Skip to content

Commit

Permalink
Merge pull request #17 from JuliaTrustworthyAI/16-serializationdeseri…
Browse files Browse the repository at this point in the history
…alization-causes-issue-with-mpijl

attempting simple fix
  • Loading branch information
pat-alt authored Nov 19, 2024
2 parents 156d666 + 376ec2f commit 6d50972
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 1 deletion.
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.

The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/), and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## Version [1.1.2] - 2024-11-18

### Changed

- Updated MPI functionality to avoid issue related to deserialization. [#16]

## Version [1.1.1] - 2024-11-13

### Changed
Expand Down
2 changes: 1 addition & 1 deletion Project.toml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name = "TaijaParallel"
uuid = "bf1c2c22-5e42-4e78-8b6b-92e6c673eeb0"
authors = ["Patrick Altmeyer <p.altmeyer@tudelft.nl>"]
version = "1.1.1"
version = "1.1.2"

[deps]
CounterfactualExplanations = "2f13d31b-18db-44c1-bc43-ebaf2cff0be0"
Expand Down
3 changes: 3 additions & 0 deletions ext/MPIExt/evaluate.jl
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,9 @@ function TaijaBase.parallelize(
MPI.Barrier(parallelizer.comm)
end

# Collect all chunks in rank 0:
MPI.Barrier(parallelizer.comm)

# Load output from rank 0:
if parallelizer.rank == 0
outputs = []
Expand Down
3 changes: 3 additions & 0 deletions ext/MPIExt/generate_counterfactual.jl
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,9 @@ function TaijaBase.parallelize(
MPI.Barrier(parallelizer.comm)
end

# Collect all chunks in rank 0:
MPI.Barrier(parallelizer.comm)

# Load output from rank 0:
if parallelizer.rank == 0
outputs = []
Expand Down

2 comments on commit 6d50972

@pat-alt
Copy link
Member Author

Choose a reason for hiding this comment

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

@JuliaRegistrator register

Release notes:

@JuliaRegistrator
Copy link

Choose a reason for hiding this comment

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

Registration pull request created: JuliaRegistries/General/119753

Tagging

After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.

This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:

git tag -a v1.1.2 -m "<description of version>" 6d509726dfad52dab185679787173ab17b3eb557
git push origin v1.1.2

Please sign in to comment.