numpy / numpy.org

The NumPy home page
http://numpy.org/
BSD 3-Clause "New" or "Revised" License
109 stars 103 forks source link

Review of GWO Case Study #182

Closed rossbar closed 4 years ago

rossbar commented 4 years ago

A few comments on the LIGO/GWO case study (related to review of EHT case study in #175)

Again, my overall take from a quick readthrough is very positive - it's a very interesting article that, in my opinion, does a great job highlighting a really compelling example!

Some thoughts on a few things that stood out to me:

Note that LIGO has made quite a few interesting observations during the first couple measurement campaigns - I would re-word this sentence to focus specifically on GW150914, which was significant in being the first observation both confirming the existence of gravitational waves, and the ability to directly measure them.

I'm not sure about the phrasing of "objects and phenomena that are made from warped spacetime". One alternative would be to re-word this sentence to emphasize that GWO allow scientists to observe and study the universe in a brand new way, ushering in the era of gravitational wave astronomy

Also - a dependency map for pycbc and/or gwpy like the one for eht-imaging would be a nice addition!

shaloo commented 4 years ago

@rossbar , thank you for these excellent suggestions. I have incorporated all other than the last one about dependency map. Creating a new git issue ticket (see #217) and updating the scripts to take care of gwpy, pycbc and others (in future) that might need a similar graph.

shaloo commented 4 years ago

@rossbar - do these look ok? - planning to add these in the case study. numpy-clean-color

rossbar commented 4 years ago

@rossbar - do these look ok? - planning to add these in the case study.

I only see the dependency graph for gwpy, but it does indeed look very nice! Your script for generating these visualizations of the dependencies is great! :+1:

shaloo commented 4 years ago

Here is the one for PyCBC... Needed to mend the script to take care of PyCBC requirements file intricacies.

pycbc-numpy-clean-color

rgommers commented 4 years ago

@shaloo I like the gwpy one. for the PyCBC one I think it would be good to exclude six and the nodes that are only connected to six. This is what you did EHT as well.