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

Update README.md with full name. #924

Merged
merged 3 commits into from
Oct 4, 2019

Conversation

apachaves
Copy link
Contributor

[please review the Contribution Guidelines prior to submitting your pull request. go ahead and delete this line if you've already reviewed said guidelines.]

What does this PR do?

Where should the reviewer start?

How should this PR be tested?

Any background context you want to provide?

What are the relevant issues?

[you can link directly to issues by entering # then the number of the issue, for example, #3 links to issue 3]

Screenshots (if appropriate)

Questions:

  • Do the docs need to be updated?
  • Does this PR add new (Python) dependencies?

Update README.md to add TPOT's full name meaning.
Update README.md with full name.
@coveralls
Copy link

coveralls commented Oct 1, 2019

Coverage Status

Coverage remained the same at 95.833% when pulling 4276330 on apachaves:master into 47d7f70 on EpistasisLab:development.

@weixuanfu
Copy link
Contributor

Thanks for the PR.,

Please use the hyperlink of the paper instead of using the link directly.

The paper was also mentioned in “Citing TPOT” section below. You may use the similar format there.

Removing link to Arxiv from the beginning and keeping full name explanation.
@apachaves
Copy link
Contributor Author

@weixuanfu, I've decided to remove the link from the beginning since it is already there on the section Citing TPOT as you said. I kept just the small addition on the text in the beginning explaining the acronym. Sorry again, just want to keep it very simple for now.

@weixuanfu weixuanfu merged commit 1fcea93 into EpistasisLab:development Oct 4, 2019
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.

3 participants