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.

Sign up for GitHub

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

Jump to bottom

Fix help on get_cells #448

Merged
merged 1 commit into from
Jul 20, 2022
Merged

Fix help on get_cells #448

merged 1 commit into from
Jul 20, 2022

Conversation

lfarv
Copy link
Contributor

@lfarv lfarv commented Jul 20, 2022

Corrected errors in the help for the get_cells function, concerning the output length and the examples

@lfarv lfarv requested a review from simoneliuzzo July 20, 2022 12:49
@lfarv lfarv merged commit ed22242 into master Jul 20, 2022
@lfarv lfarv deleted the fix_help_on_get_cells branch July 20, 2022 12:56
@simoneliuzzo
Copy link
Contributor

Dear @lfarv
the online help still show len(ring)
the +1 is not "converted" to the web doc.

@simoneliuzzo
Copy link
Contributor

https://atcollab.github.io/at/p/api/at.lattice.utils.html#at.lattice.utils.get_cells

@lfarv lfarv added the Python For python AT code label Sep 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simoneliuzzo simoneliuzzo simoneliuzzo approved these changes

Assignees
No one assigned
Labels
Python For python AT code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@lfarv @simoneliuzzo