Skip to content

Add Barro Colorado schema #99

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Mar 12, 2020
Merged

Add Barro Colorado schema #99

merged 2 commits into from
Mar 12, 2020

Conversation

Jolg42
Copy link
Contributor

@Jolg42 Jolg42 commented Feb 18, 2020

See #98

@Jolg42 Jolg42 requested a review from divyenduz February 18, 2020 10:24
@Jolg42 Jolg42 linked an issue Feb 18, 2020 that may be closed by this pull request
Scientific project "Complete data from the Barro Colorado 50-ha plot: 423617 trees, 35 years" #98
Closed
@Jolg42 Jolg42 mentioned this pull request Feb 18, 2020
Closed
@janpio
Copy link
Contributor

janpio commented Feb 18, 2020 •
edited
Loading

This can live in mysql - _academic refers to a specific "source" (https://relational.fit.cvut.cz/). It is separate because the structure is different.

If possible, please split the data part into data.sql.

@janpio janpio added this to the Preview 22 milestone Feb 18, 2020
@Jolg42
Copy link
Contributor Author

Jolg42 commented Feb 18, 2020

@janpio It's already in mysql_academic and data is 800MB so I only added the schema :)

@Jolg42
Copy link
Contributor Author

Jolg42 commented Feb 18, 2020

Ok, got the message now 😅 - I will move it to mysql

janpio
janpio approved these changes Feb 18, 2020
View reviewed changes
Copy link
Contributor

@janpio janpio left a comment

Choose a reason for hiding this comment

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

Ok, 800MB will not really work out on Github. Emtpy schema it is then.

@divyenduz divyenduz merged commit 3d1409a into master Mar 12, 2020
@divyenduz divyenduz deleted the pr-barro-colorado branch March 12, 2020 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@janpio janpio janpio approved these changes

@divyenduz divyenduz divyenduz approved these changes

Assignees

@divyenduz divyenduz

Labels
None yet
Projects
None yet
Milestone
Preview 22
Development

Successfully merging this pull request may close these issues.

Scientific project "Complete data from the Barro Colorado 50-ha plot: 423617 trees, 35 years"
3 participants
@Jolg42 @janpio @divyenduz