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

save file instead of show image in windows #2261

Merged
merged 1 commit into from
Mar 15, 2020
Merged

save file instead of show image in windows #2261

merged 1 commit into from
Mar 15, 2020

Conversation

MeepoAII
Copy link
Contributor

@MeepoAII MeepoAII commented Mar 15, 2020

I notice your comment on the ‘out_file‘ parameter of the show_result function in inference.py: If out_file is specified, the visualization result will be written to the out file instead of shown in a window, but actually it is shown in windows. (and because I run my python in no gui environment, even though i specify the out_file, it shows cannot connect to X server, which may be confused to somebody new to this)
So, I add a if statement before show image.

@hellock hellock merged commit 296cf6f into open-mmlab:master Mar 15, 2020
@hellock
Copy link
Member

hellock commented Mar 15, 2020

Thanks for the fix.

ioir123ju pushed a commit to ioir123ju/mmdetection that referenced this pull request Mar 30, 2020
mike112223 pushed a commit to mike112223/mmdetection that referenced this pull request Aug 25, 2020
@OpenMMLab-Assistant-004
Copy link

Hi @MeepoAII !First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.

We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us :https://discord.gg/raweFPmdzG

If you have WeChat,welcome to join our community on WeChat. You can add our assistant :openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends:)
Thank you again for your contribution❤

1 similar comment
@OpenMMLab-Assistant-004
Copy link

Hi @MeepoAII !First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.

We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us :https://discord.gg/raweFPmdzG

If you have WeChat,welcome to join our community on WeChat. You can add our assistant :openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends:)
Thank you again for your contribution❤

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.

None yet

3 participants