Oscar.jl icon indicating copy to clipboard operation
Oscar.jl copied to clipboard

Purge the examples folder

Open thofma opened this issue 5 years ago • 8 comments

I just spent too much time tracking down a bug which appeared when changing the version from 0.5.1-DEV to 0.5.1. The example folder files requires include code which is necessary for the non-example tests to pass. But the examples folder is only incuded, if is_dev is true.

thofma avatar Feb 12 '21 10:02 thofma

On Fri, Feb 12, 2021 at 02:26:22AM -0800, thofma wrote:

I just spent too much time tracking down a bug which appeared when changing the version from 0.5.1-DEV to 0.5.1. The example folder code requires code which is necessary for the tests to pass. But the examples folder is only incuded, if is_dev is true.

Move the stuff that is automatically tested in dev into experimental? Where it is always included?

You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/oscar-system/Oscar.jl/issues/280

fieker avatar Feb 12 '21 10:02 fieker

https://github.com/oscar-system/Oscar.jl/blob/master/src/Oscar.jl#L218-L226

thofma avatar Feb 12 '21 10:02 thofma

I don't understand why this was not moved to experimental.

thofma avatar Feb 12 '21 12:02 thofma

Essentially this is a purely "experimental" implementation from a Fach Praktikum. Wolfram is waiting for the link to the Singular library Smilar: All of Delphine's stuff should be there, but that needs coordination with her to avoid merge conflicts and frustartion

fieker avatar Feb 12 '21 13:02 fieker

@delphinepol Once your recent PR is merged, could you make a PR to move your things from examples to experimental? That would be great, thanks.

thofma avatar Feb 12 '21 14:02 thofma

OK !

delphinepol avatar Feb 12 '21 14:02 delphinepol

I think that most of the code will probably be broken right now as all of this stuff is not covered by CI

lgoettgens avatar May 08 '24 10:05 lgoettgens

Triage: @fieker likes the examples folder. Someone (TM) would need to go through all of the examples code, fix it, remove stuff that is now in src or experimental, ...

Maybe @thofma or @fingolfin can be there next triage and bring up their opinion?

lgoettgens avatar May 08 '24 10:05 lgoettgens