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

Code Cleanup (inner function related) #33

Open
4 of 8 tasks
benjaminleroy opened this issue Mar 8, 2018 · 4 comments
Open
4 of 8 tasks

Code Cleanup (inner function related) #33

benjaminleroy opened this issue Mar 8, 2018 · 4 comments
Assignees

Comments

@benjaminleroy
Copy link
Collaborator

benjaminleroy commented Mar 8, 2018

We need to standardize the following:

  • 1. change all lon variables to long (as of 6 September 2018)
  • 2. make the input parameters lon = 1, lat = 2 into the common c_position = 1:2 (just make a decision on which should be done) Ruling: now use position
  • 3. change the names of things from credible to prediction
  • 4. Change bubble_CI notation to spherical
  • 5. correct spherical and delta ball ordering of lat and lon (incorrect)
    • 5.1 spherical functions: bubble_ci_from_tclist -> ggvis_bubble_data
    • 5.2 delta ball functions: delta_ball_wrapper -> ggvis_delta_ball_contour
  • 6. For PBs should use parameter "alpha" not alpha_level to express the associated alpha level (to make 1-alpha PBs). Also alpha_ci_level should then be "alpha_ci".
@benjaminleroy
Copy link
Collaborator Author

@Mr8ND was this done?

@Mr8ND
Copy link
Owner

Mr8ND commented Jun 24, 2018

No I do not believe it was.

@benjaminleroy
Copy link
Collaborator Author

@RobinMDunn, not sure you've seen this list - but just figured I'd make sure you were aware

@RobinMDunn
Copy link
Collaborator

Got it, thanks!

@benjaminleroy benjaminleroy changed the title Code Cleanup Code Cleanup (inner function related) Aug 26, 2018
@benjaminleroy benjaminleroy self-assigned this Aug 26, 2018
@Mr8ND Mr8ND mentioned this issue Sep 3, 2018
5 tasks
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