-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Enhancement] The current status shows the JSON code #57
Comments
This is what status.json is for. Which it links to towards the top. The status.json is about the statuses not the actual status (except at the top). |
Maybe it could be higher up? But what it might say is beyond me at this time. |
BTW: the top also shows the status.json file. Maybe there is a misunderstanding? |
What do you wish to be done @lcn2 ? I can take care of it if you let me know. Thanks. |
Perhaps the Format section down to the horizontal line could be moved to the FAQ? Then instead of all that Format section text, one could add a single:
Please consider something like that as a potential solution to the enhancement suggestion by @nanochess, @xexyl. |
Okay but which FAQ? If we add an FAQ with all the details of the status.html that's going to be a long FAQ. I would guess I'm not following you right? Please clarify. |
How about either adding it as a new FAQ 10.11 that describes the contest status (i.e., pending, open, judging, closed) Or better still add, a new FAQ Section 12 about "IOCCC status", with a new FAQ 12.0 that describes the contest status (i.e., pending, open, judging, closed) A new section at the bottom of the FAQ might be a better place for this information. |
This is far more complicated than adding an FAQ and updating a markdown file though. See below.
Well that would be better .. assuming that we don't have any links using the number system (which we shouldn't). Personally I don't see the issue here but since it was brought up it's obviously an issue to some. The question is what should remain in status.html too. I guess the top part up through the contents of status.json. A much worse problem is this is not a simple matter of updating status.md! Significant changes to bin/gen-status.sh will have to be done too. Right now I'm hoping to look at the other repo to get some of the things you needed done (and those things are more important as they are required for the next contest). |
This is actually an enhancement that can be improved on either pre or post IOCCC28 opening. The idea raised by @nanochess is valid. The solution, however, should not delay the opening of IOCCC28. |
I thought those things too. Glad you agree. Just put in the other repo some thoughts/questions about an enhancement. I have to do other things now but if I don't do more today I'll certainly tomorrow morning. |
As for when the contest is in open status I won't very likely be working here. It is not clear even what I'll be doing. But I can certainly look into it after IOCCC28 though I do have some plans also. Plus I (and hopefully you can help some) have the new tools (after discussion) in jparse and it would be nice to fix the UTF-8 issue that @SirWumpus reported (and did some great work towards!). Anyway maybe I can get to this in the months following IOCCC28. |
While that code started out as a bit of plod, it turned out to be pretty interesting study. Hope you use it at some point. |
I would highly encourage you, @xexyl, to incorporate that code into the jparse repo, after IOCCC28 opens. Post IOCCC28, we can sync such a change into "the other repo". We still wonder why, assuming that your method works @SirWumpus (and we have found no case where it doesn't), why this isn't the preferred method for processing UTF-8. Thanks again, @SirWumpus On NetBSD wcYour approach, @SirWumpus, to managing non-UTF-8 byte sequences should be the norm for tools such as The NetBSD folks appears to have broken fundamental tools such as We wonder if the folk who damaged See bin/58014: wc no longer works with binary files. Who knows if they will fix the "buggering" of tools like BTW: Unless or until it is fixed, we will consider NetBSD An approach to tools such as UPDATE 0The compiler warning about While not exclusive to BetBSD, there are folk who consider it a mission and a "good feature" to make code that uses Our approach in the calc repo was to introduce modify code that used potentially unsafe functions and then to effectually ban future use in the calc code via banned.h. This is a case where an application makes a choice to move away from the use of a potentially problematic function. If a compiler is going to do that, they should add a UPDATE 1Then there are the folk who want to stop people using sigh |
Okay but you used 'I'. There's a problem there! I won't go to the usual place in 'The Black Gate Is Closed' but I'm tempted to!
The reason is quite simply that I haven't had the time to try and integrate it into the library. I believe it might take a fair amount of work and I'm trying to work on the contest code too. I will certainly be looking at it however. Of course you're more than welcome to help with this task as well
My thanks as well!
... Very interesting commentary that normally I might reply to but trying to work on txzchk. I would be inclined to agree with your points however. |
We use I explicitly because we were not speaking for the IOCCC. The IOCCC judges won't look down on a submission that used |
Oh! I always assumed that you did it because a lot of people do (I sometimes do too as does ironically and amusingly Gollum, as you know). But of course I was just going to the old joke between us anyway.
Of course. And neither would you necessarily look down on an implementation of |
No surprises there and a great compliment to him too (and well deserved)! I still have a lot of work to do with integrating it of course but that can come down the road.
I am reminded of the linux man page for Cant: a funny word that 'auto-"correct"' likes to change to 'can't'. Just like 'wont' (sometimes at least). But as you know the best one is when one changed erroneously 'shellcheck' to ... 'spellcheck'!
Hopefully not.
In many cases documenting it is just a way to say we know of it but we don't care indeed. Of course in other cases it's perfectly fine.
That would be nice wouldn't it? But alas we do not live in an ideal world. But maybe that's a good thing. If we did live in an ideal world would we really be happy? Certainly many people would not be but we also might not bother doing anything at all. That is my belief based on what I have seen in this world.
As for
I'd like that. The warning about use of pointers in some contexts is beyond ludicrous and probably the worst one I have seen. To warn about pointers in C (used as/related to an array) is insane! And the supposed misleading formatting one is also ludicrous. I had to 'fix' code just to shut that warning up years ago as at the time there seemed to be no way to disable it. Ironically by introducing such things it's more likely to cause problems because you have to fix perfectly valid code that might have got a messed up formatting for some reason. The problem is when you warn about dubious things people might actually disable all warnings or ignore them and then they will do something that actually is bad.
That bothered me at first but only because it was used in a cronjob that runs very frequently. I just changed it. It's kind of annoying but not as bad as say systemd (is anything that bad?). |
Glad it was good for you too! Thanks. I hope I can use it too. I guess it'll involve some study as well and also some work to try and integrate it into the parser. That'll be fun! |
Figured you discarded the (royal) we when you weren't talking for the Judge Collective (or when you took your crown off when going to the |
Ah yes. When it comes to humour the bog always works well for those who don't mind such sort of humour. |
Purists probably assume binary will use more efficient Base64, rather than encode arbitrary binary as though it is encoded UTF16. I think I noticed something about UTF8 errors in the C standard wide-character functions |
Okay but aren't all |
Unless you are an Irish peat moss cutter ("bog men") you probably take some offence. |
Depends on if one has a sense of humour I'd say. You could even add to this! |
@lcn2 Glad you like it. I couldn't understand why @xexyl kept saying its really complex. I had to see for myself what the fuss was about and wrote |
You misunderstand. The lower level functions in the parser are complex so I have to work it into that. Your code seems simple enough yes. |
You and I should talk more about that after IOCCC28. If a parser is that complex, then its sounds a little over engineered. I've avoid looking at that code for now cause I might get "re-writers itch". |
Sure .. though that function core I did not write myself. Do feel free to take a look if you wish. Since you wrote that code you know it better. Actually it's a bit of a mix. To fix one bug it involved a couple other UTF-8 functions including one from (I think) the linux kernel (but modified). It is the parsing of the string and calculating the necessary size that can be complex. Part of that is the fault of the JSON requirements. But we can certainly discuss it if you are up to it. Would be great. |
.. of course it could also simply be because I haven't had the energy and time and patience to look into it more carefully but anyway hopefully we will get it sorted out once IOCCC28 is over. Anyway you are certainly welcome to look at it if you like and I would be happy to look at any pull request. The same make rules that are in mkiocccentry prep and release do the same things btw. Hope you have a great night @SirWumpus and thanks again! |
BTW @SirWumpus: I just looked at it a bit but it seems macOS does not have |
|
macOS does have |
But the standard makes no mention of needing |
That curious because I used Oh well. Thanks. |
We believe we have addressed all of the current questions that still need answering at this time. If we've missed something or something else needs to be clarified, please ask again. |
QUESTIONAre we in a position to close this issue today in order to meet the soft code freeze deadline? NOTE after today, we will about 5 more days to:
So let us REALLY try to be in a position to close this issue soon, if possible. UPDATE 0We do understand that this issue addresses "rules and guidelines", so perhaps this issue can be closed and let the talks of "polishing the rules and guidelines" until IOCCC28 opens? |
Moved information about the fields of `status.json` to FAQ 10.11.
Commit fadb1dd resolved this issue by moving information about the fields of status.json to FAQ 10.11. |
Is there an existing issue for this?
Describe the bug
The current status page shows all of the JSON cases that should be preprocessed before display.
What you expect
I expect a shorter webpage and not all the cases description.
Environment
Anything else?
No response
The text was updated successfully, but these errors were encountered: