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

ROBOT export generates error in large excel exports #1131

Closed
dlutz2 opened this issue Jul 20, 2023 · 2 comments · Fixed by #1148
Closed

ROBOT export generates error in large excel exports #1131

dlutz2 opened this issue Jul 20, 2023 · 2 comments · Fixed by #1148

Comments

@dlutz2
Copy link
Contributor

dlutz2 commented Jul 20, 2023

When trying to export an ontology with more than 64,000 classes into an Excel file, ROBOT (v1.9.4) export generated an error "The maximum number of Cell Styles was exceeded. You can define up to 64000 style in a .xlsx Workbook", which I believe is coming from the POI library. I think it is caused by the the CellStyle style = wb.createCellStyle(); call in robot.export.Row.addToWorkbook() which creates a new CellStyle for each row. The export works fine when exporting to text and html.

@jamesaoverton
Copy link
Member

Yes, it looks like we could reuse a small number of CellStyles instead of creating one for each cell. I would be happy to accept a PR for that.

@dlutz2
Copy link
Contributor Author

dlutz2 commented Jul 25, 2023

Unfortunately, the team is not yet set up for public github forks/PRs. We actually have a couple of simple additions to ROBOT we've developed in house for our own purposes e.g. we want to be able to export-edit-template lossless round trips. We would like to generate PRs for those when we get out act together.
thanks

dlutz2 added a commit to dlutz2/robot that referenced this issue Sep 9, 2023
Equivalent class fails - ontodev#1145
Self-disjoint - ontodev#1144
Export Large excel - ontodev#1131
@dlutz2 dlutz2 mentioned this issue Sep 10, 2023
5 tasks
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 a pull request may close this issue.

2 participants