search

openai / automated-interpretability

940 stars 110 forks source link

Fix new simulator's call to prompt builder #31

Closed hijohnnylin closed 8 months ago

hijohnnylin commented 8 months ago

New simulator tries to initialize prompt builder with allow_extra_system_messages but that flag should be added in .build instead.

henktillman commented 8 months ago

I think once https://github.com/openai/automated-interpretability/pull/30/files is merged this won't be necessary? (thank you for attempting the fix though!)

hijohnnylin commented 8 months ago

Sounds good! Closing. Btw - I am in no rush to merge any of these pull requests - am happy iterating on them in the meantime.