Detect source encoding to properly interpret string literals

This addresses https://github.com/m-labs/pythonparser/issues/6
Buffers now always hold unicode source, whereas before they could hold bytes if that's what were passed in to the constructor. This is possible because we determine the encoding and then use that to decode() the bytes. The side effect is that Buffer.__init__ could raise UnicodeDecodeError if the input is badly encoded.
The Buffer encoding is then used by the lexer to produce a strdata token of the correct type for string literals. For unicode literals, escaping happens much as before via _replace_escape(). For bytes, there's a different code path that calls encode() using the Buffer's encoding followed by a special escaping function that ensures the value's not accidentally promoted to unicode.
The parser behavior for multi-string literals (e.g. "foo" "bar") also had to change. When any of the literals are unicode, the result is unicode. When all the literals are bytes the resulting value is also bytes.
This PR also implements the unicode_literals future feature in a similar fashion to print_function. The one shortcoming of that approach is that it does not affect all string literals in the file, only the ones after the current lexer position. I think a proper implementation would require doing a lexer pass on the input to activate future flags before parsing. I didn't want to add any more complexity to this PR since it's already getting a bit big and this seems good enough for now.

Whoops looks like python3 is busted. Will fix.

The tests that parse string literals now use UnicodeOnly and BytesOnly values in the expected values to make sure both values and types are identical. Making this work for 2.x and 3.x grammars and run properly under 2.x and 3.x interpreters required some hackery.
PTAL

By the way...

This PR also implements the unicode_literals future feature in a similar fashion to print_function. The one shortcoming of that approach is that it does not affect all string literals in the file, only the ones after the current lexer position. I think a proper implementation would require doing a lexer pass on the input to activate future flags before parsing. I didn't want to add any more complexity to this PR since it's already getting a bit big and this seems good enough for now.

it's illegal to have any actual code before the future imports, so this is immaterial.

Thanks for merging!
The module docstring would be bytes whereas it should be unicode. Certainly unlikely to cause any problems but it is a material difference.

The module docstring would be bytes whereas it should be unicode. Certainly unlikely to cause any problems but it is a material difference.

And only in Python 2 mode, right? That's fine by me, Python 2 support is only really included for completeness.

Right, Python 2 only. Also fine by me. Just making sure it's documented.


Python 3.10 support

After adding Self to hardware.py and build.py, support for version 3.10 broke.

Thanks for the report. Can you check if version 0.1.3 fixes the issue?


modify http2 server some judgment with stream id

Motivation:
To slove the problem: some proxy product, like envoy,  send first normal http2 requst with stream id 1 in prior-knowledge mode, then sofa-rpc server throw exception.
Because the http2 protocal ref only limit using 0x01 as stream id in http 1.1 upgrade to http 2 mode.
reference : https://http2.github.io/http2-spec/#rfc.section.5.1.1
Modification:
Added some judgment logic in Class Http2ServerChannelHandler.

Codecov Report

Merging #191 into master will decrease coverage by 0.04%.
The diff coverage is 71.42%.

@@             Coverage Diff              @@
##             master     #191      +/-   ##
============================================
- Coverage     71.25%   71.21%   -0.05%
- Complexity      955      956       +1
============================================
  Files           341      341
  Lines         14210    14213       +3
  Branches       2286     2286
============================================
- Hits          10126    10122       -4
- Misses         2891     2897       +6
- Partials       1193     1194       +1

Impacted Files
Coverage Δ
Complexity Δ

.../rpc/transport/http/Http2ServerChannelHandler.java
75.4% <71.42%> (-0.23%)
28 <6> (+3)

...m/alipay/sofa/rpc/module/FaultToleranceModule.java
66.66% <0%> (-26.67%)
4% <0%> (-1%)

...y/sofa/rpc/bootstrap/DefaultProviderBootstrap.java
63.95% <0%> (-2.54%)
0% <0%> (ø)

...ava/com/alipay/sofa/rpc/config/RegistryConfig.java
89.21% <0%> (-0.99%)
0% <0%> (ø)

.../alipay/sofa/rpc/registry/local/LocalRegistry.java
57.22% <0%> (-0.61%)
25% <0%> (-1%)

.../main/java/com/alipay/sofa/rpc/tracer/Tracers.java
42.3% <0%> (+2.56%)
0% <0%> (ø)
:arrow_down:

...pay/sofa/rpc/transport/ClientTransportFactory.java
78.46% <0%> (+4.61%)
0% <0%> (ø)
:arrow_down:

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 d47a851...f2a2c8a. Read the comment docs.


[User Story]: As a developer, I want to add unit and integration tests to ensure code functionality and catch bugs early on. (Connects to: Use testing)

As a developer, I want to add unit and integration tests to ensure code functionality and catch bugs early on.
Estimated Time
TBD
Acceptance Criteria
Automated tests are run via GH actions with >90% test coverage.

Part of epic #8


v2.9.0 contained a breaking change, should be semver-major

The requireCapitalizedConstructorsNew rule defaults to on - thus v2.9.0 is a breaking change in that tests fail that passed on v2.8.0 with identical config.
Please release a v2.9.1 with this defaulting to false.

We've been doing these changes for a while though https://github.com/jscs-dev/node-jscs/blob/master/OVERVIEW.md#versioning--semver
This is probably the best reason for why presets should be moved out of the repo / just versioned independently since every rule change is breaking/major right?

preset changes are likely to be major but don't have to be.
It's not new rules that are breaking - it's defaulting them to "on". Adding a new rule is always a minor, and fixing a bug is always a patch. I realize it's tricky specifically with linter rules.

Yeah it's weird and I certainly see how it's breaking - @markelog. Also would https://github.com/airbnb/javascript be willing to host/maintain the airbnb preset? https://github.com/jscs-dev/node-jscs/issues/1811

Absolutely - let's have an off-github discussion about what's involved first :-)

See http://jscs.info/overview#versioning-semver


pinpoint agent  windows  tomcat  start mode

I install pinpoint agent  monitor tomcat  when use modify catalina.bat  in windows , startup and monitor  all  is ok
but when i  use  service.bat install  service regedit service ,then  tomcat8w.exe add  in java option
-Dpinpoint.applicationName=
-Dpinpoint.agentId=
-javaagent:c:/pinpoint-agent/pinpoint-bootstrap-1.8.2.jar
not work
must modify start mode from jvm to java.
how do i do when using jvm mode , what difference between mode jvm and java i think  java
environment  config something wrong
tks

Hi @xynhaha
This looks like a Tomcat windows service issue. Pinpoint agent is added using -javaagent option just like any other agents.
Please check how to add jvm arguments when running Tomcat in jvm mode.


Update 2019-01-16.md

Correct (complete) attendee name and affiliation.

Thanks for the fix @tvbroughton !


Remove bearer token if cookie_enabled is true

Some people were asking for the bearer token to not be included when using cookie_enabled: true in the PR that added the bearer token

@rhiroshi can you fix the specs?

Any chance to get this merged?

@blackst0ne sorry, just checking this, can you rebase to make sure the build pass?

@MaicolBen
This is not my PR. :)

Oh sorry, bad tag, @rhiroshi can you rebase?

@MaicolBen Rebased. Had some tests failing but was not related to my changes. I made another commit to trigger a rerun and the tests passed. So sorry for taking so long, had a heavy workload on the end of the last year and ended up forgetting to fix things here.

@rhiroshi no problem, thank you!

@MaicolBen Could we be reviewing a way to add the header? Because it is very useful in requests that are not via cookies, and it is the default reported by the Bearer token documentation. If you agree, I will think of a solution that does not honor cookies and both can coexist, since having the token explained in several "keys" is actually even more "expensive" in terms of size, which was the reason for removing it, because we could follow the pattern and the definition of the implementation and have only the "Authorization" header, instead of sending it separately.
What do you think? Removing the other headers would imply a break change, but we can do this in parts, starting with opening requests for changes to the libraries that use this lib.

@MaicolBen Could we be reviewing a way to keep the header even using cookies? Because using cookies does not eliminate the need for headers, since shell applications (and others) that do not support cookies need them, so there are applications (like the one I develop) that have both functionalities (authentication through cookies or headers).
As the main driver of this change is due to the size of the cookie, what do you think about following the pattern of other languages and even the documentation, returning only the "Authorization" header, that is, removing all others (access-token , token-type, client, expiry, uid) that we currently have, as they are "just" the "separate" form of the token, and in turn end up making the size bigger?
I understand that removing these other headers would imply a breaking change, but we can do this in parts, starting with a flag to return in this new format and opening pull requests for changes in libraries that use this lib (eg ng-token -auth, jToker, etc).
What do you think? Could we go down this path? If so, I can start crafting.


Possibility to edit proposed run command

A cool feature would be to have the possibility to edit a proposed run command, when the user know it's not exactly matching expectations.
Something like changing the confirmation prompt from Confirm? [y/N] to Confirm? [y/e/N], where e would trigger edition

What about returning it to stdout so it can be pipe-able to other commands?

I was more thinking refocus on the prompt with the command open for edition, with a "edit" icon (to make user understand he takes control).
would keep UI coherence and minimize the work imo
for piping, you can already do that since 0.2.0, for ex:
cat some_stuff.go | yo -c generate unit tests

In addition to editing the command, you might like also to consider that you might like to add more prompt information so that it provides an updated command with the additional information.
In effect Confirm? [y/e/p/N] where p gets you a prompt to add more detail.

Yep, that's the idea.
refine is good :+1:
Here's an example of how I saw it being used:
🚀 > port forward port 5433 on my machine to localhost:5432 on the remote machine
   ssh -L 5433:localhost:5432<EMAIL_ADDRESS>

forward the local port 5433 to the remote machine port 5432 over ssh
confirm execution? [y/e/r/N]
The first way to solve this is e and edit it yourself, or, as we are now discussing, with r and add:
🚀 > The user is greg, and the remote host is example.com
   ssh -L 5433:localhost:5432<EMAIL_ADDRESS>

forward the local port 5433 to the remote machine port 5432 over ssh. The user is greg and the remote machine is example.com
confirm execution? [y/e/r/N]

looks good !
on it asap


Enable ADC resolution

Closes https://github.com/google-github-actions/get-gke-credentials/pull/124

@bharathkkb PTAnotherL


fix for custom scalar client-side usage

@glasser

is there someone trying to fix this?
or i'll give it a go?

Can't hurt! Doubt anyone will look at it until California Monday at the
earliest anyway...
On Dec 11, 2016 12:41 AM, "Hagai Cohen"<EMAIL_ADDRESS>wrote:

is there someone trying to fix this?
or i'll give it a go?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apollostack/graphql-tools/pull/224#issuecomment-266270243,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABBVFM203VPeHLvn6zwF0RL0UbOLVmyks5rG7cugaJpZM4LHypv
.

Ok,
so i did look into it, and found the bug.
basiclly, to create a scheme, first we will use graphql to generate typedef to scheme.
when doing so, a GraphQLScalarType is being created, with default parse functions.
later on, we will run the function to bind resolvers.
However, this specific line: https://github.com/apollostack/graphql-tools/blob/master/src/schemaGenerator.ts#L280
replaces the placeholder scalar type with the one provided.
and then overrides the original type with the fields from the given scalar.
https://github.com/apollostack/graphql-tools/blob/master/src/schemaGenerator.ts#L293
however, all the arguments of the fields (which has arguments ofcourse) did not replace type.
so when comparing between the two (argType === scalarType) we will get a missmatch.
so, to sum this up, we have couple of choices:

run a "fix" function on the scheme to fix arguments after resolvers placed.

IMO, this approach will hit performance of scheme compiling.

remove this chunk of code which replaces the type on map:

    if (type instanceof GraphQLScalarType) {
      const resolveFn = resolveFunctions[typeName];
      if (resolveFn instanceof GraphQLScalarType) {
        const thisScalarType: GraphQLScalarType = resolveFn;
        schema.getTypeMap()[typeName] = thisScalarType;
      }
    }

however, this means that the original scalar given is not really used, only a copy of the handling functions. (easier fix)

@Urigo / @glasser / @helfer what do you think?

@DxCx I'd say we should just replace the functions and not the actual scalar instance.

(Like your approach 2)

So literally just delete the quoted block?  That does appear to make the whole test suite pass (well, Uri's test actually has to be adjusted slightly to expect(result.errors).to.not.be.ok; instead of expect(result.errors.length

ok @Urigo,
i just pushed the fix on your branch and updated the PR.
@helfer / @stubailo please approve and merge :)


net: TestTCPSelfConnect reports failure after 34 hours of stress testing.

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
On a
rlh@rlh1:~/work/go/src$ go version
go version devel +d4b46aa Thu Dec 8 01:36:44 2016 +0000 linux/amd64
What operating system and processor architecture are you using (go env)?
rlh@rlh1:~/work/go/src$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/usr/local/google/home/rlh/work/code"
GORACE=""
GOROOT="/usr/local/google/home/rlh/work/go"
GOTOOLDIR="/usr/local/google/home/rlh/work/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build725383804=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
rlh@rlh1:~/work/go/src$ go version
go version devel +d4b46aa Thu Dec 8 01:36:44 2016 +0000 linux/amd64
rlh@rlh1:~/work/go/src$
rlh@rlh1:
This is a 6 core 12 thread machine.
vendor_id	: GenuineIntel
cpu family	: 6
model		: 63
model name	: Intel(R) Xeon(R) CPU E5-1650 v3 @ 3.50GHz
stepping	: 2
siblings	: 12

What did you do?
I ran ./run.bash in a simple loop using the following.
GOTRACEBACK=crash ./run.bash -no-rebuild
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
See above.
What did you expect to see?
Success.
What did you see instead?
After a day (34 hours) 727 successful runs I got the following failure.
..snip. ...
ok  	mime/quotedprintable	0.162s
--- FAIL: TestTCPSelfConnect (0.13s)
tcpsock_test.go:661: Dial <IP_ADDRESS>:48322 should fail
tcpsock_test.go:661: Dial <IP_ADDRESS>:48324 should fail
tcpsock_test.go:661: Dial <IP_ADDRESS>:48325 should fail
tcpsock_test.go:661: Dial <IP_ADDRESS>:48326 should fail
tcpsock_test.go:661: Dial <IP_ADDRESS>:48327 should fail
tcpsock_test.go:661: Dial <IP_ADDRESS>:48328 should fail
tcpsock_test.go:661: Dial <IP_ADDRESS>:48329 should fail
FAIL
FAIL	net	1.527s
ok  	net/http	2.550s
ok  	net/http/cgi	0.495s
... snip ..
Note missing 48323.
I have not reproduced it. The only reason this is a concern is that it limits stress testing speculative parts of the GC and runtime.

/cc @mikioh @rsc

Challenging against TCP simultaneous open is a bit tough job. Let's push out the test from short-mode testing except when running on buildbots.

If the test just rarely catches the bug, I'd rather not sweet it under the rug by making it run less often.
Is this not a real bug?
If so, what's the bug.
If not, I'd rather deflake it.

Is this not a real bug?

I'm confused. The fix (https://codereview.appspot.com/5650071) was approved by you. I'm not sure what you are asking for.
My understanding is that TCP simultaneous open is a valid behavior of TCP and the fix written by @rsc was just a try to mitigate some undesirable side effect of the behavior for helping some application. But that might be a compromise. I think it's basically impossible to avoid TCP simultaneous open completely unless we have full control of each TCP FSM and IP control block inside the kernel.

I don't see any GNU/Linux failures of TestTCPSelfConnect in the builders at all.  I do see a few failures on Solaris; for example: https://build.golang.org/log/5315cb250600796a1ffe1c4c3a8c3f626769b692.

2020-11-09T15:46:56-979e137/dragonfly-amd64

watchflakes found another one of these, reported in #57609.

I propose that we fix this by deleting the test (https://go.dev/cl/460657).
The test's condition seems to be buggy: due to a typo in https://go.dev/cl/21447, it fails if it finds any connection whose source port OR IP address matches the destination port or address, which includes much more than just self-connects. (It essentially becomes a port-scanner for the previously-closed destination port.)
But even if we fixed that bug, the cost of dialing an arbitrary, possibly-reused port is fairly high (port exhaustion, CPU time, etc.), the potential for interference with other tests is high (because these dials may be accepted in lieu of a dial expected by some other test process), and the coverage of the test is low (on non-Linux short tests it scales back by 1000x, and it can only even possibly fail in the presence of what is arguably a kernel bug).
To me, the marginal benefit of keeping this test is not worth the cost of maintaining it.

@gopherbot, please backport to Go 1.19 and 1.20. This is a trivial backport to delete a noisy test.


suggestion: distance2 operator

Pretty simple, just a gm:distance operator but doesn't find the square root of the distance. basically just returns x^2 + y^2 to reduce computation when you don't need the actual distance in e.g. comparisons.

the distance calculation uses a single display entity transform to calculate, doing a "simplified" distance calculation would require 6 display entity transform calculations.


flutter_tools asset manifest logic has surprising behavior

The flutter tool defines assets an asset manifest as a key-value pair of a file URI and a File. As a part of the build process, these files are copied out of their respective directories and into a build directory where they can be packaged up into a platform specific distribution format.
To determine where the files should be copied, the tool uses the file URI as a file name, which happens to work across all three platforms. This is surprising behavior, and even though it works it is odd to have files created on windows such as assets\foo/bar.png.
We should investigate whether this was an intentional choice to work around a limitation of the tools at the time, or whether it is a bug that happened to work.
In either case, we should make an effort to simplify the logic as much as possible and document the remaining surprises in the tool

cc @zanderso

Copying from the PR that changed this - we need it this way, so that when we uri encode the file name on the platform channel, it matches the asset in the APK on Android.


[Question]:   resizeTo() and moveTo()  not working in Chromium

Your question
resizeTo(w,h)   moveTo(x,y)
Any reason why these functions silently fail in Playwright (Chromium)  but not in the normal browser ?
They work ok in Playwright (Firefox).
I have also tried a non-playwright Chromium incognito and they also still work.

Probably because Playwright spawns Chromium with a fixed window size. You can prevent that from doing by creating the context with ViewportSize = ViewportSize.NoViewport


Do not replace hyphen with fileseparator

Exercises used to be in subfolders so fileseparator was necessary. This issue was fixed in tmc-core.

Changes Unknown when pulling 5734e7945ed4b49e2c4270b96e4be90fa1b1ecce on run-tests-subfolder-bug-fix-juha into ** on master**.

Changes Unknown when pulling 5734e7945ed4b49e2c4270b96e4be90fa1b1ecce on run-tests-subfolder-bug-fix-juha into ** on master**.

LGTM.


feat(api): bind extensions stdout and stderr to current process

Closes #3269
Unfortunately, I did not find a way to bubble up the error happening in the extension to the Portainer UI.
This implementation will make all the extension logs now available inside the same output as the Portainer logs.

@deviantony RTM


Fix bullet list formatting error

Bullet list wasn't displaying because of missing line break. Fixed now.

Thanks @strickvl for your contribution.


ENH: update package version to 4.11.0

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #31.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #31

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

No valid build backend found for Python recipe for package ydata-profiling using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

Error - ydata-profiling 0.0.dev0 has requirement wordcloud>=1.9.3, but you have wordcloud 0.0.0.
even though installed version is wordcloud:  1.9.3-py312h66e93f0_2      conda-forge


On Device Activity not working?

Just checking if others are having similar issues? It has stopped working for me for the last few hours. I mainly use this to see which device /room the event is triggered from...

Also having that same issue - everything else seems to be working without issue

Same issue here.  I rebooted the box node-red is on after some maintenance and now on device activity is dead.

Me too

https://github.com/Apollon77/alexa-remote/issues/132#issue-1011417647 looks like it is a bigger issue than we thought.

me too

Same issue here.

It also stopped working for me, I've looked it up in several places, but I haven't found anything yet :(

Working for me again.

It's been coming and going over the last few days. Let's see what happens when it settles

Same here. Stopped working about a week ago. Haven't found a workaround that is successful for me yet.

Still working for me (US). Did you try manually refreshing your cookie?

Ah - I did not realize there was a difference between just reinitializing (from the Proxy), and dropping/recreating the proxy file.
I did that, and now that I have reestablished my Amazon logon/pw, the Device Activity node appears to be working... sometimes.  I'm not convinced it is consistently working like it was.  But at least now I have had some successes.  I will continue to monitor.  Thanks for the insights.  Hopefully a permanent fix will resolve the underlying issue.

It's working intermittently this morning.  It seems to work once and then become unresponsive for some period of time.  There is a significant delay between the Alexa routine firing and the Device Activity response if it fires.

Same ... been down for me as well.

Down for me now as well... /sigh

can it be that it doesn't work again?

Hi @cakebake , are you planning on updating your fork? Many thanks for everyhing you've done..

Migrating to Appelstrudel fork as @cakebake hasn't updated and my whole home automation depends on device activity functionality https://github.com/bbindreiter/node-red-contrib-alexa-remote2-applestrudel

Mine has come back to live again but who know for how long. Anyways, can you please share how you migrated please? thanks in advanced


Floating point number is saved as Number - both in code & MongoDb command

We have the same problem (as this one https://github.com/jagi/meteor-astronomy/issues/575), both within our code:
return Gedmatches.insert({ kit1, kit2, totalCm, created, notSharing: true });
and when using MongoDb insert directly:
db.gedmatches.insert({ kit1: 'SyVRzmvn4f', kit2: 'Syfq7mD3Nz', created: new Date(), updatedAt: new Date(), totalCm: 7.32, HIR: 1 })
In both cases it changes the field value to 7 in the doc instead of 7.32
There must be an easier way to allow floating point numbers!
Originally posted by @a4xrbj1 in https://github.com/jagi/meteor-astronomy/issue_comments#issuecomment-437563068

It's rather unlikely to happen. Neither MongoDB nor Astronomy would do that
as long as you're not doing something extra. Amount of info you've provided
is not sufficient to tell.
On Sat, Nov 10, 2018, 7:50 AM Andreas West<EMAIL_ADDRESS>wrote:

We have the same problem (as this one #575
https://github.com/jagi/meteor-astronomy/issues/575), both within our
code:
return Gedmatches.insert({ kit1, kit2, totalCm, created, notSharing: true
});
and when using MongoDb insert directly:
db.gedmatches.insert({ kit1: 'SyVRzmvn4f', kit2: 'Syfq7mD3Nz', created:
new Date(), updatedAt: new Date(), totalCm: 7.32, HIR: 1 })
In both cases it changes the field value to 7 in the doc instead of 7.32
There must be an easier way to allow floating point numbers!
Originally posted by @a4xrbj1 https://github.com/a4xrbj1 in
https://github.com/jagi/meteor-astronomy/issue_comments#issuecomment-437563068
https://github.com/jagi/meteor-astronomy/issue_comments#issuecomment-437563068
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jagi/meteor-astronomy/issues/691, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAKXU_2BmOwTMvv3ipYhEq6sQ5x3uxeks5utncugaJpZM4YX3DC
.

What info do you need?
The definition for this field is:
totalCm: {
type: Number,
optional: true,
},

You have to create reproduction repository. Without that it's hard to tell. When you will try to replicate this error on the smaller code base it will probably appear to be caused not by Astronomy but by something else


The ceph client should be present not latest

The ceph clients, if replaced, can cause VM outages on ceph
environments. This change ensures that the ceph clients are always
present by default.
Signed-off-by: Kevin Carter<EMAIL_ADDRESS>

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.

recheck_bionic_no_artifacts_swift_deploy


Limitations

Curious if we can get the current limitations of this app - I am currently unable to use my corporate access to Windows 365 / AVD because they have enabled Screen Protection, which through testing the Android app (and web client for that matter) doesn't appear to work.

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

ID: f0fa52aa-a324-bfcc-373a-efc442f7b5b7
Version Independent ID: 1592d677-2272-b816-12a6-54b96ccc2b5d
Content: Use features of the Remote Desktop client for Android and Chrome OS - Azure Virtual Desktop
Content Source: articles/virtual-desktop/users/client-features-android-chrome-os.md
Service: virtual-desktop
GitHub Login: @dknappettmsft
Microsoft Alias: daknappe

@Niglb
Thanks for your feedback! We will investigate and update as appropriate.

Hi @Niglb - thanks for the feedback. Keeping a list of the supported features per app is not something we want to do as there are so many permutations it'd be confusing. What we decided to do was keep the specific detail on supported clients, versions, prerequisites, any known issues, etc. with the feature itself. For screen capture protection on Azure Virtual Desktop, you can find that here: https://learn.microsoft.com/en-us/azure/virtual-desktop/screen-capture-protection#prerequisites. Admins enabling the features should make their users aware of any prerequisites.
#please-close


Improve search

I have an app No Frills CPU control, when i try to write "CPU", there no matches.
If an app have a long name, search doesn't work

To match you would have to write " CPU". Space is interpreted to any string of charaters. Otherwise you can rename the app to have CPU first.


please divide introduction paragraph

As it is introduction first paragraph is too long with again long sentences, making it very hard to read. Please divide it to several paragraphs and change very long sentences with proper connection verbs to make it easier to understand.

As it is introduction first paragraph is too long with again long sentences, making it very hard to read. Please divide it to several paragraphs and change very long sentences with proper connection verbs to make it easier to understand.

Done.


[BUG] Pass parquet2 io errors correctly into arrow2

In https://github.com/Eventual-Inc/Daft/commit/7fe3dbceb3ca3ee0c555753f0a55bd7e0c57bba6, two changes were made that caused parquet2 io errors to be mishandled.
Firstly, a IoError for parquet2 was introduced.  However in the implementation of parquet2::error::Error for arrow2::error:Error, any parquet2 error that is not FeatureNotActive or Transport is transformed into an ExternalFormat error.
Secondly, arrow2 IO errors are intended to be marked as ByteStreamErrors, primarily so that they are handled as transient errors and can be retried appropriately. However this special case was removed.
This PR makes now classifies parquet2 io errors as arrow2 io errors, and arrow2 io errors are now marked as ByteStreeamErrors.

Sorry for the regression, folks. I must have accidentally removed a From implementation that shouldn't have been removed.


Control width of grass at the base

What problem does this solve or what need does it fill?
It would be awesome to be able to control the width of the generated grass at its base. I'm currently using explicit mode based on my own mesh triangles, but my experimentation so far from the component settings has been fruitless. Perhaps a width setting, similar to height should be accepted.
Describe the solution you'd like
Just as users can declare the height of the grass, being able to declare the width of the grass at the base would be awesome.
Describe alternatives you've considered
N/A
Describe what is for debate
N/A
Additional context
Being able to generate skinny grass, instead of all the grass being rather wide at the base, would be nice in some contexts.

Hey @thedodd,
sorry for not replying, I had to concentrate on my thesis for quite a while and simply forgot.
I assume you don't need to answer anymore but I assume that other may have the same problem
Your suggestion to have a width component which can control the width at the base seem fair but I think it posses some difficulties and would make the api a bit more confusing to use as it slowly becomes unclear which settings refer to the chunk and which to the grass blades. It would raise the question, how to detect the base of the grass (given a generic grass mesh), to name an example.
There are actually already 2 ways to archive what you want:

You could use the scale field of the Transform component to down-scale in the xz-direction. This would be rather hacky and I'd not recommend that
The mesh used for the grass is actually just a example mesh for grass I set up. In this example I swapped the default grass mesh to a completely different one. If you'd want a thinner grass mesh, you'd have to take the default mesh defined here and make it a bit thinner

I'm not opposed to have a easier way to change the width with a component.
If someone stumbles upon this and would like to have this feature feel free to write a short comment so I now it is wanted

I achieved this by swapping out the grass mesh  but i agree it would be nice if the crate had built in controls for this perhaps.   Perhaps it could Mut the 'mesh' that is inside the handle in a simple way in a system.   on   of some settings resource or something.


Upgrade black version to 22.3.0

Due to problem in black with incompatibility with click v8.1.0 (https://github.com/psf/black/issues/2964) PTF CI started to fail on format check.
According to the link, the solution is to upgrade black to version 22.3.0.

@antoninbas yes, I have fixed it and it seems everything is passing now.


Functionality to choose pet size with a number value

Please describe your idea.
Currently we have three values for pet size: nano, medium and large.
I would love to have more fine grained tuning by using a decimal number value for size.
Describe how the idea would be helpful
Could help the users better fine tune their pet sizes.

Just going to bump this up as i was trying to find a feature for sizing since i usually have to squint to find the little guy


Order of distinct and top is wrong

When using both, distinct will come after "top 20" which results in an SQL error.

Anyone to create PR and test locally?

Does it have or need tests? We have it in production at one customer and no issues since i vreated the ticket so far

If it's already proven, then just create an PR pls.

Just created a pr. Wasn't able to run the sqlsrv tests ("test") but all other tests run well.


fix the directive videogular document

Hi guys,
I find some confused description in document and try to make it right
CHANGE

remove duplicate vgPlaysInline comment
revise vgNativeFullscreen param from function to boolean

Hi @snese! Thank you for your PR :)
vg-plays-inline is different from vg-native-fullscreen.
 Property vg-plays-inline disables this iPhone feature and basically adds the attribute webkit-playsinline to the video tag.
On the other hand vg-native-fullscreen disables native fullscreen for all devices and just makes that videogular fills the whole document and sets the z-index to the highest on the document.
So this is not duplicated information as you can see.
Closing.

Hi @Elecash,
Thank you for the quick response, awesome!
I want to point out that vg-plays-inline in app/scripts/com/2fdevs/videogular/directives/videogular.js
line 13: @param {boolean} [vgPlaysInline=false] vgPlaysInline Boolean value or a String with a scope name variable to use native fullscreen (default) or set fullscreen inside browser (true).
line 107 @param {function} vgPlaysInline Boolean to play video inline. Generally used in mobile devices.
According to the videogular.js  I think the line 13 is the correct description and the line 108 @param {function} ... it should be boolean !?

Yes, it should be boolean, I will fix the docs for the next release.


when create a project with the name "test", board do not give any prompt info

In GitLab by @wknet123 on Dec 8, 2017, 15:55

In GitLab by @yhua123 on Nov 8, 2019, 07:43
should sync project(namespace) when system boot up first time.

In GitLab by @luotian-github on Nov 8, 2019, 07:43
Fixed in PR#782

In GitLab by @sokril on Nov 8, 2019, 08:16
closed

In GitLab by @sokril on Nov 8, 2019, 08:16
assigned to @luotian-github


[glusterblock-provisioner] gluster-block mode has a problem.

This is what it says in the document:
gluster-block mode : This is an experimental or test mode on which the provisioner will directly talk to gluster-block utility or command line interface of gluster-block.
Preliminary judgment, gluster-block utility is not installed in the docker image [quay.io/external_storage/glusterblock-provisioner:latest]. I found the following wrong information in my use. Personally, I feel that this way can not be realized.Because gluster-block needs to be executed in containers running with gluster-blockd processes, and gluster-blockd depends on glusterd. It needs to be clustered in glusterfs. Do we need to add containers running the program to the glusterfs cluster? If I am wrong in understanding, ask the professional to correct.
The following wrong information is :
E0417 08:39:31.133443       1 glusterblock-provisioner.go:387] command &{gluster-block [gluster-block create block-test/dept-dev_default_claim1_d8f53aaa-421a-11e8-8d03-965ec404bb2b ha 3 <IP_ADDRESS> 1GB --json] []     []     0xc4202d2e40 false [] [] [] []  } failed,exec: "gluster-block": executable file not found in $PATH

@humblec strange error '"gluster-block": executable file not found in $PATH'
/area gluster/block


[Feature] 希望加入绝区零的启动

Checklist

[X] I have already read docs/Tips.md, but my feature is not implemented.
[X] My suggested feature was not mentioned by others, and it is not a duplicate feature.

Summary
最近绝区零即将上线，希望能够加入相关启动功能
Solution or Design
就像之前那样加入

#858

非常感谢:)


Twitter: Display Tweet directly in list view

Description

Show tweet directly in the list view
This is on be default, but can be turned off in the settings.
Closes #1534

Add character counter with validation indicator

Add support for thread in Send Tweet
User can now expand a single tweet to an thread by press CMD + +

Upgrade to 1.33 and modernize codebase accordingly

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer


Fix for older versions of pytorch

Current _get_device function will raise AttributeError if mps import was failed. This patch will handle such exception and set the device to cpu.

Great! Thanks for this pr. However, could you help us by telling us about your setup? Mostly your PyTorch setup so that we can be sure.

The PyTorch version has nothing to do with this commit. It's all about the try-import code.
https://github.com/HeliXonProtein/OmegaFold/blob/84574bb40aaa2d4c14caa7aeb6dd45f37abe98b4/omegafold/pipeline.py#L44-L47

ok!


Add "Cook" to related projects in the README

An overpower wordlist generator, splitter, merger, finder, permutator, encoder, decoder.. Frustration killer. Customizable. The Wordlist Framework.

https://github.com/giteshnxtlvl/cook
I'm sure someone here will find it handy :p

It might just be my untrained eyes, but that gist doesn't look like it actually says where the data comes from?
"original 41G stash"
maybe the magnet url would be more useful than the web-archive telling people to look up


How to monitor API？

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I have created a proxy API interface now, and I want to listen to this API interface. How should I configure it?
This API interface returns a 403 status，But this status code is the result I expected
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.8
💻 Operating System and Arch
Debian 10 x86
🌐 Browser
Google Chrome
🐋 Docker Version
No response
🟩 NodeJS Version
No response

I want to listen to this API interface

It may be able to call the API via the monitors included.
This entirely depends on what type (gRPC, http/JSON/XML, MQTT, ...) your API is.
The only clue you gave is that it reurns a status of 403 which likely implies that you want to monitor an http endpoint. (=> use one of the https monitors depending on what you want to monitor)
What your definition of up is not something you have specified. (does a 403 already mean up or do you need to auth agains your API first?)
Currently, this issue does not hold enough details to help you in any meaningful way.


[Azure.Core] Add Public Model Serialization

The goal of this PR is to provide a public interface that exposes the currently internal serialization code so customers can access it without needing to use reflection or write their own translation. This interface would be applied to all models in all of our libraries today. The implementation simply uses the existing internal serialization logic since that code already knows all of the property name changes / structure changes that exist. We can at a future date change the underlying implementation as long as we maintain functional compatibility.
The SerializationOptions class introduces a couple concepts that are common across the other languages serialization implementations which are:
IncludeReadonlyProperties: This flag tells the serializer to include the readonly properties when making the payload. This is always off during client calls since the service would error out if we tried to change a readonly property, however if a customer wants to save the state of the model this can be included by setting this property to true.
HandleAdditionalProperties: This flag tells the serializer to keep any properties it didn't recognize during deserialization as an internal dictionary. If the same model instance is then serialized with the same property set to true these values will be echoed and not lost. This is useful for dealing with a large enterprise where various micro-services could be on different versions of the azure sdk library.
We should consider what defaults we want for the above properties. I would imagine most customers would want true for both in most cases and we can simply have false for both in our client invocations.
Currently the interface only takes in Stream as the input and output object we should consider other convenience overloads such as string.
We should also consider explicit / implict cast operators Model -> RequestContent and Response -> Model.
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

Re-creating this PR as we had to rename the branch

https://github.com/Azure/azure-sdk-for-net/pull/35742

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core


[Question]: Mechanism to generate slug from title

Is there a way to take a title such as "Don't pay ridiculous amounts of money..." and cast it as e.g. "dont-pay-ridiculous"?
Something along the lines of python-slugify.
If so, how do we deploy it in the extension's options/settings?

The following is a screenshot from the Custom text information in the extension options:

So in your case, you could do something like {pageTitle:kebab}. It won't trim the title like python-slugify might? But it will give you something like dont-pay-ridiculous-amounts-of-money


remove extra blank log

Remove extra log
fixes #1644

That extra log was added to separate each tick's log and will stay there.

@douglascamata why is that needed?


Merge dev into feature/platformcache

This pull request was automatically generated because an automated merge hit a merge conflict

@njjc - gonna close this PR and delete the branch. I'll grab a copy of the POC diff.


Fix missing alias for ConnectionError

ConnectionError is used in  @type error :: Error.t | ConnectionError.t (see xandra.ex), but not aliased at the top of the module.
This results in dialyzer reporting an unknown type (Unknown type 'Elixir.ConnectionError':t/0) and also in a missing link in the documentation for t:Xandra.error/0.
The PR adds an alias for ConnectionError.

Thanks @jvf! 💟


Warn when PACKAGE is missing in DEPENDS list in rocm_export_targets

Hi @pfultz2, is there any work required for this ticket?  Can I close it?  Thanks.

is there any work required for this ticket?

Yes.

Can I close it?

No.


Implement Sign/Verify Protobufs changes

Initial PR for Feedback to implement Key Sign and verify
This change implements the protobuf changes described in https://github.com/microsoft/wssdagent/pull/663

Module="admin"

dont think this should of been added, will remove

Refers to: rpc/gen_proto.sh:30 in 5b5b30a. [](commit_id = 5b5b30af8a653ed238ad65672292452cfa17e724, deletion_comment = True)


Falcons - Lions game says scorigami though it happened 12 times before.

I saw that too. I mentioned in bugs as well how The reason there was confusion is that the square was placed at 22-22 Tie, which would be Scorigami, but the final score was 23-22, which would be a no-go. I think the machine that finds out Scorigami's is bugged (Unless Jon himself manages this.) But whatever the case, I certainly got excited when it said Scorigami, and wes then disappointed when it was not.

same as https://github.com/ACMerriman/scorigami/issues/21


Errors while exporting

While exporting the assets,it occurs some errors:
Assets\FlashTools\Scripts\Editor\FTEditor\SwfEditorUtils.cs(235,54): error CS0433: The type 'CompressionMode' exists in both 'Ionic.Zip.Unity and 'Unity.VisualScripting.IonicZip
Assets\FlashTools\Scripts\Editor\FTSwfTools\SwfStreamReader.cs(199,25): error CS0433: The type 'ZlibStream' exists in both 'Ionic.Zip.Unity and 'Unity.VisualScripting.IonicZip
I do not know how to fix...

Hello! Yeah, just remove Assets/FlashTools/Plugins/Editor/Ionic.Zip.Unity.dll.
It happens because modern Unity versions use and include some zip library that I use and include too (as far as I remember in the visual scripting package).

thanks for answering.Now i have another question.
I want to know whether it can control the layers?
Because in .fla some layers may not be displayed in certain circumstances.I want to control which layer should be displayed.
Does it support?

No, you can’t control a visibility of layers, because it no exist in compiled swf. Also, you can’t control separated parts of an animation, because all this parts are baked to a frame geometry.

OK,I see.So the proper alternative is to separate the whole .fla into several complied .swf and use them independently?

Оr separated movie clips in one swf and play them as two swfclip.

Ok,i see.Thanks for replying.It is powerful enough for me! Such a good tool.


Remove duplicated ansible.cfg configurations

This patch removes duplicated Ansible configurations that match OSA's
defaults. The ssh timeout was left in the file since it differs from
the OSA default (120 seconds in RPC vs 5 seconds in OSA).
RPC previously set forks at 15 for all deployments, but OSA uses a
safer mechanism to determine the maximum amount of forks. It sets a
hard limit of 10 but it lowers that limit to the maximum amount of
CPUs in the system if the CPU count is under 10.
Connects rcbops/rpc-openstack#1639

recheck_ceph

That darned tempest test!
20:17:37         Response - Headers: {'status': '404', 'content-length': '109', 'content-location': 'http://<IP_ADDRESS>:8774/v2.1/6ed426f8fdce4d17954765d93e49009a/os-volumes/13a8a7a3-0e76-4beb-a08c-1941dc6863cd', 'x-compute-request-id': 'req-27a5e158-7836-4b7d-9307-3bdbf2f40a89', 'vary': 'X-OpenStack-Nova-API-Version', 'openstack-api-version': 'compute 2.1', 'connection': 'close', 'x-openstack-nova-api-version': '2.1', 'date': 'Wed, 04 Jan 2017 19:56:11 GMT', 'content-type': 'application/json; charset=UTF-8'}
20:17:37             Body: {"itemNotFound": {"message": "Volume 13a8a7a3-0e76-4beb-a08c-1941dc6863cd could not be found.", "code": 404}}


Fix typespecs

SystemRegistry.Processor behaviour was wrong.
Inferred types for SystemRegistry api was wrong.

NOTE:
on Elixir 1.5.0 mix dialyzer will give an error:
lib/system_registry/processor/config.ex:90: The created fun has no local return
lib/system_registry/processor/config.ex:92: The call 'Elixir.Registry':match('Elixir.SystemRegistry.Storage.State',pid@1::any(),#{_=>#{}}) will never return since it differs in the 3rd argument from the success typing arguments: (atom(),any(),atom() | tuple())

This since been fixed in upstream Elixir, just not pushed to a stable release yet.

thanks 😄  ❤


Investigate scope of slack-bot for documentation

QTD app as slack-bot.

Closing since change of scope.


Fix #214: Improved interface hierarchy

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔

@vjik I need help with Psalm here.

Done

Thank you!


Discover tasks

Fixes #29
This PR adds 2 cooperating mechanisms for discovering tasks:

An import_modules argument to worker that will import the given modules, ensuring that the tasks in those modules are registered correctly
The name of the task in the db is now the importable python path of the task, and when the worker encounters a task it doesn't know, it tries to import it on the fly.

This gives us both power and flexibility. Plus, the 2 mecanisms interact: when a task is loaded dynamically, it logs a warning to help you realize you probably have forgotten to put a path in the discovery.

Codecov Report

Merging #30 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #30   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          10     11    +1
  Lines         309    357   +48
  Branches       17     24    +7
=====================================
+ Hits          309    357   +48

Impacted Files
Coverage Δ

cabbage/store.py
100% <100%> (ø)
:arrow_up:

cabbage/exceptions.py
100% <100%> (ø)
:arrow_up:

cabbage/tasks.py
100% <100%> (ø)
:arrow_up:

cabbage/worker.py
100% <100%> (ø)
:arrow_up:

cabbage/utils.py
100% <100%> (ø)

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 298a5f9...f36aacf. Read the comment docs.


Add persistent states for export with debug, as patch on editor restarts

Fixes: https://github.com/godotengine/godot/issues/98754

I tend to agree with this statement in the bug report:

In my humble opinion, this boolean shouldn't even be in the window there, but part of the export options, but that's perhaps more of a proposal than an issue report.

(Also with the fact that it's proposal material and thus not something to handle here.)
So until we do decide to change this, I agree with making the boolean on the export dialog persistent.

Thanks!


Please version downloads

Cloud SQL Proxy broke for us in the upgrade from v1.11 and v1.12; one reason was because of the ulimit change (systemd, at least on debian, requires additional configuration to cope).  I deploy the proxy using Ansible throughout a multi-system development environment-- the Ansible recipe fetches the proxy software from dl.google.com and installs it.
Recently, we couldn't control when we started to wind up with systems running v1.12, because the download URL is always the same between versions-- e.g. https://dl.google.com/cloudsql/cloud_sql_proxy.linux.amd64.
So it would really help if the download URLs could be versioned, e.g.
https://dl.google.com/cloudsql/v1.12/cloud_sql_proxy.linux.amd64.  Ideally, please also publish SHA256 hashes for each version so that automation can check that the binaries are intact as expected.
Thanks!

We have had an internally filed issue with the same request, thanks for filing the issue, I'll up the priority internally!

As of 1.13, we've (finally) started versioning the download links. See https://github.com/GoogleCloudPlatform/cloudsql-proxy/releases


Correct some things

I tried to correct all that mess, now everything seems functional.
I hope my small modifications doesn't break anything.
Also I think 4 instead of 8 for indent size will be better (I won't modify until you want).
Some things can be replaced by variables define at the top, let me know what you thing about these.

Thank you,
It looks like it works on iSH and Termux :)
For indent size if you think 8 is better you can commit it


Run over arbitrary duplex stream

It'd be nice if there was a way to just shove a duplex stream from somewhere into this and have it work from that rather than specifying URLs and making assumptions about the transport medium.

yes, totally agree. Thanks for your suggestion.

You may want to check out Clocksy. It is a tiny library, far less ambitious than timesync, but is transport-agnostic and disables itself when it detects it's running in a hidden tab (some browsers, e.g. Chrome, optimize timers in such a case, interfering with the algorithm).


routeMiddleware laravel-cors is not working?

I use laravel5.4 + dingoapi+ jwt build my api services.when i use laravel-cors in dingoapi ,i find something wrong.
routes/api.php as:
$api = app('Dingo\Api\Routing\Router');

$api->version('v1', function ($api) {
  $api->post('login', function(){
      return response()->json(['code'=>20000, 'token'=>'fsdf234']);
  });
});

add \Barryvdh\Cors\HandleCors::class  kernel.php as follows:
    protected $middleware = [
       .....
        \Barryvdh\Cors\HandleCors::class,
    ];

I request http://myapi.app/api/login is success.
But laravel-cors is work for every route,i dont't want this. so i change kernel.php ：
    protected $middleware = [
       .....
       // \Barryvdh\Cors\HandleCors::class,
    ];
    protected $routeMiddleware = [
        'auth' => \Illuminate\Auth\Middleware\Authenticate::class,
        'auth.basic' => \Illuminate\Auth\Middleware\AuthenticateWithBasicAuth::class,
        'bindings' => \Illuminate\Routing\Middleware\SubstituteBindings::class,
        'can' => \Illuminate\Auth\Middleware\Authorize::class,
        'guest' => \App\Http\Middleware\RedirectIfAuthenticated::class,
        'throttle' => \Illuminate\Routing\Middleware\ThrottleRequests::class,
        'mycors' => \Barryvdh\Cors\HandleCors::class,
    ];

routes/api.php
$api = app('Dingo\Api\Routing\Router');

$api->version('v1', ['middleware'=>'mycors'],  function ($api) {
  $api->post('login', function(){
      return response()->json(['code'=>20000, 'token'=>'fsdf234']);
  });
});

Then request http://myapi.app/api/login is failed.
preflight request is failed,server return 404.
why?Thanks for any helps.

This is not a Dingo Api Issue, and looks like a bad instalation/configuration of Laravel Cors.
According to the Laravel Cors documentation, "If you want to allow CORS on a specific middleware group or route, add the HandleCors middleware to your group:"
`protected $middlewareGroups = [
'web' => [
// ...
],
'api' => [
    // ...
    \Barryvdh\Cors\HandleCors::class,
],

];`
Please look at:
https://github.com/barryvdh/laravel-cors#group-middleware


Live Reload CSS Changes

I've cloned this project and I've been able to make it run as expected, whilst live-reloading when the TypeScript files change. However, when I change the CSS files (or the SASS files, using the Sass plugin for Stencil) - the changes are propagated to the relevant dist/collection/{component}/{component}.css files.
The  generated by Stencil also updates the CSS. However, there is a style tag just below the one that gets updated that uses a more specific, stale version of the CSS.
Is this a known issue with a workaround?

So sorry I didn't get back to you on this @justjoeyuk.  I have noticed this myself as well, I haven't spent much time debugging it, but it sounds like you have nailed it down.  Have you made any progress on this?  If not I will try to take a look, but I can't promise I will be fast, we have been swamped. 😅

Unfortunately, I didn't manage to - but it looks like the team building Storybook are making Web Component integration a very high priority, so seems like a temporary pain point until we can get stencil working out the box!

They are in talks on how to fix this on Storybook's Github.  I will drop the link here so that we have reference.
Framework support - Stencil.js #4600


Switch OS binary build and verification CI to Engflow RBE and github actions

The build part of this uses a cache and RBE altho that is not so important in this case (#32469)
Some of this CI requires ARM vms - which is setup if not currently used
I dont think we have any ARM RBE currently available

completed


Improve README of this repo

Below are some improvements that can be made to this repo that will help users who land here
Logging this issue to get a consensus on the suggestions. Actual changes will follow once consensus is reached.

Not all services have isomorphic Javascript packages i.e not all run in browser as well as Nodejs. We should call it out and ask users to go the respective folder READMEs for further info.
Add a line about the folder structure and where to go from here.
Add a line about the difference between this repo and azure-sdk-for-node
Have a good explanation of what is control vs data plane. External users would have no knowledge of the difference between the two
The Documentation section doesnt provide much value other than the link to https://docs.microsoft.com/en-us/javascript/azure. Since this is already covered in the "Need Help?" section, this can be removed
Before we get to "Authentication", we should have a note on Samples and where to find them. This is where most users might want to go to understand how to use the libraries. Each samples folder in this repo should have a README with steps on how to run the samples
License. All our libraries use the MIT license, so why talk about the auto generated libraries specifically?
Contribute: Apart from the initial few lines, the contents of this entire section needs to move to Contributing.md
Client Library Tested Operating Systems and Node Versions: Use something other than "x". It might convey the opposite of what we want to convey to certain readers

@KarishmaGhiya, Do you want to include anyone from your team in this discussion? A few points here apply across languages as well
cc @bterlson and @AlexGhiondea

Closing this issue as the readme has gone through multiple revisions since this issue was logged


[cluster_test] Pre allocate node manually - Flexible config 1/n

This diff is part of series of changes that aim to move away from libra_init usage in cluster test and generate configs inside cluster test binary. This will allow for higher flexibility for different scenarios and will make cluster test setup closer to production (that is not going to use libra_init).
In this diff we pre-allocate node manually, instead of relying on kubernetes to do so. This will be needed later to populate config on the node before running it.
Also, update job_template to allow utilities to access node data dir.

@bors-libra r=ankushagarwal

:pushpin: Commit 737a70c has been approved by ankushagarwal

:hourglass: Testing commit 737a70cf1ab69ddf62570270efab8babfb47385d with merge bb83ab0cc3dbb5531ac5e131939d3a11f7889768...

:broken_heart: Test failed - checks-actions_land_blocking_test

@bors-libra r=ankushagarwal

:pushpin: Commit 4eae83f has been approved by ankushagarwal

:hourglass: Testing commit 4eae83f604f8f57cfdc5d0b1fdfe0e98a79bd9e0 with merge 5e1b05d9f7932b18097c704e63b9224046917ba3...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: ankushagarwal
Pushing 5e1b05d9f7932b18097c704e63b9224046917ba3 to master...


htmlgen: use XRAY_ROI_TILEGRID instead of XRAY_ROI

Signed-off-by: Alessandro Comodi<EMAIL_ADDRESS>This PR fixes https://github.com/SymbiFlow/prjxray/issues/1264.
It changes to the usage of XRAY_ROI_TILEGRID instead of XRAY_ROI. This is more accurate as it corresponds to the whole device (or at least the one described by tilegrid.json).
In addition XRAY_ROI_TILEGRID is mandatory to have in the settings files

LGTM.

@acomodi Can you look at getting htmlgen run as part of the CI for this repository?

@acomodi Can you look at getting htmlgen run as part of the CI for this repository?

Done, I have added it in Travis CI, as well as a script to run the htmlgen on all the setting files

Hmm actually this won't work in Travis, unless I add also the download of the db in the script, to get the tilegrid info. Probably needs to be added in Kokoro?

@acomodi Kokoro or Travis is fine. Testing with make download-db could also be fine?

@mithro Sure, I'll go with the make download-db solution then

Maybe as a separate travis build stage or something?

So, re-thinking at this I would say that, in case there is a PR that adds an additional part, with a new tilegrid, the database would be incomplete, therefore the test would result in a failure.
I believe we should add this test as a Kokoro stage at the end of the build

@acomodi Whatever you think is best.

@mithro All the CI build went green. I see that Kintex and Tests CI are marked as still running, but they actually went green (unsure why this happens).

Okay merged it.


Double escape for special characters in links

It seems that the text of links is escaped twice. For example: [a'a](/) is shown as a’a, that is a&rsquo;a in the source HTML.

I tested it and it's not happening with other themes, so I imagine it has to do with this theme. I tried looking at the code but couldn't figure out why this is happening. I'm not very familiar with Hugo, but I'll gladly help to fix it if someone could point me to the source of this issue.

It was my fault


Bug fix in the C++ simulator: density matrix and probabilities are sometimes erroneous

Thanks to Guy Szweigman and Mohammad Heeb, who noticed that the C++ simulator does not always display correct density matrix and probabilities vector.
This happens in the presence of an optimization, which applies to no-noise circuits where all measurements are located at the end. The optimization replaces the shots by a single shot + sampling. However in the calculation of the density matrix and probabilities vector there is a normalization by the number of shots. This results in a density matrix whose trace is 1/(number of shots), and a probabilities vector whose sum is 1/(number of shots).

Description

I fixed the number of shots when the optimization is applied.
Independently of this pull request, it should be revised whether it is correct to normalize by total_shots. Looks like there is an infrastructure to allow, in the future, that total_shots will be the accumulation of number of shots in executions on different engines. What happens if one of the engines does not support density_matrix?
Motivation and Context

How Has This Been Tested?

I updated test_qasm_simulator_cpp.py
Note: test__simulator_online_size in test_quantumprogram fails in the system.
Screenshots (if appropriate):
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[ ] All new and existing tests passed.
test__simulator_online_size in test_quantumprogram fails in the system.

I added a comment requested by Chris and rebased against the master.


Better management of verified device list

The recent early May 2019 Android update forgets (again) all the verified devices.
This matters because some users actually invest time to verifiy the devices.
It also matters for adaption because less tech-affine users get easily annoyed when yellow triangles start to appear again after an app update, and they ditch riot quickly.
Similar to key backup, one could for example give users a manual and automatic way to backup of the verified device list.

This also happens if you re-login on the same device immediately after logging out.
Your device is assigned a new device id and all the verified devices need to be re-verified.


Add: Queue implementation using Arrays

Hey ! @amritansh22 , the maintainer,

[x] Queue implementation using array.
[x] Switch case used for each type of operations performed in queue.

Please star this repository to show your support, in the meantime I am reviewing your code.

Excellent work @Uyadav207  Thanks a lot for making this PR.

@amitsaini4556 , Thanks


Append flat check note to spells under non-spellcaster Ancestral Curse

It looks like I missed the class predicate for moderate and major stage. Will fix tomorrow.


the data selected in data_process.py

Sideburns   Blurry  Wearing_Hat Double_Chin Pale_Skin   Gray_Hair   Mustache    Bald
Why these data are selected in data_process.py？

In our work, the Imbalanced ratio (IR) of each classification, defined as the proportion of minor class, is less than 7%. These traits  are chosen from the label distribution that has the imbalanced ratio under 7%


Port clients of ScrollableMixedWidgetList to LazyBlock

LazyBlock is going to replace ScrollableMixedWidgetList at some point.

@Hixie @HansMuller


Compatibility with Cloud Shell. Moved deck to port 8081

The Web Preview feature of Cloud Shell only lets you tunnel ports 8080-8084. The Spinnaker UI was running on port 9000, and when the UI would hit the /gate/credentials endpoint, it would redirect back to the ${services.deck.gateUrl} spinnaker config value which was set to http://localhost:9000 and the UI would break.
Switching the port to 8081 lets the redirect pass through a working port for the web preview proxy.
Also added the Content-Type: application/json header to all /gate responses via apache config. This is done because the Cloud Shell web preview proxy does some content-type detection and breaks Deck for several of the gate API calls.
This PR also upgrades the spinnaker components to the latest versions.

Thanks @danisla!!


Moving quicksight (2) resources to v1beta1 version

Moving 2 resources in the quicksight group:

[ ] aws_quicksight_data_source
[ ] aws_quicksight_group_membership

Need more knowledge about config types in externalname.go.

Configurations that I've tried can be found in the file configurations.txt
configurations.txt

Are there any plans to release this in the near future?
Is there any porting guideline on how one can contribute to make this feature implemented?

@project-administrator, it is not a near-term focus, but we would be very happy to provide guidance and support if you are interested in contributing to it.
You can find the quick guide for adding resources here: https://github.com/upbound/upjet/blob/main/docs/add-new-resource-short.md (there's also a more comprehensive one here)


Add a translation

I propose to adapt the application to the multilingual version. After the application is released on F-Droid, there will be users who would like the application to be in their native language.

@Valdnet Thanks for the proposition. But the current maintainers are only well versed in English. Let me know if you'd like to work on this issue. You could add the language you speak or are well accustomed to.
Thanks!

You could add the language you speak or are well accustomed to.

First you need to adapt the code and extracting the text strings into separate files. If your application is developed in Flutter, the best solution is an .arb file.
Then I will gladly translate it into Polish 😉.

But the current maintainers are only well versed in English.

You don't need to know any other languages. The translation will be done by translators who know English and will translate into their native language and more. The best solution would be to add the project to Weblate or POEditor.

Thanks @Valdnet. Will let you know once we do the required changes.

Hi @Valdnet, I've added the arb file for Polish under {PROJECT_ROOT}/lib/localization/app_pl.arb. Could please add the translations and issue a PR?
Thanks!
Also I'm assigning you on this.

@danger-ahead Please add a commit with the following translation:
{
	"title": "Kalkulator",
	"about": "O aplikacji",
	"hapticFeedback": "Dotykowa informacja zwrotna",
	"theme": "Wygląd",
	"advanced": "Zaawansowany",
	"history": "Historia",
	"historyEmpty": "Nie ma nic do pokazania :(",
	"settings": "Ustawienia"
}

Thanks 😉.

Thanks @Valdnet. Closing this issue now.

@danger-ahead Please make the following change to the translation:
"theme": "Wygląd", -> "theme": "Motyw",
Thanks 😉.


Placeholder Error

Don't know if this has been reported but when using %parkour_current_personal_best% it shows the Prefix from the strings config and displays in front of it. So when on a scoreboard it gets very long like Parkour > No time recorded.
Version: 6.0 latest build
Thanks :)

Confirmed bug in 6.0.
Thanks for reporting it.

Fixed by Steve in previous commit.


Improve FunctionNode upgrade guide

Jenkins, test this please.

Jenkins CI test (for commit f1065dc33ed131779189e2fd538f808813ff937f, target branch master) succeeded!

Jenkins CI test (for commit f5f53516c09b8c61643fb1913b00fe1096eec867, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins CI test (for commit f5f53516c09b8c61643fb1913b00fe1096eec867, target branch master) succeeded!


Material Theme Builder

I am using the Material Theme Builder to create a new theme with custom colors. However, when I replace the colors in my design and export the Flutter file, upon checking the color_schemes.g file, I notice that the colors do not match the ones I added. How can I address this issue?

Hello, Could you provide steps to reproduce along with a screenshot?


Need to optimize the memory usage of get user image methods

Currently, memory usage is high in get user image method according to the performance test.
Suggested solution: Change the implementation using java channel API.
Priority: High
Severity: Critical
Version: 1.01
Type: Improvement

Moved to https://github.com/wso2/product-is/issues/7141


Add grow_policy='lossguide' with tree_method='exact'

Add ability to use grow_policy='lossguide' with tree_method='exact' (currently supported only if tree_method is set to hist, approx or gpu_hist)
BTW, currently this params combination works without error but max_leaves param is ignored (max_depth is used instead)

Apologies, we do not plan to add the lossguide policy to the exact method.


Add ability to add custom LED order matrix

I have an LED matrix PCB with 4 LEDs per button. The LEDs aren't grouped in button order. I had some code on another microcontroller to deal with this. Please refer to the code block below. Also, using this scheme perhaps allow a low current version that only uses 1 LED from each button.
//Holds Button 1-20 LED location. Order NW, NE, SW, SE. ie. buttonLEDs[0] houses position of button 1 NW LED, buttonLEDs[4] houses position of button 2 NW LED. LED starts at position 1.
int buttonLEDpos[80] = { 25, 26, 24, 23    ,    9, 10, 8, 7   ,   27, 28, 22, 21    ,   11, 12, 6, 5    ,
                       29, 30, 20, 19    ,   13, 14, 4, 3   ,   31, 32, 18, 17    ,   15, 16, 2, 1 };

This change to neopicoleds.cpp works as a hack. I'll try to implement an better change tomorrow and submit
Line 334:
`
std::vector<std::vector> NeoPicoLEDAddon::createLEDLayout(ButtonLayout layout, uint8_t ledsPerPixel, uint8_t ledButtonCount)
{
vector<vector<uint8_t>> positions(ledButtonCount);
int buttonLEDpos[80] = { 25, 26, 24, 23    ,    9, 10, 8, 7   ,   27, 28, 22, 21    ,   11, 12, 6, 5    ,
29, 30, 20, 19    ,   13, 14, 4, 3   ,   31, 32, 18, 17    ,   15, 16, 2, 1 };
/*
for (int i = 0; i != ledButtonCount; i++)
{
positions[i].resize(ledsPerPixel);
for (int l = 0; l != ledsPerPixel; l++)
positions[i][l] = (i * ledsPerPixel) + l;
}*/
for (int i = 0; i != ledButtonCount; i++)
{
	positions[i].resize(ledsPerPixel);
	for (int l = 0; l != ledsPerPixel; l++)
		positions[i][l] = buttonLEDpos[ (i * 4) + l ] - 1;
}

`

Hey @waterbury ,
Would you mind re-submitting this on the new GitHub?
https://github.com/OpenStickCommunity/GP2040-CE/issues/


chore(*): update polkadot versions to match SDK v22 versions

Description
update polkadot versions to match SDK v22 versions
Breaking Changes
None
JIRA Link

Checklist

[ ] Updated the Readme.md (if required) ?

Also, marking the commits as chore/ci won't release a new version

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:


Deprecate/Remove configure in favor of modify_config

The latter is newer and allows method chaining

Maybe this should be called .init() or something, since it enters initialization mode


Revert "chore: rename copy-annotation to copy-annotations"

Reverts kellyselden/move-semver-tags-action#17
The breaking change didn't work?

I should have run with --dry-run first.


Moving portal (1) resources to v1beta1 version

Moving portal (1) resources to v1beta1 version

[ ] azurerm_portal_tenant_configuration

User has to be Contributor or Owner at scope / for managing this resource.

User has to be Contributor or Owner at scope / for managing this resource.


knot-dns: 3.1.9 -> 3.2.0

This version brings various XDP-related features and improvements
(namely QUIC and TCP) and many more:
https://gitlab.nic.cz/knot/knot-dns/-/tags/v3.2.0
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Ah, unit tests are broken on 32-bit.  We might wait for a bit while upstream looks into that, though it's probably uncommon use case combination, nixpkgs + 32-bit DNS server.

Resolved.

Oh yeah, I forgot to package the QUIC support 🤦🏽  I'll do that soon.

https://github.com/NixOS/nixpkgs/pull/188383


Fix DataFactory README and bump central version

API change check
API changes are not detected in this pull request.


How to judge a stranger in my dataset?

.  I want to achieve real_time_face_recognition py, it mistook a stranger people in the dataset, it should be the most similar as the stranger in the dataset, then how should I process to determine the stranger unknowner?
Thank you very much

@Tomhouxin try other classifier, if you find the classification results are very close to each other.

hi @Tomhouxin What does the "threshold" mean in the SVM classifier？
1） check the probability by model.predict_proba(emb), if the max one is below a threshold, then it is a stranger?
or
2) add a new class called "unknown" in to the classifier?

@Tomhouxin 所以你还是增加了一个新类叫stranger吗？和embedding预测的类别相似度小于阈值的都被分类为这个类别里？

@DawnHH only add a threshold

@DawnHH yes

Hello @Tomhouxin ,
Did you solve your problem ?

@menon92 , yes !

Thanks @Tomhouxin
Could you please help me that how can I do this. I have the same problem. I try to calculate `l2 distance' of two image. But it's not working well
Thanks

sorry，I'm not sure how good it is to calculate the distance. I'm using the classifier (SVM), but I'm also going to try to calculate the distance directly

@Tomhouxin
Do you just use classifier (SVM) prediction with a fix threshold to  judge a stranger ?

@menon92 ,yes ,you are right

@Tomhouxin
But it's not working for me. what could be the cause of it . However my image quality is not very good

I don't know exactly what your question is, can't you find the appropriate threshold?

@Tomhouxin
Yes, I can't find the appropriate threshold for classifying the unknown images.


client: add dynamic password (pass a function instead of a string)

This adds the possibility to pass a function to the identity.password option. It may return a string or a Promise resolving to a string, which will then be used as the connection password.

Thanks!


Some questions about Multi-GPUs

Thanks for sharing your excellent work, but I have a question if I want to run this code with multi-GPUs, how can I modify the code?
By the way, what's your GPU using for training this network, Tesla V100 or others?

Thanks for sharing your excellent work, but I have a question if I want to run this code with multi-GPUs, how can I modify the code?

You can leverage Data Parallelism in pytorch to employ multi-gpu. Please refer to https://pytorch.org/tutorials/beginner/former_torchies/parallelism_tutorial.html for the usage.

By the way, what's your GPU using for training this network, Tesla V100 or others?
We used NVIDIA GTX 1080Ti to train our model.


Lpfg 893 - remove actions from archived course in learner record

Added status to course object in model.ts
Removed actions from course overview if course has been archived

Closing as per your request 🤠


--coverage-file option does not work

I use --coverage-file <some-file> option with krun hoping to get coverage report, but I do not see any output file. Is it because the option does not work or am I missing something?

A quick look show that the option does not work because the option (defined in KRunOptions.java) is not used anywhere in the project. But it seems that it was implemented in another commit (@5413919) by @daejunpark. Is it possible to merge that into current K ?


Resolved by #16


Too much memory allocations

What version of Flyway are you using?
4.X
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
maven
What database are you using (type & version)?
Oracle 12c
What operating system are you using?
Linux
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
We have tons of database to migrate and the migration script is very long. We noticed that the migration spends a lot of time performing GC when we run a lot of data migrations in our integration tests. It turns out that SqlScript class uses a lot of String split, replace, toUpperCase, trim, replaceAll, etc which generates a lot of string garbage. I wonder whether it's possible to stream line them a little bit such as:

Use Pattern/Matcher.replace instead of String.replace which creates a Pattern every time.
Use Matcher.appendReplacement whenever you can

Please try again with 5.0.7 as there have been a number of changes in this area.
The streaming part is related to #615 . I'll look into the 2 other suggestions as well.

Any news?

We have now rewritten a number of string handling functions and profiling indicates that things are much better now.

@axelfontaine  I just tried to upgrade to 5.0.7 from 4.2.0, it seems to break our schema deployment. It looks like some of the scripts seem to run twice. I'm trying to track it down


V1.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

the tests are run from here, by the way. So currently it still attempts to run on nose

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@t0266, any idea of what's causing the new dask error here? All tests are passing in the release and the code haven't changed and this was passing a couple of days ago... dask is 0.13.

I found the issue, that sdist contains and obsolete c file. I'll fix it.

Since it takes so long for CI to finish (actually travis macOS takes days) I'll merge to try to get it into anaconda asap. (There is no drawback AFAIK.)


Adjusting ASCII art logo slightly

corrects alignment of lines over 'r' and 'e'
separates line over 'z' and 'u' to be consistent with other letters

@lodejard Can you rebase your changes on top of dev?
That should fix CI then we can merge.


pacbio fork of htslib does not install cram/md5.h

pbbam source files expect to be able to include the header file "cram/md5.h", which is part of htslib, but the pacbio fork of htslib does not install this header file.

Please report the executed commands and the error observed.

pbbam's src/MD5.cpp contains this line: #include <cram/md5.h>.  This assumes that this file is in one of the include paths.  When building pbbam with "-DHTSLIB_INCLUDE_DIRS=/path/to/pacbio-htslib/include" where /path/to/pacbio-htslib is the install prefix of the PacBio fork of htslib 1.1 then the build will fail.
In the Guix package for the PacBio fork of htslib 1.1 I fixed this by also installing all headers in the cram directory of htslib.

Htslib is bundled within pbbam https://github.com/PacificBiosciences/pbbam/blob/master/third-party/htslib/htslib/cram/md5.h, I don't see the problem, plus you still haven't provided which tool has compilation issues

I know that htslib is bundled.  When providing packages for distributions, however, bundling is strongly discouraged, so I have separately packaged the PacBio fork of htslib and installed it.  The install target of the htslib fork does not install the cram headers.
pbbam offers a cmake flag to use htslib from a different location.  I'm pointing it to the installed htslib's "include" directory.

plus you still haven't provided which tool has compilation issues

I don't understand the question.  I'm building pbbam from source with cmake.  I'm passing the following flags:
-DHTSLIB_LIBRARIES=-lhts
-DHTSLIB_INCLUDE_DIRS=/path/to/pacbio-htslib/include
-DGTEST_SRC_DIR=/path/to/gtest-sources

Since you have already forked htslib it makes sense to alter its install target such that all required files are in fact installed.  Otherwise it won't be possible to build pbbam without using the bundled htslib sources (and that means that distributors will have to patch the sources or give up on packaging pbbam).

I just had the same problem -- did this resolve?  I am using straight out of the box pitchfork no separate build occuring

pbbam doesn't bundle htslib anymore, but uses your system-wide installation.

so -- I guess the pacbio pitchfork build needs to be updated to handle this -- what is your advice -- where do I find the system-wide installation --

This is a pitchfork issue, so best I can do is ping @mhsieh :)

Thanks for that -- I'll see if I can work around it.

what I did was comment out the htslib in the settings.mk and removed the directory from pitchfork_workdir and restarted with make isoseq-core lets see if this works.

so how do I get the version of pbbam that doesn't bundle htslib?

pbbam does not require the forked htslib anymore, any htslib 1.4 version and above should work.
Anyhow, please use the provided binaries from the SMRT link tarball or alternatively contact Tech Support for further assistance. Building our complete stack from source is daunting and requires advanced knowledge of buildsystems.


Fix multi-version testing build logic

There was a large drop in the number of tests here: https://builds.gradle.org/viewType.html?buildTypeId=Gradle_Check_Platform_Java7_Oracle_Linux_codeQuality&tab=buildTypeStatistics&branch_Gradle_Check_Platform_Java7_Oracle_Linux=master
Here's the build before the drop: https://e.grdev.net/s/u6qcp7eglzys4/tests/byProject?toggled=W1swXSxbMCwwXSxbMCwwLDBdXQ
Here's the build after the drop: https://e.grdev.net/s/qrn4n4234iwoi/tests/byProject?toggled=W1swXSxbMCwwXSxbMCwwLDFdLFswLDAsMSwwXV0
Receiver of hasProperty(name) and property(name) was the task hence not seeing the project properties. Fixed by making the ifProperty() utility method an extension on Project.
Simplified ifProperty() expression along the way.

Feedback build: https://builds.gradle.org/viewQueued.html?itemId=10700121

Build after the fix: https://e.grdev.net/s/en2g4szuldmuc/tests/byProject?toggled=W1swXSxbMCwwXSxbMCwwLDBdLFswLDAsMCwwXV0


fix(plugins/plugin-electron-components): UpdateChecker does not rende…

…r the changelog as markdown
Fixes #7497

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

/lgtm


Update sentiment-analysis extension to reflect latest Kyma changes.

Updated sentiment-analysis extension README with SAP Cloud Identity Services prerequisite for JWT protection of web hook handler function.  Also added logging to assist troubleshooting in lambdas/sentiment-analysis/hander.js and removed ExposedDestination entry from commerce-impex/projectdata-register-integration-objects.impex as this can only be done from the Commerce Backoffice interface.

Hi @ArthurPerry ,
Is the exposed destination not possible via Impex a recent limitation? I set up the demo and do not recall if that was the case. Although it is quite a while ago, so cannot be sure.


fix(#patch); aave v3; handle change to BalanceTransfer event

The value parameter of the BalanceTransfer event was updated in the Aave 3.0.1 contract update to be the scaled balance in place of current balance.
This PR interprets value based on current block compared to V3 -> V3.0.1 contract upgrade for that market.
Once Aave V3 markets are upgraded to V3.0.1 via Governance, the update blocks will need to be added to this helper function and subgraphs re-deployed.

Hey @defispartan thanks for this! Do you know when we can expect this to be deployed to mainnet so we can update the blocknumber accordingly?

Mainnet is already live, the block number is just set to 0 since this market will only have the updated events and the old logic will never be used. This subgraph should good to deploy with no other changes.
All of the other V3 markets will have a block number entry once they're upgraded.

This will be necessary to do once we upgrade aave-forks to the 3.0.0 schema and we are keeping track of Transfers. #1519

@defispartan , can you help clarify about the aToken upgrade?
It seems to me that the aToken contract has identical signature for BalanceTransfer since January, this is the first aToken created on Jan 27 according to ReserveInitialized events emitted by the PoolConfigurator contract, while this one the last one on Apr 10th. I see that the value argument is divided by index before BalanceTransfer event emitting on both contracts.
Are you saying that we should start to multiplying event.params.index with ``event.params.value` on later aToken deployment, but not on earlier ones? Which block did the switch/upgrade happen? and how can we tell? thx!

@dmelotik , appreciate your input too!

@tnkrxyz to me it seems like this change only affected the value from BalanceTransfer event. And the current aave v3 code doesn't need that event.
I don't think we even need to account for this then. Do you agree? And is there a reason we would need to add BalanceTransfer back in??

@tnkrxyz to me it seems like this change only affected the value from BalanceTransfer event. And the current aave v3 code doesn't need that event.
I don't think we even need to account for this then. Do you agree? And is there a reason we would need to add BalanceTransfer back in??

@dmelotik From what I found so far, we should switch to use BalanceTransfer for aave-v2 because the Transfer event gives transfer amount that has been discounted by liquidity index. It is trickier for v3, because they flipped the amount argument between Transfer event and BalanceTransfer in middle of aave-v3 deployment (see my description above), so my current implementation is get amount from both BalanceTransfer and Transfer event, and use the large number of the two. Let me know if this makes sense.

#1993 implements the BalanceTransfer/Transfer logic for aave-v3. We can close this issue once #1993 is merged.

Closing, this was tackled in #1993 Thanks all!


VKObserver doesn't reset the top margin back to original value

The VKObserver works really well. I've tested it with various scenarios, including when the edit box and memo box are inside a TListBox control (as I do in a lot of my UI).
There are 2 issues which I'll raise as separate tickets.
If the container control already has a top margin set, this is reset to 0 when the control loses focus as the code doesn't capture the original value.

Pushed a change that should fix this

Thanks!


Example of using iron-session with Fastify?

The library claims to be working with Fastify. Is there any example of how to do it?
Thanks.

@RIP21 There's none, this library is compatible with any server that uses Node.js IncomingMessage/ServerResponse classes. Is Fastify using something else?

Seems like fastify is using some custom things, I'll remove that sentence then. There's certainly a way to make it work but it will require some change in iron-session code I believe.

On the subject of Fastify compatability:
The main thing is that, on the Fastify response object, 'headers' is a function, not an instance of Headers  or something like it.
I've only looked over this briefly, but It appears like this will only cause problems in three places (in core.ts, at least). Two of these are obvious:
https://github.com/vvo/iron-session/blob/566056747a3931d7c901cc1c9351dc2590dad7a4/src/core.ts#L166-L170
and
https://github.com/vvo/iron-session/blob/566056747a3931d7c901cc1c9351dc2590dad7a4/src/core.ts#L227-L239
In terms of fixes, probably the easiest option is to make use of the vanilla Server Response object  Fastify puts on the raw prop of their response object.
The third problem is a little more subtle:
https://github.com/vvo/iron-session/blob/566056747a3931d7c901cc1c9351dc2590dad7a4/src/core.ts#L183-L187
The Fastify response object doesn't have headersSent as a prop (it's on that raw object), so it blows past this check. The underlying issue, however,  is that because Fastify buffers modifications to Headers, I don't think you can rely on the value of reply.raw.headersSent to prevent unwanted executions of addToCookies if Fastify is calling the shots.
If it were me I'd just do something like:
....
const cookieValue = cookie.serialize(
            options.cookieName,
            seal,
            options.cookieOptions,
          );

          if (cookieValue.length > 4096) {
            throw new Error(
              `iron-session: Cookie length is too big ${cookieValue.length}, browsers will refuse it. Try to remove some data.`,
            );
          }

if(!`${res.getHeader("set-cookie")}`.includes(options.cookieName)) {
          addToCookies(cookieValue, res);
}
....

But that's assuming there is no valid reason for cookieName to already be present in the  'set-cookie' header before the corresponding seal is appended.
Hopeuflly this post is useful to someone. I'll have some free time mid October to work on a pull request, if things haven't already been resolved.

I was able to get around this by simply passing req.raw and res.raw to getIronSession. A custom plugin/middleware could create the session:
const session = await getIronSession(req.raw, res.raw, {
  cookieName: 'myapp_cookiename',
  password: 'complex_password_at_least_32_characters_long'
})

If pull request #558 is merged it might also be a fix, but I'm not sure.


Fix jsons

Existing Sample Changes
Description
I noticed that some of the FPGA code samples were not correctly categorized in the sample browser, i have updated the appropriate sample.json files to fix this.
I've also promoted one of our code samples to 'Tier 1'.
Since this is only a documentation change, no testing was performed.
Fixes Issue#
External Dependencies
List any external dependencies created as a result of this change.
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Implement fixes for ONSAM Jiras

How Has This Been Tested?
Since this is only a documentation change, no testing was performed.
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Command Line
[ ] oneapi-cli
[ ] Visual Studio
[ ] Eclipse IDE
[ ] VSCode
[ ] When compiling the compliler flag "-Wall -Wformat-security -Werror=format-security" was used

@whitepau Do you need approval and merge?

Yes please

From: Jimmy Wei @.>
Sent: Thursday, January 18, 2024 10:37:39 PM
To: oneapi-src/oneAPI-samples @.>
Cc: White, Paul @.>; Mention @.>
Subject: Re: [oneapi-src/oneAPI-samples] Fix jsons (PR #2140)
@whitepauhttps://github.com/whitepau Do you need approval and merge?
—
Reply to this email directly, view it on GitHubhttps://github.com/oneapi-src/oneAPI-samples/pull/2140#issuecomment-1899241224, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AUQVN7PWJV75X4HFJTJMIZ3YPGI2HAVCNFSM6AAAAABCABL2ZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJZGI2DCMRSGQ.
You are receiving this because you were mentioned.Message ID: @.***>
Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de http://www.intel.de
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928


v-model value don't change with SELECT value in JqueryUI

There are two "select".
display "select" created by JqueryUI ,and real select hidden by JqueryUI.
Problem:
 v-model bind with real select ,display "select" change, real select's value also change(see in console),but v-model value don't change!?
I force show the real select and change its value by mouse,v-model value change.
v-model how to work with select in JqueryUI ?

Hello @MonKong
Thank your for your interest in this project.
However, your issue is a usage/support question, and the issue tracker is reserved exclusively for bug reports and feature requests.
We encourage you to ask it on the forum  or on gitter.


Use appropriate connector instance based on protocol

This is to fix #242

@mosabua @ebyhr I'm not really sure how such cases are being handled in Trino repo, but i can take a look. If there are any suggestions please let me know.

Can you fix the test as requested @prakhar10

Can you fix the test as requested @prakhar10

Sorry for the delay, got busy with something else. I'll fix this in a couple of days.

Should we just use the changes from #294 with the test from this PR and improve in this PR as next step? Could you do that @prakhar10 ?

Sure @mosabua , but I believe we started off with a similar change and with reviews we've come to this code that I have currently in this PR.
But if it helps and if everyone agrees I can bring in the code changes from that PR into this one and we can take it forward.

Should we just use the changes from #294 with the test from this PR and improve in this PR as next step? Could you do that @prakhar10 ?

Sure @mosabua , but I believe we started off with a similar change and with reviews we've come to this code that I have currently in this PR.
But if it helps and if everyone agrees I can bring in the code changes from that PR into this one and we can take it forward.

Ideally we just address the outstanding comments from @ebyhr ... and ensure the test works as expected. I really think we need to get this merged for the next release.

By the way, test isn't mandatory in my opinion. This is pretty small change and will be removed when migrating to Airlift.

Could you rebase on master to resolve conflicts?

Could you rebase on main to resolve conflicts?

Looks like this file does not exist anymore in the main branch, do we need this PR then @ebyhr ?

@prakhar10 Please take a look at HaGatewayProviderModule:
https://github.com/trinodb/trino-gateway/blob/3c202992dd84c31829421bb1e9c791c18ccbe787/gateway-ha/src/main/java/io/trino/gateway/ha/module/HaGatewayProviderModule.java#L168-L183

@prakhar10 are you going to move this change over to the new module? Even potentially without test .. just so we can get this merged and maybe ship as part of the 8 release?

Hi @prakhar10 , do you need any assistance with this?

We discussed this in the recent a dev sync.. the reason this issue occurs and the fix partially fixes is that we use two separate HTTP ports for UI and API. However with the airlift refactor we will drop this usage down to just using one port (like Trino) and therefore this fix is not needed.