[INSTRM-1251] Remove __init__.py files to support PEP-420 Implicit Namespaces Created: 22/Apr/21 Updated: 14/Jul/21 |
|
| Status: | Open |
| Project: | Instrument control development |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Normal |
| Reporter: | cloomis | Assignee: | Unassigned |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
Some ICS python directory trees are "correct": they have python/ics/nameActor/ directories and not just python/nameActor directories. Better design, but if a module imports two of those the second import will fail if the python/ics directories have _init_.py files. According to PEP-420, the recommended mechanism is not to have _init_.py files in the directories with the shared name (the per-project python/ics/ directories). This showed up in ics_fpsActor, which has import ics.cobraCoach and import ics.fpsActor lines in one file. |
| Comments |
| Comment by cloomis [ 14/Jul/21 ] |
|
Doing this for ics_mcsActor, ics_cobraCoach, and ics_fpsActor now. |