Add Jalali calendar system support

What problem does this feature solve?
Now that the ng-zorro-antd finally added RTL support we can easily use this excellent UI library in our projects that are in RTL languages. The only issue remaining is in some RTL languages like Farsi or Persian the calendar system is Jalali instead of Gregorian. It would be awesome if the Date Picker or Calendar component support the Jalali calendar system.
What does the proposed API look like?
I see you guys use the date-fns npm package in your date utils, there is already a package called date-fns-jalali that I use a lot in my projects and has exactly the same API and method names as the date-fns library it self but the outputs are in Jalali calendar.
I think Supporting the Jalali calendar system is as easy as adding a config or an option to nzDatePicker or nzCalendar like calendarSystem that accepts gregorian and jalali and defaults to gregorian, if it was jalali the date utils should call exactly the same methods from date-fns-jalali instead of date-fns and that's it., it should work just fine. The only drawback I see is adding a dependency of a new npm package.

If you guys are busy or something I can implement it myself as long as you guys are OK with me adding the date-fns-jalali package.

Hi there,
This would make my life easy. Any progress friends?

Hi there,
This would make my life easy. Any progress friends?

Hi, @Sina7312 @omidazm94
https://github.com/date-fns/date-fns/issues/977
it would be best if Jalali calendar could be part of date-fns package

Hi, @Sina7312 @omidazm94
https://github.com/date-fns/date-fns/issues/977
it would be best if Jalali calendar could be part of date-fns package

@vthinkxie thanks for the response. I will check with the authors of date-fns-jalali and date-fns that if they can come up with a solution to merge date-fns-jalali to date-fns and add Jalali support to the core package. I will report back here.

@vthinkxie thanks for the response. I will check with the authors of date-fns-jalali and date-fns that if they can come up with a solution to merge date-fns-jalali to date-fns and add Jalali support to the core package. I will report back here.

what's the status @Sina7312 ? did you successfully communicate with date-fns authors and made any progress?

Hi,
Right now, It's not possible to merge date-fns-jalali to date-fns. But it would be easy to support the Jalali calendar system in other libraries. for example, @material-ui/pickers  extracted all date logics to date-io repo. we just copy and modify date-fns adapter and creating date-fns-jalali. These files are almost equal only imports, and some formats are changed.
Are there any date-fns adapter for this lib?

@vthinkxie Hi, based on the answer from @smmoosavi here and also his other answer in the https://github.com/date-fns/date-fns/issues/977#issuecomment-801094803, there is no possible way to add Jalali calendar to the main date-fns package. Can you think of another way to do it?

Hi @Sina7312 , I think we can support this in Dependency Injection like useFactory
here is a demo that how we support customize resizeobserver polyfill
https://github.com/NG-ZORRO/ng-zorro-antd/commit/29d44afb058cb5d78f236cdfa57be5018b49dc02#diff-f74144a89aad109f270b3d2be30adcc5b99eef97a265ce2ddb12feb24c53dfd4R80
PR welcome

step to fixing the issue:

[ ] refactor all direct date method calls to date-fns function. e.g. value.getMonth() to getMonth(value)
[ ] refactor all date-fns usage to one adapter class. some thing like this class
[ ] write date-fns-jalali adapter implement same interface

I also need to use nz zorro jalali persian calender

Hi, Please Add Jalali DatePicker

I need to use Jalali date picker for my project please add

Hi,
Any progress?

Hello everyone, I recently released a datepicker that supports the Jalali calendar and has good features, you can use it.
https://www.npmjs.com/package/@qeydar/datepicker


Remove special handling for index mismatch in RAFT refinement once supported

After resolution of https://github.com/rapidsai/raft/issues/1950, the special handling for cases where the index type and mdspan index type do not match can be removed.

/assign @Presburger

keep trace


Improve building the PDF via Makefile

Description
This fixes Makefile being outdated & improves shell.nix to be able to use Makefile more conveniently.
Checklist

[x] Commit sequence broadly makes sense and commits have useful messages
[x] Code is formatted according to CONTRIBUTING.md
[x] Self-reviewed the diff

Also, the pdf ends up in the top level directory in the file PDF.pdf.  Should it not be called cardano-ledger.pdf ...and maybe for consistency should it end up in a result directory?

The result directory is a nix thing, so I prefer not messing with that. I've renamed it to cardano-ledger.pdf & switched to latexmk.

I think your last rebase may have clobbered your improvements to the Makefile.  Sorry I missed that before merging.  I'll open a new PR with your previous Makefile mods.


nnsearch missing

Hello,
I am new to Python and I installed Anaconda and thereafter followed the instructions given
on the website.
I get the following message:
(kaolin) csverma@blackhole:$ python
Python 3.6.7 | packaged by conda-forge | (default, Nov  6 2019, 16:19:42)
[GCC 7.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.

import kaolin as kal
Traceback (most recent call last):
File "", line 1, in
File "/mnt/35f6163f-1be0-4350-89a5-b885cc687dd6/csverma/Software/MLearn/kaolin/kaolin/init.py", line 17, in
from kaolin import conversions
File "/mnt/35f6163f-1be0-4350-89a5-b885cc687dd6/csverma/Software/MLearn/kaolin/kaolin/conversions/init.py", line 1, in
from kaolin.conversions.meshconversions import *
File "/mnt/35f6163f-1be0-4350-89a5-b885cc687dd6/csverma/Software/MLearn/kaolin/kaolin/conversions/meshconversions.py", line 22, in
from kaolin.metrics.point import directed_distance as directed_distance
File "/mnt/35f6163f-1be0-4350-89a5-b885cc687dd6/csverma/Software/MLearn/kaolin/kaolin/metrics/init.py", line 2, in
from .point import *
File "/mnt/35f6163f-1be0-4350-89a5-b885cc687dd6/csverma/Software/MLearn/kaolin/kaolin/metrics/point.py", line 16, in
from kaolin.nnsearch import nnsearch
ModuleNotFoundError: No module named 'kaolin.nnsearch'

Seems to be a duplicate of https://github.com/NVIDIAGameWorks/kaolin/issues/9#issuecomment-554422485


Provide resolver options with verifyCredential API

Hi Team,
I'm currently exploring didkit for our projects with context to verifiable credential and presentation. The current implementation of verifyCredential in web or node module doesn't have resolver_options while I can see that it is very much there for cli. Would it be possible to align all three modules with resolver option so that did resolution and verification could be consumed for last updated DIDDoc?
If there is any specific reason why there is no resolver option for web or node modules, please help me understand as I might have missed or misunderstood the implementation. Appreciate your help!

Hi @nichonien,
Can you say more about what why you need the resolver options, e.g. what are the use cases?
resolver_options (ResolverOptions) in the CLI includes options to use a given DID resolver HTTP(S) endpoint instead of the built-in resolvers (did_resolver_override, -R in CLI), or as a fallback resolver for non-built-in DID methods (did_resolver, -r in CLI). These CLI options are available in the VC/VP issue/verify and DID Auth CLI subcommands. Can you confirm are these the options you want in node/web?
There are also DID resolution options ("DID resolution input metadata", e.g. accept, versionId, versionTime) that could be passed to the DID resolution function (either with the built-in DID methods, or to a remote resolver); but these are not exposed in the credential issuance/verification/presentation functions currently.
The API of didkit web and node are based on that of didkit-http, which is based on VC API. The VC API verify operations don't have an option for DID resolution options or DID resolvers to use during verification. Possibly we could extend VC API to allow this - although the trend in VC API seems to be about reducing optionality, so maybe such options would have to be added outside the VC API data structures.

Hey @clehner,
The current requirement that we are looking at is to verify a credential from the DIDDocument of the subject. So maybe we should be able to provide a resolver which resolves a did on some configured chain and then verifyCredential be able to verify the credential?
We can verify a credential known before hand, but for the case where you also want to verify let's say issuer authority for issuance as well. Then one of the approaches would be get the authoritative credential from issuer's DIDDoc. What would you suggest for the mentioned use-case?


Visual issue caused by longer clips and regions

Moving around the scroll bar at the end of long clips leads to the annotations moving around.
It appears the scroll bar and the spectrogram are misaligned with longer clips, this could be due to inaccurate pixel to sec measurement,

Fixed it: The issue was that wave surfer tries to make it easier to render long clips by breaking up the rendering into serval canvas, which causes weird little bugs whenever one canvas ends and another begins which consequently broke the synching of the audio. I turned up the maxcanvas size up by about a few ten thousand pixels and that seemed to fix up the syncing issues:
EDIT: I've been pasting my updates on this issue accidently in issue #4 so here is a list of all my previous updates:

This seems to occur in clips in which the scroll becomes slightly ahead of the progress bar

In this example, the bar is not centered as most clips are when using the scroll wheel
EDIT: the black line behind the progess bar was there for testing, i'm looking to remove it for now

Visual issue was fixed by increasing max_canvas size in annotate.js


fix(helm): Tunnel closing already closed channel

k8s client-go closes the ready channel that's passed in (see https://github.com/kubernetes/client-go/blob/master/tools/portforward/portforward.go#L171) This means that Tunnel's Close will always panic, as the client-go library will have closed then channel. This isn't reproducible unless helm.Client is externally, as the helm cli runner doesn't actually invoke Close.

cla signed, commenting to trigger update.


Cody Gray: Watch keylok.com

Cody Gray requests the watch of the watch_keyword keylok\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

keylok\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Zoe in Charcoal HQ


Cannot pull docker image from "localhost:5000"

I followed this page and deployed IoT edge on minikube successfully. However, I cannot upgrade edge agent from "0.1.0-alpha" to the latest version from both VS Code or Azure Portal by changing the desired value.
There was a bug that edge agent only supports docker registry with port number 80 (fixed by "1.0.8+"). If you used the code template generated by VS Code, you will hit this bug. After followed the link below and bind local docker registry to port number 80, I can deploy my edge module locally.

https://github.com/Azure/iotedge/issues/1191

Is there a way to upgrade edge agent and edge hub to the latest version? Thanks.

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

ID: c1a53f1e-4a0f-e340-b0ab-0e75046afbac
Version Independent ID: 14170d59-92c9-6639-7def-51e7e28da3cd
Content: How to install IoT Edge on Kubernetes
Content Source: articles/iot-edge/how-to-install-iot-edge-kubernetes.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: veyalla

@yenbohuang Thanks for trying IoT Edge! We try to reserve these issues for feedback directly related to the tutorials: like if there are errors that blocks tutorial completion or if there are updates/changes required, suggestions for improvement, etc. If this is not the case I would open an issue with your question on the Edge repository or StackOverflow MSDN dev forums where there is a community that can help (plus the engineering team)

Thanks for the reply and I will forward this question to stackoverflow.


FEATURE: Providing a feature to swap the stable-diffusion unet model via Patching

Another parking lot ticket for discussion, I don't expect or want immediate movement on this:
https://arxiv.org/abs/2207.04316
Providing an implementation or tutorial/guide for swapping out the front UNet layer on SD and then finetuning it, seems likely able to reduce the RAM requirements for users to run the model with minimal performance costs (V100s have 16GB vram, so for prosumer graphics cards like the 3080, it becomes viable to train / finetune on one's own PC).

I'd like that too. There's a growing wish list on this repository's wiki if you'd like to add this on.

Wishlist link https://github.com/lstein/stable-diffusion/discussions/195


x/mobile: panic: unsupported GOARCH: arm64

goroutine 1 [running]:
main.archNDK(...)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:298
main.(*ndkToolchain).Path(0x14000197b00, 0x140000b8000, 0x29, 0x1011be402, 0x5, 0x0, 0x14000197938)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:327 +0xc0
main.envInit(0x14000016047, 0x31)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:108 +0xa0c
main.buildEnvInit(0x1014a8980, 0x1400011dc20, 0x0)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:79 +0x1a8
main.runBuildImpl(0x10149f860, 0x0, 0x0, 0x0)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/build.go:76 +0x38
main.runBuild(0x10149f860, 0x14000140020, 0x2)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/build.go:69 +0x28
main.main()
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/main.go:73 +0x2c0
device: MacBook pro m1

please fill out the entire template.

Looks like he is hitting this panic https://github.com/golang/mobile/blob/master/cmd/gomobile/env.go#L298
And looking up NDK, there is no darwin arm64 build https://developer.android.com/ndk/downloads
cc @hajimehoshi

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

$ go version
go version go1.16.3 darwin/arm64

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="auto"
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/xxx/Library/Caches/go-build"
GOENV="/Users/xxx/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/xxx/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/xxx/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/homebrew/Cellar/go/1.16.3/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/homebrew/Cellar/go/1.16.3/libexec/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.16.3"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/qh/w0rmms1543l0501ptgw_zhxh0000gn/T/go-build2439043797=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
gomobile init

What did you expect to see?
What did you see instead?
goroutine 1 [running]:
main.archNDK(...)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:298
main.(*ndkToolchain).Path(0x14000197b00, 0x140000b8000, 0x29, 0x1011be402, 0x5, 0x0, 0x14000197938)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:327 +0xc0
main.envInit(0x14000016047, 0x31)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:108 +0xa0c
main.buildEnvInit(0x1014a8980, 0x1400011dc20, 0x0)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/env.go:79 +0x1a8
main.runBuildImpl(0x10149f860, 0x0, 0x0, 0x0)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/build.go:76 +0x38
main.runBuild(0x10149f860, 0x14000140020, 0x2)
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/build.go:69 +0x28
main.main()
/Users/xxx/go/src/golang.org/x/mobile/cmd/gomobile/main.go:73 +0x2c0

@hyangah I don't have M1 Mac then I cannot test this, but I guess gomobile should support M1 environments and this should be easy. What do you think?

And looking up NDK, there is no darwin arm64 build https://developer.android.com/ndk/downloads

Can't the toolchains be downloaded via Android Studio?

Can't the toolchains be downloaded via Android Studio?

Nope, as far as I can tell, there is no M1 build of NDK at all yet.
But M1 should be able to run x86_64 code but I'm not sure if that works with the NDK

Ah that makes sense. Then for M1 mac, we should invoke x64 toolchains with Rosetta.

ndk support should be android/ndk#1299 , comments theresay rosetta works

Can't the toolchains be downloaded via Android Studio?

Nope, as far as I can tell, there is no M1 build of NDK at all yet.
But M1 should be able to run x86_64 code but I'm not sure if that works with the NDK

” gomobile build -target=ios ” ，the same mistake

Ah that makes sense. Then for M1 mac, we should invoke x64 toolchains with Rosetta.

Unfortunately this may help for Android by now, but not for iOS as with Xcode 13:
❯ clang --version
Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
Target: arm64-apple-darwin20.4.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin


Issue with ClearAllCompleted button

Great job on implementing all the features according to the project requirements 💯 🍾
However, there is an issue that I would highly suggest fixing ♻

Please consider checking the function of the ClearAllCompleted button which is supposed to clear all the checked items (kindly check 'npm run build' again to redistribute the output code  which could possibly be the issue)

Thank you for noticing it. I have fixed the bug.


[HowTo] Whole Page Rotation

I had an issue on page rotation.
If you need to do it, there's a hack because of page kids property.
You need to reassign page dictionary to pages kid.
doc.page.dictionary.data.Rotate = 270;
// where 0 is the current page number.
doc._root.data.Pages.data.Kids[0] = doc.page.dictionary;

@wadjeroudi thank you for this! Saved me :)


[AutoPR keyvault/resource-manager] [Key Vault] reverting changes which introduce breaking changes to generated SDKs

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2462

closing as these changes will be merged into keyvault_1.0_preview prior to being merged into master


The latest version is missing the ability to prompt custom components.

Vue - Official extension or vue-tsc version
2.0.21
VSCode version
1.90.1
Vue version
3.4.27
TypeScript version
4.5.4
System Info
System:
    OS: macOS 14.5
    CPU: (12) arm64 Apple M2 Max
    Memory: 2.19 GB / 32.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 14.21.3 - ~/.nvm/versions/node/v14.21.3/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 6.14.18 - ~/.nvm/versions/node/v14.21.3/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 126.0.6478.61
    Safari: 17.5

Steps to reproduce
Create a new component and add properties. Introduced in another file with no attribute hint;
Link to minimal reproduction
No response
Any additional comments?
In the latest version, properties of custom components are not prompted; I remember the old version has always been available, but also very useful, will greatly improve efficiency. But it seems that the latest version will not prompt! I tried to install every old version and found that it didn't work. Is there any configuration I need to change? (Figure 1 is the problematic version, Figure 2 is the desired version)

？？？

We don't support ts 4.5.4


only 1655 comics viewable

I can only access Comic 1-1655, as well as on the latest.
i have the version 5.2, I have activated th Offline mode for the comics and for What-If, also the large image setting.

Ave you tried "repair offline" (settings->advanced)?

I have done this, but it didn't work.

Weird, could you provide some screenshots?

Screencast.mp4.zip
i have made an video from the bug, this should show you what I mean. Its in the zipfile

Oh ok, didn't realize you're talking about overview mode. Are you sure the "Hide read"  ("Versteckene gelesene") option is not checked?

Hide read is not checked and if i check it it only show me comic 0, because i have read all. I have made the video only of this view, because the two others have also this bug

What happens when you select "mark all as read" ?

it makes no difference if they are marked as read or not.

I'm sorry, I don't really know what's causing the bug then. Deleting app data may work

now it works. Thank for the help.


Basic Output

Basic output should be able to just show one pass/fail for an entire run op. If there's an error it should output the error as well. If a script is explicitly logging something, it should only show in the verbose log for now.

This is more or less good. Failures are showing and also short-circuiting the chain, which is what we want.


Ok method

Hi. I don't really understand what the Ok method does in the below C# part.
Could anyone briefly explain what it does and the importance of it?
public async Task<IActionResult> OnPostUploadAsync(List<IFormFile> files)
{
long size = files.Sum(f => f.Length);
foreach (var formFile in files)
{
    if (formFile.Length > 0)
    {
        var filePath = Path.GetTempFileName();

        using (var stream = System.IO.File.Create(filePath))
        {
            await formFile.CopyToAsync(stream);
        }
    }
}

// Process uploaded files
// Don't rely on or trust the FileName property without validation.

return Ok(new { count = files.Count, size });

}

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

ID: 9a5cfa38-0e6c-4346-dafc-84ea5c513fcf
Version Independent ID: c6787f0f-3e9a-6efb-0e6e-5ccb957d3f4a
Content: Upload files in ASP.NET Core
Content Source: aspnetcore/mvc/models/file-uploads.md
Product: aspnet-core
Technology: aspnetcore-mvc
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

Potential bugs in the ASP.NET Core framework and product feedback can be reported at dotnet/aspnetcore issues (include full repro steps and cross-link to a repro project in GH when appropriate). For Visual Studio, use the Report a Problem gesture or Suggest a Feature gesture within VS, which open internal issues for the VS team. For Visual Studio Code, you can open an issue at microsoft/vscode issues.


Please consider package dependencies

The emulator works simply great, even for relatively complex configurations. Here's the thing:
The PyPi package dependencies are rather outdated, and ultimately later versions of Python and Pygame cannot be installed. I work around easily enough: clone the repo and use it as such. With no problem at all.
Here's my setup I'd really love to be supported by the package:
Python 3.9.x
Pygame 2.0.1
...

Obviously, this is just cosmetics. Works as advertised anyway 👍

Thanks


Fix css #two test color set to 'blue' to match instructions so user can pass challenge

…pass the challenge
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Fix css #two test color set to 'blue' to match instructions so user can pass challenge. I was only able to pass the challenge using the color '#0000ff' for the #two css selector before. The instructions say to use color names, in this case 'blue'.

I can only pass the challenge with: #two {
color: #0000FF;
font-size: 36px;
}
Putting 'blue' in for the color doesn't work. I was stuck here until I
looked at the repot file.
You can check it with the link below.  It doesn't
pass the check for the second element with 'blue' but will with '#0000ff'.
https://www.freecodecamp.org/learn/the-odin-project/top-learn-css-foundations-projects/css-foundations-exercise-b
On Wed, Dec 20, 2023 at 7:34 PM Huyen Nguyen @.***>
wrote:

@.**** commented on this pull request.
Hi @bowen-anth https://github.com/bowen-anth, is there an issue thread
for this change? Please if there isn't.
Curriculum changes impact a lot of learners, so we want to err on the side
of caution and have the issue go through the triage process.
—
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/52674#pullrequestreview-1792033047,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANIQKXJJFNTUUAK223ZXVBLYKON2HAVCNFSM6AAAAABA5VTGZOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOOJSGAZTGMBUG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Added Chapter 1 - 1.mdx for Nepali Language

Hi @Bikram48 and @stevhliu,
I've added the Nepali translation for a new chapter in this PR.
@Bikram48: Would appreciate your review for accuracy and natural flow, @stevhliu: Please review for overall content alignment.
Thanks in advance for your review!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Make logging configurable

This would cover options like:

Logging output to other places (i.e. not to stdout)
Customizing the format/representations printed

This is an external tracking issue to:

Gauge interest from the community for this feature
Learn about what you'd want to see out of it if we worked on it.

So please:

Upvote the issue if it's important to you, and
Comment with any relevant info on your requirements use cases, etc.

Thanks!
PS We do all our internal engineering issue tracking separately in Notion, so you won't necessarily see regular updates to the project status here even once we begin work.

It would be very nice if this used a standard logger that could be intercepted. In a typical Python web app like FastAPI I will do a fair bit of work to roll up all the various vendor logs (ORMs, web framework, SDKs etc) into some nice shape and then ship them to CloudWatch, files, stderr and so on.


Timer documentation link opens up twice

Describe the Bug
When we try to open up the documentation for timers the exact same link opens up twice.
Steps to Reproduce

Create a timer event
open up the properties panel for the same and click on "Documentation: Timer events" under Timer.
We could see that the documentation link opens up twice

Expected Behavior

The link leading to documentation on timer events should open up once.
Environment

OS: [ Windows 10]
Camunda Modeler Version: [5.0.0]
Execution Platform: [ Camunda Platform, Camunda Cloud]

https://user-images.githubusercontent.com/85225325/162408343-aba9996a-3a0f-4827-9fbe-279efd301fae.mp4

I can reproduce it on Mac.

Also reproduceable on Linux with all other links in the properties panel (User Task, Multi-Instance Task)
Links in What's new don't seem to be affected

...and I can't reproduce it in 4.12. So it's a regression.

We trigger two events in the properties panel:
[new-window] new window to https://docs.camunda.org/manual/latest/reference/bpmn20/events/timer-events/#time-duration
[will-navigate] navigating to https://docs.camunda.org/manual/latest/reference/bpmn20/events/timer-events/#time-duration

The reason for that is we use target=_blank in the properties panel. In What's new, we don't

fixed with https://github.com/camunda/camunda-modeler/pull/2911


[Windows] Can't connect controllers

If i connect the controller using the usb cable i can't connect it. The light at the bottom of the controller is flashing and in the config it says.    No bluetooth adapter detected!, no bluetoioth connected controllers and no usb connected controllers. The cams and hmd does work but the controllers not

What bluetooth adapter are you using?

@POEDaley is right. Until we can determine that this is a code-related problem then you're more likely to get help in the google group.
P.S. You should only PAIR the controller when the USB is plugged in. After that you should disconnect the USB cable. A flashing light, depending on speed, just means that the controller is charging. But the log saying "No bluetooth adapter detected" is a bad sign. If you are using drivers supplied with your bluetooth dongle then try uninstalling those drivers and letting Windows handle your bluetooth adapter.
I'm closing this issue for tracking. If you want further help then you should really post in the google group first.


Adding deprecation warning message for pynapple IO

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 11 (18.18%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.3%) to 60.283%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pynapple/io/loader.py
2
11
18.18%

Files with Coverage Reduction
New Missed Lines
%

pynapple/io/neurosuite.py
1
16.63%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
1632

Relevant Lines:
2648

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 11 (18.18%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 59.882%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pynapple/io/loader.py
2
11
18.18%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
1632

Relevant Lines:
2647

💛  - Coveralls


Configuration for localhost or on prem LLaMA server

Describe the need of your request
Hi,
First of, great work with this plugin!
I was wondering if it would be possible to enable an On Prem LLaMA C/C++ configuration in the UI.
The  LLaMA C/C++ option seems to only be available in the options for Linux MacOS, but there are some configurations where Windows users could also connect to a WSL server instance on their localhost or just connect to an LAN instance.
Launching the prompt would work but would only work if the LLaMA server is configured on Localhost on the default 64531 port at the moment.
It would probably work to only disable launching the server for the  LLaMA C/C++ on Windows and keep the IP / Port configuration side.
Thanks in advance,
Best,
Thibault
Proposed solution
No response
Additional context
No response

Hi,
I agree, enabling users to configure the host and port would be sufficient before we can actually support running the server on OSs other than UNIX.
I will add the support as soon as I have some time.

Hi,
Maybe this will help, ollama just went in preview on windows. And it has an API that runs on http://localhost:11434
https://ollama.com/blog/windows-preview

@Bmooij thanks for the info! I will have a look :)


Suggestion: remove extra summary result from Data Driven tests

Summary
Data Driven tests give an extra result that is the sum of all the data driven tests. So if all are passing then the summary test passes, if one fails then the summary test also fails. We got some feedback that this extra result was confusing and does not provide them much value.

If the test fails, I am only about which set of input data caused the failure. A comment on this docs would also help eliminate the confusion.

We can document this behavior, but have we considered taking it out? Have we received contradictory feedback?

@PBoraMSFT , @pvlakshm to take a call

Hi Kendra,
The basic principle of Data driver or ordered tests is if all are passing then it passes, if one fails then it also fails. Other than a customer feedback, can you share other reasons why should this change?

Memo: test if that's correctly fixed.

I have just checked and we now correctly display only 1 test in the result.


Update Composer dependencies (2021-04-22-00-09)

Visual regression test passed!
View the visual regression test report


[security] setIconand closeIconClass can be used to inject arbitrary HTML

jquery-confirm version:
v3.3.4
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request
Current behavior:

The setIcon method and the closeIconClass parameter can be used to inject HTML.
Expected behavior:

setIcon and closeIconClass should set a class name, and not permit HTML injection.
Steps to reproduce:
// This shows succesful script execution: alert(0) is executed, while it should not be!
$.confirm().setIcon('"><img src onerror="alert(0)"><"')

Related code:
https://github.com/craftpip/jquery-confirm/blob/v3.3.4/js/jquery-confirm.js#L834
https://github.com/craftpip/jquery-confirm/blob/v3.3.4/js/jquery-confirm.js#L753
Other information:
This bug can cause security issues (XSS), when a library consumer passes external values to setIcon.
To fix this, use addClass instead of concatenating HTML strings.

🛠 A fix has been provided for this issue. Please reference: https://github.com/418sec/jquery-confirm/pull/2
🔥 This fix has been provided through the https://huntr.dev/ bug bounty platform.

Bug Bounty
We have opened up a bounty for this issue on our bug bounty platform. Want to solve this vulnerability? Go to https://huntr.dev/
We will submit a pull request directly to your repository with the fix as soon as possible. Want to learn more? Go to https://github.com/418sec/huntr 📚
Automatically generated by @huntr-helper...


for someone who needs to use@electron/remote

install vite-plugin-electron-renderer
npm install vite-plugin-electron-renderer

change vite config

...
import renderer from 'vite-plugin-electron-renderer'
...
const config = defineConfig({
  ...
  plugins: [vuePlugin(),renderer()],
  ...
})

Closing this issue as it's a small example that they it's in the readme of vite-plugin-electron-renderer itself.
Also not relevant to the template.
I appreciate sharing your findings. Feel free to share this in the discussions tab for others to find :)


GIF Search: Better results and codebase revamp

Description

The extension now uses GIPHY Pro and Tenor v2 to have better GIF results
The codebase has largely been revamped to make it easier to maintain

Checklist

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

@thomaslombart FYI, I still got a PR open for gif-search: #11794

@j3lte Could you test out this PR and let me know if it fixes what you initially fixed? This PR is revamping the codebase to make it easier to understand and simpler to maintain while still adding some nice features like caching the results.

Wow @thomaslombart, that's a serious rewrite!
Do I need to do anything to get it working, I end up here as soon as I open the extension

FetchError: request to https://gif-search.raycast.com/api/giphy?limit=20&offset=20 failed, reason: connect ECONNREFUSED <IP_ADDRESS>:9090
    at ClientRequest.<anonymous> (/Users/pernielsentikaer/.config/raycast/extensions/gif-search/search.js:23288:14)
    at ClientRequest.emit (node:events:526:35)
    at ClientRequest.emit (node:domain:488:12)
    at /Applications/Raycast.app/Contents/Resources/RaycastNodeExtensions_RaycastNodeExtensions.bundle/Contents/Resources/api/node_modules/@raycast/api/index.js:2:37408
    at Socket.<anonymous> (/Applications/Raycast.app/Contents/Resources/RaycastNodeExtensions_RaycastNodeExtensions.bundle/Contents/Resources/api/node_modules/@raycast/api/index.js:2:38379)
    at Socket.emit (node:events:514:28)
    at Socket.emit (node:domain:488:12)
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

I can't reproduce it… Could you post a recording to better see what happens? Is it the same if you switch to the Tenor provider?

I'm not sure what happened; it seems to be working much better now. I ensured that I could open the URL, so I don't know what caused it.
The search works SO much better than previous, well done @thomaslombart

Echoing this, great work @thomaslombart!
My only nit is regarding the empty states, on initial load we display a flickering empty state even for the services that automatically populates the list:

The empty state for when you get 0 hits on search is also the same generic one, instead of saying "No Results":


Could the glob route be faster?

Hey,
We experienced some slowdown when we tested the glob route of the API.
With the given glob pattern, quiet-stroll browse about 121k files in about 9 minutes (we also have a record of 49k files in 5 minutes). It is already pretty fast but we wonder if this could be event more accelerated?

Hello FlorentGodin,
I don't see any "given glob pattern" in this issue. Didn't you forget it?
Anyway, i need more info to help you. Can you please give me more details like the setup or machine specification?
It can depend of many reason, the complexity of the file system for example.
Thanks.
Philippe


SimRunner error in dict obs for Ray Policies

Error in SimRunner._computeSingleActionWrapper.
Seems to be caused by dict observations being passed into Policy.compute_single_action.
Traceback (most recent call last):
  File "/home/dylanrpenn/clockpunch/tests/simulation/test_sim_runner.py", line 238, in <module>
    sim_runner = SimRunner(env=env, policy=policy, max_steps=horizon)
  File "/home/dylanrpenn/clockpunch/punchclock/simulation/sim_runner.py", line 115, in __init__
    self.reset()
  File "/home/dylanrpenn/clockpunch/punchclock/simulation/sim_runner.py", line 266, in reset
    action_init = self._computeSingleActionWrapper(self.obs_hist[0])
  File "/home/dylanrpenn/clockpunch/punchclock/simulation/sim_runner.py", line 131, in _computeSingleActionWrapper
    action, rnn_state, _ = self.policy.compute_single_action(
  File "/home/dylanrpenn/anaconda3/envs/punch/lib/python3.10/site-packages/ray/rllib/policy/policy.py", line 545, in compute_single_action
    out = self.compute_actions_from_input_dict(
  File "/home/dylanrpenn/anaconda3/envs/punch/lib/python3.10/site-packages/ray/rllib/policy/torch_policy_v2.py", line 526, in compute_actions_from_input_dict
    return self._compute_action_helper(
  File "/home/dylanrpenn/anaconda3/envs/punch/lib/python3.10/site-packages/ray/rllib/utils/threading.py", line 32, in wrapper
    raise e
  File "/home/dylanrpenn/anaconda3/envs/punch/lib/python3.10/site-packages/ray/rllib/utils/threading.py", line 24, in wrapper
    return func(self, *a, **k)
  File "/home/dylanrpenn/anaconda3/envs/punch/lib/python3.10/site-packages/ray/rllib/policy/torch_policy_v2.py", line 1159, in _compute_action_helper
    dist_inputs, state_out = self.model(input_dict, state_batches, seq_lens)
  File "/home/dylanrpenn/anaconda3/envs/punch/lib/python3.10/site-packages/ray/rllib/models/modelv2.py", line 249, in __call__
    print(f"Line {inspect.currentframe().f_lineno}: {input_dict['obs'].shape=}")
AttributeError: 'collections.OrderedDict' object has no attribute 'shape'

Needed to convert obs to tensor first. Now seems to work. Unsure of why SimRunner was working fine a couple of weeks ago without this fix.


onchange event listener not working on useTextField

Hi everyone, sorry if I'm wrong. I have tried to create an input field using useTextField everything works as expected, but when I add the onchange event listener as a prop the input box cannot be changed. Even though the documentation says "Change, clipboard, composition, selection, and input event support".
Here is my code snippet:
const [value, setValue] = React.useState("my value");

<TextField label="Document" name="document" value={value} onChange={(event) => setValue(event.target.value)} />

And this is my TextField.js component:
function TextField(props) {
  const textFieldRef = React.useRef(null);

  const [isFocusWithin, setFocusWithin] = React.useState(false);

  const { labelProps, inputProps, errorMessageProps } = useTextField({ ...props }, textFieldRef);
  const { focusWithinProps } = useFocusWithin({
    onFocusWithinChange: () => setFocusWithin(prevState => !prevState)
  });

  returns (

      {props.label}

        <input className={classes.TextField_Input} {...inputProps} autoComplete="off" autoCapitalize="off" ref={textFieldRef} />

  );
}

export default TextField;

have i missed something?

I see you closed it, so I'm assuming you've figured it out. So anyone who stumbles across it, onChange receives a value, not the event object. I'm assuming that's what went wrong.
One other note, you don't need to spread into a new object in the useTextField hook, you can just pass props directly in


feat(setup): use versioned release artifacts in Windows "xs-dev setup"

This update to xs-dev setup for Windows mirrors the macOS setup process by:

Performing a shallow clone of the Moddable SDK repo
Using the versioned release of the Moddable SDK tools rather than building from source

Like on macOS, this approach uses the versioned release tools as both the "release" and "debug" binaries in build/bin/win/release and build/bin/win/debug, respectively.
When I get a chance, I'll make a new version of #91 built on this change. But no reason to wait on that to get this smaller change in, imho.

Like on macOS, this approach uses the versioned release tools as both the "release" and "debug" binaries in build/bin/win/release and build/bin/win/debug, respectively.

FWIW – in theory it should no longer be necessary to have the debug binaries. There were some unintentional dependencies on the debug binaries but those have been cleaned up.


Update README and docs/website to mention built-in CRI support

I noticed that the README makes no mention of Containerd now have CRI support out of the box. With the docker-shim being marked "deprecated" (https://github.com/kubernetes/kubernetes/pull/94624) and containerd being widely used as runtime for Kubernetes by major cloud providers, I think it would be good to add a section describing this feature to assist users that arrived in this repository looking for an alternative to the docker-shim.
The https://containerd.io website could probably use an update as well, as I couldn't find information about CRI (other than a link to the GitHub repository in https://containerd.io/docs/)
/cc @mikebrow @estesp @dims

Yes.. as a part of the containerd/cri merge need to:

[ ] merge the readme

[ ] merge the docs trees

[ ] blog about it

[ ] update containerd.io to include cri details

..
...

We should look into updating and resolving this issue now that kube 1.20 is out.

merge the readme - done

Moving to 1.6 milestone since we've released 1.5.


Android support?

I've tested in modern browsers on desktop and iOS, but when testing on Android devices the animation doesn't seem to trigger, however it displays in the end state which makes me think it fires on page load instead before scrolling.

I've just tested it on an android phone on firefox and chrome and it works on my end.
Please fill out the issue template so I could reproduce the issue.


IMA not working?

I'm trying to test out the IMA support here on QEMU.   I have patched poky to add initramfs into mkefidisk.wks, use systemd-boot and pass IMA bootargs and the following added to the end of conf/local.conf:

IMAGE_FSTYPES += "wic"
WKS_FILE_qemux86-64 = "mkefidisk.wks"
EXTRA_IMAGEDEPENDS_qemux86-64_append = " ovmf"

Doing a runqemu nographic serial ovmf wic results in the following console.txt
Using the ima-inspect util that I sent a PR for, I can see that when I boot with ima_appraise=off things are signed by the sample key:

ima_inspect -a ima /lib/ld-2.26.so
/lib/ld-2.26.so
security.ima
digital signature version 2
digest algorithm: sha256
key-id v2 (gpg compatible): d0fc63d5
signature length: 2048 bits
signature data:
476965e54eae61e6 88463ca03870e42a 698096d6de446a8d c6e39fc8de01cef0 8d353e698dd222d8 007f9d5081526ae7 f583f9082ddc9a35 feebe9242e6dfd92
1ffa731b3ea569af 39596f5f676d330c 1d879a38817b516d 528d36f9510be8e8 a9a8f63c1c0b8af6 bea055935b8c1df6 aea078cdb35b0a6d 1ec06cc02de44062
e1275a0cd4c57a88 3c2012ecb874fdca fbf2d4ff810c966d 6975f9b3de20ccef 3d9ded0b1ea2dcac 0b892b862e2f2a60 6f1a0e127bb4cba9 7eeaf4c8ed9e115e
b8e21ed6f2e4610f 6a426515d038a27f 9ae857ac6ca09883 fc764393a923a1fe d3f1f373bfcddbb1 c8b2d83dac0d023f 87d032724d054b9a e3f418adbfbf2624

Dumping x509_ima.der with openssl I can see:

    X509v3 extensions:
        X509v3 Subject Key Identifier:
            4D:24:84:EA:40:37:2F:88:5F:D6:B3:13:56:03:B4:B5:D0:FC:63:D5

So we have a key match.  Any ideas?  Thanks!

I finally sat down and figured out what was going wrong here.  The default oe-core (and meta-poky) local.conf.sample include image-prelink in USER_CLASSES and this in turn will modify the binaries and thus the IMA verification will correctly fail.


improving readability

Coming back to the notebook today - I've been trying to understand the SQL stuff.
In general, I'm looking at how to improve the readability of the notebook so that I can quickly understand what each cell is doing.
Some ideas I'm implementing:

removing all of the rendering logic - (slot layout diagram grid, threejs code, etc) out into a separate file. Since the rendering logic isn't related to the charge field concepts, it should be imported to reduce confusion.
making sure each cell has an element selector as necessary. having to run a cell, select a value, then run the rest of the cells is atypical for a jupyter notebook - it should just work to run through all the cells.
use markdown cells to elaborate on some of the charge field concepts

Thanks for the update.

basedphysics, its been a couple of weeks. How goes it?
I gathered your "improving readability" branch was a draft and not ready for a merge. I hope I wasn't mistaken by not merging an actual PR.
Anyway, I also cited my confusion at the 'forum'.
Cr6 has a close to complete molecular  database.


Fix/vote model str method

May be related to closed issue #27

thanks & thanks for including a test too 👍


Fix typos in changelog entry and javadocs.

Accidentally left "meta" in a few places.

Looks good to me.


resource apstra_telemetry_service_registry_entry - not all storage_schema_path values allowed?

I'm not clear on the rules which relate application_schema to storage_schema_path, but I'm having a problem creating one of  these resources:
resource "apstra_telemetry_service_registry_entry" "t" {
  service_name       = "foo"
  application_schema = "{}"
  storage_schema_path = "arp"
}

The error is:
╷
│ Error: error creating new TelemetryServiceRegistryEntry
│
│   with apstra_telemetry_service_registry_entry.x,
│   on main.tf line 46, in resource "apstra_telemetry_service_registry_entry" "x":
│   46: resource "apstra_telemetry_service_registry_entry" "x" {
│
│ {"errors": {"storage_schema_path": "Invalid choice aos.sdk.telemetry.schemas.arp"}} - http response '422 UNPROCESSABLE ENTITY' at
│ 'https://apstra-2bd238e9-290d-4f8d-9f3a-05b501becd14.aws.apstra.com/api/telemetry-service-registry?async=full'
╵

If I change arp to generic, this works okay. Only a handful of the validated storage_schema_path values seem to work. Is it because they're not valid, or because my (empty) application schema can't use that path?

I think this was resolved by #726 - please confirm @rajagopalans?


us-fl-desoto

Desoto County address points are here: https://arcgis.mobile311.com/arcgis/rest/services/Florida/Desoto/MapServer/1

Small sized Florida county


Cache NCBI BLAST+ 2.7.0 tar-balls for x86

I'm making another legacy IUC ToolShed package for BLAST+ 2.7.0 so for consistency would like the cache the source tar-balls here as previously.
Cross reference https://github.com/bioconda/bioconda-recipes/pull/6282 for the BioConda recipe for BLAST+ 2.7.0

Thank you :)


Update readmes with working examples

and alternative dashboard instructions

I think the content of this PR is no longer relevant. It's no longer possible to log in with creds from kubeconfig, since Kubernetes dropped basic auth support. The remaining docs have been moved to https://ubuntu.com/kubernetes/docs and significantly cleaned up.

I'm truly shocked this was closed ;) Thank you for cleaning up old PRs <3


ECNIM-618

ECNIM-618
Type

[ ] Bugfix 🐛
[x] New feature 🌈
[x] Change request 🤓
[ ] Documentation 📚
[ ] Tech debt 👩💻

Changes proposed ✔
<EMAIL_ADDRESS>🎉 New features

Added boxShadow, transitionDuration, transitionDelay, transitionTimingFunction and transitionProperty to Box component sprinkles. (#87 by @juanchigallego)
Added not-allowed and grab options to cursor property. (#87 by @juanchigallego)
Added disabled state to Box conditions. (#87 by @juanchigallego)

Screenshots 🏞
Related issues 🐛

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

Coverage data is based on head (6d00a00) compared to base (b06b591).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##            master       #87   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          286       286
  Lines         1667      1670    +3
  Branches       252       252
=========================================
+ Hits          1667      1670    +3

Impacted Files
Coverage Δ

...styles/src/packages/atomic/box/box.sprinkle.css.ts
100.00% <ø> (ø)

packages/core/styles/src/properties/css.ts
100.00% <100.00%> (ø)

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

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


don't crash if errors is a hash

I was using Her with an API that returns a hash for errors instead of an array:
[2] pry(#<Her::Errors::ResourceInvalid>)> @resource.response_errors
=> {:param1=>["can't be blank"], :base=>["duplicate enrollment pending"]}

lib/her/errors.rb crashed on .join() being undefined

@hubert is this one still worth merging?

@l8nite i think this responsibility falls to the parser setting up the data, meta, and errors in the expected way.


ReactNative: Error: 404 Invalid URL (POST /v1/chat/completions/)

Confirm this is a Node library issue and not an underlying OpenAI API issue

[X] This is an issue with the Node library

Describe the bug
The issue only happens when the OpenAI client is used in ReactNative code, specifically due to the following bug in ReactNative itself: https://github.com/facebook/react-native/issues/24428
Notice how the request is attempted to be sent to /v1/chat/completions/ (with a trailing slash).
It can be resolved by installing the https://www.npmjs.com/package/react-native-url-polyfill; however, it can also be resolved by trimming the trailing slash in the buildURL method here: https://github.com/openai/openai-node/blob/master/src/core.ts#L436
Something along these lines:
return url.toString().endsWith('/')
      ? url.toString().slice(0, -1)
      : url.toString();

To Reproduce

Try to send any message using the openai.chat.completions.create within a ReactNative app

Code snippets
const openai = new OpenAI({
  apiKey: process.env.EXPO_PUBLIC_OPENAI_API_KEY,
});

const chatCompletion = await openai.chat.completions.create({
        model: "gpt-3.5-turbo",
        messages: [
          {
            role: "user",
            content: "test message",
          },
        ],
      });

OS
macOS
Node version
Node v20.3.1
Library version
openai 4.11.1

React Native is not supported, as documented in the README. More details here.

See also:

https://github.com/openai/openai-node/issues/348
https://github.com/openai/openai-node/issues/270

React Native is not supported, as documented in the README. More details here.

Then What is the alternative process for React Native?

React Native is not supported, as documented in the README. More details here.

Then What is the alternative process for React Native?

I ended up building an openai react native client that supports file upload and chat completion streaming without polyfills by using react-native-sse and the expo file system. The library uses the official node sdk everywhere else along with the same types and API
https://github.com/backmesh/openai-react-native


feat: design a CR for operating and managing the stream sources inside the RisingWave

Currently, we have the RisingWave CR for managing the RisingWave instances in the Kubernetes. We can manipulate the RisingWave resources with kubectl. I'm thinking of abstracting other resources as CRs, such as the stream sources.
The stream sources should have the connection endpoint, the credentials, and other configurations supported by either the stream provider or the RisingWave. Then we can have a controller that syncs the stream sources with the RisingWave instance and reports the status.

Now it seems it's not a good idea to do this. It's just a direct dispatching of the parameters, so there's nothing different with doing it with SQL.


Queue and Cast disapear when window becomes too small

While using this theme I have noticed that the Queue and Cast buttons disapear when the window gets smaller. This appears to be specific to this theme as I have not been able to recreate this behaviour with the stock appearence.

yes this is intentional.


OSX install issues.

OSX: 10.11.6 (15G1004)
node: v6.9.1
npm: 3.10.8
$ npm install
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

$ npm start
><EMAIL_ADDRESS>start /Users/daicoinbase/station
> electron src/index.js

App threw an error during load
Error: Cannot find module 'babel-runtime/core-js/object/assign'
    at Module._resolveFilename (module.js:455:15)
    at Function.Module._resolveFilename (../station/node_modules/electron-prebuilt-compile/node_modules/electron/dist/Electron.app/Contents/Resources/electron.asar/common/reset-search-paths.js:35:12)
    at Function.Module._load (module.js:403:25)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (../station/node_modules/electron-prebuilt-compile/node_modules/electron-compile/lib/index.js:3:15)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)

Hmmm, weird. I tried it again in a different location and now it works. (The very first time I tried to install I had an older version of Node, so that was probably it - it got confused). Closing.


MSL: Support inline uniform blocks in argument buffers.

Here, the inline uniform block is explicit: we instantiate the buffer
block itself in the argument buffer, instead of a pointer to the buffer.
I just hope this will work with the MTLArgumentDescriptor API...
Note that Metal recursively assigns individual members of embedded
structs IDs. This means for automatic assignment that we have to
calculate the binding stride for a given buffer block. For MoltenVK,
we'll simply increment the ID by the size of the inline uniform block.
Then the later IDs will never conflict with the inline uniform block. We
can get away with this because Metal doesn't require that IDs be
contiguous, only monotonically increasing.

Is this urgent? I'd like to look at this after the holidays.

My main worry here is how we assign data to ids. For a pipeline layout to work in MoltenVK, we need to be able to assign a UBO blob to a single ID, irrespective of the underlying data type in the shader. If Metal requires us to assign data per sub-ID (one call for each vector) in the argument buffer, I don't think it makes sense to support this feature in SPIRV-Cross, but I don't have an Apple device handy until next year.

From the docs for -[MTLArgumentEncoder constantDataAtIndex:]:

Constants declared contiguously in the Metal shading language (in an array or struct) are contiguous in memory. You can encode contiguous ranges of inlined constant data through a pointer to the first constant.

So this should be OK.


Forwarded email missing body

Hey guys - cool toys :)
So, I have mac mail, and I'm forwarding an email into sendgrid/griddler, and the body is just:
> Begin forwarded message:
>
>

From this email:
(quite a bit of good valid html)

What are people doing when they want to get at the whole thing, like in this case?

Should be in raw_body

On Apr 9, 2015, at 9:11 PM, Troy Anderson<EMAIL_ADDRESS>wrote:
Hey guys - cool toys :)
So, I have mac mail, and I'm forwarding an email into sendgrid/griddler, and the body is just:

Begin forwarded message:

From this email:
(quite a bit of good valid html)
What are people doing when they want to get at the whole thing, like in this case?
—
Reply to this email directly or view it on GitHub.


[Sanitizer] Symbolication test no inline

This test checks that Sanitizer reports contain properly symbolicated
stacks.  Let's make sure that all of the expected frames appear in the
stack by disabling inlining.
rdar://68171463

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 0d30acadb996c87bb8106ca2f3be00bb82a28030

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 0d30acadb996c87bb8106ca2f3be00bb82a28030

Build failed
Swift Test OS X Platform
Git Sha - 0d30acadb996c87bb8106ca2f3be00bb82a28030

Swift Test Linux shows up as failed, but I don't know why.  All tests pass, including ours:
PASS: Swift(linux-x86_64) :: Sanitizers/symbolication-linux.swift (9337 of 13664)


How to document object params with particular properties

I think I must be missing something here but I cannot figure out how to properly document object parameters (with specific properties). This is some documented code using jsdoc3:
/**
 * Transition the globe from its current position
 * to the new coordinates.
 *
 * @param  {Object} pos - the position
 * @param  {Float} pos.lat - latitude position
 * @param  {Float} pos.lon - longtitute position
 * @return  {this}
 */
api.center = function(pos) {
  target = calculate2dPosition(pos);
  return this;
}

Will output:

Globe.center(pos, pos.lat, pos.lon)
Transition the globe from its current position to the new coordinates.
Parameters
pos: Object, the position
pos.lat: Float, latitude position
pos.lon: Float, longtitute position
Returns: this

The first line is what I am not really happy about (this function only has a single parameter which is an object). After looking at the code it seems like something that is easily fixed by filtering the params before constructing the paramsString, but I am pretty sure that I am missing something obvious here.

@askmike I don't think you are missing anything.  This part of the spec wasn't supported yet, and your patch looks good, although I'm not quite sure why the tests failed on the ci build.  Will look into it.


Support for Meteor 1.4, faster compilation, caching

Major refactor in the way that node and npm are handled by the buildpack.  Rather than installing our own nodejs version (pinned to 0.10.x), use the version that is distributed with Meteor.  Locate and copy that nodejs binary onto the built slug on production to retain small slug sizes without dragging in the whole meteor world.  Addresses #120 and #125.
Rather than installing the latest Meteor release and then "upgrading" (or letting Meteor downgrade on first-run), use https://install.meteor.com/?release=$APP_RELEASE to install the correct version from the beginning.  This helps toward #64.  (Thanks @tab00).
Install Meteor on the Heroku $CACHE_DIR so that the version can be retained between builds.  This helps further with #64.

Great work!
Is there a possibility to include something like #96 ?
Thanks!

Sure.  I added $BUILD_OPTIONS to the invocation of meteor build, which you can define with --debug.  Thanks for suggesting.


Regression on stopping the server

"For every web template when the server is started with dnx . kestrel I used to be able to hit q or CTRL+C. The only way I've found to stop the server is with CTRL+Z but that keep the port in use. So on the next usage of dnx . kestrel you get a port in use error. More info at https://github.com/OmniSharp/generator-aspnet/issues/191#issuecomment-113790859."

@halter73 Can you have a look for Beta5?  If we can have a contained fix in Kestrel package only, we should be able to take this in Beta 5.
cc @danroth27 @DamianEdwards

#9?
Also note that CTRL+Z merely pauses the process and puts it in the background. From there, you could work with it using the bg and fg command line tools. It's natural that the port isn't released as the process continues to live.

@Tragetaschen I think #9 is a different issue. This was working on Mac in beta3/4. #9 was filed over a year ago.

The reason this becomes visible now is that Hosting was changed. Up until beta4, the clean way to exit the process was to press Enter (Console.ReadLine). Now with beta5, this was changed to CTRL+C (Console.CancelKeyPress).
Prior to beta5, CTRL+C would kill the process without proper cleanup. Pressing it was using the equivalent of Environment.Exit and at most the finalizers of the objects would run (if at all). Now, with beta5, CTRL+C shuts down the program using the expected code paths and properly calls Dispose on all objects including the Kestrel ones. This makes this bug visible again—it remained unfixed all the time.

@Tragetaschen ok I see. Let's see what @halter73 finds out in his investigation.

I think @Tragetaschen is right.
For now, workaround would be to hit Ctrl-Z to send the process to the background, and than kill it via command line (or with Activity Monitor on a Mac)

Just to be clear: The PR mentioned near the end of the #9 discussion (#60) has never been merged? Would it be relevant if rebased on dev?

OnStopRude along with @Tratcher's fix in Hosting fix this issue

This is going in beta 6


[PL-48480] SMP 0.15.0 Release Notes

https://harness.atlassian.net/browse/PL-48480
https://harness.atlassian.net/wiki/spaces/PD/pages/21709848595/Release+Notes+tickets+for+v0.15.0+release
Artifact vernos pulled from: https://harness.atlassian.net/wiki/spaces/PD/pages/21691498500/SMP+Release+Sign+off+March+-+2024+-+build+0.15.0
For reviewers, a preview is available:
https://6601c66f829f002e37c64af9--harness-developer.netlify.app/release-notes/self-managed-enterprise-edition#march-xx-2024-version-0150
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screenshot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65fc88250168fc617fa1aa14--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://660062847fedff02815436b4--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6601c66f829f002e37c64af9--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6601d08f151dd8326ec3f965--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6603125b4935117864f0cc0f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://660313811392da7d82a748e2--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66031cce8fb0ec82348fbc19--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66045927101d562a7450a63e--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6605ffb0c09bc55b2598bf91--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6606004de4397f17486814ae--harness-developer.netlify.app

Has an AIDA items with STO, can we please review for any other AIDA items.

@pratmit Removed AIDA items. Thanks.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6606f79ab18ed82e3b608eb2--harness-developer.netlify.app


CI: no logs from fail on BeforeEach/BeforeAll

Build link: https://jenkins.cilium.io/job/Ginkgo-CI-Tests-Pipeline/2751/testReport/junit/k8s-1/7/K8sValidatedKafkaPolicyTest_KafkaPolicies/
Logs: f0c5341e_K8sValidatedKafkaPolicyTest_KafkaPolicies.zip
There is no bugtool output, or anything related to Cilium for further triage. Fix the offending test to correctly gather logs / any potential Ginkgo framework bugs.

The issue, is the Fail happens in Before* functions, no After* functions are in place yet. I need to find a workaround for this one.

BTW: kinda hard to fix this for 1.1 release, need to say that it'll be super difficult to fix without changes in Ginkgo upstream.

Not necessarily; we can make assertions within the Its  instead?


Feature/monitr image copying

You can now copy images from monitr with a right click. Dragging images out is still needs to be implemented.

@marcosfrenkel same as the inspectr PR. can you check?


Uses ACL token when get servers.

Uses ACL token when get servers.

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

@bomee Thank you for signing the Contributor License Agreement!

The feature depend on consul server and an acl_rule, i pass the code in my environment

We have existing tests that use ACLs see https://github.com/spring-cloud/spring-cloud-consul/blob/master/spring-cloud-consul-discovery/src/test/java/org/springframework/cloud/consul/discovery/ConsulDiscoveryClientAclTests.java and https://github.com/spring-cloud/spring-cloud-consul/tree/master/src/test/resources/consul_acl

Thanks.I refer to the existing test to complete the test.


Documentation links don't resolve on pypi.org

The main readme on pypi.org found at: https://pypi.org/project/openinverter-can-tool/ has links to additional documentation for CAN message mappings and shell completion. The links are bogus as only the README.md is published there.
These links work fine on github.

Fixed in the doc updates for v0.2.0


ui text box - when user clears the data - no event gets generated.

Current Behavior
I write something in ui_text box - once i leave / release focus - event gets generated - with payload of what user typed.
I deleted the data in ui_text box - no event gets geenrated. So user has no way to know the new content of ui_text
Expected Behavior
I write something in ui_text box - once i leave / release focus - event gets generated - with payload of what user typed.
I deleted the data in ui_text box - event will empty payload (undefined/empty/null - any is ok) should get generated.
Steps To Reproduce
No response
Environment

Dashboard version:1.11
Node-RED version:3.1.9
Node.js version: 20.10
npm version:10.2.3
Platform/OS:Windows
Browser:Chrome

Have you provided an initial effort estimate for this issue?
I have provided an initial effort estimate

If I recall, @colinl had opinions on this at some point, as we implemented it as you've described. I'm open to a change to still emit if no value though, especially since we've added control over when the events are emitted (on blur, etc), the user has more control

Once user add a value and event got generated - it would get used in a flow.
Then user decided to delete this value - the backend editor would have no ways to know that now the value is empty.
We may not be able to do validations that user did not input a value - as editor has event to work on.


gvim (windows) freezes when opening a file already openend in another Vim (with "simalt ~x" in GUIEnter)

This was introduced somewhere between 8.0.69 and 8.0.586 and can still be reproduced in the latest nighlty. Last tested with 1159.
To reproduce:

use a blank _vimrc with just "au GUIEnter * simalt ~x"
gvim test.txt
:w
leave the instance open and start another vim: gvim test.txt
-> The second instance is opened and freezes (uses CPU and does not respond to keyboard or mouse input)

Use case for using two instances of vim on the same file: Do some coding in a source file in the first instance and view the local changes in a source control system (which launches a gvim -d file_in_working_copy pristine_copy_of_file)

Works fine, thanks.
$VIM now points to vim\vim81 instead of vim as in former versions (last tested with 8.0.1358). I don't now when this was introduced and if it was done intentionally, though.


Creating PathPlanner Autos

create autos for the autonomous mode

[ ] 3 piece: start from middle, pickup and shoot left & middle
[ ] shoot preload game piece, pick up the left top note, shoot, pick up left middle, shoot, pick up left bottom, shoot
[ ] shoot, pick up left top, shoot, pick up right first (up to bottom),pass the line, shoot, pick up right 2nd,pass the line, shoot, pick up 3rd, pass the line, shoot
[ ] shoot, pick up-shoot left bottom, pick up-shoot middle left, pick up-shoot top left, pick up top right, pass the line, shoot pick up and do the same with the next 2 pieces from top to bottom
[x] do nothing

Merged into Dev


EXPORT and CDC file naming conventions and best practices

CDC changes:

Created new include files for external URL content. Only updated the Example file URLs at the bottom.
Added Files section to describe the naming conventions of the files that are emitted.

EXPORT changes:

Added notes about giving each export a unique destination to avoid overwriting other exports.

Closes #7374.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/049a29d4733e83bf8d0ce9ae6d69eb712e8ee03d/
Edited pages:

v20.1/create-changefeed.md
v20.1/export.md
v20.2/create-changefeed.md
v20.2/export.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2490af7670955680820a4926ef21e20b2fbeaf64/
Edited pages:

v20.1/create-changefeed.md
v20.1/export.md
v20.2/create-changefeed.md
v20.2/export.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/837ec5ebf308cfd0c0348fc54c1b9f57d57ab7d8/
Edited pages:

v20.1/create-changefeed.md
v20.1/export.md
v20.2/create-changefeed.md
v20.2/export.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f20de2b7be865d764cf7df35f70d716eea1ba05d/
Edited pages:

v20.1/create-changefeed.md
v20.1/export.md
v20.2/create-changefeed.md
v20.2/export.md

LGTM

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/fa9c9e147f08bf12eec6fc3205ff13fc66dc9da2/
Edited pages:

v20.1/create-changefeed.md
v20.1/export.md
v20.2/create-changefeed.md
v20.2/export.md


[Windows] Unknown error during installation

See https://github.com/learningequality/installers/pull/335#issuecomment-173339645 -- however, we're not able to replicate that error at this time.

This maybe be too optimistic, but as I read the comments, in #335 this is likely related to the server already running, so if we solve #351, this might go away.
In any case, it's related to an older installer.


Fix typo in summary of interface Registration.

There seems to be an extra "to" in the annotation for the interface Registration.

@CXuesong,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks.


GEODE-9434: add ZREMRANGEBYLEX command

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

I've seen an identical stress test failure to the one in zRemRangeByLexRemovesMembersFromSortedSetAfterPrimaryShutsDown() while working on GEODE-9435, so I think there may be a common issue in what the test is doing, possibly due to multiple tests running in parallel accessing the same key.


Component's type is an object rather than a string when not using jsx

import dom from 'dekujs/virtual-element'

let Form = dom('form', {}, [])

export function render () {
  return dom(Form, {}, [])
}

The above will return a vnode with type as an object:
{     attributes: Object,     children: Array[0],     type: Object }
The type object looks like: {propTypes: {transform: {type: function}}}
I believe type should equal form.  Am I using components incorrectly?

I was misunderstanding the API.  Closing this.


Resample option when pulling data through the API does not behave correctly

For example, when pulling hourly data from the api and resampling to 15min, the process fails to produce the desired result. Instead, the dataframe returned is of the same original dimension with only the local_start_times field modified.
For example, in the image below the interval_end_local column is changed to end at 15 minutes, but the actual lmp data is not resampled. The expected behavior is that for upsampling the more coarse data is repeated the required number of times to match the desired resolution. Additional functionality may be included for smoothing transitions but for this specific data (Locational Marginal Prices), repetition is the right approach.

Based on a quick glance at the source code, I assume downsampling will be okay.

Hey @ebalogun01 could you post the code you are running so we can look into this? Thanks!


Database schema has many extra unused fields

Describe the bug
Database schema for transfers is full of fields that are no longer used.
To Reproduce
See schema.sql.
Expected behavior
Schema should reflect fields that exist for transfers.
Version:
Additional context
Check all schemas, not just transfers.
Screenshots
If applicable, add screenshots to help explain your problem.

Update the issue with the actual extra fields

This is still needed.

@preethamr what is your expected result? Is it still relevant? We need more details.


Use indirect build images and dnf roots

This reduces the images as follows:

gateway: 243MiB → 174MiB
globalnet: 213MiB → 81MiB
networkplugin-syncer: 255MiB → 189MiB
route-agent: 212MiB → 80.7MiB

We use minimal dependencies and a separate build environment with a
dedicated installation root to reduce the number of required packages
and remove the support infrastructure from the target images.
See https://www.youtube.com/watch?v=HuE2Lrf5mRg for details.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

Can we still login and run bash / ipsec calls after this?
If the answer is no, I propose we wait for 0.10 to start,
If the answer is yes, I'd be fine to merge now

Yes, we still can:
$ docker run -it --rm --entrypoint /bin/bash quay.io/submariner/submariner-gateway:dev
bash-5.0# ipsec
Usage: ipsec {command} [argument] ...>
Use --help for a list of commands, or see the ipsec(8) manual page
Most commands have their own manual pages, e.g. ipsec_auto(8).
See <https://libreswan.org> for more general info.
Linux Libreswan 4.3 (netkey) on 5.10.16-200.fc33.x86_64
bash-5.0#

The only feature this patch removes from the images is the ability to use dnf inside them.

@mkolesnik I’d appreciate knowing what you think of this!

Please don’t merge this before @mkolesnik gets a chance to review it.

Please don’t merge this before @mkolesnik gets a chance to review it.
Ack, I thought it was ok after 4 days , but it's true we had a weekend in the middle.

@mkolesnik, any chance you could take a look at this?

bump

Perhaps its best to build the base once (and publish it) and each image will rely on that?

I’ve updated the PR to do this. I still need to tweak it to share the dnf cache.

This is still relevant.


Integrating generic connection-string

Implements #13 (which in turn fixes #14 + #23), to close all open issues as of today. Plus it fixes all tests to work on Windows.
This requires a major version bump, and the PR also upps the version to 3.0.0.
All syntax updated to ES6, since even the driver doesn't support ES5 anymore.

Many tests now can be retired, because this library doesn't do the parsing anymore. But for now I kept as many tests as I could.
This implementation uses connection-string underneath, while implementing the functionality as close as possible to the original, and to keep it mostly compatible with the existing driver.
This first version ignores multi-host support, but makes it very easy to activate, once the driver starts supporting it, since connection-string supports it out-of-the-box.
However, some functionality had to be changed, and below is the list of all breaking changes...

Unix sockets support is now much simpler: the host name either ends with .sock, or contains / after decoding.
Special symbols in the user name or password need to be URL-encoded, the same as any other part of the URL, as it must be compliant with the URL Connection String standard.
Since database now always comes from the first path segment, there is no more need in custom parameter db. In the simplest form, /dbname results in the database set correctly.
Minor changes as to the short-syntax expectations now reflect what the URL spec requires. For example, passing in just some-name sets the host, not the database name. For the database shortest syntax it is /some-name, and for just host + database, it is host/database. See more examples here.

For other references, see connection-string. It supports abbreviations, defaults, string generation, etc, many nice features.
UPDATE
I have added a good usage example - connection-string-demo.

@monteslu Do you think this is possible? :smile:

Will review, is this still work in progress?

is this still work in progress?

@abowerman nope, it is all done and good to go! :)

@abowerman I have made some important improvements as to how Unix sockets are supported, both here and in the connection-string. It is so much simpler, and more logical...
Any host name that either ends with .sock or contains / after decoding is recognized as a Unix socket.

I have made some important improvements as to how Unix sockets are supported, both here and in the connection-string. It is so much simpler now, and more logical.

@vitaly-t is this a breaking change?

@hjr3 Considering that node-postgres is still stuck with v0.1.3, there won't be any progress, unless we make it :wink:

@abowerman Is it ok to merge it now?

Looks like you are still making changes here.

@abowerman I've been making improvements to the library, and here only updating dependencies, nothing else. This shouldn't stop you from merging the PR ;)

@abowerman Just checking back, as it's been 12 days since...

@abowerman just pinging ;)

How come we lost all the response here after...?
@hjr3 @abowerman Could someone, please review and merge it?

Am reluctant to introduce dependencies and breaking changes here. This is fairly heavily used and currently does not have any dependencies. @hjr3 @monteslu Thoughts?

For one thing, it is marked as version 3.0.0, within the PR itself. And for another, the driver has been stuck on the ancient version anyhow. So it won't break anything, if we just follow the major version increment.

This change guts the code base and makes pg-connection-string a thin wrapper around connection-string. The code is looks fine, but there is something that just feels wrong about it.
This is an old repository. It is depended on by a lot of people. The repository itself has a reputation behind it. There is a (loose) governance model. By merging this PR, then we give all that up. Any bug or feature request will have to be redirected to connection-string. The go-forward role for the current maintainers will only be bumping the version of connection-string periodically.
Maybe connection-string is the better option though! @vitaly-t is very motivated to improve the current state of connection string parsing and this repository has been pretty quiet. If connection-string is the better option, we should probably deprecate this repository and redirect people to connection-string. No reason to make a 3.0.0 version of this repository.
My vote is to update the README that this repository is deprecated, the .2.x.x branch will receive security updates only and people should start migrating to connection-string.

@hjr3 The important thing, issues around the lack of code consolidation keep rising, but nothing happens. Here's the last one, just from today, that's why I'm trying to revive it again.

Yup, I hear you. I am all for node-postgres updating its dependency to be on connection-string instead.

As part of moving all my projects toward native TypeScript implementations, I've just published a pre-release update for this one - 3.0.0-Beta.0.

@abowerman, @hjr3
Updated to use the new 3.x version of connection-string, re-done in TypeScript.

Hello fellow. Am I right that better to migrate node-postgres to connection-string and keep pg-connection-string as is?
I have some time to work on it and deciding what to do

@zxqfox For the lack of the progress either way, to move toward a better generic approach, I no longer want to advocate any kind of integration. For people who want to use connection-string, they can just do so independently, and generate a connection object from it to be passed into the driver, as shown within the Adapters.


https://github.com/chitdayaung/chitdayaung.git

https://github.com/chitdayaung/chitdayaung.git

y

y

https://github.com/chitdayaung/chitdayaung.git

ok


Elements: 9 patch implementation

Hi guys, and specifically @daredevildave and @guycalledfrank
This issue is to discuss 9 patch implementation from https://github.com/playcanvas/engine/pull/816
The way I did (in fragment shader) has a reasoning behind it: instead of maintaining a mesh with 9 quads we use the source mesh (which can, by the way, be of any shape – consider tightly packed sprites which are not quads) without modification and ONLY update 1 uniform (being pc.Mat4) whenever the resolution or the size changes. This seems to be a huge performance benefit as updating the uniform is way faster than reuploading the whole mesh – and even you still have to update the mesh, updating 1 quad is quicker than 9 :)
And this, in fact, matters: certain UI elements get animated a lot (hover states, movements etc) meaning the image will be updated every frame. And in this case changing a uniform seems to be the way to go.
The last but not least: keeping this uniform in StandardMaterial makes it super-easy to apply 9 patches to anything outside UI elements (i.e. drawing sprites in the scene).
Looking forward to hearing your feedback.

P.S. Another thing that is quite easy to implement in the shader and much more difficult to achieve with 9 quads is tiling mode: when the center of the image gets tiled instead of being scaled (comes in useful for certain types of artwork)

Hi guys (@guycalledfrank and @daredevildave),
Just a kind reminder about the existence of this thead :)


Fixing crash on release version

Motivação
The release version of the app was crashing due to a bug in a dependency, which is facebook's fresco animation framework. The debug version worked fine, but when released the app would crash when opened.
Mudanças feitas
Changing the version of the library to a stable one.
Tarefa relacionada

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
1573

💛  - Coveralls


Collect All PySEIR output in snapshots

Currently only output/web_ui is shipped to the snapshots.
For QA purposes and additional introspection, we would like to also collect pyseir output reports and data.
Specifically a zip of the state_summaries folder is ideal as is a concatenation of all PDF files in that folder to a single PDF that can be used for rapid QA.

I kicked off https://github.com/covid-projections/covid-data-model/actions/runs/80176747 which should generate a snapshot with a pyseir_state_summaries.zip file in it.  Will update here once it's finished.
I'll look into generating the concatenated PDF separately.


Option to display only monitors that are down

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
I have more than 20 monitors, I'd like to see only those that are down on the dashboard view by selecting the appropriate filter.
✔ Solution
Offer a filter or a checkbox, maybe in the left panel, that makes it possible to filter out monitors that are up.
❓ Alternatives
No response
📝 Additional Context
No response

Would be great to have this feature

in the status page I use css override.

oh I didn't think of this. I guess I could use greasemonkey too (?)

Well it is better to see them on top of list...

This issue seems very similar to #1585 which has an open PR (#1971).
@Piste if you agree, please close this issue or edit the issue to make it more distinct  ^^
(duplicates don't help to make the ammount of issues more manageable)
@MaxPowers1337  @ozgurerdogan  @wordpresshosters
Please refrain from posting +1 / requests for updates things on issues, as this makes issue-management harder.
Issues are for discussing what needs to be done how by whom.
We use 👍🏻 on issues to prioritise work.

Thanks for the pointer, @CommanderStorm. Super helpful.
Closing in favor of https://github.com/louislam/uptime-kuma/issues/1585


Fix branch.

@ostcar Can you please fix the validator and enable the HTMLVideo type?  Thanks.

What does through do?

If you have a structured field you have to define how to find the content of this field. E. g. in an poll option the placeholder represents a meeting id which can be find by following the foreign key poll_id from option to poll where you then have the field meeting_id.


Build assets in shopware bundles

Please describe the feature you would like to see implemented.
As described here https://shyim.me/blog/you-dont-need-a-plugin-to-customize-shopware-6/ you can also have a bundle which contains assets to build.
At the moment, you're only looking for the composer type shopware-platform-plugin which is not the case for a bundle (symfony-bundle).
https://github.com/FriendsOfShopware/shopware-cli/blob/da3c091602da9ead97fac0c8cd46d6962465b3f5/extension/project.go#L84
Maybe you can introduce some kind of extra value that you're looking for which can also indicate for other composer package types that they contain assets?

We could check if that package contains a .shopware-extension.yaml 🤔

duplicates #201 ?

Or have an additional list of folders in the .shopware-extension.yaml?
I'm not sure which solution would be best suited.

I close this as it's a duplicate of https://github.com/FriendsOfShopware/shopware-cli/issues/201


Create model with enum array throws: no such table

prisma2 version<EMAIL_ADDRESS>binary version: 30a33bde5b1d9f79e0a48491749528da76002e24
Following schema.prisma contains roles an enum attribute as array.
datasource db {
    provider = "sqlite"
    url      = "file:../db/data.db"
}

generator photonjs {
    provider = "photonjs"
}

model User {
    id    Int    @id
    email String @unique
    roles Role[]
}

enum Role {
    ADMIN
    USER
}

Seed file:
import Photon from '@generated/photon';

const photon = new Photon();

async function main() {
  const user1 = await photon.users.create({
    data: {
      email<EMAIL_ADDRESS>      roles: {
        set: 'USER'
      }
    }
  });
  const user2 = await photon.users.create({
    data: {
      email<EMAIL_ADDRESS>      roles: {
        set: 'ADMIN'
      }
    }
  });
  console.log(user1, user2);
}

main()
  .catch(e => console.error(e))
  .finally(async () => {
    await photon.disconnect();
  });

The following error occurs when executing ts-node prisma/seed.ts:
Error: Error in Photon:
ConnectorError(QueryError(SqliteFailure(Error { code: Unknown, extended_code: 1 }, Some("no such table: data.User_roles"))

stack backtrace:
   0: backtrace::backtrace::trace::h5511721bea6fa8d5 (0x1032bb9de)
   1: backtrace::capture::Backtrace::new_unresolved::h28f9c9663a9420c8 (0x1032bab38)
   2: failure::backtrace::internal::InternalBacktrace::new::hbb41405b6d19ab81 (0x1032ba4a9)
   3: <failure::backtrace::Backtrace as core::default::Default>::default::h0458ad281d4247c8 (0x1032ba695)
   4: <sql_connector::error::SqlError as core::convert::From<rusqlite::error::Error>>::from::h56a3e12576e2ad6d (0x102be48cc)
   5: sql_connector::database::sqlite::<impl sql_connector::transactional::Transaction for rusqlite::transaction::Transaction>::write::hcbd321a4ccfcd7cc (0x102bcda36)
   6: sql_connector::transactional::Transaction::insert::h9f2b434c25edfa65 (0x1027cb8d4)
   7: sql_connector::transactional::database_writer::create::execute::h2c63144580998c8d (0x102bf954d)
   8: sql_connector::transactional::database_writer::<impl connector::database_writer::DatabaseWriter for sql_connector::database::SqlDatabase<T>>::execute::{{closure}}::create::hef864a06474fbea9 (0x102c36a07)
   9: <sql_connector::database::sqlite::Sqlite as sql_connector::transactional::Transactional>::with_transaction::h139aada801227fe5 (0x1027c17d3)
  10: sql_connector::transactional::database_writer::<impl connector::database_writer::DatabaseWriter for sql_connector::database::SqlDatabase<T>>::execute::hb31d5b14b577d4b8 (0x10284d486)
  11: core::executor::write::WriteQueryExecutor::exec_one::h48f2e807cc785823 (0x102ce7006)
  12: core::executor::write::WriteQueryExecutor::execute::h992b6a51be508cce (0x102ce695b)
  13: core::executor::Executor::exec_all::h5cbc538ae21b3c80 (0x102d05aa4)
  14: <prisma::req_handlers::graphql::GraphQlRequestHandler as prisma::req_handlers::RequestHandler>::handle::h7d76032243e11158 (0x10294d9ff)
  15: prisma::http_handler::h5782a9905407eb5f (0x1028e838f)
  16: <F as actix_web::with::FnWith<T,R>>::call_with::haa4a1a2255bc8611 (0x102945693)
  17: <actix_web::with::WithHandlerFut<T,S,R> as futures::future::Future>::poll::hd3a8444797502a6e (0x10294ba12)
  18: actix_web::pipeline::PipelineState<S,H>::poll::h147d5d5368789f0e (0x102825b06)
  19: <actix_web::pipeline::Pipeline<S,H> as actix_web::server::handler::HttpHandlerTask>::poll_io::h39e2cc76fc5e8dd4 (0x102818fd0)
  20: actix_web::server::h1::Http1Dispatcher<T,H>::poll_handler::hf7d43112cc4ed6f5 (0x1027eee0a)
  21: actix_web::server::h1::Http1Dispatcher<T,H>::poll::h9c1da5bf8f5108dd (0x1028be063)
  22: <actix_web::server::channel::HttpChannel<T,H> as futures::future::Future>::poll::had9f1b0a81e71dda (0x1028bcf39)
  23: <actix_web::server::channel::HttpChannel<T,H> as futures::future::Future>::poll::had9f1b0a81e71dda (0x1028bddf3)
  24: <actix_net::service::and_then::AndThenFuture<A,B> as futures::future::Future>::poll::h15ac475c98adc942 (0x1028178c6)
  25: futures::future::chain::Chain<A,B,C>::poll::hb6e62549216884f5 (0x102813b44)
  26: futures::task_impl::std::set::he4a7d579f87ab972 (0x102a2f5bd)
  27: futures::task_impl::Spawn<T>::poll_future_notify::h1a5228822adb903e (0x102a485cf)
  28: tokio_current_thread::CurrentRunner::set_spawn::h6965d5b16e375442 (0x102a29534)
  29: tokio_current_thread::scheduler::Scheduler<U>::tick::h6c1934358ae3f6a8 (0x102a358c5)
  30: tokio_current_thread::Entered<P>::block_on::he3da6e046995d1e4 (0x102a2a0ea)
  31: std::thread::local::LocalKey<T>::with::h6ed9ac0420a84d3b (0x102a407c4)
  32: std::thread::local::LocalKey<T>::with::h71f7d0c12da0393f (0x102a40d07)
  33: std::thread::local::LocalKey<T>::with::h10c82d1197b9368d (0x102a3e9ec)
  34: std::thread::local::LocalKey<T>::with::h38bd3e3c53efd4ea (0x102a3fec8)
  35: tokio::runtime::current_thread::runtime::Runtime::block_on::hbad87c088a1c5a72 (0x102a2d952)
  36: std::sys_common::backtrace::__rust_begin_short_backtrace::h690508ce542785d7 (0x102a46168)
  37: std::panicking::try::do_call::h1536644c0a3cab79 (0x102a2bcbb)
  38: __rust_maybe_catch_panic (0x1032ec4cf)
  39: core::ops::function::FnOnce::call_once{{vtable.shim}}::h3508efbc713fc3ba (0x102a469f6)
  40: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once::hf8766009c029bc09 (0x1032d36ce)
  41: std::sys::unix::thread::Thread::new::thread_start::hfdef4649a42cf26c (0x1032ebace)
  42: _pthread_body (0x7fff6cd792eb)
  43: _pthread_start (0x7fff6cd7c249)))
    at PhotonFetcher.<anonymous> (/Users/homer/wip/node_modules/@generated/photon/index.js:37:23)
    at Generator.throw (<anonymous>)
    at rejected (/Users/homer/wip/node_modules/@generated/photon/index.js:5:65)
    at process._tickCallback (internal/process/next_tick.js:68:7)

I can confirm. Currently lift is not creating a column and a join table for scalar enum lists.
While String[] is working in the current alpha 2.0.0-alpha.36, Role[] is not creating a column.

Any updates on this issue? 🤔

Ok, this is fixed now with<EMAIL_ADDRESS>binary version: 4d17a0639165e5a8e54c7e5cf538ffdf114edb53
You might need to recreate the database

Cool, thanks works for me as well.


MavenSupport: Add cache for remote artifacts

Store remote artifacts in a disk cache backed by DiskLruCache [1]. This
brings a big performance boost for Gradle, Maven and SBT dependency
resolution.
Currently the max cache size is hardcoded to 1GB, this will be made
configurable in a later commit.
[1] https://github.com/JakeWharton/DiskLruCache

This change is

Could you elaborate in the commit message why this makes sense? Naively thinking I would have assumed Maven artifacts get already cached by Maven itself in the standard local Maven directory. And even if they would not get cached implicitly, I would have assumed "turning on" the feature to cache in Maven-Local is less work than implementing an own cache with DiskLruCache.