Width of database name to narrow

Azure Data Studio Version: 1.32 / 1.33 Insiders
OS Version: Windows 10 "current"

Steps to Reproduce:

New Query
Connect to server
Choose a database that has a name longer than 24 characters, on a server where more than one database has the same prefix of at least half of those characters
Run a query and then forget what database you ran it against because the database selector is too narrow 😄 .

I know it is possible to look down the bottom right corner but that's not as convenient as having close to the query you're working on.

Does this issue occur when all extensions are disabled?: Yes

@alexangas the full form of the current context is also available at the application header bar:

hope this helps.


Add chunk history

addresses #58
Changes

Added chunking history for incremental method - necessary to keep track of during snakemake call.

Summary
Just implemented a routine to keep chunking history while using incremental method.
While reviewing maybe focus on ChunkHistory::history():
chunks := []
completed pedigrees := []
for each unique ordered batch number in sample sheets:
  sheet subset := filter sample sheet by batch number and completed pedigrees
  chunk := run chunking on sheet subset
  chunks := chunks + maxed out chunks
  completed pedigrees := completed pedigrees + new fully analysed pedigrees
chunks := chunks + open chunks

Limitations

It is not further investigating if a chunk is closed, it assumes that incremental method takes care of that.
It is recreating the whole history, it could be  improved by just running batch numbers around reasonable splits.


Efrs-839: Recognize all faces and get result on the UI for all faces when there is more than 1 face on the image

recheckcla

@pospielov could you please sign CLA

I have read the CLA Document and I hereby sign the CLA


[Windows] Session still logged in when uninstalling then reinstalling

OS: Windows 10 Build 10586.218
Procedure:

Login to Messenger for Desktop
Exit the app
Uninstall
Reinstall
Run the app

Expected Result:

App is opened in login screen

Actual Result:

Previous account is still logged in

Its a known bug, will try to fix it on my fork

Should be fixed by https://github.com/Aluxian/Facebook-Messenger-Desktop/commit/e0bc24621633bdb7fe2349d6f96211f04dd0d457


Apple Push Notification service server certificate update

Hello, I just received an email from Apple asking about updating the certificate. Do we need to do anything to accommodate this change when using this nuget package? Thanks.
Below is the email:
On March 29, 2021, token and certificate-based HTTP/2 connections to the Apple Push Notification service must incorporate the new root certificate (AAACertificateServices 5/12/2020) which replaces the old GeoTrust Global CA root certificate. To ensure a seamless transition and to avoid push notification delivery failures, verify that both the old and new root certificates for the HTTP/2 interface are included in the Trust Store of each of your notification servers before March 29.
Note that Apple Push Notification service SSL provider certificates issued to you by Apple do not need be to updated at this time.

Hi, please tell me. On my mac, everything works, but as soon as I upload a project to linux with SendAsync, I get an exception:
"The SSL connection could not be established, see inner exception."
Have you ever encountered such a thing? And a few weeks ago everything worked exactly

i fixed it
var httpClientHandler = new HttpClientHandler
{
// Return true to allow certificates that are untrusted/invalid
ServerCertificateCustomValidationCallback =
HttpClientHandler.DangerousAcceptAnyServerCertificateValidator
};
var httpClient = new HttpClient(httpClientHandler);

Yes, I think @MakcStudio solution should work. It's one of the reasons why we made http client injectable - to make sure users have flexibility of customizing communication settings like this.

Hi,
On Linux, you'll probably have to install the AAACertificate certificate or put it somewhere where the operating system is expecting it.
In case anyone is using Azure Functions I've found that you can upload the AAA Certificate Services Root (2028) within the Azure portal via the following menu
Home > App Services > Your-function > TLS / SSL > Public Key Certificates (.cer)

Once I performed the above your plugin automatically picked this SSL certificate up without any code tweaks needed.  I didn't even need to reboot the server, however, I branched the plugin to detect invalid JTW tokens during a push attempt then force a new JWT refresh.  If you're relying on refreshing it every 50 minutes then you may need to restart your function application.
Given the old GeoTrust Global CA root certificate is terminating around March 29th then installing the new AAACertificate above means you shouldn't need to change it until 2028 (my understanding is the certificate is valid from 2020 until 2028) at which point you can expect Apple to send you another certificate before it expires.


base parameter in RsXSLT.InClasspath.resolve(String, String) is unused

@yegor256 please do something about this issue

@simonjenga it's a valid concern, you're right, thanks

@simonjenga I added milestone 1.0 to this issue, let me know if there has to be something else

@simonjenga thanks for the report, I topped your acc for 15 mins, payment ID 70732064

@danff this task is for you, keep these principles in mind, and don't hesitate to ask any technical questions. This task's budget is 30 mins. This is exactly how much will be paid when the problem explained above is solved. See this for more information

@davvd Pull request has been merged

@davvd Pull request has been merged

@danff which one?

@davvd #474 - RsXSLT.InClasspath resolve() method was not using "base" parameter.

@davvd #474 - RsXSLT.InClasspath resolve() method was not using "base" parameter.

@danff thanks

@simonjenga @davvd The pull request #474 for this issue was merged and closed. Can this issue be closed? Thank you.

@danff Lets add a check to see if base is empty: if (base == null || base.isEmpty())

@simonjenga Ok

@simonjenga @davvd Pull request #479 with base.isEmpty() check has been added

@danff check this "no obligations principle".. This task is on your name for at least 15 days. If you can't close it within the next 48 hours I will have to assign someone else to it. This article should help if you're stuck... -30 to your rating, your total score is -30

@davvd The ticket actually was implemented, and the last commit (pull request #479) is waiting for review for 9 days.

@simonjenga @davvd At last the pull request #479 has been merged. Can the ticket be closed?
@yegor256 @davvd The last pull request for a cosmetic change has been reviewed and merged in 10 days. Process that I can not influence in any way. Why -30?

@davvd The ticket actually was implemented, and the last commit (pull request #479) is waiting for review for 9 days. Why -30?

@danff the task is "done" when it's closed by its author. this task is not done. when @simonjenga closes it, I will consider it done. see http://www.yegor256.com/2014/11/24/principles-of-bug-tracking.html

@davvd But the task can not be done/closed till the PR is merged. And PR was reviewed/merged by @yegor256 in 10 days. And I am penalized because of this??? I did not received any argument why I am penalized here. Only the link to the principles of bug tracking which does not answer my question.

So, as I have been advised:
@davvd "I need more time here" because it took another 10 days to merge a PR for a cosmetic change and now I am waiting for the ticket to be closed or commented.

@simonjenga Can the ticket be closed?

@danff A ticket is only closed if all requirements have been met because once closed, it cannot be opened. So, always be patient :)
I was waiting for #479 to be approved and merged by the Architect.
Closed, OK!

@simonjenga I was already too patient and as a result have my rating -60 because of merge delays. Thank you for closing the ticket.

@elenavolokhova please, check this issue for QA compliance, as per par.24

@danff
According to our quality rules:

The ticket describes one specific problem and provides a single solution.

Two PRs were created for this issue. Please try to follow this rule in your tickets in the future.
Please confirm that everything is clear and you have no questions regarding this approach.

@elenavolokhova Everything clear. Thank you.

@danff Thanks!

@davvd  Quality is acceptable here.

@davvd  Quality is acceptable here.

@elenavolokhova thanks for the review, we'll try better next time

@danff added 10 mins to @elenavolokhova, for QA review, payment code is 73998599; I just added 30 mins to your account, many thanks for your contribution (AP-5P373388N0205934W).. 426 hours and 58 mins spent here; +30 added to your rating, at the moment it is: -30


Check status once again before committing

Hello Crowdin folks!
The reason for that is that some users may want to remove all
translation before downloading them again, for instance, to make sure
there are no stale translations.
When that is the case, the first check will indicate there are changes,
since files were removed.
However, when the download files are the same as the files that were
previously removed, git add . will cause the next git status -s to
be clean. If the second check is not in place, the script will attempt
to commit the changes, but as there are none, the action will fail,
instead of just being skipped.
Snippet of a config we are using:
name: Crowdin Sync

on:
  push:
    branches: [main]

jobs:
  sync_with_crowdin:
    name: Sync with Crowdin
    if: github.repository == 'github/docs-internal-downsized'
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@1e204e9a9253d643386038d443f96446fa156a97
        with:
          fetch-depth: 0
          lfs: true

      # This will cause git status to be dirty, no matter what
      - name: Remove all translations
        run: git rm -rf --quiet translations/*/content

      - name: Sync
        uses: mjacobus/github-action@d087d57f31ccaa817e95e626ee199e6e5b839159
        with: "..."

The last lines of the output
Your branch is up to date with 'origin/remove-all-files-before-sync'.
+ git show-ref refs/heads/translations
+ '[' -n  ]
+ git checkout -b translations
Switched to a new branch 'translations'
+ git status -s
+ '[' -n 'D  translations/pt-BR/content/dummy/foo/index.md
D  translations/pt-BR/content/dummy/foo/new-bar.md
D  translations/pt-BR/content/dummy/index.md
D  translations/pt-BR/content/index.md
?? translations/pt-BR/content/' ]
+ echo 'PUSH TO BRANCH translations'
+ git add .
PUSH TO BRANCH translations
+ git status -s
+ '[' '!' -n  ]
NOTHING TO COMMIT
+ echo 'NOTHING TO COMMIT'
+ return
+ '[' -n  ]

Hi @mjacobus!
Maybe we can do something like this https://github.com/crowdin/github-action/pull/94/commits/b71982d5435733ef36eabc95909e654f74d4c10f?

@VBeytok

Maybe we can do something like this b71982d?

Totally! That reduces the cyclomatic complexity. Thank you!


Tapping back button on main decks grid page crashes the app

Tapping back button few times on the main decks grid crashes the app and app goes to a black screen. We should make sure if does nothing.

Fixed in https://github.com/vanadium/syncslides/commit/839cee7af5cb94b28b28ca957787b6a9a81ad704.

Thanks!


Whatsapp instead of E-mail

Is your feature request related to a problem? Please describe.

A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Hey I think there is a problem with sending an email to the trusted ones.

Describe the solution you'd like
---- My solution is what if we can send whatsapp messages to the closed one because as much as people are active on whatsapp they are not anywhere . and anyone's first close ones would be parents and may be some parents don't know how to check mail but they can surely check whatsapp.

A clear and concise description of what you want to happen.

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've considered.

Additional context
---- I don't know how to implement it. I am a GSSoC participant was scrolling through projects so thought of this!

Add any other context or screenshots about the feature request here.

@sonal251001 Work on this issue and add a alternate to the whatsapp message if any user does not have account on whatsapp then message should go on email and SMS both.

@sonal251001 Any updates on the issue?

No. I'm sorry but I wrote above that I don't know how to do it. I'll do it if anyone can guide me.

@sonal251001 Okay! No problem.

I think we can do it using pywhatkit. Can I try this one? @sakshikhachane

@Tarun-Kamboj Go ahead with the issue. Happy learning:)

I've implemented sending whatsapp messages to the contacts of the user.
Here are the changes that I've made.

User will now also be adding the the mobile numbers of their contacts.

Modified the views, form and model accordingly.
Added a script to send whatsapp messages.

It sends a message to one contact per minute. I know it's quite slow but it was sometimes failing to send the message if I kept the time lower than that. See the pic below for reference.

If the number is on whatsapp then it sends the message successfully.

If the number is not on whatsapp then it simply shows the following message and proceeds to the next number.

@Tarun-Kamboj Great work! Please make a PR so that if there are any changes, then they can be suggested.

@Tarun-Kamboj Great work! Please make a PR so that if there are any changes, then they can be suggested.

Done @sakshikhachane, please review.


Fix build on wasm

Fixes WASM builds (I broke them in https://github.com/tesselode/kira/pull/64)

thank you!


PendingStateManager: Light refactoring/renaming (prep for BatchId)

Description
Some light refactoring:

Rename IPendingBatchMessage to PendingMessageResubmitData to better reflect its purpose
Rename IBatch.content to IBatch.messages
Add/use helper for strongly typed access to batch metadata
Remove redundant reSubmit callback for PSM's use; it can/should always use reSubmitBatch.

Everything is always batches in Turn-Based mode, and there's no harm in treating it the same for Immediate mode.
There's no functional change here in this PR, thanks to this bug fix a few months back.

Reviewer Guidance
Suggestion: Hide (leading) whitespace. An if statement is inverted so there's a chunk of indentation-only change in PendingStateManager.

@tyler-cai-microsoft -- any other concerns before approving?


cleans login page, remove footer, remove dead links in header, per tr…

cleans login page, remove footer, remove dead links in header, per trello card #66
Co-authored-by: Rafael Mendoza<EMAIL_ADDRESS>@Retrofitt
Description
The login portal now only display a log in feature.
The header only displays the Underdog Devs logo, but still has the avatar and one navlink named "Open Tickets" for future reference. Another discussion should accompany a Trello card regarding the link names and features. A user should only be able to see the logo, not the avatar or any links if not logged in.
There should be no footer in the entire app because the main home page has all that information and that is not pertinent to the logged in user anymore. So the footer is removed from the login page and the app itself.
https://www.loom.com/share/5717639b3e5d49fa88d70f46b6d5f17b
Type of change
Please delete options that are not relevant.

[X] User Interface / User Experience

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have removed unnecessary comments/console logs from my code
[ ] I have made corresponding changes to the documentation if necessary (optional)
[X] My changes generate no new warnings
[X] I have checked my code and corrected any misspellings
[X] No duplicate code left within changed files
[X] Size of pull request kept to a minimum
[X] Pull request description clearly describes changes made & motivations for said changes

Great work as usual Richard lets get this shipped!


Atlas

Support external MONGO_URL through env.sh
Prod MONGO_URL will point to Atlas when fetched from anet-deploy

 (3) Update backend services to point to Atlas MongoDB cluster [3]


fix: style comparison error when hotUpdate

FIXED: #3358
I think it need not transform code when hotUpdate. File content comparison when hotUpdate should compare raw code, not transformed code. The transformed code include unocss:vue-scoped's generated code and raw code, generated code is not chaged.
There is a comment from @vitejs/plugin-vue
packages/plugin-vue/src/main.ts

If we don't inject styles on handleHotUpdate, all applied styles disappear after HMR (you can try to run it locally to see what happens). Most likely it's the bug of @vitejs/plugin-vue Ref: vitejs/vite-plugin-vue#301

I run it locally and everything seems to be ok. Because unocss:vue-scoped's transform hook is called after every HMR. I think applied styles will not disappear.
return {
    name: "unocss:vue-scoped",
    transform(code, id) {
      if (!filter(id))
        return;
      return transformSFC(code); // will run after every HMR
    },
    handleHotUpdate(ctx) {
      // const read = ctx.read;
      // if (filter(ctx.file)) {
      //   ctx.read = async () => {
      //     const code = await read();
      //     return await transformSFC(code) || code;
      //   };
      // }
    }
  };

Yes, you are right, transform hook is called. But updated styles are not pushed
Reproduction: https://stackblitz.com/edit/vitejs-vite-mnvzcb?file=src%2FApp.vue
Wait for dev server to start and change class text-red to text-blue. The text should change its color, but it doesn't do so
I just commented out handleHotUpdate code (patch-unocss.mjs).
But maybe we need to push this change as temporary workaround? I guess he runtime error is worse than missed hot update

Is there a temporary solution now?


Indian Currency

Icon Request

Icon name:
Tags of this icon:
Use case:
Screenshots of similar icons:

### @xiaochunjimmy Can you assign it to me

https://remixicon.com/icon/money-rupee-circle-line
https://remixicon.com/icon/money-rupee-circle-fill


Add NoPivotRoot flag, to allow k3s to work on initrd/RAMdisk (for example buildroot)

Proposed Changes
Add NoPivotRoot to configure containerd to work on initrd / RAMdisk, will add the --no-pivot-root option on server / agent.
Types of Changes
Added flag for --no-pivot-root (agent and server which boots agent), this will append configuration to the plugins.cri.containerd.runtimes.runc.options, where it will add NoPivotRoot = true (when true). The default for windows = false.
Verification
Normally you would get (in initrd / RAMdisk):
jailing process inside rootfs caused \\\"pivot_root invalid argument\\\"\"":
now containers will spawn correctly (with --no-pivot-root flag enabled)
Linked Issues
https://github.com/k3s-io/k3s/issues/510
User-Facing Change
Add --no-pivot-root flag for agent / server for running inside initrd

Further Comments
I've added it to AgentConfig, since DIsableCgroup is also inside the AgentConfig. It however seems more like a Containerd config, but I left this alone.

Is there any way for K3s to detect when it is operating in an environment where this option is required? We generally try to avoid adding CLI flags unless its absolutely necesssary.

Will try something else, according to:
https://github.com/opencontainers/runc/blob/214c16fd9a3bd9fe56a2476933e31749a7c0576b/libcontainer/rootfs_linux.go#L876

The kernel only allows a mount of a pseudo-filesystem
like procfs or sysfs if there is a full mount (the root of the
filesystem is mounted) without any other locked mount points covering a
subtree of the mount.

I guess what they mean is that the / (root) should be mounted, which isn't on RAMdisk. Do you think that is correct?
If so we could get the mountinfo ("github.com/moby/sys/mountinfo") of / (root), and if it is non existing, activate NoPivotRoot.

Hey @cwayne18 , (speaking on behalf of https://github.com/kairos-io/kairos)
that'd be really of help to have it here ! As we are working towards Trusted Boot support, this is actually something that breaks entirely starting k3s in such environments.


Pipeline to test AwsBI module

Is your feature request related to a problem? Please describe.
Create pipeline to run tests on PR merge
Describe the solution you'd like
Pipeline should invoke tests prepared in #1676
Describe alternatives you've considered
No.
Additional context
Check #1641 for information how t was done for AZBI module.

This task was separated from #1643

PR #32

related PR

The pipeline has been tested and works properly. The test results are visible in the dashboard in Azure DevOps.


Release licensed 3.0.0

3.0.0
2021-04-27
This is a major release and includes potentially breaking changes to bundler dependency enumeration.
Changed

The bundler source will return an error when run from an executable.  Please install licensed as a gem to continue using the bundler source.  Please see the v3 migration document for full details and migration strategies.

the bundler test failures are known and fixed as part of https://github.com/github/licensed/pull/354 in https://github.com/github/licensed/pull/354/commits/4a54a34d9675dbf6f21383e555c6511d1fafdf3c

I'm going to wait to merge this and put out a new version until tomorrow AM (PDT).  If the release causes any user issues I'd like to be around to address them or pull the release during my daytime 😄


KOGITO-7133: Extend MultipleInstanceSubprocess IT test vscode-extension BPMN editor & Add support for offline tests

Extend MultipleInstanceSubprocess test
Add test for UserTask On Entry/On Exit action
Add support for offline tests

IT tests for the VS Code Extensions are currently disabled on our CI, as they were too unstable and were causing random build failures. See https://issues.redhat.com/browse/KOGITO-6657
This PR introduces a command that is not run as part of the CI, so no way to verify that it's working and will continue to do so.
Can you please share what's the strategy to make the new tests part of our CI checks?

The tests are run nightly on our Jenkins without issues.
Command is useful for development, it will be tested.
We are focused on SWF now, there is no strategy for this.

@domhanak Thank for the info. If you're not having any issues on these tests, why not bring them back to the community CI?
Anyway, I'm not opposing to this change, just wanted to highlight that by not running this command often, it might become outdated and loose its value.

@domhanak @zdrapela What's the status of this PR?

@domhanak @zdrapela What's the status of this PR?

I'm working on it. Will be revised in the next few days. Sorry for the delay.


[Bug] [1.8.01]: Seems like it doesn't support sheet code from Sheets Extended

What happened?
I created a table in sheet code block like this

in Obsidian is rendered fine

but the export look like this

Reproduction

install Sheets Extended
paste the following code into a code block with language sheet

| 主層 |  <  | 電子數  |
|:----:|:---:|:-------:|
|  K   |  1  |    2    |
|  L   |  2  |    8    |
|  M   |  3  |   18    |
|  N   |  4  |   32    |
|  O   |  5  | 18 / 32 |
|  P   |  6  |   18    |
|  Q   |  7  |    8    |

export it

Last Working Version
No response
Version
1.8.01 (but also tried 1.9.0-2b)
Operating System
Windows
Obsidian Version
1.6.7
Which browsers can you reproduce this in?
No response
Relevant log output
Log:
[WARNING] 	Creating an RSS feed requires a site url to be set in the export settings.
[WARNING] 	No existing search index found. All files will be exported.
[INFO] 	Replacing "1-1 電的特性與應用" header because it was very similar to the file's title.
[INFO] 	Replacing "1-2 電的單位" header because it was very similar to the file's title.
[INFO] 	Replacing "1-3 電能" header because it was very similar to the file's title.
[INFO] 	No old files to delete
[INFO] 	Closing render window
[WARNING] 	Creating an RSS feed requires a site url to be set in the export settings.
[WARNING] 	No existing search index found. All files will be exported.
[INFO] 	Replacing "1-1 電的特性與應用" header because it was very similar to the file's title.
[INFO] 	Replacing "1-2 電的單位" header because it was very similar to the file's title.
[INFO] 	Replacing "1-3 電能" header because it was very similar to the file's title.
[INFO] 	No old files to delete
[INFO] 	Closing render window

Settings:
	settingsVersion --------- 1.8.01
	makeOfflineCompatible     false
	inlineAssets ------------ true
	includePluginCSS          1 plugins included
	includeSvelteCSS -------- true
	titleProperty             title
	customHeadContentPath --- D:\\\\GhostShadow\\Obsidian\\Electronic\\head.html
	faviconPath
	documentWidth ----------- 40em
	sidebarWidth              20em
	minOutlineCollapse ------ 2
	startOutlineCollapsed     false
	allowFoldingHeadings ---- true
	allowFoldingLists         true
	allowResizingSidebars --- true
	logLevel                  warning
	minifyHTML -------------- true
	makeNamesWebStyle         false
	onlyExportModified ------ false
	deleteOldFiles            true
	addThemeToggle ---------- true
	addOutline                true
	addFileNav -------------- true
	addSearchBar              false
	addGraphView ------------ false
	addTitle                  true
	addRSSFeed -------------- true
	siteURL
	authorName --------------
	vaultTitle                基本電學
	exportPreset ------------ documents
	openAfterExport           false
	graphAttractionForce ---- 1
	graphLinkLength           10
	graphRepulsionForce ----- 150
	graphCentralForce         3
	graphEdgePruning -------- 100
	graphMinNodeSize          3
	graphMaxNodeSize -------- 7
	showDefaultTreeIcons      false
	emojiStyle -------------- Native
	defaultFileIcon           lucide//file
	defaultFolderIcon ------- lucide//folder
	defaultMediaIcon          lucide//file-image
	exportPath -------------- D:\\\\GhostShadow\\Obsidian\\Electronic\\html
	filesToExport             3

Enabled Plugins:
	Code Editor Shortcuts
	Media Extended
	Omnisearch
	Fast Text Color
	Task Collector (TC)
	Paste URL into selection
	Mousewheel Image zoom
	Markdown Attributes
	Webpage HTML Export
	Advanced Tables
	Extended Syntax
	Sheets Extended

Additional Info

I have tested this on the internal developement branch and it works, so I can say that at least in the next release this will be fixes


Adding Visual Studio Code plugin.

As some of you already know, I've been working on a VS Code plugin (on and off for a weeks now). This is it :)
I've included a v.0.1.0 binary in the repository with a link in the README to download it. Building it is fairly simple, and involves downloading a tool and using it as described here: https://code.visualstudio.com/docs/tools/vscecli
It's not published to the marketplace yet, so you can do that yourself by generating an access token on VS Team Services, creating a publisher, and running the appropriate command line tools.
Let me know if you have any questions!
Bhargav

@bnookala This is really great, thank you once again! So from this point we could either do code review on this branch right here with you, or we could go ahead and code review/iterate on it internally. If you have time and inclination to review/iterate with us then it would be great to have you involved, but I'd also understand if you would rather that we pick it up internally from here (you've already helped us out tremendously!). What would be your preference?

@alexflint Let's go ahead with the code review!

Hey @alexflint, any updates on this? :)

So sorry for the delay @bnookala, will make sure this happens this week.

@bnookala played around with it and it works great! just a couple of minor comments otherwise

Hey @dhung09, mind taking one more look at this?

@alexflint, @dhung09 Don't mean to sound like a broken record, but it would be really great to see this pulled!

Agreed

sorry about that guys! i just got back from vacation. will be taking a look at this tonight

this LGTM! will merge this now and get to work publishing this to the marketplace and incorporating it into our installer. thanks for your helping out with this @bnookala - we all really appreciate it


fix: use duration field of AssertionResult to report test duration

Some archaeology shows that it's been there since at least facebook/jest@dc355000db, from 24.2.0 (or earlier).
I hope we can chance breaking compatibility for people using pre-2019 versions of jest...
Fixes #516

Hi, thanks for the change, it looks good! However I would be more comfortable releasing this as a major. Can you please amend the commit message so that it reports this breaking change? Then the infrastructure will release this as a major, just to be extra safe :)

Sure thing! That makes sense.
What's the magic words I need? major:? BREAKING CHANGE? (Never used semantic-release before, sorry)

It is the BREAKING CHANGE: in the commit body, please see the previous such commit that created version 2.0.0 :)

Updated the commit message! (Sorry, I thought I pushed on Friday but apparently Git decided to give up on me...)


Find a better way of experiencing hype using party button and easter eggs

Currently, the party button and some easter eggs change the entire background completely, this causes problems for people who are sensitive to light (me). I will try to find a way to substitute the background changes with some smaller CSS elements instead (such as buttons and borders).

did a solution, maybe we will find another in the future?


"Save to Album" attribute saving original image on album instead of compressed/resized one

Bug
Version Number of Plugin: 2.5.1-beta 40
Device Tested On: Moto X Play - Marshmallow 6.0.1
Expected Behavior
Android album should display the compressed or resized picture.
Actual Behavior
When taking a picture with the "Save to Album" attribute enabled with the compression and/or size attribute set to a value, the original unmodified picture is sent to the album while the modified one is saved in the temporary folder.
Steps to reproduce the Behavior
Enable the "Save to Album" attribute and set the quality and/or resize attributes to any level other than the default one and take a picture. When opening the album, the original unmodified picture will show up instead of the modified image. Write/read permissions are enabled.

This is the expected behavior and how I programmed it.


Which programs do I need to allow through my firewall?

[x] I think my problem is NOT with youtube-dl
[x] I've verified and i assure that I'm running yt-dlg 1.X.Y
[x] I assure that i am using the latest version of youtube-dl
[x] Searched bugtracker
[x] I've read the FAQs file

What is the purpose of your issue?

[ ] Bug report
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Do I need to allow any additional programs besides yt-dlg.exe through the firewall, or make any similar adjustments to my firewall so that this program is able to connect to the internet? I see youtube-dl.exe is used in the backend, so I wonder if the program gets automatically stored in an unspecified location on the disk.
The way it is right now, I have an outbound allow rule for yt-dlg.exe in Windows Defender Firewall, in Group Policy. However, downloads fail unless I turn the firewall off, or use youtube-dl without the interface.

The setting and the CLI backends like youtube-dl and yt-dlp you can find in:

Windows: %APPDATA%/yt-dlg
GNU/Linux:$HOME/.config/yt-dlg/

Reed more in this comment

Thank you!


Compare mode: Change the regenerate button to an icon

Description
Show the regenerate icon as for usual chat for compare mode.
Use case/motivation
No response
Related issues
No response
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging successfully


DO NOT MERGE

Meeting notes demonstrating how to generate root index.ts.

Meeting notes demonstrating how to generate root index.ts.

Thank you @SurferJeffAtGoogle. Working on that.


document subpath support

Document subpath support. This complements the more developer focussed documentation at https://github.com/mattermost/mattermost-developer-documentation/pull/128.

@jasonblais, re: install guides, I'm inclined to keep them on the "happy path" and reserve the subpath for more advanced configuration. Does that seem reasonable?

@lieut-data Yeah that's reasonable to me. Probably better to keep the install guide as simple as possible.


Virtual Machines with certificate stored in Key Vault can be moved across subscription as well

I havE tried this in one of my csp migration and key vault does moves please test this and edit the documentation

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

ID: 76c1c185-685f-27e0-e634-844b510b1834
Version Independent ID: 09375778-5354-ab44-c5c6-94c2aebe34bc
Content: Move Azure resources to new subscription or resource group
Content Source: articles/azure-resource-manager/resource-group-move-resources.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@Anmolgan81 Thanks for the comment. This has been shared with the content owner for further review.

@singhkays or @cynthn - can you confirm whether a VM with a certificate stored in a Key Vault can be moved to a new subscription? We have a note saying it cannot be moved. If the Key Vault is moved with the VM to the new subscription, does that work?

@Anmolgan81 Can you share your sub-id and timeframe this happened so we can take a look at the logs? Also was your VM using Managed Disks or Unmanaged

#please-close

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Extr0py/138/dom rendering

Set up a DOM rendering strategy for ONI.
Few remaining issues:

Performance regression - this re-renders the entire DOM for every change. Need to improve performance (use delta rendering to only re-render divs that were changed)
Some highlighting regressions

This is awesome! I'll give it a go shortly.

@kybaeus , thanks for trying it out! Glad it worked.
I just started working on the delta rendering. For the really performance sensitive pieces (like the rendering), I'm planning on not using React, so that we can have more flexibility in tuning it.  My first render strategy, before the canvas, was using React, and there was some performance overhead that was difficult to workaround.
I definitely prefer writing code in React, so for other things like the the autocompletion/popup menu/etc, the rendering isn't the bottleneck and it makes sense to leverage it.
With the latest changes, the performance seems better, but there is one issue with the delta rendering - sometimes it will split <span> and break the ligature rendering.
So there is the following outstanding work left:

Fix that split span case, I'm planning on adding to the logic adding a character directly to a span in cases where it makes sense (ie, merging with a previous span)
Add unit tests for some of the delta / merging / coalescing logic
Remove the force flag on the delta region tracker, since it was really just a hack
Split out the tokenrenderer items to separate files

More work:

Move console.log -> Log, only log if not debug
Modify Performance to only log if debug
Fix tests after splitting apart files
Fix bug with coalescing spans - should not coalesce whitespace / non whitespace
Fix issue with multibyte characters (make isWhitespace return correct value)
Scrolling - performance work

Have you tried rendering by running setState inside requestAnimationFrame?


Removal of live-coded hotkeys (hotkeys.cpp)

The bind-system has been going through a lot of changes recently, so I was wondering if there is any reason at all to keep the live-coding hotkey system. I currently do not use this myself, although it does allow for more advanced things, that the current bind system cannot (and will not) support.
Examples of this may be simple things such as performing different commands depending on if a program is running or not, time of day, or really anything you could think of.
If there have been no objections here after a week or so, I will simply get rid of this system.
The ability to remap keys will also be lost if this goes through.

I personally think that it can be removed especially if you plan on releasing codesigned binaries for consumption in package managers (homebrew).
The current binding system is plenty powerful and should cover 90% of use cases. The other 10% could fork the project and make modifications as they see fit.

I'm just gonna go ahead and remove this.
If anyone wants to have this, just clone latest commit that still has this system and copy paste the code inside your own fork.


Codecov migration to marketplace app

Hi, Tom from Codecov here.
We noticed that you are using Codecov with fairly high frequency, and we’re so excited to see that! However, because you are not using our app, you may have experienced issues with uploading reports or viewing coverage information. This is due to rate-limiting issues from GitHub.
In order to prevent any future outages, we ask that you move over to our GitHub app integration.
The process is extremely simple and shouldn’t require more than a few clicks, and you should not expect any downtime. By moving to our app, you will no longer need an admin or separate account to manage the relationship with GitHub as the team bot.
Let me know if you have any questions, or if I can help at all with this process.

@thomasrockhu I installed the app on opencollective-frontend, is it ok now?

@znarf yes looks great! Let me know if you run into any issues.


Review build performance

Made some improvement.  It is building on test2 (in the background) at about the same rate as last time we built, so no longer a concern.

While the build looked like it was progressing at about the same speed as our previous build when I compared the time for CON (perhaps even faster), it looks like it is significantly slower now that we are toward the end. There are of course more VLD files now than last time, but not so many more that it should account for how much extra time it is taking.  I believe it took about 2.5 days when we built in November and looks like it will complete in about 4 days this time.
I did some observations using mongotop with a few pieces in WAR. The first 10%-15% of the total time is spent creating the vld_entries; 14%-23% is dwellings/individuals/pieces.  The majority of the time is spent creating search records (65%-80% of the total time).  mongotop shows mongo as quiet for a few (4-7) seconds before the printout of how many dwellings and entries were loaded, perhaps while it catches up from all the writes and index updates?  These timings were done on relatively large pieces, for example WAR/RG12239.VLD has 7854 individuals in 1653 dwellings.
Since the build just runs in the background and is not done often, nothing further needs to be done as long as we can plan on about 4 days of build time for the final build before launch (and any future builds).  If 4 days is too long then additional developer time will need to be spent trying to improve build performance.

I am concerned also that you mention  a Piece for WAR of 7854 individuals
as being relatively large.
We have been uploading pieces just lately more than double/treble + that,
i.e. 15,000 + individuals and a couple between 25,000 to 31,000+
Will that size have any further impact on speed/time?
On Tue, 28 Mar 2017 at 21:49, dougkdev<EMAIL_ADDRESS>wrote:

While the build looked like it was progressing at about the same speed as
our previous build when I compared the time for CON (perhaps even faster),
it looks like it is significantly slower now that we are toward the end.
There are of course more VLD files now than last time, but not so many more
that it should account for how much extra time it is taking. I believe it
took about 2.5 days when we built in November and looks like it will
complete in about 4 days this time.
I did some observations using mongotop with a few pieces in WAR. The first
10%-15% of the total time is spent creating the vld_entries; 14%-23% is
dwellings/individuals/pieces. The majority of the time is spent creating
search records (65%-80% of the total time). mongotop shows mongo as quiet
for a few (4-7) seconds before the printout of how many dwellings and
entries were loaded, perhaps while it catches up from all the writes and
index updates? These timings were done on relatively large pieces, for
example WAR/RG12239.VLD has 7854 individuals in 1653 dwellings.
Since the build just runs in the background and is not done often, nothing
further needs to be done as long as we can plan on about 4 days of build
time for the final build before launch (and any future builds). If 4 days
is too long then additional developer time will need to be spent trying to
improve build performance.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeUKGen/FreeCENMigration/issues/223#issuecomment-289900196,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANC91vKbQKm6OtD7i-1L8mWdG8WBV-fuks5rqXJngaJpZM4Mla0v
.

You are right, Brenda. These with 7,000 definitely aren't the largest.  Just "relatively" large compared to many pieces that only have a few hundred or couple thousand entries. And in fact, perhaps because some of the more recently added pieces do have 15,000 - 31,000+ entries, that actually might account for part of the reason why the build is taking longer.  Even if there haven't been enough new files to account for the longer build time, perhaps if the new files are much large than typical, the overall increase in number of entries to add to the database may account for the longer build time more than I suspected.  Thanks.

timing on  my development machine (with a nearly-empty database) to process WAR/RG122399.VLD:
load/parse/errorcheck:  2 seconds
vld_parser persist_to_database: 21 sec
transform: 38 seconds
translate: 40 seconds
total time to process: 101 seconds

I have switched vld_parser/persist_to_database to use bulk inserts, and found it drops a sample file from 23 seconds down to 7.  cf 79f0864636e8005b3ad8593a21737ae234a2cf4c
I'm not sure that we can take the same approach with transform and translate, but will look into that as well.

The majority of the time was spent in translate (creating the search records) so if we can do bulk inserts there, that would give the biggest performance gain on the database build.


npm install requires installation steps. How to use it with CI?

Hi,
There are two scenarios:

when I use bower install semantic-ui I am getting all packages as expected. No installation process is involved.
when I use npm install semantic-ui I am getting installation steps questions. So that I have a problem how to use such a way with CI. Is there a way to pre-configure semantic installation so that it would install everything automatically ? Or any other possible solution ?

Thank you very much

See #1816, fix is coming

OK. Thank you


⚠ Bofrost.lu has degraded performance

In c87ea15, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1186 ms

Resolved: Bofrost.lu performance has improved in 431b5fc.


Bug:

Describe the bug
the site takes a lot of time to load and respond, makes it looks so buggy and laggy
Steps
go to the web
Expected behavior
the website should respond very fast.
Additional info
No response
Environment
No response

Thanks for creating the issue!  Performance appears normal to me, must have been a transient problem.  Please re-open with more detail if it's still happening on your end.


Using with Arion or docker-compose

I'm trying to figure out how to use nix-snapshotter to run images that are then composed using Arion , nix frontend for docker-compose.
I've set it up like this:
  services.redis = { pkgs, lib, ... }: {
    service.image = "nix:0${pkgs.nix-snapshotter.buildImage {
        name = "ghcr.io/pdtpartners/redis";
        tag = "latest";
        config.entrypoint = [ "${pkgs.redis}/bin/redis-server" ];
    }}";
  };

But when I run it, I get the following error:
❯ arion up
/nix/store/hfs26lgg6qi2cki1kz6s2vwk3x0bjm0z-docker-compose.yaml
Pulling test (nix:0/nix/store/0cdzy9y0bh1qh95blp9rgm5pgwivyhhx-nix-image-redis.tar:)...
ERROR: Get "https://nix:0/v2/": dial tcp: lookup nix: no such host

Does anyone have any experience using nix-snapshotter with Arion or docker-compose?

you could try loading the image first.
in this specific case:
sudo nix2container /nix/store/0cdzy9y0bh1qh95blp9rgm5pgwivyhhx-nix-image-redis.tar

you'd probably want a way that hooks into arion up and does this for you.

@mjmeintjes did you configure Arion to use containerd? looking more into it i got the understanding that it uses docker by default which isn't using the configured snapshotter? would be great if you could share a full config to demonstrate how you could this working.

@mjmeintjes did you configure Arion to use containerd? looking more into it i got the understanding that it uses docker by default which isn't using the configured snapshotter? would be great if you could share a full config to demonstrate how you could this working.

I could not get nix-snapshotter working with Arion, and we've since migrated most of our docker containers to nixos and nixos-containers so I haven't spent any more time on this.


feat(libp2p): command to wait for the relay to be ready

adds a command for the client to wait for the relay server to be ready. in cases where the client wants to make sure the relay is listening before trying to publish a message

@Larkooo should we consider a timeout here, or the client should not be stuck in such situation?

what do you mean by a timeout? like a naive approach where we wait a specific amount of time?
it wouldn't be sustainable because clients might act differently from others, for eg. the wasm client takes more time to receive a handshake from the relay server than a native platform client. also imo we don't want to block clients -  because in a game it would just be completely blocking and potentially introduce a freeze

and also the waiting is needed only for publishing messages - to make sure that the relay server is ready to receive it so we dont lose a message if we try to publish it and the relay server is not ready yet

Perhaps for a subsequent PR we can specify a channel we are waiting the subscription from.

Oh yeah basically, the Subscribed event is only supposed to be coming from the relay server because that event is emitted when a remote peer subscribes to a channel, and the only remote peer we have is the relay server (yes it's weird but it's because the server acts as a relay). So basically for now it's safe to just assume that any subscription event we receive means that the server is ready - because we received its subscription confirmation (to the topic where our message is going to be broadcasted)


Issue 27 fix recognition notification from threads

Fixes #27
Uses Botkit's startPrivateConversation method to set conversation context before sending notifications to receivers of a fistbump.
This fixes a bug where notifications wouldn't be sent if a fistbump was originally sent in a thread.
Updates OAuth scopes to include im:write, which is required for startPrivateConversation.
Update Botkit Slack Adapter to version 1.0.12, to properly support conversation API while using startPrivateConversation.

Awesome work on finding that fix! How'd you figure this one out? Anything else to consider when using the Conversation API?

This one was mostly guesswork unfortunately, I was simply trying out some different ways of sending direct messages with Botkit, and this one happened to work.
There should be no really effects of using the conversations API. Slack deprecated other ways of starting conversations some time ago, such as by using im.open . The Botkit version upgrade was required to make that migration.
Under the hood, Slack's API seems to suggest that using chat.postMessage, without first using conversations.open, should be acceptable and that seems to be the case given that using bot.say worked previously with non-threaded messages. My guess is that this workaround is getting around some issue in Botkit, but it's hard to say without better knowledge of the library.


gRPC c library versioning and leaks on the client side

Firstly thanks a lot for this great work, we're really excited to be using it.
Recently we've been stress testing some services and have noticed what we think is some memory leakage in the underlying c code, but only on client requests (used from high level generated code), not seemingly on server handlers.

I'm planning on doing some profiling, but I'm not an expert on debugging c from haskell, any advice on how I might go about diagnosing if my theory is correct would be greatly appreciated.

Having had a cursory look at the grpc c-lib releases over the last year, it seems there's been a bunch of memory leak fixes (pretty much all releases actually).

We were wondering if there was any plan to try to upgrade this new core library to use the later c-lib versions?

I've just had a look at building this against v.1.8.3 but the c-api has changed a bit:
src/Network/GRPC/Unsafe.chs:228: (column 7) [ERROR]  >>> Unknown identifier!
  Cannot find a definition for `grpc_call_destroy' in the header file.
src/Network/GRPC/Unsafe.chs:145: (column 3) [ERROR]  >>> Function arity mismatch!
  Parameter marshallers are missing.

I'm not really au fait with the nuts and bolts of the low-level c interface, so I'm not sure I can immediately help in this endeavour, or indeed, if it might be desired at all.
Any comments from you folks on this would be greatly appreciated. Thanks in advance for reading and helping!

@jamesthompson: Just to clarify: you're saying that the server is stable memory-wise, but a long-lived client is not?
Also, I think you can run valgrind on a Haskell binary (I have not tried this in a very long time, though, so I could be wrong)
We do plan on upgrading to a newer grpc.  It's something I'm slowly working on

@Gabriel439 That's correct, the server seems to run very happily with a small memory footprint, but long lived clients using withGRPCClient and the high-level generated code exhibits continually expanding memory consumption, despite there being multiple GC cycles (from an +RTS -s analysis).
I'll look into valgrind to see if I can get anything diagnosed.

One question is if you occasionally exit the withGRPCClient and reinitiate that, if that helps...

Any news on this?

@CMCDragonkai: No news yet.  I haven't had a chance to look into this and likely won't within the next month at least

@CMCDragonkai I haven't been using this library for a while now and have recently been trying out https://hackage.haskell.org/package/http2-client-grpc and associated libs to great success. Sorry I can't be of more help right now.

@jamesthompson we were using that library and suffered from really slow streaming bandwidth.


Credentials: Invalid token type

Hi all,
this is likely not a bug but I can't understand what is happening.
When trying to validate/logging in with my Copernicus credentials using the code below:
from cdsetool import credentials

credentials.validate_credentials(username="username", password="password")

I get this error:
DecodeError: Invalid token type. Token must be a <class 'bytes'>

I tested the same code in Google Colab and everything worked correctly. My guess is that since I'm working from the pc of my company, there may be a firewall of some sort that messes things up.
However, if I try a normal query such this:
features = query_features(
    "Sentinel1",
    {
        "startDate": "2020-12-20",
        "completionDate": date(2020, 12, 25),
        "processingLevel": "LEVEL1",
        "sensorMode": "IW",
        "productType": "IW_GRDH_1S",
        "geometry": geometry,
    },
)

everything works. So maybe the issue is with the encryption of the credential?
Thanks

I don't know about the DecodeError, but the reason you can still do queries is probably because those do not require you to be authenticated.

I may have found the problem or at least a temporary solution.
The firewall hypothesis was wrong so  I started debugging the credentials.py and realized that when executing the code line by line, the authentication was sucessful.
The issue I found is that in the constructor, self.__ensure_tokens()  is called just right after the definition of the token expiration times.
        self.__access_token = None
        self.__refresh_token = None
        self.__access_token_expires = datetime.now()
        self.__refresh_token_expires = datetime.now()

        self.__lock = threading.Lock()

        if self.__username is None or self.__password is None:
            self.__read_credentials()

        self.__ensure_tokens()

When in __ensure_tokens the expiration dates are checked:
    def __ensure_tokens(self):
        with self.__lock:
            if self.__access_token_expires < datetime.now():
                if self.__refresh_token_expires < datetime.now():

it seems like (not 100% sure though) that no time has passed and so the if conditions fail leading to a call to self.__jwk_client.get_signing_key_from_jwt(self.__access_token) with a None token (since self.__token_exchange(data) is not called).
The solution I'm using (even though quite janky) is to add a time.sleep(0.1) in the constructor just after the  self.__lock = threading.Lock().
I hope this helps in finding a more reliable solution.

What platform and Python version are you on?
Are you using PyPy or some other heavily optimizing Python implementation?

Windows and Python 3.12.2.
I'm not using PyPy or anything fancy. Just a Python installation using miniconda and an environment where I installed only cdsetool with pip.

I don't know anything about timer resolution on Windows, or optimizations in recent Python versions, but can you try the fix in #123 and see if that works for you?

Yep, that works.
Similar in concept to the time sleep but more elegant.
The timedealta can be of just 0.1 seconds.

The timedealta can be of just 0.1 seconds.

I thought a bit about this and a 1 minute/week clock drift on a laptop isn't unheard of, and depending on configuration, ntp can move time backwards.
Since we have to have the time delta in the initialization, it's better to just set it to something clearly different so it's easy to tell what kind of time it is when debugging. I used 8 hours to be far beyond reasonable clock drift and impossible to mistake AM for PM or vice versa.

Make sense, thanks.
I think that the bug should be considered corrected.


Subdomain Issues (revisited)

Could not get sessions to be cross subdomain, tried the application_controller.rb override:
  cookies[:remember_token] = {
    :value   => user.remember_token,
    :expires => 1.year.from_now.utc,
    :domain => ".#{request.host}"
  }

This would break sessions entirely, could not log in.  So I edited clearance/lib/clearance/authentication.rb directly and that seemed to work.
Any chance of including the domain option in Clearance config?

Hi @forrestgrant! In order to consider this, a patch with a failing test would go a long way

My hope is the change to the Rack session will fix this:
https://github.com/thoughtbot/clearance/pull/177
Forrest, give master a chance when you have a moment. If it's not working for you, please re-open another issue. Thanks.


Hopefully stabilize test_bad_connection.py

Problem
It seems that even though we have a retry on basebackup, it still sometimes fails to fetch it with the failpoint enabled, resulting in a test error.
Summary of changes
If we fail to get the basebackup, disable the failpoint and try again.

Addresses https://github.com/neondatabase/neon/issues/6688

It would be good to look at the connection failure probability vs.  the number of retries in compute ctl -- if we're doing e.g. three retries and 50% failure rate, then those numbers probably need adjusting (probably by retrying more times).

Currently we retry 5 times with an initial timeout of 500ms
>>> sum(500 * 2**i for i in range(5))
15500

Probability of the test failing is then 1/(2^5) = 3%. We can probably reduce the probability of test failure to 0.01% if we make it do 10 retries. Is that good enough? Or should we make it deterministically pass?

Probability of the test failing is then 1/(2^5) = 3%. We can probably reduce the probability of test failure to 0.01% if we make it do 10 retries. Is that good enough? Or should we make it deterministically pass?

That works for me.

I switched it to 10 retries, with it 1.5xing the timeout every time, making the maximum amount of time spent waiting for retries
>>> sum(500*1.5**i for i in range(10))/1000.0
56.6650390625

56 sec

Judging by
https://neon-github-public-dev.s3.amazonaws.com/reports/pr-7301/8967994121/index.html#suites/0b656b93d1c88c96d51f478c1ec4b813/9568c4d326d5d635
90 seconds might still be not enough to wait for endpoint start.
Note that for a successful test (#1 test_basebackup_error[release-pg14]) from that run, endpoint start took almost a minute:
2024-05-06 11:14:28.335 INFO [neon_fixtures.py:1453] Running command "/tmp/neon/bin/neon_local endpoint start --safekeepers 1 ep-1"
2024-05-06 11:15:25.258 INFO [neon_fixtures.py:1519] Run ['/tmp/neon/bin/neon_local', 'endpoint', 'start', '--safekeepers', '1', 'ep-1'] failed:
...
    Caused by:
        0: failed to iterate over archive
        1: db error: ERROR: failpoint basebackup-before-control-file
...

(I've stumbled upon this failure locally.)


chore: move DB Bucket schema to consumer packages

Motivation
DB Bucket entries are all defined in the db package. Some of these entries are used in the beacon node (lodestar package), some are used in the validator (validator package).
The db package should really only be used to define the abstract db logic, and doesn't need to know which buckets exist for all downstream db consumers.

See https://github.com/ChainSafe/lodestar/issues/2820

Description
Move DB Bucket schema to consumer packages
Closes https://github.com/ChainSafe/lodestar/issues/2820

:tada: This PR is included in v1.9.0 :tada:


fix(preview): add key prop to select.item component

Component / Package Name:
This PR contains:

[x] bugfix
[ ] feature
[ ] refactor
[ ] documentation
[ ] other

Are tests included?

[ ] yes (bugfixes and features will not be merged without tests)
[x] no

Breaking Changes?

[] yes (breaking changes will not be merged unless absolutely necessary)
[x] no

This pull request actually resolves issues #123
Description
Added missing key to Select.Item component in mobile.jsx file.

@lordelogos tests didn't run bc of a lint error

@lordelogos tests didn't run bc of a lint error

Fixed!


Drag 'n drop root-folders doesn't work when moving a root folders that are ancestors or descendents of another

Since I implemented the dnd root folder reordering in atom, i was curious when i saw it mentioned in the change notes for vs code. And I found a bug!
Issue Type: Bug
Open a workspace with root folders such as
{
  "folders": [
    {
      "path": "."
    },
    {
      "path": "a"
    },
    {
      "path": "b"
    }
  ]
}

Folders a and b are fairly easily moved around each other, but . cannot be moved and a and b can not be moved when dropped over ..

Thanks for the report, we accept PRs that fix this - especailly if they are not complex
Code pointer https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/parts/files/electron-browser/views/explorerViewer.ts#L979
@bpasero I believe that workspaceEditiingService should handle this case, right?

@isidorn yes the workspace editing service should work with any combination of folders.

@bpasero @isidorn I don't really understand the bug here. Is it possible to create a dir with name ., which already exists btw ? What am I missing here ?

@shobhitchittora with absolute paths, this could look something like
{
  "folders": [
    {
      "path": "/home/forivall/my-project"
    },
    {
      "path": "/home/forivall/my-project/a"
    },
    {
      "path": "/home/forivall/my-project/b"
    }
  ]
}

It's impossible to make a directory with the name ., and it just means the same folder that the example.code-workspace file is in. Hopefully that clears it up.

I'll take this up.

Now this is disabled.

I verified the issue as described in https://github.com/Microsoft/vscode/issues/47852#issue-314263301


Add option to preserve mutability when setting value to NSMutableArray properties

How to replicate problem?

Create Motis object with NSMutableArray property
Set values with mts_setValuesForKeysWithDictionary
In mts_validateAutomaticallyValue:toClass:forKey: array is parsed in first lines, in if ([*ioValue isKindOfClass:typeClass]) statement with return YES at the end.

Now arrays are not parsed at the beginning, they go to the existing if/else statement where mutability is properly checked.

@vilanovi what do you think about this fix? :)

Hello, sorry for the delay. I'll try to give it a look before this Friday.

I've been reviewing your changes and I do agree on them, mostly. Let me explain myself:

Line 1033: ok. We will return YES directly if the *ioValue class and classType matches and the class type of *ioValue is not an NSArray. In that case we will execute the validation.

Line 1157: Here I do not agree. Mainly, I don't want to "return *ioValue != nil" as I need to check the exact type of the classArray and be sure that it is something known. Otherwise we risk that Motis returns an instnace of NSArray for something that it is not. Therefore, I'd rather add another "else if" statement and include the NSArray as the last type check.

I'll merge your changes and then apply mine on top (this way you can pull).
Thanks,

Yes, you are right. There should be another else if to check if typeClass is NSArray.
Thank you!

I've also added some tests to validate it (also with sets, ordered sets, mutable sets and mutable ordered sets).
New version 1.4.3 already pushed to cocoa pods.
Thanks!


[language] [functional tests] Added a sorted linked list example

Motivation
Follow up to #1273 : A new version of the sorted linked list in a single test file
The new file implements a distributed sorted linked list, i.e. a linked list whose every node is stored as a resource in a different account. It does not touch any of the core modules and just adds a new functional test. The initial idea behind this data structure was to implement a DNS, but it can be used for many other use-cases where we need to store vast amounts of data and hence have to distribute them among accounts.
Test Plan
The PR only adds functional tests
Related PRs
#1273

Looks like some of the tests you added are failing in the CI...

Looks like some of the tests you added are failing in the CI...

I fixed this. I'll add an issue about it.

@Goharshady any reason we still don't want to land this? This is brilliant work and we'd love to have it being run against the new builds!

@bors-libra r=sblackshear

:pushpin: Commit 1d23e47 has been approved by sblackshear

:hourglass: Testing commit 1d23e4700aff3ccc7c3943e58a39307de1e0281c with merge 1587530dec0d5fda5fbb4f652ae22f27a3218ab6...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: sblackshear
Pushing 1587530dec0d5fda5fbb4f652ae22f27a3218ab6 to master...


⚠ Bofrost.at has degraded performance

In fd80706, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1619 ms

Resolved: Bofrost.at performance has improved in 2f8be31 after 6 minutes.


Add Hugging Face

the ability to import and run Discus using any LLM on Hugging Face.
do this in the model directory as huggingface.py

add HuggingFace to the SupportedModelTypes
check the model-provider in the Generator.py and call correct file
write HuggingFace.py
test HF
add to documentation


Command Discrepancy: Sequelize vs. Mongoose in db-drawer Tool

Description:
While using the db-drawer tool, I noticed a discrepancy in the commands provided for running the tool with Sequelize and Mongoose.
For Sequelize, the command is:
npx db-drawer s
However, for Mongoose, the command is:
npx db-draw m
The inconsistency in the commands could lead to confusion for users, especially those new to the tool. To improve clarity and consistency, I suggest aligning the commands for both Sequelize and Mongoose usage.
Proposed Solution:
Update the command for Mongoose usage to match the format used for Sequelize, like so:
npx db-drawer m
This adjustment will enhance user experience and ensure consistency across the tool's functionality.
You can refer below:

Hello @Bhanu-code
Sir, Please assign me this issue.

@kvdevlopment go ahead update the commands and docs as well


new URL

You can terminate futile new issues about it.

Codecov Report

Merging #218 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #218   +/-   ##
=======================================
  Coverage   98.87%   98.87%
=======================================
  Files           1        1
  Lines         892      892
  Branches      153      153
=======================================
  Hits          882      882
  Misses         10       10

Continue to review full report at Codecov.

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

Thank you, I didn't catch that.


Could not load view for: nativecanvasview

Make sure to check the demo app(s) for sample usage
Done
Make sure to check the existing issues in this repository
Done
If the demo apps cannot help and there is no issue for your problem, tell us about it
Please, ensure your title is less than 63 characters long and starts with a capital
letter.
Which platform(s) does your issue occur on?

Both
Android 9
device

Please, provide the following version numbers that your issue occurs with:

CLI: 6.7.8
Cross-platform modules: 6.5.20
Runtime(s): "tns-android": "6.5.2" | "tns-ios": "6.5.3"
Plugin(s): "@nativescript-community/ui-canvas": "^4.0.31"

Please, tell us how to recreate the issue in as much detail as possible.
Describe the steps to reproduce it.
(JS Project)
Import plugin on main:
import CanvasPlugin from '@nativescript-community/ui-canvas/vue'
Vue.use(CanvasPlugin)

Create a clean page on project:
<template lang="pug">
Page(
  actionBarHidden="true"
)
  GridLayout(
    rows="*",
    columns="*"
  )
    CanvasView(width="100" height="100" @draw="draw")
</template>

import: (need add this on doc also)
import { Color } from '@nativescript/core'
import { Paint, createRect } from '@nativescript-community/ui-canvas'

create method:
draw ({ canvas }) {
      const paint = new Paint()
      paint.setColor(new Color('black'))
      paint.strokeWidth = 10
      canvas.drawRect(createRect(0, 0, 200, 100), paint)
}

error
 {NSVue (Vue: 2.6.11 | NSVue: 2.7.1)} -> CreateElement(canvasview)
JS: TypeError: Could not load view for: nativecanvasview. TypeError: _nativescript_core__WEBPACK_IMPORTED_MODULE_0__.Property is not a constructor TypeError: _nativescript_core__WEBPACK_IMPORTED_MODULE_0__.Property is not a constructor
JS:     at<EMAIL_ADDRESS>(file: src\src\canvas.common.ts:21:30)

@luiguild it does not support N 6.x! Sorry had to make a clean break.

Ok, no problem, thanks for your quickly response!

@luiguild btw there is https://www.npmjs.com/package/nativescript-canvas which still works with 6.x but i dont really update it anymore :s


A pattern with a chest recipe does not show the chest icon when pressing SHIFT

Issue description:
A pattern with a chest recipe does not show the chest icon when pressing SHIFT
See screenshots here:
http://imgur.com/a/AToqD
Add a chest pattern to a crafter, open the crafter GUI, press SHIFT to see the pattern result icons
What happens:
A pattern with a chest recipe shows a blank icon
What you expected to happen:
A pattern with a chest recipe should show a chest icon
Steps to reproduce:
1.Add a chest pattern to a crafter, open the crafter GUI, press SHIFT to see the pattern result icons
...
Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2

Forge:v<IP_ADDRESS>4

Refined Storage:
EnderCore-1.10.2-<IP_ADDRESS>-beta
EnderIO-1.10.2-3.1.179
MCMultiPart-1.4.0-universal
refinedstorage-1.2.25

Does this issue occur on a server? [yes/no]
N/A
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]

Duplicate of #609


[Bug] Resume playback reset the active connections and download speed

Start a new torrent.
Stop the torrent and leave it running and downloading in the torrent list
After a while, resume and play it from the torrent list.
The bug: All the connection and the download speed are reset to zero and restart the connections.

I think I already open an issue on it in the past.

Images:

After resume:

Kodi Log:
https://paste.ubuntu.com/p/c2FCNFtwjv
(You can focus after ~15:30, I also enabled the debug mode)
Your Environment
Elementum 0.1.41
Kodi 17.6
Windows 10, 64bit

@elgatito
This issue is still exist in Elementum 0.1.52
Kodi debug log:
https://paste.ubuntu.com/p/mqWRkq7Hzt
Before:

After resuming from the torrent list:

@burekas7 seeds/speed was reset after you stop watching or when you re-open player?
You can see through the web page.

@elgatito
Reset when re-open player.
(When I stop the playback the speed is still stable)
What do you mean by "through web page"?

@burekas7 I mean the web part of Elementum.
http://<IP_ADDRESS>:65220/web/

After stopping the torrent:
(Remain the same - OK)

Right after resuming from the torrent list:
(Drop - Fail. Btw, if the torrent is a popular one, it back to the normal speed pretty fast, but with weak torrents it can be more problematic)

Another issue: Why upload is remain on zero all the time, even when the max upload is set in the BitTorrent settings?

Another issue: Why does the upload remain on zero all the time, even when the max upload is set in the BitTorrent settings? (As you can see in the first image)

Elementum is taking numbers from what libtorrent gives. And upload cannot be disabled, only limited, if you set rate limits in Elementum configuration.

(Drop - Fail. Btw, if the torrent is a popular one, it back to the normal speed pretty fast, but with weak torrents it can be more problematic)

Thanks for clarification. Have to see where if gets paused.

Another issue: Why does the upload remain on zero all the time, even when the max upload is set in the BitTorrent settings? (As you can see in the first image)

Elementum is taking numbers from what libtorrent gives. And upload cannot be disabled, only limited, if you set rate limits in Elementum configuration.

Yes, as I said. I set it to value of 1000kb/s, but it was still remain on zero.
I will check it later with another torrent.

@elgatito
Ignore my last comment regarding the upload, with another torrent it seems to be ok

There was a bug with pausing torrent when starting a playback.
It was there only if enable pause/resume trick for libtorrent (which is disabled by default).
In 0.1.53 it was fixed.

Thanks @elgatito
It seems to be ok now with 0.1.53


Can't maximize snapped window with FancyZones 0.58.0

Microsoft PowerToys version
0.58.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Enable FancyZones, snap a window to a zone, click "maximize" button.
✔ Expected Behavior
The window remains maximized.
❌ Actual Behavior
The window maximizes, then resizes back to the zone it was snapped to.
A workaround I found is to "unsnap" the window by moving it around first, then maximize, or simply drag the title bar to the top of the screen (which also counts as moving it around).
Other Software
No response

/bugreport

PowerToysReport_2022-05-04-10-15-54.zip

Same issue:
PowerToysReport_2022-05-04-15-36-53.zip

As I can see in the logs, the problem has the same reason as this issue, so I'll close it as a duplicate.
/dup of https://github.com/microsoft/PowerToys/issues/18057


CMS 55 and 111 "Could not find value for Population MSRPOPLEX"

CMS55 v4 and CMS111 v4 (2015 release) are again reporting "Could not find value for Population MSRPOPLEX" though the measures do not have Measure Population Exclusions.
I am seeing this error via the API. I suspect 179 has the same error but I have not tested it recently.

Matt-
Thanks.  These IDs were not removed from
CMS55, CMS111 and CMS179 for the 2015 release and CMS179 for the 2016 release.  They will be removed in the final bundle.
-Dave


Show strats on profile page

Public strats should be shown to everyone view the profile page
Private starts should be shown only if the current user is view his own profile page

Ref figma for design: https://www.figma.com/file/x6REYepp8WsBhTyJMnFq3D/csgo-nades?type=design&node-id=0-1&mode=design&t=J29cHbbH74jwyhOl-0


Apply new befta version to master

JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

wil be merged through normal process to develop not master


enable clippy lints against integer casts and arithmetic

Cc https://github.com/rust-lang/miri/issues/1236

@oli-obk I hope the remaining changes are uncontroversial. :)

They are
@bors r+

:pushpin: Commit 7f6034862d0a22c7337eb428c67fcdb23b9168ae has been approved by oli-obk
It is now in the queue for this repository.

:hourglass: Testing commit 7f6034862d0a22c7337eb428c67fcdb23b9168ae with merge 83d50ef05535af98fe11a1d2920fb652e19f327b...

@bors retry
I fixed the PR title

:hourglass: Testing commit 7f6034862d0a22c7337eb428c67fcdb23b9168ae with merge 6227e1e3a8e1a47d6ca4dfac0b52a26084bac697...

:sunny: Test successful - checks-actions
Approved by: oli-obk
Pushing 6227e1e3a8e1a47d6ca4dfac0b52a26084bac697 to master...


refactor(tracer): allow streaming threshold configuration

Description of your changes

Add setStreamingThreshold method to ProviderServiceInterface.ts
Implement setStreamingThreshold method
Add unit-tests

Initially the setStreamingThreshold method was not exposed.
Having encountered a Segment too large issue (with a lot of requests to DynamoDB), the only way to bypass the issue was to set the Streaming Threshold to a value of 0.
This is possible when using aws-xray-sdk-node directly but not aws-lambda-powertools-typescript because the setStreamingThreshold method was not exposed.

How to verify this change

Related issues, RFCs

Issue number:
PR status
Is this ready for review?: YES
Is it a breaking change?: NO
Checklist

[x] My changes meet the tenets criteria
[x] I have performed a self-review of my own code
[x] I have commented my code where necessary, particularly in areas that should be flagged with a TODO, or hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have made corresponding changes to the examples
[x] My changes generate no new warnings
[x] The code coverage hasn't decreased
[x] I have added tests that prove my change is effective and works
[x] New and existing unit tests pass locally and in Github Actions
[x] Any dependent changes have been merged and published
[x] The PR title follows the conventional commit semantics

Breaking change checklist

[ ] I have documented the migration process
[ ] I have added, implemented necessary warnings (if it can live side by side)

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

Hi Nathan, I appreciate you taking the time to contribute to this project and open a PR, however I'm not inclined to consider merging it.
We decided not to expose this method from the SDK in the provider interface because the SDK itself already sets the streaming threshold to 0 whenever running on Lambda. This is done to limit any potential segment loss which could happen due to the ephemeral nature of Lambda. With a streaming threshold always set to 0 the SDK immediately sends out a segment as soon as it's closed.
With this in mind, and considering that we exclusively target Lambda, I don't think its necessary to expose the method at this time.
For future reference, also please note that we like to try following the process described both in the docs (here) and in the contributing guidelines.
According to the process the journey for any change/addition/removal to the project, starts with an issue, continues with a discussion, and then ends in a PR only after having been agreed upon with a maintainer.
In this way, if a change doesn't fit with the project, the contributor can save time or decide to contribute on another issue/topic.

Hi @dreamorosi, thank you for your feedback.
Indeed I just saw the link to the SDK that sets the value directly to 0.
It's quite strange because when we use the tracer wrapper (from the lambda powertools) and captureAWSv3Client, when there are huge amounts of calls to DynamoDB when I use we get a Segment too large error.
And when we use the SDK itself, it works as expected.
I will open an issue in this case.

Yes please, happy to look at the issue regarding the diverging behavior between the SDK and Powertools.
When you open the issue, please include a snippet of code to reproduce the issue if possible. Thanks

In light of the discussion happening in #1310 I'm going to close this PR for the time being.
If after discussing we decide that the feature is indeed needed we'll reopen it.


Add build artifacts

can you see if the build artefact can be done on release, and attach it as a release artefact?

Actually, a build artifact cannot be attached as a release artifact. One solution could be to cache build and use it on the release workflow (But I'm not sure if it will work stable) or build it in the release workflow itself and upload it as a release asset. Second is that we can update the readme and include the URL of the artifact there. Unfortunately, I could not find a way to get the latest artifact. The third is to keep it as a built artifact, as it is now. I personally think the third solution is the best.

superceded by #1279


handle VM expunge upon host disconnection in GC

for https://github.com/zstackio/issues/issues/1827
Signed-off-by: David Lee<EMAIL_ADDRESS>

Close this PR since refactor/tests needed.


Add NUT-XX: HTTP Compression

Draft for supporting HTTP compression on the mint's REST API endpoints.
For easier viewing: https://github.com/ok300/nuts/blob/ok300-http-compression/xx.md

Do we need a NUT for this? Isn't the compression announced via HTTP headers already?
Tangential: I think we can improve the performance of /v1/keys by generating pubkeys deterministically and just giving away an XPUB. Is that worth creating a NUT? Are keys in keysets already generated deterministically? @callebtc

Do we need a NUT for this? Isn't the compression announced via HTTP headers already?

@prusnak having it as a NUT brings it to the attention of both mint operators and wallet maintainers.
IMO that's useful because most HTTP libraries don't have it turned on by default, so for a payload to be compressed this has to be explicitly enabled on both client and server. Right now less than half the mints (9 of 25) on bitcoinmints.com support it, and none of the wallets I looked at added the headers in their requests.
If there's a better place for this (an existing NUT?) I'll gladly move it there.

I think we should:

just recommend using HTTP compression (not sure in which NUT, though)
open (and fix) issues to all mints and clients which do not announce HTTP compression

2. open (and fix) issues to all mints and clients which do not announce HTTP compression

Thinking about this more ... (wallet) clients should announce they do support HTTP compression, but with mints (servers) it is more complicated - the mint does not have to support compression, but if the deployment does (nginx, cloudflare, etc.) then we are fine ...

the mint does not have to support compression, but if the deployment does (nginx, cloudflare, etc.) then we are fine

Good point. I updated the Mints section to better reflect this, as this is for the mint operator to configure.

I appreciate the performance information in the table, but overall I think this information should be in a Mint Operator Best Practices informational NUT.

@clarkmoody IMO it's a best practice for both wallets and mints, because it will only work if both sides actively enable it.
What I'm also hoping to achieve with this NUT is to help wallets and mints choose the newer compression algorithms. When compression is enabled, most platforms support gzip and deflate, which reduce the size by about 40%. But the newer brotli and zstd algorithms can give another 5-20% on top. The choice of algos also has to be explicitly set by both sides, so this is another reason both sides should consider this.
Overall this proposed NUT is similar to NUT-16. This speeds up data transfers between wallets and mints (HTTP), while NUT-16 speeds up data transfers between wallets (animated QRs).

I appreciate the performance information in the table, but overall I think this information should be in a Mint Operator Best Practices informational NUT.

Is there already such a NUT or are you proposing that we create one?
In either case, I think this is the way forward:

create Mint Operator Best Practices NUT where we describe how HTTP compression should be enabled
patch all client software (wallets) to announce and use HTTP compression

Our specs exist to ensure compatibility between wallets and mints. As long as we don't make HTTP compression mandatory (which we should not), there is no need to specify anything. The choice whether to use HTTP compression or not is entirely up to the implementor and that choice does not have to be communicated in the Cashu protocol, because it's part of HTTP.
HTTP compression is everyone's own choice and therefore I don't see why this should be part of our specifications

nACK


[Bug] DOC adding image error and I can not get pictuer data

Before you submit this issue, have you checked the following

[X] Is this really a problem?
[X] I have searched the Github Issues for similar issues, but did not find anything.

Affected packages and versions
0.2.14
Reproduction link
Adding images smoothly without errors and able to capture data
Expected behavior
Adding images smoothly without errors and able to capture data
Actual behavior
Adding images smoothly without errors and able to capture data
System information
No response

Please provide the minimal reproduction.

We are closing this issue for now due to the lack of enough information. Please feel free to reopen it once you can provide a minimal reproduction.


https://docs.github.com/en/articles/setting-guidelines-for-repository-contributors

Ok

It's done


Aot incompatible

ng2d3 seems to be incompatible with AoT compilation

Currently there is a number of small pieces still using d3 in relation to the DOM. Those would need to be removed and semi-high effort to accomplish that given a few constraints such as #8. Its on the list to do as soon as possible.

Resolved :)


[W7.7][T13-B1] Project LISA: Levels Integrated Secretarial Agent

@Juxarius @OscarWang114 @arnollim @RSJunior37 @Pujitha97
some comments added. please read and ack; please retain the PR without closing.

@Juxarius @OscarWang114 @arnollim @Pujitha97 @RSJunior37
please take a look at the new comments added

Coding standard violation here:

Try to use UML diagrams to explain your design and implementation.
Use code snippets minimally as they are hard to maintain. Any change in the actual code should be reflected in the documentation as well.

@Juxarius @OscarWang114 @arnollim @Pujitha97 @RSJunior37 for your action


[C++][Parquet] Add benchmark for boolean Arrow column I/O

Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=uwe) / @xhochy
Note: This issue was originally created as PARQUET-1094. Please see the migration documentation for further details.

Uwe Korn / @xhochy:
PR: https://github.com/apache/parquet-cpp/pull/391

Wes McKinney / @wesm:
Issue resolved by pull request 391
https://github.com/apache/parquet-cpp/pull/391


scale_color_category cannot be used with integer data

library(rdeck)library(sf)
#> Linking to GEOS 3.10.1, GDAL 3.4.0, PROJ 8.2.0; sf_use_s2() is TRUE
reprex_data <-structure(list(state_hazmat_mobilised = 0L, geometry = structure(list(     structure(c(153.408180931689, -27.9478084866912), class = c("XY",
    "POINT", "sfg"))), class = c("sfc_POINT", "sfc"), precision = 0, bbox = structure(c(xmin =<PHONE_NUMBER>31689, ymin = -27.9478084866912, xmax =<PHONE_NUMBER>31689, ymax = -27.9478084866912   ), class = "bbox"), crs = structure(list(input = "EPSG:4326",     wkt = "GEOGCRS[\"WGS 84\",\n    ENSEMBLE[\"World Geodetic System 1984 ensemble\",\n        MEMBER[\"World Geodetic System 1984 (Transit)\"],\n        MEMBER[\"World Geodetic System 1984 (G730)\"],\n        MEMBER[\"World Geodetic System 1984 (G873)\"],\n        MEMBER[\"World Geodetic System 1984 (G1150)\"],\n        MEMBER[\"World Geodetic System 1984 (G1674)\"],\n        MEMBER[\"World Geodetic System 1984 (G1762)\"],\n        MEMBER[\"World Geodetic System 1984 (G2139)\"],\n        ELLIPSOID[\"WGS 84\",6378137,298.257223563,\n            LENGTHUNIT[\"metre\",1]],\n        ENSEMBLEACCURACY[2.0]],\n    PRIMEM[\"Greenwich\",0,\n        ANGLEUNIT[\"degree\",0.0174532925199433]],\n    CS[ellipsoidal,2],\n        AXIS[\"geodetic latitude (Lat)\",north,\n            ORDER[1],\n            ANGLEUNIT[\"degree\",0.0174532925199433]],\n        AXIS[\"geodetic longitude (Lon)\",east,\n            ORDER[2],\n            ANGLEUNIT[\"degree\",0.0174532925199433]],\n    USAGE[\n        SCOPE[\"Horizontal component of 3D system.\"],\n        AREA[\"World.\"],\n        BBOX[-90,-180,90,180]],\n    ID[\"EPSG\",4326]]"), class = "crs"), n_empty = 0L)), row.names = c(NA,
-1L), sf_column = "geometry", agr = structure(c(state_hazmat_mobilised = NA_integer_), .Label = c("constant",
"aggregate", "identity"), class = "factor"), class = c("sf",
"tbl_df", "tbl", "data.frame"))

rdeck() |>
  add_scatterplot_layer(
    data = reprex_data,
    get_fill_color = scale_color_category(
      state_hazmat_mobilised
    )
  )

#> Error in `add_scatterplot_layer()`:
#> ! Failed to create layer
#> Caused by error:#> ! Assertion failed: is_discrete(x)
#> • Continuous value supplied to discrete scale

Created on 2022-04-13 by the reprex package (v2.0.1)
Slightly surprising since integer is 'discrete'.

This is for consistency with {scales} (https://github.com/r-lib/scales/blob/main/R/scale-discrete.r#L28) and {ggplot2} (scale_colour_discrete()). rdeck re-implements the DiscreteRange to preserve input order, but otherwise is equivalent to scales::DiscreteRange().
While I think a factor / character would generally be a better choice for a category scale, maybe it's needlessly restrictive. One case where ints as they are would be ok, is integer codes from abs admin boundaries.


Can we use screen shaders for upscaling Full-screen Apps

Description
Can we use upscale filters in Apps (especially Fullscreen apps).
There's an all purpose window upscaler Magpie for Windows 10/11.
I use it extensively for old games and visual novels.
NOTE: I'll be creating my own upscale filters for the same (wrapper around Anime4k possibly), screen-shaders look simple enough, i'm asking for viability of this as i'm noob to hyprland code-base and window managers, scaling inside them in general.

you can yes


it says: failed to save record: error code 429

I had this issue. My problem ended up being not having billing correctly set up for API use despite having a premium account for ChatGPT use. Would recommend checking here: https://platform.openai.com/account/billing/overview

My billing is setup correctly and still getting the 429 error code. Any suggestions?

I had this issue. My problem ended up being not having billing correctly set up for API use despite having a premium account for ChatGPT use. Would recommend checking here: https://platform.openai.com/account/billing/overview

Should we setup billing to use this plugin ?

Currently getting the same error. Have billing set up and correct API key
Version 1.4.16 (Installer 1.4.16)

I'm having the exact same issue.

i had the same issue but now it works for me

Yes it's working again for me as well!

In my case the problem was that my account in ChatGPT  is set as Pay as you Go and you have to fund the account. Previously they didn't require that, they just invoiced you at the end of the month, but now you have to add funds for it to work. Once my balance had some cash in it, everything worked and the error 429 disappeared.


Xcode debug cannot continue when All Exception is enabled

when All Exception is enabled in Xcode debugger. our app will stop at:
AndroidUtilLog_wWithNSString_withNSString_
and cannot continue

Set java.util.logging.Level's default localization bundle to null, so exception isn't thrown during its class initialization.
Please note: the JRE, like most Java library suites, uses exceptions liberally, while Objective C doesn't. There is no plan to ever provide an exception-free JRE, as it would be impossible for it to have the same documented behavior. We've made limited changes to the java.util.logging framework since it's used so much when debugging, but generally native debugging should only be done without breaking on all exceptions.


chore(cvss): improve CVSS vector parsing

Description
We have seen a few instances where we get logs about invalid CVSS vectors but with very limited information. This change improve our CVSS processing by:

Use a library which already implements this and account for non-base metrics, which is something we missed in the past
Log the entire invalid CVSS vector instead of just the single invalid metric, so we have an idea of what the vector looks like

Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Here I tell how I validated my change
CI is sufficient, as we already have unit tests
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

The idea is great, but unit tests are unhappy

Good, they should be. They were bad tests it seems. Will fix

@vikin91 unit tests seem happy again and the other test failures seem unrelated


shared: Use custom error type for RunCommand

Closes #3502
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

Cool that will help me a lot. Also means we've got some community work: Replacing all of the exec.Command().*() calls in the lvm driver for example.

Yeah, there are a few easy spots that would benefit from porting. With this the only remaining direct use of exec.Command should be for when we need to redirect stdin/stdout/stderr (migration and such).


2.1.2 For #296.

Coverage remained the same at 92.409% when pulling 8ebedb6ee2ee9d9a5e6888d0fa00bc8328f91df7 on kaixinqiqi:2.1.2-SNAPSHOT into ea13c940ce1cc9c8e7352a19ddaae73f58b71241 on dangdangdotcom:2.1.2-SNAPSHOT.

Coverage remained the same at 92.409% when pulling 8ebedb6ee2ee9d9a5e6888d0fa00bc8328f91df7 on kaixinqiqi:2.1.2-SNAPSHOT into ea13c940ce1cc9c8e7352a19ddaae73f58b71241 on dangdangdotcom:2.1.2-SNAPSHOT.


Missing file? Main.cpp import  not found

Building from the latest source, the following error is encountered:
Client/src/Main.cpp:12:10: fatal error: filesystem: No such file or directory
 #include <filesystem>
          ^~~~~~~~~~~~
compilation terminated.
make: *** [Main.o] Error 1

I can't see this file in the repository.

filesystem is a C++17 header. I'll specify the C++ standard in the .pro file later.
Thanks!


Question about searching users?

Hi, I found out that I cannot search users even though they are actually registered.
This makes me struggle to add users in private DataDoc.
I used default method of registration.
Did anyone encountered this problem previously?
I used k8s file provided to start querybook, and also I found out that user existed in both ES and MySQL backend.

Hi Yongchan, I tried it locally and it works. There is a bit of delay (~1 min) between user creation and them showing up on the search index. Can you check if elasticsearch is working correctly on k8s and all search indices are created?

 /opt/querybook/querybook/server/datasources/search.py - INFO "RequestError(400, 'search_phase_execution_exception', 'no mapping found for field [suggest]')"
I got following error - please let me know if you encountered such before.

@czgu Just found out that it works well with normal MySQL backend -- I should have waited longer. The problem is that same issue is happening when I change MySQL backend to RDS. Will it not work for RDS backend?

Just found out that it works well with normal MySQL backend -- I should have waited longer. The problem is that same issue is happening when I change MySQL backend to RDS. Will it not work for RDS backend?
Fix: It worked after restarting ES and Web pod. I am wondering if this might be some issue with initialization order and connection.

oh interesting, I have never tried it on k8s with rds. It would be useful if you can provide stack traces or error logs

Not sure about this still -- but based on what I have been testing I found out that it maybe caused due to configuration time? I may have logged in too fast to web before configuration completed for both web and elasticsearch. Will figure out and comment later.


Accuracy at 56%

I've gone through the complete training using default values for person_match.train2 and I only get 56% accuracy from the eval script on the autogenerated validation.txt0. Any ideas on what might be the problem? Can others get to the 91% accuracy?

problem was only with accuracy printing code. please try with fresh checkout for latest updates compatible with tensorflow v1.2.1


update plugin versions

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Not required at the moment.


taxgraphs.io domain expired?

Not a code issue, but just FYI, the taxgraphs.io domain appears to have had its registration lapse.

Thanks for the note - I just renewed the domain and updated with 2020 tax data and new graphs


Using USB2 with dai ros2 driver not possible

Hello,
I'm trying to use the depthai ros driver with USB2 Hardware. When launching the camera I get the following error:
Failed to find device after booting, error message: X_LINK_DEVICE_NOT_FOUND
When using the oak-d as webcam (https://docs.luxonis.com/en/latest/pages/oak_webcam/) It only works when forcing usb2.
Is there any possibility to force USB2 within the driver similar to when using oak-d as webcam? I already tried setting the i_usb_speed to HIGH instead of SUPER_SPEED but it didn't work.
Thanks in advance!

Hi, are you able to run demo scirpt? And which distro are you using?

Hello,
I am also having the same error "X_LINK_DEVICE_NOT_FOUND" with OAK -D -Pro usb camera. I am running it on jetson agx orin inside docker container built from image luxonis/depthai-ros:humble-latest

Closing due to inactivity, related issue: https://github.com/luxonis/depthai-ros/issues/363


Debug: validate user input for breakpoint hit-count

Testing #13211
Since you already seem to have a regex to check for validity of the breakpoint hit-count, why not tell the user?

This is coming from the debug adapter and currently there is no way for the UI to know this. Thus assigning to @weinand to consider adding this in the future

Ok got it, sorry for the spam in that area. I was not aware this is debug adapter specific.

@bpasero no worries :)


Provide an opt-out mechanism for ignoring key-control on elements

see comments in #43

I'm thinking we have several types of keyboard control:

command-palette: cmd-k (currently always available)
quick-start: cmd-enter, cmd-shift-enter, and cmd-alt-enter (currently always available, and turn on navigation controls)
navigation: arrow keys, page-up/down, blank slides (currently require key-control attr)

If we're providing an opt-out it should probably support a few values (names TBD):

none: no keystrokes should have any impact on slide-deck
blur-only: only listen for the esc key to blur focus on the element
no-nav: command-palette and quick-start keys are allowed, but not keyboard navigation

Something like this:
<my-widget slide-keys="none">no key actions trigger slide controls</my-widget>
<my-widget slide-keys="blur-only">only listen for the blur action (esc)</my-widget>
<my-widget slide-keys="no-nav">don't listen for keyboard navigation</my-widget>
<!-- any other value of slide-keys will allow all? -->

Along with this change, I would probably change the key-control attribute to be called key-nav or something?

To expand on this a bit:

Currently we do allow the cmd-prefixed keys even on otherwise 'private' elements. I don't think it's clear in my previous proposal when those would be available.
Blank slides have a behavior that blurs on any keystroke, so we could have e.g. only-blur and any-blur?
We could consider re-using the key-control attribute at all levels of nesting, and then always check the nearest ancestor with a key-control value?


Hi, are you still using the npm package name?

Hi, are you still using the npm package name?
I have a project I'd love to use the name for, any chance you'd consider reassigning ownership? 🙏

Sure I don't mind giving you the package name. Let me deprecate it first. Just in case there is a small chance someone is still using it they might get some heads up first.

Thank you! I really appreciate it. My email is<EMAIL_ADDRESS>🙏🙏

I added you as a maintainer to atlaspack. Let me know when you've got that setup and I'll remove myself.

Thank you!