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

chore(data/mv_polynomial): use classical logic #1391

Merged
merged 46 commits into from Sep 6, 2019
Merged

Conversation

semorrison
Copy link
Collaborator

@semorrison semorrison commented Sep 3, 2019

Per discussion on Zulip, this PR makes data/finsupp, data/polynomial and data/mv_polynomial classical.

@semorrison semorrison requested a review from a team as a code owner September 3, 2019 02:36
Scott Morrison added 2 commits September 3, 2019 13:25
@semorrison semorrison changed the title chore(data/mv_polynomial): use classical logic chore(data/mv_polynomial): use classical logic [WIP] Sep 3, 2019
@semorrison
Copy link
Collaborator Author

Sorry, there's still some more files that I missed, so I'm marking this as WIP.

@semorrison
Copy link
Collaborator Author

@jcommelin, @kckennylau, I've run into quite a few problems in power_series.lean and in free_comm_ring.lean. Would either of you be able to help out here?

@khoek khoek added the WIP Work in progress label Sep 3, 2019
@semorrison
Copy link
Collaborator Author

@jcommelin, @kckennylau, I've run into quite a few problems in power_series.lean and in free_comm_ring.lean. Would either of you be able to help out here?

Thanks! We're down to one remaining problem, described above in the discussion with Mario.

@semorrison semorrison changed the title chore(data/mv_polynomial): use classical logic [WIP] chore(data/mv_polynomial): use classical logic Sep 6, 2019
@ChrisHughes24 ChrisHughes24 merged commit a5fa162 into master Sep 6, 2019
@ChrisHughes24 ChrisHughes24 deleted the decidable_eq branch September 6, 2019 10:46
@khoek khoek removed the WIP Work in progress label Oct 13, 2019
anrddh pushed a commit to anrddh/mathlib that referenced this pull request May 15, 2020
…1391)

* refactor(linear_algebra/lc): use families not sets

* refactor(linear_algebra/lc): merge lc into finsupp

* refactor(linear_algebra/lc): localize decidability

* refactor(linear_algebra/lc): finsupp instances

* refactor(linear_algebra/lc): use families instead of sets

* refactor(linear_algebra/lc): remove set argument in lin_indep

* refactor(linear_algebra/lc): clean up

* refactor(linear_algebra/lc): more clean up

* refactor(linear_algebra/lc): set_option in section

* refactor(linear_algebra/lc): decidability proof

* refactor(linear_algebra/lc): arrow precedence

* refactor(linear_algebra/lc): more cleanup

* make data.finsupp classical

* trouble with data/polynomial

* ...

* more classical

* merge

* merge

* merge

* fix

* removing more

* minor

* ?

* progress, using convert

* working?

* remove some unnecessary converts

* fixes

* err

* oops

* various

* various

* fix free_comm_ring

* remove test lines

* fix linear_algebra/matrix.lean

* Fix errors in power_series.lean

* trying to turn instances back on

* restore some instances

* no joy

* fix mv_polynomial errors

* another convert
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

6 participants