Sfork issu 182 #216
Sfork issu 182 #216
Conversation
Merge branch 'master' of https://github.com/numpy/numpy.org into issu_82
…rtcode strings due to hugo errors
There was a problem hiding this comment.
Thanks for the updates - lots of nice improvements!
I tried to make some suggestions to more concretely implement some of the comments I made in #182. One of the major comments that should be avoided is the concept that "scientists need visualization to comprehend the results", which appears in multiple locations in the case study. I tried to highlight and eliminate them where possible as this statement is a bit misleading.
| are ripples in the fabric of spacetime, generated by cataclysmic events in the | ||
| universe such as collision and merging of two black holes, coalescing binary | ||
| stars, supernovae or [compact | ||
| binaries](https://wwwmpa.mpa-garching.mpg.de/~hsr/researchcb-en.html). Observing |
There was a problem hiding this comment.
| are ripples in the fabric of spacetime, generated by cataclysmic events in the | |
| universe such as collision and merging of two black holes, coalescing binary | |
| stars, supernovae or [compact | |
| binaries](https://wwwmpa.mpa-garching.mpg.de/~hsr/researchcb-en.html). Observing | |
| are ripples in the fabric of spacetime, generated by phenomena such as | |
| the merger of | |
| [compact binaries](https://en.wikipedia.org/wiki/Compact_star) like | |
| black holes or neutron stars. Observing |
"phenomena" is arguably more appropriate than "cataclysmic events". Also, binary black holes are an example of compact binaries, so it's more concise to include them in that super-category.
There was a problem hiding this comment.
Also updated the link to wikipedia - the MPI link was not very detailed and (for me in the US at least) took a long time to load.
| obscure phenomena in the distant universe and its impact. Two of the most recent | ||
| events in the context of gravitaional waves are the focus of this case study: | ||
|
|
||
| * [GW observations from LIGO/Virgo | ||
| GW150914](https://en.wikipedia.org/wiki/First_observation_of_gravitational_waves): | ||
| The original discovery of gravitational waves from a binary black hole merger | ||
| * [GW observations from LiGO/Virgo | ||
| GW170817](https://en.wikipedia.org/wiki/GW170817): The first direct detection | ||
| of neutron start mergers |
There was a problem hiding this comment.
I would consider moving the introduction of these particular GW events to a spot in the text after introducing LIGO. In my opinion, it flows better if the individual observations follow the introduction of the observatory(ies) itself.
|
|
||
| The [Laser Interferometer Gravitational-Wave Observatory | ||
| (LIGO)](https://www.ligo.caltech.edu) was designed to open the field of | ||
| gravitational-wave astrophysics through the direct detection of gravitational |
There was a problem hiding this comment.
| gravitational-wave astrophysics through the direct detection of gravitational | |
| gravitational-wave astronomy through the direct detection of gravitational |
| The original discovery of gravitational waves from a binary black hole merger | ||
| * [GW observations from LiGO/Virgo | ||
| GW170817](https://en.wikipedia.org/wiki/GW170817): The first direct detection | ||
| of neutron start mergers |
There was a problem hiding this comment.
| of neutron start mergers | |
| of a neutron star merger. |
| The [Laser Interferometer Gravitational-Wave Observatory | ||
| (LIGO)](https://www.ligo.caltech.edu) was designed to open the field of | ||
| gravitational-wave astrophysics through the direct detection of gravitational | ||
| waves predicted by Einstein’s General Theory of Relativity. It comprises two |
There was a problem hiding this comment.
| waves predicted by Einstein’s General Theory of Relativity. It comprises two | |
| waves predicted by Einstein’s general theory of relativity. It comprises two |
This wasn't capitalized in the other case study. I think either is okay, but whichever convention is chosen, it should be consistent between the articles.
| * Data retrieval: Deciding which data can be analyzed, figuring out whether it | ||
| contains a signal - needle in a haystack | ||
| * Statistical analysis: estimate the statistical significance of observational | ||
| data, estimating the signal parameters (e.g. masses of stars, spin velocity, |
There was a problem hiding this comment.
These bullets seem incomplete - are they notes-to-self or intended to be included? If the latter, I think they need to be developed more, though I think the above bullet about GwPy and PyCBC adequately covers components like this.
| * Visualization of data | ||
| - Time series | ||
| - Spectrograms |
| GW detection has enabled researchers to discover entirely unexpected phenomena | ||
| while providing new insight into many of the most profound astrophysical |
There was a problem hiding this comment.
| GW detection has enabled researchers to discover entirely unexpected phenomena | |
| while providing new insight into many of the most profound astrophysical | |
| GW detection has enabled researchers to observe the universe in a brand new way, | |
| providing new insight into many of the most profound astrophysical |
| phenomena known. Number crunching and data visualization is a crucial step that | ||
| helps scientists gain insights into data gathered from the scientific | ||
| observations and understand the results. The computations are complex and cannot |
There was a problem hiding this comment.
| phenomena known. Number crunching and data visualization is a crucial step that | |
| helps scientists gain insights into data gathered from the scientific | |
| observations and understand the results. The computations are complex and cannot | |
| phenomena known. Data analysis and visualization play a crucial role in | |
| helping scientists interpret data gathered from the scientific | |
| observations and communicate the results. |
| be comprehended by humans unless it is visualized using computer simulations | ||
| that are fed with the real observed data and analysis. NumPy along with other |
There was a problem hiding this comment.
| be comprehended by humans unless it is visualized using computer simulations | |
| that are fed with the real observed data and analysis. NumPy along with other | |
| NumPy along with other |
Again, this point is not correct.
Brief description of what is fixed or changed
Deployed URL
http://numpy-<pr_number>.surge.sh/