fetchRingBufferSize should be global to all the created RingBufferThread in a JVM

Discussed in https://github.com/aws/amazon-redshift-jdbc-driver/discussions/48

Originally posted by odelpozog March 24, 2022
Hello
Some Denodo Technologies customers are facing java.lang.OutOfMemoryError issues executing Redshift queries in a concurrent way with the Reshift 2.x JDBC drivers. By default, the enableFetchRingBuffer feature is enabled and multiple instances of RingBufferThread are created with 1GB of max size when multiple queries are executed.
When this happens, as the buffers are filling up, the heap memory grows and, eventually, a java.lang.OutOfMemoryError is raised:

I think that the fetchRingBufferSize should limit the total JVM memory used by all the RingBufferThread instances, not the size of each RingBufferThread instance. Otherwise, JDBC clients with concurrent queries should not enable the RingBuffer feature to avoid possible OOM issues.

Thanks for reporting the issue. We will discuss internally and get back to you.

Any resolution for this?

@jhchee Sorry for the delay! I will look into this issue and get back to you once i have an update. Thanks for submitting it!

Hi @odelpozog  , while we will not be pursuing this issue in the near future, fixing this issue is part of our long term roadmap. We have decided to close it for now. It will be reopened when we come back to this issue. The Redshift Driver team appreciates your contribution and thanks you for bringing this to our attention!


IRGen: Handle protocol inheritance in witness_method lowering.

If the conformance in a witness_method instruction represented a derived protocol instead of the exact protocol of the desired method, we would crash. Handle this by drilling down to the exact conformance needed before forming a witness ref. Fixes rdar://problem/26633668.

@swift-ci Please test and merge


:seedling: fix ControlPlaneEndpoint in metal3cluster_manager

What this PR does / why we need it:
Avoid returning err with nil in error cases

/retest

/test-v1b1-ubuntu-integration

cc @kashifest @furkatgofurov7 @smoshiur1237

PTAL

would it be possible to squash the commits

I think these two commits fix two different problems. Should not be squashed.

/approve

@furkatgofurov7 @smoshiur1237 PTAL

@kashifest @furkatgofurov7 @smoshiur1237 PTAL


Create the project as a PWA

I would like to take this up.

I would like to take this up.

yes sir go ahead, assigned you
keep in mind we're using nextJS 13 experimental version here, if there are some updates on that


I don't want to see random event & the text above them, Entity hider hiding too much

I want to turn off random events. So turn off NPCs right?
Sure, now I don't see the NPC, but I see the messages these NPCs come with. So turn off NPCs 2d right?
Okay, I'm fishing but because 'fishing spot' is considered an NPC 2d, now I can't fish.
I'd like to not see random events and their messages, but I want to see fishing spots. Entity hider solves the random event issue, but then I can't see fishing spots.
Suggestion: Disable random events & their messages while allowing me interact with fishing spots or anything related to that nature.
Is it too petty of me to not want to see random events & their messages? I like my random events like my bloatware: non existent.
Edit: apparently sounds are still on when they're disabled. Like Evil Bob's random event sounds are annoying af.

Duplicate of #2558

random events are the equivalent of bloatware, they may not affect u much but its useless. dismissing them interrupts my actions. so no, its not a worthless sugg
Sent from my Verizon, Samsung Galaxy smartphone
-------- Original message --------From: Tomas Slusny<EMAIL_ADDRESS>Date: 8/11/18  4:32 PM  (GMT-06:00) To: runelite/runelite<EMAIL_ADDRESS>Cc: doxxide<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [runelite/runelite] I don't want to see random event & the text above them, Entity hider hiding too much (#4858)
Closed #4858.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/runelite/runelite","title":"runelite/runelite","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/runelite/runelite"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Closed #4858."}],"action":{"name":"View Issue","url":"https://github.com/runelite/runelite/issues/4858#event-1783932464"}}}
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/runelite/runelite/issues/4858#event-1783932464",
"url": "https://github.com/runelite/runelite/issues/4858#event-1783932464",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [runelite/runelite] I don't want to see random event \u0026 the text above them, Entity hider hiding too much (#4858)",
"sections": [
{
"text": "",
"activityTitle": "Tomas Slusny",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@deathbeam",
"facts": [
]
}
],
"potentialAction": [
{
"targets": [
{
"os": "default",
"uri": "https://github.com/runelite/runelite/issues/4858#event-1783932464"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "MuteNotification",\n"threadId": 367344269\n}"
}
],
"themeColor": "26292E"
}
]


General update

updated at 2024-09-24

Hello, I saw some changes related to Korean language in some commits, but there are also some commits that change the sample CV entries. Did you intend to only push some of these changes in this PR and not the others?

Sorry, I made wrong pull request. I thought I made my own repository. I made a mistake. Sorry for borthering you. T.T

No problem at all!


SQLAlchemy dialect native bool support

Should be
class DatabricksDialect(default.DefaultDialect):
    """This dialect implements only those methods required to pass our e2e tests"""

    supports_native_boolean: bool = True

as otherwise bools are converted to ints which results in datatype errors

Thanks for the tip. SQLAlchemy support hasn't merged into main yet so this won't be available generally. I'll apply this to our working branch.

Any idea when this branch will be merged into main?

Answered here: https://github.com/databricks/databricks-sql-python/pull/57#issuecomment-1411162710


System apps registration reset after userspace panic

iPhone 13 Pro
iOS 15.0.1
Dopamine 1.8

Sometimes this strange thing happens. I kept my phone charged last night and when I opened it the next morning, it went unjailbroken. I saw the "UserSpace Panic" notify which says Dopamine has protected me from a UserSpace panic, then aome system apps notified me with "WLAN has been disabled for xxxxxx". Then I found that all the sideloaded apps I installed by TrollStore(including TrollStore itself) cannot be opened(notify"no longer available"). So I used TrollStore persistence helper to refresh apps registration. Then I opened tweak injection from Dopamine. And everything was all right then.(Except that I have to switch all the "closed" registration to "WLAN and cellular" in Settings--Cellular)
log.zip
This zip file includes SpringBoard, installd, bulletin(I don't know what it is so I uploaded it), userspace-panic, SystemMemoryReset. There are many SpringBoard logs within these 3 minutes, so I put the last one.If you want me to send others, feel free to ask me.

There is absolutely no way to influence this as it's done by iOS itself and not by any part of the jaiilbreak.
The SpringBoard crash looks like it's coming from Ellekit's MobileSafety.dylib, so probably something crashed SpringBoard, it entered safe mode and from there was crash looping, unfortunately the log isn't symbolicated so it's hard to say what exactly caused it...


Add fingerprint for 2018 Nissan Leaf G (Japan)

Adds the fingerprint for the 2018 Nissan Leaf G (japanese version), which has the ADAS ECU behind the instrument cluster.

What's your dongle id?

fd3b0ef05a984a10

I see a whole bunch of messages in a drive from your device that are not in a fingerprint. They are probably low frequency, but maybe good to add them all so it also fingerprints if the car is already started:
Output from compare_fingerprints.py comparing to the current fingerprint:
1792 not in f2
1872 not in f2
1937 not in f2
1953 not in f2
1968 not in f2
1988 not in f2
2000 not in f2
2001 not in f2
2004 not in f2
2005 not in f2
2015 not in f2
2016 not in f2
2017 not in f2


bug: not redirecting to homepage

In reference to #33

I opened https://socialgram.netlify.app
I logged in.
Redirects me to - https://socialgram.netlify.app/homepage
Now, if I remove homepage from the URL, and run it again, it doesn't redirect me to the homepage, it takes me to the https://socialgram.netlify.app/ and I have to log in again.

This can be handled by checking if the user is logged in, then it should redirect them to https://socialgram.netlify.app/homepage.

Yes i want to address this I forgot to add this part in code. Please make sure you do so with the tokens or if you have better way .. go ahead.


Associate asc signatures, if any, with uploaded SBOM's

Could be more than one asc for each SBOM.
I'm thinking another path in the bucket, associated by id?

How about HTTP headers for this?

Digest? Apparently deprecated. Something else?


Hc 208

https://hysds-core.atlassian.net/browse/HC-208

had to fix the jinja templating in case ~/.sds/config doesnt have a GRQ_ES_PORT, GRQ_AWS_ES or GRQ_ES_PROTOCOL value


Update c_data_insertion_api.md

Added a note that references the BDIA.

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@nateprice7  - could you also merge this, please? I don't have rights to merge.


[Java] Apply linting format and enable CheckStyle rules

Apply CheckStyle to all Java-Recognizers.
Enable CheckStyle validation.

@enzocanoo, can you resolve the conflicts? Thanks.


convert directly into directory dump instead of image file

Hi Chris,
it would be great if one could have an option/flag to convert a Docker image into a Singularity directory in addition to the flat image file (which might make an automatized roll out to CVMFS easier)
Thanks,
Thomas

This should be possible and potentially would not be hard. Would you like to take a stab at it?

yes, I can give it a try. I had already peaked into docker2singularity.sh if one could do a rsync or so from a mounted image.
However, not in the coming two weeks, since I will be on vacation ;)

hey @thdesy is this something you are still interested in? With Singularity 2.4 you can build right into a directory, so it may not be needed here:
sudo singularity build folder/ docker://ubuntu

Hi Vanessa,
many thanks for the info - I will have a look! :)
Cheers and thanks,
Thomas
On 2017-10-18 05:11, Vanessa Sochat wrote:

hey @thdesy https://github.com/thdesy is this something you are still
interested in? With Singularity 2.4 you can build right into a
directory, so it may not be needed here:
|sudo singularity build folder/ docker://ubuntu |
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/singularityware/docker2singularity/issues/21#issuecomment-337447891,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQPYsFIViz2rjmS0XSaMUmGpcHKlKi1uks5stWxKgaJpZM4OSrLZ.

hey @thdesy this is long since supported for Singularity 2.4, and is now (also) available for the version 2.5.1 branch. I'm closing this issue and please re-open (or open a fresh one) if you have any trouble. Thanks!


New site for "Built With" Page: http://bangoshoping.myshopify.com

This site doesn't seem to actually use Cart.js.


TestAccCloudflareAuthenticatedOriginPullsCertificatePerZone is failing

TestAccCloudflareAuthenticatedOriginPullsCertificatePerZone is failing with the following exception
=== RUN   TestAccCloudflareAuthenticatedOriginPullsCertificatePerZone
    testing.go:705: Step 0 error: errors during apply:

        Error: error uploading Per-Zone AOP certificate on zone "0da42c8d2132a9ddaf714f9e7c920711": error from makeRequest: HTTP status 400: content "{\"success\":false,\"errors\":[{\"code\":1406,\"message\":\"This certificate already exists for this zone.\"}],\"messages\":[]}"

          on /tmp/tf-test591619056/main.tf line 2:
          (source code not available)

Link to failure: https://github.com/cloudflare/terraform-provider-cloudflare/runs/1295364502?check_suite_focus=true#step:6:178

closed via #928


Added that subqueries related to another database work

Related to pull request #23032
I would like to apologize for the insistence of adding this compatibility but I think it can be very useful for several users, including a project that I am developing with multiple databases.
I changed the last pull request code by changing the database separator used and using interface CanCrossDatabaseShazaamInterface to implement it
Added cross database compatibility in subqueries including database name in subquery when query connection and subquery connection are different and connection implements CanCrossDatabaseShazaamInterface.
COMPATIBILITY

MySQL (Compatible)
PostgreSQL (Compatible)
SQLite (Not compatible) will work as it works now
SQL Server (Compatible)

This will not break any existing features because user no need to modify existing code.
Only the subqueries with table in another database incorporate the database name to the left of the table name.
BENEFITS
Users can use the methods has, whereHas, doesntHave and whereDoesntHave in Eloquent when the relation is in another database.
For example:
Provider::whereHas('orders')
NOW GENERATES
select * from `hvp_providers` where exists (select * from `hvp_products` where `hvp_providers`.`id` = `hvp_products`.`provider_id` and `hvp_products`.`deleted_at` is null) and `hvp_providers`.`deleted_at` is null
Sql error:
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'global.hvp_products' doesn't exist
WITH THIS PULL REQUEST GENERATES
select * from `hvp_providers` where exists (select * from `shop`.`hvp_products` where `hvp_providers`.`id` = `hvp_products`.`provider_id` and `hvp_products`.`deleted_at` is null) and `hvp_providers`.`deleted_at` is null

No plans on adding. Sorry.

It's a little cumbersome, so it's not a surprise it didn't make into the core of the framework. Personally I wouldn't recommend cross database joins. But if your mind is set on this, perhaps you should consider writing a package for Laravel that introduce this.

I have package created locally. I will publish it today!!
In my case I need it because I have a global database and then each driving school has its own
Thanks for all.

Package is now available:
https://github.com/hoyvoy/laravel-cross-database-subqueries


Check coverage for only tested functions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+13.4%) to 84.058%

Totals

Change from base Build<PHONE_NUMBER>:
13.4%

Covered Lines:
65

Relevant Lines:
69

💛  - Coveralls


Datasets displaying a incorrect DOI

There are two datasets that are displaying an incorrect DOI and not the DOI that is in our IPT.

Monitoreo de peces, calidad del agua y caracterización de la comunidad hidrobiológica en los ríos Cucuana y San Marcos, área de influencia de la Central Hidroeléctrica del río Cucuana should have this DOI https://doi.org/10.15472/4vnoip.
Artrópodos de los Parques Ecológicos Distritales de Montaña should have this DOI https://doi.org/10.15472/cwrg3e.

Thanks in advance and have a happy new year
Edit: As an additional information, the DOI that resources are showing is a DOI from another resource of the same organization that was published the same day, so maybe it was some sort of cable crossings

Unfortunately I don't know the cause of the problem. If it happens again, let me know and please don't change the resource so we can find the cause.
Also, I would strongly recommend to update your IPT to the latest 3.1.0 version (or at least 3.0.6).
Closing this for now


Add bahnexperte extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

In reference to https://github.com/raycast/extensions/pull/7925
Had no idea how to update that pull request as the publish script threw errors my way.

Hi 👋
The icon should be 512x512 px 🙂

Is there any way to run all of this checks locally? I got a machine with 64 gigs ram and a good ethernet connection. I don't want to waste 3 hours time to get that message...

Three hours? It shouldn't take more than a few seconds to lint, you have scripts in package.json for that


Use prefetched repodatas in solve()?

In py_rattler 0.5, the solve function expected a parameter available_packages with a list of SparseRepodata, which had been previously fetched with fetch() or equivalent. In 0.6, now we pass channels + Gateway. This is excellent as a high-level API!
... but now I can't pass my previously prefetched repodatas 😬 I would need a way to pass a list of (Sparse)Repodata objects to the solver again, maybe as an overloaded signature, or a different function altogether. Would it be possible to offer the available_packages logic in some way? The reason for this is integrating with conda, where the repodata must be fetched with their networking stack for different reasons.
Thanks!

Just released py_rattler 0.6.3 which contains the new function!

That was quick, thanks!!

I see it on PyPI and it's working great. Wondering if the lack of a GH Release (only the tag is published) will cause interference with the eventual conda-forge release.

nvm semiauto mode works great https://github.com/conda-forge/py-rattler-feedstock/pull/18


Document how to delete Kubeflow installation when using Terraform deployment in CI/CD?

What is the URL of the document?
Kubeflow Terraform Deployment Guide
Which section(s) is the issue in?
The Cleanup section.
What needs fixing and describe the solution you'd like?
I would like a description of how I can make the make delete command work for a CI/CD environment.
Let's say you have a CI/CD pipeline with two jobs that can be run manually:

Deploy Kubeflow (performs all prerequisite steps & terraform init && terraform plan & make deploy commands)

Builds a docker image with all the prerequisites and runs it to trigger the deployment

Cleanup Kubeflow (performs the make delete command)

Builds the same docker image with all the prerequisites and credentials but instead of make deploy we want to do make delete

Issue: The second job (cleanup job) does not have the state of the first job (it's potentially a different CI/CD runner that runs the second job). Therefore the make delete command has nothing to delete because its state is already clean.
Is there a way to save the output of the terraform deployment from the first step and share the state (eg: output to a file) so that it could then be loaded in the delete job when needed?

@ryansteakley @surajkota Any idea ?

Hi, missed responding to this issue. yes, this can be done and is an advantage of using terraform. Make sure you store the following 3 things:

.tfvar file
Export any TF_VAR environment variables which you had while deploying
tfstate file which is created during the deployment


Broken links in HID docs

I have HTTP 301 Moved Permanently code when trying to open any subpage in these pages:
https://docs.microsoft.com/windows-hardware/drivers/hid/obtaining-hid-reports
https://docs.microsoft.com/windows-hardware/drivers/hid/sending-hid-reports
https://docs.microsoft.com/windows-hardware/drivers/hid/interpreting-hid-reports
Maybe TOC is broken or something.

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

ID: c16f66f8-babb-8d23-8ce5-52f9b87c26b1
Version Independent ID: 49578792-b8e0-8a37-515e-790378ea7dfa
Content: Interpreting HID Reports - Windows drivers
Content Source: windows-driver-docs-pr/hid/interpreting-hid-reports.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @billlattimer
Microsoft Alias: billla

https://github.com/MicrosoftDocs/windows-driver-docs/commit/9f19bd5132fe04154efcb39346b864f4619e98e5
Seems these pages are removed. Why?

@EliotSeattle any updates on this topic? Seems several pages get removed.

Hi @DJm00n
We have recovered the source for the missing pages from an earlier doc set. For some reason, these were dropped in a content migration from one source system to another. @billlattimer will research the issue and reinstate the pages, assuming they are still supported. Not sure about the schedule for this task, though, as it competes with other, hi-pri work.
Thanks,
Eliot

@EliotSeattle thank you for a quick response.
Subpages of these pages was the only source on how to work with HID devices:
https://docs.microsoft.com/windows-hardware/drivers/hid/obtaining-hid-reports
https://docs.microsoft.com/windows-hardware/drivers/hid/sending-hid-reports
https://docs.microsoft.com/windows-hardware/drivers/hid/interpreting-hid-reports
Please give this task some priority.

@DJm00n Thank you again for catching this drop; the pages will go live shortly!

@billlattimer hi!
These pages are still missing: https://github.com/DJm00n/windows-driver-docs/commit/a60e0bd21570a206f626c6c97c1f645bbdd89315

@DJm00n this was my error - it should be live in a few hours. Thank you for your diligence to ensure this content was live!


_CastError (Null check operator used on a null value)

I am trying to write a simple auto login method for my application. Previously I could write the code using Provide package. Then I decided to modify my application and use GetX package. But when the application reaches the following code it gives the mentioned error.
  Future<bool> _autoLogin() async {
    final prefs = await SharedPreferences.getInstance();
    if (!prefs.containsKey('userData')) {
      return false;
    } else {
      _isAuth.value = true;
    }
    final Map<String, Object> extractedUserData = Map<String, Object>.from(
        json.decode(prefs.getString('userData') as String));
    final expiryDate =
        DateTime.parse(extractedUserData['expiryDate'] as String);

    if (expiryDate.isBefore(DateTime.now())) {
      return false;
    }
    _token = extractedUserData['token'] as String;
    _userId = extractedUserData['userId'] as String;
    _expiryDate = expiryDate;
    _autoLogout();
    return true;
  }

Error:
_CastError (Null check operator used on a null value)
And it comes from this line of the code:
final prefs = await SharedPreferences.getInstance();
What is the problem and how can I fix it?
A minimal version of the project also can be find here: https://github.com/learner00000/test

Your code calls the ! operator on a varibale that is currently null:
var nullable = null;
nullable!.toString();

From your description, this seems to happen inside SharedPreferences, so this is probably a problem with SharedPreferences and not getx. You'll probably get better answers at SharedPreferences or Stackoverflow than here.
I still tried your example and can't reproduce your error. So the problem could be your build setup. (I'm running the example on windows as pwa)
In the future, you should always add the following:

Version of the package (in this case getx and SharedPreferences)
flutter version / flutter doctor output
your build enviroment (OS, IDE, target plattform)
a stracktrace if possible


Migrate Dashboard table to the new version on table components #2682

[ ] Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
Migrate Dashboard table to the new version on table components.
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)
[ ] Unit checks
[ ] Integration checks
[ ] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

@armenuikafka please take a look at the new dashboard, everything should look (almost) the same without any functionality issues.

@armenuikafka @David-DB88 let's just get rid of this link at all

@armenuikafka @David-DB88 let's just get rid of this link at all

@Haarolean do not redirect to Topics?

Custom deployment will be available at http://pr2694.internal.kafka-ui.provectus.io

@David-DB88 , Topics count link still exists.

but I don't see the link anywhere @armenuikafka

@David-DB88 I'm checking http://pr2694.internal.kafka-ui.provectus.io/.
https://user-images.githubusercontent.com/104780608/194479582-507e5128-403b-41c6-b184-2833c33c59a6.mov

@armenuikafka a redeploy did help it seems, PTAL


A test suite would help understanding how to use the methods

I often look at the tests to understand how a library is supposed to be used.
It is difficult for me to use these extensions because there are no tests and I don't know how to use them.
For example, I am trying to use the DBObjectExtension.SetXDictionaryXrecordData Method. I see that it requires ResultBuffer data (or TypedValue[] values), but I don't know how to create them. Having a little test that shows how to create them would save me tons of time. This is just an example, the same applies to all other methods in this repository, with the exception of the ones that are very simple and self-explanatory,
I would love to create a PR, but... well, maybe in the future I will be able to do it, right now I will need to work and understand how to use those methods before being able to create the tests :(

I certainly should have specified it in the README, this library does not exempt from knowing the basics of the AutoCAD .NET API. It is rather aimed at AutoCAD programmers who would like to optimize their development workflow.
For example, the extension method DBObjectExtension.SetXDictionaryXrecordData just makes it easier to add or modify an Xrecord but its use assumes that you know what an Xrecord is and how the data is stored in it, therefore knowing the ResultBuffer and TypedValue types.

this library does not exempt from knowing the basics of the AutoCAD .NET API

I agree, and it's great as is. The library doesn't need any improvement.
My proposal was about adding some tests so this repository would have one more purpose: it would help learning how to use the library methods and the objects it extends themselves.
The official AutoCAD documentation has very few examples, some are in C++, some of the ones in C# are obsolete, some have syntax errors and don't even compile. The only way to find examples is the forum, but those are often messy.
Perhaps this shouldn't change, perhaps creating a second repository with examples and snippets would be a better solution?
I created this issue when I found this post and I thought that if that little example had been included in the repository, it would have saved a few precious hours of my life.

What you would like is not related to the library.
You give as an example the ResultBuffer and TypedValue types which have no extension methods in this library.
In fact, you would like a detailed documentation with examples for each type in the API. The task is huge and I don't have the skills to do it.

you would like a detailed documentation with examples for each type in the API

No, I would never ask for that. I would like a few simple tests, as I mentioned earlier.
I didn't ask for documentation and didn't even ask for comments in tests.
In the post I mentioned above you put some description, some code that comes almost straight from this repository and a little test that shows how to use it.
All I am saying is that after reading that post I can finally use this repository.
I didn't need the description nor the code that is already included in this repository.
All I needed was the test at the bottom. This little command is your code, and in my opinion would add great value to this repository. A test suite that runs without the interactive GetEntity might be the right thing to do, but that might also be overkilling and confusing. As is it is already a great help.
        [CommandMethod("XRECTEST")]
        public static void XrecordTest()
        {
            var doc = Application.DocumentManager.MdiActiveDocument;
            var db = doc.Database;
            var ed = doc.Editor;

            var per = ed.GetEntity("\nSelect object: ");
            if (per.Status == PromptStatus.OK)
            {
                using (var tr = db.TransactionManager.StartTransaction())
                {
                    var ent = (Entity)tr.GetObject(per.ObjectId, OpenMode.ForRead);
                    var data = ent.GetXDictionaryXrecordData("TestXrec");
                    if (data == null)
                    {
                        ed.WriteMessage("\nThe entity does not have a 'TextXrec' Xrecord, it will be added");
                        ent.SetXDictionaryXrecordData("TestXrec", new TypedValue(1, "foo"), new TypedValue(70, 42));
                    }
                    else
                    {
                        foreach (var tv in data.AsArray())
                        {
                            ed.WriteMessage($"\nTypeCode: {tv.TypeCode}, Value: {tv.Value}");
                        }
                    }
                    tr.Commit();
                }
            }
        }

I'm sure I'm not the first one that tried to use your library, failed, abandoned, found one of your forum posts, went back to your library and finally got it.
If your repository had this test, you could have answered to that forum post with a link to that test. Faster for you (less typing and no need to create all that code that works without your library) and much much much better for the reader (they would find the same help your post gave, but they would also find other helpful examples).
Yes, I know, providing the code in the forum post is always better than a link to an external resource, but GitHub is an external resource that is not going to disappear any time soon, and the added value provided by your repository with a few test examples would be so high that everybody would be happy about it. I'm pretty sure if you posted a link to the test in the repository instead of writing that long post, that would have been marked as the solution too.

The task is huge and I don't have the skills to do it

Yeah, right!
You are single handedly running the AutoCad API forum, which is where people go when the official documentation fails. If you don't have the skills to do it (whatever "it" is), no one has!

I don't quite understand what could have helped you figure out how to use the library in this test example.
In the quoted example, the expression:
ent.SetXDictionaryXrecordData("TestXrec", new TypedValue(1, "foo"), new TypedValue(70, 42));
matches exactly the signature of the method:
public static void SetXDictionaryXrecordData(this DBObject target, string key, params TypedValue[] values)
Again, if your misunderstanding has to do with ResultBuffer or TypedValue types, it is not related to this library.
Again, this library does not exempt you from knowing the basics of .NET and the AutoCAD API. If some people don't understand how to use it, it's because they lack those basics.
I do not plan to add test examples to this library, which should remain a pure class library.

I do not plan to add test examples to this library, which should remain a pure class library.

Well this closes the issue 😄
The tests don't need to be part of the class library, they would need to use it. They could be quick and dirty, interactive like the snippet I pasted earlier, and inside the same dll or in another one. Or they could be well designed and do what good tests do, that is starting their own instance of AutoCad and reporting failures if there are any. The latter would be the proper way, but the former would be easier to do.

I don't quite understand what could have helped you figure out how to use the library in this test example.

I don't know what my misunderstanding came from. All I know is that the test in that post helped me, without it I wouldn't be able to use those methods. Sometimes the signature is enough, sometimes one needs to see a working example, execute it step by step and explore it in the debugger. Much better than looking at a signature.
Here are two use cases for the tests I'm talking about:

I wasted hours trying to use those methods. After seeing that little example on that post I figured it out in 5 minutes.
You spent some time writing that post. I don't know how much time, but posting a link to the test would have been (1) faster for you and (2) more useful for the reader.

You have shown that you are available and able to help, by publishing this and by helping us all in the forum, but I don't understand (but completely respect) your approach. To me feels like you are saying "hey guys, here is a great library for you to use it, but if you have problems using it you will need to ask in the forum, because I would rather rewrite half the code in the library again and again in the forum so I can help you, then adding a few little tests here".
I'm closing this issue. I will add the tests to my local copy, by copying them from your posts in the forum as I find them.


Document whether len passed to Serializer::serialize_struct includes skipped fields

The documentation for Serializer::serialize_struct currently says:

len is the number of data fields that will be serialized.

This implies that the count of skipped field is not to be included in the len. It would be worth calling this out explicitly.

I would even like to get a slice of all the field names of the struct, including the skipped ones. Data formats which can reuse struct layouts would profit from this as using #[serde(skip_serializing_if = "Option::is_none")] or the equivalent #[skip_serializing_none] from serde_with breaks this.
Source: I am currently trying to implement struct layout reusage for my data format and as long as serde isn't telling me about the whole struct layout, I'd have to forbid the aforementioned, really useful features.


clean_regex fails with int and bool values

Hi,
The clean_regex function added in the last commit expects only a str. But searchValue can be either a bool, a int or a str. A naive fix would be
if isinstance(searchValue, str):
  regex = clean_regex(searchValue)
else:
  regex = searchValue

Thanks !

Sorry about that -- I'm not clear what would cause datatables to send int or bool, but I think the fix looks fine in any case.

The prepare_arguments function tries to convert the value it gets from the request to a int, a bool before keeping it as a str.

Closing this as it doesn't apply to to the current version of sqlalchemy-datatables


iPhone support?

I just found this plugin and seemed perfect to fix my config sync problem! But when I tried installing it on my iPhone, it says "This plugin does not support your device". Bummer!
Do you plan to add iPhone support?

Actually I managed to fix my problem, so it's no longer need! :)


feat(core): integrate rspack_resolver with input_filesystem

Summary
related to #7478

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench

!bench


Impossible to download your dataset

Hi,
I would like to download your dataset but I cannot obtain the link from this webpage you gave in the README.
Would it be possible for you to host the file in another online platform?
Thanks for your help and your time.
Best

I have added a link on Google Drive in the README

Thanks, I downloaded the dataset. Sorry but I do not find the information about how to split this dataset into train and test set? See table VI of your paper below:


Dependencies submission

Description
For documenting the dependencies of this project, we would like to submit our Gradle dependencies to the GitHub's Dependency Submission API by using the Gradle dependency submission action.
This submission should be done on each release, so in the deployment workflow.
Checklist

[x] Update the deployment workflow.
[ ] Test it by redeploying the version 4.1.0.

Testing the dependencies submission on the version 4.1.0 fail (see the corresponding action).

Running the dependencies submission on other branches or tags than main doesn't want to update the dependency graph. This seems to be a GitHub requirement.
So, we should include the dependency submission in the integration workflow instead, but only when a push event has occurred.
Here's a checklist for fixing the dependencies submission:

[ ] Remove the dependencies job from the deployment workflow in version 4.1.0 and on the main branch.
[ ] Add a dependencies job to the integration workflow that should run only on push events.
[ ] Test the integration workflow.

Done.


install/pre-install: Update containerd

Update containerd version now we have a new release

/test

I am trying to get https://github.com/confidential-containers/operator/pull/179 merged first. Let's wait a little bit, end of the day maybe.

Given that #179 hasn't been merged we aren't actually testing the pre-install image, so any failures aren't related to the change

I've faced issues to merge #179. So I opened https://github.com/confidential-containers/operator/pull/226 in order to test this PR.
Lifted the do-not-merge label.

@stevenhorsman https://github.com/confidential-containers/containerd/releases/tag/v<IP_ADDRESS> is missing the s390x build. That's why the jobs on https://github.com/confidential-containers/operator/pull/226 (that I am using to test this PR) are failing.

btw, @stevenhorsman , https://github.com/confidential-containers/operator/pull/226/commits/cf3aaef619d2597c2f00e37fcc7f6a28982605ea fixes a bunch of troubles I've faced to get kubectl installed on the pre-install image. If you and @jepio agree, could we pick up that fix on this PR too?

/test

@stevenhorsman https://github.com/confidential-containers/containerd/releases/tag/v<IP_ADDRESS> is missing the s390x build. That's why the jobs on #226 (that I am using to test this PR) are failing.

Sorry, that's my bad. I must've removed it when updating the containerd workflow as it's only in 1.7, not 1.6. I will look at that right now.

@stevenhorsman https://github.com/confidential-containers/containerd/releases/tag/v<IP_ADDRESS> is missing the s390x build. That's why the jobs on #226 (that I am using to test this PR) are failing.

Sorry, that's my bad. I must've removed it when updating the containerd workflow as it's only in 1.7, not 1.6. I will look at that right now.

Fixed and the release re-built

/test

@stevenhorsman https://github.com/confidential-containers/containerd/releases/tag/v<IP_ADDRESS> is missing the s390x build. That's why the jobs on #226 (that I am using to test this PR) are failing.

Sorry, that's my bad. I must've removed it when updating the containerd workflow as it's only in 1.7, not 1.6. I will look at that right now.

I forgive you @stevenhorsman . This time only ;)


Ping reviewers even if dirs key is empty

This gets #333 work finally. The first commit is taken from https://rust-lang.zulipchat.com/#narrow/stream/242791-t-infra/topic/highfive's.20ping.20config/near/237971862.

r? @pietroalbini

Thanks!


0.3.1 point release

Signed-off-by: Edward Z. Yang<EMAIL_ADDRESS>

@vincentqb I merged it myself.
Re #261 it looks like it was only deprecated and not removed

@ezyang I notice that the version available on pip is 0.3.0 which requires pytorch 1.2.0. I don't have access to the binaries. Can you update the version to 0.3.1?

@soumith can you please give @vincentqb maintainership on torchaudio

@ezyang @vincentqb you are already an admin of the repo Vincent. You and christian are.

Yes, I am an admin on github. Do I need to be listed as a maintainer on pypi also? I see ezyang who is a maintainer on pypi written in upload_wheels_to_pypi.sh, and my pypi username vincentqb is not on the list of maintainers on pypi.

oh, can add you on pypi. what is your pypi username?

you already gave your pypi username, i couldn't read. adding you in 1 min

added

can you please upload this version to the "pytorch" conda channel as well?

@soumith -- my username is also "vincentqb" on anaconda.com.

@vincentqb added on conda

Uploaded using builder/conda/publish_conda.sh:
cd builder/conda
./publish_conda.sh torchaudio==0.3.1

@ezyang The error in #307 does not occur anymore, but when I print the version number I see 0.3.0+120f693 instead of 0.3.1 when installing with conda instead of pip.
conda:
conda create -n pytorch-conda python=3.7
conda activate pytorch-conda
conda install -c conda-forge sox
conda install -c pytorch torchaudio
python -c "import torch; print(torch.__version__); import torchaudio; print(torchaudio.__version__);"

1.3.0
0.3.0+120f693

pip:
conda create -n pytorch-pip python=3.7
conda activate pytorch-pip
pip install torchaudio
python -c "import torch; print(torch.__version__); import torchaudio; print(torchaudio.__version__);"

1.3.0
0.3.1

@soumith @vincentqb
thanks!


create ansible roles for nostr relay

This is an experiment in creating a nostr relay following the guide on jascha's blog.
I adjusted the code from this blog in a couple ways:

modernize terminology (whitelist->allowlist)
use a json file for the allowlist instead of hardcoded into allowlist.js.  This seemed easier to update and use with an ansible script.

This PR comes with the nostr-relay and nostr-relay-nginx roles and an example inventory and playbook.
I have tested it with the site cooltesting.space.  It is up and working as a relay, and  I was able to write from coracle.social
and see the events appear in the cooltesting.space db
[
    {"content":"relay is working","created_at":1680850919,"id":"bd7d94994cca18f7784cf406bc2a05db210448e733c1c4a21d3d2fd3a1b3b38c","kind":1,"pubkey":"35ddcde92bd63056707ca9a09a4d8cf8c2b993dc905b3407901ad17aba2336e3","sig":"b1747baf4c8ce034c5c7b8eda47b85e8e405a4ba1016e8a50a0f2880b70a5b587143fd045274dcf629dc6a523913ddedee1d8bfd2feaa3d50aaeb30b6f876a22","tags":[]},
    {"content":"docker container is working and whitelisting!","created_at":1680850984,"id":"da221fbc30b43a32c05ceb69e7fdb063f4440a60fbc915376a20d9b5d656f53b","kind":1,"pubkey":"35ddcde92bd63056707ca9a09a4d8cf8c2b993dc905b3407901ad17aba2336e3","sig":"e9f16cd1a0dff8029c3b5d4683649730dd0ea94e405993686b700dd3c3508bc8efd1ac469803240636b3fa51234fa4bd27666b9853750463eb799d06ee0a2e6a","tags":[]},
    {"content":"","created_at":1680851492,"id":"db69df5734c30c23167b1c43aa8c17d26edf5cb7354794dc72c569fd64f751fa","kind":10002,"pubkey":"35ddcde92bd63056707ca9a09a4d8cf8c2b993dc905b3407901ad17aba2336e3","sig":"5b07ff0d1aca4fac1977d76836dfa854e1cd3b5e85327b7edad458708089e0c4083c55ea1118cf7ae97d18cb2814a068521071cee98f1269c9ff06362b21832c","tags":[["r","wss://brb.io","read"],["r","wss://nostr.zebedee.cloud","read"],["r","wss://nostr-pub.wellorder.net","read"],["r","wss://relay.nostr.band","read"],["r","wss://cooltesting.space"],["client","coracle"]]},
    {"content":"i am testing from coracle","created_at":1680851515,"id":"8b83b67b32ae1880ccba69ae08a9c2778569cd47cfd48051dd349d61f44656c4","kind":1,"pubkey":"35ddcde92bd63056707ca9a09a4d8cf8c2b993dc905b3407901ad17aba2336e3","sig":"61f0c9c6800933c5b4b1851a6b39d92b53cbb79e78b947663d39f06f38d3a46596a827cf0d1020294b4d5111b96f0de30795a26cba752ccd4b13ff231340ca1d","tags":[["client","coracle"]]}
]

pretty cool!
This pr is still a draft until it has decent:

documentation :: There should be better documentation around the vars needed and running the playbook
testing :: I added some planetary folks to my allowlist from our address book, but not sure if it works with them yet. I believe the allowlist requires hex keys, and our address book only has nspub style keys.
structure :: I wonder if we can rejigger the nginx stuff to be included in this role and other ways to make this easier to maintain.

v cool. I have an instance of nostr-rs-relay running in docker on my home server. I have been using this to convert npubs to hex: https://damus.io/key/


Rename iOS Static Library from "FluentUILib" to "FluentUI"

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
This is to close #306
Our iOS static library and framework had different names, which meant the compiled products had different symbols, which prevented interop if another library (like FluentUI React Native) was expecting symbols the framework provided, but got symbols the static library provided.
Renaming the Product in xcode to "FluentUI" explicitly solves this.
Let's also update our CI pipeline to still properly find the compiled libraries and pack appropriately.
Note: This is a breaking change, as every import FluentUILib must now be a import FluentUI
Verification
Locally created the static libraries and copied them into my office enlistment. Made sure that office compiled and ran for the scenario I was testing
Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

I probably need to update our YAML too...

I probably need to update our YAML too...

Nevermind. The script I updated is what our YAML refers to 😁

@rdeassis @harrieshin no objections to hardcoding the name as I did? As opposed to using some other environment variable?


remove changelog

I documented the commit conventions for semantic-release. Let me know if you have any questions @ierceg @paulmelnikow
semantic-release automatically creates releases on GitHub and puts in changelogs, see https://github.com/node-nock/nock/releases

Coverage remained the same at 92.671% when pulling a73e68878808535e7bcfe6554c6cef40ade0ac0b on remove-changelog into bd387365ff8e6df731cf5a35cc92560346826ec5 on master.

Coverage remained the same at 92.671% when pulling 2cebc1cfeaa620467b40db3c127241af7980bfd2 on remove-changelog into bd387365ff8e6df731cf5a35cc92560346826ec5 on master.

Coverage remained the same at 92.671% when pulling 2cebc1cfeaa620467b40db3c127241af7980bfd2 on remove-changelog into bd387365ff8e6df731cf5a35cc92560346826ec5 on master.

Coverage remained the same at 92.671% when pulling 2cebc1cfeaa620467b40db3c127241af7980bfd2 on remove-changelog into bd387365ff8e6df731cf5a35cc92560346826ec5 on master.

@gr2m Think we could turn off coveralls comment posting? They're mostly noise and very chatty. The result that shows up in the "checks" seems sufficient!

I heartily agree, I turn it off on all my projects.

Coverage remained the same at 92.681% when pulling 1af2622e64772cb52e6e0291a5f91a29b0f49dc0 on remove-changelog into be7a075c9ea3891e672ee862f310e52d9a60a8a4 on master.


[LIVE-11478] Support for Uniswap V3 plugin

✅ Checklist

[x] npx changeset was attached.
[x] Covered by automatic tests.
[x] Impact of the changes:

now supports communication with the Uniswap Universal Router plugin

📝 Description
Adds support for the Uniswap Universal Router plugin specified here in @ledgerhq/hw-app-eth and activates it in coin-evm

❓ Context

JIRA or GitHub link: https://ledgerhq.atlassian.net/browse/LIVE-11478

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

Hey all, this PR seems to have caused some issues downstream for the NPM package.
hw-app-eth/src/modules/Uniswap/index.ts imports from ethers which is not listed as a dependency of the package. It may have worked for you locally as it's listed under devDependencies?
Another issues is that the listed ethers version is old, and the new one exposes utils differently (each method can be imported directly instead of the whole utils object).


enable GCE as the cloud-provider for native volumes etc

Maybe it is out of scope for a generic KTHW tutorial; but since half the tutorial is GCE specific, perhaps helpful to pass in the --cloud-provider=gce flag to kube-controller-manager?
https://github.com/kubernetes/kubernetes/issues/66564#issuecomment-407603777

Needs work.


anchor detection if the link also contains a path fails

https://github.com/wessberg/scroll-behavior-polyfill/blob/45593dcfc35f98b64230579eb01ac821c5d9f67d/src/patch/anchor/catch-navigation.ts
The catch-navigation currently looks if a link is clicked with href="#anchor" but it will never catch a link to the current page using href="/url/to/current/page#anchor" because only the attribute value is considered and only if it contains nothing else.
The detection for a link pointing to the current page is actually fairly simple:
const link = e.target;
const pointsToCurrentPage =
       link.origin === location.origin
    && link.pathname === location.pathname
    && link.search === location.search;

const hash = link.hash

Here is a reproduction: https://jsfiddle.net/ey087v3o/ (tested in safari and chrome)

Hey there. Thanks a lot for your bug report and repro. This has been fixed in the latest release (v2.0.12). I took great inspiration from your detection, thanks a lot.


Rescue error raised from missing batch report

Trello: https://trello.com/c/e1vAwREO
Link-checker-api throw a 5xx error if get_batch is called for a batch report that doesn’t exist.
This isn’t usually a problem in Production, but step by step pages, and their associated link reports are copied
to integration and staging each night. This means that if when you try to view the step by step in those environments, it’s
querying link-checker-api with a batch id that was created in (and probably only exists in) production. This causes link-checker-api to
fail with a 5xx.
We should update link-checker-api to fail more gracefully if it can’t find a requested batch, but until then, allow collections-publisher
to load step by steps even if the link report details cannot be found.
Error in sentry

@sihugh I've fixed up the changes. Cheers!


"tuple concurrently updated" with Postgres

Describe the bug
Error while creating a view in postgres
  deadlock detected
  DETAIL:  Process 187 waits for ShareLock on transaction 19479; blocked by process 186.
  Process 186 waits for ShareLock on transaction 19480; blocked by process 187.
  HINT:  See server log for query details.
  CONTEXT:  while updating tuple (1,11) in relation "pg_class"

Occassionally also throws this:
 tuple concurrently updated
Steps To Reproduce
Have 2 sql files under the same model - they work when only one exists. But when two exist in the same folder (model), the error above throws.
The two sql files refer to different tables within the same schema.
Expected behavior
2 views should be created without errors
System information
Which database are you using dbt with?

[x] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.19.1
   latest version: 0.19.2

Your version of dbt is out of date! You can find instructions for upgrading here:
https://docs.getdbt.com/docs/installation

Plugins:
  - redshift: 0.19.1
  - bigquery: 0.19.1
  - snowflake: 0.19.1
  - postgres: 0.19.1

The operating system you're using:
Ubuntu 20.0.4
The output of python --version:
Python 3.8.5

Thanks for opening @alderson59. I don't quite understand what you mean by:

Have 2 sql files under the same model

Could you share the code for the two models in your project? Obviously scrubbed of any sensitive details.

It's a large table, so I try to show some of the different types of columns queried and selected.
Important to note that these queries are all working on their own - tested via the CLI, adminer, as well as via dbt (when standalone - meaning I removed one of the sql files from the directory when doing dbt run)
view1.sql:
SELECT
        id,
        TO_TIMESTAMP(CAST(date_created AS DOUBLE PRECISION)) AS date_created,
        status,
        some_json::jsonb->>'receipt' AS receipt,
        TO_TIMESTAMP(CAST(some_json1::jsonb->>'due_by' AS DOUBLE PRECISION)) AS due_by
FROM some_schema.table1

view2.sql
SELECT
        id,
        CAST(amount/100 AS DOUBLE PRECISION) AS amount,
        status,
        TO_TIMESTAMP(created) AS date_created,
        some_json1::jsonb->>'message' AS message,
        CAST(amount2/100 AS DOUBLE PRECISION) AS amount2,
        failure_message,
        (some_json2::jsonb->>'some_key')::jsonb->>'some_other_key1' AS some_other_key1,
        ((some_json2::jsonb->>'some_key')::jsonb->>'some_other_key2')::jsonb->>'some_other_other_key' AS some_other_other_key
FROM some_schema.table2

Have 2 sql files under the same model

Apologies, should have elaborated further. I also probably misunderstood what was meant by a "model" when I posted that. I now understand each model is its own .sql file
What I meant was, 2 sql files under the same folder/dir.
Directory structure:
models/views/view1.sql
models/views/view2.sql

dbt_project.yml:
models:
  project:
    views:
      +tags:
        - "some_tag"
      +materialized: view
      +post-hook: "GRANT SELECT ON ALL TABLES IN SCHEMA public,some_schema TO user;"

Ran with dbt run --models tag:some_tag - didn't seem to be any problems here.

@alderson59 Thanks for the details, it sounds like you've been able to run these models successfully. Are you still seeing the reported error when you dbt run?

yep, nothing has changed since I last posted this. Are you able to reproduce the issue?

I am also facing the same issue
Describe the bug
I have 2 models which have some source tables in common. Both are materialised as views. When running dbt on 2 threads, the creation of the 2 views happens in parallel and one of the views throws this error. When I run these models one-by-one using the model selection syntax, they are running perfectly!
It might be possible that this error is not related to having common set of source tables
Steps To Reproduce
Create 2 models whose materialization is set to view with some source tables in common. Set threads > 1 in profiles.yml. Do dbt run
It might be possible that this error is not related to having common set of source tables
Expected behavior
The models should get created without any error
Screenshots and log output
Running with dbt=0.19.1
Found 3 models, 10 tests, 0 snapshots, 0 analyses, 138 macros, 0 operations, 0 seed files, 5 sources, 0 exposures
05:42:06 | Concurrency: 2 threads (target='dev')
05:42:06 |
05:42:07 | 1 of 3 START view model schema.view1................. [RUN]
05:42:07 | 2 of 3 START view model schema.view2...................... [RUN]
05:42:08 | 2 of 3 ERROR creating view model schema.view2............. [ERROR in 1.15s]
05:42:08 | 3 of 3 START incremental model schema.incremental1...... [RUN]
05:42:08 | 1 of 3 OK created view model schema.view1............ [CREATE VIEW in 1.25s]
05:42:17 | 3 of 3 OK created incremental model schema.incremental1. [INSERT 0 34 in 2.61s]
05:42:17 |
05:42:17 | Finished running 2 view models, 1 incremental model in 5.98s.
Completed with 1 error and 0 warnings:
Database Error in model order_details (models/view2.sql)
  deadlock detected
  DETAIL:  Process 18470 waits for ShareLock on transaction<PHONE_NUMBER>; blocked by process 18471.
  Process 18471 waits for ShareLock on transaction<PHONE_NUMBER>; blocked by process 18470.
  HINT:  See server log for query details.
  CONTEXT:  while updating tuple (42,19) in relation "pg_class"
  compiled SQL at target/run/test/models/view2.sql
Done. PASS=2 WARN=0 ERROR=1 SKIP=0 TOTAL=3

System information
Which database are you using dbt with?

[x] postgres
[ ] redshift
[ ] bigquery
[ ] snowflake
[ ] other (specify: ____________)

The output of dbt --version:
installed version: 0.19.1
   latest version: 0.19.2

Your version of dbt is out of date! You can find instructions for upgrading here:
https://docs.getdbt.com/docs/installation

Plugins:
  - postgres: 0.19.1
  - redshift: 0.19.1
  - bigquery: 0.19.1
  - snowflake: 0.19.1

The operating system you're using:
Ubuntu 18.04
The output of python --version:
Python 3.6.9
Additional context
Add any other context about the problem here.

It might be possible that this error is not related to having common set of source tables

To chime in on this, I am in fact using two separate tables, but both from the same schema in the same db.

When running dbt on 2 threads, the creation of the 2 views happens in parallel and one of the views throws this error. When I run these models one-by-one using the model selection syntax, they are running perfectly!

Spot on! When running 2 at once, one of the views get created, while the other doesn't due to the lock.
DAG flow-wise, the 2 models should be running concurrently side-by-side, which probably means dbt/postgres has trouble reading two queries at once.
Can we bump this to a higher priority? Thanks.

Ok, I hacked this a little, and it's a temporary fix as I think there may be some other underlying issues here.
So, knowing that two independent models can't run concurrently from the same db, I made the second one as a downstream of the first, and to do that, we need to use the {{ref('view1')}} function without actually parsing the SQL - so, I commented the line via SQL as such -- {{ref('view1')}}. Now it works, but it runs downstream of the first.

In this case you lose the parallelism of running on multiple threads ☹ A simpler solution would be to run the views one-by-one if you con't have many views or switching to single threaded execution.
I would like to add that this only happens when 2 views are running in parallel. Not when a table and a view are running in parallel.

Bump on this issue, please :)
The hack I'm using above is causing me to have to write a bunch of dependencies in each model in my web of views, sadly. Lots of unnecessary debugging to ensure that it works as it should have, whether with or without concurrency.

@alderson59 @sdabhi23 I'm looking into this issue, but I haven't been able to reproduce it myself. I'd like to gather some more information, if possible:

Could you each share the version of Postgres you're using?
Could you each share (from logs/dbt.log) the exact SQL that dbt runs right before Postgres returns deadlock detected/tuple concurrently updated? My guess is that this happens when it tries to either rename or drop the existing version of the view.

Postgres version: 11.10
logs/dbt.log: I have post hooks running which grant permissions on all objects to various roles. Those are the last SQL statements running on either of the threads. Looks like I should have done permissions on the object being created instead of all objects

Aha! This sounds promising, and closer to a known quantity. We recommend that post-hook grants be specific to the model just created, and grants on all objects be executed from on-run-end hooks instead.
What happens if you change the post-hook grant to be specific to {{ this }}?

@jtcohen6 thanks for looking into this!
Just like @sdabhi23 , I also have post hooks running grant perms. I set this in dbt_project.yml to the tune of:
GRANT SELECT ON ALL TABLES IN SCHEMA public,schema1,schema2 TO user; since the view models reference data from different schemas. This needs to be rebuilt on each run with our use case.
I'm not sure if {{ this }} nor {{ this.schema }} would do any good here.

It seems we've found the culprit: concurrent grants on the same objects.
@alderson59 It sounds like your approach may want to be:

Once at the start of a run, grant select on all source schemas to user
After each model runs, grant select on just that model to user

In practice, that could look like:
# dbt_project.yml

on-run-start:
  - "grant select on all tables in schema public, schema1, schema2 to user"

models:
  +post_hook: "grant select on {{ this }} to user"

I'm going to close this issue—I don't believe we need to make any code changes in dbt.
just for fun
You could get even fancier with the all-sources grant, and write a macro that derives the set of source schemas from graph.sources instead of specifying public, schema1, schema2 explicitly.
-- macros/grants.sql

{% macro grant_select_on_all_tables_in_source_schemas() %}

  {% if execute %}
    {% set all_source_schemas = graph.sources.values() | map(attribute = 'schema') | list | unique %}
  {% else %}
    {% set all_source_schemas = [] %}
  {% endif %}

  grant select on all tables in schema {{ all_source_schemas|join(', ') }} to user

{% endmacro %}

# dbt_project.yml

on-run-start:
  - "{{ grant_select_on_all_tables_in_source_schemas() }}"

@jtcohen6 I agree that issue is with concurrent grants. But do you have any idea why this behaviour is flaky?
I mean sometimes it fails sometimes it doesn't. That's the behaviour I have observed. It only failed when those 2 particular views were running concurrently.
I will definitely switch to model specific grants, but just a thought

@sdabhi23 My sense is, those two grant statements need to be running concurrently to hit this error. I imagine the view models and grant statements each take <1s to execute. The error will raise only when the queries overlap, which may not be in every invocation.

Thanks, this fixes the issue and works like a charm 👍🏻
@jtcohen6 , decided to try out your fun script 😃 sadly, it didn't work, and the macro returned nothing in the graph.sources schemas. Any hints as to why? Doesn't this automatically parse on each run? (according to what I read from the docs)
What's a good way to debug this sort of error in general? Nothing on the logs (everything ran successfully). I've been testing it by outputting just the macro text into a view to select the text. Didn't return any schema, unfortunately.


"Cornetu" and "Buda" are listed in the drop-down menu

Description:
"Cornetu" and "Buda" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Cornetu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Cornetu, ILFOV (Cornetu)" and "Comuna Cornetu, ILFOV".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Buda" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Buda, ILFOV (Cornetu)".


You need to add the paths to the object locations in your collection metadata csv

Describe what feature you'd like. Pseudo-code, mockups, or screenshots of similar solutions are encouraged!
In your collection metadata csv you need to specify the location of the files (the "object" or main file, the small and thumbnail), these are in the csv columns for object_location image_small and image_thumb.
What type of pull request would this be?
None
Any links to similar examples or other references we should review?
No response

Hi Jesse, I added the file location to my csv, but there seems to be a deployment issue now.

There's an error in the Ruby environment, I'm not sure why that is
happening now since the site built and deployed previously. (
https://github.com/gbroyles/gigi-collection-builder-test/actions/runs/12396431345/job/34612555451)
Can you build the site locally?
On Wed, Dec 18, 2024 at 11:57 AM gbroyles @.***> wrote:

Hi Jesse, I added the file location to my csv, but there seems to be a
deployment issue now.
—
Reply to this email directly, view it on GitHub
https://github.com/gbroyles/gigi-collection-builder-test/issues/1#issuecomment-2551834686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2F4PJCRITAYFHLHVIBWNYL2GGSRFAVCNFSM6AAAAABRUV5CLWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJRHAZTINRYGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Add arduboy logo to boot sequence

Can we agree that (by default) unless a sketch chooses not to compile it (for space reasons) that at boot time we should scroll in the Arduboy logo?  I downsized the graphics from Arduboy.com and rasterized it exactly but if there isn't agreement on this I probably won't go further.
I thought it would be nice to have some consistency and build in a standard "device experience" that would include toggling brightness, sound, boot logo, etc.  Kind of help with the branding of the Arduboy experience.
Another option would be to compile the boot sequence into the Bootloader so that it's completely outside of a sketch... I'm very curious how much space is to spare there.  I know the default on Atmega32U4 is to use 4k for the boot loaded, but I"m not sure how much of that space is actually used by the boot loaded now.

It looks pretty slick.  I have it worked over in https://github.com/yyyc514/ArduboyLib/tree/brightness if anyone wants to try it.  I'll make a sep PR later.  I did add a define so devs could easily turn it off if they really really didn't want it - though I honestly think it should be part of the hardware (bootloader).

Is this welcome in core?  If not I'm going to go ahead and put it in the Extra lib I'm building for Arduboy.  If it's welcome I'll make a PR.

Yes, please make a PR for this. The branding is very slick. I agree that if the developer wishes to override the boot logo the option should be easily available.

The difficulty is how to turn it off... it's the kind of thing you'd want to use defines for, but the root sketch file of a project has no way to set defines that are evaluated before the entire project is compiled... and if you make it a programatic option then in my experience C is still going to link it in the final sketch (even if it's never called)... because technically it could have been called.
That means whatever space we spent storing the logo is wasted for anyone not wishing to show the logo... unless they are comfortable editing the library itself, in which case there is no problem.  All the problems go away if we accept most people might not use the core library as-is but rather would download it and then edit one of the H files to set preferences, etc.

I think we can use the assumption that most won't mind the loading sequence, and would even prefer it, and like you say: those that would rather not use it can modify the source. I say go ahead and do a PR without the option to turn it off. Kevin was interested in getting this added as well.

Sweet!

@rogosher wrote:

I think we can use the assumption that most won't mind the loading sequence, and would even prefer it, and like you say: those that would rather not use it can modify the source.

The problem with disabling the logo using  a #define in a library header is that it will affect all sketches that use the library. My assumption is that the main reason to turn off the logo is because it's necessary to free up some program space, not because it's an annoyance to someone.
What  happens if someone writes a "killer" game that many people like? Say this game requires so much space that the only way to fit it is to free up space by disabling the logo. If a user disables the logo to play this game, then it will be disabled for all other sketches that this user loads (assuming that the Arduino IDE  or codebender is used, and the #define isn't changed back and forth between loads). If the game becomes quite popular, or a number of games have the same issue, then the logo may end up being disabled by a good many people, even though they wouldn't mind seeing it on smaller games.
@yyyc514 wrote:

The difficulty is how to turn it off... it's the kind of thing you'd want to use defines for, but the root sketch file of a project has no way to set defines that are evaluated before the entire project is compiled... and if you make it a programatic option then in my experience C is still going to link it in the final sketch (even if it's never called)... because technically it could have been called.

I can think of one way around this. It's a bit radical and makes library maintenance a bit more difficult, but is fairly straight forward for a sketch developer. Instead of one library, there would be three (or more).
The current Arduboy library and its Arduboy class would be renamed ArduboyBase.
Function logoIntro() in the ArduboyBase class would be made pure virtual, thus making ArduboyBase an abstract class.
In ArduboyBase.h:
virtual void logoIntro() = 0;
The #define SHOW_LOGO and tests for it would be removed and logoIntro() would always be called.
A new library and class would be created called Arduboy, in its own directory separate from ArduboyBase. Class Arduboy would be a derived class inheriting ArduboyBase. The Arduboy.h file would be:
#ifndef Arduboy_h
#define Arduboy_h

#include "ArduboyBase.h"

class Arduboy: public ArduboyBase
{
public:
  void logoIntro();
};

#endif

Arduboy.cpp would contain the code for function logoIntro() as it's currently written.
So sketches would remain unchanged, with a #include "Arduboy.h" and the logo would be displayed as it currently is.
Now, for sketches that want to gain some space by not showing the logo, we create another library called ArduboyNoLogo, again in its own directory. This library still defines a Arduboy class (not ArduboyNoLogo). The difference is that its version of logoIntro() would do nothing, and also any bitmaps wouldn't be included (e.g. logo[] in assets.h).
In Arduboy.cpp there would be void ArduboyNewLogo::logoIntro() {}.
A sketch developer would just have to replace #include "Arduboy.h" with "ArduboyNoLogo.h" and, viola, no logo is displayed and more code space is available.
This technique makes it possible to easily have alternate logos, as well. If sketches that had lots of free code space wanted a logo that zoomed in, with fireworks around it, and played the 1812 Overture in 4 part harmony, a library called ArduboyFancyLogo could be created and included in place of Arduboy :grinning:

If it turns out that we want to address the possibility that some users may want to turn off the logo because it annoys them or they want start up to be faster, I think there should be a flag reserved in EEPROM to indicate whether the logo should be displayed or not.
The logoIntro() function itself would check if a specific button or buttons were pressed when it starts, and toggle the flag if so. It would then display the logo or just return, depending on the flag.

This doesn't need to be done with two classes and having a software time flag doesn't help you because then you don't get any of the space back - it's still there all the time wasting space.
All we need is begin() and beginWithoutLogo() if we want to make the logo super easy to remove on a per-sketch basis... or even if we don't do that it's not hard for sketches to do a custom startup sequence... I was planning on documenting that in the README IF we figure out how to get the logo working at all in a reasonable amount of space.

If it turns out that we want to address the possibility that some users may want to turn off the logo because it annoys them or they want start up to be faster, I think there should be a flag reserved in EEPROM to indicate whether the logo should be displayed or not.

I'm not opposed to this but I'll leave it up to Kevin.  Honestly I think the branding is important and we should have found a way to put it in the boot ROM so no one can even disable it at all... and it's already going to be easy enough for sketches to turn it off that want to... so I don't know if we need to go all the way and make it a configurable setting based on EEPROM.
I'll ask Kevin.

Closing this, PR for boot logo was just merged.


Server explorer shows list of tables unsorted

When I want to explore the tables on my MariaDB server
the list of tables is not sorted alphabeticaly. This is very confusing.
I'm using the addin version 0.2.6
Server version is: 10.3.36-MariaDB-1:10.3.36+maria~ubu2004

Also experiencing this.  They appear to be displayed in the order the unsorted information_schema.tables but I think most folks would prefer alpha numeric sort.

@mkn-trumpf @kpconnell we have identified the issue and have fixed it, will be released in the next version.


dry-types 1.5.1 is broken with dry-logic ~>1.3

Describe the bug
I guess it could be expected (i. e. dry-types should be updated to >~ 1.6 to make it work), but for the user it's unexpected.
To Reproduce
require 'bundler/inline'

gemfile do
  source 'https://rubygems.org'
  gem 'dry-types', '1.5.1'
  gem 'dry-logic', '~> 1.3'
end

module Types
  include Dry.Types()

  Email = String.constrained(format: /\A[\w+\-.]+@[a-z\d\-]+(\.[a-z]+)*\.[a-z]+\z/i)
  Age = Integer.constrained(gt: 18)
end

$ ruby /tmp/d.rb
…/gems/dry-types-1.5.1/lib/dry/types/constraints.rb:20:in `block in Rule': undefined method `build' for Dry::Logic::Rule::Predicate:Class (NoMethodError)

          Logic::Rule::Predicate.build(
                                ^^^^^^

Expected behavior
It works!
My environment

Affects my production application: kinda? Not really, but dependabot PR for dry-types cannot pass CI because of that.
Ruby version: 3.1.2
OS: linux

I don't think there's much to be done beyond updating dry-types. Since dry-types depends on dry-logic, one can consider it a bug in dry-types.


[WIP] Caching parsing artifacts

This enables caching code artifacts.
Should we store cache in derived data? I think that would require users to pass that data in from their build phase
cc @bejar37 @kzaher @bolismauro take a look, I'd also appreciate some performance comparison if you can :)

          2 Warnings

:warning:
PR is classed as Work in Progress

:warning:
Big PR

  Generated by :no_entry_sign: danger

btw. @ilyapuchka I had to flatten the classes because NSCoding didn't play well with nested types

@krzysztofzablocki about 83% faster with respect to master
How many files NY app has (I've read it takes less than 1 sec)

@bolismauro a bit over 300 files

I'm seeing similar improvements to @bolismauro, 👍 !!


Comments from another user not visible on Safari, requiring more than a simple refresh to recover them.

arpaddanos recently submitted new coordinates for a variant (see page below). He noticed that Malachi had commented on them but when he looked at the page on Safari, the comments weren't there. A simple refresh did not fix the problem. They were visible on my computer, regardless of browser. He could view them in Firefox but the only fix for Safari seemed to be to use the "clear web data" option in his safari preferences.
https://civic.genome.wustl.edu/#/events/genes/20/summary/variants/413/talk/revisions/list/726/summary#variant

It looks like Safari caches XHR calls, unlike the other browsers. After a bit of research, it appears that there exist a couple of work-arounds. Safari pays attention to { cache:false } attribute in the XHR request, and will not cache those responses. Also, it pays attention to Cache-Control: no-cache headers sent from the server.
I'll discuss with Adam how to best implement the fix for this.

I was unable to reproduce this on the current master. To test, I did the following:

Using Safari I navigated to a variant view, and hit the 'Edit Coordinates' button, typed in a few coordinates, hit Submit, and then View Now.
I then logged in as a different user on Chrome, navigated to the new revision detail page, and entered a comment.
Hit reload in Safari.

When I did this, Safari displayed the new comment from the other user, as expected. I believe this simulates the same sequence of actions that arpaddanos executed. I double checked, and the comments cache is definitely being cleared after a new comment is submitted. So I'm not sure what is going on here.
@kkrysiak, I'm going to close this issue, but the next time you've commented on a variant revision on the live site, could you open up the same page in Safari and verify that your new comment appears after a normal refresh? Then we'll see if this is just happening on the live site for some reason - and please reopen this issue if that is the case.

@arpaddanos notified me of the error. I'm tagging him here so if it comes up again he can reopen the issue.


Post deploy hotfixes

Coverage decreased (-0.006%) to 83.809% when pulling cd32afc7a4603f3aa22640966b9906cc4b08f7b8 on RickCarlino:master into 79da2c40d8d232438a62f51403ab5f2c4838481c on FarmBot:master.


series: Ensure test versioning consistency

Added test versioning: see #470.

My assumption is that not having any test versioning in the bonus tests (https://github.com/exercism/xgo/blob/master/exercises/series/first_test.go) is ok?
I've not added anything as we're unable to redeclare the testVersion & targetTestVersion constants and I didn't want to start adding different tests for different version constants.

as we're unable to redeclare the testVersion & targetTestVersion constants and I didn't want to start adding different tests for different version constants

Yeah, seems like a good reason too.
Just for the record, even if I'm wrong and go test -tags first doesn't run the test version test, I'm still OK with just having just the one. We can change just that one even if a chance affects just the bonus tests, I suppose.


Next.js "Warning: Prop className did not match."

Just noting the solution for me. My babel config was only applying to my server-side and not client-side.
I suggest verifying that your babel config is running on the server-side and client-side, and includes babel-plugin-styled-components.
The cause of my config not loading client-side was https://github.com/zeit/next.js/issues/4301#issue-320945743.

@vjpr Hiya could you expand on the solution with some code example of the steps you took to fix.

Add a .babelrc to the root of your Next.js project that has the styled-components Babel plugin. Something like
{
  "plugins": ["babel-plugin-styled-components"]
}

@mxstbr Thanks I already have this in my Babel, still causing this problem, I will re-open when I can minify my codebase to reproduce.


Danwt/support different app.go

Issues
Partially tackles

https://github.com/cosmos/interchain-security/issues/58

Related to

https://github.com/cosmos/interchain-security/issues/59
https://github.com/cosmos/interchain-security/issues/77

Overview
There is a goal to split provider/consumer apps so that people can build from a minimal viable consumer chain. This PR is paired with another PR in cosmos/interchain-security. The two PR's together

Split the app.go into two app.go's. One for parent and child.
Modify ibc-go testing to be able to use two different app.go's.

Therefore this PR should be merged together with

https://github.com/cosmos/interchain-security/pull/84

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

@jtremback - we said earlier to fork the testing parts of ibc-go into a separate repo and make changes there. That can still be the plan, but for now I have just changed informalsystems/ibc-go


Gemspec: Drop EOL'd property rubyforge_project

The RubyGems property rubyforge_project is removed without a replacement.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Z Index

Is there a way to configure the z-index? I have a fly out menu which shows under the sticky.

IMO, you should use a custom class for the sticky (when it stuck) and set right z-index number for it to make it works with your menu.

@tucq88, element level styles override class based styles so thats not an option.

Actually I was able to override using !important, thanks.
On Mar 8, 2016 9:25 AM, "Rickert Mulder"<EMAIL_ADDRESS>wrote:

@tucq88 https://github.com/tucq88, element level styles override class
based styles so thats not an option.
—
Reply to this email directly or view it on GitHub
https://github.com/d-oliveros/ngSticky/issues/122#issuecomment-193802910
.

Right, I forgot about !important. I just forked the project and implemented a sticky-z-index attribute. I would a PR be worthwhile?

@circlingthesun :+1:  I'm not sure about the z-index PR, because it could be done by using a custom class via options so I think we should not set it as default :D IMO, the more flexibility, the better :smile:


create a parameter file that will be saved with each model output run

create a parameter file that will also be moved into the output directory with each model run

the makefile PR from chendaniely/multidisciplinary-diffusion-model-experiments#41 creates
copies of the base directory that is used to create a simulation.  This way the config.ini file is saved along with other simulation data


Dialog component causes 'slot):not([inert]' is not a valid selector in jets

Package name(s)
Core (@salt-ds/core)
Package version(s)
1.34.0
Description
I use the Dialog component and when running unit tests using jest (v 29.5.0) and @testing-library/react (v 14.0.0), I get the following error message:
SyntaxError: 'slot):not([inert]' is not a valid selector
  at emit (node_modules/nwsapi/src/nwsapi.js:557:17)
  at Object._matches [as match] (node_modules/nwsapi/src/nwsapi.js:1400:9)
  at Array.Resolver (eval at compile (node_modules/nwsapi/src/nwsapi.js:760:17), <anonymous>:3:68)
  at match_assert (node_modules/nwsapi/src/nwsapi.js:1342:13)
  at Object._matches [as match] (node_modules/nwsapi/src/nwsapi.js:1360:16)
  at getCandidatesIteratively (node_modules/@floating-ui/react/dist/floating-ui.react.umd.js:1718:38)
  at tabbable (node_modules/@floating-ui/react/dist/floating-ui.react.umd.js:2098:20)
  at node_modules/@floating-ui/react/dist/floating-ui.react.umd.js:2852:34
  at safelyCallDestroy (node_modules/react-dom/cjs/react-dom.development.js:22932:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24103:23)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24018:11)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24062:11)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24018:11)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24018:11)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24118:9)
  at recursivelyTraverseDeletionEffects (node_modules/react-dom/cjs/react-dom.development.js:23989:5)
  at commitDeletionEffectsOnFiber (node_modules/react-dom/cjs/react-dom.development.js:24170:9)

Steps to reproduce

Write a component that renders the Dialog component
Write a test case to render this component
Observe the error

Expected behavior
Test cases for the Dialog component should pass.
Operating system

[ ] macOS
[X] Windows
[ ] Linux
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

Are you a JPMorgan Chase & Co. employee?

[X] I am an employee of JPMorgan Chase & Co.

Which version of jsdom are you running? I can't reproduce this using
npm list jsdom nwsapi
/temp/vite-salt
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>
Tried with test code from default example
  test("can render Dialog", async () => {
    render(
      <SaltProviderNext>
        <Default />
      </SaltProviderNext>
    );
    expect(
      screen.getByRole("button", { name: "Open default dialog" })
    ).toBeInTheDocument();
    userEvent.click(
      screen.getByRole("button", { name: "Open default dialog" })
    );
    expect(await screen.findByRole("dialog")).toBeInTheDocument();
    screen.debug();
  });

I'm using jest-environment-jsdom, which uses jsdom under the hood:
├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ ├── UNMET OPTIONAL DEPENDENCY<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>

I'm using jest-environment-jsdom, which uses jsdom under the hood:

which version of nwsapi?

I'm not using nwsapi. I installed it and now tests are passing. Why is this not declared as a dependency in salt-ds?

It is not salt dependency, but jsdom. Your previous dependency is out dated.

With more testing, this error is a problem specifically with nwsapi with 2.2.3 and 2.2.4. To confirm whether you're experiencing the same, run npm list nwsapi to see which version are you using.
If you have above problematic version installed, you'll need to upgrade them to at least 2.2.5 (check latest version on npm). I didn't find a straightforward command to update this, given it's likely a transient dependency in your project. Something like below should work

With npm, you can do npm install nwsapi && npm uninstall nwsapi
With yarn@1, you can do yarn add nwsapi && yarn remove nwsapi


main: Fix two memory leaks and fix order of destructor calls

When Tesseract terminates by calling the exit function,
the destructor of any local auto variable is not called.
Fix two cases by using static variables.
Signed-off-by: Stefan Weil<EMAIL_ADDRESS>

Maybe we should use return instead of exit()

Do we care about such leaks?
api and renderer hold something important?

Tesseract code includes more than 70 calls of exit(). Using return is a lot of work, even in main() (I already tried it). And static variables also reduce the stack frame size. I know someone who will be happy. :-)

Probably exit()s can be replaced with exceptions?
And print errors in main.

Yes, that sounds like the correct solution for C++ code.

but not in Google project...

Ah, yes, remember their policy. :(

The 2nd commit fixes issue #529. Now Valgrind is happy, too:
LEAK SUMMARY:
   definitely lost: 0 bytes in 0 blocks
   indirectly lost: 0 bytes in 0 blocks
     possibly lost: 0 bytes in 0 blocks
   still reachable: 8 bytes in 1 blocks
        suppressed: 0 bytes in 0 blocks

ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

You must be kidding...


Fix a nil pointer error when a job has no ResourcesConfig.

Some combination of mixing new and old APIs has lead to me having some jobs that have no ResourcesConfig attached to the task, and so when converting to a legacy jobspec, a nil pointer causes a panic.
Pre-emptively does the same with timeouts.
cf get ae17be23 (or list if you have an old enough job)

The thing is, ResourcesConfig should never be nil if the Task was normalized right after creation. This PR https://github.com/bacalhau-project/bacalhau/pull/2890 should've fixed most of those issues, but it seems there are other areas I am missing. If you have old jobs in your datastore, then that PR should've normalized the job right after deserialization

Looks like that fix didn't make it into the 1.1.1 release, so it looks like it'll be fixed in 1.1.2


Allow to put generated files into project level, to allow Rider/R# analyse the code

I have an issue, when I added .proto file into server project, installed Nuget package and R# cannot analyze the source code of generated files, because it is not in the project and created only on build step.

I would added result file into .gitignore and put it not into the temporary folder but inside the project, and have a build task as a "pre-build" code generation.

This issue is making the tool unusable for my scenario


i intall to 2ds and slow as faaakkk

what can i do to speed up the transfer?
the speed is 500kb.

It works on my machine with around 2MB/s.

It works on my machine with around 2MB/s.

what is the version you use?
and what's 3ds or 2ds console you have?

boop.exe uses the wifi speed to download

It works on my machine with around 2MB/s.

what is the version you use?
and what's 3ds or 2ds console you have?

O2DS and the ltest version of Boop. It doesn't work at Wifi's speed but it's pretty decent.

hello.
i know this is a old problem of mine, but there is the problem nowadays too.
what is the port number you using with boop?


Are there any plans to release v0.4.1?

hi all, i created a pr #225 and it has been merged, in this pr i update the surfman version from 0.4.0 to 0.4.1,  are there any plans to release v0.4.1 on https://crates.io/ ? now i can only modify cargo.lock to use new version surfman.

Published.