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.

Sign up for GitHub

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

Jump to bottom

Some Plot Choice Quantity Names #124

Open
steven-murray opened this issue Mar 27, 2021 · 1 comment
Open

Some Plot Choice Quantity Names #124

steven-murray opened this issue Mar 27, 2021 · 1 comment
Labels
bug Something isn't working difficulty: medium Medium level difficulty to solve priority: high

Comments

@steven-murray
Copy link
Contributor

Some of the names in the plot choice drop-downs aren't great (this is my fault). IN particular:

  • Halo/Tracer Concentration Mass Relation --> Halo/Tracer Concentration
  • Remove the BoxSize (or "how_many") entry
  • Wavenumber --> "Fourier Scale, k (linear)"
  • Fourier Scale --> "Fourier Scale, k (halo model)"
  • Scale --> "Comoving Scale, r"
  • Mass Function --> "Halo Mass, m"

Furthermore, the last four above should be the first four in the X-axis drop-down. Might even make sense to put a little divider between them and the rest in the dropdown (if possible).
@steven-murray steven-murray added bug Something isn't working priority: high labels Mar 27, 2021
aneuhold added a commit that referenced this issue Apr 3, 2021
@aneuhold
T335 Fix axis names and MatJax error
91bd6c1 
Fixed an error with MathJax where it wouldn't show the plot if the
MathJax library wasn't loaded yet. Did this by editing the place where
the script is pulled in and doesn't let the page load until it
successfully retrieves it.

Addresses and almost completes #124.
aneuhold added a commit that referenced this issue Apr 3, 2021
@aneuhold
T335 Reorder x axis options
27f3cd6 
Addresses #124
@aneuhold aneuhold self-assigned this Apr 3, 2021
@aneuhold
Copy link
Contributor

aneuhold commented Apr 3, 2021

First part is solved, but the divider is a little more tricky. Might leave that for another person / next sprint. Might involve changing around how the data is formatted.

@aneuhold aneuhold removed their assignment Apr 3, 2021
@aneuhold aneuhold added the difficulty: medium Medium level difficulty to solve label Apr 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working difficulty: medium Medium level difficulty to solve priority: high
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@steven-murray @aneuhold