Skip to content

replace lambda function for mac compatibility#32

Merged
ntxxt merged 6 commits intomainfrom
ci
Mar 3, 2026
Merged

replace lambda function for mac compatibility#32
ntxxt merged 6 commits intomainfrom
ci

Conversation

@rvhonorato
Copy link
Copy Markdown
Member

@rvhonorato rvhonorato commented Feb 26, 2026

when doing haddocking/haddock3#1480 I saw the actions were failing on macos with the error:

_pickle.PicklingError: Can't pickle <function HDF5DataSet.<lambda> at 0x1130d62a0>: attribute lookup HDF5DataSet.<lambda> on deeprank_gnn.DataSet failed

The solution to this is very simple, this HDF5DataSet is taking a lambda function as an argument into an object that is being passed to a multiprocessing worker. This works fine or linux but mac (for some reason) cannot handle the lambda assignment directly - so the fix is to simply pass it as a function instead.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Feb 26, 2026

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 8305a741
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (8305a74) Report Missing Report Missing Report Missing
Head commit (2919c47) 254 228 89.76%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#32) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

@rvhonorato rvhonorato changed the title update ci replace lambda function for mac compatibility Feb 26, 2026
@rvhonorato rvhonorato self-assigned this Feb 26, 2026
@rvhonorato rvhonorato requested a review from ntxxt February 26, 2026 20:22
@rvhonorato rvhonorato added the bug Something isn't working label Feb 26, 2026
@rvhonorato rvhonorato marked this pull request as ready for review February 26, 2026 20:23
Copy link
Copy Markdown
Collaborator

@ntxxt ntxxt left a comment

Choose a reason for hiding this comment

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

Looks good to me!

I didn't know lambda function is the cause of this.

@ntxxt ntxxt merged commit 351c27d into main Mar 3, 2026
8 checks passed
@ntxxt ntxxt deleted the ci branch March 3, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants