Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move initialize_files directory #662

Merged
merged 17 commits into from
Aug 9, 2024

Conversation

200km
Copy link
Member

@200km 200km commented Jul 20, 2024

Related issues

#613

Description

The initialize_files directory was moved to the root directory and renamed as settings.

Test results

See CI

Impact

NA

Supplementary information

NA

@200km 200km added priority::medium priority medium major update incompatible API changes labels Jul 20, 2024
@200km 200km added this to the Major Update v8.0.0 milestone Jul 20, 2024
@200km 200km self-assigned this Jul 20, 2024
@200km 200km requested review from sksat and a team as code owners July 20, 2024 09:22
@200km 200km requested review from ogoogo, Hiro-0110, seki-hiro, suzuki-toshihir0, t-hosonuma and conjikidow and removed request for a team July 20, 2024 09:22
@200km
Copy link
Member Author

200km commented Jul 21, 2024

CI落ちてよくわからなかったので色々試していたが、現状次のような感じ。

  • AGIからSpaceWeather関連ファイルを落としてくるところで転けている
    • Win64bit以外は全てこれ
    • 多分元のFTPサーバーが落ちているからどうしようもない
  • S2E実行時にCSPICE関連ファイルが読めていない
    • Win64bit C2A OFFのみこちら
    • Win64bitではextlibのキャッシュが働いてSpaceWeather関連ファイルのDLが動かないので、ここまで進んでいる
    • これはS2E側の問題なので、修正する必要がある

@200km
Copy link
Member Author

200km commented Jul 21, 2024

GoogleTestのフローは何も変えていないのに通るようになった。
他のものもAGIからSpaceWeather関連ファイルを落としてくる部分はクリアした。

多分、FTPサーバーが復旧したのだろう。

実行時のCSPICEファイルエラーの原因がまだわからないが、working directoryの変更などでパスが正しく指定できていないからだと思われる。

Base automatically changed from feature/move-logs-directory to feature/modify-settings August 8, 2024 22:25
@200km 200km mentioned this pull request Aug 8, 2024
@200km 200km merged commit fd45f1a into feature/modify-settings Aug 9, 2024
11 checks passed
@200km 200km deleted the feature/move-initialize-file-directory branch August 9, 2024 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major update incompatible API changes priority::medium priority medium
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants