Resolve jshint issues

Hi,
with the error ['success'] is better written in dot notation. of jshint,
the solution is written : toastr.success('foo'); is no toastr['success']('foo');

Is anyone going to work on this or can I take this up?

I'd love to look at a PR for this if you're infestered in taking a look.

Sure will do :) I will make a PR and submit for your review.

@ShashankaNataraj - any update?

Resolved in develop with https://github.com/CodeSeven/toastr/commit/c4fba3827038a1cfefaec26bea31044e7042e010.


Protected var in Report 5692 Calculate Depreciation

Hi,
Variables that are used in request page in Report 5692 "Calculate Depreciation" should be declared as protected to be used in a report extension. For example currently, from my extension, I don't know the posting date or the document No. that user fill in the request page
In general, in my modest opinion, all variables used in any request page of any report should be declared as protected, otherwise we are very limited to add new functionality.
Regards.

Wrong repo; you want https://github.com/microsoft/AlAppExtensions/issues

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add a spark-shell smoke test to premerge and nightly

Contributes to #5704
This PR aims to catch issues like #9500. It modifies run_pyspark_from_build mostly to avoid recreating the logic of figuring out jar location etc.
Currently it may not catch this if do not have Spark 3.5.0 CI yet. But this is how it could reproduce the #9500
$ SPARK_HOME=~/dist/spark-3.1.1-bin-hadoop3.2 SPARK_SHELL_SMOKE_TEST=1 ./integration_tests/run_pyspark_from_build.sh
...
+ grep -F 'res0: Array[org.apache.spark.sql.Row] = Array([4950])'
res0: Array[org.apache.spark.sql.Row] = Array([4950])
+ echo 'SUCCESS spark-shell smoke test...'
SUCCESS spark-shell smoke test
$ echo $?
0

$ SPARK_HOME=~/dist/spark-3.5.0-bin-hadoop3 SPARK_SHELL_SMOKE_TEST=1 ./integration_tests/run_pyspark_from_build.sh
$ echo $?
1

SPARK_SHELL_SMOKE_TEST=1 \
  PYSP_TEST_spark_shuffle_manager=com.nvidia.spark.rapids.spark311.RapidsShuffleManager \
  SPARK_HOME=~/dist/spark-3.1.1-bin-hadoop3.2 \
  ./integration_tests/run_pyspark_from_build.sh
+ echo 'SUCCESS spark-shell smoke test'
SUCCESS spark-shell smoke test
$ echo $?
0

SPARK_SHELL_SMOKE_TEST=1 \
  PYSP_TEST_spark_shuffle_manager=com.nvidia.spark.rapids.spark350.RapidsShuffleManager \
  SPARK_HOME=~/dist/spark-3.5.0-bin-hadoop3 \
  ./integration_tests/run_pyspark_from_build.sh
$ echo $?
1

Signed-off-by: @gerashegalov

build

added shuffle manger conf to spark shell, PTAL

build

build


UIFlow 2 is not compatible with M5Stick-Launcher

Installing the UIFlow2 from its .bin file result in black screen, this does not happed when I burn it via M5Burner

The Python core of UIFlow runs so you can connect to the computer and make some sketches, but the application that is saved in the vfs partition isn't installed yet..
I'm thinking in a way to make it work.. I will keep the issue opened to keep thinking about a solution.

Thanks! Nothing important anyway.

Now it will be compatible with custom partitioning..
https://github.com/bmorcelli/M5Stick-Launcher/wiki/Functionalities-explained

Thanks ,just installed the new Launcher,  alas the fix does not seem to work for me, still black screen on UIFlow,
could be incompatible Cardputer or SD card... who knows , nevermind.

Hey man, yes indeed... I had it working, and idk what I've done that broke it again..
Sorry
I'll keep working on that

Its ok I really appreciate your effort, no need to proof that worked, but that video is quite helpful, as in my case I don't see the 'Installing SPIFFS' , instead goes to "Updatining FAT" maybe is some step I ve missed...

No, you have not.. It was in the testing, and I haven't changed some names yet..
You did fine, its the partition copying that isn't working anymore =(


Not getting payload

From Server am using: webpush.sendWebPush('my push messsage', subscription).then(function(resolve,reject){
console.log('chrome resolve: ', resolve);
console.log('chrome reject: ', reject);
});
result of resolve:  { statusCode: 201, statusMessage: 'Created', body: '' }
on client getting push event: {data: PushMessageData, type: "push", target: ServiceWorkerGlobalScope, currentTarget: ServiceWorkerGlobalScope, eventPhase: 2…}
i always found empty object: PushMessageData {}

Hi, uqutub.
PushMessageData is special interface to interact with payload. There are several methods for data extraction. For example PushMessageData.json().
You can find more here https://developer.mozilla.org/ru/docs/Web/API/PushMessageData

thanks. will check this out.

I'm facing the same problem. Body is empty

it is resolved,
actually the issue is in all documentation we have read to get data from event by event.data.json()
but it gives error, we have get data from event.data.text();
thanks .


Encrypt String

Write two functions in utils/password.js, one for encrypting a string and the second for decrypting it. The SECRET environment variable should be used to encrypt and decrypt the string

Assigned to @chandra1899


Bundles should be unordered

What problem does this solve or what need does it fill?
Semantically, the ordering of the components in a Bundle shouldn't matter. Components aren't inserted "in order" into our entities and Bundle(A, B) should be identical to Bundle(B, A).
Right now though, they're stored as ordered structs / tuples. This is less than ideal because:

It gives the wrong mental model.
It reduces the ability to batch bundles for command processing.
It makes implementing advanced bundle reliant features like archetype invariants (#1481) or kinded entities (#1634) much more cumbersome.

What solution would you like?
Bundles are altered to use an unordered collection representation, directly analogous to a HashSet.
What alternative(s) have you considered?
Implement bundle equality, subsetting etc and other needed features in a manual and indirect fashion, by e.g. comparing the HashSet of the TypeId of their components.
Additional context
Discussed on Discord.
Probably blocked on some Rust RFC that I don't know the name of :(

From @cart on Discord, the conclusions are:

The misleading mental model isn't great but this can't be fixed without sacrifices.
The methods I really want for archetype invariants etc are actually needed at the archetype level.

Closing this, but feel free to resurrect it if you think of a really clever way to improve the ergonomics.


Update install-sensor.md

Removing Character in incorrect place on the Installation parameters area

@batamig
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


'Data sources’ section in the sidebar is collapsed by default when the edit mode is opened.

The section is collapsed by default on TEST and INT


Batch message sent twice?

Maybe I don't understand something but it seems like we post the contents of the batch message,  and then add the same batch message to the userMailbox again?
https://github.com/asynkron/protoactor-go/blob/2ecba7517929a27dc1ac0c44734b9c538b9044cb/actor/mailbox.go#L56
Shouldn't there be a return?

the envelope can also implement other interfaces. e.g. auto-respond. which can be used to signal back that something has been completed.
e.g.
process each message individually from the batch
then when the actor receives the envelope, which is also an auto-respond. it acks back to the sender automatically.
so from the actors user-code, it just looks like normal messages coming in. the actor does not deal with a batch, nor does the user code have to handle the response

just quote Roger's response on slack.

@qazwsxedckll Thanks for clearing that up :+1:


[Sema] Add a fix-it for non-optional bindings initialized with nil.

<stdin>:1:16: error: nil cannot initialize specified type 'Int'
var foo: Int = nil
               ^
<stdin>:1:10: note: add '?' to form the optional type 'Int?'
var foo: Int = nil
         ^
            ?

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please test

Awesome! Fixits like this can make Swift tooling experience much sweeter.

I'm basically going through my list of ideas for optional pointer migration work and doing the fix-it-ish ones first. :-)

@swift-ci Please test.

Linux failure still unrelated. Merging!


PR: Amending content About the service/application and copy this impacts on

Relates to #458
@irinapencheva @sanjaypoyzer content fixes discussed in 458 and further content refinement in situ.
2 things to note:

When there's a widget instead of the female pension age content, copy on the page will be less dense – but there will be more for the user to 'do'.
I'd like to take "in Argleton" off the h1 page title as they are
a) on the Argleton Council website and
b) there's a signposting copy line saying "If you don't live in Argleton find your local council".

Before

After

@sanjaypoyzer added before screenshot

@lizziebruce I think this looks great, much clearer than before with more information.
Noted that it will be a bit different if we decide to put an age checker in the page.
Happy to take "in Argleton" out of the <h1> as long as we make sure it's kept in the <title> element which is shown in Google search results. For example:

(Clicking through on those pages show that the actual H1 of the pages don't show the area.)

@sanjaypoyzer thanks. yes just the h1 on the page! Not the SEO metadata title (what we call title-tag in CMS-word).

@lizziebruce Done, merging this in now.


adding confirm button and disable onhidden

fix ENG-397

onhidden may triggered by mistake. let's use confirm button to change code/format

Didn't try, but the code LGTM.


File-select from controller

Hi,
Is there any way of prompting for file selection from a controller?
I tried $('.class-of-ng-file-upload').trigger('click') (using jQuery), but that doesn't seem to work on the element, even though it works fine on other elements on the page.
Thanks,

Can you create a jsfiddle.

Let me know if this gets the point across: http://jsfiddle.net/nmdcwf3w/342/

For security reasons you cannot click on file input programmatically.


Not all data in the datastore?

Describe the bug
I'm searching for FAO data in the datastore. Currently 12 out of there 5000+ activities are showing. They're all from the same file which is passing validation.
Do know why not all the data is there?
To Reproduce
Steps to reproduce the behaviour:

Go to https://iatidatastore.iatistandard.org/api/activities/?fields=reporting_org_identifier&reporting_org_identifier=XM-DAC-41301&format=json
See that only 12 activities are being returned.

Expected behaviour
I'd expect all 5242 activities to be returned.

This is caused by the same bug as https://app.zenhub.com/workspaces/iaticloud-5b8e865bb2cc7b6c78ece322/issues/zimmerman-team/iati.cloud/2342. I have fixed that. You can test it here. This fix will include in the new release.

Great, thanks Lu Min! Will be good to have it in the new release


Any plans to release a pretrained model?

If you have one available it would help a great deal in evaluating this method. Thank you!

Unfortunately, I currently don't have the latest models. Most of the experiments were done almost a year ago, so I could not find the model. But for reproducing similar results you need to train the network on DTU dataset for 14 epochs. If I will find the model I will share it with you. But you can also consider using this older version model which I found. The names of the fielded may be changed, but feel free to give a try.

thanks very much! Will try to give this a shot or let it train if 14 epochs is all it needs. Much appreciated!


replaced encodeURI with encodeURIComponent

Hi,
If you use a hash # in your query then it breaks the input. encodeURI() doesn't encode special characters: , / ? : @ & = + $ #. There is an alternative encodeURIComponent() which also encodes those ones.

Thanks @sambwest
It'll be part of the next release version which I promise to issue after vacation.


feat(esl-footnotes): migrate esl-note to esl-base-trigger

ESLNote has been migrated on the base trigger element
Closes: #2683

Postponed for one release late

:tada: This PR is included in version 5.0.0-beta.38 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Update] Update AATH backchannel to build image

The main change here is to add a new CI/CD job to build and publish our backchannel image. then this image can be consumed in the AATH repo, rather than us having to synchronise code between here and AATH repo. The CI/CD should do the following:

re-release/replace an image tagged with main whenever we push to main branch. the main tagged image will therefore be the "latest and greatest"
publish an image tagged with some tag version whenever we create a tagged release. e.g. 0.64. these are our "stable release" tag images

Note that i haven't done github workflows before, so please carefully check that what i've done is sane... i've mostly tried to match some of the existing patterns in the workflows main yaml.
Other Changes

remove unnecessary Cargo.lock in the aath-backchannel (unused)
fix the "version" returned from the AATH controller. this version string is what is used on the AATH reports. we will need to make sure to keep it in sync with the aries-vcx crate
update some aath deps
removed some old artifacts from the github CI (hashing dirs that don't exist, etc)
update the seed creation approach. the previous approach only worked in some environments, as some times you would end up with an ENDORSER issuer_did vs a USER issuer_did ROLE. a USER issuer_did cannot write schemas, causing some tests to fail. changes ensure that we end up with an ENDORSER issuer_did.
minor changes to the image. The previous image just straight up didn't work on newer aries-vcx code.

Fixes for ACAPY compatibility
I've also made some changes to get a bit more interop with ACApy. particularly around the DIDExchange suite of interop tests.

fix did peer key format
fix did peer service key reference
fix DID Peer 3 & 4 handling
fix in-tolerant URL_SAFE decoding

new approach is tested upstream in the AATH: https://github.com/hyperledger/aries-agent-test-harness/pull/833

I poked Shel on Discord--if we wanted we could add a second image that only gets build on release. It's a little bit more work (and could be a separate item if we want), but could be a good idea potentially.

@JamesKEbert  hey, this one is ready for review. I believe the updated CI/CD should let us support the different version testing style that you suggested. Pls see my MR overview.
tbh, we won't know for sure that this is all working until we merge it in and start getting tagged versions of the image. If it fails for whatever reason, i will iterate on it quickly of course.

Forgive me for making indirectly related changes, but when testing acapy, i found some issues with our did:peer implementations. My latest commit should address them: https://github.com/hyperledger/aries-vcx/pull/1221/commits/a3c19840b7f1ee831337bdfefe77ca449f1bd894
Will test in acapy now.

ok with my latest changes, we get very close to successful RFC0023 tests with acapy. we fail on the final step when we try to decode their DIDDoc within the DIDExchange Request they sent us... we fail here because they are using legacy DIDDocs when we are expecting a modern diddoc structure.. i suspect this is related to acapy using unqualified DIDs in these requests. i'll see if there's a way to run the tests with acapy using qualified did:peers rather than unqualified

The AATH CI pieces look good to me, even if we accidentally did something wrong with the CI I am A-okay with us doing followup fixes there :+1:

As for the DID Exchange issues w/ACA-Py, the DIDDoc appears to be eventually resolved in /aries_vcx/src/protocols/did_exchange/state_machine/responder/response_sent/mod.rs receive_request(), which calls let their_ddo = resolve_ddo_from_request(&resolver_registry, &request).await?;, which attempts to map the attachment to a DIDDoc, but if that fails it will attempt to resolve it via the resolver registry (which the AATH agent has did:sov and did:peer registered, but since in the request the DID is unqualified the parser/resolver would fail I'd assume).
The DIDDoc format corresponds to RFC 0067, but that RFC hasn't been officially accepted/adopted, so I am unsure as to whether or not this is acceptable/expected/ideal behavior. From your POV, is this a sufficient DIDDoc inside of Aries VCX?

@JamesKEbert thanks for taking a look. yea i think we are doing what the DIDExchange spec defines. Essentially:

if the request contains a did_doc attachment, then decode that as the DIDDocument. If that decode fails then fail early (.transpose()?)
else if there was no did_doc, then try use the resolver to resolve the DID.

The RFC from my reading says that implementors should either:

have an unqualified DID and a did_doc attachment set in their request, or
have a qualified (resolvable) DID and do not attach a did_doc attachment

So in the case of doing pure did:peer:4 exchanges with ACApy, we should be good, as we will hit case 2 and be able to properly resolve the DID; i've also looked at acapy code to confirm this. However when acapy is using an unqualified DID and goes with case 1, then they will attach the weird old/legacy/aries version of a DIDDocument, which does not match the real spec: https://www.w3.org/TR/did-core/#did-document-properties
I had a look at how acapy handles this situation (i.e. acapy handling receiving and processing a DIDExchange request in case 2). The following happens:

The request is funnelled into acapy's _extract_and_record_did_doc_info function.
This function extracts the DIDDoc as a raw JSON/dict (no conversion yet) and stores it in the wallet against the unqualified DID (all in raw JSON/dict)
Eventually when acapy needs to retrieve the doc, i believe it is retrieved via the LegacyPeerDIDResolver implementation, which fetches the doc from the wallet.
After fetching the raw dict/JSON doc from the wallet, it will pass it thru a LegacyDocCorrections implementation to convert the weird/legacy/aries DIDDoc into a real DIDDoc.

long story short; acapy will proactively handle both by converting the legacy format into the correct format.
So i suppose aries-vcx should consider whether it should handle this legacy doc and try convert it? i think it probably should... I'll have a go at implementing this now. if it's too much, then i'll create another ticket

Supporting DIDExchange v1.1 will also improve our interop with acapy: https://github.com/hyperledger/aries-vcx/issues/1228


feat: sitemap detection audit

Sitemap detection for EDS customers
Ticket: https://jira.corp.adobe.com/browse/SITES-18817
Related PRs:

https://github.com/adobe/spacecat-api-service/pull/136
https://github.com/adobe/spacecat-audit-worker/pull/114

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

v1.8.0
GitHub release

Your semantic-release bot :package::rocket:


configurable task timeout

Hi,
The timeout of a task is defined definition level, but the execution time of a task can very be variable depending on input parameter. Don't you think it could be interresting to define/orverload it a workflow level, or even better to define/overload it dynamically.
Regards,
Sébastien

Hi,
yes, this is indeed quite helpful, and something we have it in mind longer term.  Not supported at the moment though.

Hi,
Is this still in road map, I am finding this very useful while designing generic workflows and sub-workflows.
If there is any workaround for timeout parameter, let the thread know, I am trying to use something similar
Regards,
Joseph Joshy

@josephrjd We couldn't get to prioritizing this over other features in Conductor 2.0 or 3.0; If you have bandwidth to help with this, Please feel free to open a Github issue with a proposal to get the ball rolling. Thanks!


System dependencies

Add the possibility to have dependencies between systems of the same parent.
The 'Systems' class will resolve and assign the dependencies to every class that implements the IHasSystemDependency interface by calling the 'SetupDependecies' method.
The method should be called before the 'Initialize' so systems can operate with dependencies assigned.

Hi, thanks for the PR.
I think systems should not be dependent on other systems. Some systems might only make sense when working in combination with others, this is usually when you group them in a systems parent to make the order visible. A system shouldn't be able to see beyond its scope other than observing and working on the context and its entities.
Is there sth I'm missing? If yes, could you explain what

Hi,
I agree with you, but there are cases when a system needs to change or trigger a behaviour of another system. Of course this can handled with components and reactive systems, but it may lead in having custom components just for this interaction, when it may be easier to just have a dependency.
This may lead also in features (group of systems) that are hard to read and hard grasp the flow.
From my perspective, its not essential but a nice to have (or not have :D) possibility. It doesn’t take away from the ECS architecture but can help make things simple in some cases.
Yes, there are DI frameworks out there, but as I said above, it can help in some simple cases where a DI framework can be an overkill.
What do you think ?

I personally think that this system dependency thing is like trying to bring old OOP ways into ECS. I really like that systems don't know anything about other systems. They only care about data.

I agree with you, but there are cases when a system needs to change or trigger a behaviour of another system.

Systems don't trigger other systems. Data does. And it shouldn't be important where this data comes from - it may be another system or it can be direct player (or developer's) input. If you bind one system to another, you will basically tie data changes to that system. This means that we can't just change data by hand (via the inspector) or by means of other systems and hope that the first system will trigger - this change will have to be made from the system on which we depend on. It also means that I can't disable one system and hope that other systems will continue to work (right now they would). Making systems depend on each other will put logic on the first place again, while in ECS we say that it's data-first.
I think it will increase the complexity of the easy-to-use and easy-to-read code.
As for the "hard to grasp the flow" - I'm agree with you here, and this is a comprehension problem. It can be solved using some static code analysis tools and visual node-based tools. We don't have to spoil our code with inter-system dependencies to understand what's going on. (We have a Visual Entitas Jam for this - #475 wink wink)
What Unity does with its internal ECS and system dependencies is just a way to arrange systems order in a way that will make sense (data has to be changed before it's consumed). It's all automatic and doesn't require any by-hand dependency placements. Entitas is a step forward from that actually because in Entitas we can order systems ourselves, getting full control over the execution flow.

@KumoKairo fair point, thank you for the clear explanation ;)
I understand the usage of an ECS so i guess what you're trying to say is that having a feature in a library that goes against its usage simply defeats its own purpose.
I agree, so there is no need for this addition.


HW Upgrade - Replace PGA

Replace PGA for higher input voltage IC
Best candidate: THS7001
Comes in dual package with independant gain controls, could be used for both MAIN and AUX outputs

Added discrete PGA using Opamp and CD4051.
Resolved in 884bd39716d23852e1e5afd593aa6cd09cad86fc


Optimize ToHex

Sha1.ToHex is non-optimal
Nuget
Clean up code, conventions
Add string.RemovePrefix

Could optimize further, if overflow and invalid characters is not a concern:
https://gist.github.com/jcdickinson/9a4205287ae107e9f4e5f6764f432db9
h=(c>>6);h|=(h<<3);h+=(c&15);

Clever. But this use case does care about invalid characters. Happy to use it if you can find a way around that.


Magic labeller

Add a magic labeller function where the user draws a box around an object, initialize the model eventually and then segments the box

Released starting from v1.2.4rc35


Output truncated (output buffering bug?)

At Logtalk startup, by default, it prints a banner, the default flag values, information on the settings file being used (if any), and information on how to load the help and tutor tools. The output stops after the default flag values. If I comment out the call (in core.pl) that prints the default flag values, then I get all the remaining output. The size of the text from the banner to the default flag values is ~1500 characters. Is there any output buffering limit at play here?

To reproduce the bug, you can use comment out the goal '$lgt_print_message'(comment(settings), default_flags) at the end of the core.pl file.
The help script that I'm using is:
var readline = require("readline");
var pl = require("./node_modules/tau-prolog/modules/core.js");
require("./node_modules/tau-prolog/modules/format.js")(pl);
require("./node_modules/tau-prolog/modules/js.js")(pl);
require("./node_modules/tau-prolog/modules/lists.js")(pl);
require("./node_modules/tau-prolog/modules/os.js")(pl);
require("./node_modules/tau-prolog/modules/random.js")(pl);
require("./node_modules/tau-prolog/modules/statistics.js")(pl);
//require("./node_modules/tau-prolog/modules/system.js")(pl);

var session = pl.create( 20000 );
var show = x => console.log(pl.format_answer(x, session));
session.consult("../logtalk3/adapters/unsupported/tau.pl", {success: function() {
    session.consult("../logtalk3/paths/paths.pl", {success: function() {
        session.consult("../logtalk3/core/core.pl", {success: function() {
//            session.query("write(done), nl.");
//            session.answers(show);
        }, error: show});
    }, error: show});
}, error: show});

var max_answers = process.argv[2] || 100;
var rl = readline.createInterface({
    input: process.stdin,
    output: process.stdout,
    terminal: false
});

rl.on('line', function(query) {
    if(query.length > 0) {
        var parsed = session.query(query);
        if(parsed === true)
            session.answers(x => console.log("\n" + pl.format_answer(x, session)), max_answers);
        else
            console.log(pl.format_answer(parsed, session));
    }
});

The script is called using:
$ node --stack_size=10000 ./taulgt.js

Beware that startup is quite slow. To speed it up, change in core.pl around line 26679 the option clean(on) to clean(off). This will make startups after the first one much faster.

@jariazavalverde verified that this problem doesn't occur on Linux. I am on macOS.

Your a31ca8e5e6f6a44e5224a6e3277a90c20d85cfc7 commit solved the truncated output for me on macOS.

I guess the parser wasn't finishing parsing the entire file due to excessive recursion. I had a similar problem on Windows, where execution ended after parsing ~ 21000 lines.


No validation messages for Select field

Try adding errors.tags = 'Error'; right above the return errors; statement in the function passed to Formik's validate prop in this sandbox - no validation message is displayed or any other styling indicating that there's an error associated to that field. The same is for DatePicker. Anyone knows how to fix this?

Sorry, just noticed that the original Material-UI component doesn't have error display capabilities. Closing this issue.


Set up Powerline in PowerShell - issues after installation

Using the instructions on the page installs oh-my-posh v3. This causes issues with the git prompt as well as with some of the fonts.
Installing the last stable v2 branch solves the problem
Install-Module -Name oh-my-posh -RequiredVersion 2.0.496

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

ID: d3cbeb9a-9e7c-f8f4-5c03-5d6262d2b440
Version Independent ID: eb575c79-5b2a-45f6-646f-9d41d5d223e8
Content: Windows Terminal Powerline Setup
Content Source: TerminalDocs/tutorials/powerline-setup.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @cinnamon-msft
Microsoft Alias: cinnamon

This is a dupe of #254, which was fixed in #263


Is it possible to trigger dependabot scan on developer pull requests

(also posted in SO: https://stackoverflow.com/questions/71068152/how-to-trigger-dependabot-scan-on-developer-pull-requests)
I'm not sure if my use case is one dependabot is suited for, so hoping someone can tell me if it is or is not, and if it is, point me to some documentation on how to do what I'm describing:
I want to create workflow that:

runs dependabot scan on each developer pull request
dependabot only reports on newly introduced or updated dependencies in scanned pull request
pull request is blocked by any new dependencies with vulnerabilities of medium or higher
dependabot does not create PR as a result of a PR scan

Is this possible or outside the scope of what dependabot's use cases?

Hey, it looks like your feature request is about dependabot alerts and the dependency-graph which aren't powered by dependabot-core (I realize this is not necessarily clear), but I think some of what you're is already available with dependency review, and the rest is planned

@jurre Ah thank you so much. I found the info in dependency review shortly after asking this question. And great to see the planned info as well.


Usage for segmentation

Hi,
Is it possible use this for segmentation?
Thanks

I'm not sure, I haven't done any image segmentation myself. I would think that the pixel-level outcome would be categorical rather than ordinal though.


Research docker bridge urls

The readme config hardcodes the localstack container IP <IP_ADDRESS>. It is this IP on my machine but probably not on all. Some other method to reference the container needs to be found

fixed by https://github.com/NovelService/NovelWorkerNode/commit/c37d547336fa9d0935e701fe8529f144473ff8ea


Support for Crashlytics

Some devices are crashing with a blank screen or just freeze, so I want to add crashlytics but I cannnot find the file MainActivity.java, which leads me to think that it is not supported here, can you please confirm?
Thanks

LauncherActivity.java is part of the Support Library and can be found here.
Regarding the crashing with a blank screen / freeze, we just released a new version of the Support Library that should fix crashes when the user's default browser doesn't support TWA, and it can be enabled with the version below:
implementation 'com.github.GoogleChrome.custom-tabs-client:customtabs:d08e93fce3'

If you still see issues with blank screen / freezes with this new version, please let us know, and we'll investigate.
Andre

thank you @andreban ! it does work. It shows the top address bar but at least it doesnt crash anymore. Just FYI, it seems that the people having these issues were on samsung devices (some on s9). I dont know if samsung does not support TWA but well i guess its just matter of time for their browser to support it.
Thank you very much for the fix you rock!


Auto-scaling controller does not scale-down the Job

I submit 10 auto-scaling jobs with min-instance=2, max-instance=20, but job-0 has 20 trainers and job-9 has only 2 trainers.
There are too many PENDING trainer pod, but the logs in controller is as following:
time="2017-10-30T05:25:31Z" level=debug msg="Dry run scale job mnist7: current 0, additional 0, remaining resource: autoscaler.ClusterResource{NodeCount:133, GPURequest:0, GPULimit:0, GPUTotal:0, CPURequestMilli:1269510, CPULimitMilli:1848000, CPUTotalMilli:2348000, MemoryRequestMega:1186525, MemoryLimitMega:1186588, MemoryTotalMega:13665210}"

In the logs CPURequestMilli < CPUTotalMilli , but I think it should be CPURequestMilli > CPUTotalMilli  in fact.

This is a bug due to:
 1621 time="2017-10-30T02:10:40Z" level=debug msg="Dry run scale job mnist4: current -32, additional 0, remaining resource: autoscaler.ClusterResource{NodeCo      unt:133, GPURequest:0, GPULimit:0, GPUTotal:0, CPURequestMilli:2339510, CPULimitMilli:3933000, CPUTotalMilli:2348000, MemoryRequestMega:2105646, Memory      LimitMega:2380589, MemoryTotalMega:13665210}"
 1622 time="2017-10-30T02:10:40Z" level=info msg="Scaling plan: map[mnist4:-32]"
 1623 time="2017-10-30T02:10:40Z" level=info msg="Scaling job mnist4, diff: -32."
 1624 time="2017-10-30T02:10:40Z" level=error msg="Error updating trainer job Job.batch \"mnist4-trainer\" is invalid: spec.parallelism: Invalid value: -38:       must be greater than or equal to 0, retry remaining: 0."
 1625 time="2017-10-30T02:10:40Z" level=error msg="Error updating trainer job Job.batch \"mnist4-trainer\" is invalid: spec.parallelism: Invalid value: -38:       must be greater than or equal to 0, retry remaining: 1."
 1626 time="2017-10-30T02:10:40Z" level=error msg="Error updating trainer job Job.batch \"mnist4-trainer\" is invalid: spec.parallelism: Invalid value: -38:       must be greater than or equal to 0, retry remaining: 2."

The problem remains, so I reopen this issue.

I checked the log, it seems that total CPU request is less than the real value in the cluster.

I have added this PR as an improvement: https://github.com/PaddlePaddle/cloud/pull/456

I think this is fixed, please reopen if otherwise.


write on empty tag ios

hello @whitedogg13,please save me.
ios writing not working on new tag(unformatted),when i formate the tag with another app and then try to write with your library, it is work but i need to do it in my own app, please help me.

Please see instructions here. We have an example for naive NDEF formatting.
BTW, we have a slack channel here: https://join.slack.com/t/reactnativenf-ewh2625/shared_invite/zt-l2phclhi-AoYGQOOlCmvCSa3sBiA0IQ
You can join us for more detailed discussion.
I will close it for now, please feel free to reopen or fire another one if there’re still issues.
Thanks!


<EMAIL_ADDRESS>hash check failed

Description
Currently, the sha-1 hash of cudatext <IP_ADDRESS> (amd64 version) is c757c7d97017e55e7be92d4604218c06657a9389 according to SourceForge website, but on manifest it's 147c3d3a001b343a4091622af0b357ef14e4fa8d.
Console Output
cudatext: <IP_ADDRESS> -> <IP_ADDRESS>
Updating one outdated app:
Updating 'cudatext' (<IP_ADDRESS> -> <IP_ADDRESS>)
Downloading new version
Starting download with aria2 ...
Download: Download Results:
Download: gid   |stat|avg speed  |path/URI
Download: ======+====+===========+=======================================================
Download: ad7109|OK  |    65KiB/s|C:/Programs/Scoops/cache/cudatext#<IP_ADDRESS>#https_sourceforge.net_projects_cudatext_files_release_<IP_ADDRESS>_cudatext-windows-amd64-<IP_ADDRESS>.zip
Download: Status Legend:
Download: (OK):download completed.
Checking hash of cudatext-windows-amd64-<IP_ADDRESS>.zip ... ERROR Hash check failed!
App:         extras/cudatext
URL:         https://sourceforge.net/projects/cudatext/files/release/<IP_ADDRESS>/cudatext-windows-amd64-<IP_ADDRESS>.zip
First bytes: 50 4B 03 04 0A 00 00 00
Expected:    147c3d3a001b343a4091622af0b357ef14e4fa8d
Actual:      c757c7d97017e55e7be92d4604218c06657a9389
SourceForge.net is known for causing hash validation fails. Please try again before opening a ticket.

So, the downloaded files is actually not corrupted.

Close due to new version (<IP_ADDRESS>) released (0507cbc3a2d2a7ec3103216541ffb61ab677e76e).


Attribute selectors with comma not working

When using nested attribute selectors that contains commas, they generate incorrect styles, inserting ampersands after the comma.
For example:
    '& [style*="color: rgb(245, 240, 241);"]': {
      color: 'black !important'
    },

will result in something like
.fk7b603 [style*="color: rgb(245,& 240,& 241);"] { color: black !important; }

See more in this Codesandbox.

That indeed a bug, can be reproed in our sandbox, too.
https://github.com/microsoft/griffel/blob/31bb2215926d5224ed2bd50fe9a61677a756a02f/packages/core/src/runtime/compileCSS.ts#L35-L44
The function above is probably the source of the problem.

Thank you @layershifter ! I would definitely not use such rules in normal conditions. But sometimes, for complex and legacy apps that could be a good workaround.
BTW, can you elaborate about the performance issues? I understand that this selector is less performant than selecting by class, but do you think the performance hit will be noticeable?


Cannot specify order for admin API page results when using for_explorer

when i get sidebar pages use below url :
api/main/pages/?child_of=4&for_explorer=1&fields=parent&order=-id
response is : filtering by for_explorer without child_of is not supported
but when i get sidebar pages use below url :
api/main/pages/?child_of=4&for_explorer=1&fields=parent
response is fine,

Thanks for the report @maya413! I can confirm I'm seeing the same issue when testing against bakerydemo:
http://localhost:8000/admin/api/main/pages/?child_of=1&for_explorer=1&fields=parent returns results
http://localhost:8000/admin/api/main/pages/?child_of=1&for_explorer=1&fields=parent&order=id returns the error filtering by for_explorer without child_of is not supported
It looks like this is because the for_explorer filter relies on a _filtered_by_child_of attribute being set on the queryset, but applying an ordering replaces that queryset with a new one.
The admin API isn't really intended to be used outside of Wagtail's own code, but I'll accept this as a legitimate bug.

@gasman  It seems this can be fixed by switching the order for the filter backends to apply the ordering before the child_of is gotten, i don't think this affects the output in any way, should i make a pull request for this?


Rename SECURITY.md to SECURITY.mda

Description

Motivation & Context

Type of changes

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

Checklist

[ ] I have read the contributing guidelines
[ ] My code follows the code style of the project (using npm run lint)
[ ] My change introduces changes to the documentation
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] All new and existing tests passed

Live previews

https://deploy-preview-{your_pr_number}--twbs-bootstrap.netlify.app/

Related issues

Jawali, Katghora, Korba, Chhattisgarh, 495445', 'B-2022: 10-90347-004501', '13-08-2022', '14-08-22 08:25:26', '14-08-22 08:25:26', '7896619162', 'MALE', 'Chhattisgarh', 'INDIRA GANDHI DISTRICT HOSPITAL KORBA '),
(115, 'chitra yadav', '', '27-07-2002', 'twenty seven -July- two thousand twenty two 'MUNICIPAL CORPORATION RAJNANDGAON', 'radhika yadav', '4793 2026 9795', 'motilal yadav', '8087 7944 4455', 'khapriteli, khapriteli, khairagarh, rajnandgaon, Chhattisgarh, 491881', 'khapriteli, khapriteli, khairagarh, rajnandgaon, Chhattisgarh, 491881', 'B-2022: 10-90347-004785', 13-08-2022', '14-08-22 08:33:00', '14-08-22 08:33:00',
7896619162', 'FEMALE', 'Chhattisgarh', 'MUNICIPAL CORPORATION
RAJNANDGAON'), (116, 'MAHI JANGDE', 'MAHI JANGDE', '22-05-2015', twenty twO
May-two thousand fifteen', 'community health center sakti',
'ANJU JANGDE', '5548 4437 0777', 'SANTOSH KUMAR JANGDE', '5901 9029 0443', 'NAKTIDIH, TALDEORI, CHAMPA, JANJGIR, Chhattisgarh, 495661', 'NAKTIDIH, TALDEORI, CHAMPA, JANJGIR, Chhattisgarh,
495661', 'B-2022: 10-90347-002249', '13-08-2022', '14-08-22 08:45:35', '14-08-22 08:45:35', '7896619162', 'FEMALE', 'Chhattisgarh', 'COMMUNITY HEALTH CENTER SAKTI ').
(117, 'moksh bandhe', thousand twenty two', ','26-07-2022', twenty six -July- two 'MUNICIPAL CORPORATION RAJNANDGAON,
rohit bandhe', '8798 6134


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_2d58e495-2c81-4a10-90a5-61bcdc309962/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_2d58e495-2c81-4a10-90a5-61bcdc309962.git

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

File
Status
Preview URL
Details

:x:Error

Details

[Error-ServiceAccountCannotAccessRepository] Your repository https://github.com/VSC-Service-Account/docs-archive-test-source has enabled Docs Build service account. It seems that at least one of the accounts, e2ebd3, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master#ops-e2e-ppe) and then retry the build. Don't know who to contact? Refer https://repos.opensource.microsoft.com/VSC-Service-Account/repos/docs-archive-test-source/permissions/ to get admin list of the repo if the repo is in one of the organizations Microsoft owns.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Fix unloaded owner bug

Fix unloaded owner bug in Repository.getUnitsByUserID(..). Bug occurred when I tried pushing to a locally-running instance of gitea.

It seems this will fix #1768 ?

LGTM

let L-G-T-M work

LGTM


update to path expansion attmap fix.

we used the paradigm rgc.to_dict()["genome_server"] in a few places instead of the preferable rgc.genome_server, due to problem with Path expansion for URLs. These have now been fixed so we should update across the board.
for example:
https://github.com/databio/refgenie/blob/91a6f6424652689943390cdc0e58a263d0a94738/refgenie/refgenie.py#L308
need to also bump requirement file to require the patched attmap

This is done in refgenconf, and it appears the code that's migrated contained the remaining usages.


Update cache/checkout action versions

The right versions are on master now, must have been merged in another PR. Sorry for the duplication


fix: get rid of import alias as it still breaks everything around

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Convert to named function refactoring should preserve async modifier

The "convert to named function" refactoring should preserve the async modifier.
Starting with this code:
const foo = async () => {
  await Promise.resolve()
}

The current behaviour, after converting to a named function, produces:
function foo() {
  await Promise.resolve()
}

The async modifier has been lost during the refactoring, resulting in invalid code. There is another refactoring to "add async modifier", but it's an unnecessary second step.
Ideally it should instead produce this result in one pass:
async function foo() {
  await Promise.resolve()
}

Duplicate of https://github.com/microsoft/TypeScript/issues/40366

Closing as duplicate - fix available.

Sorry! I did try to find if it had already been reported by search but came up short. Thanks anyway

I'm a little confused about this, but what's the point of converting like this? Is the arrow function bad

There is nothing bad about arrow functions, but there are slight differences in behaviour between an arrow function and a named function - arrow functions have lexical scope, no access to arguments.
Sometimes you want/need this, sometimes you don't, so it is useful to be able to quickly convert between the two.

I know slight differences in behaviour between an arrow function and a named function ,I thought the hint there was to force me to change the arrow function to a named function

no, the refactoring works both ways :)

Ok, thank you


Error updating Focal-Error Compiling CRAS(i386)

2021-08-17 11:49:19 (817 KB/s) - '/tmp/crouton-cras.cysSKn/adhd.tar.gz' saved [919630]

Reading package lists... Done
Building dependency tree
Reading state information... Done
libspeexdsp1:i386 is already the newest version (1.2~rc1.2-1.1ubuntu1).
alsa-utils is already the newest version (1.2.2-1ubuntu2.1).
libasound2:i386 is already the newest version (1.2.2-2.1ubuntu2.4).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
gcc is already the newest version (4:9.3.0-1ubuntu2).
gcc-multilib is already the newest version (4:9.3.0-1ubuntu2).
pkg-config is already the newest version (0.29.1-0ubuntu4).
libspeexdsp-dev:i386 is already the newest version (1.2~rc1.2-1.1ubuntu1).
libc6-dev:i386 is already the newest version (2.31-0ubuntu9.2).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
libasound2-dev:i386 is already the newest version (1.2.2-2.1ubuntu2.4).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Reading package lists... Done
Building dependency tree
Reading state information... Done
patch is already the newest version (2.7.6-6).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
Compiling CRAS (i386)...
/bin/ld: skipping incompatible /usr/lib/gcc/x86_64-linux-gnu/5/libgcc.a when searching for -lgcc
/bin/ld: cannot find -lgcc
/bin/ld: skipping incompatible /usr/lib/gcc/x86_64-linux-gnu/5/libgcc_s.so when searching for -lgcc_s
/bin/ld: cannot find -lgcc_s
collect2: error: ld returned 1 exit status
Failed to complete chroot setup.
Unmounting /mnt/stateful_partition/crouton/chroots/focal...
chronos@localhost / $

Well, I install Focal in XFCE Desktop but I wanted to use Unity or Gnome but whenever i try updating nothing happens!
If known, describe the steps to reproduce the issue:
Try updating the chroot
PS: XFCE Desktop works flawlessly with audio too. but I don't know. Maybe CRAS might have got corrupted somehow?

@sanrobin,
Please show us the output of the command: sudo edit-chroot -all
Then we'll have more details about your version, device, chroot, etc. which will help in troubleshooting your issue.
Hope this helps,
-DennisLfromGA

lol sorry for late reply i found out that it was using gcc 5 to compile. I just needed to reinstall and it worked. sorry for late reply.


Warshal.cpp

Clean and concise code in  c++
Assigned Issue Number ?
#Issue_Number
Language(s) Opted ?
C/C++/Python/Java ...
Number of file(s) added or updated ?
1/2/3/4/...
Name and location of the file(s) ?
Folder => Sub_Folder => ... => File.extension
Checklist

[ ] I've followed the contributing guidelines
[ ] I've synced-up my forked repo.
[ ] Ain't pushing from the main branch.
[ ] The content added isn't directly copied from any sources like gfg, tutorialpoint, etc.
[ ] Input has been taken dynamically.
[ ] I've referred the correct issue number.
[ ] Comments have been added
[ ] I've  synced-up my forked repo.
[ ] Ain't pushing from the main branch.

[ ] Ain't pushing from the main branch.


platform target in run tests.

@dbluhm suggested seeing if adding a platform target would allow docker to run the test scripts, which worked. I expect more changes for complete mac m1 development support but this seems to be a good starting point.
Signed-off-by: Adam Burdett<EMAIL_ADDRESS>

Continues to work on Linux/Ubuntu on Intel processor, so that's good :-)

See also #1689

We'll likely want similar changes in the run_docker and other scripts

Codecov Report

Merging #1697 (49849cc) into main (845af96) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1697   +/-   ##
=======================================
  Coverage   95.29%   95.29%
=======================================
  Files         528      528
  Lines       32878    32878
=======================================
  Hits        31331    31331
  Misses       1547     1547


rdesktop slow rendering since el capitan

Hello,
Since el capitan it seems rdesktop is broken (slow rendering)
Have had no problems with yosemite.
Has anyone else noticed this?
Kind regards,
Bert

This doesn't seem likely to be a Homebrew-specific issue. If you install rdesktop outside of Homebrew on El Cap, does it show similar performance problems?

We'll accept PRs for this but we're not actively working on it at this time.


TypeError: Cannot read property 'txid' of undefined

when in its still in sync sometimes it stopping itself, i had to start it again ...
[2015-12-18T13:29:17.282Z] error: uncaught exception: [TypeError: Cannot read property 'txid' of undefined]
[2015-12-18T13:29:17.284Z] error: TypeError: Cannot read property 'txid' of undefined
at /root/mynode/node_modules/bitcore-node/lib/services/address/index.js:1474:46
at /root/mynode/node_modules/bitcore-node/node_modules/async/lib/async.js:721:13
at /root/mynode/node_modules/bitcore-node/node_modules/async/lib/async.js:52:16
at done (/root/mynode/node_modules/bitcore-node/node_modules/async/lib/async.js:246:17)
at /root/mynode/node_modules/bitcore-node/node_modules/async/lib/async.js:44:16
at /root/mynode/node_modules/bitcore-node/node_modules/async/lib/async.js:718:17
at /root/mynode/node_modules/bitcore-node/node_modules/async/lib/async.js:167:37
at /root/mynode/node_modules/bitcore-node/lib/services/address/index.js:1423:11
at /root/mynode/node_modules/bitcore-node/lib/services/address/index.js:1180:9
at ReadStream. (/root/mynode/node_modules/bitcore-node/lib/services/address/index.js:1239:5)
[2015-12-18T13:29:17.284Z] info: Beginning shutdown
[2015-12-18T13:29:17.285Z] info: Stopping insight-ui
[2015-12-18T13:29:17.285Z] info: Stopping insight-api
[2015-12-18T13:29:17.285Z] info: Stopping web
[2015-12-18T13:29:17.286Z] info: Stopping address
[2015-12-18T13:29:17.286Z] info: Stopping db
[2015-12-18T13:29:17.307Z] info: Stopping bitcoind
[2015-12-18T13:29:18.307Z] info: bitcoind shutdown.

Outdated


Feature request: hotkeys for "Indicate cursor position" and "Show icon on click"

Terrific program for those of us who demo programs or teach how to use computers! Thanks!
A really nice enhancement would be to add user-definable hot-keys to toggle on/off the "Indicate cursor position" and "Show icons on click". And probably "Show keystroke History" as well, but I rarely use that, so it is not as critical for me.
With user-definable hotkeys, I could leave the program loaded all the time and just have things toggled off. when I want to demo something, I could quickly toggle on features I want. A lot of the time, I might want just "Indicate cursor position" and then just toggle on "Show icons on click" when I am demoing something where it is important that people understand if I am using a left-click, right-click or a modifier key, and then toggle that feature off again.
And, it these hot-keys could be added, I think it would also make sense to add an option to "Load when Windows starts" so it is always there and ready for use.
Thanks again!

First suggestion is implemented and will be part of next release. (But with only a single shortcut to toggle all enabled features on/off)
The start up suggestion is difficult to implement, because the programm does not install itself to a reliable location. But there are many ways to run a programm on windows startup.


PLZ FIX README CONTRIBUTION-PIC UPDATE

Readme pic update

hi please assign me this issue

Hello, Please Assign this to me.

I already got fixed. just need 2 minutes to fix it.

The Fix is already there in the Pull request Please check.

The issue is not fixed, because in the contributor section of my readme, I want everyone's github profile pic to be displayed. But it's not showing. I have tried with this site, but it's not working


type checking in tests/ directory doesn't use tsconfig (fully?)

Working on a new file in tests/ and noticed that our vendored digidem types definitions aren't being detected in it. Most of the files in that directory don't have typechecking enabled and the ones that do are done via // @ts-check in order to gradually integrate typechecking in that directory. However, with the // @ts-check, it seems like TS is just inferring types based on reading the source module as opposed to using the tsconfig to resolve some of the external module types, which works in some cases but overall doesn't work given the complexity of some of the hypercore-related types.
if you do npx tsc --showConfig, you'll notice that - with the exception of one file, tests/schema.js (which is explicitly listed in the files tsconfig field) - none of the tests/ files with the // @ts-check annotation are listed in the files field i.e. having the tsconfig being applied to it. This leads me to believe that using // @ts-check doesn't (fully) apply the closest found tsconfig.json file as I expected. I find it hard to believe that TS will just apply its default configuration in this case, so maybe it does something clever that isn't transparent to me.
The solution would be to update the tsconfig to either

add each file in the files field (granular, but tedious)
adding the tests/**/* in the include field (less tedious but then we have to deal with all of the type errors that haven't been addressed)

Was able to confirm that vendored type resolution is fixed applying either of the above options.

I think we can add a tsconfig.json to the tests/ folder, and extend the main tsconfig in that, plus include the test files. This avoids including test files in the type tests and emitting types, but still allows the editor to check them.


Modify Bridge.hasValue to accept multiple arguments

Proposal to add functionality to Bridge.hasValue which would allow passing 1 to many argument values. Each value would be checked. If all arguments have a value, then true is returned, otherwise false is returned.
This change would add a little more logic to the hasValue functionality, but would reduce the .js size in many places throughout bridge.js.
Expected
Bridge.hasValue(a, b) // all true, or return false

Actual
Bridge.hasValue(a) && Bridge.hasValue(b)

No breaking changes would occur. This change would be enhance the functionality of the existing hasValue function.

The existing hasValue function is located at:
https://github.com/bridgedotnet/Bridge/blob/master/Bridge/Resources/Nullable.js#L2-L4
// Existing
hasValue: function (obj) {
    return obj !== null && obj !== undefined;
};

// Proposed
hasValue: function (obj) {
    if (arguments.length === 0) {
        return false;
    }

    var i = 0;

    for (i; i < arguments.length; i++) {
        if (arguments[i] === null || arguments[i] === undefined) {
            return false;
        }
    }

    return true;
};

I ran some basic performance testing with 100 million (100,000,000) iterations.
http://deck.net/f5f848b5064b1eb334cce56e1823c05f/left

Slower when one object is passed (approx. 99ms vs 167ms)
There would an approximate 200 byte savings in the bridge.js file size

With 100 million operations, the hasValue function would not be the bottleneck.
Seems like this revision would bring a nice client-side api optimization and reduce the file size of bridge.js.

As for space savings, i remember that this topic was rised already but i can't find it on the forum, to introduce modules in bridge, im not using c#Regex, Decimal, Long, WebSocket, Reflection or Grobalization which is ~250kb off from the bridge.js in my case. Maybe my example is pretty extreeme since i have plain code, didnt imported anything from ouside and i keep in mind js flow writing code.
I already looked into the bridge code and it seems like there is a lot of dependencies of those so it may not be easy to split them to actual modules, but i though about providing skeleton for each not used module on which each member would rise exception (just in case).

Also for performance testing small methods like this it's common to do loop unrolling to remove loop play in time measurements, in my browser just empty loop takes 25% of total time

We'll be working on some options for greatly reducing the bridge.js file size with dependency checking.
The majority of bridge.js is a concatenation of various sub-libraries required to support various large chunks of the .NET Framework. You listed most of them, including Regex, Decimal, Long, etc. Another is LINQ. If you strip all these libraries out of bridge.js, there's not much Bridge remaining.
If you're not using a certain .NET feature within your App, then you should have the option to exclude from the compiled output runtime library.

Instead of
if (arguments[i] === null || arguments[i] === undefined)

We can use
if (arguments[i] == null)

The basic performance testing with 100 million (100,000,000) iterations and added optimized existing hasValue (to use if (arguments[i] == null)):
http://deck.net/aac7c465582442a75f024b95dcd30cc9/left

Optimized existing hasValue (to use if (arguments[i] == null)): 105 ms
Current approach 189 ms
With arguments 237 ms

1) Optimizes existing Bridge.hasValue (faster 105 ms against 189 ms on your perf test Deck)
Current:
hasValue: function (obj) {
    return (obj !== null) && (obj !== undefined);
},

Revised:
hasValue: function (obj) {
    return obj != null;
},

2) Adds an overload Bridge.hasValue$1 to work with function arguments, a trade-off to keep the exising fast hasValue and save some bytes for bridge.js in case of multiple arguments:
hasValue$1: function () {
    if (arguments.length === 0) {
        return false;
    }

    var i = 0;

    for (i; i < arguments.length; i++) {
        if (arguments[i] == null) {
            return false;
        }
    }

    return true;
},

Saves 259 bytes in the bridge.js file size: 259 bytes = 883'226 - 882'967

Nullable.js needs to be reviewed.


Add env var for request limit

Part of https://github.com/plotly/streambed/issues/11979
Adding an ENV VAR for request limit as we need it but only for On-Prem
Tested on:

[x] Local On-Prem Instance

@mag009 a quick review please ?

:dancer:


Pin the version of hie used during testing

Currently the tests are failing since HIE's output has changed. We should update the tests and then pin the version of HIE used to tests (submodules maybe?)

This was done in 3c3116c


Email Verification Link Upon Signup

Is your feature request related to a problem? Please describe.
To confirm the person signing up is not a robot or scammer there is need to confirm the user email address before letting the person carryout core activities on the platform this API will be used.
Describe how to intend to go about adding this feature
The pseudocode goes thus:

User signed up with email and password
Collect the email address of the user
Write a logic that generates token and concatenate to a url
Setup email protocol to transport the url to user mailbox
That is all for this part

Additional context
This request is 1 of 3 that will handle the email verification.
We can talk about the others later.

You will also have to setup nodemailer and send html to confirm account
Will u do that at once for the complete 3 points?

I will not be using NodeMailer instead SendGrid. I will do that for 3 points fine.

It's yours

Great.


unknown css property horizontal-align

Issue Type: Bug
In ccs file unknown property warning error  for horizontal-align:center.  If changed into vertical-align:center this works fine.
file.css:
.textField {
horizontal-align:center;
}
I use lates VS Code with enabled from MarketPlace: Debugger for chrome and Nativescript.
I tried Installing CSSLINT;this makes no difference. Deinstalling it doesnt solve the problem either.
The code is compiled ok and works fine. The warning message is disturbing however.
VS Code version: Code 1.32.1 (05f146c7a8f7f78e80261aa3b2a2e642586f9eb3, 2019-03-08T00:51:49.278Z)
OS version: Windows_NT x64 10.0.17763

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz (8 x 1992)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.88GB (8.01GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

debugger-for-chrome
msj
4.11.3

nativescript
Tel
0.9.0

This is not an issue with vscode. CSS has no property that is called horizontal-align.
Maybe this helps you: CSS Layout - Horizontal & Vertical Align

I am sorry, but your answer is mistaken: vertical-align is a CSS property. It is even shown in the link you sent, scroll down and see:
 /* If the text has multiple lines, add the following: */
.center p {
  line-height: 1.5;
  display: inline-block;
  vertical-align: middle;
}

I am not using CSSLint, and therefore I wonder which component is triggering the error. Please help to find a solution.

@jokro you are absolutely right: vertical-align is a CSS property, horizontal-align is not.

 It was caused by the fact that Nativescript
both supports vertical-align and horizontal-align. I now understand why VS does not suport this, because it is not standard. Thx


Errors after updating ComfyUi

After updating ComfyUi to 2590[5e806f] (2024-08-21) the following error prevents the node from loading:
ImportError: cannot import name 'sd2_clip' from 'comfy' (unknown location)
When i fix it via a symlinked copy of sd2_clip.py from ComfyUI/ComfyUI/comfy/text_encoders/ to ComfyUI/ComfyUI/comfy/ this error is prevented but gives another error:
!!! Exception during processing !!! ModelPatcher.__init__() got an unexpected keyword argument 'current_device' Traceback (most recent call last):   File "/home/geistt/Applications/ComfyUI/ComfyUI/execution.py", line 316, in execute     output_data, output_ui, has_subgraph = get_output_data(obj, input_data_all, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)   File "/home/geistt/Applications/ComfyUI/ComfyUI/execution.py", line 191, in get_output_data     return_values = _map_node_over_list(obj, input_data_all, obj.FUNCTION, allow_interrupt=True, execution_block_cb=execution_block_cb, pre_execute_cb=pre_execute_cb)   File "/home/geistt/Applications/ComfyUI/ComfyUI/execution.py", line 168, in _map_node_over_list     process_inputs(input_dict, i)   File "/home/geistt/Applications/ComfyUI/ComfyUI/execution.py", line 157, in process_inputs     results.append(getattr(obj, func)(**inputs))   File "/home/geistt/Applications/ComfyUI/ComfyUI/custom_nodes/ComfyUI_ModelScopeT2V/nodes.py", line 214, in load_modelscopet2v     model = load_modelscope_checkpoint(   File "/home/geistt/Applications/ComfyUI/ComfyUI/custom_nodes/ComfyUI_ModelScopeT2V/nodes.py", line 104, in load_modelscope_checkpoint     model_patcher = comfy.model_patcher.ModelPatcher( TypeError: ModelPatcher.__init__() got an unexpected keyword argument 'current_device'

Same here.

The easy fix is to go to configs/modelscope_config.py and replace
from comfy import sd2_clip

with
from comfy.text_encoders import sd2_clip

For reference, this is the ComfyUI breaking change that caused the issue -> https://github.com/comfyanonymous/ComfyUI/commit/4ba7fa0244badcf901f2b8ddbfb8539c6398672f

The easy fix is to go to configs/modelscope_config.py and replace
from comfy import sd2_clip

with
from comfy.text_encoders import sd2_clip

For reference, this is the ComfyUI breaking change that caused the issue -> comfyanonymous/ComfyUI@4ba7fa0

thank you this worked!

anyone still not able to fix with latest comfy?

still havent found a fix?

i solved the issue for me, this is a temporary fix and i don't know if this break  anything, but for now its working for me:
in ComfyUI\custom_nodes\ComfyUI_ModelScopeT2V\nodes.py
change: current_device=inital_load_device
to: #current_device=inital_load_device
i just put the hashtag there because i dont wanna remove it incase it'll break some things.
so simply deleting the argument worked for me, i hope it works for you guys too and if theres a better fix let me know!
and im very sorry for the messiness of my message, im new to github

thanks @soundboi123
got it running with these two fixes
go to configs/modelscope_config.py and replace:
from comfy import sd2_clip
with
from comfy.text_encoders import sd2_clip
in ComfyUI\custom_nodes\ComfyUI_ModelScopeT2V\nodes.py
change: current_device=inital_load_device
to: #current_device=inital_load_device

thanks @soundboi123
got it running with these two fixes
go to configs/modelscope_config.py and replace:
from comfy import sd2_clip with from comfy.text_encoders import sd2_clip
in ComfyUI\custom_nodes\ComfyUI_ModelScopeT2V\nodes.py change: current_device=inital_load_device to: #current_device=inital_load_device

glad this works!


Make a rule to change Area Path

Make a rule to be able to change Area Path, today you can only choose Area ID, there is no way to find your Area ID, but you may want to change the Area Path when a task is put in a specific state.
(Azure DevOps)

Might be done in Logic apps instead.


Docker integration

Hi all,
I was trying to launch some Rapid nodes using Docker, but every node is throwing an exception saying that it cannot join.
Basically, I've compiled using maven and Java 8 and copied the JAR files to a Docker image. From what I could inspect, the JARs have all the dependencies needed.
I've attached a zip file with my Docker file, the container's script and a log of a previous attempt.
files.zip
Do you have any idea of what is causing this issue?
Thank you

Thanks for submitting an issue.
I looked over your script and don't see anything wrong with the java command you're running to start Rapid nodes.
Can you double check if the seed node is brought up before the others (the node with listenAddress == seedAddress)?

I checked again and the seed node is launch immediately, while the other nodes wait 60 seconds to be launched.
Also, the seed node has listenAddress == seedAddress but the issue remains.
PS: The containers are using an overlay network, but I guess this shouldn't be the problem.

It shouldn't be an issue, I've ran it on overlay networks before.
To make sure it's not the overlay network, can you try to bring up the nodes outside of docker on a single machine?

Running 3 nodes, in a single machine, as described in the README file results in every node being stuck.
Or at least, it seems that way. I'm not sure if it's a log4j problem.
Every node prints the following:
log4j:WARN No appenders could be found for logger (io.netty.util.internal.logging.InternalLoggerFactory). log4j:WARN Please initialize the log4j system properly. log4j:WARN See http://logging.apache.org/log4j/1.2/faq.html#noconfig for more info.

Yes, it can't find a log4j configuration file, and therefore does not print any information.
Here's an example log4j.configuration file you can use: https://gist.github.com/lalithsuresh/5abe5990e94c5e999550a107d67aa062
Then do:
java -Dlog4j.configuration=file:<path-to-log4j.properties-file> -jar examples/target/standalone-agent.jar --listenAddress <ip:port> --seedAddress <ip:port>

Have you verified the firewall settings on the overlay network? What about MTU sizes? I'm guessing here because the logs don't have enough info for me to go on.

@francisco-polaco thinking further about your issue, I wonder if your overlay network remapping ports (is there NAT involved)?

@lalithsuresh I checked the network configuration and I don't believe its anything related with NAT.
However, I forced java to use IPV4 and the results have changed. -- JVM flag -Djava.net.preferIPv4Stack=true
Could you check if this log represents the correct functioning of Rapid? Because I see a lot of exceptions and the cluster size went from 1 to 8.
I also send a network description hoping you can detect something that is making all these exceptions.
log.zip

It's reporting the right cluster size, but I suspect the default messaging layer we use (gRPC) is experiencing issues, judging from the errors you see. I'll dig deeper and get back to you.

Hey @lalithsuresh! Any progress regarding this issue?

My suspicion is that the default timeouts are too low, which causes some gRPC messages to timeout (during the join phase, because they take the longest). This then causes a bunch of late responses to show up at nodes, which gives the unknown stream errors.
I usually set messaging timeouts on my own when running Rapid experiments. Can you please try the version here: https://github.com/lalithsuresh/rapid/tree/issue-13?

As before, a lot of exceptions related to deadlines were thrown, but in between exceptions some nodes were added to the membership. I've bootstrapped 10, but the cluster size is 8.
pure.log

Yeah, a join timing out after 20 seconds is definitely pathological -- something is definitely wrong with gRPC's messaging between nodes here. Any instructions on how I can reproduce your docker setup?

I set up something using the Dockerfile you'd shared earlier and I can reproduce the behavior you're seeing. I'll investigate and keep you posted!

I take that back. I'm still not able to reproduce it. :)
Can you share your docker workflow here so I can try it out too?
This is what I tried and it works for me:
$: sudo docker network create --driver bridge isolated_nw1 --subnet <IP_ADDRESS>/16

# In different terminals
$: docker run --network isolated_nw1 --ip <IP_ADDRESS> rapid-docker:latest
$: docker run --network isolated_nw1 --ip <IP_ADDRESS> rapid-docker:latest
$: docker run --network isolated_nw1 --ip <IP_ADDRESS> rapid-docker:latest
$: docker run --network isolated_nw1 --ip <IP_ADDRESS> rapid-docker:latest

The network driver I am using is Overlay, creating a "closed" cluster.
Here you can find a network inspection
eptonet2.txt

@francisco-polaco : can you share the commands you used to create the network and deploy the swarm?

In this case, the swarm only have one node, since I'm trying to run it on my local machine.
I'm using swarm since my python script is ready to deploy the system in the cloud. The following commands are the conversion of the python's SDK to bash.
Swarm creation:
docker swarm init
Network creation:
docker network create --driver "overlay" --scope "swarm" --attachable --internal eptonet
Service launching:
docker service create --replicas 10 --name rapid rapid
Then use docker network connect to connect the containers to the created network. Keep in mind that the seed IP in the containers script may change.

Still seems to work for me! https://gist.github.com/lalithsuresh/40f57f402c14e9716ac0bfbe04810069
Note that some of the cancellations reported by gRPC are benign (for example, during the join phase, the joiner makes progress as soon as it hears from even one of its temporary observers -- the remaining responses get dropped).
If you however see a lot of messages with DEADLINE_EXCEEDED, that's bad.

If it helps:
$: docker --version
Docker version 18.06.1-ce, build e68fc7a

I am running the same docker version as you and this time I've run the commands by hand.
There is a lot of DEADLINE_EXCEEDED messages and the cluster size is not right.
STDOUT: https://gist.github.com/francisco-polaco/1b31078a8d7613b6255530d784f12450 STDERR: https://gist.github.com/francisco-polaco/724626e7faf6ee733c4b012262d11f85
Is it maybe something related to my compiled jar? Could you send yours?

You are building the right jar, judging from the timeouts being printed. It seems like even the most simple RPCs (like the first join request message) are timing out in your trace, whereas it isn't the case on mine. I find this odd, given that the timeouts are high. That would explain not just the DEADLINE_EXCEEDED messages, but also the "Received DATA frame for an unknown stream " errors in your STDERR (because the caller receives a response after the timeout).
Let's try increasing the timeouts one more time. Can you pull the latest version from the issue-13 branch? I've bumped up some of the timeouts to 5 seconds.
By the way, please note that the StandaloneAgent code terminates after 400 seconds -- which causes docker swarm to keep recreating the containers.

Again, the same behavior https://gist.github.com/francisco-polaco/64137ef4cf07a8074dc81ce31353eb6f
The last file is the commands I am using to set up the experiments.
To compile it, I am running mvn clean package with oracle-jdk-8

Your compilation is fine.
The previous command you'd shared with me was not using the overlay network. I'll retry again with the new commands you'd shared and get back to you.
One question: how are you forcing swarm to create a container with the first node's IP?

I just delete and create the network again. The IPs are reset.
Then, if you launch, for example, 10 replicas, you know that one replica will have an IP between <IP_ADDRESS> and <IP_ADDRESS>. Docker does this because if you launch more replicas after the initial ones, their IPs will be always consecutive.
So in the container-start-script.sh you replace the variable FIRST_NODE_IP by something like <IP_ADDRESS> and that node will be the seed node. If you replace by that IP, I think it should work like a charm :)

Thanks! I'm able to reproduce the effect now.
I tried two runs:
It works fine with this:
$: docker service create --replicas 10 --name rapid-docker --network teste rapid-docker:latest

But not when I use the flag you had to limit CPU utilization:
$: docker service create --replicas 10 --name rapid-docker --limit-cpu 0.1 --network teste rapid-docker:latest

Can you double check?

Yup the culprit was the CPU limit. In my python script, I also was limiting the CPU.
I would never figure out the cause of the problem.
I tried with the master branch and everything worked as well.
Thanks!! :smile:

Perfect. I'll close the issue now. Thanks for your patience!

@francisco-polaco : Thanks to what you found, I ended up profiling Rapid to see why limiting CPU resources caused it to react that badly. One culprit was the excessive hash computations in MembershipView which I've cut down almost entirely with some caching. Rerunning Rapid in a docker container with --limit-cpus 0.10 still throws some gRPC exceptions (with the higher timeouts on the issue-13 branch though), but the cluster does reach the right size of 10 every time. I've cherry picked that commit to the master branch.

Hashes can be quite heavy when computed multiple times. I also had the same problem with a software I was implementing.
I'm glad you found the cause for the CPU usage :) Nevertheless, I am still experimenting with rapid and the same limits with more nodes, still cause gRPC exceptions, but without reaching the right cluster size :(

@francisco-polaco Yes, I still see them too. My patch doesn't fully eliminate that issue. The bigger problem is that there are multiple threads, mainly from gRPC (which is based on Netty, which polls continuously to check for messages to send/receive). Not good if there are limited CPU resources. :)


[Bug]: Inconsistent Header Section on Home Page

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
I’ve seen that the header at the top of the home page looks different from the headers on the other pages. This difference can make the website feel less organized and confusing for users.
Add Screenshots
Header Section of Home Page

Feedback, Content, Team, Features,  Page Header

Content Page, Team page, header,
What browsers are you seeing the problem on?
Chrome
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

already assigned for navbar


Regression test for cascade persist of associations (#909)

@malarzm: I noticed that #909 had been fixed by some recent commits, but thought it be good to include the original test case.

:+1:


[NG] build for Angular 5

Current Status: jsdoc annotations, tsickle emits warnings, handled as non-zero exit code by ngc
Solved:

[x] dherges/ng-packagr#369

BREAKING CHANGES
Due to @angular/compiler: "^5.0.0", AoT metadata will be in version: 4. Version 4 Metadata targets Angular 5 and is incompatible with Angular 4!

@hippee-lee FYI

@dherges - what makes you think Source File causing the error: src/clarity-angular/button/button-group/button-group.ts is the cause? The two properties in there are initialized to empty arrays.
    menuButtons: Button[] = [];

The error is very general and so far I've been unable to locate its source.

I am trying to find a repro but can't. The code isn't really the cause, it's just where it happens...

Understood that's what I suspected. Thanks for the follow-up.

@hippee-lee WE'Ve advanced error further down!
tsickle reports warning for your jsdoc annotations and thus ngc has a non-zero exit code
src/clarity-angular/button/button-group/button-group.ts(93,5): warning TS0: the type annotation on @return is redundant with its TypeScript type, remove the {...} part
src/clarity-angular/button/button-group/button-group.ts(208,5): warning TS0: @type annotations are redundant with TypeScript equivalents
@private annotations are redundant with TypeScript equivalents
src/clarity-angular/button/button-group/button-group.ts(208,5): warning TS0: @type annotations are redundant with TypeScript equivalents
@private annotations are redundant with TypeScript equivalents
src/clarity-angular/data/datagrid/datagrid-hideable-column.ts(11,1): warning TS0: @class annotations are redundant with TypeScript equivalents
src/clarity-angular/data/datagrid/datagrid-hideable-column.ts(20,5): warning TS0: @type annotations are redundant with TypeScript equivalents
src/clarity-angular/data/datagrid/datagrid-hideable-column.ts(30,5): warning TS0: @constructor annotations are redundant with TypeScript equivalents
src/clarity-angular/data/datagrid/datagrid-hideable-column.ts(46,5): warning TS0: @function annotations are redundant with TypeScript equivalents
the type annotation on @return is redundant with its TypeScript type, remove the {...} part

Here is the list of jsdoc annotations being reported
The line numbers in the error messages may be wrong because of the resource inlining / source rewrite by ng-packagr.

@dherges - thanks for the hint on jsdoc annotations.
I was able to make a small mod and see the errors when building.
We are also going to bump ng-packagr and angular versions in the rename PR. Thanks for both ng-packagr project and the input here! I really appreciate it.
I'll leave this open until those go in.

@dherges - I made these changes in our rename pull request that upgraded the ng-packagr version so I'm closing this pull request as it also relates to issue #850.
Thanks again for the wonderful ng-packagr project!


How to let the texts align on left?

Can we make the action texts align on left?

It is not supported to modify button view style.. If you need it, I will add the interface later and will inform you


[BUG] configuration volume is None

Hello,
I encountered a bug when migrating from bTrack 0.6.3 to 0.6.4 in the method tracker.to_napari().
Before, I could get away with not setting the tracker volume as setting it created some sort of bug, with for example:
 tracker.volume = ((0,volume[0]), (0,volume[1]), (-1e5, 1e5))

used to give weird ghost points in the trajectories (I suppose a problem of interpretation of 3D points in a 2D image, I don't know if it was a problem with my line or a bug). Anyway, I had no problem by not writing this line above.
With the new version 0.6.4, I get the following error when not setting the volume:
  File "<stdin>", line 9, in <module>
  File "/home/limozin/anaconda3/envs/btrack_last/lib/python3.11/site-packages/btrack/core.py", line 670, in to_napari
    assert self.configuration.volume is not None
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

The assertion in to_napari prevents me from using the default parameter (which is the following line in to_napari().
I'm not sure if this behavior is intended or not, maybe setting the volume has always been mandatory but I used to do it wrong or there was a problem in 0.6.3 that has been fixed in 0.6.4, provided I set the volume.
As a complement, this was my tracking config made with the napari-btrack plugin:
{
    "name": "cell",
    "version": "0.6.0",
    "verbose": false,
    "motion_model": {
        "measurements": 3,
        "states": 6,
        "A": [
            1.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            1.0
        ],
        "H": [
            1.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            1.0,
            0.0,
            0.0,
            0.0
        ],
        "P": [
            20.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            20.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            20.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            200.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            200.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            0.0,
            200.0
        ],
        "R": [
            5.0,
            0.0,
            0.0,
            0.0,
            5.0,
            0.0,
            0.0,
            0.0,
            5.0
        ],
        "G": [
            7.5,
            7.5,
            7.5,
            15.0,
            15.0,
            15.0
        ],
        "Q": [
            56.25,
            56.25,
            56.25,
            112.5,
            112.5,
            112.5,
            56.25,
            56.25,
            56.25,
            112.5,
            112.5,
            112.5,
            56.25,
            56.25,
            56.25,
            112.5,
            112.5,
            112.5,
            112.5,
            112.5,
            112.5,
            225.0,
            225.0,
            225.0,
            112.5,
            112.5,
            112.5,
            225.0,
            225.0,
            225.0,
            112.5,
            112.5,
            112.5,
            225.0,
            225.0,
            225.0
        ],
        "dt": 1.0,
        "accuracy": 7.5,
        "max_lost": 10,
        "prob_not_assign": 0.01,
        "name": "cell_motion"
    },
    "object_model": null,
    "hypothesis_model": {
        "hypotheses": [
            "P_FP",
            "P_init",
            "P_term",
            "P_link"
        ],
        "lambda_time": 5.0,
        "lambda_dist": 3.0,
        "lambda_link": 12.0,
        "lambda_branch": 50.0,
        "eta": 1e-10,
        "theta_dist": 20.0,
        "theta_time": 5.0,
        "dist_thresh": 150.0,
        "time_thresh": 10.0,
        "apop_thresh": 5,
        "segmentation_miss_rate": 0.05,
        "apoptosis_rate": 0.001,
        "relax": true,
        "name": "cell_hypothesis"
    },
    "max_search_radius": 150.0,
    "return_kalman": false,
    "store_candidate_graph": false,
    "volume": null,
    "update_method": 0,
    "optimizer_options": {
        "tm_lim": 60000
    },
    "features": [],
    "tracking_updates": [
        1
    ]
}

Thank you for your attention,

Hey @remyeltorro ,
Thanks for the report!
Just wondering, if you do this:
 tracker.volume = ((0, volume[1]), (0, volume[0]), (-1e5, 1e5))

does that get rid of the "ghost points"? btrack takes dimensions as XYZ, whereas most image stacks are ZYX. Also, I'm suprised that the tracking works properly if you set the volume to None. I can remove the line that checks, but that is currently the desired behavior.

Hello @quantumjot, thank you for your reply,
On version 0.6.4, setting the volume works perfectly without creating ghost points,
 tracker.volume = ((0, volume[1]), (0, volume[0]), (-1e5, 1e5))

I can also get away with not setting the -z dimension as such (since my data has no -Z dimension):
 tracker.volume = ((0, volume[1]), (0, volume[0]))

and it works like a charm. I think I just wanted confirmation that setting the volume was mandatory after version 0.6.4. The strange result I noticed was with version 0.6.3 but I migrated to 0.6.4 since then.
I think you can consider the case closed,
Best,


Code of Conduct (for contributors)

It would be good even in these early days of Textor to have a Code of Conduct for contributing to the project.
I'd recommend the Contributor Covenant. It's used by many open source projects, including Swift itself.
The one thing that would be needed is an email address where complaints could be sent, reviewed, and actioned. Otherwise the text is easy to copy into the repo. It's licensed under the Creative Commons Attribution 4.0 International Public License which requires attribution. The attribution is built into the Contributor Covenant text itself.

From GitHub's documentation: https://help.github.com/articles/adding-a-code-of-conduct-to-your-project/


Add overloading to bindLogger

Why
const log: Logger = bindLogger(logfn, 'info') returns a type that's Logger | undefined
What changed
Add overloading so that we can get instance of loggers easily.
Versioning

[ ] Breaking protocol change
[x] Breaking ts/js API change

Maybe the better thing to do here is to expose unbindLogger()


fix backward compatibility. In 1.2 version the sum was counted from l…

…owercase script

Not sure what this was for. I added the ability to update the checksums.


Using the API token

Versions
PHP version: 7.2.0
Laravel version: 6.18.1
So I am trying to use the bagisto APIs in another application but I am facing this problem.
When I set the token to true (when logging in) "http://<IP_ADDRESS>:8000/api/customer/login?token=true" I was able to get the token and every thing went like it should but now I am trying to use other APIs like getting the costumer information or making an order I get this error:

So I think that the problem I am facing is in using the auth token I tried putting it in the header under the Authorization key and making it start with the order Bearer in case it uses the auth like the passport way but I still got the same error.
I tried reading a lot about this problem and I always found this link https://forums.bagisto.com/topic/149/i-need-to-deliver-the-api-token-from-the-json-response-not-the-header/5, But I couldn't understand properly I mean the user here is using nuxt.js and I just need to use the Api in the android app.
Could you please help me, and tell me how to use the token?
Thank you in advance

Hi @mhd-yasser-haddad ,
First you need to do login from api/customer/login,
after that you got the  authorised token in the response  after that you are able to access the next api from token.

@shivam-webkul
Thank you for your response, but you didn't add anything to the things I just said
I mean as I stated I was able to get the token, but I was unable to use it in other APIs as I said earlier.
What I am trying to do is to use the token in other APIs without it being stored in the cookies (not like the web).

please check your token is generated or not if not then you need to generate the token from the following cmd:
php artisan key:generate
php artisan jwt:secret

run after then token generated
php artisan cache:clear
php artisan config:clear

ref - https://github.com/tymondesigns/jwt-auth/issues/1729
i hope you issue will be resolved.
Thanks

And just to add, There is the Accept header it have to be added to the request:
Accept: application/json


Error in VenvSelectLog

The file https://github.com/linux-cultist/venv-selector.nvim/blob/203c9046e1c0787ec00edfd456b30e4381afbfd1/lua/venv-selector/user_commands.lua is missing an import. The following line
https://github.com/linux-cultist/venv-selector.nvim/blob/203c9046e1c0787ec00edfd456b30e4381afbfd1/lua/venv-selector/user_commands.lua#L15
contains a reference to log, but there is no log = require('venv-selector.logger').

Its because log is a global variable from init.lua, so we dont have to include it in every lua file:
log = require 'venv-selector.logger'
Since init.lua always runs first, you shouldnt get any import errors. Did you just notice it in the code or do you have some error?

Actually its not a good idea either way to have it global. Could be causing conflicts and bugs.
I switched to having it as a local variable, included in every file. Thank you.
Update the plugin and you will get the change. :)

Updated - now I get the error "Please set debug to true in options to use the logger". I have set this to true, here is my config:
cache = {
	file = "~/.cache/venv-selector/venvs2.json",
},
options = {
	on_venv_activate_callback = nil,             -- callback function for after a venv activates
	enable_default_searches = true,              -- switches all default searches on/off
	enable_cached_venvs = true,                  -- use cached venvs that are activated automatically when a python file is registered with the LSP.
	cached_venv_automatic_activation = true,     -- if set to false, the VenvSelectCached command becomes available to manually activate them.
	activate_venv_in_terminal = true,            -- activate the selected python interpreter in terminal windows opened from neovim
	set_environment_variables = true,            -- sets VIRTUAL_ENV or CONDA_PREFIX environment variables
	notify_user_on_venv_activation = false,      -- notifies user on activation of the virtual env
	search_timeout = 5,                          -- if a search takes longer than this many seconds, stop it and alert the user
	debug = true,                               -- enables you to run the VenvSelectLog command to view debug logs
	-- fd_binary_name = M.find_fd_command_name(),   -- plugin looks for `fd` or `fdfind` but you can set something else here

	-- telescope viewer options
	on_telescope_result_callback = nil,   -- callback function for modifying telescope results
	show_telescope_search_type = true,    -- Shows which of the searches found which venv in telescope
	telescope_filter_type =
	"substring"                           -- When you type something in telescope, filter by "substring" or "character"		},
},
search = {
	mambaforge_envs = {
		command = "$FD '/bin/python$' ~/mambaforge/envs --full-path --color never -E /proc -a -L",
		type = "anaconda"
	},
},

As it turns out I had to wrap all the options into a field called settings, which I forgot to do. All sorted!

You need to have a settings lua table at the top, and all your config goes under that one. You probably forgot that one and that's why your settings are not registering. :)
See the Readme for examples.


Update v0.10

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

hi @bgalvao @janjagusch - this PR fixes the issues with https://github.com/conda-forge/dice-ml-feedstock/pull/4
please review and let me know if you need anything else to merge.


Improved types for Picker component

Hello @Bur0 :wave:,
Thank you for your work! This library looks nice and feels like a desired way of picking items on mobile. However, inaccurate typescript types ruin developer experience a lot. I tried to address this issue in my PR.
Quick summary:

Improved types for Picker component - removed never and any, made generic type for Picker props.
More strict typescript configuration - added "noImplicitAny" property.
Updated examples as needed.

Explanation:
Small tricks were used - PickerProps is now a generic type, it receives item type T, and ensures that all other properties are passed correctly.
In addition, renderListItem property is now optional only when type T has the field name because otherwise, the default renderer doesn't render anything: https://github.com/Bur0/react-native-actions-sheet-picker/blob/3a475fa90cb71658d27f5302cb43cb953685490b/src/components/Picker.tsx#L66

thanks, this fixed the problem on my typescript react native app


Allow specifying a modified theme color

Resolves #953.
This allows specifying all the modifiers that Powerpoint uses internally on theme colors, which is useful when using a powerpoint file as a template for output generated with pptxgenjs.
Haven't updated the docs yet. I wanted to file this draft PR to et any feedback on the design itself first.

Hi,
Looks like this is what i'm searching for. Do you have a exemple of use ?


U2F/FIDO does not work with Vivaldi browser

Gitea version (or commit ref): 1.14.5
Git version: 2.20.1
Operating system: Debian 10 (using gitea download with systemd)

Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[X ] No (didn't receive any account activation mail)

Log gist:

There's nothing u2f-related in the debug-log.

Description

Using u2f with Firefox browser on my gitea instance works very well. u2f with Vivaldi on other sites also works. But using Vivaldi on my gitea instance doesn't work at all. I tried to de-register and re-register my U2F device (Nitrokey FIDO2) but registering the device is not possible. Gitea tells me that my browser doesn't support U2F (which is not true).
Screenshots

Can you check in Vivaldi browser in JavaScript console what does this return:
window.u2f

and
window.u2f.sign

I hope I did this right. Here are the results:

window.u2f

undefined

window.u2f.sign

Uncaught TypeError: Cannot read property 'sign' of undefined
    at <anonymous>:1:12

navigator.userAgent

"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/92.0.4515.134 Safari/537.36"

#17957