DBZ-1228: MySQL connection with client authentication does not work

Summary
A connection to SQL server with client-side SSL authentication (like Google Cloud SQL) cannot be established.
Issue analysis
The issue is caused because of the shutdown method being called here. This is why the connection test succeeds, but fails later. The next time when debezium attempts to connect to the database here, these variables are not set and the connection fails.
Solution
We can set the system variables before the call is made. Since the start method is coded in a way that it sets the values only if the configuration values are provided, we can simply reuse this. This solves the current issue but introduces another one.
On forcing debezium to have the system variables, we get an authentication error. In order to fix this, we need to create and pass a socket factory that provides the keystore config. The changes are made in a way such that any non-SSL connections are allowed when the required configurations are not provided, and the required values are only set if the keystore password is provided.

Merged, thanks @Arkoprabho and @jpechane


Add 'gRPC - RPC rebirth?' slides

Presentation at 23. NetPonto (a .NET portuguese community) community meeting in Porto on how to build gRPC services with C#.

I signed it!


[opencv] Update to OpenCV 3.4.4

Update opencv port to OpenCV 3.4.4.
It is able to builds with enabled dnn module.
Install Test: ./vcpkg install opencv[ffmpeg,ipp,dnn,contrib]

[x] x64-windows
[x] x86-windows
[x] x64-windows-static
[x] x86-windows-static

@cenit Please check this port. Thanks,
Future Works: If this OpenCV 3.4.x port is stable, Let's start move this port to opencv3 and implement update to OpenCV 4.x. Discussion https://github.com/Microsoft/vcpkg/issues/4758

I just realized that opencv[gdcm] is broken (the same in the accepted port). Shall we fix it before merging this one?
I think that the problem relies on the gdcm2 port itself, that correctly removes the debug/include folder but leaves the installation with broken paths...

Of course that would mean another PR for that problem, but I'd like to see a properly and thoroughly checked port for OpenCV once in a while... I think it deserves this treatment!

also opencv[sfm] is broken because the dependency walker (for static build at least) goes to gflags-static.lib also for debug builds, instead of gflags-staticd.lib

also opencv[ovis] is broken in the accepted port, I will analyse your PR just after I finished with that one... Just leaving comments here as notes to what to check ;)

Ogre::PixelFormat::PF_DEPTH16 required new 1.11.3 version of Ogre, please see PR #4845.

I have some problems also with opencv[dnn] due to libprotobuf linking mismatch... (again, not specifically to your PR, it is the same in the published port). It seems that opencv is used a lot here, am I the only one seeing all these problems?
It was a long time i was not updating my vcpkg installation, it seems that it was a mistake starting updating... to be honest, vcpkg concept is great, but the realization seems still very far...

Is that a problem in this port? or dependency ports?
First of all, I think that it is necessary to find out whether the cause of problem is either port.

it can be both and should be analysed thoroughly.
I did some other tests and also opencv[openexr] is broken, because the ilmimf lib which is linked against is always the debug one, also in release mode.
When I did that huge work on this port I devoted a lot of time fixing many dependencies, which meant have them properly installed, expose a working config or module for cmake, then removing the custom files that OpenCV devs like to use (instead of the official ones). I think that many regressions happened in this year and I'd prefer a 3.4.3 really working port instead of a rushed 4.0 (or even 3.4.4)

I sent pull request #4820 to update openexr. Please test this port.

#4861 fixes the opencv[dnn] feature, fixing the libprotobuf linking

I just realized that opencv[gdcm] is broken (the same in the accepted port). Shall we fix it before merging this one?
I think that the problem relies on the gdcm2 port itself, that correctly removes the debug/include folder but leaves the installation with broken paths...

gdcm2 problem shall be fixed by PR #4880


Add troubleshooting section to main farming section

Currently there is only farming troubleshooting under the FAQ. While there's lots of good info there, it's easy to miss when drilling down the nav menu and also could provide a bit more of a structure approach to troubleshooting.
Here's a high level of what I'm imagining:

Build troubleshooting (how to wipe disks, BIOS config steps, etc)
Basic post build troubleshooting (reboot node, reboot router, check the network signal path)
Advanced post build troubleshooting (connect a screen and look for errors, then info on how to address certain errors)
How to get further assistance (contact support team/use the farmers chat for community support)

Updated:


App crashing on release (android)

So I wasn't able to make the app work on release.
Compiling the example app on release raised no issues but trying to launch the app causes it to crash with:
E/flutter (14495): [ERROR:flutter/runtime/dart_isolate.cc(707)] Could not resolve main entrypoint function.
E/flutter (14495): [ERROR:flutter/runtime/dart_isolate.cc(191)] Could not run the run main Dart entrypoint.
E/flutter (14495): [ERROR:flutter/runtime/runtime_controller.cc(382)] Could not create root isolate.
E/flutter (14495): [ERROR:flutter/shell/common/shell.cc(574)] Could not launch engine with configuration.

Eventually I found out what the problem was, after a lot of digging around and searching with not much results or information about it I decided to post here if anyone else is tackling the same issue.
Basically on android you have the activity that is responsible for the main UI thread(your main code), by default android doesn't allow running processes of closed apps since android 8 to prevent slowing down and draining the battery, so for an app to have a process running even if closed they introduced service which enables you to do that with an indicating notification as long as the process is running.
This is why you need to put the <service> tag on the AndroidManifest.xml file for the foreground task to work at all, so the problem on release is that the app doesn't know from where to start the foreground task isolate (entry point).
On the example code I noticed that the function startCallback has to be on the global scope and trying to make it an anonymous function won't work even on debug, this means that that's the main entry point of the service so all I had to do was add @pragma('vm:entry-point')
above the function and it fixed the issue and worked smoothly as expected on release.
How the solution looks:
@pragma('vm:entry-point')
void startCallback() {
  FlutterForegroundTask.setTaskHandler(FirstTaskHandler());
}

Don't know if it works on ios but I do know that it also happens there so it might be from the same reason.

I had the exact same issue in release.
I try to add the @pragma('vm:entry-point') , but now my app is actually crashing instead of displaying the Dart Error: Dart_LookupLibrary ....
It is working fine in debug.

I have explored a little bit more about this crashing app and found this cause of the issue and a possible workaround.
This is all detailed in a new issue:  https://github.com/Dev-hwang/flutter_foreground_task/issues/116

my app is also crashing in release mode only after starting the service.


Github is limiting the number of issues you can create at once to 25?

I just tried to bootstrap issues on Barkedu, and I had to do it in batches of 25.

This is really really annoying me.


Remove version: Google.Chrome.Canary version 133.0.6851.0

All InstallerUrls returned 404 Not Found

https://dl.google.com/release2/chrome/acqyvlkmla7yw2rcnv4qnda3uama_133.0.6851.0/133.0.6851.0_chrome_installer.exe
https://dl.google.com/release2/chrome/acvxcwial6vgyq5qtsumpnts73wa_133.0.6851.0/133.0.6851.0_chrome_installer.exe
https://dl.google.com/release2/chrome/add5qxxjyipadptcy4lhl3edhgiq_133.0.6851.0/133.0.6851.0_chrome_installer.exe

Microsoft Reviewers: Open in CodeFlow

URL: https://dl.google.com/release2/chrome/add5qxxjyipadptcy4lhl3edhgiq_133.0.6851.0/133.0.6851.0_chrome_installer.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 923)


Setup Amazon S3 → Cloudflare R2 pipeline

Cloudflare R2 is S3 API-compatible.

Determine how to push to R2 using the AWS CLI.
Determine how to index data in R2 using the s3fs.

Done.


ODE fonts

Problem description
Current fonts of the ODE does not correspond to those used by the Open Knowledge Foundation.
OKFN fonts: here
Guidelines: here and here
Please, let me know if we need to organize a call with @lucpretti for clarifications.

Fonts added in issue #445


Update to avoid confusion

This article describes how to deploy a new SSL certificate to your AD FS and WAP servers.
Note
The AD FS SSL certificate is not the same as the AD FS Service communications certificate found in the AD FS Management snap-in. To change the AD FS SSL certificate, you will need to use PowerShell.
Replacing the SSL certificate for AD FS
Note:
First, determine which certificate binding mode your AD FS servers are running default certificate authentication binding, or alternate client TLS binding mode.

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

ID: d8733a32-8eef-bdb8-673f-065e8a5f1e0f
Version Independent ID: f111b377-0dce-d7a6-9483-8830533cacba
Content: Managing SSL Certificates in AD FS and WAP in Windows Server 2016
Content Source: WindowsServerDocs/identity/ad-fs/operations/manage-ssl-certificates-ad-fs-wap.md
Product: windows-server
Technology: ad-fs
GitHub Login: @jenfieldmsft
Microsoft Alias: billmath

Hello @bellnaf,
Thanks for contacting Microsoft. We value your feedback.
Documentation changes have been made to clarify and acknowledge this issue. The doc changes are now live.
Thank you again and have a most pleasant day.
#please-close


Double Alert on Seek

2 Alerts come up when seeking on youtube.

Resolved. Alerts were doubled because buffering on youtube counts as a buffer, pause, and play.
Alerts now only appear on play.


Add a debugMode flag to log queries to the performance timeline

This displays all SQL queries on the performance timeline. Currently only implemented in the Web SDK.
This excludes the time waiting for the global transaction lock, but includes all overhead in worker communication. This means you won't see concurrent queries in most cases.
This includes internal statements from PowerSync, including queries saving sync data, and begin/commit statements. It does not include internal statements from powersync-sqlite-core.
TODO:

~The flag currently defaults to false if process.env.NODE_ENV == 'production', similar to what React does. All other cases default to true. Should we perhaps be more conservative and require explicit opt-in, since the queries could be considered semi-sensitive data?~ Updated default to false, requiring explicit opt-in for now.

Example of the diagnostics app during initial sync:

Is it documented?

@guillempuche It's not properly documented yet, but you can see how it's used in the diagnostics app:
https://github.com/powersync-ja/powersync-js/blob/05f3dbd25b697dee0214e5bafe852d0055c9b736/tools/diagnostics-app/src/library/powersync/ConnectionManager.ts#L26-L32
I'd recommend using something like this to only enable debugMode in development:
debugMode: process.env.NODE_ENV != 'production'

The feature just displays queries in the "Performance" tab of Chrome's developer tools (after recording) - see the screenshots above.

Thanks Ralf @rkistner


Documentation update - spo feature enable

Updated documentation of below command(s):

spo feature enable

Corrected cmdlet documentation to fix the parameters in the examples given at the bottom of the documentation.

Thank you for this correction @ganesh-sanap, we'll try to review it ASAP!

Merged manually, thank you for fixing this one!


Treat types ending in "Result" as result types

It is quite common to define custom error enums and combine them with a Result type alias, e.g.:
enum CustomError {
    Test,
}

type CustomResult<T> = Result<T, CustomError>;

However, this isn't being caught by log-derive (a similar issue has been https://github.com/elichai/log-derive/pull/6).
This PR changes the is_result_type function to accept all types that end in Result, which seems to cover additional cases.
I'm not sure if this change has any negative implications, but it works nicely for my use-case. :smile:

Hmm I'm thinking if we really want this to be implicit or explicit.
i.e. some user might also have:
enum MyResult<T, E> {
    Succeed(T),
    Failed(E),
}

but won't have the needed map, map_err functions.

Hmm I'm thinking if we really want this to be implicit or explicit.
i.e. some user might also have:
enum MyResult<T, E> {
    Succeed(T),
    Failed(E),
}

but won't have the needed map, map_err functions.

But maybe if any user of this crate has this problem then we'll wait for a complaint and think of a fix? what do you think?

I don't have experience with proc macros, but wouldn't it be possible that if ok and/or err are specified, e.g. #[logfn(ok = "INFO", err = "ERROR")], that the macro just expects the return value to be able to be mapped to Ok and Err and generates the code accordingly? If that's not the case, the compiler could just throw an error (due to invalid code generated).

But maybe if any user of this crate has this problem then we'll wait for a complaint and think of a fix? what do you think?

I agree, I don't think this change will have a breaking impact on a lot of user's code.

Sure it is. the main question here is what should be the default case for #[logfn(INFO)]. should it try to infer if it's Result using the postfix or not.

Hm good question. I think if ok/err are not specified, it should just normally log the return value and not treat it as anything special (even if it is Result) If ok/err are specified, then it should behave just like with this PR.

The whole point for the current is_result_type is to log it "better" by default without the need for explicit logging

What do you think about leaving the implicitly but forcing to treat it as Result if either ok or err is specified?

I do want some good default for the lazy #[logfn].
How do you feel about leaving it as is but if you add Ok/Err I ignore my checks and assume it's a Result type.

@SilentByte what do you think about this: https://github.com/elichai/log-derive/commit/841177536549aeb6377c3f0ddea35032282bab32

How do you feel about leaving it as is but if you add Ok/Err I ignore my checks and assume it's a Result type.

I agree. I believe the way it works now is sensible. #[logfn(Info)] currently logs both Ok and Err with the Info level, as it should. When ok/err are specified, the macro should force the return type to support Ok/Err, and error out if this isn't the case.

Thank you for the contribution!


Firebase realtime database is not updating with new values

Update : i just checked that i cant even add a node to this database with any other account means this code is only running when there is no data in my database after that my database is not updating
Okay so i was trying this realtime database with the same code as written in Firebase Unity tutorial and what this code is doing is just adding my first score to my database and if i m trying to add another score it ain't doing anything and the funny part is that my OnValueChanged function is returning me this new score, i dont know how this OnValueChanged function is working, in the code below dbRef is the rootReference of my database
    public void updateLeaderboard(string userID, string name, string profilePic, int score)
    {
        if (dbRef != null)
        {
            dbRef.RunTransaction(mutableData =>
            {
                List<object> leaders = mutableData.Value as List<object>;
                if (leaders == null)
                {
                    leaders = new List<object>();
                }
                else if (leaders.Count >= MAX_ENTRIES)
                {
                    int minScore = int.MaxValue;
                    object minObj = null;
                    foreach (var item in leaders)
                    {
                        if (!(item is Dictionary<string, object>)) continue;
                        int childScore = (int)((Dictionary<string, object>)item)["Score"];
                        if (childScore < minScore)
                        {
                            minScore = childScore;
                            minObj = item;
                        }
                    }
                    if (minScore > score)
                    {
                        return TransactionResult.Abort();
                    }
                    leaders.Remove(minObj);
                }
                Dictionary<string, object> currData = new Dictionary<string, object>();
                currData["uid"] = userID;
                currData["Name"] = name;
                currData["Photo"] = profilePic;
                currData["Score"] = score;
                leaders.Add(currData);
                mutableData.Value = leaders;
                return TransactionResult.Success(mutableData);
            }).ContinueWith(tsk =>
            {
                if (tsk.IsCanceled)
                {
                    Debug.Log("Data entry task canceled");
                }
                else if (tsk.IsFaulted)
                {
                    Debug.Log("Error writing Database : " + tsk.Result.GetRawJsonValue());
                }
                else
                {
                    if (tsk.Result.Value != null)
                    {
                        this.data = tsk.Result.GetRawJsonValue();
                    }
                }
            });
        }
        else
        {
            Debug.LogError("No database instance found");
        }
    }

and this is my OnValueChanged method
    private void onValueChanged(object sender, ValueChangedEventArgs e)
    {
        if (e.DatabaseError == null)
        {
            if(e.Snapshot.Value != null)
            {
                this.data = e.Snapshot.GetRawJsonValue();
                Debug.Log(data);
            }
        }
        else
        {
            Debug.LogError(string.Format("Error reading database : {0}", e.DatabaseError.Message));
        }
    }

Debug.Log of this method is returning me the new values i m trying to upload to database
And below is my rules of database

{
"rules": {
".read": "auth != null",
".write": "auth != null",
"$transID":{
".validate":"newData.hasChildren(['uid','Name','Photo','Score']) && newData.child('uid').val() == auth.uid",
}
}
}

Okay so after days of research i found the solution for this myself and the solution is THAT TRANSACTIONS DON'T RUN ON ROOT REFERENCE OF DATABASE and i dont know why google didn't mentioned this anywhere in their docs


Fix React Native v0.59

Actually, never mind! I see that this library already has the fixes, and we merely need to upgrade. Oops.


Add Unit Tests for components

Currently, we have 0% test coverage, we should add some unit tests for components so that we can easily debug if something is breaking in components.
cc: @ashutosh1919

@Sachin-chaurasiya yes i definitely agree with you. I have little experience with setting up testing with jest in react. @Sachin-chaurasiya, Can you setup basic test? Then we all can start contributing over it.

@Sachin-chaurasiya yes i definitely agree with you. I have little experience with setting up testing with jest in react. @Sachin-chaurasiya, Can you setup basic test? Then we all can start contributing over it.

@ashutosh1919 , Yeah sure.

Hi, @ashutosh1919 & @Sachin-chaurasiya I would like to take participate in adding unit tests for the project!

Go ahead @ShaileshParmar11 .


New 'listjobs' operation

Would be handy to have an operation to just show what job(s) a service is running in.
Currently, perfinfo gives us that, but it'd be handy to have a less-verbose way of getting at that information.
Formatting it in a way that can be paste-appended to the WRKJOB command might be a plus (for instance, JOB(608887/THATUSER/JOBNAME))

Resolved at https://github.com/ThePrez/ServiceCommander-IBMi/commit/df18ada20a4f23b803be3f37d94b05969c7b1eed


[SUGGESTION] - Update to SAM 2

Will you please consider to update this wonderful project to use SAM 2 ?
I'm not a programmer but Meta just released the model:
Meta's SAM 2:
https://ai.meta.com/sam2

Hi, I've been watching SAM2, and it definitely looks impressive. I am not sure how difficult it would be to implement into this project. It might make more sense to build an entirely new application around it.
However, I am currently quite busy working on some other projects, probably through the end of the year, so I don't have time to commit to it right now. I will be keeping my eyes open to see if anyone else starts working on a similar project. If not, then I might consider it early next year.


Shown() event not fired for delegated targets

Hi, first of all thanks a lot for a great plugin. Really helpful.
I've just come across a situation where i need to run a function whenever the dialog and form is shown.
The shown() event works great but while working with delegated targets, the shown event doesn't seems to work for elements added on the fly. I tried adding the shown() event to the fiddle you already had on your site http://jsfiddle.net/xBB5x/1515/
Am i doing it the wrong way? If not is there any fix for this?

I've just run up against this bug, and it's preventing me from doing something pretty important for my use case.  I know it's been awhile since this was opened, but I figured I would check in and see if there's any update on this issue?

@sdishman
https://github.com/vitalets/x-editable/issues/86


[Bug report] Active header link plugin is not working

Description
See https://ecosystem.vuejs.press/
Reproduction
https://ecosystem.vuejs.press/
Used Package Manager
pnpm
System Info
Not needed

This is a upstream bug at core, no need to fix here.


Bug: ArkOS strange new behaviours (e.g. roms not appearing, cannot add favourites)

I have been using ArkOS problem free for many weeks now. In the last few days, I have noticed some strangeness with the OS, including:

Roms added to the SD card failing to appear
Favourites added disappear from the favs list once I restart the system
Last Played no longer updates with recently played games
I delete roms from within ArkOS but they reappear once I restart the device.
Some roms now appear twice.

Do you have any suggestions as to what the problem may be?
Thank you.

From the emulationstation menu (start button), go to advanced settings, parse gameslists only, and make sure that setting is set to off.

Any update on if the above resolved your issue?

Hi Christian,
Thank you for chasing this up! It did fix the issue that new roms loaded
onto the SD card were not being recognised. I can also now delete games.
However, I still have the same problem that I can no longer add to the
favourites list.
On another note, there are some random oddities I have noticed with the OS.
When plugging in a charger, the battery charge level jumps to a new value
immediately (not the same number shown when on battery power).
Also, on occasion the cursor will move several steps up or down by itself
when in emuelec menus.
Thanks for the continued support,
Mark
On Mon, 22 Feb 2021 at 17:54, christianhaitian<EMAIL_ADDRESS>wrote:

Any update on if the above resolved your issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/christianhaitian/arkos/issues/246#issuecomment-783248328,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASIW7F7RY5VNMNQC6YCFWZLTAISV5ANCNFSM4XUECOCA
.

A final bug I have encountered several times when attempting to update
ArkOS: The device sometimes hangs on the 'Once you proceed...' warning page
before I can click 'ok'.
On Mon, 22 Feb 2021 at 20:19, Mark<EMAIL_ADDRESS>wrote:

Hi Christian,
Thank you for chasing this up! It did fix the issue that new roms loaded
onto the SD card were not being recognised. I can also now delete games.
However, I still have the same problem that I can no longer add to the
favourites list.
On another note, there are some random oddities I have noticed with the
OS. When plugging in a charger, the battery charge level jumps to a new
value immediately (not the same number shown when on battery power).
Also, on occasion the cursor will move several steps up or down by itself
when in emuelec menus.
Thanks for the continued support,
Mark
On Mon, 22 Feb 2021 at 17:54, christianhaitian<EMAIL_ADDRESS>wrote:

Any update on if the above resolved your issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/christianhaitian/arkos/issues/246#issuecomment-783248328,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASIW7F7RY5VNMNQC6YCFWZLTAISV5ANCNFSM4XUECOCA
.

The power issue is a known issue due to the kernel.  See here for more information on why that happens.  The hanging issue will happen if you keep the unit plugged into power and using wifi on the 351p.


Ur insane.

What are you? This is insane stuff, thanks sm for the release, really helped me!

facts


Added a Canary/Prod Sample

Added an ADO sample for a Multiple Environment implementation.
I have a public ADO repo where this has been implemented https://gdoggmsft.visualstudio.com/_git/EntScaleAzOps

I'd be keen to roll the use of Azure DevOps templates into the starter example pipelines, but happy to pick this one up separately. 😀


Releases for Java 12 and Java 13

Hi,
Are there any plans for Java 12 and/or Java 13 releases, and if so what's the likely timescale?
Also is there a particular reason there is no alpine release for Java 11?

For why no openjdk 11 on Alpine: https://github.com/docker-library/openjdk/issues/272.
On the subject of openjdk 12 and 13, those change to be from oraclelinux:7-slim with no Debian version, so there would possibly be a huge shift in the image.

thank you

I've pushed up a bit of a WIP at https://github.com/docker-library/tomcat/compare/master...infosiftr:oracle (https://github.com/docker-library/tomcat/commit/2c1605d637e0b765fabb57fe42484e53d317df40 is a direct link to the current commit in case the branch ends up going away later).
The biggest TODO there is Tomcat Native support (because I haven't had time or motivation to dig into what packages are necessary to make that work -- help wanted!)

i think it was a bad decision to drop alpine image, we had to move to stretch (with very poor and painful java12 support) and now have to shift to oracle linux which is also not the first choice and breaks all the derived images and build process. I think most user prefer ubuntu or centos based image (and for me personally alpine with jdk13ea).

JDK 12 and 13 are being added in https://github.com/docker-library/tomcat/pull/164 :+1:


feat(trin-execution): allow ctrl+c to immediately stop execution

What was wrong?
Using Ctrl+C to stop trin_execution can take a lot of time, partially due to committing to db (which can't be avoided), and partially because execution doesn't stop until next checkpoint (which can take another 30 min).
How was it fixed?
Refactor how TrinExecution executes blocks (built on top of #1450).
Instead of executing until the first commit (which can come early), it will now execute all the way to the provided block but it accepts stop_signal receiver. When stop signal is received, it will commit and return.
Note: Tested it locally (around block 6'000'000).
Future ideas
We can speed up the shutdown by abandoning changes that are in memory (by not committing anything), but we would have the same problem that is described in #1440.
Even if implemented, it's questionable if this is desired behavior, so maybe we would need a flag that controls whether to commit on ctrl+c or not.
To-Do

[ ] Clean up commit history and use conventional commits.

Future ideas
We can speed up the shutdown by abandoning changes that are in memory (by not committing anything), but we would have the same problem that is described in #1440.
Even if implemented, it's questionable if this is desired behavior, so maybe we would need a flag that controls whether to commit on ctrl+c or not.

https://github.com/ethereum/trin/issues/1440#issuecomment-2346615788 as said in my  comment here we would want to switch to an ACID compliment database.
We want ctrl+c to instantly shutdown the node, as we want to prevent the Operating System from force killing trin-execution on shutdown, which could corrupt the database. Switching to an ACID compliment database, resolves this class of problems to my knowledge. We should be able to shutdown even if we are in the middle of writing to the database, as we could set it to only consider the writes valid when we are done them.
 LMDB or MXDB solves a lot of these problems

ACID
In computer science, ACID is a set of properties of database transactions intended to guarantee data validity despite errors, power failures, and other mishaps. In the context of databases, a sequence of database operations that satisfies the ACID properties is called a transaction.

This solves that whole* class of problems


Memory issue?

Hi,
Thanks for sharing your code.
I got "scripts/ddpm/train_interpreter.sh: line 6:  3842 Killed   python train_interpreter.py --exp experiments/${DATASET}/ddpm.json $MODEL_FLAGS" error.
I have ~65G RAM available on my Ubuntu. Considering your note that "it requires ~210Gb for 50 training images of 256x256."
Does it mean that it's not feasible to train the model on my system? How about evaluation?
Thanks in advance.

You can try to i) reduce the number of training images ii) reduce feature dimensionality iii) store features on the disk and preprocess them by batches.

You can try to i) reduce the number of training images ii) reduce feature dimensionality iii) store features on the disk and preprocess them by batches.

Hi, thanks for the wonderful work and codes you provided. Can you explain how would I implement the last one (iii)? How do I process the features by batches if they are too huge that I cannot even store them on the disk? I have encounted the same problem as @HOMGH and the extracted pixel representations are so huge that before the prepare_data() function return the data, the process I'm running the script will be killed. For example, when I run ddpm on the cat_15 dataset using the original experiment setting with 30 training images, when the program tries to run the following two lines, the process will crash.
X = X.transpose(1,0,2,3).reshape(d,-1).transpose(1,0)  # Here X with size of [30,8448,256,256] is converted to size [30*256*256, 8448]
y = y.flatten()

My solution is to write another prepare_data() function, which processes one image instead of all labelled training images at one time. Then during the training of the pixel classifier, at each epoch I will create a dataloader for one image and iterate through all training images. But there exists some gap between the final evaluation results I got and yours. Do you have any suggestions for that?
Thanks a lot for your help.

You can try to i) reduce the number of training images ii) reduce feature dimensionality iii) store features on the disk and preprocess them by batches.

Hi, thanks for the wonderful work and codes you provided. Can you explain how would I implement the last one (iii)? How do I process the features by batches if they are too huge that I cannot even store them on the disk? I have encounted the same problem as @HOMGH and the extracted pixel representations are so huge that before the prepare_data() function return the data, the process I'm running the script will be killed. For example, when I run ddpm on the cat_15 dataset using the original experiment setting with 30 training images, when the program tries to run the following two lines, the process will crash.
X = X.transpose(1,0,2,3).reshape(d,-1).transpose(1,0)  # Here X with size of [30,8448,256,256] is converted to size [30*256*256, 8448]
y = y.flatten()

My solution is to write another prepare_data() function, which processes one image instead of all labelled training images at one time. Then during the training of the pixel classifier, at each epoch I will create a dataloader for one image and iterate through all training images. But there exists some gap between the final evaluation results I got and yours. Do you have any suggestions for that?
Thanks a lot for your help.
Hi，
Can you send me a copy of your code, I really need your code,
thank you very much！

You can try to i) reduce the number of training images ii) reduce feature dimensionality iii) store features on the disk and preprocess them by batches.

Hi, thanks for the wonderful work and codes you provided. Can you explain how would I implement the last one (iii)? How do I process the features by batches if they are too huge that I cannot even store them on the disk? I have encounted the same problem as @HOMGH and the extracted pixel representations are so huge that before the prepare_data() function return the data, the process I'm running the script will be killed. For example, when I run ddpm on the cat_15 dataset using the original experiment setting with 30 training images, when the program tries to run the following two lines, the process will crash.
X = X.transpose(1,0,2,3).reshape(d,-1).transpose(1,0)  # Here X with size of [30,8448,256,256] is converted to size [30*256*256, 8448]
y = y.flatten()

My solution is to write another prepare_data() function, which processes one image instead of all labelled training images at one time. Then during the training of the pixel classifier, at each epoch I will create a dataloader for one image and iterate through all training images. But there exists some gap between the final evaluation results I got and yours. Do you have any suggestions for that?
Thanks a lot for your help.

Hello ,@Yi-Lynn  , can you send me your code ?
thank you .

@Yi-Lynn Could you send me your code? please?
@MariemOualha Did you implement? If you have implemented, Could you send me the code?

@Yi-Lynn Could you send me your code? please? @MariemOualha Did you implement? If you have implemented, Could you send me the code?

@SahadevPoudel Hi, Poudel! I had the same problem, and I succeeded in implementing it in the way shown above. The newly proposed method declares a class called 'DividedImageLabelDataset' from src/datasets.py , which is imported from the train code and used. Comment if you still want code about it!

@Yi-Lynn Could you send me your code? please? @MariemOualha Did you implement? If you have implemented, Could you send me the code?

@SahadevPoudel Hi, Poudel! I had the same problem, and I succeeded in implementing it in the way shown above. The newly proposed method declares a class called 'DividedImageLabelDataset' from src/datasets.py , which is imported from the train code and used. Please comment if you still want code about it!

Hello! I'm experiencing the same problem, but I don't know what I should do to fix it. Can you share the code you modified? Thanks a lot.
This is my email address<EMAIL_ADDRESS>Or you can choose other methods that are convenient for you, thanks again.


Process cleanup problems

$ ps; [status get; Zombie =] grep;
v[gen]
$ {sleep 1}
v[command-gen]
$ ps; [status get; Zombie =] grep;
v[gen (
    0: h(
        "pid":     450868
        "uid":     0
        "gid":     0
        "user":    root
        "name":    sleep
        "cmd":     ""
        "cpu":     0
        "mem":     0
        "vmem":    0
        "runtime": 2
        "status":  Zombie
        "start":   v[datetime 2023-05-26 21:41:13 UTC]
    )
)]
$

The process should be cleaned up properly.

This has since been fixed.


Enabling traffic towards multiple proxy hosts domains with the same name, but listening on different ports

I have a privately hosted domain abc.com. I would like to be able to create multiple proxy hosts domains with the same name, but listening on different ports (for http and also https connections). This comes in handy, when you do only have 1 domain on which you would like to access several ports (both http and https).
What I would like within this feature:
-> to be able to configure multiple proxy hosts with the same domain name and be able to specify several ports for that specific domain which are either listening for http or https connection.
Example:
proxy host0: abc.com, listening for http on port 8005, 8006, 8007 and for https on 8008 and 8009 -> redirects traffic to localhost:8010
proxy host1: abc.com, listening for http on port 8100 and for https on 8200 -> redirects traffic to localhost:8011
Alternatives I considered, but are impossible, because of how NPM is built: manually modifying the .conf files. Not possible, because even slightly editing a proxy host leads to the automated rewriting of the .conf file.
Additional context: I tried manually modifying the .conf file, but they do rapidly get overwritten, when performing additional settings in the app. The NPM section which manually permits you to write Nginx configurations does not help, due to the fact that I do need to actually delete the "listen 80" and "listen 443 ssl httpd" lines. Currently, NPM does not allow that. I would like the NPM to be more customizable, because it is such a great tool...it's a shame.

Should there be a work-around for my problem, please enlighten me. However, I hardly believe there is one; the internet says there is none.

I would also like this. Confused that this may not work anymore...?
https://www.reddit.com/r/homelab/comments/16ftare/nginx_proxy_manager_using_multiple_listen_ports/


add a memory safety warning in the README

See #9

As we have a proper fix, with documentation, is this still needed?

we can close this :)
But I think we need to reopen #9 until it is properly fixed (upstream, probably)


README + remove unnecessary script

I guess I should have copied the readme section from here: https://github.com/healthforge/buildkite-key-rotator#multikey

Rebased and combined the readme contents. Looks good. i'm going to merge it


Documentation: v1.0

Update the following pages of the Developer Guide:

Target user profile, value proposition, and user stories: Update the target user profile and value proposition to match the project direction you have selected. Give a list of the user stories (and update/delete existing ones, if applicable), including priorities. This can include user stories considered but will not be included in the final product.
Use cases: Give use cases (textual form) for a few representative user stories that need multiple steps to complete. e.g. Adding a tag to a person (assume the user needs to find the person first)
Non-functional requirements:
Note: Many of the project constraints mentioned above are NFRs. You can add more. e.g. performance requirements, usability requirements, scalability requirements, etc.
Glossary: Define terms that are worth defining.
[Optional] Product survey: Explore a few similar/related products and describe your findings i.e. Pros, cons, (from the target user's point of view).

Reference: W7.1: Can record requirements of a product

@zhangriqi added the new User Stories in Pull Request #39.

Resolved through 2 pull requests: #39 and #43.
Issue closed.


Orders page

We want to eventually allow ordering circuit boards from tscircuit snippets, so this issue tracks all the stuff to make that happen.
Note that the orders api backend isn't open-source because it contains code from our integration partners that is locked down, but the fake version is open-source, as well as the ordering interface.

[ ] Orders and order files schema
[ ] /orders/* and /order_files/* fake endpoints
[ ] My orders page /my-orders
[ ] Add header option to see orders by clicking your profile picture
[ ] View order page /orders/{order_id}
[ ] New order page /new-order

Use AI to generate the pages and be relatively descriptive about the contents, it'll generate something decent if you add the api into the context /add src fake-snippets-api

Some of the items were implemented in https://github.com/tscircuit/snippets/pull/46


GH-3407 : Support KafkaHeaders.DELIVERY_ATTEMP for batch listeners.

Motivation:
The developer expects to get the KafkaHeaders.DELIVERY_ATTEMPT header from the received ConsumerRecord.
It works well with the SingleRecordListener.
However, it does not work well with the BatchListener even when containerProperties.setDeliveryAttemptHeader(true) is set.
If the KafkaContainerListener can include the KafkaHeaders.DELIVERY_ATTEMPT header for each ConsumerRecord with the BatchListener, it will be more helpful to people who they expect.
Modifications

Add DeliveryAttemptAwareRetryListener class.
Add unit tests and integration tests for DeliveryAttemptAwareRetryListener.

To Reviewer
With current spring-kafka codes, it is hard to integrate it to FailedRecordTracker.
By the way, the function ErrorHandlingUtils.retryBatch() has deliveryAttempt count as their local scope and has same reference for ConsumerRecord.
Thus, the DeliveryAttemptAwareRetryListener is easier way to achieve the goal.
Because, in each fail to invocation, ErrorHandlingUtils.retryBatch() calls listeners.forEach(listener -> listener.failedDelivery(records, thrownException, attempt).
If the developer want to use this feature, they should create ContainerFactory and DeliveryAttemptAwareRetryListener. also, they set DeliveryAttemptAwareRetryListener as RetryListener.
Please refer to code below.
final FixedBackOff fixedBackOff = new FixedBackOff(1000L, MAX_ATTEMPT_COUNT1);
DefaultErrorHandler errorHandler = new DefaultErrorHandler(fixedBackOff);
errorHandler.setRetryListeners(new DeliveryAttemptAwareRetryListener());

ConcurrentKafkaListenerContainerFactory<String, String> factory = new ConcurrentKafkaListenerContainerFactory<>();
factory.setConsumerFactory(consumerFactory);
factory.setCommonErrorHandler(errorHandler);

Result:

Fixes https://github.com/spring-projects/spring-kafka/issues/3407

@artembilan thanks for your time and comments!
I made a new commit to apply your comments.
When you have time, please take another look 🙇♂

@artembilan thanks for your time for looking into changes, again! 🙇♂
I made a new commit to apply your comments.
When you have time, please take another look!
Always thanks to read my code 🙇♂

@artembilan , It's my bad!
I cleaned up the code that I thought was useless.
When you have time, please take another look 🙇♂

Thank you, @chickenchickenlove , again for contribution!
Please, take a look into this article about commit messages: https://cbea.ms/git-commit/.
Also take into account that first commit message becomes PR description automatically.
Just for a sake to save some yours and ours time:

first - no need to write separate PR description
second - no need to write extra commit message on merge
and last - the commit message is exactly what we always need to be good. The PR easily might be lost, but commit is there forever.

Now imaging time is passing and we don't have contacts with you to remind us whatever was done in that commit years ago.
but if you write a good commit message, it will be merged there into a history with respective honor outcome.

Thanks for your time 🙇♂
As you mentioned, I will try to write good commit message 👍


Make Redis default for secured deployment of EdgeX

🚀 Feature Request
Relevant Package

 Redis
This is part of #2270
Description

Add hooks to support secure EdgeX deployment with Redis
Describe the solution you'd like
 If you have a solution in mind, please describe it.
Describe alternatives you've considered
 Have you considered any alternative solutions or workarounds?

Please assign to @andresrinivasan

There are no further outstanding PR's/tickets for redis integration. Closing for now; reopen if closing in error.


Consider changing default value of requireCleanWorkingDir

I was surprised to see temporary, not excluded, files in the repository after releasing. I do understand that clean working directory is desirable, but may not be the case always. Instead of automatically adding these files and directories to the repository I would rather expect to see an error.
My proposal is to change the default value of requireCleanWorkingDir from false to true.

This has been the default since the beginnings. I simply found it useful at the time (as I tend to have a clean state and all excludes in .gitignore), but also agree it's probably the wrong default for newcomers. Once one wants to have additional changes to a release commit, this default setting can be overridden.
Having that said, the only thing before checking the clean working dir, is execution of src.beforeStartCommand. So probably the repo isn't in a clean state before release-it is started. Shouldn't those files also be git-ignored?
I'll leave this ticket open and change this default in the next major release. Until then, requireCleanWorkingDir can be overridden to true.

In version 2 of this tool I didn't notice this behavior. I have changed the configuration of my repos, but think a change of the default in next major release makes sense.

Nice to meet long-time users here :-)
Seems the option was introduced in v2.1.0, with false being the default (for backwards compatibility).

I've just released this feature in 6.0.0-alpha.0. Would be great if you could try this on your workload.
(It also features a new requireUpstream to allow releasing from a new local branch.)


Name sandboxes archive after Jenkins job.

Summary:
Currently the sandboxes archive is called sandboxes.tar.gz. To make
debugging a little easier this patch will rename it to ci-<Jenkins job>-sandboxes.tar.gz.

Since we're changing the names - could we please simplify it to `ci-PR-NNNN-1.*?


Add twitter:card

Hello! It's a really useful module, thanks!
Could you add twitter:card option? From https://developer.twitter.com/en/docs/twitter-for-websites/cards/guides/getting-started#started
This is an example:
{
  hid: 'twitter:card',
  name: 'twitter:card',
  content: 'summary_large_image',
}

Updated with the newest release! :D


style: update background color

Description
Background color update
We probably need to update the color of the image in the second screenshot.
Task Item
Closes #1903
Screenshots

Awesome! looks good
m


Adds charmcraft publish action

Adds a github action that will build the charm and will push it to charmhub or manual dispatch or when a PR merges.

@pedroleaoc This action requires the CHARMHUB_TOKEN secret to be added to this repo (see the added docs), so this action will be able to build and publish the charm, since the charm ownership was transfered to you. Can you merge this PR when that token has been added?


Add EKS Integration

First iteration of the EKS Integration (as requested in #214), which pulls clusters and their main attributes.
Next iterations will include relations with VPCs and analysis on the data (i.e., define internet_exposed)

Yay!


grantPermission doesn't work for Photos permission

Discussed in https://github.com/leancodepl/patrol/discussions/563

Originally posted by yutanui November  4, 2022
Hi Patrol team,
Thank you very much for Patrol package.
I have checked and tried to utilize the package instead of integration_test for permission grant.
Anyway, With your example, I tried to uitlize it but still cannot perform dialog permission for photos such as (All Photos) with iOS simulator (I have tested with iOS 15.0)
Sample of code as below.  (I have set up all required configuration for permission_handler library already)
Future<void> requestAndGrantPhotosPermission(PatrolTester $) async {
    expect($(#photo).$(#statusText).text, 'Not granted');

    await $('Request photo permission').tap();

    final request = await $.native.isPermissionDialogVisible(
      timeout: Duration(seconds: 2),
    );
    if (request) {
      await $.native.grantPermissionWhenInUse();
    }

    await $.pump();
    expect($(#photo).$(#statusText).text, 'Granted');
  }

Could you please suggest what resource should I looking for or provide an example of photos permission with Patrol package ?
Thank you very much

Looks like it's been resolved in #568 :)


broadcast

Original issue: https://charm.cs.illinois.edu/redmine/issues/605

No body.

Original date: 2015-02-09 22:11:53

This test basically does a point to point broadcast and measures the time taken for it.
Array element 0 sends point to point messages to other array elements, waits to receive a ack and then measures the time for point to point broadcast.

Original date: 2015-02-09 22:26:25

I don't think this test needs to be in the autobuild.

Original date: 2015-02-10 15:14:14

If it's not a useful test, then please delete it.


docs: add --libraries section

I updated the file mentioned above on master, let me know if that works for you :slightly_smiling_face:

@all-contributors add @mattsse for maintenance, docs


zpaq algorithm does not appear to match impl in zpaq upstream

Hi, I was looking at the chunking provided by Zpaq in this crate (as part of comparing it to a zpaq implimentation I've written), and noticed that it doesn't appear to match up with the upstream zpaq code.
In particular, it appears that the hash update equation doesn't match up (though I might be missing some transform you've done).
cdchunking code:
const HM: Wrapping<u32> = Wrapping(123_456_791);
...
pub fn update(&mut self, byte: u8) -> bool {
        if byte == self.o1[self.c1 as usize] {
            self.h = self.h * HM + Wrapping(byte as u32 + 1);
        } else {
            self.h = self.h * HM * Wrapping(2) + Wrapping(byte as u32 + 1);
        }
        self.o1[self.c1 as usize] = byte;
        self.c1 = byte;

        self.h.0 < (1 << self.nbits)
}

zpaq code
            if (c==o1[c1]) h=(h+c+1)*314159265u, ++hits;
            else h=(h+c+1)*271828182u;
            o1[c1]=c;
            c1=c;

In particular, zpaq adds h, c, and 1 prior to multiplying (and uses different constants), but cdchunking adds c and 1 after h is mulitiplied.
I've done some slight modification to zpaq itself so that it prints chunk lengths, and I'm not seeing a correspondence between the lengths produced by zpaq and cdchunking.
diff --git a/zpaq.cpp b/zpaq.cpp
index f468a7b..84e1f5f 100644
--- a/zpaq.cpp
+++ b/zpaq.cpp
@@ -83,6 +83,7 @@ Possible options:
 #endif
 #ifdef unix
 #define PTHREAD 1
+#include <inttypes.h>
 #include <sys/param.h>
 #include <sys/types.h>
 #include <sys/stat.h>
@@ -2416,6 +2417,8 @@ int Jidac::add() {
         assert(sz<=MAX_FRAGMENT);
         total_done+=sz;

+       printf("## %" PRIu64 "\n", sz);
+
         // Look for matching fragment
         assert(uint64_t(sz)==sha1.usize());
         memcpy(sha1result, sha1.result(), 20);

Is this difference in algorithm an oversight?
Or is it purposefully different?
If it is purposefully different, is it following another already published source?
If it is purposefully different, could the divergence be documented?

Honestly I cannot remember where I got the code. I don't think it was the official libzpaq source.
The difference is unintended, and I think we should use the official algorithm since it is named in the API. That would be a breaking change, but I don't know if there are any real users of this crate (I wrote it mostly for a proof of concept, I never got around to the project that was supposed to use it).
I suppose the crate should also have test vectors generated from the C++ implementation.

Specification appears under "Implementation notes" in zpaq206.pdf
@jmesmon Do you have test vectors that you have checked with the C++ implementation or your own?

I've only generated 2 test vectors, both use 32KiB of data generated with Pcg64 as the input and a list of chunk lengths as the output.
generator: Pcg64::new(0, 0xa02bdbf7bb3c0a7ac28fa16a64abf96)
zpaq-fragment: 3
length: 32768
cuts: [10785, 6329, 1287, 860, 4716, 7419]

generator: Pcg64::new(3, 0xa02bdbf7bb3c0a7ac28fa16a64abf96)
zpaq-fragment: 3
length: 32768
cuts: [16353, 2334, 970, 5326, 1557]

note:

that the really long number passed to Pcg64 is the specified default Pcg64 stream, I'm just tweaking the seed
zpaq (the c++ code) "fragment: 3" corresponds to "ZPAQ::new(13)" in your implementation (zpaq c++ works in KiB)
I've used the same length for both of these

The code exists in this test in this branch of my repo.
That branch also has code to generate a binary file that can be passed to zpaq.

I made the change and imported your tests as best I could, but the results don't match exactly (left is cdchunking, right is hash-roll):
---- tests::test_long_0 stdout ----
thread 'tests::test_long_0' panicked at 'assertion failed: `(left == right)`
  left: `[10785, 6329, 1287, 860, 97, 4619, 7419, 1372]`,
 right: `[10785, 6329, 1287, 860, 4716, 7419]`', src/lib.rs:659:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- tests::test_long_3 stdout ----
thread 'tests::test_long_3' panicked at 'assertion failed: `(left == right)`
  left: `[16353, 2334, 970, 5326, 1557, 6228]`,
 right: `[16353, 2334, 970, 5326, 1557]`', src/lib.rs:676:9

The last number is probably because you're only listing the cut points and not all chunk sizes, but this 97 is suspicious.

yep, minimum chunk size is likely the distinction here. zpaq c++ includes a minimum and maximum chunk size. zpaq c++ calculates it from the selected average chunk size (ie: "bits", 13, etc).
https://github.com/zpaq/zpaq/blob/9ab539f644e364f0d92e2918b90ce2534c75653f/zpaq.cpp#L2411-L2414
          if (c==EOF
              || sz>=MAX_FRAGMENT
              || (fragment<=22 && h<(1u<<(22-fragment)) && sz>=MIN_FRAGMENT))
            break;

https://github.com/zpaq/zpaq/blob/9ab539f644e364f0d92e2918b90ce2534c75653f/zpaq.cpp#L2198-L2201
  const unsigned MAX_FRAGMENT=fragment>19 || (8128u<<fragment)>blocksize-12
      ? blocksize-12 : 8128u<<fragment;
  const unsigned MIN_FRAGMENT=fragment>25 || (64u<<fragment)>MAX_FRAGMENT
      ? MAX_FRAGMENT : 64u<<fragment;

(imo: the calculations here are pretty unreadable)

I added a warning and released 1.0.1. I don't think it's worth matching the official constants, and this crate is unused as far as I know.
You seem to be doing a much more conscientious job with hash-roll, which should probably be used instead of cdchunking if people want those well-known algorithm you implemented to specs (although your Splitter trait doesn't seem written for streaming or memory-constrained operations, it needs to collect a whole chunk in a Vec).

Understood, thats fine with me.
On hash-roll: the stuff in master is pretty old. the next branch is a complete rewrite which adjusts the apis a bit (and entirely deletes Splitter). It does include some apis that support some streaming type operations, though they need a bit more polish (imo).
The some of the APIs you've written for cdchunking are things I'm keeping in mind as I'm building easier-to-use abstractions over the current low-level hash-roll::next APIs (like fn find_chunk_edge(&self, state: &mut Self::SearchState, data: &[u8]) -> (Option<usize>, usize);. No one really wants to use an api like that)
The API you've used for


Donation

Hey, I've started using the library and it saved me a lot of time.
Is there a way to donate?
Something symbolic, to buy a drink for the contributors.

Hey, you were the first one that asked! Great to get that feedback, I appreciate that a lot. That saved my day ...
If you want to support something chess-related, please do that for lichess. That is the greatest platform ever for playing chess, and I use their board named Chessground in my PgnViewerJS.
I don't think I will add the option to donate something to me, but I like your feedback. Just give me a star ;-)

Also using Chessground and did donate to them, so got it covered :)
I see, in this case all I can say then is keep up with good job. Starred the repo.


Feat Containerize the application to improve cross OS compatibility

🚀 PR Summary:
Adds a Dockerfile to support containerization as part of https://github.com/OthersideAI/self-operating-computer/issues/36.
🛠 Changes Made:

Included Dockerfile for containerization.
Used Python:3.11-slim as the base image (Considering it's lightweight and supports all libs).

🔍 Testing:

Tested successful build and execution using the Dockerfile.

👩💻 Next Steps:

Review and merge if no issues arise.

🔧 How to Build and Run:

Build the Docker Image:

docker build -t your_image_name .

Run the Docker Image:

docker run -it your_image_name

@legendkartik45 thanks for the PR. I don't know Docker super well. I'll take a look do some learning and let you know if I have any questions! :)

@legendkartik45 let me know your thoughts on the question above. Thanks

It's going to be challenging to get this to work in a container due to the GUI limitations. What I recommend is using a VM instead. For example, you could create one inside your own desktop or provision one through a cloud provider.

@tomasfernandez1212 that's my understanding as well

@joshbickett @tomasfernandez1212 I believe that makes sense, let me explore running on local VM and get back.

@legendkartik45 curious if you have any findings. Thakns

Hi @legendkartik45. Let me know if you found anything on this PR.
I'll close this PR for now until we know more. If you have any updates feel free to reopen the PR


Deprecated lifecycle method warnings with React 16.9

The following warning is shown upon upgrading to React 16.9.0:
backend.js:1 Warning: componentWillMount has been renamed, and is not recommended
for use. See https://fb.me/react-async-component-lifecycle-hooks for details.

* Move code with side effects to componentDidMount, and set initial state in the
constructor.
* Rename componentWillMount to UNSAFE_componentWillMount to suppress this warning
in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all
deprecated lifecycles to their new names, you can run
`npx react-codemod rename-unsafe-lifecycles` in your project source folder.

Please update the following components: Draggable

Yeah, I'm seeing the same if I have time I can work on a PR for it sometime if somebody doesn't beat me to it 😄

Please

If you could fix this, that would be hella lit boi

@STRML, Thanks for the quick fix for componentWillMount.
However, componentWillReceiveProps and componentWillUpdate are also deprecated.  Sorry, I should have included those in my original comment.

Any status on this? I still receive the warning.
Warning: componentWillReceiveProps has been renamed, and is not recommended for use. See https://fb.me/react-unsafe-component-lifecycles for details.

* Move data fetching code or side effects to componentDidUpdate.
* If you're updating state whenever props change, refactor your code to use memoization techniques or move it to static getDerivedStateFromProps. Learn more at: https://fb.me/react-derived-state
* Rename componentWillReceiveProps to UNSAFE_componentWillReceiveProps to suppress this warning in non-strict mode. In React 17.x, only the UNSAFE_ name will work. To rename all deprecated lifecycles to their new names, you can run `npx react-codemod rename-unsafe-lifecycles` in your project source folder.

Please update the following components: Draggable

Same for me, I still get the warning:

I'm using React 16.13.1 and react-draggable 4.3.1


Added split by regex functionality

Allows input to be parsed using a regex while the output still uses the delimiter. Very useful for cases when the input file has an indeterminate amount of white space as the column separator. Code is not elegant and should be refactored. Example file 'marks.txt' added to docs folder and example usage added to README.md

Supporting division using regular expressions is a great idea. We've received a similar issue #8 recently that addresses this concern. In that issue, there was a suggestion to enhance the existing -d option, allowing it to accept regular expressions. If the intent is to facilitate division, I believe it's better to extend the -d option rather than adding a new option. For instance, how about the following:

If the value of the -d option has two or more characters, treats it as a regular expression for division.
rec = pattern.split(line)

Adding a new option, -R REGEX, --regex REGEX, specifically for matching using regular expressions rather than division.
rec = pattern.findall(line)

I think it's a good practice to pre-compile the regular expression outside of the loop, like this:
pattern = re.compile(REGEX)
Usage examples:
$ echo "aaa    bbb ccc   ddd" | ppp rec -d '\s+' rec[0]
aaa

$ echo "aaa    bbb ccc   ddd" | ppp rec -R '\w+' rec[0]
aaa

Since '\s+' is a common choice for regular expression division, adding an option like -S, --spaces with the same meaning as -d '\s+' would be beneficial, in my opinion:
$ echo "aaa    bbb ccc   ddd" | ppp rec -S rec[0]
aaa

I see the value of precompiling regexes or any other refactoring or
improvements to the code as I just hacked something together quickly that
worked for me.
I also originally wanted to reuse the -d option, but the problem is the
output when you have more than one column in the output.
echo "aaa    bbb ccc   ddd" | ppp rec -d '\s+' rec[0:3']
aaa\s+bbb\s+ccc
Most use cases don't want to see a regex in the output, and it would
be nice to be able to choose what the delimiter in your output is.
Having said that, the -S, --spaces you suggest would probably handle
95% of the cases where people need to resort to regexes and keep
things simple.
On Wed, Oct 25, 2023 at 1:42 AM bugen @.***> wrote:

Supporting division using regular expressions is a great idea. We've
received a similar issue #8 https://github.com/bugen/pypipe/pull/8
recently that addresses this concern. In that issue, there was a suggestion
to enhance the existing -d option, allowing it to accept regular
expressions. If the intent is to facilitate division, I believe it's better
to extend the -d option rather than adding a new option. For instance, how
about the following:

If the value of the -d option has two or more characters, treats it as
a regular expression for division.
rec = pattern.split(line)
Adding a new option, -R REGEX, --regex REGEX, specifically for
matching using regular expressions rather than division.
rec = pattern.findall(line)
I think it's a good practice to pre-compile the regular expression outside
of the loop, like this:
pattern = re.compile(REGEX)
Usage examples:
$ echo "aaa    bbb ccc   ddd" | ppp rec -d '\s+' rec[0]
aaa
$ echo "aaa    bbb ccc   ddd" | ppp rec -R '\w+' rec[0]
aaa
Since '\s+' is a common choice for regular expression division, adding an
option like -S, --spaces with the same meaning as -d '\s+' would be
beneficial, in my opinion:
$ echo "aaa    bbb ccc   ddd" | ppp rec -S rec[0]
aaa
—
Reply to this email directly, view it on GitHub
https://github.com/bugen/pypipe/pull/3#issuecomment-1778694718, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWUUH4JERXXC2S4SDYZJ3YBC7HBAVCNFSM6AAAAAA6M2WPLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZYGY4TINZRHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Confusion about the Note around CDN availability for government cloud

I have had some confusion from GCC customers (they are in the commercial cloud, but a logically separated data enclave).  Can we call out in the note that the CDN is available for Commercial and GCC customers to distinguish.  Thanks!

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

ID: c8cfe2b0-c14d-1083-7c88-a91e58593cfa
Version Independent ID: 2d15076c-0dcd-6bd6-e0fb-f8ac5ffd0091
Content: Use Office 365 Content Delivery Network (CDN) with SharePoint Online - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/use-microsoft-365-cdn-with-spo.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@LauraWi Please help us in resolving this issue. Thanks

@andydlayton Thank you for your feedback. There is already a note which states The Office 365 CDN is only available to tenants in the Production (worldwide) cloud - This is the commercial cloud.
Tenants in the US Government, China, and Germany clouds do not currently support the Office 365 CDN.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @andydlayton Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Clean up code and add noise classes.

i'll accept the noise classes as working for now without testing them, playing with them is for after the pr is merged

Fixed requests but you still need to implement vector classes.


no HTTP endpoints found in dev branch

Ralph found a bug where the HTTP module fails due to querying the ManagementPoints table. If the AD schema isn't extended HTTP checks fail.

Updated HTTP module to check the computers table for existing management points.


WIP: Basic Support for C++ & Basic Queues

Closing this because it's a very limited implementation and we'll need to make something better for it to actually be useful.


Improve UX of existing widgets

Lets do a second pass of the UI once we have basic functionality in place

This UX review has been completed and has generated several other issues. Additional UX review will be done every time we onboard new users.


Increase batch request limit

Currently by default ethers is using batch requests to receive information about block number and chain id and more. We should either increase our limit or check each request and limit them according to certain tears.

What limit should this be increased to based on observance?

By default ethers(for example) sends batch requests for queries like eth_chainId, eth_blockNumber and more. This can be turned off, when new provider is initialized, however most users won't.
So we either want to increase the limit or make it in tiers based on each request in the batch. Otherwise, even one users with avarage usage will hit the limit.


Only request media for exact match on getUserFeed

When requesting media from getUserFeed, if multiple results are obtained, a 400 error is obtained for those that are not an exact match.

Thanks!


Remove setup-ovs-with-labels.b64 & custom-config.ign

This files are outcome, the ign if outcome of fcc and setup-ovs-with-labels.b64 is base64 of setup-ovs-with-labels.sh

@phoracek @RamLavi


added options to 1) get xml document namespaces recursively and 2) remove namespaces from resulting array

@tamlyn - first, thanks for creating this - has come in very handy.

I have ran into a few xml files that had namespaces that had to be found recursively - and added the option to switch  $recursive parameter of getDocNamespaces to TRUE when option is set.  by default, parameter is set to  FALSE so that the function continues to perform as originally published.
I also just committed another change to add option to remove namespaces in resulting array. Sorry didn't split this up into another branch - as it it now is included in the same pull request.

Up this (I just create similar options and see you gotcha this PR).
Thanks for this usefull function!


Home page

@Shreyashm16  Don't use unnecessary br tag , the footer will automatically comes down after writing the text it self. Anyways, Nice work able to merge.
Thanks!


Coc

Issue #16
Created code of conduct

The PR does not adhere to the guidelines:

Contribution guidelines
Branching conventions
Pull request guidelines

Closing #17 PR


Sampling delay must be greater than 50 ms.

I have installed the last version of this software (CPU Energy Meter v1.2), but when I try to run it using -e option it returns
Sampling delay must be greater than 50 ms.

whatever I use as value.
Below an example.
$ cpu-energy-meter -e=1
Sampling delay must be greater than 50 ms.

$ cpu-energy-meter -e=10000
Sampling delay must be greater than 50 ms.

$ cpu-energy-meter -e=10000000
Sampling delay must be greater than 50 ms.

I think this is a bug.

Right, thanks for reporting. The syntax with = was documented but did not work, only -e100 and -e 100. Now -e=100 works as well.


Add Amazon RedShift Container

I would like a container for amazon redshift.  This was one of the first google hits: https://github.com/GuildEducationInc/docker-amazon-redshift

@bsideup  I noticed https://github.com/HearthSim/docker-pgredshift is similar, but also implements additional functions.
That said, I now suspect https://github.com/localstack/localstack is a better and already supported: https://www.testcontainers.org/modules/localstack/


Discord bot doesnt say anything

i runned the .exe and the source prepared, nothing happen. i got ratted, and my discord server send nothing.

nvm. i fixed it
just  make sure the intentions all are true

uhh what do you mean all intentions are true? Im encountering the same issue right now

@MkjAAA how to make sure the intentions all are true?

@nikhow0 enable them from here https://discord.com/developers/applications/1275104479871635500/bot

Hey, just use the ModdedPysilon version, it covers every bug and it make sure your discord account never get banned.
Leave me a star if you like my project ;)


Add comment re: data availability

Memorializes the issues we've discussed in https://github.com/pangeo-forge/eNATL60-feedstock/issues/2 as a comment in the recipe.
Merging this PR also serves as a way to re-run the recipe (which failed initially due to data availability issues).

/run eNATL60_wtides_1d_mld

/run eNATL60_wtides_1d_tsw60m

/run eNATL60_wtides_1d_tsw600m


Changing state causes viewWillUpdate to be called on all the components instead of only those that actually changed

Hi, very excited about the promise of StateView but it seems like it's not quite working correctly for me. Here's what I have:
import StateView

struct STStashStateViewLabelKey: StateKey {
    var hashValue: Int = 1
}

class STStashChildView: StateView {

    override func render() {

        print("rendering STStashChildView")

        let label = UILabel()
        label.text = self.prop(withValueForKey: STStashStateViewLabelKey()) as! String
        place(label, key: "label") { (make) in
            make.center.equalTo(self)
        }
    }

    override func viewWillUpdate(newState: [String : Any?], newProps: [StateViewProp]) {
        print("viewWillUpdate in STStashChildView: \(newProps)")
    }

}

class STStashStateView: StateView {

    override func render() {

        print("rendering STStashStateView")

        let labelView = place(STStashChildView.self, key: "labelContainerView1") { (make) in
            make.center.equalTo(self)
        }
        labelView.prop(forKey: STStashStateViewLabelKey(), is: self.state["text"] as! String)

        let labelView2 = place(STStashChildView.self, key: "labelContainerView2") { (make) in
            make.center.equalTo(self)
        }
        labelView2.prop(forKey: STStashStateViewLabelKey(), is: "Same Text" as! String)
    }

    override func viewWillUpdate(newState: [String : Any?], newProps: [StateViewProp]) {
        print("viewWillUpdate in STStashStateView: \(newProps)")
    }

}

And I initiate state change like that somewhere else in my code:
let stashStateView = STStashStateView(parentViewController: self)
view.addSubview(stashStateView)
stashStateView.setRootView()

stashStateView.state = ["text": "some new text value"]

after stashStateView state changes I get viewWillUpdate and render calls in STStashStateView and in both STStashChildViews even though only one of them suppose to change due to new props but another one still renders "Same Text" unchanged. Seems like the diff doesn't work correctly or am I missing something?

Hey @alexvbush, you're missing a couple of things that are easy to fix.
First, the StateKey that you use in place() should be an enum. You would think "Okay, what things am I going to pass into subviews from where I am now?" and then for each one of those things, make them an option in an enum you come up with. Something like
enum STStashStateViewKey: StateKey {
    case stringForChildView
}

/* the name of this enum can be anything, it does not have to have "STStashStateView"
anywhere to be used with that view. The convention I have been using is to make a
new StateKey enum at each view with that view's name at the beginning, give it a case
for each one of the things that particular view will pass to its subviews, and move on. */

Once you have that, you can place() a subview and pass it whatever you want under one of the cases from your new StateKey enum (here, stringForChildView is a case). Then use prop(withValueForKey: StateKey) inside that subview with the same case of your StateKey to use whatever was passed in.
This way, you get Xcode autocomplete on the enums and you have a finite, easy to look at list of all the things one view possibly passes to its subviews.
Something like
enum STStashStateViewKey: StateKey {
    case stringForChildView
}

class STStashStateView: StateView {
    override func render() {

        let labelView1 = place(STStashChildView.self, key: "labelContainerView1") { make in
            make.center.equalTo(self)
        }
        labelView1.prop(forKey: STStashStateViewKey.stringForChildView, is: "I'm the first label")
    }
}

/* now that STStashStateView is using the .stringForChildView case
of STStashStateViewKey to pass the string "I'm the first label" to labelView1,
labelView1 can grab the string it's expecting with the same key... */

class STStashChildView: StateView {
    override func render() {

        print("rendering STStashChildView")

        let label = UILabel()
        label.text = prop(withValueForKey: STStashStateViewKey.stringForChildView) as? String
        place(label, key: "label") { (make) in
            make.center.equalTo(self)
            make.size.equalTo(self)
        }

    }
}

/* here, STStashChildView expects a string under the same key */

Now to have a couple of subviews and pass some data to one and some to the other, you can use a couple of instances of STStashChildView, give them strings using the key from  STStashStateViewKey, and decide which string to actually give them by keeping the strings for each in state, and passing one to one and the other to the other.
The next code sample will also show you how to set initial state. You should use this method because it happens at the right time in init.

class STStashStateView: StateView {

    override func getInitialState() -> [String : Any?] {
        return [
            "stringForFirstChildView": "Hello world",
            "stringForSecondChildView": "Today was a good day"
        ]
    }

    override func render() {

        print("rendering STStashStateView")

        let labelView1 = place(STStashChildView.self, key: "labelContainerView1") { make in
            make.top.equalTo(self)
            make.size.equalTo(200)
        }
        labelView1.prop(forKey: STStashStateViewKey.stringForChildView, isLinkedToKeyInState: "stringForFirstChildView")

        let labelView2 = place(STStashChildView.self, key: "labelContainerView2") { (make) in
            make.bottom.equalTo(self)
            make.size.equalTo(200)
        }
        labelView2.prop(forKey: STStashStateViewKey.stringForChildView, isLinkedToKeyInState: "stringForSecondChildView")
    }

    override func viewWillUpdate(newState: [String : Any?], newProps: [StateViewProp]) {
        print("viewWillUpdate in STStashStateView: \(newProps)")
    }

}

/* now labelView1 will always show whatever is in state here for stringForFirstChildView
and labelView2 will always show whatever is in state here for stringForSecondChildView.
(I also changed the constraints from both being second to top, bottom so they don't overlap */

Now, any time those values change, these views will update.
Also, don't forget to add some constraints when you do something like view.addSubview(stashStateView) because stashStateView may not appear without them.

@alexvbush closing this issue in a couple days if I don't hear from you :)


Forced resolutions of some deps of 3rd party imports

cleaned up use of Alert component in home page

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

tested, working 👍 👍


ReCAP Items and "Returned" Label

Items with the status of 'Discharged' are not requestable SCSB. We show these as a green returned label in OL. Not sure if marc liberation is the place to solve this but noting the discrepancy here.
[
  {
    "barcode": "32101088198906",
    "id": 1149588,
    "location": "rcppa",
    "copy_number": 1,
    "item_sequence_number": 1,
    "status": "Discharged",
    "on_reserve": "N",
    "label": "ReCAP"
  }
]```

This is something that will be addressed via Voyager System Administration. The 'Discharged' status will only appear for 5 minutes after an item is checked in for ReCAP items.

Closing per voyager reconfiguration as described by @mzelesky.


Add zlib package requirement to cmakelists.txt as it is a build depen…

…dancy

There shouldn't be a dependency on zlib; what is using it?

With zlib not installed I get the following build error on Debian stable with clang 3.7
Linking CXX shared library libRuntime.so
/usr/bin/ld: cannot find -lz

If it is the LLVM cmake file that should be specifying this dependency feel free to close this.


Haddock for Estimate trails off

See here:
data Estimate e a = Estimate
   { estPoint           :: !a
     -- ^ Point estimate.
   , estError           :: !(e a)
     -- ^ Confidence error estimate. It's parametrized by
   } deriving (Eq, Read, Show, Typeable, Data, Generic)

The documentation for estError cuts off at "It's parametrized by", making it hard to tell what e and a are.

Oops, this looks like a duplicate of #101.

Thank you. I'll fix on Friday.
Not quite duplicate so I'm reopening it

Fixed in e74a8a73a49fb1d42eaa57d4608894471d33e9a3


Fixup 9d8f38a4 Check whether/how the clients for Quarkus AWS extensio…

…n can be used in Camel routes #2592
This is just a test, no respin required.

The failing Splunk and FHIR tests verified locally


🐛 Fix crash with disabled modules in migrator #576

Thank you for contributing to Homarr! So that your Pull Request can be handled effectively, please populate the following fields (delete sections that are not applicable)
Category
Bugfix
Overview

Filter out disabled modules and not convert them to widgets

Issue Number (if applicable)
Related issue: #576

Waiting for confirmation that it has been fixed by the users


Update SDSU BBToolbox to version 1.6.2

Update the SDSU BBToolbox module to version 1.6.2, as per SDSU's instructions. See comments below. Update the Fortran codes and test the updated module.

Main changes are 2 parts.
(1) coda energy envelope computation change, mainly in coda.f90. This change will improve synthetic accelerations in the time domain, but not change SA bias plots significantly.
(2) add shifting scattgram according to the minimum initiation time from srf file. This change will improve simulations for multiple segment ruptures, and change SA bias plots for 3-segment-rupture for Landers.

Updated BBToolbox to version 1.6.2 as per commit 568c3abcdd20b095af1c97ea3897494412a0c6d2
Also updated to version 2 with commit 6670cdd7c231cf628b1f93ec209523e2e78c386f


Load Test failed incase more than 7 requests at the same time

Hi,
I deployed to Vertex AI with 1 compute node, "n1-highmem-4" and 'NVIDIA_TESLA_V100'.
Then I did a load test with Jmeter with Number of Threads: 8, Loop count: 1 ( 8 requests at the same time ), the result is 100% failed. It's 100% success with only 6 requests at the same time
I saw that it failed at this PLMSSampler step in txt2img.py :
samples_ddim, _ = sampler.sample(S=ddim_steps,
conditioning=c,
batch_size=n_samples,
shape=shape,
verbose=False,
unconditional_guidance_scale=scale,
unconditional_conditioning=uc,
eta=ddim_eta,
x_t=start_code
)
Regards

@dungle01 Thanks for running this test. Its good to know what a single replica can handle concurrently.
If you like to have more concurrency, you'll need start with a min_replica_count higher than 1 and you can set the max_replica_count to a higher value as well.
I'm working on making some changes to this script so that its configurable via parameters.

@entrpn the replica-count is used for autoscaling with CPU metrics. But this case is not about high CPU, it seem PLMSSampler won't work well with many concurrently run in a same VM.
Vertex AI won't distribute the traffic into new VM if the first VM still have low CPU metric.
Regards

@dungle01
But as per what is mentioned here, we have an option to specify to autoscale on basis of gpu.
And I found the code/argument to mention this metric_spec in python sdk here.

I wrote more information about this issue here.
I'll be running some experiments this week and get back to you with the results.

@dungle01 as @mln-wave mentioned, autoscaling is also based on GPU usage. However, autoscaling is slow, so is better to start with more replicas. You can run the deploy command with something like.
python gcp_deploy.py --image-uri gcr.io/<project_id>/stable-diffusion:latest --max-replica-count 5 --min-replica-count 5

The script has also been updated to replicate with a lower accelerator duty cycle.

thanks @entrpn ,
let me double-check on our side

@entrpn after I deployed latest code, I got the image data but it's not an image (see attachments).
Did you got it on your side ?

@dungle01 I am not running into this issue. I don't think you are using the latest codebase since the response object has changed to something like:
{"predictions":[{"prompt":"An apocalyptic alien city ravaged by time, highly detailed, cinematic, intricate, trending in artstation","images":["base64encodedimage","base64encodedimage"]}]}

The request has also changed to include the parameters within the instances list. Ex:
{"instances": [{"prompt": "An apocalyptic alien city ravaged by time, highly detailed, cinematic, intricate, trending in artstation", "parameters": {"ddim_steps": 30, "scale": 7.5, "n_samples": 2, "n_itter": 2, "type": "txt2img"}}]}

Inside the stable-diffusion folder there is a python script validate_response.py that you can run that will take the response.json and decode the base64 images.

@entrpn
I can managed to update latest code now, there's some wrong cache from docker build on my side.
For Load Test checking:

With test min=4, max=4 : I saw it can handle well with 10 reqs at a same time, up to 20 reqs with  low rate of error occassionally
With test min=1, max=4: appear high rate of error (sometime 90%) when reach 10 reqs. It seems that it work not well in scaling gap.

A work-around that I implemented a job queue to control the max no of concurrent reqs (~10reqs) and use min=4, max=4 mode. The job queue is implemented by Cloud Function with Firebase FireStore .

@dungle01 that's great. I am also working on a gke version to compare against Vertex endpoints but it's a bit slow to implement as I'm not as familiar with Kubernetes so I'm learning while doing it.


Node-red fails to load , when node-red-contrib-netatmo is installed

I have also tried to do a clean install , with only node-red-contrib-netatmo installed
npm install -g --unsafe-perm node-red
npminstall  node-red-contrib-netatmo -g

Yep, last commit brought some troubles. This one works just fine:
https://github.com/ssadams11/node-red-contrib-netatmo/tree/ac650175cc7b802d80fc8c93b8c28156c1bfc3a7

I am using
11 Mar 17:09:24 - [info] Node-RED version: v0.18.4
11 Mar 17:09:24 - [info] Node.js  version: v9.8.0
and only the first 3 items
get next events
get camera picture
get home data
are marked green in node-red. Dont get any error during instalation. Any idea if that Is a similar issue then this?
Thanks, Jens

I think I fixed it with this pull request: https://github.com/ssadams11/node-red-contrib-netatmo/pull/10

Try latest master

Thanks for all the help and effort and nagging (a good thing!)
version 0.2.0 loads and runs successfully in node-red 0.18.3


Migrate LaunchDarklyConfigStore to @configu/integrations

Suggestion
Migrate the LaunchDarklyConfigStore from its current setup to the new @configu/integrations module as part of the major restructuring. The goal is to move all third-party integrations, including stores, expressions, and libraries, to @configu/integrations.
Motivation
This migration is part of a larger effort to better organize and modularize the product. Moving the LaunchDarklyConfigStore will align with the new structure, making the system cleaner and more maintainable.
Context
You can refer to the already migrated stores for guidance:
Migrated stores in integrations.
Base stores implementations from @configu/ts can be found here:
Base stores in TS.
The old node-based stores were previously in @configu/node:
Node stores.
The old browser-based stores were previously in @configu/browser:
Browser stores.

I would like to work on this. Can I get this assigned?
Thanks!

All yours @ezhil56x

Interested! @rannn505 @RichardAkman

Interested! @rannn505 @RichardAkman

We appreciate your enthusiasm @su-shubham but the issue is currently assigned to @ezhil56x. You can browse other unassigned issues that pique your interest.

@RichardAkman Dropping this. Kindly assign to someone else

Hey @RichardAkman , Please assign me this issue. I would love to contribute.

All yours @SanchitMahajan236

@RichardAkman - While I am aware this issue was recently assigned, just wanted to mention that I am willing to assist here if @SanchitMahajan236 decides to drop the issue.

@RichardAkman - Seeing as @SanchitMahajan236 is inactive for over a week, can this issue be assigned to me? I have a PR ready.

You are correct @TomerPacific, you got it. @SanchitMahajan236, please feel free to pick another issue for contribution, we appreciate your willingness to contribute 🙏🏻

I was working on the issue just now and when I opened the assigned issues section to see the issue number, I found that I have been unassigned.
I apologize for the delay on my end but I think you should have atleast left a message asking if I am working on the issue @RichardAkman.

@SanchitMahajan236, thanks for letting me know. I understand the frustration, and I apologize for any miscommunication. We aim to keep issues moving smoothly, so if there’s no update or opened PRs after some time, we may reassign to help other contributors get involved. In the future, feel free to comment on the issue or open a draft PR if you're still actively working on it; this helps us stay aligned on progress. Your contributions are valued, and I appreciate your understanding. Let’s keep up the good collaboration!

I will definitely remember your advice.


cargo install --git multiple packages with binaries found hint

What does this PR try to resolve?
The issues discussed in: https://github.com/rust-lang/cargo/issues/4830
namely this one:

a multiple packages with binaries found error should give users a hint about how to specify a single crate

I think improving the error message to provide a suggestion is a simple change that would help a lot of people when this happens, sorry if I'm out of line for just opening a PR here :)
Before
cargo 1.68.0 (115f34552 2023-02-26)

After

Additional information
I added in a related test documenting existing behaviours

multiple_examples_error: "multiple packages with examples found" (i.e. not "binaries")

I added these tests which aren't necessarily related to this PR, but could be considered if the behaviour were to change

multiple_binaries_deep_select_uses_package_name: it uses the name, not the path. Is this a problem for crates with the same name?
multiple_binaries_in_selected_package_installs_all: so --bins is implied when a package is specified?
multiple_binaries_in_selected_package_with_bin_option_installs_only_one: demonstrates a use case where --bin and [crate] are both used

r? @weihanglo

Oops. Is this ready for review? I feel like it is 😅.

Thanks so much for the reviews :+1:

@bors r+

:pushpin: Commit 3363931f7fccc81ea179bdbe516faf7e07430e9e has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 3363931f7fccc81ea179bdbe516faf7e07430e9e with merge 4cce1f94fb0ed4c4df68b061c1bca43d00d49898...

:broken_heart: Test failed - checks-actions

Looks like a spurious failure due to network timeout not related to this PR.
@bors retry

:hourglass: Testing commit 3363931f7fccc81ea179bdbe516faf7e07430e9e with merge 9855f924942fae1c4e298e9fb48e255ee1732d53...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 9855f924942fae1c4e298e9fb48e255ee1732d53 to master...


haskellPackages.espial: Fix build by applying patch

Description of changes
Applying the change in https://github.com/jonschoning/espial/pull/36/

Things done
Applying a patch from master providing GHC9 support because it doesn't have a release.

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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

[X] 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
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

LGTM, thanks for fixing this!


Check if libcloud is installed

This PR implements a safeguard that checks if libcloud is installed (or reachable).
The sanity check is performed when the elibcloud is first loaded (using the -on_load(function) directive).
Before this patch, this is the error I got when I tried to use elibcloud and libcloud was not properly installed on my computer:
...CRASH REPORT Process <0.1544.0> with 0 neighbours exited with reason: no case clause matching 127 in elibcloud:libcloud_wrapper/1 line 505 in gen_server:terminate/6 line 746...

This is the error one gets now:
=ERROR REPORT==== 3-Jun-2014::09:39:58 ===
Error in process <0.34.0> with exit value: {{nocatch,libcloud_not_found},[{elibcloud,ensure_libcloud_is_installed,0,[{file,"src/elibcloud.erl"},{line,87}]},{code_server,'-handle_on_load/4-fun-0-',1,[{file,"code_server.erl"},{line,1656}]}]}

Besides getting a more intuitive error message, the timing has also been improved. As mentioned earlier, the error is now thrown the first time elibcloud is used (e.g. when creating the credentials), rather than waiting until the user calls a function triggering a call to libcloud.

Regarding the first (main) commit

The main problem the solution implemented in 78ec9aa is that any system including elibcloud will crash if libcloud is not properly installed in the system.
Actually, this is not necessarily true. It all depends on how the Erlang node is started. If the erlang node is started in interactive mode, the system would not crash until it attempts to use elibcloud (which is the expected behaviour). However, if an Erlang node is started in embedded mode, all Erlang modules are loaded when the node is started. Hence, the node will go down even if the system was never supposed to use elibcloud. IMHO, the latter is not a problem at all. If a release contains an application (elibcloud, in this case), the contained application must be usable. It is kind of pointless to include an application that one knows will never be used. It is also pointless to include an application that does not work.

Unfortunately, this is still a problem, since the only project that uses elibcloud as far as I know is Wombat, which starts its node in embedded mode, and may be run on a system without Libcloud.
So I'm closing this PR without merging. We can keep the branch though for future inspiration.
Regarding the other commits
The other commits contain minor fixes which have been added in commits 5cfc8fbd4e0232666bf55914ef5d05d017150206 and 768fdf8053bc69753f20b908655a9792fc7e18ee.


How to change shape from square to circle in crop area?

Hello,
i want to change shape from selecting area to square to circle.
which property should i use?
Please suggest me solution.

The Cropper does not support circle crop box, but you can customize it by yourself.

how can i customize to convert it circle box?
any suggestion or fiddle it?

Override the relative classes as .cropper-crop-box and .cropper-view-box.

ok sir i will try it i have no clue to done it.because i am newbie to using this plugin?

Hi,
I have been looking to achieve the same thing. Please note that you need to make use of the box-shadow property to create a rounded stroke, as outline currently doesn't follow the border-radius property in a cross-browser way.
I ended up using the following styles:
.cropper-crop-box, .cropper-view-box {
border-radius: 50%;
}
.cropper-view-box {
box-shadow: 0 0 0 1px #39f;
outline: 0;
}
Tested in Safari, Chrome and Firefox on Mac OS, if you want to support IE8 you should consider resetting the outline for that browser, as it doesn't support border-radius and box-shadow.
I also had to set highlight: false when initializing cropper.

Hii , schuma7
i have using your code..
.cropper-crop-box, .cropper-view-box {
border-radius: 50%;
}
.cropper-view-box {
box-shadow: 0 0 0 1px #39f;
outline: 0;
}
It does seems good. i want something like that round shape. good work.
Thanks  schuma7

Hi Guys i'm also looking for this, where we need to change above css?

@Sandeepwal  Here is an example from Cropper.js.

Hey all by doing some css work we can make image crop into circle

.cropper-crop-box, .cropper-view-box {
    border-radius: 50%;
}

.cropper-view-box {
    box-shadow: 0 0 0 1px #39f;
    outline: 0;
}

.cropper-face {
  background-color:inherit !important;
}

.cropper-dashed, .cropper-point.point-se, .cropper-point.point-sw, .cropper-point.point-nw,   .cropper-point.point-ne, .cropper-line {
  display:none !important;
}

.cropper-view-box {
  outline:inherit !important;
}

As rectangular outer shape with handles might be easier to use, also most users are familiar with it more than circle (UX), I have another solution: Round internal plus rectangular outer shape as below:

Very little CSS to add to make it like this:
.cropper-view-box {
	box-shadow: 0 0 0 1px #39f;
	border-radius: 50%;
	outline: 0;
}
.cropper-face {
	background-color:inherit !important;
}
.cropper-view-box {
	outline:inherit !important;
}

Have nice result with css from @Siva-Ganesh-VR  and some magic from me)

css:
.cropper-crop-box, .cropper-view-box {
    border-radius: 50%;
}
.cropper-view-box {
    box-shadow: 0 0 0 1px #39f;
    outline: 0;
}
.cropper-face {
  background-color:inherit !important;
}

.cropper-dashed, .cropper-line {
  display:none !important;
}
.cropper-view-box {
  outline:inherit !important;
}

.cropper-point.point-se {
  top: calc(85% + 1px);
  right: 14%;
}
.cropper-point.point-sw {
  top: calc(85% + 1px);
  left: 14%;
}
.cropper-point.point-nw {
  top: calc(15% - 5px);
  left: 14%;
}
.cropper-point.point-ne {
  top: calc(15% - 5px);
  right: 14%;
}

Here's another solution, derived from the one by the one from @power-media.
In my application, I think it makes more sense to retain the blue border around the rectangle since that's the most "traditional" UI. Our users respond poorly to non-traditional implementations. The semi-opaque circular area is just an indication to the user regarding how the resulting image will be used (it will be enclosed within a circle). After all, there's no such thing as a circular image.

The CSS is surprisingly quite simple! 🎉🥳
.cropper-view-box {
	border-radius: 50%;
}
.cropper-face {
	background-color:inherit !important;
}

If you'd like to remove the dashed lines (like I have in the screenshot above), that CSS is also quite simple:
.cropper-dashed {
	display: none;
}

How can I change the black transparent background for a white background without transparency please?

@AxelBriche It's all CSS - just inspect the elements and apply some CSS to make the changes. If you need more info, you may want to open a new issue.


Fix: Changed file angular-ecosystem-logos.png. Cause of missing trans…

…parency in bakground.
PR Checklist
Please check to confirm your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Fix: Changed file docs/images/angular-ecosystem-logos.png. Cause of missing transparency in bakground.

@b1tw0rker are you willing to address the pending comment?

Closing since comments have not been actioned. Please feel free to re-open when addressing the comments.