ApolloResearch / rib

Library for methods related to the Local Interaction Basis (LIB)
MIT License
2 stars 0 forks source link

Ignore 0th position #334

Closed nix-apollo closed 1 month ago

nix-apollo commented 5 months ago

Ignore the 0th sequence position

Description

This isn't well tested or carefully handled with asserts, but I'm using it for experiments.

Related Issue

313

Motivation and Context

How Has This Been Tested?

I added a single test to make sure it runs. I'll run and examine outputs.

Does this PR introduce a breaking change?

No

stefan-apollo commented 4 months ago

Discussed the motivation with Nix and said

Also Nix made a good unrelated argument

So we tentatively concluded we probably do not want to ignore the 0th position anymore, but if someone wants to finish the PR and implement the changed ablations that's also fine. Both are sensible RIB choices.

stefan-apollo commented 1 month ago

We decided to not implement this feature