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

ODict instead of OrderedDict #308

Open
mkhalil8 opened this issue Jun 15, 2021 · 4 comments
Open

ODict instead of OrderedDict #308

mkhalil8 opened this issue Jun 15, 2021 · 4 comments

Comments

@mkhalil8
Copy link
Collaborator

@salvadord,

Network.py Line 234, specs.OrderedDict gave me an error
I changed it to specs.ODict and it ran smoothly.
Is that correct?

@salvadord
Copy link
Collaborator

Ah sorry, this was an old small bug, the correct code is:

from collections import OrderedDict

... 

output = json.load(fileObj, object_pairs_hook=OrderedDict)

do you mind creating a PR for this? thanks!

@mkhalil8
Copy link
Collaborator Author

mkhalil8 commented Jun 18, 2021

Hi Salvador,

I am trying to make a pull request for the changes, I created a branch from hnn2 named hnn2dev then added the changes above mentioned to a new local repository then:

$ git push origin hnn2dev
remote: Permission to jonescompneurolab/hnn.git denied to mkhalil8.
fatal: unable to access 'https://github.com/jonescompneurolab/hnn.git/': The requested URL returned error: 403

I tried to push the changes to hnn2 by:

$ git push hnn2 hnn2dev
fatal: 'hnn2' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

Do I need a permission to push the changes?

@salvadord
Copy link
Collaborator

salvadord commented Jun 22, 2021

@stephanie-r-jones @blakecaldwell @samnemo @dylansdaniels could you please give access to @mkhalil8 and @claireshort4 so they can contribute to hnn-netpyne (hnn2 branch)? They both attended our meeting today; as you know Mostafa (@mkhalil8) is working with Mohamed Sherif, and Claire (@claireshort4) is working with me. thanks!

@samnemo
Copy link
Member

samnemo commented Jun 23, 2021 via email

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

No branches or pull requests

3 participants