Docker file not working

Hi buddy,
im looking at your code and im trying to running it..
im getting the following:
could not find /Users/xx/templates/service/host/IdentityServerHost: stat /Users/xx/templates/service/host/IdentityServerHost: no such file or directory
can you please assist

Hi mninawa,
This is a feature module created based on the ABP framework. If you want to run it in a Docker environment, you'll need to modify the docker-compose.yml file.
https://github.com/zhaofenglee/RulesEngine/commit/ae2ad34113a0327171f703e8b25b2c0cc9fcc697
If you wish to run a sample project, you can execute the host\JS.Abp.RulesEngine.Blazor.Server.Host project. Before using it, you'll need to perform a migration.


[flash] Design updates and clean-up

Clean-up and fix the usage of sizing parameters.  Previous
design represented flash sizes in bus words, leading to an
incorrect alignment of page boundaries.  This PR changes
the parameters to flash word size base and explicitly declares
the bus widths separately.

Also add an arbiter count between host and controller arbitration.
Since host reads are critical to performance, the past design
always favored hosts, with the understanding under the right access
patterns this could lead to deadlock.  Recent system changes indeed
led to such an arbitration starvation, so a counter is added to
ensure controller will not be starved.

Round robin is still not preferred because controller does not need
access priority as long as it can "eventually" finish.

Signed-off-by: Timothy Chen<EMAIL_ADDRESS>

I really should break out the arbitration count portion into a separate PR, but do take a look if can.
The idea is pretty simple.
For others, the main change is that sizing parameters are now correctly specified, whereas before I tried to be too cute and specify it in terms of bus sizes, which made things very confusing the deeper it got.

LGTM, although I wonder whether it would really be that bad if access where just RR arbitrated. i.e., do you expect that these two paths are competing for access often?

personally i think so. Mostly because if the code becomes scrambled, the odyssey of completing a flash read is probably 4-5+ cycles. The read and de-scramble eventually will be pipelined, which to me means if it's fully round robin we won't be able to take advantage of the fact that something is de-scrambling to hide the latency of a new read.
To your point about how often they would compete, it should not be often. Because most of the time I expect software to access data on another bank...but you never know.  I still may come back and make this RR.   The cost of the counter is pretty low right now, so I'm inclined to give it a try.

I don't think the sw changes will be an issue - given the FLASH_WORD_SZ reflects new 64-bit word flash, and the other parameter change is in this PR.
Given how fundamental running from flash is to our testing setup, CI should show us if there are issues here, so I'm happy from the software side if CI is passing.

Thanks @lenary ! longer term though, I think all of us need to have a conversation about HOW software would like to make use of the flash controller.  There are lingering questions around memory protection, data alignment and informational access that's not finalized yet (and might not be for some time).

I don't think the sw changes will be an issue - given the FLASH_WORD_SZ reflects new 64-bit word flash, and the other parameter change is in this PR.
Given how fundamental running from flash is to our testing setup, CI should show us if there are issues here, so I'm happy from the software side if CI is passing.

Thanks @lenary ! longer term though, I think all of us need to have a conversation about HOW software would like to make use of the flash controller. There are lingering questions around memory protection, data alignment and informational access that's not finalized yet (and might not be for some time).

Yeah, these do sound like important issues. This sounds like something that should go through the software review queue, rather than anything else.

yeah i think it's on there...somewhere.
On Tue, Jun 23, 2020 at 12:05 PM Sam Elliott<EMAIL_ADDRESS>wrote:

I don't think the sw changes will be an issue - given the FLASH_WORD_SZ
reflects new 64-bit word flash, and the other parameter change is in this
PR.
Given how fundamental running from flash is to our testing setup, CI
should show us if there are issues here, so I'm happy from the software
side if CI is passing.
Thanks @lenary https://github.com/lenary ! longer term though, I think
all of us need to have a conversation about HOW software would like to make
use of the flash controller. There are lingering questions around memory
protection, data alignment and informational access that's not finalized
yet (and might not be for some time).
Yeah, these do sound like important issues. This sounds like something
that should go through the software review queue, rather than anything else.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2588#issuecomment-648358438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSTZDVW6GAFCWHBGKD3RYD4HDANCNFSM4OFL2ACA
.

not all of these points, so i do need to make that clearer.
On Tue, Jun 23, 2020 at 12:17 PM Timothy Chen<EMAIL_ADDRESS>wrote:

yeah i think it's on there...somewhere.
On Tue, Jun 23, 2020 at 12:05 PM Sam Elliott<EMAIL_ADDRESS>wrote:

I don't think the sw changes will be an issue - given the FLASH_WORD_SZ
reflects new 64-bit word flash, and the other parameter change is in this
PR.
Given how fundamental running from flash is to our testing setup, CI
should show us if there are issues here, so I'm happy from the software
side if CI is passing.
Thanks @lenary https://github.com/lenary ! longer term though, I think
all of us need to have a conversation about HOW software would like to make
use of the flash controller. There are lingering questions around memory
protection, data alignment and informational access that's not finalized
yet (and might not be for some time).
Yeah, these do sound like important issues. This sounds like something
that should go through the software review queue, rather than anything else.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/2588#issuecomment-648358438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSTZDVW6GAFCWHBGKD3RYD4HDANCNFSM4OFL2ACA
.


failed to load bam file

Hello!
DeepMAsED fails to load the .bam file, even if it is present. This is my bam_fasta table:
bam	fasta
nosingletons_flye.singletons.bam	01.nosingletons_flye.fasta

When running DeepMAsED, I get:
2020-07-08 14:55:08,556 - Indexing file: 01.nosingletons_flye.fasta
2020-07-08 14:55:14,015 - Cannot find nosingletons_flye.singletons.bam.bai; creating...

[E::hts_idx_push] NO_COOR reads not in a single block at the end 915 -1
samtools index: failed to create index for "nosingletons_flye.singletons.bam": No such file or directory

So it is able to find and index the fasta file, but not the sam file. What am I doing wrong?
Best,
Javier

Are you able to create an index on the bam file with samtools on your own? It seems like something is wrong with the bam file.

Hello @nick-youngblut, thanks for your quick response.
Yes, I was able to index the file. Still I got the same error.
This is my directory:
-rw-r--r-- 1 tamames javier<PHONE_NUMBER> jul  8 14:26 01.nosingletons_flye.fasta
-rw-r--r-- 1 tamames javier<PHONE_NUMBER> jul  8 14:28 nosingletons_flye.singletons.sam
-rw-r--r-- 1 tamames javier<PHONE_NUMBER> jul  8 14:37 nosingletons_flye.singletons.bam
-rw-r--r-- 1 tamames javier         70 jul  8 14:55 bam_fasta.table
-rw-r--r-- 1 tamames javier  407229940 jul  8 16:26 samtools.6153.7024.tmp.0000.bam
-rw-r--r-- 1 tamames javier<PHONE_NUMBER> jul  8 16:32 nosingletons_flye.singletons.sorted.bam
-rw-r--r-- 1 tamames javier     403000 jul  8 16:39 nosingletons_flye.singletons.sorted.bam.bai
-rw-r--r-- 1 tamames javier   34227339 jul  8 16:39 01.nosingletons_flye.fasta.fai

Best,
Javier

Can you please post the entire error message and the command used to generate it?

Here it is:
(deepmased) tamames@rigaud:~/test$ DeepMAsED features bam_fasta.table
2020-07-08 16:39:40,408 - Indexing file: 01.nosingletons_flye.fasta
2020-07-08 16:39:45,759 - Cannot find nosingletons_flye.singletons.bam.bai; creating...

[E::hts_idx_push] NO_COOR reads not in a single block at the end 915 -1
samtools index: failed to create index for "nosingletons_flye.singletons.bam": No such file or directory

Traceback (most recent call last):
  File "/home/tamames/soft/anaconda2/envs/deepmased/bin/DeepMAsED", line 10, in <module>
    sys.exit(main())
  File "/home/tamames/soft/anaconda2/envs/deepmased/lib/python3.6/site-packages/DeepMAsED/__main__.py", line 57, in main
    args.func(args)
  File "/home/tamames/soft/anaconda2/envs/deepmased/lib/python3.6/site-packages/DeepMAsED/Commands/Features.py", line 69, in main
    Features.main(args)
  File "/home/tamames/soft/anaconda2/envs/deepmased/lib/python3.6/site-packages/DeepMAsED/Features.py", line 240, in main
    index_bam(bam_file, threads=args.procs)
  File "/home/tamames/soft/anaconda2/envs/deepmased/lib/python3.6/site-packages/DeepMAsED/Features.py", line 195, in index_bam
    raise IOError('samtools return code: {}'.format(rc))
OSError: samtools return code: 1

But, I cannot index the unsorted bam file with samtools:
(deepmased) tamames@rigaud:~/test$ samtools index nosingletons_flye.singletons.bam
[E::hts_idx_push] NO_COOR reads not in a single block at the end 915 -1
samtools index: failed to create index for "nosingletons_flye.singletons.bam": No such file or directory

And now, providing the sorted.bam file in the bam_fasta table, it is working

Thanks for the details! I thought that you were able to index the bam file:

Yes, I was able to index the file. Still I got the same error.

Yeah, it seems that something is wrong with your bam file. It might be helpful to convert your bam file to a sam file and inspect it.

Well, it is working when I sorted it first. I guess I needed to do that, right?

Yes you do. I should make that clear in the docs

Thank you @nick-youngblut


For Missions: Allow Use of Clear Tickets

For any mission that allows clear tickets to be used, allow option for clear tickets to be used globally or by mission.
This will be useful for things like Dimension Mission or other Quests that take a long time.

Clear Tickets can now be used mainly for Epic Quests - but it's actually a bit slower than playing through them.    Will continue working on this - but will push up the latest change here.     Check the box on the main page to use clear tickets.


New Files

Right now, if templates are added to the directory during runtime, they are not automatically watched - only files that exist on startup are watched.
This came to my attention when looking at the code, but I didn't explicitly test this, so apologies if I missed something!

Sorry - didn't look clearly enough, my bad

no problem 😄


Add basic support of IETF QUIC

This PR is used to run CI jobs against quic-latest branch, and the branch is used for catching up new drafts. Also, this enables you to point out any issues on the latest code on this PR before we make another PR for supporting a new draft on master (since draft-20 support was merged the diff is smaller than before).
~This PR will be merged to master when we agree on adding experimental support for QUIC to a future release. In the meanwhile, we are going to maintain quic-latest branch to implement QUIC collaboratively.~
Please feel free to comment on this PR, and improve it with your PRs against quic-latest branch.
The whole activity will be tracked on this project below.
https://github.com/apache/trafficserver/projects/8
Some documentations are available on our wiki.
https://cwiki.apache.org/confluence/display/TS/QUIC
Committers, please do not merge / close this PR for a while.

For building, can we make it such that if there is no TLS v1.3 support, we don't build the pieces that needs it? This can be automatically detected, such that you get QUIC support IF your OpenSSL / BoringSSL supports it. You would still need to configure ATS to listen on the UDP port (443), but that doesn't affect compile time.

I assume we're continuing development on the quic-latest branch, and will merge that branch to master when it's ready for it?

@zwoop

For building, can we make it such that if there is no TLS v1.3 support, we don't build the pieces that needs it? This can be automatically detected, such that you get QUIC support IF your OpenSSL / BoringSSL supports it. You would still need to configure ATS to listen on the UDP port (443), but that doesn't affect compile time.

That is exactly what in my mind. It would be done on #2344.

I assume we're continuing development on the quic-latest branch, and will merge that branch to master when it's ready for it?

Yes, we are going to do so.

Current status:

Works on Fedora 26, FreeBSD 11 and maxOS 10.12 (tested with ngtcp2)
All items for the first impl. have been implemented

[approve ci]

It's broken right now. Trying to moving to draft-07 with pretty big changes but patches are always welcome.

Tested handshake with ngtcp2 and minq (draft-07).

[approve ci debian]

Locks #5457

I converted this PR to draft so we don't merge this accidentally. This PR is mainly for CI.

All the changes are merged into master.


[Proto] Implement a State result type.

Closes #376

@jpmoutinho @dominikandreasseitz I know lint/test fail but works for PyQ. Just for you to tell me what you think about it.

@jpmoutinho @dominikandreasseitz I know lint/test fail but works for PyQ. Just for you to tell me what you think about it.

Closing as moved elsewhere.


Move FileCompressionType out of common and into core

Is your feature request related to a problem or challenge?
FileCompressionType appears to be the only item in common that uses several heavy dependencies in the common crate such as tokio. As result causes it to be a dependency of all crates using common, such as expr. Other than for FileCompressionType there is no usage of async in any crate except core (tests excluded)
By moving FileCompressionType, or the associated convert_X functions to core, we'd remove several heavy dependencies from all crates except core
Describe the solution you'd like
A quick look over the codebase, it seems that FileCompressionType is only used in core. So it seems natural that it could easily be moved in to core.
Describe alternatives you've considered

Make all async dependencies optional & control them via feature flags.
Keep FileCompressionType in common, but move the convert_X functions in to core.

Additional context
No response

I agree -- thank you @universalmind303  -- marking this as a good first issue as it is primarily a code motion exercise and does not require deep datafusion internals knowledge

I want to work on this issue.


Can you share with us the .obj models you used in the scene?

Can you share with us the .obj models you used in the scene?
I downloaded the bunny.obj in standford repo and used in the scense examples but it gives me errors like
Normals and texture coordinates are required!

Yea, I've been meaning to put together an assets bundle with the models since some have slight modifications (generated normals, uv coordinates). I'll look into this tomorrow, all I really need to add are some attribution links for where they're from. I'll close this when I post a link to the assets

I've put together a models bundle which you can get here, there's also a link to this zip in the readme now.

@Twinklebear, thank you for sharing the models and this awesome project as well.
One more thing, can you share the brdfs as well. I got these errors when trying out with the more complex scenes
thread '<main>' panicked at 'material::Merl::load_file - failed to open "./scenes/brdfs/black-oxidized-steel.binary" due to No such file or directory (os error 2)', src/material/merl.rs:46

Since the BRDFs don't need any modification it's easier to just link to the MERL BRDF Database where they can be downloaded, a link is also in the model bundle's readme. The names of the files are the same as those in the database, so they can all be found on the author's site here (linked from the database).


prepare for the new mainthread package

Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
Go 1.24 will introduce mainthread package https://github.com/golang/go/issues/64777 with which we can call functions from the main thread anywher! 🥳
We have to prepare Ebitengine code so that Ebitengine will accept both the LockOSThread way and the mainthread way.
For example, without runtime.LockOSThread, the init functions might not be called from the main thread as of Go 1.24. Thus, if there is an assumption that a function in init is called from the main thread, we have to create some wrappers to accept both way.
//go:build !go1.24

func init() {
    runtime.LockOSThread()
}

func RunOnMainThreadFromInit(f func()) {
    f()
}

//go:build go1.24

func RunOnMainThreadFromInit(f func()) {
    mainthread.Do(f)
}

func init() {
    RunOnMainThreadFromInit(func() {
       FunctionThatMustBeCalledFromMainThread()
    })
}

Why is this needed?
No response

https://github.com/golang/go/issues/64777#issuecomment-2316852610

I was misunderstanding. init is called from the main thread whichever LockOSThread is called or not.


Added PL release notes for FG and NG

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-796

Preview environment: https://ci.pr.harness.io/pr-796

Preview environment: https://ci.pr.harness.io/pr-796

Preview environment: https://ci.pr.harness.io/pr-796


fix: provision new ES cluster with master nodes

Add non-optional three master nodes to ES cluster. ES domain name and
security group name were changed due to provisioning of new cluster.
Changing the domain name back requires another migration while
destroying the SG caused errors.

There is no need to change the resource name from es to es_new at all. Remember this is for public use and not to be contextualized to any project.
If you are creating another Elasticsearch service, all you need to use a new module to derive from this module, i.e. you can have elasticsearch and elasticsearch_new using the same base module here with the same resource names, and they can coexist with no issues as long as the relevant AWS service names (not Terraform resource names) do not clash.

So I suggest this to remain the same as before, since the change doesn't improve the code in general?

So I suggest this to remain the same as before, since the change doesn't improve the code in general?
I see. What about the changes wrt to master nodes? I think that's something @sulaiman-alwie wants to reduce the load on data nodes in the cluster.

So I suggest this to remain the same as before, since the change doesn't improve the code in general?

I see. What about the changes wrt master nodes? I think that's something @sulaiman-alwie wants to reduce the load on data nodes in the cluster.

The original code did a bit of logic check to determine the master count, which I agree is opinionated and harder to override.
I suggest best is to make it completely generalized, which this PR can then do:
dedicated_master_enabled = ${var.es_dedicated_master_enabled}
dedicated_master_count   = ${var.es_instance_count}
dedicated_master_type    = ${var.es_master_type}

Since there are completely no defaults for the above vars based on what I see right now, they can stay that way. This allows for deriving module to easily put whatever values the module prefers.

Would it be better to have a separate var to indicate

master node count (default: 3, anything > 3 is overkill)

instead of es_instance_count? This would allow separate definition for number of master and data nodes.

Would it be better to have a separate var to indicate
* master node count (default: 3, anything > 3 is overkill)

instead of es_instance_count? This would allow separate definition for number of master and data nodes.

Okay yeah we should have a es_dedicated_master_count for it. Sorry missed the fact that es_instance_count was shared across two different params.
In concept, just make all the cluster_config assignment vars use a different var to make overriding easy.

LGTM, but before that, can trouble you to help type annotate all the vars in modules/elasticsearch/variables.tf? Doing so will make the INOUT.md more friendly with the type descriptions.

LGTM, but before that, can trouble you to help type annotate all the vars in modules/elasticsearch/variables.tf? Doing so will make the INOUT.md more friendly with the type descriptions.

Done


error NU1101: Unable to find package System.Collections.Generic. No packages exist with this id in source(s): nuget.org

Describe the bug
Recently installed  dotnet-interactive and tried to install package 'System.Collections.Generic.' in jupyter notebook. It's giving error NU1101. Same cell execution for 'System' works.
Please complete the following:
Which version of .NET Interactive are you using? There are a few ways to find this out:

In a notebook, run the #!about magic command.
At the command line, run dotnet interactive --version.
In VS Code, run "Report installed version for .NET Interactive` and copy the version number from the status popup.

.NET Interactive
© 2020 Microsoft Corporation
Version: 1.0.115407+d247e88f3f756abad4bd16b47cf8cde187aafabc
Build date: 2020-07-14T22:52:12.8236468Z
https://github.com/dotnet/interactive

OS

[x] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[x] Chrome
[x] Edge
[ ] Firefox
[ ] Safari

Frontend

[x] Jupyter Notebook
[x] Jupyter Lab
[x] nteract
[ ] Visual Studio Code
[ ] Other (please specify)

Screenshots
If applicable, add screenshots to help explain your problem.

The System and System.Collections.Generic namespaces are already available and don't need to be referenced using #r.


how to return list of model relations?

hi,
how does one access a list of model relations?
given the example
var Tag = Bookshelf.Model.extend({
  tableName: 'tags',
});

var Author = Bookshelf.Model.extend({
  tableName: 'authors',
});

var Comment  = Bookshelf.Model.extend({
  tableName: 'comments'
  author: function () {
    return this.hasOne(Author);
  },
  tags: function () {
    return this.hasMany(Tag);
  },
});

var Story = Bookshelf.Model.extend({
  author: function() {
    return this.hasOne(Author);
  },
  comments: function() {
    return this.hasMany(Comment);
  },
});

how does one access a list of Story relations: ['author', 'comments']? preferably it would also be possible to get deep relations: ['author', 'comments.author', 'comments.tags'].
i want to make a featherjs service for bookshelf (like feathers-mongoose-service) that needs to be aware of relations in order to set up appropriate routes (e.g. GET /story/:id/comments).
thanks!

We actually figured out an even easier way to handle it. We put all our relationship functions in an object relationships and the move them out on extend. This way you know which methods are relationships for related.
https://github.com/pk4media/bookshelf-relationships

Good approach, @jtwebman. Currently planning a refactor/update of how Bookshelf relations work using a similar strategy (among other changes).

Thanks looking forward to the changes, do you have a branch that I can take a look at?
We are also building an Express JSONAPI bookshelf tool to make it easy to make JSON API's. This was a small part of it to make it easier.

I'm just working on a new spec currently - a significant API change. It's not finished, but you can see my WIP here. The better documented ideas are in README.md, but there are more features (and draft implementations) in notes.md. I'm in the process of refining the ideas in notes and moving them to README. Feedback would be great.

The project leadership of Bookshelf recently changed. In an effort to advance the project we close all issues older than one year.
If you think this issue needs to be re-evaluated please post a comment on why this is still important and we will re-open it.
We also started an open discussion about the future of Bookshelf.js here https://github.com/bookshelf/bookshelf/issues/1600. Feel free to drop by and give us your opinion.
Let's make Bookshelf great again


feat: cache manifest for one day

I think that's long enough to keep the number of requests acceptably low
If we forget to change this before updating the manifest, it's also not
too long a period to have, say, broken icons.
Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

The client is in charge of keeping track of the resources age?

Nope, that's the responsibility of the browser and any caches between it and the origin.
For simplicity, imagine the only caches are the CDN (Cloudflare) and the browser. Now imagine someone visits the page for the first time (in a given browser). The browser won't have anything in its cache and so will make a request. The CDN may have the resource cached. If it does, it'll check the Cache-Control header and determine if the resource is fresh or not. If it is, then it sends it back to the browser. If not, it queries the origin (i.e the step we're trying to minimise).
Now, if the same user reloads their page. Their browser will look at the header, check the max-age, see that this is fresh and in its cache. It will simply return what it has in the cache.
At this point another user may visit the page for the first time and the CDN will definitely still have the resource and so doesn't bother the origin.
TL;DR: browser + CDN check max-age and this should result in only one request reaching the origin per day.


Add CSSO and Terser for CSS and JS minification

Add CSS minification via CSSO
Add JS minification via Terser

Released in v2.4.0


Connect Superset to ClickHouse DBMS

Hello.
Superset is a very good dashboard. I liked him at first sight.
In my projects are increasingly used ClickHouse, because it is very fast DBMS for OLAP. See performance comparison at https://clickhouse.yandex/benchmark.html#[100000000,["ClickHouse","Hive"],["0","1"]]
May be possible create connector to ClickHouse, e.g. through [JDBC] (https://github.com/yandex/clickhouse-jdbc) (do similarly to Hive)?
All the best, thank you!

I think ClickHouse is great alternative for Druid. According to documentation there are 2 ways how to make Superset work with ClickHouse:

Native support (similar to Druid)
Using SQLAlchemy
Unfortunately there is no external SQLAlchemy dialect for ClickHouse (https://github.com/Infinidat/infi.clickhouse_orm/issues/8). And I think native support would be better. Unfortunately there documentation lack description how to do that.

@sichkarev Of course it is possible if someone do the work :)
@tomsej If you want to tackle it feel free to ask any question you have, i don't think much code is needed to add a new data sources backend.

@xrmx Thank you so much! Where should I start - Models.py?

@tomsej Well it depends.  If it's not try to come up with something like DruidDatasource. I don't know if it has the same metric, cluster and column entities as Druid though.

It would be quite exciting to have the clickhouse onboard !
From my very little knowledge about clickhouse, I've heard it somewhat is similar to presto.
In that case: https://github.com/dropbox/PyHive could be helphul as an example of the sqlalchemy integration.

We've (Cloudflare) open-sourced a basic sqlalchemy dialect for ClickHouse https://github.com/cloudflare/sqlalchemy-clickhouse
I'll start a PR for ClickHouse backend integration for Superset, it works but it's still a little bit buggy.

PR #1844 was merged, so this issue can probably be closed...

Indeed, closing then.


BrowserOpenerRenderer does not read the url / uri fragment from cell correctly

https://vaadin.com/forum/#!/thread/16959344
"The URL fragment should be #1, #2 or #3, depending of the row that is clicked, but the fragment is always #3 !"

Fixed in version 2.2.10 and 1.2.4


Adding Open Vino Model Server

Adding Open Vino Model Server information.

@dleybz @yonbek @skogstrom @alongubkin Please review the commit and push to the repository.

@SnirShechter @alexiguazio @ncilfone @omesser @jsnctl Hi guys can you review and push the commit ?

@rasapala
Hey,
Sorry, I totally missed that PR. I will review it on Sunday.
Thanks!

I forgot about that too, I'll update it according to what was asked and ping
On Thu, Sep 23, 2021, 6:41 PM Snir Shechter @.***>
wrote:

@rasapala https://github.com/rasapala
Hey,
Sorry, I totally missed that PR. I will review it on Sunday.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aporia-ai/mlops.toys/pull/70#issuecomment-925931468,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEKW5KSTESEXOVYIM5PMMCLUDNDBJANCNFSM5D5G52OA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


Enable librealsense on unrooted android devices

This PR is based on the work done by @dperox and @aangerma to enable access to RealSense device via librealsense on unrooted android device.
The PR includes the following:

An Android studio project which generates a native librealsense so and/or AAR.
Java example that captures and display the color and depth streams.
Java/NDK Example that demonstrates how to access librealsense native API via JNI.

Currently the java API includes the minimal required functionality to allow basic streaming.
The Android project is located at /wrappers/android

@matkatz
The current version has a very bad delay(~10s!) when the device wants to load and show the RealSense data.
Where is this delay coming from?

@ssakhavi, sorry for the late response (twice)

Currently there is no support for additional filter operations (i.e. Alignment) Maybe something similar to the options can be written for the wrapper?
An option to enter Advanced mode and modify would be nice ( I'm not sure the current options >
covers all the parameters)

We Plan to add support in all of librealsense API in the future, currently if you are missing functionality you can work with the native API.

Saving and loading Frames is a feature that is used in many cases to communicate and do calculations on server and Android.

Playback and record are now supported, please check the examples for more details.

The current version has a very bad delay(~10s!) when the device wants to load and show the RealSense data. Where is this delay coming from?

Can you explain what do you mean by delay? Is it the time to the first frame after start or something else? Also what Android device are you using for testing?

It's okay @Matkatz
Regarding the Advanced mode and Alignment, we managed to implement it the
same way that you did.
We also implemented "background remover" as a processing block to make it
faster.
The problem we have now is that is this:
When wanting to set the parameters for Advanced mode, we need a device
handle. Getting the handle adds around 3 extra seconds in addition to
starting the pipeline. So our delay is around 8-10 seconds overall.
Our code is here if you want to look at it :
https://github.com/KroniKare/RealSense-NewAndroid/tree/KroniKare-Update/src/android/jni
On Tue, 19 Feb 2019, 22:20 MatKatz<EMAIL_ADDRESS>wrote:

@ssakhavi https://github.com/ssakhavi, sorry for the late response
(twice)

Currently there is no support for additional filter operations (i.e.
Alignment) Maybe something similar to the options can be written for the
wrapper?
An option to enter Advanced mode and modify would be nice ( I'm not
sure the current options >
covers all the parameters)

We Plan to add support in all of librealsense API in the future,
currently if you are missing functionality you can work with the native API
https://github.com/matkatz/librealsense/tree/unrooted_android_device_support/wrappers/android/examples/native_example.

Saving and loading Frames is a feature that is used in many cases to
communicate and do calculations on server and Android.

Playback and record are now supported, please check the examples
https://github.com/matkatz/librealsense/tree/unrooted_android_device_support/wrappers/android/examples
for more details.
The current version has a very bad delay(~10s!) when the device wants to
load and show the RealSense data. Where is this delay coming from?
Can you explain what do you mean by delay? Is it the time to the first
frame after start or something else? Also what Android device are you using
for testing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/pull/3083#issuecomment-465146169,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFwDe-kBF1O6mPBFpH6ucPM9dAaefn9Cks5vPAg7gaJpZM4aEmW_
.

Btw, @MatKatz, your code is awesome. I enjoyed my time playing around with it.


CI: Add in deploy tooling

TODO: update reports to perform conditional upload

Using a Makefile as it does recipes based on globs better than other tooling (we need to vary based on ipynb vs py)
rules for happy-path laid out in CONTRIBUTING.md

Lost focus a little yesterday.
Fly deploys are now in a python script; we can move to CloudApps when ready to streamline the process.
Some apps have been excluded from deploying:

background-remover does too much processing on boot, so never comes alive in time
some apps depend on dp-components. They'll go live later

Ok sounds great,

background-remover - I can perhaps rewrite this later to do that processing on first request or in the background later on next week

Merging this as-is.
As with the other PR, the secrets aren't configured yet, so it won't do anything

@mands can the dev group get rights to merge please


UBL EN16931 Invoice

https://docs.oasis-open.org/ubl/os-UBL-2.1/xsd/maindoc/UBL-Invoice-2.1.xsd
Have run test and check it pass it or not and give it true when you are passing by test cases.
https://peppol.helger.com/public/locale-en_US/menuitem-en16931

You know that we are working with EN16931 standard. How it is close we can check by
http://peppol.helger.com/wsdvs?wsdl=1
Use web services to verify the result EN16931 document

It is working !!!
https://github.com/pondersource/invoice-ubl
https://github.com/ishifoev/invoice-ubl


Ghost 5.0: Breaking Changes

The Ghost team has started working on preparations for 5.0, which will involve several breaking changes. There is no ETA or release date as yet, so please do not ask 😬
This issue serves as a living doc with all the known breaking changes as we decide on them. It is unlikely to be exhaustive until 5.0 actually drops.

Known Breaking Changes So Far
Database support changes:

MySQL5 is no longer supported in any environment
SQLite3 is no longer supported in production
MySQL8 is supported in all environments

API changes:

Ghost no longer has multiple API versions
API versions are no longer included in the URL
An accept-version header is supported instead (optional, preferred, docs TBC)

Theme changes:
The easiest way to check your theme for breaking changes is to run it against 5.x in gscan. Many old deprecated features have been removed:

@blog
lang
single authors
many more!

Why is the sqlite3 support being removed?

@JoshStrobl it is not being removed. SQLite3 will not be supported in production.

Are there any plans for better member/user support? Would love to be able to add custom attributes and data for members (bookmarks, bio, etc) without having to edit the core. And perhaps some sort of authentication or oAuth integration for using ghost accounts on another platform (eg. Mobile app for blog or custom site using content API)

IMO MariaDB support should be included, it's a popular database in use.

Dropping MariaDB as main MySQL flavor on major Linux distributions is really bad idea, even worse than the idea to drop PostgreSQL few years ago. Your product should not depend on particular database as there is no good reason for that. I didn't analyze your code but if you use knex, why would you drop any database which they support? What's so specific about Ghost? Some super complex queries to fetch posts? I would not say so.
Supporting wider range of databases would bring you more users of Ghost, more users of non-commercial version though. That's maybe not important for you so our solution to this may be to switch out of Ghost CMS.

Do you really want to drop MariaDB just because it doesn't allow datetime columt to default to "NULL"? This is wrong, "NULL" is string and not of datetime type. You should go for type strict code and switch to TypeScript and not drop MariaDB. You should also introduce back PostgreSQL support, the reasons for dropping it were so stupid, it is the most powerfull database in the world. Both MySQL and MariaDB are far down bellow. Just fix the bug in knew and support wide range of the backends.

Hey all 👋 , I've locked this thread to contributor comments only and updated the original post with an explanation around DB support.
I understand there's a lot of frustration around the clarification that MariaDB is not an officially supported database. As a small team we've always provided a very narrow set of environments that are officially supported to keep our maintenance overhead manageable: Ghost is intended for use with Ubuntu, MySQL, nginx & Node.js LTS.
We're not able to support every flavour of environment without differences being handled correctly in the upstream packages. The way to change what's supported would be to contribute the necessary fixes & improvements - we'd absolutely love to see more contributions around database interoperability in knex.

Closing this now as the official documentation has been published https://ghost.org/docs/changes/


Start using Python type annotations and checking

Partly mirrors #12, but we don't necessarily have to use dataclasses.
For example, many of the schedule passes have a bunch of boilerplate to check the types of the arguments and raise standard error messages if they're wrong. This feels like it should be managed with type annotations and a standard type checking system.
Static type checking (mypy or similar) is also worth considering.

Static type checking (mypy or similar) is also worth considering.

I can't wait to see what adt does to mypy 😉

…that's why the original issue #12 specifically combined "rewrite to use dataclasses and types" :)
I think the idea would be to have ADT generation be a static code generation process that spits out Python source, and that source would be simpler, more idiomatic dataclass declarations as much as possible.

I think the idea would be to have ADT generation be a static code generation process that spits out Python source, and that source would be simpler, more idiomatic dataclass declarations as much as possible.

That has some pros/cons... import statements are a hassle with generated Python sources. You need an external build system (AFAIK) to coordinate this. I anticipate better would be to use PEP 561 stub files (.pyi) and continue using runtime-generated classes.

Also reasonable.

A tangential issue worth considering is whether ADT errors vs type errors at the exposed API should raise different kinds of exceptions so that users can distinguish a “my problem” from a “your problem”. Current convention (possibly violated?) is that internal errors should fail assertions.

On Nov 14, 2021, at 8:23 PM, Jonathan Ragan-Kelley @.***> wrote:

Also reasonable.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

We're going tag this into the bigger Exo core rewrite issue


Listening only on host IP

Please first read the documentation. Problems affecting the client, e.g. Plex or Emby should be reported there in the forum.
Describe the bug
With the latest container update, Threadfin is now not listening do the default "<IP_ADDRESS>" or "::" - it seems to select a single address and binds. This breaks Threadfin in a docker container with host networking.
I believe the bug was introduced by https://github.com/Threadfin/Threadfin/commit/b27dfdd3c069a5e35b83b7dd3cfeba065a3f5068
Server (please complete the following information):

OS Ubuntu 23.10
Arch x86_64
Threadfin Version and Build 1.1.6

To Reproduce
Steps to reproduce the behavior:

Update the existing working threadfin container to 1.1.6
Fail to connect to threadfin

Expected behavior
Threadfin should listen on the default IPv4 and IPv6 addresses unless the user explicitly configures otherwise. Selecting Threadfin's interface using the current logic will break many configurations.
Screenshots
N/A
Desktop (please complete the following information):

OS:
Browser:
Version:

Additional context

same here and same/similar issue as in https://github.com/Threadfin/Threadfin/issues/253

I was wrong about the cause of the issue, this was introduced in 1.1.1. That is to say 1.1.0 was the last working version.

Hey guys, I can confirm it is now working with 1.8 when setting the httpdomain and restarting the service.

Hey guys, I can confirm it is now working with 1.8 when setting the httpdomain and restarting the service.

What IP address does it listen on? Is it listening on both IPv6 and IPv4?

IPv4 currently, but should also work with IPv6, but no guarantee

IPv4 currently, but should also work with IPv6, but no guarantee

So it's back to listening on "::" as it should? Or it's binding to a specific IP of the hostname?

sudo netstat -tulpn | grep threadfin gives me this:
tcp        0      0 <IP_ADDRESS>:8091          <IP_ADDRESS>:*               LISTEN      484563/threadfin
udp        0      0 <IP_ADDRESS>:1900            <IP_ADDRESS>:*                           484563/threadfin

I've created a PR for this to fix: https://github.com/Threadfin/Threadfin/pull/260

This has already been fixed in 1.1.8


Request affected firmware version<4.1

Hello, do you have a Firmware before WebHMI4.1? I want to verify this vulnerability, but the official website has been updated to version 4.1.If so, please send me the email<EMAIL_ADDRESS>Thank you very much.

Hello,
you still can download vulnerable version in their Level2 application.
The link is here.

您好， 您仍然可以在他们的 Level2 应用程序中下载易受攻击的版本。 链接在这里。请让我知道这对你有没有用。

I have successfully downloaded it, thank you very much for your answer！


Fixed potential race condition

When I moved away from the login page and was redirected back to the login page, the FirebaseAuth component disappeared. If I did it again, I'd get the error Uncaught (in promise) Error: AuthUI instance is deleted! on the following line
https://github.com/firebase/firebaseui-web-react/blob/a9d71ba85765ec03a12624f94eaaef2cba2b8ad0/src/FirebaseAuth.jsx#L92
Using logging, I determined this was because the code in the promise in componentDidMount() was running before the code in the promise in componentWillUnmount(). To prevent this, I just set firebaseUiDeletion outside of promise.

Good catch. Thanks @jarett-lee !

for some I get an exception in that line as this.firebaseUiWidget is undefined : StyledFirebaseAuth.js:96 Uncaught (in promise) TypeError: Cannot read property 'delete' of undefined at StyledFirebaseAuth.js:96
still investigating

@jarett-lee @nicolasgarnier I'm seeing the exact same behavior as described above. I don't know why it's happening.

@jarett-lee @nicolasgarnier
In FirebaseAuth.jsx:
    firebaseUiDeletion = firebaseUiDeletion.then(() => {
      this.unregisterAuthObserver();
      return this.firebaseUiWidget.delete();
    });

Does this.unregisterAuthObserver() return a Promise?
In which case it should be:
    firebaseUiDeletion = firebaseUiDeletion.then(() => {
      this.unregisterAuthObserver().then(() => {
        return this.firebaseUiWidget.delete();
      })
    });


Update stable version in repo from 0.1.1 to 0.1.2

Think about update stable version to 0.1.2
Version: stable=0.1.1 (1394 days old) => latest=0.1.2 (15 days old)
Installs: stable=136 (93.15%), latest=9 (6.16%), total=146
Click to edit

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.1.2 but the current stable version is already 0.1.2.
So this issue should be closed.


Adding DeepLink Support

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Submission validity

[x] This is not a personal support request, that should be posted on discussions community.
[X] I checked current issues and this request isn't a duplicate of an existing issue, opened or closed.

Summary
When interacting with Shorted Links on Social Media Apps (general use case), many people are nowadays on their phone and expect the link to open in the desired app. But Since Apps on Phones are Sandboxed, the Link isn't getting forwarded to the corresponding app therefore we need deep linking to solve this issue.
Motivation
Users are logged in on their social apps already.
When opening the Link in the Browser, potential new followers miss the opportunity to subscribe, since they don't wanna copy the topic of the link and open it in the desired app (e.g. a youtube link on instagram is getting opened in the instagram browser).
Describe the solution
The Solution would be an option inside of YOURLS to toggle: "Deeplink" or "No Deeplink".
If the toggle is set to "DeepLink" a deeplink script runs after, a available shorted URL gets resolved and trys to open up the app if available. If the App is not available on the phone, the fallback value would be resolving the original url (important for pc users).
In Addition to Deep Linking we run in a little Problem, which should also be faced.
Previews and Descriptions of Videos are no longer displayed in Social Media Apps like Instagram, therefore we either need to copy and passthrough the Original Thumbnail and Description or just leave it blank
So far I can speak for "MANY" Content Creators around the globe which trust in YOURLs, since it's the only easy to use, plugin solution on the market. We all need this Solution, because this is the Dealbreaker Feature of Web 2.0 Services.

Duplicate of #2504, and still plugin territory.
The wiki link referenced there has moved to https://yourls.org/docs/guide/extend/possible-with-a-plugin
As far as I can tell from searching our issues and discussions, this has been requested only twice in the past 5 years. Any feature that is to be implemented in YOURLS core must have much higher utility for the general user community.

dgw You Closed this Case way to early in my opinion.
Only because the most users are not "claiming" that they wish to have this plugin,
doesn't mean that their is no need for it.
But hey let's assume you are right and nobody needs to have it.
I can still gurantee you, that many people will appreciate this feature in the Future.
Because Links nowadays are already as small as possible, the days in which links were
tremendous so that you wanted to short them are mostly over.
But having the opportunity to open links in their corresponding app, like an audible link (shorted) in audible. Because the amazon audible link can't deeplink into the app itself since it's not implemented in their service. Would make link rewrite- and shortening great again ;)

If you assume I'm right that nobody needs to have this feature, how could you guarantee that many people will appreciate it? Users saying that they want something is the only way we have to prioritize and assess what gets added to the roadmap. If this feature was really in demand, our issues would be "noisier" about it. (Insert old saying that "a squeaky wheel gets the grease".)
You yourself described a need for fallback behavior, checking if an app is installed on the user's device and using the regular URL if it's not available. Implementing this would not be simple. It would add significant complexity to the application for little perceived benefit to the general userbase, and then the core team would also be responsible for maintaining it.
Speaking of maintenance, that's another reason this is better done in a plugin: A plugin can be updated whenever necessary to address changes in the mobile app world. A feature built into core would have to update in YOURLS' maintenance releases, which are pretty infrequent these days (as the core product is quite mature).

Solid Point. Fully understand your thought.
Other Question, is it possible that we might get this as an official plugin?
Would be nice to have it here. (I've already tried to code it myself but wasn't successful yet)

is it possible that we might get this as an official plugin?

Possible? Of course, almost anything is possible.
Likely? No.
I personally don't have any interest in it. Another maintainer might, but I also think if one of them did we'd already have such an official plugin and you wouldn't have had to ask.


[PaddlePaddle] set get_dataloader_workers to 4

Hi @astonzhang, is there something wrong for d2l CI, why CI is always pending?


Consider instantiating textNBC when setting module.exports

Like express (HTTP server), and unlike other wink libraries, NBC needs to be instantiated before using.
Express
var express = require('express');
var app = express();

NBC
var nbc = require( 'wink-naive-bayes-text-classifier' )();

In our source code, could we:
module.exports = textNBC();

The function doesn't have any arguments so there aren't two ways of doing this any way.

It is required to aid user if s/he needs more than one instance; plus in future it may accept arguments for some additional behavior. In the example require( 'wink-naive-bayes-text-classifier' )() is a shortcut!

I have a feeling that someone might miss the extra () (while writing their own code), we should at least split it in two lines, like in the case of express.


Cherry-pick to earlgrey_1.0.0: [rom_ext_e2e] Check the attestation certificates

This PR is a manual cherry-pick of #24793 & #24829.

Examine the attestation certificates and check the measurements published in the DiceTcbInfo extensions.
Report the DICE measurements in big-endian order.
Check the key IDs reported by the DICE certs.

The CI test failure is not related to this change.


[TECH-128] WIP

branch: feature/TECH-128-cloudfront-kubernetes-deploy

📕 TECH-128 Implement Cloudfront Kubernetes Deploy
Migrate CloudfrontDeploy step in mpl-modules to mpyl.
T-Shirt size: X

Compare with https://github.com/Vandebron/mpl-modules/blob/fe40c19e36549207b82dac7e930f154415b1954f/resources/nl/vandebron/nestedlib/modules/Deploy/CloudFrontKubernetesDeploy.groovy
Create a new step in mpyl/steps/deploy
🏗 Build 2 ✅ Successful, started by Ben Lynch
🚀  enrichChargeSessionsJob, job, kong-sync, nodeservice, sbtservice

Looks great in general! I like the fact that you've wrapped the S3 client. First use of mocking for this project 👍. Although it's generally best to keep mocking to a mininimum, I think there are currently some untested bits that we could test with the help of patch.
For temp folders:
https://github.com/Vandebron/mpyl/pull/122/files#diff-d15968178de9592b92d529099660ebe7dbf683c7f1d25205bec0ac16d42e0bafR33-R37
you can use https://docs.python.org/3/library/tempfile.html#tempfile.TemporaryDirectory using a with (try with resoruces), so the temp files are guaranteed to be cleaned up
I think this a good way to chain steps :thumbs: https://github.com/Vandebron/mpyl/pull/122/files#diff-d15968178de9592b92d529099660ebe7dbf683c7f1d25205bec0ac16d42e0bafR39
For https://github.com/Vandebron/mpyl/pull/122/files#diff-e8b4ceffe17bb0b13add37010de17c46af0323088877b5afccd5b6d750cf6142R83-R85 you can use https://github.com/Vandebron/mpyl/blob/a541debbc05becb2f22c6058e7c8bb53f1c09159/src/mpyl/steps/models.py#L29-L30
The logic at https://github.com/Vandebron/mpyl/pull/122/files#diff-e8b4ceffe17bb0b13add37010de17c46af0323088877b5afccd5b6d750cf6142R92-R104 is very similar to https://github.com/Vandebron/mpyl/blob/9b0151d9d22cf6f2c689793fa785725c06c8e692/src/mpyl/steps/test/dockertest.py#L79-L88, maybe you can find a way to reuse. You can probably also use container id as a unique identifier (like here: https://github.com/Vandebron/mpyl/blob/9b0151d9d22cf6f2c689793fa785725c06c8e692/src/mpyl/steps/test/dockertest.py#L74)
Do we need wrappers around such fundamental operations? https://github.com/Vandebron/mpyl/pull/122/files#diff-2142665a90e5a80a6625be7c158fac3ab9473a6b7c9fc9bc77a820c98497bef1R8 Rmtree for example already has a parameter to ignore errors: https://github.com/Vandebron/mpyl/pull/122/files#diff-2142665a90e5a80a6625be7c158fac3ab9473a6b7c9fc9bc77a820c98497bef1R28
WRT https://github.com/Vandebron/mpyl/pull/122/files#diff-96f40a0494c735a518dcd08ad1cdb2b5faa5d056475d286c333b74324d3a3d13R13. The pattern we've been following so far is to use pure data classes for config. See here for example: https://github.com/Vandebron/mpyl/blob/ae36365ef97697b2942aee6811f49efaf35701cc/src/mpyl/utilities/github/init.py. The preferred way is to use a factory method like https://github.com/Vandebron/mpyl/blob/ae36365ef97697b2942aee6811f49efaf35701cc/src/mpyl/utilities/github/init.py#L34.
I don't think we need to replicate the requiredness checks that are already in the schema, but more complicated validations can still be done in code and throw a KeyError when appropriate.
In a couple of locations i've noticed you capture exceptions and return a boolean instead. Why not let the exceptions bubble up?

There was an update that allows the mpyl config branch to specified in the test pipeline - after rebasing on main I still wasn't able to specify the branch with this PR so I created a new branch & PR
I've addressed all the previous comments within the new PR: https://github.com/Vandebron/mpyl/pull/141


Update on-prem deployment with geeky details

Description
Give some more details on what you get with on-prem deployments
Related issues/PRs
Todos
Release Note
None required

Deploy preview for calico ready!
Built with commit b07c88213cc45c45b1e7e9489160fa4e9aac33b6
https://deploy-preview-3681--calico.netlify.app


Indicator setup form: Deleting an indicator should take you back to the program page

Affects: Both versions of the indicator setup form -- the full page version and the pop-up version.
When I click "Delete this indicator" and then confirm the deletion, I expect to be returned to the program page.
Instead, I'm taken to the old Indicator list page.

This is duplicated by #807.


Increase the regularity of the code

To increase the regularity of the code:

it is recommended to remove all @author and @Date at the top of the class; #759
clean up Unused import at the top of the class; #758

Close this PR because it has too many conflicts.


Validating issuer with multi-tenant should not just turn validate issuer off.

The suggested code is:

TokenValidationParameters = new System.IdentityModel.Tokens.TokenValidationParameters
{
ValidateIssuer = false,
}

An alternative is to set the delegate: TokenValidationParameters.IssuerValidator
Control will be passed to this delegate that has a signature:
(string issuer, SecurityToken securityToken, TokenValidationParameters validationParameters)
The application can then make a dynamic decision determining if the issuer is valid.

Brent, we've provided an example delegate implementation at Utils\AadIssuerValidator.cs

Using the delegate seems correct.
How do the issuers get populated into TokenValidationParameters?

The method BuildTokenValidationParameters does it.
The sample's first step if for admins to first sign-up and admin consent the app in a tenant. Then the users can proceed to sign in.
In this method, the "signed-up" tenants are saved in the DB and this method provides them as a list of valid tenants whose users are allowed to sign-in .

Closing this issue as the problem is solved


[snapshots] implement pigz for snapshot archiving

compress a dir
tar --use-compress-program="pigz -k " -cf dir1.tar.gz dir1
uncompress
pigz -d dir1.tar.gz OR unpigz dir1.tar.gz -> this will need adjusting in all backend stack install scripts AND the 'sync processor from 0' scripts before compressing the snapshots with pigz

after testing ended up with
compression: tar --use-compress-program="pigz -k --best --recursive | pv " -cf "/storage/rsync-public/tfchain-mainnet-$(date '+%Y-%m-%d').tar.gz" * | pv
decompression: tar -I pigz -xf tfchain-mainnet-best.tar.gz -C extract/ | pv
testing a stack deploy from snapshot to be sure no issues popped up. Will then implement for all nets and apply

Big difference in compression time, but archives are about 30% bigger. In total both creating and extracting saves a lot of time so let's go on with it.

implemented on all nets


Divide by zero.

On Linux Mint 17.3
python2 tryit.py blah.wav
tryit.py:97: RuntimeWarning: divide by zero encountered in log10
y = (20.0 * np.log10(y)).clip(-120)
Traceback (most recent call last):
File "tryit.py", line 145, in
sys.exit(main(*sys.argv[1:]))
File "tryit.py", line 124, in main
plt.yscale('symlog', linthreshy=100, linscaley=0.25)
File "/usr/lib/pymodules/python2.7/matplotlib/pyplot.py", line 1522, in yscale
ax.set_yscale(*args, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/axes.py", line 2801, in set_yscale
self.yaxis._set_scale(value, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/axis.py", line 693, in _set_scale
self._scale = mscale.scale_factory(value, self, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/scale.py", line 504, in scale_factory
return _scale_mapping[scale](axis, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/scale.py", line 449, in __init __
assert linscale >= 1.0
AssertionError
python3 tryit.py blah.wav
tryit.py:97: RuntimeWarning: divide by zero encountered in log10
y = (20.0 * np.log10(y)).clip(-120)
Traceback (most recent call last):
File "tryit.py", line 145, in
sys.exit(main(*sys.argv[1:]))
File "tryit.py", line 103, in main
X = X[window_size / overlap:]
TypeError: slice indices must be integers or None or have an _index_ method
file blah.wav
blah.wav: RIFF (little-endian) data, WAVE audio, Microsoft PCM, 16 bit, stereo 22050 Hz

It appears that the reason for the crash is not the division by zero. Numpy will actually give a result of -inf, which is then 'fixed' by the call to clip. For example:
>>> x = np.array([0, 0, 0])
>>> y = np.log10(x)
__main__:1: RuntimeWarning: divide by zero encountered in log10
>>> y
array([-inf, -inf, -inf])
>>> y.clip(-120)
array([-120., -120., -120.])

The crashes you are seeing are actually related to different errors: In the Python 2 case, it appears to be an assertion error related to plot scaling, and in the Python 3 case it is due to the altered behavior of division from Python 2.
I believe I have fixed the error related to Python 3 in commit 57d6730692ae1b4d9e66e72fdf506b01195f35ae. I am unable to replicate the other error with any wav file I have on hand. Please send the clip you are using as to assist me in debugging.

Since the first error you observed appears to be related to the use of the symlog scale, you may want to see if it persists when you choose linear scaling:
python spectrogram.py input.wav 1024 linear
The ability to choice between linear/log is added in commit a1f643404c76e239ec238711f919f74202ad2169.

Just tried the latest master.
Works with both versions of python (2 and 3):
python spectrogram.py input.wav 1024 linear
Fails with both versions of python (2 and 3):
python spectrogram.py input.wav
$python2 spectrogram.py blah.wav
spectrogram.py:73: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
Traceback (most recent call last):
File "spectrogram.py", line 110, in
sys.exit(main(*sys.argv[1:]))
File "spectrogram.py", line 89, in main
plt.yscale('symlog', linthreshy=100, linscaley=0.25)
File "/usr/lib/pymodules/python2.7/matplotlib/pyplot.py", line 1522, in yscale
ax.set_yscale(*args, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/axes.py", line 2801, in set_yscale
self.yaxis._set_scale(value, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/axis.py", line 693, in _set_scale
self._scale = mscale.scale_factory(value, self, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/scale.py", line 504, in scale_factory
return _scale_mapping[scale](axis, **kwargs)
File "/usr/lib/pymodules/python2.7/matplotlib/scale.py", line 449, in init
assert linscale >= 1.0
AssertionError
#SUCCESS.
$python2 spectrogram.py blah.wav 1024 linear
spectrogram.py:73: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
$python3 spectrogram.py blah.wav
spectrogram.py:73: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
Traceback (most recent call last):
File "spectrogram.py", line 110, in
sys.exit(main(*sys.argv[1:]))
File "spectrogram.py", line 89, in main
plt.yscale('symlog', linthreshy=100, linscaley=0.25)
File "/usr/lib/python3/dist-packages/matplotlib/pyplot.py", line 1522, in yscale
ax.set_yscale(*args, **kwargs)
File "/usr/lib/python3/dist-packages/matplotlib/axes.py", line 2803, in set_yscale
self.yaxis._set_scale(value, **kwargs)
File "/usr/lib/python3/dist-packages/matplotlib/axis.py", line 694, in _set_scale
self._scale = mscale.scale_factory(value, self, **kwargs)
File "/usr/lib/python3/dist-packages/matplotlib/scale.py", line 504, in scale_factory
return _scale_mapping[scale](axis, **kwargs)
File "/usr/lib/python3/dist-packages/matplotlib/scale.py", line 449, in init
assert linscale >= 1.0
AssertionError
#SUCCESS
$python3 spectrogram.py blah.wav 1024 linear
spectrogram.py:73: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
python2 --version
Python 2.7.6
python3 --version
Python 3.4.3
dpkg -l | grep matplotlib
ii  python-matplotlib                           1.3.1-1ubuntu5                                      amd64        Python based plotting system in a style similar to Matlab
ii  python-matplotlib-data                      1.3.1-1ubuntu5                                      all          Python based plotting system (data package)
ii  python3-matplotlib                          1.3.1-1ubuntu5                                      amd64        Python based plotting system in a style similar to Matlab (Python 3)
dpkg -l | grep numpy
ii  python-numpy                                1:1.8.2-0ubuntu0.1                                  amd64        Numerical Python adds a fast array facility to the Python language
ii  python3-numpy                               1:1.8.2-0ubuntu0.1                                  amd64        Fast array facility to the Python 3 language
I'll find a way to get you blah.wav
I don't want to post here publicly because it's from Diablo II and therefore Copyrighted (the wav was directly ripped from d2sfx.mpq, which is in the Diablo II install CD). I'm 94.2% sure the sound was created in 1998 with Soundforge as stated in one of the sound files (.wav) I extracted from one of the Diablo II mpq files that I saw in some program somewhere.

I think I figured it out... I had purposely set linscaley=0.25 to reduce the displayed size of the linearly-scaled portion of the spectrum, however this is only allowed since matplotlib 1.4. You appear to be on 1.3, and I was testing with 1.5. It is not related to the specific input file you are using. You can set linscaley=1 on spectrogram.py:89 if you need compatibility with old matplotlib.
Refer to https://github.com/matplotlib/matplotlib/issues/2288/

Fixed!
python3 spectrogram.py blah.wav
spectrogram.py:73: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
python2 spectrogram.py blah.wav
spectrogram.py:73: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
diff spectrogram*
89c89
<         plt.yscale('symlog', linthreshy=100, linscaley=1)
---
>         plt.yscale('symlog', linthreshy=100, linscaley=0.25)
Is it best to refer to people that do have < matplotlib 1.4 to just come here (ie. this solution post #2) for the solution?
And you have my permission to rename this spectrogram's issue's name, which is also the beginning of the title of this page. The "divide by zero" thrown RuntimeWarning is still persistent, but the real problem was the crash caused by using an older python2 and python3 matplotlib 1.3.* library/module/package/a-bunch-of-python-code/ASCII-files/filesystem-malaygna/obligata/sanka/demois.

I can probably instrument the script to detect old matplotlib and set an appropriate linscaley (and probably throw a warning) in that case

I detect there is a 4% chance this might help make things go faster:
$python2 spectrogram_possible.py blah.wav
spectrogram_fixed.py:74: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
spectrogram_fixed.py:94: UserWarning: You are using matplotlib 1.3.* (and not >= 1.4.0). Therefore linscaley must equal 1, not 0.25
warnings.warn('You are using matplotlib 1.3.* (and not >= 1.4.0). Therefore linscaley must equal 1, not 0.25')
$python3 spectrogram_possible.py blah.wav
spectrogram_fixed.py:74: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
spectrogram_fixed.py:94: UserWarning: You are using matplotlib 1.3.* (and not >= 1.4.0). Therefore linscaley must equal 1, not 0.25
warnings.warn('You are using matplotlib 1.3.* (and not >= 1.4.0). Therefore linscaley must equal 1, not 0.25')
$python2 spectrogram_possible.py blah.wav 1024 linear
spectrogram_fixed.py:74: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
$python3 spectrogram_possible.py blah.wav 1024 linear
spectrogram_fixed.py:74: RuntimeWarning: divide by zero encountered in log10
Y = (20.0 * np.log10(Y)).clip(-120)
diff spectrogram.py spectrogram_possible.py
https://gist.github.com/psycho23/8d495f8b31c05fec6c0cbb6e5c5c40fb


Bug report 添加mongodb配置信息不正确

Contribution guidelines

[X] I've read the contribution guidelines and wholeheartedly agree

I've found a bug and checked that ...

[X] ... the problem doesn't occur with the server or web themes
[ ] ... the documentation does not mention anything about my problem
[ ] ... there are no open or closed issues that are related to my problem

Description
2023-05-07 05:58:01,243 DEBUG [http-nio-9096-exec-7] org.apache.commons.beanutils.converters.BooleanConverter [AbstractConverter.java:140] Converting 'Boolean' value 'false' to type 'Boolean'
2023-05-07 05:58:01,244 DEBUG [http-nio-9096-exec-7] org.apache.commons.beanutils.converters.BooleanConverter [AbstractConverter.java:162]     No conversion required, value is already a Boolean
2023-05-07 05:58:01,244 DEBUG [http-nio-9096-exec-7] org.apache.commons.beanutils.converters.IntegerConverter [AbstractConverter.java:140] Converting 'Integer' value '30307' to type 'Integer'
2023-05-07 05:58:01,245 DEBUG [http-nio-9096-exec-7] org.apache.commons.beanutils.converters.IntegerConverter [AbstractConverter.java:162]     No conversion required, value is already a Integer
2023-05-07 05:58:01,246 DEBUG [http-nio-9096-exec-7] org.apache.commons.beanutils.converters.StringConverter [AbstractConverter.java:140] Converting 'String' value '<IP_ADDRESS>' to type 'String'
2023-05-07 05:58:01,246 INFO [http-nio-9096-exec-7] io.edurt.datacap.spi.connection.JdbcConnection [JdbcConnection.java:70] Connection driver com.mongodb.jdbc.MongoDriver
2023-05-07 05:58:01,247 INFO [http-nio-9096-exec-7] io.edurt.datacap.spi.connection.JdbcConnection [JdbcConnection.java:71] Connection url jdbc:mongodb://<IP_ADDRESS>:30307?authSource=admin
2023-05-07 05:58:01,247 INFO [http-nio-9096-exec-7] io.edurt.datacap.spi.connection.JdbcConnection [JdbcConnection.java:73] Connection username with admin password with ***
2023-05-07 05:58:01,250 ERROR [http-nio-9096-exec-7] io.edurt.datacap.spi.connection.JdbcConnection [JdbcConnection.java:87] Connection failed
java.sql.SQLException: java.lang.IllegalArgumentException: The connection string contains options without trailing slash
at com.mongodb.jdbc.MongoDriver.getConnectionSettings(MongoDriver.java:370)
at com.mongodb.jdbc.MongoDriver.getUnvalidatedConnection(MongoDriver.java:193)
at com.mongodb.jdbc.MongoDriver.connect(MongoDriver.java:165)
at java.sql.DriverManager.getConnection(DriverManager.java:664)
Expected behaviour
配置主机IP，端口，后日志输出错误提示无法正确连接到mongodb
Actual behaviour
配置主机IP，端口，后日志输出错误提示无法正确连接到mongodb。必然出现
Steps to reproduce
1，数据源管理
2，添加数据源
3，选择mongodb
4，配置主机，端口。自定义认证authsource=admin
Package versions
docker run -d -p 9909:9096 --name datacap qianmoq/datacap
Configuration
无

System information

Operating system: ... windows7/windows10/windows11
Browser: edge

Is mongo the community open source version or the atlas version
i used docker to run it.
docker image version is :
mongo:6.0.5


test: Add tests for add tx on declare tx

This change is

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

#235  👈
#234
main

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

Join @ArniStarkware and the rest of your teammates on  Graphite

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.90%.

Additional details and impacted files
@@                       Coverage Diff                        @@
##           arni/declare/compile_in_gateway     #235   +/-   ##
================================================================
  Coverage                            68.90%   68.90%
================================================================
  Files                                   23       23
  Lines                                 1000     1000
  Branches                              1000     1000
================================================================
  Hits                                   689      689
  Misses                                 265      265
  Partials                                46       46

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

crates/gateway/src/starknet_api_test_utils.rs line 96 at r2 (raw file):
Previously, Yael-Starkware (YaelD) wrote…
(and also as in deploy_account - PR https://reviewable.io/reviews/starkware-libs/mempool/239#-)

The difference between Invoke/DeployAccount and Declare is that the declare tx is much bigger. The Sierra program is long.
I agree - this is also bad for consistency.
I think the solution is to load from a json only the contract_class / only the sierra_program.
WDYT?


[WebToolsE2E][Aspire] With SDK 8.0.300 + Aspire 8.0 GA, dotnet new aspire-starter does not work due to manifest provider returned a duplicate manifest ID 'microsoft.net.sdk.android'.

INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU

Install SDK 8.0.300

Apply feed for SDK 8.0.300

Install Aspire 8.0 GA

Apply the "darc-pub-dotnet-aspire" feed for Aspire 8.0 GA

Install cred helper (run from Windows PowerShell admin prompt)
iex "& { $(irm https://aka.ms/install-artifacts-credprovider.ps1) }"

REPRO STEPS

Run dotnet --info  -- works fine

Run dotnet workload list

Run dotnet new aspire-starter -o Aspire-StarterApp01

ACTUAL
Template creation failed

EXPECTED
Template creation should succeed

This issue can also be reproduced with a clean install of VS 17.10 GA (with Aspire 8.0 GA)+ Web workload when creating a new template using the CLI, but works fine using VS.

@joperezr are we missing an additional feed? It seems that 'dotnet new aspire-starter' doesn't work using CLI with above steps.

The error seems similar to an issue that was reported earlier as well - internal Bug 2048871.

Elena and I investigated today and it seems like a possible feed permissions issue. But still, we have two machines that Elena and I setup - she can still reproduce the issue on her machine and I cannot reproduce the issue on my machine. We'll continue doing more investigation on Monday.

I installed a new machine today and can still reproduce this issue, it happens randomly and the failure rate is about 50%.

@joperezr @marcpopMSFT with the correct feeds applied, we seem to still run into this issue about 50% of the time, even within the same Command Prompt session as you can see in Elena's screenshot. Can you investigate, please?

To summarize the investigation, there appears to be a parallelism issue within the template engine that affects workload related templates. Daniel is still digging into possible fixes but the short-term workaround (besides the customer just trying again) is to have the aspire templates remove the workload constraint. It should not be necessary within templates provided by workloads and appears to be causing duplicate resolution of the same workload.  That's our recommendation for 8.0.1 version of the aspire workload.
https://github.com/dotnet/aspire/blob/main/src/Aspire.ProjectTemplates/templates/aspire-apphost/.template.config/template.json#L26-L31

Here are the details that I found in my investigation: https://github.com/dotnet/templating/issues/7946#issuecomment-2108853692

Close this issue as it was fixed both on Aspire 8.0.1 and 8.1 Preview 1.


Model Zoo not accessible

The models mentioned in Model Zoo part cannot be accessed and loaded due to some access issues.
Is there a way to download them another way?
Thanks

Hi,
I have extended the validity of those download links. Please try again.
All the best.
Xingyang


Add version of interdigital capacitor with CPWs and ground layer

This PR adds a version of interdigital capacitor with CPWs and ground layer.
Additionally, a get_capacitance_path method is added to PDK, similar to get_sparameters_path.
This is handy for testing capacitive simulations such as the ones enabled by https://github.com/gdsfactory/gplugins/pull/42
The code isn't very clean but please tell is something may be improved

Thank you Niko!


⚠ Processors has degraded performance

In 6b6b210, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 128 ms

Resolved: Processors performance has improved in b88178e.


Add support in oesign for PE binaries

The oesign tool still has a several ELF-specific code paths that need to be generalized like the loader so that it is agnostic to the file format it is operating on when adding signatures to the app.
This is a prerequisite for the props and debug-mode tests to build successfully as part of enabling PE enclave tests on Windows (#1035)

Closing Windows PE enclave related work until we decide to pursue that again.


Sbt uses too much cpu on idle

In 1.3.0-RC1, sbt is using roughly 10% of a cpu when idling. In older versions, it was more like 2% of a cpu. This is because I reduced a number of timeouts that are causing sbt to frequently wake up to poll queues. These can be reworked to poll from a concurrent blocking queue which should both lower latency and restore the old cpu utilization characteristics.

Ref https://github.com/sbt/sbt/pull/4544/commits/fcd24ba7cd7b26c04d750ec70b026e5d5d654356
So should we change it back to Thread.sleep(50)? 0.05s should be below human perception and easy enough for machines.

Fixed in #4750


Typo in TodoContext.cs code.

On line 10, the namespace is entered as 'TodoAp', when it should be 'TodoApi'.

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

ID: 583ae3e7-5915-f22b-e54c-6c100cf2a7da
Version Independent ID: 95f2549e-29f9-e31a-b4d0-432b971643ff
Content: Tutorial: Create a web API with ASP.NET Core MVC
Content Source: aspnetcore/tutorials/first-web-api.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@ramcd86 	good catch. I'll try to get that fixed today.

@Rick-Anderson Already addressed in the linked PR. Please review.


Add unit tests for compatibility with @carbonplan/maps

As referenced in https://github.com/carbonplan/maps/issues/14#issuecomment-930787242 and https://github.com/carbonplan/ndpyramid/issues/87, we should add a utility function and unit tests for compatibility with @carbonplan/maps, based on the requirements documented in https://ndpyramid.readthedocs.io/en/latest/schema.html.

Even just a python validator that uses zarr-python to check metadata compliance would be very useful.

+1 on the idea for checking metadata compliance.
pydantic-zarr could also be helpful for validation


Introduce dirty flags to limit expensive requests

kak-lsp requests inlay hints and semantic tokens on every NormalIdle, even when all I'm doing is move the cursor. Depending on the language server and file size, the responses for that can be quite big.
Additionally, when NormalIdle is triggered by some form of "menuing" (e.g. running a command like lsp-hover-buffer that doesn't affect the buffer at all), then not even the cursor changes, and reference highlighting doesn't need updates either.
If possible, there should be some form of "dirty" flags that track cursor movement and file changes and that prevent certain requests from running when they are not necessary, at least when triggered by NormalIdle hooks.

yeah inlay hints and semantic tokens do not depend on the cursor position, so we should only re-send them if the buffer changed.
The existing code already tries to do that via lsp-did-change which does
    evaluate-commands %sh{
        if [ $kak_opt_lsp_timestamp -eq $kak_timestamp ]; then
            echo "fail"
        fi
    }
    set-option buffer lsp_timestamp %val{timestamp}

I don't understand how that even works right now. It reads to me like the first hook to run lsp-did-change makes the other hooks a nop.. but that can't be true

It does look like that. It's scoped by buffer instead of global, at least, but still.
On the other hand, evidence of all requests running always suggests that it somehow doesn't prevent anything.

I'll look into it.
At least for semantic tokens we can switch to the refresh variant (making the server is responsible for sending updates) as proposed in https://github.com/kak-lsp/kak-lsp/pull/546/commits/aa9af5e220126313dea7f035a43a0cdd80a870f5

Additionally, when NormalIdle is triggered by some form of "menuing" (e.g. running a command like lsp-hover-buffer that doesn't affect the buffer at all),

for inlay hints and semantic highlighting, that's a clear improvement but for auto-hover we may want to still show it, I'm not sure
Here's a change that should do the trick.
Probably we should refactor it to do it in the same shell call as lsp-did-change.
Or do the cache logic on the server side..
In future we should reimplement all lsp commands by echoing data to a socket (like pykak), then we'll need almost no shell calls.
diff --git a/rc/lsp.kak b/rc/lsp.kak
index ae9f0b38..9b632037 100644
--- a/rc/lsp.kak
+++ b/rc/lsp.kak
@@ -1310,7 +1310,13 @@ incomingOrOutgoing = $1

 define-command -hidden lsp-inlay-hints -docstring "lsp-inlay-hints: request inlay hints" %{
-  lsp-did-change-and-then lsp-inlay-hints-request
+    declare-option -hidden str lsp_inlay_hints_cache_key
+    evaluate-commands %sh{
+        if [ "$kak_opt_lsp_inlay_hints_cache_key" != "$kak_timestamp" ]; then
+            echo set-option buffer lsp_inlay_hints_cache_key %val{timestamp}
+            echo lsp-did-change-and-then lsp-inlay-hints-request
+        fi
+    }
 }

 define-command -hidden lsp-inlay-hints-request %{
@@ -1560,7 +1566,13 @@ define-command tsserver-organize-imports -docstring "Ask typescript-language-ser
 # semantic tokens

 define-command lsp-semantic-tokens -docstring "lsp-semantic-tokens: Request semantic tokens" %{
-  lsp-did-change-and-then lsp-semantic-tokens-request
+    declare-option -hidden str lsp_semantic_tokens_cache_key
+    evaluate-commands %sh{
+        if [ "$kak_opt_lsp_semantic_tokens_cache_key" != "$kak_timestamp" ]; then
+            echo set-option buffer lsp_semantic_tokens_cache_key %val{timestamp}
+            echo lsp-did-change-and-then lsp-semantic-tokens-request
+        fi
+    }
 }

 define-command -hidden lsp-semantic-tokens-request %{
@@ -2029,10 +2041,18 @@ define-command lsp-auto-hover-enable -params 0..1 -client-completion \
     -docstring "lsp-auto-hover-enable [<client>]: enable auto-requesting hover info for current position

 If a client is given, show hover in a scratch buffer in that client instead of the info box" %{
+    declare-option -hidden str lsp_auto_hover_cache_key
     evaluate-commands %sh{
         hover=lsp-hover
         [ $# -eq 1 ] && hover="lsp-hover-buffer $1"
-        printf %s "hook -group lsp-auto-hover global NormalIdle .* %{ $hover }"
+        printf %s 'hook -group lsp-auto-hover global NormalIdle .* %{
+            evaluate-commands %sh{
+                if [ "$kak_opt_lsp_auto_hover_cache_key" != "$kak_timestamp $kak_selections_desc" ]; then
+                    echo set-option window lsp_auto_hover_cache_key \"%val{timestamp} %val{selections_desc}\"
+                    echo '"$hover"'
+                fi
+            }
+        }'
     }
 }

so lsp-did-change always calls its callback, intentionally.
semantic tokens and inlay hints should be fixed by 9621c99d8ad358af806d4d88f43cd1e50bec87c7, thanks for reporting
I checked the other requests; most of them do not send a lot of data so it's
probably not worth caching them. Also most requests depend on the cursor
position, so caching is less effective.
Also depends on which behavior we want for auto-hover, do we want the hover info to pop back up after pressing escape etc..  I don't really use it.
So I think we're fine now.


CSS: Convert rem to px values

Overview
This changes the scss build step to Postcss with the inclusion of a Postcss plugin which converts all rem units to px units.
The reason for this change is to enable the development of a webcomponent.
In a webcomponent we have no access to the font-size of the html element. All rem units are based on the font-size of the html element which causes incorrect sizing of the interface.
Converting these rem units to px units fixes this issue.
How to test/reproduce
Build should have no visual regressions. In the compiled styles all rems are now px.
This PR is necessary to fix the styling in https://github.com/lblod/poc-say-webcomponent/pull/1

@Dietr this fix is to resolve the issues with the icons being too large in the toolbar etc. right?
It seems to me that rem values do work in web-components: when changing the font-size of the root element, the webcomponents also scale correctly.
Applications with the appuniversum css have a default root element font-size of 62.5%, while an application with the editor webcomponent has a default font-size of 100% => the icons in the toolbar are enlarged.

No, this is not related to the icons. It's related to the font-sizes and spacing. As appuniversum is using rem with a html default of 62.5% the webcomponent renders too big when the html is set to another font-size. Converting all units to px makes the webcomponent's design more consistent. You can still change font-sizes with the css custom properties provided.
To me this seems to be the only way to have a default design on the webcomponent without forcing the user to set a font-size of 62.5% on their html element.

No, this is not related to the icons. It's related to the font-sizes and spacing. As appuniversum is using rem with a html default of 62.5% the webcomponent renders too big when the html is set to another font-size. Converting all units to px makes the webcomponent's design more consistent. You can still change font-sizes with the css custom properties provided.
To me this seems to be the only way to have a default design on the webcomponent without forcing the user to set a font-size of 62.5% on their html element.

Ok, seems fair to me!


cannot diff a schema with a database connection

Frustrated by not being able use versioned migrations approach (see #2430 ) I'm trying the declarative approach. I want to take my schema from Ent.
variable "database_url" {
  type = string
  default = "postgres://postgres:postgres@localhost:55435/postgres?sslmode=disable"
}

env "ent" {
  url = var.database_url
  dev = "docker://postgres/14/dev?search_path=public"
}

schema/user.go
// Package schema defines the Ent schema.
package schema

import (
	"entgo.io/ent"
	"entgo.io/ent/dialect"
	"entgo.io/ent/schema/field"
)

// User holds the schema definition for the User entity.
type User struct {
	ent.Schema
}

// Fields of the User.
func (User) Fields() []ent.Field {
	return []ent.Field{
		field.String("foo"),
		field.String("bar"),
		field.String("id").SchemaType(map[string]string{
			dialect.Postgres: "varchar(21)",
		}),
	}
}

// Edges of the User.
func (User) Edges() []ent.Edge {
	return nil
}

When I run atlas schema apply --env=ent --to=ent://schema it gives me:
Error: cannot diff a schema with a database connection: "" <> "public"

This is a confusing error, and no way to run it in verbose mode or anything. I'm GUESSING that it has to do with "ent" not providing a postgres schema and somehow "public" is not the default.  I've tried adding the "--schema" but that doesn't help.
MAYBE it's about the search_path n the dev-url, but removing that gives me:  Error: missing schema in --dev-url. See: https://atlasgo.io/url

Thanks for reporting this. We'll give this a look, it might be an issue with the ent/schema loader.

I've solved this for my use case by annotating the User entity with an explicit schema:
func (User) Annotations() []schema.Annotation {
	return []schema.Annotation{
		entsql.Schema("public"),
	}
}

from a usability perspective it would probably still make sense if:

A: make the error more clear to indicate which side of the diff fails to provide a schema and why
B: For postgres maybe make the "public" schema the default from Ent's side


What is mean of the search guard entry？

Who can tell me about the mean of this entry?
What is the mean about the "cn","ou", "dc" ????
searchguard.authcz.admin_dn:

cn=admin,ou=Test,ou=ou,dc=company,dc=com

Pls. ask such kind of questions in the google group https://groups.google.com/forum/#!forum/search-guard . Github issues are reserved for bugs and feature requests. Thanks.


Card title not displayed correctly in Safari

Title is behind the shadow of the card.

@marcreichel I would like to work on this issue and also would like some guidance on how to make it more visible

@mohak34 I'm not quiet sure yet how to fix this. This issue would be more or less just try and error to see what would fix it I guess.


DeepVelo and GPUs

Hi,
I'm interested in understanding whether it's feasible to run deepvelo on multiple GPUs (instead of just a single GPU).
My particular questions are the following:

Does deepvelo support parallelization and does it can take advantage of multiple GPUs?
If so, does it significantly speed up the analysis?

Best,
Miona Rankovic

Hi Miona,
Currently, we don't utilize mini-batch training, due to constraints involving the model. We have to modify the model to incorporate mini-batch training on the DeepVelo graph, and this is something that is in the to-do's and we'll create a PR for this. Without this, we can't necessarily incorporate the DataParallel or DDP torch classes (https://pytorch.org/docs/stable/generated/torch.nn.parallel.DistributedDataParallel.html).
But even with one GPU, we found that DeepVelo runs in under a minute for >10000 cells, so it will scale quite well to larger datasets.
We did not explicitly perform experiments comparing one vs multi-gpu usage, but I'm confident this will lower the training time when enabled, just not sure to what extent/scaling factor.
In the meantime, please let us know if you find any limitations in training the model for your data.

PR for mini-batch/multi-gpu training opened here:  minibatch_Training


Plugin for adding dimensions from viewbox

This is a simple plugin that allows you to add height and width dimensions based off the viewbox. It was made to deal with browser issues with rendering SVGs by default with 0x0 dimensions. This can be solved alternatively by giving the SVG a height / width through CSS. Setting a width/height in the SVG may not be desired.
Definitely open to thoughts and improvements! This was based off other plugins, removeDimensions and addAttributesToSVGElement.

it's fun because I'm doing the opposite, removing width and height when there's a viewBox. Width/height are really not necessary, like for jpeg/png, it's stylable in css

@caub Totally width/height are not typically necessary. In my case, it allowed for just updating assets instead of deploying a new version of the app I'm working on.  Had I not been time constrained, I would have used removeDimensions ;-)

@nym for doing something simple like this, I would do that:
https://github.com/caub/svgo/blob/browser/test/browser.js#L38-L45
I've added .js2svg in lib/svgo.js then I can modify the result tree, remove attributes on svg, then stringify again
result: <svg xmlns="http://www.w3.org/2000/svg" width="120" height="120" viewBox="0 0 120.45 120"><sty...
result2: <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 120.45 120"><sty...

and result2 is 25 chars shorter as expected

Thanks for commitment! However, this plugin has nothing to do with optimizing and thus I cannot accept it.
Unfortunately, SVGO is already somewhat bloated with such plugins, so it needs to focus on its main purpose to keep things as simple as possible. Feel free to use it by yourself, the plugin system was done to be easy to extend like this. Best wishes!


Fullscreen to Window (Resizable bug)

Hi, good game ! thanks you
I have some(s) ideas/bugs :

Window size 1920*1080 is very high, maybe you can open in 640x480 for older machine or more can choose in setting menu
Bug : we have possibility to resize Windows (flag SDL_WINDOW_RESIZABLE) but render doesnt resize correctly :

menu setting at startup ?
save game settings ?

Thanks you very much.

I'm glad that you like it!

Window size 1920*1080 is very high, maybe you can open in 640x480 for older machine or more can choose in setting menu.

Game chooses the desktop resolution while fullscreen. Actually, HUD covering a big space is bigger problem on very low resolutions. It should be fine for most of the users nowadays.

Bug : we have possibility to resize Windows (flag SDL_WINDOW_RESIZABLE) but render doesnt resize correctly.

Yes, I'm aware of this. Code of GUI is very primitive. It should dynamically modify the size and position.
But I have currently no plans to fix this though, maybe in the future.


Error 80160000 - Import Solution Failed: Object reference not set to an instance of an object

Hi, I'm trying to import into a Production environment and have set the Power Apps component framework for canvas apps toggle to on. I am a system administrator role in the environment.
I get a failure:
Solution "Power Drag Drop" failed to import: Import Solution Failed: Object reference not set to an instance of an object.
Anything else I should check? I've tried all the managed solutions.
Cheers

This appears to be caused by a recent update to the Power Apps platform which hopefully should be fixed soon.
Which region are you trying to import into?

Ahh ok -  I'm in the UK.
I did wonder as I tried a few other things and didn't get the same error

The issue is affecting the creator kit as well:
https://github.com/microsoft/powercat-creator-kit/issues/214
https://github.com/microsoft/powercat-code-components/issues/152
I've managed to import into Europe and Canada ok - so I think a fix is rolling out.

This appears to be caused by a recent update to the Power Apps platform which hopefully should be fixed soon. Which region are you trying to import into?

Just FYI for others that we are having same issue in Australia region too.
hope it gets fixed soon.
Thanks @scottdurow for your valuable contribution, it indeed is looking promising.

Working in UK again :)

I am going to close this now as it seems the issue is fixed. If anyone has the problem again, please add a comment!

Hi Scott, unfortunately I have that same issue in Germany. I can't import the last release (power-drag-drop-1.0.27).


more flags to validate specific aspects of a corpus (eg for a specific process)

Is your feature request related to a problem? Please describe.
Currently, child-project validate corpus/ will do a series of checks that are perfect when setting up a corpus; and we have one flag that adapts the behavior of validate to the context of importing annotations by ignoring the absence of recordings.
We have other procedures that also require certain things of the corpus, which it would be great to validate to this end first:

metrics extraction procedures require annotations to be complete & local
audio processors require the raw recordings to be local
sampler requires recordings.csv and recordings of a certain profile to be local
.eaf builder requires just metadata

Describe the solution you'd like
I don't think we should create a flag for each process, but we could create flags that perform only subsections of validate. For instance child-project validate corpus/ --annotations vtc/converted/ can check that those annotations are complete & local; child-project validate corpus/ --audio raw/ converted/standard/.
I wonder whether this is not equivalent to the user doing datalad get  annotations/vtc/converted/ -- I could imagine it is not, because such a command will not alert a user that a given child or recording doesn't have its corresponding vtc/converted annotation.

On it(#284)

Done! The new implementation is described here:
https://childproject.readthedocs.io/en/latest/tools.html#data-validation


[Feature] Upload to Godot AssetLib

It would be nice to have this plugin listed in Godot AssetLib as currently there are no proper plugins for Discord RPC for Godot 3 in the AssetLib.

Submitted the asset for 3.x, waiting for approval

https://godotengine.org/asset-library/asset/1950

Nice!


voice assistant feature

Hello, this repo was an inspiration for a voice-based PWA version located here: chat-gpt-voice.w3spaces.com
I've added a link to vanilla gpt version if it's desired when opening a new "tab" (uses iframes)
I welcome any input or feedback
I believe what I am missing is the 2 previous prompt/result histories for the messages array. I assume this makes the results better?

it is a great idea to add voiced-based features for chatGPT. Glad to see you also use vanilla js.
however, your page seems is kind of PoC not really user friendly. First time I input a wrong API KEY, but not way out.
maybe you may consider fork vanilla-chatgpt, and enhance it with additional function chat.voice(...).
best regards,

@casualwriter Yes very good point. Maybe I should add button to clear the localStorage as it has been on my mind. Today I added a feature that allows user to push html/js code generated by chat-gpt (when Code checkbox is selected) to github gh-pages branch using rest api and it also lets you edit code with contenteditable but it would be nice to have syntax colors as well like with jsbin.

that's awesome. I will add it in the script this week. thanks,

Here is a method that will also speak on the area that is double-clicked starting at the word that was double-clicked. If the word is contained in a list or paragraph then it will also speak the next sibling elements:
const speaker = (lang = 'en-AU', rate = 1.2) => {
        let ss = speechSynthesis, speaking = false;
        var utter = new SpeechSynthesisUtterance();
        document.body.addEventListener('click', e => {
            let t = e.target;
            if (t.tagName === "TEXTAREA") return;
            if (e.detail === 2) {
                ss.cancel();
                const sel = window.getSelection();
                let offset = sel.baseOffset || sel.anchorOffset;
                let node = sel.extentNode || sel.focusNode;
                let arr = [node.textContent.substr(offset)]
                node = node.parentNode;
                if (node.tagName === "P" || node.tagName === "LI") {
                    while (node = node.nextSibling) {
                        if (node.nodeType === 3) continue;
                        arr.push(node.textContent);
                    }
                }
                utter.text = arr.join('');
                utter.lang = lang; utter.rate = rate;
                ss.speak(utter);
                speaking = true;
            }
            else if (speaking) {
                ss.pause();
                speaking = false;
            }
            else if (ss.paused) {
                ss.resume();
                speaking = true;
            }
        });
    }
    speaker();

I saw that sometimes there is a text node followed by paragraph elements and I did not handle that case

thanks Jay. I just update the program with voice feature, and add you in the credit list.
ps: hope you do not mind I use my code for this feature. the project aim to explorer minimal workable solution.  I am not really know the advanced usage of voice API.

@casualwriter I have just learned this built-in speech API as well along with whisper endpoint in OpenAI.
I also added HTML/JS code editing today for the 'Code' checkbox in voice-gpt.
May be an interesting addition to your module as well.
I used chat-gpt to help me explore what options are available for code editing after realizing it is very difficult to write one from scratch.
The best ones I found are Ace, CodeMirror, and Monaco and the output of how to use each is in gh-pages branch of chat-gpt-voice. I had trouble using Ace without absolute positioning although it is said to be the most lightweight so I chose to go with CodeMirror. Monaco is the engine used in VS code but apparently it works in the browser as well.

@casualwriter yes, better to keep some things separate. I think it's not a bad addition in my case but I may look to lazy load the JS for code mirror until it is requested to keep payload smaller.

hi, jay, many thanks for your contribution on the feature of voice recognition.
going to close the issue, have a nice day,