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

Removes deprecated CustomPeriodicEvent #1255

Closed
wants to merge 15 commits into from
Closed

Removes deprecated CustomPeriodicEvent #1255

wants to merge 15 commits into from

Conversation

Tawishi
Copy link
Contributor

@Tawishi Tawishi commented Aug 29, 2020

In response to #1247 (comment)

Description: Removes the missed out instance of CustomPeriodicEvent from notebook example:
https://github.com/pytorch/ignite/blob/master/examples/notebooks/EfficientNet_Cifar100_finetuning.ipynb

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Aug 29, 2020

@Tawishi thanks for the PR, however here we have to replace CustomPeriodicEvent by the new API as I described it here : #1247 (comment)

Please, update the PR according to that and if you could rerun it inside Colab it would be awesome.
Let me know if you have any questions.

@Tawishi
Copy link
Contributor Author

Tawishi commented Sep 1, 2020

@vfdev-5 I am facing this error and a warning while installations. Please help.

I ran the second command and got this in the terminal window.

1. git clone https://github.com/NVIDIA/apex && cd apex
2. pip install --no-cache-dir --global-option="--cpp_ext" --global-option="--cuda_ext" .

Screenshot from 2020-09-01 12-39-33

It seems to a Python3.3 error as stated in this post. How to go about this?

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Sep 1, 2020

@Tawishi I think it could not identify the location of cuda library. And also there is a message saying that there is no GPUs on the system... May I ask you what would you like to do ?

If you would like to test your modifications, I'd suggest you to use Colab where you can choose a system with GPU and cuda properly installed.

@Tawishi
Copy link
Contributor Author

Tawishi commented Sep 1, 2020

I will try out Colab and let you know if I face any problem.

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Sep 4, 2020

@Tawishi any updates on this PR ?

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Sep 8, 2020

@Tawishi do you need any help with the PR ? Feel free to reach me out at discord if you need to ask several questions...

@Tawishi
Copy link
Contributor Author

Tawishi commented Sep 10, 2020

@Tawishi do you need any help with the PR ? Feel free to reach me out at discord if you need to ask several questions...

Apologize for the delay.
Please send me your username and tag. I am still working on the PR.

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Sep 10, 2020

Here it is vfdev-5#4458 at https://discord.com/channels/720788516837261322/

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Oct 18, 2020

Closed in favor of #1389

@vfdev-5 vfdev-5 closed this Oct 18, 2020
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.

3 participants