-
Notifications
You must be signed in to change notification settings - Fork 6
Porting new visualizations from the NAU rcdata server #14
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
base: gh-pages
Are you sure you want to change the base?
Conversation
Hi, thank you for your reminder about the code issue. I'll update it to improve the visualization's clarity and readability. I've added 3 new data visualizations, but I've noticed that they will automatically commit to this PR if it's unmerged. As a result, I couldn't create separate PRs for each visualization as we previously discussed. |
It's not ideal but let's just review them all in this PR. |
there are two different rows with the same name "Constrained Pruned Dynamic Programming Algorithm" |
"Penalty Learning" name should be changed to "Max margin interval tree" |
This is still an issue, can you please fix? These are the links under the "source.link" column -- can you please change them to point to your forks of the code, under github.com/nhintruong ? |
For https://nhintruong.github.io/Central-American-temperature-maps/ there is an issue with the color of background (add theme_bw()?)/alpha transparency of geom_lines (should be darker and use color=violet, color_off=black) |
too many models plotted, is geom_segment missing showSelected? |
there are two pairs of data viz with the same name,
in general, please use the same names as the old gallery, unless you need to change one of them to make them have a unique name in the new gallery |
please do not add any more data viz to this PR. |
I updated the new data viz from the NAU rcdata server.
This is the rendered version of my fork of the gallery for your preview: https://nhintruong.github.io/gallery/
Could you please review this new gallery update? Thank you very much for your guidance