fix(frontend): remove pg_get_userbyid limit operator

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Since scalar subquery currently can detect filter like pk = 1 to guarantee max one row semantics, we can remove the limit 1 from \d.
The sql in planner-test is the actual sql generated by \d.

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Resolve #5297

@mergifyio requeue


Image w/ Subviews Radius!

Hi,
When adding subviews to an Image the border radius doesn't work.
(Android - RN 0.18.1)
Thank You!

I guess this is in process here #4252


Number/letter token overlays

A lot of scenarios have number or letter tokens that either remind you to read some story line or represent different special rules. These would be really good to have in the virtual bord as well, because they are often not easy to represent by the available options (we currently usually use summons or obstacles for this, but it becomes confusing pretty quickly).
Non-spoiler example for level 2: there's a number 1 on the door, which adds new rules to the scenario and wouldn't be noticed if you didn't loom into the scenario book.
Level 56 (spoiler alert) has a-tokens which represent allies and would also be missed if you don't look into the book.
Keep up the good work, it's a really great platform!!

Hi @DaniBodor,
Thanks for your feedback and sponsorship! I think this might be a duplicate of #14 , but puts it a bit better than I did there. We currently use Summons in our games too, and you're right it gets complicated quite quickly.
I'll have a look into this probably in the second-ish week of the new year, once solo scenarios are in and as part of the custom scenario creator work that I'll be doing at about the same time :-)

Just to add, there are scenarios that have you adding terrain (like water tiles) as well. A general solution to add or remove overlay tiles is needed.

Just to add, there are scenarios that have you adding terrain (like water tiles) as well. A general solution to add or remove overlay tiles is needed.

Hmmmm.. yeah looks like I'm going to need to add a few more overlays...do you know the scenarios that add terrain? Must have missed that when going through them :-/

Hmmmm.. yeah looks like I'm going to need to add a few more overlays...do you know the scenarios that add terrain? Must have missed that when going through them :-/

74 - Merchant Ship is one.

74 - Merchant Ship is one.

Similarly, I see you have support for Forgotten Circles, but I don't see the Rift as an overlay option. These are overlays the diviner can create and move around.

Similarly, I see you have support for Forgotten Circles, but I don't see the Rift as an overlay option. These are overlays the diviner can create and move around.

Hi @orphu
Thanks for the example. I'm planning on including water tiles in the next release, but at the moment I'm focussing a lot on the Scenario Creator so releases and bug fixes are a little delayed.
With regards to the Rift tokens, they should appear if there's a Diviner in your party. Otherwise they should be hidden :-)
Hope that helps :-)

Hi @orphu
Thanks for the example. I'm planning on including water tiles in the next release, but at the moment I'm focussing a lot on the Scenario Creator so releases and bug fixes are a little delayed.
With regards to the Rift tokens, they should appear if there's a Diviner in your party. Otherwise they should be hidden :-)
Hope that helps :-)

With regards to the Rift tokens, they should appear if there's a Diviner in your party. Otherwise they should be hidden :-)

Oh excellent! Thanks for the tip!

With regards to the Rift tokens, they should appear if there's a Diviner in your party. Otherwise they should be hidden :-)

Oh excellent! Thanks for the tip!

Hi @orphu ,
The water tile issue should be fixed now. Still need to add number tile overlays, but that will be available with or shortly after the Scenario Creator is ready :-)

Looks great, thanks!

This I think is now fixed as of 1.14.0


No webstorm plugin

I actually created a simple one.
Just syntax highlighting.
https://plugins.jetbrains.com/plugin/7628-emblem


0.65.0 - Font Awesome fonts don't get loaded

When trying to load Font Awesome fonts, it displays these errors in Chrome developer console:
Failed to decode downloaded font: http://wealth.test.forcrowd.org/app/css/fonts/fontawesome-webfont.woff2?v=4.7.0
wealth.test.forcrowd.org/:1 OTS parsing error: Failed to convert WOFF 2.0 font to SFNT

During "publish" taks, htmlreplace was running on all copied files, and even though it had nothing to do with fonts, font files were getting modified (encoding issue?). Now htmlreplace only runs only on the files that need replace operation.

Closed with pull #119


Problems when using modal

I have installed 1.0.24 from npm, and Iam trying to use Modal. Iam getting error all the time (as on picture below)

How to use it ?

Did you follow all the instructions from the demo usage ? If yes, can you provide a small sample on how to reproduce this ?

please show component containing this html

Yes i have followed tutorial. I have also checked directory structure of package installed from npm. The main difference I saw was that "ng2-bootstrap\components\modal\modal.component.d" have no Directive annotation as in tutorial

@Directive({
selector: '[bsModal]',
exportAs: 'bs-modal'
})

Error that i got is strictly says "There is no directive with exportAs". Is npm version is actual ?

@grzegorzgolec this should fix your issue https://github.com/valor-software/ng2-bootstrap/issues/667#issuecomment-235240497


Is there a site docs for getting started or is the README the main docs?

Just trying to get started on this bot framework. Just wondering if this is the main docs

We have a number of readme's on this project, you might want to also check out these Botkit Studio Starter Kit for Slack or for Cisco Spark or for Facebook
https://github.com/howdyai/botkit-starter-slack
https://github.com/howdyai/botkit-starter-ciscospark
https://github.com/howdyai/botkit-starter-facebook
If there is anything else you are wondering about, feel free to follow up here or try asking on our public slack:
http://dev4slack.xoxco.com/

hey @peterswimm thank you I will get back to this resources on this weekend after exams. And thanks for the slack link!

@peterswimm would you help me out starting on this? :) just to get an idea how to start it off

Feel free to post any questions here or on our slack I linked to!


Error in debian 11 with the package "libuv".

I am following the installation manual, but when I run the command "cmake .." I get an error with the "libuv" dependency, it tells me that it does not exist, I tried to install that package but it tells me that it does not exist to install it. Any suggestions.

Check out the wiki for the fix. https://github.com/polybar/polybar/wiki/Compiling
Depending on your distro, the required "libuv" version is to be installed.

sudo apt install libuv1.dev

Yes, this worked :+1:

Terribly sorry for only seeing this now.
Thanks for this. Added it to the README


Archiving locally

Am I correct (from a quick read through the archiving code) that there isn't a way to have the archive local, but rather relies on outside sources?
Sorry if I misunderstood.

Hi Bruce!
You are correct. Currently, there isn't a way to archive bookmarks locally.
If it's something you would like to have in Crestify, please open an issue
for the feature request and we'll take this further there :)
On Fri, Apr 14, 2017, 7:54 PM Bruce Dillahunty<EMAIL_ADDRESS>wrote:

Am I correct (from a quick read through the archiving code) that there
isn't a way to have the archive local, but rather relies on outside sources?
Sorry if I misunderstood.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/crestify/crestify/issues/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEWWYpLKWtaC6Yk9brkE84TiIh0bcczcks5rv4GFgaJpZM4M9yX8
.

Moved to #6. I'll go ahead and close this issue.


"Failed - Forbidden" and "Quota exceeded" when downloading resnet18_features and resnet50_features

Hi Kyle,
Congratulations on winning 2nd place in the challenge!! Great work! I want to train and evaluate your model but I couldn't download the two feature files. The download of csv files went well. I have tried Chrome and Edge, on company and personal computers, and all have failed. It would pause in the middle, and then when I clicked "resume", it said "failed - forbidden". That is for resnet18. For resnet50, the error is "download quota exceeded". Do you know how I can fix this?
Thank you for your time!
Best regards,
Chen

Hi Chen,
Thank you for your interest in our work!
That is a well-known issue of Google Drive. This occurs when there are too many users downloading the same set of files within a few days. Could you try it again in a couple of days?
If the problem persists next week, please let me know. I'll request another Google Drive link and provide the new link to you.
Thank you,
Kyle

Hi Kyle,
Thank you for your reply! Some posts suggested using the incognito window, and it worked for resnet18! It paused in the middle as well, but when I clicked "resume", it restarted and finished the download, unlike a regular window that would give a "forbidden error". So now I have something to start with. The quota of resnet50 wasn't exceeded when I tried with the regular window, but that failed, and then when I switched to the incognito window to download it again, the quota was already surpassed. I will let you know by the end of next week if I can get resnet50.
Have a great weekend!
Best regards,
Chen

Hi Kyle,
In an incognito window ( I guess any window without a valid account signed in, like my work computer), the quota is always exceeded for resnet50; otherwise, the quota is not exceeded yet the download can't be finished in those signed-in windows, with "Failed - Network error" first and then "Failed - Forbidden" error after clicking "resume". It may be my network issue. Don't worry about it, I will figure this out or ask friends to try to download it. Thank you!
Best regards,
Chen

Hi Chen,
We made a new link: resnet50_features.zip (tmp)
You can download the features using this temporary link.
Best regards,
Kyle

Hi Chen,
That is weird. The file may be too big for you. I'll split it into multiple files and provide new links in a few weeks. Please allow us some time.
Thank you,
Kyle

@gomingchen Hi Chen, sorry for this late response. We recently released a new repository named GraVi-T. If you are still interested in SPELL+ with ResNet50 features, please refer to our new repo.


play_mysonos through exception during fetch the favorites

"play mysonos - error caught from response Details: {"stack":"TypeError: Cannot read property 'startsWith' of undefined\n at sonosPlayer.getFavorites.then.response (/usr/local/addons/redmatic/var/node_modules/node-red-contrib-sonos-plus/src/sonos-manage-radio.js:160:37)\n at process._tickCallback (internal/process/next_tick.js:68:7)","message":"Cannot read property 'startsWith' of undefined"}"

Hi, what version do you use? Can you send the payload?

Using 0.4.4 with RedMatic 5.0.5, Sonos version 10.4.2. I wrote you the payload of a debug node connected to the manage radio node. No you need anything else?

I mean the msg you send to the manage_radio node.

A matching radio entry exists:
object
id: "2/22"
parentID: "2"
title: "Fritz"
artist: null
album: null
albumArtURI: null
uri: "x-rincon-mp3radio://http://www.fritz.de/live.m3u"

"get_mysonos" without topic triggers the same error.

I have to do some testing - tomorrow evening I will upload 0.4.5.

hmmmm. On my Raspberry Pi everything works fine. But after migrating to RedMatic 4.05 several commands do not work anymore.
I have to check what the reason is.

How did you manage to get x-rincom-mp3 .... to "My Sonos" list.
I don't like the ad spot TuneIn plays on some stations before the stream starts to play. You can add your personal radio stream using the Sonos Windows client.

I double checked the IP-address. It is the right one, get the same error. An inject node triggering an TuneIn station one the same Sonos node works as aspected.

Very good idea, I hate that also but did not know that TuneIn added the ads.
I will do the same and check whether that causes the error message.

Could you please use "get_mysonosall" as payload  and send the fritz box entry  (screen shot or text) .  It seems that the "fritz" entry does not have an "uri" property. Maybe there is another field I can use to activate the station.

1: object id: "2/22" parentID: "2" title: "Fritz" artist: null album: null albumArtURI: null uri: "x-rincon-mp3radio://http://www.fritz.de/live.m3u"

I was able to add that stream to My Sonos (app - browse  - Radio by Tunein - My Radio Stations - Menu - Add new Radio Station (enter http ... + title) and then add this to My Sonos.
In version 0.4.5 play_mysonos and get_mysonos works fine on my systems raspberry pi with nodeRed and CCU with Redmatic.
So there is something else...
I just submitted a new version only updating "get_mysonos". Please update to 0.4.6, enable debug logging and trigger a "get_mysonos". The log file should then show at which item in the list the error occurs.

Thanks for your great support. A added "New Releases" of the Sonos PocketCast integration to my favourites. Here is the error:
5: object id: "2/25" parentID: "2" title: "New Releases" artist: null album: null albumArtURI: "https://integrations.pocketcasts.com/images/newreleases.png"

Please try the update 0.4.7. Know all items with out uri are ignored.
I hope that play_mysonos and get_mysonos now works with your My Sonos.

It works. Thank you very much.

Great. Thanks for the feedback. I am closing this issue now.


Connect appended shapes dragged from context pad

Fixes #170
This issue was introduced by the following change in bpmn-js: https://github.com/bpmn-io/bpmn-js/commit/14bf3a32ee31990040a2b438367a98a7a1efd92f#diff-4f1febb6aaa185881a563bca38b78d333495455342c3964b899687a7d0c0a2bc
The API to the create module was changed, which we did not adapt to on our end.

Accidentally based this off a wrong branch.


Builds broken in VS2017

Just ran into this: Opening the solutions in VS2017 is currently broken; VS will think that the <IntermediateOutputPath> is not set, causing it to try to modify the project files on open. Builds are broken with the matching MSBuild because <BaseIntermediatePath> does not get set correctly and ends up pointing somewhere under Program Files.
The cause of the issue is a change in MSBuild: https://github.com/Microsoft/msbuild/issues/1525
This is triggered by line 70 in Globals.props:
        <BaseIntermediateOutputPath Condition="'$(BaseIntermediateOutputPath)' == '' ">$(EnlistmentRoot)\..\obj</BaseIntermediateOutputPath>

Specifically, the condition returns false (since it now has a value by default), so all the intermediate path settings after that yield the wrong result.
Seems like removing the condition would be the simplest fix, as I can't see a case where to avoid setting it always.
Thoughts?

Installing an RTM of VS 2017 to try it out...

@tomasr if you'd have time - please try to apply similar change: https://github.com/Microsoft/ApplicationInsights-dotnet/pull/466#pullrequestreview-26937657
Not sure it will fix this problem, but it will help with Stylecop

@SergeyKanzhelev Yes, that fixes it. And figured out why StyleCop wasn't kicking in: I never added the Settings.StyleCop file and the StyleCop.MSBuild nuget facepalm. Now I've got a lovely 6200+ warnings :/

You can exclude all existing files from stylecop analysis so only new files will need to be fixed. Later we can uncomment files one by one.

I've now fixed all of the issues in the existing source (I think), but here's a question... how are you dealing with the old StyleCop parser throwing SA0102 errors on newish C# features? Seems this is expected, so wonder if you're just avoiding the use of such constructs for now?
Seems like the recommended alternative is to move to https://github.com/DotNetAnalyzers/StyleCopAnalyzers.

I think we will be moving to analyzer where possible. But the latest stylecop should support those language constructions... Can you please try to update the StyleCop.MSBuild package to the latest?

@SergeyKanzhelev Yep, already on latest. Specifically seems to dislike using ?? throw for null-check + assignment, as in:
this.telemetryClient = client ?? throw new ArgumentNullException(nameof(client));


consistent metadata and title label over identifier

Coverage decreased (-0.009%) to 90.439% when pulling b4d228b8853e9621d9e3e997d4c706b762aed177 on metadata into 99fee85a226db755654a65ad43f672d3e67f039d on master.


Unable to copy canvas content from  Cannot read property 'getImageData' of null

I'm trying to get screenshot of page that the DOM contains  element, logging the operation, so the error I got :

18ms html2canvas: Unable to copy canvas content from <canvas height="769" width="1267" id="kl" style="display: block; touch-action: none; outline: none; background: rgb(247, 247, 247); cursor: auto;" tabindex="8888"> TypeError: Cannot read property 'getImageData' of null

The code :
html2canvas( [ document.body ], {      "proxy": "http://anfo.pl/jw/widget/proxy/proxy.php",      "allowTaint": true,      "useCORS": true,      "logging": true,      "onrendered": (canvas) => {            document.body.insertBefore(canvas, document.body.childNodes[0]);      }  });

https://github.com/niklasvh/html2canvas/pull/1037/commits/7ca94348bc1f81d752a7b0ba64d77146a8ec528e     try this way, it fixed my bug.


Add Operator release to the Releases page

Update https://aws-otel.github.io/docs/releases to add a link to download the Operator release in Public ECR.
@JamesJHPark - assigning this to you.

Resolved the issue with PR #171


Helm init and helm version error

Hi! I was trying to setup my minikube environment using helm. After installing helm once i run Helm init it says error installing: server could not find the requested resource. Helm version command runs for client but shows error could not find tiller
Output of helm version:
Client version details (v2.13.0) and an error could not find tiller
Output of kubectl version: shows output for both server and client
Cloud Provider/Platform (AKS, GKE, Minikube etc.): Minikube

duplicate of #6374


GSSoC'23: Navbar Improvement

Current Behavior

I think before making page responsive we have to make it proper for desktop as well.

I am a GSSOC'23 contributor.
I would like to work on this issue. Please assign me with this.


Testing is not working as expected

I am probably doing it wrong, but I need some help, because it's just the test that is not working...
The following test:
import { expect, test, beforeAll, afterAll, afterEach, beforeEach} from "bun:test";
import server from "bunrest";

const app = server();
let service;

beforeAll(async () => {
    service = app.listen(3000, () => {
        console.log('App is listening on port 3000');
    });
});

afterAll(async () => {
    service.stop();
});

test("should respond OK for ping", async () => {
    try {
        const res = await fetch('http://localhost:3000/ping');
        expect(res.status).toBe(200);
        expect(await res.text()).toBe('OK')
    } catch (e) {
        throw e;
    }
});

The index.ts:
import server from "bunrest";
import cors from "cors";
import router from "./src/core/Router.js";

const app = server();
const port = 3000;

// add router
//const router = app.router();
app.use('/', router);

app.use(cors());

app.listen(port, () => {
  console.log(`App is listening on port`, port);
});

For the route.ts:
import server from "bunrest";
const app = server();

// add router
const router = app.router();

router.get('/ping', (req, res) => {
    //console.log("[/ping] RECEIVED: ", req);
    res.status(200).send('OK');
});

These are the reduced versions.
Running bun test outputs:
Router.test.ts:
App is listening on port 3000
133 |         const res = that.responseProxy();
134 |         const tree: TrieTree<string, Handler> =
135 |           that.requestMap[req.method.toLowerCase()];
136 |
137 |         if (!tree) {
138 |           throw new Error(`There is no path matches ${req.method}`);
                    ^
error: There is no path matches GET
      at /Users/luismal/Projects/Sia_AA/node_modules/bunrest/src/server/server.ts:138:16
GET - http://localhost:3000/ping failed
Database connection to  mongodb://localhost:27017/SIA  established.
20 |     console.log(result);
21 |     expect(result).toBe("OK");
22 |     */
23 |     try {
24 |         const res = await fetch('http://localhost:3000/ping');
25 |         expect(res.status).toBe(200);
            ^
error: expect(received).toBe(expected)

Expected: 200
Received: 500

      at /Users/luismal/Projects/Sia_AA/tests/Router.test.ts:25:8
✗ should respond OK for ping

I think there is some problems in cors package.

I was missing app.use('/', router); on my example test.


Computing jacobian for tf.image.resize cause Check Failed on cuda

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf 2.9, 2.11.0-dev20220927
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 20.04
Mobile device
No response
Python version
3.9
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Computing jacobian for tf.image.resize cause Check Failed on cuda and session crashes.
Standalone code to reproduce the issue
import tensorflow as tf
images = tf.zeros([1, 1, 1], dtype=tf.float32)

def f(images):
    size = [224, 224,]
    method = "nearest"
    preserve_aspect_ratio = False
    antialias = False
    return tf.image.resize(images, size, method=method, preserve_aspect_ratio=preserve_aspect_ratio, antialias=antialias, )

with tf.GradientTape(persistent=True) as tape:
    tape.watch(images)
    output = f(images)
print(output.shape) # (222, 224, 1)
g = tape.jacobian(output, [images]) #  Check failed

Relevant log output
(222, 224, 1)
WARNING:tensorflow:Using a while_loop for converting ResizeNearestNeighborGrad
F ./tensorflow/core/util/gpu_launch_config.h:129] Check failed: work_element_count > 0 (0 vs. -1822093312)
abort (core dumped)

@gadagashwini
I was able to reproduce the issue on Colab by using TF v2.9. Please find the gist here for reference.
Thank you!

We intentionally don't check on GPU, and this is usually documented. On GPU, the check is not worth the extra kernel launch required. Garbage in, garbage out. The calculation has no meaning regardless.
Specifically, quoting from the docs for scatter_nd:

Note that on CPU, if an out of bound index is found, an error is returned. On GPU, if an out of bound index is found, the index is ignored.


Quick preview docker-compose seems to be broken

Quick preview docker-compose seems to be broken...? Wouldn't start and a lot of errors in the docker logs...

I've try and that works for me, can you provide more log and reopen please ?


haskdogs don't recognise package installed from source code.

I don't even know if it's possible, maybe there's no way to know where the source code directory of a package is.

Well, haskdogs uses ghc-pkg to find package containing given module name. For example, you can find that Control.Monad belongs to base-<IP_ADDRESS> (on my system) using
smironov@tenten ~ $ ghc-pkg find-module Control.Monad
/usr/lib/ghc-7.0.4/package.conf.d
base-<IP_ADDRESS>
haskell2010-<IP_ADDRESS>
/home/smironov/.ghc/i386-linux-7.0.4/package.conf.d
Could  you please try this command with a module name installed from source?

ghc-pkg find-module find the package ok, but cabal unpack can't find it.

Hm, one can introduce new option - haskell projects directory to scan in addition to cabal unpack. And here is another thing: can you manually copy (or link) your module into ~/.haskdogs/ and see whether haskdogs can find it?


Feature request: send raw tx

As a complementary feature to #57, it would be great to have the option to broadcast a raw tx by only providing a network/RPC provider.
Useful if you want to modify data created by --raw-tx manually, and then only use starkli as a convenient interface to broadcast the raw JSON data (e.g. by supplying a file or a string)

This feature makes a lot of sense once #57 is implemented. However, regarding the modify data created by `--raw-tx` manually part I'm not so sure.. If you manually modify data, the signature would change right? Then you cannot really broadcast it anymore.

Good point, in our usecase we indeed modified both data and signature. I still think this is useful to have this option as an easy way to send (or resend) raw txs


Oracle: table info read performance improvements

In some Oracle databases table read takes too much time.
One of the reasons - reading from ALL_MVIEWS system view.
We don't need to query it + we can improve base table read query by replacing it with select from ALL_OBJECTS + filter by type.

Please check that metadata read isn't broken (tables, views, mviews, ERD, etc).

12.2 Oracle was tested. Is it ok that java object does not have data in Declaration tab?

It is ok, our test db doesn't have many Java sources

Other items are correct.


Support adding details to the pager duty incident

Add the ability to add Details to the pager duty incident report.
This allows user to put in the BUILD_URL, for instance, so that when they get a PD alert, they can click on the link and go directly the failing build to get more info.
You can add any env var or string, and it works just fine.

How do I pass json objects through detail fields？ I want to get properties of custom details in pagerduty.

@lizhh123 can you provide me with example of what exactly you mean?


Allow slight tolerance for restart health check

This PR makes the NCTL health checks more relaxed when it comes to validating the number of restarts. This is to guard against flakiness, i.e. when the test finishes correctly and all assertions hold (for example: network is correctly upgraded), but there were slightly more node restarts during the process than expected.
For example, assuming that there are 10 restarts allowed, NCTL will:

report error if there are more than 10+50%=15 restarts:

NCTL :: Adjusted restarts allowed: 15
NCTL :: ERROR: ALLOWED: 15 < TOTAL: 16

warn if there are more than 10, but less than 10+50%=15 restarts:

NCTL :: WARN: Test would fail without allowed restart adjustment
NCTL :: SUCCESS: ALLOWED: 15 = TOTAL: 13

finish successfully otherwise

Closes #4360

bors r+


Use relative file name when reporting errors

Bad merge caused us to regress recent change to use relative path in error reporting in tsc. putting it back, and also enabling it for the --pretty mode.

:+1:


Rki Survey (EXPOSUREAPP-4637)

This PR puts together all the prepared work to get an authorized otp for the survey.

looking good 👌  can be merged soon
one last thing, if its not outdated:
"If authorized successfully, the client received an expirationDate from the server which indicates the validity of the OTP."
The received expirationDate must be stored along with the OTP. (Point 4 in techspec)


Add a path from a project page to a metadata page on UI

It seems nice in practice to me if there is a path from a project page (#/projects/xxx) to a metadata page (#/metadata/xxx) on the UI.
Here is a screenshot:

Should we move the button down so all buttons are grouped together?

Can we also remove the meta repository from the list? 🤔

@trustin

Should we move the button down so all buttons are grouped together? Or perhaps we could move the Create a repository button up. 😄

Sure, I'll try it.

@minwoox
IIUC, contents in meta directory and what we can see in #/metadata/xxx are different, so it seems that we don't have to remove that. Any thoughts?

Modified one

@minwoox
IIUC, contents in meta directory and what we can see in #/metadata/xxx are different, so it seems that we don't have to remove that. Any thoughts?

Yes, exactly. Sorry about the comment. 😅

Thanks @hokkun-dayo!


403 Forbidden!

I got this error when my app using old 0.3.3 version.
Could not GET 'http://dl.bintray.com/mobisystech/maven/org/readium/r2-parser/0.1.2/r2-parser-0.1.2.pom'. Received status code 403 from server: Forbidden
Disable Gradle 'offline mode' and sync project

same error. help


Be able to define per ASGs tags

I have issues
Tags currently are too prescriptive. I have a use case where I need to tag different ASGs with different tags. Im using the ability to push these tags down to node labels and taints to drive different workloads on my kubernetes cluster. Atm, it seems that I can only define tags once on the top level EKS module and these tags are used globally throughout. I would like to be able to define tags per ASGs. A sensible place to provide these seems to be the list of worker_groups maps.
I'm submitting a...

[ ] bug report
[X] feature request
[ ] support request
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
Tags are defined once in var.tags and used throughout both to tag the cluster resources itself, as well as, all ASGs that are created.
If this is a bug, how to reproduce? Please include a code sample if relevvant.
What's the expected behavior?
Should be able to provide tags in the list of worker_groups maps and these should be used to tagging the corresponding ASGs that are created for each respective worker group. If the tags are not set, they can default to the existing top-level global tag variable that is used.
Are you able to fix this problem and submit a PR? Link here if you have already.
I can submit a PR if this is reasonable.
Environment details

Affected module version:
OS:
Terraform version:

Any other relevant info

I think that sounds reasonable. You can add an extra variable into worker_groups items for tags per ASG. Feel free to make a PR.

This will be a lot easier to do when Terraform 0.12 (with a better type system, e.g. nested maps, and for-loop support) is out.

No worries. Maybe wait until 0.12 is released then.

In my all mighty laziness this is how I made this work.
                    map("asg_desired_capacity", "${var.ui_desired_capacity}",                         "asg_max_size", "${var.ui_max_size}",                         "asg_min_size", "${var.ui_min_size}",                         "instance_type", "${var.ui_instance_type}",                         "key_name", "terraform-ops",                                                "name", "${var.environment}.prod.ui",                         "target_groups","${local.prod_target_groups}",                         "pre_userdata", "sed -i '/--cluster-dns/a\\  --node-labels=application=prod-ui \\\\' /etc/systemd/system/kubelet.service\nyum install -y nfs-utils",                     ),                     map("asg_desired_capacity", "${var.quoting_desired_capacity}",                         "asg_max_size", "${var.quoting_max_size}",                         "asg_min_size", "${var.quoting_min_size}",                         "instance_type", "${var.quoting_instance_type}",                         "name", "${var.environment}.prod.quoting",                         "key_name", "terraform-ops",                         "pre_userdata", "sed -i '/--cluster-dns/a\\  --node-labels=application=prod-quoting \\\\' /etc/systemd/system/kubelet.service\nyum install -y nfs-utils",                     ),

@shamsalmon, I don't understand how you can tag a node with your solution, can you please clarify? thanks!

@max-rocket-internet  , per the use case of being able to auto scale down to zero nodes , the cluster-autoscaler requires  a certain tag be present on the ASG depending on the node label , e.g.:
{
   "PropagateAtLaunch": true,
   "Value": "bar",
   "Key": "k8s.io/cluster-autoscaler/node-template/label/foo"
}

I have an approach that solves only the above use case in a pr within our fork:
https://github.com/OpenGov/terraform-aws-eks/pull/2/files#diff-0973ac2dc1229ea1662319bdc6954f08R34
What are your thoughts on that approach ? I understand that we would like to wait on Terraform 0.12 for the more general use case of allowing arbitrary tags, but Terraform 12 realistically won't be generally available till end of Q1 and adoption to use it will probably take longer.

What are your thoughts on that approach

That is very specific for you though and, as you mentioned, it doesn't allow just a map of extra tags to be applied per ASG.
Hmmmmm
How about we add the option now and it will not support merging. So if you want per ASG tags, then you just need to supply ALL the tags, i.e. including kubernetes.io/cluster/xxx. It's not perfect but it will work until hash merging works properly. Sound OK?

Well, since this module was already putting in cluster-autoscaler specific tags for its auto discovery of ASGs feature, the changes above is a continuation of that to allow the ASGs to scale down to zero.
But I am in support of the option you mention: either working with the defaults that are currently provided or having to supply all of the tags for each ASG.

@Chili-Man this is somewhat tangential but I am having difficulty understanding the sparse documentation around this - does applying that tag mean that CA will scale the tagged ASG to zero or that the capability will now exist to scale it to zero? For example, if I have blue/green worker ASGs, the tag should be applied to both and setting the ASG size is what will allow CA to scale it to zero?
The most confusing part for me is:

If you are using nodeSelector you need to tag the ASG with a node-template key "k8s.io/cluster-autoscaler/node-template/label/" and "k8s.io/cluster-autoscaler/node-template/taint/" if you are using taints.


Fix 'must be called with valid offset encoding'

Pass the the LSP client offset_encoding when calling LSP methods

It's a little hard to track the changes here due to tab replacement. There's an editorconfig file to help with that. :grimacing:
Would you mind formatting the file according? Or, as I intend to archive this repository, if you desire to maintain a fork of it, please let me know and I'll point to it accordingly in the README.

OK, it's merged. Thank you for helping out with this!


Improve key controls, add better marking support

"Answer" numbers and "marking" numbers are now handled separately, allowing you to enter markings and then type an answer OVER them.

The markings will now always appear in numerical order
As such, markings now require a separate entry mode- selected via radiobutton, toggled via tab, or temporarily toggled while holding Shift.
This DOES override Tab's normal feature of cycling through selected objects... if that's an issue, the tab toggle can be changed to some other key.

Arrow keys now navigate the grid properly
Backspace/Delete now will delete all markings instead of the rightmost marking; or, if an answer is placed in a cell, it will delete the answer (without deleting markings that were hiding underneath the answer).
Changed the text color of user-entered numbers (Both markings AND answers) to better differentiate user-entered numbers from given digits. These colors are now static near the top of SudokuCell.cs for ease-of-changing.

Overall, these changes attempt to bring this more in line with SudokuPad used by Cracking the Cryptic.
(More ideas from this that would be useful would include multi-highlighting to enter into multiple cells at once, and a third entry mode for corner-marking, though the former is a bit beyond me, and the latter I'm not sure how to go about displaying the numbers properly - would probably require a fair bit of work, and new UI elements on each cell...)
This fixes #3, and makes #5 and #8 redundant.

project is nolonger maintained


option for specifying which files end up in coverage report

prerequisite: https://github.com/samuelroy/nim-coverage/issues/2
eg usage
# this will run coverage only on specified globs
# !std/this_will_be_ignored/**` could be used to ignore patterns (just like in ripgrep ignore patterns or gitignore)
coverage --cov:modules:'std/**,pkg/regex/bar/*,mylocaldir/src/subdir/foo/**/*.nim,!std/this_will_be_ignored/**'

https://github.com/citycide/glob can be used for parsing globs /cc @citycide

There's 2 different things here:

target directories/files that we need to compile and run in coverage mode
cleaning up lcov.info by removing unwanted data (like coverage on .nimble pkgs)

We could end up with the following arguments:

target: glob - defines the folders and files to compile and run in coverage mode
cov: glob - defines the files that will end up in the coverage report

# !tests/ exclude folder tests from the coverage report
nimcoverage --target:'tests/**/*.' --cov:'./,!tests/'

Use the flag --cov to include and exclude path from the code coverage report. See the latest readme.


Resolved. I had unclosed tags in my head tag


Substitute a replacement character for invalid UTF-16 text in a TextSpan

Fixes https://github.com/flutter/flutter/issues/84693

Bonus points if you mention this somewhere in the API docs, maybe for Text, TextSpan, and TextPainter in particular.

@jason-simmons in this line, As I understand you replace whole text with REPLACEMENT CHARACTER, Why whole text? We should replace only invalid characters

Replacing individual characters is a lot of work, I really don't think we should do that given that the whole point is to fail. If you want to replace invalid characters, that's something to do in your app.

@Hixie I understand,  Assume I want to do it, There is many many string and Text in the application, Some Text are in the dependencies of dependencies
It's very cumbersome to do it in application layer, At least the must be a Global option to override text sanitization

You would have to provide a central place in your application where strings were sanitized, yes.
But fundamentally, strings should be sanitized long before they reach Text. If they're not, that indicates a more fundamental problem. I don't think it's Flutter's job to provide APIs to make it easier to work around such fundamental problems.


The Lounge Mentions Popout

There is a issue /w thelounge dark theme /w the mentions popout. It doesnt effect dark organizr so i assume it has to do with the radiant gradiant in that theme. It does effect dark.  I am able to fix it by adding this, but I think you'd probably have a better way.
.mentions-popup {
   background:#1f1f1f
}

Without fix:

With fix:

Thanks for the issue, can you clear cache to see if its fixed now? Might take a minute or two.

Thanks for the issue, can you clear cache to see if its fixed now? Might take a minute or two.

All good, thanks for the super fast fix!

All good, thanks for the super fast fix!

Actually going to reopen... there is another part that needs a fix, class="toggle-content toggle-type-link"

Actually going to reopen... there is another part that needs a fix, class="toggle-content toggle-type-link"

Closing /w merge.

Closing /w merge.


QB-1761: Fix evm contract destroy v2

this rever the changes of this pr(https://github.com/stratosnet/stratos-chain/pull/236) and only disable the deletion of contract code.


R2D2 ConnectionManager: allow to pass queries executed for new conne…

…ctions

You can do this via CustomizeConnection
in r2d2

The above comment is correct. https://docs.rs/r2d2/0.8.2/r2d2/struct.Builder.html#method.connection_customizer


Activity bar border is missing in web

using web/safari
using "Github Light" theme
hide sidebar
Safari doesn't show a border, desktop does

This was caused by some safari specific css for the activity bar from 2 years ago. It supposedly fixed an issue with the compact menu jumping in safari, but after removing it I could not discern any regression. Since there was no linked issue, I have opted to remove it and we can bring it back as an issue and link the fixed behavior if it still remains.


Fix wrong reference to section 5 VSCode

Fixes #75

I am resolving the conflict

Obsoleted by #77


Add a button to submit feedback from a global error on mobile.

WIP

@keybase/react-hackers, I'd like to change the text in the heading on the feedback page based on whether they came from tapping on the button in the global error or just visited the feedback page from the settings tab. What's the best way to do that?

@keybase/react-hackers, this is ready for review.


add new pack untilize for transpose/fold/resnet

Ticket
https://github.com/tenstorrent/tt-metal/issues/12076
Problem description
when pack out narrow rows (16B), currently need to repack it using NoC, with the new LLK functionality, it can directly pack out w/o repack.
Checklist

[x] Post commit CI https://github.com/tenstorrent/tt-metal/actions/runs/10726581567
[x] Model perf https://github.com/tenstorrent/tt-metal/actions/runs/10727035676
[x] device perf https://github.com/tenstorrent/tt-metal/actions/runs/10727028641
[x] t3k model perf https://github.com/tenstorrent/tt-metal/actions/runs/10727733604

@yugaoTT,
How was this change tested? Please make sure that you run Resnet tests and conv unit tests for performance and functional reasons because you've changed code within LLKs that are used everywhere.

@ttmtrajkovic yes I'll run the model/device perf pipeline (contains resent50 and other models) along with post-commit (conv tests), which should cover the changes.

@yugaoTT, How was this change tested? Please make sure that you run Resnet tests and conv unit tests for performance and functional reasons because you've changed code within LLKs that are used everywhere.

@ttmtrajkovic I have been monitoring this change with Yu, he is adding an extra flag to allow pack_untilize to work with <16 datums per face_c_dim, it will not change default behaviour

@yugaoTT Can you make a separate issue that the above change needs to be propagated to Blackhole as well but because the implementation is different it will require different additions?

@rtawfik01 issue for BH is here: https://github.com/tenstorrent/tt-metal/issues/12288

FYI. @ilkoo-lee @jeongu-moreh


[Feature Request] Component manually uploads files

Existing Component
是
Component Name
upload
Description
When the upload component manually uploads the file, you want to give the submit() method a callback that will be called after all the files have been uploaded successfully.Or an asynchronous function submit(). Then (res={})

duplicate


Question about document _id's

Sorry for additional questions, but I am a little confused after reading the docs for both ember-pouch and relational-pouch.
With ember-pouch, are the document id's automatically set to "pokemon_1" or "trainer_2" as mentioned in the README for relational-pouch?
Also, does ember-pouch include the relational-pouch plugin and call the appropriate db.setSchema() methods automatically?

Document IDs in ember-pouch are determined by relational-pouch, yes. The examples you found in the README are intended to be conceptual. The actual document ID structure also contains a flag that indicates the type of the ID (string, integer, etc.). The section of the README about parseDocID has a concrete example of the actual structure.
You don't need to do anything with setSchema yourself. You can use ember-pouch for the basics without being aware of relational-pouch at all. (Awareness of the document ID structure becomes important when you get into syncing or building indexes, though.)


Checkpoint typo did not trigger FAILed tests, why?

@matejpavlovic  and I have recently found this bug in which we missed the ! before the reflect.DeepEqual. While this is an easy-to-solve typo (coming in a PR soon), it is not expected that all tests PASSed given this error when verifying the checkpoint.
We need to know why is that. We came up with the possibility that this is because the introduction of the async multisig collector means that in the first epoch all the transactions in all tests are delivered, so no checkpoint actually need to be verified in the tests. It could be this or something else, but we need to figure it out and resolve it so that the tests are comprehensive again.

Typo resolved in this PR, but issue remains that we need to understand and fix at least testing.

In addition to figuring out why the tests passed despite the error, we should write unit tests for the checkpoint, which should have discovered that bug in the first place.

Good news. It is what I suspected - the async multisig collector. The system is just too fast to notice the problem with the checkpoint, because everything gets delivered before the checkpoint. When I set the limit of sub-certificates in the multisig collector back to one, the test fails as it should.


[decompiler] Workaround for GOAL fpr -> gpr spill bug and fix >16-bit signed FP loads

Fixed types for large offset loads and added an exception to the stack stuff to allow it to use inconsistent sizes for loading/storing as a workaround for a bug in the original GOAL compiler.

Pull Request Test Coverage Report for Build 933523018

0 of 20 (0.0%)  changed or added relevant lines in 5 files are covered.
4 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 69.539%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

decompiler/IR2/AtomicOpTypeAnalysis.cpp
0
2
0.0%

decompiler/util/DecompilerTypeSystem.cpp
0
3
0.0%

decompiler/util/TP_Type.h
0
4
0.0%

decompiler/analysis/stack_spill.cpp
0
5
0.0%

decompiler/util/TP_Type.cpp
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

decompiler/util/DecompilerTypeSystem.cpp
1
62.45%

decompiler/util/TP_Type.cpp
3
54.11%

Totals

Change from base Build 932307752:
-0.02%

Covered Lines:
36419

Relevant Lines:
52372

💛  - Coveralls


Open Public Folder By DN

Added the ability the Open a PF by a DN

@drvespa  - you gonna fix the issues or were you expecting me to merge it and then fix them myself?


[Select] Add aria-label and aria-labelledby to hidden select input

[x] I have followed (at least) the PR section of the contributing guide.

This PR resolves #30085
Before:

After:

According to https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-label, hidden elements should not have the aria-label attribute:

Not all elements can be given an accessible name. Neither aria-label nor aria-labelledby should be used with non-interactive elements or inline structural role such as with code, term, or emphasis nor roles whose semantics will not be mapped to the accessibility API, including presentation, none, and hidden. The aria-label attribute is intended for interactive elements only.

According to https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes/aria-label, hidden elements should not have the aria-label attribute:

Not all elements can be given an accessible name. Neither aria-label nor aria-labelledby should be used with non-interactive elements or inline structural role such as with code, term, or emphasis nor roles whose semantics will not be mapped to the accessibility API, including presentation, none, and hidden. The aria-label attribute is intended for interactive elements only.

Oh, I didn't see that. So the problem described in the issue is just a false positive?

It seems so.

I'm closing the issue. If you still feel the current behavior is incorrect, we can discuss and possibly reopen if the arguments are valid.


Zipformer training with recent Dockerimage fails using DDP

We are currently trying to train a zipformer model on a new machine.
However (using the latest docker image) it fails with the following traceback:
2023-11-23 13:20:46,754 INFO [train.py:1124] (0/2) About to create model
2023-11-23 13:20:47,019 INFO [train.py:1128] (1/2) Number of model parameters: 27596865
2023-11-23 13:20:47,025 INFO [train.py:1128] (0/2) Number of model parameters: 27596865
2023-11-23 13:20:47,183 INFO [train.py:1143] (1/2) Using DDP
2023-11-23 13:20:47,404 INFO [train.py:1143] (0/2) Using DDP
Traceback (most recent call last):
 File "/local/path/to/icefall/egs/librispeech/ASR/./zipformer/train.py", line 1389, in <module>
   main()
 File "/local/path/to/icefall/egs/librispeech/ASR/./zipformer/train.py", line 1380, in main
   mp.spawn(run, args=(world_size, args), nprocs=world_size, join=True)
 File "/opt/conda/lib/python3.10/site-packages/torch/multiprocessing/spawn.py", line 246, in spawn
   return start_processes(fn, args, nprocs, join, daemon, start_method="spawn")
 File "/opt/conda/lib/python3.10/site-packages/torch/multiprocessing/spawn.py", line 202, in start_processes
   while not context.join():
 File "/opt/conda/lib/python3.10/site-packages/torch/multiprocessing/spawn.py", line 145, in join
   raise ProcessExitedException(
torch.multiprocessing.spawn.ProcessExitedException: process 1 terminated with signal SIGABRT

Training the e.g. yesno example on GPUs works just fine.
This can be reproduced using an official image without any customizations:
docker run -it --runtime=nvidia --shm-size=2gb --name=icefall --gpus all k2fsa/icefall:torch2.1.0-cuda12.1 /bin/bash
cd /workspace/icefall/egs/librispeech/ASR/
./prepare.sh
CUDA_VISIBLE_DEVICES="0,1" ./zipformer/train.py \
 --world-size 2 \
 --num-epochs 30 \
 --start-epoch 1 \
 --use-fp16 1 \
 --exp-dir zipformer/exp \
 --causal 1 \
 --full-libri 0 \
 --max-duration 200 \
 --num-encoder-layers 2,2,3,3,2,2 \
 --feedforward-dim 512,768,768,768,768,768 \
 --encoder-dim 192,256,256,256,256,256 \
 --encoder-unmasked-dim 192,192,192,192,192,192

I tried the above command with other images and some did work:

custom nix setup: torch.2.1.0-cuda12.1 (python 3.11.6) -> not working
official k2fsa/icefall:torch2.1.0-cuda12.1 (python 3.10.13) -> not working
official k2fsa/icefall:torch2.1.0-cuda11.8 (python 3.10.13) -> not working
official k2fsa/icefall:torch2.0.0-cuda11.7 (python 3.10.0) -> working
official k2fsa/icefall:torch1.9.0-cuda10.2 (python 3.7.10) -> working
official k2fsa/icefall:torch1.13.0-cuda11.6 (python 3.9.12) -> working
official k2fsa/icefall:torch1.12.1-cuda11.3 (python 3.7.13) -> working

The system under test

runs alma 9.2
2x NVIDIA GeForce RTX 2080 SUPER
Driver Version: 535.104.12
CUDA Version: 12.2

Any idea whats going on?
Is this a known issue with recent pytorch 2.1.0?
I first thought that it could be due to a new CUDA, but I don't understand why the yesno example should run if it is related to CUDA.
I am very grateful for any advice.

torch.multiprocessing.spawn.ProcessExitedException: process 1 terminated with signal SIGABRT

Could you post the output of dmesg immediately after you get the above error?

I get the following logs after starting the docker container:
[Thu Nov 23 15:53:52 2023] docker0: port 2(veth7e3bc5c) entered disabled state
[Thu Nov 23 15:53:52 2023] veth766da7b: renamed from eth0
[Thu Nov 23 15:53:52 2023] docker0: port 2(veth7e3bc5c) entered disabled state
[Thu Nov 23 15:53:52 2023] device veth7e3bc5c left promiscuous mode
[Thu Nov 23 15:53:52 2023] docker0: port 2(veth7e3bc5c) entered disabled state
[Thu Nov 23 15:53:59 2023] docker0: port 2(vethcdb2b28) entered blocking state
[Thu Nov 23 15:53:59 2023] docker0: port 2(vethcdb2b28) entered disabled state
[Thu Nov 23 15:53:59 2023] device vethcdb2b28 entered promiscuous mode
[Thu Nov 23 15:53:59 2023] eth0: renamed from veth6f5933d
[Thu Nov 23 15:53:59 2023] IPv6: ADDRCONF(NETDEV_CHANGE): vethcdb2b28: link becomes ready
[Thu Nov 23 15:53:59 2023] docker0: port 2(vethcdb2b28) entered blocking state
[Thu Nov 23 15:53:59 2023] docker0: port 2(vethcdb2b28) entered forwarding state

After that I don't see any further entries, although i have started the training and it has crashed again.
I'm not sure if it makes a difference here but I call dmesg as non-root user.

I get the following logs after starting the docker container:

Please post the logs immediately after the error, not after the start.

Please post the logs immediately after the error, not after the start.

There are no new entries after the training crashed …

--shm-size=2gb

Can you try a larger value?

--shm-size=2gb
Can you try a larger value?

I tried with --shm-size=20gb but same as before. It works for 'the old' imaged but not for the newer ones and there are no entries in dmesg

I have just done the experiment on another machine:
Same results:
Running zipformer for minilibrispeech using the k2fsa/icefall:torch1.12.1-cuda11.3 docker image works fine, running it with k2fsa/icefall:torch2.1.0-cuda12.1 gives
torch.multiprocessing.spawn.ProcessExitedException: process 1 terminated with signal SIGABRT

There are still no entries in dmesg
System under test:

centOS 7
2x NVIDIA A40
Driver Version: 545.23.06
CUDA Version: 12.3

Is it working for someone else?

From my past experiments with the 'official' Dockerimages we know:

official k2fsa/icefall:torch2.0.0-cuda11.7 -> working

python3.10.0

official k2fsa/icefall:torch2.1.0-cuda11.8 -> not working

python3.10.13

So my next idea was to find whether the bug comes from cuda11.7 -> cuda11.8 or from torch2.0.0 -> torch2.1.0.
Because I coudn't find a baseimage with torch2.0.0 and cuda11.8 I switched back to my custom nix workflow for further debugging.
At first I created a nix devShell with torch2.0.0 and cuda11.7 and like using the official docker image, training works fine.
After that I pumped cuda to cuda11.8 (+torch2.0.0 and python310) and still everything works fine.
So I bumped torch to torch2.1.0 (+cuda11.8 and python310) and I can reproduce the known error again (but with a longer traceback):
2023-12-01 11:16:57,596 INFO [train.py:1124] (1/2) About to create model
2023-12-01 11:16:57,886 INFO [train.py:1128] (0/2) Number of model parameters: 27596865
2023-12-01 11:16:57,893 INFO [train.py:1128] (1/2) Number of model parameters: 27596865
2023-12-01 11:16:58,040 INFO [train.py:1143] (1/2) Using DDP
2023-12-01 11:16:58,206 INFO [train.py:1143] (0/2) Using DDP
terminate called after throwing an instance of 'c10::Error'
  what():  Error: cannot set number of interop threads after parallel work has started or set_num_interop_threads called
Exception raised from set_num_interop_threads at ../aten/src/ATen/ParallelThreadPoolNative.cpp:54 (most recent call first):
frame #0: c10::Error::Error(c10::SourceLocation, std::string) + 0x57 (0x7fffccc87617 in /path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/
site-packages/torch/lib/libc10.so)
frame #1: c10::detail::torchCheckFail(char const*, char const*, unsigned int, char const*) + 0x68 (0x7fffccc42a56 in /path/to/local/workspace/.nix/_build
/pip_packages/lib/python3.10/site-packages/torch/lib/libc10.so)
frame #2: <unknown function> + 0x1820f2f (0x7fffafa20f2f in /path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/site-packages/torch/lib/libt
orch_cpu.so)
frame #3: <unknown function> + 0x706cba (0x7fffc5f06cba in /path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/site-packages/torch/lib/libto
rch_python.so)
<omitting python frames>

terminate called after throwing an instance of 'c10::Error'
  what():  Error: cannot set number of interop threads after parallel work has started or set_num_interop_threads called
Exception raised from set_num_interop_threads at ../aten/src/ATen/ParallelThreadPoolNative.cpp:54 (most recent call first):
frame #0: c10::Error::Error(c10::SourceLocation, std::string) + 0x57 (0x7fffccc87617 in /path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/
site-packages/torch/lib/libc10.so)
frame #1: c10::detail::torchCheckFail(char const*, char const*, unsigned int, char const*) + 0x68 (0x7fffccc42a56 in /path/to/local/workspace/.nix/_build
/pip_packages/lib/python3.10/site-packages/torch/lib/libc10.so)
frame #2: <unknown function> + 0x1820f2f (0x7fffafa20f2f in /path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/site-packages/torch/lib/libt
orch_cpu.so)
frame #3: <unknown function> + 0x706cba (0x7fffc5f06cba in /path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/site-packages/torch/lib/libto
rch_python.so)
<omitting python frames>

Traceback (most recent call last):
  File "/path/to/local/workspace/icefall/egs/librispeech/ASR/./zipformer/train.py", line 1389, in <module>
    main()
  File "/path/to/local/workspace/icefall/egs/librispeech/ASR/./zipformer/train.py", line 1380, in main
    mp.spawn(run, args=(world_size, args), nprocs=world_size, join=True)
  File "/path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/site-packages/torch/multiprocessing/spawn.py", line 246, in spawn
    return start_processes(fn, args, nprocs, join, daemon, start_method="spawn")
  File "/path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/site-packages/torch/multiprocessing/spawn.py", line 202, in start_processes
    while not context.join():
  File "/path/to/local/workspace/.nix/_build/pip_packages/lib/python3.10/site-packages/torch/multiprocessing/spawn.py", line 145, in join
    raise ProcessExitedException(
torch.multiprocessing.spawn.ProcessExitedException: process 1 terminated with signal SIGABRT

Does anyone have any idea what could be going wrong here?
Is it my environment or is it a general problem of k2/icefall with pytorch2.1.0 ?
(BTW: Still no information in dmesg)

This issue should be fixed in #1424
Please use the latest master.
(Note you can reuse your existing docker image. You only need to run git pull inside your container
to use the latest code)

Thanks, works now.


criteria for tests

Set tolerance as 1 capacity unit for tests with not many facilities

move legend for all plots


Multiple ruby version suport

close: #3

@kamontia
please review and merge

OK. it seems to work correctly.


Add functions for piste routes

This adds processing for giving a kind to piste routes, and a post-process filter for removing routes tagged as abandoned.
Connects to mapzen/vector-datasource#342.
@rmarianski could you review, please?

:+1:
I wonder if it makes sense to add support to the individual transforms/filters themselves to signal that a feature should be dropped, eg returning None for the shape. I almost added this for something in the past, but it got implemented in a different way so I never added support for it.

Yeah, I think it makes sense to do that - we've got more than one filter now which is about dropping certain features. It's pretty easy to do with the post-processor, but I always feel like that's a pretty "heavyweight" solution to a "lightweight" problem.

I made a new issue to capture this: https://github.com/mapzen/TileStache/issues/99


Lilv is insanely slow for some reason

I don't have that many plug-ins, but Lilv is making all my LV2-using apps having a very slow startup, like 8 seconds when I try to profile. Lilv 0.24.20 as shipped by Arch Linux
Here's a profiler trace:

I have 78 megabytes of .ttl files in my /usr/lib which isn't that much for my computer (i7-8750H with 32GB of ram), my hunch is that there's some quadratic behaviour occuring somewhere.

For hosts that load everything at startup (which is unfortunately pretty common due to getting the name, which is the case here) it will take a bit of time, but 8 seconds on a machine like that is certainly a lot more than I'd expect.  I have considerably more data overall installed and even debug builds take nowhere near that long.
I should consider building in profiling to make it easier to see where the time goes, because the data structure of plugins themselves can affect this dramatically (e.g. including all presets in the main data file), but there's nothing like that at the moment.

my hunch is that there's some quadratic behaviour occurring somewhere

Doubtful, unless the host is really going out of its way to make that happen.  This is probably several hundred thousand, if not millions, of statements.  If anything was quadratic it would be a lot slower than 8 seconds.  What's happening here is loading all of the statements into B-trees, which is roughly O(nlgn).
Anyway, please attach:

Which host(s) you're using
Which plugins you have installed (output of lv2ls will do, package list would be handy if you can get it)
The above profile with a time legend if possible, and details about what exactly it is measuring
What kind of drive this is on, and anything non-stock about the system


‘Cannot create __weak reference in file using manual reference counting’

Pod load ReactiveObjC compile errors：
Cannot create __weak reference in file using manual reference counting
To solve：
Set Weak References in Manual Retain Release:YES

yes,I have also encountered this problem, I am introducing class (MRC).


Add petting cyborgs

About the PR

Makes it possible to pet borgs by clicking them, just like petting pets or bots. Also makes clicking borgs no longer unlock them (you have to alt+click to unlock instead)
Why / Balance

Suggested at https://forum.spacestation14.com/t/it-should-be-possible-to-pet-borgs-and-mice
Also accidently unlocking yourself while playing as a cyborg or having others accidently unlock you just by clicking stuff is a major inconvenience and this change fixes that.
Technical details

None
Media

https://github.com/user-attachments/assets/ba46c572-0ab7-45dd-80e9-f28ca2c1b4b8

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

None, changed a couple yml's and 1 ftl
Changelog

:cl: HahayesSiH

add: It is now possible to pet cyborgs.
tweak: Clicking on cyborgs and opening the strip menu no longer unlocks them.

might be a hot take but I feel like there should also be some way for borgs to 'block' being pet if they so choose
I know more than a couple borg mains from vanilla ss13 who are very put off when people just spam-pet them unprompted


fix(deps): update dependency web-vitals to v3

This PR contains the following updates:

Package
Type
Update
Change

web-vitals
dependencies
major
^2.1.4 -> ^3.0.0

Release Notes

GoogleChrome/web-vitals (web-vitals)
v3.4.0
Compare Source

Make bindReporter generic over metric type (#359)
Update INP status in README (#362)
Fix Metric types for better TypeScript support (#356)
Fix selector for SVGs for attribution build (#354)

v3.3.2
Compare Source

Fix attribution types (#348)
Safe access navigation entry type (#290)

v3.3.1
Compare Source

Export metric rating thresholds in attribution build as well.

v3.3.0
Compare Source

Export metric rating thresholds, add explicit MetricRatingThresholds type (#323)
Trim classname selector (#328)
Add link to CrUX versus RUM blog post (#327)
Prevent LCP being reported for hidden prerendered pages (#326)
Add Server Timing information to docs (#324)
Fix link in onINP() thresholds comment (#318)
Update web.dev link for onINP() (#307)
Add a note about when to load the library (#305)

v3.1.1
Compare Source

Defer CLS logic until after onFCP() callback (#297)

v3.1.0
Compare Source

Add support for 'restore' as a navigationType (#284)
Report initial CLS value when reportAllChanges is true (#283)
Defer all observers until after activation (#282)
Ignore TTFB for loads where responseStart is zero (#281)
Defer execution of observer callbacks (#278)

v3.0.4
Compare Source

Clamp LCP and FCP to 0 for prerendered pages (#270)

v3.0.3
Compare Source

Ensure attribution object is always present in attribution build (#265)

v3.0.2
Compare Source

Set an explicit unpkg dist file (#261)

v3.0.1
Compare Source

Use the cjs extension for all UMD builds (#257)

v3.0.0
Compare Source

[BREAKING] Add a config object param to all metric functions (#225)
[BREAKING] Report TTFB after a bfcache restore (#220)
[BREAKING] Only include last LCP entry in metric entries (#218)
Update the metric ID prefix for v3 (#251)
Move the Navigation Timing API polyfill to the base+polyfill build (#248)
Add a metric rating property (#246)
Add deprecation notices for base+polyfill builds (#242)
Add a new attribution build for debugging issues in the field (#237, #244)
Add support for prerendered pages (#233)
Rename the ReportHandler type to ReportCallback, with alias for back-compat (#225, #227)
Add support for the new INP metric (#221, #232)
Rename getXXX() functions to onXXX() (#222)
Add a navigationType property to the Metric object (#219)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: frontend/package-lock.json

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Update Glossary tab

Create an informative issue (See each section)
Using the current documentation as a model, please complete the issue template.
Prerequisite: Start with what we have
Always look at the current website to describe the change that is needed. Use the following link:
AgStack website
Describe the changes in the doc and link to your dev story
Provide info for the following steps:

[x] Mandatory Choose the page that needs to be updated:

[ ] About
[ ] Contribute
[ ] Projects (specify project name)
[ ] APIs
[ ] Resources
[x] Glossary
[ ] New topic

[ ] Mandatory for bugs: Clearly define what the problem is, what the change is, and link to the page:

[ ] Mandatory for content:

[ ] Add steps and/or other important conceptual information here:
[ ] Add verification at the end of the task, how does the user verify success (a command to run or a result to see?)
[ ] Add link to related dev story, if applicable

(Should we group the terms? Technical vs Agriculture?)
List of words that we want to add:

asset: Property owned by the user.
GeoID: Unique geography ID for an asset.

Looking for help from contributors, maintainers, and community.
(Should we group the terms? Technical vs Agriculture?)
List of words that we want to add:
asset: Property owned by the user.
GeoID: Unique geography ID for an asset.

moved this issue to Jira, closing this issue


could you provide Python code for  generation data

could you provide Python code for  generation data

Thanks for your message. You can find the details in the corresponding article available at this link.

@sam-1219 probably this link helps: https://docs.sdv.dev/sdv/single-table-data/modeling/synthesizers/ctgansynthesizer


UI: Enhance DateTime selector with reset and date navigation

TL;DR
Enhanced the date-time selector UI with improved formatting, navigation controls, and reset functionality.
What changed?

Added date navigation with next/previous controls and date bounds
Implemented a reset button to return to current date and time
Created new utility functions for date formatting and manipulation
Enhanced time display with proper AM/PM formatting
Updated typography and styling for better visual hierarchy
Added a "Done" button with themed styling
Removed redundant SelectedDateTimeRow component

Screenshots
https://github.com/user-attachments/assets/b623ac06-40be-49ed-81b1-d58886da2e62
Why make this change?
To provide a more intuitive and user-friendly date-time selection experience with better visual feedback and navigation controls. The changes improve accessibility and make it easier for users to select and modify their desired date and time.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#398  👈 (View in Graphite)
#397
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Merge activity

Nov 29, 9:07 AM EST: A user merged this pull request with Graphite.


Please consider to add about naming conversations for Fire Wall

Please consider to add the naming conversation for Azure Firewall's Network rule and application rule.

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

ID: 0e540bb6-9f72-0529-28ea-ae82bc869835
Version Independent ID: 24e71392-985b-59fb-1552-46395b169612
Content: Naming conventions for Azure resources - Best practices for cloud applications
Content Source: docs/best-practices/naming-conventions.md
Service: architecture-center
Sub-service: cloud-fundamentals
GitHub Login: @telmosampaio
Microsoft Alias: pnp

@KeiroKamioka0000 Thanks for the feedback! We will review and get back to you.

As additional information, current portal does not mention about maximum number of fonts can be used.

@KeiroKamioka0000 Thank you for providing details. I have assigned the issue to the content author to evaluate and update as appropriate.

@MikeWasson
Hi, how this goes?
Can you confirm up limit are 80 or 64 words?
thanks.

We've confirmed with PG that the limitation on these is that it must be unique to your tenant, must begin with a letter or number, end with a letter number or underscore and may contain only letters, numbers, underscores, periods or hyphens. There is no length limit. Additionally, for fun, Unicode is now supported :)


refactor: rename address pq

New name better implies this being a transaction index, rather than an
address index.

Stack:

#318
#317
#316
#315
#314 ⬅
#313

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

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

Project coverage is 76.98%. Comparing base (3f1a3be) to head (1d03cb1).

Additional details and impacted files
@@                Coverage Diff                 @@
##           spr/main/30696bc0     #314   +/-   ##
==================================================
  Coverage              76.98%   76.98%
==================================================
  Files                     28       28
  Lines                   1382     1382
  Branches                1382     1382
==================================================
  Hits                    1064     1064
  Misses                   265      265
  Partials                  53       53

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


feat(analytics): add self hosted plausible analytics

Closes #625

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

38 of 164 (23.17%)  changed or added relevant lines in 9 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.4%) to 3.566%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/pages/elements/Tiles.jsx
0
1
0.0%

src/components/Root.jsx
0
2
0.0%

src/components/nav/GameNav.jsx
0
4
0.0%

src/components/ExportButton.jsx
0
8
0.0%

src/components/Analytics.jsx
0
10
0.0%

src/components/File.jsx
0
10
0.0%

src/components/pages/Home.jsx
0
11
0.0%

src/util/analytics.js
38
65
58.46%

src/util/query.js
0
53
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/util/query.js
1
0.0%

src/components/File.jsx
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>7:
0.4%

Covered Lines:
512

Relevant Lines:
16221

💛  - Coveralls


Consolidate HTTP and TCP Client Settings with global DestinationRule

Support DestinationRule inheritance from mesh/namespace level DestinationRules to allow admin and users to configure global settings that may otherwise require control plane restart or special permissions.
Behavior is controlled via feature flag, default OFF.
Related Design doc/discussion: here
TL;DR summary: here
Related Issue:
https://github.com/istio/istio/issues/29525
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ X ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@GregHanson: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
db260df3ca272d54e2dafb926a95257f96eb0233
link
/test release-notes_istio

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. I understand the commands that are listed here.

@GregHanson: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
db260df3ca272d54e2dafb926a95257f96eb0233
link
/test release-notes_istio

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. I understand the commands that are listed here.

@GregHanson: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
db260df3ca272d54e2dafb926a95257f96eb0233
link
/test release-notes_istio

unit-tests_istio
db260df3ca272d54e2dafb926a95257f96eb0233
link
/test unit-tests_istio

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. I understand the commands that are listed here.

@GregHanson: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
db260df3ca272d54e2dafb926a95257f96eb0233
link
/test release-notes_istio

unit-tests_istio
db260df3ca272d54e2dafb926a95257f96eb0233
link
/test unit-tests_istio

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. I understand the commands that are listed here.

@GregHanson: PR needs rebase.

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.

@GregHanson: PR needs rebase.

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.

@howardjohn PTAL with the latest changes

@howardjohn PTAL with the latest changes

@GregHanson: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
388c9d030b3ff124ccb7589fda969f2bd515c87d
link
/test lint_istio

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. I understand the commands that are listed here.

LGTM, minor env naming suggestion.

@GregHanson: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
ea52b464a35c88df41b0dbacc6ca10b025727cad
link
/test integ-pilot-multicluster-tests_istio

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. I understand the commands that are listed here.

/retest

@GregHanson: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-telemetry-mc-k8s-tests_istio
ea52b464a35c88df41b0dbacc6ca10b025727cad
link
/test integ-telemetry-mc-k8s-tests_istio

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. I understand the commands that are listed here.

/test integ-telemetry-mc-k8s-tests_istio

In response to a cherrypick label: #30066 failed to apply on top of branch "release-1.9":
Applying: add feature flag for destination rules inheritance
Using index info to reconstruct a base tree...
M	pilot/pkg/features/pilot.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/features/pilot.go
CONFLICT (content): Merge conflict in pilot/pkg/features/pilot.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 add feature flag for destination rules inheritance
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #30657

@nrjpoddar yes, the proxy's namespace is used for determining inheritance


Add Apache License

Please add this so that GitHub will identify the project as Apache licensed

Oops. Thanks!


add unlink() system call

This change is

@qsn: thanks for your guidance helping me implement this! :-)

@bollmann do plan to finish this PR? Maybe you need some help with it?


Submit and Download CSV buttons on the homepage should be separated slightly (Download CSV button should be moved down slightly)

User Story
As a user I can clearly click on the Submit and Download CSV buttons.
Acceptance Criteria

[ ] There is noticeable gap between Submit and Download CSV buttons

Implementation Todos
Front end:

[ ] Add a marginTop: "1rem" to style property in Download CSV Button

Testing:

All previous tests still pass

Guidance/Information:

Changes should be in javascript/src/main/pages/Home/Home.js
Check more on inline styling here

The name of our branch is 5pm-3-fixButtonSpacing.

Just an FYI, you shouldn't need to close issues manually unless you're deliberately choosing to not implement the issue. If you link the issue to its corresponding PR (either using a closing keyword or by adding it on the right sidebar), and the PR gets merged into the main branch, the issue will close itself automatically.


Modified types, improved options setting for zip function

Improved types for
TODO: Need better return type definition for
Added JSZip specific options for zip command to push type requirement to end user
Use file-saver for downloads

@charlieforward9 I have made some changes to your commit. I pushed out the type option so that we don't need to deal with the node case you called out. This also doesn't hardcode base64 the way you had it
I also abstracted the compression setting because we've been using DEFLATE but I want to keep the library as close to how it operated before (no type specified, DEFLATE compression). If you have a better way to define the types, let me know. I just copied from jszip to get better checking since my tsconfig isn't doing lib checks
The only thing I don't know is how best to type the return type for the function based on your streaming addition.
I also changed to use file-saver because this is a change we've used for a long time and it's been very reliable for us.
Really appreciate you pulling this together! I'll try to close and credit other PRs if we can get yours merged

my use case is very simple. My goal is to click a button that prepares the output, and call download(output, {         folder: "treesShp",         filename: fileName,         types: {           point: "mypoints",           polygon: "mypolygons",           line: "mylines",         },       });
  to get the output. With this implementation, I cannot add `options.type`, since it is a property of the `zip` function, not the `download`. Therefore, either `options.type` needs to also be added to the download function props or set to `base64` by default.

The only thing I don't know is how best to type the return type for the function based on your streaming addition.
I also changed to use file-saver because this is a change we've used for a long time and it's been very reliable for us.
Really appreciate you pulling this together! I'll try to close and credit other PRs if we can get yours merged.
If we can make these changes I feel more confident pulling your PR into the main shp-write repo. Mapbox let me take it over but I haven't had time to do much after the dbf updates.

We can leave the ReadableStream in, we didn't have any types before so this is definitely better than that at least, and it shouldn't break any existing use cases that would all expect a Promise.
As to your use case, it appears that the download function passes the options object to the zip function so we should really pass the same interface to both anyways, and then you could specify the type.
I decided to just go ahead and separate the zip options either way, and then also add them as an optional to the download function.
I would note that I actually realized we are NOT using the download function like I thought we were. We've just been generating the artifact from the zip function and using downloadjs to get the file. We could make the case to remove the download function entirely since it's provided so nicely by other libraries (like file-saver and downloadjs)
See the current changes and let me know what you think.

I decided to just go ahead and separate the zip options either way, and then also add them as an optional to the download function.

This is a DX regression in my opinion.

Uh oh, I thought it was good, but I am not getting this error:

Heres what I have implemented:
const output: GeoJSON.FeatureCollection = {
  type: "FeatureCollection",
  features: features,
};
const options: ZipOptions & DownloadOptions = {
  compression: "STORE", // Or whatever compression you want
  outputType: "blob",
  types: {
    point: "mypoints",
    polygon: "mypolygons",
    line: "mylines",
  },
};

const zippedGaps = await zip<"blob">(output, options);
const blob = new Blob([zippedGaps], { type: "application/zip" });
download(blob);

download(blob: Blob, fileName?: string) {
  // Create a download link
  const a = document.createElement("a");
  if (fileName) {
    a.download = fileName;
  }
  a.href = window.URL.createObjectURL(blob);

  // Append the link to the document
  document.body.appendChild(a);
  a.click(); // Trigger download

  // Clean up
  document.body.removeChild(a);
  window.URL.revokeObjectURL(a.href);
}

When I use FileSaver, same issue
  download(blob: Blob, fileName?: string) {
    FileSaver(blob, fileName);
  }

When expecting string,
const options: ZipOptions & DownloadOptions = {
        compression: "STORE", // Or whatever compression you want
        outputType: "string",
        types: {
          point: "mypoints",
          polygon: "mypolygons",
          line: "mylines",
        },
      };
      const zippedTrees = await zip<"string">(output, options);

You're fine. In the original PR you sent, you had default outputType to base64, and your template type for the zip call should match, so to match what you had before, I believe you would call this
const options: ZipOptions & DownloadOptions = {
        compression: "STORE", // Or whatever compression you want
        outputType: "base64",
        types: {
          point: "mypoints",
          polygon: "mypolygons",
          line: "mylines",
        },
      };
      const zippedTrees = await zip<"base64">(output, options);

With this version, we actually call it like this (this is a combination of functions so it is all in one place, so forgive the variables for the names and the object expansion to get zip options)
import shpwrite from 'shp-write';
import download from 'downloadjs';

const options = {
  folder: this.name,
  types: {
    point: `${this.name}_pts`,
    polygon: `${this.name}_bnd`,
  }
};

...

const shpZip = await shpwrite.zip<"blob">(geoJSON, { ...options, outputType: 'blob', compression: 'DEFLATE' });

download(shpZip, `${mission.name}_pts+bnds.zip`);

Let me know if that helps you out. I think the default of STORE was because of an old bug with the older jszip library.

This is the original line that was in your changes for the zip options
https://github.com/Agriculture-Intelligence/shp-write/blob/master/src/zip.js#L39

const options: ZipOptions & DownloadOptions = {
        compression: "STORE",
        outputType: "base64",
        ...
      };
      const zippedTrees = await zip<"base64">(output, options);

Yes, I tried this originally, which is why I decided to test the Blob and the string outputs. Not sure why none of them are working, I did not change anything about how I prepared my GeoJson.FeatureCollection beforehand...

I think it would be worth it to try DEFLATE vs STORE, I had problems with
my STORE zip when you made that the default (same result as you, seemingly
corrupt zip) and had better results with DEFLATE
On Sun, Aug 20, 2023, 8:03 PM Charles Richardson @.***>
wrote:

const options: ZipOptions & DownloadOptions = {
compression: "STORE",
outputType: "base64",
...
};
const zippedTrees = await zip<"base64">(output, options);
Yes, I tried this originally, which is why I decided to test the Blob and
the string outputs. Not sure why none of them are working, I did not change
anything about how I prepared my GeoJson.FeatureCollection beforehand...
[image: Screen Shot 2023-08-20 at 6 53 35 PM]
https://user-images.githubusercontent.com/62311337/261880107-09cbb410-7c70-446d-9721-f1ab9b31a5da.png
—
Reply to this email directly, view it on GitHub
https://github.com/Agriculture-Intelligence/shp-write/pull/8#issuecomment-1685432912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANLCVVFKLIXOTFX5F6BWL3XWKQWHANCNFSM6AAAAAA3VVRCCI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I got 'unsupported format' using the "DEFLATE"

const zippedTrees = await zip<"string">(output, {
        compression: "DEFLATE",
        outputType: "string",
        types: {
          point: "mypoints",
          polygon: "mypolygons",
          line: "mylines",
        },
      });

This worked. But it looked like it deflated a bit too much. Not sure if this is a bug in our own code or JSZip.
The names of the properties viewed through Shapefile viewing softwares are not matching what I am specifying in my source code.
Shapefile Viewers

Source Code

You'll notice this is being truncated to 10 characters - this is a limitation of the underlying .dbf format that field names are limited to 10 characters.
I'm not sure about the compression format, but I'm glad you've found a configuration that's working. I'm not sure I know enough about the underlying compression formats (STORE v DEFLATE) to be of much help but we can do some side by side testing if it would be helpful.

limitation of the underlying .dbf format that field names are limited to 10 characters

This is another good piece to add to the ReadMe docs.

do some side by side testing

I like the sound of this, but shp-write is working better than it was when we both got to working on it, so lets make this merge with and add this as an Issue with an RFC. What do you think?

Works for me! Sure, we can add a link. README clean to this and dbf (https://github.com/mapbox/dbf) is definitely something I want to do more of. I'll work on merging this into the main repo!


configuring javahome with org.gradle.java.home

Expected Behavior
Can the runtime environment of Gradle be separated from the compilation and execution environment of the program
When using org.gradle.java.home configuration, an error will be directly reported when the execution environment of gradle is incompatible with the project jdk;
Current Behavior (optional)
And the configuration of org.gradle.java.home seems to be related to caching, which can lead to success and failure
Context
Old version gradle cannot compile jdk21

Sorry that you're having trouble with Gradle!
Your issue appears to be a question about how to use Gradle. Your question might be answered in Gradle documentation or on our community Slack or forums. Find links to these resources on help.gradle.org.
If you think you have come across a Gradle bug, please file a new issue describing the problem and how to reproduce it.
If you think Gradle is missing a feature, please file a new issue describing your use case for the feature.

Probably what you need is Java toolchains: https://docs.gradle.org/current/userguide/toolchains.html
If we misunderstood your need, let us know and we can reopen the issue.


Support v24.2.0 compatibility

Since updating my support library dependencies to 24.2.0, I am encountering the following error when trying start my IssueReporterActivity subclass:

java.lang.ClassCastException: android.widget.FrameLayout cannot be cast to android.support.design.widget.TextInputLayout
at com.heinrichreimersoftware.androidissuereporter.IssueReporterActivity.setGuestEmailRequired(IssueReporterActivity.java:298)

The error occurs when calling setGuestEmailRequired(true) in the IssueReporterActivity's onCreate() method. Omitting the method call prevents the error from occurring.
Everything was working fine on support v23.1.1, so it seems there was an internal change in TextInputEditText in the new version of the design support library?
Thanks again for your work, and please let me know if you need help with this.
I believe the easiest fix would be to add an ID to the enclosing TextInputLayout for the air_inputEmail TextInputEditText and use findViewById() to access it instead of casting inputEmail.getParent().

Good spot! I'll look into that


Fix section component flicker

Before Alpine.js loads, a programmatically collapsed section component briefly shows its contents. I've added an x-cloak attribute via Blade to prevent this from happening.
Before:

Additionally, for the same reason, the disclosure indicator for a collapsed section is briefly rotated 180º. I've included the correct class via Blade, which on subsequent changes is handled by Alpine.
Before:

(Apologies the GIFs are so short – each loop is a page refresh)

Perfect, thank you!

No worries!


fix(RN): correct disconnect button color in light mode and other minor UI adjustments

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

Contributor NFT
Paste in your wallet address below and we will airdrop you a special NFT when your pull request is merged.
Address:

PR-Codex overview
This PR fixes the disconnect button color in light mode and adds a placeholder for implementing the transactions screen in a React Native application.
Detailed summary

Fixed disconnect button color in light mode
Added a placeholder for implementing the transactions screen in ConnectedModal.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#3748  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @joaquim-verges and the rest of your teammates on  Graphite


fix: handle getaddrinfo errors when checking dev server

Description 📖
Fixes #dev_server_running? failing with an exception in case hostname is not resolveable.
Background 📜
That's, for example, the case when using a dockerized development environment (and not running a vite-dev server at all).
The Fix 🔨
I suggest removing the exception class filter altogether like Webpacker does.
Screenshots 📷

Usually a dockerized setup would also start the Vite dev server, but I don't see a reason to be strict about the check.
Thanks for the fix!


Don't require password field for PUT /users/

At the moment, we get the following response:
HTTP/1.0 400 Bad Request
{"password":["This field is required."]}

I'm not sure that this is an error? A PUT request should update the whole resource, you can use a PATCH request to only update certain bits?

In general I think this is true, but I think passwords are a bit of an edge case.
I think a very common case for PUT is to first get the resource's description with a GET, update a field, then send the description back using PUT. We can't do this for users because we don't get the password field back since it is write only.

@rudigiesler ready for review

Just the one comment, looks good otherwise.

Ready for review.

Ready for re-review.

:+1:


FindAll and Query.All() not returning all results

Hi,
I have a table with just over 10000 entries in it. I can view the table just fine with the LiteDB viewer, however when I try and get all documents in a table, I get a very small subset of the total documents.
Here's the code I'm using to get all results:
public static IEnumerable<TestResult> GetAllResults(IDatabaseSettingsProvider settingsProvider)
{
    using (var db = new LiteDatabase(settingsProvider.DatabaseFilePath))
    {
        var testResults = db.GetCollection<TestResult>(settingsProvider.TableName);
        return testResults.FindAll();
    }
}

Am I doing something wrong? I've attached the database in question. Please help!
Thanks!
Bobby
Database.zip

Hi @bobbyg603, your database are ok. I open here and has 14858 docs in shell tool. How are you calling this "GetAllResults" method?

If you problem persist, please re-open issue


base64 decode binary metadata

According to https://github.com/grpc/grpc-go/blob/master/Documentation/grpc-metadata.md#storing-binary-data-in-metadata -bin metadata is base64 encoded in gRPC, but it's not possible to represent binary values in json. This assumes that any -bin metadata header should be decoded as base64 before passed along to gRPC.

Hello, thanks for the PR. This should be in 0.95.0.


Exclude Bin and Obj from CopyToOutputDirectory

The code to move JSON files to the output directory causes build failures after multiple builds - at least in .NetCore 2.1. The reason being that for each build the Include="***.json" syntax will pick up JSON files that exist in both the Bin and Obj folders - and maintain their file structure - but appending them to the root bin folders causing a recursive file structure problem. This eventually triggers a path to long exception in MSBuild.
After adding the current code to the project file (without the exclude condition)
<ItemGroup>   <Content Include="**\*.json"  CopyToOutputDirectory="PreserveNewest" /> </ItemGroup>
The following happens to the Bin folder:
Build1:
\bin\Debug\netcoreapp2.1*.json
\bin\Debug\netcoreapp2.1\obj*.json
Build2:
\bin\Debug\netcoreapp2.1*.json
\bin\Debug\netcoreapp2.1\obj*.json
\bin\Debug\netcoreapp2.1\bin\Debug\netcoreapp2.1*.json
\bin\Debug\netcoreapp2.1\bin\Debug\netcoreapp2.1\obj*.json
etc etc until the file path is too deep

Great catch @Kevinio04! :rocket: ... It's likely because Generic Host apps (at this point) don't use the Sdk.Web.

Happy to help, i'm impressed with the rapid merge. Good spot on the case of the folder names in the exclude. Updated my local to match :)


Add an include option for basicAuth

Is your feature request related to a problem? Please describe.
Currently, the basicAuth option accept only exclude routes options but I need to protect one API route with basic authentification (for webhook purpose).
Describe the solution you'd like
Add an include option for basicAuth to set the route I want to protect.
Describe alternatives you've considered
Maybe access to basicAuthoption inside routeRules option can be a proper alternative ?

In my personal opinion, the better approach would be to go with include option (as currently basic auth configuration is passed to private runtime config due to storing credentials).
@danielroe
Which option do you think would be better? Route Rules or maybe the include in the module configuration? (also right now, we support exclude option).

Added in https://github.com/Baroshem/nuxt-security/pull/212/commits/40c6a7245016311e42cea5a1c47d45f25ab772c5
Will be added in upcoming 1.0.0-rc.1 version :)

Nice job 🔥 I will use it for sure when it will be shipped. Thanks for your reactivity :)