Git Product home page Git Product logo

Comments (13)

loganbyers avatar loganbyers commented on May 26, 2024 5

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.

from apd-core.

SidVal avatar SidVal commented on May 26, 2024 1

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.

from apd-core.

SidVal avatar SidVal commented on May 26, 2024

It's not very clear.

Please add some example (with complete items).

from apd-core.

caesar0301 avatar caesar0301 commented on May 26, 2024

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

Here is an example: 58b3f83

from apd-core.

ishandutta2007 avatar ishandutta2007 commented on May 26, 2024

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

from apd-core.

caesar0301 avatar caesar0301 commented on May 26, 2024

@ishandutta2007 Dont worry. Let the Bullets Fly :)

from apd-core.

ishandutta2007 avatar ishandutta2007 commented on May 26, 2024

???

from apd-core.

ishandutta2007 avatar ishandutta2007 commented on May 26, 2024

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.

from apd-core.

caesar0301 avatar caesar0301 commented on May 26, 2024

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!

from apd-core.

caesar0301 avatar caesar0301 commented on May 26, 2024

@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. :)

from apd-core.

ishandutta2007 avatar ishandutta2007 commented on May 26, 2024

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.

from apd-core.

caesar0301 avatar caesar0301 commented on May 26, 2024

@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.

from apd-core.

ishandutta2007 avatar ishandutta2007 commented on May 26, 2024

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.

from apd-core.

Related Issues (12)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.