Skip to content

Commit

Permalink
Merged PR 25518: Upgrade Azure Pipelines to macos-12
Browse files Browse the repository at this point in the history
macos-10.15 will become unsupported in December 2022. Changes:
* Upgrade Azure DevOps to macos-12
* Pull marian-nmt/sentencepiece#14
* Fix clang 13 errors as in marian-nmt#939
  • Loading branch information
Roman Grundkiewicz committed Sep 15, 2022
1 parent 5d466bc commit a47912d
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 4 deletions.
2 changes: 1 addition & 1 deletion azure-pipelines.yml
Original file line number Diff line number Diff line change
Expand Up @@ -328,7 +328,7 @@ stages:
displayName: macOS CPU clang

pool:
vmImage: macos-10.15
vmImage: macos-12

steps:
- checkout: self
Expand Down
2 changes: 1 addition & 1 deletion src/3rd_party/sentencepiece
2 changes: 0 additions & 2 deletions src/training/communicator.h
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,6 @@ class DefaultCommunicator : public ICommunicator {
int totalSize = (int)graphs_[0]->params()->vals()->size();
int shardSize = (int)ceil(totalSize / (float)graphs_.size());

int pos = 0;
for(auto graph : graphs_) {
int __size__ = std::min(shardSize, totalSize);

Expand All @@ -145,7 +144,6 @@ class DefaultCommunicator : public ICommunicator {
tmpTensors_.push_back(tmp);

// move to next shard
pos += __size__;
totalSize -= __size__;
}
}
Expand Down

0 comments on commit a47912d

Please sign in to comment.