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

[BBPBGLIB-1135] Move CompartmentMapping to python #133

Merged
merged 1 commit into from
Feb 28, 2024

Conversation

jorblancoa
Copy link
Collaborator

@jorblancoa jorblancoa commented Feb 27, 2024

Context

Addresses BBPBGLIB-1135
Move the functionality of the CompartmentMapping.hoc to python and remove the mapping.py auxiliar file

Review

  • PR description is complete
  • Coding style (imports, function length, New functions, classes or files) are good
  • Unit/Scientific test added
  • Updated Readme, in-code, developer documentation

@jorblancoa jorblancoa requested review from ferdonline and WeinaJi and removed request for ferdonline February 27, 2024 14:48
@bbpbuildbot

This comment has been minimized.

@jorblancoa jorblancoa marked this pull request as draft February 27, 2024 15:12
@jorblancoa jorblancoa force-pushed the jblanco/remove_compartment_mapping branch from 30ecad4 to 345f880 Compare February 27, 2024 15:20
@bbpbuildbot

This comment has been minimized.

@jorblancoa jorblancoa force-pushed the jblanco/remove_compartment_mapping branch from 345f880 to 6ebe624 Compare February 27, 2024 16:06
@bbpbuildbot

This comment has been minimized.

@jorblancoa jorblancoa marked this pull request as ready for review February 27, 2024 17:36
@jorblancoa jorblancoa force-pushed the jblanco/remove_compartment_mapping branch from 6ebe624 to 155f00f Compare February 27, 2024 17:36
@bbpbuildbot

This comment has been minimized.

@jorblancoa jorblancoa force-pushed the jblanco/remove_compartment_mapping branch from cdc4cc3 to 1ac4668 Compare February 28, 2024 12:49
@bbpbuildbot

This comment has been minimized.

@bbpbuildbot
Copy link

@jorblancoa jorblancoa merged commit b5c1605 into main Feb 28, 2024
4 checks passed
@jorblancoa jorblancoa deleted the jblanco/remove_compartment_mapping branch February 28, 2024 17:01
WeinaJi pushed a commit that referenced this pull request Oct 14, 2024
## Context
Addresses BBPBGLIB-1135
Move the functionality of the CompartmentMapping.hoc to python and
remove the mapping.py auxiliar file

## Review
* [x] PR description is complete
* [x] Coding style (imports, function length, New functions, classes or
files) are good
* [ ] Unit/Scientific test added
* [ ] Updated Readme, in-code, developer documentation
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.

3 participants