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

Update to UIMA 3 and DKpro Core to Version 2.1.0 #45

Open
5 of 7 tasks
pagelj opened this issue May 2, 2018 · 13 comments
Open
5 of 7 tasks

Update to UIMA 3 and DKpro Core to Version 2.1.0 #45

pagelj opened this issue May 2, 2018 · 13 comments

Comments

@pagelj
Copy link
Member

pagelj commented May 2, 2018

Several issues occurred:

  • PARAM_OVERWRITE = true needs to be added to all calls of XmiWriter
  • Import of the POS libraries needs to changed (e.g. POS_NOUN instead of NN)
  • stanfordnlp API is now called corenlp
  • Unresolved issue with DocumentMetaData
  • Some libraries are now under org.dkpo. instead of de.tudarmstadt.ukp.dkpro. (but not all!)
  • commons-lang needs to be updated to commons-lang3
  • dkpro functions that return annotations now return List instead of Collection
@pagelj pagelj added this to the 1.1.0 milestone May 2, 2018
@pagelj pagelj removed this from the 1.1.0 milestone Jul 3, 2018
@pagelj
Copy link
Member Author

pagelj commented Feb 8, 2019

For the DocumentMetaData issue, also see https://groups.google.com/forum/#!msg/dkpro-core-user/-DtO5Ivnk9I/o5ZpB9v7BwAJ and https://issues.apache.org/jira/browse/UIMA-5054
Unfortunately, UIMA Java SDK >=2.8.1 does not solve the problem for us. Either it did not actually get fixed or our issue is a different one

@pagelj
Copy link
Member Author

pagelj commented Feb 8, 2019

Postponing this issue for now

@nilsreiter
Copy link
Member

dkpro is now at 1.10.0 anyway ...

@pagelj
Copy link
Member Author

pagelj commented Feb 23, 2019

true, I also tried to update to 1.10.0, but the issue still persists

@pagelj pagelj added this to the 2.1.0 milestone Jun 7, 2019
@pagelj pagelj changed the title Update DKpro Core to Version 1.9.1 Update DKpro Core to Version 1.10.0 Jun 7, 2019
@pagelj pagelj removed this from the 2.1.0 milestone Jun 7, 2019
@nilsreiter
Copy link
Member

nilsreiter commented Apr 24, 2020

It’s 1.12.0 by now 😉

Updating to UIMA 3 would also be entailed by this, and that probably has other benefits.

Https://github.com/nilsreiter/CorefAnnotator/issues/300 would also benefit from that.

@pagelj
Copy link
Member Author

pagelj commented Apr 24, 2020

Ok, I will give it another try

@pagelj pagelj added this to the 3.0.0 milestone Apr 24, 2020
@pagelj pagelj changed the title Update DKpro Core to Version 1.10.0 Update to UIMA 3 and DKpro Core to Version 1.12.0 Apr 24, 2020
@pagelj
Copy link
Member Author

pagelj commented Apr 24, 2020

Updating to UIMA 3 will probably also require nilsreiter/generic-xml-reader#12 to be solved first

@pagelj pagelj changed the title Update to UIMA 3 and DKpro Core to Version 1.12.0 Update to UIMA 3 and DKpro Core to Version 2.1.0 Aug 20, 2020
@nilsreiter
Copy link
Member

Updating to UIMA 3 will probably also require nilsreiter/generic-xml-reader#12 to be solved first

I just noticed that this was long done, but I didn't close the issue.

@pagelj
Copy link
Member Author

pagelj commented May 23, 2021

Ah yes, thanks! I already tried out the updated generic-xml-reader, it is working fine

@pagelj
Copy link
Member Author

pagelj commented May 23, 2021

@nilsreiter But actually, https://github.com/nilsreiter/uima-util is also an issue, it also needs to be updated to UIMA 3, otherwise DramaNLP will not compile

@nilsreiter
Copy link
Member

nilsreiter commented May 23, 2021

Damn, too many dependencies ...

What exactly is it that we from uima-util? Some of its functions are now also available in uima3, I noticed.

Edit: I'll try to make a quick update of uima-util, now that I'm at it.

@pagelj
Copy link
Member Author

pagelj commented May 23, 2021

Right now, it is using de.unistuttgart.ims.uimautil.AnnotationUtil (for trimming I think), de.unistuttgart.ims.uimautil.RemoveDuplicateAnnotations and de.unistuttgart.ims.uimautil.SetCollectionId (which are relevant) and

de.unistuttgart.ims.uimautil.AnnotationComparator, de.unistuttgart.ims.uimautil.ClearAnnotation, de.unistuttgart.ims.uimautil.ContextWindowAnnotator, de.unistuttgart.ims.uimautil.MapAnnotations (which are probably not super relevant)

@nilsreiter
Copy link
Member

nilsreiter commented May 23, 2021

Thanks!
uima-util now exists in version 2.0.0, which uses UIMA 3.2.0.
Should be in maven central soonish.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants