[Table] Add a TableFooter for pagination

This PR adds a new TableFooter component and extends the EnhancedTable demo to demonstrate pagination. It also adds the missing Android versions to the nutrition list. :cake: :lollipop:
Specs:

This PR:

[x] Add a TableFooter, similar to TableHead
[x] Add pixel-perfect pagination footer for tables
[x] Add tests for TableFooter
[x] Move the pagination code into a TablePagination component as suggested in https://github.com/callemall/material-ui/issues/7746#issue-249846046 and immediately add tests this time

I'll continue working on this tomorrow, it's pretty ~late~ early in the morning here in germany. :sweat_smile:

Move the pagination code into a TablePagination

@leMaik What do you have in mind for this abstraction, for instance. How do you want to design the API?

@oliviertassinari Basically, I'd just move the EnhancedTableFooter and rename and refactor it a bit. I'll also add a way to customize the strings (for i18n). I really care about the design and layout being as shown in the specs, so I'd rather not make this too flexible for now.

@leMaik Sounds good. I have found that having placeholder is what provides the maximum correctness and flexibility.

@oliviertassinari Fixed the things you mentioned in the review, now working on tests and refactoring. :gear:
Regarding flexibility: After all, the entire footer can be swapped out, this component just adds one way of doing pagination. It can be seen as a pretty big placeholder already.

Just found another major issue thanks to argos: I completely broke the sorting. :sweat_smile:

@oliviertassinari I'm done with this so far, would you mind taking another look at the PR? CI doesn't want to run because I increased the build size, but obviously adding a component increases it. :sweat_smile:

Codecov Report

Merging #8254 into v1-beta will decrease coverage by 0.08%.
The diff coverage is 87.5%.

@@             Coverage Diff             @@
##           v1-beta    #8254      +/-   ##
===========================================
- Coverage    99.42%   99.33%   -0.09%
===========================================
  Files          147      149       +2
  Lines         2242     2258      +16
===========================================
+ Hits          2229     2243      +14
- Misses          13       15       +2

Impacted Files
Coverage Δ

src/Table/TableCell.js
100% <ø> (ø)
:arrow_up:

src/Table/TableFooter.js
100% <100%> (ø)

src/Table/TablePagination.js
77.77% <77.77%> (ø)

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 7b4b53a...32008a9. Read the comment docs.

I think that we only miss the typescript definition of TablePagination and some test coverage to merge this PR 💃 .

@oliviertassinari Currently working on the test coverage and the bug that argos just discovered (EnhancedTable is not sorted after mounting).
Look at the last item in my TODO list above, there's still some work left but we're getting there. :)

@leMaik Oh ok, feel free to continue the PR then :). If you plan to keep opening great PRs like this one here, I will give you the push right access to the repository. It can make the process easier.

@oliviertassinari Could you manually confirm the changes in argos? The table is now properly sorted, it just looks different because there are new sweets.
I added more tests and typings, so now I'm really actually done as long as you don't have any further suggestions. :tada:

If you plan to keep opening great PRs like this one here, I will give you the push right access to the repository. It can make the process easier.

While I do plan to keep contributing to material-ui, I can't guarantee the interval of the contributions. :sweat_smile:

@leMaik Awesome

I feel honored. :)

You do deserve it. You have been, with your team, did a great job: https://mui.wertarbyte.com/. Yes, no pressure :).


SPF checks

I would like to include the SPF policy agent. I don't mind raising a pull request for it but I want to discuss it in a bit more detail first.
Right now the module uses flags such as use_amavisd. This is easy, relatively straight forward and you don't really have to think about it as a user. The SPF policy agent is a little different to a feature like Amavis. It needs to add a config line into in master.cf and it requires you to manage the smtpd_recipient_restrictions variable at a verify specific point. If the module does this for you it means you lose a lot of flexibility as a user because you can't manage that setting anymore.
It probably makes more sense to allow custom content in master.cf.common.erb or a new file. That would give you the opportunity to make the changes you need.
If you want to use the SPF policy agent you need to install pypolicyd-spf on CentOS or postfix-policyd-spf-python on Ubuntu yourself as it's not installed as part of Postfix. That makes another discussion: should it include this or shouldn't it?
I would make it as flexible as possible and leave out these packages and just allow custom content. What do you think?

Hi,
how i set 'use_amavisd' in this module?

@Tomelin you're hijacking a different issue. If anything is unclear you should open a new issue instead.

@ju5t 	I think I'd welcome a postfix::spf class if you know how to implement it properly in the module.

@raphink I'm afraid my time is a little limited. This wouldn't be on my todo list for the next few months probably. Although I think it can be useful to have a postfix::spf class we have our own implementation for now.

@ju5t 	understood and sorry for the long delay in responding. I'll keep this open in case someone is interested in implementing it.


Addon does not work in installations where drupal is installed as recommended-project

If Drupal is installed using the recommended project composer template, then the core directory is in /var/www/html/web/ rather than /var/www/html/. This breaks the paths in this addon placing phpunit.xml
Would it be possible to attempt to detect the existence of the core directory before attempting to copy/alter files into that directory?

discussed on Slack, summary:
this project isn't intended to support installation in that way, just for direct git clones. If Drupal is installed using the project composer template then something like Drainpipe is more appropriate


replace all references to gh-pages with master

We no longer use GitHub pages directly; we run Jekyll ourselves and push
the built site to another repository. Stop using the gh-pages branch
so that GitHub stops trying to build this repository as a GitHub pages
site (and warning about how the redcarpet gem is no longer supported).
This completes the transition by replacing all references to gh-pages
with master.

@sploiselle @jseldess if this PR successfully builds, we're good to go!

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/1e70036147211830d9a8e9bb09d63595dcb90a02/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/cb67d477b69dfe81b0c8631b7cda3818f778873c/

––you need either of us to do anything?

Review status: 0 of 19 files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from Reviewable

Just git fetch && git checkout master sometime soon!

Woohoo! In it goes!


tez java and view changes

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.CHANDUPATLA seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add 'Submit Service Request' Action

Add in some functionality (button, link, etc.) to submit a service request to the City of Boulder when the phrase 'service request' appears in the chatbot response text.

Pull Request #21 addresses this issue by appending service request information to relevant responses.


HBASE-25471 Move RegionScannerImpl out of HRegion

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 18s
master passed

+1 :green_heart:
checkstyle
1m 16s
master passed

+1 :green_heart:
spotbugs
2m 11s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 50s
the patch passed

-0 :warning:
checkstyle
1m 13s
hbase-server: The patch generated 14 new + 225 unchanged - 16 fixed = 239 total (was 241)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
19m  2s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 21s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 13s
The patch does not generate ASF License warnings.

43m 41s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2853/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2853

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux 62263b91c04c 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a414361ed9

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2853/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
84 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2853/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=3.1.12

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m  6s
master passed

+1 :green_heart:
checkstyle
1m 26s
master passed

+1 :green_heart:
spotbugs
2m 25s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 44s
the patch passed

-0 :warning:
checkstyle
1m 26s
hbase-server: The patch generated 2 new + 224 unchanged - 17 fixed = 226 total (was 241)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
23m 11s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1 3.3.0.

+1 :green_heart:
spotbugs
2m 14s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s
The patch does not generate ASF License warnings.

52m 20s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2853/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/2853

Optional Tests
dupname asflicense spotbugs hadoopcheck hbaseanti checkstyle

uname
Linux c654770d592a 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / a414361ed9

checkstyle
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2853/2/artifact/yetus-general-check/output/diff-checkstyle-hbase-server.txt

Max. process+thread count
94 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-2853/2/console

versions
git=2.17.1 maven=3.6.3 spotbugs=3.1.12

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 52s
Docker mode activated.


when it will support MAUI .net 8

i can't use it in my Maui project because of that any idea when it is supported can I have any release date for .net 8 and if there is any beta or workaround please let me know

@omernaser we will add support for it and release it by EOW.

We are also using it in a .NET 8 Maui App. I can see some analytics coming in, as well as crash reports for Android (but not for iOS, though).
@omernaser what exactly doesn't work for you?
@ndesai-newrelic looking forward to the new version!

@omernaser what errors are you getting?

@omernaser added support for net 8.0 in release 1.1.1.
https://www.nuget.org/packages/NewRelic.MAUI.Plugin/1.1.1


Fix a bug in the organisations_list test helper

Response wasn't being converted to JSON before being used in the stub

Our styleguide says that version bumps happen in a commit and Pull Request of their own: https://github.com/alphagov/styleguides/blob/master/rubygems.md#versioning

Ah, sorry, will know for next time.


Add option to increase the clickable space between the gutter and the first character of the line

This was requested a few years ago in this Issue, but there seems to have been confusion around how the feature was described.
It was actually assigned to a developer but then for some reason un-assigned. I'm not sure exactly what happened there. The issue did garner quite a bit of support as seen from the comments, but since the issue is long buried, I figured I'd try to start fresh and clarify exactly what the request is.
Issue:
The gutter is extremely close to the first character of each line. So if you want to click to place the cursor before the first character (or to begin selecting text from the first character), you often end up missing and clicking the gutter instead. Often this results in accidentally folding parts of the code. Ultimately it just makes placing the cursor there more tedious than it needs to be.
Request
Add an option, something like editor.contentLeftPadding, which allows the user to decide how much space is there. The important bit here is that it should operate such that if you click in that space, it places the cursor at the start of that line.
Some have suggested an aesthetic fix/hack, adding
.monaco-editor .lines-content {
  padding-left: 8px;
}

to "vs/workbench/workbench.desktop.main.css"
This does in fact add some visual space between the text and the gutter. But crucially, you cannot click in that space to place the cursor.
If I have not adequately explained, I pray this meme may succeed where my words have failed me

Thanks!

Thanks so much @dschil138 for leading this charge! You're absolutely right.  Excellent documentation on the needed change.
As a related aside-- it appears the related setting to change the gutter width itself to (e.g. to 0) is:
"editor.lineDecorationsWidth": 0;

https://stackoverflow.com/a/74463128/1762445
https://github.com/Microsoft/vscode/issues/30795

But in the case of what @dschil138  is asking-- It would enhance the ability to click between the first character in a line of code... and the gutter.  Currently that's only a 1 pixel space which makes it very difficult to target.

I've commented on the original issue this references in the past, and I wholly agree this issue needs to be pushed to the top of the queue. It has had a lot of support in the original issue, and therefore we can skip the whole community vote thing.
It's time devs. Time to fix this issue once and for all. It's a simple change, nothing major for a dev, but for loyal users its a significant change and will cut down frustrations and time spent mis-clicking.

So what determines if this gets picked up? What is the process? These is some kind of vote?

yes please

After years with notepad++ decided the move to VSCode (VSCodium). Enjoying the experience so far but the only thing I found difficult was how fiddly it is trying to select to the left of the first character of a line, especially if it's a function that you can collapse. Searched online for what setting to change only to discover these threads and that there is no setting to change! Would be great if this could be implemented.

@ajtompkins yeah, I'm not sure where to go from here. This issue thread got a bunch of support/comments, just like the old one, but then nothing happened. Is there something else we can do? I'm not sure
@alexdima I see you are assigned on here, would you be able to tell us: is this just something where we did not reach a threshold of response on the issue?

It's a constant annoyance, not sure how the authors of this application aren't suffering equally from it

I don't know what's more annoying, the lack of care from the devs or the act of trying to click to the left of the first character on any one of thousands and thousands of lines of code!
SO ANNOYING!!!!

I support this.
I also just wanted to leave my work arounds here so that they might help any who find this annoying.
The only reasons I can think of to want to click at the very first col of a line are:

to select the entire line
to select multiple lines starting at the desired line
to add a new line above the line (basically to move the desired line down)

firstly I set "editor.showFoldingControls": "never", which hides the folding chevrons but maintains your ability to fold the document. I run all my fold commands from Command Palette. This makes the Gutter smaller, but more importantly. moves the line numbers closer to the document:
-clicking on a line number selects the entire line
-click and drag starting on a line number creates a selection that starts at that line.
which basically means the "line number" in the gutter becomes part of the "clickable space" that the OP was requesting be made easier to use.
To move the line down by adding a \n to the beginning of the line, I use a shortcut.
editor.action.insertLineBefore which by default is "cmd+shift+enter"
so you can click anywhere on the line and hit "cmd+shift+enter" and it will push the line down and move the cursor to the start.
all that is to say that i rarely find myself actually trying to click at the start of a line in order to type more code starting from the beginning. if i needed to do that i would just click on the line and hit "cmd+left"
hope someone finds this helpful!
Image

@FloppyDisco This is definitely the best solution so far. Thank you, I will definitely be using this.
The main shortcoming is that you can't select just part of a line starting from the beginning. But if you are wanting to select multiple lines (which is probably more common) this works quite well! Certainly better than the default

The main shortcoming is that you can't select just part of a line starting from the beginning. But if you are wanting to select multiple lines (which is probably more common) this works quite well! Certainly better than the default

@dschil138 if you need just part of the line, click at the right edge of what you want and drag the mouse left!


Is it possible to connect client from localhost(http) to identityserver(https)

Please only use the issue tracker for simple questions. For general security questions, or free / commercial support options do not use the issue tracker and instead see here for more details.
For questions containing large amounts of code we recommend submitting a question on stack overflow using the Identity Server 4. Our amazing community on Stack Overflow is there to help answer all of your questions.
Finally, please keep the issue concise and to the point if your question does include code be sure it is a Minimal working example. If you paste in more code than the text for your question then it probably belongs on Stack overflow.
Question / Steps to reproduce the problem
I deployed a remote identity server in azure app service (iis) so its using https
And I have a local SPA client which uses http. I want to connect my local SPA client to this identity server in azure. But I'm encountering error in the browser whenever i login to identity server and it redirects to my local machine.
Minimal working example
I allowed everything in CORS
            services.AddCors(options =>
            {
                // this defines a CORS policy called "default"
                options.AddPolicy("default", policy =>
                {
                    policy
                    //.WithOrigins(
                    // timeKeepingOptions.WebClientUrl
                    //)
                    .AllowAnyOrigin()
                    .AllowAnyHeader()
                    .AllowAnyMethod()
                    .AllowCredentials();
                });
            });

            var cors = new DefaultCorsPolicyService(null)
            {
                AllowedOrigins = { timeKeepingOptions.WebClientUrl }
            };
            cors.AllowAll = true;
            services.AddSingleton<ICorsPolicyService>(cors);

Relevant parts of the log file
Here is the request log that has the error
Request URL: https://bizbox-accounts-dev.azurewebsites.net/connect/userinfo
Referrer Policy: no-referrer-when-downgrade
Provisional headers are shown
Access-Control-Request-Headers: authorization
Access-Control-Request-Method: GET
Origin: http://localhost:3031
Referer: http://localhost:3031/callback
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/75.0.3770.100 Safari/537.36

Additional logs from the server
2019-06-23 07:56:35.606 +00:00 [Error] Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware: An unhandled exception has occurred while executing the request.
System.InvalidOperationException: **IDX20803: Unable to obtain configuration from: '[PII is hidden]'.**
   at Microsoft.IdentityModel.Protocols.ConfigurationManager`1.GetConfigurationAsync(CancellationToken cancel)
   at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()
   at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()
   at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.AuthenticateAsync()
   at Microsoft.AspNetCore.Authentication.AuthenticationService.AuthenticateAsync(HttpContext context, String scheme)
   at IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler.HandleAuthenticateAsync()
   at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.AuthenticateAsync()
   at Microsoft.AspNetCore.Authentication.AuthenticationService.AuthenticateAsync(HttpContext context, String scheme)
   at Microsoft.AspNetCore.Authorization.Policy.PolicyEvaluator.AuthenticateAsync(AuthorizationPolicy policy, HttpContext context)
   at Microsoft.AspNetCore.Mvc.Authorization.AuthorizeFilter.OnAuthorizationAsync(AuthorizationFilterContext context)
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeFilterPipelineAsync()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeAsync()
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.Routing.EndpointRoutingMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.Diagnostics.StatusCodePagesMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Cors.Infrastructure.CorsMiddleware.InvokeCore(HttpContext context)
   at IdentityServer4.Hosting.IdentityServerMiddleware.Invoke(HttpContext context, IEndpointRouter router, IUserSession session, IEventService events)
   at IdentityServer4.Hosting.MutualTlsTokenEndpointMiddleware.Invoke(HttpContext context, IAuthenticationSchemeProvider schemes)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Cors.Infrastructure.CorsMiddleware.InvokeCore(HttpContext context)
   at IdentityServer4.Hosting.BaseUrlMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.Invoke(HttpContext context)

Btw this works perfectly if i have both identity server and the spa client in my localhost both using http

IDX20803: Unable to obtain configuration from: '[PII is hidden]'.

Usually points to HTTPS config problems on the server.

Cool thanks for the tip. @leastprivilege I did some adjustments based on that.
So basically my setup is
Identity Server in Azure App Service is using https with AddDeveloperSigningCredential just using the autogenerated certificate for now.
My SPA is also using https with self-signed cert and it runs locally - I get warnings whenever I access it but it goes away once I acknowledge it.
This particular request is always cancelled when i check the browser network log
Request URL: https://bizbox-accounts-dev.azurewebsites.net/connect/userinfo
Referrer Policy: no-referrer-when-downgrade
Provisional headers are shown
Access-Control-Request-Headers: authorization
Access-Control-Request-Method: GET
Origin: https://localhost:3031
Referer: https://localhost:3031/callback
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/75.0.3770.100 Safari/537.36

Any advice?
Should I be using AddDeveloperSigningCredential or should I use a proper certificate and use it on both ends?
Thanks

AddDeveloperSigningCredential is unrelated to TLS.

All set on this issue -- can we close?

@brockallen yeah I guess we can, I was able to sort of make it work.
After I configured both ends to use https it seems that I was able to get it to redirect to my client app.
I got stuck in an infinite loop though. But maybe its more of a client routing issue. I haven't verified that.
It always fails on connect/authorize
Anyway will update this issue once I have more info.
Thanks

@reggieboyYEAH  any update on this? I'm facing a similar issue

Yes @naulacambra turns out its a routing issue. My callback route loads late  so it fails.


Create UiState for BookmarkFilters to Encapsulate Business Logic

To simplify our Composable functions and move the business logic to UiState, we need to create a UiState for BookmarkFilters. More details can be found in the project's Screen section at https://github.com/DroidKaigi/conference-app-2023#screen.

🙋♂


outlined text field broken

<mdc-text-field label="help" outlined></mdc-text-field>
This simple line spits out the following:
ERROR Error: "Uncaught (in promise): TypeError: this._input is undefined
_getInputElement@http://localhost:4200/vendor.js:19361:9
isFocused@http://localhost:4200/vendor.js:18963:78
./node_modules/@material/textfield/foundation.js/MDCTextFieldFoundation.prototype.init@http://localhost:4200/vendor.js:121507:27
init@http://localhost:4200/vendor.js:19082:26
layout@http://localhost:4200/vendor.js:19246:14
set outlined@http://localhost:4200/vendor.js:18712:18
setInputsFromAttrs@http://localhost:4200/vendor.js:54148:37
instantiateAllDirectives@http://localhost:4200/vendor.js:53855:13
createDirectivesInstances@http://localhost:4200/vendor.js:53088:5
ɵɵelementStart@http://localhost:4200/vendor.js:61701:9
ɵɵelement@http://localhost:4200/vendor.js:61757:5
MyComponent_Template@http://localhost:4200/example-page-example-page-module.js:213:52

Angular MDC Version used: 5.0.0
"dependencies": {
    "@angular-mdc/web": "^5.0.0",
    "@angular/animations": "~9.0.0",
    "@angular/common": "~9.0.0",
    "@angular/compiler": "~9.0.0",
    "@angular/core": "~9.0.0",
    "@angular/forms": "~9.0.0",
    "@angular/platform-browser": "~9.0.0",
    "@angular/platform-browser-dynamic": "~9.0.0",
    "@angular/router": "~9.0.0",
    "rxjs": "~6.5.4",
    "tslib": "^1.10.0",
    "zone.js": "~0.10.2"
  },

Hi @mptyspace Can you provide reproducible source? The text field outlined variant is used extensively on the demo app without issue.

I am also getting a similar error with an outlined attribute on the mdc-text-field:
Example:
<mdc-text-field
  label="Email"
  outlined
  formControlName="email"
  ></mdc-text-field>

Error:
ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'nativeElement' of undefined
TypeError: Cannot read property 'nativeElement' of undefined
    at MdcTextField._getInputElement (textfield.js:923)
    at Object.isFocused (textfield.js:525)
    at<EMAIL_ADDRESS>(foundation.js:132)
    at MdcTextField.init (textfield.js:644)
    at MdcTextField.layout (textfield.js:808)
    at MdcTextField.set outlined [as outlined] (textfield.js:274)
    at setInputsFromAttrs (core.js:12941)
    at instantiateAllDirectives (core.js:12648)
    at createDirectivesInstances (core.js:11881)
    at ɵɵelementStart (core.js:20943)
    at resolvePromise (zone-evergreen.js:793)
    at resolvePromise (zone-evergreen.js:752)
    at zone-evergreen.js:854
    at ZoneDelegate.invokeTask (zone-evergreen.js:400)
    at Object.onInvokeTask (core.js:40744)
    at ZoneDelegate.invokeTask (zone-evergreen.js:399)
    at Zone.runTask (zone-evergreen.js:168)
    at drainMicroTaskQueue (zone-evergreen.js:570)
    at ZoneTask.invokeTask [as invoke] (zone-evergreen.js:485)
    at invokeTask (zone-evergreen.js:1596)

Removing the outlined attribute resolved the error, but then the text field is no longer an outlined text field

Update:
It seems to be an issue with boolean attribute coercion. In past versions, putting just outlined on the text field worked. I was able to resolve the error by using [outlined]="true" instead. Alternatively, the injection options work too:
 providers: [
    {
      provide: MDC_TEXT_FIELD_DEFAULT_OPTIONS,
      useFactory: () => defaultOptions
    },
    {
      provide: MDC_SELECT_DEFAULT_OPTIONS,
      useFactory: () => defaultOptions
    }
  ]

Hmm, that would make sense. Must be some package is causing issues, I'll see if I can fix it locally tomorrow by equalizing the my app with the demo.

@mrp67 Thank you for investigating this! Could you also verify the version value in<EMAIL_ADDRESS>Expecting to match "version": "9.0.0",

@trimox just checked it matches 9.0.0 for @angular/cdk

@mrp67 Great thanks for confirming. Investigating as potential race condition scenario in layout(). Digging further..

Duplicated.

Got it. PR incoming

@mptyspace @mrp67 Grab v5.0.2 which contains fix. Thanks again!


Pooja/ui changes

PR Changes::

splash screen also added
holds complete UI changes

Great work ma'am, I really like the new UI ! :D

Glad, you liked it! :)
On Sun, Oct 11, 2020 at 1:47 AM Pawan Tanay Tripathi <
<EMAIL_ADDRESS>wrote:

Great work ma'am, I really like the new UI ! :D
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dscabesit/Simple-OCR/pull/4#issuecomment-706605534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKZ7MAMNQQTE5ZEBDHDHZ63SKC6NXANCNFSM4SIRW5ZA
.


Peers in 'Loop' tab showing inbound liquidity for pending channels

In the Loop tab, The total inbound/outbound liquidity slider at the top only reflects balances of confirmed channels it seems.
However, the Peers section channel sliders below the main overview one shows added liquidity for pending channels. This is a problem because it is possible to get many stuck pending channels. Please fix the sliders in the Peers section to behave the same as the 'main' slider.
I have attached a screenshot showing a peer and its selection of channels. You can see the pending channel, with a yellow dot, is included in the total liquidity slider.

Thanks for the report and for catching this! We'll clean this up in a coming release!


add dates cross browser support

All Submissions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions

[x] Does your submission pass tests?
[x] Have you lint your code locally prior to submission?

Changes to Core Features

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally?

Pr to fix the #81, it seems that it was a problem with thetransform property prefixes, but now I don't have an ability to check this fix, so it would be nice if someone could confirm correct work

It's look like this is a weird bug on webkit-based browsers with using transform-box and transform-origin properties as described in this question on Stack Overflow(link) so I close this pr for now.


implemented fix respons to chat base

I am getting a lot of non-conforming responses from gpt-35-turbo, so implemented fix_test function with a prompt

@hwchase17 this one would solve a lot of issues I'm encountering and it doesn't break tests!

Nice work, this will help a lot.
I was thinking the fixing prompt can be automatically generated like this btw
https://github.com/hwchase17/langchain/pull/2656#issuecomment-1506399807

@yummydum I agree there are other ways to do it, this one worked well for me, I'm thinking maybe also allowing a str and function that you can set from the user side, where you can run other code as well, i don't think always putting in the full formatting instructions always make sense (especially from a token perspective), but with a function you might be able to build something that the first time it gives back a simple str, and when it makes a mistake multiple times that you then resubmit the full formatting instructions, or something similar!

after changes to agents and refactors, this should probably go in the output parsing now?


Handle groupadd in Ubuntu 24.04

Hi team,
May I inquire about the groupadd enhancement in Ubuntu 24.04? Below is the content.
Closes: #9502

Starting from Ubuntu 24.04, the shadow package has been upgraded to 4.13: https://launchpad.net/ubuntu/+source/shadow.
groupadd and groupmod commands can now modify members.
Add manages_members feature to Ubuntu 24.04 and above.

root@dev:~# groupadd --help
Usage: groupadd [options] GROUP

Options:
  -f, --force                   exit successfully if the group already exists,
                                and cancel -g if the GID is already used
  -g, --gid GID                 use GID for the new group
  -h, --help                    display this help message and exit
  -K, --key KEY=VALUE           override /etc/login.defs defaults
  -o, --non-unique              allow to create groups with duplicate
                                (non-unique) GID
  -p, --password PASSWORD       use this encrypted password for the new group
  -r, --system                  create a system account
  -R, --root CHROOT_DIR         directory to chroot into
  -P, --prefix PREFIX_DI        directory prefix
  -U, --users USERS             list of user members of this group
      --extrausers              Use the extra users database

root@dev:~# groupmod --help
Usage: groupmod [options] GROUP

Options:
  -a, --append                  append the users mentioned by -U option to the group
                                without removing existing user members
  -g, --gid GID                 change the group ID to GID
  -h, --help                    display this help message and exit
  -n, --new-name NEW_GROUP      change the name to NEW_GROUP
  -o, --non-unique              allow to use a duplicate (non-unique) GID
  -p, --password PASSWORD       change the password to this (encrypted)
                                PASSWORD
  -R, --root CHROOT_DIR         directory to chroot into
  -P, --prefix PREFIX_DIR       prefix directory where are located the /etc/* files
  -U, --users USERS             list of user members of this group

Can one of the admins verify this patch?


Unable to train 'PickclutterYCB' using PPO

Hi,
I am facing an issue when training PPO policy on 'PickClutterYCB' task.
Command used: python examples/baselines/ppo/ppo.py --env_id="PickClutterYCB-v1" --exp-name="state-pickclutterYCB" --num_envs=1024 --update_epochs=8 --num_minibatches=32 --total_timesteps=5_000_000 --eval_freq=8 --num-steps=20
Issue:
Traceback (most recent call last):
File "/home/jai/ManiSkill/examples/baselines/ppo/ppo.py", line 270, in
returns = np.concatenate(returns)
ValueError: need at least one array to concatenate
Am I using the wrong command line args for training this skill?

This occurs if the number of evaluation steps is too small and so none of the episodes complete during evaluation.
Try setting num_eval_steps=100 (it has to be at least as high as the max episode steps value of the environment).

Got it, thanks a lot. Its working now.


FIX: add buffer syncing for output netcdf file on each save

Add buffer syncing for output netcdf file on each save, this prevents a memory overload at the end of a run, if trying to save a large netcdf file.
Ask me how I know we need this...

Codecov Report

Merging #113 (043b0f3) into develop (36682d9) will increase coverage by 3.28%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #113      +/-   ##
===========================================
+ Coverage    76.43%   79.72%   +3.28%
===========================================
  Files           11       11
  Lines         1528     1879     +351
===========================================
+ Hits          1168     1498     +330
- Misses         360      381      +21

Impacted Files
Coverage Δ

pyDeltaRCM/iteration_tools.py
95.04% <100.00%> (+0.04%)
:arrow_up:

pyDeltaRCM/init_tools.py
99.35% <0.00%> (-0.01%)
:arrow_down:

pyDeltaRCM/model.py
93.09% <0.00%> (+2.12%)
:arrow_up:

Continue to review full report at Codecov.

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


(🐞) add py.typed file

resolves #4822

can this be merged? i'm trying to use the robot api to make a robot pytest plugin (#4834) and it would be nice if i could use the types


ITE: drivers/i2c: create pinmux phandle to the I2C driver node

Create the pinmux phandle to the I2C driver node in the
devicetree. When the pinmux_pin_set function in
i2c_it8xxx2_init can refer to the setting of this phandle.
It is more flexible to use.
Signed-off-by: Tim Lin<EMAIL_ADDRESS>

This change is

DT_NODELABEL(pinctrl_i2c_clk**##idx**) and DT_NODELABEL(pinctrl_i2c_data**##idx**) needs index parameter.


NpgsqlDatabaseCreator does not take into account PostgreSQL schema

Methods checking existence of EF model related tables does not take into account possibility of another schema being in the database and thus leading to context.Database.EnsureCreated or EnsureCreatedAync doing nothing.
Culprit is (I believe):
NpgsqlDatabaseCreator.CreateHasTablesCommand
SELECT CASE WHEN COUNT(*) = 0 THEN FALSE ELSE TRUE END
FROM information_schema.tables
WHERE table_type = 'BASE TABLE' AND table_schema NOT IN ('pg_catalog', 'information_schema')

This is satisfied with tables from other schema than the one DbContext should work with.

Could you post some code showing what you're trying to do/not working?

What do you mean by the schema "the one DbCobtext should work with"? A single DbContext can define tables in several different schemas, which is why HasTables checks if there are any tables, in any schema...

What do you mean by the schema "the one DbCobtext should work with"? A single DbContext can define tables in several different schemas, which is why HasTables checks if there are any tables, in any schema...

Does it mean that two distinct DbContexts could not live in one database separated by schema?

Does it mean that two distinct DbContexts could not live in one database separated by schema?

It depends on what exactly you're doing. If you take care to make sure each context defines database objects in different schemas, this may work. Also, you have to make sure each context uses its own migrations table to avoid a conflict.
Am going to close this issue as the provider seems to do the right thing, but we can continue the conversation and reopen if needed.

Well, it's confusing to say at least that during mapping you can define default schema (ModelBuilder.HasDefaultSchema()), which is respected - all tables are created within that schema. But the concept of schema is ignored in NpgsqlDatabaseCreator checks used during EnsureCreated(). This leads to what I believe is a false positive - thinking that required schema and tables are created since it is satisfied by just any table.
IMHO mychema1.table1 is a different table than myschema2.table1.

Ah, I think I see the disconnect here. What you're describing can indeed be confusing, but it is documented:

Ensures that the database for the context exists. If it exists, no action is taken. If it does not exist then the database and all its schema are created. If the database exists, then no effort is made to ensure it is compatible with the model for this context.

Note that this API does not use migrations to create the database. In addition, the database that is created cannot be later updated using migrations. If you are targeting a relational database and using migrations, you can use the DbContext.Database.Migrate() method to ensure the database is created and all migrations are applied.

Well, it's confusing to say at least that during mapping you can define default schema (ModelBuilder.HasDefaultSchema()), which is respected - all tables are created within that schema.

I understand what you're saying, but the fact that your can define the default schema tells you that it's also possible to step outside that default. You're free to define, on an entity by entity basis, which table it maps to in the database, including that table's schema. So it's very possible (and sensible) for a single model to define tables that go into different schemas.
This is why HasTables() does exactly what it sounds like it should do - tests whether there are any tables in the database, in any schema. The existence of any table indicates that the database isn't empty etc.

see this example
If HasTables() would have checked the tables as they have been defined - that is with the schema, it would have quite likely work.
Only logical conclusion is that it is either a bug in the "ensure" logic or the database is not supposed to host more than one DbContext regardless of how well you've isolated them by schema.

Only logical conclusion is that it is either a bug in the "ensure" logic or the database is not supposed to host more than one DbContext regardless of how well you've isolated them by schema.

It is a supported scenario to have multiple DbContext per database. But the database creation infrastructure is explicitly limited in scope. Again, per the docs:

Ensures that the database for the context exists. If it exists, no action is taken. If it does not exist then the database and all its schema are created. If the database exists, then no effort is made to ensure it is compatible with the model for this context.

emphasis added
The limitation is in using EnsureCreated, not with using more than one DbContext.
From the perspective of your first DbContext, the only database objects that should exist are those defined within it. Then by the time your second context runs, it finds the database exists in some state and makes no effort to verify or correct that state.
You can work the problem in a couple of ways:

Define a third DbContext that defines the overall layout and use it'sEnsureCreated to establish the baseline.

Just have the combined context and have it implement two interfaces which are passed around in place of the combined context.

Stick with the original setup, but use the migrations facilities to create a more customized creation procedure than what EnsureCreated offers out of the box.

@jdvor if you want to continue this conversation, it may be better to do so in https://github.com/aspnet/EntityFrameworkCore. We're only implementing the EF Core provider for PostgreSQL here but the discussion is general.

@jdvor Please link back here if you do open a discussion upstream so that we can follow along.


Can't find the origin of data in the charts

Hi everyone !
I'm currently adapting the dashboard to my project, and I wanted to edit the "Sales" graph. Problem : impossible to find where the data used in order to make the graphic are. Any idea ?
I'm talking about this graph :

Area

Donut

 Sales

If you are using Visual Studio in English, Ctrl-F is useful to search stuff.
The data is hardcoded as an example. The file is Scripts\adminlte\pages\dashboard.js
If you want to load data, then you can follow the example of the KPIs. It's basically the same.
This is what I did to put some simple data in the donut and the bar chart:
In the DashboardIndex.cshtml

In a new file named DashboardChart.ts
        static initializeChart() {
            $(function () {

                Common.DashboardPageService.CountByStatus({}, response => {
                    this.areaChart = new Morris.Bar({
                        element: 'revenue-chart',
                        resize: true, parseTime: false,
                        data: response.Values,
                        xkey: 'Month',
                        ykeys: response.StatusKeys, labels: response.StatusLabels, hideHover: 'auto'
                    });
                });

What I did was to follow the example of the Chart In Dialog.
FYI. I'm not a web developer so these examples may not be the best way of doing it.
Also, search the forum (ctrl-f), someone shared some examples of how to correctly implement this.
Gus

I have a sample that shows the implementation of the charts using the basic samples chart in dialog.
https://github.com/sayuga/AdminLTE-Widgets-in-Typscript/tree/master/Charts
You'll have to create a service call to get the data for the chart. The data being used in the current dashboard samples is being fed through a scrip that is holding dummy data in a .js file so it'd be useless to adapt that end.
I've gotten most of the Dashboard widgets rebuilt in my repository so you are welcome to use that as a starting point. I haven't finished them due to another project taking priority but let me know if you need more help.

Hey @sayuga Thank you!! this was the example I was talking about! awesome stuff!
Gus.

@gguadalupe , If your issue is resolved, please close the issue.

Hi @sayuga, this is not mine. It's @DoeJowns
Gus.

Oops. My mistake. @DoeJowns htne. If you are all set please remember to close the ticket.
Sorry @gguadalupe

@sayuga  thanks ,your example is very useful fro me .


⚠ Samsung has degraded performance

In 9b3f110, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2237 ms

Resolved: Samsung performance has improved in 50921c4.


Update Posts “welcome”

Automatically generated by Netlify CMS

👷 Deploy Preview for ashfolios processing.

Name
Link

🔨 Latest commit
5c4d267dd00e0b6f11f5440e327bfa2203956ec1

🔍 Latest deploy log
https://app.netlify.com/sites/ashfolios/deploys/644a042c245e0000076f6dd5


Run experiment in localhost in the same process

Description
Currently when we run grid it spawns new process which runs cc_train. This PR changes this behaviour, so in localhost it runs in the same process. It enables easier local debugging session as we can run json specified files.
Checklist
Answer each question with "yes", and provide an explanation if the answer isn't what is expected.

Is this PR focused on a single feature without the possibility of being divided into smaller PRs?

Answer: yes

Are all variables set with logical defaults that are backward compatible?

Answer: yes

Can you provide a link to a Neptune experiment verifying the functionality of this PR?

Answer: NA

Have you successfully executed the linter and tests?

Answer: yes

Are all functions concise and don't require splitting?

Answer: yes

Is there documentation for arguments and complex logic?

Answer: yes

Are there no temporary solutions in the code? Should a task be added to Trello to refine the code?

Answer: no

Are all functions placed in appropriate files?

Answer: yes

Is the PR name meaningful and descriptive?

Answer: yes

Is PR description provided, or unnecessary?

Answer: yes

Other questions:

Have you not made changes to the requirements or anything related to the Singularity image? If you have, did you generate and upload a new image to the clusters?

Answer: no

Is the change significant enough to notify all individuals working with this repository?

Answer: yes

Is there a need for a second reviewer?

Answer: yes

Waiting for reviewer 2


Records can't be serialised and deserialised by Conjure

mynomoto over on Slack pointed out that if you try to return a record from an eval Conjure explodes.
(defrecord Foo [bar])
(->Foo "bar")

Because that evaluates to the string
#conjure.code.Foo{:bar \"bar\"}

Which Conjure tries to read because it reads all prepl responses (for better or worse...). I think the main need to read responses is because I want to check if it was an error or not. That will go away when I overhaul how errors and exceptions work.
Would be cool to fix this ASAP, but if you hit this before then just (into {} r) where r is your record. It'll get around this issue until I can patch it out or it vanishes on it's own when I improve errors.

Patch released in v0.17.1 to mitigate this, it'll show you the parse error and not kill the connection. Full fix will come with the errors / exceptions rework I think..

Error improvement work is now waiting for this patch to go into ClojureScript https://clojure.atlassian.net/browse/CLJS-3096
Once that's in I can rewrite how the error handling system works which means I don't need to parse output from prepl evaluations which means this error will vanish. This will make Conjure a lot more robust too, right now there's a few cases where returning something out of the ordinary will cause it to choke.
If you hit one of these errors, I'm really sorry. You'll just have to pour the data into Clojure data structures or use bean until this is patched, hopefully it won't be too long!

I think I'm running into this with Manifold streams. They have a custom print-method that returns this:
<< stream: {:pending-puts 0, :drained? false, :buffer-size 0, :permanent? false, :type "manifold", :sink? true, :closed? false, :pending-takes 0, :buffer-capacity 0, :source? true} >>

and causes Conjure to report "Invalid symbol: stream:.".
No rush, just wanted to add another data point.
Thanks for making Conjure, I enjoy using it every day.

Ah damn, I'm sorry about that :sob: try pouring it into a map or bean, it might get around the printing temporarily. I'm still waiting on my patch going into ClojureScript, maybe I can do a halfway house soon to fix the Clojure side at least.
Once that CLJS patch is in I won't have to parse anything returned from the prepl which will make things a lot easier. I'll see what I can do to mitigate it until the proper fixes are possible :thinking:

I'm also being hit by this in a project I'm working on.
I did find a workaround: if you find yourself needing to evaluate a form where you know the result is a record, you can wrap it with (into {} the-form) This turns the record into a plain map.
There is still a problem if you want to evaluate a form that contains records (e.g. a map where the values include records). I found a solution for this on StackOverflow: (clojure.walk/postwalk #(if (record? %) (into {} %) %) the-form)
It's not ideal, but I think it will tide me over until a fix comes down the pike :)

I've got an idea for how I could potentially work around this until ClojureScript prepl hits feature parity with Clojure prepl :thinking: I'll see what I can do ASAP, just got talk planning to do for next Wednesday.

Testing Slack integration...

@daveyarwood great idea to use clojure.walk. Here's my augmented version that handles manifold streams:
(defn nr
  [form]
  (clojure.walk/postwalk
    #(cond
       (record? %) (into {} %)
       ((supers (class %)) manifold.stream.core.IEventStream) (pr-str %)
       :else %)
    form))

So when someone who's experienced this has some free time, would you be able to try it again but on the result-refactor branch? https://github.com/Olical/conjure/pull/47
I'll try to repro too, but it would be great if you could try to do exactly what you did before that caused Conjure to explode. I'm hoping my changes have fixed it, if not I think a slight tweak will be all that's needed after that refactor work.
Thanks a lot!

A simple repro case is to evaluate the following using your project's prepl:
;; this works because the return value is the var `my-namespace.Foo`
(defrecord Foo [])

;; this blows up because the return value is a record
(->Foo)

I just tried the result-refactor branch, and the issue appears to be fixed!

I also tried it out on the work project where I ran into this issue before, and it's fixed there too :+1:

Yessss, thank you for checking, that's great news. I'll wrap up the rest of that PR and get it out in a tagged release.

The issue is fixed for me as well. Thanks!

So this is fixed in v0.21.0 😄


Angular 4 ng build --prod does not work

Hi @loedeman
I could make your lib work in a Angular v4 project + CLI and it seems great!
I created a service where I am loading configurations for every module to register the mappings and then, just using automapper.map works smoothly.
However, I have the problem that if I run "ng build --prod" with the Angular CLI, all the values are coming as empty, so in a table of 7 rows, I get those 7 objects, but they are empty value.
During Environment config and without --prod flag works perfectly, no with that flag.
I know you didn't build this lib thinking on Angular project but Typescript project, but it would be nice to know your thoughts here.
I checked the scripts.bundle that cli create trhough that command and it seems to be added correctly, so no idea what could be happening.
I can create a project for you in order to recreate the error, anyway I think creating a new project with cli and adding the library would be enough, but let me know If I can help out.
Thanks,

Hi @ialex90, thanks for reaching out. I would be glad if you could create a project for me to recreate the error. I am afraid I cannot fully understand your issue without it, actually ;). I am afraid the Angular optimizations kicking in when building production mode is messing up with AutoMapperTS, I am curious to discover what the actual problem is and how/if it could be fixed. Hope to hear from you soon ;)! Cheers, Bert

I'm facing with a similar issue.
I'm working with an angular v4 project with webpack.
My "createMaps" are all based with a string sourceKey and a classType destinationKey pattern. The "convertToType" fluent method is not needed in this case because it automatically return my destination classType.
automapper.createMap('json', MyClassA);
automapper.createMap('json', MyClassB);

But this creates strange behaviour when using "prod" flag: wrong classes mapped.
I finally discovered that the uglify plugin I'm using UglifyJsPlugin is mangling the variable "MyClass.name" used (correctly) by AutomapperHelper.getClassName method.
Unfortunately with "prod" flag more than one class returns the same class name "t" :(
In the example above the strange behaviour was that the map of MyClassA returned a MyClassB object.
I added "console.log(MyClassA.name)" and "console.log(MyClassB.name)" and both messages returned 't'
I'm currently using the workaround of using string as destinationKey with "convertToType(MyClass*)" fluent method.
automapper.createMap('json', 'MyDestinationKeyForClassA').convertToType(MyClassA);
automapper.createMap('json', 'MyDestinationKeyForClassB').convertToType(MyClassB);

This works for me even if I'm not so satisfied. Using strings for source and destination keys makes my code less maintenable in case of refactoring...

Hi @lucaritossa
Thanks for pointing this out.
You steered me in the right direction.
the webpack -p option does by default mangle your code but you can override that by specifying the uglifyjsplugin options in your webpack config file and omitting the -p option.
this solved it for me:
`  new UglifyJSPlugin({
cache: true,
        uglifyOptions: {
            beautify: false, //prod
            sourceMap: false,
            ecma: 6,
            mangle: {
                keep_fnames: true,
                keep_classnames:true,
                reserved: ['$super', '$', 'exports', 'require']
            }, //prod
            compress:
                {
                    keep_classnames: true,
                    keep_fnames:true,
                    warnings: false,
                    conditionals: true,
                    unused: true,
                    comparisons: true,
                    sequences: true,
                    dead_code: true,
                    evaluate: true,
                    if_return: true,
                    join_vars: true,
                    negate_iife: false // we need this for lazy v8
                },
            output: {
                //     space_colon: false,
                comments: function (node, comment) {
                    var text = comment.value;
                    var type = comment.type;
                    if (type === "comment2") {
                        // multiline comment
                        return /@copyright/i.test(text);
                    }
                }
            }
        }`


Add Deserializer for server bound packets

Identical implementation as #17 but for deserializing server bound packets, for details look there.

Looks good, Thanks  :heart:


Dg sensors

Opening pull to move light sensors into main

Self-merging, I will smooth out any outstanding changes on the PS master


Working with stored procedures

deleting from a dialog box, how do I pass multiple parameters to a stored procedure which does the removal ? DeleteRequest only supports EntityId. How to get around this problem ?

dialogbox should have a reference to entityid, are you not able to pass that to DeleteRequest? maybe I don't understand but why use a stored proc when serenity handles that for you?

I have a complex stored procedure.
It collects aggregated data from the various tables and provides data to the user.
As such, EntityId, I'm getting a composite of A_Id, B_Id and C_Id.
In order to remove some entry, I need to pass to the stored procedure, not one
the key is in the form of EntityId and a few

OK so I guess you'll need to get A_Id, B_Id and C_Id first and pass these to the stored proc? If so, from the dialog you can query your other endpoints to get id for B and C using a relation from A and pass that to your stor proc. for example this is how I've done to get ID from another table.

            this.form.IntachangeUser.changeSelect2(e => {
                var selectedIntachangeUser = this.form.IntachangeUser.value;

                if (Q.isEmptyOrNull(selectedIntachangeUser)) {
                    this.setNewUserDetails({});
                    return;
                }

                //this is another way of getting the user info but it requires another service call which is not needed if data exists
                //var intachangeUsers = Q.getLookup<IntachangeStarterRow>("Deskside.IntachangeStarterUserNameLookup");
                //let user = Q.first(intachangeUsers.items, x => x.FullName == selectedIntachangeUser);

                var id = (Q.first(this.form.IntachangeUser.items, x =>
                    ((x.source as IntachangeStarterRow).FullName == selectedIntachangeUser)).source as IntachangeStarterRow).IcmdUid;

                //var id = (intachangeStarterRowUser.source as IntachangeStarterRow).IcmdUid;

                IntachangeStarterService.Retrieve({
                    EntityId: id
                }, response => {
                    this.setNewUserDetails(response.Entity);
                });

It's all to do in the constructor of the dialog ?

well in my case yes as I'm using the event from select2 but only because I need it to change values in other fields. you could probably override the delete button function and put your bits in there.

how to override the event for the delete button in the dialogue ?

that bit I don't know I'm afraid, if anyone else reads this thread maybe they could assist with that.

thank you

check #814
shows how to intercept button events

what I need. thank you

Thanks @abelal83 for your helpful contrubitions in last few days.


打包报错

Android和ios都报这个错误
:~/libXray# python3 build/main.py android
['build/main.py', 'android']
go: creating new go.mod: module github.com/xtls/libxray
go: to add module requirements and sums:
        go mod tidy
go: finding module for package google.golang.org/protobuf/proto
go: finding module for package gopkg.in/yaml.v3
go: found github.com/xtls/xray-core/infra/conf in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found github.com/xtls/xray-core/transport/internet in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found github.com/xtls/xray-core/common/net in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found github.com/xtls/xray-core/proxy/vless in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found gopkg.in/yaml.v3 in gopkg.in/yaml.v3 v3.0.1
go: found github.com/xtls/xray-core/app/router in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found github.com/xtls/xray-core/common/cmdarg in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found github.com/xtls/xray-core/common/uuid in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found github.com/xtls/xray-core/core in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found github.com/xtls/xray-core/main/distro/all in github.com/xtls/xray-core v0.0.0-00010101000000-000000000000
go: found google.golang.org/protobuf/proto in google.golang.org/protobuf v1.36.0
# github.com/xtls/libxray/share
share/clash_meta.go:384:26: undefined: conf.HTTPConfig
share/clash_meta.go:389:19: streamSettings.HTTPSettings undefined (type *conf.StreamConfig has no field or method HTTPSettings)
share/clash_meta.go:396:19: streamSettings.GRPCConfig undefined (type *conf.StreamConfig has no field or method GRPCConfig)
share/generate_share.go:291:21: streamSettings.GRPCConfig undefined (type *conf.StreamConfig has no field or method GRPCConfig)
share/generate_share.go:294:26: streamSettings.GRPCConfig undefined (type *conf.StreamConfig has no field or method GRPCConfig)
share/generate_share.go:300:33: streamSettings.GRPCConfig undefined (type *conf.StreamConfig has no field or method GRPCConfig)
share/generate_share.go:304:31: streamSettings.GRPCConfig undefined (type *conf.StreamConfig has no field or method GRPCConfig)
share/generate_share.go:309:21: streamSettings.HTTPSettings undefined (type *conf.StreamConfig has no field or method HTTPSettings)
share/generate_share.go:312:26: streamSettings.HTTPSettings undefined (type *conf.StreamConfig has no field or method HTTPSettings)
share/generate_share.go:316:26: streamSettings.HTTPSettings undefined (type *conf.StreamConfig has no field or method HTTPSettings)
share/generate_share.go:316:26: too many errors
Traceback (most recent call last):
  File "/root/libXray/build/main.py", line 32, in <module>
    builder.build()
  File "/root/libXray/build/app/android.py", line 13, in build
    self.before_build()
  File "/root/libXray/build/app/android.py", line 9, in before_build
    super().before_build()
  File "/root/libXray/build/app/build.py", line 113, in before_build
    self.download_geo()
  File "/root/libXray/build/app/build.py", line 64, in download_geo
    raise Exception("download_geo failed")
Exception: download_geo failed

Don't use the pre-release version. This lib only keeps compatible with Xray-core's stable version. Currently, it should be v24.11.30.


Cmake MKL_ThreadingLibrary

Recent building attempt on our cluster failed upon linking because
MKL_ThreadingLibrary_LINK_LIBRARY:FILEPATH
is not found. Cmake does not throw an error. Workaround only possible by adding the correct filename of libiomp5.so manually.

The lib is searched for here: https://github.com/votca/tools/blob/bfdffabb46eca082e9ebd283ffb5626f247631f2/CMakeModules/FindMKL.cmake#L222
and here
https://github.com/votca/tools/blob/bfdffabb46eca082e9ebd283ffb5626f247631f2/CMakeModules/FindMKL.cmake#L144

is there a way to search for openmp first and then set it for the mkl instead of the other way round. Because in xtp we are searching for openmp anyway?

I think this is finished for the moment


Fix load more than 100 property groups and friends

1. Why is this change necessary?
There are lots of occassions where a complete list of objects is required but not requested properly. Most of these places are visible when you see a criteria limit of 100 or 500. These are tries to load all the data but this is not all the data. This might be the case because although it is tried to have php and admin-js behave identically regarding DAL operations it is not as the criteria in admin-js works with limited pages that have a preset of 1, 25 and the php criteria works with limit offset and has a preset of null, null. Those who work with the admin-js criteria and are familiar with it from php get fooled.
2. What does this change do, exactly?
Add the php-known RepositoryIterator and some constants from the criteria. Use the repository iterator at a lot of places that behave better now.
3. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/NEXT-10262
https://issues.shopware.com/issues/NEXT-10349
And probably some more
4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Sorry for the re-open of https://github.com/shopware/platform/pull/1310. I butter-fingered the delete external button :/

Being nitpicky is not bad :D I'll have a look at it

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://docs.shopware.com/en/shopware-platform-dev-en/contribution/contribution-guideline

That moment when you thought you've already fixed it and find the PR declined :< yikes

reopen it already! 😃

@mitelg :O just reopen? All the rebases I have to do :(

I guess a new branch/PR and cherry pick would be the better solution 🙂

We will have a look at it again and I will come back to it @JoshuaBehrens. We think this is useful but have to check some possible side effects

@htkassner I am here :) ready for your side-effects-check

@htkassner hej, since the pullrequest #2049 has been declined, and this has been declined as well, what is your proposed solution and for which releases will it be scheduled? Thanks for a concise answer on that issue (NEXT-14201)

@renegr-moonshiner this is just an out-of-time decline. Not content related.
Still keen to know about the schedule though 🧐

First thin we will contribute is the solution for the sales channels which has highest prio. This is in the final stages

@htkassner good to hear, that's in the final stages. Which version will receive this bugfix?

When its working out like we want this should be in the next release in october. Right now I could not say why this wouldn´t work. So let´s c. @renegr-moonshiner

@htkassner @renegr-moonshiner @jochenmanz @mitelg @J-Rahe just a shout-out for those who are already part of this pull request. This got a sequel pull request. Updated and rebased!


METRON-465: Automatically set storm-site topology.classpath

Set storm-site topology.classpath to /etc/hbase/conf:/etc/hadoop/conf as recommendation
Display error when storm-site topology.classpath doesn't not contain /etc/hbase/conf or /etc/hadoop/conf

Tested on Docker cluster following procedure here: Metron MPack for following cases.

Adds topology.classpath to storm-site configuration and sets it to /etc/hbase/conf:/etc/hadoop/conf when it does not exist.
Appends /etc/hbase/conf:/etc/hadoop/conf when topology.classpath does not contain both paths
Append appropriate path when only one is missing.

+1
I tested this out in the openstack environment and the fix looks good to me.
Steps followed for testing:-

Built the Mpack and RPMs by pulling out the code from dlyle65535:METRON-465
Installed Metron and related services on the 12 node openstack cluster.
The storm configuration property :-"topology.classpath"  got auto populated with the value :-
/etc/hbase/conf:/etc/hadoop/conf
Attached is the image :-

Bump?

+1 by inspection


Add Zawn

Add all the definitions of [Zawn] in a file named [Zawn].json in folder [Z]
Refer CONTRIBUTING.md  for more details on how to contribute.
Comment the words you'll be adding if you're are working on this issue to avoid conflicts.
Add reference to this issue i.e Issue # in your pull request description.

In #743


Relax requirement of rest-client

Installing this gem in an existing project failed due to a dependency resolution between the ~> 2.0.2 version of rest-client and other gems in the bundle. Since the newer versions of rest-client don't seem to have any breaking changes, I've relaxed the requirement to any higher version.
I've also regenerated the lockfile, which I'm not entirely sure is necessary here. ¯\(ツ)/¯

If you can update the requirement to ~> 2.0 instead of >= 2.0.2 this will be good to go.

Alright, how's this look? https://github.com/xivapi/xivapi-ruby/pull/1/commits/f03abecb30b09988ef3d81815cc18a8f668d0a79


Navigation inside the eval buffer

Hello,
First of all thanks for your work on Conjure. Really amazing.
I was wondering if you had some workflow or mappings for navigating the eval buffer: for instance when evaluating a large sequence a lot of lines will be written. In order to go back to the beginning of the output I use something like ?eval but something better would be tu use { or } to navigate the buffer. And if they would work then it would be possible to have mapping that would yank the last output by doing something such as Gy{.
I think (but I may be wrong) that adding a blank line between the ;; (eval) and the output could do the trick but maybe you have other solutions or workflow around this type of feature.
Have a great day

Hmm one option would be adding a mapping like fugitive's next and previous item mappings:
nnoremap <buffer> <silent> <C-P> :<C-U>execute <SID>PreviousItem(v:count1)<CR>

Although not sure about that. [[ and ]] seem to work to some extent too, do you mostly just need this for navigation or do you want to combine it as a text object / motion most of the time.
It's also worth noting that results are already stored in a register of your choice, so you can already paste results automatically. See :helpgrep conjure#eval#result_register for more information on that feature.


s3 to azure storage account error [bug]

Problem: I am trying to transfer from an AWS s3 bucket to the azure storage account by SkyPlane but I’m not able to transfer. I can able to transfer from AWS to AWS in different buckets but I can't share from AWS to Azure and  Azure to Azure.
Command: skyplane cp -r s3://awsbuckrest azure://azureinvet/fromaws
I have attached the error report that happened after the command that is executed.

@abiswal2001 Can you take a look at this issue?

The same problem in my case.


Zqs 1255/refactor cirq tests

Description
Include description of feature this PR introduces or a bug that it fixes. Include the following information:
Single file is used to perform test on both qsim and cirq.
Please verify that you have completed the following steps

[X] I have self-reviewed my code.
[X] I have included test cases validating introduced feature/fix.

Codecov Report
Base: 95.63% // Head: 95.63% // No change to project coverage :thumbsup:

Coverage data is based on head (2712d7c) compared to base (276eeea).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #39   +/-   ##
=======================================
  Coverage   95.63%   95.63%
=======================================
  Files           8        8
  Lines         321      321
=======================================
  Hits          307      307
  Misses         14       14

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Closing due to GitHub confusion, will reopen as a new one.


Regarding the results of Table 4 in the paper

Is it reproduced using the Time series library?

Yes, it is reproduced using the Time series library. However, I modified some parameter settings to match the original paper, as they were different from the GitHub code.

Yes, it is reproduced using the Time series library. However, I modified some parameter settings to match the original paper, as they were different from the GitHub code.


Python Function App on GKE - CrashLoopBackOff

Scenario which you are working on
Deploying a Python Function App to GKE via Azure Arc
Sorry not entirely sure what repository to pop this in but could do with some pointers to get it working.
Describe the bug
Using 0.10.0 preview microsoft.web.appservice extension for Azure Arc and deploying a Python Function App to GKE. After deployment I get intermittent availability of the Functions because they get restarted for failing the readiness/liveness probe.
  Warning  Unhealthy  23m                   kubelet            Liveness probe failed: dial tcp <IP_ADDRESS>:80: connect: connection refused
  Warning  Unhealthy  18m (x179 over 23m)   kubelet            Readiness probe failed: dial tcp <IP_ADDRESS>:80: connect: connection refused
  Warning  BackOff    3m27s (x60 over 17m)  kubelet            Back-off restarting failed container

Correct me if I am wrong but isn't the container and Kubernetes setup all done behind the scenes, so I am not able to change port rules or probe settings?

Thank you @benbhall. @apwestgarth can you take a look?

Hi @benbhall can you share any further details about the Function App and Function itself please?
Does the function operate initially?  What type of Function are you deploying?  Is it a code app or custom container?
Can you also confirm the Functions framework version is 3 and can you share the Azure Functions Core Tools version you are using, this can be gained by running func -v from the command line on the machine you are publishing from?
Thanks

Hi @apwestgarth. Apologies for not putting this in originally - I had a couple more tests to run.
Function Tools locally is on 3.0.3904.
I'm using the template Python Functions for HTTP trigger and Blob trigger, unchanged, on Python 3.8.
Both Functions operate as expected locally and deployed normally to an Azure Function App.
When deployed to an Azure Arc-enabled Kubernetes (only tried Google Cloud), I get the CrashLoopBackOff issue only when the Blog trigger function is deployed. The HTTP trigger on its own is stable.
Deployed together I can use hit the HTTP trigger in between the forced restarts and I think that once I saw the Blog trigger working.

Further digging into deployment events.
error parsing azure blob metadata: no blobContainerName given	KEDAScalerFailed
Failed to ensure HPA is correctly created for ScaledObject	ScaledObjectCheckFailed

I can eliminate the first error by setting blobContainerName in the function.json, which feels wrong but works? This is not documented for Azure Functions, it's a KEDA thing.
Looks like there was similar for working with KEDA directly: https://github.com/Azure/azure-functions-core-tools/issues/2435
No luck with the second error. Looks like a KEDA problem still. Looks like both errors are something that the k8s app service extension should be wrapping up?

@benbhall can you please let us know the steps implemented to create the blob trigger? Secondly do you already have a blob container created in Azure? Can you please check this and let us know?

Yep @raorugan. The container for the trigger definitely exists and setup for public access. Remember, this works fine when deployed to a standard Azure Function resource.

Can you pls confirm its the same blob container that your cs code is calling into? @benbhall

Can you pls confirm its the same blob container that your cs code is calling into? @benbhall

Yes, the container name in the path matches the name of my container in the storage account (from the connection string in the app setting in connection):
      "path": "test-container/{name}",
      "connection": "AzureWebJobsStorage",

See earlier comment , that i'm being forced to add the following too:
"blobContainerName": "test-container",

@benbhall
Can you attach the output of the following commands ?
kubectl get pods -A
kubectl get events -n appservice-ns
kubectl describe pod -n appservice-ns testpythonappblob-7cff686f8d-tdrnl
(replace the name of the application POD which is in CrashLoopBackOff)
kubectl get so -n appservice-ns
kubectl get hpa -n appservice-ns
kubectl logs -n appservice-ns limaarcncsenv5-kube-k8se-build-service-668b45df7d-shgpp build-service
(replace the above name with the name of your build-service POD)
kubectl logs -n appservice-ns limaarcncsenv5-kube-k8se-keda-operator-5967c97b54-q6jrl | findstr /i /C:
(replace the above with keda-operator POD in your case and  with function app name)
I am assuming the name of namespace which has extension installed is "appservice-ns" otherwise replace with actual name in above commands.
Also, can you try to create a new function app and publish the default python blobtrigger function again? Please make sure before you publish to this function app for the first time, you have all the required properties present in function.json ( in this case "path","connection" and "blobContainerName") with proper values. function.json should look something like :-
{
"scriptFile": "init.py",
"bindings": [
{
"name": "myblob",
"type": "blobTrigger",
"direction": "in",
"path": "samples-workitems/{name}",
"connection": "AzureWebJobsStorage",
"blobContainerName": "samples-workitems"
}
]
}

Output of  kubectl get pods -A: In this gist
Output of  kubectl get events -n arc-ns: In this gist
Output of kubectl describe pod -n arc-ns arc-demo-function-b7d6d5987-2fbzc: In this gist
Output of kubectl get so -n arc-ns: In this gist
Output of kubectl get hpa -n arc-ns: In this gist
Output of kubectl logs -n arc-ns arc-app-service-k8se-build-service-6fbd9d9898-zjgdt build-service: In this gist
Output of kubectl logs -n arc-ns arc-app-service-k8se-keda-operator-647b74c97-t7zcp | findstr /i /C:arc-demo-function: In this gist

This is doing everything fresh - new cluster, new arc bits and pieces and a new function with the JSON as provided.

@benbhall , please let us know if you are facing any other issues. After our call today hope you are able to test creating a container in storage account and see blob trigger execute.

@benbhall i will be closing this for now. please reply if something else is needed regarding this issue.


JSON.parse error after build.

After running npm run build and let Apache serve the /dist directory, I get an error:
Data from html2pug_params: null main.0884b0900bc2a131845f.js:1:235143
SyntaxError: JSON.parse: unexpected character at line 1 column 1 of the JSON data

And this is probably where it borks:
if(this._bodyFormData)throw new Error("could not read FormData body as text")
return Promise.resolve(this._bodyText)},n.formData && (this.formData=function(){return this.text().then(v)}),this.json=function(){return this.text().then(JSON.parse)},this}u.prototype.append=function(e,t){e=a(e),t=l(t)
var r=this.map[e]

And this is how I got there:
npm run build
cd dist
search JSON.parse > ../x.x
cd ..
cat x.x | tr '};' '}\n;' | less

BTW: when running in node, on some port, all is good.
Also when doing reverse proxy in Apacha, all is good.

Hmm, I use nodejs for transform html to pug. Without nodejs we can't transform.

So, what is the point with all that webpack/build stuff?
I'm perfectly happy running this, in a screen session, with Apache reverse proxy.
My point here is. I does not work, after running the build command, and serving
it as a standard web app.

Webpack building the frontend into dist and nodejs serve it.
You can build frontend, serve dist folder with Apache and reverse proxy /convert to nodejs.
I do something in now.
https://github.com/dimensi/html2pug/blob/master/now.json#L7 here i build static frontend.
https://github.com/dimensi/html2pug/blob/master/now.json#L8 here i start special version of server for now.


Pubsub message encoding performance could be improved?

See: #2773
I'm not sure on the data size limits for pubsub, but if they're significantly large then there are probably more performant message encoding strategies that could be implemented.

Per https://github.com/GoogleCloudPlatform/google-cloud-python/pull/2773#discussion_r89964329 and https://cloud.google.com/pubsub/quotas
Publish request	Total size must be no more than 10MB .
Must contain no more than 1,000 messages.
Message	The maximum number of attributes per message is 100.
The maximum size of an attribute key is 256 bytes.
The maximum size of an attribute value is 1024 bytes.

Batching code is an elegant solution and is already in master and latest release


[Task] add sdk grpc examples (#3925)

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4248 (7323a2f) into master (14bc2bd) will decrease coverage by 0.36%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4248      +/-   ##
============================================
- Coverage     69.58%   69.21%   -0.37%
+ Complexity     7362     7324      -38
============================================
  Files           992      992
  Lines         27885    27876       -9
  Branches       2473     2470       -3
============================================
- Hits          19403    19295     -108
- Misses         7003     7110     +107
+ Partials       1479     1471       -8

Impacted Files
Coverage Δ

...n/alibaba/dubbo/cache/AlibabaDubboConfigCache.java
47.12% <0.00%> (-31.04%)
:arrow_down:

.../client/server/consul/ShenyuConsulConfigWatch.java
60.41% <0.00%> (-25.00%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
70.40% <0.00%> (-19.39%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
68.96% <0.00%> (-13.80%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
35.71% <0.00%> (-10.72%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
75.00% <0.00%> (-6.25%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

...nyu/plugin/motan/cache/ApplicationConfigCache.java
52.00% <0.00%> (-5.00%)
:arrow_down:

... and 37 more

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


fix:  S3 GET presigned urls

What kind of change does this PR introduce?
Bug fix
What is the new behavior?

Refactored SignatureV4 class to make it more readable

When signing keys with the AWS CLI for fetching an object, the payloadHash is UNSIGNED.
This PR will honour this behaviour

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

122 of 158 (77.22%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 80.011%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/storage/protocols/s3/signature-v4.ts
122
158
77.22%

Files with Coverage Reduction
New Missed Lines
%

src/storage/protocols/s3/signature-v4.ts
3
81.84%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
12623

Relevant Lines:
15631

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

122 of 158 (77.22%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 80.011%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/storage/protocols/s3/signature-v4.ts
122
158
77.22%

Files with Coverage Reduction
New Missed Lines
%

src/storage/protocols/s3/signature-v4.ts
3
81.84%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
12623

Relevant Lines:
15631

💛  - Coveralls


How to compile React Component with less or css?

How to compile React Component with less or css? Our UI Library in private npm registry just compiled to es5 and we didn't  compile less(We compiled less in our business project with webpack).

You can modify your bundler config and put less, sass or css modules at rules... I didn't have much time, but my idea is create something like docz-plugin-less, docz-plugin-sass, docz-plugin-cssmodules in the near future...
But if you want to modify your bundler config, just use the modifyBundlerConfig on doczrc.js:
// doczrc.js
export default {
  modifyBundlerConfig: (config) => {
    /* put your custom loaders logic here */
    return config
  }
}

@pedronauck Can you show me a loader example? I can't find it in doc. Thanks!

@YardWill
You need to put the loader of sass or less.
Here is an example using sass:
// doczrc.js

export default {
  modifyBundlerConfig: (config) => {
    config.module.rules.push({
      test: /\.scss$/,
      use: ["style-loader", "css-loader", "sass-loader"]
    })

    return config
  }
}

Read more here: https://github.com/webpack-contrib/sass-loader


Variable viewer: Button missing hover feedback

Testing #10170
High contrast:

It also should have feedback on regular themes.

Will be fixed with a built-in variables viewer https://github.com/microsoft/vscode/issues/165445


RCTUIManager.measure could not be processed.

When implementing this component in my Android app this error comes up.
I have done the same steps as in the example;
Using ReactNative 0.17.0.
code:

I write in render() function ,it work.but,in renderScene (route,navigator) function ,it isn't work.please help me.

see my answer here
https://github.com/alinz/react-native-dropdown/issues/10
@amibition521


Atom.Object.defineProperty.get is deprecated.

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/Applications/Atom.app/Contents/Resources/app/src/atom.js:54:11)
ScriptView.initialize (/Users/Daniel/.atom/packages/script/lib/script-view.coffee:26:9)

Fixed by #407, new release pending.


2.0.3 is missing preview in exports

The error I'm getting:

Missing "./dist/preview.js" specifier in "@alexgorbatchev/storybook-addon-localstorage" package

Could you please describe your setup? We are using this package in a production environment with SB8 w/o any issues.

Steps to reproduce:

Create and go to a new directory
Install Storybook:

npx storybook@latest init

Install addon

npm i --save-dev @alexgorbatchev/storybook-addon-localstorage

Add @alexgorbatchev/storybook-addon-localstorage to list of addons in .storybook/main.ts
Run storybook with npm run storybook

oskar@Oskars-MacBook-Pro ~/g/t/sb (main)> npm run storybook

><EMAIL_ADDRESS>storybook
> storybook dev -p 6006

@storybook/core v8.3.0

(node:37482) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
info => Starting manager..
info => Starting preview..
info Using tsconfig paths for react-docgen
╭─────────────────────────────────────────────────────╮
│                                                     │
│   Storybook 8.3.0 for react-vite started            │
│   197 ms for manager and 827 ms for preview         │
│                                                     │
│    Local:            http://localhost:6006/         │
│    On your network:  http://<IP_ADDRESS>:6006/    │
│                                                     │
╰─────────────────────────────────────────────────────╯
11:33:18 AM [vite] Pre-transform error: Missing "./dist/preview.js" specifier in "@alexgorbatchev/storybook-addon-localstorage" package
11:33:18 AM [vite] Internal server error: Missing "./dist/preview.js" specifier in "@alexgorbatchev/storybook-addon-localstorage" package
  Plugin: vite:import-analysis
  File<EMAIL_ADDRESS>      at e (file:///Users/oskar/git/testproject/sb/node_modules/vite/dist/node/chunks/dep-CUvs3bbV.js:45978:25)
      at n (file:///Users/oskar/git/testproject/sb/node_modules/vite/dist/node/chunks/dep-CUvs3bbV.js:45978:627)
      at o (file:///Users/oskar/git/testproject/sb/node_modules/vite/dist/node/chunks/dep-CUvs3bbV.js:45978:1297)
      ...

@alexgorbatchev Were you able to reproduce this?

I am also seeing this issue

=> Failed to build the preview
Error: [commonjs--resolver] Missing "./dist/preview.js" specifier in "@alexgorbatchev/storybook-addon-localstorage" package

I was able to reproduce the problem, but truth be told I'm not sure what the cause is. There IS ./dist/preview.js file in module folder. It's unclear where this is coming from because there are no references to preview in the dist folder of the module. There has been a number of vite related issues (https://github.com/storybookjs/storybook/issues/23338 https://github.com/storybookjs/storybook/issues/20348) all of which are currently marked as resolved.
Finally, it's unclear where ./dist/preview.js is being required from, creating this file in project root or in the nnode_modules/vite/dist/node/chunks/dist didn't fix the problem.
I would appreciate a PR or at least some pointers in the direction of the root cause.


Pulsar Multi Cluster on Kubernetes (AWS)

Is your feature request related to a problem? Please describe.
There is no documentation available on how to deploy pulsar on multi cluster on kubernetes.
Describe the solution you'd like
Would love to have a Kubernetes deployment files where we can deploy pulsar across different regions like east and west.
Describe alternatives you've considered
http://pulsar.apache.org/docs/en/deploy-bare-metal-multi-cluster/
It has solution for bare metal but not for kubernetes(AWS)

Closed as stale and no one seems to work on it. Please create a new issue if it's still relevant to the maintained versions.
@michaeljmarshall @MarvinCai does pulsar-helm-chart support this feature now so we can add some docs?


TINKERPOP-1199:Use "MicroMetrics" as the mutator of the TraversalMetrics.

https://issues.apache.org/jira/browse/TINKERPOP-1199
Eliminate iteration-time connection between profiling Metrics.
Move upstream duration calculation to finalization of Traversal.

This has been merged to a local TINKERPOP-1199 branch as more work still needs to be done. Will put up for VOTE via a PR from the local branch after @rjbriody and I get this all squared away.

This is ready to VOTE. The completed branch is actually TINKERPOP-1199 locally (not in @rjbriody's repository anymore). I can't seem to make a PR from that our local branch as doing so just links me to this ticket. Don't know how that effects VOTERs and their testing ? --- just be aware.
In short, profiling and the new sideEffect/memory model of GraphComputer have been reconciled. This was painful as profiling is a very complicated/intrusive system. However, getting it to work with the new memory-model in OLAP is a strong testament to the flexibility of the changes forthcoming in 3.2.0 OLAP. Bob and I added lots more testing as well. Also, there was a severe bug found in ComputerVerificationStrategy that has since been rectified (a profile test exposed it).
CHANGELOG
* Fixed a bug around profiling and nested traversals.

VOTE +1.

Here is the complete branch: https://github.com/apache/incubator-tinkerpop/tree/TINKERPOP-1199

Cool work guys. This makes a lot of sense functionally. It's a nice bonus that you caught that issue in the middle of it.
VOTE +1

Looks good, VOTE +1


Fix pr edit reviewers for new bitbucket

Based on the branch from #50
Reviewer groups do not pop up for me in new bitbucket because the plugin is listening for an old, deprecated dialog event that bitbucket (at least the instance we're running) no longer seems to use.
Old dialog: https://docs.atlassian.com/aui/7.8.0/docs/dialog.html?_ga=2.162510652.1878890867.1526064900-1734121638.1523970510
New dialog (and events): https://docs.atlassian.com/aui/7.8.0/docs/dialog2.html
This will allow reviewer button show on when attempting to edit an existing PR on bitbucket using their newer dialog.

Updated the PR.  Initially I replaced the code to only listen for the new dialog event.  It later occurred to me that this would break for people who might still be on an older version of stash that is still using the old dialog.  So I've refactored it to listen to both events, for backwards compatibility.

Somebody merge this...the plugin is not usable in the newer version without this

Thanks!


chore: Use java.util.Date as a DATE_TIME representative class

Fixes: #312

java.util.Date doesn't recognize timezone. need rework.