Readds auto-rotate option

Thumbnails are being generated with the wrong orientation even when using the auto_rotate option.
Digging through the source, I found out that auto_rotation option is not doing anything.
I also found out that the paperclip-ffmpeg gem did support this before: https://github.com/owahab/paperclip-ffmpeg/commit/3e3670432378bd0648b4ffd51d3038f2a3f620f2
So I basically applied the same to the paperclip-av-transcoder gem.
I'm not sure why this was removed but it seems to be working fine...

Hey @owahab, any news on this. We've been using this in production for a couple of months now and it's working perfectly.

Thank you.


Error when loading module in Expo app

Describe the bug
In our recently set up Expo app, we get a reference error as soon as the module is imported.
The error:
ERROR  TypeError: undefined is not an object (evaluating '_en.default')     // KeyboardContext.tsx:138
ERROR  Invariant Violation: "main" has not been registered. This can happen if:
* Metro (the local dev server) is run from the wrong folder. Check if Metro is running, stop it and restart it in the current project.
* A module failed to load due to an error and `AppRegistry.registerComponent` wasn't called.

To Reproduce
Import the module in a Expo app: import EmojiPicker from "rn-emoji-keyboard"
Screenshots

In this case, I'm running through Expo Go in an iOS simulator. Tested also in an Android simulator and the result is the same.

Apparently, imports referencing module itself like this https://github.com/TheWidlarzGroup/rn-emoji-keyboard/blob/master/src/contexts/KeyboardContext.tsx#L3 aren't working
Changing linked line from
import { en } from 'rn-emoji-keyboard'

to relative import fixes the problem:
import en from '../translation/en'

Thanks for reporting and providing a solution. It will be fixed in next release.


[10.x] Add insertWithCasts() to Eloquent Builder

The Problem
The method is designed to cast properties, set default values if not supplied, and set the models timestamps where applicable.
The presenting problem is spelled out more fully here in https://github.com/laravel/framework/pull/46791, though it only mentions the pain point of created_at/updated_at timestamps. I have personally either had to write code like this within multiple projects or suffered the consequences of forgetting that Model@insert() doesn't do any handy casting or setting of defaults.
How it would be done today
For context let's consider this model (and associated enum).
// App/Enums/Genre
enum Genre: string
{
    case Rock = 'rock';

    case HipHop = 'hip-hop';
}

// App/Models/Album
use Illuminate\Database\Eloquent\Model;

class Album extends Model
{
    public $timestamps = true;

    protected $fillable = [
        'artist',
        'title',
        'genre',
        'release_date',
        'stats',
    ];

    protected $casts = [
        'genre' => Genre::class,
        'release_date' => 'datetime',
        'stats' => 'json',
    ];

    protected $attributes = [
        'artist' => 'unknown',
    ];
}

Say we have an action that can accept data from a form or elsewhere.
use App\Enums\Genre;
use App\Models\Album;
use Carbon\Carbon;

$inputData = [
    ['artist' => 'Built to Spill', 'title' => 'Keep It Like a Secret', 'genre' => Genre::Rock, 'release_date' => '1999-02-02', 'stats' => []],
    ['artist' => 'Tom Petty', 'title' => 'Highway Companion', 'genre' => 'rock', 'release_date' => new Carbon('2006-07-25'), 'stats' => ['producer' => 'Jeff Lynne']],
    ['title' => 'Ill Communication', 'genre' => Genre::HipHop, 'release_date' => null, 'stats' => new stdClass]
];
$values = collect($inputData)->map(function(array $values) {
    // make sure we have a valid enum case
    if (! $values['genre'] instanceof Genre) {
        $values['genre'] = Genre::tryFrom($values['genre']);
    }
    $values['genre'] = $values['genre']?->value;

    // make sure the date is formatted for DB layer
    if ($values['release_date'] instanceof \DateTime) {
        $values['release_date'] = $values['release_date']->format('Y-m-d');
    }

    // make sure that the stats column is properly encoded to a string
    if (!is_string($values['stats'])) {
        $values['stats'] = json_encode($values['stats']);
    }

    // set the model's default value defined in Artist::$attributes
    if (!isset($values['artist'])) {
        $values['artist'] = \App\Models\Album::newModelInstance()->artist; // get the default value from the model
    }

    return array_merge($values, [
        'created_at' => now(),
        'updated_at' => now()
    ]);
});

\App\Models\Album::insert($values->toArray());

Pretty messy for something that feels standard. We can just create them one by one, but then we have repeated writes to the database. For a large database with many connections or if this whole Action is happening within a request lifecycle, that is not an ideal solution.
Solution
We add an Eloquent\Builder@insertWithCasts() method that makes the above much less painful.
The above example relying on insertWithCasts()
use App\Models\Album;
use App\Enums\Genre;

Album::insertWithCasts([
    ['artist' => 'Built to Spill', 'title' => 'Keep It Like a Secret', 'genre' => Genre::Rock, 'release_date' => '1999-02-02', 'stats' => []],
    ['artist' => 'Tom Petty', 'title' => 'Highway Companion', 'genre' => 'rock', 'release_date' => new Carbon('2006-07-25'), 'stats' => ['producer' => 'Jeff Lynne']],
    ['title' => 'Ill Communication', 'genre' => Genre::HipHop, 'release_date' => null, 'stats' => new stdClass]
]);

$albums = Album::get();
$albums[0]->release_date->format('m/d/Y'); // '02/02/1999'
$albums[1]->stats['producer']; // 'Jeff Lynne'
$albums[2]->artist; // 'unknown'

Final Thoughts
I think there's probably a better name for this method. I would prefer to call it createMany(), but as @bert-w pointed out, then there's the expectation it will emit model events.

I tried to shorten the "How to do it today" example to this:
$inputData = [
    ['artist' => 'Built to Spill', 'title' => 'Keep It Like a Secret', 'genre' => Genre::Rock, 'release_date' => '1999-02-02', 'stats' => []],
    ['artist' => 'Tom Petty', 'title' => 'Highway Companion', 'genre' => 'rock', 'release_date' => new Carbon('2006-07-25'), 'stats' => ['producer' => 'Jeff Lynne']],
    ['title' => 'Ill Communication', 'genre' => Genre::HipHop, 'release_date' => null, 'stats' => new stdClass]
];

$values = Album::unguarded(fn() => collect($inputData)->mapInto(Album::class));
Album::insert($values->toArray());

but it raises a QueryException:
   Illuminate\Database\QueryException  SQLSTATE[22007]: Invalid datetime format: 1292 Incorrect date value: '1999-02-02T00:00:00.000000Z' for column 'release_date' at row 1 (Connection: mysql, SQL: insert into `albums` (`artist`, `genre`, `release_date`, `stats`, `title`) values (Built to Spill, rock, 1999-02-02T00:00:00.000000Z, Array, Keep It Like a Secret), (Tom Petty, rock, 2006-07-25T00:00:00.000000Z, Array, Highway Companion), (unknown, hip-hop, ?, Array, Ill Communication)).

You can add a method like this to your own project. Thanks!


Customizeable redirects for HTML controllers

Adding a proper way to customize redirects for the HTML controllers.
This still needs tests, but I think this would be best tested on the controller level. Not sure if you agree
This also changes the redirect path after a confirmation signup to the "confirmation_instructions" path

👍 This is a much better way to handle the redirection than my original cut. Fantastic work.
I'd like to get specs included with this before we merge it in, so I'll see if I can add them today. If you beat me to it that's awesome as well. Either way let's get those added and then we can add this in and I'll increment the version in the hex package and update the change log.

What's up with this PR?

I've stopped using Sentinel for my project and ended up not finishing this. I think it just needs to fix the tests, so anyone can finish this work if it's still relevant.

@Ch4s3 It needs some serious TLC on the HTML side, which I'm working on, along with some general logic fixes for V2, but I need to add a note to the README that this is slow in the coming. Once again thanks to @Draiken for the stuff he's contributed to Sentinel.


Releases not being updated with new version info

I just upgraded from NSwag 12.0.11 to 13.9.4 and need to ensure some of my custom C# code generation templates are up to date. NSwag v12.0.11 (tag NSwag-Build-1045) uses NJsonSchema v9.13.13 (tag NSwag-Build-1872).  NSwag v13.9.4 (tag NSwag-Build-1118) uses NJsonSchema v10.3.1 per the published NuGet package. However, 10.3.1 doesn't show up in the
NJsonSchema Releases. The latest version listed there is v10.0.4.
How do I figure out which commit corresponds to NJsonSchema v10.3.1?

You can see it easily in the repo commit history

Doh. Yep, it is there. I was used to looking under the Releases link to find the version. Thanks for the quick response.


Tokenomy (TEN) is token ether

Please add tokenomy (TEN)

https://github.com/MixinNetwork/asset-profile/tree/master/assets/fd11b6e3-0b87-41f1-a41f-f0e9b49e5bf0
Please add your icon.png, and use index.json.
Another problem is, replace asset_id with your coin's not Ether.

https://github.com/MixinNetwork/asset-profile/tree/master/assets/fd11b6e3-0b87-41f1-a41f-f0e9b49e5bf0
Please replace asset_id with your coin's not Ether.

link given is asset id for bitcoin cash. TEN is ether token. Sorry i feel confuse to contribute request.

https://github.com/MixinNetwork/asset-profile/tree/master/assets/fd11b6e3-0b87-41f1-a41f-f0e9b49e5bf0
Please replace asset_id with your coin's not Ether.

link given is asset id for bitcoin cash. TEN is ether token. Sorry i feel confuse to contribute request.

https://github.com/MixinNetwork/asset-profile/tree/master/assets/fd11b6e3-0b87-41f1-a41f-f0e9b49e5bf0
Please replace asset_id with your coin's not Ether.

link given is asset id for bitcoin cash. TEN is ether token. Sorry i feel confuse to contribute request.

As i follow quide. that asset id for ether is https://mixin.one/snapshots/43d61dcd-e413-450d-80b8-101d5e903357
Than, token tokenomy ( TEN ) is ether token. That is why , i pull request base on asset id ether. Please advise. What is actually should we do to carry out request. Therefore i can contribute more to bring more community of each token using mixin messanger.

https://mixin.one/snapshots/c94ac88f-4671-3976-b60a-09064f1811e8
Mixin is also erc20 token, but its asset_id is c94ac88f-4671-3976-b60a-09064f1811e8
So you should use your asset_id, not ether's.

https://mixin.one/snapshots/c94ac88f-4671-3976-b60a-09064f1811e8
Mixin is also erc20 token, but its asset_id is c94ac88f-4671-3976-b60a-09064f1811e8
So you should use your asset_id, not ether's.

i got it now.  revised already...
https://github.com/MixinNetwork/asset-profile/pull/7/commits/c786b86df919572072bac71239df700be079863f

lol, why did you delete it

lol, why did you delete it

I will create anothers branch/PR. Hoppely next PR are with the right step. Than how about this PR. Should i create new one? and again re submit. Or you get already to merge. Therefore i can see the update when using mixin messanger.
Please noted. I am a common user, not developer / not expert of coding. This is my first time using github.


Add minimal support for standalone Var to visualisers

Summary
This adds best-effort only support to the visualisers for handling stand-alone Var nodes.  Most of the changes are actually in qasm3, since the visualisers use internal details of that to handle the nodes.
This commit decouples the visualisers slightly more from the inner workings of the OQ3 exporter by having them manage their own variable-naming contexts and using the encapsulated _ExprBuilder, rather than poking into random internals of the full circuit exporter. This is necessary to allow the OQ3 exporter to expand to support these variables itself, and also for the visualisers, since variables may now be introduced in inner scopes.
This commit does not attempt to solve many of the known problems around zero-operand "gates", of which Store is one, just leaving it un-drawn. Printing to OpenQASM 3 is possibly a better visualisation strategy for large dynamic circuits for the time being.

Details and comments
Close #10937 (enough, at least, not to crash - it's not perfect by any stretch).
@enavarro51: would you be able to / like to review the changes to the visualisers that I made?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

31 of 61 (50.82%)  changed or added relevant lines in 5 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.008%) to 89.438%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/qasm3/ast.py
5
6
83.33%

qiskit/visualization/circuit/text.py
19
22
86.36%

qiskit/qasm3/printer.py
2
6
33.33%

qiskit/qasm3/exporter.py
2
8
25.0%

qiskit/visualization/circuit/matplotlib.py
3
19
15.79%

Files with Coverage Reduction
New Missed Lines
%

qiskit/visualization/circuit/matplotlib.py
1
49.33%

crates/qasm2/src/expr.rs
1
94.03%

crates/qasm2/src/lex.rs
2
93.38%

Totals

Change from base Build<PHONE_NUMBER>:
-0.008%

Covered Lines:
61003

Relevant Lines:
68207

💛  - Coveralls

Yeah, the coupling in the expr renderers isn't ideal, but hopefully this PR decouples them rather more.  There's still a single private-class access, but that's also a component that's deliberately encapsulated away from the rest of the OQ3 exporter.
The coupling was a bit of a compromise - we don't have the resources to maintain the drawers as much as we'd like, and expression unparsing/formatting/whatever is not entirely trivial when it comes to determining when brackets are necessary in nested expressions.  The OQ3 exporter already has to do that, so in the choice between duplicating additional code into the visualisers that'd need to be maintained and having a not-entirely-ideal coupling between the two, we tried out the coupling.


Issue with k8s.io/docs/tasks/access-application-cluster/service-access-application-cluster/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
When trying to communicate with the node curl returns "failed to connect to..." error.
Proposed Solution:
Investigate the issue and update the page accordingly.
Page to Update:
https://kubernetes.io/...

Kubernetes Version:
1.9.3
Additional Information:
My service description:
Name:                     example-service
Namespace:                default
Labels:                   run=load-balancer-example
Annotations:              <none>
Selector:                 run=load-balancer-example
Type:                     NodePort
IP:                       <IP_ADDRESS>
Port:                     <unset>  8080/TCP
TargetPort:               8080/TCP
NodePort:                 <unset>  32497/TCP
Endpoints:                <IP_ADDRESS>:8080,<IP_ADDRESS>:8080
Session Affinity:         None
External Traffic Policy:  Cluster
Events:                   <none>

Curl commands and output:
curl http://<IP_ADDRESS>:32497
curl: (7) Failed to connect to <IP_ADDRESS> port 32497: No route to host
curl http://<IP_ADDRESS>:8080
curl: (7) Failed to connect to <IP_ADDRESS> port 8080: No route to host

This issue sounds more like a request for support and less
like an issue specifically for docs. I encourage you to bring
your question to the #kubernetes-users channel in
Kubernetes slack. You can also search
resources like
Stack Overflow
for answers  to similar questions.
You can also open issues for Kubernetes functionality in
https://github.com/kubernetes/kubernetes.
If this is a documentation issue, please re-open this issue.


Fetch Ball improvement on Falcons Variant

The falcons variant  for fetch ball performs well, but there is a edge case, where the algorithm is not efficient.
In the falcons variant of fetch ball  the rotation and translation are performed at the same time, where the assumption is that rotation is finished before the translation is done.
In a lot of cases this is correct. But when the ball is close to the robot, it will push the ball away.
A simple solution can be if the ball is close to the robot (radius) and not within an angle to the beak, the robot should first rotate till the ball is in front of the robot, then it should also add translation.
Discuss how to fix it (which option is preferred):

fix in falcons variant
create copy of falcons variant and apply fix there.

I totally forgot to demonstrate this part actually, when we were in Bordeaux.
The reason we (Falcons) never implemented it, is because in matchplay our robots follow a general design rule to always look at ball, and we most often know where it is.
However, in single-robot testing I sometimes get annoyed by this problem. So I prefer to fix it in Falcons variant at some point, maybe when I get back to motion-related topics, or when someone else beats me to it ;).


fix: fix sponsor update location

Fix sponsor location selection
This PR provides a fix to sponsor location selection functionality.
How this should be tested?

make install (if you don't have the dependencies already installed)
make run
Create a new sponsor or login with an existent one
Go to the Profile section and then to the Update Profile section
Update Sponsor location, go to Profile section and see location updated

Linked issues
closes #294

@tetogomez codefactor suggestions could not be fixed


Create github actions to test playwright tests

We have page level testing done with playwright. Our next step should be to run these tests for every PR so that we can identify any bugs or breaking changes on the deployed demo before merging them into our codebase.

Create github action to run the playwright test for ever PR raised against the main branch.
Run the UI server against the cluster api deployed on the demo inside the github action to test for any breaking changes the PR may cause in the demo environment. This way we can test the functioning of the demo before releasing any updates to the demo env.

Done in #607


toggle inlay with mouse

hi, is it possible to make custom bindkey for toggle inlay ?
i use a macro mouse and i need this feature bind to my mouse.
Or is it possible add a toggle option with the middle mouse button ?
When i need inlay, is because am reading code, and when reading code i only use my mouse.
I would like able to just hold middle button to show inlay.

Unfortunately, I don't believe it's possible. It seems to be a hard-coded limitation of the editor currently: https://github.com/microsoft/vscode/issues/128162#issuecomment-1088497398

Closing - we don't control this behaviour.


Add to Wish List from Shopping Cart

As a Customer
I need to put an item on my wishlist if I decide not to buy it anymore
So that I can easily find it back later and be reminded that I was interested in it
Assumptions:
The buyer is no longer interested in buying, so the product is no longer in the shopping cart (and the shopping amount is adjusted). The product instead is added to the wish list. The same confirmation message appears as when added from the Product Page (other user story).
Acceptance Criteria
...
Given ... I click on add to wishlist
When ... in my shopping cart
Then ... the product is removed from the shopping cart and added to my wish list

I'll find out how to link this user story to the confirmation message user story. As they are connected ideally we build them in the same sprint


[Bug]: example gnet for echo tcp uses too much memory

Actions I've taken before I'm here

[X] I've thoroughly read the documentations on this issue but still have no clue.
[X] I've searched the Github Issues but didn't find any duplicate issues that have been resolved.
[X] I've searched the internet for this issue but didn't find anything helpful.

What happened?
ran iperf on this twice and the memory used is too great
iperf -c <IP_ADDRESS> -p 9000
why does it use so much memory and how to resolve this without using gomemlimit?
memory is not reduced after iperf finished immediately.
https://github.com/gnet-io/gnet-examples/blob/v2/echo_tcp/echo.go

Major version of gnet
v2
Specific version of gnet
latest
Operating system
Linux
OS version
ubuntu 22.04
Go version
1.22
Relevant log output
information as attached.

Code snippets (optional)
No response
How to Reproduce
get the tcp echo file from example, build it, run iperf command as above, check memory with top and see it over uses memory available.
Does this issue reproduce with the latest release?
It can reproduce with the latest release

Use pprof to find out where the majority of the memory is allocated.

@panjf2000 is this really ok?
nothing extra added and using iperf... memory is bytesliced gotten from internal mechanism... if i do gomemlimit, the whole thing becomes too slow. any fix for this?
package main

import (
        "flag"
        "fmt"
        "log"
        "net/http"
        _ "net/http/pprof"

        "github.com/panjf2000/gnet/v2"
)

type echoServer struct {
        gnet.BuiltinEventEngine

        eng       gnet.Engine
        addr      string
        multicore bool
}

func (es *echoServer) OnBoot(eng gnet.Engine) gnet.Action {
        es.eng = eng
        log.Printf("echo server with multi-core=%t is listening on %s\n", es.multicore, es.addr)
        return gnet.None
}

func (es *echoServer) OnTraffic(c gnet.Conn) gnet.Action {
        buf, _ := c.Next(-1)
        c.Write(buf)
        return gnet.None
}

func main() {
        var port int
        var multicore bool
        var pprofPort int

        // Example command: go run main.go --port 9000 --multicore=true --pprofPort=6060
        flag.IntVar(&port, "port", 9000, "Set the port for the TCP server")
        flag.BoolVar(&multicore, "multicore", false, "Enable multicore processing")
        flag.IntVar(&pprofPort, "pprofPort", 6060, "Set the port for the pprof HTTP server")
        flag.Parse()

        // Start pprof server
        go func() {
                log.Printf("Starting pprof server on http://localhost:%d/debug/pprof/\n", pprofPort)
                log.Println(http.ListenAndServe(fmt.Sprintf("localhost:%d", pprofPort), nil))
        }()

        // Setup and start TCP echo server
        echo := &echoServer{addr: fmt.Sprintf("tcp://:%d", port), multicore: multicore}
        log.Fatal(gnet.Run(echo, echo.addr, gnet.WithMulticore(multicore)))
}

@panjf2000 i can confirm this is a bug,
running the example tcp echo using
iperf -c <IP_ADDRESS> -p 8080 -t 3600
u can crash it.

I've never run into this problem when benchmarking, and I think it is expected to allocate plenty of memory when the server is overloaded during benchmark testing, this happens to all network frameworks. Have you tried WithEdgeTriggeredIO?

Besides, you need to prove that gnet has this issue while other network frameworks don't before you call it a bug.

@panjf2000 , is this how WithEdgeTriggeredIO is used?
same, the memory is used til oom,
iperf -c <IP_ADDRESS> -p 9000 -t 3600
package main

import (
        "flag"
        "fmt"
        "log"
        "net/http"
        _ "net/http/pprof"

        "github.com/panjf2000/gnet/v2"
)

type echoServer struct {
        gnet.BuiltinEventEngine

        eng       gnet.Engine
        addr      string
        multicore bool
}

func (es *echoServer) OnBoot(eng gnet.Engine) gnet.Action {
        es.eng = eng
        log.Printf("echo server with multi-core=%t is listening on %s\n", es.multicore, es.addr)
        return gnet.None
}

func (es *echoServer) OnTraffic(c gnet.Conn) gnet.Action {
        buf, _ := c.Next(-1)
        c.Write(buf)
        return gnet.None
}

func main() {
        var port int
        var multicore bool
        var pprofPort int

        // Example command: go run main.go --port 9000 --multicore=true --pprofPort=6060
        flag.IntVar(&port, "port", 9000, "Set the port for the TCP server")
        flag.BoolVar(&multicore, "multicore", false, "Enable multicore processing")
        flag.IntVar(&pprofPort, "pprofPort", 6060, "Set the port for the pprof HTTP server")
        flag.Parse()

        // Start pprof server
        go func() {
                log.Printf("Starting pprof server on http://localhost:%d/debug/pprof/\n", pprofPort)
                log.Println(http.ListenAndServe(fmt.Sprintf("localhost:%d", pprofPort), nil))
        }()

        // Setup and start TCP echo server
        echo := &echoServer{addr: fmt.Sprintf("tcp://:%d", port), multicore: multicore}
        log.Fatal(gnet.Run(echo, echo.addr, gnet.WithMulticore(multicore), gnet.WithEdgeTriggeredIO(true)))
}

@panjf2000 i just tested
https://github.com/tidwall/evio/blob/master/examples/echo-server/main.go
no such issues.

@panjf2000 sorry my bad, it's working as it should. thx.


MAP WONT AUTO CENTER

MAP WONT AUTO CENTER

yeah there isnt support for auto center yet, just auto follow


Agents are not connection to the server . Test Locations are showing blank

Hi ,
I have installed server in google cloud platform in ubuntu 16.04 LTS . It is working fine .  I followed the steps to install the agents mentioned in url (https://github.com/WPO-Foundation/webpagetest-docs/blob/master/user/Private Instances/gce_agents.md)  but after creating the instance group and the VM it is not able to connect to the server . In sever UI the locations are still blank . I waited for 30 mins so that all the browsers get installed in the agents but nothing happened . Need help to fix the issue .
Regards
Kashyap

Hi Kashyap, I seem to be having the exact same issues while trying to set up the server and agent on Ubuntu 16.0.4 and 18.0.4 (on Azure). Were you able to find a fix?

Hi Shanthi, Today I was trying to install the agents in Google Cloud and faced the same issue (Test locations were showing blank). Looks like not setting the wpt_location part of the metadata, resulting in the issue. As per the instructions given, it supposed to set a default value for that, something is not right there... think so. So, I basically set the wpt_location (it should match the location value already given in the wpt_server locations.ini... Following is the metadata I used when setting up the agent
wpt_server=<WPT_SERVER_NAME> wpt_url=http://<WPT_SERVER_NAME>/work/ wpt_key=<LOCATION_KEY> wpt_location=gce-us-central1-linux

Thank you @Kanwrk ,
Will try to change config as mentioned.
Thanks again
Shanti

Hi @Kanwrk,
Connection isn't successful after executing metadata on agent. I just executed the said command as i have already setup agent via debian.sh.
Please take a look at the location.ini attached.

As agent isn't connected i'm failing to see location info in server logs.  Any advise on this please?


Impossible to implement a provider which may or may not render a metatile

What I'd like to do is have a Provider which renders from an mbtiles file below a certain level, and renders an area with mapnik otherwise.
Under the current scheme, all requests to a given layer either do or do not return metatiles; my proposed provider would (optionally, according to config) provide a metatile when rendering from mapnik, but return a pre-rendered tile from an mbtile database otherwise.
Thus, I think that providers should tell Core whether or not they're returning a metatile, rather than Core handling that decision.
If you think this is a reasonable feature request, I'd be happy to implement it myself in the way that you think would be most reasonable, so please let me know. (We can discuss ways to implement this feature if you like it conceptually, I'm just asking here whether or not you think it's reasonable.)
My wholly untested code which sketches the type of provider I'm looking for:
from MBTiles import Provider as MBTilesProvider
from Providers import Mapnik as MapnikProvider

class MBCacheProvider:
  def __init__(self, layer, max_mb_zoom, tileset, mapfile, fonts=None):
    self.layer = layer
    self.max_mb_zoom = max_mb_zoom
    self.mbtileprovider = MBTilesProvider(self, layer, tileset)
    self.mapnikprovider = MapnikProvider(self, layer, mapfile, fonts)

  def renderTile(self, width, height, srs, coord):
    if coord.zoom < max_mb_zoom:
        return self.mbtileprovider.renderTile(width, height, srs, coord)
    else:
        xmin, ymin, xmax, ymax = self.layer.envelope(coord)

        if self.layer.doMetatile():
            xmin, ymin, xmax, ymax = self.layer.metaEnvelope(coord)
            width, height = self.layer.metaSize(coord)

            subtiles = self.layer.metaSubtiles(coord)

        return self.mapnikprovider.renderArea(width, height, srs, xmin, ymin, xmax, ymax, coord.zoom)

Interesting. Have you used the Composite provider? http://tilestache.org/doc/TileStache.Goodies.Providers.Composite.html I've been thinking of extending it to include min/max zoom level flags, so your use case could be handled by setting up layers for the Mapnik and MBTiles providers, and then a third layer that switched between them based on zoom level. Would this be sufficient for what you describe?

(oops, didn't mean to close the issue there)

Ultimately, what I want is just the mbtiles < zoom X < mapnik behavior. I'll have to take a look at implementing it via the composite provider, that could definitely make sense.


Add exclude_input_in_output option to vllm backend

This PR adds exclude_input_in_output flag to vllm backend inputs.
It only impacts non-streaming case.
For streaming case, I refactored code and return only diffs, e.g.:
Prompt = "The most dangerous animal is"
Response:
" the",
            " one",
            " that",
            " is",
            " most",
            " likely",
            " to",
            " be",
            " killed",
            " by",
            " a",
            " car",
            ".",

The above case will be the only possible response in the streaming mode. Open to discussions.
cc @@mkhludnev

Makes sense. Thank you so much @oandreeva-nv! Looking forward for release.

@mkhludnev This most likely will be a part of 24.03 release, but you don't need to wait this long.
For this PR, you can simply replace /opt/tritonserver/backends/vllm/model.py with the updated version in 24.01 or 24.02 (soon to be released), and it should work, since there's no related changes in tritonserver itself, to handle this flag.

A liitle bit of context https://github.com/triton-inference-server/server/discussions/6867
Thanks, @oandreeva-nv so much!!


border-width: 10px; doesn't work, it won't let me move to next lesson.

Challenge Waypoint: Add Borders Around your Elements has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.71 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

You can find the solution of this issue in https://github.com/FreeCodeCamp/FreeCodeCamp/issues/3204.


Define usecases of a potential generic "reveal component"

As a basis for multiple components (e.g. accordion) a generic "reveal component" could make sense. This could be used in multiple usecases embedded in multiple components.
So a list with all the usecases should be created to see the full potential of this planned support component.

[ ] plain on page
[ ] inside modal dialog
[ ] Apply Button
[ ] Edit Button (copy, duplicate, delete,...)
[ ] Export (choose format)
[ ] image switch to compare old/new image or article

Create follow-up Ticketto show and discuss the reveal use cases


[ HTML5 Game ] - FPS on Exported Project Capped up to 60. Can't go more than 60

Operating system or device - Godot version:
HTML5 Platform - Any OS? - Godot 2.1alpha 60fa2b9
Issue description (what happened, and what was expected):

What happened? FPS capped up to 60. And can't go > 60.

What was Expected? It should harness the max power of the device (PC in my case) that shown by more FPS since the project is a simple one.

Steps to reproduce:
1.) Open one of examples or demos that comes with Godot.
2.) Do some setting that deal with FPS things (use_vsync, target_fps, print_fps)
3.) Export the project to HTML5 Platform
4.) Play the project using Firefox (or other)
5.) Now, FPS capped to 60.
Link to minimal example project (optional but very welcome):
Just use examples & demos that comes with Godot.

Your browser is very likely to enforce VSync or a FPS cap of some sort. AFAIK, they all do this.

So, this is a browser "mistake" and not Godot? If so, then let's close this.

Nobody's ever going to need more than 640 frames per second
On 24 June 2016 at 05:37, Scotnez Fluxio<EMAIL_ADDRESS>wrote:

So, this is a browser "mistake" and not Godot? If so, then let's close
this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5381#issuecomment-228289453,
or mute the thread
https://github.com/notifications/unsubscribe/AGVmPe4toB2IllxKUNfUywVz9xd6vZd4ks5qO5degaJpZM4I9dEr
.

Yes, I completely agree about wasting resources on FPS > 60. The one that drive me thinking about FPS of a HTML5 game made with Godot is I want to impress Local HTML5 gamedev and want to see them saying "Wow", when I show'em that shiny FPS number (while learning and making app/game with Godot, I guess I'm also promoting Godot anywhere). Because I have no idea what to impress anyone except a shiny FPS number when it comes to performance. But, I could be wrong though. :laughing: lol
[ Copied from my comment on another issue tracker ]

So now, it's safe to close this, right?


AWS CLI command to list images

Hello EveryOne,
Any body please help me with AWS cli command to list/describe images of free tier eligible Ubuntu Server .
Thanks,
Sreedhar

Hi Sreedhar,
i found  command for filtering ubuntu images, but for free tier  we have to find.
command :
aws ec2 describe-images --owner amazon --filter "Name=description,Values=Ubuntu" "Name=owner-alias,Values=amazon" "Name=architecture,Values=
x86_64" "Name=image-type,Values=machine" "Name=root-device-name,Values=/dev/sda1" "Name=root-device-type,Values=ebs" "Name=virtualization-type,Values=hvm"
Thank you

All community images are free tier images


Why is Command.Empty disabled on first render?

After spending a good 30 minutes trying to debug why I couldn't get the Empty element to show up asap when the command opens, I ended up searching to code to find that it is explicitly disabled on first render.
https://github.com/pacocoursey/cmdk/blob/main/cmdk/src/index.tsx#L822-L832
I didn't find any documentation about this and this doesn't seems to be configurable. I would like to have the empty warning show up asap, how can I achieve this? Why is this disabled on first render?

I was using cmdk with shadcn/ui, I just copied the Empty implementation and removed the ref logic
const CommandEmpty = React.forwardRef<
	HTMLDivElement,
	React.ComponentProps<typeof CommandPrimitive.Empty>
>(({ className, ...props }, forwardedRef) => {
	return (
		<div
			ref={forwardedRef}
			className={cn('py-6 text-center text-sm', className)}
			{...props}
			cmdk-empty=""
			role="presentation"
		/>
	)
})

CommandEmpty.displayName = CommandPrimitive.Empty.displayName

I need this feature.


Remove containerd v1.5.9 from go.sum

Remove containerd v1.5.9 references from go.sum.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


[Communication - Calling] Raw Media Screen Sharing not working in Group Call from Android Chrome

Package Name: @azure/communication-calling
Package Version: 1.23.2
Operating system: Android
[ ] browser

name/version: Chrome/124.0.6367.83

Describe the bug
Start screen sharing with a custom screen share stream is not working in a Group Call from Android Chrome.
To Reproduce
Use the add-1-on-1-video-calling sample, and do the bare minimum to add "Join a group call" and "Start screen sharing with a custom screen share stream" functionality.
Setup:

Setup 1 client on Windows Desktop Chrome (124.0.6367.63)
Setup 1 client on Android Chrome

Steps to reproduce the behavior:

Have each client join a Group Call
On Android Chrome client, start screen sharing a custom screen share stream

Expected behavior
Remote side (Windows Desktop Chrome) will render the custom screen share stream.
Actual behavior
Remote side (Windows Desktop Chrome) fails to render the custom screen share stream.
The error in console:
azure:ACS:error 2024-05-02T17:52:50.610Z VideoStreamRenderer:RemoteVideoStream:421:VideoStreamRendererView6:RemoteStreamRenderer4 op:Start failed, message=Failed to render stream, timeout, code=408, subCode=43203 in 12.306s
Additional context

With the exact same setup, but the call type is a 1:1 call, this works as expected.
With the exact same setup, but using package version 1.19.1, this works as expected. (Bug is reproducible starting in 1.20.1)

@lsbrettjewell thank you for reporting, can you please provide ACS call id ?

@akania ACS call id: a1fb5a6f-7d9a-46e1-abc5-3f8680b7c186

@lsbrettjewell We have fixed the issue in 1.27.1-beta
You can still try using older SDK versions, as we have implemented a workaround for them, but we recommend using version 1.27 beta

Marking Closed. Have confirmed after re-testing that the issue is no longer reproducible, even when using older version of the SDK.


[Usage]: logprobs variable in SamplingParams from ChatCompletionRequest class

I have noticed that the logprobs variable could be set to a boolean (top_logprobs) in this line https://github.com/vllm-project/vllm/blob/main/vllm/entrypoints/openai/protocol.py#L254.
Is not supposed to be an integer which displays the number of log probabilities to return per output token?

We follow the semantics defined by OpenAI API. You might have confused the Completion API with the Chat Completion API.

Many thanks for your feedback.
Indeed, from Completion API side, logprobs variable is an integer, and from Chat Completion API side it is a boolean, it is now much clearer.


notification tests

Coverage remained the same at 74.974% when pulling c16d67b5657181f8730a850d3c201912c50814d7 on RHMAP-19008_tests into e4a9c0f1191cae6e1356dd6daa10130616bc4dff on master.


Use TTGO TCall Blue LED for orange-pi

As we are dealing with new milestone improve communication between TTGO-TCall and orange pi so I think we should use ttgo-tcall's  led to show if the ttgo is properly communicating with the orange pi or not what do you say? @mhamidjamil


feat(dialog-view): add DialogView component UX

Please, provide the support for multiple dialogue texts.


istioctl analyze: check if hub/image:tag combination exists on gcr.io/docker.io hub

Describe the feature request
istioctl analyze checks if hub/image:tag combination exists on gcr.io/docker.io hub.
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Additional context
@sushicw please help find the right assignee, thanks!

I don't think this is a good idea. it means istioctl analyze has a dependency on arbitrary docker registries, which is not always possible. It's common that you could have a registry accessible from the cluster but not from a developer machine running the cli, or CI.
If you do happen to run an invalid image you will immediately see it from kubectl - no need for a fancy istioctl command.
Is there some context I am missing here?

Sure, I agree this should be in k8s. I opened this because I encountered an issue when upgrading Istio but with an incorrect image tag. That action took down the old istio pods but new pods can never enter ready state.

@elfinhe I don't think it should take down the old pods, normally kubernetes waits for the new ones to be marked "Ready" before removing the old ones? Maybe something is wrong with some configuration


chore: add mega-linter

Closes #83

:tada: This PR is included in version 1.0.0-dev.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Link "resources" appendix to wiki

https://github.com/pytti-tools/pytti-core/wiki/Resources

actually, let's delete the wiki and direct people to the pytti-book site instead.


Feature/change ga property endpoint

If a Google Analytics account does not have a default web property set, then the existing call to retrieve web properties will have no defaultProfileId. This updates API call to use a different endpoint that returns all the account's default profile ids, not relying on defaultProfileId

ping @edenspiekermann/backend


Add/insert arguments on call expressions and decorators

It's a little tricky to add arguments on non-decorator factory decorators because the identifier is moved to be inside the call expression.

This is basically a duplicate of one of the tasks in #27. Closing.
Also, adding the ability to set as a decorator factory or not should be implemented before this issue.


Error handling updates for checkjndi.ps1

Improves the error handling for checkjndi by ensuring that AccessDenied errors are reported properly to the warning stream.

This may improve the access denied situation https://github.com/CERTCC/CVE-2021-44228_scanner/issues/5

I'm not sure that it'll make a failing scan start working with this change.  However, it should be more obvious to users experiencing problems what the problems may actually be.


Updated values.yaml with latest v0.6.2 patch tag

latest release v0.6.2 values.yaml is pointing to old tag so updated to latest which is v0.6.2.
we want to this this to be updated in v0.6.2 release

@kitt1987 shall we go ahead with merging this PR in 0.6.x and cutting a new v0.6.3 release?


[Doc] GridIdent

Hello,
I think on this page: https://openui5.hana.ondemand.com/#docs/api/symbols/sap.ui.layout.GridIndent.html
it should read "... Letters L, M or S followed by number of columns from 1 to 12 that the container has to take...."

Hi spruce,
thanks for reporting. I opened internal incident<PHONE_NUMBER> for this. We will come back to you when the documentation is updated.
Best Regards,
Martin

Hi Spruce,
the Indent of the Grid can be from 0 to 11 (it is different to Span that can be from 1 to 12), so the text is correct, but the example "M12" is wrong, so I will change it in Grid's JS Doc to M11 or whatever under 11.
Thank you and best regards,
Irina


Fix home page workflow instance miss status

Purpose of the pull request

Fix home page miss some status
Remove some unused code.

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report
Attention: 64 lines in your changes are missing coverage. Please review.

Comparison is base (7308888) 37.95% compared to head (d057497) 37.86%.

:exclamation: Current head d057497 differs from pull request most recent head 3131fe0. Consider uploading reports for the commit 3131fe0 to get more accurate results

Files
Patch %
Lines

...uler/api/service/impl/DataAnalysisServiceImpl.java
38.00%
28 Missing and 3 partials :warning:

...scheduler/api/service/impl/ProjectServiceImpl.java
25.00%
6 Missing and 3 partials :warning:

...heduler/api/controller/DataAnalysisController.java
45.45%
3 Missing and 3 partials :warning:

...hinscheduler/api/vo/WorkflowDefinitionCountVo.java
25.00%
5 Missing and 1 partial :warning:

...phinscheduler/alert/plugin/AlertPluginManager.java
0.00%
2 Missing :warning:

...uler/api/controller/v2/StatisticsV2Controller.java
33.33%
2 Missing :warning:

...e/dolphinscheduler/api/vo/TaskInstanceCountVo.java
84.61%
1 Missing and 1 partial :warning:

...lphinscheduler/api/vo/WorkflowInstanceCountVo.java
84.61%
1 Missing and 1 partial :warning:

...inscheduler/plugin/task/api/TaskPluginManager.java
0.00%
2 Missing :warning:

...apache/dolphinscheduler/api/dto/DefineUserDto.java
50.00%
0 Missing and 1 partial :warning:

... and 1 more

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15193      +/-   ##
============================================
- Coverage     37.95%   37.86%   -0.09%
+ Complexity     4671     4654      -17
============================================
  Files          1278     1281       +3
  Lines         44874    44820      -54
  Branches       4870     4842      -28
============================================
- Hits          17030    16973      -57
- Misses        25988    25991       +3
  Partials       1856     1856

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LGTM

Thanks


Load MongoDB URL from the .env file

As we noticed on previous commits, the MongoDB URL was leaked. This will fix that forever.

Just add a MONGODB_URL key on the .env file.


OAuth 2.0 Consumer support?

I can't seem to find if the oauth-plugin gem completely supports OAuth 2.0 for consumer apps. While the README states support exists, I'm hesitant as I don't know if that is experimental or full 2.0 support.
Any help would be greatly appreciated.

Yes it does support OAuth 2.0 see the example in oauth_config.
You will need to use the 0.4.0 pre release though. Use 0.4.0.rc1 in your Gemfile


sell online

sell online page fixed
Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Description
[Please include a brief description of the changes or features added]
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

please assign it gssoc label

hey merge it as soon as possible so I can work on my next issues

best selller page is not up to date so  i have made some arrangements
before

after


Explicit import of the process package

When using electron, the process package isn't imported by default. We need to explicitly import it so it can fill out the data (e.g process.platform).

Thank you for the patch :)
I've done it another way, see:  a73e7d1

Now fixed in 3.4.0


error "ModuleNotFoundError: No module named 'graphmb.data'"

Hi,
I get this error at launch ModuleNotFoundError: No module named 'graphmb.data' after having installed the tool from source with pip. Is my installation missing something?

Hi,
I have the same error. Does anyone know the reason?

sorry, I'm a bit swamped with work. Can you give me more details, are you using a virtual environment? which OS are you using? Can you install from pypi (instead of installing from source)?

a quick fix is to create this directory: "src/graphmb/data" and move "data/Bacteria.ms" and "data/kernel.npz" to there. I'll push a better fix soon


[WIP] --source pika

Note: This is a draft to discuss the feature. The implementation is pretty basic and there are no new tests, but the branch runs if you want to check it out locally (tested it myself on a dev deployment of www.pika.dev).
Background
Snowpack is a post-install tool. It runs after npm install, and it essentially exists to convert your npm packages (in your "node_modules/" directory) into JS files that run in the browser without a bundler (written to a "web_modules/" directory). Because of this flow, Snowpack has an implicit dependency on npm/yarn and a direct dependency on Rollup, Babel, etc.
Pika CDN is a hosted CDN of npm packages. Whenever an npm package is requested, it is built into a JS file that run in the browser without a bundler (served in the response). If it helps, you can think of it has hosted Snowpack or Snowpack-as-a-Service.
Investigation
The question that I've been exploring over the last couple of weeks: If Pika CDN is already a hosted, pre-built source of Snowpack-built packages, can we just pull code from the CDN instead of installing/building locally every time we run Snowpack?
$ snowpack --source pika

A new CDN-powered flow might look something like this:

Read a list of packages from your webDependencies (match our current scanning logic)
Request each package from the CDN (follow any sub-package imports to request them as well)
Write each response to disk at web_modules/ (match our current folder structure)

This has some immediate benefits:

Instead of running npm/yarn and THEN running Snowpack, you just run Snowpack. You can keep using npm/yarn to manage your non-web (Node.js) dependencies.
Snowpack becomes way lighter: Rollup, Babel, Terser are all removed since the CDN does it for us.
Snowpack becomes way faster: Instead of running everything locally, just point to the pre-built resources on the CDN.

... but it also has some drawbacks:

Supporting both this new flow and the old flow complicates our current config. Not all existing config options translate to this new flow, and vice versa.
Pika CDN tree-shaking will never be as good as local tree-shaking. To fully remove any unused code, we'd need to run Rollup on top of CDN install.
Snowpack requires an internet connection Snowpack already requires an internet connection indirectly, due to its implicit dependency on npm, so this doesn't really change anything there.

Thoughts?
This PR is really just a chance to gather feedback before I move any further on this. This would be a really interesting & exciting direction for Snowpack to take (did... uh... did we just create a package manager?) but I wanted to share this before we go too far down this path.
/cc @dangodev @alex-saunders @monchi @stramel

@FredKSchott I love this idea! After chatting with people about snowpack, a common confusion I've encountered is how it's not clear that snowpack is ran after npm install (rather than as a replacement to npm install), as some people percieve snowpack to be an all-in-one web dependency package installer/manager

did... uh... did we just create a package manager?)

☝so this really resonates!
Personally, I think running rollup after the CDN install is a perfectly reasonable step to take to achieve efficient tree-shaking.
I wonder if it's worth hanging on to babel to keep as an optional transpilation step as running code through @preset-env is a nice optimisation step. Saying that, I love the idea that we can remove dependencies such as terser in order to keep snowpack as slim as possible (I've also been thinking we could have a review of snowpack's dependencies to see if there's any we can remove/replace, but that's a whole other PR!).
Overall, excited to see where this will go

I wonder if it's worth hanging on to babel to keep as an optional transpilation step as running code through @preset-env is a nice optimisation step.

Pika CDN actually does differential serving, so you could choose whether you want ES2020, ES2019, ES2018, or ES2017 transpilation and polyfills when you go to install. This isn't as granular as preset-env + browserlist, but it may be enough for the average user with an option to add Babel if desired (oh no, am I picturing a plugin system? npm install @snowpack/core @snowpack/plugin-babel? 🙀)

I really love the idea of allowing Snowpack to operate independently of npm/yarn and allowing Snowpack to become a very light implementation. I really haven't had too much of a chance to look through the source or play around with Snowpack so my thoughts are much of an outside perspective.
Not to complicate things more but this part made me think:

Snowpack requires an internet connection Snowpack already requires an internet connection indirectly, due to its implicit dependency on npm, so this doesn't really change anything there.

What if we could improve this or even eliminate the need for internet connection after an initial run? I have seen more and more packages creating a node_modules/.cache/<package_name> folder to hold their cache in. If you properly hashed the contents of the files, we could cache the packed files to eliminate additional runs and potentially all together? You could take it a step further and have a remote/global cache to reference as well.

Instead of running npm/yarn and THEN running Snowpack, you just run Snowpack. You can keep using npm/yarn to manage your non-web (Node.js) dependencies.

I was thinking maybe, if there isn't a node_modules folder present when ran, we could default to using the Pika CDN as well?

Pika CDN tree-shaking will never be as good as local tree-shaking. To fully remove any unused code, we'd need to run Rollup on top of CDN install.

I think this is the biggest drawback for me but maybe if you want to get more advanced tree-shaking that could be an "advanced setup"?

(did... uh... did we just create a package manager?)

🤣 I think I remember a few years ago you wanting to do this lol
I'm very excited to see where you go with this direction. I think it could be a very cool idea/direction! Great stuff as always Fred!

What if we could improve this or even eliminate the need for internet connection after an initial run? I have seen more and more packages creating a node_modules/.cache/<package_name> folder to hold their cache in. If you properly hashed the contents of the files, we could cache the packed files to eliminate additional runs and potentially all together? You could take it a step further and have a remote/global cache to reference as well.

+1! Each install would essentially be mirroring a slice of the CDN graph, so there's no reason you couldn't just start to build that graph locally on your machine in some cache and then only fetch if a file was not found. Repeat installs could then operate offline!

I think I remember a few years ago you wanting to do this lol

Sometimes the real package manager is the friends we made along the way, Michael :)
Thanks for your feedback/input!

did... uh... did we just create a package manager?

So I’m pretty excited overall for this direction! Releasing this would be a huge opportunity to raise awareness of something I think the JavaScript community at large is still oblivious to: ESM can import remote packages.
Following the Jurassic Park theme I don’t this is a case of “asking whether or not we should;” I think this actually moves ESM closer to where we want it to be: relying on trusted, hosted sources of JS packages means users don’t have to download React on every website they visit (aside from the conversation about whether React is worth the weight, but I digress).
It’s kinder to users, more conservative of resources, and creates a more accessible web with bandwidth not edging people out of entire experiences.

My only question for this direction is how much we need to worry about / manage package versions. I’m guessing that somewhere in the magic where Rollup bundles each dependency as its own file, it utilizes npm and package-lock.json for version resolution for top-level packages as well as resolving sub-dep mismatches when 2 dependencies require different versions.
This is pretty complicated in itself. And it’s part of the reason for Yarn’s creation (IIRC; I may be wrong), and is something npm itself is working on improving.
Is that true that we can stand on the shoulders of giants in resolving sub-dep versions gracefully, by leaning on npm’s version resolution?
Also, in a world where you simply import preact from 'preact', how can we guarantee if you wrote your application for 9.x, it wouldn’t automatically start loading at 10.x?
Maybe that’s not really an issue, as your application is still being deployed manually in this scenario, so it would remain at whatever version you built it. But basically I’m just trying to ask: are there any considerations to package versioning that we’d need to consider with --source pika?

I love this idea! Snowpack with Pika would lead Web platform in the right direction.
I have just one concern: Pika lock-in. This idea is built on the trust to the package source, so I think anyone should be able to implement their own package source.
I suggest to define something like Snowpack CDN Specification, so that Snowpack can refer any CDN that satisfy the spec via --source option.
Also, IMO, optimized tree-shaking should be supported in Snowpack toolchain (not means Snowpack core). There is no reason to skip tree-shaking, everyone use tree-shaking, so I think we should provide easy-setup for tree-shaking.
Totally I'm really excited to see the future of Web development powered by Pika/Snowpack!

@Monchi great point, thanks for calling that out. Lets definitely keep that in mind: right now there are a few very Pika CDN-specific transforms in this PR that we could easily generalize in the future. Either through some sort of "source plugin" system (where a plugin defines the URL scheme for each import) or generalizing and then standardizing on a standard ESM CDN interface (where then --source just becomes the domain name you request from).
Thanks for the feedback everyone. It's really exciting that people are excited about this! I'll look to get this PR into a better state sometime this week, and then could maybe even merge in the next week or two.

Quick update with work still todo before we can pull this out of WIP status:
non-JS asset support
We've been supporting non-JS assets in the webDependencies array for a while now, but it may be time to rethink this. In the original PR to add that support, we'd talked about a "webAssets" array for these non-JS assets, but basically hadn't wanted to add a new config option when we didn't really understand the problem / needs. Now that we do a bit better, I think revisiting that proposal would make sense. Will spin up a separate issue to discuss.
Local fallback support (or at least documentation)
I ran this branch of Snowpack on www.pika.dev, and found that we have two packages that don't currently run on the CDN (due to issues with how those packages are configured.) I think we could get both issues solved in the packages themselves, but that's not a great story for other projects who may run into their own issues in different packages. So I wonder if packages that fail "--source pika" should fallback to a traditional install, at least while we're in this ESM transitional period . Since we're still creating the same "web_modules/" directory at the end of the day, that sounds like it will work, but I want to test it out more before officially suggesting it.
Cleanup
This PR is guilty of some egregious copy-paste, and src/index.ts is now 700+ lines of code which is right around where I start to have trouble navigating a file. I'll probably end up pulling some cleanup work out of this PR and into master before merging this.

Update: I was able to kill all 3 birds with one stone by changing the thinking on this from an "alternate install flow" to a "pre-pass" step.
Essentially, we'll look at all of your install targets and try to install each from the CDN into your local cache. Then, we'll do a normal Snowpack run for you but "resolving" packages to that cache instead of your node_modules directory. This gives us your normal bundling/tree-shaking/optimizing you're used to today without needing an npm install for most packages PLUS a natural fallback for packages not yet supported by the CDN: you just pull from local node_modules like you do today.
The branch code is still a mess, but it's becoming less of a POC now and starting to take on something closer to it's (most likely) final form.

Update: This branch is ready for some early, experimental testers! Just run with --source pika to enable this new mode (everything else should work as expected). There are no longer any known issues/bugs, but if you do encounter one the error logging is pretty weak.
Error/debug logging, test fixing, and some new documentation are next up.
Screenshot

( ^ notice the delta between a normal install (old) and a pika CDN install!) Files are bigger in normal dev mode since you're serving full packages (no treeshaking at all). But --optimize installs are exactly the same as before, tree-shaken and everything!)
Install time is definitely a little slower compared to a non-CDN local install (obviously, since we're introducing a network step), but overall its way faster than npm install && snowpack.

Update: The error/info logging is a bit better, some bugs are fixed, and tests are now passing. This is definitely ready for testing now if you're interested.
Some new documentation, new tests, and code cleanup are all that's left before this should be ready for review!

Code cleaned up, and documentation added:

Last to do is tests

Ready for review: #228


Implement rest of layoutchildren

This PR implements the basic layout and scrolling/recycling features for only
the case where flexDirection = ROW and flexWrap = WRAP and the vertically
scrolling.

@yaraki @yigit this PR implements the basic layout/scroll/recycling features.
Could you have a look?

Sorry sent a commit in the middle of the review, I'll send another squashed commit.

Thanks! Sent a squashed commit https://github.com/google/flexbox-layout/pull/129/commits/587f4a17b454ec969218c9fc95f76a2673c5c556

@yigit Thanks. Addressed in https://github.com/google/flexbox-layout/pull/129/commits/2c82c6e44d313e64779133c1d109c03d1b625afb. PTAL.

Btw the CI failure looks an infrastructure error of the Firebase Test Lab. Tests passing in my local.

I think I addressed all. If there is something you notice, please feel free to leave any comments I'll address in another PR.


fix: outdated code

[x] Fix Package Name
[ ] Fix Failing Tests
[ ] Add Testing to PRs

High priority, helps in making further improvements.

@eshanvaid


Terminate NPC application if any of the watches panic

This is #3792, rebased, squashed to remove back-steps, with a couple more commits to improve error reporting and remove an unnecessary variable.
Fixes #3764
Credit to @naemono
Work-around for https://github.com/kubernetes/client-go/issues/838 / https://github.com/kubernetes/kubernetes/issues/93641

thanks @bboreham .  I've been out of the country for a while.  👍


Diff fails with NoMethodError: undefined method '[]' for nil:NilClass when config.mnemonicprefix=true

How to reproduce:

Enable mnemonicprefix in git config
Try to extract diff

The reason it fails is this where it matches against a and b, and when that config is enabled it will be i or w.

@jaimerson Thanks. Would you mind producing a failing test case (unit test) for this? It will help us narrow down the problem further, and fix it.


document.documentElement.outerHTML only return the head

Outputs in debugger

Code here: https://github.com/ndaidong/article-parser/blob/main/src/utils/stripUnwantedTags.js#L19=
Can test with parseWithHtml tests

@SettingDust is this a barrier language or can you write down a simple code that fails expectations (also written down)?!?
This is now you setting the outerHTML which I am not sure I care at all on the html level so this is a great candidate for a won't fix unless you explain to me what is it that you are trying to do ... second issue opened and I still don't understand what you are after.
Please explain with more words ... a code example in here, and feel free to use Google translator if that's the problem.

The example sets the outerHTML ... have you tried using the API as it's meant to be used instead? That means you pass the HTML to the DOMParser, not as outerHTML ... can you please use the API as shown in the README or the docs and tell me what is your expectation and what happens instead?
And once again, the document.toString() will never be the same of documentElement.puterHTML because the document includes the <!doctype html> part, when present.

assert(
  parseHTML('<title>test</title>
asdf
').document.documentElement.outerHTML,
  '<title>test</title>
asdf
'
);

will return
 Expected: <title>test</title>
asdf

 Got instead: <title>test</title>

MB, I didn't clarified. SRY

Thx a lot for patience

@SettingDust it's OK but ... outerHTML means outerHTML ... a  element with outerHTML expects ... too, not just its content, 'cause that's innerHTML, not outerHTML ... this is why these issues you opened got me confused every single time ... if you set outerHTML of a bodyyou need to pass the  in it too ... maybe doing that way it works too? I don't know, but I hope these exchanges helped 👋

The input of my use case isn't controlled. My program extract the article part of a page and input to linkedom for post process like remove unwanted tags.
Maybe I should hold the element and operate the dom tree directly instead of parse the processed html.

@SettingDust yeah ... you are using the wrong tool for the job then, sorry .. this is not a browser, not JSDOM, nothing that will ever care about your use case so this is a won't fix indeed.


Bug: 400 error on checkbox

Description
the below works fine with a pydantic form but not with a dataclass,
if the checkbox is checked, then the payload here would be enable_2fa=on which pydantic casts to a bool
with the dataclass I get {"status_code":400,"detail":"Validation failed for POST http://<IP_ADDRESS>:9000/users/me/add_2fa","extra":[{"message":"Expected bool | null, got str","key":"enable_2fa","source":"body"}]}
URL to code causing the issue
No response
MCVE
# @dataclass
# class Add2FAForm:
#     enable_2fa: bool | None = None

class Add2FAForm(BaseModel):
    enable_2fa: bool | None = None

@post("/me/add_2fa", name="add_2fa")
async def add_2fa(
        request: Request,
        conn: BuildPgConnection,
        data: Annotated[Add2FAForm, Body(media_type=RequestEncodingType.URL_ENCODED)],

Steps to reproduce
1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Screenshots
"![SCREENSHOT_DESCRIPTION](SCREENSHOT_LINK.png)"

Logs
No response
Litestar Version
litestar = "^2.0.0b4"
Platform

[X] Linux
[ ] Mac
[ ] Windows
[ ] Other (Please specify in the description above)

Funding

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified

Not a bug but intentional. Treating on as a bool is Pydantic specific behaviour.
Litestar does not aim to abstract away all differences in serialisation/validation behaviour of the various supported libraries (msgspec/attrs/Pydantic), since that would make supporting them in the first place a bit pointless.


⚠ Bofrost.be has degraded performance

In 3bc3442, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1266 ms

Resolved: Bofrost.be performance has improved in f4fbd12 after 7 minutes.


fix: referenced models should render their type as pascal case

Description
This PR fixes a naming issue between referenced models when being rendered as properties.
Related issue(s)
fixes #168

Thanks!


Replaced unwrap with proper error handling on create_image

Fixes crash when client asks for unsupported texture type

Please squash the commits as well: our merge bot doesn't know how to do it.

Its ready.


Message Enum doesn't allow for expandability

Description

The Message enum as defined in the protocol spec doesn't support future development (i.e. instead of spacing out enum values they're grouped together).
This will pose code consistency and readability issues in the near future when expanding on existing Message types (e.g. try adding a new kind of Discovery Message).

@ruslan120101 we've migrated to using Jira for issue tracking, please open an issue there and close this one. You'll need a Linux Foundation ID to login.
Thanks


Hotel and Scripts updates

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

BingAds/hotel-service/manage-hotel-campaigns.md
:white_check_mark:Succeeded

BingAds/hotel-service/reference.md
:white_check_mark:Succeeded

BingAds/scripts/concepts/errors-and-warnings.md
:white_check_mark:Succeeded

BingAds/scripts/includes/date-range-constants.md
:white_check_mark:Succeeded

BingAds/scripts/reference/AdGroupSelector.md
:white_check_mark:Succeeded

BingAds/scripts/reference/CampaignSelector.md
:white_check_mark:Succeeded

BingAds/scripts/reference/KeywordBidding.md
:white_check_mark:Succeeded

BingAds/scripts/reference/KeywordSelector.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[fix] Force the default target

@lovesh This change does work, however I'm not exactly sure why.
Included SDK Makefiles already define all and bin/app.elf targets, and I though there was some kind of override shenanigans there, so I renamed your entry point target and explicitly marked it as the default target. The rust target is built before anything else, then the default (defined as bin/app.apdu in the SDK Makefile.defines file) target is invoked.
I'm not sure exactly why your Makefile could not properly build today, as we've already deployed this application before, with at first glance the same constraints.

Yes. Thank you!


CLOSURE_PATH location issues on Win7

Hello All,
I've tried every variation I can think of to get this working.  I've used the closurePath param, I've used system variables, but I cannot get this task to run without it failing, telling me that it cannot find the jar.
Is anyone else having issues on Windows or is it just me?  My grunt.js file is in a root directory.  I've added a 'Closure Compiler' directory with the compiler.jar file inside.  It's one level up.  Here are some example values I've set for the closurePath -
closurePath: '',
closurePath: '/',
closurePath: 'Closure Compiler',
closurePath: '/Closure Compiler',
closurePath: 'Closure Compiler/',
closurePath: '/Closure Compiler/',
closurePath: 'Closure Compiler/compiler',
closurePath: '/Closure Compiler/compiler',
closurePath: 'Closure Compiler/compiler.jar',
closurePath: '/Closure Compiler/compiler.jar',
Nothing works. I've tried every variation I thought it could be.  Here's my object as it stands now:
'closure-compiler' : {
frontend: {
closurePath : 'Closure Compiler/',
js          : 'components/footer.js',
jsOutputFile: 'js/footer.min.js',
maxBuffer   : 500,
options     : {
compilation_level: 'SIMPLE_OPTIMIZATIONS'
}
}
},

Now I'm getting an issue.
Ive installed using npm package manager and I am using grunt to run it.
Downloaded jar file, pasted it into a build folder inside node_modules/grunt-closure-compiler and run
Now the issue is this, java error.
Warning: Command failed: Exception in thread "main" java.lang.UnsupportedClassVersionError: com/google/javascript/jscomp/CommandLineRunner : Unsupported major.minor version 51.0
at java.lang.ClassLoader.defineClass1(Native Method)
at java.lang.ClassLoader.defineClassCond(ClassLoader.java:637)
at java.lang.ClassLoader.defineClass(ClassLoader.java:621)
at java.security.SecureClassLoader.defineClass(SecureClassLoader.java:141)
at java.net.URLClassLoader.defineClass(URLClassLoader.java:283)
at java.net.URLClassLoader.access$000(URLClassLoader.java:58)
at java.net.URLClassLoader$1.run(URLClassLoader.java:197)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:190)
at java.lang.ClassLoader.loadClass(ClassLoader.java:306)
at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:301)
at java.lang.ClassLoader.loadClass(ClassLoader.java:247)
Use --force to continue.

Looks like version error. What should I do now? Using npm install grunt-closure-compiler doesnt download build/compiler.jar file. Ive manually downloaded jar file and used, but still problem.
I've given path like this: closurePath: 'node_modules/grunt-closure-compiler',
Any solution for me.
I am in a mac machine.


Remove futures_core::stream::Stream from public API

Motivation and Context
Addresses boxes 2 and 5 in #2413
Description
This PR removes the futures_core::stream::Stream trait from public API, as enforced by our tool check-external-types. In our codebase, we previously relied on the Stream  trait for the following use cases:

calling .next().await to retrieve items from a stream
calling .collect().await to dump elements into a collection
passing types to functions/methods accepting impl Stream

Removing the Stream trait from the use cases above affected the following types:

aws_smithy_async::fn_steram::{FnStream, TryFlatMap}
aws_smithy_http::byte_stream::ByteStream
aws_smithy_http::event_stream::sender::{EventStreamSender, MessageStreamAdapter}

The primary approach taken in this PR involves

defining async fn next(&mut self) -> XXX for types that used to implement the impl Stream
implementing a .collect stream extension method for FnStream since it was previously used in its example
using FnStream at its core when types needed to hold/pass/return Stream trait objects
using FnStream to replace usages of a stream! macro from async_stream
introducing new-types HyperBodyWrapEventStream and HyperBodyWrapByteStream for those that were previously passed to hyper::body::Body::wrap_stream, which requires impl Stream, but no longer implement the trait.

Testing
Ported existing tests (many of which used a stream! macro from the async-stream crate) to use FnStream.
Checklist

[x] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[x] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

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

Closing this PR for the reason https://github.com/awslabs/smithy-rs/pull/2910#discussion_r1319214680.


Public bernstein

Makes the Bernstein polynomial functionality (xPolynomial and yPolynomial, as well as root finding) of Bezier curves public.

Codecov Report

Merging #85 (60a3b3e) into 0.9-release (38b45b3) will decrease coverage by 0.12%.
The diff coverage is 86.44%.

@@               Coverage Diff               @@
##           0.9-release      #85      +/-   ##
===============================================
- Coverage        96.65%   96.53%   -0.13%
===============================================
  Files               40       40
  Lines             6222     6230       +8
===============================================
  Hits              6014     6014
- Misses             208      216       +8

Impacted Files
Coverage Δ

BezierKit/Library/BezierCurve.swift
96.68% <0.00%> (-3.32%)
:arrow_down:

BezierKit/Library/Polynomial.swift
97.46% <95.23%> (-1.25%)
:arrow_down:

BezierKit/BezierKitTests/PolynomialTests.swift
100.00% <100.00%> (ø)

BezierKit/Library/CubicCurve.swift
100.00% <100.00%> (ø)

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 38b45b3...60a3b3e. Read the comment docs.

Codecov Report

Merging #85 (60a3b3e) into 0.9-release (38b45b3) will decrease coverage by 0.12%.
The diff coverage is 86.44%.

@@               Coverage Diff               @@
##           0.9-release      #85      +/-   ##
===============================================
- Coverage        96.65%   96.53%   -0.13%
===============================================
  Files               40       40
  Lines             6222     6230       +8
===============================================
  Hits              6014     6014
- Misses             208      216       +8

Impacted Files
Coverage Δ

BezierKit/Library/BezierCurve.swift
96.68% <0.00%> (-3.32%)
:arrow_down:

BezierKit/Library/Polynomial.swift
97.46% <95.23%> (-1.25%)
:arrow_down:

BezierKit/BezierKitTests/PolynomialTests.swift
100.00% <100.00%> (ø)

BezierKit/Library/CubicCurve.swift
100.00% <100.00%> (ø)

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 38b45b3...60a3b3e. Read the comment docs.


ignore removeTopScreen on empty backstack

Issue https://github.com/AlexGladkov/Odyssey/issues/64

https://github.com/AlexGladkov/Odyssey/issues/64

Fixed in release 1.3


Implement DReps notifications

some notes

to perform every 1h or so..
wallets that are registered to a drep using https://api.koios.rest/#post-/account_info_cached
process in batches all accounts that have the drep.
gather drep votes (cach it) using https://api.koios.rest/#get-/drep_votes (unfortunately one by one this is why cashing is important)
show the user the link to gather information about the gov action (for that use the https://api.koios.rest/#get-/proposal_list or any website when available maybe https://gov.tools/connected/governance_actions/7fd6429add8f2611ad8d48c0cc49101463093aec285faea402e8cfde78ea58d7#0) that hash is the proposal_tx_hash. So easier to use that so you don't have to parse the external URLs

That's how it looks like


transformer package refactor

Description of changes
Create a top-level transform package for the v2 transformer chain to simplify invocation from the API Category, and simplify invocation of the transform/preprocessing system from multiple clients.
Step 1. break apart the v1/v2 logic a bit, and moving the chain generation into a separate file (which will eventually be pulled into it's own package). Merged in https://github.com/aws-amplify/amplify-category-api/pull/1528
(Current Step) Step 2. Consolidate all input param generation into a single place for the v2 transformer, and migrate GraphQLTransform construction into a single place, with a tight interface (today is overly permissive, and leaks in significant amplify cli and deploy concerns).
Step 3. Execute transform within new single location as a separate method, and accept remaining props (e.g. transform execution props, and printer callback) into the utility.
Step 4. Final cleanup of remaining code, and split into a new package.
graph TD;
    api("@aws-amplify/amplify-category-api")
    new("Additional Transformer Clients")
    transform("@aws-amplify/graphql-transformer")
    core("@aws-amplify/graphql-transformer-core")
    model("@aws-amplify/graphql-model-transformer")
    auth("@aws-amplify/graphql-auth-transformer")
    rem("...Remaining @aws-amplify/graphql-*-transformer")
    api-->transform;
    new-->transform;
    transform-->model;
    transform-->auth;
    transform-->rem;
    model-->core;
    auth-->core;
    rem-->core;

CDK / CloudFormation Parameters Changed
N/A , this should strictly be refactor of existing interfaces.
Issue #, if available
N/A
Description of how you validated changes
Unit + E2E Tests, will manually validate the flags I'm changing (specifically create api key, and dedupe resolvers still work).

[ ] Verified CreateApiKey behavior has not changed
[ ] Verified Dedupe Resolvers behavior has not changed

Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added

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

Codecov Report

Merging #1548 (66f6142) into main (7150e75) will decrease coverage by 0.01%.
The diff coverage is 71.42%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1548      +/-   ##
==========================================
- Coverage   63.14%   63.13%   -0.01%
==========================================
  Files         362      362
  Lines       21406    21404       -2
  Branches     5089     5088       -1
==========================================
- Hits        13516    13514       -2
  Misses       7162     7162
  Partials      728      728

Impacted Files
Coverage Δ

...-api/src/graphql-transformer/user-defined-slots.ts
100.00% <ø> (+2.94%)
:arrow_up:

...api/src/graphql-transformer/transformer-factory.ts
26.38% <25.00%> (-8.62%)
:arrow_down:

.../src/graphql-transformer/transformer-options-v2.ts
16.33% <44.44%> (+0.90%)
:arrow_up:

...i/src/graphql-transformer/directive-definitions.ts
50.00% <66.66%> (+8.33%)
:arrow_up:

...graphql-transformer/transform-graphql-schema-v2.ts
74.21% <81.81%> (-0.22%)
:arrow_down:

...mplify-graphql-transform/graphql-transformer-v2.ts
100.00% <100.00%> (ø)

...ategory-api/src/amplify-graphql-transform/index.ts
100.00% <100.00%> (ø)

...l-transformer-core/src/transformation/transform.ts
16.19% <100.00%> (+0.52%)
:arrow_up:

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


Add health check endpoint

Issue: Request for Health Check Endpoint
Description:
I need an endpoint on your service to add a health check to determine if I need to kill/restart the application.
Requirements:

The endpoint should return the health status of the application.
The response should include relevant metrics or indicators that can help determine the application's health.
The endpoint should be lightweight and not impact the application's performance.

Use Case:
This endpoint will be used to monitor the application's health and make decisions on whether to kill or restart the application based on the health status.
Example:
I think most suitable would be an endpoint in the root called /healthy that just returns a 2xx response code when the app is up and running.
Additional Information:
Please provide any necessary documentation or examples on how to integrate this endpoint into our monitoring system.
Thank you!

Hi there,
Thanks for the detailed feature request 😄
There is already a /config endpoint which the frontend uses to determine the configured SSO method. This would not be reachable if the application is misconfigured and otherwise returns a 200. So if you need something quickly, this is probably a good solution.
But I do see the point of a separate /healthy endpoint with a few more details. I'm just wondering in what case the endpoint would be reachable but doesn't return a 200. Currently Kviklet doesn't even start if the DB connection fails (as it runs migrations on startup), so that's not a typical case.
It could of course test all configured DB connections, but i think this is quite expensive of an operation for a health endpoint and has little to do with how Kviklet itself is doing.
Happy about any ideas here, otherwise I'll just add a /healthy endpoint without any detailed response and expand on it when required.

I think a simple endpoint is sufficient 👍

Added in #233


Wiki links are broken

It seems the wiki links are missing the /wiki/ in the path in the main wiki page.

Looks like the Markdown preview links worked but the actual docs did not -- I see others have hit this before: http://stackoverflow.com/a/8972756/3516125. I've updated the links to fix this.


Better info when no category is defined

In current situation, ugly 404 is raised, even on DEBUG = True.
Let's make this little more user-friendly and add some eye-candy.

Here is a screenshot :)


Documentation for Isolate.spawn outdated

This was noticed by a user on stackoverflow: https://stackoverflow.com/questions/73778870/is-dartdocs-about-the-entry-of-isolate-spawn-wrong-or-something
Currently the documentation says:

The function must be a top-level function or a static method that can be called with a single argument, that is, a compile-time constant function value which accepts at least one positional parameter and has at most one required positional parameter. The function may accept any number of optional parameters, as long as it can be called with just a single argument. The function must not be the value of a function expression or an instance method tear-off.

But it is possible now to pass a function expression or an instance method tear-off.
import 'dart:isolate';

void main() async {
  await Isolate.spawn((message) => print(message), 'function expression');
  await Isolate.spawn(MessagePrinter().printMessage, 'method tear-off');
}

class MessagePrinter {
  void printMessage(message) => print(message);
}

This appears to be possible since dart 2.15:

Allow closures both in inter-isolate messages as well as as entrypoints in Isolate.spawn(<entrypoint>, ...) calls.

/cc @aam


Fixed "new Identifier()" problem in Minecraft 1.21, is "Identifier.of…

…()" and Update to Minecraft 1.21

Sorry I forget commit and pull build.gradle and some 1.21 files, so I closed the pr

sorry it's push not pull

updated in my pr

Sorry I forget commit and push build.gradle and some 1.21 files, so I closed the pr


where is the jodit component store which is import in include.jodit.js

jodit latest version
file name : include.jodit.ts
import { Jodit } from 'jodit/es2018/jodit.fat.min';
import 'jodit/es5/jodit.min.css';
export { Jodit };
I did not find out 'jodit/es2018/jodit.fat.min' this path in github latest code

@MangeshPatleDatastack , you need to build it, ok when install it using npm in a folder and then look for it.


CASMINST-6340: Add missing links to operations index

Description
I found more pages which are missing from the operations index page. This PR adds them to the appropriate sections of the index.

Checklist

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[X] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.4 release/1.3 release/1.2 release/1.0


SW-2278 Show plot labels only after zoomed in

avoid showing during the zoom process to avoid labels dancing around
to test, view difference with vercel link versus staging

Is this change only for the first loading zoom right? Because when zooming in or out with the control, I don't see any change

Yes only on the zoom when the labels are dancing around.


Revert STS from 1.4 with cherry picks of integration and stress test fixes.

This is an alternate to https://github.com/istio/envoy/pull/175.  It cherry picks some integration and stress test fixes from envoyproxy/envoy-wasm in addition to the STS reverts to try to get builds passing.

I disagree that we need to get release-1.4 branch into a state that looks
like 1.4.6

I agree. We need only to get release-1.4 branch into a state of what we
want to get shipped in 1.4.7.
On Wed, Mar 4, 2020 at 6:14 PM Kuat<EMAIL_ADDRESS>wrote:

@kyessenov commented on this pull request.
In api/wasm/cpp/proxy_wasm_api.h
https://github.com/istio/envoy/pull/180#discussion_r388043518:

std::unordered_map<uint32_t, std::unique_ptr> grpc_calls_;

std::unique_ptr cur_grpc_stream_;

I disagree that we need to get release-1.4 branch into a state that looks
like 1.4.6. We did not ship from 1.4 branch, as in reality we created a
private 1.4.6 branch that was developed independently from 1.4 branch. Now
that 1.4.6 branch is done, we should simply merge 1.4.6 branch into 1.4
branch (and that was exceptional due to secrecy around the patches).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/envoy/pull/180?email_source=notifications&email_token=AAEYGXJY2VCPQSFEODJ7T3TRF4DGXA5CNFSM4LAWICCKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCYBF7MY#discussion_r388043518,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXIKVCSGZSZ6X464O5DRF4DGXANCNFSM4LAWICCA
.

OK, I can agree with that.  And John's point is accurate - we just need to get it into a state for what we want shipped in 1.4.7.
That means at this point that if there are pure bug fix SHAs they should stay in the branch.  So   @kyessenov @bianpengyuan send them in a PR or list the SHAs here and I'll get them back into the branch.   But if there are features mixed in with the SHAs, then we are still having a discussion with the community about whether we can include them in 1.4.7

I think the relevant stake holders are the release managers for 1.4, who have already looked into those PRs (in master or other branches sometimes and cherry-picked).
Please let's not call STS a google-specific extension. This is an IETF draft-ietf-oauth-token-exchange-16, which is implemented by AWS and other providers.

@kyessenov the issue is that these are features going into a patch release.  I am within my rights to object to that.
Also, I can't find a place where STS is called a google-specific extension.

@kyessenov

We don’t have a clear view of what to do with vendor specific extensions yet. But now we will have to figure it out.
Since this is the Envoy repo,  if upstream envoy accepts something, we don’t need to have independent judgment about vendor specificity of the issue.

vendor specificity  is an issue for OSs istio-proxy repo, but we can set that aside for now.
The primary issue then is feature addition in 1.4.5 of Istio. Since this code has never been shipped in a OSS release we can let it all roll back.

I get that features in a patch release is an issue, and I'm OK with pretending those changes never were approved. I just don't think we should be using the CVE release as the reason to revert those changes. It's making a bad precedent to have a bunch of changes reverted post-factum, and creates a lot of work figuring out which change corresponds to which feature. Clearly, we do want some important bug fixes in 1.4.7.
If I were to patch 1.4 six months from now (like what I had to do for 1.2), I'd be enormously confused by what happened in the commit history.

https://github.com/istio/envoy/pull/185 for the bug fix reverted by this pr.


Hello there. Loran Kloeze, if you see this problem, please contact me immediately, I can be your sponsor, and it is lasting.

Hello there. Loran Kloeze, if you see this problem, please contact me immediately, I can be your sponsor, and it is lasting.

Please post the errors you encounter.


python3Packages.fastjsonschema: 2.14.5 -> 2.15.1

Motivation for this change
Update to latest upstream release 2.15.1
Change log: https://github.com/horejsek/python-fastjsonschema/blob/master/CHANGELOG.txt
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 126735 run on x86_64-linux 1

14 packages built:

python38Packages.fastjsonschema
python38Packages.qiskit
python38Packages.qiskit-aer
python38Packages.qiskit-aqua
python38Packages.qiskit-ibmq-provider
python38Packages.qiskit-ignis
python38Packages.qiskit-terra
python39Packages.fastjsonschema
python39Packages.qiskit
python39Packages.qiskit-aer
python39Packages.qiskit-aqua
python39Packages.qiskit-ibmq-provider
python39Packages.qiskit-ignis
python39Packages.qiskit-terra

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 126735 run on x86_64-linux 1

14 packages built:

python38Packages.fastjsonschema
python38Packages.qiskit
python38Packages.qiskit-aer
python38Packages.qiskit-aqua
python38Packages.qiskit-ibmq-provider
python38Packages.qiskit-ignis
python38Packages.qiskit-terra
python39Packages.fastjsonschema
python39Packages.qiskit
python39Packages.qiskit-aer
python39Packages.qiskit-aqua
python39Packages.qiskit-ibmq-provider
python39Packages.qiskit-ignis
python39Packages.qiskit-terra

whoops. late to the party.


Command could not found openroad

Subject
[Build] for compilation-related issues.
Describe the bug
"openroad" command cannot be used after installation of the project.
I try to find command file to link to my environment but it does not work.,
I cannot find any bin or sbin/openroad file in my OpenROAD. Only a file called openroad in the direction of ./src/openroad.
Expected Behavior
I need to run the openroad command to start a design flow.
Environment
Ubuntu 22.04

To Reproduce
git clone --recursive https://github.com/The-OpenROAD-Project/OpenROAD.git
cd OpenROAD
sudo ./etc/DependencyInstaller.sh -run
sudo ./etc/DependencyInstaller.sh -dev
sudo apt-get install libspdlog-dev
sudo apt install tcl-dev
./etc/Build.sh
openroad -gui # the final row
error then happening
Relevant log output
No response
Screenshots
No response
Additional Context
No response

[100%] Built target openroad
[100%] Linking CXX executable trTest
cd /home/eeuser/OPENROAD/OpenROAD/build/src/drt && /usr/local/bin/cmake -E cmake_link_script CMakeFiles/trTest.dir/link.txt --verbose=1
/usr/bin/g++ -O3 -DNDEBUG CMakeFiles/trTest.dir/test/gcTest.cpp.o CMakeFiles/trTest.dir/test/fixture.cpp.o CMakeFiles/trTest.dir/test/stubs.cpp.o CMakeFiles/trTest.dir/__/gui/src/stub.cpp.o -o trTest  drt.a ../stt/stt.a ../dst/dst.a /usr/local/lib/libboost_serialization.a /usr/local/lib/libboost_system.a /usr/local/lib/libboost_thread.a ../dbSta/src/dbSta.a ../gui/gui.a ../odb/src/cdl/libcdl.a ../odb/src/defin/libdefin.a ../odb/src/def/libdef.a ../odb/src/def/libdefzlib.a ../odb/src/lefin/liblefin.a ../odb/src/lef/liblef.a ../odb/src/lef/liblefzlib.a ../odb/src/db/libdb.a ../odb/src/defout/libdefout.a ../odb/src/lefout/liblefout.a ../odb/src/db/libdb.a ../odb/src/defout/libdefout.a ../odb/src/lefout/liblefout.a ../odb/src/upf/upf.a ../odb/src/tm/libtm.a ../odb/src/zutil/libzutil.a /usr/lib/x86_64-linux-gnu/libtcl.so /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.15.3 /usr/lib/x86_64-linux-gnu/libQt5Gui.so.5.15.3 /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.15.3 /home/eeuser/OPENROAD/OpenROAD/src/sta/app/libOpenSTA.a ../utl/utl.a /usr/local/lib/libspdlog.a /usr/lib/gcc/x86_64-linux-gnu/11/libgomp.so /usr/lib/x86_64-linux-gnu/libpthread.a /usr/local/lib/libboost_serialization.a /usr/lib/x86_64-linux-gnu/libz.so
gmake[2]: Leaving directory '/home/eeuser/OPENROAD/OpenROAD/build'
[100%] Built target trTest
gmake[1]: Leaving directory '/home/eeuser/OPENROAD/OpenROAD/build'
/usr/local/bin/cmake -E cmake_progress_start /home/eeuser/OPENROAD/OpenROAD/build/CMakeFiles 0
real	11m16.908s
user	79m4.245s
sys	4m29.090s

As per log installation successful. Execute below command now
cd /home/eeuser/OPENROAD/OpenROAD/build/src
./openroad -gui

Ok! It do work right now, Thanks for your help! I do appreciate it!

Other way add following format in .bashrc file and execute from anywhere export OPENROAD_HOME="<path-to>/OpenROAD/build/src" source ~/.bashrc $OPENROAD_HOME/openroad -gui

repeat this. It can be source anywhere and launched.


Small helper to run remote commands

Have you considered this TF provider in case null-provider or remote-exec wasn't elegant enough?
https://registry.terraform.io/providers/loafoe/ssh/latest/docs


102224 about the conversion page

Test Dev URL:-
https://localhost:44350/school/school-conversion-key-details?appId=2147483647&Urn=101934

waiting to get a working url before I hit Approve

It works for me. Do you definitely have the secrets setup correctly with a valid trams API URI?


Improvements to schema change dialog

have a 'cancel' button. Currently to cancel you have to click on the x sign
have a link to the documentation for what changes are allowed in each compatibility mode
do not have the pop up if compatibility mode is 'undefined', since there would be no schema changes that can't be rolled back then

Awesome

👍

👍