added --header and --labels flags to print headers and labels from datafiles - #131
Conversation
…tafiles, respectively
|
Caution The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased. |
|
Warning Review limit reached
Next review available in: 41 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
📝 WalkthroughWalkthroughThe change adds ChangesCLI and runtime updates
Sequence Diagram(s)sequenceDiagram
participant splash
participant defaults_set
participant get_data
participant labels
splash->>defaults_set: initialize defaults with suppressed verbosity
splash->>get_data: read dump data and labels
get_data->>labels: establish column metadata
splash->>labels: print requested headers or labels
labels-->>splash: emit metadata
splash->>splash: exit
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/read_data_sphNG.f90`:
- Around line 1519-1525: Move the density/thermal-energy dependency handling out
of the pre-schema block and resolve it only after `irho`, `iutherm`, and
`irhorestframe` are finalized by the schema and label discovery flow, before
records are selectively skipped. When allocating `itempcol`, `ikappa`, and
ionisation output columns, mark each generated column as required so
partial-read processing computes requested derived outputs.
In `@src/splash.f90`:
- Around line 914-929: Update the quiet-setting logic around defaults loading
and the corresponding block near the later metadata-mode handling so `iverbose`
remains `-1` for `do_print_header`, `do_print_labels`, or `do_print_labelsorig`
even when `debugmode` is enabled. Ensure the re-applied quiet value takes
precedence over `get_data`’s debug verbosity while preserving normal debug
verbosity outside these metadata modes.
- Around line 1101-1110: Update the metadata-only handling around the header and
label checks in the main read/option flow so it exits before post-read option
processing such as set_options_dataread, print_example_quantities, or
calc_quantities. Preserve the existing print_headers, print_column_labels, and
print_column_labels_orig behavior, while ensuring --header and related metadata
modes produce no other output or calculation work.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: 9c7d1715-2046-4a50-a40b-682a6f646c23
📒 Files selected for processing (16)
src/adjust_data.f90src/allocate.f90src/defaults.f90src/get_data.f90src/interpolate3D.f90src/interpolate3D_geom.f90src/interpolate3D_proj_geom.f90src/interpolate3D_projection.f90src/kernels.f90src/labels.f90src/limits.f90src/menu.f90src/read_data_sphNG.f90src/set_options_from_dataread.f90src/splash.f90src/units.f90
1 participant
Header flag gives:
Labels flag gives:
Labelsorig flag gives:
pull request also fixes warnings with gfortran v116
Summary by CodeRabbit
New Features
Bug Fixes
Improvements