[BUG] save() results in OperationFailure if use_revision=True and keep_nulls=False

Describe the bug
Defining a document with use_revision=True and keep_nulls=False results in .save() raising OperationFailure when invoked on new documents. I am using beanie==1.26.0.
To Reproduce
import asyncio
import os

from beanie import Document, init_beanie
from motor.motor_asyncio import AsyncIOMotorClient

class Parent(Document):
    name: str

    class Settings:
        name = "parents"
        use_revision = True
        keep_nulls = False

async def main():
    client = AsyncIOMotorClient(os.environ["MONGO_URI"])
    await init_beanie(database=client.test, document_models=[Parent])

    # raises OperationFailure
    parent = Parent(name="P1")
    await parent.save()

    # works
    parent = Parent(name="P2")
    await parent.insert()

if __name__ == "__main__":
    asyncio.run(main())

Expected behavior
I would expect .save() to successfully insert the new document, as stated in the docs.
Additional context
I did some debugging and - I may be wrong but - the issue seems to be with the update query:
{
    "$set": {
        ...,
        "revision_id": Binary(...)
    },
    "$unset": {
        "revision_id": None
    }
}

which tells Mongo to both set revision_id to a value and unset it. This is a result of this Unset operator, which specifies to unset revision_id (as it is, at that point, a top-level None).
(First issue here, thanks for the amazing library)

Hi, thank you for the bug report!
It seems there are at least two unique issues concerning keep_nulls=False usage. See also: #1048  (duplicate)


I can't install shaperglot if Python version is 3.10

Hello @simoncozens !
There is a bug:
 ~ → pip install shaperglot --user
WARNING: Retrying (Retry(total=4, connect=None, read=None, redirect=None, status=None)) after connection broken by 'NewConnectionError('<pip._vendor.urllib3.connection.HTTPSConnection object at 0x7f92f6aea2c0>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution')': /simple/shaperglot/
ERROR: Could not find a version that satisfies the requirement shaperglot (from versions: none)
ERROR: No matching distribution found for shaperglot

 ~ → pip install https://github.com/simoncozens/shaperglot --user
Collecting https://github.com/simoncozens/shaperglot
  Downloading https://github.com/simoncozens/shaperglot
     \ 214.7 kB 4.1 MB/s 0:00:00
  ERROR: Cannot unpack file /tmp/pip-unpack-oao25x7k/shaperglot (downloaded from /tmp/pip-req-build-jj91eo86, content-type: text/html; charset=utf-8); cannot detect archive format
ERROR: Cannot determine archive format of /tmp/pip-req-build-jj91eo86

Are you on the latest pip?


Ready to pay expenses not showing up on dashboard

Users are getting in touch about expenses they expected to be paid and it turns out that  they are not being shown on "ready to pay" in the host dashboard so we are missing them.
Examples:

https://opencollective.com/network-time-foundation/expenses/9228
https://opencollective.com/hyper-star-samsung-next-decentralization-grant/expenses/11522

The first one has already been paid. The second one I am holding off on so this can be looked into, but by the time we're alerted to these the user is already unhappy at the delay and has contacted us, so we end up having to pay them right away.

I feel like I have seen this come up multiple times when the expense amount = 100% of the Collective balance (as is the case for both these examples) and I strongly suspect that's related.

Reported in the past as https://github.com/opencollective/opencollective/issues/2144 But it was closed without being fixed.

This change seems to have created a bug: now expenses with insufficient balance are showing in "ready to pay".

This is already taken care of in https://github.com/opencollective/opencollective-frontend/pull/3074. Will be part of next deploy.


Gear state not changing

Describe the bug
When loading a recorded flight of Boeing Super Hornet, the gear state did not change in the model during playback.   Skydolly was setup to record during a takeoff where the gear was raised.    I stopped the recording and played back the flight.   Gear handle position went up but not the gear.  It could be a MSFS issue.
To Reproduce
Steps to reproduce the behavior:

start a free flight of Boeing Super Hornet
start recorder on skydolly
takeoff and raise gear
Stop and replay using skydolly.

Expected behavior
Gear to go up.
Screenshots
If applicable, add screenshots to help explain your problem.
Application version:
v0.8.5a

Hello,
Thank you for your report. Unfortunately - or luckily, depending on the point of view - I was not able to reproduce this issue: both the landing gear lever in the cockpit and the landing gear (as seen from an external aircraft view) were raised. Repeatedly (even when "scrubbing" on the replay timeline in Sky Dolly).
I tried both a "Free flight" directly from a runway (LOWW - but I don't think the airport matters) and from a "cold and dark" start from the same airport (running the checklist automatically).
Do you happen to have any "hardware" (throttle or joystick with programmable buttons, a console...) with a "two state" button (e.g. such "two/multiple state buttons/switches" exist on my Hotas Warthog Throttle)? I am asking because maybe you have the gear state assigned to one of those multi-state buttons (switches) and during replay the state of that switch is "gear lowered"?
Alternatively you could also download the Microsoft SDK (via the "developer" mode in FS 2020, there exists a menu entry to download the SDK) and try with the "Simvar Watcher" example program, which by default is located here:
C:\MSFS SDK\Samples\SimvarWatcher\bin\x64\Release\Simvars.exe
The corresponding simulation variable which controls the gear up/down state is called "Gear Handle Position" (pro tip: the tooltips in Sky Dolly reveal the corresponding simulation variable name, in the "Simulation Variables" dialog (press key V)).
Once the flight is started and Simvars.exe is running:

Click on Connect (a green light should indicate a successful connection)
In the Simvar Name dropdown list, locate and select the GEAR HANDLE POSITION simulation variable
In the Units dropdown list, select Bool or Boolean (both work - Number also always works)
Click on the "Add request" button (the variable should now appear in the list, together with its current state, either 0.0 or 1.0)
Now in the Value field enter a value like 1.0 (lowered) or 0.0 (raised), depending on the current state
Click the "Try set value" button (You best try this while airborne ;))
In FS 2020, the gear should now be lowered or raised accordingly

By the way, I tried this with the "Boeing F/A 18E Super Hornet Asobo" (in other words: the one which came for free with the latest FS 2020 update), with the default livery and no add-ons installed (especially no add-ons related to the stock F/A 18).
I will close this issue for now, but please feel free to open it again in case you have further findings which help me to actually reproduce this issue.

Just for the record: I am happy to declare that also the arrestor hook and the wing folding state are properly recorded and replayed with the Asobo F/A 18 - that was one of the first things that I've tried with Sky Dolly once the F/A 18 was available :)

And to clarify what I've meant with those "multi-state buttons/switches": Sky Dolly does "freeze" the "user aircraft" (the one being normally controlled by the player), but from this point on then merely replays all previously recorded input/button/control/switch states of the cockpit (as can be observed in the "Simulation Variables" dialog - press key V).
Any other input coming from some "control hardware" (again, I am especially referring to "multi-state switches" - and I do speak from experience with my Thrustmaster Hotas Warthog throttle where one multi-state switch is assigned to "Battery on/off") will interfere and "win over Sky Dolly".
So if Sky Dolly once says "gear up" (note that variables are only sent to FS 2020 in case their recorded state actually changes) but another hardware constantly says "gear down" then the gear will remain down.

Thank you checking it out and the thorough explanation.   Subsequent testing for me at least showed it works on the first replay but starts going wonky after the replay is over and aircraft hits the ground or crashes (crash detection turned off).   It stops working right until I restart the MSFS program.   Also, I have the steam version of the simulator.  Don’t know if that has something to do with it.
you can close issue but by the way, if you are able can you post a fltrec file and it’s Csv of an F-18 flight you used.   I would like to try your file out.  Thanks.

The freshly released v0.14 has improved support for 3rd-party aircraft, specifically for gears, flaps and spoilers and other controls (but not everything is working perfectly yet, depending on the aircraft and its specific simulation logic).

Hi there! Thank you very much for this. I tried with the PMDG 737 and looks like everything is working (flaps, landing gear, airspeed brake). Very nice.
I only found a small issue. Tried recording 3 landings and it happened all the 3 times. There's a small lag right when the airplane is going to touch the ground. I've looked the generated sqlite db and found this little gap (~500) between two timestamps right when attitude becomes close to the airport attitude. Looks like the difference between all the timestamps are usually around 30. Not sure if that's the issue but just fyi.
Here is a recorded video where you can see the touchdown lag: https://drive.google.com/file/d/1T65WlKZ61qb9W-ytLkf2i5rH_-SnBzr9/view?usp=sharing
And here's a screenshot of the db, as you can see the gap between 30807 and 30808 is bigger than the others.


Watch changes in APM criteria and config for reload

Thanks Alexandre!
One question - when entering text into the "host" field which causes the typeahead list to be displayed, it immediately issues the refresh which clears all the data. Is it possible to have this field only update the config when the user presses the return key, selects an item from the typeahead list or the field loses focus?

Yeah, I have tried some approaches to have it working like that, but so far not successful. I think previously it was not always updating (didn't removed the filter when deleting the text, iirc), now it's updating a bit too much.
Will try to find a solution. Thanks for merging.


build error with node-curl on aws

Hey,
im running into this problem on a aws instance
when ever it requires node-curl, i run into :
Error: Cannot find module '/home/ubuntu/node_modules/node-curl/lib/../build/default/node-curl'
running console.log on the error : Error: Module version mismatch. Expected 12, got 1.
there also appears a build error in "node_modules/node-curl/build"
to repeat the error, i ran sudo make:
CXX(target) Release/obj.target/node-curl/src/node-curl.o
In file included from ../src/node-curl.cc:1:0:
../src/node-curl.h: In constructor ‘NodeCurl::NodeCurl(v8::Handlev8::Object)’:
../src/node-curl.h:123:11: error: ‘class v8::Object’ has no member named ‘SetPointerInInternalField’
../src/node-curl.h:124:50: warning: ‘static v8::Persistent v8::Persistent::New(v8::Handle) [with T = v8::Object]’ is deprecated (declared at /home/ubuntu/.node-gyp/0.11.3/deps/v8/include/v8.h:5514) [-Wdeprecated-declarations]
../src/node-curl.h:125:35: error: invalid conversion from ‘void ()(v8::Persistentv8::Value, void)’ to ‘void ()(v8::Isolate, v8::Persistentv8::Object_, NodeCurl_)’ [-fpermissive]
/home/ubuntu/.node-gyp/0.11.3/deps/v8/include/v8.h:5593:6: error:   initializing argument 2 of ‘void v8::Persistent::MakeWeak(P_, typename v8::WeakReferenceCallbacks<T, P>::Revivable) [with P = NodeCurl, T = v8::Object, typename v8::WeakReferenceCallbacks<T, P>::Revivable = void ()(v8::Isolate, v8::Persistentv8::Object_, NodeCurl_)]’ [-fpermissive]
../src/node-curl.h: In static member function ‘static void NodeCurl::destructor(v8::Persistentv8::Value, void_)’:
../src/node-curl.h:163:40: error: ‘class v8::Object’ has no member named ‘GetPointerFromInternalField’
../src/node-curl.h: In member function ‘void NodeCurl::close()’:
../src/node-curl.h:169:11: error: ‘class v8::Object’ has no member named ‘SetPointerInInternalField’
../src/node-curl.h: In static member function ‘static NodeCurl* NodeCurl::unwrap(v8::Handlev8::Object)’:
../src/node-curl.h:185:28: error: ‘class v8::Object’ has no member named ‘GetPointerFromInternalField’
../src/node-curl.h: In member function ‘size_t NodeCurl::on_write(char_, size_t)’:
../src/node-curl.h:198:117: warning: ‘static v8::Persistent v8::Persistent::New(v8::Handle) [with T = v8::String]’ is deprecated (declared at /home/ubuntu/.node-gyp/0.11.3/deps/v8/include/v8.h:5514) [-Wdeprecated-declarations]
../src/node-curl.h:202:17: error: expected primary-expression before ‘’ token
../src/node-curl.h:202:19: error: ‘buffer’ was not declared in this scope
../src/node-curl.h: In member function ‘void NodeCurl::on_end(CURLMsg)’:
../src/node-curl.h:215:113: warning: ‘static v8::Persistent v8::Persistent::New(v8::Handle) [with T = v8::String]’ is deprecated (declared at /home/ubuntu/.node-gyp/0.11.3/deps/v8/include/v8.h:5514) [-Wdeprecated-declarations]
../src/node-curl.h: In member function ‘void NodeCurl::on_error(CURLMsg_)’:
../src/node-curl.h:226:117: warning: ‘static v8::Persistent v8::Persistent::New(v8::Handle) [with T = v8::String]’ is deprecated (declared at /home/ubuntu/.node-gyp/0.11.3/deps/v8/include/v8.h:5514) [-Wdeprecated-declarations]
../src/node-curl.h: In static member function ‘static NodeCurl* NodeCurl::unwrap(v8::Handlev8::Object)’:
../src/node-curl.h:186:2: warning: control reaches end of non-void function [-Wreturn-type]
make: *** [Release/obj.target/node-curl/src/node-curl.o] Error 1
able to help out?

Were you able to fix this?


Support admonition markdown extension syntax

Is your feature request related to a problem?
Hi @aepfli. I came across your rule through the discussion on one of the markdownlint repos. Thanks for this rule!
I started trying it out and there is one case where this rule complains that would be nice if it supported it.
We use mkdocs-material (https://squidfunk.github.io/mkdocs-material/) and make use of the admonition extension (see https://squidfunk.github.io/mkdocs-material/reference/admonitions/).
Unfortunately all lines with !!! ... and ??? ... (e.g., !!! note) cause a max-one-sentence-per-line violation.
Describe the solution you'd like
It would be great if this admonition syntax could be excluded without requiring to specifically add disable comments on the preceding line.
Describe alternatives you've considered
See above.
Additional context
n/a

Hey, first of all, thank you for the issue. I am happy that my code helps some people out there.
The solution I propose in #52 is merely a bandaid and a hack around the problem. But till I come up with a better solution, I think it is feasible with little configuration effort.
I'll think about a better approach, but please let me know if this helps you and is sufficient for the time being.
all the best
simon

Thanks Simon for the quick response and fix. This workaround would help for the time being.


Mutual TLS

We want to support mutual TLS along side server side TLS. This should be a matter of ensuring the client can respond to a request from the server for it's certificate.
There is commented out code to support this. We need to be configured to test thoroughly before release

Hi @DMickens Is there a tentative timeline on when this will be available? Thanks

@moizarafat
Right now we don't have a strict timeline. There are lots of enhancements that can be done and we are prioritizing based on feedback and demand from the field. There is a growing demand for TLS enhancements, though, so it might be given priority in the next few months

Hey @DMickens, my company's security policy requires us to use mTLS and we have enabled this on our clusters and use it with other tech stacks too. So this feature will be really helpful for us.
Thanks

An alternative system of TLS socket management giving the user more control is being investigated and implemented. This will allow the user to construct and provide the entire tls config object that the driver uses when wrapping the tls socket after tls negotiation with the server. The original motivation is to help minimize issues for users migrating from the unsupported node-vertica to vertica-nodejs. However, this should also provide a backdoor way to enable mutual TLS, the only caveat being that the user is responsible for ensuring the proper tls config parameters are provided.


Support workspace status values a.k.a. "stamping" in run_binary

genrule has an (undocumented) stamp attribute that gives the command access to bazel-out/stable-status.txt and bazel-out/volatile-status.txt.
run_binary should have some way to access workspace status values as well, but in a more ergonomic way. That is, don't make the binary itself responsible for finding and parsing the .txt files -- build that into run_binary. Maybe stamp = True puts each workspace status key-value into the environment? Or the stamp value could be a list of workspace status keys to expose as environment variables.

Agreed. We'll add this.

@kormide Lets model this after what rules_go and rules_nodejs does.
https://github.com/bazelbuild/rules_nodejs/blob/c962f12e06d99230f17d82a732958f3c00c9438e/nodejs/private/providers/stamp_setting_info.bzl
https://github.com/bazelbuild/rules_nodejs/blob/c962f12e06d99230f17d82a732958f3c00c9438e/internal/pkg_web/pkg_web.bzl#L60

I've come around to the opinion that we should just use a  integer for the stamping value on rules rather than a custom config setting. It's just easier to type. Like rules_python.
Positive values are always stamp. Negative are never stamp. Zero is use --stamp.

Actually I think I'll take this one, I've already implemented it a few times.

https://github.com/aspect-build/bazel-lib/pull/186 is the first step, allowing custom rules to use stamping.
I'm thinking about your proposal, and there isn't a way to do exactly what you've suggested. The two status files are only paths at analysis time, and only an action can read their content. run_binary has no wrapper script, it runs the provided tool directly with ctx.actions.run, so it doesn't have a chance inside of the action to read the two status files at all. The first moment when those files can be read is when the provided tool is running.
I think the best we can do is set an environment for that tool like BAZEL_STAMP=1 BAZEL_STABLE_STATUS=bazel-out/stable_status.txt BAZEL_VOLATILE_STATUS=bazel-out/volatile-status.txt

Thanks for the quick implementation!

I'm thinking about your proposal, and there isn't a way to do exactly what you've suggested.

Ah, too bad, but understandable.

Thanks for the quick implementation!
Will this ability be inherited by js_run_binary from rules_js?

I'm thinking about your proposal, and there isn't a way to do exactly what you've suggested.

Ah, too bad, but understandable.

It will be inherited implicitly via **kwargs. I've put up a PR to add it to the attributes so it gets a docstring in the js_run_binary documentation, https://github.com/aspect-build/rules_js/pull/290

FYI, I convinced Alex to change the default stamp value to 0 so that just landed and I cut a v1.7.0 release of bazel-lib


Add bibtex by URL with public link

Instead of uploading a bibtex, it would be useful if a .bib file could be referenced by URL, ideally with a public link, like http://citationgecko.com?bib=https://github.io/mypaper/references.bib
This would allow to generate direkt links for online writing tools ("Open as seed in citation gecko"), or load a .bib file from GitHub directly.

Nice idea, I can see how that would be useful!


Inability to handle infinite streams....

The following code will likely cause a stack overflow even though the stream should not yet be evaluated:
numbers |> Lazy.List.take 10000
This is because "take" is not formulated correctly as it injects laziness after the evaluation of function arguments, which are evaluated eagerly.
Many other functions which should be able to handle infinite streams as sorce have this same problem.
This problem is in common to elm-community/lazy-list, which seems to have a common code base as per the comment there: https://github.com/elm-community/lazy-list/issues/5.
My PR there fixes this issue by changing the order of where laziness is injected, and is applicable here.

Hi, as far as I can tell the code you described above would not cause a stack overflow.
I've looked at the example in your link, and it seems the problem there lies in toList. This does not overflow the stack because it's not tail-call optimized but because at higher recursion levels the argument to toList is too big to fit in the stack. I'll look into fixing this, but I'm not sure it will be that easy.

#13 now contains a fix for this issue (as far as I can tell).
It comes with a performance cost though.

The issue for the example I posted is perhaps more related to "numbers" being a constant reference to the head of the lazy list, thus it can't be garbage collected as the list is traversed.  This has been partially alleviated in the "Lazy" used here is non-memoizing, thus there is no extra memory used as the lazy list is traversed, however that means every node is re-evaluated on every pass, at a terrible cost in execution speed for some algorithms.
The problem I was raising in this issue is the one identified by peacememories for "drop" which didn't handle infinite lists, and my solution in my PR for the elm-community version was the same: inject a helper function so that there are no direct calls to the outer function.
You'll find that many functions require this same treatment to handle infinite lists without stack overflow.
I would give more examples but am away from my computer for several months and it is difficult to set up with the on-line version of Elm on a smartphone.


ciao-controller: implement configuration update API

Once a privileged operator updates an element of the cluster configuration (see #1096), ciao-cli will create a request that should be served by the ciao-controller endpoint to perform the change (if applies).
From the API the input should be validated as described in #1095 and return the proper ResponseError if change cannot be applied.
Sane validation from the API includes:

Is the request being performed by the cluster administrator?
Is the configuration element requested to be changed valid?
Does the request contain sane input for the configuration element to change?
Is the new value usable (e.g. does the new endpoint respond successfully) ?

After a successful validation, the new configuration payload should be sent to the SSNTP server (ciao-scheduler) and a proper response for the API call.
This is a sub-task of #929 configuration: Add cluster configuration update support

Not working on this anymore, closing...


Mouse events - sendings two of the same

I am sending two mouse events rather than one per click, see:
iTerm2/tests/mouse_reporting.sh
With Terminal.app the menu options are updated once on every click, with SwiftTerm, two updates take per click.

False alarm


Possibly switch from HyperLogLog to HyperLogLog++

We currently (as soon as #191 is merged) use the HyperLogLog algorithm (Flajolet et al, 2007) to estimate the number of unique randstrobe hashes.
There is an improved variant of the algorithm called HyperLogLog++ that gives even more accurate results and permits counting higher cardinalities. I’m not sure where the limit is in the HyperLogLog implementation that we use, but if I understand the HyperLogLog++ paper correctly, using a 32-bit hash (which this implementation does) gives more inaccurate estimates the closer the cardinality gets to $2^{32}$.
The largest genome I’ve looked at is wheat (issue #192). It has 14 Gbp and gives us 1.3 billion distinct randstrobe hashes. The HyperLogLog estimate is 1.5 billion. We only need to get the order of magnitude right since the hash table size is always a power of two anyway, so it seems we are fine at the moment.
However, before looking at wheat, I had spent some time looking for C++ HyperLogLog++ implementations, and I want to at least document what I found.

https://github.com/dialtr/libcount: Requires some work to get it integrated becauses it consists of relatively many files.
https://github.com/omsitapara23/HyperLogLog-HIP: Should be easy to integrate because it consists of just a couple of header files. Only two weeks old at time of writing.
https://github.com/arashbm/hyperloglog: also multiple files; author active quite recently

I’m fine with just closing this issue.

I see. Yes I am fine with closing this issue. And if it needs a revisit we have good information on where to look for an implementation here.


Adds the ability to add a NoProxy List to a Proxy via API

...utilising the already existing functionality.
Makes NoProxy public and replaces NoProxy::new()
with NoProxy::from_env() and NoProxy::from_string(). Adds a Proxy::no_proxy() method.
Fixes #1690

I have no clue how to fix these tests, sorry, but I don't think they are related to my changes.


cmd/pprof: panics if profile is not found

Go version
go version devel go1.23-019353d532 Thu May 23 18:51:17 2024 +0000 linux/amd64
Output of go env in your module/workspace:
GO111MODULE=''
GOARCH='amd64'
GOBIN=''
GOCACHE='/home/rhysh/.cache/go-build'
GOENV='/home/rhysh/.config/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='amd64'
GOHOSTOS='linux'
GOINSECURE=''
GOMODCACHE='/home/rhysh/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='linux'
GOPATH='/home/rhysh/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/home/rhysh/sdk/go-dev'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/home/rhysh/sdk/go-dev/pkg/tool/linux_amd64'
GOVCS=''
GOVERSION='devel go1.23-019353d532 Thu May 23 18:51:17 2024 +0000'
GODEBUG=''
GOTELEMETRY='local'
GOTELEMETRYDIR='/home/rhysh/.config/go/telemetry'
GCCGO='gccgo'
GOAMD64='v1'
AR='ar'
CC='gcc'
CXX='g++'
CGO_ENABLED='1'
GOMOD='/dev/null'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -ffile-prefix-map=/tmp/go-build3616585202=/tmp/go-build -gno-record-gcc-switches'

What did you do?
$ ./bin/go tool pprof ./missing

What did you see happen?
$ ./bin/go tool pprof ./missing
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x91c737]

goroutine 17 [running]:
cmd/vendor/github.com/google/pprof/internal/driver.fetch({0x7fff1e6ef510, 0x9}, 0xffffffffc4653600, 0xffffffffc4653600, {0x7ff5e88cb1f8, 0xc000080238}, {0xb2eca0, 0xc00009e8a0})
	/home/rhysh/sdk/go-dev/src/cmd/vendor/github.com/google/pprof/internal/driver/fetch.go:514 +0x2f7
cmd/vendor/github.com/google/pprof/internal/driver.grabProfile(0xc0001ec960, {0x7fff1e6ef510?, 0x9?}, {0x7ff5e88cb1d8?, 0xf1e060?}, {0xb2fc68, 0xc0000331d0}, {0x7ff5e88cb1f8, 0xc000080238}, {0xb2eca0, ...})
	/home/rhysh/sdk/go-dev/src/cmd/vendor/github.com/google/pprof/internal/driver/fetch.go:334 +0x131
cmd/vendor/github.com/google/pprof/internal/driver.concurrentGrab.func1(0xc000209000)
	/home/rhysh/sdk/go-dev/src/cmd/vendor/github.com/google/pprof/internal/driver/fetch.go:211 +0x86
created by cmd/vendor/github.com/google/pprof/internal/driver.concurrentGrab in goroutine 7
	/home/rhysh/sdk/go-dev/src/cmd/vendor/github.com/google/pprof/internal/driver/fetch.go:209 +0x8f

What did you expect to see?
$ ./bin/go tool pprof ./missing
./missing: open ./missing: no such file or directory
failed to fetch any source profiles

Go 1.22 works.
Bisect of this repo says "63dd79c07b0026b58f421a5273c41e705ccb73d1 is the first bad commit". I haven't bisected github.com/google/pprof.

https://github.com/google/pprof/commit/a5a03c77bf0842e0592d2be8129bca0b2bd3bb60 looks like it would cause this.
@dmitshur when do we typically update the vendor'd version of pprof? If we haven't done so yet, the best bet is probably an upstream fix and then update from upstream.

Upstream is still broken, and yes a bisect there points to https://github.com/google/pprof/commit/a5a03c77bf0842e0592d2be8129bca0b2bd3bb60

@prattmic We will update it soon, now that we've entered the release freeze, as part of go.dev/issue/36905. It's also fine to update it separately for this as needed.

I've mailed out a fix for the google/pprof library per PR https://github.com/google/pprof/pull/865.


s3 'get-bucket-acl' - grantee's 'type_' is not being populated

s3_client
.get_bucket_acl(GetBucketAclRequest {
                bucket: "some-bucket".to_string(),
})
.sync()?

Output:
[Grant { grantee: Some(Grantee { display_name: Some("user"), email_address: None, id: Some("some_id"), type_: "", uri: None }), permission: Some("FULL_CONTROL") }, Grant { grantee: Some(Grantee { display_name: None, email_address: None, id: None, type_: "", uri: Some("http://acs.amazonaws.com/groups/global/AllUsers") }), permission: Some("READ") }]

Equivalent awscli call:
aws s3api get-bucket-acl --bucket some-bucket
{
    "Owner": {
        "DisplayName": "user",
        "ID": "some_id"
    },
    "Grants": [
        {
            "Grantee": {
                "DisplayName": "user",
                "ID": "some_id",
                "Type": "CanonicalUser"
            },
            "Permission": "FULL_CONTROL"
        },
        {
            "Grantee": {
                "Type": "Group",
                "URI": "http://acs.amazonaws.com/groups/global/AllUsers"
            },
            "Permission": "READ"
        }
    ]
}

I can replicate this too. Investigating:
Making the following call with an AWS V4 signature
curl -X GET \
  'https://s3.amazonaws.com/some_aws_bucket?acl=' \

returns:
<?xml version="1.0" encoding="UTF-8"?>
<AccessControlPolicy xmlns="http://s3.amazonaws.com/doc/2006-03-01/">
    <Owner>
        <ID>XXXXXXXXXXXX</ID>
        <DisplayName>drexler</DisplayName>
    </Owner>
    <AccessControlList>
        <Grant>
            <Grantee xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:type="CanonicalUser">
                <ID>XXXXXXXXXXXX</ID>
                <DisplayName>drexler</DisplayName>
            </Grantee>
            <Permission>FULL_CONTROL</Permission>
        </Grant>
    </AccessControlList>
</AccessControlPolicy>

Type is an XML attribute from the response but is handled as an XML element here:
https://github.com/rusoto/rusoto/blob/4ae62397f70556f115b1279e1300ee1742fa63d8/rusoto/services/s3/src/generated.rs#L5522


Added isogram calculator

Fixes Issue🛠

Closes #1599
Description👨💻
I have added an Isogram calculator that calculates whether a string entered by the user is isogram or not .
Type of Change📄

[ ] Bug fix (non-breaking change which fixes a bug)
[x] New feature (non-breaking change which adds functionality)
[ ] Style (non-breaking change which improves website style or formatting)
[ ] Documentation (non-breaking change which updates or adds documentation)
[ ] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project
[x] I have commented on my code, particularly in hard-to-understand areas

Screenshots/GIF📷

@Rakesh9100 This PR is reviewed & can be merged.

@Rakesh9100 when this will be merged ?

@Nihal4777 @Rakesh9100 when will this be merged ?

I have let PA know. He will review and merge as soon as possible.

@Rakesh9100 please review and merge this today .


How do I make a random menu in Source Code in FNF on Psych Engine?

Como Faz um menu aleatório em Source Code no FNF na Psych Engine?
How do I make a random menu in Source Code in FNF on Psych Engine?

Use FlxG.random.int(0, theAmountOfMenusYouWantToCreate - 1)
Then with a switch, you can "make a random menu appear" when you enter the Main Menu State.
Something like:
` var menuNumber:Int = FlxG.random.int(0, 4) //That means you have FIVE menus.
switch(menuNumber)
case 0:
//Insert your code here.
case 1:
//Insert your code here.
//Repeat for each case from 0 to 4 in this case.
`
Hope it helps :)


Support input/device groups

What problem does this solve?
Most video games that support multiple input methods need to show button prompts at some point. To find out which set of images need to be shown for a specific set of inputs, there has to be a way to check which "input group" is currently active e.g. controller or keyboard and mouse.
What solution would you like?
I would like to be able to group input maps in to input groups with either names or typed labels. I then want those groups be attached to the ActionState so it is easily retrievable.
EDIT: There is a great video that explains what I want to achieve with this feature: https://youtu.be/watch?v=d6GtGbI-now

Is there any chance for this to be on the roadmap at some point? 🙏🏼

Definitely on board with this, but I'm generally busy with Bevy itself. If you'd like to try your hand at a PR or design for this I'd be happy to help review and give design feedback.

#242 is the same idea; closing in favor of that.


Allow the management of Route53 Resolver Endpoints and Rules

with the network-mgmt role
https://issues.redhat.com/browse/OSD-7873

Codecov Report

Merging #614 (9fa863a) into master (011989f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #614   +/-   ##
=======================================
  Coverage   14.52%   14.52%
=======================================
  Files          45       45
  Lines        4325     4325
=======================================
  Hits          628      628
  Misses       3673     3673
  Partials       24       24

/hold
There is currently ongoing discussion with the BU and CU and SREP Mgmt about the extra cost this would incur for non-ccs clusters, as well as why the current options don't meet the customer's needs.

Closing in favour of #615.


TLS support includes root certificate management code that should be handled elsewhere

txaws/client/ssl.py is mostly just code that's been obsoleted by improvements to certificate authority certificate handling code in Twisted.
We can probably just get rid of the whole module (after deprecating it, I suppose).  Certainly we don't need VerifyingWebClientContext anymore.  Instead, we should use BrowserLikePolicyForHTTPS.
This will:

reduce complexity of txaws code (in a security-sensitive area)
bring support for certificate bundles (which txaws can't currently understand)
make Twisted responsible for making sure something good happens on Windows (twisted has Windows CI, txaws does not)

@Julian asked what's needed to get rid of the "probably" from the issue description so I took another look at the module.
It still looks to me as if this whole module can be deprecated and then eliminated.  We'll have to bump up the minimum supported version of Twisted so we can be sure we get BrowserLikePolicyForHTTPS and we should check for TXAWS_CERTS_PATH and respect that in the new codepaths (and also deprecate it).
So, yea, anyone who wants to can go ahead and deprecate this thing.

AFAICT, 15.5.0, which is the current minimum version, already had BrowserLikePolicyForHTTPS, and used it by default in Agent:
https://twistedmatrix.com/documents/15.5.0/api/twisted.web.client.Agent.html

Aright, I cowboy coded together some stuff in https://github.com/twisted/txaws/tree/deprecate-t-c-ssl which looks like it should get 80% of the way there to my layman's eye, but it exhausted me a bit, so will have to leave it there for now till I catch a second wind.


Fixed #729

SQLiteViewStore did not handle create attribute properly.  Fixed this.

Equivalent iOS code
https://github.com/couchbase/couchbase-lite-ios/blob/f38ef1e54338e10d299eb1ccbc6f4c4bd442eb5a/Source/CBL_SQLiteViewStorage.m#L56-L57

Tested it and the fixed solved the issue.


e2e Playwright tests are broken: only 1 test runs

Description

It seems that after changes from #1920, e2e tests run only one test instead of the 155 tests from the suite.
Reproduction

Look at the logs from #1920. They say "Running 1 test using 1 worker": https://github.com/WordPress/openverse/actions/runs/5006755962/jobs/8972488862?pr=1920
Additional context
Almost all consequent PRs ran only 1 test. Only #2122 which was merged after it, ran the full suite. I think it's because it was still using the old settings in the CI.

I've created an issue so we can enforce not using this via lint rule: https://github.com/WordPress/openverse/issues/2195


get_country() to handle *args

May be worth considering whether get_country should have *args, where *args are the set of strings that are being searched e.g.
get_country('AUS', 'UGA', 800) which would return three Country() objects.

Don't think this is that relevant anymore.


Improve homepage copy + design

@aeolianeth test

https://discord.com/channels/775859454780244028/968886189157412864/977913251889299466


PSPlaceOpenBrace and PSUseConsistentWhitespace don't work together

Steps to reproduce
Invoke-ScriptAnalyzer -ScriptDefinition @'
if ($true)
{
    "ok"
}
'@ -Settings @{
    Rules = @{
        PSPlaceOpenBrace = @{
            Enable     = $true
            OnSameLine = $false
        }
        PSUseConsistentWhitespace = @{
            Enable = $true
        }
    }
}

Expected behavior
When both rules are used, it should not trigger the PSUseConsistentWhitespace rule.
Actual behavior
RuleName                            Severity     ScriptName Line  Message
--------                            --------     ---------- ----  -------
PSUseConsistentWhitespace           Warning                 2     Use space before open brace.

When you add 1 space before open brace, it doesn't report violation.
If the whole code block is indented (more than 1 space before open brace), it reports again the problem.
Environment data
> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      5.1.18362.752
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.18362.752
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

> (Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
1.19.0

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

> (Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
1.19.0

Related: #769

Running it without PSPlaceOpenBrace, results in the same, so I don't see how it is an issue of the 2 rules together:
Invoke-ScriptAnalyzer -ScriptDefinition @'
if ($true)
{
    "ok"
}
'@ -Settings @{
    Rules = @{
        PSUseConsistentWhitespace = @{
            Enable = $true
        }
    }
}

I am aware, generally one cannot use the formatting settings and run Invoke-ScriptAnalyzer against them to ensure the style is compliant. I think this is just one of a few things that you'll find, but it's definitely worthwhile boing them down individually and raising them as you did.
It happens exactly here: https://github.com/PowerShell/PSScriptAnalyzer/blob/163d2c2f4f03a591bea71b78038010cc9c365932/Rules/UseConsistentWhitespace.cs#L238
I am thinking we could maybe enhance the check in there to only check for whitespace if the previous token is on the same line, which is the issue here I think


Where can I find the confidence value of a Prediction on Omegafold ?

Heyy,
I've already predicted several protein structures with Alphafold2.0 using a colab-notebook. But I have to predict around 100 protein structures and so I've heard about Omegafold wich can predict protein structures in a very short time compared to Alphafold.
I'm on macOS and I've used the terminal commands in order to do my prediction job.
All my proteins have been predicted but I can't find the confidence value of the prediction with Omegafold (I mean the pLDDT score, wich is given by Alphafold).
Do you know if Omegafold gives with the pdb structure prediction, a file containing the confidence value of the prediction (pLDDT) ?
If it does, Do you know how to find this file ??
Thank you for any help you can offer
Pierre

The output PDB file that OmegaFold produces has the pLDDT on the bfactor column, you can visualize it with software such as ChimeraX or extract it from the PDB as an average or from the residues you need.

Jumping in on this thread. Is there a concept of "pae" in OmegaFold? I know that their version of pLDDT is in the bfactor column (thanks @EdgarAraya ), but I was wondering if the package also has some kind of "predicted aligned error" as well. Those are two distinct confidence metrics, you can see more about the distinction in [(https://alphafold.ebi.ac.uk/faq) ]


Syntax highlight error in js code

From @ihgsm on September 21, 2016 22:51

VSCode Version: 1.5.2
OS Version: Win10

Steps to Reproduce:

In any existing JavaScript code type this: (n<<t ). Make sure there is a space after the t.
Check the syntax highlighting of the rest of the code, all the text that supposed to be white are now in dark grey. (using Dark+ color theme)

Copied from original issue: Microsoft/vscode#12413

From @chrmarti on September 22, 2016 15:41

From @staralfurbe on September 26, 2016 17:10
I've a very similar issue, i assume the root cause is the same.
In js code within a php file, "<" followed by any character will cause the same color problem for the rest of file.

This should be fixed in latest grammar by #257
It should be available as part of vscode extension


feat: let pass our own chan to ShutdownOnSignals

Passing your own channel as an argument to these methods allows you to have control over the communication channel used to receive signals. That is very useful for my custom signal handling logic.

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

Comparison is base (c635315) 86.09% compared to head (c35c527) 85.99%.

Files
Patch %
Lines

root_scope.go
0.00%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             v2-🚀      #56      +/-   ##
==========================================
- Coverage   86.09%   85.99%   -0.10%
==========================================
  Files          20       20
  Lines        1783     1785       +2
==========================================
  Hits         1535     1535
- Misses        218      220       +2
  Partials       30       30

Flag
Coverage Δ

unittests
85.99% <0.00%> (-0.10%)
:arrow_down:

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

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

Can you develop your use case please ?
Why not just call injector.Shutdown() when you need a custom shutdown logic?

I give the channel to my API so I can shutdown my server from it.
Yes for now I manage to do it by calling Injector.Shutdown() it's working as expected.
This solution just allows me to not write the signal waiting, which just saves me a few lines of code.

We recently removed some Shutdown**** helpers for simplicity. I'm not sure we want to support more cases here.
Also, I think sending a os.Signal into this channel is unsafe, since the Shutdown helper might listen to different signals.
IMO, it's better to wait with:
ch := make(chan struct{}{})

go func() {
    <-ch
    injector.Shutdown()
}()

I will keep this PR open for further discussion.

ch := make(chan struct{}{})

go func() {
    <-ch
    injector.Shutdown()
}()

Yes, that's how i'm doing it, and it's working well.


[FLINK-21445][kubernetes] Application mode does not set the configuration when building PackagedProgram

What is the purpose of the change
Application mode uses ClassPathPackagedProgramRetriever to create the PackagedProgram which doesn't set the configuration. This causes some client configurations not take effect. ClassPathPackagedProgramRetriever should refactor to have separate classes for the Python-based PackagedProgram and the PackagedProgram with and without a jar.
Brief change log

ClassPathPackagedProgramRetriever refactors to have separate implementations which are PythonBasedPackagedProgramRetriever, JarFilePackagedProgramRetriever and ScanClassPathPackagedProgramRetriever, which set the configuration to PackagedProgram.
ApplicationClusterEntryPoint gets the PackagedProgramRetriever with setting the configuration.

Verifying this change

ClassPathPackagedProgramRetrieverTest adds the test case testGetPackagedProgramWithConfiguration to verify whether get the PackagedProgram with the specified configuration.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@XComp , could you please help to review this pull request?

@SteNicholas Thanks for creating this PR. I have gone though the implementation carefully and want to share some thoughts. Instead of inheritance here, I would suggest to use the composition. Maybe we could refactor this component by the following steps.

Introduce the PackagedProgramRetrieverAdapter(maybe other better name), which will provide a builder and select a proper PackagedProgramRetriever based on the specified properties. It is just like what you have done in the ClassPathPackagedProgramRetriever#Builder
Make JarFilePackagedProgramRetriever, PythonBasedPackagedProgramRetriever, ScanClassPathPackagedProgramRetriever(could be renamed to the new ClassPathPackagedProgramRetriever) directly implement the PackagedProgramRetriever interface
Maybe we could add the AbstractPackagedProgramRetriever if the above three cases have some common member variables or methods to share.

After this refactor, we also need to split the tests in ClassPathPackagedProgramRetrieverTest to separate files. If the three cases have some common tests, then they could be implemented in the PackagedProgramRetrieverTestBase. At last, we should not forget to add some dedicated tests for the PackagedProgramRetrieverAdapter.

Sorry for not getting back earlier: I second @wangyang0918 proposal. The code should be easier to test after the proposed change. Additionally, the JarsOnClassPath singleton seems to be only relevant to what is now ScanClassPathPackagedProgramRetriever.

@wangyang0918 , @XComp , thanks for detailed review. I have followed the comments of @wangyang0918 , and taken additional remark of @XComp . Please help to review again if you are available.

@wangyang0918 , @XComp , thanks for your detailed review. I have followed the comments of @wangyang0918 and taken the additional remark @XComp . Please help to review this pull request again.

What's the state of this PR @XComp?

@tillrohrmann , I will update this pull request through following the comments above.

Ok, thanks @SteNicholas . I'll give it another pass when you're done.

Any updates from your side, @SteNicholas ?

@SteNicholas Did you have a chance to work on it? Or do you have an idea when to proceed?

@SteNicholas Did you have a chance to work on it? Or do you have an idea when to proceed?

@XComp , I’m adding the PythonBasedPackagedProgramRetrieverTest. And I will update this pull request today.

@XComp @wangyang0918 I have followed your detailed review comments. Please help to review the update of the pull request.

@XComp @wangyang0918  I have followed the detailed review comments. Please help to review the update of the pull request.

@XComp , thanks for your detailed comments for the refactoring.  Based on your finding, I thought that:

PythonBasedPackagedProgramRetriever is indeed a JarFilePackagedProgramRetriever with a specific jarFile and jobClassName, which could be removed and pass in these specific parameter values in JarFilePackagedProgramRetriever.
PackagedProgramRetriever implementation could be similified by introducing interface EntryClassInformationProvider providing getJarFile() and getJobClassName() as methods.

@XComp IIUC, what you are suggesting is using the composition instead of inheritance. I agree with you that it is easier to test and could make the contract clearer.
For JarsOnClasspath, it could be subsumed in the EntryClassInformationProvider implementation for classpath.

@SteNicholas just a kind ping. :-) do you have an idea when you could continue working on this issue?

@SteNicholas just a kind ping. :-) do you have an idea when you could continue working on this issue?

Of course, I'm addressing your comments. Sorry for later update for this request.

Hi @SteNicholas , how is the this issue's progress? I'm wondering whether you could give a time estimate on when you're able to close it. I understand that there are other things that need to be done on your side as well. I could take over if that helps.

Hi @SteNicholas , I'm going to pick up that one if that's ok with you. ...just to have it off the list.

Hi @SteNicholas , I'm going to pick up that one if that's ok with you. ...just to have it off the list.

OK.


Removed Epic and EA launchers still keep showing up

after uninstalling Epic and EA launchers, I manually removed entries in Steam as they were not removed automatically but they keep showing up again and starting then does nothing (as Launchers no longer are there)

@slimshader use the stop Scanner button. I will  be rewriting how uninstall works in the future.

Problem is I do want it to work for when I am installing and uninstalling games. I guess I will just wait for a fix for now then.

@slimshader keeps your eyes peeled o. The Decky Plugin, very soon an update will fix exactly what you are having problems with.

@slimshader hey I attempted a small fix for you, here's what I need you to do, open up NSL using the .desktop file and simply hit the uninstall button again and uninstall your launchers that you wanted. Then let me know if that fixes your shortcuts from popping up again. You should be able to now add new games without the launchers coming back.

Downloaded new NSL and when trying to re-install EA installer with it I am getting error "path not found"


bbcy_apply_adaptation has unknown variable 'k'

There is an occurrence of a variable k in line 22 of the bbci_apply_adaptation function. This variable is not defined prior to that place and therefore, Matlab throws an error. It is possibly meant to be a 1?

Yes, you are completely right. The error is fixed now. Thanks for reporting!


ORM Integration

Hey @ericlagergren.
Finally getting around to wanting to integrate this into our "orm" (https://github.com/volatiletech/sqlboiler) as the decimal library of choice.
Wanted to talk to you about a few small things though. We have two problems that we need to deal with coming from the sql world. And then I've a random question for you.

Scanner/Valuers. This you've obviously tackled for postgres, we only support 3 databases out of the box (psql, mysql, and mssql). I don't think it's that important to do bounds checking and what-not in code since the database should puke errors if it can't do something (although mysql being the clown truck that it is very likely just truncates values -_-). I'm wondering if we could create a generic implementation that's not specific to any one database.

Nullable: Unfortunately in the sql model that we have today, nullable types are typically represented by a separate type that includes a "null" flag. Although some people use a pointer to allow nulls we decided to follow the standard library model and create NullX types for all the types that we use. It appears that there's no nullable Decimal type, do you think that's something we could add to this library? If that's something you're not interested in we'll have to wrap the decimal type in our code and it's just less pretty :p

Looking at the way you've wrapped the type in the psql code, it seems to wind up requiring a reference to .V and all the operations are done on that. I suppose it's impossible to avoid since the API relies on assigning values all the time. Was just looking for thoughts on potentially avoiding the following (although it probably wouldn't work regardless since we never put pointers to anything on our generated structs):
user.Money.V.Mul(...)

Thanks for your time! Also thanks for fixing that bug I reported a few months back, that was a bit surprising :)

Hey!

That should be fine. sql/generic? I added the manual bounds checking since I figured it'd be nice to preemptively avoid a trip to the database, or at least round the decimals to the proper size beforehand. Not sure whether that was a good idea or not.

A NullDecimal should be fine in sql/generic. I'm not sure how to handle this case

type NullDecimal {
    *decimal.Big
    Null bool
}
[ ... ]
db.Exec("INSERT ... ", NullDecimal{Big: nil, Null: false}) // zero values added for clarity

The first thing that pops into my head is letting it be equal to "0". Do you know what other NullX implementations do where X is a pointer?

I don't recall why I didn't embed the decimal. For (2) the decimal can just be embedded. Is your Money type going to be type Money struct { NullDecimal } or what?

sql/generic sounds fine.
I think that it's best to let the database deal with the restrictions on the types. Consider if you move from one version of the DB to the next, and something's changed. The library now needs to be updated, but there's almost no way of knowing that the database has changed. Or even supporting two users on different versions of said database. It's also possible to get it wrong from misreading/lacking documentation. You also now do a needless check 2x on a proper value, and most of the time it's a proper value so you're penalizing performance on the normal path and optimizing for an error case which would be infrequent. Just my 2c about duplicating DB constraints as someone who had to make the same choices :)
I don't think we actually have anything that requires it to be a pointer. We count on the zero value being usable. If we were to embed the decimal type as a not-pointer it's going to make it awfully inconvenient to use isn't it? :\ If I'm right then it'd go something like this:
// square money
myStruct.Money.Big = *myStruct.Money.Mul(&myStruct.Money.Big, &myStruct.Money.Big)

In the case of null structs, the zero value is a null value. See the standard library for an example of how it's implemented: https://golang.org/pkg/database/sql/#NullBool This is the pattern we've used for all of our types.
The structs would look like:
type User struct {
   Money generic.Decimal
}

Thoughts?

Hey @ericlagergren. Quick friendly ping as we're nearing the release of v3 and need to sort this out. I also have a contributor that jumped in without knowledge of our conversation here to try and finish this (which is awesome) so I've asked him to review the details here.

Ack. If you or the contributor want to submit a PR, that’s awesome.
Otherwise, I’ll create one by the end of the week I think.
Le mar. 5 juin 2018 à 08:27, Aaron L<EMAIL_ADDRESS>a écrit :

Hey @ericlagergren https://github.com/ericlagergren. Quick friendly
ping as we're nearing the release of v3 and need to sort this out. I also
have a contributor that jumped in without knowledge of our conversation
here to try and finish this (which is awesome) so I've asked him to review
the details here.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ericlagergren/decimal/issues/95#issuecomment-394753100,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFnwZ8cwvcer4PPK-qALTzu1trS3v7R_ks5t5qNpgaJpZM4UOE0K
.

Happy no matter who does the work. Was just trying to figure out how to manage syntactically with all the pointers flying around. For example I wonder if we don't have a pointer on the User struct as above if json won't encode or what have you.

I don’t think it would, even if it’s addressable since it’s hidden behind
an interface.
Le mar. 5 juin 2018 à 13:02, Aaron L<EMAIL_ADDRESS>a écrit :

Happy no matter who does the work. Was just trying to figure out how to
manage syntactically with all the pointers flying around. For example I
wonder if we don't have a pointer on the User struct as above if json won't
encode or what have you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ericlagergren/decimal/issues/95#issuecomment-394841559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFnwZ6Sz3umOM2V-4eBjr2dsfxHT_OfYks5t5uPjgaJpZM4UOE0K
.

https://play.golang.org/p/54m4rRQCsmZ
So I was messing around a bit on the playground with how this might look. It seems if we don't use pointers anywhere, it's actually kinda okay.
Though it seems odd at first glance that MarshalText() on big takes a pointer. Is there a reason for this? Just a big object I guess (pun intended)?
Anyway, this is about how we'd like to use it. I'm wondering if you could take a good look and point out maybe how it might be incompatible or painful for our users. It doesn't look super fun, but it looks doable?

Hey there @ericlagergren. I had a branch all ready to pull request when I suddenly realized that I needed to implement our "randomize" interface. It's an interface that creates a random value for sqlboiler. All of our custom types implement this but there's a problem... It requires importing 3 packages from sqlboiler -_-. Some interface huh? (There's one argument that wants the randomize package, /sigh. Why did I do this?). One day maybe I'll refactor that out but until then it made no sense to ask your library to depend on something insane like that in any way, so I opted to implement on our side.
The implementation is here. It has that nasty interface we spoke about but that's just how life is going to be. If people want they have the ability to replace this type with any type they want via our configuration file (so they could go to shopspring to get more bugs, less speed, but have that nicer interface).
https://github.com/volatiletech/sqlboiler/blob/v3/types/decimal.go
Any tips or advice for implementing or things I've missed please let me know. Also I found that there were times the database driver was giving me back a float64, how does the postgres type you made not deal with this case?

Well dang! Perhaps some other time. I didn't see anything glaring in the code you linked. Whenever I've used postgres (via lib/pq) it've always received DECIMAL back as a string (or []byte, it's been a while since I've used it).

One day maybe I'll refactor that out

@aarondl: You already refactored the randomizing dependency out. So, can this issue be tackled again to integrate NullDecimal here?


spacefiller or filler

Dear @erc-dharma/nusantara-philology,
You have two values for <g>/@type: spacefiller and filler. Is there a difference between the 2?
Best,

it should be the same @type in every instance.
Le 14 avr. 2020 à 11:23, ajaniak <notifications<EMAIL_ADDRESS>a écrit :
Dear @erc-dharma/nusantara-philologyhttps://github.com/orgs/erc-dharma/teams/nusantara-philology,
You have two values for /@type: spacefiller and filler. Is there a difference between the 2?
Best,
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/erc-dharma/tfd-nusantara-philology/issues/4, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGMAEZ4AXUG2TXBJE4VN63RMQTSTANCNFSM4MHST76A.

As far as I remember, I use "filler" instead of "spacefiller", as indicated in EG 4.2.

It seems I use exclusively the attribute filler in my file.

I made the change, the file concerns is Astadasa which seems to be a test of some sort. If not relevant anymore, maybe you could create an archives folder.


Better documentation is badly needed

PLEASE READ THE CONTRIBUTION GUIDELINES HERE!
Failure to comply may have your issue automatically closed.
Summary
Please answer the following questions in sentences.

What is the feature?
Please provide us with better documentation based on the examples you've given, namely nested lists.

Why is having this feature important to you?
There are many features of Sortable that I don't understand how to use. I am unable to use nested lists because I don't understand how they work and how the indexing is taken care of.

Who else will find having this feature helpful?
Essentially, anyone wanting to use Sortable. I've been using Sortable in a very simple example, but I can't take it much further to use nested lists, for instance, because I just don't understand how the more advanced features work.

Additional Context
I'm using Sortable in a Firefox extension called Context Search.

Better documentation of event handler environment. Specifically what is the value of this in event handlers?


Add Trace

This is to try out a new tracing API (https://github.com/rmosolgo/graphql-ruby/pull/4344). It looks like it will work fine.
I used a lot of duplication as a quick way to test my changes, and then, if the new tracing API is adopted, it will be easy to remove the old code. But another option would be to:

Include Trace in Tracer (since it's a module anyway) and call the capture_* methods from there, to reduce duplication
Extract the assertions from the integration test and share them between TraceTest and Test, rather than duplicate them all

Let me know what you think, I'd be happy to make those changes (or others).
Along the way, I found that some tests here fail on master, presumably because my refactors to Language::Visitor and Analysis::AST::Visitor (https://github.com/rmosolgo/graphql-ruby/pull/4338) changed the order of these arrays. Is that OK?

(I signed the CLA, but I don't see an option to re-run the job myself.)

:+1: I updated it to sort those arrays before comparing them, so it should pass on all versions. And I added a new gemfile to run tests on my WIP branch.

🎉  looks good to me. The method based tracer is a much better interface

Thanks for taking a look. I added a benchmark result above. I'll sleep on it and merge that graphql-ruby PR soon, then update this accordingly :+1:

😅 Alright, tests pass locally!


Poll: Nativefier use cases

Hi guys, I'm working on WebCatalog which shares many similarities with Nativefier. At the moment, unlike Nativefier, you cannot freely install any app you like but soon, it will change. I'm looking forward to create a CLI for WebCatalog.
But WebCatalog works differently as it uses one single Electron instance instead of using independent instance for each app. So do you guys prefer Nativefier approach or WebCatalog? And what are your use cases? If Nativefier still offers advantages over WebCatalog, I'll will make a forked version of this repo and continue the development. If not, I'll concentrate my time on improving WebCatalog.

#326 #327.

From what I see webCatalog's destinations are the end users, as nativifier for devs/more tech users. I for example use it to wrap the website and offer them as download version, so web catalog is not an alternative for me.

@BTooLs So when you use Nativefier, how do you provide updates? For example, you create an app for your website and release to your users. Then you need to update Chrome. How do you do it? If the development of Nativefier moves on, do you think it's better to make it works specially for devs?

I haven't reached that far, I am just starting, but I will not concern about the Chrome version, we are still using JS5 (babel) because not all the users have new browser versions, so this problem will take at least 1Y to take effect in which it will became irrelevant (OS reinstall, user buy new PC etc). Mainstream web apps are always living in the past :))
As for Nativifier I think there's already a long list of features request in the issues. My first one was only the output size.
Thanks, for me it just worked, and the inject feature solved some issues, including a way to distinguish desktop vs web sessions.

@BTooLs Thank you! It's clear now.


updating batch after episode is terminated

In episode_runner.py, why are we still selecting one more action + updating the batch with it after the episode is terminated? https://github.com/oxwhirl/pymarl/blob/73960e11c5a72e7f9c492d36dbfde02016fde05a/src/runners/episode_runner.py#L89
        last_data = {
            "state": [self.env.get_state()],
            "avail_actions": [self.env.get_avail_actions()],
            "obs": [self.env.get_obs()]
        }
        self.batch.update(last_data, ts=self.t)

        # Select actions in the last stored state
        actions = self.mac.select_actions(self.batch, t_ep=self.t, t_env=self.t_env, test_mode=test_mode)
        self.batch.update({"actions": actions}, ts=self.t)

The environment terminating doesn't necessarily mean the final state is a terminal state (it might terminate after reaching some pre-defined limit on the length of an episode for example).
We can then use the information about which actions the agents would have taken in that non-terminal state depending on the algorithm. e.g. in SARSA we can use that final state as a bootstrapping target if we know the joint-action that would be taken.


Text-overflow on UI and PDF

Hi there!
When I type a long text, the text overflows in the UI and this overflow happens in the generated PDF too!
Here is a picture of it, notice the horizontal scrollbar..

I tried multiple themes but that seems useless.
Do I need to wipe something? Enable an option? Write less? ^^
Thank you in advance!

The text overflows appears when your write somewhere a long word (here todo)
Whitespaces will fix the issue.
Sorry, this is not as strong as an issue as I originally thought

+1 to this issue.
While it is unlikely longer words will be needed, and such cases can be worked around trough adding a space, this workaround does not work for email addresses.
This is problematic for the References section, as longer emails can be common depending on company email policies and domains (e.g. complete.name@company.com.mx).
See attached example

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :)
Thank you!


Authentication with Google SignUp Functionality

Added a sliding Login and Sign-Up Pages
LOGIN:

REGISTER:

Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊

hello @BamaCharanChhandogi
i was wondering if i can do the backend of auth too. i ensure to complete it within the schedule. since it is a different logic altogether please dd suitable labels accordingly.

Thanks for working. But you have to change the design part. The design is not looking good, fix it first of all.
Navabr and footer are not looking good with this login page.

could you review the PR? @BamaCharanChhandogi

Fix merge conflicts then I can merge

could you review the updated changes made in the PR? @BamaCharanChhandogi

Fix merge conflicts then I can merge

We don't need this right now.


How to set it to 16:10

hi
Is there a 16:10 setting, I don't know how to set it to a fixed ratio of 16:10
thank you

If I understand your issue correctly, you can use fillToContainer option as documented here. https://docs.fluidplayer.com/layout_configuration/#filltocontainer. After, you can resize the container to match whatever aspect ratio you need.
Please, feel free to re-open the issue if you need anything else.


Add serializer options to fix #68

The default C# serializer will serialize strings containing an apostrophe as \u0027 as some deserializers don't handle this correctly which can lead to escaping the containing string and causing security issues. Ollama however handles this correctly but the conversion from apostrophe to unicode code is not undone when deserialized. This leads to the prompts containing literal unicode codes instead of the intended character which can cause odd behavior from models.
Since the Ollama deserializer handles the apostrophes correctly we can use the UnsafeRelaxedJsonEscaping encoder to tell the serializer to not use this behavior and resolve this.

I saw your issue and thought a lot about it. I actually don't feel super comfy with relaxing this but on the other hand, I can't see a possibility where this might cause harm on the calling or the called side. I merged your commit to another branch where I will go over it again.
Thanks a lot for your contribution, Plootie!

See #68

I saw your issue and thought a lot about it. I actually don't feel super comfy with relaxing this but on the other hand, I can't see a possibility where this might cause harm on the calling or the called side. I merged your commit to another branch where I will go over it again. Thanks a lot for your contribution, Plootie!

Thanks! I was also dubious about changing this myself, however this appears to be the only way to make apostrophes encode properly (See my attempts on dotnetfiddle. However reading on this issue I found this Microsoft documentation which affirms that this is a safe thing to do so long as the payload will be interpreted as UTF8 Json on the receiving end which is the case here.

I noticed that this JsonSerializerOptions is utilized only for serialization. Would it be possible to provide a similar option for deserialization as well?
My primary use case is to add a TypeInfoResolverChain and support native AOT.

That's intentional, as I wanted to keep the default settings for incoming stuff, that means deserialization.
I might do the same for deserialization the coming week as I am not at home. Let me I kindly invite you to create a pull request for this, I can approve it remotely.


[FR] pre-commit integration

pre-commit (http://pre-commit.com/) might be a nice way to wrap up various linters, including the rustfmt that currently executes via ad-hoc build script. pre-commit is python-centric but Rust integrations already exist (e.g., https://github.com/doublify/pre-commit-rust).
Happy to scratch my own itch and submit a PR if there's interest.

Ah, looks cool; this might be a better thing to do then. Go ahead and submit a PR if you'd like.

btw, would be cool to integrate something to address #111 in this too!


Does critical error need to report UEC to RAS?

If a critical error occurs, should the core stall the pipeline and report the Uncorrected Error Critical (UEC) to the RAS unit?

The critical error state entered as a result of an unexpected trap. The unexpected trap may be due to a hardware error exception which is an exception that occurs when corrupted or uncorrectabe data is accessed by  an instruction. Such hardware errors that  raised the hardware error exception may report an RAS error. However entering  critical error state by itsef is not considered a hardware error and as such not reported as a RAS error.

Hope that helped.


Create output file atomically

This change is useful if you both (a) run alex on modifications to .x-files and (b) :reload GHCi on modifications to .hs-files.  It prevents that your GHCi session will see partial source files.
As things are you will initially get an error of the form:
src/Lexer.hs:1:1: error:
    File name does not match module name:
    Saw     : ‘Main’
    Expected: ‘Lexer’
Failed, no modules loaded.

(as initially the output file is empty)
Considerations:

[ ] If we go with this then we will need to drop support for GHC < 6.10.1 (as we don't have compat code for bracket and I'm not eager to burn cycles on that).
[ ] We could do the same thing for .info-files (personally however,  .hs-files is what I care about).
[ ] We could guard this behind a flag (e.g. --atomic-output or something).

Thoughts?

@sol, thanks for the PR!

If we go with this then we will need to drop support for GHC < 6.10.1

I don't see a problem with dropping GHC versions < 7 because we anyway do not test them.  CI goes back to 7.0 only.

We could do the same thing for .info-files (personally however,  .hs-files is what I care about).

This could be good for code uniformity.  At least if there is similar code in other places, it should be changed in sync.

We could guard this behind a flag (e.g. --atomic-output or something).

I don't think that would be necessary.
One question: Can the memory footprint of alex increase significantly by doing the write atomically?  (Not sure how relevant this question is, but it came to my mind and you may have an answer to this.  It could increase if e.g. the writing was triggering a lazy computation of the output, and this now doesn't work like this anymore.)

One question: Can the memory footprint of alex increase significantly by doing the write atomically?

No, it doesn't. This patch creates a temporary file and later renames it. For that reason, the memory characteristics are unchanged.
The downside of this approach is that we create that temporary file on the file system and trigger corresponding a inotify events, which I don't really like. The pros are that this is a common and well understood approach to guarantee a atomicity when creating  files + it's easy to implement without the risk of unintentional side effects.
An other approach would be to construct a ByteString (or rather a builder I guess) in memory first, and finally write that to a file. While this is not atomic it's likely fast enough so that you can get away with it + you don't get the additional inotify events for the temporary file, which I would prefer. In this case you will need ~0.5mb for the builder (for a parser of the size of what GHC uses; possibly more due to the builder overhead not sure) which I think in itself is not a big issue, but you also run the risk to change the memory characteristics unintentionally and may need to seq things away.
A third and somewhat half measured approach would be to try do a little bit more work upfront (before opening the output file), so that the output file stays empty for a shorter period of time. This may or may not be enough to get away with it. I haven't looked into this at all.

@andreasabel before moving forward with this, I will dogfeed it for a little bit longer.


update to 1.3.5 and add osx_arm64 build

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

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.
I do have some suggestions for making it better though...
For recipe:

License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

Ready for review @guyer @isuruf.

Also pinging @conda-forge/core for a review


No HTTP Agent connectivity in Loopback-routed L3 network

Bug report
General Information
Hey.
I'm trying to use cilium 1.9.6 in a fully routed network with loopback routing and can't get it to work.

Cilium version 1.9.6 38dd27a
Kernel version 5.10.36-0-lts #1-Alpine
Orchestration system version in use k8s v1.21.1, kubeadm v1.21.0, helm v3.4
Link to relevant artifacts (policies, deployments scripts, ...)
Generate and upload a system zip: Sorry, the dump seems to contain a lot of info I do not feel like sharing. The below hopefully contains all necessary info, if it does not, I'll gladly provide more after removing sensitive info.

Following is rather complete description of the setup. Hostnames and IP addresses changed to protect the innocent.
I have 3 nodes running Alpine Linux 3.13 with the above kernel.
Network setup
Each node has several IP addresses assigned to interface lo along with routes (including a default route) set with the appropriate source IP to use.
FRR runs on each member and - after enough cursing - speaks eBGP with the switches.
node1# ip a l lo
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1000
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    <SNIP>
    inet WW.XX.YY.ZZ/32 scope global lo
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/32 scope global lo
       valid_lft forever preferred_lft forever
    <SNIP>

node1# ip r
default nhid 13 via <IP_ADDRESS> dev eth2.5 proto bgp src WW.XX.YY.ZZ metric 20
<SNIP>
<IP_ADDRESS> nhid 14 via <IP_ADDRESS> dev eth2.5 proto bgp src <IP_ADDRESS> metric 20
<SNIP>

Kubernetes setup
tl;dr: Done with kubeadm, setting up kubernetes 1.21 on CRI-O 1.20.0 without kube-proxy and with dualstack enabled.
There is a haproxy in front of all the nodes, properly set as the controlPlaneEndpoint.
node1 initializes the cluster, the networking related part of the init config is this:
node1# magic-cat --only-relevant-info kubeadm-init.yaml
apiVersion: kubeadm.k8s.io/v1beta2
kind: InitConfiguration
localAPIEndpoint:
  advertiseAddress: "<IP_ADDRESS>"
  bindPort: 6445
nodeRegistration:
  criSocket: /var/run/crio/crio.sock
  kubeletExtraArgs:
    cgroup-driver: "cgroupfs"
    node-ip: "<IP_ADDRESS>"

node1# kubeadm init --skip-phases=addon/kube-proxy --config kubeadm-init.yaml --upload-certs
<SNIP>

Following nodes use a similar config (but as a JoinConfiguration).
Up until here everything works fine.
Side note: This is not much config, but it took me a long time to get the etcd cluster to properly bind and work. :(
Cilium setup
This is where things got really tricky for me.
I want to use native routing, because, after all, the only L2 networks are point-to-point links. And dual-stack IPv6, but I'm not sure if that's fully working.
Using Helm to install cilium seemed to be the easiest.
Below are the changed values from values.yaml, I didn't wanna paste the whole config here, but my actual config is a tweaked default values.yaml, in case that matters.
k8sServiceHost: my-k8s-haproxy.tld
k8sServicePort: 6443

bpf:
  masquerade: true
  hostRouting: true
  tproxy: true

containerRuntime:
  integration: crio
  socketPath: /var/run/crio/crio.sock

endpointRoutes:
  enabled: true

hostPort:
  enabled: true

hostServices:
  enabled: true

ipam:
  clusterPoolIPv4PodCIDR: "<IP_ADDRESS>/16"
  clusterPoolIPv6PodCIDR: "AAAA:BBBB:CCCC:DDDD::/104"

ipMasqAgent:
  enabled: true

ipv6:
  enabled: true

k8s:
  requireIPv4PodCIDR: true
  requireIPv6PodCIDR: true

kubeProxyReplacement: "strict"

nativeRoutingCIDR: "<IP_ADDRESS>/8"

tunnel: "disabled"

I've tried a few things before this, but I seem to get the furthest with this config.
After deploying the chart with helm, I get these pods:
node1# helm install cilium cilium/cilium --version 1.9.6 --namespace kube-system --values cilium-values.yaml
<SNIP>
node1# kubectl get pods -A
NAMESPACE     NAME                              READY   STATUS    RESTARTS   AGE
kube-system   cilium-jc2s4                      1/1     Running   0          5d21h
kube-system   cilium-ktrt6                      1/1     Running   0          5d21h
kube-system   cilium-operator-d4548c5bd-k2qmw   1/1     Running   0          5d21h
kube-system   cilium-operator-d4548c5bd-pszk8   1/1     Running   0          5d21h
kube-system   cilium-zgf46                      1/1     Running   0          5d21h
kube-system   coredns-558bd4d5db-85kmb          1/1     Running   0          6d
kube-system   coredns-558bd4d5db-pctzf          1/1     Running   0          6d
<SNIP>

That seems fine to me, but after consulting cilium-health on the cilium pod running on node1:
macu-01:~# kubectl exec -itn kube-system cilium-zgf46 -- cilium-health status
Defaulted container "cilium-agent" out of: cilium-agent, clean-cilium-state (init)
Probe time:   2021-05-21T15:31:32Z
Nodes:
  node1 (localhost):
    Host connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=440.178µs
      HTTP to agent:   OK, RTT=489.691µs
    Endpoint connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=585.353µs
      HTTP to agent:   OK, RTT=651.984µs
  node2:
    Host connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=611.354µs
      HTTP to agent:   OK, RTT=763.209µs
    Endpoint connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=652.309µs
      HTTP to agent:   Get "http://<IP_ADDRESS>:4240/hello": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
  node3:
    Host connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=548.609µs
      HTTP to agent:   OK, RTT=1.089773ms
    Endpoint connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=615.129µs
      HTTP to agent:   Get "http://<IP_ADDRESS>:4240/hello": context deadline exceeded (Client.Timeout exceeded while awaiting headers)

This really confuses me.
From the node itself, I can ping the addresses too, just like the cilium container. nmap shows no open ports, just filtered:
node1:~# nmap <IP_ADDRESS>
Starting Nmap 7.91 ( https://nmap.org ) at 2021-05-21 17:38 CEST
Nmap scan report for <IP_ADDRESS>
Host is up (0.00030s latency).
All 1000 scanned ports on <IP_ADDRESS> are filtered

Nmap done: 1 IP address (1 host up) scanned in 21.31 seconds

The logs of the containers are long (since it has been running a while), these are all the warnings I could find:
cilium-zgf46
# Seems to not like my /sys/fs/bpf mount, weird, but that's another issue.
level=warning msg="================================= WARNING ==========================================" subsys=bpf
level=warning msg="BPF filesystem is not mounted. This will lead to network disruption when Cilium pods" subsys=bpf
level=warning msg="are restarted. Ensure that the BPF filesystem is mounted in the host." subsys=bpf
level=warning msg="https://docs.cilium.io/en/stable/operations/system_requirements/#mounted-ebpf-filesystem" subsys=bpf
level=warning msg="====================================================================================" subsys=bpf

# Frequently occurs, but irrelevant.
level=warning msg="discovery.k8s.io/v1beta1 EndpointSlice is deprecated in v1.21+, unavailable in v1.25+; use discovery.k8s.io/v1 EndpointSlice" subsys=klog

# Those two lines seem odd, but I think they are also unimportant.
level=error msg="Command execution failed" cmd="[iptables -t mangle -n -L CILIUM_PRE_mangle]" error="exit status 1" subsys=iptables
level=warning msg="iptables: No chain/target/match by that name." subsys=iptables

# These seem to be problematic. eth2.5 is correct, though.
level=info msg="k8s mode: Allowing localhost to reach local endpoints" subsys=daemon
level=warning msg="Cannot determine a device from k8s Node IP addr for BPF NodePort device detection" error="Cannot find device with <IP_ADDRESS> addr" subsys=daemon
level=info msg="Using auto-derived devices for BPF node port" devices="[eth2.5]" directRoutingDevice=eth2.5 subsys=daemon

# Load Balancer of the Control Plane seems to fail a request once or twice during the day.
level=warning msg="Network status error received, restarting client connections" error="Get \"https://my-k8s-haproxy.tld:6443/healthz?timeout=32s\": write tcp <IP_ADDRESS>:49168-><IP_ADDRESS>:6443: write: broken pipe" subsys=k8s

# This seems to be a couple days in. Probably unimportant.
level=info msg="arping failed" error="interrupted system call" interface=eth2.5 ipAddr=<IP_ADDRESS> subsys=node-neigh-debug

Regarding the bpf mount:
node1# grep bpf /etc/fstab
bpffs           /sys/fs/bpf     bpf     rw,shared 0 0
node1# mount | grep bpf
none on /sys/fs/bpf type bpf (rw,relatime)

The Cannot find device with <IP_ADDRESS> warning seems bad, but I don't think it relates to the issue.
The operator logs seem fine.
I have torn down and reinstalled k8s and cilium many times.
Again, if I failed to provide any info that might help, I'll gladly provide it upon request.
I hope you can help me. :)

@brb Do we support lo for BPF NodePort now?

Do we support lo for BPF NodePort now?

We should in v1.10 (requires the L2-less dev patch which was merged in v1.10). @vifino Try with Cilium v1.10.

Hey @brb,
I tried with v1.10, took the new values.yaml and changed the same values (apart from the runtime stuff, I properly marked the /sys/fs/bpf mount as shared, option shared in fstab didn't do the trick).
Shortly after the containers are up, I get the health check to succeed, but after a few seconds it goes back to the HTTP check not succeeding.
Just like before, routes are announced to the other hosts just fine, ICMP ping works, but no HTTP traffic. Spawned a basic container, same thing happens then. ICMP works, no HTTP.
Now there are no errors or warnings in the cilium pod logs anymore, either.
What can I do?

@vifino I would suggest to check with tcpdump and cilium monitor what happens to the failing requests.

Sorry for the late response.
I've tried to capture what exactly went wrong, but best I could tell was cilium containers requests being NATed to the host's IP, which is set as the src attribute of the corrosponding route. Sorry.
The IP range is definitly in the "don't NAT" range (nativeRoutingCIDR), but it seems to do it anyway. Probably something to do with the src attribute and handling thereof.
Should only use it when NATing, not when routing.
I'd really appreciate someone else attempting this - a fully routed topology over the loopback interface - with more k8s and cilium knowledge.
I will gladly provide configs or help setting up a testing environment, but I can't figure it out, unfortunately.

cilium containers requests being NATed to the host's IP, which is set as the src attribute of the corrosponding route. Sorry.

Mind elaborating which exactly request it was (who is a sender and who is a receiver)?

The request was from node02 to the cilium pod on node01, source natted to node02's IP address.

I think your setup might be very similar to the one mentioned in https://github.com/cilium/cilium/issues/16539 (cc @Frankkkkk).

Disclaimer: My setup is only IPv6 without NAT. It works quite well however indeed the biggest "trick" is to remove the kernel route of the IPs to lo (so that the route to cililum_host is used instead). Without it the Node IPs will ping (cause it reaches the host), but not the services.
Also, contrary to your setup, we add < /128 (I think /110 or something like that) IPs to lo. This was a technical issue because we couldn't BGP announce thousands of IPs, so each node announces a (pod) range and pod ips are allocated from this range (ipam.mode = kubernetes).
On an early iteration, we had a dummy interface named bgp-announce (ip link add bgp-announce type dummy) that kept all the IPs to be announced (and it worked fine), but we decided to add them to lo instead.


How to initialize the database?

Hi and thanks for the great package. Testing it with with FastAPI I get 500 Internal Server Error trying to access http://<IP_ADDRESS>:8000/docs or http://<IP_ADDRESS>:8000/openapi.json.
Does anyone have the same problem?

Sorry I realized I forgot to create tables. Btw, how to do that? I found this but I'm not sure if it's the most proper way. https://github.com/encode/databases/issues/240#issuecomment-682136715

Thanks for the first user issue! ✨
Under the hood ormar uses sqlalchemy to manage the tables (and construct the queries).
So all sqlalchemy/alembic tutorials are valid for ormar in this regard.
I updated the docs to avoid this confusion - thanks for pointing that out!
You can visit the relevant section here: https://collerek.github.io/ormar/models/#database-initialization-migrations

I assume that solves your issue, please reopen this issue if you need or create a new one if something else pops up.


Release 4.0.8

Coverage remained the same at 31.106% when pulling 59f0672c622c6fa2ee9723ea0f99fd76bf8b2999 on Hadis-Fard:release-4.0.8 into 949ca7f27c97d909306cb9dc184cc9c656b8df7c on Microsoft:PHP-7.0-Linux.


chore(main): release 2.5.3

:robot: I have created a release beep boop
2.5.3 (2024-12-19)
Bug Fixes

. (4b8213d)

Feature Improvements

外置浏览器渲染 (ba60514)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/rainbowwarmth/KazuhaBot/releases/tag/v2.5.3 :sunflower:


Add memcache to Loki for better performance

Hi Bruce,
After many scenarios test I suggest you to implement memcache on top of Loki to have a smoother experience.
You can find some information about it here https://grafana.com/docs/loki/latest/operations/caching/.
If you need any help feel free to ask.
Ciao

Sounds like a great idea... Loki is deployed in monolith mode at the moment (I'm not too worried about performance, just that it works), but perhaps it's time to start learning the finer details. If you'd like to send even a partial PR, we can hack it together.

I've pushed the PR #99, It adds memcached nomad job.

Closed by #99


Fix XMonad.Prompt wraparound when maxComplRows not Nothing #217

Description
When tabbing through the prompt list of completions, if the list of completions does not fit due to maxComplRows, without this fix an exception would be thrown.  I am not certain why, but this exception occurs due to a pattern mismatch at https://github.com/mgsloan/xmonad-contrib/blob/dda242a4597c1ed0fa895a10b9296dcc709f4937/XMonad/Prompt.hs#L919
However, the root cause appears to be in the nextComplIndex function.  The problem is that it was assuming that dividing the total item count by the number of rows would yield the number of columns.  This did not hold when the number of rows was being limited.
Checklist

[x] I've read CONTRIBUTING.md

[ ] I tested my changes with xmonad-testing

[x] I updated the CHANGES.md file

Ping!  Any maintainer want to review this?  I've rebased it atop master.

Thanks for the PR and for the nudge!

You're welcome, thanks for merging!
On Tue, Feb 12, 2019, 07:08 Brent Yorgey<EMAIL_ADDRESS>wrote:

Thanks for the PR and for the nudge!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xmonad/xmonad-contrib/pull/291#issuecomment-462795720,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABYKrYF7Pbvwx8aYZJ4KU0Ts9GcvMKGks5vMtj5gaJpZM4aQN4F
.


Experimental Anomaly Vessel

About the PR

First on the chopping block for #21055
The experimental anomaly vessel has double point output at the cost of a mild radioactive field and an increased explosion.
Why / Balance

Anomaly vessel upgrades are stupidly strong and i'd rather just try and have some kind of balanced strong-ness rather than have it be all over the place.
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

remove: Removed upgrades from the anomaly vessel.
add: Added the experimental anomaly vessel, which researches twice as fast at the cost of being radioactive.

i want it to emit a tremendous amount of rads during the supercrit ANIMATION then explode

I think better save upgrades of default vessel.
Just decrease the max level numbers.
Like (100% = standart points):

vessel from 100% to 150% (200%?)
experimental vessel from 200% to 400% (500?)


Simplify Managed API to avoid dealing with PCollectionRowTuple

Fixes #31375
Opening up Managed API to consume a PInput and validating at expansion time. Supported types now include:

PBegin
PCollection
PCollectionRowTuple

This also sets us up in the future to add support for Java Schema'd objects

R: @chamikaramj
R: @VeronicaWasson
R: @damccorm

cc: @weifonghsia for input on this and the Managed API in genera.

LGTM. Thanks.


Data Transfer calendar release ressources

Hi @scrthq , i can't find any solution to your Start-GSDataTransfer command for apply the release_ressource parameter.
I imagine is possible cause "GAM" do it but I don't want to use gam only for this so if you got any hidden feature or whatever to do it I'll be very happy :)
For "help" there is the gam command in first and my command in 2nd i want to implement the release ressource :

gam create datatransfer<EMAIL_ADDRESS>calendar<EMAIL_ADDRESS>release_resources true

Start-GSDataTransfer -OldOwnerUserId $username -NewOwnerUserId $Ownershipuname -ApplicationID 435XXXXXXXXX -PrivacyLevel SHARED

Alex

Hey @ABricka - It looks like that Start-GSDataTransfer needs to be updated to support it! Privacy_Level is Drive and Docs specific, but I'm seeing the RELEASE_RESOURCES TransferParam on the Calendar DataTransferApplication specifically:

Considering this a feature request, I'll let you know once available!

Super :) That's a great new !
Waiting about you to use it :)
Thanks a lot for this fast reaction 👍

Hey @ABricka - added support for this in 2.31.0! Usage (note that ReleaseResources is a [Switch] parameter, so no need to pass $true):
$calendarApp = Get-GSDataTransferApplication | Where-Object {$_.Name -eq 'Calendar'}
Start-GSDataTransfer -OldOwnerUserId $username -NewOwnerUserId $Ownershipuname -ApplicationID $calendarApp.Id -ReleaseResources

Super, thanks a lot 👍

You bet!
Sent with GitHawk


Register?

This already seems to be super useful: https://github.com/JuliaDiffEq/ModelingToolkit.jl/pull/173 . When do you plan on registering it?

Waiting for this: https://github.com/thautwarm/GG.jl/pull/5#issuecomment-528067384
Maybe done in few days...

@ChrisRackauckas  Hi Chris, could you help me with the registration of a GG's dependency?
It's JuliaVariables.jl.

Let it take a few days. Ugh I was scrolling on my phone and it clicked... it should take 3 days for a new package.

There's no rush here.

Oh, it has already taken 3 days: https://github.com/JuliaRegistries/General/pull/3315

Thanks!


Microsoft cognitive Services package Seems to cause UnityEditor to hang when switching between projects.

Describe the bug
When i include the Microsoft cognitive Services package in my Project, and change application focus between unity and visual studio, when returning to unity, unity will just hang with spinning wheel, until i quit/relaunch it. if i take out cognitive services asset package, the same project works fine.
To Reproduce
Steps to reproduce the behavior:

Use the a project with cog services bundle in it.
..

Expected behavior
Normal recompile behavior, when switching between VS/Unity.
Version of the Cognitive Services Speech SDK
1.20.0
Platform, Operating System, and Programming Language

Windows
Intel x64
Programming language: C#

Additional context

The STT is working well aside from the above crash.
Any other additional information: I am developing for oculus quest 2
if you want the specific script/project that i am using to eliminate that as the cause, please send me a PM.

@Matt-Finlay
I'll try to reproduce this with our public Unity samples.
What version of Unity are you using?
Also, is any use of our package, or is there a particular sample that you have seen this happen with?
Thanks,

@Matt-Finlay
Thanks for telling us know you resolved the issue. What was the issue you fixed?
Was this related to disposing of objects correctly?
What could we document better, so you could have root caused this quicker?
Thanks,


[DOCUMENTATION] [QUESTION] Exporting fiftyone dataset as fiftyone dataset

Hi,
Let's assume the following scenario:
I started to work on unlabeled dataset on one machine and applied some calculations on the top of it (such as- calculate image uniqueness, applying predictions from pre-trained model).
What is the best way to export existing fiftyone dataset not in yolo/coco/voc format, but in original fiftyone format?
The goal is to keep all of the metadata that has been generated (such as predictions confidence, for example) and continue to visualize/analyze the results on another laptop/desktop.
I was unable to find such kind of how-to in tutorials.

You're looking for FiftyOneDataset format 🎉
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset("quickstart")

dataset.export(
    export="/tmp/test",
    dataset_type=fo.types.FiftyOneDataset,
)

dataset2 = fo.Dataset.from_dir(
    dataset_dir="/tmp/test",
    dataset_type=fo.types.FiftyOneDataset,
)


Add Thar token and Thar Logo to TrustWallet

Token Name: Thar token
Symbol: Thar
Contract Address: 0x96c30D5499EF6eA96A9c221Bc18BC39D29c97F27
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwallet.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp