Add - Get Real IP from Cloudflare

I've used this to get a clients IP address to put into a database so if someone logs in with a different account I can make them verify, just like https://discord.com does.
I am not really sure on how to make packages so I will just put the code below, incase you would like to make it a feature otherwise it can just be closed.
<?php

class CloudFlare
{
    /**
     * Gets the real user IP address
     *
     * @return string The user's IP address
     * @throws \InvalidArgumentException If no valid IP address is found
     */
    public static function getRealUserIP(): string
    {
        $ipSources = [
            'HTTP_CF_CONNECTING_IP',
            'HTTP_CLIENT_IP',
            'HTTP_X_FORWARDED_FOR',
            'REMOTE_ADDR'
        ];

        foreach ($ipSources as $source) {
            if (!empty($_SERVER[$source])) {
                $ip = $_SERVER[$source];
                if (filter_var($ip, FILTER_VALIDATE_IP)) {
                    return $ip;
                }
            }
        }

        throw new \InvalidArgumentException('No valid IP address found');
    }

    /**
     * Check if the user is using CloudFlare
     *
     * @return bool
     */
    public static function isUsingCloudFlare(): bool
    {
        return !empty($_SERVER['HTTP_CF_CONNECTING_IP']);
    }
}

@Nyxalis I think this is useful. Does PHP work with Cloudflare Workers?

@notrab  Unfortunatley PHP cannot run on Cloudflare Workers

@Nyxalis ok, I'm not sure where this best fits currently since it would be nice to have features currently that are cross platform.

I am really lost @notrab, what does getting the IP address for a user and running on cloudflare works have any link?

@Nyxalis you said PHP doesn't run on Cloudflare, so what are you trying to achieve here exactly?

@notrab  Let's say a user creates an account onto a dashboard. There password got compromised and now anyone can login.
The code I gave as an example let's them developer get the original users IP address, and if soemone tried logging into the dashboard with a different IP, just like discord the user will have to verify from there email that the IP is safe. Otherwise they can't login.

Sounds great, but I don't think this should be part of the framework, unless I'm missing something.


Broken link for  Microsoft Visual C++ 2010 Redistributable Package (x86) link in README.md's link.

https://github.com/ytdl-org/youtube-dl/blob/master/README.md#the-exe-throws-an-error-due-to-missing-msvcr100dll says "The exe throws an error due to missing MSVCR100.dll
To run the exe you need to install first the Microsoft Visual C++ 2010 Redistributable Package (x86)."
However, its https://www.microsoft.com/en-US/download/details.aspx?id=5555 link is broken. I found it in https://www.microsoft.com/en-us/download/details.aspx?id=26999, but it doesn't mention Windows 10 (64-bit) so I am not sure if it is safe to install it into my brand new, updated 64-bit W10 Pro PC.

Thank you for the quick fix, dstftw! Do I assume it is OK to install and use in an updated, 64-bit W10 Pro?

@antdude:
https://docs.microsoft.com/en-US/troubleshoot/cpp/minimum-service-pack-levels#visual-c-2010-sp1-redistributable-package
BTW, M$ documentation is known to contain omissions/false info, etc... 😞
However, since the official youtube-dl.exe binary is 32-bit, make sure you download and install file vcredist_x86.exe (32-bit variant); you may optionally co-install vcredist_x64.exe if, in the future, you come across a 64-bit application built with VS2010...

Thank you, Vangelis66. I didn't want to break my new W10! :)


Fix flaky tests.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.
[x] Have you successfully run brew tests with your changes locally?

The issue was that the Formula.installed result was getting cached as [] by a previous test.
I've made Tab#write clear the cache if the tab didn't exist before, which should stop this coming up again, but I wonder if we should clear the cache automatically in the teardown of every test as well, which would definitely stop this coming up again, and shouldn't have too much overhead. Thoughts?

What went wrong with the CI here?

Seems like Formula.clear_cache should be updated, but not cleared. Is this better than clearing the cache on teardown? I can be wrong, though.

Seems like Formula.clear_cache should be updated, but not cleared.

Yeh, this would be ideal.

Can change so new Tabs are added to the cache, but I still think the cache should be cleared in teardown, because tests are supposed to clear up any data they write after them.
Currently, Kegs are removed at the end of tests, but the cache isn't cleared. This leads to the opposite problem to what was an issue here — no kegs installed, but some formulae still being present in the Formula.installed cache.

@alyssais Nuking cache on teardown is fine, would just be nice to avoid nuking cache on write if possible.

Several hours in, this is ending up being way more complicated than it should be. I will make sure the next thing I do here is getting this working properly, but for unexpected reasons it's going to take a small refactoring elsewhere before it's possible (can go into detail if you want, but I'm posting this right before I go to bed and I'm 99.9% sure that has to be done for this can work properly, and I know what I need to do now).
I'd like to suggest, however, that this is merged as-is. It replaces behaviour that is clearly incorrect (it's only a matter of time before there's another weird bug that takes hours to track down because of it) with behaviour that, while not optimal, is correct.
I also think it's quite important to clear the cache on teardown, but again, I'd prefer to make a separate PR for that and get this merged sooner if possible, because it's definitely an improvement to the current behaviour (and it means tests don't need to be skipped!).

@alyssais Can't merge it as-is due to conflicts 😭. I think Formula.clear_cache should probably be renamed to Formula.clear_installed_formulae_cache or similar. My concern is if we end up using more caches in Formula that nuking them all is not what we want to be doing. I'm still not sure about having the cache cleared in the location it is, too; from the placement/code it's not clear which method call it's being cleared for. If it were for e.g. runtime_dependencies I'd suggest clearing it just before then would be best and perhaps splitting it into another Formula.clear_runtime_dependencies_cache to signal intent better.

think Formula.clear_cache should probably be renamed to Formula.clear_installed_formulae_cache or similar. My concern is if we end up using more caches in Formula that nuking them all is not what we want to be doing.

Agreed.

I'm still not sure about having the cache cleared in the location it is, too; from the placement/code it's not clear which method call it's being cleared for. If it were for e.g. runtime_dependencies I'd suggest clearing it just before then would be best and perhaps splitting it into another Formula.clear_runtime_dependencies_cache to signal intent better.

Can you explain what you mean by this? It's not being cleared for any particular method call, it's being cleared so that the list is up-to-date for anything that wants to use it.

It's not being cleared for any particular method call, it's being cleared so that the list is up-to-date for anything that wants to use it.

That's my point: what is it that needs to use it? Just from the e.g. diff it's unclear if it's CACHE[tabfile] =,  atomic_write or to_json or a method called from inside one of them (I know it's the latter but I had to dig in to realise that).

Nothing in Tab needs to use Formula.installed, it's just that Tab#write is the only centralised location (as far as I can tell) that will always be called when the cache needs to be cleared.

Nothing in Tab needs to use Formula.installed, it's just that Tab#write is the only centralised location (as far as I can tell) that will always be called when the cache needs to be cleared.

What does need it (in these tests I mean)? Would like to try and figure out how to get it in a "nearer" place to that.

In these tests, the call comes from Keg.find_some_installed_dependents.
Remember, though, that just because this is somewhere we know that not clearing the cache is causing incorrect behaviour, doesn't mean that it's the only place.

How about clearing the cache in formula_installer.rb instead? Presumably that's what's calling Tab.write?

Not always — a lot of tests create their own kegs without using FormulaInstaller.
Could always add it to FormulaInstaller and come up with a helper method for creating kegs in tests, I guess?

Not always — a lot of tests create their own kegs without using FormulaInstaller.
Could always add it to FormulaInstaller and come up with a helper method for creating kegs in tests, I guess?

Yep, that sounds best 👍

Thanks again @alyssais!

Oh, I'm late with some of my thoughts on this.

get this merged sooner if possible, because it's definitely an improvement to the current behaviour (and it means tests don't need to be skipped!).

Good point @alyssais.

I also think it's quite important to clear the cache on teardown, but again, I'd prefer to make a separate PR for that ...

Teardown seems to be a good place.
This fails only for three tests:
diff --git a/Library/Homebrew/formula.rb b/Library/Homebrew/formula.rb
index 7fe00cb..90454cd 100644
--- a/Library/Homebrew/formula.rb
+++ b/Library/Homebrew/formula.rb
@@ -1360,6 +1360,23 @@ class Formula
     end
   end

+  def self.update_cached_with_formula(formula)
+    racks.push(formula.rack) unless racks.include?(formula.rack)
+    installed.push(formula) unless installed.include?(formula)
+  end
+
+  def self.update_cached_with_rack(rack)
+    racks.push(rack) unless racks.include?(rack)
+    formula_for_rack = begin
+      Formulary.from_rack(rack)
+    rescue FormulaUnavailableError, TapFormulaAmbiguityError, TapFormulaWithOldnameAmbiguityError
+    end
+
+    unless formula_for_rack.nil?
+      installed.push(formula_for_rack) unless installed.include?(formula_for_rack)
+    end
+  end
+
   # An array of all installed {Formula}
   # @private
   def self.installed
diff --git a/Library/Homebrew/tab.rb b/Library/Homebrew/tab.rb
index 3bb7d8b..3902c09 100644
--- a/Library/Homebrew/tab.rb
+++ b/Library/Homebrew/tab.rb
@@ -317,9 +317,7 @@ class Tab < OpenStruct
   end

   def write
-    # If this is a new installation, the cache of installed formulae
-    # will no longer be valid.
-    Formula.clear_installed_formulae_cache unless tabfile.exist?
+    Formula.update_cached_with_rack(tabfile.parent.parent) unless tabfile.exist?

     CACHE[tabfile] = self
     tabfile.atomic_write(to_json)

Fails occur when we call .write on tabs, which is expected. And which also proves what @MikeMcQuaid  said here:

I'm still not sure about having the cache cleared in the location it is, too; from the placement/code it's not clear which method call it's being cleared for. If it were for e.g. runtime_dependencies I'd suggest clearing it just before then would be best and perhaps splitting it into another Formula.clear_runtime_dependencies_cache to signal intent better.

The right place for updating is probably formula_installer as Mike already said, however the directories are created here:
https://github.com/Homebrew/brew/blob/master/Library/Homebrew/build.rb#L133


The mouse property should be a pointer when the admin is trying to mark all notifications as read.

Bagisto Version: Master
Steps To reproduce--
1. log in as an admin.
2. Click on the notification in the header section.
3. Try to click on mark as read
Expected result: The mouse property should be a pointer when the admin is trying to mark all notifications as read.
Actual result: The mouse property is grabbed when the admin is trying to mark all notifications as read. Please find the screenshot Here.

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here. for future reference.


Update jackson and bouncycastle artifacts

Issue #, if available: #150
Description of changes:

Update com.fasterxml.jackson.core:jackson-annotations to 2.11.1
Update com.fasterxml.jackson.corejackson-databind to 2.11.1
Update org.bouncycastle:bcprov-jdk15on to 1.66
Update org.bouncycastle:bcpix-jdk15on to 1.66

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This will be resolved on the opendistro-1.10 branch, and on master once we merge those changes back


State modification using redux

Hello,
I was playing around with the cell editing feature in a react/redux app. It looks like when you enable cell editing, react-bootsrap-table automatically mutates its input object to save the value. This doesn't really congrue with the idea of redux, which is supposed to manage state. Is there a way to disable the automatic mutation of the input dataset in react-bootstrap-table?

one way may be you can use the beforeSaveCell and cancel the save(return false) and add your custom code to apply the edit changes and do a render. The event does give the row,cellname and modified cellvalue.
There may be a better way that i am not aware of.

@rrameshkumar76, thanks, your answer is a workaround and I thinks it's the only way to achieve this target currently.
BTW, react-bootstrap-table support the remote mode and this mode allow you to manage the state on your state manager(redux, mobx ..). However, we all support sorting, insert, delete, exportCSV .etc on remote mode, but missing cell editing(Because there's some issue that it's hard to explain).
Anyway, I'll try to make cell editing available on remote mode, I think it will fix this issue
Thanks 👍

Hello,
I am also wondering how I can change the default amount of rows shown when pagination is enabled. I’d like 20 or 30 rows to be shown per page by default instead of 10.

On Aug 10, 2016, at 8:28 AM, Allen<EMAIL_ADDRESS>wrote:
@rrameshkumar76 https://github.com/rrameshkumar76, thanks, your answer is a workaround and I thinks it's the only way to achieve this target currently.
BTW, react-bootstrap-table support the remote mode and this mode allow you to manage the state on your state manager(redux, mobx ..). However, we all support sorting, insert, delete, exportCSV .etc on remote mode, but missing cell editing(Because there's some issue that it's hard to explain).
Anyway, I'll try to make cell editing available on remote mode, I think it will fix this issue
Thanks 👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/AllenFang/react-bootstrap-table/issues/618#issuecomment-238903981, or mute the thread https://github.com/notifications/unsubscribe-auth/AM5gy7sVkaEyoKXwXL6Pe2sKAtbYQ2qTks5qee42gaJpZM4JgbMx.

@edulis8, check https://github.com/AllenFang/react-bootstrap-table/blob/master/examples/js/pagination/custom-pagination-table.js#L35

Fixed on v2.4.2. https://github.com/AllenFang/react-bootstrap-table/commit/f399e878102322f3da68f95c1265b5c27e1eba1b is an example for demo how to use cell edit on remote mode

Thanks for the nice remote mode.It helps us a lot.

remoto with event selectRow works?


Fix for choice accents issue (#274)

This change is

Please update the title of this PR. I now have to look at what issue #274 is. Please try to make the life of your reviewer as easy as possible so we can merge this stuff.

This could cause issues with spaces and other entities:
https://www.drupal.org/project/wysiwyg/issues/373542
We'll revisit later after we add some tests. Closing for now.


Delegate Content-Type verification solely to contentTypeMatches()

Image content returned from the initial request will have its Content-Type verified at https://github.com/willnorris/imageproxy/blob/main/imageproxy.go#L243. Prefilling the Accept header with the list of accepted Content-Type's has proven troublesome for some of our customers as some web servers have odd behaviors like:

returning an error if you provide a list of Content-Types in Accept
returning an error if you provide a Content-Type in Accept that the server does not know about (like image, which is not valid normally, but some web servers return content using that Content-Type 🙃)

The addition of Accept-Language is similar -- we've found some servers that will return an error if no Accept-Language header is supplied. Wack a mole!

Whoops, forgot that there was a PR for this branch. I'll pull that last commit into a different branch -- we're just disabling SVG all together since we can't properly sanitize it and security researchers keep using it to submit reports.


Fix escaping of term including backticks

Summary
The existing text surrounded by back-ticks does not work since there is intentionally a single back-tick in the content.  Using the triple back-tick on the outside of the text does work.

Thanks!


Allow different storage backends for biicode

Now, just the biicode backend is available, due to historic reasons. Though git is the most reasonable one, many users still use svn and others, so we just went for the most simple way: just use the biicode storage. As all the metainformation for files was storing, just storing the contents/blobs for the files was really straightforward.
This can make challenging to to implement this issue, and I think some thinking and planning is required. I will first analyse things, current status, interactions, processes and storage and retrieval, but a possible outline of the process in order to incrementally build it without breaking things, could be:

Decouple retrieval of meta information from blobs retrieval. For dependencies it can be checked here: https://github.com/biicode/common/blob/develop/deps/closure_builder.py. No new functionality here.
Introduce a level of indirection in the biicode blobs that will redirect to the real backend. No new functionality here, just running with biicode backend.
Define syntax in biicode.conf for different backends. (maybe starting with git, then zip, http... in second iteration). Just allowing the syntax.
Specifics of backends.

Ross @rharriso offered (thanks!) collaboration with this issue. Check if we are talking about the same thing and please let me check everything, and will come back to this issue, refine plan, and lets talk about concrete tasks and next steps.

I'm still familiarizing myself with the source code. But I have some questions already.

what is included in the meta data? I am assuming that blob retrieval refers to fetching of block code.

Does the metadata include build information, and location of required headers & objects?

Would these changes require an entry in the biicode repository?

This could be a way to prevent circumvention of the premium features.

Is there a test suite? Or development documentation?

Yes, metadata basically consist of:
Cell:

Name & ID
Declaration of dependencies (#includes, can be resolved or not)
Dependencies between files (E=Explicit, coming from resolved #includes, I=Implicit, coming for implementations and manual configuration).
Necessary paths to resolve depedencies (will be translated to include-paths)
If the file contains a main()

Content:

ID
Blob of data (lazily compressed) with its sha
Parsing info: symbols (classes, methods), #includes, that will generate Declarations

Both Cell and Content form a Resource.
When things are retrieved from the biicode repository, are retrieved as Resources, and that is the problem: it has to be decoupled to first retrieve the meta-information of Cells & Contents, but without the actual blobs of data, so they can be directly retrieved from other place.
About circumvention of premium features, it is always possible, but not simple. Still a biicode server is needed as the meta information is stored there. As the circumvention of premium features, it is not a big concern right now, we just want to create something really useful that solves our problems.
There are some tests within the repos, but those are just a part. We still have many more in a separate private repo, as it is also highly coupled with our backend and sensitive data. We still have tons of work to do to decouple things, so we will be continously porting tests from there to the client and common repos. Unfortunately there is no development documentation, we are small team and work a lot on the board/paper in the office, and the design/implementation has evolved wildly and very fast, so we didnt have the need and time for development documentation, though we have started some time ago to be more thorough with our docstrings, but many still to do here....

Hi @rharriso
I have started to take small steps, cleaning things and studying how to do it, I explain them here just in case you want to track changes, and what is being done.

First thing was a bit of cleaning in Blobs storage for bytes, and avoid serializating them into our DB for "opened" blocks: https://github.com/biicode/common/pull/5
Then, the dependencies of a given project were also serialized to the project DB, which it is useless, as they are computed again: https://github.com/biicode/common/pull/7
Still some work to prepare things, and still not clear how to approach the problem, lets keep decoupling things.

To handle decoupling. How should we handle notation.

tags use @tag.number
perhaps we could use git:git-pull-path, local::path/to/local/folder ( those two examples would be most useful to me )

About your changes:
This question will reveal my ignorance. How are dependencies for blocks stored if not in the local database?

I was talking about the "opened" blocks, not the dependencies. The opened blocks are those that are actually being edited, typically it is just one, but you can have many of them open at the same time.
For opened blocks, lots of unnecessary meta-information was being stored to DB, and it was unnecessary as it is computed again (and faster) when a new invocation of "bii" is done.
Another small step has been done, with another large improvement in processing speed, especially for large projects: https://github.com/biicode/common/pull/9
Also, the dependencies and their meta-information are stored in the biicode user cache, common for all projects and located in your .biicode folder, but they were also stored (again) in the project DB, which was redundant and slow, and that is what has been removed.
Note that still much work has to be done until we have to actually specify the notation for referring to git and paths. Still not sure about how to handle the decoupling, at least these small steps are clearing the way and simplifying the codebase. I wanted to ask for more input: how do you envision the retrieval from git? because now, the meta-information for files and blocks is stored in biicode DB, so actually referring to git to just retrieve the files has little gain (it is not that complex to upload the files to biicode, as we do just now). If we commit the meta-information into text or binary files into the git repo, we have to design a full new system to retrieve and handle meta-information from there, and that will not be simple.
Thanks! Cheers.

Answering your last point. The more I think about it, there isn't much gain. Either way there will have to be a definition on the server of how to build and include the block.
What I've used this for with bundler was working on my own forks (locally then on git) before committing to the root project. However, given the different work environments, the best course might be to encourage projects to use biicode, and build my own forks locally.
I still think there is utility in this, but not nearly as immediate as I once thought.

Yes, that is the point. There is a good point if we are able to store meta-information in plain text files (and manage it coherently, which is a challenge) and also interact directly with git (and other stores), so yes, you could interact directly with your local copy. Not sure how we could manage the names and version, but certainly it can be done, but could require a lot of work, and now there are many other inmediate priorities. That is why I have started a roadmap for this decoupling, and we address it when we can find some time. I would say that in about 3 months it will be ready for working on this feature and providing alternative origins. Lets keep working on it. Cheers.

Similar to https://github.com/biicode/client/issues/23


fix(import_granularity): do not merge aliases

Fixes #6027.
Related: #4991.

use {
    super::harriblex,
    super::harriblex::{
        Iraquant,
        Mediviction
    },
};

should be merged to:
use super::harriblex::{self, Iraquant, Mediviction};

and not:
use super::{
    harriblex,
    harriblex::{Iraquant, Mediviction},
};

But my change neither introduced nor fixed this. I will try to fix this as well, but this is technically a different issue and thus topic of a separate PR, right?

What you've highlighted is not an issue. I know there's been prior discussion on the topic, but I couldn't find the exact link I was thinking about. I was able to find https://github.com/rust-lang/rustfmt/issues/3750, which mentions the issue. Ultimately, use super::harriblex; and use super::harriblex::{self}; are semantically different, and we don't want to make any changes to the current formatting.

I rebased it, is there anything else left to do?

I still need to take another look at this one. After I understood what the max_depth changes were doing I questioned whether or not a.equal_except_alias(b) was the right comparison to make in the take_while closure. Did you investigate whether or not it's possible to create a new function that takes the aliases into account when comparing a and b?

@sivizius I see that you recently force pushed. Did you have a chance to consider any alternative approaches to this instead of using max_depth? The main issue I can see with this approach is that it retains more duplicates than the current behavior:
Passing the following input to rustfmt via stdin (note that all use statements are duplicated) when running the command cargo run --bin rustfmt -- --config=imports_granularity=One:
pub use foo::x;
pub use foo::x as x2;
pub use foo::y;
use bar::a;
use bar::b;
use bar::b::f;
use bar::b::f as f2;
pub use foo::x;
pub use foo::x as x2;
pub use foo::y;
use bar::a;
use bar::b;
use bar::b::f;
use bar::b::f as f2;

Your PR produces the following:
use bar::{
    a, a,
    b::{self, self, f, f as f2, f, f as f2},
};
pub use foo::{x, x as x2, x, x as x2, y, y};

And the current behavior on the latest commit 777e25a8c9, produces this:
use bar::{
    a,
    b::{self, self, f},
};
pub use foo::{x, x as x2, y};

Also, the conversion from bar::b -> bar::{self}, in the above input seems incorrect to me. What's strange is that if we only have bar::b imports we don't convert bar:b -> bar::{self} and we don't merge aliases. This behavior is consistent for your PR and 777e25a8c9:
input
use bar::b;
use bar::b::f;
use bar::b::f as f2;

output
use bar::{
    b,
    b::{f, f as f2},
};

@sivizius I see that you recently force pushed. Did you have a chance to consider any alternative approaches to this instead of using max_depth?

Not yet, but I started to work on this PR again and will take a look. I just rebased and pushed an up-to-date state yesterday.

I recently tried the current nightly rustfmt (master, not this one) and apparently, use {foo, foo::bar}; was merged to use foo::{self, bar};? Was there a change?

Not that I'm aware of, at least not a change that would have intentionally produced that result. For what it's worth, I can also reproduce that behavior when running rustfmt from source (cargo run --bin rustfmt -- --config=imports_granularity=One).
That said, this seems like a separate issue from merging aliases, and can probably be addressed in it's own PR.


docker-buildx: init at 0.5.1

Motivation for this change
Installing docker-buildx enables buildx subcommand on the client:

https://github.com/docker/buildx

Things done

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

[ ] NixOS
[x] macOS (aarch64)
[x] nixos/nix docker image (aarch64)

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

This change requires the following in ~/.docker/config.json to make it work for me:
{
    "experimental": "enabled",
    "cliPluginsExtraDirs": [
        "/Users/ivan/.nix-profile/libexec/docker/cli-plugins"
    ]
}

Here /User/ivan is my $HOME on macOS.
Ideally I'd like it to work outside of the box, but even the following patch doesn't do the trick:

https://github.com/bobrik/docker-ce/commit/a03bb6bdfdf115951fbff023cc615270a661af3c.patch

I'd love some pointers on how things like that are supposed to work with Nix.

I've split my changes into two commits as requested.
Not sure if https://github.com/LnL7/nix-darwin/issues/112 is relevant here, since I'm adding buildx support on the client. Ultimately I want Docker client to be able to find $HOME/.nix-profile/libexec/docker/cli-plugins/docker-buildx when I type docker buildx the same way man curl is able to find $HOME/.nix-profile/share/man/man1/curl.1.gz. Any pointers describing how this sort of thing is implemented with Nix would be appreciated.

Hm you are right I thought there was some mention of config.json issues there. I'm surprised the buildx command doesn't just work - e.g. for git-lfs it's just a separate binary that shows up as a subcommand along with the other git-* binaries. Does buildx need extra setup?

In Git's case it's a matter of running git-something from $PATH when git something is invoked:
$ echo -e "#!""$(which bash)\necho howdy" | sudo tee /usr/local/bin/git-ivan && sudo chmod +x /usr/local/bin/git-ivan
#!/Users/ivan/.nix-profile/bin/bash
echo howdy

$ git ivan
howdy

With Docker it tries to look up the binary in the list of hardcoded (+configured) paths:

https://github.com/docker/docker-ce/blob/master/components/cli/cli-plugins/manager/manager_unix.go#L5-L8

And $HOME/.nix-profile/libexec/docker/cli-plugins is not in this list.

Well I think checking only the nix-profile might run into issues with nix-shell or system installations - can docker consume a list of plugin directories from e.g. an environment variable? If not, how about something like docker.withPlugins to construct an env with specified plugins on the fly, similar to how python.withPackages works?

I managed to add buildxSupport argument to docker package in my last commit and it seems to work as intended. This is similar in spirit to what git does with sendEmailSupport flag.
With the following in my ~/.nixpkgs/config.nix:
{
  packageOverrides = pkgs: {
    docker = pkgs.docker.override { buildxSupport = true; };
  };
}

Building docker from my local copy of nixpkgs:
$ nix-env --file . --install docker

Produces the docker client binary that know to look for docker-buildx plugin in the right place:

/nix/store/33halip2fz0m844d3v1sq6mzm5lrjfic-docker-buildx-0.5.1/libexec/docker/cli-plugins/docker-buildx

$ docker buildx

Usage:  docker buildx [OPTIONS] COMMAND

Build with BuildKit

Options:
      --builder string   Override the configured builder instance

Management Commands:
  imagetools  Commands to work on images in registry

Commands:
  bake        Build from a file
  build       Start a build
  create      Create a new builder instance
  du          Disk usage
  inspect     Inspect current builder instance
  ls          List builder instances
  prune       Remove build cache
  rm          Remove a builder instance
  stop        Stop builder instance
  use         Set the current builder instance
  version     Show buildx version information

Run 'docker buildx COMMAND --help' for more information on a command.

With the only caveat that user has to enable experimental features in ~/.docker/config.json:
{
    "experimental": "enabled"
}

cc docker package maintainers: @offlinehacker, @tailhook, @vdemeester, @periklis

@r-burns, I amended my commits with your changes 👍

Is there anything else I should do to get this merged?

/marvin opt-in
/status needs_merger

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107302 run on x86_64-linux 1

1 package built:

docker-buildx

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107302 run on x86_64-linux 1

1 package built:

docker-buildx

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107302 run on x86_64-darwin 1

1 package built:

docker-buildx

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107302 run on x86_64-darwin 1

1 package built:

docker-buildx

Any reason not to merge?

Any reason not to merge?

Any reason not to merge?

The merge conflict and the thins I noted above.

Any reason not to merge?

The merge conflict and the thins I noted above.

Rebased on top of the latest master branch.

Rebased on top of the latest master branch.

Resolved conflicts by rebasing once again.

Resolved conflicts by rebasing once again.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107302 run on x86_64-darwin 1

1 package built:

docker-buildx

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 107302 run on x86_64-darwin 1

1 package built:

docker-buildx

Small correction was needed after a rebase: #110543.

Small correction was needed after a rebase: #110543.


Font Picker

Summary

Add font previews to font dropdown.
Add search to font dropdown.
Show previously used fonts at the top of font dropdown.

Relevant Technical Choices
This was originally implemented in this PR, but in a too complex way with a single huge component.
I restructured the current code a bit and removed all the search and recent fonts stuff to be replaced by something much simpler.
To-do

[x] Add scrollable list knowing which items are visible
[x] Preload visible font items using GFonts menu subset
[ ] Show fonts used in this story at the top
[ ] Add search in three sections
[ ] Highlight currently selection font

User-facing changes
Font previews are now visible in font selector.
Testing Instructions

Open font drop down.
See previews of all fonts
See fonts used in this story at the top
Search for a font, see search results in three sections::

fonts used in this story matching query
fonts starting with query
fonts matching query but not at start

Fixes #309
Fixes #30

@pbakaus @dvoytenko font picker #309 includes, popular fonts and suggested fonts (recent fonts) features. Any ideas for it?

@dvoytenko @barklund @spacedmonkey @miina @swissspidy Would you mind to check this PR when you have times? The functionalities are almost ready. Recent used fonts, search parts that include startwith and including parts, real time update for the font faces....
Still some updates needed but I think it's good to have some review.
Tests will be covered.

@ndev1991 Getting a few console errors when testing:

Also, only the system fonts seem to load the preview (related to the errors most likely), others not:

One more -- the list of fonts doesn't want to close:

Thoughts?

@ndev1991 Getting a few console errors when testing:

Also, only the system fonts seem to load the preview (related to the errors most likely), others not:

One more -- the list of fonts doesn't want to close:

Thoughts?

@miina I can't see any console errors. Can you give me the steps how did you product it?
And it uses popup and also useFoucsout at the same time. Means, when click the select button, useFocusout happens that closes the dropdown and then next the select button click opens it again. hmm... very funny.

I am getting console error too.

I am getting console error too.

I think this errors are not related with font picker

Testing accessibilty, seem like tabbing work which is great, but once you have a font selected, should pressing enter pick the font?

These errors seem related.

These errors seem related.

Yes, seems there is cors errors which I'm not seeing. Not but sure why... 🤒

These errors seem related.

Yes, seems there is cors errors which I'm not seeing. Not but sure why... 🤒

@spacedmonkey So you're not seeing the correct font preview, correct?

Not sure if this is what the dropdown should look like on this viewport.

@spacedmonkey So you're not seeing the correct font preview, correct?

I am getting a cors message becuase my local dev box is not hosted on localhost, but an IP.
I am guessing that issue would be on staging as well.

Should the font loading be debounced. Scrolling up fast through the font list loaded all the fonts, even if it was never really in view...

Once a font is loaded, should we keep a record of that and not try and load it again?

When reopening the font dropdown there is a jump in of fonts. See https://youtu.be/EtgjpNLzYHQ

@ndev1991  Heya 👋  Looks like merging #1273 caused a small merge conflicts here 😬  Let me know if you need some help with that or otherwise want to sync regarding that change.
I don't think the changes from  #1273 will help you here, as it won't affect getMenuFonts for example.

@ndev1991 Heya 👋 Looks like merging #1273 caused a small merge conflicts here 😬 Let me know if you need some help with that or otherwise want to sync regarding that change.
I don't think the changes from #1273 will help you here, as it won't affect getMenuFonts for example.

🤕 Yeah, seem conflict of fonts, but will be almost same. But not sure if next one will also affect current font provider? #1373

#1373 is mostly a behind-the-scenes change. It shouldn't affect things too much, and will probably land (if it lands) well after this PR. I wouldn't worry about it 🙂

@ndev1991 From testing:
I'm currently always seeing only ABeeZee in the recent fonts section, even though I've actually never used it/searched for it:

The popup not closing when clicking on the button still prevails:

Found a related bug. https://github.com/google/web-stories-wp/issues/1432

Some tests would be really useful here.

@dvoytenko Can you check this PR when you're available? Thank you

Hey @ndev1991 , what's the status on this PR? Anything blocking it or any todos?

Hey @ndev1991 , what's the status on this PR? Anything blocking it or any todos?

It's done. Waiting for review and to be merged!

@dvoytenko Now the font picker ready for review! Would you check it when you're available? Thank you

@googlebot I consent.

I have started a rewrite of this, as it was a bit slow and mostly in a single very big component. See status in PR description.
It's only a partial for now, still needs search and fonts used in this story, so there's no need for a full review just yet.

I decided to make this PR only solve a single issue - showing previews in the font dropdown (and highlight the currently selected font). Used fonts (no issue) and search (#30) is to be done in future PR's.

@barklund Doing some testing, some findings:

Currently, the "blinking" between changing to the real font takes quite a long time, it's a bit distracting:

The search doesn't seem to work anymore. E.g. if I click "R" to get to "Roboto" to see if it's highlighted as expected, nothing happens. (Does seem to work on staging)
I tried to test using it via keyboard, however, it doesn't seem to be possible to get inside the list via the keyboard. (Does seem to work on staging)

@barklund and @miina

Currently, the "blinking" between changing to the real font takes quite a long time, it's a bit distracting:

I'm wondering if it'd be better to just use font-display:block here. Perhaps it'd be better to show nothing that show flickering.

@googlebot I signed it!

In an old version of this PR, I was able to using the up and down arrow on the keyboard to select a font from the list. Now it just scroll the fonts. Other dropdown allow you to use arrow keys to select options in a dropdown. Also the esc key down not close the dropdown. Enter does not select and close the dropdown.
As the old dropdown did do this, we should at least not have a registion here.

@barklund Re-tested this and tried to use it with the keyboard again -- similar comment as from Jonny -- even though it's possible to access the list now via keyboard, it's not really possible to take further actions / select a font. It would be great to maintain the previously existing keyboard interactions.
Otherwise works well now, the flickering is gone, too!

@googlebot I consent.

@googlebot I signed it!

@barklund @swissspidy I have added a feature flag that toggles the old dropdown and the new one with the previews set to false by default. I also fixed a bug where the old dropdown would scroll into view immediately dismissing itself because there were too many items.

@googlebot I signed it!

@googlebot I consent.

@carlos-kelly, as this is now behind feature flag, it doesn't need QA, right?

@barklund Oh you're right. I guess not. It does fix the current font menu from auto-closing because of the scroll handler. But I can merge it in too.

@carlos-kelly, as discussed we also need a follow-up PR adding expected keyboard support to the new font picker.

@barklund Correct! I think this gets us to a mergeable spot then we can iterate on it. Also there will need to be some work in speeding this up. As scrolling fast through the 40 or so fonts while they pre-load can get a bit choppy.


Need to discuss accessing blobs

[Enter feedback here]
This topic does not necessarily need to be covered on this specific page, but there is no obvious page covering it anywhere else in this online document, and this page seems to me to be the most appropriate.
You need to tell the readers how to address a blob for retrieval purposes. Based on the limited information that is provided (i.e. the storage account has a URL, the container has a name and the blob has a name), one could guess that the blob is addressed as <storage account URL>/<container name>/<blob name>, but this is not actually stated anywhere. It would be helpful to make this clear.

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

ID: 4a056685-6734-380e-562c-92e30b8976a3
Version Independent ID: 1d17cc5e-f2b1-5489-79fa-e8557fadc5b5
Content: Introduction to Blob (object) storage - Azure Storage
Content Source: articles/storage/blobs/storage-blobs-introduction.md
Service: storage
Sub-service: blobs
GitHub Login: @tamram
Microsoft Alias: tamram

@MichaelBrooks-Informatyx Thanks for your feedback. We will investigate and update the thread.

@MichaelBrooks-Informatyx  Can you provide more context on the issue.

Hi Sumanth. I'll confess I'm having difficulty thinking what more context I could possibly provide. As I see it, my statement as to what is missing is clear and unambiguous. That said, I fully recognise that the reason ambiguities exist in text is because the authors already know what they mean and therefore find it difficult to put themselves in the shoes of readers who don't. One approach that might work is if you tell me which parts of my above text you do not understand, I can try to elucidate those parts further. Alternatively we could discuss it - I'm on Teams (I'm assuming you can access my email address from my profile given that Microsoft owns github).
Regards,
Michael

@MichaelBrooks-Informatyx I have assigned the issue to the content author to investigate further and update the document as appropriate.

@MichaelBrooks-Informatyx - thanks for your feedback! The article has been updated to include information about how to address blob and container resources.
#please-close


Support Rank Stabilized LoRA in the ModelConfig/LoraConfig

Tiny PR to support Rank Stabilized LoRA. To do this we:

Expose the argument in ModelConfig. It is copied straight from https://huggingface.co/docs/peft/en/package_reference/lora#peft.LoraConfig.use_rslora
Pass it through to LoraConfig in utils.py

Of course, a user could always do this outside TRL, but my motivation for enabling it this way was that I am working with a slightly modified version of the sft.py script, so it would be nice if I could enable Rank Stabilized LoRA with a single flag.

@JohnGiorgi  do we need to pin peft to some version for this?

@kashif Good question! Looks like yes... v0.8.0 based on digging through the release notes

yes i believe we will need to pin peft to that for this PR, also could you kindly also add a test?

@kashif Okay I bumped the peft dependency to >=0.8.0. I also added a test for get_peft_config (which looks to be untested), which also serves as a test to ensure use_rslora is getting passed from a ModelConfig to resulting LoraConfig

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@JohnGiorgi i believe its failing for one of the older python with:
AttributeError: 'str' object has no attribute 'removeprefix'

Fixed!

Merged, thank you @JohnGiorgi

Merged, thank you @JohnGiorgi

Thank you!


Password regex limitations

It should allow the user to type in whatever he/she wants. Why? Because they might be using password generators and this regex:
/^[a-zA-Z0-9+_-]{6,18}$/

... is limiting them to use only a small subset of characters and that is not a good practice. I think it should also allow to type in longer text, instead of 18, I would use for example 32.

Thanks, you're right, I've been intending to make this more flexible.

What about !@#$%^&*(){};:'"<>,./?~\ or 💩 or ☃? Why restrict a password to anything at all (maybe just a lower limit on length...)

Password regex is now removed


Fixed Inconsistency in column_type, allow_nulls, recovery_value

Thank you for providing such a great gem.
I created this PR because I ran into one problem while using this gem in my project.
Expect
In case of column type:" boolean " and allow nulls: false, nil should not be stored in the deleted column.
Actual
However, when the recover and recover! methods are used for the object of the class with the following conditions, nil is stored in the deleted column.
class ParanoidBooleanNotNullable < ActiveRecord::Base
  acts_as_paranoid column: "deleted", column_type: "boolean", allow_nulls: false
end

ParanoidBooleanNotNullable.create!
ParanoidBooleanNotNullable.first.destroy
ParanoidBooleanNotNullable.delete_flag # => true
ParanoidBooleanNotNullable.only_deleted.first.recover # => true
ParanoidBooleanNotNullable.delete_flag # => nil 'expected is false'

Changes
Set default value recovery_value to false for case ofcolumn_type:" boolean " and allow_nulls: false.

@mvz
Thank you for your review. It seems that I forgot to delete the library used during development.
I removed this and confirmed that the test was all green.
could you review again?

I'm experiencing some Rubocop Offenses. However, to address this warning, a refactoring that is not significantly involved in this fix is required. I'm thinking of proposing this fix in another PR again after merging this PR.
Is this PR mergeable?

Please don't worry about the RuboCop offenses for now.

Was good. I was relieved.

@mvz
The tests are all green, is this PR ready to merge?

Hi @mvz , is there something holding up this PR that needs addressing? We've hit this issue when trying to upgrade to 0.7.2 as well

Thanks for reminding me @zhengf2. I'll check this again this weekend.

Thanks, @soartec-lab!

This change has been released as part of version 0.7.2.


Allow controlling the navigation type from stripe.redirectToCheckout()

I'm having subtle UX issues that I'm having to hack around. My page flow is like this:
/pricing -> /redirect-to-stripe-checkout -> {stripeCheckoutUrl}
Currently because Stripe.js internally uses window.location.href = {checkoutUrl} it means that my 'headless' redirect-to-stripe-checkout page remains in the history stack and if the user taps the browser back button they land on back on the redirector page and therefore get stuck in an loop.
Ideally I'd like to be able to call window.location.replace(stripeCheckoutUrl) this means that the headless redirector page is removed from the history stack and hitting back would take them back to pricing.
Currently I'm having to do some hacks with history.replaceState() before redirecting to checkout which isn't great. If the API could be called as follows it would allow caller to navigate in the best way suitable to their app.
const checkoutUrl= await stripe.getStripeCheckoutUrl({ sessionId });
location.replace(checkoutUrl);

An alternative way to resolve this would be to allow getting the checkoutUrl on the server-side which would mean we could 302 to Stripe Checkout and avoid having to create a browser history entry.
Appreciate your time here 🙏

HI @wilsonpage - we've got a beta right now for server-side redirect on Checkout that you might be interested in. Can you shoot me a note to<EMAIL_ADDRESS>with your account ID and we can go from there? Thanks!

@andrewhowell-stripe I would like the same feature please. For E2E tests running with Cypress, I would need to redirect myself to the Stripe checkout page rather than Stripe doing it for me.

@andrewhowell-stripe I would like the same feature please. For E2E tests running with Cypress, I would need to redirect myself to the Stripe checkout page rather than Stripe doing it for me.

No problem, @alessioerosferri - shoot me a note to<EMAIL_ADDRESS>with your account ID and we'll go from there.

@andrewhowell-stripe I would like the same feature please. For E2E tests running with Cypress, I would need to redirect myself to the Stripe checkout page rather than Stripe doing it for me.

FYI you still won't be able to run Cypress tests against Stripe Checkout as the checkout.stripe.com has script to break out of the Cypress iframe.

If I knew the Checkout URL though I guess I could instruct Cypress to navigate to that one and complete the payment?
Am I missing something?

Oh that's interesting, when I last tested that Cypress blew up when Stripe
attempted to break out of it's iframe. Perhaps they shipped a fix.
On Fri, May 14, 2021 at 12:32 PM Alessio Eros Ferri <
@.***> wrote:

If I knew the Checkout URL though I guess I could instruct Cypress to
navigate to that one and complete the payment?
Am I missing something?
[image: image]
https://user-images.githubusercontent.com/28859897/118264893-50eec080-b4b0-11eb-8791-2d8f464d3113.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stripe/stripe-js/issues/175#issuecomment-841187460,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHZFBZNOYCLG56GOTCSLCDTNUC23ANCNFSM43BGKKWA
.

We hope to roll this out more broadly in the coming months. For future requests to access our Checkout server-side redirect beta, please reach out to<EMAIL_ADDRESS>Thanks!

Thanks for the response. Closing this issue now!


Integrate casper event toolkit: use upstream casper-types crate

incorporated these changes based on main in #108 i.e. dismissed in favor of #108


Misc issues with session docs

URL to the relevant documentation
https://docs.quantum-computing.ibm.com/run/sessions
https://docs.quantum-computing.ibm.com/run/run-jobs-in-session
Select all that apply

[ ] typo
[ ] code bug
[X] out-of-date content
[ ] broken link
[X] other

Describe the fix.
https://docs.quantum-computing.ibm.com/run/sessions:

Since data from the first session job is cached and used by subsequent jobs, if the first job is canceled, subsequent session jobs will all fail.

This sentence is only half true. Data from the first session job is no longer cached (but subsequent jobs would still fail if the root job is cancelled).

Jobs that fit within the maximum session time run back-to-back on hardware.

This line suggests that jobs in batch mode run serially. But in reality the classical processing part is actually done in parallel. We should also point that out as an advantage of using batch mode.

We should have code showing how to use session in iterative and batch modes. For example:

Batch:
The following example shows how you can divide up a long list of circuits into multiple jobs and run them as a batch to take advantage of the parallel processing.

backend = service.backend("ibm_sherbrooke")

with Session(backend=backend):
    estimator = Estimator()
    start_idx = 0
    jobs = []
    while start_idx < len(circuits):
        end_idx = start_idx + backend.max_circuits
        jobs.append(estimator.run(circuits[start_idx:end_idx], obs[start_idx:end_idx], params[start_idx:end_idx]))
        start_idx = end_idx

The quantum device is locked to the session user unless the TTL is reached.

"Locked" can be misleading. Other jobs (such as calibration or jobs already running when a session started) can still run. It'd be more accurate to say a session is active and its jobs get priority until ITTL or max timeout is reached.
Under https://docs.quantum-computing.ibm.com/run/run-jobs-in-session:

Data from the first session job is cached and used by subsequent jobs.

Same as 1 above.

Directly specify a string with the backend name. Example:

The code example should include service, i.e.
service = QiskitRuntimeService()
with Session(service=service, backend="ibmq_qasm_simulator"):
    ...

Suggestion from Jay:  Add a code example showing that you need close if you use a class


Not possible to run the full solution (api + front)

Hi, just tried to run the solution.
So faced a couple of issues:

after running "npm start" starts only frontend
after running api (several commands) it doesn't start

Expected result:
there's a readme.md file filed with all the needed information how to run the solution.
PS-могу помочь, дискорд здесь gravityperson#1679

it should be npm run dev:ssr. Readme updated
помочь с чем?

it should be npm run dev:ssr. Readme updated
помочь с чем?

с развитием angular части


Graceful deletion

Hi,
First I would to like thank you for this project, you did a great job and it helped me set up cloudfront lambda@edge on my current project.
I have a little problem regarding graceful deletion of my stack when using your custom resources. The stack always fails to delete itself because the lambda@edge function cannot be deleted.
Do you have any tips on how we could manage a graceful stack deletion using your custom resource ?
Thanks for your help !

Hi, thank you.
Unfortunately I'm facing the exact same problem and I have to manually delete things.
That's what I personally do:

First I'm deleting the stack that contains the CloudFront distribution
The deletion will take some time and will fail (because Lambda@Edge are spread all around the world and takes time to be deleted)
I manually hit the delete stack button in AWS console an retain the custom resources that contains the Lambda@Edge stacks (in us-east-1)
The stack is then completely deleted (expect for the Lambda@Edge stacks created by the custom resource in us-east-1)
Then I go to us-east-1 and manually delete the CloudFormation stacks that contains Lambda@Edge. Usually the stacks are removed straight ahead. If failure occurs, just retry and it'll work.

I'm know that's a bit hacky and needs manual intervention but I'm using this technique for 2 years now without much problem.
I hope that will help you.
If anyone know of to properly handle the Lambda@Edge deletion within the custom resource, please feel free to contribute 😉

Thanks for your answer.
I was doing the same actually... until I wanted to deploy an ephemeral environment to test against. This tests would have been launched for every commit pushed on my project. So It would hae been possible for me to delete these environments manually.
I ended up trying something else. I add Retain as deletion policy for the Custom::LambdaEdgeressources. Doing so my main stack deletion won't failed.
But I still have to delete stack created by the custom resource in us-east-1, so I added the stack name as an output of the custom resource. With this output, I can, in my main stack output the stack name created by the custom resource.
And my deletion workflow become:

Retrieve stack name of custom resources
Delete main stack
Delete Custom resources stack. This will failed, but leave the stack as Delete Failed

With these steps, I'm able to delete automatically my stack and mark the custom resources stack as Delete failed. So know, I just have to, once he while, go to cloudformation ui in us-east-1and delete the stack that failed to delete.

You can see that I tried my best to add a deletion retry strategy within the CloudFormation custom resource but this isn't enough 😞
https://github.com/yvele/aws-cfn-custom-resource-lambda-edge/blob/c9e6678154f465f561039264f81bb66a75ddcc30/stacks/custom-resource-lambda-edge/src/deleteStack.js#L40-L59
See also https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/lambda-edge-delete-replicas.html

, so I added the stack name as an output of the custom resource. With this output, I can, in my main stack output the stack name created by the custom resource.
@stonedMoose, how are you doing that? A GetAttr on the Custom::LambdaEdge resource?

@NickDarvey Sorry I've just seen your question... I think, I did modify this repository to add the output I needed:
index 4b6419a..a4945f5 100644
--- a/stacks/custom-resource-lambda-edge/cloudformation.yml
+++ b/stacks/custom-resource-lambda-edge/cloudformation.yml
@@ -129,3 +129,6 @@ Outputs:
     Value: !GetAtt Function.Arn
     Export:
       Name: CustomResourceLambdaEdgeServiceToken
+  StackName:
+    Description: Stack name
+    Value: !Sub ${AWS::StackName}
diff --git a/stacks/custom-resource-lambda-edge/src/cloudformation.yml b/stacks/custom-resource-lambda-edge/src/cloudformation.yml
index 7fd342b..0ff1dd6 100644
--- a/stacks/custom-resource-lambda-edge/src/cloudformation.yml
+++ b/stacks/custom-resource-lambda-edge/src/cloudformation.yml
@@ -114,3 +114,6 @@ Outputs:
       - UseDefaultFunctionRole
       - !GetAtt LambdaRole.Arn
       - !Ref FunctionRole
+  StackName:
+    Description: Stack name
+    Value: !Sub ${AWS::StackName}

In the end I ended switching to aws cdk instead of cloudformation :D and I'm using this package to handle lambda@edge:
https://www.npmjs.com/package/@cloudcomponents/cdk-lambda-at-edge-pattern
The deletion is handle gracefully in it, so maybe there is something to look for ^^


Broken link to libp2p glossary

Describe the bug
Link to Glossary is broken in Reference/CLI/CLI-Syntax.md: https://docs.libp2p.io/reference/glossary/#multiaddr
See https://app.circleci.com/pipelines/github/ConsenSys/doc.teku/1783/workflows/fd36e4f8-fd08-4d5d-af70-8aed852361db/jobs/4118?invite=true#step-106-485

[ ] Missing content
[x] Outdated content
[ ] Wrong content
[ ] Confusing or misleading content
[ ] Other

Change suggestion

Maybe replace link with https://docs.libp2p.io/concepts/glossary/#multiaddr

Because libp2p changed their site at the same time, the linke is still broken: see PR 220 (Nov 17th) https://github.com/libp2p/docs/pull/220/files#diff-2d9f958a427f8f649aefc70776e5fb6c2a62bde8c1645b311c7daa29163c75b7
So the new link that should work for this is https://docs.libp2p.io/concepts/appendix/glossary/#multiaddr


External Renderer pandoc and docx not working

Gitea version (or commit ref): 1.8.2
Git version: 2.17.1
Operating system: Ubuntu 18.04
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
After having success with jupyter notebooks and nbconvert, I wanted to try pandoc. So I installed pandoc and added the below to my app.ini:
[markup.word]
ENABLED = true
FILE_EXTENSIONS = .docx
RENDER_COMMAND = "pandoc -f docx -t html "
IS_INPUT_FILE = true

I have tested on the machine to ensure that pandoc is correctly configured and when I run pandoc -f docx -t html test.docx I get html output. However if I navigate to the repo I just get the 'view raw' hyperlink
I have also tested pandoc on other file types and it appears to be working - or at least doing something. With this it is like the command is not being called at all. Could it be because the file is labelled as a binary and so never even beings to render?
...
Screenshots

Looks as if it might be related to/a duplicate of #4996

As per my entry in #4996
I propose the following untested addition in order to allow external renderers to render binary files if specified by the user
	default:
		if blob.Size() >= setting.UI.MaxDisplayFileSize {
			ctx.Data["IsFileTooLarge"] = true
			break
		}

		d, _ := ioutil.ReadAll(dataRc)
		buf = templates.ToUTF8WithFallback(append(buf, d...))
		if markup.Type(blob.Name()) != "" {
			ctx.Data["IsMarkup"] = true // or ctx.Data["IsRenderedHTML"] = true
			ctx.Data["FileContent"] = string(markup.Render(blob.Name(), buf, path.Dir(treeLink), ctx.Repo.Repository.ComposeMetas()))
		}

at line https://github.com/go-gitea/gitea/blob/734aa965088fb72330f4ad744c187c614cce3d8b/routers/repo/view.go#L343

This issue is only about rendring from binary files, not to binary files? If so it is implemented by #8300 and can be closed?
It would be cool to also support rendering to binary files (mainly images), but that is maybe another issue.

Closing as implemented, reopen if anything else is needed


New relic Integration for logs and performance analysis

What is this issue related to?
Feature
Description
To be added.
Expected Behavior
To be added.
Current Behavior
To be added.
Additional Information (optional)
------------Not Applicable-----------
Confirmation Checklist

[ ] I have searched for similar issues before creating this one.
[ ] I have provided all the necessary information to understand and reproduce the issue.

@DeltaDynamo Kindly add the description.


Validator required_with bug

Laravel Version: 5.4
PHP Version: any
Database Driver & Version: any

Description:
required_with means The field under validation must be present and not empty only if any of the other specified fields are present.
but this code is failed
	$v = Validator::make([
		'test2' => null,
	], [
		'test1' => 'sometimes|string',
		'test2' => 'required_with:test1|string',
	]);

	if ($v->fails()){
		dd($v->failed());
	}

null is not a string, that's why it fails.

@themsaid  but the test1 is not exists, so test2 is not need verification

They are completely two different rules :) check this post: http://themsaid.com/laravel-advanced-validation-conditionally-adding-rules-20170110/

@themsaid ok, thanks


nix flake show --all-systems --json --show-trace fails

https://github.com/fedimint/fedimint/actions/runs/11942412321/job/33289396131
A cross dev shell on a aarch64-linux system fails to evaluate because we're missing something w.r.t cross-compiling to x86_64-android on that platform...

Weird, we are able to build these targets in CI, but somehow can't create a dev shell for them? Any idea what the reason could be (you mention fixing it in flakebox)?

I got nerd-sniped into debugging this and came up with one possible solution. Still not the most elegant, but pragmatic imo.


my device is offline

my device is connect to my pc ,ican send and receive a message from and to my pc.
BUT on android studio or on visual studio code there is NO device appears .
when i type in cmd "flutter device"
the message is gonna be "the device afhe323 is offline"!
"I Make My Device Debugging Mode"
can you help me ,please?

Hi @mariamalbarghouti
can you please provide your flutter doctor -v with device connected to your PC?
Thank you

hi @TahaTesser
i appreciate your help,the problem was "my cable when i changed it the problem solved".
after 4 days searching for a solution.
thanks

Hi @mariamalbarghouti
given your last message I feel safe to close this issue,
if you disagree please write in the comments and I will reopen it.
Thank you

@TahaTesser i'm agree :)


SCOOPER SE--Standard  (new type litter) can't get status of api

I got a new litter with type "SCOOPER SE--Standard" of catlink，conncet it with wifi, and update the lastest version in HA.
After about 3-5mins I get  entities like SCOOPER SE--Standard error/action/mode/state in HA, and id is "select.litter_box_599_xxxx_action" type ,that seems ok but state is null/unknown.
And I checked the log and statement info in my app ,the info of state and other were updated correctly, and updated the firmware of New litter , and litter restarted after that.
But until about 5 hours ,the entities of New litter (SE) wasn't refresh ,keep in null , and there're other right info in HA of Other catlink litter at same time (in same catlink account),like log and state .
I tried reboot HA twick (beofre /after updated firmware )and checked the pw/usr info in config yaml file , didn't find any issue ,and other litter info keeping refresh correclty.
Did anyone encounter this issue or solved some like that? Thanks.

Hi, I have the same problem : Got device detail for SCOOPER SE--Standard failed: {'returnCode': 4003, 'toast': False, 'msg': '未找到该设备', 'data': {}, 'timestamp': '1717307831280', 'success': False}.
I confirm that home assistant is connected to Catlink, because my phone was disconnected.
I use this model :

I can't wait for merging it ,so I just replace the file with your PR.
It works well ,you're my hero ,bro

I can't wait for merging it ,so I just replace the file with your PR.
It works well ,you're my hero ,bro

I will eventually update it with exposing other attributes and functionalities to support Lovelace UI card development, keep watchin’ ✌

After some fast testing about action /states/ enent log, I found they're all working well with .cn API
But the error entity (looks like online/offline status sensor) keeps showing unknown , maybe that's little issue thx again man.

It be solved perfectly by new updating ,thx @milosljubenovic
Issue should be closed now


⚠ Bofrost.de has degraded performance

In c97ccbb, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1200 ms

Resolved: Bofrost.de performance has improved in 5310f41.


Correct CreateRetentionPolicyIfNotExists

This function was incorrectly returning an error if the retention policy already existed.

@benbjohnson

Totally wrong.


Kitura leaking memory as crazy

Context and Description
I have created a simple HTTP based Kitura server with Basic Auth and put it under load. With every request it allocates memory that is never reclaimed back. Can be easily tracked using Xcode malloc debugger or Leaks Instrument. Happens on both Linux with swift 4.0.3/4.1 as well as on macOS 10.13.4 with Xcode 9.2 (swift 4.0.3) and Xcode 9.3 (swift 4.1).
Looks like RouterRequest and RouterResponse are kept in memory indefinitely causing loads of other classes including Socket and its underlying buffers to stay in memory forever.
Environment Details

macOS 10.13.4 with Xcode 9.2 (swift 4.0.3) or Xcode 9.3 (swift 4.1)
Ubuntu 16/04 with swift 4.0.3 or swift 4.1
Kitura 2.3.0
Kitura Credentials HTTP 2.0.1

Steps to Reproduce
Here is a simple project that demonstrates the issue:
https://github.com/mman/leaky-kitura
Expected vs. Actual Behaviour

Expected: Memory usage stable over time.
Actual: Memory increasing indefinitely with every request served.

This is the config that appears to be memory stable on both swift 4 and swift 4.1
$ swift package show-dependencies
.
├──<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   │   ├──<EMAIL_ADDRESS>│   │   ├──<EMAIL_ADDRESS>│   │   ├──<EMAIL_ADDRESS>│   │   └──<EMAIL_ADDRESS>│   │       └──<EMAIL_ADDRESS>│   ├──<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>│       └──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>│       └──<EMAIL_ADDRESS>│           ├──<EMAIL_ADDRESS>│           │   ├──<EMAIL_ADDRESS>│           │   │   ├──<EMAIL_ADDRESS>│           │   │   ├──<EMAIL_ADDRESS>│           │   │   ├──<EMAIL_ADDRESS>│           │   │   └──<EMAIL_ADDRESS>│           │   │       └──<EMAIL_ADDRESS>│           │   ├──<EMAIL_ADDRESS>│           │   └──<EMAIL_ADDRESS>│           │       └──<EMAIL_ADDRESS>│           └──<EMAIL_ADDRESS>│               └──<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>


WebDav Nginx Problem

Hay there, everytime when i try to open a folder or file via Browser on https://myurl.com//groupoffice/modules/dav/files.php it ends in 404.
example i try to open https://myurl.com/groupoffice/modules/dav/files.php/Shared i click on the Shared it ends in 404.
The Server is using NGinx and FastCGI.
What Nginx directives do i need to set?
Hope you can help me. Thx

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/39087469-webdav-nginx-problem?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F294006&utm_medium=issues&utm_source=github).

Hi, since you are running GroupOffice, you should report the bug there. GroupOffice is a user of sabre/dav. We're basically just another library for them.


Model weights when trained on HQ-Seg44k dataset

Hey there!
Segment Anything in High Quality (SAM-HQ) model released a dataset called HQ-Seg44k, which can be found here. It includes the DIS5K dataset, but also other data that might be worthwhile to train on.
We seem to be on a roll of releasing different model weights for different practical applications, I'd be curious to see how including this new dataset in the general use model with the swin_v1_large backbone could affect model quality.
Kind of like what has been done by adding a dataset here

Hey, Chris, thanks for your suggestion! I've downloaded them to have a check, and they seem to be good to put into training
👍. But currently, the GPU support from Fal might be going to be closed (they have offered me so many GPU resources for months, and I really appreciate them). Therefore, I might not have that many computation resources to conduct this kind of massive training again. But I've kept them and your suggestion, which may work in the future.


Is UnitTesting supported in Visual Studio Code's Power Query SDK

Some information about if UnitTesting is supported in Visual Studio Code code would be helpful because
I am getting "Cannot find name" error on
Fact and Facts

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

ID: 77917fa1-4cab-b13f-c4d2-d50cbe329f3d
Version Independent ID: 3d42574b-7213-c9a5-e5f9-0a825f760541
Content: TripPin 7 - Advanced Schema - Power Query
Content Source: powerquery-docs/samples/trippin/7-advancedschema/readme.md
Service: powerquery
GitHub Login: @ptyx507x
Microsoft Alias: miescobar

Thanks @karunpoudel-chr for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. However, this appears to be product feedback or a request for product support. For information about product feedback or customer support for Power Query, go to Power Query feedback.
We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Fix links with underscores

In the generic programming section, we have links such as
`Field <https://en.wikipedia.org/wiki/Field_(mathematics)>`_

in the RST.  I think these may be broken due to the underscore.  Lets figure out how to fix these in rst

I think the underscore is rST syntax for an anonymous link. But it might not work if there’s punctuation after it or something. Will check:

Usually those might be indicating to opening the link in a new tab (easy test of that theory).

My guess is that there is a way to escape characters in RST...


Progess bars not updating properly in Python Interactive Window on macOS

Steps to cause the bug to occur

Create a model that fits using a progress bar to monitor progress, such as Tensorflow.
Run model in interactive window with progress bar enabled.

Actual behavior
The progress bar does not update properly, printing a new line with every update, rather than using the existing line. Here is a screen shot:

Expected behavior
A progress bar that updates in the same line.
Your Jupyter and/or Python environment

Jupyter server running: Local
Extension version: 2020.8.108011
VS Code version: 1.49.0
Setting python.jediEnabled: false
Setting python.languageServer: Pylance
Python and/or Anaconda version: Python 3.8.5, Anaconda 3
OS: macOS Catalina
Virtual environment: conda

Python Output
See screen shot above.
Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer, @joyceerhl

Closing due to age. Please reopen if this problem still occurs in the latest build of VS Code Insiders


Create tests for Standalone Deposit / Withdraw / Transfer Page

Still valid, but will need to look into how much value specific tests for these standalone pages bring.

These pages are being retired so this wont be needed.


RabbitMQ Streaming Pipeline never acknowledges messages

If configure_consume is set there is the option to "auto_ack" the incoming messages when they arrive. However, this does not seem to work. Messages are never acknowledged.

If auto_ack is set to False, there seems to be no way to ack messages even within the code:
Your documentation  of this feature appears to be faulty as the demo code is not currently working with 0.8.86. The kwargs field of the transform function does not contain a channel entry and thus induces an exception.

Needs either bugfixes or further documentation.

Hi @gillhofer! I'm sorry you are experiencing this issue. We'll have a look and get back to you as soon as possible.

Sorry about the issue. It's a bug introduced in v0.8.80. We'll fix it.

@gillhofer This issue has been fixed in v0.8.87. Could you pull latest version and test it again?

Yes, the kwargs bug is fixed. Thanks for the quick solution.
(auto_ack appearently is still without function)

Could there be a similar issue with global variables and streaming pipelines? They do not seem to be added into the kwargs variables, although the documentation within mage saying so.


Onboarding: Interaction states for buttons

https://www.figma.com/file/mhImzKiTrvaMpSm1lQ5KcL/New-Onboarding-Flow?node-id=259%3A2134

https://www.figma.com/file/mhImzKiTrvaMpSm1lQ5KcL/New-Onboarding-Flow?node-id=259%3A2135

https://www.figma.com/file/mhImzKiTrvaMpSm1lQ5KcL/New-Onboarding-Flow?node-id=259%3A2170

Buttons here should have equal height (Primary button look shorter in height)
Hover states for secondary buttons look off

@andrewdamelio Still seeing the above. Has the update not been pushed yet?


Bump OpenResty/nginx (for Admin Router) from 1.13.x to 1.15.x

High-level description
Bump OpenResty/nginx (for Admin Router) from 1.13.x to 1.15.x
Corresponding DC/OS tickets (required)
https://jira.mesosphere.com/browse/DCOS_OSS-5320

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot label Work in Progress

@mesosphere-mergebot label Ready For Review


Better updates

ignore config.json

this is to prevent accidently uploading sensitive information that may be in config.json as well as the ability to update SquadJS via git pull without overriding config.json as it is no longer tracked

replace config.json with example-config.json

replaces the initial function of config.json with example-config.json and adds code to create config.json from example-config.json if it doesn’t already exist on initial run

@11TStudio Message updated in 2c5aa12

I'm reposting my comment because I think it may have been missed.

my understanding is any tracked file will be overwritten via git pull by the the file from the repo if it was edited but not committed/stashed, the idea being to not track the config file in order to preserve at least the basic settings for a server.
I'm almost thinking it might be best to split it into two files server_config and plugins_config so that the things that never really change can be in a static untracked file (server_config) and the plugins can be overwritten with the newest recommended default options as most people wont change them I'd think this would work for 90% of users and also provide a command to give to users for a simple way to update (git pull).

@Thomas-Smyth
I have written an alternative parser for the config file to do just this what do you think? I wanted to check with you before I get too far ahead of myself on this.

I believe I have come up with a much better solution than my original.

Instead of splitting up the file, this untracks the standard config file and merges it with the example-config  on runtime allowing for plugins to be added after the fact and for SquadJS to automatically update the untracked config file with any new plugins/settings preserving the user defined settings.

closing in favor of merging from branch instead of master


Visualize and save result pictures

Hello, author!
Thank you for the code, I would like to ask if you have written a visualization module and a saving result image module in the code.

Hello, we are still sorting out this part of the code, you can simply use the following code to implement it.
_, cd_preds = torch.max(cd_preds, 1)
cd_preds = cd_preds.data.cpu().numpy()
cd_preds = cd_preds.squeeze() * 255
cv2.imwrite(file_path + '.png', cd_preds)
And note the batch size should be set to 1 when using this.

Okay, thank you for your advice. Looking forward to your later update of the relevant code.

Hello, after I added the code to save the picture, the accuracy of the val set I got was very low. What is the situation?
INFO:root:EPOCH 40 VALIDATION METRICS{'cd_losses': 3.6140425014567423, 'cd_corrects': 90.64542682589175, 'cd_precisions': 0.491048978484091, 'cd_recalls': 0.1381559313391155, 'cd_f1scores': 0.17627280467674486, 'learning_rate': 1.5625000000000007e-05}

Could you provide more configuration information? How did it perform before adding this code? I suggest that you separate the training and testing/saving processes. It is difficult to make the model converge if the batch size is set to 1 during training.

Hello, I directly put the code you gave into the verification code, and set the batchsize to 1. Is there a better way to save it? Also, is it convenient to add a contact method.

I think you can temporarily refer to the relevant code in train.py and write a testing/saving .py file. You can contact me at<EMAIL_ADDRESS>

OK,thank you!


Fix errors in broken branches

Branches gemss, rainbow, mqdss and LUOV are broken.
Starting with gemss, find out why (some of the others still contain tabs, but that's not the only issue).
Tell me (Henning) what was wrong.

What do you mean by "broken", exactly?
I've now fixed the validation errors in the GeMSS branch. What I did was:

merge from master
fix errors in the scheme (error messages written down from memory, thus they are not correct by the letter):

must not have additional property ct/msg → this parameter has been renamed to ct|sig
side channel guarding must be an object → add information (see schema/implementation.json), or remove the whole entry
number of operations missing → add data (see schema/paramset.json)

I guess with the other schemes, the errors are similar. Can you try and fix them yourself using the instructions from above? If you encounter other problems, just ping me.

Also, I noted that the LUOV branch is based on the GeMSS branch. This makes things difficult (for example, because after git co LUOV you get errors within gemss) and it makes diffs between master and LUOV contain gemss changes.
I'd recommend you either a) create a new LUOV branch off from master where you cherry-pick the changes onto or b) wait until gemss is working and merged before you continue working on LUOV.

@Hennich I'd recommend that you continue working on the schemes using the enter-data-helper. This way you don't have to worry about creating valid .yaml files.

New problem: Locally the check works, but on github it doesn't.

Seems like there was an issue with github that canceled the workflow runs. I restarted the jobs and now they all validated.

I will close this. Further discussion can take place in the PRs.


Added things to README.md that I would have liked to have had in there.

Did not like how atom-haskligs alters fonts, especially when editing
Markdown. It took me a while to figure out how to put things back. README.md
now tells the reader how to do so.

README.md was missing the follow-up to installing the beautifier: setting it.
It now has that instruction.

You're very welcome. I'm volunteering for Snowdrift.coop, trying to improve doc & code so that newcomers get productive faster. Long story short: I was blown away by how hard it was to get setup for working on a Haskell project.
I ran into so much broken tooling. Jumping to definitions is the hardest thing to get right. For that I have to use my script + Codex. I have not found a working in-editor solution.

@JakeAndrew that is awesome! :)
I would actually recommend looking into using haskell IDE engine, which has a lot of functionality via the Language Server Protocol, and therefore also a lot of editor plugins, because it uses LSP.
It should also support jump to definition, but I'm unsure if it's currently working or not.

I followed the instructions you gave and installed the Atom packages it said to.
I still don't have a working, direct jump-to-definition.
But interestingly, the editor now supports "search results" (Ctrl-shift-r) jumping all on its own -- no external tag generation required there.
But it still cannot jump to the definition of something used in the repo but defined in a dependency, neither directly nor with "search results". So it's not dep-crawling like Codex.


Failed to open serial port

Hello! I'm a bit new when it comes to visual studio but when trying to connect my KMBox i get the following message printed
"Serial handle: -1"
"Failed to open serial port"
I've updated the port in the kmbox.cpp to the one corresponding to my kmbox (if that's the correct thing to do).
What am i missing out here for it to connect to my kmbox?

Try deleting the auto detecting line here: https://github.com/IntelSDM/Cod4DMA/blob/12ee629eaba560565cb74f2d6e98a07f71162151/Cod4DMA/Graphics/Utility/Kmbox.cpp#L64

Try deleting the auto detecting line here:
https://github.com/IntelSDM/Cod4DMA/blob/12ee629eaba560565cb74f2d6e98a07f71162151/Cod4DMA/Graphics/Utility/Kmbox.cpp#L64

tried deleting line 64 but still get the following
Connecting to KMBOX on port
Serial handle: -1
Failed to open serial port!

Well it appears that it can't open the serial port you inputted. Maybe check your devices baudrate.


Extend the tractusx-github.io "control Plane adpater" documentation

I want to get more information about the control plane adapter on tractusx-github.io
Within this PR a openAPI.yaml was generated. This should also be available in our kit documentation.
We also should be sure if the control plane adapter is documented in md.
Tasks

[ ] Adapt tractusx.github.io
[ ] Generate mdx files out of openAPI file
[ ] Link them
[ ] Add documentation to tractusx.github.io (since we dont use submodules, this has to be done manual)

@tuncaytunc-zf can we have a call on this? Is the related PR/changes? obsolete? Should we document something here?

I would reckon that this issue is obsolete at this point?
A full documentation with examples and error codes can be found on swaggerhub.


Confirm dependent elections

If possible. Confirm elections for: previous block, source or link

Awesome !

Of course with https://github.com/nanocurrency/raiblocks/pull/1303 (v18) we can confirm all dependencies, now it's limited only to active elections


Generic assembly

a few more tweaks but can be reviewed.

Can you provide a description?

New workflow:
Please

ignore reclassify_reads.py
see the sample new work flow execution at examples/assemble_blast.json and see the new classes that are created and how they are executed.


WIP remove NVCC_FLAGS from pkg_helpers

Towards https://github.com/pytorch/vision/issues/2377, just trying stuff for now.

@ezyang it seems that removing the NVCC_FLAGS leads to an IndexError in pytorch, as the packaging machines don't have GPUs. Would you know what an appropriate fix could be?

@seemethere I'm trying to replace NVCC_FLAGS with TORCH_CUDA_ARCH_LIST in torchvision, but this seems to lead to an error in pytorch's setup.py: https://app.circleci.com/pipelines/github/pytorch/vision/6799/workflows/557c1018-186a-40ea-8725-5302ef620374/jobs/455001
I'm not sure whether this is an issue with pytorch or if I'm doing something wrong. Would you mind taking a look? Thanks!

Here's the code that's failing:
    # The default is sm_30 for CUDA 9.x and 10.x
    # First check for an env var (same as used by the main setup.py)
    # Can be one or more architectures, e.g. "6.1" or "3.5;5.2;6.0;6.1;7.0+PTX"
    # See cmake/Modules_CUDA_fix/upstream/FindCUDA/select_compute_arch.cmake
    _arch_list = os.environ.get('TORCH_CUDA_ARCH_LIST', None)

    # If not given, determine what's best for the GPU / CUDA version that can be found
    if not _arch_list:
        arch_list = []
        # the assumption is that the extension should run on any of the currently visible cards,
        # which could be of different types - therefore all archs for visible cards should be included
        for i in range(torch.cuda.device_count()):
            capability = torch.cuda.get_device_capability(i)
            supported_sm = [int(arch.split('_')[1])
                            for arch in torch.cuda.get_arch_list() if 'sm_' in arch]
            max_supported_sm = max((sm // 10, sm % 10) for sm in supported_sm)
            # Capability of the device may be higher than what's supported by the user's
            # NVCC, causing compilation error. User's NVCC is expected to match the one
            # used to build pytorch, so we use the maximum supported capability of pytorch
            # to clamp the capability.
            capability = min(max_supported_sm, capability)
            arch = f'{capability[0]}.{capability[1]}'
            if arch not in arch_list:
                arch_list.append(arch)
        arch_list = sorted(arch_list)
        arch_list[-1] += '+PTX'

This suggests that the environment variable isn't being passed through.  Add this env var to the conda meta.yaml here https://github.com/pytorch/vision/blob/978ba613518dd5b8d04c2a717e6e5ccf7fb172c3/packaging/torchvision/meta.yaml#L41

I wonder if we should have TORCH_CUDA_ARCH_LIST exported from pytorch to cmake configs, so that torch vision can build consistently with the pytorch it's relying on?

@xksteven yes, I think this is a good idea and something I would love to see implemented in PyTorch. Can you create an issue in PyTorch to track this down?


Note about the need for the lang attribute to make CSS hyphenation work

To make hyphenation work it is advisable to set the lang attribute on the HTML element as suggested on quirksmode.org and adrianroselli.com.
<html lang="en">
…
</html>

Thanks!


rbac-view crushing in MacOS Monterey

I'd installed rbac-view via krew to my macbook with macOS Monterey

plugin is crushing:
❯ kubectl rbac-view
fatal error: runtime: bsdthread_register error

runtime stack:
runtime.throw(0x1e5be19, 0x21)
	/usr/local/go/src/runtime/panic.go:605 +0x95 fp=0x7ff7bfeff3c0 sp=0x7ff7bfeff3a0 pc=0x102a125
runtime.goenvs()
	/usr/local/go/src/runtime/os_darwin.go:108 +0x83 fp=0x7ff7bfeff3f0 sp=0x7ff7bfeff3c0 pc=0x10279c3
runtime.schedinit()
	/usr/local/go/src/runtime/proc.go:492 +0xa4 fp=0x7ff7bfeff430 sp=0x7ff7bfeff3f0 pc=0x102cb04
runtime.rt0_go(0x7ff7bfeff468, 0x1, 0x7ff7bfeff468, 0x0, 0x1000000, 0x1, 0x7ff7bfeff6a8, 0x0, 0x7ff7bfeff6d4, 0x7ff7bfeff710, ...)
	/usr/local/go/src/runtime/asm_amd64.s:175 +0x1eb fp=0x7ff7bfeff438 sp=0x7ff7bfeff430 pc=0x105463b
❯
❯ which kubectl-rbac_view
/Users/user92090/.krew/bin/kubectl-rbac_view
❯
❯ readlink /Users/user92090/.krew/bin/kubectl-rbac_view
/Users/user92090/.krew/store/rbac-view/v0.2.1/rbac-view
❯
❯ /Users/user92090/.krew/store/rbac-view/v0.2.1/rbac-view
fatal error: runtime: bsdthread_register error

runtime stack:
runtime.throw(0x1e5be19, 0x21)
	/usr/local/go/src/runtime/panic.go:605 +0x95 fp=0x7ff7bfeff390 sp=0x7ff7bfeff370 pc=0x102a125
runtime.goenvs()
	/usr/local/go/src/runtime/os_darwin.go:108 +0x83 fp=0x7ff7bfeff3c0 sp=0x7ff7bfeff390 pc=0x10279c3
runtime.schedinit()
	/usr/local/go/src/runtime/proc.go:492 +0xa4 fp=0x7ff7bfeff400 sp=0x7ff7bfeff3c0 pc=0x102cb04
runtime.rt0_go(0x7ff7bfeff438, 0x1, 0x7ff7bfeff438, 0x0, 0x1000000, 0x1, 0x7ff7bfeff680, 0x0, 0x7ff7bfeff6b7, 0x7ff7bfeff6f3, ...)
	/usr/local/go/src/runtime/asm_amd64.s:175 +0x1eb fp=0x7ff7bfeff408 sp=0x7ff7bfeff400 pc=0x105463b

macOS 12.0.1
rbac-view v0.2.1
krew v0.4.2
kubectl v1.21.1

@jasonrichardsmith a rebuild/new version would be immensely helpful. I think this is a fairly popular kubectl plugin.

Also got this error on Mac.
k rbac-view
fatal error: runtime: bsdthread_register error

runtime stack:
runtime.throw(0x1e5be19, 0x21)
	/usr/local/go/src/runtime/panic.go:605 +0x95 fp=0x7ff7bfeff2f0 sp=0x7ff7bfeff2d0 pc=0x102a125
runtime.goenvs()
	/usr/local/go/src/runtime/os_darwin.go:108 +0x83 fp=0x7ff7bfeff320 sp=0x7ff7bfeff2f0 pc=0x10279c3
runtime.schedinit()
	/usr/local/go/src/runtime/proc.go:492 +0xa4 fp=0x7ff7bfeff360 sp=0x7ff7bfeff320 pc=0x102cb04
runtime.rt0_go(0x7ff7bfeff390, 0x1, 0x7ff7bfeff390, 0x1000000, 0x1, 0x7ff7bfeff5b0, 0x0, 0x7ff7bfeff5d9, 0x7ff7bfeff615, 0x7ff7bfeff657, ...)
	/usr/local/go/src/runtime/asm_amd64.s:175 +0x1eb fp=0x7ff7bfeff368 sp=0x7ff7bfeff360 pc=0x105463b

Please release a new version to fix it, thanks.

@jasonrichardsmith are you actively maintaining this?

no


[BUG] fix _max_steps not initialized bug

When I do RLHF task it occured an attribute not found bug due to _max_steps may not be initialized before using it.

reproduce:
bash OpenRLHF/examples/scripts/train_ppo_llama_ray.sh

PTAL @xiaoxigua999


Merge master -> xplat

Gets up up to date with lazy, AssemblyFoldersFromConfig, and OrcasEngine/ConversionCore projects.

@dotnet-bot test OSX Build for CoreCLR please

Microsoft.Build.Tasks.csproj--can you rearrange this so the comment doesn't come unstuck from what it's describing?
 +    <!-- Needed by GenerateResource's ResXResourceReader: UNDONE: When CLR has moved this type to improve layering, remove this reference -->
++	<Reference Include="System.Runtime.Serialization" />
 +    <Reference Include="System.Windows.Forms" />

:shipit:


Issue when trying to extract in unix root directory (/): AbstractExtractFileTask

Hi,
I was running the zip4j library in a docker container. When I tried to extract something in the root directory, I received the following error:
net.lingala.zip4j.exception.ZipException: illegal file name that breaks out of the target directory: backup_keys.csv
at net.lingala.zip4j.tasks.AbstractExtractFileTask.assertCanonicalPathsAreSame(AbstractExtractFileTask.java:84) ~[zip4j-2.11.1.jar!/:na]
at net.lingala.zip4j.tasks.AbstractExtractFileTask.extractFile(AbstractExtractFileTask.java:52) ~[zip4j-2.11.1.jar!/:na]
at net.lingala.zip4j.tasks.ExtractAllFilesTask.executeTask(ExtractAllFilesTask.java:41) ~[zip4j-2.11.1.jar!/:na]
at net.lingala.zip4j.tasks.ExtractAllFilesTask.executeTask(ExtractAllFilesTask.java:17) ~[zip4j-2.11.1.jar!/:na]
at net.lingala.zip4j.tasks.AsyncZipTask.performTaskWithErrorHandling(AsyncZipTask.java:51) ~[zip4j-2.11.1.jar!/:na]
at net.lingala.zip4j.tasks.AsyncZipTask.execute(AsyncZipTask.java:45) ~[zip4j-2.11.1.jar!/:na]
at net.lingala.zip4j.ZipFile.extractAll(ZipFile.java:470) ~[zip4j-2.11.1.jar!/:na]
at net.lingala.zip4j.ZipFile.extractAll(ZipFile.java:441) ~[zip4j-2.11.1.jar!/:na]
I think I found the issue in the following file: https://github.com/srikanth-lingala/zip4j/blob/master/src/main/java/net/lingala/zip4j/tasks/AbstractExtractFileTask.java
and the following line:
String outputCanonicalPath = (new File(outputPath).getCanonicalPath()) + File.separator;
So when I'm in the root directory / it appends another /, which results in // and then the startsWith check fails. As a workaround I think I could create a subdirectory. Even though it might be strange to extract into the subdirectory, it should be technically possible in my opinion. When I was running the Spring Boot application locally, I did not face any issues. This was due to the relative path "." was not resolved to the root directory, but to the directory, where the application was running. Please let me know if this can be fixed.

I modified the code to append file separator only if it doesn't already contain one. I will include this fix in the next release.

Fix included in v2.11.2 released today


Fancy Zones: Multiple configs, based on hotkey used

Summary of the new feature/enhancement
2 Fancy Zone configs, first triggered by LeftWin Key and second triggered by RightWin key
By default, my Fancy Zones is configured to roughly:
5 -> 25 -> 25 -> 25 -> 20.
Icon Gap -> Email -> Code -> Reference/Browser -> Notes App
It would be great if I could trigger a second set of Fancy Zones:
40-40-20
Code/Text -> Reference/Browser -> Pomodoro timer
Proposed technical implementation details (optional)
Left Windows Key + Arrow Key = Triggers first set of fancy zones
Right Windows Key + Arrow Key = Triggers second set of fancy zones

@iyerusad
since we will implement what described in https://github.com/microsoft/PowerToys/issues/1411
I'm closing this as duplicate.
Thanks,


Add auto token refresh by login via developers.redhat.com

OSO does not have a service account for the master user so there is no long lived token available. Each every now and then we need to login to developers.redhat.com outside of webflow to get a fresh token on oso.
@bartoszmajsak

Didn't we "solve" it with long living token?


Failed to installed on Bluestacks

I am trying to install on bluestacks but keep saying failed to install.
I even changed to cmpatibility mode but think that have nothing to do it.
Does it hava someone to do with new version of bluestacks or need to use an old one?
I never tried it on emulator before and sorry if this was already asked before.

Android 7?


router: RouterScreen handler parameters

The RouterScreen handler parameters have several different names.
Some of them are (_,p) (context, p) (context, parameters) (context, params) etc.
Perhaps we could make all of them (context, parameters) for some form of uniformity. What do you think @pd4d10 ?

Yeah, it is better to keep consistency.

Yeah, it is better to keep consistency.


feat: add edgedb configuration

EdgeDB has a launch week and it's reaching 5.0, so it feels like a good time to add it (back) to Coolify.
This PR includes EdgeDB logo and adds the edgedb.yaml, which includes setting environment the variables, volumes, health checks, ports, and some of the Coolify magic.
Caveats
TLS:
EdgeDB would ideally have its own certificates generated with files. These could be set via the following environment variables:
# - EDGEDB_SERVER_TLS_CERT_FILE=
# - EDGEDB_SERVER_TLS_KEY_FILE=

These are included in the PR's edgedb.yaml, but commented out as above.
Currently each time the service is restarted, EdgeDB client starts warning:
Connecting to EdgeDB instance 'example-instance-name' at exampledomain.com:5656...
edgedb error: ClientConnectionFailedError: invalid peer certificate: BadSignature

Considerations
With External Database
EdgeDB can also be used with an external Postgres DB, which could be the way to go when in Coolify to leverage the backups etc.?

@andrasbacsai Any chance you could have a look at this?

any updates on this?

/bounty $50

Would love this

Thanks for the PR, this needs to be added to the codebase as a DB (I will leave this open for now).

Thanks for the PR. Databases should not be added as service templates, they need to be added differently to the codebase. You can search for keydb for example to see how other DBs have been added. Could you please update your PR with these changes? If you do not have time to do it this way, I will close the PR for now.

Oh I see. I will add this then. Thank you for the PR. I will test it and fix it if necessary.


authString bug fix

Hi,
I was trying to authenticate rstp server. After trying sometime it was not working.
I jumped to the library and found the problem with variable authString. Inside the if statement, let authString will create a new authString variable which will be destroyed once we are out of if the statement.
For this reason, the outer authString will remain empty string all the time.

Good spot, I hadn't tested auth stuff :)


Refactoring configuration to builder semantics

This PR refactors the setup of the Minimq client into a single Config struct. This removes the need to do run-time checks on the client state and makes it simpler to see which buffers are being used for what. It also cuts down on the number of arguments needed in Minimq::new()

Why not also move the broker to the config builder?

Was thinking about this as well. Seems reasonable to me :)

And is it ok to keep the state_buffer empty, i.e. why is state_buffer treated differently from rx_buffer?

The state_buffer is only needed if the user is going to be publishing above QoS::AtMostOnce. There's no need for any state buffer if you're only sending simple AtMostOnce publications