amazing work!, but low fps. no GPU acceleration

the project is awesome but when I run CSS animations like anime.js it renders out so slowly. I wonder if it is a limitation or something that can be fixed.
Also, is there a way to remove nodejs?

Yes, Now miniblink can render by GDI only,  because GPU acceleration on windows has many compatibility problems in particular windows XP.
but, there's opportunity to improve miniblink' performance , I'm doing this recently.

@weolar great! achieving 60 fps is kinda essential to make it a viable solution, cause right now the performance is too slow. Maybe having GPU acceleration for all platforms except Windows XP?

Yes, this is a solution.
Before I don't implement GPU acceleration, because :
1, some GPU will cause crash. chromium resolve this by maintain a GPU blacklist.
2, GPU acceleration on layer window( or windowless software) is difficulty to implement.
so, I try to improve  in three ways:
1, change blink thread to other thread(now it is on ui thread).
2, improve cc layer performance. (now cc layer implement by myself, not chromium)
3, GPU acceleration on some machine
the first way and second way is high priority, and finally three way.

@weolarI am not C/C++ savvy and I hope I could help, In fact, I will try my best doing so cause mini-blink can become the absolute best solution for many developers including C# (me :D).

Oh, Welcome!  I need some one to port miniblink to other Language.
You can help me, and this is some archives :
C#：
https://github.com/E024/MiniBlinkPinvoke
https://github.com/E024/MiniBlinkPinvokeDemo
https://gitee.com/kyozy/miniblinknet
delphi：
https://github.com/yangyxd/wke-for-delphi
https://gitee.com/LangjiApp/Wke4Delphi.git


Cant find pretrained model

i cant find the pretrained pytroch model

Go to https://github.com/isl-org/MiDaS/releases and you will find all the releases. Click on Assets to see the full list of pytorch models.


Conflicts with FallingTree Mod

Mod conflicts with FallingTree mod in some way.
When installed the mod doesn't work correctly, but as far as I can tell it's not throwing any errors.

Are you sure it's my mod? I'm playing with both in my world right now. What version of fallingtree are you using?

Really.. Okay..
If I remove your mod, it works correctly, so I was assuming that's where the conflict happened.
Currently installed - FallingTree-1.17.1-2.12.3
Also tried with 2.11.4
Sentimentality3 Version = 1.2.9
I do have a modified config..
sentimentality3.zip
(allTools = false + individuals, allCompressedBlocks = false + individuals, chunker, charcoal, coalChunks, Sticks, FeatherBlock, Gliders, Furnaces, Blasts, Smokers, WoolArmor, CopperArmor = false)

Oh I see. I wonder if the problem is because you have my tools disabled, and my tools try to register with the fabric tool tags, which I assume fallingtree uses too. Let me do some testing and see if thats the cause.

Yes this is the problem. Fabric fails to load tool tags completely if a mod attempts to add a non-existing tool to the tag list. Thank you for the report, I'll be working on this asap.

No problem at all!
Glad you were able to figure it out!

It looks like this is both a problem with Sentimentality and FallingTree. In 1.17, minecraft added a tool tag system and neither of us used it in our 1.17 updates. I have solved my half, which fixed FallingTree not working, but now my Axes no longer work with FT. FT will have to update to use the new system before my axes work again. I'll push this update later this week to give time for FT to update as well as put some more fixes in S3.


Will there be support of Ktor version 3

Currently library is using last stable ktor with major version 2.
Very soon it will be released ktor with major version 3.
Do you have plans to support ktor 3 in this SMILEY/ktor-swagger-ui library?

Hi,
i havent looked at it but i'll try to support/move to ktor 3.x when it releases.


Error "Cannot read property 'volume' of undefined"

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-client/lib/controllers/receiver.js:31:5

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:251:20

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at PersistentClient._updateStatus (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:323:12)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at PersistentClient.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at emitTwo (events.js:131:20)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at MediaPlayer.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at emitTwo (events.js:126:13)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at ChromecastDevice._updateClientStatus (/opt/iobroker/node_modules/iobroker.chromecast/lib/chromecastDevice.js:610:80)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at MediaPlayer.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/mediaPlayer.js:76:27)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: at PersistentClient.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:150:45)

Caught
2019-02-07 21:17:46.408
error
by controller[1]: TypeError: Cannot read property 'volume' of undefined

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-client/lib/controllers/receiver.js:31:5

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:251:20

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient._updateStatus (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:323:12)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at emitTwo (events.js:131:20)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at MediaPlayer.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at emitTwo (events.js:126:13)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at ChromecastDevice._updateClientStatus (/opt/iobroker/node_modules/iobroker.chromecast/lib/chromecastDevice.js:610:80)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at MediaPlayer.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/mediaPlayer.js:76:27)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:150:45)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: TypeError: Cannot read property 'volume' of undefined

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-client/lib/controllers/receiver.js:31:5

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:251:20

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient._updateStatus (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:323:12)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at emitTwo (events.js:131:20)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at MediaPlayer.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at emitTwo (events.js:126:13)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at ChromecastDevice._updateClientStatus (/opt/iobroker/node_modules/iobroker.chromecast/lib/chromecastDevice.js:610:80)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at MediaPlayer.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/mediaPlayer.js:76:27)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:150:45)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: TypeError: Cannot read property 'volume' of undefined

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-client/lib/controllers/receiver.js:31:5

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at /opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:251:20

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient._updateStatus (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:323:12)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at PersistentClient.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at emitTwo (events.js:131:20)

Caught
2019-02-07 21:17:46.408
error
by controller[0]: at MediaPlayer.emit (events.js:214:7)

Caught
2019-02-07 21:17:46.407
error
by controller[0]: at emitTwo (events.js:126:13)

Caught
2019-02-07 21:17:46.407
error
by controller[0]: at ChromecastDevice._updateClientStatus (/opt/iobroker/node_modules/iobroker.chromecast/lib/chromecastDevice.js:610:80)

Caught
2019-02-07 21:17:46.407
error
by controller[0]: at MediaPlayer.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/mediaPlayer.js:76:27)

Caught
2019-02-07 21:17:46.407
error
by controller[0]: at PersistentClient.getVolume (/opt/iobroker/node_modules/iobroker.chromecast/node_modules/castv2-player/lib/persistentClient.js:150:45)

Caught
2019-02-07 21:17:46.407
error
by controller[0]: TypeError: Cannot read property 'volume' of undefined

Adding some additional checks to prevent the exception is simple but I would like to know what caused it. Usually the reason is that the connection to the device closed unexpected. Could you please confirm if this is the case?
A full log of the Chromecast adapter would really help to understand the context when the error happened.

Hey,
no problem but the debug log has a lot of private informations. Have you an email adress for me?
And the issue is directly after the start.... very short green flag and than red.

It was a race condition - fixed in the cast-player library. I will release a new version with a bunch of fixes.

Thanks

Fixed in v2.2.3


Appstudio update test-component-pac-aicz

Pipelines as Code configuration proposal

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

StatusDurationName

✅ Succeeded
10 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
31 seconds
build-container

✅ Succeeded
22 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
inspect-image

✅ Succeeded
37 seconds
clamav-scan

✅ Succeeded
13 seconds
clair-scan

✅ Succeeded
9 seconds
sbom-json-check

✅ Succeeded
16 seconds
label-check

✅ Succeeded
16 seconds
optional-label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary

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

StatusDurationName

✅ Succeeded
10 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
27 seconds
build-container

✅ Succeeded
12 seconds
deprecated-base-image-check

✅ Succeeded
11 seconds
inspect-image

✅ Succeeded
11 seconds
clair-scan

✅ Succeeded
33 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
13 seconds
label-check

✅ Succeeded
13 seconds
optional-label-check

✅ Succeeded
8 seconds
show-summary

✅ Succeeded
6 seconds
show-sbom


Parameterized k6 binary download

Hey @borjacampina,
I know that it's too late to answer this PR, but we usually recommend using the latest stable version of k6 to get new features, improvements and bugfixes. While I understand your concern, I am going to close this PR, since I am moving this repo to a new address:
https://github.com/k6io/example-circleci
Feel free to send a new PR if you still need to have this feature.


Handling of the request size

Hi people,
I've played a bit with the library and found a couple of inconveniences.
The obvious - if you pass a big text to translate, it is impossible to guess when you will get an error response that the size of the request is over the limit. I know it is 128k but the headers and other data are probably included in the calculation. As the client, I need help determining when to split the data.

Hi @emartynov , we don't have a good solution for this yet unfortunately, we are thinking about a good interface for this.
Until we release that, does it work for you to reserve a few K for the headers and send at most 128K-<estimated_header_size> of text?

Is there a way to estimate header section size? Do you also wrap data in some other format in the body of the request?
As for now, I just set that string for the translation should not to be more than 100K.

That is the way we currently recommend. 120k text size should be fine in our experience - it's hard to give an estimate of header sizes on our end due to proxys etc.


added some courses to free-courses-de.md

What does this PR do?
Add Resource(s)
Description
Some German courses / Tutorlials for the Programming languages: C, Rust, Haskell and JavaScript
Why is this valuable (or not)?
They aren't any listed courses in German for the listed languages
and its free so
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Some courses:
C: C Tutorials Deutsch - The Morpheus Tutorials
Rust: Rust Programmieren Tutorials Deutsch für Anfänger - The Morpheus Tutorials
Haskell: Haskell Tutorials Deutsch - The Morpheus Tutorials
JavaScript: JavaScript Lernen für Anfänger bis Profis - The Morpheus Tutorials
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Could you please ensure all lists, categories, and sections/subsections remain in alphabetical order?
Could you please ensure all sections and subsections have 2 line breaks above them?

2 empty lines between last link and new section
1 empty line between heading & first link of its section
0 empty line between two links
1 empty line at the end of each .md file
- https://github.com/EbookFoundation/free-programming-books/blob/master/CONTRIBUTING.md#formatting

You can get more guidance on the formatting and conventions in this repository by checking out the contribution guide.
You can click on the failed checks to get more information on any check failures. You'll be able to see exactly what was wrong and how to fix it.


[Feature Request] Sample showing how to use Monitoring feature (preview) with promptflow

Is your feature request related to a problem? Please describe.
I deployed a promptflow to an online endpoint, and I would like to continuously monitor the deployed model with the Model monitoring (preview) feature). I tried to use it without success. Is this feature ready to be used with a promptflow registered model?
Describe the solution you'd like
Provide a sample showing how to use Model monitoring (preview) feature on a promptflow deployed on an online-endpoint.
Describe alternatives you've considered
I tried to create a Monitor for a standard promptflow, but the step "Generation Safety & Quality - Signal Monitor
" fails with Error encountered while attempting to get model type.

This is an internal user ran into failure when configure model monitoring. She already opened MDC.
Follow up with momo team.


Porcupine Issue: Porcupine throws runtime error on running

Try running Unity sdk demo using custom Keyword. When running the demo code (updated to accept my custom keyword) it returns the error "Make sure your access key 'vzhanAD[*******]pnYziQ==' is a valid access key". Tested in both porcupine 2.1.3 and 2.0.2 version. Tested on Windows (Editor) and Android 8.0.0. I am using a 2.1.0 custom keyword version but I can only locate 2.1.3 package In Github. When using Built in keywords, code runs successfully.
Expected behaviour
Run without errors
Actual behaviour
"Make sure your access key 'vzhanAD[*******]pnYziQ==' is a valid access key"
Steps to reproduce the behaviour
Create (SPANISH) custom keyword, download and place it in Streaming Assets folder.
Update PorcupuineDemo.cs code to
            PorcupineManager porcupineManager = PorcupineManager.FromKeywordPaths(            "${vzhanAD[************]nYziQ==}",            new List<string> { Application.streamingAssetsPath + "/[CUSTOM_WORD]_es_android_v2_1_0.ppn" },            OnWakeWordDetected,            modelPath: Application.streamingAssetsPath + "/porcupine_params_es.pv");
(Include enough details so that the issue can be reproduced independently.)
Unity Version: 2019.4.27.f1
Porcupine Package Version: 2.1.3
Porcupine Language: Es (Spanish)
SO Develop: Windows 10
SO Tested: Windows 10, Android 8

remove ${} around your AccessKey

We place that message Make sure your access key ... as a hint since the most common mistake users make when trying the demo is not putting in an AccessKey. But it will show up for any case of InvalidArgument, any of the arguments provided to Porcupine may be incorrect.
If the ${} around your AccessKey. Please double check your paths to all the files.  Are you testing the demo on Windows or on an Emulator / Android device?

Hello ErisMik, as kenarsa pointed out, when copying the AccessKey I ended up moving the "". from $"AccessKey" to "$AcessKey". Updated the code and now is fully working on Android devices.


Potential method name clash with LibreSSL

I am working on a project where I have to integrate both OpenSSL & OpenSSH and I have been experiencing a similar symptom to what it is described with the try-ciphers script in https://github.com/openssl/openssl/issues/9524 . I can SSH into my system if it's AESGCM, but not with AESCTR.
I've been trying to debug-by-printf (ugh) this situation from OpenSSH side, and I think I have some idea of what happened.
I added some tracing to various routines within OpenSSH. I discovered that the context was getting called with some IV, then we would encrypt / decrypt a handful of times, then OpenSSH tries to marshal the state (key, IV, etc) via a blob to an inferior process. When it does this, I observed it was sending out the original IV, and not the running IV.
In the LibreSSL compatibility layer of OpenSSH , there is an EVP_CIPHER_CTX_get_iv() routine. This overlaps with an OpenSSL routine of the same name, of course... so with the right defines / configuration it calls into the OpenSSL one instead. But looking at the LibreSSL routine is instructive...
int
EVP_CIPHER_CTX_get_iv(const EVP_CIPHER_CTX *ctx, unsigned char *iv, size_t len)
{
	if (ctx == NULL)
		return 0;
	if (EVP_CIPHER_CTX_iv_length(ctx) < 0)
		return 0;
	if (len != (size_t)EVP_CIPHER_CTX_iv_length(ctx))
		return 0;
	if (len > EVP_MAX_IV_LENGTH)
		return 0; /* sanity check; shouldn't happen */
	/*
	 * Skip the memcpy entirely when the requested IV length is zero,
	 * since the iv pointer may be NULL or invalid.
	 */
	if (len != 0) {
		if (iv == NULL)
			return 0;
# ifdef HAVE_EVP_CIPHER_CTX_IV
		memcpy(iv, EVP_CIPHER_CTX_iv(ctx), len);
# else
		memcpy(iv, ctx->iv, len);
# endif /* HAVE_EVP_CIPHER_CTX_IV */
	}
	return 1;
}

It seems, at least to me, that the above is meant to get the running IV and not the original IV.
When I modified the OpenSSH call to EVP_CIPHER_CTX_get_iv() and instead had it call EVP_CIPHER_CTX_get_iv_state(), it fixed the problem as observed in my setup.
There is some subsequent discussion in https://github.com/openssl/openssl/issues/9524 about the merits of changing
the name(s) before beta. I'll let project members move salient discussion points along those lines into this issue as needed...

We will rename these APIs because the current names are confusing.

OTC decision: Use one function that has a parameter that indicates running or original iv.

I'm not sure that that is what we agreed on

What did you hear :)

There was no definite agreement about what to change and how but there at least is an agreement that the current API definition of EVP_CIPHER_CTX_get_iv() and EVP_CIPHER_CTX_get_iv_state() is confusing and needs adjustments.
The proposed choices were:

Rename EVP_CIPHER_CTX_get_iv to EVP_CIPHER_CTX_get_original_iv and EVP_CIPHER_CTX_get_iv_state to EVP_CIPHER_CTX_get_running_iv

Keep just EVP_CIPHER_CTX_get_iv and add a parameter to it to indicate whether the original or running iv should be returned.

In case of 2 there still will be name clash with LibreSSL but it will result in syntax error in compilation so the users can easily see that their code needs to handle LibreSSL and OpenSSL differently.

In https://lists.mindrot.org/pipermail/openssh-unix-dev/2020-December/039003.html @daztucker proposed:

Rename EVP_CIPHER_CTX_get_iv to EVP_CIPHER_CTX_get_original_iv and EVP_CIPHER_CTX_get_iv_state to EVP_CIPHER_CTX_get_iv

In https://lists.mindrot.org/pipermail/openssh-unix-dev/2020-December/039003.html @daztucker thought about:
3. Rename `EVP_CIPHER_CTX_get_iv` to `EVP_CIPHER_CTX_get_original_iv` and `EVP_CIPHER_CTX_get_iv_state` to `EVP_CIPHER_CTX_get_iv`

That was discussed but not acceptable for OTC. Please note that the objective of the change that OTC aims for is that the current APIs are confusing and not the clash with LibreSSL. This would not remove the confusion it would just make it the other way around.

Ok.
Proposal 1. looks good to me, as libressl can keep their EVP_CIPHER_CTX_get_iv() and the libressl compat layer in openssl can call EVP_CIPHER_CTX_get_running_iv().

I see that 3.0.0-alpha10 did not fix this problem. It would be good if it could be fixed before 3.0.0 leaves alpha, otherwise it will be more difficult for OpenSSH to support OpenSSL-3.x

I see that 3.0.0-alpha10 did not fix this problem. It would be good if it could be fixed before 3.0.0 leaves alpha, otherwise it will be more difficult for OpenSSH to support OpenSSL-3.x


Would you provide a compression Reflect.min.js in the npm package?

Duplicate of #47?

@MartinJohns yes,it's duplicated.


collectCoverage breaks TypeScript source stepping

Do you want to request a feature or report a bug?
BUG
What is the current behavior?
When collectCoverage is set to true, then source stepping no longer functions correctly.
Steps:
git clone https://github.com/mikebm/jestbug.git
cd jestbug
npm install
Debug the ./tests/component.test.ts file. Attempt to step through the executeSomeLogic call on test 'should return 5'.
What is the expected behavior?
When stepping through the executeSomeLogic function, your debugger should show the correct line that you are on.
Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.
Node v8.9.0
Jest 22.4.2
npm 5.6.0

Note: The repo has been updated with decorators to prove a separate ts-jest line reporting issue. This issue is still reproducible with and without the decorators.

Is this still an issue with the latest Jest and ts-jest?

Yes, it is still an issue. I just tested with jest 23.6.0.  I upgraded the test project as well.

@huafu ideas?

I first thought it was related to: https://github.com/babel/babel/issues/6577 but after some tests doesn't seem to be (@SimenB you can confirm that bare js instrumented thru jest do can be debugged step by step when coverage is on right?). In ts-jest we do patch it, but only if babel is required. I tried to patch it beforehand without success.
We do have e2e tests running over multiple versions of jest, babel and typescript for source mapping and then coverage, but I believe we do not have for both together, will drop an eye tommo.
@mikebm can you open an issue on ts-jest with the link to your repo, I'll try to look at it asap. We're about to publish a new release but it'll wait if this is indeed ts-jest related and if it can get fixed quickly.
BTW, your jest config can be simplified a lot @mikebm:
{
  "preset": "ts-jest",
  "automock": true,
  "collectCoverage": true,
  "coverageDirectory": "<rootDir>/coverage/jest",
  "coverageReporters": ["lcov", "json"],
  "resetMocks": true,
  "resetModules": true,
  "testResultsProcessor": "jest-junit-reporter"
}

There's probably a reason why we don't factor in the source map in that case.  @jwbay @aaronabramov do you know if there are?

@SimenB any news on this? I meant, I'd be happy to provide a PR but I'm not gonna try if there is a good reason for it to be like that ;-)

Feel free to send a PR! I'm not sure how it works, but if we're able to show it's more correct, no reason not to merge it 🙂

@huafu any chance of doing the PR?

@ruiaraujo I actually had to run on another thing, and my project to hurry to finish. I have started a PR, AFAIR it was not as hard as I thought, ...was harder.
So no, not finished, and now can't even remember what was the issue I was stuck with.
FYI this is my branch, but I think I started again from sratch locally. Anyway, could give you the idea of what should be done.
I'm sorry it's not in my priority because you can always run without coverage on your local while debugging and add --coverage for the final run (or the unique on CI) to have coverage.

Just curious if this is still being worked on. Looks like the code was refactored some over the past few months, so that the ScriptTransformer.ts is in the jest-transform package. For now will go with the workaround but would like to see an overall fix.

Had to revert the fix due to a regression, so reopening this one

Pushing the source maps (between source and instrumented code with coverage enabled) to Jest's cache for non-transformed code seems to break coverage reporting.
It's certainly possible to back to only pushing out source maps for transformed file, this line in the reverted PR could be this instead:
const shouldEmitSourceMaps = (!!transform && !!map)
This puts back coverage for untransformed files, although stack traces of errors from the files can pop out with the line numbers from the instrumented code. This seems to be on par with master, so it might be a path forward, if not ideal.
One thing I'm seeing here is that there's a mapCoverage bit being set when instrumenting non-transformed code: https://github.com/facebook/jest/blob/master/packages/jest-transform/src/ScriptTransformer.ts#L265
I can disable this here and restore coverage reporting although I don't know if this is safe to do.

One thing I'm seeing here is that there's a mapCoverage bit being set when instrumenting non-transformed code

If you can remove it and coverage is still correct that sounds a good path forward. It exists exactly to map coverage correctly when source maps are in play. It used to be a top level option to jest, but was moved to be inferred in #5177. which contains a bit of useful discussion.

Opened up a new PR here, the main difference is that mapCoverage is being set to false throughout ScriptTransformer. I erred on the side of staying away from doing a deep clean or redefining types that cross module boundarie.
We seem to get all of the improvements for debugging and stack traces of the original PR, without doing harm to non-transformed coverage.


I can't find the documentation. Would love to use it for my research!

I can't find the documentation and Setup.md files returns a 404 error. is there docs available?

I am working on the documentation. Here is the Setup.md file.

I’ve set up the GitHub Pages site for the documentation. You can access it here. Please check if the documentation is useful and let me know if there are any issues or improvements needed.

@kevaldekivadiya2415 the documentation is too simple and does not talk about workers and batches. Besides batches are tricky when you are sending huge payload to the model and followed by 3/4 more operations (small yet uses workers).
Then I would love if it did a benckmark between TEI library from HF and if it had Intel Gaudi hardware support.

--workers:
This argument specifies the number of worker processes to be used for batch processing. Increasing the number of workers allows the system to handle multiple batches in parallel, improving throughput, especially when processing a high volume of requests.
--batch_size:
This parameter defines the size of each batch for processing requests.

@kevaldekivadiya2415 can you provide a benchmark on 5000 rows using batch process? I tried batch processing before with TEI Gaudi HF repo and it was a disaster. It was easier to do sequentially since I could reach 8000 rows in 27 seconds.
I have super computers and limited amounts of smalls sized computers. If you could give me a ballpark what's the throughput at a given time for an 1 page worth payload and storing the results on RAM, I would be eager to checkout your library.


Clarify config.ini usage

I spent some time figuring out where to put config.ini as well as what format was supported in the [Directory] section. I tried to add a bit of info to the README.md that might helps others.

 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.

Coverage remained the same at 93.326% when pulling bbd60f52c347b0bdcdef9f3d162870d26acd7cff on petertiedemann:patch-1 into fbc7b04ef23c85d5e887053771ba9ab7276a9b35 on jmathai:master.

Thanks for this PR. Love getting updates on documentation from people who found it lacking.


bump rules_go to 0.16.5

Minor bump from 0.16.0 to 0.16.5
Signed-off-by: Christy Norman<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

To pick up https://github.com/bazelbuild/rules_go/commit/8179a1e8c7645de76f2c8a7e5f3f208ff9839a4a

I've been running this release on my instances (Linux, MacOS, arm64 / aarch64) since it came out without issue.

LGTM, but as I'm not working today I don't want to merge it in in case it breaks the build.
Dhi, if you're around feel free to merge it.


We can delete cluster using az cli 2.0 as well

Could you update this section
https://docs.microsoft.com/en-gb/azure/hdinsight/hdinsight-apps-install-custom-applications?#use-azure-classic-cli
because we can delete a cluster with az cli 2.0 as well. https://docs.microsoft.com/en-us/cli/azure/hdinsight?view=azure-cli-latest#az-hdinsight-delete
Agree, az cli 2.0 still not covering some features, but it's good enough to delete a cluster

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

ID: d9ef77ef-2efd-e707-e771-1833988df87a
Version Independent ID: c2a6c08f-4280-e8a0-14c3-2ca0d7e5a3b1
Content: Install your own custom Apache Hadoop applications on Azure HDInsight
Content Source: articles/hdinsight/hdinsight-apps-install-custom-applications.md
Service: hdinsight
GitHub Login: @hrasheed-msft
Microsoft Alias: hrasheed

@SychevIgor
Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks @SychevIgor. We have some scheduled work to re-evaluate all of our command line customer scenarios and will be sure that we include az cli 2.0 where appropriate. Given the scope of the update it may take some time, but we are aware of the issue. Thanks!

#please-close


How to get the server version number

Description
How can I get the server version number, apart from downloading the whole openapi.json file from api/v1/openapi.json?  I would not like to download the whole API schema just for the version number only.
I could create my own endpoint for that, but am I missing a more obvious way of doing it?

Would this do it for you?
...

from fastapi import __version__ as fastapi_version

...

@app.get('/version')
def version(request: Request):
    """Retrieve version information"""

    return {'version': fastapi_version}

Not realy... :)
This would expose the fastapi project version.  I am interested in the API version that fastapi is serving!
My implementation is similar though:
from myproject import __version__

router = APIRouter()

@router.get("/version", response_class=JSONResponse)
def version():
    v = {"version": __version__}
    return JSONResponse(status_code=200, content=v)

And I just update the __version__ var in the root __init__.py file.
I also have a:
app = FastAPI(
    title=config.PROJECT_NAME,
    description="Some description",
    openapi_url="/api/v1/openapi.json",
    version=__version__,
)

in my main.py file.

I am doing something similar for my app version and I am not aware of an easier solution.

Yeah, your best bet would probably be to return request.app.version in an endpoint. There is no built-in way to surface this information externally.

request.app.version is much better than my from myproject import __version__, I might steal that...!

Thanks everyone for the help! :cake:
I think that solves your question, right @stratosgear ? Do you want to close the issue?

Yes, I'm fine with this...

one liner
pip freeze | grep fastapi


How to fix common Ner Tag for all the different text file.

Hello Team,
I appreciate for building this beautiful tool for annotating the text file for NER.
I want to annotate 10000 different text file with fixed number of common Ner Tag for all the text files. I have to every time add the same Ner Tag reputedly for all text file.
Since I am using the application in my local using localhost. Can you please help me to make the code change in my local to fix the common tag for all the text file. Since I do not have much experience in front end applications, any help would really be appreciated.
I would really appreciate if you could help me to fix the tags.

@kunalchoudhary33 There is going to some feature updates in the upcoming release. I will try to include importing and exporting of tags in them. Thanks for bringing up the usecase.


Does it work for higher dimensional point? Like 4D or more?

Thank you for sharing the code. Appreciate your work.
Just a quick question: does the code work for higher dimensional point sets?
Currently I have 2 datasets containing 4d data (may be the dimension would be higher in the future). Can I use the code?
Thank you so much!

Definitely. I think the only exception is that the current implementation won't do >3d for rigid transforms - though I might be miss-remembering. I've used it for more than 3 dimensions using affine, and non-rigid methods. E.g., in my example, I added additional features to the XYZ points like the curvature of a surface mesh associated with the points.

It seems that @gattia answered this. Closing due to inactivity.


Don't work if Debug server host is setted: websocket: connection error", "Failed to connect to localhost/<IP_ADDRESS>:7007

I use wifi to sent bundle, in Dev settings, I set Debug server host to <IP_ADDRESS>:8081, and bundle was sent. But socket seems still try to connecting <IP_ADDRESS>:7007.
I think socket should read Dev setting, and try to connect <IP_ADDRESS>:7007 if I have set it.
Seems running "storybook start -h <IP_ADDRESS> -p 7007" can't help.

Ok, seems using:
const StorybookUI = getStorybookUI({port: 7007, host: '<IP_ADDRESS>'});

can do this.

But I think it's better to read Dev settings...Since this is config, prefer not hardcoded.

Well, after searching. I still don't know how to read Dev settings, or I could have made a PR...
Tell me if anybody know the way to do this.

Hi @linonetwo
Thanks for opening the issue. If you find a way to pass some config info into the React Native app at build time without hard coding it, please let me know. So far, the only I way I found is by dynamically generating a JS file when starting storybook and importing it into the RN app. I'm trying to find a better way to do this.

In Android Studio on Ubuntu localhost needs to be changed to <IP_ADDRESS> in order for storybook to work, the emulator doesn't recognize localhost

Nowadays I just read different config files when running by different npm script....

Hi
I am getting this same error I tried the solution as suggested by @linonetwo but still no success. Can anybody help ?

adb reverse tcp:7007 tcp:7007

Please open an issue over at the new repo if you're still experiencing this issue, thanks!


How can i get country code ( dialling code) from entered phone number?

How can i extract Dialing code  and number on onChangePhoneNumber?
I am getting string country iso2 but country code.

https://www.npmjs.com/package/awesome-phonenumber

use: getCountryCode


improvements for error pages

Here are my latest thoughts:

When a 500 error occurs on any endpoint, we would intercept it on base.js:23, and trigger a (new) action
A component placed at the level of BackOffice.js:13 would have a conditional display of the main content depending on the value of this action. If everything goes well, the children are displayed - nothing changes. In case of an error it would replace the pane content by the 500 error page. Similar pattern on the frontsite
That way we ensure that if something is wrong the interactive part of the interface disappears. But most importantly we don't need to implement fine grained logic for each page as it would be handled globally
This could be extended to HTTP errors other than 500; I can think of 403, 404 and 429

It would essentially look like the following with the corresponding error page displayed (again, same for the frontsite):

Originally posted by @FlorianCassayre in https://github.com/inveniosoftware/react-invenio-app-ils/pull/176#issuecomment-686551041

Also check that inveniosoftware/invenio-app-ils#213 is implemented (backend).


created name resolution local component

Description
Created a local name resolution component that does parsing of command line arguments to daprd/daprd.exe to identify different dapr instances.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: https://github.com/dapr/dapr/issues/3256
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@beiwei30 ran go mod tidy and fixed issues

@jigargandhi Thanks for this PR! Would you please fix DCO issue? You can refer to here  about how to fix it.

@Taction Sure, I will be adding one more commit as per a review comment from @artursouza , will sign off that commit.

@jigargandhi Jigar, at the same time, pls. fix go mod tidy check failure.

please review this https://github.com/dapr/components-contrib/pull/1455 PR, closing this as messed up sign-off


Overriding html-language-features in extension

I’m writing an extension that support the Django template language. The id of the language is django-html and when it’s set this way, I’m losing the nice features given by HTML Language Features bundled extension.
On the other hand, if I set the language id as html, I’ve got my syntax, the auto-indentation and others, but I (and users) are losing the ability to switch to plain html.
This issue seems similar to #28145 but it’s not about embedding HTML here, nor injecting, as far as I understood.
I see that in html-language-features/client/src/htmlMain.ts the auto-indent is loaded based on hardcoded language id html.
What would it be the best way to use these features in a different language id?
And what if I want to add more indentation rules?
I opened a question on Stack Overflow as well.

Please have a look at https://github.com/Microsoft/vscode/issues/47288.
Clearly, you want to define your own language (django) along with a text mate grammar and language configuration.
You will need code with a scanner/parser that understands the django templating language
and can extract HTML out of the django document.
Having that you can either

create this as a virtual html document and forward requests to the HTML language server.
embed libraries to your language server that handle these calls.

We also have https://github.com/Microsoft/vscode/issues/49210, which aimes for a simpler, ideally configuration based approach.

Thanks @aeschli for the pointers, I totally missed those. I’ll have a close look.

Either way, that’s a bit overwhelming for me yet.


Initial work

@EpicEric @Theodus if you have time can you review this for "it's not clear" or "you just plain forgot to update things"?

@EpicEric can you give this another review and if you see any small grammar tweaks, push them the branch and then I will merge this and roll out a first version.

@Theodus can you give this a review?


Autogenerating system and integration identifiers

Closes #3755
Description Of Changes
Updating the system and integration forms to not require a fides key/integration identifier. The system.fides_key will be derived from the system name, and the integration identifier will be derived from the system and integration type. I also updated the demo script to use the identifiers that would have been derived from the system names.
Code Changes

[ ] Added generateIntegrationKey function
[ ] Hide the fides_key and integration identifier fields until the form is saved, then show the generated identifiers

Steps to Confirm

[ ] Create a database system and add a database integration
[ ] Create a SaaS system and add a SaaS integration
[ ] Create a new system and add a manual integration
[ ] Create a new system and add an erasure email integration
[ ] Create a new system and add a consent email integration

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

It looks like this branch needs the fix from the other PR. I'm getting an error for null properties
fides  |   File "pydantic/main.py", line 1102, in pydantic.main.validate_model
fides  |     values = validator(cls_, values)
fides  |   File "/fides/src/fides/api/schemas/connection_configuration/connection_config.py", line 130, in mask_sensitive_values
fides  |     values["secrets"] = mask_sensitive_fields(
fides  |   File "/fides/src/fides/api/schemas/connection_configuration/connection_config.py", line 81, in mask_sensitive_fields
fides  |     mask_sensitive_fields(val, prop)
fides  |   File "/fides/src/fides/api/schemas/connection_configuration/connection_config.py", line 70, in mask_sensitive_fields
fides  |     secret_schema_keys = secret_schema["properties"].keys()
fides  | KeyError: 'properties'

Thanks for the feedback, I created a follow up ticket https://github.com/ethyca/fides/issues/3880


PNPM doesn't install binaries

PNPM version: 7.0.0-rc.2
NPM version: 8.6.0
NodeJs version: v16.15.1
Hi,
When doing pnpm i in node_modules/.bin there is no protoc cmd
But protoc-gen-ts is present
npx protoc ... doesn't work after that
Thanks for the work 👍

You are using a pre-release version of PNPM. The v7 has been out for a long while, latest is 7.5.0.
Pre-release versions are considered unstable, and may have bugs. It is not wise to spend time debugging problems with this version.
Can you update to the latest stable version of PNPM and try to reproduce?

PNPM: 7.5.0
You're totally right, I should've updated before.
So even after I did, I still have the issue, but the more I looked into it, the more I think this is not related to protobuf-ts.
When I run pnpm, that's what the .bin folder looks like:

And when I run npm i instead:

So protoc is missing when using pnpm, but it seems that lot of others binaries are missing too

This worked for me:

Install @protobuf-ts/proto as extra (direct) dev-dependency:

pnpm i -D @protobuf-ts/protoc

Use pnpm instead of npx/pnpx:

pnpm protoc --ts_out . --proto_path protos protos/msg-readme.proto

For the generated code to work you also have to install @protobuf-ts/runtime:

pnpm i @protobuf-ts/runtime
The extra installs of @protobuf-ts/protoc and @protobuf-ts/runtime are necessary because they are not direct dependencies in package.json. Indirect dependencies cannot be required/imported and are also not installed in the .bin folder of package.json by pnpm.
Or in other words: pnpm does not hoist indirect dependencies (by default). See also: https://pnpm.io/pnpm-vs-npm

Can you add this information to the readme?
There should be instructions for how to use this package for pnpm users.

@Zamiell, this is an open source project created by people in their spare time. Please feel free to make a contribution to add pnpm information to https://github.com/timostamm/protobuf-ts/tree/master/packages/protoc


Event fingerprinting: can we double check that event_id is not used to generate the event fingerprint?

In the test data set there are no events that have the same event_fingerprint but different event_id values.
We would expect to generate events with the same fingerprint but different event IDs if the event is duplicated before the event_id is attached to the event. Is this possible? If so, it is just unlikely? (So we don't see it in our test data set.) Let's double check that the event_id is not an input into the fingerprint, as if it is then we will not be able to identify duplicates of this type.

The event_id ("eid") is pretty definitely not an input. I think this is fine - for snplow2 it would be very rare to have an event which is duplicated prior to the attachment of the event ID. Closing...

Just as an FYI. There are rare cases where events have the same event_fingerprint but a different event_id. I came across events sent in by this bot:
Mozilla/5.0 (compatible; Yahoo! Slurp; http://help.yahoo.com/help/us/ysearch/slurp)
Most fields (including dvce_created_tstamp) were NULL, except for ip_address and useragent (thus producing the same fingerprint).

Interesting... Thanks @bogaert
On 8 Oct 2015 3:09 pm, "Christophe Bogaert"<EMAIL_ADDRESS>wrote:

Just as an FYI. There are rare cases where events have the same
event_fingerprint but a different event_id. I came across events sent in
by this bot:
Mozilla/5.0 (compatible; Yahoo! Slurp;
http://help.yahoo.com/help/us/ysearch/slurp)
Most fields (including dvce_created_tstamp) were NULL, except for
ip_address and useragent (thus producing the same fingerprint).
—
Reply to this email directly or view it on GitHub
https://github.com/snowplow/snowplow/issues/2088#issuecomment-146557013.


nbtPathQuote and nbtPathQuoteType aren't used

:tada: This issue has been resolved in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Slow loading video initially. Is there any process to minimize the loading time? Almost all issue are created which is not understandable for me. Please, give me a reference link if this issue is already close

loading time depends on the MediaPlayer, android.media.MediaPlayer or ijkplayer.


"augur command not found" and "brunch command not found"

after a fresh install on an Amazon AMI instance (Linux), neither the augur command or the brunch commands work on
make dev

or
make dev-start

Does running sudo make install-dev allow make dev and/or make dev-start to work?

There does seem to be some glitchyness with a fresh install on Ubuntu.

Normally I have to run two commands:
make dev-install-admin
This is to install files.
sudo make install-dev
It seems like all the second does is install augur 0.7.x itself.
The command "make dev" starts augur when called after these two commands
Changing all Makefile "pip" commands to "pip3" does not fix, but it is required for installation on Ubuntu 16.04 and 18.04


Enhancement: More changing modchart functions.

It would be nice if the misc tab in the modchart documentaion had more change functions. We already have changeDadCharacter() and changeBoyfriendCharacter(). So why not changeStage(), changeArrowSkin(), changeGirlfriendCharacter(), etc.

changeStage() has already been added, as well as the ability to change any character
but yeah changeArrowSkin() would be cool


config-loader - one minor improvement

@tremes mind doing a quick rebase.  i just added a commit that fixes that travis issue

@lholmquist  sure

Pull Request Test Coverage Report for Build 297

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.09%) to 99.553%

Totals

Change from base Build 296:
-0.09%

Covered Lines:
833

Relevant Lines:
834

💛  - Coveralls


[windows] The notifications always returns the same id "1"

Hi,
I've been implementing your notification service. It's great and works perfect on android.
But on windows there is a problem that the id is always 1.
You're still able to set multible notifications but they have all the same id 1.

Thanks for letting me know.  3.1.1 on nuget that will fix this up.

Thanks!


SPARK-4579 [WEBUI] Scheduling Delay appears negative

Ensure scheduler delay handles unfinished task case, and ensure delay is never negative even due to rounding

Test build #28024 has started for   PR 4796 at commit ad6713c.

This patch merges cleanly.

Test build #28024 has finished for   PR 4796 at commit ad6713c.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28024/
Test FAILed.

LGTM Thanks @srowen !

Jenkins, test this please.

Yeah this would be great to pull into an RC... a very confusing presentation in the UI right now!

Test build #28027 has started for   PR 4796 at commit ad6713c.

This patch merges cleanly.

Test build #28027 has finished for   PR 4796 at commit ad6713c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28027/
Test PASSed.

Ok I've pulled this into master and 1.3 thanks.


JDK 12 support

Is there a plan to publish a latest-jdk12 or r a jdk-12-alpine version of this image?

Why do you need a JDK12 version? This is not the JDK that will necessarily be used for builds, just for running the agent jar. You can still run BUILDs with a JDK12 even if the container is running the agent under JDK11 or JDK8.

No plan at the moment. We can definitely deliver #71, but adding more platforms to this repo is not sustainable until we do refactoring of the packaging flow. I might have time for that in September after GSoC, but hard to commit on that.
We also do not have any kind of test automation for Jenkins with JDK12. Last time I tried it on my instance, it was failing on Debian due to some process management issues.


Add wp binary diffing use case

Looks great! Other than a couple minor suggestions, I'm all for this.

Awesome. Just made those changes you suggested. I will merge it now.


Slim UI and bug fixes

Remove some of the tile drawing from the UI to hopefully increase performance
Fix out of memory crashes using fixes described in  https://github.com/AliFlux/MapTilesDownloader/issues/12
Removed some logging from server

Added to the fork on accident,  Meant to add to my new fork, closing Pull Request


Permission denied: '/opt/airflow/logs/scheduler'

What is the bug?
Using a PV to store logs. The storage is provided by longhorn. Since the PV is RWX longhorn presents the volume as an NFS mount using NFSv4. I am trying the volume mount hack, but it's failing. I think it is because NFSv4 usually defaults to everything being owned by nobody:nobody. I don't believe this is a code issue, just an implementation issue. I am hoping maybe someone has ran into issue with NFSv4 and con provide a workaround.
I am going to submit a bug with longhorn also.
What are your Helm values?
   extraInitContainers:
      - name: volume-mount-hack
        image: busybox
        command: ["sh", "-c", "chown -R 1000:1000 logs"]
        volumeMounts:
          - mountPath: /opt/airflow/logs
            name: logs-data

  logs:
    persistance:
      enabled: true

What is your Kubernetes Version?:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.6", GitCommit:"d32e40e20d167e103faf894261614c5b45c44198", GitTreeState:"clean", BuildDate:"2020-05-20T13:16:24Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.6", GitCommit:"fbf646b339dc52336b55d8ec85c181981b86331a", GitTreeState:"clean", BuildDate:"2020-12-18T12:01:36Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

What is your Helm version?:
$ helm version
version.BuildInfo{Version:"v3.3.0", GitCommit:"8a4aeec08d67a7b84472007529e8097ec3742105", GitTreeState:"dirty", GoVersion:"go1.14.7"}

I figured it out. I was relying to much on the example to be valid. The airflow user uid/group is 50000. This ulitimately resolved the issue:

extraInitContainers:
    - name: volume-mount-hack
      image: busybox
      command: ["sh", "-c", "chown -R 50000: /logs"]
      volumeMounts:
        - mountPath: /logs
          name: logs-data


Sample code in README outdated

We updated the codebase with a fix for owner withdrawing all funds but didn't update the README sample code to match this

Noted. Thanks @elie222.
I am working on something to minimize the cost of ERC721R. You will be surprise. 😄

Awesome. I will try get README update in today as it's quite important and confusing to people right now (it has the bug in it)


Time is stuck at 10 pm

Have you tried restarting chrome?

Too much time has gone without an update


Updates rabbitmq doc

This documentation for previewing a message for a document_type has been
moved out of the main rabbitmq doc in govuk-developer-docs in this PR
but we felt it was worthwhile preserving and fits into this doc nicely.
Trello:
https://trello.com/c/0OHZT3cq/458-investigate-rabbitmq-and-improve-the-documentation
:warning: This application is Continuously Deployed: :warning:

Merged changes are automatically deployed to staging and production.

Make sure you follow the guidance for deployments before you merge.

Check your branch is being deployed in the Release app, after merging.

Sure @benthorner , have updated the text. Sorry for the lazy copy!


Upgrade the fabric client to support newer versions of k8s

This PR contains 2 items.

Upgrade the k8s client to support newer versions of k8s.
Ability to override specific Metrics Monitors

We found that when we used the TaskCountStatsMonitor in the overlord config, the peon tasks would not start because they inherit the monitors from the parent process, which used to be the Middle Manager (but that would never have that monitor originally).  So now you can override this value with the following config:
druid.indexer.runner.peonMonitors

@abhishekagarwal87 what do you think?  This should bring the client up-to-date as well as fix things for those druid users running on k8s 1.25+ that were having issues.

@churromorales - can you address the build failures?

@churromorales - can you address the build failures?
I addressed them in a comment above, but maybe it was missed.

So I had tests for the DruidKubernetesPeonClient class, then i upgraded a client library and had to change some things. but the functions were all tested before. Now i would expect that if tests pass I would be good, since it was already tested before. But turns out if you have to make any changes to make the new library apis work, then code coverage complains. It is unfortunate here, because I didn't change any behavior.  I can add more tests, but they wont be meaningful here, I believe the build problems are all because I updated the library and had to make some code changes to make it work.  There are no logic changes here, except for the issue in 1.25, which I did test.

@abhishekagarwal87 you good with this now?  I explained about the test coverage in an earlier comment.

Do you also need to make this change?
--- a/extensions-contrib/kubernetes-overlord-extensions/src/main/java/org/apache/druid/k8s/overlord/KubernetesTaskRunner.java
+++ b/extensions-contrib/kubernetes-overlord-extensions/src/main/java/org/apache/druid/k8s/overlord/KubernetesTaskRunner.java
@@ -180,7 +180,7 @@ public class KubernetesTaskRunner implements TaskLogStreamer, TaskRunner
                     completedPhase = monitorJob(peonPod, k8sTaskId);
                   } else {
                     Job job = existingJob.get();
-                    if (job.getStatus().getActive() == null) {
+                    if (job.getStatus() != null && job.getStatus().getActive() == null && (job.getStatus().getFailed() != null || job.getStatus().getSucceeded() !=null)) {
                       if (job.getStatus().getSucceeded() != null) {
                         completedPhase = new JobResponse(job, PeonPhase.SUCCEEDED);
                       } else {

I believe that is the crux of the change.  I explain why here: https://github.com/apache/druid/pull/13804#discussion_r1112268989  Or let me know if I misunderstood you.

@abhishekagarwal87 Are there any additional concerns with this PR apart from the code coverage failure? From the discussion it looks like there isn't really a way to make the coverage tool pass without adding non-meaningful tests.

Or let me know if I misunderstood you.

Do you also need to make this change?
--- a/extensions-contrib/kubernetes-overlord-extensions/src/main/java/org/apache/druid/k8s/overlord/KubernetesTaskRunner.java
+++ b/extensions-contrib/kubernetes-overlord-extensions/src/main/java/org/apache/druid/k8s/overlord/KubernetesTaskRunner.java
@@ -180,7 +180,7 @@ public class KubernetesTaskRunner implements TaskLogStreamer, TaskRunner
                     completedPhase = monitorJob(peonPod, k8sTaskId);
                   } else {
                     Job job = existingJob.get();
-                    if (job.getStatus().getActive() == null) {
+                    if (job.getStatus() != null && job.getStatus().getActive() == null && (job.getStatus().getFailed() != null || job.getStatus().getSucceeded() !=null)) {
                       if (job.getStatus().getSucceeded() != null) {
                         completedPhase = new JobResponse(job, PeonPhase.SUCCEEDED);
                       } else {

I believe that is the crux of the change. I explain why here: #13804 (comment) Or let me know if I misunderstood you.

Sorry I wasn't clear. There are two spots in the code that have that same success / failure logic, I only see one in this pr, after the job is monitored. But I also see one before the job is monitored just after it is spawned in KubernetesTaskRunner. If the logic is not necessary at that point, please feel free to ignore me.

@a2l007 - Sorry if I am missing something but if CC is failing anyway, there is no need to add makingCodeCoverageHappy() test in this PR. That's the only comment I have. LGTM otherwise.

it is a good question.  I think the way I coded it was not very clear, I apologize for that.
I tried to make the KubernetesTaskRunnerTest cover the different scenarios.
Lets look at this case....
startup overlord,
1st thing it does is go to k8s and see if there are any running tasks.  If there are tasks, watch them and do bookkeeping when they finish.
The conditional block you are referring to handles that case.   It first does a check to see if the task is not active, then checks the state.  If it is still running it does the if statement you suggested (in following lines to monitorJob())`, so no need to do it twice.    It's just a short circuit in case its already finished, grab the status and return, no need to make more k8s calls.
Does that help make sense of how things are working?  These are good questions, and I want to make sure the code is doing the correct thing.

@abhishekagarwal87 The test failing are from the other PR that was merged, I don't have cycles to look and fix whatever is wrong and we honestly forked our druid so whether it goes upstream is not a big deal.
I think the thing is the other PR is using all the old libraries, and it can't deserialize the pod template anymore.  So if you want to update this feature to work with newer k8s versions, that will have to be sorted out at some point, or you can just close off this PR and worry about upgrades when the time comes.  As it stands the mm-less wont work in newer versions of k8s so, I'll leave it up to imply what they want to do.

I understand @churromorales. would you be open to giving write access on your branch to someone who wants to take this PR forward?

closing since these changes were added to #14028


Possibility to use a content-type in multiple indexes

Description
Not sure if it is a new feature request or a bug but:
Possibility to use a content-type in multiple indexes
Currently it is possible to create custom indexes and for example create a index called products with content-type Shoes and T-shirts
But once you create one you cannot re-use the content type for another index example "offers".
For my understanding it is possible to do it with meilisearch as you can add any information to any index but it is not possible with this plugin.
Basic example
An option could be to change the config of the plugin and instead write the content type name as root of the json, use instead the index name
currently the config looks like
module.exports = () => ({
  //...
  meilisearch: {
    config: {
      shirts: {
        indexName: 'products',
      },
      shoes: {
        indexName: 'products',
      },
    shirts: {
indexName: 'offers'. //THIS is not possible right now as overwrite previous index.
}
    },
  },
})

What I propose is
module.exports = () => ({
  //...
  meilisearch: {
    config: {
      products: {
        contentType: ['shoes','t-shirt']
      },
      offers: {
         contentType: ['jackets','t-shirt']
      },

    },
  },
})

Or as it is right now but not overwriting previous information and being able to manage it over the admin panel. But I do believe it would be easier the first option.

Hey @Pablo-Aldana
It is indeed a new feature request! No reason why we shouldn't implement this, it's a good idea!
I will add it to the stack of new features to add in this plugin. Meanwhile if someone wants to contribute and implement it, it can be faster!

Any news on this?

Hello @ddegooijer and anyone reading this issue
Feel free to open a PR, we are available for review!

Hello @bidoubiwa how far are we to this update ?

hello @L-Weisz
No one implemented it yet.

Hello I created a pull request to handle multiple indexes, I also have the a fork with the changes if anyone want to use it immediately

Hello @L-Weisz thank you very much for this PR
@brunoocasali will review it once he will come back from Holidays
In the meantime, everyone is more than welcome to review it and give their feedback ❤


Issue with k8s.io/docs/tasks/administer-cluster/namespaces-walkthrough/

This is a Bug Report

Problem:
Cannot pull images from --image=kubernetes/serve_hostname.
Proposed Solution:
Please change to --image=k8s.gcr.io/serve_hostname.
for example:
kubectl run snowflake --image=k8s.gcr.io/serve_hostname --replicas=2
kubectl run cattle --image=k8s.gcr.io/serve_hostname --replicas=5
Page to Update:
https://kubernetes.io/docs/tasks/administer-cluster/namespaces-walkthrough/#create-pods-in-each-namespace

Kubernetes Version: v1.15.0
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:40:16Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.0", GitCommit:"e8462b5b5dc2584fdcd18e6bcfe9f1e4d970a529", GitTreeState:"clean", BuildDate:"2019-06-19T16:32:14Z", GoVersion:"go1.12.5", Compiler:"gc", Platform:"linux/amd64"}

/language en

Hi while deplying using kubectl run cattle --image=k8s.gcr.io/serve_hostname --replicas=5
i got error "The Deployment "cattle" is invalid: spec.template.spec.containers[0].name: Invalid value: "serve_hostname": a DNS-1123 label must consist of lower case alphanumeric characters or '-', and must start and end with an alphanumeric character (e.g. 'my-name',  or '123-abc', regex used for validation is 'a-z0-9?')"

I also got the same issue as Kartik494. How to solve this?

Hi @ra257020 i think there is some issue with image , nginx run fine here
kubectl create deployment cattle --image=nginx
o/p :- deployment.apps/cattle created


gh: 2.46.0 -> 2.47.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/cli/cli/releases.
meta.description for gh is: GitHub CLI tool
meta.homepage for gh is: https://cli.github.com/
meta.changelog for gh is: https://github.com/cli/cli/releases/tag/v2.47.0
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.47.0 with grep in /nix/store/zi9y245zdpjl7liw08g31j7saa7hzwgq-gh-2.47.0
found 2.47.0 in filename of file in /nix/store/zi9y245zdpjl7liw08g31j7saa7hzwgq-gh-2.47.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
gh
gh-f
gh-notify
github-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zi9y245zdpjl7liw08g31j7saa7hzwgq-gh-2.47.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gh https://github.com/r-ryantm/nixpkgs/archive/55cb8b6f45707c1d382602489072ccde76596220.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/55cb8b6f45707c1d382602489072ccde76596220#gh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zi9y245zdpjl7liw08g31j7saa7hzwgq-gh-2.47.0
ls -la /nix/store/zi9y245zdpjl7liw08g31j7saa7hzwgq-gh-2.47.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

3 packages built:

gh
gh-f
gh-notify

Maintainer pings
cc @zowoq for testing.

Result of nixpkgs-review pr 301415 run on aarch64-darwin 1

3 packages built:

gh (github-cli)
gh-f
gh-notify


Crash when running ember tests

We are getting a crash in PhantomJS 2.0 on Linux only, OS X works fine. When running in the debugger I get the following stack trace. Do you have any feedback on how to make progress?
Program received signal SIGSEGV, Segmentation fault.
JSC::DFG::prepareOSREntry (exec=0x7fff8fc27e78, codeBlock=0x7fff726fd000, bytecodeIndex=) at dfg/DFGOSREntry.cpp:121
121             if (!entry->m_expectedValues.local(local).validate(exec->registers()[local].jsValue())) {
Missing separate debuginfos, use: debuginfo-install expat-2.0.1-11.el6_2.x86_64 fontconfig-2.8.0-3.el6.x86_64 freetype-2.3.11-14.el6_3.1.x86_64 glibc-2.12-1.149.el6_6.5.x86_64 keyutils-libs-1.4-4.el6.x86_64 krb5-libs-1.10.3-10.el6_4.2.x86_64 libcom_err-1.41.12-14.el6.x86_64 libgcc-4.4.7-3.el6.x86_64 libicu-4.2.1-9.1.el6_2.x86_64 libjpeg-turbo-1.2.1-1.el6.x86_64 libpng-1.2.49-1.el6_2.x86_64 libselinux-2.0.94-5.3.el6_4.1.x86_64 libstdc++-4.4.7-3.el6.x86_64 openssl-1.0.1e-30.el6_6.2.x86_64 zlib-1.2.3-29.el6.x86_64
(gdb) where
#0  JSC::DFG::prepareOSREntry (exec=0x7fff8fc27e78, codeBlock=0x7fff726fd000, bytecodeIndex=) at dfg/DFGOSREntry.cpp:121
#1  0x0000000001850aaa in JSC::cti_optimize (args=0x7fffffffc100) at jit/JITStubs.cpp:2061
#2  0x00007fffb20d8568 in ?? ()
#3  0x00007fff770eb570 in ?? ()
#4  0x0000000000000000 in ?? ()

I was looking at the history of the DFGOSREntry file to see if there is a change we could cherry pick. The file changed a lot a couple of revisions after the one we have. Would it be possible to upgrade to a newer Webkit revision? I am not sure what amount of work would be required.

We probably wouldn't try to fix something like this except by upgrading Webkit.
That said, it would be really helpful if you could come up with a minimized test case.  Ideally that's an entirely self-contained .js file that, when run (phantomjs testcase.js) demonstrates the same crash.  Since this appears to be a crash inside the JavaScript interpreter, that shouldn't be that hard.  If you need to load webpages in order to provoke the bug, you can do that with the "webserver" module.

Zack,
We have been trying to isolate, no success yet. What is the process for upgrading webkit? I have some cycles...
Marius

On Aug 6, 2015, at 7:43 AM, Zack Weinberg<EMAIL_ADDRESS>wrote:
We probably wouldn't try to fix something like this except by upgrading Webkit.
That said, it would be really helpful if you could come up with a minimized test case. Ideally that's an entirely self-contained .js file that, when run (phantomjs testcase.js) demonstrates the same crash. Since this appears to be a crash inside the JavaScript interpreter, that shouldn't be that hard. If you need to load webpages in order to provoke the bug, you can do that with the "webserver" module.
—
Reply to this email directly or view it on GitHub.

I haven't worked on Webkit upgrades myself; I'm just fielding bug reports.
@Vitallium or @ariya may be able to explain what's involved.

Short version:
We do the upgrade process quite easily:

download a vanilla version of QtWebkit
apply our patches (for evaluate, printing support, etc)
test all the things
do the usual Github merge process.

But this was before Qt decided to drop support for QtWebkit. The latest and final release of QtWebkit is 5.5. We will migrate to it eventually.
QtWebkit always was behind in functionality in comparison with nightly Webkit. This leads to outdated stuff.
Webkit already removed all Qt related things from the source tree. So we can't just take and use the latest Webkit. There are a lot of things changed.

@winding-lines Can you quickly try it again with PhantomJS 2.1?


Add upcoming overclout node

@donhardman - can you please provide some evidence that overclout meets the running for 30 days criteria please? thanks!

The domain registered in November, 2021 :)
The app launched on January and actually working for 30+ days but not for public at all for now.
Going to open public beta test next month, bout just would be happy to be added before to setup things before launch.
If it's impossible, nevermind.

@donhardman - please note I updated overclout's ID to 26 as elmasapp took 24

Got it, thanks! Can we use this id for NodeSource key also?


Apple Pay Gone?

Seems like it's commented out. Are you planning to re-add it?

@Updula Apple pay is still under development. Once this is done we will create a release.

@avish-bisbeehurry-cko Good to know, we will continue using prev version until this is ready.
BTW is it possible to show the Apple Pay button in a webview app, within WooCommerce checkout page? It works only on Safari but never in app.
I assume it's because using WKWebView and not SFSafariViewController - any way to bypass this? Or does the button need to be built inside the app in this case, and create the order via WC REST.


Updated README.md documentation links

Hey, thanks for the great library!
I've updated documentation links, because URL's were changed since README.md last update.

@husudosu, Hello! I really appreciate that I fixed the links mostly in the Readme file!
Thank you for spending your time on this library.


CAMEL-19587 Fix camel-jira to add assignee using the id

https://issues.apache.org/jira/browse/CAMEL-19587

Jira cloud service requires the user id instead of the user name for the assignee field
Adds more context about the authentication using personal token

(cherry picked from commit c20af246d3e58f04d14b9fc3e5a69a68c2275437)

The checkstyle fails to
ArtifactNotFoundException: Could not find artifact org.apache.camel:camel-vertx-common:jar:3.22.0-SNAPSHOT in apache.snapshots (https://repository.apache.org/snapshots/) which is unrelated to the camel jira.

Yeah manually cherry pick


UnauthenticatedHTTP2DOSMitigation default in 1.29 is set to true

In 1.28.3 this feature gate was introduced but was set to false.  In 1.29 this feature gate is set to true by default.
This resolves https://github.com/kubernetes/website/issues/44122 for dev-1.29

/retitle Document that UnauthenticatedHTTP2DOSMitigation default in 1.29 is set to true


Typescript types

It would be great to use this library with typescript, I hope you can add type definitions soon. Great library 👏

It would be great to use this library with typescript, I hope you can add type definitions soon. Great library 👏

It works with TypeScript as well. I am sharing a Nest.js TypeScript example as a reference. Please check this to see how you can use it with TypeScript. If you still encounter issues, let us know.
Nest.js example code (with TS): https://github.com/errsole/errsole.js/blob/master/examples/proxy-middleware/nest.ts
TS config example:
import * as errsole from 'errsole';
import * as ErrsoleSequelize from 'errsole-sequelize';

// Insert the Errsole code snippet at the beginning of your app's main file
errsole.initialize({
  storage: new ErrsoleSequelize({
    dialect: 'sqlite',
    storage: '/tmp/logs.sqlite'
  })
});

Checkout our documentation: https://github.com/errsole/errsole.js/blob/master/README.md

really ? I will try now, probably I did something wrong. One more question how I can initialize the object without storage I am using express with firebase admin so I want to manually log the errors on my catch blocks. Thanks in advice

really ? I will try now, probably I did something wrong. One more question how I can initialize the object without storage I am using express with firebase admin so I want to manually log the errors on my catch blocks. Thanks for the fast response

To confirm, you are using Express for your application and Firebase for your database, and you want to store your logs in Firebase, correct?

Errsole provide custom logging too. For manual logging, you can use either console.error or errsole.error, errsole.warn. In the UI dashboard, you can filter logs to view specific groups of logs you are interested in.


Scrolling while loading with section headers leaves a gap

First off, this is one of the best implementations I've seen of pull-to-refresh, very configurable and it works well. Great work. I am however experiencing what I think is a bug.
Basically, while refreshing a table view that uses section views, the pull-to-refresh view disappears when scrolling through the table. See my images below:

While loading I scroll through the table and the Pull-to-refresh view scrolls with the table while the section headers stop scrolling at the contentInset:

Is there a way to pin the pull-to-refresh view to the top of the table, rather than have it scroll with the table? I think this is the simplest solution. The alternative is to alter the contentInset while scrolling which would ensure the section headers scroll all the way to the top of the table. If you could help me out, that would be great and I can go ahead and implement this component in the rest of the app.

Thanks, i fixed this and also added scrollToTopAfterEndRefreshing which is default to YES.

Brilliant, you rock!

Hello,
I want to +1 the initial comment of mjgaylord, this implementation is really great.
Sadly, this bug is marked as resolved but I still experience it on my side. I am sure I have the right version, because the scrollToTopAfterEndRefreshing property is present.
I adapted the example code so that the list has a few sections, here is what it looks like after scrolling down a few pixels:

Then if I scroll down enough, the section header suddenly snaps in the right place. This only happens with the first header.

Thanks for the super fast merge of my fix.
Any chance a new pod version is scheduled soon with this fix included ?

Done, 1.0.10 is on pods


🗣 feat: STT & TTS

Summary
For STT, press the button or use Shift + Alt + L

For TTS, press the button (if you hold the click, you can download the audio file)

checklist
STT

[x] Browser
[x] OpenAI Whisper
[x] Local Whisper (tested on LocalAI and HomeAssistant Whisper)
[ ] Azure Whisper (not tested yet but it should work)
[x] All the OpenAI compatible STT

TTS

[x] Browser
[x] Elevenlabs
[x] OpenAI TTS
[x] Piper
[x] Coqui
[x] All the OpenAI compatible TTS

TODO:

[x] fix hark 🤔
[x] improve STTBrowser error handling
[ ] handle audio files in the file upload and automatically transcribe them
[ ] make the settings tab scrollable

UI

Speech TAB Explanation
NOTE: This is an explanation of how the automatic conversation works. To use it, you need to enable all of the settings in the Speech tab. This feature is still in beta, and sometimes it may not work as expected. Right now, after the AI input, I'm still not triggering the TTS call
graph TD;

    UserRequest((User Requests STT)) --> CheckLocalStorage{Check Local Storage for Engine};
    CheckLocalStorage -->|Engine Browser| AutomaticBrowser((Automatic Browser STT));
    CheckLocalStorage -->|Engine External| ExternalCheck{Check Transcription Status};
    ExternalCheck -->|Transcription Active| StopTranscription;
    ExternalCheck -->|Transcription Inactive| ListenAudio((Listen to User Audio));
    ListenAudio --> CheckAudio{Check Audio Level};
    CheckAudio -->|Below Threshold| SaveAudio;
    CheckAudio -->|Above Threshold| ContinueRecording;
    SaveAudio --> DataProviderRequest((Data Provider Request));
    DataProviderRequest --> APICall("/api/files/stt");
    APICall -->|Success| SetText((Set Text in Text Area));
    SetText -->|Auto Send Text Enabled| AutoSendRequest((Auto Send Text Request));
    AutoSendRequest --> APICall2("/chat/completions");
    APICall2 -->|Success| TriggerTTS((Trigger TTS));
    TriggerTTS --> TTSRequest((TTS Request));
    TTSRequest --> APICall3("/api/files/tts");
    APICall3 -->|Success| PlayAudio((Play Audio));
    PlayAudio -->|Playback Finished| WaitTwoSeconds;
    WaitTwoSeconds --> RepeatSTT((Repeat STT Trigger));

    subgraph Loop
    RepeatSTT --> ListenAudio;
    end

    StopTranscription((Stop Transcription));

thank you @bsu3338 for the integrated browser STT & TTS
thank you @szkiu for the Azure STT #2025

Change Type

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Testing
Checklist

[x] My code adheres to this project's style guidelines
[x] I have performed a self-review of my own code
[x] I have commented in any complex areas of my code
[x] I have made pertinent documentation changes
[x] My changes do not introduce new warnings
[x] I have written tests demonstrating that my changes are effective or that my feature works
[x] Local unit tests pass with my changes
[x] Any changes dependent on mine have been merged and published in downstream modules.

@Berry-13 Thank-you for finishing it. I was just about to take another look at it, but am glad you are. Congrats to the whole team on the github trending!

@Berry-13 Thank-you for finishing it. I was just about to take another look at it, but am glad you are. Congrats to the whole team on the github trending!

you're welcome 😊

Is this still in draft?

Is this still in draft?

yes

Good that you added azure stt from @szkiu !
Now what are we waiting to merge this?
Any way we can help?

Good that you added azure stt from @szkiu !
Now what are we waiting to merge this?
Any way we can help?

there are still some things to add, fix merge conflicts, and also fix the TTS since right now it's not sending correctly the buffer to the client

Would be nice to add deepgram.io , it has whisper models as well, but it's much faster and 200 minutes per month free

Would be nice to add deepgram.io , it has whisper models as well, but it's much faster and 200 minutes per month free

sure, I'll take a look at this but in another PR. I already want to add the Google cloud STT & TTS, I'll try to add this too

@Berry-13
Any update on this PR?

@Berry-13 Any update on this PR?

Hey there! the PR is all set to go! The only thing left is to test out the Azure Whisper feature, but I'm still waiting on the key for that. Unfortunately, until I have it, there isn't much more I can do at the moment. @danny-avila mentioned he'll be reviewing it within the next few weeks

Great job @Berry-13
My initial comments are just form glancing at the code through github.
I will do a more thorough review once you make the changes and I pull down the code for testing.

Eagerly waiting for this PR

Same here. Would really help orgs that cater to employees with disabilities or ADA compliance requirements

Is this possible to integrate whisper load locally or using inference framework such as triton inference server?

Is this possible to integrate whisper load locally or using inference framework such as triton inference server?

oops, not sure why I missed this ping. You can just run Whisper locally with LocalAI and then pass the URL into the librechat.yaml file

Eagerly waiting for this PR

I built this branch and have the button there as shown in the pic and i can speak and it outputs to the console correctly.  However, I can figure out how to get ElevenLabs api working so that my ai can talk back.  I see the button to press under the message but it has no effect..  Can you give me some directions on how to finish getting this going??  Thank you!

Benefit from merging the feature now to main > Benefit from waiting 1 more month to add new features

Benefit from merging the feature now to main > Benefit from waiting 1 more month to add new features

thats no fun man...  Lol. I guess i hear you though. Props on this feature however!.  Yall are doing a good job with this project

Benefit from merging the feature now to main > Benefit from waiting 1 more month to add new features

The benefit of the current plan is less maintenance and work on a new feature which would delay planned updates.

I also advise against merging this into a fork because there are changes yet to be done in this PR

Thank you @berry-13 for continuing to work on this

@kneelesh48
@mf
@xixingya
@Fakamoto
hello you four. i don't want (didn't want) to wait any longer either, so i downloaded the branch locally, built it myself and imported the image into my docker system. done.
if any of you use docker, i have uploaded the image to a filehoster (unfortunately it only works for 21 days and max. 50 downloads). So if any of you have no idea how docker images work but would like to use this wonderful tts/sst function and have it running on docker: Here you can download the latest TTS/SST Librechat image that I have created. (note that this image will NOT be updated and should only be a temporary solution until danny merges).
I hope I can help some of you.
I can say: TTS/SST works great!
For those who want to do it themselves:
git clone -b Speech-to-Text https://github.com/danny-avila/LibreChat.git
cd LibreChat
(apt install docker.io)
docker build -t librechat_tts .
(docker save --output /path/to/librechat_tts.tar librechat_tts)

...and that's it.
you only have to do the last step in () if you want to export the image. you will then get the same image as you get from my link above.
(berry and danny, if this is a problem please delete this comment)
have a nice day guys :)

@kneelesh48 @mf @xixingya @Fakamoto
hello you four. i don't want (didn't want) to wait any longer either, so i downloaded the branch locally, built it myself and imported the image into my docker system. done.
if any of you use docker, i have uploaded the image to a filehoster (unfortunately it only works for 21 days and max. 50 downloads). So if any of you have no idea how docker images work but would like to use this wonderful tts/sst function and have it running on docker: Here you can download the latest TTS/SST Librechat image that I have created. (note that this image will NOT be updated and should only be a temporary solution until danny merges).
I hope I can help some of you.
I can say: TTS/SST works great!
For those who want to do it themselves:
git clone -b Speech-to-Text https://github.com/danny-avila/LibreChat.git
cd LibreChat
(apt install docker.io)
docker build -t librechat_tts .
(docker save --output /path/to/librechat_tts.tar librechat_tts)

...and that's it. you only have to do the last step in () if you want to export the image. you will then get the same image as you get from my link above.
(berry and danny, if this is a problem please delete this comment)
have a nice day guys :)

thanks

PS. Is there any forms of TTS that is working you could give me a hint on? Even if they are beta solutions

thanks! yes,
Local:

TTS: Piper
STT: Whisper-Base

External (paid):

TTS: ElevenLabs
STT: OpenAI Whisper

@berry-13 when do you think you will be completely finished with this pr so that it is ready to merge?

@berry-13 when do you think you will be completely finished with this pr so that it is ready to merge?

when I commit, it means the changes are ready for merging. But since Danny mentioned he's going to refactor and fix some things, I'll continue until he begins reviewing it. Besides, I'll be working with him to ensure the Conversation Mode works properly since it's only partially functional at the moment

@berry-13 have you added support for Azure and GCP TTS in this PR?
Those are the OG TTS models. Also, eleven labs is expensive and I don't like their subscription pricing model.

@berry-13 have you added support for Azure and GCP TTS in this PR? Those are the OG TTS models. Also, eleven labs is expensive and I don't like their subscription pricing model.

I personally use Elevenlabs. It has websocket support and one of the best TTS models out there. I can't add Azure TTS because I don't have a key (I can't). Google TTS is planned, and I'm working on adding support for multiple providers. I'll also be adding some other providers in the future

@berry-13 I can provide you an azure key

FYI: The current implementation crashes the whole application on login in Firefox.
Unexpected Application Error!
SpeechRecognition is not a constructor

initializeSpeechRecognition@http://localhost:3090/src/hooks/Input/useSpeechToTextBrowser.ts:2127:25
useSpeechToTextBrowser/<@http://localhost:3090/src/hooks/Input/useSpeechToTextBrowser.ts:2150:25
``

SpeechRecognition

oh, thank you for reporting this!
I'm going to fix this now

For those patiently following this feature, we are looking to finish this by end of the week


Discrepancy in validation with JSON Schema draft 2020-12

With this schema, which defines an object with a property test that contains an array of objects that can have certain properties, I am having issues where Taplo should be invalidating a test case but it isn't. This is probably due to something with the new property introduced in this draft, unevaluatedProperties (see this discussion about it).
{
	"$schema": "https://json-schema.org/draft/2020-12/schema",
	"type": "object",
	"properties": {
		"test": {
			"type": "array",
			"items": {
				"$comment": "This schema is closed, and any property used which aren't defined are invalid/not allowed.",
				"type": "object",
				"properties": {
					"foo": { "type": "string" }
				},
				"allOf": [{ "$ref": "#/$defs/bazExtension" }],
				"unevaluatedProperties": false
			}
		}
	},
	"$defs": {
		"bazExtension": {
			"$comment": "This schema can be used as an extension",
			"type": "object",
			"properties": {
				"baz": { "type": "boolean" }
			}
		}
	}
}

These tests should be valid, with any combination of foo and baz together or separate:
{
	"test": [
		{
			"foo": "foo",
			"baz": true
		}
	]
}

{
	"test": [
		{
			"foo": "foo"
		}
	]
}

{
	"test": [
		{
			"baz": true
		}
	]
}

[[test]]
foo = "foo"
baz = true

[[test]]
foo = "foo"

[[test]]
baz = true

And these should be invalid, with an additional property not in the definition:
{
	"test": [
		{
			"foo": "foo",
			"baz": true,
			"otherProperty": true
		}
	]
}

[[test]]
foo = "foo"
baz = true
otherProperty = true

You can test these on https://json-everything.net/json-schema/ for JSON and with taplo obviously for TOML. My issue is that Taplo isn't invalidating the last example with the additional property otherProperty when it should be.

Thanks for the report! However, I don't think this can be fixed by Taplo. Taplo is using jsonschema for validation and I can see 2 possibly relevant open issues: https://github.com/Stranger6667/jsonschema-rs/issues/44 and https://github.com/Stranger6667/jsonschema-rs/issues/195. I assume that once those issues are fixed, we would need to update the jsonschema dependency version to benefit from them and thus fix this issue.

Ah okay sorry. I mistakenly assumed Taplo had its own validator! I'll check those issues out.

It looks like unevaluatedProperties should have been implemented in https://github.com/Stranger6667/jsonschema-rs/pull/419 and https://github.com/Stranger6667/jsonschema-rs/pull/423 and we are on the latest version? Hmmm...

The fix is simple (see #498). However, I have no idea of the implications and why this is not automatic. Maybe it's because the draft202012 feature is not complete yet? Maybe once https://github.com/Stranger6667/jsonschema-rs/issues/195 is fixed it would be automatic.

Huh alright. Thanks for taking a look.

However, I have no idea of the implications and why this is not automatic. Maybe it's because the draft202012 feature is not complete yet?

Exactly this. As the main author of jsonschema-rs states here:

I think some changes could be done and merged, but we may not expose them (e.g. new keyword impls) until their respective drafts are ready. So, there should be no problem with accumulating changes step-by-step. Though, we can still test them high-level via some cfg(test) items - e.g. conditional Draft201909 enum variant.

How about switching from jsonschema-rs to boon?
It looks like it is 100% compatible with Draft 2020-12, 2019-09, Draft 7, Draft 6 and Draft 4.
https://bowtie.report/#/implementations/rust-boon

jsonschema-rs 0.19.0 just got released which enabled the draft201909 and draft202012 features by default.
Draft implementation status/progress details are found here.

Folks, FYI, in jsonschema==0.21.0 only unevaluatedItems & vocabulary are missing (other non-optional tests from the JSON Schema test suite depend on unevaluatedItems). I am going to implement unevaluatedItems in the next few releases + will make Draft 2020-12 default. Thank you for your patience.


IGNITE-11678 Forbidding joining persistence node to in-memory cluster

The ticket is resolved, closing PR.
Please close PRs for your resolved tickets and other non-needed PRs.
Alternatively, ask a committer

to use GitHub UI (committer can associate accounts here: https://gitbox.apache.org/setup/ )
or /scripts/apply_pull_request.sh from Apache Ignite source.
or mention in commit message 'Fixes #(a number of PR)'.

These methods will automatically close PR.
Feel free to reopen this PR if it is actual.


BUGFIX: handle EOF restart

When you receive EOF on the socket, you will continue to receive EOF, and it spams the console with logging messges. If you stop and restart gecko while you are listing on the socket this is reproducable.
Push the processing into a loop so the defer can close the socket, then on EOF, restart the socket connection.

Perfect, this appears to have been the last piece at letting Ortelius totally right itself after service outages. The entire dev_env can be torn down and recreated and the Ortelius stack will converge back to a working state once the services are available again.


Add MacBook Pro 5,3 (Mid 2009) to tested hardware

The script also works on my MacBook Pro 5,3, which has a dual Nvidia configuration: an integrated 9400M and a discrete 9600M GT.

Thank you!


Solutions Guide

New section for more general information on best practices and technical approaches.

@MarkvanMents should this be closed since work is ongoing on #3696?

@MarkvanMents should this be closed since work is ongoing on #3696?

#3696 is a PR on this fork - I think we might lose something if I close this PR - and I cant merge this branch into David's as I don't have the ability to write to write to his repo.
Therefore, I think it has to stay for now?
Unless you can think of a way to merge the two?

#3696 rebased on development so this PR can be closed.


spnavcfg: fix darwin build

Motivation for this change
ZHF: #122042
cc @NixOS/nixos-release-managers
Hydra failure: https://hydra.nixos.org/build/142944495/nixlog/1
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123566 at 92d0a489 run on x86_64-linux 1

2 packages built successfully:

spnavcfg
tests.trivial


Rename methods on ArtifactTransformOutputs for better consistency

Change the methods on ArtifactTransformOutputs so it allows only registering a dir or a file. The paths will be resolved relative to the workspace.
This makes ArtifactTransformOutputs similar to TaskOutputs and will allow to create the dir on registration or the parent directories of a registered file.

@lptr Please take another look. I changed ArtifactTransformOutputs to only have two methods: dir and file.

@lptr PTAL!

@lptr PTAL!


ionic emulate ios live reload

Since a few days I have problems with the ionic emulate ios -lcs --target="iPhone-6". Live reload and logging isn't working.
If I start ionic serve my server address is Running dev server: http://localhost:8100 if I start ionic emulate my dev server is Running dev server: http://<IP_ADDRESS>:8100although I change via ionic address the address to localhost. iOS-Simulator prevents all addresses except http://localhost...
Since the latest versions of Ionic CLI, it is not possible to connect the same app on the local IP address or localhost at the same time
OS: Mac OS X Yosemite
Node Version: v0.10.32
Cordova CLI: 4.3.0
Ionic CLI Version: 1.3.14
Xcode version: Xcode 6.1.1 Build version 6A2008a
ios-sim version: 3.1.1
Thanks for help.

Looking into this now.

Ok I've found out how to reproduce. Working on it now. Thanks for reporting!

@schnti can you try to test the latest CLI to see if it fixes the issue for you? npm install -g ionic@readline.

Hello! Is this issue finally resolve? I'm running into the same problem. How could I solve it? Thanks for the help.

I'm also still getting this problem. There seems to be no way to specify localhost when running livereload and I haven't found a way to specifically allow my IP address.