Fix lockit.gd

According to https://fr.flossmanuals.net/creating-point-and-click-games-with-escoria/i18n/ translation magic happens with the lockit.gd script.

After it runs, you will get a file called "strings.csv" with all the strings on your esc files, separated by file, and with some context information. You'll also have all the .esc.lockit files generated.

$ Godot_v3.0.2-stable_x11.64 -script ../tools/lockit.gd
OpenGL ES 3.0 Renderer: Gallium 0.4 on AMD POLARIS10 (DRM 3.19.0 / 4.14.14-ck1-ryzen, LLVM 3.9.1)
ERROR: load_source_code: Condition ' err ' is true. returned: err
   At: modules/gdscript/gdscript.cpp:792.
ERROR: load: Condition ' err != OK ' is true. returned: RES()
   At: modules/gdscript/gdscript.cpp:1856.
ERROR: _load: Failed loading resource: res://tools/lockit.gd
   At: core/io/resource_loader.cpp:186.
ERROR: start: Failed loading scene: res://tools/lockit.gd
   At: main/main.cpp:1672.
WARNING: cleanup: ObjectDB Instances still exist!
   At: core/object.cpp:1989.

It exits with 0.
mjt@lechuck:~/src/git_checkouts/escoria (devel *$%<>)$ find . -name strings.csv
mjt@lechuck:~/src/git_checkouts/escoria (devel *$%<>)$ find . -name \*lockit

I tested adding some debug prints but nothing really led me to understand wtf is up with this.
#59 should probably be related to fixing this, as well as #138

The lockit.gd script is fixed in #140. I will see if there are some other useful features that can be added to it, and then merge it, if no one objects.


Not able to auto login on index page

How to make it auto login on index page instead of hitting login on client 2? just like you login to google and on gmail you just see logged in by only hitting refresh on their public page?

Hi Atif, when logged in, ¿can you enter client2 private.html?

Yes I can but I want to login using index.html. Is that possible? Because i've seen gmail, youtube and google logging in using public pages?

Please help me with this I really want my client 2 to login from index.html just my refresh if my client 1 is already logged in.


Add a conda install option for trustllm

A conda installation option could be very helpful. I have already started working on this, to add trustllm to conda-forge channel.
Conda-forge PR:

https://github.com/conda-forge/staged-recipes/pull/25031

Once the conda-forge PR is merged, you will be able to install the library with conda as follows:
conda install -c conda-forge trustllm

:bulb: I will push a PR to update the docs once the package is available on conda-forge.

cc: @HowieHwong

The PR got merged. trustllm is now available on conda-forge.
This is the recipe maintaining feedstock: https://github.com/conda-forge/trustllm-feedstock
cc: @HowieHwong


keyholder-angular-23 Update to Angular 6

Codecov Report

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

@@           Coverage Diff           @@
##           master      #25   +/-   ##
=======================================
  Coverage   79.45%   79.45%
=======================================
  Files          59       59
  Lines         730      730
  Branches       77       77
=======================================
  Hits          580      580
  Misses        136      136
  Partials       14       14

Impacted Files
Coverage Δ

src/app/auth/auth.interceptor.ts
100% <ø> (ø)
:arrow_up:

src/app/long-press/long-press.directive.ts
52.38% <ø> (ø)
:arrow_up:

...age/storage-actions/edit-form/edit-form.service.ts
61.53% <100%> (ø)
:arrow_up:

src/app/auth/auth.service.ts
100% <100%> (ø)
:arrow_up:

src/app/alert/alert.service.ts
75% <100%> (ø)
:arrow_up:

src/app/http/http.service.ts
100% <100%> (ø)
:arrow_up:

src/app/storage/storage-api.service.ts
44.44% <100%> (ø)
:arrow_up:

src/environments/environment.ts
100% <0%> (ø)
: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 6c79865...97c8e86. Read the comment docs.


luci-app-transmission: Allow user to define custom web interface

There are exists several custom web ui for transmission. Such as:
https://github.com/ronggang/transmission-web-control
https://github.com/Secretmapper/combustion
https://github.com/fcsonline/react-transmission
But for now there are no way to use custom ui without editing init.d script.

Also implemented in transmission package https://github.com/openwrt/packages/pull/6459


chore(ava): turned on verbose mode

verbose mode seems very helpful during debugging, lifted the idea from kentcdodds/webpack-validator

Current coverage is 100.00%

Merging #25 into master will not affect coverage as of ceed84a

@@            master     #25   diff @@
======================================
  Files            2       2
  Stmts           43      43
  Branches         0       0
  Methods          0       0
======================================
  Hit             43      43
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of ceed84a

Powered by Codecov. Updated on successful CI builds.

I actually prefer not using verbose mode these days. If it's needed, you can use the flag. So I say :-1:

You can run npm t -- --verbose.


Incompatible types with esbuild 0.21.5

Current behavior

 Property 'with' is missing in type 'import("<PATH>/node_modules/@badeball/cypress-cucumber-preprocessor/node_modules/esbuild/lib/main").OnResolveArgs' but required in type 'import("<PATH>/node_modules/esbuild/lib/main").OnResolveArgs'.
npm ERR!
npm ERR! 27             plugins: [createEsbuildPlugin(config)],

Desired behavior
Since ^0.21.2 is listed, we expect this package to work with all patch version of esbuild or restrict it to < 0.21.5. Note that this is a brand new version of esbuild.
Test code to reproduce
This is not a functional issue, just a type issue that causes build errors. Setting esbuild to 0.21.3 resolved the issue, but logging as I anticipate others may run into this.
Versions

Cypress version:  "^13.6.6"
Preprocessor version: "^19.2.0"
Node version: 18.18.2

Checklist

[X] I've read the FAQ.
[X] I've read instructions for logging issues.
[X] I'm not using<EMAIL_ADDRESS>(package name has changed and it is no longer the most recent version, see #689).

The esbuild-ts example, which uses all latests versions, works seemingly fine. You need to provide a reproducible example to your issue.

Hi!
I have the same issue
.
It looks there is kind of inconsistency. As a temporary solution I'm adding highlighted line below. Please let me know if more details are needed.

Please let me know if more details are needed.

Alright - your post doesn't provide any useful information to this issue. If you want to see this issue resolved and contribute to that, you can provide a reproducible example, like OP has been asked of twice now: one in the issue template, twice in the comments.

Hi,
In your example could you please just use attached tsconfig file.
tsconfig.json
Unfortunately I cannot change moduleResolution property, so workaround was used.
I guess this is the main reason of the problem.
Thank you

I've updated dependencies, including esbuid, which I believe fixes this, released as v20.0.7. Try it out.

Everything is fine now. Thanks a lot!


Add YAMLConfigManager support

Purpose
$subject fixes https://github.com/siddhi-io/siddhi/issues/1445
Goals
To support YAML Config Manager for easy setting of system properties in SiddhiManager
Approach
As in f3c8ea2
Release note
Add support YAML Config Manager for easy setting of system properties in SiddhiManager through a YAML file
Automation tests

Unit tests - Attached herewith
Integration tests - N/A

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment
1.8.0_191

Codecov Report

Merging #1446 into master will decrease coverage by 0.26%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #1446      +/-   ##
============================================
- Coverage     67.75%   67.49%   -0.27%
  Complexity       41       41
============================================
  Files           634      642       +8
  Lines         26792    26888      +96
  Branches       4232     4242      +10
============================================
- Hits          18154    18148       -6
- Misses         6868     6969     +101
- Partials       1770     1771       +1

Impacted Files
Coverage Δ
Complexity Δ

...util/config/model/ReferenceChildConfiguration.java
0% <0%> (ø)
0 <0> (?)

...va/io/siddhi/core/util/config/model/Extension.java
0% <0%> (ø)
0 <0> (?)

...a/io/siddhi/core/util/config/YAMLConfigReader.java
0% <0%> (ø)
0 <0> (?)

...util/config/model/ExtensionChildConfiguration.java
0% <0%> (ø)
0 <0> (?)

...ddhi/core/util/config/model/RootConfiguration.java
0% <0%> (ø)
0 <0> (?)

.../io/siddhi/core/util/config/YAMLConfigManager.java
0% <0%> (ø)
0 <0> (?)

...va/io/siddhi/core/util/config/model/Reference.java
0% <0%> (ø)
0 <0> (?)

...dhi/core/exception/YAMLConfigManagerException.java
0% <0%> (ø)
0 <0> (?)

.../siddhi/core/util/event/handler/StreamHandler.java
62% <0%> (-12%)
0% <0%> (ø)

... and 7 more

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 30704d1...24d3836. Read the comment docs.


Activate User by Mail

Waiting for ApiKey security implementation

I validate, the ApiKey #5 integrate security for inter call micro-services


Azure Stack HCI 23H2 - Parentdisk + VM creation works, booting fails with 2408 ISO

creating Azure Stack HCI 23H2 using
23H2_2408_AZURESTACKHci23H2.25398.469.LCM.10.2408.1.3002.x64.en-us.iso
SHA256: BF8DF0F0397BECA33F3974A8BBD72EC988A326FD95EB11D549DF27780F00D67C
Repro:

Deploy MSlab 24.06.2 on Windows Server Insider b26296

Use parentdisks\createparentdis.ps1 and select ISO 23H2_2408_AZURESTACKHci23H2.25398.469.LCM.10.2408.1.3002.x64.en-us.iso  to create AzSHCI23H2_G2.vhdx
This will create parent disk correctly.

Use LabConfig of AzureStackHCIHOL 23H2
This will create all VMs correctly but the Azure Stack HCI 23H2 VMs will fail to boot after creation.
Additional VMs (MGMT, WAC) are not affected but using a different parent disk.

Error message:

Logs:
Deploy.log
Labconfig according to AzureStackHCIHOL 23H2
`
#basic config for Windows Server 2022, that creates VMs for S2D Hyperconverged scenario https://github.com/Microsoft/MSLab/tree/master/Scenarios/S2D Hyperconverged
$LabConfig=@{AllowedVLANs="1-10,711-719" ; DomainAdminName='LabAdmin'; AdminPassword='LS1setup!' ; DCEdition='4'; Internet=$true; TelemetryLevel='Full' ; TelemetryNickname='' ; AdditionalNetworksConfig=@(); VMs=@()}
#Azure Stack HCI 23H2
#labconfig will not domain join VMs
1..2 | ForEach-Object {$LABConfig.VMs += @{ VMName = "AzNode-2408-prod$_" ; Configuration = 'S2D' ; ParentVHD = 'AzSHCI23H2_G2.vhdx' ; HDDNumber = 4 ; HDDSize= 1TB ; MemoryStartupBytes= 24GB; VMProcessorCount="MAX" ; vTPM=$true ; Unattend="NoDjoin" ; NestedVirt=$true }}
#Windows Admin Center in GW mode
$LabConfig.VMs += @{ VMName = 'WACGW' ; ParentVHD = 'WinSrvInsiderCore_26296.vhdx'; MGMTNICs=1}
#Management machine (windows server 2025, but can be 2022)
$LabConfig.VMs += @{ VMName = 'Management' ; ParentVHD = 'WinSrvInsider_26296.vhdx'; MGMTNICs=1 ; AddToolsVHD=$True }
`

updated repro instructions. will try to repro on a different OS (W11 24H2 b26100.1742)

Host OS: Windows 11 24H2
Guest OS: Azure Stack HCI 23H2 2408
same behaviour

tested Host OS b26296
Guest OS: 23H2
ISO: 25398.469.231004-1141.zn_release_svc_refresh_SERVERAZURESTACKHCICOR_OEMRET_x64FRE_en-us.iso

Any errors when creating image? Looks like image was not correctly dismounted when creating.

There was a bit back and forth. Had to restart the PS and check for Dism stales
No 2311 and 2408 created without issues.
I believe the Hyper-V Parent OS version plays a role here.

The issue can be reproduced on WS 2022 host OS with same parent disks created in host OS 26262.
Also ran createparentdisk again for 2311 and 2408 on WS 2022.
Also here it fails booting.
As if didn't ingest unattend.xml during createparentdisk.

I ran through this with parent disks from host OS 26296 and Azure Stack HCI 23H2 2408 and could not reproduce - everything worked as designed.
@Karl-WE , can you capture the unattend.xml file that is generated on your system ($PSScriptRoot\Temp\unattend.xml) and attach it to this case?  I'd like to see if there is something unusual to it.
In particular, I am wondering if the PowerShell command Get-TimeZone is returning a localized name of the timezone, whereas unattend.xml (I think) needs an English name.

Thankfully we can close this issue.
Tested with

mslab dev
Host OS b26311
DC b26311

the Azure Stack HCI VM creation works again, without any issue in the first boot and unattend.xml.
Root cause remains unclear, possibly related to #607


Response metadata

We need to come up with an API to attach metadata (page numbers, sort directions, collection size etc.) to a response.

There's been some discussions about whether to do this in route handlers or serializers. The pragmatist in me says, why not both.

Let's add the ability to attach metadata in route handlers - this is probably where meta specific to the actual return object (model/collection) would be attached (like page info).

But also add a method to serializers, which probably has access to schema/db (and possibly the response object), for metadata that is common across multiple routes.

I think a pretty convenient way would be to have metadata directly in the collection (primaryResource) and letting the serializer handle it, like here: https://github.com/samselikoff/ember-cli-mirage/pull/916/files#diff-21df86ecb5f80627fb6dc6057c305faaR49. That way we can attach some metadata in route handlers (actually we can do it wherever, as long as we have access to primaryResource and it happens before returning final serialized body). It definitely works for pagination (check the PR above), should also work for sort directions, filtering etc.
Not sure about extra metadata being attached directly in serializers, I mean it sounds fine as an idea, but is it really needed? Any particular use case you have in mind?

Although this was discussed in #916 I wanted to make sure this is also kept in mind here. I find that it's important to give users the ability to control how data is paged, e.g., if query params were present and the returned payload is expected to conform to those query params, then one should be able to define that logic in mirage somewhere. Since query params are received in the route handler, this is where I thought one can overload the paginator, but otherwise, how would you alter what the paginator returns. Maybe it needs to be made a service that's injected into the route (if this makes sense)?

@ashrafhasson Yeah, that would definitely make sense to have the ability to modify the default behaviour from the route handler. I should find some time the following week to improve it.

I liked the options to handle in route handler and serializers.
I kwon that the ideia here is to solve the pagination using metadata. What about use the link tag too. http://jsonapi.org/format/#fetching-pagination . This could be a default behaviour  for the default json serializer?

Feature requests are being moved to Canny.
View this feature request in Canny


Fix typo in docs

@cferdinandi I think there is some small issue in the docs :

.text-tall instead of .text-large

Because Kraken uses relative sizing, you should always apply .text-tall and .text-small classes to a  element and not directly to a . Otherwise, your spacing will get all messed up. The .text-* classes can be found in _overrides.scss for better cascade inheritance.

in Typography section: https://cferdinandi.github.io/kraken/components.html#typography

Unneeded  at the beginning of code sample

Skip to content

in Visibility section: https://cferdinandi.github.io/kraken/components.html#alignment-spacing--visibility

Rename .screen-reader classes to .visually-hidden in Tabindex & Focus Styles section

Thank you! There were some conflicts with my markdown parsers and some unclosed HTML elements. Appreciate you letting me know about this!


HSTCAL Build 2.7.6

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

I am closing this PR as the auto-bot did the work from the HSTCAL release generated here: https://github.com/spacetelescope/hstcal/releases/


Fix first letter in chatbox moving player

Fix part of issue #2833 (Various isses with movement inputs persisting
through the chatbox) where the first letter typed into an empty chatbox was
causing the player to move.
This was caused by the event falling through to the normal event system,
because a path was not marked with eventHandled = true.
Note: there are still parts of #2833 that this doesn't fix, but might be related.
Contains
Fixes part of #2833
How to test
Open up a chatbox, making sure it is empty. Type a movement command (e.g. 'w'), and observe what happens.
Before the patch, the character typed would go into the chatbox, but the player would move, even though the chat box is open. After the patch, the character typed just goes into the chatbox, and doesn't make the player move.

Can one of the admins please verify this patch?


No possibility to override built-in keys

To work-around #288 I tried different things to get an own uses value.
One of the things I tried was
_customArguments = mapOf(
    "uses" to StringCustomValue("./")
)

which resulted in a duplicate key error.
It would be nice if keys from _customArguments  could override built-in keys for working-around cases not covered by the DSL yet.

Yep, totally agree.

See https://github.com/krzema12/github-actions-kotlin-dsl/issues/80#issuecomment-1200504222, it will be fixed in scope of that task.

After addressing #80, it now works properly. See e.g. this test for jobs:
https://github.com/krzema12/github-actions-kotlin-dsl/blob/9f803d557aa073c9dc35f6ecdc70807726cfe55f/library/src/test/kotlin/it/krzeminski/githubactions/yaml/JobsToYamlTest.kt#L273-L308


default datasource not created since Swarm 2016.12.0

When using Swarm from version 2016.12.0 onward, I get:

WFSWARM1005: Not creating a default datasource due to lack of JDBC driver

My guess is that there must be something missing from the fractions, but I'm not sure what.
@FroMage, @kenfinnigan, @bobmcwhirter  any idea?

Note, the exact same code works with swarmtool 2016.12.1.

So I guess it's a result of this breaking change in Swarm.

That would be my guess.
We changed to allow you to add a JDBC driver as a dependency and we'd auto create the module.xml for it

@kenfinnigan OK, but the problem is we don't use Maven, and instead call swarmtool using java -jar. And I'm not sure what to pass as an argument to have it include hsqldb.

Ah.
Could you raise a JIRA issue, as I suspect that's a gap we hadn't thought of

OK.
(I did try --fractions=com.h2database:h2:1.4.187 but that did not help.)

Issue reported as https://issues.jboss.org/browse/SWARM-1439.

Thanks @gavinking

Hi, @kenfinnigan is there some easy workaround for this issue? Some config file I can easily add to my project to force the default datasource to be created?
Thanks.

You're attempting this against a pre-built .war that you can't rebuild? If you can somehow get your driver within your WEB-INF/lib, then it should create the default DS based on that driver.

What if you add a dep to the h2 module in the Ceylon module, @gavinking?

When testing EAP quickstarts with Swarm, I found the same obstacle. I added the --dependencies switch, which takes a comma-separated list of G:A:Vs, and that solved my usecase.
So maybe --dependencies=com.h2database:h2:1.4.187?

FTR, this was tracked under https://issues.jboss.org/browse/SWARM-1100

What if you add a dep to the h2 module in the Ceylon module

Didn't help.

Where did it end up in the .war?

So maybe --dependencies=com.h2database:h2:1.4.187?

Where precisely? My swarmtool doesn't support this switch.

My swarmtool doesn't support this switch.

Oh, OK, it's added in newer versions. Fine.

@Ladicek alright, so here's my problem. swarmtool up until version 2017.3.2 didn't support the --dependencies argument. And so my app fails because the datasource can't be found.
But after 2017.3.2, the server starts, but simply doesn't deploy my JAX-RS app. This is all that is logged by the server startup:
Aug 18, 2017 4:22:41 PM org.jboss.weld.bootstrap.WeldStartup <clinit>
INFO: WELD-000900: 2.3.5 (Final)
Aug 18, 2017 4:22:41 PM org.jboss.weld.environment.deployment.discovery.DiscoveryStrategyFactory create
INFO: WELD-ENV-000020: Using jandex for bean discovery
Aug 18, 2017 4:22:42 PM org.jboss.weld.bootstrap.WeldStartup startContainer
INFO: WELD-000101: Transactional services not available. Injection of @Inject UserTransaction not available. Transactional observers will be invoked synchronously.
Aug 18, 2017 4:22:42 PM org.jboss.weld.environment.se.WeldContainer initialize
INFO: WELD-ENV-002003: Weld SE container internal initialized
Fri Aug 18 16:22:42 CEST 2017 DEBUG [org.wildfly.swarm.runtime.server] (main) Temporarily storing configuration at: /var/folders/g5/3h5202y10nlbqllm_ftn9kc40000gn/T/swarm-config-7773360740688947777.xml
Aug 18, 2017 4:22:42 PM org.jboss.msc.service.ServiceContainerImpl <clinit>
INFO: JBoss MSC version 1.2.6.Final
Aug 18, 2017 4:22:42 PM org.jboss.as.server.ApplicationServerService start
INFO: WFLYSRV0049: WildFly Swarm 2017.3.2 (WildFly Core 2.2.1.CR1) starting
Aug 18, 2017 4:22:42 PM org.wildfly.swarm.container.runtime.cli.CommandLineArgsServiceActivator activate
INFO: WFSWARM0029: Install MSC service for command line args: []
Aug 18, 2017 4:22:43 PM org.jboss.as.server.BootstrapListener done
INFO: WFLYSRV0025: WildFly Swarm 2017.3.2 (WildFly Core 2.2.1.CR1) started in 520ms - Started 28 of 32 services (4 services are lazy, passive or on-demand)
Fri Aug 18 16:22:43 CEST 2017 INFO [org.wildfly.swarm.runtime.deployer] (main) deploying 2984f714-5916-493a-ac13-e24f2c9a0a70.war
Aug 18, 2017 4:22:43 PM org.jboss.as.server.deployment.AbstractDeploymentUnitService start
INFO: WFLYSRV0027: Starting deployment of "2984f714-5916-493a-ac13-e24f2c9a0a70.war" (runtime-name: "2984f714-5916-493a-ac13-e24f2c9a0a70.war")
Aug 18, 2017 4:22:43 PM org.jboss.as.server.deployment.DeploymentHandlerUtil$1$2 handleResult
INFO: WFLYSRV0010: Deployed "2984f714-5916-493a-ac13-e24f2c9a0a70.war" (runtime-name : "2984f714-5916-493a-ac13-e24f2c9a0a70.war")
Aug 18, 2017 4:22:43 PM sun.reflect.NativeMethodAccessorImpl invoke0
INFO: WFSWARM99999: WildFly Swarm is Ready

So apparently there is also something else that changed.

Note that this is with an app that worked fine up until it was broken by swarm 2016.12.0.

@FroMage @bobmcwhirter On Swarm 2016.12.0 I tried adding com.h2database.h2-1.4.196.jar to the Ceylon overrides.xml file (equivalent to adding it to the module descriptor), and as you can see below, that resulted in the jar being correctly packaged into `WEB-INF/lib.
But the datasource still didn't get created.
gavin$ unzip -l jaxrs.example-1.0.0.war
Archive:  jaxrs.example-1.0.0.war
  Length      Date    Time    Name
---------  ---------- -----   ----
  1821816  08-18-2017 16:30   WEB-INF/lib/com.h2database.h2-1.4.196.jar
    16546  08-18-2017 16:30   WEB-INF/lib/jaxrs.example-1.0.0.jar
  1862166  08-18-2017 16:30   WEB-INF/lib/ceylon.language-1.3.3-SNAPSHOT.jar
   640912  08-18-2017 16:30   WEB-INF/lib/com.redhat.ceylon.model-1.3.3-SNAPSHOT.jar
   238061  08-18-2017 16:30   WEB-INF/lib/com.redhat.ceylon.langtools.classfile-1.3.3-SNAPSHOT.jar
   112326  08-18-2017 16:30   WEB-INF/lib/com.redhat.ceylon.common-1.3.3-SNAPSHOT.jar
   325129  08-18-2017 16:30   WEB-INF/lib/ceylon.interop.persistence-1.3.3-SNAPSHOT.jar
   199609  08-18-2017 16:30   WEB-INF/lib/ceylon.interop.java-1.3.3-SNAPSHOT.jar
   344791  08-18-2017 16:30   WEB-INF/lib/ceylon.collection-1.3.3-SNAPSHOT.jar
        0  08-18-2017 16:30   META-INF/
        0  08-18-2017 16:30   META-INF/ceylon/
     6082  08-18-2017 16:30   META-INF/ceylon/metamodel
---------                     -------
  5567438                     12 files

But after 2017.3.2, the server starts, but simply doesn't deploy my JAX-RS app.

There is probably some additional metadata required by Swarm 2017.3+, but I don't know what it might be.

@kenfinnigan @bobmcwhirter Do I need some sort of yaml file or something in the newer versions of Swarm?

You don't have to use yaml, but we're moving that way instead of custom main() usage

Well I don't have either a main() or a yaml, AFAIK. Do I need one?

Not unless you need to configure fractions in any way, such as DS config

So then I don't understand why it doesn't discover and start my JAX-RS application in the latest releases. What changed?

No error messages?
Endpoint not running at all?

Exactly.

Very weird. Is the project on GH somewhere we can play with?

Yes it is, it is https://github.com/DiegoCoronel/ceylon-jboss-swarm/ but you need to explicitly specify the version of swarm and the jdbc driver on the command line. Wait till I get home I will tell you the command I'm using.

OK, @kenfinnigan, the commands you need to compile, build, and run the jar are:
$ ceylon plugin install --force ceylon.swarm/1.3.3
Scripts for ceylon.swarm installed in /Users/gavin/.ceylon/bin/ceylon.swarm-1.3.3
$ pwd
/Users/gavin/ceylon-jboss-swarm/ceylon-wildfly-swarm-jaxrs
$ ceylon compile
Note: Created module jaxrs.example/1.0.0
$ ceylon swarm --swarm-version=2017.8.1 --dependencies=com.h2database:h2:1.4.196 --provided-module=javax.javaeeapi jaxrs.example
...
$ java-jar jaxrs.example-1.0.0-swarm.jar
...

Note that the command line options I've added are:

--swarm-version=2017.8.1 to select the version of swarmtool
--dependencies=com.h2database:h2:1.4.196 to pass the H2 jar as an additional dependency to swarmtool.

Let me know how you go.

Thanks for the details @gavinking, will take a look

Appreciated.

@gavinking Was able to reproduce the problem locally.
In the logs I see that undertow is the only fraction being detected by the call to ceylon swarm, which is why there is no JAX-RS endpoint being deployed.
Is ceylon swarm just a wrapper around SwarmTool, or is it more?

It's just a wrapper that directly calls swarmtool


Quantum geometric tensor features

What is the expected enhancement?
The quantum geometric tensor reads 〈∂kψ|∂lψ〉−〈∂kψ|ψ〉〈ψ|∂lψ〉 and the quantum Fisher information reads 4 * Re[〈∂kψ|∂lψ〉−〈∂kψ|ψ〉〈ψ|∂lψ〉].
Using the features implemented for the LinCombFull Circuit QFI methods implemented by PR 7632, we can easily compute the quantum geometric tensor using linear combination of unitaries methods.
To streamline the possibilities and clarify the methods for the user, we suggest adding a class of quantum geometric tensor methods to the gradient framework such that several QFI methods may be derived from them.
@Cryoris

Opflow is now deprecated soon to be removed. New features/enhancements are no longer a consideration.


Open modal from navigation

💬 Question
Is it possible to open a modal from a UI kitten navigation?
Using react navigation we could set the mode type on the Navigator tomode="modal" but this doesn't seem to be an option from looking at the UI Kitten docs.

Modal component has no common with navigation types. You might want to open a screen which makes Modal component visible, for example, when the screen becomes mounted.


lib/tests/formulae.rb:395: Use any_version_installed? instead of installed?

Same as #410 but for line 395
Failed log:
https://github.com/Linuxbrew/homebrew-xorg/pull/645/checks?check_run_id=702574635#step:6:73

Thanks again @maxim-belkin!


mod depends on very laggy source.

the mod requires a mod that lags my game from 30 to 1-2 fps. and my memory used jumps from 20-40 to 70-90. is it possible to make the mods self reliant to fix lag?

On what laggy mod does it depend? I don't understand what you mean

ok i just looked at some files and console logs and its not this mod. sorry for bothering you.


Any plan to support other platforms ?

Hey nice packages , it would be great if we can support other platforms as well somehow

Initial Windows support is now published on pub.dev.

@IVLIVS-III oh thats great , btw , does this plugin expecting python preinstalled in the device to work , or does it manages python virtual env. or something

This depends on your platform and environment:

the python_ffi plugin bundles the python runtime with any flutter app (no preinstallation necessary)
the python_ffi_dart plugin does not always bundle the python runtime:

on Windows, the python runtime is bundled with the platform
on macOS, a preinstallation is necassary

Is it possible to use in iOS/Android apps @IVLIVS-III ?

@devibrahimkarahan at this point in time, the plugin only supports desktop environments.
However the underlying architecture allows to add more platforms with quite low cost. The hardest part ist to compile Python itself for Android/iOS.
I personally, have no spare time left to try this compilation.
If anyone else is interested, contributions / pull requests are welcome.


MP5

Just added the mp5

The enum is supposed to be in incrementing value order, and the name vector in abc order. Could you fix that?

Also, both values are definition indexes so aren't they sopposed to be the same?

Never heard of them be the same and ups sry for messing with the abc order
I will fix that
On Sat 18. Aug 2018 at 00:43, namazso<EMAIL_ADDRESS>wrote:

Also, both values are definition indexes so aren't they sopposed to be the
same?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/namazso/nSkinz/pull/94#issuecomment-414005902, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ab_SZuQtAcmy0oxm8rY8mvjFKXmAnSy6ks5uR0cegaJpZM4WCRhp
.

Also i tested it before i pushed it ;) the mp5 + skins work

Yea, the enum values are just never used anywhere except that of knives and gloves. I think you mixed it up with classids? Could you fix that? Aince if it works that means the enum should also say 23

Oh yeah I did ah sry dude I’m really tired my brain isn’t working
On Sat 18. Aug 2018 at 00:50, namazso<EMAIL_ADDRESS>wrote:

Yea, the enum values are just never used anywhere except that of knives
and gloves. I think you mixed it up with classids? Could you fix that?
Aince if it works that means the enum should also say 23
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/namazso/nSkinz/pull/94#issuecomment-414006829, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Ab_SZndie_hPYNkKmsE5wHK9LkVCnjU6ks5uR0jBgaJpZM4WCRhp
.

Should not work on some projects when im really tired ...


bower install && grunt build issue

npm ERR! Darwin 15.0.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v4.2.1
npm ERR! npm  v2.14.7
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: bower install && grunt build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'bower install && grunt build'.
npm ERR! This is most likely a problem with the loopback-angular-admin package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     bower install && grunt build
npm ERR! You can get their info via:
npm ERR!     npm owner ls loopback-angular-admin
npm ERR! There is likely additional logging output above.
I'm on OS X El Capitan.
Thanks

I have the same issue

I eneded up getting it running by seperating out the commands and doing each with sudo and without : /

Actually running the commands differently works.

Actually, I realized later on running it all together was maxing out my ram on a tiny cloud9 VM, and running them separate were not

I have the same issue, Could anyone please post the complete solution?
It's impossible to push updates to heroku...

my complete solution to push to heroku is here https://github.com/beeman/loopback-angular-admin/issues/156
to solve grunt problems just change postinstall command to
bower cache clean && bower install && grunt build

The one-line install mentioned in the readme should work on a machine with enough capabilitues. If not, please reopen the issue.


GEOWAVE-546 - CLI Refactor

This update includes: #331 #416 #435 #546

@rwgdrummer @dcy2003 Do you guys want to review this?  Its pretty massive and I'd vote for getting it into 0.9.1 sooner than later so we can press forward.  I glanced through it and I really think thats all anyone is going to be able to do without setting aside a full day.  Let me know.

I peaked through it the other day.  Nothing stood out, so I'm good with moving forward with it.


Remove low-level coroutine suspension notifications

The opposite of #372
Fixes #1219

Tests passed locally...


Dedicated server

With the current version of Quasar it is impossible for multiple people to manage connected clients at the same time. To solve this, I recommend adding a dedicated server you can run on a Linux server which clients to connect to and whoever wants to manage the clients can connect to the dedicated server allowing multiple people to manage clients at once.

that would be great

https://github.com/quasar/QuasarRAT/blob/master/ROADMAP.md#add-http-api-to-the-server
it's great, but i yet not hope for it


Update All with Fragments and Dynamic Columns

In order to do an update_all with fragments the update bit needs to be in the ecto query via update:. However, update does not presently permit dynamic columns.
Motivating Example
I have a set of tables from which I am trying to excise identification information. There are a set of columns one each table that I want to overwrite, in this case the user name would simply become "User #{user.id}". This is demonstrable with the slightly simpler example:
col = :name
from u in User,
  update: [set: [{^col, "foo"}]]

malformed :set in update `[{^col, "foo"}]`, expected a keyword list

You get the same or similar errors with a bare col or field(u, ^col).
Issue created per conversation with @josevalim

We should probably allow using ^col there. I don't think field is a good idea, because we can update just the main table anyway - field would allow setting update for a joined table, which doesn't make much sense.

Yup agreed, ^col feels right to me here as well.


mesh check

provide utility to find where contacts and interfaces overlap with each other.  Optionally fix mesh to remove elements at intersections using contact/interface priority list

Hi, you have already output the warnings while there are some overlap between contact and interface. But I neglect them.  Q.Q


UITK rewrite

Windows:

[x] Popped-out window
[x] Docked window
[x] MainGui window
[ ] Settings window
[ ] Edit windows window
[ ] Flight Stage info window
[ ] Maneuver window header & footer
[ ] OAB Stage info window
[ ] OAB Celestial bodies selection
[ ] OAB window settings (torque)

Unity controls:

[x] BaseEntryControl
[x] TimeEntryControl
[x] LatLonEntryControl
[x] SeparatorEntryControl

Controllers:

[ ] Flight scene
[ ] OAB scene

Other:

[x] OnEntryValueChanged event
[ ] OnUnitChanged event
[x] Make window draggable
[ ] Clamp to screen
[ ] Saving window positions and states
[x] Better Settings & Close buttons
[ ] Locking windows (background alpha = 0, buttons hidden)
[ ] Window snapping
[ ] Make windows resizable
[ ] Maybe add configurable window scaling
[ ] Themes support


Reset Bosch dishwasher from the integration (Simulate pressing power button for 4 seconds)

Is your feature request related to a problem? Please describe.
In my Bosch dishwasher, I can reset the current program by pressing the power button continuously for 4 seconds, this allows me to change the program attributes after starting it.
Describe the solution you'd like
I would like to be able to do the same thing (reset) through the integration so that I can programmatically change the program to eco program for example if the dishwasher was started at night
Describe alternatives you've considered
Unfortunately, I have no alternatives
Dishwasher Model: SMS6HCB01A by Bosch
Integration version: 1.1.6

You can use the stop_program service to stop the current program, assuming that is supported by your appliance.

Yes, stop_program service works for the same purpose, thanks for your help


This line cannot work for loading bnip or nip

https://github.com/bottytools/botty/blob/859e8e725f5be68fa5c2f1957e126ca136704311/src/bnip/actions.py#L223
Need to modify to the below and that works for me.
bnip_file_paths = glob.glob(glob_bnip_path, recursive=True)+glob.glob(glob_nip_path, recursive=True)

@ruitwang Can you please provide more detail around this? (how your local directory configured? How did you have it configured when it wasn't working?)

@ruitwang Can you please provide more detail around this? (how your local directory configured? How did you have it configured when it wasn't working?)

With the original file, the program says it can not find any nip or bnip file in the bnip folder, the program always use the default bnip file as failback.
I thought it might be a file lookup issue so checked the source code.
After modifying this sentence,  the number of bnip or nip files can be correctly recognized by the program and work as expected.


Enhancement: Note Splash

why Don't you add Note splash? And also skins for note Splashe's?

siivkoi animated one, and its coming in 1.8.1


[Snowflake] Duplicate values in column filter for Date and Time columns

System information:

Win10
DBeaver CE 21.3.3

Connection specification:

Snowflake

Describe the problem you're observing:
In the column filter window for Date and Time columns you can see valгes with and without quotes. Both are inserted with quotes into the filter's search bar above the data grid.

Duplicate of https://github.com/dbeaver/dbeaver/issues/14858


can't use external mariadb container

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug

mapping the jc21/nginx-proxy-manager to my mariadb container still results in sqlite file being created
Nginx Proxy Manager Version

docker.io/jc21/nginx-proxy-manager:latest
To Reproduce
networks:
nginx-proxy:
driver: bridge
name: nginx-proxy
mariadb:
external: true
nextcloud:
external: true
services:
app:
image: docker.io/jc21/nginx-proxy-manager:latest
container_name: nginx-proxy
restart: unless-stopped
ports:
- '80:80'
- '82:81'
- '443:443'
environment:
- PUID=1000
- PGID=1000
- DB_MYSQL_HOST=mariadb:3306
- DB_MYSQL_DATABASE="npm"
- DB_MYSQL_USER="admin"
- DB_MYSQL_PASSWORD="admin"
volumes:
- ./data:/data
- ./letsencrypt:/etc/letsencrypt
networks:
- nginx-proxy
- mariadb
Steps to reproduce the behavior:

run docker compose up -d

Expected behavior

Screenshots

Operating System

Debian 12 Bookworm
Additional context

change DB_MYSQL_DATABASE to DB_MYSQL_NAME


sqrt(): argument 'input' (position 1) must be Tensor, not int

🐛 Bug
When using fairseq-train with RoBERTa model after 37 CUDA OOM's I get this error:
TypeError: sqrt(): argument 'input' (position 1) must be Tensor, not int
Cmd I used
fairseq-train 'data-bin/agora' \
    --task masked_lm --criterion masked_lm \
    --arch roberta_base --sample-break-mode complete --tokens-per-sample 512 \
    --optimizer adam --adam-betas '(0.9,0.98)' --adam-eps 1e-6 --clip-norm 0.0 \
    --lr-scheduler polynomial_decay --lr 0.0005 --warmup-updates 50 \
    --total-num-update 100 \
    --dropout 0.1 --attention-dropout 0.1 --weight-decay 0.01 \
    --max-sentences 6 --update-freq 40 \
    --max-update 100 --log-format simple --log-interval 1 \
    --restore-file 'roberta/checkpoint_best.pt' --save-dir 'my_models/agora' \
    --skip-invalid-size-inputs-valid-test --reset-optimizer

pretrained model downloaded from: https://github.com/sdadas/polish-nlp-resources/releases/download/roberta/roberta.zip

Traceback I got
2020-02-21 13:40:23 | WARNING | fairseq.trainer | attempting to recover from OOM in forward/backward pass
Traceback (most recent call last):
  File "/home/jacek/anaconda3/envs/py36/bin/fairseq-train", line 11, in <module>
    load_entry_point('fairseq', 'console_scripts', 'fairseq-train')()
  File "/home/jacek/Projects/fairseq/fairseq_cli/train.py", line 307, in cli_main
    main(args)
  File "/home/jacek/Projects/fairseq/fairseq_cli/train.py", line 102, in main
    train(args, trainer, task, epoch_itr)
  File "/home/jacek/anaconda3/envs/py36/lib/python3.6/contextlib.py", line 52, in inner
    return func(*args, **kwds)
  File "/home/jacek/Projects/fairseq/fairseq_cli/train.py", line 171, in train
    log_output = trainer.train_step(samples)
  File "/home/jacek/anaconda3/envs/py36/lib/python3.6/contextlib.py", line 52, in inner
    return func(*args, **kwds)
  File "/home/jacek/Projects/fairseq/fairseq/trainer.py", line 369, in train_step
    grad_norm = self.optimizer.clip_grad_norm(self.args.clip_norm)
  File "/home/jacek/Projects/fairseq/fairseq/optim/fairseq_optimizer.py", line 91, in clip_grad_norm
    return utils.clip_grad_norm_(self.params, max_norm)
  File "/home/jacek/Projects/fairseq/fairseq/utils.py", line 243, in clip_grad_norm_
    sum(p.grad.data.norm()**2 for p in params if p.grad is not None)
TypeError: sqrt(): argument 'input' (position 1) must be Tensor, not int

Environment

fairseq Version (e.g., 1.0 or master): master
PyTorch Version (e.g., 1.0) 1.4.0
OS (e.g., Linux): Ubuntu 18.04
How you installed fairseq (pip, source): pip install --editable .
Python version: 3.6.9 Anaconda
CUDA/cuDNN version: ?
GPU models and configuration: 750 TI

Your training command works for me. Are you sure you're on pytorch 1.4?

Yes it is 1.4. I will add tomorrow data needed to reproduce it. The thing is that command works, it trains model for about a minute and then throws an exception.

You mentioned seeing numerous OOMs. Does it work if you reduce the batch size so you don't OOM?

After changing to better GPU I didn't have any OOMs, and that problem vanished as well.


Istio ConsistentHash load balancing with http cookie and multi-cluster service mesh does not work

Bug Description
We have a multi-primary cluster service mesh and have a destination rule to route traffic to a pod based on the JSESSIONID cookie. When the pods in one cluster are shutdown the following error is given when the request lands on the cluster without the pods:
upstream connect error or disconnect/reset before headers. retried and the latest reset reason: connection failure, transport failure reason: TLS error: 33554536:system library:OPENSSL_internal:Connection reset by peer
Istio: 1.14.1
Version
[root@docker-build aeidt4]# istioctl version
client version: 1.14.1
control plane version: 1.14.1
data plane version: 1.14.1 (4 proxies)

Additional Information
No response

Need more info like what is the traffic flow like : from ingress gateway or eeast-west traffic?
What is the underlying network: flat or non flat?

east west gateway is used for cross cluster traffic. Requests for the app running in pods comes into the ingress gateway, service mesh then routes the traffic to the cluster that has running pods for that application. We can see the traffic coming into cluster 1 (deployment exists but no running pods) but it does not get routed to cluster 2 pods.
If we remove the following from the destination rule on cluster 1 then all traffic is correctly routed to cluster 2.
consistentHash:
httpCookie:
name: JSESSIONID
ttl: 0s


Camera improvements for VS2 - Part 1

Camera movement with mouse and zoom with scrollwheel

Task linked: CU-2a3b6fe Camera improvements for VS2


Bring back bank account transfers

I've seen this a few time, especially from people who don't want to use PayPal.  Maybe not right now, but someday.
Notify

[ ] https://gratipay.freshdesk.com/helpdesk/tickets/2445
[ ] https://gratipay.freshdesk.com/helpdesk/tickets/2405
[ ] https://github.com/gratipay/gratipay.com/issues/3654#issuecomment-127145743
[ ] https://gratipay.freshdesk.com/helpdesk/tickets/3028

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25443631-bring-back-bank-account-transfers?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

Blocking on https://github.com/gratipay/inside.gratipay.com/issues/119.

+1 from https://twitter.com/failvarez/status/644672998088163328

@failvarez
@Gratipay Hmm, just noticed that you guys are only accepting PayPal. Any plans to switch it up? Hard to support scum like that via fees.

+1 from https://twitter.com/untitaker/status/645356580251283456

@untitaker
@Gratipay Also I'd rather not need a Paypal account for this.


Use hostpath repo tag that includes beta snapshot changes, and fix overriding of junit results

Tested in https://github.com/kubernetes-csi/csi-driver-host-path/pull/133 and
https://github.com/kubernetes-csi/external-resizer/pull/68/files

/release-note-none


Change kyuubi.operation.result.format log level to debug

Why are the changes needed?

There is no need to log kyuubi.operation.result.format information every time a statement is executed.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

I'm fine w/ both info and debug, cc @cfmcgrady

Thanks, merged to master/1.7


Revert 'Hide Ripple effect if no onPress* method' to avoid re-rendering

The component now re-renders every time it's pressed and it makes e.g. autoFocus property of TextInput unusable because it autofocuses every time it's pressed.

Not sure if I understand this.
If you have onPress on your component, it will render the RippleFeedback in the exact same way as before.
If you don't have onPress on your component, it won't render the RippleFeedback at all so it can't cause anything.

@xotahal Of course, it was my mistake. Sorry, man. 🙂 I was dynamically changing onPress property and that's the reason why it was always re-rendering.

I realized this could be a problem after I wrote the comment ... we should probably manage it somehow.


#86 Fix thermostat for ios13

set target temperature is not working in ios13 as ios13 expected temperature unit (celsius, fahrenheit) to be changeable.  the temperature unit is used in homekit only for displaying purposes. at the protocol level the celsius is used as basis.
this patch makes temperature unit changeable.

I can confirm that this fixed my thermostat with ios13

+1


Add support for evaluating normalized Hermite polynomials

Resolves #127 .

Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines, including the relevant style guides.
[x] Read and understand the Code of Conduct.
[x] Read and understood the licensing terms.
[x] Searched for existing issues and pull requests before submitting this pull request.
[x] Filed an issue (or an issue already existed) prior to submitting this pull request.
[x] Rebased onto latest develop.
[x] Submitted against develop branch.

Description

What is the purpose of this pull request?

This pull request:

adds support for evaluating normalized Hermite polynomials

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #127

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

@stdlib-js/reviewers

@Planeshifter As I have modified the original PR in both moving the package to tools and by adding a factory method, would be good to get another review. Thanks!


[Feature Request] Add support for Verilator as a diagnostic backend in Veridian

Hi there,
I recently discovered that Verilator, as another prevailing open-source System Verilog simv toolchain, provides more informative diagnostics on System Verilog compared to verible-verilog-syntax. Therefore, I believe that adding an option in Veridian to support Verilator as a diagnostic backend would be very beneficial to its users, including myself.
To this end, I have implemented this feature in my pull request, with 3 lint levels that align with Verilator. And I believe this will notably enhance the functionality of Veridian. I kindly request that you test my implementation and provide any feedback or suggestions for improvement.
Thank you for considering my request.
Cheers,

Hilbert

Fixed by #188


Update Elixir/Erlang versions in CI

Pull Request Test Coverage Report for Build b1e4799df5711a297d8899fb2b360337b7e57cc8-PR-143
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.1%) to 99.083%

Totals

Change from base Build 761b35d385d76ef8469cc8dde37b4270471d568d:
1.1%

Covered Lines:
432

Relevant Lines:
436

💛  - Coveralls


[stable/mariadb] use credentials in exec probes

The liveness and readiness probes don't use login credentials, if specified, and fill up the Access_denied_errors counter:
MariaDB [(none)]> SHOW GLOBAL STATUS like "%Access_denied_errors%";
+----------------------+--------+
| Variable_name        | Value  |
+----------------------+--------+
| Access_denied_errors | 107706 |
+----------------------+--------+

This PR will fix this. I had to wrap the command with bash to access the valueFrom env var (see https://github.com/kubernetes/kubernetes/pull/42273)
Furthermore, I refactored the app labels to match the current best practices

/assign @seanknox

/ok-to-test

looks like this needs to be rebased @hoesler

rebased

/ok-to-test
/lgtm


Update recipe for py312_patch

Preparation for py312 release.

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:24: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: clone/recipe/meta.yaml:30: missing_pip_check: For pypi packages, pip check should be present in the test commands
Errors were found

Failed on osx64 and osx-arm64:
ModuleNotFoundError: No module named 'fsevents'


[Coverity] fixed 3 of 537 bugs found by Coverity

fixed 3 of 537 bugs found by Coverity, each one is described in it's commit
While I'll fix more bugs I'll push to coverity branch in my fork and add those fixes to this PR.
btw. @reduz and @tomreyn also have access to Coverity results

Nice work @Marqin!

Need to rebase for real now ;)

I was already rebasing when you wrote that :stuck_out_tongue:


Fix typo in README

Sorry for this PR but the small typo bothered me :)

Thanks, much appreciated!


No Chinese Simplified in Setup

By the way, I think it's better to present the latest version download link instead of JSON. It's would be more convenience to fetch update setup in about window.

Indeed, I could add Chinese to the setup. I removed all non-latin languages after some issue with the Russian setup language. Since there's no more hurry now, I can look in more detail what was the issue and restore them. As well as using the chosen language during setup as Grabber's language on first run if available.

By the way, I think it's better to present the latest version download link instead of JSON. It's would be more convenience to fetch update setup in about window.

What JSON do you mean?


Correct extension directives placement

Extension directives must occur before any non-preprocessor tokens.
this will solve shader compilation errors from below generated shader
7-17 03:56:00.100 12331 12609 D skia    : GLSL:
07-17 03:56:00.100 12331 12609 D skia    :    1 #version 100
07-17 03:56:00.100 12331 12609 D skia    :    2
07-17 03:56:00.100 12331 12609 D skia    :    3 varying highp vec4 sk_FragCoord_Workaround;
07-17 03:56:00.100 12331 12609 D skia    :    4 #extension GL_OES_standard_derivatives : require
07-17 03:56:00.100 12331 12609 D skia    :    5 precision mediump float;
07-17 03:56:00.100 12331 12609 D skia    :    6 varying mediump vec4 vQuadEdge_Stage0;
07-17 03:56:00.100 12331 12609 D skia    :    7 varying mediump vec4 vinColor_Stage0;
07-17 03:56:00.100 12331 12609 D skia    :    8 void main() {
07-17 03:56:00.100 12331 12609 D skia    :    9     mediump vec4 outputColor_Stage0;
07-17 03:56:00.100 12331 12609 D skia    :   10     mediump vec4 outputCoverage_Stage0;
07-17 03:56:00.100 12331 12609 D skia    :   11     {
:

I signed it!
On Wed, Jul 18, 2018 at 9:19 AM, googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I
signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/skia/pull/21#issuecomment-405800251, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAaqfBLyErbnYGrHW4rIwgeom-DalrAKks5uHrA9gaJpZM4VT5pa
.

--
Digish K Pandya

Closing old PRs with merge conflicts


Suggestion: Ripping other gallery formats

Would there be anyway to rip gallery formats other then just the standard (Scaps/Tags/Searchs/Ect).
P.S thanks for your work on ripme it works great.

Do you have an example URL? What's scaps? Tags? Is this for a specific site?

For websites such as hentai foundry and fur affinity there are scraps tags, this is basically where artists put works that aren't  finished or gave up on. Tags are basically search criteria, so instead of having to look for images with the content you are after you can type in a custom search and only see images with those tags.
Examples:
Search- http://www.hentai-foundry.com/search/index?query=resident&search_in=keywords
Scraps- http://www.hentai-foundry.com/pictures/user/DearEditor/scraps

At the moment rippers are designed to work by starting with the HTML at the link provided. Depending on how similar the pages are for the various queries, it could work if support for those URL formats was also explicitly added. I think this is an issue to be tackled on a ripper-by-ripper basis. I'm renaming this issu eto be specific about the hentai-foundry case.


Can we copy parts of the FFmpeg docs?

Does the FFmpeg license ([L]GPL) allow us to copy parts of the docs into our docs?
If we could somewhat automatically copy struct attribute docs, that could be extremely helpful.

I don't think we can. But we can certainly link to them.

To be honest I'm not sure the struct attribute docs help very much, as they make references to very C-specific things, like the sentinel values for arrays, etc..


New 3D control: Lock

There should be a control that can simulate a lock with either key or number combination unlock.

I am proposing we close this issue. Each lock is so different that it's going to be a custom job anyway. It's also fairly easy to build with simple Touch events. I think this should be closed

Seems kinda project specific feature. I can see it being useful to many but I would rather see this as a wip/ready PR from someone who actually needs/used a lock in their game instead of a generic idea "issue". I also suggest closing this until then.

Linked to https://github.com/thestonefox/VRTK/issues/1522


Hive

Some updates for Hive

Clean up ssh-hive tests to be less stateful
Handle CSV dialects from directories
Reduce the amount of full-database reflection that happens (this is expensive)
Accept stored_as= and external= kwargs to create non-textfile hive tables

TODO: insert syntax in PyHive doesn't work

Other comments?  I'd like to merge this for the release.


Investigate JS errors.

tweet_button.1379006964.html:1 GET http://p.twitter.com/t.gif?_=1379111570224&count=horizontal&hashtags=webper…_widget=0&twttr_hask=0&twttr_li=0&twttr_pid=v3%3A1413697886829303693496634 net::ERR_NAME_NOT_RESOLVED

follow_button.1375828408.html:1 GET http://p.twitter.com/f.gif?_=1376940868174&id=twitter-widget-2&lang=en&scre…_widget=0&twttr_hask=0&twttr_li=0&twttr_pid=v3%3A1413697886829303693496634 net::ERR_NAME_NOT_RESOLVED

follow_button.1375828408.html:1 GET http://p.twitter.com/f.gif?_=1376940868174&id=twitter-widget-2&lang=en&scre…_widget=0&twttr_hask=0&twttr_li=0&twttr_pid=v3%3A1413697886829303693496634 net::ERR_NAME_NOT_RESOLVED

(index):1 GET http://a.adroll.com/j/roundtrip.js net::ERR_BLOCKED_BY_CLIENT

include.preload.js:237 Uncaught SecurityError: Failed to read the 'contentDocument' property from 'HTMLIFrameElement': Blocked a frame with origin "http://dev.bootstrapcdn.com" from accessing a frame with origin "https://platform.twitter.com".  The frame requesting access has a protocol of "http", the frame being accessed has a protocol of "https". Protocols must match.

I don't get those errors for http://www.bootstrapcdn.com/ using Firefox 35.0.1.
The twitter errors don't make sense since we use the protocol relative URLs everytwhere. Maybe those were a one time thing?

This can also be closed, unless you still see errors @jmervine. Personally, I don't see any.

I've concluded that this is related to my ad blocker.


Wa/mobile/fix/610 revise widgets

The static widgets of our project revised excluding visitor bottom navigation bar, it will be done later.

Conflicts will be resolved.

LGTM


[FLINK-12929] Pass TypeInformation in addSource

Co-authored-by: Georg Rollinger<EMAIL_ADDRESS>
What is the purpose of the change
Fixes https://issues.apache.org/jira/browse/FLINK-12929:
Changed scala.StreamExecutionEnvironment.addSource to pass the type info for a SourceFunction that does not extend ResultTypeQueryable.
Brief change log
see above
Verifying this change
No test added as scala/StreamExecutionEnvironment.scala seems to be not covered yet
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@aljoscha What is the overall contract in the Scala API for which type information is used:
Consider this case:
env.addSource[StringTypeInfo.INSTANCE](sourceThatImplementsResultTypeQueryable);

Which info will be used / should be used in the end? Is this well defined?

I think it was somewhat well defined until this change: https://github.com/apache/flink/commit/e39699f3eb41517a7035a5e626c6bb20c6b70501#diff-8c6a04f65f5a61e021e96322e8b06432R1000
Before, the idea was that if the source implements ResultTypeQueryable, then that should take precedence. Otherwise we try to deduce a TypeInformation. I think we should go back to that, i.e. ResultTypeQueryable takes precedence (in addSource()).
The fact that the Scala API uses returns() comes from the fact that there was no StreamExecutionEnvironment.addSource(SourceFunction, TypeInformation). Nowadays, I think no internal methods should use returns().
For this PR, it means that we have to not call returns(), but always use the constructor. And then in the constructor we have to do the correct change to make ResultTypeQueryable take precedence.

@lombarde Are you interested in picking this up, following @aljoscha's suggestion?

@lombarde Are you interested in picking this up, following @aljoscha's suggestion?

We updated our code following your suggestions, but we are not completely sure we understood what you meant. Please double check.

I merged this, thanks for your work @lombarde. 😃


Revert "libhri: 0.5.3-1 in 'melodic/distribution.yaml' [bloom] (#35053)"

This reverts commit 0a47f870e5dc3993b57390d728539af2f2a37c27.
This update is failing to build; see https://build.ros.org/view/Mbin_uB64/job/Mbin_uB64__hri__ubuntu_bionic_amd64__binary/12/console .
@severin-lemaignan FYI

@clalancette -- thanks for looking into it. I'm a bit puzzled by the error, I must say:

first the build seems to use the package version 0.5.1 instead of the released 0.5.3:

/tmp/binarydeb/ros-melodic-hri-0.5.1/include/hri/face.h:33:10: fatal error: hri_msgs/NormalizedPointOfInterest2D.h: No such file or directory
 #include <hri_msgs/NormalizedPointOfInterest2D.h>
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

second, the error is about a missing header in hri_msgs, but both melodic and noetic have the same version of hri_msgs (0.8.0), and the build is fine on noetic...

not sure what is going on here!


OpenFileDialog doesn't select a file, just when it switches the focus to another app and go back

Describe the bug
I can not select a file when the OpenFileDialog is showed, it seems to be blocked, but when I switch to another app and I go back to avalonia app it works well.
To Reproduce
Steps to reproduce the behavior:

Open avalonia app
Display the OpenFileDialog
Try to select a file
See error

Expected behavior
Be able to select a file without switching the focus to another app
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: MAC
Monterey 12.6

https://user-images.githubusercontent.com/43356162/225033615-a9ec8a22-b908-4020-9eee-2fcb0941740b.mov
Additional context
I'm working with MVVM and I have the OpenFileDialog in class which is like a service.
`		async Task<string[]> OpenFile (object parent = null, string title = default, bool allowMultiple = false, string filterName = null,
string[] extensionFilter = null, string defaultFolder = null, string defaultName = null)
{
parent = parent ?? LMApp.Current.GUIToolkit.MainController;
List filters = GetFilters (filterName, extensionFilter);
		var dialog = new OpenFileDialog {
			Title = title,
			AllowMultiple = allowMultiple,
			Filters = filters,
			Directory = defaultFolder,
			InitialFileName = defaultName
		};
		return await dialog.ShowAsync (parent as Window);
	}

`

Avalonia version?

0.10.18

I am also on 0.10.18 and can confirm having the same issue. For some reason adjusting the height of the underlying window has an effect on this bug.

Please check 11.0 previews

I tried to replicate the issue on my machine using .net 6, net472 and 0.10.18 of Avalonia. It seems to be working well in Avalonia. I will close this Issue. Thanks!


iOS Wrapping react-native into dynamic library caused Fabric components not displayed

Description
I wrapped libPods-ReactNativeSDK.a into a dynamic library for the app to depend on it, but my custom Fabric component cannot display (init method not called), and there is no error message.
Steps to reproduce
1.Create dynamic library xcode project
2.install react native by Pod install
3.compile this project into .framework file
4.My app embeded the .framework file
5.Run DYSanae
React Native Version
0.72.5
Affected Platforms
Runtime - iOS
Areas
Fabric - The New Renderer
Output of npx react-native info
System:
  OS: macOS 14.2.1
  CPU: (8) arm64 Apple M1 Pro
  Memory: 288.92 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.5.1
    path: /opt/homebrew/bin/node
  Yarn:
    version: 1.22.21
    path: /opt/homebrew/bin/yarn
  npm:
    version: 9.8.0
    path: /opt/homebrew/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.11.3
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: Not Found
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 20.0.1
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react: Not Found
  react-native: Not Found
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: false
  newArchEnabled: false
iOS:
  hermesEnabled: false
  newArchEnabled: true

Stacktrace or Logs
no exception logs captured

Reproducer
https://github.com/nanatlantica/RNDynamic
Screenshots and Videos
https://github.com/facebook/react-native/assets/6023483/a1d54656-e546-4153-b87b-adf375269421

Closing as duplicate of https://github.com/facebook/react-native/issues/43343
@cipolleschi already answered you over there

So you meaning that I should link react-native as a static library, not a dynamic library?

Static libraries and dynamic libraries work with React Native.
What is not supported is wrapping react native in a single piece and drag-and-dropping it around as an XCFramework. We use cocoapods to set up several build flags and build settings that you need in order to make it work properly. If you wrap React Native in a framework and you move it around, you lose those settings.
That setup is not supported and we don't have capacity to support you in the investigation.


Feature: Proxy mode: change proxy every N-th request

If it's possible, please, add a possibility to change proxy every N-th request.
Add a variable (for setting N) and a new value for "Proxy mode" for this.

Some services like Luminati provide a single endpoint for all requests.
The right approach is to re establish the connection (after n connections, or in case of failure) to the same endpoint.
Currently, if the endpoint gets blocked, the crawl will be terminated because there's no more proxies available.
If I try to copy paste the proxy multiple times in the file, is still being aggregated in the code.


domain student.giu-berlin.de

@PyTch18 I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host studentgiu-berlin.de not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Sidenav component errors

It's just that you have errors on your showcase page... Chrome 59.0.3071.86 (64-bit)

Being tracked at #4969

Dupe of #4969


CDN packages update

Hello,
Do you think you could make a new version oh pheditor with CDN packages update.
For example:
JQuery from v3.3.1 to v3.7.0
Twitter bootstrap from v4.3.1 to v4.6.2
fontawesome from 5.15.1 to 5.15.4 (or the last v6.4.2)
CodeMirror from v5.43.0 to the last v5.56.14 version (v6 exists but not sure if there is broken changes)
JsHint from v2.10.2 to v2.13.6
JsTree from v3.3.7 to v3.3.15
...
Thanks for your work on this great project.

Hi @slolo2000,
CDN libraries and packages updated to latest version hosted on cdnjs.
https://github.com/pheditor/pheditor/commit/58362cf85f45de600d6c7aa9ecdd1b32eee7dbb5


Support TokenInfo validation for Google OAUTH 2.0

See the "TokenInfo validation" on this page from google OAUTH 2 docs: https://developers.google.com/identity/protocols/OAuth2UserAgent
This validation helps prevent faking a login callback to get the system to grant access to another user.

seems like not-OAuth2 thing (http://tools.ietf.org/html/rfc6749), but google specific.
But I do not see anything against implementing it in subscribe.
Before this, you can easily make such request to google API after getting access_token (actualy subscribe's part of work).
Hope someone will find time to code this here. I have no such needs at the moment. May be later.
Thanks for issue!


Error while running raycaster

Preprocessing executable 'raycaster' for raycaster-<IP_ADDRESS>...
[10 of 10] Compiling Main             ( src/Main.hs, dist/build/raycaster/raycaster-tmp/Main.o )

src/Main.hs:106:21: parse error on input ‘=’

@rumblesan, I will look into this later

Fixed with https://github.com/bkach/HaskellRaycaster/commit/30bc9ef3dc7fa85434bf24e61157721b54201aab


!github command linking to the bot github

You could've at least added something like: It would be nice to have a !github command that links to this repo


Vite-node uses undeclared export vite/types/hot

Describe the bug
This line uses an undeclared export:
https://github.com/vitest-dev/vitest/blob/a96bc2247323159ac413a3ac0a1b27999a9efbfa/packages/vite-node/src/types.ts#L1
When running typescript with "moduleResolution": "bundler", it fails compilation with undeclared exports.
The allowed vite exports can be found here: https://github.com/vitejs/vite/blob/126e93e6693474a038a5053b7cefb99295f21eb5/packages/vite/package.json#L21-L32
Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-spzqkv?file=package.json,tsconfig.json&initialPath=__vitest__/
System Info
n/a

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

My temporary workaround using patch-package:
diff --git a/node_modules/vite-node/dist/server.d.ts b/node_modules/vite-node/dist/server.d.ts
index a21e596..80b6bd2 100644
--- a/node_modules/vite-node/dist/server.d.ts
+++ b/node_modules/vite-node/dist/server.d.ts
@@ -1,3 +1,4 @@
+//@ts-expect-error
 import { TransformResult, ViteDevServer } from 'vite';
 import { E as EncodedSourceMap } from './types.d-7442d07f.js';
 import { g as DebuggerOptions, D as DepsHandlingOptions, f as ViteNodeServerOptions, F as FetchResult, e as ViteNodeResolveId } from './types-e8623e9c.js';
diff --git a/node_modules/vite-node/dist/types-e8623e9c.d.ts b/node_modules/vite-node/dist/types-e8623e9c.d.ts
index dfe926f..ac99fe8 100644
--- a/node_modules/vite-node/dist/types-e8623e9c.d.ts
+++ b/node_modules/vite-node/dist/types-e8623e9c.d.ts
@@ -1,3 +1,4 @@
+//@ts-expect-error
 import { ViteHotContext } from 'vite/types/hot';
 import { E as EncodedSourceMap } from './types.d-7442d07f.js';


Fixes #3929 Allow exporting functions named "default".

This PR fixes #3929.
This PR adds the keyword defaultto the argument skip_keywords for fn function_from_decl when exporting functions.

I believe that this should be safe to add. @Liamolucko would you mind double-checking me on this?

This doesn't work with --target web, since the default export is already used for init. But it works with all other targets.
I don't particularly like adding target-compatibility caveats like this, but I also can't really imagine a scenario where it's an issue - discovering that you aren't allowed to set the default export on --target web isn't really any weirder than discovering that it got silently renamed to _default.
If we do go ahead with this it'd be nice if the CLI printed a proper error when trying to create a default export on the web target, rather than delaying it until the JS is actually used and throws a 'duplicate default export' error.

If we do go ahead with this it'd be nice if the CLI printed a proper error when trying to create a default export on the web target, rather than delaying it until the JS is actually used and throws a 'duplicate default export' error.

I think that sounds reasonable enough. I'll try to extend my PR with a check for --target weband #[wasm_bindgen(js_name = default)]

I've added a check that throws an error if wasm-bindgen-cli is called with --target web and there exist a exported symbol called "default". I'm notoriously bad at creating good error messages, so feedback is welcome 😆

@daxpedda @Liamolucko Not to be pushy, but is there a chance I can get a new review ?

I agree with whats said in https://github.com/rustwasm/wasm-bindgen/pull/3930#issuecomment-2067918594, I didn't know that this can't work with the Web target and I think its not the best idea to introduce such target-incompatible changes.
In hindsight I believe that https://github.com/rustwasm/wasm-bindgen/pull/2855 should not have simply renamed functions, but instead thrown an error.
So I currently think the way forward is to instead of renaming exports where it would make a semantic difference, export them with the export { _default as 'default' } syntax (inspired by https://github.com/rustwasm/wasm-bindgen/pull/2855#discussion_r1191380905) or e.g. in the case of the no-modules target can export as-is. Where this is not possible it should return an error.
Let me know if this doesn't address your problem @nicklaswj.
Apologies for the delay, I was on vacation.

The point is to make exporting default possible on all targets.
In this case the issue was that default is a keyword in ES modules, which prevented this PR from making a fix for both targets. My suggested solution is to use the export { _default as 'default' } syntax to work around that limitation for the Web target.
This work around could be applied to other keywords as well, therefore a fix should attempt to craft a general solution.
Let me know if anything else is unclear.

In hindsight I believe that #2855 should not have simply renamed functions, but instead thrown an error.

I agree.

In this case the issue was that default is a keyword in ES modules, which prevented this PR from making a fix for both targets. My suggested solution is to use the export { _default as 'default' } syntax to work around that limitation for the Web target.

The PR already ends up doing that though: more specifically export { default1 as default } (turns out it still works without the quotes). It seems to be because of some old code here that special-cases default (but it seems like it was written to allow importing default, not exporting it).
The issue is that the default export is reserved for init on the web target, and so it can't be used to export anything else. It's also literally just an export named 'default' with a bit of syntax sugar around it, so there's no way to have both a default export and an export named 'default' - they're the same thing.

I do not really have an opinion on how we solve it, however it is blocking me. @Liamolucko, if I understand your latest comment correct, I feel like we haven't really reached a conclusion whether I should change anything in my PR or not - is that correctly understood?

@Liamolucko, if I understand your latest comment correct, I feel like we haven't really reached a conclusion whether I should change anything in my PR or not - is that correctly understood?

Well, there are 2 options:

Only error if default is exported on the web target (current state of this PR)
Error regardless for uniformity

At this point I'm leaning towards option 1 (how often do people really switch targets anyway?), but I was waiting to see what @daxpedda's thoughts were.

however it is blocking me.

In the meantime, how important is it that the default export comes directly out of wasm-bindgen? Is there anything stopping you from making a JS wrapper that reexports it as default?

The issue is that the default export is reserved for init on the web target, and so it can't be used to export anything else. It's also literally just an export named 'default' with a bit of syntax sugar around it, so there's no way to have both a default export and an export named 'default' - they're the same thing.

Well that's unfortunate, and here I thought I found a solution that could work.

Well, there are 2 options:

Only error if default is exported on the web target (current state of this PR)
Error regardless for uniformity

At this point I'm leaning towards option 1 (how often do people really switch targets anyway?), ...

Yeah I think I'm fine with proceeding with the PR then. A solution that came to mind is to add an attribute to explicitly opt-in to the target incompatibility, but I think that's overkill.


Parsing >>>>>>… takes quadratic time and eventually crashes with recursion error

>>> import markdown, timeit
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 100)).autorange(); dur / num
0.004344235559692607
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 200)).autorange(); dur / num
0.016907797248859425
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 300)).autorange(); dur / num
0.05682634799741208
>>> markdown.markdown(">" * 331)
…
RecursionError: maximum recursion depth exceeded while calling a Python object

Same results in 3.0.1 and master.

I'm curious where this is coming from. My recollection was that we ignore > unless we first have <.
In any event as per the discussion in #789, I don;t care about the performance issue. But we should address the RecursionError.

This might be related to block quotes if I had to guess.

Ah, right, infinently nested block quotes. Except that they require a space. The token is [greaterthan][space] together. Without the space, the blockquote processor should ignore it.

I’m not sure which specification you aim to follow (if any), but both Markdown.pl and CommonMark agree that the space after > is optional in the blockquote syntax. In any event, the same behavior can be observed with > > > > > > ….

Python-Markdown does not require a space between >s too:
>>> print(markdown.markdown(">>>foo"))

foo

Sorry. I guess I mis-rembered that. So yeah, infinitely nested blockquotes.
I'm not sure how to address this. When the block parser refactor in 2.0 was done, I removed most of the recursion, but there are a few places it remains. And those few locations in the code will only ever hit the recursion limit with contrived input. On the one hand I want to say: " don't do that" but  I realize it is untrusted users who are the problem here.
I don't think we can just not support this contrived input. As it turns out, most implementations handle this pretty consistently with shorter strings of >s. Although, they do start to diverge with longer strings (I did not check every iteration to find where they divergence starts).

A simple option could be to allow for maximum nesting depth setting (if that's not already the case somewhere). I can't immediately think of any use cases for over 300 nested block quotes, unless you render your work email chains into markdown for some reason.
Though I suppose that still leaves the issue if your maximum depth is above the recursion error threshold

I can't immediately think of any use cases for over 300 nested block quotes, unless you render your work email chains into markdown for some reason.

This is the problem. Years ago (before GitHub even existed) we got a bunch of bug reports because someone was trying to pass plaintext version of the complete works of Shakespeare through our parser. The document was not Markdown formatted and we were hitting recursion limits due to the weird syntax.
Additionally, we promise our users that we won't crash their server. If a random user can submit plain text to your site and that crashes your server, you won't want to use our lib anymore. Therefore, we must always assume the worst sorts of input. Sometimes that means things are less performant in the simple case. Its a tradeoff.

A simple option could be to allow for maximum nesting depth setting

As I understand it, on different systems Python's maximum recursion depth may be different based on the available resources. If someone has a valid need for a higher nesting depth, then that should just work for them with no need to adjust a setting. I think perhaps the solution is to catch the RecursionError and simply return the unparsed levels as plan text. That ensures the maximum level of nesting possible is always supported and avoids an error being raised.
The down side is that things get really slow before the error is raised. But that is a tradeoff I'm willing to make when the user is trying to parse contrived input.

I'm not sure how impactful can the latency grow in reality, but it might be worth adding an optional setting regardless in case your normal use cases don't consist of immense nesting levels. Making it enabled by default might not be necessary, though I personally wouldn't mind that either.
 So at least for my use case it would make far more sense to restrict max depth rather than continuing indefinitely until a recursion error, given the performance impact it has.

The problem with catching the RecursionError is that by that time we are already too deep. Even a single additional function call could cause the error to be raised again. Therefore, ideally we should catch the issue shortly before the error, but not too close to it. However, as one level of the nesting depth in the syntax may result in multiple levels of Python's stack, we can't directly associate the nesting depth with the stack depth. And while we could make an approximat guess, future changes to the code could alter that.
I came up with a easy fix which cuts off nesting at approximately 100 levels from Python's stack depth limit:
diff --git a/markdown/blockprocessors.py b/markdown/blockprocessors.py
index 60baca1..88ebb62 100644
--- a/markdown/blockprocessors.py
+++ b/markdown/blockprocessors.py
@@ -276,7 +276,7 @@ class BlockQuoteProcessor(BlockProcessor):
     RE = re.compile(r'(^|\n)[ ]{0,3}>[ ]?(.*)')

     def test(self, parent, block):
-        return bool(self.RE.search(block))
+        return bool(self.RE.search(block)) and not util.nearing_recursion_limit()

     def run(self, parent, blocks):
         block = blocks.pop(0)
diff --git a/markdown/util.py b/markdown/util.py
index a8db7bd..5dc144a 100644
--- a/markdown/util.py
+++ b/markdown/util.py
@@ -26,6 +26,7 @@ from functools import wraps
 import warnings
 import xml.etree.ElementTree
 from .pep562 import Pep562
+import inspect

 try:
     from importlib import metadata
@@ -156,6 +157,10 @@ def code_escape(text):
     return text

+def nearing_recursion_limit():
+    """Return true if current stack depth is withing 100 of maximum limit."""
+    return sys.getrecursionlimit() - len(inspect.stack(0)) < 100
+
 """
 MISC AUXILIARY CLASSES
 =============================================================================

Basically, when we detect a blockquote, we check the current stack depth and if it is within 100 of the limit, we consider the block of text to not be a blockquote. That results in the text being considered a paragraph. The result would look something like this significantly truncated example:

>>>>

And now for the problem. Here are the terrible performance numbers:
>>> import timeit
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 100)).autorange(); dur / num
1.2720757999999819
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 200)).autorange(); dur / num
4.7593806999999515
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 300)).autorange(); dur / num
10.332307999999955

Without the fix I'm getting the following for comparison:
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 100)).autorange(); dur / num
0.0034761609999999976
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 200)).autorange(); dur / num
0.012274940000000002
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 300)).autorange(); dur / num
0.03034806000000003

While the "fix" works, performance is so terrible that it's not a reasonable solution. Apparently the call to inspect.stack is the issue. This Stackoverflow answer provides a bunch or alternate substitutions to calling inspect.stack, but I'm not sure we want to use any of those. Any suggestions for alternate solutions are welcome.

As an aside, an easy way to test this might be to decrease the recursion limit for a single test case. Perhaps using something like this context manager:
class recursionlimit:
    def __init__(self, limit):
        self.limit = limit
        self.old_limit = sys.getrecursionlimit()

    def __enter__(self):
        sys.setrecursionlimit(self.limit)

    def __exit__(self, type, value, tb):
        sys.setrecursionlimit(self.old_limit)

with recursionlimit(20):
    # test code here

@waylan out of curiosity, how fast can you hit the recursion limit if you don't use inspect but "test" it by intentionally causing fast recursion? Here's some preliminary testing I did and it seems to be reasonably fast in comparison:
def a():
 b()

def b():
 a()

def recurse():
 try:
  a()
 except:
  pass

>>> num, dur = timeit.Timer(lambda: [recurse() for _ in range(100)]).autorange(); dur / num
0.01735958499999981
>>> num, dur = timeit.Timer(lambda: [recurse() for _ in range(100)]).autorange(); dur / num
0.021130189999999514
>>> num, dur = timeit.Timer(lambda: [recurse() for _ in range(200)]).autorange(); dur / num
0.034520720000000435
>>> num, dur = timeit.Timer(lambda: [recurse() for _ in range(300)]).autorange(); dur / num
0.05057582000000025

@MythicManiac I'm not sure I understand the relevance of that.

Basically the solution I came up with is using the recursion limit as a sort of nesting depth limit. But given it is so expensive to determine where we are in the stack, its not practical. We might need to actually set a nesting depth and then track the levels to cut it off. I was hoping to avoid that though.

So here's a bit more elaborate snippet:
def recurse(current_depth):
    try:
        return recurse(current_depth + 1)
    except:
        return current_depth

def get_remaining_recursion():
  return recurse(0)

import timeit
num, dur = timeit.Timer(lambda: [get_remaining_recursion() for _ in range(100)]).autorange()
print(dur / num)
num, dur = timeit.Timer(lambda: [get_remaining_recursion() for _ in range(100)]).autorange()
print(dur / num)
num, dur = timeit.Timer(lambda: [get_remaining_recursion() for _ in range(200)]).autorange()
print(dur / num)
num, dur = timeit.Timer(lambda: [get_remaining_recursion() for _ in range(300)]).autorange()
print(dur / num)

This outputs
0.01548541
0.01447132
0.026666679999999988
0.04390339999999995

I just tried this:
def _get_stack_depth(size=2):
    """Get stack size for caller's frame.
    """
    frame = sys._getframe(size)

    for size in count(size):
        frame = frame.f_back
        if not frame:
            return size

def nearing_recursion_limit():
    """Return true if current stack depth is withing 100 of maximum limit."""
    return sys.getrecursionlimit() - _get_stack_depth() < 100

Which results in these times for me:
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 100)).autorange(); dur / num
0.0037207730000000084
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 200)).autorange(); dur / num
0.012463729999999984
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 300)).autorange(); dur / num
0.030507339999999772

Given my unmodified times, that doesn't seem to add any noticeable time at all.

However, if I do this:
def get_remaining_stack_frames(depth=0):
    """ Return remaining number of stack frames before limit it hit."""
    try:
        return get_remaining_stack_frames(depth + 1)
    except RecursionError:
        return depth

def nearing_recursion_limit():
    """Return true if current stack depth is withing 100 of maximum limit."""
    return get_remaining_stack_frames < 100

then my times are:
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 100)).autorange(); dur / num
0.0034912980000000003
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 200)).autorange(); dur / num
0.012532085000000137
>>> num, dur = timeit.Timer(lambda: markdown.markdown(">" * 300)).autorange(); dur / num
0.030452539999999573

The difference is negligible. However, the example which counts up (to the max) is slower (relatively) that the example which counts down (the stack) for smaller size documents. As smaller documents are the norm, I think that is probably the more sensible way to go.


Question about style of file placement.

I noticed markdown.ex was not nested inside it's respective folder while on the other hand forums.ex was. I'm thinking that was possibly a mistake ("or not IDK"), but it tickled my brain and I wanted to understand your reasoning for doing it one way or the other.

I think forums.ex should not be nested inside the forums folder but that's how Phoenix generators place that file. I left it like that to stick with their conventions, but in general don't prefer the style :)


Feature request: expose Wagtail’s fonts as CSS variables

Is your proposal related to a problem?
In #7724, we changed Wagtail’s brand font to a system font stack. In #8043, we introduced a system font stack for monospace text as well. System font stacks are much harder to reuse consistently due to their length, so it’d be nice for us to simplify this.
Describe the solution you'd like
We should make both font stacks available as CSS variables.
This should be done towards the top of the :root element here: https://github.com/wagtail/wagtail/blob/86aa38329d8f4cfe71673ee15a5951d52e529e6c/client/scss/elements/_root.scss#L1
Something like:
:root {
  // Make Wagtail’s fonts available for reuse for customisations.
  --w-font-sans: $font-sans;
  --w-font-mono: $font-mono;

I haven’t tried this so it would need testing to make sure this works as expected.
Describe alternatives you've considered
We could also make both fonts reusable with utility classes only (w-font-sans should already work, w-font-mono would need to be defined somewhere before it can be used). I think this can be done in addition, but CSS variables would be the nicest for the majority of existing code using Open Sans to move over to our new font stack.
Additional context
I’m marking this for the Wagtail 3.0 release, to be added to the stable/3.x branch. Technically this would be a feature, but I believe it’s worth bypassing our policy in this case because it would greatly simplify migration for people using the old font. There’s also very little risk of this causing any regressions. If we don’t think this is worth the trouble to backport, we could also simply recommend people copy-paste the :root declarations if they need them, and add a comment to their code to remove those declarations once they upgrade to the next release.
It’d be nice to document this properly at the same time but in the interest of speed I’d be happy for us to make the change without docs for now and add this further down the line.

https://github.com/wagtail/wagtail/pull/8409 created

Smaller scoped version for 3.0 only @thibaudcolas  https://github.com/wagtail/wagtail/pull/8415

Based on recent UI team discussions, we’re going to be implementing this with CSS variables as described above. There are two open questions before we proceed with LB’s #8409:

Are we ok with w- as the prefix for our CSS variables, to indicate they are related to Wagtail? This would be used for font-related variables, but also ideally all future CSS variables we’d introduce. There aren’t necessarily big concerns of conflicts, but at least it makes it very straightforward to search for those variables in code or on the internet.
For each value (say the monospace font) – do we only want a single variable (--w-font-mono) that’s set to our "official" value by default, and can be customised – or do we want two variables, one "immutable" for our official value, and one that can be customised? If we went for two, any thoughts on naming? This kind of pattern would be relevant across most of our customisable tokens I believe, so if we went for it we’d ideally choose a naming convention that works for all kinds of CSS variables, not specific to fonts

I personally do not see the benefit compared to the added complexity of the 'official' value.
However, we should allow for malformed custom properties to be set and always use a default (in SCSS/Tailwind). Note: I am not sure how this would adapt to Tailwind, and at the moment we will likely need to duplicate a bit across both which is not ideal.
Example A - using a default value that itself is not set to a variable
:root {
  --w-focus-outline: 3px solid supergreen;
}

$focus-outline: var(--w-focus-outline, 3px solid supergreen);

.some-element:focus-within {
   outline: $focus-outline;
}

Here we are setting up an 'internal' scss variable which always uses the custom property with a default, so if --w-focus-outline is set to a malformed value (e.g. 3parx solix) CSS will usually fallback to the default.
Example B - using a default value that also needs to be set to an internal scss variable

$focus-outline-default: 3px solid supergreen;

:root {
  --w-focus-outline: $focus-outline-default;
}

$focus-outline: var(--w-focus-outline, $focus-outline-default);

.some-element:focus-within {
   outline: $focus-outline;
}

Here we have two SCSS 'internal' variables, one that should be used all over the place; the $focus-outline and one that should never be used, except for setting up the default $focus-outline-default. This avoids a small amount of duplication, but we could probably just do this in a mixin (similar to how colours are done now).
The downside here is it could be easy to 'leak' the wrong focus-outline into styles (e.g. a new contributor may accidentally use $focus-outline-default somewhere and suddenly the custom property does not work for that case.

On the two examples, I would have expected us to go for something in-between that uses Tailwind’s theme function:
:root {
  --w-focus-outline: theme('theme.focusOutline');
}

$focus-outline: var(--w-focus-outline);

.some-element:focus-within {
   outline: $focus-outline;
}

Ultimately I’d like us to consider generating the CSS variables declarations (--w-color: theme('colors.red') with Tailwind, so the available variables are defined in JS code. This will allow us to auto-generate docs for this, making sure the documentation and the code are always in sync.


DOC: How to use RemoteRunnable

Checklist

[X] I added a very descriptive title to this issue.
[X] I included a link to the documentation page I am referring to (if applicable).

Issue with current documentation:
from langchain_community.chat_models import ChatOpenAI
from langchain_core.prompts import ChatPromptTemplate, MessagesPlaceholder

_prompt = ChatPromptTemplate.from_messages(
    [
        ("system", "You are a helpful, professional assistant named Cob."),
        MessagesPlaceholder("messages"),
    ]
)
_model = ChatOpenAI()

# if you update this, you MUST also update ../pyproject.toml
# with the new `tool.langserve.export_attr`
chain = _prompt | _model

add_routes(app, chain , path="/chat", playground_type="chat")

<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="UTF-8">
  <meta name="viewport" content="width=device-width, initial-scale=1.0">
  <title>LangChain Example</title>
</head>
<body>
  <script type="module">
    import { RemoteRunnable  } from "https://esm.sh/@langchain/core/runnables/remote";

      const chain = new RemoteRunnable({
          url: `http://localhost:3000/chat/`,
      });
      const result = await chain.invoke("Hi");

      print(result)

  </script>
</body>
</html>

Idea or request for content:
I have absolutely no idea how to use the parameter passing format of RemoteRunnable and invoke in JS
I mean, when I use add_routes to create a chain, how do I pass in the parameter format corresponding to the chain in invoke? I don’t know which parameters to pass in and the format to pass in.
Is there any documentation?

Same input as the chain takes

llm = new RemoteRunnable({
		url: "http://localhost:8040/openapi",
		config: {
			headers: {'X-Token': `Bearer ${token}`},
		}
	});

对于上述代码，在使用await llm.invoke(text)时，请求头没有X-Token


Do not find the “requirement.txt”

Hi,thanks for sharing your code. However, while I try to run the code, the file "requirement.txt" is missing.could you help upload it? Thank you.

@ChangyuanZhou Hi, I just uploaded the requirement.txt. It was ignored in the .gitignore file earlier.