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

Add WMO and IGRA #56

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add WMO and IGRA #56

wants to merge 2 commits into from

Conversation

wolflegend99
Copy link

@wolflegend99 wolflegend99 commented Oct 29, 2019

Added the prospects of research of WMO and IGRA to the list of Research under Climate Change

Checklist

  • I reviewed 2 other open pull requests. Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration.
  • I read and followed the CONTRIBUTING guidelines.
  • This pull request has a title in the format Add Company Name. Or Add Company1, Company2 and Company3 if it adds multiple companies.
    • ❌ Update README.md
    • ❌ Added Orbital Systems
    • ✅ Add Orbital Systems
    • ✅ Add Orbital Systems and Bulb
  • I added entries in alphabetical order.
  • The description is simple and doesn't contain unnecessary buzzwords. It needs to say what the company does, it's not a sales pitch.
    • ❌ Disruptive company developing cutting-edge AI-enabled solutions to provide farmers with a decision support system.
    • ✅ They use satellites to help farmers monitor crops and lower their emissions by reducing their nitrogen usage.

Added the prospects of research of WMO and IGRA to the list of Research under Climate Change
Added the research prospects of WMO and IGRA to the Research section of the README.md
@wolflegend99
Copy link
Author

I have added the corresponding organization names as followed as on the guidelines page

@wolflegend99
Copy link
Author

I checked the 'sort in alphabetical order'.

@wolflegend99 wolflegend99 reopened this Oct 30, 2019
@wolflegend99 wolflegend99 changed the title Update README.md Add WMO and IGRA Oct 30, 2019
@wolflegend99
Copy link
Author

Changed the title :)

@@ -241,10 +241,12 @@ Provides solutions for precision farming, enabling farmers to monitor their crop
## Research

- [Environmental Protection Agency](https://www.epa.gov/climate-research) - Research on effects of climate change and solutions.
- [Integrated Global Radiosonde Archive](https://www.ncdc.noaa.gov/data-access/weather-balloon/integrated-global-radiosonde-archive) - IGRA is useful as input to air pollution models, for studies of the detailed vertical structure of the troposphere and lower stratosphere, for assessing the atmospheric conditions during a particular meteorological event, and for many other analyses and operational applications.
Copy link
Contributor

Choose a reason for hiding this comment

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

I would put the acronym (IGRA), on the title and then omit it in the description, i.e Integrated Global Radiosonde Archive (IGRA) ... and then the description: Aims to be an input to air pollution models, for ... and also add the acronym to WMO's title.

Also, could you please squash the commits together, you made two commits that are the same. Also, I would change the commit name to be more descriptive than Update README, e.g Add WMO and IGRA.

@aalises aalises mentioned this pull request Dec 23, 2019
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

Successfully merging this pull request may close these issues.

2 participants