Credentials in request-body as fallback per 2.3.1 of RFC

Section 2.3.1 of the RFC says:

Including the client credentials in the request-body using the two
parameters is NOT RECOMMENDED and SHOULD be limited to clients unable
to directly utilize the HTTP Basic authentication scheme (or other
password-based HTTP authentication schemes).  The parameters can only
be transmitted in the request-body and MUST NOT be included in the
request URI.

When using a REST client for mac that support oauth, I realized it was sending client_id and client_secret in the request body rather than via the Authorization header. While it's definitely not recommended for clients to use, does it make sense to have this included as part of the library as a fallback to support that section of the RFC? I'm not sure what the limitation is of sending an Authorization header, but wanted to discuss here before submitting a PR.
If it's something you think would be useful to have as a fallback I have a PR ready to submit with all the tests.

I think that it is easier to filter out the Authorization http header in logs instead of filtering specific values in bodies like client_id or client_secret. Otherwise I do not know why this method is not recommended.
I realize that some libraries implement faulty behaviour which is why I wrote this library here. I'd rather not include a fallback as I'm trying to follow all the good practices the IETF recommends.
Let me know if you disagree, I'm open for discussion! :)

I was actually thinking the same thing -- and not really sure why a client wouldn't be able to send an Authorization header. More of an inconvenience with the rest client I'm using, but I agree this library is stricter than most (which is what I like about it!) and should avoid everything but best practices. Thanks for chiming in so quickly!

Sure, thanks for your thoughts, feedback and contributions! I appreciate it!


Forbid transfering currency

I want to only swap currencies rather than to transfer currencies. How to manage such authority?

Not sure what you are asking, but assuming I understand you correctly: To my knowledge, the only way to do this is to create a proxy contract (which holds all assets) and has an endpoint for swapping that can be allowed to be called by a third address.
Regardless, this is not the right place to ask, as it has nothing to do with the library itself. Closing.

Not sure what you are asking, but assuming I understand you correctly: To my knowledge, the only way to do this is to create a proxy contract (which holds all assets) and has an endpoint for swapping that can be allowed to be called by a third address.
Regardless, this is not the right place to ask, as it has nothing to do with the library itself. Closing.

Hah, forgive me while I am new to ETH.
I mean my friend wants me to make transactions through uniswap, but he never wants me to transfer the assets away and run away, hah hah. Like api authority management in centralized exchanges, he wants me to use a limited UNISWAP api. Or limit the target  addresses to transfer assets?

As I said, you can't do that easily (the approach I mentioned would work, although if you're new to ETH it'll be a really bad idea).
Ask somewhere else, this is not a general support forum for ETH. (and please read: http://www.catb.org/~esr/faqs/smart-questions.html)


Use a whitelist for the fields returned by the Publishing API

This is more futureproof, rather than using a blacklist. This is a
problem currently, as the Publishing API returns links correctly, but
the code does not handle this.

:+1:


Remove unused parameters related to reactivity coefficients

What is the change?
 They are not used anywhere. I have run our internal integration tests and found no issues.
Why is the change being made?
These parameters are unused bloat. They should have been removed a long time ago. Many of them are ambiguous. Having these around makes working with parameters programmatically difficult.
Side note: These parameters would more properly belong in a plugin, since they are very specific and not used by multiple plugins.

Checklist

[x] This PR has only one purpose or idea.
[x] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any important changes.
[x] The documentation is still up-to-date in the doc folder.
[x] No requirements were altered.
[x] The dependencies are still up-to-date in pyproject.toml.

@john-science We should talk about this in person tomorrow, if you have some time.

Chris and I have reviewed this.
This PR is tacitly approved, based on some testing Chris is running.
 This consisted of running our integration tests and also a benchmark case that exercises this part of the code.

Test coverage only went down because we removed lines of code.
That is exceptable.

Thanks @john-science . I can coordinate the merge, since there are a few internal repos that will be merged simultaneously.

It was pointed out to me by @mgjarrett that I shouldn't remove rxFuelAxialExpansionPerPercent because it is planned to be used very soon by an internal plugin. Does it belong in that plugin instead of the framework? 😏


[L5.2.45] Laravel throttle with load balancers

Laravel Version: 5.2.45
PHP Version: 7
Database Driver & Version:
mysql/redis

Last weekend I have setup a load-balancer on forge with 2 api servers below it. I have a rate limit (throttle) active. since the load balancer is added really weird thing are happening. the "X-RateLimit-Remaining" header is decreased constantly even when there is not even a call to the service.
I have a special forge server spinned up that handles the cache which is currently based on redis.
I'm desperate and hoping to get some answers here.
In the last 10 minutes I have made 4 API calls and the rate limit goes from 1000 to 531.
When decoupling 1 server from the rate limiter ( and add it to the hosts file so I can call it) the same code base works fine. so the limit decreases 1 by 1 by 1.
PS. I post this here since I don't get any feedback on forums.

I had an issue with the throttle functionality before since I defined it in the Kernel.php file and on routes which caused limited counted twice. At the moment the throttle middleware is only called on the Route::group
Route::group( [ 'prefix' => 'v1.2', 'middleware' => [ 'throttle:'1000,60' ] ], function() {
// Routes
});

Sounds like trusted proxies are not set up correctly. :)

@GrahamCampbell can you explain how trusted proxies affects this particular middleware?

Because the IP address of your load ballencer is getting used instead of the customer.

i'm currently monitoring with this package installed: https://github.com/fideloper/TrustedProxy.
will let you know the outcome.

I use this stuff behind a load ballancer and it works fine.

Interesting, OK. Thanks @GrahamCampbell :)


Fixed |safe_email filter to return safe and escaped UTF-8 HTML [#3072]

Backported from 1.7:
https://github.com/getgrav/grav/commit/068de42e833af8baeb96fe31c170964fa2df0d4b

In 3 days nobody is able to update Grav 1.6 anymore (our updates only support the latest version). This means that Grav 1.6 will be an unsupported version because of the upgrade logic does not allow it to be supported.

In 3 days nobody is able to update Grav 1.6 anymore (our updates only support the latest version). This means that Grav 1.6 will be an unsupported version because of the upgrade logic does not allow it to be supported.


ci: update error reference docs

This PR is auto-generated by a nightly GitHub action to update the error reference docs from source code in withastro/astro.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
reference/error-reference.mdx
Source changed, localizations will be marked as outdated.

en
reference/errors/env-invalid-variable.mdx
Source added, will be tracked.

en
reference/errors/env-invalid-variables.mdx
Source added, will be tracked.

en
reference/errors/env-unsupported-get-secret.mdx
Source added, will be tracked.

en
reference/errors/invalid-rewrite404.mdx
Source added, will be tracked.

en
reference/errors/rewrite-encountered-an-error.mdx
Source added, will be tracked.

en
reference/errors/server-only-module.mdx
Source added, will be tracked.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Add Java 11/17/21 matrix for build, test and integration checks

Description
Add Java 11/17/21 matrix for build, test and integration checks
Issues Resolved
Part of https://github.com/opensearch-project/OpenSearch/issues/10334
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Depends on https://github.com/opensearch-project/common-utils/pull/560


[FLINK-13434][e2e] Add stop-with-savepoint end-to-end tests.

What is the purpose of the change
The purpose of this PR is to add some end-to-end tests for the newly introduced stop-with-savepoint command.
Instead of creating a new test (in bash), this PR modifies already existing tests to use the new feature. In the future, we should include an end-to-end test with a more complicated job that does some state manipulation at the notifyCheckpointComplete, as the exactly-once sinks do. This will actually test the new feature.
Brief change log
This PR modifies the test_stateful_stream_job_upgrad and the test_resume_savepoint, without changing their logic.
Verifying this change
This change is already covered by existing tests, which are the ones modified.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

R @azagrebin

Merged.


Fix empty marker issues

Updated version of #1230 (I didn't realize rebasing would make it so I couldn't re-open the PR)
This PR addresses a family of related problems we were seeing when depending on particular packages.
Based on what I saw in the codebase, there is a concept of an "empty" marker. However there were a few places in the code that didn't handle these empty markers properly.
Here's an example pyproject.toml where we were hitting the issue:
[tool.poetry]
name = "example_project"
version = "1.0.0"
description = ""
authors = [""]

[tool.poetry.dependencies]
python = ">=2.7, != 3.0.*, != 3.1.*, != 3.2.*, != 3.3.*, != 3.4.*"

[tool.poetry.dev-dependencies]
pytest = "^3.0.6"
pylama = "^7.6"
pylint = { version = "^2.2", python = ">= 3.5" }
pytest-cov = "^2.6"
black = { version = "18.9b0", python = ">= 3.6" }
sphinx = { version = "^2.0", python = ">= 3.5" }

[build-system]
requires = ["poetry>=0.12"]
build-backend = "poetry.masonry.api"

Doing poetry install once to generate the poetry.lock and then running poetry install again will reproduce the issue:
$ poetry install
Installing dependencies from lock file

[InvalidRequirement]
Invalid requirement, parse error at "'; <empty'"

install [--no-dev] [--dry-run] [-E|--extras EXTRAS] [--develop DEVELOP]

Also I believe this fixes #978 which seems to be hitting a similar problem.
Pull Request Check List
This is just a reminder about the most common mistakes. Please make sure that you tick all appropriate boxes.  But please read our contribution guide at least once, it will save you unnecessary review cycles!

[x] Added tests for changed code.
[ ] Updated documentation for changed code.

Note: If your Pull Request introduces a new feature or changes the current behavior, it should be based
on the develop branch. If it's a bug fix or only a documentation update, it should be based on the master branch.
If you have any questions to any of the points above, just submit and ask!  This checklist is here to help you, not to deter you from contributing!

@Nitori- While the issue is still present on the 0.12.17 release, I can't reproduce the issue with the 1.0.0b2 release. Could you test and see if that fixes the issue for you as well.

Yes the issue appears to be gone with 1.0.0b2, thanks @sdispater !


[interceptors][README.md] adds explanation from issue as seed

Found this issue quite insightful as an overview. No sweat if not merged.

THANK YOU @loganpowell - this is indeed a good start. I'm very much aware that many packages are still in serious need of proper documentation and I'm adding bits whenever I can. But this is a great help (and I completely forgot about this issue/comment).

Glad to be able to scrub in


[RESTEASY-1798]

This brings the changes from #1557 into master (Fixes in response to comments on pull request #1539).

@asoldano, Lollll. I kept thinking, "When did I do this? I don't remember doing this. Am I becoming demented?" Thanks!! ;-)

:-)


Fix multiple windows when a window is spawned later

If multiple windows are spawned at Startup, they all work correctly. If a window is spawned later in Update, they do not work. Input works, but rendering does not.
I'm not sure if this is the right solution to this problem, as I don't understand how the new RenderEntity works. At the time Window is added, RenderEntity does not yet exist on the entity. However, at the time EguiContext is added, it does exist.

I'm encountering the issue as well, I use the F1 key to toggle a debug window in my game and it's empty after updating to bevy 0.15.


InfluenceOnTrafficRouting feature in PCF

Hi,
In the PCF code as part of pcf_policyAuthorization.go:PostAppSessionsContext() I have seen the code for the trafficRouting. I wanted to understand the correct configuration to enable the traffic routing in PCF, I tried by changing the below configurations in the default pcfcfg.conf but this does not seem to work
- serviceName: npcf-smpolicycontrol
suppFeat: 1
- serviceName: npcf-policyauthorization
suppFeat: 1
When a request to sent to the PCF N5 I/f I can see an error in the PCF logs that traffic routing is not enabled:
ERRO[2020-04-21T18:51:22+05:30]/src/pcf/pcf_producer/pcf_policyauthorization.go:1571 free5gc/src/pcf/pcf_producer.sendProblemDetail() Traffic routing not supported                 PCF=PolicyAuthorization
Could you let me what is the correct configuration to enable TrafficRouting in PCF. Also please let me know if there needs to be any additional configurations to be done in the other NF's.

Hi,
We didn't implement this feature yet.

Thanks for the answer. Could you please let me know the roadmap plan on when this feature is planned to be made available.

Hi, any update on this procedure?

Hi, @grzegorzpnk,
The AF on influence routing feature has been implemented, we will release it in the near future.
BRs,
Linpoyi

Hi @linpoyi, Could you tell me which date you will release AF on influence routing

Hi, @grzegorzpnk,  The AF on influence routing feature has been implemented, we will release it in the near future. BRs, Linpoyi
Hi @linpoyi , Could you please let me know the date you will release these features?
Thanks.

Hi @ntnguyencse ,
NEF is scheduled to be released in v3.3.2, which is 2 versions from now (v3.3.0). I'll say it probably takes 3 months or more.
BRs,
Linpoyi

嗨，NEF 计划在 v3.3.2 中发布，从现在开始是 2 个版本 （v3.3.0）。我会说这可能需要 3 个月或更长时间。 BRs， 临波伊

您好，想咨询一下，NEF网元发布了吗？

Hi @xiongkai0920,
NEF is not out yet. It will be released with the R17 update. Probably be later this year.
BRs,
Linpoyi


Add some xgboost internals access

I'd like to get more informations for the trained xgboost model (Booster) like:
Map<String,Integer> | getFeatureScore(String[] featureNames) Get importance of each feature with specified feature names.
Could you add some Api to access the booster directly, or delegate some more methods?

Hi--thanks for your suggestion!
This is actually already possible using something called "transformer views", which allow you to look at the prepared/trained transformer (in this case, the trained XGBoost model, expressed as an XGBoostClassification.Prepared object, which has a public getBooster()) method for this purpose.  However, we haven't written documentation for views yet (other than Javadoc), and this is certainly non-obvious without an example.  I should be able to add one to the project and provide it to you by the end of the day--I'll add another comment here when it's ready.

Hi--thanks for your suggestion!
This is actually already possible using something called "transformer views", which allow you to look at the prepared/trained transformer (in this case, the trained XGBoost model, expressed as an XGBoostClassification.Prepared object, which has a public getBooster()) method for this purpose.  However, we haven't written documentation for views yet (other than Javadoc), and this is certainly non-obvious without an example.  I should be able to add one to the project and provide it to you by the end of the day--I'll add another comment here when it's ready.

oh, thanks, I think I got it, it really was not so easy:
PreparedTransformerView<com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> preparedTransformerView = new PreparedTransformerView<>(xgboost);
DAG1x1<P, com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> dag1 = DAG.withPlaceholder(p).withOutput(preparedTransformerView);
Result<P, com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> res1 =dag1.prepareAndApply(testData);
Iterator<com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> c1 = res1.iterator();
..

right?
Btw I also tested some dl4j model, but stumbled upon some jvm crashes there

oh, thanks, I think I got it, it really was not so easy:
PreparedTransformerView<com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> preparedTransformerView = new PreparedTransformerView<>(xgboost);
DAG1x1<P, com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> dag1 = DAG.withPlaceholder(p).withOutput(preparedTransformerView);
Result<P, com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> res1 =dag1.prepareAndApply(testData);
Iterator<com.linkedin.dagli.xgboost.XGBoostClassification.Prepared<?>> c1 = res1.iterator();
..

right?
Btw I also tested some dl4j model, but stumbled upon some jvm crashes there

Yes, that's indeed correct (you'll just need to pull the transformer from the iterator and call getBooster()).  We often try to make this a bit easier by adding convenience methods (e.g. an asBooster() method) that allow clients to avoid having to create the view themselves; I'll go ahead and add such a method (again, please expect it by EOD).
FYI, the result of a view is always a constant; this means you can get the prepared transformer in a slightly more clean way by calling dag1..prepare(...).getConstantResult().
Could you please share more regarding the JVM crashes with DL4J?

Yes, that's indeed correct (you'll just need to pull the transformer from the iterator and call getBooster()).  We often try to make this a bit easier by adding convenience methods (e.g. an asBooster() method) that allow clients to avoid having to create the view themselves; I'll go ahead and add such a method (again, please expect it by EOD).
FYI, the result of a view is always a constant; this means you can get the prepared transformer in a slightly more clean way by calling dag1..prepare(...).getConstantResult().
Could you please share more regarding the JVM crashes with DL4J?

FYI the example code has been added as SimpleTransformerViewExample.  I've also added a convenience method XGBoostClassification::asBooster() that creates the view for you, although this obviously won't make it into the repository JARs until we push a new version (since you already have written the "manual" solution I'm guessing this isn't a priority for you :) ).
With respect to the first DL4J error, I'll try to reproduce with the CharLSTM model in a similar Linux/CUDA context (this might take a while).  DeepLearning4J is beta and we (Dagli) are using their data reader API in an "advanced" way; the exception could be due to a bug in DL4J (which we could possibly work around, as we have other bugs), or (less likely, given that we've not seen this problem before) it could be due to some subtle flaw in our DL4J integration.
With respect to the second error, this looks like it's probably a bug in the ND4J C++ internals which could be difficult to work around.  It's possible that it's exacerbated by Dagli's creation of multidimensional arrays in multiple threads or the generally high number of threads in the process.  Would you mind trying the (non-multithreaded) SimpleDAGExecutor?  Usage would be like dag.withExecutor(new SimpleDAGExecutor()).prepare(...).
Thanks for both reports!

FYI the example code has been added as SimpleTransformerViewExample.  I've also added a convenience method XGBoostClassification::asBooster() that creates the view for you, although this obviously won't make it into the repository JARs until we push a new version (since you already have written the "manual" solution I'm guessing this isn't a priority for you :) ).
With respect to the first DL4J error, I'll try to reproduce with the CharLSTM model in a similar Linux/CUDA context (this might take a while).  DeepLearning4J is beta and we (Dagli) are using their data reader API in an "advanced" way; the exception could be due to a bug in DL4J (which we could possibly work around, as we have other bugs), or (less likely, given that we've not seen this problem before) it could be due to some subtle flaw in our DL4J integration.
With respect to the second error, this looks like it's probably a bug in the ND4J C++ internals which could be difficult to work around.  It's possible that it's exacerbated by Dagli's creation of multidimensional arrays in multiple threads or the generally high number of threads in the process.  Would you mind trying the (non-multithreaded) SimpleDAGExecutor?  Usage would be like dag.withExecutor(new SimpleDAGExecutor()).prepare(...).
Thanks for both reports!

I gave SimpleDAGExecutor a short try with one epoch, and it did not crash after training this time. I now try a longer run.
Thanks for your work on this great framework, it really improves the experience in this context for java. I hope java community will pick it up, and more "state of the art" special layers are added (to dl4j).

I gave SimpleDAGExecutor a short try with one epoch, and it did not crash after training this time. I now try a longer run.
Thanks for your work on this great framework, it really improves the experience in this context for java. I hope java community will pick it up, and more "state of the art" special layers are added (to dl4j).

Thanks for checking!  This suggests that the problem is indeed likely within ND4J; if it were strictly due to MD arrays being created in multiple threads we could work around this, but from other people's reports it sounds like the mere presence of other threads in the process is more likely; if we're able to reproduce we can investigate further, but hopefully the DL4J team will have this fixed in their next release :)
If there are any layer types you're interested in that are in DL4J but not a part of Dagli's abstraction, please let us know--it would be great to get community feedback to help guide us here.

Thanks for checking!  This suggests that the problem is indeed likely within ND4J; if it were strictly due to MD arrays being created in multiple threads we could work around this, but from other people's reports it sounds like the mere presence of other threads in the process is more likely; if we're able to reproduce we can investigate further, but hopefully the DL4J team will have this fixed in their next release :)
If there are any layer types you're interested in that are in DL4J but not a part of Dagli's abstraction, please let us know--it would be great to get community feedback to help guide us here.

Thanks for investigating. And thanks for adding the getBooster-method.
Just for understanding: would it be also possible to get the XGBoostClassification.Prepared from the dag via the "producers(java.lang.Class producerClass)" method (like dag.producers(XGBoostClassification.Prepared.class).findAny().get().peek())?

Thanks for investigating. And thanks for adding the getBooster-method.
Just for understanding: would it be also possible to get the XGBoostClassification.Prepared from the dag via the "producers(java.lang.Class producerClass)" method (like dag.producers(XGBoostClassification.Prepared.class).findAny().get().peek())?

This would work, but is not recommended except for debugging.  In this case it's probably safe because it's unlikely an XGBoostClassification.Prepared instance is going to be optimized away in any real-world problem, but in general it's not a good idea to depend on assumptions about the specific nodes comprising a DAG unless you disable graph optimizations (also not recommended :) ).  The reason for this is that Dagli might "optimize away" the node you're looking for or otherwise rewrite the graph; for example, if your XGBoostClassification model had exclusively constant-value inputs in the DAG, it could (and would) be replaced by a constant itself and your code that looks for an XGBoostClassification.Prepared instance in the prepared DAG would break.
Instead, the right way to do this is with views: e.g. use a PreparedTransformerView with your XGBoostClassification as its input, and then make that view an output of your DAG.  The output value will then be the prepared transformer you're looking for (your XGBoostClassification.Prepared instance).

This would work, but is not recommended except for debugging.  In this case it's probably safe because it's unlikely an XGBoostClassification.Prepared instance is going to be optimized away in any real-world problem, but in general it's not a good idea to depend on assumptions about the specific nodes comprising a DAG unless you disable graph optimizations (also not recommended :) ).  The reason for this is that Dagli might "optimize away" the node you're looking for or otherwise rewrite the graph; for example, if your XGBoostClassification model had exclusively constant-value inputs in the DAG, it could (and would) be replaced by a constant itself and your code that looks for an XGBoostClassification.Prepared instance in the prepared DAG would break.
Instead, the right way to do this is with views: e.g. use a PreparedTransformerView with your XGBoostClassification as its input, and then make that view an output of your DAG.  The output value will then be the prepared transformer you're looking for (your XGBoostClassification.Prepared instance).

With respect to the first DL4J error, I'll try to reproduce with the CharLSTM model in a similar Linux/CUDA context (this might take a while). DeepLearning4J is beta and we (Dagli) are using their data reader API in an "advanced" way; the exception could be due to a bug in DL4J (which we could possibly work around, as we have other bugs), or (less likely, given that we've not seen this problem before) it could be due to some subtle flaw in our DL4J integration.

Could you reproduce it in the meantime? I think this post:
https://community.konduit.ai/t/bertiterator-produces-npe-while-training-on-gpu/580
hints to the right direction.
Training on CPU really takes very long (several days), so I would be happy to get it run on the GPU.

With respect to the first DL4J error, I'll try to reproduce with the CharLSTM model in a similar Linux/CUDA context (this might take a while). DeepLearning4J is beta and we (Dagli) are using their data reader API in an "advanced" way; the exception could be due to a bug in DL4J (which we could possibly work around, as we have other bugs), or (less likely, given that we've not seen this problem before) it could be due to some subtle flaw in our DL4J integration.

Could you reproduce it in the meantime? I think this post:
https://community.konduit.ai/t/bertiterator-produces-npe-while-training-on-gpu/580
hints to the right direction.
Training on CPU really takes very long (several days), so I would be happy to get it run on the GPU.

The tricky part is reproducing it in a way that it's easy to debug.  My original plan for reproducing this was inside a Linux VM with CUDA CPU emulation, but apparently this has ceased to be an option.  Since I have local access to a CUDA-capable Windows machine I'll try using this--if the issue is in DL4J's Java layer the bug will likely be cross-platform.
In the meantime, if you're not already using it, you may want to try the AVX variant of the DL4J/ND4J CPU library, which takes advantage of SIMD.  It still won't be nearly as fast as a GPU, unfortunately :(

The tricky part is reproducing it in a way that it's easy to debug.  My original plan for reproducing this was inside a Linux VM with CUDA CPU emulation, but apparently this has ceased to be an option.  Since I have local access to a CUDA-capable Windows machine I'll try using this--if the issue is in DL4J's Java layer the bug will likely be cross-platform.
In the meantime, if you're not already using it, you may want to try the AVX variant of the DL4J/ND4J CPU library, which takes advantage of SIMD.  It still won't be nearly as fast as a GPU, unfortunately :(

Quick update: we've been able to reproduce the issue; it is indeed the same DL4J bug from the post you've linked to.  We also have a workaround that we're going to be testing later today.  I'll update the ticket again once we've published the new version to Maven Central (assuming the tests succeed, I expect this will happen no later than tomorrow).

Quick update: we've been able to reproduce the issue; it is indeed the same DL4J bug from the post you've linked to.  We also have a workaround that we're going to be testing later today.  I'll update the ticket again once we've published the new version to Maven Central (assuming the tests succeed, I expect this will happen no later than tomorrow).

The workaround has been shipped in Dagli 15.0.0-beta6, which should be available from Maven Central in a few hours.  As the original issue (and now this "bonus" issue :) ) have been addressed, I'll close the issue--thanks again for reporting these!

The workaround has been shipped in Dagli 15.0.0-beta6, which should be available from Maven Central in a few hours.  As the original issue (and now this "bonus" issue :) ) have been addressed, I'll close the issue--thanks again for reporting these!


Cluster thread is deleted while running

After the recent shutdown changes of the client, it is now possible that LifecycleService.shutdown is being called from within the cluster thread and this in turn calls ClusterService.shutdown and this call deletes the cluster thread object. The thread join to the calling thread always succeeds without waiting to avoid deadlocks, and this was one of the changes in recent PRs (https://github.com/hazelcast/hazelcast-cpp-client/pull/285), hence even though there is a join call in the ClusterService.shutdown, it actually will not wait since the calling thread was the cluster thread itself. It is really dangerous to delete the current running thread object. Solution: Shutdown the thread but do not delete the cluster thread on shutdown.

Test PASSed.


Global Symbol search has duplicate entries on Windows

See in the image bellow how in some cases I get duplicate results with different file path separators on Windows (/ and \\).
Only some symbol types are duplicated such as classes, interfaces and methods.

@mjbvz this feels like a TS bug or something. In this quick pick, we just call this:
https://github.com/microsoft/vscode/blob/6ed1a4fdf1c83b8cd6813fcbda0b41e0fd01f3f2/src/vs/workbench/contrib/search/common/search.ts#L68
@benibenj can you do us a huge favor and run this command:
https://github.com/microsoft/vscode/blob/6ed1a4fdf1c83b8cd6813fcbda0b41e0fd01f3f2/src/vs/workbench/contrib/search/browser/search.contribution.ts#L391-L396

Install this extension: https://marketplace.visualstudio.com/items?itemName=benmcmorran.vscode-api-explorer
Run const result = await vscode.commands.executeCommand('_executeWorkspaceSymbolProvider', 'buttonwithdropdown') in the workspace where you repro

Take a look at result and tell us if you see the same bug of duplicates.

I'm not able to reproduce this bug currently. I reported this a month ago. If this is a typescript bug, then it's possible they have fixed it in the meantime. I'm fine with closing this for the moment. If I see it happen again, I'll reopen and check what _executeWorkspaceSymbolProvider returns


symbol(s) not found for architecture x86_64

Trying to compile this on a Mac, what could I be doing wrong?  I had to replace -lOpenCL in various Makefiles with -framework OpenCL
The card I am trying to use is the built in Radeon Pro 560.
ifeq ($(BUILD_OPENCL),1)
${CXX} -DBUILD_OPENCL -o clKeyFinder.bin ${CPPSRC} ${INCLUDE} -I${OPENCL_INCLUDE} ${CXXFLAGS} ${LIBS} -L${OPENCL_LIB} -lkeyfinder -laddressutil -lsecp256k1 -lcryptoutil -lsecp256k1 -lCLKeySearchDevice -lclutil -framework OpenCL -llogger -lutil -lcmdparse
g++ -o clunittest.bin cltest.cpp cltest.cpp main.cpp  -I/projects/BitCrack/util  -I/projects/BitCrack/AddressUtil  -I/projects/BitCrack/CmdParse  -I/projects/BitCrack/CryptoUtil  -I/projects/BitCrack/KeyFinderLib  -I/projects/BitCrack/CLKeySearchDevice  -I/projects/BitCrack/CudaKeySearchDevice  -I/projects/BitCrack/cudaMath  -I/projects/BitCrack/clUtil  -I/projects/BitCrack/cudaUtil  -I/projects/BitCrack/secp256k1lib  -I/projects/BitCrack/Logger  -I/projects/BitCrack/embedcl -I/Developer/NVIDIA/CUDA-10.1/include -O2 -std=c++11 -DCL_TARGET_OPENCL_VERSION=110 -L/projects/BitCrack/lib -L/Developer/NVIDIA/CUDA-10.1/lib -lclutil -framework OpenCL
Undefined symbols for architecture x86_64:
"util::toLower(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&)", referenced from:
cl::CLProgram::CLProgram(cl::CLContext&, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator >, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator >) in libclutil.a(clContext.o)
cl::CLProgram::CLProgram(cl::CLContext&, char const*, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator >) in libclutil.a(clContext.o)
ld: symbol(s) not found for architecture x86_64

how did you resolve this?


Change dynamic array parameters to const.

Parameters should be constant at elaboration time. Creating parameter of dynamic array type (which main idea it to be dynamic at runtime) is not clearly defined as legal by LRM.
Please change it to a const array - it should express expected behaviour better.

@googlebot I signed it!

@googlebot I signed it!

@googlebot I consent.


Add predicate pushdown

@samansmink made the updates here. I tried getting rid of the data dir from the commit, but wasn't sure the best way to do that.

Enhance query performance by filtering data at the metadata level, reducing the amount of data read during scans.
Key Changes

Extended IcebergManifestEntry:

Added lower_bounds and upper_bounds maps to store column statistics.

Utility Function:

Implemented IcebergUtils::GetFullPath to resolve file paths accurately.

Metadata Retrieval:

Added GetEntries template in IcebergTable to fetch relevant manifest entries, excluding deleted ones.

Predicate Evaluation:

Created EvaluatePredicateAgainstStatistics to assess if data files satisfy query predicates based on their statistics.
Integrated logging for transparency during evaluation.

Scan Expression Modification:

Updated MakeScanExpression to filter data_file_entries using predicates before scanning.
Logged the number of files before and after filtering for debugging.

Binding Function Enhancements:

Enhanced IcebergScanBindReplace with additional logging and prepared data files based on predicate results.

This implementation optimizes Iceberg table scans by leveraging metadata for early data filtering, significantly improving query efficiency and resource usage.

@Mytherin thanks for the review! I cleaned up the issues here: https://github.com/duckdb/duckdb_iceberg/pull/78
will close this one.

closing, see above.


new(plugins): k8s audit log plugin porting

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area plugins
/area registry
What this PR does / why we need it:
This PR is part of the ongoing effort of porting the current K8S Audit Log implementation bundled in Falco as a falcosecurity plugin. Here, we implement the K8S Audit source and extraction from scratch using the Plugin SDK Go.
Which issue(s) this PR fixes:
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Another library to consider using is https://github.com/jmespath/go-jmespath. I'm not that familiar with it, but it looks like a richer way to query json objects, which would probably provide a lot of flexibility. Don't know anything about performance, though.

Hey @mstemm, I just committed a simple benchmark that stresses the extractor for all the fields (it can be useful in the future). In the benchmark, we pull the sample k8s audit json files used in Falco integration tests (see here: https://github.com/falcosecurity/falco/tree/master/test/trace_files/k8s_audit), and attempt extracting every field from each of json event. From the benchmarks results, I was able to estimate the average time taken to extract a field from an event.
I compared the current fastjson-based implementation with an experimental porting I did with jsonparser.

fastjson: ~327ns
jsonparser: ~2390ns
Although I'm sure I haven't applied special optimizations in the jsonparser porting, I would lean towards keeping fastjson for now.

I think we could postpone the discussion later in the process, perhaps with the intent of providing a Falcosecurity-provided json lib (from the json plugin maybe) made available to plugin developers to hide which library we use underneath.

LGTM label has been added.  Git tree hash: 6229780bccd0604d42e72973428fcb148853afb4

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr, mstemm
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr,mstemm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


What about adding a meteo/weather command?

https://www.wunderground.com/weather/api/ or something similar could provide today's weather, 3 day forecast, etc.  500 API calls a day for free https://www.wunderground.com/weather/api/d/pricing.html

I personally don't like this idea. Let's know what @diegomariani thinks.

Thanks for the interest, but a weather forecast notification service does not make any sense into the university bot


Allow use of haml 5. Fixes #23

When I run the test suite I see that these pending test have fixed themselves (which is counted as a failure)
Failures:

  1) RDF::RDFa::Reader w3c test cases for xhtml1 rdfa1.1 that are required test 0065: @rel with safe CURIE FIXED
     Expected pending 'CDN messes up email addresses' to fail. No error was raised.
     # ./spec/suite_spec.rb:16

  2) RDF::RDFa::Reader w3c test cases for xhtml1 rdfa1.1 that are required test 0176: IRI for @rel and @rev is allowed FIXED
     Expected pending 'CDN messes up email addresses' to fail. No error was raised.
     # ./spec/suite_spec.rb:16

  3) RDF::RDFa::Reader w3c test cases for xhtml5 rdfa1.1 that are required test 0065: @rel with safe CURIE FIXED
     Expected pending 'CDN messes up email addresses' to fail. No error was raised.
     # ./spec/suite_spec.rb:16

  4) RDF::RDFa::Reader w3c test cases for xhtml5 rdfa1.1 that are required test 0176: IRI for @rel and @rev is allowed FIXED
     Expected pending 'CDN messes up email addresses' to fail. No error was raised.
     # ./spec/suite_spec.rb:16

  5) RDF::RDFa::Reader w3c test cases for xhtml1 rdfa1.0 that are required test 0065: @rel with safe CURIE FIXED
     Expected pending 'CDN messes up email addresses' to fail. No error was raised.
     # ./spec/suite_spec.rb:16

Coverage remained the same at 95.455% when pulling 330b2cdb85c30b0011ef301131855e5bb988acbf on jcoyne:haml5 into a3b7d6419250d63bdb6639fc191a720c17999a04 on ruby-rdf:develop.

Isn't it nice when things fix themselves? I think I pushed something to the RDFa repo which caused it to reload.


Errror in Firefox (v32 and v33) in torque visualizations

TrackJS
"Index or size is negative or greater than the allowed amount"

fixed some weeks ago in torque: https://github.com/CartoDB/torque/commit/4d73c03fee89c7c2d45e940451623a93aa9ff093


Make diffs pretty like terraform

Terraform plan creates pretty diffs.  Consider how we can make diffs better

/assign

The diff command has been removed--closing.


create install script

create a script that installs the dependencies and call it from main workflow

fail if something goes wrong
introduce output grouping


fix tomcat-10x-plugin and add test case to support tomcat7.x-8.x-9.x

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[x] Update the CHANGES log.

https://github.com/apache/skywalking-java/pull/286/files#r1119753555

Somehow Spring 6 case fails? Could you recheck?

Somehow Spring 6 case fails? Could you recheck?

The only issue I could find is that we expected POST:/create call returns status code 200, but in the actual data is 201.
But I don't think this PR is relevant since for EntrySpan, the tag is set when stackDepth == currentMaxDepth. This means only the Spring plugin can set tags.
@kezhenxu94 @wu-sheng any idea?

Somehow Spring 6 case fails? Could you recheck?

The only issue I could find is that we expected POST:/create call returns status code 200, but in the actual data is 201.
But I don't think this PR is relevant. Since for EntrySpan, the tag is set when stackDepth == currentMaxDepth. This means only the Spring plugin can set tags.

This is somewhat relevant, I tested Spring 6 and expected the status code to be 201 but it was always 200, I was sure it was not an issue of Spring plugin but didn't dig deeper to the cause, so this PR accidentally fixes the bug too, so, please update the expected data for Spring 6, change status code from 200 to 201.

What do these 201 and 204 mean actually?


ROX-26742: Refactor the csv formatter

Description

Refactor the CSV formatter in Compliance Reporting. This will make the functions more testable.
User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[x] CI results are inspected

Automated testing

[x] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[x] modified existing tests

How I validated my change

[x] New unit tests pass
[x] Generating a report manually and checking the CSV are generated as expected.

/retest


small and straight forward documentation

AWESOME. I was just trying to add it.... But your's is nice


Button Colors

Might want to consider a different colour scheme for the buttons. Red is an alert/danger color and might send the wrong message to users.
Consider not using separate colours on the buttons to avoid confusion. Users don't need to know what's important.
Reply to this email directly, view it on GitHub
https://github.com/shantaram3013/covid19-resource-site/issues/31#issuecomment-830816533,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKSKYUF5VPJ55GHD2S37H6TTLVNHVANCNFSM437MLODA
.

also i'm pretty colorblind so it looked more green/maroon to me than red :clown_face:

Dang, didn't know that. Get someone else to deal with the colours xD
On Sun, 2 May 2021, 20:01 Siddharth Singh, @.***> wrote:

also i'm pretty colorblind so it looked more green/maroon to me than red
🤡
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/shantaram3013/covid19-resource-site/issues/31#issuecomment-830818513,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKSKYUCTRCSI7AUOPBZRX5TTLVO4HANCNFSM437MLODA
.

Fixed in 9059ed20a8b084504b7158851695cd8ec6b1737e.


TAJO-1991: Tablespace::getVolume should take filter predication.

See https://issues.apache.org/jira/browse/TAJO-1991.
This patch depends on TAJO-2007.

I recover the default 1GB to the original value -1. Actually, it was related to many other code parts. It should be resolved in another jira.

I rebased and fixed all tests. It is ready to review.

Rebased.

This patch looks good to me. Test failure seems not related to the changes in this PR.

Rebased. Thanks!

+1 ship it!


npx gqlPrune - 404 not found

Hi as per readme, I am trying to run npx gqlPrune but I get this error
$ npx gqlPrune
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/gqlPrune - Not found
npm ERR! 404
npm ERR! 404  'gqlPrune@*' is not in this registry.
npm ERR! 404 This package name is not valid, because
npm ERR! 404  1. name can no longer contain capital letters
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

For some reason the builder did not include the dist folder... Wokring on it <3

Just noticed the inner error: 1. name can no longer contain capital letters
npx gqlprune seems to be installing..

I tried to publish a new build but i still get the same resutl...
https://www.npmjs.com/package/gqlprune?activeTab=code
Did the package generate the gqlPrune.config.yaml file?
I will try tomorrow and see how the new package works in my project again

@Krister-Johansson package didn't generate the gqlPrune.config.yaml as it didn't proceed with even gqlprune after installing.

thank you! this works well (v1.2.2)


[Model Request] MiniChat-3B

⚙  Request New Models

Link to an existing implementation (e.g. Hugging Face/Github): https://huggingface.co/GeneZC/MiniChat-3B
Is this model architecture supported by MLC-LLM? (the list of supported models) Yes

Additional context
Already done

This is already completed.
Model: https://huggingface.co/acalatrava/mlc-chat-MiniChat-3B-q4f16_1
Libs: https://github.com/mlc-ai/binary-mlc-llm-libs/pull/48


Problem with Pytorch Lightning

Traceback (most recent call last):
File "C:\Users\wdson\Downloads\Compressed\DiffBIR-main\inference.py", line 15, in
from model.cldm import ControlLDM
File "C:\Users\wdson\Downloads\Compressed\DiffBIR-main\model\cldm.py", line 18, in
from ldm.models.diffusion.ddpm import LatentDiffusion
File "C:\Users\wdson\Downloads\Compressed\DiffBIR-main\ldm\models\diffusion\ddpm.py", line 20, in
from pytorch_lightning.utilities.distributed import rank_zero_only
ModuleNotFoundError: No module named 'pytorch_lightning.utilities.distributed'
I am getting this error, even though I have Pytorch Lightning installed. Also, I could not find "requirements.txt" in the repository, so I installed the modules as they were being required.

Haha I commented the same thing and having the same problem.
_https://github.com/XPixelGroup/DiffBIR/issues/1_

I have provided a version (1.4.2) for PyTorch Lightning in install_env.sh. Now I think it's necessary to provide a standardized requirements.txt for users. Thanks for your question! 😄

I wasn't being able to ru install_env.sh because I had copied the full command on the repo, but when I typed just the file name, it ran normally. Sorry for the inconvenience.

Could not find module 'E:\DiffBIR\diffbirenvs\Lib\site-packages\torchvision\image.pyd' (or one of its dependencies). Try using the full path with constructor syntax.
warn(f"Failed to load image Python extension: {e}")
Traceback (most recent call last):
File "E:\DiffBIR\gradio_diffbir.py", line 8, in
import pytorch_lightning as pl
File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning_init_.py", line 20, in
from pytorch_lightning import metrics  # noqa: E402
File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics_init_.py", line 15, in
from pytorch_lightning.metrics.classification import (  # noqa: F401
File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics\classification_init_.py", line 14, in
from pytorch_lightning.metrics.classification.accuracy import Accuracy  # noqa: F401
File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics\classification\accuracy.py", line 18, in
from pytorch_lightning.metrics.utils import deprecated_metrics, void
File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics\utils.py", line 22, in
from torchmetrics.utilities.data import get_num_classes as _get_num_classes
ImportError: cannot import name 'get_num_classes' from 'torchmetrics.utilities.data' (E:\DiffBIR\diffbirenvs\lib\site-packages\torchmetrics\utilities\data.py)，
How can I fix this error?

Could not find module 'E:\DiffBIR\diffbirenvs\Lib\site-packages\torchvision\image.pyd' (or one of its dependencies). Try using the full path with constructor syntax. warn(f"Failed to load image Python extension: {e}") Traceback (most recent call last): File "E:\DiffBIR\gradio_diffbir.py", line 8, in import pytorch_lightning as pl File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning__init__.py", line 20, in from pytorch_lightning import metrics # noqa: E402 File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics__init__.py", line 15, in from pytorch_lightning.metrics.classification import ( # noqa: F401 File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics\classification__init__.py", line 14, in from pytorch_lightning.metrics.classification.accuracy import Accuracy # noqa: F401 File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics\classification\accuracy.py", line 18, in from pytorch_lightning.metrics.utils import deprecated_metrics, void File "E:\DiffBIR\diffbirenvs\lib\site-packages\pytorch_lightning\metrics\utils.py", line 22, in from torchmetrics.utilities.data import get_num_classes as _get_num_classes ImportError: cannot import name 'get_num_classes' from 'torchmetrics.utilities.data' (E:\DiffBIR\diffbirenvs\lib\site-packages\torchmetrics\utilities\data.py)， How can I fix this error?

Try pip install torchmetrics==0.6.0


plugin has old android support dependencies instead of using the new AndroidX equivalent.

Hi,
I'm trying to use this plugin with capacitor and in Android Studio I get this error 'Cannot resolve symbol ActivityCompat' at line:
import android.support.v4.app.ActivityCompat;
In the capacitor documentation says that:
Error: "package android.support.* does not exist"
This error occurs when some Cordova or Capacitor plugin has old android support dependencies instead of using the new AndroidX equivalent. You should report the issue in the plugin repository so the maintainers can update the plugin to use AndroidX dependencies.
I don't know if it's the same error...
But the final question is:
This plugin has old android support dependencies instead of using the new AndroidX equivalent, do you think to upgrade?
Thanks!

Hi,
I did not face that issue with this package, but with others. Try npm install --save-dev jetifier and then npx jetify. (you can add that to your postinstall of your package.json so it will do it with any new dependency you install if needed). This will run in the background for a while as it converts to AndroidX the libraries you use that are not AndroidX ready (it does not show the progress, so give it a time before npx cap syncing / copying back).
Hope this helps!

This plugin has dependencies on cordova-plugin-androidx and cordova-plugin-androidx-adapter which add AndroidX support to Cordova projects and dynamically patch any source code (including this plugin) from the legacy Android Support Library to AndroidX.
This works fine in a Cordova environment (you can test this yourself with the example project) however it may not work in a Capacitor project. However this is a Cordova plugin and Capacitor is not officially supported.

@nereolopez  thank you very much, it works!
@dpa99c I know, I used this plugin since years with Cordova and in these days I'm trying to use Capacitor... Thank you for your job with Cordova!


Does word alignment or phone alignment availbale?

such as word/phone start time, end time, confidence, etc.

It is an on-going work in https://github.com/k2-fsa/icefall/pull/39
All of the three you mentioned are available using icefall.

It is an on-going work in #39
All of the three you mentioned are available using icefall.

That's cool.
I have been using kaldi for a long time. And I'm wondering if it is worth to upgrade to to new generation kaldi.
Currently, my system is using cuda offline decoder + pytorch nnlm to get lattice, and using lattice for alignment (phone, word). So my outputs contain transcripts and word and phone alignments (start_time, end_time, word/phone, confidence) and used for different purpose.
If it is desired to upgrade to the new generation kaldi, I will think about (the first is the most important):
(1) Can I keep my output format, i.e. compatibility, since it has been running on server for a long time.
(2) CER? Will it be lower?
(3) Speed? Will it be faster?
(4) Development cost. Currently I'm using kaldi's c++ interface.
(5) Training pipeline change. The dataset pipeline may need to change.
(6) Any other disadvantage?
Can you give me some answers?

I think it may be too soon to switch for this type of setup.
We haven't really figured out issues around deployment and C++-based
binaries (one of the things we need to do is to move
the FSA logic to C++, for instance, which we'll do in the next couple of
months).
On Fri, Sep 17, 2021 at 5:52 PM jasontu @.***> wrote:

It is an on-going work in #39 https://github.com/k2-fsa/icefall/pull/39
All of the three you mentioned are available using icefall.
That's cool.
I have been using kaldi for a long time. And I'm wondering if it is worth
to upgrade to to new generation kaldi.
Currently, my system is using cuda offline decoder + pytorch nnlm to get
lattice, and using lattice for alignment (phone, word). So my outputs
contain transcripts and word and phone alignments (start_time, end_time,
word/phone, confidence) and used for different purpose.
If it is desired to upgrade to the new generation kaldi, I will think
about (the first is the most important):
(1) Can I keep my output format, i.e. compatibility, since it has been
running on server for a long time.
(2) CER? Will it be lower?
(3) Speed? Will it be faster?
(4) Development cost. Currently I'm using kaldi's c++ interface.
(5) Training pipeline change. The dataset pipeline may need to change.
(6) Any other disadvantage?
Can you give me some answers?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/icefall/issues/48#issuecomment-921666038, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO6TQ3ENQSN5DGOF6CDUCMFV5ANCNFSM5EGDEFHA
.
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.

I think it may be too soon to switch for this type of setup. We haven't really figured out issues around deployment and C++-based binaries (one of the things we need to do is to move the FSA logic to C++, for instance, which we'll do in the next couple of months).
…
On Fri, Sep 17, 2021 at 5:52 PM jasontu @.***> wrote: It is an on-going work in #39 <#39> All of the three you mentioned are available using icefall. That's cool. I have been using kaldi for a long time. And I'm wondering if it is worth to upgrade to to new generation kaldi. Currently, my system is using cuda offline decoder + pytorch nnlm to get lattice, and using lattice for alignment (phone, word). So my outputs contain transcripts and word and phone alignments (start_time, end_time, word/phone, confidence) and used for different purpose. If it is desired to upgrade to the new generation kaldi, I will think about (the first is the most important): (1) Can I keep my output format, i.e. compatibility, since it has been running on server for a long time. (2) CER? Will it be lower? (3) Speed? Will it be faster? (4) Development cost. Currently I'm using kaldi's c++ interface. (5) Training pipeline change. The dataset pipeline may need to change. (6) Any other disadvantage? Can you give me some answers? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#48 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZFLO6TQ3ENQSN5DGOF6CDUCMFV5ANCNFSM5EGDEFHA . 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.

Python or c++ is both acceptable for our server, all my concern are listed above. Functional, cer and speed, etc.

There is no output-format compatibility.
CER might be lower with the right model (assuming you have plenty of
training data, like 1000 hours or more), but that would likely compromise
speed, right now.
On Fri, Sep 17, 2021 at 7:46 PM jasontu @.***> wrote:

I think it may be too soon to switch for this type of setup. We haven't
really figured out issues around deployment and C++-based binaries (one of
the things we need to do is to move the FSA logic to C++, for instance,
which we'll do in the next couple of months).
… <#m_-7986333989238804099_>
On Fri, Sep 17, 2021 at 5:52 PM jasontu @.***> wrote: It is an on-going
work in #39 https://github.com/k2-fsa/icefall/pull/39 <#39
https://github.com/k2-fsa/icefall/pull/39> All of the three you
mentioned are available using icefall. That's cool. I have been using kaldi
for a long time. And I'm wondering if it is worth to upgrade to to new
generation kaldi. Currently, my system is using cuda offline decoder +
pytorch nnlm to get lattice, and using lattice for alignment (phone, word).
So my outputs contain transcripts and word and phone alignments
(start_time, end_time, word/phone, confidence) and used for different
purpose. If it is desired to upgrade to the new generation kaldi, I will
think about (the first is the most important): (1) Can I keep my output
format, i.e. compatibility, since it has been running on server for a long
time. (2) CER? Will it be lower? (3) Speed? Will it be faster? (4)
Development cost. Currently I'm using kaldi's c++ interface. (5) Training
pipeline change. The dataset pipeline may need to change. (6) Any other
disadvantage? Can you give me some answers? — You are receiving this
because you are subscribed to this thread. Reply to this email directly,
view it on GitHub <#48 (comment)
https://github.com/k2-fsa/icefall/issues/48#issuecomment-921666038>, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO6TQ3ENQSN5DGOF6CDUCMFV5ANCNFSM5EGDEFHA
. 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
.
Python or c++ is both acceptable for our server, all my concern are listed
above. Functional, cer and speed, etc.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/icefall/issues/48#issuecomment-921732390, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO3Q6Q5YXZAIVB652DLUCMS7ZANCNFSM5EGDEFHA
.
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.

It is an on-going work in #39
All of the three you mentioned are available using icefall.

I have seen the alignment script. And also have some questions.
Currently in icefall, it seems word alignment or phone alignment can only be got using seperate model, not in one model.
The scripts in conformer_ctc directory seems using data/lang_bpe which can generate word alignment.
The scripts in tdnn_lstm_ctc directory seems using data/lang_phone which can generate phone alignment.
Using kaldi I can generate both phone align and word align from one model.
The best results now seem using bpe token for english and char token for Chinese.
As a kaldi user for several years, I'm confusing will phone be discarded gradually?

We may continue to use phones or syllables for some languages.
We may add examples of using alignments.
On Mon, Nov 15, 2021 at 12:02 AM jasontu @.***> wrote:

It is an on-going work in #39 https://github.com/k2-fsa/icefall/pull/39
All of the three you mentioned are available using icefall.
I have seen the alignment script. And also have some questions.
Currently in icefall, it seems word alignment or phone alignment can only
be got using seperate model, not in one model.
The scripts in conformer_ctc directory seems using data/lang_bpe which can
generate word alignment.
The scripts in tdnn_lstm_ctc directory seems using data/lang_phone which
can generate phone alignment.
Using kaldi I can generate both phone align and word align from one model.
The best results now seem using bpe token for english and char token for
Chinese.
As a kaldi user for several years, I'm confusing will phone be discarded
gradually?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/icefall/issues/48#issuecomment-968318204, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFLO5ZTEZJAN2EB2VYD4LUL7MQ3ANCNFSM5EGDEFHA
.
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.

We may continue to use phones or syllables for some languages. We may add examples of using alignments.
…
On Mon, Nov 15, 2021 at 12:02 AM jasontu @.***> wrote: It is an on-going work in #39 <#39> All of the three you mentioned are available using icefall. I have seen the alignment script. And also have some questions. Currently in icefall, it seems word alignment or phone alignment can only be got using seperate model, not in one model. The scripts in conformer_ctc directory seems using data/lang_bpe which can generate word alignment. The scripts in tdnn_lstm_ctc directory seems using data/lang_phone which can generate phone alignment. Using kaldi I can generate both phone align and word align from one model. The best results now seem using bpe token for english and char token for Chinese. As a kaldi user for several years, I'm confusing will phone be discarded gradually? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#48 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZFLO5ZTEZJAN2EB2VYD4LUL7MQ3ANCNFSM5EGDEFHA . 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.

Hi Dan,
It will be great if there is good examples for phone/word alignment in one scripts.
Currently I'm using kaldi for Chinese-English (mainly chinese, and support code switch). This is a common scenario now. I create one pronunication lexicon contains both Chinese and English, whch works well.
So I'm wondering what is tendency for icefall to recognize this kind of common scenario in China?


Installation of the theme in a Docker Environment

Node-RED version: 1.2.3
Platform/OS: DietPi / Docker
Browser: Firefox 82.0.3

Heya, yesterday I was able to install your theme, but I just checked it once.
Today I went to my node-red page and now it is all default. The files of your theme are still persistent and inside my node-red folder.
I tryed to reinstall everything now and it doesn't work anymore :*(
bash-5.0$ npm install @node-red-contrib-themes/midnight-red
npm notice created a lockfile as package-lock.json. You should commit this file.
+<EMAIL_ADDRESS>updated 1 package and audited 349 packages in 7.912s

5 packages are looking for funding
  run `npm fund` for details

found 3 moderate severity vulnerabilities
  run `npm audit fix` to fix them, or `npm audit` for details
bash-5.0$ npm ls node-red-contrib-theme-midnight-red
<EMAIL_ADDRESS>/usr/src/node-red
`-- (empty)

Maybe some can help me out?
BTW: is there any possibility to install the theme to /data/node-modules instead of /usr/scr/node-red/node_modules

ok... maybe i was able to fix this by myself as well after some more googling around 👍
due node-red installs all its additional modules into /data/node-modules we have to install the theme to this folder as well.
so what I did is pretty simple:

npm install --prefix /data @node-red-contrib-themes/midnight-red
changed the lines in our settings.js to the following:

	editorTheme: {
		page: {
			css: [
<EMAIL_ADDRESS><EMAIL_ADDRESS>			]
		}
	}

Maybe you can add this installation guide to your documentation as well for other guys using docker?


Video: Working with sub-forms and linked tables in ODK-X Survey

Make a video showing how to add, edit, and delete sub-form/linked table instances for our Youtube Channel.
Can do without sound or with audio narration (if with audio narration please check & fix the automatic captioning).
Context: ODK-X has started a youtube channel to share videos of how to do key tasks in ODK-X. Although these tasks are often explained in the documentation, we frequently get requests for videos and some things are easier to learn watching a video than reading documentation. We are working on getting videos in English & Spanish but also welcome contributions in other languages!

I would like to work on this!

@iamhks thanks for taking on this issue, I have assigned it to you.

Hi @elmps2018 , sorry for the delay. Here's the video: https://drive.google.com/file/d/1U6zQ_AnXTbmI6uV0DentrLv2ohiPuUMO/view?usp=drivesdk

Hi @iamhks! Thanks for working on this! Sorry for any confusion, but this looks like adding, editing, and deleting an instance, rather than sub-forms and linked tables. More like issue #240 or #237. I am not sure if @swaingotnochill is still working on those, let me check.

Hi @elmps2018 , since this issue is still pending, I thought I'll complete it. But I have a doubt, I was referring to https://docs.odk-x.org/survey-using/#opening-a-subform and it also showed creating a new instance. Could you point me to the doc/reference for sub-forms/linked tables in Survey?

Hi @iamhks! So the example shows creating a new instance from inside a form, which is a sub-form or linked table. You can demo this with the same household members example in the link. You would show entering the main (household) form, navigating/entering along to the members form, and then entering members data in the linked_table/sub-form, then saving that (ending back up in the household form), going back in and editing the data and saving again, and then deleting as well. For more info on tables see: https://docs.odk-x.org/xlsx-converter-using/?highlight=linked_table#linked-tables and the earlier link had the opening a subform.

Thank you so much for the elaborate clear explanation @elmps2018 , I understood where I was making a mistake.
Here's the link to the video: https://drive.google.com/file/d/1VoolhhUZzxviy4dbvLFt41KcnCdSis_d/view?usp=sharing

Hi @iamhks! Thanks for working on this! This is the right content now! Could you just slow it down (may be able to do this with video editing software or just redo slower) so that folks can more easily follow along? Particularly for picking the household form, to highlight clearly and slowly which one you are picking, and then to give people a moment to read/see the screen before adding an instance to the subform or editing. Thanks!

@elmps2018 Sure, here's the updated one: https://drive.google.com/file/d/1WPZhjyln05t--SsQ9cyEyoMXR81afg7G/view?usp=sharing
Feel free to point out if there's something I need to slow down even further.

@iamhks this is great thanks! Uploading to Youtube next.


Feature request: support subquery as a condition value

Expr seems not good enough..
subb := builder.Select("id").From("table_b").Where(builder.And(builder.Eq{"b": "a"}))
b := builder.Select("a, b").From("table_a").Where(
    builder.And(builder.Eq{
        "id":   23,
        "b_id": subb,
    }),
)
sql, args, err := b.ToSQL()
// args will contain the args from subquery:
//  subsql, subargs, err := subb.ToSQL()
//  args = append(args, subargs...)

fmt.Println(sql)
fmt.Println(args)
fmt.Println(err)

Expected output:
SELECT a, b FROM table_a WHERE id=? AND b_id=(SELECT id FROM table_b WHERE b=?)
[23 "a"]
<nil>

I found a solution, anyway, support the Builder as the value directly will be nice..
subsql, subargs, err := builder.Select("id").From("table_b").Where(builder.And(builder.Eq{"b": "a"})).ToSQL()

b := builder.Select("a, b").From("table_a").Where(
        builder.And(builder.Eq{
                "id": 23,
                // "b_id": subsql,
                "b_id": builder.Expr(subsql, subargs...),
        }),
)
sql, args, err := b.ToSQL()
fmt.Println(sql)
fmt.Println(args)
fmt.Println(err)

That's a method but not a good idea. I think treat Builder as a subquery is a better idea for performance.

The issue doesn't actually exists, my bad..

Yes, this feature has been implemented. See the test https://github.com/go-xorm/builder/commit/a4a881a4e552fcb5473f39e8b9f7d9f3457a7fb0


brew upgrade -g sometime will install old version app

brew doctor output
$ brew doctor
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed casks are deprecated or disabled.
You should find replacements for the following casks:
  dozer

Warning: Homebrew's "sbin" was not found in your PATH but you have installed
formulae that put executables in /usr/local/sbin.
Consider setting your PATH for example like so:
  echo 'export PATH="/usr/local/sbin:$PATH"' >> ~/.zshrc

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.2.15
ORIGIN: https://github.com/Homebrew/brew
HEAD: 92a4311868322188478d7a90511ec0e8e6b0d7df
Last commit: 8 days ago
Core tap JSON: 01 Apr 16:33 UTC
Core cask tap JSON: 01 Apr 16:33 UTC
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 4
Homebrew Ruby: 3.1.4 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/3.1.4/bin/ruby
CPU: quad-core 64-bit haswell
Clang: 15.0.0 build 1500
Git: 2.44.0 => /usr/local/bin/git
Curl: 8.4.0 => /usr/bin/curl
macOS: 14.4.1-x86_64
CLT: <IP_ADDRESS>.1.1708646388
Xcode: N/A

What were you trying to do (and why)?
brew upgrade -g,i want upgrade lib and app
What happened (include all command output)?
sometimes will install old version app.example: mounty 2.4 installed, brew want install mounty 2.1
What did you expect to happen?
dont install old version
Step-by-step reproduction instructions (by running brew commands)
just `brew upgrade -g`.
My macbook is old(2014 mid) and i use `opencore-legacy-patcher` to install `14.4.1`

Step-by-step reproduction instructions (by running brew commands)

This does not reproduce. Please provide more specific instructions and affected cask(s).

Closing due to no response.


Adding optional policy name

what
Allow for custom policy name if not default to module.this.id
why
Flexibility

Okay I will double check. This was to bypass the name module.this.id from adding a tenant, environment etc.. if not needed.

You can also do the following
module "policy" {
  source  = "..."
  version = "..."

  name = "banana"

  namespace   = ""
  tenant      = ""
  environment = ""
  stage       = ""
  attributes  = []

  # ...
}


UI/UX improvements, Documentation improvements, Backend tweaks.

UI/UX Improvements
Use new UI to:

Multi-project functionality (create, clean, delete projects)
Assets Management (manage assets straight on web-ui)
Edit Feed through Google Sheets
Configure and Download Google Ads CSV
Configure, generate and Upload video variations to YouTube.

Backend Improvements

Python3 Migration
minor tweaks to allow all UI/UX improvements to run.

Documentation improvements
Improved README.md and added a simple user manual.

@googlebot I fixed it.

Notes:

Do we have license to publish all files in base_project/assets, especially the font and video?
There's a move static/vogon-ui.css → base_project/logs/.keep, is that correct?
There are several other ".keep" files that I don't know what they're for, but looks like they already existed
base_project/feed.csv is added as binary, should be text

hey, just fixed all your notes on next commit. Thank you for the time and concern. Could you PTAL?


import go_vncdriver error ...

i run the example code and i get this error:
Traceback (most recent call last):
File "code.py", line 2, in
import universe  # register the universe environments
File "/home/billykon/desktop/python bot/universe/universe/init.py", line 22, in
from universe import error, envs
File "/home/billykon/desktop/python bot/universe/universe/envs/init.py", line 1, in
import universe.envs.vnc_env
File "/home/billykon/desktop/python bot/universe/universe/envs/vnc_env.py", line 18, in
import go_vncdriver
ImportError: dynamic module does not define module export function (PyInit_go_vncdriver)

That indicates some build or install problem with go_vncdriver. What OS and Python version are you using? How did you install it? What does pydoc go_vncdriver report?

It doesn't seem like it's installed at all. Did you do the final pip install -e ./ step? Can you provide a transcript of how you built and installed it?

It doesn't seem like it's installed at all. Did you do the final pip install -e ./ step? Can you provide a transcript of how you built and installed it?

yes i did tha last.
1)git clone https://github.com/openai/go-vncdriver.git
2)cd go-vncdriver
3)./build.py
4)pip3 install -e ./

yes i did tha last.
1)git clone https://github.com/openai/go-vncdriver.git
2)cd go-vncdriver
3)./build.py
4)pip3 install -e ./

ok here it is:
`billykon@billykon-K5130:~/Επιφάνεια εργασίας/python bot/go-vncdriver$ sudo ./build.sh
[sudo] password for billykon:
Env info:
export GOPATH='/home/billykon/Επιφάνεια εργασίας/python bot/go-vncdriver/.build'
export CGO_CFLAGS='-I/usr/lib/python2.7/dist-packages/numpy/core/include -I/usr/include/python2.7'
export CGO_LDFLAGS='/usr/lib/x86_64-linux-gnu/libjpeg.so -L/usr/lib -lpython2.7'
export GO15VENDOREXPERIMENT='1'
Running build with OpenGL rendering.
Building with OpenGL: GOPATH=/home/billykon/Επιφάνεια εργασίας/python bot/go-vncdriver/.build go build -buildmode=c-shared -o go_vncdriver.so github.com/openai/go-vncdriver. (Set GO_VNCDRIVER_NOGL to build without OpenGL.)
billykon@billykon-K5130:~/Επιφάνεια εργασίας/python bot/go-vncdriver$ sudo pip3 install -e .
The directory '/home/billykon/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/home/billykon/.cache/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Obtaining file:///home/billykon/%CE%95%CF%80%CE%B9%CF%86%CE%AC%CE%BD%CE%B5%CE%B9%CE%B1%20%CE%B5%CF%81%CE%B3%CE%B1%CF%83%CE%AF%CE%B1%CF%82/python%20bot/go-vncdriver
Requirement already satisfied: numpy in /usr/local/lib/python3.5/dist-packages (from go-vncdriver==0.4.19)
Installing collected packages: go-vncdriver
Found existing installation: go-vncdriver 0.4.19
Can't uninstall 'go-vncdriver'. No files were found to uninstall.
Running setup.py develop for go-vncdriver
Successfully installed go-vncdriver
`

ok here it is:
`billykon@billykon-K5130:~/Επιφάνεια εργασίας/python bot/go-vncdriver$ sudo ./build.sh
[sudo] password for billykon:
Env info:
export GOPATH='/home/billykon/Επιφάνεια εργασίας/python bot/go-vncdriver/.build'
export CGO_CFLAGS='-I/usr/lib/python2.7/dist-packages/numpy/core/include -I/usr/include/python2.7'
export CGO_LDFLAGS='/usr/lib/x86_64-linux-gnu/libjpeg.so -L/usr/lib -lpython2.7'
export GO15VENDOREXPERIMENT='1'
Running build with OpenGL rendering.
Building with OpenGL: GOPATH=/home/billykon/Επιφάνεια εργασίας/python bot/go-vncdriver/.build go build -buildmode=c-shared -o go_vncdriver.so github.com/openai/go-vncdriver. (Set GO_VNCDRIVER_NOGL to build without OpenGL.)
billykon@billykon-K5130:~/Επιφάνεια εργασίας/python bot/go-vncdriver$ sudo pip3 install -e .
The directory '/home/billykon/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/home/billykon/.cache/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
Obtaining file:///home/billykon/%CE%95%CF%80%CE%B9%CF%86%CE%AC%CE%BD%CE%B5%CE%B9%CE%B1%20%CE%B5%CF%81%CE%B3%CE%B1%CF%83%CE%AF%CE%B1%CF%82/python%20bot/go-vncdriver
Requirement already satisfied: numpy in /usr/local/lib/python3.5/dist-packages (from go-vncdriver==0.4.19)
Installing collected packages: go-vncdriver
Found existing installation: go-vncdriver 0.4.19
Can't uninstall 'go-vncdriver'. No files were found to uninstall.
Running setup.py develop for go-vncdriver
Successfully installed go-vncdriver
`

oh wait i tried instead of ./build.py to run : python3 build.py and after that i did pip3 install -e . and it looks like its running now.
Thanks for trying to help and have a good new year :)

oh wait i tried instead of ./build.py to run : python3 build.py and after that i did pip3 install -e . and it looks like its running now.
Thanks for trying to help and have a good new year :)


Prevent overly long lines of text

Pivotal
Wraps most text in a .text block to keep the line length down.

👍  looks good


EVEREST-428 rebuild RC image

EVEREST-428
Rebuild RC image on every commit to the release branch

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: arguments are dirtied an excessive amount of times even when they aren't consumed

the best way to solve this is to provide functions for things that care about performance.
For example:
class Foo {
  data = MyData.from(this, () => ({
    foo: () => this.foo,
    bar: () => this.bar,
  }));
}

And with a less awkward API via function-wrapper:
function myData(context, args) {
  return MyData.from(context, () => args);
}

// users would do this:
class Foo {
  data = myData(this, {
    foo: () => this.foo,
    bar: () => this.bar,
  });
}


test(substitute): add test for not encoding [ and ] characters

Refs #2

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Chart not showing (updateIndex / inRange is not a function)

I'm getting this multiple error when printing the canvas of some of my charts. Sometimes they show up and other times they don't (it depends on the number of charts being displayed). I don't think it's about responsive, as past bugs suggest.

I define the width & height of the canvas like this:
<canvas id="myChart" width="400" height="250"></canvas>
I've been looking for similar issues but I've not been able to find anything. Any ideas?

Please provide more context (Chart.js version, browser version, etc.), a jsfiddle and instructions to reproduce your issue.

Closing as inactive


Add CMakeLists.txt icon

I'm not a designer, but it would be nice if there was an icon for CMake's configuration file, CMakeLists.txt.
This is their official logo:

This icon should also probably apply for CMakeCache.txt, and a folder variant for CMakeFiles?

Looks awesome, thanks! @miguelsolorio


Add playlist support

not tested yet

Is there someone maintaining the repo?

Hi @roidelapluie! Sorry for the long silence. Indeed this repository is a little neglected these days because I built out what I needed for the Grafana Terraform provider and then moved on to other things. :confounded:
I no longer have a good test environment for Grafana so I can't actually verify this, but from just reading through the code it seems plausible to me and I'm happy to merge it if you can confirm that it's worked against a Grafana instance.
Long term I should find a better home for this code, but I hope I can at least unblock you for now.

Implemented in #32 with tests.


command not found: docker-compose

The step:9 - Finally run the containers. Make sure you are in the docker folder before running this command.
When I run the command blow,there is a error :command not found: docker-compose
docker-compose up -d

And if I run docker -v,reponse blow:
Docker version 1.10.1, build 9e83765

Could there is anyone do me a favor?

@wedojava docker-compose is a different software than docker you should be running docker-compose -v to see if you have it installed. Most probably you have not yet installed it. If you install the Docker Toolbox you should get it. Check out this link https://docs.docker.com/compose/install/

Thank you.I got it!
I'm a newer in docker and English.These is not a good issue.
I'd better to read document more carefully yet.
Have a good one.


Stable Diffusion XL unet fails compile for FP16

Repro script:
import torch
from diffusers import DiffusionPipeline
import torch_migraphx

model_repo = 'stabilityai/stable-diffusion-xl-base-1.0'
prompts = ["a photo of an astronaut riding a horse on mars"]
fname = 'output.png'

pipe = DiffusionPipeline.from_pretrained(model_repo,
                                         torch_dtype=torch.float16,
                                         use_safetensors=True,
                                         variant="fp16").to("cuda")

pipe.unet = torch.compile(pipe.unet, backend='migraphx')

image = pipe(prompt=prompts).images[0]

image.save(fname)

Error:
...
    mgx_module.add_instruction(migraphx.op('concat', axis=cat_dim),
torch._dynamo.exc.BackendCompilerFailed: backend='migraphx' raised:
RuntimeError: /workspace/tm_dlm/AMDMIGraphX/src/include/migraphx/check_shapes.hpp:210: same_type: concat: Types do not match

While executing %cat_default_4 : [num_users=1] = call_function[target=torch.ops.aten.cat.default](args = ([%arg3_1, %reshape_1], -1), kwargs = {})

The exported dynamo graph contains mismatched datatypes for inputs to the aten.cat ops. This is acceptable for dynamo since they have logic to promote inputs during lowering, see: https://github.com/pytorch/pytorch/blob/main/torch/_inductor/lowering.py#L1067
Fix should be to add a pass in the torch.compile pipeline to perform the same input promotion in the parsed GraphModule.


Netflix addon play not all content

I'm submitting a ...

[X ] bug report
[ ] feature request
[ ] support request

General infomration
OS Windows 7 and Windows 10 Latest updates
Kodi Alpha release Kodi v18 "Leia" Alpha 1
and Nightly latest release
Netlix addon V 0.13.5
Country (Netherlands)
Prerequisites

[X ] Are you running the latest version?
Yes

Description
for Example
All episodes of "Lost in space" play
and
All episodes of "Stranges things" won't play
Steps to Reproduce
I tried the same on multiple computers and even OS
Exactly the same happen on all systems and platforms
In Chrome all content play (same computer as Kodi installed on)
In netflix app for Windows 10 all play (same computer as kodi installed on)
Expected behavior:
Should not all content play in Kodi?
Actual behavior: [What actually happened]
When it plays the content, it plays as it should, all works fine
When it does noet play, nothing happen, (nightly release) and a message in de the alpha release show up, unable to play.
Context (Environment)
Multi Hardware and platform and OS
Installation

[X] I installed the plugin via zip from the Releases page
[X ] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[X ] Windows (x86/x64)
[ ] Linux (ARM)
[ ] Android

Additional informatin on the environment
Windows 64 bit
Debug log
[Please include a link to your debug log (use something like http://sprunge.us/) or similar, please do not paste]
Other information
[e.g. detailed explanation, related issues, suggestions how to fix, links for us to have context, etc.]
Screenshots
[Please add a screenshot if that helps understanding your problem]
[You can erase any parts of this template not applicable to your Issue.]

https://pastebin.com/ggqC1uqV
Kodi Log when netflix content play

https://pastebin.com/PpsfE3Bb
Kodi log when netflix content dont play

duplicate to #360


Primitive Plane creation: Plane can only be spawned in x-z plane.

Maybe it should pick the two largest dimensions.

We considered this, but determined the configuration space was larger than we wanted, and the user still wouldn't have certainty and consistency that the API did what they wanted. Sticking with the current design.


Minimum angle of "Controller roll limit" is high

Hi
I will set FW_R_LIM to 25 degree but i not allowed to set it under 35 degree in qgc. also some gliders must not bank more than 30 degree . please change the FW_R_LIM min .

30° is an extremely low (max!) bank angle. I don't know of any system that should not be allowed to bank so little.
If you want to have your flight with less tight turns than you should adapt turning radius and/or your route planning, and not this parameter.
Btw, you can still set any parameter in QGC to a value outside of the "normal" range, by clicking "enforce safe" or something similar in the advanced menu.

https://github.com/mavlink/qgroundcontrol/pull/7910

Btw, you can still set any parameter in QGC to a value outside of the "normal" range, by clicking "enforce safe" or something similar in the advanced menu.

i can not force save this parameter

Btw, you can still set any parameter in QGC to a value outside of the "normal" range, by clicking "enforce safe" or something similar in the advanced menu.

i can not force save this parameter

I quickly checked, works for me.


Chore/selected address modification

Summary
Handle Google Autocomplete/Experian APIs

The above APIs requires 2 steps to select an address.
This PR handles the 2nd step based on the availability of prop.

Hi @kunalkp ,
Thanks for contributing to the repo. Please follow up on the #2406 as we are working on the same changes there.


[REQUEST] Include Orc Class from Issue 5

Hi, I was talking with someone else on the Foundry Discord who would be interested to see the Orc from Issue 5 of Crawl! included in your module. Is that something you would be interested in doing?

Yeah for sure!

Nice, I am that other guy on the discord.

Fixed in #6


Navigating between notebooks and .py file creates scrolling and cell selection errors

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur
Scenario 1

Have two tabs of notebooks open.
Open any .py file, place it on to a right half divide.
Click on any cell in one of the notebook tabs.
Click on any line in the .py file on the right.
Click back to the other notebook tab.

Scenario 2

Have a notebook on left divide and a .py file on the right divide.
Click on any cell in the notebook.
Click on any line in the .py file
Click back to the cell in the notebook
(repeat until the error occurs, usually errors in 3-4 attempts)

Actual behavior
Scenario 1

Every cell in the notebook is selected, with a cursor on every cell. There is no way to unselect these cells without clicking away every cursor or restarting the notebook.

Scenario 2

The notebook automatically scrolls down without any input.

Expected behavior
There shouldn't be auto selection of every cell or auto-scrolling without user input.

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running:                       Local
Extension version:                               2020.2.64397
VS Code version:                                1.42.1
Setting python.jediEnabled:               true | false
Python and/or Anaconda version:     3.8.1.
OS:                                                    Mac OS Darwin x64 19.3.0
Virtual environment:                          conda

Developer Tools Console Output

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

Thanks for reporting this issue, closing as a dup of https://github.com/microsoft/vscode-python/issues/10194
The fix will be in the next release


0.9.5 Register new sensor and publish via carla_ros_bridge

Hi everybody!
I'm trying to add a new "own" sensor to Carla. What I've done is following these steps to register a new sensor. I just duplicated the Lidar sensor and renamed it and all its dependencies (including the files in the CarlaUE4 Plugins folder). It worked as far as I'm able to spawn the sensor and attach it to a vehicle.
Now up to the next step, the ROS-Bridge. I duplicated the lidar.py here aswell and renamed it properly.But when I try using the ROS-Bridge with the new sensor I get the following error:
terminate called after throwing an instance of 'clmdep_msgpack::v1::type_error'   what():  std::bad_cast
I believe that the sensor is unknown for the bridge or something? Maybe I missed a step registering the new sensor somewhere else?
Any ideas are welcome!
Kind regards,
Slyer

Hi @slyer0009, Did you get around with this issue?

Are there any updates on this? How did you resolve this?


Expose _rootUrl of gltf loader as public get property

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14673/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14673/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14673/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14673/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14673/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14673/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14673/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14673/merge#BCU1XR#0


Get correct client IP address from X-Forwarded-For list

https://github.com/stefanprodan/MvcThrottle/blob/master/MvcThrottle/ThrottlingFilter.cs#L359
ip = request.ServerVariables["HTTP_X_FORWARDED_FOR"];
if (!string.IsNullOrEmpty(ip))
{
    if (ip.IndexOf(",") > 0)
    {
        ip = ip.Split(',').Last(); // <--  HERE
    }
}

I think the spec says (and our LBs say 😺) that the client address should be the first IP in the list.
Wikipedia
RFC 7239

In a chain of proxy servers where this is fully utilized, the first
"for" parameter will disclose the client where the request was first
made, followed by any subsequent proxy identifiers.  The last proxy
in the chain is not part of the list of "for" parameters.

I'll get a PR for fixing that shortly.  Awesome library!

What reverse proxy does that? Azure and AWS will put the client IP as last.

Hmmm...we're using nginx behind an ELB and they're putting the client first.  Seems the internet is divided on this issue though.  Looks like nginx has a setting to switch that, real_ip_recursive, but we don't use that.
Maybe I could add a setting flag in the ThrottlePolicy that defaults to last, but can be set to first?


Fix IElixir Jupyter Kernel crashes (IEx 1.14.0 / Erlang/OTP 25)

I think #60 did exactly the same except for the tests and API call updates, but the PR was closed.
I've also updated sandbox to use elixir_erl_var.from_env instead of the removed elixir_env.env_to_scope call, and fixed some tests.
All tests passing but one, which complains about undefined symbol: cblas_sgemm and I didn't wanted to spend more time trying to figure out which library does have this symbol given that matrex seems to be not in active development.
Also a final disclaimer: I'm not an Elixir dev (that's my first time even touching Elixir code), I just wanted to get Elixir working on my Jupyter Notebook to do some experiments and those minor changes did fixed the kernel crashes and it does execute basic scenarios, but I'm pretty sure there are still broken bits (like Matrex).

Hey @JonathanxD,
thank you very much for the contribution. I was trying to refresh a little bit IElixir, however there were few dependencies, including erlzmq that were quite serious in amount of changes needed. I'll try to test your changes on my local as fast, as I'll find some time.
Thanks once again,
Piotr

I've pulled the changes from this PR into another PR - https://github.com/pprzetacznik/IElixir/pull/64
Thanks for the contribution! It really helped me refresh the codebase! 🚀


Allow partial reads

These changes allow partial reads so that the first few rows of large XLSX files can be read quickly.

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is 70.31%.

@@           Coverage Diff           @@
##           master       #2   +/-   ##
=======================================
  Coverage   78.15%   78.15%
=======================================
  Files          19       19
  Lines        2513     2513
=======================================
  Hits         1964     1964
  Misses        381      381
  Partials      168      168

Impacted Files
Coverage Δ

xmlWorkbook.go
64.51% <56.25%> (ø)
:arrow_up:

file.go
75.91% <71.42%> (ø)
:arrow_up:

lib.go
85.78% <75.6%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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