search

necst-telescope / necst

:gem: NEw Control System for Telescope
https://necst-telescope.github.io/necst/
MIT License
3 stars 0 forks source link

Implement the optical pointing #231

Closed Kirby1717 closed 9 months ago

Kirby1717 commented 10 months ago

Checks

Related Issue

This PR closes #222.

TatsumiISHIKAWA commented 10 months ago

@Kirby1717 よく考えたら光学ポインティングのneclibコードはまたリリースされていないのでテストに反映できないですね。 エラーを吐くのも納得です。neclibをリリースしたらこちらも上手くと思われるのでそれまでこっちは保留します。

Kirby1717 commented 10 months ago

ですよね、丁度全く同じこと思いました笑 neclib が更新されたら改めてレビューお願いします。

TatsumiISHIKAWA commented 10 months ago

@Kirby1717 https://github.com/necst-telescope/necst/blob/432d6e08a709bdc077593e895ede5f1f85255f90/Dockerfile#L14-L19

Direct modeが反映されたneclibバージョンがリリースされました。 DockerFileのneclibバージョンを0.24.11に変更してテストを再度回してください。

 RUN ( cd $ROS2_WS/src/necst && pip install "neclib==0.24.11" && pip install ipython )
TatsumiISHIKAWA commented 10 months ago

@Kirby1717 変更とテストありがとうございます。 テストが終わらないという問題がありますが、これはDirect Modeのテストの実装の不具合によるもののようです。 現在別のPR ( https://github.com/necst-telescope/necst/pull/230 ) でテストコードを修正することで対処中のようなのでそちらが終わり、Mergeされ次第mainからOpitical PointingのブランチにMergeしたあと、再度テストを回し直しましょう。