What requires a HOLD vote on the Parts Tracker - Printable Version +- LDraw.org Discussion Forums (https://forums.ldraw.org) +-- Forum: General (https://forums.ldraw.org/forum-12.html) +--- Forum: Parts Tracker Discussion (https://forums.ldraw.org/forum-36.html) +--- Thread: What requires a HOLD vote on the Parts Tracker (/thread-22605.html) Pages:
1
2
|
RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-01-07 (2018-01-04, 18:40)Orion Pobursky Wrote:(2018-01-04, 18:17)Willy Tschager Wrote: Your turn: 18 Good catch. On the other hand we have a: !HELP (Authors should constrain themselves to a 50-character limit to keep the text readable.) which gives a good indication that we should keep an eye on for example: 11568p01.dat - Animal Squirrel with Black Nose, Mouth and Eyes with Eyelashes and Reddish Brown Face Decorations Pattern u9310c02.dat - Fabuland House Block with Blue Door and 2 Blue Round Windows with One Semicircular and Three Sextant Panes Panes (Complete) w. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-01-10 (2018-01-04, 18:17)Willy Tschager Wrote: Your turn: 16 Autocorrect 17 Prompt 18 Prompt 19 Autocorrect 20 Prompt w. RE: What requires a HOLD vote on the Parts Tracker - Philippe Hurbain - 2018-01-10 Quote:19 WARNING Maybe leading spaces in part description.This is a complex one. DH takes care of trying to keep part length sorting OK, adding a double space in front of length numbers with only one digit in integer part, removing double space everywhere otherwise. But sometimes we don't want that (because the number is not actually a length) so user needs some control. RE: What requires a HOLD vote on the Parts Tracker - Philippe Hurbain - 2018-01-10 Quote:And regarding potential PT integration: should the PT try to prompt anything or just reject the part if such problems exist and call it a day? It needs rework anyway at that point so IMO it can't be ready for PT in that case.Just to make sure: any certification tool must report ALL the issues it finds and can't auto-correct, and not stop after finding the first error... RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-01-30 Next one: 21 ERROR Some keywords are also in part description. HOLD 22 ERROR Filename does not matches filetitle. HOLD 23 ERROR Author real name is not set. 24 ERROR Author user name is not set. 25 ERROR BFC is not set. HOLD RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-01-30 (2018-01-30, 19:59)Willy Tschager Wrote: Next one: 21 http://www.ldraw.org/article/340.html#keywords Strip the word appearing or number in the description from the keywords 22 Silly me! I cannot fine the related spec. 23 http://www.ldraw.org/article/398.html#Syntax Prompt 24 http://www.ldraw.org/article/398.html#Syntax Prompt 25 http://www.ldraw.org/article/415 Prompt RE: What requires a HOLD vote on the Parts Tracker - Orion Pobursky - 2018-01-30 (2018-01-30, 19:59)Willy Tschager Wrote: Next one: 21 Should not be holdable, change the spec 22 Reject submission to PT 23,24 These are required for licencing information, reject submission to PT 25 BFC required for all new parts, reject submission PT RE: What requires a HOLD vote on the Parts Tracker - Santeri Piippo - 2018-01-31 I have now gotten a proof-of-concept part checker up: Tests currently implemented:
The quadrilateral checks can create redundant errors, e.g. a report for concave quad when it's actually a bowtie, because technically bowtie quads are concave and there's separate tests making these results. No header tests yet because the header currently is not parsed at all. No corrections performed yet either. Also includes a test for very thin polygons (smallest angle less than 0.5°). While they are not considered a problem, they're reported as a purely informational notice since they can point to other problems (e.g. gaps). RE: What requires a HOLD vote on the Parts Tracker - Chris Dee - 2018-01-31 (2018-01-31, 13:18)Santeri Piippo Wrote: I have now gotten a proof-of-concept part checker up: Pllease don't spend time on adding header checks. I have a header checking script that I use during the Parts Update build and this will soon be incoporated into the submit process. I'll also review this thread to see if other header checks need to be added. RE: What requires a HOLD vote on the Parts Tracker - Philippe Hurbain - 2018-01-31 (2018-01-31, 13:18)Santeri Piippo Wrote: I have now gotten a proof-of-concept part checker upretty nice so far! and damn fastOne little request, even during debugging phase: please display the name of file just processed, it allows find again the file tested to check results againt other tools (easy to get lost in my huge collection of old files!) I noticed that angles for coplanarity checks are consistently a bit lower than values reported by LDDP or planarcheck. Not annoying here, but might hide a more serious problem? Quote:Also includes a test for very thin polygons (smallest angle less than 0.5°). While they are not considered a problem, they're reported as a purely informational notice since they can point to other problems (e.g. gaps).Min/max colinear theresholds are specified here: http://www.ldraw.org/article/512.html#colinear. Might be interesting to display a warning for lower/higher values... RE: What requires a HOLD vote on the Parts Tracker - Santeri Piippo - 2018-01-31 (2018-01-31, 14:18)Chris Dee Wrote: Pllease don't spend time on adding header checks. I have a header checking script that I use during the Parts Update build and this will soon be incoporated into the submit process. I'll also review this thread to see if other header checks need to be added.Interesting. I guess I'll dedicate to the body checks then. (2018-01-31, 14:41)Philippe Hurbain Wrote: Pretty nice so far! and damn fastThanks! Quote:One little request, even during debugging phase: please display the name of file just processed, it allows find again the file tested to check results againt other tools (easy to get lost in my huge collection of old files!)The commandline tool does that but for the web front I didn't think it was so important yet. But you make a point and I'll add it. Quote:I noticed that angles for coplanarity checks are consistently a bit lower than values reported by LDDP or planarcheck. Not annoying here, but might hide a more serious problem?Coplanarity is checked by splitting the quad and computing the skew angle from the angle between the two plane normals. Both triangle configurations are tested and the first one to yield a problem is reported. Come to think of it, it needs to test both and use the larger angle. Quote:Min/max colinear theresholds are specified here: http://www.ldraw.org/article/512.html#colinear. Might be interesting to display a warning for lower/higher values...I think I need to rewrite the concave and colinear tests so that I can use these angle thresholds. Right now I'm using vector math. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-02-06 25 ERROR BFC is not set. HOLD 26 ERROR License is not set. HOLD 27 ERROR Part type is not set. HOLD 28 ERROR Primitives needs to have CCW winding. HOLD 29 ERROR Use of !KEYWORDS is not allowed for this part type. HOLD 30 ERROR Part description contains Tab character. HOLD RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-02-06 (2018-02-06, 9:57)Willy Tschager Wrote: 25 ERROR BFC is not set. HOLD 25 http://www.ldraw.org/article/415 Prompt 26 http://www.ldraw.org/article/398.html Set automatically to !LICENSE Redistributable under CCAL version 2.0 : see CAreadme.txt 27 http://www.ldraw.org/article/398.html Prompt 28 http://www.ldraw.org/article/415.html#library Prompt 29 http://www.ldraw.org/article/340.html#scope Remove keywords 30 http://www.ldraw.org/article/218.html#files Prompt RE: What requires a HOLD vote on the Parts Tracker - Philippe Hurbain - 2018-02-06 Yes. 28- could be auto corrected: if primitive is CW, change type to CCW and invert winding of triangles and quads. Probably a moot point as most primitives are done with primgen, so no need to lose time on corrector... RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-02-20 31 ERROR HELP entry length. 32 ERROR Word 'new' or 'old' is used in part description. 33 ERROR Use of (Needs work). HOLD 34 ERROR Entry for !CATEGORY. 35 ERROR Use of !CATEGORY is not allowed at for this part type. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-02-24 (2018-02-20, 11:07)Willy Tschager Wrote: 31 ERROR HELP entry length. 31 http://www.ldraw.org/article/398.html#Syntax Prompt 32 ??? 33 http://www.ldraw.org/article/398.html This is clearly wrong as (Needs work) is allowed. 34 ??? 35 http://www.ldraw.org/article/340 Prompt for primitives Mike! Please help. w. RE: What requires a HOLD vote on the Parts Tracker - Michael Heidemann - 2018-02-24 (2018-02-24, 8:54)Willy Tschager Wrote:(2018-02-20, 11:07)Willy Tschager Wrote: 31 ERROR HELP entry length. I just answer right our of my mind: 31: - what do you mean with "Prompt" 32: These words are time based (now) and shall be not used - see several discussions in the old forum 33: clearly correct, as you have to describe what needs work in the comments 34: There is a problem with the !Category entry. - i need to file to see what causes this error. 35: - what do you mean with "Prompt" - primitives are not allowed to carry a !Category entry. If I remember correctly this is all written in the documentation of DATHeader. Currently I do not have the time to look into it. Sorry. cu Mikeheide - I made a wrong numbering, now fixed - RE: What requires a HOLD vote on the Parts Tracker - Orion Pobursky - 2018-02-26 (2018-02-24, 11:13)Michael Heidemann Wrote:(2018-02-24, 8:54)Willy Tschager Wrote: 31 http://www.ldraw.org/article/398.html#Syntax Prompt Based on Mike's feedback: 33: Not an error 34: Style based, not an error 35: Not an error 36: If it's for a !CATEGORY without an entry or a category that is not on the master list then hold 37: Hold for primitives and subparts RE: What requires a HOLD vote on the Parts Tracker - Michael Heidemann - 2018-02-26 (2018-02-20, 11:07)Willy Tschager Wrote: 31 ERROR HELP entry length. Now I have my code on hand and will give a more detailed information of the meaning of this short error messages. 31 - The entry in the !HELP section exceeds the length of 90 characters in a single line. The 90 characters are implemented after the 50 character limit seemed to the people too short. 32 - If we describe a parts shape as "new" it will be in maybe one year "old". So type1 or similar will work much better. 33 - (Needs work) is allowed _if_ there is a comment that explains what needs to be done. If there is no comment this error will rise. 34 - The !CATEGORY entry has to be from the list of categories that is available on LDraw.org. If this is not the case this error will rise. 35 - Part types are (Part|Subpart|Primitive|48_Primitive|Shortcut) and for some the use of !CATEGORY is not allowed. If used for the wrong part type this error will rise. I hope this will help. Mikeheide RE: What requires a HOLD vote on the Parts Tracker - Michael Heidemann - 2018-02-27 (2017-12-08, 20:51)Willy Tschager Wrote: Officially we have only the outdated L3P error list:I just like to mention here that most of the errors that are marked with HOLD are confirmed with our PT Admin on 08.11.2009. The original request for adding this HOLD vote hint came from Willy Tschager. But I am open for changes if they are useful. cu mikeheide RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2018-03-01 Mike, just read above. We are using Datheader's error list to understand what an online version on the PT should looking for. w. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2019-05-27 While in Billund we worked on Datheader's HOLD list an came up with this: LDraw HOLD Reasons.pdf (Size: 32.25 KB / Downloads: 25) The process was working through the list and compare it with the current specs as well as how things are currently handled on the PT. As you can see there is just one point left for discussion. The rest of Orion list: Quote:Things that should generate a warning but not a HOLD: is more a "political" thing. How do we handle the parts with the above errors that clearly need a (Needs work) in the description, when the original author has lost interest in the part. If it is on HOLD for more than three months, otherwise you would have to ping the PT admin to add the note to the description. I do not think that a simple comment addresses the above problems properly. Would there be a chance to add a WARNING to the PT that stands somehow between HOLD and Comment? w. RE: What requires a HOLD vote on the Parts Tracker - Chris Dee - 2019-05-27 (2019-05-27, 10:29)Willy Tschager Wrote: While in Billund we worked on Datheader's HOLD list an came up with this: How do you envisage a WARNING vote affecting the certification status? To override the reviewers existing CERT vote, like a Comment, or should it add another vote letter that nees to be cleared before the part can be released? The latter would be complex to implement but could potentially be rolled into a much-needed Parts Tracker redevelopment. Alternatively, and much simpler to implement, could it work like a comment but automatically email the PT admin for atttention. In which case merely adding a 'Send comment to admin' checkbox to the review page would be even easier to implement and of more general applicability as it could be used on other vote types (e.g. HOLD votes to request deletions, CERT votes to warn of dependencies). RE: What requires a HOLD vote on the Parts Tracker - Orion Pobursky - 2019-05-27 (2019-05-27, 11:04)Chris Dee Wrote: How do you envisage a WARNING vote affecting the certification status? To override the reviewers existing CERT vote, like a Comment, or should it add another vote letter that nees to be cleared before the part can be released? The latter would be complex to implement but could potentially be rolled into a much-needed Parts Tracker redevelopment. I see WARNING as a comment on a HOLD, CERTIFY, or NOVOTE. These aren't necessarily errors but may be poor style, minor improvements that don't really need to be fixed, or stuff that the admin can fix. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2019-05-27 (2019-05-27, 11:04)Chris Dee Wrote: How do you envisage a WARNING vote affecting the certification status? To override the reviewers existing CERT vote, like a Comment, or should it add another vote letter that nees to be cleared before the part can be released? The latter would be complex to implement but could potentially be rolled into a much-needed Parts Tracker redevelopment. I envision this a stronger comment. You could certify a part with minor flaws and add a WARNING in addition to make the PT admin aware that it might needs a (Needs Work) in description and comment if it hasn't already been added by the author just willing to add the (Needs Work) without fixing the error. Looking at the parts you can find many with very minor flaws where the reviewer preferred comment on, but not certify. The WARNING should clear this hesitation. w. RE: What requires a HOLD vote on the Parts Tracker - Philippe Hurbain - 2019-05-28 (2019-05-27, 18:26)Willy Tschager Wrote: Looking at the parts you can find many with very minor flaws where the reviewer preferred comment on, but not certify.Yes, considering the scarcity of reviewers, this kind of novote is actually close to a hold... RE: What requires a HOLD vote on the Parts Tracker - Ulrich Röder - 2019-05-28 .... for my part ... I look at the whole thing more like Magnus .. ... his "Swedish Hammer" ... met me here in my beginnings more often than I liked ... funny meant ... but that was also a good intensive school .. ... it may be that this scares off many younger aspirants ... but those who develop beyond that ... will become good authors too ... And yes ... a review needs a lot of time..to create parts, however, even more time ... accordingly. "Quality over Quantity" ... also shapes the mindset of new generation generations of "LDrawer's" ... wants to be called ... and thus also the future of LDraw. and right there ... we should ... set the lever ... ... well made and as simple as imaginable made tutorials ... summarized like a textbook ..... best still imaged .. and integrated links .... as a loadable PDF file .... .... well and manageable structured .. and updated and supplemented again and again ..... to make it easier for the beginners. The international popularity of Ldraw ... could be underlined by translations of these Tuitorials in various languages ... ..that would of course be a dynamic group work announced. Willi has already done a lot in this respect on his homepage. RE: What requires a HOLD vote on the Parts Tracker - Ulrich Röder - 2019-05-31 (2017-12-09, 9:00)Philippe Hurbain Wrote: I mostly agree that we are too strict overall... But another thing to factor in is the scarcity of reviewers. If I don't like something in a part I put a novote with comments, but there are so few reviewers, who could overwhelm my opinion, that a hold - somehow forcing the author to react - would perhaps be better... ...I totally agree with your opinion RE: What requires a HOLD vote on the Parts Tracker - Michael Heidemann - 2019-06-08 (2019-05-27, 10:29)Willy Tschager Wrote: While in Billund we worked on Datheader's HOLD list an came up with this:I do not fully understand this pdf file. All the lines mention one error and/or warning that DatHeader will give to the user based on the specs. we made. These errors/warnings are not related to the hold vote on the PT. For some errors I was asked to mention that it shall be a hold vote. So I added in the error message the word "HOLD". I will now remove the word "HOLD" on the lines where not a yellow HOLD is mentioned and add the word if in the file is a yellow HOLD. I use the word "WARNING" instead of "ERROR" if DatHeader is unable to decide if the item is in this contect correct or not. The user has to decide it. Sadly I have no idea what you have in mind if you write "Remove" in the pdf file. Shall I remove the check for this error? cu mikeheide RE: What requires a HOLD vote on the Parts Tracker - Victor Di Rienzo - 2019-06-08 (2019-05-28, 12:48)Ulrich Röder Wrote: .... for my part ... I look at the whole thing more like Magnus .. From my perspective point of view as newbie. When you submit your first parts, it's intimidating the amount of observations that you get and things you need to correct, with out anything pointing out how to fix the issue. Its demoralising and you need to have lot of force to keep on trying to fix your file. May be doing a faq with the typical (most often) errors and how to fix them will help to newbies like me. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2019-06-09 (2019-06-08, 21:16)Michael Heidemann Wrote: For some errors I was asked to mention that it shall be a hold vote. So I added in the error message the word "HOLD". And that is fine. (2019-06-08, 21:16)Michael Heidemann Wrote: I will now remove the word "HOLD" on the lines where not a yellow HOLD is mentioned and add the word if in the file is a yellow HOLD. Fine also with that. (2019-06-08, 21:16)Michael Heidemann Wrote: I use the word "WARNING" instead of "ERROR" if DatHeader is unable to decide if the item is in this contect correct or not. The user has to decide it. Yep. (2019-06-08, 21:16)Michael Heidemann Wrote: Sadly I have no idea what you have in mind if you write "Remove" in the pdf file. Shall I remove the check for this error? That was the idea. Take for example "Maybe wrong color for sticker used." A reviewer has to spot if the colors are right or wrong. If the aren't in the LDConfig it should be a direct color and it is up to the reviewer/author to decide if it was the right pick. Same for "Length of part description" since we completely dropped any length limits. w. RE: What requires a HOLD vote on the Parts Tracker - Magnus Forsberg - 2019-06-09 (2019-06-08, 23:13)Victor Di Rienzo Wrote: Its demoralising and you need to have lot of force to keep on trying to fix your file. As a reviewer and teacher I give critic, not to put you down. I want you to think, figure out what's wrong, and understand why. If you need more info to figure it out, just ask. And I want you to do that. It's my input, to becoming a better reviewer. (2019-06-08, 23:13)Victor Di Rienzo Wrote: May be doing a faq with the typical (most often) errors and how to fix them will help to newbies like me. I agree, we should be better on documenting common errors, and hinting on how to fix them. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2019-06-09 (2019-06-08, 23:13)Victor Di Rienzo Wrote: May be doing a faq with the typical (most often) errors and how to fix them will help to newbies like me. Great idea! Go ahead with your questions and let us know what got you frustrated the most. w. RE: What requires a HOLD vote on the Parts Tracker - Victor Di Rienzo - 2019-06-09 (2019-06-09, 15:32)Willy Tschager Wrote: Great idea! Go ahead with your questions and let us know what got you frustrated the most. I think i have finally learn how to make the patterns and fulfill all the needs that are requested. For me as there its a lot to read and to learn, the learning slope it is hi to get more people envolved . May be with video tutorials explaining all the proceess and adding them to a short guide to read may help too. Also im thinking about cultural differences due most of people here are not native english speakes so reading and comprehension its more complex that a video to start the proccess, then if you are still interested all the documents available will make you undestand the underlying. RE: What requires a HOLD vote on the Parts Tracker - Michael Heidemann - 2019-06-10 (2019-06-09, 7:59)Willy Tschager Wrote: And that is fine.Ok, now I understand your idea when the reason of the checks are gone, no check is needed. cu mikeheide RE: What requires a HOLD vote on the Parts Tracker - Orion Pobursky - 2019-06-16 (2019-05-27, 10:29)Willy Tschager Wrote: While in Billund we worked on Datheader's HOLD list an came up with this: I'd like to update this page: https://www.ldraw.org/library/tracker/ref/l3pmsg/ With the above guidelines. RE: What requires a HOLD vote on the Parts Tracker - Chris Dee - 2019-06-16 (2019-06-16, 3:11)Orion Pobursky Wrote: I'd like to update this page: Go ahead. Could the non-BFC column be removed, as we expect everything tpo be BFC these days? RE: What requires a HOLD vote on the Parts Tracker - Steffen - 2019-06-16 yes please. we have brought the BFC-problematic files to nearly 0 now. just a small handful of "weird relicts" are remaining: https://www.ldraw.org/cgi-bin/ptbfcstatus.cgi I suggest to delete all of them from the PT: - one is a "minifig torso + head" physical shortcut assembly which - if we would add these combinations to the library - would make the library size explode due to the possible combination - the others are mockups which have no visual resemblence with the parts they mimick and are waiting unfinished for years now on the PT. RE: What requires a HOLD vote on the Parts Tracker - Willy Tschager - 2019-06-25 (2019-06-24, 7:34)Santeri Piippo Wrote: [*]"Wrong BFC command found." What is a wrong BFC command?[*] 0 BFC CERTIFY INVERTNEXT w. RE: What requires a HOLD vote on the Parts Tracker - Michael Heidemann - 2019-11-06 (2019-05-27, 10:29)Willy Tschager Wrote: While in Billund we worked on Datheader's HOLD list an came up with this:At least I just uploaded the new version (3.0.23.0) of DATHeader for covering the changes for hold and warnings. In case you will find still errors, please just send me a PM by email, because I not very often in the forum these days. cu Mikeheide RE: What requires a HOLD vote on the Parts Tracker - Philippe Hurbain - 2019-11-07 (2019-11-06, 19:47)Michael Heidemann Wrote: At least I just uploaded the new version (3.0.23.0) of DATHeader for covering the changes for hold and warnings.Thanks Mike! |