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

Show "force" status in printProcessQueue #117

Open
jtniehof opened this issue Jan 26, 2022 · 0 comments
Open

Show "force" status in printProcessQueue #117

jtniehof opened this issue Jan 26, 2022 · 0 comments
Labels
enhancement New feature or feature request

Comments

@jtniehof
Copy link
Member

printProcessQueue, for every entry in the queue, shows the filename and the product:

$ printProcessQueue.py ./foo.sqlite 
/home/jtniehof/tmp/foo.sqlite
2022-01-26T17:00:14.402596
ProcessQueue
0	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130909_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
1	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130908_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
2	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130907_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
3	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130906_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
4	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130905_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
5	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130904_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
6	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130903_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look

It doesn't show the force status, i.e. the value of version_bump. It's sometimes nice to have this.

Proposed enhancement

Add either an indication of forcing, e.g.:

$ printProcessQueue.py ./foo.sqlite 
0*	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130909_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
1*	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130908_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
2	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130907_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
3	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130906_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
4	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130905_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
5	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130904_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
6	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130903_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look

Or the portion of the version which is forced:

$ printProcessQueue.py ./foo.sqlite 
0{1}	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130909_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
1{1}	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130908_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
2{0}	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130907_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
3	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130906_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
4	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130905_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
5	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130904_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look
6	rbsp-a_magnetometer_uvw_emfisis-Quick-Look_20130903_v1.3.1.cdf	rbsp-a_magnetometer_uvw_emfisis-Quick-Look

I'm not sure which makes the most sense or where it should be (start of line, after position in queue, after filename, very end of line?)

Alternatives

Do-nothing, always my favorite alternative. There are several display options in the example above. This also could be a default-off that gets turned on with an command line option.

OS, Python version, and dependency version information:

Linux-4.15.0-166-generic-x86_64-with-Ubuntu-18.04-bionic
sys.version_info(major=2, minor=7, micro=17, releaselevel='final', serial=0)
sqlalchemy=1.1.11

Version of dbprocessing

Current github master (fb95a08)

Closure condition

Figure out the desired approach, implement, document, and ideally unit test it. This would involve the first unit tests of printProcessQueue.

@jtniehof jtniehof added the enhancement New feature or feature request label Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or feature request
Projects
None yet
Development

No branches or pull requests

1 participant