Introduce the rawblock snapshotter

Hello,
The PR introduces a new snapshotter for containerd named rawblock, based on the original work from @laijs in hyperhq/hyperd. It is a file based block storage snapshotter. Each snapshot is a file on the host file system and can be mounted via loopback device to provide a file system view. Child snapshots are created via local file system's reflink capability (when available). The reflink capability is implemented in several Linux kernel file systems such as btrfs, xfs and NFSv4.2. With it, the rawblock snapshotter can be pretty space efficient and convenient to use.
A few interesting questions people might ask:

Does it share page cache among different layers?

No, currently not. Different layers are mounted as different file systems and each will maintain its own page cache. But it is possible to bypass the page cache completely if the host file system providing reflink capability also supports DAX at the same time. Of course one would need fast storage (e.g., nvme) to gain performance back.

How many layers of page cache are involved?

One. Loopback device double cache problem is the past. The loopback device is configured to use direct IO and thus it avoids page cache on the host file system.

Can it be used to implement other types of file based block storage snapshotter, like qcow2?

Yes, it can. The only difference of different types of file based block storage snapshotter is the methods of creating new snapshot files. We can add a type field in the snapshotter config to specify which method to use. But let's add things one at a time ;)

What are the advantages and disadvantages compared to devmapper?

To compare them is to compare dm-thin vs. file system snapshots. Here are what I can find/think of so far:

Pros:

Simplicity. dm-thin requires an additional block(btree) mapping from the thin device to the blocks in the data file, and needs a userspace daemon to monitor and increase the size of the thin-pool data file automatically. Rawblock does not have such complexity, -- the snapshot functionality is handled by host file system together with file block mapping. Free disk space is also managed by host file system internally.
dm-thin data file did not return disk spaces back to host file system after deleting thin provisionings/snapshots. There is a long standing moby issue for it moby/moby#3182. In theory it can be fixed by fallocate(2) FALLOC_FL_PUNCH_HOLE so I'm not sure if it is still the case.

Cons:

If host file system does not support reflink, rawblock snapshot creation falls back to plain data copy and can be very slow.
dm-thin allows to put metadata and data files separately. So it is possible to put metadata file on a fast storage (e.g., nvme) to gain better metadata performance.
dm-thin pool metadata and data files can be raw block devices and thus removing the extra layer of host file system, making it possible to be faster than rawblock snapshotter due to one less file system layer.

/cc @laijs @lifupan

Codecov Report

Merging #3130 into master will increase coverage by 0.04%.
The diff coverage is 44.19%.

@@            Coverage Diff             @@
##           master    #3130      +/-   ##
==========================================
+ Coverage   43.58%   43.62%   +0.04%
==========================================
  Files         104      107       +3
  Lines       11156    11511     +355
==========================================
+ Hits         4862     5022     +160
- Misses       5555     5709     +154
- Partials      739      780      +41

Flag
Coverage Δ

#linux
47.47% <44.19%> (-0.1%)
:arrow_down:

#windows
40.47% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

mount/mount_linux.go
28.99% <0%> (-2.02%)
:arrow_down:

mount/losetup_linux.go
0% <0%> (ø)

snapshots/rawblock/rawblock.go
53.55% <53.55%> (ø)

snapshots/rawblock/config.go
73.91% <73.91%> (ø)

Continue to review full report at Codecov.

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

dm-thin data file did not return disk spaces back to host file system after deleting thin provisionings/snapshots.

@bergwolf Hi, do you mean?
Although fstrim the mountpoint which will make discard request to thin device and free disk space can be returned back to the thin device,  the fs still cannot use those space?

dm-thin data file did not return disk spaces back to host file system after deleting thin provisionings/snapshots.

@bergwolf Hi, do you mean?
Although fstrim the mountpoint which will make discard request to thin device and free disk space can be returned back to the thin device, the fs still cannot use those space?

hmm,  if used by kata runtime, each thin device will be passed through to the guest,  why is there "host filesystem"?

@renzhengeek

hmm, if used by kata runtime, each thin device will be passed through to the guest, why is there "host filesystem"?

It isn't kata specific. I was referring to the dm-thin pool data and metadata files. They are usually files on the host file system.

Although fstrim the mountpoint which will make discard request to thin device and free disk space can be returned back to the thin device, the fs still cannot use those space?

dm-thin needs to translate the trim command into FALLOC_FL_PUNCH_HOLE to tell host file system to make the data file sparse again. I'm not sure if it is implemented or not. It should be able fix moby/moby#3182 though.

Hi,

dm-thin needs to translate the trim command into FALLOC_FL_PUNCH_HOLE to tell host file system to make the data file sparse again. I'm not sure if it is implemented or not. It should be able fix moby/moby#3182 though.

Aha, I probably got the point. The DM devices you talk about here use files as underlaying storage, not raw disk. Thanks.

Is there a separate repository this driver currently exists in and shown as useable through the proxy driver interface?

@dmcgowan No, there isn't yet. Due to its simplicity, I want to propose to make it builtin. The actual implementation is quite simple. Almost half of the patch is to enable the mount.All API to handle -o loop option. That part of code is actually a missing piece of the API, and can be used to simplify the already merged devmapper snapshotter as well.

@dmcgowan For testing, one can simply use the --snapshotter rawblock option. Is there any special testing utilities you want to see integrated with this snapshotter?

CI failure seems unrelated?

E0314 09:51:02.722787    3768 remote_runtime.go:200] CreateContainer in sandbox "7fb3519b6b340b0ab229a65499e59cc3262b9fb1a5954d8885cc460e1b6f4b9b" from runtime service failed: rpc error: code = Unknown desc = : failed to generate seccomp spec opts: invalid seccomp profile "/tmp/seccomp-tests102614447/block-host-name.json"

Another network issue?
Failures
 - mingw (exited 1) - mingw not installed. An error occurred during installation:
 The remote name could not be resolved: 'packages.chocolatey.org'
Command exited with code 1

Another network issue?
Failures
 - mingw (exited 1) - mingw not installed. An error occurred during installation:
 The remote name could not be resolved: 'packages.chocolatey.org'
Command exited with code 1

re-running it

Due to its simplicity, I want to propose to make it builtin.

Maybe we can accept this as a non-core project and as a gRPC plugin first.
We can consider making this built-in later if there is a huge demand.

@AkihiroSuda In that case, we still need the first commit https://github.com/containerd/containerd/pull/3130/commits/37c9ddbc23019e5e121d2a02752cf687343026ab to add loopback support to containerd's mount package. What do others think? I can split the PR if that is the decision.

In that case, we still need the first commit 37c9ddb to add loopback support to containerd's mount package.

SGTM

Please see #4178 for add loopback support to the mount package.

Loopback mounts PR is in, please open a proposal for a non-core project in order to get started with grpc version of the plugin.

Loopback mounts PR is in, please open a proposal for a non-core project in order to get started with grpc version of the plugin.


Taxon constraint: GO:1902017  regulation of cilium assembly

Please provide as much information as you can:

GO term ID label:

GO:1902017  regulation of cilium assembly

Request to add a taxon constraint:
need to check and add the same taxon constraints as cillia related terms

Request to remove a taxon constraint:
Please specify

Supporting evidence if available (e.g PMID):

duplicate


Abort in request_finished()

https://sentry.adguard.com/adg/adguard-windows/issues/21592
All crashes are on Win7, TDI Driver

A new bunch of crashes - https://sentry.adguard.com/adg/adguard-windows/issues/25791 - cl1.4.32

run stress-test with ag_set_dns_servers and try to debug find the race
if not solved, drop DNS request on which assertion is failing with warning

core/pull-requests/1477

Move back to Current Sprint when new crash reports with extended logging are available.

There is no new crashes


Update smarter encryption to latest bloom and remove service

Task/Issue URL: https://app.asana.com/0/361428290920652/1195088920676364/f
Description:

Update to latest bloom filter
Embed smarter encryption data in app
Remove backend calls to smarter encryption

Steps to test this PR:
Test fresh installation with backend data:

Install a fresh installation of the app, wait for all data to sync
Navigate to http://facebook.com and ensure this is upgraded
Navigate to some long url that doesn't exist and is thus definitely not in our list e.g http://thisUrlDoesNotExistAndIsNotInOurSmarterEncryptionList. com and ensure this is not upgraded

Test fresh installation with embedded data:

Block all calls to out smarter encryption endpoints in Charles these are https://staticcdn.duckduckgo.com/https/https* or make breaking changes to the urls at AppUrls.swift so they definitely fail
Install a fresh installation of the app
Navigate to http://facebook.com and ensure this is upgraded
Navigate to some long url that doesn't exist and is thus definitely not in our list e.g http://thisUrlDoesNotExistAndIsNotInOurSmarterEncryptionList. com and ensure this is not upgraded

Test migration:

Install and run app from develop, making sure that all data is downloaded
Now install this branch over the top
Ensure no db migration crashes occur
Navigate to http://facebook.com and ensure this is upgraded
Navigate to some long url that doesn't exist and is thus definitely not in our list e.g http://thisUrlDoesNotExistAndIsNotInOurSmarterEncryptionList. com and ensure this is not upgraded

Internal references:
Software Engineering Expectations
Technical Design Template

LGTM Thanks.
BTW, bitrise is currently failing, but the work done here is unrelated to that. Feel free to merge this PR even if bitrise fails.

Just make sure the tests pass locally before merging 👍

Thanks for the review!

Merging as per previous comments despite BitRise tests issues across our branches. Tests passing fine locally and on one BitRise (Nexus 5) device.


quic: include all required components when extension is included

Signed-off-by: Greg Greenway<EMAIL_ADDRESS>Commit Message: This causes all quiche components that register a factory to be linked into the build when the extension is included in the build.
Additional Description: This was needed to run a simple quic server test, without the config failing to load, or aborting at runtime.
Risk Level: Low
Testing: None
Docs Changes: Not needed
Release Notes: Not needed

Without :codec_lib, an exception is thrown when the HCM config tries to create the H3 codec, which causes envoy to abort. I think that any config that specifies the HCM codec as H3 will crash right now. But that config can't work at all without some changes anyways, so I don't think it's a big deal. And AUTO won't ever select H3 right now.


Document usage with Rocket

When using anyhow together with Rocket you get the following error:
 the trait `rocket::response::Responder<'_, '_>` is not implemented for `anyhow::Error`

This is related to: https://github.com/SergioBenitez/Rocket/issues/856 and can be fixed by wrapping the error type with rocket::response::Debug:
#[get("/foo")]
pub fn foo() -> Result<i32, rocket::response::Debug<anyhow::Error>> {
    Ok(1)
}

Due to the popularity of rocket my guess is that many people may stumble opon this issue, so I would suggest documenting it in the README.

I think I would prefer not to add Rocket-specific documentation in this crate. If I got that error message, the place I would think to look is the documentation of rocket::response::Responder.

How about a section of document for integrating with other popular framework/libraries?

I'd suggest that this should live on rocket's doc, not anyhow's doc

I'd suggest that this should live on rocket's doc, not anyhow's doc

Wrapping anyhow::Error with rocket::response::Debug doesn't fix the issue on the example code posted above. I get this compile error with Anyhow 1.0.38 and Rocket 0.4.6:
error[E0277]: the trait bound `std::result::Result<i32, rocket::response::Debug<anyhow::Error>>: Responder<'_>` is not satisfied
   --> src/main.rs:44:15
    |
44  | fn index() -> Result<i32, rocket::response::Debug<anyhow::Error>> {
    |               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `Responder<'_>` is not implemented for `std::result::Result<i32, rocket::response::Debug<anyhow::Error>>`
    |
   ::: /Users/mkoval/.cargo/registry/src/github.com-1ecc6299db9ec823/rocket-0.4.6/src/handler.rs:202:20
    |
202 |     pub fn from<T: Responder<'r>>(req: &Request, responder: T) -> Outcome<'r> {
    |                    ------------- required by this bound in `handler::<impl rocket::Outcome<rocket::Response<'r>, rocket::http::Status, rocket::Data>>::from`
    |
    = help: the following implementations were found:
              <std::result::Result<R, E> as Responder<'r>>
              <std::result::Result<R, E> as Responder<'r>>

I'm not sure how to debug this. It looks like it there are two different implementations of Responder for the generic Result<R, E> type, neither is applicable here. How is that possible? Are some type bounds omitted?


Extension is not deactivated on closing the editor window

From @ArtemGovorov on November 16, 2015 7:15
My extension spins up a few processes that I'd like to kill when it's deactivated.
The deactivation callback for ExtensionContext
context.subscriptions.push({dispose: () => ...});

works perfectly when I stop debugger for the Extension Development Host. However, if I just close the editor window on OSX, the deactivation callback is not invoked, so I can see all created processes hanging around and not being killed.
Copied from original issue: Microsoft/vscode-extensionbuilders#113

From @ArtemGovorov on November 16, 2015 7:34
Here's the workaround I use for now for OSX:
  process.on('SIGTERM', function () {
    // extension deactivation code
  });

From @bpasero on November 16, 2015 7:38
@ArtemGovorov is this only during extension development? Not when having the extension installed and closing the window?

From @ArtemGovorov on November 16, 2015 7:39
@bpasero Same for the installed extension (on OSX)

From @bpasero on November 16, 2015 8:2
@jrieken fyi that would be pretty bad, is the PH connected to the lifecycle service somehow?

From @egamma on November 16, 2015 8:7
@dbaeumer you shut down the language server on shutdown and it works correct?

From @dbaeumer on November 16, 2015 9:29
@egamma no, I currently fix this on the server side by observing the streams and exit the server is the input stream closes.

See also: https://github.com/Microsoft/vscode/issues/537

With 98473e0e88cecd52ba44551d512efe445b6af11e, VSCode's extension host will now call deactivate() if extensions export it and call dispose() on ExtensionContext.subscriptions on shutdown, then will wait 1s before exiting

this was in the test plan


VideoRehab: Camera change to back not working on Android devices

Describe the bug
When trying to switch from the front cam to the back camera on Chrome on Android, a "Not Readable Error" is displayed.
To Reproduce

On the main "participant" page, try to change the video source from the combo box

OR

In a session, try the change the video source from the "gear" settings

Expected behavior
Camera should be displayed properly without user error
Desktop (please complete the following information):
Mobile Chrome Android

In the participant lobby, stopping the current video stream and selecting the new one works fine.
However, in a session, the issue is strange... Even if trying to stop the current stream when switching, we're not able to change to the back camera (it takes about 20 seconds before the camera becomes available). Seems to be related to EasyRTC?
The only work-around is to preload the back camera at the start of a session, and not use it for at least 20 seconds...

Seems that recent changes (#242 ) brought back that issue...


winpos doesn't position window

If it's possible and not too difficult, another Vim GUI command that would be very helpful for me is :winpos X Y, where X and Y are numbers of pixels.   This sets the upper left coordinates of the GUI window to be at  X,Y.  (I want to combine this with the window size commands.)
I know you're busy and can't implement everything, but I thought I'd mention this for future reference.  Thank you!

@mars0i
Hi :)
Currently, it seems that neovim does not support winpos command. Therefore, it will be necessary to modify the neovim side to implement winpos command.
On the other hand, it would be possible to implement the same functionality with Goneovim's own commands.
If I have time, I would like to implement it.

Thanks @akiyosi.
(That makes sense that nvim doesn't have it.  Running nvim in iTerm2, where the winsize and set columns, etc. commands work, winpos doesn't work.  It doesn't seem like terminal apps would allow apps running in them to change the window position.)

I thought I might try to implement this function.  I don't have time to learn much about Go or Qt unfortunately, but it seems like it ought to be a relatively simple thing to implement, so I thought I'd give it a try.  If I don't succeed because I don't know enough, that's OK.
I'm finding that it's tricky to get the "setup Go binding" step on the Development page to work.  I am not sure, but it seems as if I have to have exactly version 5.13.0 of Qt, and the closest version that the installation tool offers to me is 15.13.2.  Or maybe there's a component of Qt that I haven't installed?  I don't know.  Since I don't have time to learn much about Go and Qt--which might be a mistake--I may just be stuck.
This is not your problem!  I'm just letting you know.  I might figure it out.
(I have a couple of lines of code that I added to workspace.go that might be enough, but I haven't been able test it, and it's unlikely that what I've added is enough on its own given how little I know.)

Never mind.  I figured out how to install Qt 5.13.0.  I'll see whether that works.

@mars0i
Hi :D
Thank you for your willingness to contribute.
I would like to support you in any way I can.
Now, I apologize for the lack of clarity in the documentation.
Actually, Qt versions 5.13.X, 5.14.X, 5.15.X are all fine. (I haven't tested any other Qt5 versions recently, and Qt binding doesn't seem to support Qt6.)
The documentation describes using 5.13.0 in the QT_API environment variable, but this is like the interface version of the Qt Go binding, not the version of Qt you are using. Of course, using Qt 5.13.0 is fine.

Thank you @akiyosi!
I was able to complete the qtsetup step with a full installation of Qt 5.13.0.  (  When I used 5.13.2, the qtsetup process stopped with an error that it couldn't find a file.  I could see that a file with that name was somewhere under the 5.13.2 directory, but the qtsetup process wasn't finding it.  That's why I thought it might work if I installed Qt 5.13.0, and it did work.
I did have one warning when I ran qtsetup with Qt 5.13.0:
stat /Users/myusername/go/src/github.com/therecipe/env_darwin_amd64_513/5.13.0/clang_64/lib/QtWebEngineCore.framework/QtWebEngineCore: no such file or directory

I got the same message when I ran $(go env GOPATH)/bin/qtdeploy build desktop, but the build seems to have succeeded.  I'm able to run the newly created binary.

Hi @akiyosi. I have a working version of a new :GonvimWinpos {x} {y} command here: https://github.com/mars0i/goneovim/commit/ab76d0f1c96241e69e64f7172f3f44b4ab528bb4 .
Please let me know if you have feedback at this point.  Maybe a different name for the command or some of my variables would be better, maybe the code needs more error handling, or maybe it would be better to use entirely different methods.
I'm still experimenting with the GonvimWinpos version of goneovim.  I'm finding that after a few minutes it would go to > 100% CPU.  Maybe this is related to #345, or maybe it has to do with what I added or how I was running goneovim: I just ran the binary file from inside the goneovim/cmd/goneovim/deploy/darwin/goneovim.app tree under ~/go.
I think that I am building using the latest versions of the goneovim and goqtframlesswindow source, but I'm not certain.  I'm still figuring out how to use the go build and configuration tools, and I haven't yet figured out how to pull updates to the code from the repos (goneovim, goqtframelesswindow), as I would do with git pull.
(If I run go get -d github.com/akiyosi/goneovim/... now, in order to get more recent source, will it overwrite what needs to be overwritten, and clean out what is stale?  Can I use git clone instead?)
Thanks!

@mars0i
Hi :D
Thank you for your contribution! It looks good to me.
I am waiting for your PR.

I'm still experimenting with the GonvimWinpos version of goneovim. I'm finding that after a few minutes it goes to about 100% CPU. (I haven't seen that with the goneovim version I have installed in /Applications, which is nightly-1-g0e35e77.) Maybe this is related to https://github.com/akiyosi/goneovim/issues/345, or maybe it has to do with what I added , or how I was running my new version of goneovim: I just ran the binary file from inside the goneovim/cmd/goneovim/deploy/darwin/goneovim.app tree under ~/go. Also, go test gives me warnings about object files that were "built for newer macOS version (10.15) than being linked (10.12)". Could that cause the high CPU usage?

I have tried your branch on my machine and :GonvimWinpos command is working fine and the CPU never seems to go to 100% when I am not operating anything.
Indeed, the problem with your Goneovim going to 100% after a few minutes could be the same problem with the #345.
It might be a good try to test with goneovim -u NONE.

(If there are changes in the master branch, and I run go get -d github.com/akiyosi/goneovim/... now, in order to get more recent source, will it overwrite what needs to be overwritten, and clean out what is stale? Can I use git pull instead?)

I understand that go get command uses git mechanism internally, so each repository downloaded under $GOPATH is a git repository and can be controlled by commands such as git fetch in each directory.

Excellent!  I'll submit the PR in a moment.  Being able to set the window position programmatically is important to me.  I'm happy that I was able to contribute.
About CPU usage, yes with -u NONE my new version of goneovim does not have high CPU usage.  I'll continue my remarks about this at #345, since it seems to be the same issue.


Update pom to do Apache relase

Now that Fluo has moved to Apache we need to update the pom to support creating an Apache release.

what needed to be done in this? Maven airtifact publishing need to be done in this issue
http://www.apache.org/dev/publishing-maven-artifacts.html

Essentially, we need to switch from the fluo parent POM to the ASF parent POM, which serve similar purposes. There may be things in the fluo parent POM which are not in the ASF parent POM. In those cases, we need to reconcile the differences, either by dropping them, pushing them down into the individual project POMs, or by turning the current fluo parent POM into an intermediate POM, whose parent is the ASF parent POM (which I think is probably the best route).

I like the path of making the current parent pom have Apache as a parent and reconciling the differences.

@Garvit244 responding to your email for more info here on the issue.  I am not sure exactly what needs to be done, but I will try to explain what I think needs to be done.  I may be wrong about some things.
Some quick background.  The Fluo project pom provides common functionality across Fluo project poms (Fluo pom and Recipes pom).  For example the Fluo parent pom sets up code formatting, findbugs, etc.  Therefore each project pom does not have to duplicate all of this.  The projects poms used to duplicate all of this before creating the Fluo project pom.
Now that we are moving to Apache we want to use config from the Apache pom to do releases.   We also want to keep things from the  Fluo Project pom like code formatting.  To accomplish this we can add the Apache as a parent of the Fluo project pom.
Also we need to remove the current info about releasing to Sonatype (where io.flou releases were done) from any of the poms where it may occur.
The following is the high level overview of what I think needs to be done.

Add Apache pom as parent of Fluo project pom
Reconcile plugin config that occur in both poms.  Feel free to ask about reconciling any duplicate plugins with different config.
Remove Sonatype distribution management section from Fluo project pom.  Look for other Sonatye stuff to remove. (should inherit distribution management from Apache parent pom)
Release new version of Fluo project pom
Update Fluo pom and Recipes pom to use new version of Fluo project pom.

We were planning on releasing the Fluo project pom to Sonatype because we have not transitioned that project to Apache yet.   @ctubbsii  how can the Fluo project pom release to Sonatype if it drops Sonatype distribution management and inherits Apache distribution management?

An alternative to modifiying the existing  [Fluo project pom][1] is to create a new parent for Fluo and Fluo Recipes.  This could be stored in a branch of Fluo.   Then we would have a graph like the following.  This idea was suggested by @ctubbsii yesterday offline and I just remembered it.

FluoProjectParent has a parent of Apache pom
Fluo has a parent of FluoProjectParent
Fluo Recipes has a parent of FluoProjectParent

FluoProjectParent would be brand new and would have code formatting, findbugs, etc.   FluoProjectParent would be stored in a branch of the Fluo project that only has the pom.

@keith-turner Second approach looks good to me by that I didn't need to do any changes in current Fluo project pom. When new FluoProjectParent is done we only need to change Fluo project pom until then we can use current Fluo project pom

I dont have access to create new repo in Fluo-io. Can somebody create it then I will add all common resources ther.

Few questions:
Q1:  There are some of the resources which are in Apache Parent Pom but there configurations are different from what we use. For example:
Main Flo-parent-pom:

org.apache.rat
apache-rat-plugin

check-licenses

check

In apache parent Pom:

org.apache.rat
apache-rat-plugin
0.11

DEPENDENCIES

org.apache.maven.doxia
doxia-core
1.2

xerces
xercesImpl

@Garvit244 I'll create a separate branch called "fluo-parent" for you to do a pull request against. In that pull request, you can point its parent to the Apache parent, and rename this pom to "fluo-parent", as well as reconcile the redundant areas.
You can then do a second pull request against the "master" branch to rename it's root pom's artifactId from "fluo-parent" to just simply "fluo", and point its parent to the new "fluo-parent" which is in the "fluo-parent" branch.
As for the different configurations of plugins, most of what's in the Apache parent pom are in the pluginManagement section, and only specify the versions of the plugins. Our configuration is typically setting the behavior of the plugins.

I've migrated the old fluo-io-parent to the new fluo-parent branch, so it preserves history from that, to get you started.

I will do changes in new branch of fluo-io-parent repo and will do changes in that branch.
As Keith pointed out that

We were planning on releasing the Fluo project pom to Sonatype because we have not transitioned that project to Apache yet
So, Is there a need to change the distributionManagement in new parent pom also or can we use apache distribution management.
Because when we are using apache as parent pom it has its own distributionManagement name so if we again specify distributionManagement in new parent-pom file it will override apache
distributionManagement.

@ctubbsii are you thinking that we will release the pom in https://github.com/apache/incubator-fluo/commits/fluo-parent through Apache?  I guess that could be the first release we make as an incubator project.

@keith-turner Yes.

Now that the fluo-parent POM has been pushed to the fluo-parent branch, we can proceed updating the master branch to use it.


Build distroless proxy_init image with go version of istio-iptables

On step of plan in #12122

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: kramerul
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kramerul: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kramerul: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kramerul: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-mixer-k8s-presubmit-tests-master

prow/istio-pilot-multicluster-e2e.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test istio-pilot-multicluster-e2e-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-new-install-k8s-presubmit-tests-master
/test integ-mixer-k8s-presubmit-tests-master
/test istio-pilot-multicluster-e2e-master

@howardjohn Mind giving this a review? :)

@kramerul: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-mixer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-mixer-k8s-presubmit-tests-master

prow/istio-pilot-multicluster-e2e.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test istio-pilot-multicluster-e2e-master

prow/integ-new-installer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-new-install-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kramerul: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test istio-pilot-multicluster-e2e-master

prow/integ-new-installer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-mixer-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: howardjohn, kramerul
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [howardjohn]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kramerul: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-mixer-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3-master

@kramerul: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/integ-new-installer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-new-install-k8s-presubmit-tests-master

prow/integ-mixer-k8s-presubmit-tests.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test integ-mixer-k8s-presubmit-tests-master

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
bdaca9b769a30c2496bad30788f90d61116103f6
link
/test istio-pilot-e2e-envoyv2-v1alpha3-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-pilot-e2e-envoyv2-v1alpha3-master


Various cleanup

Description of change

discard one unused variable,
remove one useless,
disable some old dead Err defines, and discard associated macros.

Targets:

[X] master
[X] branch 1.1.0
[X] branch 1.0.2 ( p5_pbev2.c only concerned, and if you want)

+1

@levitte : changes done .
PR rebased.

Merged (I did the port to 1.0.2 as well)

Thanks !


Y2K-1795 Delete account on mobile

The approach I took was adding an optional passphrase param to the RPC and prompting the user for a password in the mobile with password case.
The rest of the cases remain unchanged and work for me from local testing
@keybase/y2ksquad

we should have a test for case where the engine gets a passphrase in argument

Should confirm with Patrick this is the right way to send passwords via AVDL

cc @patrickxb

LGTM


WIP: Support batch-reading for data-types with chunksize parameter

Context
Some datasets are large and rather than dealing them with one big block, we could split the data into chunks. This PR adds batch-reading support for data formats which provides the chunksize parameter using the Pandas API.
Supported formats:

[x] CSV
[x] JSON
[x] Stata

Modifications

mlem/cli/apply.py - Added batch parameter when calling apply method - supports both importing data on/off-the-fly workflows
mlem/api/utils.py - Added batch reading support when getting Dataset value
mlem/core/errors.py - Added new errors UnsupportedDatasetBatchLoadingType and DatasetBatchLoadingJSONError for batch-reading workflows
mlem/contrib/pandas.py - Added batch-reading support for CSV, JSON data formats
tests/contrib/test_pandas.py - Added tests for supported batch-reading data formats, exception tests for unsupported batch-reading data formats

Which issue(s) this PR fixes:
Fixes https://github.com/iterative/mlem/issues/23

@mike0sv I think my forked repo is no longer synced with this repository because it's made public now (my commits are no longer updated here). Shall I make a new PR?

Closing the PR because private forks can't sync to this repository anymore since it went public. I've pushed the changes to a separate PR (https://github.com/iterative/mlem/pull/216) after forking the public repository.


TestPypi failed when deploying a new peptdeep version

TestPypi failed due to pip install current version (of peptdeep) from Pypi, which is not deployed yet. Installing from TestPypi would solve this issue
https://github.com/MannLabs/alphapeptdeep/actions/runs/10841215744/job/30084883232

Note that now TestPypi is poluted by cur version (v1.3.0), preventing us to deploy v1.3.0 on TestPypi

after uploading to test-pypi, it does also try to install from testpypi

I tried the pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple "peptdeep==1.3.0" command and it works now, so probabvly a race condition?
I added a fix here, https://github.com/MannLabs/alphashared/pull/4/commits/a4d4cb1383257e4730063c2ba0f4becaf681383c
currently re-running it

worked


Update Iron version to 0.5.

This is related to #21 and being addressed in #30.

The only possible blocker is that juniper supports the last three versions of Rust. logger 0.3.0 uses the ? operator, which wasn't introduced until 1.13. Possible resolutions:

Submit a patch to logger that replaces ? usage with try!.
Remove logger from the server example.
Wait until the next version of Rust to bump the minimum supported Rust version. 1.16 should be out on March 16.
Drop support for Rust 1.12 now.

Yeah, I think we'll drop 1.12 for now if our dependencies start requiring more recent versions. The older versions were put there mostly in order to not accidentally break support for older Rusts; not all people upgrade to the latest compiler immediately as they get released.

Release 0.7.0 with the iron-handlers feature now depends on Iron 0.5.


Event B should be unique.

FLINK-4718 Confusing label in Parallel Streams Diagram
Minor text change to clarify a mildly confusing label in a diagram.

opened a new PR with a single commit to fix the figure.  If there's a better way I should have gone about correcting this fix by closing the old PR and opening this one let me know, I can always use some tips with git.

I think this is good. Merging this...


Cannot start livestreamer-twitch-gui - Web sqlite error 5, errno 0: database is locked

Latest OS X - one patch back
latest livestreamer gui.
It worked when I downloaded it a few hours ago, came back to run it again but when I click on it in the dock, nothing happens.   No bounce, no nothing.
So I ran it from the terminal and get:
zacs-MacBook-Pro:v8toolkit xaxxon$ /Applications/Livestreamer\ Twitch\ GUI.app/Contents/MacOS/nwjs
[57755:0910/023504:ERROR:connection.cc(1060)] Web sqlite error 5, errno 0: database is locked, sql: PRAGMA journal_mode = PERSIST
[57755:0910/023505:ERROR:connection.cc(1060)] Web sqlite error 5, errno 0: database is locked, sql: PRAGMA cache_size=32
[57755:0910/023505:ERROR:connection.cc(1060)] Web sqlite error 5, errno 0: database is locked, sql: CREATE TABLE meta(key LONGVARCHAR NOT NULL UNIQUE PRIMARY KEY, value LONGVARCHAR)
[57755:0910/023505:ERROR:web_data_service_backend.cc(54)] Cannot initialize the web database: 1

I'm not exactly sure what you've done, but it seems that either the old process is still running and preventing access to the sqlite database of Chromium for the new one (although NWjs doesn't allow running multiple processes of the same application at once) or the data has somehow been corrupted. If this is the case, you should try deleting the config folder located in $HOME/Library/Application Support/livestreamer-twitch-gui/.

You were right -- there was a copy running, but it didn't show up on
cmd-tab or force kill.
But I did a ps and found it, did a killall nwjs and tried again and it
worked.
Thank you.
--Zac
On Sat, Sep 10, 2016 at 8:54 AM, Sebastian Meyer<EMAIL_ADDRESS>wrote:

I'm not exactly sure what you've done, but it seems that either the old
process is still running and preventing access to the sqlite database of
Chromium for the new one (although NWjs doesn't allow running multiple
processes of the same application at once) or the data has somehow been
corrupted. If this is the case, you should try deleting the config folder
located in $HOME/Library/Application Support/livestreamer-twitch-gui/.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bastimeyer/livestreamer-twitch-gui/issues/304#issuecomment-246119371,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIycvQERHaclbilEo6FJOn8DsoK1boRks5qotK6gaJpZM4J5rpD
.

Ok... closing this.
If it happens again, reopen the issue. I'd like to know how to reproduce this and why the first process didn't terminate correctly.

I think the problem was that I don't understand what the "OS integration" options mean.
Apparently you can make it so it doesn't show up in the dock or even in the cmd-tab list -- but I just noticed there's a little icon in the top near the time that I probably just didn't see before.
But I've never used an app on OS X that while there's a window open it doesn't show up in cmd-tab (and probably in the dock as well).   This is probably a mis-feature to allow.

Of course the fact that they are named "taskbar" and "tray" isn't a great description on os x.
Setting it to "taskbar" gives normal functionality.


update knex for security

Problem
Security updates
Changes

updated knex
update documentation regarding library move from @pipedrive namespace

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 35.518%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
466

Relevant Lines:
1284

💛  - Coveralls


Package issue updating: Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.36.32532

Brief description of your issue
When winget tries to update "Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.36.32532" it takes the VC_redist.x86.exe executable.
winget update "Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.36.32532"
Found Microsoft Visual C++ 2015-2022 Redistributable (x86) [Microsoft.VCRedist.2015+.x86] Version 14.38.32919.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://download.visualstudio.microsoft.com/download/pr/02a6d5c5-3e10-47de-8025-d97a1321d3e3/AD573D3198853FC71137A88E51ABDE844B84F29B0CE6DD91BBEC661BC0143B36/VC_redist.x86.exe
13.2 MB / 13.2 MB
Successfully verified installer hash
Starting package install...
Successfully installed

This download https://download.visualstudio.microsoft.com/download/pr/02a6d5c5-3e10-47de-8025-d97a1321d3e3/AD573D3198853FC71137A88E51ABDE844B84F29B0CE6DD91BBEC661BC0143B36/VC_redist.x86.exe is wrong
Steps to reproduce
winget update "Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.36.32532"
Expected behavior
Update "Microsoft Visual C++ 2015-2022 Redistributable (x64) - 14.36.32532"
Actual behavior
It is updating "Microsoft Visual C++ 2015-2022 Redistributable (x86) - 14.36.32532"
Environment
Windows: Windows.Desktop v10.0.22621.2134
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.20.2201.0

The proper update link should be https://download.visualstudio.microsoft.com/download/pr/02a6d5c5-3e10-47de-8025-d97a1321d3e3/5F60592799FAE0C82578112D4B621438FFC976AB39D848D8F7623F5705A83E27/VC_redist.x64.exe
I got that info from https://www.coolexe.com/1194.html and I found that site when search with google for "AD573D3198853FC71137A88E51ABDE844B84F29B0CE6DD91BBEC661BC0143B36"

This is a known issue with how installed packages are matched to available manifests.

Duplicate of #2427
Related to #2995
Related to #2893
Related to #2243


Remove/Look into Previous Page Preload

Previous page being preloaded is generally really useful when going into a content page from a list and wanting to go back to it, but with screen reading and the implementation of the feature, the previous page is before the new page in the html.
Look into and fix this.
On new page force focus to header of new page?
Remove previousPage, only leaving one?
Custom javascript to hide previous page?

Screen Reader issue dealt with in Framework 7 v2
https://github.com/framework7io/Framework7/issues/1864#event-1334036240
So upgrading to that would remove the need to remove previous page being loaded.

For now, setting up a focus on each page to a message about which page it is, which will be placed at the start of the page. Then non-visual / screen reading will go through the page as they normally would, from the beginning.


Invoking infer -g run  -- mvn compile command from Infer REPL

My Environment is like the following:

My Infer is of version 1.0.0.
I am on macOS Big Sur.
I usually run Infer with infer -g run -- mvn compile.

Hello, I was wondering if I could invoke Infer's capture and analyze facilities from the Infer-REPL. So I have looked into onDemand.ml and Maven.ml, but I still have no idea where the entry point to Infer is when I use the above command. I guess if I can get that 'main' procedure I might be able to capture and analyze from the comfort of Utop (Yes, I love Utop 🙃).
Could you please give some guidance to what I want to achieve?

Very entertaining, thanks :) We don't link the toplevel infer.ml file into the toplevel because it also runs infer when loaded, which wouldn't be desirable: https://github.com/facebook/infer/blob/dfd4dd44dd589b92ce864dd22b01ab043e2a9cbc/infer/src/dune.in#L94-L108
The functions in infer.ml could live somewhere else though, and infer.ml could just contain a single line let () = InferMain.run ().
You'd still have to restart utop everytime you want to give different config options to infer though.

@jvillard After some fiddling, I managed to scratch my own itch: invoking infer analyze -- mvn compile from Utop. Essentially, I extracted the let () = ... phrase in src/infer.ml, selected the relevant parts triggered by the above command, and wrapped those in a separate function. This lives in a new submodule declared as a library, under the directory src/Repl. [https://gist.github.com/jeongsoolee09/82c92f99a4fc17438161de243772386a][The code is shown in this gist.]
One inconvenience is that I might have to use the #cd directive to change the working directory of Utop, but that's quite minor!
Thanks for your guidance, and I am going to close this thread.


Bug Fix: Not working if overflow is on the body

Issue #98

Really appreciate the fix @canzian! Well done


Add XML support for cloudformation commands that lacked it

This lets boto3's cloudformation API work with moto.
A lot of this work was inspired by https://github.com/spulec/moto/pull/404.
fixes #444
Signed-off-by: Scott Greene<EMAIL_ADDRESS>

The same CI failures happen on my machine on master. I don't think the failures are related to my submission.

Thanks for opening. I'm excited for us to get this merged.

@spulec Got the build green!

Great, thanks!


Multiple Instances on the Same Page Has Wrapper ID Conflict

I noticed that there is an ID attached to the <div /> containing the SVG or Canvas. It's always lottie which breaks down if there are more than one instance of the player on a page. Is there a way to specify the id?

Noted. Will look into a fix.
Thank you :)

Adding the id as a prop option.

Amazing, thanks!
On Thu, Dec 10, 2020, 5:11 AM karam<EMAIL_ADDRESS>wrote:

Closed #11 https://github.com/LottieFiles/lottie-react/issues/11.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LottieFiles/lottie-react/issues/11#event-4094268533,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABVXAKVIVQIKAYCLF7QZGLSUCNE3ANCNFSM4UBOORXQ
.

Hello, it seems it still doesn't work properly. When multiple components in the same page, when you apply, for example, a speed for one of them, it applies to all lottie components in the page. Even when setting a different id for each component.

Hi @Siignature thanks for brining this to our attention. Will look in to it.

Hi @Siignature Have you updated your version of the player? I can't manage to reproduce this bug:
https://codesandbox.io/s/lottie-react-speed-test-veuux?file=/src/App.js

I am using React 16, maybe is it the problem?

@Siignature I don't think so. Are you using the latest version of lottie-react?

@samuelOsborne Yes, 3.4.1

@Siignature Could you share how you implemented it ?


Update README nomenclatures and middlewares

This commit updates the README file by adding examples of different
pages and the reasoning behind it.
It also adds information on how this prototype works by adding a
description of the middlewares.
Rendered version: https://github.com/alphagov/govuk-nav-prototype/blob/e19ad7ebdace6b616f1a5236130076878d564ad5/README.md

Thanks @alextea, I've corrected the typos 👍

Content lgtm but I can't vouch for the technical section


REPL panic instead of SYNTAX PROBLEM error

Enter an expression to evaluate, or a definition (like x = 1) to use later.
» "hello" : Str
panicked at crates/repl_ui/src/repl_state.rs:133:25:
not yet implemented: handle pattern other than identifier (which repl doesn't support).
Tip: this error can be triggered when trying to define a variable with a character that is not allowed, like starting with an uppercase character or using underdash (_).

Stack:

Error
    at imports.wbg.__wbg_new_abda76e883ba8a5f (https://www.roc-lang.org/repl/roc_repl_wasm.js:314:21)
    at console_error_panic_hook::hook::h6a88c36edf37c7ee (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[1180]:0x390cf8)
    at std::panicking::rust_panic_with_hook::h9aabd906218897c3 (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[2668]:0x410e9d)
    at std::panicking::begin_panic_handler::{{closure}}::h96d2bc381fa6ee1e (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[3094]:0x425233)
    at std::sys_common::backtrace::__rust_end_short_backtrace::ha76513a70bb070b0 (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[5226]:0x44fff4)
    at rust_begin_unwind (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[4443]:0x448ceb)
    at core::panicking::panic_fmt::h3aff855fe938c13f (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[4508]:0x449d3b)
    at roc_repl_ui::repl_state::ReplState::step::h55f73d441a8f1c10 (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[74]:0x1534da)
    at wasm_bindgen_futures::future_to_promise::{{closure}}::{{closure}}::hb7e81c410d97a7d8 (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[414]:0x2bb161)
    at wasm_bindgen_futures::queue::Queue::new::{{closure}}::h0ac4b2913c4ef62d (https://www.roc-lang.org/repl/roc_repl_wasm_bg.wasm:wasm-function[1788]:0x3d7661)

This was on https://www.roc-lang.org/#try-roc but I get a similar error in the CLI REPL
My expectation was that what the REPL prints was valid code, so I thought a type signature/annotation would be allowed. Is that not the case?

Type signatures are allowed, but not with a literal like "hello" or 123, this is the case for all Roc code, not just in the repl.
You can do:
hello : Str
hello = "hey"

We should provide a nice error message here though. When using "hello" : Str in a roc file you get a SYNTAX PROBLEM error. We should investigate why the repl does not throw this error.

Ah right, I see, type annotations cannot be used as expressions like they can in Haskell where something like ("hello" :: String) is a valid expression. I thought Elm was the same as Haskell, but it looks like it's exactly like Roc: the type annotation is not an expression but the REPL makes it look like it could be :sweat_smile:

Do you have any suggestions how we could fix/improve the expectations around this?

I guess if it didn't panic and gave an error message I would understand that type annotations aren't expressions in Roc (and I don't need them to be).
For reference here is what Elm gives me:
---- Elm 0.19.1 ----------------------------------------------------------------
Say :help for help and :exit to exit! More at <https://elm-lang.org/0.19.1/repl>
--------------------------------------------------------------------------------
> "Hello"
"Hello" : String
> "Hello" : String
|
-- UNEXPECTED SYMBOL ------------------------------------------------------ REPL

I was not expecting to run into the "has type" symbol here:

3|   "Hello" : String
             ^
Maybe you want :: instead? To put something on the front of a list?

Note: The single colon is reserved for type annotations and record types, but I
think I am parsing the definition of `repl_input_value_` right now.

Note: I may be getting confused by your indentation. Is this supposed to be part
of a type annotation AFTER the `repl_input_value_` definition? If so, the
problem may be a bit before the "has type" symbol. I need all definitions to be
exactly aligned (with exactly the same indentation) so the problem may be that
this new definition is indented a bit too much.


Title screen camera movement?

Describe the feature you'd like to see added
I've seen on another version of re4 you can move the camera at the title screen left/right slow or fast & also you can zoom in & out. I can't remember which port had these features though.
Would it be possible to add these in? I use a controller always but i think it was a keyboard feature. It would be nice to do these simple things with a controller. Best wishes!

This was already requested in issue #290, that feature actually isn't in this version of the game at all.

Thanks for reply, it's not possible through tweals at all? That's a shame.

It might be possible to replicate it, but I haven't looked at it yet. Not sure if emoose did either.

It might be possible to replicate it, but I haven't looked at it yet. Not sure if emoose did either.

Best of luck to you all


Track 2 SDK, ParseResourceType func doesn't support management group scoped provider

Bug Report

import path of package in question: "github.com/Azure/azure-sdk-for-go/sdk/azcore/arm"
https://github.com/Azure/azure-sdk-for-go/blob/28771743d1a87b2362aadce96177fce406b502fd/sdk/azcore/arm/internal/resource/resource_type.go#L85

SDK version: latest

output of go version: go version go1.19.3 darwin/arm64

What happened?

Input: /providers/Microsoft.Management/managementGroups/00000000-0000-0000-0000-000000000000/providers/Microsoft.Network
Output: error invalid resource ID: /providers/Microsoft.Management/managementGroups/00000000-0000-0000-0000-000000000000/providers/Microsoft.Network

What did you expect or want to happen?

This should be a valid ID for type Microsoft.Resources/providers
refs: https://github.com/Azure/azure-rest-api-specs/blob/e680f9bf4f154ec427ba7feac432ed8efd9665d0/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/resources.json#L2053

How can we reproduce it?

import (
	"github.com/Azure/azure-sdk-for-go/sdk/azcore/arm"
)

id := "/providers/Microsoft.Management/managementGroups/00000000-0000-0000-0000-000000000000/providers/Microsoft.Network"
resourceType, err := arm.ParseResourceType(id)

Anything we should know about your environment.

@raych1, could you help to take a look?

@ArcturusZhang , from the referenced spec - https://github.com/Azure/azure-rest-api-specs/blob/e680f9bf4f154ec427ba7feac432ed8efd9665d0/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/resources.json#L2053, /providers/Microsoft.Management/managementGroups/00000000-0000-0000-0000-000000000000/providers/Microsoft.Network/register is a valid resource id but failed in parsing. Can you confirm?

Same error for /subscription/00000000-0000-0000-0000-000000000000/Microsoft.Network, from https://github.com/Azure/azure-rest-api-specs/blob/e680f9bf4f154ec427ba7feac432ed8efd9665d0/specification/resources/resource-manager/Microsoft.Resources/stable/2022-09-01/resources.json#L2262, shoud be a valid resource id but failed in parsing.

Feel free to re-open this issue if needed.

Hi @raych1 , can we reopen this one? Incorporating support for parsing the Resource Provider is crucial as it's extensively utilized within Azure. The Provider REST API offers a range of endpoints that the Azure Portal, Azure CLI, and Azure PowerShell frequently rely on:
https://management.azure.com/providers/{resourceProviderNamespace}
https://management.azure.com/subscriptions/{subscriptionId}/providers/{resourceProviderNamespace}
https://management.azure.com/providers/Microsoft.Management/managementGroups/{groupId}/providers/{resourceProviderNamespace}

Enabling this functionality is pivotal as it will remove obstacles when managing Providers within Golang-based projects.


Can we have IN filter?

Hi! Great job for InfluxDB.
And I wonder If it can support IN filter in where clause, so far it seems no supported:
SELECT * FROM mydb WHERE service IN ('aa', 'bb');

{
    "error": "error parsing query: found IN, expected SELECT, DELETE, SHOW, CREATE, DROP, GRANT, REVOKE, ALTER at line 1, char 321"
}

version: bdd469db26c483a0acdcadbb937829a3913d9be5

We don't have support for IN right now but you can use OR, for the query you provided:
SELECT * from mydb WHERE service = 'aa' OR servce = 'bb'

I too would like this, and it should be easy with some query re-writing of the AST. Let me see if I can get to it.

A potentially easier workaround than multiple OR statements would be to use a regex. I think it should be faster, too. E.g.
SELECT * from mydb WHERE service =~ /aa|bb/

+1

Hi Guys,
Is this feature request still valid? If it is valid, can I work on this task?
Thanks

Yes, we are still interested in this feature.

Thanks, I will start working on this during weekend.

We have another usecase, the support for IN  would make it possible to integrate Grafana's multiselect feature with influx
@ch33hau do you have any update?

Hi @kalidasya sorry I was stop it half way, will working on it sooner.
However, I found that IN already exists in influxdb for other purpose. Due to the way we code, might not able to use IN for the purpose of this issue.
Is it possible to rename it to other syntax, eg, WITHIN?

@ch33hau good to hear, WITHIN to me more time related but I also cannot really come up with anything better

@ch33hau any updates on this?

This would be great guys! It's been a few years but ???

Please implement this faster, this feature is very very mich needed

Is anybody working on this? I was thinking of taking a crack at this tomorrow

There is nobody currently working on this at the moment. Feel free to give it a try.

+1

I would love this feature

+1

Looking forward to this. I would be fine using the WHERE .. OR or regex alternatives, but even won't work when the list of values passed to IN is very large (20k plus). Right now I don't see such a way to process this data without causing a "url too large" error.

So this isn't likely to happen that quickly, but I thought I should point out that we support using POST for queries and that you can put the query itself in the body. I think you can use application/x-www-form-urlencoded or you can use multipart/form-data. If you use the first, just urlencode the query. If you use the second, you can upload the query to a file with the query parameter q and then just insert the query into that.
Here's an example of doing it in Go with a GET/POST and application/x-www-form-urlencoded:
params := url.Values{}
params.Set("q", "your super long query")
body = strings.NewReader(params.Encode())

req, _ := http.NewRequest("GET", "http://localhost:8086/query", body)
req.Header.Set("Content-Type", "application/x-www-form-urlencoded")
http.DefaultClient.Do(req)

Here's a (more complicated) example where you upload a file using POST. I'm honestly not sure if, practically, they are any different from each other.
var q io.Reader // this got defined at some point and is a really large query, maybe an *os.File?
body := bytes.NewBuffer(nil)
writer := multipart.NewWriter(body)

// Retrieve the filename if we are reading from a file.
// The server doesn't actually use this information, but it's nice to include anyway.
filename := "<stdin>"
if f, ok := q.(*os.File); ok {
	filename = filepath.Base(f.Name())
}

// Create the form file and copy the contents of the io.Reader into it.
f, err := writer.CreateFormFile("q", filename)
if err != nil {
	return nil, err
}
io.Copy(f, q)
writer.Close()

req, _ := http.NewRequest("POST", "http://localhost:8086/query", body)
req.Header.Set("Content-Type", writer.FormDataContentType())
http.DefaultClient.Do(req)

I hope that helps.
Note that you can mix query parameters and the body parameters so you can put things like db=testdb in the query while putting the query itself in the body.

@kalidasya "We have another usecase, the support for IN would make it possible to integrate Grafana's multiselect feature with influx" here for this reason! i'm trying to use "all" option with a variable in Grafana but doesn't work, only if you select all them but one by one

Is this a feature that is still being developed?

I just come across this issue as well while I was trying to show data aggregated by host (so same use case as the second comment before mine above). I believe many will benefit from this capability.
I am surprised it's not already implemented..

Oh, good to see that influx is using this stalebot cancer.
Christ.

Is there something that we can do to have a IN clause or a workaround with regexes ?

I think it would be great if someone could at least settle the discussion on the performance impact of using regexp vs OR. above @beckettsean "thinks" regexp should be faster, while here @cxreg has found support for his assumption that regexp "probably doesn't make use of the index", his link however being dead.
So what's going on behind the scenes? Is the set of tags matching the regexp first being looked up and then used as a condition (should be fast for fairly sized sets), or is the tags of each entry matched against the regexp (potentially slooow) - or something completely different?

+1 - using IN would be great in transitioning from Postgres to Influx

@mjiderhamn this would be great but this issue beeing closed, I don't think there will be an update on this topic anytime soon.
I guess most of the effort of influxdata is devoted to influxdb 2.0. Unfortunately for us, There is no reason for them to change the syntax of a previous influxdb version.

Hi..I was trying to implement something similar using both reg ex and 'OR' .
Where these approaches fail :
Reg ex : if the value I have passed is contained in the column value, it reads that though it shouldn't. Reg ex is similar to 'contains'..I want to compare the exact same value.
OR : I want data for both values not either of them.
Please suggest another approach for IN clause.

I think the all point of having an IN() clause is not write
select * from myDB where service in ('aa', 'bb');
which can be achieved (painfully) with OR
but to use sub query such as
select * from myDB where service in (select distinct(k8S_container_name) from K8S_E4));
This is real purpose of in() to deal with temporary on the fly intermediary collections, ain't it ?

Reg ex is similar to 'contains'..I want to compare the exact same value.

If RegEx specifies an exact term, it is equivalent to checking for an exact value  – as opposed to "contains" or "is similar to", which one would usually associate with RegEx.

Please look at this WHERE service = 'aa' OR service = 'bb' vs. this WHERE service =~ /aa|bb/
The latter is a RegEx that specifies two exact values, aa and bb, with the | meaning "alternative", or in other words OR. … This could also be WHERE service =~ /aa/ OR service =~ /bb/`
However, the beauty in the | RegEx notation is to specify the multiple exact values in one (potentially long) =~ term, but without all the mucking around with OR.

… Still an IN(…) clause would still be awesome – +1 from me for that.


Remove proxylib limitation in observability documentation

This removes the proxylib limitation from the observability documentation that seems to now be fixed according to the last few messages in #14072

/test

This should also be in 1.14 docs, right? Will mark for backport.

@lambdanis @ldelossa Should be good now.
Do I need to make the backport or is that going to be picked up automatically or by someone else?

@darkrift Thanks! The backport will be done by someone as a part of regular process.


⚠ RPC server - xDai has degraded performance

In 296a11d, RPC server - xDai ($XDAI_RPC_URL) experienced degraded performance:

HTTP code: 200
Response time: 373 ms

Resolved: RPC server - xDai performance has improved in fc32c68 after 44 minutes.


Allow for special characters in file names / paths

This escapes the file path provided so that special characters do now break the command. For example: test_file_with(parentheses).png or test_file_with space.png

opened by mistake, will open a better one shortly.


sensor_type as a table

sensor_type is an enum, and transfo_type is a table. They are however kind of the same species. They both provide a way to translate a type id integer into its type information : name(func_name), description, schema(func_signature) for a jsonb attribute(parameters for transofs, specifications for sensors).
They could be both implemented similarly as a table. That would enable code reuse and enable extensibility for sensor types, and the ability to specify the schema of its jsonb attribute.

Talking about that, who is responsible for populating transfo_type? I am currently trying to insert transfo's through the API but I run into problems because there's no transfo type.

My guess is that the responsible is the one that implements the transform_type or that creates the postgresql extension that adds the different transform implementations. This guy will also have to create the sql function that applies transfos and transfo stacks.

Closing this for now. But please reopen if you care enough about this :)


go2tv-lite: 1.16.1 -> 1.17.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/alexballas/go2tv/releases.
meta.description for go2tv-lite is: Cast media files to UPnP/DLNA Media Renderers and Smart TVs
meta.homepage for go2tv-lite is: https://github.com/alexballas/go2tv
meta.changelog for go2tv-lite is: https://github.com/alexballas/go2tv/releases/tag/v1.17.0
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.17.0 with grep in /nix/store/crhwvsw66v17if55p992yb7w138w6dp0-go2tv-lite-1.17.0
found 1.17.0 in filename of file in /nix/store/crhwvsw66v17if55p992yb7w138w6dp0-go2tv-lite-1.17.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

go2tv
go2tv-lite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/crhwvsw66v17if55p992yb7w138w6dp0-go2tv-lite-1.17.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A go2tv-lite https://github.com/r-ryantm/nixpkgs/archive/970fc31152cbf2254335de4a67a10b5f91f7ca67.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/970fc31152cbf2254335de4a67a10b5f91f7ca67#go2tv-lite

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/crhwvsw66v17if55p992yb7w138w6dp0-go2tv-lite-1.17.0
ls -la /nix/store/crhwvsw66v17if55p992yb7w138w6dp0-go2tv-lite-1.17.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

go2tv
go2tv-lite

Maintainer pings
cc @gdamjan for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

actually this is a duplicate of (and conflicts with) https://github.com/NixOS/nixpkgs/pull/339810
so maybe this should be closed.
Should I do that?


Exclude main.py from test coverage measurement

It can't be measured outside of an editable install, and doesn't contain any logic.

Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

[ ] Documentation has been updated to reflect change.
[x] New code has tests, and affected old tests have been updated.
[x] All tests and CI checks pass.
[x] Ensured the pull request title is descriptive.
[ ] Conda lock files have been updated if dependencies have changed.
[ ] Attributed any Generative AI, such as GitHub Copilot, used in this PR.
[x] Marked the PR as ready to review.

As is very minor testing tweak that doesn't actually touch any production code, I'm going to merge without review.


DPL-nnn Cardinal - pooling of pools (from different plates)

Currently think no development will be necessary for this one
User story

We need to be able to pool various pools for sequencing (as is possible through the poolings new page).

Who are the primary contacts for this story
Stef L
Acceptance criteria
To be considered successful the solution must allow:

[ ] add a list of acceptance criteria here
[ ] ...

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Closing - this is being achieved through the poolings/new page in SS


Pass CMAKE_CXX_FLAGS through to the add_custom_command invocation of the compiler in usFunctionEmbedResources.cmake

Updating usFunctionEmbedResources.cmake to use CMAKE_CXX_FLAGS rather than building the flags itself. This should allow changes to compilation flags in other cmake files to affect the compiled empty resource file for Darwin-based platforms.

Hi,
The loop is necessary due to an odd behavior with add_custom_command and
passing in CMAKE_CXX_FLAGS using the -D option to cmake.  The loop ensures
that the quotation marks around the arguments are removed so they can be
properly parsed.
-J
On Tue, Aug 8, 2017, 11:22 Abhinay Reddyreddy<EMAIL_ADDRESS>wrote:

@karthikreddy09 requested changes on this pull request.
In cmake/usFunctionEmbedResources.cmake
https://github.com/CppMicroServices/CppMicroServices/pull/226#discussion_r131941892
:

 endif()

 if(CMAKE_OSX_ARCHITECTURES)

   list(APPEND US_OSX_CXX_FLAGS -arch ${CMAKE_OSX_ARCHITECTURES})

 endif()

 if(CMAKE_OSX_DEPLOYMENT_TARGET)

   list(APPEND US_OSX_CXX_FLAGS -mmacosx-version-min=${CMAKE_OSX_DEPLOYMENT_TARGET})

 endif()

   # This resolves an issue where passing in CMAKE_CXX_FLAGS using

  # -D CMAKE_CXX_FLAGS:STRING="..." results in a quoted string being passed to

  # the command below which results in the command being:

  #  >> clang "..." -c ... -o stub.o

  # when it should really be:

  #  >> clang ... -c ... -o stub.o

  # (pardon the elipsis for abbreviation)

  separate_arguments(TEMP_CXX_FLAGS UNIX_COMMAND ${CMAKE_CXX_FLAGS})

  foreach(flag ${TEMP_CXX_FLAGS})

Any reason why this for loop is needed? passing the output of
separate_arguments to add_custom_command works fine for me.
add_custom_command(
OUTPUT ${_source_output}
COMMAND ${CMAKE_CXX_COMPILER} ${TEMP_CXX_FLAGS} -c ${US_CMAKE_RESOURCE_DEPENDENCIES_CPP} -o stub.o
COMMAND ${CMAKE_LINKER} -r -sectcreate __TEXT us_resources ${_zip_archive_name} stub.o -o ${_source_output}
DEPENDS ${_zip_archive}
WORKING_DIRECTORY ${_zip_archive_path}
COMMENT "Linking resources zip file for ${US_RESOURCE_TARGET}"
VERBATIM
)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CppMicroServices/CppMicroServices/pull/226#pullrequestreview-54970403,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF8dmIMdI0WWe6WaPy4K0tTyiB0Wl06Bks5sWH0ogaJpZM4OwrIO
.

Hi, I updated and removed the for loop as suggested by Abhinay.  It appears to have the same effect and cmake appears to expand the list automatically in the add_custom_command call.

Codecov Report

Merging #226 into development will increase coverage by 0.76%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #226      +/-   ##
===============================================
+ Coverage        82.72%   83.49%   +0.76%
===============================================
  Files              186      184       -2
  Lines            13138     9358    -3780
===============================================
- Hits             10869     7813    -3056
+ Misses            2269     1545     -724

Impacted Files
Coverage Δ

framework/test/driver/ServiceControlInterface.h
0% <0%> (-100%)
:arrow_down:

framework/src/util/FrameworkPrivate.h
0% <0%> (-100%)
:arrow_down:

...mework/include/cppmicroservices/GetBundleContext.h
0% <0%> (-100%)
:arrow_down:

framework/test/driver/BundlePropsInterface.h
0% <0%> (-100%)
:arrow_down:

...amework/include/cppmicroservices/BundleActivator.h
0% <0%> (-100%)
:arrow_down:

framework/src/bundle/BundleStorage.h
0% <0%> (-100%)
:arrow_down:

framework/src/bundle/BundleContextPrivate.h
0% <0%> (-100%)
:arrow_down:

framework/test/driver/TestManager.h
0% <0%> (-100%)
:arrow_down:

...ramework/include/cppmicroservices/ServiceFactory.h
0% <0%> (-100%)
:arrow_down:

framework/src/util/Utils.h
0% <0%> (-85.72%)
:arrow_down:

... and 144 more

Thanks!

Codecov Report

Merging #226 into development will increase coverage by 0.54%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #226      +/-   ##
===============================================
+ Coverage        82.72%   83.27%   +0.54%
===============================================
  Files              186      184       -2
  Lines            13138     9355    -3783
===============================================
- Hits             10869     7790    -3079
+ Misses            2269     1565     -704

Impacted Files
Coverage Δ

...amework/include/cppmicroservices/BundleActivator.h
0% <0%> (-100%)
:arrow_down:

framework/test/driver/BundlePropsInterface.h
0% <0%> (-100%)
:arrow_down:

...ramework/include/cppmicroservices/ServiceFactory.h
0% <0%> (-100%)
:arrow_down:

framework/src/bundle/BundleContextPrivate.h
0% <0%> (-100%)
:arrow_down:

...mework/include/cppmicroservices/GetBundleContext.h
0% <0%> (-100%)
:arrow_down:

framework/test/driver/ServiceControlInterface.h
0% <0%> (-100%)
:arrow_down:

framework/test/driver/TestManager.h
0% <0%> (-100%)
:arrow_down:

framework/src/util/FrameworkPrivate.h
0% <0%> (-100%)
:arrow_down:

framework/src/bundle/BundleStorage.h
0% <0%> (-100%)
:arrow_down:

framework/src/util/Utils.h
0% <0%> (-85.72%)
:arrow_down:

... and 144 more


Add support for airline warning and error colors

I noticed there weren't colors defined for the errors and warnings of airline, so I've added those to the airline nord theme.

Without warninng color:

With warning color:

No worries, I know how too busy can feel overwhelming sometimes. I'll take a look at it later today or tomorrow.

Totally agreed with your comments. It's better with more contrast.
I have added a commit which implements your suggestions.

I've rebased your branch on top of the develop branch and resolved the merge conflict and will merge it to deploy it with the next release version :rocket:
Thanks again for the PR :+1:


Switch back to dev

Update version number after release 1.7.3
cc @zhang-alvin

What's the reasoning behind using the 'dev' suffix?

@zhang-alvin it is to indicate that it is a development version, and not an officially tagged/release version (which means that it is theoretically less stable)

We have a problem with the tests on conda here since some packages were likely updated in conda-forge. All tests pass on hashdist.
I am not necessarily familiar with the specific tests that are failing, but the difference between "actual" and "expected" seems a little bit too big. For example, one of the tests in proteus/tests/CLSVOF/disc_ICs/test_CLSVOF_discICs.py needs to go from a comparison of 10 decimal points (probably overkill) to 2 decimal points (probably too little) in order to pass. Some other tests have very different values from the expected ones (for example in proteus/tests/post_processing). This seems like it could be too much to be attributed to slight numerical changes due to updated dependencies.
As @zhang-alvin mentioned, this might be due to the coarsening of some tests that now use a fixed mesh, probably from https://github.com/erdc/proteus/pull/1046
\cc @cekees @zhang-alvin @davidbrochart

Merging as hashdist tests are passing and #1107 will fix conda tests issues


Adds more lobby backgrounds + round end audios

Adds more lobby background from the discord and round end audios from funny videos

Lets remove the round end audios, the videos are funny but lets keep it to music for the lobby audios.

the round end audios are the stingers that play when the round restarts, not the music that plays in the lobby

point still stands, no moar audios!!!1!

unless they are public domain music


page.php while loop cleanup

A while ago there were a couple code cleanups concerning indentation and other inconsistencies.
#1178 #1173
Despite everyone's best efforts, I believe the page.php while loop still lacks a bit cleanup, for consistency with other files.

@dgobnto , just a note..
A while ago, Automattic  _s theme went thru a major cleanup: https://github.com/Automattic/_s/commit/b26b6ebfa4a92d82a69576e69aefe44192350635

@esarantanen Then apparently it's all the other files that need cleanup! ;)

Feel free to submit a PR.

I would, but now I'm confused! 😆
Which markup should we follow?
I did a very quick research and Wordpress PHP Coding Standards refers to this kind of markup/syntax:
<?php if ( have_posts() ) : ?>
    <?php while ( have_posts() ) : the_post(); ?>
        <!-- ... -->
    <?php endwhile; ?>
<?php endif; ?>

While Automattic's _s theme recent cleanup went towards:
<?php if ( have_posts() ) : ?>
	<?php
	/* Start the Loop */
	while ( have_posts() ) :
		the_post();
		get_template_part( 'template-parts/content', get_post_format() );
	endwhile;
endif;
?>

I prefer the former, though…

I agree, the former is much easier to read.

+1


Add Russian language translate

Coverage remained the same at 93.056% when pulling 9b59c14fa20c1c42c4486d7c19595f789ed38463 on webartisans-org:master into 38a5640ca8918131a60827363b9c13b9d8d2e24b on arteria:master.


Does anyone need twitter emoji parser?

For now, if you set enableEmoji = true in your site’s configuration, hugo will parse the emoji like this:

l find a library: https://github.com/twitter/twemoji
If you use twemoji, your site's emoji could convert unicode to image, like this:

If anyone like this feature, maybe I can consider adding an option to use it.

I'd like to render emoji on client side only if there are no native support. Personally, I like Apple style more. However, it could be better to have the same style on any platform, because in different styles the same emoji could represent lightly different meaning.


Daily question

🐛 Bug Report
The following question requires an incorrect answer:
What item is used to fill a Druid Pouch?
I consulted with others and found the answer to be "Mort Myre Fungi", which isn't even a real item and should thus be removed.
However, there are multiple answers to the question which I shall list below.
Mort Myre Stem
More Myre Pear
More Myre Fungus
Therefore the daily question should also be changed to the following:
Name one of the items that can be used to fill a Druid Pouch.

Good catch, updated the trivia doc renaming the question and added the answers you listed. Should be added whenever magna merges it

resolved


Set up pyproject.toml

This PR sets up pyproject.toml for installing the code.
The code can be installed with pip install -e . For the moment this just installed the dependencies.
I was tentative to add MORS/JANUS to the dependencies, because it is not clear to me yet how they can be integrated. We can pick that up at a later stage.
I also set up the boilerplate for a Python package (src/proteus) where we can move shared code too.
Closes #106

@timlichtenberg

@nichollsh This PR is ready, could you have a look at it?

@nichollsh Ah, thanks for pointing that out. I overlooked those. I cannot find ipykernel being used anywhere, but I will add the others. I also updated the documentation.


Add toggle setting to keep pinned tab when enabling 'Scrolling To Switch Tabs'

I would like to ask for an option related to the following setting. Enabling the new setting option should prevent pinned tabs from scrolling along with other unpinned tabs.
It is inconvenient and frustrating to have to scroll back to find the pinned tabs, especially when I need to switch frequently between two main tabs.
Related setting:

The current behavior is that pinned tabs will move along with other tabs when scrolling.

Thank you @benibenj, I gave it a try and it can be helpful on occasion.
But, it would be great if VSCode could also have the option to display pinned tabs on the same row, since I don't always have a large monitor available, using workbench.editor.pinnedTabsOnSeparateRow is good but also takes me so much space.


Pulls/2.4/alternate acs

Assertion Consumer Service is not an "only one" predetermined arrangement. It is only one at a time, of course, but e.g. subdomains are allowed for with SAML by defining alternative ACS urls at the identity provider, and provided with the authentication request from the service provider.
The Silverstripe SAML module however did not support providing alternative ACS urls as part of the request, and made assumptions based no this when trying to authenticate an authentication response from the IdP - always checking against the SP entity ID, defined as the base URL including protocol per the documentation.
The ability to define alternative ACS urls is now allowed for with a predefined list (via Silverstripe config) - although identity providers should reject undefined URLs this allows us to catch misconfiguration earlier and provides a more defined behaviour to developers.
In tandem to allowing other ACS urls, we must also account for then when assessing a response - the use of the SP.EntityID has been changed to the absolute base URL as found by Silverstripe, where mismatches in "allowed domains" should be caught by nature of differing from the response (causing a rejection) or not being selected from the list that forms the request.
A hook is introduced to allow for further or reconfiguration of the ACS base URL on a project level if it is needed.
The diff appears larger than changes due to the renaming of a variable to remove confusion between the SAML configuration that is being built, and the Silverstripe Config API that is used to obtain the data for it.

Thanks, but why did you alter the commits?

Not sure what you mean by alter commits. I just merged pr

The merge has created new commits that list you as the creator (although I'm listed as the author).
It might be nice to use either a fast-forward or a normal merge then, instead of unnecessarily rebasing. Rebasing messes with the history and looks like you've touched up everyone's commits. GitHub is an increasingly important tool used in recruitment - it can give negative impressions if it looks like someone's work has had to be redressed.
My commits:

Merged commits (made by you):

This was likely caused by using the rebase merge option. I've disabled it for the repo now. @satrun77 please make sure to always use the merge commit option for merging PRs to retain the history and the original author's commit without changes. Thank you!

For future, it also would be good to add documentation for the two new features added in this commit:

How to use the extra_acs_base (you could also slim down the PHP comment and convert it into MD).
An example of using the onBeforeAcs extension hook and why it might be useful.

To be fair, the docs for the module are hardly complete, but there is a fair amount of info regarding the YML configuration: https://github.com/silverstripe/silverstripe-saml/blob/main/docs/en/developer.md#yaml-configuration

For future, it also would be good to add documentation for the two new features added in this commit

Added in #67


Update Meta “authors”

Automatically generated by Netlify CMS

👷 Deploy Preview for determined-meninsky-f29d45 processing.
🔨 Explore the source changes: 52b221f0f916b5fb8bba815e76a5dd8f74c6c09e
🔍 Inspect the deploy log: https://app.netlify.com/sites/determined-meninsky-f29d45/deploys/6139066d181cdc00076825a0


ATSAM: fall back to only setting nRESET

Running probe-rs erase --chip atsamd51p19a with a J-Link probe always ended up in an error. The problem is that reset_hardware tried to set tms and tck to known values before issuing the reset, but the probe does not have implementation for these, so the reset always failed. Otherwise, the erase was done correctly.
@jgust since you're the original author, do you see potential problems by not setting tms and tck? Should we do a "try-but-fall-back-to-just-nrst" instead?

@bugadani - I don't have means of testing right now but its probably fine to only toggle the reset pin when doing normal reset. The reason to explicitly set the tms and tck pins was to ensure that we don't accidentally trigger a reset extension instead.  If we don't set them, there is always a slim chance that their pin states will be just right to trigger a reset extension inadvertently. If that happens, the device will just be stuck in reset since there is nothing releasing it from the reset extension. This can be really confusing if you don't know what is going on :-)
When it comes to ATSAM reset extension and Jlink, I know that Jlink supports it natively, but I have no clue how it is triggered through the jlink API (if it is even possible). But if we could figure that out, I suspect that would be a better option than the fallback you suggest. The fallback to normal reset could be a last resort, but that would limit the number of use-cases where it is possible to trigger chip-erase. So at least an extra log saying that we are doing the fallback would be nice.

Unfortunately trying to find the segger propriatery way might not be feasible: the 2008 PDF they released does not contain a bistable way to control the TCK line, only everything else.

Fair enough. The reset extension stuff was added to JLink around 2018 according to this forum thread. But someone with actual experience of the jlink protocol (i.e. not me ;-) ) would have to make the call there if it is feasible or not. From your investigation, it sounds like its not.
But anyway, how about having a third reset function e.g. reset_hardware_fallback that only fiddles with nrst and we call that from both reset_hardware and reset_hardware_with_extension if the first try returns an SWJ_PINS error? That way we retain the current functionality (which is known to work according to the datasheet and i shown to work well for probes that support swj_pins) but then we fallback to the simpler and more "best effort"  version for probes like JLINK?

Okay, the fallback seems to be working, although now I'm getting warnings about unexpected core state. That's something different, though, so I'm happy to leave this PR as is.

Looking great to me :)


Integration request: https://github.com/Birch-san/k-diffusion/tree/mps/k_diffusion

Can you integrate this diffusion? It is super optimised!
BIRCH SAN - K DIFFUSION

Is there a quick way to integrate it myself? can't find the diffusion folders in the folder.... seems the structure is a bit different

Yeah, Stable Diffusion implementations that use K Diffusion are a totally other beast, sorry 🙂 My implementation is based on Hugging Face Diffusers and so has to use the pipelines and schedulers that Hugging Face provides ... or write my own. This GUI is only about a week or two old and so I am at the point that I'm re-considering the architecture. But probably not till next weekend since I don't get to do a lot of work on this till the weekend.
If I do switch to the other approach then K Diffusion should be simple enough to add in.

Nice to hear! The only gui alternative on Mac is currently diffusion bee but it is laughable.... no settings at all... BUT it is MPS optimised....
any tips how to get your ui running with other stable diffusion installations on my computer?

Nothing that would work out of the box I'm afraid. Because the server expects certain parameters and the app will receive responses back from the server. So the parameters have to be named correctly in both cases. And each developer will simply use whatever names they wish. So without taking a look at each one and doing some work it's hard to make it work immediately would be my gut instinct ... without having taken a look at each server in question that is 🙂
I saw your other post about MPS optimizations. There are no optimizations that need to be applied as such. All of the MPS speed ups come from Pytorch. So as long as you have the latest Pytorch nightly (and my code is set to use MPS) then it should all just work.


Enhance File Upload Script with AI-Powered MIME Detection, Compression, and Endpoint Optimization

This update introduces significant enhancements to the file upload script, integrating advanced AI-driven features to improve functionality and efficiency. The following modifications have been made:

AI-Powered MIME Type Detection:

Implemented a machine learning model to predict MIME types based on file content rather than relying solely on file extensions. This approach improves accuracy in determining file types, especially for files with non-standard or missing extensions.

Smart File Compression:

Added an AI-driven file compression feature that dynamically adjusts compression algorithms based on file content and size. This optimization ensures efficient storage and faster upload times while maintaining file integrity.

Predictive Upload Endpoint Selection:

Incorporated an AI model to predict the optimal upload endpoint based on historical data and current server load. This feature enhances upload speed and reliability by directing files to the most suitable endpoint.

Advanced Error Handling:

Enhanced error handling mechanisms using AI algorithms to analyze and predict potential upload issues. This proactive approach helps in identifying and resolving issues before they impact the upload process.

AI-Optimized Base64 Encoding:

Improved base64 encoding performance with AI-based optimizations that reduce processing time and resource consumption during the encoding process.

Dynamic Timeout Adjustments:

Introduced AI-driven adjustments for connection and read timeouts based on real-time network conditions and historical data. This feature helps in managing timeouts more effectively, reducing the likelihood of failed uploads.

These updates collectively enhance the script's efficiency, reliability, and overall performance by leveraging AI technologies to optimize various aspects of the file upload process.

This update introduces several significant enhancements to the file upload script, incorporating advanced AI features to improve its functionality and efficiency.
Key Updates:
AI-Powered MIME Type Detection:
Implemented a more accurate MIME type detection mechanism using machine learning models. This improves the script’s ability to correctly identify file types, enhancing compatibility and reliability in handling various file formats.
File Compression:
Added an AI-based file compression feature that dynamically adjusts compression settings based on the file content and size. This reduces file upload times and optimizes storage usage, making the process more efficient.
Endpoint Optimization:
Improved handling of signed endpoints by integrating AI-driven strategies for endpoint management. This includes enhanced URL generation and better handling of authentication headers, ensuring more secure and reliable file uploads.
Type Hinting and Code Quality:
Resolved type hinting issues, specifically for dictionary type arguments, and refactored the code for better readability and maintainability. This adheres to modern Python best practices and improves code clarity.
Impact:

Improved Accuracy: Enhanced MIME type detection ensures that files are processed correctly based on their actual content.
Increased Efficiency: AI-driven compression reduces file sizes and speeds up upload processes.
Enhanced Security: Optimized endpoint handling improves the security of file uploads.

These updates collectively improve the script’s performance, accuracy, and security, making it more robust and efficient for handling file uploads.


BQM.from_ising should ignore 0 biases in h when h is provided as a list

This would generalize the fix introduced in https://github.com/dwavesystems/dwave-system/pull/201

Upon closer inspection of the cloud-client's sample_ising function, it looks like 0 biases are treated as active except in the case of missing qubits. In that case I think it's better to maintain the current behaviour and update the fix in dwavesystems/dwave-system#201 to match the cloud client.


Human Pose Estimation using Deep Learning

I am a GSSoC22 participant, I was wondering if there was  a dataset available for this issue as I would like to work on this

@KareemElozeiri Look for any dataset. If not found, ping us.

I have already searched on Kaggle and UCI websites and I couldn't find a dataset, Could you please point me to other platforms if you know any?

@Rutikab12

@Rutikab12

check paper with code

Hello I am a GsSOC'24 contributor and I would like to work on this issue. Can you please assign it to me?

Hello @Niketkumardheeryan , please assign me this issue.


混合开发时不能用这个插件吗？

The plugin fluttertoast is built using an older version of the Android plugin API which assumes that it's running in a full-Flutter environment. It may have undefined behaviors when Flutter is integrated into an existing app as a module.
The plugin can be updated to the v2 Android Plugin APIs by following https://flutter.dev/go/android-plugin-migration.

上面是VSCode的控制台输出。
flutter版本1.12.13+bugfix.5

解决了吗

解决了吗

运行没问题，反正我直接用了[手动捂脸]

fixed in v5.0.1


Flame v1.2.0 migration

Major description of changes:
SpacescapeGame:

Dead code prepare() and onResize() methods due fact that overrides no longer exists in v1.2.0

Migrate JoystickComponent:

joystickChangeDirectional() removed
joystickAction() updated, now called by ButtonComponent
Player.update() adapt new position calculation.
Now joystick component is required for Player class.
ButtonComponent added instead actions in JoystickComponent.

KnowsGameSize:

No longer used in code, remains as is as reference for the YouTube tutorial with note about it in code.
Migrate for use gameRef.size instead.

Player:

onLoad method added.
Fill _playerData with defaultData to make sure a null exception is not thrown due update in game.dart (_playerScore.text).

Everything looks good, but there seems to be some issue with pausing the game. If I pause the game, exit to main menu and then again start the game, the game loads up in the paused state of previous session. Can you see if you can fix that?

Fixed, now when player starts a new game game engine will be in running state. Moreover I handled another issue with collision detection between Player and Enemy. Collision was detected twice because callback set enemy hp = 0, and destroy was invoked only in next update cycle. In this situation Player and Enemy hit themselves twice.