Swap aria-*, role in Attribute order section

role should be defined first, followed by an aria-* property since a lot of ARIA roles expect certain properties to be used in conjunction with.
For example,

makes perfect sense, rather than to have the attributes the other way around.

I dig that. Want to open a PR? :)

Cheers, will do and closing this ticket.


EC5Util.convertPoint() ignores withCompression parameter

Introduced in version 1.53 by this change: https://github.com/bcgit/bc-java/commit/8b015046cd1a7232bce578a51d70353c4e47f73b#diff-e02d61345a013a87477324efd23eff27R166
I worked around this in my own code (by implementing convertPoint() myself), and confirmed that the underlying code still respects withCompression.

This has finally been addressed as part of a rework of point compression, however the convertPoint method itself, while fixed, is now also deprecated (a new method has been added without the boolean withCompression parameter).
Looking at the i2p commit mentioned above, the reflection usage is hiding other deprecations too. Both ECCurve.createPoint(BigInteger, BigInteger, boolean) and ECPoint.getEncoded() are deprecated.
After our next release, I would suggest you change to using EC5Util.convertPoint(ECParameterSpec, ECPoint) followed by ECPoint.getEncoded(boolean), passing your withCompression parameter in the getEncoded call.

Appears to be resolved.


Using data-umami-event causes a full page reload when clicking the same route repeatedly

Describe the Bug
When using data-umami-event on a  component in a Next.js application, clicking the same route multiple times causes a full page reload instead of maintaining client-side navigation. This behavior bypasses Next.js' routing mechanism, which should prevent hard reloads when navigating to the same route.
Steps to Reproduce

Add data-umami-event to a  component in a Next.js application

<Link href="/some-route" data-umami-event="button-click">Go to Route</Link>

Navigate to the route by clicking the link.
Click the same link again while on the same route.

Expected Behavior
Next.js should handle the navigation without triggering a hard reload, keeping the navigation smooth and client-side.
Actual Behavior
Clicking the link again causes a full page reload, even though it navigates to the same route, disrupting the expected smooth navigation flow in the application.
Environment

Next.js Version - 14.2.3
Umami Version - 2.11.3

Database
PostgreSQL
Which Umami version are you using? (if relevant)
2.11.3
Which browser are you using? (if relevant)
Chrome latest

Can you try putting the attribute on a wrapper element?
<div data-umami-event="button-click">
  <Link href="/">Go to route</Link>
</div>

Hi @mikecao,
I'm experiencing the same issue, and I must admit that I find the suggested workaround unsatisfactory as it often leads to significant code changes in SPAs.
After quickly reviewing the JavaScript part of the code, I understand the underlying reason for the problem. However, I believe it could be resolved more elegantly by utilizing navigator.sendBeacon() instead of preventing the default behavior and then manually updating location.href for non-external links. Do you happen to know who might be responsible for this part of the code and whom I could address this suggestion to? Thanks.

Hi @mikecao, i have the same issue on static old school html website with a ajax loading page on every pages.
I try your workaround but the effect is links are no more tracked.
Do you have any ideas ?

@laurent-d If you use the wrapper around a link, it doesn't track?


nixos/clickhouse: add package option

Motivation for this change
Consistent with other services and helps to work around #130101
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Once you've made the change do you know how to test it on your machine (server or laptop) ?
if you are using flakes, you can change the input.url of the nixpkgs flake.
if you are using channels, you can probably change the channel to point to this PR.
Let me know if anything doesn't make sense.
Thank you for your contribution!

Overlays weren't sufficient?

Overlay would probably be sufficient. I kinda dislike the idea that any usage of this module (and also plausible) requires user to define a global overlay. IMO .package simplifies the default workflow here.

A package option only makes sense if you can't use an overlay because this would imply tons of (unneeded) rebuilds (systemd or sudo are examples here) or if you need to specify a certain version for the module (e.g. postgres, nextcloud). None of that applies here, so I'd rather close this.

I'm a big fan of having a package option for most modules:
Often in production you need to use a different version/override of the package than in other parts of the system (e.g. as a dev dependency). Often you also want to run different machines with different package versions, e.g. for rolling upgrades of distributed systems. That gets messy and error-prone quickly with conditionally applied overlays.
For this reason, we do this with many service modules, none of which are like systemd or sudo:

services.postgresql.package
services.mysql.package
services.redis.package
services.consul.package

Clickhouse should be equally configurable.

package options are fine... but I notice that because so many package options exist many people don't understand what overlays are, or even know that they exist. Oh well :man_shrugging:

I've taken the liberty to force-push the addressing of feedback into this PR.

Personally this looks good to me.

I have built -A clickhouse.tests. It passed.


chore: realign release candidate

Checklist

[ ] Tests written for new code (and old code if feasible).
[ ] New or updated public/exported symbols have accurate TSDoc documentation.
[ ] Linter and other CI checks pass.
[ ] Sign-off given on the changes (see CONTRIBUTING.md).

We already have this here #91

interesting, how didn't I see that? :upside_down_face:


Distributions should use cmake to find protobuf and pin a version

Protobuf fusses if the version of the headers being used don't match the version that was used to create schema files. We should (a) use CMAKE find_package to find the system cmake and (b) verify/pin the version.

We use find_package(protobuf) here, but we don't have a FindProtobuf.cmake file, so this statement is probably meaningless. Maybe. I don't always understand cmake. Maybe this would work for us? http://www.cmake.org/cmake/help/v3.0/module/FindProtobuf.html


⚖ Register Validator: 2pilot

Network Choice
dentrite-1
Who are you?
Your trusted staking co-pilot
Delegator address
axone1w072tmg3ctdzxuk7egkd4rw4xtpgm85uj78437
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"2pilot","identity":"8E102EA8D78D6769","website":"https://2pilot.dev/","security_contact":"hello@2pilot.dev","details":"Your trusted staking co-pilot"},"commission":{"rate":"0.100000000000000000","max_rate":"0.200000000000000000","max_change_rate":"0.010000000000000000"},"min_self_delegation":"1","delegator_address":"","validator_address":"axonevaloper1w072tmg3ctdzxuk7egkd4rw4xtpgm85uztt8tu","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"uHl694kaNvZo6t9HEfCr/pq7npibCHcXQGiFpnrO5AA="},"value":{"denom":"uaxone","amount":"1000000000000"}}],"memo":"4569842347acc9204971c243315f1839e89b1cc7@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"AtY/nXwkp7kmlMN4WTPJa7N1mKRTu9fKBIwe0NZA3VZt"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["EqwlB1sFnPxMLqviyYdnrKMPXRdmC/EhQGkJ1Q8KP8EEwaJsV5q/XBA58Ff4t1ry3z6bX5PzbSxNBsChGcLW5A=="]}

😉 Here is the corresponding PR: https://github.com/axone-protocol/networks/pull/810


Uncaught ReferenceError: zxcvbn is not defined

zxcvbn = require('zxcvbn') returns undefined when the whole library is packed with webpack.

Ignore this one error. Real cause was
https://github.com/seethroughtrees/react-ux-password-field/issues/13#issuecomment-120793394


Handling responses: an empty array as result will make the response be "lost in flight"

I've been tracing a bug with RLS completions, and I think I found it.
When typing, for example, "use std:", it tries to complete after the ":" character. However, the RLS doesn't understand that, since it manages to complete only after the full "::" operator: "use std::".
This would be fine by itself, but RLS sends back a successful response with an empty result list, a list of 0 items. This, I think, is a valid response by the LSP protocol.
However, the response checking code in rpc.py fails to consider that as a valid response, because on the line 168, it tests result like this: if result:. In Python, an empty list coerces to false, which makes it to consider the value as an error. But since the error value isn't set either, the response goes unhandled.
This causes the CompletionState to get stuck as CANCELLING upon the next completion request, which prevents further messaging.
There's multiple bugs to be fixed:

Allow results that are empty lists (if result is not False: instead of if result:, possibly with a comment that explains the need for it to be that way.)

Without the response message, CompletionState seems to get stuck as CANCELLING, and prevents further requests. I have no clear picture how to fix this without breaking anything; I haven't studied the code in completion.py well enough. However, even if 1) is fixed, this is a separate bug, because it would be bad if it got stuck in case a response gets lost "in flight" for some reason.

While reading the code, I noticed that there is a setting log_payloads. This isn't mentioned in the documentation. Should it?

Btw. arguably not finding completion results with use std: can be also considered as a bug, but it's a bug/missing feature in RLS. I'll report that there.

Great research, sounds like hard-to-spot issue!

Feel free to send a PR, or I can check in what I have locally. I suggest the following additions:

Check if result is not None instead
Add a final else case that logs about an incomplete result payload (this would have made this bug much easier to spot.

I'd like to avoid the complexity of handling missing responses. Getting out of CANCELLING state might require some kind of timeout hack 🤢 - reopening the document should reset completion state without restarting the language server.

We can add log_payloads to the documentation - I've used it for in-depth debugging but it would be a good requirement to add

So the current state-machine-like code is to ensure that there is no multiple completion requests at a time to prevent juggling with concurrency? If I understand it right, CANCELLING defers any further requests until the response has been got – after that it launches the pending request.
Since each request has an ID, it shouldn't be too hard to track the state of each request separately, and "expire" them when new requests are made. One could store the ID of the currently "fresh" request (the newest request sent), and ignore all the responses from stale requests. I think this would eradicate the need for the explicit state handling.
However, this would allow for multiple concurrent requests. After a quick search here https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md , I couldn't find anything that forbids that, but I wonder if all the servers play well with that. Was the current state-machine-like code there for that reason?

I quickly applied the fixes mentioned earlier so we can move onto discussing the state-machine vs. expectations about responses.
You are correct, allowing only one completion request at a time was done to make completion state a bit easier to reason about. There is enough complexity managing Sublime's api, adding multiple concurrent requests into the picture would make the solution even less maintainable.
And yes, I think it is the "safe"/conservative thing to do towards potentially expensive requests to the server.

it would be bad if it got stuck in case a response gets lost "in flight" for some reason.

as per http://www.jsonrpc.org/specification:

When a rpc call is made, the Server MUST reply with a Response, except for in the case of Notifications.

Would it be ok if we revisited this when we have a real case where this affects end users ?

Yes, that sounds entirely reasonable. Good call about that MUST — in light of that, 2) isn't a bug.

Also, thanks for the quick fix for 1)!


Enable support for multi-line comments

Currently stto supports single line comments for the supported languages. Need to enable multi-line comments and do the adjustments needed.

I can take this @mainak55512 please assign this to me

I can take this @mainak55512 please assign this to me

Done

@mainak55512 created a pull request

Please close this issue as it has been added using #6


DomainAccumulator allows accumulating a row at once

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#480  👈
#479
#478
#463
#477
#476
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @spapinistarkware and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.71%. Comparing base (93e0bb7) to head (fc7d13f).

Additional details and impacted files
@@                                   Coverage Diff                                    @@
##           spapini/03-15-rename_evalaution.rs_to_accumulaion.rs     #480      +/-   ##
========================================================================================
+ Coverage                                                 95.67%   95.71%   +0.03%
========================================================================================
  Files                                                        60       60
  Lines                                                      8807     8818      +11
  Branches                                                   8807     8818      +11
========================================================================================
+ Hits                                                       8426     8440      +14
+ Misses                                                      330      327       -3
  Partials                                                     51       51

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

Merge activity

Mar 24, 9:16 AM EDT: @spapinistarkware started a stack merge that includes this pull request via Graphite.


SCSKafka + Actuator + enable-observation creates a 2nd AdminClientConfig which doesn't copy config across

Hi,
I have an issue when deploying to our test environment where SCSKafka with  enable-observation=true starts a second AdminClientConfig but doesn't copy over the configuration security.protocol and therefore I get SSL errors (strangely the bootstrap server is copied OK!!)
https://github.com/davidmelia/spring-cloud-function-zipkin/blob/observation_bug2/src/test/java/example/SendKafkaMessageTest.java is a super simple test and I illustrate the issue by using the property retry.backoff.ms which is not consistent:
@TestPropertySource(properties = {
....
    ,"spring.cloud.stream.kafka.binder.configuration.retry.backoff.ms=200"
....
})
public class SendKafkaMessageTest {

When you run SendKafkaMessageTest, In theory  you should see AdminClientConfig, ProducerConfig and the second AdminClientConfig boot up with retry.backoff.ms = 200 but in fact the second AdminClientConfig is the default retry.backoff.ms = 100 which is incorrect:

2023-03-16T19:57:24.791Z  INFO 31069 --- [           main] o.a.k.clients.admin.AdminClientConfig    : AdminClientConfig values:
....
	retry.backoff.ms = 200
....

2023-03-16T19:57:25.004Z  INFO 31069 --- [           main] o.a.k.clients.producer.ProducerConfig    : ProducerConfig values:
...
	retry.backoff.ms = 200
....

2023-03-16T19:57:25.052Z  INFO 31069 --- [           main] o.a.k.clients.admin.AdminClientConfig    : AdminClientConfig values:
....
	retry.backoff.ms = 100
....

^^ 100 is wrong
N.B I think this might be the root cause to https://github.com/spring-cloud/spring-cloud-stream/issues/2651  and also linked to my previous issue https://github.com/spring-cloud/spring-cloud-stream/issues/2616

I think it's something to do with this bit of code

Yes, it's related to this https://github.com/spring-cloud/spring-cloud-stream/issues/2576#issuecomment-1348666138
We fixed the KafkaTemplate (and listener container) to use the correct brokers, but the template doesn't know anything about the binder's configuration.
https://github.com/spring-projects/spring-kafka/issues/2505
It now propagates the correct servers, but it doesn't copy over the SSL configs (because it doesn't know about them).
The work around is to configure the SSL globally for all Kafka (via the spring.kafka namespace).
The properties are merged in order global -> binder -> binding.
We probably need to change spring-kafka to allow the binder to inject its own admin config.

encountering the same issue

Thanks @garyrussell  - adding the below solved the issue
spring:
  kafka:
    admin:
      security:
        protocol: SSL

The spring-kafka releases containing the fix should go out today so we will be able to change the binder to inject its own admin.

Hello there. I am currently using Spring Cloud Stream 4.0.3 snapshot + Reactor Kafka 1.3.17, and still observing this issue. Am I missing something, please?

Hello there. I am currently using Spring Cloud Stream 4.0.3 snapshot + Reactor Kafka 1.3.17, and still observing this issue. Am I missing something, please?

Am I missing something

It's not implemented yet; should have a PR today...


Tokens can be initialized with invalid mint accounts

Problem
InitializeAccount doesn't check if mint_info is valid (exists, initialized, and owned by token program) and so tokens can be created with invalid mints. These token accounts are useless because without a valid mint, they can never hold a token balance.
Suggested Changes

For developer experience, we should fail fast on token creation to avoid later confusion when token accounts are attempted to be minted to.
Consider forcing mint accounts rent exempt so that token accounts are always valid

Here is an example on-chain where this happened on testnet
InitializeAccount: https://explorer.solana.com/tx/5Ep4euatukeYjqFSwaPfrNT8FTfkgeYkp5boCEBPNygb9kACU11Nn7Kpzrjru3VxyFWSqMyv5jYfim6Wp9VVYceT?cluster=testnet
Invalid mint account: https://explorer.solana.com/address/3ZE2ei22iLgivkeLZkmNVcTx3zPfkN7ALLx5MdeEJ2ss?cluster=testnet

@jackcmay @CriesofCarrots @mvines thoughts?

These token accounts are useless because without a valid mint, they can never hold a token balance.

The account can still be closed though.  I feel like the client-side libraries could mostly mitigate this.   I'm not opposed to adding a mint validity check in InitializeAccount.

Consider forcing mint accounts rent exempt so that token accounts are always valid

This should be client side IMO.   I don't feel like we need to enforce that in the program.

@CriesofCarrots - anything left to do here?

Yes, still need the check for valid mint on InitializeAccount


Institution User Views Message Centre

**This Story May need to be updated based on the results of #451 and #245
As a (Institution User) I want (View my message Centre) So That (I can see any notifications that may require my attention)
Additional Context

The UX Story is #783
The Message Centre for Institutions is the same #861
At points in the student process there is a Trigger that will initiate a generic email prompting the user to log in to their SABC Account
The images presented in this story are from the Student Dashboard but the message center for the Institution user *should operate the same.

Acceptance Criteria

[ ] The Institution User logs in and there is a new message for them to view, there is an alert (in red) indicating that there is a notification waiting for them, and the # of notifications

[ ] When the Institution User Selects Notifications from the top navigation then a Messaged 'Drawer' is displayed.

[ ] The Institution User can see the 'Title' of the message, a brief truncated description of (? characters) and a symbol indicating that the message has not been read (orange with an exclamation mark).
[ ] Once the Institution user has selected this message the status of the message is changed and a symbol is shown to indicate that it has been 'seen' (green with a checkmark)
- [ ] The Institution User has the option to "Mark All as Seen" which turns all messages to the 'seen' status.

Question
Is it marked as seen for all users or just the one User?

Removed from scope.


[Cherry-pick][BugFix][Branch-2.2] Fix potential race condition between and compaction  (#6963)

Partial Update holds _index_lock to prevent the race of the primary index. However, _apply_compaction_commit may modify the primary index without holding the lock which will lead to a reading/writing competition of the primary index.
What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #6962
Problem Summary(Required) ：

run starrocks_fe_unittest


Scala codegen doesn't work with openapi petstore.yml

Using petstore.yml: https://raw.githubusercontent.com/OpenAPITools/openapi-generator/master/modules/openapi-generator/src/test/resources/3_0/petstore.yaml and https://github.com/guardrail-dev/guardrail-sample-sbt-http4s
  /user:
    post:
      tags:
        - user
      summary: Create user
      description: This can only be done by the logged in user.
      operationId: createUser
      responses:
        default:
          description: successful operation

works:
  /user:
    post:
      tags:
        - user
      summary: Create user
      description: This can only be done by the logged in user.
      operationId: createUser
      responses:
        '200':
          description: successful operation

[IJ]clean; compile
[success] Total time: 1 s, completed May 23, 2023, 4:15:32 PM
[warn] There may be incompatibilities among your library dependencies; run 'evicted' to see detailed eviction warnings.
Error:Unknown HTTP status code: default
[error] (Compile / guardrail) dev.guardrail.sbt.CodegenFailedException
[error] Total time: 0 s, completed May 23, 2023, 4:15:33 PM

Then that unlocked a new error:
[error] /Users/ericpeters/even/play/guardrail-sample-sbt-http4s/src/main/scala/Server.scala:59:105: value Accepted is not a member of object example.server.store.StoreResource.DeleteOrderResponse
[error]         val out = m.get(orderId).fold[StoreResource.DeleteOrderResponse](respond.NotFound)(_ => respond.Accepted)
[error]                                                                                ^
[error] /Users/ericpeters/even/play/guardrail-sample-sbt-http4s/target/scala-2.13/src_managed/main/example/server/pet/Routes.scala:175:119: not found: value utf8
[error]             for (additionalMetadata <- multipart.parts.find(_.name.contains("additionalMetadata")).map(_.body.through(utf8.decode).compile.foldMonoid).sequence; response <- handler.uploadFile(UploadFileResponse)(petId, additionalMetadata, multipart.parts.find(_.name.contains("file")).map(_.body)) flatMap ({
[error]                                                                                ^
[error] two errors found

The utf8 and Multiparts issue is new, thanks for reporting! I've resolved that issue in the sample repo.
The default issue is unfortunate, since it's incorrect to assume that 100 and 500 have the same semantics.


Develop

This fix date format, because the Date in format dd/mm/yy out with string first char accent, example: '07/03/2019

I've currently no access to my testing environment but will look at is ASAP. Thanks for your contribution!

All tests passed. Thank you!

A pleasure!


New Metric Template: Update time-to-first-response.md

The following were not specified:
Contributors
Visualizations

Closing this as I think this was addressed in another PR. Thanks @yigakpoa ! 🙏


Compressing unix Timestamp

Hey,
I was wondering if there is a way to compress UNIX timestamps to "Human Readable characters." For eg: compressing 14,000,000,000 to 14B.
So I am basically designing a timeline which displays dates from the start of the universe to the time you're reading this. Since date datatype cannot display dates at the start of the universe I am using Unix timestamps to display astronomical dates. here is a screenshot.

In the screenshot, as you can see, the numbers are huge and I would like to compress them. like 14,000,000 would be 14M and 13,800,000,000 would be 13.8B.
All I need to know if there is a way to customize the timestamp or an event handler in the library that'd let me handle the times before displaying. By "handle", I mean that the event handler would provide me the timestamp and I could parse it as needed and then return the String which would take place of the original timestamp without disrupting the sequence of numbers.
Environment:
browser: chrome(for dev), and OS: Linux based server.

I'm not sure what you mean by event handler; Moment just has functions to return things and you can of course parse them as you see fit.
var timestamp = someMomentObject.valueOf();
yourPrettifyFunction(timestamp);

Perhaps there's some context I'm missing?

What exactly are you doing here? Sorry but I couldn't understand what you meant there.

I probably just don't understand your question. What are you hoping Moment can do for you?

Sorry if i wasn't clear. You can close the issue since i figured it out.


Failed deployment due to terraform modules version [bug]

I am currently trying to update to a more version of qhub to make use of the GPU fixes. I am encountering a terraform error that I do not know how to go about debugging. I wonder can someone help...
I am using the 0.3.11 docker images with qhub version b81810441010b2031ee52865cb1fd392a20dd7d8 and a branch of the terraform modules:
terraform_modules:
  repository: "github.com/quansight/qhub-terraform-modules"
  rev: "gke-internal-lb-with-dask-gateway"

The above branch has some deployment specific settings to use an internal load balancer. Previously I would rebase this branch for deployment updates that included changes to terraform modules. Any ideas what the best strategy for updates is now that the terraform modules are not stored in a separate repo?

Tail of error message during deployment

  [terraform]: module.qhub.module.kubernetes-dask-gateway.kubernetes_config_map.gateway: Modifying... [id=qhub/qhub-daskgateway-gateway]
  [terraform]: module.qhub.module.kubernetes-dask-gateway.kubernetes_config_map.gateway: Modifications complete after 0s [id=qhub/qhub-daskgateway-gateway]
  [terraform]: module.qhub.module.kubernetes-dask-gateway.kubernetes_deployment.gateway: Modifying... [id=qhub/qhub-daskgateway-gateway]
  [terraform]: module.qhub.module.kubernetes-jupyterhub.helm_release.jupyterhub: Modifying... [id=jupyterhub]
  [terraform]: module.qhub.module.kubernetes-dask-gateway.kubernetes_deployment.gateway: Modifications complete after 0s [id=qhub/qhub-daskgateway-gateway]
  [terraform]: module.forwardauth.kubernetes_manifest.forwardauth-middleware: Creating...
  [terraform]: module.qhub.kubernetes_manifest.forwardauth: Creating...
  [terraform]: module.qhub.module.kubernetes-dask-gateway.kubernetes_manifest.cluster-middleware-stripprefix: Creating...
  [terraform]: module.qhub.kubernetes_manifest.forwardauth: Creation complete after 0s
  [terraform]: module.forwardauth.kubernetes_manifest.forwardauth-middleware: Creation complete after 1s
  [terraform]: module.qhub.module.kubernetes-dask-gateway.kubernetes_manifest.cluster-middleware-stripprefix: Creation complete after 0s
  [terraform]: module.qhub.module.kubernetes-dask-gateway.kubernetes_manifest.cluster-middleware: Modifying...
  [terraform]: module.forwardauth.kubernetes_deployment.forwardauth-deployment: Creation complete after 3s [id=qhub/forwardauth-deployment]
  [terraform]: module.forwardauth.kubernetes_service.forwardauth-service: Creating...
  [terraform]: module.forwardauth.kubernetes_service.forwardauth-service: Creation complete after 0s [id=qhub/forwardauth-service]
  [terraform]: module.qhub.module.kubernetes-jupyterhub.helm_release.jupyterhub: Still modifying... [id=jupyterhub, 10s elapsed]
  [terraform]: module.qhub.module.kubernetes-jupyterhub.helm_release.jupyterhub: Still modifying... [id=jupyterhub, 20s elapsed]
  [terraform]: module.qhub.module.kubernetes-jupyterhub.helm_release.jupyterhub: Modifications complete after 29s [id=jupyterhub]
  [terraform]:
  [terraform]: Error: API response status: Failure
  [terraform]:
  [terraform]:   on modules/kubernetes/services/dask-gateway/middleware.tf line 24, in resource "kubernetes_manifest" "cluster-middleware":
  [terraform]:   24: resource "kubernetes_manifest" "cluster-middleware" {
  [terraform]:
  [terraform]: Middleware.traefik.containo.us "qhub-dask-gateway-cluster" is invalid:
  [terraform]: spec.stripPrefixRegex: Invalid value: "null": spec.stripPrefixRegex in body
  [terraform]: must be of type object: "null"
  [terraform]:
  [terraform]:
  [terraform]: Error: Kubernetes API Error: Invalid Middleware.traefik.containo.us [qhub-dask-gateway-cluster]
  [terraform]:
  [terraform]:   on modules/kubernetes/services/dask-gateway/middleware.tf line 24, in resource "kubernetes_manifest" "cluster-middleware":
  [terraform]:   24: resource "kubernetes_manifest" "cluster-middleware" {
  [terraform]:
  [terraform]:
  [terraform]:
  [terraform]: Error: spec.stripPrefixRegex
  [terraform]:
  [terraform]:   on modules/kubernetes/services/dask-gateway/middleware.tf line 24, in resource "kubernetes_manifest" "cluster-middleware":
  [terraform]:   24: resource "kubernetes_manifest" "cluster-middleware" {
  [terraform]:
  [terraform]: Invalid value: "null": spec.stripPrefixRegex in body must be of type object:
  [terraform]: "null"
  [terraform]:
  Problem encountered: Terraform error
  ERROR: Job failed: command terminated with exit code 1

EDIT: My diagnosis of what is wrong my be completely wrong. The docs still seem to indicate specifying an external repo is fine

John, the docs default to 'stable' meaning v0.3.11 at the moment, and for that version of QHub you can indeed specify external terraform_modules.
From 'main' at the moment (and the commit you are using) the modules are contained within the qhub code so you would not be able to pull in the external modules.
By the way, it might make sense to use the 'main' tagged images too, e.g. quansight/qhub-jupyterlab:main - or sha-xxxxxx to pin to a particular commit, but be aware that we don't run the docker build for every single commit, only ones where the docker files have changed, so you would have to find the most recent commit (before the commit you are using) that actually build and tagged those images.
Anyway, we need a way for you to specify these terraform_modules quirks in your qhub-config.yaml ideally - or at least have a way to still specify your own terrafom_modules repo as a fallback. Are you in a good position to summarize what you need to configure and make a new issue for that? Perhaps it's just the first two commits here.
In general, hopefully 'main' branch works, but it isn't guaranteed to be stable of course. My view is that it's good to get code into main earlier rather than later so that it gets tested in the weekly integration tests.
I think I can see what the immediate problem is from your logs above so I will fix, although main will still not incorporate your terraform_modules of course.

The terraform error should be fixed when my new PR is merged.
Thanks for opening the other issue about external terraform modules.

Thanks for fixing this one so quickly!


Replaces non-inclusive terms

Adds PRs https://github.com/saucelabs/sauce-docs/pull/1905 and https://github.com/saucelabs/sauce-docs/pull/1904
Description
Replaces "whitelist" and "blacklist" with "allowlist" and "blocklist" in the Error Reporting docs to promote more inclusive language.

Account Management | Sauce Labs Documentation

Connecting to Symbol Servers | Sauce Labs Documentation

Motivation and Context
In an effort to promote more inclusive language. This change reflects our commitment to creating a welcoming and inclusive environment for all users.
TCD-2744
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation fix (typos, incorrect content, missing content, etc.)

Checklist

[ ] I have read the contributing document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

The screenshots will be updated in a separate PR.


Data Science with Python : Stochastic Gradient Descent

Welcome to 'DSWP' Team, good to see you here
This issue will helps readers in acquiring all the knowledge that one needs to know about Stochastic Gradient Descent . Tutorial to Stochastic Gradient Descent  and how it's applied using sample code.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Machine Learning
Mentors Assigned : Dhruv Bajaj / Aishani Singh
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

Hello @prathimacode-hub
I am Sakalya from Batch 2 of DSWP
I want to work on this issue

Kindly mention serial number, choice and approach you would follow. @Sakalya100

Done. Updated @prathimacode-hub

I want to contribute through audio in this

@prathimacode-hub add me as assignee

Issue assigned to @Aaksha-sharma for audio contribution

@prathimacode-hub  maam i have already made
pr to this issue long back but it has not been acknowledged

Can you share the link of your PR? @Aaksha-sharma

https://github.com/girlscript/winter-of-contributing/pull/5987

@prathimacode-hub , Mam
I am M. Ashish Reddy. I would like to do video on this issue.
Batch number: 5
serial number:174.
Approach: A detailed explanation with examples and real life uses

Issue assigned to @ashish-reddy-20-08 for video contribution


readme: update video link/thumbnail

Hello @yuindustries, @landism,
Please review the following commits I made in branch maiamcc/new-demo-vid:
4ad81039cd57ad9dcaf4785b44b42c19d47ca6a4 (2019-05-21 14:22:06 -0400)
readme: update video link/thumbnail
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

Video thumbnail:


windows build target support

Tested on X64 Windows10 with following dependencies;

Clang(LLVM): for bindgen
CMake: Makefile in sys/whisper.cpp is not implemented for windows
Visual Studio Build Tools: for C++ library compilation

I'm gonna add some GitHub Action CI codes for normal compilations and cross compilations tests, then convert this to actual PR.

Request a review from me once this is ready that way I see it faster.

Probably superseded by #18


One test can't run without CGenFF

We don't have CGenFF here, so test_run_macha will always fail, I labeled this test as: cannot run on CI

Codecov Report

Merging #17 (9f025e7) into main (41786a2) will increase coverage by 0.08%.
The diff coverage is 19.23%.

:exclamation: Current head 9f025e7 differs from pull request most recent head 91e67cc. Consider uploading reports for the commit 91e67cc to get more accurate results

Additional details and impacted files


Additional attributes for mowers

Two attributes have been added to mowers devices.
Both are compiled from existing attributes:

error code contains NONE if there is no error and the code from last error code if an error is active. This gives us a data point that simply contains clear error status information.
status code contains a copy of activity as long as there is no error and a copy of error code if an error is active.  This corresponds to the behavior of the old device state in the integration of Wijnand Top.

Examples:
No error

Active error

No error, last error code still visible

Is there really any use for last error code with this? Isn't it better to use the Home Assistant history for that?

Is there really any use for last error code with this? Isn't it better to use the Home Assistant history for that?

last error code is actually what the Gardena API gives us. From this the more useful attributes are derived by this PR. There is actually no data point in the API that simply gives us an active error instead of some active-or-history thingy.
We could decide not to expose last error code but as this is the original API data, I would tend to keep it.

Not sure if this is posted in a suitable place but I am wondering how the activity maps to the status in the lovelace entities card?
For example "activity OK_CUTTING" shows up as "Cleaning". Is this customizable or is this one of the things you are looking into when adapting this integration to mowers?

Not really the right place as this is a pull request with a different subject.
The possible states are predefined for each HA integration. See https://developers.home-assistant.io/docs/core/entity/vacuum/#states
The only thing that can be changed in the custom component is the mapping of mower states to available vacuum states. But you will not get rid of "Cleaning" and it is the closest we have to "Mowing".

@osks can you give me feedback if you want to remove last_error_code?
If so I will send another commit.

Are you sure that the activity is None only when there is an error ? And not in other case ?
We should keep the last error code at least while we make sure there is no issue with 'ATTR_ERROR_CODE' and 'ATTR_STATUS_CODE'.

Okay, let's keep error code.

@Jpsy : Are you sure that the activity is None only when there is an error ? And not in other case ?

Nope, definitely not. This is conclusion by observation. It is very well possible that there are conditions where this simple rule fails. E.g. hibernation mode of the mower could break that logic. We may have to fine tune over time. I will observe and take care.

Sorry to create more fuzz, guys:
I just pushed another commit to this PR which may provoke your resistance.
The ... code postfixes in the attribute names were getting onto my nerves. So I removed them all. We have "codes" here everywhere (activity is actually an activity code). So why postfix some and others not.
Total changes:

last error code -> last error
error code -> error
status code -> state

Yes, I am aware that last error code is the wording from the API. But in the switch device you have already shortened it to last error, so why not in the vacuum. I am also aware that touching this attribute name introduces a breaking change. This is sad, but we should unify now, not in some months when more users are on board.
BTW: Changing from status code to state is the result of the fact that HA does not accept an attribute name of status. I guess it's a reserved word. But after all "state" is semantically the better choice anyway.

Throw your valued opinions at me, please! :D

I merge that and make a new release !
Thx a lot
Grm


MMPA error during fragment generation

RDKit Version:017.03.3
Platform: MacOS

When running the command:
python $RDBASE/Contrib/mmpa/rfrag.py < smiles.txt > frags.txt
as suggested by this tutorial, the follow error results:
Traceback (most recent call last):
  File "/Users/owner/anaconda/share/RDKit/Contrib/mmpa/rfrag.py", line 223, in <module>
    cmpd_id = line_fields[1]
IndexError: list index out of range

A sample smiles.txt file may be found here.  Any help would be more than appreciated.

This smiles file appears to be private.

However, my guess looking at the source code is that your input file is not quite correct.
If you want the code to skip malformed lines, you could try the following:
Change
    line_fields = re.split('\s|,', line)
    smiles = line_fields[0]
    cmpd_id = line_fields[1]

to
    line_fields = re.split('\s|,', line)
    if len(line_fields) < 2:
       print("Malformed line: %s"%line, file=sys.stderr)
       continue
    smiles = line_fields[0]
    cmpd_id = line_fields[1]

You are absolutely right!    Also, just fixed the private status for the pastebin - sorry about that.  Thanks for the fix!


[WIP] Add Appveyor configuration for windows CI

@samuelpowell: This seems to fail on Windows. Is it supposed to work there?

@tknopp yes and no!
We previously decided that because of the lack of support for serial ports on CI it would suffice simply to check that the library can be loaded and ports listed on Linux/Mac. At least until such time that a 'no hardware' test suite can be added.
Unfortunately, list_ports fails on windows (#22). I recall that this might be a limitation in libserialport, but I don't generally use Windows and thus it is hard for me to debug/test against the raw C library. If it can be shown that the underlying library call should pass on Windows, we can look further at this.
Marked as WIP for now.

Closed in favour of #59


fix: better error for missing links property

Before:
error: failed to run custom build command for `tauri-plugin-specta-example v0.0.0 (/Users/oscar/Desktop/tauri-specta/examples/custom-plugin/plugin)`

Caused by:
  process didn't exit successfully: `/Users/oscar/Desktop/tauri-specta/target/debug/build/tauri-plugin-specta-example-57c4280f4909e379/build-script-build` (exit status: 1)
  --- stdout
  tauri-plugin: expected build script env var CARGO_MANIFEST_LINKS, but it was not found - ensure this is called in a build script

After:
error: failed to run custom build command for `tauri-plugin-specta-example v0.0.0 (/Users/oscar/Desktop/tauri-specta/examples/custom-plugin/plugin)`

Caused by:
  process didn't exit successfully: `/Users/oscar/Desktop/tauri-specta/target/debug/build/tauri-plugin-specta-example-6cd8c958ec0b1cb7/build-script-build` (exit status: 1)
  --- stdout
  tauri-plugin: package.links field in the Cargo manifest is not set, it should be set to the same as package.name
 ELIFECYCLE  Command failed with exit code 101.

Thanks! We'll need a small changefile before we can merge it though.

Fixed, sorry!

And now it's complaining about unsigned commits 😅


Segmentation fault from BigInt comparison with float

It seems that a BigInt instance doesn't like being compared to a float.
$ cat test.cr
require "big_int"

def foo
  x = 8.to_big_i
  y = 20.16
  if x < y
    puts "bar"
  end
end

foo
$ crystal test.cr
Program exited because of a segmentation fault (11)

I can work around this by casting x or y to match the type of the other but thought it would be good to file a bug.  Let me know if you have any questions.  Thanks!

I'm curious if a solution to this would be to implement a larger concept of BigNum

I think the issue is a recursive method call that's being called (so it's a disguised stack overflow)

Thanks!


Add CLI support to clear the cache

We should have a CLI subcommand (or flag?) that allows us to clear the package cache.

I think it would benefit to have some subcommands, as we add more functionality. Maybe a cache subcommand which would allow us to do modmark cache list, modmark cache clear etc.


Split out module MergingRun

It is one of the core abstractions of the code base. This will allow us to use MergingRun in other modules without having to depend on the whole MergeSchedule.
I also untangled its usage a bit to provide a clean module interface. For some of the later changes, there are many alternative solutions, so I added them as separate commits that can be better viewed and discussed individually.

I rebased and addressed all points except for the reference handling issues in MergeSchedule.addRunToLevels and MergeSchedule.fromSnapLevels. I'll continue with those, but I think they can/should go into a separate PR, since the issues existed before and this PR already is large enough. So this PR might be merge-able already, allowing to avoid merge conflicts.

I pushed a fix for the reference handling issues.

You fixed the issue in fromSnapLevels, did you not?

Now I did, but before my commit from today, an exception after creating a MergingRun would still have lead to a double free.

Squashed a few commits, I think it makes sense to separate them as they are now.


Add FileSelection Widget

A companion PR to https://github.com/LiveSplit/livesplit-core/pull/748.
Implements the FileSelection Widget as a button in the settings Gui that opens a file dialog when pressed. The user can select a file, and the WASI-compatible path to that file gets stored in the settings map.
This code won't fully work until after the livesplit-core dependency has been updated, but to help with testing I have this branch to point the dependency at my livesplit-core PR branch: https://github.com/AlexKnauth/asr-debugger/commits/file-selection-demo

The egui_file dependency changed, so I think I need to replace my use of the FileDialog method filter, with either show_files_filter or filename_filter. Depending on which of those I use, I'll need to replace my use of the Filter type with either Filter<PathBuf> or Filter<String>.


Why id Editor can't use else customized images

URL
No response
How to reproduce the issue?
dame.
Screenshot(s) or anything else?
No response
Which deployed environments do you see the issue in?
No response
What version numbers does this issue effect?
No response
Which browsers are you seeing this problem on?
No response

Hi there - we can't respond to your question without additional information - I'm closing this issue now because it doesn't have any details, but if you wish to ask a general question rather than report a specific bug or a feature request, please open a new thread in the Discussions section with as many details as possible about what you're trying to do, what you're expecting iD to do, and what it's doing instead of what you expected. Thank you!


Test master version of Balancing Act on multiple platforms, check for strobing text

The parent issue for this is https://github.com/phetsims/balancing-act/issues/16.  In that issue, which is from a long time ago, I fixed a problem where rotating text looked bad on some platforms.  The fix was to "rasterize" the text, which means to turn it into an image before displaying it.  @samreid recently came across some items marked as TODO in the code about potentially removing the rasterized text in the future when browser support for rotating text improves.  Well, the future is here, so I thought I'd try using regular, non-rasterized text to see if it works well now.  I made the change, then tested on Chrome and Firefox, and it looked fine.
QA team: Please test the master version of Balancing Act on multiple platforms, comparing it against the currently published version, and looking for any "strobing" with rotating text, which basically means it jumps around and looks bad.  I've found the best way to test this is to set up a situation with lots of text that can rotate and then move things around so that the motion of the text is relatively slow.  I've got a screenshot of such a situation below, where if one moves the 5 kg brick around, the plank rotates fairly slowly.
Rotating text can be found on the labels for things that can be put on the plank, on the ruler, and on the "Marks".
This testing should be pretty focused on the rotating text, please don't spend much time on anything else.

Platforms:

[x] Win 10 Chrome
[x] Win 10 Firefox
[x] MacOS 11 Safari
[x] MacOS 11 Chrome
[x] ChromeOS
[x] iPadOS 14

On Win 10 the text overall looks better, but I didn't see any jittering in the published sim itself.

The text itself looks good with Mac 11 chrome/safari. I noticed some movement in the top portion and bottom border of the ruler itself when using Mac11 + safari--almost like a wave moving through.
https://drive.google.com/file/d/1IqMX4D0Im8OoWv3-oILWETZoUCKCPEdA/view?usp=sharing

It looks like the wave is caused by "aliasing", for example, see the top image on the Bresenham antialias Algorithm page:
https://en.wikipedia.org/wiki/Bresenham's_line_algorithm
I'm not sure what can be done about it.

@KatieWoe said:

...I didn't see any jittering in the published sim itself.

You won't.  It had the workaround in place, i.e. the rasterized text, so the jittering doesn't occur there.  You may, however, notice that the new text is slightly more crisp than the text in the published sim, especially on high-resolution displays.

Text jiggles around on ChromeOS on master

QA is done

Text jiggles around on ChromeOS on master

@KatieWoe - In your opinion, is this bad enough versus the published version that I should revert back to the workaround (the rasterized text)?

I'm really not sure. It was noticeable, but I probably wouldn't have made note unless it was what I was looking for. I'm not sure how bad it would have to be to revert.

Would it help to record a video on ChromeOS?

@KatieWoe said:

I'm not sure how bad it would have to be to revert.

Not bad, I'd just roll back to a point before I removed it.  Of course, I'd prefer not to, and it would be nice to get rid of the workaround, but not if it has a noticeable negative impact on the user experience.
@samreid said:

Would it help to record a video of the ChromeOS behavior?

Yes.  Either that or, if it's possible to set up a Zoom call from the ChromeOS device, we could set up a time and do that.  Or, if @KatieWoe is comfortable making a recommendation either way, that would be cool too.  If it falls to me to make a decision, I'll need to see what it looks like.

Here are some images from the chrome book. It is up to date, but it is one of our lower performing devices.
https://user-images.githubusercontent.com/41024075/128395836-f60b7bc3-0d38-42ef-b5a1-ed42f0f9c720.MOV

My conclusion after reading through the comments and looking at the recordings and trying it out myself is that the strobing and aliasing still exist, and I should leave the workaround in place, at least for now.  Thanks to the QA team for testing this.  Closing.


Refactor  template includes

Pulled navigation out of individual pages, into _includes/header.html
'active' element ID for navigation is set dynamically by page path

maybe this should be a class?

Page titles are pulled from page front matter

title: page specific title

the Showcasing good starter issues portion has been added to the index front matter so it is present on the home page

Meta tag for description is set via _config.yml and pulled into the header of each page

Thank you so much @robjloranger and thanks for being patient with me :)

No problem, as soon as I have some free time I'll start on the blog if your still interested.

Sure! Any time is appreciated.


some minor fixes:

If alerts is empty after relabelAlerts, just return to avoid
subsequent unnecessary operations

minor fix in notifier's test case

minor fix in comment

Signed-off-by: YaoZengzeng<EMAIL_ADDRESS>

Thanks!


resources.GetRemote HEAD method errors with unexpected EOF

Reference: https://discourse.gohugo.io/t/42272/3
When I do this:
{{ $url := "https://gohugo.io/img/hugo.png" }}
{{ $opts := dict "method" "head" }}
{{ with resources.GetRemote $url $opts }}
  {{ with .Err }}
    {{ errorf "Unable to get remote resource: %s" . }}
  {{ else }}
    {{ .Content }}
  {{ end }}
{{ else }}
  {{ errorf "Unable to get remote resource: %s" $url }}
{{ end }}

I get:

Unable to get remote resource: error calling resources.GetRemote: failed to read remote resource “https://gohugo.io/img/hugo.png”: unexpected EOF

But the response is captured and cached in:
/tmp/hugo_cache/my-project/filecache/getresource/6247740232174893946

A quick code scan, we should check if the response body is nil bore reading it, as a HEAD response doesn't contain a body.
https://github.com/gohugoio/hugo/blob/master/resources/resource_factories/create/remote.go#L138
should check if res.Body is nil.
I can create a PR to fix and test this next week.

should check if res.Body is nil.

I'm checking this now and in @jmooring 's test case the body is not nil, ContentLength is reported as 18210, yet Go is throwing this unexpected EOF error. Will need to ... Google this.

OK, the Mozilla docs are useful here:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/HEAD

So, the correct way to handle this is to just ignore the body when the method is HEAD. I will create a pull request.

... but that said, I don't see the value of a Resource object from a HEAD request without the ContentLength etc. response info ... I'll see if I can add that to the mix. We have a .Data map that we could stuff this info in.

🥳


SLF4J: Class path contains multiple SLF4J bindings

Installed AntiVPN 4.7.29 on a waterfall proxy and this error caught my attention.
https://pastebin.com/7EHLyeXu
The first part is the downloading process because it's a fresh install. Last lines are the ones that matter.
[08:33:47] [main/ERROR]: SLF4J: Found binding in [jar:file:/home/container/plugins/antivpn-bungee-4.7.29.jar!/org/slf4j/impl/StaticLoggerBinder.class]
[08:33:47] [main/ERROR]: SLF4J: Found binding in [jar:file:/home/container/Waterfall.jar!/org/slf4j/impl/StaticLoggerBinder.class]
[08:33:47] [main/ERROR]: SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
[08:33:47] [main/ERROR]: SLF4J: Actual binding is of type [ch.qos.logback.classic.util.ContextSelectorStaticBinder]

Happens too on a restart.
This error was present in 4.7.28 and still is alive and kicking in 4.7.29
Waterfall version git:Waterfall-Bootstrap:1.14-SNAPSHOT:1f16030:286 by md_5

I think this belongs to AntiVPN as well. Only visible on the console and not in the log.
[08:48:02 INFO]: [08:48:02] [com.zaxxer.hikari.HikariDataSource] [INFO] - HikariPool-1 - Starting...
[08:48:02 ERROR]: Loading class `com.mysql.jdbc.Driver'. This is deprecated. The new driver class is `com.mysql.cj.jdbc.Driver'. The driver is automatically registered via the SPI and manual loading of the driver class is generally unnecessary.
[08:48:02 INFO]: [08:48:02] [com.zaxxer.hikari.HikariDataSource] [INFO] - HikariPool-1 - Start completed.
[08:48:02 INFO]: [Anti-VPN] Plan was not found. Personal analytics support has been disabled.
[08:48:02 INFO]: [Anti-VPN] Enabled
[08:48:02 INFO]: [Anti-VPN] [Version 4.7.29] [2 Commands] [2 Events]

Harmless, just annoying :)
I'll find a fix for the SLF4J binding warnings at some point.

Still broken...
https://pastebin.com/e1Bq40p3

Indeed, though still harmless. It's on my to-do, anyway.


Remove integrity constraint until we can handle deleted TaskAssignments

lgtm!

Coverage decreased (-0.003%) to 91.471% when pulling bc6c91e495c920a000099c2b1befd749b918028d on taskassignment-remove-integrity into 264fa3fe6725fc871386536b2de1b47cadef2fce on master.


Transition between themes

It would be interesting to be able to have some sort of transition effect between the switch of themes. I don't know if this is possible at the moment, but it could be a very interesting visual effect.

@gzuuus we're very much welcoming ideas like this for our next major iteration of the theme system. Would you mind folding this request here. Just responding in the thread is perfect, we'll see it:
https://github.com/skeletonlabs/skeleton/issues/1142
I'd also encourage you to read through the ideas we do have proposed already. It's not a complete list, but should shed some light on the general direction we're heading with this


Feature/refactor worker todo pluginmgr

[x] after refactoring, we can just use the pluginmanager instance var instead of passing it around

:+1: LGTM


Could I implement generic ChildEventListener?

I'm trying to implement generic analog of FirebaseArray,
And I'd like to know can I somehow put generic type in
snapshot.getValue(Generic)
in methods onChildAdded, onChildChanged, onChildRemoved, onChildMoved

I found related solution for Lists with GenericTypeIndicator
firebase.com/docs/...
GenericTypeIndicator<List<Message>> t = new GenericTypeIndicator<List<Message>>() {};
     List<Message> messages = snapshot.getValue(t);

But I couldn't adapt it for single items

@CyberNyanta I'm not sure if I completely understand your question, but let me see if I can.
I think you'd like to implement a class like this:
public class MyArray<T> implements ChildEventListener {

   // ...

   @Override
   public void onChildAdded(DataSnapshot child, String prevChildKey) {
       T data = child.getData(T.class);
   }

}

Unfortunately due to the way Java implements generics you can'd do that, but you can do this:
public class MyArray<T> implements ChildEventListener {

   // ...

   private Class<T> mClazz;

   public MyArray(Class<T> clazz) {
     mClazz = clazz;
   }

   @Override
   public void onChildAdded(DataSnapshot child, String prevChildKey) {
       T data = child.getData(mClazz);
   }

}

Thank you!


sort: implement '-t' flag

The -t flag lets you specify the field separator to use when sorting values.
       -t, --field-separator=SEP
              use SEP instead of non-blank to blank transition

However, when I try to use it, I get:
error: Found argument '-t' which wasn't expected, or isn't valid in this context

USAGE:
    sort 0.0.3

Usage:
 sort [OPTION]... [FILE]...

Write the sorted concatenation of all FILE(s) to standard output.

Mandatory arguments for long options are mandatory for short options too.

With no FILE, or when FILE is -, read standard input.

For more information try --help

I ran into this when running pyenv install -l to list installed Python versions.

I've been looking into this as a first bug to fix while I'm learning Rust.
I'm not that familiar with coreutils so I had to look up how the -t flag exactly works.
So far I've found the -t flag is being used in combination with the -k flag.
The -k isn't implemented either so implementing the -t flag would require both flags to be implemented.
Can someone verify this and/or bring up other use-cases of the -t flag?

I can confirm that at least the command that led to this issue uses -k multiple times. The command being run is currently:
LC_ALL=C sort -t. -k 1,1 -k 2,2n -k 3,3n -k 4,4n -k 5,5n

This should work now: https://github.com/uutils/coreutils/pull/1996


Update setup instructions for macOS

The preferred method for installing the ARM toolchain has changed. The method currently shown results in a compilation failure:
arm-none-eabi-gcc: fatal error: cannot read spec file 'nosys.specs': No such file or directory

Refer to pico-sdk issue 1529

Thank you!!


What about browser compatibility？

Hello guys, I want to know browser compatibility？Minimum supported Chrome version？

Perfume uses a large collection of APIs, which tends to have different browser compatibility. The goal for Perfume is to fully support all the latest APIs and simply silently not return results for the old browser.
Here a quick list:
Network Information API - effectiveType: Chrome 61+, Opera 48+, Edge 76+, Chrome for Android 76+, Firefox for Android 68+
Save Data API: Chrome 65+, Opera 62+, Chrome for Android 76+, Opera for Android 46+
Hardware Concurrency API: Chrome 37+, Safari 10.1+, Firefox 48+, Opera 24+, Edge 15+, Chrome for Android 76+, Safari on iOS 10.3+, Firefox for Android 68+, Opera for Android 46+
Performance memory API: Chrome 7+, Opera, Chrome for Android 18+, Opera for Android
Device Memory API: Chrome 63+, Opera 50+, Chrome for Android 76+, Opera for Android 46+
I missed probably some particular method, like Performance Observer, but yeah feel free to ask any API or method, in particular, you are curious about.

Thanks


MAINT: Add arm64 CI using CirrusCI

Closes #11174

Looks like it only takes 37 minutes which is great! Ready for review/marking for merge fun my end


Query Engine: Asyncification

After the redesign branch is done, we can tackle this 🎉

Done to the old master here https://github.com/prisma/prisma-engine/tree/bench (as an example)

Merged to master.


Add "Mass Effect" faker

Add Mass Effect Faker with character/specie/planet/cluster -names and quotes.

Codecov Report

Merging #301 (729e169) into master (87a721b) will decrease coverage by 0.40%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #301      +/-   ##
============================================
- Coverage     94.86%   94.45%   -0.41%
- Complexity     1893     1894       +1
============================================
  Files           197      198       +1
  Lines          3797     3805       +8
  Branches        379      379
============================================
- Hits           3602     3594       -8
- Misses           99      114      +15
- Partials         96       97       +1

Impacted Files
Coverage Δ

.../main/java/net/datafaker/service/files/EnFile.java
93.33% <ø> (ø)

src/main/java/net/datafaker/Faker.java
98.35% <100.00%> (+<0.01%)
:arrow_up:

src/main/java/net/datafaker/MassEffect.java
100.00% <100.00%> (ø)

...ain/java/net/datafaker/idnumbers/EnZAIdNumber.java
83.33% <0.00%> (-8.34%)
:arrow_down:

src/main/java/net/datafaker/Lorem.java
91.66% <0.00%> (-6.25%)
:arrow_down:

...ker/idnumbers/pt/br/IdNumberGeneratorPtBrUtil.java
92.59% <0.00%> (-3.71%)
:arrow_down:

src/main/java/net/datafaker/Code.java
95.40% <0.00%> (-2.30%)
:arrow_down:

.../java/net/datafaker/service/FakeValuesService.java
86.92% <0.00%> (-1.01%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Awesome, thank you for this!


Remove user from project before deletion

Description
If a user that is associated to a project gets deleted, other users become unable to see who belongs to the project via listProjectAccounts due to a NPE. This happens because the removed user is still associated with the project.
This PR fixes the issue by removing the user from all projects before deleting it (as we already do with accounts). To normalize environments that are affected by the bug, the following script will be executed during the upgrade in order to delete project-user associations for users that were removed.
DELETE FROM `cloud`.`project_account` WHERE `user_id` IN (SELECT `id` FROM `cloud`.`user` WHERE `removed`);

Fixes #9974
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Bug Severity

[ ] BLOCKER
[X] Critical
[ ] Major
[ ] Minor
[ ] Trivial

How Has This Been Tested?
User clean-up on deletion

I created a project
I added a user to the project
I deleted the user

After step 3, I would become unable to list accounts/users that belong to the project via listProjectAccounts before the changes. With the changes, I was able to list them normally.
Normalization of affected environments
Before applying the patch, I created some broken projects by deleting their users. Then, I executed the upgrade script and verified that I was able to list their accounts/users normally.

@blueorangutan package

@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 11657

@blueorangutan test

@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests

[SF] Trillian test result (tid-11835)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 45441 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10008-t11835-kvm-ol8.zip
Smoke tests completed. 132 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_01_secure_vm_migration
Error
134.37
test_vm_life_cycle.py

test_01_secure_vm_migration
Error
134.37
test_vm_life_cycle.py


setting avatar based on user profile not gravatar

Hello,
I cannot seem to figure out how to override the gravatar avatar defaulting on each of the posted comments.
I would really appreciate any help on the matter.
Thanks.

Hello! There is a line in the code of the Dummy app where you can find an example how to overwrite it.
https://github.com/the-trash/the_comments/blob/master/spec/dummy_app/app/models/comment.rb#L13
Could you give me some information about you steps to solve your problem. What did you do? What result did you have?


Sketch 80.1 Compatible？

Sketch 80.1 Compatible？

doesn't seem to be. i tested on both 80.1 and 81.1, no luck so far.

This might not be helpful, but I have just upgraded to Sketch 85.1 and meaxure seems to be working as expected.

This might not be helpful, but I have just upgraded to Sketch 85.1 and meaxure seems to be working as expected.

doesn't work for me in Sketch 85.1, always remind me 'TypeError: null is not an object' ....

Sketch 83 works fine!


Error: migrate: migration[276]: Add RemoteAddress to mirrors failed: get remote origin's address of MyORG/MyPRJ failed: exit status 128 - fatal: not a git repository (or any parent up to mount point /)

Description
I was upgrading gitea from 1.20 to 1.21, and after recovering the pg database and gitea custom files, the following error was reported at boot.
I tried version 1.22 again with the same result.
Complete startup information: https://forum.gitea.com/t/1-20-1-21/9388
The organization/repository displayed in the error message is the mirror repository that I cloned from the Intranet.
I found someone with a similar problem in another place. https://codeberg.org/forgejo/forgejo/issues/1800
Gitea Version
v1.21.11&1.22
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.39.1
Operating System
synology-dsm-docker
How are you running Gitea?
from official image on docker hub
Database
PostgreSQL

A side question: why did you need to recovery when upgrading?


GPU usage

Similar to CPU usage, it would be nice to have a module for GPU usage. Maybe one would have to set something akin to hwmon_path to know which file to track. Also, I don't know how this would work with Nvidia's proprietary driver (Yes, I have an Nvidia GPU in addition to my RX 6800 XT, lol), but I don't see why Nvidia should be a priority anyway.

I've created a guide to do just that, I've also linked it on the wiki page for custom module

@yonatan8070 Along these lines, the custom GPU temperature thread link can be adapted to Nvidia cards with one of the nvidia-smi commands listed in the top answer here: https://askubuntu.com/questions/34449/how-to-see-the-video-card-temperature-nvidia-ati-intel
This could be a useful note to add in the wiki.

I'm a bit busy at the moment so I don't have time to add it myself, you're more than welcome to add it yourself if you have the time!

I made this tool to display NVIDIA GPU stats on the bar
See an usage example on my dotfiles
The config states that nvidia-smi is required, in reality only NVML library should be installed (which should come with the driver AFAIK)

If anyone wants you could use the NVML API directly inside Waybar, unfortunately my C++ knowledge is too limited to do that.
An example of usage can be found in nvtop (written in C)


[STM32F4 family] Assign CAN filter number properly

There are 28 filter banks which are shared between CAN1 and CAN2. By default
they are divided in half:
* CAN1 -> 0  ... 13
* CAN2 -> 14 ... 27

that's why we need to decide which filter number has to be chosen.

LGTM


Sample factory integration

Adds sample factory integration, example in examples/sample_factory_training.py
Note, there is a bug with torch.multiprocessing in pytorch 1.12 using 1.11 (pip install torch==1.11.0)
Command to run the training:
python examples/sample_factory_training.py --env=simenv --experiment=simenv --num_workers=30 --num_envs_per_worker=1 --algo=APPO --worker_num_splits=1 --ppo_epochs=1

closing as updates need to be made to sample-factory before this can be integrated