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

2to3 #103

Open
wants to merge 22 commits into
base: ow-0.8
Choose a base branch
from
Open

2to3 #103

wants to merge 22 commits into from

Conversation

slarson
Copy link
Member

@slarson slarson commented Feb 5, 2018

Upgrades c302 to work with Python 3.6.

lungd and others added 22 commits August 23, 2017 12:03
E.g. data_reader, config_package, input_list, conns_to_include
So the optimizer can also add new parameters instead of only changing the magnitude. Therefore the unit needs to be known.
parameters['AVBR_to_DB1_elec_syn_gbase'] = {'value':'0.1', 'unit':'nS'}
self.volts['MDL01/0/GenericMuscleCell/v'] = [...]
Check if type of param is dict or list (new or old version).
Use multiprocessing.pool instead of parallelpython
This reverts commit 87bf234.
 Conflicts:
	CElegans/pythonScripts/c302/tune/C302Simulation.py
Only override/add param by a regex unless there is a param defined without a regex.
new models, other changes
@pgleeson
Copy link
Member

I've incorporated these updates into here: https://github.com/openworm/CElegansNeuroML/tree/ow-0.8_py3 & made more changes. Basic examples work for me in python3...

@lungd can you test these before I merge into ow-0.8?

@pgleeson
Copy link
Member

The branch https://github.com/openworm/CElegansNeuroML/tree/ow-0.8_py3 seems to be working now that #105 is resolved.

@slarson I'd suggest closing this PR and testing with the ow-0.8_py3 branch.

@slarson
Copy link
Member Author

slarson commented Feb 17, 2018

@pgleeson OK; where should the ow-0.8_py3 branch be merged if it works? To master?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants