Defines media route host from configured domain.

Related to https://github.com/open-orchestra/open-orchestra-media-bundle/pull/161

:+1:

@itkg-canne : It's a good idea, but I'm wondering: what about the protocol ?

@itkg-ngilain if the protocol is'nt specified, the symfony generate an url with the current protocol :

Note: Maybe the solution is to set the scheme and the domain directly in the routing.yml

@itkg-ngilain
Host and allowed schemes can also be configured like this in the routing.yml of the application :
open_orchestra_media:
    resource: "@OpenOrchestraMediaFileBundle/Controller"
    type: annotation
    prefix: /media
    host: '%my.media.domain.parameter'
    schemes: [http]

@itkg-canne
We can not merge this PR. in this case where open-orchestra-media-file-bundle isn't used in the front application this refactor doesn't work


Page reuse

I'm thinking about how to optimize ord's initial sync. The largest table by far is of outpoints to ordinal ranges. Transaction outputs are often spent soon after they are created, and since we're only committing once ever 1000 blocks, we're often starting a write transaction, and then inserting a bunch of values which are removed before we commit.
This makes me wonder about allocator behavior under this workload. If the allocator doesn't reuse pages that are written to but then deleted during a commit, those pages would be synced to disk on the next fsync, even if they're not longer used in the committed table root.
Would it make sense for the allocator to prioritize reusing pages that were written to and then freed during the current commit, on the basis that they might be dirty, and thus flushed to disk on the next fsync anyways?
This also makes me wonder if there's a syscall that does something equivalent to "I don't care about the contents of this range of this mmap, so don't bother flushing any pending changes on the next fsync".

It already reuses pages via this mechanism: if a page is freed that is "uncommitted" (allocated during the transaction) then it is immediately freed for reuse.

This also makes me wonder if there's a syscall that does something equivalent to "I don't care about the contents of this range of this mmap, so don't bother flushing any pending changes on the next fsync".
There are some madvise flags that might do that. I'm not sure it'd be worthwhile though.  It sounds like your workload is different, because you delete and insert new data many times throughout the transaction, and therefore the page reuse should take care of most of it for you.

I did have one other idea for speeding up ord though. Can you parallelize any of your processing? I'm pretty sure I can make this work: #112

It already reuses pages via this mechanism: if a page is freed that is "uncommitted" (allocated during the transaction) then it is immediately freed for reuse.

Are pages freed in this way also preferentially used for new allocations during the commit? If not, then pages might be freed during the commit, but if they aren't reused during the commit, they would still be flushed to disk, even if the contents aren't needed anymore.

There are some madvise flags that might do that. I'm not sure it'd be worthwhile though.  It sounds like your workload is different, because you delete and insert new data many times throughout the transaction, and therefore the page reuse should take care of most of it for you.

Hmm, how expensive are syscalls? Although I have the main ordinals.com server set up to make it easy to sync a database in the background, so if it's something that isn't too hard to do on a branch, I could easily test it out. As you said though, it seems unlikely that it would be provide much of a speedup

I did have one other idea for speeding up ord though. Can you parallelize any of your processing? I'm pretty sure I can make this work: https://github.com/cberner/redb/issues/112

I might be able to do some work in parallel, but it wouldn't be much. The vast majority of the work involves a single table.
One thought is that we could do dependency analysis on the transactions in a block and break them into disjoint sets of transactions that operated on different parts of the key space, however they would still be the key space of a single table.
Would it be possible to parallelize operations on disjoint parts of the key space of a single table?

Is it possible to split the indexing to process different groups of blocks in parallel and then stitch them together?
Like thread1 process blocks 0 to 100k, thread2 processes 100k to 200k with placeholder ordinals, and then when thread1 finishes you update the state of thread2 with the real ordinal numbers now that you know them.

Not explicitly, but pages are allocated in ascending order, so you end up with reuse

Gotcha. I can imagine situations where you don't get reuse during a commit, but they're situations where you for some reason have pages left over from previous commits, that are earlier in the order than pages you use and then free, but that shouldn't often be the case for ord, because the database tends to grow with each commit.

Not sure exactly, and maybe it'd be ok on the free path, but I have a benchmark of read() vs copying from an mmap and read() > 10x slower, even if I reuse the buffer to read into.

That might involve more work than an advisory syscall that doesn't actually move any data. But I think that with the tendency to reuse pages, it probably wouldn't do much good.

Is it possible to split the indexing to process different groups of blocks in parallel and then stitch them together?
Like thread1 process blocks 0 to 100k, thread2 processes 100k to 200k with placeholder ordinals, and then when thread1 finishes you update the state of thread2 with the real ordinal numbers now that you know them.

That's an interesting thought. One problem with this is that the placeholder ordinal ranges would be much larger than actual ordinal ranges. An actual ordinal range takes 11 bytes (51 bit base, which is enough to encode the maximum ordinal value of 2.1 quadrillion, and a 33 bit delta, which is enough to encode the maximum range size of 50 BTC * 100,000,000 sats), whereas a placeholder range would have to reference a previous transaction output, which takes 36 bytes, plus an offset, which would be another 4 or 5 bytes.

I see. That's unfortunate that the placeholders would be bigger. One thing it would give you is that you could parallelize as much as you wanted, and even make it parallel across multiple disks (with multiple redb database files)

There might be something there, but I can't think of a way to do it without introducing a bunch of extra work. I'll have to think on this 🤔. Feel free to close this, unless you think the madvise thing is worth trying.

Nice, glad that's conclusive 😂


Remove unthrown Exceptions from EndpointGroupRegistryTest

I was OK-ish accept these 'remove redundant throws Exception clauses for test classes' PRs, mainly because it doesn't do any harm, but I'm not really sure if this ever increases our code quality or improves readability. Removal of throws Exception from a test method is even sometimes controversial because a developer may have to add a throws clause again when he or she updates the test code.
Unless there's a good reason for this change, I'm not going to accept 'remove redundant throws Exception clasues for test classesPRs from now on. I'd be happy to accept 'remove redundantthrows MoreSpecificException` clauses', though.
Let me leave this PR for a few more days just in case there's really good justification.


Several math operations don't work on SimdArray

Trying to play with the SimdArray<T,N> I immediatelly stubled upon the fact that Vc::sqrt(.) does not for SimdArray<T,N>. As the documentation is a bit scarce, I'm not sure whether I used it correctly, but I would have assumed, that I can use them the same way as I use the underlying Vector. It seems that there is a mechanism to support arbitrary operations on the SimdArray<T,N>, but again I was not able to figure it out from the current documentation.

Yes, sorry. I should have mentioned that SimdArray<T> is not yet on feature parity with Vector<T>. It's supposed to be. I'd be happy for pull requests here. This is mostly boring copy and paste work.

@mattkretz I would have provided a patch, if it would be clear how things are to be implemented. As the current state is totally undocumented, it is not clear to me how the two implementations of SimdArray<T,N> actually differ and what the intended way to implement such features is. Is there an operator which is working and which can serve as an example? Is it necessary to specialize the operators for the two implementations, or can this be done for both at once? Is there some black magic to get the loop over the underlying Vector<T>?

Yes, I'll improve the documentation on how SimdArray is implemented right away. You can find abs, isnan, frexp, and ldexp examples in common/simdarray.h (and simdarrayhelper.h). I.e. don't worry how SimdArray is implemented. I developed an abstraction to easily implement the interface with underlying Vector<T>.

Thanks for the pointers, I'll have a look.

OK, this is indeed straight forward. I started by implementing the sqrt, I think I will add more operators on demand:  chr-engwer/Vc@b6a4ee196bfdf5b7e713350ed622759b9d65e0ff

In the corresponding pull request #77 I implemented the first additional operator and it is straight forward. I added a very short test case, but in general it would be good to factor out the tests from vector and reuse the same tests for the

Seems to be working with clang now. GCC on my systems breaks down with an internal error in its error routine... best error to be had - not.

Merged to master. All functions that are tested in math.cpp are now tested on SimdArray as well. If you have any other missing function, please report or send a pull request.


added note for email based subsciption for the mailinglist

I thought this may be a valuable addition, as this method may be unknown to some
users.

I signed it :)

Thanks


Change call for deprecated method YarnAppClient#createLauncher in YarnTwillPreparer

In the YarnTwillPreparer there is a call to deprecated method YarnAppClient#createLauncher that passes user information. Change it to the one without user input argument.

+1


Added #isDefined to Binding

Coverage remained the same at 100.0% when pulling df8e70b1795116f20a9739eb5a047a24bacd6f4b on binding-testing into af68ae3bf1945875ca2a8ebdfbddb15b858b6498 on master.


[Bug]

Describe the bug
Upon opening the app on my OneXPlayer 2 Pro 8840U (windows 11), I get the following error immediately after boot.

To Reproduce
Click on app and error pops up almost immediately.
Expected behavior
No response
Environment
- OS: Windows 11
- Device: OneXPlayer 2 Pro 8840U
- Greenlight version: v2.3.1

Anything else?
No response
New issue

[X] I confirm that this is a new issue and not a duplicate

Can you check if the time and date are set correctly? This usually happens when the time is off for more then 5 minutes.

The time and date on the device or the app? I don’t see an option to change this on the app…

No, the time from windows. The apps takes over the time automatically

Ah! Yes, it was off by 5 minutes. That fixed it! Thanks


Installed apng, but it seems to be empty

I've had the same results with Python 2.7 and 3.6.

import apng
method_list = [func for func in dir(apng) if callable(getattr(apng, func))]
print(method_list)
[]

Specifically,

from apng import APNG
"Cannot find reference 'APNG' in apng.py."

Maybe I am missing a dependency?  I installed apng from the terminal for 3.6 and using the GUI in PyCharm for 2.7.

  I had my file named the same as the package.  It works perfectly now.


Add github project slug

Motivation
We did a demo of the IDP plugin today and we couldn't go through the entire flow because we were missing github project slug annotation.
Approach
Added the annotation to the template

📣 NOTIFICATION
You are receiving this message because we did not publish any packages.
Generated by @thefrontside/actions


refactor(raw/oio): Reorganize to allow adding more features

refactor(raw/oio): Reorganize the raw/oio to allow adding more features

Besides the title, I haven't seen any other explanations or documentation. Could you please provide more information about your changes and intentions? Since there seems to be a lot of modification, it would help the reader better understand this PR and help the release manager organize the changelog.

Besides the title, I haven't seen any other explanations or documentation. Could you please provide more information about your changes and intentions? Since there seems to be a lot of modification, it would help the reader better understand this PR and help the release manager organize the changelog.

Just re-org and some naming style polish, no other changes.

Just re-org and some naming style polish, no other changes.

Well, these changes introduce some break changes, it would be best to mention that. 🤔

Well, these changes introduce some break changes, it would be best to mention that.

Yes, you are right. This API adds breaking changes for raw APIs (no public API changed).


check for lambda timeout issue

In this ticket we diagnosed an issue where a global variable holding a cache client in the JS SDK was getting corrupted after being idle for longer than the AWS NLB timeout.
We need to be on the lookout for this in all languages that users are likely to use for Lambda development; I'll add that to standards and practices docs as part of https://github.com/momentohq/standards-and-practices/issues/36 .
But for now we should test this in the .NET SDK because we know users are writing Lambdas with it.
SST docs state that you can use SST to write C# lambdas, so hopefully we can set up pretty much the same test environment that we were using for typescript.

To invoke a Lambda function via SST, .NET Core 3.1 is required, but my laptop's architecture is arm64 and .NET Core 3.1 is only availabe for x64.
Therefore, no checking for timeout was done vis SST.
Using Amazon Lambda Tool for .NET, I deployed a function in this branch and the below is my observation:

Invoked the function, which resulted in creating a new SimpleCacheClient and succeeding set/get operations.
Waited for 6 minutes (this exceeds AWS NLB's idle connection timeout of 350 seconds) and invoked the function, no new SimpleCacheClient was created but still set/get operations succeeded (as Chris mentioned the global SimpleCacheClient variable seemed to be reused in this invocation).
Waited for another 7 minutes and invoked the function, a new SimpleCacheClient was created (as Chris mentioned probably a new Docker container was created to load/run the function so the global SimpleCacheClient variable was removed.)

@poppoerika that sounds perfect.
Did you just invoke it once after each of those wait periods?  Or did you invoke it several times?

@cprice404
I invoked a few times after each wait periods. But I didn't see any errors.


Active button icon not visible

It's hard to differentiate between active button icons because button color and icon color is not contrast
Theme: Mocha Blue

Also affects the Dolphin sidebar

It also affects the sub-prompts in Dolphin (triggered by pressing CTRL +C or CTRL + D):

Has anyone managed to fix this? I tried looking deeper into the Kvantum config, but I cannot pinpoint the setting that affects the highlighted panel text shown in my example, nor in the cases above.

Hey 👋,
We haven't had active contributions to this repository in a long time so apologies for letting this issue go stale. Recently, #14 has been merged which should resolve the issues in Dolphin and in other areas too.
I'm going to close this issue given how long its been and would encourage anyone from this issue to raise a new issue going forwards if they still exist after trying out #14.
Thanks!


CodeGen class preview

Backoffice: when using "Codegen preview class" on Document type, on the right side there is nothing.

It generally fails silently if something is configured "wrong".
IE. it doesn't manage to find the clr type for a datatype.
Check the configuration, and see if there's something in the log about code generation failing.


Possible bug with refinement is not applied when const is typed

Am I missing something or is this a bug?
Smallest example:
type Fn = (number) => void;

const foobar: Fn = x => {
  if (x === 'INVALID') {
    // this should be an error, but is not
  }
}

Shouldn't this Fn generic be the same as the following?
const foobar = (x: number) => {
  if (x === 'INVALID') {
    // this should gives an error
  }
}

flow try
flow version is 0.65.0

just in case tried with type Fn = (x: number) => void; but no difference.

Testing the equality of different types is allowed. Flow knows the condition will never evaluate to true and treats the block as "dead code". Here's an example:
const n: number = 0;

if (n === "string") {
  //
  // Dead code! Numbers don't implement `trim()`.
  //
  n.trim();
}

Thanks for this, simplified the code too much, I used union types and correct types when I noticed this.
updated try

You've typed the foobar declaration, but not the anonymous function assigned to it. Since x has no type annotation, its type can change. Does that make sense?

@idiostruct thanks for explaining this,
I don't understand why flow would treat them differently, I think I'm missing something.
type A = 1 | 2;
type Fn = (x: A) => A;

const f1: Fn = (x) => x === 3 ? 1 : 1;
const f2: Fn = (x: A) => x === 3 ? 1 : 1;
const f3: Fn = (x) => 3;

Flow gives an error for only f2 and f3, why the type is applied on return value but not on the arguments?
flow try
For comparison for example typescript raises errors for all; f1, f2, and f3.
typescript try


only namespace loaded, not the translation, maybe realated to react-i18next:: i18n.languages were undefined

Describe the bug
I am working on a little proof of concept where I used the simple example as blueprint and I am trying to import components from an external library which the content must be translated.
None of the contents wrapped with {t('...')} is translated but only the namespace is being load( i.e. error-with-status or title ot buttonText)
Occurs in next-i18next version
https://github.com/dario-pon/clone-nextI18Next-reaction-components
Steps to reproduce
npm install
npm run dev
Expected behaviour
I expect to see the translated content

I cannot run your example. I get the following error:
[ error ]<EMAIL_ADDRESS>Attempted import error: '../../utils' does not contain a default export (imported as 'i18n').
ModuleDependencyError: "export 'default' (imported as 'i18n') was not found in '../../utils'

I cannot run your example. I get the following error:
[ error ]<EMAIL_ADDRESS>Attempted import error: '../../utils' does not contain a default export (imported as 'i18n').
ModuleDependencyError: "export 'default' (imported as 'i18n') was not found in '../../utils'

Did you use the following to run the repo?
npm install
npm run dev

@isaachinman please try again, should be fixed, clone->npm install->npm run dev
in the /nl path I see only namespace and not the translated content

You're not running your server.js file at any point. Please read the documentation carefully.

I appreciate your time and your effort to build and maintain this library but the documentation is sometimes unclear and example are not self explanatory.
I would really appreciate more concrete help

@dario-pon This is OSS, not a product. If you feel the documentation can be improved, please submit a PR.
I've also written a detailed walkthrough on Medium, which can be found here.
Good luck!

You are completely right, but I think you are all building something meant to be used, aren't you? :)
I added the script "node index.js" to run a index.js file like in the example(not mentioned in the doc), error still there, just for the /nl subpaths


Ensure CI is running on all platforms

ImageJ-Legacy is a sensitive component. Given recent issues like https://github.com/imagej/imagej-legacy/issues/305 and https://github.com/imagej/imagej-legacy/issues/306 we should ensure that our tests are running on all platforms

It seems the Maven build isn't actually running on Windows:
https://github.com/imagej/imagej-legacy/actions/runs/10066110644/job/27826724022
Maybe adding shell: bash to the "Execute the build" step would help?


The events "collisionStart" & "collisionEnd" don't give an array with pair data.

So I first started testing with collisionStart and collisionEnd because my general idea is less calls the better. I didn't see pair data but when I use collisionActive I got exactly the data I needed.
If it is a bug: Have collisionStart & collisionEnd give the same data as collisionActive.
Example: https://jsfiddle.net/pnpywa6z/4/ (check the console for output)
Using 0.14.1

Try making a clone of event.pairs like this:
    Events.on(engine, "collisionStart", function(event) {
        console.log('start: ', event.pairs.slice());
    });

    Events.on(engine, "collisionEnd", function(event) {
        console.log('end: ', event.pairs.slice());
    });

I think the issue is that the event.pairs array is being reused somehow so when you expand console.log you're seeing a different version to when you first logged it.
So the above should work, I realise it's not obvious so I will mark this as improve.


Check Cloud desktop shortcut doesn't work?

Hi, please see the fourth entry in this post from the AIY forum https://www.raspberrypi.org/forums/viewtopic.php?f=114&t=196612&p=1241009#p1241009 by pendragn32
If believe he is right.
If I click on the desktop "Check Cloud" a window opens and closes in milliseconds and nothing appears to be amiss.
If however I execute the python "script" Check Cloud" calls in a standard terminal  I get this error *
pi@raspberrypi:~ $ /home/pi/AIY-voice-kit-python/checkpoints/check_cloud.py
Traceback (most recent call last):
File "/home/pi/AIY-voice-kit-python/checkpoints/check_cloud.py", line 26, in
import aiy._apis._speech  # noqa
File "/home/pi/AIY-voice-kit-python/src/aiy/_apis/_speech.py", line 24, in
import google.auth
ImportError: No module named 'google'
If I click Start dev terminal and execute the script it works
Build info: Tue Sep 12 00:36:20 UTC 2017 @ 3955cac
Dev terminal is ready! Hint: enter src/assistant_library_demo.py
(env) pi@raspberrypi:~/AIY-voice-kit-python $ /home/pi/AIY-voice-kit-python/checkpoints/check_cloud.py
Testing the Google Cloud Speech API...
Everything's set up to use the Google Cloud.
Press Enter to close...

if you directly run a bash script, and if it has some error, it will close automatically fast right, that is what is happening. The python script has to be run within the environment. Double clicking it does not make it run within the env and as a result of the error, it is closing fast! I dont see any issue here !

I do see an issue here.
Have you used the voicekit image to build your installation?
Have you followed the online instructions for building the voicekit?
Do you have the Check Cloud "thingy" on your desktop?
This is what the online instructionshttps://aiyprojects.withgoogle.com/voice/#makers-guide-3-1--change-to-the-cloud-speech-api  say
**_CHECK THAT IT WORKS CORRECTLY
On your desktop, double-click the Check Cloud icon. Follow along with the script. If everything is working correctly, you’ll see this:
The cloud connection seems to be working
If you see an error message, follow the details and try the Check Cloud script again._**
If I run the cloudspeech demo it works fine - therefore I would expect to double-click the Check Cloud icon and see the words The cloud connection seems to be working I do not see that - I don't see any message just a terminal window opening and closing very quickly.
If your contention is that it can never work - well what is the point of having the  Check Cloud icon on the desktop?

The script should work when run from the icon - I'll have a look at this.
Thanks for the report!

Cheers - the other two Desktop checks Check audio and Check WiFi work fine

@sheridat , Ok, i used a wrong phrase. Since you had reported the execution of python script outside the environment from a normal terminal i said it was not an issue, not about the desktop shortcut.
Regarding your queries about have i used voicekit, yes. Hmmm .. thats quite a bit of hammering.

This could be fixed either by changing check_cloud.py to use subprocess (like it used to) or changing the shortcut to invoke it inside the virtualenv.


Fix comparison error in RC+GPS triggered FTS failsafe check

This bug would cause loss of RC + loss of GPS to trigger a FTS when flying in non-manual modes with a good data link but no RC signal

@LorenzMeier would you mind taking a quick look at this to verify the expected behavior?

Sorry it took some time. I initially didn't understand it, but yes, it looks like this should go in as-is. I'm not sure if there was a typo or initial testing value which accidentally went in.
Do you guys do a lot of failsafe testing? We have not touched that much lately (its expected to work hence) and I'd be interested in recent test results.

No problem. I am a bit conservative and wouldn't say "a lot" of failsafe testing, but we have exercised these failsafes and they worked as expected.

I've cherry-picked your commit.


Lightning send failure not sure if it's LSP fault

receiver:

sender:

I have a node with some channels but no inbound
trying to send it 100k sats over lightning
payment times out... balance goes down temporarily and then comes back ~30 seconds later

sender and receiver jsons.zip

This is due to LDK not considering MPP payments being made when constructing other parts. Will be fixed in the next version.
-------- Original Message --------
On Jun 15, 2023, 6:23 PM, Paul Miller wrote:

receiver:
Screenshot 2023-06-15 at 6 15 17 PM
sender:
Screenshot 2023-06-15 at 6 19 24 PM
I have a node with some channels but no inbound
trying to send it 100k sats over lightning
payment times out... balance goes down temporarily and then comes back ~30 seconds later
Screenshot 2023-06-15 at 6 18 27 PM
sender and receiver jsons.zip
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Switched to AsyncKeyedLock

This PR will reduce memory allocations and memory usage.

@openbullet :)

Hey sorry I didn't have time to look into this yet, will do soon ^^'

Hey sorry I didn't have time to look into this yet, will do soon ^^'

gentle reminder :)

@openbullet asked me to try it.
And i noticed that jobs don't start as they don't load proxies. - without proxies it seems to be ok. (tho i did no extensive test without being able to use proxies)

@openbullet asked me to try it. And i noticed that jobs don't start as they don't load proxies. - without proxies it seems to be ok. (tho i did no extensive test without being able to use proxies) (Tested on native)

Hi,
What would be the easiest way to reproduce this issue please so that I could maybe debug? Or would you kindly be able to debug and tell me where the issue lies please?

@meinname

Simply try to start a job with proxies.
It wont start

@meinname I'm going to need more info to replicate this issue please. Also can you try again please?

@meinname please let me know. I've also updated to the latest version of the library with performance improvements.

Still. When starting a Job with proxies off it seems to work.
But as soon as i use proxies (tested from OB2 Proxy Manager and from File) a Job with a simple config only remains at Starting...
(only tested this time on native Client on Windows)

Would you be so kind as to dumb it down into steps with screenshots to reproduce? And can you confirm that without the PR it does not give an issue?

I don't know hot to dumb it down more - but i try
First i check out ob2 master to a new directory and publish it to a new directory.
Then i start the native client and create a new config - basically anything you want. - i just used one as simple as that one:
BLOCK:RandomInteger
  => VAR @randomIntegerOutput
ENDBLOCK

BLOCK:Keycheck
  KEYCHAIN SUCCESS OR
    INTKEY @randomIntegerOutput GreaterThanOrEqualTo 9
  KEYCHAIN FAIL OR
    INTKEY @randomIntegerOutput LessThan 9
ENDBLOCK

and add a few simple proxies on ob2 proxy tab to a new group.
As they aren't used, i just add some like <IP_ADDRESS>:9000  <IP_ADDRESS>:9001 .... or real ones or whatever.
I create a new multirun job and
select the config. - set bots to 10 - proxy mode to on - wordlist either use one via file or just use infinite
And then go to the job and start it.
It works. - job runs and you get successes and fails
then do the same but with the PR merged
And then you can see that after pressing start ob2 does nothing. Beside job status is and stays on "Starting..."

@MarkCiliaVincenti Hello, first thank for your contribution, could please resolve the merge conflict to get this reviewed ?

I took a look at the updated code and I tried my best to see how I could get the AsyncKeyedLock library to work with OpenBullet2 but I don't understand what that LoliCodeBlockInstance is.
More importantly for you guys is that the current AsyncLocker class is not just very inefficient but worse than that has obvious race conditions.
You will need to refactor your code.


docker build doesn't work out of the box

Issue description
I did a vanilla clone of the repo, and ran docker build . -f ./docker/Dockerfile -t "lightning-gpu-wheels". But failed with the following error:
Source code and tracebacks
#0 15.72 -- Performing Test CMAKE_HAVE_LIBC_PTHREAD
#0 15.96 -- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Failed
#0 15.96 -- Looking for pthread_create in pthreads
#0 16.17 -- Looking for pthread_create in pthreads - not found
#0 16.17 -- Looking for pthread_create in pthread
#0 16.41 -- Looking for pthread_create in pthread - found
#0 16.41 -- Found Threads: TRUE
#0 16.42 -- Found CUDA: /usr/local/cuda (found version "12.2")
#0 16.43 -- Found CUDAToolkit: /usr/local/cuda/include (found version "12.2.128")
#0 17.05 -- Could NOT find Python (missing: Python_INCLUDE_DIRS Python_LIBRARIES Development Development.Module Development.Embed) (found version "2.7.5")
#0 17.05 CMake Error at CMakeLists.txt:176 (message):
#0 17.05
#0 17.05
#0 17.05   Unable to find cuQuantum SDK installation.  Please ensure it is correctly
#0 17.05   installed and available on path.
#0 17.05
#0 17.05
#0 17.05 -- Configuring incomplete, errors occurred!
#0 17.07 /pennylane-lightning-gpu/pyenv3.8/lib/python3.8/site-packages/setuptools/dist.py:463: UserWarning: Normalizing '0.32.0-dev' to '0.32.0.dev0'
#0 17.07   warnings.warn(tmpl.format(**locals()))
#0 17.07 Traceback (most recent call last):
#0 17.07   File "setup.py", line 143, in <module>
#0 17.07     setup(classifiers=classifiers, **(info))
#0 17.07   File "/pennylane-lightning-gpu/pyenv3.8/lib/python3.8/site-packages/setuptools/__init__.py", line 153, in setup
#0 17.07     return distutils.core.setup(**attrs)
#0 17.07   File "/opt/_internal/cpython-3.8.17/lib/python3.8/distutils/core.py", line 148, in setup
#0 17.07     dist.run_commands()
#0 17.07   File "/opt/_internal/cpython-3.8.17/lib/python3.8/distutils/dist.py", line 966, in run_commands
#0 17.07     self.run_command(cmd)
#0 17.07   File "/opt/_internal/cpython-3.8.17/lib/python3.8/distutils/dist.py", line 985, in run_command
#0 17.07     cmd_obj.run()
#0 17.07   File "/pennylane-lightning-gpu/pyenv3.8/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 79, in run
#0 17.07     _build_ext.run(self)
#0 17.07   File "/opt/_internal/cpython-3.8.17/lib/python3.8/distutils/command/build_ext.py", line 340, in run
#0 17.07     self.build_extensions()
#0 17.07   File "/opt/_internal/cpython-3.8.17/lib/python3.8/distutils/command/build_ext.py", line 449, in build_extensions
#0 17.07     self._build_extensions_serial()
#0 17.07   File "/opt/_internal/cpython-3.8.17/lib/python3.8/distutils/command/build_ext.py", line 474, in _build_extensions_serial
#0 17.07     self.build_extension(ext)
#0 17.07   File "setup.py", line 85, in build_extension
#0 17.07     subprocess.check_call(
#0 17.07   File "/opt/_internal/cpython-3.8.17/lib/python3.8/subprocess.py", line 364, in check_call
#0 17.07     raise CalledProcessError(retcode, cmd)
#0 17.07 subprocess.CalledProcessError: Command '['cmake', '/pennylane-lightning-gpu', '-DCMAKE_LIBRARY_OUTPUT_DIRECTORY=/pennylane-lightning-gpu/build/lib.linux-x86_64-3.8/pennylane_lightning_gpu', '-DPYTHON_EXECUTABLE=/pennylane-lightning-gpu/pyenv3.8/bin/python3', '-DCMAKE_BUILD_TYPE=RelWithDebInfo', '-GNinja', '-DCMAKE_MAKE_PROGRAM=/pennylane-lightning-gpu/pyenv3.8/bin/ninja', '-DENABLE_OPENMP=OFF', '-DENABLE_CLANG_TIDY=0']' returned non-zero exit status 1.

Thanks for the note, @rht !
We'll check it out and get back to you soon.

I fixed the cuquantum not found by removing --no-deps and instead doing pip install cuquantum in https://github.com/PennyLaneAI/pennylane-lightning-gpu/blob/1e129b2e7dbc7d885b16da61b6c5b1a02e45970d/docker/Dockerfile#L18.
However, subsequently, I encountered lots of compile error, for example
#0 101.4 /pennylane-lightning-gpu/pennylane_lightning_gpu/src/algorithms/AdjointDiffGPU.hpp:370:76: error: could not convert ‘{<expression error>, <expression error>, <expression error>, <expression error>, <expression error>}’ from ‘<brace-enclosed initializer list>’ to ‘Pennylane::Pennylane::Algorithms::OpsData<double>’
#0 101.4   370 |         return {ops_name, ops_params, ops_wires, ops_inverses, ops_matrices};
#0 101.4       |                                                                            ^
#0 101.4       |                                                                            |
#0 101.4       |                                                                            <brace-enclosed initializer list>
#0 101.4 /pennylane-lightning-gpu/pennylane_lightning_gpu/src/algorithms/AdjointDiffGPU.hpp: In instantiation of ‘void Pennylane::Pennylane::Algorithms::AdjointJacobianGPU<T>::batchAdjointJacobian(const CFP_t*, int, int) [with T = double; Pennylane::Pennylane::Algorithms::AdjointJacobianGPU<T>::CFP_t = double2]’:
#0 101.4 /pennylane-lightning-gpu/pennylane_lightning_gpu/src/algorithms/AdjointDiffGPU.cpp:5:39:   required from here
#0 101.4 /pennylane-lightning-gpu/pennylane_lightning_gpu/src/algorithms/AdjointDiffGPU.hpp:441:66: error: ‘jac_local’ was not declared in this scope; did you mean ‘dt_local’?

I additionally had to specify Python_SITELIB to point to the virtualenv site-packages path.

Hi @rht
Thanks for posting. We haven't been using the docker builder process for some time as we run our own custom AMIs now through Github Actions (https://github.com/PennyLaneAI/pennylane-lightning-gpu/blob/main/.github/workflows/build_wheel_manylinux2014.yml). We will need some time to investigate what changes are needed to get this process back working, but I suspect the issue is a combination of compiler versions, changing dependencies, and updated C++ language features.

Yeah, I managed to make it work by consulting the GH Actions yml file. One difference is that manylinux2014 uses Red Hat Toolset 10, which works with the Docker image, but this is different from the GH Actions file, which uses g++-11 and gcc-11.
My changes in the CMakelists.txt:

used find_package (Python3 COMPONENTS Interpreter Development.Module) (i.e. Python3 instead of Python, Development.Module instead of Development)
added set(Python_SITELIB /pennylane-lightning-gpu/pyenv3.8/lib/python3.8/site-packages) (hardcoded to 3.8 because I wanted quick result ASAP)

In the Dockerfile, I replaced yum -y install cuda with yum -y install cuda-11-5.
With those changes, everything should work.

Hi @rht, I'm glad you managed to make it work! Thank you for sharing your solution here. Please let us know if you encounter any further issues.


Help with German umlauts

I am trying to convert this .Xmodmap configuration to keyd:
keycode 66 = Mode_switch
keycode 39 = s S ssharp
keycode 38 = a A adiaeresis Adiaeresis
keycode 30 = u U udiaeresis Udiaeresis
keycode 32 = o O odiaeresis Odiaeresis
keycode 26 = e E EuroSign cent
keycode 47 = colon semicolon semicolon

Swaps ; and :
When pressing capslock, add a layer for german umlauts

This is what I tried doing in the keyd configuration, based on what I read in the manpage about Unicode Support:
[ids]

*

[main]

capslock = layer(caps)
; = :

[shift]

o = ö
u = ü
a = Ä
s = ẞ
e = ¢
; = ;

[caps]

a = ä
o = ö
u = ü
s = ß
e = €

I also symlinked /usr/share/keyd/keyd.compose to ~/.XCompose.
What happens when pressing one of the keys, however, is that a 5-digit number appears instead of the unicode character. Setting setxkbmap -option compose:menu makes no difference.
What am I doing wrong?

Does this happen in every application? If not, might it be related to #285 ?

Yes, it happens in every application.

@Kuuhhl What version are you running (keyd -v)? What is your DE?

I am now using keyd v2.4.3 (d2aa2f7) with Hyprland as the compositor.
Umlauts are working in some Apps (for example Kitty Terminal, VSCode in XWayland-Mode) but don't work in others (like Firefox, VSCode in Wayland-Mode).
In the Apps where it doesn't work, the Umlauts turn into Codes, for example

ä turns into 02s
ö turns into 03a

Unfortunately you are running into a chromium(/electron) bug on wayland, which causes it to ignore the XCompose file. It may be possible to fix this by using ibus or fiddling with your chromium flags to force it to use GTK 4/X.
[1] https://bugs.chromium.org/p/chromium/issues/detail?id=506838
[2] #273

I've got a near identical issue except that all GUI windows in Gnome show the codes - like file manager, system settings, etc.  However foot, alacritty, xfce4-terminal and terminator displays the correct utf8 chars - but terminology and Gnome's terminal doesn't.
I assume this is a Gnome/GTK issue (and EFL in the case of terminology) but can't figure out what/where - I've enabled compose:menu in Gnome's keyboard config, .XCompose is linked...
Flatpak apps have the same issue as well as running from a chroot.
Any clues appreciated!
keyd --version
keyd v2.4.3 ()

gnome-shell-45.0-r0

I played little bit on that subject.
I'm using now: xkb variant altgr-intl
The ~/.Xcompose and option compose:menu is not required for that setup.
Compose is not used, but it works only with available keys in that layout variant, which should work for most European layouts?! (But it's not like sending any unicode char)
Symbols can be created by:
a) sending a macro with the dead key and the key
b) sending the key combination directly, that is mapped in the us(altgr-intl)
Symbol details are in the intl (altgr-intl) layouts: /usr/share/X11/xkb/symbols/us
So the sample should work like Kuuhhl wants to have it and explain the details for other setups:
# us with variant altgr-intl for symbols check:
# /usr/share/X11/xkb/symbols/us -> intl (altgr-intl)
# altgr-intl it is almost the same as intl, but the dead_ keys are out of the way
# choosing that variant adds a lot of extra keys in combination with altgr
[ids]
*

[main]
capslock = layer(caps)
; = :

[shift]
; = ;

# caps layer will send altgr combinations and X11/wayland
# will create the symbols from the variant layout
[caps]
s = G-s
# works as macro with dead_diaeresis
a = macro(G-" a)
# a = macro(G-S-apostrophe S-a)
# or directly mapping to the key
# a = G-q
u = G-y
o = G-p
# €
e = G-5
; = ;

[caps+shift]
# if using macro, it needs to be created for shift too
a = macro(G-" S-a)
# u/o works automatically, because the shift modifier is applied to the G-y/G-p, which is the shifted version
# ¢
e = G-S-c

# optional - remove dead_... keys
# just as sample, there are more:
# /usr/share/X11/xkb/symbols/us -> altgr-intl
[altgr]
# will remove from altgr and altgr+shift
grave = noop

[altgr+shift]
# keep dead_acute, but remove dead_diaresis
apostrophe = noop

PS: Not sure, if combining of unicode characters could be a more general approach, but no idea, how to do it. I couldn't even create the symbols in my terminal in wayland... ( https://en.wikipedia.org/wiki/Combining_character )

I've got a near identical issue except that all GUI windows in Gnome show the codes
[...]
I assume this is a Gnome/GTK issue

Yes, this is likely related to #273.

Flatpak apps have the same issue as well as running from a chroot.

I'm not familiar with flatpak, but if the apps are sandboxed (or running inside a chroot), they presumably won't have access to your ~/.XCompose file, which is necessary in order for the the (supporting) applications to properly interpret the generated keyd sequences.

Compose is not used, but it works only with available keys in that layout variant, which should work for most European layouts?! (But it's not like sending any unicode char)
[...]
Symbols can be created by:
a) sending a macro with the dead key and the key

Indeed, this is similar to how unicode support works under the hood. keyd ships a giant XCompose file containing most desirable symbols and then generates the necessary sequences to produce the character. Unfortunately XCompose support is application specific and broken in many modern toolkits (e.g GTK4), with some applications (especially on wayland) not supporting it at all.
The alternative, as you've just graciously documented, is to set your XKB layout to a specific language and the manually generate whatever sequences that layout associates with the symbol using keyd. I've been meaning to document these limitations better and find some time to experiment with alternative input methods.

Another option is find the unicode. What I do to create ù symbols is
macro(C-S-u 00f9 space)
The space at the end is optional.

Two other options to consider:
First option is find the unicode. What I do to create ù symbols is
macro(C-S-u f9 space)
The space at the end is optional.
Second option is find a key on a keyboard that creates the dead_diaeresis that you want. In my English international with dead keys that is Altgr-Shift-', so:
[ = G-S-'

Wouldn't it be possible to implement plugins (at least as Gnome extension) that provide more high-level way of "typing" in Unicode without relying on Compose? Compose works for me, but visually it's pretty bad blinking, as correct character doesn't appear immediately.


no docs on localization

There are no docs on localization, and it's possible: download locale files , include them in html, add js:
window.ParsleyValidator.setLocale('{{ STRIPPED_LOCALE }}');

I think that documentaion is needed. Maybe locale files should be included too.

@imposeren I guess localization will have to be added client side. No Django change i.e Django-parsley change should be needed for supporting localization.
http://parsleyjs.org/doc/index.html#installation-localization
Let me know if I am wrong.


Upgrade: dove, move vm version

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 53.942%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
97

Relevant Lines:
159

💛  - Coveralls


add support for go-cache backend

This adds support for a simple, ephemeral, in-memory backend. It's a pretty quick proof of concept. I don't have strong opinions for the best way to configure/signal which db backend to use.
Let me know your thoughts!
To run:
 SPIRIT_CONNECTION_URI="asdf" SPIRIT_DB_DRIVER=ephemeral make run

To keep in line with the other database drivers, I think we can just do SPIRIT_CONNECTION_URI="cache://" or something along those lines.
I do wonder if there is a better library to use since go-cache seems to be unmaintained and if it should be actually be persisted to disk or not? An ephemeral database could be very useful for testing Spacebin in a more robust fashion compared to the current setup.

SPIRIT_CONNECTION_URI="cache://" works for me, I'll make the change.
re persistence: For my use case I am looking for something that runs entirely within memory with no saved state. It makes testing easy, and also allows for stateless deployments where there is no expectation of persistence.
re: github.com/patrickmn/go-cache it's a pretty basic wrapper around go maps, I've used it in production applications several years, though it has gone fairly stagnant. There may be more modern and feature rich implementations, but this one works well for what it does. If you feel strongly, I can look into other implementations.

Sounds good. Frankly, as long as go-cache is stable, I have no problem with it.


Errors running batch files

I'm trying to run a simple bat file in my codespace, but can't get it to work for various reasons. When attempting to run a batch file by right clicking and pressing "run", I get the following output in my terminal:
@Emblin13 ➜ /workspaces/Magic-Queries-Main-Repo (main) $ cls & cd "workspaces/Magic-Queries-Main-Repo" & "workspaces/Magic-Queries-Main-Repo/codespace.bat"
[1] 17583
bash: cls: command not found
[2] 17584
bash: cd: workspaces/Magic-Queries-Main-Repo: No such file or directory
bash: workspaces/Magic-Queries-Main-Repo/codespace.bat: No such file or directory
[1]-  Exit 127                cls
[2]+  Exit 1                  cd "workspaces/Magic-Queries-Main-Repo"
VSCode also gives me a popup error saying: "Cmd.exe could not be located at C:\windows\System32\cmd.exe". My file does indeed exist in the directory above, so I'm unsure why it's failing to locate the file.
If I try to run the file using F5, I get the same popup error regarding cmd.exe, and the terminal output appears as such:
@Emblin13 ➜ /workspaces/Magic-Queries-Main-Repo (main) $ cls & cd "/workspaces/Magic-Queries-Main-Repo" & "/workspaces/Magic-Queries-Main-Repo codespace.bat"
[1] 19373
bash: cls: command not found
[2] 19374
[1]-  Exit 127                cls
bash: /workspaces/Magic-Queries-Main-Repo/codespace.bat: Permission denied

Hi,
Looks like it fails to locate cmd.exe and proceeds to create a default terminal (in your case bash), which fails to execute the .bat file.  Going to look into updating the extension to cancel the operation if cmd.exe cannot be located.
You wouldn't happen to run VS Code using WSL? That would explain why cmd.exe cannot be located as it's running in a VM.

GitHub didn't notify me for some reason, sorry about the late response. I'm running VS Code through GitHub Codespace, which I believe uses virtual machines.


add Laplace max mechanism

Closes #950

Current dependencies on/for this PR:

main

PR #934

PR #940

PR #948

PR #951   👈

PR #938

PR #684

This comment was auto-generated by Graphite.


Added missing mockery dev dependency || fixed wrong pipes

fixes #78

Don't merge it, tests a not working...

Do we use mockery in these tests?

We using Mockery in our PHPCache tests

Can you direct me to a row? I cannot find it.
Sorry.

On 20 Jun 2017, at 22:30, Daniel Bannert<EMAIL_ADDRESS>wrote:
We using Mockery in our PHPCache tests
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or mute the thread.

It only happens on my local machine -.- hmmm

@Nyholm found one https://travis-ci.org/php-cache/integration-tests/jobs/206611026

Ah okey. We need it for the simple cache bridge.
Thanks!


ability to rename foo.ts to foo/index.ts

I sometimes encounter a case where a file gets large, and I want to refactor it into multiple files, but want to keep the import the same in all consumers. For example, a utils.ts file can become a utils/ folder with an index.ts (utils/index.ts) and I can then move things out of the utils/index.ts file into the utils folder, all without affecting import statements in other places (typically when using build tools that are designed to look for index files, although I know that technically the file name change is incompatible with native ES Modules, so this wouldn't be useful in all cases, only cases with certain common setups).
The alternative is to rename the file (the refactor affects all consumers), make the new folder, then move the file into the new folder (the refactor affects all consumers again). This process is has been known to be buggy (well, a lot better lately), but also this process is more steps (the more steps, the more chance for failure).

This feature is powered by the TypeScript project. Please file a feature request upstream


Fix an intermittent test failure

Purpose
Fix test failures happen intermittently due to server startup failures.
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #21678 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #21678   +/-   ##
=======================================
  Coverage   14.72%   14.72%
=======================================
  Files          51       51
  Lines        1399     1399
  Branches      214      214
=======================================
  Hits          206      206
  Misses       1177     1177
  Partials       16       16

Continue to review full report at Codecov.

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


[Feature/extensions] Handle named wildcards (REST path parameters)

Companion PR: https://github.com/opensearch-project/opensearch-sdk-java/pull/123
Description
Adds the ability to handle REST path parameters (named wildcards).  Most changes are on the SDK repo. However, it is necessary (see Javadoc) for the prepareRequest() method to indicate which parameters were consumed.  If the user provides too many parameters, they receive an error.
In local/plugin operation, simply fetching the parameter via param(key) consumes it.  Since the parameters are being consumed remotely, this requires communicating the list of consumed parameters, which is done in the response header (normally unused and provided for purposes such as this!)
Issues Resolved
Fixes SDK #122
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

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

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

Is there a way to unit test this?

@saratvemulapalli Not really as it relies on communications from the extension.  I hope to cover it as part of my integration test issue #120.
Also, I think while this works for the "happy path" I think (but am not sure) that we need special handling for the timeout case and the exception case.  I'll handle that as part of the IT work as well.  Also full completion of this work is somewhat dependent on #111 being complete.

@dbwiddis looks like we have to rebase feature/extensions with main first and then rebase your branch with the latest feature/extensions
1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':distribution:bwc:bugfix:buildBwcLinuxTar'.
> Building 2.2.1 didn't generate expected file /var/jenkins/workspace/gradle-check/search/distribution/bwc/bugfix/build/bwc/checkout-2.2/distribution/archives/linux-tar/build/distributions/opensearch-min-2.2.1-SNAPSHOT-linux-x64.tar.gz

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
==============================================================================

2: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':distribution:bwc:minor:buildBwcLinuxTar'.
> Building 2.3.0 didn't generate expected file /var/jenkins/workspace/gradle-check/search/distribution/bwc/minor/build/bwc/checkout-2.x/distribution/archives/linux-tar/build/distributions/opensearch-min-2.3.0-SNAPSHOT-linux-x64.tar.gz

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
==============================================================================

Codecov Report

Merging #4415 (2a49d5a) into feature/extensions (14b899b) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

@@                   Coverage Diff                    @@
##             feature/extensions    #4415      +/-   ##
========================================================
- Coverage                 70.61%   70.51%   -0.11%
+ Complexity                57436    57361      -75
========================================================
  Files                      4641     4641
  Lines                    276249   276257       +8
  Branches                  40384    40386       +2
========================================================
- Hits                     195083   194796     -287
- Misses                    64847    65117     +270
- Partials                  16319    16344      +25

Impacted Files
Coverage Δ

...rch/extensions/rest/RestSendToExtensionAction.java
27.77% <0.00%> (-3.48%)
:arrow_down:

.../java/org/opensearch/node/NodeClosedException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...cluster/coordination/PendingClusterStateStats.java
20.00% <0.00%> (-48.00%)
:arrow_down:

...cluster/coordination/PublishClusterStateStats.java
33.33% <0.00%> (-37.51%)
:arrow_down:

...java/org/opensearch/threadpool/ThreadPoolInfo.java
56.25% <0.00%> (-37.50%)
:arrow_down:

...va/org/opensearch/client/sniff/SnifferBuilder.java
72.72% <0.00%> (-27.28%)
:arrow_down:

...ion/admin/cluster/node/info/PluginsAndModules.java
53.12% <0.00%> (-25.00%)
:arrow_down:

...g/opensearch/geometry/utils/StandardValidator.java
30.30% <0.00%> (-24.25%)
:arrow_down:

...rg/opensearch/index/shard/ShardSplittingQuery.java
55.72% <0.00%> (-23.67%)
:arrow_down:

...earch/client/indices/GetIndexTemplatesRequest.java
50.00% <0.00%> (-23.08%)
:arrow_down:

... and 478 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update ignore file to include ReactBuildConfig in the npm package

Summary:
Changes .npmignore file to only exclude the ReactAndroid/build directory instead of all build directories under ReactAndroid (which included the ReactAndroid/src/main/java/com/facebook/react/common/build package). This problem was caused by the newer version of NPM being used.
Closes https://github.com/facebook/react-native/issues/45204
Changelog:
[ANDROID] [FIXED] - Fixed build from source failing due to a missing file
Test Plan:
Run npm pack or npm publish -dry-run.
Before this change it includes 3774 files in the package and ReactBuildConfig isn't included. After this change it includes 3775 files in the package and ReactBuildConfig is included.

@j-piasecki can you also open a pick request for this one?


bugfix for html-entities'<' and '>'

The '<' and '>' symbols were displayed incorrectly in the image caption. Fixed that.

Fixed in #124


Codeunit 12 - Gen. Jnl.-Post Line - new OnBeforeFinishPosting event

Please add new event OnBeforeFinishPosting to Codeunit 12 (Gen. Jnl.-Post Line):
LOCAL [IntegrationEvent] OnBeforeFinishPosting(VAR GenJnlLine : Record "Gen. Journal Line")
IncludeSender & GlobalVarAccess property set to Yes
Usage:
[External] FinishPosting(VAR GenJnlLine : Record "Gen. Journal Line") IsTransactionConsistent : Boolean
///New Code
OnBeforeFinishPosting(GenJnlLine);

IsTransactionConsistent :=
  (BalanceCheckAmount = 0) AND (BalanceCheckAmount2 = 0) AND
  (BalanceCheckAddCurrAmount = 0) AND (BalanceCheckAddCurrAmount2 = 0);

OnAfterSettingIsTransactionConsistent(GenJnlLine,IsTransactionConsistent);

Signature of FinishPosting function should be changed from
[External] FinishPosting(GenJnlLine : Record "Gen. Journal Line") IsTransactionConsistent : Boolean
to
[External] FinishPosting(VAR GenJnlLine : Record "Gen. Journal Line") IsTransactionConsistent : Boolean
If change of FinishPosting signature is not possible OnBeforeFinishPosting should be called just before  FinishPosting.
Regards, Roman.

We do not provide GlobalVarAccess anymore, please specify explicitly what you need.

What country version are you working with?

Country - UK.
GlobalVarAccess is not that important. IncludeSender is important.
In this particular case after each Cust/Vend ledger application or unapplication based on application data we could generate additional G/L Entries.
The idea is:
Before application starts we record last Detailed Cust/Vend Ledger Entry No. and save it somewhere.
In OnBeforeFinishPosting event handler we analyse new Detailed Cust/Vend Ledger Entries and based on application data create new G/L Entries by Sender.InitGLEntry and Sender.InsertGLEntry
Regards, Roman.

please specify explicitly what you need.

I'm sorry, but I think that many new events introduced on user's request are too specific and not generic enough! There are many potentials events that could be used, but they do not pass required data to event handler.
I think that IncludeSender should be turned on by default allowing event handler to access sender's functionality and custom variables in future.
I see OnBeforeFinishPosting as generic place to create additional G/L Entries after journal line posting is completed.
Regards, Roman.


move pytest-lazy-fixture to dev dependencies

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

By submitting this issue, you agree to:

follow Starlite's Code of Conduct
follow Starlite's Contribution Guidelines

Description
Moves pytest-lazy-fixture from a runtime dependency to development dependencies.
Noted on https://github.com/conda-forge/starlite-feedstock/pull/43
Close Issue(s)

Thanks for the PR! Seems like this has been introduced with the latest patch. I'll get a new one out this evening.

@all-contributors add @bollwyvl code

@bollwyvl v1.51.9 is out, containing this fix!

Great, thanks for the quick turn.
Shipped 1.51.9 without patches on https://github.com/conda-forge/starlite-feedstock/pull/44
There are a couple other oddities in pyproject.toml which we template around when consuming the pyproject.toml:

mako is a hard dep, but also has a dedicated extra
thetortoise-orm compatibility pin is only listed in dev dependencies, not as an optional in the main dependencies

More broadly:

there's a bunch of sqlalchemy stuff, but there is no extra declared
it's hard to smoke test some of the extras, as they aren't actually imported until used

we just have TODOs

mako is a hard dep, but also has a dedicated extra

Looks like mako is optional to me?
https://github.com/starlite-api/starlite/blob/6ad53abd3119b8acd63be26b77e4f9b7ef6c85a6/pyproject.toml#L51

the tortoise-orm compatibility pin is only listed in dev dependencies

I think this was intentionaly, but it should also work as an extra. I'll investigate.

there's a bunch of sqlalchemy stuff, but there is no extra declared

Afaik the dev dependency is intentionally for testing only. @peterschutt?

it's hard to smoke test some of the extras, as they aren't actually imported until used
we just have TODOs

Could you elaborate on those?

Looks like mako is optional to me?

Welp, I don't know (or really trust, to be blunt) what poetry does, but the METADATA in the latest wheel looks like:
Requires-Dist: aiomcache ; extra == "memcached" or extra == "full"
Requires-Dist: anyio (>=3)
Requires-Dist: brotli ; extra == "brotli" or extra == "full"
Requires-Dist: click ; extra == "cli" or extra == "standard" or extra == "full"
Requires-Dist: cryptography ; extra == "cryptography" or extra == "jwt" or extra == "full"
Requires-Dist: fast-query-parsers
Requires-Dist: httpx (>=0.22)
Requires-Dist: importlib-metadata ; python_version < "3.10"
Requires-Dist: jinja2 (>=3.1.2) ; extra == "jinja" or extra == "standard" or extra == "full"
Requires-Dist: jsbeautifier ; extra == "cli" or extra == "standard"
Requires-Dist: mako (>=1.2.4)
Requires-Dist: msgspec (>=0.11.0)
Requires-Dist: multidict (>=6.0.2)
Requires-Dist: opentelemetry-instrumentation-asgi ; extra == "opentelemetry" or extra == "full"
Requires-Dist: picologging ; extra == "picologging" or extra == "standard" or extra == "full"
Requires-Dist: pydantic
Requires-Dist: pydantic-factories
Requires-Dist: pydantic-openapi-schema (>=1.5.0)
Requires-Dist: python-jose ; extra == "jwt" or extra == "full"
Requires-Dist: pyyaml
Requires-Dist: redis[hiredis] ; extra == "redis" or extra == "full"
Requires-Dist: rich (>=13.0.0) ; extra == "cli" or extra == "standard" or extra == "full"
Requires-Dist: structlog ; extra == "structlog" or extra == "full"
Requires-Dist: typing-extensions

There's that mako, which pip check will duly point out.
But, of course, pip check is pretty much garbage for validating [extras] (or keeping them up-to-date from successive interactive pip installs)... which brings us to...

Could you elaborate on those?

So, for example, it's easy to test if the API of the upstream is roughly correct of e.g starlite[jinja] by importing starlite.contrib.jinja. It explodes if it's wrong, and we don't ship something knowingly incompatible.
starlite[brotli], however, won't be exercised until its actually imported and used inside a function. So there's no "cheap" way to roughly gauge whether it will work, as-shipped.
I haven't looked at running the full test suite, or even if conda-forge has all the test dependencies, which would give me more warm fuzzies, but that's a task for another day...

Mako shows as optional = false in the lock file:
[[package]]
name = "mako"
version = "1.2.4"
description = "A super-fast templating language that borrows the best ideas from the existing templating languages."
category = "main"
optional = false
python-versions = ">=3.7"
files = [
    {file = "Mako-1.2.4-py3-none-any.whl", hash = "sha256:c97c79c018b9165ac9922ae4f32da095ffd3c4e6872b45eded42926deea46818"},
    {file = "Mako-1.2.4.tar.gz", hash = "sha256:d60a3903dc3bb01a18ad6a89cdbe2e4eadc69c0bc8ef1e3773ba53d44c3f7a34"},
]

compared to Alembic, another optional dep:
[[package]]
name = "alembic"
version = "1.10.2"
description = "A database migration tool for SQLAlchemy."
category = "main"
optional = true
python-versions = ">=3.7"
files = [
    {file = "alembic-1.10.2-py3-none-any.whl", hash = "sha256:8b48368f6533c064b39c024e1daba15ae7f947eac84185c28c06bbe1301a5497"},
    {file = "alembic-1.10.2.tar.gz", hash = "sha256:457eafbdc0769d855c2c92cbafe6b7f319f916c80cf4ed02b8f394f38b51b89d"},
]

I think I see the pattern - if we have a package marked as optional, but also in dev deps, it gets locked as optional = false and ends up in Requires-Dist. E.g, brotli, click, cryptography, jinja2 are all the same, and I'm sure the others would be also if I were to keep going down the list. alembic is an example of a package that isn't in dev deps and is in extras, and it is locked as optional = true and doesn't end up as Requires-Dist.
The difference with Mako is that we don't have a Mako extra - and it is listed as an optional dependency.
Do we want optional extras to end up in Requires-Dist, or not? I'm not very familiar with this level of packaging infrastructure.

the tortoise-orm compatibility pin is only listed in dev dependencies, not as an optional in the main dependencies

Given that we have the tortoise extra, we should have the optional.

there's a bunch of sqlalchemy stuff, but there is no extra declared

Can you give some more info on this please? We have the sqlalchemy extra which includes sqlalchemy and alembic.

I think I see the pattern - if we have a package marked as optional, but also in dev deps, it gets locked as optional = false and ends up in Requires-Dist

I would not expect this behaviour. Is this intended?

I would not expect this behaviour. Is this intended?

Well I could not find anything in the documentation about this being intended so I opened up an issue: https://github.com/python-poetry/poetry/issues/7747

starlite[brotli], however, won't be exercised until its actually imported and used inside a function. So there's no "cheap" way to roughly gauge whether it will work, as-shipped.

I think we should be able to build in a check for those so they raise an error at application startup when this is being configured. Raising the error on import is going to be difficult since this is intentionally dynamic.


Storage tables - support for ETags as optimistic concurrency tokens and OData JSON format for explicit property types

Current implementation of TableServiceClient only provides one operation with "IfMatch" parameter that should contain entity ETag, but it is unclear how to get ETag for the entity in the first place.
Update operations do not support ETags. Is there plan to support this feature or I should just do something about it in my own fork?
Also, is there plan to support OData JSON format for inserting and updating entities in storage tables to be able to specify explicit types for properties?

Hi Andrey, we'll need sometime to investigate this and get back to you

For further conversation on this topic, please see the issue linked above.


python312Packages.aiovlc: 0.4.1 -> 0.4.2

Diff: https://github.com/MartinHjelmare/aiovlc/compare/refs/tags/v0.4.1...v0.4.2
Changelog: https://github.com/MartinHjelmare/aiovlc/blob/v0.4.2/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 337206 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.vlc_telnet
python311Packages.aiovlc
python311Packages.aiovlc.dist
python312Packages.aiovlc
python312Packages.aiovlc.dist


Revert "[Backport 2.x] Remove static metaFields list and get version-specific values from core"

Reverts opensearch-project/security#4412 as it is causing ISM plugin failures:
https://ci.opensearch.org/ci/dbc/integ-test/2.15.0/9978/linux/arm64/tar/test-results/8315/integ-test/index-management/with-security/stdout.txt
https://ci.opensearch.org/ci/dbc/integ-test/2.15.0/9978/linux/arm64/tar/test-results/8315/integ-test/index-management/with-security/local-cluster-logs/id-0/stdout.txt)

Let's add some info around the reason for the reversion.
This is to resolve a failure seen during release integ testing in the ISM plugin.
[2024-06-15T03:57:00,516][WARN ][o.o.i.c.IndicesClusterStateService] [node_name_9200] [openactionit_index_1][0] marking and sending shard failed due to [failed to create index]
java.lang.NullPointerException: Cannot invoke "org.opensearch.index.mapper.MapperService.getMetadataFields()" because the return value of "org.opensearch.index.IndexService.mapperService()" is null
	at org.opensearch.security.configuration.SecurityFlsDlsIndexSearcherWrapper.<init>(SecurityFlsDlsIndexSearcherWrapper.java:65) ~[?:?]
	at org.opensearch.security.OpenSearchSecurityPlugin.lambda$onIndexModule$3(OpenSearchSecurityPlugin.java:697) ~[?:?]
	at org.opensearch.index.IndexService.<init>(IndexService.java:292) ~[opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.index.IndexModule.newIndexService(IndexModule.java:634) ~[opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.indices.IndicesService.createIndexService(IndicesService.java:888) ~[opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.indices.IndicesService.createIndex(IndicesService.java:775) ~[opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.indices.IndicesService.createIndex(IndicesService.java:215) ~[opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.indices.cluster.IndicesClusterStateService.createIndices(IndicesClusterStateService.java:557) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.indices.cluster.IndicesClusterStateService.applyClusterState(IndicesClusterStateService.java:292) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.cluster.service.ClusterApplierService.callClusterStateAppliers(ClusterApplierService.java:625) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.cluster.service.ClusterApplierService.callClusterStateAppliers(ClusterApplierService.java:611) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.cluster.service.ClusterApplierService.applyChanges(ClusterApplierService.java:579) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.cluster.service.ClusterApplierService.runTask(ClusterApplierService.java:502) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.cluster.service.ClusterApplierService$UpdateTask.run(ClusterApplierService.java:204) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingRunnable.run(ThreadContext.java:882) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.common.util.concurrent.PrioritizedOpenSearchThreadPoolExecutor$TieBreakingPrioritizedRunnable.runAndClean(PrioritizedOpenSearchThreadPoolExecutor.java:283) [opensearch-2.15.0.jar:2.15.0]
	at org.opensearch.common.util.concurrent.PrioritizedOpenSearchThreadPoolExecutor$TieBreakingPrioritizedRunnable.run(PrioritizedOpenSearchThreadPoolExecutor.java:246) [opensearch-2.15.0.jar:2.15.0]
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144) [?:?]
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642) [?:?]
	at java.base/java.lang.Thread.run(Thread.java:1583) [?:?]

It was seen in CloseActionIT and OpenActionIT which have integration tests around opening and closing indices.
We reproduced the NPE by closing and opening an index, but have not fully analyzed how the NPE affects FLS rules when re-opening an index.
We plan to introduce this bug fix for FLS Includes rules in a future release and add an additional test case where a closed index is re-opened and searched upon by a user that is mapped to a role with FLS Includes rules.

@DarshitChanpura Can you also update release notes?

@cwperks done. I've created an issue to track this: https://github.com/opensearch-project/security/issues/4475


Problem with StridedSlice

Hello,
I had some problems with the StridedSlice layer. I tried to slice the vector [16, 16, 6, 3] to [16, 16, 2, 3] by using as begin vector the [0, 0, 2, 0] , as end vector the [0, 0, 4, 0] and stride vector the [1, 1, 1, 1] but I received a vector with a size of [16, 2, 6, 3]. Can anyone help me?

Thanks for your comment. I tried this change but I think that the problem is on const layers.

Please read the README carefully. You can do almost anything you want.

Closed due to lack of progress.


cdist behavior with Minkowsky and np.inf

Using scipy.spatial.distance.cdist with the Minkowski distance and p=np.inf does not produce expected result. In particular, I'm not sure how cdist is producing all distance of only 1's (see Minkowski below). Using Chebyshev distance produce expected results, but they should be equivalent for $p=\infty$.
import numpy as np
import scipy.spatial as sp

a = np.array([[1.11132, 1.15693], [0.98214, 1.01018]])
b = np.array([[1.14994, 1.15386 ], [0.973258, 1.17015 ], [0.939203, 1.09205 ]])

print("a:\n{}".format(a))
print("b:\n{}".format(b))

mink = sp.distance.cdist(a, b, 'minkowski', p=np.inf)
print("Minkowski:\n{}".format(mink))

cheb = sp.distance.cdist(a, b, 'chebyshev')
print("Chebyshev:\n{}".format(cheb))

a:
[[1.11132 1.15693]
 [0.98214 1.01018]]
b:
[[1.14994  1.15386 ]
 [0.973258 1.17015 ]
 [0.939203 1.09205 ]]
Minkowski:
[[1. 1. 1.]
 [1. 1. 1.]]
Chebyshev:
[[0.03862  0.138062 0.172117]
 [0.1678   0.15997  0.08187 ]]

Scipy/Numpy/Python version information:
scipy.__version__  = 1.0.0
numpy.__version__ = 1.14.2

Thanks,
Jesse

The reason it returns incorrect results are because the implemntation doesn't special-case p=np.inf; see the implementation here:
https://github.com/scipy/scipy/blob/5e22b2e447cec5588fb42303a1ae796ab2bf852d/scipy/spatial/src/distance_impl.h#L338-L349
Two possible fixes:

Change minkowski_distance so that it falls back to chebyshev_distance_double when p is infinite

Raise a ValueError when p is infinite and tell the user to use chebyshev distance instead

I'm happy to work on either fix... any thoughts on the preferred approach?

@jakevdp seems that the first approach would be the least disruptive. I can't think of any reason to raise an error since falling back to chebyshev_distance_double provides the user with the desired answer for infinite p.


[FEATURE] Add Important checkboxes in the issue template

Description
Current

There are no checkboxes in the issue template
During labeling its difficult for project admins to label the issues
Sometimes contributor also forget to label like (GSSOC'23)

Change

I will add important checkboxes in the issue template so that it would be easy for contributror and project admin to label the issues

Screenshots

Additional information
No response

Please assign me this task.


[ENH] Full Observability Stack

Description of changes
Summarize the changes made by this PR.

New functionality

Grafana, Prometheus, Loki, Zipkin, OTEL Collector stack
Collects Traces, Metrics, Logs
Basic metrics in register_baseline_metrics
Trace ID returned on error - #1275
Configurable On/Off config options for traces, metrics and logs
chroma-client should no longer require otel deps (to be removed in future PR)

Test plan
How are these changes tested?

[x] Tests pass locally with pytest for python, yarn test for js

Documentation Changes
TBD
Screenshots of the dashboard

@beggers, as discussed, let me know if splitting the logging into a separate PR makes sense.
CC @nicolasgere


Convert to 1000m chunks

Conversion done as an example for https://github.com/Libre-TrainSim/Libre-TrainSim/pull/452

Video: https://youtu.be/SQfjWXMiTqE

Should be working now

I think we can merge this now, as it's working fine and https://github.com/Libre-TrainSim/Libre-TrainSim/pull/452 has been merged.


keybase client breaks Kingston DataTraveler Locker+ G3 software

Hello!
I'm using Kingstone DataTraveler Locker+ G3 pendrive. Unfortunately, when keybase is running I can't open Kingstone app in order to enter password and access encrypted data. I need to manually kill all keybase related processes in order to be able to do this.
I'm using latest stable Windows 10.

Wow, that's very strange, sorry about this. Would you mind running keybase log send so we can see if the logs have any hints why? Thanks!

CC @zanderz

Success! Your log ID is:
e874ef07aeb1244a4fb24b1c

Let me know if I can help you with investigating this :)

What happens exactly when you try to use your Kingston?
Your logs show an interesting mix of filesystem drivers. I suspect not all of those are specific to Kingston (you can list them by entering driverquery | find "File System"). Most likely there is a conflict with the Dokany driver we use.  I suggest uninstalling Dokan. It is likely you could then use your Kingston and the Keybase UI, but of course there would be no KBFS access.

Scenario:

Insert Kingstone
Kingston is recognized, open Kingstone app which is stored in unencrypted drive partition
The app is opening... and nothing is happening.

How should it works:

Insert Kingston
Kingson is recognized, open Kingstone app which is stored in unencrypted drive partition
App is opening, displays window with password input
After successfully entering password new Kingstone partition is visible in "My Computer"

I was able to reproduce this on a different machine with the same result.
I am able to "fix" this by killing all keybase processes kbfsdokan.exe too :-). But I need do this before inserting Kingstone USB drive.

Have you tried leaving the processes running, but uninstalling Dokan? It seems there is a conflict with that driver.


Cdxgen is not able to pull the image if docker daemon is not running

Cdxgen is not able to pull the container image if docker daemon is not running.
I am using cdxgen as part of my plugin and docker is not running on the node where the plugin is running.
Is there any plan to pull the image directly from the registry without docker? (Syft does that)

Closed in favor of discussion where the OP has been more successful.


Enable audio with user-provided voice

Integrate with XTTS2 to use your own voice!!! https://huggingface.co/coqui/XTTS-v2
Added latency is a concern and warrants proper testing.

Solved in release v0.2.3


why does codeserver show 4 processes in dashboard when it starts

Is it threads? or is it something wrong in startupcmd? ..

It's subprocess of the main code-server process


Fix submit button styling in dark mode for feedback form

Currently it's washed out:

I checked this and it seems this specifically happens when we have Leather set to a different mode than the users system.
E.g. if the users sets Leather in Dark mode but the system is different.
We are using the system setting in the sentry config.


[🐞] Error 400: redirect_uri_mismatch - OAuth Google login + NGINX reverse proxy + BunJS

Which component is affected?
Qwik City (routing)
Describe the bug
I have NGINX setup as reverse proxy for my qwik city Bun server.
When trying to login with Google, i get this error:
Error 400: redirect_uri_mismatch.
Error description shows me that callback uri contains http instead of https.
I tried adding all the different headers through NGINX, didn't help.
Tried adding headers through env file. Didn't help.
Tried adding redirectProxyUrl in<EMAIL_ADDRESS>but then I get an error:
InvalidCheck: State cookie was missing
One guy on discord seemed to solve this issue by using getOrigin function in his entry.express.ts
But I am using Bun, not express.
Looking at source code for Node middleware, I tried changing Buns middleware on lines containing: new URL(request.url) to have https. Didn't help.
Thank You in advance!
Reproduction
impossible to reproduce (bun, nginx)
Steps to reproduce
No response
System Info
<EMAIL_ADDRESS>'@auth/core'
<EMAIL_ADDRESS>nginx

Additional Information
No response

I see. I'm trying to figure out how reproduce this problem. This is really tailored on you environment.
The best thing in this kind of scenarios is to split the problems in small steps.
Is it working with preview mode in your local machine?

On my local machine, I am having http:localhost
Google auth allows http for localhost, but only https for domains.
@gioboa thank You for the qwik ( ;) ) response!

On my local machine, I am having http:localhost Google auth allows http for localhost, but only https for domains.
@gioboa thank You for the qwik ( ;) ) response!

ok, we need the https. Can you try to deploy it on a cloud vendor (Vercel, Netlify, Cloudflare, etc. etc. ) ?

Go to my website currently live: amfibija.rs/en
login page: https://amfibija.rs/en/auth
Try logging with google

I had this error

As sorry. Deployed is the solution with adding -- redirectProxyUrl: 'https://amfibija.rs/api/auth' (in<EMAIL_ADDRESS>This is an error I got logged on server when you tried logging in:

I removed redirectProxy and deployed again. Check again and you will get something like this:

Thank You a lot for trying to help! I am truly desperate with this one :D

same Errore 400: redirect_uri_mismatch error here.

Also, error details:

details: redirect_uri=http://amfibija.rs/api/auth/callback/google
Let's chat on Discord about this problem. It's something wrong in the configuration.

https://discord.com/channels/842438759945601056/1200458096322035873

@gioboa please reopen the issue. Bun adapter doesn't support setting ORIGIN

@gnemanja so I think the issue here is support ORIGIN env variable like the node one. Can you change the title pls?


ripple test faucet

can someone name a ripple faucet that works? I need some test ripple on this address
r3AeoS4gedy7YY6icSL61wGMZfjwMoBV4y

hi @kiahmed
you can use http://ripplerm.github.io/ripple-wallet/
for test payment tx.

did try that tool. looks like it can't send , got this error. Also, couldn't find the tx in explorer
Sending 20 XRP to rnSsPiFqjyRX9bagm5LExHmXExyX7fBNzo
Status:[FINAL, tejMaxLedger], Result: tefMASTER_DISABLED
tx_hash: 49B2C2FA0D04FD16957DDB920ABC4C763704789C62A19F478C195840EA472FB9

hi @kiahmed
get new address test  from https://ripple.com/build/xrp-test-net/,
update address and secret to you new test address,
and send to you new funding address from it,
this is only for test, do not use it for sending in main net.

@passionofvc I did all that and was able to make a successful tx here is the hash 83A859E91E5DB994577E2D1FB9B23A7A6CC1062CA979D29BC9CDABBC6F6C37C7  for destination r3AeoS4gedy7YY6icSL61wGMZfjwMoBV4y
unfortunately , when I query balance or account_info, or account_tx command it errors saying account don't exit almost 40 mins after transaction

@kiahmed
it seems your transaction failed with tecNO_DST.
does [currency:  "XPR"]  ok for currency?
I think it must be XRP for payment transactions.
 2018-02-23, 08:26:21 Failed transaction
Hash: 83A859E91E5DB994577E2D1FB9B23A7A6CC1062CA979D29BC9CDABBC6F6C37C7

Raw Tx:
 Object
Account:  "rBcqzVptDrcLvgrbTsbmejcGzD8GExfNR1"
 Amount: Object
currency:  "XPR"
issuer:  "rBcqzVptDrcLvgrbTsbmejcGzD8GExfNR1"
value:  "20"
Destination:  "r3AeoS4gedy7YY6icSL61wGMZfjwMoBV4y"
Fee:  "10"
Flags:<PHONE_NUMBER>
Sequence: 1
SigningPubKey:  "039DA7135DE89B52A15DA69BBDCDC7E6E1CF296ABF0606721DBC137ED633C6D8EE"
TransactionType:  "Payment"
TxnSignature:  "3045022100FD9ABEA9BE48394CD85892B56CA61FC59369F2175F1F9823EB939E446CDC7904022044374908BF2F507D7114D4B51DF16666A50F475BA41B7919CE8A26D58496E67C"
date: 572657181
hash:  "83A859E91E5DB994577E2D1FB9B23A7A6CC1062CA979D29BC9CDABBC6F6C37C7"
inLedger: 6941051
ledger_index: 6941051

Meta:
 Object
 AffectedNodes: Array [1]
 0: Object
 ModifiedNode: Object
 FinalFields: Object
LedgerEntryType:  "AccountRoot"
LedgerIndex:  "412B886710EC27F0ABE185328F872B0AADA4D960F3AD960192CB187CDD102220"
 PreviousFields: Object
PreviousTxnID:  "C80FF77D94EEB85587A09FDB810BAA3B39ABA6E194352DDB41978A8FA887751B"
PreviousTxnLgrSeq: 6880943
TransactionIndex: 4
TransactionResult:  "tecNO_DST"

but just tx call with the hash shows its succeeded . well, I definitely sent xpr because thats what I got from ripple test site. Do I have to do anything on wallet?

@kiahmed
no, your transaction failed, and also put into the ledger,
it just eat your 10 XRP fee and do nothing else.
ripple ledger also store failure transaction.
account_tx
{
  "result": {
    "account": "rBcqzVptDrcLvgrbTsbmejcGzD8GExfNR1",
    "ledger_index_max": 6959125,
    "ledger_index_min": 6881390,
    "status": "success",
    "transactions": [
      {
        "meta": {
          "AffectedNodes": [
            {
              "ModifiedNode": {
                "FinalFields": {
                  "Account": "rBcqzVptDrcLvgrbTsbmejcGzD8GExfNR1",
                  "Balance": "9999999990",
                  "Flags": 0,
                  "OwnerCount": 0,
                  "Sequence": 2
                },
                "LedgerEntryType": "AccountRoot",
                "LedgerIndex": "412B886710EC27F0ABE185328F872B0AADA4D960F3AD960192CB187CDD102220",
                "PreviousFields": {
                  "Balance": "10000000000",
                  "Sequence": 1
                },
                "PreviousTxnID": "C80FF77D94EEB85587A09FDB810BAA3B39ABA6E194352DDB41978A8FA887751B",
                "PreviousTxnLgrSeq": 6880943
              }
            }
          ],
          "TransactionIndex": 4,
          "TransactionResult": "tecNO_DST"
        },
        "tx": {
          "Account": "rBcqzVptDrcLvgrbTsbmejcGzD8GExfNR1",
          "Amount": {
            "currency": "XPR",
            "issuer": "rBcqzVptDrcLvgrbTsbmejcGzD8GExfNR1",
            "value": "20"
          },
          "Destination": "r3AeoS4gedy7YY6icSL61wGMZfjwMoBV4y",
          "Fee": "10",
          "Flags":<PHONE_NUMBER>,
          "Sequence": 1,
          "SigningPubKey": "039DA7135DE89B52A15DA69BBDCDC7E6E1CF296ABF0606721DBC137ED633C6D8EE",
          "TransactionType": "Payment",
          "TxnSignature": "3045022100FD9ABEA9BE48394CD85892B56CA61FC59369F2175F1F9823EB939E446CDC7904022044374908BF2F507D7114D4B51DF16666A50F475BA41B7919CE8A26D58496E67C",
          "date": 572657181,
          "hash": "83A859E91E5DB994577E2D1FB9B23A7A6CC1062CA979D29BC9CDABBC6F6C37C7",
          "inLedger": 6941051,
          "ledger_index": 6941051
        },
        "validated": true
      }
    ]
  }
}

okay..so whats the remedy ? I created the address few minutes before transaction

@kiahmed
you must check you dest address ,
does it exists for sending XPR.
do you set trust line for it?

until a tx is confirmed to an address it is not recognized by the network, as far as i know. what needs to be done for setting up trust line?


rename is_Mitigated to is_mitigated

came across this annoying typo and decided to fix it.
let me know what you think, might be nitpicking, might be good for readability!

This would probably need a heads-up in the release notes. I - for one - am going to be impacted with regards to scripts outside of DefectDojo.

Adding the Breaking Changes label. It's bending the label's arm a bit, dunno. Better suggestion?

hmm thought might not be too big an issue for the API, but maybe we should do this in v2.0 where some other breaking changes might take place?

Removing my approval for now to avoid a too quick merge. Let's take it on slack?

Sounds like this one will need to make it in sooner to avoid potential future headaches...


Latest release's types not working?

Excuse the horrible title, but I'm creating a PR over at trpc and some CI tool is telling me this:
$ next build
--
10:53:39.158 | info  - Loaded env from /vercel/path1/.env
10:53:39.404 | info  - Checking validity of types...
10:53:44.795 | Failed to compile.
10:53:44.795 |
10:53:44.796 | ./node_modules/zod/lib/helpers/parseUtil.d.ts:1:15
10:53:44.796 | Type error: ',' expected.
10:53:44.796 |
10:53:44.796 | > 1 \| import { type IssueData, type ZodErrorMap, type ZodIssue } from "../ZodError";
10:53:44.796 | \|               ^
10:53:44.797 | 2 \| import { type ZodParsedType } from "./util";
10:53:44.797 | 3 \| export declare const makeIssue: (params: {
10:53:44.797 | 4 \|     data: any;

I didn't have this error yesterday, so I'm assuming that this is relate to the latest release.
In fact, it looks like this github actions job is telling that there needs to be some fixes.

Shoot, sorry. I fixed this when I saw that failure but forgot to publish the new version. Fixed in 3.17.3.


3d tiles

@nileshlab did you mean to open this pull request?  The 3d-tiles branch will not be ready until the 3D Tiles spec is ready.

Closing this.  @nileshlab let me know if you had something else in mind.


Fix: Allow passing more cssModules attributes to the transform function

Hi :D
Do you need this adjustment to use the second option dashedIdents?
  /** Whether to rename dashed identifiers, e.g. custom properties. */
  dashedIdents?: boolean

https://github.com/parcel-bundler/lightningcss/blob/9b3e6f425b1132243930737f306cb864b7b892d0/node/index.d.ts#L301-L306
In that case id rather  go explicit like with the cssModulesPattern option and have a top level option cssModulesDashedIdents.

Yes, I need it for that. But my thought was to keep it close to the docs and have further options...


Error deploying web part to a SharePoint page

Hello,
Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
Web part loads on SharePoint page, accessing assets from Sharepoint Site Assets list.
Observed Behavior
Web part does not load, received error message indicating failure to load JSON asset. Exact error message:
***ERROR MESSAGE:
***Failed to load component "5b4b0e26-5361-4b19-95b0-3f088bd39d6f" (ThinkQuestionWebPart).
Original error: ***Failed to load entry point from component "5b4b0e26-5361-4b19-95b0-3f088bd39d6f" (ThinkQuestionWebPart).
Original error: Error loading https://component-id.invalid/5b4b0e26-5361-4b19-95b0-3f088bd39d6f_0.0.1
t.module(...).info is not a function
***CALLSTACK:
i/<@https://spoprod-a.akamaihd.net/files/sp-client-prod_2017-06-02.014/classic-pages-assembly_en-us_d99e510cc09bbeb6a454a0ee800b4d78.js:109:200964
Steps to Reproduce

Create new GA drop web part with latest AngularJS (v1.x).
Create Site Assets list in SharePoint. Open write-manifests.json file in web part and enter in URL to Site Assets for the cdnBasePath property.
Run "gulp bundle --ship"
Run "gulp package-solution --ship"
Navigate to ./temp/deploy folder in web part. Take the files inside and upload them to the SharePoint's Site Assets.
Upload or drag & drop the newly created client-side solution package to the app catalog in your tenant. Choose "Deploy" on pop-up window.
Open a site where you want to test the web part and go the Site contents page of the site.
Choose Add – App from the toolbar and choose the sphosted-webpart-client-side-solution app to be installed on the site.
After the solution has been installed, navigate to the page you wish to add the web part to, then add web part.
Error message is received.

Additional Info

From the network traffic, we are able to see that of the 3 asset files generated from "gulp bundle --ship", 2 are being loaded with no errors, but the JSON file is not.

We are attempting to load the web part on a SharePoint classic page.

Loading an AngularJS web part's assets from within a SharePoint Site Assets list was working in a previous SPFX drop. We're unsure exactly which drop, but we had previously created an AngularJS web part that had (a)microsoft/sp-client-base: "~0.7.0" and (a)microsoft/sp-build-web: "~0.9.0" specified in its package.json.

Web part loads correctly in local SharePoint Workbench , as well as workbench hosted in SharePoint.

I do hope that is the required information to look into the issue. If not, please let me know what other data you may require.
Best,

How are you referencing AngularJS?

We installed AngularJS (and AngularJS typings) using the commands:
npm install —-save<EMAIL_ADDRESS>npm install @types/angular --save-dev
After which we referenced AngularJS in the webpart's main entry point as:
import * as angular from 'angular';
This import statement is also used in our app-modulte.ts file, which specifies the AngularJS main module. It is used again in our AngularJS Controller file. I believe an AngularJS reference in our Controller file was providing an error when running "gulp serve". Adding in the import statement above fixed the error. I can't recall exactly which error, but I can find out should you need that information.
Please let me know if there's any other data you need.

It sounds like you're using some old versions of @microsoft/sp-client-base and @microsoft/sp-build-web. Can you update your package.json to refer to all of the latest versions of the SPFx packages?
See instructions here.

Both @microsoft/sp-client-base and @microsoft/sp-build-web are noted as versions "~1.0.0" in my package.json. I believe those are the latest versions from the GA drop documentation.

Have you defined AngularJS in the externals section of your project config or are you including it in the web part bundle?

I am including it in the web part bundle and not defining it in the externals section of my project config.

Any update on this? i'm working with @joselarios and I do not think this is a problem with the references of AngularJS. I could be wrong here but the error message is not very descriptive. We seem to be using the latest drop and it appears to be an error in Classic mode.

@juanmlarios Does this issue only affect classic pages, or does it affect modern pages as well?

@joselarios @mpasarin are you still seeing this issue?

Closing the issue. Feel free to open a new issue if you are still encountering this problem.


Create your own operators

I know that greppers allows you to create custom commands as wrapper but it would be nice if you could create your own operators as well. For example, I'd like a custom operator that adds the literal -Q flag to a search.

Hmm, the simplest solution would be adding another entry to g:grepper. E.g.
let g:grepper = {
  \ 'ag': {
  \    'grepprg': 'ag --vimgrep',
  \    'operator': 'ag --vimgrep -Q',
  \ }}

@rgrinberg ping

I gave your snippet a try but the little flaw with this is that I still have to press  at the end to run the search. A bit annoying...

Wait, the snippet above was a suggestion how to implement this, there's no operator key yet. :)
If you want the operator to search right way, use something like:
let g:grepper = { 'prompt': 0 }
nnoremap <leader>g :Grepper -prompt<cr>

Thanks @mhinz that works like a charm. The only blemish with your solution is that it's not clear how to define an operator (should work visual mode too) that actually prompts!. But since it's not useful for me I will not heckle you further.


Implement "done" toggle

This addresses #3.

Remove Bulma from NPM dependencies
Bulma is pulled in via a CDN, so there's no need to install it via NPM.
Update Bulma to 0.9.1
This is needed for the various flexbox helpers.
Restructure task HTML slightly

Build it a little more programmatically, so that the message header and body elements are referenced
Put the 'edit' button in a container element which is styled as a flexbox
Use bulma's spacing helpers rather than applying via a style attribute
Select the message an event references via Element.closest rather than (brittle) Element.parentNode calls

Allow items to be marked as 'done'

Add a boolean 'done' field to the task type
New tasks are created with 'done' set to false
The edit task form has a 'done' checkbox
Task cards have a 'done' checkbox next to the 'edit' button which allows the 'done' status to be toggled with a single click
Tasks which are marked as done have their title and description decorated with strikethrough

I must say that when I made this project, using a template string for the task panel seemed the perfect fit, but I can see the advantages of doing it programmatically. Nice catch!

It was mainly .children[1] etc that I found scary! Grabbing by selector or reference is much less brittle.
Thanks for accepting.


SLO callbacks for PTs; PT/PGT ticket clean up

Closes #519 and #1428
Issues

SLO callbacks are not never executed for PTs
PGTs and PTs are never removed from the ticket registry with /logout is invokved except when JPA Ticket Registry is used. They otherwise linger on in the cache until the cache expiration policy or the CAS cleaner kicks in, which is unnecessary waste of memory and resources.

Patch

SLO callbacks are now executed for PTs and STs
PGTs and PTs are properly removed from all registry stores when a TGT is removed.

This should be merged with 4.2.
Solution
TGTs now have a way to reference the child PGTs which they created. Upon logout, such PGTs are enumerated, PTs collected and SLO notifications broadcasted. When the parent TGT is then deleted, all its STs, PGTs and PTs are recursively deleted. Extra care is taken to ensure transaction integrity in the case of JPA ticket registry.

@frett You should also review this.

You have a couple rogue auto comment formats but it looks good otherwise.

A few comments here and there. SLO has become fairly complex. Looks good to me, but I fear some unexpected behaviours in real life. Some tests are broken in the build.

This is going to be a joint effort to work out the test issues. I should probably ask @robertoschwald to review and test this also. @frett is also looking at it, and these are folks that deploy JPA in prod. They are the best guarantee.

@leleuj you may have a point there. How about we add a property to keep the existing behavior by default, and let that setting toggle SLO for PTs? We have them off by default just like now, and we doc the property that enables them for people to reduce risk. Makes sense?

What's the risk implied by this pull request? I have the feeling that we only add HTTP calls and tickets deletion without changing the existing logic. Isn't that right?

The risk would be that applications which were authorized by PTs now all of sudden may be SLOed out once the user explicitly logs out. That may be considered unintended behavior.

Indeed, it pleads for a property to enable / disable this new behaviour.

I think that setting would violate least surprise for any new deployments. Defaulting to current behavior should be deprecated at least.

Sure. We could issue WARN statements in the log. That should grab folk's attention, and in 4.4 we could switch the new behavior on.

I realized disabling SLO for proxies in form of a setting is not practical, simply because all those tickets end up getting deleted later as part of deleteTicket(). So I am just going to leave things as they are now. Since we are targeting this change for 4.3, a "change in behavior" might be acceptable.

All systems go.

I've got the AbstractTicketRegistry stuff ready to go as soon as this is merged.

@tduehr go ahead please.


Move content of Fork changes to README.md.

This makes it easy for other people to know what has been changed when they view the repository.

Whoops
Sorry, I accidentally made a pull request to the main repo, I meant to do it to my own fork.


Provide a way to "kill" a task but mark it as successful

There are frameworks that will submit jobs to cook that are "workers" and then kill the worker when it is no longer needed. In cook this is registered as a failure but the framework is simply shutting down the worker.
We should provide a way to mark a task in this case killed but successful so that the framework and cook can have better reporting.