Output vector inconsistent with the nature paper?

Hey,
In lines 118~121 of dnc.py, the final output is by passing the concatenated controller_output and access_output throught a Linear
    output = tf.concat([controller_output, batch_flatten(access_output)], 1)
    output = snt.Linear(
        output_size=self._output_size.as_list()[0],
name='output_linear')(output)

But according to the nature paper, in the part above Interface parameters in the left column of page 477, it is stated as: Finally, the output vector y_t is defined by adding v_t to a vector obtained by passing the concatenation of the current read vectors through the RWxY weight matrix W_r.
So should the output from the controller and the output from the read heads first be concatenated then passed through a Linear layer, or should the output from the read heads first be passed through a Linear layer then be concatenated with the output from the controller? Or am I misreading sth here?
Thanks in advance!

It is actually consistent with the paper, but this may not be immediately clear.
The v_t as defined in the paper is the linear-transformed controller output (two equations up from the one that you reference). I.e. an embedding W_y is applied to the h_i. So in the code 'controller_output' is actually [h^1_t, .. h^L_t] and v_t is this vector passed through a linear to shape it to the output size. It is then added to the embedded read words.
Concatenating two vectors and passing them through a linear (as defined in the code) is trivially equal to adding the two vectors passed through separate linears (as defined in the paper). The only discrepancy is that bias terms are used within the linears in the code, whereas no biases are described in the paper.
Does that make sense?

Ah right exactly, thanks a lot for the reply :)

Oh another thing, is the output of the Neural Turing Machine using the same mechanism as DNC (by linearly transform the concatenated hidden_state and the read_vectors_of_timestep_t, then clip in between (-20.,20.)), or its output is just a linearly transformed hidden_state? Cos this part is not so clear from the NTM paper.
And just to make sure, for both NTM and DNC, is the input to the controller module always: input_sequence and the  read_vectors_of_timestep_t-1?
Thanks a lot!

And just to make sure, for both NTM and DNC, is the input to the controller module always: input_sequence and the read_vectors_of_timestep_t-1?

It is.

Oh another thing, is the output of the Neural Turing Machine using the same mechanism as DNC (by linearly transform the concatenated hidden_state and the read_vectors_of_timestep_t, then clip in between (-20.,20.)), or its output is just a linearly transformed hidden_state? Cos this part is not so clear from the NTM paper.

Both have the same output mechanism. The value clipping used here is an implementation detail to just maintain numerical stability and avoid NaNs.


Workday,Inc.PlanningforExcel version 2022.105.2718.2929

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Is this what is needed to verify the download URL? https://clickonce.adaptiveinsights.com/planning/latest/WorkdayAdaptivePlanningExcelSetup.exe

The folder would need renaming to WorkdayInc instead of Workday,Inc


Feature/pagination

Title: Implement Pagination and Sorting for Missing Persons Data In response to #58
Description:
This PR introduces pagination and sorting functionality to our Missing Persons data display. These enhancements aim to improve user experience and data organization, especially as our dataset grows.
Key Changes:

Implemented pagination to display a limited number of entries per page.
Added sorting functionality to display newest entries first based on the 'taken_time' field.
Modified the search and filter functions to work seamlessly with pagination and sorting.
Introduced new UI elements for pagination controls.

Detailed Changes:

Added new functions: displayPage(), updatePaginationControls(), and sortData().
Modified existing functions (searchFunction, filterCategory) to work with the new pagination and sorting system.
Adjusted the initial data fetch to incorporate sorting.
Added event listeners for pagination controls.

HTML Changes:

Added a new  for pagination controls below the persons container.

CSS Additions:

Basic styling for pagination controls to ensure they're visually consistent with the existing design.

Could you please resolve the minor conflicts on your PR. just a few and it will be good to go, I like the features you have been able to add in


(Request) Portability

The app in the current version saves its setting in the appdata folder. Would it be possible to make it create the file in the same directory as the program for better portability?
btw thanks a lot for making #3! It saves a lot of time.

Yeah, sure, why not.
It will take a week or two though, my vacation is almost over. :)

Done. Please check the latest version.
https://github.com/FaultierSP/civitaiapiclient/releases/tag/v0.0.5
Note that since the configuration file moved to the app's folder, you may want to check your settings again and delete old files in your system's default folder.


OTAServerDelegate class for vendor logic

Problem
Need a way to separate common code from application/vendor specific code for OTA Server Cluster, which doesn't require the developer to fork the repo.
Change overview

define OTAServerDelegate class for implementing vendor OTA Server logic
add some common logic for OTA Server Cluster commands
use OTA_SERVER_CLUSTER_EXAMPLE_IMPL to conditionally build example logic
add OTA Server Cluster to all-clusters-app and chip-tool
fix AnnounceOTAServer command direction (OTA Client client node -> OTA Client server node)

Testing
Ran chip-tool and all-clusters-app on Linux machine, confirmed that I can send query-image apply-update-request and notify-update-applied. The response to apply-update-request and notify-update-applied is EMBER_ZCL_STATUS_UNSUP_COMMAND as expected, because there is no OTAServerDelegate implemented yet. The response to query-image is a TLV parse failure because CHAR_STRING is not supported yet

Will add generated files when review of edited files is complete

/rebase

@holbrookt Can you take a peek at the failing ZAP task?

@holbrookt Can you take a peek at the failing ZAP task?

It's failing because I pushed edited ZAP files and didn't include the generated ones. I was going to wait to include the generated ones until the end of the PR approval process to avoid merge conflicts. But maybe people are waiting to review until CI is green?

PR description has been fixed to adhere to the new template.
Apologies for the messy commit history, I've been struggling with weird zap configure tool behavior, and missing zap files from an existing PR (?).

Apologies for the messy PR, I've been struggling with a few ZAP bugs and merge conflicts (see https://github.com/project-chip/zap/issues/158 and #7135).
Hand-modified files are:
src/app/clusters/ota-server/ota-server-delegate.h
src/app/clusters/ota-server/ota-server.cpp
src/app/clusters/ota-server/ota-server.h
src/app/zap-templates/zcl/chip-ota.xml
src/app/zap_cluster_list.py
src/darwin/Framework/CHIP/templates/helper.js

And modified zap files are:
examples/all-clusters-app/all-clusters-common/all-clusters-app.zap
src/controller/data_model/controller-clusters.zap

Per the updated template, can you update the PR here?

@woody-apple Please see updated description.

I believe this PR is being blocked by #7135

Seems to work after your last revert. Kicking ESP32 again...

Reverting, looks like this needs a re-base and fixes :(


Multiple App Servers & Load Balancer Help

Hello, I'm just after clarification or pointers on how best to approach laravel-websockets with my setup. I run all my servers under Laravel Forge and I use their load balancers which is a Nginx server with a reverse proxy.
My setup is 1 load balancer, 4 app servers (same codebase), a dedicated MySQL and a dedicated Redis server. Previously I ran Laravel Echo Server on the same server as the Redis DB and the 4 app servers were able to communicate, however my project now needs sockets on mobile which lead me here due to the Pusher SDK implementation.
I'm wondering how best to deploy laravel-websockets with my app. Should I run a dedicated server instance with only laravel-websockets and proxy all 4 servers there so there's a single websockets server on a subdomain or should each of the 4 app servers be running websockets:serve ?
I do intend to use Redis once v2 is ready, however I am still wondering which of the 2 solutions would be best?

Running laravel:websockets an all 4 servers in the current implementation is broken and should not be used at all. You can use 2.x, make sure to also check the 2.x PR that explains more info on how to migrate it post-rewrite (yes, it was rewritten from scratch)

Which branch do you recommend that I use? Using 2.0.0-beta.18, running sockets on each server works to a degree but when broadcasting, it's intermittently sending to one or the other servers behind the load balancer. Basically not broadcasting to all as expected.
I tried to pull a few of the latest commits but they're causing errors when running.
production.ERROR: Class name must be a valid object or a string {"exception":"[object] (Error(code: 0): Class name must be a valid object or a string at /home/vagrant/projects/mysite/vendor/beyondcode/laravel-websockets/src/Console/Commands/StartServer.php:126)
I did notice your notes about writing from scratch, is this something that's going to happen now?

The 2.0.0-beta.20 version got improved for horizontal scalability, but you need Redis.

I will be using Redis so will check this out. I have it working as is but it's not exactly how I want it so hopefully the update solves the issue I have.
My dev environment has a load balancer, 2 app servers and a dedicated Redis server. I want to broadcast events across the app for many user actions, such as company settings so all users within that company instantly have the most up to date settings.
When I click update, the event is not broadcast across all servers in unison, it's only received by some and not others. If I click update multiple times, the broadcast is received after every second click - which appears to be the load balancer (round robin) selecting which server the connection is going to. I am using Redis and running websockets:serve on both servers and still happens.
My work around was to edit the load balancer Nginx to push all /app connections to a single server and it works fine and Redis is doing it's thing.
    location /app {
        proxy_pass             http://main_server_ip:8444;
        proxy_set_header Host  $host;
        proxy_read_timeout     60;
        proxy_connect_timeout  60;
        proxy_redirect         off;

        # Allow the use of websockets
        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection 'upgrade';
        proxy_set_header Host $host;
        proxy_cache_bypass $http_upgrade;
    }

I will check out the latest beta build to see if this has changed anything.

The idea behind the Redis replication that this package provides is that when a message gets broadcasted to a channel on one instance, it automatically takes a copy of it and streams it to the other instances using Pub/Sub. I have attached a diagram.

That's exactly how I hope it will work but it wasn't doing this as of my testing of 2.0.0-beta.18 (I think) which required the Nginx config on the load balancer.
I will try 2.0.0-beta.20 later today to see if this now works.

I've just tested 2.0.0-beta.20 and it's the same issue behind a load balancer. Only some broadcasts are being received depending on which server the user has landed on.
I tried removing the Nginx config and replaced it on each app server as follows in the hope that it would resolve but it hasn't.
    location /app {
        proxy_pass             http://<IP_ADDRESS>:8444;
        proxy_set_header Host  $host;
        proxy_read_timeout     60;
        proxy_connect_timeout  60;
        proxy_redirect         off;

        # Allow the use of websockets
        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection 'upgrade';
        proxy_set_header Host $host;
        proxy_cache_bypass $http_upgrade;
    }

The only way I can get this working is if it's only the load balancer's Nginx config is changed to this below but means all broadcasts are sent from a single server but still received on all others. I've tried numerous different approaches but it just isn't broadcasting in unison and only sends to one or the other server.
    location /app {
        proxy_pass             http://main_server_ip:8444;
        proxy_set_header Host  $host;
        proxy_read_timeout     60;
        proxy_connect_timeout  60;
        proxy_redirect         off;

        # Allow the use of websockets
        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection 'upgrade';
        proxy_set_header Host $host;
        proxy_cache_bypass $http_upgrade;
    }

This approach is still slightly better than having a dedicated sockets box but this approach is still not working as intended, unless I am not approaching this right or have my configs mixed up somewhere.

Have you set the replication mode in websockets to redis via WEBSOCKETS_REPLICATION_MODE=redis?

Yes. Have you tried the setup in your diagram or is it theory that's how it should work?
I also reverted all my server settings back to how it was before and it's back to being intermittent both with user connecting and broadcasts.  I went back to my local env so it's just a single server and with 2.0.0-beta.20  both .joining() .leaving() are not triggering properly for other connected users so they do not know who's online or not now. Backed it back down to 2.0.0-beta.19 and events work again.

The diagram is in a brief detail on what actually happens. This was made side-by-side with the code. I am currently investigating how this can occur or what additional setup might inflict this problem. The .joining() and .leaving() methods not working problem is caused by the multi-node setup that doesn't replicate properly across your nodes. I can also check by the tests that the receiving via pubsub takes place.

For some kind of reason, your Redis instance might not support Pub/Sub?

For some kind of reason, your Redis instance might not support Pub/Sub?

Isn't 2.0.0-beta.19 running Pub/Sub? That version works as expected with .joining() and .leaving() both locally and on a Forge deployed setup.

I upped it back to 2.0.0-beta.20 on my local env again, .joining() and .leaving() only trigger maybe once or twice for any user logged in and another refreshes their page. After repeatedly refreshing, the events no longer trigger.
2.0.0-beta.19 I can refresh over and over, .joining() and .leaving() both trigger as expected for other users.

I just tried the 2.0.0-beta.21 from fresh and still got the issue with .joining() and .leaving() not triggering after the first refresh. I tried on my local env with both local and redis.
The console debug I can see the connections being made but they are not triggering the Echo events. 2.0.0-beta.19 still seems to be working the best for me.

If it's not working with local, then it seems it's an issue on your side and I cant properly identify what can cause that. Do you have any demo app I can try on? It doesn't have to be disclosed or something.

Make sure the config file that comes with -beta.21 is the same one as the one you got.

Why would it be working as expected for -beta.19 but broken for .20 and .21 on both a local env built on the latest Homestead release, using Laravel 8.3 and on Forge using a brand new deployed Ubuntu 20.04 (LTS) x64 server, with PHP 7.4, latest redis? Has there been any fundamental changes that require specific environment configs or software builds?
Unfortunately this implementation is for a project that I cannot disclose due to NDA which doesn't help. I'll see if I can strip this right back without disclosing the project, failing that going back to the original Echo Server setup.

I had the same issue as @AugmentBLU . I have an AWS application load balancer balancing between 2 app servers, both running websockets. The ALB has session stickiness (2 hours) enabled. The redis instance is a centralized elasticache instance (supports pub/sub) and so is the MySQL/RDS DB.
I've tried with beta 19,20 and 21 but no luck. Any broadcasts sent either go to one server or the other. The broadcast driver is pusher and the replication mode is set to redis. However I then tried to disable session stickiness on my AWS target group and group level stickiness on my application LB and it started working! Now all broadcasts are sent to clients from both servers. The session stickiness seems to have been causing the issue.

I got round the load balancer issue by pushing all /app requests to a single server but still maintained round robin. However, the main issue I'm having with -beta 20,21 now is that Laravel Echo .joining() and .leaving() events are not being triggered at all after maybe the first couple firings. This is happening on Local env, single server Forge and multi-server Forge setup, all using Redis. Also fails to trigger using local.
I have tried a very simple Echo call to check and after maybe the second or third refresh, the events stop firing.
window.Echo.join('team.' + this.team.id)
          .here((users) => {
            console.log('here', users);
          })
          .joining((user) => {
            console.log('joining', user);
          })
          .leaving((user) => {
            console.log('leaving', user);
          });

I did notice in beta 20,21 that the Redis database sets for {appID}channels and sockets were disappearing randomly from the DB but doesn't happen with beta 19.

In -beta.20 I have implemented a system that deletes stored info in Redis if the server process gets shutdown unexpectedly (no SIGTERM/SIGINT).
The idea behind it was that if a server has local connections and the server suddenly drops, having Redis as the replication mode might lead to data still stored in Redis (like users data that were in presence channels or increments for the Redis collector which can lead to wrong statistics), so if the connection didn't pong in the last 2 minutes, according to the Pusher docs, it will be marked as inactive, mock the connection details and unsubscribe it from everywhere, including deleting obsolete data: https://github.com/beyondcode/laravel-websockets/compare/2.0.0-beta.19...2.0.0-beta.20#diff-1a793d12ff9c279a1c1d734f6a00a54cR411-R427
The fact that this test pass is making me think it's something bad configured: https://github.com/beyondcode/laravel-websockets/blob/2.0.0-beta.21/tests/PresenceChannelTest.php#L64-L99
I think I'm going to set a demo Laravel 8.x repo and configure it and perhaps try it in your env somewhere to see further what's going on.

@rennokki Can we also address the issue with the load balancing? I deleted my previous comment since it was a mistake on my part to think I had it fixed. Why did @AugmentBLU have to set up forwarding for the /app path in nginx? What was your setup in your testing, did you have to do the same? The comment I'm referring to is: https://github.com/beyondcode/laravel-websockets/issues/517#issuecomment-692698236

@SelimSalihovic Having a demo application should be enough to detect what's going on.

@rennokki let me know if you need help setting up the demo repo/app. I'm open to help.

The {appID}channels and sockets sets are being deleted almost instantly after creation from a page load/refresh. Is that expected behaviour?
I had a look at my Redis config and the tcp-keepalive is set to 300 seconds, should I be checking any other settings?

I have detected issues regarding the presence channels that were more-or-less linked to the bad filtering of users. 🙄 Also, the connections do randomly disappear from the sorted set, and I'm investigating it right now since this is the main cause that makes the events to not be broadcasted at all on Redis replication. I can clearly see that the local replication works fine.

Alright, glad you managed to find something that might point towards the issues. Hopefully fixes can be found.

To sum it up, the issues were caused by the fact that I mistakenly used zrange() instead of zrangewithscore(), causing Redis to give back all connections instead of the ones that were older than 2 minutes without a single pong, so they could be cleaned up. This has to do with the Redis replication, and I really hope that -beta.22 is gonna fix the issues you got. I have tested it locally and I will update the package name and push the demo project so y'all can play with it with the local connection (although the Redis connection will work too, hopefully)

https://github.com/rennokki/laravel-websockets-demo

Nice one, I have it working on my local env using redis - I haven't tried using the local provider as I will not be needing this.
Next I'll try on my multi-server Forge setup with the load balancer. I will see if I still need the Nginx config to route /app requests to the same server as well.
I don't suppose you have tried a multi server setup with a load balancer have you?

I did but only using AWS's ALB within Elastic Beanstalk. It helps me to open up the port and route the 6001 SSL to 6001 Non-SSL internally without too much hassle.

I'll give it a shot in the next couple hours. I'm using Laravel Forge and their Nginx based load balancers, reason is I'm using Cloudflare DNS and trying to get sub-subdomains working with SSL is painful. I'm also using DigitalOcean, I did try their own load balancers but they are so bad, many people have issues with them so I stick with Forge's.
I open port 8443 as this is one of the ports Cloudflare supports sockets on, it hits the load balancer and I route any /app connections to http://private_network_ip:8444. I am running the websockets on port 8444 internally, the load balancer has the SSL only, app servers are on a private network so only use http - SSL termination basically at the LB. Everything works this way, however broadcasts sent from user A seem are quite randomly received by other users, even user A if open in another tab as they may of hit another server due to round robin.
Anyway, I'll update once I get round to pushing and testing on this setup.

The setup seems fine, you keep the LB & Internal ports to the same one, LB SSL Termination, using non-SSL for internal backend. The round-robin should not affect the way the message gets sent. As long as the server is up, it gets connected to Redis and listens to the PubSub messages if they got users in a specific channel and app id.

I have just set up the demo project with 2 servers (ec2 instances) with an AWS application LB in front of them. I am still seeing the same thing happen, so only some of the messages get sent through. This is what I noticed:

When I open the website one one device and it connects to the socket on 6001 on server 1 and on the other device I open the page and get connected to server 2 on 6001 the list of users online on the frontend only shows 1 user (the currently logged in user). Any messages sent will not appear on the other user's screen.
However, when they are connected to the same server's socket connection then I see 2 users on the users list (on both screens) and then the messages sent appear on both user's screens.

Maybe I can share the demo URL with you but I have to wait for a response from my supervisor. Maybe that could help.

@SelimSalihovic That's what I was seeing when I was testing, hence this post. The only way round it that I found was to run websockets:serve --port=8444 on all app servers and then config my LB Nginx to push all /app requests to a single server. Each server still needs websockets to run as it will error and say Pusher server couldn't be reached or something similar to that.

@AugmentBLU I understand that, but I got the feeling that the latest beta (22, or any subsequent updates) would also have removed the need for such a setup i.e. for the LB to push all /app requests to the main server. I hope I'm not misunderstanding anything.
@rennokki I've sent an email with our testing URL to your private e-mail address if you choose to test it on our URL to see the issue.

@SelimSalihovic Yeah it would be nice if it worked that way, hopefully it will but at least there is a partial workaround to at least get things working with Redis and multiple servers. Here's hoping!

Now I see that the NGINX configuration sent here is hardcoded with a single server IP that gets all the requests. However, this is not intended and it creates too much pressure on one server that will resolve all the incoming requests to that /app route.
The proper solution is to let all the servers use the same NGINX configuration that proxies the /app route to the WebSockets server so that the Load Balancer evenly distributes all the incoming connections and requests across servers while the Redis Pub/Sub does its thing. I doubt that this has already been tried and I was talking into the void here.

I have tried using setting my LB as standard, no /app request config. Each server's /app request is pushed to it's own WS server proxy_pass http://<IP_ADDRESS>:8444; but we hit this issue that broadcasts are not being sent to all users on all servers, instead it's intermittent and only received by those on the same server the broadcast is emitted from, same issue as @SelimSalihovic has witnessed.
That's why the LB was configured to route all requests to a single server, but like you said will cause pressure on that single server - however it's the only way around this issue so far.
I've still to test the latest beta version on this setup though, probably won't be till later today now as I need to sort something else.

In your examples, are you using the same Redis instance?

Yes, I have a dedicated server for Redis that each app server connects to. I have Redis running for all session/cache/queue for Laravel and works great so it is setup correctly. It also worked flawlessly with Echo Server when I was using that. I will test in a couple hours and let you know.

I've just tested and still got issues with the broadcasting side.
Both app servers Nginx config includes
    location /app {
        proxy_pass             http://<IP_ADDRESS>:8444;
        proxy_set_header Host  $host;
        proxy_read_timeout     60;
        proxy_connect_timeout  60;
        proxy_redirect         off;

        # Allow the use of websockets
        proxy_http_version 1.1;
        proxy_set_header Upgrade $http_upgrade;
        proxy_set_header Connection 'upgrade';
        proxy_set_header Host $host;
        proxy_cache_bypass $http_upgrade;
    }

Load balancer is standard Forge Nginx load balancer, set to round robin.
When 2 different users are logged in using 2 browsers, things are working as expected. One refreshes the page, the .leaving() and .joining() events trigger as expected on the other's browser, however when a button is clicked that triggers a broadcast event, it's still not consistently being sent to all users, no matter which server they are currently connected to.
Multiple clicks of the button, triggering the broadcast event multiple time eventually gets through but sometimes it's duplicated, sometimes it doesn't appear. So back to routing all /app requests at the load balancer level to a single app server.
I tested using IP Hash which is sticky session and it's sending the broadcast as expected to all those connected to the same server, however I used TOR browser for another browser connection to connect with a different IP, managed to land on a different server than the other connected users and the broadcast is duplicated so notifications are being received twice every time.
I go back to routing all /app requests at the LB level, change each app server no /app request config and works again but of course it's back to the single server dealing with all broadcasts again. I really don't see any other way to fix this. The general here, joining, leaving events are working but Laravel Broadcasts are not.

@AugmentBLU So I found out that the double-messages do happen when the messages are being sent out on the node that has channels. Basically, the message didn't get streamed further because it wasn't streamed at all. If there were users locally, they would just send to them and just that, without streaming them further. The latest 2.x-dev should make its job, I'll tag a release this noon after I come back from the gym. Here's the commit that should solve everything up: https://github.com/beyondcode/laravel-websockets/pull/447/commits/7519da4a08f062e9983a4e3e1f698b8e4e8ca83d

I just tested 7519da4a08f062e9983a4e3e1f698b8e4e8ca83d on my Forge setup and seems to be working pretty well. I have both app servers running /app requests to their own local hosted websocket server and the broadcasts are being made across all servers, without duplication. I have some further testing to do but seems good so far, great work.

I have been able to replicate the same success but I have some remarks. I also ran it with routing /app requests to their own local hosted websocket server.

For this to work I also had to enable session stickiness in the round robin setup (I had previously disabled it). I wonder if I even need to route /app requests to their own server if session stickiness is enabled.
I also noticed that the users list is always empty for me, on both browsers. It used to be populated on beta22.

Either way, the broadcasts work without flaws. Thanks for the hard work so far.

@SelimSalihovic Session stickiness is a must on WebSockets when horizontally, making sure to stick to the same node when working back-and-forth, thus using the same active connection instead of closing it and opening it on other LB.
I have noticed that the users list acts funny, so I'm going to investigate this.

Released -beta.23 that contains the docs on changes & fixes the nasty bugs. Looking into the users list issue rn.

Just FYI, when I upgrade to -beta.24 no messages get sent through. On -beta.23 this works as expected.

@SelimSalihovic Presence channels?

I'm not sure I follow but hopefully this answers it. The list of users online is empty but I see the connections being opened in the log files.

You said the messages dont get sent through, meaning it doesnt broadcast them?

I use Laravel Echo with vueJS to create a list of users online within a presence channel. Everything seems to work with -beta.24 but I have only checked locally, single server using Redis. Broadcasts are working as well, will check multi-server setup later.
I did have to clear my Redis DB after update but that's expected.

@rennokki they don't seem to be broadcasted, no. I was testing your demo application with a 2 server setup with a LB in front. I didn't clear the redis db after updating. Would you say that's mandatory?

@AugmentBLU It's not mandatory as the obsolete connections data get flushed automatically if the connections do not pong in 120 seconds (according to the Pusher docs), which obviously they won't if you close and re-open the server.
@SelimSalihovic The app has not been updated with the latest version 🤔 But I'm going to try it myself within the app. I have pushed to production the latest version and the broadcasting works as it should, even with dozen of concurrent users (like 141 or so)

@rennokki I did manually update the demo app on both servers to use beta 24 and then tested it. Again, as soon as I downgrade to beta 23 on both servers the broadcasts start working.

I just did another test and it appears to be working on beta 24 for both the demo app and our application. I'm not sure what made it to not work 2 days ago but just wanted to let you know it does. The users online list also works for me now.

I've noticed that users are appearing in Redis days after last being connected meaning they show up as online. Shouldn't this be self flushing after X amount of time?

@AugmentBLU @SelimSalihovic @rennokki I'm quite curious how this shakes out because it would be a game-changer to run load-balanced websocket servers and not have to even worry about laravel echo infrastructure. Any more updates from the stakeholders involved, production stories, etc.?

nvm, I see this is a focal point of the refactor. Thanks for all the hard work :smiley:


bug: gps.locate() doesn't return anything

Describe the bug
When using gps.locate() inside a thread, it wont return the gps coordinates
Expected behavior
Returning the gps values inside a thread.
[ x] I am running the latest version.

Guess the latest update should be a fix for this problem. I will keep it open a couple days.
c0fab23


[WIP] Make directory for graphical test artifacts

Make CALayer.renderToPDF(at:) not iOS-coupled
Create a GraphicsTestCase for dealing with the creation temporary files

Codecov Report

Merging #105 into master will increase coverage by 0.1%.
The diff coverage is 98.11%.

@@            Coverage Diff            @@
##           master     #105     +/-   ##
=========================================
+ Coverage   83.41%   83.52%   +0.1%
=========================================
  Files          52       52
  Lines        1815     1827     +12
=========================================
+ Hits         1514     1526     +12
  Misses        301      301

Impacted Files
Coverage Δ

GraphicsToolsTests/TextTests.swift
0% <0%> (ø)
:arrow_up:

GraphicsTools/StyledPathTests.swift
100% <100%> (ø)
:arrow_up:

GraphicsToolsTests/SVGTests.swift
89.74% <100%> (ø)
:arrow_up:

GraphicsToolsTests/TextBoxTests.swift
100% <100%> (ø)
:arrow_up:

GraphicsTools/CALayer+PDF.swift
100% <100%> (ø)
:arrow_up:

GraphicsToolsTests/GraphicsTestCase.swift
100% <100%> (ø)

GraphicsToolsTests/CompositeTests.swift
100% <100%> (ø)
:arrow_up:

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 919b6a7...341cb61. Read the comment docs.


Excalibrain doesn't work on my mac because of "TypeError: t.values is not a function"

Excalibrain doesn't work on my mac.
Excalidraw works normal, Excalibrain shows only start page. selected files on other panes or search will be recognized (Footer of Excalibrain drawing), but not showed. The drawing only shows "To get started":

The Console shows error:

Code for Line "3964":

Errors from startup
(1)
The first error after Startup is ExcaliBrain URLParser: Invalid URL https://qisfsv.xxxxxx with "request failed"

This is an Intranet URL here
URL is sometimes reachable (if I am inside firm VPN) and sometimes not --> normal behaviour

(2)
"Initialized Excalidraw Image Cache"
(3)
"ExcaliBrain indexed 6036 URLs from 1765 unique hosts in 2325 of 3990 markdown files in 6.4 seconds"
(4)
"Uncaught (in promise) TypeError: t.values is not a function"

Versions:
Obsidian Version 1.5.3 (Installer 1.4.13)
ExcaliBrain 0.2.11
ExcaliDraw 2.0.13

can I ask you to replace your main.js with this one?
main.zip
Download and unpack the zip
Copy the main.js to Vault/.obsidian/plugins/excalibrain folder
note .obsidian is a hidden folder.
restart Obsidian and share the results from the console.

Thank You for the file. I've replaced the file and this is the console output:

Let's do a second try. Here's a new main.js:
main.zip
On the screenshot please open the toggles as shown:

This is because you are using a form of properties on the page that I wasn't aware of and seems dataview for those properties works differently. Can you send me the markdown file that is causing the errors? Hopefully based on the detailed log you can figure out which file it might be.

Thanks. Fixed it. I've also added support for date fields. Will release soon.


*: bump Go version to 1.13

@simonpasquier @krasi-georgiev  won't we need a change here too? :
https://github.com/prometheus/prombench/blob/8fa36bf45520bfca6a42ba7b850750428c01a4a8/prombench/temp/prometheus-builder/Dockerfile#L1

yep defensively.

Just saw prometheus/prometheus is still using 1.12 so maybe not?
https://github.com/prometheus/prometheus/blob/master/.promu.yml

@geekodour once https://github.com/prometheus/prometheus/pull/6020 is merged, yes you would need to bump to 1.13 there too.


Makefile: specify dependencies for vfsgen. Add iop target.

Specify dependencies for vfsgen. Add iop target. Also update the Readme.
Having this PR out to get more ideas of how to improve the Makefile.
Resolve: https://github.com/istio/istio/issues/15237

Hi @elfinhe. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test


[Feature]: Ozone endpoints

Is your feature request related to a problem?
This is mainly a question but could work as a feature request. I will need to consume ozone endpoints. I understand they have not yet been implemented in the SDK. This means I will have to implement them on my own (buh... work!).
Describe the solution you'd like
Ideally, the SDK would implement these endpoints.
Describe alternatives you've considered
I'm wondering if there is anything I could reuse from the SDK or if I should just fall back completely to raw HTTP requests. Thanks for your support.
Priority
Low
Implementation ideas
No response
Additional context
No response

Thank you for your suggestion, @pupi1985.
Currently, my focus is on implementing only the most critical endpoints, as the SDK is still in its beta stage, and the API interface has not yet reached full stability. I am actively working on the 2.x/stable version to address these challenges.
If you're interested in contributing, a PR for these endpoints targeting the 2.x version would be greatly appreciated. However, my immediate priority is not adding new endpoints or features but refactoring existing issues to create a more maintainable codebase.
Current Issues and Challenges:

Traceability is in a poor state (see issue: #32).
Adding new endpoints, especially implementing endpoint-specific casting components, is unnecessarily complex.
The SDK's reliance on inheritance over composition has introduced difficulties in extending functionality.

Potential Improvements:

Adopting PSR standards to align with modern development practices.
Enabling "strict types" to enhance stability and reduce errors.
General enhancements to improve maintainability and extensibility.

Thank you for your understanding, and I welcome your contributions or further feedback!


Poedit goes blank on Update

Windows 8.1 (64bit)
Poedit 1.8.5
When I click "Update" the translation sections just disappears:

Find .po file in "translations" folder:
https://mega.nz/#!B0NEQbCJ!V1-00on4pUj7tAvDOmVQqPYn2uR__glVNflgrD5prY8

You actually have to configure source code extraction to find strings to extract. If you don't have any translatable strings, Poedit naturally won't find any. See the GNU gettext manual.
If you still believe there's a bug in Poedit, please provide a reproducible test case with an explanation of why you think it's misbehaving (please read http://www.chiark.greenend.org.uk/~sgtatham/bugs.html too).


Streaming for chat completion

It would be great to add chat streaming to aviod waiting for the entire answer to finish before it is visible.

Hi @iranogajec
This has been added to v1.6.8


Unlink libjpeg from Android builds

Based on Issue #522, (Fixes #522)
Android does not seem to rely on libjpeg from my research, only seems to use BitmapUtils which are primarily used for Linux and Emscripten(html)

It may be necessary to modify this file?
/platform/android/ndk/CMakeLists.txt

We’re using CMake for Android. Mk files can be deleted afaik.
Here is the main file https://github.com/coronalabs/corona/blob/master/platform/android/sdk/CMakeLists.txt
And here are where supplementary file with source file list has reference to libjpeg
And a
https://github.com/coronalabs/corona/blob/648b1dde2e3bbd90197cc37c8ede27b1db498485/platform/android/ndk/CMakeLists.txt#L728

I pushed commit with required changes. It may I ask you to test it?

 Seems appropriate, they are of no consequence.

my testcase:

build (✔) use platform/android project

display.newImage(gXXX,"1.jpg") (✔)

display.save(gXXX,"1.jpg") (✔)

I decided to delete bunch of files... Seems appropriate, they are of no consequence.
these?
xxxx.vcxproj
xxxx.vcxproj.filters
in platform/android/sdk/...

Oh... one of our developers really into Visual Studio. So just ignore those.


Cannot interrupt execution

When I click on the Interrupt button, nothing happens and batch generation goes on. No error messages are shown. I need to stop the process in the console and reload to stop it.
I'm using tunnel.

Try clicking many times until it stops

same issue here, I even clicked several times it didn't stop or skip

that's the side effect of using the "queue" to avoid images not showing in the UI

how can I fix it

if you fix it you won't be able to generate batches

I don't need generate batches I'm generating 1 for example, I'm testing a prompt but I didn't like so I want to interrupt or skip, but I can't. How can I?

in the A1111 colab, remove this line:
!sed -i 's@ui.create_ui().*@ui.create_ui();shared.demo.queue(concurrency_count=999999,status_update_rate=0.1)@' /content/gdrive/MyDrive/sd/stable-diffusion-webui/webui.py

When I click on the Interrupt button, nothing happens and batch generation goes on. No error messages are shown. I need to stop the process in the console and reload to stop it. I'm using tunnel.

Hey, I fixed it by removing all my files and reinstalling sd. Now I can interrupt 👍 thanks for help for all

When I click on the Interrupt button, nothing happens and batch generation goes on. No error messages are shown. I need to stop the process in the console and reload to stop it. I'm using tunnel.

Hey, I fixed it by removing all my files and reinstalling sd. Now I can interrupt 👍 thanks for help for all

yeah, fresh install solved the problem


Add support of gpt4free!

Hello, it's a feature request and a call for help also.
I cant find a place where the api call is made, so that i could replace it with my own POST request. If you could guide to this place it would help a lot.
I know its unethical, but i'm from russia and have no 'legal' access to OpenAI API anyway.
Please help, i really like the framework and want to use it with g4f library, i would totally understand if you don't want to associate with it though.

You should be able to use it with it already
pip install --upgrade --quiet  gpt4all > /dev/null

from langchain_community.llms import GPT4All

local_path = (
    "./models/ggml-gpt4all-l13b-snoozy.bin"  # replace with your desired local file path
)
llm = GPT4All(model=local_path, verbose=True)

# Now you can pass this to your agent definition
Agent(
#...
llm=llm
)

# rest of your code

Thank you!
You rock, dude!

@pencilvania
This should work:
class G4FLanguageModel(LLM):
    model_name: str = "gpt-3.5-turbo"

    def _call(self, prompt: str, stop: Optional[List[str]] = None) -> str:
        response = g4f.ChatCompletion.create(
            model=self.model_name,
            messages=[{"role": "user", "content": prompt}]
        )
        return response

    @property
    def _identifying_params(self) -> dict[str, Any]:
        return {"model_name": self.model_name}

    @property
    def _llm_type(self) -> str:
        return "g4f"

g4f_model = G4FLanguageModel()

agent = Agent(
    llm=g4f_model
)


:refactor: Add code improvement to work with any Oak version

Can I have a code review for these changes? 🙂

Looks good. Thanks!


Issues with retaining index data between restarts

I've been trying out lnx and it's great. However I noticed my index documents are not retained between restarts. I can see that the index configuration itself is restored on restart but the index contains no documents. I noticed that the tantivy files all seem to be getting written to a tmp directory that looks like /var/folders/r6/y3v948hd5xqcv_b8f_z38xm40000gn/T/.tmpZPbQT1/ which gets wiped when I shut the server down.
Is there a configuration change or command line option that I need to pass to change this behavior so my documents get retained?

What is your index config? Are you sure you are not using a temporary index?

@ChillFish8 can you assist? I'm just using the default cargo run and ingesting the demo data from the cli tool

Hi @ptming Sorry for the delay, the demo tool uses a pre-configured index which uses a temporary directory because it is designed to be spun up and down without worry. I would recommend manually creating an index with configurations specific to your use case. https://docs.lnx.rs/#tag/Managing-indexes

@ChillFish8 oooooh thank you, i see the storage_type values. Please disregard and thank you for your guidance


probe.gl based logging

Background
For #1516
Change List

Use probe.gl Log instance instead of custom log
Double invocation of log functions to make probe.gl link back to actual source

What's the purpose of double invocation?

What's the purpose of double invocation?

https://uber-web.github.io/probe.gl/#/documentation/articles/console-logging


chore: change envVars to env in func.yaml

There are some places where I've changed variable and function names
where it wasn't strictly necessary. If you don't mind the bit of churn
that results, changing these makes rg -i envvars a nice way to check
for anything that could be overlooked.
Signed-off-by: Lance Ball<EMAIL_ADDRESS>

I should have noted that this change was prompted by the discussion in https://github.com/boson-project/func/pull/312#discussion_r618946318.

Nit: imho this shouldn't be a chore type of PR, since we are introducing the breaking changes.

Nit: imho this shouldn't be a chore type of PR, since we are introducing the breaking changes.

Good point!

@zroubalik how about refactor! - adding the exclamation point to a prefix indicates to release-please that there is a breaking change.


docs: add simple example of event being emitted

I missed this while looking at this package, a very simple example emitting and listening to an event on the usage part would be nice.

See https://github.com/primus/eventemitter3#usage. There is a link there (https://nodejs.org/api/events.html) with a lot of examples.

Let me rephrase it. What I can see when I read the README is just that:
var EventEmitter = require('eventemitter3');

What about adding something like this? Is that the way we should be doing?
var EventEmitter = require('eventemitter3');

class MyEmitter extends EventEmitter {}

const myEmitter = new MyEmitter();
myEmitter.on('event', () => {
  console.log('an event occurred!');
});
myEmitter.emit('event');

You have that example here https://nodejs.org/api/events.html#emitteroneventname-listener. It is only 2 click away.

Yes the link is correct, try to open in different browser (it also does not work for me on Chrome).

Yep that was it, opening it on Safari worked. So my question was more about how the maintainers recommend the users to use it. In most open source projects there is are examples on the README or an examples directory containing some common use case, and I think that helps users to get started. This issue can be closed.

Yes, I understand but this module works like the Node.js EventEmitter so there is no reason to duplicate examples. This is documented.

For the API documentation, please follow the official Node.js documentation: http://nodejs.org/api/events.html

I'm closing this as answered.


ObliqueMC.com website issues.

This is a post that I'll be updating as I find issues, but since there are two issues I know of that hasn't been fixed despite being there for well over a year now, I think it's about time that I made an actual report about it.

Social icon for "Github" leads to a dead link ("https://github.com/ObliqueNetwork") (The Server -> Support link is correct, however)
Social icon for Steam group leads to an active link, but the staff of that group obviously hasn't been maintained in a long time...
The Terra Applications page needs some updating... Admiral is still listed as "Gym Lead", Elemental as "lead mod", Ekul as wizard lead... you get the picture.

Not rushing for any of these to be fixed, but as far as I could find, there wasn't any mention of it elsewhere, so here is a place that won't vanish :D

Updated our Staff page to reflect Lead changes, namely DoughnutChicken, Katadreme, Ooomaloo, https://www.obliquemc.com/staff.
Updated our homepage to reflect our single-running server, https://www.obliquemc.com/.
Updated our Map on site so that Mia doesn't look incompetent, https://www.obliquemc.com/map.
Updated our Team page to reflect @Elemental 's addition to Manager, https://www.obliquemc.com/team.
Updated our Wiki https://www.obliquemc.com/wiki page, now comfortably directing back to https://github.com/ObliqueNET/Server/wiki.
Updated our Support https://www.obliquemc.com/support page, now comfortably redirecting to https://github.com/ObliqueNET/Server/issues.
Cleaned up some awkward transitions in our submenu, and some belated mentions of Terra and Stratos.
Corrected a typo in our Enigma overlay description - you get 3 shiny ups, not 2 https://www.obliquemc.com/store.
Fixed our Steam Community button and it's availability. It is now a public group. Join here: https://steamcommunity.com/groups/obliquenetwork . It still needs some setup, and I'll be adding staff members as we move along. Old Steam group was sadly deprecated, and it was a staff initiative from a person now retired.
Updated our FREE STARTER POKÉDISGUISE pack to include Generation 7 and 8 starter Pokémon evolutionary lines.

Updated our Staff page to reflect Lead changes, namely DoughnutChicken, Katadreme, Ooomaloo, https://www.obliquemc.com/staff.
Updated our homepage to reflect our single-running server, https://www.obliquemc.com/.
Updated our Map on site so that Mia doesn't look incompetent, https://www.obliquemc.com/map.
Updated our Team page to reflect @Elemental 's addition to Manager, https://www.obliquemc.com/team.
Updated our Wiki https://www.obliquemc.com/wiki page, now comfortably directing back to https://github.com/ObliqueNET/Server/wiki.
Updated our Support https://www.obliquemc.com/support page, now comfortably redirecting to https://github.com/ObliqueNET/Server/issues.
Cleaned up some awkward transitions in our submenu, and some belated mentions of Terra and Stratos.
Corrected a typo in our Enigma overlay description - you get 3 shiny ups, not 2 https://www.obliquemc.com/store.
Fixed our Steam Community button and it's availability. It is now a public group. Join here: https://steamcommunity.com/groups/obliquenetwork . It still needs some setup, and I'll be adding staff members as we move along. Old Steam group was sadly deprecated, and it was a staff initiative from a person now retired.
Updated our FREE STARTER POKÉDISGUISE pack to include Generation 7 and 8 starter Pokémon evolutionary lines.


Roll Plugins from 160c714e75f3 to f7d26e94df0b (19 revisions)

https://github.com/flutter/plugins/compare/160c714e75f3...f7d26e94df0b
2022-05-02<EMAIL_ADDRESS>Roll Flutter from d600a3b48801 to f7df22590adf (1 revision) (flutter/plugins#5466)
2022-05-01<EMAIL_ADDRESS>Roll Flutter from a35af5b4659a to d600a3b48801 (1 revision) (flutter/plugins#5465)
2022-05-01<EMAIL_ADDRESS>Roll Flutter from e66662f1cb75 to a35af5b4659a (1 revision) (flutter/plugins#5464)
2022-05-01<EMAIL_ADDRESS>Roll Flutter from e3d39f62a26b to e66662f1cb75 (1 revision) (flutter/plugins#5463)
2022-05-01<EMAIL_ADDRESS>Roll Flutter from 0f1eb8bad788 to e3d39f62a26b (1 revision) (flutter/plugins#5462)
2022-05-01<EMAIL_ADDRESS>Roll Flutter from 43555d05040c to 0f1eb8bad788 (2 revisions) (flutter/plugins#5461)
2022-05-01<EMAIL_ADDRESS>Roll Flutter from 6755ba59f436 to 43555d05040c (1 revision) (flutter/plugins#5459)
2022-05-01<EMAIL_ADDRESS>Roll Flutter from 7b8d50020338 to 6755ba59f436 (1 revision) (flutter/plugins#5458)
2022-04-30<EMAIL_ADDRESS>Roll Flutter from 2a24ee494db4 to 7b8d50020338 (1 revision) (flutter/plugins#5457)
2022-04-30<EMAIL_ADDRESS>[webview_flutter_wkwebview] Implementation of Objective-C WKWebView Host Apis  (flutter/plugins#5341)
2022-04-30<EMAIL_ADDRESS>Roll Flutter from b0d5d8ab2930 to 2a24ee494db4 (2 revisions) (flutter/plugins#5456)
2022-04-30<EMAIL_ADDRESS>Roll Flutter from 84e099d5f467 to b0d5d8ab2930 (1 revision) (flutter/plugins#5454)
2022-04-30<EMAIL_ADDRESS>Roll Flutter from fa519eb22fe3 to 84e099d5f467 (1 revision) (flutter/plugins#5452)
2022-04-30<EMAIL_ADDRESS>Roll Flutter from 0b3c5d1228f1 to fa519eb22fe3 (15 revisions) (flutter/plugins#5451)
2022-04-30<EMAIL_ADDRESS>Roll Flutter from 3fe2cbabc421 to 0b3c5d1228f1 (7 revisions) (flutter/plugins#5449)
2022-04-29<EMAIL_ADDRESS>Roll Flutter from ad1d30017e69 to 3fe2cbabc421 (2 revisions) (flutter/plugins#5446)
2022-04-29<EMAIL_ADDRESS>[webview_flutter] fix warning (flutter/plugins#5131)
2022-04-29<EMAIL_ADDRESS>Roll Flutter from 2eed8cbf9370 to ad1d30017e69 (1 revision) (flutter/plugins#5445)
2022-04-29<EMAIL_ADDRESS>[ci] Don't run Cirrus tests on master (flutter/plugins#5444)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


ONNX export XLM-RoBERTa

I have tried exporting the xlmr.base model from torch.hub to onnx, but I receive an exception regarding ATen.
  %output : Float(1, 5, 768) = onnx::Transpose[perm=[1, 0, 2]](%1188), scope: Exported/RobertaModel[model]/RobertaEncoder[decoder] # /home/ppatrik/.cache/torch/hub/pytorch_fairseq_master/fairseq/models/roberta/model.py:367:0
  return (%output)

Warning: ATen was a removed experimental ops. In the future, we may directly reject this operator. Please update your model as soon as possible.
Traceback (most recent call last):
  File "/home/ppatrik/.conda/envs/condaenv/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/ppatrik/.conda/envs/condaenv/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/mnt/permanent/home/ppatrik/entity-recognition/src/export.py", line 168, in <module>
    main()
  File "/mnt/permanent/home/ppatrik/entity-recognition/src/export.py", line 152, in main
    output_path)
  File "/home/ppatrik/.conda/envs/condaenv/lib/python3.7/site-packages/onnxruntime/capi/session.py", line 25, in __init__
    self._load_model(providers)
  File "/home/ppatrik/.conda/envs/condaenv/lib/python3.7/site-packages/onnxruntime/capi/session.py", line 43, in _load_model
    self._sess.load_model(providers)
onnxruntime.capi.onnxruntime_pybind11_state.Fail: [ONNXRuntimeError] : 1 : FAIL : Fatal error: ATen is not a registered function/op

I met the same problem, have you solved it?

Unfortunately I could not. If a recall correctly an error originated from the cumsum operation (which is included in onnx 1.5 with opset 11) in https://github.com/pytorch/fairseq/blob/9f25ffb02ab54a443ba181f732d5dcc00db7bea8/fairseq/utils.py#L192 . However I might have fixed that and one more error persisted so I gave up and implemented my own fairseq model (Not sure if I fixed it and still got an error or just didn't bother and went straight to implementing my own). https://github.com/Mrpatekful/xlmr-finetuning I test the outputs of the two version and they are identical with atol=1e-5, but mine could be exported without and error with onnx>=1.5 .


User Agent setting ignored

Describe the bug
Despite User Agent set in settings, output is ignored:
eg: set to "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/17.5 Safari/605.1.15" in settings:

but log shows

"Error opening input file https://c.mjh.nz/abc-vic.m3u8|user-agent=Mozilla/5.0 (Windows NT 10.0%3B Win64%3B x64) AppleWebKit/537.36 (KHTML%2C like Gecko) Chrome/98.0.4758.102 Safari/537.36&seekable=0&referer= ."
Server (please complete the following information):

OS: macOS 15
Arch: Intel 64 Bit
Threadfin Version: 1.2 Build: 9

To Reproduce
Steps to reproduce the behaviour:

Modify User Agent in Threadfin Settings & save
Attempt to stream program via Plex

Expected behavior
Stream program
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: macOS 15
Browsers: Firefox - also with Windows on Edge & Apple TV client
Versions: Various

Additional context
Add any other context about the problem here.

That isnt a valid URL. That is not how you do User Agent. Why is the user agent appearing in your URL like that? Also dont use HLS, use mpegts

Hi - thanks for your reply.
It is a valid URL from https://i.mjh.nz/au/Melbourne/
It works in many apps that stream IPTV from that source: IPTVnator (cross platform - open source) & IPTV FTW (MS Windows App Store).
It has worked in threadfin/xteve before - up until updates a month or so ago (apologies - I did not note the date).
Please explain "That is not how you do User Agent", as that is a valid user agent for browser spoofing.
Also, please explain your comment re: "don't use HLS, use mpegts".
Thanks in advance.


wiki: sample proof-general keybindings

partly blocked by #54

done: https://github.com/siegebell/vscoq/wiki/Keybindings


Install fails

PS C:\Users\tmk> Install-Package WPFTouchKeyboard
Install-Package : No match was found for the specified search criteria and package name 'WPFTouchKeyboard'.

Hi,
It should be ok, here the link at nuget website https://www.nuget.org/packages/WPFTouchKeyboard

Sorry, I was in the regular Powershell. I should be in the Nuget console. It's under Tools --> NuGet Package Manager.


Setup Orchard fork of garcon to be on Orchar pypi server

The package will be named owsgarcon on our pypi server, but the import structure remains garcon. This avoids a conflict when installing the package but makes switching between the official xethorn package and the orchard pypi package easy, and will help expedite us fixing the SWF throttles.

@nsnyder3 info about our pypi server: https://github.com/theorchard/docs/tree/master/boilerplates/pypi
https://pypi.theorchard.io/pypi/


fix: Fix fqid conflicts in the Code Objects view

Fix a class map merging problem that caused duplicate ids in the Code Objects view.
This is caused by the behavior of appmap-ruby to emit package names like some/package, rather than splitting these into a parent and child.

Yeah good idea.
On Thu, Jun 30, 2022 at 9:56 AM Alan Potter @.***>
wrote:

@.**** commented on this pull request.
In src/lib/CodeObjectEntry.ts
https://github.com/applandinc/vscode-appland/pull/421#discussion_r911060055
:

@@ -0,0 +1,208 @@
+import * as vscode from 'vscode';

Ok, thanks. FWIW, I've been trying to use refactor commits to move code
around, then separate fix or feat commits when I make changes.
—
Reply to this email directly, view it on GitHub
https://github.com/applandinc/vscode-appland/pull/421#discussion_r911060055,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC65WWKBR4T2WVM7BWELVRWRRTANCNFSM52DMM2HQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

@apotterri I've made some changes and all the extension tests are passing locally.

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

v0.32.1
GitHub release

Your semantic-release bot :package::rocket:


[semantic-UI]   where  have  layout   border  ?

[semantic-UI]   where  have  layout   border  ?

I dont understand the question.


Add KeyboardEvent.which compatibility data

Testcase: http://jsbin.com/pugiqa

@Fyrd Data review complete: http://caniuse.com/#feat=keyboardevent-which


GPC_4T - move the AliveTest to the correct dir and please also commit the .s, .elf elf_.txt

move the AliveTest to the correct dir and please also commit the .s, .elf elf_.txt
see your commit 369087db3ab720637d71dbe7c56711c2d2feab6f

If I understand the scripts correctly, those files are generated based on the C source file. Is that correct? And if that's indeed the case, why are they in the repository in the first place? As a rule of thumb, it's a bad idea to keep auto-generated files/build outcomes in the repository, because you'll easily find yourself with inconsistent copies of the same thing.
The best approach is to keep only the source files and build scripts in Git, so only one version is maintained

You are correct - the ".s", ".elf", "elf_.txt" ".sv" are all generated based on the C source file.
In most cases, I agree with the rule of thumb "bad idea to keep auto-generated files/build outcomes in the repository" -
But In this case, I want us to "commit" the generated files.
This is due to some contributors (mainly from the HW part) don't have the RISCV toolchain.
Committing the files the will allow anyone with a SystemVerilog executing/simulation env to test and run the programs we already compiled, Link, and turned into SystemVerilog memory format.
Whenever anyone makes changes in the "C" code they must fun the compile and generate scripts - and commit all changes.

Files were moved to the alive dir (under apps). for some reason alive/ dir name won't change to tests/


Is there a way to state +above(tablet)?

I often end up duplicating rules for +tablet() and +desktop(), as +tablet is between it's lower and higher limits rather than above it's lower limit. What would be the best way around this? I love the functionality otherwise.

@permadesign Not sure I understood correctly, but the lowest cutoff for tablet is mobile, so you could do this:
+above(rupture.mobile-cutoff)
// OR
+above('s')

Which basically means "apply these styles if the viewport is above the largest mobile breakpoint/smallest tablet breakpoint"

That's perfect, thanks!


Add pcl cmake example

This take's @EricCousineau-TRI's PCL example from #37 and gets it working with CMake using the installed Drake.
After build and run with
$ cd drake_cmake_installed
$ mkdir build && cd build
$ cmake -Ddrake_DIR=/opt/drake/lib/cmake/drake ..
$ make
$ src/pcl/simple_pcl_example

The output is
PointCloud before filtering: 100000
PointCloud after filtering: 99403

Without resorting to some imaginative history transformation I couldn't readily preserve @EricCousineau-TRI's actual commits but I have no problem with this being rewritten to reflect proper authorship.
I did have to make some changes small changes to the source itself to build against current Drake but as long as they are not incorrect they are inconsequential.

This change is

I also forgot to mention that for this example to build you either need Drake combining https://github.com/RobotLocomotion/drake/pull/7434 and https://github.com/RobotLocomotion/drake/pull/7435 or this branch which combines them both.

Added some additional documentation. @siyuanfeng-tri could you test and review please?

taking a look.

Works as intended with pcl 1.7 that's shipped with ubuntu 16.04.
I have actually had pcl 1.8 compiled and install from source to /usr/local. In which case, this was able to build and link fine, but the example segfaults somewhere in vtk land. I am guessing that's probably caused by the vtk issued mentioned in the readme.
After purging 1.8, I was able to run the example successfully.

Looking into it, it looks like CI is failing because ${PCL_LIBRARIES} contains a reference to vtkproj4, which depends on the package libproj-dev (where the error /usr/lib/x86_64-linux-gnu/libproj.so comes in) which seems to have been a dependency of vtk5.10 (when using locate libvtkproj4.so on my machine).
It looks like it might just be an apt configuration bug in drake-shambhala. It seems like we need to ensure that we use libvtk6-dev from OSRF, not libvtk6-dev from nominal sources.
On my machine and in the docker image, I ran dpkg -s libvtk6-dev. The difference:
diff --git a/./docker.txt b/./host.txt
index 81c9924..078b7d9 100644
--- a/./docker.txt
+++ b/./host.txt
@@ -2,13 +2,13 @@ Package: libvtk6-dev
Status: install ok installed
Priority: optional
Section: libdevel
Installed-Size: [-25854-]{+25887+}
Maintainer: [-Ubuntu Developers<EMAIL_ADDRESS>Science Team<EMAIL_ADDRESS>Architecture: amd64
Source: vtk6
Version: [-6.2.0+dfsg1-10build1-]{+6.2.0+dfsg1-10build1+debian11.1+osrf1+}
Replaces: libvtk5-dev
Depends: libavcodec-dev, libavformat-dev, libavutil-dev, libc6-dev, libeigen3-dev, libexpat-dev, libfreetype6-dev, libgdal-dev, libgl1-mesa-dev | libgl-dev, libgl2ps-dev, libglu1-mesa-dev | libglu-dev, {+libhdf5-mpi-dev,+} libjpeg-dev, libjsoncpp-dev, libmysqlclient-dev, libnetcdf-dev, libnetcdf-cxx-legacy-dev, libogg-dev, libpng-dev, libpq-dev, libpython-dev, libswscale-dev, libtheora-dev, libtiff-dev, libvtk6.2 (= [-6.2.0+dfsg1-10build1),-]{+6.2.0+dfsg1-10build1+debian11.1+osrf1),+} libvtk6-java (= [-6.2.0+dfsg1-10build1),-]{+6.2.0+dfsg1-10build1+debian11.1+osrf1),+} libx11-dev,[-libxdmf-dev,-] libxft-dev, libxml2-dev, libxss-dev, libxt-dev, mpi-default-dev, python-vtk6 (= [-6.2.0+dfsg1-10build1),-]{+6.2.0+dfsg1-10build1+debian11.1+osrf1),+} python-zope.interface, tcl-dev, tcl-vtk6 (= [-6.2.0+dfsg1-10build1),-]{+6.2.0+dfsg1-10build1+debian11.1+osrf1),+} tk-dev, vtk6 (= [-6.2.0+dfsg1-10build1),-]{+6.2.0+dfsg1-10build1+debian11.1+osrf1),+} x11proto-core-dev, zlib1g-dev
Suggests: vtk6-doc, vtk6-examples
Conflicts: libvtk5-dev
Description: VTK header files
@@ -17,5 +17,5 @@ Description: VTK header files
 .
 This package provides the VTK header files required to compile
 C++ programs that use VTK to do 3D visualisation.
[-Original-Maintainer: Debian Science Team<EMAIL_ADDRESS>Homepage: http://www.vtk.org/
{+/usr/lib/x86_64-linux-gnu/libvtkproj4-6.2.so+}

Yup, just adding thing ROS packages fixed things. Just added to install_prereqs:
echo "deb http://packages.ros.org/ros/ubuntu $(lsb_release -sc) main" > /etc/apt/sources.list.d/ros-latest.list
apt-key adv --keyserver hkp://ha.pool.sks-keyservers.net:80 --recv-key 421C365BD9FF1F717815A3895523BAEEB01FA116

It's worth noting that it wasn't my intention to require using libvtk6-dev from the ROS repositories. In case that changes the desired solution.

Ah, I see. Can I ask how the solution will change in that case?

Ah, I see. Can I ask how the solution will change in that case?

I don't actually have a resolution yet. Manually installing libproj-dev together with the ubuntu libvtk6-dev still produces the error linking vtkproj4.

Thanks to @j-rivero found some history regarding the vtk6 package on the ROS buildfarm https://github.com/ros-infrastructure/reprepro-updater/issues/32
Looks like it's there to address issues with the system package and PCL. So maybe we do need it.

The upstream Ubuntu bug is https://bugs.launchpad.net/ubuntu/+source/vtk6/+bug/1573234
From comments there, the simple solution of removing the offending library from the PCL_LIBRARIES list is the most straightforward solution and it will not break use of the ROS-provided backported package. I changed workstations when I flew across the US so I have to rebuild Drake before I can test the hotfix.

I've pushed up a4ad7f0  which should allow the example to work with the upstream or ROS libvtk6-dev package.

Thanks for chasing this down @nuclearsandwich and @EricCousineau-TRI!

drake_cmake_installed/src/pcl/CMakeLists.txt, line 42 at r5 (raw file):

# by removing the missing vtkproj4 library from the list of needed files.
# It's not actually needed for this example.
list(REMOVE_ITEM PCL_LIBRARIES "vtkproj4")

What are the consequences of not linking vtkproj4?

Comments from Reviewable

What are the consequences of not linking vtkproj4?

I don't believe it's used for the example. It's part of the Big List of Hardcoded Libraries that the PCL cmake pulls in. If a drake-consuming application needs that library specifically they can install the one from the ROS repository (or another backport with the same fix applied) and omit the removal.

I just rebased this to resolve the merge conflict with the updated mac install script. I'm not sure why it's Reviewable status changed but I'm pretty sure I don't have the authorization to resolve it.

I have another request, can you try in your executable include #include "drake/common/autodiff.h"
i believe that triggers a static assertion fail. here is what i get.
[ 50%] Building CXX object CMakeFiles/clutter_clearing2.dir/clutter_clearing2.cc.o
In file included from /home/sfeng/drake_bazel_build/include/drake/common/polynomial.h:14:0,
from /home/sfeng/drake_bazel_build/include/drake/common/trajectories/piecewise_polynomial.h:10,
from /home/sfeng/drake_bazel_build/include/drake/manipulation/util/trajectory_utils.h:7,
from /home/sfeng/code/rand_obj_picking/build/include/robot_bridge/robot_bridge_common.h:6,
from /home/sfeng/code/rand_obj_picking/build/include/robot_bridge/motion_primitive.h:3,
from /home/sfeng/code/rand_obj_picking/build/include/robot_bridge/iiwa_controller.h:3,
from /home/sfeng/code/rand_obj_picking/src/clutter_clearing2.cc:1:
/home/sfeng/drake_bazel_build/include/drake/common/autodiff.h:15:1: error: static assertion failed: Drake requires Eigen >= v3.3.3.
static_assert(EIGEN_VERSION_AT_LEAST(3, 3, 3),

Working for me - are you sure you did find_package(drake) before find_package(PCL) as indicated in the README in this PR?

false alarm. i had the order wrong. switched and worked


Investigate www.nuget.org/nuget.exe endpoint

Currently this points to a very old version of nuget.exe (2.8.6) hosted in a blob storage that is not used otherwise. We should either

deprecate the endpoint if it has no use
redirect it to dist

The endpoint still has a lot of usage (13k in the last day). We should immediately redirect it to version 2.8.6 on dist and then consider migrating it to the latest version.

Linking https://github.com/NuGet/NuGetGallery/issues/4559


Java API TermQuery missing an options parameter

The documentation describes the use of a term-option for a term query to distinguish it from a word query
https://docs.marklogic.com/guide/search-dev/structured-query#id_56027
However the TermQuery class does not take an options parameter as the ValueQuery does, in addition, the term() builder methods that are intended to be used in future also do not provide the necessary override methods.
http://docs.marklogic.com/8.0/javadoc/client/com/marklogic/client/query/StructuredQueryBuilder.TermQuery.html

TermQuery(Double weight, String... terms)
Deprecated.
Use the term() builder method of StructuredQueryBuilder and type the object as an instance of the StructuredQueryDefinition interface.

term() builder methods
public StructuredQueryBuilder.TermQuery term(String... terms) {
return new StructuredQueryBuilder.TermQuery((Double)null, terms);
}

public StructuredQueryBuilder.TermQuery term(double weight, String... terms) {
return new StructuredQueryBuilder.TermQuery(weight, terms);
}

value() builder methods with String[] options overrides.
public StructuredQueryDefinition value(StructuredQueryBuilder.TextIndex index, Boolean value) {
return new StructuredQueryBuilder.ValueQuery(index, (StructuredQueryBuilder.FragmentScope)null, (String[])null, (Double)null, new Object[]{value});
}

public StructuredQueryDefinition value(StructuredQueryBuilder.TextIndex index, Number... values) {
return new StructuredQueryBuilder.ValueQuery(index, (StructuredQueryBuilder.FragmentScope)null, (String[])null, (Double)null, values);
}

public StructuredQueryDefinition value(StructuredQueryBuilder.TextIndex index, StructuredQueryBuilder.FragmentScope scope, String[] options, double weight, String... values) {
return new StructuredQueryBuilder.ValueQuery(index, scope, options, weight, values);
}

public StructuredQueryDefinition value(StructuredQueryBuilder.TextIndex index, StructuredQueryBuilder.FragmentScope scope, String[] options, double weight, Boolean value) {
return new StructuredQueryBuilder.ValueQuery(index, scope, options, weight, new Object[]{value});
}

public StructuredQueryDefinition value(StructuredQueryBuilder.TextIndex index, StructuredQueryBuilder.FragmentScope scope, String[] options, double weight, Number... values) {
return new StructuredQueryBuilder.ValueQuery(index, scope, options, weight, values);
}

Is the fact that the TermQuery Class does not take an options parameter an oversight or is there another method the customer should be using?
collapse

Closing this issue to keep Search API stable.


Coverage not reported correctly for too many line hits

Steps to reproduce

Run flutter create myapp
Create a file do_something.dart with the provided code sample
Open widget_test.dart and add the provided code sample
Run test with flutter test --coverage && genhtml coverage/lcov.info -o coverage/html

Expected results
The line if (flag == false) should always be counted as covered, regardless of nTests.
Actual results
For nTests = 10, the line is counted as covered. For nTests = 100000, the line is counted as not covered. The exact value seems to not always be the same but there is some threshold.
Placing the second expect expect(doSomething(false), equals('false')); before the loop leads to the line always being counted as covered.
Code sample
do_something.dart
String doSomething(bool flag) {
  if (flag == true) {
    return 'true';
  }

  if (flag == false) {
    return 'false';
  }

  throw ArgumentError('Invalid flag');
}

widget_test.dart
import 'package:flutter_test/flutter_test.dart';
import 'package:myapp/do_something.dart';

void main() {
  test('Test', () {
    const nTests = 10;
    for (var i = 0; i < nTests; i++) {
      expect(doSomething(true), equals('true'));
    }
    expect(doSomething(false), equals('false'));
  });
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.19.0, on Arch Linux 6.7.4-arch1-1, locale en_US.UTF-8)
    • Flutter version 3.19.0 on channel stable at /home/felipe/snap/flutter/common/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision bae5e49bc2 (12 days ago), 2024-02-13 17:46:18 -0800
    • Engine revision 04817c99c9
    • Dart version 3.3.0
    • DevTools version 2.31.1

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /home/felipe/Android/Sdk/
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /home/felipe/Applications/android-studio/jbr/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • CHROME_EXECUTABLE = chromium

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2022.2)
    • Android Studio at /home/felipe/Applications/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Arch Linux 6.7.4-arch1-1
    • Chrome (web)    • chrome • web-javascript • Chromium 121.0.6167.160 Arch Linux

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Thanks for the report @Mr-Pepe.
I can reproduce this on the latest master and stable channels.
Logs (nTests = 10)

00:01 +1: All tests passed!
Found 1 entries.
Found common filename prefix "/Users/damilolaalimi/Desktop/test_macos"
Generating output.
Processing file lib/do_something.dart
  lines=4 hit=3
Overall coverage rate:
  lines......: 75.0% (3 of 4 lines)
  functions......: no data found

Logs (nTests = 100000)

00:01 +1: All tests passed!
Found 1 entries.
Found common filename prefix "/Users/damilolaalimi/Desktop/test_macos"
Generating output.
Processing file lib/do_something.dart
  lines=4 hit=2
Overall coverage rate:
  lines......: 50.0% (2 of 4 lines)
  functions......: no data found

stable, master flutter doctor -v

[!] Flutter (Channel stable, 3.19.1, on macOS 14.2.1 23C71 darwin-arm64, locale
    en-NG)
    • Flutter version 3.19.1 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    ! Warning: `dart` on your path resolves to
      /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/damilolaalimi/sdks/flutter.
      Consider adding /Users/damilolaalimi/sdks/flutter/bin to the front of your
      path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision abb292a07e (5 days ago), 2024-02-20 14:35:05 -0800
    • Engine revision 04817c99c9
    • Dart version 3.3.0
    • DevTools version 2.31.1
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.86.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • Damilola’s iPhone (mobile)  • 00008110-001964480AE1801E            • ios
      • iOS 17.1.1 21B91
    • iPhone 15 (mobile)          • DBCC5388-DA57-470E-8F0E-477EA0AA3C3A • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-17-2 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 122.0.6261.69

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

[!] Flutter (Channel master, 3.20.0-13.0.pre.31, on macOS 14.2.1 23C71 darwin-arm64, locale en-NG)
    • Flutter version 3.20.0-13.0.pre.31 on channel master at /Users/damilolaalimi/fvm/versions/master
    ! Warning: `flutter` on your path resolves to /Users/damilolaalimi/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d8503febb1 (3 hours ago), 2024-02-25 23:42:25 -0500
    • Engine revision fd3136782a
    • Dart version 3.4.0 (build 3.4.0-175.0.dev)
    • DevTools version 2.33.0-dev.11
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    • CocoaPods version 1.14.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] VS Code (version 1.86.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (4 available)
    • Damilola’s iPhone (mobile)      • 00008110-001964480AE1801E • ios            • iOS 17.1.1 21B91
    • macOS (desktop)                 • macos                     • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin         • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)                    • chrome                    • web-javascript • Google Chrome 122.0.6261.69

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

We are having a similar error and I think it might be connected, in our case we have around 5000 loc and since the last flutter update we lost coverage in some of our switch statements
The following shows one example, where everything is covered, but the return statement itself is not.
We use very_good_cli to execute tests, and they group all test in one file automatically to optimize it

Thanks for filling this issue.
I was able to reproduce but with a pure Dart package (not a Flutter package). I don't think this is an issue with Flutter. To get this investigated upstream, please re-file this issue on https://github.com/dart-lang/sdk.
If you aren't able to repro the issue using a Dart project, let me know here.


Feature Request - Add nZEDb

https://github.com/nZEDb/nZEDb/
This would be really handy!

Hi, Thanks for the suggestion.
I've now moved the feature requests over to Feathub
Could you please vote for this there so we can start getting a feel for what people would like to be worked on next.
Thanks


⚠ SceneTime has degraded performance

In bb895be, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1577 ms

Resolved: SceneTime performance has improved in 9da8744 after 6 minutes.


Seeking support

There should be seeking support in the library.
It seems that demos already have enough data for this to be possible, eg. snapshots on every 60 sec, round-start ticks, etc...
Backward seeking through time should also be possible.
For best usability, I think library should have an API like this:
DemoParser.SaveToSnapshot(Snapshot s);
DemoParser.RestoreFromSnapshot(Snapshot s);

class Snapshot
{
    List<Entity> entities;
    List<StringTable> stringTables;
    int tick;
    long streamPosition;
    CDemoFileHeader header;

    ...
}

This is definitely do-able, I have some changes on a branch that I need to test that implement seeking to arbitrary ticks.

WIP PR here: https://github.com/saul/demofile-net/pull/49

The PR should now be in a state for you to test.
I've tested it locally jumping 77 seconds every 77 seconds, and jumping back 16 seconds every 33 seconds. It seems to read the demo fine.
Can you try it out and let me know how you get on?

This is now available as v0.12.1


Deep dive on input processing with Python

We need to investigate some options for handling keyboard input on a lower level than asciimatics provides us with. This will let us bypass OS-level keyboard settings and give us the ability to handle both key up and key down events.

Input handling libraries

https://github.com/boppreh/keyboard, requires root on linux but does not depend on X11
https://github.com/moses-palmer/pynput, may require root on mac osx and additional permissions, requires X11 on linux
https://github.com/marcusva/py-sdl2, low level, cross platform, seems to require initializing a graphical window in order to listen to events
https://liballeg.org/bindings.html, python bindings for C lib, low level, cross platform, similar to SDL2

Frameworks

https://github.com/reidrac/pysdl2-harness, provides utilities to manage game and event loops using pysdl2
https://github.com/libtcod/python-tcod, python port of popular roguelike framework, uses SDL2 for event handling, no terminal rendering?
https://github.com/pygame/pygame, the beast itself, no terminal rendering, did it support key up events? Maybe uses SDL1 for event handling?
https://github.com/arnauddupuis/pygamelib, terminal-based framework

Tutorials

http://rogueliketutorials.com/tutorials/tcod/v2/, python-tcod tutorial
http://inventwithpython.com/pygame/, free pygame book

Examples

https://github.com/rswinkle/inventwithpython_pysdl2, small pysdl2 games
https://github.com/LukeMS/pysdl2-roguelike-tutorial
https://github.com/tomdaley92/py-pong, pysdl2
https://github.com/shenjinti/pysdl2-flappybird, pysdl2

pygamelib

looks really interesting
does not seem to handle key down/up events
looks very tailored to a board/player/npc roguelike game

python-tcod

cross-platform
opens a new window, not directly in terminal (like pygame)
terminal-like rendering though since it's geared towards classic roguelikes
key down and key up events work

pysdl2-harness

cross-platform not tested
all rendering happens in a new window
key up and key down not tested

I'll admit, I haven't had much time to dig into this but this looks useful: https://stackoverflow.com/questions/27215326/tkinter-keypress-keyrelease-events

^^^ appears as though tkinter also relies on opening a second window, similar to pygame, python-tcod, etc.
Really cool that this is built into python though!

Oh, whoops. I never had a chance to run it but wasn't clear that was the case. My bad. 😳

Asciimatics input handling

reads key-press events only
cannot understand key-up or when keys are held down
can block execution to wait for an event, or check events with each step of the event loop
events can be processed via the Scene (GameController) and also in Effects

Notable side effects

when keys are held down, the key events are repeated, but after an unknown delay which is determined by the OS

it's possible this could be adjusted on an OS-specific level, but the user would be required to do this manually
archlinux example, should work for most X11 users: https://wiki.archlinux.org/index.php/Xorg/Keyboard_configuration#Adjusting_typematic_delay_and_rate
xset r rate 100 60 for 100 ms delay and 60 Hz, typing was still OK at this setting but not ideal
with this setting, there is still a small flicker but only occaisionally
a 75 90 setting, I noticed flicker less but still sometimes, and this makes it very difficult to type normally
how this can be accomplished in other OS' is unknown

Functional workarounds

remove the notion of "plucking a string" so that a key press creates both a fret press and string pluck

we would not be able to support notes held for a duration and may need to adjust the UI for this

we require the user to configure their key repeat delay and perhaps implement an optimal amount of debouncing in-game

these key settings would have to be applied before the game, and unapplied after the game
debouncing might be tricky to get right
we have a hacky "key-hold" process, if the key event is not coming in, it is reset to off

I think we should just do the first workaround. No one should expect this to be a replacement for Guitar Hero (at least for v1). If they really want a perfect clone with a keyboard, they can play Frets on Fire. :)

Okay, that sounds good to me and prevents the need to workaround OS-level stuff manually.


Misleading error message when performing "DeleteBackendEnvironment" operation by an unauthorized user

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
No response
Region
No response
Amplify Hosting feature
No response
Describe the bug
When trying to delete Amplify backend environment from console by signing in as a user who is not authorised to perform the action. Console shows the below error message

"Deletion Failed Network Failed"
It ideally should display Unauthorised/AccessDenied error.

In the browser console, the remove network call is failing with 403 error. That is as below, however the console shows the Network Failed error.

x-amzn-errortype: AccessDeniedException
Also, could see AccessDenied error in CloudTrail logs.

Expected behavior
Console should be showing an accurate error message like AccessDenied/Unauthorised.
Reproduction steps

Created an IAM user without the “DeleteBackendEnvironment” permission.
Signed as this user and tried to delete an amplify backend environment from the console.
The delete operation failed with the below error message

“Deletion Failed Network Failed"

Checked the network calls being made from the browser and could see the 'remove' call failing with 403.
Also, the CloudTrail logs show the AccessDenied error.

Build Settings
No response
Additional information
No response

This is being prioritized internally, and will be worked on.


Flutter run not working on real device

I have configured everything, enabled debug mode on my android device, however, when I do flutter run this is the output.
Launching lib/main.dart on SUNSET2 in debug mode...
Initializing gradle...                                              1,9s
Resolving dependencies...                                           6,7s
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                         4,4s
Built build\app\outputs\apk\debug\app-debug.apk.

And it opens the aplication on the device and then closes immediately. I then tried flutter attach, but it stops with this message Waiting for a connection from Flutter on SUNSET2.... The device does appear on flutter devices. What is going on?

@OnlineProjectsCloud did you find a solution?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


[FEATURE] OpenAI: Tokenizer for gpt-4o (update jtokkit)

gpt-4o uses a different tokenizer, we need to update jtokkit version once it is released

If I'm not mistaken, this is closed by https://github.com/langchain4j/langchain4j/pull/1499
(or does this project prefer to close issues after full releases?)

I understand that this is closed by https://github.com/langchain4j/langchain4j/pull/1499


Volume mounts on sidecars doesn't work unless mounted to driver first

I'm very happy to see that sidecar support was added. I had actually hacked an other version of  the operator to have this functionality. My implementation was minimally invasive, but this approach is much better.
One thing I noticed, I cannot mount a volume unless it is also mounted on the driver container. I thought maybe the functionality was completely broken, but after also adding the mount to the driver first, I can also mount on the sidecar.

That's the current behavior. The driver pod spec gets mutated to add a volume only if the volume is requested to be mounted to the driver pod/container.

Good to know!

Hi,
We have use case where we need to mount secret to sidecar which should not be visible to driver container. Is there anyway to achieve it?


Leaderboard subscore to use a different sort than score

It would be awfully nice to be able to set a leaderboard to be sorted by Descending on the score value and Ascending on the subscore value.
This would represent the use-case where a higher score is better but a smaller time (subscore) is also better (as a tie-breaker). Or in other words: get the highest score possible in the least amount of time.
Unless there is a better suggestion, it looks like I'll have to "encode" the subscore (subscore =  - time_taken) in order for the rankings to be correctly calculated.

I'd like to add to this that it would also be nice to be able to set different Operators for score and subscore.
The use case would be setting the score with INCREMENTAL and the subscore with SET.
Score is incremented, whereas subscore is set to the current timestamp, in order to break ties in favor of whomever posted the score first.


[LOG4J2-2600] Add declarative Groovy configuration DSL

See https://issues.apache.org/jira/browse/LOG4J2-2600
This is currently a draft PR as it doesn't even seem to work properly at the moment. However, the gist of what's going to be supportable is here. I may be able to re-use the Component class from the configuration builder API, but that API is a bit too strongly typed for this use case.

Anything going on here or should it be closed?

This PR is stalled for now. I'll be revisiting this later, and it can be resurrected then.

Closing per @jvz so I don't keep looking at it.


Base64 implementation

The library is pretty close to having base64 encoding support; the main tricky aspect will be handling padding correctly. Instead of leading "zeroes", b64 adds trailing = tokens depending on the amount of padding needed.

Nah.


update pytest skip for tox to increase codecov

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (fc68731) 63.03% compared to head (9483815) 63.63%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #39      +/-   ##
==========================================
+ Coverage   63.03%   63.63%   +0.59%
==========================================
  Files          10       11       +1
  Lines         790      803      +13
==========================================
+ Hits          498      511      +13
  Misses        292      292

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

ultimately did not accomplish my goals (enabling tox testing), but the small changes to some files are the reason I'm merging anyways


Revoke

Add support for Revoke token function

Finally had some time to get back to this. Fixed my git branch (still learning git), renamed revoke to revokeToken for clarity, added tests for revokeToken. Uber.js is failing code coverage. Line 183 is mine, the others are not. I have no idea how to test the "happy path" since revokeToken doesn't return anything (revoke is happy if any string is passed in). If someone knows how to test line 183 (return;) let me know and I'll fix the tests. This should be ready for code review and merging now.

@rcurrier666 thanks for the PR! could you please add tests so that the coverage check doesn't fail anymore?
https://travis-ci.org/shernshiou/node-uber/jobs/255144175#L995

solved with #64


fix: clarify option selection instruction in REPL

Closes #352

Closing since Subo's being deprecated.


Update deployment script. Make this v1.

Use githubhook instead of github-webhooks
Update shelljs
Use winston for logging
Make this v1.0.0
Update README.md with quick start info
Add an editorconfig!
Specify lowest node engine in package.json

Good stuff


WIP: Ansible LLM Schema Models

This PR introduces Pydantic models for Ansible LLM testing

Would want sign off from @dagrayvid before merging

I close this PR as we're deprecating openshift-psap/ci-artifacts in favor of openshift-psap/topsail. Please reopen the PR there if relevant.