OEWG 20110830: Difference between revisions

From GO Wiki
Jump to navigation Jump to search
Line 76: Line 76:
===Next bugs to work on===
===Next bugs to work on===
* [http://sourceforge.net/tracker/?func=detail&aid=3192397&group_id=36855&atid=418260 check def dbxref for non-standard characters when saving]
* [http://sourceforge.net/tracker/?func=detail&aid=3192397&group_id=36855&atid=418260 check def dbxref for non-standard characters when saving]
** To address this, must first fix [http://sourceforge.net/tracker/?func=detail&aid=3398179&group_id=36855&atid=418257 Extended Characters not being caught]
** To address this, must first fix [http://sourceforge.net/tracker/?func=detail&aid=3398179&group_id=36855&atid=418257 Extended Characters not being caught] (see discussion above)
* Fixing any bugs that the JUnit tests reveal
* Fixing any bugs that the JUnit tests reveal



Revision as of 13:42, 29 August 2011

OBO-Edit Working Group Meeting: Tuesday, August 30, 2011, 8:30am PDT

Conference call numbers:
US: 1 866 953 9688
UK: 0808 238 6001
PIN: 801561

Agenda/Chair: Nomi Harris
Minutes:
Attendees:

Bug and Feature Trackers

Fixed since last meeting

  • Fixing JUnit tests for OBO and OBO-Edit
    • Fixed some tests; removed some that Chris M. said were obsolete
    • 2.1-b18 now passes a lot more tests than 2.0
    • 2.0:
      • 21/43 OBO JUnit tests pass
      • 1/24 OBO-Edit JUnit tests pass
    • 2.1-b18:
      • 38/38 OBO JUnit tests pass
      • 10/24 OBO-Edit JUnit tests pass
        • None of the failures are new--all 14 that fail now also failed in 2.0.
        • Trying to fix at least the obo2obo and obodiff tests

Discussion Items

  • New file browser had bug in b17--Heiko quickly fixed it and we released b18 the next day.
  • Did anyone (other than Midori) get a chance to test obo2obo to make sure my fix for [ geneontology-Bugs-3316815 ] (filtered save doesn't save dangling parents) didn't break anything in obo2obo?
    • Midori found one issue in obo2obo but it turned out to be old and unimportant
    • Fixed problem where obo2obo recently stopped printing status messages to stdout.
    • The obo2obo tests I tried yielded the same output for OE 2.0 and 2.1-b18.
  • Amelia asked what the OE policy on repeated words was, because she was getting the warning for strings like "bilirubin-glucuronoside = bilirubin + bilirubin-bisglucuronoside".
    • The policy for repeated words is that you get a warning if there are repeated words unless they are on the list of allowed repeated words (which you can add to yourself) or--theoretically--they are in different sentences (e.g., "Protein plays a role in eye development. Development starts etc"). I say "theoretically" because this is not working right--you still get a warning if you have a period between the repeated words. This is because the imported method for checking whether the second word begins a new sentence (com.swabunga.spell.event.StringWordTokenizer.isNewSentence) doesn't work--it returns false even when the second word does start a new sentence (e.g., "development. Development").
      • Symbols such as + and = don't exempt repeated words from the repeated word warning. It would be possible to add this (probably a few hours of work) if there is agreement on the list of symbols that should exempt repeated words from the warning. (I don't think it would be possible to make "+" and "=" count as words with the current setup, because the tokenizer doesn't treat symbols as words. I'm not sure you'd want that to be the case generally, anyway.)
    • Suggestion: use the Configure buttons in the Verification Manager to turn off tests (such as repeated word or repeated whitespace) that you find annoying rather than informative.
      • After agreement from at least one OEWG member, Nomi set the repeated whitespace test (which complained about two spaces between sentences, for example) to default to "Never" (users can turn it on with the Configure interface).
    • See related bug report verification checks--repeated words, sentence boundaries
  • Next meeting: we will skip Tuesday, September 6 and meet next on September 20.
  • With a new academic year starting, should we revisit the meeting day/time or just keep it as it is?

Currently working on

  • Working towards making an official 2.1 release
    • Mike Cherry wrote, "For the production site I'm still using OBO-EDIT 2.1-beta1. The OBO2OBO script is run nightly to convert an OBO v1.0 file from the OBO v1.2 and to make an OBO diff file. If we should be using a newer version of OBO-EDIT please let me know which one is best. The need on the production site is limited to obo2obo and obodiff.

To determine if the script failed I consult the returned status and I don't use the output to STDERR."

  • Is_a closure broken
    • Midori commented, "I can confirm that I get the same thing (or at least something similar) happening with 'regulates' relationships: is_a closure saves some terms that would only be there if it followed at least one 'regulates' link.
    • Chris said "This has become a lower priority for OE, since we will be able to do most of this in Oort." Should I stop working on this?
  • Extended Characters not being caught
    • There's a Text Editing configuration option "Allow extended characters". In 2.00, OBO-Edit complained about extended characters (e.g., in "Perú") whether or not this option was checked. Then in one of the early versions of 2.1, this reversed: Perú was silently accepted regardless of the setting of "Allow extended characters". I'm trying to figure out what caused this change.
    • I do want to clarify what we want to consider as "extended characters". The OBO-Edit code (both 2.00 and 2.1) brands characters "extended" if Character.isISOControl returns true. The definition of ISO control character used by that method is: "A character is considered to be an ISO control character if its code is in the range '\u0000' through '\u001F' or in the range '\u007F' through '\u009F'."
      • Shouldn't we allow diacritical markings (e.g., "Perú") in the definition and comments? (Maybe not in dbxref, though)

Next bugs to work on

Discussion

Links

Back to OEWG meeting agenda and minutes list