Release/4.4.3

Candidates for 4.4.3 release.
Please make in this branch last minute testing and use this thread to propose PRs to be included

Includes:
https://github.com/react-native-community/react-native-video/pull/1647
https://github.com/react-native-community/react-native-video/pull/1646
https://github.com/react-native-community/react-native-video/pull/1629
Should fix:
https://github.com/react-native-community/react-native-video/issues/113 (and related, if you found more, please list them here)
https://github.com/react-native-community/react-native-video/issues/1643

Added:
https://github.com/react-native-community/react-native-video/pull/1469/
Related:
https://github.com/react-native-community/react-native-video/issues/1462#issuecomment-504999168

Updated with https://github.com/react-native-community/react-native-video/pull/1654/


Add number lookup GA

Data Plane API - Pull Request
We are releasing of number insight GA.  No changes compare with previous version.
API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue: https://skype.visualstudio.com/SPOOL/_workitems/edit/3938693

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[x] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc: https://github.com/Azure/azure-rest-api-specs/tree/main/specification/communication/data-plane/PhoneNumbers/preview/2024-03-01-preview
Updated paths: operatorInformation/:search

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Schedule a data plane REST API spec review

Getting help

First, please carefully read through this PR description, from top to bottom.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

API change check
APIView has identified API level changes in this PR and created following API reviews.
AzureCommunicationRoutingService
Azure.Communication.MessagesService
CallAutomation-PhoneNumbers


#5372: Add end to end demo for functional stable diffusion model

This PR has end to end demo for the functional stable diffusion model.

The demo is experiencing hang at various stages during each trigger on WH.
To be specific, The demo includes "run_demo_inference" and "run_demo_diffusion_db," both executed for 256x256 and 512x512 image sizes with 2 num_inference_steps.
During the initial run, the test was unresponsive for the 512x512 image size during the 2nd denoising step in "run_demo_diffusion_db."
In the subsequent run, it encountered a hang for the same image size during the 1st denoising step in "run_demo_diffusion_db."
Note: This demo hang issue is not limited to only run_demo_diffusion_db test, it is observed in run_demo_inference test at times.

CIs passed. Link here


Limit custom icons size in various editor widgets

Fixed cases reported in https://github.com/godotengine/godot/issues/83801#issuecomment-1781157294 by @chybby.

Thanks!


Android - Module doesn't work in Ti.SDK 5.1.2.GA

Hello,
I'm trying to use this module on Android but it shows me this error:
I read that it can be caused by google-play-services.jar, but if I remove this library I can't compile the module. The version 1.0 of this module doesn't work in Android 6.0.
Can anyone help me?
Failed to run dexer:
[ERROR] :
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/LocationSource;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/GoogleMap$4;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/StreetViewPanorama$OnStreetViewPanoramaChangeListener;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/StreetViewPanoramaFragment;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/GoogleMap$InfoWindowAdapter;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/MapsInitializer;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/MapFragment;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/GoogleMap$OnMyLocationButtonClickListener;
[ERROR] :  Uncaught translation error: java.lang.IllegalArgumentException: already added: Lcom/google/android/gms/maps/UiSettings;
[ERROR] :
[ERROR] :  UNEXPECTED TOP-LEVEL EXCEPTION:
[ERROR] :  java.lang.RuntimeException: Translation has been interrupted
[ERROR] :  	at com.android.dx.command.dexer.Main.processAllFiles(Main.java:608)
[ERROR] :  	at com.android.dx.command.dexer.Main.runMonoDex(Main.java:311)
[ERROR] :  	at com.android.dx.command.dexer.Main.run(Main.java:277)
[ERROR] :  	at com.android.dx.command.dexer.Main.main(Main.java:245)
[ERROR] :  	at com.android.dx.command.Main.main(Main.java:106)
[ERROR] :  Caused by: java.lang.InterruptedException: Too many errors
[ERROR] :  	at com.android.dx.command.dexer.Main.processAllFiles(Main.java:600)
[ERROR] :  	... 4 more

@annamf Please keep a watch on this ticket:
https://github.com/Sitata/titanium-google-analytics/issues/48

Ok, thank you. I hope this problem will be resolved soon.


Environment of a new notebook

It seems like creating a new notebook (File->New Notebook) starts it in the same environment as the last open notebook. I'm not sure if this is deliberate, but I find it a bit surprising & confusing (especially since there is no visual indication of the environment in the interface).
I would expect a new notebook to be in the default environment, or perhaps in the same environment that notebook() was called from.
ref #820

IJulia starts notebooks in the environment of the directory where the project and manifest files are (or if they don't exist, in the default environment).
If you have two notebooks in the same directory, they will use the same environment. To circumvent this, just use multiple directories (you have to anyway to create different project and manifest files).

IJulia starts notebooks in the environment of the directory where the project and manifest files are

I'm not sure what that means. Which project and manifest files?
My understanding is that if you open an existing notebook, it looks in its directory (where the notebook file is) and its superdirectories, which is fine.
My issue is with new notebooks that were not saved yet (so there is no notebook file). These seem to use the directory/environment of whatever notebook was last open, as far as I can tell.

I think that was a typo -- I think I meant to say

Julia starts notebooks in the environment of the directory where the notebook is (or if the project and manifest files don't exist, in the default environment).

Sorry for that.

My issue is with new notebooks that were not saved yet (so there is no notebook file). These seem to use the directory/environment of whatever notebook was last open, as far as I can tell.

This is probably a JupyterLab behavior. The notebook files always exist (as far as I know) even if you haven't saved them yet (e.g., creating a new notebook will create a file on disk called "Untitled.ipynb" or whatever).
At the end of the day, IJulia is actually not even aware of the notebook files. From IJulia's (and IPython, IRKernel, etc.) perspective, it is started in a particular working directory and then just runs a server that speaks the Jupyter protocol in order to execute code (and the Jupyter protocol is just a fancy REPL - it has no idea about your notebook files).
If creating a new notebook starts it in the same directory as the previous notebook, that's just because Jupyter is spawning the Julia process in that directory. If that's the case, then this would be an upstream issue (with Jupyter Notebook or Jupyter Lab).

When opening a new notebook with jupyterlab() or notebook(), the Untitled.ipynb file is opened in my home directory rather than in the directory from which the command was given (also where I have a Project.toml and Manifest.toml file which reference IJulia).

At the end of the day, IJulia is actually not even aware of the notebook files. From IJulia's (and IPython, IRKernel, etc.) perspective, it is started in a particular working directory and then just runs a server that speaks the Jupyter protocol in order to execute code (and the Jupyter protocol is just a fancy REPL - it has no idea about your notebook files).

This is still confusing because when running jupyter lab from e.g. an Anaconda prompt, the notebook is opened by Jupyter Lab with the root directory starting in the directory from which the command was issued. Calling the IJulia command juypterlab() should tell Jupyter where it's being called from so Jupyter can open a new notebook in that location.
A better default action would be equivalent to running:
jupyterlab(dir=".") or notebook(dir=".")


Make the loaded compiler configurable

When the compiler is hosted as an API today, the binaries are loaded from our referenced NuGet packages. This works in the vast majority of scenarios due to the back compat nature of the compiler. There are times though when it's advantageous to load a different compiler:

Want to use the exact compiler contained in the log.
Want to use a compiler from a build of dotnet/roslyn. Would be very useful for debugging.
Want to use a compiler from a specific .NET SDK.

This can be done when running on .NET Core through AssemblyLoadContext usage. Likely have to re-engineer the code a bit but this could be supported.

Think the sweet spot here is for replay based investigations. It would be really nice for instance to do the following:
> complog --compiler c:\users\jaredpar\code\roslyn\artifacts\packages\Micosoft.Net.Compilers.Toolset replay

Then could debug issues against the current compiler easily. That would help a bit with investigations where we're trying to determine if a new compiler introduced a regression or if it was present in an older one.

It's been a year and haven't really found a need for this. Closing for now until I have concrete scenarios that end up driving it.


Does not deploy

When attempting a cdk deploy the following error is given:
11:03:36 AM | CREATE_FAILED        | AWS::Lambda::Function                     | certificateCertifi...orFunction5D4BA95F
Resource handler returned message: "The runtime parameter of nodejs10.x is no longer supported for creating or updating AWS Lambda functions. We recommend you use the new runtime (nodejs18.x) while creating or updating functions.

Hey :)
I haven't touched the code in a while, seems like I need to update the CDK version etc. as well, quite a few of these components are now out of date. I can't promise you that I'll get to it quickly unfortunately :/

Hey @ryansonshine ,
I've updated the repo to CDK v2 and after upgrading a few more dependencies it works again, give it a try :)


Make error pages opt-in

As seen in https://github.com/feathersjs/feathers-mongoose/issues/56 it is very easy to make a request and get an error page which looks like a mistake. Services on the other hand always return JSON. I'm wondering if we should make the HTML error pages opt-in (passing the filename in the options) and add them to the generator instead.

I would definitely appreciate this.

It may sound stupid but I think it’s actually sort of important to support this by default because it gives people a sense of security and makes Feathers look polished. Like, _”man these guys have thought of everything. Even minimal error pages”_.
If you don't want to use the middleware you can easily just write your own or use a different express error handler.
That being said I think we should do 3 things:

you should be able to turn off HTML with something like html:false
we should also check to see if the request was XHR and it doesn't accept JSON assume the user forgot to set the Accepts header.
We should provide a default.html page that show a "General Error" for any non 404 or 500 errors.

That sounds reasonable.

Opting in has been added in #25


Saving checkouts in database

Checkout should have a dedicated model that would be stored in the database. This will allow other features such as:

sharing checkouts
creating better checkout API (multiple calls to create/edit checkout)
tracking abandoned checkouts

As we talked internally, we could try with moving fields from Checkout object to Cart model.


Creating and filtering subregistries

In a world where Os are grouped into different subregistries and Bs are expected to commit to a set of subregistries (B needs to execute this commitment step before it can begin sending video to the Os that are a member of the subregistries), a B will have to select subregistries to commit to. The node software could ship with a UI tool that displays the list of all subregistries along with their titles and descriptions (pulled either directly from an on-chain contract or from IPFS using a hash stored on-chain). But how does B decide which subregistries to commit to?
First, we acknowledge that if anyone can arbitrarily create subregistries, then the system can very quickly be spammed with subregistries. As a result, Bs would have a very hard time filtering a massive list of subregistries.
One solution is to incorporate a mechanism for curating the subregistries similar to Althea's use of a global TCR for their mesh network subnets. A global TCR would allow the network to converge on a global leaderboard that ranks subregistries based on perceived quality. However, it doesn't seem optimal for the Livepeer network to converge on a global leaderboard for subregistries since the quality of a subregistry is likely to be context specific i.e. a subregistry containing Os in the eastern US region might be of high quality to Bs that are also in that region, but might be of low quality to Bs that are located in Europe. Bs should pick the subregistries that are the best fit for them. It is unlikely that subregistries will be one size fits all solutions for Bs.
Another solution is for the system to impose a cost for creating a subregistry. For example, a subregistry creator might have to lock X ETH for Y blocks in order for the subregistry to be valid for Y blocks. This cost would at the very least mitigate Sybil attacks that spam the network with useless subregistries.
Assuming that there is some cost for creating a subregistry, why would someone create a subregistry? The creator could create the subregistry contract with a listing renewal fee that is charged on a per block basis (concept borrowed from Althea). If an O wants to be listed in a subregistry it will pay the listing renewal fee which goes to the creator. An O would want to be listed in the most popular subregistries (which might be associated with the most popular geographic regions). The more Os that join a subregistry, the more listing renewal fees accumulated by the creator. If the subregistry does not serve a broadcaster well (i.e. because many Os in the eastern US subregistry are not actually serving output renditions with the latency expected by a broadcaster that is located in the eastern US region) then the broadcaster can commit to a different subregistry. In this situation, any Os that are in that subregistry would exit. Thus, in order to maintain a stream of listing renewal fees, the creator should make sure to curate the subregistry well according to the advertised characteristics of the subregistry (i.e. for a specific geographic region). A more complex DAO governed subregistry could also take the place of the described single creator governed subregistry.

Agree that a global TCR feels sub-optimal.  I think there might already be enough metadata for ranking subregistries without the TCRs.  A few data points like avg 30-day spend or age/spend ratio might be a good starting point.
  It mitigates Sybil attacks without having to spend anything.
I'm torn about the listing renewal fee idea.  I can see this being something people try to game.  We'd have to add a lock period for the fee amount.  I'm also not sure about giving the curation ability to the subregistry creators.
A few thoughts on filtering the subregistries:

Since we envision subregistries to be used for geo-location, we can use that as one of the filters.
ONLY allowing Os to self-select into subregistries can help.  This way SPAMMERS cannot easily create subregistries that look popular.  Popularity can only be built up over time.
Default SPAM filtering in B implementation can help. (For example, client implementations can automatically remove subregistries that clearly look like SPAM - been around for a long time, no usage, and low number of Os).

Also, giving curation ability to the registry creators basically give them the ability to decide who gets work - this feels like a lot of power.

As long as subregistry membership is opt-in, the idea would be for Os to self select into subregistries, but then after that the curator has little control over who gets work because Bs would use their own orchestrator selection mechanism to choose amongst the Os in a subregistry. If the Os that the curator allows (or doesn't allow) into the subregistry is detrimental for Bs, this leaves an opportunity for another curator to create a better subregistry.

ONLY allowing Os to self-select into subregistries can help. This way SPAMMERS cannot easily create subregistries that look popular.

I think subregistries should be opt-in.


Adding Seats with No Activity in 30 days in Seat Analysis

Adding Seats with No Activity in 30 days in Seat Analysis

thanks for the PR, let me know if you have time to make suggested changes


node runs out of memory on very large input

I have a script mkexpr.js that generates a very big expression:
/* global require */

const fs = require('fs');

fs.writeFile('test.wl', mkexpr(1500), err => {
    if (err) {
        console.log(err);
    } else {
        console.log('success');
    }
});

function mkexpr(count) {
    let prefix = '';
    let suffix = '';

    for (let i = 0; i < count; i++) {
        prefix += `With[{x${i} = ${i}},`;
        suffix += ']';
    }
    return `${prefix}${[...Array(count)].map((_, i) => `x${i}`).join('+')}${suffix}`;
}

My grammar wl-grammar.ne is:
@builtin "whitespace.ne"
@builtin "number.ne"

expr -> identifier _ "[" _ "{" _ identifier _ "=" _ int _ "}" _ "," _ expr _ "]"
            {% d => `${d[0]}[{${d[6]}=${d[10]}},${d[16]}]` %}
      | identifier _ "+" _ expr
            {% d => `${d[0]}+${d[4]}` %}
      | identifier
            {% id %}

identifier -> [a-zA-Z]:+ [0-9]:* {% d => `${d[0].join('')}${d[1].join('')}` %}

Then I do node mkexpr.js && nearleyc.js wl-grammar.ne -o wl-grammar.js && more test.wl | nearley-test.js wl-grammar.js
This produces an error:
...
468: {expr → identifier _ "+" _ expr ● }, from: 30465
469: {expr → identifier _ "+" _ expr ● }, from: 30460
470: {expr → identifier _ "+" _ expr ● }, from: 30455

<--- Last few GCs --->

[95384:0x102801600]   112889 ms: Mark-sweep 1400.9 (1434.9) -> 1400.9 (1433.9) MB, 1591.6 / 0.0 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1592 ms) last resort
[95384:0x102801600]   114390 ms: Mark-sweep 1400.9 (1433.9) -> 1400.9 (1433.9) MB, 1501.4 / 0.0 ms  last resort

<--- JS stacktrace --->

==== JS stack trace =========================================

    2: arguments adaptor frame: 3->1
Security context: 0x2c9ebd3a66a1 <JS Object>
    3: map [native array.js:~833] [pc=0x3bbc530dd9e2](this=0xafd6daf4861 <JS Array[5]>,bk=0x17cbb35025f1 <JS Function stringifySymbolSequence (SharedFunctionInfo 0x781fd2964e9)>,bl=0x24eaa9d82311 <undefined>)
    4: arguments adaptor frame: 1->2
    5: toString [/Users/ksushenko/projects/cloudplatform/src/webapp/node_modules/nearley/lib/nearley.js:~18] [pc=0...

FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [/usr/local/bin/node]
 2: node::FatalException(v8::Isolate*, v8::Local<v8::Value>, v8::Local<v8::Message>) [/usr/local/bin/node]
 3: v8::internal::V8::FatalProcessOutOfMemory(char const*, bool) [/usr/local/bin/node]
 4: v8::internal::Factory::NewFixedArray(int, v8::internal::PretenureFlag) [/usr/local/bin/node]
 5: v8::internal::TypeFeedbackVector::New(v8::internal::Isolate*, v8::internal::Handle<v8::internal::TypeFeedbackMetadata>) [/usr/local/bin/node]
 6: v8::internal::SharedFunctionInfo::FindOrCreateLiterals(v8::internal::Handle<v8::internal::SharedFunctionInfo>, v8::internal::Handle<v8::internal::Context>) [/usr/local/bin/node]
 7: v8::internal::JSFunction::EnsureLiterals(v8::internal::Handle<v8::internal::JSFunction>) [/usr/local/bin/node]
 8: v8::internal::Compiler::Compile(v8::internal::Handle<v8::internal::JSFunction>, v8::internal::Compiler::ClearExceptionFlag) [/usr/local/bin/node]
 9: v8::internal::Runtime_CompileLazy(int, v8::internal::Object**, v8::internal::Isolate*) [/usr/local/bin/node]
10: 0x3bbc52f043a7
Abort trap: 6

Perhaps this is related to your work on eliminating the exponential memory consumption when working with ambiguous grammars?

Perhaps this is related to your work on eliminating the exponential memory consumption when working with ambiguous grammars?

No, sorry; the parsing algorithm in master has been stable for a while now (since I did a big push of making things faster).

I’m afraid I don’t have time to look through your grammar, but I suspect it is, in fact, ambiguous :-)
The other option is that it’s right-recursive, but I don’t think that’s the case here.
I hate to say it, but have you tried switching to a tokenizer? It tends to help with performance issues!
Sent with GitHawk

Yes, it is ambiguous, and maybe recursive. That was a part of the test. I wanted to see how much it can handle. And, yes, I am going to switch to a tokenizer. Do you mean that there is not much that can be done in the code to alleviate the memory issues? Other than being careful when writing grammars?

Ah, your grammar is definitely right-recursive, due to the expr -> identifier _ "+" _ expr rule. Earley doesn't like right-recursive rules very much; one of the things on my to-do list is to re-implement Leo's optimisation, which should help with this. For now, you should prefer left-recursive rules where possible (e.g. expr -> expr _ "+" _ identifier), since they perform better. I think that's the problem you're having here.

I don't think it's relevant here, but a note on ambiguity anyway:  While Nearley does provide all parsings for ambiguous grammars/inputs, we don't (yet) use an efficient representation (the preferred one is a "packed parse forest"). Since we end up producing an array containing each parse, performance degrades the more ambiguity you have. Generally this isn't a problem, since you want to aim to write an unambiguous grammar!


Implement closing Edge

Edge should automatically close after karma start if singleRun is enabled. Currently, it stays open but it doesn't seem to cause any issues or warnings with Karma.

I think this functionality would have to be changed in edge-launcher, so I filed an issue.

See #29

Merged #29 as a workaround. Since I'm not sure if upstream can support this directly, I will assume they won't and I'll close this issue.


Poisson variable gene selection

Hi,
One of my main bottlenecks have been selection of highly variable genes.
Here I have implemented the feature selection method from https://github.com/tallulandrews/M3Drop based on increased number of observed zeros than expected from Poisson sampling for biologically variable genes.
This implementation makes use of a sparse count matrix, and utilizes PyTorch for computation. For 300,000 and 30,000 genes it needs about 2GB of memory, either in RAM or GPU.
In the original M3Drop package a Z-test is used to quantify significance for genes with increased zeros (if I read the R code right). Here I have used binomial distributions and sampling using PyTorch distributions to compute the posterior probability of enrichment of zeros for each gene.
For 300,000 cells and 30,000 genes the process takes about 10 seconds with 16 cores of CPU, or about 3 seconds with a Tesla T4 GPU.
I added the method as a mode called 'poisson_zeros' in GeneExpressionDataset. I included it in the tests listed in test_highly_variable_genes.py.
For a demonstration of the method, see this notebook: https://nbviewer.jupyter.org/gist/vals/adeb32f2be95333166347173b03f185e

I hope we can add this to the code!
Best,
/Valentine

I can review this soon. Looks cool. Out of curiosity, have you tried the seurat v3 method with a sparse matrix? It doesn't sparsify X if it's already dense, so you'd have to do it manually. I believe it should be very fast. I also just updated this to use the loess implementation (that's also used in plotnine and written in C) and tested that it gives the same outputs as seurat v3.

also this is a general question, could this have been implemented with numba? And then you wouldn't need to store any matrix in memory? Everything would just be vectors?

I can review this soon. Looks cool. Out of curiosity, have you tried the seurat v3 method with a sparse matrix? It doesn't sparsify X if it's already dense, so you'd have to do it manually. I believe it should be very fast. I also just updated this to use the loess implementation (that's also used in plotnine and written in C) and tested that it gives the same outputs as seurat v3.

I always use sparse matrices, but I've never gotten the 'seurat_v3' mode to run without errors. (These might be related, I always use CSC matrices, could that be the problem?)

also this is a general question, could this have been implemented with numba? And then you wouldn't need to store any matrix in memory? Everything would just be vectors?

I'm not so familiar with Numba. Can it do the summing of outer products without storing the blocks I create now?
Since scVI requires both Numba and PyTorch, which is preferred for computation? Personally I like using PyTorch since I'm always running on GPU instances and I like to take advantage of that.
I implemented the summary statistic calculation in NumPy as well, switching to PyTorch gave a ~5x speed increase on CPU (I guess because of multithreading).
I see the tests are failing, I think I know why: the indices for the minibatches are probably getting calculated wrong when the number of cells are smaller than the batch size. I'll see if I can fix it.

I always use sparse matrices, but I've never gotten the 'seurat_v3' mode to run without errors. (These might be related, I always use CSC matrices, could that be the problem?)\

Have you tried with the latest release? I'm using the same code that I have in a PR at scanpy, which is passing their CI. There was a bug in the earlier code on sparse matrices (before 0.6.5), I also updated the loess implementation.

I'm not so familiar with Numba. Can it do the summing of outer products without storing the blocks I create now?

Yeah it seems like since you're just taking the row sum after the outer product (if I'm reading correctly the outer product gives genes by cells?) you can use elementary for loops. Numba is pretty easy because you can just use numpy and for loops and numpy random binomial seems to work in numba. It's also pretty fun! You just add a decorator to the function.
@numba.njit(cache=True)
def poisson_gene_selection(...)

as long as you use numpy for everything it will be happy.
    expected_fraction_zeros = np.zeros(scaled_means.shape)
    for i in range(n_cells):
        expected_fraction_zeros += total_counts[i] * scaled_means

I don't think numba likes tqdm though.

Since scVI requires both Numba and PyTorch, which is preferred for computation? Personally I like using PyTorch since I'm always running on GPU instances and I like to take advantage of that.

I'm not sure maybe @romain-lopez has a preference. I guess if numba actually reduces the memory footprint then that would be preferable no?

@vals we are refactoring the dataset module to basically only take anndata as input. I'm going to hold off on this until we have that done, and then we can add this feature. I imagine this function will then have to be something that operates directly on anndata.

Sure, sounds good.  So should be quick to reroute what it calculates on.


fix(lint): enables goerr113

Fixes #480

I think we can simply merge locally and amend the commit message with /skip-e2e when #789 is merged.


fix: make sure any disk backed stores are open with write perms

feat: move long running tests to ignore(d) runs

This is a more complete PR inspired by https://github.com/filecoin-project/merkletree/pull/92


ND4J Version Check: Add way to detect mixed Spark versions on cluster

ND4J's version check warns about mixed versions on the classpath:

Mismatched DL4J/ND4J dependencies
Mismatched Scala 2.10/2.11 dependencies

However, it has no way currently to  detect running Spark 1 code on a Spark 2 cluster (or vice versa) - the Spark dependencies are added by Spark submit at runtime (i.e., Spark adds stuff to the job classpath), not by the user when building their jar.
Errors like this can lead to obscure errors like:

java.lang.AbstractMethodError: org.deeplearning4j.spark.api.worker.ExecuteWorkerPathFlatMap.call(Ljava/lang/Object;)Ljava/util/Iterator;

For users that have not come across this sort of error before, it is likely not apparent that they are using conflicting Spark versions.
Aha! Link: https://skymindai.aha.io/features/ND4J-148

Spark 1 support was removed, so issue isn't relevant anymore.


Moving from interactions.py to hikari+lightbulb

Translation progress

[x] Main script
[x] Core
[x] Listeners
[x] Tabletop
[x] Git
[x] gw2

pain, well I hope guess it works


Bail out of training immediately if no training data batches are provided

I propose we bail out immediately if no training data is found, currently we have this:
                    batch_gen = self.corpus_reader.train_batch_gen()

                    train_ler_total = 0
                    print("\tBatch...", end="")
                    for batch_i, batch in enumerate(batch_gen):
                        print("%d..." % batch_i, end="")
                        sys.stdout.flush()
                        batch_x, batch_x_lens, batch_y = batch

                        feed_dict = {self.batch_x: batch_x,
                                    self.batch_x_lens: batch_x_lens,
                                    self.batch_y: batch_y}

                        _, ler, = sess.run([self.optimizer, self.ler],
                                        feed_dict=feed_dict)

                        train_ler_total += ler

But if batch_gen provides nothing we have an error state where the code will break at some future unspecified point.
Instead I'm thinking we can do something like this:
                    batch_gen = self.corpus_reader.train_batch_gen()

                    train_ler_total = 0
                    print("\tBatch...", end="")
                    for batch_i, batch in enumerate(batch_gen):
                        print("%d..." % batch_i, end="")
                        sys.stdout.flush()
                        batch_x, batch_x_lens, batch_y = batch

                        feed_dict = {self.batch_x: batch_x,
                                    self.batch_x_lens: batch_x_lens,
                                    self.batch_y: batch_y}

                        _, ler, = sess.run([self.optimizer, self.ler],
                                        feed_dict=feed_dict)

                        train_ler_total += ler
                    else:
                        raise PersephoneError("No training data was provided, check your batch generation")

This would make it really clear that something has gone wrong and would immediately report the failure.

This is related closely to #184 and #185

There's actually a regression here because previously working test cases now fail such as this one: https://travis-ci.org/persephone-tools/persephone-web-API/jobs/441199123

"There's actually a regression here"

Shush! The world is watching. Could you kindly refer to this as, let's say, deep regression, or (better) vanilla deep regression? :speak_no_evil:

Sorry about that, I've changed it back. I shouldn't have been working on master directly.


Setting max number of lines doesn't appear to work for DataDistributor

For the the number of lines passed in via cmd line is 80. Regardless of this number, the output is always 5 lines.
dist = dist.setMaxNumberOfLines(80);
The default example (output to console) doesn't appear to set maxNumberOfLines for the distributor, yet it produces the same number of lines on each run. Is this variable set elsewhere in the code?

We'll try to reproduce it. Just as a general rule, the number of lines are the max so as long as you're generating fewer number of lines it won't stop. Also, it won't attempt to duplicate or repeat the number of lines to reach the max.

Thanks for your help!
We tried in increase the number of permutations across the columns and that fixed it. It was as you mentioned, the engine didn't duplicate any lines to meet the maximum and we had too few permutations in our code to do this.
We are finding, however, that as we introduce ranges to columns, the operation as a whole tends to go much slower. What optimizations can be made to generate large volumes of data (i,e,  increasing thread count)?

@anjueappen yes you are correct. If you're trying to generate data more than the heap allocated for you jvm, it will eventually run out of memory and error out.
As we understand now, you've reached a solution for this issue. I'll close this bug and please let us know if you have other questions or issues.
Thanks


chore: Bump async-lsp to v0.0.5

Description

Problem*

Ref #2168
Summary*

WIP to get us on latest async-lsp changes to make sure they work and then we can request a new official release
Documentation

[ ] This PR requires documentation updates when merged.

[ ] I will submit a noir-lang/docs PR.

[ ] I will request for and support Dev Rel's help in documenting this PR.

Additional Context

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

@phated async-lsp has made a new release so we can get off of the patched version now. I'm going to let you be in charge of deciding to merge this.

One thing that I'd still like to do here is fix the errors returned so we can use our own thiserror stuff instead of creating async-lsp errors.

One thing that I'd still like to do here is fix the errors returned so we can use our own thiserror stuff instead of creating async-lsp errors.

Notifications still don't allow returning custom errors correctly so I'm going to skip that for now. This should be good to land.


_ssl.c errors

en.wikipedia.org -> _ssl.c:733: The handshake operation timed out
github.com, google.com, yandex.ru -> SSLError: [SSL: UNKNOWN_PROTOCOL] unknown protocol (_ssl.c:749)
twitter.com -> SSLEOFError: EOF occured in violation of protocol (_ssl.c:749)

In all of these cases it's very hard to know without additional details what's going on. What version of Python and OpenSSL are you using?

Anaconda custom-py36_0 based on 4.4.0-np112py36_1 x86 (32 bit) (just fully updated that version)
python 3.6.1.final.0

What about OpenSSL?

openssl 1.0.2l

Hrm. Can you show me some sample code that reproduces the issue?

https://pastebin.com/GRCYVVEd

Hrm, this is not obviously problematic. Are you familiar with Wireshark? If you are, it'd be helpful to see the pcap for one of these failing transactions.

You need to specify a port along with hostname.
--- t.original.py	2017-07-31 14:41:55.000000000 +0300
+++ t.fixed.py	2017-07-31 14:43:29.000000000 +0300
@@ -2,11 +2,13 @@
 from urllib.parse import urlparse
 from hyper import HTTPConnection

+default_port = dict(http=80, https=443)
+
 def makeActualReq(uri, referer=None, maxRedirects=10):
 	if maxRedirects<=0:
 		raise Exception("Redirects limit reached")
 	u=urlparse(uri)
-	with HTTPConnection(u.hostname, secure=(u.scheme=="https")) as conn:
+	with HTTPConnection(u.hostname, port=(u.port or default_port.get(u.scheme.lower())), secure=(u.scheme=="https")) as conn:
 		#print(u.path)
 		conn.request('GET', u.path)
 		resp=conn.get_response()

Thank you, it looks like HTTP11Connection should be modified.

Thanks for that tip @KostyaEsmukov
It helped me here: https://github.com/Lukasa/hyper/issues/408#issuecomment-486232660

See this comment: https://github.com/Lukasa/hyper/pull/407#issuecomment-486234837
I was running pytest with hyper in Python 3.7 and was getting that SSL warning. But as soon as I installed ndg-httpsclient that (same) SSL error disappeared.


abnormal palatine bone perpendicular plate morphology

abnormal palatine bone perpendicular plate morphology (child of abnormal palatine bone morphology MP:0005249)
DEF: any structural anomaly of the part of the palatine bone that extends vertically upward from the horizontal lamina; it forms part of the lateral wall of the nasal cavity
SYN: abnormal palatine bone vertical plate morphology
SYN: abnormal lamina perpendicularis ossis palatini morphology
SOURCE: https://medical-dictionary.thefreedictionary.com/perpendicular+plate+of+palatine+bone

abnormal palatine bone perpendicular plate morphology MP:0030333


Does your website send our code to your server ?

Hi all !
I think all about is in the title ;)
Regards,
Aewil

Nope!
https://github.com/js2coffee/js2coffee/tree/gh-pages
https://github.com/js2coffee/js2coffee/blob/gh-pages/assets/app.js

Quick answer, thanks you ! And fpr ur great app too ;)


HeatMap error: "Unable to get property 'create' of undefined or null reference" on h337.create

I am trying to create heatmap as on "https://www.patrick-wied.at/static/heatmapjs/example-heatmap-leaflet.html"
bt getting error on line number 56 of leaflet-heatmap.js i.e "this._heatmap = h337.create(this.cfg);" as "Unable to get property 'create' of undefined or null reference"
Can anybody tell why am I getting this???

I am having the same problem. Any advice is appreciated!

i am having same issue . Any idea how to resolve
ReferenceError: h337 is not defined

@paraskashyap are you using a script blocker? apparently heatmap.js (the file name) is on some lists that power script blockers (thus preventing it to initialize h337)

@pa7 example page is working fine .I am trying to integrate heatmap with my project and i tried in different system too and the result is same "h337 is not defined".

Same issue here on Chrome, Firefox, Brave - all with extensions disabled.

I had the same issue. The problem was that i was not importing heatmap correctly...
I was using import { h337 } from 'heatmap.js';
The correct import was import h337 from 'heatmap.js';


Appstudio update test-component-pac-kart

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-kart-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
9 seconds
appstudio-init

✅ Succeeded
14 seconds
clone-repository

✅ Succeeded
9 seconds
appstudio-configure-build

✅ Succeeded
38 seconds
build-container

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-kart-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
9 seconds
appstudio-init

✅ Succeeded
16 seconds
clone-repository

✅ Succeeded
9 seconds
appstudio-configure-build

✅ Succeeded
37 seconds
build-container

✅ Succeeded
7 seconds
show-summary


Document CGRect, etc. that have Swift methods

There are various types from Foundation that gain extra methods in Swift. For example, CGRect has a bunch of methods/properties. Here's the definition as Swift sees it:
struct CGRect {
    var origin: CGPoint
    var size: CGSize
}

extension CGRect {
    static var zeroRect: CGRect { get }
    static var nullRect: CGRect { get }
    static var infiniteRect: CGRect { get }
    init()
    init(x: CGFloat, y: CGFloat, width: CGFloat, height: CGFloat)
    init(x: Double, y: Double, width: Double, height: Double)
    init(x: Int, y: Int, width: Int, height: Int)
    var width: CGFloat { get }
    var height: CGFloat { get }
    var minX: CGFloat { get }
    var midX: CGFloat { get }
    var maxX: CGFloat { get }
    var minY: CGFloat { get }
    var midY: CGFloat { get }
    var maxY: CGFloat { get }
    var isNull: Bool { get }
    var isEmpty: Bool { get }
    var isInfinite: Bool { get }
    var standardizedRect: CGRect { get }
    mutating func standardize()
    var integerRect: CGRect { get }
    mutating func integerize()
    func rectByInsetting(#dx: CGFloat, dy: CGFloat) -> CGRect
    mutating func inset(#dx: CGFloat, dy: CGFloat)
    func rectByOffsetting(#dx: CGFloat, dy: CGFloat) -> CGRect
    mutating func offset(#dx: CGFloat, dy: CGFloat)
    func rectByUnion(withRect: CGRect) -> CGRect
    mutating func union(withRect: CGRect)
    func rectByIntersecting(withRect: CGRect) -> CGRect
    mutating func intersect(withRect: CGRect)
    func rectsByDividing(atDistance: CGFloat, fromEdge: CGRectEdge) -> (slice: CGRect, remainder: CGRect)
    func contains(rect: CGRect) -> Bool
    func contains(point: CGPoint) -> Bool
    func intersects(rect: CGRect) -> Bool
}

extension CGRect : Equatable {
}

extension CGRect : Reflectable {

    /// Returns a mirror that reflects `self`.
    func getMirror() -> MirrorType
}

The other types defined in CoreGraphics.CGGeometry also have Swift additions, although far less than CGRect (mostly just an appropriate "zero" static and protocol conformance).
It would be extremely useful if SwiftDoc.org could document this stuff similar to how it documents Foundation extensions to Swift types (e.g. String).

Similarly, the ObjectiveC module has a function autoreleasepool() that should be documented. There may be other stuff there as well, I'm not sure (this function is notable for not being a translation of an Obj-C API, since Obj-C uses the builtin @autoreleasepool construct instead).

@kballard, thanks for opening this issue. I'm definitely straddling the fence on the question of SwiftDoc.org's scope, so I think I want to have a little more discussion before moving on this. My thoughts sort of for and against:

I like that SwiftDoc.org is nearly only the Swift standard library. In the absence of actual documentation from Apple, I think it's important for there to be a resource to use both while coding and also to learn about what the Swift language offers on its own.

I added the Foundation additions to String mainly because while String is a built-in type, the Foundation additions increase its capabilities a huge amount, and let's face it, if you're using Swift today you're probably also using Foundation. However, that String already exists in Swift feels like an important distinction.

If we add CGRect, CGPoint, etc. to the types on SwiftDoc.org, where do we stop? If there are only a few additions in an extension, like for NSArray, is it confusing to only include those? Would we need to also include the rest of the NSArray declaration?

At the same time, none of the CGGeometry changes are reflected anywhere else, and they're really useful. (I covered a chunk of them in an update here.)

There are also new types that are Swift-only but live in Foundation, like NSFastGenerator and NSIndexSetGenerator, as well as functions like autoreleasepool() and NSLocalizedString() that are different than the related concepts in Objective-C.

In short, I'm not sure where I stand. :smile: Let's talk a little more!

While I generally would prefer (especially for beginners) if SwiftDoc covered the whole frameworks, given the existence of third-party Swift implementations like Silver (from RemObjects), I would recommend clearly marking things that aren't part of the standard library. The SelfHTML tutorial did that nicely: They had an icon for each browser/browser version and showed that at the top of each entry. So you could immediately see whether a call is available to you.

In a way, I feel better about including extensions to CGRect than I do about those String extensions.  In the case of CGRect, it is a struct, and the extensions are pure operations on that struct, with the same representation as in C.  The String extensions make me feel a little uneasy given the possibility they’re converting your String into an NSString and then doing operations on that.

My general feeling is that when doing iOS development, I should be able to see all of the documentation on stuff I can use by using the iOS docset and swiftdoc.org (or more accurately, the Dash docset generated from swiftdoc.org). Since the iOS docset doesn't document the Swift-specific extensions to CGGeometry types (or autoreleasepool and other things of that nature) I want swiftdoc.org to handle that.
I do see the utility in having documentation on just "pure Swift", but swiftdoc.org has already broken away from that by including Foundation extensions to String. That gives precedent to documenting other Swift-only things that come from the Foundation module (or one of the modules it includes, such as ObjectiveC).
In an ideal world, SwiftDoc would have a toggle that says "include Foundation things" that, when disabled, would give only pure Swift documentation (including removing the Foundation extensions on String). But I don't think that's useful enough to be worth the effort of implementing. Instead SwiftDoc should just be certain to clearly indicate which APIs come from Foundation.

@kballard I think you have me convinced! I'll leave this open while I get the changes built, in case anyone has any other feedback. If you want, feel free to submit a PR with the declarations you'd like to see in a new file in the data/addenda directory.


Changed the query language from JMESPath to JSONPath

Updated the proposal to use JSONPath based on the current draft https://www.ietf.org/id/draft-ietf-jsonpath-base-04.html
JMESPath was originally selected because it had a specification and an extensive test suite.  However, experimentation has shown that JMESPath is optimized to transforming the input JSON document into an output JSON document.  The goal is to select and project.  However, for Overlays, the objective is to select a set of elements in the target document.  There is no need for the ability to project new JSON objects.
Since the original draft of the Overlay specification was created, there has been an effort in the IETF to standardize the JSONPath specification.  The original JSONPath definition relied on scripting support from the underlying implementation.  This would not be suitable for the Overlay specification.  The new IETF version is taking efforts to make the JSONPath use a fully specified query expression that is not implementation dependent.

Link to the general page of JSONPath so I can remember: https://tools.ietf.org/wg/jsonpath/

@webron can you approve/merge?

I'm looking at implementing something to do overlays; can I assume this will be merged? I think it should be, if that counts at all!

Uh... current draft is https://datatracker.ietf.org/doc/html/draft-ietf-jsonpath-base


Build/deploy book using GitHub actions

Use GitHub actions to build & deploy book after updates to the master branch.

We are currently using GitHub Pages to deploy the book, which is automatically updated from the master branch. Wont use GitHub Actions for now.


dnn_dropout in DNNLinearCombinedClassifier doesn't  work

I use DNNLinearCombinedClassifier to train a wide & deep model with dnn_dropout=0.5。
m = tf.estimator.DNNLinearCombinedClassifier(
    model_dir=model_dir,
    linear_feature_columns=crossed_columns,
    dnn_feature_columns=deep_columns,
    dnn_dropout=0.5,
    dnn_hidden_units=[512, 256, 256, 128])

m.train(
    input_fn=input_fn(train_file_name, num_epochs=None, shuffle=True,
    is_eval=False,steps=train_steps,
    hooks=[logger_hook])

Then I evaluate model as follow:
m = tf.estimator.DNNLinearCombinedClassifier(
    model_dir=model_dir,
    linear_feature_columns=crossed_columns,
    dnn_feature_columns=deep_columns,
    dnn_dropout=1.0,
    dnn_hidden_units=[512, 256, 256, 128])

results = m.evaluate(
    input_fn=input_fn(test_file_name, num_epochs=1, shuffle=False, is_eval=True),
    steps=None)

I find no matter I set dnn_dropout=0.1 or dnn_dropout=1.0, the result are same. It seems that this parameter doesn't work.
My version is 1.4

TF will raise an ValueError exception if dnn_dropout=1.0, did you forget to call train at second time ? Anyway you can check it by using two different model_dir.

@jony0917 Trying dnn_dropout=0.1 and 0.8, whatever any kind of droupout ratio can get the same result.

Can anyone help？

Nagging Assignee @shivaniag: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @shivaniag: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @shivaniag: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @shivaniag: It has been 89 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I'm also waiting for the answer.

Nagging Assignee @shivaniag: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @shivaniag: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@333caowei ,
We see that you are using tf version 1.4.0, 1.x is not actively supported, please update to 2.x and let us know if you are facing same issue.Thanks!


Performance is really bad on version 1.0.34

I just updated to version 1.0.34 from a previous version, and it made IntelliJ unusable for editing.  There is a two-second lag between every key I type before it is displayed in the editor.  I tried switching between having the ESLint plugin enabled and disabled (I restart IntelliJ in beween) and the ESLint plugin is definitely the culprit.  I find it useful to have the ESLint highlighting, but I will leave it disabled until a new version is available.

Please provide information regarding your environment.
OS type and version, eslint and node versions etc.

I am on Windows 10, and I had just updated to IntelliJ version 2016.2 when I had the problem.  A lot of the problem may have been the newer version of IntelliJ, because I was getting a lot of other problems. I have now gone back to IntelliJ version 2016.1.3, and things are running much better.  So I apologize for putting the full blame on the es-lint plugin.


Add option to Customize Toolbar for SQL and Data editor preferences

Is your feature request related to a problem? Please describe.
For now we can customize SQL/Data Editor toolbars from the Main Preferences only
Describe the solution you'd like
It will be great if Customize toolbars options will be available from "local" preferences for SQL and Data editors

We have an error in the logs
 !MESSAGE Invalid property category path: org.jkiss.dbeaver.preferences.main (bundle: org.jkiss.dbeaver.core, propertyPage: org.jkiss.dbeaver.preferences.main.toolbar.customization)


installed rasa but getting this message

installed rasa but gettin this message
can anyone help
C:\Users\Sanjiw Jha>rasa
Traceback (most recent call last):
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\Sanjiw Jha\AppData\Local\Programs\Python\Python37\Scripts\rasa.exe_main.py", line 7, in
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\rasa_main.py", line 82, in main
set_log_level(log_level)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\rasa\utils\common.py", line 71, in set_log_level
update_tensorflow_log_level()
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\rasa\utils\common.py", line 112, in update_tensorflow_log_level
import tensorflow as tf
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_init_.py", line 101, in
from tensorflow_core import *
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core_init_.py", line 40, in
from tensorflow.python.tools import module_util as module_util
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_init.py", line 50, in getattr
module = self.load()
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_init.py", line 44, in _load
module = importlib.import_module(self.name)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\importlib_init.py", line 127, in import_module
return _bootstrap.gcd_import(name[level:], package, level)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python_init.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "c:\users\sanjiw jha\appdata\local\programs\python\python37\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: A dynamic link library (DLL) initialization routine failed.
Failed to load the native TensorFlow runtime.
Originally posted by @kramitjha in https://github.com/RasaHQ/rasa/issues/3852#issuecomment-658339543

I've got the same issue
(base) D:\PYTHON>rasa --version
Traceback (most recent call last):
File "d:\anaconda\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "d:\anaconda\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "d:\anaconda\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "d:\anaconda\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "d:\anaconda\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: The specified module could not be found.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "d:\anaconda\lib\runpy.py", line 193, in run_module_as_main
"main", mod_spec)
File "d:\anaconda\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "D:\ANACONDA\Scripts\rasa.exe_main.py", line 7, in
File "d:\anaconda\lib\site-packages\rasa_main.py", line 82, in main
set_log_level(log_level)
File "d:\anaconda\lib\site-packages\rasa\utils\common.py", line 71, in set_log_level
update_tensorflow_log_level()
File "d:\anaconda\lib\site-packages\rasa\utils\common.py", line 112, in update_tensorflow_log_level
import tensorflow as tf
File "d:\anaconda\lib\site-packages\tensorflow_init_.py", line 101, in
from tensorflow_core import *
File "d:\anaconda\lib\site-packages\tensorflow_core_init_.py", line 40, in
from tensorflow.python.tools import module_util as module_util
File "d:\anaconda\lib\site-packages\tensorflow_init.py", line 50, in getattr
module = self.load()
File "d:\anaconda\lib\site-packages\tensorflow_init.py", line 44, in _load
module = importlib.import_module(self.name)
File "d:\anaconda\lib\importlib_init.py", line 127, in import_module
return _bootstrap.gcd_import(name[level:], package, level)
File "d:\anaconda\lib\site-packages\tensorflow_core\python_init.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "d:\anaconda\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "d:\anaconda\lib\site-packages\tensorflow_core\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "d:\anaconda\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "d:\anaconda\lib\site-packages\tensorflow_core\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "d:\anaconda\lib\imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "d:\anaconda\lib\imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: DLL load failed: The specified module could not be found.
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

Thanks for raising this issue, @koaning will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

I even tried installing ujson
Then when i reinstalled rasa it gave me this error
ERROR: Command errored out with exit status 1:
command: 'D:\ANACONDA\envs\insrasa\python.exe' 'D:\ANACONDA\envs\insrasa\lib\site-packages\pip_vendor\pep517_in_process.py' build_wheel 'C:\Users\Sanjana\AppData\Local\Temp\tmpu7jig9ss'
cwd: C:\Users\Sanjana\AppData\Local\Temp\pip-install-lptg15wd\ujson
Complete output (5 lines):
running bdist_wheel
running build
running build_ext
building 'ujson' extension
error: Microsoft Visual C++ 14.0 is required. Get it with "Build Tools for Visual Studio": https://visualstudio.microsoft.com/downloads/
ERROR: Failed building wheel for ujson
Failed to build ujson
ERROR: Could not build wheels for ujson which use PEP 517 and cannot be installed directly
I even downloaded the build tools..could you please help out

Have you tried following the steps in this video?

Yeah I did
On Mon, Aug 17, 2020, 2:11 PM vincent d warmerdam<EMAIL_ADDRESS>wrote:

Have you tried following the steps in this video
https://www.youtube.com/watch?v=4ewIABo0OkU?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RasaHQ/rasa/issues/6217#issuecomment-674746082, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AN5NHAYKIMXCAD7JQDTRNX3SBDUMBANCNFSM4O23RP6A
.


chplx::forall over ranges

The following code in backend/test/heat.chpl segmentation faults. The sequential version of the loop is working correctly.
Parallel version
        chplx::forall(chplx::Range(0, 4), [&](auto i) {
#line 44 "heat.chpl"
            data(i) = 1 + (((i - 1) + nx) % nx);
#line 45 "heat.chpl"
            data2(i) = 0;
        });

Sequential version
        chplx::forLoop(chplx::Range(0, 4), [&](auto i) {
#line 44 "heat.chpl"
            data(i) = 1 + (((i - 1) + nx) % nx);
#line 45 "heat.chpl"
            data2(i) = 0;
        });

@hkaiser ran the unit tests and got a similar segfault. I'll have to double back and see if this is an issue on x86 (I'm working off arm at the moment).


Revert "Clarifications to bring the CP component clusters using LB"

This reverts commit a207eeb25db5dd12304be485d1b16eab77a5dd8a.

@arodoni , thank you for putting this PR together.
I think we should leave the readme file as is and not remove the DNS portion of things as it's a good example of what is needed.
The only change that I think is needed is to the yaml file adding a default value.

lets leave the README portion as is please

Did put back the README file per request.


Please do

make this packageable for distros. Arch Linux, but Ubuntu would be fine, then it can be adapted for Arch. :D

He said before that porting it to different architectures would be more of the open source communities job, as this wasn't originally supposed to be an open source project, so It would be a good idea if someone could make a fork for this.

For sure. Is there a video of this?

AFAIK there's only the small video that was posted to /r/unixporn.

niice!

As @r0nk said, this is not something I plan on doing, but I've added some explanation to the README on how to make it run.


[doc] Error when creating a tenant in IS-KM

Description:
When trying to create a new tenant, following error log can be seen. Tenant creation is successful anyway.

ERROR {org.wso2.carbon.apimgt.impl.ThrottlePolicyDeploymentManager} -  Error while deploying policy to global policy server.The service cannot be found for the endpoint reference (EPR) https://localhost:9443/services/EventProcessorAdminService

What happen here is, when a new tenant is created, it tries to deploy the throttle policies to the KM node itself which doesn't have event processor features. To fix this, we have to change below configuration in KM's api-manager.xml to point to Traffic Manager node.
<PolicyDeployer>
    <ServiceURL>https://localhost:${mgt.transport.https.port}${carbon.context}services/</ServiceURL>
    <Username>${admin.username}</Username>
    <Password>${admin.password}</Password>
</PolicyDeployer>

It is better to include this in the APIM docs for configuring IS as KM.
Suggested Labels:
Doc, Improvement
[1] https://docs.wso2.com/display/AM2xx/Configuring+WSO2+Identity+Server+as+a+Key+Manager
[2] https://docs.wso2.com/display/AM210/Configuring+WSO2+Identity+Server+as+a+Key+Manager
[3] https://docs.wso2.com/display/AM210/Configuring+an+External+IdP+as+a+Key+Manager
[4] https://docs.wso2.com/display/AM210/Configuring+an+External+IdP+as+a+Key+Manager

I’m closing this issue because it has been inactive for more than two years. Please reopen if you still encounter this issue with the latest version.


Move gtag init code to root

Problem:
Trying to load the root page (i.e utopia.app or utopia.pizza) and consent to cookies breaks this page forever
Fix:
Fix the gtag initialization code as per Remix's recommendation:
https://github.com/remix-run/examples/blob/main/google-analytics/app/root.tsx

@ruggi it doesn't work there, probably due to how Remix is doing SSR vs CSR in inner routes.
(see Remix's recommendation on where to initialize it)
It's just the initialization code though (no events are being fired).
The alternative is to set it using useEffect in the component itself.


Model: Permit

Create a Django model for storing permits
We may want to revisit our terminology. "Case" instead of "Permit"?
Fields:

case number (unique)
address
location (django.contrib.gis.db.PointField)
status
summary (what we're currently calling description)
description (longer description)

Whoops. This is redundant. See issue #65.


[gtk] Build error

Host Environment

OS: Windows

Compiler: revision
vcpkg-tool version: 2022-09-20-522aa94e9d261c7d7b2f079bf2591ca62df5c714
vcpkg-scripts version: 42cc830d8 2022-10-10 (20 hours ago)

To Reproduce
Steps to reproduce the behavior:
 .\vcpkg.exe install gtk:x86-windows
Failure logs
-- Downloading https://gitlab.gnome.org//GNOME/gtk/-/archive/aec7ca82007dbe07faee6be084d20758ebac2b91/gtk-aec7ca82007dbe07faee6be084d20758ebac2b91.tar.gz -> GNOME-gtk-aec7ca82007dbe07faee6be084d20758ebac2b91.tar.gz...
-- Extracting source C:/Users/yingshaoxo/Desktop/Git-2.38.0-64-bit.tar/Git-2.38.0-64-bit/vcpkg/downloads/GNOME-gtk-aec7ca82007dbe07faee6be084d20758ebac2b91.tar.gz
-- Applying patch 0001-build.patch
-- Applying patch 0002-windows-build.patch
-- Applying patch 0004-macos-build.patch
-- Using source at C:/Users/yingshaoxo/Desktop/Git-2.38.0-64-bit.tar/Git-2.38.0-64-bit/vcpkg/buildtrees/gtk/src/58ebac2b91-58f2f5c120.clean
-- Using cached msys-mingw-w64-i686-pkg-config-0.29.2-3-any.pkg.tar.zst.
-- Using cached msys-mingw-w64-i686-libwinpthread-git-<IP_ADDRESS>73.5be8fcd83-1-any.pkg.tar.zst.
-- Using msys root at C:/Users/yingshaoxo/Desktop/Git-2.38.0-64-bit.tar/Git-2.38.0-64-bit/vcpkg/downloads/tools/msys2/9a1ec3f33446b195
-- Found external ninja('1.10.2').
-- Getting CMake variables for x86-windows
-- Configuring x86-windows-dbg
-- Configuring x86-windows-dbg done
-- Configuring x86-windows-rel
-- Configuring x86-windows-rel done
-- Package x86-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:96 (message):
    Command failed: "C:/Program Files/Microsoft Visual Studio/2022/Community/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja.exe" install -v
    Working Directory: C:/Users/yingshaoxo/Desktop/Git-2.38.0-64-bit.tar/Git-2.38.0-64-bit/vcpkg/buildtrees/gtk/x86-windows-dbg
    Error code: 1
    See logs for more information:
      C:\Users\yingshaoxo\Desktop\Git-2.38.0-64-bit.tar\Git-2.38.0-64-bit\vcpkg\buildtrees\gtk\package-x86-windows-dbg-out.log
      C:\Users\yingshaoxo\Desktop\Git-2.38.0-64-bit.tar\Git-2.38.0-64-bit\vcpkg\buildtrees\gtk\package-x86-windows-dbg-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_install_meson.cmake:33 (vcpkg_execute_required_process)
  ports/gtk/portfile.cmake:72 (vcpkg_install_meson)
  scripts/ports.cmake:147 (include)

error: building gtk:x86-windows failed with: BUILD_FAILED
error: Please ensure you're using the latest port files with `git pull` and `vcpkg update`.

Could you please provide us with C:\Users\yingshaoxo\Desktop\Git-2.38.0-64-bit.tar\Git-2.38.0-64-bit\vcpkg\buildtrees\gtk\package-x86-windows-dbg-err.log?
Also, try to put the path of vcpkg under the drive letter, such as F: vcpkg. Too long file paths may cause other problems.

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


Retrun affected upon updating

Is your feature request related to a problem? Please describe.
Recently I was in need to get back the affected row during update.
Describe the solution you'd like
Since version 3.35.0 (2021-03-12), SQLite includes a non-standard RETURNING clause that allows for what you are looking for.
So, just like insertReturning
    final access = await into(userAccessTable).insertReturning(userAccess);

It would be great to have updateReturing

Good idea, I have added a returning variant for updates and deletes in 080ef7210c4017b3a6b5341850af08b5596a28f9.


Fix overwriting unstable error code

Stop overwriting bubbled up error with a generic error.

Has this been manually or CI tested for a non-generic error case?

I was able to force the case locally. It will return a 2 for an unstable state now instead of using the generic error code


Exception Handling Improvements

Replaces #214

@rgs258 xan you re run precommit on it?

@Andrew-Chen-Wang this is updated for the latest on main. It would be great to get this into the repo as it's non-trivial to resolve conflicts as the package move forward.

@Andrew-Chen-Wang this one is updated for the latest on main.


Set WM "Urgent" hint

Hi!
Is there a way to set window manager "urgent" hint when a message arrives?

Yes, I think so. Is there a specific reason why it should be different than normal?


List entries no worky with current atom master

It just pops up for a second, and hides. :(

Thanks for the report. I think it works now.

❤


Implement MCP in Mixer

This includes: Adding a store-backend for Mixer to listen to MCP.

https://github.com/istio/istio/pull/7451


Cherry pick 'istioctl describe' fixes

Bring in https://github.com/istio/istio/pull/17772 and https://github.com/istio/istio/pull/17922 from Master.
This gives the istioctl x describe svc <svc> subcommand to 1.4.  It also puts master and 1.4 on the same code so that the fix for https://github.com/istio/istio/issues/17833 that I am working on can go to both master and 1.4.

@esnible: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

unit-tests_istio_release-1.4
feb57c4b99cba4c9f397d81cb077b6c3ceb2da97
link
/test unit-tests_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


small fixes and new props

added ignore for .idea;
fixed css missing spaces tag title; (when a tag had spaces "no mine", and if the search term was "no" or "mi" the space dissapeared.
added prop to add custom classes to main input;
added addNew optional property for each category; I made a "custom keywords" category and let the user only add there, this way the "add new ..." will noe appear 10 times if you have 10 categories.
removed duplicate field from README
added prop inputAutoSize to control the size attribute of the main input ; if the size is controlled with CSS example bootstrap, using the "size" mess the layout.

I've done the changes you ask, and by mistake I pushed a new feature, I hope is ok https://github.com/erizocosmico/react-categorized-tag-input/pull/22/commits/8f8789bfb723e51272d9360eb6f97114ee5b05bb
It was weird as in the user sees "Fruits" category but can't see all the fruits available, Instead of making a helper page to list all the tags, the search itself can be be used to discover what tags are available. I have over 100 tags so it's a real problem for me.

:+1: for the changes.
About the new feature, I think it's okay, but I'd rather have it display all the category tags only if there is no tag that matches the input. Otherwise, if the category has the same name as a tag, it's difficult to select that tag.

Makes more sense to show all of them by default, trough sometimes will require a scroll. I do not know how to remove a commit from the pull request though. The chance that a category has the same title as a tag is kinda low if I think about it.
I made other small features too, if this works maybe I'll make another push request next week. Example to change the minimum input length (now is hard coded 1).


[release-1.24] helm chart fixes

This cherry-picks the changes from https://github.com/kubernetes-sigs/descheduler/pull/814 and https://github.com/kubernetes-sigs/descheduler/pull/813 to fix helm chart changes that should have been included in 1.24

/approve


Fatal error $this->clockwork is not present in RequestFilter::postRequest()

With Silverstripe 3.1.12 I get this error? :
PHP Fatal error:  Call to a member function getRequest() on a non-object in /clockwork/src/RequestFilter.php on line 63

Not sure if this is good practice, but it seems to work...
changing protected $clockwork; to protected static $clockwork;
http://sspaste.com/paste/show/5511b1afde02f

I think this was actually fixed in PR #5.


Hide Application URLs

Hi!
Maybe I missed the obvious, but is there any setting to hide the URL that is shown under each applications name?
Thx,
Michael

You can hide it with this custom CSS:
.AppCard_AppCardDetails__tbAhY > span {
  display: none;
}

Duplicate of #105


Dashboard demo

In this issue (which might be broken up to separate issues later), visuals will begin by providing hi-fi examples of various dashboards with our components. After this, we will determine which make sense to add as demos.
Design Folder: https://drive.google.com/open?id=1kRZClTWey5ZtDPK81Yrk2ZozRugoZyPx
Consider both full page dashboards and "dashbars."
Component lifecycle:

[x] Interaction design requirements
[ ] Visuals
[ ] First checkpoint complete and approved
[ ] CSS
[ ] React
[ ] Second checkpoint complete and approved

Contacts:

IxD
Visual
CSS

Hi @mceledonia do you mind attaching the final designs to our dashboard issue?

Sure thing!
Dashboards 3.zip

@mcoker @rachael-phillips @mceledonia and I met today to discuss next steps on this. We have a set of design examples to represent examples of dashboards. As we move towards implementation, some considerations are:

How to use the existing grid layouts in PF4 to achieve a responsive dashboard that works at any viewport size/ What our the expectations for mobile?
What about card types? We have templates for different card layouts - see https://github.com/patternfly/patternfly-next/issues/355. But how do we recommend that designers and developers apply these? Do we want a standard set of card components that encapsulate a fixed set of common layouts or provide developers with examples to help them "roll their own" card configurations. The later is more complex on the developer side but gives designers more flexibility.

We agreed that first/next steps here are to select a small number of dashboard layouts from design and prototype these as React demos. They must be created in React so we can pull in charts. @mcoker will begin by creating HTML/CSS skeleton layouts that include cards with static content to use as a basic for this. We can use this to test and refine our approach.

Pulling this back to design as @kybaker does more work on refining card examples. @rachael-phillips @LHinson FYI


event_id all null in states table

Home Assistant release (hass --version): 0.63.3
Python release (python3 --version): Docker Image
Component/platform: mssql database
Description of problem: The states table has a column called event_id with a FK to event_id in the events table. Every row in states has a null event_id. Should there be data here? Is this a leftover column that can go away?
Expected: Either data in the column or for it to be removed.
Edit: Note, I am "extending" the offical docker image to add support for mssql. Maybe this isn't adding correct support?
FROM homeassistant/home-assistant:latest

RUN apt-get update && \
    apt-get install -y freetds-dev unixodbc-dev unixodbc-bin && \
    pip3 install pyodbc pymssql

I noticed that as well. Apparently it can go away since it is not used but it appears that data is intended to be there: code link.

Thanks for pointing out the code. Almost seems like a bug somewhere. Like the entity_id isn't being populated during the session.add(dbevent) call.

You are right. I made a fix now since this is preventing some logbook optimizations that I was thinking about.


Removing focused element inside dialog or dropdown makes CPU spike in chrome.

Bug report
Current Behavior
Removing focused element inside dialog or dropdown makes CPU spike. Possibly due to focus trapping logic.
Bug
Expected behavior
CPU spike shouldn't happen
Reproducible example
https://codesandbox.io/s/confident-pasteur-30gc2q
Suggested solution
Manually handling focus trap as suggested here works
https://github.com/radix-ui/primitives/issues/1498#issuecomment-1194346289
document.addEventListener('focusout', (event) => {
  if (event.relatedTarget === null) container.focus();
})

Additional context
Related issue: https://github.com/radix-ui/primitives/issues/1498#issuecomment-1194346289
This works fine on other OS.
Your environment

Software
Name(s)
Version

Radix Package(s)
Dialog
1.0.0

React
n/a
17

Browser
Chrome (MacOS)
103.0.5060.53

Assistive tech

Node
n/a

npm/yarn

Operating System
MacOS
all

Thanks @s-yadav , I haven't had a chance to look at this in depth but given that focus should be manually redirected upon removal I wonder if this is a practical problem at the moment?

@andy-hook there are couple of examples we faced in our product.

A table with virtualisation enabled having any focusable element in row, scrolling will remove the focused element
change(content1->content2) in content of Popover based on click of actionable element in popover content.

I've also encountered this in multiple situations. Mainly for (context) menu's that have a "delete" option which deletes the parent of the menu.
I was looking into it a while back but unfortunately ran out of time. In my case I think the FocusScope might be calling .focus() on an element in a ref that is unmounted, causing the CPU spike. I don't know why it happens but can reproduce the behavior: https://codesandbox.io/s/focus-cpu-syme2v?file=/src/App.tsx

@dsdeur, I don't see any CPU spike on that sandbox. How am I supposed to use it/see it? A specific browser?

Did you try chrome ?

Yeah I see it now. Given that sandbox from @dsdeur, it seems to be a bug in Chrome itself to be honest. We'll see if we can get around it.

Indeed, I have only seen it in Chrome, I've tested FF and Safari. It is quite problematic and I have not been able to get the workaround mentioned to work for my case.
I think what happens is a timing issue where the focusout is called before the listener is unregistered. Tracking unmount in FocusScope might work, an example that seems to work: https://codesandbox.io/s/focus-cpu-forked-pwizyl?file=/src/App.tsx Idk if that is also possible in FocusScope.
Alternatively what already would help a lot is to have a way to just disable this behavior. On unmount (of the parent) it's likely necessary to manually handle focus anyway.

Indeed, I have only seen it in Chrome, I've tested FF and Safari. It is quite problematic and I have not been able to get the workaround mentioned to work for my case.
I think what happens is a timing issue where the focusout is called before the listener is unregistered. Tracking unmount in FocusScope might work, an example that seems to work: https://codesandbox.io/s/focus-cpu-forked-pwizyl?file=/src/App.tsx Idk if that is also possible in FocusScope.

That would be if the parent component unmounts, not when the specific item gets removed. This will be impossible in FocusScope, we can track mount state for every child inside it.

Alternatively what already would help a lot is to have a way to just disable this behavior. On unmount (of the parent) it's likely necessary to manually handle focus anyway.

I'm not following what you're saying here.

Alternatively what already would help a lot is to have a way to just disable this behavior. On unmount (of the parent) it's likely necessary to manually handle focus anyway.

I'm not following what you're saying here.

Sorry, not very clear. I was referring to the use case where I have seen it most; when you use a (context) menu to remove an item (example.
If there is a way to disable the automatic focus handling in Radix that triggers the issue it would be relatively easy to work around it and manually handle focus in my app for these cases.

Nice, thanks! Moving the focus elsewhere in the handler works as a workaround. I had tried the preventDefault, which wasn't enough.
(If anybody is interested, here is the updated example sandbox with this workaround)

@evertjr I have just submitted a PR for it here: #2145

Not sure for the public release, I am trying to get some more maintenance done at the moment, so will be a little bit before that, but hopefully not too long. You can use the rc in the meantime.


Edit mode is enabled on "specific version" pages

Describe the bug
It is possible to enter the edit mode on the specific version pages (which is disabled on the main version).
Screenshots

Desktop (please complete the following information):

OS: Ubuntu
Browser: Chrome
Version: 109.0.5414.74

Additional context
Add any other context about the problem here.

This issue seems to be fixed on main and the last three versions.


Error connecting to Outlook since today

Since today at 6 am I started to have problems with the library that no longer generated authentication connection, which I do not know how to solve it because there are key processes in my platform that captures data.
$cm = new ClientManager([
                    'options' => [
                        'fetch' => \Webklex\PHPIMAP\IMAP::FT_PEEK,
                        'fetch_order' => 'desc',
                        'sequence' => \Webklex\PHPIMAP\IMAP::ST_UID,
                    ],
                ]);

                static::$client = $cm->make([
                    'host'          => $email->server,
                    'port'          => $email->port,
                    'protocol'      => $email->server_type,
                    'encryption'    => $email->encryption,
                    'validate_cert' => true,
                    'username'      => $email->email,
                    'password'      => $email->password,
                ]);

                try {
                    static::$client->connect();
                } catch (\Throwable $th) {
                    dd($th);
                }

I currently have the following versions:
Windows 11 (testing)
Debian 11 (production)
Laravel 7
PHP 7.4
"webklex/php-imap":"^2.4
Provider Google and Outlook

You are probably affected by this:  https://techcommunity.microsoft.com/t5/exchange-team-blog/basic-authentication-deprecation-in-exchange-online-september/ba-p/3609437
It should be possible to switch to Oauth authentication

The question is if OAuth would let me connect through the console since the only thing I do are processes by console that captures the mail and reads and publishes them on the platform.

I think it does.  I've just posted the steps I've been using.

sample code snippet to connect to a shared mailbox on office365 via oauth, hope this helps someone:
	const IMAP_HOST='outlook.office365.com';
	const<EMAIL_ADDRESS>	const IMAP_PASSWORD='?';
	const<EMAIL_ADDRESS>
	const OAUTH_URL='https://login.microsoftonline.com/organizations/oauth2/v2.0/token';
	const OAUTH_POSTFIELDS=[
		'client_id'=>'????????-????-????-????-????????????',
		'client_secret'=>'????????????????????????????????????????',
		'scope'=>'https://outlook.office365.com/IMAP.AccessAsUser.All',
		'grant_type'=>'password',
		'username'=>self::IMAP_LOGINNAME,
		'password'=>self::IMAP_PASSWORD
	];

	function connect(){
		$ch=curl_init();
		try{
			curl_setopt($ch,CURLOPT_URL, self::OAUTH_URL);
			curl_setopt($ch,CURLOPT_POSTFIELDS, http_build_query(self::OAUTH_POSTFIELDS));
			curl_setopt($ch,CURLOPT_POST, 1);
			curl_setopt($ch,CURLOPT_RETURNTRANSFER, true);
			$oResult=curl_exec($ch);
			if(empty($oResult)) throw New Exception('Missing results.');
			$oResult=json_decode($oResult);
			if(empty($oResult)) throw New Exception('Error decoding json result.');
			if(!isset($oResult->access_token)) throw New Exception('Missing access token from result.');
			$sAccessToken=$oResult->access_token;
		}finally{
			curl_close($ch);
		}

		$cm=new ClientManager([]);

		// connect to shared mailbox
		$client = $cm->make([
			'protocol'=>'imap',
			'host'=>self::IMAP_HOST,
			'post'=>993,
			'encryption'=>'ssl',
			'validate_cert'=>true,
			'username'=>IMAP_SHARED_MAILBOX,
			'password'=>$sAccessToken,
			'authentication'=>"oauth",
		]);
		$client->connect();


[WIP] Convox setup with AWS

AWS deployment with Convox
!!! This is work in progress
Info
Unfortunately, deploying with GCP is not our option.
We're trying to automate our DevOps pipeline by using convox.com services. It's not that easy, but it's easier then doing Kubernetics/manual AWS setup.
Notes on deploy process

Change DATABASE_URL value protocol part frommysql:// to mysql2://
Make sure not to load schema, but bootstrap your project with rake db:migrate command. Or you'll have a chance of rake db:seed failing ( see https://github.com/rubykube/peatio/issues/672 )
Don't provide mysql instance in resource section on convox.yml, just use convox resource create mysql and retrieve mysql:// value
convox.yml should contain explicitly defined ENV variables allowed to be set
Don't set multi-regions availability. But be sure to provide proper region in convox (Singapore in ourcase)
Don't scale down or up default number of web services (i suggest using 2, convox does 3 by default) -- this brakes things. Try auto-scale.

@skatkov, thanks for contribution!
As our platform have micro-service architecture, we should define same files for

Barong
Trading UI

Unfortunately, we are not able to support "Convox" deployment solution for our stack, as we are not qualified with this technology, that's why I am closing this pull request.
You can mount this files to our supported peatio docker image.


Paginator update

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Cluttered template, unnecessary styles.
Fixes: N/A
What is the new behavior?
Clean up template and styles. Add colorable examples to design-land.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

lgtm


Example of using ACL's in production?

Thanks for this great module.
There is only 1 example of how to use acl_token and acl_policy. More details are needed.
https://github.com/solarkennedy/puppet-consul#policytoken-system
How would you actually use those two resources? The consul guide suggests you should create a policy and a token for every node in the cluster.
Could you provide another example of how someone might setup ACL's?
Related to #472
For example:
Should an acl token and policy be generated for every host?
Should acl_api_token be the bootstrap token?

I've opened #487 with what I believe to be a minimal viable implementation of consul_policy and consul_token.


Invalid Input Error: File header contains an unknown codec

Can you share the file please?

manifest.avro.zip

Same. I'm using zstandard codec. File generated using fastavro.

It seems like unfortunately the zstd codec is not available in the official Apache C library at this point


debug.variablesDelay has no settings docs

This was only introduced so @weinand could try it out.
We will take it out this week.
@weinand is it ok to take out this setting and use 400?

@isidorn the 400 ms worked fine for me. Let's use them for May.


Leading and trailing blank lines consumed

Any selected leading and trailing blank lines are consumed by TabSpacerSelect.
I don't know if this is intentional, but it is certainly unappealing for my purposes.
Otherwise, this extension is proving very useful, so thanks very much for the work that went into it!

Thank you for this report as well. This is fixed in the latest release.


[CI_Email] for smtp protocol in EHLO command hostname is missing

When you use
$config['protocol'] = 'smtp';
in order to connect to the server is necessary to specify a hostname for the commands EHLO/HELO, this command is sent it by _send_command function, lines 2065 and 2069 of Email.php CI_Email class
$this->_send_data('EHLO '.$this->_get_hostname());
here, the hostname is provided by the function _get_hostname()
protected function _get_hostname()
        {
                if (isset($_SERVER['SERVER_NAME']))
                {
                        return $_SERVER['SERVER_NAME'];
                }

                return isset($_SERVER['SERVER_ADDR']) ? '['.$_SERVER['SERVER_ADDR'].']' : '[<IP_ADDRESS>]';
        }

as you can see, if SERVER_NAME is set, the function returns that value, otherwise tries with SERVER_ADDR and finally if it's not set then fallback to [<IP_ADDRESS>].
In my case, I'm running NGINX and I've not set the fastcgi_param SERVER_NAME/ADDR, then the function returns an empty value for hostname, so the SMTP server response is

501 Syntax: EHLO hostname

in order to fix that I used !empty()  instead of isset() to look for a valid value of SERVER_NAME/ADDR
Here is the pull request #4728

You're not supposed to have an empty server name or address while running an HTTP server - that's not a valid use case.

If you don't have a value in SERVER_NAME/ADDR is a perfect valid use case for an HTTP Server, and your application must be secure on any layer, so your function must be secure also, in this case, the function _get_hostname() uses an external value and also has a default value <IP_ADDRESS>, so, you shouldn't return an invalid value for a hostname in any case, especially when your function provides a required value  for an external command as SMTP EHLO/HELO.
The description of the function says

Get Hostname
There are only two legal types of hostname - either a fully
qualified domain name (eg: "mail.example.com") or an IP literal
(eg: "[<IP_ADDRESS>]").

but for the case that you aren't set a value for SERVER_NAME/ADDR (which is a valid use case for HTTP Server) your functions returns an invalid value empty
If you code a unit test for that case, it will fail..

If you don't have a value in SERVER_NAME/ADDR is a perfect valid use case for an HTTP Server

No, it's not. You can't reach a nameless, addressless server - there would be no way of pointing your browser to it.
You could have no SERVER_NAME, in which case it should be assumed to equal the SERVER_ADDR (Apache does this by default), but it can't exist and be empty.
The fact that some kind of software out there allows you to do that doesn't make it a "perfectly valid" use case. It's a dirty hack at best, and in fact, since you're talking about stuff being "secure" - there are serious security implications to leaving the SERVER_NAME empty.
And you're going even further to state that you can have an empty SERVER_ADDR as well - this is completely absurd.

It's a valid case because if you are using fastcgi, those variables are not required at all. You must send it explicity (and manually), I'm talking of a configuration nginx fpmphp (fastcgi), not Apache that assumes those values.
When I said "secure" I referred to the expected value that a function must return for a given input. That function has not a valid behaviour since hostname can't be an empty value, and, also contradicts his own description....

It's a valid case because if you are using fastcgi, those variables are not required at all.

Again, that some software doesn't require you to explicitly configure them, doesn't mean anything. SERVER_NAME is an essential part of the CGI protocol - you don't get to make arbitrary conclusions about it based on what you are forced to configure or not.
The very fact that nginx actually creates the variable even if you don't provide it should speak volumes.

You must send it explicity (and manually), I'm talking of a configuration nginx fpmphp (fastcgi), not Apache that assumes those values.

Yes, everything in nginx is explicit. That doesn't mean anything.
And I have not said that you are talking about Apache. I gave an example of software that obviously doesn't give you the option to do what you're claiming to be a "perfectly valid" use case.

When I said "secure" I referred to the expected value that a function must return for a given input.

That is not what "secure" means in the context of programming. Terminology is important.

That function has not a valid behaviour since hostname can't be an empty value, and, also contradicts his own description....

Please don't make side comments based on the assumption that you're right and I'm wrong. You have to prove that you are right first.
Also, "his" is a gendered pronoun. Functions don't have genders.

The principal reason is that in a given environment, with given requirements you can have not setted a server name or addr, get_hostname takes care of that situation with a default value, assuming that the external value is correctly setted, in my opinion, not is a stable behaviour (or secure, or consistent, or reliable). I mean, as a client I can't be sure that a call to ->get_hostname(); will return me a valid hostname, even when its description says explicitly that there are only two valid values for a hostname. (btw when I said "client" I'm talking about OOP, not talking about business..)

Please don't make side comments based on the assumption that you're right and I'm wrong. You have to prove that you are right first.

I give you a scenario, some conditions and some preconditions, where an unstable behaviour is present, the only thing that I see is that you are defending your position saying that an external configuration is wrong, you're blaming the environment configuration, instead doing the correct treatment if a value is not valid

Also, "his" is a gendered pronoun. Functions don't have genders.

Sorry for my English, I meant "the function contradicts its own description"....
At read this comment, seems you're not being collaborative and only you want to argue with me, that is not the spirit of the open source, I mean, I don't want to collaborate with you anymore.
I'm sorry but, I don't want discuss with you, the reasons are there.
Regards.

The principal reason is that in a given environment, with given requirements you can have not setted a server name or addr, get_hostname takes care of that situation with a default value

That environment is CLI, where literally neither exists and not set but empty. You're again making assumptions.

in my opinion, not is a stable behaviour (or secure, or consistent, or reliable).

"stable" is a relative term and meaningless in this context.
"secure" means, again, something completely different and unrelated to what we're discussing.
"consistent" is, in my observation, something people use as an argument when they have no arguments.
"reliable" is relative as well.

I mean, as a client I can't be sure that a call to ->get_hostname(); will return me a valid hostname, even when its description says explicitly that there are only two valid values for a hostname.

Fine, I will concede that in a broken environment, you can't be sure about that.
What you have is what I consider a broken environment and we don't support such cases.

(btw when I said "client" I'm talking about OOP, not talking about business..)

When we're talking about SERVER_NAME and SERVER_ADDR, we're talking about HTTP - I'm far from thinking that you'd mean business.
Though, it's good that you're clarifying that you meant something else.

I give you a scenario, some conditions and some preconditions, where an unstable behaviour is present, the only thing that I see is that you are defending your position saying that an external configuration is wrong, you're blaming the environment configuration, instead doing the correct treatment if a value is not valid

Yes, I am blaming the environment, and you seem to be defending the position that the environment is never at fault.
Sorry, I will never agree with you on this.

At read this comment, seems you're not being collaborative and only you want to argue with me, that is not the spirit of the open source, I mean, I don't want to collaborate with you anymore.

You know you're not being collaborative either - you're not trying to prove your claims, but just using different words to directly contradict mine.
Yes, I will argue about every incorrect thing. Are you claiming that accepting incorrectness is in the spirit of open-source? Furthermore, are you claiming that another person trying to correct a mistake of yours is not in the spirit of open-source? :)
This is a petty, off-topic argument, indeed. But you're taking it the wrong way.

I'm sorry but, I don't want discuss with you, the reasons are there.

That's ok. From my POV, this was never a discussion in the first place.