add support for psycopg3

Issue #, if available: #395
Description of changes: Add support for psycopg3 (now just psycopg)
I have not been able to test this locally, as I can't find any development documentation in this repo.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hi @Dilski, thanks for contributing this instrumentation!
Some workflow unit tests are failing, can you please take a look and address the issues? Also curious if you are still unable to test this instrumentation end-to-end locally? Hopefully these steps can help:

Set up a simple sample application that makes a psycopg3 request, imports & uses the new instrumentation in this PR
Set up the AWS X-Ray daemon locally
Run the sample app once you have the daemon running as a sidecar. The X-Ray console should reflect the instrumented psycopg3 call

Please let us know if you have any questions or need more help setting up a local end-to-end test

Hi @Dilski, thanks for contributing this instrumentation! Some workflow unit tests are failing, can you please take a look and address the issues? Also curious if you are still unable to test this instrumentation end-to-end locally? Hopefully these steps can help:

Set up a simple sample application that makes a psycopg3 request, imports & uses the new instrumentation in this PR
Set up the AWS X-Ray daemon locally
Run the sample app once you have the daemon running as a sidecar. The X-Ray console should reflect the instrumented psycopg3 call

Please let us know if you have any questions or need more help setting up a local end-to-end test. (Referencing #395)

Hi @carolabadeer  - we've tried to address the issues and have managed to run the test suite at our side. Can we request a re-review?

Hey folks, any progress on this? I'd also like to take advantage of this patching logic once it's in, and am weighing whether it's worth waiting for a contribution back to aws_xray_sdk, or just patching it in my own package.

Hi, this seems to have been idle for over a year. Is there a chance support for psycopg3 will be picked up in the near future?


Arabic Texts are not rendering properly.

Can you provide an example file? I'd like to look in this more closely.

I was adding the charset at top of my .scss file.
@charset "UTF-8";
when I removed the charset, the issue was resolved.

Oh thanks for the update!

Thank you for your great work.


Make VSCode Jupyer Extension Checked by default

I recommend the jupyter checkbox is installed by default (box = checked)

Made Jupyter checked on default


Question about cuda kernel performance

Click to expand!
Issue Type
Performance
Source
source
Tensorflow Version
tf 2.11.0
Custom Code
Yes
OS Platform and Distribution
Linux/Ubuntu 20.04
Mobile device
no
Python version
3.8.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
cuda V11.2.152/cudnn 8.1.0
GPU model and memory
No response
Current Behaviour?
When running my model in eager mode on NVIDIA T4, The elapsed time for cuda kernel will become faster after several iterations (4 to be precise).
The following is the summary from Nsight system.
As shown, for eager mode, the max is 1.2ms, while the avg is about 500 us after 100 iterations;
for compat.v1.Session style with the same code base, the kernel time is about 1.2ms consistently.
The question is how does it happen, and which feature from tf2/eager mode can make cuda kernel run faster after several iters? As I understand, the cuda kernel is either from cudnn or cublas, tf itself or tracing or graph compilation should not touch that low-level logic.
// TF2.11.0 eager mode
 Time (%)  Total Time (ns)  Instances  Avg (ns)  Med (ns)  Min (ns)  Max (ns)  StdDev (ns)     GridXYZ         BlockXYZ                                                     Name
 --------  ---------------  ---------  --------  --------  --------  --------  -----------  --------------  --------------  ----------------------------------------------------------------------------------------------------
     13.4        258681374        500  517362.7  479449.5    469753   1222701     138505.1     8    5    1   256    1    1  volta_sgemm_128x128_nn

// TF 2.11.0 tf.compat.v1.Session()
 Time (%)  Total Time (ns)  Instances   Avg (ns)     Med (ns)    Min (ns)   Max (ns)   StdDev (ns)     GridXYZ         BlockXYZ                                                     Name
 --------  ---------------  ---------  -----------  -----------  ---------  ---------  -----------  --------------  --------------  ----------------------------------------------------------------------------------------------------
     16.4      607,754,406        500  1,215,508.8  1,221,934.0  1,161,743  1,228,110     18,653.6     8    5    1   256    1    1  volta_sgemm_128x128_nn

Standalone code to reproduce the issue
// eager mode
for i in tf.range(100):
    tf.print(model(x))

// sess
With tf.compat.v1.Session() as sess:
     print(sess.run(output_name_list, x))

Relevant log output
No response

Hi @mohantym,
Thanks for the reply! I couldn't share my private model so I made a simple example as below,
import argparse
import tensorflow as tf
import timeit

parser = argparse.ArgumentParser(description='TensorFlow bench',
                                 formatter_class=argparse.ArgumentDefaultsHelpFormatter)
parser.add_argument('--eager', action='store_true', default=False,
                    help='Enable eager execution')

args = parser.parse_args()
if not args.eager:
    tf.compat.v1.disable_eager_execution()

data = tf.zeros([640, 4096])
model = tf.keras.Sequential([
    tf.keras.layers.Dense(2048),
    tf.keras.layers.Dense(256),
    tf.keras.layers.Dense(10),
])

if args.eager:
    print(timeit.timeit(lambda: model(data), number=100))
else:
    with tf.compat.v1.Session() as sess:
        sess.run(tf.compat.v1.global_variables_initializer())
        print(timeit.timeit(lambda: model.predict(data, steps=1), number=100))

You can run it,
nsys profile python main.py --eager

and check the cuda kernel time distribution.
(Not sure why tf1 sess picks up the different kernels compared to tf2. Here we can just focus on tf2 kernel time)

Ok @kehuanfeng !
Thanks for the update.
@SuryanarayanaY !
Could you look at this issue.
Thank you!

Hi @kehuanfeng ,
Actually for larger models Graph mode can give better performance than eager mode.From TF 2.x onwards Eager mode made as default due to beginner friendly, easy to Debug etc and Graph mode as optional by using tf.function. I am not comparing 1.X Graphs here since we are not supporting 1.X versions now. Iam comparing performance of a function with larger epochs running on Eager mode and Graph mode where we can see Graph mode is clearly a winner. Please refer to the attached gist.
Thankyou!

Hi @SuryanarayanaY, @mohantym,
Thanks for the reply.
I updated the test sample as below,
import argparse
import tensorflow as tf
import timeit

parser = argparse.ArgumentParser(description='TensorFlow bench',
                                 formatter_class=argparse.ArgumentDefaultsHelpFormatter)
parser.add_argument('--mode',
                    choices=['eager', 'tf2graph', 'tf1graph'],
                    help='execution mode')

args = parser.parse_args()

if args.mode == "tf1graph":
     tf.compat.v1.disable_eager_execution()

data = tf.zeros([640, 4096])
model = tf.keras.Sequential([
    tf.keras.layers.Dense(2048),
    tf.keras.layers.Dense(256),
    tf.keras.layers.Dense(10),
])

if args.mode == "eager":
    def run_model():
        return model(data)
    run_model()
    print(timeit.timeit(lambda: run_model(), number=1000))
elif args.mode == "tf2graph":
    @tf.function()
    def run_model():
        return model(data)
    run_model()
    print(timeit.timeit(lambda: run_model(), number=1000))
elif args.mode == "tf1graph":
    with tf.compat.v1.Session() as sess:
        sess.run(tf.compat.v1.global_variables_initializer())
        model.predict(data, steps=1)
        print(timeit.timeit(lambda: model.predict(data, steps=1), number=1000))
(venv)

I am trying to compare the performance of eager mode, tf2 graph mode and tf1 graph mode. And to be more precise,  I add warm-up logic for each mode.
I got the following result for 1000 iterations for each mode,
TF1 graph：1.027 s
TF2 graph：1.871 s
TF2 eager：1.926 s (not very stable)

As you can see, TF1 graph mode is the fastest one and the eager mode is the slowest one.
And then I profiled the cuda kernel,
 TF1 graph：

 Time (%)  Total Time (ns)  Instances  Avg (ns)   Med (ns)   Min (ns)  Max (ns)  StdDev (ns)     GridXYZ         BlockXYZ                                                     Name
 --------  ---------------  ---------  ---------  ---------  --------  --------  -----------  --------------  --------------  ----------------------------------------------------------------------------------------------------
     80.2      258,716,663      1,001  258,458.2  258,461.0   257,950   259,325        181.9     2   10    4   128    1    1  volta_sgemm_128x64_nn
      8.5       27,481,458      1,001   27,454.0   27,424.0    27,232    28,064        110.4    40    1    1  1024    1    1  void Eigen::internal::EigenMetaKernel<Eigen::TensorEvaluator<const Eigen::TensorAssignOp<Eigen::Ten…
      6.7       21,734,422      1,001   21,712.7   21,696.0    21,568    22,271         82.0     1   20    1   128    1    1  volta_sgemm_32x32_sliced1x4_nn
      2.8        8,932,829      1,001    8,923.9    8,928.0     8,768     9,344         61.0    40    1    1  1024    1    1  void tensorflow::BiasNHWCKernel<float>(int, const T1 *, const T1 *, T1 *, int)
      1.5        4,921,638      1,001    4,916.7    4,896.0     4,863     5,280         48.5     7    1    1  1024    1    1  void tensorflow::BiasNHWCKernel<float>(int, const T1 *, const T1 *, T1 *, int)

TF2 graph：

 Time (%)  Total Time (ns)  Instances   Avg (ns)     Med (ns)    Min (ns)   Max (ns)   StdDev (ns)     GridXYZ         BlockXYZ                                                     Name
 --------  ---------------  ---------  -----------  -----------  ---------  ---------  -----------  --------------  --------------  ----------------------------------------------------------------------------------------------------
     90.8    1,946,823,575      1,001  1,944,878.7  1,867,018.0  1,789,100  3,779,700    371,002.2    16   10    8   128    1    1  volta_sgemm_128x64_nn
      6.4      136,219,164      1,001    136,083.1    130,847.0    125,375    259,165     24,835.7     2   10    4   128    1    1  volta_sgemm_128x64_nn
      2.1       45,219,493      2,002     22,587.2     23,727.5      5,376     48,224     16,882.3    40    1    1  1024    1    1  void tensorflow::BiasNHWCKernel<float>(int, const T1 *, const T1 *, T1 *, int)

TF2 eager：

 Time (%)  Total Time (ns)  Instances   Avg (ns)     Med (ns)    Min (ns)   Max (ns)   StdDev (ns)     GridXYZ         BlockXYZ                                                     Name
 --------  ---------------  ---------  -----------  -----------  ---------  ---------  -----------  --------------  --------------  ----------------------------------------------------------------------------------------------------
     90.8    1,976,980,767      1,001  1,975,005.8  1,848,266.0  1,757,708  3,780,050    457,204.1    16   10    8   128    1    1  volta_sgemm_128x64_nn
      6.3      137,923,197      1,001    137,785.4    129,407.0    123,358    259,293     30,686.3     2   10    4   128    1    1  volta_sgemm_128x64_nn
      2.1       45,871,552      2,002     22,912.9     23,984.0      5,280     50,400     17,175.4    40    1    1  1024    1    1  void tensorflow::BiasNHWCKernel<float>(int, const T1 *, const T1 *, T1 *, int)

The sgemm kernel takes the most time.
If we step into the 'Avg', 'Min', 'Max' time of sgemm kernel for each mode, we can see tf2 kernel time is not stable, regardless eager or graph mode. Is there any reason for this behavior?
Actually, if you check the profiling result through nsight gui, you will find the kernel runs slowly in the begining and  become faster after several iterations.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Depth texture in CanvasItem shaders breaks code generation (Forward+)

Godot version
4.0.stable
System information
Windows 10, NVIDIA RTX 2060 (Driver <IP_ADDRESS>84), Vulkan 1.2.175 Forward+
Issue description
Trying to use depth textures in a canvas item as is documented (using hint_depth_texture) causes the shader compiler to throw various errors.
Editor reports:
servers/rendering/renderer_rd/renderer_canvas_render_rd.cpp:2077 - Condition "!canvas_singleton->shader.canvas_shader.version_is_valid(version)" is true.
Debugger reports many issues related to code compilation. Unfortunately, hovering over one of the errors causes Godot to accumulate many gigabytes of memory and crash the system.
Steps to reproduce
Place a CanvasLayer and a ColorRect with a shader, as is typical for post-processing.
Define a depth texture in the shader using hint_depth_texture to trigger the issue.
Minimal reproduction project
DepthIssue.zip

The depth texture is not available in canvas item shaders, it is only available in spatial shaders.
We have added an in-editor warning that is more instructive in master. https://github.com/godotengine/godot/pull/74320 it will be available in 4.0.1 and 4.1


TF Explorer Invalid node information

Tested on https://explorer.dev.grid.tf
The node information for node 7 on devnet shows that the SRU usafe is 115.63%

And for node 16 the CRU usage is 116.67%

Node 17 the CRU usage is 175%

same issue available on https://explorer.test.grid.tf/ in multiple nodes. for example:
Node 1

Node 2

Node 7


Some downloads from gitlab.com in master not working but work in 1.4

See original issue on GitLab
In GitLab by [Gitlab user @valentindavid] on Apr 8, 2020, 19:22
Summary
In master branch, downloading for example https://gitlab.com/freedesktop-sdk/mirrors/tarball-lfs-mirror/raw/master/sourceforge_download/giflib/giflib-5.2.1.tar.gz
Here is some example with master, and it does not work: https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/504045076
bst-1 branch works correctly.
Here is an example where it works with bst-1: https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/472007636
Steps to reproduce
What is the current bug behavior?
What is the expected correct behavior?
Relevant logs and/or screenshots
Possible fixes
Other relevant information

BuildStream version affected: /milestone %BuildStream_v1.x

In GitLab by [Gitlab user @cs-shadow] on Apr 22, 2020, 21:06
[Gitlab user @valentindavid] is this repeatable by any chance?
If not, I am think it's more likely to be cache expiry issue, rather than something to do with gitlab.com itself.

In GitLab by [Gitlab user @cs-shadow] on Apr 22, 2020, 21:06
[Gitlab user @valentindavid] is this repeatable by any chance?
If not, I am think it's more likely to be cache expiry issue, rather than something to do with gitlab.com itself.

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 15:31
Yes, please see. https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/538573536

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 15:31
Yes, please see. https://gitlab.com/freedesktop-sdk/freedesktop-sdk/-/jobs/538573536

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 15:33
mentioned in merge request !1897

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 15:33
mentioned in merge request !1897

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 17:09
closed via merge request !1897

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 17:09
closed via merge request !1897

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 17:09
mentioned in commit 9b42678148cdc32233ff06f06bde71d0d915021a

In GitLab by [Gitlab user @willsalmon] on May 5, 2020, 17:09
mentioned in commit 9b42678148cdc32233ff06f06bde71d0d915021a

In GitLab by [Gitlab user @jjardon] on May 10, 2020, 13:18
mentioned in merge request !1898

In GitLab by [Gitlab user @jjardon] on May 10, 2020, 13:18
mentioned in merge request !1898


Stream utility for SQLAlchemy

It should be pretty easy to iterate over a result from SQLAlchemy. It would be nice if there was an example in the user guide. We can then judge if the code is complex enough to merit a utility in the stream module.

Hey, I would like to give it a try. Can you assign me this task please ?

Done! Take your time with this. Ideally it would be nice to include an example with one or two joins :).

@MaxHalford For the function, do we assume that the user already created the engine ? (I'm learning SQLAlchemy on the fly)

Good question! I think we can use as a reference the con parameter of pandas.read_sql. Pandas is flexible and seems to allow providing an engine, a connection, or a string address. I think that providing an engine and/or a connection is good enough.
To answer your question specifically: yes, we should assume that the user is capable of creating the engine.

Hi, sorry to bother you again. I feel like I'm misunderstanding something.
My initial attempt was to:
1- Load the database.
2- For each table in the database:

Fetch all rows of the table.
For each row:

Yield it.

So, what I have to do now is to do the same thing but this time, over a query right?
Thus, the function should look like:
def iter_sql(query, con):
    ...

where I should yield one row at the time from the query ?

Hi, sorry to bother you again. I feel like I'm misunderstanding something.

Don't worry you're no bother.

So, what I have to do now is to do the same thing but this time, over a query right?

Yeah the idea is to yield the results of a query.

where I should yield one row at the time from the query ?

I'm not sure I understand your question. With SQLAlchemy it's possible to loop over the results of a query as they are processed by the database. In other words, instead of called query_result.all(), you simply have to loop over the result because it's a generator. There are many examples of this online.

Thanks for you quick answer ! I'll take a look


Rename master -> main

Proposal: Rename master to main
In Order avoid unnecessary references to slavery in the field of computer science more and more associations, companies, etc. replace the term master with a more inclusive one.  For instance, a month ago Github released a feature that simplifies changing the name of a branch. <- Renaming
Therefore my suggestion would be to change the name of the master branch to main.

Hi @fynn-lemburg
This is on the team's  list of things to do, closing as a duplicate of https://github.com/flutter/flutter/issues/59445
Thank you


MB-9322: Fix console warnings emitted by client tests

Description
As part of the TRA epic to clear out console warnings/errors during front-end test runs, this pull request fulfills a ticket to resolve emitted messages from the following test fixtures:

src/components/form/fields/DutyStationInput.test.jsx
src/pages/MyMove/Profile/DutyStation.test.jsx

Reviewer Notes
Ensure that all client and integration tests pass, and that the edited test files do not emit any console messages during testing. There should be no perceptible UI changes to the user.
Setup
yarn test DutyStation

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Part of epic MB-8944.
Closes MB-9322.

Messages

:book:
:link: MB-9322

  Generated by :no_entry_sign: dangerJS against 97540c029b229b8f88b64e40e56683c792a40c6e


Why is this plugin not verified by homebridge team?

Best not to use until we get an answer. Thanks for calling attention to this.

Honestly I didn't even realize there was a verification program, and this project has been in maintenance mode for a while now since Wiz came out with MATTER support

I tried switching my Wiz bulbs to Matter, and went back to using the plugin. They’re more reliable and support adaptive lighting. Wouldn’t consider the availability of Matter to mean the plugin is obsolete.

Just submitted a verification request https://github.com/homebridge/plugins/issues/776


Update table_picker.dart

@CodiumAI-Agent /review

PR Analysis

🎯 Main theme: Implementing a new widget for editing tables in Dart.
📝 PR summary: This PR introduces a new widget, EditTableDropDown, in Dart. This widget allows users to edit a table, including adding and removing rows and columns. The PR also includes the implementation of the EditTableEnum, which provides various options for editing the table.
📌 Type of PR: Enhancement
🧪 Relevant tests added: No
⏱ Estimated effort to review [1-5]: 4, because the PR introduces a substantial amount of new code, which includes a new widget and enum. The complexity of the code and the lack of tests increase the effort required for review.
🔒 Security concerns: No security concerns found

PR Feedback

💡 General suggestions: The PR introduces a new widget which is a good enhancement. However, it lacks tests which are crucial for validating the functionality of the new widget. It is recommended to add relevant tests. Also, it would be beneficial to add error handling and null safety checks where necessary.

 🤖 Code feedback:

relevant file: lib/src/widgets/table_picker.dart
suggestion: It seems like the imagePath variable is not defined in the _getEditTableItem function. This could lead to a runtime error. Please define the variable before using it. [important]
relevant line: String value = ""

relevant file: lib/src/widgets/table_picker.dart
suggestion: The MediaQuery.of(context).size.width < 480 condition is used multiple times in the code. It would be more efficient to calculate it once and store it in a variable at the beginning of the build method. [medium]
relevant line: if (MediaQuery.of(context).size.width < 480) {

relevant file: lib/src/widgets/table_picker.dart
suggestion: The _editTableListView function has a lot of nested widgets which makes it hard to read. Consider breaking it down into smaller, more manageable functions. [medium]
relevant line: Widget _editTableListView(bool isMobileView, BuildContext context) {

How to use
  Instructions

Tag me in a comment '@CodiumAI-Agent' and add one of the following commands:
/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs: Generate docstring for new components introduced in the PR.
/generate_labels: Generate labels for the PR based on the PR's contents.
see the tools guide for more details.

To edit any configuration parameter from the configuration.toml, add --config_path=new_value.
For example: /review --pr_reviewer.extra_instructions="focus on the file: ..."
To list the possible configuration parameters, add a /config comment.

@CodiumAI-Agent /improve

@CodiumAI-Agent /ask Is this changes looks correct ?

Question: Is this changes looks correct ?
Answer:
The changes in the PR look mostly correct. The developer has introduced a new widget EditTableDropDown in Dart, which allows users to edit a table, add or remove columns, rows, etc. The widget seems to be well-structured with clear comments and appropriate usage of Dart's features.
However, there are a few things that need to be addressed:

There is a missing semicolon on the line String value = "". It should be String value = "";.

The variable imagePath is used in the _getEditTableItem method but it is not declared anywhere in the method or the class. This will cause a compilation error. The developer needs to declare imagePath before using it.

The PR does not include any tests. It would be good to add some unit tests to verify the new functionality works as expected.

The PR does not include any updates to the documentation. It would be helpful to update the documentation with the new widget and its usage.

Please address these issues and resubmit the PR.

 ✨ Usage guide:
Overview:
The ask tool answers questions about the PR, based on the PR code changes.
It can be invoked manually by commenting on any PR:
/ask "..."

Note that the tool does not have "memory" of previous questions, and answers each question independently.

  More PR-Agent commands

To invoke the PR-Agent, add a comment using one of the following commands:

/review: Request a review of your Pull Request.
/describe: Update the PR title and description based on the contents of the PR.
/improve [--extended]: Suggest code improvements. Extended mode provides a higher quality feedback.
/ask <QUESTION>: Ask a question about the PR.
/update_changelog: Update the changelog based on the PR's contents.
/add_docs 💎: Generate docstring for new components introduced in the PR.
/generate_labels 💎: Generate labels for the PR based on the PR's contents.
/analyze 💎: Automatically analyzes the PR, and presents changes walkthrough for each component.

See the tools guide for more details.
To list the possible configuration parameters, add a /config comment.

See the ask usage page for a comprehensive guide on using this tool.


Is FlaUI Supports Jenkins Integration

Hi Team,
The tests what we have written using falUI can be executed with jnekins pipeline? Can some one please explain how this can be done.
Thanks
Vasanthi

Hi,
we are using this in our automated tests in jenkins pipelines. So it is definitely possible :)
The biggest hurdle was to ensure that the framework has a desktop session (as mentioned also in the wiki)
We are using RealVNC for this: we have some VMs acting as Jenkins-Nodes which have the RealVNC-service installed and running and the pipeline then can start RealVNC-Viewer to open a client-session to localhost.

Hi,
we are using this in our automated tests in jenkins pipelines. So it is definitely possible :)
The biggest hurdle was to ensure that the framework has a desktop session (as mentioned also in the wiki)
We are using RealVNC for this: we have some VMs acting as Jenkins-Nodes which have the RealVNC-service installed and running and the pipeline then can start RealVNC-Viewer to open a client-session to localhost.

@Vasanthi30 you are asking is the test you have written can be executed in jenkins, best would be to try it out, and if it is not working for some reasons also state the errors/behavior you encounter.

Yes, FlaUI works perfectly from within Jenkins. If you have concrete questions on what you tried and what does not work, feel free to ask.


Is there a method like ydf.load_model() to load model get a instance of tfdf.keras.RandomForestModel?

In YDF https://ydf.readthedocs.io/en/latest/tutorial/getting_started/#save-model
'''
Save model
Finally, we use the same model for later use.
model.save("/tmp/my_model")
So we can load the model with:
loaded_model = ydf.load_model("/tmp/my_model")
print(f"This is a {loaded_model.name()} model.")
'''
I don't find a method to load model to get a instance of tfdf.keras.RandomForestModel.
I found when I use dtreeviz to get some visual information I must retrain the model again. This is so frustrating. Is there a better way?

Hi,
due to limitations of the Keras API, it's not possible to obtain the same model after saving and re-loading. I know this is very frustrating, and it's one of the reasons using ydf instead of tfdf can be a good idea. I don't think dtreeviz has support for ydf yet, but adding it should be easy - maybe open a FR in their repository and cc me to it?

Thanks.


How to change the color of the line?

Mine is yellow.  I updated the config with hyperline: {color: 'black'}.  Still yellow. How can I change the color?

In your ~/.hyper.js
css: `
  .line {
    background: #000000;
  }
`


fixing broken links

Broken custom-routers link in custom-navigation page

Thanks react-navigation-bot 🤖  .. the change looks all good to me!

Thanks!


Fix Alpine-Nextcloud: Bump PHP Version to 8.3

🛠 Note:
We are meticulous about merging code into the main branch, so please understand that pull requests not meeting the project's standards may be rejected. It's never personal!
🎮 Note for game-related scripts: These have a lower likelihood of being merged.

✍ Description
Provide a summary of the changes made and/or reference the issue being addressed.
The newest nextcloud install automaticaly installs PHP 8.3. It fails at this point: sed -i -E '/^php_admin_(flag|value)\[opcache/s/^/;/' /etc/php82/php-fpm.d/nextcloud.conf
I´ve bumped the PHP Version to 8.3 and also included  $STD apk add nextcloud-occ and chown -R nextcloud:www-data /usr/share/webapps/nextcloud/ to get the occ command to work correctly again. It would not work without this to changes since the Release of nextcloud 29.0.4. This is the Version where occ becomes its own package:

Related Issue: #813

🛠 Type of Change
Please check the relevant options:

[X] Bug fix (non-breaking change that resolves an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change unexpectedly)
[ ] New script (a fully functional and thoroughly tested script or set of scripts)

✅ Prerequisites
The following steps must be completed for the pull request to be considered:

[X] Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.)
[X] Testing performed (I have thoroughly tested my changes and verified expected functionality.)
[ ] Documentation updated (I have updated any relevant documentation)

📋 Additional Information (optional)
Provide any extra context or screenshots about the feature or fix here.

Sorry forgot the link: bash -c "$(wget -qLO -https://raw.githubusercontent.com/michelroegl-brunner/ProxmoxVE/refs/heads/DEV/ct/alpine-nextcloud.sh)"
But i can also merge it into dev here if you want :)

I also changed mysql_install_db to maridb-install-db because the mysql version will be deprecated in future versions.


Authorization_RequestDenied

A week ago, I have stuck with this error:
Authorization_RequestDenied
Insufficient privileges to complete the operation.
Please check that your account has sufficient access to the Microsoft App Registration Portal link below.
This FAQ helped me: https://docs.microsoft.com/en-us/bot-framework/bot-service-troubleshoot-general-problems#why-do-i-get-an-authorizationrequestdenied-exception-when-creating-a-bot
But this time, I deleted the old bot, because I want to create a new bot.
I'm getting the same error. Did I miss something?
Thank you in advance.
Joy.

Oh! It's works now.

Hi @p-phornthip
Did you sign out, and back in?  What solved this issue for you?


Remove slf4j and Use LogService

Even though using slf4j benefits users to avail several opportunities, it could be considered as a strict dependency. It is more lightweight if we could use LogService instead.

According to [1], it seems to be that SLF4J is a good choice for this.
[1] - https://ekkescorner.wordpress.com/2009/09/03/osgi-logging-part-2-logservices-vs-classic-logger/


Send out the 2024 Design System UX survey

What
Create list of questions for 2024 survey to understand user's experience of using the design system and send it out by 19 Jan.
Why
To decide whether we want to repeat questions form previous surveys or discover something new.
Who needs to work on this
User researcher
Who needs to review this
Product manager
Done when

[x] Ask the team to look through last year's UX survey in order to source new questions
[x] Scope what we want to achieve
[x] We have a list of questions
[x] The survey has been drafted
[ ] The survey has been sent out

Greetings from the Carbon Design System! We're big fans of the gov.uk design system over here and as I was cruising through your site and now your Github, I ran across this issue and was super interested in learning more about the kinds of questions you ask as I'm tackling an identical problem with a survey of our own. Happy to jump on a call at your convenience to chat more if that is of interest to you.

Hi @sstrubberg! Great to hear from you. Drop me a line through<EMAIL_ADDRESS>and I'll set up a chat.

I've had a look at the draft and it looks pretty good!

Sent out on 19 Jan.


Fix typo in README (params_with_assocs)

There was a typo in the README in the params_with_assocs section.

Good catch. Thank you @philipgiuliani!

Wow. Fastest merge i've every seen. Thanks!


[AVM Module Issue]: module should allow for optional AppGw ingress controller

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
(Optional) Module Version
0.1.0
(Optional) Correlation Id
No response
Description
Many production deployments require some form of defense in depth for internet facing workloads.  If using Azure components, that typically translates to an app gateway with WAF in front of Kubernetes.
I think this would work best as an example illustrating how to 'bring your own AppGw', and a small adjustment to the module to accommodate this as a toggle variable.

We are using Azure CNI Overlay in this module which is not compatible with Application Gateway.
https://learn.microsoft.com/en-us/azure/aks/azure-cni-overlay?tabs=kubectl#limitations-with-azure-cni-overlay


Create a clean ReactJS front end running on GitHub pages

we'll move current master branch to a new branch, and use master branch to check in working, tested code.

this is done in the mvp-react-version branch


feat: add tekton chains integration

Signed-off-by: osamamagdy<EMAIL_ADDRESS>

/cc @ankitm123
Tried to search the code base but still couldn't figure out how this will control the installation of the chart.


Release 0.6.5

Try out the new automated changelog and versioning.
I've been having some trouble with npm installing dependencies for standard-version incorrectly.
I don't get issues when using yarn instead so I've switched to that:
npm i -g yarn
Then yarn to install deps, instead of npm i
Yarn is quicker and in my experience manages its lock file better than npm. If we agree with using yarn I'll do a cleanup of npm's lock files after this release.

Hold off until #82 is fixed.

Good to go.

Suggested flow for this is:

Merge development into master
npm run release
npm run publish
Merge master into development

Not sure we need a pull request for stage 1 as each fix has been individually reviewed, but it might be good practice, thoughts?

I think that we need to create a PR for 1 to maybe discuss something and check it with Travis for the last time before release.

So I've published it to the NPM but I've got an error
arkadii@arkadii-S551LN:~/Dropbox/OpenSource/ng2-image-upload$ npm run publish

><EMAIL_ADDRESS>publish /home/arkadii/Dropbox/OpenSource/ng2-image-upload
> git push --follow-tags origin master && npm publish

Counting objects: 5, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (5/5), done.
Writing objects: 100% (5/5), 706 bytes | 0 bytes/s, done.
Total 5 (delta 2), reused 0 (delta 0)
remote: Resolving deltas: 100% (2/2), completed with 2 local objects.
To git@github.com:aberezkin/ng2-image-upload.git
   6260ee9..1873054  master -> master
 * [new tag]         v0.6.5 -> v0.6.5

><EMAIL_ADDRESS>publish .
> git push --follow-tags origin master && npm publish

Everything up-to-date
npm ERR! publish Failed PUT 403
npm ERR! code E403
npm ERR! "You cannot publish over the previously published version 0.6.5." : angular2-image-upload

And also, I've found out that I forgot to build it again before publishing. Maybe we will add build step at into release script before standart-version?

Ah - I was intending npm run publish to bundle the npm publish and the git push, if you don't want to use it like that just unlink them in package.json.
Adding build first seems safe yeah.

It also looks like the changelog wasn't correctly generated - I'm guessing the squashed commit from dev to master did it. I was assuming that things like fix()... in the commit body would work, but maybe it's titles only, oh well - can try again next time.

FYI - added some documentation via CONTRIBUTING.md and the publish script now also runs the build.


UAT | Join a case

Current behaviour:
Join a case is only possible after admitting a case.
Expected behaviour:
User should be able to join a case once scrutiny is completed (the case is validated by FSO and has moved to judge for admission

cc @atulgupta2024 @subhashini-egov please see if this is requires a design change
i am not aware how validation for join a case is designed. can you please confirm

@Taherabharmal this should be do-able. @manimaarans please take note.


Display Facebook URL field for events

For some reason this isn't showing up on the event form any more... we should investigate & fix this!

Oh hey, this was totally closed with the merge of #576!


Add new exercise accumulate

As described in #94

Only one left for tonight!
Great job @matthewmorgan!

Killin' it! This is great.

@kytrinyx I love your encouraging phrases. The most imaginative I usually come up with is a plain Thanks. I think I need to build a list and I pick one at random.

I'm sure there's a bot for that :)
I just get really excited to see so much activity in a track that has been practically orphaned for so long. The two of you have a really great rhythm going--good discussions and lots of really great work. It is lovely to see.

Yeah, I'm enjoying it a lot.

@kytrinyx I looked for a bot but didn't find it!
@rchavarria Thanks to both of you-- having fun and learning a lot.  👊

:)

It's not exactly a bot, and it doesn't generate the most appropiate compliments for Github contributions, but it's fun: ComplimentBot 4000

Hah.


Remove version: Jackett.Jackett version 0.21.205

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.21.205/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Zds 4 navbar

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

https://github.com/zopaUK/react-components/pull/422

Couple of things I noticed while playing with it
I'm aware that it's not easy to make it completely fluid using js but I'm wondering whether we can improve the responsiveness a bit

Also, the fact that we're using react-headroom leads to two edge cases that probably need better handling
When you open the dropdown and start scrolling, this is what it looks like


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit 2b4864a:
:clock10: Full build: the average full build time is 6 min(s) 7 sec(s), this is based on the last 270 build(s) for this repository.
:warning: This is a FULL build due to a configuration change (usually in .openpublishing.publish.config.json and/or docfx.json). It is strongly suggested to merge this configuration change after the business hours, as it could impact the overall build performance of other PRs on the 'pr-basebranch' branch. For more detailed info, please click here.

Docs Build status updates of commit 2b4864a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow support user to add a school in claims

Context
We need to allow a support user in claims to add a school. They will be able to search through all Gias schools and link a Gias school to a school record in the claims domain.
This uses the school.js autocomplete introduced here #73
Refactored the SchoolSuggestions controller to be a shared controller https://github.com/DFE-Digital/itt-mentor-services/pull/80#discussion_r1442793084
Changes proposed in this pull request
New controller actions in claims schools controller
New school suggestions controller
New Views
New Routes
Guidance to review

Go to claims.localhost:3000/personas
Sign in as Colin
Click Add organisation and fill in the form
Check if you're organisation is added to the school index page

Screenshots
https://github.com/DFE-Digital/itt-mentor-services/assets/11318084/80fa4462-10eb-4010-9bda-84f48152a874

 As a Support User, I want to be able to “Add a new School”

Nice one!


Fix for shape mismatch during SPMD tracing.

I'll explain tomorrow.

gpu tests failing , fixing now.


got far but stuck on insert stmt support

Overview of changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
Please indicate the type of change: bug(fix), new feature, tests, config/infrastructure, docs, etc.
Be sure to link to that issue and place it in the name of the PR.
For software test
Describe any pertinent instructions to get this running or vulnerable test areas.
Additional details

additional notes, things that were changed, areas of concern, etc.*

Important Reminders

We like PRs to be less than 300 LOC
Run the pre-commit for both hook stages! (pre-commit and pre-push)
Add tests! Do not forget that we use pytest here at Narrative Science. 80% Code Coverage recommended.
Use proper mocks in tests. Do not write tests without mocks that test external libraries or external endpoints.
Add necessary documentation to README's
Notion page with our detailed PR guidelines

Tests failed!


Howdy opencv module not importing

_Please describe the issue in as much detail as possible, including any errors and traces.
Hello, i am using: https://aur.archlinux.org/packages/howdy-beta-git
The issue i am getting is with the opencv-python module, a dependency of howdy
_If your issue is a camera issue, be sure to also post the image generated by running sudo howdy snapshot
[SpiderUnderUrBed@DaSpiderCave ~]$ sudo howdy add
[sudo] password for SpiderUnderUrBed:
Traceback (most recent call last):
  File "/usr/lib/howdy/cli.py", line 98, in <module>
    import cli.add
  File "/usr/lib/howdy/cli/add.py", line 13, in <module>
    from recorders.video_capture import VideoCapture
  File "/usr/lib/howdy/recorders/video_capture.py", line 6, in <module>
    import cv2
ImportError: libabsl_die_if_null.so.2301.0.0: cannot open shared object file: No such file or directory

I've searched for similar issues already, and my issue has not been reported yet. ✅
Linux distribution (if applicable): Arch
Howdy version (sudo howdy version):
Howdy 3.0.0 BETA

Did you try to re-install python-opencv? It seems to be an issue not on Howdy side.


Remove --no-single-branch to optimise speed for some repos

Some of the plugins I used toke quite long compared to the other ones since these ones have quite a few branches. I saw that vim-plug uses --no-single-branch as a flag. Is this required by something? Otherwise I suggest to remove this. It'll make cloning faster for some repositories.

Type:

[ ] Bug
[x] Enhancement
[ ] Feature Request
[ ] Question

OS:

[ ] All/Other
[ ] Linux
[x] OS X
[ ] Windows

Vim:

[x] Terminal Vim
[ ] GVim
[ ] Neovim

Is this required by something?

Yes, we decided that we need it to support switching of branches offline, so we won't remove it. I suggest that you remove the option from your own copy of plug.vim if you really want it removed. vim-plug is extremely stable, so you don't have to worry about keeping in sync with the upstream.


[BUG][UI] Creating new cloud credentials results in t is null

Rancher Server Setup

Rancher version: v2.7.6
Installation option: Helm chart

If Helm Chart, Kubernetes Cluster and version: RKE2

Information about the Cluster

Kubernetes version: v1.26.8
Cluster Type (Local/Downstream): local

User Information

What is the role of the user logged in? user-base

Describe the bug

When my test user (that has fairly low privileges) tries to create new Cloud Credentials, the user is presented with a blank page and the error "t is null".
To Reproduce

Log in as user foo
Go to: Cluster Management > Cloud Credentials
Click "Create" (which gets you to rancher.example.com/dashboard/c/_/manager/cloudCredential/create)

Screenshots

Additional context
Here's the error message from my browser's debug console:
t is null console.js:31:30
    e console.js:31
    e client.js:111
    Babel 7
    errorHandler client.js:113
    VueJS 15
        $n
        Bn
        _render
        r
        get
        run
        Zr
        Zn
        Wn
        (Async: promise callback)
    qn
        Zn
        ni
        update
        notify
        set
    t fetch.client.js:91
    Babel 11
    f fetch.client.js:67
    p fetch.client.js:58
    u fetch.client.js:29
    VueJS 30
        Hn
        Hr
        Fr
        $mount
        init
        v
        d
        T
        I
        pa
        _update
        r
        get
        run
        Zr
        Zn
        Wn
        (Async: promise callback)
    qn
        Zn
        ni
        update
        $forceUpdate
        l
        h
        2b0e/</$/<
        (Async: promise callback)
    In
        pi
        Fn
        Rn
        _c
    n index.vue:1
    VueJS 13
        _render
        r
        get
        run
        Zr
        Zn
        Wn
        (Async: promise callback)
    qn
        Zn
        ni
        update
        notify
        set
    t fetch.client.js:91
    Babel 2
        p
        S

This will be the result of a developer error page mistakenly being shown in production and is resolved in 2.8 and the next feature release of 2.7 (for tracking see https://github.com/rancher/dashboard/issues/9542)
There will be an underlying error, but after upgrading if there are functional issues please just let us know via a fresh gh issue.


fix(types): narrow down the return type of defineConfig

Description
Fixes compatibility with mergeConfig, as reported in https://github.com/vuejs/create-vue/issues/313
Before the change, defineConfig would return UserConfigExport, which is a union type. This would cause mergeConfig to fail, as it doesn't accept the UserConfigFn type in the union. (https://github.com/vitejs/vite/pull/13135)
After the change, defineConfig returns the same type that it receives, so it would exclude UserConfigFn from the return type whenever possible.
Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

/ecosystem-ci run

Only Qwik is failing compared to main. Iles and Histoire are known issues.

Update: I can confirm this is affected by the ["strictFunctionTypes"](https://www.typescriptlang.org/tsconfig#strictFunctionTypes) configuration.
No tsconfig is configured for the config.spec.ts file, therefore it's not under strict mode, thus the local tests missed the bug.

FWIW this was also attempted at https://github.com/vitejs/vite/pull/12021 but reverted in https://github.com/vitejs/vite/pull/12077 also because of Qwik (and VitePress).

Further tests show that the as any casting in the Qwik config may be the culprit. https://github.com/BuilderIO/qwik/blob/190f0bcd7593745b58aef07054547551253d087b/packages/qwik-worker/vite.config.ts#L42
Removing that fixes the error.
I also managed to use the in out operator to fix the issue without modifying the downstream code, but ended up breaking some other use cases… But maybe it could work with more overloads. Let me try again.

/ecosystem-ci run


Grapher multiple columns plot

Originally posted by gemGreg January 31, 2022
In the old version of PandasGUI there was a way to plot multiple signals on one line plot.

Is there a way to do it in the newest version of PandasGUI?
It seems like In the newest version you can drag/drop only one signal to X and Y:

Yep I have the same problem on version 0.2.13. Seems like an important feature to be able to plot multiple columns in the same plot.

do we have any update on when this issue could be fixed?

One workaround available now at least via the repo is to use custom kwargs to override y with an array of columns. It isn't as easy perhaps but doable. So in @adamerose 's case, add y with a value of ['Australia New Cases', 'Brunei New Cases', ...].

Below is what I think a better workaround until this gets fixed.
My use case for PandasGUI is almost exclusively comparing and analyzing time series data. Being able to plot multiple columns is a "must have" feature, at least for line and scatter plot types.  I was already making some changes to read couple more file formats, and made the below changes for the line and scatter plot types to get that feature back.
file -> jotly.py
function -> line and scatter
original argument-> y: ColumnName = None,
changed argument -> y: ColumnNameList= None,
Essentially, it changes the class of argument y from ColumnName to ColumnNameList


When I build the MixedRealityToolkit-Unity sample project, I faced the following error

Assets\MixedRealityToolkit.Providers\WindowsMixedReality\WindowsMixedRealityArticulatedHand.cs(418,73): error CS0246: The type or namespace name 'HandJointKind' could not be found (are you missing a using directive or an assembly reference?)
Assets\MixedRealityToolkit.Providers\WindowsMixedReality\WindowsMixedRealityArticulatedHand.cs(111,17): error CS0246: The type or namespace name 'HandMeshObserver' could not be found (are you missing a using directive or an assembly reference?)
Assets\MixedRealityToolkit.Providers\WindowsMixedReality\WindowsMixedRealityArticulatedHand.cs(382,33): error CS0246: The type or namespace name 'HandJointKind' could not be found (are you missing a using directive or an assembly reference?)
Assets\MixedRealityToolkit.Providers\WindowsMixedReality\WindowsMixedRealityArticulatedHand.cs(412,26): error CS0246: The type or namespace name 'JointPose' could not be found (are you missing a using directive or an assembly reference?)

See #3844. Looks like you need to make sure you are on Windows 18362.

Hi @keveleigh
For Hololens 1, is it needed to be on Windows 18362?

@Cris21395 I think it is a requirement for MRTK v2 -- at least as of the RC1 when they merged the code in for the HL2 -- so it impacts HoloLens 1 as well. I had the same issue and it was resolved by updating the Windows version.
But @keveleigh can weigh in if otherwise.

Thanks a lot @genereddick for your prompt reply. I thought that updating the windows version could resolve my problem, but I wanted to be sure before doing it.

Hi @keveleigh
For Hololens 1, is it needed to be on Windows 18362?
Thankyou @keveleigh

@Cris21395 I think it is a requirement for MRTK v2 -- at least as of the RC1 when they merged the code in for the HL2 -- so it impacts HoloLens 1 as well. I had the same issue and it was resolved by updating the Windows version.
But @keveleigh can weigh in if otherwise.

Thank you @genereddick

Same problem here on HoloLens 1 still, despite running on Windows 18362.30

@genereddick is correct, the 18362 SDK is a compile time requirement for MRTK v2. This is necessary to enable support for HoloLens 2 features.
At runtime, the applications will run on HoloLens 1 without need for a new operating system.

See @keveleigh's comment on #3844. Looks like you need to make sure you are on Windows 18362.

Now, I am udate window 18875 but not solve the problem

So, installing the 18362 SDK using the insider program do make the build success. the problem now is that that the Hololns1 don't have this SDK version, an apparently can't download or install it.
Does anyone know what is the trick here?
we need to build using  18362 SDK and that's fine, but does the Hololens1 also need to have it install on it OS? if yes, how can we install it?

@Ariel-Feldman, you should not need 18362 on the HoloLens 1. When you run your application, MRTK will ask Windows 10 if an API (ex: eye tracking) is supported on your version of Windows 10. If not, the API will not be called.
If you have any trouble or questions, please do not hesitate to ask on stack overflow (tag: mrtk).

the solutions in here, straight from the queen, mother of dragons and breaker of chains...of Hololens2 []-) Julia Schwarz
https://stackoverflow.com/a/55626479/10104036

Assets\MRTK\Providers\WindowsMixedReality\Shared\WindowsMixedRealityArticulatedHandDefinition.cs(26,17): error CS0246: The type or namespace name 'HandMeshObserver' could not be found (are you missing a using directive or an assembly reference?)
Assets\MRTK\Providers\WindowsMixedReality\Shared\WindowsMixedRealityArticulatedHandDefinition.cs(29,17): error CS0246: The type or namespace name 'HandMeshVertex' could not be found (are you missing a using directive or an assembly reference?)
Build completed with a result of 'Failed' in 6 seconds (5770 ms)
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
UnityEditor.BuildPlayerWindow+BuildMethodException: 3 errors
at UnityEditor.BuildPlayerWindow+DefaultBuildMethods.BuildPlayer (UnityEditor.BuildPlayerOptions options) [0x002bb] in :0
at UnityEditor.BuildPlayerWindow.CallBuildMethods (System.Boolean askForBuildLocation, UnityEditor.BuildOptions defaultBuildOptions) [0x00080] in :0
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)
3 Error in  Unity2019.4.13 Build

Which SDK are you trying to build with in the Unity build settings? That looks like an issue with a too-old version. You'll want to use 18362 or higher.

I got same issue with my hololense 2 (updated) and latest W11 update. Any idea?

An error occurred while resolving packages:
Project has invalid dependencies:
com.atteneder.gltfast: Error when executing git command. fatal: unable to access 'https://github.com/atteneder/glTFast.git/': Failed to connect to github.com port 443 after 21097 ms: Couldn't connect to server
com.atteneder.ktx: Error when executing git command. fatal: unable to access 'https://github.com/atteneder/KtxUnity.git/': Failed to connect to github.com port 443 after 21110 ms: Couldn't connect to server

com.microsoft.mixedreality.visualprofiler: Error when executing git command. fatal: unable to access 'https://github.com/microsoft/VisualProfiler-Unity.git/': Failed to connect to github.com port 443 after 21092 ms: Couldn't connect to server

org.mixedrealitytoolkit.data: The file [E:\Unity\org.mixedrealitytoolkit.data\package.json] cannot be found


Migrate from screen to wakelock

As I explained here, screen lacks maintenance, which causes some problems. Migrating to wakelock resolves these problems.

Thanks man


Sort the output of get_icon_names alphabetically

Now the icons are sorted alphabetically, and the users icon overrides are taken into account (by default). If get_icon_names is called with "original" as parameter, it prints the original icons.

Great idea, @dritter. Should be very useful as a debug tool.


Logbook statistics not visible for some users

Testet with market and nightly. Premium member, cleared data, default settings. On the logbook page the count at the top is missing. No errors in the log. I was able to reproduce it with all caches on this device (a Samsung, don't know the exact model, but it was white :D ), with mine (HTC Desire) everything is fine.

Did you use your account on the Samsung as well as on the HTC ?
To understand:
You are referring to the find-count shown for each user who logged the cache ?

Sadly, no. I tested it only with his account.
I mean this count:

16x Found it, 2x Didn't find it, 3x Write note

Now I understand.. I admit I never carefully looked at this count but will do for further testing.
I changes the topic a little, hope you don't mind.

I tested it with 10+ caches. Log said "already English".

The only case I know of where this statistics is missing is after having imported the caches via GPX.
But after updating them the statistics are there.

Hm, very strange. The RegEx used here <p(.+?)
 and the code in cgBase ~line 1061 seems to be very stable.

Corrects. That's why I was confused about this issue. I hope that @Lineflyer is able to reproduce it with his testing-devices, maybe it's a bug related to Samsung's Android-modifications.

A few days agao i tested on a Samsung Galaxy the login.... no avatar has been displayed there. Maybe there is really something different on Samsung device.

I am using a Samsung Galaxy s2 and everything is well.
Can you assure it is not related to PQ import ?

No. Clean database and fresh nightly install. Was the same with the market-version before.

Have tested this (clean install with new nightly) now with my SGS2 and did not find any problem up to now.
Furthermore looking on the website I do not see any setting/feature which influences the statistic-display of the logbook.
Also in case you are not logged in still the statistic is shown.
Can you ask for the exact device type ?
But I doubt, that this could be device specific.


Fix ErrorWriter connect GET protocol classifier

ErrorWriter will now correctly classifying connect GET requests if the connect protocol version header is present or the connect protocol version is set in the URL query parameters. To handle connect GET requests without the protocol version the ErrorWriter will write errors as connect unary errors if the protocol is unknown. This ensures the error is always written to the client. The connect unary error payload is always in JSON and therefore makes a nice human-readable format for the caller. Clients are still recommended to check IsSupported first to ensure the protocol will match the desired and to provide fallback to other error writers if desired.

It appears this would also address #637. I think the only potentially controversial part in here is the change to always fall back to the Connect protocol's JSON error format output even when the incoming protocol is cannot be determined. I'm okay with it, but am curious if @akshayjshah or @mattrobenolt have objections.

I think this is fine. This behavior wouldn't affect us. I use ErrorWriter.IsSupported() to spit back a plaintext error, and we don't use GET requests for anything.


Some test covering upgrade from Camel 4.3 to 4.4 are not parsed correctly

In test covering upgrade of Camel 4.3 to 4.4 are test verifying removal of xpath, xquery and tokenize methods from routeBuilders. tokenize tests work, but the other fails -> because openrewrite can not determine result type of the method. This should be investigated.

Fixed by https://github.com/quarkusio/quarkus-updates/pull/145


Prevent redeferring a function body if GC happens while creating a function object of that function body

This change is

LGTM- I'd move the new class out of the function though- I can't imagine this is the only place to guard against redeferral?


Time Remaining Not Loaded When Calibrating

The time remaining for the print gets updated to a near null value when the print is just starting (during the calibration stage). Just add some logic so then if the print time is greater than 30 days in seconds, replace with another value/message.

Data from /job request when printer is preparing
{"id":62,"state":"PRINTING","progress":0.00,"time_remaining":4294967236,"time_printing":52,"file":{"refs":{"icon":"/thumb/s/usb/TESTYC~1.BGC","thumbnail":"/thumb/l/usb/TESTYC~1.BGC","download":"/usb/TESTYC~1.BGC"},"name":"TESTYC~1.BGC","display_name":"testycube_0.4n_0.2mm_PLA_MK4_3m.bgcode","path":"/usb","size":80166,"m_timestamp":1705608860}}


[occm] Multi region openstack cluster

What this PR does / why we need it:
Openstack CCM multi region support, if it has one Identity provider.
Which issue this PR fixes(if applicable):
fixes #1924
Special notes for reviewers:
CCM config changes:
[Global]
auth-url=https://auth.openstack.example.com/v3/
region=REGION1
# new param 'regions' can be specified multiple times
regions=REGION1
regions=REGION2
regions=REGION3

Optionally can be set in cloud.conf
clouds:
  kubernetes:
    auth:
      auth_url: https://auth.openstack.example.com/v3
    region_name: "REGION1"
    regions:
      - REGION1
      - REGION2
      - REGION3

During the initialization process, OCCM checks for the existence of providerID. If providerID does not exist, it defaults to using node.name, as it did previously. Additionally, if the node has the label topology.kubernetes.io/region, OCCM will prioritize using this region as the first one to check. This approach ensures that in the event of a region outage, OCCM can continue to function.
In addition, we can assist CCM in locating the node by providing kubelet parameters:

--provider-id=openstack:///$InstanceID - InstanceID exists in metadata
--provider-id=openstack://$REGION/$InstanceID - if you can define the region (by default meta server does not have this information)
--node-labels=topology.kubernetes.io/region=$REGION set preferred REGION in label, OCCM will then prioritize searching for the node in this specified region

Release note:

NONE

@mdbooth can you take a look on this PR.
Probably I need to add more configuration checks.
Thanks.

Hi @sergelogvinov
I propose an implementation of multi cloud support for cinder-csi-plugin, which offer multiple openstack clusters support, not only multiple regions, I haven't take look of occm implementation yet, but is it possible to adapt it to support multiple cloud definitions instead of only multiple regions ?

Hi @sergelogvinov I propose an implementation of multi cloud support for cinder-csi-plugin, which offer multiple openstack clusters support, not only multiple regions, I haven't take look of occm implementation yet, but is it possible to adapt it to support multiple cloud definitions instead of only multiple regions ?

Thank you for this PR, it is very interesting. Can we have a call/chat in slack #provider-openstack (Serge Logvinov)?

/ok-to-test

I've rebased the PR. all tests passed and i've tested manually too
Can you take a look please @jichenjc @mdbooth
It will be great to merge this change into the upcoming release...
Thanks.

Is anything else we can do here? @jichenjc @mdbooth @kayrus
We had conversation how we need initialize the openstack clients
	for _, region := range os.regions {
		opt := os.epOpts
		opt.Region = region

		compute[region], err = client.NewComputeV2(os.provider, opt)
		if err != nil {
			klog.Errorf("unable to access compute v2 API : %v", err)
			return nil, false
		}

		network[region], err = client.NewNetworkV2(os.provider, opt)
		if err != nil {
			klog.Errorf("unable to access network v2 API : %v", err)
			return nil, false
		}

It seems to be a similar process to the one we followed in cinder-csi-plugin.
I believe @MatthieuFin and I can introduce multi OpenStack authentication support after this PR.
    [Global]
    auth-url="https://auth.cloud.openstackcluster.region-default.local/v3"
    username="region-default-username"
    password="region-default-password"
    region="default"
    tenant-id="region-default-tenant-id"
    tenant-name="region-default-tenant-name"
    domain-name="Default"

    [Global "region-one"]
    auth-url="https://auth.cloud.openstackcluster.region-one.local/v3"
    username="region-one-username"
    password="region-one-password"
    region="one"
    tenant-id="region-one-tenant-id"
    tenant-name="region-one-tenant-name"
    domain-name="Default"

Thanks.

/retest

/test openstack-cloud-controller-manager-e2e-test

Thank you @kayrus for refactoring of instance.go (instance-v2).
I’ve moved all my changes to instance.go. And it ready to go.
Whenever you have a moment, could you please take a look at this?

Thank you @kayrus  for reviewing my PR.
I completely agree with your point about hybrid/multi-cluster setups (bare metal + OpenStack + AWS + GCP, etc.). This should first be implemented in https://github.com/kubernetes/cloud-provider first. I hope it will be added someday.
Our case is a bit different:
Imagine an OpenStack setup with only one Keystone endpoint but multiple regions for services like Nova, Neutron, Cinder, and Glance. Each region has one available zone called "nova" (the default installation). I think this type of setup is easier to manage and upgrade. I've seen similar setups used by many cloud providers.

So, in this case, it’s not fully separated OpenStack clusters. It looks more like one region with many av-zones, similar to how well-known cloud providers organize their systems.
That’s why this PR supports only one Keystone endpoint. Using this endpoint, we can get Nova/Neutron endpoints for each region.
This feature will be introduced as an alpha feature and can be enabled using a CLI flag. Other components like the load balancer, cinder-csi and manila-csi will also need updates. However, we need to focus on supporting the cloud-node and cloud-node-lifecycle controllers first.
PS. I think I forgot to check backward compatibility with non regional OpenStack setups. I will check this soon.
Thanks.

I’ve updated the documentation and verified the setup. It works well both with OS_CCM_REGIONAL=true enabled and without it.
Could you please let me know if the documentation is clear or if there’s anything that needs improvement?
Thank you! @kayrus

Doesn't this change deserve a release note?


configurable depth of code analysis for large projects

For large projects PTVS is quite slow with intellisense and completion database. But what if the user has ability to select the level of introspection by PTVS for large projects?

Your wish is granted :)
Create a registry key at HKCU\Software\Microsoft\PythonTools\14.0\Analysis\Project and add values for any of the items in AnalysisLimits.cs (smaller means faster/lower quality). Reducing CallDepth and NormalArgumentTypes seems to have the most impact.
It's not perfect as it applies to all projects for a user, but it is an escape hatch for when you need it. We don't expose them through the UI (a) because it's hard to write good UI for this, and (b) it will lead to poor results, and we don't like it when we have to explain to people that they have "errors" or "bugs" because they changed their own configuration.
As I mentioned in another reply to you, I'm working on some major changes to this area that will likely make it much easier to handle exposing more information and options to users.


Support scientific notation for Quantity in Search

Description
When converting a quantity in the search parameter (from string to decimal), now also the scientific notation can be used.
Related issues
Addresses [#838]
Testing

Tested manually
Added an unit test

FHIR Team Checklist

[ ] Update the title of the PR to be succinct and less than 50 characters
[ ] Add a milestone to the PR for the sprint that it is merged (i.e. add S47)
[ ] Tag the PR with the type of update: Bug, Dependencies, Enhancement, or New-Feature
[ ] Tag the PR with Azure API for FHIR if this will release to the managed service
Review squash-merge requirements

Semver Change (docs)
Patch|Skip|Feature|Breaking (reason)

/azp run

This may be slightly out of scope, but what happens if the number doesn't fit inside a decimal, let's say 6.5742407583884805e+84 (https://github.com/FirelyTeam/firely-net-sdk/discussions/1635)? Can such numbers be persisted to the FHIR server in the first place?

/azp run

@albamati just a heads up that we require signing our contributor license agreement to be able to commit this PR. Can you review that (above in the comments)? Thanks!

This may be slightly out of scope, but what happens if the number doesn't fit inside a decimal, let's say 6.5742407583884805e+84 (FirelyTeam/firely-net-sdk#1635)? Can such numbers be persisted to the FHIR server in the first place?

IMHO the code in this PR should work in all those cases when the POST for the resource is working.
I tried to POST an observation with 6.5742407583884805e+84, I got a Bad Request:

... so I think it's ok if the search Observation for that quantity returns also an error:

In my opinion introducing support for big numbers like 6.5742407583884805e+84 probably requires a bigger effort. It's look like a "task" more than a "bug fix"  - Probably it is out of scope trying to fix it in this PR

/azp run


Division of thermoception into hot and cold perception

I would like to suggest that "thermoception" be divided into "detection of hot stimulus" and "detection of cold stimulus".
These functions have been shown to be carried out by different neurons, for example PMID 21335241

Hi,
A naive question: Are they mediated by different molecules ? We annotate gene products, not cells.
Thanks, Pascale

Yes, loss of function brv gene mutants lose cold-avoidance behaviour, but retain hot-avoidance (same reference).


fix(elements): keep unresolvable hashes when hash router is used

Fixes https://github.com/stoplightio/elements/issues/2205
When HashRouter is used, hashes are (obviously) used for navigation.
This poses an issue, as h2/h3/h4 headings may also contain anchors, which are supposed to navigate us to a particular fragment of the page.
Currently elements redirects you to root (/) if a given route has no node associated.
Due to the nature of HashRouter, both the heading anchors and the links adhere to the same mechanism, thus any link heading that's not associated with a subpage, will be redirected to /
Unfortunately, we cannot quite tell whether a user wants to navigate to a particular subpage or simply has an anchor, so what I ended up was: if we cannot associate any node with a particular hash, we redirect to / and keep the initial hash.
This way the navigation works correctly. That's presumably what 99% of hash router users would expect to happen.

Great PR description. Logic and behavior both seem sound.


fix: supported chains endpoint

Description
Adds a basic endpoint to return a list of supported CAIP-2 chain IDs.
Resolves #566
How Has This Been Tested?
New integration test
Due Diligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

@geekbrother oops! Yeah meant to do HTTP RPC not WS supported chains. Good catch.
I suppose I'll put the list of HTTP RPC supported chains in an http entity of the response to allow room to support ws in the future? WS is deprecated and will probably be removed. Maybe I'll just put both http and ws in there, might as well and easy enough and models current structure of everything.


Backport "Backport from metals" to LTS

Backports #21196 to the 3.3.5.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to backport-lts-3.3-21343.
Reference


CoreOS Getting Started Guide Issues

Minor issues with https://github.com/GoogleCloudPlatform/kubernetes/tree/master/docs/getting-started-guides/coreos/cloud-configs

The examples rely on cloud vendor specific variables ($private_ipv4 )
The wget command for kube-register always re-downloads

LGTM


WIP: New ModelArrayRef

Will close #249

Converting to a draft PR until I am happy with everything.

This PR is not required by the dynamics merged. It would be good to get it integrated before too much code uses the old system.
Moving it to the next milestone: functional tests.

ICCS won't be able to meaningfully review this PR but the plan is to go ahead with the merge and rely on comprehensive integration tests (part of Milestone 2) to verify the correctness of this branch. After this branch is merged into develop we want the remaining features e.g., MPI and XIOS to be based off of the develop branch.
Going forward the aim is to keep features branches small and self-contained so that they can be merged back into develop in a timely fashion.

Just to flag: When running run_simple_example, this branch gives a 'double free or corruption (out) -
Aborted (core dumped)' error. Similarly, after the rebase in #331, that branch gives the same error, while commit ' a5e4b77' is fine. Trying the develop branch with the same test script gives a 'Segmentation fault (core dumped)'

@timspainNERSC when you are back, can we fix the simple example scripts e.g. run_simple_example.sh. They do not run on this branch.

Interestingly, run_simple_example.sh does run on my computer, but only produces 1x30 output. Which it really shouldn't.

@timspainNERSC I recommend compiling with these flags -g -fsanitize=address -fno-omit-frame-pointer to see if there are any memory errors. This should help find them.
Also you can try running the code in gdb. when it crashes you can type command bt to get a backtrace. Happy to meet this week to discuss in more detail.

With #425 being merged, do I understand it correctly that now #383 needs to be merged, and then this PR will be merged and upcoming problems solved as they come up? Or are we waiting for another issue/PR?

With #425 being merged, do I understand it correctly that now #383 needs to be merged, and then this PR will be merged and upcoming problems solved as they come up? Or are we waiting for another issue/PR?

I have just been through the priority PRs and marked their dependencies. As of today, this PRs #250 (this), #383 and #394 are able to be reviewed and merged. This is the most important as it impacts so much of the rest of the code.


Release version 7.4 now that SPP is GA

This is the final PR to finalized the 7.4 release of safeguard-bash to coincide with SPP 7.4 GA.  I'm assuming I can merge the 7.4 branch into master branch after this PR is approved.  Please let me know otherwise.

@JaredCuster  or @DanielTessmer  could I get one of you to review for final release please?

@JaredCuster  or @DanPeterson could I get one of you to review for final release please?

@ChrisHardison just testing


Version sorting

http://www.jsdelivr.com/projects/react

React lastversion shows 0.14.0-rc1 instead of 0.14.0
https://github.com/jsdelivr/jsdelivr/issues/7281

The site uses the API sorting. We should short the versions correctly in the API by default. 0.14.0 > 0.14.0-rc1 > 0.14.0-beta1

Here's the code we were previously using.

I'd just use the semver package: https://www.npmjs.com/package/semver.

The problem is that not all projects use semver and IIRC that package requires valid semver versions.

The problem is that not all projects use semver and IIRC that package requires valid semver versions

Plus sometimes a project will switch between valid & invalid semver.  Sometimes I feel like we should force our versions into valid semver to get around this.  I'm trying to think of real examples where the version is too strange; usually non-semver look like v1.003 or 20151016.
I think  0.14.0-rc1 > 0.14.0-beta1 will fail compareStrings().
Maybe pad odd versions with .0 to make into good semver?
Related issue; if project changes from strange versions to semver, they won't track right (very vary rare, especially on github).

I just remembered @megawac 's genius: https://github.com/megawac/SemVish.
& someone made a CLI semverish

I think 0.14.0-rc1 > 0.14.0-beta1 will fail compareStrings().

Nope, it works correctly.

This is ridiculously easy w/ semver:
versions = versions.sort(semver.rcompare);

Now fixed in v2:

@UnbounDev Can you apply the sorting to v1 as well? The website is using v1's listing of all libraries in a single request

:+1: Will do

@jimaek fixed in #105


Nested VM can't start due to vhost-vsock issue

Required information

Distribution: Ubuntu Focal
Distribution version:
The output of "lxc info" or if that fails:

Kernel version: 5.4.0-18-generic
LXC version: latest
LXD version: 4.0.0 (rev: 14270)
Storage backend in use: ZFS

Issue description
I'm creating a nested LXD environment to run virtual machines. It looks good until I tried to launch the VM - in the nested KVM guest - and received:
rafaeldtinoco@lxdexample:~$ lxc start focalvm
Error: Failed to run: modprobe vhost_vsock: modprobe: ERROR: could not insert 'vhost_vsock': Device or resource busy
Try lxc info --show-log focalvm for more info
rafaeldtinoco@lxdexample:~$ sudo modprobe vhost_vsock
modprobe: ERROR: could not insert 'vhost_vsock': Device or resource busy
Steps to reproduce

lxc init ubuntu-daily:focal lxdexample --vm
Set cpu, memory and add a config disk
Install LXD inside it and try to run a nested VM

Information to attach
I believe vsock is broken in nested LXD VMs. I know VM feature is recent and being developed, don't want to rush things, just wanted to document.

@rafaeldtinoco can you show the output of lsmod | grep vsock please, it may be there is a conflicting vsock module loaded already.

So I'm not sure that we can actually nest vsock properly.

So I'm not sure that we can actually nest vsock properly.

You likely want what's been merged in November:
commit 24df31f8d5be1b09f9e6c6d07ddfa96cbef1782b
Merge: 798a496bf425 ed8640a9612c
Author: David S. Miller<EMAIL_ADDRESS>Date:   Thu Nov 14 18:12:19 2019 -0800

    This series adds the multi-transports support to vsock, following
    this proposal: https://www.spinics.net/lists/netdev/msg575792.html

    With the multi-transports support, we can use VSOCK with nested VMs
    (using also different hypervisors) loading both guest->host and
    host->guest transports at the same time.
    Before this series, vmci_transport supported this behavior but only
    using VMware hypervisor on L0, L1, etc.

Ah yeah, we'd indeed need that for nesting ^
@brauner is that a reasonably sized patch which we could convince @sforshee to include in the Ubuntu kernel?

Closing because it's a kernel limitation.

Ah yeah, we'd indeed need that for nesting ^
@brauner is that a reasonably sized patch which we could convince @sforshee to include in the Ubuntu kernel?

That's a pretty huge patch tbh.


Enable folder source when subsetting corpus

Currently subsetting fetches data from Github which is a bottleneck if subsetting a large number of files.

Fixed by c3affb7d7e88fbb4681a0343a661147000af0787.


Image pyramid resoution changes stop working if you switch from 2D -> 3D view -> 2D again

🐛 Bug

If you open a multi-resolution image in napari, the default 2D view will automatically load the different resolution levels as you zoom in and out. BUT if you switch the view to a 3D volume view, and then switch back to a 2D view, then the multiresolution levels don't work anymore when you zoom in and out with the mouse wheel.
cc @gazzar who found this out in the last 5 minutes of our napari hack day!
Gary says he needs to catch the train now but will add links or add any other information I forgot.
To Reproduce
Steps to reproduce the behavior:

Load a multiresolution image into napari.
Zoom in & out in the 2D mode and see the resolution levels changing automatically
Switch to the 3D volume view.
Switch back to the 2D image plane view.
Zoom in & out in the 2D mode and see that the resolution levels don't update automatically anymore :(

Gary says the code sample he used is in this gist: https://gist.github.com/gazzar/d06a1cafdba9888a47d905e2664d070f
Expected behavior

We expected the resolution levels to update like they did when we first opened the image.
Environment

Please copy and paste the information at napari info option in help menubar here:

Any other relevant information:

Additional context

@gazzar can you add this info in a comment below please?

Please copy and paste the information at napari info option in help menubar here:
Any other relevant information:

Thanks for creating this bug report for me @GenevieveBuckley
napari info option:
napari: 0.2.7+1.g0ecabae.dirty
Platform: Windows-10-10.0.17134-SP0
Python: 3.7.5 (default, Oct 31 2019, 15:18:51) [MSC v.1916 64 bit (AMD64)]
PySide2: 5.14.0
Qt: 5.14.0
VisPy: 0.6.4
NumPy: 1.17.4
SciPy: 1.4.1
scikit-image: 0.16.2
Dask: 2.9.0
GL version: '4.5.0 - Build <IP_ADDRESS>73'
MAX_TEXTURE_SIZE: 16384
Other relevant information:
The multiresolution image I used is a BigDataViewer dataset with 4 levels of pyramid representation. Unfortunately I'm not able to share the data I used.

Thanks for catching this! The 3D pyramid will only show the most zoomed out layer. It should still work again properly when switching back to 2D though. I'll have a look at where the problem is. Shouldn't be too hard to fix

This has now been fixed with our recent multiscale improvements - see this code to confirm it is no longer an issue
    data = [np.random.random((5, 2**s, 2**s)) for s in range(13, 8, -1)]
    viewer.add_image(data)

If this example didn't fully capture the bug and a new minimal example is given we can reopen. Thanks all for falgging


Simplify calculateAngleBetween3Points

Seen as a,b,c were entered in as raw values to begin with, I don't think that declaring operations (a-b) & (c-b) is/are sufficient as variables. Should probably check to see if a,b,c exists before it's being passed in to prevent possible edge cases from happening.
Probably a good idea to implement error (exits, but provides a message) and warn (pass, but provides a message) methods or something similar to prevent obscure error handling such as this. Whatever works.

Directory is no longer maintained in favor for workshop. Closing.


[DatePicker] The style for Paper not working

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
The style for Paper not working
https://codesandbox.io/s/basicdatepicker-material-demo-forked-07w0l?file=/demo.tsx
In previous versions, I could access it this way. But now it doesn't work anymore

    PrivatePickersPopper: {
      styleOverrides: {
        paper: {
          boxShadow: theme.shadows[20],
          borderRadius:20
        }
      }
    },

How can I interfere with the DatePicker components and style them?

Using the sx prop from your side was the right thing to do, however we skipped the sx prop in the pickers component, thinking that it may affect perf and we weren't sure if it will be useful. Seeing this issue, I propose we remove this. For this particular use-case this should fix it:
index 79436e4e8f..8a7eb2721f 100644
--- a/packages/mui-lab/src/internal/pickers/PickersPopper.tsx
+++ b/packages/mui-lab/src/internal/pickers/PickersPopper.tsx
@@ -42,7 +42,7 @@ const PickersPopperRoot = styled(Popper, { skipSx: true })<{ ownerState: PickerP
   }),
 );

-const PickersPopperPaper = styled(Paper, { skipSx: true })<{
+const PickersPopperPaper = styled(Paper)<{
   ownerState: PickerPopperProps & Pick<MuiPopperProps, 'placement'>;
 }>(({ ownerState }) => ({
   transformOrigin: 'top center',

Then, you would be able to customize it with - https://codesandbox.io/s/basicdatepicker-material-demo-forked-67toz?file=/demo.tsx
I propose we remove the skipSx everywhere, it is surprising that it works in some places and not in others. @mtr1990 would you like to work on this?

Hi @mnajdova and @hbjORbj, I will work on this!

Hi @mnajdova and @hbjORbj, I will work on this!

@boutahlilsoufiane Please go ahead! We always appreciate contribution of the community.

Hi @mnajdova @hbjORbj could you please review the PR! Thank you so much


[BUG] When double-clicking two sql execution tasks alternately for editing, the sql statement will display an error

*For better global communication, please give priority to using English description, thx! *
Describe the bug
When double-clicking two sql execution tasks alternately for editing, the sql statement will display an error.
To Reproduce
Steps to reproduce the behavior, for example:

Create workflow
Then create two sql scheduling tasks in succession（eq. one is create table sql, and other is insert sql）
Save&quit the workflow
Edit the two sql scheduling tasks alternately, this time you will find an error when displaying the statement

Expected behavior
no
Screenshots

Create the first sql task

create the second sql task

create connect

4.Edit after saving

Click two sql tasks alternately， then and look the error!

Which version of Dolphin Scheduler:
-[1.2.1]
Additional context
no
**Requirement or improvement

Should be a front-end bug

good issue， thanks for your report
@break60

#2366

it has been resolved in dev_1.3.0


Automatic e-mail signatures for all active members

Beheer got a request whether it was possible to automatically install e-mail signatures for all active members so that the e-mail signatures are uniform and for "uitstraling".
Example

I do not think this is an amelie issue as this should probably be configured in Google Workspaces directly.
Maybe relevant: https://support.google.com/a/answer/176652

Needs to be set using the Gmail API according to that page, probably per individual users. So this will probably need to be done when a user is created via Claudia.
It also notes that Google Workspace is required, and we might still be on the old GSuite, so some investigation is needed to see if we need to upgrade, and if so, if we keep our 'unlimited' free plan or not.

Should be doable with our current Google Workspaces, and should be pretty easy.


http: request body too large

Hello,
I'm trying to write some binary files base64 encoded onchain through the python driver and I wonder how can I increase the transaction body limit. I was able to send a file of 512k wich was synced on four nodes but for a file of 1M or bigger I received a request body too large message.
Thanks.

same.


Unable to Deploy Extended ERC20 Standard Contract Example

Hello,
I've been trying to deploy an example that extends the ERC20 standard contract using the starkli tool, but I've encountered some issues.

Steps to Reproduce:
I followed the example provided for extending the ERC20 standard contract (picture above).
I used the following command to deploy the contract:
starkli deploy --account ~/.starkli-wallets/deployer/account.json --keystore ~/.starkli-wallets/deployer/keystore.json --watch 0x0751012e95eb15318a7036d935c552bf19c526a429c90dc7d8aa611f0e530cc4 u256:100 0x052E6AD8187767eF41F2F3bc225D33d31B37DD9BBEE7B628F4b6b16b90D293EC
Expected Behavior:
The contract should deploy successfully.
Actual Behavior:
I received the following error:
Error: code=ValidationFailure, message="Execution was reverted; failure reason: [0x496e70757420746f6f206c6f6e6720666f7220617267756d656e7473]."
Any assistance or guidance on this issue would be greatly appreciated.
Thank you!

Try bravos wallet, I got same error "?Input too long for arguments", and changed to bravos, it built.

Thanks for the help @mbzdg.
I forgot to re-fetch my argentX's account file after I upgraded it.


SQLAlchemy: Maintenance support for SQLAlchemy 1.3

Hi.
While working on support for SQLAlchemy 2.0 on behalf of GH-485 and GH-488, we discovered that the updated code will no longer be compatible with SQLAlchemy 1.3.
ImportError: cannot import name 'declarative_base' from 'sqlalchemy.orm'
sqlalchemy.exc.ArgumentError: columns argument to select() must be a Python list or other iterable

While SQLAlchemy 1.3 is effectively EOL, we may still want to keep supporting it for a while for users of the CrateDB dialect.

If you agree, we will have to come up with a proper release and deprecation plan.
My proposal, already briefly discussed with @seut, would be to run a 0.30.0 maintenance release, which adds a corresponding warning for users who are still on SA13. Then, we would move on by integrating the SA20 changes, and probably release them with a crate-python 1.0.0 version (maybe after running a few 1.0.0bX releases). In this manner, the 1.x series will support SA14 and SA20, while the 0.x series would still be able to receive maintenance patches, and keeps supporting SA13.
With kind regards,
Andreas.

Inspired by comments from the upstream authors at https://github.com/sqlalchemy/sqlalchemy/issues/8631#issuecomment-1362880407, GH-497 retains compatibility with SQLAlchemy 1.3.


High-level description of how this framework works

Hi there!
I found this framework interesting and promising due to its handy API syntax. However, the documentation somewhat confuses me. Well, this is unsurprising because I'm not an expert in Web development anyway. Actually, I have received little formal education in CS, but I know the very basics of TCP thanks to Beej's Guide to Network Programming. In addition, I have previously worked with kcgi, Flask, and Django.
Anyway, from my perspective, both the comments in the header files and the HTML document are enumerating functions without sufficiently stating how they interact with each other as a whole. For example, the HTML document begins with introducing fio_protocol_s which has a lot of on_* callbacks. This part inevitably contains several forward references to stuff like FIO_PR_LOCK_TASK. When working through it I'm totally lost for a while, because I have no idea how the connections and locks are handled by the framework (without reading the rest of the document, which is kinda long and dry).
Personally, I would like to see a high-level description at the very beginning of this documentation, preferably presented with a diagram like the following one. Plain text descriptions like "On startup, we allocate a fio_protocol_s object with malloc and set its callback functions by writing a compound literal. Next, we get a UUID from somewhere. Then we can conveniently attach it to a connection with fio_attach and set up some kind of lock. ..." are also nice to have.

@sunqingyao ,
Thank you for opening this issue. I was looking at the documentation and you are right, it's missing an overview of the library.
The HTML documentation source code is at docs/_SOURCE and I will try to add an overview to that location when I can.
Right now, the example code should provide the best overview, and I hope I will be able to improve the documentation soon.
I will keep this issue open until I update the documentation.

Thanks!
The example code in the repository is great and informative, but unfortunately those in the HTML document are not. Please update them as well. For example, the echo server example doesn't even compile with facil.io 7.0.0.


Icons too small in iOS app

I have read the wiki about what to try first before creating a bug report.
Which version of HA do you use?
0.110.2
iOS app 2.0.0(69)
Which version of the card do you use?
0.1.1
Describe the bug
On a desktop browser icon size is correct.
On a mobile browser icon size is correct.
In the iOS app icon size of the home/solar/grid icons is too small.
Configuration of the card
Copy the complete configuration of the card, even if you think it isn't relevant.
  - type: "custom:power-wheel-card"
    solar_power_entity: sensor.solar_power
    grid_power_consumption_entity: sensor.grid_power_consumption
    grid_power_production_entity: sensor.grid_power_production
    solar_energy_entity: sensor.solar_energy_today
    grid_energy_consumption_entity: sensor.grid_energy_consumption_today
    grid_energy_production_entity: sensor.grid_energy_production_today
    energy_consumption_rate: 0.1883

Expected behavior
Bigger size icons in iOS app.

Because of recent changes in HA per 0.110.x I guess the iOS app might need a change for this too. Since the small window size of mobile is not the problem. (Icon size is correct on emulated mobile windows and on my mobile Chrome browser).

Got magically fixed in HA 0.110.3.
Maybe it was: Update frontend to 20200519.5 (@bramkragten - #36154) (frontend docs)


backport aspose updates to 7.8

❗ please do not add sensitive information in issues, you can provide extra information via email using issue number as reference ❗
Describe the issue
backport aspose updates to 7.8
Reporter
JJAnsenvr

There is only 1 Aspose related CVE's in the 7.8. Tika has already been updated to the latest version