Skip to content

issue/591/triaxiality: Build scripts in examples/triaxiality [version:1.18.0]#603

Open
shenmingfu wants to merge 208 commits into
mainfrom
issue/591/triaxiality
Open

issue/591/triaxiality: Build scripts in examples/triaxiality [version:1.18.0]#603
shenmingfu wants to merge 208 commits into
mainfrom
issue/591/triaxiality

Conversation

@shenmingfu
Copy link
Copy Markdown
Collaborator

@shenmingfu shenmingfu commented Jul 18, 2023

@shenmingfu shenmingfu changed the title built a folder examples/triaxiality, added a readme file inside Build scripts in examples/triaxiality Jul 18, 2023
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 18, 2023

Coverage Status

coverage: 100.0%. remained the same — issue/591/triaxiality into main

@shenmingfu shenmingfu changed the title Build scripts in examples/triaxiality issue/591/triaxiality: Build scripts in examples/triaxiality Jul 18, 2023
@akumgill akumgill self-assigned this Jul 19, 2023
akumgill and others added 19 commits July 19, 2023 14:23
Comparison of Tae's and Adhikari's implementations of quadrupole and monopole components
Includes fitting for orientation of halo and ellipticity simultaneously.
Comment thread clmm/theory/parent_class.py Outdated
Comment thread examples/demo_dataops_functionality.ipynb Outdated
Copy link
Copy Markdown
Collaborator

@combet combet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My main comment on this PR is to add as much as possible the relevant reference/equation for the new triaxiality parts, either in the docstrings or in the demo notebooks. I commented on the parts I saw, where this could be useful.

@m-aguena
Copy link
Copy Markdown
Member

@combet thanks for the comments, I updated the docstrings with the equations, and added a quick compilation of the documentation page here for you to check out.

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

Successfully merging this pull request may close these issues.

9 participants