Ticket #65 (closed defect: fixed)

Opened 1 year ago

Last modified 3 months ago

edge reader problem

Reported by: huangb@IU.EDU Assigned to: Michael Zhang
Priority: highest Milestone:
Component: DataConverter Version: 0.8.x
Severity: minor Keywords:
Cc:

Description (Last modified by huangb@IU.EDU)

If specify "directed" in the first line of .edge file, the edge reader report an error if there's a space in front of "directed"

If specify "undirected" in the first line of .edge file, the edge reader report an error if there's a space in front of "undirected".

Problem is at edu.iu.nwb.converter.edgelist.EdgeListValidatorFactory?.java

Change History

(follow-up: ↓ 2 ) 07/16/07 22:23:51 changed by anonymous

We're requiring people to edit the file to change it from directed to undirected?

I think a better solution would be to make it so load . . . algorithms pop up a gui if so desired (heck, they might already, I don't know), and give a checkbox.

As for the space, that's really up to us. Since we're defining the mini-format, its either an legitimate error to put a space there or something we can accomodate.

(in reply to: ↑ 1 ) 07/17/07 09:06:32 changed by rduhon@IU.EDU

Replying to anonymous:

We're requiring people to edit the file to change it from directed to undirected? I think a better solution would be to make it so load . . . algorithms pop up a gui if so desired (heck, they might already, I don't know), and give a checkbox. As for the space, that's really up to us. Since we're defining the mini-format, its either an legitimate error to put a space there or something we can accomodate.

This was me (Russell).

10/04/07 17:44:40 changed by huangb@IU.EDU

  • version changed from 0.7.x to 0.8.x.
  • milestone changed from 0.7 to 0.8.

10/16/07 13:17:12 changed by huangb@IU.EDU

  • owner changed from huangb@indiana.edu to Michael Zhang.
  • description changed.
  • severity changed from major to minor.

08/13/08 12:03:08 changed by mwlinnem@IU.EDU

  • status changed from new to closed.
  • resolution set to fixed.

Whitespace is now legal before or after "undirected" and "directed". Change Wiki documentation to reflect this.

08/13/08 12:03:29 changed by mwlinnem@IU.EDU

*Changed

I already changed it.