Pretalx support

This is now based upon the FOSDEM 2024 schema, and correctly parses the production conference data. I chose in the end to use the public JSON schedule format combined with a handful of calls to the API to populate the speaker information. This now mostly works, and is missing:

[ ] More thorough testing that the output conference is exactly right.
[ ] Plug gaps that were covered by the pentabarf DB. Currently I believe this is only QA names and Matrix IDs.
[ ] Write a e2e test.

This seems pretty sane to me - happy to approve as-is to keep things moving or should I wait until the checklist is complete - those could possibly be done in follow-up PRs?


Workspace - Bank account personal info isn't accepting characters with umlaut as name

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

Version Number: 1.4.85-0
Reproducible in staging?: Y
Reproducible in production?:  Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Log in with a new expensifail account
Create a workspace
Enable Workflows
Navigate to workspace settings - Workflows
Click on "Connect bank account"
Choose the manual option
Input "011401533" for Routing Number
Input "1111222233331111" for Account Number
Input "Ádám" for first name
Input "Horváth for last name

Expected Result:
Characters with umlaut should be accepted
Actual Result:
Bank account personal info isn't accepting characters with umlaut as name. Please enter a valid name error appears. These characters are accepted in the Save the world - teacher and principal flows
Workaround:
Unknown
Platforms:

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/1962573b-b68d-4a66-9efd-b0db7326e78a
View all open jobs on GitHub

@JmillsExpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Opening up to community

Payment summary:

Contributor: @eucool $250
Reviewer: @suneox $250

All contributors paid via Upwork and checklist complete. Closing this one!


⚠ Jenkins Plugins Artifactory has degraded performance

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

HTTP code: 200
Response time: 1152 ms

Resolved: Jenkins Plugins Artifactory performance has improved in e1745a6 after 20 minutes.


The page is wrong MSBuild  is not in the latest dotnet it was taken out in 2022

[Enter feedback here]

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

ID: 911666fa-55ac-cbc1-ce79-66c058859ad9
Version Independent ID: 571eec5d-8fb9-2401-fc4c-8a7896723eb7
Content: MSBuild - MSBuild
Content Source: docs/msbuild/msbuild.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@owenn
MSBuild should still be installed when you install the .NET SDK and it should be invoked when you use dotnet build. If you're seeing something different, let me know the details of what you installed and how that differed from your expectations.


Auto update when hplans CSV updates

Auto pull in changes

I've already configured a CRON type instance on the spreadsheet that updates from the hplans source data every 24hrs, so the tool is kept up to date!

@shawaj I've already configured a CRON type instance on the spreadsheet that updates from the hplans source data every 24hrs via the Apps Scripts extension, so the tool is kept up to date!

Amazing 👍👍👍


Proxy Chaining (External Proxy) Not working

Setup:

Upstream Titanium Proxy (IP =<IP_ADDRESS>, Port = 5500)
Intermediate Titanium Proxy (IP: <IP_ADDRESS>, Port = 5501) with Upstream Http/Https specified.
HttpClient with WebRequestHandler set to use Intermediate proxy doing a HTTPS request.

Expected Result:
HTTPS request goes through Intermediate proxy and then Upstream proxy and completes successfully.
Actual Result:
HTTPS request fails with "underlying connection was closed".
I have built an example console app that demonstrates the issue.
I'm not sure if I need to do some other configuration, or if this is a bug.
static void Main()
        {
            var upstreamProxy = new ProxyServer();
            upstreamProxy.AddEndPoint(new ExplicitProxyEndPoint(IPAddress.Any, 5500, true));
            upstreamProxy.ServerCertificateValidationCallback += (sender, args) =>
            {
                Console.WriteLine("Upstream Proxy Validate Server Certificate");
                args.IsValid = true;
                return Task.CompletedTask;
            };
            upstreamProxy.BeforeRequest += (s, e) =>
            {
                Console.WriteLine("Upstream Proxy Before Request");
                return Task.CompletedTask;
            };
            upstreamProxy.BeforeResponse += (s, e) =>
            {
                Console.WriteLine("Upstream Proxy Before Response");
                return Task.CompletedTask;
            };
            upstreamProxy.Start();
            var intermediateProxy = new ProxyServer();
            intermediateProxy.AddEndPoint(new ExplicitProxyEndPoint(IPAddress.Any, 5501, true));
            intermediateProxy.UpStreamHttpProxy = new ExternalProxy { HostName = "<IP_ADDRESS>", Port = 5500 };
            intermediateProxy.UpStreamHttpsProxy = new ExternalProxy { HostName = "<IP_ADDRESS>", Port = 5500 };
            intermediateProxy.ServerCertificateValidationCallback += (sender, args) =>
            {
                Console.WriteLine("Intermediate Proxy Validate Server Certificate");
                args.IsValid = true;
                return Task.CompletedTask;
            };
            intermediateProxy.BeforeRequest += (s, e) =>
            {
                Console.WriteLine("Intermediate Proxy Before Request");
                return Task.CompletedTask;
            };
            intermediateProxy.BeforeResponse += (s, e) =>
            {
                Console.WriteLine("Intermediate Proxy Before Response");
                return Task.CompletedTask;
            };
            intermediateProxy.Start();

            PutAsync().GetAwaiter().GetResult();
            Console.ReadKey();

            intermediateProxy.Stop();
            upstreamProxy.Stop();
        }

        static async Task PutAsync()
        {
            try
            {
                var handler = new WebRequestHandler
                {
                    Proxy = new WebProxy("http://<IP_ADDRESS>:5501"),// <-- Fails if we use the intermediate proxy, but works if we use the upstream proxy
                    UseProxy = true,
                    ServerCertificateValidationCallback = ValidateServerCertificate
                };
                var client = new HttpClient(handler);
                var response = await client.PutAsync("https://httpbin.org/put", new StringContent("hello"));
                if (response.IsSuccessStatusCode)
                {
                    Console.WriteLine("success");
                }
            }
            catch (Exception ex)
            {
                Console.WriteLine("failed: {0}", ex.Message);
                Exception inner = ex.InnerException;
                while (inner != null)
                {
                    Console.WriteLine("--------");
                    Console.WriteLine(inner.Message);
                    inner = inner.InnerException;
                }
            }
        }

        static bool ValidateServerCertificate(object sender, X509Certificate certificate, X509Chain chain, SslPolicyErrors sslPolicyErrors)
        {
            return true;
        }

Attached Visual Studio Test Project:
Test.zip

Any ideas what might be the problem?

Should be fixed in latest develop branch. Thx


Creation of user service

Please create a basic user service for the application. It will use spring boot and it should involve endpoints to create a user, delete the user, and get user details. It will use PostgreSQL to store the user information.
Also, configure the application spring.jpa.hibernate.ddl-auto=update so the table can be automatically created and updated.
The table structure should be the same as given in the documentation.
Please create BaseEntity which is extended by the User Entity.
BaseEntity should have the following fields:

Id (UUID)
createdAt (LocalDateTime)
updatedAt (LocalDateTime)
createdBy (String)
updatedBy (String)
status (Enum [ACTIVE, INACTIVE and DELETED)

Please refer to the schema here:
https://whimsical.com/mithiril-chatapp-dbms-ByzAEQh5SYR16i5gsYdcg8
Please check the documentation for more details, and feel free to ask anything.
Note: You don't need to add any security.

Hi @Satwik-Bhardwaj, I'm intersterested in working on this issue. Could you please assign it to me?

Hi @Satwik-Bhardwaj, I'm intersterested in working on this issue. Could you please assign it to me?

@Naresh-Kushwaha I'm happy to hear that you are interested in my project and want to make some contribution. I am assigning this issue to you. Please proceed. I hope you will code it properly. However, I am updating the description of this issue to learn more about it.

Hi @Satwik-Bhardwaj , Please tell me from where should I learn Spring framework.any course?


GR switched to BAG as data source

https://www.gr.ch/DE/institutionen/verwaltung/djsg/ga/coronavirus/info/aktuell/Seiten/fallzahlen-erklaehrung.aspx
Decision: We are continuing to take the data published on the Cantonal website until Monday (there are no data being published Sat/Sun anyway). The switch of GR makes sense; it is likely that other Cantons will follow their path. By Monday we will decide, if we are adding another column to our data structure, specifying the 'data source' and/or are taking other measures.

The data on the website was updated on Monday and Today (2020-04-08). So they still continue things.
I am not sure what they mean by 'BAG as a data source'. I guess they refer to the xlsx file updated daily offered by BAG. There is also a new tableau based visualisation from BAG that provides full breakdown by cantons, age and sex, but it is not fully available by mechanical means AFAIK (there is a csv dump of data there, but the link changes every day).

@baryluk Exactly

The entire data published by GR at https://www.gr.ch/DE/institutionen/verwaltung/djsg/ga/coronavirus/info/Seiten/Start.aspx is taken out of the ISM of the BAG, as can be seen at https://github.com/openZH/covid19_sources_caseNumbers/blob/master/COVID19_Quellen_CH_total_v2.csv. This does not mean that they discontinue publishing data, however, they seem to have exchanged the source at some point.


Misc code cleanup and JavaDoc, delete unused action

I've separately been thinking about doing a bulk refactor to add @Override annotations, since we seem to be added them on a file-by-file basis


Drop some RC4 cipher suites.

Firefox dropped these recently. I'm following their lead.
See also:
https://docs.google.com/spreadsheets/d/1C3FdZSlCBq_-qrVwG1KDIzNIB3Hyg_rKAcgmSzOsHyQ/edit#gid=0

Lgtm

@kruton Thumbs up?

When this is done, I'm going to consider our cipher suites up-to-date for another 6 months.
https://github.com/square/okhttp/issues/1084

Dropping RC4-SHA might cause some pain, but not for me :)

It's easy enough to add-it-back in applications that need it.


Configs for greenhouse

Can these values be edited via the config file?
The crops grow too quickly
public static final float GROWTH_FACTOR = 1f / (16 * ICalendar.TICKS_IN_DAY); // 24 -> 16 days
public static final float NUTRIENT_CONSUMPTION = 1f / (8 * ICalendar.TICKS_IN_DAY); //  12 -> 8 days
public static final float WATER_CONSUMPTION = 1f / (12 * ICalendar.TICKS_IN_DAY); // 12 days
public static final float NUTRIENT_GROWTH_FACTOR = 0.5f;

done


Each Consumer API call returns 401 (Unauthorized)

Readiness checklist

[ ] I added/updated unit tests.
[ ] I added/updated integration tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

I have narrowed the problem down to the Microsoft.IdentityModel.Tokens package. After upgrading to a version >6.27.0 the error occurs.

In order to avoid accidental updates to the corrupted version, I have pinned it to 6.27.0. We need to check for new versions of this package regularly. I registered for notifications on the corresponding repository (https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet).


Apache license author

Copyright of the Apache license is directed towards the individual 'Peter Karich' and we move this here to the company 'GraphHopper GmbH'. New (and old) contributors agree(d) to the new CLA
This effects all GraphHopper projects including jsprit (see the required license change issue here) and map-matching etc.

Two first words of title were a bit scary until the full message is read! :smile:

Ups & sorry, will rephrase!!

This is done here: https://github.com/graphhopper/graphhopper/commit/c6c18ce28ae1722bfb6338d517ca3093e205d1ac


Application not working fine in macOS

There are few button issues

@skimran2000 I'll review your PR soon.

okay

btw when i am trying to pull changes from upstream repo, I am getting an error:
command--> git pull upstream master
error-->          fatal: couldn't find remote ref master

Yaa try it.
Or google it this is a common issue.

I found another bug related to tax calculation. When annual income is less than deductions then also it is calculating the tax which is wrong because annual income should always be greater than deductions.
Once look into it if there is really a bug then i would like to raise an issue

Yes open this issue.

What should the title of issue be?

This will not work.
Make a separate file for macOS users. And add it.
I will merge it.

@skimran2000 any updates?


Extended Regex Options

This PR implements extended regex options as described in #753.

Flag
Behavior

g
replace all occurrences of match instead of only first one (regexp_replace only)

c
case-sensitive matching (default)

i
case-insensitive matching

m, n, p
dot does not match newline

s
dot matches newline (default)

Example:
-- global replace
SELECT regexp_replace('ana ANA ana', 'ana', 'banana', 'g');
-- banana ANA banana

-- case insensitive and global replace
SELECT regexp_replace('ana ANA ana', 'ana', 'banana', 'gi');
-- banana banana banana

This looks fantastic, thank you! I'm only a user on the Python side of things, so as soon as this lands in PIP, I can certainly test it out!

It should land on pip with the next release :) I hope it solves your issue!


Dev builds in Angular 9 are very slow

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
Yes, not present in 8.2.
Description
Dev builds in Angular 9 are very slow. Builds of 8.2 took on average 30 seconds or so, and recompiles took 2-5 seconds. This same app in Angular 9, however, takes ~45-60 seconds to build initially, and recompiles take ~15-25 seconds even if nothing changes at all. Production builds are roughly 5-10 seconds faster in 9, so those are better at least. (Note that none of these times include ngcc, that's taken care of before I attempt to build.) Additionally, the speed is slow regardless of whether Ivy is used or not.
The Angular 9 build always seems to freeze for a bit at a certain point. It frequently lists the same file, but it's not always the same file that it seems to get stuck on. There's nothing special about the file it usually lists, it's actually quite small and has no advanced Angular or TypeScript features.
I found #34699 (and #33532 by extension), but those are unrelated. I'm using TS 3.8 and it happens regardless of the value of that option in tsconfig.json.
Is there a way I can audit what's causing it to take so long? I tried ng build --watch --verbose, but it provides way more information than I can feasibly parse (and also doesn't include timings from what I can tell).
🔬 Minimal Reproduction
I'm running ng build --watch in both Angular 8 and 9. This is a work app so I can't share it, and I can't provide a repro because I don't know what's causing it to happen in the first place.
🔥 Exception or Error
N/A
🌍 Your Environment
Angular CLI: 9.1.0
Node: 13.12.0
OS: win32 x64

Angular: 9.1.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.901.0
@angular-devkit/build-angular     0.901.0
@angular-devkit/build-optimizer   0.901.0
@angular-devkit/build-webpack     0.901.0
@angular-devkit/core              9.1.0
@angular-devkit/schematics        9.1.0
@ngtools/webpack                  9.1.0
@schematics/angular               9.1.0
@schematics/update                0.901.0
rxjs                              6.5.5
typescript                        3.8.3
webpack                           4.42.0

Anything else relevant?
N/A

Same problem. Angular cli  is extremely slow in version 9.0.1.
Command: ng serve --host <IP_ADDRESS>
Step1, the cli prints:
WARNING: This is a simple server for use in testing or debugging Angular applications
locally. It hasn't been reviewed for security issues.

Binding this server to an open connection can result in compromising your application or
computer. Using a different host than the one passed to the "--host" flag might result in
websocket connection issues. You might need to use "--disableHostCheck" if that's the
case.
0% compiling

This step take at least 4 minutes in my Macbook PRO 13 (2017)
Step2: the percent starts to increase. This step takes 1 minute.
Total time 5 minutes. It's insane.

upgrade to angular 9.1 will help so much sir. It's will faster alot @vaindil  @Serginho  https://blog.angular.io/version-9-1-of-angular-now-available-typescript-3-8-faster-builds-and-more-eb292f989428?source=collection_home---4------0-----------------------

I appreciate the feedback, but I am already on 9.1. The problem remains.

@vaindil, would you be able to share the application privately? unfortunately, it will be hard to track down the issue without a reproduction.
Also can you use try Node 12 instead of 13?

upgrade to angular 9.1 will help so much sir. It's will faster alot @vaindil @Serginho https://blog.angular.io/version-9-1-of-angular-now-available-typescript-3-8-faster-builds-and-more-eb292f989428

I'm in that version as well

@Serginho, your problem should be revolved by https://github.com/angular/angular/pull/36486#issuecomment-611482640
Note: that from Angular 9.1 there is no longer recommended to run ngcc as a post install hook.

@alan-agius4 I checked, I'm not allowed to share the source even privately. Is there anything I can do to try to debug it on my own? If not, I understand if the issue has to be closed.

You can use the NG_BUILD_PROFILING=1 environment variable which outputs profile events for Chrome profiler and a JSON file with plugins stats.

I tried setting that environment variable. It creates a new empty file at the project root called chrome-profiler-events-es2015.json, but the actual build process gets up to 71% chunk graph and never gets beyond it. I left the CLI sitting here for 15 minutes, no change.

Looks like the problem comes down to AOT builds being turned on for all builds by default with Angular 9. Disabling it for dev builds brings times roughly back to where they were before (just slightly slower, but acceptably so). I would normally expect this behavior, but the docs are written in such a way as to imply that AOT builds are faster than JIT builds with Ivy, which is clearly not the case (since it nearly doubles the build time).
Looks like the solution therefore is to disable AOT for dev builds. If that's the answer, feel free to close this issue. I'll leave this open though in case this is a bug.

@alan-agius4 Sorry for the ping, I just see that this issue is still open. Not sure if it should be open based on my previous two comments or if it should be closed.

In version 9, AOT and JIT compilations time should be similar.
Unfortunately, since you cannot provide a reproduction not even privately. I don’t see how we can investigate this any further.

@alan-agius4 I understand, thanks. Do you know what could be going wrong with the build profiling? I'd like to continue to investigate locally, but when I have NG_BUILD_PROFILING=1 set the build just freezes at 71% chunk graph.

Mine also get's stuck on the 71% chunk graph

@chaseWillden I was able to fix that problem by setting giving Node more RAM with the environment variable NODE_OPTIONS=--max_old_space_size=16000 (replace with whatever number in MiB), then I just had to let it sit for a while. The output file ended up being 600 MB though and Chrome chokes trying to open it.
I found this guide for how to profile, instructions still seem to be valid. Unfortunately in my case there's no obvious thing that the build is hanging on, so it's been rather difficult to figure out what's making it take so long.
Just a note from that: To start the profile, I'm clicking the Continue button when it initially breaks, then immediately switching to the Profiler tab and starting the profile with the initial build. Builds after the initial build triggered by console.log(1) in main.ts like that issue says to do don't always have the problem.

@alan-agius4 I was able to find a project that exhibits the same behavior. Could this issue be reopened?
Here's the repo. Instructions are in the readme, but the short version is that an AOT build of the extra branch takes ~81s, whereas a JIT build takes ~32s.

I see this issue too, and also while you are watching for changes, recompiles take much longer with aot then with jit

As @alan-agius4 said me, my issue is solved. Thank you everyone.

Just wanted to chime in here to say that my ng build --watch builds went from approx 1.5-2 sec in 8.2 to between 6 and 9 seconds in 9.1.  It's very frustrating!

Having this issue as well across all our projects (which I can't share)
As a hint for reproduction our projects are a bit odd (currently). We are currently disting source projects as opposed to compiled angular libraries. (we are in the process of converting to angular libraries now that they seem to work).
Our angular app that we are compiling actually includes say 5 other angular projects that are in node_modules as ts files not compiled js files.
I think this might be why it is so slow and I hope that as we convert to angular libraries this will get faster. Currently though the compile times are unworkable.


Would be cool to have an Auth extension [Feature request]

Is your feature request related to a problem? Please describe.
I'm using the paywall extension and its APIs which are amazing! Thanks for the incredible project. I was just thinking it would be nice if there was also an extension to easily do lightning login / auth for devs
Describe the solution you'd like
Auth extension
Describe alternatives you've considered
right now i'm running a separate server to do this using lnurl-passport-react

Auth logic now on main branch


Ports are displayed twice using Docker 20.10.6

Bug description
With the latest update to Docker (20.10.6), some network and port regressions were fixed.
Apparantly, this has caused Portainer to display published ports twice for all containers (which takes a huge amount of screen space if there are many published ports - see screenshot). I guess ports are now explicitly displayed for both IPv4 and IPv6.

Expected behavior
Expected behavior is that duplicate ports are only displayed once - alternatively, that you can choose if you want to display IPv4 ports, IPv6 ports or both.
Portainer Logs
Not relevant.
Steps to reproduce the issue:

Install Docker 20.10.6
Install Portainer
Install any container that maps a port
See the list of containers, and see that each port is displayed twice

Technical details:

Portainer version: 2.1.1
Docker version (managed by Portainer): 20.10.6
Platform (windows/linux): Linux - Ubuntu 20.04.2
Browser: Firefox / Chrome
Use Case (delete as appropriate): Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes

Additional context
This is mostly a cosmetic issue, but nonetheless I assume it can be annoying that ports suddenly take up double screen space.

You are correct that those ports are showing for both IPv4 and IPv6.
That's because as of Docker 20.10.6 it is returning both:
https://github.com/moby/moby/issues/42313
From their release notes:

Fix implicit IPv6 port-mappings not included in API response

Ok, maybe this isn't a bug report then, but a "feature request" - that you in the Portainer UI can choose only to show ports once?

@ameinild, we have a preview version available. Can you give it a try by using the image portainerci/portainer:pr5239
Please let us know how that's working for you.
Note that this is a development build and should not be used in a production environment.

Thanks, I'll give it a spin!

Thanks for confirming the fix!
And please follow the documentation in order to use the Agent: https://documentation.portainer.io/v2.0/deploy/ceinstalldocker/

Any idea when this will be merged in the official release?

Should be there by October.


[Feature]: Add a salting feature allowing admins to generate a key and use it to encrypt and decrypt data in database

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
I want to start by quickly applauding the devs on this project. Absolutely game changing for my organization and so incredibly well designed.
For storing user data in apps, it would be great if we could have a feature that empowers Appsmith to handle the encryption and decryption of data stored in the database. 'Salting', I believe is the term - for added security in case the database is breached. It could be a checkbox labelled "Encrypted" in the widget settings that encrypts the input of a specific widget before sending to the database and decrypts the output of a specific widget after retrieving from the database. Keys could be encrypted and stored alongside other datastore credentials.
I am not a coder so forgive me if there is a better way to achieve this. Would be very helpful for my use-case.
Why should this be worked on?
This could be helpful in enabling admins to keep user data more secure and private, enabling organizations to use the software for broader purposes.

@Hyperei Glad you're enjoying the project! You can configure APPSMITH_ENCRYPTION_SALT in your Appsmith instance to encrypt the data!


Use right version of react-hook-form for ie11 bundle

👋🏻
Related to https://github.com/react-hook-form/resolvers/issues/64
Use rollup instead of tsc to bundle ie11.
paths allow to rewrite import in order to use the right version of react-hook-form and then fix the issue.
paths: {
  'react-hook-form': 'react-hook-form/dist/index.ie11',
},

Thanks, heaps @jorisre 1.0.1-beta.1

@c0d3x did that PR fix your issue ?

@c0d3x did that PR fix your issue ?

for the strictly-typed I did not have a version to test, but the one for resolver 1.0.1-beta.1 worked.

@bluebill1049 we could merge this PR, isn't it ?

yes sorry for the delay. @jorisre and thanks very much for your work. 🙏


close #262 by using react-snap to prerender pages

This pull request attempts to close #262.
At the time of this writing, it adds minimal CSS needed for the page to render by adding
"reactSnap": {
    "inlineCss": true
},

to package.json, adds the react-snap dependency and postbuild script for it in package.json, and updates src/index.js to attach event listeners to already created HTML if react-snap already pre-rendered HTML contents or rendered them on the server.

error at / TypeError: Cannot read property 'ok' of null 99     at isOk (/home/runner/work/tide/tide/node_modules/minimalcss/src/run.js:13:35)
This is the message that occurs on the failed run that's not Vercel.
I researched it a bit and the owner of react-snap says "This is not an error of minimalcss. This is an issue in react-snap (there is no try-catch around minimalcss https://github.com/stereobooster/react-snap/pull/263/files#diff-168726dbe96b3ce427e7fedce31bb0bcR261). Nothing to fix in minimalcss" on an issue page open in minimalcss for the same error message we received.
A more recent issue with the same error message says, "Disabling the website's service worker works!"
Is that something we want to do?


services: support verbose mode on healthchecks

NOTE
I know the passing of kwargs is not optimal (it sort of breaks the purpose when passing-extracting). A bigger refactor can be made later. This change is blocking debugging the gh actions.

Cannot cancel travis for some reason. Temporarily closing PR


feat: subscribe / handle subscription page

Settings page

Subscription (buy / manage)
Settings (just a sign-out button)

Auth flow / pages

Coherent Card setup for all "redirect" pages
See demo: https://share.cleanshot.com/CxIvnq

Fix payment race condition bug

sometimes entries had not yet been inserted before other's referenced it

https://user-images.githubusercontent.com/51714798/203587465-2865ad01-c4a6-4e9e-b848-f5329fa5443e.mp4

Tidied up the auth pages/flow too:
https://share.cleanshot.com/CxIvnq

The click on Upgrade in the extension (on the paywall tooltips) doesn't do anything for me.

I'd guess this is your stripe configuration not letting you create a customer? Do you get an error from the trpc logger link?

Payments

[ ] Upgrade button should redirect to Subscription Page
[ ] Subscription Page should look different for PRO users
[ ] Stripe session should not display a trial
[ ] Trial page should move to post sign in flow

Current demo: https://share.cleanshot.com/7FZ50w

Didn't really understand the SharedRoutes thing and left a couple of other questions.

SharedRoutes is just so that we get type-errors if we're trying to navigate to a page we haven't defined - so basically every time the calendar wants to navigate somewhere, we type what routes it can do so with. This will help making sure we don't do something like onPageChange("settings/subscriptions") when the path is really /settings/subscription.
The extension has type safe routing internally - but these navigations are done from the calendar and that doesn't know what routes the extension has on its router. By defining the SharedRoutes type we tell ourselves that OK, all apps consuming the calendar package must define these routes to make sure all navigations work properly

Thanks.
There actually is an issue about creating multiple subscriptions for the user -- which isn't exactly ideal.
This StackOverflow user proposes to:

create the session in mode: setup to collect payment details & nothing more,
set up a resolver for the checkout.session.completed event that's coming in through our webhook, and
update the existing subscription by calling something like:

await stripe.subscriptions.update(
  'sub_1M9YqBEdYMdxQoGeTqqP4NiA',
  {metadata: {...}}
);

Recorded explanation of the routing: https://share.cleanshot.com/fDb71I


Consider using my parser instead

https://github.com/kaitai-io/kaitai_struct_formats/blob/master/hardware/mifare/mifare_classic.ksy
Obviously  it needs compilation,  you can read how to do it in
https://github.com/KOLANICH/MFCTool.py/blob/master/setup.py#L16L31
Please note that kaitaiStructCompile.py is unstable and has unstaboe API, it is likely to be changed in a week.
For example I gonna default
"flags": ["--python-package", "."],

and move them into a separate place in config dict

1 you need to compile the description into python code. You need to install the compiler first,  how to do it on linux you can see in https://github.com/KOLANICH/MFCTool.py/blob/master/.ci/setupKS.sh .
For windows you need to download the nightly msi from appveyor:
pip install --upgrade git+https://github.com/KOLANICH/appveyor-artifacts.git
appveyor-artifacts -n kaitai-struct-compiler -o GrayCat -b master

(of course browser can also be used)
2 git clone https://github.com/kaitai-io/kaitai_struct_formats.git ksf
3 compile the stuff: ksc --target=python ksf/hardware/mifare/mifare_clawsic.ksy
4 ipython, then from mifare_classic import *, then d=MifareClassic.from_file("./test.mfd"), then get you familiar with the objects hierarchy
5 once you have integrated the code into your code, you can put into setup.py the linked snippet and the parser would be freshly compiled from the fresh definition from the repo. But this lib is very unstable, I have significantly improved it in my local repo, but the changes are not yet deady to be published.

Wow, so complicated :)
Maybe it is possible to use it in wed ide?
I've try with no luck. What i'm doing wrong?

WebIDE (even /devel) has an obsolete version, use python shell for now.


remove '.' from command line

I built an application with arguments and options when executed in command line,  like ./node_modules/.bin/electron . email --email_subject="hi there" --email_body="good morning."
After packaging with electron-packager, I get .exe file on windows, and it works well like this:
./app.exe . email --email_subject="hi there" --email_body="good morning."
the question:  is it possible to remove dot('.' ) from command line?   I want app execute like this:
./app.exe email --email_subject="hi there" --email_body="good morning."

The . is not necessary because your app is packaged. It is necessary when you run node_modules/.bin/electron because it needs to know where your app lives.


Alignment Issue in About Section

Describe the bug
In the About section inside cards Images and texts are not properly aligned
Expected behavior
Proper alignment of components inside cards
Screenshots

Example  Fill x { [x] }  into  bracket

[x] I have checked the existing issues *
[x] I have read the Contributing Guidelines *
[x] I have starred the repository *
[x] Working under Gssoc24*

@HimanshuNarware Please could you assign me this issue

hey i can fix this issue please assign it to me

@Prithwi32  assigned U

Pls assign it to me

@Prithwi32 assigned U

@HimanshuNarware I have done PR, please could check once

Hey @HimanshuNarware the issue still exists, Please assign me this issue

After Changes :

@HimanshuNarware gssoc tag is missing, please could add ..


Let property patches be applied last (fixes #338)

Fixes #338.

What's the status of this? Is this going to be merged?


Estimated lift much higher than actual lift in dataset.

Hi folks,
I'm wondering if you might be able to help me out with a puzzling issue that I'm running into using the GeoLift package.
I'm currently designing a test using this package. Based on the results of the GeoLiftMarketSelection, I'm creating a simulated dataset, where I take the effective size provided in the function output and apply it to the test markets for the duration provided by the function output. For example, if the first market selection is for markets 1 & 2, with an effect size of 0.05 for a 15 day test, I apply a lift of 0.05 to my test markets data to simulate this lift.
When I run the GeoLift function on this test data, the reported percent lift for these tests is often much higher than the lift that I simulated. For example, when I simulate a lift of 5%, I might get back a lift of 14.8%. I'm wondering if you might have any ideas as to why this is happening?
I unfortunately can't share my data, but I did notice that when I apply this same procedure to the datasets provided in the GeoLift walkthrough, I don't run into the same problem. In other words, the lift that I simulate is more or less the one that is returned by the GeoLift function.
Here are a few things that I've noticed or that I want to note about the dataset that I'm using:

In the output of the GeoLiftMarketSelection function, the Average_MDE is often higher than the EffectSize. For example, the EffectSize might be 0.04 but the Average_MDE is close to .08.
The abs_lift_in_zero is often higher than 0, ranging somewhere between 0.02 and 0.04.
In the dataset that I'm using, we have a relatively low number of conversions (esp. compared to the sample data provided in the walkthrough). For example, the highest number of conversions per day in any location is less than 200.
We've already moved all the way up the funnel for this test, so we likely won't be able to use an outcome metric that has a higher volume of data.
The dataset that I'm using contains one year worth of data and the test sizes that I'm simulating range from 2 weeks to 8 weeks, so I think we have ample historical data and a fairly long test window.

I'm wondering if you have any ideas about why I might be obtaining these results and whether there is anything you recommend doing to obtain more robust results?
Here are some things I've thought of and I'm wondering if you think they might be valid approaches (but would be curious to know if you have other ideas):

Aggregate DMAs to obtain a higher volume of conversions per day.
Aggregate days into weeks to obtain a higher volume of conversions per time point.

Related to the above, I'm wondering how much cause for concern there should be when

The EffectSize and the Average_MDE diverge substantially in the GeoLiftMarketSelection
When the abs_lift_in_zero is significantly above 0
And as with the above questions, what steps you might recommend for addressing these issues.

Thanks for any help here!

@djolear Hi, would you mind sharing the code that you used to simulate the problem you had but using the the Walkthrough data? While your explanation is clear, the devil lies in the details with code

@djolear I'm afraid I can't reproduce the error that you had with your dataset, as a consequence, I will close this issue but please re-open it if you manage to create a dataset we can use to test out the error.
Regarding your questions
Aggregate DMAs to obtain a higher volume of conversions per day.
What does DMA stand for? From the text I suppose it is 'locations', and if so I don't think it will help out aggregating locations. Grouping locations is equivalent to getting the weighted average of the coefficients when training with the locations separated. The only situation where I can imagine that it would be useful is

if you have a few locations with a lot of conversions/users and many with very few users, so that the regularization in the Augmented Synthetic Control basically zeroes the coefficients for all location but those few that are large
if you have many locations where the number of conversions can be 0 in some of the dates, then grouping locations to ensure that the group never has a date with 0 users. The difference to the previous point is that if a location has 0 events on a date it causes a missing date for the location, which makes it get dropped out when we use GeoReadData

Aggregating dates dates will again just contribute in the same situations as the two explained above
Same answer as above
The EffectSize and the Average_MDE diverge substantially in the GeoLiftMarketSelection
Yes, that shouldn't happen. If this happens, then the algorithm is not performing well for your data, which likely means that some of the assumptions for the correct estimation of your lift aren't satisfied
When the abs_lift_in_zero is significantly above 0
If this happens then again your data is not satisfying at least one of the assumptions necessary when using Augmented Synthetic Control. It means that even when we force a lift of 0, we still detect a lift which 'feels above' 0, which can lead to a false positive situation where we state the existence of an impact by the treatment even when there is none


boost::lexical_cast<static_string_n<>>

This allows conversion for fields without dynamic allocation

NACK, boost::lexical_cast ain't so great! But we have string_param now so this is taken care of.


GetSizedFormat breaks with type=GL_UNSIGNED_SHORT_5_6_5 (GLES)

glTexImage2D was called with: {internalformat,format}=GL_RGB, type=GL_UNSIGNED_SHORT_5_6_5
GetSizedFormat then converted internalformat to GL_RGB8 which was saved to the capture.
This sized type however is invalid with GL_UNSIGNED_SHORT_5_6_5, thereby breaking replay. See table 1 in https://khronos.org/registry/OpenGL-Refpages/es3/html/glTexImage2D.xhtml
Im not sure if this is correct, but seems reasonable as a fix:
GLenum GetSizedFormat(const GLHookSet &gl, GLenum target, GLenum internalFormat, GLenum type)
{
  switch(type)
  {
    // types can imply a sized format
    case eGL_UNSIGNED_SHORT_5_6_5: return eGL_RGB565;
    default: break;
  }

That sounds like a reasonable approach. Like you say it's not valid to use any other internal format in that case, so a PR for that change would be fine. It would probably be a good idea while you're there to handle the other oddball formats like GL_UNSIGNED_SHORT_4_4_4_4 to GL_RGBA4 and GL_UNSIGNED_SHORT_5_5_5_1 to GL_RGB5_A1.

Done. Ive added cases for all the unsized internal formats from Table 1, which are those cases that you mentioned.

Merged the PR so I'm closing this.


Enable sidekick publish on files in .helix/

As part of the content-bus config merge users will need to be able to publish the .helix/headers.json file.
Files contained in .helix/ will continue to be blocked from being served via the pipeline but the preview/publish operations in admin will trigger a merge of all config files into .helix/confilg-all.json and /helix-env.json. Where the latter does not contain any sensitive data from .helix/config.json and can be used by other helix systems (besides pipeline) to extract the production host.

So basically, only .helix/config.json should be treated as a special case?
cc @tripodsan

So basically, only .helix/config.json should be treated as a special case?
cc @tripodsan

for now, I would just disable publish on all /.helix. we can still open it up if needed.

OK, that's the current status. Then I don't understand this enhancement request. 😄

right, I forgot that there is no publish button on the edit urls. closing as won't fix.

Sorry, somehow missed this convo..
.helix/config-all.json  is unique to each partition as per.
https://github.com/adobe/helix-html-pipeline/blob/main/src/steps/fetch-config-all.js#L103
If we are not going to enable publish we should do something similar to what is done with preview operations on .helix/config where somehow (haven't looked) a preview results in .helix/config-all.json in the live partition being updated.
Do you agree @tripodsan?

I think /.helix/headers.json should also not need a publish.


Bot testing: Support testing return values & multiple responses

This takes ideas originally in #5502, splitting out the additional features into this PR to allow discussion of these separate from any helper function changes (check_expected_responses).
This PR adds support for testing:

messages which result in multiple responses from a bot;
messages/responses which depend upon the return value of send_* functions.

API comments:

function signature for assert_bot_response changes, with response and expected_method replaced by response_list and result; instead of a single response and method, response_list is a nested list of them. result is the default return value of the send_* functions, with a default success value, and can be overridden by use of a third entry in each element of response_list.
check_expected_responses identical, but adjusted to use new assert_bot_response (does not support new features)

Generally the error reporting should be an improvement, ie. a failure outputs the message, etc. causing the problem.
test_followup.py is updated to demonstrate the new call signatures (and also not using check_expected_responses @showell ). I have a simple bot which would test multiple responses, and pollbot would test that and return values.
To do/discuss:

FIXME: This concerns that multiple calls to send_* must match the return value order, but any_order would make testing simpler, and is applicable if the return values are all identical?
If the list-of-lists approach is too loosely specified, a list of namedtuples may be an option?
Other bot tests need updating (beyond just followup), though those calling check_expected_responses appear to still pass.
Would message/response templates in the bot testing or even bot lib be useful? These libraries are there to make bot writing easier, so referring to the API docs shouldn't be necessary?
Maybe result -> default_result renaming?
MyPy types could be improved?
Expand check_expected_responses to support these features, or drop it? (later PR?)

Automated message from Dropbox CLA bot
@neiljp, it looks like you've already signed the Dropbox CLA. Thanks!

Codecov Report

Merging #5587 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5587   +/-   ##
=======================================
  Coverage   87.46%   87.46%
=======================================
  Files         434      434
  Lines       54313    54313
  Branches     2098     2098
=======================================
  Hits        47506    47506
  Misses       6807     6807

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 5c9f029...d5a2c1f. Read the comment docs.

I've removed the messy multi-list style involving [[response, method]]; the revised assert_bot_response now takes the message and a a variable number of tuples, one per response expected, ie. the minimal call is now:
assert_bot_response(message, (response, method))

for a message (in dict form) triggering a response via a send_reply or send_message method, with response being text or a dict, with the send_* call being a success.
This would then also allow for cases such as:
assert_bot_response(message, (response, method), result={'result':'error'}) # method result is error
assert_bot_response(message, (response, method, result={'result':'error'}) # same as line above
assert_bot_response(message, (response1, method1), (response2, method2)) # two responses, both successful
assert_bot_response(message, (response1, method1), (response2, method2, result={'result':'error'}), (response3, method3)) # three responses, two of which succeed
assert_bot_response(message, (response1, method1, result={'result':'success'}), (response2, method2), (response3, method3, result={'result':'success'}), result={'result':'error'}) # same as line above

This should be moved to the python-zulip-api project.


BUG: Improve the color of footer elements in dark mode

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The footer elements in dark mode are not readable due to dark colours.
I would like to work on this. @anuragverma108 please assign this to me under GSSoC'24.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

work on it!!


data: lmac01

Changes proposed
Updates my LinkFree profile that was created with the previous version of LinkFree

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers
I followed all of the instructions on the editing guide docs.
I only edited the JSON file directly in GitHub (data/lmac1.json) but I don't understand why it's saying that there are other files changed - are these automatic actions from GitHub?

I didn't make these changes, they were automatically added by GitHub. Will try to revert them. Can I do that here on GitHub, or do I have to check out the project locally?
(sorry just did extra commit to add one extra link)

I didn't make these changes, they were automatically added by GitHub. Will try to revert them.
Dan in the EddieHub Discord advised me to merge from EddieHub main into my branch. I just did that and now there's only one file changed. Hope it's okay, let me know if I'm missing anything else

Thanks!
Dan in the EddieHub Discord advised me to merge from EddieHub main into my branch. I just did that and now there's only one file changed. Hope it's okay, let me know if I'm missing anything else

yep that works fine as well, you did great 😇

Thanks so much for your help @Aadarsh805 🤩


Docker image for 5.5.4 unavailable

CrateDB version
5.5.4
CrateDB setup information
Any Docker / Kubernetes install
Problem description
The 5.5.4 image is missing, but weirdly the unreleased 5.5.5 image is available, although it's actually the 5.5.4 release
Steps to Reproduce
docker pull crate:5.5.4
Actual Result
Error response from daemon: failed to resolve reference "docker.io/library/crate:5.5.4": docker.io/library/crate:5.5.4: not found
Expected Result
Released binaries should be available on Docker Hub

This is a known issue and happened by mistake:
https://github.com/docker-library/official-images/pull/16181/files#diff-3d54c8667b603b919713cf8a929ff7a68f7f8544a247d2a76c78f3f4af693a1dR11
There is however no "easy" fix for it without disrupting other processes.

No worries, just wanted to make sure you guys were aware of the issue

thx @robd003! but yes as @proddata said we are aware already of this hiccup.


Creating plugin (plugin-seed) with custom AndroidManifest settings is failing

From @NickIliev on May 10, 2018 12:12
Problem:
Creating a plugin with the plugin seed structure and using Android AAR library that needs some Android permissions. When the permissions are added in the plugin's AndroidManifest file the build is failing with
Unable to apply changes for device: emulator-5554. Error is: Failed to build plugin nativescript-sinch-new :
Error: Command failed: gradlew.bat -p D:\repositories\nativescript-sinch\demo\platforms\tempPlugin\nativescript_sinch_new assembleRelease -PcompileSdk=android-27 -PbuildToolsVersion=27.0.3 -PsupportVersion=26.0.0-alpha1
error: invalid file path 'D:\repositories\nativescript-sinch\demo\platforms\tempPlugin\nativescript_sinch_new\build\intermediates\manifests\aapt\release\AndroidManifest.xml'.

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':verifyReleaseResources'.
> java.util.concurrent.ExecutionException: java.util.concurrent.ExecutionException: com.android.tools.aapt2.Aapt2Exception: AAPT2 error:
check logs for details

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

BUILD FAILED in 1s

The issue seems to be happening only when the command npm run demo.android is executed and change is made in the AndroidManifest.xml file of the plugin (e.g. adding user permissions). If the same permissions are directly added to the application AndroidManifest.xml the issue is not observed
Copied from original issue: NativeScript/android-runtime#1048

From @RoyiNamir on May 17, 2018 21:47
@NickIliev  Hi. Happens here also .
I've tried to modfy nativescript-audio's manifest permission ( also in the aar->zip->modify->aar file)
And got this :

If I revert the permissions , It's ok.
(case : I wanted to remove the recording permission since i'm only using playback. not recording).

@RoyiNamir the issues can be resolved by applying the change in AndroidManifest.xml and then rebuilding the plugin (stopping the live sync from nativescript-plugin-seed). The issue appears (at least on my side) only when trying to sync a change in the plugin under development (nativescript-plugin-seed) and on second thought it is kind of expected as the plugin seed is using the merged AndroidManifest file of the demo project.

Closing this issue as it is related to how the CLI is updating the AndroidManifest file. Possibly related to this


[WIP] improve ceph deployment

make two improvements of ceph deployment

make osd deployed in parallel to save time

change the conditions of running ceph-update role.
We run ceph-update to restart ceph services, always ceph services should be restarted when ceph.conf changed or ceph package is upgraded.
We let ceph scale-out trigger ceph-update because sometime scale-out comes together with enabling multi-pod and enabling multi-pod changes ceph.conf.

The problem is that it may take hours to rebalance after scale-out, ceph health is not ok until rebalance is done. While we can't do ceph-update(restarting ceph services) until ceph health ok.
My design is like this:
We never do scale-out together with changing ceph.conf or upgrading ceph.
If new osd nodes are in other pod, we can enable multi-pod first. Then do scale-out.
I will update scale-out document if this design is accepted.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Tested on tardis-sl by using VMs.
deploy env: envs/example/ci-ceph-redhat
result:
deployment is finished without errors.
osd nodes are deployed in parallel.
Create VM and volume successfully, and is able to attach volume to VM.
Writing volume in VM by using dd command is fine.

@paulczar @nirajdp76 @knandya , please help review this PR.

closing this as new PR has been made to master
https://github.com/blueboxgroup/ursula/pull/2451


FITC is Fully rather than Partially independent

Just a minor typo.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Issues with power management introduction into Intel CAVS25

Describe the bug
Few Kernels tests started failing after 95168e6776d6c8803a5e1f3ed10e1a8b6bc98311 was merged:
INFO     - 1) tests/boards/intel_adsp/ssp/boards.intel_adsp.ssp on intel_adsp_cavs25 failed (Timeout)
INFO    - 2) tests/kernel/tickless/tickless_concept/kernel.tickless.concept on intel_adsp_cavs25 failed (Timeout)
INFO    - 3) tests/kernel/timer/timer_api/kernel.timer.tickless on intel_adsp_cavs25 failed (Timeout)

To Reproduce
Steps to reproduce the behavior:

west twister -p intel_adsp_cavs25 -s tests/boards/intel_adsp/ssp/boards.intel_adsp.ssp

Expected behavior
Tests shall pass
Impact
Multiple kernel tests failing.
Logs and console output
In the case of SSP, no output.  With tickless tests, the test just times out.

@kv2019i  FYI

There are a bunch of hw specific flow in this commit, looking Zephyr's pm, the way core 0 is handled in pm_state_set raises some suspicious.
+       if (state == PM_STATE_SOFT_OFF) {
+               core_desc[cpu].intenable = XTENSA_RSR("INTENABLE");
+               z_xt_ints_off(0xffffffff);
+               soc_cpus_active[cpu] = false;
+               z_xtensa_cache_flush_inv_all();
+               if (cpu == 0) {
+                       uint32_t ebb = EBB_BANKS_IN_SEGMENT;
+                       /* turn off all HPSRAM banks - get a full bitmap */
+                       uint32_t hpsram_mask = (1 << ebb) - 1;
+                       /* do power down - this function won't return */
+                       power_down_cavs(true, uncache_to_cache(&hpsram_mask));
+               } else {
+                       z_xt_ints_on(core_desc[cpu].intenable);
+                       k_cpu_idle();
+               }

how core 0 is supposed to wake up if all interruptions are masked ?
One more thing, if is not necessary to do anything in cores !=0 other than k_cpu_idle you should just remove cpu-power-states = <&d3>; in the device tree for these cores, then when these cores go idle (idle thread runs) the function pm_system_suspend will return false and the idle thread will invoke k_cpu_idle. This way you simplify a lot this code path.

@ceolin you know is the test framework setting the PM_STATE_SOFT_OFF? In sof we are forcing it in driver power off message... I thought it was not set automatically.

@ceolin hmm I guess the "new" power-off handling should be surrounded with #ifdef CONFIG_PM_POLICY_CUSTOM in power.c then the tests seem to work. it is kind of assumed here that this should be only called by custom handler in the app... this is my blind copy from ace side which I think we are not testing with zephyr tests...
So cavstools can't handle currently the ipc response sent from assembly, also for example the ssp test would need next_state handler for custom policy.

fix here: https://github.com/zephyrproject-rtos/zephyr/pull/56843

this is fixed now


dubbo feign http call and mvc annotation support

What is the purpose of the change
dubbo feign http call and mvc annotation support
Brief changelog
Verifying this change
Related Issues
#10803

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Please submit this feature to 3.2 branch.

ok


[SPARK-38236][SQL] Treat table location as absolute when the first letter of its path is slash in create/alter table

What changes were proposed in this pull request?
After https://github.com/apache/spark/pull/28527, we change to create table under the database location when the table location is relative. However the criteria to determine if a table location is relative/absolute is URI.isAbsolute, which basically checks if the table location URI has a scheme defined. So table URIs like /table/path are treated as relative and the scheme and authority of the database location URI are used to create the table. For example, when the database location URI is s3a://bucket/db, the table will be created at s3a://bucket/table/path, while it should be created under the file system defined in SessionCatalog.hadoopConf instead.
This change fixes that by treating table location as absolute when the first letter of its path is slash.
This also applies to alter table.
Why are the changes needed?
This is to fix the behavior described above.
Does this PR introduce any user-facing change?
Yes. When users try to create/alter a table with a location that starts with a slash but without a scheme defined, the table will be created under/altered to the file system defined in SessionCatalog.hadoopConf, instead of the one defined in the database location URI.
How was this patch tested?
Updated unit tests.

CC @yaooqinn @cloud-fan for a review.

Thanks @dongjoon-hyun for the guidance. Created https://issues.apache.org/jira/browse/SPARK-38236 and updated the title.

thanks, merging to master/3.2!

@bozhang2820 can you help to create backport PRs for 3.2 and 3.1?

Created https://github.com/apache/spark/pull/35591 for 3.2.
It seems that it can merge to 3.1 cleanly. Should I create one for 3.1? Also should I add "[3.2]" in the PR title in this case?

+1, late LGTM.


Update local

NOTE
Please verify that the base repository above has the intended destination!
Github by default opens Pull Requests against the parent of a forked repository.
If this is your personal fork and you didn't intend to open a PR for contribution
to the original project then adjust the base repository accordingly.

@jackkoskie please close this pr

@jackkoskie please close this pr

Whoops! Created the pr in the wrong direction


feat(flow): add flow types

adds flow types that match the nomenclature of flow more closely. fix https://github.com/avantcredit/gql2ts/issues/103

Coverage increased (+0.09%) to 96.591% when pulling fa648647c642b2bac1ee8c7e14f62288807101ff on arahansen:flow-types into 458af3101d1e11ae984edf6de8b534b1881123d7 on avantcredit:master.


fix(translations): fixed some typos and refined translations

Fixed a few translations I noticed. I'll try to take a closer look next week. Great work btw! 🙂

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make sure that fragment shader interlock instructions are not removed by DCE

@s-perron correct me if I'm wrong, but it seems like AggressiveDCE is treating OpBeginInvocationInterlockEXT and OpEndInvocationInterlockEXT by not removing them? Are there any other tests I should add?

That should be good enough. Can you look for any place that checks for OpMemoryBarrier? These instructions should be treated like memory barriers, and that should let you know which optimizations will need to handle them.

The only explicit references I see to OpMemoryBarrier in opt are in CodeSinkingPass, LoopFusion, and UpgradeMemoryModel – should I update any of these?


fix: reduce docker context building locally

modify dockerignore due to workspace changes to include node_modules from subdirectories for example

Works nicely, seeing a reduction from 83 MB to 11 MB.


createAccount

Hello,
I'm trying to create multiple accounts with your method createAccount.
But the callback is not called.
Is it deprecated?
Code :
var SteamUser = require('steam-user');
var newAccount = new SteamUser();
var email = "";
var accountNames = [...];

for (var i = accountNames.length - 1; i >= 0; i--) {
	var password = passwordGen(12, false);
	console.log("Try : " + accountNames[i] + ":" + password);
	newAccount.createAccount(accountNames[i], password, email, function(result, steamid){
		if (steamid){
			console.log("created account : " + accountNames[i] + ":" + password + " [steamID]" + steamid);
		}
		console.log(result);
	});
};

Thanks,
Eddy

You must be logged on either anonymously or as an existing individual user to use this.

Thank you


nixos/power-profiles-daemon: Add assertion with auto-cpufreq

Description of changes
auto-cpufreq is similar to tlp in that it shouldn't be run with power-profiles-daemon.  There functionality can conflict and bugs can show up.  On my system this materialized by auto-cpufreq frequently shutting down, but there may be other consequences.
This change follows the same pattern as the tlp assertion
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This is baby's first NixOS commit. I tried reading through documentation to make sure I'm following convention, but a few things were unclear.  If I've done something wrong, please don't be afraid to tell me to RTFM and send me in the general direction of docs.
The one thing that was least clear to me was how to test this change.  Most of what I saw in documentation revolved around build/testing packages and not configuration changes.
This assertion triggers on a pretty easy situation to get into, having gnome and auto-cpufreq both enabled.  While there is no situation where a user would want that, I think this will probably still be considered a breaking change and should target a future release.  I made my PR target master for this reason, but let me know if that's wrong.

Pinging @Technical27 since I think you might be the most relevant maintainer

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4070

I took a second look and think @mvnetbiz and @picnoir might be the correct maintainers.  Sorry if I'm spamming notifications, still learning the ropes here.

I really appreciate the detailed explanation of your testing steps!  Thank you very much


Delete delegate on operation queue in handleReceivedMessage

It was the only place where delegates array might be written from different queue.
@vodovozovge, @hoangnami, let's run the main app with this commit on M1 Mac to confirm if it cures the issue Yoan reported.

Tested and verified, crash can no longer be replicated with these changes.

Checked that none of handleReceivedMessage (where the change is embodied) callers are executed in operationsQueue. Good to merge.


[clarification]: Consider rationalizing the zoo of possible query return types

Specification
Data
I need clarification on ...
do we really need so many?
Additional information
In the Javadoc of @Find, and in the module-level documentation, we find some rather long lists of possible things that users might want to return, depending on the semantics of their query, their own personal aesthetic sensibilities, and, apparently, on the phase of the moon and the price of West Texas Crude:

List
Page
Slice
KeysetAwarePage
KeysetAwareSlice
Set
Stream
Streamable
Iterable
Collection
Optional (for single instances)
The entity type itself (for single instances)
Array of the entity type

The module-level doc even sneaks in LinkedHashMap as an option!
Some of these options are needed. Some of them, IMO, aren't:

Collection is a worse List
Streamable is an even worse List
Iterable is the worst List
Set doesn't really seem necessary either—if the caller wants to put the results in a Set, it can quite easily call new HashMap()
LinkedHashMap looks like he got drunk and gatecrashed a party full of posh people

I would kick at least some of these guys out.

I would say that Set is the worst of all of these because it even loses the order (unless it is something like a LinkedHashSet, but it doesn't say it is that).  We should definitely get rid of Set.
Collection and Iterable seem unnecessary given that a repository can define the return type as List and a user can still write code that treats it as a Collection or Iterable if they want.  We can get rid of these.
LinkedHashMap probably should have been removed already. It is likely left over from when we were exploring all the different options that could be possible as return types. If a user really wants the LinkedHashMap behavior, they could implement it themselves by accepting a List and putting the elements into a LinkedHashMap of their own.  I don't see why we should make the Jakarta Data provider do this for them.
Getting rid of the above doesn't stop any vendors from supporting the extra return types on their own.
I believe Streamable is something that Spring asked for (they have a similar interface) and Micronaut also liked.  @otaviojava worked on this one and might have more background information from them on it.
At first glance, it looks like a way to obtain multiple streams for the same result, but you can already do that by just returning a List and obtaining multiple streams from the list.  If we keep it, it would be nice if its JavaDoc could include an example illustrating the additional value that it provides.

I believe Streamable is something that Spring asked for (they have a similar interface) and Micronaut also liked. @otaviojava worked on this one and might have more background information from them on it.
At first glance, it looks like a way to obtain multiple streams for the same result, but you can already do that by just returning a List and obtaining multiple streams from the list. If we keep it, it would be nice if its JavaDoc could include an example illustrating the additional value that it provides.

I mean I guess I would say that if Java really needs an API for a cached stream—and I'm really not sure it does—then Jakarta Data is not the right place to define it. If it's so useful then I wonder why it's not in java.util.stream where it would seem to belong.
And so I agree with you that if this is what you want to do then List is sitting right there in java.util staring you in the face, asking you if it's chopped liver.

This was fixed by #511.


MSI support minor alignments and adjustments

Codecov Report

Merging #915 into dev will increase coverage by 0.16%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##                dev     #915      +/-   ##
============================================
+ Coverage     49.69%   49.86%   +0.16%
- Complexity     2880     2889       +9
============================================
  Files           117      118       +1
  Lines         27998    28012      +14
  Branches       4666     4673       +7
============================================
+ Hits          13915    13969      +54
+ Misses        11839    11790      -49
- Partials       2244     2253       +9

Flag
Coverage Δ
Complexity Δ

#JDBC42
49.46% <100%> (?)
2850 <0> (?)

#JDBC43
49.67% <100%> (-0.03%)
2881 <0> (+1)

Impacted Files
Coverage Δ
Complexity Δ

.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java
80.08% <100%> (+0.26%)
25 <0> (ø)
:arrow_down:

...n/java/com/microsoft/sqlserver/jdbc/DataTypes.java
79.68% <0%> (-0.2%)
5% <0%> (-1%)

.../com/microsoft/sqlserver/jdbc/SQLServerJdbc42.java
25% <0%> (ø)
0% <0%> (?)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
56.26% <0%> (+0.16%)
0% <0%> (ø)
:arrow_down:

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
52.28% <0%> (+0.22%)
213% <0%> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
42.81% <0%> (+0.22%)
310% <0%> (+2%)
:arrow_up:

.../microsoft/sqlserver/jdbc/SQLServerDataSource.java
45.12% <0%> (+0.24%)
66% <0%> (ø)
:arrow_down:

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
65.01% <0%> (+0.29%)
0% <0%> (ø)
:arrow_down:

...ncurrentlinkedhashmap/ConcurrentLinkedHashMap.java
39.22% <0%> (+0.43%)
44% <0%> (+1%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


questions about disk's features

Hi, @jatentaki
Thank you for your great work! I have some further questions:

have you tested disk's result on aachen localization benchmark and ETH benchmark (for 3D construction tasks)? How's the result?
why you choose unet as feature extraction network? or have you compared with other networks?
have you tried to do feature detection in different layers? (just like multi-scale strategy)
I found disk's keypoints number is less than other SOTA methods, but still works great. What's the major reason, is it because of the grid strategy during training?

Seems many questions...however, really looking forward to your reply. Thanks a lot in advance!

Hello.

We have tested on some scenes from the ETH benchmark. The results are reported in our paper (section 4.3, page 8). I am sceptical of this benchmark in general because it does not compare with any ground truth. In fact, I have tried DISK on the Gendarmenmarkt scene, which includes two similarly looking churches and although I got very competitive numbers (benchmark-wise), the reconstruction was completely wrong because the two churches got merged into a single thing. This particular issue, collapse of similar structures, is the main issue I found using DISK with COLMAP. I believe it stems partially from our training method: firstly, most tourist landmarks are old [-> weathered, therefore strongly textured] and asymmetric, secondly we only train on pairs of covisible images, which doesn't expose the network to the fact that there may be similar but non-matching structures. Largely, though, I would say this is an issue to be fixed on the SfM algorithm side. COLMAP starts with the initial image and tries to match as many others as possible to it, before going on to further away regions. At the same time, it has no method to recover from initial errors, so two similarly looking buildings will likely be collapsed, even though the overall structure of image-to-image matches could be explained much better with a different covisibility graph. We have not tested on Aachen.
I have not tested other architectures. UNet is the go-to dense regression network for me, so I just used it and didn't spend time optimizing this aspect.
No.
What do you mean by less than SOTA? What methods do you have in mind and how do you run DISK (detect.py flags, image resolution, etc)? Going back to the remark in section 4.3 of our paper, DISK ran without limiting feature numbers yields 60k+ detections (which mostly are correctly triangulated into landmarks), more than 4x that of other methods.

for the fourth question, i mean in the inference time, for 640*480 images, i can extract about 2.5k~3.5k keypoints without limiting feature numbers (just using detect.py), and matches quite good.  This is quite good because less keypoints can also work well. So i'm quite interested what's the major reason/strategy. Is it because of the grid strategy during training, or actually you didn't mean to reduce the kpt number in inference time, and disk just learned a differentiable score map?

For 640*480 I think it's probably mostly the lack of multi-scale extraction. That said, it's difficult to comment without knowing the specifics of the dataset.


Cobigen generates ToTest which is not supporting SortedSet

Problem
When we use SortedSet in the To the ToTest is failing in the method "private void testEqualsAndHashcode(Class<?> clazz, SoftAssertions assertion, ToExclusion exclusion)"
Expectation
The test should not fail for SortedSet
Solution hint
If the green undelayed snippet is added everything works fine again

Issue moved to devonfw/devon4j #568 via ZenHub


[Linux] read returns null when keyring is unlocked for the first time after boot

FlutterSecureStorage.read returns null for the first time unlocking the secret-service after system boots.

If the keyring is already unlocked beforehand, it works fine.
If I manually lock the keyring and then try to read, it works fine also.

This issue is only occuring in Ubuntu-22.04 or above and Debian-12 or above based Linux distributions
Mine is Ubuntu 23.04
Source code:
https://github.com/KRTirtho/spotube/tree/cf7b849cddca3260d9c3a6a064418b0ba2d63270/lib/utils/persisted_state_notifier.dart#L60-L97

Also, in a KDE based environment where KWallet is installed it still uses gnome-keyring instead of kwallet

I am closing all older issues. If this issue still exists in the latest version, please let me know.


Error validating rss feed

First of all, great work on this theme, it is amazing and I'm looking forward to using it on my podcast/blog.
But I tried to validate its RSS feed against some known tools and it failed, making me wonder if it will work when I try to import it to Spotify or iTunes.
https://castfeedvalidator.com/?url=https://wave.ghost.io/rss/
ERROR: The link tag with attribute rel='self' does not point to a valid RSS feed and will therefore be denied by Apple and Google.
WARNING: Your feed has no language tag. Apple Podcasts will not accept a feed with no language specified.
WARNING: Found 10 episodes with an invalid length. The length should be a nonzero value specified in the enclosure tag, and refers to the file length in bytes.
WARNING: Found 8 items with an enclosure URL that has already been used in your feed. These episodes will be ignored by Apple Podcasts.
ERROR: No itunes:image tag in feed. Cannot validate feed artwork.
FATAL: No artwork URL found. Did not finish artwork tests.
WARNING: Your title ID3 tag is missing.
WARNING: Your bitrate mode is not set to CBR. This is the optimal setting for podcast audio.

Hey @thiagoalmeidasa,
Went through the validation warnings and errors.

ERROR: The link tag with attribute rel='self' does not point to a valid RSS feed

Fixed this one in https://github.com/TryGhost/Wave/commit/e7c7bbb431f3c3d439fd2e55041e67d3a1f15c94.

WARNING: Your feed has no language tag.

The feed has a language tag, but the demo site hasn't been updated to Ghost 4.x yet, and that's the reason why<EMAIL_ADDRESS>not working on the demo.

WARNING: Found 10 episodes with an invalid length.

Calculating file length on the fly would be impossible for how we're generating the feed unless we use the og_description field for full info like: url="https://example.com/episode-1.mp3" length="1500" type="audio/mp3". Still not sure about this 🤔

WARNING: Found 8 items with an enclosure URL that has already been used in your feed.

That's because we used same audio URL for demo purpose.

ERROR: No itunes:image tag in feed. Cannot validate feed artwork.
FATAL: No artwork URL found. Did not finish artwork tests.

Site publication icon is used as a feed artwork here. On the demo site, we don't have any publication icon selected. When you select an image as an icon, it'll be validated with no issue.

WARNING: Your title ID3 tag is missing.
WARNING: Your bitrate mode is not set to CBR. This is the optimal setting for podcast audio.

These are related with the audio files we used on the demo.

Closing the issue. You can reopen if have any issue when you upload your podcast :)


DOCS-1756: RN for removing cluster

Product Version(s):

Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

@bmckercher123 PTAL. Thinking I'll group this with other enhancements rather than give it a grand heading.


bip21 URI scheme should be lowercased?

https://github.com/rust-bitcoin/rust-bitcoin/blob/7c6a7c34b87fa92e6432247e3ce909333ef44d05/bitcoin/src/address.rs#L985
Even if according to the spec the scheme are case insensitive
When scanning a bip21 URI like BITCOIN:TB1QENTR8XV3ZEFGGYDAEDNF99AQFVD98HZW4KUWWQ from my android generic camera, the intent is not passed to bitcoin applications like  bitcoin:TB1QENTR8XV3ZEFGGYDAEDNF99AQFVD98HZW4KUWWQ is doing.
So even there is a penalty in qr code dimension with the lowercase scheme, I think we have to bite the bullet and use a lowercase scheme.
Leaving here the QR code with the uppercase scheme if someone could try it from an iphone

Tried from Android with Blue Wallet installed and didn't get it offered. So probably iOS works correctly by treating the scheme as case-insensitive (IIUC this is the URI standard) and Android is buggy. In an ideal world Google would fix it.
I suggest let's lower case it and reserve the right to change it if the situation changes.

/// If the address is bech32, both the schema and the address become uppercase.

/// If the address is base58, the schema is lowercase and the address is left mixed case.

Where does that notion even come from? URIs should be case insensitive so sane creators would always make them lowercase..
I think we should definitely remove that and make them all lowercase.

I assume it's because in ASCII the uppercase letters have their upper two bits clear, while the lowercase ones have only the uppermost one clear.

I've looked into QR code encoding in the past and IIRC alphanuumeric is just its own encoding. And yes, I also think it's weird to encode upper case more efficiently.


Update for 243 realease - Switching to PackageApiClientService provided by the platform

Moved to platform internal APIClient service
Update JEWEL
Bump for release 243

Also tests are not passing, please fix them.


Measure along / within ROIs

Not sure if this is within the scope of the project, but it would be nice to be able to measure along ROIs (e.g. a "profile line" through that moves through space in one (more, eventually?) axis. Through lines, it could be a profile line. With polygons, it could be average intensity (or any other function that takes a set of pixels and returns a number), or a profile line along the circumference of the shape.
Something like this as an example of the profile line:

I worked on something like this a couple of years ago, and might give it a shot when I have the time, but I thought I'd leave this here so I could remember, and if it inspired anyone else to give it a shot since I think it'd be very useful!

Yes, this is definitely within scope - if someone wants to have a go and open a PR it would be very welcome! 😄


select 语句报SQL syntax错误

SELECT state, ROUND(SUM(duration),5) AS `duration (summed) in sec`
FROM information_schema.profiling
WHERE query_id = 0
GROUP BY state
ORDER BY `duration (summed) in sec` DESC

测试下来，问题出在select中字段（Field）的别名（Alias）未加上反引号导致。


Update validate.yml

Teşekkür edeRim
12 Oca 2024 Cum 22:25 tarihinde Jefri Reynaldi @.***>
şunu yazdı:

You can view, comment on, or merge this pull request online at:
https://github.com/flexwie/devcontainer-features/pull/8
Commit Summary

dbd0516
https://github.com/flexwie/devcontainer-features/pull/8/commits/dbd0516e09ac867480c057380d6de8475d3ec4cf
Update validate.yml

File Changes
(1 file https://github.com/flexwie/devcontainer-features/pull/8/files)

M .github/workflows/validate.yml
https://github.com/flexwie/devcontainer-features/pull/8/files#diff-a5c0f08bc43f53a0451cad4a4a2e435ce55ad34183350dfae5e4b2c7dca6360b
(44)

Patch Links:

https://github.com/flexwie/devcontainer-features/pull/8.patch
https://github.com/flexwie/devcontainer-features/pull/8.diff

—
Reply to this email directly, view it on GitHub
https://github.com/flexwie/devcontainer-features/pull/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCLDQG2VGOAWUGFT5IYU4V3YOGEY7AVCNFSM6AAAAABBYTGMH6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3TSNJUHEZTMMI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Build instructions?

Trying to hack on this at the citation.ccf hackday and it would be great to have a page that shows how to get a dev environment up.  Perhaps imagine you've got a new install of Mac OS X, how would you get coding on a local instance (install brew, install python, install npm, install angular).  Or use having conda installed as a starting point?

Seems like a conda environment.yml file would be a good approach here?
https://conda.io/docs/user-guide/tasks/manage-environments.html#creating-an-environment-from-an-environment-yml-file

Hello. Installation instructions are now available for the back-end API as well as the front-end web application.
I plan to improve the installation process with docker or conda down the road. But this should be a good start for anybody looking to get the app running on their local machine.