LoadBalancer API

We've gone back and forth on various public API options for the load balancer.  This issue shall serve as a placeholder for the discussion on which API is the most robust and semantically correct.
Iterator
Getting a client from the load balancer fits naturally with the Iterator interface since it follows the pull model.  However, since the load balancer list of hosts may be updated asynchronously the Iterator API can easily be violated.   hasNext() may return true but the list may be empty by the time next() is called.
public Observable<Response> submit(final Request request)  {
    return Observable.from(lb).flatMap(new Func1<Client, Observable<Response>>() {
        Observable<Response> call(Client c) {
            return c.submit(request);
        }
    }).single();
}

Observable
Implementing the LoadBalancer as an Observable give great flexibility for managing various states and error conditions.
public Observable<Response> submit(final Request request) {
    return lb.flatMap(new Func1<Client, Observable<Response>>() {
        Observable<Response> call(Client c) {
            return c.submit(request);
        }
    });
}

Func0
Using Func0 reduces the API to a simple method call that returns the next client or null if no client is available.  Since an Observable must be created to tie the client selection with request execution triggered by a subscription to the Observable we may as well just implement the LoadBalancer as an Observable in the first place.
public Observable<Response> submit(final Request request) {
    return Observable.create(new OnSubscribe<Client>() {
        public void call(Subscriber<Client> s) {
            ObClient c = lb.call();
            if (c == null) {
                s.onError(new NoSuchElementException());
            }
            else {
                s.onNext(c);
                s.onCompleted();
            }
        }
    }).flatMap(new Func1<Client, Observable<Response>>() {
        Observable<Response> call(Client c) {
            return c.submit(request);
        }
    });
}

Thanks @elandau for creating this issue, we need to nail down the contract :)

hasNext() may return true but the list may be empty by the time next() is called.

Is it true that the Load balancer works on immutable list of instances and for every change the list is swapped with a new list?
If so, then this should not be an issue, correct w.r.t hasNext() not being consistent with next()? As a worst case, one would work off an older list in the scope of a single request. If it is over two requests, then one should anyways do a fresh call to the LB.

Implementing the LoadBalancer as an Observable give great flexibility for managing various states and error conditions.

Can you explain this more as compared to an Iterable?

Func0

This is much like an Iterable of 1 semantics. In this case, would a backup request, do multiple calls to the Func0 till it receives a distinct item?

Since an Observable must be created to tie the client selection with request execution triggered by a subscription to the Observable we may as well just implement the LoadBalancer as an Observable in the first place.

I think its true that we will use it with an Observable at the end, but not sure whether that requires this API to be Observable based for the primary reason that the interaction is always pull-based.

The worst case for the iterator is the scenario where hasNext() returns true but the list is set to empty() before next() is called.  next() will end up throwing a NoSuchElementException.  To me this sounds like a violation of the contract for iterator.  For this to work correctly hasNext() must always return true and we will need to expect a NoSuchElementException from next().


Remove codecommit from the stack as this service is deprecated for the new AWS accounts

Describe the bug
Info from AWS:
https://repost.aws/questions/QUshILm0xbTjWJZSD8afYVgA/codecommit-cannot-create-a-repository
Codecommit service is deprecated for the new AWS accounts. We need to change the way we are triggering CI/CD pipeline.
Steps to reproduce

Create new AWS account
Try to deploy CI/CD stack to the AWS

System Info
.

Logs
No response
Validations

[X] Follow our Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

@mkleszcz Do you have a recommended workaround for this issue? I have just run into this when running pnpm saas infra deploy:
CreateRepository request is not allowed because there is no existing repository in this AWS account or AWS Organization (Service: AWSCodeCommit; Status Code: 400; Error Code: OperationNotAllowedException; Request ID: 3cbb3638-14f3-49c1-b9cd-5153d50e1e58;
Proxy: null)

@mkleszcz Do you have a recommended workaround for this issue? I have just run into this when running pnpm saas infra deploy:
CreateRepository request is not allowed because there is no existing repository in this AWS account or AWS Organization (Service: AWSCodeCommit; Status Code: 400; Error Code: OperationNotAllowedException; Request ID: 3cbb3638-14f3-49c1-b9cd-5153d50e1e58;
Proxy: null)

If anyone is curious, I was able to bypass this issue by submitted a support request to AWS  asking for access to CodeCommit. There may have been a quicker way to do this, but I submitted a support case, which was then forwarded to the AWS service team.
I needed to provide the following info:

Region/s:
Commercial Cloud, Gov Cloud, or CN?:
Customer was a CodeCommit Customer prior to July 25th, 2024 via asking them:

This took some time and back and forth (~8 days), but eventually, they added me to the whitelist. Someone more persistent may get this done quicker.


wip

Description
WIP
Changes include

[ ] Bugfix (non-breaking change that solves an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)
[ ] Tests
[ ] Documentation
[ ] Other (for changes that might not fit in any category)

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

Comparison is base (e13ea21) 60.42% compared to head (16e2b04) 60.42%.
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #542   +/-   ##
=======================================
  Coverage   60.42%   60.42%
=======================================
  Files          12       12
  Lines        1660     1660
=======================================
  Hits         1003     1003
  Misses        657      657

Flag
Coverage Δ

unittests
60.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Support iOS simulator

Is it possible to provide support for the iOS simulator as well? I'm currently getting the following error when deploying an app using this framework to the simulator:
dyld: Library not loaded: @rpath/ProtocolBuffers.framework/ProtocolBuffers
  Referenced from: xxx
  Reason: no suitable image found.  Did find:
	xxx/Frameworks/ProtocolBuffers.framework/ProtocolBuffers: no matching architecture in universal wrapper

I have not seen such errors. Everything works fine in the simulator. I think the problem in your project settings. How do you connect the library?

You're absolutely right. Closing this now. Thanks!

In case anyone else runs across this, I fixed the error by adding @executable_path to the project's "Runpath search paths".


fix: correct logic of requireEnv

by default, instrument will be on; env will not take any effect in this case
when requireEnv is not set, it will be assumed to be false

So after 2.1.0, the only chance that to disable the plugin is to

set requireEnv === true AND VITE_COVERAGE=false.

LGTM! Will merge when i get the semantic-release action going, so in the future pull requests can be shipped earlier. I'll have a look at it again tomorrow and hopefully the release will be out then.

thanks for providing this plugin！


Events EVENT_AFTER_INSERT and EVENT_AFTER_UPDATE documented as Event, but they are AfterSaveEvent in fact

Is AfterSaveEvent an internal stuff (so, we cannot rely on it) or is there a documentation bug?
Declaration https://github.com/yiisoft/yii2/blob/master/framework/db/BaseActiveRecord.php#L58 :
/**
 * @event Event an event that is triggered after a record is inserted.
 */
const EVENT_AFTER_INSERT = 'afterInsert';
...
/**
 * @event Event an event that is triggered after a record is updated.
 */
const EVENT_AFTER_UPDATE = 'afterUpdate';

Triggering https://github.com/yiisoft/yii2/blob/master/framework/db/BaseActiveRecord.php#L907 :
public function afterSave($insert, $changedAttributes)
{
    $this->trigger($insert ? self::EVENT_AFTER_INSERT : self::EVENT_AFTER_UPDATE, new AfterSaveEvent([
        'changedAttributes' => $changedAttributes
    ]));
}

Pull request #9822.


⚠ Jenkins Plugins Artifactory has degraded performance

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

HTTP code: 200
Response time: 1193 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 2d9390d after 33 minutes.


Deploy swan on docker

I tried to serve the application in a docker environment using the dockerfile already present on the repository, following all the steps described in the swan documentation.
Install yarn, which is required to mangage the monorepository.
$ yarn
Bundle the client applications and the server in ./server in production with the following command:
$ yarn build
Dockerize
Create a Docker image from the built files:
$ docker build -t swan .
You get this error when you try to launch the application image:
><EMAIL_ADDRESS>start
swan-partner-frontend-server-1  | > node -r ./dist/tracing.js ./dist/index.js
swan-partner-frontend-server-1  |
swan-partner-frontend-server-1  | Error: Cannot find module 'vite'
swan-partner-frontend-server-1  | Require stack:
swan-partner-frontend-server-1  | - /app/dist/client/devServer.js
swan-partner-frontend-server-1  | - /app/dist/app.js
swan-partner-frontend-server-1  | - /app/dist/index.js
swan-partner-frontend-server-1  |     at Module._resolveFilename (node:internal/modules/cjs/loader:1077:15)
swan-partner-frontend-server-1  |     at Module._load (node:internal/modules/cjs/loader:922:27)
swan-partner-frontend-server-1  |     at Module.require (node:internal/modules/cjs/loader:1143:19)
swan-partner-frontend-server-1  |     at require (node:internal/modules/cjs/helpers:110:18)
swan-partner-frontend-server-1  |     at /app/dist/client/devServer.js:38:78
swan-partner-frontend-server-1  |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
swan-partner-frontend-server-1  |     at async createViteDevServer (/app/dist/client/devServer.js:38:30)
swan-partner-frontend-server-1  |     at async Promise.all (index 0)
swan-partner-frontend-server-1  |     at async startDevServer (/app/dist/client/devServer.js:60:38)
swan-partner-frontend-server-1  |     at async start (/app/dist/app.js:610:37) {
swan-partner-frontend-server-1  |   code: 'MODULE_NOT_FOUND',
swan-partner-frontend-server-1  |   requireStack: [
swan-partner-frontend-server-1  |     '/app/dist/client/devServer.js',
swan-partner-frontend-server-1  |     '/app/dist/app.js',
swan-partner-frontend-server-1  |     '/app/dist/index.js'
swan-partner-frontend-server-1  |   ]
swan-partner-frontend-server-1  | }
swan-partner-frontend-server-1 exited with code 1

@devopsING2 Hi!
Did you switched NODE_ENV="development" to NODE_ENV="production" before running yarn build and docker build -t swan .?

I think the issue comes from the fact the provided NODE_ENV isn't "production", triggering the development mode.
you can find the list of necessary env variables here: https://swan-io.github.io/swan-partner-frontend/build-deploy#required-environment-variables

Thanks, yes that was the problem


Make European union endpoint optional - MixpanelDestination.swift

Making the enable European endpoint in the destination setting an optional one so customers who have had this destination enabled for a while are able to migrate from iOS to Swift without any issues. Currently, this causes the plugin to fail as the CDN file does not contain the enableEuropeanUnionEndpoint setting.
Reference customer ticket - https://segment.zendesk.com/agent/tickets/517739

Hey @bsneed assigning this to you for review. Please let me know if I can assign this to someone else.

Thank you!


1.82 cannot select text in integrated terminal

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

VS Code Version: 1.82.0
OS Version: Ubuntu 22.04

Steps to Reproduce:

Try to select text in the integrated terminal by shift + select text. Doesn't work.
Last working in 1.81.*

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.82.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

selection is working for me - could you provide your settings?

Actually, I see that selection works in a normal terminal by simply selecting the text (or double clicking a word, etc).
But, I use tmux in the integrated terminal and so have to hold down shift while selecting text (or double clicking a word, etc) and it is this SHIFT + SELECT text that is not working. I see that the selected text gets highlighted for a split second and then the highlight goes away.

Thanks for the report, I can reproduce.
In the meantime you can workaround this via keyboard selection by command (ctrl/cmd+up/down) or triggering the mouseup event outside of the terminal.

Not sure atm if this is upstream or in vscode.

Verified working in today's insiders

Nice. Thank you @Tyriar .

so how to fix it?

The same question, how to fix?

Use insiders or wait for v1.83

Still not working in insiders
Clean install 1.83.0-insider and WSL Ubuntu + tmux
Version: 1.83.0-insider (system setup)
Commit: 7b0ae6cf9d5a0c891ad1adc4da9c36c4dc453d4d
Date: 2023-09-14T05:33:44.297Z
Electron: 25.8.1
ElectronBuildId: 23779380
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621


feat: Release py-mtcli v1.36.0

Signed-off-by: Ankit Kurmi<EMAIL_ADDRESS>py-mtcli
Description
Release py-mtcli version v1.36.0
in favour of #258

/ok-to-test

/lgtm


refactored migrations

This PR removes useless and buggy backward compatibility between major versions of the package and closes #98.
P.S. this PR introduces breaking changes so major version(6.0) should be used.

Thank you for the PR!
I consolidated everything into just one migration with an existing name, so it wouldn't have to be a major release. When someone rollbacks, it just says the migration wasn't found and continues without throwing an exception.
Thanks again, it was about time to get rid of the legacy migrations indeed. :) Good luck!

When someone rollbacks, it just says the migration wasn't found and continues without throwing an exception

For this reason I had suggested to release a major version so with new installations the package is more clean and with less bugs.


Covalent theme won't build due to missing variables

Bug Report
Hey guys,
I'm getting this exception when building my Angular + material + covalent app:
ERROR in ./node_modules/css-loader?{"sourceMap":false,"importLoaders":1}!./node_modules/postcss-loader?{"ident":"postcss"}!./node_modules/sass-loader/lib/loader.js?{"sourceMap":false,"precision":8,"includePaths":[]}!./src/assets/theme.scss
Module build failed:
undefined
                                          ^
      Undefined variable: "$mat-fast-out-slow-in-timing-function".
      in<EMAIL_ADDRESS>(line 132, column 44)
Error:
undefined
                                          ^
      Undefined variable: "$mat-fast-out-slow-in-timing-function".
      in<EMAIL_ADDRESS>(line 132, column 44)
    at options.error (c:\something\node_modules\node-sass\lib\index.js:291:26)
 @ ./src/assets/theme.scss 4:14-222
 @ multi ./src/styles.css ./src/assets/css/loader.css ./src/assets/css/social.css ./src/assets/css/r.css ./src/assets/css/dragula/dragula.min.css ./src/assets/theme.scss<EMAIL_ADDRESS>
My theme is:
@import '~@covalent/core/theming/all-theme';
@import '~@angular/material/theming';

// Plus imports for other components in your app.

// Include the common styles for Angular Material. We include this here so that you only
// have to load a single css file for Angular Material in your app.
// Be sure that you only ever include this mixin once!
@include mat-core();

// Define the palettes for your theme using the Material Design palettes available in palette.scss
// (imported above). For each palette, you can optionally specify a default, lighter, and darker
// hue.
$app-primary: mat-palette($mat-indigo);
$app-accent: mat-palette($mat-red, 500);

// The warn palette is optional (defaults to red).
$app-warn: mat-palette($mat-red);

// Create the theme object (a Sass map containing all of the palettes).
$app-theme: mat-light-theme($app-primary, $app-accent, $app-warn);

// Include theme styles for core and each component used in your app.
// Alternatively, you can import and @include the theme mixins for each component
// that you are using.
@include angular-material-theme($app-theme);
@include covalent-theme($app-theme);

.angular-cli.json:
"styles": [
        "styles.css",
        "assets/theme.scss",
<EMAIL_ADDRESS>
      ],

Which version of Angular and Material, and which browser and OS does this issue affect?
"@angular/animations": "4.3.6",
    "@angular/cdk": "2.0.0-beta.8",
    "@angular/common": "4.3.6",
    "@angular/compiler": "4.3.6",
    "@angular/core": "4.3.6",
    "@angular/forms": "4.3.6",
    "@angular/http": "4.3.6",
    "@angular/material": "2.0.0-beta.8",
    "@angular/platform-browser": "4.3.6",
    "@angular/platform-browser-dynamic": "4.3.6",
    "@angular/router": "4.3.6",
    "@covalent/core": "1.0.0-beta.6",
    "@ngx-translate/core": "7.2.0",
    "@ngx-translate/http-loader": "1.0.2",
    "@swimlane/ngx-charts": "6.0.2",
    "d3": "4.10.0",
    "angular2-jwt": "0.2.3",
    "core-js": "2.5.1",
    "hammerjs": "2.0.8",
    "moment": "2.18.1",
    "ng2-dragula": "1.5.0",
    "rxjs": "5.4.3",
    "stacktrace-js": "2.0.0",
    "underscore": "1.8.3",
    "zone.js": "0.8.17",
    "@angular/flex-layout": "2.0.0-beta.9"

Do you have any ideas why this might be?
Thank you!

Can you target the .css file instead of the .scss? Typically we expect people to use that one, but we will start testing the .scss file addition too.

Yeah, I initially tried with the .css, but had the same result :/ Not sure
what I can test since this seems to be quite basic setup.
On Sep 1, 2017 19:25, "Ed Morales"<EMAIL_ADDRESS>wrote:

Can you target the .css file instead of the .scss? Typically we expect
people to use that one, but we will start testing the .scss file addition
too.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Teradata/covalent/issues/851#issuecomment-326638170,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGzoR1xCvmx7yjcK18IJMtTRu3GDpVX_ks5seD4CgaJpZM4PJ_pu
.

hmm.. i cant replicate this.. so im not sure why this is happening. Gonna keep it open just in case.

To be it seems like the material theme is expecting a '$mat-fast-out-slow-in-timing-function' variable to be defined in<EMAIL_ADDRESS>which is not the case on my end (using latest build of covalent + material 2.0.0-beta.8 (since the last one is not yet supported). Any idea why this might be? How does this look on your end?

turns out we were missing a variable.. that we forgot to rename to the mat- preffix.


Kong behind an outgoing corporate proxy

Hi,
how would one configure kong to respect an outgoing corporate proxy for external up_stream urls?
Right now it looks like $http_proxy, $https_proxy, $no_proxy are not respected. Looking at nginx there is something like proxy_pass as configuration option but the question is how to configure this propper without breaking kong behavoir.
The environment is: kong runs in a pod within openshift. External http traffic need to go through the outgoing http proxy.
I have tested a curl query against https://httpbin.org/ipwhich was successful and even a nodejs service is able to access resources (they do both respect the http_proxy settings).
What options do I have to configure this propper? Are there maybe any lua-(restiy) plugins that could help?
Thanks for your feedback
Best Regards
Felix
P.S.: Not sure but this could maybe relate to https://github.com/Kong/kong/issues/1271

@felixbecker you may want to consider looking at https://getkong.org/plugins/ee-forward-proxy.

Thx, I guess then this will be the way to go ...

What about Kong Community Edition?


[BUG] I didnt got the money

I did not got any money from this money print project

I want the money

💀

💸💸


Add support for Ceph RBD erasure coded pools

Since Ceph Luminous it's possible for RBD volumes to be created in erasure backed pools. LXD currently can't work with these pools since the command line arguments to create a image in an ec pool is a tiny bit different. (requires the --data-pool flag)
Is there any interest in supporting erasure coded pools that already exist in a Ceph cluster? I'm going to assume creating erasure coded pools through LXD is a bit outside the scope.
Willing to submit a PR if this is something you would considering merging.

Should be fine, especially if this can be detected somehow. If not, then we'd need a ceph.XYZ config option added to the storage pool config.
Also worth noting that this would only apply to manually built/installed LXD for the next couple of months at least as the LXD snap is still built from Ubuntu 16.04 an so only has CEPH 10.x.
Moving to a 18.04 base is planned and the work already done, but switching over now would break a lot of our users due to a bug in snapd, we need to wait for all distributions we care about to be on snapd 2.40 or higher prior to making the switch.

So anyway, if you're interested in doing the work and you understand what needs to be done, go ahead, it's fine with me :)

I took a stab at it in #6297.
I'm fine with building it from source, I was more worried about having to keep my fork and mainline in sync if you wouldn't want this added to the official code.

Thanks @maran for implementing this!

Hi @stgraber sorry for the necro on this but it seemed like the best place to discuss it.
I just upgraded my cluster from a local build LXD to the latest snap 9.20 release but it seems CEPH 10.x is still bundled with that. Hence the --data-pool feature is not working.
Was this meant to be solved by now (upgrading to a newer Ceph) or is there some other way to use a non-bundled Ceph version with the snap?

The edge channel is now built on 18.04 and so has a newer version of Ceph.
We're waiting to synchronize with some other snaps before we switch candidate/stable over too.

We're hoping to have this sorted over the next month.

Great, for now I will continue to try and build a version from the release source. Thanks.


pyranges Subtract does not work in pyranges v0.0.110

The Subtract method does not work with the latest version of pyranges.
The problem stems from a change in a newer version of the NCLS module.
I am not sure which version of NCLS is causing the problem.
NCLS version 0.0.53 works.
NCLS version 0.0.62 does not.
Error traceback:
File "/lib/python3.8/site-packages/pyranges/methods/subtraction.py", line 64, in _subtraction
    idx_self, new_starts, new_ends = o.set_difference_helper(
  File "ncls/src/ncls32.pyx", line 450, in ncls.src.ncls32.NCLS32.set_difference_helper
TypeError: set_difference_helper() takes exactly 3 positional arguments (4 given)

You should update NCLS. I should have required a minimum NCLS version in pyranges. Will fix tonight.
thanks!

Yes!
thanks @endrebak !
side-note: after my dirty fix I was also noticing a weird behavior each 1024 rows??? ... I guess my fix was really dirty
thanks a lot for the software!

The old NCLS had a bug: it only found the first 1024 intervals of overlap.
That might explain it.
Thanks for reporting and testing all of you :)
On Tue, Nov 2, 2021 at 6:33 PM Francois Serra @.***>
wrote:

Yes!
thanks @endrebak https://github.com/endrebak !
side-note: after my dirty fix I was also noticing a weird behavior each
1024 rows??? ... I guess my fix was really dirty
thanks a lot for the software!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore-ntnu/pyranges/issues/224#issuecomment-957974535,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEHURUX2KZKHAJBJSBWX66TUKAOFVANCNFSM5FDN55LQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Use executor to find swiftc path

Warnings

:warning:

Any changes to library code should be reflected in the Changelog.
Please consider adding a note there and adhere to the Changelog Guidelines.

  Generated by :no_entry_sign: Danger Swift against 56ac3d6dc3924bfd0440f3d72fcc6dbb180de0c5

merge on green


Keyboard shortcut shift+enter undesirable behavior

How can I get shift+enter to do triple duty

when text is not selected in a python file with #%%, run cell in Interactive Python pane and advance.
when text is selected in a python file with #%%,  run the selected text in in Interactive Python pane.
when text is selected in a python file without #%%,  run the selected text in in Interactive Python pane.

I used to be able to press shift+enter to do all three.  This does not work anymore.  Now shift+enter in python files without #%% always starts the Terminal.  When I change that shortcut to not point to "python.execSelectionInTerminal" anymore, then shift+enter just deletes the selected text in files without any #%%.
I have tried changing keybindings.json from
(note: the bang ! in front of !python.datascience.ownsSelection is the only difference):
    {
        "key": "shift+enter",
        "command": "python.datascience.execSelectionInteractive",
        "when": "editorFocus && python.datascience.featureenabled && python.datascience.ownsSelection && !findInputFocussed && !replaceInputFocussed && editorLangId == 'python'"
    },

to:
    {
        "key": "shift+enter",
        "command": "python.datascience.execSelectionInteractive",
        "when": "editorFocus && python.datascience.featureenabled && !python.datascience.ownsSelection && !findInputFocussed && !replaceInputFocussed && editorLangId == 'python'"
    },

The following change allows me to run selected text in the Interactive Python pane from files that don't  have 'cells'.  However, now shift+enter deletes selected text in python files with 'cells'.
(&& !python.datascience.hascodecells added to the end)
   {
        "key": "shift+enter",
        "command": "python.datascience.execSelectionInteractive",
        "when": "editorFocus && !findInputFocussed && !python.datascience.ownsSelection && !replaceInputFocussed && editorLangId == 'python' && !python.datascience.hascodecells"
    }

Ok, so it looks like I had to add all of the options to the user file keybindings.json.  Not sure if I have some unnecessary redundancy here, but the following seems to be working.
    {
        "key": "shift+enter",
        "command": "python.datascience.execSelectionInteractive",
        "when": "editorFocus && !python.datascience.hascodecells && !findInputFocussed && !python.datascience.ownsSelection && !replaceInputFocussed && editorLangId == 'python'"
    },
    {
        "key": "shift+enter",
        "command": "python.datascience.execSelectionInteractive",
        "when": "editorFocus && python.datascience.hascodecells && !findInputFocussed && !python.datascience.ownsSelection && !replaceInputFocussed && editorLangId == 'python'"
    },
    {
        "key": "shift+enter",
        "command": "python.datascience.execSelectionInteractive",
        "when": "editorFocus && python.datascience.featureenabled && python.datascience.ownsSelection && !findInputFocussed && !replaceInputFocussed && editorLangId == 'python'"
    },
    {
        "key": "shift+enter",
        "command": "python.datascience.runcurrentcelladvance",
        "when": "editorFocus && python.datascience.featureenabled && python.datascience.hascodecells && !editorHasSelection"
    },
    {
        "key": "ctrl+shift+alt+enter",
        "command": "python.execSelectionInTerminal",
        "when": "editorFocus && !findInputFocussed && !python.datascience.ownsSelection && !replaceInputFocussed && editorLangId == 'python'"
    },
    {
        "key": "shift+enter",
        "command": "-python.execSelectionInTerminal",
        "when": "editorFocus && !findInputFocussed && !python.datascience.ownsSelection && !replaceInputFocussed && editorLangId == 'python'"
    }


[BUG] Resource ID's returned from rest api do not respect case of resource group names created in portal

Describe the bug
When resource ids are returned by the az cli they respect capitalization of resource group created by the user (for example resourceGroup) but all resource group ids returned across any service invocation are always upper-cased.
To Reproduce
Example AZ cli invocations:
az group list
  {
    "id": "/subscriptions/f0319a3f-4404-48f3-8a1e-06b0d38b0dcb/resourceGroups/anuj-group",
    "location": "eastus",
    "managedBy": null,
    "name": "anuj-group",
    "properties": {
      "provisioningState": "Succeeded"
    },
    "tags": {},
    "type": "Microsoft.Resources/resourceGroups"
  }

az vm list --resource-group anuj-group
  {

    "id": "/subscriptions/f0319a3f-4404-48f3-8a1e-06b0d38b0dcb/resourceGroups/anuj-group/providers/Microsoft.Compute/virtualMachines/anuj",
    "networkProfile": {
      "networkInterfaces": [
        {
          "deleteOption": null,
          "id": "/subscriptions/f0319a3f-4404-48f3-8a1e-06b0d38b0dcb/resourceGroups/anuj-group/providers/Microsoft.Network/networkInterfaces/anuj411",
          "primary": null,
          "resourceGroup": "anuj-group"
        }
      ]
    },
    "osProfile": {
    ...
    },
    "storageProfile": {
      "dataDisks": [],
      "osDisk": {
        "managedDisk": {
          "diskEncryptionSet": null,
          "id": "/subscriptions/f0319a3f-4404-48f3-8a1e-06b0d38b0dcb/resourceGroups/anuj-group/providers/Microsoft.Compute/disks/anuj_disk1_b9821ce0ce9e44a1a0a5bda94f26378d",
          "resourceGroup": "anuj-group",
        },
      }
    },
  }

However, when retrieving the result using the SDK VM list operation we get a resource id like:
/subscriptions/f0319a3f-4404-48f3-8a1e-06b0d38b0dcb/resourceGroups/CLOUDKART-C0-RESOURCES/providers/Microsoft.Compute/virtualMachines/TestVM
Code Snippet
Any listing of a service that has a resource ID
Expected behavior
Expected that the returned resource ID should match the case sensitivity of the created resource group. Could not find any documentation around this in Azure docs.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

Thanks for filing this GitHub issue, @dclusin-cisco.  @weidongxu-microsoft could you please help route this?

@dclusin-cisco
I think resource ID from ARM is case insensitive.
An indirect ref https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/resource-api-reference.md#put-resource

The resource group names and resource names should be matched case insensitively. That means, for example, if a user creates a resource in resource group "rG1", and then calls a read operation on "RG1", the same resource should be returned even though the casing differs.


📝 Correction | Canadian Key Disclosure status

Description
Canada is listed under countries where individuals are subject to key disclosure laws, yet the link to the Wikipedia citation actually reads as the opposite - that attempts to compel individuals to turn over keys have failed in court and that the Canadian Charter of Rights and Freedoms enshrines the right to protect against self-incrimination.
Why I am making the suggestion
I don't know what the legal status actually is in Canada, but the citation provided in the list actually says the opposite of what Canada's listing in that column suggests.
My connection with the software
None

[x] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

I've update your issue, please use the templates provided in future. This is definately something to look into, so thanks for opening the issue.

Ok thanks - sorry will do in the future.

Yes, we need to put a PR in for this. I had forgotten about it https://github.com/privacytools/privacytools.io/issues/1167#issuecomment-604583930


Updates for clearing filter values after actions.

Coverage decreased (-0.05%) to 54.975% when pulling a8233a5b9a02f88e9e9a8c7aa657a4481d042343 on wknet123:dev-sync into 7d42b73f832754c555211ef94c6af4f086887559 on vmware:dev.


Is it actual?

In symfony 6.3 we see a new #[MapRequestPayload] attribute. Symfony map request to DTO (and validate).
https://symfony.com/blog/new-in-symfony-6-3-mapping-request-data-to-typed-objects

Hello
I will edit readme and add info in composer with info about Symfony feature. I don't want to set lib as depricated because still a lot of projects are on Symfony version less than 6.3 :)

Symfony doesn't have ability to work with multipart data (for example with files) so this library is still very useful

Hi @prugala
Can I still use your library for Symfony 6.4 or are there new features in Symfony that make it unnecessary? Thank you!

@m4n50n in Symfony 6.4 I would suggest using Symfony feature: https://symfony.com/blog/new-in-symfony-6-3-mapping-request-data-to-typed-objects


Added Rust Lessons

Title: Introduction to Rust
Author: dvigneshwer and FOXIOT study group Coimbatore
Prerequisites: Open mind to learn a state of art programming language
Time to Complete: 3 hr
SHORT Summary/Context/Objectives: Learn about Rust programming language and main components to write your first application with rust lang.
Link to lesson : https://github.com/dvigneshwer/studyGroupLessons/tree/master/Rust-intro
USE the tags for level, topic, and language: Rust, System Programming language

WOW, excellent @dvigneshwer ! I'm so glad you've added rust to your curriculum. 🚀


Allow passing boolean flag args like --force to cask install

Sometimes you just want to bulldoze over existing packages 😛

How does this work currently? Does it need added to every line of the Brewfile? I'm wondering if either a global DSL and/or a brew bundle --force would be better? Thoughts?

I get it now! Seems reasonable! Thanks for your contribution to Homebrew!  You rock!


Failed to get subscriptions via proxy with authentication when gateway is down

Describe the bug
Related to issue : https://github.com/Azure/azure-sdk-for-java/issues/12289
@jianghaolu  Can you please check the related issue?
When try to get subscriptions by Azure.Authenticated object via proxy that was configured in java System properties, proxy's user and password are not been send to proxy.
Issue partially resolved in case #12289 ,
But still failing when removing gateway , which is going out to internet.
In case I remove authentication from my proxy get subscriptions call success  without gateway as well
Exception or Stack Trace
Wrapped by: java.util.concurrent.ExecutionException: java.net.SocketException: Network is unreachable (connect failed)
at java.util.concurrent.FutureTask.report(Unknown Source)
at java.util.concurrent.FutureTask.get(Unknown Source)
at com.microsoft.azure.credentials.ApplicationTokenCredentials.acquireAccessToken(ApplicationTokenCredentials.java:160)
... 46 common frames omitted
Wrapped by: java.io.IOException: java.net.SocketException: Network is unreachable (connect failed)
at com.microsoft.azure.credentials.ApplicationTokenCredentials.acquireAccessToken(ApplicationTokenCredentials.java:174)
at com.microsoft.azure.credentials.ApplicationTokenCredentials.getToken(ApplicationTokenCredentials.java:139)
at com.microsoft.azure.credentials.AzureTokenCredentials.getToken(AzureTokenCredentials.java:74)
at com.microsoft.azure.credentials.AzureTokenCredentialsInterceptor.intercept(AzureTokenCredentialsInterceptor.java:36)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
at com.microsoft.azure.management.resources.fluentcore.utils.ResourceManagerThrottlingInterceptor.intercept(ResourceManagerThrottlingInterceptor.java:54)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
at com.microsoft.azure.management.resources.fluentcore.utils.ProviderRegistrationInterceptor.intercept(ProviderRegistrationInterceptor.java:40)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
at com.microsoft.rest.interceptors.BaseUrlHandler.intercept(BaseUrlHandler.java:43)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
at com.microsoft.rest.interceptors.RequestIdHeaderInterceptor.intercept(RequestIdHeaderInterceptor.java:29)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:147)
at okhttp3.internal.http.RealInterceptorChain.proceed(RealInterceptorChain.java:121)
at okhttp3.RealCall.getResponseWithInterceptorChain(RealCall.java:264)
at okhttp3.RealCall.execute(RealCall.java:93)
at retrofit2.OkHttpCall.execute(OkHttpCall.java:188)
at retrofit2.adapter.rxjava.CallExecuteOnSubscribe.call(CallExecuteOnSubscribe.java:40)
at retrofit2.adapter.rxjava.CallExecuteOnSubscribe.call(CallExecuteOnSubscribe.java:24)
at rx.Observable.unsafeSubscribe(Observable.java:10327)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:48)
at rx.internal.operators.OnSubscribeMap.call(OnSubscribeMap.java:33)
at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:48)
at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:30)
at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:48)
at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:30)
at rx.Observable.subscribe(Observable.java:10423)
at rx.Observable.subscribe(Observable.java:10390)
at rx.observables.BlockingObservable.blockForSingle(BlockingObservable.java:443)
... 14 common frames omitted
Wrapped by: java.lang.RuntimeException: java.io.IOException: java.net.SocketException: Network is unreachable (connect failed)
at rx.exceptions.Exceptions.propagate(Exceptions.java:57)
at rx.observables.BlockingObservable.blockForSingle(BlockingObservable.java:463)
at rx.observables.BlockingObservable.single(BlockingObservable.java:340)
at com.microsoft.azure.management.resources.implementation.SubscriptionsInner.list(SubscriptionsInner.java:244)
at com.microsoft.azure.management.resources.implementation.SubscriptionsImpl.list(SubscriptionsImpl.java:38)
at com.emc.cloud_dr.cdr.azure_adapter.service.impl.factory.ProxyAzureTest.testAzureCallThroughProxy(ProxyAzureTest.java:58)
at com.emc.cloud_dr.cdr.azure_adapter.service.impl.factory.ProxyAzureTest.proxyAzureTest(ProxyAzureTest.java:47)
To Reproduce

create Azure Azure.Authenticated object with withSslSocketFactory as suggested in case
https://github.com/Azure/azure-sdk-for-java/issues/12289
try to get subscriptions - call succeded
Delete gateway that going out to the internet from the machine where java is running(proxy is still reachable and internet is accessible via proxy from this machine, using  "curl -x http://prxyIp:proxyPort  --proxy-user pxuser:prxpass  -I http://google.com" cli command)
try to get subscribtions - failing

Code Snippet
ApplicationTokenCredentials credentials = new ApplicationTokenCredentials(clientId, domain, secretKey, AzureEnvironment.AZURE);
credentials.withSslSocketFactory(new SSLTunnelSocketFactory(host, port, username, password));
Azure.Authenticated authenticate = Azure.configure().withConnectionTimeout(20, TimeUnit.SECONDS).authenticate(credentials);
log.debug("authenticate:  " + authenticate.toString());
PagedList subscriptions = authenticate.subscriptions().list();
Expected behavior
connection to proxy should success
Setup (please complete the following information):

OS: [e.g. iOS]
IDE : [e.g. IntelliJ]
Version of the Library used

OS: Linux and Windows
IDE : IntelliJ
Version of the Library used: com.microsoft.azure.management:1.33.0

Thanks for filing this issue @MarinaTeper.  Someone from the Mgmt team will assist you shortly.  /cc @xccc-msft

@jianghaolu Can you please help check this? Thanks.

@jianghaolu  Can you please help

Hello, @jianghaolu  ,we're waiting for reply, can you please help?

Hey @MarinaTeper Super sorry for the late reply 😢
In the currently managed SDKs 2 proxy setups are required - one for authentication, and one for connecting to Resource manager endpoints. You code snippet has handled authentication proxy but you also need proxy configured on the Azure instance.
ApplicationTokenCredentials credentials = new ApplicationTokenCredentials(clientId, domain, secretKey, AzureEnvironment.AZURE);
Authenticator proxyAuthenticator = new Authenticator() {
      @Override
      public Request authenticate(Route route, Response response) throws IOException {
             String credential = Credentials.basic(username, password);
             Request request = response.request().newBuilder().header("Proxy-Authorization", credential).build();
             return request;
      }
};
Azure.Authenticated authenticate = Azure.configure()
    .withConnectionTimeout(20, TimeUnit.SECONDS)
    .withProxy(new Proxy(Type.HTTP, new InetSocketAddress(host, port)))
    .withProxyAuthenticator(proxyAuthenticator)
    .authenticate(credentials);
// The next 2 calls must happen after Azure.Authenticated is created. The order is important
credentials.withProxy(null);
credentials.withSslSocketFactory(new SSLTunnelSocketFactory(host, port, username, password));
PagedList subscriptions = authenticate.subscriptions().list();

This is a less than ideal workaround for proxies with authentication. In the next major version of the SDKs you will only need to configure the proxy once.

Please verify if this is still an ongoing issue, thanks!


Still randomly crashes between loading zones

I'm not sure what's causing it, but it happens with both the cartridge and digital version. Sometimes when going through loading zones, or even warping; It just randomly (very rarely) boots me out of the game and says "an unknown error had occurred". I don't have "exception handlers" enabled, but this issue STILL hasn't been fixed. Have no idea what is causing it. I have to constantly save every 2-5 minutes. It's so annoying and I have no idea what could be causing it. Again, it's very rare and usually happens only once per seed.

Oops, wrong one, meant to post this in the OoT3R one.


Close dialogue box for inline elements

When I mark a text in the visual editor, then the format line for inline elements opens and it does not close when I write more text.

this specific behavior is not fixed, but the inline-editor box is improved generally. Check if this is enough...


chore: auto update self hosted compose files

What kind of change does this PR introduce?
self-hosted
What is the new behavior?
Every stable release will create a PR https://github.com/supabase/supabase/pull/15330 to update self-hosted compose files.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 61.8%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
7
7.94%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
4745

Relevant Lines:
7678

💛  - Coveralls


Consider uploading artifacts to a central maven repository

Uploading gwt-ol3 artifacts to a central maven repository would be facilitate the use of it in maven projects. Probably this process can be supported by CI.
Ressources:
https://maven.apache.org/guides/mini/guide-central-repository-upload.html

If you have interest, I can help with this. If you want to use sonatype and deploy using the packages starting with de.desjardins you can request access sending a message like this https://issues.sonatype.org/browse/OSSRH-21191 and be able to demostrate that you have access to this domain de.desjardins. So if they grant access to de.desjardins, you will be able to deploy de.desjardins.ol3:gwt-ol3. http://central.sonatype.org/pages/ossrh-guide.html
If you go ahead, I'll create a pull request with the changes required in your pom. And you will need to configure gpg and upload your public key to a server (http://central.sonatype.org/pages/working-with-pgp-signatures.html).
Recommendation: The client lib is in the package ol, I don't like it 😜, I prefer de.desjardins.ol3 which is the maven group of your project, but if you are sure about this, at least you should move GwtOL3.gwt.xml into ol package.

Hello @ibaca ,
yes, I have interest. Thanks for your offer. I already created an account at central sonatype. What do you mean with demonstrate access to the domain de.desjardins? The domain doesn't really exist. It's only a namespace.
I also don't like the package ol but this is the original namespace of ol3. When I created the wrapper, the idea was to make it as closely as possible to the ol3 API. The advantage is that you can better find your way when developing the wrapper and you can use the apidoc of ol3. You also don't need to add jsinterop namespace annotations.
When thinking in Maven it will be usefull to change the namespace. Then all the stuff could be deployed under the same namespace, right?
Regards
Tino

Yep, maybe you are right, ol is not that bad, even is more natural. This is a JS wrapper, and be close to the JS lib might be a good idea. But, what about moving the GwtOL3.gwt.xml into ol package?
About 'demonstrate access'; just request access to de.desjardins, someone from sonatype will ask you something, or maybe they just grant access, not sure 😛 just ask.

What are the benefits when moving GwtOL3.gwt.xml into ol package? There are a lot of files directly under ol that are then not in the source path.

You can add this <source path=""/> to the GwtOL3.gwt.xml so the module can be inside its own package. IMHO is more common (used in libs like https://github.com/intendia-oss/rxjava-gwt/blob/master/src/main/resources/rx/Rx.gwt.xml). I suppose there aren't any specific benefits.

It is more common because rxjava-gwt is a reference project ?! ;-)
The module xml files are moved now.
Also the groupid com.github.tdesjardins is accepted at sonatype. So I will change it in the maven configuration. de.desjardins was not accepted because of the missing ownership of the domain.

The sonatype configuration is prepared: https://issues.sonatype.org/browse/OSSRH-24393 .
I also changed the groupid in the maven configuration and uploaded a public gpg key to a server. Now I have to configure the maven deploy plugin, right?

This pull request https://github.com/TDesjardins/gwt-ol3/pull/33 should cotain all configuration you need. You can do right now a mvn clean release:prepare release:perform, this will create various commits, one tag and will deploy to the sonatype staging repository. Don't worry if something goes wrong, you can always remove the tag, make a hard reset and push force to this repo and discard the staging repo created in oss.sonatype.org.

Thanks @ibaca ! I tried different parameters and configurations (settings.xml) with release:perform and now the checkout seems to work but I get the following error message when executing goal deploy site-deploy:
Failed to execute goal org.apache.maven.plugins:maven-gpg-plugin:1.1:sign (sign-artifacts) on project gwt-ol3: Cannot obtain passphrase in batch mode -> [Help 1]
I suppose that I have to set some settings in settings.xml for gpg signing. I already set a passphrase but   that doesn't fix the issue. Maybe I need references to the public and the private key. How the process knows where to get the gpg infos?

First of all, I see that you are using a stable version right now, you should change it to a SNAPSHOT version (you can use mvn versions:set -DnewVersion=2.5.0-SNAPSHOT), the release plugin will ask you which stable version do you want to use in the first step.
About your problem, try to execute one goal a time, I think that the problem is related to deploy, that cannot be executed in batch mode because it need the password to use gpg. In any case, do not use --batch-mode to fix this error.
And maybe you are having some problems with the site-deploy, we can see whats happening later, but for now I thinks its better that you just remove the distibutionManagement.site section. I think that deploying the showcase to this url will be much more useful.

That's a good hint. I havn't used the batch mode intentionally. I use the maven integration of eclipse which runs all commands in batch mode :-) .
With the console I managed it somehow and now there are artifacts at central maven repository. I don't know if this is a correct release because all artifacts are in a folder snapshot, also in the release repositories:
https://oss.sonatype.org/content/groups/public/com/github/tdesjardins/
https://oss.sonatype.org/content/groups/staging/com/github/tdesjardins/
https://oss.sonatype.org/content/repositories/snapshots/com/github/tdesjardins/
Could you check this?
Furthermore I think also the module gwt-ol3 should be exluded from the release.

It's not published in central, don't worry. You should make the close and release in oss.sonatype.org, but do not make it until you confirm everything is ok 😛.

User always the terminal, and do not keep open eclipse during release (at least for now, eclipse can auto-compile, adding files to the final artifact or making it fail to build, etc).
Releasing should be just mvn clean (this is a good idea), mvn release:prepare (you should set a stable version, and configure the next snapshot version), mvn release:perform (should deploy to sonatype creating the staging repository in oss.sonatype.org which should be closed and released).

Current version of the master branch is still a stable version, please change to 2.5.0-SNAPSHOT.

gwt-ol3 is now available at central maven repository. Thank you very much @ibaca !


Allow BS and SL Testing from Fork PRs

Did you allow access to GitHub secrets from forks somehow? Because otherwise, any PRs from forks will fail because they don't have access to the secrets.

 I think we should, however, should have a mechanism for PRs from forks to test in SauceLabs and BrowserStack. I am going to move this discussion to a bug!
Originally posted by @jginsburgn in https://github.com/karma-runner/karma/issues/3779#issuecomment-1095965695

I think that it important to fully test a PR before allowing it to be merged to master. For this reason, we should come up with a mechanism of allowing conducting all tests, including the ones that need credentials for 3P services (like SauceLabs and BrowserStack) from fork PRs, without openly revealing such credentials. An idea I have is to have a pre-master, which would be the base branch for fork PRs. It should be protected as the master branch, and once a merge is conducted, it should conduct all integration tests. If any fails, we should hard reset that pre-master branch and reopen the original PR displaying the failure. If they all pass, we should fast forward master to pre-master as a workflow.
@devoto13 thoughts?
@karma-runner/google-web-test-team

BTW, this is also applicable in the following repos:

https://github.com/karma-runner/integration-tests
https://github.com/karma-runner/karma-sauce-launcher
https://github.com/karma-runner/karma-browserstack-launcher

I'm not sure, it feels like it would complicate the workflow and create more friction for the external contributors. We're going to have two branches, shall we also have pre-beta branch? How can we enforce/communicate that external PRs should be opened against those branches?
IMO the current system is fine. If the PR looks risky from the cross-browser compatibility perspective, then the maintainer can always push the branch to the main repo and run BS/SauceLabs tests before merging. I think the effort would be pretty much the same.
I also think that the post-merge BS failures will become very rare if we manage to drop the IE support in the next major as all supported browsers will be evergreen.
Having said that I agree that it's nice to be able to run all tests (including 3P services) in all PRs, I just don't see how we can achieve that without introducing a lot of extra complexity.

@devoto13
What you say is true. I agree.

If the PR looks risky from the cross-browser compatibility perspective, then the maintainer can always push the branch to the main repo and run BS/SauceLabs tests before merging. I think the effort would be pretty much the same.

Let's stick with this.


Rescue pool cannot reconnect in Redis 5

When trying to update to redis 5 from redis 4.8, I get the following error:
"undefined method reconnect' for #Redis::Client:0x000055c8e8ac9c78 (NoMethodError)"`
Which is originating from my lib/tasks/resque.rake file, with the following line as recommended in the readme of this project:
Resque.redis._client.reconnect
I believe that this is because reconnect is now a private method as of redis 5, as mentioned here: https://github.com/resque/resque/issues/1825
I hope we can resolve this, I wouldn't know how to fix it myself but would help if I could

@DennisWormald just replace that by Resque.redis.close. And you should be set.

I also added ":inherit_socket => true"

I very highly recommend not to do this, that will break your app in tons of ways.

Big thanks to @casperisfine , this appears to work fine and I can leave everything else the same. Here is what I put in my rakefile (slightly refactored version of the code currently in readme.md, that is suggested to be put in "resque.rake" file). I left everything else the same, all working now as it used to I think, will test more over the next few days. I won't close the issue until we decide if the readme.md needs to be updated or not, as this is a redis-version dependent change.
task "resque:pool:setup" do
  # close any sockets or files in pool master
  ActiveRecord::Base.connection.disconnect!
  # and re-open them in the resque worker parent
  Resque::Pool.after_prefork do |job|
    ActiveRecord::Base.establish_connection
    Resque.redis.close
  end
end

Ok I hadn't tested it properly, I tried it again this morning and I now have the following issue: https://github.com/resque/resque/issues/1826
And it seems a few other people are having the same problem. Will follow and see.

okay all fixed with new updates to resque 2.4.0 and redis-rb 5.0.2 thanks @casperisfine and team


File output causes IOError because of temp_file.close

Removing the call to .close will fix it. You can't access an instance of Tempfile after close is called. Perhaps adding another method like pass.cleanup! that closes the temp file is a good idea.

Hi,
Could you please provide a use case in order to reproduce this IOError


[bug 976346] Link language names to user lists corresponding to those languages

The code is based on https://github.com/mozilla/mozillians/pull/995 with the feedback in the ticket.
Also not contain the langname_to_code because the data in the user profile is saved already as code and not language so in my code the url are: mozillians.org/it/language/en/. In that way we can support also other strange codes if supported by the editing profile that use django facilities for it.
Also I added in the profile, where there is the language list of the user links for that list by that language.

Coverage increased (+0.01%) to 92.966% when pulling 910f54f4a8e948aa50d10e47c18052f3c912f85f on Mte90:Mte90/976346 into 338097d0a54849a43e5d2c0cdc07f96f809ae5ad on mozilla:master.

Coverage increased (+0.01%) to 92.928% when pulling 7b1634218ec39815bb21e55aaae949f7a687c7ec on Mte90:Mte90/976346 into dd47e51172668eb0e1fbe6967841e99d7bb9dcfb on mozilla:master.

ok pr splitted in two commits :-D

Coverage increased (+0.01%) to 92.928% when pulling c045ae11716797f6f1e06f5a2d2314f6fafb22a9 on Mte90:Mte90/976346 into dd47e51172668eb0e1fbe6967841e99d7bb9dcfb on mozilla:master.

any news for that?

Done it, now I have to see how to merge the last commit in the first one in a simple way.

Coverage increased (+0.01%) to 92.793% when pulling<PHONE_NUMBER>2c99ceb056e06b171d1bc30fd00213 on Mte90:Mte90/976346 into 17848a478448914999e103606ac1ae178fe267f3 on mozilla:master.

I think you also have the first two commits backwards. My suggestion would be to reset and rewrite the history of these two commits, since it's not massive lines of code changes.

done :-)

Coverage increased (+0.01%) to 92.793% when pulling c30035d0a3e1ece5d6f0c1db054aaa818b2d57a6 on Mte90:Mte90/976346 into 17848a478448914999e103606ac1ae178fe267f3 on mozilla:master.

Nice! Could also ammend the second commit to add the link the home page too. This should be here.

Uhm I didn't understand :-/

When you login you are redirect to the home page. On the left sidebar you can see some basic personal information, like groups, skills and languages you speak. It would be nice if languages were linked there too.

Gotcha! Done it!

Coverage increased (+0.01%) to 92.793% when pulling 16c2052cf65675285027b2530fa75ee7456d2f13 on Mte90:Mte90/976346 into 17848a478448914999e103606ac1ae178fe267f3 on mozilla:master.

We put this on hold temporarily, since we had some more pressing issues to merge and deploy. In the meantime @Mte90 would you be interested in refactoring the code a bit? It seems that language list is almost identical with location list, both template and view. It probably makes sense to consolidate them and have on view and one template for both.

I am not sure to have time to improve the code (busy months are coming) :-/

@Mte90 that's totally fine. Thank you again for this PR. Awesome work!
@comzeradd can you please continue the work on this so that we can deploy it to prod soon?

Yes, we can leave this open to keep track till I open a new PR.

Closing this in favor of #1674


Add NHDv2 gridmet targets to pipeline

This PR adds two targets to the pipeline, including a file target that represents the NHD-aggregated gridmet driver data for the full DRB (p1_drb_nhd_gridmet) and another that subsets the gridmet data to those COMIDs where we have observations (p2_met_data_at_obs_sites).
Here's a preview of the subset data:
> tar_load(p2_met_data_at_obs_sites)
> head(p2_met_data_at_obs_sites)
# A tibble: 6 x 10
    COMID time                 tmmx  tmmn    pr  srad    vs  rmax  rmin     sph
    <dbl> <dttm>              <dbl> <dbl> <dbl> <dbl> <dbl> <dbl> <dbl>   <dbl>
1 4492036 1978-12-31 19:00:00  61.9  47.8 0.481  35.8  9.57  86.1  58.5 0.00628
2 4495680 1978-12-31 19:00:00  59.3  48.8 0.632  35.6  7.70  82.8  66.0 0.00686
3 4495656 1978-12-31 19:00:00  58.4  48.0 0.634  35.7  7.63  84.2  66.9 0.00678
4 4784831 1978-12-31 19:00:00  59.9  48.9 0.610  35.5  7.77  82.2  65.5 0.00690
5 4782619 1978-12-31 19:00:00  58.1  48.3 0.625  35.5  7.61  84.8  67.5 0.00679
6 4782585 1978-12-31 19:00:00  57.3  46.7 0.610  36.0  7.57  88.8  69.0 0.00673
>

These targets were taken from #137 and modified a bit. The gridmet driver data are treated as a plain 'file' format target as opposed to a target that integrates with S3 and has repository = aws. This assumes that the DRB-scale gridmet data are available in 1_fetch/in.

Just want to make sure I understand: the assert call within subset_nc_to_comids is to make sure that you are only filtering out 4781767, which is the zero area segment, is that correct?

Yeah, exactly. NHDv2 COMID's with AREASQKM == 0 will not have associated catchment areas and therefore won't have met driver data, either.
To aggregate the gridmet data, Jeff used catchment polygons that are created in the drb-network-prep repo (p1_nhdv2_catchments_sf here). Within that pipeline we now omit any zero-area COMID's before downloading the catchments. So this assert statement shouldn't be necessary, but we decided to leave it in for now as a sort of flag.

Awesome, thanks for the explanation, looks good then!


Update Storybook Playwright test CI job name and testing docs

Fixes

Follow-up to #2447
Description

This PR adds changes from the comments in #2447:

renames the CI job from storybook_vr to storybook to reflect that the tests now run both functional and visual regression tests.
Updates the documentation to add more details about storybook functional tests.

Testing Instructions

The changes should make sense and be readable. The CI should pass.
Checklist

[ ] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[ ] My pull request targets the default branch of the repository (main) or a parent feature branch.
[ ] My commit messages follow best practices.
[ ] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[ ] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

There are some references to the job name in the CI + CD docs, e.g. https://github.com/WordPress/openverse/blob/5a242e1052a8e06562fa5dce2274a121da9f5f10/documentation/meta/ci_cd/jobs/frontend.md#frontend-jobs that also need updating.

Thank you! I ran a search throughout the repository for storybook_vr, and that was the only occurrence (updated). Do you know of any other places I should check?

This PR should be merged after the infrastructure changes renaming the required GitHub CI check are applied.


BuildStream 1.4 not handling relative git URL's properly in submodules

See original issue on GitLab
In GitLab by [Gitlab user @nanonyme] on Sep 16, 2019, 13:36
Summary
BuildStream 1.4 not handling relative git URL's properly in submodules. URL's starting with ../ or ./ are supposed to be calculated from main Git URL.
Steps to reproduce
Have element that uses Git source with a submodule with relative URL.
What is the current bug behavior?
BuildStream tries to do a local clone.
What is the expected correct behavior?
BuildStream should clone a remote repo.
Relevant logs and/or screenshots
Possible fixes
We could calculate real URL at https://gitlab.com/BuildStream/buildstream/blob/1.4.1/buildstream/plugins/sources/git.py#L590 before creating GitMirror
Other relevant information
At least 1.4.1, also been present throughout 1.2.x. Not known if affects master.

BuildStream version affected: /milestone %BuildStream_v1.x

In GitLab by [Gitlab user @nanonyme] on Sep 16, 2019, 13:37
changed the description

In GitLab by [Gitlab user @nanonyme] on Sep 16, 2019, 15:42
mentioned in merge request freedesktop-sdk/freedesktop-sdk!1790

In GitLab by [Gitlab user @nanonyme] on Oct 14, 2019, 18:39
So this code is tested to do the right thing https://gitlab.com/BuildStream/bst-external/blob/master/bst_external/sources/git_tag.py#L690 ; I would prefer in 2.0 that submodule handling would work right straight from git plugin in core.


[Release-1.30] - go.mod requirement "inet.af/tcpproxy" unavailable, should be update

Backport fix for go.mod requirement "inet.af/tcpproxy" unavailable, should be update

#11024

validated on
https://github.com/rancher/distros-test-framework/actions/runs/11289920229


Version 1.19.2 Fabric on Modrinth

Forgive me if this is an annoying request. But I would really appropriate having an 1.19.2 Fabric release on Modrinth. Happy to help where I can

Unfortunately there isn't enough hours in the day for me to be able to code for all the versions people want. This is why I only code for the current version at the time that I am coding.


Event startup script

Is your feature request related to a problem? Please describe.
Create incident startup script to set metadata and set local feature templates
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Should accomplish the following

[ ] Set dynamic data on dynamic text table
[ ] Create feature templates
[ ] Takes webMap offline (if currently connected)
[ ] If not connected, spin up local EGDB and point data source to appropriate source
[ ] Create spatial bookmark from current extent
[ ] Set CRS (UTM?)
[ ] Set definition queries to exclude other incidents
[ ] Filter temporal data

Adding ideas to what the Event Startup Script should do:

[ ] Spatial Area Selection (Create Bookmark?)
[ ] Creates and Names selected Map Types (this is probably a later feature)
[ ] Creates and Names selected Layouts (this is probably a later feature)
[ ] If AGOL based then filters Feature Service to exclude past incidents?

@t-mahon so quite a few of these items do not have arcpy methods. I can think of a couple ways to skin this.

Use AGP Tasks - this appears to be tied to the Workflow Manager license which may make it unusable for some. This also prevents an AGOL start (although I'm not entirely sure we had that figured out to begin with).
We use a notebook. That enables both an AGOL and AGP start although it's not very clean.
We think more about relating features to the 'incident' table. This makes things more complicated but has the potential to tightly integrate all the data.

I think for the time being, I am open to creating a job-aid to fill in for this tool (at least a good portion of this tool). I think we can still integrate the following:

Setting dynamic text in 'dynamic text' table
Pulling a FS for offline use (we will need to discuss how we want that flow to happen)
Spinning up local EGDB (see above comment)
Filtering data
Spin up maps/templates

I think for a stub, I'm going to implement the dynamic text. Let's do continue to discuss follow-on actions.


Remittance reference or text row missing

Thank you for the QR code generator! It is working well for me except for one small bug.
As you can see in the examples in the standard document you have linked to in README.md both AT-05 (Remittance text and reference) should always be included even if only one of them will have content.
https://github.com/smhg/sepa-qr-php/blob/116ba2a8f4c21c024e0eb6d2e02361c51308aeda/src/SepaQr/SepaQr.php#L217
There seems to exist a fork that has this issue fixed.
https://github.com/makuser/sepa-qr-php/blob/master/src/SepaQr/SepaQr.php#L208
I have tested with Nordeas mobile app to verify that both lines should always be included.

Please correct me if I'm wrong, but I read page 2 of the standard as it can only be either of them? (see the {Or - Or} notation)
The examples show all possible (empty) fields. They don't represent the actual content well I guess?
There are other optional fields that the Nordeas app works without, right? If so, this sounds like a mistake on their end if they require both remittance fields?

I think the problem is that there needs to be an empty line for the one that is not used so that the app can know which one is used and the value from setInformation() which has last position can be read correctly.
When i scanned the code with Nordeas app the content i set with setInformation() was shown as AT-05 remittance information. This also indicates that there needs to be correct number of lines.
Here are two links that show sample payload and they both have an empty line for the one that is not used:

finanssiala.fi Finnish language but you can see the sample payload
stuzza.at Generator that shows the payload

This is fixed and released as 2.2.0. This part of the spec is now also correctly handled:

The last populated element is not followed by any character or element separator.

Previously there were trailing newline characters.

Great! Thank you for fixing it quickly!


Quality of life changes

Reordered tabs a bit, prioritising those that I feel are going the most commonly used ones by devs:

Topics, when in a Cluster
Messages, when in a Topic

For tabs that show a countable information I added a label with the count. This way you can enter a topic and see how many partitions it has without having to click the tab
The cluster switcher takes you to a different page depending on where you interacted with it:

if inside one of the Cluster pages (Topics, Brokers, etc) it takes you to the same page for the new cluster
if inside any other section, it will take you to the parent page. Eg. if you are browsing a specific topic, switching cluster will take you to the Topics page for that cluster

:+1: to the improved UX.


replaced default HFSS Solution type to Terminal for version >= 2024.1

@maxcapodi78 In the title of the PR, you mention that the default Solution type should be Terminal.
But in the code it seems it is Modal.
It is just to be sure that it is expected.

Codecov Report
Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 76.46%. Comparing base (7680ef4) to head (47d3267).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4484      +/-   ##
==========================================
- Coverage   81.43%   76.46%   -4.98%
==========================================
  Files         110      110
  Lines       53482    53484       +2
==========================================
- Hits        43553    40894    -2659
- Misses       9929    12590    +2661

@MaxJPRey You can see in desing_types that @maxcapodi78 changed the default HFSS design, the main reason is because from AEDT 2023R2, the default is Terminal.

@MaxJPRey You can see in desing_types that @maxcapodi78 changed the default HFSS design, the main reason is because from AEDT 2023R2, the default is Terminal.

Got it. Thanks @Samuelopez-ansys !


File adf-producer.py Missing from Getting Started with Snowflake and Amazon Data Firehose (ADF)

Describe the bug
The File adf-producer.py is not available.
URL of where you see the bug
https://quickstarts.snowflake.com/guide/getting_started_with_snowflake_and_aws_kdf/#4
To Reproduce
Follow the quickstart directions.
Expected behavior
The "python3 /tmp/adf-producer.py " command should execute.  It does not as there is no adf-producer.py file.
Screenshots
n/a
Desktop (please complete the following information):
n/a
Smartphone (please complete the following information):
n/a
Additional context
n/a

In case some else needs it:
import pytz,boto3,time,requests,sys,re,json
from datetime import datetime
from botocore.config import Config

global tbl,maxNumRecords
maxNumRecords=250

def dumpToKDF(rec):
 for j in range(0,len(rec),maxNumRecords):
    records=rec[j:j+maxNumRecords]
    numRec=str(len(records))
    try:
      for item in records:
        print(json.dumps(item))
        result = kdfClient.put_record(DeliveryStreamName=deliveryStreamName,Record={'Data': json.dumps(item)})
    except Exception as err:
            print("Error:", err)

#Main
if len(sys.argv) != 2:
   print("Usage: python3 ",sys.argv[0], " <delivery stream name>")
   sys.exit()

deliveryStreamName=sys.argv[1]
headers = {'Content-type': 'application/json'}
url = "http://<IP_ADDRESS>/latest/meta-data/placement/availability-zone"
region = requests.get(url).text[:-1]

kdf = boto3.Session()

kdfClient = kdf.client('firehose', region_name=region, config=Config(read_timeout=20, max_pool_connections=5000, retries={'max_attempts': 10}))

url='http://ecs-alb-1504531980.us-west-2.elb.amazonaws.com:8502/opensky'

kdfflight=[]

while True:
  try:
    results = requests.get(url, headers=headers)
    r=json.loads(results.text)
    print('Headers:')
  except:
    print(results.headers)
    pass

  ts=int(time.time())

  print(datetime.now(pytz.timezone('America/Los_Angeles')).strftime("%D %H:%M:%S"))

  for st in r:
       (icao,id,utc,lon,lat,alt,dest,orig)=(st['icao'],st['id'],st['utc'],st['lon'],st['lat'],st['alt'],st['dest'],st['orig'])
       if dest=='KSFO' or dest=='KSJC' or dest=='KOAK' or dest=='KPAO' or dest=='KSQL' or dest=='KRHV' or dest=='KNUQ' or dest=='KHWD' or dest=='KLVK' or dest=='KHAF':
         kdfRecord={}
         kdfRecord['utc']=str(utc)
         kdfRecord['icao']=icao
         kdfRecord['id']=id
         kdfRecord['lat']=str(lat)
         kdfRecord['lon']=str(lon)
         kdfRecord['alt']=str(int(alt))
         kdfRecord['dest']=dest
         kdfRecord['orig']=orig
         kdfflight.append(kdfRecord)

  dumpToKDF(kdfflight)
  kdfflight=[]

  print('-----------')
  print('Sleeping for 20 seconds')
  print('-----------')
  time.sleep(20)


Helm-Chart: make the MySQL-Secret optional

Describe the bug
The seaweedfs helm chart uses a secret to deploy the filer database credentials for MySQL. We don't Redis in our setup, so we don't really need those credentials. We would prefer not to have them, as the defaults are hardcoded. Making them optional would allow us to remove them, without breaking the deployment.
System Setup

Seaweedfs deployed via Helm
Helm 3 on Kubernetes 1.27
Helm chart 3.67
Redis as filer persistence (or any non-mysql datastore)

Expected behavior
It is possible to remove the database secert without breaking the deployment.

I didn't want to put this into the description in order not to sidetrack the ticket - but is there a specific reason to include the MySQL-Config in the chart, although it is not enabled? It would still be possible to configure MySQL through the extraEnvironmentVars / secretExtraEnvironmentVars (like all other persistence options).


Documentation build fails on Travis CI

Hi,
I am trying to switch our documentation build from normal sphinx to sphinx-multiversion. Locally, I can run everything just fine, but the deployment using Travis fails. This is the output of my job on Travis:
https://travis-ci.org/github/Happy-Algorithms-League/hal-cgp/jobs/743167629
I am trying to build the documentation for the master branch and for one release tag (0.2.0) but it fails to find both, if I understand correctly (error message saying:
Failed load config for refs/tags/0.2.0 from /tmp/tmpwju7f358/f6143c5620f203c0d60363154278b294b603734b/docs
Failed load config for refs/heads/master from /tmp/tmpwju7f358/f6143c5620f203c0d60363154278b294b603734b/docs
No matching refs found!```).

I thought that I might need to fetch all tags explicitly, that's why I added a git fetch --all --tags to the .travis.yml but that didn't help.
This is the complete travis config file: https://github.com/Happy-Algorithms-League/hal-cgp/blob/sphinx_doc/multi_version/.travis.yml
and the actual documentation build is done in a deploy.sh script, which is:
make -C docs/ clean
make -C docs/ html
touch docs/_build/html/.nojekyll

where my Makefile then executes (inside the docs/ folder):
sphinx-multiversion  -D reset_argv=1 . _build/html

This is using the latest release (0.2.4).

I ran into a similar issue on GitLab CI.
It turns out only remote branches were available (checked by adding git for-each-ref --format '%(objectname)    %(refname)      %(creatordate:iso)' refs to my ci script), and smv_remote_whitelist was not set in my configuration, disallowing all remote branches by default.
After setting smv_remote_whitelist = r'^(origin)$', the documentation is deployed as expected.
Hopefully that might help!
As an aside, @Holzhaus: is there a way to enable debug logging through a command-line switch? This would have helped narrow down the issue.

@opcon not the author, but having looked at the code it doesn't appear so. If you're willing to put a PR together it would be trivial to add an argparse option and proxy it to sphinx-build, which accepts a -P flag for debugging.

So, the underlying issue was a missing whitelist entry?


V2 - When the ICLA is already signed, Contributor is able to sign again by clicking link in the email notification of failed CLA checks

Not sure if it is a valid scenario but logging the ticket for tracking purpose
Prerequisite

Create a CLA group with only ICLA enabled
Add github repository for the CLA group
Generate template

Steps to Follow

Login to Github as individual contributor
Navigate to the repository used in prerequisite
Create PR
CLA check fails
Click on CLA NOT SIGNED
Contributor is redirected to Contributor Console
Click on Proceed as a Individual Contributor
Click on SIGN CLA when the button is enabled
Complete the docusign process
Open the Email notification for failed CLA check

Click on the link to get authorized
Observed the contributor is redirected to contributor console and is able to sign ICLA again

Email link is still active after user has previously signed. Maybe check this before user is allowed to re-sign?

Won't fix. Out of scope of current product requirements/features.


Installation failed on Linux Mint

just trying 'pip install'
Requirement already satisfied (use --upgrade to upgrade): requests>=2.5.1 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 1))
Downloading/unpacking PySide==1.2.4 (from -r requirements.txt (line 2))
  Downloading PySide-1.2.4.tar.gz (9.3MB): 9.3MB downloaded
  Running setup.py (path:/tmp/pip_build_root/PySide/setup.py) egg_info for package PySide
    Removing /tmp/pip_build_root/PySide/pyside_package
    package init file 'pyside_package/PySide/__init__.py' not found (or not a regular file)
    package init file 'pyside_package/pysideuic/__init__.py' not found (or not a regular file)
    warning: no previously-included files matching '**' found under directory 'sources/pyside-examples/examples/hyperui'
    warning: no previously-included files matching '**' found under directory 'sources/pyside-examples/mobility'
    warning: no previously-included files matching '**' found under directory 'sources/shiboken/.git'
    warning: no previously-included files matching '**' found under directory 'sources/pyside/.git'
    warning: no previously-included files matching '**' found under directory 'sources/pyside-tools/.git'
    warning: no previously-included files matching '**' found under directory 'sources/pyside-examples/.git'
    warning: no files found matching '**' under directory 'pyside_package/PySide'
    warning: no files found matching '**' under directory 'pyside_package/PySide/docs'
    warning: no files found matching '**' under directory 'pyside_package/PySide/plugins'
    warning: no files found matching '**' under directory 'pyside_package/PySide/imports'
    warning: no files found matching '**' under directory 'pyside_package/PySide/translations'
    warning: no files found matching '**' under directory 'pyside_package/PySide/include'
    warning: no files found matching '**' under directory 'pyside_package/PySide/typesystems'
    warning: no files found matching '**' under directory 'pyside_package/PySide/examples'
    warning: no files found matching '**' under directory 'pyside_package/pysideuic'
    warning: no files found matching '**' under directory 'pyside_package/pysideuic/Compiler'
    warning: no files found matching '**' under directory 'pyside_package/pysideuic/port_v2'
    warning: no files found matching '**' under directory 'pyside_package/pysideuic/port_v3'
    warning: no files found matching '**' under directory 'pyside_package/pysideuic/widget-plugins'
Installing collected packages: PySide
  Running setup.py install for PySide
    Removing /tmp/pip_build_root/PySide/pyside_package
    Python architecture is 64bit
    qmake: could not exec '/usr/lib/x86_64-linux-gnu/qt4/bin/qmake': No such file or directory
    Failed to query the Qt version with qmake /usr/bin/qmake
    Complete output from command /usr/bin/python -c "import setuptools, tokenize;__file__='/tmp/pip_build_root/PySide/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-O4qI16-record/install-record.txt --single-version-externally-managed --compile:
    Removing /tmp/pip_build_root/PySide/pyside_package
running install
running build
Python architecture is 64bit
qmake: could not exec '/usr/lib/x86_64-linux-gnu/qt4/bin/qmake': No such file or directory
Failed to query the Qt version with qmake /usr/bin/qmake
Cleaning up...
Command /usr/bin/python -c "import setuptools, tokenize;__file__='/tmp/pip_build_root/PySide/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-O4qI16-record/install-record.txt --single-version-externally-managed --compile failed with error code 1 in /tmp/pip_build_root/PySide
Storing debug log for failure in /home/kusayu/.pip/pip.log

uname -a
Linux desctop 3.19.0-32-generic #37~14.04.1-Ubuntu SMP Thu Oct 22 09:41:40 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

Looks like installation of PySide failed because it didn't find qmake. Some people reported problems with installing PySide on Linux.
Try to install PySide using: sudo apt-get install python-pyside


Duplicates in the PatternCom Dataset

In the file named quantity.csv,  there are nine images that appear twice with different attribute values.
For example, wastewaterplant270.jpg is annotated with three and one of attribute Quantity
The names of the nine images are as follows.
wastewaterplant270.jpg wastewaterplant450.jpg storagetank020.jpg wastewaterplant271.jpg wastewaterplant272.jpg wastewaterplant327.jpg storagetank225.jpg storagetank520.jpg wastewaterplant269.jpg

Hello again @jian-rookie,
Thanks for raising this issue. These duplicates are images of high ambiguity, thus it was a design choice to annotate them like this. For example, in most such cases, the whole object is not depicted in the image, thus raising ambiguity. If you spot any other annotation issue, don't hesitate to tell me about it.

Thanks for your quick relpy. I've got it. @billpsomas
When any of the above nine images is used as a query, it will also be used as a target image. However, there are not many of these images, I think it should not have much impact on the evaluation results.


Updated streamers.csv: added FixIt42 and some urls [no ci]

Added FixIt42 incl youtube.
Added youtube links for Garr_7 and InsiderPhD too as they were missing.

Thanks @SofiaEngvall!


i18n(zh-cn): Update local-image-used-wrongly.mdx

Description (required)

fix typo
Related issues & labels (optional)

Closes #
Suggested label:

Quick fix not need approve.


Schema verification of inline !vault results in "unknown tag"

Issue: !vault | results in unknown tag
Encrypting individual variables in an inventory results in schema errors. Relevant docs for ansible vault:
https://docs.ansible.com/ansible/latest/user_guide/vault.html#encrypting-individual-variables-with-ansible-vault
Here's an anonymized snippet from my ansible inventory:
snmp:
  addresses:
    - "<IP_ADDRESS>"
  rocommunity: !vault |
    $ANSIBLE_VAULT;1.1;AES256
<PHONE_NUMBER>6434366235343939393130626666383561336638613963633636393962656231646536
<PHONE_NUMBER>363038303632383464653432336534666336340a333262653237396137623564343630
<PHONE_NUMBER>3031356265653964653238633636393832643062356338666538303431333337653033
<PHONE_NUMBER>393132300a383633376465366130356237633439323639353335383338333465343935
<PHONE_NUMBER>3232663865316237656138373133393461333733313639383733391234

And screenshot from VS Code ansible extension showing issue (https://github.com/ansible-community/vscode-ansible):

Can you try adding !vault to the yaml.customTags setting in vscode like:
 "yaml.customTags": [
    "!vault"
 ]

Can you try adding !vault to the yaml.customTags setting in vscode like:
 "yaml.customTags": [
    "!vault"
 ]

Works like a charm! Thanks for the workaround.

Needed tags are now automatically added : https://github.com/ansible-community/vscode-ansible/pull/104
@ssbarnea do we close this issue ?

Fixed in 0.4.3 which adds these automatically.


feat(icons): removed corners from twitch and medium icons

This is what these two icons look like right now:

This PR is to remove the corners so look more like the others ones that has a background color:

Examples:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Problem while importing plateflex

After successful installation of PlateFlex when I tried importing plateflex by command import plateflex, it showed an error saying arviz has no module geweke. After searching on google related to this issue, I got to know that downgrading arviz version may be a solution but had no luck with that. A new error came saying "no module name plateflex.flex". At this point I am confused what to do.
I am currently using:
Ubuntu 18.04.6 LTS
python: 3.8.15
gcc: 10.0.4
For your reference, I have attached screenshots

Hi - It's unclear to me where this error is coming from. What is the version of PyMC3 installed in your environment? You should have pymc3==3.10 for PlateFlex to work properly. Can you show the result of conda list?

Hi - Thank you for your quick response. Please have a look at the attached file. As per instruction given I have installed pymc3==3.10. Though it was showing the same error.
Conda list.txt

Sir, the given version of the fortran-compliler (1.2.1) is not available. Could you please let me know how should I proceed further.

Sorry, it should be 1.2.0 to match the one I have in my environment. But I think this one is not so critical - you could perhaps avoid specifying the version number and get the latest version.

Sir, as suggested, I changed fortran compiler version to 1.2.0 and I also tried by not specifying any version. In both the situation conda was not able to create environment saying incompatible dependencies (screenshot, Ubuntu file attached). Further, I also tried installing plateflex in macos. In macos, conda was able to create pflex environment and plateflex successfully installed. But while importing plateflex, it also shown error (Macos file). Macos.txt Ubuntu error.txt

HI anirban, I am also getting the same error that you are getting in Macos.
Have you found the solution to it?

Recently, I have been working on another project. It is very difficult
for me to recall those steps. You can go as directed by Dr Pascal and use
the updated version; your problem will definitely be solved.
Thank you
On Wed, Jul 3, 2024 at 10:28 AM 23pikachu @.***> wrote:

Sir, as suggested, I changed fortran compiler version to 1.2.0 and I also
tried by not specifying any version. In both the situation conda was not
able to create environment saying incompatible dependencies (screenshot,
Ubuntu file attached). Further, I also tried installing plateflex in macos.
In macos, conda was able to create pflex environment and plateflex
successfully installed. But while importing plateflex, it also shown error
(Macos file). Macos.txt
https://github.com/paudetseis/PlateFlex/files/10227094/Macos.txt Ubuntu
error.txt
https://github.com/paudetseis/PlateFlex/files/10227097/Ubuntu.error.txt
[image: Screenshot from 2022-12-14 15-38-43]
https://user-images.githubusercontent.com/52559240/207577501-094eb5ba-b5f0-4526-8bb0-9d0077daf622.png
HI anirban, I am also getting the same error that you are getting in Macos.
Have you found the solution to it?
—
Reply to this email directly, view it on GitHub
https://github.com/paudetseis/PlateFlex/issues/10#issuecomment-2205101561,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMQ73CGJ6YBXT7HWDUZUFZTZKOAGDAVCNFSM6AAAAABKIY4ZMKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBVGEYDCNJWGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
With Regards
Anirban Biswas
SRF(Ph.D.)
IIT ISM Dhanbad
8507609073
@.*** @.**>

Ok thanks
I will try it with updated version


boards: silabs: Added support for Counter and Watchdog in efr32xg24_dk2601b

Added the support for counter and watchdog in efr32xg24_dk2601b board. Updated the peripheral_sysrtc.h, peripheral_sysrtc.c and added peripheral_sysrtc_compat.h. The files are taken from gecko_sdk 4.4.0 (https://github.com/SiliconLabs/gecko_sdk). These files are required for counter support in silabs boards.
PR in hal_silabs : https://github.com/zephyrproject-rtos/hal_silabs/pull/62

@fkokosinski, @tgorochowik could you please review this PR?

I've unassigned myself since I'm no longer maintaining SiLabs platorms (see https://github.com/zephyrproject-rtos/zephyr/pull/71219).


Add a way to allow database configurations in migrations

I am using edgedb with migrations, but I also have a usecase where I need to create some more dynamic models in a different module.
In order to do this, I have to run configure current database set allow_bare_ddl := 'AlwaysAllow';
However, there is no clean way to run this on every database without making it a manual step.
Where this matters most is in local developement. It is already part of the development workflow to run edgedb migrate whenever getting a new database. Right now, whenever a dev runs into any errors, they have to manually run configure current database set allow_bare_ddl := 'AlwaysAllow'; in order to run the code that uses DDL.
It would be nice if there was a way to add database configurations in a migration file so that edgedb migrate would automatically set the correct intended configuration.

Nevermind, you can ignore this feature request. It turns out the entire desired workflow won't work. Once I run a single DDL command, migrations are no longer possible, even though my DDL statements are on a different module and wouldn't mess up the things in default module. Looks like how I want to use edgedb isn't going to work.

edgedb error: DDL statements were applied to the database, cannot proceed normally.
Hint: Create a fixup file manually or recreate the database from scratch.

Nevermind, you can ignore this feature request. It turns out the entire desired workflow won't work. Once I run a single DDL command, migrations are no longer possible, even though my DDL statements are on a different module and wouldn't mess up the things in default module. Looks like how I want to use edgedb isn't going to work.
edgedb error: DDL statements were applied to the database, cannot proceed normally.
Hint: Create a fixup file manually or recreate the database from scratch.


The timing of tab rendered (when CSS class active is added to tab pane) is not consistent.

Hi all,
In theory, if we scheduel a $timeout in tabScope.active $watcher, the function schedueled by $timeout should execute after a full $digest cycle, during that full cycle, CSS class should be added to tab pane. (because bootstrap.tabs uses ng-class, which shares the same full cycle)
but see this plunkr, the timing is inconsistent. what causes this? It is possibly a bug.
This matters because in our current work we need to calculate DOM dimensions when tab toggles,
but display:none prevent that calculating.
$timeout() 50ms fixes this, creating some helper directive forcing to add active CSS class in tabScope.active $watcher also fixes this.

Just to fully understand, when you add your 50ms to the timer, are you adding it to the timer in your plunker?
This smells like a race condition to me.  I don't think it will change anything because I think they are the same, but if you want to watch an attribute, you use attrs.$observe.
What happens if you use an ng-if that looks for the class?  Would that be sufficient?

@icfantv yes, adding 50ms to the timer in my plunkr fixes this.
And using attrs.$observe totally fixes this! I don't know why.

So they ARE NOT the same.  I'm still trying to wrap my head around why, and chatted with some people smarter than me.  Essentially, $scope abstracts the $watch away from the DOM whereas attrs is tied directly to the DOM.  I still think it's a race condition in that the $watch is firing too soon - BTW, i was able to reproduce this consistently via the plunker on my iPad.
Some light reading:
See this for a more in-depth discussion, and this link for a better (albeit old) one.

Sorry I rarely used $observe and  misused it.
The problem still exists after using $observe.
Thanks for your references, and I'm glad that this problem could be reproduced.

@icfantv could you please reopen this issue?

Going to close this issue in favor of #4836 - the tab API is buggy, and we need to fix the source of all the issues with a fresh & 100% improved API.


Turn on -amd switch to test downstream dependents.

This PR adds the -amd switch to the build and test jobs. This will trigger them to build and test downstream dependents. The goal here is that if you change something in the repo that someone else depends on, then your PR will validate that you didn't break them (for playback test scenarios).

/azp run java - core - ci

//CC @alzimmermsft @srnagar @weshaggard
@mitchdenny I don't think that the -amd switch enabled for everything is going work. The main problem we're going to have will stem from Jacoco which uses the current versions of everything and that will cause every build/run except the From Source ones. With that being said, based upon the discussion that you, Alan and I had on Tuesday, the only times we're really going to see benefits from the -amd switch are in the From Source runs anyways. What if we only enable the -amd switch only for the From Source test runs? This would have a the desired effect, the version manipulation would allow everything to actually build and because of the -amd switch it should run all of the appropriate tests for that area and downstream dependencies. It would make the From Source runs the long pole in terms of pipeline runs but that's only 1 of the 7 test runs.

/check-enforcer reset

/check-enforcer evaluate

/azp run java - core - ci

Test comment to help with triaging Azure Dev Ops Pipelines / GitHub Integration issue.

Closing this PR down since Jim has got a good approach for this.


Deprecate output names with hyphens, add new outputs with underscores

Added

Add serverless_access_key_id output
Add serverless_secret_access_key output

Deprecated

Deprecate serverless-access-key-id output (but leave in place for now)
Deprecate serverless-secret-access-key output (but leave in place for now)

Whoops... to do this properly, I should have done it off of main, to avoid including all of the other changes already in develop. Oh well, not worth that extra time at this point.


⚠ Jenkins Plugins Artifactory has degraded performance

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

HTTP code: 200
Response time: 1208 ms

Resolved: Jenkins Plugins Artifactory performance has improved in db6182b after 34 minutes.


Expand folders when searching in the folder widgets (Loader + Launcher).

Changelog Description
Expand folders when searching in the folder widgets (Loader + Launcher).
Additional info
I'm not entirely sure whether the change implemented here is 'correct' according to the model/view separation logic we want to have in tools, but it shows the behavior artists in our studio we like more (and is logic that behaved similarly in OpenPype actually).
Without this PR when searching in the folder widget it would not expand any folders:

With this PR it defaults to auto-expanding all:

The benefit to this is that when searching you automatically can see all the results and directly can see whether you found what  you're looking for or should filter more or not.
Testing notes:

Use this branch in dev mode for ayon core
Launcher and loader tool should now auto-expand all folders when typing in the filter field.

Let me know what you think about the code @iLLiCiTiT - pointers are welcome to restructure or move this logic elsewhere.