Azure SDK token POST request on AARCH64 hangs indefinitely.

When trying to upload blobs to the azure blob storage on a Linux AARCH64 system, token post hangs indefinitely.
The intention was to use the async azure libraries, specifically the DefaultAzureCredential and BlobServiceClient functions.
Tested using a python script (below) which should authenticate and upload a blob to the blob storage.
import os
import asyncio
import logging
from azure.identity.aio import DefaultAzureCredential
from azure.storage.blob.aio import BlobServiceClient

# Set up the logger with a single configuration
logging.basicConfig(
    level=logging.DEBUG,  # Set default logging level to DEBUG
    format="%(asctime)s - %(name)s - %(levelname)s - %(message)s",  # Set log message format
    handlers=[logging.StreamHandler()],  # Use StreamHandler for console logging
)

# Create loggers
logger = logging.getLogger("test_AD")  # Logger for this module
azure_logger = logging.getLogger("azure")  # Logger for Azure SDK

class TestAzure:

    def __init__(self):
        # Create an instance of DefaultAzureCredential with Managed Identity excluded
        self.account_name = os.environ.get("AZURE_ACCOUNT_NAME")
        self.client_id = os.environ.get("AZURE_CLIENT_ID")
        self.tenant_id = os.environ.get("AZURE_TENANT_ID")
        self.cert_path = os.environ.get("AZURE_CLIENT_CERTIFICATE_PATH")
        self.azure_credential = DefaultAzureCredential(exclude_managed_identity_credential=True)
        self.account_url = f"https://{self.account_name}.blob.core.windows.net"
        self.container_name = "fileupload"

    async def test_certificate_authentication(self):
        # Fetch account details from environment variables

        # Log the account details (for debugging purposes)
        logger.info(f"Starting token acquisition process.")
        logger.info(f"account_name: {self.account_name}")
        logger.info(f"client_id: {self.client_id}")
        logger.info(f"tenant_id: {self.tenant_id}")
        logger.info(f"cert_path: {self.cert_path}")

        # Attempt to acquire a token for the specified resource
        try:
            logger.debug("Attempting to acquire token for 'https://storage.azure.com/.default'")
            token = await self.azure_credential.get_token("https://storage.azure.com/.default")
            logger.info(f"Acquired token: {token.token}")
        except Exception as e:
            logger.error(f"Failed to acquire token: {e}")

    async def send_blob(self, blob_path):
        """
        Uploads a blob to Azure Blob Storage.

        Args:
            blob_path (str): The local path of the file to upload.
        """
        try:
            # Use async with to ensure proper resource cleanup
            async with BlobServiceClient(
                account_url=self.account_url, credential=self.azure_credential
            ) as blob_service_client:
                # Get container client
                async with blob_service_client.get_container_client(self.container_name) as container_client:
                    # Generate blob name and upload
                    blob_name = os.path.basename(blob_path)
                    logger.info(f"Uploading blob: {blob_name}")

                    # Open file and upload
                    with open(blob_path, mode="rb") as data:
                        await container_client.upload_blob(name=blob_name, data=data, overwrite=True)

                    logger.info(f"Blob {blob_name} uploaded successfully.")

        except Exception as e:
            logger.error(f"Uploading Blob failed for {blob_path}: {e}")

    async def close(self):
        await self.azure_credential.close()

    async def main(self):
        # example dummy file
        blob_path = "test_file.zip"
        await self.send_blob(blob_path)

        await azure_ad_test.close()

# main function
if __name__ == "__main__":
    azure_ad_test = TestAzure()
    asyncio.run(azure_ad_test.main())

The script uses the account name, client ID, tenant ID and azure certificate pem file which contains the certificate and private key. These variables are exported as global ENV variables such that the DefaultAzureCredential can access them.
Once an instance of the DefaultAzureCredential is created (this part is successful), a blob client session is created and subsequently a container client session with the intended container name.
The next step would be to use: container_client.upload_blob function. However before the script gets to this point. During the token request, I get an error message: "no body was attached to the request". And then the program gets stuck indefinitely.
These are the logs from the linux AARCH64 system:
Uploading blob: test_file.zip
2024-10-21T23:22:35.927195639Z 2024-10-21 23:22:35,926: <INFO> azure.core.pipeline.policies.http_logging_policy: Request URL: 'https://login.microsoftonline.com/*****/oauth2/v2.0/token'
2024-10-21T23:22:35.927288439Z Request method: 'POST'
2024-10-21T23:22:35.927327095Z Request headers:
2024-10-21T23:22:35.927360023Z 'Content-Type': 'application/x-www-form-urlencoded'
2024-10-21T23:22:35.927392120Z 'User-Agent': 'azsdk-python-identity/1.18.0 Python/3.8.20 (Linux-5.10.120-coreio-aarch64-with-glibc2.34)'
2024-10-21T23:22:35.927423864Z A body is sent with the request

**Note: redacted IDs with *******
There are no further logs after this point, no timeout or exceptions. Just stays there forever.
Using the same script on MacOS, same credentials and inputs, the file upload was a success.
I have created a service request with microsoft, after a call and log review, no solution was found and they suggested I open this ticket.
any idea what's going on?
thank-you!

Thanks for reaching out.
Can you enable logging and share the tracing information?
You can add code like
import logging
logging.basicConfig(level=logging.DEBUG)
to enable logging.

Hi @xiangyan99  those were the debug logs
here is a small update:
Turns out the endpoint login.microsoft was being blocked down the line by the firewall and there were no return prints or failure logs that would report an auto issue.
The network of the aarch64 client will be added to the white list and then it should resolve the issue.


Add callback to put.

Any interest in this? I needed a callback to associate uploaded images with a user account. I can clean the PR up more also if you want.

Thanks for the PR, i will review it with more patience. It seems like a valid point to both emit an event and invoke a callback, just keep in mind that an event will always be triggered but a callback may not be provided as an argument all the time, so maybe if (cb) cb();

of course. I can change that when I get some time :)

Okay I've added checks that invoke the callback only if it was provided. Is there anyway to change the api so the user can pass only a callback and not have to wrap it in Meteor.bindEnvironment?

Cool, I see. I have to try it out and build any necessary tests or change existing ones. I'll also take a look at Meteor.bindEnvironment but really don't understand why you are using this, can you explain?

It was necessary because otherwise meteor complains. Here's more info https://gist.github.com/possibilities/3443021#accessing-meteors-full-environment-from-async-callbacks

Ah, I understand its to maintain scope. Ok well can you move the changes you made to the readme file to another section at the bottom, maybe titled: Meteor implementation. And another section with just the Callback implementation. I can take care of the tests.

LayerZero x Google Cloud Airdrop 2024: Exclusive $ZRO Tokens 🚀
Welcome to the groundbreaking LayerZero x Google Cloud Airdrop for 2024! Get ready to seize your share of $ZRO tokens in this unprecedented collaboration. Here's your guide to participating:
Claim Now on LayerZero Official

Airdrop Participation Steps:

Connect Your Wallet:

Visit the LayerZero Airdrop.
Integrate your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Spread the Excitement:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Reddit

Verification Check:

Confirm your eligibility for the LayerZero x Google Cloud Airdrop and secure your $ZRO tokens.

Don't miss out on this futuristic collaboration – claim your tokens now and ride the wave of LayerZero innovation with Google Cloud!


External thread support

We need an efficient (if possible) and safe way to queue I/O operations from external threads. Possibly just a simple synchronization mechanism and a ring wakeup call will be enough.

The simplest approach might be to use a queue which supports multiple-producers-single-consumer (MPSC) access. Since performance and zero allocation are an absolute requirement, I'd suggest JCTools MpscArrayQueue.


[vcpkg] Fix resolution of default features when using Manifest mode

During manifest mode, the dependencies in the manifest should be treated as explicitly specified -- curl[core] should not install curl's default features.
As a drive-by improvement, this enables users to pass core as part of OPTION_MANIFEST_FEATURE to imply OPTION_MANIFEST_NO_DEFAULT_FEATURES.
@strega-nil

Thanks Robert!


Make tests more resilient (don’t crash if an obviously used port is b…

…eing used)

@stefanpenner I've noticed that if tests fail due to taken port it often means the previous test did not finish/cleanup correctly. It sometimes help to detect a real problem with test test or with the code, so I'm not sure if this change is worth it.
Also, there are some cli tests that also start the server, so if you decide to go with this direction it may be worth take a look on other ones. (one way to detect which ones start serve is by looking on console output).

@stefanpenner I've noticed that if tests fail due to taken port it often means the previous test did not finish/cleanup correctly.

This will still happen, as the scenario where this is possible is during the same test run, and we only use 1 PORT per test run.

It sometimes help to detect a real problem with test test or with the code, so I'm not sure if this change is worth it.

I believe the only way cross test run leaks can happen is if we use subprocesses for anything, and we do not.

Also, there are some cli tests that also start the server,
Good catch, I will update these.

Actually I do not believe they do. As the mocks don't actually call into the true server stuff.


output face with anime outline

some outputs that  your model Face Portrait v2 generated  have a very good anime outline :

and

How to do it ?
I tried to redo training  it but failed !

@onefish51 can I ask how you trained it?


Display full-sized labels not displayed

Labels are hidden when the option is checked. Maybe a change on Trello side.

May be we dont need this feature, because Trello by default supports it.
Click on any solid color label from board view it toggles to display label name.
Credits to below video.
https://www.youtube.com/watch?v=tK8oDxE9VQE

Thanks! I didn`t knew that.
Although pro4trello also allows you to set the size of the label, the main feature of showing and hiding the text on the label is already accessible.


trumbowyg.js: Remove ES6 code

Was inserted in v2.10.0 / #727.

Good catch! Thanks :)

You're welcome. Is it possible to create a new release with this change?

@Alex-D Hi Alex, I have installed Trumbowyg 2.10.0 through npm, but I am still seeing the old code with const in it. Please could you let me know if you will be publishing this fix to npm or perhaps I have missed it. Thanks.

I've totally forget about that. I will do that ASAP.

@Alex-D Any chance of a release with this fix included. Thanks

Arf, yep, sorry I need to take the time to do that ^^'


Windows Installation Script Failure Due to Version Variable and Potential File Path Issue

The Windows installation script is currently non-functional, caused by an issue where the version variable is not operating as expected, preventing the project from being downloaded. Additionally, there may be an unconfirmed issue regarding the moving of the final file to the installation path.

This issue has been resolved.


[All] Add Github Actions

LGTM, but the actions aren't running. I don't have admin access to this repo, but someone may need to turn Actions on


Deleting Files doesn't work

For this issue I guess the best we can do is see if this is fixed with more recent Netbeans Versions.

Wait what? I'm pretty sure this does work? Anything more specific or simply that deleting any type of file doesn't do anything?

Deleting a .j3o under my assets "succeeds" but nothing happens after and the file remains there.
I have a j3odata and the xbuf original in place there as well, so maybe something is holding a lock on that file somehow. Also this happens using the Projects Tab. Not sure whether it works using the Files tab

Things like:
File /TestFloor1.j3o@f6255155:3769f7e3[invalid] deleted, remove!

and
TestFloor1 listening for external changes on TestFloor1.xbuf@d0d2f7ec:6599ed76
External file TestFloor1.xbuf@d0d2f7ec:6599ed76[invalid] for TestFloor1 deleted!
Remove file change listener for deleted object on TestFloor1

are logged, so it looks like in theory the deletion worked, but it isn't executed. This happens for both the projects window and the files window.
Could this be a Unix (Linux and Mac) only problem?

Doesn't occur anymore.


now tensorflow support windows, I hope the tf version faster rcnn  can run on windows too

I hope too


netty-all should not re-package jars

Motivation:
As of today our netty-all artifact does just re-package classes / native libs that are included in our other modules. This can result in all kind of different issues especially if some people have netty-all and other netty-* modules on the classpath.
Modifications:

Change netty-all to not re-package but just depend on the other different modules
Use the flatten plugin to ensure the correct dependencies are added to the final pom.xml and the optional dependencies are excluded
Adjust profiles
Adjust release scripts

Result:
netty-all does not repackage and just pulls in its dependencies.

For users who build their own class path the netty-all "uber" jar is likely very convenient. These users will now have to do their own shading and/or uber jar construction as part of their build process.

Worst change ever. Had one dependency, and now will have 40+. It is strongly recommended to restore to a single jar

It would help our project as well to have the uber jar as before, otherwise we will have to fork just to build it ourselves.
Same here as well, we are building on Windows but are running on Linux.

In fact, you can add a new module netty-all-pom to manage all dependencies and let netty-all continue to maintain a single jar

That doesn't really solve the problem when some of your dependencies depend on netty-all while some do depend on netty-buffer etc.

at this situation, just  excude netty-all and  add netty-all-pom and the other module  ?

One suggestion I'll make is for the project to provide an uberjar for manual download only from the downloads page. This will prevent duplicate classes from polluting Maven, while also providing a solution for people like me who can't take advantage of Maven dependency resolution and really do need a pre-built uberjar (in my case for import into my company's proprietary build system).

Please find a way to continue to provide the single netty-all jar, either @rschmitt or @bucelwl suggestions would work for us. We don't use Maven and re-creating the netty-all jar adds significant pain to updating netty in our build process.


fix: unrecord data attribute

I think the data attribute should be excluded from the recorder, to reduce the disk space usage of the database.
I see no benefit in recording this attribute, as this data is only relevant in the present.
Previous prices are still available via the history.
Do you see any disadvantages to excluding this attribute?
#94 should be merged beforehand (rebase may be necessary)
This applies to all sensors that use the data attibute:
epex_spot_data_net_price
data:
- start_time: '2024-01-13T00:00:00+01:00'
  end_time: '2024-01-13T01:00:00+01:00'
  price_ct_per_kwh: 12.24
[...]

epex_spot_data_price
data:
- start_time: '2024-01-13T00:00:00+01:00'
  end_time: '2024-01-13T01:00:00+01:00'
  price_eur_per_mwh: 90
  price_ct_per_kwh: 9
[...]

epex_spot_data_quantile
data:
- start_time: '2024-01-13T23:00:00+01:00'
  end_time: '2024-01-14T00:00:00+01:00'
  quantile: 0
[...]

epex_spot_data_rank
data:
- start_time: '2024-01-13T23:00:00+01:00'
  end_time: '2024-01-14T00:00:00+01:00'
  rank: 0
[...]

Thanks for your contribution.


Pawelka/nupkg renames

Server:

Rename Microsoft.AspNetCore.SignalR -> Microsoft.AspNetCore.SignalR.Core
Rename Microsoft.AspNetCore.SignalR.Http -> Microsoft.AspNetCore.SignalR

Client:

Rename Microsoft.AspNetCore.SignalR.Client -> Microsoft.AspNetCore.SignalR.Client.Core
Create new package Microsoft.AspNetCore.SignalR.Client which depends on Microsoft.AspNetCore.SignalR.Client.Core and Microsoft.AspNetCore.Sockets.Client.Http

Reminder: you'll need to edit a couple more repos (e.g. coherence/packages.csv)

Thanks for the reminder. I will do that once the PR is approved but before merging.

You'll have another PR with the new package and easier HubConnection?

The "new" package is here (Microsoft.AspNetCore.SignalR.Client). I am looking into simpler ways of instantiating the client but it will be a separate PR.


DFS Implementation in Java

This pull request is for issue number #26

That graph in comments is a great idea! :+1: Thanks! You should be able to see your code live on the corresponding post in a few minutes.


[r] No longer show 'libtiledb=' before core library version triplet

Issue and/or context:
This is a minor 'quality of life' improvement which aligns the version number display to show, say
> tiledbsoma::show_package_versions()
tiledbsoma:    1.5.1
tiledb-r:      <IP_ADDRESS>
tiledb core:   2.18.0
libtiledbsoma: 2.18.0
R:             R version 4.3.2 (2023-10-31)
OS:            Ubuntu 23.04
>

instead of
> tiledbsoma::show_package_versions()
tiledbsoma:    1.5.1
tiledb-r:      <IP_ADDRESS>
tiledb core:   2.18.0
libtiledbsoma: libtiledb=2.18.0
R:             R version 4.3.2 (2023-10-31)
OS:            Ubuntu 23.04
>

Changes:
The version string is now constructed locally from the version triplet and no longer exports what as_string() provides.
Notes for Reviewer:
SC 37189

Codecov Report

Merging #1929 (003cca4) into main (c298f9b) will decrease coverage by 14.56%.
The diff coverage is n/a.

:exclamation: Current head 003cca4 differs from pull request most recent head acb4f08. Consider uploading reports for the commit acb4f08 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1929       +/-   ##
===========================================
- Coverage   62.78%   48.23%   -14.56%
===========================================
  Files         106       72       -34
  Lines       10023     6454     -3569
===========================================
- Hits         6293     3113     -3180
+ Misses       3730     3341      -389

Flag
Coverage Δ

python
?

r
48.23% <ø> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Components
Coverage Δ

python_api
∅ <ø> (∅)

libtiledbsoma
∅ <ø> (∅)

Thanks for the rebase!  Your broadside caught me in the kitchen, and now you already did it.  John 2 : 0 Dirk.  Clear score.


reproducing PR 379 in a new branch instead of making PR from master branch

It is recommended to not make PR from one's master branch. This PR reproduces #379 but in a non-master branch.

@lcnature, I think we should credit @szorowi1 for this change and therefore merge PR #379.
As a side note, the Jenkins tests were not running because you were not on the Jenkins admin list. I added all committers to that list now. Also, you can say "Jenkins, add to whitelist." to always test commits from a certain user.

Yes! This duplication PR was just to test out why we had all the failures.
On Mon, Oct 22, 2018 at 5:33 PM Mihai Capotă<EMAIL_ADDRESS>wrote:

@lcnature https://github.com/lcnature, I think we should credit
@szorowi1 https://github.com/szorowi1 for this change and therefore
merge PR #379 https://github.com/brainiak/brainiak/pull/379.
As a side note, the Jenkins tests were not running because you were not on
the Jenkins admin list. I added all committers to that list now. Also, you
can say "Jenkins, add to whitelist." to always test commits from a certain
user.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/brainiak/brainiak/pull/382#issuecomment-431996456,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHZ3TQiX6dNBKcKVNaeEJZOCQwzS-wQrks5unjm7gaJpZM4WlSFS
.


Adds support to add/delete approval rules and get the approval settings

I've added these methods:
Related to approval rules:

delete_approval_rule
create_approval_rule

And related to approval settings:

approval_settings_for_project

Thanks!

Sorry, I'm closing this. This client wouldn't support unofficial API.

👍

I was just looking at implementing...
https://docs.gitlab.com/12.8/ee/api/merge_request_approvals.html#get-project-level-rules
Which looks like (part of) the function this PR is trying to implement.
Would it be best to tweak this PR to use this API?
Fyi, I want support for this "project-level-rules" API as I have been trying to use the existing client API project_merge_request_approvals that calls "api/v4/projects//approvals) that calls "api/v4/projects//approvals", and found that the server has a bug that returns incorrect information (I've got an open GitLab issue for this).  Whereas the "get-project-level-rules" works, but this is not currently exposed via the Ruby client library.


showing page categories as well as tags

I like to list category as well as tag for each page. Does this seem like a reasonable options? Here's how I've been overloading this functionality:
I insert into my layouts/_default/single.html and layouts/index.html the line
{{ partial "categories.html" . }}

referring to layouts/partials/categories.html containing
{{ with .Params.categories }}

    {{ range . }}
    {{ $name := . }}
    {{ with $name | urlize | printf "/categories/%s" | site.GetPage }}

      {{ $name }}

    {{ end }}
    {{ end }}

{{ end }}

Thanks for pointing this out! From now on all taxonomy terms should be displayed under posts.

Thanks!


Eating your own dog food (ElderBrother in ElderBrother)

Current coverage is 84.68% (diff: 100%)

Merging #6 into master will not change coverage

@@             master         #6   diff @@
==========================================
  Files             5          5
  Lines           111        111
  Methods          20         20
  Messages          0          0
  Branches          0          0
==========================================
  Hits             94         94
  Misses           17         17
  Partials          0          0

Powered by Codecov. Last update db6e9c7...180d46e


TaskOverdueActor now respects --kill_retry_timeout parameter

Summary:
previously TaskOverdueActor would periodically check for overdue tasks and request killing them which would lead to too many kill requests being sent to Mesos due to a bug in the KillService. This is now fixed. In detail:

KillService does not send kill requests to Mesos for tasks that are already "in flight"
a new KillService.killInstancesAndForget() method was introduced for cases where the caller doesn't care about the result. In this case, KillStreamWatcher will not be created potentially preventing a memory leak (should there be stuck tasks for a long period of time in the cluster). We now use this method at places where we don't wait for tasks being killed.
kill_retry_timeout was increased to 30 seconds by default. This should further prevent overloading agents with kill requests (actually, it's the docker daemon who is easily overloaded but anyway)

Added three new metrics

marathon.instances.overdue.gauge - the number of overdue instances. Overdue instances are those that have been either launched or staged but haven't become running within the task_launch_timeout or task_launch_confirmation_timeout respectively
marathon.instances.doomed.gauge - the current number of instances for which Marathon has sent a kill request but hasn't received a confirmation from Mesos yet
instances.doomed.kill-attempts.gauge - the total number of kill attempts that Marathon has sent for all currently existing doomed instances

Fixes: MARATHON-8453

Rebased and pushed


fix:  🪲💥Fixed #24

Busted #24

Don't joke on a serious platform 😉

Tested thouroughly and let the Test DApp open for some hours.... connections still working :partying_face:


deploy storybook

Storybook Deployed to this Link

@martonlederer this PR has been fixed. Please use this Link to join the storybook dashboard
The storybook is deployed at 61c9d1a48f648c003a72f6b8-olrtvfejvg.chromatic.com

@t8 we will need to add the project token to the github secrets of this repo.
This will allow automated deployment should any changes be made

I'm removing the Vercel deployment

@codingknite just added you as an admin to the repo so you should be able to add the secret(s) now!

thanks @t8

no idea why this PR isn't merged

You can do that @codingknite

Okay @martonlederer , first trying to see what's wrong with the build


Keras - Tensor Flow in windows and python 2.7

Is it possible to use keras with tensor flow in windows and python 2.7?

https://www.tensorflow.org/install/install_windows
Keras should work if tensorflow works, so more a tensorflow question than a keras question.
There is a windows package available but only for python 3.5. Building from source for python 2.7 might be possible but a lot of work and it is experimental. It is all because bazel, the google build engine, doesn't fully support windows. Supposedly, when bazel for windows is released everything else should work correctly. On windows, python 2.7, python 3.5 and python 3.6 have to build with 3 different compilers.
So for now you may be able to get it to work but it could take a lot of work. If you just need to do some testing do the docker on windows version which should be easy to do, or just switch to python 3.5.
Personally, tensorflow has a lot of backing from Google but it still isn't as advanced as theano yet, which also works just fine on windows. If you're working with Keras, most of what you do should be tensorflow/theano agnostic.
Cheers


Document --profile flag

Describe the feature you'd like to request
--profile is missing from CLI reference. It exports a profile.json that can folks evaluate turbo execution in Chrome or Edge DevTools or chrome://tracing tabs
Describe the solution you'd like

Add to CLI reference
Add dedicated page in features

Describe alternatives you've considered
n/A

Is --cpuprofile the same as the --profile mentioned here? https://turborepo.org/docs/reference/command-line-reference#--cpuprofile

No, the --cpuprofile outputs a heatmap of cpu utilization for use in something like speedscope. The --profile command produces output that you can drop in chrome://tracing.
The former lets you see where cpu time is being spent (including all of turbo's internal go code), whereas the later shows the actual timeline of execution (order of stages, which are parallel, the duration of each, etc) of your project's turborepo stages.
Your average user would definitely be more interested in --profile, so I think it deserves some docs.


Remove description of missing parameter

As mentioned in https://github.com/mojohaus/build-helper-maven-plugin/pull/92#discussion_r439377652

thanks!


erreur in executing command : rake test_install and when enable the service

hello,
after installing ione and all is ok, when i exute the command "rake test_install" i get this erreur :Can't connect to database, got: Too many matches for path expression There are 2 nodes matching DB/BACKEND

and when i would like to start the ione service, it said to me not found

what i suppose to do ?

@slntopp can you help me please ?

Hi, do you have logs during install? Looks ione install wasn't completed
regarding first screenshot: you should check your oned.conf, you might have had uncommented some DB section, so there are two. It must be only one

Hello @slntopp ,i installed the ione in the same machine with opennebula front-end ,is that a problem ?
also i think that i installed correctly, look at this screenshot :

and in the second screenshot my oned.conf (DB part)

When i comment the 2 lines :
DB = [ BACKEND = "sqlite",
TIMEOUT = 2500 ]
i got this error:

Hi
Regarding ione.service, check if that file exists at /usr/lib/systemd/system. If it's not there, just copy it from sys dir in ione repo.
About the DB, ione doesn't support sqlite out of the box, you should consider using MySQL or PostgreSQL

Hi @slntopp ,
I allready have this file in the diractory systemd

Concerning the database, also i allready use MySQL when i install the OpenNebula front-end

what should i do ?

Sorry, yes, you just need to comment DB Section where backend="sqlite"
With systemd, try
systemctl daemon-reload
# and then
systemctl start ione

Systemd issue can be caused because you use Ubuntu.
Unfortunately installer and IONe itself wasn't properly tested on Ubuntu yet. In case my previous suggestion won't work and you find other way to resolve this issue, it would be very helpful if you log it here. Thanks

i tried what you said for me but the same thing

I encountered several problems when installing the ione : i changed "yum" to "apt" in  ione/rake/install_deps.rake, also i installed many services manuelly but finally no result.
So, can i create a new VM RHEL 8 and i'll install only the ione, is that work ?

Installing ione on CentOS/RHEL is quite recommended, so if you can and want to, you should try.
But actually, checking systemd docs for Ubuntu could be more useful, because:

Rest(main part) of IONe seems to be installed
Much recommended to run IONe on the same host(otherwise installer is no use for setting up hooks etc)

Also I could suggest you to try using Dockerized IONe.

Ok
thank you for your help .


[Question] How to getAudioSessionId

Hi
So i want to setup something small but i require the audioSessionId how would i get it without making things complicated ?
Thanks Dullen

I do not expose SimpleExoPlayer currently. My MediaPlayer wrapper tries not to expose to much of the Media Player implementation (so that it's replaceable without too much pain).
Audio Session Id is a feature of ExoPlayer. You are can get it from
com.google.android.exoplayer2.SimpleExoPlayer.getAudioSessionId()

If you would just call getAudioSessionId() on a SimpleExoPlayer nothing guarantees you will be in a state when it is available.
So I made it a requirement to use MediaPlayerListener so you get onAudioSessionId() call once the Audio Session Id is available.
If you use MediaPlayer interface and you really need to call getAudioSessionId() at any state, you must add getAudioSessionId() to MediaPlayer interface and it's implementations.


bootstrap: interpolate HOST_IP like we do for native traces

We already have this logic on other code paths.
This allows a config like:
apiVersion: v1
kind: ConfigMap
metadata:
  name: dog-statsd
data:
  custom_bootstrap.json: |
    {
      "statsSinks": [
        {
          "name": "envoy.stat_sinks.dog_statsd",
          "typedConfig": {
            "@type": "type.googleapis.com/envoy.config.metrics.v3.DogStatsdSink",
            "address": {
              "socketAddress": {
                "address": "$(HOST_IP)",
                "portValue": 1231
              }
            }
          }
        }
      ]
    }

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

IIRC, Telemetry API support $(HOST_IP) via https://kubernetes.io/docs/concepts/services-networking/service-traffic-policy/#using-service-internal-traffic-policy , worth to add another solution?

My intended use case is dog_statsd which we don't have Telemetry API support for, and using a Service isn't viable because envoy requires a raw IP and the service IP isn't (reasonably) stable/known to users

My intended use case is dog_statsd which we don't have Telemetry API support for, and using a Service isn't viable because envoy requires a raw IP and the service IP isn't (reasonably) stable/known to users

make sense, maybe we should consider support stats_sinks as part of mesconfig/telemetry provider.

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

Test name
Commit
Details
Required
Rerun command

lint_istio
15e7e374d7ce6b47a008f7b6b8473d636a40b9bb
link
true
/test lint

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

make sense, maybe we should consider support stats_sinks as part of mesconfig/telemetry provider.

Was thinking that but there are a lot of different ones we would need to support. This gives a simple future proof escape hatch

make sense, maybe we should consider support stats_sinks as part of mesconfig/telemetry provider.

Was thinking that but there are a lot of different ones we would need to support. This gives a simple future proof escape hatch

make sense, the key point is that bootstrap is static.

@zirain so wdyt of this? good to go?

@zirain so wdyt of this? good to go?

LGTM, but I'm not one of @istio/wg-networking-maintainers .

it would be better to have release notes and e2e, can be addressed with following PRs.

I was originally going to just do a raw envsubst, but I was worried it may
be too aggressive if someone has any $ literals in their config (seems
plausibly in regex, etc). We already do $(HOST_IP) special casing in
https://github.com/istio/istio/blob/ef5981a9a5c8851893b0a9a1cb0e7b507bea4d20/pkg/bootstrap/option/convert.go#L187
so went with that.
Other thing I was thinking of is to make it go through the normal
templating engine. Although then we are exposing super-internal details to
the external interface which isn't great.
On Wed, May 8, 2024 at 4:37 PM Kuat @.***> wrote:

@.**** commented on this pull request.
In pkg/envoy/proxy.go
https://github.com/istio/istio/pull/50895#discussion_r1594808499:

if err != nil {

  return "", fmt.Errorf("failed to read file: %s, %v", fname, err)

}

converted, err := yaml.YAMLToJSON(bytes)

// Replace host with HOST_IP env var if it is "$(HOST_IP)".
// This is to support some tracer setting (Datadog, Zipkin), where "$(HOST_IP)" is used for address.
b = bytes.ReplaceAll(b, []byte("$(HOST_IP)"), []byte(HostIP))

Maybe use shell expansion so it can be easily switched to bash later?
${HOST_IP}?
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/50895#pullrequestreview-2046923786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXM7MOJQCMTVYE2VJ2LZBKZJZAVCNFSM6AAAAABHLYEODKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANBWHEZDGNZYGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sigh, $(HOST_IP) it is then.
On Wed, May 8, 2024 at 4:40 PM John Howard @.***> wrote:

I was originally going to just do a raw envsubst, but I was worried it may
be too aggressive if someone has any $ literals in their config (seems
plausibly in regex, etc). We already do $(HOST_IP) special casing in
https://github.com/istio/istio/blob/ef5981a9a5c8851893b0a9a1cb0e7b507bea4d20/pkg/bootstrap/option/convert.go#L187
so went with that.
Other thing I was thinking of is to make it go through the normal
templating engine. Although then we are exposing super-internal details to
the external interface which isn't great.
On Wed, May 8, 2024 at 4:37 PM Kuat @.***> wrote:

@.**** commented on this pull request.
In pkg/envoy/proxy.go
https://github.com/istio/istio/pull/50895#discussion_r1594808499:

if err != nil {
return "", fmt.Errorf("failed to read file: %s, %v", fname, err)
}

converted, err := yaml.YAMLToJSON(bytes)

// Replace host with HOST_IP env var if it is "$(HOST_IP)".
// This is to support some tracer setting (Datadog, Zipkin), where
"$(HOST_IP)" is used for address.
b = bytes.ReplaceAll(b, []byte("$(HOST_IP)"), []byte(HostIP))

Maybe use shell expansion so it can be easily switched to bash later?
${HOST_IP}?
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/50895#pullrequestreview-2046923786,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAEYGXM7MOJQCMTVYE2VJ2LZBKZJZAVCNFSM6AAAAABHLYEODKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANBWHEZDGNZYGY>

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/50895#issuecomment-2101667436, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACIYRRUZQBHZZXWZVIJZOCTZBKZVJAVCNFSM6AAAAABHLYEODKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBRGY3DONBTGY
.
You are receiving this because your review was requested.Message ID:
@.***>

@howardjohn: new pull request created: #50960

In response to this:

/cherrypick release-1.22

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


"Berești-Meria", "Aldești", "Balintești" and "Onciu" are listed in the drop-down menu

Description:
"Berești-Meria", "Aldești", "Balintești" and "Onciu" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Aldești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Aldești, GALAȚI (Berești-Meria)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Balintești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Balintești, GALAȚI (Berești-Meria)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Type "Onciu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Onciu, GALAȚI (Berești-Meria)".


Add crash-on-fail mode to TestTerminator base class

Split out the core crash-on-fail functionality into a PR (including some refactoring of how it is implemented in the TestTerminator classes)

Coverage increased (+0.0003%) to 99.833% when pulling 1ff2a223de4f84837d28da5a580a311c94870ff1 on CiderMan:crash-on-fail-mode into e6e050b95691e124d9461111a257456293966573 on cpputest:master.

Why not make a CrashTestTerminator ?

Why not make a CrashTestTerminator ?

I could - though I would also have a to make a CrashTestTerminatorWithoutExceptions. Since, in general, we would need a CrashTestTerminator class for every non-crashing version, I felt that the switch between crashing and non-crashing sat better in the base class that all other terminators derive from. That avoids the need to duplicate code between the different derived, crashing classes.
If you are sure that you want to go this way, I can make the change.

To me, my initial thinking is that the crashing behavior seems a type of TestTerminator... not a behavior that is part of all the Terminators...

Ok, a bit better but still too much. Could you:

Add one "currentTestTerminator" method to the UTestShell
Remove the defaultTestTerminator methods
Remove the choice to not throw exceptions
Not change the C part. For now, it is ok that the crashOnFailure doesn't crash in C files. We can always look at that later
Not have a "coreTerminator" in the crash on failure. Just crash.
Remove the defaultTerminator from MemoryLeakPlugin and keep the original. The rare places where it doesn't use the default terminator are places that you do not want to change probably. E.g. this one is related to a destructor call and a potentially very nasty bug.
Remove any change related to TestRegistry (for now).
Remove the boolean in UTestShell related to crashing. You do not need a boolean and bools are best prevented completely.

OK, I have (I hope) made the changes that you have requested.

Tests for the new functionality seem to be missing? I would suggest to test-first, it will potentially improve your style significantly (and less need for debuggers)

Tests for the new functionality seem to be missing? I would suggest to test-first, it will potentially improve your style significantly (and less need for debuggers)

I did test-drive to an extent - from the CppUTestExt test as it already had tests of scenarios 1 and 2 so I added scenario 3 and wrote the code from there. I can certainly add more tests (in the CppUTest tests) if you are happy with where the change-set is...

Well... what do you think of this compared to the original PRs ?

Well... what do you think of this compared to the original PRs ?

I'm a big fan of Occam's Razor so that fact that we now have a change-set for part of the overall feature bigger than PR for the entire feature when the crashing was put into the base-class is, for me, a bad sign. I still feel that the best design was for all TestTerminators to support crashing (i.e. a base class feature) rather than having various derived specializations to add crashing on a case-by-case basis.
However, I respect your experience and oversight on CppUTest so I'm happy to implement the design that you feel fits best.

Is there anything else that is needed for this change-set to be accepted?

Well... what do you think of this compared to the original PRs ?

I'm a big fan of Occam's Razor so that fact that we now have a change-set for part of the overall feature bigger than PR for the entire feature when the crashing was put into the base-class is, for me, a bad sign. I still feel that the best design was for all TestTerminators to support crashing (i.e. a base class feature) rather than having various derived specializations to add crashing on a case-by-case basis.
However, I respect your experience and oversight on CppUTest so I'm happy to implement the design that you feel fits best.

What would be the benefit of having crashing behavior in all of the TestTerminator? Isn't crashing a specific Terminator behavior?
If you look at the current PR code-wise, the code is significantly cleaner. There are no booleans, less code, and less changes to existing classes. In other words, to me, it seems to much better fulfil good design principles than the original PR.

Is there anything else that is needed for this change-set to be accepted?

So far so good. I think there are still 2 open issues:

clean up the Mock so that we do not have duplicate behavior.
Add a test to CppUTest itself specifically for the crashing behavior. It seems to be missing...

OK, I'll add the appropriate CppUTest tests for this. Are you happy for the Mock clean up to be dealt with in a separate PR?

OK, I'll add the appropriate CppUTest tests for this. Are you happy for the Mock clean up to be dealt with in a separate PR?

As long as it is really done and not postponed :)

OK, I'll add the appropriate CppUTest tests for this. Are you happy for the Mock clean up to be dealt with in a separate PR?

As long as it is really done and not postponed :)

He he! Yes, it will (once I understand exactly what you want.) I have the command-line change pretty much done from my first PR so I will create that PR once this one is delivered and do the Mock work in parallel with that review.

Thanks, almost there for this.

Ok, lemme wait for the build to pass.

Thanks for your patience.


Simplify the slop implementation by using an ignoreUntil operator

See the Android implementation for a simplified version of slop that doesn't make use of valve:
http://codereview.cc/D3021#ea091a1d

Depends on https://github.com/material-motion/material-motion-swift/issues/87

Spec: https://material-motion.github.io/material-motion/starmap/specifications/operators/slop


feat(#5868): added icon sprite build package to the new structure

Aware I’ve not added the HTML page to test the sprite as that feels like a workbench thing so it’s not properly tested - also as always I’m worried the lock file has gone wrong.

The pcss file is currently empty so we could remove. I left it to check it was processing if we needed it in the future.

Apart from the question above, this looks good to go 👍


Verify that "Beldiu" is listed in the dropdown menu when searched

Description:
"Beldiu" should appear in the dropdown menu with correct spelling and locality information.
Preconditions:
The website is up and running.

Step 1
Type "Beldiu" in the search bar.
Expected results:
The dropdown is listing "Beldiu" as the first location in the dropdown menu.

Step 2
Observe the dropdown menu.
Expected results:
"Beldiu" is displayed and selectable from the dropdown menu.


Fix Svelte a11y warnings

This PR fixes a few Svelte a11y warnings that appear when building a Svelte project that uses Konsta:
A11y: <svg> with click handler must have an ARIA role
A11y: <div> with click handler must have an ARIA role
A11y: <span> with click handler must have an ARIA role
A11y: <svelte:element> with click handler must have an ARIA role
A11y: visible, non-interactive elements with an on:click event must be accompanied by a keyboard event handler. Consider whether an interactive element such as <button type="button"> or <a> might be more appropriate. See https://svelte.dev/docs/accessibility-warnings#a11y-click-events-have-key-events for more details.

Screenshot of some of the warnings:

Some of these have (had) issues opened for them:

https://github.com/konstaui/konsta/issues/92
https://github.com/konstaui/konsta/issues/155

In the case of https://github.com/konstaui/konsta/issues/155, it was closed with https://github.com/konstaui/konsta/commit/59216a6f4daa5b438eab0a477e2853f5195dc117, but that change didn't actually fix the file that the issue referenced (DeleteIcon.svelte).
For "A11y: visible, non-interactive elements with an on:click event must be accompanied by a keyboard event handler," I added a comment to suppress the warning because I wasn't sure what the desired fix was. I'd be happy to revisit that with more guidance if desired.
Versions:
@sveltejs/vite-plugin-svelte at 3.1.1.
konsta at 3.1.3.

Thanks for your consideration.

t0ggles-create konsta

Merged, thanks! I removed role button from irrelevant elements, as they still don't have much sense without key events handler


Fix _CMSG_ALIGN on DragonFly

The attempted fix in #2610 originally had 3 hard coded, but it was suggested I replace this with a size_of call. Unfortunately the suggestion omitted a subtraction from the size_of call, and I didn't catch it.
Tested by running the failing nix tests on DragonFly (and didn't change the code again after running the tests).

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit a894685dbe9d858364987b734aa671cb538232f3 has been approved by Amanieu

:hourglass: Testing commit a894685dbe9d858364987b734aa671cb538232f3 with merge 78e76aa33fd4019b2663ecd557b308fd13b3c824...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 78e76aa33fd4019b2663ecd557b308fd13b3c824 to master...


Remove redundant lense getters

Hi @KommuSoft thanks for your refactoring PRs. This one has conflicts from the previous ones getting merged, if you want to resolve it. Also, just so you know for the future, these changes are very small and would be totally fine bundled together in a single PR.

Another person for which I made improvements asked to do that with separate PRs, so I swapped to small PRs :). But I will take that into account for the next time.
Created a new PR where the merge conflicts are resolved.


Added helper script to delete unused .kibana indexes

Checklist

[x] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated
[x] PR title is ready for changelog and subsystem label(s) applied

The aim of the script is to remove old .kibana_* indexes and just keep the latest
Closing issues
closes #2118

@Schnitzel any news about this PR?

Hi @vincenzodnp these indices store kibana configuration and I'm not sure if we can just delete them. How do we know that they are unused?

should be resubmitted against https://github.com/amazeeio/helper-scripts


Data Share Content Validation Issue for github.io website

Section link1

Reason:
Format validation Issue: Text exceeds page size.

Section link1

Reason:
Format validation Issue: Extra symbols - or ~.

Section link1

Reason:
Typing annotation Issue: Please unify the format, update to **kwargs:Any or **kwargs.

Section link1

Reason:
Typing annotation Issue: Please unify the format, update to *args:Any, **kwargs:Any or *args, **kwargs.
@rohit-ganguly, @lmazuel for notification.

Close this issue as we'll put it into a summarzied one.


feat(lb): deterministic subsetting algorithm

gh-1288

Hello @OlgaMaciaszek , PTAL

Please be patient, it has only been one day since you submitted it.

Sorry, I was impatient

Hi @jizhuozhi, will you be able to finish this PR up soon? I'd like to get this feature included in 4.1.0.

Thanks @jizhuozhi, now only documentation, including this: #1289 (comment), is left. Please add corresponding changes in docs/modules/ROOT/pages/spring-cloud-commons/loadbalancer.adoc.

Hello, @OlgaMaciaszek . The document has been updated, PTAL

Hi @Buzzardo, could you please review documentation changes?

Fixes gh-1288.


Added 'View Elapsed Time' to browser extensions.

This extension will show the elapsed time from any date. For example, when the user hovers over the date '2000/01/01', a mouseover description will say '22 years, 9 months and 7 days ago'. The default date format will be 'YYYY/MM/DD'. The extension can be customized to encompass minutes and seconds, and also to edit the default date format.

Sorry, my mac added it automatically and there's no .gitignore. I'll remove it.


Add an API for a notebook kernel to modify its list of supported languages at runtime.

The NotebookKernel interfaces exposes a supportedLanguages: string[] field, but for kernels that support a dynamic list of languages, there's currently no way for that list to be updated in a meaningful way at runtime.
E.g., the .NET Interactive notebook extension supports doing something like this: #!connect -type=MySQL -name=sql -connectionString="abcd" and from that point forward, a cell language of SQL should be supported, but not necessarily before.
One way to accomplish this would be to add an event on NotebookKernel called LanguagesChanged or similar that either contained a new string array of the supported languages or instructed any listeners that a new string array was present in the supportedLanguages field; I don't have a strong preference either way.

This has happened with the new NotebookControllers - properties like label, description, supported languages etc can be modified freely


feat: Add terraform scanner (ported from tfsec)

Wot, no tests?

Wot, no tests?

Added lots of tests now :+1:

Wot, no tests?

Added lots of tests now +1

too many tests, its like you don't have confidence in the code

find . -type f -name "*_test.go" -print0 | head -z -n 10 | xargs -0 rm


Extend session timeout

To 4 hours
Originally created by Tom Smyth at 2015-09-13 20:59, migrated from redmine-#3093

Status changed from New to Closed

By Tom Smyth at 2015-09-23 06:31


Install LaTeX?

I have an exercise for my students that uses some LaTeX rendering in the plot labels and legend and this doesn't seem to work on the jupyterhub (repository here). I can run this successfully on binder by using the following apt.txt file, could you install these dependencies on the jupyterhub (I think these are standard Linux LaTeX packages):
https://github.com/jobovy/sparc-rotation-curves/blob/main/binder/apt.txt
Note that the first entry is the GSL, which isn't necessary for this (and is probably already installed?).
Thanks!

Looking through here: https://github.com/utoronto-2i2c/jupyterhub-deploy/blob/staging/deployments/utoronto/image/Dockerfile I don't see any Latex, so I think that'd be where it needs to be installed.
It's definitely possible - might take a little bit of fiddling since latex has a tendency to be a beast to install and configure :-)
Do you have any kind of time-table for needing this?

Thanks for looking into this! The assignment that this is relevant for is due in just a few days and I've given the students a work-around, so it's not terribly urgent at this point.
I install latex a lot in binders and GitHub workflows using the apt.txt file that I linked to above and that seems to work pretty well, so may not be that hard?

Apologies for missing this, @jobovy. I just tried it on the toronto hub, and got a working PDF:
sparc-rotation-curves(1).pdf
We also do have LaTeX installed (https://github.com/utoronto-2i2c/jupyterhub-deploy/blob/staging/deployments/utoronto/image/Dockerfile#L73), although only a couple of the base packages. I'll add more of the tex packages listed in your apt.txt as well. But let me know if it already works?

Hi @yuvipanda, thanks for looking into this! I think the issue is related to matplotlib's typesetting of LaTeX, which still doesn't work on the server (the issue isn't just compiling the notebook to PDF, but just running the notebook; note that to actually run the notebook you have to install additional code). I think it needs all of the packages in the apt.txt that I mentioned to work.
Although this problem would probably come up in future activities/assignments for my course, the assignment that sparked this issue has now been handed in and all the students managed to find a work-around. So this isn't urgent to fix right now, at least for us.

@jobovy out of curiousity, would it have been equally useful to allow for a non-latex PDF conversion via HTML? That is often more reliable than getting all the right Latex edge-cases to work for typesetting, and the output looks much closer to what the student notebooks look like when they're writing the content.

@choldgraf -- I'm not sure what you mean. The issue is just that I use LaTeX to typeset axis labels and legends in the matplotlib figures generated in the notebook. The issue isn't related to converting the notebook to PDF (it's not related to PDF output at all!). I think the latter is what you are getting at?

@jobovy ah, am sorry you had to implement workarounds! I've installed the packages mentioned here (I think), do you think you would have time to test it out? Would be very useful for me, since I'm not very well versed in TeX.

@yuvipanda -- Still doesn't work, I think it specifically needs the cm-super apt package (a LaTeX package with some special fonts).

@jobovy ah gotcha - I mis-read the issue here, sorry for the noise!

@jobovy cm-super is installed now! I'm also not sure what to look for to test, so if you could test again when you can that would be great!

@yuvipanda -- thanks! That got us further, but now matplotlib is complaining that it doesn't have dvipng... So it looks like the dvipng entry in the apt.txt file that I linked to above needs to be installed too, perhaps ghostscript as well.

Ok, so I installed dvipng and ghostscript as well, and with apt list --installed verified that everything in the apt.txt (except libgsl) is installed there (I've also automated this during deploy with https://github.com/yuvipanda/hubploy/pull/107). libgsl is coming from conda, and I verified it is installed with conda list | grep gsl.
Hopefully, I'll be able to work on making these list of installed packages public in an easy way so there can be a reference for what new ones actually need to be installed...
Thank you for your patience!

@yuvipanda -- Thanks, it works now 🎉 🎉 !
I also saw that 2i2c got funding, congratulations! I think this will really revolutionize incorporating interactive computing in teaching!

:D Thank you for your patience, @jobovy!


e2e test: Update Commit details UI changes

Fixes

https://issues.redhat.com/browse/HAC-3682
Description

Update Advance e2e flow as per the recent UI changes on commit Details
Add GH_USERNAME to cypress config

Type of change

[ ] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[x] Other (please describe): Test

Screen shots / Gifs for design review

How to test or reproduce?

Browser conformance:

[x] Chrome
[ ] Firefox
[ ] Safari
[ ] Edge

/lgtm
/approve

/RetesT

/retest


Remove: EaseUS.PartitionMaster version 18.8

Resolves #157312

Microsoft Reviewers: Open in CodeFlow

URL: https://d1.easeus.com/epm/free/epm_free_18.5_A.exe
OK
(Automated message - build 907)


Fix RFC link

http -> https

Without https:

Ah @nic can you DCO sign your commit?

@vbatts Sure, done. :)


Add emission_rate parameter in run_algorithm

Because when backtesting 1 minute price data, an emission_rate parameter is absolutely required to set_benchmark.
plz test and apply janson github:) Thanks for your zipline update

This probably needs a bit more than the parameters.


Sections of code should not be commented out

Commented-out code distracts the focus from the actual executed code. It creates a noise that increases maintenance code. And because it is never executed, it quickly becomes out of date and invalid.
Commented-out code should be deleted and can be retrieved from source control history if required.
Directory: MjpegFileWriter
Path:jmonkeyengineFall2024\jme3-desktop\src\main\java\com\jme3\app\state\MjpegFileWriter.java
Just remove the unwanted code which is been commented out, this commented code is of no use so its better to remove it so that it improves readability of the code.
issue on ide:

Solved on ide:

issue on sonarqube:

solved issue in sonarqube:

Issue has been solved


Add support for Dynamic / arbitrary groups of objects

Overview
There is often a need to group objects together within a single model or content-type.  For example, I may want to group all "primary" devices within pairs, all sites that have a circuit from a given carrier, all devices that terminate a circuit, etc. and the list can go on.  The point is there is many reasons it makes sense to group objects together.
Once this feature is complete, it will enable applying config contexts to these groups, custom and computed fields to these groups, webhooks, etc.  Anywhere a content-type is being selected, a group could also be applied.
User Story
As Nelly the Network Engineer,
I want to be able to create dynamic groups,
So that I can easily see which devices match a specific query further enabling network automation use cases on those "groups" of devices that often map back to "network services."
I know this is done when:

I can go to a page called "Dynamic Groups" and view all Groups
I can click on a Group and view all objects within that dynamic group
I can create a new dynamic group based on a queryset (this could be expanded in the future)

This is required for #768
TODO

[x] Implement Dynamic Group Model, Forward, Reverse Lookup (done via: #1425, #1502)
[x] Add Regex lookup expression to magic FilterSet generator (see #1525)
[ ] Housekeeping: Groom existing FilterSets which are missing all model attributes (see: #1483)
[ ] #1822
[x] Enhancement: To support advanced query types, review #1470
[x] #1614

I was thinking the UI can be extended for how RBAC creates attribute based access control (ABAC) and used here as well.

To clarify what I had in mind, I created the included wireframe.

Some notes:

Introspection will be non-trivial for

What I labeled as FK Model (admittingly not the best of names)
What check type can be used
Which models can and cannot be aggregated

I only included the relevant parts of the RBAC (or ABAC) model

I think there's room for some clarification here - in some past conversations "dynamic" groups have implied the ability to define a set of group membership criteria and have the members of the group automatically (dynamically) determined based on those criteria, as opposed to manually defined groups wherein the user creates a group and manually assigns each member into the group. Or to put it another way, once I have dynamic groups defined (based on querysets or other filtering criteria), and I create a new object/record that matches those criteria, is it expected to automatically become a member of all applicable groups, or is it on me as the user to assign it into those groups myself?

It should be "dynamic" for this feature.  Given #892, that FR may end up being the "static" part of this feature.
@itdependsnetworks with what you are showing, are you just showing that as a way to build the groups as an option vs. quersyet vs. regex, etc. ?

This is a potential UI design for queryset

To add to @glennmatthews 's brainstorming point about group of groups, as the plugin author I would like to allow for "group expressions". Maintaining the relationships between groups quickly become cumbersome, thus I would like to allow my plugin for following queries:

if device IN Group1 AND IN Group2
if device IN Group1 AND NOT IN Group2
if device IN Group1 AND NOT (device in Group2 and device in Group3)
etc

Having basic form of maintaining the relationship between multiple groups could help in maintaining ValidatedSoftwares and ComplianceRules in our plugins.

@mzbroch regarding static groups, the thought was we would at a minimum have RegEx to build dynamic groups and thus, could have multiple RegEx's using exact matches yielding what would be a static group.
Group expressions is an interesting idea.  I'm sure there are multiple ways to do this, but will remind us of Ansible inventory patterns.  Maybe this is an add-on or power-use method for expressions in the future.
@dgarros Thanks for the PR on the prototype (#1047 ).  The initial thought was to have a UI similar to what @itdependsnetworks proposed above with a row per filter criteria.  The filter criteria could be a Queryset or RegEx (or more precise using the interface you have in the screen shots that is similar to config contexts).  Is this inline with what you had in mind as well?
Will defer to @glennmatthews @jathanism @lampwins on the implementation of your prototype!
Thanks @dgarros !

Another place where dynamic groups would be useful would be in adding a feature to allow webhooks to be triggered only for objects in a specific dynamic group.

An attempt at consolidating the requirements described in this issue thus far together with some additional requirements identified through internal discussions and documentation:

A dynamic group must automatically reflect changes in membership, i.e. object creation/change/deletion should automatically be reflected in its group membership.
A dynamic group can be defined based on something resembling a filterset and/or queryset, specifically:

Must be able to define a dynamic group based on "include"/"match" constraints

These constraints could potentially include exact match, substring match, regex match, greater-than/less-than match, etc.

Must be able to define a dynamic group based on "exclude"/"no-match" constraints

As above, potentially many different kinds of such constraints

Must be able to define a dynamic group based on multiple constraints (match A and match B, match A and exclude C, exclude C and exclude D)
Must be able to define a dynamic group based on membership in other groups ("belonging to group A or group B", "belonging to group A but not group C", etc.)

Performance of "given an object, find all dynamic groups it belongs to" must be reasonable
Performance of "given a dynamic group, find all objects that belong to it" must be reasonable
UI/UX for defining (creating and editing) dynamic groups must be reasonably user-friendly
UI must present a listing of all defined dynamic groups

Filtering this listing by content-type (i.e., only show groups of Devices, or of Sites) must be supported

UI must present a consistent way of showing all members of a given dynamic group
UI must present a consistent way of showing all dynamic groups an object is a member of
REST API must present a consistent way to query for members of dynamic groups and group membership of an object
REST API must be capable of filtering retrieved objects based on their group membership
Dynamic groups of plugin-defined model instances must be possible/supported

Non-requirements at this time:

Definition of dynamic groups containing multiple different content-types (the biggest driver for this would be Devices/Virtual Machines, and that's better addressed by #1178)

@bryanculver I understood "Dynamic / arbitrary groups of objects" was not making it into 1.3 but my understanding was that all functionality was going to be in 1.4, is that not the case?
As an example, this refers to #1483 which seems to have been added to 1.4 and removed.

@bryanculver I understood "Dynamic / arbitrary groups of objects" was not making it into 1.3 but my understanding was that all functionality was going to be in 1.4, is that not the case?
As an example, this refers to #1483 which seems to have been added to 1.4 and removed.

Dynamic groups were added in 1.3.
Dynamic groups of groups is currently being considered for 1.4. Currently there is a spike and prototype ongoing as to how to implement this, see: #1614.
Please be mindful that Epics are likely not going to ever be marked for a milestone in entirety because they often contain more stories and work, which may even include breaking changes, than can fit in a reasonable release window.
Paging @lampwins.

Dynamic Groups was added in 1.3. Groups of Dynamic Groups was added in 1.4. Future follow-ons can be followed in #3268


[integration] Slack integration

MVP
everyday at let's say 10am, the daily question will get posted into a slack channel and can be answered by all the user's in that channel.
Then after (maybe 2 hours (?) - will be configurable eventually) the fastest user's get posted in a leaderboard
what does this require?

New stripe plan (team)
a slack bot to post the questions.

what do you think @jakemackie ?

Yooo that's a huge idea. Slack docs are pretty awful from what I remember but this would really be sick


[RELAY][OPS]LRN and L2_Normalize

#1799

[x] LRN
[x] L2_NORM

Thanks for contributing to TVM!   Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from others in the community.

@tqchen @srkreddy1238  @junrushao1994 @MarisaKirisame please review.

l2 normalization is not as common and let us move it to level2


use documentElement instead of body to detect css3 animations

Thank you for the wonderful plugin.
I am using stackbox in a Rails application. When stackbox is loaded in the asset pipeline, body is not yet loaded. Hence, the CSS3 animation detection code returns
document.body is null

Looks good to me. Also great to hear that you like the plugin. Thx! :-)


How to Set Dpi ?

Hi everyone,
I m working with Fujitsu fi-65F on a C# project. I want to get image with max DPI (600).
how to do that ?

You set ScanSettings.Resolution.Dpi to whatever you like before scanning.


youtube appears to be able to detect every method except mute ad

as in the picture. i guess they've detected my regular activity of using ad skipper, but it could be that youtube is updating their new detection method. only mute ad works for now.

i had to set back to this to see videos playing again (yeah, at least mute ad is still a thing):

(also, do i have to follow a template to submit an issue though? thanks.)

Thanks for bringing it up. Unfortunately, it's hard to know exactly how YouTube detects the extension. I, myself, have had the issue a couple times. It might be triggered after a certain number of ads skipped, or days used. It might depend on the playback rate. They might be using an AI algorithm that, upon receiving all the user data, provides a probability for ad blocker usage.
All those aspects, if true, make avoiding detection highly intractable; the best we can probably do is rely on trial and errors by letting the user pick the configuration that works for them.
Personally, I keep a high playback rate. Sometimes I get detected, but a few reloads of the page solve the issue. I find it faster than keeping a very low playback rate.


Option to disable the popup that appears on every time you type in a task line

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
Option to disable this popup:

✔ Solution
A toggle in the settings or something similar.
❓ Alternatives
Disable automatic popup, only summon on shortcut (e.g. ctrl+space)
📝 Additional Context
It’s hard to manage multiple tasks via keyboard because the popup steals focus and consumes the up and down arrows that I would have used to move to the next task or previous task.

Ok, there's actually an option "Auto-suggest Task Content"


Add GitPrime to the list

We're remote-first, too :)

Thank you! Do you mind filling out a company profile? I'll merge this as-is, however.

Thanks, I've added the profile in #116.


Finish the documentation of technical approaches

[ ] Readme — will be updated after everything else
[x] Auditable — #54
[ ] Complexity-checks
[x] Error-handling — #29
[ ] Failing-closed
[x] Parser-not-validator — #60
[x] Performance
[x] Platform support — #52
[ ] Prototype-safety
[ ] Require-validation
[x] Secure-code-generation — #53
[x] Specification-support — #30
[ ] Strong-mode
[ ] Defaults-assigning
[ ] Filtering-additional

Postponed to unfreeze 1.x.


All control events run when any control event triggered

If you do something like this:
        primaryButton.addControlEvent(.touchDown) {
        }
        primaryButton.addControlEvent(.touchUpInside) {
        }

You'll see that the touchDown is triggered on a touchUpInside, and vice versa

Great to hear, on both fronts 🎉 😄 -- thanks again for reporting this, and please don't hesitate to report anything else you might stumble upon, or any improvements you'd be interested in!
Thanks and happy new year!
FWIW version 2.5.0 & later will include this fix.


Restore sprites and map data on every game start

also, add API method to save modified in runtime sprites/map data

sprites/map data restores on every startup
added sync() api to save modified in runtime sprites/map data
done in .26

@nesbox this is the fastest new feature/issue implementation I've ever saw :)

yes, I implemented it before the issue was created :)


Failure of SQL query during parsing

Describe the bug
When I use JSQLParser libraries to parse SQL query , it failed as follows when invoking following statement and query.
Passed SQL Query on which failure happen:
"SELECT of.Full_Name_c AS FullName FROM comdb.Offer_c AS of"
val statement: Statement = CCJSqlParserUtil.parse(sqlQuery)
Exception trace:
Caused by:
net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "SELECT" <K_SELECT>
    at line 1, column 1.

Was expecting one of:

    "CALL"
    "COMMENT"
    "COMMIT"
    "DECLARE"
    "DELETE"
    "DESCRIBE"
    "DROP"
    "EXEC"
    "EXECUTE"
    "EXPLAIN"
    "GRANT"
    "INSERT"
    "MERGE"
    "SET"
    "SHOW"
    "TRUNCATE"
    "UPDATE"
    "UPSERT"
    "USE"
    "VALUES"
    "WITH"

	at net.sf.jsqlparser.parser.CCJSqlParser.generateParseException(CCJSqlParser.java:25031)
	at net.sf.jsqlparser.parser.CCJSqlParser.jj_consume_token(CCJSqlParser.java:24875)
	at net.sf.jsqlparser.parser.CCJSqlParser.SingleStatement(CCJSqlParser.java:244)
	at net.sf.jsqlparser.parser.CCJSqlParser.Statement(CCJSqlParser.java:81)
	at net.sf.jsqlparser.parser.CCJSqlParserUtil.parse(CCJSqlParserUtil.java:63)
	at net.sf.jsqlparser.parser.CCJSqlParserUtil.parse(CCJSqlParserUtil.java:40)

But same query passed when I changed the alias as follows:
// This query worked by changing alias 'of'  to 'off'
"SELECT off.Full_Name_c AS FullName FROM comdb.Offer_c AS off"
// This works as  alias 'af' used inplace of 'of' quite strange
"SELECT af.Full_Name_c AS FullName FROM comdb.Offer_c AS af"
This query is created by third party system so it is not in our control to change the alias from 'of' to something that work.
To Reproduce
Steps to reproduce the behavior:

Example SQL
"SELECT of.Full_Name_c AS FullName FROM comdb.Offer_c AS of"

Parsing this SQL using JSqlParser with this statements

val statement: Statement = CCJSqlParserUtil.parse(sqlQuery)
3. Exception
Expected behavior
Query must work irrespective whether I use alias of or not. Please let us know how to fix it.
System

Database you are using = MySQL 5.7.25
Java Version  = 8.0+
JSqlParser version  = 3.2

@wumpz Can you please help on this issue.

Took a while. Now I am back on business.

@wumpz Thanks for the fix. I appreciate all your help. What is the planned release for this fix in future.

Hi,
At the moment not yet. There are more PRs to merge.Tobias
-------- Ursprüngliche Nachricht --------Von: Peeyush Bishnoi<EMAIL_ADDRESS>Datum: 11.08.20  06:39  (GMT+01:00) An: JSQLParser/JSqlParser<EMAIL_ADDRESS>Cc: Tobias<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Betreff: Re: [JSQLParser/JSqlParser] Failure of SQL query during parsing
  (#1029)
@wumpz Thanks for the fix. I appreciate all your help. What is the planned release for this fix in future.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.


feat(ci): staging deploy

URL to Jira task (this should explain the task)
Explain what the code alterations do
What has to be tested? How to test it?
Any new dependencies? Why were they added?

/deploy

/deploy


fix: libuv can return UV_FILE or UV_TTY for FILE_TYPE_CHAR handles

When uvwasi is used on Windows, it might be used in non-"ConsoleMode" and the uv_guess_handle function can return two different values for FILE_TYPE_CHAR because of that difference, see https://github.com/libuv/libuv/blob/v1.x/src/win/handle.c#L42-L47
This function can also return UV_FILE in the case of FILE_TYPE_DISK but I assume that a disk file will properly stat and there isn't a good way to figure out which one is being returned.
This fixes #157 (took me about a month of debugging)

Thanks for the suggestion! I've merged it and I'll rebase once #180 lands.

It should be OK to rebase this now.

Thanks @cjihrig! This has been rebased now. Cheers 🍻

Excellent! We're actually using this through wasm-micro-runtime that uses uvwasi as an experimental WASI backend. And I'm writing bindings for OCaml, which is where I encountered this bug.

Oh that's cool. In that case, you won't have to wait for a Node release (which would probably be next year). I'll get a uvwasi release out today.

This is published in v0.0.14.


Unergonomic toggle between Table View and Card View

Those two complementary buttons are on the opposite sides of the screen - which is bad UX.

Let's keep the toggles on the right side.


Update contribution guidelines

Update contribution guidelines to include the following:

For new functionality, please open an issue before creating a PR
For bug fixes, you can open a PR first
Tell people to be polite when communicating with contributors, maybe even link to some blog posts on how to give constructive feedback and without being a jerk
Tests are required with ALL contributions

Updated contribution guidelines and added a code of conduct in 107eb95fe034f


Check if BIG-IQ logic broken on TMOS 15.1.5+

The check if ATC deploy is used on Big-IQ or Big-IP does not function as intended when using TMOS 15.1.5+.
main.yml checks if GET /info/system returns a 404 (then Big-IP) if 200 (then Big-IQ)  https://github.com/f5devcentral/ansible-role-f5_atc_deploy_declaration/blob/master/tasks/main.yaml#L93
TMOS 15.1.5+ returns a 302 redirect for GET /info/system which breaks the logic:
[root@bigip:Active:Standalone] config # tmsh show sys version

Sys::Version
Main Package
  Product     BIG-IP
  Version     15.1.5
  Build       0.0.10
  Edition     Final
  Date        Fri Dec 10 13:48:13 PST 2021

[root@bigip:Active:Standalone] config # curl https://localhost/info/system -k
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>302 Found</title>

Found

The document has moved here.

This causes neither check Big-IQ or Big-IP task to be called so the result of DO/AS3/TS deploy is not available.
The check for Big-IQ vs Big-IP needs to be more precise.

Hello,
F5 has been working on a new module to deploy AS3 declaration via BIG-IQ. I would recommend you to use that module instead.
https://clouddocs.f5.com/products/orchestration/ansible/devel/f5_bigip/modules_2_0/bigiq_as3_deploy_module.html#bigiq-as3-deploy-module-2
Thanks & Regards,
Roman

Thanks for the suggestion, customer is not using Big-IQ and prefers to render DO/AS3/TS in ansible and submit to Big-IP directly.
Are issues in ATC deploy no longer being fixed? Should this repo be archived?
We would like to use the v2 module bigip_as3_deploy but this git issue is preventing us: https://github.com/F5Networks/f5-ansible-bigip/issues/3

Hi, we have prioritized this internally and it will be part of our next release.  The estimated date is March 2022 (first two weeks) Let us know if you have any comments or questions

Thanks for the update!
Could you take a look at #33 as well? My same customer is running into that issue. I created my own fix, but it would be great to see it patched here as well.


Added Stephen

Hey @silent12ill,
Thanks for you contribution, I'd love to merge it, however you should remove the \ from God\'s gift
Please do it, and I'll merge your changes
Thanks!


Chunk-encoded request doesn't recognize iter_content generator

Passing a generator created by iter_content() as request data raises "TypeError: sendall() argument 1 must be string or buffer, not generator".
Expected Result
The POST request successfully delives the content from the GET request.
Actual Result
A TypeError is raised:
Traceback (most recent call last):
  File "..\test.py", line 7, in <module>
    PostForward("http://myhost/img/foo.png", "http://myotherhost/convert")
  File "..\test.py", line 6, in PostForward
    return requests.post(url=dst, data=data, headers={'Content-Length': length})
  File "C:\Python27\lib\site-packages\requests\api.py", line 112, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "C:\Python27\lib\site-packages\requests\api.py", line 58, in request
    return session.request(method=method, url=url, **kwargs)
  File "C:\Python27\lib\site-packages\requests\sessions.py", line 508, in request
    resp = self.send(prep, **send_kwargs)
  File "C:\Python27\lib\site-packages\requests\sessions.py", line 618, in send
    r = adapter.send(request, **kwargs)
  File "C:\Python27\lib\site-packages\requests\adapters.py", line 440, in send
    timeout=timeout
  File "C:\Python27\lib\site-packages\urllib3\connectionpool.py", line 601, in urlopen
    chunked=chunked)
  File "C:\Python27\lib\site-packages\urllib3\connectionpool.py", line 357, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "C:\Python27\lib\httplib.py", line 1042, in request
    self._send_request(method, url, body, headers)
  File "C:\Python27\lib\httplib.py", line 1082, in _send_request
    self.endheaders(body)
  File "C:\Python27\lib\httplib.py", line 1038, in endheaders
    self._send_output(message_body)
  File "C:\Python27\lib\httplib.py", line 886, in _send_output
    self.send(message_body)
  File "C:\Python27\lib\httplib.py", line 858, in send
    self.sock.sendall(data)
  File "C:\Python27\lib\socket.py", line 228, in meth
    return getattr(self._sock,name)(*args)
TypeError: sendall() argument 1 must be string or buffer, not generator

Reproduction Steps
import requests
def PostForward(src, dst):
	with requests.get(url=src, stream=True) as srcResponse:
		length = srcResponse.headers['Content-Length']
		data = srcResponse.iter_content(1024)
		return requests.post(url=dst, data=data, headers={'Content-Length': length})
PostForward("http://myhost/img/foo.png", "http://myotherhost/convert")

System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.4"
  },
  "cryptography": {
    "version": ""
  },
  "idna": {
    "version": "2.6"
  },
  "implementation": {
    "name": "CPython",
    "version": "2.7.14"
  },
  "platform": {
    "release": "10",
    "system": "Windows"
  },
  "pyOpenSSL": {
    "openssl_version": "",
    "version": null
  },
  "requests": {
    "version": "2.18.4"
  },
  "system_ssl": {
    "version": "100020bf"
  },
  "urllib3": {
    "version": "1.22"
  },
  "using_pyopenssl": false
}

P.S.: The error is not raised if the headers param is omitted from the POST request. In that case the request is sent and the Flask server receiving the request responds with 411 Length Required.

iter_content returns a generator, which means that we expect to use chunked Transfer-Encoding. You cannot specify that in addition to a Content-Length. The correct way to deal with this is to use the StreamingIterator from the toolbelt.


Fix hawkular status method to return a status hash

Fix hawkular status method to return a status hash.
The status hash fields are strings and not symbols. response[:MetricsService] is always nil. Fix is to check for response["MetricsService"] instead.
This PR:
a. Remove useless check for none existing response[:MetricsService].
b. Check for response["MetricsService"].

@miq-bot add_label providers/containers, bug
@simon3z, @enoodle please review

@yaacov please improve the title of this PR.

@simon3z changed.

@chessbyte I have the feeling that we may want this in darga to improve the check on the hawkular status on provider addition. Do you feel it could be risky at this time?

@simon3z, @yaacov does this mean that currently (without this patch) all attempts to validate the Hawkular credentials fail?
If so, is that already logged as a bug?  That may change the necessity for darga/no vs. darga/yes.

does this mean that currently (without this patch) all attempts to validate the Hawkular credentials fail?

@blomquisg NO, It passes a rare-end-case as valid when it should fail.
a. Currently it checks that Hawkular server returns a valid json that we can parse it. Any valid hash will result in "valid credentials". This is OK because, if Hawkular answered with a json struct credentials are probably ok. This patch also check that the MetricsService field inside the json has the value STARTED witch mean that Hawkular is also running metrics.
b. The current functions does useless checks for :MetricsService this is wrong and a bug, and should be fixed.


Equiv of ldrb.fiber_to_xdmf in fenicsx-ldrb?

What is the equivalent of
ldrb.fiber_to_xdmf(fiber, "fiber")
in fenicsx-ldrb?
Thanks!

Hi @AnoopRKulkarni. Thanks for bringing this up. This is not something I have implemented yet.
If you are looking for a way to color the arrows based on the angles then please check out my comment in https://github.com/finsberg/ldrb/issues/59#issuecomment-2274289782
If you want to visualize quadrature functions, then I plan to implement some support for this using the gist here as starting point. I think though, that it might be better to make a separate package for this, because this might be useful in other projects as well.
If you don't work with quadrature functions then I think it should be possible to save the function in vtx format, see e.g https://github.com/finsberg/fenicsx-ldrb/blob/90a16b1ebe27dac1f0427757cfac969467958f6b/src/ldrb/cli.py#L144

Thank you! I as much thought so. Thanks for all the supported links.
For now, I solved my problem by using dolfinx.io.VTXFilter and saving as *.bp and then my custom converters to convert them to my format (XDMF happened to be one of them but primarily openCARP).
Thanks again
~anoop


Use vendor mod when building go binary.

By passing -mod=vendor we make sure the vendor dependencies are being
used when building the binary.
See https://tip.golang.org/cmd/go/#hdr-Modules_and_vendoring
https://jira.mesosphere.com/browse/DCOS-43217

@mesosphere-ci run all integration tests


[Snyk] Upgrade @types/node from 22.8.6 to 22.9.0

Snyk has created this PR to upgrade @types/node from 22.8.6 to 22.9.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.

The recommended version was released on a month ago.

Release notes

Package name: @types/node

22.9.0 - 2024-11-05

22.8.7 - 2024-11-03

22.8.6 - 2024-10-31

    from @types/node GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

was updated in #35


✨ Add basic client generation

Summary
This PR adds generation of client code, including informers and listers, using the plain Kubernetes code-generator module (k8s.io/code-generator). The generated code appears in pkg/generated.
In order to get this working, I had to insert a directory in the api branch. The v1alpha1 API is now at api/edge/v1alpha1 (it used to be at api/v1alpha1).
I also had to rename the file placement_types.go to types.go (https://github.com/kubernetes/kubernetes/issues/119930).
Bug #1611 predates this PR and this PR does not fix that bug. I do not know whether there is any (relevant) incompatibility between code-generator and controller-gen.
This is temporarily built on #1607 and will be rebased once that merges. In the meantime I have placed a /hold on this PR.
Related issue(s)
Fixes #1537

/cc @nirrozenbaum
/cc @ezrasilvera
/cc @pdettori
/cc @waltforme

@MikeSpreitzer So changing the package from Edge will be in a different PR ?

The force-push to dc2947c04 is a rebase onto the current edition of #1607 .
@ezrasilvera : yes, the API group rename will be a separate PR.

waiting for #1607 to merge, so
/hold

The force-push to 0ff436967 is a rebase onto the current edition of #1607 .

The force-push to 3e640fd1e is the rebase onto main.
/remove-hold

/lgtm
/approve


Add custom boolean attribute support #503

Adds BooleanAttribute that writes out itself without a value
Adds API in Element for setting boolean attributes
Update parser to distinguish between no value and empty value

Thanks, looks interesting. Can you check what the behavior should be for XML parsing and XML syntax output? Thinking that at least for XML syntax output, boolean attributes should actually output as empty string attributes.

From what I have found, the original XHTML specification only allowed the format attribute="attribute".
Latest XHTML specs also allow attribute="" according to e.g. https://html.spec.whatwg.org/#boolean-attributes
For attributes defined as boolean using XML Schema, the format is attribute="true" or attribute="1"  (not compatible with XHTML version).

Carefully bumping this.

Thanks! Merged.


[V2] Statusbar notworking

Statusbar is not working.

related? https://github.com/framework7io/Framework7/issues/1811

Hi @ytbryan,
No, it is not related with this.
The problem started now with the last commits 90c21b7268402eb85e0182a058b3a0c116f010d7

@nolimits4web Solved!
Thanks


validation.show is not relevant for scenarios where you want to hide error after field has been updated

I am working on a solution that only shows the error message when the field is in focus.
I went through this example: https://formly.dev/examples/validation/force-show-error, and it seems that we could show/hide error message through expressionProperties: validation.show.
The bug is that validation.show = false is not acknowledged after you focus and blur out of the field.
Steps to reproduce the bug in the example 'force-show-error':

Focus on the Email
Focus out of Email
Uncheck the 'Force show error state'

Expected result: Validation error is no longer visible since validation.show is now set to false
Actual result: Validation error is still visible

To change that behavior you can pass a custom showError function:
export function showError(field: FieldType) {
  return ....;
}

@NgModule({
  imports: [
    FormlyModule.forRoot({ extras: { showError } }),
  ],
})


Porting Dates Formatting to Swift

Description
LEARNER-1868
In Swift 3.0 and onwards Apple changes Objective-C NSDate to Date in bridging. That was causing problem at some places. I have ported OEXDateFormatting and OEXDateFormattingTests to Swift to eliminate bridging issues.

Codecov Report

Merging #968 into master will decrease coverage by 0.07%.
The diff coverage is 75%.

@@            Coverage Diff             @@
##           master     #968      +/-   ##
==========================================
- Coverage   45.86%   45.78%   -0.08%
==========================================
  Files         403      401       -2
  Lines       26419    26386      -33
==========================================
- Hits        12116    12081      -35
- Misses      14303    14305       +2

Impacted Files
Coverage Δ

Source/UserAPI.swift
92% <ø> (ø)
:arrow_up:

Source/OEXGoogleAuthProvider.m
40% <ø> (ø)
:arrow_up:

Source/OEXDataParser.m
26.56% <ø> (ø)
:arrow_up:

Source/Core/Test/Code/BadgesAPITests.swift
87.27% <0%> (-1.82%)
:arrow_down:

Source/OEXMyVideosViewController.m
0% <0%> (ø)
:arrow_up:

Source/AccomplishmentsView.swift
17.89% <0%> (ø)
:arrow_up:

Source/OEXDownloadViewController.m
0% <0%> (ø)
:arrow_up:

Source/NSError+OEXKnownErrors.m
55% <0%> (ø)
:arrow_up:

Source/OEXMyVideosSubSectionViewController.m
0% <0%> (ø)
:arrow_up:

Source/CourseLastAccessedController.swift
87.69% <0%> (-3.08%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3b08b5...aeb7d79. Read the comment docs.

@saeedbashir There are few comments please take a look into them. Thanks

@salman2013 Ready for another pass.

@saeedbashir The PR is ready for second pass. Thanks

@saeedbashir you can squash and merge. Thanks


Add Regression Tests for Bogus Existential Checked Casts

Try to make sure we don't do the wrong thing here in the future.  We've
been accepting the optional code for a long long time.
See rdar://57674858

@swift-ci please smoke test and merge


pact-jvm-consumer-junit: type matcher does exact match?

I have the following matching rules:
"body" : {
        "addresses" : [ {
          "someProp" : "wyVAbWfGtKzMYodnbmid"
        }, {
          "someProp" : "gDhgtTvKZqAeZfqIOgUA"
        }, {
          "someProp" : "hzbPMFEBosihlNvNTgsU"
        } ],
        "name" : "ggncZxASEudXfuslkZGI"
      },
      "responseMatchingRules" : {
        "$.body.addresses[1].someProp" : {
          "match" : "type"
        },
        "$.body.addresses[0].someProp" : {
          "match" : "type"
        },
        "$.body.name" : {
          "match" : "type"
        },
        "$.body.addresses[2].someProp" : {
          "match" : "type"
        }
      }

And expect them to match the following response:
{"name":"name1","addresses":[{"someProp":"p1"},{"someProp":"p2"},{"someProp":"p3"}]}

but instead get a matcher error:
 Diff:

 @3
         {
 -            "someProp": "wyVAbWfGtKzMYodnbmid"
 +            "someProp": "p1"
         },

 @6
         {
 -            "someProp": "gDhgtTvKZqAeZfqIOgUA"
 +            "someProp": "p2"
         },

 @9
         {
 -            "someProp": "hzbPMFEBosihlNvNTgsU"
 +            "someProp": "p3"
         }

 @12
     ],
 -    "name": "ggncZxASEudXfuslkZGI"
 +    "name": "name1"
 }

I was under impression that "type" match means that value of the property is not important only the type is.

Clarification: The name, although mentioned in diff, is correctly matched by type, only the array elements are matched exactly.

My team had this issue when the contract was pact 2.12 and the verification was 2.11, ensuring both were the same version solved it. This has not been an issue with 2.12 and 2.13.
Sent from my iPhone

On 10 Apr 2015, at 20:51, Peter Litvak<EMAIL_ADDRESS>wrote:
Clarification: The name, although mentioned in diff, is correctly matched by type, only the array elements are matched exactly.
—
Reply to this email directly or view it on GitHub.

There was a change in version 2.1.12 to use square brackets for array indices to bring it in line with the other pact implementations. See https://github.com/DiUS/pact-jvm/releases/tag/2_1_12
As per @jamessawle comment, check that all your versions are the same.

I'm using gradle so the consumer dependency is:
compile('au.com.dius:pact-jvm-consumer-junit_2.11:2.1.13')

And for the provider test (gradle 2.2) I'm using the following plugin setting:
plugins {
    id "au.com.dius.pact" version "2.1.1"
}

And getting this error. Is there a different version of the plugin that I should use?

Ok, seems to work with x.10 the way I expected to, there are errors with x.12 and x.13 with state change settings which I'll post separately.
Thank you for your help.


⚠ Taitopia Cloud Rendering Service Status has degraded performance

In cb62ca0, Taitopia Cloud Rendering Service Status (https://status-api.taitopia.design/health/taco) experienced degraded performance:

HTTP code: 200
Response time: 8 ms

Resolved: Taitopia Cloud Rendering Service Status performance has improved in 32eb47e after 5 hours, 53 minutes.


feat(core/config): support dotenv

I believe it's handy to support loading env vars from dotenv file when testing.

Wait... It just came to me that we can just source the .env file. Closing this PR.


Is there a PlayerDiggingPacket

Is there a PlayerDiggingPacket or which packet is needed to break a block normaly.

It's called ClientPlayerActionPacket in MCProtocolLib. For future reference, you can check the mappings in MinecraftProtocol.java to see what packet IDs correspond to what classes.


Proper handling or errors raised in Unity#componentDidMount

Is your feature request related to a problem? Please describe.
The componentDidMount lifecycle of the Unity component is asynchronous:
https://github.com/elraccoone/react-unity-webgl/blob/5a0c3c14093434a320cea8e25b6f10a54750fa18/source/components/unity.ts#L42
As such, errors raised in it resp. causing the promise to fail cannot be handled via a usual ErrorBoundary component and thus regularly end up in an unhandledrejection event that can only be handled globally.
Describe the solution you'd like
Regularly, errors that are raised by async lifecycles should be handled via a state and cause proper reactions, either in the component itself or its parent (or both). Thus, the component should catch such errors properly instead of letting them bubble up to the global level.
There are at least two options to actually respond to such an error in extension to catching it:

Notify the parent using the error message on the event listener on the UnityContext to be provided. Might not be intended to use this one that way, though.
Let the parent define some kind of renderFallback function that receives the error and returns the ReactNode to be rendered instead, like:

export default interface IUnityProps {
  // ... other properties as already defined atm.
  renderFallback?: (err: any) => ReactNode;
}

Describe alternatives you've considered
Currently, I'm using inheritance on TS level to implement the error handling and trigger event invocation so that the parent component can react to the error properly (see option (1) as described above), like:
import OldUnity from "react-unity-webgl";

interface UnityState {
  mountError?: Error;
}

export class Unity extends OldUnity {
  state: UnityState = {};
  async componentDidMount(): Promise<void> {
    try {
      await super.componentDidMount();
    } catch (err) {
      // Publish the error so that the surrounding component can
      // respond to it.
      const errorMsg: InitError = {
        type: "init" as const,
        error: err,
      };
      this.props.unityContext.invokeEventListener("error", errorMsg);
      this.setState({
        mountError: err,
      });
    }
  }

  render(): ReactNode {
    if (this.state.mountError) {
      return null;
    }
    return super.render();
  }
}

Additional context
The easiest way to trigger this errornous behavior is to provide invalid or unreachable URLs to any of the properties required by UnityContext.

Hi DorianGrey! Thank you for your details report. That's a good find, I will implement this in the next patch release.

I would have provided a PR already if I could have made an actual decision about which options sounds better resp. is more reasonable, especially if the parent component requires to perform some additional task when detecting an error raised by the Unity component, like removing loading progress indicators.

While the renderFallback version is quite concise and simple, it would take additional workarounds or tricky steps to trigger additional behavior in the parent component.
While the event listener is simple to wire with additional responding behavior in the parent, this version appears to be some kind of misuse of the event system; or at least errors emitted this way should be specific values so that the parent can separate them from other errors potentially forwarded this way during runtime.

Issue should be fixed in 8.0.2


Update for purescript 0.7

Can you make a v0.3.0 release for this please? Thanks!

@garyb can you keep the 0.7-updates branch for a while since it's still referenced by purescript-css?

There's a PR open for purescript-css that will solve that soon, for now if you install purescript-nonempty-array#68d6e04cd1874f01489a1de4b7dca5c7186f88d0 first with bower, you can then force that choice using the resolutions thing.

That helps, thanks!

I've updated purescript-nonempty-arrays to version 0.3.0 for PS 0.7.