Reading from .properties files?

Any chance of adding support for reading parameter values from local.properties or gradle.properties files too?
Not sure how idiomatic it is, but I've seen a few projects that use those areas to store things that I would consider to be build parameters :)

Hey @ankushg, gradle.properties will already work. Would that already be enough to support the use case you're thinking about?

Oh that's great, I must have missed that in the docs
I still think reading from local.properties would also be a good addition -- plenty of folks use those files to avoid checking in secrets to version control: https://blog.mindorks.com/using-local-properties-file-to-avoid-api-keys-check-in-into-version-control-system

Team Decision
Useful feature to separate built-in Gradle properties (like org.gradle.caching), which belong into a checked-in gradle.properties, from build parameters.
Should not be fixed to one properties file like local.properties but work for arbitrarily named property files.
Details about DSL/API and override rules need to be worked out.

+1 to this. My gradle.properties is checked into git, but secrets.properties and local.properties are not.

For consideration, we use https://github.com/stevesaliman/gradle-properties-plugin to load properties from gradle-[env].properties to override parameters depending on the environment (mainly ci and local). I'm working to incorporate this plugin and just ran into this issue, still working out how to solve it. Might be useful to consider the override rules used by gradle-properties-plugin in case the two plugins can coexist.

Some relevant details on the gradle-properties-plugin I mentioned earlier, regarding the issues that plugin has with Configuration Cache and why it is likely incompatible with this plugin: https://github.com/stevesaliman/gradle-properties-plugin/issues/39#issuecomment-1027532152.
I am happy to contribute if the team has some guidance on how this should be implemented.

This plugin is about modeling parameters that are passed via the command line with -P, not to model everything you put into gradle.properties or other properties files. For that reason we consider this out of scope.


Accessibility for Code Snippet on Copy icon

Expected behavior
Using the tab key, navigate through interactive elements and ensure that interactive elements receive focus in an order that follows the visual layout of the content, maintaining sequence and relationships.
Actual behavior

The focus order is changed after operate the copy icon
image
Steps to reproduce (including any links)
Access link: https://dev.console.test.cloud.ibm.com/docs
IBM Checkpoint: 2.4.3 Focus Order(https://www.ibm.com/able/guidelines/ci162/focus_order.html)

Step one
Step two
Step three
etc.

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/github/IBM/carbon-components-react/tree/master/examples/codesandbox

Additional information

Screenshots or code
Notes

@dakahn Can you add an accessibility label to this issue. Also, note that the same issue is present in the copyButton component.

not stale ⚡


Regenerate class diagrams

Note that the sh script states:

Run the script in the deeprank-core/deeprankcore

Not sure if it works as intended if run from tools folder.
Blocked by #181, #236, #231

Blocked by #234

@gcroci2 , why is this blocked?
Does it need to be recreated each time we adjust any class in the package?

@gcroci2 , why is this blocked? Does it need to be recreated each time we adjust any class in the package?

No absolutely, but I was waiting for reorganizing the Trainer class first. Finished that, we can regenerate the class diagram.

Waiting for #480 to be over for plotting the most updated situation

Training + testing pipeline classes and functions


CPU low load

Hi,
I am not sure whether it is a bug or a feature of this package.
I've noticed that CPU cores are loaded for no more than 6% at most. I provide all the cores available on the system and it uses all the cores but the load is very low. It happens both on my local Intel Mac and on GCC (see a screenshot below).
Here is the config.json file (everything else if default). Data is random with 10000 obs and 2 predictors.
{   "experiment": {
"logdir": None
},
"task" : {
"task_type" : "regression",
"metric": "inv_nmse",
"metric_params": (0,),
"function_set" : ["add", "sub", "mul", "div", "exp", "log", "sqrt" ,"const"] #
} ,
"training" : {  #"epsilon" : 0.05,
"n_cores_batch" : -1  #  "epsilon" : 1,
},
"prior": {
"length": {
"min_": 3,
"max_": 15,
"on": True
}}}

It seems like it is still loading one core at time although I specify to use all available cores. I would appreciate any advice here.

Your CPUs should definitely be pegged at 100% using this configuration.
As you know, setting n_cores_batch to -1 should be shorthand to use all available cores. From core.py:
if n_cores_batch == -1:
    n_cores_batch = cpu_count()
if n_cores_batch > 1:
    pool = Pool(n_cores_batch, initializer=set_task, initargs=(self.config_task,))

I'm wondering if there could be some OS/machine-specific issue with cpu_count? Could you try the following in a Python shell?
import multiprocessing
print(multiprocessing.cpu_count())

Also, how are you running this config? If it's via command-line, it it the simple python -m dso.run config.json or do you have other command-line arguments in that command?
One other thing to try that might help narrow things down is if the problem persists when removing the const token from function_set? On that note do you know if your Cython executable successfully installed? (Sometimes this fails but if so DSO falls back on a pure Python version)?

@IgorK7 FYI we just updated the repo to a new release. It had a lot of refactoring, so apologies if it breaks any existing configs; you might have to shuffle some things around in your configs following the new template config_common.json.

Hi Brenden,
Thank you very much for getting back to me.
I am using your package in my research project where I investigate properties of symbolic regressions and DSR in particular in the set up with very high noise (true R2 at 5%). So all calculations require a lot of computation capacity and time.
I can confirm that with
print(multiprocessing.cpu_count())
the core count is correct. It loads all the cores but only for a fraction of capacity.
Yes, I tested it via command-line with
python -m dso.run config.json
and the effect is the same. The load per core is very low. Tracking average load on the build-in datasets is not reasonable (unless lopping) as they have no noise and the program just runs too fast. I supply my own data of 10000 rows and 2 explanatory variables with noise and the average load per core is low (like 5%).
The inclusion/exclusion of const does not affect the load.
Cython is installed.
I run the algorithm on 3 different machines: Intel Mac with 10 physical cores, 2 instances of 48-cores on Google Compute Engine and Windows Subsystem for Linux (WSL) on an Intel PC with 8 physical cores. The results are the same - low load per core although all cores are loaded.
By the way, running the program on WSL should be a faster alternative to Docker if one needs to run it on Windows PC.
I am not sure how to make it run on CPUs at the full capacity.
Also, is there an option in the package set up to make it run on GPU? I could not find it but I am not a programmer.
Thank you very much!

I will also test the new version of the package. Thank you!

Regarding using GPU, I was able to make it utilize GPU but it still does not load to max capacity. Although it may be that my datasets are just too small (10,000 by 3).
I was able to install tensorflow==1.14 + CUDA==10.2+cuDNN==8.6.0 on Ubuntu18.04 with some intel and Nvidia TeslaT4. (resolving compatibility issues on the go as tf==1.14 should only work with CUDA=10.0).
Still, any advice for speeding up computations would be appreciated.

GPU is not going to help. The DSO LSTM is a very small network, and CPU (e.g., computing MSE on the dataset for the regression task) is going to be the computational bottleneck.


Ascii character 160 showing as

Description
Our translation service is inserting ascii character 160 before punctuation marks for french translations. This is rendering as   in the output. We are not binding the html on these elements (and doing so would be quite bad in some cases) so is there a way that we can avoid this without using bind-html?

Have you read about the security / escaping on https://angular-translate.github.io/docs/#/guide/19_security ? And maybe you should provide a complete example so we can see what goes wrong.

So a few of the problem translations are
"Est:" : "Est :", "Estimate: High to Low" : "Estimation : élevée à plus faible", "Estimate: Low to High" : "Estimation : faible à pus élevée"
between "Est" and ":" and "Estimation" and ":" there is an ascii character 160 and that is showing as   I have tried a.. of the various sanitation methods with no change. We are currently using the escape strategy as the docs note that there is a bug with the sanitize mode.

A complete example would be some plnkr according to the guidelines for issue filing. But anyway - I don't see an issue here. Look: https://plnkr.co/edit/TJm0BT?p=preview

Sorry about that. Your example has shown what the issue was and I now have it sorted out. Thanks! In short I am now using escapeParamters sanitation service.


Add a page on data stores / data sources

Revive old data store page but change emphasis
Suggest that analytics should be baked into a service
Provide contact slack channel for data services to advise
Point to the data audit and how data stores need named owners

I couldn't build locally, so was unable to view a live preview!

Closing the PR with agreement from Matthew. We're exploring other options with the information services area teams


Daijisho crashing everytime I try to edit a game cover

Everytime I try to edit any game (by pressing and holding onto it) Daijisho crashes. This is happening on every platform I try. It doesn't even open up the options to edit, it simple crashes as soon as I hold any game cover with my finger or with a physical button.
I'm using AYN Odin 2 Pro running Android 13, Daijisho v1.4.67
java.io.IOException: Error on service connection.
at com.google.android.gms.auth.zzl.zzh(com.google.android.gms:play-services-auth-base@@18.0.4:8)
at com.google.android.gms.auth.zzl.zza(com.google.android.gms:play-services-auth-base@@18.0.4:15)
at com.google.android.gms.auth.zzl.getToken(com.google.android.gms:play-services-auth-base@@18.0.4:3)
at com.google.android.gms.auth.zzl.getToken(com.google.android.gms:play-services-auth-base@@18.0.4:1)
at com.google.android.gms.auth.zzl.getToken(com.google.android.gms:play-services-auth-base@@18.0.4:6)
at com.google.android.gms.auth.GoogleAuthUtil.getToken(com.google.android.gms:play-services-auth-base@@18.0.4:3)
at com.google.api.client.googleapis.extensions.android.gms.auth.GoogleAccountCredential.getToken(GoogleAccountCredential.java:258)
at com.google.api.client.googleapis.extensions.android.gms.auth.GoogleAccountCredential$RequestHandler.intercept(GoogleAccountCredential.java:283)
at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:880)
at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(AbstractGoogleClientRequest.java:525)
at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(AbstractGoogleClientRequest.java:466)
at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.execute(AbstractGoogleClientRequest.java:576)
at com.magneticchen.daijishou.google_drive.GoogleDrive.listAllFiles(GoogleDrive.kt:250)
at com.magneticchen.daijishou.google_drive.GoogleDrive.createOrUpdateFileByInputStreamContent(GoogleDrive.kt:288)
at com.magneticchen.daijishou.google_drive.GoogleDrive.createOrUpdateFileByFilenameAndInputStream(GoogleDrive.kt:302)
at com.magneticchen.daijishou.google_drive.GoogleDrive.uploadByteArrayToFilename(GoogleDrive.kt:321)
at com.magneticchen.daijishou.google_drive.GoogleDrive$setupGoogleSignInAccount$1.invokeSuspend(GoogleDrive.kt:365)
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:108)
at kotlinx.coroutines.internal.LimitedDispatcher$Worker.run(LimitedDispatcher.kt:115)
at kotlinx.coroutines.scheduling.TaskImpl.run(Tasks.kt:103)
at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:584)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:793)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:697)
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:684)
Caused by: android.os.DeadObjectException
at android.os.BinderProxy.transactNative(Native Method)
at android.os.BinderProxy.transact(BinderProxy.java:584)
at com.google.android.gms.internal.auth.zza.zzb(com.google.android.gms:play-services-auth-base@@18.0.4:2)
at com.google.android.gms.internal.auth.zzd.zze(com.google.android.gms:play-services-auth-base@@18.0.4:5)
at com.google.android.gms.auth.zzl.zzb(com.google.android.gms:play-services-auth-base@@18.0.4:2)
at com.google.android.gms.auth.zzf.zza(Unknown Source:6)
at com.google.android.gms.auth.zzl.zzh(com.google.android.gms:play-services-auth-base@@18.0.4:6)
... 24 more

Okay, seems like it was an issue with Google Play Services. I've updated it and the issue didn't go away, but searching a little bit I found a tip saying to clear Play Services Cache/Data and after doing it Daijisho stoped crashing. Let's see how long it's gonna stay like that...

Now that's interesting seems like Daijisho hung for you trying to generate a token(?) Not sure why though 🤔. Hopefully TF can look at this when they have time


Cancel any ongoing attempt to establish new BLE connection when Close…

…AllBleConnections() is called
When CloseAllBleConnections() is called, any ongoing attempt to establish new BLE connection should be cancelled because it is no longer needed for a new BLE connection to establish. CancelConnection() of the connection delegate should be called to cancel any ongoing new BLE connection so that platform- specific BLEManager can do cleanup. This is needed because we are encountering a problem that BLEManager not doing clean up when establishing PASE session times out, causing problem in the BT layer. Since BLEManager is platform-specific, it doesn't have knowledge of when PASE times out. But BLEManager needs to do clean up whenever CloseAllBleConnections() is called.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Peter Cui seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix two issues present in Debian patches

Description
Fixes https://salsa.debian.org/games-team/libsfml/-/blob/master/debian/patches/01_remove-googleapi-css.patch and https://salsa.debian.org/games-team/libsfml/-/blob/master/debian/patches/02_build-doc-once.patch.
The first one about removing a comment is mostly a curtesy to the Debian folks but we don't like commented code either so it makes sense.
The second one about the doc target is something I agree with. I never build the docs because I have to modify a cache variable then deal with the massive console output that Doxygen produces every single time I build the library. It's much better if I can leave that option enabled but only build docs on demand. I think it would be even better if we made Doxygen an optional dependency and only configured the docs if Doxygen is found. That way we don't even need the SFML_BUILD_DOC option.

After this change the setup is kind of broken.

If I set SFML_BUILD_DOC I expect the docs to actually be built and not just when I explicitly build the doc target.
If you set SFML_BUILD_DOC and then build the install target, the build fails, because it tries to still install the docs, more specifically the SFML.tag file.
If you want to install the docs this has now become a two step process, where first have to explicitly build the doc target and then build the install target.

Personally, I'd want to challenge this change again.


Sahar

👷 Deploy Preview for adoring-leavitt-e31a11 processing.

Name
Link

🔨 Latest commit
778c42c8d00abd66a6397e8125f60d491399f0ac

🔍 Latest deploy log
https://app.netlify.com/sites/adoring-leavitt-e31a11/deploys/62a04c830bf424000896f159


Spread the word ;)

Check express-hijackresponse "Dependents" packages, and make a PR into those projects removing express-hijackresponse while using express-interceptor:

express-jsxtransform is a good start because of the hype of React.

What needs to be done?

Description updated.

express-jsxtransform PR sent.

@fer8a thanks, it's merged now ;-)
I'll just add a few more that I know of here:

gustavnikolaj/express-autoprefixer
papandreou/express-compiless

There's a few more, but those use streaming as far as I remember.

I'll be happy to do some of the work, but I cannot promise when I'll get around to it :-)

Hi again guys, express-compiless PR sent.

Consider sending an email to http://nodeweekly.com/

I think we can do it after this gets merged, and adding to the readme file a section "Who is using it" we can send the email. Also we can add streaming as part of our roadmap. cc @flockonus

BTW, this comment is awesome! :feelsgood:

@p1nox I'll review and probably merge that PR tomorrow at work.

https://github.com/One-com/livestyle is also using express-hijackresponse in https://github.com/One-com/livestyle/blob/master/lib/middleware/injectLiveStyleScriptIncludeIntoHtml.js

connect-inject would also be a good candidate for a PR, since it is also messing with the response object

Greetings everyone, express-autoprefixer PR sent.

Merged on express-compiless: https://github.com/papandreou/express-compiless/pull/10#issuecomment-98609635

Next packages waiting for PR ;):

https://github.com/papandreou/express-processimage
https://github.com/One-com/livestyle

Thanks to @gustavnikolaj @fer8a @Munter for all the help :+1:.

I'm also using interceptor in this project: https://github.com/Munter/express-systemjs-translate
Still in prototyping stage and waiting for some syncing with systemjs, so not ready to be linked to yet

Awesome @Munter! Let us know :+1:

No need to open more PRs :wink:, closing for now.


use membrane channel segmentation

Hi Vishal
when I want to segment using membrane channel, it always says "false" in the config.txt file, even though I put numbers in channel and cycle. Could you please have a look, I think it doesn't work properly.
Thanks!
Christian

Hi Christian,
Isn't the membrane channel a number value?? It can't be false.

Closing as it is outdated.


Public API

Setup public api to expose static data

@blrchen I am also in need of same data for an application that I am building. Would you be able to share how you got this data? I was not able to find any one API to get this data.

I suggest opening new issues as this is off current topic.
Could you be more specific on what data you are interested at?


Security Vulnerability: CVE-2024-38428 in alpine/helm:latest - Helm Dependency Update Required.

Summary
url.c in GNU Wget through 1.24.5 mishandles semicolons in the userinfo subcomponent of a URI, and thus there may be insecure behavior in which data that was supposed to be in the userinfo subcomponent is misinterpreted to be part of the host subcomponent.
CVE Details
Vulnerability Score(v3): 9.1
CVE: 2024-38428
Severity: Critical
Documentation: CVE-2024-24790

This is not a helm issue it is a https://github.com/alpine-docker/helm issue

As Terry points out, this is not an issue with Helm or something maintained by the Helm project. Please see the appropriate project to file the issue.


Fix input/output volume owner and permission issues

Docker volumes for job input/output are perm 0755 owned by root so algorithm containers which execute as a non-root user will have permission issues. The owner of the volume should be the "standard" unprivileged users (uid 1001) or the top level permissions should be 0777.

We've fixed the ownership so that the input and output directories are owned by Scale, but I think we want to also chmod the output_data to 0777 so the job task can write to it as any user.


Inaccurate instruction.

Instructions would not support high availability if CMS is moved to a SQL AlwaysOn backend without additional steps.

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

ID: d9082d9a-5cfe-f210-c2e7-afc56226b252
Version Independent ID: 1ad7b4fb-2c78-d0c5-d8de-2ac2c7910127
Content: Move-CsManagementServer (skype)
Content Source: skype/skype-ps/skype/Move-CsManagementServer.md
Service: skypeforbusiness-powershell
Product: unspecified
GitHub Login: @kenwith
Microsoft Alias: kenwith

@papandms can you help validate this issue and what needs to be updated in article? Thanks!


S-10.1: As a user, I want to find the nearest point of outdoor interest (restaurants, stores, etc.).

The user can see the points of interest within the campus perimeters.

[ ] Prepare data (.json) and assets (pictures of locations & reviews)

[ ] Create UI to show the building information

[ ] Show directions to Point of Interest (based on the json)


modify aggregate of fuel types Issue #54

@claytonpbarrows

@claytonpbarrows the build is erring because of the version of PowerSimulations vs PowerSystems in the test project.toml.... should be fixed when dev0-6 gets merged

@claytonpbarrows and @Lilyhanig I am going to merge this to start getting the updates going.


Add initial support for GLSL compilation

TODO:

[X] Set GLSL version through compiler options (like HLSL shader model)
[X] Add example

bors r+

bors r+

bors r-

bors r+

bors r-

bors r+

bors r-

bors r+

bors r+


Outdated personal version of rspec_sequel_matchers

Hi @rianniello ,
When reviewing #266 , I stumbled upon a line in both Gemfile and Gemfile.lock that uses a "personal version" of the rspec_sequel_matchers gem:
https://github.com/ausaccessfed/saml-service/blob/develop/Gemfile#L42
  gem 'rspec_sequel_matchers', git: 'https://github.com/bradleybeddoes/rspec_sequel_matchers.git'

I'd expect AAF would want to use either proper upstream repositories or forks under ausaccessfed...
I've traced this line back to commit 384e668c (8 years ago), with the commit message pointing to a PR addressing an issue - but the PR had been merged along ago (shortly after being made) and a newer version has been released since (0.5.0, released 2 years after merging the PR), so I believe 'rspec_sequel_matchers' can be reverted back to rubygems latest - which should pick 0.5.0.
Cheers,
Vlad

Hi Vlad,
Thanks for looking into this! I have bumped the version of rspec_sequel_matchers to 0.5.0 as recommended, everything is looking good so far with that change. I did not create a seperate PR, however I have added to to be released with the upgrade to rails 61.4.7 here: #267
Thanks


bosco install reset when node version is changed

I think bosco should store somewhere the version of node used to run npm install or yarn on a package. Then, when it doesn't match, it should remove the node_modules before reinstalling again.

:+1: this is a good idea, will do this this week.

Would it be better to warn the user and exit (unless a -f flag is specified)? It could be that the user simply has a node version mixup that could be fixed with an nvm command, whereas a complete reinstall of modules can be quite a heavyweight process requiring network access. You'd be pretty annoyed if you were sitting on an aeroplane and it removed node_modules and then wasn't able to reinstall.


Fix Kam Image reconcliation during upgrades

Describe the bug
GitOps operator does not handle the KAM image upgrades during the GitOps operator upgrades. For example, when a user upgrades from GitOps operator v1.8.0 to v1.9.0, KAM image is not upgraded to the v1.9.0 supported version.
However, someone who installs GitOps operator v1.9.0 can use the KAM v1.9.0 supported version.
To Reproduce
Steps to reproduce the behavior:

Install GitOps Operator v1.8.0
Verify KAM version
Upgrade GitOps Operator to v1.9.0
Verify KAM version

Expected behavior
GitOps operator handles the KAM Image version upgrade during the reconciliation.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

fixed with

https://github.com/redhat-developer/gitops-operator/pull/625 in master
https://github.com/redhat-developer/gitops-operator/pull/605 in v1.10
https://github.com/redhat-developer/gitops-operator/pull/565 in v1.9


chore(bower): remove moot version property

Per the Bower spec the version property is not used for anything.
One of the maintainers also says that they are not likely to use it in the future.

:+1: Didn't see this PR


Block comments not seen as comments

The rules language does treat /* ... */ as a comment.
The plugin (1.1.5-SNAPSHOT) does not render it as such.

Thanks for testing, issue will be fixed on version: 2.0.0


Add query id to hive connector to build bucket file name

In Presto e2e test, we found the bucket file name is not compatible
with one generated by Presto java if some bucket file is missing
(computeFileNamesForMissingBuckets). The difference is that java
use query id to build bucket file name but velox use the task id.
We can just use query id which is unique under a particular partition
dir so no need to use task id.

NIT: typo in description
We can just use query id as bucket id is unique under a particular...

@kewang1024 thanks!


AWM: #661 - patient age

update age display as #y #m #w #d
IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #[issue number]
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Release | Other)
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[x] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

Lgtm

Lgtm

Name your PRs more descriptively
Add the issue number/link as provided in the PR template. This helps the reviewer find the original issue scoping and requirements
Add the PR type from the given choices on the PR template
You don't need to add your abbreviation on the PR since your username, display name and avatar are available on the PR and commits


Add project root command

@kRITZCREEK think you requested this on a recent PR. Just ripped this from what I was already using

Sounds like useful functionality to have. Thanks!


emitter setXSpeed/setYSpeed does not work

This Issue is about (delete as applicable)

A bug in the API (always say which version you're using!)

this code does not work correctly.
particles can not move anywhere. (works in phaser 2.6.2)
        this.waterParticle = game.make.bitmapData(2, 2);
        this.waterParticle.rect(0, 0, 2, 2, '#ff0000');
        this.waterParticle.update();

        this.emitter = game.add.emitter(200, 200, 500);
        this.emitter.makeParticles(this.waterParticle);
        this.emitter.setXSpeed(-100, -250);
        this.emitter.setYSpeed(-100, 100);
        this.emitter.setAlpha(0.2, 1);
        this.emitter.flow(500, 20, 2, -1, true);

version is 2.7.3

Works for me: http://codepen.io/samme/pen/mRKrmG

Thanks for verifying it.
Hmm, I actually use this template.
https://github.com/lean/phaser-es6-webpack
So maybe this template or I made some mistake.
I'll close this issue.


Request to add Code of Conduct

Can we have Code of Conduct and add that link to the README.md or CONTRIBUTING.md?

Everyone's welcome to suggest their favourite code of conduct and I'll explore them when I can.

How about a modified version of The Ruby Community Conduct Guideline?
Some of the rationales for this:

It is simple (i.e. more of a code than a legal framework), which seems proportionate, as we don't really have a problem with misconduct.

It comes free of any political baggage and contention.

It was adopted by a community very similar to our own, after a lot of discussion.

@Drenmi Great idea!
How modified? Just replacing Ruby with RuboCop perhaps?

I completely agree with @Drenmi suggestion. I think it is a good opinion to match The Ruby Community Conduct Guideline if CoC will be added.


append label and remove the label for the same function

What this PR does / why we need it:
This PR is used to append label when the OperandRequest an OperandBindInfo have existing labels
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #351
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

NONE

why removing the exist labels?

@chenzhiwei  It is for this issue https://github.com/IBM/operand-deployment-lifecycle-manager/issues/329. If the RegistryNamespace or RegistryName changed in OperandRequest or OperandBindInfo CR, the old label won't be removed.


Measure performance of control plane

Understand the overall performance of the control plane and its associated components.
Measure:

end to end time for the scenario

data policy opa connector

katalog connector

deployment of data plane components (ex: arrow-flight read module)

admin config policy evaluation

[x] Test1: Test the quick start scenario which uses the katalog and opa connectors and deploys and arrow-flight server on kind

[x] Test2: Test a scenario similar to the quick start scenario but that uses a pre-deployed arrow-flight server (config module) on kind

[x] Test3: Repeat test1 on open shift

[ ] Test4: Repeat test2 on open shift

I have tested a quick-start scenario when opa and katalog connectors have been deployed on kind, no webhooks deployed.
FybrikApplication controller:

Validation of FybrikApplication: 1 ms
Getting response from the catalog connector: 14-25 ms
Getting response from the policy manager: 29-52 ms
Adminconfig: initialization - ~20 ms, each subsequent evaluation 2-4 ms
Building a plotter - 1ms
Applying a plotter resource - 10-40 ms, depending on the machine load
Overall reconcile - 95-98 ms, quite consistent between runs
Plotter controller - overall time consists of either getting a blueprint to check the status or creating a new blueprint, the processing time can be neglected. ~10ms to check the status, 12-20ms to generate a blueprint
Blueprint controller
Helm release installation: 2.3-2.5s
Helm resources status check: 30-80ms (mostly kubectl get operations)
** Although, I've tried a helm release with a single configmap, results were not much better (it took less time to check one resource than four resources, but helm install still lasts over 2 sec.)
End-to-end scenario takes ~5s. Multiple reasons/problems:
Extra reconciles caused by status update or finalizer update, or Requeue requests during the reconcile.
Problems with updating status (not the latest version). Causes another reconcile.
Idle cycles. For example, blueprint controller checks resource status again and again, while plotter controller is waiting and can't update the status.
Context switches? Manager is a single process, there are several controllers sharing the run-time.
Perhaps, some of the problems are specific to kind, I'll check on open shift clusters.

@shlomitk1, @simanadler, can we close it?

Not yet.

@shlomitk1 Can you please rerun the tests on v0.7.0?
Also, were any of the issues you raised (extra reconciles, updating status, etc) addressed already?

Problems with updating status have been resolved.

I've focused on Helm operations as they take most of the run time.
The experiment was done on an OS cluster for 0.7 version, application that required installation of the arrow-flight module.
The script below starts an application, adds a dataset, waits for its readiness, removes a dataset, deletes the application:
#!/usr/bin/env bash
echo "Create empty fybrikapplication"
kubectl  apply -f app-empty.yaml
while [[ $(kubectl get fybrikapplication demo-app -n default  -o 'jsonpath={.status.ready}') != "true" ]]; do echo "waiting for FybrikApplication" && sleep 5; done
echo "Add asset"
kubectl  apply -f app-dev.yaml
while [[ $(kubectl get fybrikapplication demo-app -n default -o<EMAIL_ADDRESS>== "Ready")].status}') != "True" ]]; do echo "waiting for asset" && sleep 5; done
echo "Remove asset"
kubectl  apply -f  app-empty.yaml
while [[ $(kubectl get fybrikapplication demo-app -n default  -o 'jsonpath={.status.ready}') != "true" ]]; do echo "waiting for FybrikApplication" && sleep 5; done
kubectl delete fybrikapplications.app.fybrik.io demo-app -n default
sleep 60

Left to explore:

Why are there 17 upgrades? Should not be any, given the fact that the 60 sec sleep interval was made to ensure that the pods are properly cleaned up.
There are frequent calls to check the status of the release and the resources. Can any improvements be made?

@shlomitk1 what was imagePullPolicy during the tests ?

Of the manager pod? Always
But I don't see the relevance to the helm installs and other operations that have been measured.

According to the provided numbers, looks like there ware 67 iterations ( 50 installs + 17 upgrades ) as well 67 pulls and loads. Is it correct ?

@shlomitk1 what was imagePullPolicy during the tests ?

Of the manager pod? Always But I don't see the relevance to the helm installs and other operations that have been measured.

The module's pod

According to the provided numbers, looks like there ware 67 iterations ( 50 installs + 17 upgrades ) as well 67 pulls and loads. Is it correct ?

50 iterations of the script I have attached above. No idea where the upgrades are coming from. Originally, there was a smaller interval between deleting an app and applying a new one, so I suspected a release was being attempted to install and failed because the old pod did not die yet. Maybe this is still the reason. I'll try to run with even a bigger time interval.

@shlomitk1 what was imagePullPolicy during the tests ?

Of the manager pod? Always But I don't see the relevance to the helm installs and other operations that have been measured.

The module's pod

Always

I suspected a release was being attempted to install and failed because the old pod did not die yet.

Can we check errors in the logs ?

I'm looking on the numbers, there is something strange in duration per call and duration per app iteration.
I can understand that if the same method was called several times per app iteration, the duration per app will be accommodated time, like in case of status.
How can you explain that upgrade took 8332.56 ms per call and 2833.07 per app iteration?

@shlomitk1 what was imagePullPolicy during the tests ?

Of the manager pod? Always But I don't see the relevance to the helm installs and other operations that have been measured.

The module's pod

Always

In order to compare results, can you repeat the test with IfNotPresent ?

I'm looking at the numbers, there is something strange in duration per call and duration per app iteration. I can understand that if the same method was called several times per app iteration, the duration per app will be accommodated time, like in case of status. How can you explain that upgrade took 8332.56 ms per call and 2833.07 per app iteration?
Or it overlapped app iterations, and got wrong time.

Duration per app iteration is just an average of the total duration divided by 50. So for an upgrade of duration X we'll have 12X/50 in average. If you think it is a meaningless number, just ignore.

@shlomitk1 While it's fine to focus on helm, since that was the main thing taking time ... in addition please run the same tests as you did on 24 Nov 2021 so that we have a comparison to the previous version.

@roytman With pullImagePolicy = IfNotPresent:

Looks much better!

Interesting, that status is 47.7 ms vs 362  and getResource is 218 vs 836. The pull image policy should not influence on these calls, or I'm missing something.

Rerunning with pullImagePolicy = Always:

Results are looking pretty much comparable. Given the concurrent run of the controllers, I am not sure how accurate a single measurement is. The idea was to measure the end-to-end run. It makes more sense to compare imagePullPolicy values at the single helm operation level rather than the entire control plane.

@shlomitk1  A couple of questions:

In blueprint_controller.go, applyChartResource, line 240 we check the status of the chart in order to determine whether to install or upgrade.  Couldn't we skip this status check, and always use the Helm Upgrade command with --install?  This upgrades if the chart exists, and if it doesn't it installs.

This would decrease the number of calls to Helm status, and would also simplify the code.

Why are there 280 calls to getResources?  It's used to check the status after deployment.  Right?  If so, you have to install, update, delete the arrow-flight-server I assume there is a status check after each.  That would give me 150, but your table shows 280.  What am I missing?

Why are there only 11 upgrade calls?  When you delete a data set doesn't that cause an upgrade to the arrow-flight helm chart?

Why are there 61 pull and load calls?

Any idea of the time it takes to log in/out of the registry?  I see in the code that we do it during every reconcile.  Wouldn't it be sufficient for the control plane to log in only once, or at least only once per FybrikApplication instance?  If the time is small we can ignore.

@shlomitk1 A couple of questions:

There is no use in setting an Install option for Upgrade. From the documentation (upgrade action):

// Install is a purely informative flag that indicates whether this upgrade was done in "install" mode.
	//
	// Applications may use this to determine whether this Upgrade operation was done as part of a
	// pure upgrade (Upgrade.Install == false) or as part of an install-or-upgrade operation
	// (Upgrade.Install == true).
	//
	// Setting this to `true` will NOT cause `Upgrade` to perform an install if the release does not exist.
	// That process must be handled by creating an Install action directly. See cmd/upgrade.go for an
	// example of how this flag is used.
	Install bool

GetResources is done in a "polling" mode, that's why it takes a number of calls until the release is ready. I need to check whether the "wait" option blocks the call until the release is ready. It did not work for me previously, but maybe with the latest version of Helm things have changed.

So here is the thing: there should be no upgrades at all. We have an extra reconcile somewhere (and I think I have found it) that causes the controller to attempt an install (upgrade) twice.

61 = 50+11, each applyChartResource method calls both load and install/upgrade. Ideally, there should be only 50.

Regarding login/logout, I don't know. You'd need to store a helm chart in a private repository and supply pull secrets for obtaining the chart, to activate the login/logout functionality. Perhaps, @cdoron has some use cases that can be measured.

Couple of comments:

There is no use in setting an Install option for Upgrade. From the documentation (upgrade action): ....

The mentioned above code comment is wrong. The comment suggests to check "cmd/upgrade.go for an example of how this flag is used".
So cmd/upgrade.go says

// If a release does not exist, install it.

I ran entire Fybrik "Get Started" process with upgrade --install options instead of the install commands, all packages include fybrik itself were successfully installed. I'll submit a PR to remove the misleading comment.

I need to check whether the "wait" option blocks the call until the release is ready. It did not work for me previously, but maybe with the latest version of Helm things have changed.

I suggest to set both flags Wait and WaitForJobs to true.  (a module might include jobs).

The last one, jut put attention that we continue using DisableOpenAPIValidation  =  true. It was set when helm had the memory leak. For better robustness we need to unset it. In the previous performance tests it was false.

Rerunning the test after changes made by @roytman, with imagePullPolicy = IfNotPresent (the results are better than with Always):

The number of status/getResources calls has been significantly removed after using install with wait option. However, the duration of an install operation has increased (9.4 sec vs 3.6 sec).
Future improvements: no need for checking "standard" resources after wait, removing extra reconciles/upgrades.
Also, since installing of a pod takes so much time, I'd suggest to test a module that installs config maps and services, for example.

Actually, in the original code, when the install operation returned, the resources were not ready yet, and the entire process required several "getResources" iterations (each with a delay of 2 sec). Now the install command returns after finishing the entire process.
@shlomitk1 do you have measurements from the original code, from start of installation until the resources were ready ? When you wrote "End-to-end scenario takes ~5s" what were the start and end of the probes ?
If now, install requires 9.4 sec plus 2 sec getResources iteration delay (in the original code),  we are getting 11.4 sec which is going to be  close to the  BU results.
We have to check how they  measure the process.

@shlomitk1 could you check please if in the manager logs there is the following message "blueprint.Status.ObservedState is not ready, will try again" ? I want to validate that when helm install --wait returns, all resources are ready.

@roytman end-to-end scenaro for 5 sec was on kind, installing a pod was not much slower than installing a configmap, on open-shift run-times are different.
Current measurements show ~11 sec.

blueprint.Status.ObservedState is not ready, will try again

I did not find this message. Found those instead: Deployment is not ready: fybrik-blueprints/demo-app-default-arrow-flight-module-arrow-flight-module-chart. 0 out of 1 expected pods are ready

the latest message is printed by helm when it is waiting and checking readiness of the objects

I understood difference between kind and OpenShif, my question is where are the start and end probe points? Is it the entire reconcile loop? something else ?

From the first reconcile after the app was created until the app gets a ready condition

Here are the helmer operations on an OpenShift cluster datamesh05 , the installed chart is our arrow-flight module,  which was preloaded into a local dir, the test ran 100 iterations. Time in ms:

Performance improvements have been made. Closing this issue.


Brenda Michelle

Due we close new issues on PR merge and voting is still possible on closed issues I will close this one. Nice Profile btw 🚀


Framed Controller, Slave & Trim do not drop when broken, even with silk touch

Version: 1.20-6.0.0
MC: 1.20.1
Forge: 47.2.21
Basically what it says in the title:
Steps to reproduce: Craft a framed controller in the regular way, then frame it with whatever blocks you want, place, and attempt to break with any axe (tested with stone, iron, and diamond). Regardless of silk touch (tested with and without) the block will not drop. This applies to Framed Controllers, Framed Slaves, and Framed Trims.
https://github.com/Sirfirelord1/Petty-s-Cozycraft/issues/12

Thanks for the report. The bug was introduced in f3724dace1ad7ca92ed8ced24028c72830731d6f by the loot tables accidentally being moved, which I didn't notice because Storage Drawers doesn't actually use loot tables to compute drawer drops. It should be fixed in 33736887c64209048e32a390a08bc912de2d9dda and thus the 1.20-6.0.1 release, which should be here when it's approved.


Bug when triggering renewal of cert in acme plugin

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
3.5.0
Current Behavior
I try to trigger a renewal of certificates and there seems to be some issue in the acme client.
This is the request I do towards the admin-api:
curl http://XX.XXX.XXX.XXX:8001/acme -XPATCH

The response from my request is this:
{"message":"Renewal process started successfully"}

This is the error message from Kong:
Jan 28 18:09:58 fra1-kong-001 docker[138569]: kong-api              | 2024/01/28 18:09:58 [error] 1280#0: *200803 [lua] job.lua:284: execute(): [timer-ng] failed to run timer unix_timestamp=1706465398520.000000;counter=13563:meta=debug off: /usr/local/share/lua/5.1/kong/plugins/acme/client.lua:512: attempt to index local 'config' (a boolean value), context: ngx.timer
Jan 28 18:09:58 fra1-kong-001 docker[138569]: kong-api              | XX.XXX.XXX.XXX - - [28/Jan/2024:18:09:58 +0000] "PATCH /acme HTTP/1.1" 202 50 "-" "curl/8.4.0"

Expected Behavior
I except the existing certificates to be renewed and not get stuck with the "old" ones.
Steps To Reproduce

Trigger a renew of certificate:curl http://XX.XXX.XXX.XXX:8001/acme -XPATCH

Anything else?
I use DecK for configuring Kong, but I've done that since months and without any problem.
If I recall correct I did not have the renewal issue when I run an older version of Kong (not 3.5.0), but I don't remember the version back then.

I think it's related to the :configure handler change that recently introduced, there's a GH discussion around it too but I can't find it right now. cc @bungle
https://github.com/Kong/kong/blob/master/kong/plugins/acme/handler.lua#L93

@fffonion, looked also there and the only one I found could be relevant was #12224 but without any further information.
But this issue must have started with PR #11703 from @bungle..
I'll add new certs manually for now so that my things don't stop work mid next week :P

Created https://konghq.atlassian.net/browse/KAG-4008 for tracking


EmscriptenApplication.js:35:2:WebXR unsupported in this browser

Hi,
I build webxr for emscripten in ubuntu, and generated :
EmscriptenApplication.js
magnum-webxr.html
magnum-webxr.js
magnum-webxr.wasm
WebApplication.css
I setup a server using python SimpleHTTPServer and access the html with my PC browser,
I could see a image in the browser but there's error log in console:WebXR unsupported in this browser
I tried it in chromium and Edge, they have the same issue.
my Chromium version is  83.0.4103.116（正式版本） （32 位）
Have anybody met this kind of issue? any help will be appreciated!

Pinging @Squareys, since he's the WebXR guru here :)

Hey @yylittlefish ,
Desktop Chromium don't support WebXR yet, I think. You could try Firefox with HTC Vive, but apart from that, you're still out of luck. It should definitely work on the Oculus Browser, though, in case you have an Oculus Quest or Oculus GO.

Hey @yylittlefish ,
Desktop Chromium don't support WebXR yet, I think. You could try Firefox with HTC Vive, but apart from that, you're still out of luck. It should definitely work on the Oculus Browser, though, in case you have an Oculus Quest or Oculus GO.

Hi, thank you for your reply!
"Desktop Chromium don't support WebXR yet" => does any other version Chromium support WebXR?
actually I finally want to run the webXR example in TV, it contains chromium browser too.
I installed firefox browser with version 78.0.1, but not connected with HTC Vive, it also shows "WebXR unsupported in this browser"
does the WebXR feature only supported when VR/XR devices are connected?
library_webxr.js calls w3c webxr apis in this page: https://www.w3.org/TR/webxr/#xr-device-concept
even if there's no HTC Vive, the APIs should be supported, right?(in both chromium and firefox).

Vaguely related, the WebXR demo is now also available on https://magnum.graphics/showcase/webxr/


Remove z-index from d2l-input-textarea.

This PR removes the z-index from d2l-input-textarea to avoid some stacking context issues. It looks like they were introduced when the component was initially converted to Lit, but it appears they aren't actually needed.
There's really two contributing factors to this issue:

d2l-input-textarea creating a stacking context that is higher than other things on the page when we don't want it to
d2l-card creating a stacking context for nested d2l-tooltips, effectively overriding the effectiveness of it's z-index. It would be good to eliminate this as well, but if it's possible, it will be more involved and can be dealt with separately.

Haha yeah... I was so happy to get rid of this... but my happiness was short-lived because I had to introduce a new one for the collapsible panel's sticky header in my next PR. 😢


Init Docker Image repository

[x] Init a docker repository to build image used by deps (for faster CI/CD)
[x] Write a bot to update and push this image
[ ] Update spaceporn dependencies workflows:

[ ] toolbox
[ ] vulkan
[ ] wayland
[ ] X11
[ ] glfw
[ ] cimgui
[ ] spirv
[ ] glslang
[ ] shaderc

Github Actions clears /usr/bin & /usr/local/bin locations before using an image so it is impossible to use custom images to save time on package/zig installation


Add support for parameters

Not sure if I am doing anything wrong or "parameters" is not supported. The "parameters" keyword wasn't get parsed properly and I was getting ReDoc error so I changed it to "parameter", which gives the following warning and also gets ignored by ReDoc.
Please update to align with the swagger v2.0 spec.
[ 'parameter' ]

Sorry opened the ticket in the wrong repo.


Gromacs XTC files

File Type
binary

Topological information
???

Positions
Yes

Velocities
Yes

Reference http://www.gromacs.org/Developer_Zone/Programming_Guide/XTC_Library

Read support was added through VMD molfiles plugins.
Leaving this open for write support.


Explore | Combine all the empty data fields

Fixes Issue
Closes #690

Changes proposed
This pull request fixes an issue in the Explore section. Previously, selecting 'all' filter with no items available displayed 'Not Found' message four times.
Changes:

Moved the empty list check to the Explore component. It now checks if all item lists are empty and displays a single 'Not Found' message if true.

This change provides a cleaner interface when no items are available.

Screenshots

Code of Conduct

[x] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).

@Som401 Hi, you handled only one case: when all data not found , the user will see only one Not Found message. However, you didn't combine cases when for example there are user details and blogs but there are not feeds and ctf , the user also should see only one message of Not Found : "Ctf and Feeds Not Found" - below the data.
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/f551b930-2571-4d3b-8517-92f0105b3b32

@ArkadiK94  I implemented a new function called renderNotFoundComponents, this function utilizes a string builder approach to construct the description for the NotFound component. It efficiently handles different scenarios, including when all data types are missing or when there are some missing data types.

@Som401 you didn't handle something right, there are some errors.
https://github.com/thecyberworld/TheCyberHUB/assets/76536506/7dae6dd9-a7bf-454b-86ba-3e0164a810c4

@Som401 It is not related to you, I will handle it...


bot.on("inline") does not work

bot.on('inline_query', function(msg) { 	console.log("inline!!"); });
Inline mod is set on, but it does not work.

@alikhil Do the other events work (like onText)?
Do you see the inline text when you mention the bot? (This text would've been set when enabling inline mode.

Other events are working and no I do not see placeholder text when mentioning the bot.

@alikhil the bot's handle should end with 'bot'. Like @somenamebot. Few days ago, I was trying to use the bot's name :P
Also make sure to give a space after the mention

@rithvikvibhu Yeah, I know about it. But it is still not working

If the placeholder text isn't working, something is wrong with your inline settings registered w/ Telegram. That text will show up regardless of if your bot is setup to receive the inline request.

@icdevin and @rithvikvibhu  Thank you. The problem was on the client side. I just restarted telegram and now it works.


Edit and additions

I added my name to both supporters.txt file and made a small edit to the diversity statement document.

Sorry, @stareagle — another commit has caused a conflict.  Can you rebase?

Adds names to two supporters files—this can be merged.
Adds "political leaning" to diversity statement, this needs community discussion period per our changes procedure. So this needs to be 2 PRs. I can get to it eventually if no one else does.

@stareagle Are you still interested in shepherding this through, or not so much?

So per my earlier comment this is sort of two PRs, but the only one we need to discuss is now redundant (political affiliation is already mentioned in the CoC). Therefore the last bit is adding @stareagle's name to the SUPPORTERS.txt which is fine and needs no discussion. If that's an incorrect interpretation, let me know — otherwise I'll merge this in a week


kpi_without_optimum vs original kpi

kpi_without_optimum is not matching with the target conversions of the original data

Hello @virithavanama !
kpi_without_optimum runs a prediction based on historical average media data. If you would like to compare against a different baseline you can always do so.
Can you further explain what you would expect it to match against?

If the total historical conversions is 48000, in the 'plot_pre_post_budget_allocation_comparison' plot preoptimized predicted target(kpi_without_optimum) is showing as 16000

That is possible if your total historic data is eg. 30 weeks and you are optimizing for 10 weeks. Could that be your case? (values are relative not absolute in my example)

No, we are using complete data for optimizing as well

n_time_periods = 52
prices = jnp.ones(mmm.n_media_channels)
budget = jnp.sum(jnp.dot(prices, ch_mean))* n_time_periods
solution, kpi_without_optim, previous_budget_allocation = optimize_media.find_optimal_budgets(
n_time_periods=n_time_periods,
media_mix_model=mmm,
#extra_features=extra_features_test,
budget=budget,
prices=prices,
media_scaler=media_scaler,
target_scaler=target_scaler,
bounds_lower_pct=0.75,
bounds_upper_pct=jnp.array([1.5,1.5,1.5,1.5,1.5,1.44,1.5,1.5,1.5]),
seed=SEED)

Thanks! Let me investigate this one and get back to you.

I am not able to recreate this great discrepancies with mock data.
What is the output when passing all the extra features as well to the optimisation?

What if you want to predict the future and then we will not have the extra features right?

Depends what those extra features are, for some (holidays, promotions, ...) you might know.  For others no, you will have to estimate them or set them to zero.
However if you extra features have a high positive impact and you want to be optimising for the future without them, output will not match historic absolute values. But I would not focus too much on the absolute value of the optimisation, in this case you know you are missing certain part of the value due to those missing extra features.

Discussion on #64 might be helpful as it is related to a very similar topic.


Calculate overage in build allocation

Extends upon work done in https://github.com/inventree/InvenTree/issues/335
The overage quantity is not included in the required part allocations when allocating parts to a build.

@SchrodingersGat Should this be taken into account given the recent build allocation fixes?

@eeintech yep, I'll add it to the list for the next release. It does need to be addressed, only took three years...


Automated Package Upgrade

:tada: This PR is included in version 1.3.31 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Allow useMatch take RouteDefinition object instead of path.

Basically I want to know which route is currently active outside of <Routes/>, so I can for example change nav bar styles on specific page, currently this is hard to do correctly. So my proposed solution is to allow useMatch take RouteDefinition object so this works:
const LIST_ROUTE = {
  path: '/list',
  component: List,
  children: [
    { path: '/' },
    { path: '/apples' },
    { path: '/oranges' },
    { path: '/banana' },
  ],
}

const routes: RouteDefinition[] = [
  {
    path: '/',
    component: Home,
  },
  LIST_ROUTE,
]

export const App = () => {
  const Routes = useRoutes(routes)
  const isListRoute = useMatch(LIST_ROUTE)

  return (
    <div>
      <Routes />
      <BottomNavBar hide={isListRoute()} />
    </div>
  )
}

Alternative today is to use
const isListRoute = useMatch(() => '/list/*') // Matches everything and is not a correct solution
// or way more complicated
const isListRoute = useMatch(() => '/list/')
const isListAppleRoute = useMatch(() => '/list/apple')
const isListOrangeRoute = useMatch(() => '/list/orange')
const isListBananaRoute = useMatch(() => '/list/banana')
const isListRouteActive= createMemo(() => isListRoute() && isListAppleRoute() && isListOrangeRoute() && isListBananaRoute())

There might be some other way to solve this like having <Routes /> fire change event with matched route id.

I think that this API is a bit awkward and I'd probably prefer something like this
useMatch('/list', { matchChildren: true })
useMatch('/list', { matchChildren: true, matchChildrenOnly: true })

See also this issue https://github.com/solidjs/solid-router/issues/127
According to the original proposal it could be
useMatch(path`/list/${['apple', 'orange', 'banana']}`)

Or the way I'd prefer, something like
useMatch('/list/(apple|orange|banana)')


Fixes GROOVY-8537 GroovyCollections.combinations(Iterable) exhibits incorrect (asymmetric) behavior.

The patch ensures no combination is returned if any of the input iterables to
GroovyCollections.combinations() is empty.
https://issues.apache.org/jira/browse/GROOVY-8537

Merged, thanks!


refactor(ci): use GitHub variables for non-sensitive info

Motivation
We've been using values that are variable across multiple workflows,
and those can only be changed if modifying the workflows, but we should
be able to change the values without committing new changes in the code
for this purpose we're now using GitHub Variables, and even moving
non-sensitive information into variables instead of secrets. Allowing
more flexibility and other scenarios that should be easier to manage,
like deploying to Mainnet or Testnet.
Closes: #6312
This will also make it easier to fix our actual main branch failures
Specifications
https://docs.github.com/en/actions/learn-github-actions/variables#defining-configuration-variables-for-multiple-workflows
Solution
Use GitHub variables for:

Our GCP Project name
Our registry URLs
GCP machine types
Other variables that have an impact on all or most workflows

Review
Anyone from the DevOps team, after test are passing (or before for desing feedback)
Check the actual values here: https://github.com/ZcashFoundation/zebra/settings/variables/actions
Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

This should also make it easier to fix main branch CI failures, when related to CPUs not being available on a specific region.

This is probably something for the next refactor, but can you change some of the secrets that don't need to be confidential into variables?
it's much easier to edit the value if we know what the previous value was.

This is probably something for the next refactor, but can you change some of the secrets that don't need to be confidential into variables?
it's much easier to edit the value if we know what the previous value was.

Can this be merged "as is" with follow ups in other PRs? Otherwise this will have to be parked until Sprint 8

This is probably something for the next refactor, but can you change some of the secrets that don't need to be confidential into variables?
it's much easier to edit the value if we know what the previous value was.

Can this be merged "as is" with follow ups in other PRs? Otherwise this will have to be parked until Sprint 8

Unfortunately, this PR uses a single variable for the machine type, but our current CI uses two different machine types. Changing all workflows to a single machine type could break CI or be really expensive, so we need to fix that before we merge:
https://github.com/ZcashFoundation/zebra/pull/6357#discussion_r1145388175

All the other suggestions could be done as follow-ups.

This should also make it easier to fix main branch CI failures, when related to CPUs not being available on a specific region.

We might want to try switching the large machine workflows from Zone 1f to 1b, because they are failing with "service unavailable" errors:
https://github.com/ZcashFoundation/zebra/pull/6426#issuecomment-1487749393

I'm bumping this down to a medium priority because I think we fixed the urgent bug another way in PR #6365

Aside from secrets.DOCKERHUB_USERNAME, all other non-sensitive information was replaced with a variable.

I have applied a quick fix to use the correct network variable, and a fix for another incorrect network default variable.

In general, it would help if we put more details in variable names, so that it's clearer what they should be used for:

vars.ZCASH_NETWORK -> vars.DEFAULT_ZCASH_NETWORK
${NETWORK} -> ${NETWORK_LOWERCASE}

Thanks for this, I'll take this into account for further refactors I have pending.

Failed in the merge queue due to #6498

@mergifyio refresh


cyclical beakerlib libraries dependency is not handled

Requiring following libraries causes the issue:
https://github.com/beakerlib/distribution/tree/master/Log, https://github.com/beakerlib/distribution/tree/master/opts
Currently the opts library has the requirement commented out to avoid the issue.
    discover
        workdir: /var/tmp/tmt/run-012/plans/default/discover
        how: fmf
        order: 50
            Run command 'git rev-parse --show-toplevel'.
            out: /home/dapospis/test/git/SP_components/tests
        directory: /home/dapospis/test/git/SP_components/tests
            Copy '/home/dapospis/test/git/SP_components/tests' to '/var/tmp/tmt/run-012/plans/default/discover/default/tests'.
            Check metadata tree in '/var/tmp/tmt/run-012/plans/default/discover/default/tests/'.
            Detected library 'library(distribution/dpcommon)'.
            Fetch library 'distribution/dpcommon'.
            Run command 'git clone https://github.com/beakerlib/distribution /var/tmp/tmt/run-012/plans/default/discover/default/libs/distribution'.
            err: Cloning into '/var/tmp/tmt/run-012/plans/default/discover/default/libs/distribution'...

            Detected library 'library(distribution/tcf)'.
            Library 'distribution/tcf' already fetched.
            Detected library 'library(distribution/Log)'.
            Library 'distribution/Log' already fetched.
            Detected library 'library(distribution/opts)'.
            Library 'distribution/opts' already fetched.
            Detected library 'library(distribution/Log)'.
            Library 'distribution/Log' already fetched.
            Detected library 'library(distribution/opts)'.
            Library 'distribution/opts' already fetched.
            Detected library 'library(distribution/Log)'.
            Library 'distribution/Log' already fetched.
            Detected library 'library(distribution/opts)'.
            Library 'distribution/opts' already fetched.
            Detected library 'library(distribution/Log)'.
            Library 'distribution/Log' already fetched.
            Detected library 'library(distribution/opts)'.
...
Traceback (most recent call last):
  File "/usr/bin/tmt", line 8, in <module>
    tmt.cli.main()
  File "/usr/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1290, in invoke
    return _process_result(rv)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1224, in _process_result
    value = ctx.invoke(self.result_callback, value, **ctx.params)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/lib/python3.9/site-packages/tmt/cli.py", line 280, in finito
    click_context.obj.run.go()
  File "/usr/lib/python3.9/site-packages/tmt/base.py", line 990, in go
    plan.go()
  File "/usr/lib/python3.9/site-packages/tmt/base.py", line 572, in go
    step.go()
  File "/usr/lib/python3.9/site-packages/tmt/steps/discover/__init__.py", line 125, in go
    plugin.go()
  File "/usr/lib/python3.9/site-packages/tmt/steps/discover/fmf.py", line 165, in go
    test.require, test.recommend, _ = tmt.beakerlib.dependencies(
  File "/usr/lib/python3.9/site-packages/tmt/beakerlib.py", line 212, in dependencies
    requires, recommends, libraries = dependencies(
  File "/usr/lib/python3.9/site-packages/tmt/beakerlib.py", line 212, in dependencies
    requires, recommends, libraries = dependencies(
  File "/usr/lib/python3.9/site-packages/tmt/beakerlib.py", line 212, in dependencies
    requires, recommends, libraries = dependencies(
  [Previous line repeated 969 more times]
  File "/usr/lib/python3.9/site-packages/tmt/beakerlib.py", line 209, in dependencies
    library = Library(dependency, parent=parent)
  File "/usr/lib/python3.9/site-packages/tmt/beakerlib.py", line 92, in __init__
    self.fetch()
  File "/usr/lib/python3.9/site-packages/tmt/beakerlib.py", line 118, in fetch
    self.parent.debug(f"Library '{self}' already fetched.", level=3)
  File "/usr/lib/python3.9/site-packages/tmt/utils.py", line 237, in debug
    self._log(self._indent(key, value, color=None, shift=shift))
  File "/usr/lib/python3.9/site-packages/tmt/utils.py", line 209, in _log
    self.parent._log(message)
  File "/usr/lib/python3.9/site-packages/tmt/utils.py", line 209, in _log
    self.parent._log(message)
  File "/usr/lib/python3.9/site-packages/tmt/utils.py", line 209, in _log
    self.parent._log(message)
  File "/usr/lib/python3.9/site-packages/tmt/utils.py", line 212, in _log
    log.write(remove_color(message) + '\n')
  File "/usr/lib/python3.9/site-packages/tmt/utils.py", line 935, in remove_color
    return re.sub(r'\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])', '', text)
  File "/usr/lib64/python3.9/re.py", line 210, in sub
    return _compile(pattern, flags).sub(repl, string, count)
  File "/usr/lib64/python3.9/re.py", line 291, in _compile
    if isinstance(flags, RegexFlag):
ecursionError: maximum recursion depth exceeded while calling a Python object

As a workaround I removed the requirement for the Log library from the opts library as normally tests / libs require Log library not the opts or both at the same time.
To reproduce the issue one needs to have two libraries which require each other.

Hi. I finally have working reproducer so I can try to handle this. Seems to be free.
Feel free to let me know if you want to work on this.

There is first draft, but it seems like interesting rabbit hole so I need to figure out how to debug tmt code running on virtual machine...
https://github.com/teemtee/tmt/pull/1149
Virtual machine output:
prepare
how: install
summary: Install required packages
name: requires
package: beakerlib and library(testprd/Logs)
... then nothing

What about just running with the full debug mode using tmt run -vvvddd? For example:
> cd tests/discover/libraries/data/
> tmt run -vvvddd discover plan --name apache
...
    discover
        workdir: /var/tmp/tmt/run-304/plan/apache/discover
        how: fmf
        order: 50
            Run command 'git rev-parse --show-toplevel'.
            out: /home/psss/git/tmt
        directory: /home/psss/git/tmt
            Copy '/home/psss/git/tmt' to '/var/tmp/tmt/run-304/plan/apache/discover/default/tests'.
            Run command 'git rev-parse --short HEAD'.
            out: 4dc7a31
        hash: 4dc7a31
        path: tests/discover/libraries/data
        tests: apache
            Check metadata tree in '/var/tmp/tmt/run-304/plan/apache/discover/default/tests/tests/discover/libraries/data'.
            Detected library 'library(httpd/http)'.
            Fetch library 'httpd/http'.
            Run command 'git clone https://github.com/beakerlib/httpd /var/tmp/tmt/run-304/plan/apache/discover/default/libs/httpd'.
            err: Cloning into '/var/tmp/tmt/run-304/plan/apache/discover/default/libs/httpd'...

            Detected library 'library(openssl/certgen)'.
            Fetch library 'openssl/certgen'.
            Run command 'git clone https://github.com/beakerlib/openssl /var/tmp/tmt/run-304/plan/apache/discover/default/libs/openssl'.
            err: Cloning into '/var/tmp/tmt/run-304/plan/apache/discover/default/libs/openssl'...

            Detected library '{'url': 'https://github.com/redhat-qe-security/certgen', 'name': '/certgen'}'.
            Fetch library 'certgen/certgen'.
            Run command 'git clone https://github.com/redhat-qe-security/certgen /var/tmp/tmt/run-304/plan/apache/discover/default/libs/certgen'.
            err: Cloning into '/var/tmp/tmt/run-304/plan/apache/discover/default/libs/certgen'...

Btw, the library fetching happens during the discover step, not prepare. Only libraries which cannot be fetched from git are handed over to prepare to try to install it using the old rpm way.

Ok. Looks like it works now. Maybe it was slow because my machine.
I think, there is another way how to fix this issue, but for now I'm happy that I was able to crack this.


Add .africa to top level domains and allow adding additional top level domains at run-time.

I've managed to work around this, issue appears to be related to installer permissions. However this version of IDFIX is not recognizing ,africa as a valid top level domain name causing false positives. Is this list stored within a configuration file in the application anywhere or would a file need to be updated  in the Github repository? I noticed in the build file download there is a file called domains.txt and I've verified that .africa is not listed there...
Originally posted by @peterjoinobits in https://github.com/microsoft/idfix/issues/36#issuecomment-713547424

Fixed in version <IP_ADDRESS>


Angular-UI-Router

lgtm overall, thanks for this
a comment left above

btw I had a discussion with Christopher some time ago and an idea was to keep 960px for bigger screens and min-width: 768px; so that we have a bit more space

I would rather just use google specs as defined here : https://material-design.storage.googleapis.com/publish/material_v_4/material_ext_publish/0B8olV15J7abPSGFxemFiQVRtb1k/layout_adaptive_breakpoints_01.png
https://www.google.com/design/spec/layout/responsive-ui.html#responsive-ui-breakpoints

Fixed bug in project review

lgtm


NavigationView - Android: screen will zoom in on the first leg of the multi stop route on Android when navigationViewController.showOverview is called.

Description
When route is initialized the screen will zoom in on the first leg of the multi stop route on Android. iOS zooms to the level of the full route is visible on the screen.
Environment details
Library version and other environment information:

react-native-navigation-sdk version: 0.6.1

OS type and version

iOS: 18.1, 18.0
Android: 35

Steps to reproduce

Pull down this branch.
Install the dependencies.
Run yarn start
Run yarn ios or yarn android
Click on navigate button
Once the screen is done loading you should see the map zoomed in on the first leg of the route for android.

Code example
Link to code example
Link to onRouteOk

Hi @rhinkle
Behaviour differs in underlaying Navigation SDK's for Android and iOS.
As a workaround support to animate camera to bounds should be implemented.
This difference need to be documented to the showOverview method.


[youtube] Regular expression for finding next page of search results is outdated.

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.07.28
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'ytsearchdate50:the']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.07.28
[debug] Python version 3.7.5 (CPython) - Linux-4.15.0-112-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: none
[debug] Proxy map: {}
[youtube:search:date] query "the": Downloading page 1
[download] Downloading playlist: the
[youtube:search:date] playlist the: Collected 20 video ids (downloading 20 of them)
[download] Downloading video 1 of 20

Description

The regular expression used to find the next page link in the results is currently not working. As a result, regardless of how many results you specify there is a limit to 20 results (the number of results on the first page).
The regex, found here should be:
r'href="(/results\?[^"]*\b(?:sp=[^"]+)?)"[^>]*>\s*<span[^>]+class="[^"]*\byt-uix-button-content\b[^"]*"[^>]*>Next'. I left in the incorrect part as optional, for any backwards compatibility issues.

I have created a pull request for this issue: https://github.com/ytdl-org/youtube-dl/pull/26163.

No longer actual.


DLNA failed to connect to Sony TV

What version of WebTorrent Desktop? (See the 'About WebTorrent' menu)
Version 0.17.2 (0.97.2)
What operating system and version?
macOS 10.12.1
What did you do?
Tried to cast movie to TV
What did you expect to happen?
To watch Movie in TV
What actually happened?
Failed to connect with errors in console

@grunjol Do you have any ideas about what could cause this?

With the info we have from the console screenshot I see its a problem with the  XML response from the TV. It seems the response is not well formed because the SAX parser (the library that decode the XML) is throwing the error. I can not solve the issue without the XML response (maybe they its sending a special character or trimming the response).
@phamviet Is this happening everytime you try to connect? did you try with another file?
Is this happening with another DLNA player?

@grunjol Even without seeing the XML response from @phamviet, we probably shouldn't have an uncaught exception when there's an error, right? Can we catch it?


DOI needs to be a searchable field

DOI needs to be a searchable field

We may wish to utilize the following
https://github.com/jprante/elasticsearch-analysis-standardnumber


hover text color in button fixed!

"When we hover over some of the buttons, the text color does not change to black!" issue fixed
added comments to the code for more understandability.

fixed the button hover issue with comments added to the code for more precission


Support models larger than 8K

It will be great to support inputs larger than 8K tokens for GPT-35-16K and GPT-4-32K.
Thanks

Hi, not sure if I understand the question, but you should be able to use these models: https://github.com/silvanmelchior/IncognitoPilot#change-model

Hi,
The larger token models were working (by following the link above) if the conversations used less than 8K tokens.
For both the models I outlined, when using a query larger than 8K (I tested with 8.7K) it hung. The prompt disappeared instead of providing an answer.
As an example, I asked it to take the input (8.7K tokens) and generate a file with that content.

So interestingly, when I try it with gpt-3.5-turbo-16k, and have a long request, then the model starts to output wrong function call arguments. So it should output {code: "<python-code>"}, but it actually outputs the python code directly. The backend then cannot parse this, because it's no valid JSON.
We could try to just assume that it's the code, if the JSON is invalid, but I'm not sure how stable it is.
So far I would just see it as a limitation of the models, maybe future versions will overcome this.


Update dependencies.sh to other medusa name

Merging this change to test new lerna tooling build on heroku


Implemented a quick and simple method to get the client's IP

I through it was a good idea to get the IP from the Agent class as it handles "all" the information from the client's.
What do you think?

its so difficult to use: $request->ip() ?


CSRF token mismatch when InitializeTenancyByRequestData middleware is used

Bug description
I'm building the APIs for my SPA using the SPA Authentication. I realized that when I apply any of the tenancy initialization middlewares such as InitializeTenancyByRequestData, I got the "CSRF token mismatch." issue. Is anyone facing the same issue?
Steps to reproduce

Make a request to get CSRF token
Make a POST request

Expected behavior
Receive 419 "CSRF token mismatch" error response
Laravel version
11.18.1
stancl/tenancy version
3.8

Please use the right channels for support questions.


Missing documentation on "configuration"

Hi
In the Template Designer Documentation it is said that

If an application configures Jinja to trim_blocks...

However, I can't find any hint how I can configure Jinja - neither on jinja.pocoo.org, nor on flask.pocoo.org nor via Google.
So my question is: How do I configure Jinja2 to accept trim_blocks/lstrip_blocks?
Best
Edit: Reading the code I've discovered this work around. Is this the way to go?
# in bootstrap
wsgi = Flask(__name__.split('.')[0])

wsgi.jinja_env.lstrip_blocks = True
wsgi.jinja_env.trim_blocks = True

From what I know you pass these settings to the Environment initialization.
See https://github.com/mitsuhiko/jinja2/blob/9b4b20aa56fde3a5cd5ac49d4feacd96eacb832d/jinja2/environment.py#L143
And https://github.com/mitsuhiko/jinja2/blob/master/tests/test_lexnparse.py#L444


[Android] CachedImage doesn't load in SFListView until after scrolled off screen

I'm finding some strange behavior whereby when I load a CachedImage into an ItemTemplate within an SFListView, I see a blank space where the image should be until I scroll.
Here's a template
<ContentView xmlns="http://xamarin.com/schemas/2014/forms"
             xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
             xmlns:xaml="clr-namespace:MyApp.SelfServe.Mobile.Xaml;assembly=MyApp.SelfServe.Mobile"
             xmlns:forms="clr-namespace:FFImageLoading.Forms;assembly=FFImageLoading.Forms"
             xmlns:controls="clr-namespace:MyApp.SelfServe.Mobile.Controls;assembly=MyApp.SelfServe.Mobile"
             x:Class="MyApp.SelfServe.Mobile.Pages.UserAccount.Projects.Templates.ProjectListTemplate">
    <Grid HorizontalOptions="Fill" VerticalOptions="Fill" Margin="0" Padding="0"
          BackgroundColor="{Binding IsSelected, Converter={StaticResource SfListViewSelectionConverter}}"
          RowSpacing="0">
        <Grid.RowDefinitions>
            <RowDefinition>
                <RowDefinition.Height>
                    <OnPlatform x:TypeArguments="GridLength">
                        <On Platform="Android" Value="120" />
                        <On Platform="iOS" Value="100" />
                    </OnPlatform>
                </RowDefinition.Height>
            </RowDefinition>
            <RowDefinition Height="1" />
        </Grid.RowDefinitions>
        <Grid.ColumnDefinitions>
            <ColumnDefinition Width="5" />
            <ColumnDefinition Width="50" />
            <ColumnDefinition Width="*" />
            <ColumnDefinition Width="40" />
        </Grid.ColumnDefinitions>

        <BoxView Grid.Column="0"
                 Grid.Row="0"
                 Color="{Binding Phase, Converter={StaticResource ProjectPhaseToColorConverter}, ConverterParameter={Binding HasAlert}}"
                 WidthRequest="5"
                 HeightRequest="999"
                 VerticalOptions="FillAndExpand" />

        <forms:CachedImage Grid.Column="1"
                           Grid.Row="0"
                           HorizontalOptions="Center"
                           VerticalOptions="Center"
                           Source="{Binding PhaseIcon}" />

        <StackLayout Grid.Column="2"
                     Grid.Row="0"
                     Spacing="2"
                     VerticalOptions="Center">
            <Label Text="{Binding Nickname}" Style="{StaticResource Common-Title}"
                   LineBreakMode="TailTruncation" />
            <Label FormattedText="{Binding RequestNumberFormatted}"
                   LineBreakMode="TailTruncation" />
            <Label FormattedText="{Binding PhaseFormatted}"
                   LineBreakMode="TailTruncation" />
            <Label FormattedText="{Binding SiteIdFormatted}"
                   LineBreakMode="TailTruncation" />
            <Label FormattedText="{Binding LocationFormatted}"
                   LineBreakMode="TailTruncation" />
        </StackLayout>

        <controls:TintedCachedImage
            Grid.Column="3"
            Grid.Row="0"
            Source="arrow.png"
            HorizontalOptions="Center"
            VerticalOptions="Center"
            TintColor="{Binding Phase, Converter={StaticResource ProjectPhaseToColorConverter}, ConverterParameter={Binding HasAlert}}"
            InputTransparent="True"
            IsVisible="{Binding IsBusy, Converter={StaticResource InverseBoolConverter}}" />

        <ActivityIndicator Grid.Column="3"
                           Grid.Row="0"
                           IsVisible="{Binding IsBusy}"
                           IsRunning="{Binding IsBusy}" />
        <BoxView Grid.Column="0"
                 Grid.Row="1"
                 Grid.ColumnSpan="4"
                 HeightRequest="1"
                 HorizontalOptions="FillAndExpand"
                 Color="{xaml:ResxColor PrimaryDarkBlue}" />
    </Grid>
</ContentView>

Page Load

Partial Scroll Down

After Scroll down and back to top

I also tried this in the codebehind to remove the binding... but it gives the same result.
public partial class ProjectListTemplate
{
    public ProjectListTemplate()
    {
        InitializeComponent();
        ResetImage();
    }

    protected override void OnBindingContextChanged()
    {
        ResetImage();
        base.OnBindingContextChanged();
    }

    private void ResetImage()
    {
        var item = BindingContext as ProjectsTabItem;
        if (item == null)
            return;

        PhaseImage.Source = item.PhaseIcon;
    }
}

Does it work when you replace CachedImage with Image?
Could you give some examples of PhaseIcon binding values?

Hey @daniel-luberda, yes it's works with <Image />, I'm actually attempting to improve our scrolling performance by exactly swapping <Image /> for <forms:CachedImage />
The PhaseIcon is set in the code behind like so.

I'm getting the same behavior in a normal listview.
It also doesn't display the placeholder image.
I upgraded from 2.2.20 and tested the following releases (all on Xamarin.Forms <IP_ADDRESS>255-pre3):
2.2.20 works
2.2.21 works
2.2.22 works
2.2.23 doesn't work
2.2.24-pre-605 doesn't work
2.2.24 doesn't work
so the last version where i get the expected behavior is the 2.2.22

@gmwilhelm Thanks for the info. So it looks like fast renderer is a problem.

Should be fixed now, thanks! :)

@daniel-luberda  Is this enough of a change to merit a new patch release (or at least pre-release)?

Yes, I want to fix issue 808 and release next stable today.

@daniel-luberda, that last update works, ty


add configuration support for Page title without Action name as prefix

Triggered by https://github.com/filamentphp/filament/discussions/8108.
It is possible to manually override a specific Page title to only show the record title (e.g. "John Doe") instead of getting the Action name prefixed (e.g. "View John Doe").
This PR adds configuration support so that the Action name is not shown in the title of any View / Edit page.
    return $panel
        // ...
        ->titleAction(showAction: false);

[x] Changes have been thoroughly tested to not break existing functionality.
[x] New functionality has been documented or existing documentation has been updated to reflect changes.
[ ] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Sorry, but I'm not a fan. You could use a trait or an abstract class to do this if you want.

Wanted to give something back to the community, but up to you, you're the boss. ;-)
The reason for it was: for all my projects I prefer this behaviour for all my Resources.
Therefore I feel config is the best place, instead of adding trait to each Resource in each project.


Get interpolated property value multiple assets

What happened:
I added multiple assets with the same asset property to the Get interpolated property value query and only saw data appear for one of the assets.
What you expected to happen:
Data should appear for each asset in separate data frames.
How to reproduce it (as minimally and precisely as possible):

Add a query for Get interpolated property value
Select multiple assets from the same asset model with data
Select an asset property
See a set of data appear for only one of the assets - unclear which one

Screenshots

Anything else we need to know?:
Environment:

Grafana version: 9.4.0
Plugin version: 1.12.1
OS Grafana is installed on: MacOS
User OS & Browser: Sonoma 14.5, Chrome
Others: Amazon Managed Grafana environment

This has already been taken care of here: https://github.com/grafana/iot-sitewise-datasource/pull/256
Will upgrade my version of the plugin.


HtmlRspackPlugin add support for alwaysWriteToDisk

What problem does this feature solve?
In hybrid application setups with mixed front-end and back-end technologies it can often be useful for the back-end to serve the index.html
This works fine in production but the index.html file is not outputted in development mode.
Current solution is to use the  HtmlWebpackHarddiskPlugin to enable alwaysWriteToDisk in HtmlWebpackPlugin - this works fine, but would be great to use the HtmlRspackPlugin and have less dependencies to install.
What does the proposed API of configuration look like?
One proposal could be to add an option alwaysWriteToDisk to the HtmlWebpackPlugin:
    new HtmlWebpackPlugin({
      template: "./public/index.html",
      meta: {
        runtimeEnv: "%ENCODED_RUNTIME_ENV%",
      },
      alwaysWriteToDisk: true, // Always write the index.html file to disk
      publicPath: "%CDN_URL%",
    }),

One could also add a similar option in the dev server options, maybe allowing a filter to prevent all assets to be written to disk

module.exports = {
// ...other configs
  devServer: {
    devMiddleware: {
      writeToDisk: (filename) => {
        return /index.html$/.test(filename);
      },
    },
  },
}

should work. please give a try

Thanks @xc2 that worked perfectly! ❤


Needs timeout value

Requests does not timeout without a timeout value being passed to it. Right now there isn't any mechanism to pass a timeout to the individual request calls that I can tell, so a down device can hang indefinitely (I've seen up to 15 minutes in my environment which is probably when my firewall is killing the session, but in theory requests will hang forever if you let it).
Should probably be able to define a timeout on the FortiOSAPI init and then update the various _session calls to include the timeout option.

Hello,
The login method does timeout, hence I did not encounter letting the session half open and the Fortigate disappear.
You are right thought and will implement the timeout method in the requests calls soon.
Thanks for the bug

Fixed in 0.10.0 version


Disable the Hubble before starting with UI via cilium-cli.

Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

Fixes: #24587

address doc section related to https://github.com/cilium/cilium/issues/24587, which might lead the user to meet the unexpected failure to start the Hubble UI.

Can you please also drop the first, unrelated commit that sneaked in into your PR? I think this is because you initially worked on the master branch, and that commit was merged there by mistake.

Can you please also drop the first, unrelated commit that sneaked in into your PR? I think this is because you initially worked on the master branch, and that commit was merged there by mistake.

done in https://github.com/cilium/cilium/commit/de019848abd38aa1155c7fbd4711081505a25234

Just FYI, created follow-up feature request to cilium-cli, https://github.com/cilium/cilium-cli/issues/1647

Doc change only, no need for the full CI suite.

thanks guys for taking the time to review.
btw, do i need to rebase? technically i do not think that is required to do so for this PR, but i can see Merging is blocked though.


Modify how narrator names the first bit of a number in Bit Flip

Fixes #385
#640 modified how CalculatorProgrammerBitFlipPanel generates texts for the Narrator, we can now easily modify the text for "0th bit" and use "least significant bit" instead.
Description of the changes:

replace "0th bit"  by "least significant bit"

How changes were validated:

Manually

(Let me know if you also want to use "most significant bit")

If it is not too much trouble :)

If it is not too much trouble :)

Will do it! It's a little more complicated than LSB (but still simple) because we need to update the text of the previous MSB and the new one when we change the bit length.


Slider value becomes on first load NaN

Describe the bug
When loading image for the first time and trying to use paint tool the range slider is not usable. It becomes NaN on click and cannot be dragged.
I am loading the image by using the infamous .loadImageFromURLmethod. I think this might be causing this bug. Because the second time I open the image editor the slider works fine.
To Reproduce
Steps to reproduce the behavior:

Open image editor
Open Draw tool
Click and draw the slider
Slider does not work and value becomes NaN

Expected behavior
After loading the image editor second time the value is correct and slider works.
Screenshots

Desktop (please complete the following information):

OS: Mac OS Catalina 10.15.3
Browser Chrome 80
Version TUI image editor 3.9.0

It doesn't reappearance well. Use a tool like codepen to tell me more.

when I add Editor in Modal I run into an issue, on changing the value from slider it is changing to NaN, But if the modal is placed on a page without modal. This is working fine