Support for JRuby 9000

When I tried to install interception under jruby-<IP_ADDRESS>.pre1, I got this error:
jrubyc --javac org/pryrepl/InterceptionEventHook.java
NameError: missing class or uppercase package name (`org.jruby.compiler.impl.StandardASMCompiler'), caused by (NameError) cannot load Java class org.jruby.compiler.impl.StandardASMCompiler
  get_proxy_or_package_under_package at org/jruby/javasupport/JavaUtilities.java:54
                      method_missing at file:/home/dbernier/.rvm/rubies/jruby-<IP_ADDRESS>.pre1/lib/jruby.jar!/jruby/java/java_package_module_template.rb:14
                              (root) at /home/dbernier/.rvm/rubies/jruby-<IP_ADDRESS>.pre1/lib/ruby/stdlib/jruby/compiler.rb:10
                              (root) at /home/dbernier/.rvm/rubies/jruby-<IP_ADDRESS>.pre1/lib/ruby/stdlib/rubygems/core_ext/kernel_require.rb:1
                             require at org/jruby/RubyKernel.java:954
                             require at /home/dbernier/.rvm/rubies/jruby-<IP_ADDRESS>.pre1/lib/ruby/stdlib/rubygems/core_ext/kernel_require.rb:54
                              (root) at /home/dbernier/.rvm/rubies/jruby-<IP_ADDRESS>.pre1/lib/ruby/stdlib/jruby/jrubyc.rb:1
                             require at org/jruby/RubyKernel.java:954
                              (root) at /home/dbernier/.rvm/rubies/jruby-<IP_ADDRESS>.pre1/lib/ruby/stdlib/rubygems/core_ext/kernel_require.rb:1
                          __script__ at /home/dbernier/.rvm/rubies/jruby-<IP_ADDRESS>.pre1/bin/jrubyc:3
Makefile:2: recipe for target 'install' failed
make: *** [install] Error 1

make failed, exit code 2

I checked the jruby repo, and confirmed that StandardASMCompiler.java isn't in the master branch. (That link points to the 1_7 branch.)
I'm guessing this is related to the new runtime they've been blogging about.
This is a low-priority problem for me. I only noticed it because I was trying to install did_you_mean, which depends on interception, but I can skip that for now.

I'm sure this is fixed. I just installed interception on JRuby <IP_ADDRESS> and it builds fine.

Very likely. I haven't messed around with this in a while.
I'll close this issue.


IdentityApi -> Just usable with ExternalId

The class IdentityApi just uses methods with ExternalId as arguments. Should also be able to use with ID String too.

Closing because it can't be used in the desired way.


i18n(fr): update guides/i18n.mdx

Description
This PR updates the French version of the guides/i18n.mdx page with the changes of #1620 and #1798.

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

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

Tracked Files

Locale
File
Note

fr
guides/i18n.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

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


Responding to challenge fails with "Provided key authorization was incorrect"

I'm trying to use letsencrypt.sh to generate certificates, but I'm getting an error always.
I initially used Debian packaged version, but I then also tried the version from git.
Error is:
ERROR: An error occurred while sending post-request to https://acme-v01.api.letsencrypt.org/acme/challenge/CENSORED/CENCORED (Status 400)
Details:
{
"type": "urn:acme:error:malformed",
"detail": "Unable to update challenge :: Provided key authorization was incorrect",
"status": 400
}

And Apache logs show no attempt to access ".well-known/acme-challenge"

Since I got it working with official cerbot client I will not be able to do additional verification. Just for information.

Not sure what was causing this, closing this ticket since it seemed to be a one-off issue.


Release, please?

Due to my computer setup, I'm having a hard time setting up fxSDK. I would love to test this out though. Could you possibly provide a release?
Thanks! 😁

Hi Lukas1h,
I've just created a new release for you to test out — I too found it initially a bit difficult getting fxSDK working when I started working on Sinecraft so I understand the struggle!
There's only a .g1a file that I can provide at the moment, so I hope it works on your calculator (though of course note that this is highly unlikely to work with colour calculators like the fx-CG50).
Have fun and let me know what you think!
-James.

Got it downloaded.
Great job with the game. Looks very reminiscent of the original PE version that I used to love to play.
On average, what FPS is it suppose to run at? I'm only getting about 2 FPS on my fx-9750GIII.
Also, what are the default controls?

Deleting a few blocks helped the fps quite a bit. I thought maybe decreasing the fov would help to, but I didn't really notice much of a difference.
A section in the wiki about controls might be a good addition.

Good point! Just added a section to the wiki for the controls: https://github.com/James-Livesey/Sinecraft/wiki/Controls


[Improvement][MOD-150] Use existing notify method for reviews

Purpose

Use existing notify() for reviews notifications. We want reviews notifications to work the same way like OSF. notifications.
Changes

Update reviews model mixin
Update notify method to handle global_reviews event from reviews.
Update store method.
Update test

Side effects

Ticket
https://openscience.atlassian.net/browse/MOD-150

@sheriefvt do you have travis running on your branches (i.e. sheriefvt/osf.io as well as CenterForOpenScience/osf.io )?

@laurenbarker there is something wrong happened either with travis or github. I ll create another new PR and close this one.

Closing this PR and start a new one #7832


feedback on launching in ecs

amazing work - got it to work - saves me a lot of time . Here are somethings I noticed

it uses default vpc for some reason even if the cluster created is not on default vpc. I tried to have the cluster on a different vpc and it did not work.

If I have multiple aws credentials, and for some reason i pick a different one initially, it "remembers" the old credential id and keeps trying to use it and ecs secret management is not usable at this point. Had to clear out credentials.

I do not know what all roles are needed for the account that's needed to run this - had to use a highly privileged policy to get it to work. Would be good to list the policies needed by the runner for it to work. If it was documented , it was not clear.

Using the newly created context is somewhat a usability issue - not a big deal - but needs messaging IMO .

👍

ECS integration indeed uses default AWS VPC by default. Would make sense we detect the cluster is using a distinct one and use this one as default. => https://github.com/docker/ecs-plugin/issues/149
Also see https://docs.docker.com/engine/context/ecs-integration/ and use of x-aws-vpc to select another VPC
We rely on aws CLI "profiles" to select the adequate credentials. If you can offer a reproduction scenario I can check if we miss something
documented here: https://github.com/docker/ecs-plugin/blob/master/docs/requirements.md#permissions, let me know if something was missing
This is beta 1 ¯_(ツ)_/¯. Kidding apart, UX will significantly evolve in the incoming months, beta 1 is mostly a feature preview

a few more questions:
5. how can i set spot instances to be launched?
6. How can i set some of the instances to just have a private IP in a private subnet vs public subnet?

ECS integration only support Fargate Launch type for now, so you have no control on the ECS instances in use
the current design is to select the default subnets for Availability Zone in selected VPC (you can select a non-default VPC using x-aws-vpc extension). Not sure this would cover your need, feel free to open a dedicated issue explaining your use case.

thanks @ndeloof  - i was referring to this: https://aws.amazon.com/blogs/aws/aws-fargate-spot-now-generally-available/
I'll try 6 out and report back. BTW I was going off the demo and not your detailed docs - will give it a go with the docs!

ok I missed that. => https://github.com/docker/ecs-plugin/issues/151

I started investigating #151 but I can't find in AWS API how a Clutser is related to a VPC.
@listaction can you please clarify where in the web console, coudformation stack or any other way you setup your Cluster, you selected a specific VPC to be used by the cluster ?


Display one error message per field in error summary

Context
The error summary displayed all field errors grouped on one line.
This commit changes the error summary to only display the first error for a field.
This PR is a draft as we would like to discuss the testing approach.
Changes proposed in this pull request
Before

After

Guidance to review
To test this feature a model with two errors on one field was needed.
We added an extra validation to the existing Person class.
We create an invalid person for these tests and then expect to see the correct message in the error summary.
This feels brittle and we would like to discuss it.

We could also have a look at how error messages are displayed next to the form field inputs and add them to this PR?

We could also have a look at how error messages are displayed next to the form field inputs and add them to this PR?

If you think it fits in with this one, sure thing. Really don't mind if they're combined or separate

Looks good. I think that this behaviour is probably worth covering with a spec There's a shared context called 'a field that supports errors that is used by all of the error message including specs.
Something like this might work, welcome to hear your thoughts on the approach
index 4ee4ba0..258932d 100644
--- a/spec/support/shared/shared_error_examples.rb
+++ b/spec/support/shared/shared_error_examples.rb
@@ -19,6 +19,26 @@ shared_examples 'a field that supports errors' do
         id: error_identifier
       })
     end
+
+    context 'when there is more than one error' do
+      let(:first_error) { "It is totally wrong" }
+      let(:second_error) { "Very wrong indeed" }
+      let(:error_messages) { [first_error, second_error] }
+
+      before do
+        allow(object.errors.messages).to(
+          receive(:[]).and_return(error_messages)
+        )
+      end
+
+      specify 'the first error should be included' do
+        expect(subject).to have_tag('span', text: Regexp.new(first_error))
+      end
+
+      specify 'the second error should not be included' do
+        expect(subject).not_to have_tag('span', text: Regexp.new(second_error))
+      end
+    end
   end

   context 'when the attribute has no errors' do

We did look into writing a test to cover this but we couldn't decide on a nice way to stub object.errors.messages.
This is a much cleaner way Cheers. 👍

Great, thanks.
With regards to the difficult you found in stubbing object.errors.message, is there anything about the way the specs are set up that could be improved? I tried to keep it generic and re-usable but perhaps that was at the expense of it being a little opaque?
I'm happy with these changes, I'll merge shortly 👍🏽


feat(drain): add HA_ENABLED env to core-agent

Add the HA_ENABLED env to core-agent with {{ .Values.agents.ha.enabled }}

Wondering if it will be a good idea to pass this flag to csi-node as well? Behavior could be, if csi-node detects that ANA is not supported, it should CLBO unless HA_ENABLED is set to false.
@tiagolobocastro @Abhinandan-Purkait ?

Wondering if it will be a good idea to pass this flag to csi-node as well? Behavior could be, if csi-node detects that ANA is not supported, it should CLBO unless HA_ENABLED is set to false. @tiagolobocastro @Abhinandan-Purkait ?

Yeah, we can do that.
Only downside maybe is we will be restricting normal operations which can still happen without HA.

Wondering if it will be a good idea to pass this flag to csi-node as well? Behavior could be, if csi-node detects that ANA is not supported, it should CLBO unless HA_ENABLED is set to false. @tiagolobocastro @Abhinandan-Purkait ?

If Ha is not enabled then agent-ha-node is not running anyway so it will not trigger any ha operations

Wondering if it will be a good idea to pass this flag to csi-node as well? Behavior could be, if csi-node detects that ANA is not supported, it should CLBO unless HA_ENABLED is set to false. @tiagolobocastro @Abhinandan-Purkait ?

If Ha is not enabled then agent-ha-node is not running anyway so it will not trigger any ha operations

I'm suggesting for csi-node, not agent-ha. Kind of fail-fast approach, user will have to then set ha_enabled = false to proceed with normal operations.

bors merge


Suggestion & bug report

Hello! I would like to recommend that you add buttons to the transcripts as well! And I noticed that when a user sends a link it doesn't appear blue like on Discord...

will be implemented in v3

added in next release:

released!


Is there an implementation for Optimal Hand eye calibration available?

Hello,
I just read this paper:
https://www.robotic.dlr.de/fileadmin/robotic/stroblk/publications/strobl_2006iros.pdf
Which seems to be quite promising. Do you have an implementation of this?

No but there are 3 solvers: https://github.com/crigroup/handeye/blob/master/src/handeye/solver.py
The class names match the reference keys here: https://github.com/crigroup/handeye/blob/master/doc/bibliography.bib

Ok, i will implement that and create a PR when I am done.

Am 25.04.2019 um 01:48 schrieb fsuarez6<EMAIL_ADDRESS>No but there are 3 solvers: https://github.com/crigroup/handeye/blob/master/src/handeye/solver.py
The class names match the reference keys here: https://github.com/crigroup/handeye/blob/master/doc/bibliography.bib
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello,
Any updates on this?

I'm afraid not. I discarded that approach because I no longer needed it. Unfortunately, I have not found an implementation of this algorithm until today.


Using fly launch vs fly apps create?

This is part question and maybe a suggestion. Why does this action use fly launch instead of fly apps create?
I recently ran into an issue where the initial PR deployment of my app would not succeed due to healthchecks not being triggered. After getting some help in the community forums, it turn out that fly launch was modifying my fly.toml file to include the [http_service] section. This is what was breaking my deployments.
It may be a good idea for this action to check for the existence of a fly.toml file in the repo. If it does not exist, then run fly launch to deploy the app, otherwise run fly apps create so that these unexpected modifications do not occur.

there is a protection to keep fly.toml unchanged on this github action tho  https://github.com/superfly/fly-pr-review-apps/blob/280ea619ee64c8fffd66cca5b846b628c1d81e6d/entrypoint.sh#L42


python3Packages.rapidfuzz: 1.9.1 -> 2.0.8

TODO

[ ] fix scikit-build tests
[ ] package Cython 3.0.0a10

I don't have the time to care of these right now, feel free to push to my branch or open your own PR.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

cc @yuuyins

cherry-picked into python-updates with little fixes


Fix Variant to string constructor ambiguity in macos/ios builds

When sending those unsigned long to the stringify function of godot as Variant it will lead to compile error stating an ambiguity as it can't defined which constructor of Variant it should use, happens on macos/ios builds, casting it as uint64_t fixes that and ensure theres always enough space for the value being casted.

Dear @HKunogi , I apologize for very long delay in development


Response on title change

Added a response to the !title command to make it consistent with other misc commands.
Also fixed a minor error in an unrelated comment.

Sweet, much obliged!


Error when attaching mixed-mode debugger

This appears every time I try to attach in mixed debugging mode to an application with an embedded interpreter:

Process: modo.exe
The requested attach was not completely successful.
Successfully attached these type(s) of code:
Native
Failed to attach to these type(s) of code:
Python: An operation is not legal in the current state.

This happens with Blender (Python 3 x64) as well as Modo (Python 2 x64).
Everything seems to work correctly despite the error.

This does not happen if you only select either Native or Python debugging, but only when you select to debug both. (I.e. the debugging steps in https://msdn.microsoft.com/en-us/library/8dbb3we5(v=vs.85).aspx do not give any more information because it works correctly when only one is specified).

I'm facing same issue. Anyone found a resolution for this?
I'm doing MMD for C++ & CPython 2.7.8

@zooba can we get some traction here? this one is hurting us the most. Let me know if any trace is needed.

I believe you need .pdb debug symbols for CPython in order to be able to do
mixed-mode cross-language debugging. Not sure if blender or modo provide
this, if not, then recompiling with debug mode is necessary.
On Friday, March 4, 2016, Tamer<EMAIL_ADDRESS>wrote:

@zooba https://github.com/zooba can we get some traction here? this one
is hurting us the most. Let me know if any trace is needed.
—
Reply to this email directly or view it on GitHub
https://github.com/Microsoft/PTVS/issues/481#issuecomment-192542304.

Hi @denfromufa,
PDBs has to match (generated from the same build of) the python build you use. I have recompiled CPython 2.7.11 release to generate its PDBs and added the symbols to VS debug settings.
If symbols do not match or not provided, VS (in mixed mode) will show a dialog requesting to update the symbols paths. That isn't what we are facing here.

@tafify, is this issue actually preventing you from debugging? I found that debugging worked normally in spite of the error message.

@Artfunkel When you say "normally", could you set breakpoints in Python code? It looks like that should not have been possible from the error message.
@tafify Thanks for the reminder. More traces would be very valuable, as we have a lot of bugs that we cannot easily reproduce, and we simply don't have the time or manpower to dedicate to trying to recreate each one. The more information we can collect in advance, the easier it will be to narrow it down to the underlying issue.

@zooba I have here logs from "Devenv /log" with my dxdiag log:
vs_logs.zip
Not sure what other logs I can provide. Can I attach another debugger to debug the "attach to process" of PTVS? if so, can you wiki me how to build PTVS and install it in VS2015? maybe I can provide you with some stack trace.

@tafify We have build instructions on the wiki already. You really only need to do a few prerequisites and then build and run the solution.
It seems like an InvalidOperationException is being raised somewhere, so if you break on those then you'll likely hit the right place when reproing the issue.

@zooba I've hit this problem again.
Breakpoints are hit, but when I hit them a window appears with the title "foo.py not found". There's a "browse and find" option, but clicking it doesn't do anything. However, the call stack and locals window are both correct. They even show the line number.
I don't recall the "file not found" problem from last time. That has probably been introduced since I opened this bug.
I'm using VS2015 and PTVS 2.2.3. I'm debugging Python 2 embedded in an x64 program.

@Artfunkel What happens if you just open the file normally (Ctrl+O)? Do we show the breakpoint then?
I don't recall what the logic is here for mapping filenames, but maybe @int19h knows?

No, the breakpoint doesn't appear. If I create one in the file it immediately disappears.

This is where mapping is handled - pretty similar to what we do for regular debugging:
https://github.com/Microsoft/PTVS/blob/master/Python/Product/Debugger/DkmDebugger/ModuleManager.cs#L30

@Artfunkel, can you see foo.py in the Modules window, and what path does it have there (if any)?

I've discovered that the file not appearing was down to some script security we had in place. To answer your question, the module does appear and the path is the one I quoted above. Presumably it's the forward slash that causes problems.
The error on mixed mode attach persists, but doesn't apparently cause any problems. It just makes newbies think everything is broken. :)

I'm 99% sure we fixed the InvalidOperationException in another issue, so I'm closing this for cleanliness reasons. If it occurs again, we can reopen or start a new bug.


Missing dependencies in package.json

when i run this command
npx create-react-app reactreact
i got error always like this "Missing dependencies in package.json"

Run npm init to download package.json file

not working


Irr System Configuration Position & PositionDataPedigree redundant to Location entity

We should likely use Location here instead.

Agreed in review with @aferreyra Oct 18, 2022.

Changes made in Score.

Closing old Irrigation issues.  Irrigation was removed from Scope of ADAPT Standard 1.0.   We will work through a revised implementation indepedently of what was in the ADAPT Framework.


API for expressing the importing with aka .innerWith in a Cypher.Call does not work for UNION subqueries.

After reading https://neo4j.com/developer/kb/post-union-processing I noticed the current API to express the Importing with in Cypher.Call does not fit well when the inside clause is a Cypher.Union.
For instance, to achieve the below:
MATCH (m:Movie {title:'The Matrix'})
CALL {
    WITH m
    MATCH (m)<-[:ACTED_IN]-(p)
    RETURN p
    UNION
    WITH m
    MATCH (m)<-[:DIRECTED]-(p)
    RETURN p
}
RETURN p.name as name

There is no option to express the WITH m that doesn't require to modify every Union clause.

Depending on the API to do this, this may be a breaking change

So, a proposal would be to add a WITH to every union subquery when the innerWith is set in the parent Call

So, a proposal would be to add a WITH to every union subquery when the innerWith is set in the parent Call

That's my first thought as well, could be worth considering it for the next major release!


Always treat paths relative to vault root

Fixes #147 (again). Simplifies the logic from #148.

On links with a display bar [[path/file|display]], it throws an error, but still creates the file properly. Other than that it works great.

Good catch! Fixed: 329098f

We're lacking on unit tests here, but I've manually gone through all of the edge cases I can think of and it seems to be working as expected. I'm going to merge now b/c this is an important fix, but let me know if you find anymore issues. ✌


Exif image rotation for ZoomableAsyncImage w/ coil

When swapping out AsyncImage with ZoomableAsyncImage exif rotation seems to be ignored. AsyncImage shows my image in the correct orientation, while ZoomableAsyncImage does not.
I saw https://github.com/saket/telephoto/issues/30 marked as fixed so this might have regressed (or never worked for Coil)?

Will you be able to share a sample image? I've got a few here but it's possible I'm missing something.
Exif rotation is applied by the bitmap decoder so it should work for all image libraries out of the box

Ah OK, that was added after 1.0.0-alpha02 though which I was using. With the 0.6.0-SNAPSHOT the rotation is correct.

Yea sorry about that. My decision to release a 1.x.x-alpha and then go back to using 0.x was a mistake.

No worries, it's a great library so thanks for that!


Make IntoAttribute more flexible for Options

This change should both remove repetition for implementing IntoAttribute for Options and help futureproof things by making sure any option of an intoattribute will always also be intoattribute.

I ran all the commands listed on the CONTRIBUTING page. Working on this has also inspired an issue for me to work on (#2131).

Could you clarify why ignore-ing unrelated tests is necessary here? If you rebase and revert those ignores in Axum, do the tests pass?

@gbj I removed the ignores and now everything works 🤷

Okay, thanks! Looks good.


ubx【RTK】:  Add event info for timestamp.

When we perform 3D modeling of surveying tasks, in order to prevent the delay of the serial port, we need to get the precise time of taking photos.
This is from ubx data sheet：

FYI: @lumeier

Is there any setup configuration required to use it? What is the setup you use?
We have a rising and falling edge timestamp. Can we add both?
There are some flags bits that are important as well. For instance we need to check it the time is valid, and whether we have utc time.
Then we should also add the newRisingEdge information to the message.

It seems no need,

Why, did you end up using a different solution?


Add more users and most are very famous

This full list came from the Survey about the End the supports of JDK 1.6. End users submit the company info in the Survey.

Codecov Report

Merging #4068 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4068      +/-   ##
==========================================
+ Coverage   26.82%   26.83%   +<.01%
==========================================
  Files        1134     1134
  Lines       24986    24986
  Branches     3607     3607
==========================================
+ Hits         6703     6705       +2
+ Misses      17686    17685       -1
+ Partials      597      596       -1

Impacted Files
Coverage Δ

...alking/apm/agent/core/commands/CommandService.java
40.42% <0%> (+4.25%)
: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 4e50132...292a560. Read the comment docs.


Failed to fetch anonymous token with DOCKER_BUILDKIT=1

For some reason docker:dind build started failing as of yesterday
ERROR: failed to solve: node:18.13-alpine: failed to authorize: failed to fetch anonymous token:
Get "https://auth.ipv6.docker.com/token?scope=repository%3Alibrary%2Fnode%3Apull&service=registry.docker.io":
dial tcp [2600:1f18:2148:bc00:e947:5a7f:a49f:9126]:443: connect: network is unreachable

This is the only related thing I see but that does nothing to resolve the issue.
docker network prune --force
Deleted Networks:
disable_ipv6

If I run the same docker build as DOCKER_BUILDKIT=0 docker build there is no such problem.
What could be the cause of this?

Closing. This is the wrong repository to open this issue in. Please ask in https://github.com/docker/hub-feedback


Cannot read properties of null (reading 'icon') added a state to the…

error :Cannot read properties of null (reading 'icon')
added a state to the cmpnt

Fixed incorrect loading of card icons.


Does it work for ReactJS?

Can I install the npm module and require it in my react code would it work?

This is meant to be a server-side module, it's not meant to work on the client-side, sorry.

Okay. Thanks for the prompt reply


Fix binder build

Seems like it's failing because cmake is not found. Adding to APT dependencies fixes the problem.
Example binder (working): https://mybinder.org/v2/gh/luizirber/evcxr/fix_binder?filepath=evcxr_jupyter%2Fsamples%2Fevcxr_jupyter_tour.ipynb

Thanks for the fix


add coverage exclusion for json converter

Coverage is at 97% now

hey @JasonBoggsAtHMSdotCom I dont think we should ignore the tests.
specially for this class, I will open another PT, so you can check the tests.
thanks

I chose to ignore this class since the converter is called by System.Text.Json and it's never called by RulesEngine directly

Closing in favor of https://github.com/asulwer/RulesEngine/pull/85


Controls do not correctly update while a DVR stream is paused

Controls do not correctly update while a DVR stream is paused, in particular the back to live button does not appear again when leaving live conditions.
Issue type
Incorrect behavior
Environment information

Library version: Tested with 4.0.0
iOS version: Any
Device: Any

Reproducibility
Always reproducible
Steps to reproduce

Open Letterbox demo and play Couleur 3.
Pause the stream and wait. The skip button never appears, even when the slider says we are not in live conditions anymore.

Fixed in commit a3a3d0b260edc5f47a13e58e4b7ad9aa93e0fcc4.


Adjust maximal amount of characters for push notfications / news

The maximal amount of characters for the push notifcations body should be adjusted to 500 (currently 250).
Beside the technical adjustment of the figure you might need to check how the app is displaying a longer push-notification, e.g. is it automatically using only the first 250 chars ands add "..." behind or do we have to cut it by ourselves?

Beside the technical adjustment of the figure you might need to check how the app is displaying a longer push-notification, e.g. is it automatically using only the first 250 chars ands add "..." behind or do we have to cut it by ourselves?

@f1sh1918
Is there anything that should be done in the backend for this point?

As far as I can see it should be fine.
Do you see any issues @steffenkleinle ?

I don't think so, no. I proposed a few times already that it might make sense to decouple the notification text (non-html) and the text that is displayed in the detail view in the app (with html text). Still makes sense imo, but thats another issue.


Add global resource examples to description about execptions in resource group update restrictions

Added "Azure DNS Private Zones" to the examples of the global resources that do not apply to the restrictions when resource groups's region is temporarily unavailable.
In the docs we can see that there is some differences between Azure DNS and Azure Private DNS.
Like in this docs.
Here, Private DNS Zone is specified as a zone-redundant service,

However, Azure DNS it self is stated as always-available service.

This could cause confusion to our cutomers, therefore, would like to explicitly add Azure Private DNS to the description.

Docs Build status updates of commit 76b9879:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/azure-resource-manager/management/overview.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@tfitzmac
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off
Thanks @nagakei05 for this update!


Cannot commission device after unparing (mDNS name conflict)

Using chip-tool I can pair my Aqara P2, when I unpair the Aqara P2, I cannot commission it again unless I restart the otbr-agent (ot-ctl br disable/enable). Restarting the agent results in mDNS entry for the unpaired device to removed.
When I pair the device these entries appear in avahi-browser logs
+ enp6s0 IPv4 6817CD2E11FB394B-0000000000000001             _matter._tcp         local
+ enp6s0 IPv6 6817CD2E11FB394B-0000000000000001             _matter._tcp         local
+ enp6s0 IPv6 91E5641FEF75C406                              _matterc._udp        local
+ enp6s0 IPv4 91E5641FEF75C406                              _matterc._udp        local

When I unpair, I only see this log
- enp6s0 IPv6 6817CD2E11FB394B-0000000000000001             _matter._tcp         local
- enp6s0 IPv4 6817CD2E11FB394B-0000000000000001             _matter._tcp         local

When I try to pair the device again, I see these lines in otbr-agent logs
Name conflict: service name 6817CD2E11FB394B-0000000000000001._matter._tcp.default.service.arpa. has already been allocated
Failed to process DNS Update section: Duplicated
Send fail

To Reproduce Information to reproduce the behavior, including:
1.790dc775144e33995cd1cb2c15b348849cacf737
2. IMX8MP
3. meta-matter from NXP
4. Its a basic test environment where there is only one wifi router, a Hub and Aqara P2
Expected behavior
When the device is unpaired the mDNS name should be free-ed so that the device could be claimed again. Right now I have to restart the otbr-agent

When the device is commissioned this is what I see
root@pseudo-Hub-d8-47-8f-21-75-ef:~# ot-ctl srp server service
6817CD2E11FB394B-0000000000000001._matter._tcp.default.service.arpa.
    deleted: false
    subtypes: _I6817CD2E11FB394B
    port: 5540
    priority: 0
    weight: 0
    ttl: 7200
    lease: 7200
    key-lease: 680400
    TXT: [SII=35333030, SAI=353530, T=30]
    host: 6AC28D105035D3AA.default.service.arpa.
    addresses: [fd63:bc10:46f1:1:ed42:29b4:b4b5:591]
CBCB398AC5666307._matterc._udp.default.service.arpa.
    deleted: false
    subtypes: _V4447,_T65535,_S6,_L1561
    port: 5540
    priority: 0
    weight: 0
    ttl: 7200
    lease: 7200
    key-lease: 680400
    TXT: [VP=343434372b38313934, DT=3635353335, SII=35333030, SAI=353530, T=30, D=31353631, CM=30, RI=303130303942343233453538344632303939334646363942323431353241364531323833, PH=3336, PI=]
    host: 6AC28D105035D3AA.default.service.arpa.
    addresses: [fd63:bc10:46f1:1:ed42:29b4:b4b5:591]
Done

AND when I unpair the device
root@psuedo-Hub-d8-47-8f-21-75-ef:~# ot-ctl srp server service
6817CD2E11FB394B-0000000000000001._matter._tcp.default.service.arpa.
    deleted: true
CBCB398AC5666307._matterc._udp.default.service.arpa.
    deleted: false
    subtypes: _V4447,_T65535,_S6,_L1561
    port: 5540
    priority: 0
    weight: 0
    ttl: 7200
    lease: 7200
    key-lease: 680400
    TXT: [VP=343434372b38313934, DT=3635353335, SII=35333030, SAI=353530, T=30, D=31353631, CM=30, RI=303130303942343233453538344632303939334646363942323431353241364531323833, PH=3333, PI=]
    host: 6AC28D105035D3AA.default.service.arpa.
    addresses: [fd63:bc10:46f1:1:ed42:29b4:b4b5:591]
Done

I have discovered that instead of restarting the otbr agent, I am able to get things back to normal by just enable/disable the srp service.
ot-ctl srp server disable
ot-ctl srp server enable

It appears your Matter device is always using the same Node ID<PHONE_NUMBER>000001. Matter requires the device to have a new Node ID every time it is factory reset.
Using the same node ID across pairings will cause name conflicts, which is what you are experiencing.

Thank you, that was it.


Avro

@martindurant is it possible for with hdfs.open(...) to finish closing before the file is globally accessible throughout the network?

I'm getting failed tests with missing files.

OK, things seem ok-ish.  I've rebased down some of the smaller commits.  Merging on pass.


nuxt: File is not defined

Reproduction
https://stackblitz.com/edit/github-vqvsuh-xhe2f3?devtoolsheight=33&file=app.vue
Describe the bug
When creating a validation with an array, I'm getting [worker] File is not defined.
Please note that while stackblitz still shows the result, on my end it doesn't load at all.
Environment

Formkit: 1.0.0-beta.4 (also was able to repro on beta.5)
Nuxt: 3.0.0-27428764.271289c (also was able to repro on latest)
Platform: Mac

I meant to put in here, a workaround to put up the top of your formkit config:
// ! TEMP FIX UNTIL BETA 5. SEE FORMKIT ISSUE 108
if (typeof global !== 'undefined') {
    Object.assign(global, { File: function () {} });
}

Fixed in @formkit/nuxt@next and will be released in beta.5


Refactor symbolics

This allows us to add other symbolics than SymToken easily.

Really nice use case for barbie!

Right? Although! I'm not yet convinced that the barbie library is the right implementation of this pattern yet. The Const functor stuff is all a bit clunky because there are some missing instances that don't really have any right to be missing....

Really nice use case for barbie!

Right? Although! I'm not yet convinced that the barbie library is the right implementation of this pattern yet. The Const functor stuff is all a bit clunky because there are some missing instances that don't really have any right to be missing....

Is this just an omission from the barbie library? Or are these instances impossible to implement?

I think they are impossible given the implementation of barbie. But I'm
also not sure how to do it better.
Den tors 20 apr. 2023 12:30Sjoerd Visscher @.***> skrev:

Really nice use case for barbie!
Right? Although! I'm not yet convinced that the barbie library is the
right implementation of this pattern yet. The Const functor stuff is all
a bit clunky because there are some missing instances that don't really
have any right to be missing....
Is this just an omission from the barbie library? Or are these instances
impossible to implement?
—
Reply to this email directly, view it on GitHub
https://github.com/input-output-hk/quickcheck-contractmodel/pull/14#issuecomment-1516094093,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBPPQJTUVG3MUGL5ZJQM3DXCEF4ZANCNFSM6AAAAAAXD6WWW4
.
You are receiving this because you modified the open/close state.Message
ID: @.***
com>


Using a BTC/OTC in the training Zipformer instead of Conformer.

Is there any script available to train the latest good model Zipformer model using Bypass Temporal Classification(BTC)/Omni-temporal Classification (OTC) (https://github.com/k2-fsa/icefall/tree/master/egs/librispeech/WSASR) to align speech with text instead of CTC (https://github.com/k2-fsa/icefall/tree/master/egs/librispeech/ASR/zipformer_ctc) ?

sorry, this is not planned so far.
Best Regards
Jin
On Thu, 11 Apr 2024 at 18:31 Kerolos ghobrial @.***>
wrote:

Is there any script available to train the latest good model Zipformer
model using Bypass Temporal Classification(BTC)/Omni-temporal
Classification (OTC) (
https://github.com/k2-fsa/icefall/tree/master/egs/librispeech/WSASR) to
align speech with text instead of CTC (
https://github.com/k2-fsa/icefall/tree/master/egs/librispeech/ASR/zipformer_ctc)
?
—
Reply to this email directly, view it on GitHub
https://github.com/k2-fsa/icefall/issues/1589, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOON42F76HJTCQFPNRJQD33Y4ZQ7BAVCNFSM6AAAAABGCBFVYKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGIZTONBQHA3TEMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Thanks, Dr. Daniel Povey, JinZr,
Hello  @DongjiGao,
1- Is there any script to create lang OTC based on Phone lexicon instead of BPE (k2-fsa/icefall/tree/master/egs/librispeech/WSASR/local/prepare_otc_lang_bpe.py), since the paper showed a better performance in the Phone Based Lexicon over (Bypass Temporal Classification)?
2- Could this Method be used to clean up the errors present in the training transcripts, something similar to original Kaldi  (/egs/wsj/s5/steps/cleanup/clean_and_segment_data.sh or clean_and_segment_data_nnet3.sh  ) ?
3- Is there any plan to extend that to be used in ONNX (decoder with int8) ?
Thanks in advance,
Kerolos

Thanks, Dr. Daniel Povey, JinZr, Hello @DongjiGao,
1- Is there any script to create lang OTC based on Phone lexicon instead of BPE (k2-fsa/icefall/tree/master/egs/librispeech/WSASR/local/prepare_otc_lang_bpe.py), since the paper showed a better performance in the Phone Based Lexicon over (Bypass Temporal Classification)? 2- Could this Method be used to clean up the errors present in the training transcripts, something similar to original Kaldi (/egs/wsj/s5/steps/cleanup/clean_and_segment_data.sh or clean_and_segment_data_nnet3.sh ) ? 3- Is there any plan to extend that to be used in ONNX (decoder with int8) ?
Thanks in advance, Kerolos

Thank you for your interest.

I can check the phone-based script if needed. We modified how we model the star token in OTC (as the average probability of all non-blank tokens instead of an individual token), so I would guess the performance would be similar to using BPE.
Yes, please refer to this script for more details. It does "flexible alignment" by replacing suspicious tokens or wrongly inserted tokens with a star, and by placing a star where a word is missing.
We do not have such a plan in the near term. Do you have any suggestions?

Dongji

Thanks for the quick response, Dongji!
1- It would be great if you could check the file in prepare_otc_lang.py (using the phone based lexicon ) and any related files that required to complete the training. I would really appreciate that.
2-It seems quite helpful. I'll definitely give it a try.
3- Perhaps my focusing on utilizing Zipformer could be a primary consideration, and then exporting it to ONNX.
BR, Kerolos

Sorry @DongjiGao for bothering you again:
1- I would like to use Phone Based lexicon script in OTC, and then compare the cleaning from this method VS original Kaldi method (clean_and_segment_data_nnet3.sh), which i had used phone based.
Thanks in advance, Dongji Gao.

Sorry @DongjiGao for bothering you again: 1- I would like to use Phone Based lexicon script in OTC, and then compare the cleaning from this method VS original Kaldi method (clean_and_segment_data_nnet3.sh), which i had used phone based. Thanks in advance, Dongji Gao.

I will submit a PR by the end of this week.

Thanks @DongjiGao for update.
I have gotten this error in otc_phone_graph_compiler.py :
(encoder_output_layer): Sequential(
(0): Dropout(p=0.2, inplace=False)
(1): ScaledLinear(in_features=512, out_features=72, bias=True)
)
)
2024-05-07 17:16:56,774 INFO [train_phone.py:936] Number of model parameters: 100857162
2024-05-07 17:16:56,988 INFO [asr_datamodule.py:325] About to get train-clean-100 cuts
2024-05-07 17:16:56,989 INFO [asr_datamodule.py:216] About to create train dataset
2024-05-07 17:16:56,989 INFO [asr_datamodule.py:224] Using DynamicBucketingSampler.
2024-05-07 17:16:57,471 INFO [asr_datamodule.py:241] About to create train dataloader
2024-05-07 17:16:57,472 INFO [asr_datamodule.py:347] About to get dev-clean cuts
2024-05-07 17:16:57,472 INFO [asr_datamodule.py:354] About to get dev-other cuts
2024-05-07 17:16:57,473 INFO [asr_datamodule.py:272] About to create dev dataset
2024-05-07 17:16:57,687 INFO [asr_datamodule.py:285] About to create dev dataloader
Traceback (most recent call last):
File "./conformer_ctc2/train_phone.py", line 1128, in
main()
File "./conformer_ctc2/train_phone.py", line 1121, in main
run(rank=0, world_size=1, args=args)
File "./conformer_ctc2/train_phone.py", line 1029, in run
train_one_epoch(
File "./conformer_ctc2/train_phone.py", line 766, in train_one_epoch
loss, loss_info = compute_loss(
File "./conformer_ctc2/train_phone.py", line 604, in compute_loss
decoding_graph = graph_compiler.compile(
File "icefall_kaldi/icefall/icefall/otc_phone_graph_compiler.py", line 140, in compile
fsa_with_self_loop = k2.remove_epsilon_and_add_self_loops(transcript_fsa)
File "miniconda3/envs/icefall-run/lib/python3.8/site-packages/k2/fsa_algo.py", line 765, in remove_epsilon_and_add_self_loops
if fsa.properties & fsa_properties.EPSILON_FREE != 0:
AttributeError: 'NoneType' object has no attribute 'properties'

Hello @DongjiGao ,
I got a large WER (1best decoding) when I used phone based lexicon, feature_type=ssl and FP16 :
Moreover, in the results it seems it is escaping or swallowing some words when decoding, for example:
a) 1688-142285-0006-2603:	ref=['I', "DON'T", 'THINK', 'MISTER', 'HALE', 'YOU', 'HAVE', 'DONE', 'QUITE', 'RIGHT', 'IN', 'INTRODUCING', 'SUCH', 'A', 'PERSON', 'TO', 'US', 'WITHOUT', 'TELLING', 'US', 'WHAT', 'HE', 'HAD', 'BEEN']
1688-142285-0006-2603:	hyp=['I', "DON'T", 'THINK', 'YOU', 'HAVE', 'DONE', 'WHAT', 'HE', 'HAD', 'BEEN']
b) 1688-142285-0008-2605:	ref=['HIS', 'FATHER', 'DYING', 'IN', 'MISERABLE', 'CIRCUMSTANCES']
1688-142285-0008-2605:	hyp=['HIS', 'MISERABLE', 'CIRCUMSTANCES']
Phone results:
[decode_phone.py:473] {'subsampling_factor': 4, 'feature_dim': 768, 'nhead': 8, 'dim_feedforward': 2048, 'encoder_dim': 512, 'num_encoder_layers': 12, 'search_beam': 20, 'output_beam': 8, 'min_active_states': 30, 'max_active_states': 10000, 'use_double_scores': True, 'env_info': {'k2-version': '1.24.3', 'k2-build-type': 'Release', 'k2-with-cuda': True, 'k2-git-sha1': '4c05309499a08454997adf500b56dcc629e35ae5', 'k2-git-date': 'Tue Jul 25 16:23:36 2023', 'lhotse-version': '1.24.0.dev+git.4f014b1.clean', 'torch-version': '1.13.0+cu116', 'torch-cuda-available': True, 'torch-cuda-version': '11.6', 'python-version': '3.8', 'icefall-git-branch': 'first_run', 'icefall-git-sha1': 'c45e9fec-dirty', 'icefall-git-date': 'Wed Apr 3 05:26:24 2024', 'icefall-path': '/mnt/srv/data/train_am/analysisTD/icefall_kaldi/icefall', 'k2-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/k2/init.py', 'lhotse-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/lhotse/init.py', 'hostname': 'Hyrican-3', 'IP address': '<IP_ADDRESS>'}, 'otc_token': '', 'blank_bias': -4.0, 'epoch': 20, 'iter': 0, 'avg': 5, 'method': '1best', 'use_averaged_model': False, 'num_decoder_layers': 0, 'exp_dir': PosixPath('conformer_ctc2/exp_phone'), 'lang_dir': PosixPath('data/lang_phone'), 'lm_dir': PosixPath('data/lm/G_3_gram.fst.txt'), 'full_libri': False, 'mini_libri': False, 'manifest_dir': PosixPath('data/ssl'), 'max_duration': 200.0, 'bucketing_sampler': True, 'num_buckets': 30, 'concatenate_cuts': False, 'duration_factor': 1.0, 'gap': 1.0, 'shuffle': True, 'drop_last': True, 'return_cuts': True, 'num_workers': 2, 'input_strategy': 'PrecomputedFeatures', 'train_manifest': 'librispeech_cuts_train-clean-100.jsonl.gz'}
4-05-10 13:32:00,131 INFO [decode_phone.py:410] batch 0/?, cuts processed until now is 14
2024-05-10 13:32:16,706 INFO [decode_phone.py:410] batch 100/?, cuts processed until now is 2224
2024-05-10 13:32:18,942 INFO [decode_phone.py:430] The transcripts are stored in conformer_ctc2/exp_phone/recogs-test-clean-no_rescore.txt
2024-05-10 13:32:18,994 INFO [utils.py:656] [test-clean-no_rescore] %WER 74.22% [39021 / 52576, 6 ins, 37768 del, 1247 sub ]
2024-05-10 13:32:19,130 INFO [decode_phone.py:442] Wrote detailed error stats to conformer_ctc2/exp_phone/errs-test-clean-no_rescore.txt
2024-05-10 13:32:19,133 INFO [decode_phone.py:456]
For test-clean, WER of different settings are:
no_rescore      74.22   best for test-clean
2024-05-10 13:32:19,673 INFO [decode_phone.py:410] batch 0/?, cuts processed until now is 18
2024-05-10 13:32:36,774 INFO [decode_phone.py:410] batch 100/?, cuts processed until now is 2612
2024-05-10 13:32:38,820 INFO [decode_phone.py:430] The transcripts are stored in conformer_ctc2/exp_phone/recogs-test-other-no_rescore.txt
2024-05-10 13:32:38,874 INFO [utils.py:656] [test-other-no_rescore] %WER 79.77% [41756 / 52343, 8 ins, 40086 del, 1662 sub ]
2024-05-10 13:32:39,019 INFO [decode_phone.py:442] Wrote detailed error stats to conformer_ctc2/exp_phone/errs-test-other-no_rescore.txt
2024-05-10 13:32:39,024 INFO [decode_phone.py:456]
For test-other, WER of different settings are:
no_rescore      79.77   best for test-other
BPE results:
{'subsampling_factor': 2, 'feature_dim': 768, 'nhead': 8, 'dim_feedforward': 2048, 'encoder_dim': 512, 'num_encoder_layers': 12, 'search_beam': 20, 'output_beam': 8, 'min_active_states': 30, 'max_active_states': 10000, 'use_double_scores': True, 'env_info': {'k2-version': '1.24.3', 'k2-build-type': 'Release', 'k2-with-cuda': True, 'k2-git-sha1': '4c05309499a08454997adf500b56dcc629e35ae5', 'k2-git-date': 'Tue Jul 25 16:23:36 2023', 'lhotse-version': '1.23.0.dev+git.1c2a1b5.clean', 'torch-version': '1.13.0+cu116', 'torch-cuda-available': True, 'torch-cuda-version': '11.6', 'python-version': '3.8', 'icefall-git-branch': 'first_run', 'icefall-git-sha1': 'c45e9fec-dirty', 'icefall-git-date': 'Wed Apr 3 05:26:24 2024', 'icefall-path': '/mnt/srv/data/train_am/analysisTD/icefall_kaldi/icefall', 'k2-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/k2/init.py', 'lhotse-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/lhotse/init.py', 'hostname': 'hyrican-1', 'IP address': '<IP_ADDRESS>'}, 'otc_token': '▁', 'blank_bias': -4.0, 'epoch': 20, 'iter': 0, 'avg': 1, 'method': '1best', 'use_averaged_model': False, 'num_decoder_layers': 0, 'exp_dir': PosixPath('conformer_ctc2/exp'), 'lang_dir': PosixPath('data/lang_bpe_200'), 'lm_dir': PosixPath('data/lm/G_3_gram.fst.txt'), 'full_libri': False, 'mini_libri': False, 'manifest_dir': PosixPath('data/ssl'), 'max_duration': 200.0, 'bucketing_sampler': True, 'num_buckets': 30, 'concatenate_cuts': False, 'duration_factor': 1.0, 'gap': 1.0, 'shuffle': True, 'drop_last': True, 'return_cuts': True, 'num_workers': 2, 'input_strategy': 'PrecomputedFeatures', 'train_manifest': 'librispeech_cuts_train-clean-100.jsonl.gz'}
2024-04-14 14:21:47,136 INFO [decode.py:476] batch 0/?, cuts processed until now is 14
2024-04-14 14:22:18,915 INFO [decode.py:476] batch 100/?, cuts processed until now is 2224
2024-04-14 14:22:23,112 INFO [decode.py:496] The transcripts are stored in conformer_ctc2/exp/recogs-test-clean-no_rescore.txt
2024-04-14 14:22:23,178 INFO [utils.py:656] [test-clean-no_rescore] %WER 7.30% [3838 / 52576, 370 ins, 599 del, 2869 sub ]
2024-04-14 14:22:23,321 INFO [decode.py:508] Wrote detailed error stats to conformer_ctc2/exp/errs-test-clean-no_rescore.txt
2024-04-14 14:22:23,325 INFO [decode.py:522]
For test-clean, WER of different settings are:
no_rescore	7.3	best for test-clean
2024-04-14 14:22:23,962 INFO [decode.py:476] batch 0/?, cuts processed until now is 18
2024-04-14 14:22:57,948 INFO [decode.py:476] batch 100/?, cuts processed until now is 2612
2024-04-14 14:23:02,000 INFO [decode.py:496] The transcripts are stored in conformer_ctc2/exp/recogs-test-other-no_rescore.txt
2024-04-14 14:23:02,063 INFO [utils.py:656] [test-other-no_rescore] %WER 17.93% [9385 / 52343, 741 ins, 1813 del, 6831 sub ]
2024-04-14 14:23:02,200 INFO [decode.py:508] Wrote detailed error stats to conformer_ctc2/exp/errs-test-other-no_rescore.txt
2024-04-14 14:23:02,207 INFO [decode.py:522]
For test-other, WER of different settings are:
no_rescore	17.93	best for test-other

Is there anything in the parameters training or decoding shall I change to obtain a close result as BPE ?

Hello @DongjiGao , I got a large WER (1best decoding) when I used phone based lexicon, feature_type=ssl and FP16 : Moreover, in the results it seems that; it is escaping or swallowing some words when decoding, for example: a) 1688-142285-0006-2603: ref=['I', "DON'T", 'THINK', 'MISTER', 'HALE', 'YOU', 'HAVE', 'DONE', 'QUITE', 'RIGHT', 'IN', 'INTRODUCING', 'SUCH', 'A', 'PERSON', 'TO', 'US', 'WITHOUT', 'TELLING', 'US', 'WHAT', 'HE', 'HAD', 'BEEN'] 1688-142285-0006-2603: hyp=['I', "DON'T", 'THINK', 'YOU', 'HAVE', 'DONE', 'WHAT', 'HE', 'HAD', 'BEEN'] b) 1688-142285-0008-2605: ref=['HIS', 'FATHER', 'DYING', 'IN', 'MISERABLE', 'CIRCUMSTANCES'] 1688-142285-0008-2605: hyp=['HIS', 'MISERABLE', 'CIRCUMSTANCES']
Phone results: [decode_phone.py:473] {'subsampling_factor': 4, 'feature_dim': 768, 'nhead': 8, 'dim_feedforward': 2048, 'encoder_dim': 512, 'num_encoder_layers': 12, 'search_beam': 20, 'output_beam': 8, 'min_active_states': 30, 'max_active_states': 10000, 'use_double_scores': True, 'env_info': {'k2-version': '1.24.3', 'k2-build-type': 'Release', 'k2-with-cuda': True, 'k2-git-sha1': '4c05309499a08454997adf500b56dcc629e35ae5', 'k2-git-date': 'Tue Jul 25 16:23:36 2023', 'lhotse-version': '1.24.0.dev+git.4f014b1.clean', 'torch-version': '1.13.0+cu116', 'torch-cuda-available': True, 'torch-cuda-version': '11.6', 'python-version': '3.8', 'icefall-git-branch': 'first_run', 'icefall-git-sha1': 'c45e9fec-dirty', 'icefall-git-date': 'Wed Apr 3 05:26:24 2024', 'icefall-path': '/mnt/srv/data/train_am/analysisTD/icefall_kaldi/icefall', 'k2-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/k2/init.py', 'lhotse-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/lhotse/init.py', 'hostname': 'Hyrican-3', 'IP address': '<IP_ADDRESS>'}, 'otc_token': '', 'blank_bias': -4.0, 'epoch': 20, 'iter': 0, 'avg': 5, 'method': '1best', 'use_averaged_model': False, 'num_decoder_layers': 0, 'exp_dir': PosixPath('conformer_ctc2/exp_phone'), 'lang_dir': PosixPath('data/lang_phone'), 'lm_dir': PosixPath('data/lm/G_3_gram.fst.txt'), 'full_libri': False, 'mini_libri': False, 'manifest_dir': PosixPath('data/ssl'), 'max_duration': 200.0, 'bucketing_sampler': True, 'num_buckets': 30, 'concatenate_cuts': False, 'duration_factor': 1.0, 'gap': 1.0, 'shuffle': True, 'drop_last': True, 'return_cuts': True, 'num_workers': 2, 'input_strategy': 'PrecomputedFeatures', 'train_manifest': 'librispeech_cuts_train-clean-100.jsonl.gz'}
4-05-10 13:32:00,131 INFO [decode_phone.py:410] batch 0/?, cuts processed until now is 14 2024-05-10 13:32:16,706 INFO [decode_phone.py:410] batch 100/?, cuts processed until now is 2224 2024-05-10 13:32:18,942 INFO [decode_phone.py:430] The transcripts are stored in conformer_ctc2/exp_phone/recogs-test-clean-no_rescore.txt 2024-05-10 13:32:18,994 INFO [utils.py:656] [test-clean-no_rescore] %WER 74.22% [39021 / 52576, 6 ins, 37768 del, 1247 sub ] 2024-05-10 13:32:19,130 INFO [decode_phone.py:442] Wrote detailed error stats to conformer_ctc2/exp_phone/errs-test-clean-no_rescore.txt 2024-05-10 13:32:19,133 INFO [decode_phone.py:456] For test-clean, WER of different settings are: no_rescore 74.22 best for test-clean
2024-05-10 13:32:19,673 INFO [decode_phone.py:410] batch 0/?, cuts processed until now is 18 2024-05-10 13:32:36,774 INFO [decode_phone.py:410] batch 100/?, cuts processed until now is 2612 2024-05-10 13:32:38,820 INFO [decode_phone.py:430] The transcripts are stored in conformer_ctc2/exp_phone/recogs-test-other-no_rescore.txt 2024-05-10 13:32:38,874 INFO [utils.py:656] [test-other-no_rescore] %WER 79.77% [41756 / 52343, 8 ins, 40086 del, 1662 sub ] 2024-05-10 13:32:39,019 INFO [decode_phone.py:442] Wrote detailed error stats to conformer_ctc2/exp_phone/errs-test-other-no_rescore.txt 2024-05-10 13:32:39,024 INFO [decode_phone.py:456] For test-other, WER of different settings are: no_rescore 79.77 best for test-other
BPE results: {'subsampling_factor': 2, 'feature_dim': 768, 'nhead': 8, 'dim_feedforward': 2048, 'encoder_dim': 512, 'num_encoder_layers': 12, 'search_beam': 20, 'output_beam': 8, 'min_active_states': 30, 'max_active_states': 10000, 'use_double_scores': True, 'env_info': {'k2-version': '1.24.3', 'k2-build-type': 'Release', 'k2-with-cuda': True, 'k2-git-sha1': '4c05309499a08454997adf500b56dcc629e35ae5', 'k2-git-date': 'Tue Jul 25 16:23:36 2023', 'lhotse-version': '1.23.0.dev+git.1c2a1b5.clean', 'torch-version': '1.13.0+cu116', 'torch-cuda-available': True, 'torch-cuda-version': '11.6', 'python-version': '3.8', 'icefall-git-branch': 'first_run', 'icefall-git-sha1': 'c45e9fec-dirty', 'icefall-git-date': 'Wed Apr 3 05:26:24 2024', 'icefall-path': '/mnt/srv/data/train_am/analysisTD/icefall_kaldi/icefall', 'k2-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/k2/init.py', 'lhotse-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/lhotse/init.py', 'hostname': 'hyrican-1', 'IP address': '<IP_ADDRESS>'}, 'otc_token': '▁', 'blank_bias': -4.0, 'epoch': 20, 'iter': 0, 'avg': 1, 'method': '1best', 'use_averaged_model': False, 'num_decoder_layers': 0, 'exp_dir': PosixPath('conformer_ctc2/exp'), 'lang_dir': PosixPath('data/lang_bpe_200'), 'lm_dir': PosixPath('data/lm/G_3_gram.fst.txt'), 'full_libri': False, 'mini_libri': False, 'manifest_dir': PosixPath('data/ssl'), 'max_duration': 200.0, 'bucketing_sampler': True, 'num_buckets': 30, 'concatenate_cuts': False, 'duration_factor': 1.0, 'gap': 1.0, 'shuffle': True, 'drop_last': True, 'return_cuts': True, 'num_workers': 2, 'input_strategy': 'PrecomputedFeatures', 'train_manifest': 'librispeech_cuts_train-clean-100.jsonl.gz'}
2024-04-14 14:21:47,136 INFO [decode.py:476] batch 0/?, cuts processed until now is 14 2024-04-14 14:22:18,915 INFO [decode.py:476] batch 100/?, cuts processed until now is 2224 2024-04-14 14:22:23,112 INFO [decode.py:496] The transcripts are stored in conformer_ctc2/exp/recogs-test-clean-no_rescore.txt 2024-04-14 14:22:23,178 INFO [utils.py:656] [test-clean-no_rescore] %WER 7.30% [3838 / 52576, 370 ins, 599 del, 2869 sub ] 2024-04-14 14:22:23,321 INFO [decode.py:508] Wrote detailed error stats to conformer_ctc2/exp/errs-test-clean-no_rescore.txt 2024-04-14 14:22:23,325 INFO [decode.py:522] For test-clean, WER of different settings are: no_rescore 7.3 best for test-clean
2024-04-14 14:22:23,962 INFO [decode.py:476] batch 0/?, cuts processed until now is 18 2024-04-14 14:22:57,948 INFO [decode.py:476] batch 100/?, cuts processed until now is 2612 2024-04-14 14:23:02,000 INFO [decode.py:496] The transcripts are stored in conformer_ctc2/exp/recogs-test-other-no_rescore.txt 2024-04-14 14:23:02,063 INFO [utils.py:656] [test-other-no_rescore] %WER 17.93% [9385 / 52343, 741 ins, 1813 del, 6831 sub ] 2024-04-14 14:23:02,200 INFO [decode.py:508] Wrote detailed error stats to conformer_ctc2/exp/errs-test-other-no_rescore.txt 2024-04-14 14:23:02,207 INFO [decode.py:522] For test-other, WER of different settings are: no_rescore 17.93 best for test-other

Is there anything in the parameters training or decoding shall I change to obtain a close result as BPE ?
have you faced a similar situation in the phone based lexicon, in which the system swallows some words ?

Please use subsampling_factor = 2 for SSL features.

Thanks @DongjiGao for your support, and sorry for bothering you again.
I have changed the subsampling_factor = 2 for SSL features in training and decoding and total loss and otc became very close to bpe (loss[otc_loss], tot_loss[otc_loss]). However, The WER still very high it drops from 74.22 to 65.83% [34612 / 52576, 18 ins, 32137 del, 2457 sub ] for test-clean, and from 79.77 to 74.65% [39075 / 52343, 27 ins, 35505 del, 3543 sub ] for test-other.
The deletion is still very high.
-The parameters used for decoding based phone lexicon:
{'subsampling_factor': 2, 'feature_dim': 768, 'nhead': 8, 'dim_feedforward': 2048, 'encoder_dim': 512, 'num_encoder_layers': 12, 'search_beam': 20, 'output_beam': 8, 'min_active_states': 30, 'max_active_states': 10000, 'use_double_scores': True, 'env_info': {'k2-version': '1.24.3', 'k2-build-type': 'Release', 'k2-with-cuda': True, 'k2-git-sha1': '4c05309499a08454997adf500b56dcc629e35ae5', 'k2-git-date': 'Tue Jul 25 16:23:36 2023', 'lhotse-version': '1.24.0.dev+git.4f014b1.clean', 'torch-version': '1.13.0+cu116', 'torch-cuda-available': True, 'torch-cuda-version': '11.6', 'python-version': '3.8', 'icefall-git-branch': 'first_run', 'icefall-git-sha1': 'c45e9fec-dirty', 'icefall-git-date': 'Wed Apr 3 05:26:24 2024', 'icefall-path': '/mnt/srv/data/train_am/analysisTD/icefall_kaldi/icefall', 'k2-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/k2/init.py', 'lhotse-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/lhotse/init.py', 'hostname': 'Hyrican-3', 'IP address': '<IP_ADDRESS>'}, 'otc_token': '', 'blank_bias': -4.0, 'epoch': 20, 'iter': 0, 'avg': 5, 'method': '1best', 'use_averaged_model': False, 'num_decoder_layers': 0, 'exp_dir': PosixPath('conformer_ctc2/exp_phone'), 'lang_dir': PosixPath('data/lang_phone'), 'lm_dir': PosixPath('data/lm/G_3_gram.fst.txt'), 'full_libri': False, 'mini_libri': False, 'manifest_dir': PosixPath('data/ssl'), 'max_duration': 200.0, 'bucketing_sampler': True, 'num_buckets': 30, 'concatenate_cuts': False, 'duration_factor': 1.0, 'gap': 1.0, 'shuffle': True, 'drop_last': True, 'return_cuts': True, 'num_workers': 2, 'input_strategy': 'PrecomputedFeatures', 'train_manifest': 'librispeech_cuts_train-clean-100.jsonl.gz'}
The training loss in Tensorboard (BPE based lexicon White -VS- Phone based lexicon Black):

@tobygodwin Can you share more details?

Thanks @DongjiGao for your support, and sorry for bothering you again. I have changed the subsampling_factor = 2 for SSL features in training and decoding. By changing that, the total loss and otc became very close to bpe (loss[otc_loss], tot_loss[otc_loss]). However, The WER still very high it drops from 74.22 to 65.83% [34612 / 52576, 18 ins, 32137 del, 2457 sub ] for test-clean, and from 79.77 to 74.65% [39075 / 52343, 27 ins, 35505 del, 3543 sub ] for test-other. The deletion is still very high.
-The parameters used for decoding based phone lexicon: {'subsampling_factor': 2, 'feature_dim': 768, 'nhead': 8, 'dim_feedforward': 2048, 'encoder_dim': 512, 'num_encoder_layers': 12, 'search_beam': 20, 'output_beam': 8, 'min_active_states': 30, 'max_active_states': 10000, 'use_double_scores': True, 'env_info': {'k2-version': '1.24.3', 'k2-build-type': 'Release', 'k2-with-cuda': True, 'k2-git-sha1': '4c05309499a08454997adf500b56dcc629e35ae5', 'k2-git-date': 'Tue Jul 25 16:23:36 2023', 'lhotse-version': '1.24.0.dev+git.4f014b1.clean', 'torch-version': '1.13.0+cu116', 'torch-cuda-available': True, 'torch-cuda-version': '11.6', 'python-version': '3.8', 'icefall-git-branch': 'first_run', 'icefall-git-sha1': 'c45e9fec-dirty', 'icefall-git-date': 'Wed Apr 3 05:26:24 2024', 'icefall-path': '/mnt/srv/data/train_am/analysisTD/icefall_kaldi/icefall', 'k2-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/k2/init.py', 'lhotse-path': '/home/ghk/miniconda3/envs/icefall-run/lib/python3.8/site-packages/lhotse/init.py', 'hostname': 'Hyrican-3', 'IP address': '<IP_ADDRESS>'}, 'otc_token': '', 'blank_bias': -4.0, 'epoch': 20, 'iter': 0, 'avg': 5, 'method': '1best', 'use_averaged_model': False, 'num_decoder_layers': 0, 'exp_dir': PosixPath('conformer_ctc2/exp_phone'), 'lang_dir': PosixPath('data/lang_phone'), 'lm_dir': PosixPath('data/lm/G_3_gram.fst.txt'), 'full_libri': False, 'mini_libri': False, 'manifest_dir': PosixPath('data/ssl'), 'max_duration': 200.0, 'bucketing_sampler': True, 'num_buckets': 30, 'concatenate_cuts': False, 'duration_factor': 1.0, 'gap': 1.0, 'shuffle': True, 'drop_last': True, 'return_cuts': True, 'num_workers': 2, 'input_strategy': 'PrecomputedFeatures', 'train_manifest': 'librispeech_cuts_train-clean-100.jsonl.gz'}
The training loss in Tensorboard (BPE based lexicon White -VS- Phone based lexicon Black):

Hi @kerolos,
Can you try different 'blank_bias' during decoding (e.g., -3, -2)? It looks like the current (-4) value is too small.
Dongji


[Google Workspace] Configure which browser to open Google Drive files in

Extension
https://www.raycast.com/raycast/google-workspace
Description
It would be very helpful if the extension could be configured to open Google Drive documents in another browser than your default browser (like Quicklinks)
Who will benefit from this feature?
People who use different browsers for different purposes (like Safari for personal browsing and Chrome for work).
Anything else?

Thanks for the feature request. Would you like to work on it? I can provide some guidance on how to contribute to this extension so that you can open a pull request. You can contact me here or in the Slack community if you need further help.
Note that opening up a PR and having it merged will earn you some credits as we want to reward contributions. You’ll then soon be able to exchange them in our swag store.

Someone from our team will take care of it 🙂


add support for "mentoring" bots

The whole vision and infrastructure of Repairnator can be used for creating mentoring bots: bots who give useful feedback to junior developers on how to program better.
This is related to the Sobo mode, ref https://github.com/eclipse/repairnator/pull/1260
This is also related to https://github.com/eclipse/repairnator/issues/1141
cc @sofi1410  @LinusOstlund

Related work by @bbaudry
https://github.com/Gaikanomer9/mentor-chatbot
https://github.com/apps/oss-mentor-bot
https://github.com/MentorMate/mentorbot

I configure the Sobo integration in a way where now there is a feedback option so that you can choose between Repair Tools or Feedback tools. Also, on the Pipeline for feedback tools, there is an option to use repair tools.
So Sobo is a feedback tool inside the feedback mode in Repairnator. Is there another tool you would like to integrate?
I'm reading #1141 , and an approach that comes to my mind is adding SORALD to the feedback mode. So, suppose the user wants to avoid receiving the patch as a PR. In that case, they can select using SORALD in feedback mode. Every time they have a violation, Sorald opens an issue in human language.
WDYT?

choose between Repair Tools or Feedback tools

That is so cool! So if we configure "repair" mode in Sobo, do students
directly get the repair?

Sorald opens an issue in human language.

How will that be different from Sobo?
On Thu, Nov 3, 2022 at 4:40 PM Sofia Bobadilla @.***>
wrote:

I configure the Sobo integration in a way where now there is a feedback
option so that you can choose between Repair Tools or Feedback tools. Also,
on the Pipeline for feedback tools, there is an option to use repair tools.
So Sobo is a feedback tool inside the feedback mode in Repairnator. Is
there another tool you would like to integrate?
I'm reading #1141 https://github.com/eclipse/repairnator/issues/1141 ,
and an approach that comes to my mind is adding SORALD to the feedback
mode. So, suppose the user wants to avoid receiving the patch as a PR. In
that case, they can select using SORALD in feedback mode. Every time they
have a violation, Sorald opens an issue in human language.
WDYT?
—
Reply to this email directly, view it on GitHub
https://github.com/eclipse/repairnator/issues/1261#issuecomment-1302301562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIMPTONZQB3FKGLNDZI3B6TWGPMIRANCNFSM6AAAAAAQNDRRIE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

No, but it should be possible without making significant architectural
changes.

Okay.

giving the solution is not a problem.

This makes sense. Otherwise, an explanation in natural language is already
provided by SonarQube. Presenting that to a developer by using Sorald won't
be novel, in my opinion. Providing a solution for it is, indeed, novel. :)
On Thu, Nov 3, 2022 at 6:18 PM Sofia Bobadilla @.***>
wrote:

That is so cool! So if we configure "repair" mode in Sobo, do students
directly get the repair?
No, but it should be possible without making significant architectural
changes.
How will that be different from Sobo?
SOBO is made for students. It presents a pedagogical explanation of
violations and gives nudging and informal messages. In the user case we are
discussing now, we are referring to developers who don't need a type of
feedback more precise, and in this scenario, giving the solution is not a
problem.
—
Reply to this email directly, view it on GitHub
https://github.com/eclipse/repairnator/issues/1261#issuecomment-1302431937,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIMPTOMUFXP24CNVPX2RST3WGPXU3ANCNFSM6AAAAAAQNDRRIE
.
You are receiving this because you commented.Message ID:
@.***>

FTR, official doc of SOBO: https://github.com/eclipse/repairnator/tree/master/doc/sobo


Fix initial issue update

fixes #233

Looks good, but also included in the other PR already (you need to make branches earlier :wink:)


Orchard does not support cookie domains per tenant.

Scenario:
You wish your site to issue auth cookies on the domain, rather than sub domain, for example; instead of issuing the .ASPXAUTH token to www.testdomain.com you wish it to be issued to .testdomain.com so that your user is signed into other sites on the same domain at the same time
Currently there is only one way to alter the cookie domain, that is to alter the forms section of web.config as below:
<authentication mode="Forms">
    <forms loginUrl="~/Users/Account/AccessDenied" timeout="2880" domain=".testdomain.com" />
</authentication>

However, this only works in a single site application, and would not work in a multi-tenant situation (unless you were fortunate enough that all your tenants were sub domains of the same root domain)
There are two routes that seem viable to me to resolve this

Cookie domain is a parameter available at tenant setup (along with Url Host and Url Prefix). This would allow you  to setup a site as Urlhost : www.testdomain.com with CookieDomain: .testcomain.com

Expose a public property on FormsAuthenticationService to allow extending code to set a value for this property

I'm keen on any feedback. If option (2) is considered a valid change to Orchard then we could submit a PR for this.

1 would seem very practical and sensible, but does bake in the assumption that cookies are used for authentication, which could be substituted for a different implementation. (On the other hand, Web.config assumes ASP.NET forms authentication is used for the whole site, so...)
2 seems like a partial solution. It adds an extensibility point, but you still have to add some other mechanism to determine the cookie domain.
I would opt for 1, as it would be a turn-key solution. Perhaps we could use a more abstract term than CookieDomain such as AuthenticationScope or something, so that it still makes sense even if a completely different mechanism is used to scope the authentication session?

Yeah exactly. For 2 one could make the cookie domain a public property on the FormsAuthenticationService and configure its value via HostComponents.config (which can be configured per tenant).

@sfmskywalker Yeah that's true - but seems a little disjointed maybe?

In what way? Isn't HostComponents.config intended to configure Autofac-registered services with properties? At the same time, HostComponents should probably be used for lower-level, core framework services, that aren't typically replaced with other implementations.
Alternatively, a site-setting could be introduced for the FormsAuthenticationService. Also tenant-specific.

Site settings to control the scope of authentication seems wrong, since you need to authenticate to even get to that site setting. Should be either ShellSettings, HostComponents.config or Web.config.
IMO by far the most logical and intuitive location would be ShellSettings. If you configure a tenant host URL in one place, it makes a lot of sense to configure the authentication scope (cookie domain) in the same place.
HostComponents.config sort of also makes sense, but it would require access to that file to even create a new tenant, which would really hurt multi-tenancy scenarios. It's worth a lot to be able to configure this from the tenants UI.

Agreed. If the name is made more abstract then a shell setting probably makes most sense.

Can I suggest this is a very specific requirement that can be solved in a separate authentication module ?

Single sign on can be done by creating a common service for all the site you want to authenticate to, and it can even spread different root domains. There are many open source system doing this, or you could rely on some public ones, AAD for instance, as there are some modules on the gallery for this.

You can even imagine having an Orchard module that would act as a sign-in authority, which would run in it's own tenant, and all other tenants would point to it for authentication.

Would it still not be a good idea to be able to have the FormsAuthenticationService to be configurable? Perhaps it could rely on a shell setting of which Orchard itself is agnostic, but FormsAuthenticationService would now which key to read from ShellSettings?
For this we would need to support 'extensible shell settings.txt', which I believe is a work in progress.

@sebastienros You can suggest it, but you'd be way off. :) This is not a very specific requirement at all. On the contrary, it's very generic. Which is why it is built into ASP.NET in the first place. The fact that a majority of sites will not use it, does not make it specific. Something can be very generic yet rarely needed.

Sadly in our situation we are not allowed to use true "Single Sign On" either (we tried and tried ;) )
I prefer option 1, but it could equally be solved if the FormsAuthenticationService took in some form of "AuthenticationScopeService" that defaulted to a null service type option

I don't believe that this is custom. In fact, it is already partially done in Orchard. Take a look the  the ISslSettingsProvider interface and it's default implementation. This exposes one property from the FormsAuthentication settings in web .config (requireSSL). How can it be argued that this is not a core implementation?
I propose that ISslSettingsProvider is replaced with IFormsAuthenticationSettingsProvider and a more complete implementation is exposed. I'll even do the work and submit it :)

Looking at the code related to ISslSettingsProvider, its sole purpose does indeed seem to be providing the FormsAuthentication.RequireSSL value and its only usage is from FormsAuthenticationService, so renaming it to IFormsAuthenticationSettingsAccessor seems to be even better. One argument against this could be that the intent of ISslSettingsProvider is to be forms authentication-agnostic. But since there aren't any usages beyond FormsAuthenticationService and a concrete need for someone to access the cookie value, I think we should change it. Alternatively, the FormsAuthenticationService class needs to be duplicated and another interface + implementation to access the cookie domain needs to be added.

Since we all seem to prefer option 1, why aren't we exploring that instead? :)

Regarding Option 1 and multi-tenancy: Will the browser read a cookie from another domain?  I don't think this is allowed.
i.e. specify cookie domain as .domain1.com and the tenant is actually running on domain2.com
(yes, this would work with sub-domain tenants or same domain tenants with URI prefixes), but not for tenants on a different domain altogether.  So, for multi-tenancy on the same root domain, the web.config setting would be sufficient.  But when the cookie is assigned to a completely different domain, the browser won't be able to access it and the tenant would appear as signed out.
This is why SSO exists.  To @sebastienros point, I think an SSO module is an optimal way of solving shared accounts in a multi-tenancy environment with different root domains ;)

There are many ways to solve SSO. Sometimes the absolute most efficient way of solving SSO is by re-using an ASP.NET authentication cookie across sites. Sometimes both sites use ASP.NET and you are in control of machine keys, etc - in such scenarios sharing the cookie is an extrelemy cost-effective way to achieve SSO.
@dcinzona as to your last point that the user needs to exist in the tenant, I don't think the requirement here is for a user to move between Orchard tenants, but rather for a user to be able to move between an Orchard tenant and some other site.
For example, let's say my business is to host car rental sites for my clients. For each client I provide:

http://www.myclient.com which is a tenant in my Orchard environment
http://analytics.myclient.com which is an instance of my custom analytics application, which also happens to be an ASP.NET application

I want my clients to be able to move between 1 och 2. My authentication solution makes sure user accounts are shared between the two. I just want to be able to control the cookie domain of 1, but since it's Orchard and running multiple tenants in the same ASP.NET web application, I can't just use Web.config for it, because for one tenant I want the cookie domain to be myclient.com and for another I want it to be myotherclient.com.
In other words, my tenants don't even share the same top-level domain.

@DanielStolt I agree with you, many ways to solve SSO...The only point I was trying to raise was that of multi-tenancy where the tenant root domains are different, in which case, I believe you cannot share a session/cookie because domainx.com can't read the session cookie from domainy.com
But I see what you are saying where you would have multiple organizations running within one orchard deployment as separate tenants, each with sub-domain tenants - in which case, yes, having this setting would greatly benefit that scenario.
As for shared users across tenants, it may not be abundantly clear that when sharing the session cookie, there is a requirement for users to be synced across tenants.

Actually, my assumption was that both www.myclient.com and analytics.myclient.com were running under orchard as tenants... You would have to be able to decrypt the cookie in that scenario, so both sites need to be designed in a way that they would be sharing a machine key or using a certificate to encrypt (and the sites would have to share the encryption certificate).
This could be an issue on Azure Web Apps because of how azure generates a machine key on deployment if not specified and also azure web apps defaulting to using DPAPI for encryption.
It may not be related, but during my SSO dev work, I had some frustrating experiences with DPAPI:
https://github.com/IdentityServer/IdentityServer3/issues/1710

@dcinzona yes it's a fair point that it might not be abundantly clear to everyone that cookie sharing also assumes username sync.
I don't think anyone would expect to be able to share a cookie between different root domains though. That doesn't even conceptually make sense.

@DanielStolt has the scenario I'm talking about exactly correct. The assumptions for this shared cookie scenario to work are:

Same Machine key for both sites
same root domain for both sites
At least one site issuing the authentication cookie against .domain.com rather than sub domain
Both sites would need to add the same "tenant name" value into the auth cookie user data

The complication in Orchard is is is a multi-tenant solution, so cannot easily partake in this scenario without affecting the cookie domain of ALL tenants.
As Daniel suggested, we have an external non-orchard site (it's Umbraco boo-hiss) that needs to share authentication with one tenant of our Orchard solution
(Interestingly there is a further issue in orchard that would stop two orchard tenants on the same landlord sharing authentication in this way - the tenant name is put into the userData value for the auth cookie and verified to ensure that two users with the same username on two different tenants cannot substitute cookies, but I think that this is an acceptable limitation)

Ok for cookie domain config.


docs/reference, how-to: add store limit topics

What have you changed? (mandatory)
Please explain IN DETAIL what the changes are in this PR and why they are needed:

This PR added

a how-to topic: configuring the scheduling rate limit
a reference topic: pd-control

What are the type of the changes? (mandatory)

Doc updates

@Hoverbear PTAL and merge.  Thanks.


Add continued fraction evaluator

Hi @fancidev. Here's a PR with the promised continued fraction evaluator. I've added a stub for iv_ratio using a different, simpler, continued fraction. That should give you an idea of how to use the evaluator, but let me know if you have any questions.
From here, you can try out Perron's continued fraction. Once you get it working well, then add some thorough tests. Unfortunately, we can't mix this generator with the ive implementation from AMOS without writing some annoying and unwanted wrapper code. Hopefully things will work well without that, otherwise this will have to wait until AMOS is translated into C++ (not as far off as you might think, we just did specfun for instance: https://github.com/scipy/scipy/pull/20243).
Unfortunately I don't have time to give more details today, but hopefully this can get you started.

@steppi Thank you very much! The startup code is super helpful! I'll spend some time to work out the numerical aspects for the specific problem, and continue discussion on the main SciPy issue to keep organised.
Btw, how is the iv_ratio related to the ellip_harm function? Wonder if I could use that as a benchmark or reference.

Btw, how is the iv_ratio related to the ellip_harm function? Wonder if I could use that as a benchmark or reference.

Sorry. I haven’t seen any specific formulas. I just know that Bessel functions arise when looking for separable solutions of the Laplace equation in spherical or cylindrical coordinates, and elliptical harmonics arise when looking for separable solutions in elliptical coordinates, which I think  you probably also know, otherwise you wouldn’t of asked about this.

No I have never heard of elliptical harmonics before :) I was asking just because I saw the comment in your code:
add_newdoc("_iv_ratio",
    """
    Internal function, use `ellip_harm` instead.
    """)

No I have never heard of elliptical harmonics before :) I was asking just because I saw the comment in your code:
add_newdoc("_iv_ratio",
    """
    Internal function, use `ellip_harm` instead.
    """)

Ah. That’s just because I copied the docstring  from another function without reading it. SciPy won’t build if you add a special ufunc without a docstring. It should just say “Internal function. Do not use.”


Move to new CDS and ADS

see https://forum.ecmwf.int/t/the-new-climate-data-store-beta-cds-beta-is-now-live/3315
Old CDS will be shut down 3 September 2024. The new grib -> netCDF converter that they use at the beta-cds is also still experimental and might change in the future. This will be difficult to support :/

also see New Coordinate variable names in netcdf conversion.


Component Sets / Variants?

Hi,
First of all: this is amazing! Thanks for creating and sharing the experiment :-)
Works like a charm for "flat" figma components, any plans to make it work for components with variants?
(e.g. buttons with a hover, pressed, active, focus etc variants?)
Thanks again!

Working on variants! For components they are split out as separate components in the file.
Still diving into the api. If you have any test figma files that would be awesome

If you have any test figma files that would be awesome

There's a really simple button here: https://www.figma.com/file/1FKxoZr1L5W4BVqaK7XoqV/ComponentCodegen?type=design&node-id=0%3A1&mode=design&t=rnOu3WDxNhjAhhaP-1
(Let me know if you need more than 'view' access!)


Explicitly disable pylint warning subprocess-popen-preexec-fn

This patch raises the question "what is the minimum version of python supported by TVM?" anyone?

by far we still aims to keep python2 compatibility, but I agree that is an issue that we should bring up the discussion again #1602


[comments] no way to close right sidebar on mobile because button offscreen

repro:

open composer on mobile
toggle the right sidebar in top right
you cannot close it, even with refresh

(for iOS Safari, the only way out is to delete all data for the dxos.org domain because right sidebar open state is stored in local storage)

What is “s2”?

@thure updated!

fwiw the canonical name for this component is “complimentary sidebar” (I shorten to “c11y sidebar”), but understood 👍
The design for this will substantially change, and this may have already been resolved by #5309.

@thure FYI this isn't resolved by the above PR. I still got stuck on dev with this over the weekend. I think it would make sense to resolve this at the same time as #5237 though.


Set Airflow's webserver hostname dynamically

Setting the hostname is not mandatory but very practical, specially when trying to consult logs.

Well, I'm forced to specify the whole URL including the port. This means I must either guess or override the port from airflow.cfg. Overriding is less error prone.

Turns out there can't be self references using GetAtt. An alternative is to inject it using the metadata query API:
curl http://<IP_ADDRESS>/latest/meta-data/local-ipv4

Metadata insertion via API query is working. Now comes an opinionated decision point: hardcore local-ipv4 or public DNS/IP? Or try the latter and fallback to former?


Fix reports table alignment

A very small CSS tweak to align the report selection table properly within the "Reports" page and stop Tessa from wincing every time she loads the page ;)
Before:

After:

YES absolutely


File dialogues broken

Prerequisites
* [x] Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
* [x] Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
* [x] Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
* [x] Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
* [x] Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
Neither File/Reopen Project nor File/Add Project Folder work if the main Atom window has been closed.

Steps to Reproduce

Launch Atom
Select File/Reopen Project or File/Add Project Folder

Expected behavior: In step 3 above I expect either Atom to open with my project or to be presented with a file dialogue.
Actual behavior: Nothing happens when selecting File/Reopen Project or File/Add Project Folder if the empty Atom window is closed.
Reproduces how often: 100%
Versions

brandons-mbp:~ Brandon$ atom --version
Atom    : 1.15.0
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0

Additional Information
Occurs in both Atom and Atom Beta on macOS 10.12.x.

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #12508. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Adding a way so that the InfoWindow can get a path to an html file

Today I wanted to style and make an InfoWindow for my marker, but it was pretty weird/difficult to design. I think that having to input and loooong html string as content is not very elegant. It would be better if I could set as the content string a path to a html file, instead of the html string.
it should look something like this:
const marker = new google.maps.InfoWindow({ content: './path/to/html/file' });
This would be a much more elegant approach and would imporve the developing experience because i wouldn't have to switch between an editing html file and copying the html document into the content string

The problem with this is that the library needs to be opinionated about the mechanism for loading that file and it will blow up with dependencies for doing so.
How are you loading the html file currently?

Currently I am using a filereader to read in the html file as a string but it is not very sophisticated because it can't read the referenced css file and reference it to the html-nodes


[Feature]: Implement sevenScuttle in GameState API

Feature Summary
We need to implement scuttling via a seven in the GameState API in order to support the full game logic in this API. See the spec for a general overview of the API, its underlying motivations, and the development approach.
Scuttling from a seven (sevenScuttle) means removing the targetCard from the opponent's points, removing the playedCard from the deck, and putting both of them into the scrap. It then requires setting the phase back to GamePhase.MAIN and incrementing the turn.
Detailed Description
Implementation should take the general approach shown in the Backend Control Flow section of the GameState API Spec and follow the example of #1025
Making this move, like all others, will be triggered via the endpoint:
POST /api/game/:gameId/move

Which move gets made is determined by the request body, which should specify a moveType field with a value from the MoveType enum. Playing a jack will be triggered by sending a request body like:
{
  moveType: MoveType.SEVEN_SCUTTLE,
  cardId: '9C',
  targetId: '8S'
};

When a moveType: MoveType.SEVEN_SCUTTLE is sent in the req body, the endpoint will attempt to utilize the sevenScuttle helpers: sails.helpers.gamestate.moves.sevenScuttle.validate() and sails.helpers.gamestate.moves.sevenScuttle.execute(). Each of these should follow the paradigm outlined in the docs:
There are four pieces to implement:

Update the api/policies/hasValidMoveBody policy to allow and validate requests with a moveType of MoveType.SEVEN_SCUTTLE
Implement the api/helpers/gamestate/moves/seven-scuttle/validate helper to verify if the move is legal
Implement the api/helpers/gamestate/moves/seven-scuttle/execute helper to create a new GameState that scuttles the targetCard with the card played from the deck.
Unskip the relevant tests by removing the cy.skipOnOnGameStateApi() commands from tests which can now pass in the GameState API

Updating the api/policies/hasValidMoveBody policy
We must also update the policy api/policies/hasValidMoveBody.js to accommodate the MoveType.SEVEN_SCUTTLE moveType. This policy is already applied to the endpoint for making a move and it uses a switch() statement on req.body.moveType to validate that a given req body has all the data necessary for the given moveType. We should add a MoveType.SEVEN_SCUTTLE case to this policy.
A moveType  of MoveType.SEVEN_SCUTTLE requires a cardId and a targetId which are both valid card ids. If both cardId and targetId are present in req.body and can be found in utils/DeckIds.json, then we can return res.next() in this switch case. Otherwise we should return res.badRequest() with an error that explains which req.body attribute is missing or invalid.
Implementing validate()
Create a new api/helpers/gamestate/moves/seven-scuttle/validate.js helper file that matches the general setup of the existing api/helpers/gamestate/moves/scuttle/validate.js helper. It should throw errors if the requested move is illegal. It should have three inputs:

currentState - the unpacked GameState from before the move is made
requestedMove -req body describing the move being made3.
playedBy - 0 or 1 for whether p0 or p1 is making the move.

validate() should throw if:

It is not the requesting player's turn (throw new Error('game.snackbar.global.notYourTurn');)
The GameState's phase !== GamePhase.RESOLVING_SEVEN (throw new Error('game.snackbar.seven.pickAndPlay');)
The playedCard is not in the top two cards of the deck (throw new Error('game.snackbar.seven.pickAndPlay');)
The playedCard is not a number card i.e. if playedCard.rank >=11 (throw new Error('game.snackbar.points.numberOnlyForPoints'))
The targetCard is not in the opponent's points (throw new Error('game.snackbar.scuttle.mustTargetPointCard')
The targetCard is bigger than the playedCard (compare first by rank then by suit to break tie, so 10 of spades is the biggest number card and it can scuttle any other point card, while the 5 of Hearts can scuttle the 5 of Clubs, 5 of Diamonds, and any Ace - Four, but not the 5 of Spades or any 6+). - (throw new Error('game.snackbar.scuttle.rankTooLow'))

Implementing execute()
Create a new api/helpers/gamestate/moves/seven-scuttle/execute.js helper file that matches the general approach of the existing execute helper api/helpers/gamestate/moves/scuttle/execute.js. It should create and return a new GameState that is the result of making the sevenScuttle move for the requested player i.e. it should remove the targetCard from the opponent's points, and the playedCard from the deck, then put both those cards into the scrap. Any cards in the attachments of the scuttled targetCard should be removed and put into the scrap as well. The playedBy, playedCard, and targetCard attributes should be included in the returned GameState. The GameState's phase should be set to GamePhase.MAIN, the turn should be incremented, the oneOff should be put into the scrap. The seven which was the oneOff should also be set as the resolved attribute of the GameState.
execute()  should have 3 inputs:

currentState - the unpacked GameState from before the move is made
requestedMove -req body describing the move being made
playedBy - 0 or 1 for whether p0 or p1 is making the move.

Unskipping Tests
Because we are approaching the GameState API project with a feature-flag approach, all changes are merged into the mainbranch and the application and tests are set to dynamically run in either the GameState API mode, or the legacy mode, based on theVITE_USE_GAMESTATE_API` env variable. In CI, we run the full suite of tests against the legacy version of the application and then run a subset of the tests against the GameState API version of the application.
Which tests are run in the GameState API version of CI is controlled via the test command cy.skipOnGameStateApi() which programmatically skips a test if the env var VITE_USE_GAMESTATE_API is enabled. As we develop new features in the GameState API, we should remove the cy.skipOnGameStateApi() command from more and more of the tests.
Essential tests
For this PR, at minimum, we should unskip the following tests, which should be able to pass once it is working correctly. All the following tests are in the tests/e2e/specs/in-game/one-offs/7_sevens folder:

playerSevens.spec.js - it('Scuttles from a seven')
playerSevens.spec.js - it('Scuttles using a NINE from a SEVEN')
opponentSevens.spec.js - it('Opponent scuttles from seven (top card)')
opponentSevens.spec.js - it('Opponent scuttles from seven (top card)')

You can run the tests against the GameState API locally by:
booting the server in gamestate mode with
npm run start:server:gamestate
Then booting the client in gamestate mode with
npm run start:client:gamestate
running the tests in gamestate mode with
npm run e2e:client:gamestate.

CLosed by #1098


duckdb too small to be a Parquet file

System Info
langchain               0.0.193
python 3.8.10
Who can help?
@agola11
@hwchase17
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
[X ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction

Im creating multiple vector stores using Chroma DB and saved these inside subdirectories inside a directory DB/ , all occurring inside of a docker container. I'm using persist method as you detail in the docs.
save and load work as expected, however, after container restart (an expected operation during development), I got the following error related to parquet file in the DB subdirectory: Invalid Input Error: File '/app/DB/SIC/chroma-embeddings.parquet' too small to be a Parquet file. During interaction with Chroma DB app only reads data not adds or overwrites data.

it seems that the reading operation overwrites the chroma-embeddings.parquet file.
Expected behavior
I expect that after the container restarts the DB could be loaded and not require to be rebuilt.

This is my DB definition using chroma.

I'm using persist due to another forum I see this method to persist data.
https://github.com/langchain-ai/langchain/issues/6938
I note that the db parquet file is overwriting when it is requested to search for similarity. I don't expect this behavior due that I suppose that is a reading operation only.


Installation fails with non-verbose conflict errors

❯ conda install -c beakerx beakerx_all
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Solving environment: /
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions

Trying to create a new env for just this package also doesn't work:
❯ conda create --name beakerx -c beakerx beakerx_all
Collecting package metadata (current_repodata.json): done
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: \
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError:

❯ conda create --name beakerx -c beakerx python=3.7 beakerx_all                    1
Collecting package metadata (current_repodata.json): done
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: |
Found conflicts! Looking for incompatible packages.
This can take several minutes.  Press CTRL-C to abort.
failed

UnsatisfiableError: The following specifications were found to be incompatible with each other:

Output in format: Requested package -> Available versions

Package python conflicts for:
beakerx_all -> notebook[version='>=5.7.4'] -> python[version='>=2.7,<2.8.0a0|>=3.10,<3.11.0a0|>=3.8,<3.9.0a0|>=3.9,<3.10.0a0|>=3.6,<3.7.0a0']
python=3.7
beakerx_all -> python[version='>=3.7,<3.8.0a0']

conda create --name beakerx -c beakerx -c conda-forge python=3.7 beakerx_all

seems to work. I can send a PR for the readme if anyone's interested.

yes please

please py 3.9

@NightMachinery
Can i get to know what's different with the last working command?