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

Map operator #278

Merged

ianspektor merged 20 commits into main from map

Oct 10, 2023

Collaborator

ianspektor commented Oct 6, 2023

No description provided.

Ian Spektor added 12 commits

October 4, 2023 14:51


          run create op

ed8af86


          base version working

858f270


          test serialize fails

7599f52


          add extras

12be004


          fixes

f2d3638


          add output_dtypes to core

24233bf


          add tests, test output dtype param

72f89b8


          create tp.types module, improve docs

93ac1c3


          fix docstrings

96f5bb9


          add usage examples

f0d15d1


          changelog

c373676


          Merge branch 'main' into map

4f6e1a5

ianspektor requested review from achoum and DonBraulio

October 6, 2023 10:37


          fix notebooks

3d079d6

github-actions bot commented Oct 6, 2023

Coverage report

Main: 90.50% | PR: 90.68% | Diff: 0.18 ✅

google deleted a comment from github-actions bot

achoum reviewed

View reviewed changes

temporian/core/event_set_ops.py Outdated Show resolved Hide resolved

temporian/core/event_set_ops.py Show resolved Hide resolved

temporian/core/event_set_ops.py Outdated Show resolved Hide resolved

temporian/core/operators/map.py Show resolved Hide resolved

temporian/implementation/numpy/data/dtype_normalization.py Show resolved Hide resolved

temporian/implementation/numpy/operators/map.py Outdated Show resolved Hide resolved

temporian/implementation/numpy/operators/map.py Outdated Show resolved Hide resolved

temporian/implementation/numpy/operators/map.py Outdated Show resolved Hide resolved

temporian/core/operators/map.py Outdated Show resolved Hide resolved

temporian/test/utils.py Outdated Show resolved Hide resolved


          address PR comments

3615ad6

ianspektor requested a review from achoum

October 9, 2023 15:17

achoum reviewed

View reviewed changes

temporian/core/operators/test/test_map.py Show resolved Hide resolved

achoum reviewed

View reviewed changes

temporian/implementation/numpy/operators/map.py Outdated Show resolved Hide resolved

achoum approved these changes

View reviewed changes

achoum reviewed

View reviewed changes

temporian/implementation/numpy/operators/map.py Outdated Show resolved Hide resolved

achoum reviewed

View reviewed changes

temporian/implementation/numpy/operators/map.py Outdated Show resolved Hide resolved

Ian Spektor added 5 commits

October 10, 2023 16:19


          reuse mapextras obj

66f0f14


          add receive_extras param

0c08a20


          add is_serializable proto attr

35f0279


          add test with builtin

dc95b14


          add tests

e5fbaf7


          Merge branch 'main' into map

github-actions bot commented Oct 10, 2023

Coverage report

Main: 90.55% | PR: 90.71% | Diff: 0.17 ✅

ianspektor merged commit 95ce3ff into main

20 checks passed

ianspektor deleted the map branch

October 10, 2023 15:32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment