es6 function import() not working in function render()

Version
2.5.16
Reproduction link
https://github.com/kpeng2016/testVueProject
Steps to reproduce

git the testvueproject
start the project by "npm run dev"
open the project by browser

What is expected?
show text "this is test page" in the page
What is actually happening?
empty

I wish it will work if I use es6 import() in vue render() function to import dynamical file.
Thank you very much!

render function is synchronous. You can't dynamically import something inside it.


Added pe.version_info_list with the list of all version info records

pe.version_info is stored in YARA just as a dictionary (hash table). However it keeps track only of the very last key, value pair it sees so if there are more keys A with values B and C, only C is going to be available in YARA.
Because of mentioned reason these were added:

pe.number_of_version_infos : Number of entries in pe.version_info_list
pe.version_info_list : Array of pairs (key, value) containing version infos

Sample: 00bb7a440abacc9d9350f1641fe4a9705246642838b927614cfc5cbe20c89751
Mentioned sample contain multiple LanguageId version info records:

LanguageId: userdefault
LanguageId: ar
LanguageId: bg
...

For a complete list of version information records, see this gist.

We'd really welcome this PR to reach upstream since we have some troubles scanning such samples with the current approach to pe.version_info. So +1.

@xbabka01 Would you mind making the corresponding change to the documentation?

@xbabka01 Would you mind making the corresponding change to the documentation?

Documentation for pe.version_info_list added


And I say PHP is the best language all through the world, nobody will disapprove，is it right ?

unha

Python says nothing and just smiles.

Java says nothing and just smiles.

Hack says nothing and just smiles.


Uncaught (in promise) Error: Expected a value of type 'Object', but got one of type 'Null'

When below code is added
`import 'package:flutter/material.dart';
import 'package:url_strategy/url_strategy.dart';
void main() {
setPathUrlStrategy();
runApp(App());
}
class App extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'vipatra',
theme: ThemeData(
primarySwatch: Colors.blue,
),
initialRoute: '/',
routes: {
'/': (context) => HomePage(),
},
);
}
}
class HomePage extends StatefulWidget {
@override
_HomePageState createState() => _HomePageState();
}
class _HomePageState extends State {
@override
Widget build(BuildContext context) {
return Container(
child: Text('hi'),
);
}
}
`
it shows this error (below)

and this code opens up

Steps to Reproduce

Add below code and press ctrl +F5.
`import 'package:flutter/material.dart';
import 'package:url_strategy/url_strategy.dart';

void main() {
setPathUrlStrategy();
runApp(App());
}
class App extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'vipatra',
theme: ThemeData(
primarySwatch: Colors.blue,
),
initialRoute: '/',
routes: {
'/': (context) => HomePage(),
},
);
}
}
class HomePage extends StatefulWidget {
@override
_HomePageState createState() => _HomePageState();
}
class _HomePageState extends State {
@override
Widget build(BuildContext context) {
return Container(
child: Text('hi'),
);
}
}
`

And click resume.

Expected results:
No errors
Actual results:
When I click ctrl + F5 it pauses and if i resume it shows all those errors and the below file opens up.

Logs

[  +89 ms] executing: [C:\Users\vignesh\flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +71 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[   +1 ms] b1395592de68cc8ac4522094ae59956dd21a91db
[   +1 ms] executing: [C:\Users\vignesh\flutter/] git tag --points-at b1395592de68cc8ac4522094ae59956dd21a91db
[  +58 ms] Exit code 0 from: git tag --points-at b1395592de68cc8ac4522094ae59956dd21a91db
[   +1 ms] 2.0.4
[  +60 ms] executing: [C:\Users\vignesh\flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +46 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[   +2 ms] origin/stable
[   +1 ms] executing: [C:\Users\vignesh\flutter/] git ls-remote --get-url origin
[  +47 ms] Exit code 0 from: git ls-remote --get-url origin
[   +2 ms] https://github.com/flutter/flutter
[ +299 ms] Unable to locate an Android SDK.
[   +6 ms] executing: [C:\Users\vignesh\flutter/] git rev-parse --abbrev-ref HEAD
[  +56 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[  +58 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[   +4 ms] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +124 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +136 ms] Multiple devices found:
[  +30 ms] Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.128
[   +2 ms] Edge (web)   • edge   • web-javascript • Microsoft Edge 89.0.774.76
[   +2 ms] [1]: Chrome (chrome)
[        ] [2]: Edge (edge)
[   +3 ms] Please choose one (To quit, press "q/Q")
[   +1 ms] :

Analyzing dashboard...
No issues found! (ran in 3.4s)

[√] Flutter (Channel stable, 2.0.4, on Microsoft Windows [Version 10.0.19042.928], locale en-IN)
    • Flutter version 2.0.4 at C:\Users\vignesh\flutter
    • Framework revision b1395592de (2 weeks ago), 2021-04-01 14:25:01 -0700
    • Engine revision 2dce47073a
    • Dart version 2.12.2

[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[√] VS Code (version 1.55.2)
    • VS Code at C:\Users\vignesh\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.21.0

[√] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.128
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 89.0.774.76

! Doctor found issues in 2 categories.

@vigneshksaithal
url_strategy is a 3rd party plugin, please open an issue in the dedicated github
or provide  a minimal reproducible code sample without 3rd party plugins

Duplicate of #78017 and this issue solved this error.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/288 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 5a3d3f1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

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


Typo "Azure Open AI"→"Azure OpenAI"

https://learn.microsoft.com/en-us/azure/machine-learning/prompt-flow/transparency-note?view=azureml-api-2
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/machine-learning/prompt-flow/transparency-note.md
#PingMSFTDocs

Thanks so much @hyoshioka0128


README: Add PSC's in charge

I don't know if it's possible, but this could be a help for contributors to have PSC's name and links (to odoo-community.org) in repos README.

@OCA/core-maintainers

Makes sense to me.
Could be an additional question for copier.

The information is in the OCA instance. Dont know how to recover it to générate readme...

The information is in the OCA instance. Dont know how to recover it to générate readme...

Yes that's a point I don't have answer.

isn't https://github.com/OCA/maintainer-tools/blob/master/tools/oca_projects.py where the info about PSCs/repos lives?

I suggested to keep it simple, and make it a copier question.

It can be done by a controller in https://odoo-community.org, where all that information is already stored.
For example, Copier could just put a link to https://odoo-community.org/psc-teams/find-by-repo?repo=OCA/project in the readme which just makes a 302 redirection towards https://odoo-community.org/psc-teams/project-service-28.

Any update on this? I resorted to this workaround 😅

I suggested to keep it simple, and make it a copier question.

Not sure it's simple if we duplicate the information in two places. (in odoo-community + in copier answers file). There will be harder to maintain, don't you think ?

Good news: we are going to manage repos via github actions. I'm sure we can handle this too.
We are going to have a talk during OCA days but if you want to have a look at the prototype, here you go https://github.com/OCA-devel/repo-maintainer :wink:

Why having a different organization for such repo?

Why having a different organization for such repo?

It's temporary. We needed a sandbox to play with. We'll move what's needed to OCA when ready.


Register

@JuliaRegistrator register()

Error while trying to register: "Pre-release version not allowed"

@JuliaRegistrator register()

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/115945
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" f643fc8357a39282679ce8bd2ce68cb3e222786a
git push origin v1.0.1

Also, note the warning: This looks like a new registration that registers version 1.0.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.

@JuliaRegistrator register

Error while trying to register: Unexpected error in registration

@JuliaRegistrator register

Error while trying to register: Unexpected error in registration

Registration pull request updated: JuliaRegistries/General/115945
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.0.1 -m "<description of version>" efbc4c108192a58c95505f4ed6c098868f2e6a9d
git push origin v1.0.1

Also, note the warning: This looks like a new registration that registers version 1.0.1.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.


can i add nested wizards?

I need nested wizards, is this possible?
And is it possible that the nested wizard is dynamic, i mean:
main wizard: step1 | step2 | step3
then, step 2 would have a nested wizard inside like subtab1 | subtab2 ...
This substeps would be created dinamically in step1 but instead of being added to the main wizard they would be added to nested wizard, like this:

I've readed the doc and i can add tabs dinamically, but i can't find anything about nested ones.

Has this been merged into master so that its available in latest version download ?

Has this been merged into master so that its available in latest version download ? Can we get a demo?

@grandstylus16 this example does not work?
https://jsfiddle.net/7jkt6/6/

it works but I wanted for the nested step to work with the main step previous/next button and also to not show a second pagination header

You can check current step on onStepChanged for nested wizard. And if
true, process it.
You can disable pagination, see options.
On Thursday, August 20, 2015, grandstylus16<EMAIL_ADDRESS>wrote:

it works but I wanted for the nested step to work with the main step
previous/next button and also to not show a second pagination header
—
Reply to this email directly or view it on GitHub
https://github.com/rstaib/jquery-steps/issues/23#issuecomment-132793033.

--
С уважением,
Хандрамай Павел Александрович


Update va_code.js

Discovered this bug while running tests on linkify-citations.

Gah! Apparently I have no tests for the id field!  I added tests in e43ee58 and verified that your changes were the difference between their passage and failure.
Thanks, @vzvenyach!


Node test failing: Call propagation Cancellation With a client stream call

Error message:
Error: timeout of 8000ms exceeded. Ensure the done() callback is being called in this test.
There doesn't seem to be a stack trace. Logs.

Dupe of #5294.


LightGBM doesnt have hyper-parameters as attributes in the java object

Is your feature request related to a problem? Please describe.
Currently it seems the hyper-parameters of LightGBM models do not exist as attributes in the java object.
When I fit a model
lightgbm_classif = LightGBMClassifier(
featuresCol='features',
isUnbalance=True,
labelCol='label',
)
lgbm_model = lightgbm_classif.fit(train_data)

When I retrieve the java obj and try to get learningRate
lgbm_java = lgbm_model._java_obj
lgbm_java.getParam("learningRate")

Got error:
java.util.NoSuchElementException: Param learningRate does not exist.
Describe the solution you'd like
I think it would be helpful to attach the hyper-parameters as attribute to the model object. I think after hyper-parameter tuning we would like to get the best set of hyper-parameters. Currently for LightGBM models, either doing thru the CrossValidator or TuneHyperparameters, I can't extract best set of parameters for LightGBM models.
Additional context
N/A

@victorconan sorry about the issue you are encountering, I think this issue might actually be fixed on latest master but will need to check, this PR may have (hopefully?) fixed this:
https://github.com/Azure/mmlspark/pull/864


Updated template service to use templateEngine on options object

I recently updated to version 1.4.4 and encountered an error that seems to be introduced when you streamlined the passing of the options object to the templateEngine.  The error is:
node_modules/campaign/src/templateService.js:37
            options.engine.renderString(template, model, getCallback(model, do

It looks like the correct call is options.templateEngine.render or renderString based on the previous inputs.  I verified the error in your tests and then updated the code and retested with your tests passing.
Thanks for a great client, appreciate the hard work!

Thanks. Published as 1.5.0 on npm

Awesome, thanks for the quick turnaround.


[Demo] Subtotal calculates a negative amount.

What I'm trying to achieve
If a user will add "-" before the selected quantity in the product view then ADD TO BASKET button becomes active. After clicking on it the amount charged will be reduced by the quantity of negative products added.
Steps to reproduce the problem

Go to https://demo.saleor.io/
Open random product, e.g. https://demo.saleor.io/product/light-speed-yellow-paint/64/
Select size e.g. 5l
Fill Quantity with e.g. 50 and click ADD TO BASKET
Check MY BAG and SUBTOTAL
Once more fill the Quantity with e.g. 200 and then add a minus before the number.
Click on the active button ADD TO BASKET
Open MY BAG
Look at the SUBTOTAL
Click on the Checkout
Click on the CONTINUE AS A GUEST
Fill Email field and click on the CONTINUE TO SHIPPING

Look at the last Response of Request in DevTools - attachments added.
What I expected to happen
It should be not possible to add negative products.
Screenshots

System information
Operating System:
Ubuntu 18.4
Browser:
Firefox

Issue moved here since this is a frontend issue.


Running wurstscript.exe does nothing

I'm in the process of helping @Cokemonkey11 with adding WurstScript to WEX(#529), and I've hit a problem. When I run wurstscript.exe nothing happens, here are the things I've tried

Running wurstscript.exe in cmd/powershell
Running wurstscript.exe -gui in cmd/powershell
Running wurstscript.exe <absolute path to common.j> <absolute path to blizzard.j> <absolute path to map>

In all cases, nothing happens. The process starts and immediately closes with no output and with an exitcode of 0.
I've tried running wurstscript_b.exe to see if it behaved differently, and here I got an error about being unable to connect to the server. This leads me to believe I might be missing a dependency which is only used in wurstscript.exe.
I'm running Windows 10 with the most recent updates and I have JRE x86 and JRE x64 installed. I have not modified any environment variables, just installed both JREs and then tried using wurstscript.exe.

Also he has no debug logs in AppData\Local\Temp\wurst\.
One further diag: I suspect he's using the x86 JRE since wurstscript was previously complaining that he was missing java, so that's a fresh install and cleared that problem.

@Frotty can you take a look when you wake up

Works perfectly as expected.

make sure you have Java 8 installed and set JAVA_HOME correctly.

I have Java 8 and running wurstscript.exe --about does nothing. Do I have to set JAVA_HOME manually?

The jars were missing, resulting in a silent error. An improvement would be to detect this.

While I agree that an error could help, unless you want to build on your own, you would never download the source repo, so the .jars would always be there for a normal user that follows instructions.

I think your answer leaves opportunity for discussion, but better more definitive one is probably just "we should delete wurstscript.exe since wurstpack is not supported anymore"

nope.

How did you achieve that?

by not giving up after first attempt


[UTXORPC] FollowTip function fails to return Tip when only Hash is passed

cardano-node-api:local (gouroborus:v0.90.0)
cardano-node:8.9.4
https://github.com/utxorpc/go-sdk
network: preprod
followTip(ctx, client, "235f9a217b826276d6cdfbb05c11572a06aef092535b6df8c682d501af59c230", -1, nil)
Server logs:
2024/07/18 13:00:28 Got a FollowTip request with intersect [hash:"#\x9a!{\x82bv\xd6\xcd\xfb\xb0\\x11W*\x06\xae\xf0\x92S[m\xf8Ƃ\xd5\x01\xafY\xc20"]
2024/07/18 13:00:28 Intersect: [hash:"#\x9a!{\x82bv\xd6\xcd\xfb\xb0\\x11W*\x06\xae\xf0\x92S[m\xf8Ƃ\xd5\x01\xafY\xc20"], Len intersect: 1
2024/07/18 13:00:28 BlockRef: idx: 0, Blockhash: 235f9a217b826276d6cdfbb05c11572a06aef092535b6df8c682d501af59c230
2024/07/18 13:00:28 Starting point: slot: 0, hash: 235f9a217b826276d6cdfbb05c11572a06aef092535b6df8c682d501af59c230
2024/07/18 13:00:28 ERROR: chain intersection not found
followTip(ctx, client, "5f929015ab29faaa4ad779c6b52ff3fb5a7a4b218e0f154b441dc7023275ca4a", -1, nil)
2024/07/18 13:04:55 Attempting to call Utxorpc on block
2024/07/18 13:15:42 Got a FollowTip request with intersect [hash:"\x92\x90\x15\xab)\xfa\xaaJ\xd7yƵ/\xf3\xfbZzK!\x8e\x0f\x15KD\x1d\xc7\x022u\xcaJ"]
2024/07/18 13:15:42 Intersect: [hash:"\x92\x90\x15\xab)\xfa\xaaJ\xd7yƵ/\xf3\xfbZzK!\x8e\x0f\x15KD\x1d\xc7\x022u\xcaJ"], Len intersect: 1
2024/07/18 13:15:42 BlockRef: idx: 0, Blockhash: 5f929015ab29faaa4ad779c6b52ff3fb5a7a4b218e0f154b441dc7023275ca4a
2024/07/18 13:15:42 Starting point: slot: 0, hash: 5f929015ab29faaa4ad779c6b52ff3fb5a7a4b218e0f154b441dc7023275ca4a
2024/07/18 13:15:42 ERROR: chain intersection not found
Intersect length check
https://github.com/blinklabs-io/cardano-node-api/blob/main/internal/utxorpc/sync.go#L191

Closing, it works as expected.


Resolution of https://github.com/xlab-si/xopera-examples/issues/31

Looks good, thanks @philippemerle!


fixes 3D extension backend mixup

This PR fixes some copy paste errors in the 3D extensions for both CUDA and AMDGPU
@albert-de-montserrat: Does AMDGPU need to be a deps in the Project.toml or can it also be moved to a weakdeps?

It should be a weakdep because its an extension

Codecov Report
Attention: Patch coverage is 0% with 22 lines in your changes are missing coverage. Please review.

Files
Patch %
Lines

src/ext/AMDGPU/3D.jl
0.00%
14 Missing :warning:

src/ext/CUDA/3D.jl
0.00%
8 Missing :warning:

Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!

Do we actually need it as weakdeps? Extension would not require it as weakdeps, unless you maybe want to control versions...

I guess so. This is from the Julia docs:

A package "extension" is a module that is automatically loaded when a specified set of other packages (its "extension dependencies") are loaded in the current Julia session. Extensions are defined under the [extensions] section in the project file. The extension dependencies of an extension are a subset of those packages listed under the [weakdeps] section of the project file. Those packages can have compat entries like other packages.


Budget row height pacing fix

Simple fix for #116, where the pacing column numbers weren’t properly
centered when using slim view.

* {
  merge-request: !important;
}


Updates for purescript-aff-4.0.0

Thanks! Could you tweak the package.json to inclue -- before the psa arguments to pulp too? That'll fix the build error (unrelated to this change) that is occurring on Travis.

Done.

Thanks!


test: scenarios-feature_branch_happy_path (2024-03-06 22:05:11.325981+00:00)

automatic test cleanup


Support Ignores for Type-Checking Blocks

Right now there's the ability to exclude specific submodules, like tests:
exclude:
- tests
- docs

It would additionally be helpful to exclude all imports under an if TYPE_CHECKING block as not being counted against breaking module boundaries. importer-linter is an example of a tool that supports this.
from module_a import ...

if TYPE_CHECKING:
   from module_b import ... # Boundary for module_b is not violated

This is included in 0.1.8!
Enable it with ignore_type_checking_imports: true in your tach.yml


Language server memory leakage

First of all, thank you for bringing this greak tool!! I love it!
BUT,
there seems a memory leakage caused by language serve, as shown below:

My system is Ubuntu 20.04 and vscode information is listed in the following:

I need to restart the vscode every two hours, to avoid a system crash now..

Sorry some Chinese characters are in the image.. I try to translate them here:
version: 1.92.1
Submission: eaa41d57266683296de7d118f574d0c2652e1fc4
date: 2024-08-07T20:16:19.455Z ( 6 days ago)

same to me. Has it been solved?

same to me. Has it been solved?

Sadly, not yet.
An oberservation is not opening any big library (like pyqt or CV2) may avoid this. I do not understand.

This repository is for the vim extension, not VS Code.

This repository is for the vim extension, not VS Code.

Oh, sorry I made a mistake. I close it now.


No module named torch_utils.engine error while training

Hello, I have been having some trouble training with the smoke custom dataset and I keep receiving the error below.
Traceback (most recent call last):
File "train.py", line 10, in
from torch_utils.engine import train_one_epoch, evaluate
ImportError: No module named torch_utils.engine
The problem seems to have risen from the python version match, and I could not find exactly which python version is used for this whole code. The version in my enviroment is 3.10.4.
Thank you.

@Bleking
The Python version seems to be fine. The code supports Python 3.10.x.
Were you able to successfully install all the requirements according to instructions?
Also, it seems that is unable to locate the torch_utils.engine module which is odd. This is pretty odd. Can you share the following things:

PyTorch version.
Torchvision version.
OS (Ubuntu/Windows)
The complete command that you used.

torch 1.12.0
torchvision 0.13.0
I ran it on MobaXTerm (Linux environment)
python train.py --config data_configs/smoke.yaml --epochs 100 --model fasterrcnn_resnet50 --project-name smoke_training --batch-size 16
I used the very default command for the smoke dataset that you have provided us.

If you are using Linux environment with CUDA already installed globally, please do pip install -r requirements.txt. Right now, the latest version of PyTorch, that is, PyTorch 1.12.0 is not supported because of some pretrained weight changes.


Create my-first-branch

I have created a new branch, created a file, and made a commit!

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] A subject matter expert (SME) has reviewed the technical accuracy of the content in this PR. In most cases, the author can be the SME. Open source contributions may require a SME review from GitHub staff.
[x] The changes in this PR meet the docs fundamentals that are required for all content.
[x] All CI checks are passing.

PROFILE.md

PROFILE.md


Launching not working as expected?

Trying to launch a Rails/Sinatra app as described here:
https://github.com/isaiah/jubilee/tree/master/examples/chatapp
When launching using vertx run...
Here's my config.ru:
require ::File.expand_path('../config/environment',  __FILE__)
run Machineshop::Application

And application.rb:
require File.expand_path('../boot', __FILE__)
require 'sunspot'
require "action_controller/railtie"
require "action_mailer/railtie"
require "active_resource/railtie"
require "rails/test_unit/railtie"
require "sprockets/railtie"
require 'bundler/setup'
require 'vertx'
...

Any thoughts? What am I missing?

Scratch that...got around the issue by requiring vertx before others. Now I'm running into the same thing as https://github.com/isaiah/jubilee/issues/19. Not sure how to implement the solution outlined there. Help?

It is. Thank you.

John Cox

On Feb 3, 2015, at 9:37 AM, Isaiah Peng<EMAIL_ADDRESS>wrote:
@johncox00 Is it solved?
—
Reply to this email directly or view it on GitHub.


win: implements uv__udp_try_send

This PR implements the missing uv__udp_try_send method for Windows.

Thanks @cjihrig for the feedback, fixed them.

Any Windows people care to give this a review?

Maybe you should enable the udp-try-send test in this PR?

@txdv He already did.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/210/

@gemabarni ping! :)

It looks like @supercairos has picked this up. Moving to #1385.


Provide API for host to force progress from finality certificates

If a node gets really stuck, we expect the host to fetch finality certificates and use those to force a node up to a new instances without passing through the full protocol. We need some API capability for this; it might also overlap with initial bootstrapping.
Since validating a finality certificate depends on previous certificates, probably the host will do the validate then just tell the GBPFT participant where to skip to (and provide the necessary history of power tables for it to validate messages). But other orchestrations might be possible too.
FYI @Kubuxu

cover #245?


Evaluation error: KeyError: 'categories'

100%|██████████| 1322/1322 [22:19<00:00,  1.01it/s]Loading and preparing results...
Traceback (most recent call last):
  File "/home/oliver/PycharmProjects/SLN-Amodal/amodal_eval_COCOA.py", line 670, in <module>
    evalute_amodal(dataset_val, model, limit=args.limit,args=args)
  File "/home/oliver/PycharmProjects/SLN-Amodal/amodal_eval_COCOA.py", line 440, in evalute_amodal
    coco_results = amodalGt.dataset.loadRes(results)
  File "/home/oliver/PycharmProjects/SLN-Amodal/pycocotools/coco.py", line 324, in loadRes
    res.dataset['categories'] = copy.deepcopy(self.dataset['categories'])
KeyError: 'categories'

Bugs report.

I was using the COCO api download from official github repository.  But I suddenly find that your modified version is just in your repo, and it's correct.
Maybe it's better to make it be more clear in README.
Thank you for your work and the codes!


Is this done through Flamite? As when login just redirected to page saying Flamite is no more...

As above

delete flamite.


[Proposal] Limit the memory usage of Compaction

With the widespread use of the new load framework, existing compaction strategies are no longer work in some scenarios. This document focuses on the problems that the new load framework brings to the compaction logic and how to improve it.
Problem
In the new load framework, the load data forms a serial of Memtables in memory. When the size of a memtable reaches the threshold (default is 100MB), it will be written to the disk to form a Segment. A batch of load is corresponding ti a Version. When a batch of loaded data is relatively large, or a row of a table is large, a batch of load may generate thousands of segments.
In the compaction logic, at least one version is selected for one compaction. Compaction is an external sorting that will open a RowBlock for each segment, with 1024 rows per RowBlock. So a RowBlock occupies a memory size of (1024 * row size).
Assuming that a Compaction has 1000 Segments and each row is 4K in size, RowBlock will take up 4G memory. When multiple Compactions are running at the same time, the system OOM may be caused.
Solution
This proposal is to ensure that Compaction can run stably with less memory by estimating and limiting the amount of memory used by Compaction. This work is divided into the following three steps.
Compaction ratio statistic
To estimate the amount of memory used by a Compaction, it is mainly to estimate the size of a row in memory. We can simply use the ratio of the size of a memtable in memory to the size of file it is written on disk as the compaction ratio. With this ratio, the size of the data file on the disk, and the number of rows in file, we can calculate the approximate occupancy of a single row of data in memory.
Supported compaction within a version
Currently only Compaction with at least one version is supported. And if there are too many Segments in a single version, it still consumes a lot of memory. So we need to support compaction with a subset of segments with a single version.
Limiting Compaction memory usage
With the previous two steps, it has been possible to estimate and limit the memory usage of a single Compaction. Finally, we need an overall limit to ensure that the memory overhead can be within a reasonable range when multiple Compactions are running at the same time.

Compaction ratio statistic

To estimate the amount of memory used by a Compaction, it is mainly to estimate the size of a row in memory. We can simply use the ratio of the size of a memtable in memory to the size of file it is written on disk as the compaction ratio. With this ratio, the size of the data file on the disk, and the number of rows in file, we can calculate the approximate occupancy of a single row of data in memory.

Why not estimate average row length by computing ratio of write bytes to number of rows when generating a rowset. And this value can be recorded in rowset meta.

Supported compaction within a version
Currently only Compaction with at least one version is supported. And if there are too many Segments in a single version, it still consumes a lot of memory. So we need to support compaction with a subset of segments with a single version.

It will be better to merge enough segments into a bigger segment when loading other than leaving it when compaction. 1. This is simple for compaction logic; 2. this is good for read. Even if we have 1000 segments for a rowset, it will lead to terrible read performance.

Why not estimate average row length by computing ratio of write bytes to number of rows when generating a rowset. And this value can be recorded in rowset meta.

Do you mean the memtable size / num of rows?

It will be better to merge enough segments into a bigger segment when loading other than leaving it when compaction

I am afraid it will increase the mem consumption of a load process. And the concurrence of loading is much larger than compaction. If doing this when loading, we need to rethink about how to limit the memory of loading.

Why not estimate average row length by computing ratio of write bytes to number of rows when generating a rowset. And this value can be recorded in rowset meta.

Do you mean the memtable size / num of rows?

No. Now when write a rowset, RowsetWriter will estimate how how many bytes to decide if it will generate a segment. We can use this value divided by row count as average row size.

I am afraid it will increase the mem consumption of a load process. And the concurrence of loading is much larger than compaction. If doing this when loading, we need to rethink about how to limit the memory of loading.

what I want to do has no effect for current load process. It will be done before we add this rowset to StorageEngine. If we found there are too many number of rowsets, we can try to merge some of them to a bigger rowset. Actually we can do it for all load operation, because it will improve our read performance.
Even though we can do this, but I think it is not very urgent. For you example, when a segment has 1000 segment, the rowset size will be 100GB(1000 * 100MB), it will lead to other serious problem. Such as tablet balance, read performance. So I think for this proposal, you can leave this case aside.

what I want to do has no effect for current load process. It will be done before we add this rowset to StorageEngine. If we found there are too many number of rowsets, we can try to merge some of them to a bigger rowset. Actually we can do it for all load operation, because it will improve our read performance.

I think the motivation for compaction within a rowset is to reduce the number of overlapped segments and improve query performance.  However when the number of segments is large, single round of compaction would consume too much memory. So we need to decide how many number of segments to compact at a time based on estimated RowBlock size and available memory.
My question is the previous compaction strategy will generate a rowset with a new version (like 0-6) to replace input rowsets with overlapped version, but when doing compaction within a rowset, you end up with two rowset with the same version. Is it a problem? And if it is, how do you plan to solve it?

what I want to do has no effect for current load process. It will be done before we add this rowset to StorageEngine. If we found there are too many number of rowsets, we can try to merge some of them to a bigger rowset. Actually we can do it for all load operation, because it will improve our read performance.

I think the motivation for compaction within a rowset is to reduce the number of overlapped segments and improve query performance. However when the number of segments is large, single round of compaction would consume too much memory. So we need to decide how many number of segments to compact at a time based on estimated RowBlock size and available memory.
My question is the previous compaction strategy will generate a rowset with a new version (like 0-6) to replace input rowsets with overlapped version, but when doing compaction within a rowset, you end up with two rowset with the same version. Is it a problem? And if it is, how do you plan to solve it?

In my point of view, we should create another type of operation to generate a new rowset from rowset with many segments other than compaction. And this operation's input and output are segments, then this will works fine, because Doris already have many segments which are not sorted in one rowset.

After the research, most of the memory overhead was concentrated on opening the segments(RowBlock) and creating the Readers(StreamBuffer). And this part of the memory overhead is strongly related to the number of columns of data and the column type. And we can't accurately estimate the size of a row of data in memory at this time, so we can't estimate the memory required for a compaction.
Compaction within a rowset is a way to reduce the memory, but the root cause still exists, which is: How many segments should we selected for a compaction by given certain amount of memory. And this requires the statistic of size of a row of data in memory.
I will leave this Issue opened and try to resolve it sometimes later.


Implement function summarize(text)

Engshell fails with text data larger than the maximum prompt size. Need a non-GPT based summarize(text) function for processing large text data.

Why must it be non GPT based? Wouldn't recursively summarizing chunks of the conversation using the gpt3.5 chat endpoint be sufficient?

Why must it be non GPT based? Wouldn't recursively summarizing chunks of the conversation using the gpt3.5 chat endpoint be sufficient?

It's much better to vectorize large texts using the ADA model and then process it from there. Much cheaper and faster.
"This model's maximum context length is 4097 tokens. However, your messages resulted in 685229 tokens. Please reduce the length of the messages."

Getting the embeddings of the large text doesn't help to summarize it as you can't usefully convert the embeddings back to text. What do you mean by "process it from there"?

Why must it be non GPT based? Wouldn't recursively summarizing chunks of the conversation using the gpt3.5 chat endpoint be sufficient?

I believe chunking the text and then doing this would work as well, yes.


hrls will not start after upgrade to version 8 of mysql-connector-java

In #53 the mysql-connector-java version is updated from 5.1.x to 8.0.y
This causes the following issue. The hrls servlet will not load and gives the following error:
Caused by: com.mysql.cj.exceptions.InvalidConnectionAttributeException: The server time zone value 'CEST'

In version 8 it needs a timezone to be set: https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-other-changes.html
CEST is just not known.
This can be fixed by adding a timezone to the connection string for handlereverselookupservlet.properties . An example being:
sqlConnectionString = jdbc\:mysql\:///master_database?autoReconnect=true&serverTimezone=Europe/Amsterdam

Here following is added:
&serverTimezone=Europe/Amsterdam

Fixed in merged issues.. Close request.


TN-2781 Integrate SDC service

TN-2781

Integrate SDC at service level

Hello @ivan-c! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/models/questionnaire_response.py:

Line 332:80: E501 line too long (87 > 79 characters)
Line 334:9: E265 block comment should start with '# '
Line 334:80: E501 line too long (95 > 79 characters)

In the file portal/tasks.py:

Line 368:80: E501 line too long (93 > 79 characters)
Line 370:5: E265 block comment should start with '# '
Line 370:80: E501 line too long (81 > 79 characters)
Line 374:5: E265 block comment should start with '# '

In the file portal/views/assessment_engine.py:

Line 1580:80: E501 line too long (94 > 79 characters)


changed the design of footer along with background image

Description
changed the background-color of footer ,so that it matches with theme
Fixes:  #554
Type of change

[x ] New feature (non-breaking change which adds functionality)

Checklist:

[ x] I have made this from my own
[x ] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code

[x ] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@myhacktech fix this issue.

make the footer for entire width of the screen.

@myhacktech fix this issue.  make the footer for entire width of the screen.

Ok

@DEEPIKA-SUK please review this...

And yes, keep the same font style same throughout the website and write Maroom in bold the way it is written in navbar

https://deploy-preview-583--cool-front-end-templates.netlify.app/maroom - interior design landing page/


there is no grub in linux machine

Describe the bug
using orbstack Version 1.6.3 (17138)，linux machine is ubuntu 22.04， on mac os（apple arm64）
i need to change cgroup version to v1，but there is no /etc/default/grub，how can i add some kernel boot parameter， like GRUB_CMDLINE_LINUX=“systemd.unified_cgroup_hierarchy=0”
To Reproduce
No response
Expected behavior
No response
Diagnostic report (REQUIRED)
No response
Screenshots and additional context (optional)
No response

OrbStack doesn't use grub. It's very unlikely that we'll support cgroup v1, but you can subscribe to #66 for updates on that.


@Dependabot API ignore major version seemingly not working

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
Maven
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update
pom.xml
dependabot.yml content
version: 2

updates:
- package-ecosystem: github-actions
  directory: "/"
  schedule:
    interval: weekly
    day: "sunday"
  open-pull-requests-limit: 5
  labels:
    - "Type: Dependency Upgrade"
    - "Priority 1: Must"
  milestone: 96
  reviewers:
    - "AxonFramework/framework-developers"
  groups:
    github-dependencies:
      update-types:
        - "patch"
        - "minor"
        - "major"

- package-ecosystem: maven
  directory: "/"
  schedule:
    interval: weekly
    day: "sunday"
  open-pull-requests-limit: 5
  labels:
    - "Type: Dependency Upgrade"
    - "Priority 1: Must"
  milestone: 96
  reviewers:
    - "AxonFramework/framework-developers"
  groups:
    maven-dependencies:
      update-types:
        - "patch"
        - "minor"
        - "major"

Updated dependency
No response
What you expected to see, versus what you actually saw
By using the Dependabot bot ignore rules inside a pull request, I expected that my project does not upgrade dependency that no longer supports JDK8.
As it stands, dependencies of for example Mockito and Spring Boot have been upgraded to a none-JDK8 supporting version for the last 4 weeks.
I was originally led to issue #10798 by @amazimbe.
Although the issue is marked completed, I still noted the unexpected behavior this week and last week.
I replied to the issue referred to last week with this comment, but I figured making an issue would be the right approach.
An example culprit that does not adhere to the ignore rules I have set up is pull request #3182 from the Axon Framework project.
I have an additional 10 or so other repositories that show similar behavior, though.
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
Issue #3182 of the Axon Framework project.
Smallest manifest that reproduces the issue
No response

Just wanted to let all readers know that all repositories I am maintaining still receive dependabot upgrades for major releases that are ignored through the ignore-rules.
Hence, to me, it seems as if #10798 has not been resolved fully yet.
Any form of update on this would be very much appreciated! :pray:

If you remove your current ignore conditions and then add them again it should work. You shouldn't see 5.a in the range anymore as that should be 5.a0

If you remove your current ignore conditions and then add them again it should work. You shouldn't see 5.a in the range anymore as that should be 5.a0

So a "fresh" rule set should do the trick. Thanks for sharing that, @amazimbe! I'll give it a try.

So a "fresh" rule set should do the trick. Thanks for sharing that, @amazimbe! I'll give it a try.

@smcvb did it work?

@smcvb did this work?

@abdulapopoola, it did the trick for some, but not all.
However, what's good to know, is that I moved towards ignore rules in the dependabot.yml instead of the comment-approach discussed above.
To be frank, I find the GitHub comment-based ignore rules rather tiresome to work with.
I was missing future like "give me all existing rules," as well as support to adjust everything in one go.
I comprehend it is tough to provide such a future in a comment-based API, so nothing wrong said about what's there.
The approach just didn't suffice for the amount of rules I had to set.
Hence, I went through the process of checking all dependencies for potentially active ignore rules based on comments, and unignored all of them.
Then, I moved all my rules to the yml file.
Although that solved my personal issue, it also means I cannot state whether the ignore major version support is working as intended fully.
My apologies for that.
From my perspective, it is fine to close this issue, though.

Thanks @smcvb
Yes, we want to improve our ignore experience and hopefully should have this in the new future. I apologize for the current state.

Yes, we want to improve our ignore experience and hopefully should have this in the new future. I apologize for the current state.

Thanks for that, @abdulapopoola, and no worries at all. As a maintainer of several open source repositories, I know all to well the desire for change throughout the sheer amount of tasks in front of you.
If anything, thanks a ton for all the work you (and others!) have already put into this. :pray:


Update documentation after removing AddTemporarySigningCredential - issue with SecurityTokenSignatureKeyNotFoundException

Is your feature request related to a problem? Please describe.
This request is related to documentation and connected to issue i am encountering currently.
I am using test host for API tests, and i've updated IdentityServer4.AccessTokenValidation from 1.2.1 to latest 2.6.0 and IdentityServer4 from 1.2.1 to 2.2.0.  I needed to change setup from:
services.AddIdentityServer()               .AddTemporarySigningCredential() // <-- to change               .AddInMemoryClients(Config.GetClients())               .AddInMemoryIdentityResources(Config.GetIdentityResources())               .AddInMemoryApiResources(Config.GetApiResources())               .AddTestUsers(new List<TestUser>...
to
 services.AddIdentityServer()              .AddDeveloperSigningCredential();  // <-- changed 	     .AddInMemoryClients(Config.GetClients()); 	     .AddInMemoryIdentityResources(Config.GetIdentityResources()); 	     .AddInMemoryApiResources(Config.GetApiResources()); 	     .AddTestUsers(new List<TestUser>...
and this is basically only thing i changed in Test Host setup...
I am hitting the API endpoint in Main project from tests with HTTP client. Main project is also updated with the newest versions. With previous test host setup and old versions, all works fine, but after update and change from AddTemporarySigningCredential()  to AddDeveloperSigningCredential(), i am getting error:
Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler:Information: Failed to validate the token "my-token"
Microsoft.IdentityModel.Tokens.SecurityTokenSignatureKeyNotFoundException: IDX10501: Signature validation failed. Unable to match keys: 'f2084371f0983f0da5c341d6a2235754',
token: '{"alg":"RS256","typ":"JWT","kid":"f2084371f0983f0da5c341d6a2235754"}.{"nbf":1538643011,"exp":1538646611,"iss":"http://localhost","aud":["http://localhost/resources","api"],"client_id":"app","sub":"1","auth_time":1538643011,"idp":"local","company":"companyA","role":"ADMIN","scope":["company","openid","profile","role","api"],"amr":["pwd"]}'.
at System.IdentityModel.Tokens.Jwt.JwtSecurityTokenHandler.ValidateSignature(String token, TokenValidationParameters validationParameters)
at System.IdentityModel.Tokens.Jwt.JwtSecurityTokenHandler.ValidateToken(String token, TokenValidationParameters validationParameters, SecurityToken& validatedToken)
at Microsoft.AspNetCore.Authentication.JwtBearer.JwtBearerHandler.HandleAuthenticateAsync()
Also one more point: when i am hitting Main project endpoint from Postman with correct Bearer, it works fine(Authority in this case is my User Service with old versions implementation), and in another case when running tests, authority is Test Host with new versions. In both cases Main project uses all new versions.
I can't figure out what is going on for around 2 days...
Describe the solution you'd like
I would like to have transparent documentation after every bigger update and to have clear working examples after breaking changes, so i can just update my working code and make it work with new versions.
Hope you can help me solving this issue,
Thanks!

I'm not sure about your kid issue.
Our QS docs will be getting a bit of a reworking soon once we release 2.3.

When do you expect 2.3 release, maybe that will solve my problem?

Check the milestones.

Not sure -- I've seen that when manually validating JWTs and the JwtSecurityTokenHandler doesn't match on kid for some reason (only x5c). I assumed that was done properly in the UseJwtBearer, which is what our AccessTokenValidation  wraps. Can you try with just UseJwtBearer directly and see?
In short, we need a repo to help diagnose. If you can get our samples to exhibit the same problem, then it will be easy for us to sort it out and provide a fix if necessary.

I manged to solve the issue:
I needed to create HttpMessageHandler from TestServer (i am using test host), and then to pass it and register in Web API  Startup so IdentityServer can use this particular handler when tests are running.
So, in configuration, if test are running - use handler from test host, in all other cases use default.
Seems to me it is not a bug in IdentityServer4, just some edge use case. I don't know if there is some kind of explanation of handler usage (test environment) in docs, i couldn't find it.
In any case, thanks for support, good luck and keep going! :)


gitpod.io fixes

github.io integration currently breaks in multiple ways:
Issues

Halts for user interaction
Missing yaml python package
Demo data installed to wrong directory

Fixes

Ignore user prompt for install
Install pyyaml
Fix demo data installation

@SchrodingersGat isn't pyyaml part of the dependencies?

It is but it need to be present before calling any of the invoke tasks (like invoke install)

@SchrodingersGat my bad - did not take that into account.


Create output directory

If the output folder /outfile/parameter_file/instance_name/ is not created before running the solver, it is not created anywhere and the solution files are written nowhere, hence the impossibility to run the validator.
We should create the folder automatically when not present nor user-specified.

Close with #27


raspberrypiWirelessFirmware: symlinks for Zero 2W

This adds symlinks for WiFi firmware for the RPi Zero 2W. Tested by building and booting on my Pi Zero 2W, which now successfully loads the firmware and brings up the network interface.
% dmesg | grep brcmfmac
[   19.465879] brcmfmac: brcmf_fw_alloc_request: using brcm/brcmfmac43430b0-sdio for chip BCM43430/2
[   19.483492] usbcore: registered new interface driver brcmfmac
[   19.803425] brcmfmac: brcmf_fw_alloc_request: using brcm/brcmfmac43430b0-sdio for chip BCM43430/2
[   19.863147] brcmfmac: brcmf_c_preinit_dcmds: Firmware: BCM43430/2 wl0: Oct  9 2020 14:44:32 version <IP_ADDRESS> (test) (f149b32@shgit)  (r679549) FWID 01-f40f3270

Description of changes

Things done

Built on platform(s)

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

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

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

[x] 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/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

It looks like the latest upstream version already has the symlinks, so can we just update it?

While those symlinks point to the same file, the names are different than the ones in this PR (brcmfmac43436 vs brcmfmac43430b0).
The driver's firmware mapping doesn't mention 43436 at all. From looking at the patch that added this entry, it seems like the vendor's naming is a bit inconsistent.

Also, why is there a bunch of whitespace followed by a pipe at the end of each line?

Whoops, that's from copy-pasting from another vim window. Thanks for catching that: it didn't break my build, and due to the pipe at the end it didn't trigger git-diff's red highlighting of trailing whitespace.


Add release workflow

Description
This PR adds a release GitHub workflow. It:

automatically creates releases on GitHub and PyPI,
handles bumping version numbers,
and creates tags for the new versions.

Type of Change

[x] New feature

Testing
Limited due to the nature of workflows.
Impact
No impact on users. Adds a release workflow.
Additional Information
Publishing to PyPI requires the PYPI_TOKEN secret to be added to the repository.
Checklist

[x] I have documented my code with docstrings and comments, particularly in hard-to-understand areas.
[x] My changes adhere to the coding and style guidelines of the project and pass linting.
[x] My changes generate no new warnings.

To create a "staging" release, make a PR into staging.
Ditto for "stable release" -- release.
In addition, please protect the following branches:

staging
release

Sounds good to me.  We will have to establish standards for the transitions between the branches.


Add support for Webpack 4

Resolves boopathi/image-size-loader#13

Thanks

@boopathi Can you publish a release with this update?

This change is not in the npm release, you have to make changes yourself


Update the DHCP workflows

Debugging some DHCP issues here, I've found the following improvements:

DHCP library does not need to be replaced anymore
The request/release/renewal could be simplified to rely only on the nclient4 and some DHCP modifiers.

For example, when releasing an IP, previously it was forcing the usage of unicast client. But, the ReleaseFromAck already sets a false broadcast request.
The same applies to rebind/initReboot, which could be simplified to a new discovery request (multicast) but specifying their IP address.
There is a follow-up coming on the next PR, that we need to correctly parse if errors of nclient4.ErrNak (today they are also not parsed, just returned).
In this case, the client should go into requesting a new IP without trying to rebind it, because the server may be communicating that the IP is not available anymore (this can happen on renew, rebind, initReboot).
This means, that a new IP address can be allocated, so the syncServices reconciliation should be triggered to reflect the new allocated IP.
I will come late today (or next week) with a fix for this one as well :)

found  problem on renew, fixing it  :) please don't merge it yet

oook I think this is fine now :)

Ooo, just one small issue @rikatz and we should be good to merge 👍

buuuuh got by linters... trying again (sorry for the delay, day was a mess)

I've run the tests locally and things passed now

Tests are running!

💥 boom

Is https://github.com/harvester/dhcp/commit/28b38eafefe3fcd4d9ce0d3fd605ae54070489e9 / https://github.com/insomniacslk/dhcp/pull/458 not needed anymore? I assume that is why the fork was used in the beginning?


First parameter must either be an object or the name of an existing class

Version information

Laravel: 5.5.*
Voyager: ^1.1
PHP: 7.2
Database: Postgress 9.5.3

Description

When a bread with select dropwdown input type is created and if kept options field empty then the browse feature of voyager will not work.
Steps To Reproduce
Steps to reproduce the behavior:

Create New Table with at least one field.
Create new Bread for that table and keep the input type as select dropdown and left empty the options for that rows
Try to browse the newly created bread.
See error

Expected behavior
It should browse the bread.

#1157 #2464

Fixed in https://github.com/the-control-group/voyager/pull/3637


RotateTowardsVector resets references when I hit play

In Edit mode:

After I hit play:

Maybe the new statement is the bad guy here.
public class RotateTowardsVector : Action
{
    [Tooltip("Current managed vector")] public BlackboardEntrySelector current = new BlackboardEntrySelector(); // <-- this one

    [Tooltip("Target vector")] public BlackboardEntrySelector target = new BlackboardEntrySelector(); // <-- this one

    [Tooltip("The maximum angle in radians allowed for this rotation")]
    public BlackboardEntrySelector<float> maxRadiansDelta;

    [Tooltip("The maximum allowed change in vector magnitude for this rotation")]
    public BlackboardEntrySelector<float> maxMagnitudeDelta;

    [Tooltip("Blackboard variable to store the new rotated vector in"), WriteOnly]
    public BlackboardEntrySelector rotated = new BlackboardEntrySelector(); // <-- this one

Fixed by #17


[FEAT] Add streaming + parallel CSV reader, with decompression support.

This PR adds streaming + parallel CSV reading and parsing, along with support for streaming decompression. In particular, this PR:

Adds support for streaming decompression for brotli, bz, deflate, gzip, lzma, xz, zlib, and zstd.
Performs chunk-based streaming CSV reads, filling up a small buffer of unparsed records.
Pipelines chunk-based CSV parsing with reading by spawning Tokio + rayon parsing tasks.
Performances chunk parsing, as well as column parsing within a chunk, in parallel on the rayon threadpool.
Changes schema inference to involve an (at most) 1 MiB file peak rather than a full file read.
Gathers a mean row size in bytes estimate during schema inference and propagates this estimate back to the reader.
Unifies local and cloud reads + schema inference.
Adds thorough Rust-side local + cloud test coverage.

The streaming + parallel reading + parsing leads to a 4-8x speed up over the pyarrow reader and the previous non-parallel reader when benchmarking large file (~1 GB) reads, while also resulting in lower memory utilization due to the streaming reading + parsing.
TODOs (follow-up PRs)

[ ] Add snappy decompression support (need to essentially do something like this)

@samster25 I reverted the estimated row size piping from schema inference, added a good bit more test coverage, and addressed your primary review comments, PTAL!


[bug] Replace failed node missing

Version
Checklist:

[x] I am not using Home Assistant. Or: a developer has told me to come here.
[x] I have checked the troubleshooting section and my problem is not described there.
[x] I have read the changelog and my problem is not mentioned there.

Build/Run method

[x] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - yarn install - yarn run build )

Zwavejs2Mqtt version: 5.11.0
Z-Wave JS version: 8.7.5
Describe the bug
Replace failed node option is missing from menu.
To Reproduce
Steps to reproduce the behavior:

Go to Advanced menu on a node
Notice missing "Replace" under failed nodes

Expected behavior
Replace option under failed nodes as before.
Additional context
None failed node:

Failed node:

Never mind it was moved to Manage Nodes


rename conflicting file

when two render steps create files with the same name, only one wins during a copy and we got the "wrong" one.

@mfojtik @sttts why didn't I have a bindata update?

@sjenning thanks for the issue.  This and the other should fix it.

lgtm'd via slack.

Comparing this with openshift/cluster-kube-controller-manager-operator#50, it looks like they used to have the same name and same content:
$ (cd cluster-kube-controller-manager-operator && git show origin/pr/50 -- bindata/bootkube/manifests/configmap-client-ca.yaml)
commit fde6d6a092ed70c44b26e37d5c29fe36413f904c
Author: David Eads<EMAIL_ADDRESS>Date:   Tue Oct 16 13:57:11 2018 -0400

    rename conflicting file

diff --git a/bindata/bootkube/manifests/configmap-client-ca.yaml b/bindata/bootkube/manifests/configmap-client-ca.yaml
deleted file mode 100644
index e0044cc..0000000
--- a/bindata/bootkube/manifests/configmap-client-ca.yaml
+++ /dev/null
@@ -1,9 +0,0 @@
-apiVersion: v1
-kind: ConfigMap
-metadata:
-  name: client-ca
-  namespace: {{ .Namespace }}
-data:
-  ca-bundle.crt: |
-    {{ .Assets | load "kube-ca.crt" | indent 4 }}
-
$ (cd cluster-kube-apiserver-operator && git show origin/pr/73 -- bindata/bootkube/manifests/configmap-client-ca.yaml)
commit 2f0de2bdab59ddaa22051498df31dab62949b8e6
Author: David Eads<EMAIL_ADDRESS>Date:   Tue Oct 16 13:56:55 2018 -0400

    rename conflicting file

diff --git a/bindata/bootkube/manifests/configmap-client-ca.yaml b/bindata/bootkube/manifests/configmap-client-ca.yaml
deleted file mode 100644
index e0044cc..0000000
--- a/bindata/bootkube/manifests/configmap-client-ca.yaml
+++ /dev/null
@@ -1,9 +0,0 @@
-apiVersion: v1
-kind: ConfigMap
-metadata:
-  name: client-ca
-  namespace: {{ .Namespace }}
-data:
-  ca-bundle.crt: |
-    {{ .Assets | load "kube-ca.crt" | indent 4 }}
-

Does the filename still matter?  Or are these somehow getting set up without per-project prefixes?  I'm not particularly familiar with how the filename affects the pushed asset, so feel free to ignore if this is a noob Kubernetes issue ;).

The filename does matter here.  The CVO just copies all these manifests from all the operators into a directory (single namespaces for all rendered files).  Operator can overwrite one another if they have the same file name, as was the case here vs the cluster-kube-controller-manager-operator.

The CVO just copies all these manifests from all the operators into a directory (single namespaces for all rendered files).

Without per-project prefixes?  Maybe the CVO should borrow that logic from the release-image generator?

Operator can overwrite one another if they have the same file name, as was the case here vs the cluster-kube-controller-manager-operator.

But was the file content actually different between these two operators?  The templates were the same, and the only variable is .Namespace.

This has nothing to do with CVO, but it due to the simplicity of the bootkube script: https://github.com/openshift/installer/blob/master/pkg/asset/ignition/bootstrap/content/bootkube.go#L112

yes, my bad.  these are the bootkube resources not the release resources :man_facepalming:


UI function isMiniaturized called in non-main thread :)

Describe the bug
Main Thread Checker: UI API called on a background thread: -[NSWindow isMiniaturized]
PID: 29621, TID: 8817659, Thread name: (none), Queue name: com.apple.root.user-initiated-qos.cooperative, QoS: 25
Backtrace:
4   libgtk-4.1.dylib                    0x000000010199f410 _gdk_macos_toplevel_surface_present + 344
5   libgtk-4.1.dylib                    0x000000010184a1ec gtk_window_update_toplevel + 52
6   Demo                                0x00000001001a4844 $s7Adwaita6WindowV13setProperties6window8templateyAA015GTUIApplicationB0C_ACtF + 516
7   Demo                                0x00000001001a58ac $s7Adwaita6WindowV6update_3app5forceyAA0B7StorageC_AA7GTUIAppCSbtF + 816
8   Demo                                0x00000001001aaa44 $s7Adwaita6WindowVAA0B5SceneA2aDP6update_3app5forceyAA0B7StorageC_AA7GTUIAppCSbtFTW + 72
9   Demo                                0x000000010003d5e4 $s7Adwaita3AppPAAE05setupB0AA7GTUIAppCyFZySbcfU0_ + 1724
10  Demo                                0x000000010001faf8 $sSbytIegnr_SbIegy_TR + 36
11  Demo                                0x000000010001fa08 $s7Adwaita13UpdateManagerO11updateViews5forceySb_tFZ + 516
12  Demo                                0x0000000100019d18 $s7Adwaita5StateV11updateViews5forceySb_tFZ + 40
13  Demo                                0x0000000100019ba0 $s7Adwaita5StateV12wrappedValuexvs + 252
14  Demo                                0x000000010000424c $s4Demo5ModelC5itemsSayAA12NewsListItemVGvs + 160
15  Demo                                0x00000001000030f0 $s4Demo5ModelC5fetch6beforeySSSg_tYaKFTY2_ + 332
16  libswift_Concurrency.dylib          0x0000000261dab2e0 _ZN5swift34runJobInEstablishedExecutorContextEPNS_3JobE + 436
17  libswift_Concurrency.dylib          0x0000000261dac470 _ZL17swift_job_runImplPN5swift3JobENS_11ExecutorRefE + 72
18  libdispatch.dylib                   0x000000010133eac0 _dispatch_root_queue_drain + 404
19  libdispatch.dylib                   0x000000010133f6d0 _dispatch_worker_thread2 + 188
20  libsystem_pthread.dylib             0x00000001013cfd04 _pthread_wqthread + 228
21  libsystem_pthread.dylib             0x00000001013d7a94 start_wqthread + 8

To Reproduce

Expected behavior
Update UI should be main thread :)
Additional context

Workaround :)

It is interesting that your workaround works on macOS as the code items = response.items never gets executed on Linux. What should be used to update the UI from an asynchronous context is Idle which is based on GLib.idle_add. It seems to solve your problem with NSWindow on macOS as well.
Idle {
    items = response.items
}

It is interesting that your workaround works on macOS as the code items = response.items never gets executed on Linux. What should be used to update the UI from an asynchronous context is Idle which is based on GLib.idle_add. It seems to solve your problem with NSWindow on macOS as well.
Idle {
    items = response.items
}

Yes, it is also work :)


clean and fix string split

Please provide a description of this PR:
if strings.Split have not substring. it will return [""],so it is unnecessary use
if len(parts) > 1 {
		return parts[0]
	}

	return ""

in addition, add len(knn) to avoid panic

Hi @rokkiter. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@rokkiter: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
b8105c93f020fa5de7937492f0ffba81098fa194
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@rokkiter: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
b8105c93f020fa5de7937492f0ffba81098fa194
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

yes, you are right.i'm so sorry I'm confused by the strings.spilt example.


Add the documentation engine.

Public function parameters, etc.

https://github.com/cbou/markdox

Finally we're adding it to the README directly.


Store layer digests on pull

Currently digests are not stored on pull, causing a simple re-tag or re-push to send up all layers. Storing the digests on pull will allow subsequent pushes to the same repository to not push up content.
This does not address pushing content to a new repository. When content is pushed to a new repository, the digest will be recalculated. Since only one digest is currently stored, it may cause a new content push to the original repository.
Fixes #13883

LGTM. This should go in the next RC, if possible.
@dmcgowan This looks like a good stop gap to avoid obvious re-pushes, assuming this gets set on all pushes. Please ensure that if the content is pushed with another digest, the local digest is correctly. This may also cause issues of the layer is pushed back and forth between separate repositories. However, the proposed behavior is better than any of the current behavior.

Not sure what I should see here.
docker pull debian:jessie && docker images --digests yields <none> under digests.

@cpuguy83 Those won't show since that digest is stored as a tag, only when being pulled by digest. This affects the layers, stored internal digest, in a checksum file in /var/lib/docker/graph.

Ok.
LGTM

cherry-picked

what am i missing why dosnt this compile on the RC


How many consumers can be supported per CPU core?

Hi, would you be able to comment on how LiveKit fares in terms of performance when compared to other SFUs i.e. MediaSoup? MediaSoup claims to support 500 consumers per CPU core:

Depending on the host CPU capabilities, a mediasoup C++ subprocess can typically handle over ~500 consumers in total. If for example there are 4 peers in a room, all them sending audio and video and all them consuming the audio and video of the other peers, this would mean that:
Each peer receives audio and video from 3 peers, so 3x2 = 6 consumers in total.
There are 4 peers, so 4x6 = 24 consumers in total.

In particular, the LiveKit blog post "Going beyond a single-core" seems to imply that the number of 1600 down-tracks equates to the number of video tracks being consumed. If this assumption is correct, then 1600 video tracks divided by 16 cores comes out to 100 per core. Is the takeaway from said article then that LiveKit, on a 16-core VM, can only handle up to 100 consumers per core before experiencing performance degradation? And do the recent improvements cited in "Bringing Zoom's end-to-end optimizations to WebRTC" possibly improve the situation here?

As noted in the blog post, LiveKit uses a multicore architecture so we are not limited to using a single core. We don't quote a simple "it can support N consumers" because the answer depends on what the consumers are doing. As an example, in last set of benchmarks we performed, a 16-core instance is capable of handling:

1 publisher publishing audio and 720p video, 3000 subscribers - 3,000 consumers
10 publishers publishing audio, 3000 subscribers - 30,000 consumers

In many cases, network becomes a bottleneck before CPU does. (3000 720p videos would be pushing 5Gbps)
When it comes down to raw CPU performance, I would think mediasoup is likely more optimized given its written in C++. We are more focused on achieving end-to-end efficiencies and enabling applications to achieve their use cases by spreading the load across cores & instances.


ImportError: No module named user on WSGI Deployment

I'm deploying on DigitalOcean with a barebones Ubuntu setup and I'm getting this error:
File "/var/www/myapp/app.wsgi", line 22, in <module>
from myapp import create_app
 File "/var/www/myapp/env/lib/python2.7/site-packages/myapp-0.1-py2.7.egg/myapp/__init__.py", line 3, in <module>
from app import create_app
File "/var/www/myapp/env/lib/python2.7/site-packages/myapp-0.1-py2.7.egg/myapp/app.py", line 10, in <module>
from .user import User, user
ImportError: No module named user

from the apache error logs
I've done the vHost and run fab and all that works out fine. I have added to the application, but nothing major. Am I missing something?
Thanks for the help.

Can not reproduce. Pls check out the latest version.


chore(master): release 1.0.1

:robot: I have created a release beep boop
1.0.1 (2022-06-28)
Bug Fixes

package json (#11) (92bb879)

This PR was generated with Release Please. See documentation.

Release v1.0.1 addresses this.


Bugfix/imds get token to fix #35166

Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
If the ImdsCredential obtains an HttpResponseError with a status code other than 400 (such as a 504), the ChainedIdentityCredential crashes unexpectedly. That is due to the _check_forbidden_response call which can't handle HTML. If the call to the Imds server is made behind a corporate proxy (other another API gateway), the request times out with a 504 and the gateway returns an HTML response. At this point, the Imds -> ManagedIdentityCredential should fail and move on to the next auth mechanism, but instead it fails. This PR fixes that bug.
All SDK Contribution checklist:

[X] The pull request does not introduce [breaking changes]
[X] CHANGELOG is updated for new features, bug fixes or other significant changes.
[X] I have read the contribution guidelines.

General Guidelines and Best Practices

[X] Title of the pull request is clear and informative.
[X] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

@Chuck1823 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree [company="BlackRock"]

@Chuck1823 the command you issued was incorrect. Please try again.
Examples are:
@microsoft-github-policy-service agree

and
@microsoft-github-policy-service agree company="your company"

@microsoft-github-policy-service agree company="BlackRock"

Thank you @Chuck1823 - we really appreciate this!
@pvaneck and @xiangyan99 - are you able to review?

Thanks for contributing.
As I commented in https://github.com/Azure/azure-sdk-for-python/issues/35166#issuecomment-2050153559, it is the design that we want to raise ClientAuthenticationError to stop the DefaultAzureCredential from attempting next if we receive 504.


-nofit option for gmx_cluster

The same group is used for trajectory aligning and RMSD calculation in gmx cluster (as far as I know). Thus, if one wants to align with respect to one group (e.g. all backbone heavy atoms) and compute the RMSD with respect to another group (e.g. active site residues) the only possibility is to align the trajectory before applying gmx_cluster (for example with gmx trjconv through gmx_image) and then apply gmx cluster with the -nofit option activated. Which is not available through gmx_cluster yet. Thank you! :)

Hi!
Please, could you paste here the specific Gromacs command line here. I have added the -nofit flag property however gmx cluster still only asks for just one group for both: fitting and RMSD calculation.
Command line:
  gmx cluster -g gmx_cluster_cluster.log -dist gmx_cluster_rmsd-dist.xvg -o gmx_cluster_rmsd-clust.xpm -s /private/tmp/biobb/unitests_2/gmx_cluster/5c8d32ce-0b55-45c1-a725-8e223c5c7c24/topology.tpr -f /private/tmp/biobb/unitests_2/gmx_cluster/5c8d32ce-0b55-45c1-a725-8e223c5c7c24/trajectory.trr -cl /private/tmp/biobb/unitests_2/gmx_cluster/5c8d32ce-0b55-45c1-a725-8e223c5c7c24/output_pdb.pdb -cutoff 0.1 -method linkage -n /private/tmp/biobb/unitests_2/gmx_cluster/5c8d32ce-0b55-45c1-a725-8e223c5c7c24/index.ndx -nofit

Select group for least squares fit and RMSD calculation:
Group     0 (         System) has 29509 elements
Group     1 (            DNA) has   758 elements
Select a group: 0
Selected 0: 'System'

Select group for output:
Group     0 (         System) has 29509 elements
Group     1 (            DNA) has   758 elements
Select a group: 0
Selected 0: 'System'

I think that is correct, you are not missing any flag as far as I know.
The -nofit flag should deactivate the fitting part even though the group is still called "least squares fit and RMSD calculation".
With this option available now we can fit the trajectory beforehand using trjconv and then compute the RMSD with -nofit. Like this we can fit with respect to one group and compute the RMSD with respect to another.

Solved in 5347ca5
Pablo, let me know either if you need a new release or you can use the development version.


Make the project compilable with clang.

Clang generates more warning messages, such as pointing out a lack
of newline at end of file and issues with the size of a strncopy.
Moving -Wno-unused-parameter to the end of the flags avoids it being
overwritten by -Wextra.
Removing -Werror avoids compilation errors that may arise from new
compiler warnings that might come from newer versions of compilers.
Additionally, the -Werror flag made clang error out when compiling ly before the warnings it detected were corrected.

Out of curiosity, all pars_count parameters are unused for now. What's their intended purpose?

Thank you @ericonr;
pars_count gives the length of pars, which is a string array.
It's not used by Ly but the goal of argoat (and configator) was to use a single
function pointer format for the sake of simplicity and interoperability :)


Adding changes to provide descriptions of schemas

Minor changes:

Updating AMF to 3.5.0
Passing an input directory as an argument

Now we are collecting also vocabulary files and building an ontology index with the URIs and descriptions for each term.
When parsing dialects, the classTerm and propertyTerms mapped in the shapes and property mappings are looked up in the ontology index and the description applied to the schema.

Please run standard --fix src/* after you've done working on the code.

@postatum: standard is not a dev dependency for some reason. maybe it should be there and setup in one of the "scripts".

@postatum Please, review changes.
Thanks for the comments

Please run standard --fix src/* after you've done working on the code.

@postatum: standard is not a dev dependency for some reason. maybe it should be there and setup in one of the "scripts".

I've added it as a dev dependency in this PR branch.
I thought about adding it to pre-commit hook, but didn't figure out how yet. Not sure if I need more rights for that.


Feature Request: Prevent installations of incompatible PSEditions

By default, modules that are marked as incompatible with the PSEdition in use should not be installed.  The current behavior results unnecessarily in incompatible modules being installed and causing issues later.
Below, Posh-Git is given as an example, since it is tagged with PSEdition_Core, but can be installed, without any warning on WIndows PowerShell.
Ideally, both modules tagged with PSEdition_Core and not tagged with PSEdition_Desktop should result in an error (or at least a ShouldContinue prompt) when installed on Windows PowerSHell, and modules with PSEdition_Desktop but not PSEdition_Core should be similarly guarded from installation on PowerShell Core.
Steps to reproduce
In WIndows PowerShell:
Install-Module posh-git -AllowPrerelease

Expected behavior
Error saying that this module is incompatible with the current PowerShell Version

Actual behavior
Module is installed

This should be an opt-in check like -AuthenticodeCheck. When -PSEditionCheck is passed then if  CompatiblePSEditions is defined in the module manifest it uses that to validate editions. There could be some consideration if tags should be used if CompatiblePSEditions is not defined.


Updated legacy Flask example

This PR is supposed to update the legacy Flask starter to v2.2

Hey @DeviousLab, I am closing this PR as we've restructured how we manage templates. Do you mind submitting a PR to the new repo?

@FarazPatankar Just a gentle reminder about the pull request to the new Flask repo


Make it possible to override just an image repo in helm chart

Tell us about your request
Split controller.image and webhook.image to repository and version
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
I'm trying to use local images and right now I have to hardcode version in helm values:
controller:
  image:<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/mirror/public.ecr.aws/karpenter/controller:v0.16.1

webhook:
  image:<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/mirror/public.ecr.aws/karpenter/webhook:v0.16.1

It would be easier to override just repository, like:
controller:
  image:
    repository:<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/mirror/public.ecr.aws/karpenter/controller

webhook:
  image:
    repository:<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com/mirror/public.ecr.aws/karpenter/webhook

and do not hardcode version

Can you link to where you'd like to see this separation? or an example where this overriding ability is implemented?

Sure, you can look here
https://github.com/helm/examples/blob/main/charts/hello-world/values.yaml#L7
image:
  repository: nginx
  # Overrides the image tag whose default is the chart appVersion.
  tag: ""

https://github.com/helm/examples/blob/main/charts/hello-world/templates/deployment.yaml#L20
image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"

I have the same request for this! Because I want to reuse value.yaml, when I upgraded chart version, I also modify image. It's a little inconvenience.

I would like to see this implemented, I'd go one step further and request to include registry
image:
registry:<PHONE_NUMBER>.dkr.ecr.eu-west-1.amazonaws.com
repository: mirror/public.ecr.aws/karpenter/webhook
tag: v0.16.1
This assists with consuming docker proxies with tools like Nexus. You can then just change the registry.


Refactor MainWindow file operations to FileManager

Attempt to follow Facade pattern by separating file operations from MainWindow into FileManager
Add documentation for methods
FileManager is currently under seedu.programmer.model

Lmk if there's a better way to do this!

Looks good actually!! Can I also ask if there is a difference between making the methods in the FileManager class static as compared to initialising it each time we need to use it?


Update docs for @rancher/components

Summary
This updates the @rancher/components readme to include information on

Porting criteria
Developing tips
Porting and publishing

Fixes #5930

@Sean-McQ @cnotv @richard-cox each of you requested some docs on developing @rancher/components in #7425. Let me know if there are any additions or clarifications that you would like to see 🙂


Drone can end up at an angle when unpossessing

This is in standalone netmode. Possess the drone, look down a bit, then unpossess. Ideally the fix works in client netmode as well.

DevNote:
Blocked until 802 issue is merged. I don't want to maintain 3 PRs for the same asset.


Add rpm macros for godot packaging

These macros should be useful for packaging games, made with the
Godot engine, for RPM-based operating systems.

I'm not convinced by the added value of these macros to be honest. As a fellow RPM packager, I'm not sure I would go through the trouble of setting up those macros for the feature they provide.
In particular, these macros depend on the installation of a pre-made export_presets.cfg file, which is not part of the upstream distribution but generated by the editor for a given project. I assume that you ship your own as extra Source in the openSUSE package, but you would then have to keep reviewing changes that we might do to its format to adapt this downstream copy.
I would rather have the Godot binary generate its export preset with suitable values if missing than hardcode something like this in a RPM macro. Most of the time games should provide their own preset anyway that matches the export options.
Additionally, the current command line interface is a bit wonky and will likely see a refactor for Godot 4.0 (there are already some PRs working on that), so the macros would have to be adapted.
I don't feel that the workflow is repetitive and complex enough to require abstraction. So I'm not sure I would pull in those macros myself for my Mageia and Fedora packages.

Is there someway to generate the export_presets.cfg file from the command line? The only way that I found was through the gui. Yes, we do include a pre-made one with the godot-headless package just in case one is not provided in a game's source.
What I was trying to solve with these macros was standardization and simplification of the spec file for Godot games. I liken these macros to the CMake macros that abstract away the small details so the package can be built according to a distribution's standards.

Sorry for the lack of update the past few months.
I still think that these macros are a bit too magic for a workflow which does not support it properly; as mentioned export presets actually often need to be provided by the upstream developer as they can contain non-default settings that are important for the game functionality, such as filters for files to include/exclude from the pck.
For my Mageia and Fedora packaging I prefer to handle this on a case-by-case basis, and thus without macro. It's perfectly fine for openSUSE to use macros for that if it's working though, but as such I'd opt for not merging upstream.
What I do want to implement (hopefully for Godot 4.0, but maybe also cherry-picked to a stable 3.2.x release) is a way for the command line export to generate a default export template if missing. We also have WIP pull requests to refactor the command line interface and make it better parsed and more user-friendly, which should reduce the trial and error in command line exports and thus the need for macros to automate it.
Thanks for the contribution nevertheless! The PR is still available for any RPM packaging interested in using such macros (though do check the openSUSE package for potential updates).


DNS zone option

Currently the zone is inferred, however this is not always ideal—something like:
{
  "stages": {
    "production": {
      "domain": "api.example.com",
      "zone": "api.example.com"
    }
  }
}

{
  "stages": {
    "production": {
      "domain": "v1.api.example.com",
      "zone": "api.example.com"
    }
  }
}

+1

+1

-2 :trollface:

An alternative may be to recursively go through the hosted zone names. First look for api.example.com then look for example.com then create a zone as last option.
Would require no config changes nor a drastic change to Up


Failed: Update autorest version to prerelease 3.0.0-alpha.20240519.3

Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4707

API change check
API changes are not detected in this pull request.


HTTP2 Server Push

Known browsers that support:

Chrome & Opera: yup
Safari: don't think so
Firefox: maybe!
Edge: maybe..

Test pages:

https://http2-push.appspot.com/
https://http2.cloudflare.com/ (apparently)

FWIW, http2.cloudflare.com does not appear to actually push any resources when I visit it (according to Chrome's Net Internals tool).

@aaronlink It does now, I believe.

There is a list of browsers that support HTTP/2 server push in this article.
In their latest incarnations:

Firefox
Chrome
Internet Explorer 11
Edge
Opera

Not supporting it:

Safari.

Also, info on safari and iOS:
https://twitter.com/colinbendell/status/816662477207379968
https://twitter.com/colinbendell/status/816662703305588737

For unshipping in Chrome see https://github.com/Fyrd/caniuse/issues/5688.
Now also being unshipped in Firefox: https://groups.google.com/a/mozilla.org/g/dev-platform/c/vU9hJg343U8/m/4cZsHz7TAQAJ?pli=1