Inlining fonts behind HTTP proxy

🐞 Bug report
Command (mark with an x)

[ ] new
[x] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
No
Description
Since 11.0.0 there is the new feature to inline fonts. The problem is that it does not work behind HTTP proxy. We have correctly set HTTP_PROXY/http_proxy and NO_PROXY/no_proxy environment variables, but it is ignored.
🔬 Minimal Reproduction
There is no support for proxy: https://github.com/angular/angular-cli/blob/871dd6a434fdd9674905bde307b5ad79d5d59603/packages/angular_devkit/build_angular/src/utils/index-file/inline-fonts.ts#L107
There is not really a reproduction scenario. It fails only when running behind HTTP proxy.
🔥 Exception or Error

✖ Index html generation failed.
Inlining of fonts failed. An error has occurred while retrieving https://fonts.googleapis.com/icon?family=Material+Icons over the internet.
connect ETIMEDOUT <IP_ADDRESS>:443

🌍 Your Environment

Angular CLI: 11.0.1
Node: 14.15.0
OS: linux x64

Angular:
...
Ivy Workspace:

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.1100.1 (cli-only)
@angular-devkit/core         11.0.1 (cli-only)
@angular-devkit/schematics   11.0.1 (cli-only)
@schematics/angular          11.0.1 (cli-only)
@schematics/update           0.1100.1 (cli-only)

Anything else relevant?

This is affecting me as well.  Our build servers don't have access outside our firewalls.  Is there a way to disable inline fonts?

On how to disable it, see https://angular.io/guide/workspace-config#optimization-and-source-map-configuration

Had the same issue behind a corporate proxy. Thought of using an index.html referencing the google fonts in dev config and replacing the index.html with a prod version referencing a local copy of the fonts that would be made in the build procedure of my docker image in my pipeline. However, this would have required additional scripting for the woff2 files referenced in the css files. I ended up using fontsource. This way ng build does not need to reach fonts.googleapis.com.

Had the same issue behind a corporate proxy. Thought of using an index.html referencing the google fonts in dev config and replacing the index.html with a prod version referencing a local copy of the fonts that would be made in the build procedure of my docker image in my pipeline. However, this would have required additional scripting for the woff2 files referenced in the css files. I ended up using fontsource. This way ng build does not need to reach fonts.googleapis.com.

Had the same issue behind a corporate proxy. Thought of using an index.html referencing the google fonts in dev config and replacing the index.html with a prod version referencing a local copy of the fonts that would be made in the build procedure of my docker image in my pipeline. However, this would have required additional scripting for the woff2 files referenced in the css files. I ended up using fontsource. This way ng build does not need to reach fonts.googleapis.com.

Why not just disable font optimization?

Had the same issue behind a corporate proxy. Thought of using an index.html referencing the google fonts in dev config and replacing the index.html with a prod version referencing a local copy of the fonts that would be made in the build procedure of my docker image in my pipeline. However, this would have required additional scripting for the woff2 files referenced in the css files. I ended up using fontsource. This way ng build does not need to reach fonts.googleapis.com.

Why not just disable font optimization?

I am facing the same problem. While changing the font optimization under angular.json for production from
"optimization" : true
to
"optimization": {
    "font": false
}

gives me schema validation failed error.
The suggestion in the link: https://angular.io/guide/workspace-config#optimization-and-source-map-configuration
does not work and seems to have not been updated.
Is there any fix for this problem ?

I am facing the same problem. While changing the font optimization under angular.json for production from
"optimization" : true
to
"optimization": {
    "font": false
}

gives me schema validation failed error.
The suggestion in the link: https://angular.io/guide/workspace-config#optimization-and-source-map-configuration
does not work and seems to have not been updated.
Is there any fix for this problem ?

If I set ONLY fonts optimization to false
"optimization": {
    "fonts": false
}

are scripts and styles property set to false or are they still true by default ?
I am not able to find any info on this. Could someone help ?

If I set ONLY fonts optimization to false
"optimization": {
    "fonts": false
}

are scripts and styles property set to false or are they still true by default ?
I am not able to find any info on this. Could someone help ?

@austi10,
They are true, by default. See https://angular.io/guide/workspace-config#optimization-configuration

@austi10,
They are true, by default. See https://angular.io/guide/workspace-config#optimization-configuration

Yeah, if you think about the code behind that property, the type is likely a simple boolean or some optimization option interface.  Defining it as a simple boolean is the equivalent of setting it with an option that has all properties as true.  So if you want to just turn off one, provide an object that has all true, but the one(s) you want turned off as false.  Seems pretty straightforward.

Yeah, if you think about the code behind that property, the type is likely a simple boolean or some optimization option interface.  Defining it as a simple boolean is the equivalent of setting it with an option that has all properties as true.  So if you want to just turn off one, provide an object that has all true, but the one(s) you want turned off as false.  Seems pretty straightforward.

This is the new error I get with the patch. I have tried with hardcoding my proxy credentials into %HTTP_PROXY% too, but still no luck.
× Index html generation failed.
Inlining of fonts failed. An error has occurred while retrieving https://fonts.googleapis.com/css?family=Roboto:300,400,500&display=swap over the internet.
Socket is closed

🌍 Your Environment
Angular CLI: 11.1.2
Node: 14.15.1
OS: win32 x64

Angular: 11.1.1
...
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1101.2
@angular-devkit/build-angular   0.1101.2
@angular-devkit/core            11.1.2
@angular-devkit/schematics      11.1.2
@angular/cli                    11.1.2
@angular/flex-layout            11.0.0-beta.33
@schematics/angular             11.1.2
@schematics/update              0.1101.2
rxjs                            6.6.3
typescript                      4.0.5

This is the new error I get with the patch. I have tried with hardcoding my proxy credentials into %HTTP_PROXY% too, but still no luck.
× Index html generation failed.
Inlining of fonts failed. An error has occurred while retrieving https://fonts.googleapis.com/css?family=Roboto:300,400,500&display=swap over the internet.
Socket is closed

🌍 Your Environment
Angular CLI: 11.1.2
Node: 14.15.1
OS: win32 x64

Angular: 11.1.1
...
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1101.2
@angular-devkit/build-angular   0.1101.2
@angular-devkit/core            11.1.2
@angular-devkit/schematics      11.1.2
@angular/cli                    11.1.2
@angular/flex-layout            11.0.0-beta.33
@schematics/angular             11.1.2
@schematics/update              0.1101.2
rxjs                            6.6.3
typescript                      4.0.5

@alexaka1, are you using a rotating proxy?, If so, can you try using Node.js 12?
Note: only the HTTPS_PROXY environment variable.

@alexaka1, are you using a rotating proxy?, If so, can you try using Node.js 12?
Note: only the HTTPS_PROXY environment variable.

@alan-agius4 No we don't have rotating proxy, but still I have installed nvm4w, and got Node 12.20.1 installed, and building with that no cmd error is given, however, the final index.html doesn't have the font css, instead it has this inside


Update auto_encoder.py

Tensorflow keras has changed names to MeanSquaredError. Please accept for the next users.
Could not find any issues on that task. If not accepted, please add a info on the TF version number.
I am using TF 2.15.0
All Submissions Basics:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully ran tests with your changes locally?

New Model Submissions:
Updated AutoEncoder

I have seen the l2 import had to be changed also to L2 (line 24 + 29...)

Hi there, the auto tests failed :) feel free to fix them first. thx


[Triangulation] Determine current location from dead reckoning

Select beacon (last known location)
Enter distance traveled from the beacon (straight line)
Mark direction to the beacon
Location is calculated

This might not be accurate, as judging distance traveled is difficult. The most accurate measure for this is pace counting with a known pace length.

The most accurate measure for this is pace counting with a known pace length.

Agree. If this is measured it would be a good start for the calculation.
Average speed and time travelled might also be a reasonably good measure. This is basically the input you'd use for planning the hike, so one may have at least an idea about it. Naismith's rule comes to mind: Time taken = 4km/h + 1min per 10m climbed. Where the magic 4 is a bit of a personal variable.
(Allowing a base on speed/time would also allow to include other activities then walking.)

The most accurate measure for this is pace counting with a known pace length.

Agree. If this is measured it would be a good start for the calculation.
Average speed and time travelled might also be a reasonably good measure. This is basically the input you'd use for planning the hike, so one may have at least an idea about it. Naismith's rule comes to mind: Time taken = 4km/h + 1min per 10m climbed. Where the magic 4 is a bit of a personal variable.
(Allowing a base on speed/time would also allow to include other activities then walking.)

isRelatedTo: #142

Closed for the same reason as #142


Can't load default.vert

I cloned repository with LFS, opened package in Xcode 15.2, run TestApp1 and got runtime error:
[DKGame:debug:E8944C00] Notification: name = NSApplicationWillFinishLaunchingNotification, object = Optional(<NSApplication: 0x12861ba50>), userInfo = nil [DKGame:error:E8944C00] SPIRV-Cross: Invalid SPIRV format. [DKGame:error:E8944C00] Shader.compile() error: "Invalid SPIRV format.", spvc_result(rawValue: -1) [DKGame:error:E8944C00] URL(file:///Users/fedor/Library/Developer/Xcode/DerivedData/SwiftVVD-ayyxphwodmnkithitdfowuberhio/Build/Products/Debug/DKGame_DKGUI.bundle/Contents/Resources/SPIRV/default.vert.spv) error: LoadError(message: "Failed to load shader: default.vert") DKGUI/GraphicsContext+Pipeline.swift:452: Fatal error: GraphicsPipelineStates.sharedInstance(commandQueue:) Error: LoadError(message: "Failed to load shader: default.vert")
I've tried on MacBook Pro with M1 and Sonoma 14.5

Some of the shader files seems to be broken.
The shader files are located in the 'Sources/DKGUI/Resources/Shaders/SPIRV' folder.
Please try downloading them again and if you are still having issues, you may want to download the files directly from your web browser and compare them.  github files
(check the file hashes if possible, or at least compare the file sizes)
If the shader file is not the problem, it could be a packaging issue in Xcode.
I recommend running it from the terminal directly.  In the root folder of this project, run the following command
swift run TestApp1

Oh... I see. Shader files was broken.
I've downloaded again and it works fine. Thank you!

Oh... I see. Shader files was broken.
I've downloaded again and it works fine. Thank you!


{"code":401,"reason":"Unauthorized","message":"Authentication Failed"}

Does not work anymore:

nissan.0
2021-09-19 21:30:28.819
error
{"error_description":"Invalid Token ID","error":"invalid_grant"}

nissan.0
2021-09-19 21:30:28.818
error
Error: Request failed with status code 400

nissan.0
2021-09-19 21:30:28.812
error
Access token

nissan.0
2021-09-19 21:30:28.295
error
{"code":401,"reason":"Unauthorized","message":"Authentication Failed"}

nissan.0
2021-09-19 21:30:28.294
error
Error: Request failed with status code 401

nissan.0
2021-09-19 21:30:28.293
error
Post JWT

Was this working for you? Do you really using the Nissan Connect App and not Nissan Connect EV

I'm using the "Nissan EV" App. At the moment i use fhem with carwings to get the values to iobroker. A working iobroker adapter would be nice.

0.0.7 unterstützt auch Nissan EV für Modelle vor 2019


user reporting notes disappear with new build

User reported that after upgrading to the nightly 1.0.22 beta builds, their notes for old transactions disappeared. New notes are there though.

In the new release, notes are saved in a different place and by a different key. This new place is in mwc-qt-wallet/context/notes.dat. Previously notes were saved in the app context in the file mwc-qt-wallet/context/context.dat. So the notes from older versions of Qt wallet needed to be migrated to their new format and location.
To test:
Open and older version of Qt wallet and add some transaction and output notes, then exit the wallet.
Now open the same wallet instance with the new version of Qt wallet. The notes should be migrated and you should be able to see them now using the new Qt wallet version.
The transaction notes are actually migrated when you select the transaction screen. So if you open Qt wallet and don't go to that screen and then close the wallet again, you should not lose your notes. The next time you open the wallet and go to the transaction screen you should see that your notes have been migrated and appear ok.
The output note migration is similar. It doesn't happen until you open the output window. If you open Qt wallet and never go to the output window, you will not lose your notes. When you finally do open Qt wallet and go to the output window, you will see your notes.

I installed version 1.0.21, made multiple accounts, processed some transactions and left transaction and output notes. I then upgraded the wallet to v1.0.22 and all of the notes were still there.


Update waitForAppliedIndex to always be > than the current snapshot index

This deals with the edge case where a snapshot could be provided by a peer
that is much higher than the current written WAL index.  Also provide
additional logging in the case of a Docker test failure decrypting the
snapshot, which has been difficult to reproduce.
Signed-off-by: cyli<EMAIL_ADDRESS>cc @aaronlehmann

Current coverage is 55.15% (diff: 85.00%)

Merging #1798 into master will increase coverage by 0.02%

@@             master      #1798   diff @@
==========================================
  Files           102        102
  Lines         16931      16940     +9
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           9334       9344    +10
+ Misses         6437       6436     -1
  Partials       1160       1160

Powered by Codecov. Last update c8916c5...8e3694f

LGTM


Add option for timeout and custom procdump

DO NOT MERGE, this is for experiments only
Description
VSTEST_TESTHOST_TIMEOUT to set how many ms we will wait till killing testhost
VSTEST_DUMP_PROCDUMPARGS  to provide custom procdump parameters
Related issue
Kindly link any related issues. E.g. Fixes #xyz.

@nohwnd Is there anything specific you want to test/handle before turning this into mergeable PR?

:shipit:

Closed in favor of #3466


Need ImageDeleted Event

We need a new event ---- 'ImageDeleted'.
I need to "DELETE" the image url by axios when the picture was deleted.

A new event has been added as of 2.10.0. This new event is imageRemoved.
I plan to add more information to the Docs soon.
However, I have created an example on CodeSandbox.
Closes #175

I was trying this solution, but I am getting error "document is not defined"  I am sure this is popping up because of quill.  How do i import Quill properly? I have been trying this from last two days.
Your sandbox code doesn't have any plugin import for Quill not sure why it is not throwing error.  I am trying to work this on Nuxt
@davidroyer any help is appreciated. It's been two days I am struggling with this.


Fix running out of memory when creating the compilation database

When I tried to use the extension against a larger project (Unreal Engine 4), I found that it was running out of memory (devenv.exe is 32 bit). Seems like an easy fix.

Hi @TTimo, thanks for looking into this! Yes, it really doesn't make any sense to keep all that data in memory. I already reviewed your changes and ran some tests on the new plugin build and it's looking really good. I'll merge this PR now and we'll probably have a new plugin release this week!


Discordant number of Exons

I was checking the exons of the gene TTC28 on GRCh37, pyensembl (0.6.8) found 39 for that gene but other sources (also based on GRCh37) states that there are only 23. Could you explain why there is such a difference?
The link of the "other sources" are:
http://www.ensembl.org/Homo_sapiens/Transcript/Summary?db=core;g=ENSG00000100154;r=22:27978014-28679865;t=ENST00000397906
http://www.nextprot.org/db/entry/NX_Q96AY4/exons
http://www.omim.org/entry/615098
http://www.rcsb.org/pdb/gene/TTC28?chromosome=chr22&range=27999260
The code I have used is the following:
 >>> data = EnsemblRelease(75, auto_download=True)
 >>> data
 EnsemblRelease(release=75, species=homo_sapiens, genome=GRCh37)
 >>> a = data.exon_ids_of_gene_name('TTC28')
 INFO:root:Cached file Homo_sapiens.GRCh37.75.gtf from URL ftp://ftp.ensembl.org/pub/release-75/gtf/homo_sapiens/Homo_sapiens.GRCh37.75.gtf.gz
 >>> a
['ENSE00001556591', 'ENSE00001551541', 'ENSE00003640178', 'ENSE00003492820', 'ENSE00001492788', 'ENSE00001492787', 'ENSE00001231767', 'ENSE00000879682', 'ENSE00001173060', 'ENSE00001173052', 'ENSE00001173076', 'ENSE00000651956', 'ENSE00001297696', 'ENSE00000879679', 'ENSE00000651949', 'ENSE00000651948', 'ENSE00000651947', 'ENSE00003685416', 'ENSE00000651945', 'ENSE00001321146', 'ENSE00000651944', 'ENSE00003566454', 'ENSE00001559219', 'ENSE00001726746', 'ENSE00001625734', 'ENSE00001852426', 'ENSE00003550588', 'ENSE00001840432', 'ENSE00001878331', 'ENSE00003602135', 'ENSE00001859545', 'ENSE00001757870', 'ENSE00001654926', 'ENSE00001862939', 'ENSE00003566593', 'ENSE00003667769', 'ENSE00001817484', 'ENSE00001938107', 'ENSE00001917942']
 >>> len(a)
 39

Hey Alec,
You're seeing all the distinct exon IDs across all the transcripts of "TTC28". The primary transcript from ensembl75 has 23 exons, and the others are either incomplete or are retained processed transcripts.
In [1]: for transcript in ensembl_grch37.genes_by_name("TTC28")[0].transcripts:
   ...:     print(transcript.name, len(transcript.exons))
   ...:
TTC28-001 23
TTC28-005 6
TTC28-006 3
TTC28-004 3
TTC28-003 3
TTC28-007 4
TTC28-002 2

Clear! Thank you.


⚠ ClamAV has degraded performance

In 2c7812d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 559 ms

Resolved: ClamAV performance has improved in de0cfc7.


Update dependency @types/jasmine to v3.10.7

This PR contains the following updates:

Package
Type
Update
Change

@types/jasmine
devDependencies
patch
3.10.6 -> 3.10.7

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (3.10.7). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Ran out of memory try to allocate 2.20GiB, but I have a 8GB GTX1080. style2paints45beta1214B

using style2paints45beta1214B.
OS: Win7x64
CPU: i7-4790K
MEM: DDR3 16G
Graphic Card:  Gigabyte GeForce® GTX 1080 G1 Gaming 8G (Driver version: v442.19)
here is a screenshot.

Tell me if you need more debug log or whatever.

It seems that you have more than one gpu.  If that is the case,  maybe you can try disabling other gpus and enforce s2p to use your 1080.
---Original---
From: "Cat<EMAIL_ADDRESS>Date: Tue, Feb 18, 2020 18:17 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [lllyasviel/style2paints] Ran out of memory try to allocate 2.20GiB, but I have a 8GB GTX1080. style2paints45beta1214B (#136)
using style2paints45beta1214B.
OS: Win7x64
CPU: i7-4790k
GPU: Nivdia GTX1080 with 8GB (v442.19)
here is a screenshot.
Tell me if you need more debug log or whatever.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It seems that you have more than one gpu.  If that is the case,  maybe you can try disabling other gpus and enforce s2p to use your 1080.
…
---Original--- From: "Cat<EMAIL_ADDRESS>Date: Tue, Feb 18, 2020 18:17 PM To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [lllyasviel/style2paints] Ran out of memory try to allocate 2.20GiB, but I have a 8GB GTX1080. style2paints45beta1214B (#136) using style2paints45beta1214B. OS: Win7x64 CPU: i7-4790k GPU: Nivdia GTX1080 with 8GB (v442.19) here is a screenshot. Tell me if you need more debug log or whatever. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

I don't get it.
I only have one Graphic Card and it is Gigabyte GeForce® GTX 1080 G1 Gaming 8G. I don't know the GPU number. The Core Number in Specification said CUDA® Cores was 2560 (A Nvidia Thing).
What should I next?

Is it possible that you have an integrated graphics card?
Since you have an Nvidia graphics card, you should be able to open the "Nvidia Control Panel" and check which graphic card gets used by default.

I'm using an old i7-4790K, not brandnew F-series, and it has an integrated GPU.
In my NVIDIA Control Panel(Version 8.1.940.0), there isn't "Prefered graphics processor" section. I'm using the default setting of NVIDIA Control Panel.

When you right click a program, do you have a Run with graphics processor option? From there, you should also be able to change the default graphics processor.

Are you running other GPU software like photoshop or unity or dynamic wallpaper or music player or something else? Maybe you could try close all other software and run s2p again.

When you right click a program, do you have a Run with graphics processor option? From there, you should also be able to change the default graphics processor.

I'm using desktop, not laptop, so there isn't such a thing like Run with graphics processor

Are you running other GPU software like photoshop or unity or dynamic wallpaper or music player or something else? Maybe you could try close all other software and run s2p again.

Nope. Just s2p only.
This Graphic Card has 8G memory but the debug log says it failed on allocate 3GB memory.
Normally how much memory does s2p use?

Are you running other GPU software like photoshop or unity or dynamic wallpaper or music player or something else? Maybe you could try close all other software and run s2p again.

Oh. Sorry I just realized I ran Chrome at that moment(with hardware acceleration off).
I reinstalled my computer and didn't reinstall Chrome (Use Firefox instead), not it's fine.
Can't reproduce now.


Fix table formatting

Fix table formatting

Codecov Report

Merging #275 (6654c59) into master (6ae1669) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #275   +/-   ##
=========================================
  Coverage     56.30%   56.30%
  Complexity      765      765
=========================================
  Files           161      161
  Lines          3307     3307
  Branches        363      363
=========================================
  Hits           1862     1862
  Misses         1319     1319
  Partials        126      126

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 6ae1669...6654c59. Read the comment docs.


User Defined Function Restrictions not clear

What specifically does this statement at https://docs.microsoft.com/en-us/azure/azure-resource-manager/resource-group-authoring-templates#functions mean?
"The function can't access template parameters. That is, the parameters function is restricted to function parameters."
Perhaps a rewording will help convey the meaning better.

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

ID: 0eb51e04-9b4e-c22d-be8c-4ddf4a59a100
Version Independent ID: 2390761b-e1b9-21d1-ea81-7b8d669d0cf2
Content: Azure Resource Manager template structure and syntax
Content Source: articles/azure-resource-manager/resource-group-authoring-templates.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

@manjnath Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

@manjnath - thanks for this feedback. I am clarifying the text to indicate that you can only use parameters declared in the function.

Update has been published.
#please-close


Don't plot all estimates

Don't plot forecasts.

No because nightmarishly plots get made and saved in inividual epinow calls and in regional_secondary calls. I've just pushed a commit so this should now happen.


upgrade to<EMAIL_ADDRESS>

This would finally get rid of entire lodash.js being included into webpack and maybe browserify.
related casesandberg/reactcss#28

Done<EMAIL_ADDRESS>

Hey @casesandberg
Sorry for not responding earlier. This works, however I'm still not able to get rid of the single lodash dependency from my build.
The problem seems to be in modules/react-material-design/src/components/Link.js and modules/react-material-design/src/components/Tabs.js
Are these maintained by you?

I can corroborate, here's webpack-analyse output for the lodash/index module: https://www.dropbox.com/s/jidzk0bjuuysbm1/Screenshot 2016-03-29 12.10.36.png?dl=0


Hyper parameter tuning with optuna

https://optuna.org/

The reason I am raising the issue is that I realized that there might be much room to turn hyper parameters. In particular, I realized that the increasing number of epochs from 50 to 200 dramatically improves the quality of synthetic voice and also it achieves better validation loss on the NIT-song070 dataset.

Note to self: I should definitely work on this

I've done the first cut. It works pretty nicely.

https://medium.com/optuna/easy-hyperparameter-management-with-hydra-mlflow-and-optuna-783730700e7d

Fixed by #109

https://r9y9.github.io/nnsvs/optuna.html


[CP] [pkg/dart2native] Handle cases where linker-signed is not an option.

Commit(s) to merge
4e520ec7
Target
stable
Issue Description
As seen on #49010, attempting to compile dart code to a standalone executable on older versions of macOS (<= 11.0) fails when attempting to override the original signature for dartaotruntime. This is because the codesign executable does not support the linker-signed option, which we use to add a signature that can be overwritten by the developer without requiring the -f/--force flag be passed to codesign.
This wasn't seen in testing because only X64 runs macOS <= 11.0, and tests are run using a locally built unsigned SDK, and the code for signing standalone executables only executes if dartaotruntime is signed.
What is the fix
The basic fix is to not fail immediately when codesign fails when passed the linker-signed option. Instead, we then fallback to overriding the signature without that option, and only fail at this step if that second attempt also fails.
In the full fix, since linker-signed signatures still require being force overridden for versions of macOS < 11.0 even when that option is recognized, we only perform the linker-signed attempt on macOS >= 11.0, and also print a warning if that fails and we fall back to the non-linker-signed attempt.
The commit also adds a test for signing dartaotruntime in the locally built SDK and then attempting to compile a standalone executable with it, triggering the standalone executable signing behavior even on X64.
Why cherry-pick
Currently Dart users that cannot upgrade their development platforms past macOS 10.15.x cannot create standalone executables.
Risk
low
Issue link(s)
https://github.com/dart-lang/sdk/issues/49010
Extra Info
No response

/cc @mkustermann

cc @athomas

SGTM, I think we should pick this

This change is fairly low risk and addresses a P1 issue with codesigning on macOS. The Fix essentially checks for the macOS version and uses the 'linker-signed' option only when the version is > 11.0. There is a test case associated with the change. I would recommend approving this cherry pick.

Merged to stable in edf0ab222395c4b13c3e52a18900dd62702b639f (2.17.1).


WaitForLoadStateAsync (CustomNetworkIdle)

Hi,
Feature request:
An overload for the WaitForLoadStateAsync where we can provide an options object with some definitions for when the network should be considered idle.
This is similar to an existing feature in puppeteer (networkidle2), but I would like the ability to provide my custom definition of what I define idle.
Right now I'm working with a site that is always pinging servers for tracking/analytics purposes every few seconds so this feature could account for those events.

I would not use any variation of network idle. As you already figured out, this signal is pretty much useless in the wild, it only works in a controlled environment of your tests. I can't imagine the terms you will be expressing the CustomNetworkIdle in, so I would recommend using WaitForSelectorAsync or WaitForFunctionAsync.

The problem with those methods is that it's quite difficult, if not impossible, to use them across iframes when you don't have a way to identify each one of them.
Currently, I need to wait for a page to completely load so I can grab a screenshot. The main page creates iframes (that are created based on network requests, so the order in the source is always changing) and each iframe does something similar (more iframes inside iframes).
Main page also pings servers about every second. NetworkIdle never fires.
I was thinking in something like CustomNetworkIdle { int NetworkConnections, int Timespan }, so I can decide if I want to emulate networkidle2 I can use new CustomNetworkIdle(2, 500).

A custom network idle is not supported, we recommend waitForSelector to ensure elements are there or waitForResponse to ensure a http resource got loaded.
If you are still interested in a custom network idle timeout, please open a feature request upstream: https://github.com/microsoft/playwright


[doc] Clarify that AnimationNode virtual methods need to be implemented rather than called directly

Fixes #62379

Hey, this would be nice to have backported (is this the correct term?). Godot 3.x suffers from the same lack of clarity, but it's actually worse since these methods don't even have an underscore.
If backported, I think this would also fix #35272.
My only feedback is that some methods add a bit more information about why they should be overriden:

(used by editors inheriting from [AnimationRootNode])
Only useful when inheriting [AnimationRootNode]

But other methods don't. I think all affected methods should have a uniform and explicit "Only used for custom AnimationNodes". To me, the current wording implies that you can implement these methods for core AnimationNodes like AnimationNodeStateMachine but I don't think that's how they are meant to be used.

Thank you @Hiiamwilliam. I made it explicit that these are used when inheriting AnimationRootNode.
Once the wording is settled on and this is merged, I will create a parallel PR for 3.x.
(This PR now conflicts with #64333 so one will have to be fixed depending on which is merged first.)

Looks good to me, thanks! 🎉 Merged.


UnboundLocalError at first try

I cant figure out the error, but the test script throws this.
fresh install
from __future__ import print_function
import time
import pychromecast
pychromecast.get_chromecasts_as_dict().keys()

i get
>>> from __future__ import print_function
>>> import time
>>> import pychromecast
>>> pychromecast.get_chromecasts_as_dict().keys()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Python27\lib\site-packages\pychromecast\__init__.py", line 131, in get_chromecasts_as_dict
    **filters)}
  File "C:\Python27\lib\site-packages\pychromecast\__init__.py", line 88, in get_chromecasts
    cc_list = set(_get_all_chromecasts(tries, retry_wait, timeout))
  File "C:\Python27\lib\site-packages\pychromecast\__init__.py", line 39, in _get_all_chromecasts
    hosts = discover_chromecasts()
  File "C:\Python27\lib\site-packages\pychromecast\discovery.py", line 89, in discover_chromecasts
    browser.cancel()
UnboundLocalError: local variable 'browser' referenced before assignment
>>>

i first thought dongle could not be found but that aint the case with second test script.
from __future__ import print_function
import time
import pychromecast
from pychromecast import *

CHROMECAST_HOST = '<IP_ADDRESS>'
globals = [0]
bob = pychromecast.get_device_status('<IP_ADDRESS>')
print (bob)
cast = pychromecast.Chromecast(CHROMECAST_HOST)

it finds dongle but get second error here.
DeviceStatus(friendly_name=u'Pegasus-Cast', model_name=u'Eureka Dongle', manufacturer=u'Google Inc.', api_version=(1, 0), uuid=UUID('**********-18ab-1a1a-d241-ceceb703f7e0'), cast_type='cast')
No handlers could be found for logger "pychromecast.socket_client"

I am encountering the same issue with both Python 2 and 3 on Ubuntu.

get_chromecasts_as_dict() has been removed from pychromecast. See https://github.com/balloob/pychromecast/releases/tag/0.8.0.


Error while running yarn dev

Bug Report
While running yarn dev the following error occurs:
ERROR in ./node_modules/payload/dist/admin/components/elements/ReactSelect/index.js 3:0-89
Module not found: Error: Can't resolve 'react-sortable-hoc' in '/.../node_modules/payload/dist/admin/components/elements/ReactSelect'

Expected Behavior
No error should be thrown.
Current Behavior
Error is thrown.
Possible Solution
Seems like react-sortable-hoc is set as dev-dependency, hence not available in the production build.
Steps to Reproduce

run npx create-payload-app
run yarn dev

Detailed Description
Seen in version 1.0.25

Created PR #969

fixed in v1.0.27


ISO Header Struct Fixes

Fixes use of wrong ISO header structure, and adds a bitmask to ensure that iso header length value ignores RFU bits:

@carlescufi Can we merge this today please? It's blocking some of our internal development.

Is there anything here that should be backported?

Is there anything here that should be backported?

Not sure. ISO is still considered an experimental feature, so not sure if that really warranted. It also isn't a fatal bug, and will only be an issue with non-compliant controllers.


fix: fixes typescript and intellisense

Summary

Fixes #2548
Both intellisense and typescript tests were broken on master. Though the issue mentions webstorm as the editor I had the same with vscode.
Converted the index.ts into index.js with CJS syntax as per eslint requirement. Also fixed the typescript tests and added them to npm run test:full script - now that they are working.
Test plan

For testing I ran npm run build
 ~/r/exceljs | fix/2548/fix..intellisense                                                                                51s | 20.8.0 node | 2.7.5 rb
> npm run build

><EMAIL_ADDRESS>build
> grunt build

Running "babel:dist" (babel) task

Running "browserify:bare" (browserify) task
>> Bundle ./dist/exceljs.bare.js created.

Running "browserify:bundle" (browserify) task
>> Bundle ./dist/exceljs.js created.

Running "browserify:spec" (browserify) task
>> Bundle ./build/web/exceljs.spec.js created.

Running "terser:dist" (terser) task
>> 1 grunt.util.pluralize(createdFiles, 'file/files') created.

Running "terser:bare" (terser) task
>> 1 grunt.util.pluralize(createdFiles, 'file/files') created.

Running "exorcise:bundle" (exorcise) task
Exorcising source map from ./dist/exceljs.js
Exorcising source map from ./dist/exceljs.bare.js

Running "copy:dist" (copy) task
Created 20 directories, copied 332 files

Done.

Then npm run test (the output is really big so I omit it here).
Also to check intellisense working and to make sure that different module approaches are still working I created 3 files: one ESM, one CJS and one TS. I then use npm install from the local source code of my branch to have the changes in. And both checked the intellisens and ran the code in the files to see if the lib is working.
See below:

Related to source code (for typings update)

I actually got the same exact problem and this PR fixed it. To ensure proper development, I had to specifiy
"exceljs": "github:groozin/exceljs#fix/2548/fix-typescript-and-intellisense"
in my package.json in the devDependencies
Please fix this.

Any update @Siemienik @groozin ?

@marisuxma @alexandre-embie I tried both versions 4.3.0 and 4.4.0 and can't reproduce with the latest vscode. Meaning all is working well for me now. Lets see how to progress here.

I'm using webstorm. I think that's where the problem is from, but your package is the only one I have that has this problem so there has to be something setup wrong

Hi, i have this issue too


EPICS_HOST_ARCH and other environment variables can override system-defined settings

Issue
Settings in the activation script such as this:
https://github.com/conda-forge/epics-base-feedstock/blob/1084f602acb591c807347ffe4adf78ac5a33125f/recipe/build.sh#L50
Can override the existing environment's settings undesirably.
Example
Specifically, at the LCLS we have a custom EPICS_HOST_ARCH defined for our machines that depends on the RedHat release version. That is, we have rhel6-x86_64, and rhel7-x86_64 whereas upstream epics-base only defines linux-x86_64.
(ours) https://github.com/slac-epics/epics-base/blob/091367acbd3112bb2a6fc8397e0e5b1006d79d70/startup/EpicsHostArch.pl#L35-L36
(theirs) https://github.com/epics-base/epics-base/blob/3e371da8ec327509621845c8c4a67c1e9cb61462/src/tools/EpicsHostArch.pl#L35
Such an activation script can then break the existing environment where rhel7-x86_64 is not equivalent to the feedstock-defined linux-x86_64.
Workaround?
A few possible workarounds that come to mind are the following:

Could the activation script look for a flag that enables/disables the current functionality?
Could the activation script only set variables that were previously unset?

Alternatively, is there a mechanism in conda for disabling specific activation scripts that we might be unaware of?
cc @zllentz @tangkong

Hello,
As far as I know there is no mechanism to disable activation scripts.
I'm not sure to understand the use case.
The goal of conda is to create binary packages that are independent of the OS. This package should allow to install epics-base on Debian or CentOS (not sure if rhel6 is supported, but for sure rhel7 or rhel8 should work), so using the default linux-x86_64 makes sense. Libraries will be installed under <conda env>/epics/lib/linux-x86_64. Changing the EPICS_HOST_ARCH will probably break things.
If you want to use this conda epics-base package, you should compile everything using it. You shouldn't try to mix it with another environment / different epics-base.
Either you use the conda-forge epics-base and compile your IOC with it, either you use your own epics-base. Both environment can coexist on the same machine. The conda one will only set the proper variables when activated. If we don't some things might probably break for this environment.
Or did I misunderstand the issue?

If you want to use this conda epics-base package, you should compile everything using it.

I'm afraid that's not a possibility for us. We have thousands of IOCs and support modules on epics-base versions from 3-14 to 7 on various architectures unsupported by your distribution.
Is everything EPICS built with conda at your facility? I would be interested to see how you have it set up.

You shouldn't try to mix it with another environment / different epics-base.

We do not specifically want to use your package, but rather we are forced to use it because it is a hard dependency of other Python packages present in our environment (regardless of if we use those parts or not).
The summary is that sourcing an environment with the provided activation script breaks other tools we have that expect the correct (to us) host architecture environment variable. We were hoping for a mutually agreeable mechanism to allow disabling the activation script for this package, but I see that your view of how this package must be used will not allow for that.
Thanks for your time.

Is everything EPICS built with conda at your facility? I would be interested to see how you have it set up.

I did some experiment when working at ESS but this isn't the solution that was chosen. I don't work there anymore, so I don't know the current status. You can find some (old?) documentation and some links here:

http://ics-infrastructure.pages.esss.lu.se/conda/quickstart.html
https://gitlab.esss.lu.se/e3-recipes
https://gitlab.esss.lu.se/conda-recipes

You shouldn't try to mix it with another environment / different epics-base.

We do not specifically want to use your package, but rather we are forced to use it because it is a hard dependency of other Python packages present in our environment (regardless of if we use those parts or not).
The summary is that sourcing an environment with the provided activation script breaks other tools we have that expect the correct (to us) host architecture environment variable. We were hoping for a mutually agreeable mechanism to allow disabling the activation script for this package, but I see that your view of how this package must be used will not allow for that.
Thanks for your time.

OK, I understand. That's tricky.
Not sure I have a good solution. Maybe patching the conda package on your side is an option?

I did some experiment when working at ESS but this isn't the solution that was chosen. I don't work there anymore, so I don't know the current status.

To shed some light on the current ESS implementation: It is true that a majority of the IOCs at ESS are deployed by other means, but we do also have (and intend to keep having) IOCs deployed using Conda.
Our IOCs deployed with Conda are intended to be distribution agnostic and do not rely on system dependencies. We maintain our own recipe for epics-base (https://gitlab.esss.lu.se/e3-recipes/epics-base-recipe/) as well as all modules needed and the packages are distributed through our internal Conda channel(s), i.e. we're not using conda-forge directly.
I share @beenje 's view, that it should be considered discouraged to mix environments since it's a recipe (pun?) for problems. @klauer, I'm afraid I don't have any better ideas to solve your issues than what's already been mentioned.


Building a standalone web component

Hi. This is very nice work you've done with this forms library.
My question: How can I use the example here with the current version of this repo? Basically, how would I build something like https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>

yes the current version which is based on vue3 and vuetify3 in the master branch is currently not yet released since I'm waiting for few more issues to be resolved before the release. I expect that most likely in about few weeks there will be a release that you can use - either the the fixes that I'm talking about or without them depending on how much changes are neededed in the jsonforms project

👍 Thanks for the update.


Add developer tools.

I had a previous pull request which suffered quite a bit from my poor git skills.
This is intended to resolve #1090.
This pull request intends to introduce developer tools into Yew. The idea is that if you compile your crate with the dev feature enabled, a browser extension will give you helpful information about your component heirarchy, etc. It is intended to be somewhat similar to the React Developer Tools.

Added skeleton browser extension.

👀

There's not much there at the moment. For some reason Travis isn't running against my commits, so I'll need to fix that first.

Ah ok, still exciting!

For some reason Travis isn't running against my commits, so I'll need to fix that first.

Seems like a travis issue 🤷

Yup. On the Travis page it says that the branch is “missing a merge” so I’ll rebase from the master branch and see if that helps.

On 31 May 2020, at 11:34, Justin Starry<EMAIL_ADDRESS>wrote:
Ah ok, still exciting!
For some reason Travis isn't running against my commits, so I'll need to fix that first.
Seems like a travis issue 🤷
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/yewstack/yew/pull/1221#issuecomment-636452573, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKFSTPPBQA7MG7MUZKTA6JLRUIXCDANCNFSM4M7WILCA.

This has been blocking for a while on adding some additional functionality to indextree which has now been approved 🎉 !
Should be able to move further ahead with this now.

This isn't completely finished yet, but I'd value some input into what thoughts there are on this implementation.

Hey @teymour-aldridge! What's the status of this PR?
I looked it over and have some high level feedback 😄

I would prefer the changes to the core yew crate to be as minimal as possible for now. It looks like most of the dev module could be moved out into a separate yew-devtools-client crate.
If a developer enables the "devtools" feature on yew, the devtools connection should be connected in the yew::initialize() method. And then each time a component lifecycle method is called, the devtools connection should fire a message.

I think the devtools would be a lot more useful with prop information. We can conditionally require Properties to implement Serialize / Deserialize when the "devtools" feature is enabled and then have the extension display JSON nicely. Thoughts?

I think the devtools would be a lot more useful with prop information. We can conditionally require Properties to implement Serialize / Deserialize when the "devtools" feature is enabled and then have the extension display JSON nicely. Thoughts?

I agree completely!

I would prefer the changes to the core yew crate to be as minimal as possible for now. It looks like most of the dev module could be moved out into a separate yew-devtools-client crate.

Will do.

devtools connection should be connected in the yew::initialize() method

That's a neat solution!
I also need to work out some proper handling for people running multiple Yew applications on the same device so that messages from different apps don't get confused.

Uh oh. Looks like the rebase failed.
Apologies to all the people pinged 😬

Ok everything now seems to be OK except somehow these two commits crept in 🤷🏼.

I think I've managed to banish those two commits with the magical power of the interactive rebase.

@teymour-aldridge I'm going to close this as stale for now to clean up the PR backlog. Feel free to re-open when you have time to dig back in 😄

Cool – I'll try and finish this up (at some point) before reopening.


Added logo to README.md

Personally I'd prefer the logo a bit smaller and centered. But it is just a matter of taste.

What about the "organization" logo? See e.g. https://github.com/TorXakis

@pjljvandelaar Just did that too! Nothing to version though.


[clusteragent/admission/controllers/webhook] Use mock configs

What does this PR do?
Modifies some tests in the admission controller to use a mock config.
The second commit is just a cleanup to group similar tests together.
Motivation
Not using a mock config can lead to difficult to debug problems if several tests rely on the same config params and they're not reset properly.
Describe how to test/QA your changes
Skip.

/merge


Fix upload of image assets to iTunes Connect

iTunes Connect requires height, width, size and checksum when uploading screenshots

looking forward to see this - as it blocks current usage of deliver. @KrauseFx

as mentioned in https://github.com/fastlane/fastlane/issues/6426


Remove stripping .py extensions

The main branch of workerd no longer requires stripping the .py extension from Python modules (and in fact, crashes if Python modules do not end in .py).
This PR makes it so that Wrangler no longer strips the .py extension from Python modules.

This will likely need a workerd bump in Miniflare before we can land it. Are the workerd changes in a release yet?

This one contains the changes necessary: https://github.com/cloudflare/workerd/releases/tag/v1.20240208.0.
@garrettgu10  We should update to it in this PR.

@penalosa I bumped the workerd version :-) but it looks like I'm failing a bunch of random test cases? Not sure what's going on... :-/

@mrbbot I'm not sure what you mean by that. I thought updating the PythonModule names to include .py would fix things, but it looks like a bunch of unrelated tests are failing?

@mrbbot does this work now? It seems to still fail CI but the D1 test does pass now


Rollouting local AEM instances  by configuration

Imagine:
gradle aemUp  will which will dependsOn([aemSetup aemSatisfy])
aemSetup will create
~/.gradle/aem/example/local-author/[crx-quickstart.jar,licence.properties,start.bat|sh]
~/.gradle/aem/example/local-publish/[crx-quickstart.jar,licence.properties,start.bat|sh]
then aemSatisfy will install needed dependencies
then aemUp will run AEM in background, corresponding aemHalt can stop it (like in Vagrant).
aemSetup {
   jar = 'smb://some-server/aem/6.2/crx-quickstart.jar'
   license = 'smb://some-server/aem/6.2/license.properties'
}

motivation to create that improvement is fact, that AEM on Vagrant is noticeable more slower.... and that full automation project related could be very helpful in case of small projects, PoC in which SE is not available to configure Vagrant for us.

change has many breaking changes, but provides local instances management tasks. due to that facts this feature will be included within 2.0.0 release

released in 2.0.0-beta


Terminal fails to refresh/display tabs after VM resumes from suspend

Environment
Windows build number: 10.0.19041.388
Windows Terminal version (if applicable): 1.3.2651.0

Any other software?
Parallels Desktop 16.0.1 VM running Win10

Steps to reproduce
Launch Windows Terminal
Open a few tabs
Run any command
Suspend the VM:  Go to Parallels Desktop (MacOS app), and choose Action -> Suspend
Resume the VM
The display does some sort of redraw
Windows Terminal renders the content in the active tab (good)
Entire titlebar/tabs region is gray (bug)
Typing into the content area doesn't result in a refresh.
Pressing Ctrl-Tab to switch tabs fixes the issue, but is annoying.
Resizing the Terminal window also fixes the issue.
Expected behavior
Tabs should refresh when VM resumes
Actual behavior
Entire tab area is gray.
Let me know if there is some tool I can run (I tried Spy++) to see what messages the Terminal window is receiving during suspend/resume.

Hmmmm. This seems like the inverse issue from #3446. Nobody on the team has ever seen either, which is somewhat alarming.
I'm concerned that this is a platform issue (in XAML islands.)
I wonder what's happening when Parallels resumes the VM.

This is gonna be a weird question, but if you run c:\windows\system32\PerceptionSimulation\PerceptionSimulationInput.exe and then suspend/resume... does its UI do the same thing as Terminal?

Good call -- yes, I see the same issue with PerceptionSimulationInput.exe
Screenshot attached.

When Parallels resumes, the display goes blank for a moment, as if it's restarting the video card driver.

Thanks! I filed this upstream on XAML!
/dup https://github.com/microsoft/microsoft-ui-xaml/issues/3472


Error: Invalid response

Hey, Joshua, the issue with downloading appeared a couple days ago. I updated the package for the newest version,  here's the stack trace:
ytdl-mp3 https://www.youtube.com/watch?v=7TREIsVM08Q
Unable to extract all song tags from YouTube
Enter title:
C:\Users\admin\AppData\Roaming\npm\node_modules\ytdl-mp3\src\utils.js:26
reject(new Error('Invalid response: ' + response));
^
Error: Invalid response:
at C:\Users\admin\AppData\Roaming\npm\node_modules\ytdl-mp3\src\utils.js:26:24
at [_onLine] [as _onLine] (node:internal/readline/interface:422:7)
at [_line] [as _line] (node:internal/readline/interface:885:18)
at [_ttyWrite] [as _ttyWrite] (node:internal/readline/interface:1263:22)
at ReadStream.onkeypress (node:internal/readline/interface:272:20)
at ReadStream.emit (node:events:527:28)
at emitKeys (node:internal/readline/utils:358:14)
at emitKeys.next ()
at ReadStream.onData (node:internal/readline/emitKeypressEvents:64:36)
at ReadStream.emit (node:events:527:28)
Node.js v18.0.0
I use windows 10.
The error occurs if the "Enter title:" or "Enter artist:" is empty. Before that it worked like a charm if I left them empty. Do you have any suggestions about this problem?

Hey, so yesterday I tried to download something and got this error, which appears to be a problem with the ytdl-core library. In addition, the method I was using to fetch metadata for the songs is now useless due to a change in the YouTube API which was causing me to be prompted to enter the details each time. So, I made a number of changes (including rolling back the ytdl-core version) which might have broken this.

Thank you for the answering, will wait for update!

Hey, no worries! So I made some major updates, since the changes YouTube made broke the (easier) way of getting the tags. Now, they are fetched from iTunes based on the video title, so it should actually work now unlike the previous method. However, if you don't want to add any tags, I added an option ("-n") that you can add and then it should skip adding or fetching any tags. Let me know if there are any other issues :)

Cool, that's nice to hear. I will check out the update in the evening and test things out. Thank you for your fast reply and solution!


Fix implementation of batch normalization

Small fixes to the implementation of batch normalization (which I've measured to have good effect on its efficacy):

Initialize the gamma parameter (which is the nn.BatchNormalization's weight parameter, as seen here) to 0.1 as recommended in Cooijmans et. al.
Unset the default values of self.eps and self.momentum because those parameters can be passed as nil into nn.BatchNormalization, so unless there's a good reason for setting rnn-specific initializations it should be a transparent API. Also, the default value of 0.1 for eps is too high - the default inside nn is 1e-5, which seems more appropriate for a divide-by-zero fudge factor. In my experiment setting it to 0.1 caused the network to converge slower.
Remove :noBias() since self.o2g is already a LinearNoBias (this is just for code cleanliness).

@rachtsingh Great fix. Thanks for taking the time. And sorry for the long wait!


Implement oniguruma in pure Rust

JRuby and TruffleRuby use a Java port of oniguruma called joni.
C2Rust is able to transpile onig_sys to Rust, although the code does not compile.
Try transpiling oniguruma to Rust to remove a C dependency.

Upstream bug for the compile failure: https://github.com/immunant/c2rust/issues/149

Oniguruma transpiles and compiles in Rust for me now. The const macro conversion option doesn't work for oniguruma yet, but I'll look into that.

Originally posted by @rinon in https://github.com/immunant/c2rust/issues/149#issuecomment-536142439

Upsteam (non-blocking): https://github.com/immunant/c2rust/issues/166

A much better alternative to this is to bind to Onigmo, which is the MRI fork of oniguruma. See work upstream in https://github.com/rust-onig/rust-onig/issues/149


Prevented the Operator from overriding Secrets/ImagePullSecrets on ServiceAccounts

Fixes #521 by not overriding the Secrets, ImagePullSecrets nor AutomountServiceAccountToken fields from service accounts.
Signed-off-by: Juraci Paixão Kröhling<EMAIL_ADDRESS>

@objectiser right now, we do not set those fields. We might set them in the future (#404 for instance), but if we ever do that, we can just build a merge function to keep the values added from processes outside of the operator.

@samcrutt9900, this should fix #521. If you want to give this one a try, I published the following image: jpkroehling/jaeger-operator:521-Do-not-override-secrets-imagepullsecrets-on-serviceaccounts

cc @knrc, @jwendell, @gbaufake, with this change, the Jaeger Operator should keep your changes to the ServiceAccount objects.


refactor(transformer/es2021): move all entry points to implementation of Traverse trait

follow-up: 4881

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5014
#5013  👈
#5012
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite


mute/unmute feature added, fixes #330

PR Description 📜
Added a mute/unmute feature to the dinorun game
Fixes issue #330

one with muted, other unmuted

muted:

unmuted:

Thank you  @Lakshyyaa , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Tree view for topology

Having a tree view for a topology would be really helpful.

https://www.npmjs.com/package/archy


python312Packages.pydeconz: 117 -> 118

Diff: https://github.com/Kane610/deconz/compare/refs/tags/v117...v118
Changelog: https://github.com/Kane610/deconz/releases/tag/v118

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

4 packages built:

home-assistant-component-tests.deconz
home-assistant-component-tests.zha
python312Packages.pydeconz
python312Packages.pydeconz.dist


[#446][Part 2] Add OpenAPI to support creating API documentation with OpenAPI

Implement #446
What happened 👀
This part mainly adds the UI for the generated project which is mounted at: /api/docs/openapi
Insight 📝
After generating the project, it will:

Copy the view template rendered by Stoplight
Add a new route that redirects to that view

Mock Server will be added in part 3, it's also the last part of this series.
Proof Of Work 📹
Tested the generated project without any additional setup
https://github.com/nimblehq/rails-templates/assets/63148598/0ed76042-f91a-4d87-932a-0464f6695afd

Rebase needed 🙇


Warning fix

Fix warning on Travis CI

Thanks! I merged this via a squash in https://github.com/kinetiknz/cubeb/commit/b0f1a6331f327b4fc127c8d206bca944f1141b80.

Thanks for the help and correction about my wrong fix for resampler.c. You save the code!
I'll try upstreaming the fixes on its own repo!

No problem, thanks for the patches! Let me know if I can help with the speexdsp upstreaming at all.


Mouse pointer not showing up for Steam controller

This may not be completely sway related but it's where I noticed it.
I ran sway maybe a month ago and when I used my Steam controller (with Steam running) I was able to control the mouse cursor (It uses uinput to present devices)
I just setup sway again today and something weird happens. When I plug in the controller it moves "a mouse" but it's invisible. I can see it moving by seeing things changing color as I hover over them. The regular sway cursor doesn't move at all. Moving my regular mouse then moves the sway cursor as it should.
I'd be happy to provide any info as needed.

Video https://www.youtube.com/watch?v=j0ClJC1K2xI

Update: Seems to be however Steam is initializing it with uinpt/udev.
When the controller is wired and Steam isn't running it presents itself as a hardware mouse and keyboard and performs fine in sway.
Wired with Steam running has the same behavior as wireless with Steam running. "Invisible" mouse.
Wireless without Steam running doesn't do anything (Wireless is only fully initialized while Steam is running)
Third party https://github.com/ynsta/steamcontroller and using wireless works so I'm just going to chalk this up as a Steam issue and close it out.


[TVMSCRIPT] Using diagnostics for TVM Script

This PR modifies TVM Script to use the new diagnostics interface. This should make error messages a lot clearer.
This PR depends on #6795.
@jroesch @tqchen @junrushao1994

cc @spectrometerHBH

cc @tkonolige @mwillsey

Please wait until the jenkinsfile get updated, we will need a separate PR to update the Jenkinsfile first

docker images has been updated

@spectrometerHBH Would you like to take a look? Thanks!

Is this PR ready for review? Looks like it encountered problems in CI.

@spectrometerHBH Feel free to review it now, I need to bump the dependency in docker.

Do we prefer to maintain synr as an external dependency, and make a release every time, or include synr as part of tvm?

Do we prefer to maintain synr as an external dependency, and make a release every time, or include synr as part of tvm?

I think it is valuable to maintain outside of the TVM code base. I know monorepo'ing everything provides some conveniences but when there are reusable components which can be used outside of TVM it makes sense to maintain them as separate pieces of code. For example there have been multiple other projects which have built similar approaches to synr and I think it is valuable to provide it in an easily reusable component, for example TensorFlow and PyTorch have both opt'd not to do this and even though they do AST rewriting there is no easy way to take advantage of all the work they did.

@leandron Does this PR look good to you now? Or are there any other changes you would like?

Thansk @tkonolige @leandron @u99127 @jroesch @junrushao1994 @spectrometerHBH !


Fix for going over CONFLUENCE_MAX_TITLE_LEN while handling title conflict

Increasing trimming length over CONFLUENCE_MAX_TITLE_LEN results in failed publishing attempt. Instead, trimming using tail length so that the title won't go over the limit. Signed-off-by: Anton Mihalkov<EMAIL_ADDRESS>

Added a signed off message in my commit, yet there still seems to be a problem according to the DCO. Please advise.

Thanks for the pull request. For sure the original implementation was just broken. I had one concern with respect to this change. For these lines in the original implementation:
https://github.com/sphinx-contrib/confluencebuilder/blob/99f2658504fcad0f22c1dd139e1c822e01cffa38/sphinxcontrib/confluencebuilder/state.py#L91-L93
I assume when a post-fix is set, it can result in the same problem with bumping try_max (I assume the original implementation should have been subtracting this length instead). If try_max is improperly set, I assume it would still cause issues with the corrects you are introducing in the title-conflict check.
Any chance you are willing to look at that? I could look at trying to add some unit tests later to sanity check title trimming/duplication logic (unless you'd be interested in adding some as well).

@jdknight don't plan to add any unit tests at this point

@AntonMihalkov, thanks for the pull request. 🙇


REFPLTV-2539:[MFRHal]Serialized Data is not getting properly

Reason for change: [MFRHal]Serialized Data is not getting properly for some parameters
Test Procedure: Run the MFRHAl Scripts
Risks: Low
Signed-off-by<EMAIL_ADDRESS>

Hi @DorababuPragada ,
Thank you for this changes. Found more gaps while reviewing this and tried to address those in this https://github.com/rdkcentral/rdkvhal-mfrlibs-raspberrypi4/pull/13 including the changes from this PR.
Kindly review


ACME/Let's Encrypt does not work with split horizon DNS.

Environment

OS: Fedora 36, fully updated
Kubernetes/K3s: v1.24.4+k3s1 (c3f830e9)
AWX Operator: AWX Operator 0.28.0
cert-manager: 1.9.1

Description
I am using split-horizon DNS, and would like to use Let's Encrypt/ACME steps to secure my AWX installation. I am using CloudFlare as my external DNS, with BIND9 for internal DNS.
Step to Reproduce

Deployed certmanager using the cert-manager yaml as per instructions.
kubectl apply -f https://github.com/jetstack/cert-manager/releases/download/v1.9.1/cert-manager.yaml
configured issuers.yaml for Cloud Flare:

apiVersion: cert-manager.io/v1
kind: Issuer
metadata:
  name: awx-issuer
spec:
  acme:
    # The email address to be registered with ACME CA
    email<EMAIL_ADDRESS>
    # The URL of the ACME API endpoint.
    # In Let's Encrypt, this is one of the following:
    #   Production: https://acme-v02.api.letsencrypt.org/directory
    #   Staging   : https://acme-staging-v02.api.letsencrypt.org/directory
    server: https://acme-staging-v02.api.letsencrypt.org/directory

    privateKeySecretRef:
      name: awx-issuer-account-key

    solvers:
    - dns01:
        cloudflare:
          apiTokenSecretRef:
            name: cloudflare-api-token-secret
            key: client-secret```

3. configured kustomization.yaml as per documentation on cert-manager.io and here:
  ```---
apiVersion: kustomize.config.k8s.io/v1beta1
kind: Kustomization
namespace: awx

generatorOptions:
  disableNameSuffixHash: true

secretGenerator:
  - name: cloudflare-api-token-secret
    type: Opaque
    literals:
      - client-secret=<redacted>

resources:
  - issuer.yaml

check DNS to see that the _acme-challenge.awx. has propagated:

; <<>> DiG 9.16.31-RH <<>> @<IP_ADDRESS> _acme-challenge.awx.<domain> txt
; (1 server found)
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 42270
;; flags: qr rd ra; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 1232
;; QUESTION SECTION:
;_acme-challenge.awx.<domain>. IN	TXT

;; ANSWER SECTION:
_acme-challenge.awx.<domain>. 120 IN TXT	"<acme key redacted>"

;; Query time: 41 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Mon Sep 05 16:11:08 CST 2022
;; MSG SIZE  rcvd: 118

At this point, cert-manager just hangs. According to the cert-manager documentation, if split-horizon DNS is being used cert-manager needs to be invoked with some extra parameters to control the DNS servers. See https://cert-manager.io/docs/configuration/acme/dns01/#setting-nameservers-for-dns01-self-check. In my case, I would have to use:
--dns01-recursive-nameservers-only --dns01-recursive-nameservers=<IP_ADDRESS>:53,<IP_ADDRESS>:53

see https://github.com/cert-manager/cert-manager/issues/5415

@str8edgedave
Hi, thanks for filling the issue.
First, my guide is just aimed at prividing the simplest example implementation, so the detail of how to use cert-manager is a bit out of scope of my repo. Thanks for understanding.

I would have to use:
--dns01-recursive-nameservers-only --dns01-recursive-nameservers=<IP_ADDRESS>:53,<IP_ADDRESS>:53

F.Y.I., you can edit arg parameter for the Deployment resource for cert-manager bykubectl edit, or downloading cert-manager.yaml and appending args then apply it instead of kubectl appy using YAML file from URL directly.
Or, of course you can use helm to install cert-manager on K3s.

Thank-you very much!!!
I'm just getting started with Kubernetes, and with your suggestions I was able to figure it out. I just had to use the kubectl edit command, then delete the stale Certificate, order and challenge and it worked!
On suggestion for your documentation. In the issuer.yaml, you have the staging ACME api. You may want to add text about using the production issuer once everything is working with staging.
I really appreciate all you do for the community.


Broadcast cuts off using spaces

Broadcast will only send the first word before a space.
This is a known issue with Palworld, I am waiting on them to fix it, nothing can be done as of yet.
If you absolutely need broadcasts, use underscores.

Added a note in the README and enabled the command.


How to set header background color

Hey thanks for this great plugin 🙏🏾  is there a possibility to set background color for header like # ## ### ... ? Coudnt find the hl_groups.
Would like to have something like here: https://github.com/folke/todo-comments.nvim
thanks :)

I would like to know too!
#392

Figured it out after like a week...
You can use :Inspect to return the highlight of a specific element under your cursor. This way you can add custom highlights using vim.cmd[[highlight...]]
Would have been nice if someone pointed that out to me. But hope it helps.

Thanks for now :))
Just getting this from Inspect:
Treesitter
  - @markup.heading.2.marker.markdown links to Title markdown

any idea, how to set colors for this?

yeah as i said you add a vim command to your lua config
vim.cmd[[highlight elementName guifg="#FFFFFF" guibg="#FF0000" gui=underline]]
you can specify fg color, bg color, and display type like bold italic, etc
your cursor is over the marker for ## not the h2 element itself
for changing the marker, its
vim.cmd[[highlight @markup.heading.2.marker.markdown  guifg="" ]]

Great, it works. But now Im able to colorize marker and heading, not space between 😆  any idea how to get the whole line?

Great, it works. But now Im able to colorize marker and heading, not space between 😆 any idea how to get the whole line?

no highlight for the space in between, as far as im aware, shouldnt matter too much...
use google and these commands if confused about highlights.
:highlights
:Inspect
:InspectTree
this type of stuff should be documented somewhere in the plugin, if the creator sees it, them im happy to make an entry into docs.

this is what it looks like for me

Thats pretty cool! Thanks. Im gonna check that out.

thanks, but dont like it. Looks to chaotic to me ... just wanna highlight marker of headings like
vim.cmd([[highlight @markup.heading.1.marker.markdown guibg=#FF5733 guifg=#FF5733]])
vim.cmd([[highlight @markup.heading.2.marker.markdown guibg=#C70039 guifg=#C70039]])
vim.cmd([[highlight @markup.heading.3.marker.markdown guibg=#900C3F guifg=#900C3F]])
vim.cmd([[highlight @markup.heading.4.marker.markdown guibg=#FFC300 guifg=#FFC300]])

and it worked like charm some weeks before ... now there is no @markup.heading.4.marker anymore :'(((

tbh ive never used past headings 2s...
Youre probably better off using headlines.nvim


IconMenu, DropDownMenu etc. are not getting copied to node_modules during "npm install"

Hello everyone! I am dealing with a strange issue here and going clueless. Please help me if you know what is going wrong. Any pointer to identify the issue will be highly appreciated.
My problem is that some of the components are not getting installed as part of the material-ui npm package.
For example:
IconMenu (folder is not even there under node_modules\material-ui),
MenuItem
DropDownMenu
Error:
./src/components/NavBar/LoggedNav/index.js
Module not found: Can't resolve 'material-ui/IconMenu' in '/Users/xyz/proj/myproj/src/components/NavBar/LoggedNav'
However, I am able to use Avatar, IconButton etc. Tried uninstalling and installing the package a few times from stable as well as @next, but the behavior has always been the same. I had used all these components successfully in a couple of learning projects over the past couple of months without any issue and those projects still have these components under node_modules.
It just doesn't make sense why some components are not getting copied as part of the package while most are.
Works:
import IconButton from 'material-ui/IconButton';
import Avatar from 'material-ui/Avatar';
import Paper from 'material-ui/Paper';
import Menu from 'material-ui/Menu';
Doesn't work:
import MenuItem from 'material-ui/MenuItem';
import IconMenu from 'material-ui/IconMenu';
There are more that do not work; the above is just to show how I imported in the code - both working and non-working cases.
Kindly help resolve this as I am unable to make progress unless I go for a different package like grommet. I really want to stick with material design and this is the one I liked the most.

Material-UI: 1.0.0-alpha.16 (tried the latest stable version as of today, 6/11/2017 as well)
React: 15.5.4
Browser: Safari and Chrome (shouldn't matter)

Installation commands tried (both multiple times, each time after uninstall):
npm install material-ui@next --save
npm install material-ui --save

Doesn't work:
import MenuItem from 'material-ui/MenuItem';

Use:
import { MenuItem } from 'material-ui/Menu';

// or
import { MenuItem } from 'material-ui';

Doesn't work:
import IconMenu from 'material-ui/IconMenu';

The IconMenu component hasn't been ported. Have a look at the demo regarding the right import path. There is many breaking changes with that alpha version.

Thank you so much @oliviertassinari. That worked for MenuItem. IconMenu showed up when I did npm install without the "--save" option. Can't understand why though. I tried this a few times to make sure that is what is happening. With --save, IconMenu and DropDownMenu are missing always and without --save they always show up under the package list. Thanks again and have a great day!


Add project icon

All you have to do is put the image to .idea/icon.png
https://github.com/DroidKaigi/conference-app-2021/blob/main/.idea/icon.png
And we have an icon in app-android res folder.

🙋

Thanks! Assigned 👍


Support Bazel --override_repository flag

@justbuchanan, thanks so much for sending in your PR! Lemme run the tests and merge this.


Show update type in UpdatePrompt

r? @gabriel @maxtaco

And this comes all the way from the API server?

I talked to @gabriel about this.  Currently there's a json file in an s3 bucket and it has the update type.  The API server isn't being used for updates...

yeah currently comes from update json in s3 bucket: https://s3.amazonaws.com/prerelease.keybase.io/update-darwin-prod.json
this file is created and uploaded automatically after each prerelease (and nightly) build
:ship:

:+1:


Dedicated invoice view when clicking on an invoice

Possibly add two additional buttons:
Add Payment  and Edit
Add Payment just updates the amountPaid field and the BE logic toggles the paid boolean, if amountPaid  >= balance
Edit takes the user to the Edit view, so they could edit anything else

done


Add location service to calculate distance between two locations.

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #3039
Engineer Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have added any strings visible on UI components to the strings.xml file
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[ ] I have built and run the FHIRCore app to verify my change fixes the issue and/or does not break the app
[ ] I have checked that this PR does NOT introduce breaking changes that require an update to Content and/or Configs? If it does add a sample here or a link to exactly what changes need to be made to the content.

Code Reviewer Checklist

[ ] I have verified Unit tests have been written for any new feature(s) and edge cases
[ ] I have verified any strings visible on UI components are in the strings.xml file
[ ] I have verifed the CHANGELOG.md file has any notable changes to the codebase
[ ] I have verified the solution has been implemented in a configurable and generic way for reuseable components
[ ] I have built and run the FHIRCore app to verify the change fixes the issue and/or does not break the app

Closing this in favor of #3107


[warning] [cmake-tools]: Cannot find key cmake-tools.configuration.cmake.options.advanced.statusBarVisibility.variant.description , And git have been waiting.

Brief Issue Summary

CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information

Just open the project in a new window, don't use the old workspace file.

@cyrus123456 it should be fixed now in the newest pre-release channel update, if you could install it and confirm, thanks!


Failed to load config file! Error: config.yml: file not found

Hi. Disclaimer - not an expert, using portainer.
Have this error:
Failed to load config file! Error: config.yml: file not found
I've double checked typos, have the config.yml in the folder below, config.yml has rw-r-r
version: "3.0"
services:
frigate-notify:
image: ghcr.io/0x2142/frigate-notify:latest
container_name: frigate-notify
environment:
- TZ=Etc/UTC
volumes:
- "/home/1/frigate-email:/config/app/"
restart: unless-stopped
Any thoughts please welcomed? Ready to facepalm if I'm missing something obvious!

Hi @35nine, the image has the following built-in settings:
$ podman image inspect ghcr.io/0x2142/frigate-notify:latest -f '{{ .Config.WorkingDir }}'
/app

As for the business logic of the application, it will try to find config.yml in the working directory if it is not overridden. This behavior is also mentioned in the documentation, please check (https://frigate-notify.0x2142.com/install/).
You have 2 ways to solve this issue:

Add an environment variable FN_CONFIGFILE with the content /config/app/config.yml.
Provide the volumes configuration as /home/1/frigate-email/config.yml:/app/config.yml.

NOTE:
Please do not try to mount the entire /home/1/frigate-email directory into /app as this will override the /app/frigate-notify binary and break the container's Entrypoint:
$ podman image inspect ghcr.io/0x2142/frigate-notify:latest -f '{{ .Config.Entrypoint }}'
[/app/frigate-notify]

Hi there @35nine - Thanks for using this app! And thank you @freefd for jumping in to help here 😊
By default the app looks for the config file at /app/config.yml. If you're running the binary, this is fine - but with docker it needs to be overridden. Both options provided by @freefd should resolve this for you. I run mine with the first method & I'll provide a sample of my docker compose file below using your config path:
version: "3.0"
services:
  frigate-notify:
    image: ghcr.io/0x2142/frigate-notify:latest
    environment:
      - 'FN_CONFIGFILE=/app/config/config.yml'
      - 'TZ=America/New_York'
    volumes:
      - /home/1/frigate-email:/app/config
    restart: unless-stopped

Hope that helps! Let us know if you're still having trouble

@freefd and @0x2142
Awesome, no more errors in logs, time to experiment now.  Certainly helped!  Using this fixed me up:
volumes:
- /home/1/frigate-email/config.yml:/app/config.yml
Have a good day both, appreciate the replies


Allow Log group prefix

Adding use_log_group_prefix for using log_group_name as a prefix to match several groups.
Lambda, RDS and other PaaS services generate log groups and this allows to ingest them all.
Also adding add_log_group_name and log_group_name_key for writing the log group name to the record and being able to distinguish them later.
A new example is provided.

I can't get the tests to run in my machine
$ rake test

File does not exist: mocha/test_unit

rake aborted!
Command failed with status (1)
/var/lib/gems/2.5.0/gems/rake-12.3.2/exe/rake:27:in `<top (required)>'
Tasks: TOP => test
(See full trace by running task with --trace)

You should specify AWS_REGION. AWS_ACCESS_KEY_ID, and AWS_SECRET_ACCESS_KEY as environment variable before running tests. Note that don't post real your aws access key and secret on th internet.

Thanks, just managed to run the tests.
Nothing old breaks as of the last commit.
I will add tests for the new functionality, wait for them before merging :)

Hi! May I ask for the status of this PR?
We are very interested in the capability of using log_group prefix to get all /aws/ecs/ and /aws/lambda/ logs instead of having one source per log group.

Hi! May I ask for the status of this PR?

Just staled. Thanks for paying attention to the PR.

I'd created a subsequent work PR for this PR.


Sample Request: New Language Cognitive Service (QnA Maker replacement)

Is your feature request related to a problem? Please describe.
QnA Maker has recently migrated to use the new Language Cognitive Service. The Bot Framework does not appear to have a package to integrate with this service.
Describe the solution you'd like
It would be great to have a sample of a starter bot that fulfills questions from a project/knowledge base created within the Language Studio.
Describe alternatives you've considered
The provided QnA Maker samples do not work with the new service as Language Studio does not provide a KB ID (needed in appsettings.json)
Additional context
Add any other context or screenshots about the feature request here.

We are in the same position - The QnA maker portal is pushing users to upgrade but from what I can see the bot-builder-dotnet SDK which we consume has not published updated packages which actually integrate with the new Language Studio.
I did manage to find a working sample by deploying a bot through the new language studio, then reverse engineering the ARM template which is used to spin up the app service which revealed a zip file which contains a modified version of the 49.qnamaker-all-features sample which integrates with the new language studio.
For reference the zip file in question can be found here: https://qnamaker-data.trafficmanager.net/qnamakerdata/botservice/qnav2bot-cs.zip

See 12.customQABot


Extra Form Fields

Is there a way to add extra fields to the application form? If there is, where can I read more about it.

Heya,
For adding extra fields to the application form, there may need to be some additional work done depending on the actions required.  Most fields that are added would need some sort of authentication process to authorize you to write data to the fields in the database.  I suggest checking out some of Bullhorn's developer documentation found here .
Please let me know if you have any questions.

That's great. Thank you.


juliacall 0.9.22 can result in hanging precompilation

Affects: JuliaCall
Describe the bug
Upgrading to 0.9.22 causes PySR's CI to hang upon precompilation of Zygote.jl on macOS. I haven't been able to reproduce it locally but can confirm that downgrading to 0.9.21 is enough to fix the issue for PySR. So perhaps the new GC is causing some infinite loop somewhere, for some kind of PythonCall object? The error is randomly encountered so it has been hard for me to track down.
Your system
Please provide detailed information about your system:

The operating system: macos-latest
The version of Julia: 1.10
Python 3.12

Here's an example of the test failing, with 0.9.22:

https://github.com/MilesCranmer/PySR/actions/runs/10345006064/job/28631387696#step:7:277 (timeout at 1hr)

And succeeding, on 0.9.21:

https://github.com/MilesCranmer/PySR/actions/runs/10345785401/job/28633325703#step:7:317

It is random but it has so far this CI seems to always work on 0.9.21 while it fails 75% of the time on 0.9.22. It's also only on macOS for whatever reason.

Is there a way I can manually trigger the GC to run? I wonder if I can debug it (or work around the issue) by manually triggering the GC at different stages?

Hmm not sure what's going on here. I wonder if it's due to the presence of the immortal GCHook() object? You could try commenting out this line. If that helps, we could avoid constructing GCHook() while precompiling.

Is there a way I can manually trigger the PythonCall GC to run? I wonder if I can debug it (or work around the issue) by manually triggering the GC at different stages?

Yep you can do PythonCall.GC.gc().

That would mean there would be no garbage collection during precompilation, right? Is that desired?
I wonder if there’s some infinite loop possible somewhere, where Julia GC calls the finalizer on the GC hook, but the finalizer is called immediately, and this process repeats indefinitely.

Do any other packages use a similar pattern with a self-regenerating self-terminating GC object? If not, are we sure it’s safe?

I’m also curious what happens if one were to call
Base.GC.gc(#=full=#true)

Does this trigger any sort of infinite loop of finalizers?

That would mean there would be no garbage collection during precompilation, right? Is that desired?

It was just a suggestion to narrow down what's causing the problem. My suggestion shouldn't prevent GC - the GCHook is just there to free any Python objects that were finalized while the GIL was released (e.g. on a different thread). Anyway, it will all be collected when the precompile process ends, so unless you are allocating a lot during precompilation I wouldn't worry.

Do any other packages use a similar pattern with a self-regenerating self-terminating GC object? If not, are we sure it’s safe?

I don't know and I don't know. I had thought someone commented at some point that this behaviour is documented to work, but I can't find that now so maybe I dreamt it.

I’m also curious what happens if one were to call
Base.GC.gc(#=full=#true)

Would this trigger any sort of infinite loop of finalizers?

I don't think so - I called that function a lot when testing the new GC and didn't observe any hangs.

Maybe something about running precompilation a second time triggers it?

Maybe, I don't know enough about the mechanics of precompilation to say.

version 0.9.22 is creating endless loop with linux and macos triggered by:
error in running finalizer: UndefVarError(var=:PYNULL)
it produces 9k lines of errors:
error in running finalizer: UndefVarError(var=:PYNULL)
ijl_undefined_var_error at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/rtutils.c:134
ijl_get_binding_or_error at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/module.c:471
enqueue_all at /home/runner/.julia/packages/PythonCall/flx5V/src/GC/GC.jl:106
pyobjectarray_finalizer at /home/runner/.julia/packages/PythonCall/flx5V/src/JlWrap/objectarray.jl:30
unknown function (ip: 0x7ffb6a98cd45)
_jl_invoke at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gf.c:2895 [inlined]
ijl_apply_generic at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gf.c:3077
run_finalizer at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gc.c:318
jl_gc_run_finalizers_in_list at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gc.c:408
run_finalizers at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gc.c:454
jl_mutex_unlock at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/julia_locks.h:80 [inlined]
jl_generate_fptr_impl at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/jitlayers.cpp:529
jl_compile_method_internal at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gf.c:2481 [inlined]
jl_compile_method_internal at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gf.c:2368
_jl_invoke at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gf.c:2887 [inlined]
ijl_apply_generic at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/gf.c:3077
jl_apply at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/julia.h:1982 [inlined]
jl_f__call_latest at /cache/build/builder-amdci4-0/julialang/julia-release-1-dot-10/src/builtins.c:812
#invokelatest#2 at ./essentials.jl:892 [inlined]

@ppalmes that specific issue has just been fixed by #544

https://github.com/JuliaPy/PythonCall.jl/pull/544 fixes this as well.
I think this is probably a good reason to add some variant of #461 and other integration tests to help prevent this type of issue, which otherwise goes unnoticed by a test suite containing only unit tests.


Bo hebrew - israel

Please add Hebrew language (israel).
Also cant search apps in the search bar.

Closed with https://github.com/Flo354/XposedAppLocale/commit/1484a1562d77e2f090dbdb4bb5a69f50e30112f5


Choosing PUT over Patch

Hi,
I have a problem with my update. Regardless of my PreferredUpdateMethod Simple OData always use PUT to update my model on specific records.
Below is my Model in my Client Application that uses the Simple OData Client
public partial class subcategory
{
[System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage","CA2214:DoNotCallOverridableMethodsInConstructors")]
    public subcategory()
    {
         this.inventories = new HashSet<inventory>();
    }
    public string sbc_ccode { get; set; }
    public string sbc_code { get; set; }
    public string sbc_name { get; set; }
    public string sbc_createby { get; set; }
    public Nullable<System.DateTime> sbc_createdate { get; set; }
    public string sbc_updateby { get; set; }
    public Nullable<System.DateTime> sbc_updatedate { get; set; }
    public Nullable<sbyte> sbc_Status { get; set; }
    public Nullable<sbyte> sbc_migration { get; set; }
    [NotMapped]
    public Nullable<System.DateTime> sbc_logdatetime { get; set; }
    [NotMapped]
    [System.Diagnostics.CodeAnalysis.SuppressMessage("Microsoft.Usage", "CA2227:CollectionPropertiesShouldBeReadOnly")]
    public virtual ICollection<inventory> inventories { get; set; }
}

Here's the value of the above model where Simple OData Client selects PUT over Patch to update.
inventories: Count = 0
sbc_Status: 1
sbc_ccode: "00001"
sbc_code: "00002"
sbc_createby: "100000"
sbc_createdate: {6/16/2016 10:27:17 AM}
sbc_logdatetime: null
sbc_migration: 0
sbc_name: "NONE"
sbc_updateby: "100022"
sbc_updatedate: {4/15/2017 1:17:40 PM}

However  the below value of the above model was saved successfully and Simple OData selects Patch over Put to update.
inventories: Count = 0
sbc_Status: 0
sbc_ccode: "00001"
sbc_code: "00001"
sbc_createby: "100001"
sbc_createdate: {5/4/2016 12:00:00 AM}
sbc_logdatetime: null
sbc_migration: 0
sbc_name: "NONE"
sbc_updateby: "100022"
sbc_updatedate: {4/15/2017 1:17:47 PM}

Those values were retrieved from the server with the below command.
var subcatClient = await cloud.For<subcategory>()
                   .Filter(f => f.sbc_logdatetime >= ttd.downloaddate)
                   .FindEntriesAsync();

The first record was saved successfully using PATCH, but I don't know why it fails on the second record  and Simple OData Client is selecting PUT to update the second record. The error that I'm getting is with the sbc_logdatetime. I'm using MySQL and the sbc_logdatetime has a default value of CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP and DateTime data type.
Any idea why it's behaving that way? I thought Simple OData will no longer select PUT if I set the Update method to PATCH and I don't really quite understand why one record is being saved using Patch and the other using Put when they both came from one collection and untouched?
Here's the Inner Exception
Modifying a column with the 'Identity' pattern is not supported. Column: 'sbc_logdatetime'. Table: 'ServiceModel.Store.subcategory'.
Here is my code for updating the record.
var subcatUpdate = await LAN.For<subcategory>()
                        .Key(subcat.sbc_code)
                        .Set(subcat)
                        .UpdateEntryAsync();

And I have this setup above.
var _settings = new ODataClientSettings();
_settings.IgnoreUnmappedProperties = true;
_settings.PreferredUpdateMethod = ODataUpdateMethod.Patch;
_settings.BaseUri = new Uri(CloudUri);

var _lanSettings = new ODataClientSettings();
_lanSettings.IgnoreUnmappedProperties = true;
_lanSettings.BaseUri = new Uri(LANUri);
_lanSettings.PreferredUpdateMethod = ODataUpdateMethod.Patch;

var cloud = new ODataClient(_settings);
var LAN = new ODataClient(_lanSettings);

Here's the complete update code for that specific Model.
var subcatClient = await cloud.For<subcategory>()
.Filter(f => f.sbc_logdatetime >= ttd.downloaddate)
.FindEntriesAsync();

if (subcatClient.Count() > 0)
{
   DispatcherService.BeginInvoke(() =>
   {
       ttd.updaterecords = subcatClient.Count();
       ttd.finishflag = Utility.ProcessFlag.Downloading;
   });

   foreach (subcategory subcat in subcatClient)
   {
       try
       {
           var subcatUpdate = await LAN.For<subcategory>()
                    .Key(subcat.sbc_code)
                    .Set(subcat)
                    .UpdateEntryAsync();
           success = true;
       }
       catch (Exception ex)
       {
           try
           {
               var subcatInsert = await LAN.For<subcategory>()
                  .Set(subcat)
                  .InsertEntryAsync();
            }
            catch (Exception e)
            {
                success = false;
                DispatcherService.BeginInvoke(() =>
                {
                    ttd.finishflag = Utility.ProcessFlag.Error;
                 });
             }
       }
       finally
       {
            if (success)
            {
                DispatcherService.BeginInvoke(() =>
                {
                    ttd.downloadprogress++;
                 });
             }
        }
        if (!success || CanDoStartClick)
            break;
    }
}
else
{
    DispatcherService.BeginInvoke(() =>
    {
        ttd.finishflag = Utility.ProcessFlag.NoUpdate;
    });
}

Any help would be highly appreciated.
Thanks

I can think of the case when a record being updated has an updated field that is a part of its key. PATCH can only be applied to an update with no key data modified. For example, if you have a record with compound key Column1 + Column2 and the updated payload contains modified value for Column2, then the library has no choice but run PUT on the update. Can you verify that none of key data is affected by the update?

Hi Vagif,
I only have sbc_code as the primary key, but your comments gave me an idea that there could be something wrong with my Web API and upon checking I found out that the patch sent to the Web API actually uses sbc_ccode as the primary key and in WebApiConfig I actually have this code.
builder.EntitySet<subcategory>("subcategories").EntityType.HasKey(k => k.sbc_ccode);

After replacing with the correct primary key field now everything works well.
Thank you for your help and my apology.
Dennis


Return type of EntityRepository.persist()

Describe the bug
The documentation specifies that the return type of EntityRepository.persist() is Promise<void>, however, the type is on EntityManager that EntityRepository inherits from is void.
Expected behavior
persist should return a promise or the documentation should be updated.
Additional context
I believe the documentation is out of date, since flush is what marks the end of the transaction. If that's the case, I can put in a PR to update the documentation.
Versions

Dependency
Version

node
12.18.2

typescript
4.2.4

mikro-orm
4.5.3

your-driver
postgresql

That whole page should go, API docs are now generated from code and elsewhere (link in the header).