| [20:14:31] | <symbioquine[m]> | <symbioquine[m]> "Anybody else getting timeouts..." <- https://status.discourse.org/ |
| [20:15:06] | <symbioquine[m]> | ACTION uploaded an image: (256KiB) < https://libera.ems.host/_matrix/media/v3/download/matrix.org/ZacfvjaDozn... > |
| [04:00:22] | * riotmiked[m] has quit (Quit: You have been kicked for being idle) |
| [09:30:51] | <mstenta[m]> | Forum is back up it seems |
| [09:30:56] | <mstenta[m]> | Looks like they moved us to a different region |
| [09:31:11] | <mstenta[m]> | ACTION uploaded an image: (56KiB) < https://libera.ems.host/_matrix/media/v3/download/matrix.org/iVGpDTSbJJl... > |
| [11:44:34] | <symbioquine[m]> | <symbioquine[m]> "Left two questions on the..." <- mstenta: I hope I wasn't too pushy with my comment that there might be a race condition in those constraint checks. I don't think it's worse than it was, even if the race condition is there - it just informs whether the change makes circular relationships impossible or just really unlikely. |
| [11:45:45] | <mstenta[m]> | Not at all! Its a good question and I'm pretty sure it does NOT protect against that |
| [11:46:11] | <mstenta[m]> | I think there are other situations where it will be possible to end up with a circular relationship too |
| [11:46:22] | <mstenta[m]> | I plan to follow up on your comment next week |
| [11:47:12] | <mstenta[m]> | I think this change will prevent the most likely scenario, so its probably worth starting here and maybe opening a followup up for tracking the hypothetical scenarios |
| [13:01:23] | <lordeddi[m]> | just out of curiosity: that CSV bug #497 (https://github.com/farmOS/farmOS/pull/622). If i understand correctly, the column containing multiple quantities has newlines etc.... |
| [13:01:24] | <lordeddi[m]> | and the solution for now would be to only have the 1st quantity outputted in the csv? or only that line `preg_replace` which strips tabs, newlines, extra spaces etc... ? |
| [13:01:24] | <lordeddi[m]> | i find this interesting, since there is the need of outputting more than 1 thing in a CSV field/column.. and it triggers database-normalization warnings in my brain :) |
| [13:01:24] | <lordeddi[m]> | * just out of curiosity: that CSV bug #497 (https://github.com/farmOS/farmOS/pull/622). If i understand correctly, the column containing multiple quantities has newlines etc.... |
| [13:01:24] | <lordeddi[m]> | and the solution for now would be to only have the 1st quantity outputted in the csv? or only that line `preg_replace` which strips tabs, newlines, extra spaces etc... ? |
| [13:01:24] | <lordeddi[m]> | i find this interesting, since there is the need of outputting more than 1 thing/value in a CSV field/column.. and it triggers database-normalization warnings in my brain :) |
| [13:01:36] | <symbioquine[m]> | Right now the formatting is bad enough with multiple items (and whitespace) to be basically broken |
| [13:02:31] | <symbioquine[m]> | Multiple values can be retrieved as their own rows via the quantities view. |
| [13:04:41] | <symbioquine[m]> | e.g. /quantities/standard |
| [13:05:20] | <symbioquine[m]> | ACTION uploaded an image: (221KiB) < https://libera.ems.host/_matrix/media/v3/download/matrix.org/geMFpzLtqIF... > |
| [13:06:52] | <symbioquine[m]> | <symbioquine[m]> "Right now the formatting is..." <- In the CSV export that is. |
| [13:06:53] | <lordeddi[m]> | yes, i see what you mean. but that bug was mostly about exporting CSV, no? and for CSV i found it interesting: it makes total sense to export some logs which contain multiple values. |
| [13:06:53] | <lordeddi[m]> | anyhow, don't want to stir up dust about neat pull requests just before a release 😅 |
| [13:06:59] | <symbioquine[m]> | ACTION uploaded an image: (20KiB) < https://libera.ems.host/_matrix/media/v3/download/matrix.org/oFrfpsoTOiR... > |
| [13:07:17] | <lordeddi[m]> | yes this 👆️ |
| [13:07:31] | <symbioquine[m]> | > <@lordeddi:oblak.be> yes, i see what you mean. but that bug was mostly about exporting CSV, no? and for CSV i found it interesting: it makes total sense to export some logs which contain multiple values. |
| [13:07:31] | <symbioquine[m]> | > anyhow, don't want to stir up dust about neat pull requests just before a release 😅 |
| [13:07:31] | <symbioquine[m]> | No, it's a good question |
| [13:07:44] | <lordeddi[m]> | so what was the proposed solution? cutting out the newlines etc as in that preg_replace line in the pull requests |
| [13:08:11] | <lordeddi[m]> | so it would be many values with spaces between them, but still all the values? -> which would keep the value of that log |
| [13:08:34] | <symbioquine[m]> | ACTION uploaded an image: (8KiB) < https://libera.ems.host/_matrix/media/v3/download/matrix.org/Lcapzklpkcw... > |
| [13:09:29] | <lordeddi[m]> | ah, i see |
| [13:10:28] | <symbioquine[m]> | There's also https://github.com/farmOS/farmOS/pull/619 which is the one that makes the logs view only show one value |
| [13:11:20] | <lordeddi[m]> | i missed that one! |
| [13:11:49] | <lordeddi[m]> | i just saw it hahah. |
| [13:12:26] | <lordeddi[m]> | so, then there are the 2 solutions |
| [13:12:26] | <lordeddi[m]> | 1. only showing one value |
| [13:12:26] | <lordeddi[m]> | 2. making sure that one value is shown without all that extra stuffs |
| [13:12:41] | <symbioquine[m]> | yep |
| [13:12:48] | <lordeddi[m]> | thanks for pointing it out symbioquine |
| [13:13:39] | <symbioquine[m]> | And obviously one can use the API and a script to create a CSV with any desired structure. |
| [13:13:56] | <lordeddi[m]> | yeah that #619 answers all my questions |
| [13:14:56] | <lordeddi[m]> | its an interesting problem i think, next time this is discussed i would love to eavesdrop and or discuss along with y'all :) |
| [13:15:32] | <lordeddi[m]> | somewhen after the release and when everyone has lots of free time :) |
| [13:37:09] | <symbioquine[m]> | <lordeddi[m]> "somewhen after the release and..." <- hah! |
| [14:21:55] | <mstenta[m]> | lordeddi: https://twitter.com/tunguz/status/1604101663517683712 |
| [14:27:12] | <lordeddi[m]> | 😅 yes something like that |
| [14:27:14] | <lordeddi[m]> | :) |
