-
Notifications
You must be signed in to change notification settings - Fork 26
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
Options to select data to plot #15
base: master
Are you sure you want to change the base?
Conversation
Added parameter -w --wavelength to select data to plot. Plots all data if left empty. Multiple data rows can be selected as e.g. -w 215nm -w 280nm.
Wouldn't it be better to say -w1 -w2 -w3 etc? This way one does not need to know the wavelength that was actually used? |
I agree it would be easier if the user doesn't have to know (or remember) the wavelength in advance. Also, it should handle corner cases like the user asking for |
I think if the users don't remember the exact wavelengths, they would usually plot all wavelengths first and then see which they like. Determining the data row by the wavelength is also very precise. Therefore I would like to keep this option. I agree, the alternative option may be useful for someone, however, I see the following problems:
|
It would work like this: Judging by the systems/files that I have come across:
The reason I prefer the -w1/2/3 is that the command line does not change because of the wavelength, which is especially helpful when integrating pycorn-bin in a script. I think w1/2/3 would be sufficient, since I think there are no systems which have more than 3 wavelengths. |
I have access to two ÄKTA Purifiers, having three UV detectors, 215, 254, and 280 nm. The result files are, however, not entirely equal - the wavelengths are ordered differently: 280, 254, 215 on the first machine, 280, 215, 254 on the second one. Therefore, using for example Also, I think, by default, the script should plot everything - it might otherwise lead to confusion of new users. |
Add parameter -o --output to specify output file name (default still works)
Regarding ordering etc.. do the data series have a prefix like UV1_, UV2_ etc? Current behaviour is to plot everthing unless it is set to 0nm ("off"). |
Could you send the output file naming patch separately? |
This reverts commit 7b7d10e.
The data series from both machines have the same names - UV1_280nm, etc. The latest commit is reverted - sorry, I didn't realise that it will affect this pull request. |
Could you make a separte pull request with the output file naming only? Since that one can be merged easily. |
Yes, I'll do it as soon as possible. I'd like to first clean it up and possibly add more functionality regarding the output file names. |
Added parameter -w --wavelength to select data to plot. Plots all data if left empty. Multiple data rows can be selected as e.g. -w 215nm -w 280nm.