Skip to content

Multiple backend using narwhals #145

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
Open
wants to merge 43 commits into
base: develop
Choose a base branch
from
Open

Multiple backend using narwhals #145

wants to merge 43 commits into from

Conversation

@movsesyanae
Copy link
Contributor

@movsesyanae movsesyanae commented Feb 7, 2025

No description provided.

Alexander Movsesyan and others added 30 commits September 27, 2024 00:20
initial draft of base reader
cb55f9e
updates to base reader
8d7d9e2
base reader working without cct creation
ceae939
Base Reader for single threaded reading
8f7880a
changes to make core reader more parallel-friendly
65e4332
updates to have strided unique_id, take care of instant events, and t…
b959fee 
…o concat trace dfs
style updates
1fd2bc9
@movsesyanae
Merge branch 'develop' into abstract-base-reader
4c15683
@zoreoo
Implement otf2 reader with core reader
0aa4669
@zoreoo
Fix metrics for non-paired events
4501825
@zoreoo
Fix another thing with metrics
e615e01
@zoreoo
Ensure metrics_dict is reset
5a0157e
minor update
02a00e5
updated non-existing int values to -1
27e95f8
core_reader -- initial implementation to use narwhals df and support …
83939a5 
…multiple backends
otf2_reader -- converting to use nw frame, failing mp, so forcing 1 p…
a434f2b 
…rocess
trace -- updated constructor to use nw frame
8e31839
otf2 reader -- removing forced 1 process
5b2b08b
requirements.txt -- added narwhals as dependency
7c6107b
otf2_reader -- default num processes restored to None
a6ea249
otf2_reader -- updated matching timestamp to account for shift
8ac685f
core_reader -- removing int cast for matching timestamp
c0a7567
trace -- updating match_events to use nw dfs (polars api)
a98cf1f
trace -- updated match_caller_callee to use polars API with nw frame
54e7437
trace -- updated calc_inc_metrics to use nw frame
80d898e
trace -- updated match_exc_events for nw and minor update to match_in…
4674b35 
…c_events
trace -- fixed minor bug in match events and caller/callee
fe92ed7
trace -- minor bugfix for calc_exc and reduced to 1 join op
eae91c1
trace and core_reader -- changing match_event to shared matching_even…
9e8940a 
…t_id
trace and core_reader -- moved back to matching_event and inc metrics…
6860bcd 
… updated to be only one join
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@movsesyanae @zoreoo