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

error for example file #78

Open
teddy106913 opened this issue Dec 21, 2022 · 5 comments
Open

error for example file #78

teddy106913 opened this issue Dec 21, 2022 · 5 comments

Comments

@teddy106913
Copy link

Hi Peter, I appreciate your providing a convenient program to make flux calculation more accessible. However, I found one problem when I try to use an example file to run the pyfluxpro. It shows the error information like ''17:01:07 ERROR Unknown sonic type (CSAT-3A) for Diag_SONIC''.
I tried to debug for this error, and I found out it's because the file didn't give the correct instrument name for Diag_SONIC. In the example file L1.txt, the variable 'Diag_SONIC' shows the instrument name is 'CSAT.' The error information disappears when I use 'CSAT3' for the instrument name.
I just tried to that you know this message. So maybe you can check this problem and renew the example file. And very much appreciate your program. It can help me to do a lot of research.
Best regard,
kuan-yin Chen

@teddy106913
Copy link
Author

teddy106913 commented Dec 21, 2022

Hi Peter, Maybe I overlooked that. Where can I find the file to describe different instruments for each variable? Because I get the new error information for different variables. Sorry for disturbing your time.
'17:23:52 ERROR Unknown IRGA type (dummy profile) for Sco2_profile'
17:23:52 ERROR No known irga types found
17:23:52 ERROR Unknown sonic type (CSAT3 and Li-7500) for Fh_EP
17:23:52 ERROR Unknown sonic type (CSAT3 and Li-7500) for Fh_EP_QC
17:23:52 ERROR 'instrument' attribute must have 2 entries separated by a comma, got CSAT3 and Li-7500 for Fco2_EP
17:23:52 ERROR 'instrument' attribute must have 2 entries separated by a comma, got CSAT3 and Li-7500 for Fco2_EP_QC
17:23:52 ERROR 'instrument' attribute must have 2 entries separated by a comma, got CSAT3 and Li-7500 for Fe_EP
17:23:52 ERROR 'instrument' attribute must have 2 entries separated by a comma, got CSAT3 and Li-7500 for Fe_EP_QC

Best regard,
Kuan-yin Chen

@pisaac-ozflux
Copy link
Collaborator

Hi teddy106913,
Apologies for the errors. I recently changed the code to be much more rigorous when checking the instrument type. I've upgraded my local copy of PFP_examples but have not yet pushed those changes to GitHub. I will do that this morning.
I haven't had time to update the Wiki (see https://github.com/OzFlux/PyFluxPro/wiki) with the new changes, hopefully today. In the meantime, you can see the currently supported instruments in the PyFluxPro/scripts/constants.py file, see the instruments dictionary.
Feel free to contact me again with any questions. I'm also happy to do a Zoom session to help you get started.
Cheers,
Peter

@pisaac-ozflux
Copy link
Collaborator

Hi teddy106913,
I have updated the PFP_examples repository (https://github.com/OzFlux/PFP_examples.git) to reflect the recent changes in the PyFluxPro code. I have tested the updated examples by running all L1 control files (using PFP V3.4.9) and no errors were generated.
Hope this helps. Feel free to contact me if you have further problems.
Cheers,
Peter

@pisaac-ozflux
Copy link
Collaborator

Hi teddy106913,
I have also added a description of the instrument variable attribute to the PyFluxPro wiki, see https://github.com/OzFlux/PyFluxPro/wiki/Variable-names-and-attributes#variable-attributes-instrument. Let me know if you think more information is required.
Cheers,
Peter

@pisaac-ozflux
Copy link
Collaborator

I've also added the wiki to the Help menu. Clicking on Help/Wiki menu in PyFluxPro will open your default browser and take you to the Home page of the PyFluxPro wiki.

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

No branches or pull requests

2 participants