Review coroutines stack size

Ok now that I found it I remember looking at it before, but this time I'll record here that it was because of stack space.
The vulkan driver calls a bunch of functions, wrapping MainWindow inside the following fixes it.
(def test-wire (Wire "" :Looped :LStack
 ...
))

Maybe anyways we should increase the default stack size, currently it's 128kb (1MB in debug)
Originally posted by @guusw in https://github.com/fragcolor-xyz/shards/issues/438#issuecomment-1370098354

Maybe we can have a mechanism that during compose shards can request a certain amount of stack space (in this case MainWindow)

Fixed by #514?

Fixed by #514?

Not really but for now it's enough. Eventually we want to research some proper numbers on how many coroutines a big game/project will need.


add net standard support

I can't use the package until I use an import's statement for netcore5.0 which I'd rather not do.

Also, is there any reason for supporting net46? Would net451 work?

Ah, no, sorry.
The synchronization primitives are actually all using a replacement core implementation (AsyncWaitQueue and friends, which most people aren't even aware exists). The new core depends heavily on the TaskCreationOptions.RunContinuationsAsynchronously flag, which was introduced in .NET 4.6. This is necessary to prevent thread pool starvation and stack dives, which can happen with the old implementation.

In fact, the dependency will be on net462. Once it's released...

Thanks for the heads up!

1.0.0-beta-2 supports netstandard1.3 with proper (trimmed) dependencies.
Upon review, the bug that exists in 4.6's support for RunContinuationsAsynchronously is IMO rare enough that AsyncEx.Coordination will officially take a dependency on net46 / netstandard1.3.

Awesome! Just a heads up the rtm bits and tooling leaked. Check Twitter :)
Get Outlook for iOS
On Sat, Jun 25, 2016 at 9:55 PM -0500, "Stephen Cleary"<EMAIL_ADDRESS>wrote:
1.0.0-beta-2 supports netstandard1.3 with proper (trimmed) dependencies.
Upon review, the bug that exists in 4.6's support for RunContinuationsAsynchronously is IMO rare enough that AsyncEx.Coordination will officially take a dependency on net46 / netstandard1.3.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

That is extremely unfortunate.
Also, I apparently don't follow the right people on Twitter for leaked software. :)

https://twitter.com/h0x0d and me of course (https://twitter.com/blaken)

I believe all the changes you need to make to support rtm is update tooling version in global.json and update dependencies :)

Probably. Then further changes when the tooling RTMs.


fix multiple changing of AnimatedSource property to same value

Hi! Thanks for the good lib!
However, i ran into strange problem with non-animated images. This is steps to reproduse:

Create WPF window with Image
Set from code AnimatedSource property with NOT animated image (e.g. any png or jpg image)
Show window
Create and Show second instance of that window (OR: close first then create and show second window)
App crashes with 'System.ArgumentException' - The provided DependencyObject is not a context for this Freezable.
But if I set my image from XAML it works well.
While debugging I have found out that on show of second window it sets AnimatedSource property of Image second time (I have no idea why). So this is quick fix for that case.

PS i can provide sample project if you want

Hi,
Thanks for the PR!
The fix seems simple enough and probably isn't too risky, but I'd like to try to reproduce the issue myself to understand the root cause first. I'll get back to you ASAP.

OK, I reproduced the issue, but I don't really understand what's going on... somehow, setting imageControl.Source triggers a call to AnimatedSourceChanged, which doesn't really make sense because the value didn't actually change. AnimatedSourceChanged in turn tries to set imageControl.Source, but it fails because it's already being set (there must be a guard against reentrancy).
Anyway, now that I see what's going on, your fix makes sense, so I'll merge it and publish a new release.
Thanks!

The new release is on NuGet

Great!


Removing IsValid

Hi, first off I want to thank you for this library. It's made my life so much easier.
There are definitely ways to cause UB in safe Rust with this library. The main one I ran into today was needing to call is_valid before calling any other methods on some types.
I think it would be more "rust-y" if maybe_wrap was the default and wrap was unsafe and only used when the check is too expensive for performance reasons and we already knew it was valid for another reason. This would mean if I had a SBFoo I'd know it's already been checked for validity.
I'm not up for making this change to everything at once but I'm open to doing this for some types as I work on my own project, if that sounds useful to you?

I like the IsValid trait in the lldb bindings from the vscode extension.
Having that have a .check() is nice ... and if I had an impl of Into / From that let you do:
let raw = unsafe { CreateSBWhatever() };
raw.into().check()

That'd be nicer than what is done now with wrap / maybe_wrap.


feat: a new simple (correct) test that may break some accepted solutions

Some of the community solutions incorrectly try to locate STOP codons from input and strip anything that follows the first STOP codon.  This is a neat idea, but if the regex is not anchored, or if it does not align to groups of 3 - it's possible that a valid pair of codons (AUGAUG, for example,) creates a pattern that can cause the regex mis-identify a sequence as a STOP codon when it is not (UGA, in the previous example).
  AUGAUG
   ^^^ - not a STOP codon

  UUAAUG
   ^^^ - not a STOP codon

Summary
A couple of the accepted solutions to this problem have a subtle bug that the tests are not catching.  This PR corrects that by putting a simple test in place.
Click to see examples...

Bonus
I noticed that the contractions in the test descriptions were not rendering correctly, so I changed them to non-contractions.

Checklist

[ ] If docs where changed, run ./bin/configlet generate to ensure all documents are properly generated.
[ ] CI is green

Thanks for the pointer!  I opened https://github.com/exercism/problem-specifications/pull/2458 - for cross referencing.

Do you want to submit the jq version of the test now? Use bin/configlet sync and bin/generate_tests

Woops. I did a sync PR. Feel free to ignore.

Isaac created #243. Do you want to review that, or try to do it yourself?


Get generic parameters for a struct field

I am not sure if I am missing something or if this is not possible with the current graphql schema.
Example:
Let's say I have the following struct in my crate:
struct MySettings {
  a: HashMap<String, MyConfig>
}

struct MyConfig {
  value: u16
}

I now want to query rustdoc like so:
query {
  Crate {
    item {
      ... on Struct {
        name @filter(op: "=", value: ["$struct_name"])

        field {
          field_name: name @output
          raw_type {
            type_name: name @output
          }
        }
      }
    }
  }
}

I would get something like that in the query output:
{
    "field_name": "a",
    "type_name":  "std::collections::HashMap",
}

I want to retrieve the HashMap generic parameters to run another query that would retrieve data for MyConfig.
Context:
I am trying to automate generation of a config reference documentation. initially I tried to use schemars which seemed to fit the usecase perfectly, until I stumble on the same issue with Map generics parameter not being parsed (see: https://github.com/GREsau/schemars/issues/124).

Hi @oknozor, unfortunately this isn't possible yet, but I have a possible workaround. We're making steady progress toward expressing this in the schema, but I don't have a concrete timeline to offer.
I think using a more recent version of the adapter (after #528) might make it so that the generics are printed as part of type_name in your query, but I'm not sure. (The RawType schema type is documented as "permanently unstable" and intended for experimentation, so I'm not 100% sure about its status right now.) But that's not as part of the schema, so it isn't a full solution by any stretch.
If the above isn't good enough, I'd recommend grabbing the id property of the StructField type (inside the field edge) and looking up the field type inside the rustdoc index directly. This is a workaround that will get you the full type info, though you'll still have to deal with the Rust type system's complexity mentioned here 👇
As for the full solution: modeling Rust types properly in our schema is quite hard: we have to represent everything from &impl Trait + 'a to for<'a> Fn(&'a i64) -> (&'a i64, String) to <Self as Iterator>::Item and everything in between. Recently, we extended the schema to expose generics information on top-level items (structs, enums, unions, traits, impls, functions) and on trait bounds: #514, #554, #560.
If that work is useful to you, I'd appreciate it if you'd consider sponsoring my work! I'd love to pay my rent with this work so I can spend more time doing it ❤

Also, not sure if I've already invited you to join the Trustfall / cargo-semver-checks community Discord — here's an invite link just in case: https://discord.gg/HGg95qSf

Thanks, seems I will be able to parse generics from the raw type field using the latest version. Feels a bit hacky but that will do for now.

Neat, good luck!


Trust: should the dialog talk about "windows" and not "workspaces" in empty window

Testing #124991
Here I had an empty window opened but the dialog talks about workspaces:

Workspaces is the correct term here because the decision applies to any workspace. Not just empty windows.
I could see "... all windows and workspaces" to be generic. please let me know if you think that's worth it. otherwise, closing for now.


feat: add debug and progress messages

This commit set progress and message for rtti recover script. The analysis for some binaries can take hours, so we need a better progress indicator..
The setProgress and setMaximum had to be added inside the loops because functions called inside them reset the progress.
The messages are based on the methods comments.

I'm in the middle of a big refactor of parts of this PR so cannot accept at this time. I will add similar messaging to the code once the refactor is done. Thanks for your interest in this very prototype script that needs a lot of cleanup.

I'm in the middle of a big refactor of parts of this PR so cannot accept at this time. I will add similar messaging to the code once the refactor is done. Thanks for your interest in this very prototype script that needs a lot of cleanup.

I understand, thank you !


[x10] Number::ordinal() Values Adjustment

After seeing this new implementation I found that we could improve some things, based on the fact that I have a package that is intended to convert number values into letters.
Adjust the ordinal method as follows, since not all languages handle the single English standard and the output type must be defined.
Number::ordinal(2);
// "2nd"

Number::ordinal(2, Number::ORDINAL_DEFAULT);
// "Second"

Number::ordinal(2, Number::ORDINAL_MALE, "es");
// "Segundo"

Number::ordinal(2, Number::ORDINAL_FEMALE, "es");
// "Segunda"

See related documentation: https://rmunate.github.io/SpellNumber/usage/numbers-to-ordinal.html

No plans to implement this atm.

No plans to implement this atm

@taylorotwell OK, thank you very much, engineer. I hope to contribute something valuable later


Separate Docker image with tags for postgres and Cockrockdb

Your Environment

Nakama: X.X.X
Database:  postgres
Environment name and version:
Operating System and version: truenas nas scale Debian
https://www.truenas.com/docs/scale/scaleuireference/apps/docker/

Unable to use docker compose file with the truenas scale docker system. Default image pull from docker is the cockroachdb yml build. Is it  possible to have these tagged separately?

The Nakama Dockerfile does not specify any database relationship at all, so I'm not sure this issue report is fully valid.
If you're using docker-compose (not to be confused with docker, the two are of course related but not exactly the same thing) and want to use Postgres then please use the appropriate YML config file: https://github.com/heroiclabs/nakama/blob/master/docker-compose-postgres.yml


[Spotify Player] ...

Extension – Spotify Player
Author: @mattisssa

Description
Steps To Reproduce

Open Raycast
Search for Menu Bar Player
Hit enter.=

The current behavior
Menu bar player not popping up.
The expected behavior
Menu bar player should be popping up.
Raycast version
Version: 1.41.1

It was working until I updated to Ventura


Update utils.py

Before submitting
Please complete the following checklist when submitting a PR:

[ ] All new features must include a unit test.
If you've fixed a bug or added code that should be tested, add a test to the
test directory!

[ ] All new functions and code must be clearly commented and documented.
If you do make documentation changes, make sure that the docs build and
render correctly by running make docs.

[ ] Ensure that the test suite passes, by running make test.

[ ] Add a new entry to the .github/CHANGELOG.md file, summarizing the
change, and including a link back to the PR.

[ ] The PennyLane source code conforms to
PEP8 standards.
We check all of our code against Pylint.
To lint modified files, simply pip install pylint, and then
run pylint pennylane/path/to/file.py.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context:
Description of the Change:
Benefits:
Possible Drawbacks:
Related GitHub Issues:

i tried to type in most of the changes that i saw sugested here: https://github.com/XanaduAI/pennylane/pull/535
should i change my approach?


[REQUEST] Timers as ISO 8601

Is your feature request related to a problem? Please describe.
https://www.schema.org/Recipe uses their Duration type for things related to time. That type uses ISO8601. It would be nice to get the values as such to be able to convert to schema.org in renderers.
Describe the solution you'd like
A getter for it would be nice
Describe alternatives you've considered
Apart from parsing it myself none.
Additional context
I guess this may need a spec change to cooklang itself as well.

I think until the spec changes we will pause this.


Ingredient Mappings

METHOD <init> (Lnet/minecraft/class_1799;)V ARG 1-> stack
stackArray -> matchingStacks (stackArray is not descriptive of the purpose of the variable)
createStackArray -> cacheMatchingStacks (this one calculates the matching stacks once to be used multiple times)
element -> jsonElement (element doesn't say a lot)
getStackArray->getMatchingStacks
METHOD test (Ljava/lang/Object;)Z -> matches (This one is similar to the IIngredient#matches method in CraftTweaker.)
ARG 1 itemStack

Conflicts and needs another review please.


Update WDTK to 0.11.1

Closes #2526.
The build will initially fail because it will take some time for the artifacts to be published on Maven Central, we can just re-run the build before merging.

@wetneb : I'm curious, so we now have a v3.3.1, and we should use it and test it?
P.S. if I can help with the wiki, the testing or else, email me, I'll help.
Regards, A.

We don't have a 3.3.1, but now we have snapshots every time we push something to the master branch. Please do use them! The more we use them, the more likely we are to find bugs that need fixing before a stable release.
https://github.com/OpenRefine/OpenRefine-nightly-releases/releases

I am personally just going to continue constantly building / running Trunk to ensure consistency.
I may occasionally be required to download a snapshot to try the Windows Portable version once we have that, of course.


Standalone msi installer for windows

Several people are asking about a standalone windows msi installer. I couldn't find a relevant issue, but feel like this has been discussed (perhaps on slack?).

Dup of #343 and #413


Feat/CIV-2649-update camunda changes to send notification to solicitor one

LRvLR: 1v2 different sol - Camunda changes.
camunda changes for sending notification to solicitor one
JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-2649
Change description
LRvLR: 1v2 different sol - Camunda changes.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

closing PR


[Program]: Write a Swift program to convert string to reversecase

Description
Write a Swift program to convert string to reversecase
Reversecase is a style of writing in which all lowercase letters are converted to uppercase and all uppercase letters are converted to lowercase.
Input  : "hello world"
Output : "HELLO WORLD"

How to contribute

Save the solution in program/program/convert-string-to-reversecase/ConvertStringToReversecase.swift file

!assign

!assign


[BUG] Trying to use docker integration but get [2021-12-06 09:05:34.734 UTC+0] [ERROR] Cannot read properties of undefined (reading 'split')

Deployment details:

App version [e.g. v1.7.4]: 2.1.1
Platform [e.g. amd64, arm64, arm/v7]: amd64
Docker image tag [e.g. latest, multiarch]: latest

Bug description:
Trying to define a bunch of things with labels but I seem to be upsetting flame, which makes me sad.

Steps to reproduce:

Deploy Flame with this docker-compose. I use tecnativa/docker-socket-proxy for security:

version: '3.6'

services:
  flame:
    image: pawelmalak/flame
    container_name: flame
    volumes:
      - /home/flame/data:/app/data
    networks:
      - web
      - default
    secrets:
      - password # optional but required for (1)
    environment:
      - PASSWORD_FILE=/run/secrets/password # optional but required for (1)
    labels:
      - "traefik.enable=true"
      - "traefik.docker.network=web"
      - "traefik.http.routers.flame.entrypoints=websecure"
      - "traefik.http.routers.flame.rule=Host(`flame.private`)"
      - "traefik.http.services.flame.loadbalancer.server.port=5005"
    restart: unless-stopped

  docker-socket-proxy:
    image: tecnativa/docker-socket-proxy:latest
    container_name: flame-docker-socket-proxy
    privileged: true
    volumes:
     - /var/run/docker.sock:/var/run/docker.sock:ro
    environment:
      - CONTAINERS=1
    restart: unless-stopped

# optional but required for Docker secrets (1)
secrets:
  password:
    file: /home/flame/secrets/password

networks:
  web:
    external: true

I am also running paperless with labels to add it to flame:
version: "3.4"
services:
  broker:
    image: redis:6.0
    restart: unless-stopped

  webserver:
    image: jonaswinkler/paperless-ng:latest
    restart: unless-stopped
    depends_on:
      - broker
    networks:
      - web
      - default
    healthcheck:
      test: ["CMD", "curl", "-f", "http://localhost:8000"]
      interval: 30s
      timeout: 10s
      retries: 5
    volumes:
      - /home/paperless/data:/usr/src/paperless/data
      - /home/paperless/media:/usr/src/paperless/media
      - /home/paperless/export:/usr/src/paperless/export
      - /home/paperless/consume:/usr/src/paperless/consume
    environment:
      PAPERLESS_REDIS: redis://broker:6379
      USERMAP_UID: 1111
      USERMAP_GID: 1111
      PAPERLESS_TIME_ZONE: Australia/Perth
    labels:
      - "diun.enable=true"
      - "flame.type=application"
      - "flame.name=Paperless"
      - "flame.url=https://paperless.private/"
      - "traefik.enable=true"
      - "traefik.docker.network=web"
      - "traefik.http.routers.paperless.entrypoints=websecure"
      - "traefik.http.routers.paperless.rule=Host(`paperless.private`)"
      - "traefik.http.services.paperless.loadbalancer.server.port=8000"

networks:
  web:
    external: true

Go to https://flame.private/settings and log in with the password
Go to the docker settings and set the Docker host to flame-docker-socket-proxy:2375
Head back to the homepage and you'll see the eternal loading graphic of doom
Check the flame logs to find something like this:
[2021-12-06 08:45:29.424 UTC+0] [ERROR] Cannot read properties of undefined (reading 'split')

What else have I tried?
I gave flame temporary access directly to the docker socket but still get the same issue.
I have tried with another application, but still get the same issues.
I'm quite interested in the idea of a label-defined dashboard, so I might poke into this issue on a future weekend.

I have the same error ^^

@enzofoucaud Can you share the details of your old server? What practices have you changed when migrating to the new server? Are there any new containers with funky labels you're running?

Okay, this is weird. I just added Flame labels to a couple of more containers and now it works..?

example ?

I had only Overseerr and Paperless-NG and was getting the error message. Then I added a weather API key and also Vikunja, TTRSS, and Home Assistant, and it started working.

Don't work form me .. I wait

For future people, I have Flame running for months, and today I noticed that it wasn't able to fetch apps. I got the same error as OP (by searching the error I came across this issue).
The problem was running Vikunja. I stopped that, and suddenly it works again.

Had same problem, as a workaround added flame tags to one of the containers, but I don't really want that container to be on the app list.
Seems like the problem manifests itself when some containers don't have flame tags set up.

Can confirm this seems to be a Vikunja + flame incompatibility.


⚠ Jenkins Plugins Artifactory has degraded performance

In 00f6488, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1205 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 5f93381 after 21 minutes.


Remove deprecated --delivery option from chef generate cookbook

This command has been deprecated and currently does nothing. We should remove it entirely in the next major release of DK / Workstation.

Nuked!


Support use of ByteBuffer for request-side of code-gen unary

A feature request for callback API to support certain use cases: when you want to send the same request message to multiple backends, you can just serialize it once and then send it as a separate request to each backend. Currently only providing this feature for unary; will consider other cases later if requested.

LGTM

@yang-g : this PR escalated quickly. So making ByteBuffer a first class citizen in the .grpc.pb.h files meant that SerializationTraits<ByteBuffer> would get pulled in by files in the codegen directory. As a result, ByteBuffer::operator= had to get moved out of src and into codegen. Additionally, we had to change to using from Status::OK to core_codegen->ok(). So far no problems, IMO, since we're already using core_codegen features in the implementation of ByteBuffer.
 The Library instantiator there didn't actually initialize the library, so there was nothing that was bringing alive the core_codegen now needed by ByteBuffer for the ByteBuffer microbenchmarks. So, this needs to get another review, IMO.
Thanks in advance!

@yang-g: Thanks for the prompt re-review!
@mhaidrygoog: Does this help?


process.env check outputs complete dump of environment settings

We are using a NPM package that depends on this package. When compiling to JS, I see that it contains all my environment settings with local paths, etc:
{....A LOT OF ENV SETTINGS...}.REFLECT_METADATA_USE_MAP_POLYFILL === "true"
If I look into following line 678, it looks like a coding error. It supposed to do a check, but in my case it outputs a complete dump of settings:
https://github.com/rbuckton/reflect-metadata/blob/3aeb98af4030be664a66f49bfd164936e0ba1825/Reflect.ts#L678C60-L678C60
I am using Windows. Am I right? Is this something that can be fixed?

Are you using a bundler? A bundler might be inserting the entire process.env object there, although it shouldn't. The code could also do !!process.env if that was a problem worth preventing.

This definitely sounds like something a bundler is doing, which is unfortunate because the point of the other checks on that line are to only test for the environment variable in a host that actually supports it.

The code could also do !!process.env if that was a problem worth preventing.

Are you indicating that !!process.env would bypass the bundler injecting an object here? It's already checking for the presence of env via &&:
const usePolyfill = typeof process === "object" && process.env && process.env["REFLECT_METADATA_USE_MAP_POLYFILL"] === "true";
//                                                 ^^^^^^^^^^^

So unless the !! is some kind of heuristic a bundler recognizes, I'm not sure it will be much help.
The only other thing I could think to do is do something like process["en" + "v"] or similar, assuming that's enough to bypass a bundler.

True - I would assume/hope that a bundler that is providing process would know that process and process.env are always an object, and thus the boolean would be replaced by true and then go through DCE.
I'd still love to know which bundler tho, since that sounds like a pretty bad bug.

I'm using webpack via nx and am also encountering this problem.
nx uses webpack's DefinePlugin to automatically load all envars starting with NX_ into process.env. As a result, if reflect-metadata is included in the bundle, all the the envars getting included in the bundler's output.
Granted this behaviour is not a bug; however, it does result in some undesirable behaviour (envars being included into your bundle). Given that process.env is used to see if polyfills should be included, could you expose an additional polyfill-less entrypoint (eg. import 'reflect-metadata/no-polyfills';)? This would also probably address https://github.com/rbuckton/reflect-metadata/issues/73


enable reliable broadcast-mode events by making queue names predictable

hm. not sure what i think about this. will try to find some time to consider properly
concerned about backwards incompat and "non-safe" default, but also more generally about having semantics that make sense. e.g. if we have distinct use cases for the current and this new mode of operation, maybe we should allow both. or is this "obviously" the way one would like to use broadcast?

This change doesn't prevent the "old" behaviour so in a sense it does allow both modes of operation. To summarise:
Old BROADCAST behaviour

reliable_delivery not possible (queues set to exclusive and auto-delete)
queue names appended with a uuid created on service start

Therefore any messages sent to a previous instance of the service are not seen by new instances (e.g. after a restart)
New BROADCAST behaviour

reliable_delivery is possible, but not mandatory
you can supply a differentiator callable to provide the value appended to the queue name
default differentiator uses socket.gethostname so queues will be reused between service restarts

It is therefore possible to use the old behaviour by passing uuid.uuid4 (or similar) as the differentiator and setting reliable_delivery=False.
I would argue that both modes of operation are valuable. Semantically, the old behaviour is "broadcast to all service instance processes", and the default new behaviour is "broadcast to all service instance deployments".
If we have to pick one the new behaviour is more useful, and we would want to keep the ability to provide a differentiator (for example if socket.gethostname didn't return something sane or useful) which basically you get the old mode for free anyway. Though we might choose not to document it.
This change is backwards incompatible in the sense that reliable_delivery in broadcast mode now defaults to True. Actually this brings it in line with the other modes, which also use reliable delivery by default. #104 has been open for some time to make us "safe" (i.e. non-reliable) by default, but I think this is a separate concern.

so have been thinking a bit about this, but not sure i can articulate my concerns very well
i think there are a few independent issues (which keep getting muddled in my head). an attempt
1: changing the default behaviour: i guess i'm not convinced that the new behaviour is "better enough" than the old one to change the default. esp. given that (#104) we might eventually end up going the other way on this. so maybe we it would be preferable to enable the new behaviour but leave the old default
2: i'm not so keen on this api for supplying a differentiator.
a) it's an argument that only applies/means something for certain combinations of other arguments
b) it feels a bit limiting, and would maybe be better exposed by the extension using a method to produce the queue name (which can be overridden by a subclass for your use-case, either in nameko or in your app/shared lib for your projects)
c) defaulting to gethostname can lead to strange behaviour if there are multiple processes running with the same service on the same host. i guess when you call it deployments there's an implicit assumption that this isn't the case

OK thanks. Let me try to address these concerns:

There are two things that are backwards incompatible here:

a) reliable_delivery for BROADCAST mode now defaults to True
b) the "differentiator" changes from uuid.uuid4 to socket.gethostname (so we go from process broadcast to deployment broadcast).

For 1.a, we could keep the old behaviour by mapping a handler_type to a default value for reliable_delivery. But would then have to explain why it's like that, and I don't think there is a good reason, except for maintaining backwards compatibility. We would want to remove the anomaly eventually, so all we're doing is postponing the backwards-incompatible change. We include #104 in these changes too, which would compress the two into a single backwards-incompatible release.
1.b can be solved by changing the default differentiator (or its equivalent) back to uuid.uuid4. We can definitely do this, but there's an argument that we're making things less useful for the sake of backwards-compatibility.

I understand all of these concerns. I will have a crack at an implementation using a method that can be overridden. This would be more in line with the nameko way of doing things. Being able to change the entire queue name is an interesting idea.

Regarding gethostname -- I wanted something that would be unique to the service instance but survive restarts, and for most people I figured hostname was a reasonable guess. It makes for nice readable queue names too ;) It won't always apply, but that's why you can specify it yourself.
It turns out that broadcast queues aren't exclusive, but they should be, which would also let us detect the situation where multiple service instances end up with the same broadcast queue name.

There's also hidden option number three: explicitly add PROCESS_BROADCAST and DEPLOYMENT_BROADCAST as separate modes. Although in the latter case you have to be able to provide a way to differentiate one deployment from another, since gethostname won't always be valid.

OK, this version is better I think. I've made the following changes:

Removed differentiator in favour of a broadcast_identity property, which can be overridden
Restored the default to a uuid that is created when the service starts
Added docs explaining this behaviour and some alternatives

The behavioural difference now is that reliable_delivery defaults to True in broadcast mode, and that we emit a warning in this case rather than raise an exception. Since we used to raise, there can't be any working code out there that will now behave differently.
I also made the broadcast queues exclusive, although it turns out you can't easily detect the rejected connection so I didn't bother going further.

as discussed, let's raise instead of just warning in the default implementation of broadcast_identifier if reliable_delivery is set to True.
This stops a naive user from shooting themselves in the foot by just enabling broadcast and leaving the rest as default, and also provides a loud warning to anyone upgrading about the backwards incompatible change (allowing them to explicitly set it back if they need)

:+1: modulo spelling


Bring fork up to date

(No issue, because I can't open issues on this repo)

merge conflicts @BatmanAoD

Replaced by #11


Exception handling for backup_volumes

Also got rid of some whitespace as I found it annoying

Will create a new pull request with a fix.

Silly me - did not notice that it updated.


Yield event handler, then return event handler misses second event

Describe the bug
When yielding events from an event handler, the final returned event is not processed.
To Reproduce
import reflex as rx

class State(rx.State):
    def handler(self):
        yield rx.console_log("foo")
        return rx.console_log("bar")

def index() -> rx.Component:
    return rx.fragment(
        rx.vstack(
            rx.button("Log Console", on_click=State.handler),
            spacing="1.5em",
            font_size="2em",
            padding_top="10%",
        ),
    )

app = rx.App(state=State)
app.add_page(index)
app.compile()

Expected behavior
When the button is clicked, the browser console should display "foo" then "bar".
Instead, we just see "foo".
Specifics (please complete the following information):

Python Version: 3.11.3
Reflex Version: main
OS: macOS 13.4.1
Browser (Optional): Firefox / Chrome

issue is also observed with non-special events defined on the State as well.
attempting to make the handler that yields into an async generator throws an error:
  File "/Users/masen/code/reflex-dev/repro-event-yield-return/repro_event_yield_return/repro_event_yield_return.py", line 12
    return rx.console_log("bar")
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: 'return' with value in async generator

It turns out that the return value of a generator is mostly thrown away!
See discussion here: https://stackoverflow.com/questions/34073370/best-way-to-receive-the-return-value-from-a-python-generator
I think this is probably a little known detail of python that will result in confusion as users mix yield and return haphazardly (like yield for state updates, and then ultimately return and event handler -- this will NOT work!)

Fixed in 0.2.3 via https://github.com/reflex-dev/reflex/pull/1390


How to evaluate on ohter dataset

hi, @ruotianluo, sorry to bother you.
I want to train and evaluate my own network on RSICD dataset. This dataset has dataset_rsicd.json file, which is similar to dataset_coco.json. However it doesn't has an annotation file like captions_val2014.json. Now i have successfully train my network, but how can i get BLEU/METEOR scores?
Thanks a lot!

https://github.com/ruotianluo/self-critical.pytorch/blob/master/scripts/prepro_reference_json.py


SyntaxError: The requested module 'vuepress' does not provide an export named 'defaultTheme'

Hi,
no matter what I am doing I get the error
SyntaxError: The requested module 'vuepress' does not provide an export named 'defaultTheme' when running npm install on my vuepress project.
I have tried NodeJs in the versions 16,20 and 22 everytime I get the same error.
My package.json looks like:
"dependencies": {
    "vue": "^3.4.29"
  },
  "devDependencies": {
    "@condorhero/vuepress-plugin-export-pdf-v2": "^2.0.1",
    "@vuepress/bundler-webpack": "^2.0.0-rc.13",
    "@vuepress/plugin-register-components": "^2.0.0-rc.34",
    "@vuepress/plugin-search": "^2.0.0-rc.34",
    "@vuepress/theme-default": "^2.0.0-rc.33",
    "sass": "^1.77.4",
    "sass-loader": "^14.2.1",
    "vuepress": "^2.0.0-rc.13",
    "yaml": "^1.10.0"}

Hello, Please in install the latest version of @condorhero/vuepress-plugin-export-pdf-v2(v3.0.3 now) and retry.
I notice you are using the old version.

It did work!
I had do to a little detour because I got:
file://..../node_modules/@vuepress/plugin-git/node_modules/execa/lib/utils/max-listeners.js:1
import {addAbortListener} from 'node:events';
        ^^^^^^^^^^^^^^^^
SyntaxError: The requested module 'node:events' does not provide an export named 'addAbortListener'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:123:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:189:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)
    at async loadUserConfig (file:///..../node_modules/@vuepress/cli/dist/index.js:77:18)
    at async CAC.serverApp (file:///..../node_modules/@condorhero/vuepress-plugin-export-pdf-v2/dist/shared/vuepress-plugin-export-pdf-v2.564f7dfd.mjs:136:42)

Which I do only state here for documentation purposes.
The Reason for this error is that "node:events" only works for the NodeJs version >= 16.9 (source)
Thank you.

Excellent.


Address GRC Bugs

See each commit for details.

/cc @JustinKuli @dhaiducek

/test unit-tests-sonarcloud

/unhold

/cherry-pick release-2.11

/cherrypick release-2.11


hadolint-sarif: init at 0.4.2

Description of changes
hadolint-sarif is a CLI tool to convert hadolint diagnostics into SARIF. It is a part of the sarif-rs group of projects
related to #266558, #266570, #266572, #266575
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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.


Introduce a hub for users on major protocols

Describe the feature request
An avenue for users to connect and address ongoing on-chain issues live.
Spam policy

[X] I verify that this issue is NOT SPAM and understand SPAM issues will be closed and reported to GitHub, resulting in ACCOUNT TERMINATION.

Stop spamming useless requests if you want to keep your account.


Create a rg-tag component

Some CSS tweaks and refactoring might be needed

@rockabox/qa over to you :+1: :)

Ready for merge to QA


metro-config should have an option to not populate watchFolders

What happened?
Sometimes you want to be more precise about what folders are added to watchFolders.  Currently metro-config always adds all packages within the whole repo.
A new option should be added to metro-config to opt out of the auto population of watchFolders
Affected Package
@rnx-kit/metro-config
Version
1.3.4
Which platforms are you seeing this issue on?

[X] Android
[X] iOS
[X] macOS
[X] Windows

System Information
N/A

Steps to Reproduce
N/A
Code of Conduct

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

I've just had a look and it turns out we already support this. If you set watchFolders in the config you pass to makeMetroConfig, it will override the auto-populated one. I've submitted a change to skip the auto-population to save some time since it was just discarded.


"snapshot for the following tabs are missing:" error on find-leaks

memlab find-leaks --baseline snapshots/Heap-baseline.heapsnapshot --target snapshots/Heap-target.heapsnapshot --final snapshots/Heap-final.heapsnapshot --snapshot-dir snapshots --work-dir workdir

Error: snapshot for the following tabs are missing:{
"1": {
"name": "baseline",
"url": "",
"type": "baseline"
},
"2": {
"name": "target",
"url": "",
"type": "target"
},
"3": {
"name": "final",
"url": "",
"type": "final"
}
}
The error is
Use memlab help or memlab <COMMAND> -h to get helper text
snapshot for the following tabs are missing:
baseline
target
final

There are three ways to specify inputs for the memlab find-leaks command:

--baseline, --target, --final specifies each snapshot input individually
--snapshot-dir specifies the directory that holds all three heap snapshot files (MemLab will assign baseline, target, and final based on alphabetic order of the file)
--work-dir specifies the output directory of the memlab run or the memlab snapshot command

Please only use one of the three ways to specify the input. I guess your command failed because --work-dir is not the auto-generated working directory of the memlab run or the memlab snapshot command.
Can you try this command instead:
memlab find-leaks --baseline snapshots/Heap-baseline.heapsnapshot --target snapshots/Heap-target.heapsnapshot --final snapshots/Heap-final.heapsnapshot
(Also double check that ls snapshots/Heap-*.heapsnapshot can list the files)

That command worked. Thank you.


[FLINK-18521][release] Add script for creating snapshot branch

Adds a script for creating the release snapshot branches (e.g., release-1.11).
Covers the git branch creation, updates to docs/_config.yml and flink-python/pyflink/version.py.
For testing:
export SHORT_RELEASE_VERSION=1.12
export RELEASE_VERSION=1.12.0

cd tools
./releasing/create_snapshot_branch.sh

I guess you would use this script rather with RELEASE_VERSION=1.12-SNAPSHOT instead of RELEASE_VERSION=1.12.0? Or am I misunderstanding something?

I guess you would use this script rather with RELEASE_VERSION=1.12-SNAPSHOT instead of RELEASE_VERSION=1.12.0?

That's a good point; I'll adjust it to use ${SHORT_RELEASE_VERSION}-SNAPSHOT where appropriate. I'd like this one to work without having to specify additional variables, if the variables from the release guide have been declared.

Actually, none of the fields are supposed to contain -SNAPSHOT, as seen in the release-1.10 branch / release guide. So you use the script as is without specifying  anything special, apart from what is written  in the release guide.

I'm not sure I understand what you are saying.
The script is fine, just the example that you provided confused me, because it created a "release-1.12" branch, where the version was set to "1.12.0" instead of "1.12-SNAPSHOT".
I think you can merge this as-is.

The script is fine, just the example that you provided confused me, because it created a "release-1.12" branch, where the version was set to "1.12.0" instead of "1.12-SNAPSHOT".

The properties shown in the example will be set as part of the usual release process (see the Create a release branch section in the guide.
As such, the example shows what will happen during the normal release process.
You would not use it like this: RELEASE_VERSION=1.12-SNAPSHOT

Okay, you are right. I assumed this script was also setting the version in the pom file. However, it doesn't (I should check the code I'm reviewing closer).
Sorry for the confusion. This is good to go.


Error: HandshakeException: Connection terminated during handshake

I am receiving the below error on Android
Error: HandshakeException: Connection terminated during handshake
This started after updating to Flutter 3.13.1. It will not let me sign into my app. It also will not let someone that is signed in authenticate with the saved credentials.
[✓] Flutter (Channel stable, 3.13.1, on macOS 13.5.1 22G90 darwin-arm64, locale en-US)
    • Flutter version 3.13.1 on channel stable at /Users/rickeylee/Desktop/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e1e47221e8 (8 days ago), 2023-08-22 21:43:18 -0700
    • Engine revision b20183e040
    • Dart version 3.1.0
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at /Users/rickeylee/Library/Android/sdk
    • Platform android-33, build-tools 33.0.2
    • ANDROID_HOME = /Users/rickeylee/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14E300c
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)

[✓] VS Code (version 1.81.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.70.0

[✓] Connected device (4 available)
    • SM F926U (mobile)            • RFCR802EMHY                              • android-arm64  • Android 13 (API 33)
    • rickey lee’s iPhone (mobile) • d0545eb2ff26e1c087d1b29c26f8e15da19284bb • ios            • iOS 16.6 20G75
    • macOS (desktop)              • macos                                    • darwin-arm64   • macOS 13.5.1 22G90 darwin-arm64
    • Chrome (web)                 • chrome                                   • web-javascript • Google Chrome 116.0.5845.110

[✓] Network resources
    • All expected network resources are available.

• No issues found!

I think this is an issue with flutter. Can you do a http request to any other website via the http package?

@Vinzent03 it might be I was digging in a bit more I will try to see if I can.

@Vinzent03 You are right I will close this and open a new issue under flutter


[SPARK-44837][SQL] Change ALTER TABLE ALTER PARTITION column error message

What changes were proposed in this pull request?
Improve the error message for ALTER TABLE ALTER COLUMN command on partition columns.
Why are the changes needed?
Currently, if a user executes ALTER TABLE ALTER COLUMN on a partition column, the error message provided is cryptic and does not help users. The improved error message makes it clear that the command is not supported for partition columns.
Does this PR introduce any user-facing change?

New output when the command is executed on a partition column.

org.apache.spark.sql.AnalysisException: [CANNOT_ALTER_PARTITION_COLUMN] ALTER TABLE (ALTER|CHANGE) COLUMN is not supported for partition columns, but found the partition column i in the table non-delta table spark_catalog.default.t. 	at org.apache.spark.sql.errors.QueryCompilationErrors$.alterTableChangeColumnNotSupportedForPartitionColumn(QueryCompilationErrors.scala:2586)

Old error message when the ALTER TABLE ALTER COLUMN is executed on a partition column

org.apache.spark.sql.AnalysisException: Can't find column i given table data columns [k]. 	at org.apache.spark.sql.errors.QueryCompilationErrors$.cannotFindColumnError(QueryCompilationErrors.scala:2843) 	at org.apache.spark.sql.execution.command.AlterTableChangeColumnCommand.$anonfun$findColumnByName$2(ddl.scala:490) 	at scala.Option.getOrElse(Option.scala:189) 	at org.apache.spark.sql.execution.command.AlterTableChangeColumnCommand.findColumnByName(ddl.scala:490) 	at org.apache.spark.sql.execution.command.AlterTableChangeColumnCommand.run(ddl.scala:447)

How was this patch tested?
All tests pass.

@cloud-fan could you review?

Seems like a related test failure:
[info] - Error classes match with document *** FAILED *** (209 milliseconds)
[info]   "... table `<tableName>`[.]
[info]
[info]   ### CANNOT_CAST_DA..." did not equal "... table `<tableName>`[]
[info]
[info]   ### CANNOT_CAST_DA..." The error class document is not up to date. Please regenerate it. (SparkThrowableSuite.scala:336)
[info]   Analysis:
[info]   "... table `<tableName>`[.]

+1, LGTM. Merging to master.
Thank you, @michaelzhan-db and @beliefer @holdenk for review.


Highlight recognized image

Hi, i wan't to highlight a recognized image through ViroARImageMarker and onAnchorFound by putting a rectangle hover the image, but ViroArPlane does not position itself correctly hover my image.
How can i achieve that?
I want the rectangle to be same size of my image. It's possible?
in swift you can achieve that with this simple lines of code
   let plane = SCNPlane(width: referenceImage.physicalSize.width,
                         height: referenceImage.physicalSize.height)
    let planeNode = SCNNode(geometry: plane)
    planeNode.opacity = 0.25

    planeNode.eulerAngles.x = -.pi / 2

    planeNode.runAction(self.imageHighlightAction)

thanks

Hi @Tamiyadd ,
Thanks for reaching out. Can you explain more what you mean by but ViroArPlane does not position itself correctly hover my image? Can you share your code snippet where you are trying this? Did you try getting the width, height & position, rotation of the Anchor given in onAnchorFound? Pasting our anchor object's docs for reference here https://docs.viromedia.com/v2.7.0/docs/viroarscene#anchor

Hi @Tamiyadd ,
Thanks for pointing that out. We should probably make it clear in our documentation. You are right in that anchor object returns width and size for ViroARPlane objects. In the case of ViroARImageMarker, onAnchorFound should give you the parameters for the detected image "plane" in the scene. Let us know if this works for your use case.
Thanks!

Unfortunately the info returned by the anchor object are not sufficient to place a ViroQuad hover the image :( or maybe I'm not able to do it
 {_.map(this.objectsArray, function(el) {
          const marker = el.marker;
          return (
            <ViroARImageMarker
              target={marker}
              key={marker}
              pauseUpdates={_this.state.animateQuad === marker}
            >
              <ViroQuad
                onHover={(isHovering, position, source) =>
                  _this.onHover(isHovering, position, source, el.variable, marker)
                }
                materials={['whiteColor']}
                position={[0, 0, 0]}
                width={el.width}
                height={el.height}
                opacity={0.09}
                rotation={[-90, 0, 0]}
                animation={{
                  name: 'animateQuad',
                  loop: false,
                  run: _this.state.animateQuad === marker,
                  onFinish: () => _this.finishAnimation(marker),
                }}
              />
            </ViroARImageMarker>
          );
        }

now i'm doing in this way but it's not reliable

Hi @Tamiyadd ,
Can you try using  instead of  to "highlight" the recognized image? That would be more reliable. Your code would looks something like below
<ViroARImageMarker
              target={marker}
              key={marker}
              pauseUpdates={_this.state.animateQuad === marker}
              onAnchorFound={this._onAnchorFound}
            >
              <ViroImage
                       height={this.state.height}
                       width={this.state.width}
                       position={this.state.position}
                       rotation={this.state.rotation}
                       source={{your image uri here}}
                        ...whatever else you might need here
                         .../>

            </ViroARImageMarker>

_onAnchorFound(anchor) {
    this.setState({
      width: anchor.width,
      height: anchor.height,
      position: anchor.position
    });
}

As you see above, you'll rely on the width, height, position, rotation of the anchor passed in onAnchorFound(anchor) to set the correct height, widht, position of the image.

there something strange, how can this work if the onAnchorFound anchor in ViroARImageMarker does not return width and height? :(

Hi @Tamiyadd,
Sorry for the delay in responding(the team was away for a couple of conferences last week). I confirmed in our own implementation on ViroARImageMarker.  Currently the way that the ViroARImageMarker works is that the developer knows the physical width/height of the marker and provides it as a parameter into the component itself (https://docs.viromedia.com/docs/ViroARImageMarker). As such, it is assumed that the developer knows the physical width/height (in meters) of their marker in the real world and the platforms (ARKit/ARCore) simply return the position (center) of the marker.
Apologies for the incorrect guidance earlier. You can get around the onAnchorFound not returning width/height by using the measured width  / height parameters in meters along with the position of the center returned in the callback.
Hope this helps. Let us know if this still doesnt work for you.

solved using physical size to set width/height


Fail to run with polars 0.19.14

When running with polars 0.19.14 getting error:
pyo3_runtime.PanicException: called Result::unwrap() on an Err value: DlSym { desc: "/home/user/somecode/polars_ds_extension/python/polars_ds/_polars_ds.cpython-310-x86_64-linux-gnu.so: undefined symbol: _polars_plugin_get_version" }

When running with polars 0.19.14 getting error:
pyo3_runtime.PanicException: called Result::unwrap() on an Err value: DlSym { desc: "/home/user/somecode/polars_ds_extension/python/polars_ds/_polars_ds.cpython-310-x86_64-linux-gnu.so: undefined symbol: _polars_plugin_get_version" }

I am aware of this. Polars 0.19.14 was just released and depends on Rust Polars 0.35 and I haven't made the update.Current I am stuck with some overtime work and I will  likely update later tonight

https://github.com/abstractqqq/polars_ds_extension/pull/18


Seg Fault when handling mid-call REGISTER for call using tcp

Hi Dave.
When you have a chance, can you please take a look at the logs that I've attached for a crash that is occurring during the handling of a REGISTER when the webrtcproxy is configured to use tcp for outgoing INVITE's? This only happens when the REGISTER comes in for a user that is on an active call. REGISTER's when there is no active call are succeeding.
I've tried forcing the REGISTER to use tcp, but that ends up hitting the seg fault even when there is no active call.
If there is any other info that may help, let me know. Thanks!
coredump_drachtio.txt
crash_tcp_midcallRegister.txt
register_ForceTcp.txt

what version are you running?  Also, are you running in docker or a host machine?

I'm running v0.8.9-rc2-2-g0004a185c on a host machine (logs are from my Mac, but I originally encountered the issue in our linux environment).

the stack trace wasnt all that helpful.  Could you try running drachtio server from within gdb (or lldb in mac) and when it crashes get the stack trace from there?

@rhondahollis can you test the branch bugfix/issue-42 ?  I believe it should fix this issue.
Note: be sure to update submodules, as the fix is in a newer version of sofia-sip
i.e. after fetching that branch do
git submodule update --init --recursive

@davehorton This issue has been resolved with your bugfix/issue-42 branch. I am also only seeing two tcp connections created now for the SIP traffic. This looks like it is all working well, but if you can take a peek at the attached log to confirm the tcp connections are behaving the way you expect them to, that would be great. Thanks!
These are the two connections that I see now:
tcp        0      0 drachtio01-dev-s0:35197 <IP_ADDRESS>:sip       ESTABLISHED
tcp        0      0 drachtio01-dev-s03:sip  <IP_ADDRESS>:8309      ESTABLISHED
drachtio_tcpFix.txt

Thanks, this has now been merged into main and tagged as v0.8.11-rc1.
That log looks clean to me, and as expected.  Thanks for finding and helping me fix a rather important bug!


Co-Speakers do not get Email Invitation

Problem and impact
When a speaker enters the email of a co-speaker into the sign-up form the co-speaker does not get an email.

How to regenerate the issue?

I tried replicating this, but my co-speaker got the invitation email


Spark/passing spark conf parameters to OL client

Problem
Whenever we want to set up new type of consumer of spark lineage messages we first have to add it in OL client and then we have to have all the parsing logic inside spark integration.
Closes: #1331
Solution
With new mechanism Spark will only pass on the values of parameters defined in its configuration, then the parameters will be parsed out inside the client, that way adding new consumer will be transparent to the integration.
The way its implemented:

the parameters are filtered for having a spark.openlineage. prefix
Json structure is build from the parameters, the ones with transport.urlParams and transport.properties prefixes are assumed to be map structures with dot separated keys, the ones with ; inside the value are treated as arrays.
Json structure is parsed to OpenLineageYaml (new name will be necessary) which contains FacetConfig and class implementing TransportConfig

Note: All schema changes require discussion. Please link the issue for context.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[x] Your changes are accompanied by tests (if relevant)
[x] Your change contains a small diff and is self-contained
[x] You've updated any relevant documentation (if relevant)
[x] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2022 contributors to the OpenLineage project

Codecov Report

Merging #1383 (0c9cb44) into main (db17a57) will decrease coverage by 3.09%.
The diff coverage is 70.83%.

@@             Coverage Diff              @@
##               main    #1383      +/-   ##
============================================
- Coverage     68.15%   65.06%   -3.10%
- Complexity       74       80       +6
============================================
  Files            86       20      -66
  Lines          3925      332    -3593
  Branches         25       27       +2
============================================
- Hits           2675      216    -2459
+ Misses         1216       86    -1130
+ Partials         34       30       -4

Impacted Files
Coverage Δ

...a/io/openlineage/client/transports/HttpConfig.java
12.50% <0.00%> (-1.79%)
:arrow_down:

...penlineage/client/transports/KinesisTransport.java
47.36% <0.00%> (ø)

.../io/openlineage/client/OpenLineageClientUtils.java
56.86% <25.00%> (-2.72%)
:arrow_down:

...o/openlineage/client/transports/HttpTransport.java
79.38% <100.00%> (+5.97%)
:arrow_up:

...o/openlineage/client/transports/KinesisConfig.java
83.33% <100.00%> (ø)

client/python/openlineage/client/constants.py

...flow/openlineage/airflow/extractors/dbapi_utils.py

...rflow/openlineage/airflow/extractors/extractors.py

...ow/openlineage/airflow/extractors/sql_extractor.py

integration/common/openlineage/common/models.py

... and 59 more

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


chore: replace refine' with refine

A few regular expression replacements to remove ~40% of the remaining refine's
Also see #11884

I'm running the linter manually, and will fix the few ERR_LIN errors...

see also discussion here:
https://leanprover.zulipchat.com/#narrow/stream/287929-mathlib4/topic/reviewdog/near/434389568


http2_keep_alive_interval doesn't work

Bug Report
Version
master branch (da482359933f52e84c0263b28a5a83ab1efe6c33)
Platform
$ uname -a
Darwin AkiranoMacBook-Pro.local 23.1.0 Darwin Kernel Version 23.1.0: Mon Oct  9 21:27:27 PDT 2023; root:xnu-10002.41.9~6/RELEASE_X86_64 x86_64

Crates

tonic
tonic-build

Description
I made a reproducer.
https://github.com/akiradeveloper/tonic-hyper10-debug
the test fails with this message
thread 'main' panicked at /Users/akira/.cargo/registry/src/index.crates.io-6f17d22bba15001f/hyper-1.3.1/src/common/time.rs:37:17:
You must supply a timer.

I expect it to complete without error because it was okay with 0.11.0.

+1,
Seems to be coming from here: https://github.com/hyperium/hyper/blob/master/src/common/time.rs#L36
I think it can be fixed via https://docs.rs/hyper-util/0.1.5/hyper_util/server/conn/auto/struct.Http2Builder.html#method.timer after this line: https://github.com/hyperium/tonic/blob/da482359933f52e84c0263b28a5a83ab1efe6c33/tonic/src/transport/server/mod.rs#L550
However, I think it also requires an implementation of: https://docs.rs/hyper/1.3.1/hyper/rt/trait.Timer.html

Oh, there's one already made: https://docs.rs/hyper-util/0.1.5/hyper_util/rt/tokio/struct.TokioTimer.html

I'll raise a PR with a fix tomorrow if nobody else gets to it first.


Option to use native Kubernetes load-balacing at global level

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Currently, Traefik forwards incoming traffic directly to pods, bypassing native Kubernetes services. However, when utilizing service meshes to manage cluster traffic from Ingress to pods, it is necessary to forward the traffic to meshed Kubernetes services. Currently, the option to achieve this behavior involves setting nativeLB at the Ingressroute level. It would be beneficial if there were an option to specify this at the provider level, especially when dealing with large production clusters, eliminating the need to consistently set this parameter at the route level.

Hey @prajithp13,
Thanks for your suggestion.
We are interested in this issue, but we’re unsure about the use case and the traction it will receive. We are going to leave the status as kind/proposal to give the community time to let us know if they would like this idea.
We will reevaluate as people respond.
Conversation is time-boxed to one month.

This feature might be required only for those who use Service Mesh with Traefik. Can we consider this?

Hey @prajithp13,
IMO it can be useful for other use cases. For instance, when the Kubernetes cluster has many pods starting and shutting down, using the native load balancer allows Traefik to refer to the K8S Service IP instead of the the pods Ip and thus avoid 502 errors (we have already helped people with such an issue).


Make notes read from data

At the moment, the notes on the home page show dummy data. They should be changed to read data from whichever kind of localstorage is selected

The Hive package has been selected to store data

This has now been completed


Make a cheat sheet with commands

just like docker-katas
Could be inspired by:
https://github.com/tolitius/git-cheat-sheet/blob/master/git-cheat-sheet.markdown

Possibly also re-use some things done already and do something extra, real handy stuff etc. By ready made meant, for example, https://github.github.com/training-kit/downloads/github-git-cheat-sheet.pdf , https://education.github.com/git-cheat-sheet-education.pdf , https://www.atlassian.com/git/tutorials/atlassian-git-cheatsheet , https://about.gitlab.com/images/press/git-cheat-sheet.pdf , https://gist.github.com/5310/01a41f21eb62d6ca391f etc.

I see that the git cheatsheet is already good or extensive enough in the Cheatsheet secion on the git-katas home page - https://github.com/praqma-training/git-katas.
Is it about facelifting the way we want to present this cheatsheet or is something still missing?

Seems like an issues that has not been propperly updated, by quick check.
Have you seen if some of the links @mdrocan have linked to has more to add @schiluveri ?

Honestly, I think the value of this whole project lies in the katas.  The cheat sheets to be found online are good enough.  Referring to them should suffice.


Move plugin specific options under pluigin settings

Summary of the Pull Request
What is this about:

What is include in the PR:

Added AdditionalOptions property to ISettingProvider. List of options(checkboxes).
Render plugin specific options in plugin settings
Move disable drive detection warning to the indexer settings

How does someone test / validate:

Delete PowerToys Run\settings.json
Check that Disable drive detection warning for the file search plugin is under Indexer settings

I just noticed that the indexer plugin has the wrong name. It is probably an issue with resources not in the scope of this issue.
Quality Checklist

[X] Linked issue: #9655
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Can we structure this a bit mor?
Activation

global results
activation phrase
shortcut (comming later)

Additional settings

indexer

I just noticed that the indexer plugin has the wrong name. It is probably an issue with resources not in the scope of this issue.

@mykhailopylyp
I think there is a mistake in line 34 of main.cs from indexer plugin.
Instead of Plugin.Indexer.Name it should be Plugin.Indexer.Plugin.Name, I think.

Can we structure this a bit mor?
Activation
* global results
* activation phrase
* shortcut (comming later)

Additional settings
* drive warning

We can consider it. I am a little bit worried that Additional settings section would not look good only with one checkbox. @enricogior, what do you think?

Can we structure this a bit mor?
Activation
* global results
* activation phrase
* shortcut (comming later)

Additional settings
* drive warning

We can consider it. I am a little bit worried that Additional settings section would not look good only with one checkbox. @enricogior, what do you think?

I am not sure if the title "Activation" is a good idea.
If we don't want to have the title we should at least sort the settings in groups.

@htcfreek
It is not a good idea to have Plugin settings inside of plugin settings. Maybe Additional settings is a better option here. On the other hand, we introduce Additional settings inside of the collapse section and I think that the collapse section should be simple and flat(Maybe I am mistaken here).
Anyway, here how it looks without any headings but kind of with grouping. Indentations are not ideal.

@mykhailopylyp @htcfreek
I like the idea of Additional settings since it makes very clear the distinction between the activation settings and the plugin's specific settings.
Some plugin may have several additional settings so it's a good idea introduce it.

@mykhailopylyp
Regarding the wording of "Plugin Settings" or "Additional settings" I agree with you.
"Additional settings" is the better choice.
Regarding the grouping I would split between "action phrase" and "drive detection warning". My intention is that we have general settings (global results, activation) and additional settings like the drive warning.

@mykhailopylyp
Can you please update the screenshot. Thank you.

@mykhailopylyp
can you rebase the branch?

@htcfreek

Btw: We should add a work item like "Optimize layout and margins. And swap Plugin group with Appearance group." to the parent issue. (Doing this first makes sense if all ux elements are added.)

We already have an issue for it #9653

@enricogior @htcfreek
Before merging we should consider https://github.com/microsoft/PowerToys/issues/5117#issuecomment-716742089. The drive detection field is going to have complex behavior. Our current approach is not good for that. Also, I do not want to introduce "UI in code specification".

@htcfreek

Btw: We should add a work item like "Optimize layout and margins. And swap Plugin group with Appearance group." to the parent issue. (Doing this first makes sense if all ux elements are added.)

We already have an issue for it #9653

Sory. I meant #9653.

@enricogior @htcfreek
Before merging we should consider https://github.com/microsoft/PowerToys/issues/5117#issuecomment-716742089. The drive detection field is going to have complex behavior. Our current approach is not good for that. Also, I do not want to introduce "UI in code specification".

If we rework this setting, we could consider adding a link to settings app.

@mykhailopylyp
is there anything in this PR that you consider a must have for future changes to implement https://github.com/microsoft/PowerToys/issues/5117#issuecomment-716742089? If not we can merge it and as @htcfreek suggested, work on that in another PR (that is not high priority right now).

@enricogior

is there anything in this PR that you consider a must have for future changes to implement #5117 (comment)?

No, I can not think about anything. It just seems that the current approach with additional plugin settings would be completely rewritten for #5117. And it would become a large cost item with this PR.
#5117 will not be so hard to do if it is not under index plugin settings.

@mykhailopylyp
for #5117 we can reconsider the approach:

add a link to the Windows Settings
add a link to the online documentation
We don't have to over-engineer the PT settings to explain every detail there.
And in case we decide we have to over-engineer, we move it outside of the plugin settings, if that is a better solution.


Restrict the user login

Some times, we need to disabled one user login system.
Whether or not add a new field restrict the user login.

Thanks your feedback!

+1

Add a field "active" on the user

Merge this thread to https://github.com/gogits/gogs/issues/2937 which has implemented this feature.


Final changes from paper plot edits

Change Description
Changes that fix all plotting scripts, that allows replication of paper plots.
Also updated numpyro sampler to sample multiple light curves in series really really fast.
Also adds SNANA file reader file.
Code Quality

[x] I have read the Contribution Guide
[x] My code follows the code style of this project
[x] My code builds (or compiles) cleanly without any errors or warnings
[x] My code contains relevant comments and necessary documentation

Recommend doing a "squash and merge" for this PR, since there are so many intermediate commits.


unicode/utf8: add some methods to handle strings

Solved the problem of getting the correct index for non-English.
Solved the problem of non-English getting substrings.

@googlebot I signed it!

Closing per the comments on the CL.


[Feature request] Simpler usage of Ctrl + L

Hello folks,
It's been a while since I used and enjoyed mitmproxy last time, and seems like many things are changed. The changes are mostly awesome, and I always thank you guys for building this great project :)
But, IMHO, we need more helpful documents for the new features.. For example, when I tried to copy a url from a flow view. Ctrl + L was somewhat confusing, and I guess it would be difficult for beginners, too. The help page only shows me that ctrl l is to send cuts to clipboard, and when I typed the ctrl l it shows me cut.clip [flow] [cut]. I think [flow] and [cut] are not general terms, so we need more explanations or examples for them, how to type the values. It was very difficult for me to figure out what the [flow] and [cut] mean. So, I read your docs. Since your documentation page don't provide a search option, so I just read the docs pages one by one, and I found what the [focus] means and what the right value is. And, I searched and read github issues to understand what the [cut] means, and I finally guessed the value based on an example in one of github comments. So, now I'm using this command cut.clip @focus request.url.
I also want to claim that maybe we can reduce the number of typed characters to copy a url. I cannot understand why we have to type @focus even in a flow view. I think we can omit the @focus because we already select a flow. Maybe we can also omit request.. Typing just url is not ambiguous because only request has a url.

Hi there! Thanks for taking the time to write this up - it's excellent user feedback. I have a few things coming down the pipeline to improve usability here.
First, In #2650 we're discussing a help specification for key bindings. Key bindings are now all expressed in terms of a lower-level command language. While this is powerful, expressive and flexible, it's not always clear what commands (especially compound commands) do by inspection. To address this, we'll have short help text for each keybinding that can be specified for built-ins and for user bindings. This help text will be displayed when viewing contextual help (with a toggle to see underlying commands).
Second, in #2718 we're discussing adding Actions, which you can think of as unbound key bindings, that users can select from a palette. These will have the same help mechanism as key bindings, and let us put together easy-to-use user actions expressed in the command language.
Finally, @focus is a flow specification, and can be something different from the current focus. This needs to stay in the command language to keep things explicit and predictable. We might not have to display this to the user on the command prompt, though - so we can imagine a key binding that requires an argument also being able to over-ride the command-prompt.
I think the suggestion that url could be a shortcut for request.url is worth considering. The current specification is a shortcut for the way you would access flow attributes in code, where "flow.request.url" gives you the URL, but just "request.url" will fail.
Thanks again for the detailed feedback.


support inline math rendering

single dollar

Currently, two dollar is (too big) inline, which is even worse.
One dollar should also force a newline break...
So:
Something $f(x) = x$
should get translated into
Something ![img]
Something $$f(x)=x$$
should get translated into
Something

![img]

We also need to fix the CSS for that, I think? This requires a weekend day of time...


package.json has broken "main" field

When you npm install the package as is and use require(''gl-matrix) this library will not work in nodejs anymore because at some point the main field in the package.json was changed to dist/gl-matrix where it should be src/gl-matrix.
You dont want to use the webpacked version when you use node.
Please correct this because otherwise nobody can use this package anymore

(Relevant issue: https://github.com/toji/gl-matrix/issues/277 )
@andrevenancio You probably know more about this than I do.

Hi @Fuzzyma, you are right, the main property in the package.json points to the "compiled" version. I don't know why module was added. It was introduced by @toji with this commit
From what I can ascertain the module property is meant to be used by webpack and rollup as ES6 modules (allowing three shaking).
With that said, I just want to point out that the exported version on the dist/ folder (dist/gl-matrix.js) though it has some comments introduced by webpack, its not a webpack version. Webpack is exporting with the UMD option which exposes the library under all the module definitions, allowing it to work with CommonJS, AMD and as global variable.
We can remove the comments from that version (and we should @stefnotch ) but still the only thing that's happening is that you're merging all the files into one.
Unless I'm missing something this should not be a problem.

@Fuzzyma  Can you pull my version https://github.com/toji/gl-matrix/pull/312 and give it a go on your node app?
I've tested this locally with something very simple, and it seems to work
test.js
var vec3 = require('gl-matrix').vec3;
console.log(vec3.create());

and running
node test.js

returns the expected Float32Array [ 0, 0, 0 ]
Give it a go too @stefnotch

the windows was were my code was breakig. However this was just reported to my repo. I cannot test it atm but I am pretty sure it should work now. I will report back when my people close the issue on my repo


Using kind with a cluster not named kind results in image pullbackoff

I found using kind and naming the cluster something other than the default that kind uses : kind results in ko apply pods failing to start, erroring with ImagePullBackoff.
Likely related to this comment: https://github.com/google/ko/pull/180#discussion_r469332236

/cc @markusthoemmes

The workaround is to set KIND_CLUSTER_NAME.
I would expect to use the kind cluster that is currently being used by the kubecontext.


bug(radio): updated radio.ts to add support for ngModel on md-radio-group

Added support for ngModel on md-radio-group
Updated Documentation
Added Demo App Example

This should fix #208 that I submitted earlier.

Changes seem good, just needs unit tests now.

I have cleaned up the code as suggested and added a unit test.

LGTM aside from one more unit test.
cc @rfru

@doctorrustynelson forgot one last thing: can you squash your commits into one? The commit message should be something like
feat(radio): support ngModel

Thanks!

@jelbourn Thank you for all the suggestions; I have done as requested and squashed the commits.

LGTM

Thanks for your contribution!


Dev-tools using "sed" command failing on Mac M1

Describe the Bug

On a Mac M1 (maybe in general on a Mac) some dev-tools fail due to a different set of arguments of the "sed" command.
Steps to Reproduce

Install the cms on a Mac (M1).
Run ./dev-tools/run.sh.
See error

Additional Information
The following error is shown:
processing locale de
sed: illegal option -- -
usage: sed script [-Ealnru] [-i extension] [file ...]
        sed [-Ealnu] [-i extension] [-e script] ... [-f script_file] ... [file ...]

there is at least a temporary fix that you can use gsed instead of sed.
-> See: https://gist.github.com/andre3k1/e3a1a7133fded5de5a9ee99c87c6fa0d

In the meantime, we implemented various strategies to set up the project, ensuring smooth compatibility with the new MacBook chips. #2732 #2730


Allow for HTML in breadcrumbs.

Updated to permit the change made here: https://github.com/parameter1/base-cms/pull/209
EDIT: This also needed to be done in the theme-monorail package.

 Html tags on Webcast page title


feat(extension): migrate chrome extension v2 to v3

update chrome extension package.json
add dist/output.css
migrate chrome extension v2 to v3

Great! Thank you very much!


StackStorm with Hubot adapter for botframework

Microsoft has posted a Bot Framework Hubot adapter[1]. Have anyone manage to actually get hubot connected with MS teams?
I am looking forward StackStorm integration with MS team using Bot Framework Hubot adapter.
[1]. Hubot adapter for botframework: https://github.com/Microsoft/BotFramework-Hubot

Here is discussion about Hubot for Microsoft Teams: https://github.com/hubotio/hubot/issues/1260.

Initial support for ChatOps response/reply is incoming in StackStorm 3.0. Adding end-to-end tests with MS Teams and supporting chatops.post_message should be complete in StackStorm 3.1.

Just a confirmation update that Microsoft Teams was indeed added in StackStorm v3.0:
https://stackstorm.com/2019/04/29/its-here-stackstorm-3-0/
Feel free to use it and share your feedback or issues.


Mapfish print module : sending a POST request, then changed in GET ?

Hello everyone,
I have an issue with the size of my print request.
I send a POST http request because I know that the spec parameter can be
really big (I have a custom parameter CQL_FILTER to filter some layers to
print). The POST request is well sent with "/geoserver/pdf/create.json".
When I check the generated PDF, I get an empty map with the following error
near the zone of the map : "Error java.io.IOException : Error (status=414)
while reading the image from http://.......(the rest of the big url)...".
When I check the log file of GeoServer, I see the same error : "ERROR
[print.PDFUtils] - Server returned an error for http://.....(the rest of the
big url)... : Request-URI Too Large"
I understand the error, URI too long, but why my POST request is finally
converted in GET request to retrieve the image of my report ? Why doesn't it
stay in POST request ? Do I have to configure something in the print plugin
?
Any help appreciated.
Kind regards,
JCB

You POST request is sent to GeoServer, then it's GeoServer choosing to do a GET to MapFish Print. Therefore, this ticket should be open against GeoServer, not MapFish Print. I'll close it, here.

It's funny, in GS they had closed this issue for the same reason: https://osgeo-org.atlassian.net/browse/GEOS-8480?focusedCommentId=78596&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-78596


chmod 0600 tests/data/dcos.toml

Otherwise you get errors when running integration tests:
> /Users/abramowi/dev/git-repos/dcos-cli/cli/tests/integrations/test_config.py(336)_get_value()
-> assert returncode == 0
(Pdb) stderr
b"Permissions '0o644' for configuration file '/Users/abramowi/dev/git-repos/dcos-cli/cli/tests/data/dcos.toml (pointed to by tests/data/dcos.toml)' are too open.
File must only be accessible by owner. Aborting...

Cc: @tamarrow

Can one of the admins verify this patch?

We chmod in make test and not in this file, because we would need to chmod for all test files :)

Oh I can probably move this to a fixture in conftest.py then and it would be used no matter what tests are run.
See http://pytest.org/2.2.4/plugins.html#conftest-py-plugins

We are in the process of switching tests instructions / CI setup to use the dcos cluster setup command (#1036), so soon enough it won't be necessary anymore to chmod that file.


Add /annotation enpoint

Add an endpoint to list, search, create, modify, and delete annotations according to a subset of the Web Annotation Data Model.

GET /annotation list and search annotations with optional filter fields:

creator: URI of the creator
target: URI of the target
motivation
bodyValue
...

POST /annotation add a new annotation
PUT /annotation/{id} modify an existing annotation
GET /annotation/{id} get an individual annotation
DELETE /annotation/{id} delete an individual annotation

An annotation is a JSON object with the following structure (see discussion at https://github.com/gbv/cocoda/issues/190):
{
  "@context": "http://www.w3.org/ns/anno.jsonld",
  "type": "Annotation",
  "id": "http://example.org/annotation/{id}",
  "creator": "http://uri-of-the-creator.org",
  "target": "http://example.org/uri-of-the-mapping"
}

The target value must (should?) be an existing mapping URI in the same jskos-server.

The creator field is handled like the creator.uri field in JSKOS

The annotation further has timestamp fields created and modified, filled automatically

The annotation MUST further have fields

either field  motivation with value assessing and field bodyValue with value +1 (upvote) or -1 downvote
or field motivation set to one of commenting, questioning and bodyValue set to a non-empty string
or field motivation set to bookmarking (this allows public bookmarks)

validation: yes, a simple JSON schema is enough - this could be included in jskos-tools (and maybe later moved to JSKOS specification).

I will add this next week.

"should everyone be able to change/delete any annotation?" no, annotations belong to their user, but we later may need an admin interface to delete around (#39).
deleting: just delete. We can think about undeleting once we may have full versioning

These two points are already implemented that way now.

the endpoint also needs to be documented in README.md.

True. 🙈 Also next week.


@uncle9999 #149

@uncle9999 #149
Originally posted by @uncle9999 in https://github.com/LeCoupa/awesome-cheatsheets/pull/71#issuecomment-720158281

#18


Configuration screen in FancyZones

Microsoft PowerToys version
0.80.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
FancyZones Editor
Steps to reproduce

Hello, sorry for my bad english. I want to report a problem that i'm having with FancyZones Editor. When I am at work i have 3 screens in 1920x1080 and at home i have 1 screen in 5120x1440. When I installed PowerToys I was at work. When I run FancyZones at home there is an issue because FancyZones remember the configuration that i have at work but don't recognize actual configuration (1 screen). So I can't do anything to adjust and apply my personalization screen. My computer is Windows 11. Thank you so much for reading. And thank you for your tool, it's very appreciable.
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

@ebrancaleone
Can you please upload a /bugreport for debugging.
@SeraphimaZykova
Can you look into this?

I don't know if there is any coincidence but FancyZones is pined to my windows taskbar and when the issue produce when I launch it by this shortcut. I tried to launch by tray icon of PowerToys and the issue disappeared. Do you still want me to send you the bugreport zip ?

PowerToysReport_2024-04-18-08-30-10.zip
Here is the bugreport zip. Thank you. This morning when i was back to my work, FancyZones display the configuration screen of my home (yesterday). I confirm you that if FancyZones is pined to my taskbar the configuration screen is not actualize at start. But when i open the editor by the tray icon the configuration updates.

Have you tried closing and re-opening PowerToys?

Of course. Even try to reboot computer. The issue is still exists. When I switch between work and home I shutdown my computer. So PowerToys is reboot.

I don't know if there is any coincidence but FancyZones is pined to my windows taskbar and when the issue produce when I launch it by this shortcut. I tried to launch by tray icon of PowerToys and the issue disappeared.

It's not a coincidence. When you launch the Editor via shortcut or from settings, FancyZones collects information about the monitor configuration and writes it to the file, which is used by the Editor then.  This doesn't happen if you launch it just from the taskbar and the Editor uses the data from previous launches.


Aim towards ES6

The current modular structure is fair simple for conversion to classes, but it will entail to rely on some sort of bundler. This probably entails an increase in weight for end file.
More bold proposition is also to abandon jQuery dependency.
What do you think?

Proposes:

[x] Provide as module for loaders/bundlers (AMD, CommonJS, ES6 Import etc.)
[x] Convert to the ES6 Classes
[ ] Abbadon jQuery

Some thoughts for the future after 1500 stars 😉
Using classes would be nice. While the code is already using a quite modular approach, this would be indeed not that hard.
Moving away from jQuery is understandable, because a lot other libraries are trying to avoid it. On the other hand, a lot of front-end developers are still using jQuery in their projects. So they're including jQuery anyways. From a developer perspective, jQuery doesn't offer huge advantages compared to plain javascript these days. A move would be a lot of work - rebuild from ground number 3, I guess.

This would be super nice!

v3.0.0 release is here 🎉


Can't find icon file after making.

Hey there.  This seems like a really awesome tool although, I do seem to be having one issue.  When running the created .exe I get the below error.  Any ideas on how I can fix this?
Thanks!

What options are you exporting with?

Thanks for responding so fast!  Here are the options.   I appreciate any assistance you can provide.
When I looked at package.json it seems that // were added for each directory listed i.e. "mac_icon": "assets//images//icon_800.png"  Thanks again!

Hmm your options look good. I just tried exporting using your options and they work fine. What bothers me is the "Failed to load extension" message. Can you post your package.json contents?

Hey Joey, it was something I did on my end.  Sorry to take your time with this!
Great tool!


GFLOPs for attention and deformable attention

Hello, I add forward_dummy and dummy_meta code, and use get_flops.py to calculate the flops.
But seems many calculate function (torchstat, ptflops, etc.) doesn't support attention layer or deformable attention layer very well.
Can you please share the code to calculate flops of attention layer and deformable attention layer?
Thank you.

Hi, the mmdet supports the calculation of the deformable attention layer, just using get_flops in mmdet is ok. But mmdet doesn't support the calculation of the attention layer.

Hi. Can you share how to add forward_dummy and dummy_meta code?
Thanks.

def forward_dummy(self, img):
    from mmdet3d.core.bbox.structures.box_3d_mode import LiDARInstance3DBoxes
    dummy_metas = {
                'scene_token': 'c3ab8ee2c1a54068a72d7eb4cf22e43d',
                'can_bus': np.random.random((18)),
                'lidar2img': [np.random.random((4,4))]*6,
                'img_shape': [(928, 1600, 3)]*6,
                "box_type_3d":LiDARInstance3DBoxes,
                }
    return self.forward_dummy_test(img=img, img_metas=[[dummy_metas]])

def forward_dummy(self, img):
    from mmdet3d.core.bbox.structures.box_3d_mode import LiDARInstance3DBoxes
    dummy_metas = {
                'scene_token': 'c3ab8ee2c1a54068a72d7eb4cf22e43d',
                'can_bus': np.random.random((18)),
                'lidar2img': [np.random.random((4,4))]*6,
                'img_shape': [(928, 1600, 3)]*6,
                "box_type_3d":LiDARInstance3DBoxes,
                }
    return self.forward_test(img=img, img_metas=[[dummy_metas]])

Hello, thank for your work, but i want to know what the input_shape is,
or which command do you run?
can you explain for me?
thx


Add support for Symfony 3

@nilportugues ping :)

Php-CS-Fixer has a fixer that replaces this library, so I will retract this PR


Refactor routing

Routing function is huge and needs some work doing to tidy it up, any small commands like help etc should be broken off into other functions. Need to figure out a good way to handle feature flags too but that'll be a different issue.

Split up the bot message and ! Check logic so the bot doesn't trigger other random replies like the bezos one.

Fixed the bot self triggering in #72


User can set restrictions for group when creating a group and when ed…

User can add major/university restrictions.
This completes issue #105

I get this error
Android Bundling failed ...
Unable to resolve "../utils/groupUtils" from "app\screens\Explore.js"

I don't see a groupUtils file that you git added, did you forget to git add it maybe?


feat: setup Tailwind configurations

Description
Added existing tailwind configuration from master to the current migration branch, so that we can get appropriate CSS properties from TailwindCSS.

I checked it with some example components, but looks like it isn't working


Mit or GPL license

I am a little confused about the licensing. The link to the jQuery license makes no mention of the GPL: https://jquery.org/license/
What exactly does the dual licensing entail? Could this project be effectively treated as MIT?

When any project is licensed under multiple licenses, you can pick any of those licenses and ignore the others. I believe jQuery was originally released under both, which means this code retains the GPL licensing from that time. jQuery is now only MIT, but that doesn't affect this project.

@ljharb The link you provided in the readme file lists JQuery as being Apache 2.0 licensed (https://jquery.org/license/).  JQuery's license may have changed since you posted the link, but it's a little confusing as it's written now because there isn't any context as to why the link is provided.  Regardless, I plan on treating your software as MIT (at my choice).  But you may want to remove the link or provide some clarification.


Find highlight is not seen when cell edit mode is active

On Mac OS X, with a given cell editor active, the scroll to position of the current highlighted hit (in orange) is just out of view. See image below:

When the editor is NOT active, the highlighted hit is not obstructed by the UI in any way. See image below:

Fixed by the #15149


Raw Ores Graphics issue

Mod source
Factorio Mod Portal
Which mod are you having an issue with?

[ ] pyalienlife
[ ] pyalternativeenergy
[ ] pycoalprocessing
[ ] pyfusionenergy
[ ] pyhightech
[ ] pyindustry
[ ] pypetroleumhandling
[ ] pypostprocessing
[X] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[X] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[ ] Pypostprocessing failure
[ ] Other

What is the problem?
I installed all Pyanodon mods and now I have graphics issue with all Factorio textures.
I've tried to reinstall Factorio and mods and still have this issue.
Steps to reproduce

Install all Pyanodon mods
Restart Factorio

Additional context
Here is a screenshots of bug: https://imgur.com/a/i8qKqfO
Factorio version: 1.1.61
Pyanodon Raw Ores version: 2.2.7
Pyanodon Raw Ores Graphics version: 1.0.9
OS: Win10x64
Log file
Here is a log-file: https://pastebin.com/ZhsEt2Y1

looks like your computer is trying to kill itself. not something we can fix in the pymods

looks like your computer is trying to kill itself. not something we can fix in the pymods

It's not a problem with my gpu or drivers. When I disable these 2 mods, it's working pretty fine.

I tried to manually remove Factorio from computer and installed it again. After this is has started working fine.


Submitting a new profile results in two web3 transaction requests

Describe the Bug
As a new user when I submit a new profile MetaMask prompts me to confirm two transactions (they are byte-for-byte identical). Confirming both would end it wasting gas as one would be denied.
Two users reported this to me so I decided to check it out for myself. It caused confusion because they thought they needed to send both transactions to request registration.
To Reproduce

Clear cache.
Start application with a funded MetaMask wallet (for an address that isn't registered on PoH).
Click "Submit Profile"
Fill out required information.
Click Submit
MetaMask asks you two confirm two transactions

Expected Behavior
I expect the application to only submit one transaction to the web3 wallet.
Screenshots

Video of reproduction: https://drive.google.com/file/d/11R3jva-KwWmUOBHNT_xuKGCfRdTzTA2K/view?usp=sharing
Device and Browser Information
Please see video

Parsing through some newer submissions, I can see people have made the mistake of confirming both transactions:

https://etherscan.io/address/0xe2dc6c29b31a39c416ac325bee52944677187f79
https://etherscan.io/address/0x9f82a22b62686e77964e71ddec96d3646990404b
https://etherscan.io/address/0xa64092c19f547f33cfafb15d5a1380dc80545d14
https://etherscan.io/address/0x3af98916d560ccb524180597e6a35f2201ef5df3
https://etherscan.io/address/0xf2e9ec4ba09bae56177e795c8ae7d4a9e9b16669
https://etherscan.io/address/0x3057587f759808688bce6f0022115269295ac398

Looking through mainnet transactions again it appears some users are also clicking Submit more than once, and approving all the resulting transactions 😳

I also have this issue. The double submission also affects other transactions like fund submission button or the challenge removal button. People on telegram groups also report this issue, so it looks like it is widespread.

I also have this issue. The double submission also affects other transactions like fund submission button or the challenge removal button. People on telegram groups also report this issue, so it looks like it is widespread.

Agreed, playing around with debugging it seems state management in the UI needs some love.

This commit is what created the current bug: https://github.com/Proof-Of-Humanity/proof-of-humanity-web/commit/c561920fede8e82458a7aa61cf2943b021ef385e#diff-5d04351f46fd31e3ab5abf320c7ada976f3ba8b63366edf5f1d1aba3f3991b66L439-L440

I just ran into this bug (or something similar) today while registering. I was using MetaMask 10.8.0, Firefox 95.0.1 and macOS 11.6.
I initially thought I might have clicked Submit twice accidentally, but after rejecting the first two transactions and resubmitting I got prompted with two more transactions.
If you'd like me to try reproducing it on the same setup let me know what logs/debugging info to record :pray:


Crash Report - Testnet 3.3 - CrisF#3405

Snarkworker and blockproducer running for 4+ hours.
Just observing at the time of the crash.
Many prints such as:
coda_crash_report_2020-09-17_08-31-49.154225.tar.gz
2020-09-17 08:54:26 UTC [Info] validating a new pubsub message ...
2020-09-17 08:54:27 UTC [Info] why u fail to validate :(
2020-09-17 08:54:27 UTC [Info] why u fail to validate :(
2020-09-17 08:54:30 UTC [Info] starting refreshing cpl 0 with key CIQAAAMVU4AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA (routing table size was 0)
2020-09-17 08:54:30 UTC [Info] failed when refreshing routing table
``` ```
2020-09-17 08:57:26 UTC [Info] would have validated but it's from us!
2020-09-17 08:57:26 UTC [Info] validated!
2020-09-17 08:57:26 UTC [Info] why u fail to validate :(
2020-09-17 08:57:26 UTC [Info] validated!

Crash report attached.
coda_crash_report_2020-09-17_08-31-49.154225.tar.gz

Prints as above continue upon restart.

One of the snark worker errors that should be fixed by #6036


Rendering issue with mouse movement

Windows 10 latest update
V 4.0.1 GDMP
**Issue Descriptions:
When moving the mouse anywhere with the desktop player open there seems to be a rendering issue. several components flicker when moving mouse anywhere on screen. The album art, three-dot hash, thumbs up thumbs down and volume control will flicker If I move my mouse in particular below the track progression bar.
Further, With the track queue open and moving mouse over various tracks there is flickering of album art and the dark highlight intended with current mouse hover jumps around from multiple tracks in the list when moving mouse.
**Steps to Reproduce: happens all the time music playing or not. It seems specifically to occur when mouse moves over a selectable object.
I hope I explained this well enough.

Glad to know I'm not the only one! When I'm highlighting a song, it rapidly switches between the song four or so spaces down and the one under my cursor. The mini player behaves the same way, and I can't pause/play/rewind etc. from it.

Duplicate #1777
Currently the theory is that this is partially to do with touch screens.

@MarshallOfSound just a note. My laptop does not have any touch screen functionality. Samsung Series 9. I will look at #1777