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

CALL FOR NEW DATA #1

Open
caesar0301 opened this issue Jan 15, 2018 · 13 comments
Open

CALL FOR NEW DATA #1

caesar0301 opened this issue Jan 15, 2018 · 13 comments

Comments

@caesar0301
Copy link
Member

caesar0301 commented Jan 15, 2018

Please follow the instruction to join the awesome public data community,

https://github.com/awesomedata/apd-core/blob/master/CONTRIBUTING.md

@SidVal
Copy link
Contributor

SidVal commented Feb 22, 2018

It's not very clear.

Please add some example (with complete items).

@caesar0301
Copy link
Member Author

Hi @SidVal, a quick note: Fork apd-core -> Add new data file -> Give a merge request.

Here is an example: 58b3f83

@SidVal
Copy link
Contributor

SidVal commented Feb 23, 2018

You did not understand me.

I mean that the document: https://github.com/awesomedata/apd-core/wiki/HOW_TO_CONTRIBUTE-%E5%A6%82%E4%BD%95%E8%B4%A1%E7%8C%AE not It is very clear, because the example you provide does not explain how each point is completed and what impact it has on the final file that the bot creates.

You could edit the Wiki better to add more documentation and that way more people would collaborate.

@loganbyers
Copy link
Contributor

From having just added a dataset to APD, I can say there seems to be only four variables in the YAML that currently transfer to the APD readme:

  • title -- used as the visible text
  • homepage -- used as the link/href
  • category -- used to place the dataset under the appropriate heading
  • description -- appended to the title in the visible text

The YAML -> Markdown conversion looks like:

[{title}]({homepage}) - {description}

with description truncated as needed.

swmpkim pushed a commit to swmpkim/apd-core that referenced this issue May 1, 2018
changed some things after seeing this comment:
awesomedata#1 (comment)
@ishandutta2007
Copy link
Contributor

Why are so many pull requests kept in pending state inspite of passing travis ci build tests?

caesar0301 pushed a commit that referenced this issue Dec 3, 2018
@caesar0301
Copy link
Member Author

@ishandutta2007 Dont worry. Let the Bullets Fly :)

@ishandutta2007
Copy link
Contributor

???

@ishandutta2007
Copy link
Contributor

ishandutta2007 commented Dec 3, 2018

It seems now you have merged some. I think it's better if you add a short comment mentioning the reason for the rest, doing so will help new contributors to be on the same page.

@caesar0301
Copy link
Member Author

I explained why I gave labels to some PRs in #67 (comment). I really want this to be a pure high-quality data sharing repo. No advertisement! No spam! No reputation promotion! Maybe, sometimes actually, I gave a wrong judgement about the quality of a request. I want you some people, jump out, and point out that I gave a wrong decision. I will fix my label when I am convinced and give each request a responsible review. Thank you all contributors!

@caesar0301
Copy link
Member Author

caesar0301 commented Dec 3, 2018

@ishandutta2007 "Let the bullets fly", a modern Chinese film name. We sometime use this phrase to tell people to take it easy. The metaphor means the bullet you fired out will achieved its target, eventually. If not, its justing flying its way. :)

@ishandutta2007
Copy link
Contributor

My point is, if it fails your criteria it should fail the test right a way. is it too hard to automate it ?
The amount of time that you might have spent manually reviewing might be more than what writing a quick test to handle that scenario.

@caesar0301
Copy link
Member Author

@ishandutta2007 Yes I know. Am trying to automate validation as best as I can. But to my programming experience, some criteria are hard to implemented as code, say fake submission for ads.

@ishandutta2007
Copy link
Contributor

ishandutta2007 commented Dec 11, 2018

Create a pending validation list, and share it, let the open souce community help you on those too, I will see if I can take up any.
Let's keep that list on issues itself with "feature request" label and we will close it ony when all from that list gets done. If new validation comes up, we will just add to it and keep the issue open.

ppival pushed a commit that referenced this issue Feb 20, 2019
Update American-Ninja-Warrior-Obstacles.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants