search

LSSTDESC / rail

Top level "umbrella" package for RAIL
MIT License
8 stars 4 forks source link

Add get_data instructions to pipeline running instructions #46

Closed OliviaLynn closed 1 year ago

OliviaLynn commented 1 year ago

Adds documentation to support rail_base PR #20

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (2cca1e5) 100.00% compared to head (36f4a1e) 100.00%.

:exclamation: Current head 36f4a1e differs from pull request most recent head 58b5b7d. Consider uploading reports for the commit 58b5b7d to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #46 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 1 1 Lines 1 1 ========================================= Hits 1 1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

OliviaLynn commented 1 year ago

Thanks. Maybe:

  1. Point out that ceci gets installed along with rail_base, so you can do pip install rail_base and that will get you both
  2. Clarify what “point at the root of rail” means. I think that is just the same as activativectge conda env, but I’m not sure

Word, that makes sense that ceci would be bundled in now. I'll just remove that line in that case

Fair, that could stand to be reworded - the pipeline needs to be run from the rail/ directory, or the relative paths in the pipeline won't work