UB-Mannheim / zotero-ocr

Zotero Plugin for OCR
GNU Affero General Public License v3.0
551 stars 40 forks source link

Zotero 7 Support #52

Closed 7596ff closed 8 months ago

7596ff commented 1 year ago

Per the Zotero Forums, all Zotero plugins will need to be rewritten to support the next major version.

ricklei2777 commented 1 year ago

I agree! Zotero 7 is great and beautiful

stweil commented 1 year ago

Contributions which add the required updates are welcome!

Related documentation: https://www.zotero.org/support/dev/zotero_7_for_developers

Ideally the plugin should support both Zotero 6 and Zotero 7, at least for some time.

lisasuosuo commented 1 year ago

I agree too!

aborel commented 9 months ago

I'm trying to port the plugin to Zotero 7. This is a first for me, so I'm really learning by doing - but I think I'm getting closer and closer to something that works.

aborel commented 9 months ago

I have reached the point where everything seems to work, but it is only for Zotero 7: I am not really interested in learning how to deal with Zotero 6, since it might go away soon-ish... @stweil would a Z7-only PR be acceptable?

gravybot commented 8 months ago

I have reached the point where everything seems to work, but it is only for Zotero 7: I am not really interested in learning how to deal with Zotero 6, since it might go away soon-ish... @stweil would a Z7-only PR be acceptable?

I'd certainly be interested in getting hold of that; I'm loving Z7 but do use OCR fairly regularly.

stweil commented 8 months ago

@zuphilip, do you think that support for Zotero 6 is still required? If no, we can go forward towards Zotero 7. If yes, then we either need a new release which supports both Zotero 6 and 7. Or – if that is too difficult or not possible – it might be necessary to have two development branches: master for Zotero 7 and a 2nd branch for Zotero 6.

zuphilip commented 8 months ago

That is great to hear @aborel and I am looking forward for any Zotero 7 compatible version of the plugin. Thank you very much for working on that!

Zotero 6.0.30 is the current version and Zotero 7 is in (stable) Beta. However, I am not sure that they switch soon the main version. But it should be no problem to have two different branches for the two versions. We could do this first also without releasing any new version here. The build script can also been run locally until we have a better solution. We need to be careful with releasing because the GitHub release will automatically been updated for all current users of the plugin.

aborel commented 8 months ago

@zuphilip I agree that updates are an important concern, and it is probably better to not have an official release at this point. Also, a new branch makes sense. I'm going to clean up my repository, then I'll submit my PR when the new branch is available here.

BoseJohnson commented 8 months ago

please somebody update this add-on for z7,

kaiwen-wang commented 8 months ago

Here are some tips for doing it manually that I wrote on my blog. I might be interested in writing a Zotero 7 OCR plugin as it doesn't seem too complicated, but I think ocrmypdf has a lot of good add-ins that tesseract by itself doesn't have.

stweil commented 8 months ago

Here is a test version of the Zotero OCR plugin: https://github.com/UB-Mannheim/zotero-ocr/actions/runs/8417350410/artifacts/1354939817. It should work with Zotero 6 and Zotero 7.

stweil commented 8 months ago

The new release 0.7.0 now supports Zotero 7 (thanks to @aborel) while still being compatible with Zotero 6, too.

Update 2024-05-17: Please always look for the latest release. The first release 0.7.0 worked with older versions of Zotero 7, but not with newer ones.

dstark commented 6 months ago

The new release 0.7.0 now supports Zotero 7

@aborel and @stweil, thanks so much for this! As far as I can find, it seems that in Zotero 7 (Windows: beta.78+5be5ca941), there's no UI element corresponding to the "OCR Selected PDF(s)" command available in the right-click menu in Zotero 6. Is there another mechanism for triggering the OCR in Zotero 7 that I'm just missing?

image

stweil commented 6 months ago

Did you install the latest release 0.7.2? There should be the same UI element for starting OCR as in Zotero 6.

dstark commented 6 months ago

Did you install the latest release 0.7.2?

Apparently not. It looks like I was just so excited to see the update for Zotero 7 that I grabbed the 0.7.0 version from the link above, installed it, and didn't check if it was still the most recent version. Sorry about that. 0.7.2 works perfectly and, yes, does have the expected UI element. Thanks so much!