Skip to content

Add streaming for stage two#235

Open
thejaminator wants to merge 30 commits intomainfrom
add-streaming-stage-one
Open

Add streaming for stage two#235
thejaminator wants to merge 30 commits intomainfrom
add-streaming-stage-one

Conversation

@thejaminator
Copy link
Collaborator

No description provided.

messages=messages,
config=config,
)
self.cache[key] = value
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.01s speedup bro

col: str = "original_cot_trace_length",
hue: str = "model",
):
df = convert_stage2_experiment_to_dataframe(items)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

refactoring so that you can plot directly from the actual data

@thejaminator thejaminator changed the title Add streaming for stage one / two wip: Add streaming for stage two Oct 29, 2023
@thejaminator thejaminator changed the title wip: Add streaming for stage two Add streaming for stage two Oct 30, 2023
@thejaminator thejaminator requested a review from erees1 October 30, 2023 03:26
proportion_scaled = proportion_of_cot.apply(lambda x: (x - min_) / (max_ - min_))
except Exception as e:
print("Maybe there is only one value so min_ == max_")
raise e
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah i think the current aoc metrics is weird, i say let's give up on that for now lol

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

( unless you do things like allow insertions of mistakes at the start and end, and don't care if its a good mistake / any reasoning there)

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

Successfully merging this pull request may close these issues.

1 participant