[channel,rdpecam] MJPEG input format support

Adding support for MJPEG camera format, using FFMPEG library decoder.
Higher end web cameras, supporting 720p and higher resolutions at 30 fps, are using MJPEG format. This allows to overcome USB 2.0 throughput limitation, that would otherwise be encountered with uncompressed formats, such as YUYV. Even laptop cameras typically use USB 2.0 bus internally, so MJPEG support is required for 720p and 1080p at 30 fps.

My 2 cameras that support MJPEG don't show this problem.
I don't insist on this change going in, in case MJPEG format is known to cause these issues. Or we can commit the code with
	{ CAM_MEDIA_FORMAT_MJPG, CAM_MEDIA_FORMAT_H264 },

line commented out or under some #ifdef.

Hum rather strange, I ran guvcview and it seems like capture works fine with MJPEG

@hardening : guvcview is using its own MJPEG decoder. We are using FFMPEG library decoder, which may be more sensitive to USB errors. Could you please provide more details:

your camera make/model;
is it USB external camera, are you using USB hub, long cable or anything special;
is the error always the same, i.e. error y=54 x=1 or you see different y and x values every time? Can you share a  log with multiple errors please?

If you share more details, I can try to find the same camera/model and hopefully reproduce it. Thanks!

@oleg0421 so my cam is a Logitech, Inc. HD Pro Webcam C920 (046d:08e5), connected on the monitor's USB hub. It works perfectly with guvcview, without any decoding errors of MJPEG (I mean I don't have any long cable).
The error position varies (30 < y < 54).
Debug logs: https://gist.github.com/hardening/eaf129889c90a13ea32aff9503bd7982 (running webcamtest site and then youcam)
In guvcview you're right they have their own JPEG decoder, I noticed that they check that the MPEG frame is at least 0xaf, and except if I read the code wrongly when decoding fails it's the previous frame that is shown. Anyway I've not seen any decoding failure error.

@hardening : thank you for sharing additional information!
Somehow, the log doesn't show any ffmpeg library output (no mjpeg_decode_dc: bad vlc: 0:0 error), but I can see many "Skip sample: no credits left" messages. Let me try to sort this out:

"Skip sample: no credits left" could mean you hit congestion in your network uplink. In the latest patch that went in, we optimized H264 bitrates, but you still need to have at least 1 - 2 Mbps uplink for camera redirect to work;

Regarding guvcview approach of showing the last good frame: IMO, masking USB layer 1 - 2 errors is not what you ultimately want to do. If USB setup has a problem, it will impact your video, network or other high bandwidth devices. You'd rather want to troubleshoot and eliminate this problem. Even with raw video formats (where there's no decoder), you'll see distorted video, artifacts, etc.;

I also have Logitech C920 (earlier revision 046d:082d) and I don't experience this error.

Could you please try to connect your camera directly to USB port on your computer (bypassing monitor's USB hub)?

@oleg0421 is this pull ready or still stuff to fix?

@oleg0421 with direct connection of the cam I get the same result.

@hardening: what do you mean by same result? Could you please share the new log when camera is connected directly? The log with mjpeg_decode_dc: bad vlc: 0:0 errors.

Just for tests I've tried skipping the mjpg to h264 transcoding (by setting mjpg -> mjpg in the convertion array), so that native mjpg is sent, but that doesn't change anything (always the no credits message).

To passthrough MJPG, the required uplink bandwidth could be around 50 Mpbs. As mentioned, no credits left message indicates uplink saturation.
Let's please not mix these two things up. Slow uplink is one problem.  mjpeg_decode_dc: bad vlc: 0:0  is a different problem, which may point to a USB hub with low throughput.

@oleg0421 is this pull ready or still stuff to fix?

@akallabeth: don't have enough information to troubleshoot @hardening's problem. Not sure what's wrong with proposed change (if anything). It would be helpful, if somebody else helps with testing (camera needs to support MJPEG format).
In general, adding MJPEG format is essential to support 720p and higher resolutions. But it may also require adding hardware acceleration (VAAPI) support for h264 encoding, because with higher resolutions h264 encoding shows high CPU load.

I gave a look at the code and I'm wondering if the pb is that when buffers are submitted to the sampleCallback, a full frame could be composed of multiple v4l buffers and so if we don't have credits, then we could possibly drop/mistreat just a piece of the encoded frame.
So when we're capturing raw traffic that gives artifacts (like before this PR), and when the capture format is more "encoded" (MJPG), that gives "unable to decode in ffmpeg".
Could it be an explanation @oleg0421 ? Perhaps we need another callback that is triggered when all the buffers have been treated and would be called here https://github.com/FreeRDP/FreeRDP/blob/master/channels/rdpecam/client/v4l/camera_v4l.c#L467 meaning "end of frame"  (and so we would send the sample in that callback ) ?

@hardening:

I gave a look at the code and I'm wondering if the pb is that when buffers are submitted to the sampleCallback, a full frame could be composed of multiple v4l buffers

In V4L2 API description:
https://www.kernel.org/doc/html/v4.9/media/uapi/v4l/io.html
it's not mentioned that single video frame can be stored in several buffers. When buffers are allocated, the size for each buffer is big enough to fit in entire frame based on dimensions and pixel format.
The purpose to have multiple buffers is for parallel processing, i.e. one buffer is being processed by the application, while the other is being populated by the driver with next frame. I suspect it would be enough to have 2 buffers for our case (single capture thread), but there are 4 just in case.

and so if we don't have credits, then we could possibly drop/mistreat just a piece of the encoded frame. So when we're capturing raw traffic that gives artifacts (like before this PR), and when the capture format is more "encoded" (MJPG), that gives "unable to decode in ffmpeg". Could it be an explanation @oleg0421 ?

My feeling is that in your case buffer has incomplete frame (tail cut off) due to USB bandwidth limitation / issue, but the missing part of the frame is not in another buffer, it's just lost. Please correct me if I'm wrong.

Perhaps we need another callback that is triggered when all the buffers have been treated and would be called here https://github.com/FreeRDP/FreeRDP/blob/master/channels/rdpecam/client/v4l/camera_v4l.c#L467 meaning "end of frame" (and so we would send the sample in that callback ) ?

Definitely, in case V4L2 API would have such a provision (that frame can be stored in multiple buffers), then we would have to reconstruct the frame before giving it to decoder. Maybe I don't see it in the documentation, please let me know.

@oleg0421 I just tested this PR in my thinclient project (https://github.com/jens-maus/thinRoot) and can say that RDP MJPG camera redirection works with this PR.
The camera that I am currently using is:
$ v4l2-ctl --info --list-formats
Driver Info:
	Driver name      : uvcvideo
	Card type        : C270 HD WEBCAM
	Bus info         : usb-0000:00:15.0-1
	Driver version   : 6.6.52
	Capabilities     : 0x84a00001
		Video Capture
		Metadata Capture
		Streaming
		Extended Pix Format
		Device Capabilities
	Device Caps      : 0x04200001
		Video Capture
		Streaming
		Extended Pix Format
Media Driver Info:
	Driver name      : uvcvideo
	Model            : C270 HD WEBCAM
	Serial           :<PHONE_NUMBER>01
	Bus info         : usb-0000:00:15.0-1
	Media version    : 6.6.52
	Hardware revision: 0x00000021 (33)
	Driver version   : 6.6.52
Interface Info:
	ID               : 0x03000002
	Type             : V4L Video
Entity Info:
	ID               : 0x00000001 (1)
	Name             : C270 HD WEBCAM
	Function         : V4L2 I/O
	Flags            : default
	Pad 0x01000007   : 0: Sink
	  Link 0x02000016: from remote pad 0x100000a of entity 'Extension 8' (Video Pixel Formatter): Data, Enabled, Immutable
ioctl: VIDIOC_ENUM_FMT
	Type: Video Capture

	[0]: 'YUYV' (YUYV 4:2:2)
	[1]: 'MJPG' (Motion-JPEG, compressed)

Looking forward to seeing this PR integrated.

@jens-maus : thank you for verifying and confirming MJPG format works!

@oleg0421 can you address the remaining issues and rebase this to current master?

@akallabeth : I re-based it to current master. There are no issues, other than what was originally reported by @hardening, which I believe is USB hardware specific.

@hardening , @akallabeth: I was able to get a hold of Logitech Webcam C930e and do some testing with it. I observed some JPEG decoding errors such as:
[mjpeg @ 0x7f9d102824c0] overread 8
[mjpeg @ 0x7f9d102824c0] EOI missing, emulating

or
[mjpeg @ 0x7f9d47583c40] mjpeg_decode_dc: bad vlc: 0:0 (0x7f9d47580048)
[mjpeg @ 0x7f9d47583c40] error dc
[mjpeg @ 0x7f9d47583c40] error y=134 x=117

In my case, all errors show up only while decoding first frame from camera.  Beginning from second frame, there are no problems with decoding.
I inspected the dumps of the first frame and I can confirm there is random partial data corruption. My guess is that something happens during the capture start between the camera, driver and kernel, which results in this data corruption. So far I don't see anything wrong with the application code itself and this problem doesn't happen with other cameras that I have.
One simple workaround is to always drop 1st frame in case of MJPEG format and it works ok (no decoding errors). Not sure, if this is acceptable as a solution, please let me know.

I would not be shocked if we decode beginning at the second frame, or perhaps we must not make the decoding error fatal, or we can live with a certain rate of decoding errors ?

Currently when the 1st corrupted JPEG frame is being decoded, FFmpeg library detects and prints the problem, but API doesn't return an error (considers it minor). We then try to re-encode to H264. Since it's a 1st frame, it becomes bad H264 I-frame, which results in distorted image for ~10 seconds or so, until next good I-frame is generated. The effect of single corrupted frame is over-amplified.
There's a special flag in FFmpeg API (AV_EF_EXPLODE), that tells MJPEG decoder to abort decoding on minor error detection and return error. It will allow us to skip corrupted frames. WIll prepare a small patch in the next few days.

We then try to re-encode to H264.

Btw (and sorry for talking to a slightly different direction): Now that RDPCAM support in FreeRDP works quite well, I recognized that there is almost no Webcam out that that directly provides H264 streams anymore, even thought Logitech explicitly announces H264 support for their higher (C925e, C930) models. v4l-cal always return only raw and MJPEG support for these models. I just read that these models seem to attach H264 frames to the MJPEG streams for applications aware of that to pickup that data in that way. Does anyone in here is aware of that or perhaps knows if there is a way to get out that MJPEG attached H264 data? Or do we have any chance to tune FreeRDP or it's ffmpeg dependency to extract and use that H264 data? Because that would greatly help clients to keep CPU usage low of course...

@jens-maus : a great question, something I've been wondering myself after seeing some forum posts about attaching H264 frames to the MJPEG streams by Logitech cameras. It seems unlikely to me, considering it goes against Logitech's official stance on hardware H264 encoding:
https://www.logitech.com/en-us/video-collaboration/resources/think-tank/articles/article-logitech-and-h264-encoding.html
Another consideration is:  why would they do it this way, but not as separately advertised H264 format, like older C920 cameras do? Just to give some partner competitive advantage by risking potential technical and legal issues - not worth it.
Having said that, if you or someone finds good technical details how it's done (activation of this mode and how to extract from MJPEG frames), then please share and we can try to implement it.
I recognize the importance of optimization, as current MJPEG decoding followed by H264 encoding takes a lot of CPU, especially for higher resolutions and while running on thin cliens. One way to address that could be adding hardware acceleration (VAAPI) support for H264 encoding, which seems possible, considering FreeRDP already uses VAAPI for H264 decoding.

@oleg0421 I could also indeed not believe that Logitech has actually gone that road in somewhat hiding the H264 data within the MJPEG frames, but it really seems that for their cameras where the mention H264 support this is actually the case. In fact, see here a 5 year old feature request for ffmpeg which actually explains it in more detail:
https://trac.ffmpeg.org/ticket/8413
And in this ticket you will find other links to e.g. gstreamer where they already implemented a kind of demuxer to somewhat extract the H264 data from the MJPEG frames. See here:
https://github.com/GStreamer/gst-plugins-bad/blob/master/sys/uvch264/gstuvch264_mjpgdemux.c
Or for a general stackoverflow talk on that matter:
https://stackoverflow.com/questions/55439184/getting-unable-to-decode-app-fields-while-playing-usb-webcam-stream-through-ff
So it would actually be great if you could have a look at all that H264 smuggling that logitech seem to have put in their recent camera generations. It is really astonishing that even the same hardware models which previously directly provided H264 streams are now – when you buy them freshly – solely announcing MJPEG when plugged in and analyzed via v4l-ctl, but they use that MJPEG smuggling to actually provide certain apps (e.g. Skype & co) with some benefit while standard apps do not know about that "innovative" way of hiding the H264 data. In indeed, for thinclient environments, especially where e.g. FreeRDP might be integrated it would be great if the hided H264 data could be extracted. So please let me know what you will find out and how I could assist in getting such a demuxer implemented in FreeRDP.


Fix a bug in the Kythe URI spec. [phab:D1040]

Created by fromberger at 2016-09-09 14:08:05:
The "root" attribute of the URI should not require a leading "/".
The existing implementations are already correct (i.e., they do not
enforce a leading "/"); the bug was in the spec.

shahms accepted this revision at 2016-09-09 15:41:48

fromberger closed via commit 983aabc5ea636a4c6fec85d0652df7ee95b66923 at 2016-09-09 15:43:14


Hardware back button not working

Hey
I just upgrade to 1.22.0-10.0.pre.65 on Channel master and after that, I have a problem with hardware back button in android.
It doesn't work properly and most of time it doesn't work.
Flutter version 1.22.0-10.0.pre.65
Framework revision 2032a448ed
Dart version 2.10.0

Hi @z85510,
There's similar issue that was fixed. Also see this comment and I'd suggest you to keep an eye on the framework engine rollout to master to see the fix. Once it lands, you may upgrade master and see if that's resolved.
Closing this as duplicate.


Bug DFBUGS-922: [release-4.15-compatibility] Reverting back CVE fix for DFBUGS-922 and DFBUGS-934

https://issues.redhat.com/browse/DFBUGS-922
https://issues.redhat.com/browse/DFBUGS-934

/retitle Bug DFBUGS-922: [release-4.15] Reverting back CVE fix for DFBUGS-922 and DFBUGS-934

/approve

/lgtm


Question: Does docker keep the mount.ID same across container restarts?

I am trying to maintain state across the container restarts and they persist data into a folder that is created when the container starts for the first time.
If the container restarts, I want to give the same dir path again to the container using my custom docker volume plugin.
My question is does docker provide the same mount.ID for the same container to a docker volume plugin?

If you do not provide a name for a volume, then docker will generate one when it's created. This name is stored in the container config, so will not change (also not between restarts). However, a volume that is not explicitly named, is considered "ephemeral" in some cases, for example:
start an interactive container with an "unnamed" volume for /foobar:
docker run -v /foobar -it --rm --name some-container alpine sh

In another shell:
docker inspect --format '{{ json .Mounts }}' some-container
[{"Name":"d88bccf10009985f9336278a8b1b44371ff7256da926ce16dd0e846e602ac77c","Source":"/var/lib/docker/volumes/d88bccf10009985f9336278a8b1b44371ff7256da926ce16dd0e846e602ac77c/_data","Destination":"/foobar","Driver":"local","Mode":"","RW":true,"Propagation":""}]

docker volume ls --filter name=d88bccf10009985f9336278a8b1b44371ff7256da926ce16dd0e846e602ac77c

DRIVER              VOLUME NAME
local               d88bccf10009985f9336278a8b1b44371ff7256da926ce16dd0e846e602ac77c

However, the moment the container is stopped, it will be removed including the volume:
docker kill some-container
some-container

docker volume ls --filter name=d88bccf10009985f9336278a8b1b44371ff7256da926ce16dd0e846e602ac77c

DRIVER              VOLUME NAME

To make sure a container persist, either name it explicitly, or create the volume first, and specify the automatically generated name for the container;
Example 1: explicitly name the volume:
docker run -it --rm -v my-volume:/foobar --name some-container alpine sh

Example 2: create the volume "out of bound", and use the generated (random) name:
docker volume create
3a975286dc446b51792c9b7ca95837408057f7852ba5c0555391e27ca8130712

docker run -it --rm -v 3a975286dc446b51792c9b7ca95837408057f7852ba5c0555391e27ca8130712:/foobar --name some-container nginx sh

you can do the above in one go, using:
docker run -it --rm -v $(docker volume create):/foobar --name some-container alpine sh

I'm closing this issue, because this is a question, not a bug report, and our GitHub issue tracker is not really intended for that, but feel free to continue the discussion

I'm believe we do provide the same opaque id, but will have to check the code, I can't remember... Or you could test it.

oh, wait, this is about mount id, not the volume name, sorry for the confusion on my side :smile:


Enhancement: A popup message should be displayed when member is added to group.

Currently if the member you are trying to add does not exists or is already a member then message is displayed through an HTTP response.
A pop up message should appear on the same page.
Tasks:

[ ] #21
[ ] #22
[ ] #23

Please assign it to me.


API.AI in readme file, instead of Dialogflow.

I can take this up

Destino?

Sorry I didn't get. Shall I take this up?


It's not working on real device.

It's working fine with the simulator but it's not formatting the number on real device. I am testing on iPhone 12 pro.
Here is my source code.
PhoneView.swift.zip
iPhoneNumberField("(000) 000-0000", text: $phoneNumber)
.maximumDigits(10)
.font(.custom(AvenirFont.medium.rawValue, size: 16))
.keyboardType(.phonePad)
It's formatting number in this way : 5765 656 585 but i need in this way : (5765) 656-585

Seems like you have an issue with detecting the correct region for your desired formatting.

Closing the issue due to inactivity. Feel free to reopen if needed.


Tag limit on GitHub container repos

I am using several images from the linuxserver repositories on ghcr.io, and the image reflector is only returning 1000 tags for them even when there are more than 1000 tags. This is preventing those repositories from matching newer tags and being updated by the automation controller. I configured authentication on one of the repos but the 1000 tag limit is still being hit.
$ flux get image repository tautulli
NAME            READY   MESSAGE                                 LAST SCAN                       SUSPENDED
tautulli        True    successful scan, found 1000 tags        2021-05-08T09:22:11+08:00       False

If I use curl on the API tags list URL for tautulli (https://ghcr.io/v2/linuxserver/tautulli/tags/list) I get 1088 tags returned.
$ curl -s -H "Authorization: Bearer $(echo <token> | base64)" https://ghcr.io/v2/linuxserver/tautulli/tags/list | jq '.tags | length'
1088

The PHP Docker Hub registry is returning over 5000 tags so it might be something specific to GH.
$ flux get image repository php
NAME    READY   MESSAGE                                 LAST SCAN                       SUSPENDED
php     True    successful scan, found 5033 tags        2021-05-08T09:22:03+08:00       False

Running image-reflector-controller v0.9.1

I see that distribution/distribution#3143 was merged about 15 days ago. Does this mean there can be progress on this issue, or can it be closed? (I think it still blocks on GCHR taking action.)

It looks like it's resolved, I am seeing over 1000 tags returned for the repositories I have configured.
$ flux get image repository tautulli
NAME            READY   MESSAGE                                 LAST SCAN                       SUSPENDED
tautulli        True    successful scan, found 1436 tags        2021-09-24T16:19:18+08:00       False


mips64el support

These are the hopefully-uncontroversial parts of my mips64el nixpkgs support patches.
The commits in this PR are also the only ones needed in order for Hydra to produce a usable bootstrap-files.tar.xz (#161162, #161159, #161163, #161925, #161925, and #162070 are needed to use the tarball, but not to generate it).
Mostly this just adds definitions.  MIPS has an uncharacteristically-large space of (architecture,abi,endianness) permutations; I added all of them to lib/systems/platforms.nix but elsewhere (in the next PR) reference only the ones used.
Currently lib/systems/inspect.nix has a single isMips predicate, which is a bit ambiguous now that we will have both mips64 and mips32 support (and, for mips64, two different ABIs neither of which is better than the other in all cases).  I have therefore added four new predicates (isMips32, isMips64, isMips64n32, and isMips64n64).  I suggest that we treat the ambiguous isMips as deprecated moving forward, and encourage people to use the more-specific predicates.  These predicates are used mainly for enabling/disabling target-specific workarounds, and in practice it is extremely rare that both mips32 and mips64 need a workaround and also need exactly the same workaround.
The separate predicates for ABI are, unfortunately, useful.  Mips64n32 is great for low-RAM routers (most have much less than 4GB).  Unfortunately neither golang nor boost supports n32.  In the case of boost there is hope: the needed support is just a single assembler file that saves/restores the CPU state.  Unfortunately until that is written nix can't bootstrap on n32.
So, that's the reason for having four different predicates.  If the need for them is still unclear, take a look at the next few PRs I submit for example usage.
Motivation for this change
Octeons are awesome.  That is motivating for me.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

This is the blocker PR for all my other mips64 PRs, including #161165 which has been marked as "draft" until this pull is merged.
Reviews would be much appreciated.

Merging this PR is enough in order for Hydra to cross-build a usable bootstrap-files.tar.xz.  Once that happens I will submit another commit with pkgs/stdenv/linux/bootstrap-files/mips64el.nix pointing at those outputs and their hashes.
Here is the command that I used to build the bootstrap-files.tar.xz I used for testing:
nix-build  -E '(import /nix/git/bootstrap/pkgs/stdenv/linux/make-bootstrap-tools-cross.nix { }).mips64el-linux-gnuabi64.build'

Is there something else that needs to be done in order to tell Hydra that it should evaluate the expression above and build the derivation?  I'm following the example of a5851f18c5c87348d86b36741290201eb43258ac where https://hydra.nixos.org/build/159891432 was used to produce the trustable values for pkgs/stdenv/linux/bootstrap-files/riscv64.nix.
Merging the other mips64-related PRs is not needed for the bootstrap-files; the other PRs affect the build of the bootstrap nix binary (which isn't in the bootstrap tarball) and the copy of nixpkgs used on the bootstrap host.

Ping...
There are at least six other PRs blocked waiting for this one.  Could somebody please review it?

To allow people without mips64 hardware to test things, I have put together a shell.nix to get you a booted mips64el qemu VM, with the host's /nix/store mounted via 9p inside the guest.  Details: https://github.com/NixOS/nixpkgs/pull/162070#issuecomment-1055518234

https://github.com/a-m-joseph/mips64-nixpkgs-qemu/pull/1/files

Merged, thanks for catching that.  I also found a way to write the shellHook without having to duplicate builder/args.

Latest push squashes e8fe59f9ca1909e381f6b18f6530f5a14d3a0012 to a single commit and simplifies the wording of the first paragraph of the commit message; no other changes.

The job running on runner GitHub Actions 10 has exceeded the maximum execution time of 360 minutes.

Is there any chance we could have CI not mark issues with the "red X of shame" if the only non-successful jobs were timeouts?  Three of my PRs are currently in this state.

Due to the number of rebuilds this PR causes it needs target staging.
The CI checks that are timing out only run for PRs targeting master.

Due to the number of rebuilds this PR causes it needs target staging.
The CI checks that are timing out only run for PRs targeting master.

Ah, thank you!
I will rebase against staging.

The CI checks that are timing out only run for PRs targeting master.

Hrm, the base branch for #161925 was already set to staging, but it is getting the "red X due to timeout".
Also after changing the base branch for this PR to staging I think CI got stuck in a funny state... the "Basic evaluation checks" cancelled after 7 minutes.

I cancelled it. Need to push to the branch as well ... with the way github actions work just changing the branch doesn't reset it.

I wouldn't worry about getting CI to pass for your other PRs for now as they'll need to rebased anyway after this is merged.

@a-m-joseph Sorry for so may rounds of review. We are getting close.

@a-m-joseph Sorry for so may rounds of review. We are getting close.

No worries, take your time.  Getting this stuff right matters.

Please note that in my latest batch of pushes is one new change, 0ec63a6.
This removes the float = "hard" from the mips entries for platforms.nix.
I never should have included those in all of the entries -- it was a copy/paste error.
It turns out I should not even have included it in the mips64 entry.  I had mistakenly believed that all mips64 chips had floating-point cores, but it turns out that this is not correct.  mips is popular in routers, which don't have much need for floating point.  For example, my Unifi Security Gateway (USG-4pro) is an Octeon II with no FPU.
So, we can't assume anything about the FPU simply from the platform tuple.
Fortunately -- unlike arm32 -- the presence or absence of an FPU does not change the calling convention.

Ping...

Ping...

At long last, this looks really great! Thanks for your patience, @a-m-joseph, and writing those extra comments/cleanups to make this easier for the next person. Excited to merge this!

Oh, and sorry I didn't notice this over the past week. Ping me on matrix if I keep on missing something.

Fixing the commit messages or squashing when merging would have been good ...

Thank you thank you thank you!!!

@a-m-joseph please tidy the commits once you think a pr is in a mergeable state

@a-m-joseph please tidy the commits once you think a pr is in a mergeable state

This PR has already been merged; did you mean to write that comment in a different PR?

I think he meant in the future

I think he meant in the future

Ok, can you answer this then?
I rely on this "who uses this declaration" mechanism in all the other programming languages I code in (Haskell, ML, Rust, C/C++, even Java...) in order to avoid making mistakes like the one I made here.  I'd like to know how to do the equivalent thing in nixlang so this doesn't happen again.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/packages-marked-as-broken-should-come-with-an-explanation/19187/26


IMPORT Doc Update

Re: IMPORT
Issue Description
document a CSV import flag strict_quotes. If import CSV file has quotes, multiple rows can be treated as single rows. strict_quotes flag will produce reasonable output to hint at an underlying issue row 1914: reading CSV record: parse error on line 1914, column 28: extraneous or missing " in quoted-field
Suggested Resolution
document flag strict_quotes if import files have quotes within rows. This flag doesn't take any input. Example: CSV         DATA ('gs://artem_cockroach_bucket/aka_name.csv') WITH     strict_quotes, delimiter = ',';

@lnhsingh, any updates here? Would be great to wrap this up in Jan. Requested by the customer success team.


(Request) add option to expire locks

As a server operator I would like a config option where I can set a number of days/hours that chest lock/ownership expires in. Example I want players who are inactive for a month (haven't logged into the server) to have their chest unlocked when any other player interacts with it :)

This is really important for our server because the chests of inactive players start to really add up after some months and liter the landscape and can even be used as unbreakable griefing blocks. If there was a way to have the locks expire after 30 days it would be very helpful. It would also be cool that players exploring could happen upon an expired chest of an inactive player and get cool loot and clear up the landscape by breaking it


Run Renggli on ImageNet on whole WP1 model pool

Run Renggli on the whole of the selected pool of models for WP1 on the ImageNet-1k development set, including hyperparameter scans of results vs. number of images.

Calling this done for WP1 - see plots in July partner meeting slide.


Dashboard details are slow in loading

Azure Data Studio Version: master/1.2.3-insider

Steps to Reproduce:

Connect to a server. I used an Azure SQL DB for this.
Click on the "V" button on the top right to show the server details.

Expected - Details shouldn't take a lot of time to show.
Actual - Details take a lot of time to show. If the details are collapsed and expanded again, it takes the same amount of time to load. It can be easily cached for a server.

Closing older issues.  This should be address as part of current dashboard feature work.


Edit font size, type, etc to optimize readability

@hazelshapiro In which direction do we need to move to optimize? Are certain fonts too small/large? Etc.

We have a graphic designer we have worked with in the past and we'll get
his input. Generally larger fonts are better while making sure it still
fits on people's screens.
On Sat, Jan 6, 2024 at 10:56 AM Matt Fisher @.***>
wrote:

@hazelshapiro https://github.com/hazelshapiro In which direction do we
need to move to optimize? Are certain fonts too small/large? Etc.
—
Reply to this email directly, view it on GitHub
https://github.com/nsidc/usaon-benefit-tool/issues/188#issuecomment-1879768252,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4RGLH3VLQZ7WW4PQU6MBC3YNGF6FAVCNFSM6AAAAABADLQDN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZG43DQMRVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Hazel Shapiro
Program Analyst
US Arctic Observing Network
https://www.iarpccollaborations.org/united-states-arctic-observing-network.html
& Interagency Arctic Research Policy Committee
https://www.iarpccollaborations.org/
I currently reside on the unceded land of the Arapaho, Cheyenne, and
Núu-agha-tuvu-pu (Ute) people
she/her (what's this? https://www.glsen.org/activity/pronouns-guide-glsen)

This is more of a general note to check this and adjust as needed, rather
than a specific action
On Mon, Jan 8, 2024 at 11:40 AM Hazel Shapiro @.***>
wrote:

We have a graphic designer we have worked with in the past and we'll get
his input. Generally larger fonts are better while making sure it still
fits on people's screens.
On Sat, Jan 6, 2024 at 10:56 AM Matt Fisher @.***>
wrote:

@hazelshapiro https://github.com/hazelshapiro In which direction do we
need to move to optimize? Are certain fonts too small/large? Etc.
—
Reply to this email directly, view it on GitHub
https://github.com/nsidc/usaon-benefit-tool/issues/188#issuecomment-1879768252,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4RGLH3VLQZ7WW4PQU6MBC3YNGF6FAVCNFSM6AAAAABADLQDN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZZG43DQMRVGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Hazel Shapiro
Program Analyst
US Arctic Observing Network
https://www.iarpccollaborations.org/united-states-arctic-observing-network.html
& Interagency Arctic Research Policy Committee
https://www.iarpccollaborations.org/
I currently reside on the unceded land of the Arapaho, Cheyenne, and
Núu-agha-tuvu-pu (Ute) people
she/her (what's this?
https://www.glsen.org/activity/pronouns-guide-glsen)

--
Hazel Shapiro
Program Analyst
US Arctic Observing Network
https://www.iarpccollaborations.org/united-states-arctic-observing-network.html
& Interagency Arctic Research Policy Committee
https://www.iarpccollaborations.org/
I currently reside on the unceded land of the Arapaho, Cheyenne, and
Núu-agha-tuvu-pu (Ute) people
she/her (what's this? https://www.glsen.org/activity/pronouns-guide-glsen)

I've updated the title and labels, please feel free to apply your own edits if you see more opportunities for clarity :)


Improve mesh cleaning algorithm

The mesh cleaning algorithm can throw an unhandled exception causing the pipeline to terminate early. In addition, the algorithm should be improved to rely more on local isometric remeshing than on cutting and hole-filling.
The current version of the cleaning algorithm deals with self-intersecting faces and folded faces by deleting a section of the mesh surrounding them and then filling in the hole. This can create non-manifold faces, but in line 243 we're calling close_holes() again before fixing these non-manifold faces which throws an exception.
To do:

Add exception handling to calls of clean_mesh() so this and any other issues don't kill the pipeline. We should at least log that the cleaning failed and move on to the next mesh.
Fix this specific problem by checking and fixing manifoldness before calling close_holes() again in line 243.
If possible, replace this holes-filling process with calls to dialate_selection and remesh_isotropic on the selection. It's possible that cutting and hole-filling works better for self-intersections but local remeshing works better for things like folded faces.
It's probably worth adding a check for faces with an aspect ratio less than 0.02 or maybe even 0.10 and fixing them with a local remeshing process.

It is also slow, and its possible it just isn't exiting from the cleaning loop properly.


perf(proofs): root calculation using hash builder

Replace triehash root calculation with custom using HashBuilder
Benchmarked in https://github.com/paradigmxyz/reth/pull/2513

Codecov Report

Merging #2517 (bdecbf5) into main (1743169) will increase coverage by 0.09%.
The diff coverage is 80.64%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #2517      +/-   ##
==========================================
+ Coverage   71.68%   71.78%   +0.09%
==========================================
  Files         489      489
  Lines       61062    61146      +84
==========================================
+ Hits        43773    43891     +118
+ Misses      17289    17255      -34

Flag
Coverage Δ

integration-tests
17.95% <0.00%> (-0.04%)
:arrow_down:

unit-tests
66.81% <80.64%> (+0.10%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

crates/consensus/auto-seal/src/task.rs
0.00% <0.00%> (ø)

crates/payload/basic/src/lib.rs
0.19% <0.00%> (ø)

crates/storage/provider/src/post_state.rs
93.55% <0.00%> (ø)

crates/primitives/src/proofs.rs
94.50% <85.71%> (+0.73%)
:arrow_up:

crates/consensus/common/src/validation.rs
80.88% <100.00%> (-0.04%)
:arrow_down:

crates/interfaces/src/test_utils/generators.rs
98.23% <100.00%> (ø)

crates/revm/src/executor.rs
89.57% <100.00%> (ø)

crates/rpc/rpc-types/src/eth/engine/payload.rs
91.37% <100.00%> (+0.11%)
:arrow_up:

... and 16 files with indirect coverage changes


Ensure system is not leaking FDs

@bkueng @davids5 We might need to teach NuttX to keep track of the FD count for each task and report that.

We have better instrumentation in now.


how to use the example files

Hi, I am testing racons with the test files you have on repository. I was wondering if you could help.
Currently getting error like this:
$ racon -u --threads 48 sample_reads.fastq sample_overlaps.paf sample_reference.fasta
WARNING: Not mounting /var/tmp (already mounted in container)
[racon::Polisher::initialize] loaded target sequences
[racon::Polisher::initialize] loaded sequences
[racon::Polisher::initialize] error: empty overlap set!
$ racon --version
1.3.2

Hello,
you have to run racon on sample data either with:
racon -u --threads 48 sample_reads.fastq sample_overlaps.paf sample_layout.fasta

or
racon -u --threads 48 -f sample_reads.fastq sample_ava_overlaps.paf sample_reads.fastq

sample_reference.fasta is used to calculate the edit distance between the consensus and the actual reference. You can also build racon_test which will run racon automatically on the test data. To obtain it add -Dracon_build_tests=ON while running cmake (run make afterwards).
Best regards,
Robert

Thank you!


GEODE-6702: optimize equals by not calling compareTo

The compareTo implementation needs to get the InetAddress as a byte array
which causes to byte array allocations.
By changing InternalDistributedMember.equals to not call compareTo InetAddress.equals
can be used instead which prevents the byte array allocations.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Won't the runtime optimize the compareTo call?

I don't know. I know that compareTo has to ask the InetAddress for a byte array representation of the address so that it can figure out if one address is greater or lessor than the other. It was these byte array allocations that showed up in the profiler. I think this would be garbage that the collector has to deal with. The new equals logic also has fewer conditions that it needs to test for but that was not the reason for this pull request and I doubt it would be much of a difference. If I could have found a way to change compareTo to not do the byte array allocations (by calling InetAddress.getAddress) I would have made that change add kept equals implemented by calling compareTo.


Migrating away from GAMS

What is this about?
Since GAMS is proprietary software, receiving one of the expensive licenses is often a bottleneck for using message_ix, especially for researchers from the Global South. It would thus greatly benefit our inclusivity and bolster our open source spirit if we moved away from GAMS.
There are several replacement options: pyomo and linopy come to mind most readily. I saw a presentation about linopy and a performance comparison during the OpenMod workshop hosted at IIASA last year, which is why I chose linopy (using the HiGHS solver underneath) to implement a proof of concept for the ixmp4 transport tutorial. A similar migration could be done for the actual MESSAGE GAMS code, though this comprises a lot of code and would thus likely require a significant time investment.
During today's weekly meeting, I brought up my wish to eventually perform this migration. Several colleagues shared their experiences and made suggestions right away:

@behnam-zakeri shared that we could look into gamspy, too. This, together with GAMS' move to make licenses free for academic institutions would present an alternative to using linopy/HiGHS. We would still have a python interface to the GAMS code, while possibly avoiding the actual rewriting of our GAMS code and likely avoiding the workflow problem presented by @SiddharthJoshi-Git below. The downside would be that our results would still not be completely publicly reproducible and we would still support the closed-source GAMS.
@SiddharthJoshi-Git shared part of his usual workflow: for quick assessment of results or copying of data, he often uses the .gdx files, which seem to be much more performant than dumping the results to Excel or .csv. Our migration should take this into account and provide ways to quickly and conveniently access the same information stored in the .gdx as well as the .log and .lst files.
@yiyi1991 expressed her support for this migration and stated that the OseMOSYS team has already done something similar. This might be a place to look for examples as well as an opportunity for collaboration.
@khaeru noted that we would need to ensure before migration that all GAMS features we use are also available with our choice of tools and that performance is at least at comparable levels.

All of this means that we're not likely to start this migration anytime soon, much less finish it, but I wanted to record this discussion. For future reference and for you, dear enthusiastic community member: please reach out to us to get information on the current status of this project! Any help will be greatly appreciated :heart:

I would like to add one experience where we used GAMS Base Module (academic license) at UVic and then use open-source solvers such as CBC. I briefly checked the results and the results are looking similar on a country scale model with 13 nodes.

I would like to add one experience where we used GAMS Base Module (academic license) at UVic and then use open-source solvers such as CBC. I briefly checked the results and the results are looking similar on a country scale model with 13 nodes.

Thanks @awais307! A clarifying question from my side, if I may: Do you use GAMS and CBC simultaneously, i.e. for the same Scenario.solve() workflow? Or do you use either GAMS or CBC for each individual Scenario.solve() workflow and you compared both versions for one model now?
If the former, what do you consider the "GAMS Base Module" and which functions does it fulfill (contrasting to CBC)? Also, could you point to a version of the code that does this somewhere on GitHub?


documentation clarifications

Several changes to grammar, typos and made clarifications

Changes overridden by subsequent PR and incorporated, where remaining


Support byte array passwords - Part 2

You recently added byte[] support in 1.0.3 via #51 but can this also be added for the other functions that currently take a char[] as well? Such as the SQLCipherUtils.decrypt() & SQLCipherUtils.encrypt() functions specifically? Thank you!

OK, I'll look into it -- thanks for the suggestion!

I can offer byte[] editions of those, but not of rekey() (SQLCipher for Android doesn't have a byte[] variant of the changePassword() method that I use in rekey()).
Are you using the AndroidX or Android Support version of SafeRoom? As with 1.0.3, I'd like to publish a SNAPSHOT release of a 1.0.4 for you to try.

I think that is fine as I don't personally use the rekey() method but I am on the AndroidX version and will be glad to test the snapshot, thanks!

OK, try 1.0.4-SNAPSHOT. It has byte[]variants of encrypt() and decrypt(): https://github.com/commonsguy/cwac-saferoom/commit/fd6d8050c7f3a45c2dc18f66a31943d07a22045b

Tested both encrypt() and decrypt() and it is working as intended like the char[] version, thanks!

OK, 1.0.4 final is out with this change -- thanks again!


Fix single-node scenario

Currently, distributed scenario works without problems. The single node scenario delivers errors.

If the errors can be solved by a usage of a Request SpringData JPA repository instead of a QueryGatewayRepository - just go for it...


refactor: centralize range expand logic

Perhaps we can have a better name and better place to hold this logic. Just noticed that we repeat a few time to expand range bounds and they're subtly different.

I prefer repeating in hand rather than introducing a macro.
The ideal solution will be using Range<u64> or offset, size directly in OpRead. But this could be another breaking change. May be we can implement this change in next release?

Good to know. Let me make another PR later.


Update 6.28.4

Closes #206
Closes #200
Closes #201
Closes #207
Closes #205

Logs for locally built packages:
osx_arm64_numpy1.23python3.11.____cpython.log.zst.zip
osx_arm64_numpy1.21python3.10.____cpython.log.zst.zip
osx_arm64_numpy1.21python3.9.____cpython.log.zst.zip
osx_arm64_numpy1.21python3.8.____cpython.log.zst.zip


Bot not catching lured pokemon, seems to get in infinite loop state of noticing them.

I have this as my catch configs in config.json
      {
        "type": "CatchVisiblePokemon"
      },
      {
        "type": "CatchLuredPokemon"
      },

I have this at the top level of the config
   "catch": {
      "any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or"},
      "// Example of always catching Rattata:": {},
      "// Rattata": { "always_catch" : true }
    },

In the older version from yesterday, and the newer version from today, I get in an infinite loop state of the bot "noticing" lured pokemon but not catching them. It looks like this:

Try changing your location.

if that doesn't fix it (if it's for all lured pokemon)

Use my config as a reference. http://pastebin.com/UStttHL2

if that still doesn't work post your full config and I'll see if I can fix it. :/

Cool, will do

Cool! It work for me :D
I think my pokemon is full, so i can't catch anything.

I had the same problem, it ended up being my catch logic was to blame
"any": {"catch_above_cp": 300, "catch_above_iv": .85, "logic": "and"}
Seems it would cause the bot to get surrounded by pokemon that did not fall into this range and it would keep "looking" at them.

Cool! Glad to hear you guys fixed it.


Timeout?

I'd like to be able to specify (or implement) a timeout for all my requests and responses.
I tried doing so with a behavior, but without any luck.
I initially thought I would implement it by altering the cancellation token that gets passed on to the next behaviors, but that's not possible apparently.
Is there any way to implement a timeout in Mediatr? I am okay with it being on the cancellation-token level, and only working when cancellation tokens are used.

As long as you are respecting the cancellation token in your handlers you should be able to use that.
public class TimeoutBehavior<TRequest, TResponse> : IPipelineBehavior<TRequest, TResponse>
{
    public Task<TResponse> Handle(TRequest request, CancellationToken cancellationToken, RequestHandlerDelegate<TResponse> next)
    {
        var source = new CancellationTokenSource(TimeSpan.FromSeconds(30));
        cancellationToken = source.Token;

        return next();
    }
}

SO For reference https://stackoverflow.com/questions/30875279/how-to-cancel-a-cancellationtoken

But that cancelation token won't be passed on to the "next" behavior and functionality, will it?

Since its the same cancellation token being referenced by all the handlers and behaviors I believe it will.

This wouldn't change subsequent calls, you're replacing the reference on the parameter.

Oh, based on the SO I thought it would update the reference for all.  In that case maybe making a mediator extension method that creates the token source with a timeout and sends with that token?

The token is simply passed through from _mediator.Send, so if you want a different cancellation token, you can pass in something else from there.

@jbogard that makes sense - I was still hoping I could make a behavior alter the timeout duration of a request or command, based on some condition.

@hhko that is exactly the problem I am having. I believe it would be nice if the cancellation token could be altered by a behavior.

I had the exact use case as being discussed above, i.e. the ability to change the cancellation token from inside a behavior.
@jbogard would this be a possible solution?
   public delegate Task<TResponse> RequestHandlerDelegate<TResponse>(CancellationToken cancellationToken = default);
and then update the Handle method in RequestHandlerWrapperImpl as below.
    ```

public override Task Handle(IRequest request, CancellationToken cancellationToken,
ServiceFactory serviceFactory)
{
Task Handler(CancellationToken ct) =>
GetHandler<IRequestHandler<TRequest, TResponse>>(serviceFactory).Handle((TRequest) request, ct);
        return serviceFactory
            .GetInstances<IPipelineBehavior<TRequest, TResponse>>()
            .Reverse()
            .Aggregate((RequestHandlerDelegate<TResponse>) Handler,
                (next, pipeline) => ct => pipeline.Handle((TRequest) request, ct, next))
            (cancellationToken);
    }

You can decorate IMediator with Scrutor and implement polly optimistic timeout in decorator.

@jbogard this was marked as completed. Can you link to documentation or the commit where it was fixed? 🙏

GitHub's UX marked it as completed. Not me. I did a bulk-close and GitHub marked it as such.

In any case, this is better suited as a discussion, not an issue.

I know it's been a while but I have a PR out that I think makes this possible now.
#1058


How do I enable ssl for email? Add mail.domain.com to cert?

First, thank you for this wonderful plugin. I came from the beta command line interface from letsencrypt, and while that worked for me on my own home server, setting up the same on a centOS RHEL box with heavy integration with cPanel/WHM was not an ideal solution.
Finding this cpanel plugin was a life saver - thank you!
There are a few last pieces I'm attempting to rectify, one of which is enabling ssl for mail handled by our server. How would I go about adding mail.domain.com to the certificate?
Thank you!

You have to create mail.domain.com subdomain first and then you can generate cert for the same using letsencrypt plugin.


Add initVariable (and Pose3) tests

Will do this weekend. Note the blocking indefinitely is likely due to the new event pattern as I have mentioned before. We should change this function to now check two endpoints in the interim. The events endpoint has the new feature events. I can update the SDK. If you are blocked, here is the snippet I would add:
GQL_GET_EVENTS_BY_ID = """
  query events_by_id(\$eventId:String) {
    test: events(where: {context:{eventId:\$eventId}}) {
      context {
        eventId
      }
      status {
        state
      }
    }
  }
"""
function WaitForCompletion(eventId, timeoutInSeconds; totalRequired=1, completeRequired=1)
  elapsedInSeconds = 0
  incrementInSeconds = 5
  while elapsedInSeconds < timeoutInSeconds
    get_event_response = client.query(QueryOptions(
      GQL_GET_EVENTS_BY_ID,
      Dict(
        "eventId" => eventId
      )
    ))
    payload = JSON.parse(get_event_response.Data)
    events = payload["data"]["test"]
    completeEvents = filter(event -> event["status"]["state"] == "Complete", events)
    if size(events)[1] >= totalRequired && size(completeEvents)[1] >= completeRequired
      return true
    end
    failedEvents = filter(event -> event["status"]["state"] == "Failed", payload["data"]["test"])
    if size(failedEvents)[1] > 0
      return false
    end
    elapsedInSeconds += incrementInSeconds
    sleep(incrementInSeconds)
  end
  return false
end

we just posted at the same time.  I'll try the code snippet thanks

Few tweaks but think that may have done it -- thanks!
julia> runInitVariableTests(sessionId="INITVARIABLE_0010")
context = NVA.Client(userId, robotId, sessionId) = Client<EMAIL_ADDRESS>Robot=TESTING, Session=INITVARIABLE_0010
┌ Info: Wait on addVariable eventId
└   resultId = "01GA9T9ZX1S0YJJYFQ0BWZ5EE8"
┌ Info: waitForCompletion on initVariable
└   eventId = "01GA9TA3HZNQYYJ3E01HTB3PCR"
Test Summary:          | Pass  Total
run initVariable tests |    1      1
Test.DefaultTestSet("run initVariable tests", Any[], 1, false, false)

currently under NVA.waitForCompletion2 which we can consolidate before tagging.  Will create new issue for that.

So the new NVA.waitForCompletion2 does solve that problem and is included in this PR thanks.  The SDK.jl test for initVariable now passes.  This on on guest however, my other local test against examples does not seem to be updating numerics with initVariable.

There also might be another waitForCompletion type issue on older solveSession test which used to pass.  Think this might be related to the new eventing.  Easiest way to recreate is just run the SDK.jl tests locally:
pkg> test NavAbilitySDK

Need to keep moving even though test are failing.  Related issues opened.


if use on Windows

Expected Behavior

npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for fsevents@ 1.1.1: wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"} )
Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. PHP 5.4 on nginx 1.9.1):
Server type and version:
Operating System and version:
Link to your project:

fsevents is an optional dependency, it shouldn't matter if it's skipped on install on Windows, so you can ignore the warning. :)
If you run into any issues though, let us know!

thanks for your replying


[HDFS-483] Weird parsing workaround by number->string type conversion

Nima, this seems like a pretty large change, especially considering that we
now lose all validation that happens when values are supposed to be
integers.
I have not had a look at the ticket for more context.
Evan Lezar
On 21 Oct 2017 02:05, "Nima Vaziri"<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/mesosphere/dcos-commons/pull/1762
Commit Summary

Changed type of JSON field from number to string to get around weird
parsing behaviour

File Changes

M frameworks/hdfs/universe/config.json
https://github.com/mesosphere/dcos-commons/pull/1762/files#diff-0
(731)

Patch Links:

https://github.com/mesosphere/dcos-commons/pull/1762.patch
https://github.com/mesosphere/dcos-commons/pull/1762.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/dcos-commons/pull/1762, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA42Qafv3sl2aaztpRvJpDe09wKJzEQ1ks5suTVGgaJpZM4QBWyu
.

I would postulate that we could get more crashes now because of non-numeric strings being passed.

There is truth to that, although we're deciding to pass integers as strings only for settings that we're passing through to the base tech. For other settings, we're still preserving our validation (i.e. number for cpus, mem, etc)


Does not save edit or add new values

Hello,
nice work you have done! :)  I get always a "An Action has been performed..." notification but the PlistManager didn't save any values, but recognized when I'm saving an existing one. Where could the problem be?
Thanks!
 ✌

I thought I was having the same issue today but:
In my case the "SwiftyPlistManager.shared.save()" function unexpectedly returns true when you use a key that dose not exist in the plist. In this case the save function won't store the new key/obj pair.
When I swapped over to "SwiftyPlistManager.shared.addNew()" the library started creating new values correctly.


rewrite devtools memory profiling doc

Plz R @pbakaus
Staged doc will be available (in a few mins) at https://pr-3092-dot-weasel-dot-web-central.appspot.com/web/tools/chrome-devtools/profile/memory-problems/memory-diagnosis

Just added some minor nits, otherwise great work!


Document's getBlob() does not return the data of the blob if content_type is specified

The getBlob() method of the Document class does not return the blob's data even if the content_type is set.

Team, we are hitting this issue. Is there any workaround?

This is fixed in 0.1.9 - @azaddhirajkumar can you please re-run your tests to validate.  I updated the Example app Get Document and Editor pages to inject in multiple blobs on a single document and then the Get Document page will get the content if it's a text blob.  Get should now properly return hello world for the text content on any document from the Editor page if you select a generated document that has a blob type.

Fixed in 0.1.9


Packages: Move patterns to new structure

Context

Summary
The design system currently loads some patterns-related code from edit-story (which itself relies on the design system), causing a circular dependency.
Every util functions around patterns (i.e. dealing with hex and rgb colors) should be moved to a dedicated package.
Relevant Technical Choices

To-do

User-facing changes

Testing Instructions

QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #7824

@spacedmonkey Looks like some tests are failing now.

@googlebot I consent.


Option to disable MathJax but enabled by default

Fixes previous commit which allowed disabling MathJax, but would have disabled MathJax if the site parameter mathjax was not included. This would have broken existing sites that don't have that parameter in their config.toml. Now MathJax is controlled by the disable_mathjax site parameter. If it does not exist, MathJax will be enabled, so nothing is broken...yay!

Great!


Passing Resources to CellTable does not style table

Hey guys, I'm looking for a bit of clarification on this issue.
In GwtBootstrap3 there is a constructor: public CellTable(final int pageSize, final Resources resources)
If this was a standard GWT CellTable the Resources would allow a developer to style elements of the table with css, like this:
.cellTableHeader {
	background-color: #eee;
}
.cellTableSortableHeader {
	cursor: pointer;
}
.cellTableSortableHeader:hover {
	background-color: #ddd;
}
.cellTableCell {
	font-size: 13px;
}

I noticed that the current implementation of CellTable doesn't apply those classes to the table as the native GWT one would. They are all just class="gwtb3-d" whether sortable or not, etc.
Is there a way to achieve this? Is it something that could be included in a future PR? I see in the code for CellTable those classes have been deliberately removed to allow Bootstrap styling but wasn't sure exactly what was going on.
Cheers,
Drew

Just use the native GWT CellTable if you do not want to have your's styled like a bootstrap table. We only have an implementation of CellTable so that we can provide the Resources to make it look/feel like a bootstrap table.

The thing is, I want the bootstrap styles, but I also want to add a couple of very basic style rules on top of them - to change the colour of the header on hover if that column is sortable, for instance.
It's fine if it's just something that hasn't been implemented yet, I just thought I'd ask the question as I was trying to make sense of the CellTable code and didn't quite get what was going on.

It depends on the type of styling you want but you should be able to use some custom css to do this via either a ui:style element in UIBinder or a global css file.


How can I use platypus to create a javafx app

I use shell like
#!/bin/bash

cd "${0%/*}"
./jdk/bin/java -Xmx64m --upgrade-module-path ./ -m mymod

This works, but will open another window.

After I exit javafx application, the app not exit.

Java apps in macOS automatically register as their own "apps" as far as the OS is concerned, so you're stuck with an additional Dock icon unless you can find a way to suppress this behaviour.
As for quitting when the Java app quits, you can deselect "Remain running after execution". Then the Platypus application wrapper will quit once the script exits.


Security Stuff

Add security features by using django package called "django-secure".

REMAINS!!

Done.


Andstatus defaults to an arbitrary or strange timeline

Whenever I open AndStatus from the icon, or return from a conversation view by pressing the button in the upper left corner of the app, AndStatus loads an odd "default" timeline. For a while it was "Friends Combined", but for a long time now it's been "Search 'hishgraphics'".
I think I've done nothing in Settings to cause this behavior; it just started on it's own, and changed itself arbitrarily after I performed a search for "hishgraphics". When I look in Settings->Timeline, there is an option called Manage Timelines that says "Timeline that will be opened by a Launcher: "Search 'hishgraphics' combined" that I never specified, and under it is a list of timelines with checkboxes --some checked, some not-- which don't give me a clue how this behavior started or how to fix it for good.

@EricxDu I also noticed that in v.27 there is no concrete default Timeline until you set one explicitly in the Manage timelines view.
See https://github.com/andstatus/andstatus/issues/395
In v.28 this is fixed: If not set explicitly in the "Manage Timelines" view, Home Timeline of the Default Account is now the Default Timeline - it opens on the Application launch. This is natural for new users and simplifies initial configuration.
See http://andstatus.org/changelog.html
v.28 is in the Open Beta testing channel https://play.google.com/apps/testing/org.andstatus.app

This is good news, thanks. I'll be awaiting .28
Eric Duhamel
http://noxbanners.net/
Yuri Volkov<EMAIL_ADDRESS>wrote:

@EricxDu I also noticed that in v.27 there is no concrete default Timeline until you set one explicitly in the Manage timelines view.
See #395
In v.28 this is fixed: If not set explicitly in the "Manage Timelines" view, Home Timeline of the Default Account is now the Default Timeline - it opens on the Application launch. This is natural for new users and simplifies initial configuration.
See http://andstatus.org/changelog.html
v.28 is in the Open Beta testing channel https://play.google.com/apps/testing/org.andstatus.app
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.￼
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/andstatus/andstatus","title":"andstatus/andstatus","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/andstatus/andstatus"}},"updates":{"snippets":[{"icon":"PERSON","message":"@yvolk in #413: @EricxDu I also noticed that in v.27 there is no concrete default Timeline until you set one explicitly in the Manage timelines view. \r\nSee https://github.com/andstatus/andstatus/issues/395\r\n\r\nIn v.28 this is fixed: If not set explicitly in the "Manage Timelines" view, Home Timeline of the Default Account is now the Default Timeline - it opens on the Application launch. This is natural for new users and simplifies initial configuration. \r\nSee http://andstatus.org/changelog.html\r\n\r\nv.28 is in the Open Beta testing channel https://play.google.com/apps/testing/org.andstatus.app"}],"action":{"name":"View Issue","url":"https://github.com/andstatus/andstatus/issues/413#issuecomment-243668560"}}}

Seems to be fixed at least by AndStatus v.29


serverspec example files are not created

Issue Type

Bug report

Molecule and Ansible details
ansible <IP_ADDRESS>
molecule, version 1.17.0

Molecule installation method: pip
Ansible installation method: pip

Desired Behaviour
when running:
molecule init --verifier serverspec

The serverspec section of the documentation says "Example files are created" but they are not.  Ideally if the docker driver is also specified it should create a spec_helper.rb that facilitates testing of the docker container.
Actual Behaviour (Bug report only)
No serverspec example files are created.

@m4rkw molecule init --role shit --verifier serverspec appears to create examples, correct?

ah so it does, it just doesn't seem to do it if you init into an existing role.  in any case i've figured out how to make docker work now but there would have been a lot less friction and lost time if that had worked.

@m4rkw sorry to have disappointed :)  Thanks for the bug report.  Will fix it.


datafusion-cli: unstable-2022-04-08 -> 15.0.0

Description of changes
https://github.com/apache/arrow-datafusion/compare/9cbde6d0e30fd29f59b0a16e309bdb0843cc7c64...15.0.0
https://github.com/apache/arrow-datafusion/blob/15.0.0/datafusion/CHANGELOG.md

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thank you !


filter by the context type we're checking for

@robmoffat

How come a typed listener is needed in these basic tests? An untyped listener should still receive the contexts broadcast via a null typed listener. There are filtered tests later in both sets...

The issue is that in FDC3 implementations that retain history on channels, the current context on a wildcard listener can be contaminated by previous tests and produce failures.  If we're in agreement that adding more precision here doesn't alter the meaning and purpose of the test, then is there harm in doing so?

That's fine with me.  though I think for this test, it may be 6 one way /
half-dozen the other.  In this case, I believe it is just validating that
its received a context of type 'fdc3.instrument'.
On Mon, Mar 20, 2023 at 5:14 PM Kris West @.***> wrote:

I was under the impression that this was already dealt with when we added
the testId to the contexts so we could ignore all messages from outside the
tests...
I can see the testId being added but it looks like it's not being used in
setupAndValidateListener where it is in other functions like
waitForContext.
I think matching the testId is probably the better approach as you could
still have collisions on a filtered type.
—
Reply to this email directly, view it on GitHub
https://github.com/finos/FDC3-conformance-framework/pull/194#issuecomment-1476943444,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIY7GX2FA44FGLZZLYLOS3W5DCDHANCNFSM6AAAAAAWBERHVM
.
You are receiving this because you authored the thread.Message ID:
@.***>

@kriswest  does the above explanation make sense to you?

Hi Nick,
Can we have some details on what is failing and why this is necessary?  In order for @gaganahluwalia to make some changes to the code, we're going to need to have something that breaks and then be able to fix it.  Does that make sense?
thanks,
Rob

@robmoffat
the relevant code looks like this:
    const resolveExecutionCompleteListener =
        cc.initCompleteListener(scTestId1);
      let receivedContext = false;
      await cc.setupAndValidateListener1(
        null,
        null,
        "fdc3.instrument",
        errorMessage,
        () => (receivedContext = true)
      );
      const channel = await cc.retrieveAndJoinChannel(1);
      await cc.openChannelApp(scTestId1, channel.id, JOIN_AND_BROADCAST);
      await resolveExecutionCompleteListener;

This adds a context listener (with the validator for context type of 'fdc3.instrument') for any context type and then joins a channel (e.g. 'red').  When the channel is joined, whatever the most recent context is will be pushed to the handler - regardless of type.  If the type isn't the expected 'fdc3.instrument', the test fails.
Connectifi will save channel history in a session.  So, if you rerun the tests in the same session, a test after this one has set a different context type on the same channel.  Which means that when this test runs it will receive the unexpected context on first joining the channel - resulting in a failure.  These tests fail consistently if there is previous session state, reseting the session consistently results in a pass.
Can't really speak to why these tests aren't failing on other systems or why they started failing.  I believe with Sail, we we're probably starting the runtime fresh with each test run.  Its also possible that different systems have different handling for sending context on channel join when a listener hasn't specified a context type.

Ah, multiple runs of the tests without a reset, that'd explain it.

This has been superseded by #195 (merged to main). @nkolba you could test that again and close this if the problem is resolved.

thanks @kriswest  just tested against main and the tests pass now.


IBAN not accepted

My IBAN ends with the letter "K", but this is not accepted by this library. I verified the IBAN on various sites (e.g. https://rechnerli.ch/iban-pruefen-validieren) and it is valid.
Could you please add support for IBAN with letters at the end?

SwissQRBill uses the npm module iban to verify the provided IBAN.
Unlike the web page you have mentioned, the module verifies not only if the checksum is valid, but also whether the structure of the IBAN matches country specific specifications. Based on the source code of the iban module, letters in the account number should be supported. Therefore I think the error is more likely to be found in these structure checks.
Since only IBAN numbers from Switzerland and Lichtenstein are allowed, and both have the same structure, this additional checks could be simplified.
I have made a branch, which removes the iban dependency and implements the simplified IBAN checks directly into the library. You can install it using
npm i https://github.com/schoero/SwissQRBill.git#Fix-#215-iban-validation
Please let me know if your IBAN passes the simplified checks.

I've just tested this, but I'm still getting an error:
/usr/local/bin/node ./RechnungMedlist.js
(node:89421) UnhandledPromiseRejectionWarning: Error: Creditor Reference requires the use of a conventional IBAN.
    at PDF._validateData (.../node_modules/swissqrbill/lib/swissqrbill.js:416:27)
    at new PDF (.../node_modules/swissqrbill/lib/swissqrbill.js:47:14)
    at new PDF (.../node_modules/swissqrbill/lib/node.js:42:13)
    at .../my.js:270:21
    at new Promise (<anonymous>)
    at generateQrInovice (.../my.js:269:12)
    at /my.js:317:17

Thank you for testing.
I just realized that I messed up the error messages there. The correct error message would be QR-Reference requires the use of a QR-IBAN (and vice versa).
I'm guessing you are trying to use a 27-digit QR-Reference in conjunction with your normal IBAN?
This is not allowed according the specifications (page 32, "Customer references"). If you want to continue using a 27-digit reference number (as in the orange ISR), you must request a QR-IBAN from your bank.

Thanks for the hint. I did not know that. I will check this with my bank.

I'm going to close this issue for now.
I have released v2.3.1 on npm including the changes regarding the iban checks and the wrong error messages.
If you need any further assistance, feel free to comment or create a new issue.


Support [model-paths]

dbt_project.yml now uses 'model-paths' instead of 'sources-path'. Refer to issue #3.

Worked for me and someone should merge it already

Sorry for the delay. Merged.

I am fixing this change to support 0.x and I will cut a new release


getAssetDecimals

Description
Get assetDecimals doesn't work for ETH.PEPE

Ideally https://midgard.mayachain.info/v2/pools will be updated with the correct decimals.


Paste from the system clipboard

With jupyterlab-vimrc and {"unnamedplus": true} I was able to copy TO clipboard system register, but not FROM it.
Is there a way to do it in both ways with vimkeys (all works with CTRL+V)
There was a similar yanking to clipboard problem, and  that was the reason: https://github.com/codemirror/CodeMirror/issues/6341
@ianhi had proposed the workable solution: https://github.com/codemirror/CodeMirror/issues/3075#issuecomment-612319694
Now it is the part of this repo and works perfectly under jupyter-lab.
Can it also be implemented for the paste function?

I am bad in JS and web development, but from my point of view it can't be done only in some sort of firefox extension, which is not looks like jupyter-lab friendly way

Sorry I didn't respond earlier. Tragically this was also my takeaway :(. If there is a firefox extension that enables this on localhost then I think we could work towards this - but I don't.
So until then I think we're stuck with entering insert mode and doing Ctrl-V.


Add casts to avoid some 'non-existent property' warnings from Closure…

… Compiler.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements

[ ] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

Googler here. This PR is required for an internal cleanup. @IgorMinar advised to send PR instead of submitting internally.

Which branches does this need to go in?

Internally it's v1_5, so any branches that correspond to that.

LGTM. can we please get this merged to master soon and cherry-picked to v1.5.x to unblock an internal refactoring? Thanks!

Done


[UI] Inconsistency in the sidebar icons

Current Behavior

Only few icons in the sidebar  get blue, when user has selected them.
Desired Behavior

All icons in the sidebar shold get blue, when user has selected them. So that, user can know where he is.
Screenshots/Logs

https://user-images.githubusercontent.com/103871275/211491333-f2b90467-b52c-4c8e-9566-416d2b2acb07.mp4
Environment

Browser: Firefox
Host OS:Linux
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Kubernetes

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hey @leecalcote I would like to work upon this

Thank you for the issue @hs05june.


Add aria2 to tools list (step 2)

Add aria2 to the list of tools
Add the aria2 path to the cmake parameters.

Requires https://github.com/microsoft/vcpkg/pull/20971 first.

Well I didn't mean to push my suggestions into your branch but when I attempted to push to my demo branch it pushed to your branch instead.
Can you take a look at the changes I submitted, and confirm that you agree with them? Otherwise, feel free to chop my edits off (git push -f) and do something else. Point is though that we should find the aria2 from the system and always return that rather than potentially sending broken values into cmake.

Test result on Linux:
usr@usr:/home/vcpkg# ./vcpkg fetch aria2
/usr/bin/aria2c

Please note apt only provide aria2 version 1.33.1, the user may need to compile and install it manually. @BillyONeal

Thanks for adding the fun fetch!


Add access to 3D Tiles Loaded / Progress events

In Cesium Native, there are progress events for when 3D tiles are loaded, but we don't "bubble them up" to Cesium for Unity. It'd be nice to let the user access them.

This is frustrating. If my users are on a slow connection everything else finishes loading way before the tiles and they see a lovely empty screen for a bit, or a patchwork.  Please add this capability.

We address this in #356, which shipped in v1.6.0.


Could not find class: apple.awt.CGraphicsEnvironment

When I execute multi screenshots in a test method, Spoon would generate a gif for those screenshots.
I meet an error "Could not find class: apple.awt.CGraphicsEnvironment" in OS X environment. Does it has workaround about this condition?

What version of the plugin are you using?

I use plugin version in "com.stanfy.spoon:spoon-gradle-plugin:1.0.1".

And does example in this repo work for you?
https://github.com/stanfy/spoon-gradle-plugin/tree/master/example
One of the tests ("Set Text") in this example generates multiple screenshots and I can see a generated gif in the report (on Mac).
You might want to change plugin used plugin version at
https://github.com/stanfy/spoon-gradle-plugin/blob/master/example/build.gradle#L9
(to 1.0.1).

It runs multiple flavors, you can see a report opening
example/app/build/custom-report-dir/free/debug/index.html

I can not find 1.0.2-SNAPSHOT version in mavenLocal. Do I need another setting to set path of mavenLocal?

Just set it to 1.0.1 to get from maven central.

I use example and set 1.0.1 and still meet the same problem. But it is no problem to use Mac of my colleague to run test. Does the configure of computer environment influence the result about generating gif? However, I have no idea about this. Do you have any idea about this? Thanks.

You probably have some problems with Java setup (paths, missing jars in jdk libs folder, anything).

Ok, I see. I would try setup Java again. Thanks for your help!

@fantasy1022 were you able to fix that issue?

I do not find solution to fix this issue.

I'm having the same problem when I run spoon from android studio. I think that maybe is related with this issue
https://code.google.com/p/android/issues/detail?id=162448.
When running from the terminal works fine for me.


Plugin Load Error

Hyper 1.3.1
Electron 1.4.7
darwin x64 16.5.0

you're better off opening an issue at https://github.com/Kikobeats/hyper-flat


Add installation instructions for a released version

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Which issue(s) this PR fixes:

Part of #197
Special notes for your reviewer:

/lgtm


Add support for huggingface_hub

Is your feature request related to a problem? Please describe.
We currently download stuff using pooch from huggingface. This however has its own API that can be used somewhat similarly to tensorflow_hub` from what I can see.
Describe the solution you'd like
Later, people should be able to upload their models and data to huggingface, and we would probably want to encourage that to have a reliable interface. Therefore, the current model download approach should be replaced with something that uses huggingface_hub.
Describe alternatives you've considered
Generic download is what it`s currently doing, but since ML hubs exist and are widely used, it seems best to use them where possible.

closed because we took a more flexible approach.


Add a variant using the pulp crate

It's a bit different architecturally, but seems pretty cool.

@sarah-quinones if you ever have the time, you could probably do this way faster than I could: I'm trying to implement all the different API variants in one repo.

sure thing, i actually made one last week. im a bit busy right now but i can clean it up and send it your way next weekend

#7 opened a PR

Fixed by #7.
Interestingly on my computer pulp is somewhat slower than std::simd (but faster than wide, so second fastest).

if you don't have avx 512, the reason might be that the larger vector width with std simd is making it unroll the loop

I'm on i7-12700 so yeah there's no AVX-512.


doc: add ability to live reload and regenerate HTML

Just to make things easier when editing the documentation.
R=@libuv/collaborators

LGTM but shouldn't you have a variable for sphinx-autobuild like there is for sphinx-build?

Added the variable and landed, thanks Ben!


refactor: plugin system for projects and dependencies

What I did

fixes: #398
How I did it
How to verify it
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

OKAY this PR has reached the point of massive-ness so I am going to open smaller PRs to help with this.

Currently, we don't support downloading from private repos on GitHub. Maybe a ape-github first class repo would make sense to keep that out of core ape?

@fubuloubu As far as I can tell, it is about ready, though I should probably come back to it with fresh eyes later on.


Is there any way to embed Facebook post like Instagram using post id?

Hi @ansarizeeshan, there isn't, sorry.
It looks like Facebook provide a plugin to care of this pretty well, so I'm not sure if it's something worth trying to add: https://developers.facebook.com/docs/plugins/embedded-posts

Thanks @cbotman .


Empty datatip below the normal one

I observe this artefact since I installed this package:

I tried out changing the theme, but it didn't help. I suspect that it could be conflicting with the ide-ui package, but I don't know how to test or resolve it.
Atom 1.36 (was the same with 1.35).

Can you give me a link to a sample project to verify it? What Atom language | ide-package are you using?

I was using it with the ide-scala package (I'm the author) for Scala development. But I just tried it on a typescript project and got the same result:

This is from the atom-languageclient project.

Ok, I will have a look. Could you please check if you have disabled the data tip and signature help functionality from you ide-ui package after installing the replacements from the atom-ide-community?

Looks like having something to do with your setup... I've no ide-ui package installed; working just with the datatip and signature-help ones gives me that:


Auto install isn't working

I run the script, but after rebooting nothing happens

Please check that the dependencies are installed, you have to install them manually if you are not using archlinux. https://github.com/Deathemonic/SweetDots/wiki/Documentation-(Xorg)#dependencies
Sorry for the long response I was busy from school


Styling master document from stylesheets defined in html is deprecated (bug)

I create a polymer 2 starter kit project from polymer cli and after running I checked the console on opera and found following error,

app seems to work fine but I thought this error/warning might be worth considering for the polymer team :)

Duplicate of #4679. Fixed by #4849
See this comment explaining why this warning appears regardless of the fix.


Tutorial for FSI VaR MonteCarlo

Work in progress. Trying to get demo ready for SC23.
Not completed:

readmes
full workflow

Created modules for:

vertex ai notebook
pubsub topic / subscription
BigQuery Dataset / Table.

PTAL let me know if this is on track

I don't see that I can add a label. I assume that is on purpose.
Thanks!

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun

@jrossthomson I went ahead and addressed the majority of my findings.
Please review 1533b714bdafe7c6a8b9b1799d56b7fdf8738e4f and make sure you agree with my changes. Generally I tracked these in comments if you want more context.
I also needed to incorporate updates from develop to get tests to pass. I did not want to force push over your history so I manually merged in the latest copy of develop.
There are still a few open items for you to address. Feel free to reach out to discuss.

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Do not release the PinotDataBuffer when closing the index

The PinotDataBuffers are tracked and maintained inside SegmentDirectory
for ImmutableSegment and PinotDataBufferMemoryManager for MutableSegment.
They are created when loading the indexes and released when closing the
segment. If the PinotDataBuffer gets released when closing the indexes,
and if the buffer manager decides to reuse the buffer, the following read
on the buffer will cause JVM to crash. This can be triggered in
SegmentPreProcessor when the same indexes need to be opened twice in two
different preprocessors.
This PR standardize the behavior of indexes to not release (close) the
PinotDataBuffer when closing the index.
We don't need to worry about accessing the index after it is already
closed because that is already addressed in #4764

I guess the claim is that memory manager needs to be closed at the close of a segment. It will be good to put that comment in the memory manager interface. If for any reason the memory manager is retained across interfaces, a lot of memory will not be released.

Both segment directory and memory manager are closed when destroying the segment, which is the expected behavior. They are responsible of allocating and releasing the buffers, and might reuse the buffer across different calls.

The 'reused later' comment needs some explanation. What may be the re-use of those buffers?

In SegmentDirectory, we cache the buffer for each index and return the same buffer if the same buffer is requested twice.


Missing changelog entry Text <IP_ADDRESS>

What the summary says.

Fixed by https://github.com/bos/text/commit/9db9d45e1fc5eb511665e86dcfe5663043ad566d.


Scrolling Multiple Selection Dropdown?

I'm trying to get the tokenizing dropdown to behave like a regular one and rather than expanding the input height - keep the tokens on one line and scroll the overflow.
I've added white-space: nowrap and overflow: scroll to the dropdown which obviously cuts the menu - but then removing relative positioning from the dropdown, adding it to a parent wrapper and then making the menu fixed should pull this out to the viewport and display it and it doesn't!
http://codepen.io/bezoerb/pen/AGgcu
You can see how this should work here, but I can't find out why the the fixed menu element is still relative to the dropdown - any ideas? Codepen below:
http://codepen.io/adamccc/pen/xRdGvY

Hi @adamcollingburn,
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost this issue, please use one of our other support resources:

Gitter chatroom
Official forums
StackOverflow


Add doc for using ROS

Add documentation for publishing joints state to ROS, based on https://github.com/robotology/community/discussions/545

@pattacini @traversaro

fyi @lrapetti @S-Dafarra

I think in AnDy demos we also display the iCub robot on RViz @lrapetti @Yeshasvitvs, in a following PR we may add this info as well to this documentation.

I think in AnDy demos we also display the iCub robot on RViz @lrapetti @Yeshasvitvs, in a following PR we may add this info as well to this documentation.

Thanks @traversaro, we have not been visualizing iCub on RViz since quite a long time, but it might be handy in the future

Some minor comments. I wonder if we should clarify that this page is related to ROS 1 and does not cover at all ROS2 .

I concur with this suggestion: it would help out.

Hi @traversaro, @pattacini,
Thank you for the feedback, I took it into account. Let me know if I missed anything!

Thanks @cedricgoubard for the amendments.
Would it be possible/easy for you to commit with your username instead of the generic icub?

Sorry about that, it should be fixed now.

I think in AnDy demos we also display the iCub robot on RViz

Reference: https://github.com/robotology/human-dynamics-estimation/tree/master/conf/ros/launch

I think in AnDy demos we also display the iCub robot on RViz

Reference: https://github.com/robotology/human-dynamics-estimation/tree/master/conf/ros/launch

For future reference, also this doc seems related: https://github.com/robotology/icub-models/tree/v1.21.1/ros .


add external-secret support for tencent cloud

This commit will add Tencent Cloud Secrets Manager support into kubernetes external secret

It supports polling by taking secret ID and secret Key from environment values
It supports roleName, which can be consumed by CVM attached role

@Flydiverny Dear Markus, I have successfully verified the code in Tencent Cloud Kubernetes clusters (TKE and EKS). Could you please help me review the code?

@yagrxu hey! We are migrating efforts from KES to ESO:
https://github.com/external-secrets/external-secrets
Would you want to implement this provider there since KES will eventually be deprecated?

@knelasevero sure, I can do both~