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

Replace items not just based on their name #43

Open
senekor opened this issue Nov 23, 2023 · 0 comments
Open

Replace items not just based on their name #43

senekor opened this issue Nov 23, 2023 · 0 comments

Comments

@senekor
Copy link
Contributor

senekor commented Nov 23, 2023

I think this test case is inadvertantly exposing a sever limitation of the current representer. This function and its local variable have the same name and are assigned the same placeholder. That should not be the case however, as they refer to different items.

For running the representer in production, this means that two identical solutions may not receive the same representation, simply because two two different items shared or didn't share the same name.

Originally posted by @senekor in #30 (comment)

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

No branches or pull requests

1 participant