Skip to content
This repository has been archived by the owner on Nov 4, 2019. It is now read-only.

Performance issues #9

Open
jcklie opened this issue Sep 14, 2018 · 0 comments
Open

Performance issues #9

jcklie opened this issue Sep 14, 2018 · 0 comments
Assignees
@jcklie
Labels
⚙️Refactoring
Milestone
0.1.1

Comments

@jcklie
Copy link
Contributor

jcklie commented Sep 14, 2018

Loading large CAS XMI, e.g. germeval 2014 de, takes up to 50 seconds. This should be much quicker.
@jcklie jcklie added 🐛Bug Something isn't working 🆕Enhancement New feature or request labels Sep 14, 2018
@jcklie jcklie added this to the 0.1.1 milestone Sep 14, 2018
@jcklie jcklie self-assigned this Sep 14, 2018
jcklie added a commit that referenced this issue Sep 14, 2018
@jcklie
#9 - Performance issues
c47bb5e 
- Use dictionary instead of list for feature loopup
- Use a set instead of a list for ids
@reckart reckart added ⚙️Refactoring and removed 🐛Bug Something isn't working 🆕Enhancement New feature or request labels Sep 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jcklie jcklie

Labels
⚙️Refactoring
Projects
None yet
Milestone
0.1.1
Development

No branches or pull requests
2 participants
@jcklie @reckart