Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

問題追加 polynomial_root_finding #1084

Merged
merged 7 commits into from
Jan 21, 2024
Merged

Conversation

maspypy
Copy link
Collaborator

@maspypy maspypy commented Jan 17, 2024

#924
つくりました

correct.cpp:定数倍はほとんど気にしていない雑実装ですが、一応計算量オーダーは適切なはず。
正しさについては、correct.cpp 内の test_1, test_2 は確認済なので、大丈夫かなと思っています。

制約: $4000$ としてみました、1.5sec 前後。乱択要素があるからか、 $1$ 次式 $N$ 個の積のランダムケースでも、実行時間が1 ~ 1.5secなどとぶれます。

@maspypy maspypy linked an issue Jan 17, 2024 that may be closed by this pull request

@{lang.ja}
次数 $N$ の多項式 $f(x) \in \mathbb{F}_{998244353}[x]$ が与えられます.
$f(a)=0$ を満たす $a\in \mathbb{F}_{998244353}$ をすべて求めてください.
Copy link
Collaborator

Choose a reason for hiding this comment

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

$a \in \mathbb{Z}\ (0 \leq a < {998244353})$ の方が正確だと思いました( $\mathbb{F}_p$ 上にもともと順序関係はないので )

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ありがとうございます。

ここの問題というよりは、ここと現状の出力パートを合わせた話ですよね。
なんか解が一意の方が分かりやすいかなと思って無理に順序を入れて書き方に悩んでいたんですが、
そもそも順不同で特殊ジャッジにする方が設定も自然だと思いなおしたので、そうしようと思います。

@maspypy
Copy link
Collaborator Author

maspypy commented Jan 21, 2024

というわけでいくつか変更しました。

  • task.md を更新しました
  • checker.cpp を実装しました
  • correct.cpp は適当順序で解を出力します
  • print_sorted_cpp は sort された状態で出力して,wrong = false で通ることを確認しました

@maspypy
Copy link
Collaborator Author

maspypy commented Jan 21, 2024

review ありがとうございます

何かをコピーしてきたときのファイルで使っていないものがあったので削除しました
自分で一通り見直して良さそうだったらマージします

@maspypy maspypy merged commit a8d78f7 into yosupo06:master Jan 21, 2024
6 checks passed
@maspypy maspypy deleted the mod_p_equation branch February 10, 2024 16:31
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.

[問題案] mod p equation
2 participants