Skip to content

Commit

Permalink
Fix some issue with relation updates
Browse files Browse the repository at this point in the history
  • Loading branch information
jmccrae committed Jan 17, 2025
1 parent 43be7bb commit 23ab580
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 5 deletions.
4 changes: 2 additions & 2 deletions src/automaton.rs
Original file line number Diff line number Diff line change
Expand Up @@ -126,14 +126,14 @@ pub fn apply_automaton(actions : Vec<Action>, wn : &mut Lexicon,
change_manager::insert_sense_relation(wn,
sense.clone(),
SenseRelType::from(&relation)
.ok_or(format!("Bad relation {}", relation))?,
.ok_or(format!("Bad relation {}.", relation))?,
target_sense, changes);
},
None => {
change_manager::insert_rel(wn,
&source,
&SynsetRelType::from(&relation)
.ok_or(format!("Bad relation {}", relation))?,
.ok_or(format!("Bad relation {}.", relation))?,
&target, changes);
}
}
Expand Down
6 changes: 5 additions & 1 deletion src/rels.rs
Original file line number Diff line number Diff line change
Expand Up @@ -514,7 +514,9 @@ impl SenseRelType {
"domain_region" => Some(SenseRelType::DomainRegion),
"has_domain_region" => Some(SenseRelType::HasDomainRegion),
"exemplifies" => Some(SenseRelType::Exemplifies),
"exemplifies_sense" => Some(SenseRelType::Exemplifies),
"is_exemplified_by" => Some(SenseRelType::IsExemplifiedBy),
"is_exemplified_by_sense" => Some(SenseRelType::IsExemplifiedBy),
"similar" => Some(SenseRelType::Similar),
"other" => Some(SenseRelType::Other),
"agent" => Some(SenseRelType::Agent),
Expand All @@ -531,7 +533,9 @@ impl SenseRelType {
"destination" => Some(SenseRelType::Destination),
"body_part" => Some(SenseRelType::BodyPart),
"vehicle" => Some(SenseRelType::Vehicle),
_ => None
_ => {
None
}
}
}

Expand Down
4 changes: 2 additions & 2 deletions src/wordnet.rs
Original file line number Diff line number Diff line change
Expand Up @@ -476,8 +476,8 @@ impl Lexicon {
None => {}
}
} else {
match self.synset_by_id_mut(source) {
Some(ss) => ss.insert_rel(&rel, target),
match self.synset_by_id_mut(target) {
Some(ss) => ss.insert_rel(&rel, source),
None => {}
}
}
Expand Down

0 comments on commit 23ab580

Please sign in to comment.