Not compatible with React 16.9.0?

I'm on a React 16.9 project and when I build, I get the following error:
node_modules/ditto-react/dist/index.modern.js
@monument/patient: Cannot find module: 'react/jsx-runtime'. Make sure this package is installed.
From what I understand, this error message means a React upgrade is required. Is there possibly a workaround to get ditto working otherwise?

Hi @BrandLibel, thanks for opening!
Internally, we're using ditto-react on React version 16.12.0. I also found a template online that I tweaked to spin up a React project on version 16.9, and I was able to integrate the latest version of ditto-react without any apparent problems.
Is it possible to make a minimum reproducible example of this for us to look at?

@azjgard thanks for sharing that you got it working on React 16 versions. I actually tried upgrading from react 16.9 to 16.14 and this ended up fixed it for me. Not sure how you got 16.9 working but I'm guessing there was a change somewhere between React 16.9 and 16.14 that added the necessary module


Multiple occation of mutable default argument

Checklist

[ ] I'm reporting a bug unrelated to a specific site
[x] I've verified that I'm running yt-dlp version 2021.09.02
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] The provided URLs do not contain any DRM to the best of my knowledge
[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[ ] I've read bugs section in FAQ

Description
It caught my attention, that there are numerous occasion of mutable default argument in function or method definition.
These can cause very nasty behaviour and generally needs to be avoided.
I'll try to collect and fix them later. Till then here are a couple of examples:
https://github.com/yt-dlp/yt-dlp/blob/7738bd32722154a26f70006e0fe586f40d06e606/yt_dlp/downloader/__init__.py#L10
https://github.com/yt-dlp/yt-dlp/blob/7738bd32722154a26f70006e0fe586f40d06e606/yt_dlp/extractor/adobepass.py#L1384
https://github.com/yt-dlp/yt-dlp/blob/7738bd32722154a26f70006e0fe586f40d06e606/yt_dlp/extractor/brightcove.py#L471
https://github.com/yt-dlp/yt-dlp/blob/7738bd32722154a26f70006e0fe586f40d06e606/yt_dlp/extractor/common.py#L654
https://github.com/yt-dlp/yt-dlp/blob/7738bd32722154a26f70006e0fe586f40d06e606/yt_dlp/extractor/common.py#L717
...
and not just dict, but lists also.

This is inherited code from youtube-dl. I was aware of the issue and have already fixed a few instances that were more problematic. These ones are currently not doing any harm. ofc, it is best to fix them anyway


Backport Qt 5.5 improvements to release-15.09

These are all generic Qt 5.5 improvements that I could find on master that isn't in the stable branch.
Currently testing the change (DON'T MERGE YET).

I haven't run nox-review yet (low disk-space and I have some other CPU hogging processes on my system today). But one of the goals of this PR was to be able to use Qt 5.5 in QtCreator, and that doesn't work. It fails the same way on master*:
$ ./result/bin/qtcreator
file:///nix/store/zmpf6ydrjdydd85wh6splpywv6aj4782-qtcreator-3.4.2/share/qtcreator/welcomescreen/welcomescreen.qml:31:1: module "QtQuick" is not installed
     import QtQuick 2.1
     ^

This breaks the welcome screen (all white / blank).
* I use the following patch to build qtcreator with qt55:
diff --git a/pkgs/development/qtcreator/default.nix b/pkgs/development/qtcreator/default.nix
index 8dabd33..f920226 100644
--- a/pkgs/development/qtcreator/default.nix
+++ b/pkgs/development/qtcreator/default.nix
@@ -16,7 +16,7 @@ stdenv.mkDerivation rec {
     sha256 = "1asbfphws0aqs92gjgh0iqzr1911kg51r9al44jxpfk88yazjzgm";
   };

-  buildInputs = [ makeWrapper qtLib.base qtLib.script qtLib.quickcontrols qtLib.declarative ];
+  buildInputs = [ makeWrapper qtLib.qtbase qtLib.qtscript qtLib.qtquickcontrols qtLib.qtdeclarative ];

   doCheck = false;

@@ -45,7 +45,7 @@ stdenv.mkDerivation rec {
     Categories=Qt;Development;IDE;
     __EOF__
     # Wrap the qtcreator binary
-    addToSearchPath QML2_IMPORT_PATH "${qtLib.quickcontrols}/lib/qt5/qml"
+    addToSearchPath QML2_IMPORT_PATH "${qtLib.qtquickcontrols}/lib/qt5/qml"
     wrapProgram $out/bin/qtcreator \
       --prefix QT_PLUGIN_PATH : "$QT_PLUGIN_PATH" \
       --prefix QML_IMPORT_PATH : "$QML_IMPORT_PATH" \
diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix
index a91d3da..e682ea2 100644
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -8107,7 +8107,7 @@ let
   });

   qtcreator = callPackage ../development/qtcreator {
-    qtLib = qt54;
+    qtLib = qt55;
     withDocumentation = true;
   };

qtcreator wrapper on master:
$ cat ./result/bin/qtcreator
#! /nix/store/xag5ayq906w9zhlxs8wayv4kvpiyqphq-bash-4.3-p42/bin/bash -e
export QT_PLUGIN_PATH=/nix/store/msack3cmh2p7kf5bcp26cqkwaky7rlyx-qtbase-5.4.2/lib/qt5/plugins:/nix/store/8bx376rlb148x6r3l2d4khmmfidblwh8-qttools-5.4.2/lib/qt5/plugins:/nix/store/87a2w6l3zzq1639fvsqzpxhpafpkvdrf-qtdeclarative-5.4.2/lib/qt5/plugins:/nix/store/k5r82x2xkqszjmqkq56zzgdi7n1scchm-qtsvg-5.4.2/lib/qt5/plugins:/nix/store/2wx1xyd3c8diwxv0j8xkzi6hz52qsg34-qtwebkit-5.4.2/lib/qt5/plugins:/nix/store/4zmw4j52c12c687m26gc9q524imwbw1d-qtlocation-5.4.2/lib/qt5/plugins:/nix/store/0anpmz654yfsaz5ygjv8x4xp3ghkd2rs-qtmultimedia-5.4.2/lib/qt5/plugins:/nix/store/xzbza4qh0f8adlsf955896bkv2spyk2b-qtsensors-5.4.2/lib/qt5/plugins${QT_PLUGIN_PATH:+:}$QT_PLUGIN_PATH
export QML2_IMPORT_PATH=/nix/store/87a2w6l3zzq1639fvsqzpxhpafpkvdrf-qtdeclarative-5.4.2/lib/qt5/qml:/nix/store/2wx1xyd3c8diwxv0j8xkzi6hz52qsg34-qtwebkit-5.4.2/lib/qt5/qml:/nix/store/4zmw4j52c12c687m26gc9q524imwbw1d-qtlocation-5.4.2/lib/qt5/qml:/nix/store/0anpmz654yfsaz5ygjv8x4xp3ghkd2rs-qtmultimedia-5.4.2/lib/qt5/qml:/nix/store/xzbza4qh0f8adlsf955896bkv2spyk2b-qtsensors-5.4.2/lib/qt5/qml:/nix/store/g18b9lxwr023m2l0iq2a1l89c9c0z184-qtquickcontrols-5.4.2/lib/qt5/qml${QML2_IMPORT_PATH:+:}$QML2_IMPORT_PATH
exec -a "$0" /nix/store/k65jp38x3v6k1cxxmwil9s717mgynd9i-qtcreator-3.4.2/bin/.qtcreator-wrapped "${extraFlagsArray[@]}" "$@"

qtcreator on master + build with qt55 (see patch in previous post):
$ cat ./result/bin/qtcreator
#! /nix/store/xag5ayq906w9zhlxs8wayv4kvpiyqphq-bash-4.3-p42/bin/bash -e
export QML2_IMPORT_PATH=/nix/store/8nzfzjd1ihkv7djl9wyasj96gzjr7mrb-qtquickcontrols-5.5.0/lib/qt5/qml${QML2_IMPORT_PATH:+:}$QML2_IMPORT_PATH
exec -a "$0" /nix/store/zmpf6ydrjdydd85wh6splpywv6aj4782-qtcreator-3.4.2/bin/.qtcreator-wrapped "${extraFlagsArray[@]}" "$@"

Ah, it's missing qtdeclarative. This works:
QML2_IMPORT_PATH=/nix/store/mvcib9a1dwyq8w7dma1pnpi53h7n3snx-qtdeclarative-5.5.0/lib/qt5/qml/ ./result/bin/qtcreator

I added qtdeclarative to qtcreator in ff4d8513b435cbb9858a6e714f5e50e0d9ae069a. CC @ttuegel.

nox-review succeeded locally.

According to nox-review (which completed successfully), it only touches these derivations:
qtcreator stellarium teamspeak_client qt5Full. I also tested qtcreator with qt55 (local change). It works.
I think it's ready for merge.

Any objections?

Looks good to me. Thanks!


docs: added missing config arguments for docker example

Codecov Report
Patch coverage has no change and project coverage change: -0.04% :warning:

Comparison is base (6c49772) 71.06% compared to head (babeedb) 71.02%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #218      +/-   ##
==========================================
- Coverage   71.06%   71.02%   -0.04%
==========================================
  Files          27       27
  Lines        2737     2737
==========================================
- Hits         1945     1944       -1
- Misses        670      671       +1
  Partials      122      122

see 1 file with indirect coverage changes

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


-skip-mime for pull

First, let me thank you for this program.  It does a very nice job and is the only reason I am able to do the current project I'm working on.  Thank you for all of your hard work.
I am working on a digital picture frame.  I would like to pull jpeg's from my google drive but skip the mp4 movie files.  The documentation states that pulling by matches is supported.  The program itself states that the -skip-mime="" flag is supported.  I can't seem to get either to work.  I know that in issue #498, a user asked for -match-mime support in list.  I am able to make that work, sort of (it lists the directories instead of the files).  You suggested a PR for the -match-mime for pull command.  If i had the knowledge to do it myself, I would.  Things I have tried:
-skip-mime="mp4"                        #mp4 files are still listed for pull
--match jpg                 #mp4 files are still listed for pull
-match-mime jpg                  #flag not recognized
I tried to pipe drive list -skip-mime into drive pull.  This froze the console.
This may just be error on my part.  I tried everything I could and read the wiki multiple times before asking.  I am using drive version 0.3.4 with Raspbian 8.0 on the console.
Thanks in advance,
Pat

@pdneumann, the mime-type for mp4 is "video/mp4", not "mp4". I don't have a folder with mp4 files, so I couldn't test that mime-type, but I could successfully use it with: drive list -skip-mime="application/pdf". It skipped all the PDF files in the directory.

Thank you for your response. I tried so many different forms of syntax with the commands I listed that I couldn't remember them all.  So in a directory with mp4 files, drive list -skip-mime="video/mp4" DIRECTORY works as expected, listing all but the mp4 files. However, if I use drive pull -skip-mime="video/mp4" DIRECTORY the mp4 files are listed to be pulled.  It does work as if -skip-mime is implemented for the pull command though.

Hey there @pdneumann thanks for reporting this issue and welcome to drive.
My apologies for the late response, I am totally swamped with work and school.
This is an actual bug in pull because a plain pull doesn't have skip-mime but pull matches might.

@odeke-em please don't apologize. This app is great, I was asking a question because I thought I was doing this incorrectly.  If my syntax is correct:
drive pull --matches -skip-mime="video/mp4" DIRECTORY
Results in response "Everything is up-to-date." even if I know that it is not.
drive list --matches -skip-mime="video/mp4" DIRECTORY
Results in "no matches found!"
I wasn't originally using --matches because I wasn't confident about how to use that flag even though I did see it in the wiki and did briefly try to figure it out. The above may be in the wrong syntax and if so I apologize. Thanks again for all the hard work!

@pdneumann thank you for the kind words. In deed as you have further noticed this bug is present for pull, I probably might have ripped it out or hadn't fully implemented it. However, it works alright for list of which it is listed for list in the README. This definitely needs some fixing.


how to add example values in api json to use in swagger-ui.html

Hi

Using latest version: 2.9.0
This is a question how to do

Swagger api-docs and swagger-ui.html is very much useful.
I'm looking for below one

how to provide example values (may be in path, query, object) while running the "try this operation" while using swagger-ui.html
(example values, to supply, may be in the form of annotations in RestController, or in the Swagger configuration level)

Thanks
Kishore

For scalar examples and for complex examples.


add github link to index

there is no obvious link to gh on the frontpage; there should be

Windows build test failures unrelated.

also yay, rtd builds working!


Chore/update packages

Short summary
update packages recommended by snyk
Link to issue:
PR Checklist

[x] I have performed a self-review of my own code
[x] I have written a short summary of my changes in the PR
[ ] I have linked related issue to the PR

[!TIP]
To deploy the PR to Test, use the Manual deploy fusion app TEST🚀 action.
Remember to deploy any backend changes to Test as well!

[!CAUTION]
⛔ I understand by merging my PR, the changes will be deployed to production immediately ⛔

Please deploy changes to test before merging, just as a precaution

Please deploy changes to test before merging, just as a precaution

Already done :)


Move Create[Metric] from Extensions to MetricFactory

I have issue when methods with conversion String to Tuple cannot be used from Extensions without timestamp since v5.
Look commit: https://github.com/prom-client-net/prom-client-healthchecks/commit/ced97c89b4cfda3f2b0e32764b268855bc60474f

You should reference namespace Prometheus.Client. for proper extension resolution. Not sure how it worked before =)

@sanych-sun I checked this, doesn't work. I beleive, that some extensions better move to MetricFactry it is more consistency and this fix issue.


Fun Facts section is a bit misleading

I think this data is pretty cool, but the "Fun Facts" section feels a little misleading to me.
Using my page as an example:

I definitely do not prefer camelcase.
I've used ruby for closer to 15 years, but SVN was a thing before github existed 😛
Same for JS, CSS, HTML
My most "starred" repo is in Crystal, as well as I contribute to mainly Crystal repos these days. What would be the metric for it say how long I've been using Crystal?
I think it's saying I'm most productive between noon and 6pm, but I take lunch from 12-1, and leave the office around 4:30pm... This may be accurate, but I'm not sold on it. I'm also not really sure why some of the bars in that graph are light gray and some are dark almost black.

On a side note, if I just create a javascript repo and accidentally committed my node_modules directory, would that really mess up this data? ... not that I've ever done that before 🤐

@yaronskaya - could you debug this case?

Hi @jwoertink! Thank you for your comments.

We are debugging that.
2, 3. Currently, we don't support svn repositories, so the Ruby has 10 years.
We've added how long you've been using Crystal.
The dark means night time:) How does the graph should look like to your perception?

Maybe it should say something along the lines of "I've used Ruby for at least 10 years" and maybe add an info button that explains that the respective information is deduced from GitHub (or GitLab) activity and might not be 100% accurate.

Yeah, I like the "for at least", even though it does make it a bit verbose. Maybe could use ">= Xyrs"? I think it's all starting to make a bit more sense now. Thanks @yaronskaya!

Fixed calculation of variable naming fact. Should be snake_case now.
We will add mention of possible older contributions soon. Thank you @jwoertink!

It's looking great! Thanks for the update.


click and dblclick are not exclusive?

What I'm trying to do is based on
"Zoom to Bounding Box II - Updated for d3 v4"
https://bl.ocks.org/iamkevinv/0a24e9126cd2fa6b283c6f2d774b69a2
I have the first click to do exactly what that map does.  Then after than initial (single) click I ignoring single clicks and zoom via wheel or double click. That is, pick a state, bring it "forward" and then drill down. So far, so good.
The problem is, if the initial click is actually a double click, those clicks seem to be treated by D3 as two successive single clicks.  Even with  svg.call(zoom).on("dblclick.zoom", null).
What happens is the first click starts but before it can finish the second click cuts it short and stops the transform before it's complete.  That is, before waiting to make sure a click is not one of two (i.e.,dblclick) D3 takes off on the first click, but (afaik) makes to effort or option to control / interpret a second click.
I've tried preventDefault(). I've tried flag to change configuration of the d3 object. For example, remove svg.call(zoom)  from svg.call(zoom).on("dblclick.zoom", null) after the first zoom start. Nuttin'.
Something isn't right. But maybe it's me? Is the UX outlined above not possible?

D3 overlapping click and dblclick aside, the solution that seems to be working for me is to not add  svg.call(zoom);  until the callback function for the zoom on("end", fn_cb)
This solution was REALLY difficult to uncover. Was it just me?

This sounds like standard browser behavior. A double click gesture consists of two click events followed by a dblclick event.
Please use Stack Overflow tag d3.js to ask for help. Stack Overflow provides a better collaborative forum: thousands of D3-related questions have been asked there, and some answers may be relevant to you.
When asking for help, please include a link to demonstrate the issue, preferably as an Observable notebook. It is often impossible to debug from code snippets alone. Isolate the issue and reduce your code as much as possible before asking for help. The less code you post, the easier it is for someone to debug, and the more likely you are to get a helpful response.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗


[FEATURE]: Create the Tooltip component for the Storybook

Is your feature request related to a problem?
This component is part of a reusable design system for Fonoster's dashboard. It helps achieve consistency and reduces repetitive implementation work.

Describe the solution you'd like
The Tooltip should follow the design in the Figma file and be implemented as a standalone React component with the following props:
The closest component to this in MUI is the Tooltip.
(Note: MUI’s Tooltip doesn’t seem to support multiline text, which is a requirement for our implementation)
Describe alternatives you've considered
Building the component from scratch if using MUIs as a base is impractical.
Additional context
Before working on this item, please check the CONTRIBUTING document. Details about the Tooltip component can be found here.
Acceptance Criteria:

[ ] Matches Figma design
[ ] Storybook story with documentation
[ ] Includes basic unit tests (if applicable)

Hi @psanders
I will like to take up this next

Ofc! All yours, friend.

Hi  @psanders
The Tooltip should follow the design in the Figma file and be implemented as a standalone React component with the following props:
The props weren't stated in the issue, here is what I am proposing
type TooltipProps = {
    content: string | string[],  // array of string for multiline text
    placement?: TooltipPlacement,
    children: React.ReactNode
}

type TooltipPlacement =
    | 'top'
    | 'right'
    | 'bottom'
    | 'left'

Is this okay by you ?

@obrucheoghene
Good catch. What would the children mean in this instance?

The children is the component that the tip will be on
example
  <Tooltip content="Request Video Call" placement="top">
       <Button>Video Call</Button> // button is children
   </Tooltip>

@obrucheoghene Thanks for clarifying. Makes perfect sense.
I agree with your proposed props.

Closed by #685


feat(Range) : using LEFT or RIGHT keys : knob is moved, but value is not modified, the event is not fired.

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Current behavior:
Using keyboard keys (left arrow or right arrow) to decrease or increase the ion-range value, the knob is moved, but the value model is not modified. Also, the (ionChange) event is not fired.
Expected behavior:
Moving Knob via LEFT, UP, DOWN or RIGHT keys on keyboard should modify the value model and fire the event ionChange as the knob is moved.
Steps to reproduce:
In a WEB browser with a keyboard, move the focus to the knob's . Then press LEFT, RIGHT,  UP, DOWN key to move the knob. The model value is not modified.
Related code:

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having? Thanks for using Ionic!

Hello,
You can see the bug on the demo in ionic doc/api/components/range.
https://ionicframework.com/docs/api/components/range/Range/
On the displayed phone on the right (displayed as demo on large screen),
1 => click above one of the ranges (in an empty zone) to set the focus on the phone app demo.
2 => Press the [Tab] key 1, 2 or 3 times.  One of the knobs will get the keyboard focus. You should see a selection square around one of the knob on a range.
3 => Now you can move the Knob using the keyboard arrows.
You can see that the displayed value in the badge never changes.
A lot of people like blind people, uses only keyboard to control applications, because they can't easily use a mouse. So this bug prevents them to use any application with Range.

Thank-you. Was able to duplicate.


No Download Certificate

Hi, it is from 27/09/21 that the certificates are no longer downloaded. What may have happened?

UP!

The used servers https://github.com/eu-digital-green-certificates/dgca-verifier-app-android/blob/main/app/src/acc/assets/verifier-context.jsonc#L22 and https://github.com/eu-digital-green-certificates/dgca-verifier-app-android/blob/main/app/src/tst/assets/verifier-context.jsonc#L22 give 404

Do you have a solution to solve?

Nope, unfortunately not. I used those servers myself and don't know what happened or if there are other servers/mirrors we could use.

C19 app work, can you use it?

If you wont, i have repo.

I have found this online: https://get.dgc.gov.it/v1/dgc/signercertificate/update

Please see https://github.com/eu-digital-green-certificates/dgca-verifier-app-android/issues/218#issuecomment-925566524
The test environment is a service provided for the EU member states and countries in the official onboarding process.
Due to the high amount of traffic it is only available during limited periods for testing in schedule with the EU onboarding processes.


C#: Autoformat

For C#, we like to keep all our QL code auto-formatted. Since we now sync new IR files with C++, that means autoformatting them as well (just like we autoformat the shared dataflow implementation). I hope that is OK.

@dave-bartolomeo and @rdmarsh2, shout if this interferes with your open PRs or other ongoing work.

I think we'll always have ongoing work that it could interfere with. I'd rather do it now rather than waiting around for a gap.


Website page development

A Responsive website page.

I'm GSSoC 24 contributor. I have recently completed my frontend development.I want to contribute in creating a website page. So I request you to please assign me this task!
@RituKumari98

I'm GSSoC 24 contributor.  So I request you to please assign me this task!
@RituKumari98


Error finding offsets

Iphone 6 plus ios version 12.1.3
Unc0ver 5.0.1/5.0.2
Errors

And

Yep same on all my iOS 12.4 devices. Right when the ad pops up it causes it to fail.

Yep same on all my iOS 12.4 devices. Right when the ad pops up it causes it to fail.

Same but i fixed this you must dowlnload an older version of unc0ver
Follow this guide and download unc0ver 4.3.1
https://youtu.be/hy1JbWBbJ3M

Yes.  unc0ver 5 and up, is meant for ios 13.


UMD bundle unsuitable because other libs deep-import

Last night's new version of store and core include UMD bundles in the box. That's great, but there is a problem.
Store, as I write this, in the very first line of https://github.com/ngrx/store/blob/master/src/store.ts  performs a deep import from core.
This runs into the exact same problem, when you try to actually use the UMD bundles, as we have seen with RxJS + UMD + SystemJS: the combination of UMD and the system loader is unable to "see" deep imports in a UMD. This problem was also present in Angular itself for quite a while, until numerous Angular source files were changed such that the various sub libraries import from each other at the top level only. Since the published RxJS API makes extensive use of deep imports, the solution there has been to just not attempt to use RxJS with system and UMD..
Possible solutions for NgRx:

Make core expose select at the top level façade, further follow the same rules as Angular itself, where each of the family of related libraries only imports from each other at the top level.
Make a custom store UMD bundle that includes the one tiny bit of core needs.

  And that means I'm still making my own bundle of store instead of using the shiny new one in the box.
(Of course this only affects those neural cases where people care about loading all these libraries via UMDs. Demo apps. Plunkers. Training material, that one is my use case.)

Update from @MikeRyan52 : Core already exports the select stuff at the top - Store can be made to use that.
I think that means there is no problem to solve any further from the "core" side. Things can be fixed by making store use the new top-level export.


modification of influxdb config file initiates a new setup process

Hi team, I am very new to influxdb and have been trying to set it up for the past 2 days.
I started influxdb with no config file presented, therefore it's all default; with the default setting I created a new user, org and whatnot.
Then I created a config file for influxdb; I was presented with the setup process again (new user, new password, etc); and every time I modify the same config file (e.g., delete/add a config entry), I was presented with a new setup process. Is this normal? If yes, is there reason why this is the case?
I am MX Linux-21.2 with Influxdb 2.6, Linux kernel version is 5.18.

That's not behavior I've heard of before. How are you running influxdb, Systemd/cli/etc? Can you share the options you are passing to it?


[Feature] Add Collide event listener

Note: for support questions, please use one of the following channels.
Reddit: /r/gbstudio
Discord: Join Chat
This repository's issues are reserved for feature requests and bug reports.

I'm submitting a ...

[ ] bug report
[X] feature request
[ ] support request => Please do not submit support request here, see note at the top of this template.

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

Feature.

What is the current behavior?

There's no way to trigger an event based on Sprite (Actor) collisions.

What is the expected behavior?

It would be helpful to be able to trigger events based on Actor collisions.

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

Many uses, but in my case, it would help in the constructions of a minimal combat system.

Please tell us about your environment:

Version: 1.0.0
Platform: all

I was looking for something like this. Is there any workararound?

I was looking for something like this. Is there any workararound?

You can use invisible projectiles to trigger collisions between actors


Carousel forward/backward tooltips are localized to initially set language, not updated language

OpenUI5 version: 1.108.0 (but really any version that supports Carousel)
Browser/version (+device/version): any (tried Chrome and Firefox)
Any other tested browsers/devices(OK/FAIL): nope
URL (minimal example if possible): https://plnkr.co/edit/YCVZLAms72cm9oY8?preview
Steps to reproduce the problem:

Open example above
Switch UI5 language to one that is not your browser language
Note how the localized text saying there are no table columns changes as expected
Hover over right Carousel button. Note how the text is in your browser language (assuming it's supported).

What is the expected result?
Text for the Carousel buttons should be displayed in the language set for UI5, like all other messages (e.g. Table in the example)
What happens instead?
Text is localized to the browser language
Any other information? (attach screenshot if possible)

Hello @rg1 ,
I made simplified sample: https://plnkr.co/edit/TqMfy6BjSGXXMKvq
And the Carousel arrows tooltips seems to work fine.
Best Regards,
Lidiya

So, I seem to be wrong about the browser language. Maybe it's the language initially set for UI5. After switching the language, the tooltips are off. Please reopen the bug.

Hello @rg1 ,
Please see the documentation for "setLanguage" method: https://sapui5.hana.ondemand.com/#/api/sap.ui.core.Configuration%23methods/setLanguage
It is described, that:
"The framework does not guarantee that already created, language dependent objects will be updated by this call. It therefore remains best practice for applications to switch the language early, e.g. before any language dependent objects are created. Applications that need to support more dynamic changes of the language should listen to the localizationChanged event and adapt all language dependent objects that they use (e.g. by rebuilding their UI)."
Best Regards,
Lidiya

@LidiyaGeorgieva You're correct, I did not see/remember, nor expect, that part of the documentation. Given that you're updating all the other language dependent strings I've encountered so far, I've come to expect this to work for native UI5 controls. Also, with this statement you're asking us to check every control we use. This makes me quite unhappy, but I guess I'll have to take it,

@rg1 this won't help you, but let me add some comments to complete the picture:
We discussed this feature already several times in the past (e.g. due to #2709, #1256, #741).
But management decided that the use cases for a dynamic language change of an already created UI are too rare to justify the significant effort for a complete solution. For the known use cases, a change of the language + restart seemed to be acceptable, albeit not being optimal for the end user.
Some details reg. the necessary effort can be found around https://github.com/SAP/openui5/issues/2079#issuecomment-1058908149 .


Setting ColumnsFilters create an error when working with expanded rows

Describe the bug
When I set a value for a columnFilters I have the following error :
Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application.
This only happens with the expanded configuration (This bug can be found in the example provided for the case in the doc)
Your minimal, reproducible example
Go to https://tanstack.com/table/v8/docs/examples/react/expanding
Steps to reproduce
1 . Go to https://tanstack.com/table/v8/docs/examples/react/expanding
2 . type something in one of the column input
3 . 💥
Expected behavior
being able to use column filtering and subrows together
How often does this bug happen?
Every time
Screenshots or Videos

Platform

MacOs

react-table version
8.0.0-beta.8
TypeScript version
4.5.5
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

yes, I'm running into this too. Max call stack size exceeded
Didn't realize it was the expanding rows that were causing it, but I find that to be the case too

same on 8.0.13

Seems to be fixed in the latest version.


[BUG] Can't get working with deno in AWS Lambda

Environment

chromium Version: 117
puppeteer / puppeteer-core Version: 21.4.1

Expected Behavior
The package should be installed and the script ran
Current Behavior
The AWS Lambda function fails with the following error code
Error: The input directory "/_virtual/esm.sh/v133/<EMAIL_ADDRESS>does not exist.
    at Function.executablePath (https://esm.sh/v133/@sparticuz/chromium@117.0.0/denonext/chromium.mjs:18:6013)
    at handler (file:///MY_SCRIPT_FILE)
    at file:///tmp/runtime.js:78:24
    at eventLoopTick (ext:core/01_core.js:165:11)

Steps to Reproduce
import puppeteer from<EMAIL_ADDRESS>import chromium from "https://esm.sh/<EMAIL_ADDRESS>
chromium.setHeadlessMode = true;
const browser = await puppeteer.launch({
  executablePath: await chromium.executablePath(),
  headless: chromium.headless,
  defaultViewport: chromium.defaultViewport,
  args: chromium.args,
});

At this point, we're cjs. Looking to change this very soon with the release of node 20 on Lambda.


Spoofing and leakage protection

Is there a way to avoid third party spoofing -for example, network sniffing and replaying the GET requests with different lat / long values? And... how secure is Traccar against known attacks such as SQL Injections, XSS etc, to avoid position leaks?
Thank you!

There is no protection against spoofing at the moment.
As for Traccar server security:

SQL injections are not a problem because Traccar properly uses parametrized queries
XSS does not apply because it's not possible to add any dynamic content

How about persistent XSS (via the Control panel's fields -or- via GET requests of lat & long values)? Thank you so much for the answer.

It's impossible to inject any dynamic code into any of the values.


Bug - Export Resultset to SQL Insert - Binaries: Hex, wrong values

I got table with column: attributes, BLOB, NOT NULL (DB: MySQL)
I did export resultset from filtered table and it generated this code:
INSERT INTO items (player_id,sid,pid,itemtype,count,`attributes`) VALUES
(1083505,101,3,2589,1,0x)
,(1083505,102,13,2589,1,0x)
,(1083505,103,101,2594,1,0x)
,(1083505,104,102,2594,1,0x)
;

When I tried to run it as query on other database, it returned error:

Unknown column '0x' in 'field list'

I had to replace every '0x' with '0x0' to run query.
For me it looks like DBEaver bug.

Cannot reproduce it for now.
If issue is still actual for you - feel free to reopen the ticket


feat: store proposal, vote and follower count on space

I've created 3 new columns on the "spaces" table, proposal_count, vote_count and follower_count, this get automatically updated if some action happening on the space, this will allows us later to remove some heavy queries that we have currently on the hub.

Need to update the schema.sql on the hub with the new column, and reflect it to https://github.com/snapshot-labs/snapshot-sequencer/blob/master/test/schema.sql

@ChaituVR What's blocking here ?


Wrong output type of function

Version
1.22.0
What happened?
sqlc generates function (pg_catalog.shobj_description) output type as "string", but function can return NULL.
In runtime I getting error from pgx "cannot scan NULL into *string".
Type overrides not work for function output types.
Relevant log output
No response
Database schema
No response
SQL queries
No response
Configuration
No response
Playground URL
https://play.sqlc.dev/p/fe73abcb3230f7757740515b6aa48e6fc022fd21f9d3ff413f24dca65ecb41b4
What operating system are you using?
macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

I looked into this. https://github.com/sqlc-dev/sqlc/blob/main/internal/engine/postgresql/pg_catalog.go#L24691 should get ReturnTypeNullable: true.
However, that file is generated by https://github.com/sqlc-dev/sqlc/tree/main/internal/tools/sqlc-pg-gen and I've failed to figure out how we can find whether a function can return NULL. I tried JOIN pg_catalog.pg_type t ON t.oid = p.prorettype and looking at t.typnotnull, but that seems to always be false. (Confirmed by https://www.postgresql.org/docs/current/catalog-pg-type.html, which writes "typnotnull represents a not-null constraint on a type. Used for domains only.").
Does anyone know how to figure out whether a function can return NULL programmatically?


[bug] [bitcoin-core test] '2147483647 DUP ADD<PHONE_NUMBER> NUMEQUAL'

Issue
The following bitcoin-core test is failing to give the correct result :
2147483647 DUP ADD<PHONE_NUMBER> NUMEQUAL

This code should be failing due to an error.
More info from the test: NUMEQUAL must be in numeric range
Replicate
To replicate the issue you can use the latest main branch with the following command:
scarb cairo-run '[[],4372357539058384277836571663884584193442884,18,[],1164013830132011234147657419853840811581325644,19]'

Please can I fix this bug?

@Oshioke-Salaki sure, I'll assign you for this one. Thank you


Sphinx >= 1.5a1 doesn't work

I've disabled this version of Sphinx in 541d3554de99568ff6bc82c45bcb02bea9b0766f.
$ ./test_rstcheck.py
F..FF.FF.FF..FFFF...
======================================================================
FAIL: test_check (__main__.Tests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "./test_rstcheck.py", line 59, in test_check
    """))
  File "./test_rstcheck.py", line 19, in assert_lines_equal
    self.assertEqual(set(line_numbers), set(dict(results)))
AssertionError: Items in the first set but not the second:
6

======================================================================
FAIL: test_check_doctest_in_code_block (__main__.Tests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "./test_rstcheck.py", line 329, in test_check_doctest_in_code_block
    """))
  File "./test_rstcheck.py", line 19, in assert_lines_equal
    self.assertEqual(set(line_numbers), set(dict(results)))
AssertionError: Items in the first set but not the second:
7
Items in the second set but not the first:
4

======================================================================
FAIL: test_check_doctest_in_python_code_block (__main__.Tests)
I'm not sure if this is correct, but I've seen people do it.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "./test_rstcheck.py", line 345, in test_check_doctest_in_python_code_block
    """))
  File "./test_rstcheck.py", line 19, in assert_lines_equal
    self.assertEqual(set(line_numbers), set(dict(results)))
AssertionError: Items in the first set but not the second:
7
...

https://travis-ci.org/myint/rstcheck/builds/166606676
This is related to #8.

This is because importing sphinx no longer has the side effect of registering roles and directives (https://github.com/sphinx-doc/sphinx/issues/2799).


Cannot connect to a VM Postgres through pgbouncer-k8s

Steps to reproduce
# This is not the exact order I used but it should get you to my state, I lost the steps and cannot retrace my steps at the moment
# Expect: lxd controller named lxd-local with a model named collectivedb
# Expect: microk8s controller named microk8s with a model named collectivedb-k8s
juju switch lxd-local:collectivedb
juju deploy postgresql --channel 14/edge -n 2
juju deploy data-integrator --channel edge
juju offer postgresql:database

juju switch microk8s:collectivedb-k8s
juju deploy pgbouncer-k8s --channel 1/edge
juju consume lxd-local:admin/collectivedb.postgresql
juju relate pgbouncer-k8s lxd-local:admin/collectivedb.postgresql
juju offer pgbouncer-k8s database

juju switch lxd-local:collectivedb
juju consume microk8s:admin/collectivedb-k8s.pgbouncer-k8s
juju relate data-integrator microk8s:admin/collectivedb-k8s.pgbouncer-k8s
juju config data-integrator database_name=test
juju run-action data-integrator get-credentials --wait

# This returns service name for the pod for the hostname, which does not work
# So, I get the pod IP from microk8s, which works slightly better
psql -h 10.x.x.x -p 6432 -U <uname> -d test
# First time around this gave an error about a query timeout, I lost the message
# Then, I went into the pgbouncer container to debug and I could psql to the database from there
# I can also psql from local machine
# After some poking and prodding the psql command above started failing to find pgbouncer
psql: error: connection to server at "<IP_ADDRESS>", port 6432 failed: Invalid argument
	Is the server running on that host and accepting TCP/IP connections?

Expected behavior
I expect to be able to connect to pgbouncer via Postgres client software.
Actual behavior
I get query timeouts or invalid argument errors as described in the reproduction steps.
Versions

Operating system: Ubuntu 22.04.2 LTS

Juju CLI: 2.9.44-ubuntu-amd64

Juju agent: 2.9.42

postgresql charm revision: 312
pgbouncer-k8s charm revision: 61

microk8s: MicroK8s v1.27.4 revision 5643
Log output

Juju debug log:
unit-pgbouncer-k8s-0: 14:05:37 ERROR juju.worker.uniter.operation hook "pgbouncer-pebble-ready" (via hook dispatching script: dispatch) failed: exit status 1
unit-pgbouncer-k8s-0: 14:05:37 ERROR juju.worker.uniter pebble poll failed for container "pgbouncer": failed to send pebble-ready event: hook failed
unit-pgbouncer-k8s-0: 14:05:44 ERROR unit.pgbouncer-k8s/0.juju-log Uncaught exception while in charm code:
Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/./src/charm.py", line 660, in <module>
    main(PgBouncerK8sCharm)
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/main.py", line 441, in main
    _emit_charm_event(charm, dispatcher.event_name)
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/main.py", line 149, in _emit_charm_event
    event_to_emit.emit(*args, **kwargs)
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/framework.py", line 344, in emit
    framework._emit(event)
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/framework.py", line 841, in _emit
    self._reemit(event_path)
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/framework.py", line 930, in _reemit
    custom_handler(event)
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/./src/charm.py", line 189, in _on_pgbouncer_pebble_ready
    container.replan()
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/model.py", line 1915, in replan
    self._pebble.replan_services()
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/pebble.py", line 1680, in replan_services
    return self._services_action('replan', [], timeout, delay)
  File "/var/lib/juju/agents/unit-pgbouncer-k8s-0/charm/venv/ops/pebble.py", line 1761, in _services_action
    raise ChangeError(change.err, change)
ops.pebble.ChangeError: cannot perform the following tasks:
- Start service "metrics_server" (cannot start service: exited quickly with code 1)
----- Logs from task 0 -----
2023-08-04T11:05:40Z INFO Service "pgbouncer_0" already started.
----- Logs from task 1 -----
2023-08-04T11:05:40Z INFO Service "pgbouncer_1" already started.
----- Logs from task 2 -----
2023-08-04T11:05:40Z INFO Service "pgbouncer_2" already started.
----- Logs from task 3 -----
2023-08-04T11:05:41Z INFO Service "pgbouncer_3" already started.
----- Logs from task 4 -----
2023-08-04T11:05:41Z INFO Service "pgbouncer_4" already started.
----- Logs from task 5 -----
2023-08-04T11:05:41Z INFO Service "pgbouncer_5" already started.
----- Logs from task 6 -----
2023-08-04T11:05:42Z INFO Service "pgbouncer_6" already started.
----- Logs from task 7 -----
2023-08-04T11:05:42Z INFO Service "pgbouncer_7" already started.
----- Logs from task 9 -----
2023-08-04T11:05:43Z INFO Most recent service output:
    ts=2023-08-04T11:05:43.783Z caller=collector.go:138 level=error msg="error setting up DB connection" err="error pinging pgbouncer: pq: password authentication failed"
2023-08-04T11:05:43Z ERROR cannot start service: exited quickly with code 1

Additional Info
Pastebin with some more information.

Hi, sounds related to https://github.com/canonical/postgresql-operator/issues/197. Can you retry using the workaround to open the ports mentioned there?

Not sure if I did it correctly but it did not work.
### Actions
$ juju run --unit postgresql/0 open-port 5432/tcp
$ psql -h <IP_ADDRESS> -p 6432 -U relation_id_2 -d test
psql: error: connection to server at "<IP_ADDRESS>", port 6432 failed: FATAL:  query_wait_timeout

# <IP_ADDRESS> is the IP of the pod running pgbouncer-k8s

### juju debug-log --replay
...
controller-0: 15:42:37 ERROR juju.worker.remoterelations cmr error in remote application worker for pgbouncer-k8s: opening facade to remote model: cannot resolve "controller-service.controller-microk8s.svc.cluster.local": lookup controller-service.controller-microk8s.svc.cluster.local on <IP_ADDRESS>:53: server misbehaving
controller-0: 15:42:53 ERROR juju.worker.remoterelations cmr error in remote application worker for pgbouncer-k8s: opening facade to remote model: cannot resolve "controller-service.controller-microk8s.svc.cluster.local": lookup controller-service.controller-microk8s.svc.cluster.local on <IP_ADDRESS>:53: server misbehaving
unit-postgresql-0: 15:43:07 ERROR unit.postgresql/0.juju-log Failed to list PostgreSQL database users: could not translate host name "None" to address: Temporary failure in name resolution

unit-postgresql-0: 15:43:07 ERROR unit.postgresql/0.juju-log Uncaught exception while in charm code:
Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-postgresql-0/charm/lib/charms/postgresql_k8s/v0/postgresql.py", line 338, in list_users
    with self._connect_to_database() as connection, connection.cursor() as cursor:
  File "/var/lib/juju/agents/unit-postgresql-0/charm/lib/charms/postgresql_k8s/v0/postgresql.py", line 107, in _connect_to_database
    connection = psycopg2.connect(
  File "/var/lib/juju/agents/unit-postgresql-0/charm/venv/psycopg2/__init__.py", line 122, in connect
    conn = _connect(dsn, connection_factory=connection_factory, **kwasync)
psycopg2.OperationalError: could not translate host name "None" to address: Temporary failure in name resolution

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-postgresql-0/charm/./src/charm.py", line 1226, in <module>
    main(PostgresqlOperatorCharm)
  File "/var/lib/juju/agents/unit-postgresql-0/charm/venv/ops/main.py", line 439, in main
    framework.reemit()
  File "/var/lib/juju/agents/unit-postgresql-0/charm/venv/ops/framework.py", line 843, in reemit
    self._reemit()
  File "/var/lib/juju/agents/unit-postgresql-0/charm/venv/ops/framework.py", line 922, in _reemit
    custom_handler(event)
  File "/var/lib/juju/agents/unit-postgresql-0/charm/./src/charm.py", line 799, in _on_start
    self._start_primary(event)
  File "/var/lib/juju/agents/unit-postgresql-0/charm/./src/charm.py", line 833, in _start_primary
    users = self.postgresql.list_users()
  File "/var/lib/juju/agents/unit-postgresql-0/charm/lib/charms/postgresql_k8s/v0/postgresql.py", line 344, in list_users
    raise PostgreSQLListUsersError()
charms.postgresql_k8s.v0.postgresql.PostgreSQLListUsersError
unit-postgresql-0: 15:43:07 ERROR juju.worker.uniter.operation hook "start" (via hook dispatching script: dispatch) failed: exit status 1
...

@mertalpt, I'm still trying to publish the new revision. I'm gonna update you here and on MM when it's done.

@mertalpt, revision 63 has the fix for this issue.


Support Cache Glyph Revison 2, issue #367

No Fonts desplayed on iOS with the Microsoft Remote Desktop 10.2.2(1291), issue #367

Overall, LGTM.
@jsorg71 @speidy Any thoughts?

Let's merge, it actually works fine for me. If any problems, consider & fix it later.


Ci/workflow trigger via open pr

Currently, the build is triggered manually or with a push.  This does not allow external contributors to trigger the build.  With this change, their opening of a new PR should trigger the build (though new contributors will still require approval for security reasons, to prevent malicious build modifications that do annoying things).
The creation of this PR should itself be an example.
There is more complex logic we can use to determine when a build is triggered (ie pr to a specific branch, pr is in a waiting approval state, pr has been approved, etc) but for now this should cover the use case.
Also, this PR contains the approved change from PR #1188 just to have it build (this branch was forked from that PR).

Thanks @DevopsGoth

I believe this can be safely closed, now that the PR it was related to is merged.


Vote validation code is duplicated in account_create_evaluator

Code in account_create_evaluator here checking committee size does not exceed number of witnesses voted for is duplicate of code in account_options::validate() here.  The latter code is called here so we can safely remove the duplicate functionality from the evaluator.

Since the code in the evaluator is common to account_create_evaluator and account_update_evaluator, let's factor it out into a common method while we're here.  This will ease implementation of #607 and any future change to vote checking logic.


there will be freeze on some low phones

Thanks code.
I found that there will be freeze on some low phones
https://github.com/megasanjay/vue3-marquee/blob/6cb63103711d5a77905e8f44bfad9086c82a7299/packages/vue3-marquee/src/vue3-marquee.vue#L307-L314
There will be a frame freeze for a period of time when text scrolling
Is this a problem with the phone? cpu not good ?
or
hava some way to avoid this problem ?

Are you using this for only text or something more dynamic?

just text and image. like that
    <Marquee>

xxxxx

    </Marquee>

bus the text content is large
the arr  is an array of length 10 .
This only happens on some phones. I think it might have something to do with the phone's GPU
Is there any way to circumvent this problem?
I can't find any alternatives to fix this right now . 😔. I have the same problem with canvas

To add some info to this, does a carousel on autoplay work for this issue?


Update fixes with V.2.0.0

Update fixes branch to refactor corrections that were made

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

307 of 307 (100.0%)  changed or added relevant lines in 15 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
442

Relevant Lines:
442

💛  - Coveralls


Reported line numbers are wrong for IJ1 Macro scripts

When running an IJ1 Macro script containing an error, the reported line numbers are off by 1+the number of script parameters. Likely an issue with the script pre-processing, but specific to the IJ1 macro language.
Test scripts:
x = nonExistent

(error reported to be in line 2)

#@ String (required=false) text1
#@ String (required=false) text2

x = nonExistent

(error reported to be in line 7)

This is a longstanding issue. It happens because in order to inject the parameters into an IJ1 macro, we must physically alter the script to prepend lines, then feed the altered script to IJ1.
In order to fix it, we would need to post-process the error message to parse the line numbers and adjust them accordingly.

Would it help to factor out the additional lines into a macro function, append it to the end of the macro (similar to how ImageJ1 does it with additional functions), and then only call this function in a single prepended line? This would at least reduce the offset of the reported line number to be always one, instead of increasing with the number of script parameters present.

Interesting idea! But I do not think it would work; I tested the following:
injectParameters();
print(greeting + ", " + name + "; x=" + x);
function injectParameters() {
	x = 1;
	name = "Curtis";
	greeting = "Hello";
}

And it gives the error:
Undefined variable in line 3:
print ( <greeting> + ", " + name + "; x=" + x ) ;

Does the macro language have a way to define a variable with global scope from within a function?

Oh, yes, I forgot about variable scope. We really miss the goto command in the macro language ;-D
The following works, but is useless indeed:
var x; var name; var greeting; injectParameters();
print(greeting + ", " + name + "; x=" + x);
function injectParameters() {
	x = 1;
	name = "Curtis";
	greeting = "Hello";
}

But I think this cannot happen, because the #@ ... line will be replaced with `` and thus the first (blank) line will never have errors.

It still can happen if the newly inserted lines contain errors (or, in case we add everything in the beginning of the first line, also if the first line contains errors).

I tried using this pattern for parameter initialization (see https://github.com/imagej/imagej-legacy/commit/b7bc93a89ad3f669d69be36d7d5125707f21f22b):
var x, name, greeting; initializeSciJavaParameters();
print(greeting + ", " + name + "; x=" + x);
function initializeSciJavaParameters() {
	x = 1;
	name = "Curtis";
	greeting = "Hello";
}

but it turns out that the bindings do not work when variables are declared global with the var statement in the macro language: IJ1MacroLanguageTest.testBindings is failing with the commit linked above.

It turns out the global variables were not correctly parsed: in the String [] array returned by getVariables(), the variable names contain a " (g)" suffix when they're global. Removing that suffix when present fixes the bindings.
The tests now pass, and the line numbers reported by Macro Error dialogs are now correct.
I noticed that in the Debug window, you can still see all values of variables, even when declared with style=password, but this was the case before this fix, and it might also be the desired behavior, as we expect to see all values when debugging, right?

It still can happen if the newly inserted lines contain errors (or, in case we add everything in the beginning of the first line, also if the first line contains errors).

What is the advantage of creating a function at the end of the macro, versus simply declaring all variables with assignments at the beginning of the first line? Either way, we have to modify the first line of the macro to include some extra characters.

What is the advantage of creating a function at the end of the macro, versus simply declaring all variables with assignments at the beginning of the first line? Either way, we have to modify the first line of the macro to include some extra characters.

The advantages would be:

we can keep modifications to the first line as short as possible (only variable names, no values)
the error message dialog would not contain any (potentially confidential) information about passwords etc. (although the debug window would still do)
the call("net.imagej.legacy.plugin.IJ1MacroEngine.saveInterpreter"); line would also be hidden from the error message dialog.

@imagejan Great, I understand now, and am convinced of the approach. I merged it.


Remove references to merge requests and update links GitLab->GitHub

🪄

Codecov Report

Merging #6022 (26de18d) into main (fb03ada) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #6022      +/-   ##
==========================================
+ Coverage   90.03%   90.05%   +0.01%
==========================================
  Files         264      264
  Lines       19046    19046
==========================================
+ Hits        17148    17151       +3
+ Misses       1898     1895       -3

Impacted Files
Coverage Δ

tests/meltano/cli/test_run.py
99.20% <0.00%> (+0.59%)
:arrow_up:

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 fb03ada...26de18d. Read the comment docs.


Help: week detection (read the post)

What do you need help for ?
Please note that the documentation is still under construction.
how can i detect from what week the song is from (for a freeplay menu)

PlayState.actualModWeek
its a structure with these parameters:

PlayState.actualModWeek

but everytime i use it it says:
invalid access to field actualModWeek am i doing something wrong?

i'm trying to use it like
if (PlayState.actualModWeek == 'weekendX')

is this right?


Is segmatch deterministic?

Hi all,
I have tried the loop closure demonstration and also some own datasets and noticed that different runs with exactly same parameters return different outcomes. I have made small changes to the code so I cannot determine if I introduced the non-deterministic behaviour.
Did you also notice non-deterministic behaviour in your experiments and have an idea where it comes from?

Yes, I have gotten that before. I think it is because SegMatch is multi-threaded.

Thanks @vkee, that was also my guess besides the random forest and the ROS message system.
Good to know that I didn't break its :)


♻ refactor: rebrand okp4 to axone

📋 Steps

[x] Rename contracts names
[x] Rename packages names
[x] Update docs with goods name
[x] Update CI
[x] Change assets link uploaded through .releaserc
[x] Change tests okp4 wallet address to axone wallet address
[x] Update docs
[x] Update template

⚙ Remarks

I've disabled the update-schema workflow on an old repository that is not maintained. (https://github.com/okp4/contracts/pull/530/files#diff-6fdb29ee56278ca668526e27dcf7192c00693f6cd4fd163c411f5f4546616636R29)

🔗 Related links

Needs #531
Needs #532
https://github.com/okp4/okp4d/pull/616

size-limit report 📦

Path
Size

target/wasm32-unknown-unknown/release/okp4_objectarium.wasm
0 B (-100% 🔽)

target/wasm32-unknown-unknown/release/okp4_law_stone.wasm
0 B (-100% 🔽)

target/wasm32-unknown-unknown/release/okp4_cognitarium.wasm
0 B (-100% 🔽)

target/wasm32-unknown-unknown/release/okp4_dataverse.wasm
0 B (-100% 🔽)

0 B (+100% 🔺)

I've updated the secrets in the CI and also disabled the release auto comments on PRs :)


DownmixMono channels_first wrong default value

While the docs state the default value for channels_dim is True, it seems to be set to None instead as shown here.
This would yield to the default channel dimension as dim1:
channels_first = None
ch_dim = int(not channels_first)
print(ch_dim)
> 1

Reported in the forum here.
If you want, I could create a fast fix using channels_first=True as the default value or update the docs instead.
Best,
ptrblck

Removed DownmixMono here https://github.com/pytorch/audio/pull/152


Start tauri-based frontend from main binary

This PR finally integrates tauri, and thus fixes #96.

   Compiling bazed-tauri v0.0.0 (C:\Users\Cpli\Documents\proto\bazed\crates\bazed-tauri)
error: failed to run custom build command for `bazed-tauri v0.0.0 (C:\Users\Cpli\Documents\proto\bazed\crates\bazed-tauri)`

Caused by:
  process didn't exit successfully: `C:\Users\Cpli\Documents\proto\bazed\target\debug\build\bazed-tauri-3c627ece64243281\build-script-build` (exit code: 101)
  --- stdout
  cargo:rerun-if-changed=C:\Users\Cpli\Documents\proto\bazed\crates\bazed-tauri/build.rs
  cargo:rerun-if-changed=C:\Users\Cpli\Documents\proto\bazed\crates\bazed-tauri/svelte/src
  cargo:rerun-if-changed=C:\Users\Cpli\Documents\proto\bazed\crates\bazed-tauri/svelte/package.json
  cargo:rerun-if-changed=C:\Users\Cpli\Documents\proto\bazed\crates\bazed-tauri/svelte/svelte.config.js

  --- stderr
  thread 'main' panicked at 'npm ci failed: Error { kind: NotFound, message: "program not found" }', crates\bazed-tauri\build.rs:13:10


Which languages will be supported and based on which criteria they get selected

What is missing
It is missing on which facts the selection of further supported languages will be based to reach the most people possible living in Germany but not speaking german sufficiently.
Why should it be included
The selection should be made based on official destatis information publicly available to eliminate any selection bias and prevent discrimination.
Where should it be included

E01.06 3.

I have a lot of non German Speakers in my Team. They are from all over the world but everyone speaks Englisch. That Language would be highly apreciated!

@Lhendrichs English and German are both supported in the initial release of the app.

@Lhendrichs and the English version works perfectly as I have my iPhone set to English and therefore use the live version in English right now.

ok I will ask my team mates to try it. Thanks
Am Di., 16. Juni 2020 um 10:33 Uhr schrieb benekuehn <
<EMAIL_ADDRESS>
@Lhendrichs https://github.com/Lhendrichs and the English version works
perfectly as I have my iPhone set to English and therefore use the live
version in English right now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/corona-warn-app/cwa-documentation/issues/152#issuecomment-644619250,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AP7EKDZ4YBVGRZ7OMNEGBUTRW4U4LANCNFSM4NKLJ6CA
.

My Friend just tried with his phone set to Chinese.. it will show the App in German. Maybe is there a way to redirect a Chinese Phone to Englisch instead of German (or create a Language selection button)?.

@Lhendrichs English and German are both supported in the initial release of the app.

right but you can't select it and it looks like a none English Phone (for instance Chinese) will always show German.

@Lhendrichs if that's the case, could you kindly open an issue in the respective app repository:

iOS
Android

According to the scoping document, this would be a bug (https://github.com/corona-warn-app/cwa-documentation/blob/master/scoping_document.md)

Thanks Thomas. Bug report submitted: https://github.com/corona-warn-app/cwa-app-ios/issues/636

There should be a setting which allows the user to select the language manually if he/she doesn't like the default.

@darkdragon-001 filed a request for this one  almost a month ago.

New languages are constantly being added, e.g., PRs for Turkish, russian and arabic are already underway.
Please use https://github.com/corona-warn-app/cwa-backlog/issues/4 to track upcoming languages.


Refactor: Moving StorageClusterInit in a new file.

Moved the logic for StorageClusterInit in a new file for better clarity and understanding.
Signed-off-by: RAJAT SINGH<EMAIL_ADDRESS>

CSV changes should be a separate commit. In fact, they aren't even needed.

Ok I will  make them in a separate commit. But why they aren't  needed?.

Changes to pkg/apis/ocs/v1/zz_generated.deepcopy.go is not relevant here. Remove it.

These changes came from running make update generated and make latest-gen-csv

Changes to purgeOSDScript in pkg/controller/storagecluster/reconcile.go should not be a part of this PR.
shellcheck-stable/README.txt is not relevant. Remove it.

/retest

Code looks good. Please rename the file to storageclusterinit.go without camel case.

Done, please merge.

@jarrpa @umangachapagain @rajatsingh25aug  -  this should have been reviewed a bit more carefully....
The first patch accidentally removes a hunk of code from reconcile.go that it did not mean to remove (part of the purge-osd-script).
Then the second patch re-introduces that hunk but messes up the whitespacing. Amusingly, this second patch is called "Removed extra whitespaces added by previous contributor". The whitespaces that this patch effectively removes were actually not extra but  good and intentional (indentation of blocks).
The fact that this has not been caught in reviews now makes backporting of #589 unnecessarily difficult.
I wish a bit more scrutiny to be applied to the commits and commit messages in reviewing PRs. :wink:

@jarrpa @umangachapagain @rajatsingh25aug - this should have been reviewed a bit more carefully....
The first patch accidentally removes a hunk of code from reconcile.go that it did not mean to remove (part of the purge-osd-script).
Then the second patch re-introduces that hunk but messes up the whitespacing. Amusingly, this second patch is called "Removed extra whitespaces added by previous contributor". The whitespaces that this patch effectively removes were actually not extra but good and intentional (indentation of blocks).

But may I know why do we need extra spaces at the end of a line?. My vscode removes any unnecessary extra whitespaces from everywhere(end of a line, lots of extra space between lines).

The fact that this has not been caught in reviews now makes backporting of #589 unnecessarily difficult.
I wish a bit more scrutiny to be applied to the commits and commit messages in reviewing PRs. 😉

And now working on re-adding the indentation, I see what must have been @rajatsingh25aug 's intention:
To remove the trailing white spaces. That would have been fine (in a separate PR).
In addition, and probably unintentionally, the indentation was removed too.
First of all this should not have been part of this PR.
Secondly, it was only spotted because the accidentally removed block of code. And re-adding was done with sacrificing white spaces at the beginning of the line, and that was put as a separate commit that claimed only to remove white spaces, but in fact it added back a whole block of code. :-)

I see now. It should've been a separate PR. My bad that I didn't thought of
this earlier.
But now, what should we do next?. Should I raise a new PR?.
Anything else that you can suggest?.
On Fri, 3 Jul 2020 at 8:39 PM Michael Adam<EMAIL_ADDRESS>wrote:

And now working on re-adding the indentation, I see what must have been
@rajatsingh25aug https://github.com/rajatsingh25aug 's intention:
To remove the trailing white spaces. That would have been fine (in a
separate PR).
In addition, and probably unintentionally, the indentation was removed too.
First of all this should not have been part of this PR.
Secondly, it was only spotted because the accidentally removed block of
code. And re-adding was done with sacrificing white spaces at the beginning
of the line, and that was put as a separate commit that claimed only to
remove white spaces, but in fact it added back a whole block of code. :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/ocs-operator/pull/563#issuecomment-653589555,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDUM2HELVXOTEEOKDT74OTRZXYCHANCNFSM4N7QUFEQ
.

@rajatsingh25aug writes:

But may I know why do we need extra spaces at the end of a line?

Ah, at the end of the line, we don't usually need them.
But such unrelated changes should go in a separate PR.

My vscode removes any unnecessary extra whitespaces from everywhere(end of a line, lots of extra space between lines).

Apparently, your editor tried to be a bit too clever here....
Because this was part of a string that actually contains a script, it may not have understood that this is a script and removed the indentation.
In general, I am not in favour of tools that interfere too much with the editing.
At the very least we should always critically check if the tool has done the right thing.
The first accident in this PR was not the whitespace change though, but the fact that first patch https://github.com/openshift/ocs-operator/commit/a79a94584b8969038e349fde79674f15efbca75c removed almost the whole script. And the second commit https://github.com/openshift/ocs-operator/commit/74c06f2b51c49640f3ec6c3a88236bb01172d2a7 brought back the script but with whitespace changes.
So the intentions were good. But two problems:

error in the first commit undone in the second commit - this is what makes backports difficult.
the editor removed too many white spaces.

Cheers - Michael

@rajatsingh25aug writes:

I see now. It should've been a separate PR. My bad that I didn't thought of this earlier.

No issues, this is also what reviews are for. Review should have been more diligent. :wink:
@anubhav-here actually spotted that there was something strange, but convinced all is good.

But now, what should we do next?. Should I raise a new PR?.

I have it almost ready.

Anything else that you can suggest?.

No, keep this feed-back in mind. Otherwise, keep up the good work. :smiley:

thanks a lot Miachel 😬 and again from next time, we will keep these nifty
things in mind.
On Fri, 3 Jul 2020 at 8:48 PM Michael Adam<EMAIL_ADDRESS>wrote:

@rajatsingh25aug https://github.com/rajatsingh25aug writes:
I see now. It should've been a separate PR. My bad that I didn't thought
of this earlier.
No issues, this is also what reviews are for. Review should have been more
diligent. 😉
@anubhav-here https://github.com/anubhav-here actually spotted that
there was something strange, but convinced all is good.
But now, what should we do next?. Should I raise a new PR?.
I have it almost ready.
Anything else that you can suggest?.
No, keep this feed-back in mind. Otherwise, keep up the good work. 😃
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openshift/ocs-operator/pull/563#issuecomment-653593119,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDUM2HH6KFNIRLQETZZ6LLRZXZD5ANCNFSM4N7QUFEQ
.


Credential Parameter should prompt

In PowerShell, you can specify the username as part of a Credential parameter, and PowerShell will prompt for the password.

But this behavior isn't supported in New-OpenADSession. I haven't tried anything else yet.

Thanks for pointing that out, looks like I need to add the [Credential()] attribute to this parameter at https://github.com/jborean93/PSOpenAD/blob/main/src/Commands/OpenADSession.cs#L60.


setfenv disables several Luau optimizations

There are two very helpful optimizations in Luau that can increase the speed of many scripts: imports and fastcalls.
Imports are global chains that can be evaluated at script load time (e.g. math.huge) rather than evaluating them every time they're used at run time; the values are placed in the constant table.
Fastcalls are special versions of builtin functions that don't require things like setting up a new stack and all the hassle for calling functions.
However, these optimizations fall when getfenv/setfenv is used. The safeenv property of the functions is turned to false because it is no longer reliable to expect that the values that were evaluated at runtime are the same as they are now with a modified environment.
This project uses setfenv to give some globals to virtual scripts, which completely disables both of these optimizations.
https://luau-lang.org/performance#importing-global-access-chains
https://luau-lang.org/performance#specialized-builtin-function-calls
I was wondering if there could be a different way of setting these globals, other than using getfenv/setfenv, since these will make scripts run slower.

I just realized that loadstring also disables these optimizations, so unless this is possible to do without it, it might not be possible.

This is a bit out of scope, so I'm closing it for now. Bundling your code with remodel is a better solution if you need these optimizations.


Dart client with grpc-dotnet server: some interop test cases are flaky

Affected test cases:
cloud_to_cloud:dart:aspnetcore_server:timeout_on_sleeping_server:tls
cloud_to_cloud:dart:aspnetcore_server:cancel_after_begin:tls

A few recent failures:
https://source.cloud.google.com/results/invocations/7caa00ca-fcd9-4dcf-82f2-7b0cd4e60293/targets/github%2Fgrpc%2Finterop_test/tests;group=tests;test=cloud_to_cloud:dart:aspnetcore_server:cancel_after_begin:tls;row=1
https://source.cloud.google.com/results/invocations/7892334e-d125-468a-9b74-c1ecea19669f
https://source.cloud.google.com/results/invocations/313f6357-e840-47a8-86c2-3e9c58c3ebb1
https://source.cloud.google.com/results/invocations/8b003997-a952-431b-9621-c3beb65685bd
https://source.cloud.google.com/results/invocations/e048167f-428f-4ec5-a079-73bcdd3779bc
From the go/grpc-flaky dashboard

The error message is similar to some flakes we've seen the for grpc-dart client in the past, so we can assume that the problem is on the grpc-dart side.
Passed.
Unhandled exception:
NoSuchMethodError: The method 'read' was called on null.
Receiver: null
Tried calling: read()
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:50:5)
#1      _Socket._onData (dart:io-patch/socket_patch.dart:1785:27)
#2      _RootZone.runUnaryGuarded (dart:async/zone.dart:1314:10)
#3      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:336:11)
#4      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:263:7)
#5      _SyncStreamController._sendData (dart:async/stream_controller.dart:764:19)
#6      _StreamController._add (dart:async/stream_controller.dart:640:7)
#7      _StreamController.add (dart:async/stream_controller.dart:586:5)
#8      _RawSecureSocket._sendReadEvent (dart:io/secure_socket.dart:1004:19)
#9      Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:21:15)
#10     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:382:19)
#11     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:416:5)
#12     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)
``

@mit-mit  @sigurdm can you take a look?
The error message looks similar to https://github.com/grpc/grpc/issues/21755

It should be possible to reproduce with tools/run_tests/run_interop_tests.py -l dart -s aspnetcore --use_docker (after a few attempts)

I'm able to reproduce locally with DOCKERHUB_ORGANIZATION=grpctesting tools/run_tests/run_interop_tests.py -l dart -s aspnetcore --use_docker very reliably.

I tried locally ~10 times without being able to reproduce:
DOCKERHUB_ORGANIZATION=grpctesting tools/run_tests/run_interop_tests.py -l dart -s aspnetcore --use_docker
No module named apiclient
Seen --use_docker flag, will run interop tests under docker.

IMPORTANT: The changes you are testing need to be locally committed
because only the committed changes in the current branch will be
copied to the docker environment.
START: Building interop docker images.
PASSED: build_docker_dart [time=24.2sec, retries=0:0]
PASSED: build_docker_aspnetcore [time=56.6sec, retries=0:0]
SUCCESS: All docker images built successfully.
START: interop_server_aspnetcore
PASSED: cloud_to_cloud:dart:aspnetcore_server:timeout_on_sleeping_server:tls [time=4.1sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:empty_stream:tls [time=4.1sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:server_streaming:tls [time=4.1sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:cancel_after_begin:tls [time=4.2sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:client_streaming:tls [time=4.2sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:cancel_after_first_response:tls [time=4.2sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:unimplemented_method:tls [time=4.3sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:unimplemented_service:tls [time=4.3sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:empty_unary:tls [time=4.4sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:status_code_and_message:tls [time=4.4sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:large_unary:tls [time=4.4sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:ping_pong:tls [time=4.5sec, retries=0:0]
PASSED: cloud_to_cloud:dart:aspnetcore_server:custom_metadata:tls [time=4.4sec, retries=0:0]
SUCCESS: All tests passed
Removing docker image grpc_interop_aspnetcore:22fd58c1-7859-4c68-adf2-bc5b1e9ca948
Removing docker image grpc_interop_dart:858c56c2-fa94-4f8d-a33d-82130d66ed71
Traceback (most recent call last):
  File "tools/run_tests/run_interop_tests.py", line 1693, in <module>
    report_utils.render_junit_xml_report(resultset, _TESTS_XML_REPORT)
  File "/usr/local/google/home/sigurdm/projects/grpc/tools/run_tests/python_utils/report_utils.py", line 59, in render_junit_xml_report
    create_xml_report_file(tree, report_file)
  File "/usr/local/google/home/sigurdm/projects/grpc/tools/run_tests/python_utils/report_utils.py", line 82, in create_xml_report_file
    tree.write(report_file, encoding='UTF-8')
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 802, in write
    file = open(file_or_filename, "wb")
IOError: [Errno 13] Permission denied: 'interop_test/sponge_log.xml'

Not sure what the log-error means, but it looks like the tests pass.
This is with grpc, grpc-dart and grpc dotnet all at todays master

@sigurdm  that's odd, because I can reproduce pretty consistently.
Would the trick that fixed similar error on the dart server in https://github.com/grpc/grpc-dart/pull/274/files help fix the client part?

Hmm - that's hard to say without further debugging.
I'll try to poke a bit more at it today, and look if something like that could be the reason.

Looking at the corresponding location of the stacktrace in the current sdk (https://github.com/dart-lang/sdk/blob/master/sdk/lib/_internal/vm/bin/socket_patch.dart#L1981) this should not be able to happen. (We have just checked that _raw is not null.
That check has been in the sdk for 9 moths: https://github.com/dart-lang/sdk/commit/f664a808500c09dcdcc14765a262a8eaf646892a
Fixing https://github.com/dart-lang/sdk/issues/37522.
Probably the tests are running an older sdk.

I'm OOO next week. So will not be able to get to the bottom of this before that. But updating the dart sdk used for the tests might be enough to fix this.

I updated the dart SDK in the interop test to 2.7. Let's see if that helps.

The last occurrence of this flake is https://source.cloud.google.com/results/invocations/0293e59e-e6fa-4caf-98aa-153048a06b56 (just before this PR got merged). We haven't seen the flake since.
Closing this issue.
@sigurdm  thanks for help!

Sure - and thanks for yours!


Fix query execution lock-up for large frames.

The send_frame function did use write method to send out
frame data. The method actually writes just a portion of the data provided
(usually first 16K). The rest was not written and got lost.
That caused Cassandra node peer send no reply waiting for more frame data,
which in turn led to send_frame hanging infinitely.
The write_all method has to be used instead.

Hi @a1ph i believe that repo doesn't have manteiner. Do you want to send PR to https://github.com/krojew/cdrs-tokio if apply?

Hi @a1ph i believe that repo doesn't have manteiner. Do you want to send PR to https://github.com/krojew/cdrs-tokio if apply?

Hi @a1ph i believe that repo doesn't have manteiner. Do you want to send PR to https://github.com/krojew/cdrs-tokio if apply?

@fggarcia That is on my list. I still hope @AlexPikalov may build a bug fix release with this one. It's a pretty weird bug.

Hi @a1ph i believe that repo doesn't have manteiner. Do you want to send PR to https://github.com/krojew/cdrs-tokio if apply?

@fggarcia That is on my list. I still hope @AlexPikalov may build a bug fix release with this one. It's a pretty weird bug.

@a1ph Thank you for the PR

@a1ph Thank you for the PR


Initialize driver in COMMAND mode and make sure it will be in DATA mode

Send ATO in command mode will guarantee that the device will be in DATA mode.
If the device is in DATA mode, it will be transmitted as burst data (no +++AT)
and as this command does not require a response, it doesn't break the task.

Depends on https://github.com/Brazilian-Institute-of-Robotics/drivers-usbl_evologics/pull/19


Upload butterfly.jpg

Could you include the "../test/butterfly.jpg" into the repo? That would make it easier to actually run the demo and play around with it. Thanks!

Sure, I added it.


Refactor and Document Parquet Tools

make Parquet Tools runnable from a bundle, create documentation on the usage of it.
So the use can use parquet tools with:
hadoop jar parquet-tools xxxx
Reporter: Tim / @tsdeng
Assignee: Tim / @tsdeng
Note: This issue was originally created as PARQUET-105. Please see the migration documentation for further details.

Julien Le Dem / @julienledem:
Issue resolved by pull request 69
https://github.com/apache/incubator-parquet-mr/pull/69


Purpose of this library?

What does it offer that I couldn't get from the standard date  API?

Default Date API offers only millisecond based date comparisons, calculations and manipulations. Just check the Instadate API vs Date API, they are completely different.

So your library just multiplies MS by X?

Not really.. Please take a look at the API docs for an overview of everything this library does


Undefined Behavior: incorrect layout on deallocation, at Bytes.advance()

Just running cargo miri test. Moving the pointer does not dealloc it correctly, producing UB.
let mut buf: Bytes = vec![0u8; 8].into();
buf.advance(2);
drop(buf);

Should dealloc when advance it as it will not be used again.

Can you provide the full main function you used?

With coverage

It does not appear to fail on the playground when you choose miri in the tools on the following:
use bytes::{Bytes, Buf};

fn main() {
    let mut buf: Bytes = vec![0u8; 8].into();
    buf.advance(2);
    drop(buf);
}

playground
What did you do to reproduce this?

It's my fault, I close.


Configmap not updated when upgrading apps

I have a chart that includes lots of children charts.
One of those children charts creates a configmap.
It utilises a pre-install,pre-upgrade hook so that the configmap is created before all other charts are deployed.
The problem I am having now is that the configmap does not update when I upgrade the app in Rancher.
However, if I deploy my application with helm install and then use helm upgrade to upgrade it, I can see the configmap is successfully updated.
Here's what my configmap looks like
{{- $url := include "neon.url" . -}}
{{- $database := include "neon.database" . -}}
apiVersion: v1
kind: ConfigMap
metadata:
  name: "{{ .Values.global.customer }}-{{ .Values.global.environment }}"
  annotations:
    "helm.sh/hook": pre-install,pre-upgrade
    "helm.sh/hook-weight": "5"
data:
  CLIENT_DOMAIN: "{{ $url }}"
  DB_HOST: "{{ $database }}"
  STACK_DOMAIN: "{{ $url }}"
  STAGING_DOMAIN: "{{ .Values.global.vault.vault_template }}"
  VAULT_ADDR: "{{ .Values.global.vault.vault_addr }}"
  VAULT_PORT: "{{ .Values.global.vault.vault_port }}"
  ENVIRONMENT: "{{ .Values.global.environment }}"
  PARTITION: "{{ .Values.global.customer }}"
  CUSTOMER: "{{ .Values.global.customer }}"

I added a new key in data section and the configmap couldn't be updated.
Environment:
Rancher 2.5.2
Kubernetes 1.19
OS Ubuntu 16.04
Docker 18.9.9

Can anyone have a look at this issue. I have to manually delete the configmap every time I upgrade the application.


fix: construct ASSIGNEES_URL to the correct url

The issue: The action fails to assign assignees to the created PR**
Logs sample:
START: Running Pull Request on Branch Update Action!
Found ${GITHUB_EVENT_PATH} at /github/workflow/event.json
Branch prefix is
No branch prefix is set, all branches will be used.
Pull requests will go to main
PULL_REQUEST_DRAFT set to a value: created PRs will be draft PRs.
No preference for maintainer being able to modify: default is true.
PULL_REQUEST_ASSIGNEES is set, HasanKhatib
.
.
github response: 'assignees_url': '[https://api.github.com/repos/ingka-group-digital/resolutions/assignees{/user}](https://api.github.com/repos/ingka-group-digital/resolutions/assignees%7B/user%7D)'

The fix:

Based on GitHub REST API, use the correct endpoint to add assignees to an issue /repos/:owner/:repo/issues/:issue_number/assignees

This has been fixed in the add_assignees function.

Can you clarify how this is different? The current implementation already has the repo url, but it's represented in the issue URL.
ISSUE_URL = "%s/issues" % REPO_URL

Can you please show me a before and after to clarify?

Before: The incorrect URL was %s/%s/assignees, which incorrectly concatenates the issue URL and the issue number directly without the /issues/ prefix, leading to an invalid endpoint.
After: The corrected URL is %s/issues/%s/assignees, ensuring the correct endpoint /issues/{issue_number}/assignees is used for adding assignees to the pull request.

Ah! now I see ISSUE_URL = "%s/issues" % REPO_URL. My bad!

Argh! I see now.
I will close the PR and create an issue instead, as the issue is still there. It reads the assignees and show it in this log message PULL_REQUEST_ASSIGNEES is set, HasanKhatib , but then returns assignees': [],

#105


Add get_var_shape and get_value_ptr

I found the Julia libribasim tests hard to deal with.
Specifically, I completely failed to get a working Ptr{Ptr{CVoid}} for get_value_ptr so I leave them for now.
The API itself works fine though.


Unforce openh264 version

It's been a couple weeks, hopefully they should have pushed a fix on the 2.3 version update for the imports to work correctly.
Resolves #317

Yes they did! Ready to merge :slightly_smiling_face:


Fix bug in client stats summary uploading where it uploaded incorrect…

… or badly formatted data

Great, thanks. I'll deploy this asap.

deployed


Can DatabaseQueue::getLockForPopping() use anything other than PDO::ATTR_DRIVER_NAME.

Laravel Version: 8.79.0
PHP Version: 8.0.13
Database Driver & Version: CockroachDB (pdo_pgsql)

Description:
I'm using CockroachDB with Laravel - which uses the PostgreSQL PDO driver. However, according to https://github.com/cockroachdb/cockroach/issues/40476 it does not support SKIP LOCKED (yet anyway).
I've been enhancing nbj/cockroachdb-laravel (my fork is  leenooks/cockroachdb-laravel), and what I cannot figure out is an easy way to ensure that DatabaseQueue::getLockForPopping() passess through the DB checks and returns true. Unfortunately that routine is using PDO::ATTR_DRIVER_NAME to determine what the DB is, but in this case it's not actually PostgreSQL.
Ideally, this routine should use something else PDO/Driver/... to determine the driver for the database and make its decision to include SKIP LOCKED based on the PHP driver, not the PDO driver.
Without this being changed, it doesnt seem that I can run job queues with CockroachDB.

Heya, thanks for submitting this.
This seems like a feature request or an improvement so I'm converting this to a discussion instead. It's best to post these in the discussions section in the future to get support for your idea. After that you may send a PR to the framework. Please only use the laravel/framework issue tracker to report bugs and issues with the framework.
Thanks!


flops and params

can you tell me how didi you get it?thank

you can get it from "model.summary()".


Android Studio

Eclipse is old.
Nowadays Android developers use Android Studio to create Android applications.
By the way, Torque 2D doesn't include support to Android Studio, but Eclipse only.
This summer Casey Doran added support to Android Studio (https://www.garagegames.com/community/forums/viewthread/142325/) in his fork of Torque 2D (https://github.com/elementc/Torque2D/tree/android-studio).
I have tested this cod, it works very fine!
I think you should add this cod to original repository.

I'm on board with this.  If you're using his support for Android Studio are you able to create a pull request with the needed files?  I would gladly merge such a request.

Ok, I'll make this


Regression 4.6.0 / Postgres index relation already exists

First happened on this PR, which bumped the recipe version that PQSQL is tested against from 4.5.x-dev to 4.6.x-dev, so possibly it's related to that
Otherwise, may be something in https://github.com/dnadesign/silverstripe-elemental/compare/4.3..4.4
The postgres failure:
https://travis-ci.org/github/dnadesign/silverstripe-elemental/jobs/698651852
PHP Fatal error:  Uncaught SilverStripe\ORM\Connect\DatabaseException: Couldn't run query:

create index "ix_a32d98fbe585c49c2c18e1e14f7cdbe4" ON "Page" ("ElementalAreaID");

ERROR:  relation "ix_a32d98fbe585c49c2c18e1e14f7cdbe4" already exists in /home/travis/build/dnadesign/silverstripe-elemental/vendor/silverstripe/framework/src/ORM/Connect/DBConnector.php:64

Stack trace:

#0 /home/travis/build/dnadesign/silverstripe-elemental/vendor/silverstripe/postgresql/code/PostgreSQLConnector.php(234): SilverStripe\ORM\Connect\DBConnector->databaseError('Couldn't run qu...', 256, 'create index "i...', Array)

#1 /home/travis/build/dnadesign/silverstripe-elemental/vendor/silverstripe/postgresql/code/PostgreSQLConnector.php(246): SilverStripe\PostgreSQL\PostgreSQLConnector->preparedQuery('create index "i...', Array, 256)

#2 /home/travis/build/dnadesign/silverstripe-elemental/vendor/silverstripe/framework/src/ORM/Connect/Database.php(159): SilverStripe\PostgreSQL\PostgreSQLConnector->query('create index "i...', 256)

#3 /home/travis/build/d in /home/travis/build/dnadesign/silverstripe-elemental/vendor/silverstripe/framework/src/ORM/Connect/DBConnector.php on line 64

That appears to be a regression caused by https://github.com/silverstripe/silverstripe-framework/commit/052c5cbc38210476ffce7d98dc052fa09b4e5e5f (part of 4.6.0-rc1).
I'm investigating what exactly is going on there.

A patch with a fix is here - https://github.com/silverstripe/silverstripe-framework/pull/9555
Should be in 4.6.1 release.


Add dynamic multiselect

Add dynamic multiselect
changes examples for Select

Coverage increased (+0.8%) to 91.304% when pulling cc3b7b93af3a4b57582980132de1978d2be476b1 on tb:connected-selects into c03f1a882d3dbeef5feb08538f2048559cc0ed9e on mseemann:master.


MonitoredTrainingSession/Parameter server with large embedding table

i want to train a model with MonitoredTrainingSession, and here is a large embedding table(K*D, K is almost 100 millions, D is 16), and i have to put the tensor(embedding_table) to one parameter server(ps.0), which will make a bottleneck.
So, i want to know is there any method to solve this problem. For example, put the large embedding table to be parted and each PS have one of them.

@colourful-tree As this issue is not related to bug/performance, build/install, feature request or doc related issues, Please post this issue in stack overflow where there is a wider community to respond. Thanks!


support adding validation annotation to list of object in query param…

…eter

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:


Uncaught AttributeError in sendchat.py line 30

Aider version: 0.56.1
Python version: 3.12.1
Platform: Linux-6.5.0-1025-azure-x86_64-with-glibc2.31
Python implementation: CPython
Virtual environment: No
OS: Linux 6.5.0-1025-azure (64bit)
Git version: git version 2.46.0
An uncaught exception occurred:
Traceback (most recent call last):
  File "default.py", line 72, in map_httpcore_exceptions
    yield
  File "default.py", line 236, in handle_request
    resp = self._pool.handle_request(req)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "connection_pool.py", line 216, in handle_request
    raise exc from None
  File "connection_pool.py", line 196, in handle_request
    response = connection.handle_request(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "connection.py", line 99, in handle_request
    raise exc
  File "connection.py", line 76, in handle_request
    stream = self._connect(request)
             ^^^^^^^^^^^^^^^^^^^^^^
  File "connection.py", line 154, in _connect
    stream = stream.start_tls(**kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "sync.py", line 152, in start_tls
    with map_exceptions(exc_map):
  File "contextlib.py", line 158, in __exit__
    self.gen.throw(value)
  File "_exceptions.py", line 14, in map_exceptions
    raise to_exc(exc) from exc
httpcore.ConnectError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "chat.py", line 725, in make_sync_call
    response = client.post(
               ^^^^^^^^^^^^
  File "http_handler.py", line 373, in post
    raise e
  File "http_handler.py", line 356, in post
    response = self.client.send(req, stream=stream)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "_client.py", line 926, in send
    response = self._send_handling_auth(
               ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "_client.py", line 954, in _send_handling_auth
    response = self._send_handling_redirects(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "_client.py", line 991, in _send_handling_redirects
    response = self._send_single_request(request)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "_client.py", line 1027, in _send_single_request
    response = transport.handle_request(request)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "default.py", line 235, in handle_request
    with map_httpcore_exceptions():
  File "contextlib.py", line 158, in __exit__
    self.gen.throw(value)
  File "default.py", line 89, in map_httpcore_exceptions
    raise mapped_exc(message) from exc
httpx.ConnectError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "main.py", line 1586, in completion
    response = anthropic_chat_completions.completion(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "chat.py", line 1117, in completion
    completion_stream = make_sync_call(
                        ^^^^^^^^^^^^^^^
  File "chat.py", line 736, in make_sync_call
    raise AnthropicError(status_code=500, message=str(e))
litellm.llms.anthropic.chat.AnthropicError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "base_coder.py", line 1124, in send_message
    continue
  File "base_coder.py", line 1396, in send
    self.stream,

  File "sendchat.py", line 86, in send_completion
  File "utils.py", line 1086, in wrapper
    raise e
  File "utils.py", line 974, in wrapper
    result = original_function(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "main.py", line 2847, in completion
    raise exception_type(
          ^^^^^^^^^^^^^^^
  File "utils.py", line 8194, in exception_type
    raise e
  File "utils.py", line 6680, in exception_type
    raise litellm.InternalServerError(
litellm.exceptions.InternalServerError: litellm.InternalServerError: AnthropicException - [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1000). Handle with `litellm.InternalServerError`.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "aider", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "main.py", line 698, in main
    return 1
  File "base_coder.py", line 735, in run
    return self.io.get_input(
                ^^^^^^^^^^^^^^
  File "base_coder.py", line 778, in run_one
    def check_for_urls(self, inp):
            ^^^^^^^^^^^^^^^^^^^^^^^
  File "base_coder.py", line 1126, in send_message
    interrupted = True
       ^^^^^^^^^^^^^^^^
  File "sendchat.py", line 30, in retry_exceptions
    litellm.llms.anthropic.chat.AnthropicError,
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'litellm.llms.anthropic' has no attribute 'AnthropicError'

Thanks for trying aider and filing this issue.
This looks like a duplicate of #1278. Please see the comments there for more information, and feel free to continue the discussion within that issue.
I'm going to close this issue for now. But please let me know if you think this is actually a distinct issue and I will reopen this issue.


Fixes #128

This change is

Codecov Report
Attention: 47 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

...c/SIL.Machine.AspNetCore/Services/S3WriteStream.cs
0.00% <0.00%> (ø)

:loudspeaker: Thoughts on this report? Let us know!.


yeoman-test for ensuring template generation

generator-teams changes should be tested via. yeoman-test package for ensuring, that the the projects can be generated successfully. It should also included in build pipeline.

Initial tests included in version 2.7.1 (7c45f921ef44af81a909295724a4dd387f4bc1e1)


Drop RSpec/EmptyExampleGroup customization

We're providing a flexible way of customizing RSpec syntax thanks to #956
This is not based on #956, but rather includes it (can't specify a base branch from another fork).
Those are the changes https://github.com/rubocop-hq/rubocop-rspec/pull/1007/commits/42f77084fe01f5abe333bbbe1a8ba7b402c25e2c
This is the change needed to avoid mutating the default config between the examples https://github.com/rubocop-hq/rubocop-rspec/pull/1007/commits/718b398dcdfc794017d93bb8578466b7e65f22a3
This way it's possible to drop customization of this cop instead customize via .rubocop.yml configuration, i.e.:
AllCops:
  RSpec:
    Language:
      Includes:
        Example:
        - it_has_special_behavior

There's a problem with specs, though, specifically with this:
RuboCop::ConfigLoader.default_configuration.for_all_cops.dig('RSpec', 'Language')

for_all_cops is memoized in Config:
# rubocop-0.89.1/lib/rubocop/config.rb:132:

def for_all_cops
  @for_all_cops ||= self['AllCops'] || {}
end

It's not so obvious to notice, since in this cop the example that is modifying the language is in its own context, and it runs last.
Even if you add an example after it, due to how RSpec orders examples, it will still be executed last.
However, if you add a context with an example, you'll see that the language changes leaked.
~We need to figure out a way to reset the config between examples, and do it in a performant way.~
~Thoughts? dup in before and assign back in after?~
Fixes #969

Before submitting the PR make sure the following are checked:

[no] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Updated documentation.
[ ] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

If you have modified an existing cop's configuration options:

[x] Set VersionChanged in default/config.yml to the next major version.

I'll rebase this one.

I'll rebase this one.

Nevermind, #956 comes first.

It's time. I'll rebase this one.

Rebases on top of the current #956

JRuby crashed on rake internal_investigation, which is quite interesting. I guess I'll have to add :config to all Concept specs.

I can reliably reproduce the issue with --seed 37363:
Failure/Error: Language.config['HookScopes'].include?(element.to_s)
undefined method `[]' for nil:NilClass
./lib/rubocop/rspec/language.rb:127:in `all'

There seems to be something broken in the edge RuboCop, I'm looking into it.

The failure on edge-rubocop is irrelevant. Notified RuboCop guys about a breaking API change here https://github.com/rubocop-hq/rubocop/commit/51ff1d7e29c985732fe129082c98d66c531a2611#r43837211


Hidden Abilities unavailable

just pushed a change (c7fb974) that will allow the edit of the ability ✅
should be up in 1-2 minutes

thank you!


Make WinUI3 support creation of transparent windows

Does the WinUI3 support the creation of transparent windows?
And when will it supports the following window properties that WPF supports as of now
AllowsTransparency="True"
ResizeMode="NoResize"
ShowActivated="False"
ShowInTaskbar="False"
Topmost="True"
WindowState="Normal"
WindowStyle="None"
Background="Transparent"

and WinUI3 desktop app does not support height and width attributes.

xlink:

#8353
#1247
#8139

I'm sure there are other dupes hanging around on the repo too.