Using show.bind on mdc-toolbar results in error

I tried to use show.bind on mdc-toolbar and got an error.
The toolbar:
<mdc-toolbar class="mdc-elevation--z2" fixed="true" show.bind="true"></mdc-toolbar>

The error:
ERROR [app-router] TypeError: Cannot read property 'remove' of undefined
    at Animator.removeClass (http://localhost:9000/scripts/vendor-bundle.js:19227:25)
    at Show.valueChanged (http://localhost:9000/scripts/vendor-bundle.js:25773:23)
    at Show.bind (http://localhost:9000/scripts/vendor-bundle.js:25780:12)
    at Controller.bind (http://localhost:9000/scripts/vendor-bundle.js:22567:24)
    at View.bind (http://localhost:9000/scripts/vendor-bundle.js:20565:24)
    at Controller.bind (http://localhost:9000/scripts/vendor-bundle.js:22559:19)
    at Controller.automate (http://localhost:9000/scripts/vendor-bundle.js:22504:12)
    at ready (http://localhost:9000/scripts/vendor-bundle.js:26849:40)
    at RouterView.swap (http://localhost:9000/scripts/vendor-bundle.js:26878:14)
    at http://localhost:9000/scripts/vendor-bundle.js:17389:29
    at Array.forEach (<anonymous>)
    at http://localhost:9000/scripts/vendor-bundle.js:17388:20
From previous event:
    at NavigationInstruction._commitChanges (http://localhost:9000/scripts/vendor-bundle.js:17387:33)
    at CommitChangesStep.run (http://localhost:9000/scripts/vendor-bundle.js:17220:36)
    at next (http://localhost:9000/scripts/vendor-bundle.js:17189:20)
    at iterate (http://localhost:9000/scripts/vendor-bundle.js:18349:14)
    at inspect (http://localhost:9000/scripts/vendor-bundle.js:18326:16)
    at http://localhost:9000/scripts/vendor-bundle.js:18342:20
    at processPotential (http://localhost:9000/scripts/vendor-bundle.js:18460:14)
    at iterate (http://localhost:9000/scripts/vendor-bundle.js:18341:18)
    at processActivatable (http://localhost:9000/scripts/vendor-bundle.js:18352:12)
    at ActivateNextStep.run (http://localhost:9000/scripts/vendor-bundle.js:18238:14)
    at next (http://localhost:9000/scripts/vendor-bundle.js:17189:20)
    at iterate (http://localhost:9000/scripts/vendor-bundle.js:18268:14)
    at processDeactivatable (http://localhost:9000/scripts/vendor-bundle.js:18271:12)
    at DeactivatePreviousStep.run (http://localhost:9000/scripts/vendor-bundle.js:18226:14)
    at next (http://localhost:9000/scripts/vendor-bundle.js:17189:20)
    at iterate (http://localhost:9000/scripts/vendor-bundle.js:18349:14)
    at inspect (http://localhost:9000/scripts/vendor-bundle.js:18326:16)
    at http://localhost:9000/scripts/vendor-bundle.js:18342:20
From previous event:
    at processPotential (http://localhost:9000/scripts/vendor-bundle.js:18430:35)
    at iterate (http://localhost:9000/scripts/vendor-bundle.js:18341:18)
    at processActivatable (http://localhost:9000/scripts/vendor-bundle.js:18352:12)
    at CanActivateNextStep.run (http://localhost:9000/scripts/vendor-bundle.js:18214:14)
    at next (http://localhost:9000/scripts/vendor-bundle.js:17189:20)
From previous event:
    at LoadRouteStep.run (http://localhost:9000/scripts/vendor-bundle.js:18490:68)
    at next (http://localhost:9000/scripts/vendor-bundle.js:17189:20)
    at iterate (http://localhost:9000/scripts/vendor-bundle.js:18268:14)
    at processDeactivatable (http://localhost:9000/scripts/vendor-bundle.js:18271:12)
    at CanDeactivatePreviousStep.run (http://localhost:9000/scripts/vendor-bundle.js:18202:14)
    at next (http://localhost:9000/scripts/vendor-bundle.js:17189:20)
    at http://localhost:9000/scripts/vendor-bundle.js:17654:16
From previous event:
    at BuildNavigationPlanStep.run (http://localhost:9000/scripts/vendor-bundle.js:17652:58)
    at next (http://localhost:9000/scripts/vendor-bundle.js:17189:20)
    at Pipeline.run (http://localhost:9000/scripts/vendor-bundle.js:17202:14)
    at http://localhost:9000/scripts/vendor-bundle.js:18801:25
From previous event:
    at AppRouter._dequeueInstruction (http://localhost:9000/scripts/vendor-bundle.js:18774:32)
    at AppRouter.activate (http://localhost:9000/scripts/vendor-bundle.js:18751:12)
    at http://localhost:9000/scripts/vendor-bundle.js:18729:23
    at <anonymous>
From previous event:
    at AppRouter.registerViewPort (http://localhost:9000/scripts/vendor-bundle.js:18728:16)
    at new RouterView (http://localhost:9000/scripts/vendor-bundle.js:26768:19)
    at Object.invokeWithDynamicDependencies (http://localhost:9000/scripts/vendor-bundle.js:12534:20)
    at InvocationHandler.invoke (http://localhost:9000/scripts/vendor-bundle.js:12509:168)
    at Container.invoke (http://localhost:9000/scripts/vendor-bundle.js:12780:25)
    at ProviderResolver.get (http://localhost:9000/scripts/vendor-bundle.js:21149:74)
    at Container.get (http://localhost:9000/scripts/vendor-bundle.js:12719:23)
    at Container.elementContainerGet [as get] (http://localhost:9000/scripts/vendor-bundle.js:21204:17)
    at HtmlBehaviorResource.create (http://localhost:9000/scripts/vendor-bundle.js:23181:58)
    at applyInstructions (http://localhost:9000/scripts/vendor-bundle.js:21306:33)
    at ViewFactory.create (http://localhost:9000/scripts/vendor-bundle.js:21527:9)
    at HtmlBehaviorResource.create (http://localhost:9000/scripts/vendor-bundle.js:23193:41)
    at http://localhost:9000/scripts/vendor-bundle.js:23679:18
From previous event:
    at CompositionEngine.createController (http://localhost:9000/scripts/vendor-bundle.js:23678:10)
    at CompositionEngine._createControllerAndSwap (http://localhost:9000/scripts/vendor-bundle.js:23640:19)
    at CompositionEngine.compose (http://localhost:9000/scripts/vendor-bundle.js:23726:21)
    at TemplatingEngine.compose (http://localhost:9000/scripts/vendor-bundle.js:23989:38)
    at Aurelia.setRoot (http://localhost:9000/scripts/vendor-bundle.js:13230:21)
    at http://localhost:9000/scripts/app-bundle.js:328:22
From previous event:
    at Object.configure (http://localhost:9000/scripts/app-bundle.js:327:35)
    at http://localhost:9000/scripts/vendor-bundle.js:12059:29
From previous event:
    at config (http://localhost:9000/scripts/vendor-bundle.js:12054:56)
    at http://localhost:9000/scripts/vendor-bundle.js:12090:14
From previous event:
    at bootstrap (http://localhost:9000/scripts/vendor-bundle.js:12089:26)
    at http://localhost:9000/scripts/vendor-bundle.js:12076:9
    at <anonymous>
From previous event:
    at run (http://localhost:9000/scripts/vendor-bundle.js:12071:61)
    at Object.<anonymous> (http://localhost:9000/scripts/vendor-bundle.js:12096:37)
    at Object.execCb (http://localhost:9000/scripts/vendor-bundle.js:5484:33)
    at Module.check (http://localhost:9000/scripts/vendor-bundle.js:4671:51)
    at Module.enable (http://localhost:9000/scripts/vendor-bundle.js:4964:22)
    at Object.enable (http://localhost:9000/scripts/vendor-bundle.js:5345:39)
    at Module.<anonymous> (http://localhost:9000/scripts/vendor-bundle.js:4949:33)
    at http://localhost:9000/scripts/vendor-bundle.js:3922:23
    at each (http://localhost:9000/scripts/vendor-bundle.js:3847:31)
    at Module.enable (http://localhost:9000/scripts/vendor-bundle.js:4901:17)
    at Module.init (http://localhost:9000/scripts/vendor-bundle.js:4576:26)
    at http://localhost:9000/scripts/vendor-bundle.js:5248:36

I drifted badly into containerless mode in the start to make the design work. Such errors when different functions can't access element is some of the side effect of this.
I removed the containerless decorator of the toolbar and compared doc's before and after. New version on its way :smile:

Thanks. Does using containerless cause bad problems? I've found a few situations where I have to use it because the mdc css isnt expecting an extra element. For instance an mdc-checkbox inside of a list-item or if I use a custom element to output an mdc-grid-cell.

I used containerless for almost all components in the bridge. Like you say; the css isnt expecting extra element. I started to get some trouble like the one in this issue. It was hard to debug. And attributes like class needed to be handled.
I ended up in testing and removing containerless wherever I could. I also left some. Maybe time to check if some of these also can be removed.
External css style problems is one good reason to use containerless decorator.
These items still use containerless:

grid-cell
simple-menu
tab
toolbar-row
toolbar-section
toolbar-title

.. and has not been tested without.. yet.

Thanks, good to know.


We might need to upload our data and make a data/ folder

I know we aren't using any local data and our data is accessed through a library, but we might need download the data from the titanic package and include it.

Yes that would be nice to anyone trying to contribute

that is a good point. Issue has been fixed


Datamart Integration is broken in branch annoint-merge

While trying to upload an annotated spreadsheet, I am getting this error,
This file is already present in the project's data files
Traceback (most recent call last):
  File "/Users/amandeep/Github/t2wml/backend/application.py", line 212, in upload_data_file
    i_f = ItemsFile.create_from_dataframe(project, combined_item_df)
  File "/Users/amandeep/Github/t2wml/backend/models.py", line 370, in create_from_dataframe
    wf = cls.create_from_filepath(project, filepath)
  File "/Users/amandeep/Github/t2wml/backend/models.py", line 237, in create_from_filepath
    saved_file.add_to_api_proj()
  File "/Users/amandeep/Github/t2wml/backend/models.py", line 374, in add_to_api_proj
    self.project.api_project.add_wikidata_file(self.relative_path)
AttributeError: 'Project' object has no attribute 'add_wikidata_file'
'Project' object has no attribute 'add_wikidata_file'
INFO  [werkzeug] <IP_ADDRESS> - - [09/Sep/2020 17:33:38] "POST //api/data/12 HTTP/1.1" 200 -`

To reproduce, follow these steps,

Run these commands in order in a terminal

git clone https://github.com/usc-isi-i2/t2wml-annotation

git clone https://github.com/usc-isi-i2/t2wml-api
cd t2wml-api
git checkout development

cd ..
git clone https://github.com/usc-isi-i2/datamart-api
cd datamart-api
git checkout development

python3.6 -m venv datamart_env
source datamart_env/bin/activate
pip install --upgrade pip

pip install -r requirements.txt
pip install -e ../t2wml-annotation
pip install -e ../t2wml-api

cd dev-env
docker-compose up -d # This above step might take a while to complete as it will start and populate the database if  this is the first time running this command. Wait until the database is up and running and available to accept requests

cd ..

python app.py

Update the datamart api endpoint in t2wml/backend/app_config.py

DATAMART_API_ENDPOINT = 'http://localhost:12543'

Start T2WML from the branch annoint-merge

Upload this file to a project in T2WML - https://github.com/usc-isi-i2/datamart-api/blob/development/test/test_data/04_worker_incidents.xlsx

See the above mentioned error

I merged the branch annotation2 to annotation-integration. We should continue working from annotation-integration.


Fix for PublishSingleFile Launch Crash

PR Details
Stride applications built for Windows will crash when PublishSingleFile is set to true.
Description
When a Stride application is launched, Assembly.Location is used to find the executable directory and create cache folders. This returns a null value when PublishSingleFile is true. It is instead recommended to use AppContext.BaseDirectory to check for the executable directory.
Additional Info: https://learn.microsoft.com/en-us/dotnet/core/deploying/single-file/warnings/il3000

Related Issue
https://github.com/stride3d/stride/issues/2204
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] I have built and run the editor to try this change out.

Might be able to also close https://github.com/stride3d/stride/issues/1947 with this fix.

Might be able to also close #1947 with this fix.

That's a good idea, inline with that the docs should also be updated at some point to include PublishSingleFile instructions.
https://doc.stride3d.net/latest/en/manual/files-and-folders/distribute-a-game.html


[WIP] Transform PHP 8 qualified names

Two concerns here:

Does this code actually transform every occurrence? Does the foreach still get the tokens that were shifted right to make room after transformation?
Why does this not transform anything. :trollface: Can someone help? // cc @SpacePossum

When you say "Why does this not transform anything.", which exactly do you mean? Are your tests failing?
Are you running your code in PHP 8.0.0RC1? If so, I'm surprised you didn't run into the Call to undefined method PhpCsFixer\Tokenizer\Tokens::rewind() backwards-compatibility issue.

Are your tests failing?

Yes, see the travis build.

Yes, see the travis build.

Oops, I should have clarified! I checked the Travis build and noticed that it failed the static code analysis (since it runs on PHP 7.4), so tests didn't run.  That's why I asked about your tests.
Managing the CI builds will be tricky as we introduce changes for PHP 8.  I've been pondering on that and might propose something.

Oh. Forgot travis was configured in that way.

Hi and sorry for not coming back on this earlier, struggled to find time to work on PHP8 support : (
Thanks for your WIP PR.

Does this code actually transform every occurrence? Does the foreach still get the tokens that were shifted right to make room after transformation?

yeah, that will work fine

Why does this not transform anything. :trollface: Can someone help? // cc

I went ahead and wrote a similar transformer and sure run into the same bug;
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/5166/files#diff-f0f05b598c6bff06cf727fc98b5d1191f18793eb4f9dce0c61b7d93ec8a7a334L980
The issue was that when inserting tokens of a different kind but with the same content the replace was not done :tada:
Let me know if you spot anything off in my PR or if you've time to work on some other PHP8 issue here :)

👍


Openblas crash on android

Hi,
I did successfully finish the cross-compile(TARGET=ARMv7) according to https://github.com/xianyi/OpenBLAS/wiki/How-to-build-OpenBLAS-for-Android. When linking the generated static library with NDK, I got the error:
uses VFP register arguments, output does not
Then I did set  the params in my Android.mk, and the error gone.
LOCAL_CFLAGS += -mhard-float -D_NDK_MATH_NO_SOFTFP=1
LOCAL_LDFLAGS += -Wl,--no-warn-mismatch -lm_hard.
But when  I ran the simple example, my app crashed without any info:
#include <cblas.h>
#include <stdio.h>
void main()
{
int i=0;
double A[6] = {1.0,2.0,1.0,-3.0,4.0,-1.0};
double B[6] = {1.0,2.0,1.0,-3.0,4.0,-1.0};
double C[9] = {.5,.5,.5,.5,.5,.5,.5,.5,.5};
cblas_dgemm(CblasColMajor, CblasNoTrans, CblasTrans,3,3,2,1,A, 3, B, 3,2,C,3);
for(i=0; i<9; i++)
printf("%lf ", C[i]);
printf("\n");
}
I will really appreciate if you can help me.

Is it a spelling error?
s/LOCAL/TARGET/g

Not a spelling error. It did seems that 'LOCAL' cleans the link error, though the apk still cannot run. And I have tried 'TARGET'，but it didn't work. I did not find some variables named "TARGET_CFLAGS " or "TARGET_LDFLAGS " at https://developer.android.google.cn/ndk/guides/android_mk.html , while I was wondering why did you use 'TARGET'. Or did I have any misunderstand?

The warning message you show comes exactly from not setting TARGET_ variables.

Can you make consistent set build log and backtrace from crash?
Preferably with single-threaded openblas if it crashes in that case (easier to debug with your hands)

The TARGET_ variables works for me finally! The crash is just one naive mistake having nothing to do with openblas! thx so much!

@gaoqiang
hey ! I just have the same problem with you , can you explain more detail about solutions?  Where did you set the  "TARGET_" variables?

@brada4
Hey! I just want to ask you where should I set "TARGET_" variables?  in Android.mk ?

Typically in application.mk, system environment is valid too.

@brada4 Thanks for quick reply.  I will try.

I set "TARGET_" variables in Android.mk and it worked. I did not try application.mk.

@gaoqiang  THX ! I will try it .


Update README with GitHub issues line

https://guides.github.com/features/issues/

https://github.com/blog/2051-protected-branches-and-required-status-checks

https://help.github.com/articles/closing-issues-via-commit-messages/


Configuring a new Snowflake provider using the account id of the generated account

Provider Version
The provider version you are using.
snowflake-labs/snowflake v0.73.0
Terraform Version
The version of Terraform you were using when the bug was encountered.
Terraform v1.6.1
Describe the bug
A clear and concise description of what the bug is.
In a single deployment, I want to create a Snowflake ACCOUNTADMIN Provider, where the account field is the id generated by creating a new account, using the resource snowflake_account(using a different provider, based off ORGADMIN)
The issue has to do with creating the new provider
provider "snowflake" {
  alias    = "accountadmin"
  role     = "ACCOUNTADMIN"
  account  = snowflake_account.account.id #Issue arises here
  username = <insert user>
  password = <insert pass>
}

provider "snowflake" {
  alias    = "orgadmin"
  role     = "ORGADMIN"
  account  = <insert>
  username = <insert>
  password = <insert>
}

resource "snowflake_account" "account" {
  provider             = snowflake.orgadmin
  name                 = "<insert>"
  admin_name           = <insert>
  admin_password       = <insert>
  first_name           = "<insert>"
  last_name            = "<insert>"
  email                = "<insert>"
  must_change_password = true
  edition              = "STANDARD"
  comment              = <insert>
  region               = <insert>
}

When trying this, I get this error
Error: could not build dsn for snowflake connection err = 260000: account is empty
│
│   with module.main.module.snowflake.provider["registry.terraform.io/snowflake-labs/snowflake"].accadmin,
│   on ..\..\modules\snowflake\main.tf line 19, in provider "snowflake":
│   19: provider "snowflake" {

Expected behavior
A clear and concise description of what you expected to happen.
Successful terraform plan, where the Snowflake ACCOUNTADMIN provider can be created, depending on the read_only id of the snowflake_account, in one deployment. This new provider will be used to create resources under the new account
Code samples and commands
Please add code examples and commands that were run to cause the problem.
provider "snowflake" {
  alias    = "accadmin"
  role     = "ACCOUNTADMIN"
  account  = snowflake_account.account.id # This is where the issue arises
  username =<insert>
  password = <insert>
}

provider "snowflake" {
  alias    = "orgadmin"
  role     = "ORGADMIN"
  account  = <insert>
  username = <insert>
  password = <insert>
}

resource "snowflake_database" "env_db" {
  provider = snowflake.accadmin
  name     = "${var.env}_env_DB"
}

resource "snowflake_account" "account" {
  provider             = snowflake.orgadmin
  name                 = "<insert>"
  admin_name           = <insert>
  admin_password       = <insert>
  first_name           = "<insert>"
  last_name            = "<insert>"
  email                = "<insert>"
  must_change_password = true
  edition              = "STANDARD"
  comment              = <insert>
  region               = <insert>
}

Additional context
Add any other context about the problem here.
Let me know if this makes sense or if you have any questions. I can add anymore details if needed.
Thank you!

Hey @AyushSVarma. Thanks for creating the issue.
Unfortunately, this is not a bug but a Terraform limitation.
In the Terraform docs on provider configuration:

You can use expressions in the values of these configuration arguments, but can only reference values that are known before the configuration is applied. This means you can safely reference input variables, but not attributes exported by resources (with an exception for resource arguments that are specified directly in the configuration).

In your case, the second provider block is using unknown variables exported by a resource that has yet to be created.

@sfc-gh-asawicki
Thank you for the answer!


not working correctly when only one gpu is available

Hey,
when I try to use 3 GPUs but only 2 are available the library behaves as expected and the third process crashes.
python gpu-acquisitor.py --backend pytorch --id 1 --nb-gpus 1 & python3 gpu-acquisitor.py --backend pytorch --id 2 --nb-gpus 1 & python3 gpu-acquisitor.py --backend pytorch --id 3 --nb-gpus 1
[1] 351
[2] 352
GPUOwner3 2024-08-26 10:36:12,762 [INFO] acquiring lock
GPUOwner3 2024-08-26 10:36:12,762 [INFO] lock acquired
GPUOwner1 2024-08-26 10:36:12,767 [INFO] acquiring lock
GPUOwner2 2024-08-26 10:36:12,803 [INFO] acquiring lock
GPUOwner3 2024-08-26 10:36:12,805 [INFO] Set CUDA_VISIBLE_DEVICES=0
GPUOwner3 2024-08-26 10:36:22,043 [INFO] lock released
GPUOwner3 2024-08-26 10:36:22,043 [INFO] Allocated devices: [0]
GPUOwner1 2024-08-26 10:36:22,044 [INFO] lock acquired
GPUOwner1 2024-08-26 10:36:22,081 [INFO] Set CUDA_VISIBLE_DEVICES=1
GPUOwner3 2024-08-26 10:36:31,044 [INFO] Finished
GPUOwner1 2024-08-26 10:36:31,214 [INFO] lock released
GPUOwner2 2024-08-26 10:36:31,214 [INFO] lock acquired
GPUOwner1 2024-08-26 10:36:31,214 [INFO] Allocated devices: [1]
GPUOwner2 2024-08-26 10:36:31,252 [INFO] lock released
Traceback (most recent call last):
  File "/home/fuchsfa/foundation-models/gpu-acquisitor.py", line 77, in <module>
    safe_gpu.claim_gpus(
  File "/home/fuchsfa/foundation-models/.venv/lib/python3.12/site-packages/safe_gpu/safe_gpu.py", line 153, in claim_gpus
    gpu_owner = GPUOwner(nb_gpus, placeholder_fn, logger, debug_sleep)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/fuchsfa/foundation-models/.venv/lib/python3.12/site-packages/safe_gpu/safe_gpu.py", line 132, in __init__
    raise RuntimeError(f"Required {nb_gpus} GPUs, only found these free: {free_gpus}. Somebody didn't properly declare their resources?")
RuntimeError: Required 1 GPUs, only found these free: []. Somebody didn't properly declare their resources?
[2]+  Exit 1                  python3 gpu-acquisitor.py --backend pytorch --id 2 --nb-gpus 1

But when I try to use 2 GPUs when only 1 is available both processes get the one available GPU. Can I prevent this?
python gpu-acquisitor.py --backend pytorch --id 1 --nb-gpus 1 & python3 gpu-acquisitor.py --backend pytorch --id 2 --nb-gpus 1
[3] 6117
GPUOwner2 2024-08-26 10:35:35,666 [INFO] Running on a machine with single GPU used for actual display
GPUOwner2 2024-08-26 10:35:35,667 [INFO] Set CUDA_VISIBLE_DEVICES=0
GPUOwner1 2024-08-26 10:35:35,670 [INFO] Running on a machine with single GPU used for actual display
GPUOwner1 2024-08-26 10:35:35,670 [INFO] Set CUDA_VISIBLE_DEVICES=0
GPUOwner1 2024-08-26 10:35:44,742 [INFO] Allocated devices: [0]
GPUOwner2 2024-08-26 10:35:44,850 [INFO] Allocated devices: [0]
GPUOwner1 2024-08-26 10:35:53,743 [INFO] Finished
GPUOwner2 2024-08-26 10:35:53,850 [INFO] Finished
[2]   Done                    python gpu-acquisitor.py --backend pytorch --id 1 --nb-gpus 1
[3]+  Done                    python gpu-acquisitor.py --backend pytorch --id 1 --nb-gpus 1

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Hallo @f-fuchs,
thanks for the bringing this up! I'm now super busy with other stuff (thesis deadline in fact ;-) ), I will look into this in September, hopefully next week already. Please poke me if I don't.

Hey @ibenes,
I hope everything went well with your thesis. Have you had a chance to look into this yet?

Hi @f-fuchs!
Thank not thank you for asking ;-) It's done now, on to safe-gpu issues!
As far as I can tell from your example, your 1-GPU machine is not in an exclusive mode. This case has special handling in safe-gpu: The default for us is GPUs in our local cluster, which are in exclusive mode and need to be allocated appropriately. The other one is occasional GPUs in people PCs, which are not exclusive, running a couple of processes associated with GUI etc.  Here we don't want to check that the card is free (because it is not) and safe-gpu simply sends the current process there.
Is that roughly your situation? I know the test for the exclusivity is not perfect; if you'd like the behaviour of safe-gpu to change in your case, could you attach the output of nvidia-smi -q here?

okay, now i am confused 😕 I ran nvidia-smi -q while having one gpu and while having four but both times the  Compute Mode  was set to Default but with four gpus it is working correctly.
if this is the intended behavior that's also fine, I currently don't need need it to work with one GPU. 👌
just happy it works with multiple ones 😄

Thanks for the input, we will update the test for exclusivity; I will keep this Issue open until then :muscle:


sudo environment pass-through behavior is confusing with Go

Describe the bug
When using the setup-go action to specify an exact release of the Go toolchain, if any follow-on commands in other steps use sudo and then, within context as the root user, perform any step which relies on a Go toolchain binary, the GOROOT environment used will default back to the latest Go release installed on the ubuntu-18.04 image; today that is Go 1.14.x. This will generate errors, for example, if compiling code which tries to link in Go language standard library content, using the Go binaries in the $PATH which will generate mismatch errors given the setup-go environment of some non-Go1.14 release:
# runtime/internal/atomic
compile: version "go1.14.1" does not match go tool version "go1.13.10"

Passing GOROOT through the sudo command does not work; unlike other environment variables which do appear to pass through. For example I can add sudo GOROOT=$GOROOT _GOROOT=$GOROOT PATH=$PATH GOPATH=$GOPATH to a command and find that within the environment of the commands run that my special "underscore-prefixed" _GOROOT does pass through, and apparently all the other variables, but GOROOT itself is reset!?
Output before sudo:
GOROOT="/opt/hostedtoolcache/go/1.13.10/x64"

Output after sudo:
GOROOT="/usr/local/go1.14"
_GOROOT="/opt/hostedtoolcache/go/1.13.10/x64"

Area for Triage:
Go
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[x] Ubuntu 18.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
I expect that, as long as I pass-through the right Go environment it will be preserved in the context of the commands run with sudo. I also would have hoped sudo -E would work, but it may be turned off by the configuration of the virtual Ubuntu environment. There is no documentation on the configuration and operation of sudo at this time to validate behavior.
Actual behavior
I described the behavior of at least GOROOT being inexplicably reset during the call through sudo but other variables seem to be preserved if passed through.
I created a repo with actions that reproduces the problem.

Hi @estesp , thank you for the report of this issue! The cause of the issue is Linux security policy that overrides system $PATH to default value for sudo. As workaround you can use next step after setup-go task to create symlinks in /usr/bin directory for installed Go version:
- name: create symlink
  run: sudo ln -f -s $GOROOT/bin/* /usr/bin/

and use sudo -E for preserving user environment in following steps.

Hi @estesp , I close this issue, please, feel free reopen it if you have any concerns, thank you!

FWIW, the simpler solution here is to combine sudo -E with an explicit PATH, ala: sudo -E PATH="$PATH" ...


remove deprecated features

Coverage increased (+5.2%) to 92.513% when pulling e230a1872f1cc55754938553a035b01a9b0284dd on joostvanzwieten:drop-deprecated into a521df21266913d482bad91490f236bc99193351 on nutils:master.


[Bug]: Port not included on login

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Running locally on docker and using a different port in my docker-compose.yml:
services:
  appsmith:
    image: appsmith/appsmith-ce:latest
    container_name: appsmith
    ports:
      - "8080:80"
      - "443:443"

Everything works fine but during the login after submitting the Email/Password I get redirected to localhost/applications (w/o the port).  Either manually adding the port or going back will load the correct page at that point.
Steps To Reproduce

Docker install - use port 8080 on the host to 80 in the container (see above).
Login with credentials.
Get redirected to the page w/o the port. :(

Public Sample App
No response
Version
Self-hosted 1.7.1

Hi @disl-pdavid , We believe it is an inginx issue since 8080 is being used by the backend server process. Meanwhile until we find a solve, you can use any other port other then 8080 on the host.

Thanks @sum35h this works for me!


Automatically remove volunteering opportunities that have expired

After x amount of days strike through text, and after y amount of days remove entry.

on /volunteering it would be nice if expired opportunities had a strike through

I wrote the inital functionality d762178bd37a78873d614ad44c00842ea23a81ae


Measure usage of Unstructured in file upload feature

Summary
This PR enables:

counting API calls for Unstructured in file upload feature
sending it to telemetry backend

Related Issue

this PR closes #198

as subtask of #197

Changes

instrument to API call for Unstructured
refactor preexistent wrapper function and instrumentations

Testing

confirmed that telemetry data arrived after uploading a file ✅

Note
Although we still need to instrument for Vercel Blob to record usages along file-uploading feature, I did not include it in this PR because the amount of changes is already large at this point.

I'll merge this after resolving conflicts

I'll merge this because I confirmed that the preview env works well after rebasing 👍


[Redistributable] Experimental instance of Visual Studio doesn't find custom debug engine

After changing GUIDs as stated in Redistribution the MIDebugLaunch command no longer finds custom debug engine, instead it loads installed version of the engine. Is there a way to start my version of engine with MIDebugLaunch command?

@Dmitri-Botcharnikov if you change GUIDs you are creating a new side-by-side debug engine. So, if you still want command window integration, you would want to still include the MIDebugPackage, but you would need to rename the 'MIDebugLaunch' command to your own thing. I will add notes on this to the wiki page.

I've renamed launch command but now it seems like experimental instance of VS doesn't find my engine by GUID:
The debugger is not properly installed.  Cannot debug the requested type of code.  Run setup to install or repair the debugger.

This message is result of COM exception form call to debugger.LaunchDebugTargets4
Should I do something to register my debug engine?

The debugger engine is registered through the .pkgdef file. Make sure you have the new GUID there. You can check what is in the registry by looking at HKEY_CURRENT_USER\Software\Microsoft\VisualStudio\14.0_Config[Exp][_number]. Where the '[Exp]' suffix is used if you are using the experimental instance of VS (this will be true if you are F5'ing the package project). The '[_number]' suffix can sometimes be there if there are multiple copies of the 'config cache'. If you can't figure out which one is being used you can delete them all, do a clean build of the project, and it should come back.

@gregg-miskelly: thanks a lot! I finally found cause of error: one need to change CodeBase parameter value in the .pkgdef file to reflect renaming of .dll file.


Support of compilation optimizations when using enums (backward compatible enum access)

Is your feature request related to a problem? Please describe.
If during development a lot of work is done with enums, which have very small as well as very large values, then the compiler can perform optimizations or space saving when accessing the enumeration during the generation of the binary program.
Imagine the following example:
If a program part now accesses only the lower values, then a simple byte pointer is used in the assembler code instead of a dword pointer.
So far it is not possible to map this in Ghidra. Enums always have a predefined size of 1,2,4 or 8, so the decompiler cannot recognize accesses of an enum of 4 with a byte-pointer. This in turn leads to the fact that the resolution of the enum values sometimes works, sometimes not.
It would be desirable to be able to map at least dword enums into byte variables, so that one can read the generated code better.
Describe the solution you'd like
The simplest solution would be that if the variable is a byte, for example, and the associated enum has a length of 2, 4, or 8, that at least the values up to the length of 1 are displayed. The lower values can be represented thereby fully, and furthermore it is not to be assumed that higher values come, because that does not permit the size of the byte variable technically.
Describe alternatives you've considered
Another possible solution would be that enums can also have a variable length. For this one would have to supplement the fixed predefined lengths of 1, 2, 4 or 8 by a further option "variable" or "dynamic". The length is then specified at runtime.
Additional context

@ryanmkurtz
One of the simpler solutions would be really helpful here. I increasingly have the problem that I can track the values of enums very badly when they have been optimized. I think that would improve readability significantly if enum values were always displayed, regardless of cast or data type. The fixed sizes only make sense in structures from my point of view, while in stack variables the size shouldn't really matter.

Relates to #446, #2581, #3206

Relates to #446, #2581, #3206

And probably #4616 too!

As @0xBEEEF says, while the size of an enum only makes sense in structures (and parameters), I would like to see the "size" being specified changed to be a base type instead.  Doing this enables the same enum to be used throughout an application suit where different compilers with differently sized types (e.g. int sometimes 2 bytes, others 4 or even 8) have been used.

I would like to see the "size" being specified changed to be a base type instead. Doing this enables the same enum to be used throughout an application suit where different compilers with differently sized types (e.g. int sometimes 2 bytes, others 4 or even 8) have been used.

Taking this a little further, you could base the new enum on an existing one to add new values.  For example, this would assist when REing Windows apps that define their own messages (above 0x400) whereby each different "wndProc" could have their own message enumeration that contained the standard ones and their private ones, thus enabling better maintenance of a single copy (instance) of the standard ones.  Onwards another step, multiple inheritance could help when using 'List Boxes', 'Combo Boxes' etc.

@caheckman what do you think of the above suggestion?

I would like to see the "size" being specified changed to be a base type instead. Doing this enables the same enum to be used throughout an application suit where different compilers with differently sized types (e.g. int sometimes 2 bytes, others 4 or even 8) have been used.

Taking this a little further, you could base the new enum on an existing one to add new values. For example, this would assist when REing Windows apps that define their own messages (above 0x400) whereby each different "wndProc" could have their own message enumeration that contained the standard ones and their private ones, thus enabling better maintenance of a single copy (instance) of the standard ones. Onwards another step, multiple inheritance could help when using 'List Boxes', 'Combo Boxes' etc.

Is this feasible?  Should it be a separate Feature Request?


LaTeX Support in mobile

The Zulip app currently does not render LaTeX formatting correctly.
A simple example from a user in my realm:

Correctly rendering visual of post in desktop app:

Incorrect render on Android:

source code:

\text{DRYLAB WEEK OF 6/10/18}
```
```latex
General
```
- These updates will either come in the form of Hardware/Software/Simulations or Chromastat/Simulations or Chromastat/Game/Simulations

```latex
Chromastat
```
We will be meeting on Tuesday to talk about expectations, deadlines, and get started with work. We have gone in depth with how the device is built and programmed so far.

```latex
Simulations

What's the status on this?  I just got a report about it from a friend.

There's some work in #2690 on the underlying infrastructure in the app to enable us to support this. It's a bit tricky, and it's been stalled for a while; I expect we'll return to it after we complete some other significant upgrades like #2788 .

I expect we'll return to it

I think that day is today :) (for the PR review part)

Thx for taking up the challenge of figuring out how to do this for react native.
Nobody is able to solve this problem, within react-native (to add the capability to display latex or MathML).
I have been researching this on and off for a couple of months, and the only thing I could find is  that this need is listed in React-native feature request tracker:
https://react-native.canny.io/feature-requests/p/latex-or-mathml-rendering-in-reactnative
It seems like rendering latex/mathML from a Webview is the only viable path at the moment.

hey @vladp!
Yup, we will be rendering latex in the webview itself. In fact our whole message list is webview. :)

Bumping the priority because we just got another request for this feature -- with Tim's comment above, I guess that means at least 5 different people asking for it :-) , which is quite a lot.

After trying multiple platforms I just settled for zulip only because it supported latex.. now I discover it does not yet in mobile..  Please add this support ASAP.. any road map?

I second this request.

With #3636 landed, our app infrastructure is now in much better shape to be able to implement this feature! So I'm hoping we'll have this done sometime soon.

I've just started using Zulip mobile and LaTeX rendering half works: the maths is correctly rendered, but then there's a unicode approximation that follows the rendered maths immediately afterwards.

More voices of people eager for this feature, over on #3744:

I'd just like to comment that this is a really compelling feature for the many research groups that grumble daily about Slack lacking math and language-specific code. "That all works in Zulip" is a much stronger pitch than "it works with some clients". Thanks, @ray-kraesig, for implementing, and thanks in advance for the review, @gnprice.

and

We've been waiting for this for a long time. Looking forward to it!

I get the same problem as thosgood  on the iPad app. It is fine in the browser.

LaTeX support in the mobile app is now released! The new version is 26.24.147 if you're on iOS, and 26.25.148 on Android. (They're the same app built from the same code, just different version numbers due to a hiccup in the build.)
@Yoshi8765 @parthi2929 @berceanu @thosgood @jedbrown @pyrocto @cortner and everyone else who's been wanting this -- please update, try it out, and don't hesitate to file new issues if you find something that doesn't look right :slightly_smiling_face:


hds: validate messages on receive to catch a missing field exception

Commit Message:
Validate messages to catch missing fields in received specifiers in health discovery service.
Additional Description:
Use MessageUtil::validate over the HealthCheckSpecifier proto on message receive. HDS then instead ends its current task rather then exiting over std::terminate.
Risk Level: Low
Testing: Added an unit test test based on the example given in the related issue #11014 which calls the modified onReceiveMessage function with an invalid Health Check Specifier. Another unit test passes a valid specifier followed by an invalid specifier and ensures the previous valid configuration continues to be used.
Docs Changes: N/A
Release Notes: N/A
Fixes #11014

For those curious, the CI two failures are related to a linting error in the HDS constructor calling initialize(). This linting error was already in the repository before this change and is unrelated to the changes proposed in this PR.

@drewsortega do you think it would be super simple to fix those or more involved? If the former, might as well clean it up.
@lizan I'm seeing this kind of issue a lot these days; we have unrelated clang-tidy issues appearing in a PR where a file is modified. Is there anyway to make clang-tidy smarter about only calling out actually modified lines?

@htuch yes, the change is quite simple and I went ahead and cleaned this up. I simply expanded the initialize function into the constructor so it doesn't try to call the virtual method. I left the original initialize function intact as well.

/retest

/rerun

/retest

/retest


Inidicator bubbles for carousel

Is your feature request related to a problem? Please describe.
I'm looking for indicatior bubbles to help the users know where they are in a carousel.

For e.g. in this screenshot they are on the first page of four.
Describe the solution you'd like
Indicators below the image. Hiding them, when there is only one image. Ideally let users of the lib choose their own indicators.
Describe alternatives you've considered
No response
Additional context
Trying to switch to this library coming from react-responsive-carousel as we have been using this for the lightbox usecase for some time now.

Thank you for proposing this feature. I've considered implementing such plugin, but ultimately decided to postpone and leave it in the backlog due to limited utility of this feature. Navigation bubbles serve their purpose only when there is just a handful of slides in the lightbox, and do not work well in the case of dozens of slides. There is an option to implement Instagram-like bubbles, but then they do not provide any visual context on your current position in a carousel.
With that beings said, it's quite unlikely I'll be working on this feature in the foreseeable future. However, I'll be more than happy to accept a PR if somebody will be willing to contribute a plugin for this feature.

I guess I can come up with an external component controlling the lightbox from the outside. The only thing that doesn't seem to work (or I can't find an api for) is changing the current index, but with an animation?
As changing index only seems to be targeted at first load.

This should help - https://yet-another-react-lightbox.com/documentation#ControllerRef

This should help - yet-another-react-lightbox.com/documentation#ControllerRef

Yes, that seems to help. But animations seem problematic, as soon, as I go forward/backward more then 2 images (3 and above).
Screencast from 2023-07-16 22-45-00.webm
Seems to forget about the current image, while it's loading what to do?

At any given point in time the lightbox renders 2 * carousel.preload + 1 slides in DOM (the default value is 2 * 2 + 1 = 5). So when you navigate 3 or more slides forward/backward, the current slide immediately gets evicted from DOM.
I can suggest 2 different solutions here.

You can increase carousel.preload value to cover all slides:

carousel={{ preload: Math.max(Math.ceil((slides.length - 1) / 2), 0) }}

You can limit the number of navigation bubbles to let's 5 to match the lightbox sliding window size.


Better error message for Rack::JsonSchema::RequestValidation::Validator::LinkNotFound

Hopefully this will help someone in the future :D

aha this needs a fix, i'll work on it tomorrow :(

fixed :)

Nice improvement :+1:

woot!


Catch with ios13

flutterBeacon.initializeScanning catch and terminate with ios13. When i by pass this process app can work, But beacon.ranging don't work.
how can i fix it?
Thank you.

Hi @doodii
When you start using this library by executing flutterBeacon.initializeScanning, this will check

CBCentralManager.state, if the state is not CBManagerStatePoweredOn then this will throw an Exception
But if your bluetooth is already on, then this will request for authorization
Then if your CLLocationManage. authorizationStatus either kCLAuthorizationStatusDenied or kCLAuthorizationStatusRestricted so it will throw an exception too.

It is better to block it execution with try and catch so you can handle the exception that throw. Hope it helps.

Hi @AdnanKazi
My iPhone 6 stuck at iOS 12, can't upgrade it. Can you please debug and run it at Xcode and then please copy-paste the debug log here?

sure will post it but first i need to know which other permission do i need to add in info.plist file may be the issue is about permission

@alann-maulana I got the solution I haven't put the bluetooth permission in info.plist file
hi @doodii
just add following line in info.plist file your code will works fine.
<key>NSBluetoothPeripheralUsageDescription</key>    <string>need blutooth permission</string>    <key>NSBluetoothAlwaysUsageDescription</key>    <string>need blutooth permission.</string>

Great @AdnanKazi
Thanks for your great information. I think there are new property list regarding Bluetooth usage starting iOS 13 NSBluetoothAlwaysUsageDescription
I will add these required privacy on to README documents later.


Upgrade Client App to React 18

I would like to propose upgrading the client app of this project to React v18.
Problem
The current version of React used in the project may lack some of the latest features and optimizations introduced in the modern versions. Additionally, the application may face compatibility issues with the latest React ecosystem, libraries, and tooling.
Upgrading to modern versions will address these problems and provide developers with access to the most recent advancements in React .
Solution

Upgrade to Modern React Version (includes verification of dependencies compatibility).
Additional improvements/features:

Evaluate areas that would need new features or improvements.
Discuss and plan the addition of features that align with the goals of the Spring PetClinic project.

Potential new features

Enhanced data visualization
Improved user experience
Any functionality that can add value to the application.

Benefits

Improved development experience and productivity.
Ensuring compatibility with the evolving React ecosystem.
Potential to enhance user experience and provide additional value to the Spring PetClinic application.

References

Link to React documentation: React

I believe that upgrading and adding new features will enhance the development experience. This will enable developers to work with the latest technologies while providing additional value to the users of the Spring PetClinic application.
Please let me know if you have any questions or if there's any further information I can provide to support this enhancement proposal.

I agree that it needs an upgrade, and at this stage I haven't managed to start the application successfully (it might be due to my version of Node.js being too recent).

Like @michaelisvy , an update of the React front and Spring Boot backend would be a good thing.  Since you seem motivated and are motivated @firasrg , you have free rein and could work on it. I think @nilshartmann  will agree too.

Hi all!
I think that's a very good idea to upgrade this. Unfortunatly I hadn't found the time yet, so I'm more than happy if anyone else would support.
Tbh I think "upgrade" in this case is more or less a re-write 😬
Does it make sense, before starting with the upgrade to could discuss a tech stack that we want to use?
For example:

should we go with create-react-app or vite?
use TanStack Query for data fetching?
CSS: "plain", modules, css-in-js, Tailwind, ... ?
should we implement frontend tests? should they run in GitHub actions? 🙀

Hi all
@michaelisvy I got the client app working without backend and data, just UI.
@nilshartmann yeah it needs to be rewritten, I have a good experience with React I can provide a good initiative on that. As for techs, I imagine we would need CRA (create react app) for sure, React Query for datafetching yes! Tailwind CSS or something similar for styling and layouts.
As for tests, I think : Jest + React Testing Library
I have a good background with these tools and some others.

@firasrg not sure, if create-react-app is still a good idea. That project seems (currently) more or less abandoned while more and more people tend to use Vite. That would probably mean vitest instead of jest, but that should also work with RTL.

TBH I don't know Vite, but based on a first comparison, it seems a good tool that brings better performance over CRA. But I think for beginners and intermediate levels, CRA would be more suitable as it's mature, popular and easier to use.  As PetClinic project is intended to be used by new people or immigrants, I think it would be better to start with CRA.
It's up to you

I think it would be better to start with CRA.

I just got an information from trusted React community members that CRA is no more maintained. I agree with what you said

hello, in case the work can be split, I'll be happy to help out (either for React part or for the Backend part)

hello, in case the work can be split, I'll be happy to help out (either for React part or for the Backend part)

Hi @michaelisvy,
thank you very much!!!
I think it makes sense to consolidate the backend and frontend issue, so please find my ideas here: https://github.com/spring-petclinic/spring-petclinic-reactjs/issues/26#issuecomment-1586740581

Alright@nilshartmann , in this case I close this issue.


ADAS relevant signals are not found

Hi!
Thanks a lot for the efforts and making this public.
We are trying to get some ADAS info, specifically the following signals:
DAS_ACC_report
GPSLatitude04F
GPSLongitude04F
DAS_objectId
DAS_leadVeh2Dx
DAS_leadVeh2Dy
DAS_leadVeh2Id
DAS_leadVeh2RelevantForControl
DAS_leadVeh2Type
DAS_leadVeh2VxRel
DAS_leadVehDx
DAS_leadVehDy
DAS_leadVehId
DAS_leadVehType
DAS_leadVehVxRel
DAS_leadVehRelevantForControl
DAS_autopilotState
UI_accFollowDistanceSetting
DI_vehicleSpeed
Although we could get some signals regarding the current etc, the ADAS ones where not there.
BTW, we are doing this in Europe. Is it possible that the dbc files are a different here?
Thanks a lot!

In Europe, there is no way to get this data on pre 2021 cars because we haven't figured out a way to access the Chassis bus.


add consul module

HashiCorp Consul
https://www.consul.io/

@DDtKey should I do anything else?

No, that's alright. I changed PR name to match rules of conventional commits
Thank you! 👍


Include Enhancement request for One-To-One

Hi,
I am finding myself doing a lot of Include calls for one to one properties.
Your code suggest you pass in an <Expression<Func<T,T2>> to Include for the relationship property, but that can actually be inferred by T2. I realize that there can be more than one property of T1 that is of type T2, but am fine with it in this case.
I wrote this little helper method, but it's awkward because I have to pass in Database.  It would make more sense as part of your QueryProvider as a simplified include:
IQueryProviderWithIncludes<T> Include<T2>(JoinType joinType = JoinType.Left) where T2 : class;

static class IncludeHelp {

    public static NPoco.Linq.IQueryProviderWithIncludes<TEntity> Include<TEntity, TInclude>
        (this NPoco.Linq.IQueryProviderWithIncludes<TEntity> query
        , NPoco.IDatabase database
        , NPoco.Linq.JoinType joinType = NPoco.Linq.JoinType.Left)
        where TInclude : class
    {

        //get all the One to One References that that TEntity has.
        var oneToOneMembers = database.PocoDataFactory.ForType(typeof(TEntity))
            .Members.Where(x => x.ReferenceType == NPoco.ReferenceType.OneToOne
                                && x.MemberInfoData.MemberType == typeof(TInclude));

        //Add an include for each member of type TInclude
        foreach (var o2oMember in oneToOneMembers)
        {

            // build the automatic join property expression: Expression<Func<TEntity, TInclude>> expression
            //Define the parameter for the lambda
            ParameterExpression entityParam = Expression.Parameter(typeof(TEntity), "entity");

            //Build the lambda expression.
            var joinProperty = Expression.Lambda<Func<TEntity, TInclude>>(
                    Expression.PropertyOrField(entityParam, o2oMember.Name) // Build the field Expression //x.TInclude...
                , entityParam);
            //add Include
             query.Include<TInclude>(joinProperty, joinType);
        }
        return query;
    }
}

Sounds like a good idea, so i've created a hook to get access to the database, which will be in V4 this weekend.
    static class IncludeHelp
    {
        public static NPoco.Linq.IQueryProviderWithIncludes<TEntity> Include<TEntity, TInclude>
        (this NPoco.Linq.IQueryProviderWithIncludes<TEntity> query
            , NPoco.Linq.JoinType joinType = NPoco.Linq.JoinType.Left)
            where TInclude : class
        {

            //get all the One to One References that that TEntity has.
            var oneToOneMembers = ((INeedDatabase)query).GetDatabase().PocoDataFactory.ForType(typeof(TEntity))
                .Members.Where(x => x.ReferenceType == NPoco.ReferenceType.OneToOne
                                    && x.MemberInfoData.MemberType == typeof(TInclude));

            //Add an include for each member of type TInclude
            foreach (var o2oMember in oneToOneMembers)
            {

                // build the automatic join property expression: Expression<Func<TEntity, TInclude>> expression
                //Define the parameter for the lambda
                ParameterExpression entityParam = Expression.Parameter(typeof(TEntity), "entity");

                //Build the lambda expression.
                var joinProperty = Expression.Lambda<Func<TEntity, TInclude>>(
                    Expression.PropertyOrField(entityParam, o2oMember.Name) // Build the field Expression //x.TInclude...
                    , entityParam);
                //add Include
                query.Include<TInclude>(joinProperty, joinType);
            }
            return query;
        }
    }

Need to check this exists for v5


Updates to support for 3.7.0

Updated

LGTM assuming the failing Travis checks are a false alarm

LGTM again


chore: default to Postgres in docker compose

Description

It was strange that we had a Postgres in the docker compose file, but didn't use it when we docker compose up. So let's use it. With these changes we start a Postgres container, start OpenFGA using that container, and perform the migration.
References

Review Checklist

[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[ ] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Codecov Report

Merging #195 (3b14c41) into main (905cf9f) will increase coverage by 0.01%.
The diff coverage is 41.89%.

@@            Coverage Diff             @@
##             main     #195      +/-   ##
==========================================
+ Coverage   75.19%   75.21%   +0.01%
==========================================
  Files          79       78       -1
  Lines        8845     8811      -34
==========================================
- Hits         6651     6627      -24
+ Misses       1788     1781       -7
+ Partials      406      403       -3

Impacted Files
Coverage Δ

pkg/cmd/migrate.go
0.00% <0.00%> (ø)

pkg/cmd/run.go
0.00% <0.00%> (ø)

pkg/cmd/service/service.go
87.00% <0.00%> (ø)

server/authn/authn.go
21.42% <ø> (ø)

server/authn/oidc/oidc.go
50.90% <44.44%> (ø)

storage/postgres/postgres.go
64.54% <57.89%> (+0.10%)
:arrow_up:

server/server.go
32.39% <61.53%> (-3.44%)
:arrow_down:

server/authn/presharedkey/presharedkey.go
100.00% <100.00%> (ø)

server/errors/encoded_errors.go
94.08% <100.00%> (+2.15%)
:arrow_up:

server/commands/check.go
85.99% <0.00%> (-0.29%)
:arrow_down:

... and 2 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Create Pages “resume”

Automatically generated by Netlify CMS

👷 Deploy Preview for koytak processing.

Name
Link

🔨 Latest commit
8941951cf9b0e074f5f5ce3fcfb306dba4c2872e

🔍 Latest deploy log
https://app.netlify.com/sites/koytak/deploys/66a0e255405d2e0008adf7e0


Add unix_socket_path that exposes more Unix socket control

The existing unix_socket method has some limitations in its design:

You cannot unset a previously set path.
You cannot set non-UTF-8 paths.

This adds a second method named unix_socket_path that instead takes an Option<impl AsRef<Path>> which more closely aligns with what libcurl accepts for this option. We could consider soft-deprecating unix_socket, but I left it alone for now.

Seems reasonable to me!


List.shuffle seems to leak a midway stage

Dart Version: 2.7.0
Steps:

Have FutureBuilder to wait on a future List of items
In the snapshot.hasData if block run snapshot.data.shuffle
Pass that snapshot.data on to a new view
In the new view displaying the first item in the List.
Change state not related to the List.
Which causes a rebuild again displaying the first item in the list -- but it's a different item than in 4

Expected Behavior:
Step 4 and 6 the first element of the list should be the same.
I also commented out the snapshot.data.shuffle line to isolate the problem. When the line is commented out 4 and 6 display the same data.

Thanks for the report! Could you post a snippet of source code showing the issue? That would help us to triage; thanks.

Yep, I made a demo flutter app that exhibits the bug. Check it out the relevant bit at https://github.com/joyfulmantis/dartbugdemo/blob/master/lib/main.dart I am also attaching images of the demonstration below

I'm guessing this is not a plain Dart list. Are you shuffling an ObservableList? If so, it's an issue with that, not the default List.shuffle implementation.


complete PSCIS submission for Phase 2 sites

has not been started so can be done with BCEID/NewGraphEnvironment  (ltd company iD vs sole proprietorship)

Data has been submitted.

Phase 2 data has been successfully submitted

Habitat confirmation data is ready for review (need Al's review) and submission


Publishing docker images for Datafusion Ray on tagging

To make Datafusion Ray available to a larger audience not interested in building from source the Docker image, we would like to publish on ghcr.io docker images.

We are now successfully publishing thanks to #32 see https://github.com/apache/datafusion-ray/pkgs/container/datafusion-ray


Importing scalaparse.Scala._ breaks compilation

After the line import scalaparse.Scala._ is executed in the interpreter, all other lines fail to compile like this:
@ import scalaparse.Scala._
import scalaparse.Scala._
@ val a = 10
Compilation Failed
Main.scala:65: macro is now a reserved word; usage as an identifier is disallowed
  macro,
  ^

Fixed in 8b78621a00edba5eb87696ddad0cd9582d8c3123


Explorer does not work with Hyperledger Fabric v3.0.0 and FabricGateway.ts file should be corrected/updated

What happened?
There is a problem about using Hyperledger Fabric v3.0.0 with Explorer. It does not show chaincodes and transactions. It looks like https://github.com/hyperledger-labs/blockchain-explorer/blob/main/app/platform/fabric/gateway/FabricGateway.ts file should be updated to be compatible with HLFv3.0.0. When I check the logs on the explorer container with docker logs CONTAINER command, it shows the error:
[2024-11-21T12:02:03.592] [ERROR] Sync - <<<<<<<<<<<<<<<<<<<<<<<<<< Synchronizer Error >>>>>>>>>>>>>>>>>>>>>

[2024-11-21T12:02:03.592] [ERROR] Sync - Error: error in simulation: failed to execute transaction d00a5575f86467db94ce1ba5e0dec11f0e837fd23b41c0ccc526128ee7c0a93e: could not launch chaincode lscc.syscc: error building chaincode: error building image: failed to get chaincode package for external build: could not get legacy chaincode package 'lscc.syscc': open /var/hyperledger/production/chaincodes/lscc.syscc: no such file or directory

The start point of the error is, I think, async queryInstantiatedChaincodes(channelName) function. There is a problem with the lifecycle of chaincode.
What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?
async queryInstantiatedChaincodes(channelName) function in FabricGateway.ts file should be updated.
Anything else we need to know?
No response
OS version

# On Linux:
$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.5 LTS"

NAME="Ubuntu"

VERSION_ID="22.04"

VERSION="22.04.5 LTS (Jammy Jellyfish)"

VERSION_CODENAME=jammy

ID=ubuntu

ID_LIKE=debian

HOME_URL="https://www.ubuntu.com/"

SUPPORT_URL="https://help.ubuntu.com/"

BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"

PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"

UBUNTU_CODENAME=jammy
$ uname -a
Linux umit 5.15.0-124-generic #134-Ubuntu SMP Fri Sep 27 20:20:17 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

I had to downgrade the peer and orderers from v2.5.10 to v.2.5.9 for the explorer to work. Once peers and orderers were downgraded I just restarted the explorer and it worked.


Celery worker status

Hi,
I run 3 workers (each using its own queue) with different number of worker processes. When I try to "ping" them by doing:
./manage.py celeryctl inspect ping
I get pong only from those workers that have one or more idling worker processes. Workers with all busy  processes do not replay at all. Same thing then i try to get status information via python using "inspect" from "celery.task.control".
Not sure if it's a bug or a feature, but how do I reliably get the worker status ?
(Broker: RabbitMQ 2.7.1, Celery: 2.4.6)
Thanks,
lauris

On Feb 22, 2012, at 5:39 PM, PonasNiekas<EMAIL_ADDRESS>wrote:

Hi,
I run 3 workers (each using its own queue) with different number of worker processes. When I try to "ping" them by doing:
./manage.py celeryctl inspect ping
I get pong only from those workers that have one or more idling worker processes. Workers with all busy  processes do not replay at all. Same thing then i try to get status information via python using "inspect" from "celery.task.control".
Not sure if it's a bug or a feature, but how do I reliably get the worker status ?
(Broker: RabbitMQ 2.7.1, Celery: 2.4.6)
Thanks,
lauris

Reply to this email directly or view it on GitHub:
https://github.com/ask/celery/issues/619

Have you disabled rate limits? Issues related to this have been fixed in the development version (2.5 will be released this Friday)

I had CELERY_DISABLE_RATE_LIMITS set to True yesterday. Today I set it to False, also set CELERYD_PREFETCH_MULTIPLIER to 4 (from 1) and it works just fine at the moment.
Looking forward to 2.5 .


add write permissions for pdfs to video action

Allow the docker image to write out pdfs - failing at present.

yeah hadn't had time to get back on to debugging this, cheers. If it was really just this missing, I owe ya a beer.


Fixing / Removing new vulnerability found 11th April in spring-web 6.1.5

New vulnerability found in spring-web 6.1.5
https://spring.io/security/cve-2024-22262
Manually overriding spring web version to new 6.1.6 where vulnerability doesn't exist
Original bug:
bug: modify CHANGED_RESOURCE_URI to include private
JU-247

CI: No security warnings found


Implement FilterDsl for BoxedSelectStatement

We had to switch the type of the where clause to an option, and handle
whether or not to put WHERE in the query explicitly to handle the
AND case when filter is called twice. I had tried to instead box up
WhereAnd, but ultimately if you try and write
impl WhereAnd<Box<QueryFragment<DB>>>, the output type ends up being
infinitely recursive, as the output type would be a Box<WhereAnd>, and
would need the output type specified, etc.

Review? @mfpiccolo @mcasper

Looks good!


permit headers directly before eof

previously something like
[foo]

would fail to parse, due to there not being a newline after the header.
I haven't tested this with zig nightly yet, but it should just work.

Implementation looks good and the tests pass on my machine. Please could you make the PR only include the header change rather than that and your escape code one please?
You could cherry-pick 3f0686f onto a new branch or do an interactive rebase and delete the older commit.

sorry this took so long, done now


Corrected number of digits for Australian 1300 and 1800 numbers.

Apologies, my initial PR incorrectly expected 8 digits after the 1300 and 1800 numbers, when it should have been 6.

Thanks a lot! :tada: Happy holidays :)
Released in 2.15.11.


FR: ParseEnv support for sensitive API/sign keys

I'm currently referencing the Imgix API and sign key with getenv() in config/imager.php, however this isn't ideal and not recommended for the best security practices.
https://docs.craftcms.com/v3/extend/environmental-settings.html
It would be nice if Imager would support being able specify these values through the special Craft environment syntax of $SECRET_ENV_VAR.
Something like:
'imgixApiKey' => '$IMGIX_API_KEY',
 'imgixConfig' => [
   'default' => [
       'domains' => ['domain.imgix.net'],
       'useHttps' => true,
	'signKey' => '$IMGIX_SIGN_TOKEN',
        'sourceIsWebProxy' => false,
        'useCloudSourcePath' => true,
	'shardStrategy' => 'cycle',
	 getExternalImageDimensions' => true,
         ....
    ]
]

Where did you read that the use of getenv() is not recommended? The use of the special $<SECRET_ENV_VAR> is necessary if a plugin has a control panel settings interface, to make sure that those values aren't parsed and stored in the DB or project config. But Imager doesn't have control panel settings.
The only thing that happens in the Craft::parseEnv method, is that the $ is matched, and getenv() is run on the variable name. The only difference I can think of is if the config files are actually compiled at some point and stored in a compiled state somewhere - but I find that highly unlikely.
Then again, I've been known to be wrong, so let me know where you got the recommendation from. :)

Thanks for your response! I think I might have misinterpreted some guidelines somewhere! I think, because with Craft 3.1 and the project.yaml, using getenv() can expose secret keys and such in plain text unless using the $ENV_VAR syntax but as far as I know Imager doesn't write any such info like this. If it did in the future then maybe it might be a problem. That's I think where I'm getting a bit paranoid.
Environment variables while are generally secure, there are some concepts where super sensitive keys are stored in something like secrets.json, which can be passed to Craft as environment variables, but their storage in production is in an encrypted file, rather than plain text.
However, if getenv() is generally fine for settings not stored in the DB, then that's fine with me.

Yeah, Imager doesn't store any settings in the DB or project config, so it doesn't matter.
I'll probably start using Craft::parseEnv at some point, just for consistency since I already parse alias' for some settings. But since it requires 3.1, I want to hold off as long as possible so I don't need to bump the requirements.


npm run server:dev:hmr error

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Report a bug

What is the current behavior?
When executing npm run server:dev:hmr an error is shown in web console, and application is not shown

Uncaught Error: [HMR] Hot Module Replacement is disabled.

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

What is the expected behavior?
Run with HMR without errors in JS console

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 5
Browser: all

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

in package.json do you have --hotOnly or --hot, it must be --hot to replace  --hotOnly

@almothafar That fixed it for me. Suggest that change should be committed to the repo.

I'm trying to use HMR the gdi2290/angular-hmr or the angular-cli way. None of them work, the page simply refreshes. Is HMR working with gdi2290/angular-starter or not? There's any doc about how to use it?

+1 @adrhc state used to be kept across reloads, but that is lost, any guidance on how to maintain the state would be good


Use CircularBuffer in EmbeddedChannel.

Motivation:
When running load through EmbeddedChannel we spend an enormous amount of
time screwing around with removing things from Arrays. Arrays are not a
natural data type for removeFirst(), and in fact that method is
linear-time on Array due to the need for Array to be zero-indexed. Let's
stop using (and indeed misusing) Array on EmbeddedChannel.
While we're here, if we add some judicious @inlinable annotations we can
also save additional work generating results that users don't need.
Modifications:

Replace arrays with circular buffers (including marked versions).
Avoid CoWs and extra allocations on flush.
Make some API methods inlinable to make them cheaper.

Result:

Much cheaper EmbeddedChannel for benchmark purposes.

Actually in most of these cases the allocation counts are too low. Empty arrays don't allocate, but all CircularBuffers do, so we traded better runtime performance for more allocations on EmbeddedChannel construction. I think this is a good trade, but it's a real tradeoff.


add GTEx widget pages to all genes for which GTEx has information

right now the GTEx widget page is only on ~10 genes; we should expand this.
we need to get a list of genes from GTEx.

I created a list of genes from GTEx_Analysis_2017-06-05_v8_RNASeQCv1.1.9_gene_reads.gct.gz from https://gtexportal.org/home/datasets and used it to annotate 19352 gene pages. See recent commits in #60


Add changelog generation

Checkout https://github.com/camunda-community-hub/camunda-helm/tree/main/.chglog

Alternative we could generate it via github api https://docs.github.com/en/rest/reference/repos#generate-release-notes-content-for-a-release and use the content

https://github.com/helm/chart-releaser supports now a changelog file

We use the artifact.io changelog, which is added to the chart.yaml file and manually add the changelog via the github web ui. it is enough for now


Pass context to ArgumentType.parse

This Pull Request implements the suggestion outlined in #76
It has been implemented to be non-breaking unless an implemented argument type can return a nullable value. Since the CommandContext is not yet available at this point in time, we can pass the command source, from which the required context can usually be extracted.

this is incredibly important to merge!!! well implemented, would need that😂

i don't quite get if this is sarcasm or actual comment of approval?

this is incredibly important to merge!!! well implemented, would need that😂

i don't quite get if this is sarcasm or actual comment of approval?

I am not joking i REALLY NEED THIS NOW PLS

you can just clobe PR repo and build it from source, doesn't take all that
long
On Tue, Sep 21, 2021, 21:48 Paul Dietrich @.***> wrote:

this is incredibly important to merge!!! well implemented, would need that
😂
i don't quite get if this is sarcasm or actual comment of approval?
I am not joking i REALLY NEED THIS NOW PLS
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Mojang/brigadier/pull/77#issuecomment-924279690, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALVAGAIZQWYHQIHY4RFDMD3UDDHQFANCNFSM4SRYS5CQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I won't be continuing work on this feature, as there's a lot of complications mentioned in the issue linked in the first post.


Add source created for VSCODE

Reasons for making this change:
TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

Thanks, but see Global\VisualStudioCode.gitignore


Tentative first pass at making simulcast egest possible

https://github.com/webrtc-rs/webrtc/issues/289#issuecomment-1271677760
Forgot to make a PR and stick my long-winded text there, but whatever, is is the draft PR :)
(I'll fix the tests momentarily, forgot I'd changed a structural thing for the API!)

@robashton This looks great! I'm going to look closer next week. I would advise that you review the changes that undo work from https://github.com/webrtc-rs/webrtc/pull/217 however. These fixed bugs and made us more spec compliant and shouldn't definitely remain(might need changing though)

Ah, if I have a source as to "where it came from" then I can do my best to restore it

ah, yes - now I see where it was being set. By the time I realised that code wasn't working any more I'd lost that line.
Fine, that's easy to restore, will sort that along with the pesky Mutex that shouldn't be a Mutex.

Great, I think it was only #217 that introduced these changes, but you if you're in doubt it should be clear from git blame. Any part of JSEP that talk about msid and the parts of the spec that discuss RTCRtpSender.[[AssociatedMediaStreamIds]] are relevant to read to make sense of this

That should be a bit closer to the finish line - I think I got the placement right for the msid / simulcast directives in the SDP

I think it might be better to leverage an enum for track_encodings, i.e.
enum TrackEncodings {
  Simulcast(Vec<TrackEncoding>),
  Single(TrackEncoding)
}

I'm not against this, let's get ourselves happy with the rest of the minor bits and I'll have a look at it.

Just tidying up this thread so I can see what is outstanding, quite keen to get this pushed over the finish line so I can move onto the next task in my queue (not webrtc related).
The SRTP Reader stuff seems the most pressing, I've re-looked at this in the whole and it looks as though it's not 'wrong' per se, we've got independent streams on a shared transport and you want separate binds/interceptors for each of those (or that shared state will get hairy quickly).
The comment about the SRTP Stream being shared I don't think is correct, as we create the SRTP stream per encoding at the top of the function
        let ssrc = rand::random::<u32>();
        let srtp_stream = Arc::new(SrtpWriterFuture {
            closed: AtomicBool::new(false),
            ssrc,
            rtp_sender: Arc::downgrade(&self.internal),
            rtp_transport: Arc::clone(&self.transport),
            rtcp_read_stream: Mutex::new(None),
            rtp_write_session: Mutex::new(None),
        });

        let srtp_rtcp_reader = Arc::clone(&srtp_stream) as Arc<dyn RTCPReader + Send + Sync>;
        let rtcp_reader = self.interceptor.bind_rtcp_reader(srtp_rtcp_reader).await;

Calling close on each of these in sequence is fine (I think). This is spiritually quite close to what Pion is doing with its stack (although I appreciate a lot of that has changd and it may well have fixes in to deal with bugs that might arise with this decisio that I haven't implemented here)

The comment about the SRTP Stream being shared I don't think is correct, as we create the SRTP stream per encoding at the top of the function

Aight, I misread.
What about RTCP?

Urgh, I didn't mean to push that rebase, I was just looking at what had changed upstream and forgot I'd pulled it when I came back from coffee. What a mess.

The comment about the SRTP Stream being shared I don't think is correct, as we create the SRTP stream per encoding at the top of the function

Aight, I misread.
What about RTCP?

I've re-read the code around this, and as far as I can see, the infrastructure-per-encoding model works because each encoding has its own SSRC and that is how the SrtpWriterFuture discriminates when it sets itself up. (The SrtpWriterfuture being where the RTCP reader comes from) and that being how read_simulcast then works.
It's a bit of a janky model, but it's the one we've got - I don't know how you'd avoid setting this up per-encoding without changing a heap of other code?

Could you try and clean up the git strangeness and I can do a final pass

If I can work out how, I totally messed this one up, usually I'd just do a git fetch and a rebase on final pass (and from the other side). That I did a pull to check what had changed upstream and then accidentally pushed it is a total cluster.
I imagine my best bet might be to do an interactive rebase locally and re-apply my changes without any of that noise in it.


[WIP] Add link to finder from each section

Needs to be rebased against #558
Trello: https://trello.com/c/b9mzRtCa
Add a link to filterable list page from each supergroup section.
Escapes the query string in case passing / as part of the taxon base_path causes any issues.

Replaced by #566


State Express Route requirements for Hosting/Enterprise Architecture team

Dependancy level is higher than expected - we will need other teams to move to be able to deploy express route.


fix:removed white spaces

Fixes Issue

Changes proposed

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.

Screenshots

Note to reviewers

just removed white spaces as this is my first pull request i did a small contribution

when will my pull request will be accepted?

Congratulations on your contribution :) 🎉  You did a great job. Wait for a few hours and @eddiejaoude will accept this PR.


Image feature extraction could be cached efficiently

In a given Python session, we are now potentially using the same image feature extraction several times (either for the same purpose with different parameters, or for multiple purposes, in a development loop). For instance, one might annotate a dataset (with several calls to annotate, each of which will extract features), then train an image classifier, then evaluate. Each step does its own feature extraction and nothing is shared.
Because the extracted features are small, we could keep a fairly small session LRU cache (say 16 or 64 MB) that could give a performance boost to these types of in-session reuse of extracted features.

I like this idea, but I think we'll have to store more than just the deep features in the cache, since we'll have to be able to map images to deep features.
We could probably do something smarter than storing the images themselves in the cache. I believe tc.Image() type is immutable, so perhaps we could just store the memory address of the images. Or we could do something like use storing the md5sum of the images, but then it would be possible to incorrectly map images to deep features.

We could use a fast hash of the image, though I agree we'd have to disambiguate or risk (with low probability) wrong results. Memory address could also work.


feature request: expose options for managing lifecycle of AsyncIOSystems

this could be by making AsyncIOSystem.beacon_location public so users can manually manage the entity manually or adding such a method to AsyncWorld
i also noticed this comment
// TODO(Bevy 0.13):
// The AsyncIO and AsyncIOBeacon structs are hacks to enable sending IO to/from a system
// until Bevy 0.13 lands with https://github.com/bevyengine/bevy/pull/10380.
// When that happens, this file should be reverted to how it was in 886204a201eb94e54d85c8fb88d5cc722042d244.
// (and probably rewritten to match the refactor that got rid of operations)

so maybe this will be covered there
otherwise, i believe the beacon entity leaks?

Yes, the concept of these AsyncIO and the corresponding beacons will be entirely removed once 0.13 hits.  I plan to update as soon as the new version is released.

To clarify, there will be no entities created or associated with an AsyncIOSystem.  It will behave very similarly to the current AsyncSystem

just wanted to follow up on this, so I understand that there are no entities from this library that will leak now but it looks like the bevy internal entity still leaks if not explicitly .remove_system-ed https://github.com/bevyengine/bevy/blob/2aed777435d26c357ed71cdb8c7b858de35e582e/crates/bevy_ecs/src/system/system_registry.rs#L146-L163
so i think we still need AsyncIOSystem.id to be pub so we can do async_world().apply(|world| world.remove_system(system.id))?


Windows on RKE 2 MVP Checklist for 2.6

Confirm working before GA:
custom provisioning
v1 and v2 apps working
...other use cases to validate?

Part of https://github.com/rancher/windows/issues/4


docs(navigation): what's the id of the root nav?

Ionic version:  (check one with "x")
[X ] 3.x
I'm submitting a ...  (check one with "x")
[X ] support request (linked with https://forum.ionicframework.com/t/getrootnav-deprecated-use-getrootnavbyid-whats-the-value-of-the-root-nav-id)
Current behavior:
Following the introduction of Ionic 3.5.0, getRootNav was marked as deprecated. I would like to use then getRootNavById but I've got no idea what's the id of the root nav or how I could set my own id.
(getRootNav) is deprecated and will be removed in the next major release. Use getRootNavById instead.

Expected behavior:
Document what's the root nav Id or how to set the root nav Id
Steps to reproduce:

Update to ionic-angular >= 3.5.0
Use getRootNav

Other information:
global packages:
@ionic/cli-utils : 1.5.0
Cordova CLI      : 7.0.1
Gulp CLI         : CLI version 1.2.2 Local version 3.9.1
Ionic CLI        : 3.5.0

local packages:
@ionic/app-scripts              : 2.0.2
@ionic/cli-plugin-cordova       : 1.4.1
@ionic/cli-plugin-gulp          : 1.0.2
@ionic/cli-plugin-ionic-angular : 1.3.2
Cordova Platforms               : android 6.2.3 ios 4.4.0
Ionic Framework                 : ionic-angular 3.5.2

System:
Node       : v8.1.3
OS         : macOS Sierra
Xcode      : Xcode 8.3.3 Build version 8E3004b
ios-deploy : 1.9.0
ios-sim    : 5.0.13
npm        : 5.2.0

Same problem here. Near identical sistem configuration.
I'm using this function to force tabs to disappear when loggin out.

This issue was moved to ionic-team/ionic-site#1197


Free channel ID after being closed

The channel ID resource is never released when a channel is closed. Fix this bug by subscribing to its closeFuture.

thanks :)


OSGI Buncle packaging

Imported from Google Code issue #205 created by reda.bendiar on 2012-10-17T17:03:31.000Z:

Is it possible to have the java version packaged as osgi bundle ?

I don't know whether a CLA was signed for that patch, or whether it's enough for the process to be automatic along with our regular releases.
I'll close this issue as there hasn't been interest or contributions since we moved to GitHub.
Please re-open if you can help!

I'm looking to utilize this library in an OSGi environment, so I resurrected the patch and made a couple minor adjustments


Typescript compiler errors on React Native

Hello.
I keep getting following errors from TypeScript compilation (using react-fetchiung-library on React Native):
node_modules/react-fetching-library/lib/index.d.ts:15:19 - error TS2304: Cannot find name 'RequestCredentials'.

15     credentials?: RequestCredentials;
                     ~~~~~~~~~~~~~~~~~~

node_modules/react-fetching-library/lib/index.d.ts:16:13 - error TS2304: Cannot find name 'RequestCache'.

16     cache?: RequestCache;
               ~~~~~~~~~~~~

node_modules/react-fetching-library/lib/index.d.ts:17:12 - error TS2304: Cannot find name 'RequestMode'.

17     mode?: RequestMode;
              ~~~~~~~~~~~

node_modules/react-fetching-library/lib/index.d.ts:18:22 - error TS2304: Cannot find name 'ReferrerPolicy'.

18     referrerPolicy?: ReferrerPolicy;
                        ~~~~~~~~~~~~~~

node_modules/react-fetching-library/lib/index.d.ts:22:16 - error TS2304: Cannot find name 'RequestRedirect'.

22     redirect?: RequestRedirect;
                  ~

Is there any types package that I need to install in order to get these typings on React Native?
I've tried @types/node, @types/request and @types/node-fetch but without any success.
Thank you so much!

It looks like I will need to define it in library, I will try to fix it later today.
For now you can add .d.ts file in your project with
type RequestCredentials = "include" | "omit" | "same-origin";
type RequestCache = "default" | "force-cache" | "no-cache" | "no-store" | "only-if-cached" | "reload";
type RequestMode = "cors" | "navigate" | "no-cors" | "same-origin";
type ReferrerPolicy = "" | "no-referrer" | "no-referrer-when-downgrade" | "origin" | "origin-when-cross-origin" | "same-origin" | "strict-origin" | "strict-origin-when-cross-origin" | "unsafe-url";
type RequestRedirect = "error" | "follow" | "manual";


CoolerControl v1.2.2: daemon does not detect the nvidia proprietary drivers.

Describe the bug
CoolerControl does not detect Nvidia GPU when enabled via options
Steps To Reproduce
Steps to reproduce the behavior:

Nvidia GPU with proprietary drivers (version 555, but might not matter)
programs.coolercontrol.enable = true;
programs.coolercontrol.nvidiaSupport = true;
journalctl -u coolercontrold spits out error about missing nvidia-smi and nvidia-settings

Expected behavior
CoolerControl to detect Nvidia GPU
Screenshots

Additional context
Logging output:
May 22 21:27:04 cortex coolercontrold[143375]: Logging Level: INFO
May 22 21:27:04 cortex coolercontrold[143375]: Initializing...
May 22 21:27:04 cortex coolercontrold[143375]: Configuration file check successful
May 22 21:27:07 cortex coolercontrold[143375]: Communication established with Liqctld.
May 22 21:27:07 cortex coolercontrold[143375]: Initialized Liquidctl Devices: ["Corsair Lighting Node Pro"]
May 22 21:27:07 cortex coolercontrold[143375]: Initialized CPU Devices: ["Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz"]
May 22 21:27:07 cortex coolercontrold[143375]: Error communicating with nvidia-smi. If you have a Nvidia card with proprietary drivers, nvidia-smi and nvidia-settings are required.

If I manually stop the systemd process and start sudo coolercontrold it detects the requested binaries correctly, which makes me think that the coolercontrold binary just doesn't search the correct paths.
Notify maintainers
@codifryed @OPNA2608
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.32, NixOS, 24.11 (Vicuña), 24.11.20240526.e2dd4e1`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(vilsol): `""`
 - channels(root): `"nixos-23.11, nixos-unstable, nixpkgs-unstable"`
 - nixpkgs: `/nix/store/62ca8xfz3n92p57sdds15r4s07kyls6i-source`

Add a :+1: reaction to issues you find important.

the coolercontrold binary just doesn't search the correct paths

The tools should be on PATH if you check the generated service file.
    # Nvidia support
    (lib.mkIf cfg.nvidiaSupport {
      systemd.services.coolercontrold.path = with config.boot.kernelPackages; [
        nvidia_x11 # nvidia-smi
        nvidia_x11.settings # nvidia-settings
      ];
    })

And restarting the daemon would support that I think, since it shouldn't pick up any new paths from the env AFAIU.

Error communicating with nvidia-smi. If you have a Nvidia card with proprietary drivers, nvidia-smi and nvidia-settings are required.
No GPU devices detected

I'm pretty sure the second part of that message comes from nvidia-smi itself.
https://gitlab.com/coolercontrol/coolercontrol/-/blob/029cbe9a8a8bb26c0b8d4a9404a871e60a800e67/coolercontrold/src/repositories/gpu_repo.rs#L189-196
https://gitlab.com/coolercontrol/coolercontrol/-/blob/029cbe9a8a8bb26c0b8d4a9404a871e60a800e67/coolercontrold/src/repositories/utils.rs#L70-126
So nvidia-smi might not like whatever state your system is in whenever coolercontrold first launches?
I sadly don't have any nvidia system to test this with… I think if you try something like this you could delay the service startup to check if the above guess is correct?
systemd.services.coolercontrold.serviceConfig.ExecStartPre = "${lib.getExe' pkgs.coreutils "sleep"} 10";

(Or override the package with an overlay and patch it into the service that way if this doesn't work the way I think it does)

Restarting the service does not work, as that throws out the same identical error.
After adding that to my config, rebuilding and restarting the service, I get this output:
● coolercontrold.service - CoolerControl Daemon
     Loaded: loaded (/etc/systemd/system/coolercontrold.service; enabled; preset: enabled)
    Drop-In: /nix/store/j0sjhvl62mhl4q2kaw445pzn4kj4f669-system-units/coolercontrold.service.d
             └─overrides.conf
     Active: active (running) since Wed 2024-05-29 15:55:06 EEST; 51s ago
    Process: 246351 ExecStartPre=/nix/store/php4qidg2bxzmm79vpri025bqi0fa889-coreutils-9.5/bin/sleep 10 (code=exited, status=0/SUCCESS)
   Main PID: 246614 (coolercontrold)
         IP: 0B in, 0B out
         IO: 0B read, 8.0K written
      Tasks: 18 (limit: 75809)
     Memory: 23.6M (peak: 25.0M)
        CPU: 348ms
     CGroup: /system.slice/coolercontrold.service
             └─246614 /nix/store/racw52cqg7llndjqgndj3mr14nd7njav-coolercontrold-1.2.2/bin/coolercontrold

May 29 15:55:18 cortex coolercontrold[246614]: Initialized CPU Devices: ["Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz"]
May 29 15:55:18 cortex coolercontrold[246614]: Error communicating with nvidia-smi. If you have a Nvidia card with proprietary drivers, nvidia-smi and nvidia-settings are required.
May 29 15:55:18 cortex coolercontrold[246614]: No GPU Devices detected
May 29 15:55:18 cortex coolercontrold[246614]: Initialized GPU Devices: []
May 29 15:55:18 cortex coolercontrold[246614]: Error reading temperature value from: "/sys/class/hwmon/hwmon5"/temp1_input - No data available (os error 61)
May 29 15:55:18 cortex coolercontrold[246614]: Initialized Hwmon Devices: ["nvme1", "acpitz", "nvme0"]
May 29 15:55:18 cortex coolercontrold[246614]: Initialized Custom Sensors: []
May 29 15:55:18 cortex coolercontrold[246614]: Applying all saved device settings
May 29 15:55:18 cortex coolercontrold[246614]: API bound to IPv4 and IPv6 addresses: <IP_ADDRESS>:11987, [::1]:11987
May 29 15:55:18 cortex coolercontrold[246614]: Daemon successfully initialized

which looks like sleep succeeded, but not sure how that would help so I did this instead:
systemd.services.coolercontrold.serviceConfig.ExecStartPre = "${lib.getExe' config.boot.kernelPackages.nvidia_x11_beta "nvidia-smi"} -L";

Which I either did wrong, but it seems to print the GPU fine?
$ journalctl _SYSTEMD_INVOCATION_ID=`systemctl show -p InvocationID --value coolercontrold.service`
May 29 16:04:22 cortex nvidia-smi[340429]: GPU 0: NVIDIA GeForce RTX 2080 Ti (UUID: GPU-0bb60bf8-aed8-cafd-fbf1-bf60e4024959)
May 29 16:04:22 cortex coolercontrold[340431]: Logging Level: INFO
May 29 16:04:22 cortex coolercontrold[340431]: Initializing...
May 29 16:04:22 cortex coolercontrold[340431]: Configuration file check successful
May 29 16:04:33 cortex coolercontrold[340431]: Communication established with Liqctld.
May 29 16:04:34 cortex coolercontrold[340431]: Initialized Liquidctl Devices: ["Corsair Lighting Node Pro"]
May 29 16:04:34 cortex coolercontrold[340431]: Initialized CPU Devices: ["Intel(R) Core(TM) i7-8700K CPU @ 3.70GHz"]
May 29 16:04:34 cortex coolercontrold[340431]: Error communicating with nvidia-smi. If you have a Nvidia card with proprietary drivers, nvidia-smi and nvidia-settings are required.
May 29 16:04:34 cortex coolercontrold[340431]: No GPU Devices detected
May 29 16:04:34 cortex coolercontrold[340431]: Initialized GPU Devices: []
May 29 16:04:34 cortex coolercontrold[340431]: Error reading temperature value from: "/sys/class/hwmon/hwmon5"/temp1_input - No data available (os error 61)
May 29 16:04:34 cortex coolercontrold[340431]: Initialized Hwmon Devices: ["nvme0", "acpitz", "nvme1"]
May 29 16:04:34 cortex coolercontrold[340431]: Initialized Custom Sensors: []
May 29 16:04:34 cortex coolercontrold[340431]: Applying all saved device settings
May 29 16:04:34 cortex coolercontrold[340431]: API bound to IPv4 and IPv6 addresses: <IP_ADDRESS>:11987, [::1]:11987
May 29 16:04:34 cortex coolercontrold[340431]: Daemon successfully initialized

But that made me realize one difference. I am on beta drivers, but as per your code block, you pull in the stable ones... Could that be the distinguishing factor? In which case, maybe it might be better if it pulled in the field hardware.nvidia.package rather than hardcode to stable drivers?

If I manually stop the systemd process and start sudo coolercontrold it detects the requested binaries correctly

Oh sorry, I thought that read "restart coolercontrold".

I am on beta drivers, but as per your code block, you pull in the stable ones... Could that be the distinguishing factor?

Maybe.

In which case, maybe it might be better if it pulled in the field hardware.nvidia.package rather than hardcode to stable drivers?

Likely.