Failing to register to FRPS hangs on workspace create

Describe the bug
When the server fails to connect to FRPS, the user is unable to create a workspace and receives no error. The request just hangs.
To Reproduce
Reproduction of the issue will be difficult since the FRPS connection issue occurred because the ISP blocked the handshake.
We should find a way to replicate the connection failure.
After the connection failed, attempt to create a workspace and observe that the request hangs.
Expected behavior
There are 2 things to handle here:

If the server can not connect to FRPS while starting, it should error and let the user know.
The connection should not hang requests towards the server but return an appropriate error.

Screenshots

Desktop (please complete the following information):

OS: MacOS 14
Daytona Version: v0.12.1

Not having Internet at all should also reproduce the issue

Found the source of the issue.
Before the request to create a workspace is even sent to the server, the CLI attempts to establish a tailscale connection that is then used to check if the agent successfully started in the project. (https://github.com/daytonaio/daytona/blob/main/pkg/cmd/workspace/create.go#L97)
Half of the solution to this issue would be adding a timeout for establishing a connection (3 seconds should be enough I think). This would then notify the user that something is wrong with the FRPS connection and that the workspace can not be created.
The other half would be to implement a health check for the FRPS connection in the server and we should check the health when starting the Daytona server. If the health check fails, we should throw an error and the start of the server should fail.
@vedranjukic thoughts?

@Tpuljak i would suggest that if it establishes a connection under 3 seconds, then proceed as it does now. After 3 seconds would show a log to the user something along the lines of "Connection to tailscale is taking longer than usual" With this the user is prepared and informed what the issues is. 5 seconds after that we can display an error along the lines of "Connection to tailscale could not be established and WS creation aborted. Please check your Internet connection or the status of the tailscale"
Part 2. yes there should be a public health check that can be linked to the error message above but also agree that if health is degraded that it shows that error right away and does not proceed

@Tpuljak i would suggest that if it establishes a connection under 3 seconds, then proceed as it does now. After 3 seconds would show a log to the user something along the lines of "Connection to tailscale is taking longer than usual" With this the user is prepared and informed what the issues is. 5 seconds after that we can display an error along the lines of "Connection to tailscale could not be established and WS creation aborted. Please check your Internet connection or the status of the tailscale"

#443 added a 30sec timeout. The tailscale connection takes more than 3sec half of the time anyways. We can add a message to the user after 5secs so they're more informed. I'll open another PR for that.

Part 2. yes there should be a public health check that can be linked to the error message above but also agree that if health is degraded that it shows that error right away and does not proceed

That's part of #445.


how to add animation when go to next month?

I can't find api  about animation.

Hey @looping84, unfortunately one of the downsides to using template rendering is that it's difficult to incorporate animation. This is because the entire DOM tree for the calendar is replaced each time it is rendered.
That said, it is possible to do if you create your own controls and use clndr's API to call next, previous etc. after you've completed your own animations (e.g. fading to white before re-rendering).
Closing this for now, but I am happy to answer specific questions if you have them!


ASUS ZenFone Zoom Video Recording Issues

Crashes with various errors, irregularly after taking first video in demo app. Occurs with both engines.

Closing as not reproducible, as it went through the demo/ fine just now.


Fix DNS record alignment issue

Description
Closes #673
Demo
https://user-images.githubusercontent.com/50856799/234981029-65283f23-3b93-4864-b151-4592a37e6de7.mov

Thank you for investigating this.
As for the tablet size, I think we need to come up with a new layout entirely. I am planning to work on it as part of #203 sometime next week


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Don't make changes to guns

I'm currently refactoring so please don't make changes to it until it's done as it will just make the PR much more painful.

Is it fine to make changes to explosives?

Explosive and flash grenade sure.

Compilation of issues/suggestions related to guns for your PR.
#3732 #3527 #913 #2595 #2164 #1839

I think this is a minor bug you might wanna check out, but if you're refactoring guns for maximum pick n mix of functionality it should be fine. #4174

@metalgearsloth is this still relevant?

Yeah, depending what the changes are I may be able to work around it. Its mainly avoid touching the code side as it's all liable to be bulldozed.

In order for #4262 to work, it makes minor changes to HitscanComponent, ProjectileComponent and HitScansystem (making damage type datafields get damage prototypes, and changing a damageableComponent change damage call). its only a couple of lines, but might still be annoying if it were to be merged before the gun refactor is.

Just make the changes then I won't hold up body system for it.

This is a related issue that might be worth fixing in the refactor
https://github.com/space-wizards/space-station-14/issues/2283


Storage - file.js. Any way to set Source Encryption Key?

From @conundrum888 on May 21, 2017 14:26
Hi,
In File.js of the Storage package, the function setEncryptionKey() helps set the "x-goog-encryption-key" header. Is there a function to set the "x-goog-copy-source-encryption-key header"? Can a function be added for this please?
Copied from original issue: GoogleCloudPlatform/google-cloud-node#2322

From @frankyn on July 20, 2017 18:28
Hi, I'm just pinging the issue as I was trying to use the File.copy method to rotate customer-supplied encryption key using node.
I tried the following, but I didn't have any luck.
var file = bucket.file("file_name");
var options = {baseHeader: "x-goog-copy-source-encryption"};
file.setEncryptionKey(encryptionKey, options); // file was already set with encryption key
file.copy("file_name", function(err, copiedFile, apiResponse) { // used the same filename
   copiedFile.setEncryptionKey(new_encryptionKey);
   copiedFile.save(); // rotate encryption key
});

Hi @subra-vr,
Thanks for reporting this.
I just submitted #2324 which should give you what you need. Let me know if this solves your use case?

From @conundrum888 on May 21, 2017 21:16
That should work, thanks.

From @stephenplusplus on July 20, 2017 21:47
Sounds good to me, and thanks for the reference.

From @frankyn on July 20, 2017 21:46
+1 I agree can we move forward with this feature request? Ruby has this helper method in its client library. (Documentation).

From @stephenplusplus on July 20, 2017 21:43
Maybe we should have a method:
file.rotateEncryptionKey(newKey, function(err) {});

This would handle doing the rewrite, and managing the headers. What do you think?

From @frankyn on August 3, 2017 16:22
@stephenplusplus is there a status on this request?

From @stephenplusplus on August 3, 2017 16:45
@lukesneeringer has GCS on a code freeze while we are doing some code/repo re-organization. He can address the timeline aspect of this request.

PR sent: #81


Site accessibility issue: requires color, incompatible with forced colors

Currently, Can I Use is very difficult for me to use with forced colors.
The issue
“Forced colors” is a standard browser feature implemented by Windows High Contrast Mode and by Firefox on all platforms
When I override colors to my preferred palette, it’s impossible for me to distinguish between “supported”, “not supported”, and “partially supported” states without selecting each entry and reading the infobox. Without enabling JavaScript, I have to read the tooltip with the mouse (!!).
Reproduction
In Firefox, on any supported platform: go to “colors” in about:preferences, select “Manage Colors”, and set overrides to “always”.
Alternatively: on Windows, enable High Contrast Mode and use Firefox or any Chromium-based browser.
Severity
This is a violation of the WCAG level A success criterion 1.4.1, Use of Color.
With JavaScript enabled, the site is difficult but possible to use; I have to look at cell details one-at-a-time since the infoboxes cover other cells. Without JavaScript, the site is impossible for me to use without squinting at tooltips exposed by the mouse, or switching to a screen reader.
Proposed solution
I can think of two solutions:

Add a visual indicator to each cell, such as a check-mark or “X”. The cells already have accessible text alternatives; these textual alternatives can be assigned to these symbols.

Alternatively, follow the design of the browser compatibility tables on MDN by putting visible text in each cell to describe supported/not-supported status. If this text is good enough, it eliminates the need for visually-hidden text specifically for the accessibility tree.

Screenshot
This is a screenshot of the table from the page “CSS at-rule: @media: forced-colors media feature” as it appears on Firefox with forced colors enabled:

Ironically I ran into this issue myself while using Can I Use to check on forced-colors support, here's a small example of rendering the cross mark emoji and the check mark button emoji for supported/not supported. Unknown could be left empty and this could be applied to when forced-colors media query is active.

Properly fixed now. :)


Add class name to children util

Proposed Changes
Implemented a small util that adds className to children (or any react node).
// node
Hello react-vapor!;

const newNode = addClassNameToChildren(node, 'new-class');

// newNode
Hello react-vapor!

Potential Breaking Changes
None
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1231   +/-   ##
=======================================
  Coverage   99.66%   99.66%
=======================================
  Files         354      354
  Lines        8917     8917
  Branches     2113     2113
=======================================
  Hits         8887     8887
  Misses         30       30

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 0460f13...231b513. Read the comment docs.

I was thinking about this util and I think it would better if we wrapped the child with a span that has the className when the child is a string.


Reset index in loop to restart member search

Description
Found from audit, we need to reset the start of the loop when looking for members
Test Plan
Added unit test

If your changes are not user-facing and do not break anything, you can skip the following section. Otherwise, please briefly describe what has changed under the Release Notes section.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

thanks! plz fix tests

will do. I thought this was all green, I am a bit at a loss now, but it seems it is a snapshot to fix so will do...


(PUP-9448) Rephrase ILLEGAL_ASSIGNMENT to not use a/an. Add period.

CLA signed by all contributors.

include in single PR with multiple commits


refactor:return error type for FindCluster in cluster placement strategy interface

Description
Changes the following methods to return an error type instead of the KAS Fleet Manager ServiceError. This is based on what was agreed upon in the Control Plane team's technical discussion. Functions should not return ServiceError unless there is a strong need for it which wasn't the case for these functions.
Cluster Service

FindAllClusters: will now return database error as is. This is only called in the cluster placement strategy methods which now returns an error type.
FindCluster: will now return database error as is. This is called in the following places:

Cluster placement strategy: This now returns an error type.
Cluster manager: in the reconcileDeprovisioningCluster, this method already returns an error type.

FindKafkaInstanceCount: will return database error or GetKafkaInstanceSize error as is. This is called in the following places:

Cluster placement strategy: This now returns an error type.
Cluster manager: in the reconcileClusterWithManualConfig and setKafkaPerClusterCountMetrics, both of these methods already returns an error type.

FindStreamingUnitCountByClusterAndInstanceType: Returns database and GetKafkaInstanceSize errors as is. This method already returns an error type. This is used for the cluster placement strategy and in the Kafka manager (setClusterStatusCapacityMetrics) which already returns an error type.

Cluster Placement Strategy

FindCluster: interface was changed to return an error instead of a ServiceError. There wasn't any strong need for it to return a ServiceError as the errors it returns will mostly be caused by database errors or configuration error.

Follows up from https://github.com/bf2fc6cc711aee1a0c2a/kas-fleet-manager/pull/1167#discussion_r925801278
Verification Steps

[ ] CI Checks passing

Checklist (Definition of Done)

[x] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] All acceptance criteria specified in JIRA have been completed
[ ] Documentation added for the feature
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

please have a look when you get a chance @ziccardi @machi1990 @miguelsorianod @VanillaSpoon


feat: influx CLI Packager support for explicit schema buckets

This PR teaches the influx CLI how to export and apply buckets with explicit schema from Cloud 2. The existing CLI subcommands to export and apply templates require a significant amount of logic be duplicated from the server to validate and parse the template prior to upload. Preferably, the template would be passed to the server for parsing and validation and an appropriate response sent to the client to display to the user. Changing this would require significant refactoring of the HTTP and CLI APIs, which is beyond the scope of this feature.
NOTES
The Packager CLI does not show diffs of hierarchical data, so it is unable to show detailed differences when new measurements are added or new columns to an existing measurement. An existing example of this is Dashboards, which are limited to showing only the number of charts (Num Charts column):
https://github.com/influxdata/influxdb/blob/00afd95cb7cca048dcf7c15fbf1430d5a0724f93/cmd/influx/template.go#L1358-L1362
Following the precedent set by Dashboards, the following UI will be shown when applying changes to a bucket that include schema updates:

[ ] CHANGELOG.md updated with a link to the PR (not the Issue)
[x] Well-formatted commit messages
[x] Rebased/mergeable
[x] Tests pass

Rebased due to conflicts

Rebased due to conflicts


PublishTestResults Task fails on Linux vso agent

Publish test results task does not compile typescript to javascript file (missing PublishTestResults.js file on the agent tasks folder) and does not import / copies necessary node_modules for runtime.
I Created a separate task containing the Javascript and the node_modules folder and the publish works fine.
i would like to understand how the agent compiles the typescript file and imports the node modules in runtime Or whether my solution is a valid one (i feel it is not best practice)

@grawcho We internally compile the entire vso-agent-tasks and upload the npm package with dependent modules and compiled js files. So your vsts-agent doesn't compile on the fly.
If you want to patch tasks, you should compile locally and patch the compiled task to your TFS

thanks.
so the issue in fact is that PublishTestResults.js is missing from the uploaded content of the agent tasks on my server (on-prem TFS2015.2)?
if this is the case than i guess my solution will suffice and you should make sure that this file is added (or already added) in future updates.

We don't check in the compiled files as source code. If you look into our Read Me on how to patch tasks, it clearly explains this. Closing this issue now


Promise with success type of Self

Quite new to Swift so this may well be an issue with my knowledge rather than Bright Futures itself but should it be possible to create a Promise that has a success type of Self in a protocol extension?
extension Promiser {
    func promise() -> Future<Self, Error> {
    let promise = Promise<Self, Error>()

    return promise.future
  }
}

Gives me the exceptionally helpful xcode error Consecutive statements on a line must be separated by ';'

As you suggest, this is something that is not possible in Swift and is not a problem related to BrightFutures. I'd look into adding an associatedtype to Promiser and use that instead of Self.


”Boțești”, ”Gănești”, ”Gugești”, ”Tălpigeni” are listed in the drop-down menu

Description:
When user writes ”Boțești”, ”Gănești”, ”Gugești”, ”Tălpigeni” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Boțești"

Expected results
The location appears in the drop-down menu as "Sat Boțești, VASLUI (Boțești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Gănești"

Expected results
The location appears in the drop-down menu as "Sat Gănești, VASLUI (Boțești)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Tălpigeni"

Expected results
The location appears in the drop-down menu as "Sat Tălpigeni, VASLUI (Boțești)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Tălpigeni"

Expected results
The location appears in the drop-down menu as "Sat Tălpigeni, VASLUI (Boțești)".


Fix package api listing

Codecov Report

Merging #299 (6306a9c) into integration (147639b) will not change coverage.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##           integration     #299   +/-   ##
============================================
  Coverage        61.73%   61.73%
============================================
  Files               98       98
  Lines            26687    26687
============================================
  Hits             16475    16475
  Misses           10212    10212

Impacted Files
Coverage Δ

resqpy/organize/__init__.py
100.00% <ø> (ø)

resqpy/surface/__init__.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Set up documentation

We want this repository to use Sphinx for its own documentation, as it will help for demonstrating the capabilities of the Sphinx plugin we will later develop.
The documentation should be structured as follows:

Installation
Usage
Development

Reference
Design
Changelog

Before installation: "What is this?"

Before installation: "What is this?"


black screen

when i disable i get the screen
i dont have an adblcoker other than the thing from "enhancer for youtube"
when i enable it is a black screen

Common Issues
"Ad Blockers violate Youtube Terms Of Service" OR Just a black screen
This happens when there is an adblock other than the script running. Common ways you may be blocking ads accidentally on Youtube is:

Adblock extension: Disable or whitelist YouTube on your ad blocker extension.

Browser built-in adblock: Some browsers have built-in ad blockers. Disable or configure them.

Other Tampermonkey scripts: Scripts like YouTube enhancer may conflict. Disable them one by one to identify the issue.

Wi-Fi/DNS/Firewall adblock rules: Check network settings for any ad-blocking configurations.

Windows hosts file rules: Inspect the Windows hosts file (C:\Windows\System32\drivers\etc) for ad-blocking rules.


"Add-in Manager" is now "Extension Manager"

The installing CocosSharp section says to use the "Add-in Manager", but this doesn't seem to exist any more. The Extension Manager doesn't find the CocosSharp ad-in

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

ID: 52b14e35-4b8c-95e3-3552-1e81d3933b09
Version Independent ID: fe72a978-d48f-8da1-0289-18ebec44135d
Content: Creating a Multi-Platform CocosSharp Project - Xamarin | Microsoft Docs
Content Source: docs/graphics-games/cocossharp/first-game/part1.md
Service: unspecified
GitHub Login: @charlespetzold
Microsoft Alias: chape

Hi @richardlawler. Thanks for calling this out. I'll provide an update for this doc!

Hi @richardlawler, I've updated and reorganized these docs a bit to remove the parts that reference the templates, since they are no longer available. I'd suggest opening an existing project (take a look at BouncingGame) to understand the project structure, references, etc. Thanks for calling this out!


Usernames involving special symbols (email, etc.) can't log in / view userpage

Seems to occur occasionally. Creating this issue to centralize the related threads:
#662, #555, #547, #401, #349

Closing #764, re-opening this as not resolved(?)

Good catch. I probably closed this by accident when I linked it to the other duplicate.

Clicking on my name in the top right corner does nothing but bring me to the same welcome example page thing with all the instructions on it.

Currently, I seem to be unable to access my user profile. When I try and access my profile by clicking on my email in the top right corner, I'm sent back to the main page instead of my user page, meaning I can't access any of the brews that I've currently made or any I make in future. I've tried this on chrome and firefox and I've got the same issue on each browser.

However this link works, revealing a published brew<EMAIL_ADDRESS>
So, maybe usernames as emails work .. but that says nothing about other edge cases so leaving open.
Note however that the username is case-sensitive<EMAIL_ADDRESS>
We need more examples of failures.

I think the link generated in the profile button is url encoded, turning the @ symbol into %40 or whatever. We just need to make the button not do that.

https://github.com/naturalcrit/homebrewery/blob/master/client/homebrew/navbar/account.navitem.jsx#L23-L25
return <Nav.item href={`/user/${global.account.username}`} color='yellow' icon='fa-user'>
	{global.account.username}
</Nav.item>;

https://github.com/naturalcrit/naturalcrit/issues/63 is also interesting to solve this and prevent similar issues.


Perf: improve loading speed for legacy table chart

CATEGORY

[x] Enhancement (new features, refinement)

SUMMARY
One of the main pain points with the table chart is that it loads too slow for large datasets. This PR upgrades it to an updated version with improved performance. See superset-ui-plugins PR#385 for details.
In addition to upgrade the table chart plugin, this PR also refactors some styling code related to it.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Tested with a ~10,000 rows, 7 columns table in Superset, pagination enabled:
Before

Takes about 6 secs to load.

After

Takes only 3 secs to load!

TEST PLAN

Query a large dataset, increase row limit to a very high number
Feel the difference in reactiveness and loading speed of the table chart

ADDITIONAL INFORMATION

[x] Has associated issue:  this does not fix #8273
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[x] Removes existing feature or API

REVIEWERS
@kristw @graceguo-supercat @etr2460 @rusackas

this change will automatic/force to show pagination on large table? Or it's up to user to show pagination or not?
The perf improvement from 6 seconds to 3 seconds, is because the table only show top 50 rows?

this change will automatic/force to show pagination on large table? Or it's up to user to select show pagination?
The perf improvement from 6 seconds to 3 seconds, is because the table only show top 50 rows?

No, it will not. This PR only refactors the model and disabled a couple of already-broken features. The perf gain mainly comes from other optimizations.

Codecov Report

Merging #9234 into master will increase coverage by <.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #9234      +/-   ##
==========================================
+ Coverage   58.93%   58.93%   +<.01%
==========================================
  Files         373      373
  Lines       12014    12017       +3
  Branches     2945     2946       +1
==========================================
+ Hits         7080     7082       +2
- Misses       4755     4756       +1
  Partials      179      179

Impacted Files
Coverage Δ

superset-frontend/src/chart/ChartRenderer.jsx
40.5% <66.66%> (+1.03%)
:arrow_up:

Continue to review full report at Codecov.

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

@ktmud Please rebase


Rename util.do_not_optimize to util.barrier (or something).

The op represents an optimization barrier along the use-def chain - it's unclear today that is the case (the form "don't optimize %0" doesn't really match; it doesn't mean "don't optimize the producer of %0" or "don't optimize the consumer of %0", just "don't optimize across %0", which is a barrier).

This also manifests in silly ways today with the following:
%0 = flow.dispatch @a
%1 = flow.dispatch @a
%2 = flow.dispatch @a
%dno0 = util.do_not_optimize(%0)
%dno1 = util.do_not_optimize(%1)
%dno2 = util.do_not_optimize(%2)

->
%0 = flow.dispatch @a
%dno0 = util.do_not_optimize(%0)
%dno1 = util.do_not_optimize(%0)
%dno2 = util.do_not_optimize(%0)

That's definitely not what I'd expect from "do not optimize %0/%1/%2"


Fix incorrect parameter order when calling RenameInternal

destinationPath and destinationFileSystem are switched in the Rename and RenameAsync methods when calling the RenameInternal method.

Hey @chrisjshields Thanks for your contribution and good catch on the bug.
Looks like you would have to rerecord all the affected tests by this change. I know it's probably difficult on your end to do so, since that would use your resources to do that. Would you be okay if I made a PR to resolve your issue, I can do the rerecording for you.

@amnguye Yes, no problem, go ahead. Happy to help!


[Suggestion] Toggle VRC chatbox

Add a button/checkbox in GUI to be able to turn off vrc chatbox. being able to toggle it would be a welcome addition since sometimes you dont wanna have duplicates

It will be a planned feature. It should be easy enough to add.


Missed observations on Bitcoin

Describe the Bug
Tokens sent on BTC Testnet are not send on Zetachain Testnet as tBTC
BTC Testnet transaction : https://blockchair.com/bitcoin/testnet/transaction/124d23127a54c5df8f13c2898542b567ca4c85d8370ef97d7363ce484f424b9f
I used hex:: in XDEFI in memo and I assume that it should works right now.
Related issue : https://github.com/zeta-chain/node/issues/1907

@SebaBau https://github.com/zeta-chain/node/issues/1907#issuecomment-2006762096


feat(NumberTheory/LegendreSymbol): @[csimp] rule for jacobiSym and legendreSym

Main definitions:

def jacobiSym.fastJacobiSym (a : ℤ) (b : ℕ) : ℤ
@[csimp] theorem jacobiSym.fastJacobiSym.eq : jacobiSym = jacobiSym.fastJacobiSym
def legendreSym.fastLegendreSym (p : ℕ) [Fact p.Prime] (a : ℤ) : ℤ
@[csimp] theorem legendreSym.fastLegendreSym.eq : legendreSym = legendreSym.fastLegendreSym

Also added tests

I added the algorithm description in the module docstring

@digama0 Could you please have a look at the meta code?

This failed on bors. I merged master; hopefully this will make the error show up here.

Thank you very much @digama0 and @Ruben-VandeVelde ! The branch should compile now.


source.list.d file is not created

Getting following Error when using this module.
Error: Failed to apply catalog: Could not find dependent Exec[apt_update] for File[/etc/apt/sources.list.d/mariadb.list] at /etc/puppet/modules/apt/manifests/setting.pp:63
Puppet Version: 3.4.3-1
Distro: Ubuntu 14.04/12.04

You need example42 apt module (but it might wok also with puppetlabs' apt module).
Alternatively manage the repos with a custom class passed via the parameter repo_class of the mariadb class


Update inference to support multi-gpu and offloading

This PR:

Adds native support for running inference on multiple GPUs by adding -g ID_1:vRAM_1 ID_2:vRAM_2 where ID_X is CUDA ID and vRAM_X is the amount of vRAM (in GiB) to allocate to loading the model. See the Inference README for more info.
Added option to run inference on specific CUDA devices.
Auto offload the model to RAM/Disk when running on devices with vRAM < size of model allowing the model to be loaded on consumer hardware.
Added documentation for inference with hardware requirements

This should fix most issues people have been having relating to multi-gpu inference, documentation, and lower-end (consumer) hardware.
List of things that can be fixed in the future:

Add CPU, RAM, and disk requirements to documentation
Generalize documentation and bot.py to work correctly on other models
Look into whether inference is possible to run only on CPU. Most likely, but if it is, it'll be incredibly slow. Even so, I think it's a good option to have.
Getting error RuntimeError: probability tensor contains either inf, nan or element < 0 on 2x 4090 24 GB but works fine on 2x 3090 24 GB and also on 1x 4090 24 GB. Will need further investigation.
Update some of the package versions to the latest versions, for example, the latest transformers version has a progress bar when loading model shards.
Apply a similar approach to training

Let me know if anyone has any suggestions!

Can solve:

OOM issues:

#28
#49

Multiple GPU:

#20

Hardware Requirements:

#60
#62

@justin-together can I get a review?

Looks great, @orangetin. I left a small nit ("vRAM" -> "VRAM") but I'm happy to merge this as is as well.

Fixed

Thanks, @orangetin!

Fantastic work, @orangetin.  Thanks!


Update version from 0.0.88 to 0.0.89

This is an automated update of version in pyproject.toml. This action is performed by publish.yml.

updated version from 0.0.88 to 0.0.89, @SuleyNL pls review


Configure parts of config.json in the boot partition

and this was also closed by #16

yeap, closing this as well


Dynamic add theme attribute to component

react-native, react and native-base version
"native-base": "^2.0.13",     "react": "15.4.2",     "react-native": "^0.42.2",
Expected behaviour
Sory for this issue request, but i don't know how to dynamic component theme attribute base on value. For example
<Button small  onPress={() => onPressButton()}       >         <Text >{onText}</Text>       </Button>
and i want to dynamic add "light" attribute to Button base on value on Button click.
Thanks !

@redwind applying boolean conditions on the prop should do the trick for you.
Something like this
<Button light={(condition) ? true : false}>
Let me know if that works for you.

@shivrajkumar it's work.
Thank so much !


EPIC: Alerts

User story
I am a public health official for Allegheny County in Pennsylvania. I check the dashboard as often as I can, but I don’t want to miss anything important. I am on the lookout for spikes in doctor’s visits related to COVID because I understand that is an early warning sign for the resurgence of an outbreak.
I want to be alerted if there is a rise in Doctor’s Visits in Allegheny County or any of the surrounding counties (Butler, Armstrong, Westmoreland, Washington, or Beaver counties)
I want my alert to come in the form of a text message at 9AM anytime Doctor’s Visits in the areas I care about increase.
“Subscribe to alert” screen / mode
I am going to go to the map and see a button that says “Subscribe to data alerts” - I click on that and I am taken to a new screen that explains what alerts are and how I might find them useful.
Choosing an alert method
Below the explanation, there are buttons for me to decide how I would like to receive the alert (Email or Text) and I choose text.
Gaining user trust
I want to know who I am giving my phone number to and that they won’t be texting me too often or about other things I don’t care about, but there is some text explaining this is from CMU and this number won’t be shared and will only be used for COVIDcast alerts, and is easy to unsubscribe from.
Choosing an alert
Once I select how to receive my alert, I go to a “choose alert” screen. It gives me some examples of different useful alerts and explains what they are looking for and how they can be used. I choose “Increase in Doctor’s Visits” as my alert.
Alert confirmation
The site confirms that my alert has been created and sends a text message to my number that tells me I have been subscribed.
I leave the page and forget that I set an alert.
Receiving an alert
A few days later I receive a text message. It reminds me that I set an alert using COVIDcast and tells me that doctor’s visits have increased by X count and Y percentage and gives me a link to the COVIDcast site so that I can see it for myself. It also reminds me that I can unsubscribe by saying “STOP” over text or deleting the alert on the website. I click the link in the text and go to the COVIDcast site and begin clicking around at my county and neighboring counties to get more information.

Tooling
After looking at a few ways to provide notifications, it looks like notifme-sdk is the most robust, and provides the most options in terms of delivery from SMS to email. This allows alerts to be sent from a node script as simply as
const NotifmeSdk = require("notifme-sdk").default

const notificationRequest = {
  email: {
    from<EMAIL_ADDRESS>    to<EMAIL_ADDRESS>    subject: 'COVIDcast alert: threshold crossed',
    html: '<b>Metric: </b> value',
    replyTo<EMAIL_ADDRESS>  }
}

const notifmeSdk = new NotifmeSdk({
  channels: {
    email: {
      providers: [{
        type: 'sendgrid',
        apiKey: 'SENDGRID-API-KEY'
      }]
    }
  }
})

Architecture questions
When a user "creates" an alert, is that "unique" alert tied to that user?
Or does a user simply subscribe to a pre-configured alert?
I think it makes much more sense, when a user creates an alert, that it is added to a pool of "shared" alerts that any other user can "subscribe" to. This will also make it easier for architecture and caching when multiple users create the same alert.
One can also imagine a two-tier system, where qualified researchers could create alerts, and the general public could simply subscribe to alerts from a list of those created by other users (and organized by different needs). There could also be "featured alerts" that are designed for a general audience or for specific use cases.

Figma containing the sketches above:  https://www.figma.com/file/QHWVg6ieuyInmsTzWDNYVZ/COVIDcast-Alert-UI?node-id=0%3A1

Closing this as obsolete.  Alerting is hard to do well, and we don't currently have plans for a general alerting system like this in the next few months.  We can revisit need in the future.


🙏  数值 hover 不显示 tooltip

🖋 Description

:tada: This issue has been resolved in version @antv/s2-v1.4.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version @antv/s2-v1.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use WebRTC when possible

WebRTC data channels let browsers pass data directly between themselves, without relying on the server.  If this system could use data channels for actual content delivery, the server would only have to route contact information between browsers, rather than the actual data.  This dramatically improves the scalability of the system.  Sharefest does basically this using BitTorrent.

WebRTC is not trustworthy for anonymous use because of IP-leaks


fix: normalize cache get and get! paths arguments

We hope that this change fixes the get and get! arguments on Windows when run from git bash

bors merge
Thanks!

bors r-

bors merge
(I edited the message)


Staff role for engagement lead: mandatory or not ?

Is your feature request related to a problem? Please describe
Why a Testing lead user must have the staff role ? As a standard user (ie. not staff) who interact with DefectDojo through the API, I am frustrated when I have to specify a staff-user for lead engagement. I may work in a context where there is no staff-user, and it does not make sense to specify the 'admin' user as engagement lead.
Describe the solution you'd like
As a standard user using DefectDojo API, I want to create engagements for which I have the Testing lead, so that :

I do not need to be staff myself. Otherwise I would have too many permissions on other products
I do not need to designate another staff-user as lead, because it does not make sense for my product

Describe alternatives you've considered
The solution would be to remove the exclusion filter isStaff=False in dojo/forms.py:669

Agreed the model is not what it should be. We welcome work on refactoring that ;-)
For my API users which import or re-import tests from pipelines, I do the following:

Only Active (no staff)
Through the /admin django interface, I assign specific wider permissions like

This only works for the API, however.
Hope this helps.


Not working with react 17 (or below)

I am encountering the error "You are using React 17 or below. Install with "npm install use-local-storage-state@17"
This can't be corrected using the instruction as the module requires the use-local-storage-state@18 which itself requires react@18. This has been since version 1.0.10.

Hey @flipersclub thanks for your highlighting this issue.
Indeed it looks like the use-local-strage-state package that this library uses does not support react < 18.
We've got two options:

Require users of this library to have a React version at least >=18
Implement our own useLocalStorageState hook and drop the dependency of the use-local-storage-state library.
Put use-local-storage-state in peerDependencies and let the user of this library install the 3rd party library independently.

At the moment I'm opting to go with option 1. Obviously, option 2 is the most ideal.
As far as option 3 goes, I think that it would be a bit of an "overkill" right now to require users of this library to install 3rd party packages themselves.
Of course if we realize that the need for react<=17 is getting higher, then we're going to revisit this :)


save tk dict to a json file

I needed to write the tk dictionary to a JSON for my own purposes. If you think you might need it sometime, here it is

@ABurrello can you check this on the regression? I think it is a reasonable thing to be added, could also be useful on our side.


[PN-64] Add ROSBag_Recorder Utility

Adds a wrapper around ros2bag::record to simplify recording rosbags when creating python nodes
This will be followed up by a PR to bdai that will update/add examples on how to use the utility with the walk forward example
In order to test, refer to the example_walk_forward_with_recording.py from this PR to bdai

Closing in favor of this PR to bdai where ROSBagRecorder will now be implemented as a singleton in record.py


Automatic recordScreenViews not working in Android & iOS (simulator environment)

A user indicated that this feature was not working for them:

I am trying to use automatic recordScreenViews in react-native. In the android simulator no $screen events are captured and on the iOS simulator the only screen names I get are UI and RCTModalHost . Am I missing something additional I have to do to make this work?

The user confirmed that manually sending a screen view works.

This was a factor of our limited v1 implementation. v2 is fully built for React Native and hooks into the react-native-navigation stack.


Finish (or at least improve) the documentation

https://genomespy.app/docs/

[x] Add a live example on the docs main page
[x] Add an example of a Unit spec and explain basic principles. Now the docs just link to Vega-Lite docs.
[x] Mark properties (some props are missing, extract them all from the JSON schema)
[x] "Sample faceting" is unclear, perhaps it could be called "cohort" to emphasize that it's completely different from the regular "facet" operator.
[x] View titles (implemented but undocumented)
[x] View size (it behaves in a bit differently than in Vega-Lite. There's already something on "concat" docs)
[ ] Expose the public ScaleResolution interface on the API docs page
[x] "Working with Genomic Data" is a mess. There's plenty of broken stuff.
[x] Cohort App Spec (how to specify a cohort visualization)
[x] Cohort App Users's Guide (at least briefly describe the available interactive features)
[x] Cohort App Getting Started (There's only the core package on the Getting Started page)

https://genomespy.app/ (the landing page)

[ ] Add some cohort stuff to https://genomespy.app/ (now it focuses mainly on the core package's capabilities)

Mostly done


Nuxt Edge : adding/handling babel polyfill?

I am starting using Nuxt Edge and trying to update a project based on Nuxt 1.4.2.
On Nuxt v1 I've added the babel polyfill for a IE11 compatibility following documentation and info n the forum, and it works.
I've used to install the npm package, and the nuxt config file I added the babel polyfill as vendor.
Now on Nuxt Edge the vendors are gone.
When I run Nuxt, the CLI give me a warning about vendor: "vendor has been deprecated due to webpack4 optimization"
So my question is: how can add on Nuxt Edge Babel Polyfill?
Anyone could give me some tips?
I see articles where people talking about the removal of vendors, but I didn't find anything explaining how to handle the old dependencies using vendor …
thanks :)

This question is available on Nuxt community (#c7787)

I have the same question. My Nuxt@2 website is not working in IE11.

Me too.
My nuxt2.2 project is not working in safari 9~10.
How do i fix?

I have the same question. My Nuxt@2 website is not working in IE11.

This should fix that
head {
 ...
 meta: [
    {'http-equiv': 'X-UA-Compatible', content: 'IE=edge'}
  ]
}


Ability to unmask currency as a float

Describe the Feature
Currently, if you type in a value using currency masking, ie:
type="currency"
options={{
    prefix: '$',
    decimalSeparator: '.',
    groupSeparator: ',',
    precision: 2
}}

And type in the input something like "$4,732.55" then try to unmask it through the onChange event:
onChangeText={(text, rawText) => {
  console.log(text) // $4,732.55
  console.log(rawText) // 473255
}}

You get the completely unmasked string, which isn't what I'd think most people would expect. I expect the string returned to be 4732.55, but it looks like the library removes all the separators and prefixes. Right now, I have a function that checks if the length of the value is greater than whatever the precision is and if it is, return the value with the decimal at -${precision}, instead, but I don't really know if returning an unmasked value without the precision included is helpful to anyone and it should probably just be built into the library.
Possible Implementation
Right now I'm taking the unmasked value and running this function on it:
interface IParseCurrency {
    value: string;
    precision?: number;
}

const parseCurrency = ({ value, precision }: IParseCurrency) => {
    const numberVal = Number(value);

    if (!precision || !numberVal) {
        return value;
    }

    const cutoffIndex = value.length - precision;
    const initialSubstring = value.substring(0, cutoffIndex);
    const endingSubstring = value.substring(cutoffIndex, value.length);

    return `${initialSubstring || 0}.${endingSubstring.length < 2 ? '0' + endingSubstring : endingSubstring}`;
};

Which seems to return everything how I'd expect. I assume the library is probably doing the masking with some sort of Regex pattern though, which would be a lot more simple to implement in reverse.

Yeah that cannot be the expected behavior at all. Periods should definitely be part of the raw text.

@DinsmoreDesign will you be implementing this enhancement? I too would like to see this altered

the library is open source, you can create a PR whenever you want!

we also support Issuehunt to budget desired issues from the community, or my GitHub sponsors to get special features requests or reviews/test
if you cannot give your time creating a PR or pay for maintainers contributors to implement it, I understand that the feature is not that important for you, so why would it be for me?

the feature is not that important for you, so why would it be for me?
Ouch. Was just asking a question


Fix minor typos in documentation and styles

Changes

Corrected typos in four files:

1_verify-contract-with-basescan.md
introduction-to-remix.md
styles.module.css
docusaurus.config.js

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

Hi! @wbnns Done, check it

@dramarereg Please rebase and resolve the merge conflict 🙏


Gallery Uploader bug

Gallery Uploader bug
When changing the title, description, etc. attributes, the values of all uploaded images or files change, that is, the value selected for one image is set for all uploaded ones when the Save button is pressed

Fixed


Resolves #2063

Add ssl example.
https://github.com/MicrosoftDocs/windows-powershell-docs/issues/2063

Docs Build status updates of commit 9119a73:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 3ed1e1a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@dariomws @get-itips @tseward
Peer review needed for this PR.
cc @JohanFreelancer9 for copy editing.
Thank you.

Also OK once the changes are applied.

Docs Build status updates of commit 3b17eb6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 3b17eb6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Actually, revisiting this -- the cmdlet examples don't show how to get into the IIS drive. Should we instead just use PS C:> for the examples?

Docs Build status updates of commit 2f40f1b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 2f40f1b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit cdb1325:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ee97c80:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012-ps/webadminstration/New-WebBinding.md
:white_check_mark:Succeeded

docset/winserver2012r2-ps/webadministration/New-WebBinding.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


MACDC-5745 Add different cleanser for OTL SRVC_ENDG_DT

What is this?
Addresses https://cms-dataconnect.atlassian.net/browse/MACDC-5745
How would you classify this change (feature, bug, maintenance, etc.)?
Bugfix
How did I test this (if code change)?
Generated SQL before and after change.
Executed here https://databricks-val-data.macbisdw.cmscloud.local/#notebook/3695012/command/3695029
Is there accompanying documentation for this change?
Issues Encountered
Optional section to include any challenges encountered, options considered, and the approach chosen.
PR Checklist

[x] Is the issue number and a short description in the subject line?
[ ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Credit to TMSIS PR template and https://embeddedartistry.com/blog/2017/8/4/a-github-pull-request-template-for-your-projects

@C1KB-tensileai The change to to_date('1960-01-01') turned out to be necessary. I updated the PR description with a link to the testing notebook.


Area chart only fills values from zero

I have an area chart with values ranging from -50 to +100. I would like the area to always fill from the bottom, but unfortunately it seems to only shade the area between the value and 0. Is there any way to override this?

v0.4.9 has been released, so please let me close.

hi @masayuki0812; have you released it at the end?
It does not seem to work to me, but maybe I didn't get it how to use it.


python3Packages.rdkit: 2022.03.3 -> 2022.03.4

Description of changes
https://github.com/rdkit/rdkit/releases/tag/Release_2022_03_4

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python310Packages.rdkit
python39Packages.rdkit

Thanks!


Fix autocomplete 'arg_lead' matched as regex and add autocompletion to pull from elsewhere action

Closed in favour of #421 which is its own branch


Jetpack recipes

It seems in the Latest beta SimplyJetpacks2-1.10.2-<IP_ADDRESS>, all the jetpacks don't seem to have a recipe for themselves.

Just a silly mistake, sorry for that...
It is fixed in build 46 which is currently under review on CurseForge.


Unable to Verify the First Certificate Error When Trying to Add Azure Account

Issue Type: Bug
When trying to add a Azure account to Azure Data Studio I am greated by the following error "unable to verify the first certificate" as soon as I click the Add an account button.
I have added "http.proxyStrictSSL": false to my settings but it did not make any difference.
Azure Data Studio version: azuredatastudio 1.13.1 (d08d07fe39c7fe74f9265db02cb963c38d048299, 2019-11-14T23:08:09.800Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Xeon(R) Gold 6126 CPU @ 2.60GHz (2 x 2600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: disabled_softwaremultiple_raster_threads: disabled_offnative_gpu_memory_buffers: disabled_softwareoop_rasterization: unavailable_offprotected_video_decode: unavailable_offrasterization: unavailable_offskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabled_readbackwebgl2: enabled_readback

Load (avg)
undefined

Memory (System)
4.00GB (0.78GB free)

Process Argv

Screen Reader
no

VM
100%

Extensions: none

I was able to fix this issue by bypassing our security web proxy (which had TLS inspection turned on).

@olivesandtrees thanks for providing the additional context.  ADS uses Node.JS for http communication, which doesn't use the system Certificate Authorities.  This creates issues with packet inspection firewalls that decode HTTPS traffic.  As noted, disabling SSL certificate validation with "http.proxyStrictSSL": false is the correct workaround at this time.


S3 Constructor the S3Client Occur the Assertion failed:s_HttpClientFactory line 192.

What platform/OS are you using?
windows7.
What compiler are you using? what version?
visual studio 2013  sdk c++ core & s3 1.6.25
What's your CMake arguments?
Local Windows debugger with VS2013
Can you provide a TRACE level log? (sanitize any sensitive information)
######The Code Like this:
// define the variable like BucketAndObjectOperationTest.cpp
// Create a client
ClientConfiguration config;

// init config like BucketAndObjectOperationTest.cpp
config.executor = Aws::MakeShared<Aws::Utils::Threading::PooledThreadExecutor>(ALLOCATION_TAG, 4);
Client = Aws::MakeShared<S3Client>(ALLOCATION_TAG,
         Aws::MakeShared<DefaultAWSCredentialsProviderChain>(ALLOCATION_TAG), config,
         AWSAuthV4Signer::PayloadSigningPolicy::Never /*signPayloads*/, true /*useVirtualAddressing*/);

when come to the last sentence Occur the Assertion failed:s_HttpClientFactory line 192.

solved with another method https://github.com/awsdocs/aws-doc-sdk-examples/issues/331


Node.js  Loaders Team Meeting 2023-08-01

Time
UTC Tue 01-Aug-2023 18:00 (06:00 PM):

Timezone
Date/Time

US / Pacific
Tue 01-Aug-2023 11:00 (11:00 AM)

US / Mountain
Tue 01-Aug-2023 12:00 (12:00 PM)

US / Central
Tue 01-Aug-2023 13:00 (01:00 PM)

US / Eastern
Tue 01-Aug-2023 14:00 (02:00 PM)

EU / Western
Tue 01-Aug-2023 19:00 (07:00 PM)

EU / Central
Tue 01-Aug-2023 20:00 (08:00 PM)

EU / Eastern
Tue 01-Aug-2023 21:00 (09:00 PM)

Moscow
Tue 01-Aug-2023 21:00 (09:00 PM)

Chennai
Tue 01-Aug-2023 23:30 (11:30 PM)

Hangzhou
Wed 02-Aug-2023 02:00 (02:00 AM)

Tokyo
Wed 02-Aug-2023 03:00 (03:00 AM)

Sydney
Wed 02-Aug-2023 04:00 (04:00 AM)

Or in your local time:

https://www.timeanddate.com/worldclock/fixedtime.html?msg=Node.js+Foundation+Loaders Team+Meeting+2023-08-01&iso=20230801T1800
or https://www.wolframalpha.com/input/?i=06PM+UTC%2C+Aug+01%2C+2023+in+local+time

Links

Minutes Google Doc: https://docs.google.com/document/d/1LHjGUvVSvOJcoNuNj4Jy1AztRLd4zzpqTLpJP2PmjcE/edit

Agenda
Extracted from loaders-agenda labelled issues and pull requests from the nodejs org prior to the meeting.
Invited

Loaders team: @nodejs/loaders

Observers/Guests
Notes
The agenda comes from issues labelled with loaders-agenda across all of the repositories in the nodejs org. Please label any additional issues that should be on the agenda before the meeting starts.
Joining the meeting

link for participants: https://zoom.us/j/97506450082

For those who just want to watch: https://www.youtube.com/c/nodejs+foundation/live

Hi, is there anything to discuss? @izaakschroeder are you good re https://github.com/nodejs/node/pull/48842? @bizob2828 @cspotcode anything regarding globalPreload or other upcoming changes?

Nothing from me at the moment, thanks.

I'm good too, thanks. I plan on attending and just listening in.

Well I won’t hold the meeting if there’s no agenda 😄  So unless there’s anything to discuss, I’ll just cancel it. I’ll wait until @izaakschroeder responds or it’s the day before, whichever comes first.

@GeoffreyBooth I know you talked in slack about most of the items landing soon. I won't have bandwidth to build from source and explore before release. Do you have an idea when loaders will be stable. I seem to remember mid August?

Do you have an idea when loaders will be stable. I seem to remember mid August?

No, mid-August is the earliest that https://github.com/nodejs/node/pull/48842 could be released, which includes register and initialize. There’s also https://github.com/nodejs/node/pull/47999 that needs to be finished and landed and released before we’ve completed our “must finish before stable” to-do list. See https://github.com/nodejs/loaders#milestone-2-stability
Once both of those PRs are released, though, the Loaders API won’t be immediately stable; we’ll wait for a while, possibly months, to respond to any bugs that users report. Once things quiet down and we think we’ve squashed all important bugs and made any final API revisions that we discover the need to make per user feedback, then we’ll make it stable. So the soonest it can enter this “release candidate” phase is late August/early September, but since no one is actively working on https://github.com/nodejs/node/pull/47999 at the moment I can’t predict how far off its landing will be.

Hi, is there anything to discuss? @izaakschroeder are you good re https://github.com/nodejs/node/pull/48842?

All good from my end. Sorry work has been hectic this week. I should be able to get that PR wrapped up this weekend though 🤞

Sounds good. I’ll leave this open until Monday in case anything comes up over the weekend that you want to discuss on Tuesday, otherwise on Monday I’ll cancel the meeting.

@izaakschroeder do you think we need a meeting? https://github.com/nodejs/node/pull/48842 looks pretty good, I think we can sort out the register signature async probably, and the other notes all seem pretty minor.

Works for me @GeoffreyBooth I don't see any significant changes that need to be made either 😄 Like you said, the signature and any other minor doc updates can probably be sorted out async, and if we ever built import.meta.register (as https://github.com/nodejs/node/pull/48740 now seems? to suggest might be more ok than I originally believed) it would  be easy to do separately and there's no need to block current work over.


Bigscreen beyond fan speed locked at 100%

It seems no matter what sleep mode or not my fan speed is always locked at 100% Is this intended?

Please switch to the Beta branch on Steam. This issue is already fixed there and should land in v1.13 soon.


Solder linking problem

Hello, I have a problem when I want to link my solder to the website TechnicPack.net. I got this error :
[curl] 23: Failed writing body (0 != 39) [url] http://odysse.myftp.org:46000/TechnicSolder/public/index.php/api/verify/c5af03180b8cfe2803c6bedbb4c7256426a33f991cb47b8df364e68b2cc7d870
Anybody can help me please ? Genpage ? : D
I'm on Debian 7.7 and I using Solder <IP_ADDRESS> and in update checker I have this :
Shell Access: Yes
Git Installed: Yes
Git Repository Found: Yes

Link your Solder please.

I try to do it, but if failed

http://odysse.myftp.org:46000/TechnicSolder/public/index.php
Is this your solder install? Because if it is, I can't access it.

Yes it's my solder install

I don't know, that's on your end bud.


Making compatible with omniauth-oauth2 version 1.2.0

:+1:

@masylum you are really fast xD

:+1: great work @alexandrebini

@bpedro Could you do the magic? =)


Keys method on Godot.Collections.Dictionary<TKey, TValue> converts to TKey when a typecast exception would be expected

Godot version
3.3.2.stable
System information
Windows 10
Issue description
When I use Godot.Collections.Dictionary<int, string>() to wrap a mixed-bag Dictionary, Keys converts each key to int.
For example, if the dictionary contains "1", it's converted to 1 by Keys, which makes the key "invalid": dict.ContainsKey(key) returns false!
I'd expect Keys to throw a typecast exception when it hits the keys with the bad type, instead.
Practical example: if I've deserialized some JSON, I might not know what the contents are, so I might want to iterate through Keys to see which keys have values in my dictionary. But if the type doesn't match up, Keys gives me "fake" keys that don't actually have any values. (Arguably I should just use Godot.Collections.Dictionary and handle the object keys myself since that matches what the engine is doing behind the scenes, but it is nice sometimes to get automatic static types.)
This is not a big issue, it's just hard to debug because it's difficult to realize that the Keys API isn't working as expected. Someone hit it on Discord so I figured I should file this to at least document what we ran into. 😄
Steps to reproduce
var dict = new Godot.Collections.Dictionary
{
	["42"] = "string 42",
	[9001] = "int 9001",
	[new Godot.Object()] = "object",
};
var typedDict = new Godot.Collections.Dictionary<int, string>(dict);

foreach (int i in typedDict.Keys)
{
	if (typedDict.TryGetValue(i, out string value))
	{
		GD.Print($"key '{i}' has '{value}'");
	}
	else
	{
		GD.Print($"key '{i}' has no value");
	}
}

The string and Godot.Object keys are converted to ints, which aren't valid keys, so this prints:
key '42' has no value
key '9001' has 'int 9001'
key '0' has no value

Minimal reproduction project
GodotCSTypedDictionaryKeyConvert.zip

It seems this is a consequence of #22460.


exception when processing incoming waveform report

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
File "C:\src\sdcsim.sdcsim_venv_AMD64_3.9.6\lib\site-packages\sdc11073\mdib\clientmdibxtra.py", line 303, in _on_waveform_report
accepted_states = self._mdib.process_incoming_waveform_states(received_message_data.mdib_version_group,
File "C:\src\sdcsim.sdcsim_venv_AMD64_3.9.6\lib\site-packages\sdc11073\mdib\clientmdib.py", line 477, in process_incoming_waveform_states
states_by_handle = self._process_incoming_states(
File "C:\src\sdcsim.sdcsim_venv_AMD64_3.9.6\lib\site-packages\sdc11073\mdib\clientmdib.py", line 364, in _process_incoming_states
if self._has_new_state_usable_state_version(old_state_container, state_container,
File "C:\src\sdcsim.sdcsim_venv_AMD64_3.9.6\lib\site-packages\sdc11073\mdib\clientmdib.py", line 635, in _has_new_state_usable_state_version
diff = int(new_state_container.StateVersion) - int(old_state_container.StateVersion)
TypeError: int() argument must be a string, a bytes-like object or a number, not 'NoneType'
Expected Behavior
no exception
bug is caused by wrong modeling of Stateversion, instead of default_py_value = 0 it must be implied_py_value = 0.
Minimal Reproducible Example
No response
Python Version
all
Operating system
all
Sdc11073 Version
2.0.0a4
Link to sdc11073 Logs
No response
Further Information
No response
Participation

[ ] I am willing to submit a pull request to fix this bug.

There are some more places where default_py_value should be implied_py_value.


Mod not working?

Tried out the mod with your new instructions but I seem to be unable to get it to work. Video of what I tried.
Added an active effect to an existing feature and gave it an aura of 10 feet. For the effect I chose the token magic glow aura. When the feature is added to a character sheet the glow effect is applied correctly. But the aura doesn't seem to transfer the effect to surrounding tokens?
Or perhaps I am misinterpreting the instructions again. Sorry again for the trouble.
Mods Active:
TidyUI, TidySheet, Active Auras, DAE, Token Magic FX.
Cheers 👍

Do you get any errors in the console ?

Do you get any errors in the console ?

I do get this error when i remove the feature ("Aura of Protection") from the wolf, as seen in the video.
No errors when i add the feature granting the aura to the wolf, and none when i move the wolf close to the bigger wolf.

I do get this error when i remove the feature ("Aura of Protection") from the wolf, as seen in the video.
No errors when i add the feature granting the aura to the wolf, and none when i move the wolf close to the bigger wolf.

could you PM me an invite to your game and make an assistant account so I can locally test?

could you PM me an invite to your game and make an assistant account so I can locally test?


IIIF Manifest for Multi-Volume Work should display scanned resources under it

Note that these should be iiif collections: http://prezi-21.iiif.io/api/presentation/2.1/#collections


170125 snr plots

should be good now, circle permitting
also had to add vAxis in a few place to remove the min/max for other graphs as it carries across when viewing in fullscreen and upon coming out of fullscreen could have unexpected reults.
Also I'm like 99% sure vAxes is a typo and won't do anything, so I removed some references to that.

@iamzakr Merging then if you're happy.
Test and move off the git issues if required.


Updating series in turbo mode causes browser to hang indefinitely

Version
nuxt-highcharts: 2.0.4
nuxt: 2.15.8
Nuxt configuration
mode:

[ ] universal
[x] spa

Nuxt configuration for nuxt-highcharts
 modules: [
    // https://go.nuxtjs.dev/axios
    '@nuxtjs/axios',
    // https://go.nuxtjs.dev/pwa
    '@nuxtjs/pwa',
    'nuxt-highcharts',
  ],

Reproduction Link

Steps to reproduce

Use following code, place under pages directory (i.e. using the default template).
Press add button
Press add button

<template>

    <v-btn v-on:click="addSome()">add</v-btn>
    <highchart :options="chartOptions" />

</template>

What is Expected?
2 points added to the chart
What is actually happening?
1 point is added, after the second click to add button, browser hangs indefinitely with high CPU utilization

Performance analysis?

And what happens if you try the BasicChart.vue copied in to your components?

Ok, those are all concerns with the highcharts API usage and Vue fundamentals, not the original issue. At that point, the underlying libraries deal with that logic. The docs here mention what data types to use: https://www.highcharts.com/docs/advanced-chart-features/boost-module#configuration-data-options .

In boost mode, turbo mode is always turned on. That means all data points should be configured as an array of numbers (e.g. [1, 2, 3]) or a two dimensional array of numbers (e.g. [ [1, 2], [2, 3], [3, 4] ]).

I am able to use the boost module like this w/o issue:
<highchart :modules="['boost']" ... />

How are you using / configuring it?

I don't currently have the time right now to look at this but perhaps later tonight. .push should work and I agree with you it's preferred to performing a direct copy of the array.

Ok, I believe I know what's happening, but I believe it's out of my control. I think when the series.update underlying method is called, it may be modifying the data that's given to it, and if it does, it will trigger Vue's watchers to keep firing (so an endless loop).  The way to overcome this is to then disable the watching, but then update the chart as desired. This way you can do the array push instead of array copy.
<highchart ref="chart2D" :options="chartOpts" :update="[]" />

Setting update to an empty array will disable the watchers. But then the chart updating has to be implemented here:
methods: {
  append () {
    this.points.push([this.cnt, Math.random() * 10])
    this.$refs.chart2D.chart.series[0].update(this.chartOpts.series[0])
    this.cnt++
  }
}

Pushed a working example here: https://github.com/richardeschloss/nuxt-highcharts/blob/feat/v3-rc/components/charts/Data2D.vue


process: add process.status attribute

Related Issue: #1181
Changes
This PR deprecates system.process.status and replaces it with process.status. The previous usage of system.process.status has been replaced with process.status, and process.status has also been add as a resource attribute to the process resource.
Merge requirement checklist

[x] CONTRIBUTING.md guidelines followed.
[ ] Change log entry added, according to the guidelines in When to add a changelog entry.

If your PR does not need a change log, start the PR title with [chore]

[x] schema-next.yaml updated with changes to existing conventions.

Based on the discussion in SemConv SIG on 8/12, the only blocker is listing process.status as a resource attribute. We can unblock this PR (if it makes sense) by defining  process.status attribute in the registry, but not referencing it in process resource definition. It can also be used on metrics as a regular attribute.

I could do that if it would help. I don't suspect anything in the process or system namespace will use this attribute, but if there may be a use case for it I can adjust this PR to just be a definition in the registry.

Hey @braydonk, I wonder if this could be modeled as a metric instead. It was also suggested at https://github.com/open-telemetry/semantic-conventions/issues/1032#issuecomment-2420399295 and it seems it's already done for the hw.status : https://opentelemetry.io/docs/specs/semconv/system/hardware-metrics/#hwbattery---battery-metrics

This is going to be redone to model it the same as hw.status as @ChrsMark mentioned above. Since this is so old, I'm just going to close this and redo it.


Should all receivers be moved into the completion signals?

 Issue by msimberg
Monday Jan 24, 2022 at 11:53 GMT
Originally opened as https://github.com/NVIDIA/stdexec/issues/422

I don't know if this makes any difference in the wording, but there's currently quite a mix of how receivers are passed into completion signals:

C-style cast (related: https://github.com/brycelelbach/wg21_p2300_std_execution/issues/379):
https://github.com/brycelelbach/wg21_p2300_std_execution/blob/e3627f3b90e914591cdf2d0a93aae98aa252eb27/std_execution.bs#L4520
No move/cast:
https://github.com/brycelelbach/wg21_p2300_std_execution/blob/e3627f3b90e914591cdf2d0a93aae98aa252eb27/std_execution.bs#L4962
Move:
https://github.com/brycelelbach/wg21_p2300_std_execution/blob/e3627f3b90e914591cdf2d0a93aae98aa252eb27/std_execution.bs#L5242

Should they all just be std::moved? Or something else? I'll happily attempt to clean this up (including #379).

 Comment by ericniebler
Monday Jan 24, 2022 at 23:17 GMT

The answer depends. When the algorithms mention out_r, what they should be saying is that they are manipulating an rvalue (or xvalue to be more precise) that refers to a copy of out_r that lives in <place>. ("<place>" most likely is within the operation state.) If for the sake of the specification, you give that rvalue a name like out_r', then out_r' can appear in a set_[value|error|stopped] expression without a cast or a move, because it is already an rvalue.
Also see #403.


Gamma Strategies QA (Ethereum) Schema Version 1.2.1 Subgraph Version 1.0.0 Methodology Version 1.0.0

Description
Value

Subgraph Reviewed
https://subgraphs.messari.io/subgraph?endpoint=https://api.thegraph.com/subgraphs/name/messari/gamma-ethereum

Date Reviewed
June 29 2022

Schema Version
1.2.1

Subgraph Version
1.0.0

Methodology Version
1.0.0

Evidence Spreadsheet
https://docs.google.com/spreadsheets/d/1R7zUw68xAlzHYwjzj2Y1PuXkkFeCAiTd3FTT1guyGA0/edit?usp=sharing

Metrics To Review
Protocol Metrics

Section
Metric
Issue

financialsDailySnapshots
totalValueLockedUSD
Trend and data for most points matches up. Between 10/01/2021 - 12/14/2021 Defillama has a max TVL of 25m and the subgraph is showing 35m ... not sure if this is wrong, but big enough discrepancy to take a look

Vault Overview

Pool
Metric
Issue

ALL POOLS
ALL METRICS
No outputting anything here

Vault Metrics

Pool
Section
Metric
Issue

xWBTC-ETH05
vaultDailySnapshots
totalValueLockedUSD
Values are good, but daily time series is skipping weeks at a time

xWBTC-ETH06
vaultDailySnapshots
inputTokenBalance -  xWBTC-ETH05
Negative values on 04/04/2022 and 03/30/2022. Rest of values look good, but daily time series is skipping weeks at a time

xWBTC-ETH07
vaultDailySnapshots
outputTokenSupply -  xWBTC-ETH05
Negative values on 04/04/2022 and 03/30/2022. Rest of values look good, but daily time series is skipping weeks at a time

xWBTC-ETH08
vaultDailySnapshots
outputTokenPriceUSD -  xWBTC-ETH05
Values are good, but daily time series is skipping weeks at a time

xWBTC-ETH09
vaultHourlySnapshots
totalValueLockedUSD
Values are good, but daily time series is skipping weeks at a time

xWBTC-ETH10
vaultHourlySnapshots
inputTokenBalance -  xWBTC-ETH05
Negative values on 04/04/2022 and 03/30/2022. Rest of values look good, but daily time series is skipping weeks at a time

xWBTC-ETH11
vaultHourlySnapshots
outputTokenSupply -  xWBTC-ETH05
Negative values on 04/04/2022 and 03/30/2022. Rest of values look good, but daily time series is skipping weeks at a time

xWBTC-ETH12
vaultHourlySnapshots
outputTokenPriceUSD -  xWBTC-ETH05
Values are good, but daily time series is skipping weeks at a time

This has been fixed here (also upgraded to 1.3.0): https://github.com/messari/subgraphs/pull/829
Also fully indexed so can be reviewed again.

Description
Value

Subgraph Reviewed
https://subgraphs.messari.io/subgraph?endpoint=https://api.thegraph.com/subgraphs/name/messari/gamma-ethereum&tab=protocol

Date Reviewed
August 22 2022

Schema Version
1.3.0

Subgraph Version
1.1.1

Methodology Version
1.0.0

Evidence Spreadsheet
https://docs.google.com/spreadsheets/d/1oJJ5WpaST6qQuPbEE7s9m7FN3BNQCt-qeLDLMPOpM6o/edit?usp=sharing

Metrics To Review
Protocol Metrics

Section
Metric
Issue

financialsDailySnapshots
totalValueLockedUSD
The trend generally looks ok, but between 11/2021 - 1/2022 the max TVL is 25M on defiLlama and the subgraph's is 35m (roughly 50% difference in TVL)

usageMetricsDailySnapshots
totalPoolCount
We have 77 pools listed when the Gamma UI only has 16. I am guessing we are counting historic and low TVL pools, while the Gamma UI is filtering them out. But could use a second look.

Vault Overview
Comments - The Gamma UI (https://app.gamma.xyz/dashboard) only has 16 pools in it and the largest TVL pool is 100k. I am not sure if the pools we are reporting are correct or not, but going to make a table of the larger pools (100k+ TVL) missing from the Gamma UI. It does seem that the pools that Gamma is showing, we are reporting the correct TVL for them. If these pools exist and the TVLs seem correct, then Gamma UI is most likely incorrect. Based on my quick smart contract scan, looks like these vaults exist and the TVL is correct, but good to get a second look.

Vault name
Vault ID

Visor WBTC-ETH Uni v3
0x97491b65c9c8e8754b5c55ed208ff490b2ee6190

LQTY-ETH Uni .3%
0x586880065937a0b1b9541723619b75739df8ef13

rETH-THETA - ETH
0x24fe0b138d9b10a7f0502e213212ee6648926ecb

Visor ETH-THX Uni .3%
0xe065ff6a26f286ddb0e823920caaecd1fcd57ba1

GAMMA-ETH Uni .3%
0x33412fef1af035d6dba8b2f9b33b022e4c31dbb4

xH2O-USDC05
0xe1ae05518a67ebe7e1e08e3b22d905d6c05b6c0f

Visor ETH-BABL.2 Uni .3%
0x5e6c481de496554b66657dd1ca1f70c61cf11660

Visor ETH-BABL Uni .3%
0xf19f91d7889668a533f14d076adc187be781a458

xagEUR-USDC01
0x717a3276bd6f9e2f0ae447e0ffb45d0fa1c2dc57

Visor ETH-TRU Uni v3
0xc92ff322c8a18e38b46393dbcc8a7c5691586497

Visor ARCX-ETH Uni 1%
0x1b56860eaf38f27b99d2b0d8ffac86b0f1173f1a

xPSDN-ETH1
0x0ec4a47065bf52e1874d2491d4deeed3c638c75f

Visor ENS-ETH Uni .3%
0x5230371a6d5311b1d7dd30c0f5474c2ef0a24661

Vault Metrics

Vault
Section
Metric
Issue

xWBTC-ETH05
vaultDailySnapshots
inputTokenBalance -  xWBTC-ETH05
Subgraph.xyz is outputting an error that there is negative values being output. Looks like from 03/30/2022 on there are slight negative values. When querying the subgraph it seems like there are larger negative values. Link to query - https://api.thegraph.com/subgraphs/name/messari/gamma-ethereum/graphql?query={
++vaultDailySnapshot+(id%3A+"0x35abccd8e577607275647edab08c537fa32cc65e-19081")+{
++++inputTokenBalance
++++timestamp
++++vault+{
++++++name
++++++id
++++}
++}
}++

xWBTC-ETH06
vaultDailySnapshots
outputTokenSupply -  xWBTC-ETH05
Same comment as above

On the discrepancy of number of vaults, other than historic and low-TVL vaults, gamma also does treasury management for protocols.  My understanding is that these are not open to the public, and only specific address can deposit/withdraw.  These vaults do not appear in the UI.
Haven't looked in depth for the others issues yet - will do that and report back.

On the discrepancy of number of vaults, other than historic and low-TVL vaults, gamma also does treasury management for protocols. My understanding is that these are not open to the public, and only specific address can deposit/withdraw. These vaults do not appear in the UI.
Haven't looked in depth for the others issues yet - will do that and report back.

Ahh that would make sense. Thanks!

The negative values seems to be a result of a vault that was added "late" to the registry contract, hence some deposit events were missed.  Working on a fix which is syncing at l0c4t0r/laboratory2.

@bye43 The fixes have now been merged and fully synced for review. Thanks!

@bye43 The fixes have now been merged and fully synced for review. Thanks!

Just took a look and all the issues look solved to me! @l0c4t0r thanks! @this-username-is-taken this is good to be frozen and I am going to close the issue.


actions/checkout@v3 use node version that is going to be deprecated #199 fix

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

@galbaneim-orca Thank you for creating the PR. I've tested the failing check, it's green 👍


Update [platform].mdx

Issue #, if available:
Description of changes:

[x] Redirects needed?

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

@Mergifyio refresh


openjdk: Merge passthrus

Fixes #104680.

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I will merge this when ofborg will confirm that there are no rebuilds triggered.


doc: Rename stable API change to breaking change

It is often confusing for users and developers alike to see the sentence "stable API change" in a label or in the release notes. Stable APIs can change in at least two ways (retaining compatibility or not), and so it is preferrable to use a term that clearly describes the change as incompatible, by using the common term "breaking".
Note: I will rename the label once this is merged.

@kartben had to push to fix a minor doc warning.


Fix arrayOrdering optimization

(to avoid compound ordering creation if it's not necessary)

Codecov Report

Merging #5040 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5040      +/-   ##
============================================
+ Coverage     82.16%   82.23%   +0.06%
- Complexity    10720    10722       +2
============================================
  Files          1298     1298
  Lines         44646    44646
  Branches       5656     5656
============================================
+ Hits          36685    36716      +31
+ Misses         5301     5272      -29
+ Partials       2660     2658       -2

Impacted Files
Coverage Δ
Complexity Δ

...eration/projectors/sorting/OrderingByPosition.java
86.66% <100%> (+4.44%)
18 <3> (+1)
:white_check_mark:

...n/java/io/crate/exceptions/JobKilledException.java
0% <0%> (-66.67%)
0% <0%> (-1%)

...crate/executor/transport/TransportShardAction.java
77.5% <0%> (-20%)
8% <0%> (-2%)

...va/io/crate/operation/projectors/DMLProjector.java
77.77% <0%> (-14.82%)
5% <0%> (-1%)

...tor/transport/distributed/SingleBucketBuilder.java
66.66% <0%> (-9.53%)
6% <0%> (-1%)

...eration/projectors/ColumnIndexWriterProjector.java
73.33% <0%> (-8.89%)
7% <0%> (-1%)

...java/io/crate/protocols/postgres/SimplePortal.java
80.18% <0%> (-8.5%)
23% <0%> (ø)

...ava/io/crate/jobs/AbstractExecutionSubContext.java
84% <0%> (-4%)
17% <0%> (ø)

...rc/main/java/io/crate/blob/BlobTransferTarget.java
84.1% <0%> (-3.32%)
25% <0%> (-1%)

.../src/main/java/io/crate/exceptions/Exceptions.java
67.01% <0%> (-3.1%)
32% <0%> (-3%)

... and 34 more

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 996f232...c944607. Read the comment docs.


loss=0 in step=101(after two step)

hello i utilize your useful program for face recognition,my implemented network is inception resnet v2 and

batch_size=16,
2.lr=0.001
3.embeding_size=128
4.train_size=708
5.eval_size=396
6.margin=0.5
7."triplet_strategy": "batch_all"
loss = 0.5458313, step = 1
loss = 0.0, step = 101 (3229.453 sec)
without pretrained weights and only with your implementation two layers CNN ,loss  for evaluation goes to about 0.35(batch_size=64,lr=1e-4)with data augmentation .i use embeddings vector from PREDICTION mode as input of  my SVM and KNN classifier but  accuracies of both are too low about 10%. what do think  about this low accuracy for recognition?which part  do you think have caused this low accuracy (triplet part   or classification  part)?

If the batch size is too low (16) and you have a lot of different classes (ex: 100), you may have no valid triplet in your batch, which might give you a loss of zero.
Or maybe the model is not converging because the learning rate is too high.


Avoid setting a negative memory limit when no cgroup mem limit is imposed

Hi,
I wanted to try out what happens when there's no cgroup memory limit set and this library was in use. The result is that debug.SetMemoryLimit() is called with a negative number.
Not sure about the consequences, but this PR should avoid this incorrect call.
Now I could probably check the value after multiplying it with the ratio, but I think that a limit above 2**63 generally means "no limit", so we should avoid making calculations with it entirely.

HI, @LubosD
Thank you for your attention and contribution!
But sadly, I can't reproduce the situation you issued in the simple testing through SetGoMemLimitWithProvider(Limit(math.MaxUint64), 1).
As I understand about Go spec, float64 to int64 conversion will automatically cap the value if the float64 value is greater than the max value of int64.
If possible, can you provide a recipe for reproducing that situation?

Right, maybe I failed to spot the right location of the problem. So I know that memlimit.FromCgroup() returns<PHONE_NUMBER>3709551615 on my system.
When I use this trivial program:
package main

import (
	_ "github.com/KimMachineGun/automemlimit"
)

func main() {
}

and run it, the following gets printed:
$ AUTOMEMLIMIT_DEBUG=true ./memlimit
2022/11/20 11:12:36 GOMEMLIMIT=-9223372036854775808

LGTM!

Great! Thank you for your contribution again 😄


Skyplane is slow for large number of small files

Currently, for large number of small files, Skyplane is bottlenecks on the chunk dispatch because the file listing is much slower than Skyplane's ability to transfer the data
Proposed solution:

Use multiple processes/threads to list objects in parallel (this can be done by listing random prefixes)
Use aync HTTP requests to send chunk requests, rather than waiting

Have a look at S3P if you haven't already; it seems to have implemented a partial strategy for this that could be good to take inspiration from. This is a HUGE use case for us.


Feature: check redis when init

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've discussed this feature request in the OpenIMSDK Slack and got positive feedback

Is this feature request related to a problem?
❎ No
Problem Description

now missing some check when chat start
Solution Description
check redis
Benefits
more thorough check
Potential Drawbacks
No response
Additional Information
No response

Could you please provide an update on whether there is a PR in progress for this issue? Additionally, could you kindly describe the approach for implementing the pre-launch check for the chat feature? Thank you.


Should be “Recipients” instead of “Recepients” field

tap send by email for any widget on dashboard
look at “Recepients” field

Result: Should be “Recipients” instead of “Recepients” field

just fixed by @Boniara


Allow Neve to be used as a Nixvim module

Context
Currently, Neve is being installed as a package by Nix. While it's easy to install, it forces users to clone the project and make their own configuration.
The goal of this PR is to provide an alternative that allows us to modify Neve's configuration without touching the flake sources directly.
Example
programs.nixvim = {
  enable = true;
  imports = [ inputs.Neve.nixvimModule ];
  # Then configure Nixvim as usual, you might have to lib.mkForce some of the settings
  colorschemes.catppuccin.enable = lib.mkForce false;
  colorschemes.nord.enable = true;
};

This keeps the default Neve configuration while allowing users to disable the catppuccin theme and enabling Nord. The code for this is fairly straightforward.
For now, we only use nixvim configuration options but you could define your own to allow for some extra configuration

Thank you! :smile:


Cancelling on Result Err

Hi I have a function, which returns an Result<(), Error>. I want to early return on errors.
I saw in result.rs  something, that uses while_some may be the right method here, although I don't know how to use that. I couldn't find something in the documentation.
Also a while_ok method would probably be nice.

Thanks, I wasn't aware of the type hint, but seems pretty obvious in hindsight.

while_ok would allow you to have a filter_map that stops depending on a predicate.
while_some does not work with filter_map for obvious reasons 😁

Hello, I'm doing
fn main() -> anyhow::Result<()> {
    0..123123
        .into_par_iter()
        .map(|num| {
            /* something with lots of foo()? */
            println!("Hi from {num}");
            anyhow::Ok(())
        })
        .collect::<Result<Vec<_>, _>>()?
    ;
    Ok(())
}

And then running with cargo run | head -n 10. So when Bash closes the pipe to head after 10 successful printlns, errors start getting throw for every println with the broken pipe error, as expected. However, this doesn't seem to stop Rayon, as it keeps dumping panic errors on stderr non-stop.
What am I doing wrong please?

@YuriGeinishO the println! macro doesn't return any errors, even though it reports them. Try something like this:
use std::io::{stdout, Write};
// ...
            writeln!(stdout(), "Hi from {num}")?;


Breadcrumb: Active State

In this PR:
Add breadcrumb active state is-active with bold font and grey color and related root variables.

Screenshot(s):
Before

After

Pre-Merge Checklist:
Reviewer # 1

[ ] Review Source code.
[ ] Review Test code.
[ ] Review Snapshot.
[ ] Review PR Storybook.

NOTE: To review PR Storybook for the last build, download PR storybook from CI build Artifacts.
PR Owner

[ ]  Pipeline is green.

Looks good thank you!


no option to set color axis limits in mne_nirs.visualisation.plot_nirs_source_detector()

Describe the new feature or enhancement
As far as I am aware, the function mne_nirs.visualisation.plot_nirs_source_detector() has no option of setting the axis limits for the color bar. An argument in the function such as "clim" would be handy to make sure the color axis is representing similiar values when plotting e.g. GLM results of different conditions, to make plots visually comparable.

agree that it seems like a good idea to expose that functionality. Are you up for making a pull request to add it?  In MNE packages we usually call such variables vmin/vmax (scalars) or vlim (tuple of two scalars), with a preference for migrating to vlim everywhere (eventually / gradually). I'm not super familiar with the NIRS codebase though, so if it's exposed everywhere else in the mne_nirs API as vmin / vmax it's fine to follow suit here.


fix: add modal submit interaction to APIInteraction union & make data required

Please describe the changes this PR makes and why it should be merged:

https://discord.com/channels/222078108977594368/941291502372552735/941658634973245531

Reference Discord API Docs PRs or commits:

https://github.com/discord/discord-api-docs/pull/4486

Might also need to check if it's similar to chat input commands with DM/Guild interactions 👀

Looks great! Found the same missing guild/dm interactions for autocompletes that anyone can fix in a subsequent PR 👀