PLaplacianTests fail with Julia 1.7.3

オープン
#129 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
julia

調査の方向性

test/PLaplacianTests.jl の11〜13行目付近から始め、Julia 1.7.3 で失敗を再現してから、Julia 1.9.3 での成功した実行と比較します。参照されているコミットを含め、一覧にある互換性および PartitionedArrays.jl のオプションを確認し、PLaplacianTests を再実行して選択した修正を確認します。

索引モデルが issue の本文から書いたものです。

説明

The error is ERROR: Scalar indexing on DebugArray is not allowed for performance reasons. and it occurs at this line

With Julia 1.9.3 it runs sucessfully

The reason for this behavior is the change in the implementation of copy!(dst::AbstractVector, src::AbstractVector) which indexed dst directly in v1.7.3, see here, whereas it calls copyto! in v1.9.3, see here

Possible solutions I see:

  1. increase compat
  2. implement copy! for DebugArray and MPIArray in PartitionedArrays.jl
  3. eliminate this overload which is not required after the this commit to PartitionedArrays.jl (although the test will fail with previous releases). I verified that copy(a::PSparseMatrix) and copy!(b::PSparseMatrix,a::PSparseMatrix) work with latest PartitonedArrays.jl.

@fverdugo , @amartinhuertas , @JordiManyer any thoughts?

主要言語
Julia
スター
150
フォーク
28
PR マージ指標
30日以内にマージされた PR はありません

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

gridap/GridapDistributed.jl のほかの issue

gridap/GridapDistributed.jl の issue をすべて見る

似ている issue

Julia の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。