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

periodic for millions of elements... #125

Closed
fischer1 opened this issue Oct 30, 2016 · 6 comments
Closed

periodic for millions of elements... #125

fischer1 opened this issue Oct 30, 2016 · 6 comments
Labels
bug

Comments

@fischer1
Copy link
Contributor

There is a genmap issue -- fix suggested by Scicong/Elia -- that is required
for millions of elements when using periodicity.
@fischer1 fischer1 added the bug label Oct 30, 2016
@stgeke
Copy link
Contributor

stgeke commented Dec 22, 2016

@fischer1 I believe one of your students fixed that. Can you ask him to submit a pull request with the fix.

@pschlatt1 pschlatt1 mentioned this issue Dec 22, 2016
Closed
@stgeke
Copy link
Contributor

stgeke commented Dec 22, 2016

genmap_issue.pdf

@pschlatt1
Copy link
Contributor

Great, thanks! I guess we can work with that now, and try to create a PR.

@emerzari
Copy link

emerzari commented Dec 22, 2016 via email

@pschlatt1
Copy link
Contributor

@emerzari: could you send it to us, then we can test it on the concrete example we have right now.

@pschlatt1
Copy link
Contributor

It seems I directly pushed the simple fix to the wrong repo, sorry.

@stgeke stgeke closed this as completed Dec 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fischer1 @pschlatt1 @emerzari @stgeke