RMS icon indicating copy to clipboard operation
RMS copied to clipboard

Define .config Path Relative to RMS Root

Open Cybis320 opened this issue 1 year ago • 8 comments

Currently, RMS can only function when the working directory is the RMS root directory because the .config path is defined relative to the working directory in the .pyxbld files. This is inconvenient when, for instance, we want to use RMS functions in other projects. This PR defines the `.config' path relative to the RMS root directory regardless of the actual working directory path.

Cybis320 avatar Sep 20 '24 05:09 Cybis320

it seems so. In multi-station configuration, the .config path is passed as parameter -c, which most of RMS functions understand.

satmonkey avatar Sep 23 '24 15:09 satmonkey

it seems so. In multi-station configuration, the .config path is passed as parameter -c, which most of RMS functions understand.

True. And I guess the pyxbld stuff already reads from the default config in these situations so at least this will ensure it is consistent.

markmac99 avatar Sep 23 '24 16:09 markmac99

No. Actually, in multicam the pyxbld files are still grabbing the config file from the working directory not the station’s.

On Sep 23, 2024, at 8:10 AM, Mark McIntyre @.***> wrote: @markmac99 commented on this pull request.

Is this going to conflict wth the multicamera build, which locates the config file in a camera-specific folder?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Cybis320 avatar Sep 23 '24 16:09 Cybis320

To clarify, the PR does not affect how multicam or single cam operate.

Currently, both single and multicam setups use the built parameters specified in the config file located in the working directory, no matter what other config file might be specified at launch.

The PR only change is that the built parameters will be coming from the config file in the RMS root directory not the working directory.

Cybis320 avatar Sep 23 '24 17:09 Cybis320

Then, there is no problem with the single user multicam. The current workflow starts with creating the single camera at standard location, and then cloning the .config for multicam. The build info remains untouched.

Dne po 23. 9. 2024 18:20 uživatel Luc Busquin @.***> napsal:

To clarify, the PR does not affect how multicam or single cam operate.

Currently, both single and multicam setups use the built parameters specified in the config file located in the working directory, no matter what other config file might be specified at launch.

The PR only change is that the built parameters will be coming from the config file in the RMS root directory not the working directory.

— Reply to this email directly, view it on GitHub https://github.com/CroatianMeteorNetwork/RMS/pull/424#issuecomment-2368901641, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABIU5B2YINCPMVA6TU57KUDZYBEUPAVCNFSM6AAAAABORJEI3KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRYHEYDCNRUGE . You are receiving this because you commented.Message ID: @.***>

satmonkey avatar Sep 24 '24 00:09 satmonkey

Without the PR, the pyxbld files use a hardcoded ./.config. It causes an error If any RMS module is accessed when the working directory does not contain a .config file.

The use case for the PR is when trying to work with RMS modules outside our standard environment. The PR has no impact on current single or multicam operations.

Cybis320 avatar Sep 24 '24 01:09 Cybis320

Note that currently the build parameters in the station's config files on multicam are ignored. Only the working directory config build params are used. This is fine as it would be nonsensical to have different build parameters for different cameras. This PR would simply point to the RMS root config for the build params. Currently the same thing is achieved as we are forced to manually set the working directory to the RMS root directory.

Cybis320 avatar Sep 24 '24 01:09 Cybis320

All makes sense. Thanks.

markmac99 avatar Sep 24 '24 08:09 markmac99