doesn't work when other Dateformat

example:

does work
<datetime format="DD/MM/YYYY" width="300px" v-model="val"></datetime>

does not
thx

@08-15at Could you please explain what does not work properly?

uhh, sorry,
there was a line missing:
standard Date format

works,
as soon as I try to reorder day and month to european style format the complete Browser Tab freezes.

Have to close and reopen the tab.
to be sure, I'm not passing wrong values in val I set this to
val: "05-05-2018 00:00:00"
I'm using date-fns functions in my code, too installed version is<EMAIL_ADDRESS>thx for your help

@08-15at Please confirm if the issues is fixed now.

works perfectly now, thank you very much for your fast fix.


too slow, whats the problem?

I made a simple empty test on my 15 retina mac, (8gb) but is so slow...
could you explain about it?
Thanks!
1 server
1 client
1 thread
for i := 0; i < 200000; i++ {
       nc.Request("test", []byte("ok?"), Time1s)
}

1 subscriber
nc.QueueSubscribe("test", group, testsimpleempty)

subscribe function
func testsimpleempty(m *nats.Msg){
    nc.Publish(m.Reply, []byte("OK"))
}

and here is...
1887-2600 request per seconds:)
https://gyazo.com/e72b8f976bd15241ba8025787437fd31
how to get 30k, 100k, 1m?

server run via docker, maybe so big overhead?

Request() is a request reply, which means a round trip with the server. The number seem low, but still, a request/reply is a synchronous operation and not going to perform as well as simple publish or subscribe.
A single stream of request/reply will be bounded. To improve performance you would need to increase the number of processes/go routines/threads/etc. doing request/reply in parallel.

thank you!


Make AwaitableInfo readonly struct

This was proposed during a PR review: https://github.com/grpc/grpc-dotnet/pull/48#discussion_r252778441.
cc @JamesNK

Note that we need to update this in https://github.com/grpc/grpc-dotnet as well

https://github.com/aspnet/AspNetCore/pull/7169

Close?


wWBTC-BTCtz / wUSDC - USDtz / wUSDT - USDtz

Swap between wWBTC to BTCtz

Adding pairs to the request


🐝 Feature request: root.go declusterify

Is your feature request related to a problem? Please describe.
root.go for goji is a bit clustered needs simplification
Describe the solution you'd like

migrate functions to other files in the config dir
simplify and cleanup the commit logic

Describe alternatives you've considered
No response
📖 Additional contex
No response
🧨 Issue Checklist

[X] I have checked for other similar issues
[X] I have explained why this change is important
[X] I have added necessary documentation (if appropriate)

fixed in #26

fixed in #26


Idea! different configs depending on git branch

It would be cool to be able to specifiy a different upload location depending on the checked out git branch.
A bit of background:

As it may contain passwords and is specific for each dev, .remote-sync.json is normally in .gitignore, right?

When I switch branch, I may have different sync areas, like dev branch to dev server, master branch to live server

But this doesn't change automatically, so there's a risk that you switch branches and deploy to the wrong target

A possible approach:
Allow users to enter a key with the branch's name, like this:
{
  "key": "master",
  "uploadOnSave": true,
  "username"<EMAIL_ADDRESS>
and
{
  "key": "dev",
  "uploadOnSave": true,
  "username"<EMAIL_ADDRESS>
remote-sync would then automagically read the current branch's name and use that config.
What do you think? Or is there an existing solution for that use case already?

I'm searching for exactly the same problem/solution.
It seem too simple for no implementation at all. I guess there's something hard on read git branch, or there's a simpler solution i don't know. Please help us.

a workaround is to use a shell script to switch between branches (instead of your favorite git gui or git), and this script would at the same time switch out the remotesync-config

#!/bin/bash
git checkout $1
cp .remote-sync.json.$1 .remote-sync.json


[dhcp6] add multicast solicit and accept more general replies

Adding a compiler flag to enable OpenThread devices to transmit multicast DHCPv6 Solicit messages as well as adding code to enable compatibility with DNSmasq DHCPv6 Server replies.

Sorry about that, I used the GitHub API to update my fork and it didn't use my correct email

@jrodriguez42, this PR looks good but is failing the code style check. The patch is provided in the log. Otherwise, you can also use clang-format to automatically reformat the code (see STYLE_GUIDE.md for more details).

Sorry about that. I realized I pushed some changes that caused another check to fail and it took me a few tries to roll it back and properly fix it.

@jrodriguez42, thanks for the updates. I think you accidentally added FETCH_HEAD and dnsmasq_dhcp6_server. Can you remove those from this PR?


Add metaData / user data to AuthUser object

Is your feature request related to a problem? Please describe.
I currently have to manage user state through either Redux, or useContext.
I was wondering, since AuthUser is already everywhere in the app, why not use it to store User Data as well?
Describe the solution you'd like and how you'd implement it
Perhaps a setter function to set AuthUser Metadata.
Is this a breaking change?
Not really. If implemented the right way, it should be totally backwards compatible.

Thanks for the suggestion. This feels a little out of scope to me. An app typically will manage more than just user-related state, and of course people shouldn't attach non-user app state to AuthUser. In my mind, the better pattern is the reverse: if you have a state management system, you could consider putting AuthUser data into it, if it's useful to have things in one place. Thoughts?

Thanks for the suggestion. This feels a little out of scope to me. An app typically will manage more than just user-related state, and of course people shouldn't attach non-user app state to AuthUser. In my mind, the better pattern is the reverse: if you have a state management system, you could consider putting AuthUser data into it, if it's useful to have things in one place. Thoughts?

Yes, that's how I'm currently doing it. It's just the problem of sync and revalidation. For example, if I do X action, I'd like to refresh AuthUser.

Trying to understand this better: What do you mean by "refreshing" AuthUser? Do you mean force-refreshing the ID token (for example, to update claims)? How are you currently accessing AuthUser data and refreshing it?

Closing this, as I believe this is out of scope for this library.


omgx integration

I've deleted omgx-sync-tests.yml  because I think it's a duplicate of sync-tests.yml.
These tests really just test verifier and replica.
omgx-integration.yml has commented out the original Optimism integration tests. The old develop had them enabled and the reason for it was that it allowed us check whether our containers somehow interfere with the original containers. Just putting this out there in case @jliphard decides to enable them back.

I see that creating an omgx docker - compose overlay was commented out:
# yq eval '(.services.[].image | select(. == "ethereumoptimism*")) |= sub("ethereumoptimism", "omgx")' ${ORIGINAL_DOCKERFILE} | \
# yq eval '(.services.[].image) += ":latest"' - \
# > ${DOCKERFILE}

This means we're relying on Optimisms push of container images into their docker hub. Not ours. So their latest container could in in front of our containers and there could be discrepancies the way the system is functioning.

Codecov Report

Merging #146 (2969210) into develop (c7ebd2c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #146   +/-   ##
========================================
  Coverage    86.05%   86.05%
========================================
  Files           49       49
  Lines         1936     1936
  Branches       307      307
========================================
  Hits          1666     1666
  Misses         270      270

Continue to review full report at Codecov.

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

Most changes incorporated into #149.

okay, so should this be closed?

yes, can be closed


Add proposal to TLS encrypt the Kafka Connect REST API

This proposal reduces access to the Kafka Connect REST API by enabling
TLS encryption on the Kafka Connect REST listener when the connectors
running on the Kafka Connect cluster are managed by the
KafkaConnector or KafkaMirrroMaker2 operators.
Signed-off-by: Andrew Borley<EMAIL_ADDRESS>

Thanks for the proposal!


Readme tyops

"linmath.h provides the most used types required programming computer graphice:"
maybe better:
"linmath.h provides the most used types required for programming computer graphics:"

Applied the suggested changes.


PACKAGES: mod-chat: extend to full functionality using tencent API

CHat works but needs much more of the chat functionality people expect.
Below is an excellent base that works on Web and mobiles.
Its has zero dependencies also.
https://pub.dev/packages/tencent_im_plugin
https://intl.cloud.tencent.com/document/product/1047/33527

On the client we can use the grpc API to wrap the existing dart code
On the server we can implement the GRPC API

uses minio for images, sounds and video
uses GPRC NATS for messages

Envoy and OPA does the Security.

Push Server is needed to send Notifications to the client if the client app is not open.
For notifications we can use the golang server that wraps the apple and google gateways.
gorush
https://github.com/appleboy/gorush/tree/master/k8s

Has GRPC API to it: https://github.com/appleboy/gorush/tree/master/rpc/proto
Docker support for ARM and AMD64: https://github.com/appleboy/gorush/tree/master/docker
Storage: https://github.com/appleboy/gorush/tree/master/storage
badger, reids, etc
Web Push: https://github.com/appleboy/gorush/pull/310

Will be needed for Web (https://developers.google.com/web/fundamentals/push-notifications/web-push-protocol)
Use JWT so the receiver can be sure the message is from the correct sender.

This is also pretty clean but lacking also
https://github.com/GetStream/stream-chat-flutter

https://maintemplate.ci.getcouragenow.org/#/chatBeta

its basically a broken mess.

We should run the 2 candidates above inside chatbeta, and then decide ?

ChatBeta is broken because the mock data is not generating correctly on web...
But its much easier to use TenCent. I'll get working on that.


images number?

Does it support reconstruction of hundreds or thousands of images？

Hi there, I am afraid not due to the extensive memory cost of costvolume and deformable attention, < 10 views is a proper choice.


Woking with GPU

I tried models work on gpu but torch is not working. Program gives a simle GPU errors.
Like (Input type (torch.FloatTensor) and weight type (torch.cuda.FloatTensor) should be the same or input should be a MKLDNN tensor and weight is a dense tensor).
I put my veriables(input and model) to cuda on gpu with ".to(device)" or ".cuda()". It's still not working. But if all veriables and model were staying "CPU" code is working. How can I solve this problem? Feature Extractor model is on "GPU" too. But error is same. Could I code rewiev for my problem or this error can be lib error. But I have used Torch1.13 - cuda11.7.

can you paste 1) your training or inference code here?
2) you detailed error code

Second error is : "[RuntimeError: Input type (torch.FloatTensor) and weight type (torch.cuda.FloatTensor) should be the same]"
But you will see in the code  I send data and model to GPU!.But feature extractor return None. Debug the code and I see this error comes from feature extractor. Feature extrator was in GPU but input was not. Than I changed input type and tried but None type error.  Can u explain this error? I have searched so much but I didn't solve this. Thanks.

It seems that the model you are loading and the model definition have a mismatch. Either up/downgrade the timm such that the resnet definition corresponds, or load another model.

It seems that the model you are loading and the model definition have a mismatch. Either up/downgrade the timm such that the resnet definition corresponds, or load another model.

I can run this code with torch-cpu(2 different env). If I got mistmatch I couldn't run. But I will check timm version. Thanks for reply.

Instead of using torch.load, can you use the timm model creation? Then you can be more sure it's a version mismatch.

Thanks. I checked my lib versions. And I change to timm version (0.6.12 --> 0.4.12) and install faiss again. This solve my problem. But I want to ask something about prediction time. I ran my model on GPU and prediction time was : 40 ms avg. Than ran to CPU prediction time was :40 ms? I didn't understand and I am sure about the veriables are in cpu or gpu. I have 64 RAM 12 GB 3060 GTX. Do you have an idea for this issue?

I haven't tried measuring gpu<>cpu performance yet, but gpu should be quite a bit faster than cpu.


Cannot drop next to nested siblings

This one is a bit cryptic but becomes clear in this exaggerated example below; basically it seems that you cannot drop an element in the middle of a nested tree.. you need to adjust the level of indention at the last node that is indented.

This isn't a show stopping bug but its pretty close.  However, without  #777 this component is unusable for me.  Expect updates on one of these two very shortly.


Refactor config loading

#33 revealed how tricky the configuration loading in adc is. We should refactor it to be simpler and easier to understand.

This was done in the v1.0.0 release.


An extension problem caused the software to crash

If Extension Manager (ID: gjldcndgmdknpinoemndlidpcabkggco) is used, kiwi will automatically exit to the mobile desktop and can not use Kiwi normally.

This issue has been fixed . Consider reopening the issue if bug still persist


Vet-Clinic: Multiple tables

Hiiiiiiiii,

Activities Done:

[x] Created new tables called owners and species.
[x] Inserted data into the new tables just created.
[x] Updated the animals table by removing the species column and adding new columns species_id and owner_id.
[x] Made the new columns be foreign keys for the previously created tables.
[x] Performed the given queries.

Screenshots of the required queries are attached below:

@uwadonat Thank you so much, have a great day.


cmd/go: go get -u with GIT_TERMINAL_PROMPT=1 combines separate git username/password prompts, which can lead to password being exposed

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

$ go version go1.14.1 linux/amd64

Does this issue reproduce with the latest release?
Not Tested
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/nibin/.cache/go-build"
GOENV="/home/nibin/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/nibin/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/snap/go/5569"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/snap/go/5569/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/nibin/golang/mod-exp/sample-mods/mysample/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build009035298=/tmp/go-build -gno-record-gcc-switches"

What did you do?
I tried to update go module dependency ( 2 separate go modules residing in 2 repos) ( the dependency(s) resides in a private repo ) using GIT_TERMINAL_PROMPT=1 GOSUMDB=off go get -u

What did you expect to see?
Assume Password Entered is : mypassword
Username for 'https://gitlab.com': myusername
Password for<EMAIL_ADDRESS>Username for 'https://gitlab.com': myusername
Password for<EMAIL_ADDRESS>
What did you see instead?
Assume Password Entered is : mypassword
Username for 'https://gitlab.com':
Username for 'https://gitlab.com': myusername
Password for<EMAIL_ADDRESS>
Password for<EMAIL_ADDRESS>

@jayconrod @bcmills @matloob

This is happening because the go command runs git commands for multiple repositories concurrently. By default, it sets GIT_TERMINAL_PROMPT=0, but it allows users to override that.
The narrow solution here would be to only run one git command at a time if GIT_TERMINAL_PROMPT=1. That doesn't seem like a great experience though.
@bcmills Should we allow GIT_TERMINAL_PROMPT=1 at all, now that ~/.netrc is supported? Does it cover this case?

We probably should not allow GIT_TERMINAL_PROMPT.
At one point I started adding explicit serialization of Git terminal operations (first setting to 0, then acquiring a lock and setting to 1 if it is set in the containing environment), but after discussion with Russ we decided that that wasn't a great UX, especially given .netrc and the Git credential cache in general.

+1 to disallowing GIT_TERMINAL_PROMPT. Let's make sure .netrc and other procedures for accessing private repos are better documented first though. I suspect in most cases, people are only using one private repo and won't run into this issue.

There needs to be a way to serialize the request for passwords / passphrases. Otherwise people who don't want to write their passwords in plain text cannot even use the old workaround of GIT_TERMINAL_PROMPT=1 to be prompted for the passwords and makes go run and go get no longer usable.

@RubenGarcia, there are other, cleaner ways to pass credentials to git without storing them on disk. (See, for example, git-credential-cache.)

What would be the way that a user can go get from private repo and avoid this with minimal fuss?  I run into this issue frequently at and it is pretty frustrating.  I'm often on different computers so saving my password on each of them doesn't seem ideal.  With the git cache, wouldn't I first need to git clone something to get my password in the cache? It seems kind of silly to git clone a repo just to get my password saved and turn around and use go get to pull it into whatever project I'm working on.
Whatever the method is, I need to be able to explain it to my co-workers so they can avoid the issue too and it should be something simple.  Cloning or downloading code should be really simple.  The need for GOPRIVATE and GIT_TERMINAL_PROMPT is already far more complication than it really should have :(
I think the very basic old fashion magic of being able to type in your username and password should work.  If that means when using GIT_TERMINAL_PROMPT=1 that go only calls git commands sequentially then that's a far better trade off (to me) then making it not work at all and not allowing someone to just simply type their password in.
I don't run into this as often at my work computer but when I'm on a different system it pops up - like today there was an issue and I needed to try and look at something quickly from home and when I tried to go get a package I got 3 user prompts piled on top of each other and it's pretty much impossible to figure out rather I should be typing a username or password or some magic of both at the same time.  :(

@bwmarrin, one option is to always fetch using HTTPS, with a personal access token stored in a .netrc file.
Another option is to fetch the private repo using SSH, and to use the ControlPath option to set up the connection outside of cmd/go itself; see https://github.com/golang/go/issues/49515#issuecomment-969247066 for some options that seem to work for that. (If you run into trouble with those, please let me know on that issue.)

Thanks for the options @bcmills  - I do appreciate the suggestions.  These suggestions and the other's I've read in this issue and elsewhere all seem like "work arounds" and that's fine as it helps solve the issue for right now but I don't think they're very good solutions.
Option 1 isn't terrible, I suppose - but it's not very ideal when working on multiple computers and having to leave your auth in a file on all of them.  I'm sure there are plenty of cases where leaving your auth info on systems just isn't very viable. Option 2 seems wildly complicated.
I'll re-iterate that I believe downloading a repo should be incredibly easy and in these cases of private repos and/or self hosted repos that isn't the case and I feel like that's a short coming of the Go toolset :(.  I still think having a fall back option of being able to simply type your password should exist and work.


feat: include ScrewMPCMpAgent (manipulability)

Changes

Include an Agent for considering manipulability.

I don't think I really love the idea of two different agents doing almost exactly the same thing. Also, copying over the exact same code, when really only one line needs to change (the initialization of the motion generator). I think this should be a simple parameter of the agent (use manipulability maximization or don't). Either I merge now and refactor later or you can do the refactor if you like.

Okay, adjustments underway ;)

Changes:

include flag for using manipulability.


A few issues

Hi @MKM12345,
This is some constructive criticism I have for this.

Indentation Problems
There are a lot of problems to do with indentation in this project. I'm guessing this is because you used the in-built code editor in GitHub, which isn't ideal. For better results, use a proper code editor such as VS Code.
Incorrect File Tree
A lot of files seem like there where just placed there with no real attempt at organising them. Because of this, there are a lot of import issues in your project. In the README, you put that you are meant to import ai_commands like this: import ai_commands. But as init.py is empty, and you have another file containing the library's contents, you are meant to use from ai_commands import ai_commands for the import to happen successfully, however incorrect formatting (as mentioned before) makes this also unusable.
ai_summarise()
To me, it looks like this method should be a class. There are quite a few 'nested methods' which isn't clean Python and can make it hard to read. I'd turn this into a class and have all the methods listed inside of it. Then create the object at the beginning of the program and use it. Or, you could take all the methods out and separate them , which is the easiest thing to do considering the current state of the library.

That's it for the criticism, but this is a great project! It mixes AI with a lot of things and, although it is unlikely poeple will use this in their mainstream projects considering the size and use cases,, it is awesome starter projectt on your journey to making libraries on PyPi.
Also, as I am a contributor on this project, I can fix these errors for you too (if you are not free).

Hey I am already fixing these @hamdivazim hamdivazim. Please wait till then.

Alright, just pointing what you need to work on

I am using this for experimental usage and is not ready for production.

Alright @hamdivazim, very good points. I will work on it! Thanks for the info!

If you could submit your ideas in a pr please, it would be helpful @hamdivazim


[Bug]: CheckBOMStructure() in COD99000769 "Production BOM-Check" uses uninitialized global variable Item

Describe the issue
A call of CheckBOMStructure() in COD99000769 with a production bom in which a line has set a routing link code leads to the error message 'Routing Line with Routing Link Code ... cannot be found. Check Production BOM No. ... Line No. ....', although the (supposedly referenced) routing has a line with this connection code.
This is because CheckBOMStructure() uses the global record variable Item, which is not initialized when called from outside the codeunit.
Expected behavior
It should be possible to pass a record variable for Item to CheckBOMStructure(). Either directly as a parameter or via an overload in which Item is set from a parameter.
Steps to reproduce

Create a routing link code.
Create a routing with an operation with this routing link code.
Create a production bom with a line with the same routing link code.
Create an item and set routing and production bom.
Write a function that calls CheckprodBOM() for the production bom.

Additional context
No response
I will provide a fix for a bug

[X] I will provide a fix for a bug

This could be something we'll approve (we'd have to take a closer look), but since the code you are referring to is in the BaseApp, you'd have to suggest and fix this code in the BaseApp contribution pilot: https://github.com/microsoft/businesscentralapps. For access to the pilot, please fill out the following form: https://forms.office.com/r/E9wyE5a58f.
This repository is only for extensibility requests or pull requests towards the 1st party Business Central apps.
Would you like me to move the issue to the contribution pilot for you?
IMPORTANT: Microsoft is not going to address the reported issue for you. You will have to get the issue approved, and then push a pull request with the suggested changes yourself. If you would like Microsoft to address the issue, please create a support request.

Closing due to inactivity. If you'd like to address this issue, please create it in the BaseApp repo mentioned above.


Add missing EventEmitter.emit args/types

Coverage increased (+0.01%) to 68.967% when pulling ea008411e4c88824fd8a0775c7639036d0d2eab0 on Tyriar:add_missing_emit_arg into a95ac990ffd3fab501388c8501c835d969f58cbc on sourcelair:master.


Fallback fonts

XFCE terminal does not have every Unicode character we are using. We should create fallback fonts, somehow, or maybe we can show emojis only on particular terminals.

I never did something like this before, I don't know how to set fallback fonts but we can start by setting fonts only on specific terminals? I don't even know if it's terminal-related or system-related or what else.

Maybe it's related on both terminal and system, we should investigate 🔬


Remove all fields not used as search criteria from Elasticsearch

Our Elasticsearch cluster contains many fields that are not used directly in search, including:
    identifier = Keyword()
    creator_url = Keyword()
    created_on = Date()
    url = Keyword()
    source = Keyword()
    meta_data = Nested()

Cutting these fields from Elasticsearch may significantly lower the size of our data in memory, which would allow us to run a leaner and cheaper Elasticsearch cluster, or at the very least improve search performance.

The feature in itself is not hard to implement, but how would a test be written for this part, since the sample_data.csv is quite small, and the performance gains won't be noticeable


Versioning tag

You still seem to missing the GitVersion.yml and the package

And the csproj (like https://github.com/morganstanley/dotnet-please/blob/main/DotNetPlease/DotNetPlease.csproj#L27-L30 )

pr updated with required files

You need to set fetch-depth on the checkout action:
https://github.com/morganstanley/dotnet-please/blob/main/.github/workflows/build.yml#L14
GitVersion needs the whole commit history of the branch being built, as explained here:
https://gitversion.net/docs/reference/requirements

@BalassaMarton @bagdxk kindly review my pr and give me feedback. thank you

@BalassaMarton @bagdxk kindly review my pr and give me feedback. thank you

TBH I'm not sure what would happen, but since we're not publishing the packages, we can run this without risk.

So they are trying to replicate the setup from Please here for versioning and release to nuge.org.


social login returns {"exist": false}

I'm trying to integrate google and facebook sign ups and whenever I hit the auth path, I get this response:
{ "exists": false }
It was working well back in November, haven't updated the package or changed any config in google cloud platform or meta developer account
System information
Medusa version (including plugins): v1.17.4
medusa-plugin-auth: v1.8.3

Can you upgrade your auth package version please. I will soon deploy the updated doc as well

I have updated the auth package to v1.9.1 and still the same error occurs.

@adrien2p , how you please help me on this?

Had the same issue. I was in 1.11.0 and upgraded to 1.11.1 and it solved the issue. I have dug much but my intuition is that the exists: false is throw by Medusa, not by the plugin. Why is a mystery.


Weather app

Create a single page web app, that displays the Weather in the area of  your current position in the world, this project can be done using either React / Vanilla JS

[ ] OPTION: display weather in a specific location in the world
[ ] OPTION: display weather of the entire weak

Hello, can I work on this?

@panvicka yes you're now assigned to it!


showFullScreenButton not found

I am enable to link showfullscreenbutton and other attributes like showYouTubeButton,showVideoCurrentTime etc in XML i am only tyring to use  showfullscreenbutton but other options does not show either

dude i am also having the same issue. did you find any solution??
AAPT: error: attribute showVideoDuration (aka com.diltech.firebaserecv:showVideoDuration) not found.

Hi, which version of the library are you using?

Hi, which version of the library are you using?

implementation 'com.pierfrancescosoffritti.androidyoutubeplayer:core:11.0.1'
Am trying to get videoID from firebase Realtime Database in recyclerView

@PierfrancescoSoffritti
hi, thanks for replying
i am trying to get videoIds from firebase realtime database in recyclerview. Everything works fine the videos gets stream in recyclerview perfectly. but these attributes that i have add does not work and i get an error like this.
Android resource linking failed - AAPT: error: attribute showSeekBar (aka com.diltech.firebaserecv:showSeekBar) not found.

however when i remove these attributes everything just works fine
here are the dependencies that i use

Version 11 does not support those attributes, see release notes: https://github.com/PierfrancescoSoffritti/android-youtube-player/releases/tag/11.0.0

I'm Also find the same error
attribute showFullScreenButton (aka com.webandappdevelopment.serviceshop:showFullScreenButton) not found.


Set httpcore5 to 5.2 to be compatible with httpclient5 5.2.1

Description
org.apache.httpcomponents.client5:httpclient5:5.2.1 is compatible with httpcore5:5.2 and the recent upgrade to 5.2.2 is causing problems running an extension using the SDK. I am opening a PR to downgrade the version to 5.2 which is compatible with httpclient:5.2.1
Issues Resolved
https://github.com/opensearch-project/opensearch-sdk-java/issues/857
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@reta There were issues in the SDK related to the httpcore5 update to 5.2.2 so I created a PR to set it to 5.2 which is compatible with httpclient:5.2.1.
I created another PR in opensearch-java too: https://github.com/opensearch-project/opensearch-java/pull/566

Isn't the fix to upgrade httpcore to 5.2.2?

@dblock Looking at the source code for HttpClient5:5.2.1, it references HttpCore5:5.2 not HttpCore5:5.2.2
This isn't the case with HttpClient5:4.5.14 which references HttpCore5:4.4.16

Isn't the fix to upgrade httpcore to 5.2.2?

We use 5.2.2 explicitly, the transient 5.2 dependency from HttpClient5 should not be used

Fix is sent to https://github.com/opensearch-project/opensearch-sdk-java/pull/861

Thank you @reta ! Closing this PR.

For the record, rest-high-level dependencies:
> Task :client:rest-high-level:dependencyInsight
org.apache.httpcomponents.core5:httpcore5:5.2.2
  Variant runtime:
    | Attribute Name                 | Provided     | Requested    |
    |--------------------------------|--------------|--------------|
    | org.gradle.status              | release      |              |
    | org.gradle.category            | library      | library      |
    | org.gradle.libraryelements     | jar          | jar          |
    | org.gradle.usage               | java-runtime | java-runtime |
    | org.gradle.dependency.bundling |              | external     |
    | org.gradle.jvm.environment     |              | standard-jvm |
    | org.gradle.jvm.version         |              | 11           |

org.apache.httpcomponents.core5:httpcore5:5.2.2
\--- project :client:rest
     \--- runtimeClasspath

org.apache.httpcomponents.core5:httpcore5-h2:5.2.2
  Variant runtime:
    | Attribute Name                 | Provided     | Requested    |
    |--------------------------------|--------------|--------------|
    | org.gradle.status              | release      |              |
    | org.gradle.category            | library      | library      |
    | org.gradle.libraryelements     | jar          | jar          |
    | org.gradle.usage               | java-runtime | java-runtime |
    | org.gradle.dependency.bundling |              | external     |
    | org.gradle.jvm.environment     |              | standard-jvm |
    | org.gradle.jvm.version         |              | 11           |

org.apache.httpcomponents.core5:httpcore5-h2:5.2.2
\--- project :client:rest
     \--- runtimeClasspath


fix unused var check in release mode

The vars are actually used in #[cfg(debug_assertions)], the warnings will pop when building in release mode
Test:

run cargo build -r

total_coins as well?

hmm didn't know that using debug assert would silent the unused var warning. Did you confirm with a release build?

@lxfind yes, I did not know that either, TIL.
here is the output running cargo build -r https://app.warp.dev/block/SkF0aVPb5uQavW4bZ3bdnE


ruff-ecosystem: Add indico/indico repo

It's a pretty big codebase using lots of different stuff, so a good candidate for finding obscure problems.
I didn't look more closely which options are used (I have the feeling --select ALL is not implied, since I see you adding it via check_options for certain entries but not for others), the repo itself has a pretty large ruff.toml - but assuming ecosystem just cares about differences between base and head of a PR, ALL most likely makes sense.

I'm gonna start it out without ALL, since it already has a bunch of rules enabled and it can actually be helpful not to override the selectors when possible (since it can surface other kinds of bugs).

Thank you for this!


Fix/unsafe replace

Summary
The unsafeReplace may can't find search string,
We should handle this.
Related Links

Checklist

[ ] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

!canary

!canary

!canary

!canary

!canary


how to use mx.sym.dot

In mxnet, I HAVE no idea about how to use the mx.sym.dot, hope u can help me

For example, i want a row vector in a matrix TO dot-multiply with a row vector in another matrix ，but I do not want to do for operation, is there any easy ways to solve this,,,

@Eshter Use batch_dot for the case http://mxnet.io/api/python/symbol.html#mxnet.symbol.batch_dot

why are u closing this？

@Eshter
For example, assume MatA has shape (B, 1, C) and MatB also has shape (B, 1, C). Our goal is to get the ret of size (B, 1, 1) by dot-multiply the corresponding row-vectors in two matrices.
There are two ways to achieve this:

Elementwise multiplication and use mx.nd.sum

ret = mx.nd.sum(MatA * MatB, axis=-1, keepdims=True)

Use mx.nd.batch_dot

ret = mx.nd.batch_dot(MatA, MatB, transpose_b=True)

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Driver-specific extensions of the web interface

Describe the solution you'd like
Sometimes there is printer- or driver-specific functionality which cannot be necessarily described by extra options ("vendor options") on the "Printer Defaults" page.
For PostScript for example there are:

For (nearly) each option there is PostScript code in the PPD file with which one can poll the setting of the option from the printer.
In the PPD file is a group of options (usually named "Installable Options") with which one can tell which optional accessories (like extra trays, mail boxes, duplex unit, ...) are installed in the printer. UI constraints describe which choices of the actual job options (the ones to be listed on the "Printer Defaults" page) are available and which not depending on whether the accessory is installed or not. Sometimes there is even PostScript code so that one can apply (1) to auto-detect the printer's hardware configuration.

I have now thought about how one can make use of this functionality. Without any changes on the web interface it is rather restricted, one could poll the printer for option default settings and installed accessories when one creates the queue for it, but

The printer gets woken up (for PostScript the poll is a zero-page PostScript job sending back the settings as answer) on driver setup (creation of the queue and every start of the Printer Application)
The poll cannot get easily repeated when accessories are added/removed or printer-internal defaults changed. Having to restart the Printer Application is no very intuitive and in the previous point we also told that we do not have a poll during startup.
With the options of the "Installable Options" group on the "Printer Defaults" page together with the job options one would need to modify settings of one option when the setting of another is changed which gets difficult on web forms.

So what I need would be driver-specific customizations of the web interface. For the PostScript Printer Application this would be:

The print queue pages (and also the printer entries on the main page) have one extra button, named "Installable Options". This button opens a page which shows all the options of the "Installable Options" group of the PPD file. When submitting the settings, the driver setup callback is re-run, taking into a account these settings by not adding the conflicting choices of the options to the driver data structure. The "Installable Options" page could in addition have a button which, when one clicks on it, triggers the printer being polled for the settings of all the "Installable Options".
The "Printer Defaults" (and also "Media") have an extra button to poll the value from the printer. The page also has an extra button to poll all the option values.

So what I would like to have that there are callback functions which allow to add extra buttons to print queue entries and to pages like "Media" and "Printer Defaults", ideally also to each option. I also would like to have the possibility to add extra configuration pages to the print queues.

@tillkamppeter You can add custom web pages with papplSystemAddResourceCallback and add links with papplPrinterAddLink.
That said, ordinarily this is something that the status callback is supposed to do automatically. What I would suggest is to query the installable options once per minute (or every five minutes), keeping track of the last time you checked since PAPPL will call you up to once per second.


Improper/missing use of modern Javascript features.

Expected Behavior
Yes.
Actual Behavior
No.
Steps to Reproduce the Problem

Not using Set when more appropriate than Map. Note: use of Map vs Set will double memory usage.
See hullGeom2.js, line 17 & 26, hullGeom3.js line 20 & 29.

When using Map() with vectors or other numbers, you sometimes convert to string. This just isn't necessary, Map and Set can be indexed by any Javascript type. This is done in hullGeom2&3.js. See vertexMap and edgeCount in repairSlice.js (These should be Maps!).

insertTjunctions.js violates everything, you misuse an object, indexed with a string and used as a set.
It starts with "sidestocheck" at line  165. Also all use of "tags" there are unnecessary.

Generally all Objects and Map and Set usages should be audited. Or at least put on the list of things to look out for.

Specifications

Version:
Platform:
Environment: (browser, local server, node.js etc)

2. When using Map() with vectors or other numbers, you sometimes convert to string. This just isn't necessary

Disagree on this point. We are trying to find unique points by value, not by pointer. It's important that duplicate points, but with different underlying pointers, still map to the same entry. So two different points with [1,2] and [1,2] should map to the same entry, regardless of how they are created.
Not what we want:
const m = new Set()
m.add([1,2])
m.add([1,2])
m.size === 2

This is what we want:
const m = new Set()
m.add(`${[1,2]}`)
m.add(`${[1,2]}`)
m.size === 1

Overall great feedback though! I agree that we should audit Maps and Sets across the codebase to make sure we are using them optimally.

Interesting... this is the problem with really only having book learning on the subject.
When I was translating... I had to often read the Javascript spec and it says that all types work.
Apparently they don't consider lists a type.
Note, I do this in C# all the time. For a class, you have to have IEquatable implemented.
For tuples (anonymous structs) you don't have to do anything, it just works.
I.E. You can:
Dictionary<(double, double), object>()[(1,2)] = "some stuff" and (1,2) will act exactly as you expect.
I of course implemented IEquatable on my Vec2/3 class (really all of them).

Just wondering, have not seen any articles or mentions ...
could it be that there is performance improvement if using classes Vec3 with declared x,y,z fields instead of arrays. JIT compiler is likely to have more options for optimisations. I am wondering if performance is the reasong threejs and babylonjs use classes Vec3 vec4 Color3 Color4 ...

@platypii You got me curious... so C# doesn't let me do it with Arrays or Lists.
However Python does. Python is the dynamic language I use the most so is likely why I interpreted the spec to mean it work for lists too.

Interesting... this is the problem with really only having book learning on the subject.
When I was translating... I had to often read the Javascript spec and it says that all types work.
Apparently they don't consider lists a type.
Note, I do this in C# all the time. For a class, you have to have IEquatable implemented.
For tuples (anonymous structs) you don't have to do anything, it just works.
I.E. You can:
Dictionary<(double, double), object>()[(1,2)] = "some stuff" and (1,2) will act exactly as you expect.
I of course implemented IEquatable on my Vec2/3 class (really all of them).

SIGH. The original implementation was using Map with the vectors, but it turns out that converting to string provided a huge performance improvement. I suspect that the issue is with the comparison of objects which occurs dead last after all other types have been passed by. Anyway, we are at the mercy of the javascript engines.

GAGA. I hope the code isn’t using OBJECT for hashing. If so, that needs to be fixed.

4. Generally all Objects and Map and Set usages should be audited. Or at least put on the list of things to look out for.

geometries/geom2/toOutlines.js:  const vertexMap = new Map()
geometries/geom3/validate.js:  const edgeCount = new Map()
operations/booleans/scissionGeom3.js:  const indexesPerPoint = new Map()
operations/expansions/expandShell.js:  const vertices2planes = new Map() // {vertex: [vertex, [plane, ...]]}
operations/expansions/expandShell.js:  const edges2planes = new Map() // {edge: [[vertex, vertex], [plane, ...]]}
operations/expansions/offsetFromPoints.js:    const pointIndex = new Map() // {point: index}
operations/hulls/hullGeom2.js:  const found = new Map()
operations/hulls/hullGeom3.js:  const found = new Map()
operations/modifiers/insertTjunctions.js:  const sidemap = new Map()
operations/modifiers/mergePolygons.js:  const edgeList = new Map()
geometries/geom2/toOutlines.js:  const vertexMap = new Map()
geometries/geom3/validate.js:  const edgeCount = new Map()
operations/booleans/scissionGeom3.js:  const indexesPerPoint = new Map()
operations/expansions/expandShell.js:  const vertices2planes = new Map() // {vertex: [vertex, [plane, ...]]}
operations/expansions/expandShell.js:  const edges2planes = new Map() // {edge: [[vertex, vertex], [plane, ...]]}
operations/expansions/offsetFromPoints.js:    const pointIndex = new Map() // {point: index}
operations/hulls/hullGeom2.js:  const found = new Map()
operations/hulls/hullGeom3.js:  const found = new Map()
operations/modifiers/insertTjunctions.js:  const sidemap = new Map()
operations/modifiers/mergePolygons.js:  const edgeList = new Map()

Closing as the use of map vs set has been reviewed. only the hull logic was adjusted.

Reopening as there are additional functions using... yup... objects for maps.

DONE slice.repairSlice()
DONE reTesselateCoplanarPolygons() in booleans

If anyone knows of more then please add a comment.

@platypii would you like to rework insertTjunctions() ? If not then I'll start working on the changes.

I'll take a shot at it, unless you've already started working on it

Go for it.

I believe that all the functions are now using proper maps. Closing this issue.


Update forecastle to v1.0.34

The aim of this PR is to update forecastle to v1.0.34, this way we can use the forecastle.stakater.com/url to overwrite the URL inferred from the ingress to allow, for example, for non-standard ports.
I left out the new CRD on purpose as I don't think it could be of any use to us.

added the bats tests for forecastle, it passed. LGTM


feat(rbac): add type checks with generics for audit log

What does this pull request do:
Add type checks for audit log metadata object for type safety.

/lgtm

/cherry-pick 1.2.x


Localize screenshots to all languages

[ ] Localize screenshot 1 for all languages
Estimated Time: 1 hour

[ ] Localize screenshot 2 for all languages
Estimated Time: 1 hour

Screenshot 1
Screenshot 2

All screenshots uploaded to Chrome Developer Dashboard :)


Provider for Document DB

Are there any roadmaps for EF provider for Document Db?

Dupe of #1035.


I can export a PNG/JPEG or SVG image of the diagram

You can use canvg (https://github.com/gabelerner/canvg) and do something like this:
        function generatePng (svg) {
          var canvas = document.createElement('canvas');
          canvg(canvas, svg);
          var imgSrc = canvas.toDataURL('image/png');
          return imgSrc;
        };

+1 (in hope of raising priority of merging the code in upstream asap :wink:)


add IHI postgrad studies

Going to nest these under academic category in Resources for now

thanks Chris. I think we can close


HttpResponseForbidden if user is logged in else redirect

Hi,
I needed little modification and I wasn't able to do it without modification of django-guardian. I didn't find way how to redirect anonymous users to login page and show HttpResponseForbidden for users, who are already logged in. So I made this patch.
It adds user_specific parameter to permission_required. If user_specific is True, than it will check if user is logged in. If it is so, and user hasn't enough permissions to view the page, error 403 is returned. If user isn't logged in and if anonymous users can't view the page, than he will be redirected to login_url.
By default, this option is disabled in permission_required. So it keeps backward compatibility. For easier use, I add also permission_required_user_specific decorator, which is a shortcut to permission_required with user_specific parameter turned on.
I don't know, if it fits to django-guardian philosophy, but I posted it here, to save someone else time.
P.S.: I don't know, if user_specific is the best name for this functionality. But I didn't find out better label.

Closing as there were no response in a long time. Let me know if that's still a requested feature.


Rowspan causes issue with swipe

When I add a rowspan to one of the table cells it thinks the first cell of the remaining table rows in that rowspan should be a tablesaw-cell-persist
See this jsfiddle for an example

Related: #48

Hi,
Is this issue fixed in the latest version of tablesaw?
Thanks,
Mike

Simple support for rowspan has been added. There are still a few limitations, take note here: https://github.com/filamentgroup/tablesaw#limitations


Update compass-icon icon-file-document-outline to match icon font rename

Summary
The icon icon-file-document-outline was recently renamed in Compass Icons to icon-file-text-outline, this PR reflects that change in the Webapp now that the Webapp is using the Compass Icons package directly.
Release Note
NONE

@jgilliam17, the icon is used in the channel header file icon button, the no file results RHS and the file button in the "What are you searching for" drop down when clicking on the empty search box.


node v12: Error: listen EADDRINUSE: address already in use

Trying to run a simple app in a Digital Ocean kubernetes cluster using the node v12 template but getting this error when invoking the function:
Error: listen EADDRINUSE: address already in use :::3000
I've definitely not got anything running on this port (tried to use other arbitrary ports, always getting the same error).
Any help would be appreciated.
Thanks
Your Environment
Kubernetes running on Digital Ocean

What version of Kubernetes are you using? kubectl version v1.21.5

Hi @rvarkonyi without some more details I don't know if we can really help debug this. Can you include more information about your app and any other software you installed in your cluster, including how you installed openfaas? Ideally, if you can share a link to your app code and give concrete instructions for the comands you ran on your cluster, that will allow us to attempt to help.

Reviewed on the Office Hours call today, no response from original poster, so closing.
If someone else runs into this, please raise your own issue.

/lock


(swedish) Update reactivedrop_swedish.txt

Bruh... what now?

i analyzed closely. there is a missing quote mark here that i believe to be causing the error. line 4670

I will probably need to correct all those speedrun achievments into swedish at some point. Right now all of them are english


Fix our labels to align with our program management

Updated to include 3 labels for our P3 structure - people, products, pathways.


Nuxt 3: Objects are not valid as a React child

Tried to add react to components to vue app with vite, worked perfectly. Thanks a lot!
Trying to add the same react components to vue in nuxt 3, got errors :(
Objects are not valid as a React child (found: object with keys {__v_isVNode, __v_skip, type, props, key, ref, scopeId, slotScopeIds, children, component, suspense, ssContent, ssFallback, dirs, transition, el, anchor, target, targetAnchor, staticCount, shapeFlag, patchFlag, dynamicProps, dynamicChildren, appContext, ctx}). If you meant to render a collection of children, use an array instead.
My deps if that helps
"devDependencies": {
    "@nuxt/devtools": "latest",
    "@types/node": "^18",
    "nuxt": "^3.6.1"
  },
  "dependencies": {
    "@vitejs/plugin-react": "^4.0.1",
    "@vitejs/plugin-vue-jsx": "^3.0.1",
    "react": "^18.2.0",
    "react-dom": "^18.2.0",
    "veaury": "^2.3.12"
  }

Maybe you already know where I should dig into. Thanks.

@augustjourney
SSR is currently not supported.

@augustjourney @ekkaiasmith
Try<EMAIL_ADDRESS>https://github.com/gloriasoft/veaury?tab=readme-ov-file#using-veaury-in-the-ssr-projects


[Feature][Mongodb-CDC] Add multi-table e2e testcase

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
[Feature][Mongodb-CDC] Add multi-table e2e testcase
add testcae into
https://github.com/apache/seatunnel/tree/dev/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-cdc-mongodb-e2e
reference
https://github.com/apache/seatunnel/blob/dev/seatunnel-e2e/seatunnel-connector-v2-e2e/connector-cdc-mysql-e2e/src/test/java/org/apache/seatunnel/connectors/seatunnel/cdc/mysql/MysqlCDCIT.java#L177
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Hey @hailin0 ,
I've created a pull request, but there are errors in the Github Actions. I'll fix them and let you know.

Dear @hailin0,
After careful consideration and multiple attempts to address the issue, I have come to the realization that it is beyond my current expertise and skill set.
I deeply apologize for any inconvenience this may cause to the team and the progress of the project. It was not my intention to hinder the development process, and I regret that I am unable to fulfill the requirements of this particular task.
Thank you for your understanding and support during this time.


Lower module params

Adds a step after canonicalization that lowers module params in Declarations.

[x] Top-level functions get one extra argument for the params (if any)
[x] Imported module calls are extended with one more argument referencing the ImportParams def
[x] Top-level value defs are converted into function to take the params (if any)
[x] Imported module references are converted into calls that pass the params
[x] Nested lambdas capture params from higher scopes
[x] Home function calls are extended by passing params from current function
[x] Home value references are converted into calls that pass params
[x] Top-level value defs referenced in calls need to be first applied with the params
[x] Top-level functions passed to another module are wrapped in a lambda that captures params
[x] Params don't show up in language server types
[x] Type errors updated to hide params argument

@agu-z it's related. Basic-cli task was merged already, so anything using latest of that platform will break, AKA that test. So if you want to override merging, I think that's fine

After #6836 is merged, we can pull in main here and the tests should pass, assuming that there are no merge conflicts.


Add toggle API method

I needed to make a "combobox" and found that adding a toggle API method helped with that immensely. Thought it might be more generally helpful.
It handles 3 states:

Dropdown is empty
Dropdown is hidden
Dropdown is open

For 2 and 3, toggle simply calls the corresponding open or close method (ensuring the proper event is fired). For 1, minChars is set to 0 (to ensure the full data list is shown) and evaluate is called (which also fires the awesomplete-open event).

Hi Stephen, thanks for contributing!
This looks like it could be a separate plugin or example, since it’s not used internally, it just extends the API. Please make an advanced example with this functionality? That’d be great!
If many people find it useful, we can add it to the API later.

I'll update this PR to be a documentation change instead.

Thanks! Let me know when you do so I can merge this in :)

Sorry for the crazy delay, but life got crazy. I've updated this PR to include only an additional "Advanced Examples" section along with a small number of CSS additions to style to combobox input.

Better late than never :)
Could you please scope the CSS to #combobox (or [id="combobox"] to avoid the extra specificity) to avoid future conflicts and confusion?

Done :)

Merged, thanks!


docker-app init: validate compose file

Currently, the compose file is just copied as it is. A validation of the compose file format here can help. Currently I'm running docker-app render to see if everything is correct.

We're currently doing some specification work around the format in https://github.com/docker/app/pull/269. That PR has some internal tools for validation which we're discussing exposing as docker-app validate as well.
I think however that init may fail currently if it's not provided a valid Compose file? @mnottale

We now have a validate command in master from https://github.com/docker/app/pull/290, this will be in the next release.


Write blog post about metanorma setup and Fontist

Inheriting some information from #271 , let's have a blog post that describes the thinking of metanorma setup and how it is used, where the fonts are setup, etc.

This task is still relevant. We need to document how to use Metanorma with Fontist to install fonts (using a local executable Metanorma, or Metanorma docker which requires setting up fonts on a new volume).

Because we removed metanorma setup command, it's nor relevant anymore, thanks for note


Add instruction to copy/paste in nano

If there are instructions instructions to close the nano editor, it seems like there is an assumption of the level of familiarity with the editor.  It would make sense to add instructions to copy and paste as well.

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

ID: e2289383-6ce2-a036-c710-7d1fad6e1700
Version Independent ID: 3b902941-0876-17c3-3f0b-a4b7f7770531
Content: Quickstart Azure IoT Edge + Linux
Content Source: articles/iot-edge/quickstart-linux.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@theangellmethod Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@theangellmethod I can see your point, but I don't think this is a high priority for the user experience in docs at the moment. I can add this to my backlog for updates to the documentation. Did you know that you also have the ability to edit any article on docs.microsoft.com? The Edit button on each page takes you to the GitHub web UI for quick edits. Any pull requests made from public contributions are assigned to the article author to approve or give feedback.

I do but I'm not sure when to use the quick edit if I have an opinion (as opposed to fixing a typo) and in the event I make the edit across multiple docs for this kind of thing (because there are different language tutorials) and they all get rejected or some accepted/some rejected then it adds complexity to go back and resolve each one individually.

@theangellmethod I've decided not to incorporate this suggestion into the documentation. Copy+paste steps aren't unique to nano, as the save and exit steps are. Instead, copy+paste steps depend on which interface you're using to configure the IoT Edge device, which may vary between readers.
I'm going to ask that @Mike-Ubezzi-MSFT #please-close this issue. If you still feel strongly that these steps are necessary for the documentation, feel free to reopen the issue and provide more information about how the docs should be updated.

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


add turbofish-like syntax

Problem
In Rust, the "turbofish" syntax (::<>) allows users to explicitly specify generic type parameters in function calls. This feature has proven to be useful for disambiguating type inference in certain scenarios.
Happy Case
fn main() {
    let mut slice = [];
    slice = slice.push_back(1);
    slice = slice.push_back(2);
    let slice = slice.as_array::<2>();
}

Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

Converting to discussion for community chime in and votes.


Error: URL using bad/illegal format or missing URL

Expected Behavior
Hey, when I have "Automatically encoded special characters in URL" unchecked I seem to be getting an error when using a space " " inside the URL Error: URL using bad/illegal format or missing URL - This never used to happen in Insomnia and I've been using it for years, this was the primary reason I've been using insomnia is the ability to easily control and malform requests for my security work. It should still make the request using the unencoded space character.
Actual Behavior
It returns an error: Error: URL using bad/illegal format or missing URL
Reproduction Steps
Use any URL with a space " " and make sure you have "Automatically encoded special characters in URL" unchecked
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnia Version
2022.1.1
What operating system are you using?
macOS
Operating System Version
macOS Monterey 12.1
Installation method
Download via website
Last Known Working Insomnia version
No response

I'm able to reproduce this issue. @Kong/team-insomnia I'm not entirely sure, but this might be related to the recent changes we did in order to support recent Electron versions.
Here's an example comparison, between 2021.7.2 and 2022.2.0:

The version previous to the one @unreleased mentions in the issue description is affected as well, so might be also from the changes we did to bump to Electron 12.

This issue was caused by an update to libcurl. We're working on a workaround for this problem, which should make it into a beta in the near future (#4659)

Hey @unreleased, follow-up to what @johnwchadwick mentioned above, we've released the fix for this issue in our latest beta (2022.3.0-beta.2).
Thank you for reporting this, feel free to report any other issues you come across 👟 😉

Thanks for the quick response and fix!

It was same for me when I tried to listen to a port on a local node server. Turns out I only did app.get() and didn't wrote app.listen(). The moment I listened on the port, insomnia started working as intended!
Hope that helps!

fatal: unable to access 'https://github.com:Yoni1984/alx-low_level_programming.git/': URL using bad/illegal format or missing URL  my screen shows me this message what is the problem is there any one who help me


Doesn't work with core::error::Error

For some reason publicsuffix::Error implements core::error::Error but not std::error::Error.
I think it should be possible to wrap around that type using just #[from], but currently it's a compilation error.
 1  error[E0599]: the method `as_dyn_error` exists for reference `&Error`, but its trait bounds were not satisfied
   --> src/suffix.rs:23:18
    |
 23 |     ParseError(#[from] Error)
    |                  ^^^^ method cannot be called on `&Error` due to unsatisfied trait bounds
    |
   ::: /home/james/.cargo/registry/src/index.crates.io-6f17d22bba15001f/publicsuffix-2.2.3/src/error.rs:7:1
    |
 7  | pub enum Error {
    | --------------
    | |
    | doesn't satisfy `_: AsDynError<'_>`
    | doesn't satisfy `publicsuffix::Error: std::error::Error`
    |
    = note: the following trait bounds were not satisfied:
            `publicsuffix::Error: std::error::Error`
            which is required by `publicsuffix::Error: thiserror::__private::AsDynError<'_>`
            `&publicsuffix::Error: std::error::Error`
            which is required by `&publicsuffix::Error: thiserror::__private::AsDynError<'_>`

A related issue is this is just not a good error message, but that's probably a bit harder to fix.

core::error::Error and std::error::Error are the same trait. It is not possible to implement one but not the other.
According to this code, publicsuffix::Error implements Error only if feature="std" is enabled. Probably you have not enabled that feature.


[Feature Request] Breadcrumbs block type

Autonav is too slow to show breadcrumbs because it creates navigation from parent to child.
However, we can get parent pages from the current page to render breadcrumbs, and it should be much faster than autonav.
Additionally, it's not friendly to the editors. A lot of my clients asked me how to add breadcrumbs to pages.
IMO version 9 is good timing to add it.

I agree. Auto-Nav was a good idea but really it should just be used for big sitemaps. A dedicated, simple, efficient Breadcrumb Navigation block would be a great addition.

How about this one:
https://www.concrete5.org/marketplace/addons/free-simple-nav-menu-block/

It's GPL3 package, we can't take it in the core

I can make another one with a different handle with any license you wish

Thank you @linuxoid  – but that block does a lot more than just Breadcrumbs, which is what we're looking for. There's no reason this block shouldn't just be named Breadcrumb, with the handle breadcrumb_navigation, that traverses from the home page down to the current page. It probably doesn't need any options, except maybe a checkbox as to whether to include the current page in the breadcrumb trail or not.

Sorry @aembler but I'm confused which one is that and which one is this :)
Should I do anything with my block ab_nav_menu?

@linuxoid not at this time, no. Thank you!

@aembler Two simple options that could be left to view templates, but I believe basic users may appreciate...

Page path separator option (/, >, |, etc), be it a 1 character text input, or a dropdown of options to allow.
Option to keep the Home page out of the crumbs display or use of font awesome in its place to display a house for instance.

Throwing ideas around !!

IMO we don't need to add the separator option, because the markup of bootstrap's breadcrumb doesn't have it
https://getbootstrap.com/docs/4.5/components/breadcrumb/

I don't know if this is not planned anyway, but I would add the option to make that block respect the page attribute "Exclude From Nav".

"Exclude From Nav".

It's a good idea for me too


TEK002: Remove donor page and add separate cards for individual and corporate donors

Requirement

Instead of two separate cards for Individual and Corporate donors showing up on clicking the Donate your smartphone card on the landing page, the idea is to have two separate cards on the landing page itself.

Implementation

We can remove the Donate page and modify the landing page configuration to add the additional card with the appropriate redirect link.

The donatePageCards from the config.js can be removed and added to the homePageCards

The "Donate Your Phone" can be removed from the homePageCards in the config

Additional Considerations

Upon adding an additional card to the landing page, the styles will have to be suitably adjusted to make sure there is no breakage for both small and large screens.

On large screens, the viewport can be made to scroll given five cards might not fit in a single page.

The card text for individual and corporate donors should read:

"Donate a Smartphone as an Individual Donor" for individual donor
"Donate a smartphone as a corporate donor" for corporate donor

mobile screen :

large screen :

mobile screen:

large screen:

 Device Donation Portal


Discrepancy in value, value_start, and value_end in Scan Results

Hi,
I've encountered an issue with the scan results related to the indexes of value_start and value_end not aligning correctly with the value field. I found the discrepancy while scanning a URL containing a Key. When I scan a file containing the following line with default rules in version 1.5.9.
https://maps.google.com/maps/api/js?key=wredzaSyCj4RVnfs3eU72CZXDIB9UG4NRt-VF1yOA&libraries=places

The scan yields the following finding:
[
    {
        "api_validation": "NOT_AVAILABLE",
        "ml_validation": "VALIDATED_KEY",
        "ml_probability": 0.9186561107635498,
        "rule": "Key",
        "severity": "medium",
        "line_data_list": [
            {
                "line": "https://maps.google.com/maps/api/js?key=wredzaSyCj4RVnfs3eU72CZXDIB9UG4NRt-VF1yOA&libraries=places",
                "line_num": 3,
                "path": "test/urls.txt",
                "info": "",
                "value": "wredzaSyCj4RVnfs3eU72CZXDIB9UG4NRt-VF1yOA",
                "value_start": 40,
                "value_end": 98,
                "variable": "key",
                "entropy_validation": {
                    "iterator": "BASE64_CHARS",
                    "entropy": 4.8854165898713005,
                    "valid": true
                }
            }
        ]
    }
]

The value field correctly shows wredzaSyCj4RVnfs3eU72CZXDIB9UG4NRt-VF1yOA. However, the value_start and value_end indexes do not accurately reflect the start and end of this value within the line.
Demonstration using Python:
>>> text = "https://maps.google.com/maps/api/js?key=wredzaSyCj4RVnfs3eU72CZXDIB9UG4NRt-VF1yOA&libraries=places"
>>> print(text[40:98])
'wredzaSyCj4RVnfs3eU72CZXDIB9UG4NRt-VF1yOA&libraries=places'

I'm curious to know if this discrepancy is already a known issue. Ideally, the value should align precisely with the value_startand value_end indexes provided in the result. I tired to look at the code but could not figure out why this discrepancy is occurring.

Thank you for discovering the issue.
Wrong position appears because the position has been got from regex values but after "sanitize" the value is reduced.
https://github.com/Samsung/CredSweeper/blob/ea2ee9605e8683c7da4c600e9fda491635933db0/credsweeper/credentials/line_data.py#L107


WebApp tutorial link above leads to an ASP.NET tutorial, not ASP.NET Core

This is a page for .NET Core, so a bit confusing not having a link to a webapp tutorial for ASP.NET Core.

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

ID: f432109f-6ecb-502b-2612-f8b6eaaf9ffc
Version Independent ID: 4c316f01-502d-d9cf-e5c9-f908357c662d
Content: Azure Cosmos DB: SQL .NET Core API, SDK & resources
Content Source: articles/cosmos-db/sql-api-sdk-dotnet-core.md
Service: cosmos-db
GitHub Login: @rnagpal
Microsoft Alias: rnagpal

@jjgriff93 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Reference:
The tutorial and the code samples are for .NET and not .NET Core.

@jjgriff93 the Get started link in the doc points to the .Net core tutorial: https://docs.microsoft.com/en-gb/azure/cosmos-db/sql-api-dotnetcore-get-started let us know if that doesn't help

Thanks @SnehaGunda , this is related to the issue I've just commented on the other thread. It says Web App tutorial and therefore should be an ASP.NET Core tutorial, as this documentation is a section on getting started with .NET Core - we shouldn't then be linking people through to a .NET WebApp tutorial from this, can be misleading

@jjgriff93 I had a chat about this with our dev team and they confirmed that same sample should work with ASP.Net core as well. Did you had a chance to try it or run into any issues?

@SnehaGunda - there are some inconsistencies which I was able to work through but some people might not be able to:

Configuration works differently in asp.net core - there is no web.config file for you to put the Cosmos DB connection strings in, you have to use appsettings.json and dependency injection instead. For somebody just looking to start out this might take them a while to figure out
You should use the Azure.DocumentDB.Core package instead of the regular one specified
On the step Open Global.asax.cs and add the following line to the Application_Start method - global.asax doesn't exist in ASP.NET Core either, this would need to be put in start up instead

So it doesn't work out of the box with ASP.NET Core, I think a new version that's modified from the .NET example needs to be included so we have every platform covered - this is something that's done in most of our other docs

@jjgriff93 thanks for the detailed feedback.  I have shared this feedback with our dev and PM teams, we have a work item in our backlog to provide new samples for .Net core. Currently, we are revamping some conceptual docs, & how-to guide docs once those are done, we will get to these tutorials probably around end of this month.
We will close this issue and track it as an Azure DevOps work item.

#please-close


FkBroker.java:29-31: Broker has too many methods We...

The puzzle 292-fab4489f from #292 has to be resolved:
https://github.com/eo-cqrs/eo-kafka/blob/34ab706b18fcc368602c1cf04d4a4a3eb69f8d88/src/main/java/io/github/eocqrs/kafka/fake/FkBroker.java#L29-L31
The puzzle was created by @h1alexbel on 12-Jun-23.
Estimate: 60 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0pdd thank you for reporting this. I'll assign someone to take care of it soon.

@l3r8yJ this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-06-22T16:19:05.771309.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

@zoeself deregister

@zoeself deregister

@h1alexbel ok, I've removed this task from scope. I'm not managing it anymore.

@zoeself register

@zoeself register

@h1alexbel I've just registered this ticket as a task and will assign it to someone soon. Thanks!

@h1alexbel this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-06-22T20:09:04.176448.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.


booleanWeekendsColor does not seem to work for mini calendar?

Thank you very much for the nice plugin!
It seems that disabling booleanWeekendsColor in the settings menu doesn't disable the coloring of weekend days in the mini calendar above the title. Is it the intended behavior? Is there a way to change their colors in the mini calendar as well?

Also, is it possible to expose more styling options for the mini calendar in the settings menu? Such as the border color for the current day (green). I would like it to be less prominent. Since the day spans are generated programmatically, it's not straightforward to use custom css to style it.

It seems after refreshing Logseq the colors for weekend days are now correctly disabled in the mini calendar. Would still be nice to be able to set the highlight colors by the users though.

It was a bug that it was not reflected immediately.
Do you want to set colors for each day of the week?

It was a bug that it was not reflected immediately.
Do you want to set colors for each day of the week?

No, I would only like to change the border color of the selected day. Currently it's green if the opened journal page is today. The border is orange (?) if it is a day in the past or in the future. It would be nice to have them as "accent color" in the settings for users to set their colors of choice.


Container options panel not usable with different DPI

I have a 4k screen so I had to increase screen DPI
Windows 10 creators update, staxrip <IP_ADDRESS> was working fine, in staxrip <IP_ADDRESS> the container options panel is not usable
tried to disable DPI scaling in windows, but nothing happened
screenshot:

I believe this was fixed, could you please try the newest test build?
https://github.com/stax76/staxrip/blob/master/md/test-build.md
How much DPI are you using (I use 288 with 4K screen)?

Yes, it has been fixed in the latest test build
DPI... I am using 150% zoom, that means... 144 dpi?
anyway it works, thanks
ps: the onedrive link is broken

Yes, 150% Zoom equals 144 DPI, can you please try the onedrive link again? I just tried it and it worked.

Still doesn't work, if i try in incognito mode it forces me to login - looks like a permission problem

I guess it works only with login, it's OK as long one of the two works otherwise I can try dropbox in the future.


[api-docs] Fixed capitalization in Stage Discovery

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/52f10e8c9a0aa7d2d49fd813373e519d5b67173e

Irrelevant


Touch scrolling

Hi there, I'm using Annotorious for my mobile app and I have found that I cannot scroll on a "zoomed in" image that has Annotorious enabled on it as I presume the touch gets taken up by Annotorious. If I setVisible(false) I can scroll up & down and left and right as standard. Any advice?

Hi,
yes, setVisible will hide the SVG annotation layer (using CSS display:none). Just to confirm: are you using Annotorious standard, or the OpenSeadragon version?
Either way, touch events on the image are, by design, going to trigger drawing (in Annotorious standard) or image navigation (Annotorious OSD). If the image is taking up the whole screen and you still want to be able to scroll the page, there's no way around building your own toggle button. This way, you could keep setVisible to false by default, allowing normal page scroll. And only when the user clicks a "Start Drawing" button (or whatever), setVisible would get switched on, scrolling would be blocked, and drawing would work.


663 Add feature image on markdown

Faq entry about markdwon feature images , related to 663.
Not sure about the title of the link and the post OK abou the content but inputs are welcome.

Thanks a lot for your contribution! That's a very nice idea to add stuff like that <3
Like explained on the chat, we are currently thinking on a better approach to sort all FAQ entries, maybe in subsection with dedicated pages. Your content may even end up in the Getting Started page!
So this will definitely be merged soon, can't tell when, but it will be ;)

Going to make some adjustement & move that in the "Getting started" as it's a nice example! Thank you! ❤


Disabling jobs only in CI.

In https://github.com/coq-community/paramcoq/pull/75, I had to put coqPackages.mathcomp.job = false; because I had also put coqPackages.mathcomp.override.version = "master"; but there was no reason to have a CI job building the full mathcomp.
In https://github.com/jwiegley/category-theory/pull/21, since no Cachix was set, I wanted to avoid the inefficiency of having dependencies built in separate jobs (only to be rebuilt in the main job). Therefore, I had to put a number of job = false; (more for the "dev" bundles than for the others). In the case of Coq, the unwanted consequence is that Coq is silently skipped when we try to build it, and it is only actually built later on, while trying to build ssreflect: https://github.com/Zimmi48/category-theory/runs/3493747463?check_suite_focus=true#step:10:9
There should be a way of excluding a job from CI without making nix-build --job a no-op.

Related to this, something that I still didn't understand is why some jobs marked as job = false; are still showing up in the GitHub Actions generated for this repo (e.g. coq-shell, and heq and mathcomp-bigenough in the master workflow).

@Zimmi48

I had to put coqPackages.mathcomp.job = false; because I had also put coqPackages.mathcomp.override.version = "master";

I do not understand this sentence. Why put job = false?

Because otherwise the generated workflow contained a job building mathcomp (full) even though I only needed one for mathcomp-ssreflect IIRC.

Related to this, something that I still didn't understand is why some jobs marked as job = false; are still showing up in the GitHub Actions generated for this repo (e.g. coq-shell, and heq and mathcomp-bigenough in the master workflow).

This was a bug, I fixed it in #75, why exclude coq-shell, bigenough and heq though?

Coq-shell, I don't know, but the rest of them are not really expected to be compatible with Coq master, are they?

Coq-shell, I don't know, but the rest of them are not really expected to be compatible with Coq master, are they?

I see I misread, I thought all of them were suppressed for released versions.
I do not see why we would remove coq-shell I will keep it.


New extension: Hack Club Ship

Description
Be able to quickly ship and display brand new projects to the Hack Club Slack!
What are ships?
Ships are when you publish your project, and share it with your peers. Hack Club strongly encourages ships, and they have a whole channel in their Slack that allows you to write a message detailing your ship!
I wanted to make an extension on Raycast that could speed up the whole process without ever needing to open Slack, so here it is!
Screencast

Checklist

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

hey! thanks so much for the review.
i'll get on it!

I converted this to a draft until you're ready 🔥


Braintree Drop in Issue in ie with angularjs

We have been trying to integrate braintree dropin in ie11 but the form never loads. Here is the url for the same
https://staging.healthlynked.com/#!/doctors/login?plan=STANDARD.

I was able to load Drop-in on your site in IE11.

Are there any console messages?
Can you provide the exact version and build of your version of IE11? Can you try running any software updates that are available?
Also, are you able to view the demo site in your copy of IE11? https://braintree.github.io/braintree-web-drop-in/

Hi, It seems the error was on my side I was using flex feature in IE which was hiding the content. Thanks allot for the prompt reply.

:+1: Glad you figured it out.


Doesn't work on Nordic NRF51 devboards (PCA10028 etc) - fix supplied

Original Nordic dev boards have somewhat different hw but use same UART TX/RX as the BLE400 board; however they also require RTS/CTS set up.  Adding this at the start of main, to existing 'custom serial pin' stuff; additional setup for the PCA boards..

#ifdef YOTTA_CFG_TXPIN
  #ifdef YOTTA_CFG_RXPIN

    //Nordic NRF51 devboard (PCA10028) and probably others
    //use p0_08 is CTS, 0_09=RxD, 0_10=RTS, P0_11=TXD
    //The onboard J-Link seems to want RTS/CTS set correctly.
    //We need to deactivate the "display" we don't have one anyway - and assign the gpios correctly
    //or the device will transmit over the JLink UART but not receive.
    //Again - this is only for the Nordic PCA devboards.
    #pragma message("Btlejack firmware for Nordic PCA10028 dev board RTS/CTS pins")
    uBit.display.disable();
    DigitalOut cts(P0_8,0); //must set rts/cts for Nordic board
    DigitalIn  rts(P0_10);
    /// End Nordic PCA devboard

    #pragma message("Btlejack firmware will use custom serial pins")
    uBit.serial.redirect(YOTTA_CFG_TXPIN, YOTTA_CFG_RXPIN);

  #endif
#endif

Curious to see if this will work with my nRF52840 boards.

Hey @raplin I've just come across this issue while trying to get btlejack to work for my device, the same PCA10028 board that your fix here is for. I've been having a lot of trouble putting together an environment to properly compile the firmware with the changes you shared in your post, I think the ARM mbed Online has been discontinued and for the life of me I cannot get Yotta installed on any of my systems. You wouldn't happen to have a pre-compiled version of the firmware for the PCA10028 that you could share, would you? I'm sorry to ask like this, especially since it's been so many years since you opened the issue. Thanks.

sorry that was on an old computer, don't have a binary of it any more (nor NRF tools installed on this computer)


Upgrade critools version in 1.6

Description
In 1.6 release, critools version has not been updated for a long time since last year. Should we consider to update the version to 1.29?
Steps to reproduce the issue

Describe the results you received and expected
critools version is updated accordingly
What version of containerd are you using?
1.6.36
Any other relevant information
No response
Show configuration if it is related to CRI plugin.
No response

Fixed in #10929


Immersive reader mode not supported.

This page doesn't support immersive reader mode in the Microsoft Edge Chromium browser.

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

ID: 158425f1-86b1-101a-9172-14c944867784
Version Independent ID: 6386f636-99b3-93c3-ca9e-0134f3e3be10
Content: Methods - C# Guide
Content Source: docs/csharp/methods.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

moving to Microsoftdocs/feedback.

Closed and moved to MicrosoftDocs/feedback#3275


MD009 I can't set br_space parameter

I am trying to set br_spaces to 2 to insert an explicit line break/br element, as suggested here at MD009 documentation
Is my syntax wrong? Thanks for this awesome extension!
"markdownlint.config": {"MD009": 2, false}  // not working
"markdownlint.config": {"MD009": false}     // working...not ideal tough

This is the correct syntax: https://github.com/DavidAnson/markdownlint/blob/master/test/trailing_spaces_br.json

Thank you so much @DavidAnson
Have a nice day Sir!


Modern C++ - [Enhancement]

Describe the enhancement
Use modern C++.

Kind of hard to mind material for this so I will close this issue.


Update Rack dependency to v2.0.0

Is there any value to upgrading the gemspec to use rack >= 2.0?

Not that I'm aware of. As it stands now, it should work with both ~1.0 and ~2.0 to allow people on older apps to continue using it while also supporting Rack 2 (which is a dependency of Rails 5). The support for Rack 2 was added nearly a year ago, and appears to be working just fine judging by the dearth of issues reported.


mros1_reasoner does not store qa values for FG properly in the KB

The desired behavior is that only one value for each QA type is stored in the KB, and that value is continuously updated1
Example of desired printouts:
FG Statuses:
fg_standard      None    [('obs_mvp.safety', 1.0), ('obs_mvp.energy', 0.685)

But we are getting this:
FG Statuses:
fg_standard      None    [('obs_mvp.safety_18', 1.0), ('obs_mvp.safety_36', 1.0), ('obs_mvp.safety_54', 1.0), ('obs_mvp.safety_70', 1.0), ('obs_mvp.safety_88', 1.0), ('obs_mvp.safety_106', 1.0), ('obs_mvp.energy_133', 6.20881714097e-10), ('obs_mvp.safety_124', 1.0)]

and apparently in some runs no energy value is associated to the FG (to be confirmed)
[1] Note: this is not how ontologies are supposed to work, so this requires further analysis from an ontological perspective

Fixed in 7717bcd77ecea96fde5ef58f8cb98420dc8a1890


Update README.md

The project moved from 1.7 ThirdPartyResource to the newer CustomResourceDefinition on Aug 8th 2017 with commit abb279b251f79eaa391afb3f0b1ac60db462da48. Updating the main page with this change.

Hello contributor, thanks for submitting a PR for an this project!
I am the bot who triggers "standard-CI" builds for this project.
As a security measure I will not run automated tests on PRs that are not from white-listed contributors.
In order to allow automated tests to run, please ask one of the project maintainers to review the code and then do one of the following:

Type ci test please on this PR to trigger automated tests for it.
Type ci add to whitelist on this PR to trigger automated tests for it and also add you to the contributor white-list so that your future PRs will be tested automatically.
If you are planning to contribute to more than one project, maybe it's better to ask them to add you to the project organization, so you'll be able to run tests for all the organisation's projects.

Note: You might see multiple messages like this showing up on a single PR, this is due to a known bug in the GitHub trigger plugin on Jenkins, the infra team is working on fixing this via new testing flow in the near future, so we'll be able to post a single message per PR.

Thanks @cliffy94!

ok to test


Version 73cd50432cb8e9d634b04320dff328c066249dcf does compile in release config? (both x86 and x64)

2>------ Build started: Project: peview, Configuration: Release x64 ------
1>C:\Program Files (x86)\Microsoft Visual Studio\Preview\Community\Common7\IDE\VC\VCTargets\Microsoft.CppBuild.targets(402,5): warning MSB8038: Spectre mitigation is enabled but Spectre mitigated libraries are not found.  Verify that the Visual Studio Workload includes the Spectre mitigated libraries.  See https://aka.ms/Ofhn4c for more information.
2>C:\Program Files (x86)\Microsoft Visual Studio\Preview\Community\Common7\IDE\VC\VCTargets\Microsoft.CppBuild.targets(402,5): warning MSB8038: Spectre mitigation is enabled but Spectre mitigated libraries are not found.  Verify that the Visual Studio Workload includes the Spectre mitigated libraries.  See https://aka.ms/Ofhn4c for more information.
2>C:\Program Files (x86)\Microsoft Visual Studio\Preview\Community\Common7\IDE\VC\VCTargets\Microsoft.CppBuild.targets(402,5): warning MSB8038: Spectre mitigation is enabled but Spectre mitigated libraries are not found.  Verify that the Visual Studio Workload includes the Spectre mitigated libraries.  See https://aka.ms/Ofhn4c for more information.
2>Done building project "phlib.vcxproj".
2>LINK : fatal error LNK1181: cannot open input file 'noarg.obj'
2>Done building project "peview.vcxproj" -- FAILED.
1>LINK : fatal error LNK1181: cannot open input file 'noarg.obj'
1>Done building project "ProcessHacker.vcxproj" -- FAILED.

You need to enable the Windows SDK (Spectre) package using Visual Studio for compiling Release builds.

cool. thanks!

Hey janwilmans,
Can you describe how you fixed the issue? I have been googling about enabling the Windows SDK Spectre package but to no avail. I just need to build the code in Release, yet I am still getting the same problem.
Best,

Ed

@ERGeorgiev
You just select the spectre libraries in the Visual Studio installer. See screenshot:

@ERGeorgiev
You just select the spectre libraries in the Visual Studio installer. See screenshot:

Found it in the C++ Development package in Visual Studio Installer for 2017 (haven't moved to 2019 yet). Thank you!


The time label is not match crosshair's time? How to fix it?

The label day is not match crosshair date?  There is a api to change the label date?

Can you please provide steps to reproduce the issue? Can you provide a code to reproduce it as well?

Can you please provide steps to reproduce the issue? Can you provide a code to reproduce it as well?

@timocov
It is found that there is a problem with the time transmitted from the background, which is transmitted in UTC time, which conflicts with the local time.

which conflicts with the local time.

What do you mean?


Revert the new JavaVersion#getMajorVersionNumber method (#29176)

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle merge

Can't merge a draft PR.

WARN: Based on labels, this pull request addresses notable issue but no changes to release note found.

The merge queue build has failed. Click here to see all failures.

WARN: Based on labels, this pull request addresses notable issue but no changes to release note found.


Create a modal to answer questions from recommandation

Create a modal template to answer questions
Handle request to retrieve questions files

Due to wrong structure - the structure will be fixed in this issue to (ex: create id correspondence for the front)


How to disable logging when migrating

Hello,
I'm running artisan migrate:reset on my app before seeding new data for testing. Problem is that the package tries to log even when the activity_log table has been deleted.
Any way around that? Or should I make pull request with the solution?
Best wishes,
Siim

I'd accept a PR that fixes that problem. Be sure to include a test as well.

Resolved with PR#51

Thank you very much for that PR.
Changes tagged as version 1.7.0


Interesting project name

The name of this project is interesting, what's the purpose of this project?
Hi from Indonesia! #openedx

Hello.
It's a python version of a machine learning application I'm doing for my master thesis.


File versions API is not described in the docs

The API for manipulating previous versions of a file for which tests are exist is not described in the docs.

Does this mean onedrive API supports listing and downloading past versions of files?

It means we don't yet support this feature. But we might be working on it.

I'm going to close this out as by design right now, since we don't offer this functionality in the API, but hopefully this will be coming soon.

Can you by any chance share the unsupported api endpoint to download a specific file version, if such an endpoint exists?

The tests were written for OneDrive personal, which has a /versions collection off the item objects. Support for ODB is still being worked on, but I hope to have a better story to share in a few weeks time.


basis_universal: Unbundle jpgd to fix symbol conflict, use our newer copy with SSE2 support

Should fix the basis_universal LTO warning from #88504:
thirdparty/basis_universal/encoder/jpgd.h:123:15: warning: type 'struct jpeg_decoder' violates the C++ One Definition Rule [-Wodr]
  123 |         class jpeg_decoder
      |               ^
thirdparty/jpeg-compressor/jpgd.h:124: note: a different type is defined in another translation unit
  124 |         class jpeg_decoder
      |
thirdparty/jpeg-compressor/jpgd.h:276: note: the first difference of corresponding definitions is field 'm_has_sse2'
  276 |                 bool m_has_sse2;
      |
thirdparty/jpeg-compressor/jpgd.h:124: note: a type with different number of fields is defined in another translation unit
  124 |         class jpeg_decoder
      |

Our thirdparty/jpeg-compressor copy is the latest from https://github.com/richgel999/jpeg-compressor, which has some additional commits not present in the basisu version. I'll see if Rich is interested in updating this dep upstream too.
CC @BlueCube3310

This seems to make the binary slightly larger for some reason (Linux x86_64 editor build with optimize=speed lto=full and stripped symbols):
107,091,040  godot.linuxbsd.editor.x86_64  <-- This PR
107,086,912  godot.linuxbsd.editor.x86_64.master

That's pretty weird, but I can reproduce the size difference too with scons production=yes builds on Fedora 39:
$ ls -l bin/godot.linuxbsd.editor.x86_64*
-rwxr-xr-x. 1 akien akien 100693632 Feb 18 22:10 bin/godot.linuxbsd.editor.x86_64
-rwxr-xr-x. 1 akien akien 100701824 Feb 18 22:11 bin/godot.linuxbsd.editor.x86_64.pr88508

It's a bit counterintuitive...

So I think the size difference can be explained by the SSE2 support code present in thirdparty/jpeg-compressor which didn't exist in the basisu copy.
I made a test build of this PR + the SSE2 support disabled, and the size exactly the same as master:
-rwxr-xr-x. 1 akien akien 100693632 Feb 18 22:19 bin/godot.linuxbsd.editor.x86_64.master
-rwxr-xr-x. 1 akien akien 100701824 Feb 18 22:11 bin/godot.linuxbsd.editor.x86_64.pr88508
-rwxr-xr-x. 1 akien akien 100693632 Feb 18 22:25 bin/godot.linuxbsd.editor.x86_64.pr88508+no_sse2

So this means this LTO warning wasn't benign, but actually a sign that the basisu copy of jpgd was taking precedence over the copy in jpeg-compressor, and thus we were losing the (enabled by default) SSE2 support.
So this should fix a bug for regular JPEG decompression, which may become faster? The size increase comes from the added SSE2 code.

Ran a test import 500 JPGs with master, this PR, and this PR + JPG_USE_SSE2=0 which should be equivalent to master...
Running on a Linux laptop with balanced CPU profile, on AC. The results aren't exactly clear.
$ hyperfine -iw3 -p "rm -rf .godot" "~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.master -e" "~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508 -e" "~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508+no_sse2 -e"
Benchmark 1: ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.master -e
  Time (mean ± σ):     20.479 s ±  0.232 s    [User: 175.239 s, System: 8.655 s]
  Range (min … max):   20.079 s … 20.864 s    10 runs

Benchmark 2: ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508 -e
  Time (mean ± σ):     19.680 s ±  0.384 s    [User: 167.595 s, System: 8.489 s]
  Range (min … max):   18.658 s … 20.009 s    10 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet system without any interferences from other programs.

Benchmark 3: ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508+no_sse2 -e
  Time (mean ± σ):     19.678 s ±  0.203 s    [User: 168.102 s, System: 8.064 s]
  Range (min … max):   19.456 s … 20.125 s    10 runs

Summary
  ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508+no_sse2 -e ran
    1.00 ± 0.02 times faster than ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508 -e
    1.04 ± 0.02 times faster than ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.master -e

Same benchmark with performance CPU profile:
$ hyperfine -iw2 -r5 -p "rm -rf .godot" "~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.master -e" "~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508 -e" "~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508+no_sse2 -e"
Benchmark 1: ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.master -e
  Time (mean ± σ):     16.587 s ±  0.103 s    [User: 143.993 s, System: 6.877 s]
  Range (min … max):   16.523 s … 16.766 s    5 runs

Benchmark 2: ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508 -e
  Time (mean ± σ):     16.158 s ±  0.145 s    [User: 138.665 s, System: 6.785 s]
  Range (min … max):   15.975 s … 16.380 s    5 runs

Benchmark 3: ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508+no_sse2 -e
  Time (mean ± σ):     16.084 s ±  0.333 s    [User: 138.148 s, System: 7.016 s]
  Range (min … max):   15.512 s … 16.323 s    5 runs

Summary
  ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508+no_sse2 -e ran
    1.00 ± 0.02 times faster than ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.pr88508 -e
    1.03 ± 0.02 times faster than ~/Godot/godot/bin/godot.linuxbsd.editor.x86_64.master -e

So there seems to be a significant JPG import performance gain between this PR (with or without SSE2) and master, though I can't really explain it. The difference between pr88508 and pr88508+no_sse2 doesn't seem meaningful, but then it seems to mean the SSE2 code doesn't bring significant performance improvements for me (on x86_64).

Cherry-picked for 4.2.2.


[Feature] Download as mp4

I would love to be able to download an asciicast as a video format to upload it to YouTube and other video sharing sites.

Generating video files from the recordings is not something I consider in the scope of asciinema-server project, but...
This can be achieved with asciicast2gif + converting the gif to a video format.

Generating video files from the recordings is not something I consider in the scope of asciinema-server project, but...
This can be achieved with asciicast2gif + converting the gif to a video format.