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

Ejhumphrey/ismir18 archive #26

Merged
merged 32 commits into from
Mar 27, 2019
Merged

Ejhumphrey/ismir18 archive #26

merged 32 commits into from
Mar 27, 2019

Conversation

ejhumphrey
Copy link
Collaborator

all of the changes to get ISMIR18 to Zenodo. woof.

@stefan-balke
Copy link
Collaborator

I think a lot happened here. Could you resolve the conflicts so I can give you a quick review?

@ejhumphrey
Copy link
Collaborator Author

conversation elsewhere led to my discovery that this was still pending, @stefan-balke do you still have cycles to give this a look? plznthx 🙏

@stefan-balke
Copy link
Collaborator

Alright, gave it a quick look. All changes look reasonable for the time being.
Starting to build unit-tests is a good move...thanks for this.
Realistically, we gonna have some more fun after ISMIR deadline, what you think?

@stefan-balke stefan-balke merged commit d55e238 into master Mar 27, 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.

2 participants