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

Is jdk7 -y needed? #159

Closed
3 tasks
HomeStayMom opened this issue Feb 28, 2023 · 6 comments
Closed
3 tasks

Is jdk7 -y needed? #159

HomeStayMom opened this issue Feb 28, 2023 · 6 comments
Assignees

Comments

@HomeStayMom
Copy link

Hi, I wonder if there is specific need to utilize jdk 7 considering I have Java 19 STS from Azul installed.
https://www.azul.com/downloads-new/?os=windows&package=jdk#zulu

I would be happy to learn if this could work else the proposed method as
choco install jdk -y results in the following error.
Looking forward hearing from you

C:\Windows\System32>choco install jdk7 -y
Chocolatey v1.3.0
Installing the following packages:
jdk7
By installing, you accept licenses for the packages.
jdk7 not installed. The package was not found with the source(s) listed.
Source(s): 'https://community.chocolatey.org/api/v2/'
NOTE: When you specify explicit sources, it overrides default sources.
If the package version is a prerelease and you didn't specify --pre,
the package may not be found.
Please see https://docs.chocolatey.org/en-us/troubleshooting for more
assistance.

Chocolatey installed 0/1 packages. 1 packages failed.
See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Failures

C:\Windows\System32>

  • a possible bug
  • a question about package functionality
  • a suggested code or documentation change, improvement to the code, or feature request

If you are reporting (1) a bug or (2) a question about code, please supply:

  • ensure that you can install and successfully load rJava
  • a fully reproducible example using a publicly available dataset (or provide your data)
  • if an error is occurring, include the output of traceback() run immediately after the error occurs
  • the output of sessionInfo()

Put your code here:

## rJava loads successfully
# install.packages("rJava")
library("rJava")

## load package
library("tabulizer")

## code goes here


## session info for your system
sessionInfo()
@pachadotdev
Copy link
Contributor

JDK 7 is required because posterior changes break some methods
we need to fix this during 2023

@HomeStayMom
Copy link
Author

Hi, I would be happy to see the improvement. Considering the jdk 7!is way out of lifecycle where install jdk results in error as well jdk 8 is old, would be nice some improvement where more recent and free jdk could be used. Looking forward to it. Good luck and hope to hear from the repo soon!

@pachadotdev
Copy link
Contributor

Hola @jazzido

Sigo con la idea de actualizar tabula. ¿Te parece si nos reunimos durante Marzo para corregir issues como este?

@HomeStayMom
Copy link
Author

Guys, as a Stay home bored mom who enjoys playing around with tabulaR and trying making to work, I feel would be a great idea.!!!
An other topic might be interesting:
As per Tabula
" Caveat: Tabula only works on text-based PDFs, not scanned documents. If you can click-and-drag to select text in your table in a PDF viewer (even if the output is disorganized trash), then your PDF is text-based and Tabula should work."

@pachadotdev
Copy link
Contributor

Guys, as a Stay home bored mom who enjoys playing around with tabulaR and trying making to work, I feel would be a great idea.!!! An other topic might be interesting: As per Tabula " Caveat: Tabula only works on text-based PDFs, not scanned documents. If you can click-and-drag to select text in your table in a PDF viewer (even if the output is disorganized trash), then your PDF is text-based and Tabula should work."

hi @HomeStayMom , we'll work on making it work nicely with regular pdfs first, and then see if we can expand to images in pdfs

@pachadotdev
Copy link
Contributor

@HomeStayMom , I made changes to make it work with OpenJDK 11

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

2 participants