Global Scope

This is a topic that I've discussed independently with a few people here, and I'm still not sure that I'm 100% convinced by any of the solutions.
I guess the best phrasing of the problem is:
"Most PHP application instances will require no more than a single event loop. How can we ensure that this event loop can be easily accessed by code within the application?"
From here I believe there are a few options (feel free to add):

Straight forward dependency injection. The benefits of this is that it's explicit, you can see immediately whether a class uses the event loop and you have type safety. The negative is that it's repetitive and requires a lot of boiler plate code.

Make the event loop global. Have a class that can act as a locator with static properties (or functions) so that code anywhere in the application can retrieve the instance of the event loop.

Some hybrid between these, which allows users to choose whether they wish to explicitly require the event loop as a class dependency or just access it statically when they need it.

A big factor at play here is testability. I'm not convinced that anything other than option 1 is a good bet for writing testable code - but I have been told that the other options don't interfere with this. Has anyone got an example of well tested classes that retrieve their instance of the event loop from global scope? I'm happy to be convinced on this point :)
Another question I have is - can anybody think of a use case where a single PHP instance would wish to have multiple event loops operating at any one time?
Interested to hear your thoughts,
Andrew

Nah, please don't go with option 3. That's worse than any of 1 or 2.
There are situations where you'd like to have multiple event loops, but there are none where you'd operate two event loops in parallel (that's basically only possible via a busy loop switching between both).
Quick example: https://github.com/amphp/aerys/blob/master/lib/WorkerProcess.php#L22-L29
But this is such a rare case that you can comfortably back up the static instance and put it back after task done.
The use case basically is to have nothing interfering with your code… it's a defacto halting of the global reactor …
So, we still have intent to be able to swap event loops, do something on the new instance and put the old instance back, but not to operate multiple event loops in parallel.

For a long time we had the first option in Amp. It worked nice … but it brought literally no benefits except much boilerplate. For the sake of readability we made the overhead (in source) as small as possible, which is just simple functions. \Amp\once(). \Amp\onReadable(). \Amp\repeat(). etc.
Regarding tests, I'm just resetting the global loop to a new instance at the end of each test leaving unclean state.
Also we still could use mocks if needed by just setting the global loop to the mock. - So that's a non-issue.
Another note about class. I'm really in favor of using simple global functions (no static class - why a class? does this actually bring any benefit to have it being a class?).
For the simple reason that it abstracts the event loop completely away. You don't code to an event loop anymore, but you code to async primitives. [That's by the way also why I proposed #9 - it leaks information about the loop nature of it.] - I schedule something and it just happens. I have to know exactly nothing about the existence of the event loop in the background [well, except that I'm calling run() once … that's the only annoyance left].
KISS. Seriously, just KISS. Choose option 2: global event loop.

Here are my thoughts on the matter:

Defining an interface for event loop I don't think is enough. I want to encourage as many people as possible to write packages in an asynchronous way where applicable, but in most cases, they should be able to do so without depending on any particular loop implementation. For that to work, there needs to be some generic way of retrieving an event loop.
The idea here is that libraries can be written purely on the event loop interface, and it is up to applications to choose an implementation which is then used by all async code in the app.

Essentially the definition of an event loop is that it is shared. You can't easily run multiple event loops in parallel (on the same thread, anyway :stuck_out_tongue:), so for an event loop to be useful, all parts of an app need to use the same one.
So, there is only one, and it needs to be shared. That's pretty much the definition of a global.

If we go with (2), I don't have a strong preference between a static class or static functions; only that in either case, EventLoop::onReadable() and EventLoop\onReadable() read fine in source code, but just onReadable() would be really confusing.

So to summarize, I'm in full favor of (2).

@bwoebi The only practical benefit of a static class is that you can have distinct get() and set() methods for getting and setting the event loop instance, respectively, where you would need a single function to do both operations (unless you used global, which wouldn't be awful either).

Independently of this, I'm not sure where this is heading to - see also #8 ... If we provide functions to directly access a global instance, we also need to provide actual implementations of the interfaces.
@coderstephen hehe, I wanted to use global in Amp too, but we didn't … for esthetic reasons I guess :-D … also not sure if explicit get/set is actually a particular benefit.
Additionally, if we could use a shared namespace for async foundations (eventloop, promises, coroutines), it'll all read nicely as async\onReadable(), async\coroutine(), async\Promisor etc. :-D

+1 for static (option 2).
With static loop accessors, would the interop group be responsible for implementing that library?

With static loop accessors, would the interop group be responsible for implementing that library?

Might seem heavy-handed, but I would say yes absolutely. It is the only way to guarantee that packages that want to use interoperable async are using the same accessor, otherwise we're back to square one.
I would provide the loop accessors in the same package as the loop interfaces themselves.

The only thing related is the global-ness, but some of my opinions were inspired by this particular logging library for Rust. It spread in the Rust community rapidly because it accomplished this idea, where libraries can all use the same logger without knowing the implementation, and the application can choose the logger that is used by every single dependent library.

I'm not so sure about logging (after all different events might go to different loggers? … that's more a thing for something package global (if such a thing would exist)) … but that's unrelated^^
Actually, it's not necessary to have these accessors. … the important part is that things arrive in the same and unique global loop.

Yeah, I'm not certain that the design they chose is best for logging necessarily, but said design I do think would work extremely well for event loops.

There's only (2), all other options are non-options.

(1) Users will start using lazy injection, constructing another reactor otherwise. Having two reactors leads to subtle bugs. We had that in Amp, never again.
(3) Worst of all options, leads to the same issues as (2).

I don't really care if we choose functions or static methods, but I think I prefer functions, just for the sake of simplicity and the ability to alias namespaces like
use Async\Interop\EventLoop as Loop;

Loop\onReadable(...);

I'm a strong believer that this group should not seek to provide implementations of any of the standards that it produces. The intention to do so would get a lot of resistance in the FIG (it would never pass), and there are good reasons for this. I'd prefer to keep that particular discussion in #8, and I can expand on this if people want. The gist is that implementations require a far higher level of maintenance and are much less inviting to couple to than an interface.
Back onto the scope debate, it seems like 2 is the winning option (I thought it might be).
I could only live with functions if they were a proxy for a driver that was configured. As an example:
use Async\Interop\EventLoop;
use SomeAsyncProject\LibEventLoop;

$loop = new LibEventLoop();
EventLoop\set($loop);

// elsewhere...

EventLoop\onReadable(...);

//

EventLoop\run();

However, I think the version of this using a static class as a proxy to an underlying driver (below) is superior.
use Async\Interop\EventLoop;
use SomeAsyncProject\LibEventLoop;

$loop = new LibEventLoop();
EventLoop::set($loop);

// elsewhere...

EventLoop::onReadable(...);

//

EventLoop::run();

I think a static class is superior as it has the benefit of encapsulation, and can store the underlying driver as a static property:
class EventLoop
{
    private static $driver;

    public static function set(EventLoopInterface $driver)
    {
        self::$driver = $driver;
    }

    public static function get()
    {
        return self::$driver;
    }

    // ...
}

Using functions, there's no way to do this without creating an exposed global variable or a single function that you use to both get and set the driver (as AMP have done). I don't really like either of those options, the first loses type safety and the second breaks SRP.
On the other hand, I can't see any reason not to use a static class (except for having to type an extra character).
Interested to hear thoughts :)

Correction, you can use functions without breaking SRP - as you can do this:

function get()
{
    return loop();
}

function set(EventLoop $loop)
{
    return loop($loop);
}

function loop(EventLoop $assign = null)
{
    static $loop;

    if (null !== $assign) {
        $loop = $assign;
    }

    return $loop;
}

Still, this feels like a messy solution to a problem that static classes solve better.

@AndrewCarterUK Regarding get() and set(), I had the following discussion: https://github.com/async-interop/event-loop/pull/13#discussion_r54335136
which would render this redundant and it'd be a purely internal construct (no public facing set).

@bwoebi - How would you set the default event loop without set()?

I'm a strong believer that this group should not seek to provide implementations of any of the standards that it produces.

@AndrewCarterUK I think I misunderstood; what implementation are you referring to here? Sorry if it was already clarified.

@coderstephen I'm referring to any implementation of the EventLoopInterface (or whatever it ends up being named). That shouldn't be our concern, this should only focus on the Interface.
@bwoebi - I re-read, are you suggesting that all asynchronous code is run within a with() call?

@AndrewCarterUK yes I am suggesting a with() call for all async code, allowing proper scoping.

I think that, if we were to choose a get() and set() approach, that Andrew's comment puts into words much more clearly than what I was babbling about a couple days ago on why I prefer the static class approach. Of course, I'm not totally against functions.

I just feel that set() is a bit weird … it sounds like something allowing setting at any time. A set-once function is … well … bit weird to me. And with() is going around that problem elegantly IMO.

@bwoebi - I'm not opposed to that. I agree that with() is more elegant as it's very clear where the scope of the event loop starts and finishes. Would be interested to hear other opinions.

The alternative is what Bob is suggesting: a single with() method. To reiterate in this thread, looks like this:
EventLoop\with(new MyLoop(), function () {
    // do some async code
});

Only one function is required here, but all asynchronous code must be run inside the above closure. Suggested advantages are

Stack trace for async code will bubble up to the given closure.
The loop dependency is specified in the same place that it is run.

@bwoebi Perhaps give the functions better names, like init()?
EventLoop::init(new MyLoop());

// do some async code

EventLoop::run();

I feel that this system is akin to the way stream wrappers are set up, class autoloaders are registered, and the like. The main thing I'm not sure that I do like is that creating the event loop has to be in the same place as the beginning of the program; or rather before since it may very well be laid out like this:
use Async\Interop\EventLoop;
use Vendor\Loop;

// Do setup work first, which may possibly look like this for some vendor.
$factory = new Loop\LoopFactory();
$loop = $factory->createLoop();

// Now do the "real" program in a closure?
EventLoop\with($loop, function () {
    // Do async stuff in here
    // Which is indented from the above code...
});

I dunno, maybe I'm weird (actually, I definitely am!), but that indenting situation looks strange to me.
(sorry for many subsequent posts)

I need to have a play with a few examples, but I can see this approach working.
At the moment, I'm playing with:
EventLoop::execute(function () {
    // ...

    EventLoop::onReadable($stream, function ($stream) {
        // do handling
    });

    // ...
}, new MyLoop());

This would give us a proxy class that looked like:
class EventLoop
{
    private static $loop = null;

    public static function execute(callable $block, EventLoopInterface $loop)
    {
        self::$loop = $loop;

        $block();

        self::$loop = null;
    }

    private static function inScope()
    {
        if (null === self::$loop) {
            throw new RuntimeException('Not within the scope of an event loop');
        }
    }

    public static function onReadable($stream, callable $callback)
    {
        self::inScope();

        return self::$loop->onReadable($stream, $callback);
    }

    // ...
}

https://3v4l.org/1ZF30

I actually believe that stream wrappers are something horrible and should be banished from PHP ;-D
The core difference between autoloading and this is that you don't have to ever start it (nothing like run()), that it actually is falling back (you can have multiple registered autoloaders) and that it is truly global.
Don't think you can compare these well (apples and oranges ;-)).

@AndrewCarterUK yeah, basically this (just that you'd place self::$loop = null; inside a finally … but for the general idea, yeah) :-)

As an amendment to the example I posted, I believe the run() call would be implicit and should  also be within the try/finally block of the execute() method.

The code I posted also doesn't account for embedding event loops within event loops. This is the version I'm playing with now: https://3v4l.org/oSZ80
class EventLoop
{
    private static $loop = null;

    public static function execute(callable $code, EventLoopInterface $loop)
    {
        $previousLoop = self::$loop;

        self::$loop = $loop;

        try {
            $code();

            self::run();
        } finally {
            self::$loop = $previousLoop;
        }
    }

    private static function inScope()
    {
        if (null === self::$loop) {
            throw new RuntimeException('Not within the scope of an event loop');
        }
    }

    public static function run()
    {
        self::inScope();

        self::$loop->run();
    }

    public static function onReadable($stream, callable $callback)
    {
        self::inScope();

        return self::$loop->onReadable($stream, $callback);
    }

    // ...
}

Allowing for:
EventLoop::execute(function () {
    // ...

    EventLoop::onReadable($stream, function ($stream) {
        // do handling
    });

    // ...
}, new MyLoop());

@trowski @bwoebi @assertchris - Can I get your opinions on the discussion that has happened here?
The proposals are quite structural and I think we've reached a bit of a fork in the road until we can answer this question.

Reflecting the discussion in #5, I'd like to make a point about extensibility. As @AndrewCarterUK pointed out, offering additional functions other than the ones defined in the standard poses a problem for interoperable code. If a specific library provides the capability to listen for Windows process messages, for example, they might add an onWindowMessage() method. Now proprietary methods aren't bad, but we can't guarantee they exist in the standard.
The solution would be to allow arbitrary access to the event loop instance. For example:
EventLoop::execute(function () {
    EventLoop::get()->onWindowMessage(function ($message) {
        // do handling
    });
}, new MyWindowsLoop());

We can't guarantee that onWindowMessage() exists, but the application or library presumably knows already what specific loop they are compatible with, and so they use that interface. The solution would not be to try and extend EventLoop to have onWindowMessage(). It wouldn't be necessary.

@coderstephen - It's my honest opinion that we shouldn't try to encourage extensibility in any form. If you need functionality that isn't offered by the event loop - you need a different event loop. There's no point making any attempt to couple to a generic event loop - you need to couple to a specific event loop and this standard isn't useful to you.
There's nothing wrong with trying to solve the 95% use case, and that only.

I'm only suggesting that the interface can't promise anything other than the standard, but we should probably allow access to the actual event loop instance for the reasons I described.

I guess that would allow people to hack the event loop, but I'm not convinced it's a good idea to encourage. We should make sure if we refer to this behaviour in examples we do it like this:
function some_code()
{
    $driver = EventLoop::get();

    if ($driver instanceof MyWindowsLoop) {
        $driver->onWindowMessage(function ($message) {
            // do handling
        });
    }
}

EventLoop::execute(function () {
    // do stuff

   some_code();

   // do some other stuff
}, new MyWindowsLoop());

That is, if there is any chance that a different event loop should be used, they should be coupling directly to that implementation rather than the standard. The only use case I can see for this, is if there is an optional feature that is used when present.

To sum up your reply, yes to all of it. :wink:

Here's what I propose: either a static function structure, or a static class structure. The static function structure:
function get()
{
    global $__eventLoop;
    return $__eventLoop;
}

function run(callable $closure, EventLoop $loop)
{
    global $__eventLoop;
    $prev = $__eventLoop;
    $__eventLoop = $loop;
    try {
        $closure();
    } finally {
        $__eventLoop = $prev;
    }
}

// Other wrapper functions around EventLoop methods.

I really wish we had C-style statics that were file-level visible, but oh well. Or the static class equivalent:
class EventLoop
{
    private static $loop;
    public static function get()
    {
        return self::$loop;
    }
    public static function run(callable $closure, EventLoopDriver $loop)
    {
        $prev = self::$loop;
        self::$loop = $loop;
        try {
            $closure();
        } finally {
            self::$loop = $prev;
        }
    }
}

@AndrewCarterUK The problem with coupling directly is that we'd need to persist it separately again and get things messed up when stacking etc.
Hence the event loop should be accessible in any way from the exterior.

We need also some mechanism to store global state bound to a reactor. This can for global state for things like a default DNS resolver or filesystem driver.
Since we have I/O bound applications, most operations will have something to do with networking operations. Most of these need DNS, so it might make sense to have a global accessor for DNS just like for the event loop itself.
Otherwise it's possible to pass it optional and create a new one if no resolver is specified. Multiple instance aren't a problem for DNS, just for the event loop.

I think we've determined that we're going to use global scope ... This issue can be closed, I think.
@kelunik May you please open a separate issue for that specific point of global state storing?

As per usual, I'll close and if anyone has anything to add we can reopen


Add AbstractFileAccessor::GetFileInfo() for use in geFilePool

Created AbstractFileAccessor::GetFileInfo(), and switched calls to khGetFileInfo inside geFilePool.cpp use it instead.

FYI, I created a new file-accessor-plugin branch and retargeted this PR to it. That will keep experimental code out of the master branch.


fix: check for gsettings in audit-secureblue before extensions test

Closes #401

closing in favor of https://github.com/secureblue/secureblue/pull/406


Unclear interpretation of Scan Results

Hi,
first, thank you for sharing this tool, it is very helpful.
However, we have an issue with the interpretation of the scan result of log4j-scan.    Upon pointing it to a known vulnerable test system, it delivers consistent results.   We observe, that the remote address in the output is equal to the ip address of the system which is being tested.
We now found an anomaly scanning live systems.  The scan results are positive, but the remote address points to internet ip's which are totally unrelated to the live system.   They point largely to google-cloud servers or WoodyNet IPs.  Is this an indicator of compromise or just an anomaly?  We can't pinpoint where that exactly comes from.  Obviously it has something to do with interactsh, but it is unclear at which point the external IPs are fed in.  Here's a snippet of the output (target IP and token obfuscated, obviously):
{'timestamp': '2021-12-14T11:00:02.489479218Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:02.664853784Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:02.843801487Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:03.020504231Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:03.198737042Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:06.71736305Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:06.719712788Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:06.731693918Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:06.91019229Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
{'timestamp': '2021-12-14T11:00:07.08665417Z', 'host': 'xxx.xxx.xxx.xxx.[token].interact.sh', 'remote_address': '<IP_ADDRESS>'}
Any thoughts on this?  Has this also been observed by others, maybe?
ty
Roger

Hi @roger-wilco-66,
Thank you very much for the detailed explanation.
The remote_address on all cases will be the DNS server that was seen to be generating the DNS query. If you send
dig A sample.token.interact.sh @<IP_ADDRESS>

you would see that the remote address is coming from Google servers.
It maybe that the systems resolvers are using a resolver that is forwarding queries to these IPs. It should not be a reliable way to pin-point vulnerable systems.
The most reliable way is testing the remote address (or host) in the prefix of the DNS callback: this can lead to the vulnerable system.
All the best!

Mazin


my project fails without explicitly including lodash as a global dependency

my project doesn't use lodash, and it doesn't include inspire-tree-dom as a global script. Everything is included via my bundler. If I don't include lodash directly in my project and wire it to global it breaks at runtime.

resolved, thanks!


[ja] Fix links for k/design-proposals-archive

Fix those 404 links by resyncing with the en version of #34155.

/lgtm

This looks good to me. Thank you!
__
  Summary: :ok_hand: This code looks good  |  Tags: Efficient

This looks good to me!
__
  Summary: :ok_hand: This code looks good  |  Tags: Efficient

@annajung Would you like to add something to it?

/retest

sorry for late reviewing
/assign
/approve


Update all hold skill to use elemental hold tag

Should be using -> AttackTagElementalArtHold
Also should update all weapons/damage that check for ElementalArt

Is there anything in game that checks against this tag? If not then I'd prefer to just not deal with it at all right now...


Swapped out node-fetch for Fetch API

Fix for #6

Thanks @remiBoudreau!

Hey @remiBoudreau! Here's the pull request to update one of the libSQL internal packages to use the new isomorphic fetch: https://github.com/libsql/hrana-client-ts/actions/runs/7799693771/job/21271024422?pr=11 Any ideas why it's failing? I can debug this myself too a bit later.


Create generator-generic-ossf-slsa3-publish.yml-

new web browser-addition
Closes [issue link]
What's being changed:

Check off the following:

[x] For workflow changes, I have verified the Actions workflows function as expected.
[x] For content changes, I have reviewed the style guide.

completed


Gherkin info

Hi,
I'm having some trouble getting the Gherkin steps to show on a failing test.  I don't see how to have that working.  I see you have a TestResult.txt file, but I'm not sure where that came from and how it's being used.
love the design btw
-tomato

Hi,
It's probably too late, but I will reply to this in case it was useful to any other person.
As stated in the Usage, you need to get your specflow report generator to output the .txt file for you. See here for instructions.


Implement horizontal scrolling

Vertical scrolling works but there is a TODO to implement horizontal.
I have had a look at this and think I have a solution so will submit a pull request tomorrow.

That sounds great :-)
On Jun 15, 2015 9:15 AM, "stretch4x4"<EMAIL_ADDRESS>wrote:

Vertical scrolling works but there is a TODO to implement horizontal.
I have had a look at this and think I have a solution so will submit a
pull request tomorrow.
—
Reply to this email directly or view it on GitHub
https://github.com/One-com/knockout-dragdrop/issues/23.

Fixed by #24


Density tooltip edit

Density tooltip should note: "Density may vary drastically within a Community District."

Addressed in PR #375


Getting rid of sub panel

It doesn't seem to be used a lot (partially because it's hidden).

Changed to tuneable parameters.
https://github.com/damellis/sensors/commit/118d833153c8fcda42eb6668c260554e9326bf88


Moor + sqlcipher ... Invalid argument(s): Failed to load dynamic library (dlopen failed ... libsqlite3.so not found)

There are two ways to use an encrypted database with moor:

Using the encrypted_moor package: In this case, you shouldn't use a VmDatabase at all. Instead, use an EncryptedExecutor.
Using the sqlcipher_flutter_libs package: Here, you can use a regular VmDatabase after a one-time setup:

open.overrideFor(
        OperatingSystem.android, () => DynamicLibrary.open('libsqlcipher.so'));

The docs don't really explain the second option, I'll improve them.

What is the difference between the 2 methods ?

Basically, they're using two different implementations:

encrypted_moor uses an encrypted version of the sqflite package, so it uses Flutter platform channels and a platform-specific implementation for Android and iOS
The ffi-based approach is a little more independent of Flutter since it doesn't require platform channels or a plugin implementation. We have a special sqlcipher_flutter_libs package, but it doesn't contain any Dart/Java/Kotlin/ObjC/Swift/whatever code. Its only responsibility it to bundle the compiled SQLCipher libraries with your app.

For regular moor I usually recommend the ffi-based library since it's a bit smaller and can be more efficient for read-heavy apps. But both approaches work well in practice.

encrypted_moor works fine for me. The problem is I tried to do the 2nd approach ffi-based without success.
ffi-based error:
SqliteException(26): file is not a database, file is not a database (code 26)

Should I myself include the file libsqlcipher.so ? or is it enough with the pubspec dependencies as shown above ?
because I noticed the import in main.dart is from sqlite3 NOT Sqlcipher ... you know better
Finally, appreciate your fantastic work with moor. Thank you!

SqliteException(26): file is not a database, file is not a database (code 26)

It sounds like libsqclipher.so was properly included, but the password is incorrect? If libsqlcipher.so wasn't available, you'd get a "Failed to load dynamic library" error.
It's definitely weird that it works with encrypted_moor though... Are you using moor with isolates? You need to call overrideFor on every isolate that uses a VmDatabase later, but that's only the main isolate by default.

Nope, I don't use isolate.
The password is the same like in encrypted_moor
Can I try plain sqlite db + with empty password '' ? will it work ?

Opening a normal Sqlite db with the Sqlcipher library should work normally.

Opening a normal Sqlite db with the Sqlcipher library should work normally.

Yep it worked

@kemist88  Are you trying to open an existing Sqlcipher database copied from assets? Maybe you are providing an incorrect password or the database is encrypted with SqlCipher 3.x or lower, which uses a different format than 4 and above.

@kemist88 Are you trying to open an existing Sqlcipher database copied from assets? Maybe you are providing an incorrect password or the database is encrypted with SqlCipher 3.x or lower, which uses a different format than 4 and above.

The password is correct cuz I already tried it with encrypted_moor with success.
The version of sqlcipher I don't know it. I encrypted it with sqlitemanager. till now i can't find the version. I will search the vesrion

I will stick with encrypted_moor
I am also getting this error (when trying moor with desktop)

Any solution ?

I am also getting this error (when trying moor with desktop)

I think both approaches only work on iOS and Android by default. There is some documentation around this, but effectively you'd need to download the native libraries for each OS and abi you want to support and ship them with your app.

I think both approaches only work on iOS and Android by default. There is some documentation around this, but effectively you'd need to download the native libraries for each OS and abi you want to support and ship them with your app.

actually i like this encrypted_moor  very much :) it's working fine. Besides, I think Desktop is in alpha and too early for me to look into it this much. Thank you guys for this great lib and keep it on.


Decimal values linq tests fail on none US machines

When I run the tests on my machine which uses , for decimals some ST tests fail because ST can't convert the decimal values. When i change
    <TheDocumentsAre-row Name="C" Double="6.5" />
    <TheDocumentsAre-row Name="E" Double="10.1" />

To
    <TheDocumentsAre-row Name="C" Double="6,5" />
    <TheDocumentsAre-row Name="E" Double="10,1" />

All tests pass. This is however no solution. I guess this needs to be fixed in ST so that the conversion to decimal always use InvariantCulture?

Yeah, ST itself, and yep. I'm a little surprised this hasn't happened before. I can maybe get that addressed on Monday. I had some other ST todo items coming from Marten work.

@nieve has a workaround for this kind of thing in the integrated fixtures to just force the Thread.Culture to "en-US." I'll probably do the same here.
Go ahead and say "ugly American"

@tim-cools How bad would it be if I solved this by forcing ST to do all conversions with a given culture? Something like st.exe open [folder] --culture en-US ?

@jeremydmiller sounds good to me!
Initially I was thinking about an application setting, but this makes more sense in the case of ST...


Use new gcloud command-based appengine-maven-plugin

The new plugin deprecates the use of application and verion tags in the
appengine-web.xml file, so this change also removes those references.
Also adds a README with deployment instructions.

Added DO NOT MERGE label until I can test this with the Neos tutorial.

I tested this out on Cloud Shell in the tutorial draft and it worked well.
I filed https://github.com/GoogleCloudPlatform/app-maven-plugin/issues/118 but even though it can't find the SDK in Travis Compute Engine instances, it seems to work fine from Cloud Shell.

@lesv TBR
I need to update the Neos tutorial to use the master branch instead of gcloud or newplugins.


Fix pre-commit issues

In order to enable prow on install_yamls repo. We need to fix pre-commit issues discovered on this pr
https://github.com/openshift/release/pull/36396.
This patch fixes the same.

Thanks, Chandan
Apart from these failures I am seeing more issues locally:
Ansible-lint.............................................................Failed
- hook id: ansible-lint
- duration: 2.09s
- exit code: 2

INFO     Set ANSIBLE_LIBRARY=/home/sandeepyadav/.cache/ansible-compat/8ef565/modules:/home/sandeepyadav/.ansible/plugins/modules:/usr/share/ansible/plugins/modules
INFO     Set ANSIBLE_COLLECTIONS_PATH=/home/sandeepyadav/.cache/ansible-compat/8ef565/collections:/home/sandeepyadav/.ansible/collections:/usr/share/ansible/collections
INFO     Set ANSIBLE_ROLES_PATH=/home/sandeepyadav/.cache/ansible-compat/8ef565/roles:/home/sandeepyadav/.ansible/roles:/usr/share/ansible/roles:/etc/ansible/roles
INFO     Discovered files to lint using: git -c safe.directory=/tmp/install_yamls ls-files --cached --others --exclude-standard -z
INFO     Excluded removed files using: git -c safe.directory=/tmp/install_yamls ls-files --deleted -z
INFO     Discovered files to lint using: git -c safe.directory=/tmp/install_yamls ls-files --cached --others --exclude-standard -z
INFO     Excluded removed files using: git -c safe.directory=/tmp/install_yamls ls-files --deleted -z
INFO     Executing syntax check on ci/playbooks/collect_logs.yaml (0.45s)
INFO     Executing syntax check on ci/playbooks/deploy_podified_openstack.yaml (0.46s)
WARNING  Listing 17 violation(s) that are fatal
ci/playbooks/collect_logs.yaml:2: name[play]: All plays should be named. (warning)
ci/playbooks/collect_logs.yaml:5: fqcn[action-core]: Use FQCN for builtin module actions (file).
ci/playbooks/collect_logs.yaml:11: name[play]: All plays should be named. (warning)
ci/playbooks/collect_logs.yaml:14: fqcn[action-core]: Use FQCN for builtin module actions (file).
ci/playbooks/collect_logs.yaml:20: ignore-errors: Use failed_when and specify error conditions instead of using ignore_errors. (warning)
ci/playbooks/collect_logs.yaml:55: name[play]: All plays should be named. (warning)
ci/playbooks/collect_logs.yaml:58: fqcn[action-core]: Use FQCN for builtin module actions (include_role).
ci/playbooks/collect_logs.yaml:58: name[template]: Jinja templates should only be at the end of 'name'
ci/playbooks/deploy_podified_openstack.yaml:6: name[play]: All plays should be named. (warning)
ci/playbooks/deploy_podified_openstack.yaml:10: fqcn[action-core]: Use FQCN for builtin module actions (include_role).
ci/playbooks/deploy_podified_openstack.yaml:15: no-changed-when: Commands should not change things if nothing needs doing.
ci/playbooks/deploy_podified_openstack.yaml:18: no-changed-when: Commands should not change things if nothing needs doing.
ci/playbooks/deploy_podified_openstack.yaml:18: risky-shell-pipe: Shells that use pipes should set the pipefail option.
ci/playbooks/deploy_podified_openstack.yaml:35: no-changed-when: Commands should not change things if nothing needs doing.
ci/playbooks/deploy_podified_openstack.yaml:43: no-changed-when: Commands should not change things if nothing needs doing.
ci/playbooks/deploy_podified_openstack.yaml:43: risky-shell-pipe: Shells that use pipes should set the pipefail option.
ci/playbooks/deploy_podified_openstack.yaml:51: no-changed-when: Commands should not change things if nothing needs doing.
You can skip specific rules or tags by adding them to your configuration file:
# .config/ansible-lint.yml
warn_list:  # or 'skip_list' to silence them completely
  - experimental  # all rules tagged as experimental
  - fqcn[action-core]  # Use FQCN for builtin actions.
  - name[template]  # Rule for checking task and play names.
  - no-changed-when  # Commands should not change things if nothing needs doing.
  - risky-shell-pipe  # Shells that use pipes should set the pipefail option.

                           Rule Violation Summary
 count tag               profile    rule associated tags
     4 name[play]        basic      idiom (warning)
     1 name[template]    moderate   idiom
     2 risky-shell-pipe  safety     command-shell
     1 ignore-errors     shared     unpredictability, experimental (warning)
     5 no-changed-when   shared     command-shell, idempotency
     4 fqcn[action-core] production formatting

Failed after min profile: 12 failure(s), 5 warning(s) on 46 files.

bashate..................................................................Failed
- hook id: bashate
- exit code: 1

devsetup/scripts/crc-setup.sh:53:1: E003 Indent not multiple of 4
1 bashate error(s) found

bashate error is noticed in CI as well: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/36396/rehearse-36396-pull-ci-openstack-k8s-operators-install_yamls-master-precommit-check/1642759263900143616#1:build-log.txt%3A129

Thank you @Sandeepyadav93 for highlighting those issue. I fixed it now.

Thanks Chandan, looks good now. Tests are passing locally with the updates.


Support for Mobile WallpaperService

I'd like to use my assets and code for my Watchface on mobile as a WallpaperService.
Is this easy to support?

It's a bit harder to support, but it's doable.


Configuring standard kubernetes networking policies to take priority over calico GlobalNetworkPolicy

Hi,
I am having some trouble configuring a GlobalNetworkPolicy to work within environments that already have plain kubernetes (networking.k8s.io/v1) policies installed.
I am the administer of a cluster and would like to block access to a number of IPs but otherwise not interfere with any of my users traffic. My understanding is that calico will treat networking.k8s.io/v1 network policies with an order of 1000 and so if I use a GlobalNetworkPolicy with a lower order the kubernetes policy should take priority, is this correct?
The issue I am having is that the calico GlobalNetworkPolicy always appears to take priority over the kubernetes one. I expect this is an issue on my part with understanding the documentation, any pointers would be massively appreciated.
Converting users policy from kubernetes NetworkPolicies to calico NetworkPolicies in this case in unfortunately not immediately feasible, due to a large number of tenants.
Current Behavior
In my environment I am applying the following policy:
apiVersion: crd.projectcalico.org/v1
kind: GlobalNetworkPolicy
metadata:
  name: metadata-restriction-policy
spec:
  order: 999
  egress:
    - action: Deny
      destination:
        nets:
          - xxx.xxx.xxx.xxx/32
          - yyy:yyy::yyy/128
      protocol: TCP
      source:
        selector: all()
    - action: Deny
      destination:
        nets:
          - xxx.xxx.xxx.xxx/32
          - yyy:yyy::yyy/128
      protocol: UDP
      source:
        selector: all()
     - action: Allow
  types:
    - Egress

Implementing a default deny all policy (mimicking what users of the cluster may do).
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: default-deny-egress
  namespace: app-1
spec:
  podSelector: {}
  policyTypes:
  - Egress

Applying some test resources:
apiVersion: v1
kind: Pod
metadata:
  name: nginx
  namespace: app-1
  labels:
    app: nginx
spec:
  containers:
  - name: nginx
    image: nginx
    ports:
    - containerPort: 80
      name: http
    command: ["/bin/sh", "-c"]
    args:
    - |
      sed -i 's/listen 80;/listen 8080;/g' /etc/nginx/conf.d/default.conf && nginx -g 'daemon off;'
---
apiVersion: v1
kind: Pod
metadata:
  name: ubuntu
  namespace: app-1
  labels:
    app: ubuntu
spec:
  containers:
  - name: ubuntu
    image: ubuntu
    command: ["/bin/sh", "-c", "sleep infinity"]
---
apiVersion: v1
kind: Service
metadata:
  labels:
    app: nginx
  name: nginx
  namespace: app-1
spec:
  ports:
  - port: 80
  selector:
    app: nginx

I would expect to not be able to connect from the ubuntu pod to nginx, as I believe I have nothing in my GlobalNetworkPolicy that should affect this request and I have a default deny all policy in my user namespace. However my request is accepted:
 kubectl exec --stdin --tty -n app-1 ubuntu -- /bin/bash
root@ubuntu:/# wget  nginx.app-1.svc.cluster.local:80/
--2024-09-30 08:26:44--  http://nginx.app-1.svc.cluster.local/
Resolving nginx.app-1.svc.cluster.local (nginx.app-1.svc.cluster.local)... <IP_ADDRESS>
Connecting to nginx.app-1.svc.cluster.local (nginx.app-1.svc.cluster.local)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 615 [text/html]
Saving to: 'index.html'

index.html.11                           100%[==============================================================================>]     615  --.-KB/s    in 0s

2024-09-30 08:26:44 (2.90 MB/s) - 'index.html' saved [615/615]

Cheers for your help with this!
Your Environment

Calico version: v3.27.3
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes (v1.30.2)
Operating System and version: Fedora CoreOS 39 [2024-05-23]

     - action: Allow

I would expect to not be able to connect from the ubuntu pod to nginx, as I believe I have nothing in my GlobalNetworkPolicy that should affect this request

I believe this final rule that unconditionally allows egress traffic (with the exception of any traffic that was dropped by the previous rules) is coming into effect here.
This allow rule is being executed prior to the default deny implemented by the default-deny-egress NetworkPolicy`.
This is a bit tricky - the GlobalNetworkPolicy you have created is going to implement a default deny of its own, so the default-deny-egress NP is actually redundant in this scenario. So, you can explicitly deny traffic but then your namespaced policies will also need to explicitly declare the egress traffic that they want to allow.
There is actually a feature coming in Calico v3.29 - tiered policy - that will make this scenario easier, so keep an eye out for it!

hey casey - cheers for getting back to me so quickly, hugely appreciated :)

GlobalNetworkPolicy you have created is going to implement a default deny of its own, so the default-deny-egress NP is actually redundant in this scenario

okay I see, for our use case we would ideally leave it up to the tenants of the clusters to write network policies if they choose to. Each tenant has varying experience with kubernetes so expecting all to allow list all their traffic (while great) will be too challenging for some.
In this case I will probably wait for v3.29 to implement this, thank you for the heads up on this.
closing


[Cherry-pick 2.3][BugFix] Record query result if forward to leader (#11339)

Fixed a legacy bug that existed long before I joined the company.
Sometimes a SQL received by a follower FE may end up forwarding to the leader. The leader will process the statement and send the result back to the follower FE. The result will be stored in the ShowResultSet field of the thrift response message sent from the master to the follower.
However, if the forwarded statement is a query statement, the query result has no place to store in the response and will be directly written to the send buffer of the leader MySQL channel. After the follower receives the response message from the leader, it will find out that the ShowResultSet field is empty and will regard the master as a lower version of FE; thus, it will send a simple EOF to the MySQL client. The MySQL client receives an unexpected EOF and will report a malformated packet error.
This bug is seldom triggered because we never forward a query statement unless when the progress of the follower is far behind the leader. Even in that scenario, we would try to fix the FE so that the follower can catch up with the leader. As a result, this bug is ignored.
Luckily, in developing materialized view, we rewrite the show materialized statement to a query in the information_schema.materialized_views table. This bug has finally revealed its veil.
To fix this bug, we need to add a list of Bytebuffer in the thrift response message from the leader to the master. All the packets that are supposed to send to the MySQL channel of the leader will be stored in this list so that they can send to the MySQL client after the follower receives the response.
Fixes #1081
Manually cherry-picked from 243952ca39de7b127bde069ef5b648c83f20fd15

run starrocks_admit_test

run all


[FLINK-18695][network] Netty fakes heap buffer allocationn with direct buffers

What is the purpose of the change
This PR modifies the NettyBufferPool to also allocate direct buffers for the heap buffer request. This enables us not to change the memory footprint when upgrade Netty to 4.1.50-FINAL. In the future we could further decide how to adjust the Netty memory management.
Brief change log

3ef9f728f6a8005a7f91846bf32df96ee718c626 returns direct buffers for all the heap buffer allocation methods.

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Very sorry for not checking the tests at the first hand :( , I fixed the tests.

@flinkbot run azure


Added evaluation functions and cli interface

TODO

[x] Add tests
[x] Add functions docstrings
[x] Add example for cli interface for training and evaluation

:books: Documentation preview :books:: https://metatensor-models--21.org.readthedocs.build/en/21/

I added a first version for the usage from the command line.
To not dublicate files I am now using symbolic links. I know that this is problematic for people who use Windows but we can add an info to the readme.
If you have a better idea @frostedoyster or @Luthaf I am open to change it.

I think the overriding of the parameters of the architecture and of the trainer shouldn't go under "getting started", as they are more advanced functionality. I would rather have them under something like "How to / how to modify hyperparameters"

It still installs cmake 3.28 despite by commands, let's disable the MacOS tests


Merges latest fixes from Release/9.11.1 RC phase back into develop

Merges latest fixes from Release/9.11.1 RC phase back into develop

This is a release management task, as per our policy, we are self-approving it.


COnvert AlignedVec to vec without copy?

Hi,
Using Dioxus I want to send the result of the serialization over the network without copying a second time to a Vec beforehands.
The issue for the codec I opened can be found here.
to_vec() will reallocate. Unsafe casting didn't make it either. A Box will reallocate as well.
What are my options on the Rkyv side ?

Forget that, not necessary, I don't need that.
Sorry guys

I reopen this as multiple tests showed that it ends being necessary.
So, what is the best way to convert an AlignedVec to a Vec<u8>, without reallocating ?

You can't and should serialize directly into a Vec<u8> instead. AlignedVec allocates its memory with a alignment 16 and Vec assumes its storage is always allocated with the alignment of the contained type. When Vec<u8> gets dropped, it will free its memory assuming its layout has alignment 1, when it was actually allocated with alignment 16. This is unsound, and so the memory has to be reallocated.
If the global allocator is smart/permissive enough, then we might be able to improve the performance of AlignedVec -> Vec<u8> conversion by calling realloc instead of copying the data. In the worst case realloc will copy the data, but most allocators will not force a copy just to lower the alignment.
In 0.8 (main branch), you can serialize directly into a Vec<u8> with:
let bytes = rkyv::to_bytes_in::<_, Panic>(&value, Vec::new()).unwrap();

where Panic is the desired error type from rancor.
In 0.7 it's still possible but more annoying:
let value = "hello world".to_string();

let mut serializer = CompositeSerializer::new(
    WriteSerializer::new(Vec::new()),
    // or whatever your scratch space and shared serialize registries are
    FallbackScratch::<HeapScratch<1024>, AllocScratch>::default(),
    SharedSerializeMap::new(),
);
serializer.serialize_value(&value).unwrap();
let bytes = serializer.into_serializer().into_inner();

Looks like rust's realloc function doesn't support changing the alignment, only the size. So there's no more efficient way to do this conversion.

Thanks a lot, I will make it work that way


Made calico_ctl.node.get_host_ips fail-safe

Switched get_host_ips to use subprocess instead of sh since sh was returning blank responses on fedora and subprocess works cross platform. Also updated get_host_ips so it swallows exceptions, allowing 'calicoctl node' to continue with the node creation even if the IP check fails

Did you talk to @alexhersh about the Python library he was playing with on Friday?  From what I recall, it appears to return you the system's IP addresses in nicely formatted python objects - might be worth trying out.
Not sure what it uses under the covers, but it could make this code obsolete.

@tomdee Can you take a look at this? I'd prefer to implement this minimum impact fix which unblocks fedora before considering adding a new library.

LGTM


Pure Browser

Pure Browser not detected #240
Before
You are using an unknown browser that imitates Firefox 60.0 on Linux

After
Pure Browser 60.9 on Linux

Coverage remained the same at 100.0% when pulling cb8900b8619e5431c09cdb36f615db760e63e1fc on ayumi-cloud:pure into 4899110cd2f87b01e04ced62dbb9dec541031dee on WhichBrowser:master.


I cant run app on ios 17 physical device from android studio it switch to xcode and then android studio prompt "Error launching application on iPhone."

Steps to reproduce
Have a functioning Flutter project that builds on iOS 16.4
Update to MacOS Sonoma, iOS 17, and the Xcode 15
Run flutter doctor -v and see that everything checks out
Run flutter run --debug -v and see that it doesn't work
Expected results
I just want to run my app using android studio. For debug, Hot restart and hot reload
Actual results
I had a project that was working fine, with iOS 16, but I recently updated to the iOS 17. The project runs fine from Xcode using Product > Run, but it's a pain to have to rebuild every time. Note that the logs show that I am using the beta tool, but I get the same output on the stable tool. I want to debug my app using android studio
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration

[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.16.5, on macOS 14.2.1 23C71 darwin-arm64, locale
    en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.1)
[✓] VS Code (version 1.85.1)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

Untitled document.docx

From the file you shared above, I see below two errors:
Error: No pubspec.yaml file found.
           This command should be run from the root of your Flutter project.

 Make sure the path from which you run the command has pubspec.yaml file.
Error: Browsing on the local area network for iPhone. Ensure the
device is unlocked and attached with a cable or associated with the same local
area network as this Mac.
           The device must be opted into Developer Mode to connect wirelessly.

This indicates your device was probably locked when you tried running the app. Make sure the device remains unlocked.

@darshankawar the phone is attached and unlocked it was working very fine when I was on iOS 16 after I updated to iOS 17 the app is running but only from the Xcode side not from the android studio side. you can see this as well https://github.com/flutter/flutter/issues/133465 still no progress from the flutter side to resolve this issue.

@umairali4433 Do you still have the same error above after your device is connected?
Also, do you see this issue if running app from non-AndroidStudio editors (like Terminal or VSCode)?

the i issue i am having right now is if i run app with android studio it will not run but from xcode yes it does

Can you try this from Terminal or VSCode as well?
And please also try with a new Flutter project instead of your old project on your Android Studio. If it still occurs, can you let us know what the Flutter plugin version on Android Studio is?

i tried with vs code and with new project still not working

Maybe this is a genuine issue, as also raised on SO here.
@umairali4433  In order to track this issue separately, we would like to see what error are you getting when you run flutter run using AS or VSCode. Make sure your device is attached and unlocked and try to run the command flutter run -v and provide the verbose logs here. The one you provided earlier doesn't seem to properly replicate the error you are reporting, ie, we couldn't find Error launching application on iphone.

i tried with vs code and with new project still not working

Can you provide the verbose logs for this ?

you can see this as well #133465

Maybe it is similar as the root-cause mentioned in it. But we need to confirm if you are getting any other errors in the log while running on AS or Vscode.


Hostlisting should be moved to configuration file

The host listing, which require constant editing, should be moved to the configuration file instead of being kept inline in the GUI template.
jonasbn

Fixed with merge from #issue5 branch


Make call_create with host=NULL provide a default host name

In all situations that currently matter at least.
Fixes #2635.

LGTM otherwise. The format seems off, run clang-format?

Done.


Add Network Static Route/Gateway support

This commit enables static route configuration on EthernetInterface Implements CreateStaticRoute method which creates a new d-bus object with StaticRoute interface.
Tested By:
Run StaticRoute D-bus method and verified D-bus object and configuration Delete StaticRoute object
Change-Id: I3fbc6f85ede00b6c1949a0ac85f501037a69c831

upstream commit: https://gerrit.openbmc.org/c/openbmc/phosphor-networkd/+/63033


Italian translation of not translated levels (start with clone.js in remote level)

Hi
This is my first translation on learnGitBranching e my first pull request on GitHub.

Looks great!!

(Ill wait a few days to push the site in case you have any other PRs coming, its easier to do in one batch)

pushed it to production, @egiantu  feel free to reload the site and see your changes :)

pushed it to production, @egiantu feel free to reload the site and see your changes :)

Hi, Peter, I have reload the site, but I don't see changes. I load site with another browser too, but there aren't changes.
Regards

Oh good catch -- this was just an issue with renaming master to main. think it just got actually pushed now:
https://github.com/pcottle/learnGitBranching/commits/gh-pages


Code does not match class material

Isn't Louie or Gurik assigned to this? I don't really know what Professor Andrej Milivojevic wants.


Feature.apply() can fail silently

/**
* Once application has added a feature to an overlay, it can change one or more attributes of the Feature and invoke the apply method to
* have those changes reflected on the display. This is typically used to change position(s) as battle field objects move around.
*/
@Override
public void apply() {
try {
storageManager.apply(this, true);
} catch(EMP_Exception e) {
e.printStackTrace();
}
}
We should flow the exception to application else application presumes that apply was successful. One example is where apply is invoked on a Feature that was never added to the Map.

Closed as non-issue.  The CMAPI exceptions are passed to the caller.  The others are handled within EMP.


Support for Statistics Graph

Hi,
thanks for this great integration ! I'm glad to be able to monitor my containers on my HA :)
I'd love to put some CPU and memory stats in a Statistics Graph card but the entities are not listed. Apparently devs need to update something for the entities to be picked up by this card, here is the link :
https://www.home-assistant.io/more-info/statistics/
Could you please make the modification to allow us to use the statistics graph card ?

For long term statistics like disk usage, memory usage it would be awesome feature. Now showing chart for week of few containers is quite a pain because huge amount of data. Month is unusable. Statistics is a way to go, and it seams quite easy to get. I believe that entity have to define new state_class attribute and thats it. :)


Unable to login

最新版本，無法登入
Failed to login API: An error occurred when refreshing Living room support code.
2:01:21 PM – (ERROR) JciHitachi TW (custom integration) - message first occurred at 1:53:29 PM and shows up 2 times

The latest system status check is passing as you can see the badge on the top of the readme. Please check your environment. #31

Now system status is changed to failing, can you fix it? THX~


Publish configured cluster offer with AAG, AAD, and DB

Publish as 1.0.23.

arm-oraclelinux-wls-cluster-1.0.23-arm-assembly.zip

arm-oraclelinux-wls-cluster-1.0.23-arm-assembly.zip

arm-oraclelinux-wls-cluster-1.0.24-arm-assembly.zip

Published on Partner Center

arm-oraclelinux-wls-cluster-1.0.250000-arm-assembly.zip
Fix defaultValue problems found in acceptance testing.

Published on Partner Center

@jacobt123 I found another problem with the Configured Cluster offer: it was using the database pids from the dynamic cluster offer because there were no pids for the configured cluster database offer.
I fixed this in https://github.com/wls-eng/arm-oraclelinux-wls/issues/76 and the result is baked into 1.0.27 at https://github.com/wls-eng/arm-oraclelinux-wls-cluster/actions/runs/101885802 .  Can you please update the configured cluster offer with 1.0.270000 zip?

Use zip from https://github.com/wls-eng/arm-oraclelinux-wls-cluster/actions/runs/137861535 .

Please use Microsoft.Compute.UserNameTextBox. It comes with built-in validation for Windows and Linux user names
https://solutiontemplates.visualstudio.com/marketplacesolutions/_git/oracle/pullrequest/7661
Same as the other PR(7661), change all of user names to Microsoft.Compute.UserNameTextBox
https://solutiontemplates.visualstudio.com/marketplacesolutions/_git/oracle/pullrequest/7659

https://solutiontemplates.visualstudio.com//marketplacesolutions/_git/oracle/pullrequest/7660


Fix strict application to function-after with use_enum_values

Fix https://github.com/pydantic/pydantic/issues/9257
Selected Reviewer: @samuelcolvin

Please review @dmontagu

Yep. Added said issue to the 2.8 milestone!

Here's that issue: https://github.com/pydantic/pydantic/issues/9281


Thanks!

Wonderfully written and very helpful--thank you!

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

ID: c3a2bc83-7b1a-f28b-2ba6-4ec3c0d3f18f
Version Independent ID: c90f4bc6-9865-e5c7-53de-914dd398baaa
Content: Microsoft's brand voice : above all, simple and human - Microsoft Style Guide
Content Source: styleguide/brand-voice-above-all-simple-human.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

Thanks! We're very glad you find the style guide to be helpful.


Question: Integrate IdentiyServer with another authetication rest api.

Please only use the issue tracker for simple questions. For general security questions, or free / commercial support options do not use the issue tracker and instead see here for more details.
For questions containing large amounts of code we recommend submitting a question on stack overflow using the Identity Server 4. Our amazing community on Stack Overflow is there to help answer all of your questions.
Finally, please keep the issue concise and to the point if your question does include code be sure it is a Minimal working example. If you paste in more code than the text for your question then it probably belongs on Stack overflow.
Question / Steps to reproduce the problem
We want to integrate identity server with our internal access/authorization api. Any docs how it is possible?
Minimal working example

An external client ask auth credential to identity server
Identity server before granting credentials asks to our internal api.
The internal api grants access.
The identiry server grants access.

<code goes here>

Relevant parts of the log file
<log goes here>

not enough information

The autentication server can forward its requests to a rest api.

Sorry - either you take the time and effort to ask a proper and actionable question - or we can't help you.


ROX-25421: Compact error msg through removing duplicates

Description
The error message produced by the preflight check "Kubernetes authorization" contains many duplicated names of k8s resources. This makes it unnecessary long and unreadable - see screenshot.
In this PR, I remove the duplicates and sort the names of resources in the error message.

⚠ This PR does not fix the cause of the missing authorization - for that a follow-up may appear soon.
User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing

[ ] inspected CI results

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

[ ] Manually by triggering it on the cluster and comparing visually - TODO: paste screenshot "after"

/test gke-upgrade-tests

Well, that is not what I expected 😅

Let me give it another try

That dac74f9 should do the job. I will paste new screenshot as soon as the image is available

I updated the PR description with the screenshots obtained with  dac74f9

CI failures seem unrelated - I will merge this.


Adding option to Update and Delete Tickets

Feature Request
Feature Description
There should be an option to update and delete tickets as well.
Would you like to work on the issue?
Yes
@iamareebjamal I would like to work on this.

No, deleting tickets is involved in refunding orders and backend is not equipped with checks on pending or existing orders. So we already implemented this but it is hidden by default

Yeah, but you can update the ticket but still not sure how well it'd translate on the actual real world scenario. So let's skip this for the moment


RightClickCommand missing

I have a requirement to show the application window on any interaction with the tasktray icon.
LeftClickCommand works great, but there is nothing for right-click.
I tried handling MouseRightButtonUp but it doesn't seem to work. I think the event doesn't originate from the notify icon.
        <tb:TaskbarIcon x:Name="MyNotifyIcon"
                        NoLeftClickDelay="True"
                        IconSource="{Binding Path=DynamicAppIcon}"
                        DoubleClickCommand="{Binding ShowWindowCommand}"
                        LeftClickCommand="{Binding ShowWindowCommand}">
            <i:Interaction.Triggers>
                <i:EventTrigger EventName="MouseRightButtonUp">
                    <i:InvokeCommandAction Command="{Binding ShowWindowCommand}" />
                </i:EventTrigger>
            </i:Interaction.Triggers>
        </tb:TaskbarIcon>

Would it be possible to create a RightClickCommand ?

Handling the MouseRightButtonUp event normally in the view .cs file, the event still does not get fired.
This is the error message when attempting to use the command binding above:

System.Windows.Data Error: 2 : Cannot find governing FrameworkElement or FrameworkContentElement for target element. BindingExpression:Path=ShowWindowCommand; DataItem='ViewModel' (HashCode=6956456); target element is 'InvokeCommandAction' (HashCode=51797270); target property is 'Command' (type 'ICommand')

I  used to use TrayRightMouseDown for right click command.


FWI-3154 - Update the CI plugin to succeed on content errors and report them in action items

Checklist

[x] I have signed the CLA
[x] I have updated/added any relevant documentation

Description
What's the goal of this PR?

Continue processing reports for errors that are "soft," such as one failure out of many helm template commands, downloading images to be scanned, or processing OPA policies across multiple manifests.
Report CI errors as action items, via a new scan-errors report type.
Report-processing functions now return no report as nil, to differentiate whether  a report has data, since these functions can now return both a report and errors encountered during report processing.
Provide a CI_BASE_PATH environment variable to override the default base path of /app/repository - useful when testing locally.

What changes did you make?

Return a new ScanErrorsReportResult type for CI errors that we want to reflect as action items. This type satisfies the Go Error interface, and allows specifying additional context and metadata at the time the error happened.
Update the CIScan.ProcessRepository() function to add the new scan-errors report type to the slice of reports, instead of short-circuiting and failing a CI scan when errors are encountered.
Update error-processing throughout the CI plugin, to return the above ScanErrorsReportResult type instead of a traditional Error.

Internal Ticket FWI-3154

Great changes Ivan, I think the only required change is regarding the Filename json tag missing

Thanks @vitorvezani
Did you also see the comments I left in the PR, calling out a few things? One example is this question about whether we should continue if merging images fails.

@ivanfetch-fw, just did... I think you have made good judgment calls, I left some thoughts, but none are really any definition or request for change

@ivanfetch-fw we might wanna to hold-off the merge until the fix for:

FWI-3386 [CI Plugin] skip images.docker imgs that uses env. variables on CI plugin

gets in, since we cannot release hotfixes


[Feature request] Layouts code splitting

Clear and concise description of the problem
Is it technically possible to implement code splitting for layouts?
Some layouts can be very different from the main layout. However, they can be large and not very significant from the user's point of view. It would be great to be able to put them in a separate chunk.
Suggested solution
// vuepress.config.ts
import { defineUserConfig } from '@vuepress/cli'
import { path } from '@vuepress/utils'

export default defineUserConfig({
  clientConfigFile: path.resolve(__dirname, 'client'),
})

// client.ts
import type { ClientConfig } from '@vuepress/client'

import { defineClientConfig } from '@vuepress/client'

import NotFound from '~/pages/not-found.vue'
import SpecialPage from '~/pages/special.vue'
import Layout from '~/pages/layout.vue'

export default async (): Promise<ClientConfig> => {
  let Special = (await import('~/pages/special.vue')).default
  return defineClientConfig({
    layouts: {
      NotFound,
      Special,
      Layout,
    },
  })
}

<!-- content/special.md -->
---
layout: Special
---

I do think you should be able to define a AsyncComponent and use it as layout component.
References:

https://vuejs.org/guide/components/async.html

// client.ts
import { defineClientConfig } from '@vuepress/client';
import { defineAsyncComponent } from 'vue';
import type { ClientConfig } from '@vuepress/client';

import NotFound from '~/pages/not-found.vue'
import SpecialPage from '~/pages/special.vue'

export default async (): Promise<ClientConfig> => {
  let Special = (await import('~/pages/special.vue')).default
  return defineClientConfig({
    layouts: {
      NotFound,
      Special,
      Layout: defineAsyncComponent(() => import('~/pages/layout.vue')),
    },
  })
}


Multimonitor choppy mouse when not focused on a window

Been using hyprland without any issues.
For some reason the mouse movements seem super choppy if no window is focused on any of my 3 monitors. If it's over a window it's smooth.

I think it's a dupe of #267 and #290 but holy shit you actually made me be able to reproduce this yooo

It's pretty subtle for me so I will ask: does it go away if misc:no_vfr is set to true?

@vaxerski for some reason I can't see this option as valid tried appending either
no_vfr=1

or
misc{
no_vfr=1
}

But says that there is no such configuration choice

The wiki tells you how to use categories. You have a bunch in your config already.
misc {
    no_vfr = 1
}

@vaxerski ok tried it and it didn't help, the behavior is very weird, like at first everything works great and after you open a window it starts chopping

after you open a window

You said it happens when there are no windows, though?

No window focused, not in general, sorry for my poor explanation 😅

try with 09495375b52de47ddd97a46be1159721d207c862

@vaxerski beast! Works like a charm :D Thank you so much for your swift response


Make typeasync write slower

i know making typeasync write slower in javascript but i cant figure out how to do it using puppeteer sharp

TypeAsync has a delay option.


(WIP) Update supporting versions of spree and ruby

Update supporting spree version to 3.2, 3.5, 3.6
Update ruby version to 2.3.7, 2.4.4, 2.5.1

Right now globalize gem doesn't support Rails 5.2


Object.Object.defineProperty.get is deprecated.

Requiring $ from atom is no longer supported.
If you are using space-pen, please require $ from atom-space-pen-views. Otherwise require jquery instead:
{$} = require 'atom-space-pen-views'
or
$ = require 'jquery'
Add "atom-space-pen-views": "^2.0.3" to your package dependencies.
Or add "jquery": "^2" to your package dependencies.
Object.Object.defineProperty.get (/Applications/Atom.app/Contents/Resources/app.asar/exports/atom.js:48:11)
Object.<anonymous> (/Users/rainer/.atom/packages/supercollider/lib/post-window.coffee:2:1)
Object.<anonymous> (/Users/rainer/.atom/packages/supercollider/lib/post-window.coffee:102:4)
Module._compile (module.js:452:26)
Object.requireCoffeeScript (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/coffee-cash/lib/coffee-cash.js:85:19)
Module.load (module.js:347:32)

ah, I missed one. fixed

There are 4 more left ;) Just run the package in current Atom (.204). It doesn't feel friendly to report them all individually... They are saying they will remove deprecated API's today...

I did that and I already updated those and made a new release.
I don't understand why its not showing them to me. Probably because its installed with link (development mode) and there is some bug in deprecation cop.

ah now its showing them to me.


MSVC compiler spams version information

In AXConfigDefine.cmake there's:
if (FULL_MSVC)
    list(APPEND _ax_compile_options /Bv)
endif()

This adds /Bv option that outputs compiler's pass versions for each compiled file like this:
[661/1090] Building CXX object engine\axmol\core\CMakeFiles\axmol.dir\base\Director.cpp.obj
Compiler Passes:
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\cl.exe:        Version 19.39.33523.0
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\c1.dll:        Version 19.39.33523.0
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\c1xx.dll:      Version 19.39.33523.0
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\c2.dll:        Version 19.39.33523.0
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\c1xx.dll:      Version 19.39.33523.0
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\link.exe:      Version 14.39.33523.0
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\mspdb140.dll:  Version 14.39.33523.0
 C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1439~1.335\bin\Hostx64\x64\1033\clui.dll: Version 19.39.33523.0

This adds a lot of unnecessary noise. Can this be removed or at least made optional?

just delete it


React step 5 - Endicott

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

What a website!  I think you should build on this to create kaiyik-zon!


Question on UIActivityIndicatorView compatibility

Hi, I have tried the library it is very perfectly if I am adding an UILabel or UIView. But now I am trying to add UIActivityIndicatorView, seems it doesn't work. Is this syntax below correct ?. I tried this but the view is not showing anything,  but once I try with UILabel or UIView, it works fine.
UIActivityIndicatorView(style: .large)
                .swiftUIView(layout: .fixed(size: CGSize(width: 100, height: 100)))
                .set(\.color, to: UIColor.blue)
                .set(\.isHidden, to: false)
                .fixedSize()

second question, how did I trigger a function that preserved from one of the subclass of UIView ? Let say this activity indicator has an open function called as startAnimating(). however I can't using the declarative syntax in above code, is it expected to only set a value but can't call any derived method ?
Thank you before, @AvdLee

It might not be visible because it's not animating. It's likely better to wrap your activity indicator in a UIViewRepresentable and see if that works

It might not be visible because it's not animating. It's likely better to wrap your activity indicator in a UIViewRepresentable and see if that works

Got it. I try with UIViewRepresentable and it is works.
I will closed this issues. Thank you @AvdLee

Got it. I try with UIViewRepresentable and it is works.
I will closed this issues. Thank you @AvdLee


Swagger file modification for Fileshare snapshot API's

What this PR does / why we need it:
Fileshare snapshot Api modification in swagger file
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

These changes already merged in PR #1112, so closing this PR


How to load Pre-trained model

How to get the pre-trained LayoutLMv2 model? Is it's code publicly available ?

Yes, we show the pre-trained model in this section. If you use the script in the fine-tuning example, it will automatically download and load the pre-trained model.


Refactored and modified private api for resize functional op

Description:

Refactored output size computation into a single method _compute_output_size and removed max_size arg from F_t.resize and F_pil.resize.

We can also reuse _compute_output_size in proto for bboxes (right now max_size is not taken into account)

Minor changes and cosmetics in tests
Temporarily I also have to modify prototype functional implementation resize_image_*. This will be fixed in https://github.com/pytorch/vision/pull/6187

@vfdev-5 @datumbox
@vfdev-5 pointed me to this PR from our discussions in https://github.com/pytorch/vision/pull/6209#discussion_r908294878
As discussed over there, functional_pil and functional_tensor are technically public, unfortunately. I understand some of these changes are useful for the prototype area (where breakages are OK), but would it be possible to preserve BC here? It looks like the removal of the parameters aren't strictly necessary.

For example, perhaps it's possible to call _compute_output_size within functional_pil.resize and functional_tensor.resize instead of calling it in functional.resize?

It's possible if we introduce additional _utils.py as a common place for functional_pil.py and functional_tensor.py

I guess we could just define it in functional_tensor and have functional_pil rely on it?

@NicolasHug as far as I know those APIs are private and this change has been done many versions ago. I don't think we should worry about BC here.

Sorry I was referring to when the API became private not to this PR. Let's continue our discussion at https://github.com/pytorch/vision/pull/6209#discussion_r908474578 to avoid further confusion


Charts ideas

Post your ideas here!
Reminder of the available fields for analysis:

start: command start time
stop: command stop time (we can compute duration)
uuid: session id, shared with subshells
parents: process parents, newline-separated commands, something like:/bin/bash
/usr/bin/python /usr/bin/x-terminal-emulator
sh -c x-terminal-emulator
tint2 -c /home/pawamoy/.config/tint2/tint2rc
/sbin/init

host: hostname
user: username
tty: something like /dev/pts/0 or /dev/tty1
path: command working directory
shell: shell used (/bin/bash, /bin/zsh, etc.)
level: shell level (usually 2 in a GUI, 1 in a Ctrl-Alt-F1/6 TTY)
code: command return code
cmd: the command itself

Usage of builtins, aliases, functions, GNU tools. Maybe it would require a new field to be added to history, because we might not be able to determine afterwards the type of the command.

Stats on number of different sessions per day
Average command duration
Put more series in hourly/daily/monthly with different hosts/users


Break core rules and UI into separate repos

The core rules and the UI need to be broken into separate repos. As it is, it's too hard too manage.
A sub-point of this is that the util directory needs some fundamental reworking since it's a hairball.

This is now done!


Appstudio update hacbs-test-project-afwt

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-afwt-on-pull-request-
creating pipelinerun hacbs-test-project-afwt-on-pull-request- in namespace rhtap-demo-pudd-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


the default  size of the database connection pool(5) should greater than threads num in puma(8~16).

https://devcenter.heroku.com/articles/concurrency-and-database-connections

@0x01f7 好建议, 但需要一个压力测试来验证后再进行调整.


feat: Add support for drizzle orm also

Is your feature request related to a problem? Please describe.
when users run npx create t3-app the users are getting only prisma. It is better to add the drizzle orm also
Describe the solution you'd like to see
after the adding solution it should ask to include drizzle
Describe alternate solutions
nothing
Additional information
No response

drizzle is included in npm create t3-app@latest


Add Arusha technical college

The official College URL website is www.atc.ac.tz
The URL for IT course longer than one year is: https://atc.ac.tz/index.php/courses/diploma-in-information-technology

@fule201 First stage of verification passed, official domain is atc.ac.tz as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@fule201 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Add flag to cmds to use AWS profile

Add a new flag to command line utilities to allow to choose a AWS profile from the aws credentials file

Codecov Report

Merging #1788 into v0.13.x will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           v0.13.x    #1788   +/-   ##
========================================
  Coverage    25.27%   25.27%
========================================
  Files           98       98
  Lines         5088     5088
========================================
  Hits          1286     1286
  Misses        3660     3660
  Partials       142      142

Continue to review full report at Codecov.

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


Missing button for tutorial CTA

What is the issue?
In the overview page, the tutorial box is missing the CTA button. It is the only box that doesn't have a button, which doesn't match the flow of the rest of the CTAs.
Reference:

It should look like the tutorial box here:

resolved with https://github.com/dagger/dagger/pull/2933


Fix some comparisons to appease ruff

It seems like I just broke the CI by merging some bot PRs. Now things are failing because of a (presumably new) ruff check. Not sure why the CI didn't catch it, but in any case it's easy to fix.

Yeah, that would make sense


modularize the wing model

Allows easy use of wing model for adding components and use as vtail or htail

test models please

@bqpd tests have been in like this for a while (Waiting for status to be reported).  Do you know what is going on?

test models please

I think the merge conflicts have to be fixed before you can test

test models please

@bqpd dah, still in this waiting state...

test models please

@galbramc do you know why the status is "waiting for status to be reported"? It's been that way for some time, despite attempts to trigger another test using "test models please"

hmm, no let me check jenkins (you can also always check the progress here https://acdl.mit.edu/csi/)

test models please

I was looking there, but couldn't figure out what had happened.

did this repo get renamed?

yeah I think it was library a day or two ago and now its gplibrary

Ok. I don't actually see a test on jenkins for this repo. Did @bqpd try to set one up?

Yeah we had it testing thursday I think

Is this repo supposed to be tested just like SPAircraft and turbofan for example?

No it's supposed to set up more like gpkit, testing the "research repo" like SPaircraft and turbofan and gassolar and jho

Oh is it the gpkit_commons_PR_Models and gpkit_commons_Push_Models tests?

I believe its gpkit_commons_PR_Models

test models please

that's seemed to work!

yeah when you rename a repo you have to update the names in jenkins

How is this repo related to models? Do you really want to say "test models" to run tests here? would you prefer "test this"?

ahh, ok thanks!

I setup "test models" with gpkit so you could run all the tests in all the model repos when you are making changes to gpkit. Does that somehow apply to this repo as well?

I don't know.  @bqpd is the one who knows that

test models please

test models please

your tests are chaotic. You get different results every time you run them without making changes. That does not make them particularly useful tests.

It's a problem with the turbofan repo, which my changes don't depend on... not sure how to best fix that.

test models please

derp, sorry, forgot the renames break jenkins

test models please


Add support for Field Serialization

This PR adds 2 helper functions to the field class - serialize(Kokkos::View<>) and deserialize(), allowing the discontinuous mesh fields to be serialized into a Kokkos view.

@CKegel I assume this is ready to be merged?

@CKegel I assume this is ready to be merged?

It is ready to merge.


Data Migration Tool failing on Settings #37

Hello,
I'm dealing with the connection issue and on Nexcess hosting.
I've tried the IP of the host, and name as well as localhost but all with the same error...
Couldn't connected to Magento 1 database: Access denied for user 'username'@'localhost' (using password: )Also, attached the screenshot.
I am migrating data from Magento 1.7.2 to Magento 2.1.2

Please help!

Hi Cis-shalini,
The script on Github is our old version which we no longer maintain.
We have replaced it with a newer version - UB Data Migration Pro. A few notes about the difference of our Pro version for your reference:

Ongoing updates and bug fixes and fully compatible with latest Magento 2.1.4
Code base is completely revamped for better performance, especially for large scale database
Support new features like Delta migration, Command-Line interface etc.,

We did use the Pro version ourself to help Winetasting.com – Migrate 382,047 customer entities to Magento 2.
And this case study as well - Freelaunch.nl used UB Data Migration Pro to migrate 230,000 products to Magento 2.
If you have any additional question, please reach us via info (at) ubertheme.com.
PS. If you wish to learn more about our Pro version, please take a look at our Tutorial video for latest version — UB Data Migration Pro v3: https://youtu.be/R4UBLe2kf-I
Regards,
Ubertheme team

Ok but if i work on the local server then connection has established and i am able to migrates every thing like stores, categories and product.
The problem is only when i am using this tool on website which is hosted on nexcess not on the local server...so can it be related with the script(doesn't support the magento 2.1 versions) or it's just the database access issue. Please let me know...

Hi Cis-shalini,
The script here at Github was outdated, and not maintained for long, especially for new Magento releases. So unexpected issues are beyond our control.
We are sorry for not being of much help in this case.
Thanks for your understanding.
Sincerely,
Ubertheme team


Comparison plots are reversed (set #1 on the right, set #2 on the left... )

Oops... I probably should have noticed this sooner, but I didn't.  I'll have to figure out where I'm switching these around...

Fixed, I think... but this is the next issue: comparisons/49/chromatography/rt_ms1max_rt_ms2_for_ids_sec

It only plots half the image in the time rendered plot... Apparently the scale of the time plot is chosen based upon the new dataset only.

Fixed with commit: b820da88fac49d7d1f54370854408b7c629d3dd3


Update Japanese info text of JSON and HTTP Request node

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes

I updated Japanese info of JSON and HTTP Request node.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage remained the same at 75.14% when pulling 2acc31a4e7a93ec0c8f50ee0a8c07cd0462e0c0e on node-red-hitachi:update-join-httpreq-info-jp into 8226f1fa7529fa3e70ef4d4337344013216c1dde on node-red:master.


google_sheet_localize_action.rb Option

It should be nice to be able to do not generate CD DATA and export only strings
google_sheet_localize_action.rb.zip
after each update l must remove in google_sheet_localize_action.rb
CD DATA references

@mariohahn ping ?

oh I haven't seen that issue, will look into it the next days

hello Ping ?

Could you please check it ?

@mariohahn Could you have a look ?

Hi, can you send me a PR? (:

Why would you need to remove CDATA?

Why would you need to remove CDATA?

@AlexGrafl
to do not have it in string.xml in android studio

@AlexGrafl   because UNREADABLE
before :

after

I don't know, seems reasonably readable to me.
Using this plugin you should manage your strings in the Google Sheet anyways and not have to look into the strings.xml most of the times.

@AlexGrafl right for the google sheet.
However, when debugging and testing, it's not clear, as I have 6 different languages for each application.

Well I'm still not convinced, it's just the <CDATA[[ tag.
But if you insist, feel free to implement an option for the Android variant to be able to exclude the CDATA tags and open a PR.

@AlexGrafl as l said in my PR l did it but not to much developper to set it as feature flag or option
https://github.com/tailoredmedia/fastlane-plugin-localize/pull/8


Reorg dir structure to add src dir.

This moves the core module files under a src dir.  BTW can we move install.ps1 to the chocolatey dir?  Shipping this file in the module may lead to confusion.  Also both it and CheckVersion.ps1 check if git is installed.

Will this cause merge headaches for the outstanding PRs?

Will this cause merge headaches for the outstanding PRs?

Kind of. I wasn't comfortable with the overlap between this and #358, so I've overwritten your changes here based on the non-Chocolatey changes that ended up over there. Net result:

profile.example.ps1 has been moved back up to the project root, to avoid breaking those who have used install.ps1 or Chocolatey until now.
I also added a warning to that file indicating it will be removed in a future version (presumably 1.0).
install.ps1 no longer uses profile.example.ps1, instead calling Import-Module directly.

This is kind of a breaking change, as fresh installs will no longer call Start-SshAgent -Quiet. Thoughts? (cc #338)

Finally, the module files are moved under src/ along with some missing fixes for references to those files.

Note: there are currently a few commits in this branch marked as fixup!. If you see that, it should be a clear indication that the branch is meant to be rebase -i'd before it's merged (autosquash FTW).

So should I run git rebase -i master on this branch and then git push --force-with-lease before merging it?

So should I run git rebase -i master on this branch and then git push --force-with-lease before merging it?

Yep, looks good as merged.


⚠ ClamAV has degraded performance

In 494d959, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 221 ms

Resolved: ClamAV performance has improved in f0edd28.


[feature] Allow running task & it's deps in same shell

Is your feature request related to a problem? Please describe.
When i'm writing two tasks that are exploiting local variables , the variables are lost when the second one is executed (i assume a new shell is spawned)
Describe the solution you'd like
Add an option on a task to keep the same shell for the downstream (deps)
Describe alternatives you've considered
Adding commands in the same task (merged two tasks into one) but it's not readable as using the deps
Additional context
tasks definition
  first_shell:
    platform: 'system'
    command: export AB=5

  sec_shell:
    platform: 'system'
    command: echo "$AB"
    options:
      uniqueShell: true # Must be great to add
    deps:
      - first_shell

The sec_shell task outputs nothing because of the shell separation

At first thought, I'm not sure how possible this would be. All tasks are ran as child processes, and the parent process can't access their children's env vars.
Can you use a .env file here?

If the variables are dynamically set by other tools it may be useful to keep one shell instance for each tasks and it's deps

@spljs Are you aware of other tools that achieve this? I'm not aware of any.

Well for example if can have poetry & pyenv for python, which are modifying the vars in order to activate the venv of the current project. In each task i have to source .venv/bin/activate so if one instance of a shell could be kept accros each project, we could just source it one time

Do you think it can be achieved easily ? For custom dynamic environment it will be more "DRY"

My gut reaction is no. I'm honestly not sure how I would implement this without a massive rework. This is something that probably won't land for a long time.

Alright we may close this issue if it doesn't fit the moon implementation. Thanks again for your reactivity and awareness !


unzipper failed. Emitted 'error' event on DuplexWrapper instance at #289

Issue Type

[x] Bug Report
[ ] Feature Request
[ ] Other

Current/Missing Behaviour
Expected/Proposed Behaviour
Additional Info

Package version:
Operating System:
Node version:
NW.js version:
Repro link:
...

nwbuild({
  version: '0.82.0',
  src: './temp',
  platform: 'win',
  arch: 'x64',
  cacheDir: './cache',
  outDir: './out',
  flavor: 'normal',
  logLevel: 'debug',
  downloadUrl: 'https://npmmirror.com/mirrors/nwjs',
}).then(() => fs.removeSync('./temp'));

yarn run v1.22.21
$ node build.mjs --platform=win64
node:events:492
      throw er; // Unhandled 'error' event
      ^

Error: FILE_ENDED
    at Parse.pull (D:\own\wang-comic-downloader\nwjs\node_modules\unzipper\lib\PullStream.js:83:28)
    at Parse.emit (node:events:514:28)
    at Parse.<anonymous> (D:\own\wang-comic-downloader\nwjs\node_modules\unzipper\lib\PullStream.js:20:10)
    at Parse.emit (node:events:526:35)
    at finish (node:internal/streams/writable:937:10)
    at node:internal/streams/writable:918:13
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)
Emitted 'error' event on DuplexWrapper instance at:
    at Parse.<anonymous> (D:\own\wang-comic-downloader\nwjs\node_modules\duplexer2\index.js:45:12)
    at Parse.emit (node:events:514:28)
    at Parse.pull (D:\own\wang-comic-downloader\nwjs\node_modules\unzipper\lib\PullStream.js:83:14)
    at Parse.emit (node:events:514:28)
    [... lines matching original stack trace ...]
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

What version of nw-builder?

What version of nw-builder are you on?

4.5.0/4.4.2-beta.4/4.4.2
I found that the issue seems to occur only when decompressing macOS zip file on a Windows platform.

After decompression on a Windows platform, the .app file cannot be opened normally on the macOS platform, and I found that many files are missing.
I have tried using decompress, adm-zip, unzipper, and extract-zip, but all of these library have the same issue.

Is this be an issue with the operating system architecture, where it is not allowed to decompress macOS app package on Windows system?

Can you reproduce this on v4.5.3?

Turns out unzipping files fails uniformally across different tools - yauzl, yauzl-promise, unzipper, native unzip.
On error, we could ask user to either retry decompression or redownload?

@ayushmanchhabra ,
Decompression does not necessarily fail in each version, however it is certain that this one corrupts the files with unzipper. If you are in LTS version of nodeJS or higher (16+ and 18+) this is completely blocked. With each unzip it is the folder which is corrupted the quickest solution and to replace unzipper with unzip-stream which allows you to keep exactly the same structure of the code without having a single error in sight.
the correction can be done like this:
import unzipper from "unzipper";

to
import unzipper from "unzip-stream";

this resolve the problem with wine #1002 too

I have replaced with unzipper with yauzl-promise. I think this package is a good long term solution.


Send banner notification when a user sends message?

Hi, I'm wondering if agora rtm supports shooting a banner notification to user B when user A sends a message to user B. Or is there any documentation related to this functionality?
Thanks in advance!

you should use other notification library to implement it.


Update terraform-create-complete-vm.md

Updated to use Managed Disks (OS Disk), SSH and newer version of Ubuntu. Also made random_id resource more robust when run multiple times. Gave tags consistent values.
References:
https://docs.microsoft.com/en-us/azure/virtual-machines/linux/terraform-create-complete-vm

@paulbouwer,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@paulbouwer : Thanks for your contribution to the Azure documentation! The author, @echuvyrov, has been notified to review your proposed change.

LGTM, approved!
#sign-off


LBRY crashes to desktop when trying to upload on Windows 7 starting at v0.5.36

Basically, when trying to upload a file at the upload screen, either using the Browse button to select a file via explorer or using drag-n-drop on the file results in the app crashing to desktop. This doesn't happen on 0.5.35, but does on 0.5.36 and 0.5.37.
To repro:

go to your uploads and click the "Upload" button
click "Browse" in the file box and select a file or drag and drop a file to upload
app crashes a few seconds later

Of course, expected behavior is the file is accepted and the app functions normally.

LBRY Daemon version: 0.110.0
LBRY App version: 0.5.36 / 0.5.37
LBRY Installation ID: 6MWuUBTneBsyYMS4VJtwNyU3KWX6mT5oEoUJGMoxg4gprTjGN3DkeWGMiHWVFiCUhP
Operating system: Windows 7 SP1

I do see that in 0.5.36, there's an entry for "fixed various upload bugs including transcoding" which might be significant.

I have the same issue in Linux (Fedora 37) . This bug happens when the file is large. Even with smaller file, such as a few mb short video, you can clearly feel that the lbry-desktop app is freezing for a few seconds.

Not using transcoding
720p MP4 (h.264) / files are large between 500MB - 1GB and are 4+ hours long. Weekly uploader.
I do see the preview/confirm upload screen

Looks like disk usage spikes to 100% on file select. Looking into it.

Not checking the transcoding.
1 to 1.5 GB, 1 to 1.5 hours long, formats are MP4 H.264, WEBM VP9 (both crashed).
I have "preview enabled", but different from Sessh22, I can't even see the preview window. Right after selecting the file, the app would freeze and then crash in seconds. I can't even click the submit button.

Perhaps I misunderstood. I have the preview enabled, but I get nowhere near being able to see it because as already stated, the app crashes a few seconds after selecting the file to upload. I thought he was asking if I had it enabled at all. I figured it was already assumed I wasn't actually seeing it because the app crashed almost immediately after selecting the file. Sorry for the confusion.

I've reproduced the problem. It is described in the limitations section of this link:
https://www.npmjs.com/package/get-file-object-from-local-path
I'll have a fix out today or tomorrow. The disk usage spikes were not the problem - that was my old windows vm.

Great I'll be sure to test that out when it's available.

The alpha release with the fix is in releases now!

I grabbed it and it now lets me select a file and everything appears to be normal. I can get to the preview screen now and did everything except push it through. Looks good!

Tested on the last git pull. I can upload in Linux now.

yay!