When TextEdit::multiline has a dynamic length, the button event cannot be selected for focus

// impl eframe::App
                ui.horizontal(|userinfo| {
                    userinfo.with_layout(Layout::left_to_right(Align::RIGHT), |lable| {
                        lable.set_width(112.0);
                        lable.with_layout(Layout::right_to_left(Align::RIGHT), |ui| {
                            ui.add(Label::new("Request Token:"));
                        });
                    });
                    // userinfo.add(TextEdit::singleline(&mut self.setting_form.token));
                    userinfo.add(TextEdit::multiline(&mut self.setting_form.token));
                    // userinfo.text_edit_multiline(&mut self.setting_form.token);
                });

                ui.horizontal(|sure_button|{
                    sure_button.with_layout(Layout::right_to_left(Align::LEFT), |button| {
                        if button.add_enabled(true, Button::new("submit")).clicked() {
                            self.setting_form.submit = true;
                        }
                    });
                });

When the multiline exceeds the default rows, or contains special characters, buttun is no longer available. If the error triggered by special characters is not accepted, there is no panic
The above submit is gray and contains text, which cannot be selected for submission, and I have not selected special characters

This problem seems to be caused by the insufficient height allocation in horizontal layout. I assign ScrollArea to TextEdit to control the height, or manually assign the height to TextEdit, which can now be used


data fields display in editor but not in XML preview

A cataloger reported that she input a variety of fields in a description, but they do not display in the posted record. They also do not display in Marva when the record is re-loaded.
I have duplicated this behavior on preprod-3001.
https://preprod-3001.id.loc.gov/bfe2/editor/edit/e0811612
I updated the Date of Work (bf:originDate) and Moving Image Technique (bflc:movingImageTechnique) fields. They do not display in the XML preview and they do not display in the preprod database either, https://preprod-8210.id.loc.gov/resources/works/a4d148af-55bd-4e5d-81c3-66b119b9fdaf

bflc:movingImageTechnique is not a valid property.

transferred to Gitlab


Eradicate false positive

Hi,
# SPDX-License-Identifier: BSD-3-Clause

is deleted by eradicate even if it is not a valid code if uncommented.

You can configure Ruff to allow those comments via task-tags! Check out #2064.

I could work on it, I seem to remember that there is a helper to check if a python name is valid. As kebab-case is invalid, it would suffice to correct the false positive.

Kebab-case is not a valid python name so it should not count as valid code.
I do not remember the name of the helper that checks if a python name is valid or not.
It could be used to remove this false positive.
I could work on it if you want.

Sure thing!

(We have an is_identifier method in src/python/identifiers.rs.)

It is not possible to fix.
In the context of a dict. This is a valid code if SPDX, License, Identifier, BSD, and Clause are int variables.
{
    SPDX-License-Identifier: BSD-3-Clause
}

Ah ok. Thanks for taking a look. At least we have a valid workaround.

You are welcome.


Incorrectly placed diacritical characters in Calibri

For the following text, the two diacritical characters in appearing incorrectly for Calibri font compared to any other font like Arial.
بِّ

Thank you for contacting us. The position of the two diacritical characters on the letter ‘Beh’ in your example is acceptable in both styles.
When the marks are combined, bot marks are placed on top of the base character as shown in Calibri.
When the marks are not combined, the ‘Kasra’ mark is placed below the base character as observed in Arial.
It’s a design choice. To my understanding, in modern style Arabic writing Calibri style is preferred. However, in traditional writing the Arial style is more common.


Configurable local xml-rpc callback port

DONT'T DELETE THIS.
Is your feature request related to a problem? Please describe.
Currently there seems to be no way to have custom_homematic work as part of a home-assistant installation running as a container, without it using host-network. This is due to the xml-rpc-server in hahomematic seemingly using a random callback-port, making it impossible to setup a stable, working port mapping.
Describe the solution you'd like
Make the local port under which the xml-rpc-server is running configurable.
Describe alternatives you've considered
Other solutions could be: use the configurable "callback_port" not only for communication, but also as the port the server is started with - reduces the number of config options, but is less flexible.
Using the host-network route is no option, since ha is running as part of a larger collection of containers in a locally setup kubernetes cluster with multiple nodes. I need a way to setup a fixed port and ip (ingress, load balancer etc) and host-network would not allow for a fixed ip.
Additional context
Reading through other issues regarding ports and container based setups, the current solution seems to be to not do it. I know of the limitations regarding auto discovery but disregarding homematic I was able to work around all other limitations.

Version 1.11.2 will fix an issue with XmlRPCServer.
Then callback_host and callback_port can be used.


import error

Hi!
Sorry to take up your time with this, but I have a small issue when trying to use the package on chemrxiv/biorxiv and medrxiv.
The import fails, and I have a no module found error (I have attached a snapshot).

I was wondering if I had missed something?
Thank you very much for making this package open source, I look forward to using it!
Best regards,
Claire

Hi @donnate thanks a reporting this, I think I understood where the issues is (a missing __init__.py most likely).
Will keep you posted and let you know once we have a fix ready.

This should be addressed once this PR is merged: https://github.com/PhosphorylatedRabbits/paperscraper/pull/5.
Afterwards, the version 0.0.2 that will be available from PYPI via pip should not cause the import issue.

HI @donnate the fix has been released and the new version is available on PYPI: https://pypi.org/project/paperscraper/.
Please try again updating the package version:
pip install paperscraper==0.0.2

In case the import error is solved, please close the issue.

Hi,
thanks a lot!
There seems to be a small typo in one of your init files:

Best ,
Claire

Hi,
Thanks I think it works!
Will keep you posted if I see anything else, but thanks again for the package, this looks great!


Toolbar icon-navigation-expand-more not showing

I am trying to use that and somehow the icons are not showing. It seems that when i tried different icon also in app bar they werent showing also.

I figured the matierial icon are not included . To install them npm install material-design-icons. Infos : http://google.github.io/material-design-icons/#getting-icons


feat: Add support for custom test attributes

You may now add the attribute #[parameterized_macro(...)] after a #[parameterized(...)] attribute. The ... in the parameterized_macro may be replaced with a custom test attribute such as tokio::test.
As an additional treat (because the above would not half be as useful without some of these), the parameterized test function now also parses and provides codegen for the async, const and unsafe keywords, in addition to the also newly added support for specifying return types.
closes #80

This is now supported and will be released as 2.0.0
Example usage: https://github.com/foresterre/parameterized/blob/e0186210baf99632c5a454100d93f64cf475b217/src/lib.rs#L95-L111


Big query

hi,
I am getting the following error on high queries. Can you help me with this?

Please Reinstall, we release a bug fix

Unfortunately it continues to give the same error.

please download and reinstall again, as we release fix.
Also share how you cause this error?

I have a search of 36628 records. there is no problem in the first run. But when I turn off the PC and turn it on again, the previous error appears and the output screen freezes like the picture below

I suggest resetting the app and starting from scratch again, by going to
Support > Factory Reset App and Clear All Data.
This error should not reoccur again, given that you followed the previous comment I told you

The error I mentioned has been fixed. Thank you.
Now I am getting a different error

SyntaxError: Bad control character in string literal in JSON at position 3145728 (line 103498 column 48)
at JSON.parse ()
at o.refresh (C:\Users\winserver\AppData\Local\Programs\googlemapsextractor\resources\app.asar\dist\main\main.js:2:193450)
at new o (C:\Users\winserver\AppData\Local\Programs\googlemapsextractor\resources\app.asar\dist\main\main.js:2:193300)
at new s (C:\Users\winserver\AppData\Local\Programs\googlemapsextractor\resources\app.asar\dist\main\main.js:2:194062)
at c (C:\Users\winserver\AppData\Local\Programs\googlemapsextractor\resources\app.asar\dist\main\main.js:2:169287)
at h.saveCachedTask (C:\Users\winserver\AppData\Local\Programs\googlemapsextractor\resources\app.asar\dist\main\main.js:2:169621)
at t.TaskExecutor.markTaskAsSuccess (C:\Users\winserver\AppData\Local\Programs\googlemapsextractor\resources\app.asar\dist\main\main.js:2:163256)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async t.TaskExecutor.runTask (C:\Users\winserver\AppData\Local\Programs\googlemapsextractor\resources\app.asar\dist\main\main.js:2:161640)

Please download and use latest version, let me know if it works?


Add region flag to dataflow.go

Support specifying GCP Region
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand:

[ ] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @herohde

Please test these changes on Dataflow and ensure the Go precommit passes. I was too quick to LGTM before.

@tgroh my favorite gentleman might you deign to merge this humble pull request

Tested in us-central1 (default) and us-east1 (manual override).

Let me know

Turns out the problems were on the Dataflow side; fixed.


Create 1074-number-of-submatrices-that-sum-to-target.py

File(s) Modified: 1074-number-of-submatrices-that-sum-to-target.py
Language(s) Used: Python
Submission URL: https://leetcode.com/submissions/detail/1159672861/

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@divamkumar Thanks for your contribution 👍


libcuda.so.1: cannot open shared object file: no such file or directory

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
TensorFlow version: 1.13
Python version: 3.7
Installed using virtualenv? pip? conda?: Conda
CUDA/cuDNN version: cudatoolkit version 10 and cudnn version 7.6 (according to conda list)
GPU model and memory: 2080ti 11gb 🥇

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
I create a new conda environment and run conda install tensorflow-gpu. Next I start up a python terminal and import tensorflow as tf. My error message received is in next section.
Any other info / logs

Traceback (most recent call last):
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: libcuda.so.1: cannot open shared object file: No such file or directory
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/init.py", line 24, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/init.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/imp.py", line 242, in load_module
return load_dynamic(name, filename, file)
File "/home/ian/anaconda3/envs/tf_gpu/lib/python3.7/imp.py", line 342, in load_dynamic
return _load(spec)
ImportError: libcuda.so.1: cannot open shared object file: No such file or directory
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

@iantimmis Just to verify, Did you install CUDA toolkit in the same environment? Thanks!

I think so. I'm under the impression that conda handles that for you. When I run "conda list", I see that cudatoolkit version 10 is present.

@iantimmis Looks like similar issue, please take a look at #26209. Let us know if that helps. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

The isssue is still exisiting

The issue is still exisiting

Yes, same issue with same configuration

The issue is still existing

Just ran into it myself


composite: images do not have same numbers of bands (Vips::Error)

it works with a .jpg, .jpeg, but now thi a .png:
-  image: "/assets/static-pages/yaro-avatar.png"
+  image: "/assets/static-pages/yaro-cindy.jpg"

What version of libvips do you have? I used .png image and everything was good.

ruby-vips (2.2.0)

I'm encountering the same issue, with ruby-vips (2.2.2) @igor-alexandrov

I will look at this during the weekend.


setPosition not working

hey there!
i got your plugin working an its beautiful. i just have a problem when trying to scroll to a section on the page. when i click my button it just says scrollbar is not defined. Can you tell me what am I doing wrong?
Thank you so much!
Jaro

var scrollbar = Scrollbar.get(YourElement)

thanks for taking care of me.. would you please a bit more precise? i am new to javascript. what element do i have to get?

The element you initialized scrollbar to:

Scrollbar.initAll()
var scrollbar = Scrollbar.get(document.getElementById('your-scrollbar')

thank you so much! i that helped me a lot. thought i could use query to get the element?

You meant jQuery? Try
Scrollbar.get($(selector).get(0))

wow. thank you!! one last question..
i use scrollIntoView() for my anchors now. can i adjust the duration of the animation somehow like you can do for .scrollTo()?
:)

Sadly, by now you cannot change the scrolling duration :(

is it possible to use the scrollTo() function somehow and use the offset of the element to trigger its position?

Yeah it's possible. But you need to calculate the offset yourself. You can refer to the implement of scrollInToView: https://github.com/idiotWu/smooth-scrollbar/blob/113ead2046530c1d0592d6507c5b141887cc9c89/src/apis/scroll-into-view.js#L34-L41

uh.. i guess that goes beyond my understanding. but thanks for your help. you really are doing a great job. any idea if a duration parameter is coming to scrollIntoView()? also with maybe selecting custom easing curves? :)

I'm afraid there will never be a duration option for this method 😥

hey again. can you help me with the scrollInToView() method then? i tried a lot yesterday but couldn't make it work.

Can you post some example code?

actually i was trying to understand what happens within your plugin.. and also the snippet you sent me. but i did not understand it. my javascript knowhow is very limited. what exactly would i have to do to find the offset of an element on the canvas?

That's a bit complicated. I'd suggest you ask someone in real life ;)


Add kubescape to Zarf init package

I'm not really sure if this is something we want to do, but I did it as part of my kubescape work so I thought I'd share and have a discussion about it.
What/Why

Add kubescape to the Zarf init package so that it can be used in airgapped environments

Have you played with the yaml-only scanning in kubescape? I would love to see this as a first wag at pre-package scanning we've discussed before. Something that makes the report about the package before you ever run.
As far as being in the init package, I'd rather keep it under "examples" for now as an add-on capability until we can discuss further. Super valuable add either way though.

I have not. That sounds awesome, but definitely bigger scope and different use case.
Moving it to examples works for me. I'll add it to the Big Bang Core example.

Yeah thanks, examples for now is probably good, and then we can look again at it in the future, just trying to be careful of initial binary size, it's already fatter than I would like.  Thanks, this is a really useful addition.


Service Broken?

It seems that the service is reporting 0 loc for any project that you feed it.

https://github.com/XAMPPRocky/tokei_rs/issues/123 it has been gone for months


Add option to open a text editor for the commit's description

It feels nice to write short stuff like the scope and the header of the commit right from the CLI, but when it comes to a potentially larger text such as the description of the commit, I'd rather write it on a text editor.
Currently, we have the -e flag, but this option makes the whole process occur in the text editor, which isn't as convenient as having the CLI questioning you about the smaller fields.

I see. Would a last question with a default to no be good for this case? If the user presses y, the editor would open with the already generated text in it.
Another option would be to change the current behavior of the -e flag and make it open after all questions have been asked.

I like the idea of allowing the user to review/edit the final message.
Both approaches are nice, but I think changing -e may be the way to go, initially. IMO, its current behavior makes using the editor too manual, as you have to build each part of the message yourself. Also, changing the behavior of an option rather than the main flow looks less impactful.
It think it would be good for the user experience to add a "do you want to review?" question in the end of the process, though. Currently, the CLI writes the commit right after it got all the information from the user, without allowing its review.

It think it would be good for the user experience to add a "do you want to review?" question in the end of the process, though. Currently, the CLI writes the commit right after it got all the information from the user, without allowing its review.

I didn't add it at the beginning as you could always run git commit --amend to basically do the same. I do see why someone might want that though :smile:

v0.2.0 has been released. You can either download a pre-built binary or build it from source using crates. homebrew tap is in the works :)

Homebrew tap is done as well. You can install it via brew install sirwindfield/tap/git-cm. Should work for macOS and Linux as well (if homebrew is installed) :)


libev appears to build properly in OS X

But it's not part of the platforms list:
diff --git a/pkgs/development/libraries/libev/default.nix b/pkgs/development/libraries/libev/default.nix
index b0c74c9..c2833c3 100644
--- a/pkgs/development/libraries/libev/default.nix
+++ b/pkgs/development/libraries/libev/default.nix
@@ -21,6 +21,6 @@ rec {
       a.lib.maintainers.raskin
     ];
     platforms = with a.lib.platforms;
-      linux ++ freebsd;
+      linux ++ freebsd ++ darwin;
   };
 }

Converted to PR.


Settings

There is a settings part in MicRichTextComposer>>preferencesSettingsOn:.
Reading the comments of what the various settings does, I believe they make more sense in the BeautifulComments project. For example one can choose not to render in the MicRichTextComposer - which make make no sense to me.

Yes this is probably a leftover of the split into beautiful comments.

Fixed by #11


Missing authorizer/enterer data

Authorizer/enterer names late in the alphabet (e.g. "M. Uhen") do not appear in the Navigator auto-search.

Should be fixed in 1.2 update: #29


Unable to fetch SamplingRules configured in AWS console

Hi,
With X-ray 2.0, I wanted to try out the option to manage sampling rules within the console itself. However the rules are not being applied/fetched from the console. In the stacktrace, I can see that it fails in the RulePoller trying to fetch the sampling rules.
We use a config equivalent to the following to initialize the global recorder:
            AWSXRayRecorderBuilder builder = AWSXRayRecorderBuilder.standard()
                .withPlugin(new EC2Plugin()).withPlugin(new ElasticBeanstalkPlugin())
                .withSamplingStrategy(new CentralizedSamplingStrategy(MyConfig.class.getResource("/xray/rules.json")))
                .withContextMissingStrategy(new NoOpXrayContextMissingStratergy()); //skips logging
            AWSXRay.setGlobalRecorder(builder.build());

Stacktrace:
2018-09-04 11:27:41,312 ERROR [pool-5-thread-1] c.a.x.s.sampling.pollers.RulePoller  Encountered error polling GetSamplingRules:
com.amazonaws.xray.exceptions.SegmentNotFoundException: Failed to begin subsegment named 'AWSXRay': segment cannot be found.
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
        at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
        at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
        at org.springsource.loaded.ri.ReflectiveInterceptor.jlrConstructorNewInstance(ReflectiveInterceptor.java:1002)
        at com.amazonaws.xray.strategy.RuntimeErrorContextMissingStrategy.contextMissing(RuntimeErrorContextMissingStrategy.java:17)
        at com.amazonaws.xray.contexts.ThreadLocalSegmentContext.beginSubsegment(ThreadLocalSegmentContext.java:23)
        at com.amazonaws.xray.AWSXRayRecorder.beginSubsegment(AWSXRayRecorder.java:418)
        at com.amazonaws.xray.handlers.TracingHandler.beforeRequest(TracingHandler.java:152)
        at com.amazonaws.http.AmazonHttpClient$RequestExecutor.runBeforeRequestHandlers(AmazonHttpClient.java:782)
        at com.amazonaws.http.AmazonHttpClient$RequestExecutor.doExecute(AmazonHttpClient.java:726)
        at com.amazonaws.http.AmazonHttpClient$RequestExecutor.executeWithTimer(AmazonHttpClient.java:719)
        at com.amazonaws.http.AmazonHttpClient$RequestExecutor.execute(AmazonHttpClient.java:701)
        at com.amazonaws.http.AmazonHttpClient$RequestExecutor.access$500(AmazonHttpClient.java:669)
        at com.amazonaws.http.AmazonHttpClient$RequestExecutionBuilderImpl.execute(AmazonHttpClient.java:651)
        at com.amazonaws.http.AmazonHttpClient.execute(AmazonHttpClient.java:515)
        at com.amazonaws.services.xray.AWSXRayClient.doInvoke(AWSXRayClient.java:1257)
        at com.amazonaws.services.xray.AWSXRayClient.invoke(AWSXRayClient.java:1226)
        at com.amazonaws.services.xray.AWSXRayClient.invoke(AWSXRayClient.java:1215)
        at com.amazonaws.services.xray.AWSXRayClient.executeGetSamplingRules(AWSXRayClient.java:568)
        at com.amazonaws.services.xray.AWSXRayClient.getSamplingRules(AWSXRayClient.java:539)
        at com.amazonaws.xray.strategy.sampling.pollers.RulePoller.pollRule(RulePoller.java:53)
        at com.amazonaws.xray.strategy.sampling.pollers.RulePoller.lambda$start$0(RulePoller.java:41)
        at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
        at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
        at java.lang.Thread.run(Thread.java:748)

Few details:
Versions:
com.amazonaws:aws-xray-recorder-sdk-core:2.0.0
com.amazonaws:aws-xray-recorder-sdk-aws-sdk:2.0.0
com.amazonaws:aws-xray-recorder-sdk-aws-sdk-instrumentor:2.0.0
com.amazonaws:aws-xray-recorder-sdk-apache-http:2.0.0
com.amazonaws:aws-java-sdk-xray:1.11.400
Deployed on:
ElasticBeanstalk Java 8 running on 64bit Amazon Linux/2.7.4

I think this ticket might be a duplicate of #27

Yes #27 is a duplicate of this issue.

Hi anaynayak,
Sorry for the issue. As per comment on #27 the AWSXRay client requests to the new Centralized Sampling API should not be traced as they are not part of the application's request code path. As of now, it looks like an exception is thrown due to the segment context expected but missing, and is not able to fetch the sampling rules as a result. We'll be investigating this and pushing a fix as soon as we can.

This bug is fixed at 2.0.1 which was just released. Please note it takes some time for the new version to propagate on maven central. Thanks.


MISUV-4674: fix taxYear format

change taxTear format while requesting calcs list:
https://confluence.tools.tax.service.gov.uk/pages/viewpage.action?pageId=561971911

Description
Please include a brief summary of the change you've made and the issue that it fixes.
Add a link to the relevant story in Jira
[SASS-CHANGE-ME](ADD URL HERE)
Checklist PR Reviewer
Before Reviewing

[ ]  Have you pulled the branch down?
[ ]  Have you assigned yourself to the PR?
[ ]  Have you moved the task to “in review” on JIRA?
[ ]  Have you checked to ensure all dependencies are up to date?
[ ]  Have you checked to ensure its been rebased against the current version of main?

Whilst Reviewing

[ ]  Have you run the tests?
[ ]  Have you run the journey tests?
[ ]  Have you looked at the JIRA story to make sure all Acceptance Criteria has been met?

After Reviewing

[ ]  Have you checked for merge conflicts or any changes in the current main that may affect the current pull request? i.e. does it need another rebase?
[ ]  Have you checked to make sure there are no builds in the pipeline before you merge?
[ ]  Have you moved the task to “in pipeline” on Jira?

Checklist PR Raiser
Before creating PR

[ ]  Have you run the tests?
[ ]  Have you run the journey tests? (where applicable)
[ ]  Have you addressed warnings where appropriate?
[ ]  Have you rebased against the current version of main?
[ ]  Have you checked code coverage isn’t lower than previously?

After PRs been raised

[ ]  Have you checked the PR Builder passes?

taxYear format would remain as YY-YY after clarification and cross checking with spec


RDM-11933: Case reference type changed to TEXT

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RDM-11933
Change description
Case reference type changed to TEXT
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X ] No

Merged straight after approval and green build, because of urgency of build fix.


publish to homebrew

I want to 'brew install msbuild' on my mac
http://docs.brew.sh/Formula-Cookbook.html

Until that happens, if you need to programatically get msbuild for vm setup and such, you could bring down dotnet CLI which also packages msbuild. They have a script for it: https://github.com/dotnet/cli/blob/rel/1.0.0/scripts/obtain/dotnet-install.sh
Not ideal, but it is a one liner command line.

We don't currently have any plans to do this. On non-Windows, you'll always need a runtime host (either dotnet CLI or Mono) to execute MSBuild, and both of those include MSBuild, so there's not much reason for us to go through the packaging effort. In addition, there are all the problems involved with getting the right set of SDKs, tools, targets, and tasks to be able to do something useful with MSBuild--which are solved by the CLI installer.

both of those include MSBuild

Do they? I don't think the mono package on brew does.  It certainly doesn't on Ubuntu (it's in a separate package called msbuild, but I don't think there's an equivalent for that in homebrew)

My understanding is that Mono carries MSBuild as of (some version number I don't know). @radical or @directhex, am I misunderstanding?

Mono-the-project does (starting in 5.0, IIRC). Mono-the-package doesn't, at least on Ubuntu and IIUC on brew, currently.

Mono for Mac bundles msbuild as part of the mono.pkg download. But there is a LOT in that package beyond what's in mono.git.
RE Linux: There are msbuild packages on download.mono-project.com, but msbuild as-is is not suitable for inclusion directly into Debian/Ubuntu.

Also note that the mono that is on homebrew is not maintained by us (Xamarin/Mono team) but some third party. It's best to use the .pkg download from http://www.mono-project.com.

@rainersigwald You're wrong, says @akoeplinger – you need msbuild to be installed for mono to work properly nowadays.

Looks like homebrew now bundles msbuild with its distribution of mono https://github.com/Homebrew/homebrew-core/pull/29422


Enabling example generation results in issues generating documentation for some types

Describe the bug
I recently upgraded graphql-markdown in order to get access to the feature for displaying examples in the documentation.  In doing so, I've started hitting issues where I get warnings for certain types in the schema that are preventing those types from having their documentation generated.  The specific error behind the warning appears to be a RangeError that looks a lot like a recursive loop.  However, in closer debugging I noticed that the files getting pulled into my node_modules folder for the printer-legacy module within graphql-markdown are noticeably out of date.  The ZIP file dowloaded from NPM for graphql-markdown contains the correct files but the ZIP file for the printer-legacy submodule contains the old files.  I'm concerned that having this old code may well be the reason that I'm having issues with what appears to be a recently introduced feature so I wanted to provide all of that context as I think it's the right place to start.  Note that if I set printTypeOptions.exampleSection to false, then this issue will not occur, so this is definitely related to the example generation code.
To Reproduce
yarn run docusaurus graphql-to-doc

GraphQL schema

If applicable, add an extract of the GraphQL schema.
public_schema.txt

Docusaurus configuration

If applicable, add an extract of the Docusaurus configuration.
        printTypeOptions: {
          exampleSection: true,
          hierarchy: 'entity',
        },

Expected behavior
I would expect the command to run without errors while generating appropriate documentation for all types in the schema.
Screenshots
Sample output showing how this error manifests when running the tool:
yarn run docusaurus graphql-to-doc
...
[WARNING] An error occurred while processing "VotingLocationConnection"
[WARNING] An error occurred while processing "VotingLocationEdge"
[WARNING] An error occurred while processing "HasCandidacies"
[WARNING] An error occurred while processing "HasRaces"
[SUCCESS] Documentation successfully generated in "docs/api/graphql/reference" with base URL "docs/api/graphql/reference".
[INFO] 173 pages generated in 1.956s from schema "../ballot_factory/graphql/public_schema.graphql".
✨  Done in 4.74s.

Relevant code in repo from what I can tell: https://github.com/graphql-markdown/graphql-markdown/tree/main/packages/printer-legacy/src
Contents of local directory, clearly still JS and not the new TS files:
trhardy@MacBook-Pro-2 @graphql-markdown % cd /node_modules/@graphql-markdown/printer-legacy/dist
trhardy@MacBook-Pro-2 dist % ls
badge.d.ts common.js example.js group.js printer.d.ts section.js
badge.js const frontmatter.d.ts index.d.ts printer.js
code.d.ts directive.d.ts frontmatter.js index.js relation.d.ts
code.js directive.js graphql link.d.ts relation.js
common.d.ts example.d.ts group.d.ts link.js section.d.ts

Additional context
If you're looking for a fuller picture of the issue, I added a little bit of debugging code to my local copy of example.js to try and understand what is causing the warnings.  This is an example of the output of that debugging code for one of the affected schema types.  It just prints out the error that is being caught that results in the warning being printed.  All the types that are generating warnings have underlying errors that look like that.  Again I strongly suspect we should address the misaligned packaged code first but feel like I should share everything I have.
RangeError: Maximum call stack size exceeded
    at parseTypeFields (/Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:33:25)
    at parseExampleDirective (/Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:100:19)
    at /Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:52:23
    at Array.forEach (<anonymous>)
    at parseTypeFields (/Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:47:10)
    at parseExampleDirective (/Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:100:19)
    at /Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:52:23
    at Array.forEach (<anonymous>)
    at parseTypeFields (/Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:47:10)
    at parseExampleDirective (/Users/trhardy/Documents/Git_Repos/developer-documentation/node_modules/@graphql-markdown/printer-legacy/dist/example.js:100:19)
[WARNING] An error occurred while processing "HasRaces"

Thanks for the report. I’ll be looking into it over the weekend.

Quick comment on Contents of local directory, clearly still JS and not the new TS files: the package is built for esnext, so it is normal that build/dist files in the npm package are JS.

I managed to reproduced the issue, and you are right the reason is a recursive loop created by the recursive type Organization in your schema.
type Organization implements DatabaseIdentifiable, Node, Timestamps {
  children: [Organization!]!
}

I'll add support for recursion in types 🤔

Fixed with release 1.26.4 ✅


feat: morgan.ini: +multiline values

closes #15

I'd like to note this that also fixes a problem I just ran into. I was having trouble with pyyaml not being able to build because Cython<3.0 hadn't been pulled in. Turns out it's because morgan's just seeing that "Cython" has already been processed, without paying attention to the requirement. A previous package had some other requirement for Cython which was fulfilled by 3.0.8, which clearly isn't fulfilled by <3.0, but checking the name instead of the full requirement missed it.
This also fixes my issue #17 another similar but slightly different issue.

That's great, I knew from the beginning that the _processed_pkgs map would not be sufficient. I think with these changes there'll still be recursion if two different requirement strings resolve to the same version. I'll run some tests tomorrow and try to get this in ASAP.


Attachments from '1_Test/Attachments' are pointed by html file

Attachments are copied from directory '1_Test/Attachments' to directory 'Attachments' while creating html report. But still the files from directory '1_Test/Attachments' are pointed in the html report

Attachments are not copied over to the Attachments folder by xchtmlreport. The result bundle is provided by Xcode like that.


Treasury - Reallocate Funds from Previous Farmer to Current Farmer

The amount requested to transfer:
All assets in the following wallets:
0x4c3ADd465413A85EfcD24a523A3cEb68B7566F95
cosmos1zzfvpqr2kwvxf4x23l8dn5n3vkcyjcpa4rtw52
terra1elzncl4032mvaxyvfynvwwahplp2pfhadj6wd2
thor1uucdcz7nq69mrtxgxy27rg878r78wn0q69pace
The recipient of the amount. Each additional recipient will receive the specified amount in the proposal:
0x4c3ADd465413A85EfcD24a523A3cEb68B7566F95
The purpose of the transfer. This can include the allocation of the funds to delegated traders/farmers, to conduct purchases of specific assets, to perform token buybacks, or to pay external parties for services:
Allocation of funds to current delegated trader, @eggwards (telegram username).
Justification for the transfer, if one or any of the recipient addresses are not known to the team:
Not required.

I like the idea . I do believe it should be clarifying the value of these wallets of which ed will end taking control off.

Instead of transferring The funds in only those wallets ? I think we should transfer 75% of the treasury or even 100% if we are happy with EDs transparency and risk management. A 5% gain on 400k is obviously much better than on 120k so why not utilise this market. IMO I anticipate ETH doing a good 15% pump toward the merger date 15th sept 2022. Lot of potential to buy the hype now and sell 1-2 days before.

We cant leave the money to much to dust into the wallets slowly decreasing in vlaue. In favor of moving the the funds into the wallet of the delegated trader.

For sure we have to move the funds from Diego wallet and use them to trade more with Ed.

This has been approved unanimously by the aggregatedfinance PM's.
This proposal has been moved to vote on snapshot :-
https://snapshot.org/#/aggregatedfinance.eth/proposal/0xba703f9423f38d1f63bdf43f700b8399e6f04a79303e8f17683f50b9c0614936

Voting result FOR. Agfi official and trader ed will arrange the transfer


The ip ranges are NOT the latest, please keep the docs synced, thanks!

[Enter feedback here]
https://docs.microsoft.com/en-us/microsoft-365/enterprise/urls-and-ip-address-ranges?view=o365-worldwide
I didn't found the IP: <IP_ADDRESS> in the document above, but this IP is come from Graph Notification Validation Request!
it caused plenty of "Subscription validation request timed out" error in Graph SDK from our service.
I don't know if there are any other IPs which will send Graph Notification Validation Request, if so, please keep document SYNCED so that we could sync security group of our service, plz, plz, plz, thanks !!!

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

ID: 66bd8666-3566-a901-b25b-eb6baba6264a
Version Independent ID: e9f6310f-2207-8c81-d31a-3d5a80beb26b
Content: Office 365 URLs and IP address ranges - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/urls-and-ip-address-ranges.md
Service: o365-administration
GitHub Login: @JoeDavies-MSFT
Microsoft Alias: josephd

@Code2Life Not all of the IP addresses are listed. See this blog post for an explanation: https://techcommunity.microsoft.com/t5/office-365-networking/why-are-some-office-365-network-endpoints-provided-as-urls-only/m-p/310490#M97
Hope this helps.


Path module updates and fixes

Hello,
Just a suggestion...
We have 2 major path to support : "unix" and "windows".
Most of the difference stay arround of string manipulations.
IMHO It should be interesting to have the both implementation available at the same time.
we will be able to test the both implementation from unix and windows.
It should be setup inside in path.windows.isabs and path.unix.isabs
Finaly the default function should just be mapped to the proper platform (path.isabs should be path.windows.isabs or path.unix.isabs).
Regards,

@tst2005 I did run across that problem, while reworking this module, that is an interesting solution.


[WFCORE-2335] It is possible to set buffer pool buffers-per-slice att…

…ribute to negative value
JIRA: https://issues.jboss.org/browse/WFCORE-2335

@ctomc please approve


Create cedarridgeacademy.txt

@aregius I am sorry, your school website does not look like a website of an established institution. There is an institution with the same name (and different website) which is an educational institution, but they do not seem to provide any IT-related courses.
Thus, I have to close your request without merging because the institution you ask us to add to the repository does not match the rules. You can reopen the request and comment, if necessary. Thank you for the understanding.


Tool not configured yet

Hi, I installed it on a Windows 10 PC with one admin account and one regular account. Logging in to the regular account / windows profile I get now everytime the message "Tool not configured yet" - no matter if I configured it in between. After rebooting, again the message "Tool not configured yet".
How can I solve the issue?
Cheers

Hey!
It should work regardless of admin or regular user account as the config is written to the user's appdata folder such as C:\Users\username\AppData\Local\DNS_Swapper\DNS-Swapper.exe_Url_2vkzryni5blagkgybxtiai1dd3qne5zd\<IP_ADDRESS>\user.config. The configuration is saved when you exit the tool or do it manually via File -> Save. It probably didn't save if you just configured it and rebooted afterward.
I should probably add automatic saving if a field is changed.
Could you please try manually saving or exiting the application and let me know if that works?
Please note that changing the DNS settings requires elevated permissions, so an Admin would need to enter his / her credentials every time the user wants to change the DNS.

Hi, thanks.  Before I did not save it explicit because I thought it will store it automatically.
Anyhow, great tool!
Thank you for your support!
I am by the way searching something similar for Linux ... if somebody here does have a tip ... would be appreciated ;-)


🌐 Add Japanese translation for docs/ja/docs/advanced/templates.md

This PR translates advanced/templates.md in Japanese, help me review when you have time. Thank you!

📝 Docs preview for commit fd64969f51a262501568241dcb3716c274f14220 at: https://649a17df51d130136dc7df41--fastapi.netlify.app

As this PR had requested changes to be applied but has been inactive for a while, it's now going to be closed. But if there's anyone interested, feel free to create a new PR.


Update docstrings for correct parameter names

At some point we switched from using key to using field but never
reflected those changes in the docstrings.

:+1:


Use php8.1, support symfony 5 or 6

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 98.529%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
67

Relevant Lines:
68

💛  - Coveralls


Easy Auth doesn't supply tokens blocking use of Graph API

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
First of all, I love Container Apps and I love Easy Auth - thank you for your work on them! ❤
I'm a long time user of Easy Auth and was delighted when it landed in ACAs. I uncovered an issue early on around how to get .NET to recognise the X-MS-CLIENT-PRINCIPAL header that is supplied with Easy Auth and wrote up a workaround: https://johnnyreilly.com/azure-container-apps-easy-auth-and-dotnet-authentication
This has served me well, however I'm now trying to make use of the Graph API on behalf of my logged in user.  I cannot.
Reading the docs I believe I should be trying to use the "on behalf of" approach:
var scopes = new[] { "https://graph.microsoft.com/.default" };

// Multi-tenant apps can use "common",
// single-tenant apps must use the tenant ID from the Azure portal
var tenantId = "common";

// Values from app registration
var clientId = "YOUR_CLIENT_ID";
var clientSecret = "YOUR_CLIENT_SECRET";

// using Azure.Identity;
var options = new OnBehalfOfCredentialOptions
{
    AuthorityHost = AzureAuthorityHosts.AzurePublicCloud,
};

// This is the incoming token to exchange using on-behalf-of flow
var oboToken = "JWT_TOKEN_TO_EXCHANGE";

var onBehalfOfCredential = new OnBehalfOfCredential(
    tenantId, clientId, clientSecret, oboToken, options);

var graphClient = new GraphServiceClient(onBehalfOfCredential, scopes);

The problem is, there's not JWT token that we can make use of in the headers.  This is supported in App Service which has a token store and supports the following headers:
X-MS-TOKEN-AAD-ID-TOKEN
X-MS-TOKEN-AAD-ACCESS-TOKEN
X-MS-TOKEN-AAD-EXPIRES-ON
X-MS-TOKEN-AAD-REFRESH-TOKEN

If there was a populated X-MS-TOKEN-AAD-ACCESS-TOKEN then using the "on behalf of" approach above would work.  As there are not such headers in Easy Auth with Azure Container Apps, this is not possible. To be clear X-MS-CLIENT-PRINCIPAL is not a JWT - it's a base 64 encoded JSON representation of the principal.
There's definitely some overlap between this issue and https://github.com/microsoft/azure-container-apps/issues/479 - it's not clear to me that they are exactly the same issue though; so I've raised this instead.  I'll copy the content across to the other in case this a dupe in which case this can be closed.
Steps to reproduce
Add a controller to your app to surface headers:
    [HttpGet("api/headers")]
    public async Task<IActionResult> GetHeaders()
    {
        var headers = new Dictionary<string, string>();
        foreach (var header in Request.Headers) {
            if (headers.Keys.Contains(header.Key))
            {
                headers[header.Key] = header.Value.FirstOrDefault() ?? "";
            }
            else
            {
                headers.Add(header.Key, header.Value.FirstOrDefault() ?? "");
            }
        }
        return Ok(headers);
    }

Take a look at what comes out.
Expected behavior [What you expected to happen.]
Going to /api/headers surfaces headers which include the following:
X-MS-TOKEN-AAD-ID-TOKEN
X-MS-TOKEN-AAD-ACCESS-TOKEN
X-MS-TOKEN-AAD-EXPIRES-ON
X-MS-TOKEN-AAD-REFRESH-TOKEN

i.e. JWT tokens in headers that can be consumed and used.
Actual behavior [What actually happened.]
Headers do not include the hoped for tokens. These are the headers I encountered:
[
    "Accept",
    "Host",
    "User-Agent",
    "Accept-Encoding",
    "Accept-Language",
    "Cookie",
    "traceparent",
    "Upgrade-Insecure-Requests",
    "Content-Length",
    "sec-ch-ua",
    "sec-ch-ua-mobile",
    "sec-ch-ua-platform",
    "DNT",
    "sec-fetch-site",
    "sec-fetch-mode",
    "sec-fetch-user",
    "sec-fetch-dest",
    "x-forwarded-for",
    "x-envoy-external-address",
    "X-Request-ID",
    "x-envoy-expected-rq-timeout-ms",
    "x-k8se-app-name",
    "x-k8se-app-namespace",
    "x-k8se-protocol",
    "x-k8se-app-kind",
    "x-ms-containerapp-name",
    "x-ms-containerapp-revision-name",
    "x-arr-ssl",
    "x-forwarded-proto",
    "X-MS-CLIENT-PRINCIPAL-NAME",
    "X-MS-CLIENT-PRINCIPAL-ID",
    "X-MS-CLIENT-PRINCIPAL-IDP",
    "X-MS-CLIENT-PRINCIPAL"
]

Hi all, now containerapp has supported token store feature since api-version 20230502preview. users can provide a blob storage sas url (need contain enough permission) to containerapp secret (name could be arbitrary) and then use following command:
az containerapp auth update -g myResourceGroup -n cotanierappName --token-store true --sas-url-secret-name {secretName}
After that, we can get those headers from incoming request:
X-MS-TOKEN-AAD-ID-TOKEN
X-MS-TOKEN-AAD-ACCESS-TOKEN
X-MS-TOKEN-AAD-EXPIRES-ON
X-MS-TOKEN-AAD-REFRESH-TOKEN
and /.auth/me and /.auth/refresh can work fine

Thanks for sharing @njuCZ! Is there any documentation on this? Do you know if a more plug and play version of this is likely to become available?  I think that App service etc don't require that you have to manually set up a storage sas URL etc?

Hi @johnnyreilly, app service supports two types of token store feature: default shared file system token store and blob storage sas url token store. However, containerapp currently only supports blob storage sas url token store. There is no document on this currently, I will work with PM for the official document

Thanks @njuCZ.  Do you know if app service will support an equivalent to default shared file system in future?
Also, in my world we're a Bicep shop - we deploy everything with Bicep.  If the blob storage SAS URL token store is the only option, would you be able to provide documentation on how to use it with that?  Thanks!

@johnnyreilly currently there is no plan to support the default shared file system token store feature.
after the document is ready, I will share here. You could use the azure cli command I provided to enable it

Thank you!

Hi all, now containerapp has supported token store feature since api-version 20230502preview. users can provide a blob storage sas url (need contain enough permission) to containerapp secret (name could be arbitrary) and then use following command:
az containerapp auth update -g myResourceGroup -n cotanierappName --token-store true --sas-url-secret-name {secretName}
After that, we can get those headers from incoming request: X-MS-TOKEN-AAD-ID-TOKEN X-MS-TOKEN-AAD-ACCESS-TOKEN X-MS-TOKEN-AAD-EXPIRES-ON X-MS-TOKEN-AAD-REFRESH-TOKEN
and /.auth/me and /.auth/refresh can work fine

@njuCZ I'm using 2023-08-01-preview based on the deployment arm templates in the resource group. I've added the --token-store and the blob sasUrl.
{
"type": "Microsoft.App/containerApps/authConfigs",
"apiVersion": "2023-08-01-preview",
....
"login": {
                    "routes": {},
                    "tokenStore": {
                        "enabled": true,
                        "azureBlobStorage": {
                            "sasUrlSettingName": "token-store-sas-url"
                        }
                    },
                    "preserveUrlFragmentsForLogins": false,
                    "cookieExpiration": {},
                    "nonce": {}
                },
...
}

I'm still not able to see the X-MS-TOKEN-AAD-ACCESS-TOKEN header.

What are the specific permissions on the sas url? I've given all permissions (everything checkbox is checked) and used the url to the blob endpoint.

Has there been a regression and I should use the 20230502preview version specifically to get the access token headers to show up?

Are there specific logs in the EasyAuth container that I could look at to debug further?

Thanks,

Hi @ltrain777, it seems you provide the wrong sas url, you could follow the doc here: https://learn.microsoft.com/en-us/azure/container-apps/token-store
This should be the right portal for getting a blob storage sas url  .

@njuCZ Thank you for pointing me to the correct documentation. I updated the secret with the sas url to the private container sas url. I created another revision and updated the container. I'm still not getting the x-ms-token headers. Are there debugging logs that might help me find the issue?
One interesting difference I found was that when token-store is true I don't get 404 when going to /.auth/me. That endpoint now returns empty array. The /.auth/refresh endpoint gives me 403.

you should see some error logs in your console logs if you have configured the logging. There should be nothing special, several points to check:

check the secret value is correct sas url.
enable easyauth and enable tokenstore. you can use azure cli to show the right values.

If you still encountered issue, you could send email to acasupport at microsoft.com with your containerapp info, then we could take a check for you.

I think this part of the documentation may be incorrect:

Additionally, you can create your store using an ARM template

This doesn't appear to be an ARM template reference. Is there an ARM / Bicep template reference available please?

Hi @johnnyreilly thanks for reporting it, we will fix the link soon.

Hi @anthonychu - forgive me if I'm getting the wrong end of the stick, but I don't think that's an ARM template?

@ltrain777 you should see some error logs in your console logs if you have configured the logging. There should be nothing special, several points to check:

check the secret value is correct sas url.
enable easyauth and enable tokenstore. you can use azure cli to show the right values.

If you still encountered issue, you could send email to acasupport at microsoft.com with your containerapp info, then we could take a check for you.

@njuCZ I had too many sas urls generated on my scratchpad and used the incorrect one with only read permissions to the container. I used the correct sas url with read, write, delete and I'm now able to get the access tokens. Thank you for your help!

Hi @njuCZ token store make life so much easier. All working fine, except i'm not able to get the X-MS-TOKEN-AAD-REFRESH-TOKEN header.
I also tried to add OAuth permissions to the SPN

Any plans to allow access to the token store storage account using managed identity, rather than having to manage SAS?

Hi @hbuckle , we are going to release this feature soon. Once it's available, we will update here.

the feature token store auth using msi has been released in api-version: 2024-10-02-preview. Storage Blob Data Contributor role should be assigned and the swagger is as following:


Telemetry-only Phoenix support

In the current version of the Plug integration, we use Appsignal.Plug to add instrumentation to an application endpoint, which wraps the application’s endpoint call to create a root span. The new version of AppSignal for Plug is a zero-setup Telemetry-based instrumentation, which automatically hooks into Plug applications that have the Plug.Telemetry included.

Through working on this issue, I found that what I originally wanted to do will work, but not for pure Plug apps. Therefor, in https://github.com/appsignal/appsignal-elixir-phoenix/pull/56, we’re implementing this change for Phoenix specifically.
The issue here is that although we can get endpoint start and -stop events from Plug.Telemetry, there’s currently no way to catch errors through Telemetry alone. In Phoenix, there’s an event for that we can hook into.
By implementing this change for Phoenix, we’re able to remove the dependency on AppSignal.Plug. The Plug library will remain as is for now.

Closed by #56.


Add rpsene and aishwaryabk as reviewer, remove bpradipt as approver

Signed-off-by: CS Zhang<EMAIL_ADDRESS>

/approve

/lgtm


ftl-project.toml should be generated if doesn't exist when running ftl init

I can grab this since I'm already working on https://github.com/TBD54566975/ftl/issues/1561

IIRC the reason for this was so the VSCode extension would detect an FTL project, though strangely I do not have this issue in a new project I create.

Aaah, because it looks for ftl in bin/ftl I think @wesbillman ? What are the heuristics it uses?
Regardless, I think creating an ftl-project.toml in the Git root is a very good idea.


san-mui官网增加首页

san-mui官网增加首页home.san

Coverage remained the same at 100.0% when pulling 08bf0c181c98191e8addc770ca128524a6bdae6e on ceadaTian:master into 0a5cae4b83877359cdb45b3815ac5ec990a4729c on ecomfe:master.


Add TypeScript Typing

Make your typescript program run, in the least degree.
Solve #7.

Awesome! I was in need for this yesterday haha
Thanks for contributing

Just published it under version 1.0.13


confusion

@Toneyaya
Nice work !

In this line, self.cgattention2 is shared with the last two Group Transformer Layers.

The implementation of Eq. 8 using this line is inconsistent with such illustration in paper that needs to compute the similarity between the referent query token and the global linguistic feature e.

The implementation of LoadLayer does not consider pe_shape <= 30, meaning that the LoadLayer in the first Group Transformer Layer  is discarded, and in each  LoadLayer, only the referent query token is interacted with word features.

The paper claims that different query tokens can represent different objects/regions and the first one is to highlight referring object.   However,  in your implementation, only two query tokens are introduced, one is as referent query token and the other is to represent background. This may  not match your motivation, and  more query tokens (>2) may be a reasonable settting and meet your claim.

@Toneyaya
Thanks for your reply.
For current 2-tokens version, its results are definitely different from that in paper. So could you provide the corresponding results on RIS datasets (or RefCOCO) as a reference and comparison ?


Batarangle state doesn't update on page refresh

Steps to reproduce:

Run the todo-mvc-example
Add some tasks
Open batarangle: the components are shown as expected.
Refresh the browser window

The tasks go away in the app (because the example app isn't persistent).  However batarangle still shows all the components from the previous run.
Workaround: close and reopen dev tools.

yes, same problem, workaround works

@midroid we are working on that target is to fix it with beta.3 fixes

This issue still exist on my newest version of batarangle? Any idea?

@xinrui-ma If you are using latest version of Angular and Batarangle this shouldn't be the case if you reload your angular 2 application.


Plugin management section is not taken into acount

We have recently added the spottles plugin and set up the plugin version only in the plugin management section.
Despite the fact that it is the most recent version of the plugin, action reports vulnerability that existed in the old version and has already been fixed.
if we add a version of the plugin in the plugin section of pom.xml, the review passes without problems.

Hello, thanks for reporting this 🙇
Can you provide more detail (specific repo, plugin/versions, example POM.xml etc.) and context for reproducing this?


Saving stacking

Is it possible to dump the stacking ensemble using joblib dump just as scikit-learn? will this store all the estimators within as well?
On another note, It would also be useful to have a save method where it will iterate through the estimators and store each one individually (should consider pipelines as well).

Yes. The stacking estimator should pickle just fine. It has an attribute for accessing the individual estimators (named_base_estimators), so dumping them should be a loop
for name, est in stk.named_estimators.items():
    with open(name + '.pkl', 'w') as fp:
        pickle.dump(est, fp)

You can dump the meta estimator using its attribute as well (meta_estimator).
My preference is to not have an extra method since it would have to make a lot of assumptions about file names, etc.
Let me know if you have more questions.

Thanks it worked.
I think you are right about your reference. Cheers!


Rebuilding ChromiumFXRenderProcess.exe

Latest /app-x.x.x/ChromiumFXRenderProcess.exe is getting flagged by Windows Defender. I'm submitting in to Microsoft as a false positive now. But I think I need a quick way to make a custom build of it.
So I have a couple of questions:

Could you share how to build ChromiumFXRenderProcess.exe?
Just curious, what is another ChromiumFXRenderProcess.exe in the root of the app directory? Doesn't seem to be used.

Ok, thank you. I got this file unflagged by Microsoft, but nice to know it's easy to rebuild it too.


Require python<3.13 and add typing-extensions to requirements

Fix #363
From @nelimee:

When trying to install tqec, I had an issue due to Python 3.13 being the newest Python version but most of our dependencies (and in particular stim) do not have yet pre-built binaries, leading to some issues related to CPython imports in stim.

The typing-extensions package seems to be missing from our requirements. I though it was a standard python package, it seems like this is not the case.

So Python 3.13 is likely not an issue here. I do not think we have anything in the code base that would prevent us using Python 3.13. The issue was really local to my environment, likely due to the delay between the first Python release and the availability of pre-compiled binaries for our dependencies. I think we should still allow Python 3.13, people using it probably know that there will be some edge-cases due to the fact that it went out recently.


Fix Machines Pages Buttons

This fixes the buttons on the machines pages so that they actually work.

This is good from my end. @tpetr @kwm4385 @wolfd

I took the opportunity to replace the OldTable component with SimpleTable. Along with #1140, this almost fully removes that component.

LGTM


Returning 308 Redirect on Merge Request Update

Hello, thank you for the great library!
I'm attempting to PUT to a merge request via the UpdateMergeRequest function but am getting a 308 redirect response which I think is preventing me from making the PUT. This seems quite similar to another issue from a few years ago: https://github.com/xanzy/go-gitlab/issues/523
This request works via Curl, for instance like this:
curl --location --request PUT 'https://www.gitlab.com/api/v4/projects/45056707/merge_requests/9' \
--header 'PRIVATE-TOKEN: MY_TOKEN' \
--header 'Content-Type: application/json' \
--data '{
    "description": "This is a new description"
}'

Is there something that I need to do to configure the client to allow it to follow these redirects? I'm initializing the client like this:
client, err := gitlab.NewClient(authToken, gitlab.WithBaseURL("https://www.gitlab.com/api/v4"), gitlab.WithCustomLogger(l))

And the call is pretty straightforward:
mergeRequestOptions := gitlab.UpdateMergeRequestOptions{
    Description: gitlab.String("Some description"),
}

_, res, err := client.UpdateMergeRequest(c.projectId, c.mergeId, &mergeRequestOptions)

EDIT: This seems to be happening because I'm using gitlab.withBaseURL when initializing the client. Is there some way for me to configure the client with a a custom base URL that doesn't cause this issue?
This is using v0.83.0

Apologies, must have done something wrong in my code, I'm not able to replicate this with a simpler repository. Closing this issue.

@harrisoncramer ,
This library appends /api/v4 internally for each call. So in your case, final url for api would've been https://www.gitlab.com/api/v4/api/v4/.....
Can you try with setting url to just https://www.gitlab.com instead?

You don't need to set a base URL when using gitlab.com. That is only needed when you want to use a private GitLab server.

Got it, that makes sense.
By the way, I'm building a Gitlab client for Neovim on top of this library, if anyone would like to contribute, it's here: https://github.com/harrisoncramer/gitlab.nvim
I'll make a note in that repository's README that you should not pass in a base URL unless you are using a self-hosted instance of Gitlab.


Add jp2k  et al to FORMATS_WITHOUT_PROFILE_SUPPORT

FORMATS_WITHOUT_PROFILE_SUPPORT  now includes jpc jpt jp2 j2c j2k.
These formats all triggered the same error when running tests. Checking the rubydocs confirmed that jp2ksave and its friends jp2ksave_buffer and jp2ksave_target don't have a profile option, while savers for formats like jpeg and png do.
In addition, while running tests

Added ** when using DragonflyLibvips.symbolize_keys as an arg.
Changed tests to use _(result.property).must as suggested by minitest.

Also
My local system is still getting test failures for encode converting anything to .raw
I'll open another issue for that.

Thanks @anitagraham! Let me know when ready, and I'll merge and release new version.

Added raw to the unsupported output formats.
encode_test.tb was failing with Minitest::UnexpectedError:         Vips::Error: VipsForeignSave: ".raw" is not a known buffer format, as described in #17
Checking with  vips -l | grep _target as advised (on stackoverflow) does not include raw as one of the targets.
On the other hand, rubydocs lists VipsImage.rawsave.  Maybe I will rethink this.

So, it appears that ruby-vips has VipsForeignSaveRaw and VipsForeignSaveRawFd (saves to file and to file descriptor), but not VipsForeignSaveRawBuffer (according to vips -l | grep raw. ( vips-8.11.3-Wed Aug 11 09:29:27 UTC 2021).
Encode (and all the processors) return a buffer.
Perhaps it is just a matter of asking for VipsForeignSaveRawBuffer to be added to ruby-vips.

Thanks @anitagraham!


Remove implicit input casting to strings

This PR addresses an issue discussed in #15 where any input type can be converted to string if the feature type is string.
Example: rouge expects a single string as reference. If a list of strings is passed instead (like for bleu) the list is currently converted to a string without an error.
Since pa.array casts to the provided input types we need to throw an error before that. The method Value.encode_example could be a good place. This PR applies a patch to the Value objects in the metrics features.
However, there is an issue with pickling the patched object. Any ideas @lhoestq @albertvillanova?

Ok, switch to a standalone function. Used encode_nested_example to traverse the nested structure and check each subtype. What do you think?


Tree shaking not applied for node modules

When i use rollup with node_modules unused exports also appear in the bundled file.
Shouldnt it apply tree shaking also for node_modules? Or just im making something wrong.
Of course these unused exports disappear when uglified, but according to the official Rollup examples these should not be there before uglify.
I made an example repo to show my problem.
https://github.com/blacksonic/es2015-tree-shaking
I use RxJS, and it always applies operators for example, which makes the size bigger. Even if i uglify.

Closing in favor of rollup/rollup#663.


test merge with NLog.Contrib

Targets of NLog.Contrib are merged with the core (https://github.com/NLog/NLog/pull/833). But will this give issues if NLog.Contrib is used with NLog 4.1? (same types in different assemblies)
We should test/check that.

Test successfull:
test code
        static void Main(string[] args)
        {
            Logger _logger = LogManager.GetCurrentClassLogger();

            MappedDiagnosticsLogicalContext.Set("a","BBBB2");
            _logger.Info("123 ");
        }

   <column layout="${message} ${mdlc:a}" name="Message"/>

[x] works with nlog.extended package (<IP_ADDRESS>)
[x] works without nlog.extended package (<IP_ADDRESS>) (NLog 4.1 RC1)


Use median file covered_strength for project covered_strength to mitigat...

...e outliers.

Closing due to inactivity...  related to https://github.com/colszowka/simplecov/issues/306


Add fred repo to #modding-info-feed

Added with the same configuration as the SML webhook. I'll wait until we receive something on the webhook before closing


toc/GOVERNANCE.md needs to go away

Any content in this file that is unique should be placed somewhere else. This file should be gone by the end of the process.

@cloudfoundry/toc Looking over toc/TOC.md, toc/ROLES.md and toc/working-groups/*.md, I don't think there's anything from toc/GOVERNANCE.md that isn't covered elsewhere. I'd appreciate a second pair of eyes to confirm that though, before I go ahead and raise a PR to remove the doc.

fixed in #398


Model zoom

In Sceneform sample app we are able to zoom the model to a level only, beyond that level zoom in zoom out does not work. I want to make the model zoom in zoom out to any level. How can we achieve this.

I got the solution. Thanks