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.
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
Support for Hyperlink Templates on Dimensions #200
Conversation
b6b6860
to
cb09711
Compare
…he reacttable widget
cb09711
to
80cf07f
Compare
fireant/slicer/widgets/reacttable.py
Outdated
@staticmethod | ||
def map_hyperlink_templates(df, dimensions): | ||
""" | ||
WRITEME |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add this header comment now rather than later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
def map_hyperlink_templates(df, dimensions): | ||
""" | ||
WRITEME | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this docstring is still to be written
fireant/slicer/widgets/reacttable.py
Outdated
arguments = [format_dimension_key(argument[1:-1]) | ||
for argument in pattern.findall(hyperlink_template)] | ||
missing_dimensions = set(arguments) & set(df.index.names) | ||
if not missing_dimensions: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor but I think this could have a clearer name. Instead of missing_dimensions
this could be dimensions_to_hyperlink
or something.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could rename it to unavailable_hyperlink_parameters
. This is a set of the dimensions required to fill in the hyperlink template that are not in the result data set.
Added support for hyperlink templates in dimensions which work with the reacttable widget