Feature request - Token substitution sets

Currently, there seems to be no easy way of specifying multiple values to a check using token substitution. So far I have got around this by building a check template that accepts a comma-separated list of values and checking each one.
My suggestion would be to allow a dict or array in the check definition that is iterated over and supplies the check tokens.
For example:
...
"check-service": {
  "command": "check-service.rb -name :::service::: -state :::state:::",
  "tokens": [
    {
      "service": "dns",
      "state": "running"
    },
    {
      "service": "dhcp",
      "state": "stopped"
    }
  ]
}
...

In the above example, each of the items in the array must supply tokens that will be used for substitution. This lets a single check definition check multiple things without needless separate checks. The issue with this example is that I'm not sure how to separate the output of each check.

Alternatively, tokens may need to be a dict, and each item would be under a unique key so that each check result is given a unique name automatically, for example:
...
"check-service": {
  "command": "check-service.rb -name :::service::: -state :::state:::",
  "tokens": {
    "dns": {
      "service": "dns",
      "state": "running"
    },
    "dhcp": {
      "service": "dhcp",
      "state": "stopped"
    }
  }
}
...

And the check results would be generated under check-service-dns and check-service-dhcp. This would ideally not show each individual check within the API, only in the results (For sake of keeping the API & Uchiwa clean).
Any thoughts or suggestions?

I feel like there is too much implementation logic here to translate cleanly. I think you can accomplish this fairly easy with defining the start and stopped services as two check definitions. Imagine something like this:
"check-service-running": {
  "command": "check-service.rb -name :::services_running::: -state running",
}

"check-service-stopped": {
  "command": "check-service.rb -name :::services_stopped::: -state stopped",
}

With some decent CM that should be really easy to accomplish. I might be totally missing important I am not seeing but this sounds like asking sensu to have some crazy data structure to support almost any combination of args together which seems very hard to account for misconfiguration.

Well, my biggest issue was the pollution to Uchiwa and the checks API. I started down this route before but ended up with over 20 different checks for different services, some of which only appear on a single server. I solved this somewhat by writing the check-multi-template stuff for Windows, but it just felt like a hack having to do this in the check plugin. I had the same remark from somebody in Slack, as they felt that having a single check with a command like the following:
  "command": "powershell.exe -file check-service-multi.ps1 -critical 'service1,service 2,service$3,...'"

was a bit unwieldy, and passes the issue off to the CM tool instead of getting Sensu to handle it, in what could be a more effective manner.
In my services example, I generally only need to check the service name, the state was just to illustrate multiple tokens. But another example might be having different warning values per disk, per client - Which isn't really possible with the current level of granularity.

I guess what I am trying to say is that only your check script can know the relation of tokens to each other in these use cases. All sensu knows how to do is retrieve values and pass it to your check, if you need something like this you probably need to devise a more elaborate data structure in your check and have some string representation to pass on cli.
Some examples:

--service 'telnet:stopped, bind9:running, mybotnet:running'
--disks 'disk1:crit,disk2:crit'
--support-plan client1:pro,client2:enterprise

If you need multiple args that relate to other other args that I think they should be separate checks to make it clear what the check is looking for. If you think this is wrong then you need to somehow pass a mapping of relations and the simplest solution is something like what I proposed above.

was a bit unwieldy, and passes the issue off to the CM tool instead of getting Sensu to handle it, in what could be a more effective manner.

For who a person or machine? For a machine there is nothing wrong with a really long line and at the end of the day it will be passed like that to the cli anyways. I have a check for dynamodb tables that after rendering from CM (chef) the list of tables is over 1300 chars but I interact with my CM in a way that makes it easy to handle with essentially no effort. I generally solve that with something like this in my CM "check-dynamodb-throttle.rb -r us-west-2 -t #{node['MY']['CUSTOM']['NAMESPACE']['DYNAMODB_TABLES'].join(',')} --warning-over 5 --critical-over 10". This seems far better to ask your CM to do something it is really good at doing than asking sensu to become CM.
I might be missing something so I will hit you up in slack and maybe a chat would best help me understand what the real issues and potential solutions are.

So we talked about this over slack and one of the desires of this feature was to not just to do token substitution the goal was to on the fly create more instantiations of the check with the token substitution, building on the concept of a proxy check.
For example:
"check-service": {
  "command": "powershell check-service -name ::service:::",
  "service": "dhcp,dns"
}

Would actually create 2 checks like this:
"check-service-dhcp": {
  "command": "powershell check-service -name dhcp"
}

"check-service-dns": {
  "command": "powershell check-service -name dns"
}

Aside from the fact that CM already makes doing that fairly trivial (to define the same check over and over again from say an array of values). In this case you are directly aware of the performance penalty being applied to invoke the check 50 times loading up any interpreter/vm, libraries, sdks, etc over and over. By moving that into sensu's problem to solve you are now hiding the fact that this is what is going to happen behind the scenes. While in some cases this might be OK there are others where it would bring your client to it's knees.
Basically the end result is already solved by CM, it hides a lot of performance penalties that were not apparent even to advanced sensu users, and does not really bring anything new to the table. We could not find a valid use case where CM and/or rewriting the checks gave us the same or better experience.
To be clear even when using CM to accomplish this you may want/need to rewrite your checks to be multi resource aware because of the performance penalty of not is too high. Take for example the aws checks loading the aws-sdk is quite expensive about a year or so ago I had to add a bunch more instances of the sqs messages check. Doing so put that server (client) at 100% CPU all the time, even even after doubling and then quadrupling the number of CPU cores. After rewriting the check to accept multiple queues in a smaller number of instances of the check and removed the old ones the CPU utilization dropped to an average of 50-60% .
We should make users life easier when it is the right thing to do, when the right thing to do is non trivial it means that we get to have these conversations to help solve the problem several layers down rather than one on the surface. Just my opinions, feel free to disagree.

Nothing else to add, you summed it up perfectly :smile:
Thanks for taking the time to discuss it!


[1.4] Update documentation

Updated Developer Guide.
Renamed all "repo" to repository.
Added IO user stories.
Added Suggestion user stories.
Added IO use cases.
Added Instructions for Manual Testing.
Added OpenCommand Implementation.
Added OpenCommand sequence diagram.
Added ExportCommand Implementation.
Added ExportCommand activity diagram.
Added SaveCommand Implementation.
Added ImportCommand Implementation.
Added Suggestion commands implementation.
Updated User Guide.
Added images in open and import to show their differences.
Added note to better explain FILE_PATH and INDEX_RANGE in File management.
Added section on Suggestion commands.
Added Patient Teeth per record [Coming in v1.5]
Added Adding images to records [Coming in v1.6]

Coverage increased (+13.6%) to 66.466% when pulling 5b36a291e5dc7c2c237c0166d368253db3ffd38a on JiaHaoLim:update-documentation into f4c6d09eeb912a164247cdcc643132ca0457fea5 on CS2103-AY1819S2-W17-2:documentation.


Quantification migration #8036

This closes #8036.

@javitonino CR this, please.

:+1: Left a minor comment about specs.


Talk with Team 1 to get their model structure for training their CIFAR10 model

then replace last layer for binary classification

see #30


Add more logs in RSP for troubleshooting

Currently, I found the RSP do not have too much logs, so it seems a bit difficult to debug it when customer have issues, it is better add more logs in RSP to improve troubleshooting.
/cc @irfanurrehman can you please add a label as good-first-issue? Thanks.

Hi @gyliu513 I can work on this.. can you assign this to me?

Thanks @johnarok , seems I can only assign to members of federation v2, but I think you can go ahead to create the PR, as I think no one is working on this for now. ;-)

/remove-lifecycle stale
FYI @nzoueidi

/reopen
/remove-lifecycle rotten

/assign


Add random colors wrapper for plotting instance segmentation

Would be nice to have a function wrapper to generate random colors for displaying instance segmentation with matplotlib (not sure if we have one already). I have a function which I use often, I'll refactor it and take care of it.

Updated at https://github.com/constantinpape/torch-em/pull/244


Remove harmony branch.

Closes #1172.

Why clear it out? Just delete the branch.

LGTM :+1:

The change has been landed (not sure why this PR wasn't automaticall closed).


TS Playground-v2 automatic type acquisition: "Cannot find module 'react'."

There currently seems to be a problem with react types / automatic type acquisition in general (or I just don't have got the hang of the new Playgroundv2 yet):
import React, { useState } from "react"; // error
import ReactDOM from "react-dom"; // error
import {pipeable } from "fp-ts" // works

const App = () => {
    const [state, setState] = useState(42);
    return
test {state}
;
};

ReactDOM.render(<App />, document.getElementById("root"));

Playground link
Above code first resulted in "react" and "react-dom" not found errors, "fp-ts" was found. Adding custom npm modules didn't help either. ATA is enabled as seen down under, esModuleInterop and jsx option checked as well. Strangely enough, sometime it doesn't work with any library  (I cannot reproduce it consistently).
Is this an unhappy coincidence with the new site's testing weekend or a known issue?

Hrm, the ATA code in v2 hasn't changed since Jan 2nd - will add some more debugging tools to the v2 code to see if something is up there

hah, this is because we added React as an exposed variable to playground plugins


Timestamps of mp4 and subtitle files used to be the same as the download time but are now different

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
I don't want to open this as a bug because I'm not sure it is.  I'm just looking for guidance here.
I've downloaded tons of video + subtitle combinations using youtube-dl for a few years now and more recently been doing the same with a switch to yt-dlp.  Whenver I would download a video + subtitles in the past using --write-subs, both would get recorded to my local filesystem as new files with the current time as the timestamp listed as the last write to my local filesystem.  This appeared to be the case whether the files were segmented or not (at least on the video side of things).
The huge benefit of this is that when I batch download vids + subtitles, when I later sort a folder by timestamp, the downloaded files are in a very easy to follow and organized sequence that is consistent over time as new batches come in.  By that I mean that:

the files are always in the order that they were download
each subtitle file is always right next to it's parent video file

This makes both using and managing the files very easy because pairs of files are always together and you have an easy way to chronologically see your downloads as they occured over time.  You can't beat this style.
However, since either the switch to yt-dlp or simply the switch to newer segmented subtitle files or some other unexplained reason, the downloaded subtitle files and sometimes even the video container files themselves take on the date / timestamp that they were uploaded to the provider server and not the timestamp when they are recorded to my system.   This throws the ordering and pairing of the files off on the local filesystem.
I'm mostly seeing this in terms of francetv video links (francetv extractor) because this is what I download the most.
For instance, if francetv uploaded a video and subtitle file to it's servers two weeks or a year ago, and I download it today, while downloading is in process, the initial video, audio and subtitle .part files  are always initially stamped with the current time.  But after segment download is completed, some final "file processing" occurs that resets the subtitle file timestamp to an older date then the original and current download time in my own filesystem.
This often leaves the mp4 container at the current timestamp but reset's the subtitle file to what I presume is it's original provider upload timestamp.
Example:  You can see that I downloaded all of these files today (around midnight straddling Oct 29th and 30th).  The listing is in chronological order.  The mp4 files have retain the local download time timestampbut the vtt subtitle files get reset back anywhere from a few hours to a few days earlier which throws them out of sync with their parent mp4 files.
total 4229104
drwxr-xr-x  15 mbpme  staff        510 Oct 30 00:50 ./
-rw-r--r--   1 mbpme  staff  720183494 Oct 30 00:11 C dans l'air - Macron _ Biden  - après la trahison, la confrontation !-89725f6e-3da4-4a35-830a-5a5063d69d56.mp4
-rw-r--r--   1 mbpme  staff  281383092 Oct 30 00:06 Plus belle la vie-472fb7e3-03ac-4f6b-aa8d-3b400b0e628a.mp4
-rw-r--r--   1 mbpme  staff  598473322 Oct 30 00:02 Germinal-af8c03fb-a354-4829-b493-2916324ea7c9.mp4
-rw-r--r--   1 mbpme  staff  564823570 Oct 29 23:56 Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.mp4
-rw-r--r--   1 mbpme  staff      21381 Oct 29 20:59 Plus belle la vie-472fb7e3-03ac-4f6b-aa8d-3b400b0e628a.fr.vtt
-rw-r--r--   1 mbpme  staff      74615 Oct 29 19:43 C dans l'air - Macron _ Biden  - après la trahison, la confrontation !-89725f6e-3da4-4a35-830a-5a5063d69d56.fr.vtt
-rw-r--r--   1 mbpme  staff      41668 Oct 27 23:20 Germinal-af8c03fb-a354-4829-b493-2916324ea7c9.fr.vtt
-rw-r--r--   1 mbpme  staff      36856 Oct 27 22:33 Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fr.vtt

Why does this happen?  Is there a way to override this behavior?  As I said it was never like this in the past and the behavior only changed in the last few months.
Here's the command I use for one of the example files above.
yt-dlp -f best[ext=mp4][width<=960][format_id^=hls]/bestvideo[ext=mp4][width<=960][format_id^=hls]+bestaudio[ext=mp4] -Uv -o %(title)s-%(id)s.%(ext)s --write-subs https://www.france.tv/france-2/germinal/germinal-saison-1/2830613-episode-1.html

Thanks

--no-mtime is the option you are looking for

Whenver I would download a video + subtitles in the past using --write-subs, both would get recorded to my local filesystem as new files with the current time as the timestamp listed as the last write to my local filesystem. This appeared to be the case whether the files were segmented or not (at least on the video side of things).

To verify this claim, you are going to have to show the verbose logs of running both youtube-dl and yt-dlp with the same options

Thanks for that tip!  Hopefully that takes care of it.
I have 4T / 4 years worth of files to prove it as I use them extensively / daily so that when things stop working and the files become hard to work with it's obvious and not an oversight.  But I totally understand where you're coming from too.
The thing is that since the youtube-dl francetv IE is broken, there is no way to run a side by side comparison.   But even if I could, I'm not sure it would matter because this behavior seemed to creep in around the time the francetv IE broke and there was a move to the mpd formats and segmented subtitle files.   So my guess it has something to do with the combined francetv backend / frontend IE changes that took place a few months ago as that is when the change in behavior started happening.
That said, per my other "ard Tatort" tests that I've posted in the other issue, I can see that the "ard" IE does process the subtitle files correctly maintaining the timestamps.   So if you run the following two commands, you will get different subtitle timestamp results for the different IEs.  We're not comparing old fork and new fork francetv IEs to one another but why does current fork "ard" IE maintain the timestamps and "francetv" IE doesn't?   Where or what in the code determines the final timestamp of the subtitle?   (I guess I need to search for --no-mtime in the codebase)
ard first:
 yt-dlp -Uv -f hls-162 --write-subs https://www.ardmediathek.de/video/tatort/absturz/mdr-fernsehen/Y3JpZDovL21kci5kZS9iZWl0cmFnL2Ntcy84OTIxZTdmYy01YmJjLTRkZmYtYTE5Yy1kYTlkNmUxMzMxMGY/
[debug] Command-line config: ['-Uv', '-f', 'hls-162', '--write-subs', 'https://www.ardmediathek.de/video/tatort/absturz/mdr-fernsehen/Y3JpZDovL21kci5kZS9iZWl0cmFnL2Ntcy84OTIxZTdmYy01YmJjLTRkZmYtYTE5Yy1kYTlkNmUxMzMxMGY/']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, err utf-8, pref UTF-8
[debug] yt-dlp version 2021.10.22 (source)
[debug] Lazy loading extractors is disabled
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Git HEAD: fccf50211
[debug] Python version 3.9.7 (CPython 64bit) - macOS-11.2.3-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.4.1, ffprobe 4.4.1, rtmpdump 2.4
[debug] Optional libraries: sqlite
[debug] Proxy map: {}
yt-dlp is up to date (2021.10.22)
[debug] Using fake IP <IP_ADDRESS> (DE) as X-Forwarded-For
[debug] [ARD:mediathek] Extracting URL: https://www.ardmediathek.de/video/tatort/absturz/mdr-fernsehen/Y3JpZDovL21kci5kZS9iZWl0cmFnL2Ntcy84OTIxZTdmYy01YmJjLTRkZmYtYTE5Yy1kYTlkNmUxMzMxMGY/
[ARD:mediathek] Y3JpZDovL21kci5kZS9iZWl0cmFnL2Ntcy84OTIxZTdmYy01YmJjLTRkZmYtYTE5Yy1kYTlkNmUxMzMxMGY: Downloading webpage
[ARD:mediathek] 94131462: Downloading media JSON
[ARD:mediathek] 94131462: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id
[debug] Downloading subtitles: de
[info] 94131462: Downloading 1 format(s): hls-162
[info] Writing video subtitles to: Absturz [94131462].de.ttml
[debug] Invoking downloader on "https://www.ardmediathek.de/subtitle/554906"
[download] Destination: Absturz [94131462].de.ttml
[download] 100% of 133.30KiB in 00:00
[debug] Invoking downloader on "https://mdronline-vh.akamaihd.net/i/mp4dyn2/8/FCMS-8921e7fc-5bbc-4dff-a19c-da9d6e13310f-,41dd60577440,730aae549c28,d1ceaa57a495,be7c2950aac6,70dc5e5f2727,1af20647950e,_89.mp4.csmil/index_2_av.m3u8?set-akamai-hls-revision=5"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 532
[download] Destination: Absturz [94131462].mp4
[download] 100% of 142.18MiB in 01:59
[debug] ffmpeg command line: ffprobe -show_streams 'file:Absturz [94131462].mp4'
[FixupM3u8] Fixing malformed AAC bitstream of "Absturz [94131462].mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i 'file:Absturz [94131462].mp4' -c copy -map 0 -dn -f mp4 -bsf:a aac_adtstoasc 'file:Absturz [94131462].temp.mp4'

 $ ▶ ll -t
drwxr-xr-x  18 mbpme  staff        612 Oct 30 02:44 ./
-rw-r--r--   1 mbpme  staff  127453739 Oct 30 02:44 Absturz [94131462].mp4
-rw-r--r--   1 mbpme  staff     136503 Oct 30 02:42 Absturz [94131462].de.ttml

francetv
yt-dlp -Uv -f best[ext=mp4][width<=960][format_id^=hls]/bestvideo[ext=mp4][width<=960][format_id^=hls]+bestaudio[ext=mp4] -Uv -o %(title)s-%(id)s.%(ext)s --write-subs https://www.france.tv/france-2/germinal/germinal-saison-1/2830613-episode-1.html

[debug] Command-line config: ['-q', '--progress', '-f', 'best[ext=mp4][width<=960][format_id^=hls]/bestvideo[ext=mp4][width<=960][format_id^=hls]+bestaudio[ext=mp4]', '-Uv', '-o', '%(title)s-%(id)s.%(ext)s', '--write-subs', 'https://www.france.tv/france-2/germinal/germinal-saison-1/2830613-episode-1.html']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8 (No ANSI), err utf-8 (No ANSI), pref UTF-8
[debug] yt-dlp version 2021.10.22 (source)
[debug] Lazy loading extractors is disabled
[debug] Plugins: ['SamplePluginIE', 'SamplePluginPP']
[debug] Git HEAD: fccf50211
[debug] Python version 3.9.7 (CPython 64bit) - macOS-11.2.3-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.4.1, ffprobe 4.4.1, rtmpdump 2.4
[debug] Optional libraries: sqlite
[debug] Proxy map: {}
yt-dlp is up to date (2021.10.22)
[debug] [FranceTVSite] Extracting URL: https://www.france.tv/france-2/germinal/germinal-saison-1/2830613-episode-1.html
[FranceTVSite] 2830613-episode-1: Downloading webpage
[debug] [FranceTV] Extracting URL: francetv:dbc8e2d8-003c-4381-ae5d-5fb4b6f89706
[FranceTV] dbc8e2d8-003c-4381-ae5d-5fb4b6f89706: Downloading desktop video JSON
[FranceTV] dbc8e2d8-003c-4381-ae5d-5fb4b6f89706: Downloading mobile video JSON
[FranceTV] dbc8e2d8-003c-4381-ae5d-5fb4b6f89706: Downloading signed dash manifest URL
[FranceTV] dbc8e2d8-003c-4381-ae5d-5fb4b6f89706: Downloading MPD manifest
[FranceTV] dbc8e2d8-003c-4381-ae5d-5fb4b6f89706: Downloading signed hls manifest URL
[FranceTV] dbc8e2d8-003c-4381-ae5d-5fb4b6f89706: Downloading m3u8 information
[debug] Formats sorted by: hasvid, ie_pref, lang, quality, res, fps, hdr:12(7), vcodec:vp9.2(10), acodec, filesize, fs_approx, tbr, vbr, abr, asr, proto, vext, aext, hasaud, source, id

[debug] Downloading subtitles: fr
[info] dbc8e2d8-003c-4381-ae5d-5fb4b6f89706: Downloading 1 format(s): hls-1582+hls-audio-aacl-96-Audio_Français
[info] Writing video subtitles to: Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fr.vtt
[debug] Invoking downloader on "https://cloudreplay.ftven.fr/693c6333b9716/1050200180_france-domtom_TA.ism/ZXhwPTE2MzU1NjU5OTZ+YWNsPSUyZjY5M2M2MzMzYjk3MTYlMmYxMDUwMjAwMTgwX2ZyYW5jZS1kb210b21fVEEuaXNtKn5obWFjPTI3ZWQ2ZWVmMDk3NjkyMjQzMmNlNDRmNWQ5ZmUyODEzMTYzYjc2ZDAwZTMyZDc5ODg2MzI5MzJkOGJlNjg0Mjk=/1050200180_france-domtom_TA-textstream_fre=1000.m3u8"
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 301
[download] Destination: Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fr.vtt

[download] 100% of 35.99KiB in 00:45

[debug] Invoking downloader on "https://cloudreplay.ftven.fr/693c6333b9716/1050200180_france-domtom_TA.ism/ZXhwPTE2MzU1NjU5OTZ+YWNsPSUyZjY5M2M2MzMzYjk3MTYlMmYxMDUwMjAwMTgwX2ZyYW5jZS1kb210b21fVEEuaXNtKn5obWFjPTI3ZWQ2ZWVmMDk3NjkyMjQzMmNlNDRmNWQ5ZmUyODEzMTYzYjc2ZDAwZTMyZDc5ODg2MzI5MzJkOGJlNjg0Mjk=/1050200180_france-domtom_TA-video=1400000.m3u8"
[hlsnative] Downloading m3u8 manifest
WARNING: The stream has AES-128 encryption and pycryptodomex is not available; extraction will be delegated to ffmpeg
[download] Destination: Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-1582.mp4
[debug] ffmpeg command line: ffmpeg -y -loglevel quiet -headers 'User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3657.1 Safari/537.36
Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Encoding: gzip, deflate
Accept-Language: en-us,en;q=0.5
' -i https://cloudreplay.ftven.fr/693c6333b9716/1050200180_france-domtom_TA.ism/ZXhwPTE2MzU1NjU5OTZ+YWNsPSUyZjY5M2M2MzMzYjk3MTYlMmYxMDUwMjAwMTgwX2ZyYW5jZS1kb210b21fVEEuaXNtKn5obWFjPTI3ZWQ2ZWVmMDk3NjkyMjQzMmNlNDRmNWQ5ZmUyODEzMTYzYjc2ZDAwZTMyZDc5ODg2MzI5MzJkOGJlNjg0Mjk=/1050200180_france-domtom_TA-video=1400000.m3u8 -c copy -f mp4 file:Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-1582.mp4.part
[ffmpeg] Downloaded 526983929 bytes

[download] 100% of 502.57MiB in 02:43[debug] Invoking downloader on "https://cloudreplay.ftven.fr/693c6333b9716/1050200180_france-domtom_TA.ism/ZXhwPTE2MzU1NjU5OTZ+YWNsPSUyZjY5M2M2MzMzYjk3MTYlMmYxMDUwMjAwMTgwX2ZyYW5jZS1kb210b21fVEEuaXNtKn5obWFjPTI3ZWQ2ZWVmMDk3NjkyMjQzMmNlNDRmNWQ5ZmUyODEzMTYzYjc2ZDAwZTMyZDc5ODg2MzI5MzJkOGJlNjg0Mjk=/1050200180_france-domtom_TA-audio_fre=96000.m3u8"

[hlsnative] Downloading m3u8 manifest
WARNING: The stream has AES-128 encryption and pycryptodomex is not available; extraction will be delegated to ffmpeg
[download] Destination: Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-audio-aacl-96-Audio_Français.mp4
[debug] ffmpeg command line: ffmpeg -y -loglevel quiet -headers 'User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3657.1 Safari/537.36
Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,*/*;q=0.8
Accept-Encoding: gzip, deflate
Accept-Language: en-us,en;q=0.5
' -i https://cloudreplay.ftven.fr/693c6333b9716/1050200180_france-domtom_TA.ism/ZXhwPTE2MzU1NjU5OTZ+YWNsPSUyZjY5M2M2MzMzYjk3MTYlMmYxMDUwMjAwMTgwX2ZyYW5jZS1kb210b21fVEEuaXNtKn5obWFjPTI3ZWQ2ZWVmMDk3NjkyMjQzMmNlNDRmNWQ5ZmUyODEzMTYzYjc2ZDAwZTMyZDc5ODg2MzI5MzJkOGJlNjg0Mjk=/1050200180_france-domtom_TA-audio_fre=96000.m3u8 -c copy -f mp4 'file:Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-audio-aacl-96-Audio_Français.mp4.part'
[ffmpeg] Downloaded 37013119 bytes

[download] 100% of 35.30MiB in 01:37[debug] ffmpeg command line: ffprobe -show_streams 'file:Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-audio-aacl-96-Audio_Français.mp4'

[Merger] Merging formats into "Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel repeat+info -i file:Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-1582.mp4 -i 'file:Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-audio-aacl-96-Audio_Français.mp4' -c copy -map 0:v:0 -map 1:a:0 -bsf:a:0 aac_adtstoasc file:Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.temp.mp4
Deleting original file Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-audio-aacl-96-Audio_Français.mp4 (pass -k to keep)
Deleting original file Germinal-dbc8e2d8-003c-4381-ae5d-5fb4b6f89706.fhls-1582.mp4 (pass -k to keep)

Thanks

Without --no-mtime, the timestamp is set to whatever the server says is the "last modified time" of the file. So it is possible that it is a server side change like you said.
Anyway, check with --no-mtime and close the issue if it serves your needs

Where or what in the code determines the final timestamp of the subtitle? (I guess I need to search for --no-mtime in the codebase)

It is in http.py and fragment.py. Search for 'updatetime' to quickly find it

Ok. give me a few days to try it and post back.  Thanks for the followup.

Didn't realize I could close these issues myself.  I will do just that.  I just tried --no-time and it did the trick.  Thanks for the tip.


If there is a Pipfile or .python-version file, trigger discovery when autoselecting

THIS ISSUE MAY NOT BE NEEDED ONCE WE HAVE WORKFLOW EXTENSIONS MANAGING THEIR OWN ENVS: https://github.com/microsoft/vscode-python/issues/22836
When opening a new workspace, before auto-selecting begins:

[ ] If there is a Pipfile, make sure to trigger Pipenv discovery:
[ ] If there is a .python-version file, make sure to trigger Pyenv discovery

This is where we should do it: https://github.com/microsoft/vscode-python/blob/b9109cfd49fe1361d7a76bbd9b22fe4f4300f81e/src/client/interpreter/autoSelection/index.ts#L207-L209

I commented on 23114, does this issue have anything to do with launch.json not respecting my pipenv virtualenv? this used to work, but now i am getting errors that a module is not found if i run python through the debug console.. it is installed in my pipenv, but for some reason it is looking in my global pyenv for the module??

@karthiknadig - curious if you think this is still necessary or no longer in scope

No longer i scope. If users contribute an extension for pipenv or pyenv, they can control this there.


[Suggestion] Special Interaction with RF Tools Storage Scanner and Crafters

It would be nice to allow special setups between Xnet and those blocks to set up on demand auto crafting within the Storage Scanner interface somehow. Similar to how you can with AE2 or RS, but without the actual "computer storage" aspect because that already exists in RF Tools modular storage.
Basically have some kind of interface that showed what was craftable by connected crafters within the storage scanner and allow them to be requested so that Xnet delivers the components and returns the crafted items to the modular storage.

At this moment I'm working on the base of XNet as a standalone thing. But integration with my other mods (specifically RFTools/RFTools Control) is planned


added tyzbit hosts

Includes Windows 10 tracking sites not caught by previous entries.
source: https://github.com/10se1ucgo/DisableWinTracking

I like this!   But the update.info file is incorrect...


Assign only duties to roles

Hi there,
In this article, there is a recommendation to "assign only duties to roles". Is this recommendation basing on the assumption that each privilege will be always assigned to one (and only one) duty? Or  it will be a case-by-case scenario?
For instance,  the privilege "VendProspectiveVendorDataAreaSelectDeny" does not belong to any duty. So, this privilege has to be assigned to a role directly.
And the privilege "ProjCostPriceExpenseView" is assigned to two standard duties. If these two duties are assigned to one role, and if the privilege "ProjCostPriceExpenseView" needs to be removed from the role, then two new customized duties (with the privilege removed) need to be created to replace these two standard duties in the role. In this case, actually, it easier to assign the privileges to the role directly, so the customization of the new duties can be avoided.
As a result, to assign only duties to roles, is it a recommendation, a best practice or maybe just a case-by-case scenario?
Thanks and regards,
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 09327be4-4437-628d-65b0-45e5824a06d5
Version Independent ID: 1bbaa6dd-6e50-77e1-36c2-10c2dbd55459
Content: Role-based security - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/sysadmin/role-based-security.md
Service: dynamics-ax-platform
GitHub Login: @sericks007
Microsoft Alias: sericks

@FINOPSAOSCEWA, thanks for posting this!
@robadawy, can you please review and answer the questions.

@ChrisGarty, please review and answer the questions, and update the topic if needed.

This is just a best practice. It needs to be evaluated on a case-by-case basis. I updated the documentation slightly to provide the reason for the recommendation: grouping.
"However, we recommend that you assign only duties to roles, so that the privileges are grouped together into a duty first for easier maintenance. "


add hg-tui project who use tui-rs

Description
Add a new project hg-tui which uses tui-rs in README
https://hellogithub.com is a GREAT website to view Github's project
Checklist

[x] I have read the contributing guidelines.
[x] I have added relevant tests.
[x] I have documented all new additions.

Done in https://github.com/fdehau/tui-rs/pull/659


Publisher throwing an error 'No such file or directory'

Release version
v1.2
Describe the bug
While running the publisher, it's failing with an error 'No such file or directory' while publishing artifacts. I have print statements to ensure that publisher was downloaded but not sure what else could be causing this.

 I don't know why but it started working now without a change.


Would like an "Align Only" option on "Align\Sort line by columns" the same as the existing "Sort Only" option.

Is your feature request related to a problem? Please describe.
I would like to align by column but do not want to sort the lines.
Describe the solution you'd like
I would select "Align\Sort line by columns", put a ',' in the Separator field, select the "Align Only" box, leave the "Sort Only box" unchecked, and press OK.
Note that any other UI incantation to get alignment without sorting is fine. Just would like the anti-sort feature.
Describe alternatives you've considered
The alternative is to live with sorted lines or use another plugin, like "Vertical Align" or "Column Align", both of which have other idiosyncrasies that may them "less than perfect".
Additional context
Add any other context or screenshots about the feature request here.

If you do not fill Sort by columns field, then it will not sort by anything, so it will in fact align only.

Makes perfect sense but is not intuitively obvious. Thanks.


🐞[Bug]: Text not visible in footer

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
the subscribe button in the footer of home page has to be changed to dark color to enhance visibility.
Expected behavior
Text inside subscribe button will be enhanced
Add ScreenShots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Can you assign me this issue

I would like to work on this project. Please assign it to me.


View Order Label Config dosent rename the View Order text in Gmail Highlights

I added the config for "View Action Label" and expected that in the Gmail I will get the renamed ViewAction button name, but it was still the same "View Order"
The config dosent work.

What is the use of this Config field?


Bugfix/azurerm downgrade

azurerm Downgrade and Terraform Deployment Testing
Temporarily downgrading the the azurerm Terraform provider to fix a bug preventing deployments.
Also adding a Terraform test deployment workflow to prevent future PRs from breaking deployments.  This involves adding a new reusable GitHub Action workflow that deploys the Terraform.  We use it for the this PR flow and for our original staging deployment.  This test deployment only happens when we edit anything in the /operations/ folder.
Issue
#79.

Ah, yeah.  Hah!  I'll approve, and you can merge.


Focused section - fold all other sections beside current one

Helpers plugin has a lot of sections (Commands, Scripts etc)
When working with given section it is need to fold all other to allow easy operation on given section.
Button for focusing on one can speed up work and improve focus of on chosen activity.
Button can be placed as refresh button in SHOW TRACE.

It looks like this function is not available right now.
https://stackoverflow.com/a/73347770/4285480
https://github.com/microsoft/vscode/issues/88219
https://stackoverflow.com/questions/72170016/i-want-to-collapse-a-vscode-tree-view-programatically/72174813#72174813


Smooth line joins

libGDX provides no line joining utilities/settings.

http://www.bluevoid.com/opengl/sig00/advanced00/notes/node290.html
The following SO answer shows how to compute Milter join points (in general terms, although the question was about libGDX in specific).
An easy round join solution:

Draw a circle centered at the common vertex with radius half the line width. Sounds easy but if not drawn carefully, there would be serious overdraw.

http://artgrammer.blogspot.com/search/label/opengl

If drawing a curve, a texture can be applied to triangle strips. Example:

https://badlogicgames.com/forum/viewtopic.php?f=11&t=18940

Update
libGDX renders lines using OpenGL.

One of the sad things about using the OpenGL rendering option in Processing is the lack of control over line weights, caps and joins.

http://www.tom-carden.co.uk/2008/02/09/new-processing-hack-line-caps-and-joins-in-opengl

Unfortunately, drawing lines is a weak point of OpenGL. The GL_LINES drawing mode is limited: it does not support line joins, line caps, non-integer line widths, widths greater than 10px, or varying widths in a single pass.

https://blog.mapbox.com/drawing-antialiased-lines-with-opengl-8766f34192dc

Maybe the simple solution is still good enough:
Draw a circle centered at the common vertex with radius half the line width.


assert.equal() missing arguments missed by unassertify

Expected behavior
unassertify should remove all assertions in the code
Actual behavior
Call to assert.equal on https://github.com/yoshuawuyts/choo/blob/master/index.js#L120 isn't passed any arguments and appears to be missed by unassertify. This causes an exception when app.toString() is called.
Steps to reproduce behavior
Build a Choo app that calls .toString, run unassertify on it, run the app.

good catch, sorry about this :'(
On Thu, Jun 1, 2017 at 8:33 PM Dan Wilkerson<EMAIL_ADDRESS>wrote:

Expected behavior
unassertify should remove all assertions in the code
Actual behavior
Call to assert.equal on
https://github.com/yoshuawuyts/choo/blob/master/index.js#L120 isn't
passed any arguments and appears to be missed by unassertify. This causes
an exception when app.toString() is called.
Steps to reproduce behavior
Build a Choo app that calls .toString, run unassertify on it, run the app.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yoshuawuyts/choo/issues/509, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACWlehTz12iRf3egRdxsZszFsR5m9f1fks5r_wPpgaJpZM4NtXst
.


Period 3 Brown, Cipher Team

Our frontend page mostly completed it has all of our levels including our home page

This pull request cannot be accepted.
Reason: Out of Zone changes (index.md and config.yml were edited)


Upload of output directories within container

Currently the upload of output files & directories is done outside the container. In some cases it may make sense to do this from within the container (?) However, if there are multiple tasks, what to do?

Closing this. Will re-open if there is an actual use case for this.


Can't see comments or favorites

Comments aren't visible at all and the only favorites that show up are the ones i added through the app (Yes i'm logged in, and have restarted the app)

This can be a both a new problem or an code error @Dar9586
By the way whats your app version/website selection (nhentai.net or nhent.ai)/and whether you use VPN or not.
@Bannkora

For the comments part they seems to work, could you provide an ID of a gallery where it doesn't?
Same for the online favorites, once logged they shows up, at least in the last version, in the android browser your favorites shows up?


Feature/get institution project endpoints

@Eronavrp Can you change the structure to the new one with services? And then we can merge this PR.

@Eronavrp Can you change the structure to the new one with services? And then we can merge this PR.


New version: Zoom.Zoom.EXE version 6.1.1 (41705) (<IP_ADDRESS>705)

Created by 🥟 Dumplings in workflow run #6269.

Log
Updated: <IP_ADDRESS>135 → <IP_ADDRESS>705
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/issues/142854


Poll devices in parallel

Fixes performance regression introduced by https://github.com/TestArmada/flank/pull/584

Codecov Report

Merging #586 into master will decrease coverage by 0.03%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #586      +/-   ##
============================================
- Coverage      76.8%   76.76%   -0.04%
  Complexity      608      608
============================================
  Files            81       81
  Lines          2250     2251       +1
  Branches        320      319       -1
============================================
  Hits           1728     1728
- Misses          308      311       +3
+ Partials        214      212       -2


Example image (possibly a gif)

Can you provide a screenshot or a screenrecording of the library.

A screencast (posted on youtube) showing various use cases would be even better.


Add Button Components

Add amazing button components in this section
This commit introduces several new button components designed to enhance user interaction within the application.

Added primary and secondary button styles.
Implemented hover and focus states for better accessibility.
Included customizable properties for flexibility in usage.

These components aim to improve the user experience and maintain design consistency across the application.
Here are the buttons:

if there is anything to change , please tell me

@biswajit-sarkar-007 I'm merging this code.
Don't Forget to star the repository!!!
Keep Contributing!!!


How to get depth of a custom BBOX or by passing a  2d-point (x,y) to Oak-D depthai implementation.

Hi, Recently got one Oak-d-Poe, I've gone through the depthai example.
In Face detection+depth example , it's giving depth(z-coordinate) of face detection(BBOX).
What if i need depth of a keypoint(that can be any 2d-pixel coordinate(x,y)).
Or, how to get depth from a custom BBOX.
If someone has done it or has any reference, please share.
Thanks.

Hi @s91-maker .
Please have a look at Spatial Locaton Calculator, which does exactly that:)
Thanks, Erik


Near duplicate Image detection

Hello, first of all thanks for creating this package It is really good package for detecting Duplicate images.
I have tried this package I have found that it is able to detect images which are 100% similarity but I have found that it was not able to detect the images when similarity is not 100% even if similarity is 99.99% or less not able to detect image.
I have tried to play with the pixel values and similarity but than also it was not able to detect. So, is there ways to detect such image which having similarity score less than 100% by using difpy package.
I have attached few images which it was not able to detect.
Note:- The percentage values which I have refereed many times found from matchTemplate method the images which are attached having similarity is 99%.

Hi @dhruvbhatnagar9548,
Thanks a lot for opening the issue and for the feedback!
I just tested difPy on your images, and indeed it does not recognize them as duplicates. This is because of to the difference in tensors that the images have: due to the frame on some images, and the crop difference. DifPy computes the following MSE values for your images ("weareone" = a, "bus" = b):

a1 and a2 = 7298
a1 and a3 = 13218
b1 and b2 = 9774

DifPy only considers images as being similar if they have an MSE of less than 1000 (with the similarity parameter ='low'). Therefore, in order to solve your issue, you would need to adjust the MSE threshold. You can do this yourself by forking the difPy code and adjusting it, or wait until the next difPy release which will include an option to set the MSE threshold value directly from the similarity parameter.
I hope this helps!
Again, thanks a lot and all the best,
Elise

Thanks for the reply. I will try the solution that you have mentioned. Thanks.


Event Listener

You have really done a nice job but I can see some issues that needs to be improved. Instead of using multiple click event listener, I sugest you use event delegation.
Multiple event listener increases memory consuption and performance overhead.

Great work so far however there are some areas that need improvement.
Click events only work for desktops. Using touchStart will make your webpage work across all mobile platforms in other browsers.


The lack of tensor-train RNNs for latest tf/keras

Hey, pardon for posting a similar issue twice, but I think this matter needs to some attention.
There are no proper working tensor-train RNNs for the latest tf/keras which is very odd to me. It would  be fantastic to process sequences with this kind of power and speed.
As this is the most popular and active tensor library maybe you guys could give some serious thought on this matter?
Best regards
Martin

I agree with that. Tensor-train RNNs perform well on several tasks.


Reorder reports

Closes #178

Codecov Report

Merging #181 into master will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #181      +/-   ##
==========================================
+ Coverage   98.75%   98.84%   +0.08%
==========================================
  Files          31       31
  Lines        1929     1899      -30
==========================================
- Hits         1905     1877      -28
+ Misses         24       22       -2

Impacted Files
Coverage Δ

robocop/checkers/spacing.py
100.00% <ø> (+2.70%)
:arrow_up:

robocop/checkers/__init__.py
86.66% <100.00%> (ø)

robocop/reports.py
100.00% <100.00%> (ø)

robocop/run.py
97.46% <100.00%> (+0.09%)
:arrow_up:

tests/utest/test_thresholds.py
95.45% <0.00%> (-0.20%)
:arrow_down:

robocop/checkers/duplications.py
96.82% <0.00%> (-0.05%)
:arrow_down:

robocop/checkers/lengths.py
97.94% <0.00%> (-0.03%)
:arrow_down:

robocop/checkers/naming.py
99.25% <0.00%> (-0.01%)
:arrow_down:

robocop/config.py
100.00% <0.00%> (ø)

tests/e2e/test_e2e.py
100.00% <0.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c858f16...c7d4420. Read the comment docs.

Closes #188


How do you to turn off and on (disable/enable) a channel that is stuck in a feedback loop?

Hi,
I have a question about what I can best do when the servo is stuck in a feedback loop. When I set a target for the servo channel, it sometimes happens that the target it not fully reached and the servo keeps trying to get to the indicated position. This has happened for different target values. Unfortunately, I cannot tell it to stop the PWM sending, as only one of the channels has that option, and I plan on using two channels. I tried to fix it with this code, by simply restating the target when getting close to it, but it doesn't always solve the problem:
if abs(servo.getPosition(0) - target) < 5.:
    servo.setTarget(0, servo.getPosition(0))
    servo.close()

Even when you close the servo, the servo keeps working and you have to disable it manualy in the Maestro Control Center. Besides, you'd rather not have to fully close and reconnect the servo after each target statement, but rather only temporarily the channel that is experiencing issues. It's just that I don't know how best to define the rapid back-and-forth positions when it's stuck, since you don't have an iterative overview of all positions as the target it set.
I've tried using the fact that it's moving as a control condition, but while jammed/stuck in the loop it claimes it's not moving, even though you can hear it vibrating and working. This makes it difficult for me to pinpoint a solution, other than "keep the Maestro Control Center at hand in case of emergency".
In the Maestro Control Center I've also experienced this same problem, but there I could avoid damage to the servo controller by manually disabling the respective channel. Is there a way to also do this through python? Or are there other suggestions?
Are the settings like speed and acceleration relevant for this? I tend to use v=10 and a=5, but have this problem for different values as well.
Thanks!
Vera

Discussed in the "How do you immediately stop a servo's movement?" issue. Might be something that should be solved in a different way than through the Maestro code.


build: collect the build artifacts in a central location

This structures the build for Foundation similar to LLVM and most of the
other libraries.  Having a regular structure makes it easier to find the
artifacts and makes it easier to automate builds.

@swift-ci please test

CC: @millenomi @drexin @spevans

Please test with following PRs:
apple/swift#3489
apple/swift-corelibs-xctest#310
@swift-ci please test

Please test with following PRs:
apple/swift#33489
apple/swift-corelibs-xctest#310
@swift-ci please test


Add the abilty to set WEB_SERVER_NAME/ALIAS and WEB_INCLUDES in ...

... php-apache images

tested against #336 Spryker Apache images, and works as expected

tho YVES_WEB_SERVER_NAME not being rendered :(

appears to be a bug in that PR rather than this one


Alarm Acknowledgement Handling and Override Issues

There is an issue with the handling of alarms that require acknowledgment. Specifically, when an alarm is raised with acknowledgment required, and the same alarm is raised again without acknowledgment, AlarmWatcher no longer requires acknowledgment. This results in the user potentially missing out on important alarm.
The situation becomes even more problematic when an alarm is raised with high priority and requires acknowledgment, but is then overridden by a new low priority event without acknowledgment, causing the alarm to be completely missed.
Expected Behavior:

If an alarm is raised with acknowledgment required, it shall always maintain the acknowledgment state until it is manually acknowledged.
The alarm class of an ack alarm shall only be allowed to increase in priority, not decrease (this may need further consideration).

Proposed Solution
The solution involves adding three new columns to the alarm database:

time_after_ack
class_after_ack
state_after_ack

When an alarm is raised again while it already exists, the behavior depends on the req_ack flag:

req_ack = false
The alarm is overwritten as usual.

req_ack = true
Additional checks are performed:
a) New alarm has the same alarm_class as the existing one: The alarm is simply overwritten (except req_ack, which remains true).
b) New alarm has a higher alarm_class: The alarm is updated with the higher class and the new alarm state (except req_ack, which remains true).
c) New alarm has a lower alarm_class: Only delay and duration are updated. alarm_class, raise_time, and alarm_state remain unchanged but are saved in the new columns (time_after_ack, class_after_ack, and state_after_ack).

Handling Acknowledged Alarms
When an alarm is acknowledged, the system should check if the new columns (time_after_ack, class_after_ack, state_after_ack) contain data. If they do, the data in these columns should move to the main columns, overwriting their values.
Delay Behavior
Delays should be handled carefully. Delays will still apply as usual.
If a delay results in a change to the alarm_class, the same process outlined above should be followed.
Additional Observations
It was also noticed that the issue extends to clearing alarms:
It should not be possible to clear an alarm that still requires acknowledgment (req_ack = true).
Currently, it is possible to clear such alarms, which needs to be addressed.


More Error(UnsupportedType) Feedback

Really helpful library, thanks so much for your work on it.
This issue is borne of https://github.com/dan-fritchman/Layout21/issues/33, in which:

I have a big tree of structs, with lots of fields, internal types, and external types along the way
I pass the biggest top-most one to toml::to_string()
What comes back is this:

failures:

---- tests::empty_lib_to_toml stdout ----
thread 'tests::empty_lib_to_toml' panicked at 'save failed: Error(UnsupportedType)', gds21/src/tests.rs:214:50
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Question: is there any good way to get more than UnsupportedType as feedback here?
(Set a setting, submit a PR to improve the error message, whatever.)
Thanks again!

Yes, ideally we report more details to help people with this.  However, a lot of things will be changing with #340 which I'm actively working on so I'd like to hold off on other changes until then.

Ok, cool, understand.


installing spconv

hi, Could i use
pip install spconv-cu102

command to install spconv package?
thanks

yes, as long as your cuda version is 10.2


closure: make it build on Windows x86_64

Windows x86_64 is a LLP64 environment.  unsigned long is not size_t.
Adjust the parameter to be the correct size for the allocator.  This is
compatible with the previous implementation on LP64 environments.
Prefer to use the __sync_bool_compare_and_swap over the MSVC
intrinsic which is overly strong.  While in the area, remove an unused
compare and swap method.

CC: @parkera

@swift-ci please test

@swift-ci please test
The test failure seemed unrelated

@swift-ci please test

@swift-ci please test

Seems reasonable. I thought we were actually depending on the swift-corelibs-libdispatch version of this file now, though.

Hmm, actually, yeah, the size_t probably is better.  Hmm, using the version that is in libdispatch is fine, but, we should then have a single copy of the sources.  Would it be okay to remove these then?

Switched to size_t.  I took a look at the build.py.  Seems that if LIBDISPATCH_SOURCE_DIR is not specified, then it will fall back to this source.  Is it really possible to build CoreFoundation/Foundation without libdispatch at this point (i.e. is that just dead code)?

@swift-ci please test

@compnerd I have been wondering that myself recently as well. We did need this during bringup (dispatch was not yet ready), and I wonder if it is still useful and still working for other platforms that are in that stage.

Like Windows 64 bit, apparently. :)

@swift-ci please test

@swift-ci please test

That failure is really weird, its not at all related to this change, and the crasher there doesn't even reference Foundation.

@swift-ci please test

@swift-ci please test

@spevans guess it likes you better than the rest of us :-p.  @parkera good to merge?  I have the mirror'ed commit in libdispatch up as well so that we don't drift apart until we can unify the codebase.

Yup, ok.


New-SSHSession Credential passing failure

I am getting the error:-
New-SSHSession : Sequence contains more than one element
when trying to pass in a credential to a New-SSHSession
I can create a new session if I use the cmdlet without specifying a -Credential parameter.
$cred   = Get-Credential -username Oracle -Message "Oracle password for server"
$sess  = New-SSHSession -ComputerName $server -Credential $cred -AcceptKey -Verbose
VERBOSE: Using SSH Username and Password authentication for connection.
VERBOSE: Fingerprint for<server.dc.dc.com>: aa:bb:cc:dd:ee:ff:00:10:ad:dd:94:4d:56:d8:b4:41
VERBOSE: Fingerprint matched trusted fingerprint for host <server.dc.dc.com>
New-SSHSession : Sequence contains more than one element
At line:1 char:14
+ ... sess   = New-SSHSession -ComputerName $server -Credential $cre ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [New-SSHSession], InvalidOperationException
    + FullyQualifiedErrorId : System.InvalidOperationException,SSH.NewSshSession

I have tried various options, but keep getting the same error message.
What am I doing wrong?
Thanks

upvote, getting the same issue but intermittently

In my case the problem was the wrong username/password combination.  Kind of a confusing error message.

Weird in my test with a bad password, none existent user, none permitted user all gave me permission denied and the method of auth in the error
PS C:\Users\Carlos Perez\Documents\ADAudit> New-SSHSession <IP_ADDRESS> (Get-Credential carlos)
New-SSHSession : Permission denied (password).
At line:1 char:1

New-SSHSession <IP_ADDRESS> (Get-Credential carlos)

  + CategoryInfo          : NotSpecified: (:) [New-SSHSession], SshAuthenticationException
  + FullyQualifiedErrorId : Renci.SshNet.Common.SshAuthenticationException,SSH.NewSshSession

PS C:\Users\Carlos Perez\Documents\ADAudit> New-SSHSession <IP_ADDRESS> (Get-Credential carlo)
New-SSHSession : Permission denied (password).
At line:1 char:1

New-SSHSession <IP_ADDRESS> (Get-Credential carlo)

  + CategoryInfo          : NotSpecified: (:) [New-SSHSession], SshAuthenticationException
  + FullyQualifiedErrorId : Renci.SshNet.Common.SshAuthenticationException,SSH.NewSshSession

PS C:\Users\Carlos Perez\Documents\ADAudit> New-SSHSession <IP_ADDRESS> (Get-Credential root)
New-SSHSession : Permission denied (password).
At line:1 char:1

New-SSHSession <IP_ADDRESS> (Get-Credential root)

  + CategoryInfo          : NotSpecified: (:) [New-SSHSession], SshAuthenticationException
  + FullyQualifiedErrorId : Renci.SshNet.Common.SshAuthenticationException,SSH.NewSshSession

On Mar 11, 2016, at 7:52 PM, karezza<EMAIL_ADDRESS>wrote:
In my case the problem was the wrong username/password combination. Kind of a confusing error message.
—
Reply to this email directly or view it on GitHub https://github.com/darkoperator/Posh-SSH/issues/90#issuecomment-195606417.

against what OS and type of SSH server is this?

Closing issue since no reply has been received. Please re-open if still having the issue


Vue.js support ?

Hi.
Is Vue.js single-file template support considered?
It would mean scanning .vue files and then considering only the TS code inside the

Right now, it's not really planned ;-)

Ok thanks for for the answer.
I've just managed to make the Reorganize feature work because I really
wanted it but it's poorly done, for instance I'm replacing everything above
the JS code with spaces so the calculated indexed are preserved.
I'd like to work on Vue support but I'm not sure I'll have enough time atm.
On lun. 2 oct. 2017 à 17:32 Christoph Bühler<EMAIL_ADDRESS>wrote:

Right now, it's not really planned ;-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/buehler/typescript-hero/issues/302#issuecomment-333571262,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE4EFM_c9bevydhG7fDms9VsXYkZqs3ks5soQILgaJpZM4Pe7GY
.


No display of another "Given" in Scenario

If in "Background" section a "Given" is listed, the Scenario "Given" is not displayed with CukeManager.

need more details eg.: screen shot and example scenario.

Severity: HIGH
Priority: HIGH
Reported By: Leila
Reported On: 07-04-2015
Reason: Defect
Status: New
Environment: Windows 7
Description:
"Given" section is not displayed with CukeManager.
Steps To Reproduce:

Open "Feature:UpdateServer - TaskManagement - flow of tasks - Initiate and finish" -> "New task for "Files upload" process" Scenario in Test Scenarios editor AND in CukeManager.

Expected result:
The Scenario starts with " Given User logs in as "pm", "pm" " in the CukeManager as in Editor is displayed
Actual result:
The Scenario starts with " And User starts "Upload new file" process on "Files" section " in the CukeManager.

Fixed.


Undefined Method getTargetUrl()

My tests began breaking with the errors, [Symfony\Component\Debug\Exception\FatalErrorException]   Call to undefined method Illuminate\Http\Response::getTargetUrl() after I migrated to Laravel 5.2.35.
I began to debug the error until I figured out that the error was caused by a call in the trait Illuminate\Foundation\Testing\Concerns\InteractsWithPages::followRedirects()
while ($this->response->isRedirect()) {
            $this->makeRequest('GET', $this->response->getTargetUrl());
        }

When I reverted to 5.1.36, all my tests ran successfully without the errors above.

This method is in "RedirectResponse" class of "symfony/http-foundation": http://api.symfony.com/3.0/Symfony/Component/HttpFoundation/RedirectResponse.html#method_getTargetUrl
Check that all composer packages are updated.
I have tried an upgrade from Laravel 5.1.36 to 5.2.35 and all works fine for me.


Is this repo actively maintained? If not, suggestions for a fork that is? or interested in another maintainer to help?

We are seeing some build warnings here and wanted to see about fixing them but this looks unmodified for years now.
Anyone have a fork they are maintaining they'd recommend using instead? or an alternate library?

 Feel free to do any spring cleaning that's due :) The library used to build warning-free but that was years ago, and compilers have become better over the years.

Also - even despite the lack of attention, I've been using the library in commercial project with success. It simply became feature-complete enough for me to stop adding new things - but of course there are still features/bugs that need work. I stopped paying active attention when the old CI setup broke and I couldn't easily verify changes; of course the world has changed and we have GitHub Actions now - but there's still the matter of setting things up...
PS. I am totally an idiot on the internet :D

Heheheh
@kosma would you mind adding my private account as an maintainer? https://github.com/chmorgan
We use this library a bit and having it maintained is important to us so it makes sense for us to help out. Let me check on the CI setup as a first step, Github actions is pretty helpful.

@kosma and you can reach me at<EMAIL_ADDRESS>since I think github, for whatever reason, doesn't provide a way to reach out to users directly.

@kosma and I'm reaching out to some of the active forks with interesting improvements to see if they'd be interested in pushing their changes upstream.


Ubuntu binary release again (last release is 1.12.1)

Description of issue
On Ubuntu 16.04 and some other friends, binary debian packages are available but the version, 1.12.1 (in early 2017) IINM (and made through ROS buildfarm around https://github.com/ros/rosdistro/pull/13649), is very old compared to the latest SDK release 2.19.1. And recently for ROS2 Crystal (which runs on Bionic for Ubuntu), 2.16.5 was released but not for the older ROS distro.
Suggestion
Is there a plan to make backport releases to older platforms, e.g. Ubuntu 16.04, which is LTS so will be supported by Canonical by 2021? I apologize if there has been discussions about the release policy, and I understand release policy is totally up to the community, but it'll be quite convenient if new releases can be made into older LTS.
Particularly early 2017 was still very early for librealsense. So even just making one binary release, can be final, will be beneficial to the community IMO.

As I mentioned in #3373, I finished the packaging in the following way. I created librealsense2, librealsense2-dev and rs2-utils packages. I'm thinking in adding the udev rules file in another package and add a dependency in librealsense package.
When I have it I will create a PR to include it in the master branch.

Not sure if this is the right thread. I've seen https://github.com/ros/rosdistro/pull/19958 contributing an entry to ROS2 crystal rosdistro, so I wonder what needs to happen to do the same for ROS1 melodic, so the ROS wrapper realsense2_camera pkg can depend on librealsense2 pkg provided on this repo, and how I could help to make that happen. Thanks.

How about trying below and build the ros wapper from source, which is maintained? Is it feasible?
https://github.com/intel-ros/realsense

How about trying below and build the ros wapper from source, which is maintained? Is it feasible?
https://github.com/intel-ros/realsense

@RealSenseCustomerSupport Thank you for the response. I'm not sure, however, if you're replying to a wrong thread or not. On this ticket we've been specifically talking about possibility of resuming binary release of librealsense, not the ROS wrapper (intel-ros/realsense).
If the release maintainers who did the release of earlier version are unavailable and the current maintainers need some help, I'm happy to assist (I'm an active Realsense user, and I've been making releases of many packages to rosdistro for 5+ years.

Hello @doronhi and @dorodnic,
any comments about this?

Hi @130s
Currently we publish packages to private PPA (see here)
In general, upstreaming it to Ubuntu official PPA is in our plans but we couldn't get to it yet, and as you mentioned, may lack some experience in that area.
My main concern is the DKMS package we publish to streamline Kernel patching. Any potential issue you see with that? Other packages are pretty straightforward, and we can definitely publish the relevant scripts we use.

@dorodnic

Currently we publish packages to private PPA (see here)

True. In my OP I referred to ros archive, but that's not "public" as it requires setting apt source, same as your ppa setup. So I updated the title to clarify what I requested.

My main concern is the DKMS package we publish to streamline Kernel patching. Any potential issue you see with that?

Do you mean sending pull requests to Linux Kernel repos? If so, that means even more review for the patch to be merged in (*1). I can't agree more.
*1...Please don't get me wrong...no criticism for the maintenance on this repo. My comment is made under assumption that review in Linux Kernel development is very mature and strict by nature.

@RealSenseCustomerSupport I appreciate the effort in opensourcing the software for your product. That means a lot to the users community I believe (so to your product maintenance too). Like many other tickets on this repo, however, I cannot appreciate you closing a ticket without discussion getting settled. On this ticket, the last post was from me asking some clarification https://github.com/IntelRealSense/librealsense/issues/3528#issuecomment-480713980 (cited below for the record).
I cannot even re-open this even I'm the original author. Please re-open, and also stop closing unclosed tickets.

My main concern is the DKMS package we publish to streamline Kernel patching. Any potential issue you see with that?

Do you mean sending pull requests to Linux Kernel repos? If so, that means even more review for the patch to be merged in (*1). I can't agree more.

@RealSenseCustomerSupport , could you cite where this was resolved? Else, please reopen.


Dev warning when can.Model is extended without static properties.

When can.Model is extended without static properties like:
Todo = can.Model.extend({
  findAll: "GET /todos"
});

This is almost always a mistake.  We should warn users with something like:

can/model/model.js: can.Model extended without static properties.

VOTE FOR THIS FEATURE HERE:  http://bithub.com/event/25835

This helped me discover something unusual. If you do this in canjs 2.2.5:
steal('can/model', 'can/map/attributes', function() {
can.Model.extend('HiThere',
// @Static
{
  attributes: {},
  findAll: function() {
    // doesn't matter
  }
},
// @Prototype
{}
)
})

It will throw this error and the model is empty, without definitions. Incomplete. Dead.
But I'm giving it the name, static props and prototype props.
The cause of this is the stealing of can/map/attributes, which breaks it completely and causes it to break. It is deprecated, but it's removed from docs but not from canjs package, so I didn't even knew it was deprecated :-DThis helped me discover something unusual. If you do this in canjs 2.2.5:
steal('can/model', 'can/map/attributes', function() {
can.Model.extend('HiThere',
// @Static
{
  attributes: {},
  findAll: function() {
    // doesn't matter
  }
},
// @Prototype
{}
)
})

It will throw this error and the model is empty, without definitions. Incomplete. Dead.
But I'm giving it the name, static props and prototype props.
The cause of this is the stealing of can/map/attributes, which breaks it completely and causes it to break. It is deprecated, but it's removed from docs but not from canjs package, so I didn't even knew it was deprecated :-D


[Docs] - Roadmap show wrong "latest version"

Describe the bug

The image in the roadmap incorrectly highlights Preview 3 as the latest version.

Steps to reproduce the bug

Steps to reproduce the behavior:

Go to https://github.com/microsoft/microsoft-ui-xaml/blob/master/docs/roadmap.md#winui-30-timelines-and-milestones
See that Preview 3 has an adornment indicating it as the latest release

Expected behavior

Preview 4 is the latest released version and should be indicated as such.
Screenshots

see above.
Version Info

n/a
NuGet package version:

Windows app type:

UWP
Win32

n/a
n/a

Windows 10 version
Saw the problem?

Insider Build (xxxxx)
n/a

October 2020 Update (19042)

May 2020 Update (19041)

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
n/a

Xbox

Surface Hub

IoT

Additional context

@anawishnoff FYI


IK control frame

Thank you for sharing this great codebase!
I have a question regarding the control frame in sawyer.
What is the reason why cartesian 6d control is expressed in the world's orientation?
https://github.com/deepmind/dm_robotics/blob/667a0776fbbe217867fc65d31a3de6749bbf3d54/py/moma/effectors/cartesian_6d_velocity_effector.py#L380
Is there a simple way to modify it to express velocity targets in the local orientation frame?

Hello,
This is configurable.  You should use control_frame of the model_params argument.
Internally, the cartesian controller uses another controller to convert from cartesian velocity to joint velocities (it is this controller that has the world frame orientation assumption).
If you pass a control frame, then the cartesian command will be converted from that frame to the one that the underlying controller expects.  This will achieve the result you are looking for.

Just to round this off, a control_frame like geometry.PoseStamped(frame=my_mujoco_element) might be the control frame you want.  If you wanted to control a point that is off the element itself (E.g. a point in the middle of a gripper or in front of it) then you could also supply a pose to the PoseStamped initializer that would then be an offset from the frame itself.

Closing, assume resolved.


Only apply replication updates if the new value is different from the existing value

This could be useful to avoid triggering change detection

might require a PartialEq bound. Or should we make PartialEq optional?

This should probably use the equality_fn, not PartialEq?

Maybe, if we go though the optional PartialEq work

The problem with the optional PartialEq approach is that it would add more verbosity to the protocol:
app.register_component::<C>()
     .add_default_equality_fn();   // available only for types that are PartialEq

or
app.register_component::<C>()
     .add_equality_fn(equality_fn);

Also the equality function for change detection during replication might not be the same as the one used for rollback checks (the thresholds might be different).
One solution could be to require all networked type to be PartialEq

Maybe requiring PartialEq is easiest for now

I assumed PartialEq was already necessary for this kind of thing, so I don't think it'd be too much of an imposition to require it now. Users who need custom behaviour can always newtype it and provide their own PartialEq implementation, right?


Blender: pre pyside install for all platforms

Brief description
Pre pyside install could work for all platforms not only windows.
Description
pre_pyside_install.py hook is updated to get the blender python executable path for Darwin and Linux platform and install pyside using a subprocess python pip install.
issue : #3399

I'll do one more test, then we can merge.


Echoing DB_URL environment variable may leak secrets

Describe the bug
In reflex-0.2.5, overriding the config with environment variables logs a message to the console containing the value. Since the DB_URL likely contains the database credentials, it shouldn't be echoed to the logs.
To Reproduce
DB_URL=postgresql+psycopg2://postgres:secret@db/postgres reflex run

Info: Overriding config value db_url with env var DB_URL=postgresql+psycopg2://postgres:secret@db/postgres

Specifics (please complete the following information):

Python Version: 3.11.4
Reflex Version: 0.2.5
OS: -
Browser (Optional): -

Picking this up

Thank you 🙏

Can this be reviewed?
https://github.com/reflex-dev/reflex/pull/1681


NODE_ENV not inherited from .env file when running in a different mode

Version
3.7.0
Reproduction link
https://github.com/RickMeijer/bugreport-vuecli-environment
Environment info
  System:
    OS: macOS Mojave 10.14.4
    CPU: (8) x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz
  Binaries:
    Node: 10.13.0 - ~/.nvm/versions/node/v10.13.0/bin/node
    Yarn: 1.15.2 - /usr/local/bin/yarn
    npm: 6.9.0 - ~/.nvm/versions/node/v10.13.0/bin/npm
  Browsers:
    Chrome: 74.0.3729.131
    Firefox: 66.0.3
    Safari: 12.1
  npmGlobalPackages:
    @vue/cli: 3.7.0

Steps to reproduce
Either:

Clone project
npm run build
npm run build -- --mode=foo

Or:

Create new project
Create .env
In the .env file, set NODE_ENV to production
Create empty .env.foo
Run npm run build -- --mode=foo

What is expected?
NODE_ENV should be 'production'
According to the docs; all environments load the .env file. That should mean it also inherits the variables set therin.
What is actually happening?
NODE_ENV is 'development'

When setting NODE_ENV in the [mode] file, everything works as expected.
It becomes impossible to set a default though contrary to every other environment variable.

According to the official wiki, when you do npm run serve without --mode, it neans --mode=development. So process.env.NODE_ENV is 'development', even though you define it production in .env.
In this case, you can make a .env.development like below:
NODE_ENV=production TITLE=inheritance works (from .env.development)
Then do npm run serve output console:
Current build environment is production
Fortunately, inheritance works (from .env.development)
wiki:
https://cli.vuejs.org/guide/cli-service.html#vue-cli-service-serve
https://cli.vuejs.org/guide/mode-and-env.html#modes

Here is a basic reproduction of the issue: https://github.com/GiantZOC/vue-cli-environment-reproduction
I created a new project with vue-cli, added a .env file and then tried to output the value in the console with main.js.


Add project last update stdout

Add method to Project model that fetches stdout capture of its last update job. It uses 'related' property of the Project. Returns nil if there is no update job for the Project.
Fetching and presenting this information is a workaround for cloning repositories from HTTPS servers with untrusted SSL certificates. Tower API does not offer an option to disable SSL check, so with the stdout user can at least see, why the cloning actually failed.
Alongside fix some subtle bugs in the Api class:

Non-JSON responses were not logged correctly. This bug was hidden because with the default log level debug logs are skipped.
Faraday was not available in the Api class, resulting it raising LoadError while catching Faraday::* exception.

https://bugzilla.redhat.com/show_bug.cgi?id=1513616
Required for #72 in manageiq-providers-ansible_tower.

Not sure whom should I ask for review. Maybe @bdunne or @jrafanie?

I fixed the require’s and the respond_to?’s. I also simplified the last_update method to return the whole object and not only the stdout. This avoided URI building. I stuck to using related['last_update'] though as they are used quite a lot in this gem and this approach is IMO more readable than building a custom query.
Is it better now, @jrafanie?

@jrafanie I squashed the commits. I also bumped the version from 0.13.0 to 0.14.0 so our manageiq-providers-ansible_tower “gem” can depend on the new method.

@Glutexo please mention the people requesting changes if you're done implementing the changes... We don't always notice pushes to your PR branch or know if you're finished.  Thanks!


Provisioning template failed: App catalog not found

Category

[ ] Question
[ ] Typo
[X] Bug
[ ] Additional article idea

Expected or Desired Behavior
Provision the template when the app catalog is existing in the tenant (for more than 30 minutes).
Observed Behavior
Provisioning Microsoft 365 learning pathways template failed with error message "In order to provision the template you need to have an App Catalog in your tenant. Please, create one (for instructions you can read this document: https://go.microsoft.com/fwlink/?linkid=2087251), wait up to 30 minutes, and try again. "
PnP Correlation ID: 5802fd27-629f-4c83-8dc6-596e211cbaa3
Steps to Reproduce
Go through provisioning process on page: https://provisioning.sharepointpnp.com/home/details?packageId=3df8bd55-b872-4c9d-88e3-6b2f05344239
App catalog has been available for at least a week.
Provisioning fails.

Effective June 2019, the official guidance is to post all issues related to the PnP provisioning service to the following GitHub repository: https://github.com/SharePoint/sp-provisioning-service.
We've transferred your issue to the issue list for that repo. In the future, please ensure you use the https://github.com/SharePoint/sp-provisioning-service repo's issue list for all issues, questions & bug reports.


Add information about WebJobs SDK logging setup to this main App Service logging section

This section (add log messages in code) should contain a link to the section about WebJobs SDK logging, because special considerations need to be taken:
https://docs.microsoft.com/en-us/azure/app-service/webjobs-sdk-get-started#enable-console-logging

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: bfc316af-0d30-b449-a04f-3019b7a16bc1
Version Independent ID: 01c4c435-fda9-374a-70b3-f1d7f13c7651
Content: Enable diagnostics logging - Azure App Service
Content Source: articles/app-service/troubleshoot-diagnostic-logs.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thanks for your feedback! We will review and update this doc shortly.

Thanks for the feedback I have submitted a PR to update the doc. The change is merged internally and will go-live in the public docs shortly.

Thanks you @Grace-MacJones-MSFT !! :) 👍


Fixing Ideafast FS CI

As it turns out, Minio server fails sometime in the CI environment, and tests improperly silently swallow the error.

There seem to be issues with the resource allocation on the CI environment too.

Mongo memory server is also unpredictably failing to start.


prevent a startup failure with invalid computed values

Scope & Purpose
Backport of https://github.com/arangodb/arangodb/pull/16639
Prevent a startup failure when there are invalid computed values definitions stored for a collection.
While it is not possible for a user to store an invalid computed values definition for a collection, an existing computed values definition may become unintentionally invalid after an upgrade, for example if in a future ArangoDB version the computed values' AQL query string becomes invalid for some reason (introduction of new keywords, disallowed language constructs etc.).
In that case, starting an instance would immediately fail because of the invalid computed values definition, and there is no way anymore to make it start.
This PR instead disables the computed values in that case, logs an error the log containing the problem and the original computed values description, and continues the startup. So that the invalid computed values definitions can be fixed later, but availability of the instance is not affected.

[x] :hankey: Bugfix
[x] :pizza: New feature
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification

Checklist

[ ] Tests

[ ] Regression tests
[ ] C++ Unit tests
[ ] integration tests
[ ] resilience tests

[ ] :book: CHANGELOG entry made
[ ] :books: documentation written (release notes, API changes, ...)
[ ] Backports

[ ] Backport for 3.9: -
[ ] Backport for 3.8: -
[ ] Backport for 3.7: -

Related Information

[ ] Docs PR:
[ ] Enterprise PR:
[ ] GitHub issue / Jira ticket:
[ ] Design document:

Found some issue with it. Please do not merge for now.


Update development dependencies

Upgrading dependencies to run on the latest version of OSX 10.13.2
Description
node-sass v4.5.3 -> v4.7.2 (v4.5.3 is not supported in latest version of OSX)
electron-spellchecker v1.2.0 -> v1.1.2 (v1.1.2 is the latest release - https://github.com/electron-userland/electron-spellchecker/releases)
Motivation and Context
Enables development on the latest version of OSX
How Has This Been Tested?
Verified that the project compiles and runs on OSX 10.13.2
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the code style of this project (run $ yarn lint).

Any update on this?


Enter/Shift Enter for text input

Enter now accepts text input; multiline input would require a rethinking of text label layout which is currently out of scope.


Migrate OnConflictStrategy to enum

OnConflictStrategy (used by @update and @insert) now is an enum instead of integers. Thereby, the compiler supports us and the library's users with its type safety.
No migration for end users needed.

Btw, I think we should do the same for the ForeignKeyAction.

Yes, I've started working on this and will open another PR soon.


Add --token-usage Flag to Display Token Usage Information

I would like to propose a new feature that adds a --token-usage or -t command-line flag to the tool. This flag would output information about token usage to stderr, helping users better understand how many tokens were used during both the prompt and the completion.
Feature Details:
When the --token-usage flag is set, the tool should:
*Output the number of tokens used in the prompt (e.g., prompt_tokens).
*Output the number of tokens used in the completion (e.g., completion_tokens).
*Output the total tokens used (i.e., completion_tokens + prompt_tokens).

The token usage information will be extracted from the response provided by the LLM provider (e.g., OpenAI, Groq, etc.).

If the flag is set, the tool will display the following information to stderr:
Prompt Tokens: 57
Completion Tokens: 17
Total Tokens: 74

Why This Feature is Useful:

Knowing how many tokens are used is important for both cost management (since most LLM providers charge based on the number of tokens) and for ensuring that the request fits within the model's context length.
This feature will allow users to debug and optimize their usage of LLMs, ensuring they stay within token limits and avoid potential errors caused by exceeding the token budget.
Proposed Approach:

Add support for the --token-usage or -t flag in the command-line options.
Extract token usage information from the LLM response (e.g., usage.prompt_tokens, usage.completion_tokens, usage.total_tokens).
Output the token information to stderr when the flag is enabled.

Let me know if you approve this feature request, and I would be happy to work on it!

Nicely done


Hotfix/subfolder middleware debug

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
#197

Added test case
SubFolderMiddleware renamed to SubFolder ( comment )

Ready?

Merged. Thanks!


Match results wrong for larger values of overlap

Steps to reproduce

Run RegexSearcherTests.MaxMatchCountTest with overlap set to pattern.Length and observe it fails

Note: suspect that deduping is not correctly handling the case where multiple matches are present in the overlap window

The real issue is that the maxMatchCount parameter is approximate.  The reason for this is that maxMatchCount runs before deduplication.  So when duplicate matches are discarded you end up with fewer matches.  I have documented the behavior in RegexSearcher.


Can't access data captured by StreamRecorder

HI,
I am testing out the StreamRecorder app. Was able to run fine on the device. Trying to get the data off the device and it doesn't work.
a. Using the device portal I get "failed to get files for package: ..."
b. Using the python code - what is the path?
I tried connecting via wifi and USB (the appdata folder is empty in USB but in wifi device portal it is there)
How can I get the data?

@sitzikbs  Hi, maybe you can look at the StreamRecorder\README.md, which provides a lot of helpful information. Actually, you can use python StreamRecorderConverter/recorder_console.py --workspace_path <output_folder>  --dev_portal_username <user> --dev_portal_password <password> to download data from hololens, and this #4 maybe helpful.

Thanks!
#4 worked (uncheck SSL connection from Preferences under system at Windows Device Portal).
Also, I found the documentation for workspace_path confusing - I thought it meant the path on the device (not on the local machine - but maybe it is just me).
closing now.


Don't use ssh to setup vclusters

[ ] Access control plane directly via local kubeconfig instead of ssh-ing into one of the control hosts
[ ] have the control attribute on vcluster hosts then actually mean, that they are allowed to run the vcluster's control plane, instead of meaning the host to ssh into (or yeet the attribute if not possible)

Allo opt in to ssh via specifying host cluster control hosts in control_hosts group explicitly


Copyright

Just wondering, am I able to use some of the skybox code in my own project? I'll just be using the fragment shader and the sky colour blending (from Skydome.cpp).

Yes go ahead! 👍
I'll add some licensing information to this repo too.

According to the LICENSE file, you are free to use all code under MIT license:
https://github.com/lragnarsson/Emerald-Engine/blob/master/LICENSE

Ha I totally missed that Lage added this a few hours ago (went straight from email notification) 🤗


Typo on site page - 'ligher' should be 'lighter'.

Under LINES, textures.lines.lighter() is misspelt.

Fixed, thanks.


[ENHANCEMENT] Add echo-server to stack

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the solution you'd like
Add echo-server to stack for monitoring
Additional Information
No response

Decided not work the lift as it is pointless


Use closefrom when available

Commit 0917867 could instead be handled by first regenerating the existing autoconf files with autoconf 2.69, then regenerating again after 0032b330ec171e838e561ed1d5ee502f7c869fa4.


Server Hosting/Specifications

This shall outline the key challenges, approaches & specifications for setting up the SB-RC in production.

Do we have anything documented at the moment? What is expected ETA of this?

@sukhpreetsamagra the work is in progress,
right now I'm creating the helm charts that can be used for the deployment

https://docs.sunbirdrc.dev/developer-documentation/installation-guide


Hyrule Warriors Gamepad Resolution

The Gamepad Resolution cannot be altered which was, as far as I know, possible in older versions.
I hope that feature will return, as it is absolutely critical for a awsome co-op experience.

I do not see a gamepad pack for hyrule warriors in the old v2 latest pack available. However it is easy to make a gamepad map, if ofcourse you mean just increase the internal resolution?

As far as I remember the old resolution graphic pack increased both resolutions.
The graphic pack doesn't seem to work.
I tried it with and without the normal resolution pack.

seems more like resolutions tied to gamepad (besides from the gamepad resolution itself) are not scaled at all...

That works.
may i suggest editing the description to state that it'll also change the gamepad resulution.

i think it'll be fine having it as is, considering there's other resolution packs that touch gamepad but don't mention that they do

alrighty and thank you.


Provide a pointer to stylish-haskell version in CONTRIBUTING.md

          I think the version we use is important. Perhaps we can "automate this" in the sense that we can provide a pointer to the place in CI where the version of stylish-haskell can be seen and therefore it would be always up to date.

Originally posted by @Jasagredo in https://github.com/input-output-hk/ouroboros-consensus/pull/1#discussion_r1061452709

Right now, that would be this place:
https://github.com/input-output-hk/ouroboros-network/blob/8def57b2f8ed306c9f32f888569c4ca5516a83b6/nix/pkgs.nix#L27

This is now the case: https://github.com/input-output-hk/ouroboros-consensus/blob/main/CONTRIBUTING.md#formatting-the-code
Currently, this has to manually be kept up-to-date with the "actual" version as used by Nix, but we only change it quite rarely, so that seems acceptable.
Also, CONTRIBUTING.md now has a section on how to easily inspect such things, i.e.
 $ nix eval --json .#stylish-haskell.version
"<IP_ADDRESS>"


includes and requires in the module broken

Continuing from discussion here
zend-application/library/Zend/Application/Resource/Modules.php line 78
zend-application/library/Zend/Controller/Dispatcher/Standard.php line 357
zend-validate/library/Zend/Validate/Hostname.php line 1780
zend-controller/library/Zend/Controller/Dispatcher/Standard.php line 357
zend-translate/library/Zend/Translate/Adapter/Array.php line 55
zend-view/library/Zend/View.php line 155 and 157
zend-http/library/Zend/Http/UserAgent/AbstractDevice.php line 787
zend-db/library/Zend/Db/Profiler/Firebug.php line 99
Maybe zend-cache/library/Zend/Cache.php on 148 and 190 as they refer to using include paths

@Shardj 1.13 incoming with sorted out autoloader incompatibilities

Thanks, I'm probably going to stick with my current repo though. It's proving popular and I don't want to have to spend time going and figuring out how to swap the projects.


Expression shouldn't be allowed as computed property name

Chakra doesn't throw a SyntaxError for this code:
({ [ 1, 2 ]: 3 });

but the specification defines ComputedPropertyName as [ AssignmentExpression ]

Similar to #147. @nicolo-ribaudo feel free to tackle this one as well at the same time.  Thanks for finding and reporting this.

@ianwjhalliday Should I open a different pr or can I just add a commit to #471?

wait, why would that throw a syntax error? I thought computed properties could be any JS expression - { ['foo']: 3 } is certainly valid, why wouldn't { [2]: 3 } be?

{ [2]: 3 } is valid, but { [ 1, 2 ]: 3 } isn't, because 1, 2 isn't an AssignmentExpression

@nicolo-ribaudo one PR for both is fine

@nicolo-ribaudo Is the comma operator not an AssignmentExpression?

@ljharb no it isn't. It is the only thing that is not an AssignmentExpression.  Expression is defined in the grammar as
Expression[In, Yield]:

AssignmentExpression[?In, ?Yield]
Expression[?In, ?Yield] , AssignmentExpression[?In, ?Yield]

gotcha, thanks for clarifying

I just pushed the fix at #471


Change the way sets and dicts are handled in the library

This pull request backports yet unmerged PR from hiredis and additionally changes the way dicts are handled.
This changes are necessary because RESP3 allows:

dict value to be a set
dict key to be a set or an array
have a dict inside a set

None of these things are allowed in Python. Therefore it was decided to replace sets with lists and dicts with lists of tuples. The client is supposed to handle the types and convert if necessary themselves.
This is a breaking change

huh, apparently PyPy works in a slightly different. Investigating.

When you say RESP3 dict, it's a RESP3 map, right? (Dict is a Python term, and a Valkey internal datastructure.)
Even if RESP3 allows complex map keys and set elements, I don't think Valkey ever returns values like that. Hm... COMMAND seems to be the exception. How about some special handling just for COMMAND, replacing the set with a list only for COMMAND's response?
I guess modules can return stuff like that too. We could possibly forbid modules from doing that in Valkey. That would be a breaking change too, but I think could be good to forbid values like that, not only for Python but also JavaScript and probably other languages too.
Alternatives:

Add an option to return list for sets and list-of-pairs for maps.
Simply fail if some response contains these values, if the new option is off.

Even if RESP3 allows complex map keys and set elements, I don't think Valkey ever returns values like that. Hm... COMMAND seems to be the exception. How about some special handling just for COMMAND, replacing the set with a list only for COMMAND's response?

AFAIK the API exposed by libvalkey-py is that of a simple RESP2/3 parser. With no knowledge about commands etc.
I think the best way to go is:

Add an option to return list for sets and list-of-pairs for maps.
Simply fail if some response contains these values, if the new option is off.

Thanks @aiven-sal and @zuiderkwast ! I'll fix it accordingly

we will just keep it always on in valkey-py

@aiven-sal so this only affects the communication beween valkey-py and libvalkey-py? It does not affect how valkey-py returns the replies to the end users?

we will just keep it always on in valkey-py

@aiven-sal so this only affects the communication beween valkey-py and libvalkey-py? It does not affect how valkey-py returns the replies to the end users?

No, it's only about communication between valkey-py and libvalkey-py. By "client" here I meant exactly valkey-py :)
There won't be any breaking changes in valkey-py as it's meant to be widely-used

I think I'll close this one in favour of creating a new one from scratch, as the proposed changes would require me to change even a commit that doesn't belong to me.


Could we expose the bool flag for calculating bounds in Mesh.SetVerts?

Describe the feature
I noticed the default case is true, which isn't able to be set to false from the C# interop.
Thanks! :)

Oh good call, thanks for catching that! This is in /develop now. I also added Mesh.SetData while I was in there, if you're ever setting both verts and inds at the same time, it's a small win too :)
c7e82b9c3a1df39ae035531a2f4032ca06c6b62e

Sweet, you're the man!

I'm gonna leave this open until it goes into a preview build, it's a little easier for me to track that way :)

Hey Nick! Is there a preview.4 release on the roadmap? It's not really blocking but just curious when to expect a Nuget release after this issue. Thanks :)

I've been doing a lot of work on a branch that hasn't gotten merged into /develop yet, so I haven't been shipping a ton of preview builds lately! I do preview builds either on request, or critical mass :D
I can look into doing a small release towards the end of the week to make this available, it has been a little long.


[Feature]May I grant permission? I used some code in my script

How to get some license？No contact information .

https://greasyfork.org/zh-CN/scripts/455741

https://greasyfork.org/zh-CN/scripts/455741


Solarized dark theme background colors off

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description

Notice the background blue color. It's one color around the cursor and then above and below it's another slightly different color. Here's it in the color checking tool built into MacOS:

Notice that the G and B in RGB change by 1 in between the two. It seems to be related to whatever is being drawn over the background. For instance it's only "blank" areas that ever change color. Over text or the cursor highlighting it's always the right color, if that makes sense. Changing the location of the text cursor to an area that's colored wrong changes it back to the right color. Lmk if you need any more information. I've used a bunch of different themes and only ever noticed this in Solarized Dark, never anywhere else. I also see in on both my iMac and MacBook.
Steps to Reproduce

Open Atom
Set Solarized Dark Theme
Sometimes see background color off

Expected behavior: No color differences
Actual behavior: Color differences
Reproduces how often: 1/10th of the time
Versions
You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.
⇒  atom --version
Atom    : 1.34.0
Electron: 2.0.16
Chrome  : 61.0.3163.100
Node    : 8.9.3

⇒  apm --version
apm  2.1.3
npm  6.2.0
node 8.9.3 x64
atom 1.34.0
python 2.7.15
git 2.11.0

MacOS: 10.14

Update
For others: I found it, setting the color profile in settings to sRGB fixed it :)


Fixing redundant check

In the last commit: Fix Optional's def causing issues It has a redundancy in the checks, preventing the error of missing arguments from being called. This way, when using a command without putting the arguments, the values are passed as null instead of sending missing arguments.
I partially fixed it by removing a piece of code that was previously removed. However, as I haven't tested all the features of the code, I don't know if it would cause any errors in the future, apparently it's working.

Appreciate the work. In the next version though, @Optional will no longer take a def value, as this functionality will be exclusive to @Default. Therefore I think it's better to just wait until the next release. Thanks for the heads-up though!


proto() method for objects

Should return the prototype of a given object.

I'll start on this now.

Done locally. Awaiting merge.


K9 Mail Dark Theme

Expected behavior
After setting theme to "Dark", I'm expected to see dark theme.
Actual behavior
Even though theme set to "Dark", I'm seeing light theme.
Steps to reproduce

K-9 Mail 5.112 @ Android 7.0 (PL:10/5/16), Nexus 6p
Settings -> GLOBAL -> Display - Theme = switch to Dark theme

Environment
K-9 Mail version: 5.112
Android version: 7.0 (PL:10/5/16)
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

See #1666

I'm unable to switch to dark theme outside of message and without using fixed message theme as well.
Is it really duplicate?

You're using different ways to trigger it, but it's the same bug.

Ok, thanks!