Skip to content

missing resources.has_content#14

Merged
OmerShubi merged 1 commit into
mainfrom
pymovements-resources
Mar 23, 2026
Merged

missing resources.has_content#14
OmerShubi merged 1 commit into
mainfrom
pymovements-resources

Conversation

@SiQube
Copy link
Copy Markdown
Contributor

@SiQube SiQube commented Mar 23, 2026

No description provided.

@OmerShubi
Copy link
Copy Markdown
Contributor

I am getting


> dataset.definition.resources.has_content['precomputed_events']
Traceback (most recent call last):
  File "<string>", line 1, in <module>
TypeError: 'method' object is not subscriptable

dataset = pm.Dataset(dataset_def, f'data/{dataset_name}').load()

if dataset.definition.has_content['precomputed_events']:
if dataset.definition.resources.has_content['precomputed_events']:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
if dataset.definition.resources.has_content['precomputed_events']:
if dataset.definition.resources.has_content('precomputed_events'):

logger.info(f'{dataset_name} has no precomputed events...')

if dataset.definition.has_content['precomputed_reading_measures']:
if dataset.definition.resources.has_content['precomputed_reading_measures']:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
if dataset.definition.resources.has_content['precomputed_reading_measures']:
if dataset.definition.resources.has_content('precomputed_reading_measures'):

Copy link
Copy Markdown
Contributor

@OmerShubi OmerShubi left a comment

Choose a reason for hiding this comment

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

[]->()

@SiQube SiQube force-pushed the pymovements-resources branch from b1bbc9d to d42795c Compare March 23, 2026 09:11
@OmerShubi OmerShubi merged commit 68b6d08 into main Mar 23, 2026
2 checks passed
@OmerShubi OmerShubi deleted the pymovements-resources branch March 23, 2026 09:13
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.

2 participants