[enhancement] create sklearnex/test/utils package for sklearnex testing

Description
move _utils.py and _utils_spmd.py to a separate package to make further modifications/expansions of these utilities easier.
Renames _sklearn_clone_dict to sklearn_clone_dict
Removes a relative import from test_run_to_run_stability.

Checklist to comply with before moving PR from draft:
PR completeness and readability

[ ] I have reviewed my changes thoroughly before submitting this pull request.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation to reflect the changes or created a separate PR with update and provided its number in the description, if necessary.
[ ] Git commit message contains an appropriate signed-off-by string (see CONTRIBUTING.md for details).
[ ] I have added a respective label(s) to PR if I have a permission for that.
[ ] I have resolved any merge conflicts that might occur with the base branch.

Testing

[ ] The unit tests pass successfully.
[ ] I have run it locally and tested the changes extensively.

Performance

[ ] I have measured performance for affected algorithms using scikit-learn_bench and provided at least summary table with measured data, if performance change is expected.
[ ] I have provided justification why performance has changed or why changes are not expected.

/intelci: run

/intelci: run


Updating from 11.2.5 to 11.2.6 throws an error

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[x] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

 Yes, the previous version in which this bug was not present
Description
 A clear and concise description of the problem...
When I tried to update from 11.2.5 to 11.2.6 with ng update @angular/cli , I receive an error.
🔬 Minimal Reproduction
ng update @angular/cli
🔥 Exception or Error
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/@angular-devkit/build-angular
npm ERR!   dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/@angular/localize
npm ERR!   peerOptional<EMAIL_ADDRESS>|| ^11.2.0-next" from<EMAIL_ADDRESS>npm ERR!   node_modules/@angular-devkit/build-angular
npm ERR!     dev<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

🌍 Your Environment

Angular CLI: 11.2.5
Node: 15.10.0
OS: win32 x64

Angular: 11.2.6
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1102.5
@angular-devkit/build-angular   0.1102.5
@angular-devkit/core            11.2.5
@angular-devkit/schematics      11.2.5
@angular/cdk                    11.2.5
@angular/cli                    11.2.5
@angular/material               11.2.5
@schematics/angular             11.2.5
@schematics/update              0.1102.5
rxjs                            6.6.6
typescript                      4.1.5

Same issue here.

Can confirm that it is also broken when coming from 11.2.5 with npm 7 and node 14.16. I get the exact same error message.
ngular CLI: 11.2.4
Node: 14.16.0
OS: linux x64

Angular: 11.2.5
... animations, common, compiler, compiler-cli, core, forms
... language-service, localize, platform-browser
... platform-browser-dynamic, platform-server, router
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1102.4
@angular-devkit/build-angular   0.1102.4
@angular-devkit/core            11.2.4
@angular-devkit/schematics      11.2.4
@angular/cdk                    11.2.4
@angular/cli                    11.2.4
@angular/material               11.2.4
@nguniversal/builders           11.2.1
@nguniversal/express-engine     11.2.1
@schematics/angular             11.2.4
@schematics/update              0.1102.4
rxjs                            6.6.6
typescript                      4.1.3

This is unfortunately an issue with npm 7 and its new peer dependency resolution process.
<EMAIL_ADDRESS>satisfies the SemVer range of<EMAIL_ADDRESS>|| ^11.2.0-next" but npm 7 is treating it as a failure.

So maybe passing --legacy-peer-deps to npm on npm 7 could help? But I don't see an option to tell ng update to pass any flags to npm.

That, unfortunately, would most likely require every invocation of npm to use the flag.
As a temporary solution, there are two potential options: 1. use npm 6; 2. add an .npmrc file to the project that sets the --legacy-peer-deps flag.  Neither of which are an ideal long-term solution as the new npm behavior is intended to catch legitimate package version errors.

Let's continue tracking this issue in npm/cli#3152 since this is not actionable by Angular.


Bug fix

The redeploy at https://harold-ohana-web.herokuapp.com/ had a bug that was resolved by adding an extra set of parentheses at lines 16 and 22.
I'm not sure if this is necessary (or even desirable) for other redeploys.

Coverage remained the same at 99.366% when pulling cd0bed9d434b3d1c7c1603d7338d35a8761bf109 on HumanTaxa:master into 46170b98dc8e32663bc2fb957457510eaa67093a on codeforamerica:master.

Thanks. I tried that command locally on my Mac, in Docker, and in a bash terminal in Heroku, and all worked for me.


Code blocks get truncated when nested inside the description list.

When you place a code block inside the description list it is not being split between pages correctly. The first part of the code block is being rendered properly as if it was to be continued on the next page, however, nothing gets rendered on the following page.
SQL query::
+
[source, sql]
----
...
code
more code
more code
more code
more code
more code
more code
more code
more code
even more code
----

==== Schedulers

Incorrect split

When code block is detached from the description list it is properly split.
SQL query::

[source, sql]
----
code
more code
more code
more code
more code
more code
more code
more code
more code
more code
even more code
...
----

Correct split

I'm not able to reproduce this. Please specify which version of Asciidoctor PDF you are using. Also, please provide a complete example that demonstrates the problem. In other words, the AsciiDoc sample itself needs to be enough to reproduce the problem without modification.

I'm using Asciidoctor PDF 2.3.9 that is fetched by Asciidoc IntelliJ plugin 0.39.9.
I think the key to reproduce the problem is to make the description list [horizontal].

A horizontal dlist is essentially a two-column table (with no borders). In Asciidoctor PDF, a table cell cannot exceed the height of a page. Therefore, a horizontal dlist has the same limitation of tables in Asciidoctor PDF. (I'll update the docs to emphasize this fact). There's nothing we can do about this limitation.


Can't deactivate some Rules (ex. AL1027)

Describe the bug
Rule AL1027 doesn't deactivate after custom ruleset setting.
To Reproduce
I've attached a tiny example with 3 Rules deactivated but only AL1027 is steel showing the info warning
RulesetIssue.zip
settings.json
{
    // Sets the list of paths to code analyzers to use for performing code analysis.
    "al.codeAnalyzers": [
        "${AppSourceCop}",
        "${CodeCop}",
        "${UICop}",
        "${PerTenantExtensionCop}"
    ],
    // Specifies whether the code analysis should be performed for all source files in the opened folder
    "al.enableCodeAnalysis": true,
    // Sets the path to the file containing the customized rules to use when running code analysis.
    "al.ruleSetPath": "./rules/my.ruleset.json",
}

my.ruleset.json
{
    "name": "My Ruleset",
    "description": "These rules must be respected by all the AL code written within the company.",
    "rules": [
        {
            "id": "AA0005",
            "action": "Hidden",
            "justification": "This is OK -> Avoid AA0005 warning notification for BEGIN .. END only with compound statements"
        },
        {
            "id": "AL1027",
            "action": "Hidden",
            "justification": "This is KO -> Avoid AL1027 info notification for PermissionSet definition matches"
        },
        {
            "id": "AS0015",
            "action": "Hidden",
            "justification": "This is OK -> We don't need translations at the moment."
        }
    ]
}

Expected behavior
All rules must be deactivated or reduce their level of importance in the same way, without exceptions.
Screenshots

Versions:

AL Language: 2.0.48254
Business Central: ES Dynamics NAV 13.0 (24630) OnPremise

I'm Sorry to hear this... the AL1027 rule messages are a litle anoying...

It's a pity.
Modify AL rule "Error" to "Ignore" is bad idea.
But why not modify ""Warning" to "Error" or "Info" to "Hidden"?

Thanks, it does make sense. We will consider the Warning -> Error and Info -> Hidden as a suggestion.

I am closing this issue because it appears that it has been resolved as part of analysis framework changes in the 2023 Wave 1 timeframe. Please open a new issue if you believe this has not been resolved and reference the current issue

Hi @SBalslev,
due to Microsoft's Universal Code Initiative, we are trying to use Scope=Cloud in all of our new BC projects and trying to follow ALL official Best Practices by enabling all CodeCops...
BUT, when we try to use for compliance reasons some Spanish SII [OnPrem] functions to verify the "VAT Registration Number", we get Compiler error AL0296
We can't avoid that Error either by using #pragma or by a ruleset.json file for changing from Error to Warning, Info or Hidden.
ruleset.json
{
    "name": "Name",
    "description": "Description",
    "rules": [
        {
            "id": "AL0296",
            "action": "Hidden",
            "justification": "Avoid AL0296 The application object or method '{0}' has scope '{1}' and cannot be used for '{2}' development."
        }
    ]
}

Any idea how we can follow MS policies in these cases?


make migration callbacks optionally asynchronous

Do you think it's painful to change them such they're always async? Or does that get nasty for a non-async user?

Do you think it's painful to change them such they're always async? Or does that get nasty for a non-async user?

I think it would be too painful, but it's an option. Here are some options:

The user defines per context or per region or per sub-region behavior via a new flag. This flag tells libvfio-user whether it must send a response after the user's region callabck has returned or do so when the user explicitly notifies completion via a new callback.
The user's region callback returns, along with the error code, whether or not libvfio-user must immediatelly send a response.
Make it always asynchronous, as @jlevon suggested.

Thoughts?

Irrespectively of the high level behavior we choose, we need to move all variables into a dynamically-allocated struct (the request context) so that we can send the response at a later time. We also need to consider whether we need locks for writing to the socket, since the user of thel library might be completing requests from multiple threads. We also need to consider what the behavior of the library should be when there is a pending request and another, potentially conflicting request arrives: e.g. what happens when the device is being set to a particular migration state and in the mean time a write to a BAR arrives?
I think for now we can allow only the migration region to be accessesed asynchronously. When this happens we can set a flag that makes process_request immediatelly return that there is no work to be done, without checking the socket. This can only work with LIBVFIO_USER_FLAG_ATTACH_NB.

Pausing for now, WIP stashed in https://github.com/nutanix/libvfio-user/tree/tmakatos/issues/296.

In https://review.spdk.io/gerrit/c/spdk/spdk/+/7617 it became evident that highly asynchronous applications such as SPDK will have a very hard time supporting live migration. We should re-prioritize implementing this.

The simplest way to make the migration state callback asynchronous is for the caller to indicate e.g. via a return value whether or not the transition occured. If it did occur then it's no different from the current implementation. If it didn't occur we record the fact that there is a migration state pending (e.g. introduce a bit in the libvfio-user context) and then we don't process anything (vfu_run_ctx() should check whether the bit is set and if so it should return -1 and set errno to EAGAIN) until the user tells libvfio-user that migration state transition finished. We can implement a similar solution for making DMA map/unmap asynchronous.


Add hidden --preview / --no-preview options to ruff check

Per discussion at https://github.com/astral-sh/ruff/discussions/6998

Summary

Adds a --preview and --no-preview option to the CLI for ruff check and corresponding settings. The CLI options are hidden for now.
Available in the settings as preview-mode = true or preview-mode = false.
Does not include environment variable configuration, although we may add it in the future.
Test Plan

cargo build
Future work will build on this setting, such as toggling the mode during a test.

Should we introduce a PreviewMode enum that has Enabled and Disabled variant to make it easier to search for preview usages (instead of having to search for booleans named preview)?

I like it


Improve the poor performance of the current data-integration via SPARQL implementation

Our automatic data integration pipeline works by executing a series of SPARQL INSERT queries to create non-duplicate integrated resources: first creating records of a data source, then performing a series of updates from other data sources before inserting - not yet linked records - of the following data source. The updates avoid that new records will be created for already existing records (see https://github.com/kbrbe/beltrans-data-integration/issues/89 for the approach and https://github.com/kbrbe/beltrans-data-integration/issues/130 for the added feature of generating SPARQL queries instead of having to provide many similar hard coded queries).
Additionally we added the feature of manual curation lists where we specify in one row that a certain person or book has certain identifiers in the data sources. For example person1, name1, kbrID1, bnfID33, ntaID1337.
Based on provenance information encoded in the RDF representation of the correlation list, the SPARQL INSERT queries use a FILTER statement to exclude correlation list entities from the automatic data integration (see https://github.com/kbrbe/beltrans-data-integration/issues/176)
However, the performance of the current implementation of this approach is not good and has gotten worse.
The series of update queries is implemented as separate SPARQL queries per identifier that additionally are executed several times, i.e. a query trying to find links via ISNI, a query trying to find links with VIAF, etc.

This led to a data integration time of roughly 30 minutes.
Adding the additional FILTER NOT EXSTS for the exclusion of the correlation list records, resulted in a data integration time of roughly 3 hours.

This slow runtime is not only annoying, but also slows down our work significantly. During the development of new features we have to schedule several hours for trial and error because of the slow pipeline.
The slow runtime can most likely be improved by rewriting the SPARQL queries (that we currently generate automatically).

All identifiers are encoded in a similar fashion by using the bf:identifiedBy property of the BIBFRAME ontology. Hence a generic triple pattern can probably be used to identify links between records in a single SPARQL query, thus avoiding a query per identifier which also brings overhead in terms of network.
We can investigate different ways to apply the CorrelationList filter, e.g. by encoding the information differently or perform query-rewriting to avoid costly joins or materializations

The commits above added a slightly different implementation whose performance was tested and whose correctness was verified by a new integration test. In the following some more details.
New generic update query
Instead of several queries per linking-identifier, a generic query is used. This requires updates in the RML mapping files that generate instances of bf:Isni.
Semantically, the following two statements provide the same information:
ex:person1 bf:identifiedBy [
  a bf:Isni ;
  rdf:value "0000000508565337" .
] .

ex:person1 bf:identifiedBy [
  a bf:Identifier ;
  rdfs:label "KBR" ;
  rdf:value "21621349" .
] .

However, it is differently encoded. The BIBFRAME ontology already provides a class for ISNI that avoids the need of a specific label. Yet for our data integration where we want a single query, we have to treat those two patterns. Only providing SPARQL variables for the type of identifier and identifierValue creates cross-products which we observed in the test data. If we have the KBR identifier "123" and an ISNI identifier "123", those would be mixed up when not differentiating also between the label of the identifier. In reality it does likely not occur that one identifier is used also by another provider, but this should also not be assumed! In the end these are just local identifiers and not URIs.
To fix this, we also have to add a rdfs:label "ISNI" property for instances of bf:Isni. Like this we can also have a SPARQL variable ?typeLabel to differentiate the type and only identify correct links between the same types of identifier. This was necessary as having an OPTIONAL triple pattern for the label did not avoid the cross products.
https://github.com/kbrbe/beltrans-data-integration/blob/a1084ae23307a558d1e6ad067107b8f6252cb91c/tools/sparql/query_builder.py#L1176-L1190
Correlation list filter
We used the WHERE clause of the SPARQL INSERT query together with a SELECT to test different ways of filtering for records that are the result of a btm:CorrelationActivity.
Instead of using a FILTER NOT EXISTS, we added the following OPTIONAL triple pattern and a binding check that one the one hand created results for our current data in roughly 3 seconds (compared to more than 20 before), and on the other hand works for the test data.
https://github.com/kbrbe/beltrans-data-integration/blob/a1084ae23307a558d1e6ad067107b8f6252cb91c/tools/sparql/query_builder.py#L1195-L1197

The commit above adapted the RML mapping files such that future data processing will add explicit rdfs:label "ISNI" properties for instances of bf:Isni.
To adapt already existing data, I used the following query in the Blazegraph interface:
prefix bf: <http://id.loc.gov/ontologies/bibframe/>

INSERT {
  graph ?g { ?s rdfs:label "ISNI" . }
}
WHERE {
  graph ?g { ?s a bf:Isni . }
}

COMMIT: totalElapsed=20035ms, commitTime=1682671806964, mutationCount=138289

Using a single update query for manifestations
Apparently, so far we only use the bf:identifiedBy construct only for contributor identifiers (see query and result below).
In order to use a single update query for manifestations, we have to adapt the different means generating RDF (RML mapping rules and SPARQL queries) to use bf:identifiedBy [ a bf:Identifier ; rdfs:label "ISBN-13" ]  in addition to bibo:isbn13 (similar for ISBN-10). And for already existing data we have to perform an update via SPARQL.
prefix bf: <http://id.loc.gov/ontologies/bibframe/>

SELECT ?g ?label (count(?s) as ?number)
WHERE {
  graph ?g { ?s a bf:Identifier ;
                rdfs:label ?label . }
}
GROUP BY ?g ?label

g
label
number

http://isni-sru
BNF
23052

http://isni-sru
VIAF
25052

http://isni-sru
NTA
19336

http://isni-sru
Wikidata
9300

http://isni-sru
KBR
20915

http://kbr-linked-authorities
VIAF
2001

http://kbr-linked-authorities
KBR
49174

http://bnf-contributors
VIAF
8069

http://bnf-contributors
Wikidata
3946

http://kb-linked-authorities
VIAF
6335

http://kb-linked-authorities
Wikidata
3529

http://beltrans-contributors
VIAF
36410

http://beltrans-contributors
Wikidata
13887

http://beltrans-contributors
KBR
57156

http://beltrans-contributors
BnF
8455

http://beltrans-contributors
NTA
24727

http://beltrans-contributors
Unesco
13967

http://beltrans-contributors
UnescoLong
10809

http://unesco-linked-authorities
Unesco
14423

http://unesco-linked-authorities
UnescoLong
14423

For current data I used a SPARQL INSERT query to add ISBN10 properties according to the BIBFRAME ontology with the following query (a similar one for ISBN13)
prefix rdfs: <http://www.w3.org/2000/01/rdf-schema#>
prefix rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
prefix dcterms: <http://purl.org/dc/terms/>
prefix schema: <http://schema.org/>
prefix bibo: <http://purl.org/ontology/bibo/>
PREFIX bf: <http://id.loc.gov/ontologies/bibframe/>

INSERT {
  graph ?g {
    ?manifestation bf:identifiedBy ?publicationISBN10Entity .

    ?publicationISBN10Entity a bf:Identifier ;
                             rdfs:label "ISBN-10" ;
                             rdf:value ?isbn10 .
  }
}
WHERE {

  graph ?g {
    ?manifestation a schema:CreativeWork ;
                   dcterms:identifier ?identifier ;
                   bibo:isbn10 ?isbn10 .
  }

  BIND(iri(concat("http://kbr.be/id/data/identifier_isbn10_", ?identifier, "_", ?isbn10)) AS ?publicationISBN10Entity)
}

This issue has been addressed. The data integration time is now around 40 minutes which is acceptable. The nice-to-have refactoring of the data integration script was moved to a different issue.


Move to ibmcloud CLI

The bx CLI is being renamed to ibmcloud, update the examples in this repo:
https://console.bluemix.net/docs/cli/index.html#overview

updated in PR #6


Add cpp sample project

The example project to use with cmake.

minor fix in cmake usage tutorial

:+1:


Improve Memory Consumption

This is an incremental improvement as we work through improvements that will allow larger caches. The crux of this improvement was to store the Environment in an Arc which allows us to share the same memory location between threads.
Also up'd some deps and applied some clippy suggestions.

Changed the caching strategy to be per env. This should allow us to keep a much larger cache and not waste memory on unused envs per sha.
Also removed the template all route as it will pollute the cache. If this behaviour is required we can replicate with a call to the get envs route and repeated calls to template.


[IE11] [Crash] "Access is denied" when try to read frameElement from getFrameElement function

It's IE11 specific issue.
Steps to reproduce:

You need App1 with iFrame
set document.domain of app1 to domain.com
You need App2 that load into App1 iframe
set document.domain of app2 to sub.domain.com
Usage of function getFrameElement with crash with error: Access is denied

Because IE allow access top window ONLY in case when domain fully matched!
Impact:
Very popular UI library antd - just stops work with error that very hard to find and reproduce.

PR with the fix is included in version 1.0.14
The release is available on https://www.npmjs.com/package/compute-scroll-into-view/v/1.0.14


Multiple Ramp Capacities

Is your feature request related to a problem? Please describe.
I'm using only icons in my polybar config. I'd like to be able to use different ramp-capacities when charging instead of animation-charging. In a way that it could show the current battery status without animation.
Why does polybar need this feature?
I think it would be helpful for those with minimal configs who dislike animations in the bar.
Describe the solution you'd like
I'd like to be able to set ramp-capacity and ramp-capacity-charging. So I could use one set of icons for charging and another for discharging.
Describe alternatives you've considered
I've considered using a custom script module to set it, but I think it would be great to have this option by default.

This will be possible, once we implement #1752

Awesome!


dialect_snowflake_test.go uses an unsupported type

In https://github.com/go-gorp/gorp/blob/main/dialect_snowflake_test.go, there is a test case for checking the SQL type of []uint8{1}, except bytea is not a supported type by Snowflake: https://docs.snowflake.com/en/sql-reference/intro-summary-data-types.
If updated to binary instead of bytea, this test will pass.

Pending PR: https://github.com/go-gorp/gorp/pull/452


Add ABI generation to CI

If you made a contract change, make sure to complete the checklist below before merging it in master.
Refer to our documentation for more details about contract security best practices.
Contract change checklist:

[ ] Code reviewed by 2 reviewers.
[ ] Copy & paste code review security checklist below this checklist.
[ ] Unit tests pass
[ ] Slither tests pass with no warning
[ ] Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)

Merging this in, since it works with the branch now. Gonna test it with release tags once on master


openspg有发相关论文吗 想看一看

No description provided.

@article{liang2024kag,
title={KAG: Boosting LLMs in Professional Domains via Knowledge Augmented Generation},
author={Liang, Lei and Sun, Mengshu and Gui, Zhengke and Zhu, Zhongshu and Jiang, Zhouyu and Zhong, Ling and Qu, Yuan and Zhao, Peilong and Bo, Zhongpu and Yang, Jin and others},
journal={arXiv preprint arXiv:2409.13731},
year={2024}
}
@article{yikgfabric,
title={KGFabric: A Scalable Knowledge Graph Warehouse for Enterprise Data Interconnection},
author={Yi, Peng and Liang, Lei and Da Zhang, Yong Chen and Zhu, Jinye and Liu, Xiangyu and Tang, Kun and Chen, Jialin and Lin, Hao and Qiu, Leijie and Zhou, Jun}
}

https://github.com/orgs/OpenSPG/discussions


REL300 doc and image updates in response to feedback

Issue #, if available:
Description of changes:

Move RDS above servers in diagram
Make it clearer what the student is viewing when they look at the DB table output
Move link above the image
Limit size of displayed image
Add note to prevent race condition when AZ failure is run after RDS failure

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

Shouldn't the comment say "Move RDS below servers in diagram"

And I would rephrase the descriptions of the IP records to say something like "Website URL access requests are shown here for traffic against the website. These include IPs of browser traffic as well as IPs of load balancer health checks".


merge from arlac77/npm-package-template

.travis.yml

chore(travis): remove node versions 12.8.1
chore(travis): add node versions -10,-11,-12,12.9.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Refactor developing-dapr documents for consistency

Description
Address consistency issues mentioned in #2529:

Consolidate all dapr development environment setup instructions in setup-dapr-development-env.md and remove duplication from developing-dapr.md.
Add golangci-lint as a prerequisite for make lint.
Add linux build-essentials package as requirement for make & gcc.
Remove incorrect Windows instruction to set GOPATH=C:\go. GOPATH should default to %USERPROFILE%\go as indicated by golang install instructions.
Add links to dapr docs for debugging dapr in Kubernetes.

A separate PR will address the remaining concerns regarding the use of mingw and WSL2 for Windows dev environments.
Issue reference
Addresses most concerns in #2529, requires additional PR to close on mingw32-make issues on Windows.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@CodeMonkeyLeet Please, comment on https://github.com/dapr/dapr/issues/2529 so it can be assigned to you.


Provide only PRs which are mergeable.

It would be useful to my pipeline to only get a version from check if the PR in question is ready to merge.

Closing this in favor of the PR.


Hello, is this code complete and capable of training?

Hello, is this code complete and capable of training?

@dgo2dance yes it is


MOV conversion creates an absolutely gargantuan file that's unreadable on Mac

Hi using the docs I ran this command successfully
backgroundremover -i "/path/to/video.mp4" -tv -o "output.mov"
I had an input file created from an
iPhone, 15MB, 1440x1440, Linear PCM, Timed Metadata, HEVC. 7 seconds, 15MB
The output file when I ran the local copy of this was
1.22GB (that's GIGABYTES, nearly 100x the original size), 1440x1440, MPEG-A AAC, Animation, encoding software Lavf59.27.100
Most crucially, when I open it in QuickTime I get this error "This file contains some media that isn’t compatible with QuickTime Player."
What can I do to get this working so a) it's readable on a Mac and b) it's hopefully approximately the same size 🙂

What version of ffmpeg are you using?

Also, have you tried opening the file on https://mpv.io/ their player is what I use on ubuntu and it seems to work just fine.

ffmpeg v 5 1.2.
Even if it's readable, it's 100x the original size. Maybe the file is raw uncompressed frames (so the compression step failed?)

I have no tried with ffmpeg 5 yet, only 4.4 I presume it would still work with 5+ though.  Can you try opening the file with the mpv player?

And I reference specifically this comment when I was building it https://superuser.com/a/1648130/535078

@nadermx Your version of ffmpeg is too old and does not support shortest=1. Use ffmpeg -i video.mp4 -i mask.mp4 -filter_complex "[1][0]scale2ref[mask][main];[main][mask]alphamerge" -c:v qtrle output.mov or upgrade your ffmpeg. What I meant in my last comment is that I tried 4.2.4 without shortest=1, and then viewed output.mov with the player named mpv and it had a transparent background. Best recommendation is to upgrade your ffmpeg. –
llogan
May 11, 2021 at 21:25

I think this is expected.  It might not be playable just because of the sheer bandwidth required. You can try converting it to WebM, that keeps the transparency.

I think it’d be useful to use the original codec to recompress the file.
It’s how other tools work I’ve used anyway.
On Mon, 13 Feb 2023 at 16:45, Rena Kunisaki @.***>
wrote:

I think this is expected.  It might not be playable just because of the sheer
bandwidth required. You can try converting it to WebM, that keeps the
transparency.
—
Reply to this email directly, view it on GitHub
https://github.com/nadermx/backgroundremover/issues/49#issuecomment-1428271498,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABD62PQOEBBJVZJZ4GZOSTWXJQLTANCNFSM6AAAAAAUIUO5MI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Pull requests are accepted
On Mon, Feb 13, 2023 at 10:50 AM Julian Harris @.***>
wrote:

I think it’d be useful to use the original codec to recompress the file.
It’s how other tools work I’ve used anyway.
On Mon, 13 Feb 2023 at 16:45, Rena Kunisaki @.***>
wrote:

I think this is expected.  It might not be playable just because of the sheer
bandwidth required. You can try converting it to WebM, that keeps the
transparency.
—
Reply to this email directly, view it on GitHub
<
https://github.com/nadermx/backgroundremover/issues/49#issuecomment-1428271498
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AABD62PQOEBBJVZJZ4GZOSTWXJQLTANCNFSM6AAAAAAUIUO5MI
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/nadermx/backgroundremover/issues/49#issuecomment-1428277816,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABYACXFG3YRW37PGKU5ZKVLWXJQ33ANCNFSM6AAAAAAUIUO5MI
.
You are receiving this because you were mentioned.Message ID:
@.***>

I tried converting it to VP9 with transparancy but it didn't work.

So just to clarify, you want me to find a way to compress the transparent mov?

That’d be amazing. I was thinking of doing it at some point but I’m on
other projects at the moment so it might be a few months away.
On Fri, 5 May 2023 at 21:06, Johnathan Nader @.***>
wrote:

So just to clarify, you want me to find a way to compress the transparent
mov?
—
Reply to this email directly, view it on GitHub
https://github.com/nadermx/backgroundremover/issues/49#issuecomment-1536720739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABD62PXVUJRSQVW3NK2FTTXEVMVFANCNFSM6AAAAAAUIUO5MI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Maybe moviepy can do it?
On Fri, 5 May 2023 at 21:22, Julian Harris @.***> wrote:

That’d be amazing. I was thinking of doing it at some point but I’m on
other projects at the moment so it might be a few months away.
On Fri, 5 May 2023 at 21:06, Johnathan Nader @.***>
wrote:

So just to clarify, you want me to find a way to compress the transparent
mov?
—
Reply to this email directly, view it on GitHub
https://github.com/nadermx/backgroundremover/issues/49#issuecomment-1536720739,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABD62PXVUJRSQVW3NK2FTTXEVMVFANCNFSM6AAAAAAUIUO5MI
.
You are receiving this because you authored the thread.Message ID:
@.***>


[20.03] chromium: 85.0.4183.121 -> 86.0.4240.75

Motivation for this change
Backport of #99975.
cc @Frostman for testing.
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.

nixosTests.chromium passed (x86_64)


VMware: SystemVMs not cleared after forcing them to stop

ISSUE TYPE

Bug Report

COMPONENT NAME

CLOUDSTACK VERSION

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

Occupy a host with a few VMs and make sure both system vms are there as well
MariaDB [cloud]> select name,state,removed from vm_instance where type = 'SecondaryStorageVm' or type ='ConsoleProxy';
+---------+-----------+---------------------+
| name    | state     | removed             |
+---------+-----------+---------------------+
| s-1-VM  | Destroyed | 2019-11-21 09:37:59 |
| v-2-VM  | Destroyed | 2019-11-21 09:48:16 |
| s-28-VM | Destroyed | 2019-11-21 09:38:58 |
| s-29-VM | Destroyed | 2019-11-21 09:45:29 |
| s-30-VM | Destroyed | 2019-11-21 09:46:29 |
| s-31-VM | Destroyed | 2019-11-21 09:47:29 |
| s-32-VM | Destroyed | 2019-11-21 11:50:29 |
| v-33-VM | Destroyed | 2019-11-21 11:44:56 |
| v-39-VM | Destroyed | 2019-11-21 11:45:59 |
| v-40-VM | Destroyed | 2019-11-21 11:50:29 |
| v-41-VM | Running   | NULL                |
| s-42-VM | Destroyed | 2019-11-21 11:51:29 |
| s-43-VM | Destroyed | 2019-11-21 11:52:29 |
| s-44-VM | Destroyed | 2019-11-21 11:53:29 |
| s-45-VM | Destroyed | 2019-11-21 11:54:29 |
| s-46-VM | Destroyed | 2019-11-21 11:55:29 |
| s-47-VM | Destroyed | 2019-11-21 11:56:29 |
| s-48-VM | Running   | NULL                |
+---------+-----------+---------------------+
18 rows in set (0.00 sec)

notice how s-29, s-32, 40 are still part of the list but according to cloudstack DB they are removed
STEPS TO REPRODUCE

Make sure the host is the last one in the cluster and set it to maintenance

Observe that all VMs are shut down

Cancel maintenance and observe new system vms will be created

Login to vCenter and see that old system vms are not destroyed

EXPECTED RESULTS

ACTUAL RESULTS

@borisstoyanov do still see this with latest master?

I can confirm this still happens in 4.15. I wonder if it is on purpose as we are talking about the last host in the zone, might it be thought to be usefull to keep the images of the svms around?
i think we can call this valid and setting it to 4.14.1 for now (we might need to postpone ...

I can confirm this still happens in 4.15. I wonder if it is on purpose as we are talking about the last host in the zone, might it be thought to be usefull to keep the images of the svms around?
i think we can call this valid and setting it to 4.14.1 for now (we might need to postpone ...

Edge case, not critical issue moved to next milestone.

Fixed in https://github.com/apache/cloudstack/pull/4644


[Bug] (1.0.7) - Sprite Sheet Node Skips First Animation Frame

When generating a sprite sheet with the Sheet node I noticed the sequence of the sprites depends on the current frame of the animation timeline. If the timeline is set to the first frame, the first cell of the generated sprite sheet will be blank. If the timeline is set to any other frame the sprite sheet will start with that frame's sprite then loop through all the other frames correctly.
Additionally, I noticed that the shadows were not being included in the sprite sheet for some reason. Padding and other modifications like the dithering overlay seem to make it in so it's not just pulling the original sampled sprites. Not sure what is going on there.
Original Sprites:

Sheet Generated By Starting Animation At Frame 1:

Exported File Starting At Frame 7 (of 8):

Pixel Composer:

Thanks for reporting. I did fix the first frame not rendering. For the shadow problem I'm not sure what happened here as I can't replicate it on my device. Can you show me the full node or save file?

Thanks for reporting. I did fix the first frame not rendering. For the shadow problem I'm not sure what happened here as I can't replicate it on my device. Can you show me the full node or save file?

Sure! Here's a screenshot of the full node setup:

And the save file (I had to convert it to a .txt to upload, but you should be able to just change it back to a .pxc):
RunRainbow.txt

I check your save and found no shadow missing problem. Maybe the first frame fix also fix this one? I'm not sure you might want to check again once I release 1.0.9

Maybe it has to do with the Spliced Sprite node somehow? I'll check when the new release is out 👍

Looks like the 1.0.9 update fixed these issues. All sprites and shadows are present in the correct order. Nice work!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/399 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 2960871:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-02-07-03-47-37 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-02-07-03-47-37

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add diagnostic to move 'async' and 'throws' in 'ArrowExpr' in front of '->'

After parsing the -> for an ArrowExpr, eat trailing throws or async and include them as unexpected nodes after the ->. This is safe because the expression after -> cannot start with throws or async.
This changes the syntax nodes to allow unexpected tokens after the last expected token. Previously @rintaro had concerns about unexpected nodes at the end of a node because this make it even more ambiguous where the unexpected tokens should be placed, but it’s super convenient for this diagnostic, so allowed it again.

I'm fine with adding UnexpectedAfterLastChild

https://github.com/apple/swift/pull/61482
@swift-ci Please test

https://github.com/apple/swift/pull/61482
@swift-ci Please test

https://github.com/apple/swift/pull/61482
@swift-ci Please test macOS


Vanilla Example not working

I am in Chrome (Ubuntu), and the Vanilla Example in the web page https://foliotek.github.io/Croppie/ doesn't work.
This is the result, after clicking on Resut button. The image is totally transparent.

I had a similar problem working on my app, so I would understand why it happen.
I tried to clear site data, hard reloading and incognito page, but the problem persist.
Chrome version: Version 63.0.3239.108 (Official Build) (64-bit), Ubuntu
It works on Mozilla Firefox.

I solved this issue updating chrome version.

@fabio-C , would you mind closing this issue if it's no longer relevant?


terminal completions are not indicated to screen reader users

enable terminal.integrated.suggest.enabled and (optional) terminal.integrated.suggest.enableExtensionCompletions
use a screen reader and enter screen reader mode
type in the terminal, for example, c
completions widget is visible, but it's not announced to screen reader users

Completions in the editor aren't announced to screen reader users, but when they're invoked via ctrl+space, they are and say the suggestion then x of count.
cc @jooyoungseo

We'll need to fix this first https://github.com/microsoft/vscode/issues/233921

It's verbally announced on Windows when tested with the latest JAWS. However, the terminal suggestion is not presented in a braille display.

@meganrogge my guess is this is either the terminal's announcing conflicting with it or the simple suggest widget missing some a11y stuff.


Add scram-sha-256 to UnsupportedAuthenticationSchemeException and fix…

… typo in filename

Codecov Report

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

@@           Coverage Diff           @@
##           master     #267   +/-   ##
=======================================
  Coverage   84.75%   84.75%
=======================================
  Files         110      110
  Lines        1286     1286
  Branches       25       25
=======================================
  Hits         1090     1090
  Misses        196      196

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 03c9be6...f703a73. Read the comment docs.


Added tag as a Flag option to fsoc solution fork

Description
This change allows a tag to be passed as a flag when running fsoc solution fork.   Allowing a tagged version other than stable to be forked.
Type of Change

Bug Fix

Checklist

[ x] I have read the contributing guidelines
[ ] Existing issues have been referenced (where applicable)
[ x] I have verified this change is not present in other open pull requests
[ x] Functionality is documented
[ x] All code style checks pass
[x ] New code contribution is covered by automated tests
[ x] All new and existing tests pass

Codecov Report

Merging #130 (1266b29) into main (62d908f) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #130   +/-   ##
=======================================
  Coverage   32.34%   32.34%
=======================================
  Files          27       27
  Lines        3568     3568
=======================================
  Hits         1154     1154
  Misses       2323     2323
  Partials       91       91


two-digit numbers disappear

When i use SyntaxHighlighter 3.0.83 on pre-elements where the only content is a two-digit number, the output is a single   and nothing else. One or three digits works just fine, but not two digits. I thought it was a brush problem (using JScript brush), but get the same behavior with the plain brush.
Example HTML:

42

Settings in use:
SyntaxHighlighter.defaults['tab-size'] = 4;
SyntaxHighlighter.defaults['gutter'] = false;
SyntaxHighlighter.defaults['toolbar'] = false;
This is the generated HTML:

I've submitted a fix #467 for this.


ability to open admin windows in iframes inside windows

- Do you want to request a feature or report a bug?
bug
- What is the current behavior?
ability to duplicate the admin iframe inside of the admin screen (see screenshots)
- If the current behavior is a bug, please provide the steps to reproduce.
begin a new blog post, click in the empty preview on the headline-card for the post before doing anything else.
- What is the expected behavior?
unable to load the admin in an iframe already inside the admin
- Please mention your node.js, and operating system version.
this was observed on the site running from a static server.  It was built using Node.js v 6.9.1 on ubuntu14.04 (netlify buildbots).

screenshots:


Fix version constraint on queue lib

Currently locking the bundle on ^v0.0.1, pulls versions of the queue library that it doesn't support (v0.1.0).
This PR fixes that. When merged we need to tag a version v0.0.5.

lgtm

Minor correction: the library constraint was ">=0.0.1", which indeed installs v0.1.0. The caret notation ^0.0.1` would have also worked, it's basically the same as this change.
lgtm

@pkruithof what do you mean exactly? This is the current constraint in version 0.0.1 of this bundle (not the lib):
https://github.com/treehouselabs/queue-bundle/blob/574e95f9c99afd3476649a6e68492a4f790f855f/composer.json#L18

I meant that this would also fix your problem:
"treehouselabs/queue": "^v0.0.4"

It's the same as the diff in this PR. I was reacting on your comment that the current constraint was ^v0.0.1 and pulled in v0.1.0.


terraform deploy times out using the chart

I am trying to deploy the chart using the terraform plan below (I commented out values to test it completely default). However, the chart never successfully deploys, continuously saying helm_release.traefik: Still creating... [XmXXs elapsed] until it finally times out. I can see a singular pod created as per the defaults and the pod is a healthy 1/1 and the logs simply say time="2020-05-11T02:19:41Z" level=info msg="Configuration loaded from flags." with no errors (kubectl decribe output posted below). If I just deploy the helm chart as per the README the deploy completes successfully and the pods come up just the same. Is there something I am missing for the deploy to complete successfully?
traefik.tf
resource "kubernetes_namespace" "traefik" {
  metadata {
    name = "traefik-system"
    // We don't care about these, ignoring them with lifecycle
    annotations = {
      "cattle.io/status" = "placeholder"
      "lifecycle.cattle.io/create.namespace-auth" = "placeholder"
    }
  }
  lifecycle {
    ignore_changes = [
      metadata[0].annotations["cattle.io/status"],
      metadata[0].annotations["lifecycle.cattle.io/create.namespace-auth"]
    ]
  }
}

resource "helm_release" "traefik" {
  name = "traefik"
  namespace = kubernetes_namespace.traefik.metadata[0].name
  repository = "https://containous.github.io/traefik-helm-chart"
  chart = "traefik"
//  values = [
//    file("traefik-values.yaml")
//  ]
}

 kubectl describe pod/traefik-566477c64f-kcgft -n traefik-system
Name:         traefik-566477c64f-kcgft
Namespace:    traefik-system
Priority:     0
Node:         cc-kube-wkr02/<IP_ADDRESS>
Start Time:   Sun, 10 May 2020 22:19:40 -0400
Labels:       app.kubernetes.io/instance=traefik
              app.kubernetes.io/managed-by=Helm
              app.kubernetes.io/name=traefik
              helm.sh/chart=traefik-8.1.4
              pod-template-hash=566477c64f
Annotations:  cni.projectcalico.org/podIP: <IP_ADDRESS>/32
              cni.projectcalico.org/podIPs: <IP_ADDRESS>/32
Status:       Running
IP:           <IP_ADDRESS>
IPs:
  IP:           <IP_ADDRESS>
Controlled By:  ReplicaSet/traefik-566477c64f
Containers:
  traefik:
    Container ID:  docker://8f5d9f2bd1130ef11fc3d87eee26a51bf9f62355b9c205beb401e75630162206
    Image:         traefik:2.2.1
    Image ID:      docker-pullable://traefik@sha256:ad4442a6f88cf35266542588f13ae9984aa058a55a518a87876e48c160d19ee0
    Ports:         9000/TCP, 8000/TCP, 8443/TCP
    Host Ports:    0/TCP, 0/TCP, 0/TCP
    Args:
      --global.checknewversion
      --global.sendanonymoususage
      --entryPoints.traefik.address=:9000
      --entryPoints.web.address=:8000
      --entryPoints.websecure.address=:8443
      --api.dashboard=true
      --ping=true
      --providers.kubernetescrd
    State:          Running
      Started:      Sun, 10 May 2020 22:19:41 -0400
    Ready:          True
    Restart Count:  0
    Liveness:       http-get http://:9000/ping delay=10s timeout=2s period=10s #success=1 #failure=3
    Readiness:      http-get http://:9000/ping delay=10s timeout=2s period=10s #success=1 #failure=1
    Environment:    <none>
    Mounts:
      /data from data (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from traefik-token-gjslw (ro)
Conditions:
  Type              Status
  Initialized       True
  Ready             True
  ContainersReady   True
  PodScheduled      True
Volumes:
  data:
    Type:       EmptyDir (a temporary directory that shares a pod's lifetime)
    Medium:
    SizeLimit:  <unset>
  traefik-token-gjslw:
    Type:        Secret (a volume populated by a Secret)
    SecretName:  traefik-token-gjslw
    Optional:    false
QoS Class:       BestEffort
Node-Selectors:  <none>
Tolerations:     node.kubernetes.io/not-ready:NoExecute for 300s
                 node.kubernetes.io/unreachable:NoExecute for 300s
Events:
  Type    Reason     Age        From                    Message
  ----    ------     ----       ----                    -------
  Normal  Scheduled  <unknown>  default-scheduler       Successfully assigned traefik-system/traefik-566477c64f-kcgft to cc-kube-wkr02
  Normal  Pulled     6m38s      kubelet, cc-kube-wkr02  Container image "traefik:2.2.1" already present on machine
  Normal  Created    6m38s      kubelet, cc-kube-wkr02  Created container traefik
  Normal  Started    6m38s      kubelet, cc-kube-wkr02  Started container traefik

Hello @khumps,
thanks for your interest.
As you mentioned, the pods deploy healthy so I don't see an actual issue with that helm chart. I rather suspect an issue with terraform somewhere close.
Therefore, I'll close that issue.

In my case, the traefik helm deployment timed out  because the traefik service had LoadBalance type and trying to obtain an external ip, which isn't provided by MiniKube. As a workaround, I just manually editted the service type and changed to NodePort and the deployment completed successfully immediately.

I'm facing something similar to @chance2021, but on AWS. I think it takes too long for the IP Address to get assigned.
What I don't get is why the behavior is different while using Helm CLI directly. It says deployed instead of failed, and way quicker.

For reference, I was missing the Load Balancer Controller.


[New] adding default and alias option to nvm install

For: nvm install 8.12.0 after install:
--alias=8 is equivalent to nvm alias 8 8.12.0
--default is equivalent to nvm alias default 8.12.0
implements #1867

Duplicated by #1929. It'd be great if you could undelete the branch so I can reopen the PR and keep the two in sync.

@ljharb yeah. Apparently github was flakey this morning. I can do either PR, this was the original but out of date, the other has your feedback. I'll push changes to both and you can decide.

Hm, it’s still not letting me reopen it; let’s jus abandon this one and keep the other.


feat: expose IResizeLayer in dynamo.conversion.impl

Description
#2219
Type of change
Expose IResizeLayer in dynamo.conversion.impl
Checklist:

[ ] My code follows the style guidelines of this project (You can use the linters)
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[ ] New and existing unit tests pass locally with my changes
[ ] I have added the relevant labels to my PR in so that relevant reviewers are notified

@bowang007 - actually, I realized the upsample_bilinear2d op is not in the Core ATen IR set, so it is okay to omit it. That error arises when the decorator specified for the converter is something like aten.upsample_bilinear2d instead of aten.upsample_bilinear2d.default

In that case, how can we add converter for aten.upsample_bilinear2d?

We can add this converter using the decorator @torch.ops.aten.upsample_bilinear2d.default as opposed to @torch.ops.aten.upsample_bilinear2d.

Our converters come from the schema. The decorator's key should be one of the scheme's functions.

yeah that makes sense! Thanks!


repos folder is included in the go build path

The 'repos' folder which  holds all the code to be statically analyzed are included in the build path when I run
go build ./...
Since I have been working with a fork, I have found this getting in the way often when I want to run the build all command. I have renamed the directory as '_repos' since go build automatically ignores directories that have names beginning with an underscore. I propose that this is done in this project as well. I see two long-term benefits

It becomes a problem when we clone repos that do not compile properly and this certainly is problematic if we were to compile the goreportcard code.

Right now IDEs like Gogland are indexing all the code that is located in the repos folder. This is unnecessary and time-consuming. Since the underscore prefixed names have special meaning in golang, I predict that IDEs will automatically mark such folders as 'excluded'.

@shawnps @hermanschaaf Let me know your thoughts on this? The changes are simple and they have been working fine on my code.

@tariq1890 That sounds fine to me, if you want to make a PR with the code changes I'll merge and then update the server accordingly.

Created PR #204


[Bug]: Intermittent issues seen with git actions - Discard & pull, pull from remote.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Intermittent issues seen with git actions - Discard & pull, pull from remote.
Steps To Reproduce
Issue 1:
Uncommitted changes are seen even after discarding them.
Steps:
1. Create a git connected app
2. Add a few pages (Page1, Page 2) and commit them
3. Add another page (Page 3) and discard and pull
4. Notice that page 3 still remains in the app. However it does get removed post refreshing the page
Loom: https://www.loom.com/share/c71f31bd0d424df381f4454a8b86402e?sid=a8f075f5-3f89-4535-b1fd-0a6afaf4ea16
Repo: https://github.com/laveena-en/june20-ee
I created a branch called test-bug post this issue to help with debugging via the diff.
Issue 2:
Changes aren't seen on pulling remote changes.
Steps:
1. Create a git connected app
2. Add a few pages (Page1, Page 2) and commit them
3. Delete one of the pages from the remote repo
4. Pull changes on the app
5. Notice that the changes don't reflect until the page is refreshed
Loom: https://www.loom.com/share/71c72615e7e6498fb1c18416743f0173
Repo: https://github.com/laveena-en/june20-ee
Public Sample App
No response
Environment
Production
Severity
Medium (Frustrating UX)
Issue video log
No response
Version
v1.28

Issue 3:
Though already committed, the pages show up as uncommitted, after discard and pull of a new page.
Steps:

Create a new app and connect it to git.
Note that Page1 already exists on the app.
Add a few pages (Page 2, Page3) and commit them
Now add Page4.
Click on the commit modal, and discard the changes.
Now note that though Page4 is discarded, Page 2 and Page 3 show up as changes to be commited, though they were already committed in step 3.
On committing again, the diff in repo shows up as below:

The recording of the issue:
https://github.com/appsmithorg/appsmith/assets/101863839/63f2790e-50df-4f34-bd42-1335ac1a891e

Not reproducible as part of latest version.


<EMAIL_ADDRESS>fix dotnet dependency

Per version 2.0 it uses the dotnet6 sdk: https://github.com/BartoszCichecki/LenovoLegionToolkit/commit/78fc6c1b18f760ab9a2f8641ec469740f33168f1

Closes #11565

[x] I have read the Contributing Guide.

/verify


provider validation of hostnames apparently isnt very good

while setting up my provider I goofed up one of the hostnames with an invalid character. I got this error
Feb 17 18:13:10 ams0 akash[1745601]: E[2021-02-17|18:13:10.856] applying ingress                             module=provider-cluster-kube err="Ingress.extensions \"web\" is invalid: spec.rules[0].host: Invalid value: \"8uhrdv0kdpegf231vkrbiv1qag.*.pub.vufvuptyod.hydrogen18.com\": a wildcard DNS-1123 subdomain must start with '*.', followed by a valid DNS subdomain, which must consist of lower case alphanumeric characters, '-' or '.' and end with an alphanumeric character (e.g. '*.example.com', regex used for validation is '\\*\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?(\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')" lease=akash19dnhammcj3vf9dzgmtgg7pc8ujecn5azlzcu09/2470/1/1/akash1ls5wpjvaslysse33k0yyv0fh9lgc40744klv44 service=web expose="{Port:80 ExternalPort:80 Proto:TCP Service: Global:true Hosts:[]}"
Feb 17 18:13:10 ams0 akash[1745601]: E[2021-02-17|18:13:10.856] execution error                              module=provider-cluster cmp=service cmp=deployment-manager lease=akash19dnhammcj3vf9dzgmtgg7pc8ujecn5azlzcu09/2470/1/1/akash1ls5wpjvaslysse33k0yyv0fh9lgc40744klv44 manifest-group=westcoast state=deploy-active err="Ingress.extensions \"web\" is invalid: spec.rules[0].host: Invalid value: \"8uhrdv0kdpegf231vkrbiv1qag.*.pub.vufvuptyod.hydrogen18.com\": a wildcard DNS-1123 subdomain must start with '*.', followed by a valid DNS subdomain, which must consist of lower case alphanumeric characters, '-' or '.' and end with an alphanumeric character (e.g. '*.example.com', regex used for validation is '\\*\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?(\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')"

This needs to be checked early and if it is invalid just bail

come to think of it this is a spot that is using regexes that probably shouldn't.

This only runs once on startup and not on data from the blockchain, so I think we should be OK to use a regex here.


Add MPS support

Description
Supports basic training on Mac M series.
Motivation and Context
It partially solves Mac support.
How has this been tested?
Ran a train job with lora-mps.yml from start to finish.

This is awesome!  @maximegmd

I'm not sure what's up with packaging and CI. I've run into this issue with our CI before and I think I just gave up on using it at one point.

I can always "parse" version numbers with a quick regex but it's not ideal

Thanks @maximegmd!! @winglian is MLX support still planned?

Thanks @maximegmd!! @winglian is MLX support still planned?

The issue with supporting MLX is that we will pretty much have to rewrite axolotl entirely to replace all torch specific code (all of it) to MLX, so no bitsandbytes, no xformers, no flash-attn, no transformers... At this point I think we are better off waiting for the transformers support of MPS to reach an acceptable state.

Thanks for this, @maximegmd !!
May I know how long it took to fine-tune and what are your M specs?

@maximegmd For installation, since flash-attn & deepspeed aren't supported on macOS, should I use pip install -e . instead of pip install -e '.[flash-attn,deepspeed]'?

Right just install the base without any options.
I use accelerate :
accelerate launch -m axolotl.cli.train examples/tiny-llama/lora-mps.yml

Nice, thx! I'll try it out


create EtcdConfig

It is my first try to commit EtcdConfig, hope there is some review, thanks!

Close this.


Update ch02-02-data-types.md

The previous code was able to run because the compiler inferred the type of the variable. However, this could be confusing to understand.

Hm, i'm not sure we should change it. The default type when nothing is specified is felt252, this is explained a bit earlier I believe? + this is a good example of type inference
@fricoben wdyt?

Alright, let's favor strong defined types for clarity, thanks! :)


Configuration API

It would be nice to define the configuration structure (keys, value types) in some sort of json object. That way it's easy for third-party (Python, JS, etc) tools to generate custom installer configuration files that can produce a custom netinst.

refs #50

What about adding some python unit tests that verify the config structure?

The config files are already similar to ini, maybe they could be used with something like Python's ConfigParser?

Think so too @mpmc

So I'd like ot introduce some python tests that parse and verify a config file. Thoughts?


Reports using allure

https://github.com/input-output-hk/cardano-node-tests/blob/d15e1475189194eb574509277c150917fd55502c/publi.sh
https://github.com/input-output-hk/cardano-node-tests/pull/135
https://github.com/input-output-hk/cardano-node-tests/pull/134
https://github.com/input-output-hk/cardano-node-tests/pull/125


Add chrom parameter to SNPs.merge

Fixes #102.

Codecov Report

Merging #106 into develop will increase coverage by 1.54%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #106      +/-   ##
===========================================
+ Coverage    92.76%   94.31%   +1.54%
===========================================
  Files            8        8
  Lines         1424     1424
  Branches       253      253
===========================================
+ Hits          1321     1343      +22
+ Misses          65       43      -22
  Partials        38       38

Impacted Files
Coverage Δ

src/snps/snps.py
95.29% <100.00%> (+0.98%)
:arrow_up:

src/snps/resources.py
94.73% <0.00%> (+0.40%)
:arrow_up:

src/snps/ensembl.py
70.73% <0.00%> (+39.02%)
: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 77224b3...01e8cd6. Read the comment docs.


Earlgrey: missing support for drive strength

Yosys currently doesn't support driving strength, but it is used in prim_generic_pad_wrapper.sv file.

Selecting xilinx as default prim implementation (instead of generic) doesn't give us this file and xilinx implementation doesn't have drive strength stmt.


Tutorial notebook is outdated

The official tutorial is not compatible with v1.0 release. Please refer to the issue here.

I have also encountered this problem. Do you know how to apply other data?


Introduce context provider in view component

Fix #6394

📚 Documentation preview 📚: https://volto--6395.org.readthedocs.build/

@ksuess Thanks for the PR. However, I have some doubts that I'd like to discuss this further in the Volto Team Meeting. Let me mention some of my concerns here:

I might be wrong, but we are not using React.Context anywhere in Volto (yet).
Whenever we needed something like React.Context, we used Redux (eg. recently, with the addition of the form reducer to keep track of the form changes. See https://github.com/plone/volto/pull/5721)
We introduced (as experimental) jotai in Volto some time ago. For the use case that you are trying to achieve, maybe it's better to use something like that.

I'm fine with using a different technique. And I am fine with talking about this during the next meeting.
Just a note why I don't use component state or global Redux state:
The view renders an array of blocks.
Each block checks successively for a match of a glossary term.
One block finds a term and updates the context.
The preceding blocks should NOT be re-rendered.
But a re-rendering is done if working with a state, be it component state or global state.
So I use context, update it in a block if a term matches. Every subsequent block knows about the updated context and does not check for a match of this term already found in a preceding block.

I'm fine with using a different technique. And I am fine with talking about this during the next meeting.
Just a note why I don't use component state or global Redux state:
The view renders an array of blocks. Each block checks successively for a match of a glossary term. One block finds a term and updates the context.

How do the blocks update the context? How do the blocks check for a match?

The preceding blocks should NOT be re-rendered.

Every time you do a useContext you are subscribing the component to changes to the context (like useSelect does, since it does use useContext too under the hood) Thus, the blocks will be re-rendered if you change the context value.

But a re-rendering is done if working with a state, be it component state or global state. So I use context, update it in a block if a term matches. Every subsequent block knows about the updated context and does not check for a match of this term already found in a preceding block.

Are you sure that they do not re-render?

I'm fine with using a different technique. And I am fine with talking about this during the next meeting.
Just a note why I don't use component state or global Redux state:
The view renders an array of blocks. Each block checks successively for a match of a glossary term. One block finds a term and updates the context.

How do the blocks update the context? How do the blocks check for a match?

The preceding blocks should NOT be re-rendered.

Every time you do a useContext you are subscribing the component to changes to the context (like useSelect does, since it does use useContext too under the hood) Thus, the blocks will be re-rendered if you change the context value.

But a re-rendering is done if working with a state, be it component state or global state. So I use context, update it in a block if a term matches. Every subsequent block knows about the updated context and does not check for a match of this term already found in a preceding block.

Are you sure that they do not re-render?

I understand that the official documentation seams to say that all components under the provider that call 'useContext' gets re-rendered on context changes. https://react.dev/reference/react/useContext#caveats But this is not the case. They subscribe to the context, yes, but are touched only if they really change.
I did a small example, to show that React's diffing algorithm considers the previous blocks (in fact the previous slate text leafs) that are already handled, are unchanged Virtual DOM nodes.
https://codesandbox.io/p/sandbox/only-first-occurence-view-context-provider-h5p4fv
My solution with the context provider in 'View.jsx' plus the PR mentioned in the issue (https://github.com/rohberg/volto-slate-glossary/pull/8) is fullfilling the request 'tooltips only on first occurrence on a page (over all blocks). It does it in a minimal invasive manner.
I am sorry that there seams to be no solution without a change in Volto and that I take up your time.

@ksuess as @sneridagh said let's discuss this at the Volto team meeting. Personally, I want to have the chance to test your add-on with these changes. Besides this, I am curious to see what would happen if for instance Jotai is used or even Redux.
Once I have these experiments and the full picture, I can better communicate my opinion. Right now, I am too uninformed to give a proper opinion on this change and its entitles.
What I have noticed is indeed our lack of usage of React context providers up to this point in Volto and what would this entail as opposed to using our current approaches.

@ksuess I just installed collective.glossary and your branch from volto-slate-glossary

As soon as I try to edit the frontpage I get this error and what's strange is that the error counter continues to increase as I remain on the error page.
Has this ever happened to you and if so what did you do to solve it?

New concept for volto-slate-glossary: Instead of generating the tooltip enhanced markup in each Slate leaf,
we generate all tooltip enhanced leaf texts on route change, store them with jotai atom and use the appropriate ones in the leafs.
https://github.com/rohberg/volto-slate-glossary/pull/10
The advantage of this is, that we do not write, but only read, in slate text leaves. This means we don't have to worry about re-rendering the leaves when the status changes.
BTW Global state management with Jotai is a pleasure!

Closing in favor of a solution without the need of a change request in Volto core.
https://github.com/rohberg/volto-slate-glossary/pull/10


Change how we use modals

We currently render modals as children of the components or buttons that will trigger them.
This is very limiting, as it doesn't allow a good way to manage the modals and to open another modal from within a modal.
Modals should be managed by a global service that abstracts them. Actions to open a modal will open a modal, same for closing.
If a modal is opened while another one is already displayed it will be rendered as a child modal - and clicking outside will close the latest modal.
case study: opening wallet connect modal from within the deposit modal.
Currently there's no good way to do it.
In the suggested way the modal at the root will display the deposit modal and will render a child modal - the wallet connect modal.

comment from #349 from @ashachaf
after adding 3 different tokens to bancor network v3 pools, i was unable to scroll down to view the bottom of the screen on the portfolio page. (actually, even when navigating to other pages, i was unable to scroll down). after refresh, it was working again.
https://drive.google.com/file/d/10iTYWyJZs4jimBWPy_oA6RuHA7KtbkfP/view?usp=sharing


Undefined error starting test where cursor is

Following the #49 fix, I now get an error message "Cannot read property 'replace' of undefined" when I place my cursor inside a test and try to run. If I place my cursor in the test name (the string inside the it(...)), it works, but with the cursor on any line inside the test, I get the error.
My version is 0.4.9

Yes, before we used different approach for parsing the test. We where traversing Up from the current line. now i took the test Parser from jest itself. i can now identify the Test with the Line Numbers. Which made alot stuff easier. Downside is when you do Not click in the exact Line i cant find the test.

Ah but there is a Line range for each Test, maybe i can add this Feature back.

fixed in 0.4.11, please verify

Yes, fixed. Thanks


atom: patchelf ctags binary

Motivation for this change
The fix (introduced by https://github.com/NixOS/nixpkgs/pull/16056) got removed and the issue https://github.com/NixOS/nixpkgs/issues/15528 can be reproduced.
This adds the fix back in.
Things done

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

[x] 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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg  build atom

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: atom
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

No attempt on aarch64-linux (full log)
The following builds were skipped because they don't evaluate on aarch64-linux: atom
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: atom
Partial log (click to expand)
/nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2/share/atom/resources/app/apm/node_modules/node-gyp/gyp/gyp: interpreter directive changed from "/bin/sh" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/sh"
/nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2/share/atom/resources/app/apm/node_modules/open/vendor/xdg-open: interpreter directive changed from "/bin/sh" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/sh"
/nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2/share/atom/resources/app/apm/node_modules/ajv/scripts/prepare-tests: interpreter directive changed from "/usr/bin/env sh" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/sh"
/nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2/share/atom/resources/app/apm/node_modules/ajv/scripts/travis-gh-pages: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/bash"
/nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2/bin/.atom-wrapped: interpreter directive changed from "/bin/bash" to "/nix/store/qckzjk3406va7h6s40cy9s75z2w715rq-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2...
cannot find section .dynamic
file /nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2/share/atom/atom had a PT_GNU_STACK program header, converted
file /nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2/share/atom/resources/app/apm/bin/node had a PT_GNU_STACK program header, converted
/nix/store/qv6pncykspnsg4vprdln85p8cc678sz2-atom-1.27.2


feat(golang): namespace aware types

Adds logic for emitting submodules and namespace aware types.
Creates a Module base class that is extended by both Package and Submodule. Module represents a single .go source code file that has a package xxx declaration at the top. For the root file, this is the package name. For all submodules, this is the submodule name. All instances of Module have a reference to the root package so they are able to search the entire package for a type by fqn.
All instances of GoType now have a property parent that is the instance of Module that the type is defined in. This is used to pass to any instances of GoTypeRef that the type may refer to. GoTypeRef can resolve the corresponding instance of GoType using the types parent Module as described above.
The combination of these things means that starting at the root Package, an AST is defined that mimics the structure of the .jsii assembly but with all go specific emit logic. This makes it possible to access distant nodes in the tree which is what allows for namespace resolution of foreign type references.

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

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 6038e1fac464e03205c7aab394483385b4896ded
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: a5c9801c001dd04b3324c463db140ea82967e9dd
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 46c5ca0ed8544bd0f04244940b2e0ac8c430843e
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: ccb1261b563e3b0803b932c14199a36885e3d520
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 251d7b13e92742ddc6e330489d65fad26eee5c8f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 0db0dbbdafe0af724c2788d94b566ca4fcbe346d
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 19a12b9db73ade0b686072fef1332ee78fcf45ca
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-Blkkw9bQFn8A
Commit ID: 572f1514c8cc67c075f0cf807510b02e3ea728d0
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Updated readme

Minor typos

Hi Matt!
Thank you for the PR! That's a first, hope you're having fun with the code! I merge it right away.


Add getBranch metod to JPetReader

Add getBranch method to JPetReader for EventDisplay to determinate type of file selected by user. TTree is holding branch named after class.

Ok, I found work around with getObjectFromFile getting tree and then branch name, so I close pr.


Export ICountOptions and use Generics.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #253   +/-   ##
=======================================
  Coverage   96.71%   96.71%
=======================================
  Files         115      115
  Lines        1004     1004
  Branches      129      129
=======================================
  Hits          971      971
  Misses         15       15
  Partials       18       18

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 634a9ff...4224066. Read the comment docs.

@RobinBuschmann Are you fine with this change ?

@santiagodoldan Thanks for contributing @santiagodoldan @bilby91 There is one thing left - see the my comment :)

@RobinBuschmann done, thanks to you ☺


fix: disable unintended parallel processing in sumcheck stage 1

Fixed #511
Root cause
We have 2 stage sumcheck

stage 1: each rayon job isolated processing
stage 2: single thread aggregate results.

So in stage 1,  a rayon job shouldn't invoke parallel rayon internally, otherwise it will invoke deadlock, since there is 0 rayon idle thread in pool, so the parallel job will hang and never generate 1st round of univariate evaluation, so the main worker thread never get enough message to processing.
This only happened when extrapolate  was invoked on the situation when virtual polynomial got different degree monomial terms. In other word, it tend to happened on opcodes with different degree of monomial terms in a constrain.
The fix is we need to have stage 1 strictly serialized and do not invoke rayon parallel internally.
Another minor optimisation
Previously main thread also be one of worker thread, thus it also exchange data via channel which might incur a unnecessary cost. This PR also remove this cost and exchange data locally.
Testing

cherry-pick commit into PR #368 and doen't hang anymore on ceno-server, previously with command RAYON_NUM_THREADS=64 RUST_LOG=debug cargo run --release --example fibonacci_elf -- --nocapture it's 100% reproduce
before/after the change benchmark on 2^20 instances add remain the same, which is expected since there is no constrain in add opcode which got different degree monomial terms.

Thanks for the detailed PR description.!

Hey @kunxian-xia originally I planned to merge this to master, but I believe you want to narrow down integration test debug scope in this period, so you might avoid having regular sync with master to your branch, also cherry pick might also troublesome.
how about we merge this PR to integration test feature branch instead? I'm fine with this plan :)

how about we merge this PR to integration test feature branch instead? I'm fine with this plan :)

Sounds good to me!

how about we merge this PR to integration test feature branch instead? I'm fine with this plan :)

Sounds good to me!

Ok done 👍

Could we have this in master as well?


usage with rlwrap

Came up in https://github.com/babel/babel/issues/1965.  I don't have knowledge on strictly what conditions NODE_NO_READLINE=1 or --always-readline are required.  They have been important on macosx at varying times in the past.

@willxy thanks!


Promote standalone component helpers to public API

Command
add
Description
With https://github.com/angular/angular-cli/commit/e895fc7c1012796442cb7a4a3b9b3a0ec3845b0e a few standalone helpers were moved from the CDK to the schematics package. I am wondering why they were moved to a "private" section and aren't considered public API. Was this just because standalone components were still in developer preview at that time? Now that angular 15 is going to make standalone components stable could the helpers be moved outside the private section and be treated as public API?
Describe the solution you'd like
With standalone components becoming stable these helpers should become public API as well.
Describe alternatives you've considered
No response

This is in the works related to supporting --standalone more thoroughly in our schematics. Our current implementation was a bit haphazard to meet the immediate needs of standalone at the time. As we expand our own schematics we'll want to share more of these helpers between Angular-owned repositories and iterate on the designs. Once that is in a good place, we're interested in making them more broadly available to the community.


SAK-31953 Pointer arrow now showing for mobile tools menu.

Move the :after pseudo class so that the arrow pointing at the tool menu is displayed for a mobile sized screen.  Add an overflow property so that the arrow is actually shown.

Have resolved the conflict.  Can someone please check and merge?

can somebody now merge


intentionally minimal commit, testing launchpad-staging saas deploy

for osd v4

@quintesse  PTAL: we will need to add a serviceAccount reference to any dsaas+osio deployments, for the move to osdv4. to keep consistent i'll use "dsaas-deploy" across any relevant updates (or, another name, if that's not ideal for the project).
thanks!

@mmclanerh should I merge yet? Not sure you're already finished with this or not.

@quintesse yep! this is a-ok to move forward. didn't want to merge anything in here, without the proper consent/awareness.  thanks!


New Server Setup : Error : FAILED TO JOIN: SERVER IS FULL - when the server is empty

Hi there, trying to setup a dedicated server to play with a friend and having a hard time to make it work, when trying to join the server, it claim it's full, but there is 0 player in it.

More info on my Setup :

Server is running on a Fresh virtualised instance Windows Server 2022

Client is on Windows 11

Downloaded the Dedicated Server files from Steam directly.

Added the AstroLauncher to the Server Folder and Launched as Admin

I see the Firewall rules in Windows Firewall setup.

I have setup Portforwarding on my router.

Both the Local Server Website and the servercheck.spycibot.com show the server running fine :

I have tried to add the net.AllowEncryption=False in the engine.ini on both client & server and it made no difference.

I have a password setup for the server, but regardless what I type in the password field, I get the same error.

I am kind of lost at this point, any suggestion to make it work?

The net.AllowEncryption=False is only required if Linux clients are involved.
Head on over to the Astroneer discord in the #self-host-talk channel where people can help you more easily than here.
https://discord.gg/astroneer


Update HTTP gem version dependency

The version currently in use has an incompatibility with Ruby 2.7: https://github.com/httprb/http/issues/582
The earliest version with a fix for this is 4.3.0 (as per noted here:
https://github.com/elastic/apm-agent-ruby/issues/665#issuecomment-572664916)

@wilsonsilva: More Ruby 2.7 changes 🎉

FYI, this is where I identified the error and debugged it right up to the use of HTTP gem.

Lovely, thank you! Are you using this gem in production? If so, I can put a bit more effort onto it and keep it up to date with Loqate's API.

Thank you @wilsonsilva !
Yes, we are making use of it in production.


Compare reports table summary refinement

Enrich the summary outcome.
PR checklist

[x] Ensure you have added or ran the appropriate tests for your PR.
[x] DCO signed

What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
sc-29300
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Sample output

Comparison Summary

Table
Rows
Columns

performance_weekly (-)
-
-

performance (+)
12
4

project (!)
245 (+93)
6 (+0)(changed=1)

project_block
1 (-98)
1 (+0)

session
54 (+3)
14 (+0)

user (!)
54 (+3)
8 (+0)(added=1, deleted=1)

user_block
0 (-3)
1 (+0)

user_internal
34 (+0)
1 (+0)

Tables Summary (added=1, deleted=1, schema changed=2)

performance_weekly (-)

Column
Type
Valid %
Distinct %

week (-)
TIMESTAMP_NTZ
-
-

active_users (-)
DECIMAL(18, 0)
-
-

active_projects (-)
DECIMAL(18, 0)
-
-

project_per_user (-)
DECIMAL(24, 6)
-
-

performance (+)

Column
Type
Valid %
Distinct %

week (+)
TIMESTAMP_NTZ
100.0%
100.0%

active_users (+)
DECIMAL(18, 0)
100.0%
75.0%

active_projects (+)
DECIMAL(18, 0)
100.0%
91.67%

active_project_per_active_user (+)
DECIMAL(24, 6)
100.0%
100.0%

project (!)

Column
Type
Valid %
Distinct %

project_id (!)
VARCHAR(16777216)VARCHAR
100.0% (+0.0%)
100.0% (+0.0%)

event_count
DECIMAL(18, 0)
100.0% (+0.0%)
7.76% (-4.74%)

user_count
DECIMAL(18, 0)
100.0% (+0.0%)
1.22% (-0.75%)

first_event_at
TIMESTAMP_NTZ
100.0% (+0.0%)
99.59% (-0.41%)

last_event_at
TIMESTAMP_NTZ
100.0% (+0.0%)
99.59% (-0.41%)

lifetime_in_hours
DECIMAL(9, 0)
100.0% (+0.0%)
6.94% (-4.25%)

project_block

Column
Type
Valid %
Distinct %

project_id
VARCHAR(16777216)
100.0% (+0.0%)
100.0% (+0.0%)

session

Column
Type
Valid %
Distinct %

unique_session_id
VARCHAR(32)
100.0% (+0.0%)
100.0% (+0.0%)

user_id
VARCHAR(16777216)
100.0% (+0.0%)
100.0% (+0.0%)

events_per_session
DECIMAL(18, 0)
100.0% (+0.0%)
51.85% (+2.83%)

projects_per_session
DECIMAL(18, 0)
100.0% (+0.0%)
25.93% (+4.36%)

session_started_at
TIMESTAMP_NTZ
100.0% (+0.0%)
100.0% (+0.0%)

session_ended_at
TIMESTAMP_NTZ
100.0% (+0.0%)
100.0% (+0.0%)

session_length_in_minutes
DECIMAL(24, 6)
100.0% (+0.0%)
87.04% (+0.76%)

session_started_at_day
TIMESTAMP_NTZ
100.0% (+0.0%)
59.26% (-1.53%)

session_ended_at_day
TIMESTAMP_NTZ
100.0% (+0.0%)
62.96% (+0.22%)

user_session_number
DECIMAL(18, 0)
100.0% (+0.0%)
1.85% (-0.11%)

last_session_ended_at
TIMESTAMP_NTZ
0.0% (+0.0%)
-

last_session_ended_at_day
TIMESTAMP_NTZ
0.0% (+0.0%)
-

is_first_user_session
BOOLEAN
100.0% (+0.0%)
1.85% (-0.11%)

minutes_in_between_sessions
DECIMAL(38, 0)
0.0% (+0.0%)
-

user (!)

Column
Type
Valid %
Distinct %

user_id (-)
VARCHAR(16777216)
-
-

amplitude_user_id (+)
VARCHAR(16777216)
100.0%
100.0%

user_created_at
TIMESTAMP_NTZ
100.0% (+0.0%)
100.0% (+0.0%)

first_session_at
TIMESTAMP_NTZ
100.0% (+0.0%)
100.0% (+0.0%)

last_session_at
TIMESTAMP_NTZ
100.0% (+0.0%)
100.0% (+0.0%)

total_events_per_user
DECIMAL(18, 0)
100.0% (+0.0%)
51.85% (+2.83%)

total_sessions_per_user
DECIMAL(18, 0)
100.0% (+0.0%)
1.85% (-0.11%)

average_session_length_in_minutes
DECIMAL(38, 12)
100.0% (+0.0%)
87.04% (+0.76%)

average_minutes_in_between_sessions
DECIMAL(38, 6)
0.0% (+0.0%)
-

user_block

Column
Type
Valid %
Distinct %

user_id
VARCHAR(16777216)
-
-

user_internal

Column
Type
Valid %
Distinct %

user_id
VARCHAR(16777216)
100.0% (+0.0%)
100.0% (+0.0%)

Fix #510


Add country name to /visits responses

Queries to the following endpoints return country_id and not the name of the country. It is possible to look up the country name using /countries/{.id} but most uses of the /visits API will be to display visits by a particular user and will require the country name.

/visits
/visits/{.id}

Current visit object:
{
  "id": 1,
  "user_id": 1,
  "country_id": 1
}

Desired visit object:
{
  "id": 1,
  "user": {
    "id": 1
  },
  "country": {
    "id": 1,
    "name": "Canada"
  }
}

Modifying the visit object to include nested objects for both country and user will give us the flexibility to include further properties for these entities in future. This specific issue requires the country object to include both the id and the name property.
Hints:

The country name is not stored in the visits table and so a join will be required
You will get a flat object back from your database query. You will need to decide where to convert it to a nested object; model or controller.
Existing tests might need re-work as we are changing the format of API responses.
New tests will be required to check that calls to /visits include the country name.

I will be on this

Awesome, I've assigned it to you.

Thanks for the fix in #23. This has now been merged and deployed. You can see the changes live here: https://travel.glvr.io/visits


Better logging for a flaky test

Coverage decreased (-0.03%) to 83.232% when pulling a7f3b1d914a1d8492df65591456a71cc7a3670e8 on chao-flaky into 2548d2d111b039a8e3b3cd3441e88d076f4e367b on master.


Does the Azure Firewall Count as a Next Generation Firewall?

Does the Azure Firewall Count as a Next Generation Firewall? I would prefer not to add another vendor to the mix if Azure resources can provide this functionality.

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

ID: b13aae31-93a0-b479-5876-ddf296c65580
Version Independent ID: 37924c46-6514-f9eb-de7f-ae98bfc28551
Content: Add a next generation firewall in Azure Security Center
Content Source: articles/security-center/security-center-add-next-generation-firewall.md
Service: security-center
GitHub Login: @TerryLanfear
Microsoft Alias: terrylan

@mjcarrabine Thanks for your feedback! We will investigate and update as appropriate.

@mjcarrabine Azure Firewall is currently in preview mode and it may not have all the capabilities as compared to Next-generation firewall.  Also, Azure Firewall is currently recommended not to use for production workloads. You may have to go for a next generation firewall from a Microsoft partner.

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


Improve Readme.md from ./pb

I am trying to follow the guide strictly, to allow mass adoption to use this amazing project, colleagues and I encountered a misunderstanding to newbies, so we decided to reference it! <3

Merged.
I'm glad that you find this project useful. Can you tell me how you are using it?

I am used to work with pocketbase in every area, hobby, work and also some independent works, in case, is as a "indie" project directory forum for my free opensource projects, My idea is to launch a unique Free and accessibility 100% forum <3


Made modifications to the dataloads components and dataload service

@oblodgett The package.json and package-lock.json are needed to build the node_modules.

@adamgibs not in the root of the project

@oblodgett Ah, didn't catch that. Yeah, they shouldn't be there.


Not Authorized for ecr inside eks cluster

we get HTTP 401 for kube-proxy in a eks cluster
https://602401143452.dkr.ecr.eu-west-1.amazonaws.com/v2/eks/kube-proxy/tags/list": http: non-successful response (status=401 body="Not Authorized\n")

Since ECR does not allow public images, fetching a token via the AWS SDK might be the way to go
https://docs.aws.amazon.com/sdk-for-go/api/service/ecr/#example_ECR_GetAuthorizationToken_shared00


toggling voice over should update the screen reader status bar item

On macOS Ventura, with Insider's already open, I expect to see the status bar entry when I turn VoiceOver on, but do not.
https://user-images.githubusercontent.com/29464607/204815613-a89362f5-1ada-4ff3-ac64-a2a0d9ded73c.mov
Originally posted by @meganrogge in https://github.com/microsoft/vscode/issues/167627#issuecomment-1332195806

To verify:

be on macOS ventura
toggle voice over on/off without and verify that the status bar item is updated correctly

It automatically turns screen reader mode on, but not off. It not only keeps the statusbar entry, but also QuickPick doesn't select the first item (which it only does when screen reader mode is off). So I'm not sure this is fixed. #167776 only seems to refer to the statusbar item.

I can repro in insider's, but not oss so must be a timing issue

after a discussion with Kai, we've confirmed this is the same as https://github.com/microsoft/vscode/issues/167627
/duplicate #167627

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


[Feature]: Expose OTLP Ports in all-in-one docker image

Requirement
In the all-in-one docker image Open Telemetry protocol ports (4317, 4318) need to be exposed. https://github.com/jaegertracing/jaeger/blob/main/cmd/all-in-one/Dockerfile
Problem
We have a testcontainer setup where we rely on the ports marked as exposed by the image. Eg. when running the image with "publish all" flag ie. docker run -P the OTLP ports are not published
Proposal
Let me know if this is okay to expose. Will raise PR post discussion.
Open questions
No response

Yea they should be. We don't seem to include them in the collector either

Okay. Will add it


Experimentation double bridge IRC - Mattermost

Question : Is it possible to to chat between 2 MI  passing through 2 bridges ?
Hypothesis :

installation of the 2 most advanced bridges (IRC and Mattermost ? tbc)
intent to chat between IRC and Mattermost

Tasklist :

[ ] search for the most evolved bridge after IRC
[ ] shadow user / puppet managed by IRC / Mattermost ?
[ ] IRC installation on Kubernetes cluster (?) or use an existing server ?
[ ] IRC bridge installation
[ ] Mattermost installation Kubernetes cluster (if not possible to join to beta.gouv)
[ ] Mattemost bridge installation
[ ] Analyse in case of impossibility
[ ] Description of configuration to be set in case of discussion

Definition of Done :

Discussion available => double  bridge works
Discussion not available => analyse of the causes

Here's a docker-compose version with discord and slack : https://github.com/ad2ien/matrix-double-bridge

From what I can see, it should be possible to achieve something that would cover our needs :

it's possible for the bridge to listen to specific events, create MX ghost users and send messages on their behalf

Challenges:

the puppeting, however, highly depends on the 3rd party application and can be tricky if the bridge can't create puppets on demand
the bridged end-to-end encryption is an other challenge


LibWeb: Children don't inherit parent's opacity in a grid layout

Example:

Chrome
Ladybird

The issue is no longer reproducible on build from master.


Events and delegates of C# is not supported. When it will be supported?

Events and delegates of C# is not supported. When it will be supported?

This is already tracked by https://github.com/mono/Embeddinator-4000/issues/351
The Embeddinator is a community project, and right now does not have many active developers. If this if functionality that you are interested in PRs are welcome to add it. :)


Assignment-3 - iframes

Working but twitter/insta/linkedin clickbaits only taking to their images and not to actual sites

review my assignment-iframes

One of the iFrames provided padding. All the files checked and running.

okay sir I'll make it correct


add undocumented objects/functions to flytekit api ref

Signed-off-by: Niels Bantilan<EMAIL_ADDRESS>TL;DR
This PR adds adds a bunch of undocumented objects/functions to the flytekit API reference
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin
[X] Docs

Are all requirements met?

[X] Code completed
[ ] Smoke tested
[ ] Unit tests added
[X] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
See TLDR
Tracking Issue
NA
Follow-up issue
NA

fixes https://github.com/flyteorg/flyte/issues/3271


fix: simplify collapsible panel summary item example

Simple change to reduce the amount of code displayed in the demo page for the d2l-collapsible-panel-summary-items.

Note: this reduces the code behind the scenes, but this won't reduce the amount of code displayed in the Daylight demo. I know this still conveys the general idea of changes to the panel contents being reflected in the summary, but we may want to check with Jeff which way he prefers. If we do want these changes, we probably also want to update the margin in the styles to reduce the overall height of the demo container, given that the "add-ons" are removed and there would be excess whitespace.

Clos


docs: steps for adding testing against new Kubernetes release

https://github.com/kubernetes/test-infra/pull/20559 implements the first step (updating Prow jobs).

https://github.com/kubernetes/test-infra/pull/20559 implements the first step (updating Prow jobs).

/release-note-none
We concluded the support Kubernetes 1.20 according to this checklist, so I think it is complete.
@msau42 let's merge it.

/lgtm