Skip to content
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

fix session view #518

Merged
merged 1 commit into from
Jan 8, 2024
Merged

fix session view #518

merged 1 commit into from
Jan 8, 2024

Conversation

durandom
Copy link
Member

@durandom durandom commented Jan 8, 2024

No description provided.

@op1st-prow op1st-prow bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jan 8, 2024
@durandom durandom merged commit 629b5ff into b4mad:main Jan 8, 2024
2 of 3 checks passed
@durandom durandom deleted the fix_session_view branch January 8, 2024 22:28
@@ -28,7 +29,7 @@
def get_session_df(self, session_id, measurement="laps_cc", bucket="racing"):
file_path = f"{self.temp_dir}/session_{session_id}_df.csv.gz"

if os.path.exists(file_path):
if os.path.exists(file_path) or False:

Check failure

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a
user-provided value
.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant