Skip to content

Fix #2414: Add ExclusionModel support to profile export#2434

Draft
Sayeem3051 wants to merge 1 commit intoborgbase:masterfrom
Sayeem3051:patch-1
Draft

Fix #2414: Add ExclusionModel support to profile export#2434
Sayeem3051 wants to merge 1 commit intoborgbase:masterfrom
Sayeem3051:patch-1

Conversation

@Sayeem3051
Copy link

Added ExclusionModel handling to profile export and database methods.

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have read the CONTRIBUTING guide.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

I provide my contribution under the terms of the license of this repository and I affirm the Developer Certificate of Origin.

Added ExclusionModel handling to profile export and database methods.
@m3nu
Copy link
Contributor

m3nu commented Mar 15, 2026

This looks better, but you sniped the issue away from someone else who asked to work on it properly. So we'll give him time to come up with their own PR. If they end up not working on it, we can consider this.

#2414 (comment)

Working on an open source project also includes understanding the context and working with other contributors. It's not that the fastest will win.

@m3nu m3nu marked this pull request as draft March 15, 2026 13:01
@Sayeem3051
Copy link
Author

Thank you for the feedback, @m3nu. I understand and completely agree - I should have checked the issue context more carefully before working on it. I see now that someone else had already asked to work on this issue.

I apologize for jumping ahead without consulting with the community first. You're absolutely right that collaboration and understanding the context is important in open source projects. It's not about speed, but about being a good team member and respecting others' contributions.

I'll close this PR and let the other contributor have the opportunity to work on it. If they don't end up completing it, I'd be happy to contribute then.

Thank you for the learning opportunity - I'll make sure to check issue discussions and be more thoughtful about picking up issues in the future.

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