Getting a error reading 'AuthHandler'

I get a error 500 page with the server error below after using both pnpm or yarn with the base scaffold project.
Not able to get around this one.
Tried both node version 16 and 18
Don't know if it helps but I'm on a M1 Macbook Pro
Cannot read properties of undefined (reading 'AuthHandler')
TypeError: Cannot read properties of undefined (reading 'AuthHandler')
    at Object.eval [as handle] (/node_modules/.pnpm/@auth+sveltekit@0.1.5_svelte@3.55.0/node_modules/@auth/sveltekit/index.js:37:19)
    at respond (file:///Users/mikhailkaran/Development/sveltekit-setup/node_modules/.pnpm/@sveltejs+kit@1.0.0_svelte@3.55.0+vite@4.0.1/node_modules/@sveltejs/kit/src/runtime/server/index.js:229:40)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async file:///Users/mikhailkaran/Development/sveltekit-setup/node_modules/.pnpm/@sveltejs+kit@1.0.0_svelte@3.55.0+vite@4.0.1/node_modules/@sveltejs/kit/src/exports/vite/dev/index.js:444:22

@mikhail-karan I am experiencing the same issue.

This happens not only with this starter but with just following the documentation on a new sveltekit project.

Same - and the function signature for AuthHandler takes two arguments that aren't being passed in the example code and makes a call to /api/auth/session - which doesn't exist in SvelteKit.
I tried passing the Request that comes in as a prop from the load function on +layout.server.ts but I think the issue is deeper than that?

Experiencing the same as well, also on an M1 Macbook for what it's worth.

Okay it seems this is an issue with @auth/sveltekit's dependency still being on @auth/core v0.1.3. If I manually update the @auth/core dependency of @auth/sveltekit in my node_modules folder to 0.1.4, then it seems to work.

Looks like it just needs to be repackaged and republished to npm, which should fix it (running pnpm pack in the directory pins the core dependency correctly as 0.1.4)

GitHub issue created here: https://github.com/nextauthjs/next-auth/issues/6068

I had the same issue on development server but when i made a build for it it gave me another other which was
[next-auth][error][UNTRUST_HOST_ERROR]
https://next-auth.js.org/errors#untrust_host_error Host must be trusted. URL was: http://<IP_ADDRESS>:4173/api/auth/session UntrustedHost: Host must be trusted. URL was: http://<IP_ADDRESS>:4173/api/auth/session
    at AuthHandler (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/node_modules/.pnpm/@auth+core@0.1.4/node_modules/@auth/core/index.js:204:23)
    at getSession (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/chunks/hooks.server.js:12:26)
    at Object.event.locals.getSession (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/chunks/hooks.server.js:35:39)
    at load (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/entries/pages/_layout.server.ts.js:4:33)
    at load_server_data (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/chunks/index.js:946:42)
    at file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/chunks/index.js:1935:24 {
  code: 'UNTRUST_HOST_ERROR'
}
Error: There is a problem with the server configuration. Check the server logs for more information.
    at getSession (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/chunks/hooks.server.js:19:9)
    at async load (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/entries/pages/_layout.server.ts.js:4:14)
    at async load_server_data (file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/chunks/index.js:946:18)
    at async file:///Users/shivammeena/WebstormProjects/SvelteKit-Auth-Example/.svelte-kit/output/server/chunks/index.js:1935:18

When i looked more into it i found this piece of code
function SvelteKitAuth(options) {
  const { prefix = "/auth", ...authOptions } = options;
  authOptions.secret ??= AUTH_SECRET;
  authOptions.trustHost ??= !!(env.AUTH_TRUST_HOST ?? env.VERCEL ?? dev);
  return SvelteKitAuthHandler(prefix, authOptions);
}

Here we can see   authOptions.trustHost ??= !!(env.AUTH_TRUST_HOST ?? env.VERCEL ?? dev);  which is looking for AUTH_TRUST_HOST in env so we need to add this in our .env file. then you can make a build it will work.

Thanks, please reopen in the main repo https://github.com/nextauthjs/next-auth


HeapSizeOf for HashSet

r? @SimonSapin @notriddle @nox

Seems like a no-brainer to me. r+

@bors-servo r=notriddle

:pushpin: Commit e1712c8 has been approved by notriddle

:hourglass: Testing commit e1712c8 with merge 145c108...

:sunny: Test successful - travis


(feature) Add small library for interest rate calculation

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>

Coverage remained the same at 99.114% when pulling 4d966c969ee296779bf751caf53dea8586f0719f on jeromesimeon:js-feature-interests-lib into d42e7e622e96552b096b84e9e6c6e9f40d4c4ff5 on accordproject:master.


cmd/vet: false positive printf-like function detection for github.com/leonelquinteros/gotext

What version of Go are you using (go version)?

$ go version
go version go1.19.2 linux/amd64

Does this issue reproduce with the latest release?
Yes: https://github.com/divVerent/aaaaxy/actions/runs/3685154878
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/rpolzer/.cache/go-build"
GOENV="/home/rpolzer/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/rpolzer/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/rpolzer/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go-1.19"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go-1.19/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.19.2"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build3901397724=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Used "go vet" on a program that calls Get("100%") on a Po object. The method is defined here:
https://github.com/leonelquinteros/gotext/blob/6998e37f95499dad532f617cb0ff07fdbabc6ef7/po.go#L83
What did you expect to see?
This is valid, as the call goes through
https://github.com/leonelquinteros/gotext/blob/6998e37f95499dad532f617cb0ff07fdbabc6ef7/domain.go#L252
https://github.com/leonelquinteros/gotext/blob/6998e37f95499dad532f617cb0ff07fdbabc6ef7/helper.go#L37
I.e. this is only a printf-ish function if receiving more than one arg - if exactly one arg is provided, it is NOT interpreted as a format string.
What did you see instead?

# github.com/divVerent/aaaaxy/internal/font
Error: internal/font/charset.go:23:9: (*github.com/leonelquinteros/gotext.Po).Get format %& has unknown verb &
# github.com/divVerent/aaaaxy/internal/playerstate
Error: internal/playerstate/playerstate.go:237:10: (*github.com/leonelquinteros/gotext.Po).Get format % is missing verb at end of string
Error: internal/playerstate/playerstate.go:262:10: (*github.com/leonelquinteros/gotext.Po).Get format % is missing verb at end of string
Error: internal/playerstate/playerstate.go:266:10: (*github.com/leonelquinteros/gotext.Po).Get format % is missing verb at end of string

Looks like as a workaround I have to call
po.Get("100%s", "%")
to get both the correct output AND no go vet errors. This is kinda sad. Not so great for the translator, who will needlessly see a format string.
On https://github.com/golang/go/issues/17058 it has been agreed that we want NO annotation mechanism to silence vet warnings - so I guess this means we need to fix this false positive instead.

(BTW, as for why I use gotext and not x/text/messages - I really insist on a file format with lots of good editing tools such as gettext's AND need to load them from an in-game VFS [possibly the user's file system just like other game data]; I couldn't figure out how to do that with gotext).

@michaelmatloob @timothy-king

How exactly would you propose to fix this?

So maybe not detect the function as a printf function if the call to a
printf function is conditional?
On Tue, Dec 13, 2022, 09:25 Than McIntosh @.***> wrote:

How exactly would you propose to fix this?
—
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/57288#issuecomment-1348680036, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB5NMHIFSYANPYYQRW3PDDWNCBMVANCNFSM6AAAAAAS5IE6GI
.
You are receiving this because you authored the thread.Message ID:
@.***>

https://github.com/golang/go/issues/57288#issuecomment-1348764620 I think comes down to whether gotext.Printf is a "printf-like" function:
func gotext.Printf(str string, vars ...interface{}) string {
	if len(vars) > 0 {
		return fmt.Sprintf(str, vars...)
	}
	return str
}

I am curious what the author of this code (@leonelquinteros) thinks the decision should be. Is "gotext.Printf" a wrapper of "fmt.Sprintf" and it is desired for go vet users to be warned on format strings here? Or no because a different format (like having a trailing"%") should be supported? Or only when len(vars) > 0**?
This does not necessarily dictate what vet should do, but I am curious.
** I doubt a len(vars) > 0 requirement would make sense for vet. I am just interested if that would change opinions.

So maybe not detect the function as a printf function if the call to a
printf function is conditional?

Right now the printf.Analyzer checks whether there is some forwarded call. There are a few different ways we could choose to restrict based on a condition.

We could still consider a function a wrapper if all paths reach a printf wrapper. Pro: precise. Con: Challenging to retrofit & requires control flow.
We could choose to only consider it a wrapper if a call to a printf-like is in the func's block and no return statement or panic precedes it (i.e. some printf-like dominates the function's entry). Pro: Seems like it would capture most cases in the wild? Somewhat simple to implement. Con: Won't capture as many cases as the previous suggestion, if p { return fmt.Sprintf(fmt, args...) }; ...; fmt.Printf(fmt, args...); return "lorem ipsum";. Not clear if this distinction arises in practice.
We could stop if there is any preceding branching. Pro: Easy to retrofit. Con: Somewhat restrictive so hard to predict the impact, e.g. if p { irrelevant(); }; fmt.Printf(fmt, args...) would not longer be a wrapper.

Given the relative rarity of the issue, I think we should test candidate solutions before making a decision.

po.Get("100%s", "%")

@divVerent An alternative stopgap is to have a function that printf.Analyzer cannot infer is a wrapper. Such as:
 func foo(fmt first, args ...interface{}) {
   id := func(s string) string { return s } // https://github.com/golang/go/issues/57288
   po.Get(id(first), ...args)
 }

Not a great solution, but it may be preferable to po.Get("100%s", "%"). You would need one of these per printf-like function from another package.

Obscuring the format string with an id() wrapper or similar won't work as then xgotext will no longer detect these strings as translatable and put them in the .pot file.
However I found a workaround - it seems like xgotext just looks at the package name of the type of the receiver, not the type itself. And the package is so nice to define a Translator interface that *Po implements - so I can go via that interface whenever I need to evade go vet's printf detection.
This is still dirty and against the philosophy in https://github.com/golang/go/issues/17058 to NOT require explicit annotations of any form to evade false positives - instead go vet should not HAVE false positives.

What speaks AGAINST skipping printf detection if the function is conditional is, of course, constructs like log.V(1).Infof("hello, world"). And to be honest I don't really see a good way how go vet can distinguish
func (v V) Infof(format string, args ...any) {
  if v <= *verbosity {
    fmt.Printf(format, args...)
  }
}

(this SHOULD count as printf-like)
from
func (p *Po) Get(format string, args ...any) string {
  if len(args) != 0 {
    return fmt.Sprintf(format, args...)
  } else {
    return format
  }
}

(this is what this package is doing)

The verbosity example you gave would be blocked by all of my suggestions. So none of those seem like the right call.

The only real difference is that here in theory an analyzer could go through the entire thing and notice that the comparison is only based on the arguments and literals, while in the log.V case, a flag or otherwise external value is involved and the analyzer can prove nothing.

It might just be reading args or format in a context other than a matching call? Maybe it is just using one before a matching call?
Another distinguishing characteristic is fmt.Sprintf vs fmt.Printf.
Worth also pointing out that someof printf's unit tests would likely need to be modified to support the types of restrictions needed: https://cs.opensource.google/go/x/tools/+/refs/tags/v0.4.0:go/analysis/passes/printf/testdata/src/b/b.go

Hello,
I'm catching up with the thread after being summoned by Timothy.
In my opinion (which I just formed quickly after reading the thread), I think gotext.Printf shouldn't be considered a fmt.Sprintf wrapper, my reasons for that are:

The most common usage of gotext.Printf is without parameters and so not going through fmt.Sprintf
It's common to include special chars in translation strings that may or may not be using fmt.Sprintf
Another use case got gotext is to translate strings that will be used by some printf-like function after translation, so it's expected these contain control chars inside.
go vet is a tool that should help developers, and on this case the balance between true and false positives will be very even if not favoring the false ones, so I'd say better to skip this one.

For the specific needs of validating format strings and function calls inside gotext, I'd say it's better to create a tool inside this package to perform this tasks, I think this matches the spirit of the package as well.
That said, I'm not sure how vet is detecting the wrapping of fmt.Sprintf, so it may not be straight forward to exclude gotext.Printf from the detection heuristics.

That sounds good - xgotext can be the tool to do format string validation, after all.
Based on the above, vet apparently transitively follows the call chain to a printf function. There may be tricks to make it not notice a format string, like fmt.Sprintf(""+format, args...), although these tricks may stop working at some point when vet gets more clever.

Related #26555.

An alternative to fmt.Sprintf(""+format, args...) is _ = &format; fmt.Sprintf(format, args) as an "annotation" that a function is not "printf-like". There is an explicit check for it, and I don't think vet will get too clever for taking addresses any time soon.

I think gotext.Printf shouldn't be considered a fmt.Sprintf wrapper

It is a printf wrapper, since it may forward its arguments to printf. It does so only conditionally, but that's true of a lot of printf wrappers, such as loggers with a verbosity control. The only problem I see is that gotext.Printf interprets its arguments inconsistently: sometimes as a format string and sometimes as a string literal. This seems just as likely to confuse human readers as mechanical onset: if a Go programmer calls a function named Printf and wants to see the output "100%", they will follow Go conventions and pass the argument "100%%", and the function will not do what they expect.
I suggest you change gotext.Printf to delegate to fmt.Sprintf unconditionally.

I disagree with this solution - yes, the function called Printf really should not be called that.
But the functions I was about are NOT called Printf. They are called Get, GetC, GetN, GetNC.
And apparently the author intended them to not take format strings if there is just one arg.
In a /v3 of the package that may be something to reconsider - how do we handle this in /v2 though?
Suggestion: can we maybe have a global variable (with a function to enable this mode) in the package that enables always-formatstring handling so I can migrate my project to that?

yes, the function called Printf really should not be called that.
But the functions I was about are NOT called Printf. They are called Get, GetC, GetN, GetNC.

It's not just the name, it's the behavior of the function that's the problem. 'Get' interprets its 'format' argument inconsistently: sometimes as a format string, and sometimes not. The convention throughout the Go libraries is that functions choose one or the other (and ideally reflect this choice in their name).

And apparently the author intended them to not take format strings if there is just one arg.

I'm asserting that this was a design mistake, since it confuses not only vet, but human readers too. (Perhaps vet should attempt to report "function interprets format string argument inconsistently"?)

In a /v3 of the package that may be something to reconsider - how do we handle this in /v2 though?

If you want to disable vet checking for this function, then change its body as described above to obscure the delegation to printf. Or you could split Get into two functions, one Printf-like and one Print-like. If you can't change Get itself, you can write two wrappers, and use them instead. You can add if false { _ = fmt.Sprintf(format, args...) } to a function to force vet to interpret it as a printf wrapper, or use ""+format, as you pointed out, to do the opposite.)

Suggestion: can we maybe have a global variable (with a function to enable this mode) in the package that enables always-formatstring handling so I can migrate my project to that?

I'm not sure what you mean. If you can change the function, you can trick vet into treating it as as printf wrapper or not. If you can't change the function, there is no way to change vet's interpretation.

no way to get both the correct output AND no go vet errors. This is kinda sad.

To be clear, I agree, it is sad that an API design mistake in one of your dependencies causes vet to report a false positive. In effect, vet imposes a stronger type on Get than Go itself, and the type of Get cannot be expressed in this stronger type system, and, like any type system, it rejects some programs that would run just fine. But since there is a design mistake in the code and there are simple workarounds, I don't think it's worth trying to change vet to accommodate this case.

So the idea is that leonelquinteros puts in the ""+format hack into the library's /v2 as it is right now, and /v3 will use separate functions for literals and format string.
@leonelquinteros is that OK with you?

@divVerent I'm not sure about maintaining multiple versions of the package just for this change.
About the design decision, it's not a mistake, it was made on purpose and it's documented, in fact, it's the only documentation for that function: https://pkg.go.dev/github.com/leonelquinteros/gotext#Printf , so readers should be safe.
I agree it's ugly when seen from this perspective, but it was due to organic grow, as it deviated from gettext standard, but found useful for some package users.
As I said before, if you need to make vet not warn about this, even when the documentation explicitly say that  Vet uses heuristics that do not guarantee all reports are genuine problems (https://pkg.go.dev/cmd/vet) in the very first paragraph, there are several workarounds for your case and some have been already discussed in this thread.

I can foresee a /v2 version that has Get and GetWithParams (or something) for the 2 separate use cases, but starting a /v2 version would include a lot of other decision changes that have been discussed in the project before and that will require more effort, so ETA for that would be really long given the maintenance rate i put into the project this days.
PRs are always welcome though :) we can start in a branch for your specific use case and move from there


Feature/23 refactor component

Refactored vue components into installable vue plugins
Separated components to be used on different screen vda-account and vda-login
Added library documentation

NB: This supports only vue3 ATM due to compatibility issues with vue
see question about this on stack overflow https://stackoverflow.com/questions/63663608/can-vue2-components-be-used-in-vue3
#23

Name on login screen in Vault not coming from .env

Avatar image still not there


启动插件PHP错误，类已声明。

PHP错误：Fatal error: Cannot declare class Metowolf\Meting, because the name is already in use in /wp-content/plugins/nm/functions/Meting.php on line 11

你其他地方用了呗。。。


Changed -LocationName to -Location under ### View LTR backups

Changed -LocationName to -Location under ### View LTR backups

@jesinj22 : Thanks for your contribution! The author, @anosov1960, has been notified to review your proposed change.

@anosov1960 please review

#sign-off

@CarlRabeler: I'm sorry - only the author of this article, @anosov1960, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Port the postData fix

microsoft/playwright#5736

Test have moved: https://github.com/microsoft/playwright/commit/53a62a3af8c822806e91fe1aa2307b5779e3045b


[Code Quality] Too nested loop in UniqueSscList

Found in UniqueStudent/StudentScore/GradedComponentList

Abstract away middle part into function


dlb: add retry logic

When big traffic come, dlb_send() may fail due to temporarily unavailable resource.
Add retry logic to ensure every connection is sent successfully by dlb.
Signed-off-by: Loong<EMAIL_ADDRESS>Risk Level: Low
Testing: Add
Release Notes: N/A

ci errors look genuine:
contrib/network/connection_balance/dlb/source/connection_balancer_impl.cc:49:3: error: use of undeclared identifier 'retry_times'
  retry_times = dlb_config.retry_times();

@phlax thanks, have fixed it.

@adisuissa could we merge it?


AccessDenied when calling the PutPublicAccessBlock operation

Steps to reproduce the problem
from qiskit_braket_provider import AWSBraketProvider

provider = AWSBraketProvider()
backend = provider.get_backend("Aria 1")

qc = QuantumCircuit.from_qasm_file("...")
task = backend.run(qc, shots=100)

What is the current behavior?
Executing a Qiskit circuit, targeting a backend, throws this error:
botocore.exceptions.ClientError: An error occurred (AccessDenied) when calling the PutPublicAccessBlock operation: Access Denied
What is the expected behavior?
No error

Thanks for reporting the issue @epelofske-LANL! It might be the case that the IAM role/user you are using doesn't have the s3:PutBucketPublicAccessBlock permission. Could you please try adding that permission?

Closing this issue. Please reopen if you have further questions. Thanks!


Ask textual feedback only if bad review

Description
Is it possible to ask for textual feedback only if the review is negative or is not the best emoji review ?
Good feedback might be less interesting that bad feedback

Yes could be a nice feature. Is this something that you would be able to contribute?

I can take this one!


Crashing on choosing device

Hi, thanks for quick reply on LICENSE issue, that was really fast
I am working on Windows 10 x64
I finally managed to build the project and now I am crashing on choosing capture device.
I tried to look for some log files or something, but, honestly, I don't have much experience with C++, so I might have missed something.
Here is output of the program (I used my android smartphone as video device using DroidCam):
20:13:50: Starting C:\Code\ScoreboardOCR\scoreboardocr\build\Desktop_Qt_6_7_1_MinGW_64_bit-Debug\debug\scoreboardocr.exe...
Initializing filter manager...
Initiating caputure manager...
Initializing recognition manager...
Initializing capture scene...
Initializing capture scene...
Initializing capture scene...
Initializing main worker...
Initializing settings manager...
Loaded svm: "C:/Code/ScoreboardOCR/scoreboardocr/SVMs/7segment.svm"
SVM var count: 256
Found device: "\\\\?\\root#media#0001#{e5323777-f976-4f5b-9b55-b94699c46e44}\\global" | "DroidCam Source 3"
Found device: "\\\\?\\root#media#0001#{e5323777-f976-4f5b-9b55-b94699c46e44}\\global" | "DroidCam Source 3"
20:13:59: C:\Code\ScoreboardOCR\scoreboardocr\build\Desktop_Qt_6_7_1_MinGW_64_bit-Debug\debug\scoreboardocr.exe crashed.

When I press "Start Capture" everything crashes
So yeah, the main question for now is do you have logs generated somewhere?
If not then second question is how can I possibly debug this behaviour?

Hello @dmkirichen
Sorry for late response, had other things to do.
About logging - only way to get more logs is to add two OpenCV environment variables to the build:
OPENCV_LOG_LEVEL=DEBUG
OPENCV_VIDEOIO_DEBUG=1

This would enable logging for OpenCV, but there are no additional logs that you can enable in this project :)
I checked out this problem on my PC on several cameras and either the cameras worked or a appropriate error message was shown. Then I tried DroidCam and I could get the program to crash. Seems that OpenCV has a hard time with DroidCam, as it is what makes the program crash (in capturemanager.cpp->CaptureManager::initUSBCapture() you can add breakpoint to the code, where video capture is initiated to check this problem out).
I tried to fix this problem somehow (for openCV), but could not do it properly, maybe you can try to find a solution if you want :)
Anyways, I saw that DroidCam offered a possibilty to view video through IP, so I added IP camera support. That means that you can go to capture tab->IP capture and enter IP address of your camera (for example: http://<IP_ADDRESS>:4747/video , check DroidCam for this IP cam access), then press "start capture" and it should start working. Tried it with my phone and video was caputred and displayed in ScoreboardOCR.
If any more questions, feel free to ask :)
If not, you can close this issue.

Thanks, that is helpful feature since most of the sport operators use ip connections with cameras
I've managed to connect to DroidCam that way, so the issue can be closed


GH908: Add ability to upload test results when running under AppVeyor

This change adds an alias to the AppVeyor environment for uploading test results so that it integrates nicely with AppVeyor. This is supported by the AppVeyor API here: http://www.appveyor.com/docs/running-tests#uploading-xml-test-results
Currently test results display is supported by AppVeyor, but only if you use the test runners and framework using the appveyor.yml config.
This change allows tests to be run from Cake and then upload the outputted XML file to AppVeyor's test results page (see screenshot).
Fixes: #908

@robertcoltheart thanks for submitting this PR, looks really useful, left som minor feedback, could you adjust and rebase against latest develop. Ping us when you feel ready for review again.

@devlead Thanks, updated. Please take a look!

@robertcoltheart could you rebase / squash into 1 commit and I'll take a look tomorrow.

@devlead Done, although ugh, it seems to have changed the line endings...

@robertcoltheart I've tested and it works as expected, one thing would need to be changed though
To not need do enter full namespace to enum please add the following
[CakeNamespaceImport("Cake.Common.Build.AppVeyor")]
[CakeNamespaceImport("Cake.Common.Build.AppVeyor.Data")]

to
         public static ITeamCityProvider TeamCity(this ICakeContext context)
         {

in https://github.com/cake-build/cake/blob/develop/src/Cake.Common/Build/BuildSystemAliases.cs#L63
If you can manage to sort the lineending issues that would be great, otherwise no worries I can fix that while merging.
Ping me when addressed an I'll do a final review before merging.

@devlead Hi, do take a final look and let me know if anything is amiss.

@robertcoltheart LGTM :+1: Merged! Thanks for your contribution.

@devlead Np, thanks for your assistance!

Hi,
Is there any DSL or documentation for this?

It's documented here
http://cakebuild.net/api/cake.common.build.appveyor/3c90c0b2/cf706cd4
But basically it's like below:
if(BuildSystem.IsRunningOnAppVeyor)
{
    // Upload artifact to AppVeyor.
    BuildSystem.AppVeyor.UploadTestResults("./tests.xml", AppVeyorTestResultsType.XUnit);
}

Thanks!


Fail gracefully if region is down

Which version of kubefirst are you using?
v.2.2.2
Which cloud provider?
None specific
Which installation type?
CLI
Which distributed Git provider?
None specific
What is the issue?
If one of the cloud region is down (currently experiencing it with Civo & NYC1), the CLI will display this error
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x102816860]

goroutine 1 [running]:
github.com/kubefirst/runtime/pkg/civo.(*CivoConfiguration).createAccessCredentials(0x14000ebeac8, {0x14000ac4d80, 0x24}, {0x102df4e75, 0x4})
	github.com/kubefirst/runtime@v0.1.75/pkg/civo/objectStorage.go:129 +0x120
github.com/kubefirst/runtime/pkg/civo.(*CivoConfiguration).GetAccessCredentials(0x14000ebeac8, {0x14000ac4d80, 0x24}, {0x102df4e75, 0x4})
	github.com/kubefirst/runtime@v0.1.75/pkg/civo/objectStorage.go:67 +0x1dc
github.com/kubefirst/kubefirst/cmd/civo.createCivo(0x1400060fb00?, {0x102df5411?, 0x8?, 0x8?})
	github.com/kubefirst/kubefirst/cmd/civo/create.go:402 +0x1fac
github.com/spf13/cobra.(*Command).execute(0x1400060fb00, {0x14000116300, 0x8, 0x8})
	github.com/spf13/cobra@v1.7.0/command.go:940 +0x5c8
github.com/spf13/cobra.(*Command).ExecuteC(0x1058e0dc0)
	github.com/spf13/cobra@v1.7.0/command.go:1068 +0x35c
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.7.0/command.go:992
github.com/kubefirst/kubefirst/cmd.Execute()
	github.com/kubefirst/kubefirst/cmd/root.go:48 +0x30
main.main()
	github.com/kubefirst/kubefirst/main.go:87 +0x590

When in reality, the region is down, and unreachable or 404 as shown in the logs
2023-07-12T13:40 INF runtime/pkg/civo/(*CivoConfiguration)/objectStorage.go:101 > looking for credential: k1-state-store-kubefirst-fred-2obfo1
2023-07-12T13:40 INF runtime/pkg/civo/(*CivoConfiguration)/objectStorage.go:104 > Error: Unknown error response - status: 404 Not Found, code: 404, reason: {"code":"database_region_not_found","reason":"Failed to find the region within the internal database"}

2023-07-12T13:40 INF runtime/pkg/civo/(*CivoConfiguration)/objectStorage.go:62 > Error: Unknown error response - status: 404 Not Found, code: 404, reason: {"code":"database_region_not_found","reason":"Failed to find the region within the internal database"}

2023-07-12T13:40 INF runtime/pkg/civo/(*CivoConfiguration)/objectStorage.go:66 > credential name: k1-state-store-kubefirst-fred-2obfo1 not found, creating
2023-07-12T13:40 INF runtime/pkg/civo/(*CivoConfiguration)/objectStorage.go:127 > Error: Unknown error response - status: 404 Not Found, code: 404, reason: {"code":"database_region_not_found","reason":"Failed to find the region within the internal database"}

The CLI should fail gracefully with a message mentioning the region is down at the moment, and let them know they should use the --cloud-region flag to choose another than the default one.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

It shouldn't be an issue anymore as the revamp of the CLI will use the API instead of its own code.
If the problem persist with v2.3+, please add a comment or create a new issue.


fix bug for export async ignore exception

What is the purpose of the change
fix bug for export async ignore exception
when I use wrong threadPool config that does't exist and use async export model ,such as dubbo-thread-pool as below code:

        ProtocolConfig protocol = new ProtocolConfig();
        protocol.setName("dubbo");
        protocol.setPort(12345);
        protocol.setThreads(16);
        protocol.setIothreads(2);
        protocol.setKeepAlive(true);
        //this is bad thread-pool
        protocol.setThreadpool("dubbo-thread-pool");
        protocol.setAccepts(4);
        protocolConfigList.add(protocol);

    DubboBootstrap dubboBootstrap = DubboBootstrap.getInstance();
    dubboBootstrap.service(service);
    dubboBootstrap.configCenter(configCenterConfig);
    dubboBootstrap.registry(registryConfig);
    //export by async !!!
    dubboBootstrap.exportAsync();
    dubboBootstrap.start();

in this situation , export operation will run in executorRepository , it will be fail if export catch exception because the workThread in threadPool will be die if catch error !!!
Brief changelog
fix bug for export async ignore exception
Verifying this change

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #7507 (b38c86f) into master (cc421ba) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #7507      +/-   ##
============================================
- Coverage     59.18%   59.14%   -0.04%
+ Complexity      502      501       -1
============================================
  Files          1079     1079
  Lines         43344    43415      +71
  Branches       6324     6335      +11
============================================
+ Hits          25651    25678      +27
- Misses        14850    14897      +47
+ Partials       2843     2840       -3

Impacted Files
Coverage Δ
Complexity Δ

.../apache/dubbo/config/bootstrap/DubboBootstrap.java
42.10% <0.00%> (-0.21%)
0.00 <0.00> (ø)

...ntext/event/AwaitingNonWebApplicationListener.java
48.48% <0.00%> (-24.25%)
0.00% <0.00%> (ø%)

...in/java/org/apache/dubbo/common/utils/JVMUtil.java
81.13% <0.00%> (-11.33%)
0.00% <0.00%> (ø%)

...che/dubbo/remoting/transport/mina/MinaChannel.java
35.52% <0.00%> (-10.53%)
14.00% <0.00%> (-1.00%)

...ng/transport/dispatcher/all/AllChannelHandler.java
82.75% <0.00%> (-6.90%)
0.00% <0.00%> (ø%)

...dubbo/remoting/exchange/support/DefaultFuture.java
80.34% <0.00%> (-6.84%)
0.00% <0.00%> (ø%)

.../dubbo/remoting/transport/netty4/NettyChannel.java
59.40% <0.00%> (-1.99%)
0.00% <0.00%> (ø%)

...e/dubbo/remoting/transport/netty4/NettyClient.java
69.56% <0.00%> (-1.45%)
0.00% <0.00%> (ø%)

...vent/listener/ServiceInstancesChangedListener.java
0.00% <0.00%> (ø)
0.00% <0.00%> (ø%)

...e/dubbo/remoting/exchange/codec/ExchangeCodec.java
80.16% <0.00%> (+0.33%)
0.00% <0.00%> (ø%)

... and 6 more

Continue to review full report at Codecov.

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

大佬英明!!!


Doggy's a landing page

please merge this .this is my first project of html ,css and bootsrap based.

please merge this page


Thompson /w commando perk

Incendiary Thompson gets bonus ammo /w Commando perk, but doesn't get damage bonus or damage counted towards perk.
20 round Thompson is my favourite weapon in the game and I don't see a reason to remove it from Commando. I like the idea to make it Incendiary, but it would make sense to keep the cross perk option for Commando.
Especially with tactical reload, the weapon feels so fun to play with commando.

Also, I'm loving the mod! And the KF1/KF2 review episodes on YT are excellent aswell!

You can give Commando bonuses to Inc. Thompson by adding this line to ScrnBalanceSrv.ini:
PerkedWeapons=3:ScrnBalanceSrv.ScrnThompsonInc

Adding the line worked :)
But how could I also edit e.g. ScrnThompsonInc.uc? Does it need compiling?
Also can I get working Discord link pls?
And have you thought about working with Tripwire to get ScrNMod released in original KF1 as like "KF Remastered" or something?

The best way to modify the weapon is to create a separate package and extend the weapon class there. For example:
// MyPackage/Classes/MyScrnThompson.uc
class MyThompson extends ScrnThompsonInc;

// put your modifications here

Then do the same with the pickup class:
class MyThompsonPickup extends ScrnThompsonIncPickup;

Compile the package, then add your weapon to ServerPerks.ini:
TraderInventory=3:MyPackage.MyThompsonPickup


chore: bdd-sbrp-import-1585523838 to 0.0.1

chore: Promote bdd-sbrp-import-1585523838 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Doesn't work with Windows

I tried to use this in Windows but it fails. I traced the issue to the use of single quotes. Windows doesn't recognize 'just one parm' as a single parameter, but as 3 parameters which with bash/unix would have been typed 'just one parm'. In the example of running "npm run watch" I got an error telling me that 'npm isn't an executable (notice the leading quote). I added some logging and I got this:
C:\usr\local\angular\projects\npm-scripts-example-master>parallelshell 'where npm'
[ 'node',
  'C:\\usr\\local\\angular\\projects\\npm-scripts-example-master\\node_modules\\parallelshell\\index.js',
  '\'where',
  'npm\'' ]

C:\usr\local\angular\projects\npm-scripts-example-master>parallelshell "where npm"
[ 'node',
  'C:\\usr\\local\\angular\\projects\\npm-scripts-example-master\\node_modules\\parallelshell\\index.js',
  'where npm' ]

As you can see, single quotes and double quotes aren't considered the same in windoze. :-( (You can thank Billy for that.)
So, I was able to get a bit further along by doing something like this:
"watch": "parallelshell \"npm run watch:test -s\" \"npm run watch:build -s\"",

I also noticed that this was frowned upon by npm:
"watch": 'parallelshell "npm run watch:test -s" "npm run watch:build -s"',

It wants the "watch" parameter to start with a double quote and gives this error:
npm ERR! Failed to parse json
npm ERR! Unexpected token '
I also tried with Cygwin and bash provided in Git, without success, and this problem is not just limited to 'parallelshell' but even this fails:
"build": "npm run build:scripts -s && npm run build:styles -s && npm run build:markup -s",

So, in the end even if this seems like a good idea, it didn't work but it would be nice if it did.

Hey, from my latest experience it works just fine if you replace all ' (single quote) with \" (escaped double quote)…

Christian,
I hope this will help as I am not trying to complain.
I installed your example from the downloaded ZIP. Then I ran "npm
install" to get all the dependencies. The "npm run watch" still fails
but some commands work, like the "npm run build". I tried this on 2
machines as the corporate computer is sometimes tempermental but my
personal computer isn't. I got the same results.
Here is some of the run samples: (I highlighted the command lines but
the email program highlighted what looks like email addresses)
C:\usr\local\angular\projects\npm-scripts-example-master>npm run watch

<EMAIL_ADDRESS>watch
C:\usr\local\angular\projects\npm-scripts-example-master
parallelshell "npm run watch:test -s" "npm run watch:build -s"

npm run watch:build -s failed with exit code 1
npm ERR<EMAIL_ADDRESS>watch: parallelshell "npm run  watch:test -s" "npm run watch:build -s"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>watch script.
npm ERR! This is most likely a problem with the npm-scripts-example
package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     parallelshell "npm run watch:test -s" "npm run watch:build
-s"
npm ERR! You can get their info via:
npm ERR!     npm owner ls npm-scripts-example
npm ERR! There is likely additional logging output above.
npm ERR! System Windows_NT 6.1.7601
npm ERR! command "C:\usr\local\nodejs\\node.exe"
"C:\usr\local\nodejs\node_modules\npm\bin\npm-cli.js" "run"
"watch"
npm ERR! cwd C:\usr\local\angular\projects\npm-scripts-example-master
npm ERR! node -v v0.10.33
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!
C:\usr\local\angular\projects\npm-scripts-example-master\npm-debug.log
npm ERR! not ok code 0
C:\usr\local\angular\projects\npm-scripts-example-master>npm run
watch:build

<EMAIL_ADDRESS>watch:build
C:\usr\local\angular\projects\npm-scripts-example-master
nodemon -q -w assets/ --ext "." --exec "npm run build"

npm ERR<EMAIL_ADDRESS>watch:build: nodemon -q -w assets/  --ext "." --exec "npm run build"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>watch:build script.
npm ERR! This is most likely a problem with the npm-scripts-example
package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     nodemon -q -w assets/ --ext "." --exec "npm run build"
npm ERR! You can get their info via:
npm ERR!     npm owner ls npm-scripts-example
npm ERR! There is likely additional logging output above.
npm ERR! System Windows_NT 6.1.7601
npm ERR! command "C:\usr\local\nodejs\\node.exe"
"C:\usr\local\nodejs\node_modules\npm\bin\npm-cli.js" "run"
"watch:build"
npm ERR! cwd C:\usr\local\angular\projects\npm-scripts-example-master
npm ERR! node -v v0.10.33
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!
C:\usr\local\angular\projects\npm-scripts-example-master\npm-debug.log
npm ERR! not ok code 0
C:\usr\local\angular\projects\npm-scripts-example-master>npm run build

<EMAIL_ADDRESS>prebuild
C:\usr\local\angular\projects\npm-scripts-example-master
npm run clean -s

<EMAIL_ADDRESS>build
C:\usr\local\angular\projects\npm-scripts-example-master
npm run build:scripts -s && npm run build:styles -s && npm run
build:markup -s

generated dist\main.css.map
compiled dist\main.css
rendered dist\index.html
C:\usr\local\angular\projects\npm-scripts-example-master>
Although I would very much like to be able to use npm to configure the
builds, it seems that there still is some reason why other build tools
were developed.
Thanks.
Gerard Perreault
------ Original Message ------
From: "Christian Seel"<EMAIL_ADDRESS>To: "keithamus/npm-scripts-example"
<EMAIL_ADDRESS>Cc: "cpu999"<EMAIL_ADDRESS>Sent: 1/15/2015 1:23:40 PM
Subject: Re: [npm-scripts-example] Doesn't work with Windows (#5)

Hey, from my latest experience it works just fine if you replace all '
(single quote) with " (escaped double quote)…
—
Reply to this email directly or view it on GitHub.

@cpu999 try downloading the example again, it should work now.

Thanks @christianseel for suggesting the fix :smile:

Actually, it still didn't work. I replaced all the single quotes by " and now the "npm run build" works. However, the "watch:build" still fails. I suspect it may have to do with the use of "nodemon", the "--exec" parameter. If I recall correctly, "exec" requires an executable and "npm" is not one, it is a script.
This is what I had to change it to:
{
"name": "npm-scripts-example",
"version": "1.0.0",
"description": "An example of how to use npm scripts over a build tool like Grunt or Gulp",
"main": "index.js",
"author": "Keith Cirkel<EMAIL_ADDRESS>(http://keithcirkel.co.uk/)",
"license": "MIT",
"devDependencies": {
"browserify": "^6.3.2",
"hashmark": "^2.0.0",
"http-server": "^0.7.3",
"jade": "^1.7.0",
"jshint": "^2.5.10",
"karma": "^0.12.28",
"karma-browserify": "^1.0.0",
"karma-cli": "^0.0.4",
"karma-mocha": "^0.1.10",
"karma-phantomjs-launcher": "^0.1.4",
"live-reload": "^0.2.0",
"minifyify": "^4.4.0",
"mocha": "^2.0.1",
"nodemon": "^1.2.1",
"opener": "^1.4.0",
"parallelshell": "^1.0.0",
"rimraf": "^2.2.8",
"s3-cli": "^0.11.1",
"stylus": "^0.49.3"
},
"scripts": {
"clean": "rimraf dist/*",
"prebuild": "npm run clean -s",
"build": "npm run build:scripts -s && npm run build:styles -s && npm run build:markup -s",
"build:scripts": "browserify -d assets/scripts/main.js -p [minifyify --compressPath . --map main.js.map --output dist/main.js.map] | hashmark -n dist/main.js -s -l 8 -m assets.json \"dist/{name}{hash}{ext}\"",
"build:styles": "stylus assets/styles/main.styl -m -o dist/ && hashmark -s -l 8 -m assets.json dist/main.css \"dist/{name}{hash}{ext}\"",
"build:markup": "jade assets/markup/index.jade --obj assets.json -o dist",

"test": "karma start --singleRun",

"watch": "parallelshell \"npm run watch:test -s\" \"npm run watch:build -s\"",
"watch:test": "karma start",
"watch:build": "nodemon -q -w assets/ --ext \".\" --exec \"cmd /c npm run build\"",

"open:prod": "opener http://example.com",
"open:stage": "opener http://staging.example.internal",
"open:dev": "opener http://localhost:9090",

"deploy:prod": "s3-cli sync ./dist/ s3://example-com/prod-site/",
"deploy:stage": "s3-cli sync ./dist/ s3://example-com/stage-site/",

"serve": "http-server -p 9090 dist/",
"live-reload": "live-reload --port 9091 dist/",

"dev": "npm run open:dev -s & parallelshell \"npm run live-reload -s\" \"npm run serve -s\" \"npm run watch -s\""

}
}
Notice that on the --exec I added "cmd /c" and now it works. However, that makes the build not compatible in both Unix and Windows. I'll let you know if I figure out something.

Christian,
I tried it. It wasn't totally fixed. I posted this:
Actually, it still didn't work. I replaced all the single quotes by "
and now the "npm run build" works. However, the "watch:build" still
fails. I suspect it may have to do with the use of "nodemon", the
"--exec" parameter. If I recall correctly, "exec" requires an executable
and "npm" is not one, it is a script.
This is what I had to change it to:
{ "name": "npm-scripts-example", "version": "1.0.0", "description": "An
example of how to use npm scripts over a build tool like Grunt or Gulp",
"main": "index.js", "author": "Keith Cirkel<EMAIL_ADDRESS>(http://keithcirkel.co.uk/)", "license": "MIT", "devDependencies": {
"browserify": "^6.3.2", "hashmark": "^2.0.0", "http-server": "^0.7.3",
"jade": "^1.7.0", "jshint": "^2.5.10", "karma": "^0.12.28",
"karma-browserify": "^1.0.0", "karma-cli": "^0.0.4", "karma-mocha":
"^0.1.10", "karma-phantomjs-launcher": "^0.1.4", "live-reload":
"^0.2.0", "minifyify": "^4.4.0", "mocha": "^2.0.1", "nodemon": "^1.2.1",
"opener": "^1.4.0", "parallelshell": "^1.0.0", "rimraf": "^2.2.8",
"s3-cli": "^0.11.1", "stylus": "^0.49.3" }, "scripts": { "clean":
"rimraf dist/*", "prebuild": "npm run clean -s", "build": "npm run
build:scripts -s && npm run build:styles -s && npm run build:markup -s",
"build:scripts": "browserify -d assets/scripts/main.js -p [minifyify
--compressPath . --map main.js.map --output dist/main.js.map] | hashmark
-n dist/main.js -s -l 8 -m assets.json "dist/{name}{hash}{ext}"",
"build:styles": "stylus assets/styles/main.styl -m -o dist/ && hashmark
-s -l 8 -m assets.json dist/main.css "dist/{name}{hash}{ext}"",
"build:markup": "jade assets/markup/index.jade --obj assets.json -o
dist", "test": "karma start --singleRun", "watch": "parallelshell "npm
run watch:test -s" "npm run watch:build -s"", "watch:test": "karma
start", "watch:build": "nodemon -q -w assets/ --ext "." --exec "cmd
/c npm run build"", "open:prod": "opener http://example.com",
"open:stage": "opener http://staging.example.internal", "open:dev":
"opener http://localhost:9090", "deploy:prod": "s3-cli sync ./dist/
s3://example-com/prod-site/", "deploy:stage": "s3-cli sync ./dist/
s3://example-com/stage-site/", "serve": "http-server -p 9090 dist/",
"live-reload": "live-reload --port 9091 dist/", "dev": "npm run open:dev
-s & parallelshell "npm run live-reload -s" "npm run serve -s" "npm
run watch -s"" } }
Notice that on the --exec I added "cmd /c" and now it works. However,
that makes the build not compatible in both Unix and Windows. I'll let
you know if I figure out something.
Gerard Perreault
------ Original Message ------
From: "Keith Cirkel"<EMAIL_ADDRESS>To: "keithamus/npm-scripts-example"
<EMAIL_ADDRESS>Cc: "cpu999"<EMAIL_ADDRESS>Sent: 1/15/2015 5:25:20 PM
Subject: Re: [npm-scripts-example] Doesn't work with Windows (#5)

@cpu999 try downloading the example again, it should work now.
—
Reply to this email directly or view it on GitHub.

Sorry, I didn't realize that an email reply got posted (and messed up).

We had the same issue with nodemon under windows. It seems like you can't execute npn run from nodemon. Thats why we replaced it with watch:
like "watch:build": "watch  \"npm run build\" assets/sources"

Nodemon has an issue here: remy/nodemon#389. Anyone experiencing this issue with nodemon should file it there - it looks like it may be coming toward a fix, but I'm sure your experiences would help.

thanks guys! i had the same issue and now it works!


"SIGSEGV: segmentation failure" when initialising connection

Hi, all!
I hope I'm missing something here... I've copied the example from the examples/ directory to try out using this library, but upon running the application, I get the following error:
$> CGO_ENABLED=1 GOOS=linux go run .
SIGSEGV: segmentation violation
PC=0x4f387c m=0 sigcode=1
signal arrived during cgo execution

Any help will be greatly appreciated! :)

This doesn't seem to be an issue anymore, all working as expected.


Can't build realm-swift using Xcode 8 beta 3

Goals
Build Realm.framework and RealmSwift.framework for Swift 2.3 using xcode-build with Xcode 8 beta 3
Expected Results
Compile with no errors
Actual Results
Compilation fails with the following:
=== BUILD TARGET Realm OF PROJECT Realm WITH CONFIGURATION Release ===

Check dependencies
No signing certificate 'iOS Development' found:  No 'iOS Development' signing certificate with a private key was found.

** BUILD FAILED **

Steps to Reproduce
Run ./build.sh build using the latest version of code from master whilst using Xcode 8 beta 3.
Version of Realm and Tooling
ProductName:	Mac OS X
ProductVersion:	10.11.5
BuildVersion:	15F34

/Applications/Xcode 8.0.app/Contents/Developer
Xcode 8.0
Build version 8S174q

/usr/local/bin/pod
1.0.0.rc.1
(not in use here)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin15)

/usr/bin/git
git version 2.8.4 (Apple Git-73)

The Xcode 8 betas introduced and subsequently removed the need to codesign iOS frameworks.
You'll need to change the team used for code signing for each target via Xcode's target editor prior to running build.sh build.
The changes and conversations in #3801 should help shed more light on this for you.

I wasn't able to get the build to work after manually setting code signing (issues with Swift compatibility) but the fix in pull request #3896 has resolved everything. Thanks for the quick support.


Cannot use custom method in $Parameters collection

I'm trying to use a custom method in $parameters but for some reason it is not getting converted.
$Parameters[$name: $CalculatedType][, ]

My method signature is string CalculatedType(Type m). Is this the wrong signature?

Since the custom method is called from within a $Parameters scope the method signature must be string CalculatedType(Parameter p) (in the method you can use p.Type to get the type)

That worked! Thanks!

@frhagn I have created SO tag http://stackoverflow.com/questions/tagged/typewriter


fix(footer): Adjust chatbot icon alignment

Issue Title:  fix Bug(footer): Adjust chatbot icon alignment
resolve issue: #253
Type of change ☑
What sort of change have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

How Has This Been Tested? ⚙
The changes are tested locally by running the application and verifying the alignment of the chatbot and Gmail icons in the footer. Both icons are now visible and properly spaced out, resolving the overlap issue. Testing is also performed across different screen sizes and resolutions to ensure the fix works as expected.
Screenshots-
Before:

After:

@singodiyashubham87 Please have a look at my PR and merge it.

@DebapriyaJha Neither you have to change the package.json && package-lock.json files nor you are advised to write custom CSS, Use TailwindCSS classes for styling.

okay i will make the changes

@DebapriyaJha Any updates?

Yes I am working on it will complete and try to submit my PR in 2 days


backintime: 1.3.2 -> 1.3.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/bit-team/backintime/releases.
meta.description for backintime is: Simple backup tool for Linux
meta.homepage for backintime is: https://github.com/bit-team/backintime
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.3 with grep in /nix/store/8b4nfpig436yz2nxissmr5m0wwp7dpv9-backintime-qt-1.3.3
found 1.3.3 in filename of file in /nix/store/8b4nfpig436yz2nxissmr5m0wwp7dpv9-backintime-qt-1.3.3

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
backintime
backintime-common
backintime-qt

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8b4nfpig436yz2nxissmr5m0wwp7dpv9-backintime-qt-1.3.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8b4nfpig436yz2nxissmr5m0wwp7dpv9-backintime-qt-1.3.3
ls -la /nix/store/8b4nfpig436yz2nxissmr5m0wwp7dpv9-backintime-qt-1.3.3/bin

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

2 packages built:

backintime
backintime-common

Maintainer pings

@ofborg build backintime backintime-qt backintime-common


Doesn't work with Teams created sharepoints sites?

I wanted to try this tool to move large files from my computer to a Microsoft Teams I am part of.
In Teams i select "Open in Sharepoint" and then copy the URL to the migration tool.
But whatever I try I get "Sorry, you don't have access to this site. Please enter a different Sharepoint online sit." (yes, it cuts of there). . .
I can upload files through Teams just fine so I DO have access!
Can you please fix this in the application? Or at least let the documentation reflect that it doesn't work with Microsoft Teams created Sharepoint-sites?

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

ID: 193d7cc4-4270-f69e-262b-6384a273f835
Version Independent ID: 6625a0f7-e4d0-2352-d78a-0a82273b1f07
Content: Introducing the SharePoint Migration Tool
Content Source: migration/introducing-the-sharepoint-migration-tool.md
Product: sharepoint-server-itpro
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

If you are using the exact URL you are sent to after you "Open in SharePoint", that might not work. Try removing everything after and including "/Shared%20Documents/..." in the URL - just provide the URL to the SharePoint site - the Migration Tool will then list the available document libraries. Be sure to upload into one of the Teams' channels subfolder (i.e., "General"). Also, make sure you don't migrate or create new permissions during the migration - you don't want to break the existing permissions on the document library for the Team.

Is that tool supports migration from SP 2010 to SharePoint Online?

@gurramsubrahmanyam no it does not.


inputs outputs blog post

This is a rewrite of the first version which was based on the video text. It has been rewritten to follow the example more closely instead of the video,

Should we close this, or maybe move to pulumi-hugo?

not published


Adding links to http://crafting-guide.com

I've been working on a project to make it easier to deal with complex mods (especially tech mods), and I recently added support for EnderIO. The gist of the site is that you tell it what item(s) you want to make, and it will figure out all the raw ingredients you need along with step-by-step instructions. For example, here's how you make a Dimensional Transceiver.
The maintainers of mods I've added (e.g., BuildCraft / Applied Energistics) have found it useful to include links to Crafting Guide in their official documentation, and I was hoping you would feel the same.  I'd be happy to make the changes to the wiki myself if you like the idea.
Thanks!

Crafting Guide looks amazing. I really hope @CrazyPants & @tterrag1098 like your idea too, since they are the main coders of Ender IO. I really like the idea myself though!

@tterrag1098 — If you don't mind having them, I'd be happy to add the links myself.  Just let me know, and I'll get to work!

I don't mind, but I'll need to add your wiki account manually.
On Mon, Feb 9, 2015 at 3:37 PM, Andrew Miner<EMAIL_ADDRESS>wrote:

@tterrag1098 https://github.com/tterrag1098 — If you don't mind having
them, I'd be happy to add the links myself. Just let me know, and I'll get
to work!
—
Reply to this email directly or view it on GitHub
https://github.com/CrazyPants/EnderIO/issues/1833#issuecomment-73585078.

Great, thanks!  Let me know if there's anything else I need to do.

@tterrag1098 Any update on that new account?  :-)

@tterrag1098 Sorry to be a bother, but I'm kinda stuck without that account...  :-(

Agh, sorry I keep forgetting. You should now have an email on your @mac.com email (the one listed on your github account).

Awesome, thanks!

Should be all set!  I added a link for each item, and fixed up a few issues I ran across while doing so.  Thanks again!


Update googleAuthWrapper.js

If clientSecrets filename had the .json extension, it was invalidating the final path because it's appended later. To avoid this, if ".json" is found at the end of clientSecrets, it's dropped early then appended as before. This way it doesn't matter if ".json" is provided initially.

Seems reasonable to me, when I get a chance I'll validate with existing dependencies.


Pythonic bitinformation

This is work-in-progress for a pythonic way to retrieve the mutual information content.

This branch is now in a state where the basic functionality of get_bitinformation is ported to python and works in first tests reasonably well with Dask. Due to #98 and other issues with the wrapping of Julia code, large datasets could not be processed straight forward. Ideally, Julia and python would work better together in the future, but currently this seems to be a reasonable alternative.
However, the following features are still missing in the python implementation and should be added in a next step:

enable masked values
enable set_zero_insignificant (currently is set to False for the python implementation)
any additional arguments to xb.get_bitinformation

I second that the duplication is not ideal and I would like to avoid it, but currently it seems (at least to me) the easiest way to allow for larger datasets.

julia stays the default.

julia stays the default.

Yes!!


odin: fix not including base library

Description of changes
The merge of #285742 broke the Odin package
This PR rectifies this, fixing #286441
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

This fix has been tested against hellope.odin found on the odin website, before which it didn't compile

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1437


Vanilla Recipes

Hi, your mod adds additional recipes for Vanilla Recipes like Stairs or Trapdoors etc, would it be possible to add a config to disable it?

Absolutely. Will continue to work on this.

Alright I made the serializer and now I just have to make 200+ recipes work with it lol

I actually removed those recipes as I didn't like them that much to begin with. If the outcry is large I'll readd them with config settings


User limit can be bypassed

Everyone has Move Members permission in their temporary channels to allow users to kick others from their channel. However, this permission also gives the channel owner an ability to move users to channels that are visible to the owner and joinable by the one who is going to be moved, ignoring the channel's user limit (if set).
This way any user can move themselves to a channel that is already full.
Looks like this works only on mobile version of Discord app. Tested on Android, not sure about iOS.
Client info
App: 198.13 (198113) betaRelease
Manifest: N/A
Build Override: N/A
Device: RMX2040 (RMX2040) OS 30

By the way, you cannot reconnect from the desktop or web versions of Discord to a channel you are already connected to from mobile device, if the channel is full.

Although this issue has now been fixed for Doorman's temporary channels, there's little that can be done to prevent users from joining full channels not related to Doorman.
Of course, Doorman could manage those channels as well, but it should not, because no one expects the bot to touch channels other than temporary and hubs. It's best to wait until Discord fixes it.


Update Jenkinsfile

Codecov Report

Merging #3054 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3054   +/-   ##
=======================================
  Coverage   63.15%   63.15%
=======================================
  Files         484      484
  Lines       27628    27628
  Branches     4281     4281
=======================================
  Hits        17449    17449
  Misses       9027     9027
  Partials     1152     1152

Continue to review full report at Codecov.

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

I'll merge it without review approval to unlock the CI work. Please let me know if you have any concerns.


Update ETNavBarTransparent.swift

Fix for "Redundant conformance of 'UINavigationController' to protocol 'UINavigationBarDelegate'" if target project is already conforming to UINavigationBarDelegate

Please accept my pull request to fix the issue


update: maintainers list and evolution resources

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind documentation

/kind feature

/kind incubating

/kind officialsupport

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area governance

/area proposals

/area utils

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
The CI job is not fully working, so I'm manually creating this PR. Hoping to fix the automation soon.
/cc @jasondellaluce

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

Needs approval from an approver in each of these files:

OWNERS [leogr]

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

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

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

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


project:adddependencies doesn't detect a subtab object dependency

Your environment
OS : Ventura 13.0.1 (22A400)
Node.js version : v18.11.0
suitecloud-cli version : 1.6.2
Terminal/CMD tool : Terminal/iTerm
Describe the bug
Command suitecloud project:adddependencies does not discover an object dependency. This is a transactionbodycustomfield object, and within the subtab element, there is a [scriptid=XXX] that does not get picked up. This also isn't picked up by suitecloud project:validate. Once suitecloud project:deploy --validate is run, the error appears.
To Reproduce
Steps to reproduce the behavior:

Run suitecloud project:adddependencies
Run suitecloud project:validate
Run suitecloud project:deploy --validate

Actual Behavior
suitecloud project:adddependencies &  suitecloud project:validate miss a necessary dependency and suitecloud project:deploy fails.
Expected Behavior
It would be ideal if the dependency file were added to the manifest before deploy.
Additional context
none at this time.

Hi @magic-madrigal
We are reviewing past issues and I have not been able to replicate this issue.
Could you please try it with last Suitecloud CLI version?
If the error persists Could you please share the files that produce this error?
Thanks.


Support u and r prefixes (unicode and raw strings)

Based heavily on #4 and #5 (f and b prefixes), adds support for the r and u prefixes.
Also bumps version to v0.5

Tested on a large code base here:
https://github.com/biopython/biopython/pull/2192
Almost all the manual changes afterwards were for #12.


Raw mode is not supported on the current process.stdin

🐛 Bug report
Describe the bug
I have a script that migrates env-vars on deploy, like this:
echo $MY_SECRET | npx -y wrangler@alpha --env main secret put MY_SECRET

I get this error, now:
 ERROR  Raw mode is not supported on the current process.stdin, which Ink uses as input stream by default.
        Read about how to prevent this error on https://github.com/vadimdemedes/ink/#israwmodesupported

Expected behavior
There should be a way to set secrets, in scripts (without user-input.)
Environment and versions
Fill out the following information about your environment.

operating system: OSX 12.2 (Monterey)
output of wrangler -v: 0.0.0-046b17d
output of node -v: v17.6.0

This was fixed in 0.0.20 with #559. Could you try the latest version and see if it works for you?

Seems to be working with wrangler@alpha. Thanks!


Fix bad host.

LINK2 is OK for external links, right?

LINK2 is OK for external links, right?

Yep.

Like this?

Auto-merge toggled on

WEB is even better

Why is it better? Ref #4411.


dynamic routes with loadChildren

testLazyModule.zip

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)
Ubuntu 16.04

Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:
angular-cli: 1.0.0-beta.26
node: 7.4.0
os: linux x64
@angular/common: 2.4.5
@angular/compiler: 2.4.5
@angular/core: 2.4.5
@angular/forms: 2.4.5
@angular/http: 2.4.5
@angular/platform-browser: 2.4.5
@angular/platform-browser-dynamic: 2.4.5
@angular/router: 3.4.5
@angular/compiler-cli: 2.4.5

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.
This was created using ng new

The log given by the failure.

Normally this include a stack trace and some more information.
EXCEPTION: Uncaught (in promise): Error: Cannot find module 'app/modules/lazy/lazy.module'.

Mention any other details that might be useful.
When doing loadChildren from the route, it works fine but dynamically modifying the route with loadChildren isn't working
this works if I add the lazy path app.routing.ts
`
path: 'app',
component: HomeComponent,

children: [

  { path: '', component: HomeOutletComponent},

  { path: 'home', component: HomeOutletComponent }

  ,{ path: "lazy", loadChildren: "app/modules/lazy/lazy.module#LazyModule"}

]

This doesn't worklet routerConfig = this.router.config;
if (!this.loaded) {

  let r: Route = { path: "lazy", loadChildren: "app/modules/lazy/lazy.module#LazyModule"};

  routerConfig[1].children.push(r);

  this.router.resetConfig(routerConfig);

  this.loaded = true;}`

Thanks! We'll be in touch soon.

Dupe of https://github.com/angular/angular-cli/issues/4234


Vane strips lore already existing on items on anvil combine / rename.

With Vane enchants enabled:

With Vane Enchants Disabled:

Root cause: https://github.com/oddlama/vane/blob/main/vane-enchantments/src/main/java/org/oddlama/vane/enchantments/Enchantments.java#L151
Vane currently clears all lore, every time it updates, what it should be doing, is instead using the existing lore, and only removing lore that it's added previously.
We need some way of marking this lore, the easy way would be to use the translatable components, since they are (presumably) namespaced under vane, we can see them pretty easy.
However this directly clashes with the ability to server-side translate the chat components.
So instead I propose we insert some form of invisible marker, so we can see what lore belongs to vane-enchant, remove all that match, and then update as normal, appending the vane enchants to the list again.
That would mutate the item stacks on every enchanting operation, but it would be a LOT easier then scanning and updating individual enchants + levels.

[memory log from discord discussion]:
Best would be to tag the lines added by vane with a hidden text component, to later update them if necessary but never to delete existing content.

Another thing: Existing books can then no longer be used to craft stuff because of NBT mismatch. Maybe it's time to address this NBT matching issue somehow :/

Another thing: Existing books can then no longer be used to craft stuff because of NBT mismatch. Maybe it's time to address this NBT matching issue somehow :/

Another thing: Existing books can then no longer be used to craft stuff because of NBT mismatch. Maybe it's time to address this NBT matching issue somehow :/

I talked to the Recipe Manager dev, and short of using Recipe Manager 2 as a dependency, the path is using Material Choices for the recipes, then validating it afterwards, either in the various recipe result events, or inventory events.
I havn't tested it, but he suspects that they still render the example item correctly.

I'm 99% certain they wont render correctly in the preview on the client. But I guess it's the only way to move forward for us (at least for now)

I just had a new idea to mostly prevent breaking backwards compatibility. If a lore entry is already just a translatable component referencing vane*:something, it can always be considered part of vane, no need for a prefix to keep track of it. And that should be the case for 95% of the installs. I'll use your nice prefixing logic when client side translations are disabled, but add those simple translatable components to the list of vane-owned components.
It might cause duplicated texts on those other 5%, but we cannot realistically do anything about that. After all, vane is supposed to be used with client side translations. And if some other plugin stringifies the lore, it's not our problem. It isn't anything we can prevent and therefore I will not create any weird hacks to support it.

[50% done]: Crafting will still be an issue going forward. I can see two paths going forward:
A) We introduce a lightweight item watcher that converts items when they are encountered in some events (maybe item pickup/move/...). Can be done cleanly, and has the added benefit that we can use it for more things later, for example if we want to make breaking changes to custom items.
B) Make the crafting recipes more general (i.e. Material only) and reject invalid materials in an event. This will cause items to be wrongly rendered in recipes on the client recipe book though.
Realistically I think we need to go for A.

The converter that will be introduced next breaking release can also take care of updating items for these situations.

B) Make the crafting recipes more general (i.e. Material only) and reject invalid materials in an event. This will cause items to be wrongly rendered in recipes on the client recipe book though.

I wonder if it's possible to do the opposite, make the items NBT specific, and use the events to make them more general.
They wouldn't shift click nicely from the recipe books though.
An alternative I thought of as I was writing this, is if that fails, register them twice?
Gives a recipe that 'shows up right' and a different one that 'works right' but looks wrong.
Neither situation would be good, but I wonder if that's a better compromise.
VanillaTweaks / datapacks tend to use a green recipe book thing as a placeholder, and detect that being given to the player, we don't need to do that luckily, but it's also an option potentially as some players would be used to seeing these placeholders.

Sounds like you'll always be in some kind of workaround situation. It would just be so good if mojang added support for this natively :/


fix: Update intermediate CA and PKI role when common_name is changed.

Description
Fixes: #367
When the common name is changed a new CSR will be generated to get a new intermediate CA that will replace the old one and the pki role will be updated too.
Note to reviewer:
This pr sets allow_bare_domains to true in the PKI role config
"allow_bare_domains": True,
This will allow the requirer to request certificates using the same common name configured in vault as the CA.
For example if the common name configured in Vault is example.com the requirer will be able to request a certificate for example.com. Prior to this change the requirer would be allowed to request certificates for subdomains only, for example sub.example.com.
Please let me know in case of any objections to this change.
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have added tests that validate the behaviour of the software
[x] I validated that new and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Turning PR to draft until integration test issues are figured out.


saner defaults for compilation faces?

Seems to be that all of the special faces for rustic-compilation-mode inherit from default, meaning by default you get a really bland compilation buffer. They should inherit from the appropriate compilation faces instead.
So

rustic-face
compilation-face

rustic-compilation-warning
compilation-warning

rustic-compilation-info
compilation-info

rustic-compilation-error
compilation-error

rustic-compilation-column
compilation-column-number

rustic-compilation-line
compilation-line-number

S.N. I didn't even realize you could jump to line numbers/errors until I tried to.

I think the idea was to make the colors that are emitted by cargo the default. Default compile.el doesn't translate control sequences so I thought rustic should use different defaults.
Any ideas how to improve the docs ? I apprecaite any help I can get, since I currently don't even use rust.

I see. #127 seems fine then.


This library is still mantained?

Hi! Is this library still mantained?
There is so many opened issues and I didn't see any commits on this year.
Thanks!

18,219 downloads per week, and this package just killed my android app because of an issue first reported in January....


[Feature Request] Add listener loader

What do you want to request?
Like commands, allow for dynamic loading of listeners
The implementation should be doable enough and it could possibly re-use code
Do you have anything to tell us about your request?
I can implement it after I finished up with #134

closing in favor of #137


docs: improve reference description in the readme file

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


Run WandB offline

https://docs.wandb.ai/guides/technical-faq/setup#can-i-run-wandb-offline

@r-sarma if I well remember you did sth similar already... Do you have an example code snippet?

I can take this up.


useMeasure dimensions out of sync in react18

React version: 18
Due to automatic batching in react18, state updates in useMeasure's ResizeObserver may be applied after paint. This can cause visual glitches if we depend on useMeasure dimensions for visual updates.
Same issue is raised in react: https://github.com/facebook/react/issues/24331
They are recommending that state updates in ResizeObserver should be wrapped in flushSync from ReactDOM.
Can we do this in useMeasure or at least add optional support for flushSync in useMeasure. So that consumers can opt-in if they are using the dimensions for visual update(For eg. in useLayoutEffect).
Link to code example: https://codesandbox.io/s/amazing-montalcini-j6lz46?file=/src/App.js

This library is great and works with React 18 https://www.npmjs.com/package/react-use-measure

Not sure about this, since same issue is reported there: https://github.com/pmndrs/react-use-measure/issues/93
@streamich


SizeOverLifetimeModifier disrupts OrientModifier

Crate versions
bevy version: 0.12.1
bevy_hanabi version: 0.9 and 0.10-0-dev
Describe the bug
If you have a SizeOverLifetimeModifier with screen_space_size = true then the OrientModifier in-plane rotation is ignored. If screen_space_size = false then the OrientModifier works as required.
Expected behavior
SizeOverLifetimeModifier and OrientModifier to be independent transforms.
To Reproduce
Patch billboard.rs like so:
diff --git a/examples/billboard.rs b/examples/billboard.rs
index 6d384b4..b6d5208 100644
--- a/examples/billboard.rs
+++ b/examples/billboard.rs
@@ -160,8 +160,8 @@ fn setup(
                 rotation: Some(rotation_attr),
             })
             .render(SizeOverLifetimeModifier {
-                gradient: Gradient::constant([0.2; 2].into()),
-                screen_space_size: false,
+                gradient: Gradient::constant([500.; 2].into()),
+                screen_space_size: true,
             }),
     );

The clouds no longer have a random rotation.
Screenshots

Thanks for the bug and repro. This made it immediately clear that the render shader, when using screen space size, ignores the axis attribute. This should hopefully be a simple fix but I need to get the math right first 😅

There's actually 2 bugs. The other is that there's still a perspective divide. We can see that the particles at the back are smaller, although the code assigns a constant screen-space pixel size. We can also see that the assigned size (500 px) is quite wrong, the particles are much smaller than that.


[ci][build] fix vllm-flash-attn

I find that it actually works for ci, because ci uses python setup.py to build the wheel.
what's wrong is when we use pip install -e . , which will miss the python files.
anyway, after this pr, pip install -e . should work too.
FYI @ProExpertProg

Man I was wondering why it kept failing build on my dev environment and I thought it's something wrong with my env 🤦
Thanks for fixing!

Again, thank you @youkaichao 🙏

@ywang96 🤝


Add the "Light Bulb Generator" app

This PR adds the "Light Bulb Generator" application:
To generate rapidly a visible glowing light source with its light beam.

This application is generating an entity that is running a client entity script to generate the light and material as local entities.
This script (lightBulb.js) will be located in this application repository.
This means that everywhere that such a light build will be used, every single visitor will run it. The script will be then read from the community-app web server at least one time to be kept in cache.
This is the first problem. Are we ok to support this load? (Maybe at short or medium term)
The second problem is that it will induce a dependency to the community-app web server for the content that are using this.
Is this ok ? I'm not sure it is.

I'd say that such dependency is perfectly fine. I also tested the script right now and it works like it should

:

If I remember correctly how this works, you need to regenerate that metdata.js. After that I would merge it.


SPI is not generic over word size

I am working on a driver for AD9833 and co. waveform generators / DDSs.
This device requires SPI transfers of 16-bit words but the current SPI mock implementation is hard-coded to use u8.
I tried to make spi::Mock generic over the word size but then the default implementations of embedded_hal::blocking::spi::{Write, Transfer} from the embedded_hal::spi::{write, transfer} modules conflict with the implementations in this module.
Here is my attempt, which results in conflicts: diff, branch
Any ideas?

Nevermind, the hardware seems to accept sending [u8; 2] just fine.


Documentation left pane view cuts title name #39621

resolved issue #39621

Sorry, but I think this is the wrong fix.
Looking at the deploy-preview-40275, there are few pages I spotted where the overall website pages look broken.
Eg: https://deploy-preview-40275--kubernetes-io-main-staging.netlify.app/docs/setup/production-environment/tools/kubeadm/install-kubeadm/
Attaching screenshot below: Top content
Bottom content

May I ask what screen size you are on?

May I ask what screen size you are on?

@KhushPatibandha  I'm on native default MacOS resolution of 1728 x 1117. I have update the previous comment's screenshot. The current page without the PR changes looks fine without the issue highlighted in the screenshot.

Sure I'll look into it today and get back to you asap.

I still am stuck on this problem and can't really figure out what's going wrong but I've thought of another approach that I can try

Now the problem here is that the the sidebar is resizable but the main documentation is not, so basically the sidebar is resizing but the main documentation is not shrinking. can you help me with that ?

I believe you will have to introduce some jQuery component to add a vertical splitter or things like that.

@dipesh-rawat How about this ?

@dipesh-rawat How about this ?

IF this look ok they can you please help me locate the css files for this

@KhushPatibandha You may want to check #40369 and see if you can continue working towards that direction. I'm too occupied to fine tune the css look and feel.

I'll surely have a look.

Hello @KhushPatibandha , gentle reminder! Are you still working on this? If not, it is completely okay to say so since we understand that you may have bandwidth issues & can't continue working.
In any case, please let us know by 15th August, 2023. If there is no response from your side, we shall be closing off the PR.
/remove-lifecycle stale

Closing the PR off per above comments.
/close


Blank main window

When attempting to run FlightCore 2.9.3 via its appimage or manually compiled release binary on Ubuntu, The main window starts but renders nothing.
I even went as so far as to compile the application from source in an attempt to resolve the issue, alas the problem persists.

$ src-tauri/target/release/flight-core
libEGL warning: DRI3: Screen seems not DRI3 capable
libEGL warning: DRI2: failed to authenticate
 INFO  app::github::release_notes > Checking GitHub API
 INFO  app::github::release_notes > Fetching releases notes from GitHub API
 INFO  app                        > Titanfall2.exe exists in path? true
 INFO  app::northstar             > /mnt/86905182-2968-4fc9-8e44-207e0a1e5b08/steam/steamapps/common/Titanfall2
 INFO  app::github::release_notes > Done checking GitHub API
 INFO  app::util                  > total_player_count: 183
 INFO  app::util                  > server_count:       166
^C

~/Downloads/titanfall/FlightCore on  main ⌚ 20:56:19
$ git rev-parse HEAD
4d61ffe0439dd1e145ec5ea992ea48978034bd78

Host information:
OS: Ubuntu 22.04.3 LTS x86_64
Kernel: 5.15.0-83-generic
GPU: NVIDIA GeForce GTX 1050 Ti Mobile

Caused by upstream bug: https://github.com/tauri-apps/tauri/issues/5143#issuecomment-1311815517
setting WEBKIT_DISABLE_COMPOSITING_MODE=1 allows the main window to render.

Oh wow, good catch. Thanks for the detailed report and corresponding PR <3


AltspaceVR: World Editor page for SDK apps should hide URI when using appID

The AltspaceVR world editor is currently allowing users to change the URI if an MRE has been created using a registered app. This shouldn't be allowed - if it is a registered app, the appID should be used to look up the URI from ALtspace's app register every time
There are 2 separate changes needed

In the World editor page to change settings on an MRE, if the MRE has an App ID then the world editor should disable the URI editor
During MRE instantiation, if an MRE has an App ID, then the App ID should be used to look up the URI, rather than trust the URI coming in from the space component

One question mark with this issue (which I agree with in general) is how query arguments should be handled. Some (many?) apps rely on them to be useful.

good point. Maybe instead of hiding the field it could show mre:12345678?parameters or  ws://uri?parameters, depending on if it is using an appid or a uri.

I don't think we should expose the app ID to the users. Maybe we can make the URL a label instead of a text field, but keep a text field for the query params?

Good idea to separate the two. There already is a separate label field for all MREs, so maybe we could separate parameters and URL, and just hide URL field when we have an appID.

Agree with Steven on the approach in separating the two.


Disable OSMFLUTTER tap/drag

I would like to disable touch and drag on the map, as I only use it as a viewer, and I am having a problem showing dialogs in front of the map, as the focus remains on the map and not on the dialogs so I can click on their buttons, if you can help me Help me resolve this, I'd be grateful!
I've used many things to disable the touch on the map, but I wasn't successful, I used GestureDetector to capture the touches and disable it, I used Positioned to position the dialog on the map, I used IgnorePointer, Listener, Positioned.fill...
Example of my code:
Container(
height: 300,
margin: EdgeInsets.only(top: 20),
width: MediaQuery.of(context).size.width - 40,
child: Stack(
children: [
OSMFlutter(
controller: Controller.to.mapController,
osmOption: OSMOption(
enableRotationByGesture: false,
zoomOption: ZoomOption(
initZoom: 14,
maxZoomLevel: 19,
),
staticPoints: Controller.to.staticPoints,
),
),
],
),
)

flutter doctor -v
[√] Flutter (Channel stable, 3.22.2
• Flutter version 3.22.2
• Dart version 3.4.3
• DevTools version 2.34.3
flutter_osm_plugin: ^1.0.5

i will add that feature soon

If it's not too much to ask, can you tell me how long this option will be available?
I'm waiting for this issue...

i will work on small feature for custom tile and this one
it should not  be so difficult so
probably this weekend or beginning of next week

Excellent, I look forward to hearing from you, thank you!!

sorry for being late
the new version will published today or tomorrow at maximum

the release will up in one hour
preparing just the docs
I created new map widget called as you can see in this example
you should use with that widget SimpleMapController
       OSMViewer(
             controller: SimpleMapController(
               initPosition: GeoPoint(
                 latitude: 47.4358055,
                 longitude: 8.4737324,
                ),
                  markerHome: const MarkerIcon(
                    icon: Icon(Icons.home),
                  ),
                ),
                zoomOption: const ZoomOption(
                  initZoom: 16,
                  minZoomLevel: 11,
                ),
              )

Thank you for your help Liodali!!
It really helped me to block the map touch, it was static.
However, regarding the dialogs, the map continues to overlap them, this is a problem, as I need to click on their buttons and I am unsuccessful, my buttons seem to be disabled due to the map...

See my code sample:
Dart version 3.4.3
Flutter version 3.22.2
flutter_osm_plugin: 1.2.0
Container(
height: 300,
margin: EdgeInsets.only(top: 20),
width: MediaQuery.of(context).size.width - 40,
child: OSMFlutter(
controller: Controller.to.mapController,
osmOption: OSMOption(
enableRotationByGesture: false,
zoomOption: ZoomOption(
initZoom: 16,
maxZoomLevel: 19,
),
staticPoints: Controller.to.staticPoints,
),
),
)

Get.defaultDialog(
title: "Salvar",
middleText: "Você possuí alterações, deseja salvar?",
backgroundColor: Colors.white,
titleStyle: TextStyle(color: Color(0xff01243a), fontWeight: FontWeight.bold),
middleTextStyle: TextStyle(color: Color(0xff424242)),
textConfirm: "Salvar",
textCancel: "Não Salvar",
cancelTextColor: Color(0xff424242),
confirmTextColor: Colors.white,
buttonColor: Color(0xff01243a),
onConfirm: () async { },
onCancel: () async {},);

is your app running on web?

yes

add this package to pubspec
pointer_interceptor: ^0.9.3+3
and put your button under the pointerInterceptor widget it will work as expected thats issue related to iframe in flutter


Post file returns Max Content-Length exceeded

This is from chrome dev tools capturing the file upload on the destination website where the file upload works just fine.

Request Headers:
Accept:text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,/;q=0.8
Accept-Encoding:gzip,deflate
Accept-Language:en-US,en;q=0.8
Cache-Control:max-age=0
Connection:keep-alive
Content-Length:10573
Content-Type:multipart/form-data; boundary=----WebKitFormBoundaryAauA5ite0s0Ci520
Cookie:JSESSIONID=9D456DCA826972CFA443D62E11AF2B15
Host:destination.site.com
Origin:https://destination.site.com
Referer:https://destination.site.com/upload_file.jsp?id=0000&type=round&companyid=0000&d_type=O
User-Agent:Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/38.0.2125.104 Safari/537.36

Request Payload:
------WebKitFormBoundaryAauA5ite0s0Ci520
Content-Disposition: form-data; name="todo"
upload
------WebKitFormBoundaryAauA5ite0s0Ci520
Content-Disposition: form-data; name="id"
0000
------WebKitFormBoundaryAauA5ite0s0Ci520
Content-Disposition: form-data; name="companyid"
0000
------WebKitFormBoundaryAauA5ite0s0Ci520
Content-Disposition: form-data; name="type"
round
------WebKitFormBoundaryAauA5ite0s0Ci520
Content-Disposition: form-data; name="d_type"
B
------WebKitFormBoundaryAauA5ite0s0Ci520
Content-Disposition: form-data; name="uploadfile"; filename="loadFile.xlsx"
Content-Type: application/vnd.openxmlformats-officedocument.spreadsheetml.sheet

------WebKitFormBoundaryAauA5ite0s0Ci520--

And this is my script:

$post_fields = array(
'todo'          => 'upload',
'id'     => '0000',
'companyid'     => '0000',
'type'          => 'round',
'd_type' 	   => 'B'
);
$request = $client->createRequest('POST', 'https://destination.site.com/upload_file.jsp', ['cookies' => $jar]);
$postBody = $request->getBody();
foreach ($post_fields as $key => $value) {
$postBody->setField($key, $value);  }
$postBody->addFile(new PostFile('uploadfile', fopen($filePath, 'r')));
$response = $client->send($request);

I tried setting every single HEADER like in chrome but am still getting a 500 error.
I do not have control over the destination server, the most they would give me is that it produces a Java error "java.io.IOException: Max Content-Length exceeded"
Any insight would be greatly appreciated.
Thanks

Doing a triage. Closing until there's more information that allows me or someone else to reproduce the error.


Problem with same ID

Hello, I've created a dimension and they created with the same ID without deleted previous dimension.
Screen ->
http://prntscr.com/f7j603
http://prntscr.com/f7j63p

If you use exactly the same dimlets you get exactly the same dimension. i.e. there will be no new dimension created. Use 'digit' dimlets to make new random dimensions if that's what you want to do


Native Molecule Hashing

Is your feature request related to a problem? Please describe.
mol1 == mol2 should use get_hash() not whatever comparison is native to Pydantic.
Describe the solution you'd like
Override __hash__ in Molecule to call get_hash() and maybe deprecate that function so that hash(Molecule) works just as well.
@mattwelborn

Is there any weirdness that could happen here? For example in Fractal, when we record a geometry optimization with tight convergence thresholds, will the last steps all end up in the molecules table? Or will they be seen as

This relates to #89. There will be a field saying that Molecules cannot be hashed and therefore cannot be compared.
We do see sometimes with geometric where we get the same molecule in the optimization trajectory. It is rare, but does happen due to our thresholds.

Overring __hash__  has two issues:

__hash__ must return an integer
"hash() truncates the value returned from an object’s custom hash() method to the size of a Py_ssize_t. This is typically 8 bytes on 64-bit builds and 4 bytes on 32-bit builds."

We could make get_hash return our current hashes and __hash__ cast it to an integer, but I prefer to just change __eq__ to use get_hash() and not deprecate get_hash()
Another issue is whether compare should be deprecated.


IGNITE-944 HTTP REST log command doesn't work

Hi,
I have updated the PR with review feedback.
Regards
Saikat

Hi,
I have updated the PR with review feedback.
Regards
Saikat

Hi Saikat, thanks!
I'm on holidays and will be able to review your changes at the end of the
next week.
Denis
On Friday, January 1, 2016, Saikat Maitra<EMAIL_ADDRESS>wrote:

Hi,
I have updated the PR with review feedback.
Regards
Saikat
—
Reply to this email directly or view it on GitHub
https://github.com/apache/ignite/pull/383#issuecomment-168302962.

Hi Saikat, thanks!
I'm on holidays and will be able to review your changes at the end of the
next week.
Denis
On Friday, January 1, 2016, Saikat Maitra<EMAIL_ADDRESS>wrote:

Hi,
I have updated the PR with review feedback.
Regards
Saikat
—
Reply to this email directly or view it on GitHub
https://github.com/apache/ignite/pull/383#issuecomment-168302962.

Thank you Denis. Let me know if any further changes are required.
Regards
Saikat
On Fri, Jan 1, 2016 at 9:42 PM, Denis Magda<EMAIL_ADDRESS>wrote:

Hi Saikat, thanks!
I'm on holidays and will be able to review your changes at the end of the
next week.
Denis
On Friday, January 1, 2016, Saikat Maitra<EMAIL_ADDRESS>wrote:

Hi,
I have updated the PR with review feedback.
Regards
Saikat
—
Reply to this email directly or view it on GitHub
https://github.com/apache/ignite/pull/383#issuecomment-168302962.

—
Reply to this email directly or view it on GitHub
https://github.com/apache/ignite/pull/383#issuecomment-168314485.

Thank you Denis. Let me know if any further changes are required.
Regards
Saikat
On Fri, Jan 1, 2016 at 9:42 PM, Denis Magda<EMAIL_ADDRESS>wrote:

Hi Saikat, thanks!
I'm on holidays and will be able to review your changes at the end of the
next week.
Denis
On Friday, January 1, 2016, Saikat Maitra<EMAIL_ADDRESS>wrote:

Hi,
I have updated the PR with review feedback.
Regards
Saikat
—
Reply to this email directly or view it on GitHub
https://github.com/apache/ignite/pull/383#issuecomment-168302962.

—
Reply to this email directly or view it on GitHub
https://github.com/apache/ignite/pull/383#issuecomment-168314485.