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.

Sign up for GitHub

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

Jump to bottom

Ignoring identity transforms is unjustified #248

Open

rptb1 opened this issue Jun 16, 2023 · 1 comment

Labels

essential review-raise

Member

rptb1 commented Jun 16, 2023

m7. It seems dangerous to quietly ignore identity transforms, the client might like to know if this means there's a problem in there code. If there's a good reason why ignoring identity transforms meets a requirement, we should refer to it here

Originally posted by @thejayps in #214 (comment)

This needs investigation to discover

What, if anything, depends on this restriction?
Is it just a convenient optimisation?
Does it matter to the client programmer?
Should it be an error?

rptb1 mentioned this issue

Make transforms part of the public MPS #214

Merged

thejayps added the review-raise label

Contributor

thejayps commented Jun 19, 2023

See also #251

thejayps mentioned this issue

Transforms is estimated to contain more than 2 undiscovered major defects #251

Open

thejayps added the essential label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment