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

check python version before import imp #30

Closed
wants to merge 1 commit into from
Closed

Conversation

wsyxbcl
Copy link

@wsyxbcl wsyxbcl commented Aug 13, 2024

imp been removed in python 3.12
python/cpython#98040

@Erotemic
Copy link
Owner

I have a similar patch in #29 looks like there is a bit more CI work to do

@wsyxbcl
Copy link
Author

wsyxbcl commented Aug 13, 2024

I have a similar patch in #29 looks like there is a bit more CI work to do

Didn't see the dev branch at all :D

BTW., I'm playing around with ibeis today—great work! I noticed a significant delay when dragging the annotation box while adding annotations. I'll check it later (any idea?) and will probably file an issue.

@wsyxbcl wsyxbcl closed this Aug 13, 2024
@Erotemic
Copy link
Owner

I merged and uploaded the patch.

WRT to the IBEIS annotations: I'm not surprised at all. All of those interactions are written in matplotlib which is not known for its interaction scalability or speed. It might be possible to speed it up, but you may want to use labelme / labelstudio / cvat, export the annotations and then write a script that ingests them into ibeis rather than being forced to use the shoddy in-app annotation abilities.

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 this pull request may close these issues.

2 participants