Skip to content

Fix disptach method problem#38

Open
mxrcx wants to merge 16 commits intodevfrom
fix-disptach-method-problem
Open

Fix disptach method problem#38
mxrcx wants to merge 16 commits intodevfrom
fix-disptach-method-problem

Conversation

@mxrcx
Copy link
Copy Markdown
Collaborator

@mxrcx mxrcx commented Jun 25, 2023

No description provided.

Comment thread src/pyicu/tbl_utils.py Outdated
def dur_unit(x):
return dur_col(x).units # Changed from units(dur_col(x))

@singledispatch
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This doesn't choose the correct case

Comment thread src/pyicu/tbl_utils.py Outdated
from pyicu.utils_cli import warn_dots
from functools import singledispatch

@singledispatch
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

if the default method is included it chooses the default --> stops run

Comment thread src/pyicu/callbacks/concept.py Outdated
if mode == "match_vals":
print(x)
print(x['po2'])
print(TableAccessor.is_id_tbl(x))
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

is_ts_tbl() returns False, whereas is_id_tbl() returns an AttributeError: 'dict' object has no attribute 'is_pandas'

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