Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -451,7 +451,7 @@ impl OnTheFlyRuntimeMetadata {
|| component != expected_component
|| projection.runtime_name != *runtime_name
|| parameter.name != format!("{runtime_name}.{component}")
|| (previous_runtime_name == Some(runtime_name)
|| (previous_runtime_name == Some(runtime_name.as_str())
&& previous_kind != Some(parameter.kind.as_str()))
{
return Err(RusticolError::integrity(
Expand Down
8 changes: 7 additions & 1 deletion rust/crates/rusticol-core/src/engine/physics.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1172,7 +1172,13 @@ pub(super) fn certifies_lc_direct_total_source(
.iter()
.map(|group| group.id)
.collect::<BTreeSet<_>>();
if raw_group_ids != physics.reduction_by_group_id.keys().copied().collect() {
if raw_group_ids
!= physics
.reduction_by_group_id
.keys()
.copied()
.collect::<BTreeSet<_>>()
{
return false;
}
let mut covered_members = BTreeSet::new();
Expand Down
4 changes: 2 additions & 2 deletions rust/crates/rusticol-core/src/engine/runtime_load.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2026,7 +2026,7 @@ fn build_helicity_recurrence_runtime_from_view(
!= represented_complete_domains
.union(&structural_zero_domains)
.copied()
.collect()
.collect::<BTreeSet<_>>()
{
return Err(RusticolError::artifact(
"helicity amplitude and structural-zero domains do not cover every complete selector domain",
Expand Down Expand Up @@ -2450,7 +2450,7 @@ fn validate_helicity_materialized_source_routes(
factor: route.factor,
});
}
if routed_source_ids != schema.sources.keys().copied().collect() {
if routed_source_ids != schema.sources.keys().copied().collect::<BTreeSet<_>>() {
return Err(RusticolError::artifact(
"helicity materialized source routes do not cover every final runtime source",
));
Expand Down
4 changes: 2 additions & 2 deletions rust/crates/rusticol-core/src/loop_runtime.rs
Original file line number Diff line number Diff line change
Expand Up @@ -175,7 +175,7 @@ impl RectangularDualWeights {
}
let requested: BTreeSet<_> = columns.iter().map(|c| [c[0], c[1]]).collect();
require(
requested == self.banks.iter().copied().collect(),
requested == self.banks.iter().copied().collect::<BTreeSet<_>>(),
"rectangular banks differ from weight direction/sector keys",
)?;
let mut covered = BTreeSet::new();
Expand Down Expand Up @@ -1191,7 +1191,7 @@ impl LoopRuntimePlan {
&& ids.insert(&member.process_id)
&& member.external_pdgs.len() == legs
&& map.len() == member.species.len()
&& targets == map.keys().copied().collect()
&& targets == map.keys().copied().collect::<BTreeSet<_>>()
&& map.iter().all(|(&a, &b)| {
a != 0
&& a != b
Expand Down
2 changes: 1 addition & 1 deletion rust/crates/rusticol-core/src/metadata.rs
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@ impl ProcessPhysics {
)));
}
}
if momentum_slots != (0..self.external_particles.len()).collect() {
if momentum_slots != (0..self.external_particles.len()).collect::<BTreeSet<_>>() {
return Err(RusticolError::artifact(
"external particle momentum slots must form a complete zero-based permutation",
));
Expand Down
7 changes: 6 additions & 1 deletion rust/crates/rusticol-core/src/recurrence/process.rs
Original file line number Diff line number Diff line change
Expand Up @@ -3132,7 +3132,12 @@ impl<'a> RecurrenceProcessInputView<'a> {
.iter()
.map(|row| row.source_slot)
.collect::<BTreeSet<_>>();
if endpoint_slots != expected_orientations.keys().copied().collect() {
if endpoint_slots
!= expected_orientations
.keys()
.copied()
.collect::<BTreeSet<_>>()
{
return Err(invalid(
"fermion-pairing endpoints do not cover the LC open-line endpoints",
));
Expand Down
Loading