Bank tags only show the first tag in its Tags

My bank tags will only show the very first tag I have it listed under.
For example: I have my Slayer Helmet tagged with "kq sire dk goril" and the item will only show under the "kq" tag.
This happens will any tagged item, and always only shows the very first tag.
Any help or advice would be greatly appreciated!

Note that your tags need to be comma separated, is this the case?

Yes it was, that was fast! It had always worked without them for me until a week ago, thanks!

That's because they weren't quite working correctly for a long time. :P


Using plugin as Segmented Tabs

Hello again. I have tested in my real project but below code seems does not work. This approach is right or not?
<cards:CarouselView>

                <cards:CarouselView.ItemsSource>
                    <x:Array Type="{x:Type View}">

                        <ContentView BackgroundColor="Orange">

                                <ListView
                                    x:Name="xBindableLayout"
                                    CachingStrategy="RecycleElementAndDataTemplate"
                                    GroupDisplayBinding="{Binding Key}"
                                    HasUnevenRows="True"
                                    IsGroupingEnabled="True"
                                    ItemTemplate="{StaticResource xPolicyDetail}"
                                    ItemsSource="{Binding PolicyItemsAsGroup}"
                                    SeparatorVisibility="None"
                                    VerticalScrollBarVisibility="Never">

                                    <ListView.GroupHeaderTemplate>
                                        <DataTemplate>
                                            <ViewCell>

                                                <yummy:PancakeView
                                                    Margin="10,-1,10,1"
                                                    Padding="10,5"
                                                    BackgroundColor="{Binding Key.BackgroundColor}"
                                                    CornerRadius="5,5,0,0"
                                                    HasShadow="False">

                                                    <yummy:PancakeView.Behaviors>
                                                        <cbehaviors:ViewTappedBehavior
                                                            AnimationType="Fade"
                                                            Command="{Binding Path=BindingContext.ExpandItemsCommand, Source={x:Reference xBindableLayout}}"
                                                            CommandParameter="{Binding Key}" />
                                                    </yummy:PancakeView.Behaviors>

                                                    <yummy:PancakeView.GestureRecognizers>
                                                        <TapGestureRecognizer />
                                                    </yummy:PancakeView.GestureRecognizers>

                                                    <Grid ColumnSpacing="5">
                                                        <Grid.ColumnDefinitions>
                                                            <ColumnDefinition Width="*" />
                                                            <ColumnDefinition Width="Auto" />
                                                            <ColumnDefinition Width="30" />
                                                        </Grid.ColumnDefinitions>

                                                        <Label
                                                            Grid.Column="0"
                                                            FontSize="22"
                                                            HorizontalOptions="StartAndExpand"
                                                            Text="{Binding Key.Title}"
                                                            TextColor="White"
                                                            VerticalOptions="Center" />

                                                        <Label
                                                            Grid.Column="1"
                                                            FontSize="14"
                                                            HorizontalOptions="StartAndExpand"
                                                            Text="{Binding Key.SubTitle}"
                                                            TextColor="White"
                                                            VerticalOptions="Center" />

                                                        <Label
                                                            Grid.Column="2"
                                                            FontFamily="{StaticResource FontAwesomeSolid}"
                                                            FontSize="24"
                                                            HorizontalOptions="EndAndExpand"
                                                            Text="{Binding Key.StateIcon}"
                                                            TextColor="White"
                                                            VerticalOptions="Center" />

                                                    </Grid>

                                                </yummy:PancakeView>

                                            </ViewCell>
                                        </DataTemplate>
                                    </ListView.GroupHeaderTemplate>

                                </ListView>

                        </ContentView>
                        <ContentView BackgroundColor="Red" />
                        <ContentView BackgroundColor="Green" />

                    </x:Array>
                </cards:CarouselView.ItemsSource>

            </cards:CarouselView>

I am getting below warning too
NU1603: CardsView 2.4.6 depends on Xamarin.Forms (>= 3.0.0) but Xamarin.Forms 3.0.0 was not found. An approximate best match of Xamarin.Forms <IP_ADDRESS>6417 was resolved.
My Xamarin Forms version is <IP_ADDRESS>6

I am getting below warning too
NU1603: CardsView 2.4.6 depends on Xamarin.Forms (>= 3.0.0) but Xamarin.Forms 3.0.0 was not found. An approximate best match of Xamarin.Forms <IP_ADDRESS>6417 was resolved.
My Xamarin Forms version is <IP_ADDRESS>6

that's very strange. >= 3.0.0
But don't think it's the reason

<cards:CarouselView>
                <cards:CarouselView.ItemsSource>
                    <x:Array Type="{x:Type View}">
                        <ContentView BackgroundColor="Orange">
                        <ContentView BackgroundColor="Red" />
                        <ContentView BackgroundColor="Green" />
                    </x:Array>
                </cards:CarouselView.ItemsSource>
            </cards:CarouselView>

Does it work? (without listView)
And did you add CardsViewRenderer.Preserve() in AppDelegate and MainActivity?

<cards:CarouselView>
                <cards:CarouselView.ItemsSource>
                    <x:Array Type="{x:Type View}">
                        <ContentView BackgroundColor="Orange">
                        <ContentView BackgroundColor="Red" />
                        <ContentView BackgroundColor="Green" />
                    </x:Array>
                </cards:CarouselView.ItemsSource>
            </cards:CarouselView>

Does it work? (without listView)
And did you add CardsViewRenderer.Preserve() in AppDelegate and MainActivity?

Yes that is working.

<cards:CarouselView>
                <cards:CarouselView.ItemsSource>
                    <x:Array Type="{x:Type View}">
                        <ContentView BackgroundColor="Orange">
                        <ContentView BackgroundColor="Red" />
                        <ContentView BackgroundColor="Green" />
                    </x:Array>
                </cards:CarouselView.ItemsSource>
            </cards:CarouselView>

Does it work? (without listView)
And did you add CardsViewRenderer.Preserve() in AppDelegate and MainActivity?

Yes that is working.

then there is an issue with your ListView (maybe bindings are wrong)

When I assign a x:Name for ListView I am getting error.

Hello again I think that is bug. For example below code is working
cards:CarouselView>
                <cards:CarouselView.ItemsSource>
                    <x:Array Type="{x:Type View}">
                        <ListView x:Name="Test" ItemsSource="{Binding Items}" />
                        <StackLayout BackgroundColor="Red" />
                    </x:Array>
                </cards:CarouselView.ItemsSource>
            </cards:CarouselView>

But when I add ItemTemplate for ListView code can not build and work as below
   <cards:CarouselView>
                <cards:CarouselView.ItemsSource>
                    <x:Array Type="{x:Type View}">
                        <ListView x:Name="Test" ItemsSource="{Binding Items}">
                            <ListView.ItemTemplate>
                                <DataTemplate>
                                    <ViewCell>
                                        <Label Text="{Binding Title}" />
                                    </ViewCell>
                                </DataTemplate>
                            </ListView.ItemTemplate>
                        </ListView>
                        <StackLayout BackgroundColor="Red" />
                    </x:Array>
                </cards:CarouselView.ItemsSource>
            </cards:CarouselView>

Console error output
Severity	Code	Description	Project	File	Line	Suppression State
Error		Value cannot be null.

@kerbooo ah yes... It's known Xamarin xaml bug :(
I think you will have to use code

Try CollectionView. As I remember it should work

No way :(
My project can not compile with CollectionView too. Do you have any suggestion for this approach?

No way :(
My project can not compile with CollectionView too. Do you have any suggestion for this approach?

yes, just try to add your views from C# code :)

I am using MVVM prism. It can be possible?

I am using MVVM prism. It can be possible?

yes of course, why not?

So last question. Do you know any Tabbed View Plugin that has SelectedIndex Binding and animation properties like CarouselView controller?

Yes, it's already embedded in CardsView. PanCardView.Controls.TabsControl


Magic numbers

There currently are quite a few magic numbers in the codebase. I’m referring to this type:

Unique values with unexplained meaning or multiple occurrences which could (preferably) be replaced with named constants

Can you be a bit more specific?

Things like
  static const float a_hp[2] = {-1.99599, 0.99600};
  static const float b_hp[2] = {-2, 1};

in "denoise.c". This is also repeated.

These are the coefficients (numerator and denominator) of the high-pass filter used as input.


Authenticated Socket Mode

For some features it would be neat to have not only request/response one time use socket connections, but a continuous socket connection where multiple requests can be sent.
This could be combined with a one time "auth token" that can be used to establish this connection.
This way, the flatpak could pass a one time token to the daemon and not require authentications all the time for each action.

This could also be used to fix all pinentry issues on the flatpak builds by providing a nicer, GTK based pinentry.

https://github.com/quexten/goldwarden/issues/67
https://github.com/quexten/goldwarden/issues/52


powerfulseal fails while running pod scenario

Hi!
I have standalone kraken installed on my VM and I'm trying to run pod scenario to kill pods in Openshift 3.11.
I faced the following issue:
2021-10-14 20:30:04 ERROR k8s_client (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Audit-Id': '70745aa7-6743-4983-a3ee-7e4bd3b9f743', 'Cache-Control': 'no-store', 'Content-Type': 'application/json', 'Date': 'Thu, 14 Oct 2021 17:30:05 GMT', 'Content-Length': '174'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"the server could not find the requested resource","reason":"NotFound","details":{},"code":404}

Traceback (most recent call last):
  File "/home/centos/.local/lib/python3.6/site-packages/powerfulseal/k8s/k8s_client.py", line 231, in get_scenarios
    crds = self.client_extensionsApi.list_custom_resource_definition().to_dict()['items']
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api/apiextensions_v1_api.py", line 613, in list_custom_resource_definition
    return self.list_custom_resource_definition_with_http_info(**kwargs)  # noqa: E501
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api/apiextensions_v1_api.py", line 730, in list_custom_resource_definition_with_http_info
    collection_formats=collection_formats)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 353, in call_api
    _preload_content, _request_timeout, _host)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 184, in __call_api
    _request_timeout=_request_timeout)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 377, in request
    headers=headers)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/rest.py", line 243, in GET
    query_params=query_params)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/rest.py", line 233, in request
    raise ApiException(http_resp=r)
kubernetes.client.exceptions.ApiException: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Audit-Id': '70745aa7-6743-4983-a3ee-7e4bd3b9f743', 'Cache-Control': 'no-store', 'Content-Type': 'application/json', 'Date': 'Thu, 14 Oct 2021 17:30:05 GMT', 'Content-Length': '174'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"the server could not find the requested resource","reason":"NotFound","details":{},"code":404}

Traceback (most recent call last):
  File "/home/centos/.local/bin/powerfulseal", line 8, in <module>
    sys.exit(start())
  File "/home/centos/.local/lib/python3.6/site-packages/powerfulseal/cli/__main__.py", line 656, in start
    main(sys.argv[1:])
  File "/home/centos/.local/lib/python3.6/site-packages/powerfulseal/cli/__main__.py", line 630, in main
    metric_collector=metric_collector
  File "/home/centos/.local/lib/python3.6/site-packages/powerfulseal/policy/policy_runner.py", line 86, in run
    policy = self.read_policy()
  File "/home/centos/.local/lib/python3.6/site-packages/powerfulseal/policy/policy_runner.py", line 52, in read_policy
    scenarios = self.k8s_client.get_scenarios()
  File "/home/centos/.local/lib/python3.6/site-packages/powerfulseal/k8s/k8s_client.py", line 231, in get_scenarios
    crds = self.client_extensionsApi.list_custom_resource_definition().to_dict()['items']
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api/apiextensions_v1_api.py", line 613, in list_custom_resource_definition
    return self.list_custom_resource_definition_with_http_info(**kwargs)  # noqa: E501
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api/apiextensions_v1_api.py", line 730, in list_custom_resource_definition_with_http_info
    collection_formats=collection_formats)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 353, in call_api
    _preload_content, _request_timeout, _host)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 184, in __call_api
    _request_timeout=_request_timeout)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/api_client.py", line 377, in request
    headers=headers)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/rest.py", line 243, in GET
    query_params=query_params)
  File "/home/centos/.local/lib/python3.6/site-packages/kubernetes/client/rest.py", line 233, in request
    raise ApiException(http_resp=r)
kubernetes.client.exceptions.ApiException: (404)
Reason: Not Found
HTTP response headers: HTTPHeaderDict({'Audit-Id': '70745aa7-6743-4983-a3ee-7e4bd3b9f743', 'Cache-Control': 'no-store', 'Content-Type': 'application/json', 'Date': 'Thu, 14 Oct 2021 17:30:05 GMT', 'Content-Length': '174'})
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"the server could not find the requested resource","reason":"NotFound","details":{},"code":404}

2021-10-14 20:30:05,087 [ERROR] Failed to run powerfulseal autonomous --use-pod-delete-instead-of-ssh-kill --policy-file scenarios/customapp_pod.yaml --kubeconfig /home/centos/.kube/config --no-cloud --inventory-kubernetes --headless, error: Command 'powerfulseal autonomous --use-pod-delete-instead-of-ssh-kill --policy-file scenarios/customapp_pod.yaml --kubeconfig /home/centos/.kube/config --no-cloud --inventory-kubernetes --headless' returned non-zero exit status 1.

Scenario I run:
config:
  runStrategy:
    runs: 1
    maxSecondsBetweenRuns: 30
    minSecondsBetweenRuns: 1
scenarios:
  - name: "delete glusterfs pods"
    steps:
    - podAction:
        matches:
          - labels:
              namespace: "glusterfs"
              selector: ""
        filters:
          - property:
              name: "state"
              value: "Running"
          - randomSample:
              size: 1
        actions:
          - kill:
              probability: 1
              force: true
    - podAction:
        matches:
          - labels:
              namespace: "glusterfs"
              selector: ""
        retries:
          retriesTimeout:
            timeout: 180

        actions:
          - checkPodCount:
              count: 5

Cluster works, and responses:
NAME                                          READY   STATUS    RESTARTS   AGE
glusterblock-storage-provisioner-dc-1-q59xs   1/1     Running   0          6d
glusterfs-storage-jgv6n                       1/1     Running   0          6d
glusterfs-storage-mlgmr                       1/1     Running   0          15d
glusterfs-storage-zkf5n                       1/1     Running   0          6d
heketi-storage-2-xckf4                        1/1     Running   0          52d

Should I create an issue in powerfulseal github, or it might be an issue in internal kraken logic?
Thank you!

Hi @chaitanyaenr ,
I'm facing this error while running the pod scenario. Can you please confirm if this solution is still relevant to the new version of the Kraken tool as well?
HTTP response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"the server could not find the requested resource","reason":"NotFound","details":{},"code":404}


Security Risk: When initializing the database connection, the connection string is logged which will display database user name and password in the log file

There is a big security risk when initializing. The constructor allows for the database connect string to connect to the database. When the connection to SQL is successful it logs the connection string which can contain the database user name and password. If the password exists in the connection string is should not be added to the logging or at least don't display that actual password in the log.

Thanks for reporting this. I looked into the issue, and I'm pretty certain that I never expose the actual connection string. There's no logging integrated in this library, I can only assume that you're concerned about https://github.com/wcabus/DapperContext/blob/master/DapperContext.NET35/DbConnectionFactory.cs#L31 where the ConfigurationErrorsException is thrown. I only expose the name of the connection string, not the connection string itself.


Introduction to Containers and Docker

Article checklist
ATTENTION: In your PR - add the ISSUE (topic form) # that this PR corresponds to. You can do this by also adding a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
If you have not already please go over our Resources Page for more tips and tricks on how to improve our overall technical writing.
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Please place your article through a 3rd party plagiarism checker? We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution guidelines
For first-time contributors and for more details, see Contributing Guidelines
This fixes #2154

@Evanslodoctor
Good afternoon and thank you for submitting your article for publishing.
Your article has been entered into our queue and should be reviewed (and published pending any changes) as soon as a peer reviewer is finished with the ones in the queue before it. 🚀
Thank you.

Hi @Evanslodoctor I forgot to welcome you to our Engineering program. I have had a good read. You're very knowledgeable on the subject. Don't worry about making mistakes as this is your first article. You'll learn to better your writing skills with time. Have a look at the minor edits I've done on your article to avoid making similar mistakes in the future. In case of anything you don't understand, feel free to ping me on the Slack channel.
Please work on the requested revisions then we can proceed for final review.

@Evanslodoctor
👋 Good afternoon - and welcome to the EngEd program.
I'll be running a final review on your PR when ready.
Upon running your article through our 3rd party plagiarism checker it seemed to raise a few flags and the % was higher than we typically accept.
Please see attached PDF - and revisit the article to ensure we are contributing wholly unique and original content.
Be sure to see our resources page to see more info on plagiarism and what is considered as such.
Introduction to Containers and Docker #2852.pdf
cc @Bayler

hello @Bayler
done correcting requested changes. please check
thank you

hello @Bayler
done correcting requested changes. please check
thank you

@Evanslodoctor did you also make changes concerning plagiarism?

Hello @Bayler
plagiarism corrected.

hello @Bayler wished to inquire the progress my article. Thank you

hello @Bayler wished to inquire the progress my article. Thank you

I'm done reviewing it @Evanslodoctor. However, an article cannot be added the "Ready for final review" label until Plagiarism check has been performed on your article. Once that is done, it'll be ready for final review.
cc: @hectorkambow

plagiarism check completed - all good
2120420-22996517a603422142fc-1626802383624.pdf

@Evanslodoctor Please adjust the size of your hero image. It's too large. The acceptable dimension for a hero image should be 600px by 400px. It's very clear on the article checklist section. Please avoid making such mistakes in your future article submissions.

hello @Bayler I have added the requested hero image with 600 by 400px.
Please check. Thank you

Hi, @Evanslodoctor. From previewing your article, your hero image has a black border on the left and the image (particularly the text) looks a bit squashed. Could you please adjust this?

Thank you for submitting your article.
After some careful consideration, and after reviewing your article for publication, it struck us that this topic is already well covered in our EngEd library and it would be repeating some previous articles.
Due to this we will be closing this PR to avoid topic overlap. We encourage you to submit a more in depth topic or one that builds upon what we already have. Thank you.
cc - @Bayler.

@louisefindlay23 and @hectorkambow that's not fair.
Closing a PR after topic approval, passing all reviews then entering final review then all over a sudden its all in vain. All time committed while writing just for it to be closed after all that work is done is pains.  Thank you

@Evanslodoctor
Our sincerest apologies and any inconvenience - and we truly encourage you to submit a more in depth version on this topic for you to publish.
We do our best to avoid overlap in topics (and catch in in the beginning of the topic suggestion phase) but with so many articles being contributed we can only catch so much - that is why we try to make it very clear to student to check that their topic does not overlap with existing articles/topics as well.
We to need to ensure the content we are producing and curating is adding value to the larger community of developers.
It is also not fair to have contributors repeat information and get credit for it when it adds no extra value to the community.
Hope that helps shed some light on the situation - of course every situation is different and we try to handle them case by case.


user-settings: avoid using undeclared translation keys

Hey, I just made a Pull Request!
On top of #19968, this one just makes sure we're not assuming that certain translation keys exist based on theme IDs. I think it's technically safe since it will only be used when theme IDs aren't set, but either way I'd like to avoid this precedent  for now and see if we can come up with something more strict. Also slight restructuring of the messages + fixed the dev setup.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Tbh i'm piggybacking all of these on the original i18n PR changesets


Transform snake_case to camelCase

Is this feature already implemented in Flutter https://github.com/supabase/supabase/discussions/7136 ?

Hi @avalanche-tm,
I don't know if transforming snake case to camel case should be a part of Supabase library. In Flutter, you can easily transform the properties from snake case to camel case like this, but would this not be enough? Let me know if you have any specific use case that you would want this feature!

Talking about Flutter, it is even easier than that:
You need:

json_serializable
build_runner
json_annotation

import 'package:json_annotation/json_annotation.dart';

part 'my_data_type.g.dart';

@JsonSerializable(
  includeIfNull: false, // Optional
  fieldRename: FieldRename.snake,
)
class MyDataType {
  MyDataType({
    String? id,
    int? numberOfSomething,
  });

  /// Connect the generated fromJson function to the `fromJson` factory.
  factory MyDataType.fromJson(Map<String, dynamic> json) =>
      _$MyDataTypeFromJson(json);

  /// Connect the generated toJson function to the `toJson` method.
  @override
  Map<String, dynamic> toJson() => _$MyDataTypeToJson(this);
}

run flutter pub run build_runner build, this generates the types automatically.

@christianfl that looks very promising, will try out. Thanks!


Dynamic sysvars

Uses dynamic rent sysvars.
I've left the requirement to pass in the rent account to avoid breaking existing clients. However, due to the fact that clients dedup pubkeys when constructing the transaction, clients can now pass in dummy accounts instead of the rent sysvar to shrink the transaction size by one pubkey.

Closing in favor of https://github.com/project-serum/serum-dex/pull/155


Build for 2.13

Would like a new build published for Scala 2.13 if possible.

Duplicate of #230

1.3.9 has been released with 2.13 support.


Update baselines

Updating credscan baselines file to correspond with recent updates to messaging library recordings.

/azp run net - servicebus

/azp run net - eventhub

@benbp:  Not sure if you were trying to validate the management libraries against the baseline, but they're in a different set of pipelines:

net - eventhub - mgmt
net - servicebus - mgmt

Want me to kick those?

/azp run net - eventhub - mgmt

/azp run net - servicebus - mgmt


Add github action to auto-reply when a new pr is opened

Add github action to send auto-message to the contributors whenever they send a new pr
message for example - "We got your code , @username please wait for our team to review it."
or any other message as per the mentor's wish

I would like to work on this issue

We already have that @AB-san


Support for pandoc markdown's raw_tex extension

By default, pandoc markdown has raw_tex activated. It allows to do things like:
Some test with inline math: $f(x)=x^2$. And now some raw tex:

\begin{align}
   x &= 2 \\
f(x) &= 4
\end{align}

The align environment in the above document is currently dropped when using pandoc-katex (but not when using eg mathjax). I think this is because in the pandoc AST it has type RawInline rather than Math. Hence pandoc-katex passes it along, and it gets dropped by pandoc during output (doc says Inline LaTeX is ignored in output formats other than Markdown, LaTeX, Emacs Org mode, and ConTeXt.)
I think transforming the RawInline/tex nodes into RawInline/html as is currently done for Math nodes would work. What do you think?
Note, to be complete there is RawBlock/tex as well, eg with (given in the doc):
\begin{tabular}{|l|l|}\hline
Age & Frequency \\ \hline
18--25  & 15 \\
26--35  & 33 \\
36--45  & 22 \\ \hline
\end{tabular}

Sorry, for the late response.
However, there is an issue that the RawBlock or RawInline can contain arbitrary tex code, which may not be supported by katex. For example, \cite{} command in the official document or the tabular environment in your comment. Passing them to katex directly would yield hard error.
So unless there is a solution to differentiate math tex code with normal tex code, this feature is unlikely to be implemented.

That's a good point. Isn't it also the case with Math though? For example, this document:
$$
\begin{tabular}{|l|l|}\hline
Age & Frequency \\ \hline
18--25  & 15 \\
26--35  & 33 \\
36--45  & 22 \\ \hline
\end{tabular}
$$

It produces a Math node, and as you pointed out it trips katex because katex doesn't know the tabular environment. As I understand, RawBlock/tex and RawInline/tex would work the same as Math.

No, what you show in the example is invalid tex code even in a normal LaTeX document. So a hard error is what we should expect in this case.
RawBlock on the other hand can contain arbitrary non math tex code, and it is perfectly valid.
I will close this issue for now. Because I don't believe pandoc's raw tex extension is intended to be used to render html file. It is likely to design to generate proper LaTeX pdf documents using markdown.

Got it. You're right about RawBlock potentially containing arbitrary (non-math) tex code. Then I wondered: how come pandoc --mathjax is able to pick up the align environment from the first comment?

So unless there is a solution to differentiate math tex code with normal tex code, this feature is unlikely to be implemented.

Turns out the pandoc HTML writer has a list of hard-coded environments to pass through: https://github.com/jgm/pandoc/blob/604d777c03ad1bd8a7f9279a255b2a693fe8c18d/src/Text/Pandoc/Writers/HTML.hs#L1687-L1725
However it seems to only work with pandoc's built-in math modes, not filters doing their own math rendering. I wonder if we could request that feature in pandoc.


[DNM][RFC] SDL based display emulation driver

This driver introduces an emulated LCD display for the native POSIX board.
The emulated display driver makes use of SDL2 to render the displays frame
buffer into a dedicated desktop window.
Note that only the last 2 commits are applicable for this pull request, the remaining 5 commits are part of open pull request where this one is depending on.
Bellow is a screenshot of the LVGL sample build and running for native_posix board:

Relate dto #7833 & #6826

@SebastianBoe Note that only the two last commits are the actual pull request, the remaining commits are part of #6826 and #7833. I will take your comments along in #6826.

@aescolar Could review the last commit (native_posix: Added polling for SDL events)?

Changes:

Removed dependency on lvgl (#6826) and use character framebuffer subsystem instead (including sample).
@nashif can you remove DNM label as this PR has no longer any dependencies on other PR?
Added extra posix signal to sdl event loop to prevent crash in case events pop up during driver initialization
Small corrections in cfb and SSD1306 & SDD1673 drivers (#9035)
@jfischer-phytec-iot  please review

Codecov Report

Merging #9149 into master will decrease coverage by 0.01%.
The diff coverage is 28.57%.

@@            Coverage Diff             @@
##           master    #9149      +/-   ##
==========================================
- Coverage   53.26%   53.25%   -0.02%
==========================================
  Files         215      215
  Lines       25827    25840      +13
  Branches     5695     5697       +2
==========================================
+ Hits        13758    13761       +3
- Misses       9753     9763      +10
  Partials     2316     2316

Impacted Files
Coverage Δ

boards/posix/native_posix/hw_models_top.c
78.12% <28.57%> (-14.04%)
:arrow_down:

Continue to review full report at Codecov.

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

It seems the underlying GitHub entry holding this PR got quite corrupted. I see my last review either not at all, or several times when I refresh this page. Maybe you want to close the PR and create a new one with this same branch?

@aescolar Github is experiencing some difficulties (https://status.github.com/messages) at the moment let see how it turns out else I will create a new PR

recheck

As this PR is adding a new "peripheral" for native_posix, it would be nice adding a new section in https://docs.zephyrproject.org/latest/boards/posix/native_posix/doc/board.html#peripherals for it. You could also mention there that it will require pkg-config and libsdl2-dev:i386.
At least in my case, I also needed to export PKG_CONFIG_PATH=/usr/lib/i386-linux-gnu/pkgconfig/ before calling cmake to get it to build the demo app.

@vanwinkeljan I have just tried the app in a VM, and it runs very slowly. Are you experiencing that in your side?

@aescolar In my case the scrolling line wraps around in 1 to 2 seconds.
In case it is much slower at your side could this be caused by the fact that the main loop is a tight loop (k_cpu_idle has been added)?

In case it is much slower at your side

It is around 1 second here.. I guess "slow" can have different meanings :)

@SebastianBoe : Is your review still valid?

@vanwinkeljan : I have just run that sample app with valgrind's callgrind tool. In 1 scroll, there seems to be 450.000 calls to SDL_UpdateTexture() which lead to a baffling amount of activity inside the SDL library (over 2.5 thousand million == 2.5 short billion instructions in the SDL library are executed for each wrap).

@aescolar  I'm not surprised about the number 450.000. This s due that SDL rendering buffer is ARGB and the CFB sample uses monochrome displays, the function sdl_display_write_mono naively takes care of the conversion pixel per pixel. Further CFB always writes the complete screen and the screen size is 250x120 so it seems per wrap there are 15 full screen writes.
On the other hand I'm a bit surprised by the number of calls in the SDL library I was assuming that the reddening was delayed until a single call in the function sdl_display_write.

@aescolar Quickly googled it and as far as I understand it will for each call to SDL_UpdateTexture transfer the data to the GPU and for the monochrome display this is per pixel.  I could try to optimize it a bit but no idea how much it will improve, first time playing around with SDL.
Fyi from the remarks section of SDL_UpdateTexture (https://wiki.libsdl.org/SDL_UpdateTexture ;)

This is a fairly slow function, intended for use with static textures that do not change often.
If the texture is intended to be updated often, it is preferred to create the texture as streaming and use the locking functions referenced below. While this function will work with streaming textures, for optimization reasons you may not get the pixels back if you lock the texture afterward.

I could try to optimize it a bit but no idea how much it will improve

I would guess that if you built the whole display in one rectangle, and called UpdateTexture once per sdl_display_write, it would speed up things 30.000 times.

I would guess that if you built the whole display in one rectangle, and called UpdateTexture once per sdl_display_write, it would speed up things 30.000 times.

That was actually my initial plan but that would have require an extra buffer and I had already the texture available. I will try the streaming texture + locking as mentioned above and see how it turns out, if it does not improve I will create local buffer.

that would have require an extra buffer

That extra buffer is trivially cheap in native_posix. You could have it preallocated since you initialized the display driver.

Added display peripheral to documentation
Performance improvement: Use SDL_LockTexture/SDL_UnlockTexture instead of SDL_UpdateTexture

@vanwinkeljan : Uhm..., trying this new version I'm meeting with two errors occasionally. Sometimes everything runs fine, sometimes I get a:
XIO:  fatal IO error 14 (Bad address) on X server ":0.0"
      after 288 requests (286 known processed) with 0 events remaining.

Sometimes (more often) I get a segfault in the SDL library.
Both always after the cfb_framebuffer_finalize() finalize has completed and before the next iteration of the loop starts. So they seem related.

@vanwinkeljan By the way, regarding the performance issue, it is obviously better. But you can also see the numbers here (both for 1 full scroll). Old commit 6afa8a53a469661f:

New commit (b389754472e4378f699):

The 3 first columns are:

Total instructions executed in that function (including children)
Total instructions executed in the function itself (no children)
Number of times the function was called

The total zephyr.exe instruction count in the new commit was 717M

@aescolar Thx for all the debugging possibly this is caused by the fact that SDL assumes everything is rendered from a single thread.
As mentioned in a earlier comment:

Thx for all the input I will go for the timer thingy especially because of the warning in the SDL_PumpEvents API documentation:
WARNING: This should only be run in the thread that initialized the video subsystem, and for extra safety, you should consider only doing those things on the main thread in any case.

So two options here:

Move everything towards the main hw loop
Put a message queue in place between the event loop thread and the main hw loop.
I already had a proposal ready to support keyboard based GPIO via SDL that contains this solution.
You could have a look at following 2 difss to get an idea:
https://github.com/vanwinkeljan/zephyr/commit/c762321b84ead7c1951c96eb3b8b52765f95096a#diff-b34741663195be2a7c40e7fa916080f0
https://github.com/vanwinkeljan/zephyr/commit/c762321b84ead7c1951c96eb3b8b52765f95096a#diff-d5441623e86f5884cfbfaa5e58f58daf

@aescolar I will have a closer look at buffer write functions. In the end I managed to reproduce the issue by putting k_cpu_idle back in place and sending the window to the background and then when I bring the window to the foreground the crash occurs. But I still have the feeling that it is just a multi threading issue, especially as the issue is gone after placing a return at the beginning of the function sdl_handle_window_event.

In the end I managed to reproduce the issue ..

Curious, here it crashes maybe one out of every 3 times on its own (no need to touch anything)

But I still have the feeling that it is just a multi threading ..

It may be. But note that the thread that calls the sdl_write function is anyhow a Zephyr thread, while the HW models are another thread.
( It can be quite annoying to track down a bug like this which is exposing itself deep in the guts of a 3rd party library. )

@aescolar Added synchronous event handling code to handle the render event in the hw loop.
This should prevent concurrent calls to SDL_RenderPresent as the zephyr threads are not executing at the same time as the hw main loop thread. Could you test this from your side to see if it resolves the problems, for me it is at least solved.

just coming back to trying this out after a while...
i am getting this:
[100%] Built target zephyr_prebuilt
Scanning dependencies of target run
dbus[1429]: arguments to dbus_message_new_method_call() were incorrect, assertion "path != NULL" failed in file ../../dbus/dbus-message.c line 1362.
This is normally a bug in some application using the D-Bus library.

Any hints?

@nashif

Any Hints?

Are you on ubuntu 18.04?
If this is the case see: https://bugs.launchpad.net/ubuntu/+source/libsdl2/+bug/1775067
Comment 7 has a posible workaround.

Are you on ubuntu 18.04?

Fedora 28

Are you on ubuntu 18.04?

Fedora 28

Does the workaround work?

Does the workaround work?

yeah, workaround 1 in comment 7 works

@aescolar any chance you found some time to test the latest fixes?
@nashif Did you encounter any instabilities during your testing?
@SebastianBoe could you revisit you review?

any chance you found some time to test the latest fixes?

@vanwinkeljan : I just took a quick look now.. I hope I have some time in the next couple of days.

@aescolar at first thx for the quick feedback

I just took a quick look now.. But I hope I'll have some time in the next couple of days.

Actually I was only hooping for a test, I just grapt this code from some "work in progress" for test purposes.

I just grapt this code from some "work in progress" for test purposes

Then if you don't mind let's add the DNM tag back.

@aescolar In case the crashes do not show up in your test I will rework the code to handle all events in the SDLEVENTSTIMER case of the hwm_main_loop

In case the crashes do not show up in your test

They did not.
But note that all I do is run it in a Xubuntu18.04 VM , so you do not need me just for testing it a bit..

I wonder if the reason why it does not crash now is because of changing the threading, or because of the changes you did in the driver itself with the texture locking and so forth.

But note that all I do is run it in a Xubuntu18.04 VM , so you do not need me just for testing it a bit..

I did test locally first but still thought it was a good idea it was tested on different machines.

I wonder if the reason why it does not crash now is because of changing the threading, or because of the changes you did in the driver itself with the texture locking and so forth.

Once I found a scenario to reproduce the issue was still there with the reworked driver  and it always originated from SDL_RenderPresent, stack trace indicated that came from inside the OpenGL driver. Making sure that the SDL_RenderPresent was called sequentially  resolved the issue.

Updated code to handle SDL events in hwm_main_loop instead of dedicated event loop tot prevent concurrent calls to SDL functions.

Moved SSD1673 & CFB commits to separate PR #11351, only 3 last commits are now part of this PR

Moved SSD1673 & CFB commits to separate PR

I'm a bit confused with this: the SSD1673  commits are still in this PR branch, did you forget to remove them?

@vanwinkeljan : shouldn't the commits:
driver: SDL based display emulation driver
native_posix: Use synchronous SDL event handling
be squashed toghether?

Moved SSD1673 & CFB commits to separate PR

I'm a bit confused with this: the SSD1673 commits are still in this PR branch, did you forget to remove them?

This is a bit of a github issue as it can not track dependencies between PRs.
What happend here is that this PR now dependence on PR #11351 so it shows also all commits part PR #11351

@vanwinkeljan : shouldn't the commits:
driver: SDL based display emulation driver
native_posix: Use synchronous SDL event handling
be squashed together? (if you want to go with the synchronous solution, that is. But if you are not sure yet then leave it in 2 commits)

Yes they should but I thought it would be more clear to reviewers to see what got changed.

This is a bit of a github issue

This PR is a request to merge the branch sdl_display into master.
 Everything after that is what this pull request is requesting to merge (and would be merged if somebody pressed the merge button).
If that set of commits has something in common with other branches, or other PRs, is not relevant from GitHub's point of view (One cannot expect GitHub to read the author's mind and guess what is it that the author is trying to do with common commits in different branches/PRs/repos).

This is a bit of a github issue

@aescolar Maybe I had better word this as it is a "highly requested feature", of course it would be up to the author to mark the dependencies.

@vanwinkeljan : Is this PR actually blocked by #11351? or simply this PR branch branched from that other one?

@aescolar Thx for all the review effort!

@SebastianBoe could you revisit you review?

@SebastianBoe could you revisit you review?

I see that the original review comment stated that this would require a bump in the CMake version. Which has not happened. Is this still valid then?

Correct there is no need to up step CMake anymore as west will be used instead and actually the up step was not needed for this PR but for PR #6826.

@nashif : I did. This PR is DNM, being blocked by #11351

@aescolar Thx was just typing my self that this PR blocked

@nashif replaced subsys with cfb in commit message

please rebase

@nashif done


Allow controling font modified date

Fixes https://github.com/googlei18n/ufo2ft/issues/161

It would be good to describe the new modifiedTime argument in the functions' docstrings, a perhaps also mention the environment variable in the README. I can also do it later.

Thanks!


Calculate session replay percentage based on stable identifier

Hi all! Right now we're using record_sessions_percent to roll out session replay to a small percentage of our users. However, due to the way that Mixpanel uses Math.random to calculate a page view-level eligibility for session replay, it's hard to compare replays in a funnel or between different sessions by the same user.
Instead, Mixpanel should hash the user's distinct_id to calculate a stable identifier that can be used for determining whether the user is in the "record sessions" cohort, so that replays from the same user can be analyzed together.

This is largely a product decision (whether you want user-based sampling or not), but we are indeed planning to offer some more complete controls over sampling in future releases. In the meantime, if you want user-based sampling, the recommended workaround is to write your own sampling logic and then initialize with record_sessions_percent set to either 0 or 100 based on your determination of whether the current user falls in sample or not.


▶ ERROR Bad password: Invalid password. Server rejected login attempt..

I get this error when I try to create a new GPG key on keybase.io
My Keybase log ID: 701835f5240464f188c2a81c
osioke@Osioke-XPS:~$ keybase pgp gen --multi
Enter your real name, which will be publicly visible in your new key: Osioke Itseuwa
Enter a public email address for your key<EMAIL_ADDRESS>Enter another email address (or <enter> when done):
Push an encrypted copy of your new secret key to the Keybase.io server? [Y/n] Y
When exporting to the GnuPG keychain, encrypt private keys with a passphrase? [Y/n] Y
▶ INFO PGP User ID: Osioke Itseuwa<EMAIL_ADDRESS>[primary]
▶ INFO Generating primary key (4096 bits)
▶ INFO Generating encryption subkey (4096 bits)
Please enter the Keybase password for osioke (8+ characters):
▶ ERROR Bad password: Invalid password. Server rejected login attempt..

Nevermind, my mistake. I was to enter my Keybase password but kept entering what I wanted to use as the passphrase


Annotation: @alwaysThrows

It's often nice to have a _fail() method or similar.
Often if it's used in a method with a return type, the analyzer will warn about a missing return type.
Solution:
@alwaysThrows (or similar) on a method definition

Analyzer confirms that all code paths through the method throw (as opposed to no-op or return)
Effectively replaces an inline throw in code, eliminating the "does not end in a return statement" hint

This is filed under analyzer, but the implementation would also involve package:meta, right?

Yes, but the authors are the same, so...

Fair enough. Just understanding the implementation. I guess there's no "area-meta" label.

Committed.


[Feature] throttle account delete request [OSF-6000]

Note: This PR is branched off of #5487, which will need to get merged into develop first
Purpose
Users can request account deactivation as frequently as they wish, and  each time that button is clicked an email gets sent to the osf help account.
Changes
Only let users request account deactivation once every 30 seconds.
Side effects
None.
Ticket
https://openscience.atlassian.net/browse/OSF-6000

Cherry-picked 7a34932 into #5487


error after bazel build a python program

Issue:
build apollo, then build a customized python program, find that the build results of apollo in /apollo/bazel-bin or /apollo/bazel-bin/cyber/tools partly disappear
System information

Linux Ubuntu 20.04:
Apollo master branch, January 12, 2022:
Output of ./apollo.sh config:

[cyber@in-dev-docker:/apollo]$ ./apollo.sh config
[INFO] Apollo Environment Settings:
[INFO]     APOLLO_ROOT_DIR: /apollo
[INFO]     APOLLO_CACHE_DIR: /apollo/.cache
[INFO]     APOLLO_IN_DOCKER: true
[INFO]     APOLLO_VERSION: master-2021-12-28-463fb82f9e
[INFO]     DOCKER_IMG: dev-x86_64-18.04-20210914_1336
[INFO]     APOLLO_ENV:  STAGE=dev USE_ESD_CAN=false
[INFO]     USE_GPU: USE_GPU_HOST=1 USE_GPU_TARGET=1

Steps to reproduce the issue:

prepare a python program in apollo project, refer to Cyber RT Python API : An Example`:

$ tree apollo/atest
apollo/atest
├── BUILD
└── my_demo.py

0 directories, 2 files

build apollo and program`:

./docker/scripts/dev_start.sh
bash docker/scripts/dev_into.sh
./apollo.sh clean --all
./apollo.sh clean --bazel
./apollo.sh build_opt_gpu
# check build
ls /apollo/bazel-bin
ls /apollo/bazel-bin/cyber/tools
# build python program
bazel build //atest:my_demo
# check build again, I meet the error, presented in the following screenshots

Supporting materials (screenshots, command lines, code/script snippets):
build results of apollo in /apollo/bazel-bin partly disappear, with subsequent command not found error

solved
according to scripts/apollo_build.sh, use Bazel Options: --config=opt --config=gpu to bazel build the program if running ./apollo.sh build_opt_gpu before
bazel build --config=opt --config=gpu //atest:my_demo


Case sensitivity in code systems

New validator issues:

codesystem\v3-TimingEvent.xml:
error: CodeSystems SHOULD NOT have a stated value for the caseSensitive element when they are a supplement
codesystem/ihe-pharmaceuticaladvicestatuslist.xml:
information: CodeSystems SHOULD have a stated value for the caseSensitive element so that users know the status and meaning of the code system clearly
codesystem/edqm-standardterms.xml:
information: CodeSystems SHOULD have a stated value for the caseSensitive element so that users know the status and meaning of the code system clearly

For the 2 informations, we would have to add the element <caseSensitive value="true"/>.
For the error, see the discussion in zulip.

need to clarify if the two codesystems are case-sensitive or not

All codes should be case-sensitive within CDA-CH-EMED and I will treat them as such. There are issues with some code systems (e.g. UCUM) and the Schematron does a case-sensitive comparison with value sets codes.

TBD: Für beide Code Systeme recherchieren und definieren.

@oliveregger : Anpassung bei der Generierung noch notwendig


select() cannot handle fd >= 1024

This is an issue reported by a customer.
https://github.com/Azure/azure-sdk-for-cpp/blob/21214f9e387375ad32791dca4464273de6eeda9e/sdk/core/azure-core/src/http/curl/curl.cpp#L41
fd_set is an array of bits and can only handle fd < 1024. Some customers have more than a thousand fds in the process, thus some fd values are larger than 1024. these fd cannot be put in fd_set, so select() just won't work.

fixed in https://github.com/Azure/azure-sdk-for-cpp/commit/fdcb150621f05bcce7fb705d000d1f21b20073ef


Make rules using $< without prerequisites

A number of rules in the top-level Makefile lack prerequisites yet use $< in the recipe:
%.fst:
          $(call run-with-log,\
            $(MONO) $(VALE_HOME)/bin/vale.exe -fstarText \
              -include $*.types.vaf \
              $(VALE_FLAGS) \
              -in $< -out $@ -outi $@i && touch -c $@i \
            ,[VALE] $(notdir $*),$(call to-obj-dir,$@))

obj/%.ml:
          $(call run-with-log,\
            $(FSTAR) $(notdir $(subst .checked,,$<)) --codegen $(CODEGEN) \
              --extract_module $(subst .fst.checked,,$(notdir $<)) \
            ,[EXTRACT-ML] $(notdir $*),$(call to-obj-dir,$@))

obj/%.krml:
          $(call run-with-log,\
            $(FSTAR) --codegen Kremlin \
              --extract_module $(basename $(notdir $(subst .checked,,$<))) \
              $(notdir $(subst .checked,,$<)) && \
            touch $@ \
            ,[EXTRACT-KRML] $*,$@)

What is the intended value for $< in these cases? The second of these looks like it should include prerequisite of obj/%.fst.checked, though that doesn't work for a target like Lib_Memzero0.ml. Without the prerequisite, I get an error for missing argument to --extract_module because $< is empty.

Target dependencies are generated in the .depend then the pattern rules kick in to provide the recipe.
For the particular case of Lib_Memzero0.ml we recently cleaned up a hack, so now you may need to regenerate your .depend, and remove a few .ml in obj --  I suggest attempting to start from a clean build.

A clean build from the latest master resolved this issue for me.


Cant use a "Comma" in the Settings of a device

If i try to put in a comma, when i set a setting of e.g. a volumepump, i cant set it to maybe "0,25". can copy it in, but if im hitting the "," key on my keyboard, it wont work.

Wow, really :)
I'll fix it

Fixed :+1:


Slack Integration

Can we add support slack tool integration

You may

@adamcooke is there any plans to create a plugin framework so we can add features like this with out break future roadmap?

I will be adding a simplified Slack notification piece in the coming days/weeks.  I started down a path that was going to be far more complicated than really necessary I think.  So i'm just going to punt and clone/tweak the Email model to suit.  I figure that with Attachments, it should be pretty easy to get a decent looking slack message without a lot of work.
What i'm shooting for will be as simple one-or-nothing approach; if you choose to notify subscribers, it'll also send to the configured slack webhook (which, again, punting here, will be in the environment.yml), using the channel, username, icon, etc. also defined via ENV parameters.
Perhaps down the road we can add actual settings that can be configured from the WebUI, or associate specific channels to specific services somehow; but for now, I personally just need it working so I'm going to carve out the time to make it work in the most simplistic way that accomplishes what we need.  There's always time to make it fancy later.

Hello, any update here? Would love a Slack integration.

Any update?

we ended up deploying Hund. Staytus just didn’t offer enough and was going to require enough dev and upkeep that the $29/mo for Hund was a net win. so i won’t be adding the integration i proposed, at least not anytime in the foreseeable future.
Sent from my iPhone

On Aug 11, 2019, at 15:27, JonathanPaluch<EMAIL_ADDRESS>wrote:
Any update?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Automated cherry pick of #3457: build: show diffs made by checks

Cherry pick of #3457 on release/2.12.
#3457: build: show diffs made by checks

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@515e293). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.12   #3458   +/-   ##
==============================================
  Coverage                ?   7.03%
==============================================
  Files                   ?     496
  Lines                   ?   82516
  Branches                ?       0
==============================================
  Hits                    ?    5808
  Misses                  ?   76221
  Partials                ?     487

Flag
Coverage Δ

#aFlag
7.03% <ø> (?)

Continue to review full report at Codecov.

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

/lgtm

/approve


Return txid to user

Display txid to user upon success to help with support issues.

I can only provide you the tx id if the tx is done via 'send button'


Option to allow 'async fns' to return 'impl Future' instead of 'Box'

Hi,
First of all, thanks a lot for your great code that has helped us (some developers at work, SAP) a lot. The background of this PR is, we extensively use GAT, generics, and asynchronous functions all over the code base for our project. However, it has been a pain in the a** to manually specify Future types in a trait to emulate async fn, and one of the developers came up with an idea of writing our own macros that would convert #[x]async fn in a trait into type X = impl Future; fn -> X. The first attempt was half-successful that it only covered few cases. So, we decided to shamelessly use your code as a basis to achieve what we want!
=> The goal is, write a set of macros that automatically generate GATs in a trait for some performance-critical functions where heap-allocation is not an option.
=> The new macro #[static_future] makes it possible to do so; instead of boxing a future, an impl Future<Outut = Ret> is returned by implicitly adding GATs for subjected async methods.
#[async_trait]
pub trait A {
    #[static_future]
    async fn(&self) -> usize;
}

#[async_trait]
impl A for () {
    #[static_future]
    async fn(&self) -> usize {
        11
    }
}

The generated code for async fn(&self) -> usize returns an impl Future<Output = usize> + 'async_trait.
The current state of the change has some limitations, such as, it requires the user to explicitly activate certain unstable features including GAT, #[static_future] doesn't work with a method having a default implementation. However, it turns out that the coverage is quite good that it can already be applied to some performance-critical traits in our code base.
That said, it'd be really appreciated if you could point out anything inappropriate in the interface and code, or it's even OK for you to say "it is absolutely unacceptable".
Thanks and best regards,
wvwwvwwv

@dtolnay Can you please review this PR?


feat: separate price oracle vs vol oracle

Codecov Report

Merging #96 (acc14c8) into master (4e24b54) will decrease coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
- Coverage   24.07%   23.81%   -0.26%
==========================================
  Files          21       21
  Lines         889      886       -3
  Branches      116      116
==========================================
- Hits          214      211       -3
  Misses        615      615
  Partials       60       60

Impacted Files
Coverage Δ

script/Deploy.sol
0.00% <ø> (ø)

src/core/Oracle.sol
62.50% <ø> (-4.17%)
:arrow_down:

...e/engines/advanced-margin/AdvancedMarginEngine.sol
79.38% <ø> (ø)

...core/engines/advanced-margin/AdvancedMarginLib.sol
0.00% <ø> (ø)

...ore/engines/advanced-margin/AdvancedMarginMath.sol
0.00% <ø> (ø)

src/test/mocks/MockOracle.sol
100.00% <ø> (ø)

src/test/shared/AdvancedFixture.t.sol
0.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix triton-arith-linalg not using the common patterns

The triton-arith-to-linalg-pass wasn't using the common ConversionPatterns.hpp file, leading to some inconsistencies in the number of patterns populated.

@haishanzzzz FYI, I just noticed this because we recently merged a few more patterns in ConversionPatterns.hpp but the new pass didn't pick them up.

Thank you for the fix @nhat-nguyen! It turned out that I updated our internal code but didn't do that for triton-shared. This two-copy business is madness :D


RFC: source layout change?

Proposal:
src/ almost everything
include/ quickjs.h
Reasoning: I broke embedding in CMake projects because I made the include paths private to avoid a weird CMake error (now fixed). It feels more natural to not jumop through hoops and use a "better" layout.
Bonus: the GH UI looks better since our README would actually be seen :-P

I kind of like flat layouts but no strong disagreement. If it makes things better, go for it.

Maybe if / when we add Meson or another build system things become more obvious. As it stands it Just Works (TM).

I really like the single file flat layout :)

It would remain a single file, just on src/ with the rest of the files, except the header files which would go on include/

except the header files which would go on include/

Plural or singular? quickjs.h is the only header file that's intended for public consumption; cutils.h, libbf.h, etc. should probably go into src/.

Singlular. Onlt quickjs.h would go in there. Maybe quickjs-libc.h too since we conditionally export it too...

Cool! I think @bnoordhuis showed interest in having it. Would you like to send it as a PR?

Sure! Since my work is modeled after the original Makefile, I need some time to digest the brand new CMake build script.


Autoscroll results in offset for dropzones in sticky parents

Problem Description
In a case where draggable elements are placed inside a scrollable area, the auto-scroll feature is super helpful as it scrolls when a currently dragged element comes close to the edges.
Normally this works just fine and the options allow pretty fine grained control of the auto-scroll.
However, when there is a dropzone inside that scrollable area that has position: sticky to prevent it from scrolling out of view, the auto-scroll introduces an offset with every scrolling action that is applied by the auto-scroll to the drop-zone rect before the collision detection is performed.
Optically nothing breaks initially, but a drop-zone that has a hover effect will show the hover effect when the offset bounding box is hovered instead of the visible bounding box.
Sidenotes
While trying to work around this issue I noticed that the auto-scroll feature can be turned off globally by options, but can only be turned off locally by nesting another DndContext. Unfortunately this would break the global drag'n'drop support for the application I'm working on.
It might be a nice addition to be able to exempt certain scroll containers without turning it off globally as well (e.g. via attribute or class name).
On the other hand the ability to nest multiple DndContext where children serve as a configuration device and the parent does all the heavy lifting (drag overlay, drop handling, etc.) would be a nice addition as well. But I guess that's a bigger refactoring and beyond the scope of this issue.
Reproduction
I made a small example in this sandbox: https://codesandbox.io/s/dndkit-autoscroll-with-sticky-dropzone-70m37f
Scrolling horizontally via auto-scroll will prevent reaching the sticky drop zone (dashed border shows successful hovering over the drop zone)
https://user-images.githubusercontent.com/16539774/186900278-f2620221-6004-423e-a1f8-565d050ed761.mov
p.s.: I haven't forgotten about this library and my plans with it, but time is a very limited resource for me right now - more contributions to come soon hopefully ;)

After some more investigation I found a temporary solution to this problem.
I noticed that the measurements for droppables were done before and after the drag, but not while dragging.
Using a number as the measuring frequency for droppables instead of 'optimized' forces the measurement even while dragging and corrects any offset formerly applied to the sticky droppables.
This is my workaround:
  const measuring = useMemo(() => {
    return {
      droppable: {
        strategy: MeasuringStrategy.Always,
        frequency: isDragging ? 500 : MeasuringFrequency.Optimized,
      },
    };
  }, [isDragging]);

@5ebastianMeier thank you! You saved my time :)

Thanks @5ebastianMeier , there is another issue that while scrolling you can't drop into the child of a sticky element.


style whatsapp share into a button

@RDjarbeng, you have to take a look at the changes I made in the exportToWhatsapp() You might want to suggest edits before merging this PR.
The issue with it is that only chromium browsers support the very easy to use navigator.uadata. my workaround for other browsers is not working so well (the else if block). The else part is for desktop.
what to do?

Well the icon is supposed to be hidden on larger screens. You can use media
query min-width(500px) to hide it. I think this solves the desktop issue.
I also wanted text that shows share to WhatsApp for a few seconds before
minimizing unto the whatsapp icon, I think that will help first time users
👥.
On Fri, 4 Mar 2022, 12:03 netlify[bot], @.***> wrote:

✔ Deploy Preview for rcountdown ready!
🔨 Explore the source changes: 7130a30
https://github.com/RDjarbeng/countdown/commit/7130a3004c9a50174da5517b6ed5b9b000ce8ab3
🔍 Inspect the deploy log:
https://app.netlify.com/sites/rcountdown/deploys/6221ffec94f2db0008311c39
😎 Browse the preview: https://deploy-preview-47--rcountdown.netlify.app
—
Reply to this email directly, view it on GitHub
https://github.com/RDjarbeng/countdown/pull/47#issuecomment-1059102997,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANY6G45KXUKJCXC757RR3G3U6H3X7ANCNFSM5P5LLMXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Well the icon is supposed to be hidden on larger screens. You can use media query min-width(500px) to hide it. I think this solves the desktop issue. I also wanted text that shows share to WhatsApp for a few seconds before minimizing unto the whatsapp icon, I think that will help first time users.

Icons are very intuitive, so having to explain yourself again is not needed. Anyways I'll use set the title attribute so it serves this purpose. Everyone happy :-)


Kubernetes broken

Issue description
#35330 broke building Kubernetes.
Detected go version: go version go1.10 linux/amd64.
Kubernetes requires go1.9.1 or greater.
Please install go1.9.1 or later.

all-packages.nix probably just needs a go = go_1_9;.
Edit: Please do make sure the K8s tests work.

seems like the versions are compared lexicographicaly somewhere

Another distro just removed the version check:
https://github.com/clearlinux-pkgs/kubernetes/blob/724a51af37ec57615cf5c2f77392b4919fe5888c/build.patch

Some Arch people suggested this patch
https://github.com/kubernetes/kubernetes/commit/fc37221db5d08eb9553273ad6202a2557cfde131.patch
I haven't tried it yet

@xeji It seems to be already merged upstream 12 days ago: https://github.com/kubernetes/kubernetes/pull/58207

@volth successful build with that patch. will run the tests later

failed to build the tests in nixos/tests/kubernetes on master. looks like a separate issue, the tests are broken somehow.
I'll go ahead and make a PR for the version check patch anyway.


Estimote library add permission READ_PHONE_STATE (targetSdkVersion < 4)

When building an apk with your aar library attached, the generated apk will require the permission:
android.permission.READ_PHONE_STATE
This implies that publishing on the google play store that apk, google will ask a privacy policy.
I'm sure that the problem derive from your library because of the log that I can found in:
app/build/outputs/logs/manifest-merger-release-report.txt
In particular:
ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:13:5-88 	android:name 		ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:13:29-85 service#com.estimote.sdk.repackaged.dfu_v0_6_1.no.nordicsemi.android.dfu.DfuBaseService ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:26:9-115 	android:name 		ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:26:18-112 activity#com.estimote.sdk.DefaultRequirementsCheckerCallback$Activity ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:28:9-30:56 	android:theme 		ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:30:13-53 	android:name 		ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:29:13-88 receiver#com.estimote.sdk.service.BeaconService$ScanAlarmBroadcastReceiver ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:32:9-102 	android:name 		ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:32:19-99 service#com.estimote.sdk.connection.internal.DeviceConnectionProviderService ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:34:9-36:40 	android:exported 		ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:36:13-37 	android:name 		ADDED from [com.estimote:sdk:0.16.0] /home/nagash/.android/build-cache/8d300f9c495671d3da2ac07a68e7eec629dbf839/output/AndroidManifest.xml:35:13-96 uses-permission#android.permission.READ_PHONE_STATE IMPLIED from /home/nagash/workspace/AppWebBrowser/app/src/main/AndroidManifest.xml:2:1-75:12 reason: org.webkit.android_jsc has a targetSdkVersion < 4
Last lines:
uses-permission#android.permission.READ_PHONE_STATE
IMPLIED from /home/nagash/workspace/AppWebBrowser/app/src/main/AndroidManifest.xml:2:1-75:12 reason: org.webkit.android_jsc has a targetSdkVersion < 4
Probably the issue is caused by org.webkit.android_jsc, hope there is a new version of org.webkit.android_jsc or another way to fix the issue.
I resolved adding this line in my AndroidManifest.xml:
<uses-permission android:name="android.permission.READ_PHONE_STATE" tools:node="remove" />
hoping does not cause problems at runtime (I have not tested yet with a beacon).
Thanks

Dear @nagash91
we take all such requests seriously. We took some time to investigate the source of this problem, always starting with the documentation of basic things like Android framework.
So, with the knowledge that our sdk DOES NOT use android.permission.READ_PHONE_STATE, we looked at the permission itself. According to the Android documentation:

READ_PHONE_STATE
Allows read only access to phone state, including the phone number of the device, current cellular network information, the status of any ongoing calls, and a list of any PhoneAccounts registered on the device.

Note: If both your minSdkVersion and targetSdkVersion values are set to 3 or lower, the system implicitly grants your app this permission. If you don't need this permission, be sure your targetSdkVersion is 4 or higher.

I think this is the clue of your problem. Please consider targeting minSdkVersion > 4.
By the way -  according to the Android dashboard, market adoption of API 10 (Gingerbread) is around 1.0%. This is why our sdk minSdkVersion = 9.
Have a good day!

Ok sorry, I messed up.
The problem comes from library org.webkit.android_jsc that I assumed wrongly to be used by Estimote.
The problem comes from another library I have included in my project.
Thanks!


allow to specify a proxy server for package downloads

sometimes we need to specify a proxy to download the consul package. So we can just pass the corresponding puppet-archive parameter.

Is the use-case that only this specific download needs use a proxy server, but nothing else in the catalog?
Can you read https://github.com/solarkennedy/puppet-consul/issues/269 first and see if it is relevant?

Thank you for the issue pointer. I also think it is cleaner to specify the proxy to the consul module instead of the archive class, because this would create a hidden dependency that breaks if you happen to use something different to download the consul package.
And yes, we do use the archive module for other use cases that do not require the proxy.

Resolves #269, cc @walkamongus

@solarkennedy thanks for the followup!


Instances of uninferred RAM logic

I tried to implement the neorv32 core to a cyclone II device EP2C8Q208C8. The tool I use is Quartus II 13.0 64-bit.
I followed the well documented user guide step by step to build a simple project. But I ran into errors during Analysis & Synthesis.
Info (276014): Found 8 instances of uninferred RAM logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_imem:\neorv32_int_imem_inst_true:neorv32_int_imem_inst|mem_ram_b1" is uninferred due to asynchronous read logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_dmem:\neorv32_int_dmem_inst_true:neorv32_int_dmem_inst|mem_ram_b1" is uninferred due to asynchronous read logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_imem:\neorv32_int_imem_inst_true:neorv32_int_imem_inst|mem_ram_b0" is uninferred due to asynchronous read logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_dmem:\neorv32_int_dmem_inst_true:neorv32_int_dmem_inst|mem_ram_b0" is uninferred due to asynchronous read logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_imem:\neorv32_int_imem_inst_true:neorv32_int_imem_inst|mem_ram_b3" is uninferred due to asynchronous read logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_dmem:\neorv32_int_dmem_inst_true:neorv32_int_dmem_inst|mem_ram_b3" is uninferred due to asynchronous read logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_imem:\neorv32_int_imem_inst_true:neorv32_int_imem_inst|mem_ram_b2" is uninferred due to asynchronous read logic
Info (276007): RAM logic "neorv32_top:neorv32_top_inst|neorv32_dmem:\neorv32_int_dmem_inst_true:neorv32_int_dmem_inst|mem_ram_b2" is uninferred due to asynchronous read logic
Error (276003): Cannot convert all sets of registers into RAM megafunctions when creating nodes. The resulting number of registers remaining in design exceeds the number of registers in the device or the number specified by the assignment max_number_of_registers_from_uninferred_rams. This can cause longer compilation time or result in insufficient memory to complete Analysis and Synthesis
I doubt it is a resource limitation from this device, I tried to reduced the default size of IMEM and DMEM anyway, it is not helpful, still the same error -8 instances of uninferred RAM logic.  Can someone give me some hints? Thanks!!!

I do remember this problem from working with old Quartus versions at college... 🙈
According to the Altera forum this seems to be a tool issue. However, you could try using the "recommended coding style for RAMs" for the Cyclone II - maybe the old version of Quartus can identity that and actually infer block RAM.

💾 neorv32_dmem.default.vhd.TXT

Download the file, remove the .TXT suffix and replace the default rtl/core/mem/neorv32_dmem.default.vhd file by this one. This modified DMEM version uses a register for buffering the address for read-accesses. The same modifications could be done for the IMEM and the bootloader ROM without effecting functionality.

Yes, :)
Unfortunately the last version of Quartus which support Cyclone II is 13.0..
I implemented the suggested changes to DMEM and IMEM (not the boot rom, that file seems to be OK), now the compilation is done successfully! Thank you!

@stnolting, what about adding that file to a subdir in setups/quartus, along with a README?

@umarcor
Good idea. But what about putting them as neorv32_*mem.cyclone2.vhd or neorv32_*mem.altera.vhd in rtl/core/mem?

@stnolting that sounds good to me! You are the one not convinced about adding multiple descriptions in rtl/core/mem 😆 . I pushed for the creation of that subdir precisely for use cases such as this one.

Seems like reality convinced me once again :smile:
How should we name those new *MEM files? altera or cyclone2 might be misleading since there are no primitives used. It is just a different HDL style. Maybe alternate?

@stnolting I think it is good to make it explicit that this is for some older version of quartus. According to @ownway88, there is some "last version of Quartus supporting Cyclone II". Therefore, I think neorv32_*mem.cyclone2.vhd is correct. If we find this description to be required for some other tool/device, we can later rename it.


Preprocess SQL Lab query prior to checking syntax

Syntax checking doesn't work if jinja templates haven't been prerendered.
Also remove unreachable return statement. Fixes #4288.

There doesn't seem to be any tests for sql_lab.execute_query(). If the maintainers see the need for one for this specific case, I can add one.

Codecov Report

Merging #4686 into master will increase coverage by <.01%.
The diff coverage is 55.55%.

@@            Coverage Diff            @@
##           master   #4686      +/-   ##
=========================================
+ Coverage    71.4%   71.4%   +<.01%
=========================================
  Files         190     190
  Lines       14935   14934       -1
  Branches     1102    1102
=========================================
  Hits        10664   10664
+ Misses       4268    4267       -1
  Partials        3       3

Impacted Files
Coverage Δ

superset/sql_lab.py
74.6% <55.55%> (+0.39%)
:arrow_up:

Continue to review full report at Codecov.

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


[CLOSED] Dry up Attachments controllers

 Issue by Themitchell
Wednesday 23 July 2014 at 13:20 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/181

This is a first pass at drying up attachments controllers across the board. I have taken a look at the ManualDocumentAttachmentsController to see if it would be reasonable to dry this up too however it feels that it is different enough to the others that we can probably accept that it is different from the other AttachmentsControllers. I am open to discussing people thoughts on this. Perhaps we could do some work to allow ManualDocumentAttachments to use the same views as the other attachments do although there are some differences here such as the breadcrumbs.

Themitchell included the following code: https://github.com/alphagov/specialist-publisher/pull/181/commits

 Comment by tommyp
Wednesday 23 July 2014 at 16:47 GMT

LGTM :+1:


useQuery fires refetch even If the query key did not change

Describe the bug
I am building a page, where I load different components on menu clicks (tabs). I noticed that If I change tabs, react query fires the network request each time I change tabs and come back. The component is not rendered when I am on a different tab, so when I change back and it gets mounted, a network request fires. Even If the query key is just a single string (and did not change).
I changed the refetchOnMount option to false on the query, but now it won't fire the first time or even If I change query key.
I tried to change the staleTime as well, with a number > 0, it also won't ever fire the query.
Your minimal, reproducible example
https://codesandbox.io/s/usequery-not-firing-onmount-cv1wkp?file=/src/App.tsx
Steps to reproduce
Change tabs between 1 and 2. The data is refetched each time to you change it to tab 1. The ["test-query"] query key does not changes.
Expected behavior
I expected the query to fire the first time the component mounts and each following time it mounts again, It should not start a query until the query key changes.
I set the cacheTime so the data should be cached and I am using the same query key. Shouldn't it survive a component remount?
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
Windows. Firefox and Chrome.
react-query version
4.2.2
TypeScript version
4.4.2
Additional context
No response

This is what staleTime is for. It tells react-query to treat data in the cache as "fresh" for a specific amount of time, so that no background refetch happens.
Your problem comes from combining staleTime with initialData, as initialData is put into the cache and is treated as valid data - just as if you had fetched it. So by setting initialData to an empty array, this will be put into the cache. And because you set stale time, that empty array is considered fresh for that time, so no real fetch is happening. That is per design. What you are looking for is likely a combination of staleTime and placeholderData.
Here is a fork with:

staleTime: Infinity
placeholderData: []

https://codesandbox.io/s/usequery-not-firing-onmount-forked-5zovq2?file=/src/App.tsx
Also, have a look at my extensive blog post on that topic: https://tkdodo.eu/blog/placeholder-and-initial-data-in-react-query


Powershell api-version

In the sample PowerShell script, api-version is an error.
/hybridCertificate?api-version=202020-11-01-preview
$DAGName + "?api-version=2021-05-01-preview"
Please use the following api-version.
/hybridCertificate?api-version=2021-08-01-preview
$DAGName + "?api-version=2021-08-01-preview"
Thanks.

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

ID: 9dc7ce4a-4c4b-5b8f-3f2e-9a72efde78a5
Version Independent ID: 6faf5c8b-2bba-bbf7-d4b4-111c1c3c7394
Content: Replicate database with link feature with T-SQL and PowerShell scripts - Azure SQL Managed Instance
Content Source: articles/azure-sql/managed-instance/managed-instance-link-use-scripts-to-replicate-database.md
Service: sql-managed-instance
Sub-service: data-movement
GitHub Login: @sasapopo
Microsoft Alias: sasapopo

Hi @yang-jiayi Thanks for your findings,We will investigate and update as appropriate

Hi @yang-jiayi team will be updating the script by tomorrow we will let you know once done.

Hi @yang-jiayi /hybridCertificate?api-version=2021-08-01-preview has been updated.

@yang-jiayi We will proceed to close this issue now.
If there are further questions regarding this, please comment and we will gladly continue the discussion.


Cannot re-register node after using ConfigurationId when switching back to ConfigurationNames

WMF5, xPSDesiredStateConfiguration upgraded from 3.7 to 3.9 (switched to ESENT database).
I had previously had a successful pull client enrolled in my pull server when using partial configurations--i.e. it was able to pull config and update itself.
I've since been testing Composite Resources instead of partials, so I'm now trying to switch the pull client from using a ConfigurationId + Partial Configurations to using a ConfigurationNames now. Instead of multiple MOF files, I now have a single MOF file ("TestConfiguration.mof").
When I run the LCM configuration against the pull client, this is what I get back:
Registration of the Dsc Agent with the server https://pullserver/PSDSCPullServer.svc failed. The underlying error is: The attempt to register Dsc Agent
with AgentId 1C4637FD-7E90-11E5-8124-005056A4CA09 with the server
https://pullserver/PSDSCPullServer.svc/Nodes(AgentId='1C4637FD-7E90-11E5-8124-005056A4CA09') returned unexpected response code NotFound. .
    + CategoryInfo          : InvalidResult: (root/Microsoft/...gurationManager:String) [], CimException
    + FullyQualifiedErrorId : RegisterDscAgentUnsuccessful,Microsoft.PowerShell.DesiredStateConfiguration.Commands.RegisterDscAgentCommand
    + PSComputerName        : lab-pull-client

So, it applied the configuration fine (LCM:  [ End    Set      ]), but when trying to register the node, it fails.
I also checked the EDB file with ESEDatabaseView and see that no devices are registered in the database:

What could be causing the failure to register? The Agent ID was originally provisioned when I first set up the pull client (with partials). Could that cause it? Is there a way to reset the AgentID (remove it) and "start from scratch" with the LCM? Is there a way to see exactly what is going wrong? The event logs don't necessarily share anything interesting except that it can't send a status report (NotFound) since the device isn't registered.
I still need to test enrollment with a blank VM--to see if a brand new client can enroll without issues.

ConfigurationNames only works with AgentID registration, not ConfigurationID

Yes, sorry maybe I wasn't clear.  I am using agent registration with
configuration names currently, but when it was working I had been using
configuration ID and partials. Currently, the same node is not working with
agent registration and names.
On Mon, Apr 18, 2016, 07:32 Arie Heinrich<EMAIL_ADDRESS>wrote:

ConfigurationNames only works with AgentID registration, not
ConfigurationID
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/PowerShell/xPSDesiredStateConfiguration/issues/110#issuecomment-211359162

will have to see your pullserver and lcm scripts to see how you set it up to try and figure out why its not registering. I assume youre using <IP_ADDRESS>

Yes, 3.9.
The LCM meta config that gets generated (and applied) to the target node is pretty straightforward:
/*
@TargetNode='lab'
@GeneratedBy=MYUSER
@GenerationDate=04/20/2016 12:20:36
@GenerationHost=MYPC
*/

instance of MSFT_WebDownloadManager as $MSFT_WebDownloadManager1ref
{
SourceInfo = "foo.ps1::52::9::ConfigurationRepositoryWeb";
 ServerURL = "https://pullserverhost/PSDSCPullServer.svc";
 ResourceID = "[ConfigurationRepositoryWeb]PullServer";
 RegistrationKey = "GUID";
 ConfigurationNames = {
    "Sandbox"
};

};

instance of MSFT_DSCMetaConfiguration as $MSFT_DSCMetaConfiguration1ref
{
RefreshFrequencyMins = 30;
 RefreshMode = "Pull";
 RebootNodeIfNeeded = False;
 CertificateID = "PUBLIC KEY";

  ConfigurationDownloadManagers = {
  $MSFT_WebDownloadManager1ref
 };
};

instance of OMI_ConfigurationDocument
{
 Version="2.0.0";
 MinimumCompatibleVersion = "2.0.0";
 CompatibleVersionAdditionalProperties= { "MSFT_DSCMetaConfiguration:StatusRetentionTimeInDays" };
 Author="MYUSER";
 GenerationDate="04/20/2016 12:20:36";
 GenerationHost="MYPC";
 Name="PullClient";
};

Though its not the scripts needed, and the pull server one is missing, your LCM mof file seems ok,
so i dont think its the LCMs fault. The pullserver script link might help in finding the problem.
Did you delete the old pull server web site, app pool and files (including the devices.mdb local database) from the IIS and rebuilt the pull server ?
Note that according to the LCM mof file you linked, the mof file on the pull server should be named Sandbox.mof and not the initial "TestConfiguration.mof" you noted on the first post.
The AgentID is regsitered in the windows registry at:
HLKM\Software\Microsoft\DesiredStateConfiguration
Even if you delete the value of the AgentID key, the LCM will still hold it as its in the memory.
Getting a new AgentID can be achieved by running Get-DSCLocalConfigurationManager
But the registration process does it too, which gives the node its initial AgentID

Thanks, I will try playing with it. I enabled Failed Request Tracing and
saw the failed PUT request returning a 404. It's almost as if it is not
trying to register, unless that is what the PUT does.
Yes, I deleted the pull server website files, upgraded to 3.9 and switched
to ESENT, so it is blank.
On Thu, Apr 21, 2016, 01:42 Arie Heinrich<EMAIL_ADDRESS>wrote:

Though its not the scripts needed, and the pull server one is missing,
your LCM mof file seems ok,
so i dont think its the LCMs fault. The pullserver script link might help
in finding the problem.
Did you delete the old pull server web site, app pool and files (including
the devices.mdb local database) from the IIS and rebuilt the pull server ?
Note that according to the LCM mof file you linked, the mof file on the
pull server should be named Sandbox.mof and not the initial
"TestConfiguration.mof" you noted on the first post.
The AgentID is regsitered in the windows registry at:
HLKM\Software\Microsoft\DesiredStateConfiguration
Even if you delete the value of the AgentID key, the LCM will still hold
it as its in the memory.
Getting a new AgentID can be achieved by running
Get-DSCLocalConfigurationManager
But the registration process does it too, which gives the node its initial
AgentID
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/PowerShell/xPSDesiredStateConfiguration/issues/110#issuecomment-212768663

@ArieHein where are the DSC pull server trace events written to? Is it in one of the event logs? I'm referencing the caught exceptions and such. I'm wondering if maybe the pull server can't write to the ESENT database or something deeper.

Turned out to be totally unrelated, our IIS configuration was presenting PUT and DELETE at the application host-level. As soon as I allowed those verbs in the web.config, it worked.
Thanks for your help!

@kamranayub can you give a little more info how you fixed this PUT and DELETE issue?

@dotps1 IIS by default doesn't restrict verbs but it turned out our base image for web servers internally blocked some verbs.
You can find out if that's the case by going to IIS Manager. The rules can be added to the Server level or Website level, so might want to check at the Server level first (the first tree node) and then check an individual website.
Go to Request Filtering:

Then HTTP Verbs:

If there's any in the list, then someone's modified the defaults.

yeah, I'm not getting any FRTs I have tried disabling the firewall on both node and server, I'm only having issues registering the node when I try using the devices.mdb.  the devices.edb works but I get a slew of log files from that method.

This can always be a result of someone hardening the server by URLScan module and limiting verbs


Update KcpTransport.cs

commented out defines

sorry, these are necessary for the MirrorTransport to stay in the repo (mirror isn't known in here).
might just delete it and keep in Mirror later

sorry, these are necessary for the MirrorTransport to stay in the repo (mirror isn't known in here).
might just delete it and keep in Mirror later


Collections without paginator

For fetchAll the DoctrineResource does by default return a paginator. Is there also a possibility to return a collection without a paginator? Would there be interest in a pull-request to make this possible?
We have some smaller collections for which adding a paginator is totally overkill.
Or is the only way setting page size to -1.

Have you tried creating a Query Provider and returning something which looks like a paginator but isn't?  https://github.com/zfcampus/zf-apigility-doctrine/blob/master/src/Server/Query/Provider/AbstractQueryProvider.php#L57
If setting the size to -1 works then that would work for me.  If you really feel not using a paginator by not creating your own Query Provider is a better way I'll be sure to look at a PR.

@Wilt anything to add or can I close this?

Our focus moved on other things. We are no longer working on implementing zf-apigility-doctrine. I can get back on this later. You can leave open or close (I can request reopening when needed later) whatever you prefer. Thanks!


[GLUTEN-7577][VL] Add pattern match for extension rules

What changes were proposed in this pull request?
Fixes: #7577
How was this patch tested?
CI

Conflict resolved, can it be merged? @zhouyuan Thank you.


remove testnet

This removes Testnet as a valid network target. This is a response to the de-facto deprecation of this network, as well as a direct response to Blockfrost removing Testnet

BTW, not sure how to re-generate the docs page, since there was a change there.

BTW, not sure how to re-generate the docs page, since there was a change there.

You don't have to. It all happens automatically ;)


Contact information/use lazy load query

The type of this PR is: TYPE
Use the new hook useLazyLoadQuery for getting the user info

This PR resolves []
Description

PR Checklist (tick all before merging)

[ ] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have tested my changes on iOS and Android.
[ ] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[ ] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added an app state migration, or my changes do not require one. (What are migrations?)
[ ] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against dca3c74cebc0a03e9691be7a72765020c087e30f


custom realtionship attributes doesn't work until make bread for the model

Version information

Laravel: v7
Voyager: v1.3
PHP: 7
Database: [type] [version] (e.g. MySQL 8.0)

Description

custom realtionship attributes doesn't work until make bread for the model
After performing the steps mentioned in the link below
https://voyager-docs.devdojo.com/customization/custom-realtionship-attributes
custom realtionship attributes doesn't work for the model and Not seen in the display fields for relationship in BREAD form.
To solve this problem, just make a bread for your model.

I'm not sure I understand the issue.  Of course it doesn't work without BREAD, since it's sole purposes is to add fields to the BREAD.

Also, Voyager 1.3.* does not support Laravel 7.  Please make sure you're providing accurate version numbers.

For example, when we want to use the $ additional_attributes field that we created in the teacher model for the message table bread,
`namespace App;
use Illuminate\Database\Eloquent\Model;
class Teacher extends Model
{
protected $table = 'teacher';
public $additional_attributes = ['full_name'];
public function getFullNameAttribute()
{
    return "{$this->name} {$this->family}";
}

}`
See the photo at the link below:
http://kiansite.ir/image-share/github00.png
we do not see the $ additional_attributes(full_name) title when selecting the display field in the message bread.
To solve this problem, just make a bread for teacher model.
See the photo at the link below:
http://kiansite.ir/image-share/github01.png
I hope you understand my explanation.
Thanks to you for Voyager

To make it short additional attribute is not shown in the modal while creating a new relationship but only while editing it.

I think I got it wrong, you are probably editing another BREAD like Students and you are saying additional attribute for Teachers doesn't appear if a BREAD for teacher hasn't been created.
Anyway as asked please provide correct version numbers. https://github.com/the-control-group/voyager/issues/4836#issuecomment-602634849

I think I got it wrong, you are probably editing another BREAD like Students and you are saying additional attribute for Teachers doesn't appear if a BREAD for teacher hasn't been created.
Anyway as asked please provide correct version numbers. #4836 (comment)

Hi @MrCrayon, this is my first contribution here, and I don't know if I can complement this issue or need to create a new one. But same happens with me, and my environment is:
PHP: 7.4
Laravel: 8
Voyager: 1.x-dev
The scenario is exactly that you described above. In my case, I created the BREAD for my model that i needed the additional attribute to be used in the relationship on another BREAD and hidden it from the menu and restricted the user access using the roles. But I think its not clear in actual documentation that need.
I propose that we can user additional attributes without the need of creating a BREAD for the model or specify that need in the docs to easy the way from future devs that needs to use it.