argocd:make install idempotent

Signed-off-by: Xieql<EMAIL_ADDRESS>What type of PR is this?
/kind bug
What this PR does / why we need it:
part of #53
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

BTW, please split this to two commits:

fix argcd issue
refactor typemeta


Enhance logging capabilities

This PR allows to output the test results as plain text, or csv; and/or to output the result to a file

Thanks!


DFS : Connected Cell in a grid in C++

Related Issue
Solved issue number: #365
Describe the changes you've made
Adding Source Code for DFS : Connected Cell in a grid in C++
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Language used for coding
C++
Checklist:

[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have documented the code.
[x] New and existing unit tests pass locally with my changes.

Please review the code ;-)


Support for running in Linux virtual server

Although it can run successfully in linux, for virtual server like wsl, vps, etc. it is unrealistic to install firefox. Just like this #30.
Is there any way to run this script successfully on a virtual server?

btw, It is also difficult to use github actions.

Unfortunately there's no way to get the script to run without a browser of some kind (at least that I've found)


[ADD]: Data Visualization Application Using JAVA

Describe the project you want to add with tech stack

The desktop application provides the means to convey the data of the library to the members of the library.
made using java swing
Expected behavior

Slideshow of the data in a visual and easy to understand manner.
Users(Admin) can add/update/delete the library data according to the requirement based on which graphs are also updated.

@MohitGupta121 @TusharKesarwani Can you assign me this issue.


tab completion not working

I have two accounts on my Macbook, both are installed oh-my-zsh. But one account cannot use the tab completion. When I uninstalled oh-my-zsh, tab completion  worked well.

This seems like a non-reproducible error.

It is happening to me too, even after updating to last version the tab completion isn't working. Not sure what happened, it suddenly stopped.

it happened to me too, i update the brew, zsh,  oh my zsh, still not working. I have tried to remove the dump file, but not work.

it happened to me too,tab working on zsh without ohmyzsh,but can not work with ohmyzsh,

I have solved it!  in .oh-my-zsh/oh-my-zsh.sh, look at 14th line:
:${ZSH_DISABLE_COMPFIX:=true}
turn true to false,  save it,then execute  source ~/.zshrc ,restart your terminal,you will know some directory which are insecure,then execute chmod 755 <insecure-directory>,
at last，back to .oh-my-zsh/oh-my-zsh.sh,the 14th line,turn false to true, and source ~/.zshrc
it will be solved!!!

@1nv3rse Awesome, it worked for me, thank you.

@1nv3rse  U are really awesome, it's really useful!!! Thanks very much!!

its not work on me -_-

I removed the account which I used to install oh-my-zsh,  and also remove oh-my-zsh, then rebuild an account and reinstall oh-my-zsh, finally everything back to normal.

I started having the same problem but I could find no solution. I have two users and I need oh-my-sh to work for both, applying @1nv3rse solution has shown it is complaining about Cellar being gropu writable or writable on by another user. Is there a way to disable that group check?

I, too, have this problem, but, as @snood1205 mentioned, it appears to be a non-reproducible error. At the time of this writing, I'm on the up-to-date version of omz: b908feeb.

I had a similar problem but it seems like it's specific to iTerm2 - tab completion fails in zsh or bash using iTerm.app, but succeeds in either shell using Terminal.app (maybe your problem is different, or maybe the issue is not with zsh)

@1nv3rse, You are awesome!

@1nv3rse You are awesome!

@1nv3rse For me, simply setting ZSH_DISABLE_COMPFIX to false, executing ,.zshrc and opening a new tab was sufficient. I don't get any warnings about any insecure directory.
I suspect that installing an OSX update my cause tab completion to break, as it was working shortly before I applied the latest Sierra update.

guys it ain't working for me, It works fine with the normal terminal, but not with this one.. Halp?

I just started having this issue after my Mac login was moved to a network login. When I tried to uninstall/re-install zsh, I found I no longer had permissions to several of the directories. Once I updated the permissions for my current user, tab-completion began working again.

I happened to solve this a few time ago, been too busy too reply, sorry. I delete the account and recreate the same one, then everything works fine. Don't know why. Maybe it's the problem of my macbook.

Interesting I have to source ~/.zshrc in a zsh shell to get various auto bash completion to work (e.g aws zsh autocomplete) while a new session in tmux didn't work, regardless of setting true/false on ZSH_DISABLE_COMPFIX.
I really have no explanation for this behavior, I don't really have any file that should load after .zshrc (according to https://shreevatsa.wordpress.com/2008/03/30/zshbash-startup-files-loading-order-bashrc-zshrc-etc/)

thx a lot!

It works for me. thanks a lot! @1nv3rse

@AnthonyWC I had the exact same issue as you.  When new terminal windows opened, I'd have to then re-source my ~/.zshrc before my aws autocompletion would work.  For me, the issue ended up that I was installing pip within a pyenv that was sourced/activated later in the file after the oh-my-zsh invocation.  I guess it inherited the path to the aws_zsh_completer.sh script from that later activation when I sourced the file again manually.
Moving up the pyenv activation to before ZSH fixed it for me!

If you're seeing this, I've just updated OMZ to ignore permission issues with your completion scripts and load them anyway, but you have to set this in your zshrc file:
# this goes before oh-my-zsh.sh is sourced
ZSH_DISABLE_COMPFIX=true

For me, there were no permission issues on my completion scripts so the above didn't fix my issue. Since this the top thread for tab completion not working in oh-my-zsh, I figured I'd post my solution for anyone in the same boat.
If the above doesn't work for you, try removing any .zcompdump files in your home directory. Credit: https://unix.stackexchange.com/a/210931.
rm ~/.zcompdump*

For me, there were no permission issues on my completion scripts so the above didn't fix my issue. Since this the top thread for tab completion not working in oh-my-zsh, I figured I'd post my solution for anyone in the same boat.
If the above doesn't work for you, try removing any .zcompdump files in your home directory. Credit: https://unix.stackexchange.com/a/210931.
rm ~/.zcompdump*

This was to fix for me after trying all the other options. I would like to know what could cause that problem though so if posible we can add a fix to prevent this from happening again in the future.

I have tried all the solutions above and NONE of them work, and test under iTerm2 and Terminal. 😭 😭 😭

I have tried all the solutions above and NONE of them work, and test under iTerm2 and Terminal. sob sob sob

meet the same problem, tried too many way
emmm , apt remove --purge zsh then install zsh
it work

Guys! It's more simple than that.
Just add a single space in front of the opening/closing parenthesis, execute a source ˜/.zshrc and all plugins start working!

For me, there were no permission issues on my completion scripts so the above didn't fix my issue. Since this the top thread for tab completion not working in oh-my-zsh, I figured I'd post my solution for anyone in the same boat.
If the above doesn't work for you, try removing any .zcompdump files in your home directory. Credit: https://unix.stackexchange.com/a/210931.
rm ~/.zcompdump*

This fixed my problem. Thank you!

@claymcleod Give That Man a Bells! Thanks rm ~/.zcompdump* solved my problem!


fix: allow users to scroll to bottom of multis-elect filters

The type of this PR is: Bugfix
Description
When a user scrolls to the bottom of a long multi-select filter (like the categories filter on an artwork grid) they are not able to keep the bottom 2 options in view long enough to select them before the snap to hide behind the "Apply" button.

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

@dzucconi I assigned this ticket to you because we spoke about it during our mob session on Monday.


Make translator friendly

Some languages such as Japanese use different word order with
English. If the stream name must be the last word, translating to these
languages is difficult.

Automated message from Dropbox CLA bot
@kou, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @kou!


Makefile: Restrict $KUBECONFIG definition to 'test' target (#8697)

Instead of making it global. It was causing issues on some e2e
tests.

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

Needs approval from an approver in each of these files:

OWNERS

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

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
4db0b3264d58432e852b134cee607588a1e222cc
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai I have the same fix in my PR, and does not fix anything :-(

@hklai: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
f8f51fc578d01c2e7dc5365589bcdd362daa86e6
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


/ob/purgecache is hanging

The API call doesn't seem to be working and is hanging somewhere.

No hanging after rebase


Publish Helm chart to an OCI-compatible repo

Could you push the packaged Helm chart also on ghcr.io, so we can pull/install the helm chart like this:
helm install keptn oci://ghcr.io/keptn/lifecycle-toolkit-charts --version x.y.z

What is the reasoning/use case behind this? Is there anything that is not supported by normal helm repositories?


Bad email address in bulk emailing

Hi,
I am not sure if this is more of an issue on the gem side or on Mandrill's mail server side.
I tried sending to multiple recipients at once and sometimes it fails with the following exception:
Error Class 	Net::SMTPServerBusy
Error Message 	401 4.1.3 Bad recipient address syntax
In the past this has happened when I have email addresses with bad format, e.g<EMAIL_ADDRESS>(double dots). Note that I've done regex validation but some addresses just slip past my validation -- that's the reality of regex email validation (See http://stackoverflow.com/questions/201323/using-a-regular-expression-to-validate-an-email-address/719543#719543)
When sending emails in bulk, it's not possible for me to find out which email address is bad because I am sending to 1000 recipients in one go.
The sending fails for that whole batch so that even though I have an auto-retry solution (using sidekiq), this batch always keeps failing.
Ideally the gem or Mandril should tell me which of the email addresses is invalid (may be via webhook or using a more explicit exception that can be rescued and processed), so that I can blacklist the offending address. So even though the sending fails the first time, it will work on the next retry because by that time I've blacklisted the bad address.
Thanks

@hgani the mandrill-rails gem is not involved in mail sending .. that's a conversation between your stack and the SMTP gateway. NB: tho you can use mandrill-rails to receive inbound mail and webhook callbacks from mandrill related to mail delivery. So this is not really the place to find a good solution .. hence why I'm closing the issue, sorry!
That being said, yes, invalid addresses often cause the SMTP gateway to immediately reject the mail even before it gets into the queue (and not give you much information in the process).
I've tried various regex/gem solutions for email address validation, but none are perfect. Rather than try and re-invent the wheel, I've found the most reliable approach is to leverage some stuff lurking in ActionMailer (assuming you are sending from Rails). There isn't an obvious interface for this (I don't think it was an anticipated use-case), but here's one trick:
  def invalid_email?(email)
    Mail::ToField.new email
    raise Mail::Field::ParseError.new('Mail::AddressListsParser', email, "email is not fully qualified") unless email =~ /@/
    false
  rescue Mail::Field::ParseError => e
    "(#{e.message})"
  end

invalid_email<EMAIL_ADDRESS> => false
invalid_email<EMAIL_ADDRESS> => "(Mail::AddressListsParser can not parse<EMAIL_ADDRESS>was: Expected one of !, #, $, %, &, ', *, +, -, /, =, ?, ^, _, `, {, |, }, ~, \r\n, (, \", ., @, :, < at line 1, column 15 (byte 15) after .com)"
invalid_email? "test"
 => "(Mail::AddressListsParser can not parse |test|\nReason was: email is not fully qualified)"

Note: the reason the custom raise Mail::Field::ParseError is to also handle cases where email does not have an @domain qualifier (Mail::ToField.new "test" is considered "valid")

@tardate Thanks for the quick response and detailed explanation.
I've implemented something very similar to your suggested solution
But I decided to use a gem (https://github.com/validates-email-format-of/validates_email_format_of) because the gem claims to adhere to RFC 2822 and RFC 3696 which should be compatible with Mandrill (on the other hand, Mail::ToField doesn't seem to be well documented so perhaps it's not meant to be used publicly)
def self.email_invalid?(email)
  ValidatesEmailFormatOf::validate_email_format(email).present?
end


Uppercase keyspace causes Kong migrations to fail

$ kong start
[INFO] Using configuration: /etc/kong/kong.yml
[INFO] Proxy port.........8000
       Admin API port.....8001
       Database...........cassandra keepalive=60000 port=9042 timeout=1000 hosts=kongdb.org keyspace=Yoda
[INFO] Database not initialized. Running migrations...
[ERR] Cassandra error: Cassandra returned error (Invalid): "Keyspace 'yoda' does not exist"

The keyspace is lowercase in the error so I changed it to lowercase in kong.yml and it then worked.

Actually migrations are successful but a keyspace will always be lower case, so a fix would be to force it to be lower case in the configuration loading I guess

Sorry, I meant the keyspace creation is successful*

@thibaultCha is this closed?

No

Ok fixed by #271!


update for google cloud platform source for additional step for creat…

…ing log sync
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@sumoanema, I went ahead and updated the article with my suggestions


x/tools/gopls: symbol definitions not found after latest vscode/gopls upgrade

What did you do?
Clone a github repo, e.g. https://github.com/eliben/gemini-cli
$ cd gemini-cli
$ code .
Navigate to main.go and click "Go to definition" on the Execute symbol on line 10
What did you expect to see?
Expected to jump to definition.
What did you see instead?
"No definition found for Execute" popup
Build info
golang.org/x/tools/gopls v0.15.2
<EMAIL_ADDRESS>h1:4JKt4inO8JaFW3l/Fh9X1k/5JQn+iUOpdc4/Lpi0mOs=
<EMAIL_ADDRESS>h1:9F2/+DoOYIOksmaJFPw1tGFy1eDnIJXg+UHjuD8lTak=
<EMAIL_ADDRESS>h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
<EMAIL_ADDRESS>h1:2O2DON6y3XMJiQRAS1UWU+54aec2uopH3x7MAiqGW6Y=
<EMAIL_ADDRESS>h1:SernR4v+D55NyBH2QiEQrlBAnj1ECL6AGrA5+dPaMY8=
<EMAIL_ADDRESS>h1:5BMeUDZ7vkXGfEr1x9B4bRcTH4lpkTkpdh0T/J+qjbQ=
<EMAIL_ADDRESS>h1:vcVnuftN4J4UKLRcgetjzfU9FjjgXUUYUc3JhFplgV4=
<EMAIL_ADDRESS>h1:ScX5w1eTa3QqT8oi6+ziP7dTV1S2+ALU0bI+0zXKWiQ=
<EMAIL_ADDRESS>h1:No0LMXYFkp3j4oEsPdtY8LUQz33gu79Rm9DE+izMeGQ=
<EMAIL_ADDRESS>h1:KUas02EjQK5LTuIx1OylBQdKKZ9jeugs+HiqO5HormU=
<EMAIL_ADDRESS>h1:oFEHCKeID7to/3autwsWfnuv69j3NsfcXbvJKuIcep8=
<EMAIL_ADDRESS>h1:G3QvahNDmpD+Aek/bNOLrFR2XC6ZAdo62dZu65gmwGo=
<EMAIL_ADDRESS>h1:lyBNOm8Wo71UknhUs4QTFUNNMyxy2JEIaKKo0RWOh+8=
go: go1.21.1

This happened today, after an auto-upgrade of vscode.
$ code --version
1.87.2
863d2581ecda6849923a2118d93a088b0745d9d6
x64

I don't see errors in the gopls (server) in the Output terminal. Extension Host terminal says:
2024-04-04 06:36:26.948 [info] ExtensionService#_doActivateExtension golang.go, startup: false, activationEvent: 'onLanguage:go'

Hi Eli, thanks for filing.
I can reproduce this by installing gopls with go 1.21, and it is fixed by installing gopls with go 1.22.  I believe VS Code should have prompted to reinstall tools upon the version mismatch, but it didn't, for me (CC @hyangah).
I'm haven't yet investigated the failure mode, which is rather disconcerting: no errors, just a broken experience.

This is a manifestation of #66525: (1) go/types fails fast with an error when the language version is newer than its runtime version, and (2) this error is not reported through the error handler, therefore not seen by gopls. Hence the silent brokenness of your workspace: go list happily lists packages and gopls happily type checks them, but they are empty.
We need to guard against this when built with 1.21.x, since this is a very bad failure mode. We should capture a synthetic type checking diagnostic, and unset types.Config.GoVersion to proceed with type checking.

Thanks for the quick response, @findleyr
Indeed, removing gopls and re-installing it with 1.22 solved the issue for me.

@hyangah -- I'm sorry: I just tried again and got the popup (but it went away after ~10s). I must have missed it.


⚠ Bofrost.be has degraded performance

In 21fca33, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 890 ms

Resolved: Bofrost.be performance has improved in 3bca938.


Manual models and TabularSHAP

I am working on a ML solution that uses an ensemble approach to classification. We train two base models and combine them with other features in a final score with a random forest. I would love to implement explainability with TabularSHAP but the specifics of our use case mean this is tricky. We can run TabularSHAP on each model individually but I would love to implement a version that returns SHAP values on untransformed inputs. The SHAP values for the random forest return the two input model results as the most important features.
I tried to create a class that combines the two model predictions and scores with the random forest in a transform method. This method works when run independently but fails when used in TabularSHAP (I'm running into threadlock errors).
I am able to run a similar script with models created in sklearn and using the Python shap library and it does produce appropriate SHAP values for raw features.
Are there any recommended approaches for "manual" ensemble models in this context?

Can you combine the three models in a PipelineModel and run TabularSHAP on that?

Yes, it looks like is possible to combine the models in a PipelineModel. I will have to change how I've trained the random forest since I would have to rely on a custom transformer in the PipelineModel to use it as it is currently set up. That version failed with an error: object has no attribute '_to_java'.


Bug located in api reference not published for topodata

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
src/ansys/meshing/prime/autogen/topodata.py not generating api references.
📝 Steps to reproduce
No api docs generated for this autogen file
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
NA
🐍 Which Python version are you using?
3.8
📦 Installed packages
NA

Added tfs bug to track the same
https://tfs.ansys.com:8443/tfs/ANSYS_Development/Portfolio/_workitems/edit/953912/


Pointer Events: FrameContext.Pointers looses pointer positions

FrameContext.pointers is meant to keep track of the current position/tilt/buttons etc of all of the pointers being used. However, currently as soon as the main pointer (in my scenario a mouse) stops moving the pointer is removed from the Batch of pointers. I suspect this is because we treat the pointers in a similar way to the mouse events, but it needs a bit more thinking about and tinkering with

I've had a quick look and the implementations are very different. I think Mouse could do with a bit of a clean up, and Pointers probably needs to work a bit more like mouse does.
It would be helpful to list a couple of examples of the expected / desired behaviours. (I don't really use Pointers :sweat_smile:)


get() method

Currently get() is useless:

Its $params doesn't do anything.
It is useless and is there only because FactoryInterface inherits ContainerInterface.

How about removing get() along with inheritance from ContainerInteface?

I was confused by the very existence of the container in this library, and even more so by the inheritance of the Factory from the Container. These are essentially different things, albeit similar in method names. The purpose of the Factory is to create objects by parameters, and the purpose of the Container is to store entry by IDs and give object on demand.
The user code could use Factory in the ContainerInterface::get(id) method. With the current state of affairs, it is not clear to me where to use your factory, which is also a container.

Unfortunately it cannot be simply deleted.
It is required for definitions to work.
See https://github.com/yiisoft/factory/blob/master/src/Definitions/DefinitionInterface.php
public function resolve(ContainerInterface $container);

Some workaround is necessary.


UCP: Print panic info into tikv_stderr.log

Description
Print panic information into tikv_stderr.log to make it easier to search.
Related code is here https://github.com/tikv/tikv/blob/master/components/tikv_util/src/lib.rs#L466
Difficulty

Easy

Score

200

Mentor(s)

@hicqu

Recommended Skills

Rust

Learning Materials

Standard error output is redirected into tikv_stderr.log
Grep panic_hook in TiKV source code tree to find where need to change.

/pick-up-challenge

/pick-up-chanllenge

@hicqu this is pr

/pick-up-challenge

/pick-up-challenge

I think there are several things need to be considered:

tikv_stderr.log doesn't rotate, the file can become very large if tikv keeps panicking.
I remember tikv_stderr.log was truncated on every start in tidb-ansible deployment, should check if it's still true for latest version and tiup deployment.
Searching may not be easy as it seems. When searching panics, there will be contextual information needs to be looked up. Print to stderr will have to search twice.
It seems inconsistent when users specify --log-file but get some logs in stderr.

@BusyJay Do you mean a fix on this issue might be less useful? If it's true, should I stop working on this issue?
I have read the releated code, I think the 4th consideration you point out is expected behavior, because if --log-file specified, tikv will use an async file logger, when it panics, the global logger will be swithed to a terminal logger wrapper stderr, all the pending logs will be logged to stderr.

...should I stop working on this issue...

You can still work on the issue, but I think there should be more discussions. For example, adding a configuration to log the trace to stderr in addition to log seems a balance between context and search.
/cc @hicqu

..when it panics, the global logger will be swithed to a stderr terminal logger wrapper...

What you mention is implement details, what I'm talking about is the use experience.

Users also cannot search in tikv_stderr.log when using TiDB Dashboard, so this change may make it harder to search.


Feature/add domains to structure

Description
In this feature I added the begin of DDD implementation for in the future help on increase the package on integrate with others API's.
Types of changes

[x] New feature - Added DDD to base structure

Thanks @MichaelKlemersson, but unfortunately I have no intention  of integrinting with others  API's, I wanna keep this package as small as possible, so I think that this new strucuture wont be needed right now.
I was thinking in add a service provider for Laravel, but I dont think that will be needed too, that package is small and simple enough, a service provider would only get in the way of that.
But I do like the idea of validating the dimensions before sending requests to the Correios webservice, so if you could keep that feature, but in the same current structure, that would be awesome :D
Again, many thanks :)


Update PowerShell completions script

Walks up the directory tree to find the gulpfile
supports gulpfiles of other languages (ie, iced coffeescript)

No more changes are being made to the CLI in gulp master. Please submit to the gulp-cli repo for us to review.


Audio not working anymore

I think return audio(speech).play(stream_url) isn't working anymore.
Alexa replies "Something went wrong with the response".
I tried to execute the exact example demo inside folder "audio".

Can anyone else confirm with an existing skill? I can't reproduce this.

I tried three examples. Both from the audio folder and one of my own. All three played.

Thank you @Christina96 - closing this issue until more info is provided or someone can reproduce and include debug info / stacktraces

I had a similar issue, and in my case it was due to the following error
The requested skill has not declared that it implements the AudioPlayer interface. Please update the skill's configuration from the Developer Console.
The library only returns a 400 error and fails silently. It is probably a good idea to let errors like these bubble up.

When you fill the Skill information (Alexa Skill Kit), did you check the field for Audio Player?

yes. I fixed the problem before writing the comment. thanks. My comment was mostly about making the errors more visible to the user.

hi @Christina96 ,
I already checked the field for Audio Player
But when I tried 2 projects from the audio sample folder
Alexa didn't play any sound from url. Alexa just return the statement then I didn't hear any song.
I copied those mp3 url and open them in my browser, those mp3 url are still alive.

I tried to run the Audio samples... but couldn't... the https respose is INTERNAL SERVER ERROR and here is the stack trace
Traceback (most recent call last):
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1985, in wsgi_app
response = self.handle_exception(e)
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1540, in handle_exception
reraise(exc_type, exc_value, tb)
File "C:\apps\Python\Python36-32\lib\site-packages\flask_compat.py", line 33, in reraise
raise value
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1614, in full_dispatch_request
rv = self.handle_user_exception(e)
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1517, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "C:\apps\Python\Python36-32\lib\site-packages\flask_compat.py", line 33, in reraise
raise value
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "C:\apps\Python\Python36-32\lib\site-packages\flask\app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "C:\apps\Python\Python36-32\lib\site-packages\flask_ask\core.py", line 594, in _flask_view_func
result = self._map_intent_to_view_func(self.request.intent)()
File "C:\apps\PycharmProjects\TestProj\Alexa\ask_audio.py", line 26, in demo
return audio(speech).play(stream_url, offset=93000)
File "C:\apps\Python\Python36-32\lib\site-packages\flask_ask\models.py", line 154, in play
directive['audioItem'] = self._audio_item(stream_url=stream_url, offset=offset)
File "C:\apps\Python\Python36-32\lib\site-packages\flask_ask\models.py", line 208, in _audio_item
push_stream(stream_cache, context['System']['user']['userId'], stream)
File "C:\apps\Python\Python36-32\lib\site-packages\werkzeug\local.py", line 377, in
getitem = lambda x, i: x._get_current_object()[i]
TypeError: 'NoneType' object is not subscriptable

seems like the issue is with this statement:
return audio(speech).play(stream_url, offset=93000)

@skgali how are you sending requests to the application? Chance are you're not forming a valid context object with a userId in your request.
Make sure you're structuring requests containing context objects per Amazon's examples.
If you're still experiencing issues after adding contexts, please open a separate issue so we can track this more specifically as an issue related to context processing when handling audio requests.


Serialising NaN float produces invalid JSON.

Serialising an arrow vector (?) of dataType Float with a NaN value produces invalid JSON.
I hope it's ok that I have a test from Polars as that is the way I interact with arrow.
    #[test]
    fn serialize_row_with_nan_float() -> Result<()> {
        let mut df = df! {
            "a" => [4.0, 0.0,f64::NAN]
        }?;
        let mut buf = Vec::new();
        JsonWriter::new(&mut buf)
            .with_json_format(JsonFormat::Json)
            .finish(&mut df)?;
        let encoded = std::str::from_utf8(&buf).unwrap();
        dbg!(encoded);
        let test_res = serde_json::json!([4.0, 0.0, f64::NAN]);
        dbg!(test_res);
        let res: serde_json::Value = serde_json::from_slice(&buf)?;
        dbg!(res);
        Ok(())
    }

produces:
[src/test.rs:775] encoded = "[{\"a\":4.0},{\"a\":0.0},{\"a\":NaN}]"
[src/test.rs:778] test_res = Array([
    Number(
        4.0,
    ),
    Number(
        0.0,
    ),
    Null,
])
Error: expected value at line 1 column 27

Above can be seen that the arrow JSON wrote: {\"a\":NaN} but the normal rust serde_json writes Null for the NaN value.
If needed I can try rewriting the test into arrow2 specific code.

Indeed - well spotted. This is a good first issue - let me know if you would like to PR :)


Alpha bug: Empty list display

An empty list of links should be more descriptive such as informing users nothing was found rather than displaying an ambiguous white box

Related to #126


unicode-font got error

emacs 30.0.50 native-comp
manjaro linux

It seems to be an Emacs issue https://github.com/emacs-mirror/emacs/commit/263d6c38539691c954f4c3057cbe8d5468499b91#commitcomment-105809455

thks : )


Add compatibility with awscli v2

This addresses the issues I was running into with https://github.com/emdgroup/awscli-s3touch/issues/10
I'm not sure if this is the way you'd want to do it but it allow it to be installed and used with the v2 client.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Andrew Morton seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fire and forget Hyperlink handling to break deadlock

Fire and forget on the hyperlink handler inside the TermControl.
PR Checklist

[x] Closes #7994
[x] Tested manually
[x] Hi, I work here.

Detailed Description of the Pull Request / Additional comments
In TermControl, _HyperlinkHandler is called by _PointerPressedHandler which has taken a write lock for all its friends. However, _HyperlinkHandler downstreams to ShellExecute which can pump the message queue looking for something. That pumping of the queue can trigger messages that also want the write lock to update state. They get stuck. Everything hangs.
_HyperlinkHandler really only needs read lock and really only for as long as it takes to fill up its parameters before it's invoked... but the simpler and more contained solution is to just fire and forget the rest of the method that causes the deadlock to a continuation at the tail of the dispatcher queue so _PointerPressedHandler can complete and naturally drop the write lock.
Validation Steps Performed

Launched main manually on my box and clicked the hyperlink that is detected when Powershell starts and it froze.
Launched this change manually on my box and clicked the hyperlink that is detected when Powershell starts and it did not freeze.

Over to @PankajBhojwani for another signoff in the morning :smile:

@msftbot make sure @PankajBhojwani signs off on this


Authorize policy attribute not compatible with dynamic policy provider

I was attempting to implement something like the DynamicPolicyProvider in your unit tests where the idea is that you can specify the policy name at runtime and built it on the fly. While this passes your unit tests, if you try to actually declare something like [Authorize("MyPolicy")] on an action/controller you get an exception that states
InvalidOperationException: The AuthorizationPolicy named: 'MyPolicy' was not found.
Is this by design, and if so, what am I missing? I assume this happens because the policy isn't declared at runtime, but how is that compatible with the idea of building it at runtime?

@HaoK can you show how to wire this up?
Then give to @blowdart to document this more completely.

Issues is that we aren't using the IPolicyProvider in AuthorizePolicy.Combine, which MVC AuthorizeFilter is using to combine policies

Will this make it in RTM?

Yeah it should make it.

Awesome because this will make my life a lot easier


Fix hellow world sample component name

I think there was a bit of copy pasta

Thanks for catching that!


Validator Middleware is not compatible with Symfony 2.3

Turns out our advertised compatibility with Symfony 2.3 is not correct.
Symfony\Component\Validator\Validator\ValidatorInterface; used in the ValidatorMiddlware is actually not available until 2.5, it is formerly known as Symfony\Component\Validator\ValidatorInterface;
Thinking what approach is best here.

I'd vote to just upgrade the requirements to 2.8.

Given anything below 2.7 is now gone, and 2.8 is the new LTS, I agree.

#24 drops support for any symfony version older than 2.8

Indeed resolved by #24


I’m wondering if a virtual display can function as the only display without a physical monitor connected to the computer.

I’m wondering if a virtual display can function as the only display without a physical monitor connected to the computer. Specifically, if I initially set up the virtual display using Parsec-VDD with a physical monitor connected, then disconnect the monitor, will the virtual display still be operational?
Additionally, if I connect to the computer through a remote control app like Moonlight after disconnecting the monitor, would the virtual display continue to work in that scenario?
I’m currently using a laptop and can't test this myself, so I’m reaching out to see if anyone has tried this and can confirm whether it works.
Thanks in advance for your help!

Yesh, just enable the 'fallback display' option and allow the app to run on startup, your headless PC will be able to connect from any remote desktop app.

That's great! It has helped me a lot. I'd like to thank you for you development.


fixed few typos/formatting errors

variatiant -> variation;
decending -> descending;
removed – from titles.

Thanks for catching these typos, Aleks!


Altering protection to prevent changing data of other users

Hello Dear OrbitDB Team,
I want to create a system that has a register/log-in mechanism using IPFS and OrbitDB. The software I am going to build is intended to write username-password into OrbitDB database. Since OrbitDB is decentralized, the data is going to be accessible and changed by others. I know that there is an "Access Control" mechanism. However, everyone needs to have a write permission to register and for other database operations and this permission causes anyone to change other people's password.
I need anyone to be able to create a new data but everyone can alter only the data that was created by them. Is it possible to apply a permission for every user of the software? If not, what do you think about building a feature like this?
Thank you so much.

My suggestion would be to create a database for each user. This would ensure only the user can edit their own data.
However, if you have a use case which requires adding user details to a single database, you could develop a custom access controller which checks the user's public key against the entry being edited.
Please be aware that the data you are storing to OrbitDB is in clear text unless you are explicitly encrypting it yourself.

I think I will try custom access controller function. I didn't know this feature. Thank you so much.


feat: uses a different gha to generate preview release notes

Description
The semantic release GHA won't work in PR events. This does work as tested in some other repos.
Changes

feat: uses a different gha to generate preview release notes

🚀 PR created with fotingo

Agreed! I think push is a good compromise to get the full configuration working 👍


Introduce spotless maven plugin for linting Scala and Java code

Proposed changes
A code style linter is the key to making all the git history traceable and readable.

introduce the spotless maven plugin for linting Scala and Java code
spotless maven plugin 2.30.0 is the last available version for Java 8
use ScalaFmt for Sala code linting, the style config file .scalafmt.conf also help lint on local IDE
use Google Java Format for Java linting, 1.7  is the last available version for Java 8
auto checked in CI builds with maven, and easy to apply fixes by running mvn spotless:apply

The scala style rules are mainly referred to Apache Kyuubi's practice here (https://github.com/apache/kyuubi/blob/master/.scalafmt.conf).
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

cc @JNSimba @hf200012

Thank you for your contribution, but there are too many style-related changes, and it will be troublesome to pick pr later, so it is not recommended to change, thank you


Change name policy to trust

Fixes https://github.com/projectatomic/skopeo/issues/191

I am perfectly fine with renaming --policy to --trust, but we do need a $name for the file or the concept which works in situations like "the image is not signed and $name requires a signature” or “the signature for this image is not accepted per $name”.  “signature verification policy” is a mouthful but works; “trust”, to me, doesn’t.

I think we can drop this back and just leave policy in lots of places but begin talking about it as a "trust" policy.
@aweiteka WDYT?

The two really user-visible pieces to which we kind of commit to in the public release are the --policy option to skopeo, and the /etc/containers/policy.json path. (We could of course change the default names in future releases and read the older names for compatibility, but it would be a bit awkward.)
I am perfectly fine with changing these two to --trust and trust.json; it is less clear what names to use for the internal signature.Policy{,Context} API naming and the error messages, but we can change them ~anytime.

I simplified the original pull request to just change the way we refer to the file,  as a trust policy file.  Lets talk at the meeting.

Fixes https://github.com/projectatomic/skopeo/issues/191

@aweiteka and I have agreed on this.
@mtrmac @runcom PTAL and lets get this merged.

LGTM as is (@mtrmac?)

ACK. A similar patch to containers/image would touch 3 error messages.


TextField inputProps invalid font causes other TextField to lose it styling

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
<TextField                         label="label wrong fonts"                         inputProps={{                             sx: {                                 fontFamily: '"'                             }                         }}                     />                     <TextField                         label="Should be fine"                     />
Current behavior 😯
Something goes wrong with the styling, and it propagate to all TextFields in the same page

Expected behavior 🤔
Font in the first input should not change, or at least not cause the component to change its styling.

Context 🔦
A TextField for inputing text and previewing it inside the TextField in the font that was selected.
Your environment 🌎
OS: Linux 5.15 Linux Mint 21 (Vanessa)
Binaries:
Node: 16.18.1 - ~/.nvm/versions/node/v16.18.1/bin/node
Yarn: Not Found
npm: 8.19.2 - ~/.nvm/versions/node/v16.18.1/bin/npm
Browsers:
Chrome: Not Found
Firefox: 111.0
npmPackages:
@mui/base:  5.0.0-alpha.60
@mui/icons-material: 5.4.2 => 5.4.2
@mui/lab: 5.0.0-alpha.60 => 5.0.0-alpha.60
@mui/material: 5.4.3 => 5.4.3
@mui/private-theming:  5.11.2
@mui/styled-engine:  5.11.0
@mui/styled-engine-sc: 5.4.2 => 5.4.2
@mui/styles: 5.4.2 => 5.4.2
@mui/system: 5.4.3 => 5.4.3
@mui/types:  7.2.3
@mui/utils:  5.11.2
@mui/x-data-grid: 5.5.1 => 5.5.1
@mui/x-data-grid-pro: 5.5.1 => 5.5.1
@mui/x-license-pro:  5.5.1
@types/react:  17.0.52
react: 17.0.2 => 17.0.2
react-dom:  17.0.2
styled-components: ^5.3.5 => 5.3.6
typescript:  4.9.4

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://mui.com/r/issue-template), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

FYI @phoenixstudiodz I can't reproduce this, see this sandbox: https://codesandbox.io/s/https-github-com-mui-material-ui-issues-36612-fzd9i6


Issue: SharePoint SSO Component

Suggest improvements to the code sample and/or setup guide.
I did setup an SPFx extension with the same package.json packages and same components but I got error with running locally with gulp serve.
I tried uninstall the packages then install again but still the same issue. Didn't find something useful when search for the errors.
Error - [webpack] 'dist':
./node_modules/micromark-util-decode-numeric-character-reference/index.js 23:11
Module parse failed: Identifier directly after number (23:11)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|   code > 126 && code < 160 ||
|   // Lone high surrogates and low surrogates.
>   code > 55_295 && code < 57_344 ||
|   // Noncharacters.
|   code > 64_975 && code < 65_008 || /* eslint-disable no-bitwise */
 @ ./node_modules/mdast-util-from-markdown/lib/index.js 138:0-97 1061:14-45
 @ ./node_modules/mdast-util-from-markdown/index.js
 @ ./node_modules/botframework-webchat-component/lib/Attachment/Text/private/LinkDefinitions.js
 @ ./node_modules/botframework-webchat-component/lib/Attachment/Text/private/MarkdownTextContent.js
 @ ./node_modules/botframework-webchat-component/lib/Attachment/Text/TextContent.js
 @ ./node_modules/botframework-webchat-component/lib/index.js
 @ ./node_modules/botframework-webchat/lib/addVersion.js
 @ ./node_modules/botframework-webchat/lib/index.js
 @ ./lib/extensions/copilotComponent/components/ChatBot.js
 @ ./lib/extensions/copilotComponent/CopilotComponentApplicationCustomizer.js
./node_modules/micromark-util-sanitize-uri/index.js 86:22
Module parse failed: Identifier directly after number (86:22)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|     }
|     // Astral.
>     else if (code > 55_295 && code < 57_344) {
|       const next = value.charCodeAt(index + 1)
|
 @ ./node_modules/micromark/lib/compile.js 44:0-55 770:10-21 781:10-21 1089:8-19 1105:22-33
 @ ./node_modules/micromark/index.js
 @ ./node_modules/mdast-util-from-markdown/lib/index.js
 @ ./node_modules/mdast-util-from-markdown/index.js
 @ ./node_modules/botframework-webchat-component/lib/Attachment/Text/private/LinkDefinitions.js
 @ ./node_modules/botframework-webchat-component/lib/Attachment/Text/private/MarkdownTextContent.js
 @ ./node_modules/botframework-webchat-component/lib/Attachment/Text/TextContent.js
 @ ./node_modules/botframework-webchat-component/lib/index.js
 @ ./node_modules/botframework-webchat/lib/addVersion.js
 @ ./node_modules/botframework-webchat/lib/index.js
 @ ./lib/extensions/copilotComponent/components/ChatBot.js
 @ ./lib/extensions/copilotComponent/CopilotComponentApplicationCustomizer.js
./node_modules/htmlparser2/lib/esm/index.js 48:9
Module parse failed: Unexpected token (48:9)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|  * They should probably be removed eventually.
|  */
> export * as ElementType from "domelementtype";
| import { getFeed } from "domutils";
| export { getFeed } from "domutils";
 @ ./node_modules/sanitize-html/index.js 1:19-41
 @ ./node_modules/botframework-webchat/lib/markdown/renderMarkdown.js
 @ ./node_modules/botframework-webchat/lib/index.js
 @ ./lib/extensions/copilotComponent/components/ChatBot.js
 @ ./lib/extensions/copilotComponent/CopilotComponentApplicationCustomizer.js

What does your serve.json look like?

The default:
'use strict';

const build = require('@microsoft/sp-build-web');

build.addSuppression(`Warning - [sass] The local CSS class 'ms-Grid' is not camelCase and will not be type-safe.`);

var getTasks = build.rig.getTasks;
build.rig.getTasks = function () {
  var result = getTasks.call(build.rig);

  result.set('serve', result.get('serve-deprecated'));

  return result;
};

build.initialize(require('gulp'));

That is not serve.json
https://github.com/microsoft/CopilotStudioSamples/blob/master/SharePointSSOComponent/config/serve.json

Sorry, accidently past the gulp.js
serve.json file same as the code sample.

Same as the sample, i.e. untouched?

Same as the sample and populate with my own values. I think I figured out where is the issue.
The issue has been resolved locally by adding the following function in the gulp.js file but it hasn't been resolved when deploying the extension using Azure DevOps pipelines.
build.configureWebpack.mergeConfig({
  additionalConfiguration: (generatedConfiguration) => {
    generatedConfiguration.module.rules.push(
      {
        test: /\.js$/,
        exclude: /node_modules\/(?!htmlparser2)/,
        use: {
          loader: 'babel-loader',
          options: {
            presets: ['@babel/preset-env']
          }
        }
      }
    );
    return generatedConfiguration;
  }
});

Same as the sample and populate with my own values. I think I figured out where is the issue.
The issue has been resolved locally by adding the following function in the gulp.js file but it hasn't been resolved when deploying the extension using Azure DevOps pipelines.
So, gulp bundle in DevOps pipeline is giving the previous error however it works locally fine. gulp bundle --ship
build.configureWebpack.mergeConfig({
  additionalConfiguration: (generatedConfiguration) => {
    generatedConfiguration.module.rules.push(
      {
        test: /\.js$/,
        exclude: /node_modules\/(?!htmlparser2)/,
        use: {
          loader: 'babel-loader',
          options: {
            presets: ['@babel/preset-env']
          }
        }
      }
    );
    return generatedConfiguration;
  }
});

Can you open a new issue so I could investigate Azure DevOps support (can't commit to a timeline though)


[BEAM-6865] DefaultJobBundleFactory: create using default environmentFactoryProvi…

…derMap
I'm using the exact same environmentFactoryProviderMap in the Spark runner, so it makes sense for this code to be moved.
R: @robertwb, @angoenka
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

LGTM
Can you also add the jira in the commit message.

Can you also add the jira in the commit message.

Done


(NCO) [Bug] Blindness still causes flicker occasionally

Thanks for fixing the issue! Unfortunately, it seems that at 0:01, blindness still flickers occasionally (every 5-10 seconds or so, a single flicker happens). :C
During that split second flicker, the sky is gray (not white, because the timer hasn't gotten close enough to 0:00)- you can see everything for a split second and then the blindness comes back. I think it may be because the number dips somewhere below 0:01, and since blindness does not have a fade-in effect (i.e. the world just suddenly pops into view), it looks like a flicker. Maybe increasing to 0:02 might fix this issue.
I tested running around with a 20 second blindness effect and the flicker does not happen at all, so I think I've ruled it out being some sort of lag/graphics issue.

Hmm, must have not tested for long enough - perhaps this build does the trick?

I ran around for a good minute without flickering. I think that fixes it! Thanks. ^_^


Upgrade prettytable-rs to 0.8

I wasn't getting any color with TERM=tmux-256color, neither automatic nor forced always, so I think it didn't understand that terminal. I do get automatic color with prettytable-rs 0.8.

@BurntSushi ping?

@cuviper Thanks. There appear to be a whole bunch of unrelated changes to the Cargo.lock though. Did you just run cargo update -p prettytable-rs or cargo update? I'd prefer the former.

Hmm, I don't remember why I wanted the broad cargo update, though at least I did that in a separate initial commit. I think that may have been to deal with indirect updates breaking 1.28, but I'll try to redo this with the minimal change.

Updating the Rust version is okay. Latest stable is fine with me for a tool like this.

Basically, my main thing here is that I like to do dependency updates myself so I can see what's going on. Maybe that would be quicker. Would you be okay with me doing that and closing this PR?

I don't mind if you want to handle this yourself. I just wanted to fix my colors. :slightly_smiling_face:

Ug. I was afraid of this happening. But it looks like this causes a update to the term dependency of prettytable-rs, and that in turn brings in a host of new dependencies. And in particular, it adds a dependency on the dirs crate, which I'm not comfortable putting in any of my dependency trees (for non-technical reasons).
I'm not sure what to do about this. Seems like the only option is to abandon prettytable-rs. I'm surprised people are still using term.

I pushed an update to as many dependencies as I could to master. But I'd guess that won't fix your problem.

Yeah, the updated master is still uncolored for me.
I also tried prettytable-rs 0.7, which does fix my colors, but that's pretty much the same dependency story. I guess it's the update from term 0.4 to 0.5 that started recognizing my terminal.

And in particular, it adds a dependency on the dirs crate, which I'm not comfortable putting in any of my dependency trees (for non-technical reasons).

It looks like that was added in term 0.5.2, whereas 0.5.1 only depended on byteorder and winapi.

It seems that was only added to avoid the deprecated std::env::home_dir -- https://github.com/Stebalien/term/commit/84cfdb51775b327fedf21784749d862fdffa10b4, so maybe there's another dependency that could fill this need. (But I don't know your reasons for avoiding dirs.)

Nice, works for me. Should be fixed in cargo-benchcmp 0.4.3 on crates.io.
And yes, I would have just kept to std::env::home_dir and stuck an allow(deprecated) on it.

Thanks!


Version 1.0

Description

Symfony 6 only
PHP 8 only
Phpstan level 9
Fixes deprecation with SF 6.3 about missing return types

Besides the obvious BC breaks due to added return types, there is one BC break that needs attention.
The UserInterface whose method changed from getIdentifier() to getUserIdentifier().
As latter method is part of the Symfony User Interface, this shouldn't be an issue, as we target SF 6 only.
Types of changes

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

Checklist

[x] I updated the documentation (see here)
[x] I agree that this code will be published under the MIT license

This is great!   Can you also add Symfony 7, which will be out in 2 months?
My branch of the demo works with Symfony 6.4 and I've been removing deprecation warnings.

What about those issues @kevinpapst?

My changes are BC breaks, that's why we need to bump to 1.0.
Also I am not a big fan of the "I am waiting for years until 1.0" release tactic.
It is time to signal that the bundle is stable.
And there is always a time for new major release if we do BC breaks in the future (e.g. remove |trans filter).
Lets move the tickets to a new milestone and close the linked one.
SF 6.3 is out and there are deprecations that need to be fixed now.
SF7 is coming soon, we should be prepared.
I agree, we can always release a 1.1 to support SF7 in two month.
Regarding your code comments: these files were NEVER changed since I committed them 2 years ago.
I am working mostly by myself since years, so you have better insights in merge problems. Larger teams on active codebases obviously have different needs than projects like this one 😄
But I do not see a big risk here, due to the rare commits in the last months 🤷 which is actually a good sign of stability IMO.


Making enum exclusive constraint trait

Does it make sense to allow enum trait to coexist with length or pattern? I don't think so. In any case, the values of the enum are not constrained by the other two traits; this model builds fine:
$version: "1.0"

namespace com.amazonaws.simple

use aws.protocols#restJson1

@restJson1
@title("SimpleService")
service SimpleService {
    operations: [
        AnOperation,
    ],
}

@http(uri: "/operation", method: "GET")
operation AnOperation {
    input: AnOperationInput,
    output: AnOperationOutput,
}

structure AnOperationInput {
    enumMember: MyEnum
}

structure AnOperationOutput {
}

@enum([
    {
        value: "t2.nano",
        name: "T2_NANO",
    },
    {
        value: "t2.micro",
        name: "T2_MICRO",
    },
    {
        value: "m256.mega",
        name: "M256_MEGA",
    }
])
@length(min: 1, max: 3)
@pattern("\\d+")
string MyEnum

FWIW we're making enums shapes in v2, you could very easily add :not(enum) to their selectors

While I agree with the spirit of this issue, I don't think like it's worth the risk of breaking existing models (particularly the tens of thousands inside of AWS). It's a stretch, but a use case for mixing enum and pattern/length might be that a team wants to restrict how future enum values are modeled by the service team -- they can use length and pattern to constrain it in the future even if it doesn't make sense to enforce server-side.
Given enums are going to be promoted to shapes in IDL v2, I think this change has more risk than reward.

The EBS model has this:
https://github.com/awslabs/smithy-rs/blob/e1e9a29d5db1330eb51c1e84e5c6c3acb9b4f65e/aws/sdk/aws-models/ebs.json#L1195-L1217

they can use length and pattern to constrain it in the future even if it doesn't make sense to enforce server-side.

Then Smithy CLI should enforce this, not each code generator.


Source link is sometimes treated as an internal route rather than an external link in image detail views

Image detail view source links sometimes route users to a non-existent page within CC Search.
Sometimes, links in our database include the protocol (e.g. https://), and other times it does not. When the protocol is included, the user will be correctly routed, and when it is missing, they will be routed to the non-existent internal page.
This can be fixed by either:

Adding some logic in the frontend to account for inconsistent inclusion of protocols (stick 'https://' in front of the URL when required after fetching it from the API)
OR
Amending the database so that all source links include the 'https://' protocol prefix.

Example where the source link works correctly:
https://ccsearch.creativecommons.org/photos/92df8057-86e3-4d33-93fe-a51cb315d8ba
'Source' routes to https://www.flickr.com/photos/mikefats/387594564
Example where it's broken:
https://ccsearch.creativecommons.org/photos/ab41edc5-6075-4450-88a3-87bb99f629c0
'Source' routes to https://ccsearch.creativecommons.org/photos/www.flickr.com/photos/92082510@N04/15161614328

I discussed this with @sclachar and we have decided that the best way forward is to handle it outside of the database layer. The incomplete URLs are from upstream content partners.
I'm going to patch the API so it always returns results with a protocol in the 'source' link.


Adding instructions on compiling Substrate on Mac M1 chip

A few comments has been seen in our Element technical channel on how to run Substrate on Mac M1 chip. Let's add a section here (will be shown here finally) on how to make Substrate run on Mac M1 chip.
@hammewang Could you help? Thanks.

@jimmychu0807 Of course. Since lots of changes in the latest substrate, it will take me some time.

This blog seems to outline a solution that works for most people. Although I have been telling people that really it's a pain and if possible, do not use M1 ARM for builds/running substrate until upstream rust deps get stable for use on them.
https://vikiival.medium.com/run-substrate-on-apple-m1-a2699743fae8

updated instructions here: https://github.com/substrate-developer-hub/substrate-node-template/issues/179#issuecomment-843522331


Documentation on how to format columns

The documentation doesn't show how to edit the column data (i.e. alignment etc.). The core revogrid documentation shows the angular/react etc. way to do it -- is there any help available on how to do so in the "svelte" way?
Also - might I suggest setting up a discord channel for the library for the community?

https://rv-grid.com/guide/column/#Column-Formats


{Request} search settings XD

No content

Issue created by BlueBubbles Bot in the BlueBubbles commnity on behalf of @monicow in #requests.

Prob create a map of widgets with the key as a list containing 2 things:

The title of the setting item
The description of the setting item if applicable

Then when the user searches we grab those two items individually and use simple String.contains() to check if the search matches either the title or description in some way, and load all the widgets with matches. Don't load SettingsTiles though maybe? Idk for sure right now.

Prob create a map of widgets with the key as a list containing 2 things:

The title of the setting item
The description of the setting item if applicable

Then when the user searches we grab those two items individually and use simple String.contains() to check if the search matches either the title or description in some way, and load all the widgets with matches. Don't load SettingsTiles though maybe? Idk for sure right now.

What if we did it like this:

Create the map for each option. The map should contain the name and description. But also the navigation page it's on (The GetX location string), as well as the global key so we can "scroll to" it.
Search will show the results and tapping will bring you to the correct settings page


Blockquote not correctly closed

Hi I have the following HTML:
<blockquote>  This is a quote </blockquote>

<img alt="alt" src="https://path/to/file.jpg" />

The generated markdown for this is
> This is a quote ![alt](https://path/to/file.jpg)

Somehow the newline before the image is missing after a blockquote. It happens for all blockquote elements I tested that where followed by an image. If there is text after the quote everything works fine.
I'm using reverse_markdown version 1.0.3 and ruby 2.3.1p112

@kanedo Thanks, I've just release 1.0.4 to include the change. Sorry for the long wait, I've been a bit busy 😄


[Bug]: Account not opening after clicking on Windows message for received transaction

Issue description
Hey guys, I sent a transaction to my wallet and got some notification from Windows/ Firefly. After I clicked the message, my Firefly went in "I'm not able to click anything in the app" mode. I restarted it, and now I can't get over the loading Firefly symbol after entering my PIN. Its loading forever.
Expected behaviour
Logging in to my wallet.
Actual behaviour
Infinite loading screen without anything happening.
Can the issue reliably be reproduced?
Yes
Steps to reproduce the issue
Starting the firefly application.
Entering the PIN code for my profile.
Nothing happens except the Firefly loading symbol.
Diagnostics
Firefly v1.3.3
Plattform: win32
Plattformversion: 10.0.19043
Plattform-Architektur: x64
CPU-Anzahl: 8
Gesamtspeicher: 8026.2 MB
Freier Speicher: 2749.9 MB

Errors
Mon, 14 Feb 2022 13:26:40 GMT
ReqwestError: error sending request for url (https://mainnet-node.tanglebay.com/api/plugins/participation/addresses/iota1qpegl8wh2m52g3wtsdef589qvvsjae4866kyfhu5yyc5zqtktq5kg5hrmzn): error trying to connect: dns error: Der angegebene Host ist unbekannt. (os error 11001)

Mon, 14 Feb 2022 13:26:40 GMT
ClientError: `error sending request for url (https://mainnet-node.tanglebay.com/api/v1/addresses/iota1qrjtfxnd6qlcyecpuwmvd4s39jcdrvzvs4ydzdc03zggwh5cd3upwvuk4hf/outputs?include-spent=true): error trying to connect: dns error: Der angegebene Host ist unbekannt. (os error 11001)`

Mon, 14 Feb 2022 13:26:40 GMT
ClientError: `error sending request for url (https://mainnet-node.tanglebay.com/api/v1/info): error trying to connect: dns error: Der angegebene Host ist unbekannt. (os error 11001)`

Mon, 14 Feb 2022 13:26:40 GMT
ClientError: `error sending request for url (https://mainnet-node.tanglebay.com/api/v1/info): error trying to connect: dns error: Der angegebene Host ist unbekannt. (os error 11001)`

Mon, 14 Feb 2022 13:26:40 GMT
ReqwestError: error sending request for url (https://mainnet-node.tanglebay.com/api/plugins/participation/events): error trying to connect: dns error: Der angegebene Host ist unbekannt. (os error 11001)

Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

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

Hi I had the same problem.
I was able to fix it.
1. uninstall the Firefly app.
2. go to the following folder: %appdata% > Roaming > Firefly
3. delete the Firefly folder
4. install Firefly again
5. log in with your account
Voilà !

Hi I had the same problem.
I was able to fix it.
1. uninstall the Firefly app. 2. go to the following folder: %appdata% > Roaming > Firefly 3. delete the Firefly folder 4. install Firefly again 5. log in with your account
Voilà !

Thanks man, you saved my day! That perfectly helped :)


Does pugixml support on Linux Environment?

Hello Team,
Sorry for asking this in as an issue first.But no other way out to know this.
I am working on a Swift Project and need to use XML Parser that has xPath support too.Will pugixml work on Linux environment?
Thanks.

I've used it on my Linux PC before, so that should be a "yes".

Yes, pugixml fully supports Linux (as well as other major operating systems). This is mentioned in the manual (http://pugixml.org/docs/manual.html#install.portability) and sort of mentioned on pugixml.org.


Sheet component is not working inside Dropdown Menu.

Description:
When attempting to select an item from the dropdown menu, the associated sheet fails to activate as expected.
Expected Behavior:
The selected item from the dropdown menu should activate and display the associated sheet.
Actual Behavior:
Upon selecting an item from the dropdown menu, the associated sheet does not activate, and the expected content is not displayed.
Code
 <DropdownMenu>
          <DropdownMenuTrigger asChild>
            <Button>
              <PlusIcon className="-ml-0.5 mr-1.5 h-5 w-5" aria-hidden="true" />
              Add contacts
            </Button>
          </DropdownMenuTrigger>
          <DropdownMenuContent align="end">
            <DropdownMenuGroup>
              <DropdownMenuItem className="gap-2 px-4 py-2.5">
                <button className="flex items-center justify-start gap-2">
                  <CloudIcon />

                    Import contacts

                </button>
              </DropdownMenuItem>
              <DropdownMenuItem className="gap-2 px-4 py-2.5">
                <Sheet>
                  <SheetTrigger asChild>
                    <button>
                      <PlusIcon className="h-5 w-5" aria-hidden="true" />

                        Create a contacts

                    </button>
                  </SheetTrigger>
                  <SheetContent>
                    <SheetHeader>
                      <SheetTitle>Create contacts</SheetTitle>
                    </SheetHeader>
                  </SheetContent>
                </Sheet>
              </DropdownMenuItem>
            </DropdownMenuGroup>
          </DropdownMenuContent>
        </DropdownMenu>

Screenshots:

Thank you for your attention to this matter! Let me know if any additional information is needed.

What was the fix to this? I am also experiencing a similar issue.

I found a fix, don't use SheetTrigger, instead use isOpen prop to toggle the sheet. And separate the sheet component from the dropdown component

@obah can you please send us an example of fix ?
That would be awesome, thank you :)

const [isOpen, setIsOpen] = useState(false);
const toggleSheet = () => setIsOpen(!isOpen);
return (

<Button size={"full"} onClick={toggleSheet}>
Open Sheet

    <SheetContent>
        ...
    </SheetContent>
</Sheet>

)
Basically something like this. You will pass the states and function to the component where the sheet will be triggered from


asynchronous load fails , getting error code : 7002

Here is the code i have used for angular project:
from ts :
import { Component } from '@angular/core';
const shaka = require("node_modules/shaka-player/dist/shaka-player.compiled.js") // ./dist/shaka-player.compiled.js');
//declare var shaka: any;
@Component({
  selector: 'app-app-shaka-player',
  templateUrl: './app-shaka-player.component.html',
  styleUrls: ['./app-shaka-player.component.css']
})
export class AppShakaPlayerComponent {

  manifestUri = '//storage.googleapis.com/shaka-demo-assets/angel-one/dash.mpd';
  constructor() { }

  ngOnInit() {

    this.initApp();

  }

  //document.addEventListener('DOMContentLoaded', this.initApp());

  initApp() {
    // Install built-in polyfills to patch browser incompatibilities.
    shaka.polyfill.installAll();

    // Check to see if the browser supports the basic APIs Shaka needs.
    if (shaka.Player.isBrowserSupported()) {
      // Everything looks good!
      this.initPlayer();
    } else {
      // This browser does not have the minimum set of APIs we need.
      console.error('Browser not supported!');
    }
  }

  initPlayer() {
  // Create a Player instance.
  const video = document.getElementById('video');
  const player = new shaka.Player(video);

    // Attach player to the window to make it easy to access in the JS console.
//window.player = player;

// Listen for error events.
player.addEventListener('error', this.onErrorEvent);
player.setTextTrackVisibility(true);
// Try to load a manifest.
// This is an asynchronous process.
player.load(this.manifestUri).then(function () {
  // This runs if the asynchronous load is successful.
  console.log('The video has now been loaded!');
}) //.catch(error => {this.onError(error)});  // onError is executed if the asynchronous load fails.
}

onErrorEvent(event:any) {
  // Extract the shaka.util.Error object from the event.
  this.onError(event.detail);
}

onError(error: any) {
  // Log the error.
  console.error('Error code', error.code, 'object', error);
}

}

Here is the html part:
<div class="center">
    <video id="video"
           width="640"
           poster="//shaka-player-demo.appspot.com/assets/poster.jpg"
           controls autoplay></video>
</div>

Im getting the error code : 7002 at here when trying to load manifest url :
player.load(this.manifestUri).then(function () {
  // This runs if the asynchronous load is successful.
  console.log('The video has now been loaded!');
}) //.catch(error => {this.onError(error)});  // onError is executed if the asynchronous load fails.

In my personal machine this works fine , but in my VM from Azure  facing this error. Immediate solution for this issue would be greatly appreciated :)
Thank you !

Take a look at the docs for error 7002: https://shaka-player-demo.appspot.com/docs/api/shaka.util.Error.html#value:7002

Name
Value
Type
Description

NO_VIDEO_ELEMENT
7002
number
The call to Player.load() failed because the Player does not have a video element. The video element must either be provided to the constructor or to Player.attach() before Player.load() is called.

At this point in your code:
  const video = document.getElementById('video');
  const player = new shaka.Player(video);

Is video null?  If video is part of the component's HTML, it may not exist during ngOnInit.  You have to wait until ngAfterViewInit to access it.
Does this help?


Fixed-size integers larger than Int64

Provide a family of [U]Int128, 256, ... up to a reasonable threshold where flexibly-sized bignum becomes competitive (most likely 1024 bits). This should be done as generically as possible so that support for additional sizes can be added as needed with minimal complexity.
These types should conform to FixedWidthInteger.

I cannot emphasize enough how great it would be with Int256, UInt256, Int512, UInt512 etc! But why not also a BinaryInteger of non fixed width. There is already a prototype by Apple here. So far I’ve been relying on Great attaswift/BigInt by @lorentey , would be nice with BigUInt as well!

But why not also a BinaryInteger of non fixed width

It's just a separate issue.

What are the odds that a generically written UInt128 will be optimised into use of the specialised instructions available on some supported platforms?

What are the odds that a generically written UInt128 will be optimised into use of the specialised instructions available on some supported platforms?

Do you have an example? I'm not aware of any processor with hardware support for 128-bit integers (that being said, newer PPC processors have 128-bit floating point). Or do you merely mean "use the carry/borrow flag/bit" available in most processors?

Or do you merely mean "use the carry/borrow flag/bit" available in most processors?

Sorry, I should have been clearer: yes, I mostly meant that, though in the event some weirder platforms have actually got 128-bit integer operations that would be nice too. As a concrete example, here is a naive-but-reasonable implementation of 128-bit unsigned integer checked addition (that might be used when writing this generically across FixedWidthInteger), and the associated x86_64 assembly is:
output.UInt128.add(output.UInt128) -> ():
        push    rbp
        mov     rbp, rsp
        add     qword ptr [r13], rdi
        mov     rax, qword ptr [r13 + 8]
        jae     .LBB1_2
        add     rax, 1
        mov     qword ptr [r13 + 8], rax
        jb      .LBB1_4
.LBB1_2:
        add     rax, rsi
        mov     qword ptr [r13 + 8], rax
        jb      .LBB1_3
        pop     rbp
        ret
.LBB1_3:
        ud2
.LBB1_4:
        ud2

Whereas a Rust version gives us:
example::add:
        push    rax
        mov     r8, rdx
        mov     rdx, rsi
        mov     rax, rdi
        add     rax, r8
        adc     rdx, rcx
        jb      .LBB0_1
        pop     rcx
        ret
.LBB0_1:
        lea     rdi, [rip + .L__unnamed_1]
        call    qword ptr [rip + core::panicking::panic@GOTPCREL]
        ud2

I'd like us to have the Rust version rather than the naive Swift version if we could.

If your example is simplified, then the code gen is optimal:
https://swift.godbolt.org/z/ES6j7q

Sadly your simplification makes the program incorrect. Specifically, you cannot safely perform an unchecked addition of the carry bit in case that addition itself overflows. You can see this by running your code with these extensions:
var s1 = UInt128(high: 0, low: 1)
let s2 = UInt128(high: .max, low: .max)
s1.add(s2)
print("high: \(s1.high), low: \(s1.low)")

This will print `high: 0, low: 0", which indicates that we have overflowed, but the program did not crash, though it should have. The Rust program correctly crashes in this case.

What are the odds that a generically written UInt128 will be optimised into use of the specialised instructions available on some supported platforms?

If for some reason this isn’t possible, we simply wouldn’t use a generic implementation. Part of the API contract for such a type should be that it is essentially optimal.

Thanks for clarifying @stephentyrone, that's good to know, especially as this was a limitation with DoubleWidth back when that was floating around.

@Lukasa – Whoops. You're right. I was too focused on getting the right ASM out. Nevertheless, both Rust and Swift use LLVM, and the standard library integers are "just" a wrapper around LLVM intrinsics. We just need an intrinsic or pattern of intrinsics that can reliably lower to an "ADC" on x86 (or similar instructions on other ISAs). Rust is either using such an intrinsic, or their just using LLVM's i128 type. Swift can do the same. This isn't hard.

Swift can do the same. This isn't hard.

I agree, unless the requirement is actually to use an intrinsic. If it is, writing the code generically gets pretty gnarly as you require compile time type checking to replace the generic slow path with the intrinisic fast path.
I also do not know how to use LLVM intrinsics safely from within a Swift package. If that’s straightforward then all is well, but if it isn’t then that raises further questions.
Hence the question: do we believe that LLVM can be convinced, either by the generic code or by means of LLVM intrinsic calls from a swift package? So far both you and @stephentyrone have said “yes”. You two are both more expert than I am, and so I believe you both, but I do think it is telling that so far no-one has actually convinced the Swift compiler to emit the pattern we want.

I want to stress here: I am satisfied that my desired outcome is possible. I really mean it when I say I believe you both! What I am trying to say here is that I do not see how to do it (a personal limitation, not something that limits either of you), and I haven’t seen it done yet, so I am currently operating entirely on faith, not evidence. 👍

Okay, more old-school, but it works this time. The compiler fails to elide a CMP instruction and a MOV instruction, but otherwise it's "perfect":
https://swift.godbolt.org/z/JHDVUq

Yup, that looks a lot better. It’s a shame about the cmp but it seems like something that could be optimised away at least in principle.

The compiler fails to elide a CMP instruction and a MOV instruction

This would be a good bug to file if you can spare a minute to write it up (if not, let me know and I'll do it). I've probably already reported an equivalent issue, but I'm not finding it right now.

Looks like various carry chain related bugs exist:
https://bugs.llvm.org/buglist.cgi?quicksearch=addcarry

A quick reading suggests that none of those is a great fit for this particular issue, which looks (to me) considerably easier to resolve.

We're drifting way, way off topic. I looked into this. I think LLVM bug 36243 and 39464 are the same bug, and the same bug as this bug. In short, LLVM doesn't model status flags during generic (not target specific) abstract optimization passes. This design tradeoff has pros and cons. The "pro" is many programming languages (and some processors) don't expose status flags, so the compiler has to recognize idioms that are effectively checks of status flags. But the con is that LLVM easily "forgets" that a status flag might hold the answer it wants and then creating redundant "TEST" or "CMP' instructions.

(Hopefully not going too much further into the weeds) I think they're broadly the same class of bug, I just think that this particular case may be easier to address without completely solving the general problem. I'll poke around at some nebulous future point and see if that actually turns out to be true.

https://reviews.llvm.org/D70079

I don’t know if the code will be useful for this PR, but I tried to implement Int128/UInt128, but I never had the time to put it out there. https://github.com/drodriguez/swift/tree/integers-128 has the code I had around 3 months ago rebased onto a current apple/swift master. I haven't checked that it compiles or pass the tests, but it used to.
My interest in this was to implement Float128, which is needed for interacting with long double in some platforms (like Linux AArch64). The code for https://github.com/drodriguez/swift/tree/float-128 is even older (around 1 year?). I think I never got it to work. I don’t think I have seen any PR with interest for f128, but if anyone needs inspiration…
I don’t know if I would be able to work a lot on those, so if anyone wants to take them as a starting point, be my guest (if there's a lot of interest, I will try to find some time myself, but I cannot promise anything).

@Lukasa DoubleWidth is still floating around, and I would still like it form a basis for an (at least initial) implementation of this issue.  @stephentyrone any reason why not?

I think @davezarzycki convinced me it’s a decent basis for an initial implementation. I can always moan about it if I’m not happy with the codegen. :wink:

Also it’s hard to be too picky about work that others volunteer to do when you yourself are not volunteering to do it.

@dabrahams There are a bunch of reasons not to use the DoubleWidth pattern. Most are minor, but two of them are significant:

Sizes that are not powers of two start to become interesting as you go past 128b. There are a bunch of real applications for e.g. 192 and 384b operations--rounding these up to 256 and 512b makes multiplication (the dominant operation in most bignum arithmetic usage) half as efficient as it could be.
For sizes past 256b, using the DoubleWidth pattern starts to put a lot more strain on the optimizer to re-organize code in multiplication to pull out optimal sequences. What you generally want to end up with is a loop over e.g. 1x8 or 2x8 tiles of partial products, doing mul-add or mul-mul-add-add chains with carry. Those natural tiles are spread across multiple leaves of the number representation in the DoubleWidth formulation--you can work around this by simply taking a pointer to self and reinterpreting as a buffer of words or similar, but then you're effectively not using the DoubleWidth structure.

FYI I've written a package called swift-int2x a couple of years ago.
https://github.com/dankogai/swift-int2x

Thanks to SE-0104, making your own integer types is easier than ever.  This module makes use of it -- creating double-width integer from any given FixedWidthInteger.

import Int2X

typealias U128 = UInt2X<UInt64> // Yes.  That's it!
typealias I128 = Int2X<UInt64>  // ditto for signed integers

@dankogai That's the DoubleWidth approach that was mentioned earlier.  @stephentyrone it seems to me that this pattern in still useful in that it offers a path to easily implemented and tested functionality that may be needed in contexts where the width of inputs is generic and it's important not to overflow.  Even when the binary tree algorithms become inefficient they can at least be used to cross-check other implementations in testing.  Linearizing the algorithms can always be done as an optimization step, and if you're writing generic algorithms over arbitrary-width integers you'll still need something like DoubleWidth to handle the results of multiplications.

For an arbitrary-width integer type,DoubleWidth would be Self. But sure, I'm not saying binary-tree based approaches are useless, rather that they're generally suboptimal.

@stephentyrone if you want to go all technical precision on me… when you hear me say “generic algorithms over arbitrary-width integers” please hear that I mean generic algorithms over integers that are arbitrarily wide, not algorithms (why would they need to be generic?) over integers that have dynamic width.
To be 100% clear, I'm saying that something like DoubleWidth fills a niche that is covered neither by integers of dynamic width nor having an expanded library of fixed integer sizes.

@stephentyrone if you want to go all technical precision on me… when you hear me say “generic algorithms over arbitrary-width integers” please hear that I mean generic algorithms over integers that are arbitrarily wide, not algorithms (why would they need to be generic?) over integers that have dynamic width.

Apologies, I misinterpreted you; "arbitrary-width" is something of a term of art here.

Not a problem, Steve.  I'm wondering, though, whether you still think DoubleWidth is a suboptimal avenue for expanding the integer widths available in the Swift standard library, and if so, why?  As I've said, it seems like a useful and easily-implementable way to start that can always be optimized later and could eventually share a linearized implementation with a FixedSizeArray<Word>-based design that supports non-power-of-2 widths.

Hi @dabrahams – I don't think there is anything wrong with DoubleWidth as a generic type. But generally speaking, I think this is one of those cases where it's really easy for the users of generics to race far ahead of what the back-end optimizations are able to pattern match. Said differently, you'll know the back-end is ready when we can replace UInt64 with DoubleWidth<DoubleWidth<DoubleWidth<UInt8>>>

I have no illusions about the back-end's ability to optimize (it still leaves a lot to be desired), and in the case of DoubleWidth<DoubleWidth<DoubleWidth<UInt8>>>, either:

The implementation of operations is written in terms of the tree representation, and it could never hope to optimize perfectly because that's not the best algorithm for 8x width operations and I don't expect the optimizer to ever reconstruct a UInt64 division from the algorithm for division expressed in terms of that tree, or
The implementation of operations is linearized over Words, in which case there's every chance it might optimize perfectly today.

But whether the backend is ready for these types is totally beside the point.  There are plenty of applications that just need the functionality of Int128 or Int512, no matter how poorly they optimize.  And there are others that would benefit greatly in their development from being able to use a working DoubleWidth, and then dropping down to lower-level operations when they discover the need for better performance.  Swift in general suffers from a lack of useful prepackaged abstractions, and withholding them from the standard library until they can be implemented optimally is counterproductive.
Imagine if we had done that with Codable!  The results shown in this pitch show that performance could be far better than it is for many applications.  Yet, people use Codable every day to get important work done.

What is _TestSupport and how would I use DoubleWidth today?

Not sure if I should start a new thread but I have created a proposed UInt128 module.
Where should I upload the proposal? @stephentyrone
Here's the basic interface for discussion:
UInt128.doccarchive.zip

[U]Int128 probably fits better in the standard library itself (there's already a partial implementation there to support Duration.


Prepare v1.0.0 release

@MarioRodriguezS please use v1.0 as the head branch since we are releasing the first stable version


Fix the location of CS header files using relative paths

Determines relative path of _static to find the chemiscope headers

Note to self: if @Luthaf tells you to fix the paths , fix the paths.


Update file-uploads.md

add additional info, used to hint users in html form uploads.

Hello @AliBaghernejad ... We have a large number of updates going in on https://github.com/aspnet/AspNetCore.Docs/pull/12344. Check back after the PR is merged within a few days or weeks to let us know if this point isn't covered. I think it is in the updates.


feat: added home page components

Pull Request
Scope
Create a Homepage
Work Done
Added two components Home.vue and SearchBar.vue (this can be reused for navbar in other pages) and accompanying tests. Added style sheet and updated App.vue to show placeholders for navbar and footer.
Limitations: Router link for logo is disabled, search button is not connected.
Steps to Test
npm run start
Screenshot

It seems I used the wrong git commit message type. It should be test instead of chore. Anyway to fix this?


Fix typos in login page and docs for search

Just fixes up a few small typos I picked up in PR review for James :)

Thanks for the heads up! I've changed this quite a bit locally, so I've incorporated your fixes.


author name checking is too weak

@rultor release, tag is 0.0.15

@rultor release, tag is 0.0.15

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.15

@yegor256 Done! FYI, the full log is here (took me 3min)


bug: correctly set compiled_class_hash in DeclareTransaction

#744 has a bug in transaction/mod.rs where inside impl TryInto<DeclareTransaction> for &Transaction we set the compiled_class_hash equal to class_hash. This is because compiled_class_hash is not yet implemented in CallEntryPoint in blockifier @LucasLvy. Once this is done, we can fix this issue.

Hey, is the the CallEntryPoint in blockifier implemented? I can't find it but neither can I find an issue related to its implementation

Are we able to fix it now ?

I think yes, so may I take this one ?

Sorry for the late reply guys. This hasn't been implemented it in the blockifier yet. @LucasLvy, can someone do that as a part of this issue itself?

There will be big change in the implem of this soon. I'm working on it. It should most likely solve the issue

Hey,
Is this still open and if so can i take it? This is my first contribution so might need some help.

Hey @mbtariq82, thanks for taking the initiative. Unfortunately, this issue is closed now, we should have closed it earlier. However, there are a lot of other issues you can pick up! Do let me know if you need help in finding an issue.


feat(community): add mmr search to pgvector

Adds new features to pgvector class

Add flag to include vectors in similarity search
maxMarginalRelevanceSearch implementation

@jacoblee93 Please take a look when possible.


Add missing audio file extensions

Fixes

Fixes #1347 by @dhruvkb / @obulat
Description

This shows all the currently existing audio file types that we have. Although most of them still don't work due to pending resolution of WordPress/openverse-api#615.
Testing Instructions

Run the project and see the new types in the filter sidebar

Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[N/A] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

I think we should just go ahead and rename the filter from "file type" to "extension", because file type can mean format, and that is quite ambiguous. For example, oga can mean several different things.

Interesting! TIL that "OGA is an audio extension for OGG file". That makes me wonder if we should actually group all the extensions by file type, as OGG can group .opus, .ogm, and more. The same will apply to .mp3 and .mp32 for example. It will avoid cluttering the search sidebar.
I'll do as you say here and we can change it later if it is agreed between all 🙂

Side note: It's cool to see the visual regression tests reflect a translation update, but unfortunate that it can happen!

Olga and I discussed a way around this would be to use an automatically generated locale in Playwright tests. For example a pirate locale could be a good one to test LTR languages. I wonder if there is a "pirate" equivalent for an RTL language. Can we automatically generate Arabic or Hebrew script in a safe and consistent way that would be useful for testing the application?

What is a Pirate Locale

This link describes it well: https://make.wordpress.org/polyglots/2017/01/05/hi-4/
Basically a fake locale that is auto-generated based on the default local (en in our case) just for testing i18n stuff. Something for another PR for sure! Was just mentioning it could be an option for removing that external factor from e2e tests.

why are Playwrithg tests in the CI not getting updated RTL strings as in local

I don't know... that is really odd. I'm looking at this PR now and will see if I can reproduce it. Sorry for the delay in the review!

Hmmm, when I run the tests locally they didn't have an issue with the translation updates :thinking: How curious...


[chore] add vendor/ssl.lua and remove reset_db()

ssl.lua needs to be installed whenever Kong is installed (imagine if ones purges the rocks tree). It is a dependency when installing OpenResty but also when kong is running.
removed a reset_db call from one of the tests, and removed the method from the spec_helper (migrations make the tests too slow to afford using it)

Inifinite loop on the tests causing it to break, same as locally.


Make sam deploy output more user-friendly

Describe your idea/feature/enhancement
The new sam deploy command is fantastic, but the output can be slightly more UX-friendly.
For example, the output is nicely formatted, but when I try the copy the URL, which is in the multiple lines, I get the URL with spaces.
Here's the screenshot of the action:

Expected result of the copy action:
https://abcdefghij.execute-api.eu-central-1.amazonaws.com

Actual result:
https://abcdefghij.execute-api.eu-
                                                      central-1.amazonaws.com

or this, if the paste destination doesn't support new lines:
https://abcdefghij.execute-api.eu-                                                      central-1.amazonaws.com

It's even more confusing if I try to copy something with the long key. For example, if I try to copy the Arn from the previous screenshot, I'll get the "Arn" string in the middle of my actual Arn.
Proposal
Not sure what's the best solution to this, but it would be much more useful if I can copy the output.

Thanks for the feedback! this is definitely on our mind to fix.

I would like to see this with a --output=json or similar flag to allow processing the output with jmespath or jq.
If appropriate I can file a separate issue for this, but to me they're related.


Touchup (12 Nov 2017): Update Documentation + minor fixes

Documentation Updated.
Re-collation of files.
Added 3 more Easter egg messages to Why Command.

Changes Unknown when pulling b15b708a0890d5d121f925dcb1167842a38ce550 on arnollim:Touchup into ** on CS2103AUG2017-T13-B1:master**.


connectors are not showing properly when bidirectional edges

https://ibb.co/2tfFqjw
In the above image department3 to Student3 line is not shown at edges
can this be done like this?
https://ibb.co/2cYzhWk
having curves on edges if they are hidden

Hey @srinivasuk
this is not built in but you could implement a custom edge logic https://reactflow.dev/docs/api/edge-types/

we are not familiar with that side of development, anyone in your team
available to do it. I can pay for the cost. thanks
On Fri, Jul 23, 2021 at 4:57 AM Moritz Klack @.***>
wrote:

Hey @srinivasuk https://github.com/srinivasuk
this is not built in but you could implement a custom edge logic
https://reactflow.dev/docs/api/edge-types/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wbkd/react-flow/issues/1379#issuecomment-885529696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHUJ7T6XZ3ILPN26JVB7CDTZE4HBANCNFSM5A2K7VVA
.

{
const edgePath = getBezierPath({
sourceX,
sourceY,
sourcePosition,
targetX,
targetY,
targetPosition,
});
const markerEnd = getMarkerEnd(arrowHeadType, markerEndId);
const [edgeCenterX, edgeCenterY] = getEdgeCenter({
sourceX,
sourceY,
targetX,
targetY,
});
return (
<>

currently we are using this edge, how can we change this code to show edges
clearly at connectors?
On Fri, Jul 23, 2021 at 3:30 PM Srinivasu Kota @.***>
wrote:

we are not familiar with that side of development, anyone in your team
available to do it. I can pay for the cost. thanks
On Fri, Jul 23, 2021 at 4:57 AM Moritz Klack @.***>
wrote:

Hey @srinivasuk https://github.com/srinivasuk
this is not built in but you could implement a custom edge logic
https://reactflow.dev/docs/api/edge-types/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wbkd/react-flow/issues/1379#issuecomment-885529696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHUJ7T6XZ3ILPN26JVB7CDTZE4HBANCNFSM5A2K7VVA
.

[image: image.png]
i am trying to get like this as in google drawing (google docs)
[image: image.png]
when i move the box line gets automatically adjusted to curve around the
boxes and comes over the boxes
i feel this should be default behavior
Appreciate if someone point us in right direction, thanks
On Fri, Jul 23, 2021 at 3:57 PM Srinivasu Kota @.***>
wrote:

{
const edgePath = getBezierPath({
sourceX,
sourceY,
sourcePosition,
targetX,
targetY,
targetPosition,
});
const markerEnd = getMarkerEnd(arrowHeadType, markerEndId);
const [edgeCenterX, edgeCenterY] = getEdgeCenter({
sourceX,
sourceY,
targetX,
targetY,
});
return (
<>

currently we are using this edge, how can we change this code to show
edges clearly at connectors?
On Fri, Jul 23, 2021 at 3:30 PM Srinivasu Kota @.***>
wrote:

we are not familiar with that side of development, anyone in your team
available to do it. I can pay for the cost. thanks
On Fri, Jul 23, 2021 at 4:57 AM Moritz Klack @.***>
wrote:

Hey @srinivasuk https://github.com/srinivasuk
this is not built in but you could implement a custom edge logic
https://reactflow.dev/docs/api/edge-types/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wbkd/react-flow/issues/1379#issuecomment-885529696,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHUJ7T6XZ3ILPN26JVB7CDTZE4HBANCNFSM5A2K7VVA
.