Skip to content

Fix HiggsDataLoader due to missing class methods#1641

Merged
rahulga1 merged 1 commit intosecurefederatedai:developfrom
rahulga1:higgs_fix
May 20, 2025
Merged

Fix HiggsDataLoader due to missing class methods#1641
rahulga1 merged 1 commit intosecurefederatedai:developfrom
rahulga1:higgs_fix

Conversation

@rahulga1
Copy link
Copy Markdown
Collaborator

Summary

Added required methods to make it compatible.

Type of Change (Mandatory)

Specify the type of change being made.

  • Bug fix

Description (Mandatory)

missed adding required methods for Higgs dataloader class

Testing

Locally testing.

This update introduces two new methods, `get_feature_shape` and `get_num_classes`, to the HiggsDataLoader class. These methods provide easy access to the feature shape and the number of classes for the Higgs dataset, enhancing the usability of the data loader.

Signed-off-by: Rahul Garg <rahul.garg@intel.com>
@rahulga1 rahulga1 changed the title Fix HiggsDataLoader due to feature issue Fix HiggsDataLoader due to missing class methods May 20, 2025
Copy link
Copy Markdown
Collaborator

@teoparvanov teoparvanov left a comment

Choose a reason for hiding this comment

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

LGTM!

@rahulga1 rahulga1 merged commit e716c88 into securefederatedai:develop May 20, 2025
37 checks passed
Copy link
Copy Markdown
Collaborator

@payalcha payalcha left a comment

Choose a reason for hiding this comment

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

Tested the cahnges its working as expected.

tayfunceylan pushed a commit to tayfunceylan/openfl that referenced this pull request May 23, 2025
…ederatedai#1641)

This update introduces two new methods, `get_feature_shape` and `get_num_classes`, to the HiggsDataLoader class. These methods provide easy access to the feature shape and the number of classes for the Higgs dataset, enhancing the usability of the data loader.

Signed-off-by: Rahul Garg <rahul.garg@intel.com>
Signed-off-by: Tayfun Ceylan <tayfun.ceylan>
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.

3 participants