We use cookies to ensure you get the best experience on our website
Home Explore Help News
Register Sign In
NeuralEnsemble
/
ephy_testing_data
9
6
Fork 15
Files Issues 10 Pull Requests 4 Wiki
Labels Milestones
New Pull Request

#39 add openephys binary files

Merged
sprenger merged 4 commits from NeuralEnsemble/openephysbinary into NeuralEnsemble/master 3 years ago
Conversation 4 Commits 4 Files changed 88
Samuel Garcia commented 3 years ago
There is no content yet.
Samuel Garcia commented 3 years ago
Owner

Hi @sprenger. have a look to this and merge if it like like ok.

Hi @sprenger. have a look to this and merge if it like like ok.
sprenger commented 3 years ago
Owner

Hi @samuelgarcia, thanks for the PR! Looks like a lot of files, but good. Only one small change is needed: Could you rename your 'readme.txt' to 'README.txt' for it not to be captured by annex, but by git.

Hi @samuelgarcia, thanks for the PR! Looks like a lot of files, but good. Only one small change is needed: Could you rename your 'readme.txt' to 'README.txt' for it not to be captured by annex, but by git.
sprenger commented 3 years ago
Owner

Is there a reason to have the test_multiple_2020-12-17_17-34-35 twice nested? If not it I would remove one level here. `

Is there a reason to have the `test_multiple_2020-12-17_17-34-35` twice nested? If not it I would remove one level here. `
sprenger commented 3 years ago
Owner

Hi @samuelgarcia, it looks like you uploaded the files in an unlocked state. Are you sure you ran gin lock * before the upload?

Hi @samuelgarcia, it looks like you uploaded the files in an unlocked state. Are you sure you ran `gin lock *` before the upload?
sprenger closed 3 years ago
This pull request has been merged successfully!
Sign in to join this conversation.
Labels
Clear labels
No Label
Milestone
Clear milestone
No Milestone
Assignee
Clear assignee
No assignee
2 Participants
Write Preview
Loading...
Cancel
Save
There is no content yet.