Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove nrt-specific output dir code #167

Merged
merged 2 commits into from
Sep 27, 2024
Merged

Conversation

trey-stafford
Copy link
Contributor

Originally we were only going to produce daily nrt files and they were going to be a part of G02202. Now they'll ne a new version of G10016 and the directory structure for nrt files should reflect this.

Originally we were only going to produce daily nrt files and they were going to
be a part of G02202. Now they'll ne a new version of G10016 and the directory
structure for nrt files should reflect this.
@trey-stafford
Copy link
Contributor Author

Results in dirs that look like this:

/share/apps/G10016_V3/v03r00/dev/trst2284/complete
├── north
│   ├── checksums
│   │   └── daily
│   │       └── 2024
│   └── daily
│       └── 2024
└── south
    ├── checksums
    │   └── daily
    │       └── 2024
    └── daily
        └── 2024

Copy link
Contributor

@sc0tts sc0tts left a comment

Choose a reason for hiding this comment

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

Sure is simpler not to have kwarg is_nrt in so many places!

Base automatically changed from nrt-improvement-for-ops to ecdr-at-25km September 27, 2024 14:38
@trey-stafford trey-stafford merged commit 686febd into ecdr-at-25km Sep 27, 2024
1 check passed
@trey-stafford trey-stafford deleted the update-paths-for-nrt branch September 27, 2024 14:38
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.

2 participants