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

Moved from fvcore's PathManager to iopath's #93

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

thomasehuang
Copy link
Collaborator

This fixes recent issues about the demo that is caused by the fvcore update.

@thomasehuang thomasehuang requested a review from xinw1012 March 4, 2021 03:01
@lgtm-com
Copy link

lgtm-com bot commented Mar 4, 2021

This pull request fixes 1 alert when merging 34028a7 into 6b0769b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@lgtm-com
Copy link

lgtm-com bot commented Mar 4, 2021

This pull request fixes 1 alert when merging 4944930 into 6b0769b - view on LGTM.com

fixed alerts:

  • 1 for Unused import

@Anthuang
Copy link
Collaborator

Still seeing the following when running the demo:

** fvcore version of PathManager will be deprecated soon. **
** Please migrate to the version in iopath repo. **
https://github.com/facebookresearch/iopath

Should change these too:

https://github.com/ucbdrive/few-shot-object-detection/blob/model_zoo/tools/visualize_json_results.py#L7

https://github.com/ucbdrive/few-shot-object-detection/blob/model_zoo/datasets/prepare_voc_few_shot.py#L2

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