When using a NONEXISTENT_REGION, get_id of mock_cognitoidentity fails

When using a NONEXISTENT_REGION, get_id of mock_cognitoidentity fails because the request made by botocore does not contain the Authorization header
I have tried this with python>=3.8
Here it follows an example get_id request:
{
'X-Amz-Target': 'AWSCognitoIdentityService.GetId',
 'Content-Type': 'application/x-amz-json-1.1',
 'User-Agent': '... Botocore/1.29.23',
 'amz-sdk-invocation-id': '...f',
 'amz-sdk-request': 'attempt=1',
 'Content-Length': '86'
}

And here where you parse the region (lines 327-328):
https://github.com/spulec/moto/blob/4ec748542f91c727147390685adf641a29a4e9e4/moto/core/responses.py#L317-L334
I would like to help you with a PR, but I don't get why the region depends on Authorization header
Traceback
python3.8/site-packages/botocore/client.py:530: in _api_call
    return self._make_api_call(operation_name, kwargs)
python3.8/site-packages/botocore/client.py:943: in _make_api_call
    http, parsed_response = self._make_request(
python3.8/site-packages/botocore/client.py:966: in _make_request
    return self._endpoint.make_request(operation_model, request_dict)
python3.8/site-packages/botocore/endpoint.py:119: in make_request
    return self._send_request(request_dict, operation_model)
python3.8/site-packages/botocore/endpoint.py:202: in _send_request
    while self._needs_retry(
python3.8/site-packages/botocore/endpoint.py:354: in _needs_retry
    responses = self._event_emitter.emit(
python3.8/site-packages/botocore/hooks.py:405: in emit
    return self._emitter.emit(aliased_event_name, **kwargs)
python3.8/site-packages/botocore/hooks.py:253: in emit
    return self._emit(event_name, kwargs)
python3.8/site-packages/botocore/hooks.py:236: in _emit
    response = handler(**kwargs)
python3.8/site-packages/botocore/retryhandler.py:207: in __call__
    if self._checker(**checker_kwargs):
python3.8/site-packages/botocore/retryhandler.py:284: in __call__
    should_retry = self._should_retry(
python3.8/site-packages/botocore/retryhandler.py:307: in _should_retry
    return self._checker(
python3.8/site-packages/botocore/retryhandler.py:363: in __call__
    checker_response = checker(
python3.8/site-packages/botocore/retryhandler.py:247: in __call__
    return self._check_caught_exception(
python3.8/site-packages/botocore/retryhandler.py:416: in _check_caught_exception
    raise caught_exception
python3.8/site-packages/botocore/endpoint.py:278: in _do_get_response
    responses = self._event_emitter.emit(event_name, request=request)
python3.8/site-packages/botocore/hooks.py:405: in emit
    return self._emitter.emit(aliased_event_name, **kwargs)
python3.8/site-packages/botocore/hooks.py:253: in emit
    return self._emit(event_name, kwargs)
python3.8/site-packages/botocore/hooks.py:236: in _emit
    response = handler(**kwargs)
python3.8/site-packages/moto/core/botocore_stubber.py:59: in __call__
    status, headers, body = response_callback(
python3.8/site-packages/moto/core/responses.py:222: in dispatch
    return cls()._dispatch(*args, **kwargs)
python3.8/site-packages/moto/core/responses.py:364: in _dispatch
    return self.call_action()
python3.8/site-packages/moto/core/responses.py:451: in call_action
    response = method()
python3.8/site-packages/moto/cognitoidentity/responses.py:61: in get_id
    return self.backend.get_id(identity_pool_id=self._get_param("IdentityPoolId"))

Hi @dariocurr, thanks for raising this. If possible we try and read the region from the original URL (e.g. us-east-2.ec2.amazonaws.com), but if that's not available we can fallback and extract the region from the Authorization-header.
There are a few operations in CognitoIDP that do not have the Authorization-header, which means we don't have any way of knowing which region to connect to. For some operations, we tend to just cycle through all regions until we find the correct IdentityPoolId.
Not sure why this fails for get_id(), it would need some investigation what's the root cause here.

Thank you for your response and for this useful tool.
I see your point, but isn't the region always specified on the full_url parameter?
Ex: https://cognito-idp.fake-region.amazonaws.com/ for fake-region

Correct, it is part of the URL. We use a regex to extract it, but that was too specific: it only matched regions in the traditional format [a-z]{2}-[a-z]+-\d{1}
I'll raise a PR shortly to fix this.

Thank you for your support! Glad to help


is it possible on mousewheel function to  turn image instead of zoom? Thanks..

@shejin-abu5 Right now it isn't possible, but i have added it to my future roadmap. Hopefully i can find time to work on it and release it in this month. Will keep you posted. Thanks :)

@shejin-abu5 Test out the latest version i just published and let me know if that works for you. Thanks :)


Simplify navigation code

Description of problem:
Currently we have way to many things that set up and manipulate navigation.  Needs to be simplified in to a simple controller and service.   Currently it is in
navigation.config.js
navigation.provider.js
navcounts.service.js
navigation-controller.js
layout.config.js

@miq-bot assign chalettu

@miq-bot add_label wip

@miq-bot remove_label wip


cant bind to leftshift and leftcontrol

i use app.conf:
this work (accidentally found out):
[blender]
mouse2 = rightshift
mouse1 = rightcontrol
this dont work:
[blender]
mouse2 = leftshift
mouse1 = leftcontrol
why?

Can you clarify what you mean by "doesn't work"? What key combinations are problematic? In general you should be using layer(control) instead of  leftcontrol  to avoid problems with modifier stacking.


python3 support

since, the project has no dependencies, it should be relatively easy to extend support for python3.
something like 2to3 can come in really handy. https://docs.python.org/2/library/2to3.html

Even better than that, we could write it in Python 2 and 3. at this point, I think it should be trivial to fix the minor details that block us from running python 3.
https://wiki.python.org/moin/PortingToPy3k/BilingualQuickRef


eclipse-scala-sdk: 4.4.1 -> 4.7.0

Motivation for this change
Even though it's old, Scala IDE 4.7.0 is newer than the version 4.4.1, which is currently available in nixpkgs, but broken.
This commit fixes the break and updates Scala IDE to its most recent version.
Things done

Unfortunately scala-ide is based on eclipse 4.7 (oxygen).
Thus is requires a java SDK 8 and GTK2.
The java SDK is easily overwritten. It would have been possible to have an override gtk3 = gtk2; as well, but I opted for the

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thanks! Rebased to master in 2355d9d545fe2b6ee8a603913d7ec3f6bafe9bef.


Arc vs Rc

When writing a library with Rc<T>, sometimes the author must choose whether they will

limit use cases for the sake of Rc performance vs Arc performance
allow for more use cases (e.g. multithread programs) by replacing Rc with Arc.

The anchor_client is one such library. I have some multi-threaded system that uses a struct with an anchor_client Client as a field, which internally contains an Rc<Keypair>. This makes the type !Send and !Sync and thus unusable for my multithreaded system.
I suspect that the minor performance cost incurred from going from Rc<Keypair> to Arc<Keypair> is worth allowing using a client in a multi-threaded program. So, I propose we do that. I have my local fork where this is done for my own use case, but would like to see this done upstream.

It could even be behind a feature flag to give users of the library the option between Rc<T> and Arc<T>

With this, should come other things like expanding the type signature to Arc<dyn Signer + 'static + Send + Sync>

So it sounds like you would like the anchor client to start using Arc instead of Rc to enable multithreading, which seems like a pretty reasonable request from a rust client. I haven't looked much into anchor's rust client, so i don't know where or for what it's currently using rc but I'd definitely be open to discussing this in a PR if you have any specific suggestions or changes.

I don't want to introduce anything that requires nightly, but I'll see if I can get something going in stable Rust this weekend.

Just checking that you know Solana compiles with its own custom version of rust. It's only 1.59 for the 1.13 cli but might be up to 1.64 whenever 1.14 becomes the stable version

Cool, check out https://github.com/coral-xyz/anchor/pull/2321

Just checking that you know Solana compiles with its own custom version of rust. It's only 1.59 for the 1.13 cli but might be up to 1.64 whenever 1.14 becomes the stable version

This is not relevant for the client, btw. The clients aren't compiled to bpf.

Added in https://github.com/coral-xyz/anchor/pull/2321


Build NettyFinder API

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
An API for the NettyFinder project
Should return phone number prefix number validity and network provider
Describe alternatives you've considered
Node JS
Additional context
Add any other context or screenshots about the feature request here.

@bytenaija @CITGuru

http://nettyfinder.herokuapp.com/ Thats the api endpoints I created on heroku. Haven't had extra time to push it. Its just basic django json response. No rest api framework. I'll also enable CORS for ajax and fetch support. @BolajiAyodeji

@CITGuru Still open?
Kindly re-visit this issue thread.

Was the api for this, still done? or I can create an issue and create api for this with it's documentation @BolajiAyodeji

Was the api for this, still done? or I can create an issue and create api for this with it's documentation @BolajiAyodeji

Hi @sayoadesokan! Thanks for indicating interest.
I'm working on revamping the entire project, and the API will come with that. Once that is live, I'd be open to further contributions and improvements. Thank you!

I'll close this issue now, too, since it's not needed anymore. Thank you.


Imcompatible with Jquery no conflict

If you add
$.noConflict(); to anywhere and even if you don't add any other library the design Bug. The link that scrool stop working and other stuff

Is this still an issue? I'm using the current version and get:
TypeError: $ is undefined
... new Error("AdminLTE requires jQuery");$.AdminLTE={},$.AdminLTE.options={navbarM...
TypeError: $(...) is null
$("#box-widget").activateBox();

My config:

        <link href="/bundles/sonataadmin/vendor/admin-lte/dist/css/AdminLTE.min.css" type="text/css" rel="stylesheet" />
        <link href="/bundles/sonataadmin/vendor/admin-lte/dist/css/skins/skin-black.min.css" type="text/css" rel="stylesheet" />

Some box widget:

Expandable

      <button class="btn btn-box-tool" data-widget="collapse"></button>

First of all it looks like the noConflict mode has to be set after the app.js:

        <link href="/bundles/sonataadmin/vendor/admin-lte/dist/css/AdminLTE.min.css" type="text/css" rel="stylesheet" />
        <link href="/bundles/sonataadmin/vendor/admin-lte/dist/css/skins/skin-black.min.css" type="text/css" rel="stylesheet" />

Now I get:
TypeError: a.AdminLTE.boxWidget is undefined
...se strict";a.fn.activateBox=function(){a.AdminLTE.boxWidget.activate(this)}}(jQu...

TypeError: $(...) is null
...92,lg:1200}},$(function(){"use strict";$("body").removeClass("hold-transition"),...

Does the boxWidget require an extra js file?

Next I wrapped the widget into the ready event:

Currently I get:
TypeError: $(...) is null
...92,lg:1200}},$(function(){"use strict";$("body").removeClass("hold-transition"),...

I just checked the versions, are they fine?

AdminLTE app.js @version 2.3.0
jQuery v2.1.4
jQuery UI - v1.10.4
Bootstrap v3.3.5

I also downgraded to jQuery 1.12.0:

And adding the option:

Still getting:
TypeError: $(...) is null
...92,lg:1200}},$(function(){"use strict";$("body").removeClass("hold-transition"),...

Will check my config and get back to you. But first what libraries are you using aside of Jquery ? Because so far i see only jquery based code in code. And if you are not using any other library aside of Jquery like prototype js for example, there is no need of jquery no conflict

Thanks for having a look at this @kressly . Indeed I missed the prototype lib:

It is added after this block:
https://github.com/almasaeed2010/AdminLTE/issues/174#issuecomment-202818201
And indeed if I remove these files everything works fine! It is a very old version:
Prototype JavaScript framework, version 1.6.0

What file did you remove and is everything working now ? What file did you remove ?

I removed all the prototype files I added in the last comment.

Ok. So are you satisfied now or you need help ? IF you want to keep prototype files, you will have to add the no conflict mode to each jquery file. If you are satisfied let me know and i will close the discussion. If you are not, i will provide you with more details help after i do that with the new design

Thanks! This should work fine for me as I will no longer be loading the prototype files in the new module.
Guess it makes no sense to do further research what is actually causing the issue with such an old prototype lib?

I have taken my time and downloaded the latest template of Adminlte and i have tried to put it in no conflict but just like you i am facing a couple of challenges: Everythings works perfectly except the fact that the calendar, the visitor and the graph are all broken.
I am using the latest version of prototype js prototype_1_7_3 and the latest version of Jquery. So i think the problem is NOT with prototype js. I think there is something we are missing somewhere

Ouch! I'm currently working for a different company now. But if you have anything to test with the boxWidget I could recall my code.

Yes please show me code. I am facing challenges with some of the codes in the new version when it comes to no conflict mode

I have realized that the problem has to do with the jquery ui library. In no conflict mode it is not working. So the question here is how to put jquery ui in no conflict mode

I always thought the UI has to be used w/ jQuery itself that's why UI would adapt the noConflict mode.
Maybe this one helps?
https://github.com/twbs/bootstrap/issues/6303#issuecomment-11494897

https://github.com/twbs/bootstrap/issues/6303

Tks a lot . I was able to put bootstrap in noconflict mode. My problem now is how to effectively put app.js in noconflict mode because in no conflict mode when i click on the menu nothing happens, it no longer reduces or increases.

Actually the boxWidget was the only thing I ever tried. The original implementation for other use cases always came from the Sonata Admin bundle.

Oh ok otherwise i just solved also the app.js
Now my last challenge on the first page is the fact that on the chart when i click on donut , Area disapears


scanDevices() in foreground service doesn't work / times out unless android:foregroundServiceType="location" is defined in AndroidManifest.xml

This almost certainly isn't a problem with the library, but I felt it worth mentioning anyways.
I am relatively new to BLE and I have inherited a project that utilizes this library extensively for BLE work. When I took code I knew was working in my Activity and tried utilizing it in a Service and saw it not working, my immediate location to look for answers / support was this Github. I imagine others might be in the same boat (being relatively new to Android BLE and looking to this library as the first entry point into diagnosing issues)
I am hoping that maybe a blurb could be added in the documentation about using this library in a service.

A while back I created this Gist to make it easier to use Services in a reactive application. It might help you: https://gist.github.com/RobLewis/fe7fad0381b761bae2842dc0d2cd8f2c
Also, I had to do some things to keep the Service running, like using a WakeLock, and posting a Foreground Notification.

A while back I created this Gist to make it easier to use Services in a reactive application. It might help you: https://gist.github.com/RobLewis/fe7fad0381b761bae2842dc0d2cd8f2c
Also, I had to do some things to keep the Service running, like using a WakeLock, and posting a Foreground Notification.

Yeah, that is what I ended up getting working was using a foreground service. I don't need the connection to stay alive forever, just long enough to scan, connected, and pull from data from the device. So I used work manager to schedule starting of the foreground service every hour or so to do that task. Once I added the foregroundServieType to the manifest, all was working perfectly then. Just took me a few days to come to that conclusion :)
Thank you again for the gist!

Yeah, that is what I ended up getting working was using a foreground service. I don't need the connection to stay alive forever, just long enough to scan, connected, and pull from data from the device. So I used work manager to schedule starting of the foreground service every hour or so to do that task. Once I added the foregroundServieType to the manifest, all was working perfectly then. Just took me a few days to come to that conclusion :)
Thank you again for the gist!

Added a mention to the background operations Wiki page
Hope that helps! Cheers!

awesome - appreciate it :)


Upgrade to scala-logging Library

We cannot really use pact-jvm due to your usage of the obsolete (dead would probably be the more appropriate description) scalalogging library.
Is it possible to upgrade to scala-logging?

Sure, I'll add in a hyphen for you.

I'm unable to update the project as there is no scala-logging artifact for scala 2.10. There is only one for 2.11.

Hmm, you've got a point here. Well then, I have to find some way around my problems.
But please remember this issue once you abandon Scala 2.10.
Thanks,
Achim.

What problems are you having? As far as I can see, it is just a rename of the repository and a different base package.

Our (gradle) projects all use the new scala-logging library. So far we did not manage to avoid the conflicts resulting.
But perhaps I have to just try harder and dive deeper into gradle's dependency management or so.

It is only 3 classes, so I could inline them and that should not cause a conflict with your project

That sounds like a great idea.
Thanks so much!

Logging in-lined, should not conflict with your project now.

Again, thanks a lot!

version 2.2.6 released with this fix

Please see https://github.com/DiUS/pact-jvm/issues/401


TypeScript assertion function

Hello,
Recently I've been adding an assert utility function in my projects that is simply the Valibot parse function wrapped and types as a TypeScript assertion function.
export function assert<TSchema extends v.GenericSchema>(
    schema: TSchema,
    value: unknown,
    config?: v.Config<v.InferIssue<TSchema>> | undefined,
): asserts value is v.InferInput<TSchema> {
    v.parse(schema, value, config);
}

function main(something: unknown) {
  assert(v.string(), something, { message: 'Can only lower case a string' });
  return something.toLowerCase();
}

console.log(main(4));

(in the playground)
It's pretty useful to do type narrowing with abort and the Valibot error messages. Maybe this method could be used in the Valibot library ?

Sorry for the late reply. I will try to have a look and give you feedback in the next few days.

Great idea! Will implement it!

v1.0.0-beta.1 with the assert method is available


[macOS] Remove sensitive data from a download log

Description
During the process of downloading the provisioner installer, potentially vulnerable data got into the log. There won't be any more.
Related issue:
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

abandon due to https://github.com/actions/runner-images/pull/8121


Support Resource Groups in ARM Template deployment

When using this plugin based on an Azure VM Image, you are able to specify the Resource Group which will be used for the VM. However, the ARM Template based option does not allow you to specify a Resource Group.
Can we get Resource Groups  on the ARM Template deployment method as well?

It would seem I totally missed that part of the UI.


Fix typo in extend schema documentation

Description
There is a typo on the website https://www.graphql-code-generator.com/docs/custom-codegen/extend-schema
Type of change

n/a Bug fix (non-breaking change which fixes an issue)
n/a New feature (non-breaking change which adds functionality)
n/a Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):

Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
n/a I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
n/a My changes generate no new warnings
n/a I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
n/a Any dependent changes have been merged and published in downstream modules

Thank you @ertrzyiks !


Adding mongo line

@nialldonnellyfh :eye: ?

I don't seem to have permissions to assign reviewers (or don't know how).

LGTM, build fail is not related to this change.


Web - Contact method - App is signed out after coming from old dot and entering secondary contact

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.expensify.com.
Log in with a new account (unverified).
Create a new free policy.
After being redirected to new dot, go to Settings > Profile > Contact method.
Request a new magic code and enter the magic code to verify the account.
Enter secondary contact method.

Expected Result:
App is not signed out
Actual Result:
App is signed out automatically
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/3f328e46-7b01-4682-80d8-c7523b332bc7
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

okay let me do this now!

i can't actually get on staging on safari:

i've been unable to repro this one / no proposals have come in, so im going to close this out for now.


build: release candidate

Release Summary
Fixes

Resolve minor Sonar issues (#390)
Addressed minor SonarCloud issues across the codebase.
Readonly properties in icons.tsx (#389)
Ensured icons.tsx handles readonly props correctly.
Prefer nullish coalescing over logical OR (#391)
Updated conditional logic for improved code clarity.
Cron job export errors post-update (#378)
Refactored the cron job handling to resolve errors after system updates.
Linting command for the database package (#392)
Integrated a new linting command in the DB package.
High and medium severity bugs in SonarCloud (AI-637) (#379)
Fixed critical issues flagged in SonarCloud.
Address major issues in SonarCloud (#393)
Resolved significant codebase maintainability issues.
Fix dialog icons rendering issues (#398)
Improved how icons display within dialogs.
Corrected Cloudflare email display on the Help page (#399)
Addressed email rendering issues.
Revert tabindex changes (#404)
Restored previous tabindex behavior to fix regressions.
Improve intentionality in async promise-based question generation (#403)
Enhanced reliability in asynchronous code.
Memoized sidebar context provider values (#408)
Optimized sidebar component state management.
Avoid inline component definitions (#413)
Refactored inline-defined components to enhance readability.
Proper key usage and void returns for forms (#409)
Fixed improper key usage in array mappings and return types for onSubmit.
SonarCloud maintainability fixes:

Issue 1 (#394)
Issue 2 (#395)
Issue 3 (#403)
Issue 4 (#405)
Issue 5 (#414)

Chores

Add Jest code coverage in SonarCloud (#387)
Enabled Jest code coverage metrics for SonarCloud reports.
Check for cyclic dependencies (#396)
Introduced a check to prevent circular dependency issues.
Remove outdated migration scripts (#412)
Cleared redundant scripts from the repository.
Feature flags and fixes for Math recommendation system (AI-6) (#403)
Enhanced the Math rec-sys functionality, added images for exports.

Tests

Stories for supplementary chat messages (#401)
Added storybook tests for additional chat scenarios.
Chat messages and parts testing (#407)
Expanded test coverage for chat functionality.

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


Add support for Prometheus

Maybe more details about Prometheus ? :smiling_face_with_tear:
or better explaining of the point of the issue

@jaoleal Prometheus is an application used for event monitoring and alerting. It records metrics in a time series and makes it easy to export them through an HTTP endpoint. @Davidson-Souza can correct me, but the point of the issue is simply adding Prometheus to the project. We can them observe the metrics and analyze them through Grafana, for example.

Hi @Davidson-Souza , I can work on this. Can you assign me, please?


Support for Docker install for Node-RED

It would be mighty nice to have Node-RED build and run neatly under Docker, and to have the documentation and/or scripts for that included in the distribution (somewhere). There has been discussion on the mailing list.
A few bits that are a useful start:

A Dockerfile from @dceejay at https://gist.github.com/dceejay/9435867 that uses node:0.10.36-slim
A Dockerfile from @crouchingbadger at https://gist.github.com/crouchingbadger/c0a0e3eba4242baebd07 that uses node:0.10.38
A blog post from http://ivyco.blogspot.com/2015/03/docker-awesomeness.html that describes Docker on a Raspberry Pi running Node-RED

Opening up an issue here to deal with that...if it belongs in another repo we can move it.

Also a Dockerfile from @bortels from mid last summer using Ubuntu 12.04 and Node-RED 0.8.1.
https://github.com/bortels/docker-node-red/blob/master/Dockerfile

IIRC, the biggest issue (unsolved) was the web sockets. This was quite some time ago; I should go look at it again, it may have simply been docker being so young... Or just me not quite getting it right.

And of course now that 0.10.6 is out, it would be worthwhile to update the Dockerfiles to use that.

This is certainly on our list to do. What has stopped us has (aside from it not being a top of the list priority) is deciding what the image should look like and how it would be used. Talking it through with @dceejay today, I think we came up with at least 3 different patterns of usage - all valid, but all very different in what it would expect of a node-red docker image.
Short for time right now, but will endeavour to list out the scenarios this evening so we can discuss what approach we want to take.

FYI - we have node-red running under a docker instance, but as part of an embedded app using strongloop. The websockets seem to work just fine.
However, having said that, one of the major goals for us is to make both nodes and flows loadable from a database, so that we are not constrained by having to have "data" in a docker container - if a db is used as the source / target then the nodes & flows can be updated / changed / added without having to do a rebuild for the container. (and no, I don't want to have to -v them ;)  - we want the docker image to be pulled onto any machine, connect to the database and go, not be beholden to having to have certain files in certain places on the filesystem
thanks

@vielmetti - "my" gist pulls from latest stable - so if re-built now it should be v0.10.6...
@jmls  - yes all part of the chat @knolleary and I were having - about how easy/complex we need to make it.

Some current thoughts for comment
Simple
add an on-build style docker file to root of main project
Advantages

easy to build
it can pick up local already installed customisations
quick way to "clone" a working system
works best if Node-RED installed without -g into a simple node-red dir as can then copy in any extra nodes already installed locally , flow files etc. But will work from /usr/lib/node_modules/node-red - just awkward

Disadvantages

works best if... (as above...). This is a tiny bit more complex if you want to also get local flow files into the image (as they are now normally in $HOME/.node-red and Dockerfiles can't just accept $HOME as a scriptable variable)
not pre-built (but is easy to do :-)

Just a Dockerfile
just provide a "full" Dockerfile - i.e. not using the on-build method
Advantages

could document Dockerfile line by line to show options
could add extra npms to the file
fully customisable
show how to add extra commands (eg non- Node-RED) commands
would work as a "gist" that could be pointed at by a docker build command

Disadvantages

more complicated than "simple" if you just want to build something that works, or make a container of your working setup.
not pre-built

Pre-built on Docker Hub
provide a pre-built image on Docker Hub of latest stable version
Advantages

no need to build - just use Docker run node-red/node-red (sort of thing)
Dockerfile is automatically available if someone wants to copy/modify

Disadvantages

would need sensible defaults for ports and volumes (storage for flow files) to be sorted in advance - see below

Volumes - storage of flow files
The storage of flows files in this environment from simple but dangerous to complex but safer, with several levels of head scratching in between... indeed to my mind this is the most complex aspect of this ... how to make it easy for beginners but show folks how to extend it.

Do nothing - flows exist in the container, when you restart they are wiped... Yikes ! Let's not go there.
Simple volume - flows exist outside the container, so you can stop, restart, and even upgrade the container without losing the flows. (I quite like this as a level-0).
BUT if you recreate the container then the link to the external storage gets broken and while the data still exists on the host drive (so you can re-find them ... eventually) it doesn't stay linked easily.
Named volume - flows exist outside the container in a named location on the host machine. Good for some use cases - but is less transportable. (This approach is normally used for test/debug scenarios...) If you're not careful you may share files between machines you may not intend to... if you use the same shared space.
Storage container - by setting up a non-running - but named container you can create a moveable but linkable space that is much
more in the Docker style of doing tings... however you now have to set up two containers and link them, and the documentation starts getting much more complex...
Database container - as previous but using a running database container to store everything - which can then use the database pluggable storage within Node-RED, and could work across networks etc etc... loads of documentation (and indeed testing required to see if it even works :-)

On-build plus meta package
One quite neat way of combining some of these is to use the simple on-build style  Dockerfile - but combine it with "just" a local meta package.json file that contains links to a list of required npms (including node-red of course). So minimal footprint required to build a working image. - but again does some local customisation and building.
Automated builds
If we can get a sensible default build sorted then we can let docker hub automatically build images from github
    https://docs.docker.com/docker-hub/builds/

That is if we want to use the github version - not sure how we only use "stable" for that.

#3 +1 ;)
storage : have the choice of either a storage container or database container. Using docker compose, this can be handled in a single yaml file, with the data container selected by default, but comments to show how to enable a db container
We would certainly want to use a db container.
we've played an awful lot with docker over the past year and it's getting simpler by the day to manage "complex" environments

I think a key part of this is to think about what users want to be able to do with a dockerized version of node-red - rather than just what shape it may be.
The main use case I would like us to satisfy is:

A user wants to try node-red with minimum effort

Having a pre-built image is key to this. By default, keeping it all in the container is okay - restarting the container (stop/start) will not wipe your data. Only if you kill the container so it no longer exists, then yes it gets wiped - but that's what you expect when you kill a container. But that's good - clean up after yourself and not worry about bits of data lying around. As an out-of-the-box option, not config needed, we should do this.
If that image also has a means to point it at some external storage, then that's good too - and it ought to be via volume or named storage container in the first instance. Pointing at a db container adds quite a lot of complexity - the base image would need the storage plugins to be included and some means to pick which one you want. We could get there eventually, but I don't consider that a priority.
Experience tells us that there are lots of ways you can run node-red, and lots of ways you may want to run it in the context of docker. If we try creating a single thing that caters for lots of these uses, it risks not doing any one use case well.
I would rather we didn't jump straight to the complex scenarios - lets get the basics done first.
The on-build/meta package approach is a good one - make it easy for users to create their own customised containers based on that.
But... would like to think through how we could make a pre-built image customisable - in particular, enabling adminAuth.

Don't disagree with most of that :-)... But I do think the basic volume for user data would be "a good thing". 1) it does keep the date somewhere in case of " oops I overwrote the container", albeit slightly hard to find, but recoverable. 2) you can upgrade the underlying image and then restart the container... Means people can keep up with our releases without having to save flows out and over to a new machine image somehow. 3) updating the underlying contains means any other images based on it get updated too, rather than having to update each image individually.

Just to layer another use case on top of all of these. One of my goals eventually is to have Node-RED running inside docker, and then to have those containers provisioned using salt and salt-cloud. The relevant open issue on the other project is https://github.com/saltstack/salt/issues/22322 for reference (there's a bunch of work to be done before this can really work).
One of the things I see about docker as a piece of the Node-RED system is getting some kind of isolation between independent Node-RED development efforts. For example, the old development Mac I ran on had a single Node-RED instance with 8 or 9 separate pages all really doing separate applications all jumbled together into a single image. It would be great for production to peel out a single self-contained page, minimize its dependencies, package up just the essentials of what it's doing, and deploy it to a stable environment.

If it's running inside and nicely isolated then surely it doesn't care what it is running on - so we can leave the Salt side of things to you ;-)
The on-build style mentioned above can pretty much do that... you can get it all running how you want, then clone it into a container... repeat for the other 8 pages you have...

Great discussion. I´d like to add our planned usage scenario:

We have a docker host VMWare image where we want to deploy probably a dozen node-red instances which manage a logical piece each in a complex network environment.
Our flow files are all hosted in individual git repositories for version control and data safety.
Development: The developer checks out and runs a flow for testing and improvements on his local machine in a staging environment. When he has finished he can push the new flow to the docker host (e.g. with deis/dokku) which replaces the current docker instance related to this repository.

For us it seems to make sense to keep a Dockerfile in the flow file repository which takes care of setting up node-red in a container and copying the flow and settings file. We would use node-red containers only as a service with immutable flow files and therefore without the web frontend.

I've been doing an unofficial Dockerised Node-Red for a little while now at https://registry.hub.docker.com/u/cpswan/node-red/
It's just been updated to 0.10.6, and I've vastly simplified the Dockerfile this time around to use the official Node image and the new npm install mechanism for Node-Red.

I've been using this Docker image on my Raspberry Pi since yesterday:
https://registry.hub.docker.com/u/nieleyde/rpi-nodered/
It's from @nieleyde and it seems to work fine, though it doesn't have a Dockerfile that I can find so it's not as useful for learning.

The source for @nieleyde 's rpi-nodered is at https://github.com/nieleyde/rpi-nodered
The slight complication of rebuilding it is that it's running on Arch Linux for ARM and that distro no longer has the 0.10.x node.js available. https://github.com/nieleyde/rpi-nodered/issues/1 is the relevant issue.

Hi,
I've been working on this for a while https://github.com/muka/redzilla
Contains a docker image and a build script and a minimal nodejs based api for further customizations (like multi instances)

+1 Nice

Going back to the question of "volumes for storage of flow files", another part of what I seem to be heading towards is a volume with a set of nodes in it, so that I can extend or experiment with new nodes without building a whole new base configuration. I think that's a small task but I'm not sure yet.

yes - fairly easy, just use the -volume option to mount a directory into the image...
https://docs.docker.com/userguide/dockervolumes/
the simplest way is to just mount a directory from the host (fine if you own the host of course) - though the docs show how to use a data container to do it and then link the containers.

Thanks @dceejay @jmis for help. I managed to put together a simple Docker Compose file that launches Node-RED and mosquitto both on a Raspberry Pi (so that I have a local MQTT broker) based on images from @nieleyde , you can see that here at
http://vielmetti.typepad.com/vacuum/2015/07/bringing-up-node-red-and-mosquitto-under-docker-compose-on-a-raspberry-pi.html
You'd need a slightly different version picking up different images to run on x86, but this got me going and there is an example.

You can also try my version = NodeRed v0.12.4 with a number of built in Nodes, support for storing the flows externally and and example of passing a USB serial device through to the docker.
###https://hub.docker.com/r/spants/nodered/
example commandline:
docker run -d --name nodered -p 1880:1880 --device=/dev/ttyACM0:/dev/ttyACM0  -v /mnt/user/docker/appdata/nodered:/data:rw spants/nodered

serial device ttyACM0 passed through to NodeRed docker
flow stored in /mnt/user/docker/appdata/nodered on the host (linked to /data in the docker image)

Reviewing the comments, feedback and suggestions above, we've now started to publish official Node-RED Docker images. The existing Docker images the community had already published were a great source of inspiration.
Images: https://hub.docker.com/r/nodered/node-red-docker/tags/
Repository: https://github.com/node-red/node-red-docker
We've focused on enabling new users to get started quickly, with sensible defaults and customisation via environment variables. There are three different images, a full-Linux base image, a smaller image using Alpine and a Raspberry Pi compatible version.
The project's README has a comprehensive introduction to using and customising the images.
Feel free to use this issue to provide feedback, feature requests or concerns (...or send a PR 😄).
Future Plans

Node-RED Version Images. We've set up a CI to build the images automatically whenever Node-RED is updated. In the future, we would like to configure this to publish the new image as that version tag.
Node-RED Official Docker Hub Repo. Follow the process to register the Node-RED organisation as an official Docker Hub repository.
Docker Compose Samples. Publish docker-compose.yml files for common patterns to help users spin up Node-RED and connected services e.g. MQTT server.


Add documentation about how to render the log data

I was totally confused for about ten minutes before I figure out from the shell:$ tensorboard --logdir=<logdir name>.
It would be good to add this to your walkthru.
(nice project!)

Thanks! Will do.


vendor/bin/selenium-server-standalone script does not work with parameters in 3.0.1

$ vendor/bin/selenium-server-standalone -Dwebdriver.gecko.driver=/opt/selenium/geckodriver
Exception in thread "main" com.beust.jcommander.ParameterException: Unknown option: -Dwebdriver.gecko.driver=/opt/selenium/geckodriver
	at com.beust.jcommander.JCommander.parseValues(JCommander.java:742)
	at com.beust.jcommander.JCommander.parse(JCommander.java:282)
	at com.beust.jcommander.JCommander.parse(JCommander.java:265)
	at com.beust.jcommander.JCommander.<init>(JCommander.java:210)
	at org.openqa.grid.selenium.GridLauncherV3$1.setConfiguration(GridLauncherV3.java:219)
	at org.openqa.grid.selenium.GridLauncherV3.buildLauncher(GridLauncherV3.java:147)
	at org.openqa.grid.selenium.GridLauncherV3.main(GridLauncherV3.java:73)
$

It appears the options now need to come beforehand, e.g.:
$ /usr/lib/jvm/java-8-openjdk-amd64/jre/bin/java -jar -Dwebdriver.chrome.driver=/opt/selenium/chromedriver vendor/se/selenium-server-standalone/bin/selenium-server-standalone.jar

Updating the script like so:
- exec java -jar "${SELF_PATH}.jar" "$@"
+ exec java -jar "$@" "${SELF_PATH}.jar"

seems to work, but I'm not sure what else might break. I'll open a PR anyway.

Ah, things like -port 4445 break, so maybe not such a good idea. I think the workaround here is to make sure chromedriver are in the path so the options aren't required, not sure how else to solve this nicely without parsing the parameters in the script and splitting out -D params to before the .jar file in the command.

Thank you a lot for finding this out. So maybe then we have to remove support for the bin file or find a better way in handling driver support. @javigomez You have a lot of experience with webdrivers. What do you think?
At the end the bin file is just a proxy and I assume deprecrating it is an option before everyone updates to 3.0.1. @asgrim How come you did update so fast, what was your version inside composer?

@asgrim What geckodriver version are you using? I'd like to setup a test case in 'tests/SE/SeleniumServerStandalone/Test/ExecutableTest.php'.

@sveneisenshmidt I installed from Fresh, just starting a new Behat suite with it, so I didn't have unbounded constraints ;)
The geckodriver I'm using is the latest from https://github.com/mozilla/geckodriver/releases - which is currently 0.11.1. It's platform dependent, but I grabbed the linux64 flavour :)

@asgrim I won't have any time to upgrade the bin file in the near future to provide a fix so I rather choose to remove the bash bin file for  >=3.0.1. Any objections from user perspective?

I don't think it's necessary to remove it. Just consider it a "known bug" that whatever web driver user requires must be in PATH when using the script. I'd prefer to keep the script and use that workaround personally :)

Also, that'd be a BC break, if you're following semver, although I appreciate the versions just follow selenium versions anyways :)

@asgrim If we are introducing a new major version we can have a breaking change right?

Sure; but you'd lose the parity between this package and the real selenium-server-standalone version, which I'd argue is one of the more useful aspects.

@asgrim Maybe I miss understood you. I think we keep parity between version numbers but only introduce a BC break in our additional code. Although I am fine with introducing the issue as a known bug and move forward.
PS: I pulled v3.0.1 stable of this package and did a pre release with v3.0.1-beta1 until a decision on this issue is found. I want to strongly gather opinions before making a decision.

I think that's a little extreme; once a release is made, it shouldn't be removed really :s clients may already depend on that version (and indeed, our application does)...
As for the issue; as I said, it's likely just a known issue; it's not possible to pass the -Dwebdriver.whatever.driver options, but the workaround (ensuring the webdriver is in PATH) is perfectly acceptable. It's almost a non-issue really (it's less convenient to have to provide the path to the webdriver every time anyway).

@asgrim I've documented the new behaviour, any objections? PR #19

Closed with #19. @asgrim Big thanks to you. New release is under way.

Thanks @sveneisenschmidt :+1:

@asgrim Are you fancy becoming a collaborator?

@asgrim Done. With your next potential PR/update please add yourself as an author to the composer.json file. Thank you for being part of this package.

Thanks! :)

Adding a check if firefox or chrome is in $path, would be very helpfull.
Sample if none of this two are found :
<alert>No driver found, consider adding firefox, chrome or IE to your $PATH variable environnement.</alert>

(selenium start)

Sample if firefox found
<info>Only firefox would be avaible as driver, if you want have more of more driver, add those to your $PATH variable environnement.</info>

(selenium start)

I can make a PR if it sound interessting to you

ping @sveneisenschmidt

Hello @fezfez,
I am pn vacation this week. I like your idea. Let us have a chat about it when I am back.

@fezfez fwiw, this seems like a reasonable approach imo

@fezfez Are you fancy working on this? I think we think it is a good idea to have this. Especially because of issue #17.

@sveneisenschmidt : I start :)

@fezfez What about having a nice checklist?
Webdriver status:
-----------------

✓ chrome (found in PATH)
×  gecko
×  ...
×  ...

For a full list plase see https://github.com/joomla-projects/selenium-server-standalone/tree/master/bin/webdrivers. Some months ago I had contact with the maintainers and I porposed merging both projects so your feature idea would be a very good enhancement.

Good idea, i look into

I found a way more efficient strategy but i need more time than expected.
I keep you up to date


Difficult to find users to delete from group in large groups

References

Fixes DSpace/dspace-angular#2869
Requires Angular PR https://github.com/DSpace/dspace-angular/pull/3286

Description -

The list of current members on the edit group page is searchable now;

List of changes in this PR -

For searching the current editing group members (includes frontend and backend task) :-
a) - In the Frontend, we have added a search form;
b) - In the Backend, we have added a search method("findIsMemberOf") that receives current editing group id and query parameter which
searches in the current member's list of the editing group;

Steps to reproduce:-
Task 1- Search option in the group edit page (in current epeople members section)

Login as admin user
Go to Access Control option in the side pannel
Then go to Groups section in the Access Control
Then choose a group to edit and enter into the edit group screen
In the current members of EPeople section, now you will get a search option to search a member

Rebasing with DSpace Main branch.


[Task]: Add createdAt and updatedAt to sample postgres users table available on prod

Is there an existing issue for this?

[X] I have searched the existing issues

SubTasks
Having that column will help us sort entries in select query and see latest updates at the top.

@abm17 which template is this for? We have a new user table on the new mock_db_v2 that has the updated and created fields

@jacquesikot
Attaching a loom video of where createdAt and updatedAt isn't showing up for users postgres table in mock database : https://www.loom.com/share/c8d229b96aff4035bd737cb29709f28b

Created a PR for this: https://github.com/appsmithorg/cron/pull/35


Add switch to run impersonated with admin permissions?

This module would be of great help in scenarios where you have to install packages from Intune MDM that requires admin permissions but that can't run as NT AUTHORITY\SYSTEM.
Currently, RunAsUser seems to run without admin permissions. Is this something that can be added with a optional boolean?
I tested with following code, running as SYSTEM.
# Test command
Invoke-AsCurrentUser -ScriptBlock {
    # Assets
    $FilePath = [string]('{0}\whoami-{1}.txt' -f ([System.Environment]::GetFolderPath('Desktop'),[datetime]::Now.ToString('yyyyMMdd-HHmmss')))
    $OutText  = [string]('UPN:   {0}{1}' -f ((whoami /upn),[System.Environment]::NewLine))
    $OutText += [string]('UN:    {0}{1}' -f ((whoami),[System.Environment]::NewLine))
    $OutText += [string]('Admin: {0}{1}' -f (
        [string]([System.Security.Principal.WindowsPrincipal][System.Security.Principal.WindowsIdentity]::GetCurrent()).IsInRole([System.Security.Principal.WindowsBuiltInRole]::Administrator),
        [System.Environment]::NewLine
    ))

    # Output file
    $null = Out-File -FilePath $FilePath -Encoding 'utf8' -InputObject $OutText
}

Seems possible and like a good plan. Marked as enhancement and will see about adding soon. Thanks! :)

Implemented in a local test version but seems to behave erratic. Works when UAC is completely disabled, otherwise it works 30% of the time. Still in research :)

Can't get it to work consistently. I think I'd need a C# and UAC expert for this. Adding the 'help required' tag. :)


fixed issue#81 clean list comprehensions

i am not really sure of where to put the function , how many list comprehensions are there so i am just fixing this one , keep merging committing into this as i found them further or you could just give me the list

@4the4ryushin Succint!


Sync the current kube context in the kubeconfig file while creating the context for TAE and Kubernetes-cluster-endpoint

Describe the feature request
The current kubecontext in the kubeconfig file should be updated/synced to the context resulted while creating the context for TAE(Tanzu Application Engine) and Kubernetes-cluster-endpoint(TKG endpoint supporting pinniped auth)
If user wish to not update/sync the kubecontext whenever CLI changes the currentContext in CLI config file(eg: context use, context create command) , they should be able to skip it by setting the environment variable TANZU_CLI_SKIP_UPDATE_KUBECONFIG_ON_CONTEXT_USE to true
Describe alternative(s) you've considered
Additional context

Addressed as part of PR #514


Fatal error 4: Illegal instruction when installing emacs via puppet-emacs

If I run boxen and include emacs, the installation process works, but in running emacs, I get an error:
Geoffs-MacBook-Pro:repo goxberry$ emacs
Fatal error 4: Illegal instruction

If I manually uninstall emacs and reinstall it, it seems to work.
Geoffs-MacBook-Pro:repo goxberry$ brew uninstall emacs
Uninstalling /opt/boxen/homebrew/Cellar/emacs/24.3-boxen1...
Geoffs-MacBook-Pro:repo goxberry$ brew install emacs
==> Downloading http://ftpmirror.gnu.org/emacs/emacs-24.3.tar.gz
######################################################################## 100.0%
==> ./configure --prefix=/opt/boxen/homebrew/Cellar/emacs/24.3 --without-dbus -
==> make
==> make install
==> Summary
/opt/boxen/homebrew/Cellar/emacs/24.3: 3843 files, 96M, built in 4.1 minutes
Geoffs-MacBook-Pro:repo goxberry$ emacs --version
GNU Emacs 24.3.1
Copyright (C) 2013 Free Software Foundation, Inc.
GNU Emacs comes with ABSOLUTELY NO WARRANTY.
You may redistribute copies of Emacs
under the terms of the GNU General Public License.
For more information about these matters, see the file named COPYING.

I suspect this error is related to https://github.com/boxen/puppet-tmux/issues/1, https://github.com/boxen/puppet-homebrew/issues/12, and https://github.com/boxen/puppet-homebrew/issues/17.

This module has been deprecated in favour of using the in built Homebrew/Caskroom support. Please give that a try and let me know if you're still having issues.


Retrieve ABI from sourcify / metadata

Since you, @krzkaczor , mentioned that you would like to retrieve the ABI directly from the metadata as well instead of from etherscan, I just want to quickly explain the steps. We are working on an npm package to maybe do part of this, but it's really not complicated (pseudocode):
code = eth.getCodeAt(addr).hexToBytes()
metadataSize = code[-2] * 0x100 + code[-1]
metadataCBOR = cbor.decode(code[-metadataSize - 2:-2])
ipfsLink = metadataCBOR['ipfs']
// the link is binary-encoded, you might have to base58-encode it, depending on the
// way you access ipfs
metadata = ipfs.retrieve(ipfsLink)
abi = metadata['output']['abi']

This is the recommended decentralized way to do it. You can either use ipfs "directly" or through an http gateway. My experience with http gateways (at least the cloudfront one) is that they will also pin data that is retrieved through them, which is nice.
The access via ipfs only works if contracts are fully verified on sourcify (including the correct metadata hash).  This guarantees that neither the source nor the ABI have been modified after deployment.
There is a centralized http interface to sourcify that also allows you to access partially verified contract's abis: https://github.com/ethereum/sourcify/blob/master/docs/api/server/getByChainAndAddress.md
If you have further questions, you can also come to our gitter / matrix channel: https://gitter.im/ethereum/source-verify

@chriseth thanks! This is a huge help. I'll make sure to prioritize this issue.

I'm not in Lisbon but @chriseth is there so you might bump into each other. I think it's more or less clear and seems to be quite parallel with what we want to build. I will be letting you know about the progress, thanks!

@hasparus did initial implementation in https://github.com/dethcrypto/eth-sdk/pull/42
@kuzdogan first big piece of feedback is that we need s CJS build instead of ESM. You should support both in your package IMO.

Thanks, these are valuable feedback. I agree that they should both be available. We'll look into it.
Let me know if you have any other remarks, both on the package or Sourcify in general. @hasparus @krzkaczor

Basic support for sourcify got released in: https://github.com/dethcrypto/eth-sdk/releases/tag/%40dethcrypto%2Feth-sdk%400.2.2
Currently, it uses sourcify API (backend), we could migrate to IPFS once npm package is done and exposes a simple interface for us to use.


[text-field] When required is set an asterisk (*) is added to the label even if there is no label

Describe the bug
If required is set for the component it adds an asterisk (*) is added to the label even if there is no label. If you are using an external label, this looks very odd.
To Reproduce
Steps to reproduce the behavior:

<mdc-text-field outlined required></mdc-text-field>
See error

Expected behavior
I would expect it to check if there is a label. If so, append asterisk (*) as per spec. If there is no label, don't append.
Screenshots

(Looks bad when an external label is used without a floating label)
What Angular MDC version are you using?
5.1.1

Closing, looks like to ab MDC issue, will defer to: https://github.com/material-components/material-components-web/issues/1363

Reopening. @tinybigideas Thanks for reporting on this. There was a time I had this working, will review..

Got it.
https://github.com/trimox/angular-mdc-web/blob/master/packages/notched-outline/notched-outline.html#L3


[Bug] Triple default timeout is inconsistent with Dubbo

Pre-check

[X] I am sure that all the content I provide is in English.

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache Dubbo Component
Java SDK (apache/dubbo)
Dubbo Version
3.2.11
Steps to reproduce this issue
Triple default timeout is inconsistent with Dubbo
What you expected to happen
Triple default timeout is the same as Dubbo
Anything else
No response
Are you willing to submit a pull request to fix on your own?

[ ] Yes I am willing to submit a pull request on my own!

Code of Conduct

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

code in dubbo

code in Triple

@guohao @EarthChen PTAL

可以修改，和 Dubbo 行为保持一致

https://github.com/apache/dubbo-go/blob/f04c8d08753765f6028d2a84b01aa00e42175a04/global/consumer_config.go#L24
in dubbo-go the defalut timeout is 3s


Fix bare metal install on OCP 4.10

https://github.com/stolostron/backlog/issues/20792

/cc @jeswanke

/retest-required


OpenSearch datasource integration (vector)

Add a new datasource type "opensearch"
Add support for services:

OpenSearch AWS serverless
OpenSearch standalone

For agents:

"query", "query-vector-db": pass JSON as query
"vector-db-sink": sink record into index
"asset":

opensearch-index (with vector support)

Resolved by #574


type abbreviations not allowed in signature when defining new datatypes

The criterion for well-typedness when defining datatypes is too syntactic
let x = Type
type t : x = | A

The type of t is x, but since this is the result type of a constructor its type should be Type

Pushing a fix to call the normalizer on the type before doing that check.

Thanks @mtzguido that was quick !

...Still too syntactic, the following fails
let f (a:Type) = a -> Type
type t : f unit = | A : t ()

Hopefully re-Fixed by https://github.com/FStarLang/FStar/commit/ab3aaaf88999005603f870d92432a70e4d060b16


COMP: Add export specification to ITKSpatialObjects

Closes #743

Thanks for the review.
MetaImageMaskConverter, SpatialObjectFactory, and ObjectFactory are header-only, so they should not have the export specification. NiftiImageIOFactory looks like it has the export specification.

You are correct, those two other files that I mentioned are header only. Should we come up with a script (used in a test) that verifies the export symbols?

We should have errors on the dashboard if export specifications are not correct. It was not an issue with the ITKSpatialObjects module because ENABLED_SHARED was not set on the module -- it was always being built statically.


Papers found on Google Scholar that have no citations get -2 instead of 0

https://scholar.google.de/scholar?q=Partially Safe Evolution of Software Product Lines
Is found, but has no citations so far. As result, I would expect 0 as the citation count.

Should be fixed. Tested with the example above.

Nice!


Add: ability to create association with labels: false by default

As brought up on Gitter, this PR adds the ability to create an association where model_class != false but the cypher generated still does not specify labels for the target node. Currently, the only way to have an association's cypher not specify labels for the target node by default is to usemodel_class: false, which prevents someone from easily query chaining.
Note: I've commented out one spec I added for the has_one association which is failing. As far as I can tell, the problem is not with the code, and is only a problem with the spec (the exact same test works fine in the console). I think it has to do with how the class in the spec is being stubbed, but I really just guessing. I don't know much about stubbing. See the code for more info
Additional
As I discovered while writing this PR, the ability to remove the target node labels on a case-by-case basis already existed as an association getter option (e.g. person.friends(labels: false)). What this PR does is add the ability to specify default association getter options when you create the association.
In this PR, the only configurable default option is :labels, but this can easily be changed in the future. I also added a section to the docs covering this PR

I've noticed that if model_class: [] then neo4jrb still adds in the node labels as part of a where clause. i.e.
class Person
  has_many :out, :entities, type: :ENTITY, model_class: [:LabelOne, :LabelTwo], labels: false
end

person.entities
# =>
# MATCH (person)-[:ENTITY]->(result)
# WHERE result:`LabelOne` OR result:`LabelTwo`

Note that this seems to be a bug in the existing labels option. Since this PR just allows the labels option to be turned on or off by default for an association, it doesn't fix this.

Codecov Report

Merging #1485 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1485      +/-   ##
==========================================
+ Coverage   96.85%   96.85%   +<.01%
==========================================
  Files         205      205
  Lines       12620    12654      +34
==========================================
+ Hits        12223    12256      +33
- Misses        397      398       +1

Impacted Files
Coverage Δ

spec/e2e/has_one_spec.rb
100% <100%> (ø)
:arrow_up:

spec/e2e/has_many_spec.rb
100% <100%> (ø)
:arrow_up:

lib/neo4j/active_node/has_n/association.rb
95.94% <100%> (ø)
:arrow_up:

lib/neo4j/active_node/has_n.rb
98.73% <100%> (+0.03%)
:arrow_up:

lib/neo4j/shared/attributes.rb
94.52% <0%> (-0.4%)
:arrow_down:

Continue to review full report at Codecov.

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

@cheerfulstoic, assuming the specs pass, this is ready to go


Extend extrude-sketch operation to also allow for "cut" operation

Within SC and other CAD products at Ansys, one can draw a sketch and by default it will perform a extrude operation. However, you can also configure it to perform a "cut" operation (i.e. subtract operation). Let's enable this option as well - should be fairly straightforward.

Reported initially by @agvarghe

I'd expect something like:
comp.extrude_sketch(..., cut=True)

By default, cut should be False, users should always want to extrude by default


Stop downloading tr.rdf constantly

Never heard of "HTTP HEAD"?

this got fixed a long time ago in 724391d699a05a32dc151534540f9651bd9598c2


PyTorchWildlife 1.0.1

Fixed batch processing memory leak.
Adjusted documentation.
Changed default Gradio Image type to PIL Image.

Removed unnecessary import from datasets.py
Adjusted default number of workers in gradio_demo.py


[WIP] 2.1 Sample updates (Round 2)

Addresses #5495
Takes care of topics and samples in the fundamentals/configuration TOC area ...

fundamentals\url-rewriting.md (1 sample)
fundamentals\configuration\options.md (1 sample)
fundamentals\configuration\platform-specific-configuration.md (2 samples)

@Tratcher Ran into a problem on one of these and need a tip.
This worked before ...
.UseKestrel(options =>
{
    options.Listen(IPAddress.Loopback, 443, listenOptions =>
    {
        listenOptions.UseHttps("testCert.pfx", "testPassword");
    });
});

... but it's throwing now ...

An attempt was made to access a socket in a way forbidden by its access permissions

I have the option of dropping this (AddRedirectToHttpsPermanent) example from the sample, but I'd prefer to get it working if possible.

That error means some other server is still using port 443, there's likely nothing wrong with the sample.


followup for #1203: deploy.json fix for src/common/*

hopefully this works to make github-deploy-repo catch all the files from src/common/ (#1203 added a file to src/common/covid_hosp/, and there are a total of 3 in that subdir now)

I am not 100% sure if recursive will do what we think it will do

i think/hope it will?  i cribbed it from some usages at the bottom of the same file, and a quick look at the source seems promising.


[Docs] Remove [, on] argument from add when using name as first argument

Test: AddSpec.js
it("currently skips 'on' object when using method name as first argument", function () {
  $.add("bar", function () {}, {
    array: false
  });

  expect([1,2,3]._.bar).to.exist;
  expect($(".foo")._.bar).to.exist;
  expect($.bar).to.exist;
});

Perhaps this should be fixed though?
https://github.com/LeaVerou/bliss/issues/110 had a solution but was put on hold.
We should at least modify the documentation until it's fixed.

i have this fixed in the pull request #148

@dperrymorrow Ah, my bad!
I only read the test-case name it currently skips 'on' object when using method name as first arg
I can see that you changed the assertion. I will close this!

no worries, thanks for all the tests  you wrote, it made it much easier to do that refactor in #148


String.Split Method (Char[], Int32) - does not respect count

Description
string.Split using the count argument - returns all splits instead of a limited number.
Repro steps
#r "node_modules/fable-core/Fable.Core.dll"
let s = "a b c d"
let splits = s.Split ([|' '|], 2)
printfn "%A" splits

Expected behavior
Note a count of 2 returns an array of 2 elements - one split and the remainder. This differs from js split with count. js split with count returns ["a", "b"] which is not what .net produces.
fsi index.fsx
[|"a"; "b c d"|]

Actual behavior
node out/index.js
["a","b","c","d"]

Related information

Operating system win 7
0.3.24

String.Split Method (Char[], Int32) working as expected in 0.3.26 with fable-core 0.1.10


Bluetooth Remote can lockup Zero

If you use the remote function of the iOS app and turn off Bluetooth,  you need to reboot the Zero.
I think if Bluetooth gets disabled from phone, keys on Zero should still work.
A left + back reset still works, but all other button combinations seem to have no effect.

Can't reproduce. Feel free to reopen if you still have this issue.


Add docker compose

It's hard to set up this project locally because of the folder structure. So I've created docker services to run the project locally with a single command.
After setting env vaiables in client and server folders run docker-compose up --build
Note: Make sure you've Docker Engine installed.

We’ve noticed that your PR is causing issues in our deployment. Currently, we have specific plans for Docker in the future, particularly when scaling this application. As such, we haven’t implemented Docker at this time.
To run the application, after installing all the packages in both the server and client, simply navigate to the client directory with cd client, and then execute npm run dev. This is a straightforward process, and we don’t believe it involves an excessive amount of code.


NoMethodError: undefined method []' for nil:NilClass in proc/scanner/dstring.rb:49:in start_tag'

Running with rails console:
Loading development environment (Rails 4.1.5)
>> def la &block
>>   binding.pry
>>   end
=> :la
>> la { p "asd" }
[1] pry(main)> block.to_source
NoMethodError: undefined method `[]' for nil:NilClass
from /usr/local/var/rbenv/versions/2.1.5/lib/ruby/gems/2.1.0/gems/sourcify-0.5.0/lib/sourcify/proc/scanner/dstring.rb:49:in `start_tag'

Using ruby '2.1.5' and 'rails', '4.1.5'

Hey @joaohornburg , thank you for reporting the issue .. sourcify was written in the pre ruby-2 days. i'm sorry, this won't be fixed in the short to mid term, as no longer have the resource & interest to work on this project.

Are you able to add a notification about this to the top of the project readme? Otherwise more issues like this are bound to be opened.

Agree, lets do it now.

Thank you for adding that. :) I'm sure it will save many a lot of hassle, figuring out what is wrong.


AP and switch renaming

Hello,
I apologize in advance for my question as I'm relatively new to Zabbix.
During a maintenance operation this weekend, I performed the following:

Renamed an access point (due to its relocation).
Renamed a switch (similarly) and changed its IP address.

In Zabbix, I still see the hosts with their old names, and the switch is shown as unreachable, even though it is fully accessible in reality.
I was considering deleting the hosts and allowing the discovery process to recreate them. Is this the right approach?
Thanks for your help.

Hi @Isilorn,
yes, in this case deleting the old one and let the template to automatically recreate them is the right approach.


⚠ Bofrost.at has degraded performance

In db953bb, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2226 ms

Resolved: Bofrost.at performance has improved in 113c2a3 after 34 minutes.


Integration Test - Release - 2019/08/29 21:47:45

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest3 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Add support for wildcards in --focus and allow to specify focus in pipeline.sh

What this PR does / why we need it:
This allows to wildcard the version of an artifact when using --focus.
It also adds an optional argument to pipeline.sh which allows to specify
the focus.
Special notes for your reviewer:
Merge after #65
Release note:

A wildcard can be passed as artifact version in '--focus' to target all versions of an artifact

/test pull-containerdisks-pipeline


Dapr extension services block is not recognized

Describe the bug
I want to specify the metrics port for each service in dapr extension area. But when its included according to the schema it throws an exception.
To Reproduce

Add following lines to the tye.yaml.

extensions:
  - name: dapr
    components-path: ./components/tye
    config: config
    services:
     identityapi:
        metrics-port: 9093

Run command tye run
An Exception is thrown.

Loading Application Details...
Error parsing tye.yaml: (8, 5): Expected scalar value for key: "services".

Further technical details

tye --version output

0.10.0-alpha.21420.1+7cec0655cc474aee8d3c3a3d180678985209d439

Debug output

Loading Application Details...
Error parsing tye.yaml: (8, 7): Expected scalar value for key: "services".

Platform - Windows

@chan4lk I apologize for the confusion; the per-service Dapr configuration properties are only available on the 0.11.x builds, not yet officially released (but can be installed, if desired).  At the moment, we don't have a great way to document the Tye configuration schema on a per-version basis (see #1204).

Closing in favor of #1204.


[Refact](inverted index) refact no need read data opt logic #36969

cherry pick from #36969

run buildall


Hairy CPU profile stalls out UI

Hey Roman! cpupro is badass. :D nice work
I just came across the project today. I'm excited to dig in and learn quite a few things.

But first thing I did was drag in a particularly heavy CPU profile.  The tab gets totally hung (at least for 30sec).
Here's the repro: large-profile.cpuprofile.zip
I figured you'd want these cases. :)
You'll probably hear from me again soon. Cheers!

@paulirish Is this artificial profile? Never saw such thing before 🤯

@paulirish Thank you for your feedback and report. I appreciate it!
Yes, I'm very interested in such cases. This way, neither you, the users, nor I will encounter problems when analyzing any profile :)
I took a look at your profile. It uses parent instead of children for the call tree nodes. I've never seen such a structure in profiles from the V8 CPU profiler (.cpuprofile files), only in the Chromium Performance Profile files (as far as I can tell, CPP may contain children, parent, or both in ProfileChunks depending on the Chromium version, so cpupro supports all the cases). Therefore, cpupro expects children for .cpuprofile files, and the logic to transform parent into children is located only in CPP file processing. It seems that applying this logic to .cpuprofile as well files fixes the problem.
A patch is on the way.

@paulirish The issue has been resolved. Please try your profile now. It takes only 0.4 seconds for me.
I also discovered another issue with your profile. For some reason, the time from the last sample until endTime is significant. I suspect it's the time needed to conclude the profiling session or adjustments from excluding idle samples at the end. Regardless, this time is now excluded from the "Profiling time" used to compute time percentages. I've also updated the hint content to explain this:


Spring Data Pageable with PageableDefault

Referring to my question on stackoverflow
I have been trying to expose a Feign Client for my rest api. It takes Pageable as input and has PageDefaults defined:
    @GetMapping(value = "data", produces = MediaType.APPLICATION_JSON_VALUE)
     	@ApiOperation(value = "Get Data", nickname = "getData")
     	public Page<Data> getData(@PageableDefault(size = 10, page = 0) Pageable page,
     			@RequestParam(value = "search", required = false) String search) {
   		return service.getData(search, page);
     	}

And here is my feign client:
     @RequestMapping(method = RequestMethod.GET, value = "data")
     	public Page<Data> getData(@RequestParam(name = "pageable", required = false) Pageable page,
     			@RequestParam(name = "search", defaultValue = "null", required = false) String search);

Now the problem is regardless of whatever page size and page number I send to Feign Client it always applies PageDefaults (0,10).
When I call the rest service directly it works:
http://localhost:8080/data?size=30&page=6
I am using Spring Boot 2.1.4.RELEASE and Spring Cloud Greenwich.SR1. [Recently a fix was done to support Pageable]((https://github.com/spring-cloud/spring-cloud-openfeign/issues/26#issuecomment-483689346).
However I am not sure if using Pageable as RequestParam with PageDefault is supported yet.

yes, it won't work as @RequestParam as of now, you can pass plain pageable object in your Feign client, however, like so:
@RequestMapping(value = "invoicesPaged", method = RequestMethod.GET, produces = MediaType.APPLICATION_JSON_VALUE)
	ResponseEntity<Page<Invoice>> getInvoicesPaged(
			org.springframework.data.domain.Pageable pageable);

Is this helpful?

Thank you. It works without @RequestParam


build error: Inlined implicit external

descroption
I fork the unocss repo then clone to my pc, but build failed after I have installed all deps.
what is expected
build success
what actually happened
build error

reproduction
https://github.com/zoeyzhao19/unocss
just excute ni and nr build. (I use https://github.com/antfu/ni globally to manage packages)
environment
System:
OS: Windows 10 10.0.22000
CPU: (8) x64 11th Gen Intel(R) Core(TM) i7-11370H @ 3.30GHz
Memory: 9.64 GB / 23.74 GB
Binaries:
Node: 16.10.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.11 - C:\Program Files\nodejs\yarn.CMD
npm: 7.24.0 - C:\Program Files\nodejs\npm.CMD
Browsers:
Edge: Spartan (44.22000.120.0), Chromium (106.0.1370.37)
Internet Explorer: 11.0.22000.120

In my computer run build successful, macOS13.0

@sudongyuer . I think this commit  have solved my problem. But I am still doubt that the code segement  ctx.options.rollup.inlineDependencies || id[0] === '.' || id[0] === '/' || id.match(/src[\\/]/) || id.startsWith(ctx.pkg.name!) in file. /src[\\/]/ seems to be invalid on regex101


Unify definition of astro constants

Describe your use case
Some constants, like MU_... and R2D are defined both in architecture/utilities/astroConstants.h and in utilities/astroFunctions.py. It would be good to define them in the same place.
If a barebones astroConstants.i is added, SWIG will find all the constants and make them available in Python side, so I think that is the best way.
We would need to come up with some transition plan, e.g., have astroFunctions.py set its constant values from Basilisk.architecture.astroConstants marking their use as deprecated, then eventually remove them from astroFunctions.py altogether in favor of direct usage of Basilisk.architecture.astroConstants.
Describe alternatives solutions you've considered
None
Additional context
None

I too have been thinking about this and so I second this suggestion. There may be one or two other utilities like files which whose use could be rationalized/reorganized. Even if the .py files stay around, we might be able to find ways to split things so it's clear for a user which they should use when (e.g. name spaces/packages). I'll provide some suggestions shortly.

@patkenneally Do you have any updates about your thoughts on this?
Otherwise I will just submit the small change I made on our internal fork as a merge request to this project.
One potential issue is how to warn users of astroFunctions constants that the constant is available in astroConstants and will be removed later. https://peps.python.org/pep-0562/ is probably a good.

howdy @sassy-asjp , I'm glad to take a look at your branch to provide feedback, and run this by my BSK researchers.  I agree that we'd like to depreciate old functionality gracefully if at all possible, but certainly in a documented manner. 😄

@schaubh I have submitted the change I mentioned. Sorry for the delay, I said I'd submit it soon a while ago, but we had higher priority things we wanted in upstream like the debugging fix and build system changes.


Update MD file on new branch 2017-18-20_03-11-23

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ButtonWithIcon disabled and loading props

This PR introduces optional disabled and loading props to the ButtonWithIcon component and displays the button state accordingly.
A button with loading state is always disabled too.
Tasks:

[x] The IconButton should receive to addiational input parameters/booleans: loading and disabled.
[x] The Icon button should reflect the Input parameters with animated spinner and disabled appearance, onClick event handler should be disabled when diabled=true
[x] Add unit tests, especially for disabled feature.
[x] The download button should display a animated Spinner when compile state is IN_PROGRESS.
[x] The Download button should be disabled when monaco editor is not ready (that means monacoUri is not set in context state)

@achimcc The small "Jump" you described is coming from the requirement: "The download button should display a animated Spinner when compile state is IN_PROGRESS." in the issue #114 It is instantly in "Result" state, because there is no compilation process running.
I can add the same functionality to the Compile button.
Should it be removed from the "Download" button? Maybe that one should only be disabled, while compilation is running.

Edit: Ah, okay, with the merged main branch I see the subtle movement too. I will fix that.

Exactly, the Text button on the right should not move/jump to the left when the state of the button changes ;)

I cannot reproduce the "jumping" button. It looks like you are missing the latest changes in your screen recording. The compile button would be in loading state too, if they were applied.
This is how it looks for me: https://cloud.deni.one/s/ctDfQ4XooL5RXsX


Awake scheduling like from 8am to 11 pm then in the night sleep.

Description of the new feature / enhancement
I work the entire day and I want awake during working hours. The current options doent allow me to set away from specify time range like 8 am to 11 pm.
Scenario when this would be used?
I work the entire day and I want awake during working hours. The current options doent allow me to set away from specify time range like 8 am to 11 pm.
Supporting information
No response

Duplicate of #27790

/dup #27790

It is a good idea, just already captured by some other users. Check out this list of common issues. If you have more ideas, then you can see if anyone else has already suggested it and +👍 their issue.
https://github.com/microsoft/PowerToys/wiki/Common-Triage-Issue


Document ports that need to be exposed on hosts

We should document the default ports that need to be exposed on host for everything to work correctly (stats, logs, etc). Currently appears to be 9342 - 9346 (see https://github.com/rancherio/python-agent/blob/master/cattle/init.py).
Ports should go here:
https://github.com/rancherio/rancher#register-docker-nodes

FYI @vincent99.


[💡 FEATURE REQUEST]: RPC method to manage process num

Plugin
No response
I have an idea!
I have an idea, listen to me!!
It would be great to have the ability to  manage process num by RPC. Something like
$rpc->call('http.AddProcess');
$rpc->call('http.KillProcess');

$rpc->call('jobs.AddProcess');
$rpc->call('jobs.KillProcess');

dup: https://github.com/roadrunner-server/roadrunner/issues/97


Function after timer completes?

I can't seem to get a function to execute after the timer has completed.
I've tried
swal({
  title: 'Auto close alert!',
  text: 'I will close in 2 seconds.',
  timer:2000,
}).then(function(){alert('done')}).done();

This works if confirming via the button but not when the timer finishes.  Is there a way to do this or is this a bug?
Thanks.

swal({
  title: 'Auto close alert!',
  text: 'I will close in 2 seconds.',
  timer:2000,
}).then(
  function() {
    // Confirm button clicked
  },
  function(dismiss) {
    if (dismiss === 'timer') {
      // handle close by timer
    }
  }).done();


Trying benchmarking ConvNets in PyCaffe2 fails.

Hi there!
I'm currently trying to run a Python script located atcaffe2/python/convnet_benchmarks.py. Unfortunately, it breaks in runtime. My OS is Ubuntu 16.04 this time. It'll nice if you could help me out here.
I built caffe2 framework with no GPU support and now executing the following script from the <caffe2-home>/build directory:
PYTHONPATH=.:$PYTHONPATH PATH=<caffe2-home>/build/caffe2:$PATH python caffe2/python/convnet_benchmarks.py --batch_size 128 --model AlexNet

It fails with the following message:
WARNING:root:This caffe2 python run does not have GPU support. Will run in CPU only mode.
WARNING:root:Debug message: No module named caffe2_pybind11_state_gpu
AlexNet: running forward-backward.
Traceback (most recent call last):
  File "caffe2/python/convnet_benchmarks.py", line 657, in <module>
    Benchmark(model_map[args.model], args)
  File "caffe2/python/convnet_benchmarks.py", line 564, in Benchmark
    workspace.RunNetOnce(model.param_init_net)
  File "/home/kostya_by/git-repos/caffe2/build/caffe2/python/workspace.py", line 151, in RunNetOnce
    return C.run_net_once(StringfyProto(net))
RuntimeError: [enforce fail at operator.cc:106] op. Cannot create operator of type 'XavierFill'. Verify that implementation for the corresponding device exist. It might also happen if the binary is not linked with the operator implementation code. If Python frontend is used it might happen if dyndep.InitOpsLibrary call is missing. Operator def: output: "conv1_w" name: "" type: "XavierFill" arg { name: "shape" ints: 64 ints: 3 ints: 11 ints: 11 } device_option { device_type: 1 cuda_gpu_id: 0 }

A help message in the header of the script suggests to add ../gen path to the PYTHONPATH variable, but I believe this suggestion is a bit outdated as no gen directories are generated during the current framework installation process.
Thanks in advance!

Would you mind elaborate a bit?

@Yangqing Hi! Sorry I mistakenly reported the issue before I even started describing it. The details are added now.

(closing this for now, please feel free to follow up)


Implement expression class

Coverage decreased (-7.4%) to 92.593% when pulling 0cffa1f8367caf433a5e8528c8f60277c7b3f003 on implement-expression-class into 890c81cd5e8db73b884eab03b745aed935575859 on master.

Coverage remained the same at 100.0% when pulling fa578bfe82c72c778cfa85513df2367a85d247d1 on implement-expression-class into 890c81cd5e8db73b884eab03b745aed935575859 on master.

Coverage remained the same at 100.0% when pulling d874fe94a50e339d6abaca502c4dee99a6e1a7dd on implement-expression-class into 890c81cd5e8db73b884eab03b745aed935575859 on master.


Frontend Project "Unit Converter" Box Design Enhancement

Describe the Feature you want to end
Changing few design properties to make it look attractive.
Expected behavior
Attractive box design.
Screenshots (optional)
Current Design

Expected Updated Design

Additional context (optional )
N/A

@MohitGupta121


runtime(jq): fix undefined variable s:save_cpoptions in ftplugin

This PR fixes an issue about undefined variable in the jq ftplugin file. It is recommended not to setting 'cpo' if unnecessary (https://github.com/vim/vim/pull/14619#discussion_r1575249285), I removed the resetting lines instead of adding let s:save_cpooptions = &cpooptions and set cpo&vim.

thanks, see also #14663 I'll merge those two together :)


Moved @nestjs/common to dev and peer dependency

#16

Pull Request Test Coverage Report for Build 27

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.045%

Totals

Change from base Build 25:
0.0%

Covered Lines:
127

Relevant Lines:
129

💛  - Coveralls


Phase 1 Partial - Colorization + share

Partial of phase 1, incorporating colorization into the CMake Tools repository, and adding internal development sharing.

TODO: CHANGELOG update.


Implement Loader for Interim Constitution Page

As a user, I want to see a visual indicator while the Interim Constitution page is loading, so I know that the page is responsive and that the content is being loaded in the background.
Reasoning:
Currently, the page takes a few seconds to load due to the library used for scanning the document and generating the table of contents. Without a visual identifier, users may not know if their click was registered or if the page is unresponsive.
ACCEPTANCE CRITERIA:

Visual Indicator Implementation: Add a loading animation or a similar visual indicator (does not need to be a traditional loader) to notify users that the page is loading.
The visual indicator should be visible immediately after the user clicks to access the Interim Constitution page and remain until the content is fully loaded.
Design Consistency: The visual indicator should align with the overall design language of the platform (e.g., color scheme, style).
Performance Optimization: Ensure that the visual indicator does not impact the performance of the page or increase loading times.
Page to Update: TBD

This has been deployed to both SanchoNet and MainNet.


Cannot see any configuration options

Description of the issue
Hi, I can only see this in my config when I install the add-on:
enabled: false
master: pty,raw,echo=0,link=/tmp/ttyZ2M,mode=777
slave: tcp-listen:8485,keepalive,nodelay,reuseaddr,keepidle=1,keepintvl=1,keepcnt=5
options: "-d -d"
log: false
I can't add my mqtt username etc, any ideas how I can do this?
Addon version
1.29.0-1
Platform
Home Assistant 2022.12.8
Supervisor 2022.12.1
Operating System 9.4
Frontend 20221213.1 - latest
Logs of the issue (if applicable)
No response

First of all, I'm also a newbie trying to install this addon with a CC2652P2 TCP Zigbee Coordinator from cod.m
When you say : Hi, I can only see this in my config when I install the add-on:
Do you mean, in the Configuration tab of the add-on?

At first I didn't have also these entries, so I did go by 'Edit in yaml' at the 3 dots menu, to add them.
I wonder where this configuration is written, because in the specific zigbee2mqtt configuration yaml file, I do not see this configuration, only this:

But now it is failing with this:

Searching further :-)


[BUG]- It is not working even after I reference the CDN in my project

Hello Team,
I have added below CDN to my project but still toggle button is not working as expected.
Please can you help?

Hi can you provide screenshots and code to reproduce the error?


Update topics with grpc-timeout option in header

Fixes Issue #92

Updated several topics with grpc-timeout. I'm also making some improvements to those REST sections. :)


[stable/mysqldump]: Add support fir existing secret for mysql password

Signed-off-by: Marcus Sonestedt<EMAIL_ADDRESS>
What this PR does / why we need it:
Getting the password for mysql root user from another existing secret was not possible. This PR implements that.
Special notes for your reviewer:
I took the liberty to adress some markdown linting issues in README.md. Hope it's ok. 😊
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @axdotl


Add files via upload

this is one example keybinding to add CTRL A to approved macro.

using https://docs.keymapper.club/quick-start/


TextToSpeech Cognitive services No connection to the remote host. Internal error: 1. Error details: Code: 11001.

I am trying to run sample program for TextToSpeech Cognitive services and got error No connection to the remote host. Internal error:
here is code that I am following:
public static async Task SynthesisToSpeakerAsync()
{
// Creates an instance of a speech config with specified subscription key and service region.
// Replace with your own subscription key and service region (e.g., "westus").
// The default language is "en-us".
var config = SpeechConfig.FromSubscription("9604324d07454f21934aaffdb4eb705c", "centralcanada");
        // Creates a speech synthesizer using the default speaker as audio output.
        using (var synthesizer = new SpeechSynthesizer(config))
        {
            // Receive a text from console input and synthesize it to speaker.
            Console.WriteLine("Type some text that you want to speak...");
            Console.Write("> ");
            string text = Console.ReadLine();

            using (var result = await synthesizer.SpeakTextAsync(text))
            {
                if (result.Reason == ResultReason.SynthesizingAudioCompleted)
                {
                    Console.WriteLine($"Speech synthesized to speaker for text [{text}]");
                }
                else if (result.Reason == ResultReason.Canceled)
                {
                    var cancellation = SpeechSynthesisCancellationDetails.FromResult(result);
                    Console.WriteLine($"CANCELED: Reason={cancellation.Reason}");

                    if (cancellation.Reason == CancellationReason.Error)
                    {
                        Console.WriteLine($"CANCELED: ErrorCode={cancellation.ErrorCode}");
                        Console.WriteLine($"CANCELED: ErrorDetails=[{cancellation.ErrorDetails}]");
                        Console.WriteLine($"CANCELED: Did you update the subscription info?");
                    }
                }
            }

            // This is to give some time for the speaker to finish playing back the audio
            Console.WriteLine("Press any key to exit...");
            Console.ReadKey();
        }
    }

    static async Task Main()
    {
        await SynthesisToSpeakerAsync();
    }
}

Thank you for bringing this to our attention.  To better assist you, could you please identify which document this is in reference to?

@MartinJaffer-MSFT Thanks for helping out. here is link that I am following:
https://github.com/Azure-Samples/cognitive-services-speech-sdk/blob/master/samples/csharp/sharedcontent/console/speech_synthesis_samples.cs

@MartinJaffer-MSFT any update on this enquiry ?

I think you are using wrong region name. Please try the region short name as canadacentral, The referenced issue is using the correct short name. You can run the CLI command az account list-locations to get the correct short name list.

@RohitMungi-MSFT No, I don't think so. I had other sample code program with the same region mentioned is running fine.
here is link https://docs.microsoft.com/en-us/azure/cognitive-services/translator/quickstart-translator?tabs=csharp and code for other sample program: namespace textspeechconsole
{
class Program
{
    private static readonly string subscriptionKey = "f3b8513946af4fe88b6ccc5937efdbeb";
    private static readonly string endpoint = "https://api.cognitive.microsofttranslator.com/";

    // Add your location, also known as region. The default is global.
    // This is required if using a Cognitive Services resource.
    private static readonly string location = "canadacentral";

    static async Task Main(string[] args)
    {
        // Input and output languages are defined as parameters.
        string route = "/translate?api-version=3.0&from=en&to=de&to=it";
        string textToTranslate = "Hello";
        object[] body = new object[] { new { Text = textToTranslate } };
        var requestBody = JsonConvert.SerializeObject(body);

        using (var client = new HttpClient())
        using (var request = new HttpRequestMessage())
        {
            // Build the request.
            request.Method = HttpMethod.Post;
            request.RequestUri = new Uri(endpoint + route);
            request.Content = new StringContent(requestBody, Encoding.UTF8, "application/json");
            request.Headers.Add("Ocp-Apim-Subscription-Key", subscriptionKey);
            request.Headers.Add("Ocp-Apim-Subscription-Region", location);

            // Send the request and get response.
            HttpResponseMessage response = await client.SendAsync(request).ConfigureAwait(false);
            // Read response as a string.
            string result = await response.Content.ReadAsStringAsync();
            Console.WriteLine(result);
        }
    }
}

}

Yes, Please use canadacentral. The code snippet from the first comment is using centralcanada.
var config = SpeechConfig.FromSubscription("xxxxxx", "centralcanada");
This seems like a config issue rather than a documentation issue. We would request you to please raise an issue on the issues page of SDK repo if the samples are not running as expected. Thanks.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply.


Run HA/NA analysis with separate distance matrices for HA and NA

Now that pathogen-embed supports multiple input distance matrix files and alignments, we should update the HA/NA analysis workflow to build separate distance matrices for HA and NA instead of concatenating the gene sequences into a single file. This updated analysis should produce more accurate embeddings for the distance-based methods, since the pathogen-distances command will be able to strip trailing gaps from HA and leading gaps from NA that were included in the concatenated sequences.

See https://github.com/blab/cartography/pull/122 for details on why this is closed.


Hugo deprecation warnings

Hugo warns not to use certain language constructs anymore:
WARN 2019/05/30 01:17:21 Page's .Hugo is deprecated and will be removed in a future release. Use the global hugo function.
WARN 2019/05/30 01:17:21 Page's .URL is deprecated and will be removed in a future release. Use .Permalink or .RelPermalink. If what you want is the front matter URL value, use .Params.url.
WARN 2019/05/30 01:17:21 Page's .UniqueID is deprecated and will be removed in a future release. Use .File.UniqueID.
Total in 13 ms

A suitable patch could look similar to

https://lab.allmende.io/ecobytes/commons/hugo-theme-ecolearn/compare/251f206a7dae4cfdcf21bc246fdc03a97a634544...36d70302e81b17bd17b1bc7ac2285cb6e2464e07

We now get
WARN 2019/05/30 02:11:39 .File.UniqueID on zero object. Wrap it in if or with: {{ with .File }}{{ .UniqueID }}{{ end }}

from the menu.html partial. Unfortunately my go and go templating skills are not enough to construct this into the one-liner where it's needed.

Me too


Bring back tilted attribtue

Fixes #329

Coverage decreased (-0.03%) to 88.924% when pulling fe0228ddb40e1ecc29d962c6dd2c8ac10816c01d on bring_back_tilted_attribtue into b4457c3c18d298161e5882ef19c8c5aaf2da1e99 on dev.