Thread: [brlcad-tracker] [ brlcad-Bugs-3197208 ] mged's red won't save combination edits
Open Source Solid Modeling CAD
Brought to you by:
brlcad
From: SourceForge.net <no...@so...> - 2011-03-02 15:42:27
|
Bugs item #3197208, was opened at 2011-03-02 06:11 Message generated for change (Comment added) made by tbrowder2 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Nobody/Anonymous (nobody) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- >Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 09:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-03-02 12:11:42
|
Bugs item #3197208, was opened at 2011-03-02 06:11 Message generated for change (Tracker Item Submitted) made by tbrowder2 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Nobody/Anonymous (nobody) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-03-02 18:23:20
|
Bugs item #3197208, was opened at 2011-03-02 06:11 Message generated for change (Comment added) made by tbrowder2 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Open Resolution: None Priority: 8 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Nobody/Anonymous (nobody) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- >Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 12:23 Message: No, I was trying to subtract an inside object from its "parent". The combination editor works, but not red with either of vi or gedit. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-02 11:02 Message: That still sounds concerning, though, because I don't see any changes that have occurred to ted or red since 7.18.2 ... The one limitation noticed is that those commands do not presently support changing the object name during the text edit. Is that perhaps what you were doing? ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 09:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-03-02 17:02:26
|
Bugs item #3197208, was opened at 2011-03-02 12:11 Message generated for change (Comment added) made by brlcad You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Open Resolution: None >Priority: 8 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Nobody/Anonymous (nobody) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- >Comment By: Sean Morrison (brlcad) Date: 2011-03-02 17:02 Message: That still sounds concerning, though, because I don't see any changes that have occurred to ted or red since 7.18.2 ... The one limitation noticed is that those commands do not presently support changing the object name during the text edit. Is that perhaps what you were doing? ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 15:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-03-02 18:25:20
|
Bugs item #3197208, was opened at 2011-03-02 06:11 Message generated for change (Comment added) made by tbrowder2 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Open Resolution: None Priority: 8 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Nobody/Anonymous (nobody) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- >Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 12:25 Message: To duplicate, create a new TGM, a single rpp solid, and make it a region. Then make an inside solid of the first solid (ahem, primitive object). Use red to subtract the inside object--doesn't take. Use the combination editor--it takes. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 12:23 Message: No, I was trying to subtract an inside object from its "parent". The combination editor works, but not red with either of vi or gedit. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-02 11:02 Message: That still sounds concerning, though, because I don't see any changes that have occurred to ted or red since 7.18.2 ... The one limitation noticed is that those commands do not presently support changing the object name during the text edit. Is that perhaps what you were doing? ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 09:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-03-03 20:46:38
|
Bugs item #3197208, was opened at 2011-03-02 07:11 Message generated for change (Comment added) made by starseeker You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None >Status: Closed >Resolution: Fixed Priority: 8 Private: No Submitted By: Tom Browder (tbrowder2) >Assigned to: Cliff Yapp (starseeker) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- >Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 15:46 Message: This behavior appeared when compiling against a system regex (i.e. not using the local regex in src/other) and was being caused by interference from the Tcl copy of regex. Commit 43259 renamed the Tcl copy and allowed proper use of the system regex, which is why recent trunk checkouts worked. ---------------------------------------------------------------------- Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 15:46 Message: This issue has been resolved in the latest (unreleased) version of the source code. Thank you for bringing the issue to our attention! You're welcome and encouraged to test and make sure this issue is resolved after the next source release of BRL-CAD. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 13:25 Message: To duplicate, create a new TGM, a single rpp solid, and make it a region. Then make an inside solid of the first solid (ahem, primitive object). Use red to subtract the inside object--doesn't take. Use the combination editor--it takes. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 13:23 Message: No, I was trying to subtract an inside object from its "parent". The combination editor works, but not red with either of vi or gedit. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-02 12:02 Message: That still sounds concerning, though, because I don't see any changes that have occurred to ted or red since 7.18.2 ... The one limitation noticed is that those commands do not presently support changing the object name during the text edit. Is that perhaps what you were doing? ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 10:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-03-04 06:06:03
|
Bugs item #3197208, was opened at 2011-03-02 12:11 Message generated for change (Comment added) made by brlcad You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Closed Resolution: Fixed Priority: 8 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Cliff Yapp (starseeker) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- >Comment By: Sean Morrison (brlcad) Date: 2011-03-04 06:06 Message: To get even more specific for anyone searching for cause/solutions to similar problems, the actual problem was the compiler parsing against the tcl regex.h instead of the system header, but then using the system library. That normally would be innocuous as the regex API hasn't really changed much in the last decade, but there are #define magic numbers that differ from implementation to implementation. In this particular case, REG_NEWLINE was being used in an expression which was 300 in one version of regex.h and 10 in the other implementation, causing a runtime failure given it was compiled with the wrong one. ---------------------------------------------------------------------- Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 20:46 Message: This behavior appeared when compiling against a system regex (i.e. not using the local regex in src/other) and was being caused by interference from the Tcl copy of regex. Commit 43259 renamed the Tcl copy and allowed proper use of the system regex, which is why recent trunk checkouts worked. ---------------------------------------------------------------------- Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 20:46 Message: This issue has been resolved in the latest (unreleased) version of the source code. Thank you for bringing the issue to our attention! You're welcome and encouraged to test and make sure this issue is resolved after the next source release of BRL-CAD. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 18:25 Message: To duplicate, create a new TGM, a single rpp solid, and make it a region. Then make an inside solid of the first solid (ahem, primitive object). Use red to subtract the inside object--doesn't take. Use the combination editor--it takes. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 18:23 Message: No, I was trying to subtract an inside object from its "parent". The combination editor works, but not red with either of vi or gedit. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-02 17:02 Message: That still sounds concerning, though, because I don't see any changes that have occurred to ted or red since 7.18.2 ... The one limitation noticed is that those commands do not presently support changing the object name during the text edit. Is that perhaps what you were doing? ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 15:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-08-16 16:33:24
|
Bugs item #3197208, was opened at 2011-03-02 06:11 Message generated for change (Comment added) made by tbrowder2 You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Closed Resolution: Fixed Priority: 8 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Cliff Yapp (starseeker) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-08-16 11:33 Message: I'm using BRL-CAD version 7.20.2 and get this warning when using red: WARNING: The 'red' command is incomplete and may inadvertently change model data. Make backups of your data. Consider alternative edit methods. I think that the user should not be allowed to use red at all until it has been thoroughly tested, or at least make the user confirm that he really wants to use it after seeing the warning. I can attest that it still has problems, but I don't have any more specifics at the moment. I''l try to get an example later. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-04 00:06 Message: To get even more specific for anyone searching for cause/solutions to similar problems, the actual problem was the compiler parsing against the tcl regex.h instead of the system header, but then using the system library. That normally would be innocuous as the regex API hasn't really changed much in the last decade, but there are #define magic numbers that differ from implementation to implementation. In this particular case, REG_NEWLINE was being used in an expression which was 300 in one version of regex.h and 10 in the other implementation, causing a runtime failure given it was compiled with the wrong one. ---------------------------------------------------------------------- Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 14:46 Message: This behavior appeared when compiling against a system regex (i.e. not using the local regex in src/other) and was being caused by interference from the Tcl copy of regex. Commit 43259 renamed the Tcl copy and allowed proper use of the system regex, which is why recent trunk checkouts worked. ---------------------------------------------------------------------- Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 14:46 Message: This issue has been resolved in the latest (unreleased) version of the source code. Thank you for bringing the issue to our attention! You're welcome and encouraged to test and make sure this issue is resolved after the next source release of BRL-CAD. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 12:25 Message: To duplicate, create a new TGM, a single rpp solid, and make it a region. Then make an inside solid of the first solid (ahem, primitive object). Use red to subtract the inside object--doesn't take. Use the combination editor--it takes. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 12:23 Message: No, I was trying to subtract an inside object from its "parent". The combination editor works, but not red with either of vi or gedit. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-02 11:02 Message: That still sounds concerning, though, because I don't see any changes that have occurred to ted or red since 7.18.2 ... The one limitation noticed is that those commands do not presently support changing the object name during the text edit. Is that perhaps what you were doing? ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 09:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |
From: SourceForge.net <no...@so...> - 2011-08-17 02:54:21
|
Bugs item #3197208, was opened at 2011-03-02 12:11 Message generated for change (Comment added) made by brlcad You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Geometry Editing Group: None Status: Closed Resolution: Fixed Priority: 8 Private: No Submitted By: Tom Browder (tbrowder2) Assigned to: Cliff Yapp (starseeker) Summary: mged's red won't save combination edits Initial Comment: Using BRL-CAD 7.18.2, Ubuntu 10.10, 64-bit, and using either vi or gedit for the editor, changes to a region or a group appear to be saved (i.e., no warnings or errors from the editor or mged), but the changes aren't there after exiting the editor. ---------------------------------------------------------------------- >Comment By: Sean Morrison (brlcad) Date: 2011-08-17 02:54 Message: Tom, the warning message was added after this and other bug reports were received but the message is actually out of date now (and should be removed). A lot of effort went into fixing up 'red' for 7.20.0, though, including the addition of a slew of tests. Those tests are now integrated as part of our release and regression testing. They poke and prod 'red' for proper command behavior including a lot of bizarre "user error" cases. It's now passing all of our tests including some independent manual testing, so if you still are encountering problems please do post a new, more specific report.. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-08-16 16:33 Message: I'm using BRL-CAD version 7.20.2 and get this warning when using red: WARNING: The 'red' command is incomplete and may inadvertently change model data. Make backups of your data. Consider alternative edit methods. I think that the user should not be allowed to use red at all until it has been thoroughly tested, or at least make the user confirm that he really wants to use it after seeing the warning. I can attest that it still has problems, but I don't have any more specifics at the moment. I''l try to get an example later. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-04 06:06 Message: To get even more specific for anyone searching for cause/solutions to similar problems, the actual problem was the compiler parsing against the tcl regex.h instead of the system header, but then using the system library. That normally would be innocuous as the regex API hasn't really changed much in the last decade, but there are #define magic numbers that differ from implementation to implementation. In this particular case, REG_NEWLINE was being used in an expression which was 300 in one version of regex.h and 10 in the other implementation, causing a runtime failure given it was compiled with the wrong one. ---------------------------------------------------------------------- Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 20:46 Message: This behavior appeared when compiling against a system regex (i.e. not using the local regex in src/other) and was being caused by interference from the Tcl copy of regex. Commit 43259 renamed the Tcl copy and allowed proper use of the system regex, which is why recent trunk checkouts worked. ---------------------------------------------------------------------- Comment By: Cliff Yapp (starseeker) Date: 2011-03-03 20:46 Message: This issue has been resolved in the latest (unreleased) version of the source code. Thank you for bringing the issue to our attention! You're welcome and encouraged to test and make sure this issue is resolved after the next source release of BRL-CAD. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 18:25 Message: To duplicate, create a new TGM, a single rpp solid, and make it a region. Then make an inside solid of the first solid (ahem, primitive object). Use red to subtract the inside object--doesn't take. Use the combination editor--it takes. ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 18:23 Message: No, I was trying to subtract an inside object from its "parent". The combination editor works, but not red with either of vi or gedit. ---------------------------------------------------------------------- Comment By: Sean Morrison (brlcad) Date: 2011-03-02 17:02 Message: That still sounds concerning, though, because I don't see any changes that have occurred to ted or red since 7.18.2 ... The one limitation noticed is that those commands do not presently support changing the object name during the text edit. Is that perhaps what you were doing? ---------------------------------------------------------------------- Comment By: Tom Browder (tbrowder2) Date: 2011-03-02 15:42 Message: The bug is in version 7.18.2, but on the trunk as of revision 43629 it works okay. I will consider this bug closed when the next release occurs and mged/red works then, too. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=640802&aid=3197208&group_id=105292 |