[Bug] Unhandled exception. System.MissingMethodException: Method not found: 'System.Collections.Generic.IReadOnlyList`1<System.Object> Avalonia.Markup.Xaml.XamlIl.Runtime.IAvaloniaXamlIlEagerParentStackProvider.get_DirectParents()

Check the following items

[ ] I have looked up relevant Issue

Description of the issue
When attempting to setup SukiUI I added the theme to my App.Styles collection like so:
<Application.Styles>
    <FluentTheme></FluentTheme>
    <suki:SukiTheme ThemeColor="Blue"/>
</Application.Styles>

Which fails to build with the callstack:
System.IO.IOException: Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host. ---> System.Net.Sockets.SocketException: An existing connection was forcibly closed by the remote host
   at System.Net.Sockets.Socket.EndReceive(IAsyncResult asyncResult)
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
   --- End of inner exception stack trace ---
   at System.Net.Sockets.NetworkStream.EndRead(IAsyncResult asyncResult)
   at System.Threading.Tasks.TaskFactory`1.FromAsyncTrimPromise`1.Complete(TInstance thisRef, Func`3 endMethod, IAsyncResult asyncResult, Boolean requiresSynchronization)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Avalonia.Remote.Protocol.BsonStreamTransportConnection.<ReadExact>d__15.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Avalonia.Remote.Protocol.BsonStreamTransportConnection.<Reader>d__16.MoveNext()
13:46:49.704 [Information]  Started AvaloniaDesigner.CreateCompletionMetadataAsync() for "C:\Users\jercl\Projects\Toybox\Build\bin\net8.0\Editor.dll"
13:46:49.705 [Information]  Starting previewer process for '"C:\Users\jercl\Projects\Toybox\Build\bin\net8.0\Editor.dll"'
13:46:49.709 [Information] 6852 Started previewer process for '"C:\Users\jercl\Projects\Toybox\Build\bin\net8.0\Editor.dll"'. Waiting for connection to be initialized.
13:46:49.807 [Information] 6852 Connection initialized
13:46:49.973 [Error] 6852 <= "Unhandled exception. System.MissingMethodException: Method not found: 'System.Collections.Generic.IReadOnlyList`1<System.Object> Avalonia.Markup.Xaml.XamlIl.Runtime.IAvaloniaXamlIlEagerParentStackProvider.get_DirectParents()'."
13:46:49.974 [Error] 6852 <= "   at SukiUI.SukiTheme.!XamlIlPopulate(IServiceProvider, SukiTheme)"
13:46:49.975 [Error] 6852 <= "   at SukiUI.SukiTheme.!XamlIlPopulateTrampoline(SukiTheme)"
13:46:49.979 [Error] 6852 <= "   at SukiUI.SukiTheme..ctor()"
13:46:49.980 [Error] 6852 <= "   at Toybox.App.!XamlIlPopulate(IServiceProvider, App) in C:\Users\jercl\Projects\Toybox\Editor\Editor\App.axaml:line 14"
13:46:49.982 [Error] 6852 <= "   at Toybox.App.!XamlIlPopulateTrampoline(App)"
13:46:49.984 [Error] 6852 <= "   at Toybox.App.Initialize() in C:\Users\jercl\Projects\Toybox\Editor\Editor\App.axaml.cs:line 15"
13:46:49.986 [Error] 6852 <= "   at Avalonia.AppBuilder.SetupUnsafe()"
13:46:49.988 [Error] 6852 <= "   at Avalonia.AppBuilder.Setup()"
13:46:49.989 [Error] 6852 <= "   at Avalonia.AppBuilder.SetupWithoutStarting()"
13:46:49.991 [Error] 6852 <= "   at Avalonia.DesignerSupport.Remote.RemoteDesignerEntryPoint.AppInitializer.ConfigureApp(IAvaloniaRemoteTransportConnection transport, CommandLineArgs args, Object obj)"
13:46:49.994 [Error] 6852 <= "   at Avalonia.DesignerSupport.Remote.RemoteDesignerEntryPoint.Main(String[] cmdline)"
13:46:49.997 [Error] 6852 <= "   at Avalonia.Designer.HostApp.Program.Main(String[] args)"

Package Version
6.0.0-beta7
Environment
Windows 11 Home Version 23H2
Expected Behavior
I expect to be able to use the theming without errors being thrown on build as well as the .axaml previewer to work correctly, currently neither work due to this exception when using a SukiTheme in my application.
Reproduction
Installed via the nuget package manager in VS 2022, then followed the steps provided here: https://github.com/kikipoulet/SukiUI/wiki/1.-Installation to modify my app with SukiUI theming and use the SukiWindow for my main window.
Additional Information
The application I'm attempting to setup with SukiUI is publicly available in this repo, its the Editor cs app under the Editor folder: https://github.com/Konfus-dev/Toybox

I think the problem is beacuse the nuget package is not updated.
I introduced the newest package from the CI process, and it works for me.

assembly config:
example window:

I'm using the latest available (6.0.0-beta7), is that not the true latest? If not could the nuget package be updated?

You can try the latest .dll generated by CI to troubleshoot the problem

I think it's time to upgrade the version in nuget. Using nuget to manage libraries is a very friendly way.

I think it's time to upgrade the version in nuget. Using nuget to manage libraries is a very friendly way.

Guys be patient, it's still in beta state. (Even if it's not far from final version now 🙏).
Releasing a beta in a middle of major changes just because there is a Avalonia new minor version isn't my priority to be honest. Be patient, a new beta version will come in few weeks with latest changes, closer to final version and to the ideal design the library aimed from the start.


Bump pg-promise version

Underlying pg library doesn't work on node 14 with this pg-promise version. Needs pg-promise bump to fix this: https://github.com/brianc/node-postgres/issues/2180

LGTM!

LGTM!


Github API hitting rate limit when unauthenticated

I'm using dependabot-script with GitHub Enterprise.
For some public dependencies, dependabot queries public GitHub.
But, GitHub doesn't like being queried without authentication.
Any suggestions, to make it either more robust, do less queries, or workarounds?
Traceback (most recent call last):
        30: from generic-update-script.rb:121:in `<main>'
        29: from generic-update-script.rb:121:in `each'
        28: from generic-update-script.rb:178:in `block in <main>'
        27: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/pull_request_creator.rb:56:in `create'
        26: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/pull_request_creator.rb:76:in `github_creator'
        25: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/pull_request_creator/message_builder.rb:56:in `pr_message'
        24: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/pull_request_creator/message_builder.rb:354:in `metadata_cascades'
        23: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/pull_request_creator/message_builder.rb:373:in `metadata_cascades_for_dep'
        22: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/pull_request_creator/message_builder.rb:454:in `commits_cascade'
        21: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/pull_request_creator/message_builder.rb:558:in `commits'
        20: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/metadata_finders/base.rb:96:in `commits'
        19: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/metadata_finders/base/commits_finder.rb:42:in `commits'
        18: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/metadata_finders/base/commits_finder.rb:158:in `fetch_github_commits'
        17: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/clients/github_with_retries.rb:86:in `method_missing'
        16: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/clients/github_with_retries.rb:109:in `retry_connection_failures'
        15: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/clients/github_with_retries.rb:89:in `block in method_missing'
        14: from <myenv>/.rvm/gems/ruby-2.5.3/gems/dependabot-core-0.88.3/lib/dependabot/clients/github_with_retries.rb:89:in `public_send'
        13: from <myenv>/.rvm/gems/ruby-2.5.3/gems/octokit-4.13.0/lib/octokit/client/commits.rb:198:in `compare'
        12: from <myenv>/.rvm/gems/ruby-2.5.3/gems/octokit-4.13.0/lib/octokit/connection.rb:19:in `get'
        11: from <myenv>/.rvm/gems/ruby-2.5.3/gems/octokit-4.13.0/lib/octokit/connection.rb:156:in `request'
        10: from <myenv>/.rvm/gems/ruby-2.5.3/gems/sawyer-0.8.1/lib/sawyer/agent.rb:94:in `call'
         9: from <myenv>/.rvm/gems/ruby-2.5.3/gems/faraday-0.15.4/lib/faraday/connection.rb:138:in `get'
         8: from <myenv>/.rvm/gems/ruby-2.5.3/gems/faraday-0.15.4/lib/faraday/connection.rb:387:in `run_request'
         7: from <myenv>/.rvm/gems/ruby-2.5.3/gems/faraday-0.15.4/lib/faraday/rack_builder.rb:143:in `build_response'
         6: from <myenv>/.rvm/gems/ruby-2.5.3/gems/faraday-0.15.4/lib/faraday/request/retry.rb:128:in `call'
         5: from <myenv>/.rvm/gems/ruby-2.5.3/gems/octokit-4.13.0/lib/octokit/middleware/follow_redirects.rb:61:in `call'
         4: from <myenv>/.rvm/gems/ruby-2.5.3/gems/octokit-4.13.0/lib/octokit/middleware/follow_redirects.rb:73:in `perform_with_redirection'
         3: from <myenv>/.rvm/gems/ruby-2.5.3/gems/faraday-0.15.4/lib/faraday/response.rb:8:in `call'
         2: from <myenv>/.rvm/gems/ruby-2.5.3/gems/faraday-0.15.4/lib/faraday/response.rb:61:in `on_complete'
         1: from <myenv>/.rvm/gems/ruby-2.5.3/gems/faraday-0.15.4/lib/faraday/response.rb:9:in `block in call'
<myenv>/.rvm/gems/ruby-2.5.3/gems/octokit-4.13.0/lib/octokit/response/raise_error.rb:16:in `on_complete': GET https://api.github.com/repos/theintern/intern/compare/4.2.2...4.4.1: 403 - API rate limit exceeded for <myip>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.) // See: https://developer.github.com/v3/#rate-limiting (Octokit::TooManyRequests)

Hello @greysteil For users which only want to use GHE, is there a way to completely disable interaction with GitHub.com?


nixos/tests/networking: fix flaky scripted.dhcpSimple test

The underlying issue is unknown, but starting the router first and then the client makes the test not flaky.
It's also weird that I can't reproduce this issue on my machine (Ryzen 7 5700U laptop, kernel 6.6.63) while it's reliably reproducible on our aarch64 community box (kernel 5.15.163).
https://github.com/NixOS/nixpkgs/issues/360089

Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofbord test networking.scripted.dhcpSimple networking.networkd.dhcpSimple

@ofborg test networking.scripted.dhcpSimple networking.networkd.dhcpSimple

Testing looks good to me.  On an idle larger machine I did a few runs of both before and after this PR, getting 100% failures and 100% successes.

I may be reproducing something like this on bare metal (24.11.20241205.4dc2fc4 (Vicuna)). Unsure if it's the same issue, but dhcpcd does this and now I'm suspicious of this fix:
Dec 12 20:14:12 monomyth systemd[1]: Started DHCP Client.
Dec 12 20:14:12 monomyth dhcpcd[168047]: lan: probing for an IPv4LL address
Dec 12 20:14:14 monomyth dhcpcd[168047]: arp_announce1: Invalid argument
Dec 12 20:14:17 monomyth dhcpcd[168047]: backbone: using IPv4LL address <IP_ADDRESS>
Dec 12 20:14:17 monomyth dhcpcd[168047]: backbone: adding route to <IP_ADDRESS>/16
Dec 12 20:14:17 monomyth dhcpcd[168047]: backbone: adding default route
Dec 12 20:14:17 monomyth dhcpcd[168047]: lan: using IPv4LL address <IP_ADDRESS>
Dec 12 20:14:17 monomyth dhcpcd[168047]: lan: adding route to <IP_ADDRESS>/16
Dec 12 20:14:18 monomyth dhcpcd[168047]: backbone: deleting default route

I have two virtual interfaces configured using VLANs, lan and backbone. When I unplug and replug the ethernet cable, it suddenly starts working again.
Further, when dhcpcd renews, I get this:
Dec 12 20:14:09 monomyth dhcpcd[168047]: arp_probe1: Invalid argument
Dec 12 20:14:09 monomyth dhcpcd[168047]: arp_probe1: Invalid argument
Dec 12 20:14:10 monomyth dhcpcd[168047]: arp_probe1: Invalid argument
Dec 12 20:14:10 monomyth dhcpcd[168047]: arp_probe1: Invalid argument
Dec 12 20:14:11 monomyth dhcpcd[168047]: backbone: probing for an IPv4LL address
Dec 12 20:14:12 monomyth dhcpcd[168047]: arp_announce1: Invalid argument
Dec 12 20:14:12 monomyth dhcpcd[168047]: arp_announce1: Invalid argument

The sequencing here, ARP errors, and failure to get a DHCP address are making me wonder why this test got flaky.

The permission errors might be because of https://github.com/NixOS/nixpkgs/pull/336988
Maybe you could try reverting it locally and check if it makes any difference


RA-TLS cert verification call back

I am Using RA-TLS certificate and private key in my gRPC server.py as you know with RA-TLS, there is no certificate chain anymore.There is only a self-signed certificate that the server generates on the fly (using a special RA-TLS function) and that the client verifies (using a special RA-TLS-specific callback). In other words, RA-TLS throws away the whole classic PKI infrastructure, with root CAs and cert chains.
I have  to find a function in gRPC that allows me to install my own verification callback. In OpenSSL, for example, this function is called SSL_CTX_set_cert_verify_callback(). In mbedTLS, for another example, this function is called mbedtls_ssl_conf_verify().
I read the documents and all needed root certificate or CA, is there any function in gRPC to have this verify_callback to verify RA-TLS certificate in client side?

This functionality does not currently exist. There is an issue open tracking the work of adding this. Closing this issue as a duplicate of https://github.com/grpc/grpc/issues/32635


Finalize module renaming from external-storage to blob-storage

Resolves #23

@shevron how much work to finish this one? It would be great to get it in soon 👍

Work on this is finished and should work for 0.6.1 compatible code. Before this is merged, we need to create a branch in ckanext-datahub that changes all references to this extension.
In addition, any extension that relies on this code will need to be updated in accordance, or be pinned to older versions of this extension / datapub.

Work on this is finished and should work for 0.6.1 compatible code. Before this is merged, we need to create a branch in ckanext-datahub that changes all references to this extension.
In addition, any extension that relies on this code will need to be updated in accordance, or be pinned to older versions of this extension / datapub.


Makes pachd set the cluster-id if it's unset.

Fixes #1268

LGTM

I'm clicking the merge button since we want this in for a point release now

:+1:


Revert heading, pitch, roll changes

Reverts #5848 to fix #5868.
I'll add the changes to another branch when this is merged.

@bagnell, thanks for the pull request! Maintainers, we have a signed CLA from @bagnell, so you can review this at any time.
I am a bot who helps you make Cesium awesome! Thanks again.

Looks good @bagnell, thanks!

@kaktus40 sorry, but we decided to revert the heading/pitch/roll counter clockwise changes you made.  It was hard to justify making such a big breaking change that would effect the vast majority of our user base.  See the discussion in #5868.
We may decide to make this change eventually, but we need to discuss it more so we understand the benefit of using CCW compared to the cost of forcing everyone to switch.  We'll continue discussion in #5666

It's OK for me, I just hope that the community make a choice after thinking about it.


Don't need host dev mount for pre-created partition

Signed-off-by: Sven Dowideit<EMAIL_ADDRESS>add debug option passing, and fix pre-made partition suport (broke AMI building)

Thank you very much @SvenDowideit
I have tried the script https://github.com/rancher/os/blob/master/scripts/run-install using RancherOS v0.9.1
Works like a charm.


App crash after slide of 10 pages

RN 0.60
I have 600 pages. This is code. After fast lsiding of 10 pages app crash.
Doas viewager has RTL support?
   <ViewPager
         style={{flex:1}}
          initialPage={5}>
          {dataar.map((item, index) => {
                return(
                    <View key={index + 1}>
                      {Object.values(item)[0].map((it, key) => {
                          return key == 0 ?
                                <TouchableOpacity key={key} activeOpacity={1} style= .
   {styles.imageView} onPress={this.showHeaderFooter}>
                                       <Image style={{ width: IMGWIDTH, height: IMGHEIGHT }}
    source={it.url} />
                                  </TouchableOpacity>
                             : null
                            })
                          }
                    </View>
                    )
                  })
               }
 </ViewPager>

This component renders all children immediately, so in above case app will crash. We need to figure out, how to handle this case.

I reduced it to 30 items and again app crash.

@IsmetGlumcevic you should probably add few pages and then add them dynamically while you move between them. Could you also share a repository where we can reproduce it?

@ferrannp  this scenario happened with us with low memory phones after logging it on android studio it showed this error that it's out of memory after 100 scroll.
As for rtl I just did the a replica with the following function edited in ReactViewPage.java
/package/ void addViewToAdapter(View child, int index) {
getAdapter().addView(child, 0);
}
so instead of adding at the end of the list I've added at index 0 with no performance issues!


Blobs are not dumped correctly

When I try to dump tables with blobs, the module is going to generate MySQL syntax errors by adding an additional single quote before the value.
Example:
BLOB Value: 613a303a7b7d
Output in INSERT statement:
[...] '1',NULL,'',NULL,NULL,''613a303a7b7d',NULL [...]
                             ^

This leads to problems with exporting BLOB-including tables.

Thanks for reporting i`ll check soon.


Update 1.1 API tests to use latest python client library

This uses the latest api1.1 client library pip install on_http_api1_1 generated off monorail.yml (with auth settings).   Keeps library imports consistent with redfish 1.0 and 2.0 api's.
@RackHD/corecommitters @zyoung51 @WangWinson @uppalk1 @keedya @geoff-reid @brianparry

:+1:

:+1:

+1

:+1:


RFC: Toga API for style-related operations

This is a request for comment on a design decision we have to make about the APIs exposed by Toga for style-related operations.
The problem
What API should we expose for style-related changes to a widget? Consider the following operations:

Set the font face, font size and text color of this label
Set the background color of this widget
Hide/show this widget (either "don't display, but retain layout", or "remove from layout entirely")

These are all relatively common things for a widget toolkit to need to do. There will inevitably be an API on platform backends for supporting these operations.
However, what is the API that the Toga interface layer should expose? We use Colosseum to apply styles - however, the style API is deliberately abstracted so that you can pass in any style object you want. In theory, you could define your own style objects implementing a different layout approach (e.g., constraint-based layout?) I'm not sure if anyone will actually ever want to do this - but the option exists.
But should we force all style operations through the style API? Or should we also expose a more traditional functional interface for style operations? i.e., would you rather see:

Fonts:

label.style.set(font=toga.Font(face='Times New Roman', size=12*pt); or
label.font = toga.Font(face='Times New Roman', size=12*pt)

Background color:

box.style.set(background_color='#abcdef'); or
box.background_color = '#abcdef'

Show/Hide:

box.style.set(display=None, visibility=False); or
box.hide() and box.visible = False

If we do provide both interfaces, we're faced with some questions:

Which one is the "canonical" definition. If I set the font with CSS, then change the font with a widget API, what happens?

The widget is the canonical definition, but the CSS declaration is updated; or
The widget is the canonical definition, but the CSS declaration is not updated. CSS is just a way of getting an "initial" style defined; or
The CSS declaration is canonical. Widgets have no style state; Style objects are expected to expose an API that matches Toga, so if you call hide() on a widget, it calls hide() on the style object, which sets display=None on the style object, which pushes that down to the widget implementation.

In documentation and examples, which API do we favour?

I've been going back and forth in my mind about this; I'm torn between the purity and internal consistency of using CSS for all style operations, and the simple utility of having methods like hide() available. I'm interested in hearing other opinions.

I personally favor the CSS approach because I believe consistency and familiarity make API's easier to learn. Additionally, I think the complexity of the current API can be helped. If the CSS class implemented __setattr__ and __getattr__, It would allow an API with similar look and feel to native HTML5 style:
label.style.font = toga.Font(face='Times New Roman', size=12*pt)
box.style.background_color='#abcdef'
box.style.display = None and box.style.visibility = False
The main purpose would just be to bring back some "simple utility" as well as making those familiar with CSS feel at home :)
Of course, I would ultimately like to see stylesheets made possible, which includes having ids and class names for widgets. Maybe not so much for example apps, but for larger scale applications where many widgets will share the same theme colors etc., and most widgets will have custom padding/margin.
This leads me to a question: should Toga widgets define their own default styles? For example, toga.Button has no default margin, so when one button is placed adjacent to another, they "touch" (at least on GTK). A good portion of the example apps is used avoiding this by adding margin/padding. I would suggest that each widget gives itself default margin and padding, say 10px, so that it "just works" out of the box.

I really like that proposal @DariusMontez! One CSS style API, but with a simple and consistence implementation would really be optimal.

I really prefer the "Python" way like. Although it is "odd" compared to commonly use toolkits, I think the whole point is to make Toga exploit Python constructs... setters and getters
So :+1: label.font = toga.Font(face='Times New Roman', size=12*pt)
and :-1: label.style.set(font=toga.Font(face='Times New Roman', size=12*pt)
BUT, following this same line of reasoning
box.background_color = '#abcdef' feels like a "hack" so it perhaps could be more like:
box.background_color = toga.Color('#abcdef')
box.background_color = toga.Color(r=150, b=200, g=190, alpha=200)

?

@DariusMontez That's already possible - I was just using the widget.style.set() API in my examples. Under the hood, style.set() is just a proxy for "turn all these kwargs into setattr" calls.
Default widget styles (and platform-specific default widget styles) are both entirely possible, as are style sheets. Default widget styles are a little easier, because we can define them directly on the widgets themselves; stylesheets require an implementation of the CSS selection and cascading module - which is entirely possible, just more work. But that's why I started Colosseum... :-)
However, I agree that having default paddings and margins (just as you do in a browser - H1 has a default font, margin, padding, leading, etc) would be desirable.

@goanpeca To be clear: Is your objection to:

label.style.set(font=toga.Font(face='Times New Roman', size=12*pt), or
label.style.font = toga.Font(face='Times New Roman', size=12*pt)?

i.e., is it to pushing all the style operations to the style object, or on the set() API specifically? As I indicated to @DariusMontez, the set() API is just a convenience for multiple assignments - the individual attribute assignment approach is also possible.
The Color approach you describe is already possible (or will be, once the Colosseum 0.2 is finalised). Colosseum 0.2 provides an rgb(), rgba() etc definitions, matching those provided by CSS. However, just as CSS does, it also allows '#abcdef' and 'lightgoldenrodyellow' as string definitions.
However, for me, this is probably the biggest reason to stick to widget.style.background_color = 'xxx' (rather than widget.background_color = 'xxx'. It means the style definitions are deliberately kept apart from the widget itself, in the same way that the data definitions are kept apart from the widget itself. This has the (theoretical) side effect that you could assign any other style object you wanted.
So, you could (theoretically) define a "CSS, but disallowing string assignment of colours" style module, and you wouldn't require any changes on Toga itself. You could even mix your CSS styles on one widget with your "no-strings-CSS" styles on a different widget in the same app.

I guess set() feels weird on python, plus having a set type is double weird, maybe I am just being too picky.

The Color approach you describe is already possible (or will be, once the Colosseum 0.2 is finalised). Colosseum 0.2 provides an rgb(), rgba() etc definitions, matching those provided by CSS. However, just as CSS does, it also allows '#abcdef' and 'lightgoldenrodyellow' as string definitions.

Yeah, something like what CSS provide, the Color example was Qt messing with me again... :-\

However, for me, this is probably the biggest reason to stick to widget.style.background_color = 'xxx' (rather than widget.background_color = 'xxx'. It means the style definitions are deliberately kept apart from the widget itself, in the same way that the data definitions are kept apart from the widget itself. This has the (theoretical) side effect that you could assign any other style object you wanted.

Yes, it makes sense now
And feels more clean to do
widget.style.load_from_css(css)
print(widget.style.css(css))

So, you could (theoretically) define a "CSS, but disallowing string assignment of colours" style module, and you wouldn't require any changes on Toga itself. You could even mix your CSS styles on one widget with your "no-strings-CSS" styles on a different widget in the same app.

Interesting and powerful but probably not something we should advice :-p

@goanpeca I hadn't considered the API overlap with the set() object, but I completely agree. Maybe update()would be a better name for that API (and one that has precedent withdict` objects).
Any help with Colosseum would be most gratefully accepted :-)
The only use case I can think of for a custom style class is to use a completely different layout scheme - for example, someone who wants to do constraint-based layout for widgets, or have a really simple grid-based layout. I don't think it's a big use case, but if Django has taught me anything, it's that if you don't get your architecture clear to start with, you're gonna be fighting it for life.

My $0.02: I'm in favor of the widget.style.{property} = {type} form. It feels like the right blend of CSS and Pythonic ways of doing things.
As a bonus, we can make the properties on style discoverable through dir(). Having .update() functions doesn't feel particularly pythonic unless the repr of the style object and all sub-objects is a dict.
All of the above is my opinion, but it seems to be in keeping with the group.

I support @DariusMontez , it shold be made as simple as possible
label.style.font = toga.Font(face='Times New Roman', size=12*pt)
box.style.background_color='#abcdef'
box.style.display = None and box.style.visibility = False

Closing this on the basis that the "widget.style.{property} = {value}" approach has effectively been implemented in the 0.3 branch.


[Bug] Insert selected suggestion doesn't work on sidepanes

Pretty self-explanatory. I've got both Tab and Esc assigned to Insert selected suggestion, and neither of them works for auto-completions in sidebar panes.
And yes, they do work perfectly on main panes, in the same instance, no changes in-between.
EDIT: clicking on the suggestion does work, so it's a thing with the hotkey.

Obsidian issue. They've been unable to change 3 lines of code for some reason.
https://forum.obsidian.md/t/hotkeys-commands-should-apply-to-sidebar-notes-when-they-are-focused/2740/23
https://forum.obsidian.md/t/workspace-getactiveviewoftype-does-not-return-markdownview-docked-in-sidebar/37743

Fixed in Obsidian 0.15.0 https://forum.obsidian.md/t/obsidian-release-v0-15-0-insider-build/38948


Make ButterknifeProcessor use significantly lesser memory during compilation

Currently, Butterknife processor scans and parses all available R class files and their resources. When working with modules which have a large number of dependencies, the many merged R files from different packages and same resource identifiers lead to significant amount of memory required to maintain mappings of ids.

This change makes Butterknife only maintain mapping of ids that are actually referenced in the various annotations.
Some element instances have been converted to not be serialized to strings until necessary.

Doing so consumes significantly lesser memory and leads to faster compilation as well. Some snapshots of heap usage during compilation before and after
Before

After

The retained heap after the change is due to another bug not related to Butterknife.
As for build time, one of our largest modules (~2.5k java files) that uses butterknife modules went from 130s to 90s to build (about 30 % savings)

Merged manually with some indentation fixes. Thanks.


CharBERT will not generate vocab.txt?

Just curious whether the vocab.txt will be generated. It should not be right?

Hi @Believer215 , I'm not sure what you mean by "will be generated". In any case, you are right to assume that there is no vocab.txt when using CharacterBERT since it does not rely on wordpieces and thus no vocabulary 😊


truncate should always be treated as warning in cast decimal as decimal

What problem does this PR solve?
Issue Number: close #7348
Problem Summary:
Accoring to TiDB's implementation, truncate in cast decimal as decimal should be always treated as a warning instead of error.
https://github.com/pingcap/tidb/blob/96fa4692b09b43ae587ea449e5891a9d9fd6e5c8/types/datum.go#L1512-L1519
What is changed and how it works?
Check List
Tests

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/merge

/cherry-pick release-6.5-20230425-v6.5.2


Runtime Error: Invalid Memory Address. After Upgrade to 1.6.0

My environment
Egress and livekit server are installed on two different servers in the same local network. The redis server is installed on the same server like the livekit server. Egress communicates with redis and livekit over the local network, not public. for example: ws://<IP_ADDRESS>:7880
Issue
I updated my egress docker image from version 1.5.5 to version 1.6.0.  I call startRoomCompositeEgress function to start stream and I get successful response. But after a while egress gives an error and the stream is stopped. And my livekit server logs contains my servers public ip addresses but i didn't communicate via public ip adress. I shared it with you as LIVEKIT_SERVER_PUBLIC_IP and EGRESS_SERVER_PUBLIC_IP in the livekit logs.
I could not find the cause of the problem. When I revert back to version 1.5.5 it works correctly.
StartRoomCompositeEgress typescript:
const encodingOptions: EncodingOptions = {
        width: 1920,
        height: 1080,
        framerate: 30,
        videoBitrate: 9000,
        videoCodec: VideoCodec.H264_MAIN,
        audioBitrate: 128,
        audioCodec: AudioCodec.AAC,
        depth: 24,
        audioFrequency: 44100,
      };

      const options: RoomCompositeOptions = {
        layout: roomSettings.streamLayout,
        encodingOptions: encodingOptions,
      };

const rtmpOutput: StreamOutput = {
        protocol: StreamProtocol.RTMP,
        urls: roomSettings.streamUrls.map((url) => url.url),
      };

      const streamInfo = await this.egressClient.startRoomCompositeEgress(
        roomMember.room.uid,
        rtmpOutput,
        options,
      );

Egress Log
2023-03-13T11:53:00.020Z        DEBUG   egress  service/handler_deprecated.go:108       handling request        {"nodeID": "NE_VypyLcqobuTd", "handlerID": "EGH_bkHiTb4T42sJ", "clusterID": "", "egressID": "EG_aT2Jmgw7nVbY", "egressID": "EG_aT2Jmgw7nVbY", "requestID": "RPC_aaciiAcWw4VF"}
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x14735e5]

goroutine 1 [running]:
github.com/livekit/egress/pkg/pipeline.(*Pipeline).removeSink(0xc000627480, {0xc00061a960, 0x57}, {0x0, 0x0})
        /workspace/pkg/pipeline/pipeline.go:282 +0x145
github.com/livekit/egress/pkg/pipeline.(*Pipeline).UpdateStream(0xc000627480, {0x1d75810?, 0xc000040028?}, 0xc0002ae690)
        /workspace/pkg/pipeline/pipeline.go:263 +0x24d
github.com/livekit/egress/pkg/service.(*HandlerV0).Run(0xc0005df4d0)
        /workspace/pkg/service/handler_deprecated.go:112 +0x7c5
main.runHandler(0xc00028a580?)
        /workspace/cmd/server/main.go:215 +0x3dd
github.com/urfave/cli/v2.(*Command).Run(0xc00028a580, 0xc0001f7480, {0xc0003dc000, 0x7, 0x7})
        /root/go/pkg/mod/github.com/urfave/cli/v2@v2.25.0/command.go:273 +0xa42
github.com/urfave/cli/v2.(*Command).Run(0xc00028a840, 0xc0001f72c0, {0xc00003c180, 0x8, 0x8})
        /root/go/pkg/mod/github.com/urfave/cli/v2@v2.25.0/command.go:266 +0xc8a
github.com/urfave/cli/v2.(*App).RunContext(0xc00047b0e0, {0x1d75810?, 0xc000040028}, {0xc00003c180, 0x8, 0x8})
        /root/go/pkg/mod/github.com/urfave/cli/v2@v2.25.0/app.go:332 +0x605
github.com/urfave/cli/v2.(*App).Run(...)
        /root/go/pkg/mod/github.com/urfave/cli/v2@v2.25.0/app.go:309
main.main()
        /workspace/cmd/server/main.go:71 +0x47c
2023-03-13T11:53:00.029Z        ERROR   egress  service/manager.go:132  process failed  {"nodeID": "NE_VypyLcqobuTd", "clusterID": "", "error": "exit status 2"}
github.com/livekit/egress/pkg/service.(*ProcessManager).awaitCleanup
        /workspace/pkg/service/manager.go:132
2023-03-13T11:53:00.030Z        WARN    egress  service/handler.go:202  egress failed   {"nodeID": "NE_VypyLcqobuTd", "clusterID": "", "egressID": "EG_aT2Jmgw7nVbY", "request_type": "room_composite", "output_type": "stream", "error": "exit status 2"}
2023-03-13T11:53:00.032Z        INFO    egress  service/service_deprecated.go:33        shutting down   {"nodeID": "NE_VypyLcqobuTd", "clusterID": ""}

Livekit server logs
0|livekit-server    | 2023-03-13T11:52:56: 2023-03-13T11:52:56.273Z     INFO    livekit service/egress.go:189   using deprecated egress client
0|livekit-server    | 2023-03-13T11:52:56: 2023-03-13T11:52:56.276Z     INFO    livekit service/twirp.go:128    API Egress.StartRoomCompositeEgress     {"service": "Egress", "method": "StartRoomCompositeEgress", "room": "2083548a-3a91-496b-af78-b2454bb312b0", "baseUrl": "", "outputType": "*livekit.RoomCompositeEgressRequest_Stream", "egressID": "EG_aT2Jmgw7nVbY", "duration": "3.850166ms", "status": "200"}
0|livekit-server    | 2023-03-13T11:52:57: 2023-03-13T11:52:57.271Z     INFO    livekit rtc/room.go:327 new participant joined  {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "pID": "PA_GgcRkb8xAZoT", "participant": "EG_aT2Jmgw7nVbY", "protocol": 8, "options": {"AutoSubscribe":true}}
0|livekit-server    | 2023-03-13T11:52:57: 2023-03-13T11:52:57.272Z     INFO    livekit rtc/room.go:263 participant state changed       {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "state": "JOINED", "participant": "EG_aT2Jmgw7nVbY", "pID": "PA_GgcRkb8xAZoT", "oldState": "JOINING"}
0|livekit-server    | 2023-03-13T11:52:57: 2023-03-13T11:52:57.272Z     INFO    livekit rtc/participant.go:1105 sending offer   {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "participant": "EG_aT2Jmgw7nVbY", "pID": "PA_GgcRkb8xAZoT", "remote": false, "transport": "SUBSCRIBER"}
0|livekit-server    | 2023-03-13T11:52:57: 2023-03-13T11:52:57.274Z     INFO    livekit service/rtcservice.go:258       new client WS connected {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "participant": "EG_aT2Jmgw7nVbY", "pID": "PA_GgcRkb8xAZoT", "remote": false, "connID": "CO_FREEYkrHWgRm"}
0|livekit-server    | 2023-03-13T11:52:57: 2023-03-13T11:52:57.290Z     INFO    livekit rtc/participant.go:490  received answer {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "participant": "EG_aT2Jmgw7nVbY", "pID": "PA_GgcRkb8xAZoT", "remote": false, "transport": "SUBSCRIBER"}
0|livekit-server    | 2023-03-13T11:52:58: 2023-03-13T11:52:58.297Z     INFO    livekit rtc/transport.go:599    selected ICE candidate pair     {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "participant": "EG_aT2Jmgw7nVbY", "pID": "PA_GgcRkb8xAZoT", "remote": false, "transport": "SUBSCRIBER", "pair": "(local) udp4 host LIVEKIT_SERVER_PUBLIC_IP:53126 <-> (remote) udp4 prflx EGRESS_SERVER_PUBLIC_IP:37456 related :0"}
0|livekit-server    | 2023-03-13T11:52:58: 2023-03-13T11:52:58.306Z     INFO    livekit rtc/room.go:263 participant state changed       {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "state": "ACTIVE", "participant": "EG_aT2Jmgw7nVbY", "pID": "PA_GgcRkb8xAZoT", "oldState": "JOINED"}
0|livekit-server    | 2023-03-13T11:53:00: 2023-03-13T11:53:00.007Z     INFO    livekit service/twirp.go:128    API Egress.ListEgress   {"service": "Egress", "method": "ListEgress", "duration": "1.341723ms", "status": "200"}
0|livekit-server    | 2023-03-13T11:53:00: 2023-03-13T11:53:00.011Z     INFO    livekit service/twirp.go:128    API Egress.ListEgress   {"service": "Egress", "method": "ListEgress", "duration": "5.043286ms", "status": "200"}
0|livekit-server    | 2023-03-13T11:53:00: 2023-03-13T11:53:00.017Z     INFO    livekit service/twirp.go:128    API RoomService.ListParticipants        {"service": "RoomService", "method": "ListParticipants", "room": "2083548a-3a91-496b-af78-b2454bb312b0", "duration": "363.806µs", "status": "200"}
0|livekit-server    | 2023-03-13T11:53:00: 2023-03-13T11:53:00.031Z     INFO    livekit service/rtcservice.go:231       finishing WS connection {"room": "2083548a-3a91-496b-af78-b2454bb312b0", "roomID": "RM_aQGx8LQCyaSr", "participant": "EG_aT2Jmgw7nVbY", "pID": "PA_GgcRkb8xAZoT", "remote": false, "connID": "CO_FREEYkrHWgRm"}
0|livekit-server    | 2023-03-13T11:53:00: 2023-03-13T11:53:00.031Z     ERROR   livekit service/ioinfo.go:89    egress failed   {"egressID": "EG_aT2Jmgw7nVbY", "error": "exit status 2"}

Hey @ozgur-d, it looks like the panic was caused by the UpdateStream RPC being called with a url that wasn't part of the pipeline.
I've fixed the panic in https://github.com/livekit/egress/pull/301.
As a side note, it's recommended to add
egress:
  use_psrpc: true

to your livekit config to upgrade the internal communication system (https://github.com/livekit/livekit/blob/master/pkg/config/config.go#L253)


Dynamically get latest binary release

Current download links on this page hardcode references to 1.4.1 - proposed change retrieves download URL for latest.
Would be cleaner to use jq, but would create a dependency.

This change is

CLA signed.

@jfenton I'm not able to test wget on my machine. Is there someone else that you know of that can do the tech review? Also, can you resolve the merge conflict? Thx

@jfenton,

Do you want to move forward with this if we can find a tech reviewer?
I did a quick test on Linux and got a failure (see below).
It looks like people have been updating the hard-coded version: https://github.com/kubernetes/kubernetes.github.io/commit/582f6f5bd3a483c350ffd6e6dbf873239d2977d8#diff-809955e4761fac8b74780ff37771d45a, https://github.com/kubernetes/kubernetes.github.io/commit/108a4051fe66415d28ea8153f515457c3b87395b#diff-809955e4761fac8b74780ff37771d45a

@jfenton, @chenopis, We might try something like this:
wget https://github.com/kubernetes/kops/releases/download/{{page.fullversion}}/kops-linux-amd64

sudo wget -O /usr/local/bin/kops curl -s https://api.github.com/repos/kubernetes/kops/releases/latest | grep browser_download_url | grep amd64 | head -n 1
--2017-09-08 22:26:25--  ftp://%22browser_download_url%22/
=> ‘.listing’
Resolving "browser_download_url" ("browser_download_url")... failed: Name or service not known.
wget: unable to resolve host address ‘"browser_download_url"’
"https://github.com/kubernetes/kops/releases/download/1.7.0/kops-darwin-amd64": Scheme missing.

@jfenton ping

@chenopis I suggest we wait two more days and close this.

Closing for now. @jfenton Please reopen if you want to move forward with this.


VS Code Terminal Won't Map Option Key to Meta on OS X

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.63.2
OS Version: OS X 10.15.7
iTerm2: Build 3.4.12
Terminal: Version 2.10 (433)

Steps to Reproduce:
The "terminal.integrated.macOptionIsMeta": truesetting is not working on my system (https://code.visualstudio.com/updates/v1_20#_option-as-meta-on-macos).
I made this same change to my system Terminal app and iTerm2. iTerm is the only one that works. In iTerm the option key is mapped to Meta but not in the VS Code terminal or Terminal.
settings.json
{
  "editor.tabSize": 2,
  "window.zoomLevel": 1,
  "code-runner.preserveFocus": false,
  "code-runner.showExecutionMessage": false,
  "code-runner.saveAllFilesBeforeRun": true,
  "code-runner.runInTerminal": true,
  "editor.formatOnPaste": true,
  "editor.formatOnSave": true,
  "js/ts.implicitProjectConfig.experimentalDecorators": true,
  "editor.defaultFormatter": "esbenp.prettier-vscode",
  "[prisma]": {
    "editor.defaultFormatter": "Prisma.prisma"
  },
  "javascript.updateImportsOnFileMove.enabled": "always",
  "workbench.iconTheme": "vscode-icons",
  "vim.easymotion": true,
  "vim.incsearch": true,
  "vim.useSystemClipboard": true,
  "vim.useCtrlKeys": true,
  "vim.hlsearch": true,
  "vim.insertModeKeyBindings": [
    {
      "before": ["j", "j"],
      "after": ["<Esc>"]
    }
  ],
  "vim.normalModeKeyBindingsNonRecursive": [
    {
      "before": ["<leader>", "d"],
      "after": ["d", "d"]
    },
    {
      "before": ["<C-n>"],
      "commands": [":nohl"]
    }
  ],
  "vim.leader": "<space>",
  "vim.handleKeys": {
    "<C-a>": false,
    "<C-f>": false
  },
  "terminal.integrated.macOptionIsMeta": true
}

VS Code

iTerm2

Terminal

Closing. This appears to be an issue with OS X, not VS Code.
The relationship between Option/Alt and Esc/Meta does not work well. Specifically, Option will work as Meta in an Option+j scenario but will not work in an Option+Left scenario. When using the arrow keys you must use Esc as Meta.

For me terminal.integrated.macOptionIsMeta didn't work until I switched from ABC - Extended to U.S. keyboard layout


Validation Perplexities

Thanks for sharing this amazing work. This will hopefully help in developing a better understanding of how LLMs work.
I had one question. Are the validation perplexities for each of the models available (ideally with every model snapshot) so that we can compare models on equal footing?

Hi! We're actively working on gathering all evals and getting them posted in this repo, including LAMBADA perplexity. If you're wondering about validation perplexity on the Pile, unfortunately in order to save compute we did not run on the validation set during training very frequently at all.
We plan to have LAMBADA perplexity up for all models on this repo for 15 evenly spaced checkpoints (steps 3000, 13000,..., 133000, 143000) asap! Would you want or need more granular results for any experiments?

Hi! Thank you for your kind response.

We plan to have LAMBADA perplexity up for all models on this repo for 15 evenly spaced checkpoints (steps 3000, 13000,..., 133000, 143000) asap! Would you want or need more granular results for any experiments?

That's great! Thank you for the awesome work. I was particularly using the aforementioned checkpoints, so having the Lambada perplexity (which should be proportional to the validation perplexity on PILE) should help.

fantastic, we should have those up and all corrected within a week ideally!

Hi @shoaibahmed ! I believe all models' evaluations should now be up-to-date.
Let me know if any evals look suspect or anything's missing! Hopefully all is well though, barring some cleanup of filenames (also, all lambada tasks are the lambada_openai task in the Eleuther lm-evaluation-harness.)
Also let me know if having more granular evals/PPL would be helpful to your research for any reason :)

That's really awesome. Thank you for your prompt action on this. I will go ahead and close this issue now.
I will open a new issue in case something else comes up which might be beneficial. Thanks once again for your time and effort.


bug: Add tmp in .gitignore and correct config for tmp

Bug Report
Beerus version:

Current behavior:

Expected behavior:

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

I can take this one.

As this issue is related to DATA_DIR, do you mind guys if I rename the DATA_DIR variable? It's not enough specific to Beerus, and that may conflict with some other programs.
Proposal: BEERUS_DATA_DIR.
Other variables are fine because they are specific. But this one is too generic.
This can stay called data_dir in structs, but the environment variable may change to something like BEERUS_DATA_DIR.

By the way, the issue is not related to gitignore. 👍

CI fixed. Sorry it was from my side.

The CI bug will be fixed by #401.

Fixed.


Handle vCenter connection expiry

If a command takes a lot of time to execute, the vCenter connection
can go stale and expire. Handle this by checking the validity of
the connection before each function call.

LGTM


asmjs-unknown-emscripten target, disabled

This target is failing.
https://github.com/cross-rs/cross/runs/5084819916?check_suite_focus=true
https://gist.github.com/Emilgardis/3283deb43cdab4fd6f21877022921736
Disabled for now.

https://github.com/cross-rs/cross/pull/609#issuecomment-1030889137

Partially fixed as of #840, depends on the upstream patch for https://github.com/rust-lang/rust/pull/98216.


supply PR to substrate for gov2 nonlinear curves

https://github.com/paritytech/substrate/blob/74a6370e805ebaf88b7939e496818531f762cadf/frame/referenda/src/types.rs#L251
Add "exponential" curve to `Curve type, using our lib
configure by:

start value at t=0
halving time (blocks?)

goal: bring substrate-fixed into substrate codebase for better support of all our fixed types

WIP: https://github.com/pifragile/substrate/pull/1

closing, hopeless to get upstream


const, let and var keywords no longer highlighted

Hi,
I ran PluginInstall to install some other plugins and I think yajs must have updated or something because several keywords are no longer highlighting at all.
Notice how const, let and var are not highlighted even though they are keywords.

Can't reproduce.
Maybe its caused by your theme color.

No further information

Yes. Sorry about that. This was due to my color scheme. You were right.


IL tests from TypeSystem tests are causing a warning in Visual Studio

The following warning is displayed in VisualStudio when compiling the TypeSystem.sln solution:
The referenced component 'ILTestAssembly\ILTestAssembly.ilproj' could not be found.	TypeSystem.Tests

Looks like Visual Studio has no idea about .ilproj file.

That's because the ilproj is not in the solution file. We could add it, but it won't build from VS anyway.
We would need to fix IL.targets to work in VS first. It lives in the dotnet/buildtools repo.

Submitted dotnet/buildtools#640 to address the first problem.


Error while trying to run demo.

Current Behaviour
After installation, I am starting server using command :
./../../system/sysbin/fb_dbcommands -s localhost:7509/MANAGER -load -f ../../../../addonlibs/administration/FBD/openAAS_Demo_17_10_20.fbd as mentioned in README.md.
I am getting error:
Fehler beim Laden der Datei '../../../../addonlibs/administration/FBD/openAAS_Demo_17_10_20.fbd' in Server 'localhost:7509/MANAGER'.
    Nr. 0x1 (KS_ERR_GENERIC)

Log
Some lines from openAAS_Demo_17_10_20.log:
ERR Instance /TechUnits/openAAS/AASFolder/Motor_M23.Body/LifeCycle.LifeCycleArchiv couldn't be updated.  Error : KS_ERR_GENERIC
     [ Variable 'identifier' not found ]
     [ Variable 'idH' not found ]
     [ Variable 'idL' not found ]
     [ Variable 'creationtime' not found ]
     [ Variable 'objectstate' not found ]

Detailed log here : https://gist.github.com/nikhilbchilwant/0e4bea118668cd937fa3a927ecdc8cd2
Environment
I am using Ubuntu 16.04 64 bit.
What is going wrong?

Hi,
i was at the plugfest on Jan. 29th. Now i wanted to try out the server. but i'm having a similiar issue like described above.
My error message:
Fehler beim Laden der Datei '../../../../addonlibs/administration/FBD/openAAS_Demo_17_10_20.fbd' in Server 'localhost:7509/MANAGER'.
Nr. 0x11 (KS_ERR_BADPATH)
Im running this on a 64Bit Ubuntu 16.04 machine.
Is there a way how to solve this?
Thanks
Ibrahim


revert: "fix: add a build.lock file to gate the buildengine (#1670)"

This reverts commit e711523cabbc597ea7acaf0434c9e89106adb263.
=== Failed
=== FAIL: buildengine TestDissallowMultipleEngines (0.00s)
    /home/runner/work/ftl/ftl/buildengine/engine_test.go:65: Did not expect an error but got:
        failed to lock file at: /home/runner/.ftl/build.lock

DONE 383 tests, 1 skipped, 1 failure in 115.533s
Error: === RUN   TestDissallowMultipleEngines
    buildengine/engine_test.go:65: Did not expect an error but got:
        failed to lock file at: /home/runner/.ftl/build.lock

cc @safeer
This is occurring because go test runs each packages tests in parallel, so multiple tests will be running concurrently and thus multiple build engines.

Fixed in #1737

Ohh nice. Thank you @alecthomas!


Albums from previously deleted songs show on the 'Albums' section.

I have songs that are no longer on my Apple Music, who's album title, with a blank cover show on the albums section. Furthermore, when I click on them, there are no tracks.

Thanks for the report. I'll see if I can reproduce this and submit this to Apple.


moment object struct is different between linux and windows

moment version: 2.29.1
moment object in window has "_a" and "_f" attributes, but there is not in linux version.
does it matter?

This surprises me, but I don't think it will affect the functionality.
Thanks for reporting this.


Minimize constraints to avoid ghc bug

Consensus tools tests stumbled upon the same GHC bug that was mentioned in this https://github.com/input-output-hk/cardano-ledger/pull/2950#issuecomment-1207298011
I went through all of the sites where bug could potentially be triggered and removed this possibility by removing redundant constraints.
There are no semantic changes in this PR, just expansion of class instances' constraints to the minimal possible set, but only for those instances that can trigger the bug.

Fixed that :joy_cat: rdano and squashed into one commit with a better message. Will merge as soon as PR goes green.


error: could not find native static library unwind_protect, perhaps an -L flag is missing?

Hi! I updated savvy and I am now getting this error. Do you happen to know what's the issue here?

Cargo check works fine btw.

with 0.6.4 it didn't work, so I tried with git version and it worked. I don't know what change made it work... Do you want me to close it anyway?

Thanks for reporting. I forgot in what situation this happens, but I think cargo clean would have saved you because this is something related to build cache and nothing was changed between 0.6.4 and the current GitHub version.

I did tons of cargo clean and it didn't work anyway :(
I will close it then. Thanks!

Please feel free to file a new issue if you find the reproducible steps, thanks!


BUG: RESHAPE op is not always compatible with the resize_tensor_input method of the TF Lite Interpreter

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): macOS Big Sur 11.2.3
TensorFlow installed from (source or binary): binary
TensorFlow version (or github SHA if from source): v2.4.0-49-g85c8b2a817f (2.4.1)

Provide the text output from tflite_convert
N/A
Description
RESHAPE op (as converted from tf.reshape + tf.shape functions) is not compatible with the resize_tensor_input method of the TF Lite Interpreter when the tflite model is obtained from the model with the known input shape (e.g., the known shape being (1, 1, 16000)). In this case the allocate_tensors method called after resize_tensor_input (e.g., with the new input shape being (1, 1, 32000)) results in the
RuntimeError: tensorflow/lite/kernels/reshape.cc:69 num_input_elements != num_output_elements (32000 != 16000)Node number 0 (RESHAPE) failed to prepare.

On the other hand, when the tflite model is obtained from the model with partially unknown input shape (e.g. (1, 1, None)) then resize_tensor_input works correctly.
Standalone code to reproduce the issue
Please see a Colab Notebook here for a more detailed presentation of the issue (the last cell in the notebook results in the RuntimeError mentioned above).
Any other info / logs
N/A

This is an intended behavior. If you want to resize input tensors without any issues, the corresponding TF model should have dynamic dimension support on the specific dimensions, which should be overided.


parser delegate non string to money

Why
Avoid parsing non string inputs
For now we want to avoid a scenario like 8% tax
Money.parse(1.66 * 0.8) #=> Money.new(1328), should be Money.new(1.33)

What

delegate anything passed to Money.parse that is not a string to Money.new

Change looks good to me but I don't follow the explanation: our core extension for the Numeric class already calls Money.new?
https://github.com/Shopify/money/blob/5df7243f39e2b1ae5f8b36cadc066e8c97e9df48/lib/money/core_extensions.rb#L4-L8
So this is an issue only when people call Money.parse(1.328)?

@bdewater yes, just updated the description :D


install/rpm: wipe all zypper repos before adding repos-under-test

Our tests should be designed to not rely on any pre-existing repos. If there are
any pre-existing repos, these can potentially skew test results by causing
unexpected packages/package versions to be installed.
Therefore, wipe any pre-existing repos before adding the repos-under-test.
Signed-off-by: Nathan Cutler<EMAIL_ADDRESS>

@susebot run deploy

@smithfarm I saw this code make the job failing with error like:
2019-11-11T14:38:53.957 INFO:teuthology.orchestra.run.target-libcloud-008:Running:
2019-11-11T14:38:53.957 INFO:teuthology.orchestra.run.target-libcloud-008:> sudo zypper repos -upEP && sudo rm -f /etc/zypp/repos.d/*
2019-11-11T14:38:54.142 INFO:teuthology.orchestra.run.target-libcloud-008.stdout:Warning: No repositories defined.
2019-11-11T14:38:54.143 INFO:teuthology.orchestra.run.target-libcloud-008.stdout:Use the 'zypper addrepo' command to add one or more repositories.
2019-11-11T14:38:54.187 DEBUG:teuthology.orchestra.run:got remote process result: 6```

@smithfarm could you review the #1348


Several problems noticed

I followed the webarena instructions.
The first thing I notice is the gitlab docker image on the AWI somehow went into some issue, I reloaded it again using the .tar file and it works fine. (Also, there is a space issue with the sql-related code in this part, but it's easy to fix)
Seems like the webarena file on the AWI is not the same version as this the one in this repo. I used this repo version.
The "context_length" field was declared in line 140 in openai_utils.py, but not between line 149-156. Also in line 41 - 48 of utils.py should add a line of "context_length=lm_config.gen_config["context_length"],".
Then I met an issue with the list of stop_tokens. I checked that the list always be empty, so I just comment out line 43 in lm_config.py, and according places.
For the suggested command to run :
python run.py
--instruction_path agent/prompts/jsons/new_action_prompt.json \ # this is the reasoning agent prompt we used in the paper
--model gpt-3.5-turbo
--mode completion
--observation_type html
--action_set_tag id_html_nasc_tree
--result_dir <your_result_dir>
--test_start_idx 0
--test_end_idx 1
notice that if anyone wants to use gpt-3.5-turbo, he/she should change the mode to chat instead of completion. If you want to use the completion choice, you need to change to model to gpt-3.5-instruct. (Details:)
I used gpt-3.5-instruct, but the output shows that the task failed. When I tried to use chat option, the program somehow crashed saying there is an unhandled error.
Did not know what excatly happened, hope someone could tell me in the future, and apologies if I made mistakes in my statements.

The dtails link should be https://stackoverflow.com/questions/75617865/openai-chat-completions-api-error-invalidrequesterror-unrecognized-request-ar


work in ios 13?

pls

I will check and update the library soon. Thanks for opening this issue.

I will check and update the library soon. Thanks for opening this issue.

big thx!!!


The output of K.dropout is not as expected!

For example:
x = numpy.array[[1,2,3],[4,5,6]]
x = K.variable(value = x, dtype = 'float64')
out = K.dropout(x, 0.4, seed=1334)
K.eval(out)

The results:
array([1.6666666, 3.3333332, 4.9999998], [0. , 8.33333 , 0. ])
But, in my opinion, the dropout operator only sets entries in x to zero at random,  the results should be like: array([1, 2, 3], [0. , 5 , 0. ])
why are other elements also modified?
If I want to implement the function like my description,  which method should I use? Any opinions or suggestions would be appreciated!

Hello,
Because Keras use inverse scaling for dropout.
See https://github.com/fchollet/keras/issues/3305
and https://github.com/fchollet/keras/issues/5357
for more information.

Thanks for your reply.


Fine Tunned Large model on Squad 2.0 using 8GB GPU

Thanks for share your implementation.
I did a fine tuning on Squad 2.0 using a 8 GB GPU and achieved great results even tough training partially the network.
Here's the repo: https://github.com/renatoviolin/electra-squad-8GB

Nice work!


Archive does not contain a directory named onlyoffice - ownCloud

When I'm trying to enable add-on in "Productivity" section, I see an info: "Archive does not contain a directory named onlyoffice". ownCloud version: 9.1.4.

Hello
Please note that ownCloud ONLYOFFICE integration app should be placed into the apps/onlyoffice
cd apps/ git clone https://github.com/ONLYOFFICE/onlyoffice-owncloud.git onlyoffice
You probably used other folder name or default folder "onlyoffice-owncloud" has been created (just rename the folder to "onlyoffice")

Hello @dafeer
In the release of the last version we took into consideration your remark on Tarbomb. The current .zip-file contains a subdirectory 'onlyoffice'.
Thank you for the interest in our product.

Thanks, it works. The installation via the ownCoud-internal app manager does not throw errors anymore.
Confirmed for (SHA1) 9943d825078f905d331005295d4c1cebc3418406  174798-onlyoffice.zip


Lapr/core removing

Proposed Changes
Remove Core from project. Fix UWP Update Viewport. Add fix to Sample.

What is the current behavior?
Cartography.Core is necessary for map to work.
What is the new behavior?
Cartography.Core is gone.
Checklist
Please check if your PR fulfills the following requirements:

[ ] Documentation has been added/updated
[ ] Automated Unit / Integration tests for the changes have been added/updated
[ ] Contains NO breaking changes
[ ] Updated the Changelog
[ ] Associated with an issue

Need to remove Cartography.Core from existing project.
Other information

Please separate removing of Core and fixes to the sample


Add other Git websites

Add other Git websites like Codeberg, Bitbucket, Sourcehut, Launchpad, NotABug, SourceForge

@m3thm4th how would you order these in terms of a priority?
Do you have links to sample projects in each to use for reference/testing?
I'm also happy to review PRs if anyone wants to add implementations themselves.

The project also has some coroutine based http wrappers to simplify async request code.

I don't know. I would probably see Codeberg as more important in that it's often preferred to GitHub or GitLab and the others as nice to have, I can't think of projects on any of them at the moment. I just like the idea of this app, I would like to have it on desktop too, if I remember about projects on any of them I will inform you.

@jroddev at the moment we struggle with adding the NonFreeNet anti-feature (F-Droid inclusion). If you could at least include self-hosted GitLab (just needs replacing the hostname), that would already help the decision against that.
I'd suggest we add your app with NonFreeNet for now and you drop us a note once a free service (self-hosted GitLab, or Codeberg/Gitea) is supported so we remove it again?

At the moment I use the hostname to determine whether to use GitHub or GitLab implementation.
If there's a way to auto-detect what is being used (through api call perhaps?) then self-hosted GitLab should be trivial to add.
At the moment only the url you provide is saved, everything else is calculated on the fly. I plan to save other fields in future to reduce the number of API calls and allow overriding when my detection logic fails. A dropdown to select the resolver (github, gitlab, etc, etc) could be added.

If there's a way to auto-detect what is being used

Not that I know of. But for starters, a passable logic could be: If it's neither github.com nor gitlab.com but carries the term gitlab in its host name, it's most likely a self-hostet GitLab (to my experience, about 90% of those self-hosted GitLabs have the term "gitlab" in their host name). Not perfect, but already an improvement.

At the moment only the url you provide is saved

In the future, you could also save the determined/specified service and, unless auto-detected, prompt the user for a list of services supported to choose from (don't allow to save if no working variant was found of course).

A dropdown to select the resolver (github, gitlab, etc, etc) could be added.

Excactly. And where absolutely clear (e.g. github.com, gitlab.com and codeberg.org/gitea.io) either have the proper service pre-selected or, if absolutely sure, just skip the popup.
That said: Do you prefer us to apply NonFreeNet now and merge the MR to have your app listed (removing NonFreeNet once we can) – or rather want us to wait until that anti-feature is no longer needed? IMHO the first step (identifying self-hosted GitLab by the term gitlab in its hostname and allow saving that when the API call succeeded) would be sufficient to not needing the anti-feature.

@IzzySoft sorry for the delay - life.
Now in 0.1.5 I am just making the assumption that if it doesn't contain the word "github" in the URL that it should use the GitLab resolver. Naive - yes, but gives us self-hosted GitLab support until I do some proper refactoring.
Looking for the phrase "gitlab" is the logic I had previously, but I have seen some self-hosted GitLab instances use git.<company>.com instead of explicitly mentioning GitLab.
before
if (repoUrl.contains("github")) {
  GitHub()
} else if (repoUrl.contains("gitlab")) {
  GitLab()
} else {
  null
}

after
if (repoUrl.contains("github")) {
  GitHub()
} else {
  GitLab()
}

sorry for the delay - life

No need to apologize. I just wanted to get the task done, and we can fix up anytime.

I am just making the assumption that if it doesn't contain the word "github" in the URL

I'd restrict that to the host name, not the full URL. Imagine I'd have a repo at https://gitlab.example.com/funnybear/github-faker, I'd totally fool your logic. Cannot fool it that easy if you restrict the filter to the host name (gee, yes, github.mydomain.com, but meh).
You could also consider making that a case statement, to be prepared for gitea/codeberg with a future version :wink: Apart from that: Yes, that was what I had in mind. Please let me know once that's up at F-Droid, and I'll take care for removing the AF (you could also watch the build monitor – once it's in there, the corresponding build section exists and I can edit/update the YAML so it's part of the next cycle already, if you want to speed it up a few days :see_no_evil:). Thanks a lot!

Maybe a solution could be to see if it matches a hostname and if it doesn't, or it's a custom one, give the user a multiple selection option

@m3thm4th that will definitely make sense as soon as support for Gitea is added. For now, the above approach should be fine: it's either github.com, gitlab.com or a self-hosted GitLab instance – all else is not yet supported. And there is no such thing as a self-hosted Github :wink:

Closing this issue for now. Once I have added the ability to override what is auto-detected then gitea, etc will become viable.


Added code to deleteCookies over jssh

I needed to clear the cookies when doing tests in Cucumber, so quickly hacked this XUL method to delete Cookies matching a regexp. I have absolutely no clue about ruby so forgive me if the code is terrible. The javascript can be extended to read out the list of cookies as well:
function listCookies() {
  var cookieManager = Components.classes["@mozilla.org/cookiemanager;1"].getService(Components.interfaces.nsICookieManager);
  var iter = cookieManager.enumerator;
  while (iter.hasMoreElements()) {
    var cookie = iter.getNext();
    if (cookie instanceof Components.interfaces.nsICookie){
      print("nsiCookie " + cookie.host + " " + cookie.name + " " + cookie.path);
      print("\n");
    }
  }
}

Closing this since FireWatir is deprecated and should not be used anymore.


Allow GlobalImageCache.clear to, optionally, only remove the actual data (PR 11912 follow-up)

When "Cleanup" is triggered, you obviously need to remove all globally cached data on both the main- and worker-threads.
However, the current the implementation of the GlobalImageCache.clear method also means that we lose all information about which images were cached and not just their data. This thus has the somewhat unfortunate side-effect of requiring images, which were previously known to be "global", to again having to reach NUM_PAGES_THRESHOLD before being cached again.
To avoid doing unnecessary parsing after "Cleanup", we can thus let GlobalImageCache.clear keep track of which images were cached while still removing their actual data. This should not have any significant impact on memory usage, since the only extra thing being kept is a RefSetCache (essentially an Object) with a couple of Sets containing only integers.

/botio test

Good idea!


Add unique ID for the execution of a challenge

For analysis it would be convenient to have an identifier (hash or UUID) that uniquely identifies the execution of a specific challenge added to all metrics records related to it. This makes it possible to easily drill in using a single parameter in Kibana and makes it easier to work with a central metrics store where multiple Rally instances concurrently write metrics to.

At the moment Rally uses the race timestamp for that. For each race we store the following information:
{
  "car": "4gheap",
  "target-hosts": [
    "some-host:9200"
  ],
  "selected-challenge": {
    "operations": [
      "index"
    ],
    "name": "append-no-conflicts"
  },
  "user-tag": "",
  "track": "percolator",
  "pipeline": "from-sources-skip-build",
  "environment": "nightly",
  "trial-timestamp": "20170209T075251Z",
  "revision": "f707132",
  "laps": 1,
  "distribution-version": "6.0.0-alpha1"
}

and we use the environment and the  trial-timestamp as the "natural" key usually (e.g. when identifying races in tournament mode). Is this sufficient or do you think we need an technical id in addition?

In order to use it efficiently in Kibana visualisations I have usually created it as a not_analyzed/keyword field. Am not sure how easy it would be to use a date field in the same way. I therefore think a UUID might be a good addition.

Sounds reasonable. But I think we should have only one user-facing key for a race (e.g. for tournaments). Ideally, we will also not present the full-length UUID to the user but rather shorten it (similar to what git does).
So, while the actual change is pretty easy and isolated, I think we should go the extra-mile and make this consistent across the codebase.

I like the idea around the shortened UUID. I think this would be more useful than the starting time of the benchmark.

@cdahlqvist The trial-timestamp('s) can be hardcoded into a search or with a boolean AND query, Or left out of the search and used in the visualization on the x axis as a terms aggregation.
Personally I think adding a UUID does not add much. A string describing the particular trial would be more useful. At the moment I name my visualizations in sequence , matching the sequence of dates in the legend. Which obviously cant work when using the field to fetch the X most recent trials.. (Although it sounds like I can do this by changing the environment each trial, some users might want to retain env to its specific meaning)


Overload wrapper

Adds the possibility to wrap calls to an overloaded function with a wrapper function, which receives the method to dispatch to as its first argument (or None, if there is no method to dispatch to).
Additionally, if the initial_state function is provided, bootstrap=True is implied and top level invocations will initialize a state. The state will be kept through recursive invocations through self but each top level invocation creates a new state. This allows things like keeping track of seen objects.
Overload also now automatically receives a name based on the first function or wrapper to be registered to it.

It doesn't appear to forward the docstring from what I can see.


modify 'UPDATE DATASOURCE' to preserve existing metadata

I modified the implementation of 'UPDATE DATASOURCE'.
Previous version has bugs that existing metadata (e.g., comment, category) would disappear after 'update datasource'.

You're right.
Current logic has some limitations.
Actually, I think the problems you mentioned are no big deal.
Only the custom metadata (description, category) would be lost, when the object name is changed.
I think the change of the object name (or type) means the previous custom metadata become obsolete.


fix: add StreamOutput deps

Cannot find trait StreamOutput in this scope [E0405]

Oh thank you but I am still working on it, cannot promise it functional well untill first version published😓.

I just published the first version and it should work now.


Add tests for sf::VideoMode

Description
I added tests for sf::VideoMode. I was unsure how to test sf::VideoMode::getDesktopMode, sf::VideoMode::getFullscreenMode, and sf::VideoMode::isValid in a way that would work on all machines including the presumably headless CI runners. The test-sfml-window executable has always been compiled but until now didn't actually include any test assertions so now this test executable will provide some meaningful feedback.
Tasks

[ ] Tested on Linux
[ ] Tested on Windows
[x] Tested on macOS
[ ] Tested on iOS
[ ] Tested on Android

SFMLBuildMaster: Build this please.

SFMLBuildMaster: Build this please.

By the way, the comparison logic can be vastly simplified with std::tie like this:
https://github.com/ChrisThrasher/SFML/commit/253ba16d2de0e84994518ca107adb62ece548070
Either way the logic is still correct but this pattern is a lot easier to get right if you're writing operator< from scratch.

SFMLBuildMaster: Build this please.

Any other comments on this? @vittorioromeo @Bromeon @eXpl0it3r


refactor ./go to go/krmfn go/api and go/cmd

Move KRM function code to go/fn
Keep api under go/api, but using a new module github.com/GoogleContainerTools/kpt-functions-sdk/go/api
Separate Makefile for go/api and and go/fn
Update github CI for go-api (TODO: add a CI for go/fn)

Separate Makefile for go/api and and go/fn

Why are we doing this? If the problem is things don't work with multiple modules. Maybe we can do something like https://github.com/GoogleContainerTools/kpt/blob/343b21c254a7a064488c5fce226a6fbcc8c02643/porch/Makefile#L98?

Why are we doing this? If the problem is things don't work with multiple modules. Maybe we can do something like GoogleContainerTools/kpt@343b21c/porch/Makefile#L98?

not for "things don't work with multiple modules". the API, fn and CMD (scaffolding CLI) have independent goals and use cases. User who need the kptfile schema does not need to import the fn and vice versa.

not for "things don't work with multiple modules". the API, fn and CMD (scaffolding CLI) have independent goals and use cases. User who need the kptfile schema does not need to import the fn and vice versa.

I meant we can still use one MAKEFILE even though we have multiple go modules. Using one single MAKEFILE for golang may be more convenient for users.

not for "things don't work with multiple modules". the API, fn and CMD (scaffolding CLI) have independent goals and use cases. User who need the kptfile schema does not need to import the fn and vice versa.

I meant we can still use one MAKEFILE even though we have multiple go modules. Using one single MAKEFILE for golang may be more convenient for users.

I see. works for me. Thanks!


WebDAV server

Would it be possible to support WebDAV servers?
This would open up more possibilities for many clouds.
Great application. congratulations

More cloud backend support is planned at the very beginning. The reason why Nextcloud now is simply because I use it as my home server, easy for me to do testing. But as you already know, WebDAV is a standard, backend communication code is reusable, just need to figure out UI flow.
Still tied up at camera roll management feature which is schedule in release 1.4. Will tackle this later. Thank you for using this app.

More cloud backend support is planned at the very beginning. The reason why Nextcloud now is simply because I use it as my home server, easy for me to do testing. But as you already know, WebDAV is a standard, backend communication code is reusable, just need to figure out UI flow.
Still tied up at camera roll management feature which is schedule in release 1.4. Will tackle this later. Thank you for using this app.

Any update about the implementation of the webdav protocol?
This would probably help us solve the Kdrive connection issue 45.

I had this in mind ever since the 1st day. but after more than a year's work, i realise it's not that simple. nextcloud (or i should say owncloud) has a well documented API for file syncing with client, and les pas rely heavily on that to make things work. i doubt others (by that i mean other free hosting software and providers)would have the same, or near the same mechanism, though i also haven't done any serious research on them.


XR simulated mouse input

Simulate a mouse using right controller.
Uses "Khronos Simple Controller Profile", which should be available on most devices.
Controller input is interpreted as follows:

/user/hand/right/input/select/click as left button
/user/hand/right/input/aim/pose as cursor using ray cast

Thank you Peter!


Remove unit files created in /usr/lib/systemd/system by previous cookbook versions

Description
Remove unit files created in /usr/lib/systemd/system by previous versions of this cookbook.
Issues Resolved
[List any existing issues this PR resolves]
Check List

[x] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[x] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[x] The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

My comment is just a clarification question, nothing that should keep you from merging this, I suppose.


Adnane preferences

Description

Designed the preferences choice on sign up, using a pop up window.
Tests
Toggles and ensuring correct API calls are fully tested.
MISSING TEST: Loading data from storageSession. Will open up a different PR.
UI Changes
You can access the preferences at /preference:

Pop up:

Issues
#66 #68 #47

Additional Notes
Add any other information about the PR here.

@aezouhri please make your pr name more descriptive of what was done

@aezouhri please make your pr name more descriptive of what was done

Just updated it. Thanks for the reminder @saanbe16


Point-and-click deletion of Lofts and Offset Planes

From https://github.com/KittyCAD/modeling-app/pull/4605#pullrequestreview-2479827749 @franknoirot

@Irev-Dev whenever you have a bit of time I bet you could help me or @pierremtb understand how to add new deletable entities pretty quick and knock both these out in one session

And I guess second question is: do we want to wait for the Operations Timeline to be available to Right-click > Delete things like Lofts and Shells? Instead of extra buttons?

Ok this workflow is new to me! No idea that del or backspace was triggering a selection-based delete, was looking for a button haha! Cool that makes sense as interim.
I guess we first need to make the selection highlight work on lofts and shells? Cause it's not working now


Added v22.1.1 anchor link target

v22.1.1 is live today, adding direct anchor link to release notes.

This change is


Use preconditions and aspects.

Now that Ada 2012 compilers are more mature and widespread we should make use of at least some of its  features, in particular aspects (especially import one as it would cut down on some code in bodies) and preconditions (to prevent one from supplying incorrect enums into subprograms).

Aspectization of Import and Inline pragmas is finished.


Error pulling latest changes for branch: main: Not possible to fast-forward, aborting.

There should be an automated reaction to this one (may be reclone or another fix).

Reclone with backup, or stash with reset.


Disable proxy caching of bookshelf

TODO: determine whether and how to cache bookshelf responses.
HTML and JSON responses were being cached under the same
proxy_cache_key, leading to JSON being returned for the main bookshelf
page.  If caching is enabled, it should use the suggested
proxy_cache_key value to ensure that JSON and HTML do not get mixed up.
This is related to the frontend's design decision to have the same
resource return not only different representations given different
Accept: content types, but different objects (data), as well.

:100:


Checks fail with "timed out waiting for pod to exit"

Describe the bug
One of my custom checks keeps failing with "timed out waiting for pod to exit". I can see that the pod for the check comes up and the logs show that the check was able to run correctly.
Steps To Reproduce

Running a custom check. Note that I have around 7 custom checks running and this happens with only one of them.

Expected behavior
Check should pass.
Screenshots
NA
Versions

Kubernetes Version: 1.17
Env : GKE version 1.17.17-gke.4900
Kuberhealthy Release v2.4.0

Interesting.  Did the pod fully shut down and transition into Error or Completed state?  Can you get the logs off the pod after it shuts down with kubectl logs <podName> --previous?

Could this have to do with #903?  Essentially in that error, pods created in namespaces outside of the Kuberhealthy namespace are automatically deleted.

Interesting. Did the pod fully shut down and transition into Error or Completed state? Can you get the logs off the pod after it shuts down with kubectl logs <podName> --previous?

Thanks. I see the pod is in the Running state even after the pod logs show the test is done and dusted. I suspect this might be a problem with the check's code which leaves non-daemon threads hanging around. I'll check and get back.

Could this have to do with #903? Essentially in that error, pods created in namespaces outside of the Kuberhealthy namespace are automatically deleted.

I don't think so - as we deploy all checks in the kuberhealthy namespace.

I checked our code - and it looks like there was a thread pool with non-daemon threads lying around. Shutting it down took care of it. Thank you.


portus_web cannot start with volume .:/portus

Hi,
i wanna start Portus with ./compose-setup.sh, but it failed. And i found the portus_web container exited with error. Then i remove the volume in docker-compose.yml and it worked.
Another question: since we have add the total dir into docker image on building, why the volume is needed on running?

i wanna start Portus with ./compose-setup.sh, but it failed. And i found the portus_web container exited with error. Then i remove the volume in docker-compose.yml and it worked.

Please provide more details about the failure and your host system. We cannot help you otherwise.

Another question: since we have add the total dir into docker image on building, why the volume is needed on running?

The compose setup is intended to be used to play with the latest version of Portus or even develop it. Hence we want the latest code to be used by the container.

@flavio ,thanks...sorry but i finally found selinux making that trouble on my new PC, and i've fixed it...


feature: query lineage visualizer advanced styling & interactive component handling

Issue #, if available:
Description of changes:

advanced styling including color by lineage entity, star shape startarn nodes and legend section
callback events on selecting nodes which nodes will be highlighted on click

Testing done:
tox -e black-format & tox -e flake8,pylint,twine,black-check,docstyle --parallel all
tox -e py39 tests/unit/sagemaker/lineage
tox -e py39 tests/integ/sagemaker/lineage
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[x] I have read the CONTRIBUTING doc
[x] I certify that the changes I am introducing will be backward compatible, and I have discussed concerns about this, if any, with the Python SDK team
[x] I used the commit message format described in CONTRIBUTING
[ ] I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
[ ] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have added unit and/or integration tests as appropriate to ensure backward compatibility of the changes
[ ] I have checked that my tests are not configured for a specific region or account (if appropriate)
[ ] I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The graph looks nice!


ParameterDefaultReader in swagger does not honor other annotations usage of DefaultValue

When annotations other then ApiParam are used, the defaultValue set by them is not honored.
If an annotation such as Springs @RequestParam is used with a defaultValue set and @APIParam is not used, the current implementation will override the defaultValue of the parameterBuilder to  empty string.
Note: springfox.documentation.spring.web.readers.parameter.ParameterDefaultReader correctly evaluates the defaultValue of RequestParam and correctly assigns the defaultValue (not overriding previous values set if the annotation doesn't exist).

hi @dilipkrish,
I am unsure if those tests were not working properly, but I was able to make a modification to ParameterBuilder#defaultValue that works with those 1.2 annotation tests.

@ctruzzi I thought about doing that fix that you did, but then what if we had the case where we want the default value to be a blank string?

I believe it would work properly as long as you do not have a conflicting annotation, since the initial value is set to null, as long as that value does not get set to anything prior to defaultValue = "" it would assign to that properly in this conditional argument.
However if we had something like
@RequestParam(defaultValue="value")
@APIParam(defaultValue="")
The new code revision would keep defaultValue assigned to value. I am fine with either code modification, and the first one makes more sense if indeed those tests were doing incorrect checks.

Philosophically the way it works is that we layer information on top of the other. The layer happens based on priority.

Spring Web annotations are applied first
Bean Validation annotations go next (recently introduced)
Swagger annotations are applied last

So the way I see it playing out is that if someone puts in the swagger annotation that overwrites the spring default value, then it will be apparent where the problem is. They would need to set the @ApiParam#defaultValue to a non-default value. Let me know if that makes sense.
This way there is no conflicting annotations. Its always predictable and there is not conditional code to handle just this particular edge case. Hope that makes sense

Since there is that defined priority, I would agree that the initial pull request would be the most correct one to keep the code simple and working properly.
Is there an ETA for 2.4 milestone? (Currently using 2.0.1 and require some changes from 2.0.2 but want a single upgrade time).

Its set for  March 1st, but I want to get as many features as possible by then.


(699) Update view page

This updates the view page to make some text tweaks, as well as allowing the schedule to be edited.
Mural

Code

Trello card: https://trello.com/c/nYOtTtr2/699-update-view-page

@Harriethw I might just bin off the edit schedule stuff from this PR and do it seperately. We should probably make this a more deliberate act


Android USB permissions

This fixes #58.
Android intent-filters are added to the plugin.xml to allow the application to be listed as an option for opening when a Star printer is connected via USB.
In the event permissions are not granted to the device by the use of the intent-filters, when an action is executed we loop through connected devices and request permissions if a Star printer we don't have permission for is found. Implemented the check and request permission here since permissions need to be re-requested each time the printer is plugged back in, so don't want to be surprised by a lack of permissions at any time.

Replaced with #62 which supersedes this one by handling bluetooth permissions for Android 12 as well.


[fix](group commit) Fix the timeout to wait wal finished when schema change

Proposed changes

modify regression case to add some schema change;

when schema change, there is a dead lock:

the schema change job hold table write lock, and wait for BE wal num is 0
the group_commit commit failed because can not get table write lock
the replay wal failed because can not get table read lock to generate a plan fragment

so move the wait wal out of the table write lock.

If BE encounter memory hard limit, the load will stuck until the hard limit is ended. This may lead to the commit time interval of group commit is much larger than users' config, so improve the check timely.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run p0


Scalar arrays in UBOs generate the wrong array stride

kernel void foo(global int *out, constant int *in) {
  *out = *in;
}

When -constant-args-ubo is enabled the resulting UBO has an ArrayStride of 4. This is invalid. It must be:
Any ArrayStride or MatrixStride decoration must be an integer multiple of the base alignment of the array or matrix from above.

This means a multiple of 16 bytes is necessary.

Validator does catch this.

This was fixed by https://github.com/google/clspv/commit/3d9e20152fea932e7d102d8b2872a3f542caa00f.


Need Attribution-NonCommercial-ShareAlike 4.0 International

Attribution-NonCommercial-ShareAlike 4.0 International (CC BY-NC-SA 4.0)
I'm very sorry. I need Attribution-NonCommercial-ShareAlike 4.0 International on my Github.
I hope you can add it to your website. I think you can do it.
Many thanks.

Probably a duplicate of https://github.com/github/choosealicense.com/issues/481, but if you want a license added https://github.com/github/choosealicense.com/blob/gh-pages/CONTRIBUTING.md#adding-a-license

No need to apologize. Out of scope for this project per the link Nick posted, but you can use whatever license you want, as you've done https://github.com/TangelScratchStudio/TangelStudio-OfficialWeb/blob/57524dc5734990203c2f08bebde8b42551d3c2cc/LICENSE (though note you've specified a different CC license in that project's README; I'd guess you'd want those to match).


net: tcp: Revert notify TX window avaialbility

Due to simulations merge of a few PRs, the main is broken. Suggest for the time being revert the PR https://github.com/zephyrproject-rtos/zephyr/pull/45626
After sorting the issues out the PR should be re-submitted.

@rlubos @ssharks
You are referring to reverting incorrect behavior coming from https://github.com/zephyrproject-rtos/zephyr/commit/c668199b5d09add7ef2ed83233828d7ac733c07e, but it is still in Zephyr 3.2
Could you please clarify what kind of behavior is that?
Should it still be reverted?

@AndreyDodonov-EH Neither of these commits was wrong. They came from different PRs and happened to break each other after being merged. I believe this has been fixed by https://github.com/zephyrproject-rtos/zephyr/pull/45890/commits.
For that reason this PR is obsolete.


AST Node has no document

When working on a new grammar / language, I see the issue that in Linking and other situations, Langium fails with
Error: AST node has no document.
at getDocument (c:\Users\agraf.vscode\extensions\undefined_publisher.XXXXX-0.0.1\out\language\main.cjs:10601:11)
Since in @cdietrich 's report on the formatter he mentions "the" AST node has no document bug, I assume it is already known. Is there further information on that bug?
Regards,
Andreas

Hey @grafandreas,
can you post a reproduction example? I've seen some complaints about it, but failed to reproduce the issue as of now.

i have encounter similar issue by override getDocumentHighlight method in DefaultDocumentHighlightProvider

@zrwsmd Can you reproduce this consistently? Can you share a reproducible example?

Probably this is not the reason but I ran into this error when accidentally using ?= to assign a value to a property of type boolean combined with a rule.
In this example (which might not be very reasonable) :
entry Model:
    persons+=Person*;

Person:
    'person' name=ID isXYZ?=PersonPredicate;

PersonPredicate:
    'knows' other=[Person:ID];

Using this input:
person Foo knows Bar
person Bar knows Foo

The error AST node has no document. occurs (related to the cross references after knows).
This might be just because of an incorrect definition but maybe this can serve as a litte hint.