Skip to content

Conversation

@eacharles
Copy link
Collaborator

No description provided.

@eacharles eacharles requested a review from sschmidt23 May 11, 2023 22:20
@eacharles
Copy link
Collaborator Author

Main changes:

  1. Added src/rail/core/introspection.py with a class called RailEnv that keeps track of packages and namespaces that are available in the rail ecosystem in the current python environment
  2. Added src/rail/stages packages that uses RailEnv to dynamically populate itself with available RailStage objects
  3. Added unit tests for same.

@codecov
Copy link

codecov bot commented May 11, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (142a156) 100.00% compared to head (c858484) 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##              main      #381    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           38        40     +2     
  Lines         2633      2860   +227     
==========================================
+ Hits          2633      2860   +227     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...eation/degradation/observing_condition_degrader.py 100.00% <ø> (ø)
src/rail/core/__init__.py 100.00% <100.00%> (ø)
src/rail/core/data.py 100.00% <100.00%> (ø)
src/rail/core/introspection.py 100.00% <100.00%> (ø)
src/rail/stages/__init__.py 100.00% <100.00%> (ø)

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

Copy link
Collaborator

@sschmidt23 sschmidt23 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, seems to function as intended, thanks.

@eacharles eacharles merged commit 09739f6 into main May 12, 2023
@eacharles eacharles deleted the issue/374/introspection branch May 12, 2023 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants