[DAMD-30] Please do not use LSST module in pfsArm.py Created: 13/May/18 Updated: 17/Aug/18 Resolved: 17/Aug/18 |
|
| Status: | Won't Fix |
| Project: | Data Model |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Story | Priority: | Normal |
| Reporter: | Kiyoto Yabe | Assignee: | price |
| Resolution: | Won't Fix | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Description |
|
ETC uses datamodel as a submodule previously linking to commit 32f4791382779375721ca802f9ca5d70f18e146d . I was trying to update the linkage to the latest one 9420aace891f41e64870a4a9890779816162dc96 , so that we can also use Python3 with the futurized datamodel. Then, one gets ModuleNotFoundError if there is no LSST modules installed. ETC should be standardized, so, if possible, I'm happy if you do not use LSST stuff in at least pfsConfig, pfsArm.py and pfsObject.py. |
| Comments |
| Comment by price [ 14/May/18 ] |
|
I'm hoping to tackle this as part of the cleanup in |
| Comment by hassan [ 17/Jul/18 ] |
|
So price: has this ticket been addressed as part of the |
| Comment by price [ 20/Jul/18 ] |
|
Just pushed a change to address this. Thanks for the reminder! |
| Comment by hassan [ 17/Aug/18 ] |
|
price is this ticket is resolved also now given that |
| Comment by price [ 17/Aug/18 ] |
|
Yes, this has been done in |