question about types, services, optional fields

Hi there!
I like your idea of having a GDScript compiler for Godot. However, I need more advanced features of the Protocol Buffers for my use case, thus a few questions.
What is the reason for not supporting types, services, and optional fields?
It is because of the complexity of marshaling or missing features in Godot?
If so, what would be a reasonable approach and the next steps to implement support for them?
Do you consider making a protoc plugin reasonable since that imho would be the regular way to go?
Thanks!

Hi!

What is the reason for not supporting types, services, and optional fields? It is because of the complexity of marshaling or missing features in Godot?

Optional fields are defined only in the specification proto2. In proto3, as far as I understand, they abandoned the optional fields. For proto2 Godobuf suppors optional fields.
What "types" are you mean?
Services assumes use of interfaces or something abstract constructions, in the GDScript they seem to be absent.

If so, what would be a reasonable approach and the next steps to implement support for them?
I do not plan to support services in the near future.

Do you consider making a protoc plugin reasonable since that imho would be the regular way to go?

It would be simpler and smarter, but not convenient to use. Godobuf is convenient, because built into IDE, does not require any additional software and does everything in one click.

Hi!
As I was reading "Any types" under "Not supported" I mistakenly assumed you do not support defining any message type.
But now I understand that you mean the "keyword" Any for Any-message types.
Just a little bit of confusion on my side :-)
Thanks!

It would be strange not to support any types, then what to support at all :-)
Can I close the issue?

I agree. Yes.

Optional fields are defined only in the specification proto2. In proto3, as far as I understand, they abandoned the optional fields. For proto2 Godobuf suppors optional fields.

Protobuf 3.15 (Feb 18, 2021) introduced optional messages and I need that feature to know If my Vector coordinates are NULL or 0.0
Example from: https://stackoverflow.com/a/62566052/12619313
syntax = "proto3";

message Foo {
    int32 bar = 1;
    optional int32 baz = 2;
}

A has_baz()/hasBaz() method is generated for the optional field above, just as it was in proto2.

@oniksan should I create a new issue for this?

Yes, please create new.

Cheers! https://github.com/oniksan/godobuf/issues/28


Perfetto SDK support App tracing in production

Perfetto sdk size(.so about 1MB) is too large for app tracing in production
Can you trim out most of the unnecessary functionality and provide a lightweight SDK?

If you build with -Wl,--gc-sections the linker will strip most unused parts.
That should shrink it to ~200KB.
We are also working on a shlib C-only API, details on https://bit.ly/perfetto-c , but that will take some time to become a thing.
Note: you'll need to join this group to get access to the doc: https://groups.google.com/forum/#!forum/perfetto-dev


Revert "README updates and copyright headers."

Reverts google/cloud-android-orchestration#203
Breaks building the debian package as goutil script does no longer start with shebang (#!)

Sounds like you just need to put the #!/bin/bash line at the beginning, not revert the whole thing. Unless you want to submit it again with the fix.


Is there a wrapper of coreNLP for open information extraction?

Among the Stanford CoreNLP API documents I couldn't find the api for stanford's Open Information Extraction (OpenIE) annotator.
Is there any?
thanks a lot

@Vicky-Meng: Please post your question to a more appropriate place, thanks


KOGITO-4545 Disable header validation for generated sources and fix t…

…emplates to create valid header
Many thanks for submitting your Pull Request :heart:!

The Linux check is successful.

@danielezonca agree but that will require some work. If fine with you, I would like to just fix the examples and look into that in a follow-up PR/JIRA.

@danielezonca agree but that will require some work. If fine with you, I would like to just fix the examples and look into that in a follow-up PR/JIRA.

Sure, fine for me 👍


Pure JSON API

The siad API now uses JSON exclusively. The API is documented in API.md.
siac is still fairly broken. To make things cleaner, I'm thinking of adding some helper functions/structs to siad/api.go and importing siad in siac. This should make it easier to change the API without having to fix things in 3 different places.

I'm not sure what's causing the build fail, but given that it passed on an earlier commit I think it just got unlucky with the transactions.

I decided to merge this prematurely so Seve can use the changes.
Please review and submit new changes when you get the chance

Seve could have just worked off my branch. I guess it would be a bit uglier if I made additional changes though.


How-To Pages

Describe the solution you'd like
A new page e.g. /how-to which is a manual on how to use different types of features (Later on maybe even Setup etc?). This pages could utilise markdown files, so that they might also be added to the GitHub Wiki.
It might be better to separate the admin-tabs from the how-to pages, so the how-to pages can be a standalone documentation for anyone who is interested.
For now the chat features would suffice.

I started using the wiki for this a while back, I'd be fine with extending it https://github.com/joschahenningsen/TUM-Live/wiki


README: update badge

🦟 Bug fix
Fix Ubuntu CI badge to use focal job instead of bionic
Summary
Also test macOS CI with gdal dependency using https://github.com/osrf/homebrew-simulation/pull/1790
Checklist

[X] Signed all commits for DCO
[ ] Added tests
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[X] codecheck passed (See contributing)
[X] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

homebrew CI needs https://github.com/ignition-tooling/release-tools/pull/626

I've updated the jenkins homebrew job configuration to use the scpeters/brew_doctor_ignore_proj7 branch of release-tools

@osrf-jenkins run tests please


Refresh SAML Metadata

Please see: https://github.com/pac4j/pac4j/issues/1109
Will someone take another look at this?
It is a standard feature in other SAML SP Implementations, e.g. spring-security-saml or shibboleth.
If the IdP Metadata is an URL, you refresh it from that URL. If it is a file, you simply refresh it from that file (the user could change the file), although that is much less important a use-case. The interval for refreshing should be configurable.
The interval-based refresh is necessary to capture changes to the Metadata that occur before the validUntil expires.
In addition, there should be a refresh when validUntil is over.
This behaviour is in fact implemented in opensaml, see: https://github.com/apigee/java-opensaml2/blob/master/src/main/java/org/opensaml/saml2/metadata/provider/HTTPMetadataProvider.java
So all you would have to do is expose the configuration in pac4j-saml...

Do not create duplicate issues please. See https://github.com/pac4j/pac4j/issues/1109#issuecomment-539522096

is this added?


Unable to build Docs.elm

First of all, I really really like this library, thank you @terezka for all the time and effort and thought you put into it.
I have been struggling to understand how to use hints from the documentation, so I thought I would play around with the example code in https://terezka.github.io/elm-plot/ .
However, after cloning the repo, I couldn't find any way to build Docs.elm, there is no Readme nor Makefile, so I tried elm-make Docs.elm and a few more variations of that.
Eventually, it seems like the code uses an outdated version of elm-plot, because Docs.elm is trying to import a PlotComposed module that is not declared anywhere.
Thank you for your time and please let me know if I can help.

I think docs/Docs.elm is an artifact to be removed.
The updated one is located at docs/src/Docs.elm.
to build, run elm-make docs/src/Docs.elm --output=docs/docs.js

Thanks! 💛 Did @skvale's answer help? I'm working on the hint situation! I realize it's a little awkward at the moment! There should be an example use here.

@skvale thank you, that worked!
@terezka eventually I went through the library code to get a better idea of how to use the it.
We are going to use elm-plot in production, so I hope I'll be able to contribute some real-life example.
Looking forward to the hints update!


Tracking community engagement

Was curious if you've thought of a way of tracking and promoting those who engage with communities that they support / contribute to the collective directly in site content form / commits downstream to repos that feed it, money (which I know you "track" in some form). I bring this up because I think you could build a bit of an engagement / tracking engine pretty easily or at least worth thinking about thanks to a standard called xAPI.
xAPI is short for Experience API. It's popular in training / education but also with analytics / advertising people. There's also a great open source xAPI store (in industry called LRS) which might be willing to get involved in exchange for the benefits this group provides -- https://learninglocker.net/
If you want to learn more about xAPI, here's some stuff assembled about xAPI in a public elmsln space I'm working on https://courses.openulmus.org/xapi/
Basic idea: Track activity / everything people do back from distributed places to centralized LRS (learning locker). Then query the engine nightly (or weekly or whatever) and email reports to people in open collective highlighting who had the most engagement that week, who contributed the most money, etc.

@xdamman you might be interested in this

Yes we would love to have an easy way to create monthly activity reports for backers.
See also https://github.com/OpenCollective/OpenCollective/issues/312#issuecomment-279915527
Ideally, we should have "plugins". Eg a plugin for github.com, one for Facebook, medium, meetup.com, etc.
Each of those plugins would create a monthly digest for the collective. This would include:

a short summary:
Webpack had 10 new issues, 7 new pull requests and 30 new comments.
a digest:
Top issues this month:

release of webpack 2.0 (20 comments)
now you can order your webpack swag (12 comments)
Latest release notes:
xxxx
etc.

You could imagine the same with the number of meetups and number of people who signed up for them, most popular posts shared on twitter or Facebook by the collective, totally number of engagement through comments and likes. Etc.
Feel free to prototype one of those plugins for the platform of your choice. This could be the base for others to build other plugins. Let me know what you need.

I referenced this and #769 as potential components the Drupal community could use to sustain projects. http://buytaert.net/comment/130991#comment-130991
Based on the feedback from the founder of the project others in the community, I think Drupal would be a good test case.

@btopro What about running a learning program built on https://www.elmsln.org/ to prove some concepts and even work on some of these integrations? I could see OpenCollective as a tool for funding those putting together tutorials, curriculum, or training programs.

@gusaus That's kind of my vision w/ OpenUlmus as a Free deploy that's community supported and has free "courses" / training. Are you thinking that this could be funded by OpenCollective to cover costs of hosting or you mean like pay contributors to build tutorials in the system or what?
I'd be open to helping get that going w/ some idea of a roadmap and realization that I fall off the map when we get in crunch periods of contribution to the tool itself :)

@btopro All of the above! One of the goals of https://opencollective.com/drupalopenlearning, still in limbo, is to provide a way to sustain a community supported, free, learning program for Drupal. Funds could be allocated to the people developing/maintaining the website and managing the program, and covering other costs like hosting.
OpenCollective could also be an option for those needing funds to create their own training program , curriculum, tutorials, etc. Ideally this functionality would be built into the system where users would be able to create their own campaign.
Probably don't want to get too into the weeds in this issue (I've already set up the topic in Slack), but with ELMS also running on Drupal, it seems like a good project to test the waters in the open and engage the community.

We're currently iterating and updating older issues. Unfortunately we have no plan to support that in the immediate future.


Cannot set 'scaleX' to Float.NaN exception when PressedScale is set in Android

I am testing v2.0.5 but I've seen the same issue in older versions as well. When in Android and set PressedScale to lets say 0.9 then the app crashes with the Exception Cannot set 'scaleX' to Float.NaN .

@ssotir small sample please..
can you reproduce it on iOS ?

No, iOS works fine

What forms version do you use?
Can you replicate it with latest version?
I think it's XF bug

XF 3.6.0344457

@ssotir hmmm
https://www.google.com/search?q=Cannot+set+'scaleX'+to+Float.NaN&oq=Cannot+set+'scaleX'+to+Float.NaN&aqs=chrome..69i57j69i60j0l2.121j0j7&sourceid=chrome&ie=UTF-8
A lot of similar reports. I think it's XF issue

@ssotir could you please fork my repo and reproduce this issue there? and send me branch with it
I guess, we need report it to XF proj

Use Android support version <IP_ADDRESS> (latest version)
and target platform 8.1
(works fine for me)

target platform 8.1 isn't accepted bij Google anymore, should be 9.0 (Api 28). But then this issue appears. XF 4.2. Any idea why this is happening?

@KeesAlderliesten could you please provide a sample?

Hello Andrei,
I will try to make one later. For now I removed the touchview on Android and it don’t crash anymore on Pie (9.0). Now I can release my app.
Earlier I had to remove the TouchImage on Android because that caused also crashes. I sent you a mail about it 2 weeks ago. Did you receive that?
Kind regards,
Kees Alderliesten
Van: Andrei<EMAIL_ADDRESS>Verzonden: dinsdag 17 september 2019 14:25
Aan: AndreiMisiukevich/TouchEffect<EMAIL_ADDRESS>CC: Kees Alderliesten<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Onderwerp: Re: [AndreiMisiukevich/TouchEffect] Cannot set 'scaleX' to Float.NaN exception when PressedScale is set in Android (#10)
@KeesAlderliestenhttps://github.com/KeesAlderliesten could you please provide a sample?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/AndreiMisiukevich/TouchEffect/issues/10?email_source=notifications&email_token=AB6MVIQ6FIZXCJMSWZ6SP3TQKDEARA5CNFSM4HKUTMJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD64K6BI#issuecomment-532197125, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AB6MVIWHNPHJTUQVAU7WHYDQKDEARANCNFSM4HKUTMJQ.

Hello Andrei, I will try to make one later. For now I removed the touchview on Android and it don’t crash anymore on Pie (9.0). Now I can release my app. Earlier I had to remove the TouchImage on Android because that caused also crashes. I sent you a mail about it 2 weeks ago. Did you receive that? Kind regards, Kees Alderliesten Van: Andrei<EMAIL_ADDRESS>Verzonden: dinsdag 17 september 2019 14:25 Aan: AndreiMisiukevich/TouchEffect<EMAIL_ADDRESS>CC: Kees Alderliesten<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Onderwerp: Re: [AndreiMisiukevich/TouchEffect] Cannot set 'scaleX' to Float.NaN exception when PressedScale is set in Android (#10) @KeesAlderliestenhttps://github.com/KeesAlderliesten could you please provide a sample? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#10?email_source=notifications&email_token=AB6MVIQ6FIZXCJMSWZ6SP3TQKDEARA5CNFSM4HKUTMJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD64K6BI#issuecomment-532197125>, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AB6MVIWHNPHJTUQVAU7WHYDQKDEARANCNFSM4HKUTMJQ.

Hm, not sure. Perhaps it is in spam

Switch TouchEffectSample.Android to target Pie (9.0), clean project en rebuild.
Run sample app on Android 9.0 device of emulator and tap button ‘(FADE EFFECT)’
This results in Java.Lang.IllegalArgumentException: 'Cannot set 'scaleX' to Float.NaN'
<EMAIL_ADDRESS><EMAIL_ADDRESS>As temporary workaround I replaced the TouchView on Android with an AbsoluteLayout with a TapGesture and in the tap event handle I put this:
<EMAIL_ADDRESS>Doesn’t work as nice as your control (doesn’t stay scaled if I keep it pressed). but for now I’m goed, app  can now be published.
But somehow it crahses on your call to ScaleTo, and mine doesn’t. I don’t get why…
Kind regards,
Kees Alderliesten
Van: Andrei<EMAIL_ADDRESS>Verzonden: dinsdag 17 september 2019 14:25
Aan: AndreiMisiukevich/TouchEffect<EMAIL_ADDRESS>CC: Kees Alderliesten<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Onderwerp: Re: [AndreiMisiukevich/TouchEffect] Cannot set 'scaleX' to Float.NaN exception when PressedScale is set in Android (#10)
@KeesAlderliestenhttps://github.com/KeesAlderliesten could you please provide a sample?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/AndreiMisiukevich/TouchEffect/issues/10?email_source=notifications&email_token=AB6MVIQ6FIZXCJMSWZ6SP3TQKDEARA5CNFSM4HKUTMJ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD64K6BI#issuecomment-532197125, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AB6MVIWHNPHJTUQVAU7WHYDQKDEARANCNFSM4HKUTMJQ.

found workaround

fixed in 4.1.1

Getting this in last version <IP_ADDRESS>7

Getting this in last version <IP_ADDRESS>7

the latest version of this package is "4.1.51" ;)

Getting this in last version <IP_ADDRESS>7

please install the latest version from nuget (4.1.52) and check


Fix rare bug where some k8s events would be emitted without a timestamp

What does this PR do?
Fixes a rare bug that would happen where some kubernetes events would be emitted with no timestamp, resulting in them getting dropped upstream
Motivation
While testing another apiserver check change, I noticed that the Scheduled pod event emitted by the kube-scheduler was not showing up in the UI when it was set to be unbundled. After investigating, I noticed it was because this event type does not have firstTimestamp or lastTimestamp set. Upstream dropped events metric

Example:
apiVersion: v1
kind: Event
metadata:
  name: hello-28719074-dc8qz.17e9d6bd239aefa9
  namespace: workload-stress-test-ad
  uid: 4bbd5526-50f3-4516-8a20-d2b52ef80853
  resourceVersion: '1400320'
  creationTimestamp: '2024-08-08T19:14:00Z'
  selfLink: >-
    /api/v1/namespaces/workload-stress-test-ad/events/hello-28719074-dc8qz.17e9d6bd239aefa9
involvedObject:
  kind: Pod
  namespace: workload-stress-test-ad
  name: hello-28719074-dc8qz
  uid: 402d6de8-a7cf-4c69-9d95-3091e2542075
  apiVersion: v1
  resourceVersion: '1400316'
reason: Scheduled
message: >-
  Successfully assigned workload-stress-test-ad/hello-28719074-dc8qz to
  steve-containerd-new
source: {}
firstTimestamp: null
lastTimestamp: null
type: Normal
eventTime: '2024-08-08T19:14:00.280440Z'
action: Binding
reportingComponent: default-scheduler
reportingInstance: default-scheduler-steve-containerd-new

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Deploy the agent via helm with the following values:
clusterAgent
  confd:
    kubernetes_apiserver.yaml: |
      init_config:
      instances:
      - unbundle_events: true
        bundle_unspecified_events: true
        collected_event_types:
          - kind: Pod
            reasons:
              - Scheduled
              - Started

Verify that Scheduled events show up, and that there are no dropped upstream (use this metric query, with an updated timeframe)

/merge


Duplicate 'E' access key in Brazilian Portuguese

VSCode Version: Versão: Commit de 1.33.1 (user setup): 51b0b28134d51361cf996d2f0a1c698247aeabd8 Data: 2019-04-11T08:27:14.102Z elétron: 3.1.6 Chrome: 66.0.3359.181 node. js: 10.2.0 V8: 6.6.346.32 OS: Windows_NT x64 10.0.17134
OS Version: Windows 10 Pro and Linux Mint 19 Cinnamon

Steps to Reproduce:

Press Alt + E
Submenu Exibir (View) is open; can't open submenu Editar (Edit).

#71075 was almost exactly the same, but regarding Arquivo/Ajuda. Didn't realize the same was going on for Editar/Exibir.

Does this issue occur when all extensions are disabled?: Yes

Should change &Exibir to E&xibir (underlining 'x' instead of 'E'). Editar should be kept the same. This is the usual behavior, like Notepad below:

Fixed.
new tgt: E&&xibir


Assert only specific error messages in cucumber

Feature Request
Description of Problem:
Currently we have 3 steps for asserting error messages:

"Then I am presented with an error message" - asserts any exception was thrown during execution
"Then the profile is invalid" - asserts any one of InvalidProfile, JSONParse, IllegalArgument or ClassCast Exceptions
"Then the profile is invalid because..." - asserts that an InvalidProfileException has been thrown which has a message as specified

Realistically we only ever want to check the most specific case - knowing exactly why the input steps have failed in generation.
Potential Solutions:

Re-purpose the "Then the profile is invalid because..." step to read more generically, e.g. "I am presented with the following error message...".
Remove both of the other steps from the framework.
Adjust all existing cucumber scenarios to use the new specific error message step

Acceptance Criteria:

[ ] A step for asserting error message contents is visible in cucumber
[ ] Both the "Then I am presented with an error message" and "Then the profile is invalid" steps are removed from the cucumber framework
[ ] All scenarios which check failure cases or exceptions check specific messages

This should include the removal of assertions of different types of exception in the the profile is invalid step?
Currently we say the profile is invalid whenever one of the following exceptions is thrown:

InvalidProfileException - correct, should be used whenever a profile is considered invalid
ClassCastException - incorrect, thrown when casting types when they're read from the JSON, e.g. casting the operand to a Number (maybe we need a new ticket for this fix?)
IllegalArgumentException - not thrown anymore from a quick analysis


Feature: If html has custom attributes, copy and paste will end up removing them

Copy and paste text such as:
Hello
When it is pasted:
Hello
The custom attributes are removed. Any ideas why it is being designed this way? Would be nice if we preserve any custom attributes.

You can overwrite the custom importDOM logic of overridden default anodes to translate the HTML attributes into editor state. Otherwise though, we just strip out any unrecognised HTML as part of the normalization process on paste.

Ok thanks for the tips


[dv] use the correct label for e2e_bootup_success test

This fixes #14209 by fixing the Bazel label that refers to the above
test in the dvsim config file that runs this test.
Signed-off-by: Timothy Trippel<EMAIL_ADDRESS>

For visibility, this test took a little over 1hr to run in DV.


Error opening containing folder with "+" in its name

This is an alert of Tracker. I opened a ticket for it at https://dev.haiku-os.org/ticket/14912 .


TopicDisallowed when using VoIP or Watch "Complications"

Took a bit of time to figure out what this was supposed to be set to. I originally thought it was just the app id, i.e com.example.yourApp as noted in the README -- however for VoIP pushes, you have to add a .voip suffix (see "APNs Provider Certificates" in the Apple Docs), same goes for Watch where you have to add a .complication suffix.

topic = 'com.example.yourApp.voip'

I'm not sure how to clarify it in README, maybe add mention that topic is not always equal to bundle id or even add some examples


Improve error message from errImagePull

See #1706 as an example; the message is so close to being extremely helpful. The error state was identified and reported as errImagePull and specifies which container is the problem. But it doesn't show which image it is trying to pull.
If we just tweak that error message it would be much more helpful.

Hi @johnSchnake I want to take this up


Migrate PHPUnit configuration

This PR migrates the PHPUnit configuration as suggested by PHPUnit when running it.

Yeah sure will update the PR tomorrow.
Sent from my iPhone

On 18 Aug 2020, at 23:03, Markus Podar<EMAIL_ADDRESS>wrote:

@mfn approved this pull request.
In phpunit.xml.dist:

    convertErrorsToExceptions="true"

    convertNoticesToExceptions="true"

    convertWarningsToExceptions="true"

    processIsolation="false"

    stopOnFailure="false">

   <testsuite name="Unit Tests">

       <directory>tests</directory>

   </testsuite>

   <whitelist>

       <directory suffix=".php">src/</directory>

   </whitelist>

+
Can we keep this split over multiple lines for readability as not to have to scroll 400 characters to the right to read everything?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I've updated the PR and reworked the update configuration a bit to reduce the diff compared to the old version.


feat(batch): support auto execution mode

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?

Decide execution mode for batch queries automatically. The decision is primarily based on range scans of BatchSeqScan.

Checklist For Contributors

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[x] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[x] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

#8273

It seems deterministic simulation e2e, ci-3cn-2fe-3meta, recovery, batch runs slower than before.

It seems deterministic simulation e2e, ci-3cn-2fe-3meta, recovery, batch runs slower than before.

Almost all batch e2e tests should be explicitly set to a query mode like in local_mode.slt and distribution_mode.slt, so there should be no much coverage of Auto? 🤔

It seems deterministic simulation e2e, ci-3cn-2fe-3meta, recovery, batch runs slower than before.

For small amount of data, distributed mode is slower than local mode.

Let's wait for #8292 to get merged.


docs(release.md): remove typo

Signed-off-by: Leonardo Grasso<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap-engine-udig

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, Molter73
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Molter73,leogr]

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

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr, Molter73
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,Molter73,leogr]

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


Make loading page

Make a loading page for slow processing

You mean loading overlay?

Yes, till all assets are loaded and landing page is displayed. Landing page is currently being addressed by #140

Ohk, can you assign it to me?


Ft 25 deep linking aliases 457

Is this the right branch? The changes contain the git branch switcher, not deep linking aliases?

Is this the right branch? The changes contain the git branch switcher, not deep linking aliases?

Yes, It's right. This branch is just created from switching branch branch, that's why.

Is this the right branch? The changes contain the git branch switcher, not deep linking aliases?

Yes, It's right. This branch is just created from switching branch branch, that's why.

It'd be better if all our branches were based off of main, not of a branch that is currently in development, especially as the other one is not connected to this feature at all. If that other branch turns out non-mergeable we won't be able to merge this one either.

I am gonna send another PR regarding FT-25 so I close this PR.


About visualization and kitti dataset

Hi,
Excellent work! I have trained the model and obtained close results to the paper on FlyingThings3D dataset. I have some problems. 1). How to visualize the scene flow like your paper? 2).  When will the training and test on the KITTI dataset be released? And, can you provide the pre-trained model? Thank you very much!
Best,
Zhai

Hi，Dr.Zhai，I just saw your issue 24 minutes ago，I want to ask that what study you are interested in，and I want to use scene flow to study 3D tracking and LiDAR SLAM. I have not run this code by now，so I want to know the real-time performance of this method，thanks.

Hi,
Excellent work! I have trained the model and obtained close results to the paper on FlyingThings3D dataset. I have some problems. 1). How to visualize the scene flow like your paper? 2). When will the training and test on the KITTI dataset be released? And, can you provide the pre-trained model? Thank you very much!
Best,
Zhai
Hi，Dr.Zhai，I just saw your issue 24 minutes ago，I want to ask that what study you are interested in，and I want to use scene flow to study 3D tracking and LiDAR SLAM. I have not run this code by now，so I want to know the real-time performance of this method，thanks.

Hi,
Excellent work! I have trained the model and obtained close results to the paper on FlyingThings3D dataset. I have some problems. 1). How to visualize the scene flow like your paper? 2). When will the training and test on the KITTI dataset be released? And, can you provide the pre-trained model? Thank you very much!
Best,
Zhai

Hi，Dr.Zhai，I just saw your issue 24 minutes ago，I want to ask that what study you are interested in，and I want to use scene flow to study 3D tracking and LiDAR SLAM. I have not run this code by now，so I want to know the real-time performance of this method，thanks.

@ymxlzgy Hi, I have not tested the time. But I think the speed is fast .

Appendix G describes running time.

Whether any update to release code in KITTI dataset?

KITTI dataset has been released.


Es6/fix tests

@ibgreen I think this PR can be merged now, Ib. Then the lessons PR will just be for the lessons.


Dissertation Visual #12: NC1868 Spectrogram

Spectrogram of borrowings in NC1868. Should show heavy reliance on one or another New York Code, with scattered provisions coming from elsewhere.

DT1868 should be excluded from rankings. How much is in that "Other" category? If it's mostly NY1850, it should definitely be tagged as such.

Oops. I see NY1850 is purple. I meant NY1853. Anyway, are there any NYs in the "other" category?

What remains in the "Other" category? I wonder if we should condense the NY Codes, but standardizing the color scheme might clear this one up.


Upgrade serverless to v3

Cirrus is currently using serverless v1 for deployments: https://github.com/cirrus-geo/cirrus-geo/blob/main/src/cirrus/core/constants.py#L5
Serverless v1 and v2 default to using Node 12, which means that the custom resource lambdas that they deploy use Node 12 as well. This was fixed in Serverless, and is part of the 3.0.0 release.
Node 12 reached end-of-life on April 30, 2022. AWS has stated:

Starting November 14, 2022, Lambda will no longer apply security patches and other updates to the Node.js 12 runtime used by Lambda functions, and functions using Node.js 12 will no longer be eligible for technical support. In addition, you will no longer be able to create new Lambda functions using the Node.js 12 runtime. Starting December 14, 2022, you will no longer be able to update existing functions using the Node.js 12 runtime.

Cirrus needs to be updated to use serverless v3, or there's a risk that deployments will stop working in the next few months.

This issue is effectively encapsulated in #139, as we need to support newer versions of serverless to support containerized lambdas, but calling it out explicitly as a separate issue is not a bad thing.
The work to support that issue and inherently serverless >=3.0.0 is already in the PR #150. As such I will add this issue to milestone 0.7.0 and ensure it is closed out when that PR is merged.


fix: syndicate jobs now have the type

What did you fix?
Syndicate jobs now have the "type" attribute, which was missing

Reproduction steps

Evidence/screenshot/link to line

Considerations

Does this contain a new dependency? [No]
Does this introduce opinionated data formatting or manual data entry? [No]
Does this pr include updated data files in a separate commit that can be reverted for a clean code-only pr? [Yes]
Have I run the linter? [Yes]
Is is a bug fix, feature request, or enhancement? [Bug Fix]

:tada: This PR is included in version 5.0.1 :tada:
The release is available on:

v5.0.1
GitHub release

Your semantic-release bot :package::rocket:


CK3 Duchy Compatibility - Part B

d_utr and pals are intentional and they work
Il Mar 6 Ott 2020, 21:48 Ivo Elezović<EMAIL_ADDRESS>ha scritto:

@Zemurin requested changes on this pull request.
In EU4toV2/Data_Files/configurables/ck2_title_map.txt
https://github.com/ParadoxGameConverters/EU4toVic2/pull/591#discussion_r500554095
:

@@ -3664,7 +3664,7 @@ link = { name = "mazedonien" title = "d_mkd" region = "e_byzantium" }

Empires
link = { name = "ajuraan" title = "e_ajuraan" region = "e_ajuraan" }
link = { name = "ajuran" title = "e_ajuraan" region = "e_ajuraan" }
-link = { name = "guinea" title = "e_guinea" region = "e_guinea" }
+link = { name = "guinea" title = "e_guinea" region = "e_guinea }
Close quote after Guinea
In EU4toV2/Data_Files/configurables/country_mappings.txt
https://github.com/ParadoxGameConverters/EU4toVic2/pull/591#discussion_r500554244
:

@@ -457,6 +457,7 @@ link = { eu4 = LUX v2 = LUX } # Luxembourg
link = { eu4 = d_luxembourg v2 = LUX } # Luxembourg
link = { eu4 = NED v2 = NET } # Netherlands
link = { eu4 = UTR v2 = UTR } # Utrecht
+link = { eu4 = d_utr v2 = UTR } # Utrecht

d_utr doesn't seem right
In EU4toV2/Data_Files/configurables/country_mappings.txt
https://github.com/ParadoxGameConverters/EU4toVic2/pull/591#discussion_r500554368
:

@@ -1456,6 +1457,9 @@ link = { eu4 = d_tafilalt v2 = TFL } # Tafilalt
link = { eu4 = d_udmurtia v2 = UDM } # Udmurtia
link = { eu4 = d_uzboy v2 = UZB } # Uzboy => Uzbekistan
link = { eu4 = d_yatenga v2 = YAT } # Yatenga
+link = { eu4 = d_asy v2 = ASY } # Chaldea => Assyria

Also these 3 are off
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ParadoxGameConverters/EU4toVic2/pull/591#pullrequestreview-503303906,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF3JX6B7FHKMD47HDQO6TSTSJNYBHANCNFSM4SGPD2JQ
.

No worry, I'll fix guinea then (again, your last commit had the same error), but please fix it locally or sync after I do.


Custom templates and blocks are not showing in the editor

Hi!
First of all, thanks for building this great package.
I'm trying to add custom templates and blocks by added them to the 'resources/views/vendor/laravel-grapejs/templates' and 'resources/views/vendor/laravel-grapejs/gjs-blocks' folders, but somehow they aren't showing in the editor.
Even the two files that were already there (named example.blade.php) aren't showing.
Any idea what I'm doing wrong?
Thanks.

👍

@JerrelZ please look at this section in the readme and let me know if this has not helped you.

I have the same issue.

I have the same issue.

@JerrelZ please look at this section in the readme and let me know if this has not helped you.

This is a Slack link. I don't have access to your workspace.

Apologize, I've updated the link. Review now and let me know if it doesn't resolve your problem.

still blocks and templates not working

From what I can tell, templates fail to show if they include Laravel directives and possibly Tailwind, which would make it useless for me. I'm not entirely sure but I can get at least get Blocks to show up but only if they are vanilla HTML. Hopefully @jd-dotlogics can look into this. Should Laravel directives and Tailwind work?? Also, where are buttons and other elements in the edit panel??

I had this this issue a while ago and I found out that when two blade files with similar names were in the resources/views/vendor/laravel-grapejs/gjs-blocks  or resources/views/vendor/laravel-grapejs/templates directories, this issue would occur.
I tried giving each block and template unique names with kebab-case naming convention and it was fixed. I hope it also works for you.


Log introspector job info on failure

Add one INFO and one FINE log messages on introspector failures. Those log messages will be generated at the first failure of each introspector job, and when the job is considered done and deleted by the operator.
Write two unit tests for the changes in this PR.

Remote Jenkins jobs are running.
https://build.weblogick8s.org:8443/view/all/job/weblogic-kubernetes-operator-kind-new/553/
https://build.weblogick8s.org:8443/job/weblogic-kubernetes-operator-model-in-image-tests-10/447/


Generate metadata.json files so AoT compilation can be done

Also updated typescript and thus removed typings dependency

I've integrated your PR into my PR: https://github.com/xmaestro/angular2-recaptcha/pull/28

Merged through #28


i18n add a new language: Occitan

I open this issue to add a new language.
What file should I use as a template?
Best regards
Steps

[x] lib/translations.py
[x] share/translations/oc.txt
[x] help: share/translations/help-oc.txt
[x] v2 translation
[x] we-lang translation

Hi @Quenty-tolosan!
Thank you for your help
I've added the list of steps that we need to do for the translation.
They are fully independent and can be done in any order.
Just for the repository, translate it, and open PRs as you advance,
after each step, or after all steps done, as you wish.
The former is probably better because we can see and test the results sooner

@iago-lito Do you speak Occitan too? If I remember it correctly Montpellier was in the Occitanie region?
Should we just fork Occitane translation from the French one, and then just fix the difference,
or it is not so easy?

@chubin Unfortunately I do not, although you are right about Montpellier and I'm very happy to read that there is an interest in translating to local languages :)
This said, I doubt it would be useful to fork from the French translation, because I think that very few words will be unchanged. It sounds familiar to the ear of a French speaker, but just enough to understand the context, and I recall that the writing differs much.

Hello! No it would take more time to change from the French then using a Translation Memory ;)
I do have it in Occitan and some other software for weather were translated so I would only need to validate and translate the rest :) I'm planning to work on it this week end

@Quenty-tolosan I'm afraid that you will find no time for the wttr.in translation, and we lose such a great opportunity to have Occitan support for wttr.in. It is very hard to find someone who can help with it, and that was very bug luck that you created this issue. So I hope that you will not disappear.
If you currently have no time for forks, PRs etc, we can start slowly,
with translation of the most basic words:
"Morning",
"Day",
"Evening",
"Night",
"Weather report for:",
"Weather",

And the next portion (from the v2 view):
"Timezone",
"Now",
"Dawn",
"Sunrise",
"Zenith",
"Sunset",
"Dusk"

Though some of them maybe actually are not so easy as I thought: Dawn, dusk, sunrise, sunset...

Hello!
I'm back and here is my first PR: https://github.com/chubin/wttr.in/pull/501
And:
"Morning", => Matin
"Day",  => Jorn
"Evening", => Vèspre
"Night",  => Nuèch
"Weather report for:",  => «Previsions metèo per :»
"Weather",  => Metèo
"Timezone",  => Zòna orària
"Now",  => Ara
"Dawn",  => Auròra
"Sunrise", => Alba
"Zenith", => Zenit
"Sunset", => A solelh colc
"Dusk" => Entrelutz
I have to say i don't personally make a clear different between dawn an sunrise, or sunset and dusk xD

I have to say i don't personally make a clear different between dawn an sunrise, or sunset and dusk

Actually, there is substantial difference; it is even possible
that there is dusk, but no sunset (or there is is a sunrise, but no dawn):
$ curl -ks v2.wttr.in/Mo-i-Rana?F | tail -5 ; echo
Weather: ⛅  Partly cloudy, +20°C, 49%, ↓6km/h, 1011hPa
Timezone: Europe/Oslo
  Now:    18:30:47+0200 | Dawn:    --------  | Sunrise: 02:33:20
  Zenith: 13:09:23      | Sunset:  23:41:35  | Dusk:    --------
Location: Mo i Rana, Rana, Nordland, 8622, Norge [66.3166,14.1496]

Translation:
It seems to work and it looks pretty cool:
https://oc.wttr.in/Tolosa, Occitània
curl https://oc.wttr.in/Tolosa,Occitània

@iago-lito it is, by the way, a good example of a case that I mentioned in #500
Ordinary weather services couldn't resolve Tolosa,Occitània,
they have no idea what it is, but not wttr.in.
Back to the translation:
help, messages, etc. work also.
v2.wttr.in translation was also added; it does not work yet, but it is a problem
with the v2 view itself, not with the translation.
@Quenty-tolosan, I added your account name (because I have no your real name)
to https://wttr.in/:translation
Please say (or make a PR) if the name must be hidden/changed

Brillant! Thinks for having taken the time to fix the resolution of Tolosa :D
It means a lot :D
I will have to shorten some strings.
A question: where do the strings for short name of day come from?

Quentin, thank you again!


Wait for delayed tasks before stopping test server

This is handy for automatic tests of asynchronous handlers that return a result before finishing their work. That is possible with asyncio.create_task.

I can see how this would be helpful, but I also can see how it might not be. Seems like it should be configurable. Can you add a wait_for_all_tasks (or some other) kwarg?

Since users cannot possibly handle worker processes' pending background tasks themselves, all tasks should always be run to completion by Sanic, until GRACEFUL_SHUTDOWN_TIMEOUT, after which I suppose they should be cancelled (by exception) but never just abandoned. Abandoning an async task easily leads to data corruption, orphaned processes left running and other severe problems.

@yunstanford and @Tronic , I agree with you both. which is why if it will be added it should be configurable and not required.

I don't see why user can not handle pending background tasks themselves, when launching background tasks, can always stored the task as app context for proper tracking... Personally, i don't think it's good practice to just launch sth. there and don't care, but in the very end assume the framework to handle that. Also shutting down those background tasks may also involve customized logic, e.g. the order could matter, etc.

@yunstanford Do you mean waiting for remaining tasks in a custom after_server_stop listener? After that the loop gets closed and the worker process terminates. All the rules break for any tasks still running: execution just stops, finally and with __exit__ are not run, functions don't return etc. Bad stuff.
One of the big points of Trio is to ensure that this can never happen. Any nursery blocks and eventually the trio.run() call won't exit until all tasks within have completely finished execution.

Closing. Please remake PR here.
https://github.com/huge-success/sanic-test


System Files and deferred bindings using database connection of parent model instead of default

Cross posted from #385 on behalf of @informas
I have tried to reproduce the problem raised in my previous posting regarding errors that appear when moving user tables (and associated) to a secondary database.
The following are the steps followed for me.
1) Create 2 new MYSQL databases:
winter_test_user (Main)
winter_test_user_data (Auxiliary)
2) Download WinterCMS
composer create-project wintercms/winter wintertestuser
3) Install WinterCMS
php artisan winter:install
   Database type [MySQL]: 0
   Database Name []: winter_test_user
   Would you like to change the backend options? (URL, default locale, default timezone) (yes/no) [no]: no
   Configure advanced options? (yes/no) [no]: no

5) Install plugin users
php artisan plugin:install Winter.User
php artisan plugin:list
+-----------------------+---------+-----------------+----------------+
| Plugin name           | Version | Updates enabled | Plugin enabled |
+-----------------------+---------+-----------------+----------------+
| Winter.Demo           | 1.0.1   | Yes             | Yes            |
+-----------------------+---------+-----------------+----------------+
| Winter.User           | 2.0.0   | Yes             | Yes            |
+-----------------------+---------+-----------------+----------------+

6) Copy the tables of the Winter.User plugin from the main database to the auxiliary database
+---------------------------------+
| Tables in winter_test_user_data |
+---------------------------------+
| user_groups                     |
| user_throttle                   |
| users                           |
| users_groups                    |
| winter_user_mail_blockers       |
+---------------------------------+

7) Create a new InforMAS.UserExtended plugin
php artisan create:plugin InforMAS.UserExtended
8) Modify the Plugin.php file of the created plugin, as follow:
<?php namespace InforMAS\UserExtended;

use Backend;
use System\Classes\PluginBase;

// Modules used by Winter.User plugin
use Winter\User\Models\User as UserModel;
use Winter\User\Models\UserGroup as UserGroupModel;
use Winter\User\Models\Throttle as UserThrottleModel;
use Winter\User\Models\MailBlocker as MailBlockerModel;

/**
 * UserExtended Plugin Information File
 */
class Plugin extends PluginBase
{
    public $require = ['Winter.User'];
    public $DB4Data = 'mysql_custom_data'; // Auxiliary DB for custom data

    /**
     * Returns information about this plugin.
     *
     * @return array
     */
    public function pluginDetails()
    {
        return [
            'name'        => 'User Extended',
            'description' => 'Clone de Winter User for work with tables in auxiliary database',
            ...
        ];
    }

    /**
     * Register method, called when the plugin is first registered.
     *
     * @return void
     */
    public function register()
    {
    }

    public function boot()
    {
        $this->extendModels();
    }

    protected function extendModels()
    {
        UserModel::extend(function($model) {
            $model->setConnection($this->DB4Data);
        });

        UserGroupModel::extend(function($model) {
            $model->setConnection($this->DB4Data);
        });

        UserThrottleModel::extend(function($model) {
            $model->setConnection($this->DB4Data);
        });

        MailBlockerModel::extend(function($model) {
            $model->setConnection($this->DB4Data);
        });
    }
}

9) Modify config\database.php
I add a new item to the connections array similar to mysql but changing the database name for the auxiliary database
        'mysql_custom_data' => [
            ...
            'database' => 'winter_test_user_data',
            ...
        ]

10) Refresh the plugin
artisan plugin:refresh informas.userextended
11) Enter in Backend
12) I'm trying to create a new user
I press the "+ New User" button and an error immediately appears
ERROR:
We're sorry, but an unhandled error occurred. Please see the details below.
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'winter_test_user_data.system_files' doesn't exist (SQL: select * from `system_files` where (system_files.id in (select `slave_id` from `deferred_bindings` where `master_field` = avatar and `master_type` = Winter\User\Models\User and `session_key` = 4eBL0FXKqn49pBY5cxcyPtlFpfa3YEq4dnJ1ldhW and `is_bind` = 1)) and system_files.id not in (select `slave_id` from `deferred_bindings` where `master_field` = avatar and `master_type` = Winter\User\Models\User and `session_key` = 4eBL0FXKqn49pBY5cxcyPtlFpfa3YEq4dnJ1ldhW and `is_bind` = 0 and id > ifnull((select max(id) from deferred_bindings where slave_id = system_files.id and master_field = avatar and master_type = Winter\User\Models\User and session_key = 4eBL0FXKqn49pBY5cxcyPtlFpfa3YEq4dnJ1ldhW and is_bind = 1 ), 0)) order by `sort_order` asc)
E:\websw10\wintercms\wintertestuser\vendor\laravel\framework\src\Illuminate\Database\Connection.php line 669

13) I copy the system_files table (which is currently empty) from the main database to the auxiliary database.
14) I'm trying to create a new user
I press the "+ New User" button and immediately another error appears different from the previous one
ERROR:
We're sorry, but an unhandled error occurred. Please see the details below.
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'winter_test_user_data.deferred_bindings' doesn't exist (SQL: select * from `system_files` where (system_files.id in (select `slave_id` from `deferred_bindings` where `master_field` = avatar and `master_type` = Winter\User\Models\User and `session_key` = eRK6auj7sSyduuX6esRhwQVaGPcPpAQnFEGuM2mJ and `is_bind` = 1)) and system_files.id not in (select `slave_id` from `deferred_bindings` where `master_field` = avatar and `master_type` = Winter\User\Models\User and `session_key` = eRK6auj7sSyduuX6esRhwQVaGPcPpAQnFEGuM2mJ and `is_bind` = 0 and id > ifnull((select max(id) from deferred_bindings where slave_id = system_files.id and master_field = avatar and master_type = Winter\User\Models\User and session_key = eRK6auj7sSyduuX6esRhwQVaGPcPpAQnFEGuM2mJ and is_bind = 1 ), 0)) order by `sort_order` asc)
E:\websw10\wintercms\wintertestuser\vendor\laravel\framework\src\Illuminate\Database\Connection.php line 669

15) I copy the deferred_bindings table (which is currently empty) from the main database to the auxiliary database
16) I'm trying to create a new user
I press the "+ New User" button and it works perfectly
If instead I try the same situation on the table Groups (of users) ...
17) I remove the system_files and deferred_bindings tables from the auxiliary database
18) I'm trying to create a new group
I press the "+ New Group" button
It shows the form "Create User Group", I enter the corresponding data, but when pressing the "Create and close" button an error appears in a popup
POPUP ERROR
"SQLSTATE[42S02]: Base table or view not found: 1146 Table 'winter_test_user_data.deferred_bindings' doesn't exist (SQL: select * from `deferred_bindings` where `master_type` = Winter\User\Models\UserGroup and `session_key` = bKKpuCwcV1yD16UTXfCS7EbIY2x2uo72Ed1uLSmT)" on line 669 of E:\websw10\wintercms\wintertestuser\vendor\laravel\framework\src\Illuminate\Database\Connection.php

19) If I copy deferred_bindings table from main database to auxiliary database then I can create the groups without problems.
Take note: in the case of groups it seems that the system_files table is not necessary.
Any comments?
Thanks in advance!

I can see a rationale for attached data (files, deferred bindings) being stored in the same database as the parent model - for example, if one were setting up a secondary database that's perhaps more secure (or audited) for user data, it might be a security risk storing attachments or deferred data for those users in the main DB.
The only issues that I can see with this implied logic is that: a) this is not automated, so people who set up auxiliary databases will need to manually create the files and deferred binding tables on that DB, and b) the automatic truncation of old deferred binding records may not work.

It's also not documented behaviour AFAIK, and it should at least be configurable.

Probably because it's not an common scenario to be running multiple databases in this fashion, so it's probably never come up before. Now that we know it, we definitely should document it at a minimum, and look at making it configurable down the track. I'm just thinking it's not a "bug", per se.

@informas I believe this is fixed as of https://github.com/wintercms/storm/commit/2cd7370a6677b0dcd76c3d82101bc321de71d9f4


Charge mode resetting

Version of the custom_component
Latest from HACS
Describe the bug
I have a day/night rate with changes at 9am to day rate and midnight to night rate.
I created an automation to change my Zappi chargers from Fast to Stopped and Stopped to Fast mode at those times, to make sure cars only charge on the night rate.
This is working mostly fine, however every once in a while the mode re-sets immediately :

As you can see above for example, the automation changed the mode to Fast at midnight as expected, but 55 seconds later it changed itself back to Stopped without showing a source.
I don't use the myenergi app so that's not where that change originated.
I've seen this both ways (Stopped -> Fast -> Stopped like here and Fast -> Stopped -> Fast) a few times.
Any idea what could be doing that ?
Thanks

It's happening now again, even when I try to trigger it by hand.
 I suppose the integration tries to send the API call and the mode changes in HA, but the call fails and the change in HA is undone to reflect the fact that it failed.

They seem to be doing some major changes in their backend, hopefully things will improve when the dust settles :)

This is actually awful this week, my automation has failed every day since I created this issue, and even trying to manually do it in the morning takes ten or fifteen re-tries. And the morning is too late of course, only have time to get a few percents in.
I just edited my automation to do a repeat until the status stays changed, hopefully that'll help. The car is getting dangerously low on battery ..
Should that retry logic be added to the integration ? I went with a delay of 2 minutes to be safe, since it looks like it always 'rolls back' within 50 seconds.

Talked to Dr Chris from myenergi and it turns out the issue isn't the API, it's weak signal / interference between the hub and the charger.
My CT Clamp readings were good but I suppose the other direction (hub -> Zappi) wasn't great. I moved my hub a bit and it's been behaving a lot better, although still not great.
Anyway the retry loop in home assistant works well so I'll keep it, and for anyone seeing the same behavior try moving your hub closer.
As for why it started behaving horribly lately I have no explanation, but I did notice that my Aerial TV has also been getting wild SNR fluctuations, maybe there's something new around here causing the issue.
Anyway, I'll close this. Thanks

Weather is a big contributor to signal propagation, for example high air pressure improves performance; as does dry air, but low pressure and high rain rates are known to absorb RF energy as the energy heats the water molecules.
You will always get improved reception when air pressure is high as has been the case across much of Europe.
Electrical interference is also a contributory factor.

As an update I ended up buying a PoE splitter, which allowed me to move the hub into the crawl space right above the chargers. Works like a charm


[CLEANUP] Placed dependencies and plugins in parent pom

dependencyManagement and pluginManagement. Placed and defined remaining
properties for dependencies, plugins and features.xml bundles versions
in parent pom.

@graimundo, please review

@JoaoLMPereira added line comments
we use the test scope for dependencies (ex: mockito, bean matchers) we want to use while testing that are only needed by the test classes and should not be included at runtime
we need to build some test jars so that other modules can reuse test logic
provided scope is for dependencies that are to be looked for outside of karaf
${project.revision} is the only property that will correctly change from 6.0-SNAPSHOT to 6.0-NIGHTLY in feature files
there may be cases were we don't want to filter some resources, it's safer to have a separate folder that is obviously filtered so that people don't assume things with ${} remain as they are in the source file

Hello, thank you for the reviews so far. I am preparing a new commit, could you please review the new commit?
I pushed down the dependencies into dependencyManagement of project root pom. I did the same for the plugins, so the scope tests etc.. are described there. I have to change this to only push down shared dependencies and plugins. I am really sorry.

@JoaoLMPereira It's not a big deal. I just closed to prevent the merge button from being clicked accidentally and wanted to explain why :smile:.

@brosander do you see any drawbacks on defining both the version and scope on the parent POM instead of having them scattered in the child POMs with the explicit ${x.version} variable? IMO having them defined in the parent POM and using Maven inheritance for the children makes the children POMs less cluttered.
Regarding the ${notification.version} and other properties, what @JoaoLMPereira did was to create intermediate variables that still point to ${project.version}. So if the build team changes that one only property it should still work. Nonetheless, I am not fully aware of how the build team overrides these values so please let me know if I am wrong.
@JoaoLMPereira I agree with @brosander on the filtered resource directory. Although the name is not standard maven directory layout, with it is easier to understand what resources are filtered and what are not. Besides we are using this naming on other pentaho projects.

@graimundo @JoaoLMPereira I should have looked more closely at the parent pom, I'd mistakenly assumed the dependencies were all being converted to compile, if all of the scope removals were just changes to use inheritance, that's fine with me, sorry about that.
I also didn't realize that notification.version had been changed to use project.version.
So besides the filtered-resources (which wasn't that high on my list of concerns) I'm good with the rest.  Sorry for not looking more closely first.

No problem. Thank you for the feedback and looking into it :) @JoaoLMPereira will still need to check what should really go into the parent POM as opposed to have everything there ;)


[Device Farm] Add information about long waiting time and a way to minimize it

CI fail is a little bit strange. Not sure if it's because of my changes or different reason...
The command "awesome_bot README.md --allow-redirect --allow-dupe --allow 429 -w .amazonaws.com,news.ycombinator.com" exited with 1.

Nope, swagger.io had a header failure. With 1200 links or so, these things happen. The check is mainly to help us avoid bit rot!

@artemnikitin Thanks for the contribution-- I've made a couple of wording changes to your PR just for flow. If you're happy with them, I'll merge this in.
Also, you're one of the first people I've seen in the wild using Device Farm. Are you on the Open Guide Slack team? I'd love to ask you a few questions about your experience.

If you're happy with them, I'll merge this in.

I'm completely ok with it. No problem. You can merge.

Also, you're one of the first people I've seen in the wild using Device Farm. Are you on the Open Guide Slack team? I'd love to ask you a few questions about your experience.

I had no idea about this Slack existence. I will join it and feel free to ask any questions :)

@QuinnyPig is everything ok with pull request or something is still missing?

Sorry for the delay-- thanks for the commit!


feat: implement pages and checkpoints

Implement pages and position
Implement checkpoints to have values available to next modules
implement breaking changes module

Task linked: CU-862krj3ap feat: implement pages and checkpoints


ci(Mergify): configuration update

This change has been made by @rhahao from Mergify config editor.

:tada: This PR is included in version 2.44.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Feature Request: TCN Layer (temporal convolutional networks)

Describe the feature and the current behavior/state.
The feature is for adding the different layers introduced in An Empirical Evaluation of Generic Convolutional and Recurrent Networks for Sequence Modeling paper to tfa.layers. There is already existed an Keras implementation here: https://github.com/philipperemy/keras-tcn/, I feel like this is helpful and would have more visibility in tfa so I ask him in this issue if I can help out porting the TCN layers portion into tfa and he says yes :)
If you feel like this would be a good addition let me know, very happy to contribute.
Relevant information

Are you willing to contribute it (yes/no): yes of course :)
Are you willing to maintain it going forward? (yes/no): yes of course :)
Is there a relevant academic paper? (if so, where): yes An Empirical Evaluation of Generic Convolutional and Recurrent Networks for Sequence Modeling
Is there already an implementation in another framework? (if so, where):
No but was implemented in pytorch in this repo, https://github.com/locuslab/TCN, and Keras in this repo https://github.com/philipperemy/keras-tcn (will be working on porting the layers portion into tfa, asked owner for permission)
Was it part of tf.contrib? (if so, where): no

Which API type would this fall under (layer, metric, optimizer, etc.)
layer
Who will benefit with this feature?
Developers who want to experiment with TCN layers.
Any other info.
Github issue that initialize the conversation to port TCN layers from @philipperemy's repo to tensorflow addons (see last two comments in this issue).
https://github.com/philipperemy/keras-tcn/issues/73

Good news! This is indeed in the scope of addons. Feel free to file a PR for it. Thanks!

@WindQAQ thanks! Will do so when I have time :)

TensorFlow Addons is transitioning to a minimal maintenance and release mode. New features will not be added to this repository. For more information, please see our public messaging on this decision:
TensorFlow Addons Wind Down
Please consider sending feature requests / contributions to other repositories in the TF community with a similar charters to TFA:
Keras
Keras-CV
Keras-NLP


color transform color: #fff to text="[#fff]"

Current:

Excepet:
text-#fff or text-[#fff]

and bg="#5DA85E" border="[1px_solid_rgba(233,233,233,1)]". Is this expected logic?

yes, toUnocss is for attr, used like <div text="[#fff]" border="[1px_solid_rgba(233,233,233,1)]" />.
I add toUnocssClass that used in class can be transformed to text-#fff


Add Resiliency Metrics to SAM

Add resiliency metrics to SAM models, leveraging REopt API

added in PR SAM #235 and SSC https://github.com/NREL/ssc/pull/324


List mode: Translations

Task/Issue URL: https://app.asana.com/0/1207418217763355/1207932816245026/f
Smartling job URL: https://dashboard.smartling.com/app/projects/b01b53ede/account-jobs/b01b53ede:iuwowrxuokvu
Description
Translations for new strings.

@anikiki All the translations are in, this is ready for a review.


[boost] /safeseh

Describe the pull request

What does your PR fix?
Builds boost on windows x86 with /safeseh. Otherwise you get linker errors if you build your application with /safeseh but boost is not build with /safeseh.

Should boost always be built with /safeseh for all users in all circumstances?
Yes! It improves the security of the resulting application (see here) and has no downsides.

Related to old PR https://github.com/microsoft/vcpkg/pull/16142

should it be promoted to a toolchain flag?

should it be promoted to a toolchain flag?

Maybe not an bad idea. Maybe create a Issue or PR to test if the packages will build with it.


I cannot set the responseType in the HttpRequestConfig

When using my internet destination, I want to use other response types than 'json' (for instance to retrieve an image). For now I cannot override this setting as it is no part of the HttpRequestConfig. It is however a setting in the AxiosRequestConfig...
I now need to do this:
const axiosRequest = await buildAxiosRequestConfig({destinationName: destination, jwt: token}, config);
axiosRequest.responseType = 'arraybuffer';
const result = await axios.request(axiosRequest);

Hey @blaart,
thank you for your feedback. We have recently talked about allowing this configuration as well. It is good to know, that there is an actual need for this. We will most probably implement this, but it might take a bit until we get to it. We would still have to discuss the API design on this.
Are you trying to use this in an OData client or are you using executeHttpRequest directly?

Hi @marikaner,
No problem, I have a workaround.. I'm using this with a direct http call/destination (not OData).

I will close the issue since we have a backlog item to do this in the future and you can work around it currently.


UI: Allow org admin to hide change email and change password

Organizations using LDAP don't need the "Change Email" and "Change Password" UI elements.

We need to determine if the fact that LDAP auth is used is surfaced anywhere for the FE.

Also, this would likely be helpful in scenarios where a user authentication is OAuth.

Oauth I think we can get, I just wonder if we can use the existing PREVENT_EMAIL_PASSWORD for the other scenarios

This might need some further thought. I'm not entirely certain that preventing OAuth users from changing their email address is correct.
Let's take a scenario here:

A user<EMAIL_ADDRESS>authenticates with OAuth - a user account is created, and the ID from the OAuth provider is stored against the user
The user changes their email address (to<EMAIL_ADDRESS>
In this case:

The next time they try to authenticate with the original OAuth method, a new account will be created
If they re-authenticate with the same OAuth method, but with the new email address, the ID from the OAuth provider will be updated

So maybe, what we should do is to re-add the ability to change your email as an OAuth user, but we should clear the ID from the OAuth provider when an email changes?

we should clear the ID from the OAuth provider when an email changes

As long as we're not doing it silently, I'm in favour of this option.


File info button tooltip box empty

Description
File information button tooltip should say something i presume
Steps To Reproduce
Hover over button that shows file onformation, tootlip shows up but contains no text
Expected behavior
Actual tooltip in the tooltip box
Files Version
<IP_ADDRESS>
Windows Version
Windows 11 22H2 22598.100
Relevant Assets/Logs

This issue should be resolved with #9028


Controls chart: issues with logarithmic scale for SLA and Grid features

Describe the bug
SLA and Grid features use linear scale when calculating line position between adjacent axis ticks.
To Reproduce
Steps to reproduce the behavior:

Go to Controls chart
Choose a date when the blue "Rolling average" line is somewhere in the middle of two adjacent Y axis ticks (not too close to axis tick's value)
Set mouse pointer there and look at the "Rolling average" value
Set SLA to the same value
Enable linear Grid and drag it so that any its line match the "Rolling average" value for the chosen date

Expected behavior

SLA line should cross "Rolling average" on the chosen date
Value on the Grid line should be the same as "Rolling average" value on the chosen date

Screenshots
See the screenshot.
SLA is set to 6, but it's much lower than Rolling average's 6.3. It's where 6 is expected to be if we have linear scale.
Grid's 2 SP line says "8.5 days" that is somewhat bigger than expected ~6.3.

Desktop

JIRA 8

From the docs:

If the maximum Elapsed Time value on the chart is less than 30 days, then a linear scale is used for the y-axis. If it is 30 days or greater, than a cube-root power scale is used.


Recursive stackop

PR for other half of #10.
This change allows you to capture the next (or previous) element into the sibling form containing your cursor. Another way to say it is that capture will recurse up ancestor forms until it finds an element to capture and moves it into that ancestor.
See the issue for more details.
Cheers

To flesh out several distinct concepts of emit and capture, I've documented 3 candidate "emit" commands, and 2 candidate "capture" commands. Would appreciate feedback from @snoe and others on the usefulness of the proposed commands. If the consensus is that only one command in each set is particularly useful (or one is significantly more useful than the others), perhaps the best approach is to change sexp's current emit/capture to work accordingly. OTOH, if multiple variants are deemed useful, perhaps it makes sense to provide several distinct emit/capture variants; alternatively, a single emit/capture command could be provided, along with an option to configure its behavior.
Note: I'm not overly attached to the proposed names - just needed a way to distinguish between the various commands...
Note: In all of the examples below, | indicates cursor position.
Emit commands
Emit
sexp_emit_head_element
sexp_emit_tail_element
Description: Emit [count] elements from current list, preserving cursor position unless element under cursor is emitted from list, in which case, cursor will be moved to the list bracket.
Note: The only difference between this and the original sexp emit is that the original always positions the cursor on list bracket (even when element under cursor is not emitted).
Example:

Direction
Count
Before
After

tail
3
(((foo (a |b c)) d e))
(((foo (|) a b c) e))

tail
2
(((foo (|a b c)) d e))
(((foo (|a) b c) d e))

Note: I can see an argument in favor of removing lists made empty by the emit: e.g., the empty list before a in the 1st example above.
Barf
sexp_barf_head_element
sexp_barf_tail_element
Description: Emit a single element from current list [count] times, preserving cursor position relative to the element beneath it, even when the element under cursor is emitted from the list.
Note: This is essentially the snoe PR. It is also closest to emacs paredit behavior, hence the name.
Example:

Direction
Count
Before
After

tail
3
(((foo (a |b c)) d e))
(((foo (a) |b) c d e))

Eject
sexp_eject_head_element
sexp_eject_tail_element
Description: Push element under cursor out of [count] containing forms, pushing any elements in its path.
Note: AFAIK, there is no comparable emit command in either vim-sexp or emacs paredit.
Example:

Direction
Count
Before
After

tail
3
(((foo (a |b c)) d e))
(((foo (a))) |b c d e)

head
2
(((foo (a |b c)) d e))
((foo a |b ((c)) d e))

Capture command
Capture
sexp_capture_head_element
sexp_capture_tail_element
Description: Pull [count] elements into the current list from ancestor forms, preserving original cursor position.
Note: This command is similar to the original sexp capture, but allows elements to be pulled from lists other than the immediate parent.
Example:

Direction
Count
Before
After

tail
3
(foo ((a |b) (c d)) e f)
(foo ((a |b (c d) e f)))

head
2
(foo ((a b) (c |d)) e f)
(((foo (a b) c |d)) e f)

Slurp
sexp_slurp_head_element
sexp_slurp_tail_element
Description: Pull a single element into an ancestor list [count] times, preserving original cursor position.
Note: This is essentially the snoe PR. It is also closest to emacs paredit behavior, hence the name.
Example:

Direction
Count
Before
After

tail
3
(foo ((a |b) (c d)) e f)
(foo ((a |b (c d) e)) f)

@bpstahlman great write up. I think your capture and emit commands could work but would take some getting used to. There might be some small benefit to matching paredit because the commands are so ingrained across the ecosystem. On the other hand, if capture/emit feel more vim-like then I think that would fall inline best with this plugin.
Honestly I would be happy with any change that keeps cursor position getting to master; I see colleagues fumble around this issue all the time.

@snoe Thanks. So maybe the best path forward for now is your recursive_stackop branch. I like your basic approach, and the bugfixes seem manageable. Additional flavors of emit/capture could always be added independently at a later date if enough users deem them useful...
I strongly concur on the importance of preserving cursor position. While adding a few features on my own branch, (enhanced-sexp-objects), I added a non-trivial amount of logic to ensure cursor position is preserved across command execution. Incidentally, if you or your colleagues ever want to take a look at any of the features on that branch, I'd welcome any feedback. I've got a significant amount of code cleanup to do yet, and nothing's fixed in stone, but the features are all documented in their current form, and should be basically working. It started as a branch to enhance sexp objects to allow the ie/ae objects to expand with multiple applications (just like Vim's iw, aw, etc.), but I ended up adding several other features I find useful. Also, if you have an emacs background and haven't updated recently, you might be interested to know that convolute and several new motion commands were merged to master over a year ago.


[0.1.0] CTGAN Optimization

Problem

When large amount of real data is used to train a CTGAN model, the current implementation is not working well.
Since all the data (DataFrame) is loaded into the memory when training, this will cause huge memory consumption, which is not an elegant solution.
Proposed Solution

Fortunately, in this refactoring, sdgx provides the new DataLoader and the NDArryLoader under development.
We can use these new data-related components to modify the Data transformer, Data sampler, and CTGAN model.
The data will not be loaded into the memory all at once, instead, the data will be loaded into the memory in rows or columns (chunks) according to needs, then the data will be used to train the model.
This will effectively reduce memory consumption and provide larger data processing capabilities.
Additional context

TBD

CTGAN encodes all discrete columns one-hot, if there are random strings present, they will form a huge matrix during vectorisation, leading to memory overflow.
Based on this, we need to identify random discrete columns (and things like home addresses, names, etc. for random discrete columns) in DataProcessor and Metadata, and process them probabilistically or on-the-fly using tools like Faker.

CTGAN encodes all discrete columns one-hot, if there are random strings present, they will form a huge matrix during vectorisation, leading to memory overflow.
Based on this, we need to identify random discrete columns (and things like home addresses, names, etc. for random discrete columns) in DataProcessor and Metadata, and process them probabilistically or on-the-fly using tools like Faker.

In response to this problem, I will start the design of metadata and data processor, and update it in the issue or descussion section.


[#5865] Remove the link that appears to be linked by zero in the serv…

…er map

Codecov Report

Merging #5866 into master will increase coverage by <.01%.
The diff coverage is 69.23%.

@@            Coverage Diff             @@
##           master    #5866      +/-   ##
==========================================
+ Coverage   40.77%   40.77%   +<.01%
==========================================
  Files        2950     2950
  Lines       88675    88687      +12
  Branches    11665    11668       +3
==========================================
+ Hits        36156    36163       +7
- Misses      49383    49388       +5
  Partials     3136     3136

Impacted Files
Coverage Δ

...oint/web/applicationmap/DefaultApplicationMap.java
64.28% <69.23%> (+1.78%)
:arrow_up:

...pc/server/LoggingServerMessageListenerFactory.java
90% <0%> (-10%)
:arrow_down:

...p/pinpoint/rpc/stream/StreamChannelRepository.java
75% <0%> (-10%)
:arrow_down:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
80.59% <0%> (-2.99%)
:arrow_down:

...cluster/zookeeper/ZookeeperClusterDataManager.java
52.9% <0%> (-1.94%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
72.4% <0%> (-0.65%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
19.76% <0%> (+1.16%)
:arrow_up:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
59.57% <0%> (+2.12%)
:arrow_up:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
85.36% <0%> (+2.43%)
:arrow_up:

...tor/metric/datasource/DefaultDataSourceMetric.java
78.12% <0%> (+3.12%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


How to get a work or school email?

My issue is this: Powerapps is a strong 'citizen developer' tool, but how can someone in between jobs skill up on the platform? How do we get an Azure backed email? Would a free Outlook account be enough? Otherwise it's difficult to access. If there is a free/developer//trial tier of 365 which would provide the sufficient email type, please can you link to it or name it?

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

ID: 43c7f027-9449-93d3-d418-c1a62a514e51
Version Independent ID: 281c1943-8e5f-94a3-fa21-c4744a8580e3
Content: Power Apps 30-day free trial - Power Apps
Content Source: powerapps-docs/maker/signup-for-powerapps.md
Service: powerapps
Sub-service: common
GitHub Login: @alaug
Microsoft Alias: alaug

Thanks for your comment, @zorbag. This issue has been assigned to the doc author for review.


gifs preview for foot in lf

so basically I added
preview gifs .gif {{
    chafa -f sixel -s "$w"x"$h" "$f"
}}

to my ctpv config
if I type ctpv filename.gif
it will work perfectly but if I try to preview the same gif with lf It will not work

You can try adding exit 1 at the end of your preview. It's needed to prevent lf from caching the preview.

But that is wrong, ctpv is supposed to handle gifs by default, yet it doesn't. The problem is that chafa tries to animate the gif, which doesn't work in lf. I attempted adding an option to the chafa call to disable animation but some users reported that their image previews broke after that. I suspect it's because this option was introduced somewhat recently and many distros still use old versions of chafa.
AFAIK chafa also provides a library, maybe I can statically link it with ctpv and use internally? This way the gif previews will work everywhere and I won't have to rely on whatever chafa version user has installed on their system.

well I wanted the gif to be animated in the first place that's why I got through the hassle of creating my own commands
so as I understand lf doesn't support animation right

No, apparently it does not, which is a pity. But maybe you can try ueberzug if you use X11? I used to use it and, if I remember correctly, gifs were animated.

ohh I didn't know that nice
since I'm on wayland and I really don't want to switch then I guess I should wait until lf support sixels properly in the first place to ask for this feature request

https://github.com/jstkdng/ueberzugpp might help!
It's a drop-in replacement for ueberzug that supports Wayland and gif images.


Allow disabling particular binder from health

My app uses binders for rabbit and kafka. When rabbit is down my app should return 200 OK on actuator health. However when the kafka is down it should not return 200 OK. With current API I need to use nasty reflection to achieve this behavior:
import java.lang.reflect.Field;
import java.util.Map;
import lombok.RequiredArgsConstructor;
import org.springframework.cloud.stream.config.BindersHealthIndicatorAutoConfiguration.BindersHealthContributor;
import org.springframework.context.ApplicationListener;
import org.springframework.context.event.ContextStartedEvent;
import org.springframework.stereotype.Component;

@Component
@RequiredArgsConstructor
public class DisableRabbitBindingContributor implements ApplicationListener<ContextStartedEvent> {

    private final BindersHealthContributor bindersHealthContributor;

    @SuppressWarnings({"squid:S3011", "sqid:S112"})
    @Override
    public void onApplicationEvent(ContextStartedEvent event) {
        try {
            Class<? extends BindersHealthContributor> bindersHealthContributorClass = bindersHealthContributor.getClass();
            Field contributorsField = bindersHealthContributorClass.getDeclaredField("contributors");
            contributorsField.setAccessible(true);
            Object contributors = contributorsField.get(bindersHealthContributor);
            if (contributors instanceof Map contributorsMap) {
                contributorsMap.remove("rabbit");
            }
        } catch (ReflectiveOperationException e) {
            throw new RuntimeException(e);
        }
    }
}

It would be nice to add some optional bean for filtering binders contributors.

It sounds like you are running the app in a multi-binder environment. Did you try to disable the health indicator in the Rabbit binder's environment? https://docs.spring.io/spring-cloud-stream/reference/spring-cloud-stream/health-indicator.html.

Closing the issue due to no activity. Feel free to re-open it if you are still facing this issue.


Select environment for ports

Feature:
I'm on Windows, using VSCode with Remote SSH extensions to a linux machine.
Now the device I want to connect to is not on the linux but on my local (windows) machine.
I wanted to see the serial connection stuff in vscode but sadly I can only see ports like "/dev/ttyS0" but no "COM4" and so on.
COM ports do show but only when I open a new vscode window and do not connect with the remote extension.
Is there a way to select which environment (remote , local) it looks for ports? This would be super useful, otherwise I constantly have to switch back and forth from other tools.
Have a good day!

I just found out it is possible, by looking more through other issues searching for "environment" this time which i didn't before.
anyway found this https://github.com/microsoft/vscode-serial-monitor/issues/77
and apparently i have to put this:
    "remote.extensionKind": {
        "ms-vscode.vscode-serial-monitor": [
            "ui"
        ]
    }

Now it lists the COM ports!

Glad you figured it out! Closing this issue.


Display proper messages according to wallet connection status. Design reference.

https://aeternity-blockchain.atlassian.net/browse/SW-722
Story: Display more informative messages replacing the missing data when SH wallet is offline. Note that this proposed design is a placeholder for offline status messages that can be used in different places according to actual needs. It might be the case that some of the suggested cases are not needed because SH wallet can actually fetch the data or display last available data. Currently the wallet is displaying .chain names, assets and balance even if the device is not connected to the Internet. We should discuss and decide whether we want to keep it this way. @Paolo
Acceptance criteria:
Messages (displayed at the bottom of screen) according to wallet connect status:
You are offline. Go online to get real time data.
Unable to connect to the chosen node.
Middleware is offline.

Not connected icon + message to be displayed at the place of missing data such as balance, transactions history, assets or .chain names owned by the account:

Unable to load balance.
Assets list is not available offline.
Transactions history is not available offline.
Names list is not available offline.

“Unable to load balance.” message for the account card.

Design:

https://github.com/superhero-com/superhero-wallet/assets/147718725/9a4882cd-7194-426b-a3d8-fa31f99b8102

Figma Reference: https://www.figma.com/file/3oGLWzSH0oJljo4RETZtur/Superhero-Wallet-UI-(✔Updated)?node-id=22724%3A314628&t=mBfGnff3xvTQWXRa-0Connect your Figma account

I would remove "list" from wording as it is too technical: "Assets list" => "Assets", "Names list" => "Names"


@ Reply mentioning user under user-popover should add mention to the end of compose

When you go under user-popover and click "Reply mentioning user," the mention should be appended to the text in the compose box. If none, it should just be added, which is the current implementation.

@zulipbot claim

@samatdav thanks for working on this! One way to solve this issue is to git grep .mention_user which is the class name of those buttons. However, the lines of code that are relevant to you are in static/js/popovers.js. Lines 543-552 and 572-581.

Pretty much the situation you are seeing is when textarea.val('@**' + name + '** '); erases the compose box and adds the mention at the beginning. You need to change this.

@lonerz Thank you! I will take a look!

@samatdav feel free to mention me if you need help :)

@zulipbot label "bite sized"

$("#new_message_content").caret() (from the jquery-caret library) will nicely add things at the end; we use this for the emoji picker (etc.) and ti's probably what you want for this.

@samatdav are you still working on this?

Yes, I would still want to fix it if there is time. I am sorry for the
delay.
On Mar 13, 2017 5:24 AM, "Joshua Pan"<EMAIL_ADDRESS>wrote:
@samatdav https://github.com/samatdav are you still working on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/issues/3945#issuecomment-286000914, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJDh7nNhy1pzod9o4sVMOh8xoN8xRxTBks5rlKjegaJpZM4MTYK-
.

I just created a [WIP] PR for this issue.
Initially I tried to only change val to caret at textarea.val('@**' + name + '** ');
I think this is already a better that the existing approach because it does not delete the inputted text anymore. However, this way, the '@**' + name + '** ' is always added to the beginning of the message since the cursor position is lost while clicking to user-popover.
Thus, I present for your review an approach, where the position gets stored in
var position = $("#new_message_content").caret();
This way, the '@**' + name + '** ' gets added to where cursor last was, just like emoji do.
What do you think? I think the second approach is better, but I am not sure if having a variable to store the positions is the best way. Or maybe the first approach would be better because of the simplicity of change and a better behavior in some interaction scenarios.

I'm not sure; let's just start with adding it at the beginning, and then see what feedback we get from users.  Might not be important to do something fancier, and there's a good amount of complexity there.

@lonerz if this issue is still not solved, I would like to work on this. Can I?

@shreyanshdwivedi sure

@zulipbot  claim

This was fixed in dcac002213c231b6f4a810610e28ca56195085a4.