[libmariadb] Fix patch

Due to version update, fix outdated patche.
Related: #10024.
Note: this port doesn't contain any features.

/azp run

I added the commits by myself to my vcpkg folder (in my PC) and tried to rebuild still same error or adding it by myself isn't an option?

@Moustafa1337 Could you provide failure logs?

I checked this branch out and tested it; works for me.

Please, could someone tell me how to apply this libmariadb patch: x64-windows? I'm trying to install but the error.

@Moustafa1337 @ainz7892 please pull this branch.

Still
`Computing installation plan...
The following packages will be built and installed:
libmariadb[core]:x64-windows
Starting package 1/1: libmariadb:x64-windows
Building package libmariadb[core]:x64-windows...
-- Using cached E:/vcpkg/downloads/MariaDB-mariadb-connector-c-8e9c3116105d9a998a60991b7f4ba910d454d4b1.tar.gz
-- Extracting source E:/vcpkg/downloads/MariaDB-mariadb-connector-c-8e9c3116105d9a998a60991b7f4ba910d454d4b1.tar.gz
-- Applying patch md.patch
CMake Error at scripts/cmake/vcpkg_apply_patches.cmake:50 (message):
Applying patch failed.  Patch needs to be updated to work with source being
used by vcpkg!
Call Stack (most recent call first):
scripts/cmake/vcpkg_extract_source_archive_ex.cmake:134 (vcpkg_apply_patches)
scripts/cmake/vcpkg_from_github.cmake:139 (vcpkg_extract_source_archive_ex)
ports/libmariadb/portfile.cmake:8 (vcpkg_from_github)
scripts/ports.cmake:90 (include)
Error: Building package libmariadb:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: libmariadb:x64-windows
Vcpkg version: 2020.02.04-nohash
Additionally, attach any relevant sections from the log files above.`

@Moustafa1337 Could you provide your cmake version?

cmake version 3.16.4

@JackBoosY Applying this patch fixes the issue for me

/azp run

/azp run

@Moustafa1337 @NoFr1ends @Derimax3 After installed cmake 3.16.4, I still can't repro that.
Could you provide failure logs VCPKG_PATH/buildtrees/libmariadb/patch-x86-windows-*-err.log ?

@Moustafa1337 @NoFr1ends @Derimax3 After installed cmake 3.16.4, I still can't repro that.
Could you provide failure logs VCPKG_PATH/buildtrees/libmariadb/patch-x86-windows-*-err.log ?

Hello  JackBoosY this is log from patch-x86-windows-0-err
error: corrupt patch at line 13

@Moustafa1337 @NoFr1ends @Derimax3 After installed cmake 3.16.4, I still can't repro that.
Could you provide failure logs VCPKG_PATH/buildtrees/libmariadb/patch-x86-windows-*-err.log ?

Should I reinstall Cmake? Sorry i am totaly beginner in this. What version should i have? Because i have cmake-3.17.0 .

@Derimax3 I think it has nothing to do with cmake version.
Can you delete VCPKG_PATH/buildtrees/libmariadb/src/ and try again?
Please provide full log.

@Derimax3 I think it has nothing to do with cmake version.
Can you delete VCPKG_PATH/buildtrees/libmariadb/src/ and try again?
Please provide full log.

I deleted /buildtrees/libmariadb/src/ and still same error.

C:\Users\Adorius Black\vcpkg>vcpkg install boost-iostreams:x64-windows boost-asio:x64-windows boost-system:x64-windows boost-filesystem:x64-windows boost-variant:x64-windows boost-lockfree:x64-windows luajit:x64-windows libmariadb:x64-windows pugixml:x64-windows mpir:x64-windows cryptopp:x64-windows
Computing installation plan...
The following packages are already installed:
boost-asio[core]:x64-windows
boost-filesystem[core]:x64-windows
boost-iostreams[core]:x64-windows
boost-lockfree[core]:x64-windows
boost-system[core]:x64-windows
boost-variant[core]:x64-windows
cryptopp[core]:x64-windows
The following packages will be built and installed:
libmariadb[core]:x64-windows
luajit[core]:x64-windows
mpir[core]:x64-windows
pugixml[core]:x64-windows
Package boost-filesystem:x64-windows is already installed
Package boost-asio:x64-windows is already installed
Package boost-iostreams:x64-windows is already installed
Package boost-lockfree:x64-windows is already installed
Package boost-system:x64-windows is already installed
Package boost-variant:x64-windows is already installed
Package cryptopp:x64-windows is already installed
Starting package 1/4: libmariadb:x64-windows
Building package libmariadb[core]:x64-windows...
-- Using cached C:/Users/Adorius Black/vcpkg/downloads/MariaDB-mariadb-connector-c-8e9c3116105d9a998a60991b7f4ba910d454d4b1.tar.gz
-- Extracting source C:/Users/Adorius Black/vcpkg/downloads/MariaDB-mariadb-connector-c-8e9c3116105d9a998a60991b7f4ba910d454d4b1.tar.gz
-- Applying patch md.patch
CMake Error at scripts/cmake/vcpkg_apply_patches.cmake:50 (message):
Applying patch failed.  Patch needs to be updated to work with source being
used by vcpkg!
Call Stack (most recent call first):
scripts/cmake/vcpkg_extract_source_archive_ex.cmake:134 (vcpkg_apply_patches)
scripts/cmake/vcpkg_from_github.cmake:139 (vcpkg_extract_source_archive_ex)
ports/libmariadb/portfile.cmake:8 (vcpkg_from_github)
scripts/ports.cmake:90 (include)
Error: Building package libmariadb:x64-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: libmariadb:x64-windows
Vcpkg version: 2020.02.04-nohash
Additionally, attach any relevant sections from the log files above.

@ras0219-msft Can you take a look?
I still can't repro it.

@JackBoosY You maybe misunderstood me but your PR is working for me.

@JackBoosY You maybe misunderstood me but your PR is working for me.

Hi. Can you tell me what you changed ? :)

@JackBoosY You maybe misunderstood me but your PR is working for me.

@NoFr1ends Hi. Can you tell me what you changed please? :)

I have had the same issue when trying to install libmariadb (error while trying to apply the md.patch file) and this PR fixes it on Mac, Linux and Windows.
Can we merge this into master?

@Derimax3 Are you sure that you have checked out this PR instead of the master, or at least have applied this PR as a patch to your vcpkg folder?

I have had the same issue when trying to install libmariadb (error while trying to apply the md.patch file) and this PR fixes it on Mac, Linux and Windows.
Can we merge this into master?

Hi how you fixed it ? Can you tell me please?

I have had the same issue when trying to install libmariadb (error while trying to apply the md.patch file) and this PR fixes it on Mac, Linux and Windows.
Can we merge this into master?

Hi how you fixed it ? Can you tell me please?

You should learn how to use git and GitHub first.

I have had the same issue when trying to install libmariadb (error while trying to apply the md.patch file) and this PR fixes it on Mac, Linux and Windows.
Can we merge this into master?

Hi how you fixed it ? Can you tell me please?

You should learn how to use git and GitHub first.
Download: https://patch-diff.githubusercontent.com/raw/microsoft/vcpkg/pull/10043.patch
And then you can do for example git apply 10043.patch

One more question. How to download it? Just save it in vcpkg as text with name 10043.patch? or should i donwload it in git?

I have had the same issue when trying to install libmariadb (error while trying to apply the md.patch file) and this PR fixes it on Mac, Linux and Windows.
Can we merge this into master?

Hi how you fixed it ? Can you tell me please?

You should learn how to use git and GitHub first.
Download: https://patch-diff.githubusercontent.com/raw/microsoft/vcpkg/pull/10043.patch
And then you can do for example git apply 10043.patch

One more question. How to download it? Just save it in vcpkg as text with name 10043.patch? or should i donwload it in git?

Just download it so you have a file called 10043.patch (or anything else, but then you have to change the git apply command)

I have had the same issue when trying to install libmariadb (error while trying to apply the md.patch file) and this PR fixes it on Mac, Linux and Windows.
Can we merge this into master?

Hi how you fixed it ? Can you tell me please?

You should learn how to use git and GitHub first.
Download: https://patch-diff.githubusercontent.com/raw/microsoft/vcpkg/pull/10043.patch
And then you can do for example git apply 10043.patch

One more question. How to download it? Just save it in vcpkg as text with name 10043.patch? or should i donwload it in git?

Just download it so you have a file called 10043.patch (or anything else, but then you have to change the git apply command)

I downloaded it and i did i use this command: git apply 10043.patch
error: patch failed: ports/libmariadb/CONTROL:1
error: ports/libmariadb/CONTROL: patch does not apply
error: patch failed: ports/libmariadb/md.patch:1
error: ports/libmariadb/md.patch: patch does not apply
:-/

On which branch are you and are you on the newest version?

On which branch are you and are you on the newest version?

https://github.com/otland/forgottenserver/wiki/Compiling-on-Windows-(vcpkg)
I have cmake-3.17.0

On which branch are you and are you on the newest version?

https://github.com/otland/forgottenserver/wiki/Compiling-on-Windows-(vcpkg)
I have cmake-3.17.0

That wasn't my question..
Can you please do git pull then git apply 10043.patch again?

On which branch are you and are you on the newest version?

https://github.com/otland/forgottenserver/wiki/Compiling-on-Windows-(vcpkg)
I have cmake-3.17.0

That wasn't my question..
Can you please do git pull then git apply 10043.patch again?

C:\Users\Adorius Black\vcpkg>git pull
Already up to date.
C:\Users\Adorius Black\vcpkg>git apply 10043.patch
error: patch failed: ports/libmariadb/CONTROL:1
error: ports/libmariadb/CONTROL: patch does not apply
error: patch failed: ports/libmariadb/md.patch:1
error: ports/libmariadb/md.patch: patch does not apply
Where should i donwload 10043.patch? Desktop,Vcpkg or it doesnt matter?

I have had the same issue when trying to install libmariadb (error while trying to apply the md.patch file) and this PR fixes it on Mac, Linux and Windows.
Can we merge this into master?

Hi how you fixed it ? Can you tell me please?

In this case you don't even have to bother with applying a PR to your code, simply just download the whole contents of the md.patch file from the PR and copy it to ports/libmariadb and you are done. And make of course sure that you have a clean install of Vcpkg.
Because this was basically just what I did when I tested this PR, because after I initially ran into this issue I had fixed it on my own and only after I was done I had the idea to actually look if other people ran into this issue as well and if someone already submitted a PR, which was of course the case. ;-)

Please follow the steps below:

git clone https://github.com/JackBoosY/vcpkg.git
git checkout -b dev/jack/10024 origin/dev/jack/10024
./bootstrap-vcpkg.bat
./vcpkg install libmariadb

Please follow the steps below:

git clone https://github.com/JackBoosY/vcpkg.git

cd vcpkg

git checkout -b dev/jack/10024 origin/dev/jack/10024
./bootstrap-vcpkg.bat
./vcpkg install libmariadb

Building package libmariadb[core]:x86-windows... done
Installing package libmariadb[core]:x86-windows...
Installing package libmariadb[core]:x86-windows... done
Elapsed time for package libmariadb:x86-windows: 2.286 min

Total elapsed time: 2.286 min

There is another way to just switch to a branch that contains exactly the changes of this PR:
git clone https://github.com/Microsoft/vcpkg.git
cd vcpkg
git fetch origin pull/10043/head:pr-fix-libmariadb
git checkout pr-fix-libmariadb
./bootstrap-vcpkg.sh
./vcpkg install libmariadb

This creates a new branch pr-fix-libmariadb from this PR.

Por favor, siga os passos abaixo:

git clone https://github.com/JackBoosY/vcpkg.git
git checkout -b dev/jack/10024 origin/dev/jack/10024
./bootstrap-vcpkg.bat
./vcpkg install libmariadb

Thank you very much JackBoosY, my error has been corrected and now I can compile.

There is another way to just switch to a branch that contains exactly the changes of this PR:
git clone https://github.com/Microsoft/vcpkg.git
cd vcpkg
git fetch origin pull/10043/head:pr-fix-libmariadb
git checkout pr-fix-libmariadb
./bootstrap-vcpkg.sh
./vcpkg install libmariadb

This creates a new branch pr-fix-libmariadb from this PR.

Thanks this fixed it for me, other solutions didn't work.

This looks good! (although I wonder how it ever worked...) Thanks! 😄

@strega-nil I just judate the patch. Looking at the changes in detail, you will find that the source code content related to the patch has been changed.

Shouldn't a test have caught the build failure before it was merged into master?

@OlafvdSpek Well, obviously the pipeline test didn't catch this regression.

Wouldn't that be an issue by itself?
They should've caught this right?

@OlafvdSpek because mariadb clashes with mysql, we don't test it; this is an issue with our current CI system, and will be fixed at some point. Sorry!


Fix ack when RabbitListener returns Mono

This fixes issue 1006.
Note that I don't think it's possible to actually verify that it works automatically since I couldn't find a way to find the number of unacknowledged messages in a RabbitMQ queue programmatically. I actually tried to make an HTTP request to the management API (see here) but since it's not possible to know the username/password then this is not a feasible solution either.
I've verified that it works manually though.

@johanhaleby Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@johanhaleby Thank you for signing the Contributor License Agreement!

@johanhaleby ,
can you take a look into this https://github.com/spring-projects/spring-amqp/issues/1009?
For me it looks like related problem and can be fixed smoothly in your PR as well.
Thanks

OK. Since we have a fix for that issue and this one should be addressed as well, but no feedback from you, @johanhaleby , I'm going to take this task and issue a new PR.
Thanks for your effort anyway!
I guess I'm going to re-use some of your code and therefore will place your name into the @author list.

Superseded with: https://github.com/spring-projects/spring-amqp/pull/1011

Sorry, I'm currently on vacation and I don't have my computer at hand so
that's why I haven't answered. I'm just glad that it's going to be fixed :)
Thanks for reviewing my PR, sorry for not being able to have a better
conversation. And thanks for a great project.
tors 30 maj 2019 kl. 21:52 skrev Artem Bilan<EMAIL_ADDRESS>
OK. Since we have a fix for that issue and this one should be addressed as
well, but no feedback from you, @johanhaleby
https://github.com/johanhaleby , I'm going to take this task and issue
a new PR.
Thanks for your effort anyway!
I guess I'm going to re-use some of your code and therefore will place
your name into the @author list.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spring-projects/spring-amqp/pull/1007?email_source=notifications&email_token=AABNVFNDAULJF5VQZQWUT5TPYAV6VA5CNFSM4HORDUW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWTKYYQ#issuecomment-497462370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABNVFJ32JWPE4EMROISYI3PYAV6VANCNFSM4HORDUWQ
.

--
Sent from my phone


Can't connect internet after install

Did the install, mostly everything to default. After installation is complete (and reboot), the PI can't connect to the internet anymore. E.g., apt-get update will fail because no server can be found and ping www.google.com doesn't work anymore. (all this worked before installing).
Tried adding dns from google to ethernet and resolve, but nothing unfortunately.

Had to do with something changing the resolv.conf after pivpn was installed (on each reboot)

i have the same problem.... have you been able to resolve it?

can you paste in your install output?  also what distro/version etc?

I had this problem and the only way to fix it was to change the static domain_name_servers entry in /etc/dhcpcd.conf from my router (so <IP_ADDRESS> in my case) to <IP_ADDRESS>.
Reboot and you should be sorted.

Been trying to fix this problem for a week, its for part of an assignment so I was super stressed.

I had multiple line with the static domain_name_server but event if I changed them all, removed some or what ever I still doesn't have internet when I connect to the VPN :-(

Had the same problem. Thanks, @cabsandy

@cabsandy @AzaIndustries @brianpadley
the installer uses your default gateway as the dns server.  this typically works as the gateway is typically a home router who will forward dns requests out of its wan interface to the dns of your internet provide.  In your 3 cases this seems not to be true.
Can you guys tell me on a normal computer on your local nework, what is the gateway of that computer vs what dns servers does it use?  If they are in fact different then I'd need to ask that.
Also you'd only get in this state I believe if you used the installer to set a static IP on the raspberry pi.  Can you all confirm this as well?

I used static IP, I think it's the only option I had.
The DNS server on my pi was the router, and on normal computer it points to OpenDNS.


Reconsider usage of AnonCredsPredicate in an AnonCreds VP

I've been re-reading the DIF PE specification to create test vectors for AnonCreds W3C credentials and exchange messages.
I now noticed in the spec the following sections:

This requires that the returned value is a boolean, only if a predicate is used. I think this is based on the assumption that each attribute will only have one predicate query associated with it (so not age > 10 and age < 50 as then you won't be able to use a boolean anymore).
Based on this I think it may be good for now to limit anoncreds w3c presentations to only a single predicate per attribute (I've never seen different), and use the approach as described in DIF PE.
Then we will be aligned with the spec, and this would actually make it a lot easier for us to get this working with e.g. the PEX implementation we use in JS which is not created by us (https://github.com/sphereon-Opensource/pex).
cc @swcurran @Artemkaaas @gvelez17 @2mau

Interesting.  The “”age”: true” approach is what I thought we would do.
I think it would not be particularly limiting to allow only one predicate per attribute of a given name. In theory, multiple predicates could be referencing like-named attributes in different source credentials (which would be OK), but that’s really an edge case.

Yes. I can update to use boolean value instead object.
I believe we even can avoid making mentioned above limitations regarding the predicates' usage (single predicate per attribute and unique name across all predicates) by reintroducing explicit mapping put under encoded proof value.
          "mapping": {
            "revealedAttributes": {
              ....
            },
            "requestedPredicates": {
              "0_age": {
                "attr_name": "age",
                "p_type": ">=",
                "value": 18
              },
              "1_age": {
                "attr_name": "age",
                "p_type": "<=",
                "value": 30
              }
            }
          },

Hmm now that you say this, I think maybe multiple predicates should even be possible without the mapping right?
I can create the following DIF PD:
{
  "id": "5591656f-5b5d-40f8-ab5c-9041c8e3a6a0",
  "name": "Age Verification",
  "purpose": "We need to verify your age before entering a bar",
  "input_descriptors": [
    {
      "id": "age-verification",
      "name": "A specific type of VC + Issuer",
      "purpose": "We want a VC of this type generated by this issuer",
      "schema": [
        {
          "uri": "https://www.w3.org/2018/credentials/v1"
        }
      ],
      "constraints": {
        "limit_disclosure": "required",
        "fields": [
          {
            "path": ["$.issuer"],
            "filter": {
              "type": "string",
              "const": "did:key:z6MkwXG2WjeQnNxSoynSGYU8V9j3QzP3JSqhdmkHc6SaVWoT"
            }
          },
          {
            "path": ["$.credentialSubject.name"]
          },
          {
            "path": ["$.credentialSubject.height"]
          },
          {
            "path": ["$.credentialSubject.age"],
            "predicate": "preferred",
            "filter": {
              "type": "number",
              "minimum": 18,
			 "maximum": 21
            }
          }
        ]
      }
    }
  ],
  "format": {
    "di_vc": {
      "proof_type": ["DataIntegrityProof"],
      "cryptosuite": ["anoncredspresvc-2023", "eddsa-rdfc-2022"]
    }
  }
}

This will translate to two different predicates:

age >= 18
age <= 21

I think the only thing that is important here is that we need a consistent way to map from a PD to an AnonCreds proof request (mainly the group names are important here I think)

For doing credential subProof verification, we have to pass predicate type and value as well.
When we change a credential predicate representation to have a bool as a value, we need to find the corresponding entry in the AnonCreds proof request.
In case of multiple predicates with the same name, we can take a wrong one if there is no restriction.


fix: Convolution in convolve_2d.

Description
This PR fixes the issue explained in #722.
References

Tasklist

[x] Add test case(s)
[ ] Ensure all CI builds pass
[ ] Review and approve

The fix looks good to me, https://en.wikipedia.org/wiki/Sobel_operator also agrees

Since the previous values come from #392, I'd give @simmplecoder a chance to check the proposed fix.
If none arrives and our tests are green, I'm happy to merge it.
Thank you!


v1.15.0

New Features

Added nav
Added means to duplicate a list item
Added task to associated video file
read in file with invalid yaml fields, just knowing invalid fields


Redirect when route scn-social-auth-user/register

When performing non-social registration, now redirectioning to configured route.

Thanks for the fix!


Adding Java to the CI

Description
Adds testing to the java CI. This is a direct port from the Java v0 so testing in this PR.
Running JavaSpoltless to get a clean build
Related
Closes #410

[X ] I have reviewed the Guidelines for Contributing and the Code of Conduct.

Closing to reopen with a nice clean PR

Well gave up on making the commits prettier here. Will squash before full merge instead


Block scoped typing

Search Terms
block, scoped typing

Suggestion
I would love to tell to TS from this bracket to this bracket the type of this variable is Foo
This feature works when used with instanceof operator but when using some libraries (e.g. axios)
instanceof won't work there as AxiosError is not a value (it is an interface which doesn't exist during runtime).
Currently, we need to define a helper variable to cast this type.
Use Cases
if(err instanceof Foo) {
  // err is now treated as a Foo
} else if((err as any).isAxiosError) {
  // err is not treated as a AxiosError so TS will treat it as Error so I can't access other properties without casting
}

Examples
if(err instanceof Foo) {
  // err is now treated as a Foo
} else if(((err as any).isAxiosError) as AxiosError) {
  // err is now treated as a AxiosError
}

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

Isn't a custom type guard sufficient for this?
function isAxiosError(e: any): e is AxiosError {
    return !!e.isAxiosError;
}


Coach taking up "competitive spawns" on maps where there are only 5 "competitive spawns"

Coach is taking up "competitive spawns" on maps where there are only 5 "competitive spawns", pushing players to non-default spawns. A big deal on certain maps where there are instant smokes based off the five spawns (ancient t and ct, anubis ct, vertigo t and ct, inferno ct) pracc and refrag have managed to avoid this issue by somehow spawning the coaches above the players(maybe they just spawn normally and then manually check the positions of the coach/5 players, then teleport?). Love the plugin and all the great work you've been putting in aside from this!

Hey!
Yes I think we can teleport the player taking the competitive spawn somewhere else and bring back the actual player to the competitive spawn to avoid this issue.
I'll also explore if any better way is available and I'll try to fix this asap
Thanks! :D

any update on this? I know this, as well as coach having to run out of the way of players is keeping many teams from switching to this over pracc.com/refrag servers.
thanks for your work!

Hey!
Apologies for the delay, couldn't get the chance to work on it, but I'm trying to resolve it ASAP
Thanks!

Hey!
This has been resolved in the latest releases of MatchZy!
Thanks!


Improve DataFrame#assign, #rename to accept [key, values] Array

Current API in DataFrame#assign and #rename only accepts { key => value } Hash pairs.
(value is Array, Arrow::Array or Vector)
It will be convenient to accept also [key, values] Array.
Because Array#map will return an Array, it is easy to pass Array to #assign or #rename.

I will try to use pattern matching to parse overloaded parameters.
It will make the code for parsing simpler.
One concern is that pattern matching was experimental feature in Ruby 2.7. Warning message will be displayed.

I implemented in #61 .


Broken build due to kind

What steps did you take and what happened:
Travis is currently failing due to a problem with getting/installing kind.
49.72s$ go get sigs.k8s.io/kind
# sigs.k8s.io/kind/pkg/kustomize
../../../sigs.k8s.io/kind/pkg/kustomize/kustomize.go:108:7: undefined: k8sdeps.NewFactory
../../../sigs.k8s.io/kind/pkg/kustomize/kustomize.go:109:26: not enough arguments in call to build.NewCmdBuild

What did you expect to happen:
Successful build.
Anything else you would like to add:
I think they recently went to using gomodules and so we may need to tweak something to properly utilize them. (Could be wrong, but locally I hit that because I was in my GOPATH)

https://github.com/kubernetes-sigs/kind/issues/509

TL;DR; it seems that we just need to get off from doing go get ... and do a clone/checkout of a specific version. With modules and the lack of vendor now go get is having problems which aren't really problems with kind; master just isn't guaranteed, only tagged releases.

GO111MODULE=on go get sigs.k8s.io/kind also works for me locally. Being the fast/easy fix this is sufficient for our our needs in travis IMO.

Thanks and sorry about that @johnSchnake, I'd additionally ask to consider using a fixed version so we don't surprise you again :(
0.3 will be out this week, perhaps at that point using a pre-built release binary will be sufficient?

Sounds good to me; I had limited availability today so I just did the fastest fix I could find. Happy to fix the version especially if a new release is getting tagged soon.

Thanks for understanding and apologies again. 0.3 will indeed land before the end of next week I promise, and should let go get without @master pick that instead of HEAD. I'll send a note when it's out.


Breaking Changes {On Update}

Found an issue or bug with electron-vue? Tell me all about it!
Breaking changes on update as the default file extension is src/index.ejs
Describe the issue / bug.
receive the error message;
ERROR in Template execution failed: ReferenceError: process is not defined
ERROR in   ReferenceError: process is not defined
- index.ejs:11 eval
  [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:11:2

- index.ejs:16 module.exports
  [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:16:3

- index.js:284
  [elect]/[html-webpack-plugin]/index.js:284:18

- task_queues.js:97 processTicksAndRejections
  internal/process/task_queues.js:97:5

Child html-webpack-plugin for "index.html":
Asset      Size  Chunks  Chunk Names
index.html  1.37 MiB       0
Entrypoint undefined = index.html
[./node_modules/html-webpack-plugin/lib/loader.js!./src/index.ejs] 1.14 KiB {0} [built]
How can I reproduce this problem?
follow the instructions to update then yarn run dev
If visual, provide a screenshot.
no screenshort
Tell me about your development environment.

Node version: v13.11.0
NPM version: v16.15.5
vue-cli version: 4.5.4
Operating System: Linux Mint 19.3 Tricia

Have the same error. Using macOS Catalina. What is going on? Any help here?

Same issue on Windows.

Same issue, attempted with yarn and did not include any extras on init, all attempts result in the same issue.
Node: 14.15.1
NPM: 7.5.6
Vue CLI: 4.5.9
Operating System: macOS Catalina 10.15.7


Corrected fix to issue #940

Hi @antonsmislevics, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Manually merged. Closing this PR now.


New tab mouse hover and border styles

There are some new themable color options nord could support, though it's certainly not necessary. The new colors are:
tab.hoverBackground
tab.hoverBorder
tab.unfocusedHoverBackground
tab.unfocusedHoverBorder

I have mine set like this
"workbench.colorCustomizations": {
        "tab.hoverBackground": "#434c5e",
        "tab.hoverBorder": "#d08770",
        "tab.unfocusedHoverBorder": "#88c0d0"
}

Thanks for your help, I've quickly read over the release notes of version 1.20 some days ago and already added a note card for the new theme keys.

The new keys have been added and can be tested in PR #89.


Consider the new DV reason ImagePullFailed

CDI v1.60.1 introduces a new reason ImagePullFailed for the DataVolume Running condition. Take it into account to properly update the printable status of a VM and to report the error.

What this PR does
Before this PR:
The test "[test_id:7679]should report an error status when data volume error occurs" fails with the latest CDI v1.60.1
https://github.com/kubevirt/kubevirt/blob/a3a9e83da6fff2de37b7fd87efb43466cd6e07d4/tests/vm_test.go#L736
After this PR:
The test passes.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/lgtm
/cc @mhenriks @nunnatsa @tiraboschi

/approve

/test pull-kubevirt-e2e-k8s-1.31-sig-storage

/cherrypick release-1.3


fixed mem issue when generating uuid

Currently, whenever we create an id, we are allocating 600 bytes of memory for each id.
This is because chromium has a bug that does not allow using the '+' operator to allocate memory correctly: https://bugs.chromium.org/p/v8/issues/detail?id=2869
Changing this to a join will fix the size difference:
Concatenate: Shallow Size 40, Retained Size 600
Join: Shallow Size: 64, Retained Size: 64

It'd be nice to have a more concrete recipe for reproducing this.  E.g. what diagnostic tools are you using?   Specific steps to see the result.
Also, is this causing problems for in-the-wild code, or is this just an academic issue?  That Chromium issue has been open for 4+ years.  That nobody has complained about it here before has me questioning how important this is.
My apologies for being mulish, but I'm not a fan of patching code for the sake of fixing issues in upstream projects.  There are a couple problems with this.  One, it increases code complexity.  E.g. before accepting your PR, I'll ask that you add a comment explaining why the code is structured the way it is, and referencing the above issue.  Two, there are always trade offs.  E.g. your PR results in the creation of a [transient] array, thus adding [admittedly trivial] GC perf hits elsewhere.
But, really, this just boils down to the fact it's silly for dozens or hundreds of current and future module owners to have to inject this hack when this should be fixed in Chromium.
[Leaving this open for the time being, since this is a legitimate issue and change request.  'Just not 100% sure it's worth worrying about.]

V8 developer here, just to quickly note that this is unrelated to https://bugs.chromium.org/p/v8/issues/detail?id=2869 (which is about substrings of long strings, not string concatenations created with +).
The reason for the difference in memory consumption is that V8 makes string additions fast by simply creating a "cons string" that refers to the two inputs, rather than copying all the characters. Certain operations later cause such "cons strings" to be "flattened", so depending on what you use the generated UUIDs for, memory consumption may well decrease later.
My opinion is that it is fine to land this patch, as the .join trick makes it easy for any JavaScript engine to recognize what you're doing (i.e. concatenating 20 one-character strings), rather than requiring complicated analysis to figure out what's going on. That said, I also wouldn't worry about it unless you're actually seeing an issue in practice.

Sorry that's the bug for a different issue I'm working on. Here is one open bug filled that is about concatenations   (in this case they ran out of memory due to the issue).
Our app is using this to create our ids - many at a time, so we are noticing the difference in memory.
If you would like to reproduce this I made this test app: http://plnkr.co/edit/gonu837kTr0IS8uG4SLt?p=preview
I used the chrome developer tools to get the heap snapshot after it created guids using both methods.    These were the results I found:

@jakobkummerow Thanks for chiming in.
For the sake of others that may stumble across this, do you happen to know what other operations will cause cons strings to be flattened?  (I.e. is Array#join() the most elegant hack here or, for example, is there a way to flatten without needing to create an array?)

For anyone else stumbling across this, it appears that some (but not all) functions that transform a string have the side-effect of flattening the underlying "cons strings".  E.g. str.toLowerCase(), str.trim(), str.replace(/^/, ''), and (str + 'x').substr(0, str.length) all appear to work (in v8).  str.substr(0) does not, however.
@sava-smith Knowing this, we could do ( ... ).toLowerCase() on the concatenated string instead of [...].join(). three.js is doing this (with toUpperCase instead).  But... I'm guessing the join() pattern is a little more efficient(???) so let's stick with that for now.


test: setup detox

closes #469

Motivation
Setup Infra for testing

Test plan

[x] Detox Android
[x] Detox ios

Hi, I am trying to use https://github.com/react-native-community/react-native-circleci-orb for circleci setup. If I can have access for it, I will try to set up or else I will remove the last commit and we can try another PR for circleci

Any update on this??

Hi, @chakrihacker
What do you want us to do?

I am getting this error when using orbs
Orb<EMAIL_ADDRESS>not loaded. To use this orb, an organization admin must opt-in to using third party orbs in Organization Security settings.
please enable use of third party orbs

hey @chakrihacker, it's already enabled in circleci settings. maybe it's because the branch is under your personal user? I just gave you write access to this repo so you can try by creating a branch.

Thanks for access, I will finish this weekend

I think it's time to request mac plan.
More details here https://circleci.com/open-source/
need to put a mail to circleci team

Thanks! I'll check it.

@chakrihacker @satya164
Maybe we can use Travis for it?

Whichever is easier :)

Travis ci doesn't support orbs

We'll contact circleCI for it! We do also want to use it bc Travis does not support Android virtualization well.

@chakrihacker I emailed circleci and we already have a Mac OS plan active for the community org. Can you move this branch to the repo so we can check if the tests run?

ok


Waste of our time maintaining this as open source TBH


Button image color highlighting

Emu doesn't know whether button images should be highlighted or not

fixed in 7014766e88da1cc5934aab1bb66172ebed4ddcdf


Update README.md

Problem
...
Changes

...

Testing
...

Main branch dependencies

Neodora Analysis ⚠
 ⚠ Missing cache in Github Workflow

Message: Github workflow should be using caching when step uses actions/setup-node

Workflow file
Job name
Step index
Actual cache value
Expected cache value

functional-test.yml
docker
1
Not found
npm

🔍 How to fix it

 ⚠ Unknown tribe name

Message: The property owner in repository.json must be a valid tribe name present in mission-tracking
Repository file: repository.json

Owner status

'CORE' is an unknown tribe name

🔍 How to fix it

 ⚠ Missing exclusions in .dockerignore

Message: .dockerignore should contain .git and .deploy exclusions
File: .dockerignore

Missing exclusion

.deploy

🔍 How to fix it

 ⚠ Missing cache in Github Workflow

Message: Github workflow should be using caching when step uses actions/setup-node

Workflow file
Job name
Step index
Actual cache value
Expected cache value

on-commit.yml
test
2
Not found
npm

🔍 How to fix it

 ⚠ Missing jsonpFunction

Message: jsonpFunction must be defined in webpack configuration if using webpack version lower than 5.0.0

Webpack file missing jsonpFunction

webpack.config.cjs

🔍 How to fix it

Commit cc2dcda  Job 15428


API reference guide links are broken

Describe the bug
There are few API reference on the documentation sites which have broken links. These links are mentioned on main docs site.
Broken Links

https://chainsafe.github.io/lodestar/packages/lodestar-cli
https://chainsafe.github.io/lodestar/packages/lodestar-validator
https://chainsafe.github.io/lodestar/packages/lodestar-utils
https://chainsafe.github.io/lodestar/packages/lodestar-types
https://chainsafe.github.io/lodestar/packages/lodestar-spec-test-util
https://chainsafe.github.io/lodestar/packages/lodestar-params
https://chainsafe.github.io/lodestar/packages/lodestar-config

Expected behavior
All the links on the website should be live.
Screenshots

@wemeetagain should we just delete the typedoc documentation? I have never used it or seen someone using it since 2020

Yea i think we can delete for now. I think it would be nice to have some api docs (especially for things like the config, types, api, lightclient, etc). But it might just be better to start from a clean slate.

Closing for https://github.com/ChainSafe/lodestar/issues/4311


Add upgrade for rename getUsername to getUserIdentifier

This will already rename the sulu users getUsername to getUserIdentifier.
Try out this PR by copy following in your unix shell:
Upgrade for: https://github.com/sulu/sulu/pull/6626
git clone git@github.com:sulu/sulu-rector.git
cd sulu-rector/
git remote add alex git@github.com:alexander-schranz/sulu-rector.git
git fetch alex
git checkout feature/replace-user-name-with-user-identifier-method
composer install
composer test

I don't get why this is failing in the CI 🤔 it seems to work correctly locally.

@TomasVotruba Sorry for pinging you but I'm little bot lost again. Do you have any hint what is going wrong here? It works locally for me:
git clone git@github.com:sulu/sulu-rector.git
cd sulu-rector/
git remote add alex git@github.com:alexander-schranz/sulu-rector.git
git fetch alex
git checkout feature/replace-user-name-with-user-identifier-method
composer install
composer test

but the rule just don't work in the CI which is very strange. Even checked if the CI installs other dependencies but they are the same. Also did cat the files the looks also correclty in the CI not sure what is going wrong here. Even updated the composer install actions still the rule test does not work in the CI.

I've just tested for a while locally to find this fix: https://github.com/rectorphp/rector-src/commit/cbebb9a4de7d15fedd55d5035c11603397c85c27
Basically this rename would break parent interface contract. Untill the Symfony require parent interface, the method rename would cause a BC break:
public function getUsername();

Yet, in this case, the change could be done in an interface :+1: I'll see if we can improve it

This should help: https://github.com/rectorphp/rector-src/pull/2362
Could you try rector dev-main?

See https://github.com/sulu/sulu-rector/pull/8


仓库中vue3+vite的代码示例报错

Describe the bug
本仓库中vue3+vite的代码示例在IDE中报红(已经正常pnpm install)。

Reproduction
no need
System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (12) x64 AMD Ryzen 5 5600X 6-Core Processor
    Memory: 3.79 GB / 15.91 GB
  Binaries:
    Node: 16.18.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
    npm: 8.19.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (109.0.1518.70)
    Internet Explorer: 11.0.19041.1566

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

fixed

got the same issue with unplugin-vue-define-options 1.2.0

经定位该问题是从1.7.0版本开始的，之前版本正常


Run danger against PRs

I would like to implement http://danger.systems/ for mithril PRs.

They've got a nodejs version that I implemented against modular-css as an experiment, it's nice.
Here's my current list of ideas for checks:

🛑 PR targeting master instead of next
🛑 lint failures (since they don't fail the build)
🛑 Require the "use strict"; pragma at the top of any .js files
⚠ Request a change-log.md entry for .js file changes
⚠ Request a descriptor of the semver change required for the PR to land (if .js files changed)

Any other ideas? Anyone vehemently against this proposal?

I'm vehemently for this proposal

I like this proposal a lot myself. I'll note that the "use strict" requirement is already mandated by the ESLint settings, so making DangerCI check it is redundant.

@isiahmeadows fair, edited. I think the danger-style output is a lot nicer to read than eslint results though. Especially since lint failures don't fail the build.

yet.


"user-error: Not on a heading"

Hello,
Thanks for the cool-looking tool!
I am trying to set title, author, etc. using #+OPTIONS, but I keep getting the error "user-error: Not on a heading". This seems to happen whenever the first line of the .org file is not a heading (e.g. * xyz).

I have the same problem.    If I remove the #+ items, it works.

I'm having this problem too. Would love a solution or workaround.

I have recently delved back into the hell that is Emacs development in order to release a new version of the package. I've managed to reproduce this with org 9.1,9 so I think there is some backwards-incompatible change in the API between that and the current 9.4. I haven't been able to test any of the intermediate versions because of course Emacs isn't capable of installing a specific version of a package. Could you tell me what version you are all running (SPC SPC org-version)?

Thanks for this, it's very handy!
I'm on org-mode version 9.6 and see this problem.
It seems to be triggered by the call to (org-get-tags) in ox-json-transcode-headline; if I comment out the enclosing push call it works.
It looks to me like the call isn't occurring at a header. The docs for org-get-tags mention an optional pos-or-element argument; I tried it with (org-get-tags headline nil) but this gave me an error Wrong number of arguments: (0 . 0), 2 which I don't understand,  and I'm out of my depth.

Just to confirm - you are on 9.1.9 and not on 9.6?

Yes, sorry for the confusion! I was running against 9.1.9 on another machine and stupidly checked org-version locally.

Okay, I'm going to go ahead and close this issue because currently stated minimum org version is 9.2 and I don't expect this will change.


Doesnt work

Describe the bug
A clear and concise description of what the bug is. Errors and logs go down below, not here!
To Reproduce
Steps to reproduce the behavior:

Do '...'
Go '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Plugin version

[x] Using the latest released version of the plugin
[ ] Using the legacy jar

Server version
The Minecraft version your server is running. Bonus points for the full version including the hash.
Additional context and logs
Add any other context about the problem here. Also post error messages and logs (use a service like Hastebin for logs, please!) in this area.

it doesnt work eventhough i added the worlds to the config file

I get the same thing.

I have successfully done all I needed to do with the config and world files, but my inventory keeps getting wiped as I go into the nether. When I go back it brings back my inventory as if both worlds still have separate inventories.


[Nova] MacOS Unittests on Nova

As discussed in the Torchaudio Migration Proposal, this PR moves MacOS unittest job to Nova tooling. Note that this does not touch anything within the existing CircleCI job at the moment.
Passing job: https://github.com/pytorch/audio/actions/runs/4932497525/jobs/8815581251?pr=3324

Failure on latest MacOS test is due to network error, but the job looks otherwise fine:
requests.exceptions.ConnectionError: HTTPConnectionPool(host='localhost', port=8000): Max retries exceeded with url: /torchaudio_unittest.backend.dispatcher.ffmpeg.load_test.TestFileObjectHttp.test_frame_0_-1/test.wav (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fc8bca91f10>: Failed to establish a new connection: [Errno 61] Connection refused'))


Add support for webpack dev server to inject in-memory build assets into AEM served pages

Description
Webpack dev server should proxy content pages from AEM and inject it's in-memory assets in the page. This way the dev server will create the bundles in memory. Webpack provisioned clientlibs should not be proxied.
For the ui.frontend.react, this requires rewiring the CRA Webpack config.
Related Issue
#631
Motivation and Context
Webpack dev server only proxies, but still requires a build to provision the clientlibs. Live reloading is not supported with watch.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Hi @robinalaerts1, I am testing this PR locally, I built the archetype and generated project based on the new version using your branch. After running npm install and then npm run start in ui.frontend (general FE module used) I got the following error:
× ｢wds｣: Invalid configuration object. Webpack has been initialised using a configuration object that does not match the API schema.
 - configuration has an unknown property 'publicPath'. These properties are valid:
   object { amd?, bail?, cache?, context?, dependencies?, devServer?, devtool?, entry?, externals?, infrastructureLogging?, loader?, mode?, module?, name?, node?, optimization?, outp
ut?, parallelism?, performance?, plugins?, profile?, recordsInputPath?, recordsOutputPath?, recordsPath?, resolve?, resolveLoader?, serve?, stats?, target?, watch?, watchOptions? }
   For typos: please correct them.
   For loader options: webpack >= v2.0.0 no longer allows custom properties in configuration.
     Loaders should be updated to allow passing options via loader options in module.rules.
     Until loaders are updated one can use the LoaderOptionsPlugin to pass these options to the loader:
     plugins: [
       new webpack.LoaderOptionsPlugin({
         // test: /\.xxx$/, // may apply this only for some modules
         options: {
           publicPath: …
         }
       })
     ]

Is this some kind of version miss-match ? Not sure whether it's important - I am testing this on windows 10, npm 6.14.6, node v12.18.3
Sample project generated using the following command:
mvn -B archetype:generate \
 -D archetypeGroupId=com.adobe.aem \
 -D archetypeArtifactId=aem-project-archetype \
 -D archetypeVersion=27-SNAPSHOT \
 -D appTitle="My Site test" \
 -D appId="mysite-test" \
 -D groupId="com.mysitetest"

To fix that locally I added the following output object inside webpack.dev.js
output: {
            publicPath: '/'
        },
        devServer: {
            inline: true,
            proxy: [{
                context: [
                    '/content',
                    '/etc.clientlibs/'
                ],
                target: 'http://localhost:4502',
            }],
            before: (app, server, compiler) => {
                app.use(injectScripts(compiler));
            },
        }

Also it seems that '/etc.clientlibs/*/clientlibs/clientlib-base.*.*' does not work properly as it causes error:
Error: [HPM] Invalid context. Expecting something like: ["/api", "/ajax"] or ["/api/**", "!**.html"]


Expose --storage.tsdb.retention.time in a metric

Proposal
As I have many Prometheus instances with different retention settings defined different repositories, I'd love to have a dashboard where to combine them all with a Prometheus query.
I can find the metrics prometheus_tsdb_retention_limit_bytes for --storage.tsdb.retention.size (clear enough) as well as prometheus_tsdb_size_retentions_total and prometheus_tsdb_time_retentions_total (no idea what these are) but I don't find a metric reflecting the --storage.tsdb.retention.time setting (I've not tried --storage.tsdb.retention as I don't use it and I understand it's deprecated since a while ago).
So, would it be possible to add a metric like prometheus_tsdb_retention_time_limit_days or something, with an appropriate conversion from --storage.tsdb.retention.time units?

Hey! 👋 Is it something the project/community would be interested in? If so, I can open a PR and hash out the details there.
I believe it should be something in this direction. https://github.com/prometheus/prometheus/commit/89449ce5e7fba6a37448e82c58918f72adaec694
(just a draft. still need to think of which unit & metric name to use) I'd love to give a try!

Hi there @beorn7  :wave: (hopefully I'm tagging the right person, but lmk if not)
Is this issue something the project/community would be interested in?
If yes, I'll continue & work in a PR :runner:

Thanks for this idea! I think it's worth accepting, contributions are welcome 🤗
Perhaps prometheus_tsdb_retention_seconds - concise and in base units as our metric name style guide recommends.
(Triaged during Prometheus bug scrub meeting)

Maybe we can close this as implemented

Indeed. Thanks everyone!


Lock down helper traits like Dimension

We need to be stricter (at least in the docs) and hide trait methods. These traits are not intended to be used as more than trait bounds.

Fixed by #82


Short vibration

It would be great if the slider could generate a short vibration when the circle goes through each step

Sounds great!

Implemented in 1.3.0


Metabase stuck in a loop

Hello!
I'm trying to start the system via docker compose up . All pieces come up nice, except for Metabase which seems stuck in a loop with this error message:
metabase  | 2023-04-19 07:14:05,812 INFO db.update-h2 :: H2 v1 database detected, updating...
metabase  | 2023-04-19 07:14:05,812 INFO db.update-h2 :: Creating v1 database backup at /tmp/metabase-migrate-h2-db-v1-v2.sql
metabase  | 2023-04-19 07:14:06,340 INFO db.update-h2 :: Moving old app database to /conf/metabase.db.v1-backup.mv.db
metabase  | 2023-04-19 07:14:06,341 ERROR db.update-h2 :: Failed to update H2 database: #error {
metabase  |  :cause /conf/metabase.db.mv.db -> /conf/metabase.db.v1-backup.mv.db
metabase  |  :via
metabase  |  [{:type java.nio.file.AccessDeniedException
metabase  |    :message /conf/metabase.db.mv.db -> /conf/metabase.db.v1-backup.mv.db
metabase  |    :at [sun.nio.fs.UnixException translateToIOException UnixException.java 90]}]
metabase  |  :trace
metabase  |  [[sun.nio.fs.UnixException translateToIOException UnixException.java 90]
metabase  |   [sun.nio.fs.UnixException rethrowAsIOException UnixException.java 106]
metabase  |   [sun.nio.fs.UnixCopyFile move UnixCopyFile.java 481]
metabase  |   [sun.nio.fs.UnixFileSystemProvider move UnixFileSystemProvider.java 266]
metabase  |   [java.nio.file.Files move Files.java 1430]
metabase  |   [metabase.db.update_h2$update_BANG_ invokeStatic update_h2.clj 81]
metabase  |   [metabase.db.update_h2$update_BANG_ invoke update_h2.clj 68]
metabase  |   [metabase.db.update_h2$update_if_needed invokeStatic update_h2.clj 98]
metabase  |   [metabase.db.update_h2$update_if_needed invoke update_h2.clj 90]
metabase  |   [metabase.db.data_source.DataSource getConnection data_source.clj 29]
metabase  |   [com.mchange.v2.c3p0.WrapperConnectionPoolDataSource getPooledConnection WrapperConnectionPoolDataSource.java 161]
metabase  |   [com.mchange.v2.c3p0.impl.C3P0PooledConnectionPool$1PooledConnectionResourcePoolManager acquireResource C3P0PooledConnectionPool.java 213]
metabase  |   [com.mchange.v2.resourcepool.BasicResourcePool doAcquire BasicResourcePool.java 1176]
metabase  |   [com.mchange.v2.resourcepool.BasicResourcePool doAcquireAndDecrementPendingAcquiresWithinLockOnSuccess BasicResourcePool.java 1163]
metabase  |   [com.mchange.v2.resourcepool.BasicResourcePool access$700 BasicResourcePool.java 44]
metabase  |   [com.mchange.v2.resourcepool.BasicResourcePool$ScatteredAcquireTask run BasicResourcePool.java 1908]
metabase  |   [com.mchange.v2.async.ThreadPoolAsynchronousRunner$PoolThread run ThreadPoolAsynchronousRunner.java 696]]}

I assume it's a matter of permissions, because the H2 DB file resides on the local drive, although I tried with chmod 777 on data, and it's the same. Could you please share the version of the Metabase container that you used for base in building ?
Thank you!

Hi, I was using the Metabase version 0.44.6. Can you try to change the file access permissions of the /conf folder?

Thanks for getting back.


RSDK-7924: Fix building of example projects

This PR includes changes that were needed to get the example projects building on my machine. The work here is pursuant to a related ticket which would build the example projects as part of CI as well to prevent them from breaking in the future.
Other than the more obvious changes to source file paths in the Makefile and CML.txt, there were some less obvious changes needed to the way rpaths are set in installation. After fixing the source file paths, there was an error where the build would succeed but running the executables would fail on startup with an rpath error loading the dynamic libraries. The issue seemed to be that

rpath was not being set at all for Makefile projects
In CMake projects the rpath contained a stuttered directory which was pointing to a nonexistant path, i.e., /path/to/sdk/install/*lib*/lib/libviamsdk.noabi.so

I have removed a line in the sdk CML.txt that seems to have been producing issue 2, and now issue 1 is solved by passing the rpath to the linker in the Makefile
I was encountering some additional build failures in the Makefile project which I've addressed by adding some overridable variables with ?=

rpath was not being set at all for Makefile projects
In CMake projects the rpath contained a stuttered directory which was pointing to a nonexistant path, i.e., /path/to/sdk/install/*lib*/lib/libviamsdk.noabi.so

Here are some details on why we are getting the stuttered lib directory:

When the example gets built in a standalone cmake project, it gets built with -Wl,-rpath,/Users/andrewmorrow/Documents/viam/dev/viam-cpp-sdk/build/install/lib
12:52
As well as …/build/install/lib/libviamsdk.0.0.9.dylib on the link line
12:53
Now, …/build/install/lib/libviamsdk.0.0.9.dylib has an install name of:
$ otool -D build/install/lib/libviamsdk.dylib
build/install/lib/libviamsdk.dylib:
@rpath/lib/libviamsdk.noabi.dylib
12:54
So it gets recorded in the example program as:
Load command 14
cmd LC_LOAD_DYLIB
cmdsize 64
name @rpath/lib/libviamsdk.noabi.dylib (offset 24)
time stamp 2 Wed Dec 31 19:00:02 1969
12:54
But the -rpath argument set the rpath to: .../build/install/lib
12:55
So when the @rpath in @rpath/lib/libviamsdk.noabi.dylib gets expanded in the context of the example, it comes out as …/build/install/lib/lib/libviamsdk.noabi.dylib
12:56
Which of course doesn’t exist
12:58
If I relink the example with an edited -rpath argument of -Wl,-rpath,/Users/andrewmorrow/Documents/viam/dev/viam-cpp-sdk/build/install (removing the lib) then it works
12:59
So somehow we need to get those .cmake files to drop the lib out of the rpath that gets into consumers
12:59
I am …. not sure how to do that
12:59
OR, we need to rethink the scheme we are using for @loader_path and @rpath and such for the SDK


this.windowProvider?.on is not a function

I am getting this.windowProvider?.on is not a function error if we add on react project.

I think it's just dependency conflict especially on the wallet connection.
I am using rainbow kit provider along with react version 18.2.0.
Do you know why this happens?

@sbvhev Hey, could you please share a reproducible example? 🙂

I am not sure how you can reproduce this error. You should probably use rainbow kit as an example of wallet connection and use react 18.2.0. That way, you can see that error.

@sbvhev Can you at least share how you use the widget with the rainbow kit? 🙂


Mapping of 'RelatieZorgepisode' in MedicationAgreement and DispenseRequest

The MP9 dataset has the concept RelatieZorgepisode that links the MedicationRequest to an EpisodeOfCare. With R4, the reference to EpisodeOfCare has been removed. Only a reference to Encounter can be made natively which in turn can reference the EpisodeOfCare.
In the comment on MedicationRequest.encounter is written: 'If there is a need to link to episodes of care they will be handled with an extension.'. However, the only available extension that looks usable is not allowed to be used in MedicationRequest in R4: https://hl7.org/FHIR/r4/extension-workflow-episodeofcare.html.
In the build version this is solved: http://build.fhir.org/extension-workflow-episodeofcare.html.
For now I propose to place an implicit mapping on MedicationRequest.encounter that states that the mapping of RelatieZorgepisode is placed on Encounter.episodeOfCare. Would this work? Or do we need to create a custom extension.

Please also look at what has been done / will be done for information standards general practitioner / ketenzorg. The requirement for these relations have their source in the world of general pratitioners (and are very ubiquitous in that environment). It would be nice to have a consistent solution.

@ahenket , can you think along for how the information standard GP data will want to tackle this problem? Any helpful thoughts?

@pieter-edelman-nictiz , do you have an opinion about this?

The FHIR native version cannot be used on MedicationRequest in R4, but the build version that you linked allows for this context. We could mock this FHIR extension in order to pre-adopt this wider range, but then you would need to restrict/add the target profile to the nl-core version in the using profiles, while you'd still not use an actual core extension. So in this case it would maybe be better to just use a custom extension.
The current version of said extension can be found here: https://simplifier.net/nictizstu3-zib2017/extension-context-nl-core-episodeofcare and is a generic mechanism to link an nl-core-episodeofcare instance.

I will mimic the STU3 custom extension for R4 and make a comment/note that from R4 onwards, the extension will likely be replaced by a HL7 core extension.
I also edited my initial issue description to make it more clear that the extension is not usable in R4, but the build version shows this will be fixed in R5.

This has been added to the profiles, therefore I will close the ticket. It can be reopened if the approach needs to be fundamentally different. Normal feedback will follow during the review procedures.


Working bundle code reference

Check out
https://twitter.com/nicksdjohnson/status/1490891360144347136?t=64k6I1CloffFgSat6kv8nA&s=19

https://docs.flashbots.net/flashbots-auction/searchers/advanced/rpc-endpoint#authentication


Fix issue with missing transcripts in recent videos

We needed to load the video player before loading transcripts since they
go directly to the video player via a notification
JIRA: https://openedx.atlassian.net/browse/MA-358

@hanningni please review

:+1:


Every HybridOp should implement map_wires

Or alternatively, a base class should implement map_wires.

I do wonder, why should they implement this method (in what context does it come up or cause issues)?

Well, I had this test here, but it got deleted :sweat_smile: https://github.com/PennyLaneAI/catalyst/pull/555/files
The issue came from a template or transform that uses map_wires in operations. If HybridCtrl didn't implement it, then it would fail. I believe with the recent changes where HybridCtrl inherits from another control it might be solved, but I am not a 100% sure.

Ah thanks, that makes sense! I don't think it's fully solved yet.


fix(RingTheory/GradedAlgebra/Basic): rewrite simps lemma by hand

The simps lemmas generated here were bad.
The porting note matches the one on decomposeAddEquiv which also needed manual lemmas.
This also adds new lemmas that help keep simp confluent.

bors merge


Compile error on cuda 12.3

Hello, I try compiling the flash atten on the cuda 12.3 and got the error of segmentation default. It works well with cuda 11.8. I located the error on this function. https://github.com/Dao-AILab/flash-attention/blob/main/csrc/flash_attn/src/flash_fwd_kernel.h#L139
Appreciate it if anyone has an idea about it.  Many thanks!

How do you solve this, decrease the version of CUDA or find a right version of flash_attention? Thank you.


Add regulator to filesystem

This PR adds a regulator wrapper around the Filesystem driver.
The regulator ensures that there is an upper bound on the number
of concurrent blocking filesystem operations which spawn new
threads.
By default maxthreads is set to 100. This means the FS driver is
limited to 100 concurrent blocking file operations. Any subsequent
operations will block in Go until previous filesystem operations
complete.
This ensures that the registry can never open thousands of simultaneous
threads from os filesystem operations.
Note that maxthreads can never be less than 25.
PS: Thanks jlhawn for the regulator implementation.
Edit:
snip

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "add-regulator-to-filesystem" git@github.com:tonyhb/distribution.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "add-regulator-to-filesystem" git@github.com:tonyhb/distribution.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

related: https://github.com/golang/go/issues/7903

Looking good.
This will need documenting in docs/configuration.md and docs/storage-drivers/filesystem.md

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "add-regulator-to-filesystem" git@github.com:tonyhb/distribution.git somewhere
$ cd somewhere
$ git rebase -i HEAD~7
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

[Current coverage][cc-pull] is 53.81%

Merging [#1674][cc-pull] into [master][cc-base-branch] will decrease coverage by -7.86%

4 files (not in diff) in registry were created. more
5 files (not in diff) in ...istry/storage/driver were modified. more

Misses +812
Partials -94
Hits -722

10 files (not in diff) in registry/storage were modified. more

Misses +4
Partials +5
Hits -13

2 files (not in diff) in registry were modified. more

Misses +9
Partials +1
Hits -10

2 files (not in diff) in registry/storage were deleted. more
File ...a1/config_builder.go (not in diff) was modified. more

Misses 0
Partials 0
Hits -2

@@             master      #1674   diff @@
==========================================
  Files           120        123      +3
  Lines         10489      10773    +284
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           6469       5797    -672
- Misses         3423       4472   +1049
+ Partials        597        504     -93

Powered by Codecov. Last updated by [5d08dfa...d68178a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/docker/distribution/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/docker/distribution/compare/5d08dfa70c978033fdd2d8aa8db87d6bcd4aaad7...d68178aad4a40c324f21243b609ff4a6afeac002
[cc-pull]: https://codecov.io/gh/docker/distribution/pull/1674?src=pr

Thanks for addressing the feedback!

Not quite sure what happened here. @SvenDowideit: was this intentionally closed?

@aaronlehmann I'll reopen with a squashed series of commits


Middleware Tests

Missing

Codecov Report

Merging #9 (148f4e1) into main (064a9b9) will decrease coverage by 14.28%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##              main       #9       +/-   ##
============================================
- Coverage   100.00%   85.71%   -14.29%
============================================
  Files            1        2        +1
  Lines           26       63       +37
============================================
+ Hits            26       54       +28
- Misses           0        6        +6
- Partials         0        3        +3

Impacted Files
Coverage Δ

middleware/logger.go
75.67% <0.00%> (ø)

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 064a9b9...148f4e1. Read the comment docs.


Jhaessler amba oracle

Overview/Summary
added Oracle on Azure IaaS baseline alerts.
This PR fixes/adds/changes/removes
added oracle on azure iaas baseline alerts
Breaking Changes

Replace me
Replace me

As part of this Pull Request I have

[ x] Read the Contribution Guide and ensured this PR is compliant with the guide
[x ] Checked for duplicate Pull Requests
[x ] Associated it with relevant GitHub Issues or ADO Work Items (Internal Only)
[ x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x ] Ensured PR tests are passing
[x ] Updated relevant and associated documentation (e.g. Contribution Guide, Docs etc.)

#assign-reviewer:kisshetty

@jessiehaessler, please fix Lint errors.  Looks like line 35 in oracle on azure iaas/_index.md is missing a '|' at the end.

@judyer28 fixed the lint issues. Ready to go now.


Error With Xcode 14

now use Xcode14.
when use -> Install the runner with brew install emergetools/homebrew-tap/ettrace

The command line tools need to be installed, it looks like this message is coming from homebred not ettrace. Could you try running xcode-select -p and make sure it's pointing to the latest version of Xcode?

Closing this since it appears to be unrelated to ETTrace, let us know if you're still experiencing the issue!


Twitch Clone

Description
Twitch.com clone
Code of Conduct

[X] I follow Contributing Guidelines of this project.

/assign


chore: Update Dialogflow agent display name

This is to test #8676's IT issues without running the full suite of ITs.
Issue opened: #8681

It seems that the agent got renamed or replaced + recreated
https://dialogflow.cloud.google.com/#/editAgent/gcloud-devel/:

It seems that the setUp()/ tearDown() logic for these ITs doesn't create/ destroy an agent. I'm guessing this might be due to the fact that it's either expensive or extremely slow.
We're currently relying on this one agent to be there and unchanged for the ITs: https://github.com/googleapis/google-cloud-java/blob/b2a5f7207e9f9b2f933e75ef4fd44973a234326f/java-dialogflow/google-cloud-dialogflow/src/test/java/com/google/cloud/dialogflow/v2/it/ITSystemTest.java#L98
Perhaps this is something that we can continue to look into as part of the issue.

@meltsufin Do you know why it's adding in the .gitignore file again? I believe you recently made the change to exclude the file: https://github.com/googleapis/google-cloud-java/blob/b2a5f7207e9f9b2f933e75ef4fd44973a234326f/java-dialogflow/owlbot.py#L106

"generation diff / owlbot-py (pull_request)" is failing.

OwlBot is adding the gitignore back somehow. https://screenshot.googleplex.com/BmRid3PQPwrMcw7

Created an issue to follow up https://github.com/googleapis/google-cloud-java/issues/8683 I cannot troubleshoot it right now.

No worries! Thanks for the context @burkedavison! I could have sworn DialogFlow's IT suite was passing before and I had no idea how it started to fail.
Personally, I think it makes more sense for the test to create/ delete the agent anyways (unless it takes way too long). I can take a look in a bit to see how feasible it is. If not, I can just rename the agent to what it was before and delete this PR. We also have the owlbot issue #8683 to investigate.

Actually on second thought, I'm not sure how I could make creation/ deletion work for parallel invocations of ITs. Given the quota limitations, it would make sense to only have one agent. Does anyone have any objections if I rename this agent back to google-cloud-java-tests?
@burkedavison @suztomo @meltsufin

@lqiu96 No objections. Go ahead with the rename.

Does "owlbot:ignore" label work? Trying.

I've renamed the agent back to google-cloud-java-tests. Going to create a new PR to fix: modified_files=$(git diff --name-only "${KOKORO_GITHUB_PULL_REQUEST_TARGET_BRANCH}...${KOKORO_GITHUB_PULL_REQUEST_COMMIT}").
Closing this PR. We can address the issue separately.


Add dokuwiki

What this PR does / why we need it:
Add DokuWiki, an open source wiki that doesn't require a database

This was a test PR to see if squashing could be done in the PR. It can't. Closing.
Nice to know linting passed, though.

I can squash it when I merge...

Squashed in #434
Thanks for an awesome project. I'm learning a lot with the added bonus of reproducible deployment.


Support for Xiaomi Vacuum Robot S12

Hi,
Just bought a Xiaomi Vacuum Robot S12 and there seems to be unsupported.
Is it possible to add support for it?
Thank you in advance.

No, it's not.
But there seems to be support for RoboRock's older models.
The model is b106eu.
The official integration doesn't support either and I had hope you could help.


(maint) Fix typo in options hash

The :executor option in the concurrent promise instantiation had a typo that resulted in non-parallel download resulting in non-serial execution of download_file.

This was easy to overlook when only a single task is downloaded. It also got passed some manual testing when the cache was being populated as failed attempts (for fixing a different, LOUDER bug) succeeded in eventually downloading the files and thus masking the error.


Make the PDPI translation APIs have a consistent TranslationOptions p…

sonic-buildimage/src/sonic-p4rt/sonic-pins$ ~/Tools/keyword_checks.sh .
Keyword check Passed.
sonic/src/sonic-p4rt/sonic-pins$ bazel build $BAZEL_BUILD_OPTS ...
INFO: Analyzed 408 targets (0 packages loaded, 0 targets configured).
INFO: Found 408 targets...
INFO: From Compiling lib/basic_traffic/basic_traffic.cc:
lib/basic_traffic/basic_traffic.cc: In function 'absl::lts_20230802::StatusOr<absl::lts_20230802::flat_hash_map<std::__cxx11::basic_string, pins_test::basic_traffic::{anonymous}::PrecomputedPacket> > pins_test::basic_traffic::{anonymous}::PrecomputePackets(const absl::lts_20230802::flat_hash_map<std::__cxx11::basic_string, std::__cxx11::basic_string >&, const absl::lts_20230802::flat_hash_map<std::__cxx11::basic_string, thinkit::InterfaceInfo>&, absl::lts_20230802::Span, absl::lts_20230802::Span)':
lib/basic_traffic/basic_traffic.cc:125:39: warning: comparison of integer expressions of different signedness: 'int' and 'absl::lts_20230802::Span::size_type' {aka 'long unsigned int'} [-Wsign-compare]
p4_fuzzer/oracle_util_test.cc:37:13: warning: 'p4_fuzzer::SwitchState p4_fuzzer::{anonymous}::EmptyState()' defined but not used [-Wunused-function]
37 | SwitchState EmptyState() {
|             ^~~~~~~~~~
p4_fuzzer/oracle_util_test.cc:29:5: warning: 'int p4_fuzzer::{anonymous}::AclIngressTableSize()' defined but not used [-Wunused-function]
29 | int AclIngressTableSize() {
|     ^~~~~~~~~~~~~~~~~~~
INFO: Elapsed time: 212.165s, Critical Path: 28.24s
INFO: 96 processes: 5 internal, 91 linux-sandbox.
INFO: Build completed successfully, 96 total actions
sonic/src/sonic-p4rt/sonic-pins$ bazel test $BAZEL_BUILD_OPTS ...
INFO: Analyzed 408 targets (0 packages loaded, 0 targets configured).
INFO: Found 261 targets and 147 test targets...
INFO: Elapsed time: 106.427s, Critical Path: 25.80s
INFO: 88 processes: 85 linux-sandbox, 18 local.
INFO: Build completed successfully, 88 total actions
//gutil:collections_test                                        (cached) PASSED in 0.5s
//gutil:io_test                                                 (cached) PASSED in 0.5s
//sai_p4/instantiations/google/tests:p4_constraints_integration_test     PASSED in 0.8s
//sai_p4/tools:packetio_tools_test                                       PASSED in 0.7s
//sai_p4/instantiations/google/test_tools:table_entry_generator_test     PASSED in 23.0s
Stats over 5 runs: max = 23.0s, min = 1.0s, avg = 5.9s, dev = 8.6s
Executed 83 out of 147 tests: 147 tests pass.
INFO: Build completed successfully, 88 total actions

Co-authored-by: Srikishen Pondicherry Shanmugam<EMAIL_ADDRESS>


Large roi annotations to masks

Allow get_all_rois_from_slide() to handle very large ROIs by optionally  tiling them .. a simple function parameter controls this behavior.
This should be merged after PR #883 .

@Leengit Thanks. No, the increase does not affect any of the existing software .. it's only meant to slightly relax the threshold for throwing errors, which was too stringent when I tried smaller magnifications for some of my current work.

@Leengit Thanks. No, the increase does not affect any of the existing software .. it's only meant to slightly relax the threshold for throwing errors, which was too stringent when I tried smaller magnifications for some of my current work.

@Leengit OK, I think this is ready to merge. Like PR #883 (which is a subset of this one), the tests pass locally, and it seems that the errors are from some circleci issue and arise from tests/test_cli_common.py and tests/test_feature_extraction.py .. I didn't change either of those files or any of their dependencies, so I assume this is an old error.

@Leengit OK, I think this is ready to merge. Like PR #883 (which is a subset of this one), the tests pass locally, and it seems that the errors are from some circleci issue and arise from tests/test_cli_common.py and tests/test_feature_extraction.py .. I didn't change either of those files or any of their dependencies, so I assume this is an old error.

I do not know anything about those circleci errors and am not confident enough to ignore them.  Let's ask @manthey.

I do not know anything about those circleci errors and am not confident enough to ignore them.  Let's ask @manthey.

@kheffah I've merged in the removal of Python < 3.6 rather than add code to accommodate those versions.  Aside from one comment to simplify the code, this looks good to me.

@kheffah I've merged in the removal of Python < 3.6 rather than add code to accommodate those versions.  Aside from one comment to simplify the code, this looks good to me.

@kheffah I've merged in the removal of Python < 3.6 rather than add code to accommodate those versions. Aside from one comment to simplify the code, this looks good to me.

@manthey Thank you! OK I've incorporate the correction.

@kheffah I've merged in the removal of Python < 3.6 rather than add code to accommodate those versions. Aside from one comment to simplify the code, this looks good to me.

@manthey Thank you! OK I've incorporate the correction.


Look into what's wrong with the ranking with https://codaprotocol.com/memberProfile?allTimeRank=753&allTimePoints=1&phaseRank=717&phasePoints=2&releaseRank=322&releasePoints=1&genesisMember=false&name=Hekit%235820 . There is mismatch between Google Sheet and Web page

Unable to reproduce.  Closing.


Report issues

[x] BuildingID 700 is generating a completely blank Building Report.
[x] The Building Reports uniformly seem to be displaying text that say Comments 0.
[x] The Find Financial Incentives button is showing the text of the link below the button itself.

┆Issue is synchronized with this Asana task

Comments: seems to be related to the fact that the 2022 data uniformly has comment = 0 and previous years had this as blank. I haven't pinpointed where this comes in but there seems to be the option to print a comment on a report. If no comments are desired, then comment should be blank/null
Find Financial Incentives. According to issue #56, this is how this has worked, since before we came on to the project. The main change we made last year was to support a mailto link, and if mailto, not to show that href

The issue with building 700, and all buildings with property_type of "Supermarket / Grocery Store" is that only 2021-22 data have this property_type. Previously it is always "Supermarket / Grocery", and that is the key defined in seattle.json

Actions in CARTO:

Change "0" to "" for comments
Change all Supermarket / Grocery Store to Supermarket / Grocery, because this requires no code change

On the next data update, we'll want to do the reverse, which is update all Supermarket / Grocery to Supermarket / Grocery Store, and update seattle.json to match

Keeping this open in order to keep active the last item in the previous comment:

On the next data update, we'll want to do the reverse, which is update all Supermarket / Grocery to Supermarket / Grocery Store, and update seattle.json to match

The current EPA data has Supermarket / Grocery Store, so to reflect that, do the following:

[ ] Update seattle.json to key Supermarket / Grocery Store
[ ] In CARTO change all previous years data from Supermarket / Grocery to Supermarket / Grocery Store
[ ] See if that change can be applied upstream, so future data updates don't have this issue

Following up on this, the latest 2015-2023 data has all of these keyed as Supermarket / Grocery Store, so really the only TODO is the first:

[ ] Update seattle.json to key Supermarket / Grocery Store


gex: 0.4.0 -> 0.5.0

Description of changes
simple update
Things done
used gex 0.5.0 to stage and commit this very PR :)

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

ping @Azd325 @Br1ght0ne

cc @Piturnah also

Result of nixpkgs-review pr 242765 run on aarch64-darwin 1

1 package built:

gex


Implement queue_event move assignment via static move/dtor_impl

Addresses issue described in https://github.com/boost-experimental/sml/issues/279

Added a regression test for the issue I saw - it's a little verbose because the issue only manifested when the defer event was being destroyed from the back half of the deque.
Verified that the test fails in master, and passes after this fix:
Before fix:
Test project /Users/plarson/upstream-sml/sml/BUILD
Constructing a list of tests
Done constructing a list of tests
Updating test list for fixtures
Added 0 tests to meet fixture requirements
Checking test dependency graph...
Checking test dependency graph end
test 20
Start 20: test_actions_defer
20: Test command: /Users/plarson/upstream-sml/sml/BUILD/test/ft/test_actions_defer
20: Test timeout computed to be: 1500
20: /Users/plarson/upstream-sml/sml/test/ft/actions_defer.cpp:289:e1data3.use_count() == 1
1/1 Test #20: test_actions_defer ...............***Failed    0.00 sec
0% tests passed, 1 tests failed out of 1
Total Test time (real) =   0.01 sec
The following tests FAILED:
20 - test_actions_defer (Failed)
Errors while running CTest
After fix:
Test project /Users/plarson/upstream-sml/sml/BUILD
Constructing a list of tests
Done constructing a list of tests
Updating test list for fixtures
Added 0 tests to meet fixture requirements
Checking test dependency graph...
Checking test dependency graph end
test 20
Start 20: test_actions_defer
20: Test command: /Users/plarson/upstream-sml/sml/BUILD/test/ft/test_actions_defer
20: Test timeout computed to be: 1500
1/1 Test #20: test_actions_defer ...............   Passed    0.01 sec
The following tests passed:
test_actions_defer
100% tests passed, 0 tests failed out of 1


Update accidental thread keywords (#296)

Add !close to the default list of accidental thread keywords

@dragory actually, does this support wildcards? Just occurred to me that this will only catch the default prefix.


Custom Keywords Meta Schema

Do you have a meta schema that includes your custom keywords by any chance?
I would like to validate schemas themselves and intend to allow your custom keyword extensions when creating schemas.
Essentially I am looking for the equivalent to this meta schema: http://json-schema.org/draft-07/schema
Thanks for the excellent library!

Keywords definitions include fragments of meta-schemas for each one - they can be used in any combinations. Also, extending meta-schemas is a difficult in JSON-Schema (see how hyperschema metaschema looks for example).


Provider Workers fail to deploy on podified

After creating an EMS the orchestrator fails to create the provider worker deployments due to a data type mismatch in the environment variables for the EMS_ID:
MIQ(ManageIQ::Providers::Vmware::InfraManager::RefreshWorker.sync_workers) Workers are being synchronized: Current: [], Desired: [\"ems_4\"]
MIQ(MiqServer::WorkerManagement::Kubernetes#sync_workers) Failed to sync_workers for class: ManageIQ::Providers::Vmware::InfraManager::RefreshWorker: HTTP status code 400, Deployment in version \"v1\" cannot be handled as a Deployment: json: cannot unmarshal number into Go struct field EnvVar.spec.template.spec.containers.env.value of type string for POST https://<IP_ADDRESS>/apis/apps/v1/namespaces/manageiq/deployments"}
{"@timestamp":"2024-09-25T19:11:25.470533","hostname":"orchestrator-54949864-xcmdg","pid":7,"tid":"972c","service":"evm","level":"err","message":"[Kubeclient::HttpError]: Deployment in version \"v1\" cannot be handled as a Deployment: json: cannot unmarshal number into Go struct field EnvVar.spec.template.spec.containers.env.value of type string  Method:[block (2 levels) in <class:LogProxy>]"}
{"@timestamp":"2024-09-25T19:11:25.470561","hostname":"orchestrator-54949864-xcmdg","pid":7,"tid":"972c","service":"evm","level":"err","message":"/opt/manageiq/manageiq-gemset/gems/kubeclient-4.12.0/lib/kubeclient/common.rb:130:in `rescue in handle_exception'
/opt/manageiq/manageiq-gemset/gems/kubeclient-4.12.0/lib/kubeclient/common.rb:120:in `handle_exception'
/opt/manageiq/manageiq-gemset/gems/kubeclient-4.12.0/lib/kubeclient/common.rb:406:in `create_entity'
/opt/manageiq/manageiq-gemset/gems/kubeclient-4.12.0/lib/kubeclient/common.rb:253:in `block (2 levels) in define_entity_methods'
/opt/manageiq/manageiq-gemset/gems/kubeclient-4.12.0/lib/kubeclient/common.rb:101:in `method_missing'
/var/www/miq/vmdb/lib/container_orchestrator.rb:27:in `create_deployment'
/var/www/miq/vmdb/app/models/miq_worker/deployment_per_worker.rb:6:in `create_container_objects'
/var/www/miq/vmdb/app/models/miq_worker.rb:355:in `start_runner_via_container'
/var/www/miq/vmdb/app/models/miq_worker.rb:348:in `start_runner'
/var/www/miq/vmdb/app/models/miq_worker.rb:393:in `start'
/var/www/miq/vmdb/app/models/miq_worker.rb:314:in `start_worker'
/var/www/miq/vmdb/app/models/mixins/per_ems_worker_mixin.rb:57:in `start_worker_for_ems'
/var/www/miq/vmdb/app/models/mixins/per_ems_worker_mixin.rb:34:in `block in sync_workers'
/var/www/miq/vmdb/app/models/mixins/per_ems_worker_mixin.rb:33:in `each'
/var/www/miq/vmdb/app/models/mixins/per_ems_worker_mixin.rb:33:in `sync_workers'
/var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:40:in `block in sync_workers'
/var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:39:in `each'
/var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:39:in `sync_workers'
/var/www/miq/vmdb/app/models/miq_server/worker_management/monitor.rb:24:in `monitor_workers'
/var/www/miq/vmdb/app/models/miq_server.rb:210:in `block in monitor'

I thought this was because of my environment variable refactoring here https://github.com/ManageIQ/manageiq/pull/23197/files#diff-d79db3eff027e013c5bb27b85d8c756574ebb3e9b33950c79b9aa0402ef96062R14
But I hit the same error on a pure master build: https://github.com/ManageIQ/manageiq/commit/1a21589
$ oc rsh orchestrator-54949864-xcmdg
sh-5.1$ cat /var/www/miq/vmdb/BUILD
20240925001811_1a21589

Ahh it used to be [].join(",") before https://github.com/ManageIQ/manageiq/pull/23195/files#diff-56487075e313126652963fef186a0e79517048b9f872a77695b35ea3ddbe4a17L8


Ability to set default profile (and temporary switch)

Title says it all.
Some devices are more personal, (like phones and laptops) while other devices are more shared (televisions, laptops, etc.). Ideally, it should be up to the user.

A temporary solution is to just save it to Favorites. However, this should be implemented fully.

Possible solution as pull request:
Enable auto login #160

related pr is merged - available in next build.


fix(deps): update dependency glob-parent from 5.1.2 to 6.0.2, CVE-202…

…1-35065

paulmillr
do you close pull requests without leaving a comment?

@AdelSkiDzNew yes

@paulmillr I'm also seeing this issue.

@Bartlebyy Where eyes are shut, nothing can be seen.

@Bartlebyy he answers meanly because he is a Ukrainian...

@AdelSkiDzNew I answer meanly because you can't do a simple 5 second search, which makes me waste my time.

https://github.com/paulmillr/chokidar/issues/1191


po.imshow sometimes breaks w/ multiple images to display

Breaks w/ reshaping or with handling of multiple titles.
This is a placeholder issue. Will edit and fill this out later when I get a chance.

Closing because this might be a duplicate of #72 and we can't reproduce anymore.

Closing because this might be a duplicate of #72 and we can't reproduce anymore.