CSS animations of added/deleted table rows

Hi!
I wanted to try out ReactCSSTransitionGroup (https://facebook.github.io/react/docs/animation.html) on react-toolbox TableRow's.
If try to do it this way:
<ReactCSSTransitionGroup transitionName="example" transitionEnterTimeout={5000} transitionLeaveTimeout={3000}>

    <TableHead>
      <TableCell numeric>Id</TableCell>
    </TableHead>
    {items.map(item => (
      <TableRow key={item.id}>
        <TableCell numeric>{item.id}</TableCell>
      </TableRow>
    ))}

</ReactCSSTransitionGroup>

it does not work due to the fact that I cannot control className prop of TableRow,  I see no changes in tr DOM elements class property (and maybe it does not work because it also requires (I don't know if it does) animated elements to be direct children. Not sure if this is the case, I never used ReactCSSTransitionGroup before)
If try to do it this way:

  <TableHead>
    <TableCell numeric>Id</TableCell>
  </TableHead>
  <ReactCSSTransitionGroup transitionName="example" transitionEnterTimeout={5000} transitionLeaveTimeout={3000}>
    {items.map(item => (
      <TableRow key={item.id}>
        <TableCell numeric>{item.id}</TableCell>
      </TableRow>
    ))}
  </ReactCSSTransitionGroup>

then there is no rows rendered due to the fact that TableRows are not direct children of Table now, so Table does not detect it and does not render.
What is the right way of doing this? Thanks!

Having the same problem. Would love to know, if there's a proper way to get transitions for table rows.

Have you guys figured this out?

Adding component="tbody" to ReactCSSTransitionGroup seems to be the suggested approach, I however still can't get animations working for table rows unfortunately.


CI will build in master

Before we start shipping we need to adopt a "CI in dev branch"->"merge to master branch from dev branch" strategy

Hi @xee5ch
Sorry for the extreme delay with this answer. We've been thinking of using GitHub Actions to do our CI/CD.
If you are still willing to take a crack at this please do so. There is an existing branch call add-github-actions which I wouldn't use. The branch is quite old and not worth working with IMO.


Make changes to type testing in table.py

Changes to type testing using the python_type attribute of a column to compare to native Python types and select correct agate type. I tried to run the tests, but I am having an as yet unresolved issue with pyasn1.

Hey Adrian,
So I'm staring at this and wondering if it's possible I didn't discover python_type in my hours of fiddling with this. It's either that or I tried it this way and ran into an issue. Regardless, if this works, it's a huge improvement! Thanks for taking a run at it!
One issue I think you're going to run into is float conversion. agate.Number intentionally breaks on floats, because they can't always be accurately converted to Decimal. So in that case we'll need to pre-convert them. (I'd certainly be open to modifying this agate behavior if you've got any ideas.)

On 10/29/2015 04:21 PM, Christopher Groskopf wrote:

Hey Adrian,
So I'm staring at this and wondering if it's possible I didn't discover
|python_type| in my hours of fiddling with this. It's either that or I
tried it this way and ran into an issue. Regardless, if this works, it's
a huge improvement! Thanks for taking a run at it!
One issue I think you're going to run into is |float| conversion.
|agate.Number| intentionally breaks on floats
https://github.com/onyxfish/agate/blob/master/agate/data_types/number.py#L66,
because they can't always be accurately converted to |Decimal|. So in
that case we'll need to pre-convert them. (I'd certainly be open to
modifying this agate behavior if you've got any ideas.)

How did you deal with it before?
Given :
In [1]: from sqlalchemy.types import *
In [2]: real_type = REAL()
In [3]: real_type.python_type
Out[3]: float
In [4]: float_type = FLOAT()
In [5]: float_type.python_type
Out[5]: float
Then:
if sql_type in [BIGINT, DECIMAL, FLOAT, INTEGER, NUMERIC, REAL, SMALLINT]:
column_types.append(agate.Number())
would have put floats in an agate.Number() field

—
Reply to this email directly or view it on GitHub
https://github.com/onyxfish/agate-sql/pull/5#issuecomment-152354991.

--
Adrian Klaver
<EMAIL_ADDRESS>

That is an excellent question. To be 100% honest I do not know why it works. My best guess is that I simply failed to test thoroughly. (The unit tests for this module are exceptionally thin.)

On 10/29/2015 05:11 PM, Christopher Groskopf wrote:

That is an excellent question. To be 100% honest I do not know why it
works. My best guess is that I simply failed to test thoroughly. (The
unit tests for this module are exceptionally thin.)

Part of the issue may be that Sqlite tends to be forgiving when it comes
to types:
http://www.sqlite.org/datatype3.html
2.0 Type Affinity
Each column in an SQLite 3 database is assigned one of the following
type affinities:
 TEXT
 NUMERIC
 INTEGER
 REAL
 BLOB

A column with NUMERIC affinity may contain values using all five storage
classes. When text data is inserted into a NUMERIC column, the storage
class of the text is converted to INTEGER or REAL (in order of
preference) if such conversion is lossless and reversible. For
conversions between TEXT and REAL storage classes, SQLite considers the
conversion to be lossless and reversible if the first 15 significant
decimal digits of the number are preserved. If the lossless conversion
of TEXT to INTEGER or REAL is not possible then the value is stored
using the TEXT storage class. No attempt is made to convert NULL or BLOB
values.
A string might look like a floating-point literal with a decimal point
and/or exponent notation but as long as the value can be expressed as an
integer, the NUMERIC affinity will convert it into an integer. Hence,
the string '3.0e+5' is stored in a column with NUMERIC affinity as the
integer 300000, not as the floating point value 300000.0.
A column that uses INTEGER affinity behaves the same as a column with
NUMERIC affinity. The difference between INTEGER and NUMERIC affinity is
only evident in a CAST expression.
A column with REAL affinity behaves like a column with NUMERIC affinity
except that it forces integer values into floating point representation.
(As an internal optimization, small floating point values with no
fractional component and stored in columns with REAL affinity are
written to disk as integers in order to take up less space and are
automatically converted back into floating point as the value is read
out. This optimization is completely invisible at the SQL level and can
only be detected by examining the raw bits of the database file.)

—
Reply to this email directly or view it on GitHub
https://github.com/onyxfish/agate-sql/pull/5#issuecomment-152362108.

--
Adrian Klaver
<EMAIL_ADDRESS>

That does explain the tests passing. Locally I've been testing against Postgres as well, though I'm not certain I've tested any floats. (My focus until now has been on agate core.)

On 10/29/2015 05:27 PM, Christopher Groskopf wrote:

That does explain the tests passing. Locally I've been testing against
Postgres as well, though I'm not certain I've tested any floats. (My
focus until now has been on agate core.)

The below looks interesting(asdecimal part) , just not sure yet where to
do the conversion yet.
http://docs.sqlalchemy.org/en/rel_1_0/core/type_basics.html
class sqlalchemy.types.Float(precision=None, asdecimal=False,
decimal_return_scale=None, **kwargs)
.....

—
Reply to this email directly or view it on GitHub
https://github.com/onyxfish/agate-sql/pull/5#issuecomment-152364256.

--
Adrian Klaver
<EMAIL_ADDRESS>

Good find! If that works universally that would make things tremendously easier.

Added code to deal with floats. So in Postgres:
test=> create table agate_test (id int, fl_fld float);
CREATE TABLE
test=> insert into agate_test values(1, 2.35);
INSERT 0 1
test=> insert into agate_test values(2, 1.333);
INSERT 0 1
test=> insert into agate_test values(3, 5.698);
INSERT 0 1
new_table = agate.Table.from_sql('postgresql://aklaver:@localhost:5432/test', 'agate_test')
In [15]: new_table.rows[0][1]
Out[15]: Decimal('2.3500000000')
In [16]: new_table.rows[1][1]
Out[16]: Decimal('1.3330000000')
In [17]: new_table.rows[2][1]
Out[17]: Decimal('5.6980000000')

Just saw test fail. Forgot  about Python 3. How are you dealing with 2/3 conversions?

I'm not 100% sure I know what you are asking, but we do want 2/3 compatibility for this module. You're welcome to add a dependency on six (agate already depends on it). The string check should probably use six.string_types.

Oh and there is a tox config already in the repo for testing against the 6 versions of Python agate supports.

On 10/30/2015 09:19 AM, Christopher Groskopf wrote:

I'm not 100% sure I know what you are asking, but we do want 2/3
compatibility for this module. You're welcome to add a dependency on six
https://pythonhosted.org/six/ (agate already depends on it). The
string check should probably use |six.string_types|.

Thanks, that is what I am looking for. I have just started rewriting my
own code to be 2/3 compatible and the thing I have found is that there
is a multitude of ways to do that. I did not want to introduce another
method to this code.

—
Reply to this email directly or view it on GitHub
https://github.com/onyxfish/agate-sql/pull/5#issuecomment-152572788.

--
Adrian Klaver
<EMAIL_ADDRESS>

Much appreciated! agate is my third or fourth library that's 2/3 compatible so lemme know if you run into any issues.

Add code to deal with string differences between Python 2 and 3. If code is being used under Python 2 convert str and unicode types into basestring. This allows test against six.string_types to work in the if/elif/else type checking section. Tested locally against Python 2 and 3.

Actually, there is a cleaner way to handle this:
elif py_type is six.string_types:

Note is, not in.All string types are subclasses of six.string_types regardless of which version of Python your in. So you don't need the if six.PY2 block at all.

Adrian, I went ahead and did some very minor cleanup and merged this in. Thanks again for hacking on it! I've added you to the AUTHORS file. Feel free to pick up any other tickets that interest you!
(I was wrong in my previous comment, since its a type and not an instance you have to use issubclass, but you can still avoid the if block.)

On 10/30/2015 05:36 PM, Christopher Groskopf wrote:

Adrian, I went ahead and did some very minor cleanup and merged this in.
Thanks again for hacking on it! I've added you to the AUTHORS file. Feel
free to pick up any other tickets that interest you!

Thanks. The issubclass test is a lot cleaner then my solution. I can
take a swing at #2. I see the TimeDelta data type in agate and the
commented out entry in SQL_TYPE_MAP. SQLAlchemy has the INTERVAL type
that outputs datetime.timedelta. Is it just a matter of hooking the
types together or is there something I am missing?

(I was wrong in my previous comment, since its a type and not an
instance you have to use |issubclass|, but you can still avoid the |if|
block.)
—
Reply to this email directly or view it on GitHub
https://github.com/onyxfish/agate-sql/pull/5#issuecomment-152680150.

--
Adrian Klaver
<EMAIL_ADDRESS>

I think it's just a matter of wiring it together, but I'm honestly not sure. I didn't get as far as figuring out what databases actually generate that type.

I will move the TimeDelta discussion over to #2.


Adding kotlin extension for basic operators

Signed-off-by: fbrunacci<EMAIL_ADDRESS>What changes were proposed in this pull request?
Adding kotlin extension functions on basic operations to make symbolic representation (like + or *) available.
How was this patch tested?
unit tests:
nd4k/src/test/kotlin/org/nd4k/INDArrayExtTest.kt

I'm against doing it this way. If we want to support the syntactic sugar for Kotlin, those few functions should be added directly on INDArray and SDVariable.
Overall however, I'm not sure if the syntactic sugar is worth the inconsistency across languages. It will create additional confusion for the benefit of being able to use +,-, /, * instead of add, sub, div, mul, and if you want to use the inplace versions to improve the performance of the implementation where possible you now get a weird usage mix of syntactically sugared and unsugared methods.

For Scala we offer those though
With best regards,
raver119
12 мая 2020 г., 22:57 +0300, Paul Dubs<EMAIL_ADDRESS>писал:

I'm against doing it this way. If we want to support the syntactic sugar for Kotlin, those few functions should be added directly on INDArray and SDVariable.
Overall however, I'm not sure if the syntactic sugar is worth the inconsistency across languages. It will create additional confusion for the benefit of being able to use +,-, /, * instead of add, sub, div, mul, and if you want to use the inplace versions to improve the performance of the implementation where possible you now get a weird usage mix of syntactically sugared and unsugared methods.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I proposed this pull request because I am currently writing notebooks (jupyter / kotlin) and I needed to make the code more readable. (for presentations / courses)
In one of my kotlin notebooks (inspired by a python notebook), it allowed me to pass from this code
// forward
val layer_1 = layer_0.mmul(weights_0_1)
val layer_2 = sigmoid(layer_1.mmul(weights_1_2))

// back propagation
val layer_2_error = layer_2.sub(getTargetForLabel(label))
val layer_2_delta = layer_2_error.mul(sigmoidDerivative(layer_2))

val layer_1_error = layer_2_delta.mmul(weights_1_2.transpose())
// layer_1_delta = layer_1_error

val diff_1_2 = layer_1.transpose().mmul(layer_2_delta)
weights_1_2.subi(diff_1_2.mul(learning_rate))
val diff_0_1 = layer_0.transpose().mmul(layer_1_error)
weights_0_1.subi(diff_0_1.mul(learning_rate))

to this code
import org.nd4k.*

// forward
val layer_1 = layer_0.dot(weights_0_1)
val layer_2 = sigmoid(layer_1.dot(weights_1_2))

// back propagation
val layer_2_error = layer_2 - getTargetForLabel(label)
val layer_2_delta = layer_2_error * sigmoidDerivative(layer_2)

val layer_1_error = layer_2_delta.dot(weights_1_2.T())
val layer_1_delta = layer_1_error

weights_1_2 -= layer_1.T().dot(layer_2_delta) * learning_rate
weights_0_1 -= layer_0.T().dot(layer_1_delta) * learning_rate

and here is the same portion of code in python / numpy (they also offer this sugar)
# forward
layer_1 = self.layer_0.dot(self.weights_0_1)
layer_2 = self.sigmoid(layer_1.dot(self.weights_1_2))

# Output layer
layer_2_error = layer_2 - self.get_target_for_label(label)
layer_2_delta = layer_2_error * self.sigmoid_output_2_derivative(layer_2)

layer_1_error = layer_2_delta.dot(self.weights_1_2.T)
layer_1_delta = layer_1_error

self.weights_1_2 -= layer_1.T.dot(layer_2_delta) * self.learning_rate
self.weights_0_1 -= self.layer_0.T.dot(layer_1_delta) * self.learning_rate

Moreover, the fact of making this writing available via a kotlin extension allows you to choose whether or not to use it (via the import of com.nd4k. * in koltin) and allows the user to choose whether he wishes to use this sugar or no.
Which seems less possible if we integrate it directly into INDArray.

Yep, we also do that in C++.

@fbricon overall looks good as a start. Can you add +=,-=,.. for the inplace operators (anything with an i at the end)?

@agibsoncc you got the wrong @fbrunacci :-)

@fbricon overall looks good as a start. Can you add +=,-=,.. for the inplace operators (anything with an i at the end)?
@agibsonccc Inplace operators are already define because arithmetic operators are used by kotlin to generate augmented assignments.
https://kotlinlang.org/docs/reference/operator-overloading.html#assignments

I have already used some of them in my projects. (example here: https://github.com/fbrunacci/kotlin-notebook-samples/blob/1a8a4ead8c5d0706291ab0e011958f6eb22a10ba/notebooks/deeplearning4j/SentimentClassificationDemo.ipynb )

Closing this. We're not going to add any jvm lang specific modules in the mainline project due to maintenance cost increase. (We recently retried all of our scala syntax enhancements) I'll be happy to put something like this  in our new contrib folder.


🚩: Final Milestone: React Capstone Project

🚩: Final Milestone: React Capstone Project - React Redux Toolkit
Hi there! @Ruthmy here.
In this project I have developed a Weather Metrics App using the Current Conditions API from AccuWeather. The app is called Weather App and it is a single page application (SPA) that allows users to check various weather metrics for different locations.

The app has been deployed to GitHub Pages and you can find it here.
To get the data for the weather, I have used the Current Conditions API from AccuWeather.

Website Goals:
The goals of this project are to:

Create a mobile web application to display weather metrics.
Implement a user-friendly interface with a list of weather metrics.
Allow users to filter weather metrics by location.
Provide detailed information for each weather metric.
Apply a visually appealing design based on the provided design guidelines.

Project Goals:

[x] The project is developed as a single-page application (SPA) using React and Redux.
[x] Data retrieved from the AccuWeather API is stored in the Redux store.
[x] Utilize React Router for efficient navigation.
[x] Each page has a unique route within the SPA.
[x] Implement React components to create a seamless user experience.
[x] Integrate Redux to manage state and data retrieval from the API.
[x] Write unit tests have been implemented to ensure the functionality and reliability of the application.
[x] Deploy the application online using GitHub Pages.
[x] Record a 5-minute video demonstrating the app's features and code highlights.
[x] Follow the GitFlow as a workflow for version control.
[x] Follow the best practices for HTML & CSS and JavaScript.
[x] Documented the work in a professional way.

Presentation Video - JavaScript Capstone

Thank you for taking the time to review this PR ⭐!

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

Hi @topeogunleye  I really appreciate your time in reviewing my code and the feedback you have given me.
Even so I need to add that, I already implemented this changes you asked for first, to include a filter or search by categories. So I implemented a search bar.
But almost an hour later you invalidate the review by GitFow.
Anyway, I reread the project requirements and the GitFlow guide, and yes, my PR is the merge from dev to main.
I took a few minutes to check the profiles of other colleagues, for example this one, and they are approved and under the same workflow, several feature branches with merge in dev and at the end with a merge at the end from dev to main.

But almost

Hi @topeogunleye I really appreciate your time in reviewing my code and the feedback you have given me.
Even so I need to add that, I already implemented this changes you asked for first, to include a filter or search by categories. So I implemented a search bar. But almost an hour later you invalidate the review by GitFow.
Anyway, I reread the project requirements and the GitFlow guide, and yes, my PR is the merge from dev to main. I took a few minutes to check the profiles of other colleagues, for example this one, and they are approved and under the same workflow, several feature branches with merge in dev and at the end with a merge at the end from dev to main.
I would be very grateful if you could please tell me what happened.

@Ruthmy I asked my Senior Code Reviewers and they suggested that I invalidate. Congratulations on the approval. Congratulations on your Approval 🎊


[Request] Light theme

I have astigmatism and find it easier to read on a light theme (light background with darker fonts).
Could a toggle be implemented so we may switch from the dark to the light theme?

Thank you for the feature request!
I would love to support a theme toggle with light theme, but realistically do not have the time to implement it right now.
If someone reads this, it would be great if someone can get started on this in a PR, I would be happy to collaborate to get it to completion.
Preferably if someone has experience setting up tailwind themes in a good way.


Uncaught Exception in DefaultAzureCredential

Package Name: azure-identity
Package Version: 1.13.0
Operating System: AzureML Instance: Ubuntu 20.04
Python Version: 3.8

Describe the bug
When using DefaultAzureCredential, there is an exception coming from ManagedIdentityClient that causes DefaultAzureCredential to stop trying different methods and ends up failing to get credentials.
I can work around this by using AzureCliCredential (or one of the other credential methods) but I cannot use this workaround when using other packages like mltable or azure-fsspec since they seem to run their own credential routine.
To Reproduce
Steps to reproduce the behavior:

I don't know what it is about my Managed Identity setup that causes this, but you need to trigger the raise here.
The ClientAuthenticationError makes it back to this try/except where it is caught as an Exception and the for loop is broken.

Expected behavior
DefaultAzureCredential loops though all selected credential routines until it receives a token.
Screenshots
Only 2 credentials attempted.

Additional context
I don't seem to have this problem from my local computer but I have to work on AzureML instances and that is where this pops up.
There is likely a deeper problem here but I found that specifically catching the ClientAuthenticationError allows the routine to proceed and get a proper credential.

Thank you for the feedback @PhysWiz314 . We will investigate and get back to you asap.

Thanks for reaching out.
Could you share more information?
Is this a machine that managed by machine learning studio?
Did you configure managed identity on that machine?
Thanks.

Thanks for reaching out.
Could you share more information?
Is this a machine that managed by machine learning studio?
Did you configure managed identity on that machine?
Thanks.

Yes, this is an instance that is set up inside Machine Learning Studio. There is no Managed Identity set up on the machine. To give a bit more context on why it works on my laptop, here is the history of attempts to find a credential. For some reason the error code for ManagedIdentityCredential is different and properly caught by the loop in get_token.

Thanks for the information.
It is by design that DefaultAzureCredential continues to attempt other credentials if managed identity is not configured and stops if managed identity is configured but we fail to get a valid token.
It is a known issue that managed identity in machine learning studio does not work correctly with identity library.

@xiangyan99 Thanks for letting me know that it is a known issue. Is there a fix coming for this? We are upgrading our internal packages to Azure SDK v2 but this is a huge blocker since mltable and azure-fsspec can't be used to load data so we can work with it on our ML studio instances.

To help me understand better, do you want to use managed identity for the authentication?
If not, you can either not configure managed identity for those hosted machines or use DefaultAzureCredential(exclude_managed_identity_credential=true) to opt out the managed identity in your code.

No I don't want to use managed identity for authentication and yes I can use the exclude_managed_identity_credential for my own code. Unfortunately this does not work for packages like mltable or azure-fsspec.
Here is the situation that led me to discover this issue.
I would like to load a data asset that is registered in my ML workspace. I was following this example in the documentation.
The issue is that mltable and azure-fsspec don't seem to use the token/credentials that I have already loaded and try to get a new token (pictured below) which runs into the issue I described.

Small Update: I looked into mltable and azure-fsspec and they hide the call to DefaultAzureCredential pretty deep within private functions making it essentially impossible for a user that is having this problem to use them or exclude credential methods with flags.

and stops if managed identity is configured but we fail to get a valid token

Wouldn't it be a better idea to throw a warning (so that the user knows that ManagedIdentity wasn't used) instead of stopping and continue to try other credential methods? This would mitigate the issue described here but still let users know there is an issue with their ManagedIdentity setup if something goes wrong.
In my case, Managed Identity is not set up but the code behaves as if it is so I know I can safely ignore the warning.

/unresolve
As it turns out, mlflow also uses DefaultAzureCredential() to authenticate which again fails when it tries to use ManagedIdentityCredential to get a token. Again, I cannot provide arguments to this call to exclude ManagedIdentityCredential so I am now unable to use mlflow, azure-fsspec and mltable which are key components of being able to run code on an AzureML workspace compute instance. Is there any way to get some help with this?

It is a known issue that managed identity in machine learning studio does not work correctly with identity library.

Again, is this being worked on? Is there a way for me to track progress (i.e. a bug report, issue in GitHub etc)?
Another very simple solution would be to move ManagedIdentityCredential to the bottom of the list of authentication methods to try. That way it is still tried but the user would be able to use another method to authenticate properly.

@xiangyan99 Can you reference any open issues regarding allowing users to specifiy their own credentials for the ML SDKs? I feel like something like credential instances should definitely be provided by the user and not instantiated from within the SDK itself.
When DefaultAzureCredential should continue with the authentication flow versus when it should terminate the flow has been a bit of debatable topic. The current prevailing sentiment is that for deployed service credentials (such as EnvironmentCredential and ManagedIdentityCredential), if the prerequisites for a credential are met and an attempt to retrieve an access token is made, if the token request to Azure Active Directory fails to yield a token, then the flow should stop with a thrown exception. This is to provide a more predicatable deployed behavior without any surprise authentications with developer credentials.
Overall, I think the solution to this problem needs to be solved in the libraries like mltable itself.
In the meantime, I think one way of circumventing the use of ManagedIdentityCredential is to remove "prerequisite" envvars that are checked to determine if a specific credential should be used. For Azure ML, MSI_ENDPOINT and MSI_SECRET are checked. Removing MSI_ENDPOINT from the python envvars might be able to do the trick assuming it persists through the usage for where mltable, mlflow, etc. are being called. Worth a shot, I think.
import os
from azure.identity import DefaultAzureCredential

os.environ.pop('MSI_ENDPOINT', None)

credential = DefaultAzureCredential()
credential.get_token("https://management.azure.com/.default")

Since the issues are in mlflow, azure-fsspec, @luigiw do you know who is the best person to take a look?

Thanks @pvaneck for the detailed response!

The current prevailing sentiment is that for deployed service credentials (such as EnvironmentCredential and ManagedIdentityCredential), if the prerequisites for a credential are met and an attempt to retrieve an access token is made, if the token request to Azure Active Directory fails to yield a token, then the flow should stop with a thrown exception. This is to provide a more predicatable deployed behavior without any surprise authentications with developer credentials.

This makes sense.

Overall, I think the solution to this problem needs to be solved in the libraries like mltable itself.

I agree with this sentiment. Ideally users would be able to provide the credential to the library or at the very least arguments that would enable/disable credential methods that the DefaultAzureCredential class sets up. I plan to open up issues/feature requests to see if this can be done. I was hoping that the issue could easily be solved here instead of having each individual library have to change their use.
I tried out your workaround and that works well. That was going to be my next line of thinking so you have saved me a bunch of time trying to track something like that down. Thanks!

FYI I did end up trying to set up a Managed Identity to get that authentication method to work. Unfortunately this results in the same error when using a Compute Instance in an ML Workspace.
I was able to trace the issue to the AZURE_CLIENT_ID environment variable not being set in the instances (it seems to be held in DEFAULT_IDENTITY_CLIENT_ID instead). This means that the Managed Identity doesn't have a client_id to work with and when the request for a token is made, a 404 is returned which isn't caught and ends with the "No token returned" error.
I confirmed that this was the issue by adding a managed_identity_client_id argument to DefaultAzureCredential. This adds the required query parameter to the URL and I could successfully get a token.

Next, I tried to set the AZURE_CLIENT_ID environment variable so that the ManagedIdentityCredential could use it, but ran into issues with Environment checks. Ultimately DefaultAzureCredential was throwing errors trying to use EnvironmentCredential instead of the ManagedIdentityCredential. I'm not sure if this is known or not but figured I would share what I found in case it is useful.
The only think I haven't tested is if this is the same cause for the "No token returned" error when I didn't have a Managed Identity set up. I'll have to wait for my IT team to remove the Managed Identity from my compute instance before I can test that properly.

Thanks for the additional details! Didn't know about the DEFAULT_IDENTITY_CLIENT_ID envvar. I will need to investigate to see if we should try checking for that variable inside our ManagedIdentityCredential as a fallback for client_id or perhaps instead try to have AZURE_CLIENT_ID be populated in the ML Compute instance by default.
When you had set the AZURE_CLIENT_ID envvar yourself, what error were you receiving from DefaultAzureCredential? From what I understand it should skip EnvironmentCredential if no complete set of environment variables is found (reference).

I am not too familiar with Azure ML, but trying on a newly created Azure ML compute instance, I was not able to use ManagedIdentityCredential (getting the same 404 and "no token received") until I found and clicked this "Authenticate" button,

Then I was able to authenticate using DefaultAzureCredential without setting AZURE_CLIENT_ID.
DEBUG:urllib3.connectionpool:http://<IP_ADDRESS>:46808 "GET /MSI/auth?api-version=2017-09-01&resource=https://storage.azure.com HTTP/1.1" 200 None
INFO:azure.core.pipeline.policies.http_logging_policy:Response status: 200
Response headers:
    'Content-Type': 'application/json'
    'Date': 'Fri, 18 Aug 2023 23:03:17 GMT'
    'Transfer-Encoding': 'chunked'
<azure.core.pipeline.PipelineResponse object at 0x7f8fefb42b20>
DEBUG:msal.token_cache:event={
    "data": {},
    "response": {
        "access_token": "********",
        "expires_in": 1508,
        "expires_on":<PHONE_NUMBER>,
        "resource": "https://storage.azure.com",
        "token_type": "Bearer"
    },
    "scope": [
        "https://storage.azure.com"
    ]
}
DEBUG:azure.identity._internal.get_token_mixin:AzureMLCredential.get_token succeeded
DEBUG:azure.identity._internal.decorators:ManagedIdentityCredential.get_token succeeded

Not sure what this  "Authenticate" step specifically, but seems like it did some extra setup to allow the /MSI endpoint to work.

Not sure what this "Authenticate" step specifically, but seems like it did some extra setup to allow the /MSI endpoint to work.

Hey @pvaneck I finally had a chance to try this out. I had a bit of an odd experience since the "Authenticate" button seemed to fail. I shutdown and re-started my instance and the message to Authenticate disappeared and the credentials started working.
I had a colleague reproduce our steps and they had the same experience as you. No Token before Authenticate but works fine after so I think it is a good work around.
I typically connect to my instance through VSCode so I had never seen the Authenticate button before. It would be nice to know what steps that runs so that it can be duplicated in terminal or code so that users aren't forced to log into the portal if they otherwise don't need to.

When you had set the AZURE_CLIENT_ID envvar yourself, what error were you receiving from DefaultAzureCredential? From what I understand it should skip EnvironmentCredential if no complete set of environment variables is found

Since I've Authenticated using your method, I can't seem to reproduce the Environment Variable error I was getting. I'm not sure how often the authentication has to be re-done so if I see it needing to be done again, I'll try it out and post the error here.
Thanks for all of your help with this!

Thanks, @PhysWiz314
@luigiw @azureml-github Is there any documentation available that has information about what exactly that "Authenticate" button in the portal does and if it's possible to replicate it programmatically?

I am facing a similar issue with DefaultAzureCredential failing when ManagedIdentityCredential is not disabled via exclude_managed_identity_credential=True. For me, the workaround of removing MSI_ENDPOINT and MSI_SECRETE from environment variables did not work. Similar to @PhysWiz314, I need to use azureml-fsspec which does not allow me to configure DefaultAzureCredential to disable ManagedIdentityCredential.
More context: the job is running in a Linux HPC development environment without any domain links. I use azure-cli for authentication.
Are there other workarounds that would allow me to disable ManagedIdentityCredential without needing to pass any arguments to DefaultAzureCredential and that would propagate to other tools like azureml-fsspec?

Hey, apologies all for the late loop back on this. I'll try to give a write-up based on what I have found from my testing with the current state of Azure ML.
Regarding managed identity endpoint behavior in Azure ML
The behavior of the managed identity endpoint in an Azure ML compute instance relies on the instance's configuration. The compute instance can be configured to use a system-assigned managed or user assigned identities, but not both. Also, by default, SSO is enabled for Azure ML compute instances.
These are the following scenarios when no client ID is provided to the ManagedIdentityCredential/DefaultAzureCredential (MIC/DAC):

SSO is enabled and managed identity is enabled: MIC/DAC will authenticate the user identity (logged-in user).
SSO is enabled and managed identity is disabled: MIC/DAC will authenticate the user identity (logged-in user).
SSO is disabled and managed identity is enabled: MIC/DAC will authenticate the managed identity.
SSO is disabled and managed identity is disabled: I don't believe this is possible, as you need to provision with one or the other.

In essence, if SSO is enabled, and a user wants to authenticate with the managed identity, the user needs to pass in the app/client ID of the managed identity which will be stored in the DEFAULT_IDENTITY_CLIENT_ID envvar.
cred = DefaultAzureCredential(managed_identity_client_id=os.environ.get("DEFAULT_IDENTITY_CLIENT_ID"))

or
cred = ManagedIdentityCredential(client_id=os.environ.get("DEFAULT_IDENTITY_CLIENT_ID"))

Dealing with DAC instances you don't have control over
However, since some of these ML libraries (mltable, azure-fsspec, etc.) have embedded DACs that users can't customize or configure, users can't pass in the client ID of the managed identity. If the MI envvars are set, and we try and fail to use them, then it's intentional for it to halt the chain (as mentioned in the above comment). I don't think we are changing that pattern anytime soon. In this case, depending on what the user wants to do, they can either:

Set the AZURE_CLIENT_ID envvar to the client ID of the managed identity. This will allow the libraries to authenticate with the managed identity. os.environ["AZURE_CLIENT_ID"] = os.environ.get("DEFAULT_IDENTITY_CLIENT_ID", "")
Force DefaultAzureCredential to skip ManagedIdentityCredential by deleting some environment variables that are used for checking the presence of a managed identity environment.

os.environ.pop("MSI_ENDPOINT", None)

If your environment also has access to the Azure IMDS endpoint (http://<IP_ADDRESS>), you'll also need to force that check to fail to prevent ManagedIdentityCredential from being used with IMDS. This can be done by setting the following environment variable to some value that will fail when a request is sent.
os.environ["AZURE_POD_IDENTITY_AUTHORITY_HOST"] = "FakeAuthorityHost"

Conclusion
If SSO is enabled, then the user identity will be used in the default case where nothing is specified. This setting can be checked in the compute settings, like in the image below. Role assignments to resources should be assigned to the user.

Workarounds and approaches for using the managed identity, if desired, are outlined above. However, ultimately, I feel credential customization should be something that is enabled by the ML libraries themselves. I'm unaware of how/where mltable and azure-fsspec are maintained, and if a mechanism for allowing user-supplied credentials is something that is in the works.
If credential customization options are something that is still needed/desired, I encourage you or anyone else to open a new, targeted issue requesting this feature with specifics of the library being used.

Thanks a lot for this write up @pvaneck! I appreciate the response and work that went into figuring this issue out.


[0.36.0] v8 : Maximum call stack size exceeded

I'm trying to build a native node module written in v8 and use it in electron.
The main goal of this module is to call a C function and stream all the output back to the JS.
Unfortunately I'm getting the following error: "Maximum call stack size exceeded".
More info here: http://stackoverflow.com/questions/34167060/v8-maximum-call-stack-size-exceeded
After further investigation, the problem seems to be related to electron.
Do you guys have any clue on this ?
Thanks
EDIT
Using the following code, I determined that the max size within electron is 17753
function computeMaxCallStackSize(size) {
    size = size || 1;
    return computeMaxCallStackSize(size + 1);
}
computeMaxCallStackSize();

I've tried to override it with electron src --js-flags="--stack-size=90000" but problem still occurs.

Issue can be deleted.
Thanks


Investigate Doc Generation

https://github.com/aws/aws-sdk-go/tree/master/apis provides a JSON structure with all the documentation from AWS for each action. If it's possible to easily generate the documentation from this that might be a good idea.

Gonna close this as generation is alive and well in a branch.


[EVENT Change Request] report 5880 "Calc. Phys. Invt. Order Lines" -  OnCreateNewPhysInvtOrderLineOnAfterCalcQtyAndLastItemLedgExpected

Could you please include the sender on the event OnCreateNewPhysInvtOrderLineOnAfterCalcQtyAndLastItemLedgExpected
//+Change
//-[IntegrationEvent(false, false)]
[IntegrationEvent(true , false)]
//-Change
local procedure OnCreateNewPhysInvtOrderLineOnAfterCalcQtyAndLastItemLedgExpected(QtyExpected: Decimal; LastItemLedgEntryNo: Integer; ItemLedgerEntry: Record "Item Ledger Entry"; PhysInvtOrderLine: Record "Phys. Invt. Order Line"; var InsertLine: Boolean; WarehouseEntry: Record "Warehouse Entry")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 43300.


Consider secret early bird tickets as sold out

We need to have tickets available for companies paying by invoice but no more available publicly.
We can switch them to secret tickets so they should be considered the same as soldout.

This should be closed automatically by merged PR.

@frenzymadness yeah, I used one word that doesn't do that by mistake 🤦♂
close
closes
closed
fix
fixes
fixed
resolve
resolves
resolved


Faster lookup

This PR depends on:

https://github.com/ignitionrobotics/ign-math/pull/406

It supersedes #188 and only focuses on the lookup thus making the reviewers life easier.
It implements a fast lookup for grid data like the data provided by MBARI. It also adds an integration test for the science data. Currently we are only checking data that does not vary with time. I have removed all interpolation code from this PR to keep it clean. I will be adding it back in the next PR. On my Laptop's Ryzen 9 5900HX this code takes between 3-15 microseconds per lookup.

I'll re-review when we merge #191 in.


Azure: Use random string rather than infraID as storage account name

The storage account is in the resource group so its name doesn't really
matter for cleanup. It has however some limitations wrt format, it must
be max 24 characters long and lower alphanumeric. This broke when we
added defaulting for the infraids, which also introduced a new format
that is too long and contains hyphens. This leads to failure:
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_release/26666/rehearse-26666-periodic-ci-openshift-hypershift-main-periodics-e2e-conformance-azure/1499088629878755328#1:build-log.txt%3A79
What we do here matches what the installer does:
https://github.com/openshift/installer/blob/8fca1ade5b096d9b2cd312c4599881d099439288/data/data/azure/vnet/main.tf#L23

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.
/cc @csrwng @sjenning

/lgtm


feat(overlay): overlay rtl support

Blocked on #1591 (needs rebase). Adds RTL support to overlay ref and position strategy.  APIs need discussion.
r: @jelbourn

LGTM


Tag suggestions

It would ease user's life if the app would suggest tags and user could just apply them.
This would come in 2 forms:

"Quick tags" section in Gallery's tags edition dialog.
Right now, "quick tags" don't contain resource-specific tags, only collection-specific (the most popular tags).
We can also add "suggestions" into "quick tags".
Resource Grid screen can have pop-up dialog, displayed right after indexing is over.
The pop-up would list unlabeled resources with "suggestions" next to them.
User can approve or ignore them on per-tag basis.

"Suggestions" can be generated by similarities found.
Image-based similarity search would consume too much power for mobile device, so this criteria is left for future Desktop/Server implementations. Lightweight search would include the following parameters:

parts of filename, Levenstein distances
EXIF-metadata of images
text content of documents

Requires better specification

@sisco0 we also can use something like dssim in addition to neural networks (like cnnembed)


add CI/CD (Github Actions) tests for Windows and MacOS

Currently we only run out rust tests on Linux. Would be good to also add windows and MacOS,
to ensure cross platform compatibility way, in at the very least this basic manner.
Targets to be run on multiple platforms: check/test + integration tests.
Can be added to the existing CI.yml file.

Examples and all other tests now run also on MacOS and Windows.


Add kylin datasource.

Add kylin datasource support (combined with this repo as kylin orm: https://github.com/rocky1001/pykylin/tree/caravel-kylin)
Because 'timestamp' is the sql keywords that kylin not supported, use '_timestamp' as the label when using kylin datasource.

Can you rebase? There are new hooks to customize for specific backends
https://github.com/airbnb/caravel/blob/master/caravel/models.py#L440

Please re-open a PR that takes into account recent code changes.

Ok, I will try to add Apache Kylin backends with the latest code and
re-open a PR,
Thank you.
2016-06-14 0:38 GMT+08:00 Maxime Beauchemin<EMAIL_ADDRESS>
Please re-open a PR that takes into account recent code changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/airbnb/caravel/pull/605#issuecomment-225637118, or mute
the thread
https://github.com/notifications/unsubscribe/ADIR7NVNK3O0le8oeBagp_Gn6auErrltks5qLYeVgaJpZM4Iz7r2
.

Any progress? @rocky1001

Any progress? @rocky1001


Kata voting is not always guaranteed that vote of one account is counted not more than once

Describe the bug
This kata is only completed by one user (me) but has 2 'Somewhat Satisfied' votes
To Reproduce
So i just finished it at about 12.29 12:10+0800 and tried to gave a 'Somewhat Satisfied' vote
But accidentally clicked on 'Very' just after clicking 'Somewhat' button before it responsed
After it got responsed, i refereshed the page and seeing

Where the counter shows 75% of 2 instead of xx% of 1
And if i switch to 'Somewhat' vote

I thought it was just accidentally count the vote twice so i toggled off the vote and got

When i re-vote again, it still reports 2 votes
To clearify, im the only one completed that kata and there were no votes before i voted (apparently)

In conclusion, a phantom vote is created by such operations.

I think I've seen this happen before. Kata voting has some black magic that I don't fully understand yet.
I've fixed the data for this one for now. I'll look into the cause.
The main problem is that the actual votes are stored separately and it should update the value on kata when changed, but sometimes it fails to for various reasons.

The screenshots shown show 2 completions (the circles icon). Total Code Submissions may only say 1 because the author's doesn't count.
Authors shouldn't be able to vote on their own kata, thats a bug (or at least a design flaw)

The vote wasn't by the author.
The only user with vote was @ZEDCWT and his vote was counted twice. It was "stuck" due to race condition (second vote was added while the first one was still being processed).
I think this is what happened because I was able to fix the data by simply making it recalculate.


5.7.0 BizTalk 2016 Visual Studio 2015 Properties for MSI Installer

I'm trying to figure how to install the BizTalk Deployment Framework 5.7.0 from commandline for Visual Studio 2015. but I have no idea how to do it.
Ideally something like
msiexec /a /visualstudio2015  DeploymentFrameworkForBizTalkV5_7.msi

If i run
msiexec /a DeploymentFrameworkForBizTalkV5_7.msi

the installer will not install anything
Though I've found the following properties; but I have no idea how to use any of them except for ALLUSERS.
According to the log, I can create using the command
msiexec /lp! msilog.log /i DeploymentFrameworkForBizTalkV5_7.msi

most of these properties are environmental properties and has nothing to do with initializing the installation of a given Visual Studio setup.
ALLUSERS
BTS_PRODUCTNAME
ESBTOOLKIT21_DIR
ESBTOOLKIT22_DIR
ESBTOOLKIT23_DIR
ESBTOOLKIT24_DIR
VS2010_IDE_VCSHARP_PROJECTSYSTEM_INSTALLED
VS2010_SCHEMAS_DIR
VS2010DEVENV
VS2012_EXTENSIONS_DIR
VS2012_IDE_VCSHARP_PROJECTSYSTEM_INSTALLED
VS2012_SCHEMAS_DIR
VS2012DEVENV
VS2013_EXTENSIONS_DIR
VS2013_IDE_VCSHARP_PROJECTSYSTEM_INSTALLED
VS2013_SCHEMAS_DIR
VS2013DEVENV
VS2015_EXTENSIONS_DIR
VS2015_IDE_VCSHARP_PROJECTSYSTEM_INSTALLED
VS2015_SCHEMAS_DIR
VS2015DEVENV

Yep, that's correct. It's a Developer machine with Visual Studio 2015.
I'll uninstall the msi and see what happens when I run the command :)

Tried
msiexec /i DeploymentFrameworkForBizTalkV5_7.msi ADDLOCAL=RuntimeFeature,DevToolsFeature,VS2015IntegrationFeature

Thought the installer wouldn't pop up but, it came up anyway
I think you need to add /quiet for it to be completely silent.

Did the second one with /quiet work as expected?

Absolutely :)


Unit test support in non-English countries.

Hello.
Some of unit tests don't work in my area.

So we added the function to set the default culture.
However, this may not be the best as not all test classes inherit AbstractTestFixture.
Are there any other good ideas?

I think if you added it into the static constructor, it should also work https://github.com/OmniSharp/omnisharp-roslyn/blob/master/tests/TestUtility/SharedOmniSharpHostFixture.cs

Thank you for the advice.


pkg/trace/stats: ensure stats payloads container Container{ID,Tags}

Changes

When agent hostname CLI outputs success and an empty hostname, the trace-agent will use it now (as empty).
ClientStatsPayload now contains a new Tags field. In Fargate environments and when the fargate_stats feature flag is enabled, this field will be populated by the tagger service using the client send Datadog-Container-ID HTTP header.

TODO:

[ ] Unit & Integration tests (for client-computed and agent-computed)

Superseded by https://github.com/DataDog/datadog-agent/pull/8383


Admin drag and drop image

Does not work, drag and drop when selecting multiple files when creating the product:
https://vimeo.com/189140941

This is a feature, you can't upload more than one image anyway.
If it would be an issue, you would have to report on django-filer issues.

I'm going to close this therefore.


Driver has problem with view referencing files in the same folder as the database

Considering that duckdb has not reach 1.0 yet and build-to-build there are database version incompatibilities.
Also, I would like to incrementally add to my database month-by-month without having to re-create the database and restart the metabase container to release the lock on the duckdb database file.

My data file is named raw_state.parquet files and is located in the same folder/volume as the duckdb database.
In the duckdb database I created a view: create view raw_state as select * from 'raw_state.parquet'; which works fine when querying it in duckdb.exe (eventually I'd change this to query across multiple raw_state files, not just one.)
I can connect to the duckdb database in Metabase as ID_PARQUET without any reported problems, and it appears to work with the sync notifications appear in Metabase
In metabase when I go to Browse data > ID_PARQUET, I can see the Raw State view - so far so good

However, when I open the Raw State view, I get the following error:
IO Error: No files found that match the pattern "raw_state.parquet"
I'm guessing duckdb is looking for the raw_state.parquet file in the same folder where the metabase.jar is running, not the folder where the duckdb database is located.
I tried specifying the full path to the raw_state.parquet file when creating the view on my local machine, but I don't have a /home/data/duckdb folder so duckdb.exe complains.
Any thoughts on how we might get the view query to run and look for the files in a path relative to the duckdb database?

I modified a bit the driver and now you can run without errors the SQL 'write' queries (like CREAT TABLE and etc), but Metabase engine sets DB connection to readOnly, so all of your changes will not be persistent. All you can do now is to create a view in each query session. For example:
create view raw_state as select * from '/app/database/*.parquet';
select * from raw_state;
...

But, I found a workaround how to do exactly what you want. For example, you mounted the 'my.duckdb' database folder into /app/database Metabase container folder. Let's pretend that all of you *.parquet files in the same folder.

Restart the docker container with Metabase and don't open any Metabase webpages (or do not refresh any already opened pages) to avoid duckdb database locking.
Login into container "$ docker exec -it <container_id> bash"
container:$ apt-get install unzip
container:$ cd /app/database
container:$ wget https://github.com/duckdb/duckdb/releases/download/v0.5.0/duckdb_cli-linux-amd64.zip
container:$ unzip duckdb_cli-linux-amd64.zip
container:$ ./duckdb
container:$ D .open my.duckdb
container:$ D create view v1 as select * from '/app/database/*.parquet';
container:$ D .exit
go to the Metabase web pages, open admin settings, open duckdb DB page and push buttons.

later all you need is just copy the new raw_view_NNN.parquet file into the folder with the ducked database (mounted into container) and refresh Metabase web page.


ENYO-5852: Enact-ui-tests: DatePicker and TimePicker - running same test on different builds gives different result

Checklist

[x] I have read and understand the contribution guide

[ ] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[x] Documentation was added or is not needed

[ ] This is an API breaking change

Issue Resolved / Feature Added
In certain cases, most notably in TimePicker, an undefined value is incorrectly overriding a supplied defaultValue on render.
Resolution
I opted to use a state value (derived), initially false, that is checked in getDerivedStateFromProps and prevents updating in the case where a decorated component is rendered open.  Once the props have been derived the first time, the state is updated and the component will update properly.
Additional Considerations
Component and UI tests seem to work fine with this change, but there might be a better way to handle this?
Links
ENYO-5852

I pushed an update to move the state determination from the constructor into getDerivedStateFromProps which allows us to avoid the extra state prop. I also added into this PR the fix for cancelling which I wanted to test while verifying this change.


Expire date is Jan 1970 when ban is permanent.

Hello
Whenever I set the ban to permanent it shows up on the webpage as expiring on January 1st 1970, I have attached a picture.
Any solutions to make it say "never" or "permanent"?
Thanks

Are you sure you are using the latest archive from GitHub? I made some fixes after the last update on SpigotMC so you should make sure you download again. The expire field should show “N/A” when a punishment does not expire.

All fixed by downloading new panel, cheers bud :)

Glad the updates fixed your issues. Thanks for reporting.


CFamily: Lex identifiers after case as constants

Add a state for marking identifiers preceded by  a case keyword as constants.
Additionally, refactor the label rule to no longer permit a case keyword before a label.
Consequentially, identifiers after a case keyword (like foo in case foo:) are no longer wrongly lexed as Name.Label, but as
Name.Constant.
In addition, this fixes https://github.com/pygments/pygments/issues/2076, as multiple case keywords in one line are lexed the same.

I'll leave this for a few days in case other maintainers would like to take a look.

I made a small oversight, so I pushed a couple of commits with a fix. I am sorry for the inconvenience, but I think it should be all good now.
Instead of lexing foo::bar, as Name.Constant, it's now lexed as:
foo  Name.Constant
:    Operator
:    Operator
bar  Name.Constant

Which is more correct and also consistent with how the lexer treats the :: operator.

This looks good to me. Thanks for working on this!


Datagrid : Expanding a column after a total row deletion crashs the Datagrid

Describe the bug
DataGrid_VisualCrash SIMPLE.zip
A. After a total deletion of all the rows present in Datagrid, the column resizing crash on visual.cs
Complementary/Very similars bugs :
B. After 50 element added then all deleted, it crash on visual.cs
C. (Same project) A scrollviewer Visible without any row in the Datagrid trigger a crash on visual.cs
Steps to reproduce the bug
A. Column resize bug

Click on 'Add 10'
Click on 'Erase All'
Resize the column width with mouse
See error "The specified Visual is not an ancestor of this Visual"

B. 50 elements total deletion bug

Click on 'ADD 10' (5 times, so 50elements)
Click on 'Erase All'
See error "The specified Visual is not an ancestor of this Visual"

C. Scrollviewer visibility bug

In XAML, set the Scrollviewer visibility to "Visible"
Launch the project
See error "The specified Visual is not an ancestor of this Visual"

Expected behavior
No crash
Screenshots
DataGrid_VisualCrash SIMPLE.zip
NuGet package version
None
Windows version
Windows 11 (22H2): Build 22621
Additional context
.NET concerned : .NET Framework 4.7.2 to .NET 7
This issue is reported because of a more problematic issue which is happening on datagrid on a more complex C# Solution : when the datagrid is displayed, after 2 total deletion of the rows, the same crash appears. So I'm trying to isolate any crash source.
DataGrid_VisualCrash SIMPLE.zip

<UseWPF>true</UseWPF>

Wrong repo.


feat(client): add entity and query functions

PR Description
PR Checklist
Testing

[x] added relevant test coverage
[ ] no tests needed

Docs

[x] added relevant docs

preview them at https://homebase.io/docs/homebase-react/{BRANCH_NAME}/overview

[x] updated relevant sections in the README.md
[x] updated relevant docstrings in index.d.ts
[ ] no docs needed

Typescript

[x] added or edited relevant Typescript type declarations
[ ] no type declaration updates needed

Merging
For maintainers.
To merge, select "Squash and Merge".

This looks good to me! Cool we could do this with such little code.
One thing: I can't check this since I'm sick and AFC, but does it make sense to update typescript types here?

Yep, just updated them. I noticed them in the checklist. Love that thing!


added disclaimer to the new tutorial

rule change and version change due to beginning of the new month

@lenastodal If there is already a new version of this tutorial, you should retire this one.


feat: download list of wsdl from a csv

command
tests
documentation

Ciao Artur, thanks for building this.
I did some changes because I needed a command to download a number of  wsdl files in a folder.
I hope the effort is appreciated but if it doesn’t suit the goal of the project feel free to close the PR, I understand!


Implement first version of pipeline

This PR wraps the sis-style checksum script within a Nextflow pipeline.
The script was adapted to have clear input/outputs and so that the script can run from nextflow's work directory.

Thanks! I've addressed your comments now :)


Add Tabs to the right side of (empty space) panel

I assume by the empty space you mean the right side of the menu button:

If so, delete the /*Disable_Horizontal_Tabs */ section (line 3 to 105) of the user_Chrome.css file and paste the following code block to the section.
:root:-moz-any([inFullscreen], [tabsintitlebar]) #TabsToolbar:not(:-moz-lwtheme) { -moz-appearance: none !important; }
#TabsToolbar {
  background-color: #000000 !important;
  margin-bottom: 0px !important;
  margin-top: 0px !important;
  margin-left: 35vw !important;
  margin-right: 0px !important;
  max-height: 35px !important;
  height: 35px !important
}

#nav-bar {
  margin-bottom: 0px !important;
  margin-top: -35px !important;
  margin-left: 0px !important;
  margin-right: 65vw !important;
  border-top: none !important;
  height: 35px;
  background-color: black !important;
}

This is the final result:

Try to edit the pixel values to fit your liking.

Since it has been two days, I will close the issue for now. But if the solution doesn't work, feel free to open it again.


primitives: use alloy Withdrawals

Describe the feature
At the moment, we have defined a Withdrawals structure inside the primitive-traits crate:
https://github.com/paradigmxyz/reth/blob/eec861fe9fc787900c18fc2d080f73c89620f5e3/crates/primitives-traits/src/withdrawal.rs#L35
After the merge of https://github.com/alloy-rs/alloy/pull/1462, this definition could be removed in reth and replaced by a simple import of the equivalent  alloy type.
Additional context
No response

Seems not too hard, can I work on it? What do you think? @tcoratger

Seems not too hard, can I work on it? What do you think? @tcoratger

I think it could be a good first issue wdyt @mattsse? Maybe we can provide some guidance if needed :) Happy to help. This is blocked by https://github.com/alloy-rs/alloy/pull/1462 at the moment

Should be unlocked by new alloy release https://github.com/alloy-rs/alloy/commit/3117f7625e508c50353e60b9026b26e810610371

Ready to work on this one if it is possible @mattsse.


gulp usage feels bad

As a gulp user with builds for sass, js, svg-maps, and so on; the sprity.src() syntax feels really weird. Also I have a few problems with this.
It doesn't translate 100% into a gulp stream. For example, if I want to build sprites for each subdirectory, I can't do it like the gulp docs suggest. The merged streams generate totally wrong sprites and css-files. Because of this flaw I'm unable to use sprity in my projects.
Yeah, there's a split-option, but as a gulp user I should be able to be completely free in my workflow. (Also, the split-option seems to generate strange output, too.) That's the first time I see a module not using the normal stream with gulp.src() and I already ran into problems.
Don't get me wrong – I just could use another module. But besides the sprity.src, this module feels really solid and I would love to use it in future projects.
So, see it as a suggestion: In my opinion the gulp.src() syntax would be much better.

Do you want to do something before starting sprity in your task? Just asking because sprity.src uses the same internal mechanism as gulp.src:
in Gulps source code: Gulp.prototype.src = vfs.src;
in sprity's source code: return vfs.src(opts.src)
The syntax is just a little bit different. In sprity the file glob is given via the options object.
For example:
sprity.src({
  src: ['./images/*.png', './pictures/*.jpg']
});

The only thing you can't do is use sprity in the middle of a gulp stream (Thats why I was asking). To me it wouldn't make that much sense using sprity in the middle of the task as long as you don't want to do something with the images before generating the sprites (I really can't think of something I would want to do with the images before genrating sprite). You can still use the suggested way from the gulp docs just with a little different syntax:
gulp.task('sprites', function() {
   var folders = getFolders(spritesPath);
   var tasks = folders.map(function(folder) {
       return sprity.src({
         src: path.join(spritesPath, folder, '/*.png')
      })
      ...
      .pipe(gulp.dest(spritesPath));
   });
});

Keep in mind: If you do that and also let sprity generate the css file, there will be naming collisions of css classes. Thats why there is the split option.
I also have to say that sprity is not a gulp plugin. It is a standalone sprite generator that happens to work quite well with gulp because it uses the same internal streaming mechanism.
Also, the split-option seems to generate strange output, too.
What do you mean by that? Could you please give me an example of the strange output? It seems like you found a bug, because sprity should return a stream with the sprites for each folder.

Thank you for the quick response!
No, I don't want to use sprity in the middle of the gulp stream – I can't imagine a scenario where this is necessary, too. :D
I wanted to use the merged tasks for each directory, and I used the exact same code you wrote.
gulp.task('sprites', function () {
  var folders = getFolders(spritesPath);
  var tasks = folders.map(function (folder) {
    return sprity.src({
      src: path.join(spritesPath, folder, '/*.png'),
      name: 'sprite-' + folder,
      style: './mypath/_sprite-' + folder + '.scss',
      dimension: [{
        ratio: 1, dpi: 72
      }, {
        ratio: 2, dpi: 144
      }]
    })
    .pipe(gulpif('*.png', gulp.dest('./dist/')))
    .pipe(gulpif('*.scss', gulp.dest('./dist')))
  });
  return merge(tasks);
});

I made an example with two directories. Each file is 200x200px.
test1/a.png, test1/b.png
test2/a.png, test2/b.png, test2/c.png, test2/d.png
The generated sprite-test1.png looks like this:

The generated sprite-test2.png looks like this:

The generated variables in _sprite-test1.scss are:
$test1-a: -0px -0px 208px 208px;
$test2-a: -0px -208px 208px 208px;
$test1-b: -0px -416px 208px 208px;
$test2-b: -0px -624px 208px 208px;
$test2-c: -0px -832px 208px 208px;
$test2-d: -0px -1040px 208px 208px;

The generated variables in _sprite-test2.scss are:
$test1-a: -0px -0px 208px 208px;
$test2-a: -0px -208px 208px 208px;
$test1-b: -0px -416px 208px 208px;
$test2-b: -0px -624px 208px 208px;
$test2-c: -0px -832px 208px 208px;
$test2-d: -0px -1040px 208px 208px;
$test1-a: -0px -0px 208px 208px;
$test2-a: -0px -208px 208px 208px;
$test1-b: -0px -416px 208px 208px;
$test2-b: -0px -624px 208px 208px;
$test2-c: -0px -832px 208px 208px;
$test2-d: -0px -1040px 208px 208px;

Note: This is no custom processor template, it's sprity-sass.
Regarding the split option
Seems like I wrote the configurations somehow wrong. I tried it just now and it works. Sorry for that! Unfortunately this is no option for me, because I need to separate my sprite images in groups. That's why I tried it with the merged streams.

Thats seems wrong to me too. I will have a look into it.

One last thing I like to mention: apart from that sprity is great. The build-process is superfast, the downscaling to support retina/nonretina is perfect and it keeps projects clean from @2x.png files.

Why not just make it a gulp plugin? The CLI can just pipe the options directly to it.

The way this works aside from just gulp.src() makes it so I can't handle any errors that occur.
I'm in a situation where we might not always use sprites on a project. If we do, I'd love to be able to use sprity. If we don't, then.. that should be fine too.
The problem is if I run the sprite task without any images in the directory, it throws an unhandled error and stops gulp from running entirely. Which I can't really do anything about at the moment, it seems. Any error handling I try to do never triggers, as it all seems to happen inside sprity.src()

@loremipson In 1.0.4 the error handling with gulp should work better. It now emits an error on the stream. So something like:
sprity.src({
  src: './images/*.png'
})
.on('error' function (err) {
  console.log(err);
});

should work. Error handling in gulp should than work like with every other plugin. Please leave a comment if you still got problems with that.

I close the issues since it seems to be fine now.


exclude translatesAutoresizingMaskIntoConstraints = NO under ios 11

Description
AdView be placed with frame location under ios 11. But translatesAutoresizingMaskIntoConstraints has NO flag.
Related Issues
https://github.com/flutter/flutter/issues/27457
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[x] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[ ] No, this is not a breaking change.

I signed it!

Sure .

May I know when this PR will be merged? It's quite bothering that I have to apply this change locally in order to have a workable plugin for my app.

Same! Hoping for a review and release soon. 🤞

Maybe this will get some attention if you add [firebase_admob] to the start of the title?

Tagging @collinjackson to make sure this doesn't fall through the cracks. Thanks!

Thanks for suggestions i changed to start of title @westy92 also i am tagging to @collinjackso n.


enhancement: resolve composer validation warnings and add test script

Fixed warnings in composer validate by specifying version constraints
Added composer test script for easier test execution

this PR should fix #7 ,
Additionally, add predefined test script


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #805 (8d6f933) into main (eee4700) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main     #805   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17


Missing input file(s)

As it stands, when I clone the repository and run the first cell, it fails. This is because the input file name is hard-coded. Hopefully, this will be addressed later by your program taking an input file as an argument. For now though, could you include an example input file?

Now, a Test file is added to the repository.


customdata and userdata should agree on casing

What happened?
I tried to provision a device with the following parameters:
forProvider:
  userdata: |
     ...
  customdata: |
     ...

This failed to provision and I spent a while scratching my head wondering if the client was wired up correctly.
I did not receive an error when updating the resource to include this parameter, but it was eventually removed in a subsequent reconciliation.
I discovered that customData was the name that was correct and accepted.
It is confusing that userdata and customData do not follow the same standard.
Pick one.
Without knowing why the wrong case was accepted and later removed from the resource, users may attempt to use userdata or userData (similar for customdata) and experience the same problem. Should this change have been rejected earlier?

How can we reproduce it?
Use customdata in a device spec.

What environment did it happen in?
Crossplane version: v0.14
EM Provider: v0.0.5

Closing for #69
Problem should be resolved in https://github.com/crossplane-contrib/provider-jet-equinix


Fix calling non-ISR functions from CAN ISR

csp_can1_rx calls csp_can_pbuf_new, which calls csp_can_pbuf_free. All of these functions can be called from an interrupt service routine, which is why csp_can1_rx and csp_can_pbuf_new accept a task_woken parameter that is non-NULL if called in an ISR. But that parameter is not passed to csp_can_pbuf_free, which then calls functions that are not safe to call from an ISR, eventually causing an assertion in FreeRTOS queue code.
The fix is just to propagate the task_woken and call the ISR safe functions if it is not NULL.

I agree that we need tp pass task_woken to any function using csp_get_ms_isr() or any other ISR related functions.  And I approve this PR.
But, the root cause of this is getting ridicurous.  We should have abstract version of csp_get_ms() or we should drop  portTICK_TYPE_IS_ATOMIC == 0 of FreeRTOS.  Does any one using FreeRTOS with non atomic tick?
I can't even find one arch on the official repo.
grep -Pre '#define portTICK_TYPE_IS_ATOMIC.+0'
FreeRTOS/Source/include/FreeRTOS.h:    #define portTICK_TYPE_IS_ATOMIC    0

rep -Pre '#define portTICK_TYPE_IS_ATOMIC.+1' | wc -l
77

Our tick type is atomic, so we never encountered a problem with calling the csp_get_ms from the ISR, but csp_buffer_free was crashing consistently because it calles some queue function from FreeRtos which then asserts that it is not in an interrupt handler context.
So even if you drop the support for non-atomic tick types and declare csp_get_ms as ISR safe, I don't think it's possible to get around passing the task_woken, especially because that information about whether or not a task was woken needs to be propagated up to the interrupt handler so it can do a context switch and make sure the correct task is resumed after exiting from the ISR.

Im sorry for the inconvenience.
We have the same fix in spaceinventor/libcsp but i havent merged it back to upstream. Your fix is good.


Test the checks

While the issue #316 is still being discussed I have started writing integration tests for the database checks.
Please review the current approach.

Not altering anything that will affect current tests, adding functionality for the future. Happy to move this forward


Crash when closing tab while loading power shell modules

Environment
Windows build number: 10.0.18363.657
Windows Terminal version (if applicable): 0.9.433.0
Steps to reproduce

Open new Windows Power Shell Tab
Close the tab as soon as the module loading message appears.

Expected behavior
No crash
Actual behavior
Crash of the power shell process.

Sorry, is powershell crashing or is terminal crashing?

powershell is crashing but if I do the same steps directly on the powershell the powershell does not crash

So, all Terminal does is send a standard close event to powershell; there shouldn't be any magic in how it tears down within versus without terminal. It might be worthwhile to file a bug on the powershell repository, as they may have more insight into what's going on.
Thanks!


mpirun support

From<EMAIL_ADDRESS>on 2012-01-28T10:49:56Z
Hello spyder community,
It would be awesome if the run configuration dialog had an option to run the incumbent file through mpirun -n <number_of_worker_processes> <file.py>. I ignore the technical implications of enabling such functionality (i.e. console output from the various mpi processes, etc.)
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=922

From ccordoba12 on 2015-01-02T17:44:14Z
This seems too much specialized and out of our scope.
Status: WontFix


Add mdformat-gfm for Github Flavored Markdown support

@williamboman @anand-bala Is it possible to add mdformat-gfm and other options listed here? They seem to each have their own mdformat executable.

Originally posted by @tnthi115 in https://github.com/mason-org/mason-registry/issues/2132#issuecomment-1865412278

When trying to push a new feature branch, I get the following error:
$ git push --set-upstream origin feat/add-mdformat-gfm
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.


Restore backup directly from MinIO

What did you do?
I want to migrate cassandra data, i already created the backup using Medusa and it is stored on minIO, I want to restore this data to another cassandra cluster. MedusaTask for sync-ing with MinIO does not work. I cannot see my MedusaBackup object.
Environment
Source Cluster: k8ssandra/k8ssandra 1.5.1
Destination Cluster k8ssandra/k8ssandra-operatorr 1.7.1

K8ssandra Operator version:
docker.io/k8ssandra/k8ssandra-operator:v1.7.1

Kubernetes version information:
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.2", GitCommit:"8b5a19147530eaac9476b0ab82980b4088bbc1b2", GitTreeState:"clean", BuildDate:"2021-09-15T21:38:50Z", GoVersion:"go1.16.8", Compiler:"gc", Platform:"linux/amd64"} Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.2", GitCommit:"8b5a19147530eaac9476b0ab82980b4088bbc1b2", GitTreeState:"clean", BuildDate:"2021-09-15T21:32:41Z", GoVersion:"go1.16.8", Compiler:"gc", Platform:"linux/amd64"}

K8ssandra Operator Logs:

draganas@msnd01srv:~$ kubectl logs k8ssandra-k8ssandra-operator-6fdc6474cb-hdwpr -n k8ssandra-operator | grep b8fa99f3-b5cd-44ad-aeb3-cb0963bd737b
2023-08-16T18:07:22.704Z        INFO    Starting reconciliation for MedusaTask  {"controller": "medusatask", "controllerGroup": "medusa.k8ssandra.io", "controllerKind": "MedusaTask", "MedusaTask": {"name":"sync-backups-1","namespace":"k8ssandra-operator"}, "namespace": "k8ssandra-operator", "name": "sync-backups-1", "reconcileID": "b8fa99f3-b5cd-44ad-aeb3-cb0963bd737b", "MedusaTask": "k8ssandra-operator/sync-backups-1", "cassdc": "dc1"}
2023-08-16T18:07:22.704Z        INFO    Starting sync operation {"controller": "medusatask", "controllerGroup": "medusa.k8ssandra.io", "controllerKind": "MedusaTask", "MedusaTask": {"name":"sync-backups-1","namespace":"k8ssandra-operator"}, "namespace": "k8ssandra-operator", "name": "sync-backups-1", "reconcileID": "b8fa99f3-b5cd-44ad-aeb3-cb0963bd737b", "MedusaTask": "k8ssandra-operator/sync-backups-1", "cassdc": "dc1"}
2023-08-16T18:07:22.724Z        INFO    Listing Backups...      {"controller": "medusatask", "controllerGroup": "medusa.k8ssandra.io", "controllerKind": "MedusaTask", "MedusaTask": {"name":"sync-backups-1","namespace":"k8ssandra-operator"}, "namespace": "k8ssandra-operator", "name": "sync-backups-1", "reconcileID": "b8fa99f3-b5cd-44ad-aeb3-cb0963bd737b", "MedusaTask": "k8ssandra-operator/sync-backups-1", "cassdc": "dc1", "CassandraPod": "k8ssandra-dc1-rack-3-sts-0"}
2023-08-16T18:07:22.755Z        INFO    Deleting Cassandra Backup       {"controller": "medusatask", "controllerGroup": "medusa.k8ssandra.io", "controllerKind": "MedusaTask", "MedusaTask": {"name":"sync-backups-1","namespace":"k8ssandra-operator"}, "namespace": "k8ssandra-operator", "name": "sync-backups-1", "reconcileID": "b8fa99f3-b5cd-44ad-aeb3-cb0963bd737b", "MedusaTask": "k8ssandra-operator/sync-backups-1", "cassdc": "dc1", "Backup": "cassandra-db-rs-backup"}
2023-08-16T18:07:22.765Z        INFO    Deleted Cassandra Backup        {"controller": "medusatask", "controllerGroup": "medusa.k8ssandra.io", "controllerKind": "MedusaTask", "MedusaTask": {"name":"sync-backups-1","namespace":"k8ssandra-operator"}, "namespace": "k8ssandra-operator", "name": "sync-backups-1", "reconcileID": "b8fa99f3-b5cd-44ad-aeb3-cb0963bd737b", "MedusaTask": "k8ssandra-operator/sync-backups-1", "cassdc": "dc1", "Backup": "cassandra-db-rs-backup"}
2023-08-16T18:07:22.765Z        INFO    finished task operations        {"controller": "medusatask", "controllerGroup": "medusa.k8ssandra.io", "controllerKind": "MedusaTask", "MedusaTask": {"name":"sync-backups-1","namespace":"k8ssandra-operator"}, "namespace": "k8ssandra-operator", "name": "sync-backups-1", "reconcileID": "b8fa99f3-b5cd-44ad-aeb3-cb0963bd737b", "MedusaTask": "k8ssandra-operator/sync-backups-1", "cassdc": "dc1", "MedusaTask": "sync/k8ssandra-operator"}

Hi @networkingana,
your settings for the storage backend mustn't exactly match. K8ssandra v1 used to set the prefix to the cluster name (or .k8ssandra, I can't remember exactly), while k8ssandra-operator makes that an explicit setting (the prefix one).
That prefix is the name of the root folder that will contain an index folder and then one folder per node in the cluster.
Once you get the settings right, they should sync.
There's a slight chance you run into trouble at restore time though because of using different versions of Medusa for backup and restore, and we've introduced some breaking changes to better create restore mappings in later versions.

Hi @adejanovski,
Thank you for your answer.
Not sure I understand correctly. I'm aware of the prefix setting for the k8ssandra-operator, but the backup that was created with k8ssandra 1.5.1 is not placed in any root directory, the index and the node folders are directly placed inside the bucket on MinIO:
@wknd03srv:/mnt/minio_data_new/k8ssandra-medusa$ ls index  k8ssandra-dc1-rack-1-sts-0  k8ssandra-dc1-rack-2-sts-0  k8ssandra-dc1-rack-3-sts-0  k8ssandra-dc1-rack-4-sts-0  k8ssandra-dc1-rack-5-sts-0
k8ssandra-medusa from the path, is my bucketName
please correct me

How did you configure Medusa in your K8ssandraCluster CR?

Try specifying an empty prefix instead.
We set the prefix to a default value (the K8ssandraCluster name) if it's not set: https://github.com/k8ssandra/k8ssandra-operator/blob/fix-replication-dcname-overrides/pkg/medusa/reconcile.go#L71-L75

UPDATE:
Now my configuration look like this:
medusa:
   storageProperties:
     bucketName: k8ssandra-medusa
     concurrentTransfers: 1
     host: minio.minio.svc.cluster.local
     maxBackupAge: 0
     maxBackupCount: 0
     multiPartUploadThreshold: 104857600
     port: 9000
     prefix: ' '
     secure: false
     storageProvider: s3_compatible
     storageSecretRef:
       name: medusa-bucket-key
     transferMaxBandwidth: 50MB/s

And it seems that the backup is successfully synced with my other Cassandra cluster.
Thank you for your support

I run into a issue with the restore process: 4/5 finished successfully according the logs from the medusa-restore container but the rack 1 has a error and the pod is stuck in crashloopbackoff
[2023-08-17 12:49:04,275] DEBUG: http://minio.minio.svc.cluster.local:9000 "HEAD /k8ssandra-medusa HTTP/1.1" 200 0
DEBUG:urllib3.connectionpool:http://minio.minio.svc.cluster.local:9000 "HEAD /k8ssandra-medusa/192-168-84-177/backup-mk/meta/schema.cql HTTP/1.1" 404 0
[2023-08-17 12:49:04,276] DEBUG: http://minio.minio.svc.cluster.local:9000 "HEAD /k8ssandra-medusa/192-168-84-177/backup-mk/meta/schema.cql HTTP/1.1" 404 0
ERROR:root:No such backup
[2023-08-17 12:49:04,277] ERROR: No such backup
Mapping: {'in_place': True, 'host_map': {'k8ssandra-dc1-rack-3-sts-0': {'source': ['k8ssandra-dc1-rack-2-sts-0'], 'seed': False}, 'k8ssandra-dc1-rack-5-sts-0': {'source': ['k8ssandra-dc1-rack-1-sts-0'], 'seed': False}, 'k8ssandra-dc1-rack-2-sts-0': {'source': ['k8ssandra-dc1-rack-5-sts-0'], 'seed': False}, 'k8ssandra-dc1-rack-4-sts-0': {'source': ['k8ssandra-dc1-rack-4-sts-0'], 'seed': False}, 'localhost': {'source': ['192-168-84-177'], 'seed': False}}}
Downloading backup backup-mk to /var/lib/cassandra

Not sure why there is an IP address accessing the schema.cql from minio in ends with error code 404 Not found

Yeah, that's the kind of incompatibility I was afraid of. K8ssandra v1 didn't allow to resolve all pods hostnames correctly, which then lead to what you're seeing.
We then did some breaking changes which don't make k8ssandra-operator backwards compatible with K8ssandra v1 backups.

So I decided to upgrade the k8ssandra to k8ssandra-operator, probably that will help with this issue.
Can I somehow stop the current restore, even if all data is gone?

I migrated the source cluster to k8ssandra-operator, and this is the result from the restore
INFO  [SSTableBatchOpen:1] 2023-08-18 21:21:50,473 SSTableReaderBuilder.java:351 - Opening /var/lib/cassandra/data/system/repairs-a3d277d1cfaf36f5a2a738d5eea9ad6a/nb-19-big (1.875KiB)
INFO  [SSTableBatchOpen:1] 2023-08-18 21:21:50,474 SSTableReaderBuilder.java:351 - Opening /var/lib/cassandra/data/system/repairs-a3d277d1cfaf36f5a2a738d5eea9ad6a/nb-17-big (12.250KiB)
INFO  [SSTableBatchOpen:1] 2023-08-18 21:21:50,475 SSTableReaderBuilder.java:351 - Opening /var/lib/cassandra/data/system/repairs-a3d277d1cfaf36f5a2a738d5eea9ad6a/nb-18-big (2.980KiB)
INFO  [main] 2023-08-18 21:21:50,515 QueryProcessor.java:106 - Initialized prepared statement caches with 10 MB
ERROR [main] 2023-08-18 21:21:50,676 CassandraDaemon.java:909 - Cannot start node if snitch's rack (rack-1) differs from previous rack (rack-3). Please fix the snitch configuration, decommission and rebootstrap this node or use the flag -Dcassandra.ignore_rack=true.

I tried to configure this in the jvmOptions.additionalOptions, but no luck.
Can you please help with this?


'image' not found when using this macro within #Preview macro

Not sure if this is a limitation of swift macros, or a bug in this library, but I thought I'd leave this here.
Seems the SFSymbol macro does not work within swiftui #Previews
Sample code:
import SFSymbolsMacro
import SwiftUI

@SFSymbol
enum Symbol: String {
    case archive = "archivebox"
    case arrowUpRightSquare = "arrow.up.right.square"
    case chevronForward = "chevron.forward"
    case checkmark = "checkmark"
    case checkmarkCircleFill = "checkmark.circle.fill"
    case checkmarkShieldFill = "checkmark.shield.fill"
    case clockArrowCirclepath = "clock.arrow.circlepath"
    case dismissKeyboard = "keyboard.chevron.compact.down"
    case ellipsis
    case exclamationmarkCircleFill = "exclamationmark.circle.fill"
    case message = "message"
    case personCircleFill = "person.circle.fill"
    case phone = "phone"
    case questionMarkCircleFill = "questionmark.circle.fill"
    case trash
    case xmarkCircleFill = "xmark.circle.fill"
}

Does not work:
#Preview {
    Label {
        Text("Some Text")
    } icon: {
        Symbol.checkmarkShieldFill.image // Value of type 'Symbol' has no member 'image'
    }
}

Works:
let icon =  Symbol.checkmarkShieldFill.image
#Preview {
    Label {
        Text("Some Text")
    } icon: {
        icon
    }
}

This seems like a limitation of Swift macros. Apparently macro generated members can't be accessed in the Preview macro. An additional workaround is to use PreviewProvider instead of the #Preview macro.


add unit test for instaslice res del

Unit test that checks InstaSlice resource deletion on node

Posted this on our internal slack.
I think we should have a plan in place on what our various tests would look like. This looks to me that you are writing an integration test. I would suggest looking into envTest for testing controller logic.
https://book.kubebuilder.io/cronjob-tutorial/writing-tests
Kueue or JobSet have pretty good examples of separation.
EnvTest sets up a fake k8s control plane so it mostly tests the creation of your CRDs without interacting with the host system.
e2e tests would be actually using GPUs or even installing the emulator plugin on an actual k8s cluster.
I have found mocking to be incredibly painful for kubernetes controllers so I like the approach of testing unit tests where it makes sense. Either inject the objects that the controls would create and write tests that way. Or we could use mocking,

/lgtm


Add Green Tears of Guthix Timer (fixes #12858)

This PR adds an optional Green Tears timer to the Tears of Guthix Plugin (fixes #12858).

I really don't think this needs to be configurable

@abextm Not a problem, I've gone ahead and cut the configuration option from this PR.

Hi,
Would it be possible if you could re-add the toggle for the green tears timer. I can't differentiate the two colours as i use a blue light filter on my screen

Hi,
Would it be possible if you could re-add the toggle for the green tears timer. I can't differentiate the two colours as i use a blue light filter on my screen

Hi,
Would it be possible if you could re-add the toggle for the green tears timer. I can't differentiate the two colours as i use a blue light filter on my screen

This has been added

Hi,
Would it be possible if you could re-add the toggle for the green tears timer. I can't differentiate the two colours as i use a blue light filter on my screen

This has been added

Thank you very much to you and the teams hard work 😊
Have a great day!
From<EMAIL_ADDRESS>Sent: 24 January 2021 15:44
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [runelite/runelite] Add Green Tears of Guthix Timer (fixes #12858) (#12862)
Hi,
Would it be possible if you could re-add the toggle for the green tears timer. I can't differentiate the two colours as i use a blue light filter on my screen
This has been added
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/runelite/runelite/pull/12862#issuecomment-766370355, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASSLCBGNCYNQOJOF3VO4ISLS3Q54DANCNFSM4UG23MKA.

Thank you very much to you and the teams hard work 😊
Have a great day!
From<EMAIL_ADDRESS>Sent: 24 January 2021 15:44
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [runelite/runelite] Add Green Tears of Guthix Timer (fixes #12858) (#12862)
Hi,
Would it be possible if you could re-add the toggle for the green tears timer. I can't differentiate the two colours as i use a blue light filter on my screen
This has been added
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/runelite/runelite/pull/12862#issuecomment-766370355, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASSLCBGNCYNQOJOF3VO4ISLS3Q54DANCNFSM4UG23MKA.


Don't stack PR tab menu on small screens

the stacking takes up screen space - display the tabs as the navigation bar. github uses the same layout.
Screenshots (left before, right after):

Could you please do it in the CSS file? https://github.com/go-gitea/gitea/blob/be23b73e8523738a1d3c214a01c0f162146da1c0/web_src/css/repo.css#L750

sorry - not sure what I should do in the CSS file?

Please set
overflow-x: auto;
overflow-y: hidden;

inside the existing CSS class for .pull.tabular.menu instead of adding helper classes. They are a good idea if there is no existing class for an element or if it is using a generalized class, but if there is already a class for the element, it is cleaner to use it.

I am afraid this change is not right on desktop view.
IIRC the "bottom line" of the menu would be broken

Good catch - i've now added a divider below and fixed layout for large screens.
Screenshots in PR description are updated.


feat: auto import sorting

This PR...
Changes

WIP: Added auto import sorting. To discuss with the team

Do we have to install an extension for this?

Yes. I found one for VS Code called TypeScript Import Sorter https://marketplace.visualstudio.com/items?itemName=mike-co.import-sorter. Do we need the one which works not only in VS Code?

@Toxer10 we need one that works with es-lint or something similar - it should be automated as a pre-commit hook and  shouldn't require any specific editor


Add fallback for HealthIcon

if the file for the current character's icon doesn't exist, then the game will fallback to bf's icon
this will only work if the game is being compiled for sys (just like sm support)
some cleanup to the file has also been done (removing redundant code and some formatting fixes)

Since icons are hardcoded wouldn't it better to check with Assets.exists(); instead of FileSystem so its not sys only?

oh I didn't think about that, good catch


Publish should display an interactive warning if host.json is not found in current directory

You can accidentally publish a function to a Function App if you happen to be in the function folder.
Should do the following:

Look for host.json in the current folder
If not found, display the following message: Warning: host.json not found in current folder. Proceed with publishing the current directory to Azure Function App $name?
Add --ignore command line argument (-i for short) to not prompt

Alternatively we can throw an error if host.json is not found and have the flag override it.
We can also travel up the directory tree to find a host.json and then use that directory, throw error if not, use current directory if --ignore is passed (how about --ignoreHostJsonNotFound instead. I think --ignore might be too generic?)
What do you think?

Agreed on both counts. I like traversing up to find host.json, that's a nicer customer experience.


Improve logging

Feedback from Vasek:

There is no info that I try to enqueue invalid URL, it was with the old crawler.
There is no info that I try to visit some page for the second time and it skips it, it was with the old crawler.
There is very little logs even when I turn it on, would be great to tell what is my log and what is log of the website, cause they are mixed together.

Is this still up to date? On what level/component should this be implemented?

It was feedback from Vasek when we moved from legacy crawler product to 100% SDK and a lot has changed since this so I'd close this


179: add-model-to-sites-table

Pull Request
Description
This PR addresses the issue #179 to handle the ability to assign a model to one or more site.

Thanks so much @hanaawad24 for doing this.
I've added this in already - https://github.com/openclimatefix/pv-site-datamodel/blob/main/pvsite_datamodel/write/user_and_site.py#L381
I wondered if you could improve the current function, and include those extra tests you have added?

I'll close this for the moment, but feel free to add those test improvements on this PR, or a new one

I'll close this for the moment, but feel free to add those test improvements on this PR, or a new one

@peterdudfield Thank you for your time and feedback. I did another pull request #186  Thank you


How to change it's content to ours

How to change it's content to ours

Most of the content can be configured in src/configs/.


Seems to be wrong release version. Example not working for using unsupported method

Using IntelliJ and importing as
compile 'com.rockerhieu.emojicon:library:1.0'
However, in EmojiActivity.java, none of
mEditEmojicon.setUseSystemDefault(b)
mTxtEmojicon.setUseSystemDefault(b)
EmojiconsFragment.newInstance(useSystemDefault)
could be found.
Did I config the dependencies wrong or the library has a version mismatch with the example?
Thanks,

Felix

Hi @rockerhieu
Any plans to release 1.1 to the Maven Central?
Thanks
Egor

@Egorand you could continue using 1.1-SNAPSHOT. If there is nothing critical coming up, I will release it to maven in couple days.

1.1 is deployed! Please give it sometimes to sync to maven central.

The deployment was be rejected. Checking...

Confirmed, 1.1 is on maven central


improve type inference for enumerate

This is a work in progress to optimize for in enumerate() in more cases.
This enables the following optimized usages:
def find_digit_end(value: str):
    for idx, c in enumerate(value):
        if c in '0123456789':
            break

see #3787
def list func():
    return []

def example():
    for idx, c in enumerate(func()):
        pass

and
def example():
    for idx, c in enumerate(range(10)):
        pass

However since it is the first time working on this there are still a couple of things, which are unclear to me:

I only changed the code in FlowControl.py however pretty much the same exists in TypeInference.py as far as I can see only the if not is_special: uses a different implementation (even though the comment in both cases states that an IndexNode is used). Whats the difference between the two? I assume I need to perform similar changes in TypeInference.py as well.

I am not quite sure whether this is the right approach. There are other things that might profit from a more optimized implementation. E.g.

def example(value1: str, value2: str):
    for a, b in zip(value1, value2):
        pass

It would be good if it is somewhat simple to extend to nested things like for idx, (a,b) in enumerate(value1, value2): as well. E.g. for CArray this currently infers the correct type, but still converts it to a list first.

is there a simple way to set a start value for the counter variable, which would allow implementing the optimization for enumerate(value, start=10). I tried to set value/constant_result in the IntNode, but it appears like that's not how it works.

How can I check which index type to use for C++ classes (see #4072). Is it possible to check at generation time, whether the class has a size_type typedef, so it can be used for the enumerate counter variable. At least for the standard containers (vector, string ...) this should usually be size_t, but I think it would still be cleaner to check the typedef when available.

How can I check which index type to use for C++ classes (see #4072). Is it possible to check at generation time, whether the class has a size_type typedef, so it can be used for the enumerate counter variable. At least for the standard containers (vector, string ...) this should usually be size_t, but I think it would still be cleaner to check the typedef when available.

I would be tempted not to do it this way. I think Cython only relies on begin() and end() for C++ iteration (rather than size and operator[]). You probably want to keep that so that the generated code looks something like:
some_type counter=0;
for (container::iterator it=container.begin(); it != container.end(); ++it) {
    ++counter;
}

Therefore counter has nothing to do with the container size_type, but is based on whatever rules Cython uses generally (probably whether the start index is negative?)

Don't know the answers to your other questions right now.

Therefore counter has nothing to do with the container size_type, but is based on whatever rules Cython uses generally (probably whether the start index is negative?)

Hm negative indexes are generally problematic, since both ssize_t and size_t can not cover the whole range that might be required. AFAIK negative indexes already cause issues with the current range implementation (see #4075)


pg_regress reloption test failure

One more floating bug in CI.
https://github.com/zenithdb/zenith/runs/2487396618
+++ /home/runner/work/zenith/zenith/integration_tests/.zenith/regress/results/reloptions.out	2021-05-02 18:15:35.707502682 +0000
@@ -131,7 +131,7 @@
 SELECT pg_relation_size('reloptions_test') = 0;
  ?column?
 ----------
- t
+ f
 (1 row)

May be the same problem as in #85.

see also
https://www.postgresql.org/message-id/flat/CAD21AoCFLA%2BmWMoZpGeQ3-88JxuQz7v%2BFeMMJWgT8v8tu470Rg%40mail.gmail.com#2ff703383e6701e45905b9416296e265


[un-revert] Occupancy sensing updates for Matter 1.4

PR #34293 was merged by mistake with ESP32 break. This PR brings back the code to fix and land.

Updated occupancy sensing cluster SDK implementation based on XML changes(PR#34163) for Occupancy Sensing rev 5

Updated occupancy sensing cluster sample app(all-clusters-app) implementation to Rev 5 in the zap and matter file of all-clusters-app

Remove "occupancy-sensor-server" from "ClientDirectories" in zap_cluster_list.json to resolve a building issue found from rebuilding the chip-tool

Fast-tracking, since assuming CI passes this is just a clone of https://github.com/project-chip/connectedhomeip/pull/34293 (which had approvals), with a CI fix.