Zulip Chat Archive

Stream: mathlib4

Topic: add_port_comments.py


Eric Wieser (Nov 20 2022 at 11:08):

You need the unreleased version of mathlibtools; I mentioned that in the description of #17600:

pip install git+https://github.com/leanprover-community/mathlib-tools

You might need to uninstall your existing mathlibtools first

Eric Wieser (Nov 20 2022 at 11:10):

I agree this is sort of problematic, hence me creating this thread on moving the porting stuff out of mathlibtools so that we don't have to deal with release issues

Riccardo Brasca (Nov 20 2022 at 11:11):

I've done that, but nothing changed. :thinking:

Eric Wieser (Nov 20 2022 at 11:12):

Did you uninstall the old mathlibtools first?

Riccardo Brasca (Nov 20 2022 at 11:13):

Ah, I didn't

Eric Wieser (Nov 20 2022 at 11:13):

Maybe you need pip install git+https://github.com/leanprover-community/mathlib-tools --upgrade to force it to do that

Riccardo Brasca (Nov 20 2022 at 11:18):

Uninstalling mathlibtools and doing pip install git+https://github.com/leanprover-community/mathlib-tools did something, now there is no error. But there is no output at all

Riccardo Brasca (Nov 20 2022 at 11:18):

In any case I think we shouldn't say in the wiki to use the script until it is ready

Eric Wieser (Nov 20 2022 at 11:32):

The script gives no output, it modifies all the files in place

Notification Bot (Nov 20 2022 at 12:00):

10 messages were moved here from #mathlib4 > Mathlib4 porting meeting series by Eric Wieser.

Eric Wieser (Nov 20 2022 at 12:06):

Riccardo Brasca said:

In any case I think we shouldn't say in the wiki to use the script until it is ready

We can remove the instruction entirely after #17641


Last updated: Dec 20 2023 at 11:08 UTC