Query to return species metadata

Current in the web app, I'm hardcoding in the species. It'd be good to have an endpoint to get the following info:
[
  ...,
  {
    "common": "Brewer's Yeast", // common name, human friendly, capitalized, for display purposes
    "scientific": "Saccharomyces cerevisiae", // scientific name, also for display purposes
    "search": "brewers-yeast" // key that is put into species search endpoint url
  }
  ...,
]

Also side note, Google seems to say that "fruit fly" is two words, so query?species=fruitfly should probably be query?species=fruit-fly.

Here's the PR that I'm working on to show you how I'm hard coding it currently
https://github.com/biothings/mygeneset.info-website/pull/14/files#diff-ae13e3f616decf65761c7604f81fcf8e157404f63e6837b4405967a04c5766ae

Looks like I already have the info I need to do this myself with existing APIs:
https://mygene.info/v3/query?q=*&facets=taxid
http://t.biothings.io/v1/taxon/9913

A better way to obtain all the species that contain at least one gene:  http://t.biothings.io/v1/query?q=has_gene:true&fields=taxid,scientific_name

Since there are so many organisms (> 24k), I wont be able to do the nice icons for each. However, perhaps we could do a selection of the most commonly used ones. @ravila4 could you put the top 100 organisms here in order?
In terms of what metric is considered "top", would that be "most genes of that organism"? Or "most searched for"? Or some other metric?

What about number of gene sets? I'm not sure if we have a way to query that efficiently.

We can count the number of genesets we have for each species using this query: https://mygeneset.info/v1/query?q=*&facets=taxid&facet_size=100
It's not a perfect method, because facet_size has a maximum value of 1000, but it 's not a big problem for now, as we don't have that many species in mygeneset.
I wrote a small bash script to get a csv of counts for each species in mygeneset and their scientific names:

#!/bin/bash

# Genesets aggregated by taxid
aggs=`curl -s "https://mygeneset.info/v1/query?q=*&facets=taxid&facet_size=100"`
taxids=`echo $aggs | jq -r '.facets.taxid.terms | map(.term) | @csv'`
counts=`echo $aggs | jq -r '.facets.taxid.terms | map(.count) | @csv'`

# Query scientific name for each taxid
resp=`curl -s -X POST -d "q=${taxids}" "http://t.biothings.io/v1/query"`
species=`echo $resp | jq -r 'map(.scientific_name) | @csv'`

echo "${taxids}
${species}
${counts}" | rs -c, -C, -T

Here's the output:

taxid
scientific name
number of genesets

9606
homo sapiens
59283

9031
gallus gallus
14771

9913
bos taurus
14520

9823
sus scrofa
14158

9615
canis lupus familiaris
13942

10090
mus musculus
716

10116
rattus norvegicus
669

7955
danio rerio
434

559292
saccharomyces cerevisiae s288c
401

3702
arabidopsis thaliana
366

6239
caenorhabditis elegans
339

208964
pseudomonas aeruginosa pao1
329

7227
drosophila melanogaster
328

9598
pan troglodytes
46

180454
anopheles gambiae str. pest
14

3694
populus trichocarpa
5

9796
equus caballus
5

39947
oryza sativa japonica group
1

@vincerubinetti , if you need to query many  taxids at the same time, it will be faster to use a batch POST request as in the example.

@ravila4 Is there a reason you did http://t.biothings.io/v1/query?q=TAXID rather than http://t.biothings.io/v1/taxon/TAXID? With the former, tax id 9031, for example, returns "carpinus sp. wen 9031" (tax id of 559512) as the first result, and "gallus gallus" (tax id of 9031) as the second result.

@vincerubinetti Because POST queries only work with the query endpoint, not taxon.
There is also some difference in the fields that GET and POST queries search by default.
The first query using GET GET: http://t.biothings.io/v1/query?q=9031  would search all the default fields, including taxid, and scientific name. The way to limit it to searching only taxid is to use q=taxid:9031. I suspect that is why you are getting "carpinus sp. wen 9031" as a response.
However if you use POST method, the default search field is _id, which is the same as taxid for this API, but you can also be explicit and pass a scopes=taxid field .
Here's an example:
POST: http://t.biothings.io/v1/query?q=9031,9606&scopes=taxid

Here's some documentation on POST queries from mygene.info: https://docs.mygene.info/en/latest/doc/query_service.html?highlight=post#batch-queries-via-post
The same should apply to the taxonomy API, and mygeneset.


Mitotic cell cycle - alt id for mitotic nuclear division

GO:0007067 'mitotic nuclear division'
has
alt_id: GO:0007067
But this was the old id for
GO:0140014 'mitotic nuclear division'
So, GO:0007067 should be made secondary for GO:0140014 'mitotic nuclear division'
and removed from
GO:0007067 'mitotic nuclear division'
We need to do a GO update tomorrow, so can you make this high priority.

Hi @hattrill
GO:0007067 is the alt_id for GO:0000278 (mitotic cell cycle)
and I did create a new term that represents GO:0140014 'mitotic nuclear division'
The explanation is here: https://github.com/geneontology/go-ontology/issues/11709
So indeed some annotations were mapped 'up'. Is this not OK for you ?
Thanks, Pascale

The problem is that "mitotic nuclear division" was GO:0007067. Which means that we have to resolve the conflict in the database that we have the same name space with a different id and the same id as an alternative to something else.
I'll have to see if we can get round this somehow.

Hi Pascale,
Leave it for now - I'm going to check with our developer - we've run into a couple of similar issues with complex edits like this before (mistakes and intentional). Hopefully she can make a work around now we know the lay-of-the-land.
Thanks, Helen

A similar thing happened in in the merge of 'cilium assembly' and 'cilium morphogenesis' where the IDs got swapped in the sense that the ID that was kept as primary originally had the term name that became a synonym, and the new primary term name was originally associated with what became the alt_id. It would be really good to have some documentation for what is desired practice, and how to avoid the unintentional situations since @ukemi said it is trickier to do merges in Protege than it was in OBI-Edit.

‘chromatin modification' was another one.

Hi @hattrill
How did the release go? Should I close this ?
Thanks, Pascale

Hi Pascale,
Leave it open for now. Our dev is currently running checks on the file and then we will work through the problems by some process - I hope.
It takes us a long time to do the update of the GO in chado - usually takes about a week as we check it, load it into a test instance and review the impact on existing data. If all goes well, we will load it into production chado on Thursday and I should know the outcome by Friday.

Hello @hattrill
I am reassigning this to you - you can decide when it should be closed or let me know if I need to do something.
Thanks,
Pascale

That's fine - think we're almost there!
Thanks,
Helen

All fixed and fine!

:)


ros-indigo-segment-and-classify_binarydeb_trusty_amd64 misses more dependencies

OK, I have fixed a number of dependencies already, but here's another one in segment-and-classify somebody who knows it better than I should fix.
/tmp/buildd/ros-indigo-segment-and-classify-0.0.9-0trusty-20151124-1019/src/test.cpp:9:29: fatal error: pcl_conversions.h: No such file or directory
 #include <pcl_conversions.h>

see https://lcas.lincoln.ac.uk/jenkins/job/ros-indigo-segment-and-classify_binarydeb_trusty_amd64/3/console

all other wrapper built and are now released again, I wanted to add. Also relevant for @mzillich


[Improvement] Provide git support in composer

 Issue by erandidesilva
Friday Nov 17, 2017 at 07:57 GMT
Originally opened as https://github.com/ballerina-lang/composer/issues/4639

When switching between branch1 and branch2, currently the composer does not load the files accordingly. Since switching may cause some changes in files, it's better to reflect this in composer. Hence it would be good to provide git support by reloading files automatically when a change is detected.

 Comment by NipunaMarcus
Monday Nov 20, 2017 at 05:46 GMT

There is a file sync menu item in the file explorer menu. User can use that to sync the project with the file system's latest updates. We will work on the feature for reloading project structure automatically.

Composer is discontinued hence closing the issue.


Bug: Scale option don't like floatting numbers

https://github.com/A-tG/VoicemeeterFancyOSD/issues/20
https://github.com/A-tG/VoicemeeterFancyOSD/releases/tag/v<IP_ADDRESS>
; Scale of OSD. Maximum: 2, minimum: 0.5
Scale = 1.2

This version does not like floatting numbers and rounding to an integer.

It works fine if I use comma, instead of dot. I'm not sure what the best practice is here for these types of configurations tho :)

Probably because it was trying to parse according to Region settings. Should be fixed now in <IP_ADDRESS>


Train Using Precomputed RPN Proposals?

Do you plan to support training Using Precomputed RPN Proposals?

I have finished the code for training Using Precomputed RPN Proposals. And multi-scale train and test, and

I have finished the code for training Using Precomputed RPN Proposals.
Detectron: Fast & Mask R-CNN Baselines Using Precomputed RPN Proposals
R-50-FPN 1x :33.7
Use this pytorch code:
R-50-FPN 1x :33.76
And multi-scale train and test, and some code of PANet.
My code is ugly, so I didn't organize the code and integrate it into this repo.
https://github.com/qinhuan/mask-rcnn.pytorch_v0

Hi, do you know how to test the network with the given proposals?


created function to calculate person age

#59 Created a function that calculates a personage by birthday that accepted the date object function.

Looks good.  Next time don’t merge it as I’ll provide comments (code review) on your Pull Request and then you’ll write more tests.
Tonté

On Apr 11, 2021, at 6:54 PM, Susie @.***> wrote:
Merged #60 https://github.com/byte-pushers/bytepushers-js-core/pull/60 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/byte-pushers/bytepushers-js-core/pull/60#event-4579140865, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFCN443NK5WTONJRS6AP5DTIIZBRANCNFSM42YGM5XA.


docs: update ng test command

Related to: https://github.com/angular/angular-cli/issues/13154

I opened #175 and were happy that you might have provided the solution, but I still have the same problem of unverified breakpoints. But maybe im an exception.

@WtfJoke, it does. I'll update it.

LGTM. Thanks @alan-agius4

Also experiencing this issue.  I read through issue #175 and this one but when the debugger starts and tests run, all breakpoints are showing as unverified.


go.mod: unexpected module path "k8s.io/sample-controller"

What happened:
The Error about go module happened when I built cnat-controller by go build -o cnat-controller ..
What you expected to happen:
I can build cnat-controller successfully.
How to reproduce it (as minimally and precisely as possible):
$ go build -o cnat-controller .
go: finding github.com/programming-kubernetes/cnat/cnat-client-go/pkg/generated/listers/cnat/v1alpha1 latest
...
go<EMAIL_ADDRESS>parsing go.mod: unexpected module path "k8s.io/sample-controller"
go: error loading module requirements

Anything else we need to know?:
I can build successfully cnat-controller when I replaced go.mod module path from k8s.io/sample-controller to github.com/programming-kubernetes/cnat/cnat-client-go.
-module k8s.io/sample-controller
+module github.com/programming-kubernetes/cnat/cnat-client-go

Environment:
Mac OS X 10.14.5
go version go1.12.9 darwin/amd64
Thank you.

Hit the same problem. Had to rename the module package to the github.com/programming-kubernetes/cnat/cnat-client-go.
Hope the author can update the instruction soon.

Thanks for flagging this @govargo and for confirming it @gangchen03! This was indeed an oversight of mine, fixing it now.


Multiple fixes to apiv1swagger.json

Description
This PR fixes the following issues with the Swagger specification:

Fixed 2 typos that refer to non-existing response 'TooManyRequsts'
Updated description of PUT method for rule
Updated description of PUT method for trigger
Created new definition ActivationBrief
Changed response type for GET activations to refer to ActivationBrief instead of EntityBrief, which didn't match the actual response
Added attribute statusCode to definition ActivationBrief and Activation
Modified description of GET activations (#2271)
Added operationId to GET activation logs and result (#318)
Removed the init attribute from the ActionExec definition (#4201)
Use of 2 spaces for indentation, which is the default when working with the Swagger editor, instead of the four spaces previously used

Related issue and scope

[x] Doc issue with "Get activation IDs" (#2271)
[x] In Swagger file, operationId is missing for activation logs and results (#318)
[x] Update swagger for action put #4201

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[x] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4294 into master will increase coverage by 21.17%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #4294       +/-   ##
===========================================
+ Coverage   59.79%   80.97%   +21.17%
===========================================
  Files         163      163
  Lines        7594     7594
  Branches      502      502
===========================================
+ Hits         4541     6149     +1608
+ Misses       3053     1445     -1608

Impacted Files
Coverage Δ

.../org/apache/openwhisk/core/entity/EntityPath.scala
100% <0%> (+1.88%)
:arrow_up:

...a/org/apache/openwhisk/core/entity/Parameter.scala
95.45% <0%> (+2.27%)
:arrow_up:

.../org/apache/openwhisk/http/PoolingRestClient.scala
90% <0%> (+3.33%)
:arrow_up:

...hisk/core/controller/actions/SequenceActions.scala
92.1% <0%> (+3.5%)
:arrow_up:

...pache/openwhisk/core/containerpool/Container.scala
88.6% <0%> (+3.79%)
:arrow_up:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
4% <0%> (+4%)
:arrow_up:

...tainerpool/docker/DockerClientWithFileAccess.scala
96% <0%> (+4%)
:arrow_up:

...rg/apache/openwhisk/core/entity/ExecManifest.scala
97.87% <0%> (+4.25%)
:arrow_up:

...he/openwhisk/core/database/CouchDbRestClient.scala
87.23% <0%> (+4.25%)
:arrow_up:

...a/org/apache/openwhisk/core/entity/TimeLimit.scala
95.45% <0%> (+4.54%)
:arrow_up:

... and 92 more

Continue to review full report at Codecov.

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

Travis build is failing multiple tests:
org.apache.openwhisk.core.containerpool.test.ContainerProxyTests > ContainerProxy should stay in Running state if others are still running FAILED
    java.lang.AssertionError: assertion failed: expected:  but got unexpected message Transition(Actor[akka://ContainerProxys/system/testActor-10/$g#670589197],Running,Ready)

org.apache.openwhisk.core.containerpool.test.ContainerProxyTests > ContainerProxy should complete the transaction and reuse the container on a failed run IFF failure was applicationError FAILED
    java.lang.AssertionError: assertion failed: expected class akka.actor.ActorRef, found class akka.actor.FSM$Transition (Transition(Actor[akka://ContainerProxys/system/testActor-10/$g#670589197],Ready,Running))

org.apache.openwhisk.core.containerpool.test.ContainerProxyTests > ContainerProxy should complete the transaction and abort if container creation fails FAILED
    java.lang.AssertionError: assertion failed: expected class akka.actor.ActorRef, found class org.apache.openwhisk.core.containerpool.NeedWork (NeedWork(WarmedData(ContainerId(testcontainer),invocationSpace,ExecutableWhiskAction/actionSpace/actionName@0.0.1,2019-02-18T09:26:42.302Z,1)))

...and more
Surely not because of changes to the Swagger??

The first commit is from another PR - should be dropped here?

Note to reviewers: Use w=1 to ignore white space diffs. https://github.com/apache/incubator-openwhisk/pull/4294/files?utf8=✓&diff=unified&w=1

@rabbah Sorry about the extra commit, that was for the other PR (#4295). Removed that one from this PR.


[Question]: Not able to import Modulus sym classes

Is this a new feature, an improvement, or a change to existing functionality?
New Feature
How would you describe the priority of this feature request
Critical (currently preventing usage)
Please provide a clear description of problem you would like to solve.
installed the latest version of Modulus is using PyPi.
But, unable to add classes that a common docker installation can implement, as follow
from modulus.continuous.constraints.constraint import (
PointwiseBoundaryConstraint,
PointwiseInteriorConstraint,
IntegralBoundaryConstraint,
PointwiseConstraint,
)
What if I want to do this in nvidia-modulus 0.4.0?
Describe any alternatives you have considered
No response

Hi @YouqiongLiu
Did you install Modulus symbolic using:
pip install nvidia-modulus.sym

Thx @NickGeneva
I have installed nvidia-modulus: 0.4.0 and the latest version of nvidia-modulus.sym is using PyPi.
But, the following programs cannot be run . Error importing pysdf. Make sure 'libsdf.so' is in LD_LIBRARY_PATH and pysdf is installed.
How can I solve this problem?
import modulus.sym
from modulus.sym.geometry.tessellation import Tessellation

Hi @YouqiongLiu
PySDF is only available inside the Modulus docker container. Not the python package. We recommend using the docker container if you wish to use this feature.

I am using the docker container (latest image version 24.12) but still got the error:
from modulus.sym.geometry.tessellation import Tessellation
ModuleNotFoundError: No module named 'modulus.sym'
is it possible to make the modulus.sym available not only in the docker container? We want to benchmark our model on XAeroNet.


Missing properties from GetAllOptions methods.

I noticed an issue with IRealTimeOptionsQuote data model. It's a simple fix, however I am going to use this issue for learning some GitHub features.
Issue is present in v1.0.5

Issue resolved v1.0.6


BlockStorage does not work when block contains multiple transactions

Place this into tests/unit/BlockStorageMultiTx:
/*******************************************************************************

    Create blocks with multiple transactions and test that they are
    read properly.

    Copyright:
        Copyright (c) 2019 BOS Platform Foundation Korea
        All rights reserved.

    License:
        MIT License. See LICENSE for details.

*******************************************************************************/

module unit.BlockStorageMultiTx;

import agora.common.Amount;
import agora.common.Block;
import agora.common.BlockStorage;
import agora.common.crypto.Key;
import agora.common.Data;
import agora.common.Hash;
import agora.common.Transaction;
import agora.consensus.Genesis;

import std.algorithm.comparison;
import std.file;
import std.path;

/// The maximum number of block in one file
private immutable ulong MFILE_MAX_BLOCK = 100;

/// blocks to test
const size_t BlockCount = 300;

///
private void main ()
{
    string path = buildPath(getcwd, ".cache");
    if (path.exists)
        rmdirRecurse(path);

    mkdir(path);

    BlockStorage.removeIndexFile(path);
    BlockStorage storage = new BlockStorage(path);
    Block[] blocks;

    blocks ~= getGenesisBlock();
    storage.saveBlock(blocks[$ - 1]);

    Transaction[] last_txs;
    foreach (block_idx; 0 .. BlockCount)
    {
        // create enough tx's for a single block
        auto txs = makeChainedTransactions(getGenesisKeyPair(), last_txs, 1);

        auto block = makeNewBlock(blocks[$ - 1], txs);
        storage.saveBlock(block);
        blocks ~= block;
        last_txs = txs;
    }

    //// load
    Block[] loaded_blocks;
    loaded_blocks.length = BlockCount + 1;
    foreach (idx; 0 .. BlockCount + 1)
        storage.readBlock(loaded_blocks[idx], idx);
    size_t idx;

    assert(loaded_blocks == blocks);
}

/// Copied over from agora.common.Transaction
private Transaction[] makeChainedTransactions (KeyPair key_pair,
    Transaction[] prev_txs, size_t block_count)
{
    import agora.common.Block;
    import std.conv;

    assert(prev_txs.length == 0 || prev_txs.length == Block.TxsInBlock);
    const TxCount = block_count * Block.TxsInBlock;

    // in unittests we use the following blockchain layout:
    //
    // genesis => 8 outputs
    // txs[0] => spend gen_tx.outputs[0]
    // txs[1] => spend gen_tx.outputs[1]...
    // ..
    // tx[9] => spend tx[0].outputs[0]
    // tx[10] => spend tx[1].outputs[0]
    // ..
    // tx[17] => spend tx[9].outputs[0]
    // tx[18] => spend tx[10].outputs[0]
    // ..
    // therefore the genesis block and the 1st block are unique here,
    // as the 1st block spends all the genesis outputs via separate
    // transactions, and subsequent blocks have transactions which
    // spend the only outputs in the transaction from the previous block

    Transaction[] transactions;

    // always use the same amount, for simplicity
    const Amount AmountPerTx = 40_000_000 / Block.TxsInBlock;

    foreach (idx; 0 .. TxCount)
    {
        Input input;
        if (prev_txs.length == 0)  // refering to genesis tx's outputs
            input = Input(hashFull(getGenesisTx()), idx.to!uint);
        else  // refering to tx's in the previous block
            input = Input(hashFull(prev_txs[idx % Block.TxsInBlock]), 0);

        Transaction tx =
        {
            [input],
            [Output(AmountPerTx, key_pair.address)]  // send to the same address
        };

        auto signature = key_pair.secret.sign(hashFull(tx)[]);
        tx.inputs[0].signature = signature;
        transactions ~= tx;

        // new transactions will refer to the just created transactions
        // which will be part of the previous block after the block is created
        if (Block.TxsInBlock == 1 ||  // special case
            (idx > 0 && ((idx + 1) % Block.TxsInBlock == 0)))
        {
            // refer to tx'es which will be in the previous block
            prev_txs = transactions[$ - Block.TxsInBlock .. $];
        }
    }
    return transactions;
}

Maybe https://github.com/bpfkorea/agora/pull/158 fixes it?
But if so, then we cannot merge https://github.com/bpfkorea/agora/pull/162 before #158 is merged first.

I have a different type of failure with #158. The test hangs.

Ok the test hang with #158 was because of a wrong rmdirRecurse. I fixed the code above to:
diff --git a/tests/unit/BlockStorageMultiTx.d b/tests/unit/BlockStorageMultiTx.d
index ec0371a..44deb0b 100644
--- a/tests/unit/BlockStorageMultiTx.d
+++ b/tests/unit/BlockStorageMultiTx.d
@@ -39,8 +39,8 @@ private void main ()
     string path = buildPath(getcwd, ".cache");
     if (path.exists)
         rmdirRecurse(path);
-    else
-        mkdir(path);
+
+    mkdir(path);

The unittests in Henry's PR still fail with #158 though. So something still seems to be broken.

looked into the cause of this phenomenon.
Commit '64238cbf9f6877e9237096303dce8471143580d0', Change Map File Size'
This is a problem caused by reducing the size of the map file in .
Because multi-transaction is introduced and block size is increased, memory map size 64KByte is not processed.
Procedure is as follows.

To change to an existing size.
The other is what PR 158 is.

PR158 is an upgraded source that can be handled as the block grows in size.
First, let's add another PR that returns the memory map size to 640KByte.

I looked into the cause of this phenomenon.
Commit '64238cbf9f6877e9237096303dce8471143580d0', Change Map File Size'
This is a problem caused by reducing the size of the map file in .
Because multi-transaction is introduced and block size is increased, memory map size 64KByte is not processed.
Procedure is as follows.

To change to an existing size.
The other is what PR 158 is.

PR158 is an upgraded source that can be handled as the block grows in size.
First, let's add another PR that returns the memory map size to 640KByte.

Issue resolved with https://github.com/bpfkorea/agora/pull/158


Display GoDoc when autocompleting in Vim

Just a feature request:
When using omnicomplete for languages like python, vim can display the documentation for the function.
This would be awesome if gocode did it with godoc.
Here's an example of what the python autocomplete it looks like in vim:

Just chiming in to say this feature would be quite useful for GoClipse as well. In typical Eclipse fashion, doc comments can be show on the code completion popup:

Unfortunately I don't have the time either to try to find a solution. :cry:


General updates

Started off as Cardiology project demo magic, but in the end it's a lot more

@rickygv99 @elvzhangg I'm going to merge the branch right away because Travis is not set for tests yet. There might be more work, but that will come in more reviewable PRs, hopefully.


MenuFlyoutItem Tapped event callback method was not called

Describe the bug
The tapped event callback method of MenuFlyoutItem  was not called.
Steps to reproduce the bug
When i click on the MenuFlyoutItem  the Tapped event callback method was not called.
Expected behavior
No response
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.4.0: 1.4.230822000
Windows version
Windows 11 (22H2): Build 22621
Additional context
The sample app repo url : https://github.com/jhonToni/WinuiMenuFlyoutSample

This is happening in my application also.

@jhonToni: This behavior is by design - the Tapped event is only fired in response to touch (Mouse click should trigger 'Click' event).

That seems to be inconsistent with other controls, for example on a Button I can used the Tapped event and it works with the mouse.

@jhonToni: This behavior is by design - the Tapped event is only fired in response to touch (Mouse click should trigger 'Click' event).

This is not what the documentation describes:https://learn.microsoft.com/en-us/windows/windows-app-sdk/api/winrt/microsoft.ui.xaml.uielement.tapped?view=windows-app-sdk-1.4

Touch, mouse devices and pen devices can all produce a Tap action. For more info, see Handle pointer input.


Update README.md

Added a Preview Section to better Understanding of this project

@kumarankit999 please mention the issue no along with the pr.

@kumarankit999 please mention the issue no along with the pr.

@kumarankit999 ??

Because of no response from your side, I am closing the pr.


Issue with different table name or columns name

Hello,
I am including this package into my project but I have to modify a lot of things before it works... Because my users table is not in English and even my columns...
I think it will be a better way to place these data on the config file no?

You know what, the funny thing here is...
It is placed in a configure file.
As it is with laravel packages you need to publish the config with artisan command.

All what I see on config it is the Users Model location. I'm talking about table name and column name that you hard-coded on the source (Thread.php etc...)

You can overwrite it that is also quite easy to see if you look at the source but take a look at #39 that should clear things up for you.

Yes I know, I edited and it works perfectly but I think it will be better to include it on the source for these who don't know.
PW: I am using the example controller and I see that every message is visible for everyone not only for the participants... I think I have to use scopeForUser instead of getAllLatest method but not sure what to give on first paramter.

From the example file controller
$threads = Thread::forUser($currentUserId)->latest('updated_at')->get();
Where currentUserId is the logged in user

Cool it works :+1:


[SPARK-43158][DOCS] Set upperbound of pandas version for Binder integration

What changes were proposed in this pull request?
This PR proposes to set the upperbound for pandas in Binder integration. We don't currently support pandas 2.0.0 properly, see also https://issues.apache.org/jira/browse/SPARK-42618
Why are the changes needed?
To make the quickstarts working.
Does this PR introduce any user-facing change?
Yes, it fixes the quickstart.
How was this patch tested?
Tested in:

https://mybinder.org/v2/gh/HyukjinKwon/spark/set-lower-bound-pandas?labpath=python%2Fdocs%2Fsource%2Fgetting_started%2Fquickstart_connect.ipynb
https://mybinder.org/v2/gh/HyukjinKwon/spark/set-lower-bound-pandas?labpath=python%2Fdocs%2Fsource%2Fgetting_started%2Fquickstart_df.ipynb
https://mybinder.org/v2/gh/HyukjinKwon/spark/set-lower-bound-pandas?labpath=python%2Fdocs%2Fsource%2Fgetting_started%2Fquickstart_ps.ipynb

Merged to master and branch-3.4.

Merged to branch-3.3 too.


About old model

Hello. Thank you for your great work!
Is the old model used during training? or is it just used to fuse the learned model as post processing.
Thanks.

Hello!
You are right. The old model is not used for training. We only use the prototypes to mitigate forgetting during training.


fix misspelling in dist

A PR to fix spelling Xenial as Zenial in the travis config.

@dependabot rebase

Travis is no longer used in the deployment process! Closing this PR 😸


Update Settings design: Add new SettingsListItem

Task/Issue URL: https://app.asana.com/0/1207908166761516/1208785622935231/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5318  👈 (View in Graphite)
#5300
develop

This stack of pull requests is managed by Graphite. Learn more about stacking.

Android CI Test failing (not related to this work) but a fix has been raised: https://github.com/duckduckgo/Android/pull/5316


Fix my sessions clear when sessions update.

Issue

#282

Overview (Required)

Update to Orma 4.2.0-rc, Replace deletion and insertion to Relation#upsert().
Session has nullable associations (topic, speaker, room), It seems to be difficult to replace. So try insert once, and update if fail.

Links

https://speakerdeck.com/lvla/ormahamaripointo-star-v2-dot-5-2 (japanese)

Screenshot

None

:eyes:

Relation#upsert()

Interesting 🆒

@gfx Can you help to review? 🙇

Great PR 👍
However #upsert() should work but it's not complete. Can you wait for the next RC?

@gfx Thank you for review! Okay, we can wait to merge.
Do you have a plan to release next RC? If you can release next week, I'll keep this PR. But if you can't, I'll close this once.

Upsert now works! https://github.com/yatatsu/conference-app-2017/pull/1

After merge https://github.com/yatatsu/conference-app-2017/pull/1 , I'll review and merge this 👍

I merged yatatsu#1, which is cool feature from Orma v4.2.0. Please review it!

LGTM! 💯 Thanks @yatatsu @gfx
I'll merge this after CI Success

👏


Can't deploy to C8 SaaS

Describe the bug
I can't deploy to C8 SaaS

Steps to reproduce

Create a C8 diagram
deploy it to saas
It fails

Expected behavior
I can deploy to SaaS
Environment
Camunda Modeler system information

Version: 5.4.0-rc.1
Operating System: Windows 10 amd64
Plugins:
Execution Platform: BPMN - Camunda Platform 8

Additional context
zeebe-node has breaking changes in a minor update:

My appologies for blindly bumping zeebe-node. :see_no_evil:
Let's revert to 8.0.x to fix this.

I'll invest 30 minutes to see if we can use non-deprecated API. If this does not work, we can revert and re-release

closed via https://github.com/camunda/camunda-modeler/pull/3224


Mismatching docker-compose.yml mapped port and localhost port.

File: get-started/part3.md
Error on the docu.
In the docker-compose.yml code, the 80 "internal" port of the service is mapped to the 4000 external port on localhost
 ports:
      - "4000:80"

A few lines after, the documentation says:

You can run curl -4 http://localhost several times in a row, or go to that URL in your browser and hit refresh a few times."

The documentation needs to indicate the 4000 port on the curl parameter.
curl -4 http://localhost:4000

Closing this ticket due to its age, and the impending refactor. If you think this is in error, feel free to reopen. Thanks!


Finish resource implementation

Overview
These changes successfully implement the use of resources within Select components. Given how hard-coded the eventual JSON string returned by the resource controller is, I imagine more work will be necessary to allow a list of, say, directory search components but this is a good first step
Checklist

[ ] tests/ added or updated
[ ] CHANGELOG.md's Unreleased section is updated
[ ] Documentation is updated

Coverage decreased (-0.9%) to 89.602% when pulling ac6e23fb859a37d4eab32aff713e8478381105ae on finish-resource-implementation into cc8ab2db57af380b45a19f943117fcbc8b064c15 on develop.


Install libllbuild next to libllbuildSwift

libllbuildSwift only contains the Swift bindings, so we need to install
libllbuild as well. It would be nice if libllbuildSwift contained the
entire thing but this should be ok as well.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test


supabase db push --dry-run doesn't correctly test migrations

Bug report
Describe the bug
Generally when a command uses the dry-run property, it will show the computed result of if something is applied. But with db push --dry-run, it only shows what migration files haven't be pushed. I would expect it to validate if the migrations are valid without actually applying them, and output the same thing as a regular run.
This is extremely useful for running migration checks as part of CI, so pull requests can validate that any migrations in it will succeed before actually applying them. The recommended way to handle migrations across environments is to have an action that applies the migration after the merge, but that can easily put the repo in a state where the code has the migration change in the target branch while the supabase project does not (because it failed).
Alternatively, maybe I am misinterpreting what --dry-run is for, and what I'm looking for is some --test flag. If that fits better, I would be very happy to use that flag 😄
And even further as an alternative, if there was some way for the github action to cache the docker(?) image for supabase start, that would also work I think. Right now it takes 3-5 minutes for it to download every time, even though starting and applying it is maybe a couple seconds.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Create a new migration pnpm supabase migration new this_should_fail
Put invalid SQL in it like abcdef or alter table i_do_not_exist add column dummy text
Run pnpm supabase db push --dry-run

Expected behavior
I would expect it to output the same thing as pnpm supabase db push (other than the this will not be applied message):
Pushing migration<PHONE_NUMBER>0451_this_should_fail.sql...
Error: ERROR: syntax error at or near "abcdef" (SQLSTATE 42601)
At statement 0: abcdef

Instead, it prints this:
$ pnpm supabase db push --dry-run
DRY RUN: migrations will *not* be pushed to the database.
Would push migration<PHONE_NUMBER>0451_this_should_fail.sql...

Where it says "would push migration...", when in fact, it would not push that to the DB.
System information

OS: Windows
Version of Node.js:18

Hello, thanks for the suggestion. I agree it would be nice to perform this check with --dry-run. However, it's blocked on a few implementation details.
For one, CLI doesn't do any client side sql validation, so checking for syntax error would require actually sending that statement to a postgres server.
An alternative is to wrap the migration file in a transaction and rollback at the end of dry run. But since we support pushing multiple migration files in one command, rolling back would mean that subsequent migration files could fail.
The current recommendation is to run db start in your CI before db push. This command starts a postgres container locally and runs all migrations to ensure they are valid. In addition, you can also lint and unit test your schemas before pushing to remote project.
Putting them all together:
supabase db start
supabase db lint
supabase db test
supabase db push

Hope it helps.

Thank you for the information, I did not know about the db lint command, will take a look at it.
One thing for my use case, I am primarily trying to speed up the testing in our CI pipeline, as we are using supabase db start to test PRs before allowing merges. But the issue is that it takes several minutes for the images to get downloaded, while the actual migrations take a few seconds to run. Across multiple projects and many rapid PRs, it starts to pull a lot of hours into our GitHub actions. Was just looking for ways to have it run faster for both cost and dev-experience.
I think we can easily adopt a workflow of getting that tested locally, but for regular code changes/PR feedback, it will still result in a new CI run each time. As an alternative, if there was some supabase configuration/utility that allowed the GitHub actions to cache the docker image for a faster run-time, that would be cool too 😄
Curious for your final thoughts 🎩

We looked into caching in a similar request here https://github.com/supabase/setup-cli/issues/88#issuecomment-1370519155
The best we can do right now is to exclude unnecessary containers from starting. Using setup-cli@v1 and db start should reduce your ci time to around 1 min.

Thank you @sweatybridge, we are using setup-cli@v1 as of right now and it is definitely better than no CI, so I appreciate the work being done there! We still see it running for a few minutes, but I will see if we have some configuration error on our part.

@yoont4 just an update: we have further reduced the size of supabase/postgres image so it should start much faster on CI now (around 1m40s for the full stack).


Select up to two functions for the Favourite button on your hob.

Is your feature request related to a problem? Please describe.
There is an option on my hob to use a favourite button, could be nice if you are cooking and you want to turn on an automation.
But if i press the buttons there is nothing visible in the log of HA.
Any chance you can look into this? :) Thx!!

If you enable debug logging and don't see anything coming through when you press those buttons then they don't publish any events and the integration can't know they were pressed.

I meant in the device log. Maybe there is an event in the logs of HA.
Will export the logs when I’m back.


Defining named regular expressions using nested let statements.

If I define a named regexp using nested let statements I get an error when trying to use it.
For example:
let int_lit =
  let digit = [%sedlex.regexp? '0'..'9'] in
  [%sedlex.regexp? Plus digit]

Then later on:
match%sedlex buf with
| int_lit -> ...

Sedlex: unbound regexp int_lit.

I've noticed this same problem. In general let ... in seems fragile for some reason.


CPU-Adam fix for scalar mode

This PR fixes the issue for the DS CPU-Adam when it goes through the scalar mode, when running on the CPUs that SIMD instructions are not supported.
Regarding the performance, I see 6x to 7x improvement over torch.adam. For instance it reduces the optimizer time from 56 seconds to 8.2 seconds to an 11.6B parameter GPT2 model.
TODO: CPU-Adam currently supports AVX instructions from Intel architecture. In the next PR, I will try to add the AMD AVX support to speed up the performance for these architectures too.

Can we add a unit test related to this change? Do we have a way to repro the old issue and create the test based on that?

To repro the issue, ds-adam requires to be compiled in scalar mode! Currently, we have it automated based on the architecture capability. I think one possibility is to use a parameter to select between different compute sources and test the old issue. Do you  have another opinion here?

Can we add a unit test related to this change? Do we have a way to repro the old issue and create the test based on that?

To repro the issue, ds-adam requires to be compiled in scalar mode! Currently, we have it automated based on the architecture capability. I think one possibility is to use a parameter to select between different compute sources and test the old issue. Do you have another opinion here?

That sounds like a good idea to me. Maybe we can add a new environment variable to this part of the code (https://github.com/microsoft/DeepSpeed/blob/master/op_builder/cpu_adam.py#L24) to allow for different SIMD_WIDTH values. If the var isn't set then we fall back to detection like we have currently. Then in the unit test we'll need a way to force re-compile the op with JIT, which can be done by wiping out the folder at TORCH_EXTENSIONS_DIR. Once you have the test that triggers with scalar i'd be happy to write the hooks needed do the env variable pieces.

Can we add a unit test related to this change? Do we have a way to repro the old issue and create the test based on that?

To repro the issue, ds-adam requires to be compiled in scalar mode! Currently, we have it automated based on the architecture capability. I think one possibility is to use a parameter to select between different compute sources and test the old issue. Do you have another opinion here?

You could monkey patch the function that returns the arch capability for the duration of the sub-test and lie that it's AMD ;)

Can we add a unit test related to this change? Do we have a way to repro the old issue and create the test based on that?

To repro the issue, ds-adam requires to be compiled in scalar mode! Currently, we have it automated based on the architecture capability. I think one possibility is to use a parameter to select between different compute sources and test the old issue. Do you have another opinion here?

You could monkey patch the function that returns the arch capability for the duration of the sub-test and lie that it's AMD ;)

I think that's a great idea for another unit test here. I think we want (at least) two unit tests here: (1) test that auto-detect AMD triggers the scalar log, (2) test that the new scalar logic works when enabled. Unfortunately our CI setup doesn't include any AMD cpus currently so this will be tricky for now but I think we can add an A100 runner temporarily to test this. We've been experimenting with adding temporary github action runners and it seems to be working out pretty well so far.

Thanks for the good suggestions, I will later add these unit tests.


diffdock for RNA

Hello,
is it possible to get the DiffDock to work for RNA also, assuming there is a training data set of know RNA-ligands.
Best,
Amir

Hi @amir-tagh
Yes indeed one could train a version of DiffDock for docking to RNA structures, however, there are further challenges such as relatively limited data and higher receptor flexibility for which, I think, this would be an interesting research project in itself!
Best
Gabriele

Hi, I have a similar question. So can I use the provided model checkpoint for docking RNA? Do you think that would be applicable as a baseline, or do we need to retrain the model?

I would not recommend to use it to dock RNA (there is none in the training data as far as I know, and the method is not designed for molecules of that size), but in theory you should be able to run it


[bug] wallet_addEthereumChain or useSwitchChain on Rainbow Wallet not working

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
2.0.0
wagmi Version
2.5.6
Current Behavior
I already add a custom chain to getDefaultConfig

When I used switchChain from useSwitchChain wagmi for Rainbow wallet to switch to this chain, error occured like this:

When I used switchChain for other wallets like MetaMask, it worked, but rainbow didn't

When I tried using wallet_addEthereumChain to add this chain to Rainbow wallet, though I approved the addition of this chain, the error occured. I checked Rainbow wallet networks, this custom chain is already added but error showed:

Expected Behavior
When I used switchChain from useSwitchChain wagmi for Rainbow wallet to switch to a custom chain, it should add the custom chain to RainBow wallet and switch to it
If I use wallet_addEthereumChain, Rainbow wallet should add the chain and not show any error
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

@kingnight153 Could you please tell what chain you're switching to ?
Also would be super helpful if you can show your RainbowKit and Wagmi configuration setup 🙏

Here is the chain info: https://chainlist.org/?search=carbon

Yeah so that's an unsupported network from rainbow wallet. We work on improving the custom network so hopefully should be fixed soon. What you can do now is add the network manually and try to switch to the chain.

@kingnight153 Thanks for reporting. Bumping this to our browser-extension repo to further investigate


Handle case where text file cannot be encoded by CP1252 gracefully.

If the user has selected CP1252 output, but the text file contains symbols that cannot be converted to CP1252, the syncer crashes.
2023-02-26 23:03:50 [ERROR] Uncaught exception:
    Traceback (most recent call last):
      File "usdb_syncer\song_loader.py", line 159, in run
      File "usdb_syncer\song_loader.py", line 271, in _maybe_write_txt
      File "usdb_syncer\songtxt_init.py", line 61, in write_to_file
      File "encodings\cp1252.py", line 19, in encode
    UnicodeEncodeError: 'charmap' codec can't encode character '\u2032' in position 2232: character maps to <undefined>

Maybe the settings dialogue could be extended and if the user selects CP1252, he/she can decide how these cases should be handled (skip song/replace characters with ?/replace characters with unicode escape sequence/ask for each song/...).


New CLI version is very slow on Windows

Dapr CLI v1.7.0-rc.2 is considerably slower than v1.6.0 on Windows.
Script used: https://gist.github.com/shubham1172/80846320d8a33bd4a39f65edd9177133
############################
       dapr --version
############################
v1.6.0

Avg: 175.652ms
Min: 137.4927ms
Max: 1076.538ms

v1.7.0

Avg: 1009.2455ms
Min: 894.5307ms
Max: 1164.4052ms

############################
       dapr --help
############################
v1.6.0

Avg: 172.6062ms
Min: 152.1281ms
Max: 211.8084ms

v1.7.0

Avg: 1026.8171ms
Min: 952.4243ms
Max: 1228.9317ms

############################
       dapr help
############################
v1.6.0

Avg: 182.6562ms
Min: 154.2751ms
Max: 299.4941ms

v1.7.0

Avg: 1240.0274ms
Min: 962.4161ms
Max: 2512.6125ms

############################
       dapr list
############################
v1.6.0

Avg: 394.1212ms
Min: 280.7828ms
Max: 897.2138ms

v1.7.0

Avg: 1687.809ms
Min: 1476.0607ms
Max: 2296.0588ms

Release Note

RELEASE NOTE: FIX Performance on Windows

/assign

Some more findings:

Dapr CLI v1.7.0-rc.2 is not slow if Dapr runtime is uninstalled
Dapr CLI v1.7.0-rc.2 is slow even if it points to an older Dapr runtime version (v1.6.0 instead of v1.7.0-rc.1)
On timing the individual cli/pkg functions, they perform almost the same for v1.6.0 and v1.7.0-rc.2.

A major change from v1.6.0 to v1.7.0-rc.2 was Go upgrade from 1.16 to 1.17.

The problem is with the go compiler version. I tried using different compiler versions with different CLI versions and here are the results (100 runs per command)
############################
       dapr --version
############################
v1.5.1 release version, compiled with go 1.16

Avg: 328.5328ms
Min: 275.4623ms
Max: 487.1119ms

v1.6.0 release version, compiled with go 1.16

Avg: 354.2701ms
Min: 276.5343ms
Max: 641.7528ms

v1.6.0 compiled with go 1.17.6

Avg: 1103.3831ms
Min: 864.4201ms
Max: 2993.4135ms

v1.7.0 release version, compiled with go 1.17

Avg: 1149.8473ms
Min: 996.1999ms
Max: 1666.1498ms

v1.7.0 compiled with go 1.17.6

Avg: 1279.1886ms
Min: 1004.9528ms
Max: 3213.9073ms

v1.7.0 compiled with go 1.18

Avg: 1292.3167ms
Min: 978.8401ms
Max: 2791.5266ms

Now, CLI v1.7.0 cannot be downgraded to use go 1.16 because of a pkg dependency introduced by https://github.com/dapr/cli/pull/894. See https://github.com/kubernetes-sigs/controller-tools/issues/643.

Phew, test results from a brand-new Windows VM:
############################
       dapr --version
############################
v1.6.0 release version, compiled with go 1.16

Avg: 155.8055ms
Min: 134.0364ms
Max: 218.5515ms

v1.7.0 release version, compiled with go 1.17

Avg: 125.6285ms
Min: 110.9377ms
Max: 160.8203ms

############################
       dapr help
############################
v1.6.0 release version, compiled with go 1.16

Avg: 150.4167ms
Min: 129.6965ms
Max: 256.1846ms

v1.7.0 release version, compiled with go 1.17

Avg: 123.6194ms
Min: 110.8603ms
Max: 159.5841ms

The problem was likely with my computer, which had tons of software running. Closing this issue!


external fonts not loaded in iframe

may want to use
http://exportkit.com/plugin/environments/html5/add-google-fonts-to-html5
or https://github.com/typekit/webfontloader (looks more complex & bulky)

closed by 573f50601a19d7ee0d8c829250a7bd52f1e04d69


Update README.md

On third part it says "adb rebbot" but because it's already on adb shell it won't work. Changed to "reboot"

@enes4949 hi，Your version is so supportive

so supportive

What do you mean by that?


🎛 Migrate window.initDateChooser|initTimeChooser|initDateTimeChooser inline script usage to Stimulus w-init controller

ℹ Part of the Stimulus 🎛 RFC 78

Is your proposal related to a problem?
Our current approach for bootstrapping the various date picker widgets rely on inline scripts.

initDateChooser
initDateTimeChooser
initTimeChooser

This approach is no longer necessary and is not CSP compliant, hence it would be good to find a better way to achieve the same goals.

Time picker example (bakerydemo location page)

Date picker example (scheduled publish)

Describe the solution you'd like
With #11644 completed via #11695 we now have an ability to easily dispatch events when an element appears in the DOM using our Stimulus [InitController](https://github.com/wagtail/wagtail/blob/main/client/src/controllers/InitController.ts).
We could adopt a similar approach for the date/time widgets as follows.
Example for AdminTimeInput (1 of 3)

Update the input widget to use the data attributes approach in wagtail/admin/widgets/datetime.py, no longer using widget.config_json but simply the built in Django widget attrs adopting the Stimulus InitController attributes as needed.
Delete wagtailadmin/widgets/time_input.html as this will no longer be needed
Update client/src/entrypoints/admin/telepath/widgets.js so that instead of calling window.initTimeChooser it creates a DOM element with the same attributes as the datetime widget.
Update the JavaScript to include an event listener that calls the global init function in client/src/entrypoints/admin/date-time-chooser.js
Update the JavaScript to mark the global window.initTimeChooser as deprecated in  client/src/entrypoints/admin/date-time-chooser.js
Complete a similar update for the other two inputs AdminDateInput & AdminDateTimeInput (with different event names though)
Update unit tests as needed to ensure there are no longer references to the old approach & the new approach is sufficiently tested
Add upgrade considerations (this can be added on the PR or in the release notes file) that window.initTimeChooser will be deprecated and using the widget or Block (write which ones) is the official way to use these widgets

The code diffs below are just a rough start, these have not been fully tested.
# wagtail/admin/widgets/datetime.py
# ...

class AdminTimeInput(widgets.TimeInput):
--    template_name = "wagtailadmin/widgets/time_input.html"

    def __init__(self, attrs=None, format=None):
--        default_attrs = {"autocomplete": "off"}
++        default_attrs = {
++            "autocomplete": "off",
++            "data-controller": "w-init",
++            "data-w-init-event-value": "w-time-chooser:init",
++        }
        if attrs:
            default_attrs.update(attrs)
        fmt = format
        if fmt is None:
            fmt = getattr(settings, "WAGTAIL_TIME_FORMAT", DEFAULT_TIME_FORMAT)
        self.js_format = to_datetimepicker_format(fmt)
        super().__init__(attrs=default_attrs, format=fmt)

    def get_config(self):
        return {"format": self.js_format, "formatTime": self.js_format}

    def get_context(self, name, value, attrs):
++        attrs = attrs or {}
++        attrs["data-w-init-detail-value"] = json.dumps(self.get_config())
        context = super().get_context(name, value, attrs)
--        context["widget"]["config_json"] = json.dumps(self.get_config())
        return context

// file: client/src/entrypoints/admin/telepath/widgets.js
// ...

class BaseDateTimeWidget extends Widget {
  constructor(options) {
    super();
    this.options = options;
  }

  render(placeholder, name, id, initialState) {
--    const element = document.createElement('input');
--    element.type = 'text';
--    element.name = name;
--    element.id = id;
++    const element = Object.assign(document.createElement('input'), {
++      id,
++      name,
++      'type': 'text',
++      'data-controller': 'w-init',
++      'data-w-init-event-value': this.initEventName,
++      'data-w-init-detail-value': JSON.stringify({ ...this.options }),
++    });

--    this.initChooserFn(id, this.options);
    placeholder.replaceWith(element);

    const widget = {
    // ...
// ... further down in the same file

class AdminTimeInput extends BaseDateTimeWidget {
--  initChooserFn = window.initTimeChooser;
++  initEventName = 'w-time-chooser:init';
}
window.telepath.register('wagtail.widgets.AdminTimeInput', AdminTimeInput);

// client/src/entrypoints/admin/date-time-chooser.js

function initTimeChooser(id, opts) {
  //... initTimeChooser implementation should not change
}
++document.addEventListener('w-time-chooser:init', (event) => {
++  initTimeChooser(event.target.id, event.detail);
++});
++
++ /** @deprecated RemovedInWagtail70 - Remove global.initTimeChooser in a future release */
window.initTimeChooser = initTimeChooser;

Describe alternatives you've considered

There are plans to fully remove this JavaScript, however, this is likely still 6-18 months away depending on browser support for accessible date/time inputs. https://github.com/wagtail/wagtail/issues/8056
There was another approach to fully migrate the date widgets to a starting Stimulus implementation, however, this has not been done due to us wanting to remove the date picker JS completely eventually. https://github.com/wagtail/wagtail/pull/10261
We could leave the code as is, however, this is one of the last handful of inline script usages and we already have a released & proven approach to simply dispatch and listen to events for basic initialisation behaviour.

Additional context

This will be an almost identical migration approach to https://github.com/wagtail/wagtail/issues/11644 & https://github.com/wagtail/wagtail/pull/11695
This is very similar to the approach in See https://github.com/wagtail/wagtail/issues/11287 & https://github.com/wagtail/wagtail/pull/11294
This issue is similar in goals to https://github.com/wagtail/wagtail/issues/11597 but taking an intentionally different approach to just leverage an existing controller

Working on this

Anyone can contribute to this if they are able to work with Stimulus and create Jest tests, plus feel confident in modifying Python tests.
View our contributing guidelines, add a comment to the issue once you’re ready to start.

Blocked by https://github.com/wagtail/wagtail/issues/8056


Not parsing directories with spaces in their names.

C:\Users\Jason Crockett.rbenv-win\bin>rbenv
The system cannot find the path specified.
The system cannot find the path specified.
Input Error: There is no file extension in "C:\Users\Jason".
The system cannot find the path specified.
'C:\Users\Jason' is not recognized as an internal or external command,
operable program or batch file.
C:\Users\Jason Crockett.rbenv-win\bin>

a console log dump.


Fix Create Host Inventories Load More

Load more in inventories dropdown now works correctly.

These changes work well! But using PageFormSingleSelectAwxResource component instead of pageFormAsyncSingleSelect might be a solution more consistent with the rest of the UI. This component wraps pageFormAsyncSingleSelect and has the queryOptions built in so they don't have to be set manually https://github.com/ansible/ansible-ui/blob/main/frontend/awx/common/PageFormSingleSelectAwxResource.tsx#L36. PageFormOrganizationSelect is currently using this component as well. Something like the code below should work
<PageFormSingleSelectAwxResource<Inventory>
  name="inventory.id"
  id="inventory"
  label={t('Inventory')}
  placeholder={t('Select Inventory')}
  queryPlaceholder={t('Loading inventory...')}
  queryErrorText={t('Error loading inventory')}
  isRequired={true}
  url={awxAPI`/inventories/`}
  tableColumns={inventoryColumns}
  toolbarFilters={inventoryFilters}
/>

along with setting the columns and filters above like so
const inventoryColumns = useInventoriesColumns({ disableLinks: true });
const inventoryFilters = useInventoriesFilters();

Ok, thanks, I will try. I had something very similar in our Hub too, however I was then focused to finish the hub project rather then do things in framework. I will use that :-)


Searching in a page doesn't work, r=jsantell. Fixes #155

https://github.com/mozilla/tofino/issues/155

Nice, r+, although surprised getCurrentWebView needed to be changed as its powering navigation

Coverage remained the same at 83.459% when pulling d5bcbec897149beffa8053ee22e1f9cf38a909b3 on victorporof:fix-search into 727ce9cb39c05edf3ff65e6858b56a76e073df48 on mozilla:master.

Coverage remained the same at 83.459% when pulling c68454596f911e9831488c7f60a1780c95c7cabf on victorporof:fix-search into 15d6b658a61d771d83ccdc2df28af5d3b7e8f6ad on mozilla:master.


Improve parser error reporting

The parser works reasonably well, but could be user-friendly: print the whole mismatched word instead of just the first letter, show the error location in prettyprinted output.

Probably goes well with the refactoring in #28.


Example config_template.yaml has broken links

The following two lines contain broken links:
https://github.com/DataDog/datadog-agent/blob/master/pkg/config/config_template.yaml#L537,L541
https://github.com/DataDog/datadog-agent/blob/master/Dockerfilesagent/README.md
Should be:
https://github.com/DataDog/datadog-agent/blob/master/Dockerfiles/agent/README.md

Hey @FierceElements, thanks for the report!
Both links are now fixed


Change logging level at runtime when writingthread = true

I am running some apps in Kubernetes and generally, whenever I encounter an issue and want to enable debug logging, I currently have to update the logging level in a file and restart the pod again. Is there a way to change the logging level if I update a system property or if tinylog.properties is changed?
I have writingthread set to true, so i am not sure if this solution will work or not. That example mentions that it will not work if writingthread is true ? can you explain why not?

I think a useful feature that tinylog could copy that is in my logging framework is allowing certain logger names to change levels dynamically.
https://jstach.io/doc/rainbowgum/current/apidocs/#level_resolvers
Basically Rainbow Gum is static and immutable for all other loggers unless you specify it in config.
logging.global.change=true          # this is required to turn on level changing.
logging.change.com.mycompany=level  # only logger names starting with com.mycompany can have their levels changed.

Then if you did
logging.level.com.mycompany=INFO  # Was ERROR
logging.level.com.mycompany.anything=DEBUG # Was INFO

it would update the loggers similar to logback and log4j2 regardless of writers (and outputs).

tinylog 2 is designed to use frozen logging configurations as these allow the JVM to detect disabled logging levels and to remove the corresponding log statement code completely. This is the reason why tinylog 2 has such a great performance: https://tinylog.org/v2/benchmark/#discarding-log-entries.

I have writingthread set to true, so i am not sure if this solution will work or not. That example mentions that it will not work if writingthread is true ? can you explain why not?

The linked workaround solution uses refactoring to makes final fields mutable at runtime. Depending on your JVM, it is very likely that the writing thread will see the changes with a delay or even not at all. With newer JVMs, it can even happen that refactoring operations are blocked at all.
However, tinylog 3 will optionally support logging configuration changes at runtime out-of-the-box. And the logging API of tinylog 2 and 3 is source code compatible.


Update token-stats endpoint

Please sync this endpoint https://balanced.sudoblock.io/api/v1/stats/token-stats
with these token lists:
https://github.com/balancednetwork/balanced-network-interface/blob/master/src/store/lists/communitylist.json
https://github.com/balancednetwork/balanced-network-interface/blob/master/src/store/lists/tokenlist.json
Meaning we would like to have all those tokens in the endpoint response and to have it automatically reflect newly added tokens to any of those lists.

To clarify, no real processing is needed, you just want those lists to be merged? Is this going to feed into some stats that I will need to calc?

@robcxyz I think this is depricated issue, the new endpoint https://balanced.icon.community/api/v1/tokens has all the tokens that exist, regardless Balanced lists. Balanced token lists now serve just as a filter for displaying relevant tokens.

Cool - closing


Investigate supporting Tahoe-LAFS mutable capabilities

The keys which constitute Tahoe-LAFS mutable capabilities are derived from RSA keys (passed through a custom "tagged" sha256d hash function as per hashutil.py). Investigate how/whether this library could or should support generating equivalent Tahoe-LAFS capabilities without the need to depend on Tahoe-LAFS itself.

Some other reasons to do this:

The allmydata package isn't really designed or intended to be used as a python library/API.
Deriving a Tahoe-LAFS capability string currently requires making an mkdir call to the Tahoe-LAFS web API -- which, in turn, requires being connected to a live grid; it would be useful to be able to reconstruct caps "offline" as a pure function, etc.
Attempting to re-create a Tahoe-LAFS directory that already exists (i.e., by supplying the same RSA key to the mkdir call) throws an UncoordinatedWriteError and may lead to data-loss.
Another implementation (and common test vectors?) helps verify existing functionality.

The recently-published "lafs" crate provides a working Rust implementation of the 'allmydata.util.*' functions necessary to derive Tahoe-LAFS mutable capabilities/URIs completely "offline" (i.e., without depending on a running python Tahoe-LAFS node). Integrating such functionality directly into this crate, however, is presently blocked on: https://github.com/crwood/lafs/issues/12
In the interim, Python applications wishing to derive a Tahoe-LAFS mutable URI from a BIP-39 phrase can still do so by interfacing with each library separately -- i.e., by using deterministic_keygen to derive an RSA key and then calling lafs.derive_mutable_capability with that key to derive the final URI.


[BUG] cdk8s init throws error

Describe the bug
cdk8s init typescript-app throws file already exists error.
OS : Windows 8.1
Node version : 10.11.0
To Reproduce
cdk8s init typescript-app
Expected behavior
New project should have been created.
Additional context
Getting started using https://github.com/awslabs/cdk8s/blob/master/docs/getting-started/typescript.md

It looks like you were trying to init hello/ and the folder already existed (which explains the OS error)

No.

(base) ➜  cdk8s git:(master) ✗ cd hello
(base) ➜  hello git:(master) ✗ cdk8s init python-app
Initializing a project from the python-app template
cdk8s init TYPE
Create a new cdk8s project from a template.
Positionals:
TYPE  Project type                                                                                [required] [choices: "python-app", "typescript-app"]
Options:
--version        Show version number                                                                                [boolean]
--help           Show help                                                                                [boolean]
--dist           Install dependencies from a "dist" directory (for development)                                                                               [string]
--cdk8s-version  The cdk8s version to use when creating the new project                                                                   [string] [default: "0.21.0"]
TypeError: Cannot read property 'mkdir' of undefined
at Object.sscaff (/Users/xyz/.npm-global/lib/node_modules/cdk8s-cli/node_modules/sscaff/lib/sscaff.js:18:25)
at Object.handler (/Users/xyz/.npm-global/lib/node_modules/cdk8s-cli/bin/cmds/init.js:29:24)
at
at process._tickCallback (internal/process/next_tick.js:188:7)
at Function.Module.runMain (module.js:686:11)
at startup (bootstrap_node.js:187:16)
at bootstrap_node.js:608:3

Hey, could you try with Node 12?

heh ... hold on..

(base) ➜  hello git:(master) ✗ node --version
v12.16.3
(base) ➜  hello git:(master) ✗ cdk8s init python-app
Initializing a project from the python-app template
Unable to find "pipenv". Install from https://pipenv.kennethreitz.org

So, the solution (update the docs maybe unless I missed these two things in there... ) is to use some version of node >= 12.x (I was using v8.9.4) and to install pipenv (brew install pipenv). So, I take it back, guess my node was crusty.

So, the solution (update the docs maybe unless I missed these two things in there... ) is to use some version of node >= 12.x (I was using v8.9.4) and to install pipenv (brew install pipenv). So, I take it back, guess my node was crusty.

can you submit a PR for doc update?

Thank you, upgrading the Node version to LTS 12.16.3 helped create the typescript-app templated project successfully.
Next I went on to init using python-app template. Similar to @jmmars , I encountered the pipenv installation message.
Went on to install pipenv - pip install --user pipenv
Followed it up with
cdk8s init python-app
in a new folder name hello_python
Received the error as seen in below screenshot

Read somewhere that which keyword is not supported in Windows.
Switched to using git bash instead.
It failed at the import part
cdk8s import k8s -l python

Realized it maybe a permission issue , so ran git bash in Administrator mode and now it seems jsii bin folder is missing within cdk8s-cli node_modules folder

can you submit a PR for doc update?
I'll  see if I can patch this getting-started/python.md this weekend after looking over your contrib guide. Basically, it just needs to be clearer on prereqs.

Looking forward: At some point, for some systems might be able to do a more programmatic check to make sure base reqs will enable an easy install.

Is this still relevant? Closing for now.


add profiler search

Nice! I'm looking forward to have this!
Have you checked out https://procomponents.ant.design/en-US/components/table#left-and-right-structure ?
The "pro" table seems to have a lot of extra functions we may not need. But it is a table with toolbar having search and actions. Might be good as an inspiration for a code.
Also if I'm not mistaken they may be composing the prop interfaces (like we discussed today) ...
https://github.com/ant-design/pro-table/blob/df4fc21be7fc3b1060715920ca73765f0abaac26/src/Table.tsx#L176


ClientRegistration Event - Suggestions

Small change suggestion.
You may want to change getInterval() to getSendMetricInterval() as this could easily be confused with the fetchTogglesInterval value from the UnleashConfig if the source code isn't examined.
Also, I noticed that the ClientRegistration is pulling a several values from the UnleashConfig.  Would it make more sense to instead just store the entire UnleashConfig as an event variable instead of the few selected fields? That way, someone implementing the clientRegistered() method of the UnleashSubscriber can pick/choose which fields are relevant for them to log.

I agree with your suggestions. Will look in to it soon, when I have some time for it.
Feel free to contribute a PR for it, but be careful about breaking changes. (E.g. we probably should deprecate "gerInterval' when adding the new method).

After trying to write a PR and thinking about this a bit more, I think this is a wontfix.
UnleashConfig has a scheduled executor and a reference to a subscriber that doesn't lend itself easily to serialization. As such it doesn't make sense to include the entire config object in the event.
If there are specific fields from the config you're missing, feel free to submit a PR adding those fields to the event you need it in. But as of now, we won't include the entire config in the event.

After trying to write a PR and thinking about this a bit more, I think this is a wontfix.
UnleashConfig has a scheduled executor and a reference to a subscriber that doesn't lend itself easily to serialization. As such it doesn't make sense to include the entire config object in the event.
If there are specific fields from the config you're missing, feel free to submit a PR adding those fields to the event you need it in. But as of now, we won't include the entire config in the event.

With regards to interval vs sendMetricsInterval, we do agree that the name may be confusing. However, the client specification requires the field to be serialized as interval for now. We might revisit that for a later major release.

With regards to interval vs sendMetricsInterval, we do agree that the name may be confusing. However, the client specification requires the field to be serialized as interval for now. We might revisit that for a later major release.


Add Monitor BufferStatProcessing config, to reduce CPU

What does this PR do?
Add Monitor BufferStatProcessing config, to reduce CPU
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[x] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?

Add a traffic_monitor.cfg key stat_buffer_interval_ms set to 5
Change traffic_monitor.cfg stat_flush_interval_ms to 1000
Verify monitor behaves correctly. The behavior should not change, except to improve poll times and/or reduce CPU.
If want to verify the code is executing, enable Info logging and look in the log for messages StatHistoryManager
To observe buffer behavior at the macro level, set the stat_buffer_interval_ms to 20000 (where the Flush interval is <10s), and observe the actual stat span in the UI around 20s.

The Monitor doesn't have an Integration Test framework, and I'm not sure how to unit test this, as it's part of the StatManager and not really isolatable. I added extensive comments, which should help convey the intention and behavior, though.
Added extensive documentation, explaining the new config options, as well as the existing related configs.
Tested building the documentation.
Tested running the monitor under load, verified stats and health are polled correctly, verified history is correct, verified various settings of buffer and flush configs behave as expected.
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2753/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2760/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2794/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2795/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2796/
Test FAILed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2829/
Test PASSed.


Create/Select ShippingRate in Stripe from Backoffice

Rather than manually copying the shippingrate ID from Stripe, it should pull in the existing stripe shippingrates and/or push into Stripe

Done


Migration to organization

It is possible to migrate the personal account to organization?
Converting:

https://help.github.com/articles/converting-a-user-into-an-organization/

It is possible to rename this account and create the organization and move/transfer the repository to the organization.
Rename account:

https://help.github.com/en/enterprise/2.13/user/articles/changing-your-github-username

Create an organization:

https://help.github.com/en/github/setting-up-and-managing-organizations-and-teams/creating-a-new-organization-from-scratch

Transfer a repository:

https://help.github.com/en/github/administering-a-repository/transferring-a-repository

Like:

http://github.com/gnome
http://github.com/kde
...

I am not sure what this is... I will need some elaboration if this is sincere.

@AdamKorcz: @AdaLogics is a personal acount.
Please do not close this ticket before the solution.

I see. Why do we need to change the account type?

@AdamKorcz: Personal account is for a real person, not a company.
And it is easy to see the difference.

@AdamKorcz: Personal account is for a real person, not a company. And it is easy to see the difference.

Yes, that is correct, but that does not really answer my question. Why do we need to switch the account type?

@AdamKorcz: It is easy: @AdaLogics is:

A person?
A company?

Yes, sure, but what is the issue using the library if it is in a personal Github account? I am not saying there isn't one btw. Just trying to understand the use case.

@AdamKorcz: It will better to manage repositories if @AdaLogics becomes an organization.
It is free.


Duplication with egglog vscode extension

Hi! It looks like this extension is more fully featured than the one in the egglog repo:
https://github.com/egraphs-good/egglog/tree/main/vscode/eggsmol-1.0.0
Would you be interested in merging it into egglog?
Or perhaps we should make this the official one, and move it to egraphs-good/egglog-language-server?

Yes!
I have no preference about merging it into egglog or making this the official one. Either is OK!
Perhaps we need to also consider about https://github.com/hatoo/tree-sitter-egglog
egglog-language-server depends on it.

I think that adding both tree-sitter-egglog and this repo to the egglog repository would be my vote

That being said, we are struggling to maintain egglog right now- there are lots of features we want and clean-ups to do.
If you merge into egglog, it would be great if you maintain it in the future when we change language features!

I think that adding both tree-sitter-egglog and this repo to the egglog repository would be my vote

Looks good.

That being said, we are struggling to maintain egglog right now- there are lots of features we want and clean-ups to do.
If you merge into egglog, it would be great if you maintain it in the future when we change language features!

Yeah, I'm happy to do that!
The main reason I wasn't quite active at egglog-language-server is just the lack of feedback.
I think it will be better after merging to main repository.
So, how do we proceed with merging?
May I create PR about this?

That's great!
Yes, I think a PR would be great. I'll encourage people to start using it and give feedback on your PR!


StreamNotificationService Exception

I keep on getting this error at seemingly random times when I play a song. I don't know why but there are consistent pauses when the music plays and this Exception appears, even on AWS.
OS:  Ubuntu 16.04.3 LTS
Run Method: pm2
0|Nadeko   | 0 19:45:54 StreamNotificationService | StackExchange.Redis.RedisTimeoutException: Timeout performing SCAN, inst: 1, queue: 1, qu: 0, qs: 1, qc: 0, wr: 0, wq: 0, in: 0, ar: 0, clientName: , serverEndpoint: <IP_ADDRESS>:6379 (Please take a look at this article for some common client-side issues that can cause timeouts: https://stackexchange.github.io/StackExchange.Redis/Timeouts)
0|Nadeko   |    at StackExchange.Redis.ConnectionMultiplexer.ExecuteSyncImpl[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\ConnectionMultiplexer.cs:line 2101
0|Nadeko   |    at StackExchange.Redis.RedisServer.ExecuteSync[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisServer.cs:line 571
0|Nadeko   |    at StackExchange.Redis.RedisBase.CursorEnumerable`1.CursorEnumerator.MoveNext() in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisBase.cs:line 276
0|Nadeko   |    at NadekoBot.Core.Services.Impl.RedisCache.<GetAllStreamDataAsync>d__27.MoveNext() in /home/earth/NadekoBot/NadekoBot.Core/Services/Impl/RedisCache.cs:line 166
0|Nadeko   | --- End of stack trace from previous location where exception was thrown ---
0|Nadeko   |    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
0|Nadeko   |    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
0|Nadeko   |    at NadekoBot.Modules.Searches.Services.StreamNotificationService.<<-ctor>b__8_5>d.MoveNext() in /home/earth/NadekoBot/NadekoBot.Core/Modules/Searches/Services/StreamNotificationService.cs:line 65
0|Nadeko   | 0 19:47:27 StreamNotificationService | StackExchange.Redis.RedisTimeoutException: Timeout performing SCAN, inst: 1, queue: 1, qu: 0, qs: 1, qc: 0, wr: 0, wq: 0, in: 0, ar: 0, clientName: , serverEndpoint: <IP_ADDRESS>:6379 (Please take a look at this article for some common client-side issues that can cause timeouts: https://stackexchange.github.io/StackExchange.Redis/Timeouts)
0|Nadeko   |    at StackExchange.Redis.ConnectionMultiplexer.ExecuteSyncImpl[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\ConnectionMultiplexer.cs:line 2101
0|Nadeko   |    at StackExchange.Redis.RedisServer.ExecuteSync[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisServer.cs:line 571
0|Nadeko   |    at StackExchange.Redis.RedisBase.CursorEnumerable`1.CursorEnumerator.MoveNext() in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisBase.cs:line 276
0|Nadeko   |    at NadekoBot.Core.Services.Impl.RedisCache.<GetAllStreamDataAsync>d__27.MoveNext() in /home/earth/NadekoBot/NadekoBot.Core/Services/Impl/RedisCache.cs:line 166
0|Nadeko   | --- End of stack trace from previous location where exception was thrown ---
0|Nadeko   |    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
0|Nadeko   |    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
0|Nadeko   |    at NadekoBot.Modules.Searches.Services.StreamNotificationService.<<-ctor>b__8_5>d.MoveNext() in /home/earth/NadekoBot/NadekoBot.Core/Modules/Searches/Services/StreamNotificationService.cs:line 65

Are you running the latest Nadeko? That error isn't related to music streaming, so it likely is a side effect of the server memory/CPU usage when streaming is stressing the server. Kwoth pushed a specific fix for this (potentially) a few days ago, and I haven't had any errors since, but my server has a lot of extra resources. The specific line that the error is thrown is a call to get the status of stream notifications from Redis cache.

Someone else was using top to examine the resources used but it never really went beyond 20% on average on both memory and CPU so I'm really wondering why would the server be stressed at that point.
Case in point.
I also recently let the server auto-update and the problems still persist. So 🤷♂

While you experience the problems or play music, if you can run this:
sudo redis-cli --intrinsic-latency 150
And post the output.

earth@kagari:~$ sudo redis-cli --intrinsic-latency 150
Max latency so far: 1 microseconds.
Max latency so far: 25 microseconds.
Max latency so far: 40 microseconds.
Max latency so far: 42 microseconds.
Max latency so far: 1310 microseconds.
Max latency so far: 1892 microseconds.
Max latency so far: 4320 microseconds.
Max latency so far: 5949 microseconds.
Max latency so far: 7049 microseconds.
Max latency so far: 8272 microseconds.
Max latency so far: 8776 microseconds.
Max latency so far: 9578 microseconds.

239766249 total runs (avg latency: 0.6256 microseconds / 6256.09 nanoseconds per run).
Worst run took 15310x longer than the average latency.
earth@kagari:~$ sudo redis-cli --intrinsic-latency 150
Max latency so far: 1 microseconds.
Max latency so far: 18 microseconds.
Max latency so far: 21 microseconds.
Max latency so far: 24 microseconds.
Max latency so far: 72 microseconds.
Max latency so far: 1536 microseconds.
Max latency so far: 2475 microseconds.
Max latency so far: 5451 microseconds.
Max latency so far: 6587 microseconds.
Max latency so far: 10282 microseconds.

244468012 total runs (avg latency: 0.6136 microseconds / 6135.77 nanoseconds per run).
Worst run took 16757x longer than the average latency.

I ran it on a few servers, the first one is the one I use for Nadeko (among other things). Does it happen only when you play music?

vlexar@node1:~/main/vlexar$ sudo redis-cli --intrinsic-latency 150
Max latency so far: 1 microseconds.
Max latency so far: 20 microseconds.
Max latency so far: 23 microseconds.
Max latency so far: 24 microseconds.
Max latency so far: 28 microseconds.
Max latency so far: 38 microseconds.
Max latency so far: 40 microseconds.
Max latency so far: 46 microseconds.

4445529895 total runs (avg latency: 0.0337 microseconds / 33.74 nanoseconds per run).
Worst run took 1363x longer than the average latency.

vlexar@frontend-dev-perf1:~$ sudo redis-cli --intrinsic-latency 150
Max latency so far: 1 microseconds.
Max latency so far: 23 microseconds.
Max latency so far: 110 microseconds.
Max latency so far: 459 microseconds.
Max latency so far: 3847 microseconds.

2563338951 total runs (avg latency: 0.0585 microseconds / 58.52 nanoseconds per run).
Worst run took 65741x longer than the average latency.

Hmm, it does seem to persist without music.
earth@kagari:~$ sudo redis-cli --intrinsic-latency 150
Max latency so far: 1 microseconds.
Max latency so far: 30 microseconds.
Max latency so far: 41 microseconds.
Max latency so far: 170 microseconds.
Max latency so far: 228 microseconds.
Max latency so far: 697 microseconds.
Max latency so far: 1034 microseconds.
Max latency so far: 1371 microseconds.
Max latency so far: 1665 microseconds.
Max latency so far: 1743 microseconds.
Max latency so far: 3674 microseconds.
Max latency so far: 8251 microseconds.
Max latency so far: 8395 microseconds.
Max latency so far: 8401 microseconds.
Max latency so far: 8572 microseconds.
Max latency so far: 11467 microseconds.

236749000 total runs (avg latency: 0.6336 microseconds / 6335.82 nanoseconds per run).
Worst run took 18099x longer than the average latency.

I guess time for Plan B. I'm switching to CentOS and see if that changes things.

It almost certainly isn't an OS issue. There could still be a race condition or lock in the code, the fix Kwoth pushed was a possible fix. On servers that are powerful, it won't be noticeable. But the OS is not going to really be the culprit here, on the same hardware. At least there shouldn't be any real difference between two Linux distributions when it comes to this.

This is the redis run on AWS micro2 btw.
First run is without music, the 2nd is with music.
I also I haven't tried using tmux yet for either configuration.
ubuntu@ip-172-31-11-182:~$ sudo redis-cli --intrinsic-latency 150
Max latency so far: 1 microseconds.
Max latency so far: 3 microseconds.
Max latency so far: 12 microseconds.
Max latency so far: 100 microseconds.
Max latency so far: 155 microseconds.
Max latency so far: 258 microseconds.
Max latency so far: 280 microseconds.
Max latency so far: 3414 microseconds.
Max latency so far: 4488 microseconds.

775206468 total runs (avg latency: 0.1935 microseconds / 1934.97 nanoseconds per run).
Worst run took 23194x longer than the average latency.
ubuntu@ip-172-31-11-182:~$ sudo redis-cli --intrinsic-latency 150
Max latency so far: 1 microseconds.
Max latency so far: 11 microseconds.
Max latency so far: 12 microseconds.
Max latency so far: 13 microseconds.
Max latency so far: 1456 microseconds.
Max latency so far: 1457 microseconds.
Max latency so far: 1601 microseconds.
Max latency so far: 1648 microseconds.
Max latency so far: 3175 microseconds.
Max latency so far: 7067 microseconds.

720239875 total runs (avg latency: 0.2083 microseconds / 2082.64 nanoseconds per run).
Worst run took 33933x longer than the average latency.

Should be working fine by now. Made a few changes.

Bug persist on version 2.6.11
0 02:28:59 StreamNotificationService | StackExchange.Redis.RedisTimeoutException: Timeout performing SCAN, inst: 1, queue: 32, qu: 0, qs: 32, qc: 0, wr: 0, wq: 0, in: 466, ar: 0, clientName: , serverEndpoint: <IP_ADDRESS>:6379 (Please take a look at this article for some common client-side issues that can cause timeouts: https://stackexchange.github.io/StackExchange.Redis/Timeouts)
   at StackExchange.Redis.ConnectionMultiplexer.ExecuteSyncImpl[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\ConnectionMultiplexer.cs:line 2101
   at StackExchange.Redis.RedisServer.ExecuteSync[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisServer.cs:line 571
   at StackExchange.Redis.RedisBase.CursorEnumerable`1.CursorEnumerator.MoveNext() in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisBase.cs:line 295
   at NadekoBot.Core.Services.Impl.RedisCache.<GetAllStreamDataAsync>d__27.MoveNext() in /home/nadeko/NadekoBot/NadekoBot.Core/Services/Impl/RedisCache.cs:line 168
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NadekoBot.Modules.Searches.Services.StreamNotificationService.<<-ctor>b__8_5>d.MoveNext() in /home/nadeko/NadekoBot/NadekoBot.Core/Modules/Searches/Services/StreamNotificationService.cs:line 65
0 02:33:37 StreamNotificationService | StackExchange.Redis.RedisTimeoutException: Timeout performing SCAN, inst: 1, queue: 3, qu: 0, qs: 3, qc: 0, wr: 0, wq: 0, in: 14, ar: 0, clientName: , serverEndpoint: <IP_ADDRESS>:6379 (Please take a look at this article for some common client-side issues that can cause timeouts: https://stackexchange.github.io/StackExchange.Redis/Timeouts)
   at StackExchange.Redis.ConnectionMultiplexer.ExecuteSyncImpl[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\ConnectionMultiplexer.cs:line 2101
   at StackExchange.Redis.RedisServer.ExecuteSync[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisServer.cs:line 571
   at StackExchange.Redis.RedisBase.CursorEnumerable`1.CursorEnumerator.MoveNext() in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisBase.cs:line 276
   at NadekoBot.Core.Services.Impl.RedisCache.<GetAllStreamDataAsync>d__27.MoveNext() in /home/nadeko/NadekoBot/NadekoBot.Core/Services/Impl/RedisCache.cs:line 168
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NadekoBot.Modules.Searches.Services.StreamNotificationService.<<-ctor>b__8_5>d.MoveNext() in /home/nadeko/NadekoBot/NadekoBot.Core/Modules/Searches/Services/StreamNotificationService.cs:line 65
0 02:36:20 StreamNotificationService | StackExchange.Redis.RedisTimeoutException: Timeout performing SCAN, inst: 0, queue: 5, qu: 1, qs: 4, qc: 0, wr: 0, wq: 1, in: 7, ar: 0, clientName: , serverEndpoint: <IP_ADDRESS>:6379 (Please take a look at this article for some common client-side issues that can cause timeouts: https://stackexchange.github.io/StackExchange.Redis/Timeouts)
   at StackExchange.Redis.ConnectionMultiplexer.ExecuteSyncImpl[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\ConnectionMultiplexer.cs:line 2101
   at StackExchange.Redis.RedisServer.ExecuteSync[T](Message message, ResultProcessor`1 processor, ServerEndPoint server) in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisServer.cs:line 571
   at StackExchange.Redis.RedisBase.CursorEnumerable`1.CursorEnumerator.MoveNext() in C:\code\StackExchange.Redis\StackExchange.Redis\StackExchange\Redis\RedisBase.cs:line 295
   at NadekoBot.Core.Services.Impl.RedisCache.<GetAllStreamDataAsync>d__27.MoveNext() in /home/nadeko/NadekoBot/NadekoBot.Core/Services/Impl/RedisCache.cs:line 168
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at NadekoBot.Modules.Searches.Services.StreamNotificationService.<<-ctor>b__8_5>d.MoveNext() in /home/nadeko/NadekoBot/NadekoBot.Core/Modules/Searches/Services/StreamNotificationService.cs:line 65

@TiBarification by any chance are you using a VM as well? (AWS EC2, etc.)

No, just raw ubuntu 16.04

@Kwoth can you reopen that issue?

@Kwoth can you reopen this issue?


Give feedback form

What do you need?
https://github.com/AusDTO/gov-au-content-guide/issues/220

Moved tracking of issue to Trello.


Private networking documentation

We need to have documentation for the private networking, I would suggest keeping the documentation in the current repo in Wiki section.

https://github.com/majidkarimizadeh/leaseweb-go-sdk/wiki/Private-Networking


getSubCategoryList and getParentCategoryList migrated to retrofit

As a part of #3026 , this PR migrates ApacheHttpClientMediaWikiApi#getSubCategoryList to CategoryClient#getSubCategoryList and ApacheHttpClientMediaWikiApi#getParentCategoryList to CategoryClient#getParentCategoryList.

Codecov Report

Merging #3055 into backend-overhaul will increase coverage by 0.02%.
The diff coverage is 38.46%.

@@                 Coverage Diff                 @@
##           backend-overhaul   #3055      +/-   ##
===================================================
+ Coverage              4.71%   4.74%   +0.02%
===================================================
  Files                   261     261
  Lines                 12195   12149      -46
  Branches               1043    1033      -10
===================================================
+ Hits                    575     576       +1
+ Misses                11581   11533      -48
- Partials                 39      40       +1

Impacted Files
Coverage Δ

.../nrw/commons/category/SubCategoryListFragment.java
0% <0%> (ø)
:arrow_up:

.../java/fr/free/nrw/commons/di/NetworkingModule.java
0% <0%> (ø)
:arrow_up:

...rw/commons/mwapi/ApacheHttpClientMediaWikiApi.java
0% <0%> (ø)
:arrow_up:

...a/fr/free/nrw/commons/category/CategoryClient.java
94.73% <83.33%> (-5.27%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks, @ilgazer. You are very quick with your PRs and the changes look good. :)

You're welcome :) When you have spare time, can you give me some feedback on my tests? The ones I write seem somewhat useless to me :|

You're welcome :) When you have spare time, can you give me some feedback on my tests? The ones I write seem somewhat useless to me :|

Yes, IMO for network layer we should use MockWebServer to write tests. Those tests can actually simulate different responses received from the server.
For ref: https://android.jlelse.eu/unit-test-api-calls-with-mockwebserver-d4fab11de847


Add support for Amazon Linux 2022

We need to ensure that we support  Amazon Linux 2022 at launch.

Packages produced and builds tested on  Amazon Linux 2022
Proper Ohai detection
dokken images and testing in Test Kitchen

Added


location used by .net wrapper for trained data

Hi....
I have setup tesseract api before using your .net wrapper for tesseract. For that i have setup Environment variable  TESSDATA_PREFIX for traineddata files location.
now i am using your stuff. now problem is that when i run the code it goes to "C:\Program Files (x86)\Tesseract-OCR\tessdata" for finding traineddata file instead of location that is specified while initializing [ TesseractEngine engine = new<EMAIL_ADDRESS>language, EngineMode.Default)  ] .
now i want that engine goes to location specified in TesseractEngine constructor for finding traineddata files.
can you give me some idea how can i do that ?
whether Tesseract .net wrapper actually need Tesseract  to be installed or NOT ????
Thanks in advance....

select files, put properties and that is equal to mine

select files, put properties and that is equal to mine

Thank you so much! It works :)

Just be aware that will only clear the TESSDATA_PREFIX environment variable
while the console window is open (sets it for the process not system).
That's assuming that was the issue all along.
On 2 Jul 2014 21:50, "JayeshBhanderi"<EMAIL_ADDRESS>wrote:

i think the problem was solved by running this command :
    set TESSDATA_PREFIX

ouput: Environment variable TESSDATA_PREFIX not defined
i have run this command at last so it is possible that it solved the
problem.
—
Reply to this email directly or view it on GitHub
#103 (comment).

HI Charles,
I am facing the same issue.
I tried using the log writing logic but geting the following errors. I have uploaded my log file.
Please advice me to run my application
tesseract.log

Provided you've specified the tessdata path, ideally as an absolute path,
then TESSDATA_PREFIX should be ignored. I believe it's only used these days
if the tessdata path isn't specified (empty string) when creating a
TessEngine instance.
On Tue., 26 Mar. 2019, 22:26 kalyanakumar619<EMAIL_ADDRESS>wrote:

Just be aware that will only clear the TESSDATA_PREFIX environment variable
while the console window is open (sets it for the process not system).
That's assuming that was the issue all along.
On 2 Jul 2014 21:50, "JayeshBhanderi"<EMAIL_ADDRESS>wrote:
i think the problem was solved by running this command :
set TESSDATA_PREFIX

ouput: Environment variable TESSDATA_PREFIX not defined
i have run this command at last so it is possible that it solved the
problem.
—
Reply to this email directly or view it on GitHub
#103 (comment)
https://github.com/charlesw/tesseract/issues/103#issuecomment-47766430.
HI Charles,
I am facing the same issue.
I tried using the log writing logic but geting the following errors. I
have uploaded my log file.
Please advice me to run my application
tesseract.log
https://github.com/charlesw/tesseract/files/3007889/tesseract.log
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/103#issuecomment-476584658,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyNqHnaWs9rQHXfRdo7lNKIEsJBisks5vagPSgaJpZM4CJDuP
.

Tesseract.TesseractException: 'Failed to initialise tesseract engine.. See https://github.com/charlesw/tesseract/wiki/Error-1 for details.'

i get a error message "See https://github.com/charlesw/tesseract/wiki/Error-1 for details.'"
If the app run on foreground is ok, when i want to run on background with task scheduler not working.
Can you help me? Thanks.

i get a error message "See https://github.com/charlesw/tesseract/wiki/Error-1 for details.'"
If the app run on foreground is ok, when i want to run on background with task scheduler not working.
Can you help me? Thanks.

Try using tessdata_fast rather then tessdata version
.Anyone knows how to read data from table in image with this library?