-
Notifications
You must be signed in to change notification settings - Fork 159
Python packages - qceff table pretty printer #911
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
Open
hkershaw-brown
wants to merge
5
commits into
main
Choose a base branch
from
python-packages
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Note I am not sure on the gamma & beta closed vs open [ vs ( have to check the .f90 code. Currently printing [ for closed. |
mjs2369
reviewed
Jul 9, 2025
mjs2369
reviewed
Jul 9, 2025
hkershaw-brown
commented
Jul 11, 2025
|
||
import csv | ||
import sys | ||
from typing import Dict, List, Any |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
python 3.8, could switch to 3.9+ but is python 3.8 still knocking about on some clusters.
Install packages in pytools with pip install -r pyDART.txt test for pyqceff only testing if QTY name is in the output, not enough.
Co-authored-by: Marlena Smith <[email protected]>
parser as separate function
avoids accidentally installing a pyqceff from the python path
db661f7
to
31d675a
Compare
@hkershaw-brown chat to @mjs2369 about namespace package |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
The qceff table has a lot of info, which is hard to read.
This python package provides
display-qceff-table
which pretty prints the qceff table for qceff.csv files.It is in a python package so people can install with:
pip install -e DART/pytools/pyqceff
and use
display-qceff-table
without the full path.Also allows packages to be put individually on pypi
There is also pyDART.txt which is list (one package in it currently) to install DART/pytools
pip install -r pyDART.txt
Example of pretty-print:

Usage:
Fixes issue
No issue
Types of changes
Documentation changes needed?
No docs, draft pull request to see what people think.
Tests
Please describe any tests you ran to verify your changes.
There is a test directory, but this test is only checking the QTYs are in the printed output.
cd DART/pytools
pytest
Checklist for merging
Checklist for release
Testing Datasets