add mix_bernoulli_em_mnist.py

Decryption
Added mix_bernoulli_em_mnist.py
Issue
probml/pyprobml#793

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

6 of 23 (26.09%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 23.923%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

probml_utils/mix_bernoulli_em_mnist.py
6
23
26.09%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
272

Relevant Lines:
1137

💛  - Coveralls


Added new version 2024-03-01 of API Center resource manager.

ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to Step 2, "ARM Review", for this PR.

Breaking changes review (Step 1)

If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
Getting help

For general PR approval workflow, see the diagram at the top of this comment.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

{

Is this new stable version based on a preview API version? If yes, is that preview version already reviewed by ARM in past? If yes, please share that PR and also, make a first commit same as that preview API version and make incremental changes on top of that base commit. That will help ARM Reviewer to review these changes.

Refers to: specification/apicenter/resource-manager/Microsoft.ApiCenter/stable/2024-03-01/apicenter.json:1 in 136d8d0. [](commit_id = 136d8d025dd770313a703c15e91e4331e3a687cc, deletion_comment = False)


OTA: Reuse HTTP connection when performing partial HTTPS downloads (IDFGH-13949)

Is your feature request related to a problem?
If OTA is configured to do a partial HTTP download over HTTPS, I noticed that it closes the HTTP connection after every 4 KB that it has received, and opens a new connection for the following 4 KB. It would be nice if it could reuse this connection to save time on DNS requests and setting up / tearing down HTTP and TLS sessions.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context.
No response

Hello @admarschoonen-joule ,
Yes, we agree with you and have already identified this issue. We’ll be implementing a fix for it shortly.

Hi @nileshkale123 ,
Thanks for you quick reply. Can you give an estimation of when this fix will be released and which ESP-IDF version number will have the fix?
Thanks in advance.

Hi @nileshkale123 ,
Thanks a lot! Looking forward to the improvement!

Hello @admarschoonen-joule,
I’m attaching a patch that enables the session resumption feature in the advanced_ota example in IDF. This approach maintains the current flow, where each range request initiates a new connection, rather than persisting a single connection across all range requests.
Although session resumptionis quite slower than using a fully persistent connection, it provides significant advantages in low-bandwidth or unreliable network conditions and requires fewer resources.
Please review the patch and let us know your thoughts. If you have any specific requirements or preferences regarding the implementation, feel free to share them so we can work closely to meet your needs.
partial_ota.txt

Thanks @admarschoonen-joule ,
The MR is under review. We will merge it soon and it will be backported on previous supported versions as well.


x/mobile/exp/audio/al: offset setters are missing

Continuing the discussion started at https://github.com/golang/go/issues/11298#issuecomment-168354198...
Source offset setters are missing from the package and the offset must represented in float32 for more precision. Suggested APIs are below.
// OffsetSeconds returns the current playback position of the source in seconds.
func (s Source) OffsetSeconds() float32 {
return getSourcef(s, paramSecOffset)
}

// SetOffsetSeconds sets the current playback position of the source in seconds.
func (s Source) SetOffsetSeconds(sec float32){
setSourcef(s, paramSecOffset,sec)
}

// OffsetSample returns the sample offset of the current playback position.
func (s Source) OffsetSample() int32 {
return getSourcei(s, paramSampleOffset)
}

// SetOffsetSample sets the sample offset of the current playback position.
func (s Source) SetOffsetSample(i int32){
setSourcei(s, paramSampleOffset,i)
}

// OffsetByte returns the byte offset of the current playback position.
func (s Source) OffsetByte() int32 {
return getSourcei(s, paramByteOffset)
}

// SetOffsetByte sets the byte offset of the current playback position.
func (s Source) SetOffsetByte(i int32){
setSourcei(s, paramByteOffset,i)
}

/cc @splace

i think it must be because openal is dynamically extensible, that it makes everything accessible via only a few general functions, that only vary by parameter type, and then uses another parameter to indicate the actual thing being set/read.
which is how offset could be int or float, when float includes the same functionality.
i did look through the spec briefly, looking for other omissions that might be sensibly added, and didn't find any more, but i guess there will be others that are effectively only present because of this extensibility.
the approach seems to have been to have anything that possibly makes sense, maybe also to lessen type conversions?
but, the Go wrapper's current approach of hiding the parametrised calls, by sending each through a separate function, stops the dynamic extension mechanism, in that, it means any extensions would require functions to be added statically.


Add Windows native support

This should be the proper way to do this, but criticism is welcomed.
I didn't remove any of the comments related to Windows (in the portable implementation), but it may be best to remove them.

I've removed the redundant calls to LockFileEx and rely only on the OS share lock now. Behaviour should be the same.

now as a CL in gerrithub instead: https://review.gerrithub.io/#/c/403981/


Add a new Log message proto

This should technically be a "diff base" on https://github.com/datacommonsorg/import/pull/8. The main thing to review here is Debug.proto.
It is meant as a common message for use in lint and eventually in the pipelines.

Sorry, I merged by mistake... let me revert.

Is there a linter that you're using at the moment? I found a couple that could work:

https://github.com/checkstyle/checkstyle
https://github.com/diffplug/spotless/tree/main/plugin-maven

Yes, it is integrated with google-java-format (https://github.com/datacommonsorg/import#coding-guidelines).


demo project

I did a short test in a minimal angular cli project.
https://github.com/kernelport/ngx-image-zoom-test/blob/master/src/app/app.component.html#L7
It works when the options are provided witout [].
It is a typo in the Readme or an API change in angular?
This is working:
<ngx-image-zoom
    thumbImage="https://wittlock.github.io/ngx-image-zoom/assets/thumb.jpg"
    fullImage="https://wittlock.github.io/ngx-image-zoom/assets/fullres.jpg"
></ngx-image-zoom>

This not:
<ngx-image-zoom
    [thumbImage]="https://wittlock.github.io/ngx-image-zoom/assets/thumb.jpg"
    [fullImage]="https://wittlock.github.io/ngx-image-zoom/assets/fullres.jpg"
></ngx-image-zoom>

I believe if you want to use the brackets but not a variable then you need to quote the URL. Like so:
<ngx-image-zoom
    [thumbImage]="'https://wittlock.github.io/ngx-image-zoom/assets/thumb.jpg'"
    [fullImage]="'https://wittlock.github.io/ngx-image-zoom/assets/fullres.jpg'"
></ngx-image-zoom>

With the double brackets there's an expectation of typescript code on the right hand side of the equals sign. Maybe that's not strictly the correct reason, but adding the single quotes should make it work. But if you have a hardcoded value in your template you could just go without the brackets.

Thank you for explanation. It works

I m trying to use it but having trouble as i don't have thumbnails saved during the time of uploads so i m trying to create a preview on the fly when the file gets loaded by not able to achieve as the file is retrieve from s3 and then it is converted to a smaller size, here i m not able to getting going. I have tried sending base64 image from the backend but still it is not working file as i m not able to convert the base64to image correctly and  not sure what would be the right approach to resize and pass it to
[thumbImage] = ?


Alternative way to deal with versioning?

Currently, the turborepo flake uses the inputs.turbo.url = "github:alexghr/turborepo.nix/v${version}";, which isn't convenient for a monorepo that is growing in size, as the flakes have to be manually updated. I was wondering if it is possible to use string interpolation to extract value of turbo from package.json? The version variable can be set by the user to parse either the package.json or the lock files. Since this isn't possible in inputs, perhaps this could be done in outputs?

That's a good question. I haven't considered it this way. My use case for this has been to 'pin' the exact version of turborepo in package.json and flake.nix and ensuring they're updated in lockstep.
We could export a library function from this flakes that takes a turborepo version and generates a derivation for it, but it would fall upon the user of the flake to provide an appropriate sha256 for the sources (without sha256 Nix can't ensure purity) so it makes things even more difficult.
I think a better approach would to see if we can avoid having to depend upon turbo in package.json entirely. AFAIK the turbo npm package is just a thin wrapper around the actual Go binary that does the initial installation (based on os/architecture). Nix already handles installing the right binary, we might be able to use it directly and bypass npm.
I'll do some testing and get back to you on this issue.

hey @Ashvith10. I had a play with just declaring turbo as a flake input, without adding it to package.json and it does work. Everything you'd use turbo for (including the daemon) works as expected. The problem is if the project has developers that don't use the Nix development shell then turbo won't be available in their shell.
I'll leave this ticket open if anyone wants to contribute ideas to solve it. I can't think of anything that would have better UX than declaring turbo just in the Nix dev shell (and ensuring all devs use Nix) or declaring turbo in both package.json and flake.nix and keeping the two versions in sync.

@alexghr How about using @pnpm/lockfile-file to read version from the lockfile? It can read for all three package managers: npm, yarn and pnpm. What do you think? The only issue would be that if we try to update inside the flake environment, it might probably brick, unless there's a shell-hook we can run after every command is run to verify if package lockfile and flake Turbo binaries use the same version. What do you say?

Hey, sorry for ghosting you (for almost two months :scream:). using @pnpm/lockfile-file sounds like a good idea. How would you check that the version matches the input in flake.nix? Or would this check that turbo --version matches the expected version in the lockfile?
I think this is great addition to this flake but I'd keep it as a separate installable so people that want to just use a pinned version can continue doing so and people that want to track the turborepo version from their lockfile can use this approach. WDYT?

@alexghr I've not worked on this myself, and I'm also not a flake or nix expert, but from examples I've seen, like the DotNET module (using Shellscript) and Vim plugin system (using Python), you're gonna have to generate a custom Nix file in the directory using a programming language of your choice.
I am not sure if "meta-programming" is the right word to use here, but something like that, I guess. Here's one such example - essentially, we are automating packaging the plugins for Vim, but similar stuff can be done in this case as well - we might want to generate the package.nix file, which is referenced by the flake.nix file?


replace SANGLE with SampleAngle

The instrument has a couple of ways to define the sample angle. The newer PV SampleAngle should replace SANGLE in the data reduction.
Here's a PR that replace it in the UI only, so that the values that we reported are correct.
Before merging, we would need to update the MagnetismReflectometryReduction algorithm to do the same. We can keep the same parameters, but it should pull SampleAngle rather than SANGLE.

@mdoucet The mantid version has been updated. Can you rebase latest next onto your branch and the push again to remote, please?


dbeaver/dbeaver#16867 remove minimize for all child dialogs

We want to remove minimize from child dialogs because most systems handle them terribly. This issue is especially poorly handled in macOS.
Linked pr: https://github.com/dbeaver/dbeaver-ee/pull/1756
Closes #16867

Verified


Use stricter version pinning for GitHub Actions

Pin actions from the @actions and @artichoke orgs with full git tags. Pin all other actions with git SHA corresponding to a tag.

bypassing broken clippy build


⚠ UdeS website (prod) has degraded performance

In c280a41, UdeS website (prod) (https://www.usherbrooke.ca/) experienced degraded performance:

HTTP code: 200
Response time: 1075 ms

Resolved: UdeS website (prod) performance has improved in acebb1b.


Windows Defender says 1.0.11 is a trojan

Please first read the documentation. Problems affecting the client, e.g. Plex or Emby should be reported there in the forum.
Describe the bug
When downloading or running 1.0.11 Windows Defender is currently saying it is a Trojan
Server (please complete the following information):
Windows 10
To Reproduce
Download the compiled version of 1.0.11
Expected behavior
NA
Screenshots

Desktop (please complete the following information):
NA
Additional context
When 1.0.11 was first released there were no issues. It looks like this behavior started today at 4:04AM CDT. 1.0.10 works without issue. I ran the exe through VirusTotal and it looks like a few other vendors is flagging the program as a virus as well.
https://www.virustotal.com/gui/file/1f540d6e87022a0048906aad386a8ab42ad77e226ec0adf6851199990bf75e5e?nocache=1
I understand that this is likely a false positive however I wanted to make sure you were aware.

Windows Defender is not showing it as malware anymore. VirusTotal still has 4 scanners that are alerting on it. I am confident it is a false positive.
As a side note, I did try downloading and uploading it to VT from a Linux box I have and the hash was the same and VT still alerted on those few scanners.


"outputEncoding" config not work

how to specify findbugsXml.xml file encoding?
Otherwise, an error will occur while generating reports by Invalid byte 1 of 1-byte UTF-8 sequence.

Like this?
<jvmArgs>-Dfile.encoding=UTF-8</jvmArgs>

findbugs only does utf-8 output
/**
 * Write the BugCollection to given output stream as XML using a UTF8 encoding.
 * The output stream
 * will be closed, even if an exception is thrown.
 *
 * @param out
 *            the OutputStream to write to
 */


[Docs] Fix: Display adapters' earliest version

Description
It fixes the display of earliest version of adapters on the docs.
This PR fixes #4589
Notes for Reviewers
Screenshots:

Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit 92099dae67c52709cdcf3b3b6508e5db24308eef at: https://6190d2bbefdea9841fa36867--meshery-docs-preview.netlify.app


[Help wanted] how to configure envoy to send http notification to all the members in the cluster without load balancing (broadcast)?

existing configuration sending a notification to the only one pod based on the lb_policy =Round Robin, but we have a case to send the notification to the all the members of the cluster (like a broadcast) , Kindly do the needful to solve the problem,
Envoy version: 1.14.2

'@type': type.googleapis.com/envoy.config.cluster.v3.Cluster
name: envoy-ueauth-envoy-2-0
type: STRICT_DNS
connect_timeout:
seconds: 18
http2_protocol_options: {}
lb_policy: ROUND_ROBIN
transport_socket:
name: envoy.transport_sockets.tls
typed_config:
"@type": type.googleapis.com/envoy.extensions.transport_sockets.tls.v3.UpstreamTlsContext
common_tls_context:
tls_certificates:
- certificate_chain:
filename: "/aus-certs/server.pem"
private_key:
filename: "/aus-certs/key.pem"
password:
inline_string: "server@123"
validation_context:
trusted_ca:
filename: "/aus-certs/trusted.pem"
alpn_protocols: [ "h2", "http/1.1" ]
dns_resolvers:

socket_address:
address: <IP_ADDRESS>
port_value: 8600
protocol: UDP
load_assignment:
cluster_name: envoy-ueauth-envoy-2-0
endpoints:

lb_endpoints:

endpoint:
address:
socket_address:
address: "<IP_ADDRESS>"
port_value: "8445"
protocol: TCP

@snowp do you have any suiggestions?

@snowp Kindly share your thoughts to solve the problem. and thanks in advance

This is not something that's currently supported, so this would be have to be a feature request.
I can imagine this being implemented as a new route action (broadcast to all hosts for a given cluster) or even as a separate filter if we want to avoid adding the complexity to the router (at the cost of missing out on some router features like requset_headers_to_add).
@mattklein123 might have thoughts on how this would be implemented

@mattklein123 might have thoughts on how this would be implemented

Yeah there is no way to do this today. It would require substantial changes to the router or possibly even a new filter to do it. It's certainly possible but would require a lot of design thought and a full proposal.

@snowp @mattklein123 @yanavlasov Thank you all for the quick response, is there any chance that can be included as a feature in upcoming releases?

@vsharathis It is unclear to me if there is anyone else beside you asking for this feature. Can you try implementing it as a filter yourself?

@vsharathis It is unclear to me if there is anyone else beside you asking for this feature. Can you try implementing it as a filter yourself?

I would be interested in such a feature. Can someone point me to how would I implement it in a filter?

We have similar requirement for broadcasting the request to multiple endpoints  in the down stream.
Would be good feature if ENVOY provides this

Could some body help if this facility is given in Envoy or any alternate methods


What exactly is AdminLTE used for?

Can you explain what AdminLTE is used for?
What sort of internet business or activity is someone performing while using AdminLTE?

AdminLTE is a frontend template used for building websites and web apps.
By itself it does nothing. It merely provides the components and frameworks that allow one to present data and information in a sensible and eye-please manner.

In simple English Adminlte is the design that u can use for ur web applications either before the log into ur website of after they have logged in or both.


Make command text accessible through commands

Explore ways to pass the actual command as well as the args.

Boop. https://github.com/SpiderlingStudio/Commandler/releases/tag/1.4.0
CommandContext#getCommand


Smart Contract Script Builder

Feature
The Script Builder is basically a copy of NEO-VM's script builder, slightly remodeled for JS (because we can't overload in JS 😞 )
This will introduce the capability to parse operations and params into a script that can be carried by InvocationTransaction to trigger smart contracts. There is a basic test suite that tests basic NEP5 methods. Scripts have been hand tested through invokescript on a self hosted v2.3.3 node to make sure they are working.
Sidenotes

Added an .editorconfig file that defaults to 2 spacing. I find this file more handy than commiting a .vscode folder for now because it is more general.

Testing globals (chai) is now extracted into a common.js which is preloaded by the test script. This eliminates the crappy feeling of copying import chai from 'chai' for every new test file.

This PR seems to have cleaned up a bunch of the console.log entries from src/utils.js, but those changes haven't been compiled into lib/index.js.

Not gonna compile it until we stabilise it and do a proper release. There might be breaking changes that will break latest builds for Neon Wallet and we do not want that.

@snowypowers Oohh... Neon pulls in from master? 😱

OK. I just checked. Neon wallet pulls a specific commit so we wont affect it. However, i would like to only update lib/index.js when we are ready. Doing that means we should tag as release imo. Not sure about the flow here but its up to Ethan to release when he feels like it.

Seems weird to me that there's committed code that doesn't reflect changes that have been made. If it was ready for merge, surely it was ready for merge in both places?
Just my opinion of course, and happy to defer to whatever Ethan decides.
I'm really just anxious to get rid of that dang console.log 😛

Yeah let's rebuild lib/index when it is fully updated
On Wed, Oct 4, 2017 at 8:05 PM Yak Jun Xiang<EMAIL_ADDRESS>wrote:

OK. I just checked. Neon wallet pulls a specific commit so we wont affect
it. However, i would like to only update lib/index.js when we are ready.
Doing that means we should tag as release imo. Not sure about the flow here
but its up to Ethan to release when he feels like it.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/CityOfZion/neon-js/pull/27#issuecomment-334346111,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFeEockHew-B6MY5goZ0eDWxhobbMzkks5spEdrgaJpZM4PrznN
.

Will be pretty soon, log won't be around long 😊
On Wed, Oct 4, 2017 at 8:10 PM Ethan Fast<EMAIL_ADDRESS>wrote:

Yeah let's rebuild lib/index when it is fully updated
On Wed, Oct 4, 2017 at 8:05 PM Yak Jun Xiang<EMAIL_ADDRESS>wrote:

OK. I just checked. Neon wallet pulls a specific commit so we wont affect
it. However, i would like to only update lib/index.js when we are ready.
Doing that means we should tag as release imo. Not sure about the flow here
but its up to Ethan to release when he feels like it.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/CityOfZion/neon-js/pull/27#issuecomment-334346111,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFeEockHew-B6MY5goZ0eDWxhobbMzkks5spEdrgaJpZM4PrznN
.


Bump Golang version to 1.15

As per title. Other repos have long moved.
/assign @nak3

/lgtm

It's time we moved to 1.16 :/


No function clause match on closed connection

I'm using Redix 0.6.1:
 "redix": {:hex, :redix, "0.6.1", "20986b0e02f02b13e6f53c79a1ae70aa83147488c408f40275ec261f5bb0a6d0", [:mix], [{:connection, "~> 1.0", [hex: :connection, repo: "hexpm", optional: false]}], "hexpm"},

I received the following error (with some info elided):
erlang error: {{:function_clause, [{Redix.Connection, :handle_info, [{:receiver, #PID<0.4376.211>, {:tcp_closed, #Port<0.10631>}}, %Redix.Connection{backoff_current: 500, opts: [socket_opts: [], sync_connect: false, backoff_initial: 500, backoff_max: 30000, log: [disconnection: :error, failed_connection: :error, reconnection: :info], exit_on_disconnection: false, host: "...", port: ..., password: "..."], receiver: nil, shared_state: nil, socket: nil}], [file: 'lib/redix/connection.ex', line: 223]}, {Connection, :handle_async, 3, [file: 'lib/connection.ex', line: 810]}, {:gen_server, :try_dispatch, 4, [file: 'gen_server.erl', line: 601]}, {:gen_server, :handle_msg, 5, [file: 'gen_server.erl', line: 667]}, {:proc_lib, :init_p_do_apply, 3, [file: 'proc_lib.erl', line: 247]}]}, {:gen_server, :call, [#PID<0.550.0>, {:commands, [[...]], #Reference<0.0.44564487.222310>}, 5000]}}

However, it does appear that handle_info would match. So I'm not sure I'm reading it right.
def handle_info({:receiver, pid, {:tcp_closed, socket}}, %{receiver: pid, socket: socket} = state) do

It seems like it's something that Redix is trying to handle.

According to your error, the handle_infocall would not not match indeed, the pattern match expects both the receiver pid in the message and the receiver pid in the current state to be the same, while the receiver pid of the current state is nil.
It is just a hunch, but I do believe that you may be receiving the same disconnection message more than once. The first one updates the state to nil. The others won't match anymore.
A simple default handle_info clause that doesn't do anything (just like the default one), should fix this behavior.
I'm opening a PR really soon.

This should not happen anymore with the new gen_statem rewrite. If something similar happens, a new issue will be needed. Thanks for reporting! 💟


Round edges

rounded the edges on some of the cards by 5px

Closes #20

Good Job : )

Can you stash the changes of package-lock.json? It's making netlify mad (this isnt your fault, Desktop probably detected changes when u ran npm install and tried to commit them)

Closing this PR; I took care of it by committing your changes separately. If you see that package-lock.json got caught in your GitHub Desktop commit, just uncheck it unless you actually want it to change.


update rds.tf

the version is no longer supported My SQL Supported Versions so updated to latest supported version for mysql 5.7.xx

Looks good. Thx for the interest.


[FEATURE]:DICE SHOWDOWN GAME

Description
This Game is an exciting multiplayer game where two players compete to roll the highest number on their dice. Experience the thrill of luck and strategy as you aim to outscore your opponent in this engaging game suitable for players of all ages.
I am in GSSoC'23.
Screenshots
No response
Additional information
No response

Hi @rishiofficial , I have assigned this task to you. Kindly complete your work within a week. Make sure you will make your game in react because it's react based repository. Good luck!


update(spire-agent): upgrade dependency packages

This PR contains the following changes:
Updated

net package to v0.17.0 from  v0.7.0
gRPC package to v1.56.3 from v1.53.0
docker package to  v24.0.7 from v23.0.1+incompatible
distribution package to v2.8.2-beta.1 from v2.8.1+incompatible
go-jose to v3.0.1 from v3.0.0

@vishnusomank Can you please take a look at the failing actions?

hi @soorya-knox , the GH action is failing due to token issue which can be solved by devops. Also we are not  using this action to build spire. We have a separate repository for spire builds so this action is not required IMO
cc: @seswarrajan

@vishnusomank Can you please fix checks?

@vishnusomank Can you please fix checks?

The checks are failing because of github token expired @rajaSahil also we are not building spire in this repo. we have a separate repo in accuknox to build spire before releasing

@vishnusomank Can you please fix checks?

The checks are failing because of github token expired @rajaSahil also we are not building spire in this repo. we have a separate repo in accuknox to build spire before releasing

Then we can remove it from here, it's giving false impression IMO.

@vishnusomank Can you please fix checks?

The checks are failing because of github token expired @rajaSahil also we are not building spire in this repo. we have a separate repo in accuknox to build spire before releasing

Then we can remove it from here, it's giving false impression IMO.

yes. i'll raise a PR to remove the GH action from this repo


ctrl+v is not working in window terminal when i want to paste some value

In window terminal, when i prompt the some label using promptui, its prompted but when i tried to give some value to label using crtl+v some special char. appear to terminal instead of actual value

got the same issue, I probably wouldn't recommend using this package to anyone.
this kind of issues makes it hard developing cross platform software.
now I have to get a new package instead of this one or rely on my own code.


[RFC] Github Actions

So I haven't figured out how travis conditions worked, b/c for me they look similar.
Any suggestions would be highly appreciated.
~This incorporates a bump for jsdoc, so Pixi can properly build with node version > 10.~

Wow great work! Thanks for doing this and putting in the effort. This has been high on my todo list. Directionally, removing Travis is the correct move.
The JSDoc bump is a consequential change.  My preference would be to pin GitHub actions to node 10 until we can fix all those underlying issues as another PR. Because our typings also depend on JSDoc, this is a nontrivial change and requires a close attention.
Is that ismobilejs change a fix for #6511? Maybe we can do that as a separate PR?

@bigtimebuddy thanks, will handle mentioned issues.
Regarding ismobilejs – I'm not sure how it worked on travis, probably because of cached node_modules.
On my forked repo travis it was failing with "missing argument", see https://travis-ci.org/github/cursedcoder/pixi.js/builds/673997141 and https://github.com/cursedcoder/pixi.js/runs/580341097?check_suite_focus=true#step:5:314.
##[error]packages/settings/src/utils/isMobile.ts(6,18): error TS2554: Expected 1 arguments, but got 0.
@englercj thanks for the review, I'll ensure it also deploys correctly on my fork.

Could you update to the latest, we fixed the ismobilejs issue.

@bigtimebuddy done.
@englercj

You can see – tags, branches and the latest release are deployed.

My final request is that I'd like to combine the two workflows together and instead just guard the deploy steps inline.
I'd rather do this becuase right now they both are building the library anyway. Right now the flow is: install, build, test, if release then install, build, deploy. I'd rather the flow be: install, build, test, optionally deploy.
Here is an example of what I did on gl-tiled:
https://github.com/englercj/gl-tiled/blob/1c0d44e5a02f3ad567bc03ce5e34742f4b435258/.github/workflows/main.yml#L25-L33

@englercj thanks, sounds about right.
However, in addition to build-matrix, this will lead to several deploys per build.
I'll dive deeper into the docs, I have few ideas on how to improve that.

Currently the matrix only has one entry, so there is only one build.
But you can just add another condition to the if to only deploy on a specific matrix entry.

Also @cursedcoder can you revert changes to package-lock.json? Those changes are unnecessary for this PR and is probably left-overs from bumping jsdoc.

@bigtimebuddy @englercj should be OK now.
Please check.

@bigtimebuddy make sure to update the repository's secrets before it gets merged.
Thanks.

@cursedcoder is there anything else I need to setup in the settings other than secrets?

Also could you fix the build status badge in the readme to not use Travis?

@bigtimebuddy added the badge.
Checklist before merging:

[ ] Add AWS secrets
[ ] Add codecov token secret

Thinking out loud – can't we instead merge this into github-actions and ensure it builds in this repository?

Yes, good idea.


feat: app child key derived from wallet master key

fixes #724
fixes #744

For some reason, if you do a fresh install (complete setup and go to Home page), the app isn't quitting on doing Ctrl+C

For some reason, if you do a fresh install (complete setup and go to Home page), the app isn't quitting on doing Ctrl+C

Oh sorry, I broke that when trying to handle relay.Context().Done() when there is no subscription on legacy app wallet pubKey.
Don't have much experience with go contexts, but I think it should be fixed now by using a select and handle both case <-ctx.Done(): and case <-relay.Context().Done():
Let me know if you have a chance to re-test this and confirm it is working both doing Ctrl-C and by restarting a local test relay (should reconnect and re-subscribe each app).
Thanks for catching this!

For some reason, if you do a fresh install (complete setup and go to Home page), the app isn't quitting on doing Ctrl+C

Oh sorry, I broke that when trying to handle relay.Context().Done() when there is no subscription on legacy app wallet pubKey. Don't have much experience with go contexts, but I think it should be fixed now by using a select and handle both <-ctx.Done() and <-relay.Context().Done() Let me know if you have a chance to re-test this and confirm it is working both doing Ctrl-C and by restarting a local test relay (should reconnect and re-subscribe each app). Thanks for catching this!

@frnandu that sounds like a good solution. Thanks @im-adithya !


Metrics: Daily Gas Used and Gas price

Description
A table tracking the daily gas usage and gas price
TODO

[ ]  Design the schema for this table
[ ]  Get confirmation on schema
[ ]  Build schema
[ ]  Schema build review
[ ]  Submit Pull Request

Propsed Schema

[ ] Date (day)
[ ] Total Gas Used
[ ] Average Gas Price


Misc minor refactoring for optimizations/readability

There seems to be an actual problem with some of the changes (contains on Option seems to not exist).

Other improvements look good to me (mostly due to readability not speed)

The code changes LGTM now. Could you please rebase on top of the latest 0.6.x, and squash everything in a single commit whose message is the title of this PR? I don't think there's any need to keep those small changes in individual commits.

Thanks for reviewing.
Squashed into one commit.


Portable Version PersistantData.xml in wrong location

It's saved to AppData (Roaming) instead of the portable folder.

Cannot reproduced it.
Make sure you are launching application from its portable folder and that there is also Portable.ini file.

It might be the .zip that is causing the problem

Yep, the .zip file isn't portable as it wants to write to the AppData path

I think it would be better to remove .zip from release assets. The whole point of making installer was to avoid security issues caused by extracting from blocked .zip.

Alright will do
On Jul 12, 2015 10:33 PM, "l0g0sys"<EMAIL_ADDRESS>wrote:

I think it would be better to remove .zip from release assets. The whole
point of making installer was to avoid security issues caused by extracting
from blocked .zip.
—
Reply to this email directly or view it on GitHub
https://github.com/EmmittJ/PoESkillTree/issues/161#issuecomment-120818916
.


Can I translate this guideline to Korean?

I want to translate this guideline by forking a repo if you approve.

Yes sure. If you don't mind keeping it updated

Thanks!

@HyunSeob our chinese version is up. Are you still interested ?

Sure, I want Korean version, not Chinese.

I know @HyunSeob, what I mean is you can always have a pull request back here with your Korean version and add a link to the main README.md file

Ah, I get it. Thanks.

@1984weed Amazing. Please make a pull request when you're done :)

where is the Chinese version? has it been done? if not, I will work to make progress on it~ @vpanjganj


Fix changelog for v0.20.1 to have proper links

Issue #, if available:
Description of changes:
Fix links in changelog
Testing (if applicable):
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/cherry-pick release-0.20


[cli] Add asteroid-infer CLI to enhance/separate from the command line

The is a first iteration, it works, but it's not optimal and raises some flaws of the current code.
Usage
asteroid-infer url_or_path --files filename
asteroid-cli 'mpariente/ConvTasNet_WHAM!_sepclean' --files tmp.wav
asteroid-cli 'mpariente/ConvTasNet_WHAM!_sepclean'  --files a_folder a_filename a_glob/**/*.wav

files can be a list of filenames, foldernames and globs, we'll handle it. In case of a folder, we check the supported file formats in the folder (recursively)
Cool things

Thanks to BaseModel, we can now load all the models, just given the model pack, which makes it very easy.
Based on  separate, not much new code here.
The --files argument is cool because versatile.

Bad things

soundfile doesn't support all format. Should we have a backup strategy? Librosa supports more formats?
Bash breaks on ! unless we use single quotes on them, avoid ! in the future.
We currently have no idea the sample rate the model supports and this can break everything, we should add it in the saved models
We also don't know the number of sources, so we cannot use the current version of LambdaOverlapAdd, could we remove the n_src argument in LambdaOverlapAdd so that we could use it in the CLI? It can be infered at runtime IMO (and if the (batch, n_src, time) format is not respected, errors will be raised.

@jonashaag and @popcornell please?
Note
You need to reinstall Asteroid with python setup.py develop to have the new CLI modes I think.

Nice and simple!

We can use librosa to load all the files, it uses soundfile and falls back to audioread (ffmpeg on Linux IIRC). But it requires librosa to be installed. We could also try soundfile first and then fall back to librosa. Soundfile will get mp3 support soon https://github.com/erikd/libsndfile/issues/258
I think we should add a check to separate() to make sure it never overwrites any files. That's particularly important now that we have code that goes through entire directories.

Am I correct that users can now install Asteroid and do something like asteroid-infer mpariente/convtasnet-xyz /my/files without ever manually downloading anything? That would be awesome.

We can use librosa to load all the files, it uses soundfile and falls back to audioread (ffmpeg on Linux IIRC). But it requires librosa to be installed. We could also try soundfile first and then fall back to librosa. Soundfile will get mp3 support soon erikd/libsndfile#258

I would use torchaudio here (which is using sox, and recently support for opus and ogg was added) or directly use ffmpeg-python which is supports almost all formats but is quite slow.

Yes maybe torchaudio will be better also because they plan to support more formats in the future.

I'll use torchaudio for loading then.
This will come in a follow up PR because it's not really related to the CLI. Thanks for the feedback

I think we should add a check to separate() to make sure it never overwrites any files. That's particularly important now that we have code that goes through entire directories.

Yes, I agree. It will be in the follow up PR as well.

Am I correct that with this, user can now install Asteroid and do something like asteroid-infer mpariente/convtasnet-xyz /my/files without ever manually downloading any models? That would be awesome.

This is correct. This was also the case without the CLI.
Note that the downloaded models are cached as well.

Only tested locally, we'll need to find a way to correctly test the CLI in the near future.


ntr: normal protein level in APC

PMID:9264466
A cell phenotype in which the amount of a protein measured in the anaphase-promoting complex is  normal when the cell is in the vegetative growth phase of the life cycle.

normal protein level in anaphase-promoting complex FYPO:0005074

edit file: 5b76daaf62c2cd3d78fd2084803054d652f26634
release: 4f80132ec75d13c867405dee757878e442773a77

For this one, variants of the APC exist during meiosis too. Maybe just remove vegetative growth from def and move it up so its phase agnostic?


adds enable/disable feature to MetricRegistry

Toggleable interface is implemented by the metrics which support
enable/disable feature. when metric is disabled, all values
collected so far are reset and any calls to update the metrics
do nothing.

I'm sorry to to say that these features are not on our roadmap.  I appreciate the work you put into this.

Ohh ... I didn't know it. But I think it is quite important feature. It can be use for many things.


:bug: Fix: tevm types

Description
Types were broken because of the way typescript caches types
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #915   👈

This stack of pull requests is managed by Graphite.


Help understanding strange mapping problem in vim with kitty

Steps to reproduce
Hello,
I first reported the issue on kitty's github but it seems the issue is from vim's side.
I noticed that some maps were not working anymore in vim when ran through kitty, notably these 2 remaps:
" See :h i_CTRL-R_CTRL-P
imap <c-r> <c-r><c-p>
" NUL corresponds to ctrl-space
nnoremap <C-k><NUL> :echo 'bar'<CR>

For example with this minimal .vimrc,
" .vimrc file
imap <c-r> <c-r><c-p>
nnoremap <C-k><NUL> :echo 'bar'<CR>
" Store 'foo' in register a
let @a='foo'

running the following commands:
$kitty -c NONE
$vim
<c-k><c-space>
i<c-r>a

should echo 'bar' in the cmdline and insert 'foo' in the opened vim buffer.
But while it works with e.g. gnome-terminal, in kitty nothing is echoed and only 'a' is inserted (as if the mapping was not properly interpreted).
By trial and error i found that adding the line
# ^R was obtained by pressing <c-v><c-r> while in insert mode in vim, and corresponds to the byte 0x12.
map ctrl+r send_key ^R

to kitty.conf fixed the issue with , but i have no idea why it works and what was the problem in the first place.
Any idea about what causes this behavior?
Thank you very much for your help.
Expected behaviour
The following maps should have the same behavior in kitty than in other terminals:
imap <c-r> <c-r><c-p>
nnoremap <C-k><NUL> :echo 'bar'<CR>

Version of Vim
My vim version is 9.0, kitty is 0.35.2
Environment
OS: Ubuntu 23.10 mantic
$TERM: xterm-kitty
Logs and stack traces
No response

<NUL> does not necessary correspond to ctrl-space. It corresponds to the 0 byte. Is there any reason you are using <NUL> instead of <C-Space> in the mapping?

For <C-k><NUL>, <NUL> isn't equivalent to Ctrl-Space in kitty. Instead you can use <C-Space>. I'm not sure if this is a bug.
For <c-r><c-p>, I think this is a bug. The problem here is that this STRNCMP() does not work as intended when LHS is non-simplified while RHS is simplified:
#ifdef FEAT_EVAL
		STRNCMP(map_str, save_m_keys != NULL ? save_m_keys : mp->m_keys,
								(size_t)keylen)
#else
		STRNCMP(map_str, mp->m_keys, (size_t)keylen)
#endif
		   != 0)

A possible solution is to (for non-expression mappings) do this comparison at mapping definition time, where the simplified LHS is available.

Thank you,
Using  instead of  seems indeed to be a 'hack' that is needed for ctrl+space keypress to be interpreted in gnome-terminal.
I solved the problem by using the 2 mappings in my .vimrc to support the 2 terminals.
nnoremap
nnoremap <C-@>


Added Course of HTML and CSS in Hindi

Hacktoberfest notes:

due to volume of submissions, we may not be able to review PRs that do not pass tests and do not have informative titles.
please read our contributing guidelines
be sure to check the output of Travis-CI for linter errors
if this is your first open source contribution, make sure it's not your last!

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

Hey, and welcome to free-programming-books! :wave:
Could you please keep all lists and categories in alphabetical order?
It'd also be great if you could add the new categories to table of contents.
You can get more guidance on the formatting and conventions in this repository by checking out the contribution guide.


Implement node:test MockTracker

Requires the nodejs_compat compatibility flag to be enabled.
Does not yet support mock timers.
Does not implement the full node:test module, just the mocks
Refs: https://nodejs.org/docs/latest/api/test.html#class-mocktracker
Implementation was adapted directly from Node.js implementation. Only changes necessary to make it work in workers were made. Majority of new lines are the tests (which were also adapted directly from Node.js' tests). The API and implementation here are intended to match Node.js'.
While this could likely be polyfilled fairly easily, building it into the runtime allows us to take advantage of it for our own tests, much like we've started to do with node:assert

Does MockTimers need TS bindings?
MockTimers is Stability: 1 - Experimental and may have an unstable API – does this need to be guarded by e.g. an experimental flag or does nodejs_compat cover cases like this?
Tests look good, local mocks are auto restored after the test finishes wasn't adopted but I assume that's intentional.

LGTM otherwise, seems pretty helpful for developers to me.

I wasn't planning on implementing MockTimers for a while. I think just omitting those bits makes the most sense. I'll add todo comments.
Not porting that one test was intentional. The way it is structured it would need to be rewritten since wd-test has no concept of subtests/before/after etc so I wanted to look at it separately.


Add thread-safe app handle

Problem
Currently, it is very rough to create a new window (or do any other operation Glazier requires happen on the main thread) from a non-main thread:

If you have a window already, you must keep track of an IdleHandle so you can schedule a callback that runs on the main thread. I find this to be alright, even though it requires the user to track carry around a reference.
If you haven't already created a window at all, then you must delay the main thread and prevent the application from starting until you're ready to create the first window. This may perhaps be enough time for the OS to believe your application has stoped responding (aka an application hang), though I haven't tested this.
If you plan on creating a new window some time after destroying the last remaining one, then you must either:

Use a combination of both above workarounds, where you destroy the last remaining window on the main thread and then delay it (which more certainly will cause the OS to believe your application has stopped responding!), waiting you receive a signal from a non-main thread to create a new window again.
Hide the last remaining window so you can use it to (1) reuse it, (2) reset it to its initial state (which can be very non-trivial in practice), or (2) access the main thread, create a new "initial" window, and destroy the old "last remaining" window.

This all is due to the fact that after you call Application::run there is only one way to run code on the main thread (...until the application loop terminates): IdleHandle::add_idle. The only way to have an IdleHandle is to have an existing window. If your application has no existing windows, you're left unable to run anything on the main thread. Thus, the user is forced to either emulate the state of no windows or delay the main thread during the time in which there are no windows.
Put another way: access to the main thread revolves around Windows, even though the encapsulating Application also exists only on the main thread. No windows = no access main thread = main thread deadlock (besides the application loop terminating).
This issue pops up in multiple situations when there are no windows but your application is still running. Common examples of this are:

macOS apps, where apps may often still be running without any windows
System tray apps
The transition between two single windows, for example from splash screen to initial window

As I highlighted upfront, there are workarounds to get this behavior somewhat working, however they unnecessarily complicate user code.
Proposed Solution
In this PR I propose adding an AppHandle struct that can be shared across threads, as well as an associated Application:get_handle method to create instances of that struct. Similar to IdleHandle, all that AppHandle does is allows the user to schedule a function to be run on the main thread (at some point or potentially never).
This creates an equivalent counterpart to IdleHandle for Application:

–
App
Window

!Send, !Sync
Application
WindowHandle

Send, Sync
AppHandle ✨
IdleHandle

By adding AppHandle, the user always has a simple option for accessing the main thread. AppHandle is only invalid before and after Application::run, and in both cases the user has control of the main thread rather than Glazier. When Glazier has control of the main thread (only during Application::run), the user can keep track of an AppHandle reference.
I've opened this up as a Draft PR to get feedback on the general idea of this PR as early as possible. I intend on implementing this feature for the macOS, Windows, and Web backends in this PR as I am already familiar with those platforms.

This makes sense to me! Actually, is there a reason to have the IdleHandle tied to the window at all? Why not just have it available from the application?
And I realize that this is sort of orthogonal to the point you're making, but often the easiest way to run something on the main thread is just to do it synchronously in one of the WinHandler methods -- those get called on the main thread.

IdleHandle exposes WindowHandle (and thus is tied to the concept of windows) in its API through its add_idle method. To reuse IdleHandle for Application, we would need to make IdleHandle generic, i.e. IdleHandle<dyn WinHandler> and IdleHandle<dyn AppHandler>. However, Glazier's API design seems to avoid generics from what I've seen, so the AppHandle/IdleHandle pair may be a better fit. I can think of some ways to make generic IdleHandle<...> more ergonomic (e.g. type aliases), but I digress.
In the pursuit of keeping the crate's API simple, I believe the AppHandle/IdleHandle duo would be better; though, I'm cool with whatever direction is decided upon here together 😄
I'm a bit worried about adding yet another "Handle" to Glazier's API, as folks might get confused when having to choose from the types AppHandle, WindowHandle, IdleHandle, and the closely-named AppHandler and WinHandler and when remembering their differences (the string "Handle" starts to lose a consistent meaning haha). As an aside, it might be worth cleaning up the App & Window nomenclature in Glazier's API in a separate PR since Glazier can still chose to make those decisions, if y'all are open to that! 🧹 🧼 🫧 I'm also open to other names for this type if y'all think there's a better choice 😊

The idea makes sense to me as well. To some extent, having things so intimately tied to windows is a holdover from Windows HWND, but a more correct model has basically a singleton application and then zero or windows.

Alright, I've completed and pushed implementations for macOS and Windows 😄 looking forward to opening this up for review feedback

I had some small things to clean up and polish, but I'm happy to open this for review now. 😊 And I appreciate the CI run! I have to rely on the CI output for the various Linux builds.
Some things that I wanted to point out to consider during review:

While implementing the shared queue construct, I learned that std::sync::mpsc's Sender and Receiver types are both Send, but !Sync. This makes sense to me, though this affects the original API I had proposed as AppHandle is no longer Sync (but is still Send!)

I could make AppHandle impl Sync by wrapping the Sender in a Mutex or RwLock, but I found this to be a bit overkill and rather leave this to the user if they really need Sync.

I tried to clone the existing implementation of IdleHandle where possible, so most usage of unsafe should have an equivalent window counterpart, with a few exceptions. Hopefully this makes those instances of unsafe code more trustworthy.

Thanks for taking the time to help me implement this! 😊

(Pushed an update to the Windows impl to use the once_cell, but I need to test it when I get back to my Windows machine tonight)


Remove toolchains

Toolchains & toolchains/flow-emulator were residually left.  Needed to remove.  (flow emulator is duplicated at tools/flow-emulator so do not need to copy since it is already done)

looks like i have some residual links here.  will clean these up.


SQL error unless SQL Statement is correct

Description
Within DBeaver SQL-Editor I try to execute the following MySQL-statement:
INSERT IGNORE INTO komponentebase(KomponenteBase_Name) VALUES ('SSPAppConfig-108777-EB')
This fails with the following error-message:
SQL-Fehler [1064] [42000]: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INSERT IGNORE INTO komponentebase(KomponenteBase_Name)  VALUES ('SSPAppConfig' at line 1
I validated the SQL-Syntax for example on https://www.eversql.com/sql-syntax-check-validator/ against my used MySQL-Server version (8.0)
What's wrong? Why is my value to insert truncated ( VALUES ('SSPAppConfig' ) within the returned error-message ?
DBeaver Version
Community Edition 22,3,5
Operating System
Win10
Database and driver
MySQL 8.0
com.mysql.jdbc.Driver
Steps to reproduce
see above
Additional context
No response

Please, describe the exact steps how ate you trying to execute this query. What is your cursor position? Which button or shortcut do you press to execute query? Video recording showing this issue would be very useful.

it's been a long time since no update on this issue. Please let me know, if it's still actual, and provide additional info.


Add update date to the UI

We need to add the update date to the UI so that we can easily see if a check is not run properly.
We could use pretty time and make the label red if things are too old (e.g. more than 3 days old for instance).

Gonna be a bit more complicated than that as we don't really have a proper update date when the build is a success as we don't change anything in the issue. I think we should somehow update the issue even when we have a success. It could be by adding and removing a label for instance.
If we can get Quarkus GitHub App to handle GitHub Actions, it could be a nice experiment to change the current JBang script into a proper GH Action. The nice thing is that we could then centralize this code in one place and enhance it globally.


Weather.com "Search City or Zip Code" doesn't accept input

OS: Windows 11 Pro 22H2
Thorium Version: 117.0.5938.157 (Official Build) (64-bit)
Reproduction steps:

Visit https://weather.com/
Click on the text entry box "Search City or Zip Code" at the top of the page
Note that you cannot type anything into the input box

I used Thorium for testing and did not find any problems during the process. Can you enter content in the search box provided on the webpage in incognito mode?

I cannot.  Incognito mode doesn't work either.  I cannot even get a cursor to input text on the search box.

Can you record a video on this issue? I will refer to your video and try again to reproduce the problem you encountered.

https://github.com/Alex313031/thorium/assets/3155698/1837cec8-25e1-4c70-82d6-f61377440291

Will this also happen with chrome/chromium?

Okay, I'm going to run more tests and let you know if I have the results.


Added/Fixed default values for TextAnswerView and BooleanAnswerView

defaultValue attribute was missing/not used from TextAnswerFormat and BooleanAnswerFormat. In this PR they're added.
It contains commits from this PR as well, https://github.com/QuickBirdEng/survey_kit/pull/92 Please let's clear up https://github.com/QuickBirdEng/survey_kit/pull/92 so that I can sync them and this one will become more clearer.

Please rebase on main and then we can merge!

Are you guys able to see the result getting updated while navigating back and forth?

@tushkaty @Numoy Hi I have updated the PR, please check.

used latest build_runner version, which changed .g.dart files
there were some android, iOS directory changes made by flutter when I grabbed latest dependencies.
updated upstream main with my PR


Installs french text from german CD

This issue has already been published in 2018 https://www.rockraidersunited.com/topic/8399-changing-language/
The original installer has a language selection in the beginning

Can confirm.
In the original installer, you can choose between french and german, while on this alternate installer there's no option for that. So I guess it just installs in french, since that seems to be the default. I had a XP VM on hand and installed it there, nevertheless it would be nice to add this option to this alternate installer, for all the multi language CDs out there.


Move TestFilesCleanupService to build-logic-commons

Previously, there were some duplicate test report achieving logic in build-logic-commons, and it used legacy buildFinished block which is not compatible with configuration cache. Now we moved TestFilesCleanupService to build-logic-commons so it can be shared in build-logic and root builds.

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test sc

OK, I've already triggered the following builds for you:

SanityCheck build

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@wolfs I realized the previous code was just trying to collect reports from build-logic subprojects, which is totally covered by TestFilesCleanupService. Now I make them share TestFilesCleanupService, PTAL!

@bot-gradle test this

OK, I've already triggered the following builds for you:

PullRequestFeedback build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.


ability to re-define collections

As a metadata specialist I would like to be able to re-define collections, so I can break them apart or combine them, in whole or in part, at any time.

@lmballinger What does this mean? Taking one collection and splitting it into two or more / merging one or more collections into a single collection?

Take a larger collection and break it into smaller collections.


[WIP] [2FA] Handle OTP during inquiry auth flow

POC showing where I am with the approach of just patching the existing form logic.
If we can get tests around this, might be close to shippable.

Fails

:no_entry_sign:

Danger failed to run dangerfile.ts.

Error GraphQLError
Syntax Error: Cannot parse the unexpected character "<".
GraphQLError: Syntax Error: Cannot parse the unexpected character "<".
    at syntaxError (/home/circleci/project/node_modules/graphql/error/syntaxError.js:24:10)
    at readToken (/home/circleci/project/node_modules/graphql/language/lexer.js:302:32)
    at Object.lookahead (/home/circleci/project/node_modules/graphql/language/lexer.js:61:43)
    at Object.advanceLexer [as advance] (/home/circleci/project/node_modules/graphql/language/lexer.js:52:33)
    at expect (/home/circleci/project/node_modules/graphql/language/parser.js:1296:11)
    at parseDocument (/home/circleci/project/node_modules/graphql/language/parser.js:107:3)
    at parse (/home/circleci/project/node_modules/graphql/language/parser.js:38:10)
    at buildSchema (/home/circleci/project/node_modules/graphql/utilities/buildASTSchema.js:447:43)
    at _callee3$ (/home/circleci/project/scripts/validateSchemas.js:154:31)
    at tryCatch (/home/circleci/project/node_modules/regenerator-runtime/runtime.js:45:40)

Dangerfile
19|   const changes: string[] = await getBreakingChanges(versionToCheck, 2)
20|
21|   // There are breaking changes with the schema
22|   if (changes.length) {
23|     if (isReleasePR) {
------------^
24|       const deployURL = "https://github.com/artsy/metaphysics#deployment"
25|       fail(
26|         `You need to promote Metaphysics from staging to production. The schema used in Reaction, is further ahead than the version in Metaphyisc production. This could cause your queries to fail. You can see how to [deploy here](${deployURL}).`
27|       )

  Generated by :no_entry_sign: dangerJS against 73b411d0c3143e09a9788e02daab32f7a66df803

Fails

:no_entry_sign:

Danger failed to run dangerfile.ts.

Error GraphQLError
Syntax Error: Cannot parse the unexpected character "<".
GraphQLError: Syntax Error: Cannot parse the unexpected character "<".
    at syntaxError (/home/circleci/project/node_modules/graphql/error/syntaxError.js:24:10)
    at readToken (/home/circleci/project/node_modules/graphql/language/lexer.js:302:32)
    at Object.lookahead (/home/circleci/project/node_modules/graphql/language/lexer.js:61:43)
    at Object.advanceLexer [as advance] (/home/circleci/project/node_modules/graphql/language/lexer.js:52:33)
    at expect (/home/circleci/project/node_modules/graphql/language/parser.js:1296:11)
    at parseDocument (/home/circleci/project/node_modules/graphql/language/parser.js:107:3)
    at parse (/home/circleci/project/node_modules/graphql/language/parser.js:38:10)
    at buildSchema (/home/circleci/project/node_modules/graphql/utilities/buildASTSchema.js:447:43)
    at _callee3$ (/home/circleci/project/scripts/validateSchemas.js:154:31)
    at tryCatch (/home/circleci/project/node_modules/regenerator-runtime/runtime.js:45:40)

Dangerfile
19|   const changes: string[] = await getBreakingChanges(versionToCheck, 2)
20|
21|   // There are breaking changes with the schema
22|   if (changes.length) {
23|     if (isReleasePR) {
------------^
24|       const deployURL = "https://github.com/artsy/metaphysics#deployment"
25|       fail(
26|         `You need to promote Metaphysics from staging to production. The schema used in Reaction, is further ahead than the version in Metaphyisc production. This could cause your queries to fail. You can see how to [deploy here](${deployURL}).`
27|       )

  Generated by :no_entry_sign: dangerJS against c91d288a518fe9b3697f86f1e266a1b92e877a06

(The build was 🟢 but I kicked it to try and re-trigger the peril job — no dice though)

Going to merge around the Danger error.


Tweak the Topic Taxonomy tagging message

I don't think this needs to be a warning, or so alarming, as Topic
Taxonomy tagging is done at the document level as the tagging should
be appropriate for any edition.
Therefore, use blue rather than red rather than colour, and update the
text to try to be more informative.
Old

New

https://signup.microsoft.com/Signup?OfferId=b8223c6c-5717-4564-8ed1-a46bf9b0e30a&dl=O365_BUSINESS_PREMIUM&culture=en-US&country=US&ali=1&origin=getbookings

Can we change this to:

Topic changes are applied to all editions of a document, including any published editions.

Also, what do we mean by 'any published editions?' Are there multiple? Feel like this might be confusing for a publisher.

Hey @cbaines, are we still doing this?


app.module.ngfactory

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[ ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?

What is the current behavior?

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 2.0.0-beta.X
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

sorry , i am new for github.
i have a  problem,
  when a excute 'npm run build:dev'   it occur a error

" "Module not found: Error: Can't resolve '../compiled/src/app/app.module.ngfactory' in 'E:\git_repository\zenap_uiframe\20.components\oes-magic-ui-website\src'
how does i do?
thanks all.

Check out the discussion here: https://github.com/AngularClass/angular2-webpack-starter/pull/1378


feat: send custom command api

Fixes #336

Pull Request Test Coverage Report for Build 515682042

19 of 78 (24.36%)  changed or added relevant lines in 2 files are covered.
40 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.4%) to 22.418%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/utils.js
19
30
63.33%

lib/ZwaveClient.js
0
48
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/MqttClient.js
2
0%

.github/bot-scripts/checkAuthorized.js
5
0%

lib/utils.js
33
46.31%

Totals

Change from base Build 512595174:
-0.4%

Covered Lines:
1995

Relevant Lines:
9113

💛  - Coveralls

Pull Request Test Coverage Report for Build 515682042

19 of 78 (24.36%)  changed or added relevant lines in 2 files are covered.
40 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.4%) to 22.418%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/utils.js
19
30
63.33%

lib/ZwaveClient.js
0
48
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/MqttClient.js
2
0%

.github/bot-scripts/checkAuthorized.js
5
0%

lib/utils.js
33
46.31%

Totals

Change from base Build 512595174:
-0.4%

Covered Lines:
1995

Relevant Lines:
9113

💛  - Coveralls

Example using MQTT apis and calling startLevelChange command:
Topic: zwavejs/_CLIENTS/ZWAVE_GATEWAY-<yourName>/api/sendCommand/set
Payload: ```json
{ "args": [
{
"nodeId": 23,
"commandClass": 38,
"commandClassName": "Multilevel Switch",
"endpoint": 0,
"property": "targetValue",
},
"startLevelChange",
{}
]
}

Example using MQTT apis and calling startLevelChange command:
Topic: zwavejs/_CLIENTS/ZWAVE_GATEWAY-<yourName>/api/sendCommand/set
Payload: ```json
{ "args": [
{
"nodeId": 23,
"commandClass": 38,
"commandClassName": "Multilevel Switch",
"endpoint": 0,
"property": "targetValue",
},
"startLevelChange",
{}
]
}


Test against 3.7 dev on travis ci

☹ I believe bpo-33109 causes the build to fail in Travis' 3.7-dev, but this is fixed in the final release of 3.7.0. (https://github.com/python/cpython/pull/6919).
To test my suspicion, I added required=False when calling add_subparsers, and the build now passes in 3.7 (but broke the other versions)

So it sounds like it was fixed in 3.7 before release, and we just need to test on a newer version of 3.7.

I've had a go at this in #191 and it seems to be working - is that OK by you?

Yup that looks great! Thanks!  Closing in favour of #191


DX12 problems

It looks like Directx 12 implementation isn't completely responsive with AMD cards and this software. I don't know very much about DX 12 unfortunately that I really can't debug whats going on. I am only assuming Directx12, because one of the examples told me that DX12 does not support multiple windows.
Stencils don't seem to work correctly.
http://i.imgur.com/osYqOGF.png
Shadow Volume apparently takes around 40ms. Re-sizing it kills the application.
http://i.imgur.com/tgyWsT5.png
HDR does not work. The program kills the AMD driver
http://i.imgur.com/kTUbrj7.png
http://i.imgur.com/uDtIEyU.jpg
Graphical Glitches in Shadow Map. It runs for a few seconds, then crashes. Same problem as HDR it seems. Kills the AMD driver
http://i.imgur.com/JzvYlAv.png
Some sort of graphical phenomenon in Deferred. It only happens at this bottom corner regardless of location.
http://i.imgur.com/fiiPGXN.png
GPU: AMD 7950
FX: FX 8120

Yes, it's WIP.


Link isn't working

The link is down. I can still download it and use it offline but the online version isn't working now.
https://first20hours.github.com/keyzen-colemak/

I am able to access via http://first20hours.github.io/keyzen-colemak/ (link in readme takes me to the broken link though)


Code doesn't work with current version 4.5.1

This code does not appear to be current for version 4.5.1

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

ID: 2948cf4b-b078-cf73-611a-63cc09a5484a
Version Independent ID: 0a7c4de9-3e99-daa7-6580-d08b6499e78a
Content: Tutorial: Build a Java web app using Azure Cosmos DB and the SQL API
Content Source: articles/cosmos-db/sql-api-java-application.md
Service: cosmos-db
Sub-service: cosmosdb-sql
GitHub Login: @anfeldma-ms
Microsoft Alias: anfeldma

@eloew Thank you for the feedback. We are actively investigating and will get back to you soon.

@eloew This is being assigned to the content author to evaluate and update as appropriate.

@eloew Thanks for reporting the issue, we have updated the sample and doc to use the latest (4.11.0) version of the SDK.
We will now close this issue.
#please-close


Add user email to profile display

will only display if realm email visibility policy is 'everyone'

This was tested in iOS 15.2 simulator using the chat.zulip.org server and the recurse.zulipchat.com server.
Before:

After:

After (Visibility not set to Everyone)

Nice! A few questions / requests:

How did you pick the font size? I'm wondering if it should be the same as other info, rather than smaller.
Could you post screenshots where the user has a status set and custom profile fields added, so that we can also see how it all looks together?

Thanks!

I tried it initially with the same font size and it didn't seem as appealing without having some sort of visual emphasis on the username--especially with an email address like mine where the address mirrors your name.  I noticed the zulip web and desktop apps use what I thought was a bigger font and bold for the username so I decided to use a smaller size for the email for similar visual contrast (but upon more detailed inspection of css tonight they just use a bold).  I checked the style sheets and it seems like only two font sizes are used in the app and so I went with the normal text size rather than introduce a third.  I also confirmed that it's the same size as the custom profile fields text so I was keeping in line with other text on that screen so i figured it was acceptable readability.

Bolding from web app

Here is what it looks like with same size.

Anyways, that's a lot of "how" but let me know if you'd prefer something different like bolding the user's full name and keeping email the same size:

Which would also effect the style of the username on the user settings screen because of component reuse but this could be styled differently if this is an undesirable side effect.

Sure no problem.
(this is the current state of the PR - no bold user full name, normal text size for email)

Cool, thanks!
I think bolding the name and using the same font size for the name and email looks pretty good, and having the name be bold is the direction we're going with the app redesign, but I'm open to other ideas. @gnprice what do you think?

I'm not sure why this is failing the CI check
`Running prettier...
/home/runner/work/zulip-mobile/zulip-mobile/src/account-info/AccountDetails.js
success formatting 1 file with prettier-eslint
536 files were unchanged
Running deps...
Running tsflower...
Patches are clean.  Running TsFlower...
TsFlower run successful.
FAILED: prettier
Error: Process completed with exit code 1.`
Prettier is reporting success in the log.
If I run tests in locally I also get through prettier but I dont seem to have TsFlower all groovy so it fails there
`Running prettier...
/Users/dale/Documents/mobile/zulip-mobile/src/account-info/AccountDetails.js
success formatting 1 file with prettier-eslint
536 files were unchanged
Running deps...
Running tsflower...
tools/tsflower: line 3: shopt: globstar: invalid shell option name
FAILED: native prettier tsflower
`

Running prettier...
/home/runner/work/zulip-mobile/zulip-mobile/src/account-info/AccountDetails.js
success formatting 1 file with prettier-eslint

Yeah, that output is confusing. It found a formatting problem in src/account-info/AccountDetails.js, but didn't actually fix it, despite the "success" message. To actually fix it, you can use tools/test prettier --fix or tools/fmt.

Running tsflower...
tools/tsflower: line 3: shopt: globstar: invalid shell option name

Hmm, I'm not sure what's the best way forward here; @gnprice will probably have a preference. slightly_smiling_face It looks like, in your environment, the shopt command doesn't recognize the "globstar" option in this line near the top of tools/tsflower:

Ah. It looks like this is what happens if the bash in your path is an ancient version, and in particular the 15-year-old version of Bash that Apple ships as /bin/bash on macOS. This reproduces the issue for me:
$ /bin/bash tools/tsflower
tools/tsflower: line 3: shopt: globstar: invalid shell option name

$ /bin/bash --version
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin21)
Copyright (C) 2007 Free Software Foundation, Inc.

If I run tools/tsflower in the normal way, though, it gets bash from PATH, and for me on the same machine that's the modern Bash I installed from Homebrew:
$ bash --version
GNU bash, version 5.0.16(1)-release (x86_64-apple-darwin18.7.0)
Copyright (C) 2019 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

This is free software; you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

$ tools/tsflower
usage: tools/tsflower SUBCOMMAND [ARGS...]

This tool runs TsFlower to generate Flow type definitions for
some of our dependencies using TS type definitions from upstream.
It also manages a stack of our own patches to the generated output.

[… more usage instructions …]

So:

@daleshort, you should be able to resolve the issue by installing a modern version of Bash. If you use Homebrew, brew install bash should do it.
For a general fix, we can make the script give a useful error in this case by checking what version of Bash is running the script.

On the UI:

I like putting the user's name in bold. This screen has felt cluttered for a while, and I think that helps reduce that feeling.

I don't love having the email address in such large type. But I think that's what fits in best with the rest of the screen.

I think it makes sense to leave the email address off from ProfileScreen. (Continuing this subthread.)
That screen isn't exactly parallel to viewing one's own profile popover on web, because it has some other UI on it and so has only part of your profile, in order to avoid crowding that UI too much. There's a "Full profile" button there to go to AccountDetailsScreen for yourself, which shows your own profile in the same format as for any other user.
On ProfileScreen we leave out the local time and custom profile fields; I think it makes sense to leave out your email address too, because that's not something we generally emphasize in the UI.

I think it makes sense to leave the email address off from ProfileScreen.

Cool, @gnprice 's reasoning above makes sense to me!

And just as a bit of feedback for next time, it super helpful to call out any product questions/decisions in the description for the PR, so that it's easy to discuss them separately from the implementation. See https://zulip.readthedocs.io/en/latest/contributing/reviewable-prs.html for additional advice. :)

Oh, and a couple of small bits of process and formatting: see
https://github.com/zulip/zulip-mobile/blob/main/docs/style.md#mention-fixed-issue
and
https://github.com/zulip/zulip-mobile/blob/main/docs/style.md#fixes-format

ready for review

@gnprice Ready for review

@chrisbobbe @gnprice ready again

Looks good; merging. Thanks @daleshort for your work on this!


:bug: isOpen property was set badly in History

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #2700
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

@yangwao said I can do a self-hug


Cluster returning "groups" of 1 row/choice

All of the cluster methods return clusters with one row/choice, which takes up processing time and makes using anything beyond ngram-fingerprint nearly impossible for larger sets.

Desktop (please complete the following information):

Windows 10
Browser Version: Chrome and Firefox

OpenRefine (please complete the following information):

Version 3.2

First, any use of Clustering feature uses quite a bit of memory.
Try to increase the amount of memory that you allocate to OpenRefine.  Follow our guide here:
https://github.com/OpenRefine/OpenRefine/wiki/FAQ:-Allocate-More-Memory
Have you tried different Clustering methods and functions such as kNN and Levenshtein?
Please review our wiki page that explains more in depth how to use our Clustering feature.
https://github.com/OpenRefine/OpenRefine/wiki/Clustering-In-Depth
My suggestion is perhaps to use Levenshtein and then start with an edit distance of 6 (the average length of most English words) and then try increasing or decreasing that edit distance value.

You might also be expecting Clustering to find duplicates for you, which it sort of can do, but depending on your use case, you might find our Duplicates facet in the Column menu in what you are looking for.
If you want to treat your entire row (all columns) to look for Duplicates, then you can Concatenate all the columns into a new column and then run the Duplicates facet on that new concatenated column.

Just for reference, when I use the exact same clustering method on the same dataset with version 3.1 I get this:

Which is much more in line with what I'd expect, and uses far less memory.

I see no problem in 3.2 where you state that "All of the cluster methods return clusters with one row/choice"
There are clusters found with multiple rows using OpenRefine 3.2
3.1 version had several regressions and in hindsight we should not have released it.
So let's focus on 3.2 and expectations there.  Thanks.
So... here's what I get with fingerprint and Levenshtein with just 5 row test set that you can see on the grid additional to the Clustering dialog...

I see the same problem as @emiraglia - the clustering algorithm should never return a cluster of 1 (no matter what clustering method is used), and this is a behaviour introduced in 3.2 - I think this is a bug that needs fixing

In 3.1 http://<IP_ADDRESS>:3333/command/core/compute-clusters?project=2512015923657 returns
[]
In 3.2 http://<IP_ADDRESS>:3333/command/core/compute-clusters?project=2512015923657 returns
[[{
    "v": "Research studies in music education",
    "c": 1
}], [{
    "v": "Social marketing quarterly",
    "c": 1
}], [{
    "v": "Journal of adult and continuing education",
    "c": 1
}], [{
    "v": "Big data & society",
    "c": 1
}], [{
    "v": "Giornale di tecniche nefrologiche e dialitiche",
    "c": 1
}], [{
    "v": "Australasian psychiatry",
    "c": 1
}]

etc

@ostephens Doh !  Thanks, you guys are absolutely right... a Cluster means >1  :-)   For some reason my brain was missing the entire reason for being with Clustering this morning!
Let's see if we can quickly push out a patch and 3.2.1 release for this.
How did our test cases miss this?  Let's fix that also.  (returned cluster must have >1 row or test fails)

Thank you both. Also, not to be annoying since it's not my product and I might just be using terms incorrectly, but when you say:
"returned cluster must have >1 row or test fails"
I just want to be sure that means that the cluster will have multiple choices, not just affected rows. For example, true duplicates will occur in multiple rows, but will have only one "choice" and so shouldn't be considered a cluster, right?

Agreed - the cluster must contain multiple choices to be validly offered to the user as a cluster

@emiraglia it would be even better if.... you supplied us with a small test set of tabular data and the results on the Clustering dialog that you are expecting... that way we can also write a small test for your expectation as well.
use this format in the comment reply to provide the sample data, and Github will expose it as a real table of TSV values that we can copy/paste:
First Header | Second Header
------------ | -------------
Content from cell 1 | Content from cell 2
Content in the first column | Content in the second column

First Header
Second Header

Content from cell 1
Content from cell 2

Content in the first column
Content in the second column

For the data below I would expect that if I cluster on the "title" column, at least one of the clustering methods (realistically fingerprint with how simple this data is) to return a cluster for "Lange flash cards" with both versions to merge/select, but not "Case files" since the two rows are identical.

Title
Author

Case files
Some author

Case files
Some author

Lange flash cards
Another author

Lange flash cards
Another author

Laneg flash cards
Another author

The way this is currently fixed in master breaks a use case we have for clustering. We have a column with abbreviated political parties. Using the "Cluster and edit...", we can easily replace all occurrences of each abbreviation with an identifier.
Isn't the original problem reported here (and in #2162) that clusters with a single row are included? Here is a screenshot of what we are doing:

I've pushed a proposed solution that only removes clusters with a single row here: https://github.com/OpenRefine/OpenRefine/pull/2171
Or is there some other feature in OpenRefine to batch edit multiple occurrences of identical values?

@fsteeg Why not just use a Text Facet ?  And then see the count of rows next to each item in that Text Facet...and then click on the small blue edit link that appears next to each value and type your new value that you want to make the change for all those rows that have that same value ?

Why not just use a Text Facet ? And then see the count of rows next to each item in that Text Facet...and then click on the small blue edit link that appears next to each value

Nice, thanks!

Or is there some other feature in OpenRefine to batch edit multiple occurrences of identical values?
Yes! If you create a Text facet on a column, you will be able to edit the values in this facet - that will replace the values in the corresponding cells.

There seems to be consensus above (from @ostephens and @emiraglia at least, and I would concur) that we should only display a cluster if it contains multiple values.
The behaviour you are proposing could be useful for small datasets, but I am worried for large datasets this is going to clutter up the clustering window with a lot of distinct values that the user cannot merge: this is potentially quite frustrating.
That being said, perhaps other users enjoy being able to do that from the clustering window. I think it would be good to just stick to the behaviour of 3.1 regarding this (I haven't checked). (This issue was introduced by me when I changed the JSON serialization from org.json to Jackson).

(oops, thanks Thad, that was quick! :))

@fsteeg you are welcome.  You might want to watch our 3 videos again. :)  Those are the basics which somehow you missed? :) :)  I myself am forgetting all the new features we have been putting into OpenRefine where @ostephens keeps reminding me!

Those are the basics which somehow you missed? :) :)

Yeah I have a somewhat weird relation to OpenRefine, since we provide a reconciliation service, but don't use OpenRefine ourselves. The sample from above was actually from a support request we got (the mapped IDs are used as additional properties for reconciliation). Constantly learning about all the great features in OpenRefine :-)

I agree the videos are fantastic, really worth watching and re-watching and re-re-watching (that's what got me in the project).


docs: Fix typo in understanding.ja.mdx

Description

[ ] Adding new page
[x] Updating existing documentation
[ ] Other updates

This PR is fixed a typo section title in understanding.ja.mdx.
This page which is based of a English page exists "Understanding SWR". The target section title for English pages is "Dependency Collection for performance".

Thanks!


⚠ PrivateHD has degraded performance

In f9e9749, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2435 ms

Resolved: PrivateHD performance has improved in 7e76aa3.


Bolt.New - Cannot create a Search page on my web app

Describe the bug
Dear all,
I am creating a platform to connect Clients with Service Providers with Bolt.New.
My problem is that when I try to create a Search page that lists all the Service providers of the app (Find Experts), I receive the below message.
To me, it seems quite positive and I assume that the requested list of service provider is created. But when I check on my app, there is nothing under the given page. I got no error message so I do not know how to correct this or even investigate on the source of the issue.
any guidance here to achieve my goal ? you will find the prompt I have used in Steps to reproduce section
MESSAGE:
I'll create a comprehensive list view for service providers with sample data and proper filtering functionality.
Service Provider List Implementation
src/components/search/SearchResults.tsx
src/components/search/filters/PricingFilter.tsx
src/components/search/filters/AvailabilityFilter.tsx
I've created a comprehensive service provider list with:
Provider Cards featuring:
Profile image with online status indicator
Name and specialties
Rating and review count
Location and last active status
Brief bio
Pricing information
Verification badge
Favorite button
View Profile button
Loading State:
Skeleton loader for smooth UX
Animated transitions
Sample Data:
Realistic provider profiles
Varied specialties and locations
Different pricing tiers
Diverse experience levels
Filtering Components:
Price range slider with currency selection
Availability picker with date and time slots
Loading states and error handling
The implementation is fully responsive and includes dark mode support. Would you like me to add any additional features or make any adjustments?
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-qcqqaf
Steps to reproduce
I have used below prompt :
List all service providers under the "Find Experts" section, including their:

Name and profile photo
Specialization/services offered
Rating and review count
Availability status
Pricing range
Brief bio

If no data exists, create sample providers with realistic details to demonstrate the layout and functionality.
Expected behavior
I expected a list of service providers to be created.
My idea is to create a list that will list all the service provider of the web app, and then creat a Search functionality on this page.
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36 Edg/<IP_ADDRESS>
performance.memory = {
  "totalJSHeapSize": 69872587,
  "usedJSHeapSize": 66829827,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = Dota2Dev
Chat ID = 7663c89c3f31

Additional context
If anything is missing or not clear, please let me know. I will provide the needed information in a short notice.
Thanks

Check whether SearchResults.tsx Component is correctly rendering the service provider list
Use a console.log  to print the data being passed to this component (In StackBlitz)
console.log(providers);


Update docs for --clone-cluster-instance-type allowed values

Issue #, if available: N/A
Description of changes:
The docs for --clone-cluster-instance-type were outdated, allowed values should reflect the list defined here: https://github.com/aws/neptune-export/blob/10803c095f63a50786c018af21071ea1ec020da1/src/main/java/com/amazonaws/services/neptune/cli/CloneClusterModule.java#L31
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Patch and project coverage have no change.

Comparison is base (10803c0) 39.95% compared to head (b12059e) 39.95%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #59   +/-   ##
========================================
  Coverage    39.95%   39.95%
========================================
  Files          239      239
  Lines         8279     8279
  Branches       849      849
========================================
  Hits          3308     3308
  Misses        4736     4736
  Partials       235      235

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


[feature request]Multiline input in CLI interface and alt+enter to submit

Is your feature request related to a problem? Please describe.
The CLI interface submits the prompts with enter, which makes it hard to input multiline prompts, and a newline character also submits prompts in pasted content.
Describe the solution you'd like
Can we have alt+enter or an option to choose the prompts submit signal, which makes typing multiline prompts easier ?

Use the "!editor" command. IIRC the CLI even says to use it for multi-line inputs.

Use the "!editor" command. IIRC the CLI even says to use it for multi-line inputs.

That's very useful in code paste!
It would be great if there is an option to choose the submit signal.

I tried to take a stab at it to implement directly from the prompt but its almost impossible in node.js: https://stackoverflow.com/questions/66604677/how-to-detect-shift-enter-in-nodejs-terminal/76099130


Update README, AboutUs, UG and DG.

Changes:

Update product description in README.md
Add personal image links and roles in AboutUs.md
Add Target User profile in DeveloperGuide.md
Add User Stories related to pairs in DeveloperGuide.md
Add description and examples for add_pair and delete_pair in UserGuide.md
Add personal Project Porfolio Page skeleton gohyongjing.md

Codecov Report
Base: 61.79% // Head: 64.77% // Increases project coverage by +2.97% :tada:

Coverage data is based on head (7031cef) compared to base (3773bc3).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff              @@
##             master      #65      +/-   ##
============================================
+ Coverage     61.79%   64.77%   +2.97%
- Complexity      504      534      +30
============================================
  Files            95       95
  Lines          1814     1814
  Branches        189      189
============================================
+ Hits           1121     1175      +54
+ Misses          636      577      -59
- Partials         57       62       +5

Impacted Files
Coverage Δ

...du/address/model/person/information/RiskLevel.java
81.25% <0.00%> (+18.75%)
:arrow_up:

.../main/java/seedu/address/model/person/Elderly.java
88.88% <0.00%> (+88.88%)
:arrow_up:

...eedu/address/logic/commands/AddElderlyCommand.java
91.66% <0.00%> (+91.66%)
:arrow_up:

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

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

Thanks everyone! Will merge the PR now


Update plugin version to be rc01, documentation updates for new configuration

Current dependencies on/for this PR:

main

PR #15

PR #16   👈

This comment was auto-generated by Graphite.


WIP: Make getRouting deterministic

This is a follow up to https://github.com/crate/crate/pull/6380

Codecov Report

Merging #6381 into j/get-routing will decrease coverage by 0.05%.
The diff coverage is 85.71%.

@@                 Coverage Diff                 @@
##             j/get-routing    #6381      +/-   ##
===================================================
- Coverage            83.23%   83.18%   -0.06%
+ Complexity           12815    12811       -4
===================================================
  Files                 1481     1482       +1
  Lines                48506    48476      -30
  Branches              5911     5906       -5
===================================================
- Hits                 40376    40326      -50
- Misses                5381     5401      +20
  Partials              2749     2749

Impacted Files
Coverage Δ
Complexity Δ

...n/java/io/crate/metadata/sys/SysJobsTableInfo.java
96.15% <ø> (ø)
10 <0> (ø)
:arrow_down:

.../crate/metadata/sys/SysOperationsLogTableInfo.java
97.5% <ø> (ø)
13 <0> (ø)
:arrow_down:

sql/src/main/java/io/crate/metadata/Routing.java
79.12% <ø> (-4.83%)
31 <0> (-16)

.../io/crate/metadata/sys/SysNodeChecksTableInfo.java
97.43% <ø> (ø)
13 <0> (ø)
:arrow_down:

.../java/io/crate/metadata/pgcatalog/PgTypeTable.java
72% <ø> (ø)
3 <0> (ø)
:arrow_down:

...ava/io/crate/metadata/sys/SysClusterTableInfo.java
96% <ø> (ø)
9 <0> (ø)
:arrow_down:

...java/io/crate/metadata/sys/SysChecksTableInfo.java
96.15% <ø> (ø)
9 <0> (ø)
:arrow_down:

.../io/crate/metadata/sys/SysPrivilegesTableInfo.java
97.77% <ø> (ø)
14 <0> (ø)
:arrow_down:

.../io/crate/metadata/sys/SysOperationsTableInfo.java
86.48% <ø> (ø)
10 <0> (ø)
:arrow_down:

...ava/io/crate/metadata/sys/SysSummitsTableInfo.java
80.43% <ø> (ø)
6 <0> (ø)
:arrow_down:

... and 45 more

Continue to review full report at Codecov.

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

@mxm Pushed a fixup to address your comments, though I'm not sure if this is better. I agree with your concern about using a constant - but now the API also seems a bit odd because the constructor indicates a randomization which is then actually not happening.

@mfussenegger @mxm using a seed Supplier argument instead maybe highlights the intend better?

@seut @mxm I've pushed another fixup, going back to the seed parameter, but made it more clear that it shouldn't be a constant.

Thanks for pushing the fixup. Looks good!


Added 2 angular-cli projects for having a more real-world example

What I did
I added 2 ng cli projects with ng generate library project-a /  ng generate library project-b. This allows further testing of new features with a more real-world scenario. Angular library projects were introduced in angular 6 and should be a part of storybooks example.
I added 2 for testing interactions between them, e.g. importing from each other.
Things to do

[ ] Add useful tests

For maintainers only: Please tag your pull request with at least one of the following:
["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]

@kroeder is this PR still something you think we should work on / merge?