Fix setup.py file

This means we have inline source maps, we should consider reverting that, it grows the bundle quite a bit! I should check for ipyvolume as well.

Ok :) I would be happy to revert it. But should the source maps be deployed by the setup.py anyway?

Not sure, for debugging it makes sense to have, lets revert it to what it was, and determine widget wise what we should do.

Merging as it fixes the build in master.
I am fine with re-adding source maps later.

@maartenbreddels In order to put the mapping file back I just need to remove the inline-source-map option in the karma config right?

No, I think you replace by source-map, and include the files again in setup.py

Ok :) Thanks, will do that


Command 'make build' not downloading dependencies.

When running the command make build or make test, no dependencies are downloaded. Fixing the issue by replacing 'go get' to 'go install'.

Just a note: this is since go 1.18 (kubebuilder reflected this in its templates here https://github.com/kubernetes-sigs/kubebuilder/pull/2486)


NTR: multiple packaging integrity terms

NTR: food packaging integrity
Parent Term/Class:
‘food object quality’ FOODON:0002470
Definition:
quality of the stability and completeness packaging material
NTR: food packaging intact
Parent Term/Class:
'food packaging integrity' (above)
Definition:
container, packaging or wrapping in original form and undisturbed
NTR: food packaging not intact
Parent Term/Class:
'food packaging integrity' (above)
Definition:
container, packaging or wrapping not in original form and/or noticeably disturbed
NTR: food packaging opened by consumer
Parent Term/Class:
'food packaging not intact' (above)
Definition:
container, packaging or wrapping opened by consumer

For "food packaging not intact" how about "food packaging compromised" to line up with "food packing medium compromised"?

Yep!

Hi Kayla, following your new term requests we've added the following terms to FOODON:

food packaging integrity = food packaging integrity FOODON:03601000
food packaging intact = food packaging intact FOODON:03601001
food packaging not intact = food packaging compromised FOODON:03601002
food packaging opened by consumer = food packaging opened by consumer FOODON:03601003

These will be available in the next foodon release, or alternatively you can view them now in the latest foodon-edit.owl


Limit alarm handler walltime to 15minutes

This PR adds a wall time timeout for alarm handlers.
When the timeout for an alarm it hit, we cancel deferred alarm deletion and abort the currently running handler.
The alarm timeouts are counted against limits, so they get deleted once the maximum retry limit is reached.

Updated PR with new expected behaviour where alarms will be deleted after 1 run + 6 retries.

Updated description
Added additional logging
Alarm retries will now count against limits.

Updated PR with timeout as kj::Duration instead of uint32_t.

Updated PR:

Addressed @justin-mp comments
Rebased


[Bug]: Import from Kettle - Get System Info -> transformation name / pipeline name error

Apache Hop version?
2.7.0
Java version?
11.0.19
Operating system
Windows
What happened?
When Importing from Kettle, the Transformation name / pipeline name output from Get System info gives  values.

Issue Priority
Priority: 2
Issue Component
Component: Import, Component: Transforms

Hi @Fjelley85 I tried to reproduce your issue using Apache Hop 2.8 but everything is working perfectly. Can you try once again with Hop 2.8 and let me know? In case of any error, can you please post the kettle transformation that, when migrated, gives you the problem you reported so that we can analyze it better?

Seems ok now


Application endpoint cache

Implements a generic Redis cache which as of now is used to store the endpoint information for a given application on message creation. This is read on repeated message  creations for the same application and when fetching endpoint information in the worker.

I just merged #283 which means you'll need to rebase over main and fix warnings. :)


Remove detection of old basic auth secret

I have added that check in 42ff2920ea5bb7b6db37834314aa49bdf69b4011 (April 2022) when we migrated from one secret name to another.
We can safely remove it now but keep that function for future checks if needed when doing a similar migration.
Improve documentation for the developers on how to add new checks.
Changes

Submitter Checklist

[ ] 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

[ ] ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

[ ] ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

[ ] 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

[ ] 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

[ ] 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

[ ] 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

lgtm


Dual Knobs would be great!

What exactly do you mean by dual knobs? Isn't example 9 and others dual knobs?
http://angular-ui.github.io/ui-slider/demo/demo.html


Fields like avatarId or other non-required fields should be typed as Option

Error:
Error: Request(reqwest::Error { kind: Decode, source: Error("missing field `avatarId`", line: 1, column: 72073) })

Thanks for the bug report, @khayyamsaleem.
The issue is that I couldn't find any resources listing which fields in the REST response are optional and which are mandatory, so I went by trial and error.
Would you happen to know which fields other than avatarId are optional, or where I could find the information?

Hi @khayyamsaleem,
I've released version 1.1.0 of the crate with several fields now correctly marked as optional. See if this helps.
There might be more such fields, but my testing with several Jira instances hasn't found any more.

Hi! Sorry for the delayed response; I also hadn't found any source of truth on which fields were required or optional after checking the Jira Issues API documentation :c but I can try out this iteration and let you know if I come across anything. Thank you for the super fast turnaround on this!

Hey, I'm getting a similar error for saying missing field progress.  Could that one be made an option as well?


publish button failing on dandiset 53

https://user-images.githubusercontent.com/844306/129488742-e396ab78-5ae4-4326-a103-f62bfe2700d5.mp4

The logs suggest this is a problem with relatedResource in the metadata:
ValueError: identifier has to be from the list:
{'LISSN', 'ISTC', 'arXiv', 'bibcode', 'w3id',
'LSID', 'URN', 'ISBN', 'PMID', 'ARK', 'IGSN',
'EAN13', 'PURL', 'DOI', 'Handle', 'URL', 'UPC',
'ISSN', 'EISSN'}, but https provided

Presumably this shouldn't pass validation, but it obviously has.

https://github.com/dandi/dandischema/blob/92f303e5cc9edeaf1c1369930e37bc203e97f2e0/dandischema/datacite.py#L175-L198 is the problematic code. Looks like it's not equipped to handle an identifier with a URL.

@bendichter - one way to handle this is to make the identifier a doi as datacite wants it: DOI:10.1038/s41467-021-20936-8
@dchiquito -  we indeed should augment the link processing in that section in dandischema knowing that in general identifiers can be urls. @djarecka - do you want to take a stab at this? i.e. the doi code should parse the url to see if it matches some known entity like doi.org and others and then map it to that type or map it to url type.

@satra OK, but it's pretty unclear from a user perspective what form to use for the identifiers. The automatic ones in the Asset Summary seem to suggest that we want them as URLs.

@bendichter - we should allow URLs there, just that the dandischema code needs to be updated. i'm simply suggesting the DOI: route as a short term fix, if you would like to have publish work before the code fix is in place.

so, the problem is that attributes["relatedIdentifiers"][i].identifier could have url and we should process it in the same way as in the elif part?

you would then have to parse the url to see if it matches any of the known identifier patterns, if so reduce the url to identifier and proceed. if not, process in the same way as the elif part.

This worked for me.
@Saksham20, could you use this work-around for dandiset 54 as well?

Yes, removing the url and suffixing the doi with DOI worked on dandiset 54

could we close the issue now?

@Saksham20, I'll close this issue, but please reopen if you have any further trouble!


[blosc] Update to 1.18.1

update to 1.18.1 and export blosc-config.cmake:
find_package(blosc CONFIG REQUIRED)
target_link_libraries(main PRIVATE blosc)

Thanks for your PR. Could you please remove openvdb:x64-windows-static in scripts/ci.baseline.txt, it makes windows build failed.

Thanks for the PR!


Fix race condition in observability config package

In pkg/config/v0/observability.go we're experiencing a race condition in which the client attempts to delete the ObservabilityStackDefinition and ObservabilityStackInstance objects in quick succession. Since the latter requires reconciliation and the client doesn't wait for this to occur, an exception is thrown when the former is attempted. On subsequent attempts to clean up the ObservabilityStack, an error is thrown because the ObservabilityStackInstance has already been cleaned up, and the ObservabilityStackDefinition is not cleaned up.
This can be fixed by a) refactoring this package to use the util.Operations abstraction and b) add a wait in the Delete() method to ensure that the ObservabilityStackInstance is deleted and reconciled before attempting to delete the ObservabilityStackDefinition.

Included in 0.5 release


Failed installation of optparse-applicative-<IP_ADDRESS>

Description
Failed to install optparse-applicative-<IP_ADDRESS>
Build log ( /Users/titto/.etlas/logs/eta-<IP_ADDRESS>/optparse-applicative-<IP_ADDRESS>-Cl5llHZYFjy5i7ettrONls.log ):
etlas: Entering directory '/var/folders/jz/3q6hyr7d1078r8hyvm8v0nxc0000gq/T/etlas-tmp-32239/optparse-applicative-<IP_ADDRESS>'
Configuring optparse-applicative-<IP_ADDRESS>...
Preprocessing library for optparse-applicative-<IP_ADDRESS>..
Building library for optparse-applicative-<IP_ADDRESS>..
[ 8 of 16] Compiling Options.Applicative.Help.Coreeta: While in operation Call:
Exception: {handle: fd:15}: Eta.REPL.Message.remoteCall: end of file
eta-serv terminated (1)
Error: Unable to access jarfile
etlas: Leaving directory '/var/folders/jz/3q6hyr7d1078r8hyvm8v0nxc0000gq/T/etlas-tmp-32239/optparse-applicative-<IP_ADDRESS>'
Steps to Reproduce
etlas install optparse-applicative-<IP_ADDRESS>
Context
I am trying to compile and test this package:
https://github.com/Quid2/flat
Your Environment

Did you install an older version of Eta/Etlas before?
Yes, but I removed before installing the latest version.

Current Eta & Etlas version:
eta-0.8.6b2.

etlas version <IP_ADDRESS>
compiled using version <IP_ADDRESS> of the etlas-cabal library

Operating System and version:
macOS Sierra

Link to your project:
https://github.com/Quid2

Much better.
I got this message:
etlas: symlink-bindir is not defined. Set it in your cabal config file or use
--symlink-bindir=
I set the --symlink-bindir in the cabal config file but I still got the same message so I passed it as a parameter and it compiled ok.
Unfortunately, continuing with my attempt of compiling the 'flat' package, I got another problem:
In order, the following will be built (use -v for more details):

unix-<IP_ADDRESS> (lib) (requires download & build)
wcwidth-0.0.2 (lib:wcwidth) (requires download & build)
tasty-<IP_ADDRESS> (lib) (requires download & build)
tasty-quickcheck-0.10 (lib) (requires download & build)
tasty-hunit-<IP_ADDRESS> (lib) (requires download & build)
flat-0.3.3 (test:spec) (first run)
Downloading [source] wcwidth-0.0.2...
Downloading [source] unix-<IP_ADDRESS>...
Configuring wcwidth-0.0.2 (all, legacy fallback)...
Building wcwidth-0.0.2 (all, legacy fallback)...
Downloading [source] tasty-<IP_ADDRESS>...
Configuring unix-<IP_ADDRESS> (lib)...
Building unix-<IP_ADDRESS> (lib)...
Downloading [source] tasty-hunit-<IP_ADDRESS>...
Downloading [source] tasty-quickcheck-0.10...

Failed to build unix-<IP_ADDRESS>.
Build log (
/Users/titto/.etlas/logs/eta-<IP_ADDRESS>/unix-<IP_ADDRESS>-f35914fc1d8c6a3b9590bf1ab79e654b5f06997893ceb2ab1b8ee74415a2ff94.log
):
Configuring library for unix-<IP_ADDRESS>..
Preprocessing library for unix-<IP_ADDRESS>..
etlas: can't find source for System/Posix/Resource in ., dist/build/autogen,
dist/build/global-autogen
Is there any way out?
Thanks

I am perplexed.
etlas build unix
In order, the following will be built (use -v for more details):

unix-<IP_ADDRESS> (lib) (requires build)
Configuring unix-<IP_ADDRESS> (lib)...
Building unix-<IP_ADDRESS> (lib)...

etlas build wcwidth
In order, the following will be built (use -v for more details):

wcwidth-0.0.2 (lib:wcwidth) (requires build)
Configuring wcwidth-0.0.2 (all, legacy fallback)...
Building wcwidth-0.0.2 (all, legacy fallback)...

But then this:
etlas build tasty
In order, the following will be built (use -v for more details):

unix-<IP_ADDRESS> (lib) (requires build)
wcwidth-0.0.2 (lib:wcwidth) (requires build)
tasty-<IP_ADDRESS> (lib) (requires build)
Configuring wcwidth-0.0.2 (all, legacy fallback)...
Configuring unix-<IP_ADDRESS> (lib)...
Building wcwidth-0.0.2 (all, legacy fallback)...
Building unix-<IP_ADDRESS> (lib)...

Failed to build unix-<IP_ADDRESS>.
Build log (
/Users/titto/.etlas/logs/eta-<IP_ADDRESS>/unix-<IP_ADDRESS>-f35914fc1d8c6a3b9590bf1ab79e654b5f06997893ceb2ab1b8ee74415a2ff94.log
):
Configuring library for unix-<IP_ADDRESS>..
Preprocessing library for unix-<IP_ADDRESS>..
etlas: can't find source for System/Posix/Resource in ., dist/build/autogen,
dist/build/global-autogen
Failed to build wcwidth-0.0.2.
Build log (
/Users/titto/.etlas/logs/eta-<IP_ADDRESS>/wcwidth-0.0.2-edd4a4a43de870517c2cccec870740d2e8146e28de902658cda7eb596e9d6b7c.log
):
Configuring wcwidth-0.0.2...
Preprocessing library for wcwidth-0.0.2..
Building library for wcwidth-0.0.2..
[1 of 1] Compiling Data.Char.WCWidth
eta: panic! (the 'impossible' happened)
(Eta version 0.8.6b2):
tcCheckFIType: Unsupported calling convention.
ccall unsafe "value wchar.h wcwidth"
Please report this as a Eta bug: http://github.com/typelead/eta/issues
etlas: Failed to build unix-<IP_ADDRESS> (which is required by tasty-<IP_ADDRESS>). See
the build log above for details.
Failed to build wcwidth-0.0.2 (which is required by tasty-<IP_ADDRESS>). See the
build log above for details.
Why building unix and wcwidth separately does not give an error, but it fails when building them as dependencies of tasty?

Yeah, it is quite strange. I've downloaded the actual master version (4ec7438041dbbe6d01ebbe0a2f11960ce44d5) and etlas-<IP_ADDRESS> builds it correctly for me. I've checked that there are a lot of comments in flat.cabal, maybe are you building another version?

Otoh if i try to install unix inside flat i've got an error:
C:\Users\Javier\dev\lang\eta\flat>etlas build unix
Resolving dependencies...
etlas: Cannot build the package unix, it is not in this project (either
directly or indirectly). If you want to add it to the project then edit the
cabal.project file.

I got this message:
etlas: symlink-bindir is not defined. Set it in your cabal config file or use
--symlink-bindir=

@tittoassini it seems you kept the local dirs of eta and etlas, could you try again removing ~/etlas and ~/eta dirs?

Thanks for your help in this matter :-)
I tried to delete both directories but ... no joy.
I just released flat-0.3.4 and this is what I get (on both macOS Sierra and ubuntu):
etlas update;etlas unpack flat-0.3.4;cd flat-0.3.4;etlas build --enable-tests
.....
....
Downloading [source] tasty-quickcheck-0.10...
Failed to build unix-<IP_ADDRESS>.
Build log (
/root/.etlas/logs/eta-<IP_ADDRESS>/unix-<IP_ADDRESS>-2d5cf6072ceb67ca8f5d36f8fcd8cd67df3c8fda0b3ded0f0938ad9fceb535b9.log
):
Configuring library for unix-<IP_ADDRESS>..
Preprocessing library for unix-<IP_ADDRESS>..
etlas: can't find source for System/Posix/Resource in ., dist/build/autogen,
dist/build/global-autogen
Failed to build wcwidth-0.0.2.
Build log (
/root/.etlas/logs/eta-<IP_ADDRESS>/wcwidth-0.0.2-d5c124531c05325d90e29469dd1f64933b0d3ceb3b6d7b875027131491e6941c.log
):
Configuring wcwidth-0.0.2...
Preprocessing library for wcwidth-0.0.2..
Building library for wcwidth-0.0.2..
[1 of 1] Compiling Data.Char.WCWidtheta: panic! (the 'impossible' happened)
(Eta version 0.8.6b2):
tcCheckFIType: Unsupported calling convention.
ccall unsafe "value wchar.h wcwidth"
Please report this as a Eta bug: http://github.com/typelead/eta/issues
etlas: Failed to build unix-<IP_ADDRESS> (which is required by test:spec from
flat-0.3.4). See the build log above for details.
Failed to build wcwidth-0.0.2 (which is required by test:spec from
flat-0.3.4). See the build log above for details.
Any idea?
As it works for you, it must be some misconfiguration on my side but I have no idea where to look.
I would love to be able to certify that 'flat' works with eta.

The java versions I am using:
java -version
openjdk version "1.8.0_181"
OpenJDK Runtime Environment (build 1.8.0_181-8u181-b13-1ubuntu<IP_ADDRESS>-b13)
OpenJDK 64-Bit Server VM (build 25.181-b13, mixed mode)
java -version
java version "1.8.0_25"
Java(TM) SE Runtime Environment (build 1.8.0_25-b17)
Java HotSpot(TM) 64-Bit Server VM (build 25.25-b02, mixed mode)

Ok, it seems etlas is picking a version of tasty package that is not patched (see https://github.com/typelead/eta-hackage/blob/master/patches/tasty-<IP_ADDRESS>.patch#L150-L155). In fact i guess it is picking the last one <IP_ADDRESS>, released in 2018/10/10.
As i am on windows the condition in the cabal file removes those dependencies for me even without the patch, that's why it worked for me.
So patching the new version should fix the issue. In the meanwhile, as workaround, you can pin the tasty version to <IP_ADDRESS> and check if it doesnt throw more errors,.

Now we are talking :-)
I fixed that and also modified my cabal to use all the latest patched versions of the dependencies.
I was then able to compile and run both the test suite and a benchmark.
I built test and benchmark with:
etlas build --enable-tests --enable-benchmarks
After that etlas test will run the test (that gets stuck and doesn't pass, but that's another issue) but etlas bench won't run the benchmark, I was however able to run the benchmark invoking directly the generated executable.
The benchmark indicates a speed about 3-7 times slower than the ghc version, is this a normal slowdown?

Another issue with the criterion-based benchmark is that it will crash when criterion prints out a unicode char (µ I believe).
Is this a known issue?

I think the slowdown is within the usual range, taking in account that is jvm vs native code. However maybe it could be analyzed and improved.
Afaik there is no open issues about the incorrect behaviour of etlas bench and the bug in criterion, so feel free to open them
Once https://github.com/typelead/eta-hackage/pull/123 etlas should build the package with that version

@jneira

Yes, it can certainly be analyzed and improved. @tittoassini It would be really awesome if you have a dual stack/etlas setup that we can simply run and measure the difference. The more cases of "I want this particular type of program to run faster" that we have, the easier it is for us profile, figure out the bottleneck, and optimize the runtime, codegen, or the library itself. Please do file a new issue on this with a link to reproduction steps.
We have yet to backport the new-bench command which should be fairly easy to do.
Merged.

@tittoassini Regarding the crashing upon unicode and the test suite hanging - can you file individual issues for those so we can fix them individually? Thanks!

OK, I will work on these points in the next few days.
In the meantime, is it a known bug that 'read' does not work with records?
data REC = REC {l::Int,r::Int} deriving (Read,Show)
main = print (read "REC {l=3,r=4}" :: REC)
Exception in thread "main" eta.runtime.exception.EtaException:
Prelude.read: no parse
I could not find it in the list of known issues.
On Wed, 7 Nov 2018 at 08:15, Rahul Muttineni<EMAIL_ADDRESS>wrote:

@jneira https://github.com/jneira

Yes, it can certainly be analyzed and improved. @tittoassini
https://github.com/tittoassini It would be really awesome if you
have a dual stack/etlas setup that we can simply run and measure the
difference. The more cases of "I want this particular type of program to
run faster" that we have, the easier it is for us profile, figure out the
bottleneck, and optimize the runtime, codegen, or the library itself.
Please do file a new issue on this with a link to reproduction steps.
We have yet to backport the new-bench command which should be fairly
easy to do.
Merged.

@tittoassini https://github.com/tittoassini Regarding the crashing upon
unicode and the test suite hanging - can you file individual issues for
those so we can fix them individually? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/typelead/eta/issues/894#issuecomment-436527012, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABYwUfDZv7kZ3orEQqREsuxkBr9dghCks5usoh6gaJpZM4YNSBo
.

--
Dr. Pasqualino "Titto" Assini
http://networkpolitics.svbtle.com
http://quid2.org/

@tittoassini Haven't seen it before - go ahead and file an issue. Feel free to file issues if you feel something is wrong. Don't worry about duplicating - we'll let you know if it is. I'd rather have a duplicate bug report than not having the report in the first place!

I have reported the major bugs that I have encountered during the porting of 'flat' to eta elsewhere so this can be closed.


Add pre-commit-hooks.nix integration in the flake

This PR adds to the flake an output lib.${system}.pre-commit-hook that contains a configuration allowing to use Topiary in pre-commit-hooks.nix. It is a first draft and I am fine changing everything again. In particular:

I only enabled the pre-commit-hook on .json, .toml, .ml and .mli files (and not on .sh files). For that I based myself on the output of --help that mentions the legal uses of --language. I am happy to add .sh to the list of matched files.

The bit of documentation the README is quite prominent, maybe too much for what this is. It made sense to me to put it in the “getting started” section. Maybe it should move after the usage examples though?

As an example, I integrated it in one of my toy projects, and it works just fine.

I'll leave this to Nix-experts to comment, but regarding .sh files: I think leaving them out makes sense, for now, as Bash formatting is considered experimental.


Radar graph without scale displayed with bottom indent

Expected Behavior
display graph to 100% height
Current Behavior
display with indent

Steps to Reproduce (for bugs)
Example on JSFiddle

Radar is circular: https://jsfiddle.net/azsf2jpt/
Try with even number of points.
You can hide it with padding:
https://jsfiddle.net/e4Lydwum/

Updated fiddles with v3.0.0-beta.3
https://jsfiddle.net/dpuwtzr9/
https://jsfiddle.net/4Lymjs2p/


chore: fix linter error

Fixes #920
No bug, warning because the promise constructor was returning another promise (fetch), instead of returning void.
This can be fixed by just changing
new Promise((resolve, reject) => fetch(...)) to
new Promise((resolve, reject) => { fetch(...) })
but we don't necessarily need another new promise, since fetch already returns a promise.
Here, it is semantically preferable to throw an Error type, so type annotations are also updated accordingly.

Thanks @anukul !


Uncaught Error: This buffer's content is too large to fit into a string.Consider using APIs lik...

Select a Large CSV file approx. 300mb try to convert. Voila!
Atom: 1.34.0 x64
Electron: 2.0.16
OS: Microsoft Windows 10 Home
Thrown From: json-converter package 0.2.4
Stack Trace
Uncaught Error: This buffer's content is too large to fit into a string.
Consider using APIs like getTextInRange to access the data you need.
At C:\Users\julia\AppData\Local\atom\app-1.34.0\resources\app\static\<embedded>:11

Error: This buffer's content is too large to fit into a string.

Consider using APIs like `getTextInRange` to access the data you need.
    at TextBuffer.getText (~/AppData/Local/atom/app-1.34.0/resources/app/static/<embedded>:11:486046)
    at TextEditor.getText (~/AppData/Local/atom/app-1.34.0/resources/app/static/<embedded>:11:12850)
    at Object.csvToJson (/packages/json-converter/lib/json-converter.coffee:114:18)
    at /packages/json-converter/lib/json-converter.coffee:100:94)
    at CommandRegistry.handleCommandEvent (~/AppData/Local/atom/app-1.34.0/resources/app/static/<embedded>:11:350017)
    at CommandRegistry.dispatch (~/AppData/Local/atom/app-1.34.0/resources/app/static/<embedded>:11:348492)
    at AtomEnvironment.dispatchApplicationMenuCommand (~/AppData/Local/atom/app-1.34.0/resources/app/static/<embedded>:1:671034)
    at EventEmitter.t (~/AppData/Local/atom/app-1.34.0/resources/app/static/<embedded>:1:678418)
    at emitThree (events.js:136:13)
    at EventEmitter.emit (events.js:217:7)

Commands
     -3:14 github:toggle-git-tab (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui)
     -3:10.7.0 github:toggle-github-tab (atom-workspace.workspace.scrollbars-visible-always.theme-one-dark-syntax.theme-one-dark-ui)
     -0:47.8.0 find-and-replace:show (input.hidden-input)
     -0:46.1.0 core:paste (input.hidden-input)
     -0:45.4.0 core:undo (input.hidden-input)
     -0:27.5.0 json-converter:csv-to-json (input.hidden-input)

Non-Core Packages
json-converter 0.2.4

Guessing it is related. I am trying to open a large(ish) csv of around 250 megs (1.6m lines). Every 30 seconds or so, I get an error saying the application is not responding. I've tried waiting many times,, but nothing ever happens. Is there a work around?


customerUpdate() doesn't update user field

Hi there, Thanks for putting the time and effort on this awesome package.
I've been trying to update customer data via this code
      await ShopifyCustomer.instance.customerUpdate(           customerAccessToken: await ShopifyAuth.currentCustomerAccessToken,           firstName: user.firstName,           lastName: user.lastName,           email: user.email,           phoneNumber: user.phoneNumber);
but when i try to retrieve the current user via this code
      ShopifyUser shopifyUser = await ShopifyAuth.instance.currentUser();
it returns a user object with the old values, is this some sort of caching issue ?

I addressed recently the problem with silent errors in b459a828d5406f94c80de8a784118670a92142cf. If you use the version from master it should be better to find the problem

I'm closing it for now because of no response from the original author.
If the issue does still persist, feel free to leave a comment and I will reopen the issue :)


peanjs generator

hi, is there available generator for peanjs like yo meanjs?

Hi @indraganiarto,
Sorry for the late response. Hmmm... as the Yo generator is used for scaffolding, I can't see any reason why you "couldn't". I've not tried this myself though. Give it go maybe?
GL!


Threadsafe user data and thread unsafe implementation

Hi,
I'm in the following case:
let surface = thread_local.compositor.create_surface(|surface| {
    surface.implement(thread_local_implementation, threadsafe_handle)
}).unwrap();

where thread_local_implementation is not Sync nor Send, but threadsafe_handle is.
However in the current implementation of NewProxy::implement_closure, the associated user data is declared thread unsafe just because the implementation is. Users should have the choice (I need to have the choice).
One (breaking change) solution would be to change the signature of the NewProxy::implement* functions, to directly take a UserData type instance so that the user can instantiate it with UserData::new or UserData::new_threadsafe which ever is appropriate.
If you want to avoid breaking the API, one other solution would be to add a NewProxy::implement_user_data_threadsafe function that takes a thread unsafe implementation with a threadsafe user data.
If one of those solution pleases you (or another one), I can make a pull request.

For now, lets go the simplest route, using implement_user_data_threadsafe. It unfortunately creeps up the number of methods, but I'd rather keep the things simple for now.


function has_value: Array and index mixup

I guess that in the file library.awk in line 504 in the function has_value(A, value,    k) it should be
A[k] == value

instead of
k[A] == value.

A is the array and k is the index. This issue was found by mawk.
Many thanks for this great and very helpful awk library.

Thanks, fixed.


login screen android

rebased to main (has Dependency Injection)
validation check for login
minor xml changes

Failing because SignupFragment is empty

 Login Screen - Android


translator

Description of the new feature / enhancement
just like the snipping tool or the text extractor but with a translator either it coppys the text to clipbord or shows it over the orignal text
Scenario when this would be used?
reading trough a web page not in english but you only want sertain bits translated but not all like google chrome can already do
Supporting information
No response

great idea, not something the PowerToys team would build out due to needing a service to query.
MS Edge can translate parts of a site too, fyi :)


There has a bug with AnimationPlayer copy/paste.

Godot version:
3.1.1
OS/device including version:
mac10.14
Issue description:
do copy an animation then paste. It will create a new animation. But it is not a new one. If I change eather two it always be change to same. The right way is duplicate the animation.(It should be forbid use copy/paste dup)

Might you also be wondering about this?  https://github.com/godotengine/godot/issues/24950
I feel like copy/duplicates creating new resources should be the default behavior, and an extra option for reference copies should be available.
I feel it would also be less mysterious to have clear indications in the UI that the Animation resource is pointing somewhere else.

Closing as duplicate of #24950


As a user, I want tasks in the Calendar to be arranged in order of priority

Issue #35.

Completed in #72.


Installer: Path too long on installation (warning/error)

The 2.2.6 windows 64bit installer threw a warning at me, that the path would be too long.
I assume it's the environment variable Path, which is indeed quite long (1,534 characters).
But since I'm able to extend the content of this variable manually, I wonder why the installer isn't able to append something.

Hey you should try LongPathTool for this problem..

Just want to inform you, that it's still present in 2.5.1 setup.

This is a known issue of NSIS:
https://github.com/dynamorio/drmemory/issues/1029
http://nsis.sourceforge.net/Environmental_Variables:_append,_prepend,_and_remove_entries
There is a special build with long string support:
http://nsis.sourceforge.net/Special_Builds

We tried replacing the dlls with those from the site; The result was that the installer would crash on start.

Any idea why that happened @dothebart?
The path too long-exception is a really ugly problem of NSIS, I see it often during installations...
Could this specific operation be carried out by own code instead?

Yo should try "Long Path Tool" to handle path too long exception.

@gamicklisafisher do you have any hints on how to use it from nsis? do you have a faq for this?

I found the following two native code snippets, that are said to circumvent NSIS' string length limitation:

https://www.smartmontools.org/browser/trunk/smartmontools/os_win32/installer.nsi?rev=4110#L636
http://stackoverflow.com/a/31342128/2044940

I also now know what caused the installer crashes: we have NSIS plugins which are compiled with old compilers; this doesn't work with the alpha NSIS and causes crashes.
We need to recompile these plugins in order to use the prerelease NSIS.

We switch to a flat directory structure as a workaround.

Same problem here when I try to install the Client Tools 3.7.11 to Win 10 64bit from the NSIS package.
The installer said: Your PATH environment variable is already too long. The installer is unable to add ArangoDB to it.
Then the installer is stoped without an error code but at approx half of the installation process.


Fix #1582, adding global object save/restore capabilities

This is a fix for issue #1582, which reports that functions in the global object that were poisoned during a test (e.g. stubbed or otherwise altered) were not restored upon test completion.
The PR creates a module that provides for saving/caching the global object (during the first call to GlobalObject.save()), as well as restoring it from cache when needed (by calling GlobalObject.restore()).
Runnable now includes a call to GlobalObject.save() just before kicking off a test, and a call to GlobalObject.restore() on the callback invoked upon test completion (whether successful or not).
Example
In the example below, every test stubs the console.log() method, rendering it useless, unless it is explicitly restored.
$ cat example.js
describe('suite', function() {
  before(function() {
    console.log('before: should work');
  });
  it('example 1', function() {
    console.log('example1: should work');
    console.log = function() {};
    console.log('example1: should not work');
  });
  it('example 2', function() {
    console.log('example2: should work');
    console.log = function() {};
    console.log('example2: should not work');
  });
  after(function() {
    console.log('after: should work');
  });
});

Before Fix
When console.log() is stubbed in test example 1, it remains so, even after the test is complete. As a result, output from successive tests, the after() hook, and the reporter are all suppressed.
$ ./bin/mocha -R spec example.js

  suite
before: should work
example1: should work

After Fix
I corrected this by restoring the global object, upon test completion. In my example, now console.log() functionality is restored on every successive test, as well as the after() hook. Reporter output is, likewise, no longer suppresed.
$ ./bin/mocha -R spec example.js

  suite
before: should work
example1: should work
    ✓ example 1
example2: should work
    ✓ example 2
after: should work

  2 passing (35ms)

I think this is part of your program design and not part of Mocha.
IMO, by testing your "unit" you may want to test its own behaviour, and log to stdout/console it's not part of it (i.e: we might want to test if it's called, but not call it directly, because this not the responsibility of our "unit").
You can simply win it by loose coupling and using DI for example.
e.g: your code may look like this:
function MyService(logger) {
  // getting logger service, used by this service
  // it could be `console`, `winston` or stub for testing
  this.logger = logger;
}

MyService.prototype.myVerySmartFn = function(val) {
  // ...
  // logger used by this service
  this.logger(val);
  // ...
}

and your test may be something like that:
describe('MyService', function() {
  var ms = new MyService(stubLogger);
  // ...
  describe('Call `myVerySmartFn` with invalid argument', function() {
    it('should log error', function() {
      ms.myVerySmartFn(null);
      stubLogger.error.called.should.be.true;
      // ...
    });
  });
  // ...
});

This is how I see it, and I'd love to hear your opinion @JohnnyEstilles

+1 @a8m, i'm not too keen on this change

@a8m, the console.log() statements in my example are merely for illustration purposes only. It's simple to demonstrate that if a test stubs/changes the behavior of methods in the global object and doesn't clean up after itself, it may affect the rest of the tests, or like in this case, reporter output. Unfortunately, performing this type of cleanup in an after() or afterEach() hook doesn't really work, because other Mocha code (including the reporter), may run before them.
I understand if there's no interest. I was just responding to the apparent interest expressed in #1582.
Closing ...

My test is not to testing console.log but stdout. I have to change the behaviour of process.stdout.write to achieve this.
Each test should be sandboxed in mocha and users should be able to do whatever dodge things inside each test. Users can change process.stdout.write but they should still be able to see the report (although the report writes to process.stdout, this shouldn't concern users).

@stevemao your argument is perfectly valid. I was just using console.log() to illustrate the point. Maybe I should have used process.stdout.write() instead. Either way, I was just trying to should how "not" restoring the state of the global object after a test can affect Mocha's execution (like in this case, suppressing Reporter output).


Extended planning service with validator

Example of capturing output of the validator by using pddl:adding, pddl:deleting, and pddl:updating properties in the existing plan (pddl:current is used to capture the current value of a fluent before the update is applied as returned by the validator):
<plan>
        a pddl:Plan ;
        pddl:cost 2.0E0 ;
        pddl:step [ a pddl:Step ;
                    pddl:action [ a pddle:move ;
                                  pddle:move-b pddle:c ;
                                  pddle:move-x pddle:a ;
                                  pddle:move-y pddle:table
                                ] ;
                    pddl:adding [ a :on ;
                                  :on-x :move-b ;
                                  :on-y :move-y
                                ] ,
                    pddl:deleting [ a :on ;
                                    :on-x :move-b ;
                                    :on-y :move-x
                                  ],
                    pddl:updating [ a pddl:Increase ;
                                    pddl:current 0 ;
                                    pddl:parameter [ a :moved ;
                                                     :moved-m :move-b
                                                   ] ;
                                    pddl:argument 1
                                  ] ,
                    oslc:instanceShape pddl:StepShape ;
                    sh:order 1
                  ] ,
                  [ a pddl:Step ;
                    pddl:action [ a pddle:move ;
                                  pddle:move-b pddle:a ;
                                  pddle:move-x pddle:table ;
                                  pddle:move-y pddle:c
                                ] ;
                    oslc:instanceShape pddl:StepShape ;
                    sh:order 2
                  ] ;
        pddl:time 0.0E0 ;
        oslc:instanceShape pddl:PlanShape .

Predicate pddl_generator:generate_pddl/3 now supports resources of type pddl:Plan and generates PDDL text compatible with input of VAL validator.

@likelion , should Imply also have left and right, since it is binary and the order is important?
How is it currently handled?
e.g.
--b [ a pddl:Imply ;
pddl:argument
[a pddl:And ;
..
] ,
[a pddl:Or ;
..
]
]

@swarupmohalik Yes, pddl:Imply should use pddl:left and pddl:right because the order is important.


#167: add commits, hoc, last_commit to release-published fact

@Suban05 can you review it, please?

@Yegorov thanks!

@Yegorov Hey there, awesome work on your contribution! 🎉 You've snagged +25 points: 20 for showing up, and 5 for those 108 hits-of-code you cranked out. Nice job keeping it under 200 HOC, avoiding any deductions there. Your total balance is now +269. Remember, more reviews and comments can boost your points, and keep those contributions coming – they're adding up nicely!

@Suban05 🌟 Fantastic job on the review! You've earned 22 points: 15 base + 7 for your insightful comments. Your dedication is paying off - your balance now stands at 357 points. Keep up the great work!

@yegor256 Thanks for your thorough code review! 🎉 You've earned +22 points: +15 base points, and +7 for your 7 insightful comments. This aligns with our policy of rewarding reviewers, especially those who provide valuable feedback. Your current balance stands at -548. Keep up the great work in future reviews!


Make unionWith use a merge instead of an alter loop

I think unionWith could do a merge because both maps are already sorted. This would be O(n). Currently it does an alter loop which is O(n log n).

Same as #1?

Ganz toll!


daos: support for additional container property

Addition of DAOS_PROP_CO_SCRUBBER_DISABLED property for serialization.
Signed-off-by: Ryon Jensen<EMAIL_ADDRESS>

Fixed an issue. Have internal verification running.

@adammoody I merged since this only touches DAOS code


'useStaticOnly' includes volatile request headers

These two at least:

cookie string

referrer

solved by https://github.com/evantahler/browser_fingerprint/pull/8


.NET MAUI 9: IBorder obsoleted

In .NET MAUI 9, IBorder has been obsoleted. Scan the docs/samples for its usage and replace.
More info: https://github.com/dotnet/maui/pull/23224

Associated WorkItem - 335726

No action required.


React-navigation makes flow-typed install fail

Hi, when trying to install new typings for my project, I get an error caused by react-navigation having a CONTRIBUTING.md file in their folder.
Maxence-VirtualBox% flow-typed install
• Found 29 dependencies in package.json to install libdefs for. Searching...
• rebasing flow-typed cache...
UNCAUGHT ERROR: Error: react-navigation_v1.x.x/CONTRIBUTING.md: Unexpected file name. This directory can only contain test files or a libdef file named `react-navigation_v1.x.x.js`.
    at validationError (/home/maxence/.config/yarn/global/node_modules/flow-typed/dist/lib/validationErrors.js:14:11)
    at /home/maxence/.config/yarn/global/node_modules/flow-typed/dist/lib/npm/npmLibDefs.js:113:53
    at Array.forEach (<anonymous>)
    at extractLibDefsFromNpmPkgDir$ (/home/maxence/.config/yarn/global/node_modules/flow-typed/dist/lib/npm/npmLibDefs.js:95:23)
    at tryCatch (/home/maxence/.config/yarn/global/node_modules/regenerator-runtime/runtime.js:65:40)
    at Generator.invoke [as _invoke] (/home/maxence/.config/yarn/global/node_modules/regenerator-runtime/runtime.js:303:22)
    at Generator.prototype.(anonymous function) [as next] (/home/maxence/.config/yarn/global/node_modules/regenerator-runtime/runtime.js:117:21)
    at tryCatch (/home/maxence/.config/yarn/global/node_modules/regenerator-runtime/runtime.js:65:40)
    at invoke (/home/maxence/.config/yarn/global/node_modules/regenerator-runtime/runtime.js:155:20)
    at /home/maxence/.config/yarn/global/node_modules/regenerator-runtime/runtime.js:165:13

I see two solutions to this. Either support having a CONTRIBUTING.md in the type definitions folders or delete react-nativation's CONTRIBUTING.md. What do you guys think ?

This is already supported. You'll need to update to the latest version of flow-typed.

It seems the latest version is 2.4.0, which I have tried, and flow-typed install is still failing for this reason.  What version are you referring to that works, @GAntoine?

Can you verify there's no mixup about local vs. global installs and which version is being used?

No worries. The nuance of yarn global add <x> when it was previously installed with npm trips up all of us.


[iOS] Open DroidKaigi official information links

Idea Description

Please implement open link about DroidKaigi official information

Medium
YouTube
Twitter

Reference images and links

I'd like to try this issue!

@NakaokaRei
Thanks! Assigned 👍🏼
(Already created PR .. 👀 )

@ry-itto
Sorry, I created PR before I was assigned🙇

No problem! Great work 👍 :+1


Asset upload through ASC compatible component

ASC provides a vibrant contemporary UI/UX for assets in DAM, however there is no equally exciting component which we can use to upload assets.
we created a component that provides an upload drop zone where any asset can be dragged and dropped on browser and the component starts file upload to configured path.

upload path can be configured while authoring

it also provides option of extensibility in form of JS methods to program pre-upload and post-upload actions.
Files being uploaded after drop on designated area:

@deepkamal looks super cool! can you talk a little bit about where these assets get uploaded, and the how they get on AEM Author so they can undergo review/approvals/etc.

@deepkamal looks super cool! can you talk a little bit about where these assets get uploaded,

The assets get uploaded at configured path, defined via dialogue box

and the how they get on AEM Author so they can undergo review/approvals/etc.
currently the demo version works on Author only and there is no mechanism implemented for porting contributed assets from Publish to author - suggested approach is to activate a workflow once the assets are uploaded at Publish, the workflow definition can be included as one of the config in doalog box, pl suggest your views on this approach.

@deepkamal ah - i think we'd need to figure out how to get this to work on AEM Publish of course (since if youre on AEM Author, just upload via the normal web ui :)) ..
I'd be interested in ideas that don't involve uploading to AEM Publish, since it can be a brittle reverse-replicating back, etc.

we can have asset approval workflow implemented in Publish, all uploaded assets will be going through asset-approval based on defined steps, I think approach can not be generalized here.


Cocoapods support

IQAudioRecorderController is a very useful controller. But I use cocoapods & would like to pull this in using cocoapods. Can you add support for cocoapods?

Will add support for cocoapods, Thanks for your suggestion

Added cocoapods


move controllers and cameras off in steamvr

I have a set up with 3 playstation eyes. Configering the controllers in the psmoveservice config tool was no problem. When I test the tracking all looks good, the controllers seem to be configered and tracked quite well. But when I open steamvr both move controllers are a bit off to the side when I define their position as steamvr tells me. Also they are loosing their orientation very fast when I move around. And I can see in the grid few that my cameras are not showing up in front of me like the are for real, but also moved to the side.
Got no clue how to solve this.

Any issues related to SteamVR should be asked first in the google group then in PSMoveSteamVRBridge.


Inconsistent fetchPlan result

I just want to execute some simple querys and come across this strange behaviour.
http://orientdb.com/docs/last/Network-Binary-Protocol.html#request_command
Binary driver as OSQLSynchQuery correct result
select from #12:0 fetchplan fetchPlan sponsoredBy:1

Results:
[ { '@type': 'd',
    '@class': 'Node',
    DisplayName: 'peter',
    Email: 'peter@local',
    Points: 0,
    Position: 'Member',
    Active: false,
    Created: Tue May 12 2015 21:39:13 GMT+0200 (W. Europe Daylight Time),
    SponsoredBy:
     { '@type': 'd',
       '@class': 'Root',
       Name: 'test',
       DisplayName: 'test',
       Email: 'test@local',
       Points: 100,
       Position: 'Admin',
       Active: true,
       Created: Tue May 12 2015 21:39:13 GMT+0200 (W. Europe Daylight Time),
       References: [Object],
       InsertedAtDepth: 0,
       out_BroughtIntoLeft: [Object],
       out_BroughtIntoRight: [Object],
       Mid: '555256e41ea7cb4c1f959812',
       '@rid': [Object] },
    References: [ [Object], [Object] ],
    in_BroughtIntoLeft: [ [Object] ],
    InsertedAtDepth: 0,
    out_BroughtIntoLeft: [ [Object] ],
    Mid: '555256e41ea7cb4c1f959814',
    '@rid': { [String: '#18:1001'] cluster: 18, position: 1001 } } ]

Binary driver as OCommandSQL doesn't works
[ { '@type': 'd',
    '@class': 'Node',
    DisplayName: 'peter',
    Email: 'peter@local',
    Points: 0,
    Position: 'Member',
    Active: false,
    Created: Tue May 12 2015 21:39:13 GMT+0200 (W. Europe Daylight Time),
    SponsoredBy: { [String: '#19:78'] cluster: 19, position: 78 },
    References: [ [Object], [Object] ],
    in_BroughtIntoLeft: [ [Object] ],
    InsertedAtDepth: 0,
    out_BroughtIntoLeft: [ [Object] ],
    Mid: '555256e41ea7cb4c1f959814',
    '@rid': { [String: '#18:1001'] cluster: 18, position: 1001 } } ]

Binary driver as OCommandScript doesn't works
[ { '@type': 'd',
    '@class': 'Node',
    DisplayName: 'peter',
    Email: 'peter@local',
    Points: 0,
    Position: 'Member',
    Active: false,
    Created: Tue May 12 2015 21:39:13 GMT+0200 (W. Europe Daylight Time),
    SponsoredBy: { [String: '#19:78'] cluster: 19, position: 78 },
    References: [ [Object], [Object] ],
    in_BroughtIntoLeft: [ [Object] ],
    InsertedAtDepth: 0,
    out_BroughtIntoLeft: [ [Object] ],
    Mid: '555256e41ea7cb4c1f959814',
    '@rid': { [String: '#18:1001'] cluster: 18, position: 1001 } } ]

Why it is different? and by the way can you explain me which command is the right tool for which situation?
OrientDb: 2.0.8
Driver: Oriento

@StarpTech
do you still need help on this?

Closing because no answer after months. In case please reopen/comment this.

@lvca if you compare the dates
Created:  13 May 2015
First Response: 10 Jun
you can see that I got no response of this and nobody inspect or try this issue. What you expect from me? The issue cannot be solved itself.

@StarpTech, I run a script that closes the issues in status "waiting reply" but without any answer since long time. This is to avoid zombie.
In this particular case the issue could be still there, let's reopen it and put it in a higher priority to avoid it's there for months again.

@tglman could you please look at this asap, maybe it's already fixed because last fix on the return objects?

this is fixed in 2.2.x branch, will be released in 2.2.14

Hi @tglman great News can you reference the commit.

hi,
I linked it to the github issue, here it is: https://github.com/orientechnologies/orientdb/commit/4fcb97b1a94b4d89891c326a12b1a7ebcb7e307c
bye

Hi,
This has been release in 2.2.14,
i'm closing this.
Regards


Question about process_wadi.py

Hi,
Thank you for the open source code. But when I ran the process_wadi.py file, I had this error, and I want to ask how can I fix this?

Hi @dxbj986, I ran into the same problem. You should take a look at this line: https://github.com/d-ailin/GDN/blob/9853899da860682669a134e4af315d036aab4eca/scripts/process_wadi.py#L50 . I had to change it to test= test.iloc[:,2:] and it worked.


where is the statefulset.yaml

each clickhouse-server instance running on separate stateful set, which rendered by clickhouse-operator directly, please look at https://github.com/Altinity/clickhouse-operator/blob/master/pkg/model/creator.go#L281


Support resume experiment better

Currently, if the nnictl is stopped due to reboot computer or other reasons, the running trials cannot be resumed, even the trials run on remote platforms. It should be able to get them back, so that the trials don't need to be rerun.

The running trials should be able to resend intermediate results, if server doesn't receive them.
The completed trials may be recycled by remote platform. If so, they should be able to rerun automatically.

I agree. Getting trials back is very useful, if the machine on which you run nni is unstable and sometimes reboots, for example, Windows.
In my case, it's a problematic driver.


Add a ramp-down phase

Similar to a warmup phase, we should also consider a part of the samples as ramp-down samples which are not included in measurement.

Analysis in #175 has shown that an explicit ramp-down phase does not buy us much at the moment. We'll maybe add it later should the need arise.


DDEV v1.21.2 Test Plan - Community Help Requested

It's testing time again here in the DDEV factory. We love community help with every item that is interesting or relevant to you, and just using the release candidates is hugely helpful, because you may stumble across something in your regular workflow. Please use DDEV HEAD for your testing (or brew unlink ddev && brew install --HEAD --fetch-head drud/ddev/ddev) or other techniques at [Testing Latest Commits].(https://ddev.readthedocs.io/en/latest/developers/building-contributing/#testing-latest-commits)
I don't think in general you'll have trouble using this in your daily workflow, so would love you do do that. You can switch back and forth , see How can I install a specific version of DDEV in the FAQ.
If you find a bug that you're pretty sure is a bug, then just open an issue for it.
To give feedback or ask questions, please just reply on this issue or discuss in discord or any other support channel.
I am maintaining the provisional release notes for v1.21.2 and would love any comments on those.

PHP 8.2: Configure a project with PHP 8.2 and confirm correct behavior, and that you can use it with some site. Drupal 10 is probably nearly ready, latest TYPO3 v12 might be.
Craft CMS:

Configure a Craft CMS project from scratch using the Quickstart
ddev craft should work
If project type is set to php then ddev craft should tell you to use the real project type.

TYPO3 v12:

Use the TYPO3 Quickstart to do a TYPO3 v12 install.
Use the same quickstart with ddev composer create "typo3/cms-base-distribution:^11" --no-install to install TYPO3 v11 and verify that it also works.

Mutagen:

Use ddev mutagen logs to see full verbose output
Run multiple projects at the same time using mutagen.
Mess up DDEV with docker kill ddev-router with a project running, then try ddev start to get out of trouble and see if you get out of trouble.
Extra credit: Run mutagen outside ddev and verify it doesn't cause any conflict. Consider [installing mutagen](brew install mutagen-io/mutagen/mutagen) and verifying it has no impact when you run mutagen daemon start. Verify that you can delete ~/.mutagen and have no impact on DDEV.

Global project_tld:

Verify that you can set the global project_tld and it takes effect for all projects.
Verify that you can override it with project-level project_tld

Yarn and npm caches:

With a project that has yarn v3, verify that https://github.com/drud/ddev/issues/4088 is not a problem.
Verify yarn and npm cache behavior using tests like @hanoii 's https://github.com/drud/ddev/pull/4234#issuecomment-1263540074

Old configured database (v1.18 database with this version of DDEV).

Create a project with a v1.18.2 version of DDEV (Just download the binary from releases). It can be an empty project, or a composer-created one, doesn't matter. ddev start it with old ddev. Then ddev poweroff and ddev start with this version of ddev. It should work. See the problem in https://github.com/drud/ddev/issues/4129

Try using ddev with an old docker-compose v1 version. (https://github.com/drud/ddev/issues/4135). You can do this in Docker Desktop by switching to docker-compose v1 in the settings. Then ddev config global --use-docker-compose-from-path. DDEV's output should help you figure out what's wrong.
Use a docker-compose.*.yaml and remove the version line. It should still work fine. For example, ddev get drud/ddev-adminer and then edit the docker-compose.adminer.yaml to remove the version line and then ddev restart and use the adminer interface.
postgres:9 detection: Create a project using --database=postgres:9 and start it. Then ddev restart. It should work. Then move to postgres:10, it should complain. Use. ddev debug check-db-match to see if the installed database matches the configured database with a variety of differences.
Database migration to different version: Try migrating a mariadb or mysql database to another version using ddev debug migrate-database <target_type:target_version>
composer_root and $PATH: Create a project with a nonstandard composer root. For example, put the composer.json in docroot and set composer_root to docroot. Then after ddev composer install things in docroot/vendor/bin should be in $PATH. For example, for Drupal, ddev drush should work. Or ddev exec typo3cms.
 Test out the no-internet lifestyle.
Use ddev debug test:

It should refuse to run if not in a project
It should fail to run in home directory
It should show no obvious errors
It should refuse to run if the parent directory is not writable
Review its output of course

Verify DDEV behavior with fig, see https://github.com/drud/ddev/pull/4216
Using mutagen, make sure that .ddev/.downloads is available inside ddev ssh and that pull commands like ddev pull platform that use it still work with mutagen enabled. (https://github.com/drud/ddev/issues/4193)
Verify that zsh completion works with ddev installed using homebrew. (#4179)
If mutagen is enabled, and project type php or a project type without a default upload_dir (like craftcms), verify that ddev warns about that on start, and that the warning goes away if you set an upload_dir.
Mutagen on traditional Windows: binaries should be executable using mutagen in traditional Windows, so try out things like ddev drush or ddev exec typo3cms to make sure they work using Mutagen.

✅ PHP8.2: I had no trouble installing Drupal 10.0.0-dev with PHP8.2 (rc3) using macOS on M1. I did have serious slowness issues on WSL2 with docker-ce inside, but couldn't figure out if it was the computer or something else. The commands, even ddev config and ddev describe had a noticeable delay.

I think it's good!

✅ PHP8.2

Using WSL2 with Docker Desktop I didn't have any slowness.

Setup Drupal 10

ddev config --project-type=drupal10 --docroot=web --create-docroot

install Drush

ddev composer require drush/drush

Install site

ddev drush si --site-name=d10-dev -y

Visited site in browser and confirmed working.

Test

Check PHP

$ ddev php -v
PHP 8.1.11 (cli) (built: Sep 29 2022 22:17:15) (NTS)

Re-install site

ddev drush si --site-name=d10-dev -y

Took 32s

Stop DDEV and (manually) update config to PHP 8.2
Check PHP

$ ddev php -v
PHP 8.2.0RC3 (cli) (built: Sep 29 2022 22:12:49) (NTS)

Re-install site

ddev drush si --site-name=d10-dev -y

Took 22s

ddev descibe felt about the same (or snappier)

Thanks so much @tyler36 !


Tainted canvas or CORS policy issue

Setup 1
    <croppa
      v-model="imgData"
      placeholder="Upload Picture"
      :accept="'image/jpeg,image/jpg,image/png'"
      :quality="1.5"
      :width="$vuetify.breakpoint.xs ? 130 : 150"
      :height="$vuetify.breakpoint.xs ? 130 : 150"
      @image-remove="onImageRemoved()"
      @new-image-drawn="onNewImageDrawn()"
      @move="onMove()"
      @zoom="onZoom()"
    >
      <img v-if="initialUrl" slot="initial" src="initialUrl">
    </croppa>

Issue 1
Using setup 1, I'm ending up getting a Tainted canvases may not be exported. error when selecting a new picture locally (users can upload a new profile picture) and trying to generateBlob from the image data in orderto later save it on the server.

Setup 2
Adding crossorigin="anonymous" in front of :src="initialUrl"
Issue 2
Using setup 2, I'm ending up getting a No 'Access-Control-Allow-Origin' header is present on the requested resource. error on the initial load of the image which is stored and loaded from firebase storage. The image is there but cannot be shown by the img-element. Otherwise I have zero issues with firebase files so I don't see a point in experimenting with the CORS policy on that end, changing settings on the server.
Any idea how I can resolve that issue? Funny enough, I had it working with setup 1 for quite a while without an issue but after some changes I cannot fully reproduce, it has now stopped.

https://stackoverflow.com/questions/55570883/tainted-canvas-or-cors-policy-issue-with-croppa

I am also facing same issue @alexeigs.

@alexeigs same issue, did you find a solution?


in iscsi deployment we need to use only he_iscsi_portal_addr and not he_storage_domain_addr

in iscsi deployment, we have 2 different variables that are the same:

he_iscsi_portal_addr
he_storage_domain_addr

I think that we need to use:

he_storage_domain_addr in case of nfs and glusterfs deployment
he_iscsi_portal_addr in case of iscsi deployment

he_iscsi_portal_addr appears just in tasks/iscsi_discover.yml and tasks/iscsi_getdevices.yml which are helpers tasks files for the the interactive iSCSI discovery via CLI and cockpit.
For the deployment only he_storage_domain_addr is relevant.


Latest commit (88e63a8) issues

After updating to the latest commit I'm running into a few issues.
My previous config (below) now has errors because line 17 in neoscroll/config.lua is expecting a table but is getting nil. The docs still use require("neoscroll").setup() in the quickstart section, so I don't think this is intended. This error will obviously go away if passing an empty table or any custom opts.
  {
    "karb94/neoscroll.nvim",
    event = { "BufReadPost", "BufNewFile" },
    config = function()
      require("neoscroll").setup()
    end,
  },

If the error is ignored then no scrolling animations happen. If i pass a table (empty or with custom opts) and then for example use zb  I get the following error happening in neoscroll/init.lua line 298 "attempting to perform arithmetic on local 'half_win_duration' ( a table value)".

I'm getting the very same error after update.
Failed to run `config` for neoscroll.nvim

.../share/nvim/lazy/neoscroll.nvim/lua/neoscroll/config.lua:17: bad argument #1 to 'pairs' (table expected, got nil)

Any fix is much appreciated.

same here

Some new interfaces have bugs in duration calculation. For example: https://github.com/karb94/neoscroll.nvim/blob/88e63a89fee0f6112f0477a5709a460c45e5dee9/lua/neoscroll/init.lua#L242C31-L242C48

Update: these bugs don't affect many users because they remove all default mappings, and set up mappings with require('neoscroll.config').set_mappings, which calls the bug-free <cmd> interfaces.

Sorry about this. I'll fix it in a few hours.

Let me know if the last commit didn't fix all of your issues.

i have this config:
local scroll = require("neoscroll")
scroll.setup({
    mappings = { "zt", "zz", "zb" }
})
local mappings = {
    ["<m-j>"] = function() scroll.ctrl_d({ time = 100 }) end,
    ["<m-k>"] = function() scroll.ctrl_u({ time = 100 }) end,
    ["<m-h>"] = function() scroll.scroll(0.1, { move_cursor=false; time = 100 }) end,
    ["<m-;>"] = function() scroll.scroll(-0.1, { move_cursor=false; time = 100 }) end,
}
for key, func in pairs(mappings) do
  MAP({'n', 'v', 'x'}, key, func)
end

and this error

E5108: Error executing lua: .../share/nvim/lazy/neoscroll.nvim/lua/neoscroll/scroll.lua:127: attempt to perform arithmetic on field 'duration' (a nil value)
stack traceback:
.../share/nvim/lazy/neoscroll.nvim/lua/neoscroll/scroll.lua:127: in function 'compute_time_step'
...al/share/nvim/lazy/neoscroll.nvim/lua/neoscroll/init.lua:179: in function 'new_scroll'
...al/share/nvim/lazy/neoscroll.nvim/lua/neoscroll/init.lua:102: in function 'scroll'
...al/share/nvim/lazy/neoscroll.nvim/lua/neoscroll/init.lua:201: in function 'ctrl_d'


correct msg display in allocator

In the log message, I found that it is listing nodes while displaying services. I think it may be an error, so I tried to fix this.
Thanks
Signed-off-by: gexinlu<EMAIL_ADDRESS>

seems make sense.

Thanks @gexinlucode
LGTM

Current coverage is 55.07% (diff: 0.00%)

Merging #1443 into master will increase coverage by 0.13%

@@             master      #1443   diff @@
==========================================
  Files            82         82
  Lines         12901      12901
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           7088       7105    +17
+ Misses         4826       4814    -12
+ Partials        987        982     -5

Powered by Codecov. Last update 9028bcb...0ae4dd5

LGTM


using external data for reference price?

Hi. I was hoping to experiment using stakemachine with UIA rather than Bitassets. Do you have plans to expand examples or functionality to take price from a local file (e.g. which would look up an rss feed) instead of the "feed" that is only available to Bitassets? Then it can be used with regular UIA assets, getting price data from centralized exchange api's. Thanks!

I'm working on a solution in the same vein, not least of all because this doesn't seem to support bitAsset:bitAsset pairs, off the git-shelf.


OpTestCAPI: Add CAPI Testcases.

This patch is ported from the below PR. Which is the initial
work done by Philippe Bergheaud.
https://github.com/open-power/op-test-framework/pull/94
Signed-off-by: Pridhiviraj Paidipeddi<EMAIL_ADDRESS>

@philippe56 I have ported #94 to new framework, Can you have a look at it. I don't have a system with CAPI card to test this. It will be good, if you can test this.

@stewart-ibm @philippe56 Got a system with CAPI card and ran the tests, all are working fine.

[PEXPECT]# test -x /tmp/cxl-tests/libcxl_tests;
test -x /tmp/cxl-tests/libcxl_tests;
[PEXPECT]# echo $?
echo $?
0
Executable file /tmp/cxl-tests/libcxl_tests is available
[PEXPECT]# test -x /tmp/cxl-tests/libcxl/libcxl.so;
test -x /tmp/cxl-tests/libcxl/libcxl.so;
[PEXPECT]# echo $?
echo $?
0
Executable file /tmp/cxl-tests/libcxl/libcxl.so is available
cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./libcxl_tests >/tmp/libcxl_tests.log;
[PEXPECT]# cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./libcxl_tests >/tmp/libcxl_tests.log;
cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./libcxl_tests >/tmp/libcx^Mxl_tests.log;
[PEXPECT]# echo $?
echo $?
0
sysfs abi libcxl_tests pass

[PEXPECT]# modprobe cxl
modprobe cxl
[PEXPECT]# echo $?
echo $?
0
[PEXPECT]# lsmod | grep -i --color=never cxl
lsmod | grep -i --color=never cxl
cxl                   246458  0
pnv_php                15588  1 cxl
[PEXPECT]# echo $?
echo $?
0
cxl module is loaded
[PEXPECT]# test -x /tmp/cxl-tests/memcpy_afu_ctx;
test -x /tmp/cxl-tests/memcpy_afu_ctx;
[PEXPECT]# echo $?
echo $?
0
Executable file /tmp/cxl-tests/memcpy_afu_ctx is available
[PEXPECT]# test -x /tmp/cxl-tests/libcxl/libcxl.so;
test -x /tmp/cxl-tests/libcxl/libcxl.so;
[PEXPECT]# echo $?
echo $?
0
Executable file /tmp/cxl-tests/libcxl/libcxl.so is available
cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./memcpy_afu_ctx -p1 -l1 >/tmp/memcpy_afu_ctx.log; echo $?
[PEXPECT]# cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./memcpy_afu_ctx -p1 -l1 >/tmp/memcpy_afu_ctx.log; echo $?
cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./memcpy_afu_ctx -p1 -l1 >^M>/tmp/memcpy_afu_ctx.log; echo $?
0
[PEXPECT]# echo $?
echo $?
0
memcpy_afu_ctx tests pass

[PEXPECT]# grep -q -w 1 /sys/class/cxl/card0/psl_timebase_synced;
grep -q -w 1 /sys/class/cxl/card0/psl_timebase_synced;
[PEXPECT]# echo $?
echo $?
0
[PEXPECT]# test -x /tmp/cxl-tests/memcpy_afu_ctx;
test -x /tmp/cxl-tests/memcpy_afu_ctx;
[PEXPECT]# echo $?
echo $?
0
Executable file /tmp/cxl-tests/memcpy_afu_ctx is available
[PEXPECT]# test -x /tmp/cxl-tests/libcxl/libcxl.so;
test -x /tmp/cxl-tests/libcxl/libcxl.so;
[PEXPECT]# echo $?
echo $?
0
Executable file /tmp/cxl-tests/libcxl/libcxl.so is available
cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./memcpy_afu_ctx -t -p1 -l1 >/tmp/memcpy_afu_ctx-t.log; echo $?
[PEXPECT]# cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./memcpy_afu_ctx -t -p1 -l1 >/tmp/memcpy_afu_ctx-t.log; echo $?
cd /tmp/cxl-tests && LD_LIBRARY_PATH=libcxl ./memcpy_afu_ctx -t -p1 -l^Ml1 >/tmp/memcpy_afu_ctx-t.log; echo $?
0
[PEXPECT]# echo $?
echo $?
0
memcpy_afu_ctx -t tests pass
OK (8.150s)

----------------------------------------------------------------------
Ran 4 tests in 34.046s

OK

Generating XML reports...

Great achievement, thank you.

Thanks! Merged to master as of 9672fa433c89


Johluo/revert to 3.1

Going back to netcoreapp3.1

@davidfowl to confirm.
Also, reopen any issues that were closed by reverts.


Revert "Prefer memory-fs over real fs"

Reverts zinserjan/mocha-webpack#152 due to #163

Codecov Report

Merging #164 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #164   +/-   ##
=======================================
  Coverage   96.77%   96.77%
=======================================
  Files          28       28
  Lines         745      745
=======================================
  Hits          721      721
  Misses         24       24

Impacted Files
Coverage Δ

src/util/registerRequireHook.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks for the fix @zinserjan !

No problem, it's already released ;)


koda art announcement message on landing

once confirmed by devs, we will soon post about it on twitter and it would be nice to put a announcement on a landing page

its in the figma handoff - small things page
https://www.figma.com/design/JXNtryaNYhY0XJNArzvPQx/Handoff?node-id=1287-18959&t=CTRefK6KIq5Q9YQM-11
design
its the same as when announcing our warpcast drops, but instead of the preview there is information icon
notes
merge the pr once twitter post is done - will let you know

its the same as when announcing our warpcast drops, but instead of the preview there is information icon

It also displays in the same position on landing page? @exezbcz
I'm think about we could still show warpcast drops announcement on the landing if user has already using koda.art.


what is a good loss value finally?

I download your code and trained among 100 million sentences, eventually I got a loss of 0.44, can it be a good one?

It depends. The final quality cannot be directly measured by loss values.


Log a minor retriable error as info instead of an error

How to categorize this PR?

/area quality
/kind enhancement
What this PR does / why we need it:
There is an transient error that gets logged as an actual error. We will just log it as info
Which issue(s) this PR fixes:
Fixes: https://github.com/gardener/gardener/issues/8148
Release note:

NONE

/retest

/retest

/retest


snapshot restore for worker nodes

https://github.com/rancher/rancher/issues/25647

Adding a separate approach for restoring rancher worker nodes, and not reusing the restore flag for rke config is a good idea, and using annotation sounds good to me. When resetting that annotation back to false, do we need retries for cluster update?

@mrajashree Yeah, I'll add retries for cluster update

Ik we already do it but is there something we can do instead of gating with conditions more?

@rmweir I understand, but probably not at this point since I've been using cluster/node conditions heavily for this feature


Ability to search for & launch Mono executables on all platforms

See also: https://github.com/itchio/itch/issues/1606 https://github.com/OpenRA/OpenRA/pull/14703 https://github.com/ZeroK-RTS/Zero-K-Infrastructure/issues/2076
I don't see any .go files mentioning Mono. Currently, I have to pass Itch mono game.exe in the manifest with my own symlink to mono. It should be able to directly handle game.exe (or look for it).
If this isn't what dash is for, I apologize. I'm making assumptions of what this tool does :)

Note that even though we don't have a mono prereq, dash will properly identify .NET assemblies:
 bleed ツ butler configure . --no-filter
• Not filtering, by request
✓ Configured in 84.648427ms
✓ Final candidates are:
| 98 MiB .
|-- 486 B data/Bleed script- sh(interpreter=/bin/bash)
|-- 12 MiB data/Bleed.bin.x86 linux-386
|-- 15 MiB data/Bleed.bin.x86_64 linux-amd64
|-- 6.7 MiB data/Bleed.exe windows-386 win(dotNet)
|-- 1.3 MiB data/lib/libSDL2-2.0.so.0 linux-386
|-- 182 KiB data/lib/libmojoshader.so linux-386
|-- 608 KiB data/lib/libopenal.so.1 linux-386
|-- 1.3 MiB data/lib64/libSDL2-2.0.so.0 linux-amd64
|-- 188 KiB data/lib64/libmojoshader.so linux-amd64
|-- 620 KiB data/lib64/libopenal.so.1 linux-amd64
|-- 26 KiB guis/libmojosetupgui_gtkplus2.so linux-386
|-- 38 KiB meta/xdg-utils/xdg-desktop-menu script- sh(interpreter=/bin/sh)
|-- 2.7 KiB meta/xdg-utils/xdg-open script- sh(interpreter=/bin/sh)

(this is Bleed)


Date of update on firmware?

Hi Eried. Can you please add a date or a version number to the page of the firmware update?
Its easier to keep track when firmware is updated to your mayhem hat.
Thanks

hi, yes good idea. I still need to structure things, so it is mainly WIP for now. Thats why I havent polished things

Thanks a lot buddy. It's just easier to know when a new firmware is
available

yes, next update will at least have datetime somewhere 👍


Video Playback through main video or screen share.

Describe the issue
I am attempting to record the screen share from a participant then after stopping the recording I want to playback what was recorded through the bot into the meeting.
Code Snippet
In the bot initialization:
`AudioVideoFramePlayerSettings settings = new AudioVideoFramePlayerSettings(new AudioSettings(20), new VideoSettings(), 1000);
framePlayer = new AudioVideoFramePlayer((AudioSocket)this.audioSocket, (VideoSocket)this.mainVideoSocket, settings);`
Pipe buffer from the screen share into a list of buffers:
`List screenShareBufferList = new List();
public void recordScreenShare(VideoMediaBuffer buffer)
{
if (shouldRecordScreenShare)
{
//var newVideo = new VideoSendBuffer(buffer.Data, buffer.Length, VideoFormat.NV12_1920x1080_15Fps, buffer.Timestamp);
screenShareBufferList.Add(buffer);
}
}`
Play the recorded video mostly taken from some other example to try and make it work:
`private const double TicksInOneMs = 10000.0;
private const double MsInOneSec = 1000.0;
public async Task PlayScreenRecord()
{
var videoMediaBuffers = new List();
var referenceTime = DateTime.Now.Ticks;
var packetSizeInMs = (long)((MsInOneSec/  VideoFormat.NV12_1920x1080_15Fps.FrameRate) * TicksInOneMs);

foreach (var buffer in screenShareBufferList)
{

    /*nint data = buffer.Data;
    IntPtr unManageBuffer = Marshal.AllocHGlobal(data);*/
    var videoBuffer = new VideoSendBuffer(buffer.Data, buffer.Length, VideoFormat.NV12_1920x1080_15Fps, referenceTime);
    videoMediaBuffers.Add(videoBuffer);
    referenceTime += packetSizeInMs;
}
await SendFrameVideo(videoMediaBuffers);
//await SendFrameVideo(screenShareBufferList);

}`
The frame player, I am not sending audio right now, but I feel like this should not be a factor:
`public async Task SendFrameVideo(List buffer)
{
try
{
    await framePlayer.EnqueueBuffersAsync(null, buffer);
}
catch (Exception ex)
{
    this.logger.Error(ex, $"[OnVideoReceived] Exception while calling SendFrameVideo");
}

}`
Expected behavior
I expect to have video play over the screen share. Currently when I play it will terminate my local debugging session and provides no errors in the try catch block.
Any help would be appreciated, I am not doing this too far away from how the audio is being handled and I can play audio over the meeting. If I attempt to use the vbss socket, I lose all sharing with the bot and still have the same issue of it crashing the app.

Not sure I fully understand your code, but it looks like you are not saving the buffer properly. You only keep references to buffers which are then dropped. Hard to tell though.

Not sure I fully understand your code, but it looks like you are not saving the buffer properly. You only keep references to buffers which are then dropped. Hard to tell though.

I am using the variable "screenShareBufferList" to keep the buffers when in the 'recording' mode then when I attempt to send the video I am trying to recreate the video buffers to strip away the original format and things because I originally got errors when attempting to send over the video pipeline.

So for the most part I am able to get a single frame to show up on the media pipeline. But when I perform the working code it is not storing the data and seems to just grab the latest copy of the image and show it. When I debug it looks like the pointer is being reference by each of my list items. I am not super familiar with how video images are handled, but it looks to be a series of bytes, but when I try and copy the bytes they seem to just be overridden. The code is as follows:
This is to get a new copy of the buffer and load it into a new VideoSendBuffer object, in theory I would think it is a new copy of the frame.
byte[] newBuf = new byte[buffer.Length];                 Marshal.Copy(buffer.Data, newBuf, 0, (int)buffer.Length);                 var videoBuffer = new VideoSendBuffer(newBuf, (uint)buffer.Length, VideoFormat.NV12_1920x1080_15Fps);                 screenShareBufferList.Add(videoBuffer);
then I send these objects to the frame player but all I see if the latest image from the screen share. When I debug the list of objects they all seem to point to the same pointer, which makes no sense since I made new objects with what I thought was a new copy of the byte array.

Hey ebade, if you're working on recording and playing back screen share video buffers through a bot, you might want to check out the Recall.ai API.
It’s a simple 3rd party API that lets you use meeting bots to get raw audio/video/metadata from meetings without you needing to spend months to build, scale and maintain these bots.
Here are the API docs: https://docs.recall.ai


support more read/write options

PTAL @zhangjinpeng1987 @siddontang

Please add test for the options.

LGTM


Latency and cycle literals should not start with uppercase letter

§<IP_ADDRESS>.2:

Certain sets of names and function signatures are always reserved to the implementation:
— Each name that contains a double underscore _ _ or begins with an underscore followed by an uppercase letter (2.12) is reserved to the implementation for any use.
— Each name that begins with an underscore is reserved to the implementation for use as a name in the global namespace.

https://github.com/MIPT-ILab/mipt-mips/blob/d162f41637c1acc32a6bd0918749a433a8ab65da/simulator/infra/ports/timing.h#L51
Ooops...

That's even worse. Nothing prevents some person or linter tool from adding a space here, and it may silently lead to future bugs.


fix duplications detected in evidence sources

@elipapa commented on Wed May 16 2018
We currently use a set of fields in each evidence to create a hash/unique id and use that to write in ES.
We found that we actually have a lot of collisions, most of which are false positives.
We need to determine a list of specific fields for each datatype that uniquely determines the identity of an evidence and use that as the hash.
number of duplicated evidences= 1145976
number of uniq ids with duplication  566525
datasource        ids    duplications
eva_somatic                 1             2
progeny                11            22
reactome                21            42
eva               239           481
expression_atlas             40537         93629
europepmc            520482       1040964
gene2phenotype                13            26
phewas_catalog              4577          9396
gwas_catalog               533          1086
intogen               111           328

@ckongEbi commented on Wed May 16 2018
The use of unique_association_field from JSONS to generate the unique evidence id in the platform backend can be problematic, as it might not fit the uniqueness requirements thus duplicates.

From @ckongEbi on July 17, 2018:

EVA - 106 (Despite adding rs_id to unique_association_field, probably due to >1 clinvar traits mapping to same EFO terms, needs following up with EVA & @gkos-bio)
Update 17JUL2018: Partially fixed in 18.08. Ontology mapping required @adam and Gautier.

GWAS - 784 (Olga has flag this to GWAS curator for checking)
Update 17JUL2018- Under investigation.

ATLAS - 40459 (Suhaib planned to add probe_id to unique_association_field aim this will solve the issue)
Update 17JUL2018: Fixed in 18.08.

EUROPEPMC - 519807 (example sent to them but no feedback yet)
Update 17JUL2018: Need to follow-up.

PHEWAS - 117 (check with Eliseo)
Update 17JUL2018: Need to follow-up.

And another email later that same day:
Look for rows for counts > 1, which will be duplicates

EVA: 1806_dup_eva_2.txt
GWAS: 1806_dup_gwas_2.txt
PHEWAS: 18.06_phewas_2.txt

EPMC & ATLAS:
https://drive.google.com/open?id=1levWwb8FgsmGm2bPsbS8ZZGOK1OAPSFv

https://github.com/opentargets/platform/issues/60
Closing this on roadmap as being tracked and addressed under platform repository data related issue.


Indigenous consultation level in history

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

:exclamation: No coverage uploaded for pull request base (develop@e7323b4). Click here to learn what that means.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1820   +/-   ##
==========================================
  Coverage           ?   68.23%
==========================================
  Files              ?      252
  Lines              ?     8606
  Branches           ?        0
==========================================
  Hits               ?     5872
  Misses             ?     2734
  Partials           ?        0

Flag
Coverage Δ

epictrack-api
68.23% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Only for 64 bit?

On my phone it gives me parsing error. My phone is 32 bit, Android 10. So I guess it either doesnt support 32 bit or android 10.

@osjasmine Try version 0.1.3 https://github.com/jroddev/android-oss-release-tracker/releases/tag/0.1.3
I have reduced the minSdk version from 30 to 29 to support Android 10 and everything seems to work.
So far only tested in an emulator so it would be good to get confirmation on an actual device.

Thanks the latest version installed with no problems.


Fix desktop endpoint set by browser flag

This pull request fixes an issue where the desktop endpoint was not correctly set by the browser flag. By passing --browser  to the configuration, the application now listens on the specified request port, ensuring proper handling of desktop requests.
Close #154

Thank you so much for the fix! Appreciate it!


OpenSearchVectorStore's similarity search functions can't handle k>10 with script_scoring

System Info
Langchain version 0.0.225
M1 Mac
Python 3.11.3
Who can help?
@naveentatikonda
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.embeddings import HuggingFaceEmbeddings
from langchain.vectorstores import OpenSearchVectorSearch
from opensearchpy import RequestsHttpConnection
from requests_aws4auth import AWS4Auth
import os

aws_auth = AWS4Auth(
    os.environ["AWS_ACCESS_KEY"],
    os.environ["AWS_SECRET_KEY"],
    "us-east-1",
    "es"
)
embeddings = HuggingFaceEmbeddings(
    model_name="sentence-transformers/distiluse-base-multilingual-cased-v2",
    model_kwargs={'device': 'cpu'}
)
opensearch_vector_search = OpenSearchVectorSearch(
    os.environ["OPENSEARCH_URL"],
    "testindex",
    embeddings,
    http_auth=aws_auth,
    connection_class=RequestsHttpConnection,
)
texts = []
for i in range(1, 41):
    texts.append(f"This is document {i}")
opensearch_vector_search.add_texts(texts)

query = "This is a document"
search_result = opensearch_vector_search._raw_similarity_search_with_score(
    query,
    k=30,
    metadata_field="*",
    search_type="script_scoring"
)
print("Number of docs in response: ", len(search_result))

Output:
Number of docs in response:  10

When using script_scoring, the code does not pass a "size=k" option to the client query, so it returns a maximum of 10 hits by default.  Therefore, the user cannot receive more than 10 results regardless of the value of k.  I believe all that is necessary it to add a size=k parameter to line 537 in opensearch_vector_search.py, and then the [:k] in line 539 is superfluous.
Expected behavior
The above code should produce 30 results, not 10.

I just reproduced this with AWS OpenSearch as well.  My index has 41 documents, I request 30, and only get 10.
All texts are the same in my index.
results = docsearch.similarity_search("What's the best way to enjoy butter?", k=30, search_type="script_scoring",metadata_field="*")
print(len(results))

I'm looking into this now.  I think the bot's suggestion doesn't hurt but I'm not sure it will fix this.

Ok, so the bot's suggestion should work
def _raw_similarity_search_with_score(
        self, query: str, k: int = 4, **kwargs: Any
    ) -> List[dict]:
        # ... existing code ...
        elif search_type == SCRIPT_SCORING_SEARCH:
            # ... existing code ...
            search_query["size"] = k  # Add this line
        # ... existing code ...
        return [hit for hit in response["hits"]["hits"][:k]]

Likely the reason why you encounter this is that OpenSearch's default k is 10, so it needs to be specified.  The [:k] doesn't hurt and may need to remain for other search methods.
Here's a similar question about its cousin ElasticSearch
https://discuss.elastic.co/t/elasticsearch-returns-only-10-records-but-the-hit-is-507/136605
Here's my own test putting "size": 30 into the query and you can see the number of results is about 46800/1536 (dividing by the vector's dimension) is 30.4...
I'll open a PR in a sec.

Thanks @aerickson-clt.  Looking at the code it seems like the "painless_scripting" option might have the same problem, but I haven't tested it.

Yes, I fixed that too.


Add PSR11 Wrapper Convenience Method

Hi! Our team loves Pimple with a passion. We've recently started using more libraries that contract against PSR-11 containers. We end up wrapping the Pimple container at the call site in a new instance of the Psr11\Container but this proposed change would allow us to just pass in $container->toPsr11() without our implementations having to be aware or coupled to that Psr11\Container class at all.
I'd imagine as the spec is adopted further in other libraries this could be more useful as time goes on.
I know the goal of this library is to stay small and trim, so I completely understand if it's not a worthwhile addition. Thanks for all your hard work on it over the years. It's the backbone of many great apps!

HA! I should have read the top of the README first. I was looking for a non existent "contributing" file lol. Sorry.


allows snapping to grid when using multi arrow and line

This PR fixes #2009 and allows for the snapping to the grid when using a multi arrow or multi-line element

Ideally, the point preview (while moving mouse) should snap to grid as well (same as it does when creating a new point via Alt/Option while editing an existing line):

Will update to include point preview

A friendly reminder @mauriceLC92 :)

@lipis reviving this now! 🙏

@mauriceLC92 forgot about this PR. What's the current status? AFAIK this PR doesn't do anything other than aligning to grid points in one branch which are already grid-aligned :).


Angular 1.5 RC 0: ng-show bug

We attempted to upgrade from 1.4.8 to 1.5 RC to start using .component. However, when we did this, the following line of markup stopped working in Chrome and IE 11 (but it works in Firefox):
<div ng-repeat="ticketType in vm.ticketTypes" ng-show="vm.ticketTypeCount(ticketType) > 0">

Under 1.5, all of the items generated by the ng-repeat get hidden. This is very puzzling. Are we doing something wrong here, or is this a bug?

@azurelogic, could you create a live demo (e.g. using CodePen, Plnkr etc), so it is easier to reproduce and investigate the issue ?

I tried to reproduce it here, but everything works as expected.

@gkalpak, that's interesting. Your sample works fine. I'll have to try to work up a plunker when I have a moment.
It's a really weird bug. I thought it was something in our ticketTypeCount function, but it worked in Firefox, and there are no console errors. Funny thing is that it works in Chrome if I put in ng-show="1 > 0" too. I thought maybe it had to do with the actual expression evaluation. It's just so strange.

That's indeed strange. Could it be something with the data (based on which ticketTypeCount() calculates its return value) ? Try logging the return value of vm.ticketTypeCount(ticketType) and see what you get in each browser.

@azurelogic Have you tried with the current snapshot:
http://code.angularjs.org/snapshot/angular.js
http://code.angularjs.org/snapshot/angular-animate.js

Closing due to no feedback. Feel free to reopen if you have more information


Can't create a new client when using custom domain name

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?

[ ] elastic.v2 (for Elasticsearch 1.x)
[x] elastic.v3 (for Elasticsearch 2.x)

Please describe the expected behavior
When creating a new client, I should be able to pass my custom domain/port to it and get a successful connection.
logElastic := "http://elasticsearch:9200"
client, err := elastic.NewClient(elastic.SetURL(logElastic))
logrus.Infof("New client: %v - url: %v", client, logElastic)
if err != nil {
  logrus.Fatal(err)
}

Please describe the actual behavior
The client returns nil :(
Any steps to reproduce the behavior?

Create a new elasticsearch instance with a custom domain of http://elasticsearch:9200
Create a new client with the above url and see that it fails with:

INFO[0005] New client: <nil> - url: http://elasticsearch:9200
FATA[0005] no Elasticsearch node available
make: *** [cli-dev] Error 1

Create a curl request for the same url and see it work:

curl -i http://elasticsearch:9200
HTTP/1.1 200 OK
Content-Type: application/json; charset=UTF-8
Content-Length: 313

{
  "name" : "Dusk",
  "cluster_name" : "elasticsearch",
  "version" : {
    "number" : "2.3.0",
    "build_hash" : "8371be8d5fe5df7fb9c0516c474d77b9feddd888",
    "build_timestamp" : "2016-03-29T07:54:48Z",
    "build_snapshot" : false,
    "lucene_version" : "5.5.0"
  },
  "tagline" : "You Know, for Search"
}

Odd... I do that all the time, so maybe it's a regression.
But it looks like the URL is not the source of the problem.
Are you using Docker? Does the Wiki section on connection problems help? Can you temporarily disable sniffing to see if that makes a difference  (see here)?

I am indeed using this stack on Docker - Thanks for the different links for help, I'll see if I can get it sorted it out.

A bit more info - I used your docker-compose setup, which uses the new network stuff.
Doing so, I can get my app to connect. Using the old stuff, when I do:
ips, err := net.LookupIP("elasticsearch")

I get the following error:
FATA[0010] lookup elasticsearch on <IP_ADDRESS>:53: no such host

Long story short it seems to be an issue within the container networking, not the actual lib. Thanks for helping me troubleshoot this.

If you enable sniffing (the default), the URL you specify is merely the starting point of a lookup process. It uses your URL to do a cluster_info API calls to get all nodes of the cluster. Then it uses all the found nodes to run requests. If the nodes returned by the cluster info API are not accessible, you've got a problem. So you need to ensure that the nodes in your cluster are available to the outside, as written in the Docker wiki entry.

In case you were wondering I managed to get this resolved, thanks to your https://github.com/olivere/elastic-with-docker repo.
I was still using golang:1.5-alpine as the base image, switching that over to golang:1.6-alpine made everything work like magic!
Thanks for the tip.


refactor(pkg): added missing nameserver fields in the client area Transfer action

:tada: This PR is included in version 1.1.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Production build results in mangling of class extends

System Info
System:
OS: macOS 13.3.1
CPU: (8) arm64 Apple M1 Pro
Memory: 2.97 GB / 32.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.16.0 - ~/.nvm/versions/node/v18.16.0/bin/node
Yarn: 1.22.19 - /opt/homebrew/bin/yarn
npm: 9.5.1 - ~/.nvm/versions/node/v18.16.0/bin/npm
Browsers:
Chrome: 112.0.5615.49
Safari: 16.4
Details
Input code:
import { WrappedIotsType } from '../WrappedIotsType';

export class StringEncodedObject extends WrappedIotsType {
  constructor(
  ) {
   ...
  }
}

Production build result:
__webpack_require__(\"902067\");\nclass StringEncodedObject extends WrappedIotsType { ...

I know the required module number is correct (in other parts of the bundle I see code like new _902067.WrappedIotsType). It seems like in this specific case, there is no variable assignment of the require. Ie I would expect
var _902067 = __webpack_require__(\"902067\");\nclass StringEncodedObject extends _902067.WrappedIotsType { ...

This seems to work fine in development mode.
var _wrappedIotsTypeTs = __webpack_require__(\"../common_types/src/defs/WrappedIotsType.ts\"); class StringEncodedObject extends _wrappedIotsTypeTs.WrappedIotsType {

Reproduce link
No response
Reproduce Steps
Extend a class in production mode.

Would you like to provide a reproduction using https://github.com/web-infra-dev/rspack-repro?

I was able to fix the issue by setting sideEffects: false in the package.json.

Would you mind providing more info about this bug, although you have worked around it, but if you give us more info about this issue like providing a reproduce repo link, we could fix it.