FR: Add support for ethersjs v6

Is your feature request related to a problem? Please describe.
Usage of alchemy-sdk with ethers v6 does work due to updates to the provider structure

Leaving this PR open to gauge developer demand. If you are using ethers v6 and want the sdk to support it, please comment what features you're looking for from v6. Thanks!

@thebrianchen  nextjs 14 and the alchemy sdk are currently unusable without this upgrade. See:
https://github.com/ethers-io/ethers.js/issues/4469
and on this repo
https://github.com/alchemyplatform/alchemy-sdk-js/issues/400

@SohailKh Would adding the option to pass in your own connectionInfo overrides to the underlying provider based on the issue in ethers thread help?
The reason we are not considering upgrading to ethers is v6 without sufficient developer demand is because it will a breaking change the to the alchemy SDK. Currently, ethers v5 has 5x more weekly downloads than v6. If there is a particular set of features that developers require from v6 that aren't supported in v5, we will re-prioritize. Thanks!

Hello to the team and thank you for your work!!
As said above, ethers 5 is not usable with nextjs 14 ... We did the nextjs migration few months ago and there was no other way than migrating ethers from 5 to 6 in order to make them both working together.
If migration is not considered now, it might lead being stuck with a deprecated stack and make future migration even more painful...
I know ethers changed quite a lot of things between both versions (I have to go through it on my side) but it seems to be for the better... -> no more BigNumber to be used throughout the code if I have to pick one "randomly" ...

@thebrianchen The main reason I've updated my apps to ethers.js v6 is the use of the native BigInt type over the BigNumber library used in ethers.js v5. There's not really any new functionality I need from ethers.js v6 aside from that.
I realize the hesitation to update to v6 because of breaking changes but could it possibly be added an option instead? Some kind of flag when when creating your alchemy instance or something? In some of my apps I've installed ethers.js v6 alongside ethers.js v5 in a different namespace so that we can slowly migrate over (example below).
Package.json:
"ethers": "5.7.2",
"ethers-v6"<EMAIL_ADDRESS>"ethers-v6,": "link:@typechain/ethers-v6,",

Code:
import { ...ethersV5Methods } from 'ethers';
import { ...ethersV6Methods } from 'ethers-v6';

Then you can have both in the same project easily and possibly slowly start supporting it.

@SohailKh Would adding the option to pass in your own connectionInfo overrides to the underlying provider based on the issue in https://github.com/ethers-io/ethers.js/issues/4469#issuecomment-1932145334 help?

Yes. Is that possible to do?

@SohailKh @Bookcliff I've added support for the ConnectionInfo override in v3.3.1. This should you workaround the error in #400 by passing your own AlchemySettings.connectionInfoOverrides option.


Fix php-cs-fixer version to 3.13.0

See https://github.com/codeigniter4/CodeIgniter4/pull/6990

@paulbalandan Can you release this as a hotfix?

Sure.

Thank you!


refactor: search results and quick search

Will complete #185 when merged to main.
To test:

Run the app.
Ensure that the QuickSearchForm displays per updates described in #185.
Search for data sources, ensure the search page displays per updates described in #185.

@joshuagraber Seeing that weird routing behavior again leading to no results:
http://localhost:8888/search/calls/undefined/search-tokens?endpoint=quick-search&arg1=calls&arg2=chicago

@joshuagraber I reverted my base url variable but the 5173 port is still hardcoded somewhere:
http://localhost:5173/search-tokens?endpoint=quick-search&arg1=calls&arg2=chicago

@joshuagraber Not that I can see

Not that I can see

@mbodeantor I'm not sure what it might be, then. I've got nothing hardcoded to any localhost value locally, env var or otherwise.

Weird, not sure where it was getting that but a clean virtual env seemed to fix it. I'm seeing the changes except for the quick search tip "For example, you could search for..."

Weird, not sure where it was getting that but a clean virtual env seemed to fix it.

Haha, when in doubt, clear the cache.

I'm seeing the changes except for the quick search tip "For example, you could search for..."

Oops, sorry, I should have mentioned: need to run npm i from /client, as we updated design-system with those changes. Mea culpa.

Didn't seem to work for me

Hmm, I've got it locally @mbodeantor. Does your /client/package.json on this branch have "pdap-design-system": "^2.5.0",?

Yeah my package.json matches

Yeah my package.json matches

Weiiiiiird. Did you ever link the design system locally using npm link to test anything? If so, that symlink may still be active.  If pdap-design-system is in the output, run npm unlink pdap-design-system, then (maybe) rm -rf node_modules then npm ci.
If that doesn't solve it, then I'm baffled 😅.

Nope, no symlinks.

@mbodeantor Then I really don't know what might be happening here, to be honest. Could try clearing npm cache? npm cache clean (might require --force flag)?
@josh-chamberlain Can you try verifying this, as a sanity check?

Clearing the cache did it!


Jesse: Watch digital[\W_]+pro[\W_]+hackz

Jesse requests the watch of the watch_keyword digital[\W_]*+pro[\W_]*+hackz. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

digital[\W_]*+pro[\W_]*+hackz has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Changed alias gsta='git stash' to alias gsta='git stash save'.

Right now doing gsta "my commit message" prints the git stash usage instead of stashing the changes with a commit message. This pull request fixes this.

Why did you close this? I was gonna merge it right away

I usually do gsta pop ... and I couldn't do it with this change. Just realized there is gstp for that. So reopening it.

Oh, right, I hadn't thought about it. Anyway, most git stash possible commands have an alias already, so I'm merging this. Thanks!


How can Kong retry target in upstream

Summary
In the test, if the service fails to be called, we find that Kong's service can be retried. But if the target in upstream is not available, how to make Kong retry the next target like nginx.
Steps To Reproduce
1.Set up routes, services, and upstream
2.Two targets in upstream (eg: <IP_ADDRESS>:6080 and <IP_ADDRESS>:6180)
3.Make a target (eg: <IP_ADDRESS>:6180) unavailable
4.Request Kong with a certain amount of traffic, but some will fail, while nginx will not
Additional Details & Logs

Kong version 2.0.1
Kong error logs (<KONG_PREFIX>/logs/error.log)
kongtest@eb7024:/home/kongtest/smekong/work/kong_ingress/logs>grep "info/21912" *
access.log:<IP_ADDRESS> - - [04/Mar/2020:21:31:55 +0800] "POST /test/http/v1.0/echo/info/21912 HTTP/1.1" 502 77 "-" "callhttp_sbi/0201A0ABADA6853151E040C0"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21912 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21912", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21912 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21912", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21912 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21912", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21912 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21912", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21912 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21912", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21912 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21912", host: "kong.ebupt"
kongtest@eb7024:/home/kongtest/smekong/work/kong_ingress/logs>grep "info/21916" *
access.log:<IP_ADDRESS> - - [04/Mar/2020:21:31:55 +0800] "POST /test/http/v1.0/echo/info/21916 HTTP/1.1" 502 77 "-" "callhttp_sbi/0201E0ABADA6853151E040C4"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21916 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21916", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21916 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21916", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21916 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21916", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21916 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21916", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21916 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21916", host: "kong.ebupt"
error.log:2020/03/04 21:31:55 [error] 23770#0: *146252 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /test/http/v1.0/echo/info/21916 HTTP/1.1", upstream: "http://<IP_ADDRESS>:6180/test/http/v1.0/echo/info/21916", host: "kong.ebupt"
Operating system CentOS Linux release 7.6.1810

Dear contributor,
We're closing this issue as there hasn't been any update to it for a long time. If the issue is still relevant in the latest version, please feel free to reopen it. We're more than happy to revisit it again.
Please have a look at our pledge to the community for more information.
Sincerely,
Kong Gateway Team


Lara gets stuck in fall animation and dies when she lands on hard collisions statics

Lara get stuck in fall animation if she jumps on statics with hard collision. This bug is repeatable over and over.
The video says it all
https://youtu.be/a4QUQIpWlR0

I can add something since I'm experiencing this bug with balustrades too: It occurs when Lara land on a row of static, in the direction of the row. As I understand, currently, when Lara is landing on a static she is pushed forward or backward to land safely on the ground, but in this case, there is no "safe" place for Lara to land because of the row of static, so she gets stuck then dies.


feat: unit listing specialist

Description
Music year: {owa_music_year}
-Adds specialist unit listing page, query and schema
Issue(s)
Fixes #
How to test

Go to https://deploy-preview-2308--oak-web-application.netlify.thenational.academy/teachers/specialist/programmes/communication-and-language-applying-learning/units
Should be 404, check on migration oak, everything else should be the same.

Screenshots

## Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

just one thing I notice in your screen shot, shouldn't there be tabs for the other development stages?

:tada: This PR is included in version 1.383.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


chore: Upgrade controller-runtime to v0.10.1

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Updates the controller-runtime to the latest version (v0.10.1 as of this submission).
Which issue(s) this PR fixes:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

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 42.721%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
179

Relevant Lines:
419

💛  - Coveralls


Use clipboardData property of copy event to store HTML content

...and fall back to browser default on error.

What is the purpose of this rewrite? Is is solving a bug or improving functionality in some way?

With this Ctrl+V in (for example) Outlook mail editor will paste HTML, not code of HTML.
Without e.clipboardData.setData("text/html", str):

With  e.clipboardData.setData("text/html", str)::


Add basic usage testcase

Closes #1

I think this PR is more related to https://github.com/hoeg/PadDown/issues/1 . This is a great example of how to use PadDown.
For unittesting I had imagined tests of plaintexts of size: 1, BLOCKSIZE, BLOCKSIZE+1 and a longer plaintext, no need to use AES or some other cipher, the hasValidPadding function could just return some predefined values.
Also we should add a task to the .circle file to make sure the tests are run on CircleCi.
Consider if we should have a test for a PadChecker this is not implemented correctly. How do handle failure to decrypt in this case?

Don't merge this yet, the tests are failing. I'll look at it during the week.

Lets get this added to CI. I'll suggest that we use github for this and move away from the circle ci that has been started. Thoughts?

Github Actions is definitely useable, however it has poor support for removing artifacts.
Perhaps docker is also overkill for now? We could just activate a venv, build and pytest.

By removing artifacts, we run out of space and money on github fast :)

I agree that docker is overkill for now. My stand is that the artifact from a release is the code with a requirement.txt file such that users can incorporate the project into their own containers. Anyhow, we use what is easiest for us and provides us with the best feedback 🤷♂


API to change users' discord nickname

Create an API that can change the users' nicknames in the Real Dev Squad server.
Description
Create an API under the /discord-actions API which will call upon the endpoint exposed in cloudflare workers named: /guild/members
API contracts for the /guild/members

method: patch
authentication: JWT signed with rds-serverless-private-key
request body

{
    discordId: <Discord id is available in the user's data once they link their discord id>, (Remember `I` is capital in Id )
    userName: <rds username of the selected user also available in users' database>, (Remember `N` is capital in username)
}

Overview of the feature
Use case 1

users join in Real Dev Squad Server
Run the command /verify
come to my site
Accept terms and conditions
click on authorize
this will add a discord id in the users collection for the specific user
now once the discord id is added the name change API must be called automatically and change the nickname of the user in the server

Use case 2

users mark themselves OOO on my site
the API needs to be called automatically
set the nickname based on the standards defined (Will be defined in the near future)

Expectations

Follow TDD
Keep a minimum test coverage of 80%
Make sure API can not be called by users directly (Apart from superuser)

Need help? Try looking here;

Stuck with failed authentication/bad request signature response?

Look here how the auth token is set: https://github.com/Real-Dev-Squad/website-backend/blob/dead12da7ad289dd89dbf0b9105b94b1e19abb44/controllers/discordactions.js#L36

unable to make a call to the cloudflare workers?

Look here how the fetch call is made: https://github.com/Real-Dev-Squad/website-backend/blob/dead12da7ad289dd89dbf0b9105b94b1e19abb44/controllers/discordactions.js#L110

I will be taking maximum two weeks time
1 week - to do the task
1 week - for test
I will try to complete it before that.

  * Middleware to check if the user has verified themself with the discord bot.
  *  If user has not verified, then no actions based on discord bot should be allowed.
  * Note: This requires that user is authenticated hence must be called after
  * the user authentication middleware. We are calling it from within the
  * `authenticate` middleware itself to avoid explicitly adding this middleware
  * while defining routes.
  *
  * @param {Object} req - Express request object
  * @param {Object} res - Express response object
  * @param {Function} next - Express middleware function
  * @returns {Object} - Returns unauthorized object if user has been restricted.
  */
 const checkIsVerifiedDiscord = async (req, res, next) => {
   const { discordId, roles } = req.userData; ```
   imp point


Add support for header union stack

Flatten header union stack variables into its individual elements. All occurrences of the header union stack variables are replaced by the elements in the stack.
For ex:
header_union U {
    Hdr1 h1;
    Hdr2 h2;
}

struct Headers {
    Hdr1 h1;
    U[2] u;
}

is replaced by
struct  Headers {
    Hdr1 h1;
    U u0;
    U u1;
}

References to u[0] is replaced by u0. Likewise for all stack elements.

Closing as upstreamed


'database is destroyed' error while using 'withIdPrefix'

If I store a reference to a prefixed store I want to use throughout my application in its own file which I require from others, I get an database is destroyed-error after signing in or out.
I tried some things, but had no luck so far, for example using a new reference in every file with const prefixedStore = hoodie.store.withIdPrefix() get’s me a possible memory leak detected...-warning after using the application for a while.
Maybe I am misusing the method here, would appreciate some guidance then on how to improve :)
I created a test repo which should reflect both failure cases: flootr/hoodie-withidprefix-bug-test-case

Thanks @flootr! I didn’t yet figure out the problem but just wanted to let you know that I was able to reproduce it thanks to your repository, very much appreciated! This is on top of my Hoodie-things now, I’ll keep you posted. Mondays are usually rather busy, but hope to get to it on Tuesday or Wednesday latest

I created a test case for the bug you found:
https://github.com/hoodiehq/hoodie-store-client/pull/150
It most likely is a problem with the reset method. We have another issue open to clean up its implementation: #97. It looks like the reference to the PouchDB instance is not updated on the scoped store APIs (returned by .withIdPrefix()) when we call store.reset()
That’s how far I got so far

I tried to find a simple temporary workaround for this but it’s complicated :tm: becaues of who the implementation is split up between @hoodie/store-client and pouchdb-hoodie-api.
I’m tempted to deprecate pouchdb-hoodie-api altogether and move all the implementations of methods like .add, .update etc over to @hoodie/store-client. The only thing holding me back is that we use pouchdb-hoodie-api in @hoodie/store-server-api.
But we might simply add an option to @hoodie/store-client which would connect to a remote database directly and would not expose any of the sync APIs, as it’s something that we’ll need in the client API anyway.
So far my thoughts, sorry this takes so long. I hope it’s not blocking you

So far my thoughts, sorry this takes so long. I hope it’s not blocking you

Not at all 👍  I'm very grateful for what you're doing.
Right now it sounds like a bigger, more complicated change to me but if you think I can help at some point, ping me.

hey @flootr I prepared an issue with lots of context, would you like to give it a go yourself? https://github.com/hoodiehq/camp/issues/121

@gr2m I saw you already claimed that issue which is really cool and I'm really looking forward to your solution 👍 🎉 😎

yeah sorry! I had a flight and that was perfect to make it a productive one. I’m ~50% through already

@flootr could you test it with this PR? https://github.com/hoodiehq/hoodie-store-client/pull/153
If you could review it, that’d be great, too :) Ask away all the questions you might have about it 👍

Yeah, of course 👍

@gr2m I wasn't able to reproduce the error by updating @hoodie/store-client in the bug-test-repo mentioned above 🎉


new release bumping bitflags dependency

the last release is a while back and bitflags has seen a major version bump. dependencies like ron not being released with a version that supports building with 2.x of bitlags keeps downstream dependencies from moving on. luckily its already on master and we only need a release. pretty please :)

@kvark @torkleyy If you could give me the necessary rights to publish releases on crates.io, I could do so now and in the future

@juntyr any progress on getting publish rights?
@kvark @torkleyy maybe there is a way to build a CD pipeline that allows publishing?

@extrawurst Unfortunately I haven’t heard back from @torkleyy or @kvark yet. I hope I will soon :)

https://github.com/ron-rs/ron/releases/tag/v0.8.1 has now been released to https://crates.io/crates/ron/0.8.1 and https://docs.rs/ron/0.8.1

Hey @juntyr sorry for the delay, you should have write access, right?

Hey @juntyr sorry for the delay, you should have write access, right?

Yes, I got access yesterday :)
And no worries about the delay - with now a third maintainer, it’ll be a lot easier to take breaks.


Add missing update params

This adds the missing update params as discussed in #70.

@tomtaylor How does this look?

Sorry for the slow reply - this looks great. I'll get it merged and released in the next couple of days.


Zombies don't spawn

Zombies are set to spawn at all times from edge of the map, with days until set to 0. No zombies spawn.
Tried changing to soft ground, tried changing to only at nighttime, no effect. Using Zombieland <IP_ADDRESS> With RimWorld 1.1.2552 64-bit, no additional mods.
Zombies can be successfully spawned using dev mode.

Try the latest release. If it does not work, please use dev mode and look at the inspector panel (the spyglas button) and post a screenshot of it.

Still having issues, unfortunatenly

Colonists without weapons don’t count.

After equipping weapons, zombies spawn correctly. Closed! However, it is a very odd start to the game that zombies don't even spawn until you have a weapon.


unable to build on mac m1 max

Please use this template for reporting suspected bugs or requests for help.
Issue description
when build with no=binary and with bundled
run
ZMQ_PREFIX=bundled pip install --no-binary pyzmq pyzmq
shows error:
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:372:5: error: static assertion failed due to requirement 'is_same<zmq::secure_allocator_t<unsigned char>, std::allocator<unsigned char>>::value': [allocator.requirements] states that rebinding an allocator to the same type should result in the original allocator
Environment

libzmq version (commit hash if unreleased): :pyzmq-25.1.1.tar.gz
OS: macOS

The workaround is to install zeromq prior to installing pyzmq, e.g. via brew install zeromq.
This is really a libzmq issue. libzmq 4.3.4 cannot be built without libsodium on current macOS with default parameters, and fails with this error. I'm not sure what compiler flags, if any, can fix it. That's a question for libzmq folks.
The full error, building zeromq 4.3.4 (entirely without pyzmq):
export CXXFLAGS=-Wno-deprecated-declarations # avoids warnings turned into errors
./configure --without-libsodium
make V=1

gives:
Making all in doc
make[1]: Nothing to be done for `all'.
/bin/sh ./libtool  --tag=CXX   --mode=compile g++ -std=gnu++11 -DHAVE_CONFIG_H -I. -I./src           -D_DARWIN_C_SOURCE -D_REENTRANT -D_THREAD_SAFE  -fvisibility=hidden     -Werror -Wall -Wno-long-long -Wno-deprecated-declarations -Wno-atomic-alignment -Wno-tautological-constant-compare -MT src/libzmq_la-ctx.lo -MD -MP -MF src/.deps/libzmq_la-ctx.Tpo -c -o src/libzmq_la-ctx.lo `test -f 'src/ctx.cpp' || echo './'`src/ctx.cpp
libtool: compile:  g++ -std=gnu++11 -DHAVE_CONFIG_H -I. -I./src -D_DARWIN_C_SOURCE -D_REENTRANT -D_THREAD_SAFE -fvisibility=hidden -Werror -Wall -Wno-long-long -Wno-deprecated-declarations -Wno-atomic-alignment -Wno-tautological-constant-compare -MT src/libzmq_la-ctx.lo -MD -MP -MF src/.deps/libzmq_la-ctx.Tpo -c src/ctx.cpp  -fno-common -DPIC -o src/.libs/libzmq_la-ctx.o
libtool: compile:  g++ -std=gnu++11 -DHAVE_CONFIG_H -I. -I./src -D_DARWIN_C_SOURCE -D_REENTRANT -D_THREAD_SAFE -fvisibility=hidden -Werror -Wall -Wno-long-long -Wno-deprecated-declarations -Wno-atomic-alignment -Wno-tautological-constant-compare -MT src/libzmq_la-ctx.lo -MD -MP -MF src/.deps/libzmq_la-ctx.Tpo -c src/ctx.cpp -o src/libzmq_la-ctx.o >/dev/null 2>&1
mv -f src/.deps/libzmq_la-ctx.Tpo src/.deps/libzmq_la-ctx.Plo
/bin/sh ./libtool  --tag=CXX   --mode=compile g++ -std=gnu++11 -DHAVE_CONFIG_H -I. -I./src           -D_DARWIN_C_SOURCE -D_REENTRANT -D_THREAD_SAFE  -fvisibility=hidden     -Werror -Wall -Wno-long-long -Wno-deprecated-declarations -Wno-atomic-alignment -Wno-tautological-constant-compare -MT src/libzmq_la-curve_client.lo -MD -MP -MF src/.deps/libzmq_la-curve_client.Tpo -c -o src/libzmq_la-curve_client.lo `test -f 'src/curve_client.cpp' || echo './'`src/curve_client.cpp
libtool: compile:  g++ -std=gnu++11 -DHAVE_CONFIG_H -I. -I./src -D_DARWIN_C_SOURCE -D_REENTRANT -D_THREAD_SAFE -fvisibility=hidden -Werror -Wall -Wno-long-long -Wno-deprecated-declarations -Wno-atomic-alignment -Wno-tautological-constant-compare -MT src/libzmq_la-curve_client.lo -MD -MP -MF src/.deps/libzmq_la-curve_client.Tpo -c src/curve_client.cpp  -fno-common -DPIC -o src/.libs/libzmq_la-curve_client.o
In file included from src/curve_client.cpp:35:
In file included from ./src/msg.hpp:40:
In file included from ./src/metadata.hpp:33:
In file included from /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/map:2356:
In file included from /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/functional:515:
In file included from /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/__functional/boyer_moore_searcher.h:26:
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:372:5: error: static assertion failed due to requirement 'is_same<zmq::secure_allocator_t<unsigned char>, std::allocator<unsigned char>>::value': [allocator.requirements] states that rebinding an allocator to the same type should result in the original allocator
    static_assert(is_same<allocator_type, __rebind_alloc<__alloc_traits, value_type> >::value,
    ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:64:60: note: in instantiation of template class 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>' requested here
        std::vector<uint8_t, secure_allocator_t<uint8_t> > hello_plaintext (
                                                           ^
./src/curve_client_tools.hpp:110:60: error: no matching constructor for initialization of 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>')
        std::vector<uint8_t, secure_allocator_t<uint8_t> > welcome_plaintext (
                                                           ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'int' to 'const std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>' for 1st argument
class _LIBCPP_TEMPLATE_VIS vector
                           ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit move constructor) not viable: no known conversion from 'int' to 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>' for 1st argument
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit default constructor) not viable: requires 0 arguments, but 1 was provided
In file included from src/curve_client.cpp:38:
In file included from ./src/curve_client.hpp:37:
./src/curve_client_tools.hpp:121:53: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        int rc = crypto_box_open (&welcome_plaintext[0], welcome_box,
                                   ~~~~~~~~~~~~~~~~~^~
./src/curve_client_tools.hpp:129:47: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        memcpy (cn_server_, &welcome_plaintext[crypto_box_ZEROBYTES], 32);
                             ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:130:47: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        memcpy (cn_cookie_, &welcome_plaintext[crypto_box_ZEROBYTES + 32],
                             ~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:154:60: error: no matching constructor for initialization of 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>')
        std::vector<uint8_t, secure_allocator_t<uint8_t> > vouch_plaintext (
                                                           ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'int' to 'const std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>' for 1st argument
class _LIBCPP_TEMPLATE_VIS vector
                           ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit move constructor) not viable: no known conversion from 'int' to 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>' for 1st argument
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit default constructor) not viable: requires 0 arguments, but 1 was provided
In file included from src/curve_client.cpp:38:
In file included from ./src/curve_client.hpp:37:
./src/curve_client_tools.hpp:159:36: error: no member named 'begin' in 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>'
        std::fill (vouch_plaintext.begin (),
                   ~~~~~~~~~~~~~~~ ^
./src/curve_client_tools.hpp:160:36: error: no member named 'begin' in 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>'
                   vouch_plaintext.begin () + crypto_box_ZEROBYTES, 0);
                   ~~~~~~~~~~~~~~~ ^
./src/curve_client_tools.hpp:161:33: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        memcpy (&vouch_plaintext[crypto_box_ZEROBYTES], cn_public_, 32);
                 ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:162:33: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        memcpy (&vouch_plaintext[crypto_box_ZEROBYTES + 32], server_key_, 32);
                 ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:169:50: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
          crypto_box (vouch_box, &vouch_plaintext[0], vouch_plaintext.size (),
                                  ~~~~~~~~~~~~~~~^~
./src/curve_client_tools.hpp:169:71: error: no member named 'size' in 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>'
          crypto_box (vouch_box, &vouch_plaintext[0], vouch_plaintext.size (),
                                                      ~~~~~~~~~~~~~~~ ^
./src/curve_client_tools.hpp:177:60: error: no matching constructor for initialization of 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>')
        std::vector<uint8_t, secure_allocator_t<uint8_t> > initiate_plaintext (
                                                           ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'size_t' (aka 'unsigned long') to 'const std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>' for 1st argument
class _LIBCPP_TEMPLATE_VIS vector
                           ^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit move constructor) not viable: no known conversion from 'size_t' (aka 'unsigned long') to 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>' for 1st argument
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/c++/v1/vector:348:28: note: candidate constructor (the implicit default constructor) not viable: requires 0 arguments, but 1 was provided
In file included from src/curve_client.cpp:38:
In file included from ./src/curve_client.hpp:37:
./src/curve_client_tools.hpp:181:39: error: no member named 'begin' in 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>'
        std::fill (initiate_plaintext.begin (),
                   ~~~~~~~~~~~~~~~~~~ ^
./src/curve_client_tools.hpp:182:39: error: no member named 'begin' in 'std::vector<unsigned char, zmq::secure_allocator_t<unsigned char>>'
                   initiate_plaintext.begin () + crypto_box_ZEROBYTES, 0);
                   ~~~~~~~~~~~~~~~~~~ ^
./src/curve_client_tools.hpp:183:36: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        memcpy (&initiate_plaintext[crypto_box_ZEROBYTES], public_key_, 32);
                 ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:184:36: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        memcpy (&initiate_plaintext[crypto_box_ZEROBYTES + 32], vouch_nonce + 8,
                 ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:186:36: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
        memcpy (&initiate_plaintext[crypto_box_ZEROBYTES + 48],
                 ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~
./src/curve_client_tools.hpp:189:40: error: type 'std::vector<uint8_t, secure_allocator_t<uint8_t>>' (aka 'vector<unsigned char, secure_allocator_t<unsigned char>>') does not provide a subscript operator
            memcpy (&initiate_plaintext[crypto_box_ZEROBYTES + 48 + 80],
                     ~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make[1]: *** [src/libzmq_la-curve_client.lo] Error 1
make: *** [all-recursive] Error 1

I think libzmq 4.3.5 won't have this issue simply because the affected feature has been removed. This, in turn, is part of what's blocking pyzmq's upgrade to 4.3.5, since it's a major breaking change in pyzmq distributions to lose CURVE support. So I have to figure out how to bundle libsodium and replace the abandoned Windows builds of libzmq, which don't exist for 4.3.5 before pyzmq can update the bundled libzmq. So I think this will be closed by #1903.

if I use brew install zeromq first ,will ZMQ_PREFIX=bundled works if I want to build an extension?

Ah, no, it should link to the installed libzmq (omit ZMQ_PREFIX=bundled, or it will ignore the installed libzmq). But that's usually ought to be a good thing. If you want to repack it for redistribution, use delocate, which will bring the linked libzmq into the wheel. This will ~always produce better results than the bundled libzmq, which is used only as a fallback when the a desirable 'real' build of libzmq is unavailable.

I am also seeing this error on fedora 38
✘ 11:27:28 [yennefer] $ g++ -std=c++17 -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -fPIC -DZMQ_HAVE_CURVE=1 -DZMQ_USE_TWEETNACL=1 -DZMQ_USE_EPOLL=1 -DZMQ_IOTHREADS_USE_EPOLL=1 -DZMQ_POLL_BA
SED_ON_POLL=1 -Ibundled/zeromq/include -Ibundled -I/home/tcaswell/.virtualenvs/bleeding/include -I/home/tcaswell/.pybuild/bleeding/include/python3.11 -c bundled/zeromq/src/curve_client.cpp -
o build/temp.linux-x86_64-cpython-311/bundled/zeromq/src/curve_client.o
In file included from /usr/include/c++/13/ext/alloc_traits.h:34,
                 from /usr/include/c++/13/bits/stl_tree.h:67,
                 from /usr/include/c++/13/map:62,
                 from bundled/zeromq/src/metadata.hpp:33,
                 from bundled/zeromq/src/msg.hpp:40,
                 from bundled/zeromq/src/curve_client.cpp:35:
/usr/include/c++/13/bits/alloc_traits.h: In instantiation of ‘struct std::__allocator_traits_base::__rebind<zmq::secure_allocator_t<unsigned char>, unsigned char, void>’:
/usr/include/c++/13/bits/alloc_traits.h:94:11:   required by substitution of ‘template<class _Alloc, class _Up> using std::__alloc_rebind = typename std::__allocator_traits_base::__rebind<_Alloc, _Up>::type [with _Alloc = zmq::secure_allocator_t<unsigned char>; _Up = unsigned char]’
/usr/include/c++/13/bits/alloc_traits.h:228:8:   required by substitution of ‘template<class _Alloc> template<class _Tp> using std::allocator_traits< <template-parameter-1-1> >::rebind_alloc = std::__alloc_rebind<_Alloc, _Tp> [with _Tp = unsigned char; _Alloc = zmq::secure_allocator_t<unsigned char>]’
/usr/include/c++/13/ext/alloc_traits.h:126:65:   required from ‘struct __gnu_cxx::__alloc_traits<zmq::secure_allocator_t<unsigned char>, unsigned char>::rebind<unsigned char>’
/usr/include/c++/13/bits/stl_vector.h:88:21:   required from ‘struct std::_Vector_base<unsigned char, zmq::secure_allocator_t<unsigned char> >’
/usr/include/c++/13/bits/stl_vector.h:425:11:   required from ‘class std::vector<unsigned char, zmq::secure_allocator_t<unsigned char> >’
bundled/zeromq/src/curve_client_tools.hpp:64:76:   required from here
/usr/include/c++/13/bits/alloc_traits.h:70:31: error: static assertion failed: allocator_traits<A>::rebind_alloc<A::value_type> must be A
   70 |                         _Tp>::value,
      |                               ^~~~~
/usr/include/c++/13/bits/alloc_traits.h:70:31: note: ‘std::integral_constant<bool, false>::value’ evaluates to false

Confirm that installing zeromq-devel avoids the problem.
$ gcc --version
gcc (GCC) 13.2.1 20231011 (Red Hat 13.2.1-4)
Copyright (C) 2023 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

I dread it a bit, but I feel like this and the removal of tweetnacl from libzmq is going to force me to rewrite bundling to include 'proper' installation of libzmq and libsodium and building via something like scikit-build or meson. I think that will be better in the long run, but figuring out the linking on Windows is going to take a lot of trial and error, and certainly reintroduce lots of build failures fixed over the years on less common platforms.
If anyone with C++ knowledge knows what's changed with recent compilers to introduce these failures and what flags and/or patches will keep us limping along while I figure that out, it would be much appreciated and take a lot of pressure off.

upstream fix appears to be https://github.com/zeromq/libzmq/pull/4480 I'll see if I can apply a patch to the bundled sources

If anyone affected can test with the sdists from this run and confirm that it fixes the issue, I can merge #1913 and make a 25.1.2 with it.

this should be fixed with last month's 25.1.2


Lost triangles in 2D Voronoi diagram

I am trying to analysize the geometrical features of triangles in 2D Voronoi diagram generated by SciPy. Although triangles can be generated , they are not recognized as polygons. In addition, the vertex coordinates, edge number, and neighbor edges of the triangle can not be extracted. Do you know how to get the geometrical information of triangles?

The Voronoi diagram of a set of points is not composed of triangles. Its dual, the Delaunay triangulation, is. scipy has routines for both, but to help you do what you want, you'll need to clarify more precisely what it is that you want to do. Can you show us what you've tried so far?

Thank you! In 2D Voronoi diagram, each seed point corresponds to a polygonal region, and each polygon contains at least three edges. I noticed that the triangle can be generated by SciPy (see attached image), but its information can not be extracted.

The regions attribute has the lists of points for each of these Voronoi cells. They are indices into the vertices attribute to get the actual coordinates. The documentation describes this more fully.
https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.Voronoi.html

I'm closing this because there's no action that we need to take. If you have more questions, you can still ask them, or join us on the mailing list (preferred for usage questions).


fix setting transmit power and channel, prevent reset hang, update documentation

This PR combines 4 changes:

methods and members in header file order by accessibility and functionality
moved method comments before declaration and improve some comments
fix not being able to set the transmit power to level 0 or change the channel via config callback, also includes a fix to assign new channel number to the member variable
improvement to prevent a indefinite loop waiting for a specific status after reset when calling reset

Thank you.


Remove comments from Persian ToC

cc @kambizG

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


added Syncalicious project

[x] Edit applications.json instead of README.md.
[x] Only one project/change is in this pull request
[x] Screenshots(s) added if any
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 1 link issue, a project collaborator will take care of these, thanks :)

Link issue by awesome_bot

Line
Status
Link

7779
404
https://raw.githubusercontent.com/AlexPerathoner/SlimHUD/master/IconFull.png

  Generated by :no_entry_sign: Danger

@skywinder Thank you! 👍


Textbox longer than browser window

Describe the bug
The min-height seems to not be used as the initial height at all.
To Reproduce
Make a textarea in a form and give it an initial value like
<textarea name="myname">
This is my initial value.

## Look, it does headers!
</textarea>

Then start easymde with
var easymde = new EasyMDE({element:document.getElementById('post'), minHeight:'50px', promptURLs:true})

Expected behavior
The editor space should be about 50px. It ends up being about 100vh, so I can't even get a screenshot of the whole editor window.
Screenshots
https://files.catbox.moe/obgfuz.png
** Version information **

OS: Linux, MacOS, Windows
Browser: Firefox, Chrome
EasyMDE version: 2.8.0

Additional context
None comes to mind.

I'll try to work up a minimal example.

    <meta content="width=device-width" name="viewport" />
    <link<EMAIL_ADDRESS>rel="stylesheet" type="text/css" />

          <form action="/" method="post">
            <textarea id="post" name="post">Look at me, "I have strings!" I'm also way too long.</textarea>
          </form>

But I was also able to remove more tags.

    <link<EMAIL_ADDRESS>rel="stylesheet" type="text/css" />

    <textarea id="p1" name="p1">Look at me, "I have strings!" I'm also way too long.</textarea>

Declare the <!DOCTYPE html> at the start of the file.

Works! Thanks.

@Vaelatern  I used from to submit theformand encountered the same problem as you
I used the same method as you and successfully solved the problem.
Thanks: .CodeMirror-scroll{height:auto}


Fix memory leak caused by File#deleteOnExit

A temporary file is created for each blob that is uploaded. This
temporary file registered itself to the DeleteOnExitHook - but since
Crate is a long lived process this hook is never called and just keeps
growing.
To fix that the delete operation of the temporary files is now coupled
to the DigestBlob & TransferStatus instances.

@seut please review

Current coverage is 81.10% (diff: 66.66%)

Merging #4490 into master will increase coverage by 0.08%

@@             master      #4490   diff @@
==========================================
  Files          1297       1297
  Lines         44096      44102     +6
  Methods           0          0
  Messages          0          0
  Branches       7003       7004     +1
==========================================
+ Hits          35728      35770    +42
+ Misses         5679       5651    -28
+ Partials       2689       2681     -8

Powered by Codecov. Last update 3ec50b0...68e7b65


CCE Minor Summer Experience and Other Engagements approvals

We need a review and approval process for CCE Minor Summer Experience and Other Engagement applications. Once approved, these experiences should be selected for the CCE minor and visible with the other engagements on the main page

 CCE Minor Summer Experience and Other Engagements approvals


[consecutive command] Grammatical error for commands

Commands that have "-s" at the back create a grammatical error in the way the command is presented

The issue extends to also tag, where taged is also generated

I think can change it to "command :  has been ran already"


Plugin not running at all

Environment:
OSX El Capitan 10.11.6
Sketch 48.2 (latest as of now)
Symbol instance locator 1.7 (latest as of now)
When I select a layer and click Symbol Instance Locator from Plugins menu, Sketch does not do anything. It's not being slow or loading, it just does nothing.
Things I have tried

Reinstall Sketch (cache and old files included)
Remove and redownloaded the plugin
Used different design files

Has anyone experienced the same issue as me?

I downloaded and installed Sketch 48 or 48.1, still no luck.

Same problem here. Did you solve it already?

No, still same problem :(
On Tue, Mar 27, 2018, 6:12 AM Lars • upstruct<EMAIL_ADDRESS>wrote:

Same problem here. Did you solve it already?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sonburn/symbol-instance-locator/issues/5#issuecomment-376519373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFk4VT1hvdQK7_NknsfT86WoL7bHLtmWks5tijqggaJpZM4R9hmY
.

At our studio it works on a machine with Mac OS 10.12.6
(same: Sketch 48.2, Symbol instance locator 1.7)
So it seems to be a Mac OS 10.11 issue. Hmmm …

@seanoh1989 and @lars-upstruct , sorry for taking so long to respond. I don't have El Capitan installed on any device, so if someone could provide a log event, it would be helpful. There are two ways to do this... Console.app or Terminal.
The Console.app changed in Sierra, and I don't recall the exact location of the log output back in El Cap, but in Sierra the log location is ~/Library/Logs/com.bohemiancoding.sketch3. Just open Sketch, run the plugin in your document, and check Plugin Output.log in the above location. This should give us a clue as to what failed.
The Terminal method is as follows. Close Sketch, and open Terminal, and enter the following "/Applications/Sketch.app/Contents/MacOS/Sketch" sans quotes. This will open Sketch and start logging within Terminal. Run the plugin in your document, and check output in the Terminal. This should give us a clue as to what failed.

@sonburn  no problem. Thanks for getting back to us.
I ran the Sketch on Terminal to get this log
```

Mar 29 15:17:37  Symbol Instance Locator (Sketch)[35330] : SyntaxError: Unexpected token '>'.
Plugin “script”, line 88:
» 					var buttonToSelect = symbolType.findIndex(o => o == pageInstance);
2018-03-29 15:17:37.892 Sketch[35330:1037053] Exception: {
line = 88;
sourceURL = "/Users/Sean/Library/Application Support/com.bohemiancoding.sketch3/Plugins/Symbol Instance Locator.sketchplugin/Contents/Sketch/script.cocoascript";
}

@seanoh1989 Thank you, this was very helpful. I will see what I can do.

@seanoh1989 and @lars-upstruct , could you please try the updated plugin at the link below? I would like to verify it works before publishing.
https://www.dropbox.com/sh/h3nzkfnoxmg7uxq/AAD9SbhzG4VSBHltGBXX1yRGa?dl=0

@sonburn looks like getting replacing that inline function worked! I can verify the fix. Thanks a lot 👍

@seanoh1989 Happy to hear it, sorry again for taking so long to get to this. I just published this update for the masses. Thanks for bringing it to my attention, and for using the plugin!


correctly handle I64 (and other) values returned from imported functions

I was attempting to upgrade to the new wasmer version when I found that any imported functions that returned I64 caused "panic: Cannot convert value to int32". it seems it was due to always trying to convert the return value to I32. This PR resolves that issue.

Do you need a new release now?

@Hywan I can point to the commit for now, so not an issue, up to you :)

I'll make a new release probably today :-)


new commands

LmsStart  - starts lms in all gamemodes by this command
LmsEnd  -  ends lms in all gamemodes by this command
RandomMinion (count) (name) - give minions for random player

RandomMinion isn's such a good idea. Just do minion and type in a random number.

^ yes

@t11tanYT ^^
Also, having a command to start/end lms whenever you want in any gamemode kindof ruins the point of having a specific gamemode just for lms.


astrocytes are not immune cells

GO:0045321 : immune cell activation (374)
GO:0048143 : astrocyte activation (5)
Astrocytes are glial cells (only exist in the brain and
support neurons; they are not found in the blood), so
they are currently mis-classified as immune cells.
Reported by: *anonymous
Original Ticket: "geneontology/ontology-requests/1627":https://sourceforge.net/p/geneontology/ontology-requests/1627

thanks www.movied.org


Add support for new oracledb package

It looks like Oracle is changing their driver package to be (mostly) pure Python, and is changing some of the interfaces.
Here's a link to some documentation that shows all of the differences:
https://python-oracledb.readthedocs.io/en/latest/user_guide/appendix_c.html

Thanks for the issue, I’ll take a look at adding that this weekend!

@zschumacher I made a pretty basic PR that's just a copy/paste job from the cx_Oracle implementation. I'm happy to continue working on it based on any code review comments you have.

Thanks! It's been generally pretty easy so far. Had to figure out how to install extras using Poetry.
Also, switching to using the new oracledb driver as the default would be helpful for anyone getting started since it doesn't require the Instant Client to do basic database operations.

yeah the instant client is a pain for sure.  I opened #28 and will consider doing that when I 1.0


[backport] Fix testing condition of diff and unwrap

Backport of #2124

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 4e7d0ff1cf50b5c22c773ef01457f187a766fad4, target branch v6) failed with status FAILURE.


TM profile parameter "heartbeat.polling.interval" is undocumented

The docs have:

health.polling.interval: An interval in milliseconds on which to poll for health statistics

but the parameter heartbeat.polling.interval doesn't have any documentation. It would be useful to have documentation for both parameters, as it appears TM uses both of them, but it's unclear what the difference between the two parameters are. Looking at the code, it seems like heartbeat.polling.interval might be a poll for less data (load avg, cpu usage, memory usage?) as opposed to all of the astats data.
Can someone with TM expertise confirm this so we can clearly document both parameters?

New information -- this is the data that is polled by heartbeat.polling.interval:
{
  "ats": {
    "server": "7.1.4"
  },
  "system": {
    "inf.name": "bond0",
    "inf.speed": 20000,
    "proc.net.dev": "bond0:<PHONE_NUMBER>95376<PHONE_NUMBER>01    0    4    0     0          0  41936537<PHONE_NUMBER>185921<PHONE_NUMBER>466    0  668    0     0       0          0",
    "proc.loadavg": "4.06 3.39 3.57 1/1021 8196",
    "configReloadRequests": 198,
    "lastReloadRequest":<PHONE_NUMBER>,
    "configReloads": 2,
    "lastReload":<PHONE_NUMBER>,
    "astatsLoad":<PHONE_NUMBER>,
    "something": "here"
  }
}

which would allow for determining available bandwidth and load average, whereas health.polling.interval includes that data and the much larger set of ATS astats (data that is mostly sent to InfluxDB for Traffic Stats).
So, it probably makes sense to have heartbeat.polling.interval poll more often than health.polling.interval just because of the amount of data returned by each of them, respectively.

@rawlinp - is this still an issue?

Yes.


 This problem isn't related to the code that is contained within the sample, but rather an API and speech recognition engine problem. The continous speech recognition stops suddenly and it doesn't start again. This isn't a local problem, I've tested the uwp continuous speech recognition on multiple systems with multiple architectures. I tried to test the source of the problem by using tail recursion, timers and conditional tail recursion in order to force the speech recognition engine to start again if a problem occurs, but surprisingly, still doesn't work. I know that the uwp's speech recognition engine stops if the app goes in the background and I ensured, that the test app doesn't deactivate, but the speech recognition engine still stops. The only function that works from the uwp's speech recognition engine is the non-continous speech recognition. If you'll think that I initiated the speech recognition wrong.... believe me, I didn't. I even used using statements and condional statements with which I manually
De-allocate the speech recognition engine from the memory and reinitiate it if it shutting down, it will still not stay activated.

Sorry, this repo is for the samples, not for general Windows developer support. You can follow the instructions on Windows developer support for support options. (Click "Forums and community".)


Integrate stevenblack/hosts

Hey @dhilipsiva, good work, I see some nice ideas here.
Feel free to pull source data, or amalgamated hosts collections, from https://github.com/StevenBlack/hosts.

Wow @StevenBlack , that is a nice collection. Thanks a lot for sharing. Will do once I reach my home 😄


[bug] Double-click data-tauri-drag-region to toggle "maximize" status of windows without decorations not working correctly

Describe the bug
https://user-images.githubusercontent.com/1064052/149616848-202a63ab-2daf-4a1c-92b4-397aa580da9c.mov
Reproduction

Checkout next brunch of tauri
Run API example
Click "Window" on the left sidebar
Uncheck "Has decorations" option
Double-click data-tauri-drag-region

Expected behavior
No response
Platform and versions
Operating System - Mac OS, version 11.6.1 X64

Node.js environment
  Node.js - 14.18.1
  @tauri-apps/cli - 1.0.0-beta.10
  @tauri-apps/api - 1.0.0-beta.8

Global packages
  npm - 6.14.15
  pnpm - Not installed
  yarn - 1.22.17

Rust environment
  rustc - 1.56.1
  cargo - 1.56.0

Rust environment
  rustup - 1.24.3
  rustc - 1.56.1
  cargo - 1.56.0
  toolchain - stable-x86_64-apple-darwin (environment override by RUSTUP_TOOLCHAIN)

App directory structure
/dist
/node_modules
/public
/src-tauri
/src

App
  tauri.rs - 1.0.0-beta.8
  build-type - bundle
  CSP - default-src blob: data: filesystem: ws: wss: http: https: tauri: asset: customprotocol: 'unsafe-eval' 'unsafe-inline' 'self' img-src: 'self'
  distDir - ../dist
  devPath - http://localhost:5000/
  framework - Svelte

Stack trace
No response
Additional context
No response

It should be working, can you try a simple app instead of the api example? it might be broken only in the api example.

It should be working, can you try a simple app instead of the api example? it might be broken only in the api example.

see this repo
with decorations
https://user-images.githubusercontent.com/1064052/149647674-672d0b23-e000-4f40-bc4c-ee86ceb1a046.mov
without decorations
https://user-images.githubusercontent.com/1064052/149647693-8ba61892-1262-4715-8d5d-946388286f9d.mov

thanks. we will look into this. might be an issue at Tao.

@amrbashir can you check this one again or can I help somehow? I've seen this issue too.

/upstream tauri-apps/wry

/upstream tauri-apps/wry


Modify preserveOutliers GraphQL API

Desc
change response in preserveOutliers api

Result<usize> -> Result<Vec<PreserveOutlierOutput>>

Reference
Close #50

Result<usize> -> Result<Vec<PreserveOutlierOutput>>

Could you follow the rule how to use backticks in our policy?

With backticks, I suggest this title:
Modify the preserveOutliers GraphQL API


ARROW-10030: [Rust] Add support for FromIter and IntoIter for primitive types

This is the associated draft implementation of a draft proposal on ARROW-10030, with associated document here: https://docs.google.com/document/d/1d6rV1WmvIH6uW-bcHKrYBSyPddrpXH8Q4CtVfFHtI04/edit
Benchmarks where the builder was replaced by this from for the `cast`: +5% to -25% improvement over the builder (with the same memory utilization)

cast int32 to int32 512 time:   [25.493 ns 25.498 ns 25.505 ns]
                        change: [+1.2840% +1.7037% +2.1989%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 15 outliers among 100 measurements (15.00%)
  4 (4.00%) high mild
  11 (11.00%) high severe

cast int32 to uint32 512
                        time:   [6.8259 us 6.8323 us 6.8390 us]
                        change: [+3.6216% +5.6472% +7.0263%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 8 outliers among 100 measurements (8.00%)
  1 (1.00%) high mild
  7 (7.00%) high severe

cast int32 to float32 512
                        time:   [5.7733 us 5.7763 us 5.7795 us]
                        change: [-14.264% -13.671% -13.106%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 10 outliers among 100 measurements (10.00%)
  1 (1.00%) low mild
  2 (2.00%) high mild
  7 (7.00%) high severe

cast int32 to float64 512
                        time:   [5.2627 us 5.2667 us 5.2711 us]
                        change: [-23.365% -22.746% -22.147%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 10 outliers among 100 measurements (10.00%)
  4 (4.00%) high mild
  6 (6.00%) high severe

cast int32 to int64 512 time:   [5.2307 us 5.2381 us 5.2438 us]
                        change: [-21.490% -20.984% -20.509%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high severe

cast float32 to int32 512
                        time:   [6.2755 us 6.2794 us 6.2833 us]
                        change: [-10.606% -9.9724% -9.3770%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 9 outliers among 100 measurements (9.00%)
  2 (2.00%) high mild
  7 (7.00%) high severe

cast float64 to float32 512
                        time:   [6.1900 us 6.2032 us 6.2206 us]
                        change: [-14.505% -13.963% -13.416%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 7 outliers among 100 measurements (7.00%)
  4 (4.00%) high mild
  3 (3.00%) high severe

cast float64 to uint64 512
                        time:   [6.4047 us 6.4189 us 6.4347 us]
                        change: [-12.287% -11.651% -11.050%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 2 outliers among 100 measurements (2.00%)
  1 (1.00%) high mild
  1 (1.00%) high severe

cast int64 to int32 512 time:   [6.3697 us 6.3913 us 6.4168 us]
                        change: [+3.5998% +4.2837% +4.9649%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 9 outliers among 100 measurements (9.00%)
  4 (4.00%) high mild
  5 (5.00%) high severe

cast date64 to date32 512
                        time:   [8.4563 us 8.4631 us 8.4702 us]
                        change: [+0.2066% +0.9093% +1.6066%] (p = 0.01 < 0.05)
                        Change within noise threshold.
Found 10 outliers among 100 measurements (10.00%)
  3 (3.00%) high mild
  7 (7.00%) high severe

cast date32 to date64 512
                        time:   [8.0482 us 8.0610 us 8.0823 us]
                        change: [+0.2428% +0.8807% +1.5657%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 7 outliers among 100 measurements (7.00%)
  1 (1.00%) high mild
  6 (6.00%) high severe

cast time32s to time32ms 512
                        time:   [2.2259 us 2.2294 us 2.2333 us]
                        change: [-2.3114% -1.6027% -0.7039%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 5 outliers among 100 measurements (5.00%)
  2 (2.00%) high mild
  3 (3.00%) high severe

cast time32s to time64us 512
                        time:   [7.8948 us 7.9060 us 7.9179 us]
                        change: [-27.022% -25.727% -24.361%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 5 outliers among 100 measurements (5.00%)
  2 (2.00%) high mild
  3 (3.00%) high severe

cast time64ns to time32s 512
                        time:   [11.546 us 11.558 us 11.571 us]
                        change: [-0.9659% -0.3235% +0.2847%] (p = 0.33 > 0.05)
                        No change in performance detected.
Found 19 outliers among 100 measurements (19.00%)
  2 (2.00%) low severe
  8 (8.00%) low mild
  1 (1.00%) high mild
  8 (8.00%) high severe

cast timestamp_ns to timestamp_s 512
                        time:   [27.676 ns 27.690 ns 27.714 ns]
                        change: [+0.9641% +1.2917% +1.6195%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 14 outliers among 100 measurements (14.00%)
  4 (4.00%) high mild
  10 (10.00%) high severe

cast timestamp_ms to timestamp_ns 512
                        time:   [2.8195 us 2.8238 us 2.8300 us]
                        change: [-0.4123% +0.7148% +1.6494%] (p = 0.19 > 0.05)
                        No change in performance detected.
Found 7 outliers among 100 measurements (7.00%)
  1 (1.00%) high mild
  6 (6.00%) high severe

cast timestamp_ms to i64 512
                        time:   [398.49 ns 399.60 ns 400.91 ns]
                        change: [-6.2549% -5.5279% -4.7295%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 17 outliers among 100 measurements (17.00%)
  5 (5.00%) high mild
  12 (12.00%) high severe

I'll take a look at this during the week

As @jorgecarleitao  mentions on https://github.com/apache/arrow/pull/8303#issuecomment-701328143 -- his goal in adding this functionality is to make it easier to improve the performance of common operations (by avoiding an intermediate Vec which is then just copied into a raw buffer)


Add a new Playlist

You can fork this repository and make your required changes by creating a PULL REQUEST instead of assigning an issue.


Add priority to commands

Would be very cool if you could also add a priority int proprty to the commands for example if you use it for let's say a rails app or something. You may need to start the docker db and redis db before launching sidekiq and the rails server.
It would look something like this:
procs:
  rails:
    cmd: ["rails", "server", "--port", "3000"]
    cwd: "/Users/ricardofabilareyes/projects/backend"
    priority: 1
  sidekiq:
    cmd: ["bundle", "exec", "sidekiq", "-C", "config/sidekiq.yml"]
    cwd: "/Users/ricardofabilareyes/projects/backend"
    priority: 1
  database:
    cmd: ["docker", "start", "mento_db"]
    priority: 2
  redis:
    cmd: ["docker", "start", "redis-stack"]
    priority: 2

This way both database and redis would start before rails and sidekiq.
I love this program man, I even added it to my own pkg manager 😄

Yeah I just created a simple bash script that does basically
sleep 5
rails serve

as my current work around, it works for my purposes, but I agree stopping can be an issue for some :nod:


Solve an issue: work for multi-input model

Solved: LRFinder doesn't work with Multi-Input data (Issue #6)
https://github.com/surmenok/keras_lr_finder/issues/6#issue-348216817

Thank you, Girishkumar!


Upgraded to Babel 6

Default presets: es2015 and stage-0, basically everything is available on a use-what-you-want basis
Only the babelhelpers that are actually used are included in the build
No polyfills are provided for ES2015 features, you are required to include the ones you need (see core-js and regenerator-runtime)
JavaScript intended for the browser is transpiled to AMD (see here and here)
Gulpfile is now also transpiled
Rewrote all gulp tasks to ES2015
Removed all useless postfixes (-compile, -transpile) from tasks
Added run-sequence to have series/parallel functionality without upgrading to gulp 4.
Moved browsersync settings to the config
Moved copying of fonts to a separate gulp task and file
The .eslintrc changes are due to the new ES2015 syntax, and the new code written inside tasks/js.js

After applying the patch file I get the following error when running npm install:
[21:02:57] Requiring external module babel-register
[21:02:59] Using gulpfile ~/Documents/frontend-bootstrap/gulpfile.babel.js
[21:02:59] Starting 'githooks-clean'...
[21:02:59] Finished 'githooks-clean' after 10 ms
[21:02:59] Starting 'githooks'...
[21:02:59] Finished 'githooks' after 27 ms
When running gulp dev this happens
[21:03:06] Failed to load external module babel/register
/Users/christiaanstegeman/Documents/frontend-bootstrap/gulpfile.babel.js:1
(function (exports, require, module, __filename, __dirname) { import 'babel-polyfill';
^^^^^^
SyntaxError: Unexpected token import
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:387:25)
at Object.Module._extensions..js (module.js:422:10)
at Module.load (module.js:357:32)
at Function.Module._load (module.js:314:12)
at Module.require (module.js:367:17)
at require (internal/module.js:16:19)
at Liftoff.handleArguments (/usr/local/lib/node_modules/gulp/bin/gulp.js:116:3)
at Liftoff. (/usr/local/lib/node_modules/gulp/node_modules/liftoff/index.js:192:16)
at module.exports (/usr/local/lib/node_modules/gulp/node_modules/liftoff/node_modules/flagged-respawn/index.js:17:3)

gulp CLI version 3.9.0
gulp Local version 3.9.1
node version v4.1.2
I got the same error as @cstegeman, looked into it (thanks to @branneman) and it looks like babel is the real issue here.
C:\Projects\branneman-frontend-bootstrap\frontend-bootstrap>node -e "require('babel-core/register')"
module.js:338
    throw err;
    ^

Error: Cannot find module 'babel-runtime/helpers/interop-export-wildcard'
    at Function.Module._resolveFilename (module.js:336:15)
    at Function.Module._load (module.js:286:25)
    at Module.require (module.js:365:17)
    at require (module.js:384:17)
    at Object.<anonymous> (C:\Projects\branneman-frontend-bootstrap\frontend-bootstrap\node_modules\babel-core\node_module
s\babel-types\lib\index.js:13:30)
    at Module._compile (module.js:434:26)
    at Object.Module._extensions..js (module.js:452:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Module.require (module.js:365:17)

The errors @cstegeman and @wouterkroes ran into were fixed after a Node.js upgrade. It turns out this needs a very modern version of Node.js: 5.9+


Add BTP and ILDCP modules

Fix sub-module versioning (all inherit from parent)
Add core libs and codecs
Drop submodules file (relic of crypto-conditions)
TODO:

Unit tests (Done for BTP Core)

Waiting for unit tests before approving.

Suggest we merge and track unit tests debt under:

#143
#144
#145

Ping @sappenin


Modify Makefile in kmm-kmod

Modify Makefile to use running kernel version if KERNEL_SRC_DIR is not set.

/cc @qbarrand

Codecov Report
Base: 73.47% // Head: 73.47% // No change to project coverage :thumbsup:

Coverage data is based on head (fb9f665) compared to base (bdd871f).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #118   +/-   ##
=======================================
  Coverage   73.47%   73.47%
=======================================
  Files          17       17
  Lines        1772     1772
=======================================
  Hits         1302     1302
  Misses        404      404
  Partials       66       66

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/ok-to-test

/approve

/lgtm

/lgtm


Access to sessionKey

Trying out, good stuff! Do you think its possible for you to expose sessionKey or make a getter for it?
I am trying to restrict some multi tabs logins with different users, and i need the sessionKey, i can see the value on the cookie on the next request but that is too late
Thanks!

Manage to achieve this without sessionKey


skip debugger libcython tests on pypy

These tests probe PyObject internals and cannot succeed on PyPy.

The test-suite does have some lists of tests to skip for PyPy - this is probably the most suitable for this test https://github.com/cython/cython/blob/master/tests/pypy_implementation_detail_bugs.txt
I'm not 100% sure whether those lists would work for this particular test (because it isn't in the main "tests" directory). But if it did work then it'd be consistent to add the test to the list instead.

if it did work

I couldn't figure out how to get it to work. In fact, when I run the tests locally <python> runtests.py Debugger, when <python> is CPython it runs these tests, when python is <pypy> they do not run, even without this PR. And this PR didn't work: the tests are still running. I will close this and try again.


DBZ-4334 Convert mysql time type default value error

For date/datetime convert method, I add try/catch to handle the invalid format value, the part zero value in this case, and for convertToDuration method, I use the regular expression to get time part value if matched.
https://issues.redhat.com/browse/DBZ-4334

LGTM, applied. Thanks a lot, @harveyyue. I'll never get why MySQL allows all these non-sense values in the first place, but here we are.


Run buttons are not displayed by default

Godot version
3472bdd6b
System information
Linux mint 21, NVIDIA GeForce GTX 1050 Ti, Vulkan API 1.2.0
Issue description
These buttons do not display properly after the project is first loaded unless the project is run once.

Steps to reproduce

Reload Current Project
Play with F5/F6

See the difference in play buttons
Minimal reproduction project
No response

Possible regression from #64756 @Calinou?


Update documentation with new frature

Objective
I need to add to the documentation the new feature that is in which you can change the default keys inside dictionary generated at the moment of validating the basic_auth, also for decoding if you have change this then you can decode by this keys that are not the default ones.
Important
Maybe its better to move this to attributes, something like you need to build a dictionary and pass that in a specific attribute in the instantiation of the class

New feature added to documentation


Fix issue with vercel compatibility

This PR refactors dynamic requires to simpler static ones to fix the following issue in vercel deployment:
2021-02-10T17:48:10.778Z	undefined	ERROR	Cannot find module './main/packages/Cosine'
Require stack:
- /var/task/node_modules/string-comparison/lib/index.js
- /var/task/node_modules/article-parser/src/utils/chooseBestUrl.js
- /var/task/node_modules/article-parser/src/utils/parseFromHtml.js
- /var/task/node_modules/article-parser/src/main.js
- /var/task/node_modules/article-parser/index.js
- /var/task/api/index.ts
- /var/task/___vc_launcher.js
- /var/runtime/UserFunction.js
- /var/runtime/index.js
2021-02-10T17:48:10.788Z	undefined	ERROR	Did you forget to add it to "dependencies" in `package.json`?
RequestId: ea2ddc67-287f-4183-a996-6794719a4207 Error: Runtime exited with error: exit status 1
Runtime.ExitError

Hi sorry to get back to you so late.  Vercel uses ncc under the hood when packaging serverless functions for deployment.  Unfortunately, this means dynamic requires like the ones in the current code fail because ncc relies on static analysis of requires.


Unallowed admission reviews to not reply to the requestor with http status code 200

/kind bug
Expected Behavior
The admission controller to reply to k8s API server on failed admission requests with http code != 2XX.
Actual Behavior
Even when the request is not allowed, the admission handler just return the admission review to the api server without explicitly setting HTTP status code, therefore the default status code of 200 is used. For k8s, this is considered as successful admission review and therefore the failure policy Ignore is not considered, hence the request fails.
The code writing the response is here https://github.com/knative/pkg/blob/12be06090b5122f58fa8b3a9870e26fb6de0bb20/webhook/admission.go#L136.
Steps to Reproduce the Problem

Get access to some k8s cluster, or just create a fresh one with kind,

$ kind create cluster

Deploy customized version of cosigned

$ kubectl create -f https://gist.githubusercontent.com/vpnachev/acd2b9ba3f457d6d05e44fd9c3a69fca/raw/111be95a393ea6cc2c63d593459553aec09219f7/cosigned-v1.5.2-with-ignore-failure-policy.yaml

this is just rendered template of the chart with changed failure policy to Ignore and added Namespace resource.
also a dummy public key is added

Create a namespace and label it with cosigned.sigstore.dev/include=true

$ kubectl create namespace test-ignore-policy
$ kubectl label namespace test-ignore-policy cosigned.sigstore.dev/include="true"

Wait the cosigned-webhook in the cosigned namespace to become Running.

Try to run pod in the test-ignore-policy namespace and make sure the create/pod request fails

$ kubectl -n test-ignore-policy run test --image=alpine:3.15.0
Error from server (BadRequest): admission webhook "cosigned.sigstore.dev" denied the request: validation failed: one of verifier or root certs is required: spec.containers[0].image
index.docker.io/library/alpine@sha256:21a3deaa0d32a8057914f36584b5288d2e5ecc984380bc0118285c70fa8c9300

Make sure there is no pod test in the test-ignore-policy namespace

$ kubectl -n test-ignore-policy get pod test
Error from server (NotFound): pods "test" not found

Additional Info

https://kubernetes.slack.com/archives/C0EG7JC6T/p1650375336872579
https://github.com/kubernetes/website/issues/33039

Does this: https://github.com/knative/pkg/blob/12be06090b5122f58fa8b3a9870e26fb6de0bb20/webhook/admission.go#L137 not set the status code to 500?

No, it does not. json.NewEncoder(w).Encode(response) returns nil and nothing else is explicitly written to http.ResponseWriter. When the HTTP code is not explicitly set, the default 200 is used, ref

// If WriteHeader has not yet been called, Write calls
// WriteHeader(http.StatusOK) before writing the data.

Very interesting. Talking with @mattmoor about this now.

Judging from this portion of the slack conversation:

I think the only potential issue here is that the json encoder may partially write to the response before erroring out, so the http.Error will be ineffective because as we start to write bytes the response writer will automatically use 200.
However, since this should result in a partial object, Kubernetes should reject the response because it is an invalid AdmissionReview, so this is mostly defensive (e.g. if nothing is written).  So short of buffering the response so we can definitively return a 500, I think what we're doing is correct.

The admission controller to reply to k8s API server on failed admission requests with http code != 2XX.

This statement from the expected behavior above seems inconsistent with Jordan's third comment in my screenshot.  failurePolicy does NOT control the behavior when the admission controller returns a valid AdmissionReview rejecting the request.
So unless I'm missing something, which is entirely possible, this seems like it is WAI to me.

It depends how the failure policy is used and what is the expectation.
During the last 4 years I was always expecting that failure policy Ignore will allow even requests rejected by admission controller to be completed. And we have couple of tens of admission controllers that were used like this. In the initial phase we deploy them with Ignore and just later, when we have some experience and the bugs are fixed, we switch the failure policy to Fail.
Now, I want to deploy cosigned (component using knative.dev/pkg as SDK for admission controller checking if OCI images are signed), but I know cosigned will reject a lot of the requests initially, so I wanted to deploy it with failure policy Ignore, and observe in the monitoring which images are failing the check. After some time, all our images will be successfully admitted, and then I can change the failure policy to Fail. I just want to use some mechanism to use silent failures without causing disruptions and enforce the validation only after all prerequisites are met.

I may be misunderstanding @liggitt but my impression is that what you describe is not what failurePolicy was intended to handle.

Yes, I find yesterday that I was having wrong understanding how failure policies work. We (maybe also others) were just using it as loophole to configure admission controllers to fail silently.


Support multiple keys/filesets for a bucket using export map

Pull request checklist

Requirements:
1.  [Support multiple keys/filesets for a bucket](https://github.ibm.com/IBMSpectrumScale/scale-core/issues/7280)
2.  [Add makeActive variable for the cache filesets](https://github.ibm.com/IBMSpectrumScale/scale-core/issues/7577)

- First scale changes needs to be merged before this PR merge.
This PR depends on the changes of both the below PRs to be merged.
[(EWM-336519) Inconsistent response from Node Mapping with cos Delete REST API](https://github.ibm.com/IBMSpectrumScale/scale-core/issues/7559)
[(EWM-336293) Changes need to be done in GUI to align key verification fixes with AFM](https://github.ibm.com/IBMSpectrumScale/scale-core/issues/7542)

RTC details are:
[https://jazz07.rchland.ibm.com:21443/jazz/web/projects/GPFS#action=com.ibm.team.workitem.viewWorkItem&id=336293](https://jazz07.rchland.ibm.com:21443/jazz/web/projects/GPFS#action=com.ibm.team.workitem.viewWorkItem&id=336293)
[https://jazz07.rchland.ibm.com:21443/jazz/web/projects/GPFS#action=com.ibm.team.workitem.viewWorkItem&id=336519](https://jazz07.rchland.ibm.com:21443/jazz/web/projects/GPFS#action=com.ibm.team.workitem.viewWorkItem&id=336519)

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature Enhancement
[ ] Test Automation
[ ] Code Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Community Operator listing
[ ] Other (please describe):

What is the current behavior?

What is the new behavior?

How risky is this change?

[ ] Small, isolated change
[x] Medium, requires regression testing
[ ] Large, requires functional and regression testing

Image: quay.io/badri_pathak/ibm-spectrum-scale-csi-driver:afmCacheExportMap_v17

New driver Image: quay.io/badri_pathak/ibm-spectrum-scale-csi-driver:afmCacheExportMap_v18

This PR depends on the changes of both the below PRs to be merged.
(EWM-336519) Inconsistent response from Node Mapping with cos Delete REST API
(EWM-336293) Changes need to be done in GUI to align key verification fixes with AFM

Image: quay.io/badri_pathak/ibm-spectrum-scale-csi-driver:afmCacheExportMap_v24

New image with cosKeys deletion issue: quay.io/badri_pathak/ibm-spectrum-scale-csi-driver:afmCacheExportMap_v27

The issue https://github.ibm.com/IBMSpectrumScale/scale-core/issues/7765 has been fixed.


Fixed Unicode response info rendering in Python 2.

This patch fixes the following:
Traceback (most recent call last):
  File "/home/phil/build/apidocs/env/local/lib/python2.7/site-packages/sphinxcontrib/swaggerdoc/swaggerv2_doc.py", line 238, in run
    section += self.make_method(path, method_type, method)
  File "/home/phil/build/apidocs/env/local/lib/python2.7/site-packages/sphinxcontrib/swaggerdoc/swaggerv2_doc.py", line 180, in make_method
    swagger_node += self.make_responses(responses)
  File "/home/phil/build/apidocs/env/local/lib/python2.7/site-packages/sphinxcontrib/swaggerdoc/swaggerv2_doc.py", line 110, in make_responses
    response.get('description', '')))
UnicodeEncodeError: 'ascii' codec can't encode characters in position 0-4: ordinal not in range(128)

Looking at the rest of the code, I concluded that the original %-style formatting is preferred, and it is both Python 2 and Python 3 compatible and Unicode-agnostic as well.

Thank you @tuffnatty for your contributions!


Write a TypeScript program to find the rank of a matrix

Description
Write a TypeScript program to find the rank of a matrix
The rank of a matrix is the dimension of the vector space generated by its columns. The rank of a matrix is also equal to the dimension of the vector space generated by its rows. The rank of a matrix is also equal to the number of pivots in its reduced row echelon form.
Input  :
1 2 3
4 5 6
7 8 9

Output :
2

How to contribute

Save the solution in program/FindTheRankOfAMatrix.ts file
Add FindTheRankOfAMatrix.ts file in find-the-rank-of-a-matrix folder

!assign

!assign

This issue has already been resolved with PR #7133
Here is the file FindTheRankOfAMatrix.ts


Why do you weight latencies from different scales differently?

In your code you weight the latency coming from each branch differently with
scale_latency_weights=[3./12, 4./12, 5./12]

in:
https://github.com/TAMU-VITA/FasterSeg/blob/bb52a004ff83f64d3dd8989104234fdb862d1cc5/search/model_search.py#L474
Can you elaborate on why you do this?

Hi @maaft ! I just updated the repo. You don't have to use this weight.


"Close" button remains disabled after cancel

Fixes #2162

To explain a bit more - the problem was that the promise in context.txt wasn't ever resolving. The extension has to send either _replyMessage or _throwError to respond back to the webview. Once it gets a response, the webview is able to update the button state


Bioinformatics Pipeline Standards and Review Process

Creation of standards by which bioinformatics resources can be systematically assessed plus a review process defining the assessment criteria and procedure

Useful starting resources to get us started:

Ten recommendations for creating useable bioinformatics command line software
Existing Workflow Systems
Workflow Execution Service (WES) API
GA4GE Tool Execution Challenge
bio.tools tool registry

More useful resources:

JOSS review checklist
How to write bioinformatic software no one will use
Global coalition regulatory science research

https://www.slideshare.net/torstenseemann/how-to-write-bioinformatics-software-no-one-will-use
https://www.nature.com/articles/s41591-020-0935-z
^ "Ten recommendations for supporting open pathogen genomic analysis in public health"  - Allison Black, Duncan R. MacCannell, Thomas R. Sibley & Trevor Bedford


Merge all changes into 1 branch

This PR contains changes made by me in #46 and #48.

📝 Changed routes:

/about/faq
/about

(view full size)
(view full size)

/about/questions
/admin

(view full size)
(view full size)

…n/resolve/[questionId]
/admin/users

(view full size)
(view full size)

And 24 other routes:

/api/announcements
/api/applications
/api/members
/api/questions
/api/questions/pending/[questionId]
/api/questions/pending
/api/resume/upload
/api/tokens
/api/userinfo
/api/users
/dashboard/Components/AnnouncementCards
/dashboard/Components/DocLinks
/dashboard/Components/MentorCard1
/dashboard/Components/MentorCard3
/dashboard/Components/Sidebar
/dashboard/Components/SpotlightCard
/dashboard/Components/SpotlightCardScroll
/dashboard/hackerpack
/dashboard
/dashboard/scan-in
/dashboard/submit
/
/profile
/register

Commit 950899d940530bdbc8568652b4c95a7ed89dfbec (https://hackportal-jlpyiftuj-acmutd.vercel.app).

There are some changes that would end up deleting and replacing some of the code that should be used in the final products. The smartphone nav(in app header) was deleted and replaced with a dropdown in this pr but the smartphone nav should ultimately be used.


Feature reduce

Added a new feature where users can "reduce" an image by a X factor.

Codecov Report

Merging #1 (8070c15) into main (e9a7584) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #1   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          217       241   +24
=========================================
+ Hits           217       241   +24

Impacted Files
Coverage Δ

properjpg/__init__.py
100.00% <100.00%> (ø)

properjpg/cli.py
100.00% <100.00%> (ø)

properjpg/processing.py
100.00% <100.00%> (ø)

tests/test_cli.py
100.00% <100.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 e9a7584...8070c15. Read the comment docs.


fix: fix error handling issues from sonarqube

Description
Fix error handling issues found by sonarqube
Type of change

[ ] Refactor/improvement

How Has This Been Tested?
Checklist

[ ] My commit message is conventional
[ ] I have run pre-commit on all files before creating the PR
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have checked my code and corrected any misspellings

Codecov Report
Attention: 8 lines in your changes are missing coverage. Please review.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #956      +/-   ##
===========================================
- Coverage    84.11%   84.03%   -0.09%
===========================================
  Files           28       28
  Lines         2027     2029       +2
===========================================
  Hits          1705     1705
- Misses         322      324       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:tada: This PR is included in version 1.10.0-beta.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Open profile when search results tapped on mobile

The necessary plumbing for profiles + back button on the route stack was already done on mobile. Just needed to diverge the desktop and mobile behavior when results are clicked. I decided to do this by checking the platform constant in the smart component. Thoughts on this?
:eyeglasses: @keybase/react-hackers

👍


Specify thread safety requirements for GLFW memory allocator

The GLFW documentation for the new 3.4 memory allocator that users can initialize and pass to GLFW, does not specify threading requirements. It is unclear if this allocator must support multi-threaded access, or if it will only be used on the main thread.

There is a thread-safety line for each callback type, but I definitely agree this is too little. Thank you for spotting this!


feat: enable new endpoint type for cluster config

Description
In order to fully support OCP 4.13 with disabled public endpoint, the IBM provider team has added a new parameter endpoint_type to the ibm_container_cluster_config data block to specify an endpoint type that may not be the default.
Added a new input parameter to provide the endpoint type to this module, and use this module in the ibm_container_cluster_config data block.
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Added new input variable cluster_config_endpoint_type that will be passed into the new parameter endpoint_type for the ibm_container_cluster_config data block used in this module.
This change will allow you to specify a different endpoint for cluster configuration than the default, for instance if using OCP 4.13 with public endpoint disabled, and you do not want to use the default endpoint of VPE, you can now specify "private" to use the private endpoint.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

:tada: This PR is included in version 2.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[SPARK-33591][3.1][SQL][FOLLOWUP] Add legacy config for recognizing null partition spec values

What changes were proposed in this pull request?

This PR is to backport https://github.com/apache/spark/pull/31421 and https://github.com/apache/spark/pull/31421 to branch 3.1
This is a follow up for https://github.com/apache/spark/pull/30538.
It adds a legacy conf spark.sql.legacy.parseNullPartitionSpecAsStringLiteral in case users wants the legacy behavior.
It also adds document for the behavior change.
Why are the changes needed?

In case users want the legacy behavior, they can set spark.sql.legacy.parseNullPartitionSpecAsStringLiteral as true.
Does this PR introduce any user-facing change?

Yes, adding a legacy configuration to restore the old behavior.
How was this patch tested?

Unit test.

Test build #134787 has started for PR 31439 at commit e5229d7.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39374/

Test build #134787 has finished for PR 31439 at commit e5229d7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/134787/

Merged to branch-3.1.


Component Library

Use component library for button

added a component library, however still needs adjustment for it to work properly


npgsql does not respect default RC2 table naming conventions

From @crozone on June 7, 2016 4:22
As per this announcement , the default naming convention for tables in RC2 has been changed from using the entity class name as the default, to using the DbSet property name as the default.
Additionally, Migrations are now stored in ./Data/Migrations, instead of ./Migrations. this is an unrelated change.
When using the Npgsql adapter, the legacy RC1 naming conventions are still used.
Copied from original issue: npgsql/npgsql#1147

Moving to the EFCore repo, all EFCore issues should be opened there.

Thanks for this, I'll apply this change ASAP.

From @crozone on June 7, 2016 4:53
For reference, here is the relevant pull request that performed this change, which also includes the SqlServer and SQlite provider changes.


Write absolute AppDataPath to app.ini when installing

If the APP_DATA_PATH isn't written into the config when installing, then its value is uncertain because some Gitea command doesn't run with correct WorkPath.
This is a quick fix for #25330  and can be backported.

Thank you for reviewing and approving, I just realized that there are more details and this PR is not ideal for 1.21.
I would propose a new separate PR for 1.20 later. 🙏

OK, hard decision ..... let's just do this one-line patch, and leave more problems to #25330


scylla_cql: autoderive Default for Consistency

As the awaited issue in num_enum crate has been resolved in its 0.6 release, it is now possible to derive Default for Consistency automatically.
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[ ] I have provided docstrings for the public items that I want to introduce.
[ ] I have adjusted the documentation in ./docs/source/.
[ ] I added appropriate Fixes: annotations to PR description.

@piodul as we are already heading towards 0.9, can we merge this?

As the awaited issue in num_enum crate has been resolved in its 0.6 release

What issue? The description should be more specific.

Please improve the commit message & PR description and then it will be good to merge.

done.


docs: add jwt demo

fix: #14
inherited from @yance-dev pr#18, thanks for contribution

@rahulsingh7 @leeqvip plz review


fix: error when base image doesn't support target platforms

Fixes #3563: Converts warnings to exceptions.

[X] Create a new issue at https://github.com/GoogleContainerTools/jib/issues/new/choose.
[ ] Ensure that your implementation plan is approved by the team.
[X] Verify that integration tests and unit tests are passing after the change.
[X] Address all checkstyle issues. Refer to the style guide.

Thank you; we'll review next week.

LGTM! Thanks again @gsquared94!