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

Include the formatted citation in the default Site Report #22

Open
clnsmth opened this issue Jun 29, 2022 · 2 comments
Open

Include the formatted citation in the default Site Report #22

clnsmth opened this issue Jun 29, 2022 · 2 comments

Comments

@clnsmth
Copy link

clnsmth commented Jun 29, 2022

At the May LTER Water Cooler on EDI Reporting Tools, @seagrinch requested the default Site Report include the formatted citation rather than offering it as a separate table. Doing so would streamline use.

Currently, to combine and access this information, the user must generate two site reports (the default report and one with the formatted citation) then join these tables on the "Package ID" field.

The proposed change: https://docs.google.com/spreadsheets/d/13oMlFiQivWadzli8X8cCx47Jne3qXTwOh5sPoevL-tY/edit#gid=0

Does this look right to you @seagrinch?

Note: The current implementation posts a warning about the formatted citation option being slower. Maybe this is slow enough that the proposed change here is a no-op.

@seagrinch
Copy link

seagrinch commented Jun 29, 2022

Yes that looks good. If you wanted to provide more options, you could provide details-only, citation-only and combined, with the caveat being the 2nd and 3rd options might be slower.

The citation report for PAL shows up in ~2-3 seconds, so the overhead isn't huge for a small site scope. But neither report currently works for edi, so I'm not sure what that means. Pagination is a good way to speed up web page access, and folks are willing to wait longer for downloads. For the full report, I'm guessing the download option is preferred anyway.

@clnsmth
Copy link
Author

clnsmth commented Jun 30, 2022

Thanks @seagrinch. We'll take these comments into consideration.

@servilla servilla added the EDI label Oct 31, 2023
@servilla servilla removed the EDI label Oct 10, 2024
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

3 participants