nixos/krb5: Cleanup, fix and RFC42-ify

Description of changes

This replaces the krb5 module's options with RFC 42-style krb5.settings option*, while greatly simplifying the code and fixing a few bugs, namely:

#243068
not being able to use mkIf etc. inside subattributes of krb5.libdefaults, e.g. krb5.libdefaults.default_realm = mkIf ...

* leaving an escape hatch in form of krb5.extraConfig in case extra syntax is needed (e.g. `include' directives)
See #144575.
Closes #243068.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (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.

Can you rebase?

Done!

Actually, since this has incompatible changes anyway, might it be a good idea to move this module under security (like security.ipa)?

Actually, since this has incompatible changes anyway, might it be a good idea to move this module under security (like security.ipa)?

sure, why not

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2871

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3036

@2xsaiko I wrote earlier that moving the format generation to it's own file wasn't really necessary for this PR, but after having looked deeper into the manpages, I think we should do it anyway.
I have made a bunch of changes in https://github.com/h7x4/nixpkgs/commit/88ebceb9f6054b236ec20e9079817ef2950adb5f (patch), some of which should probably be rebased into the existing commits. Feel free to leave comments on the commit if you're wondering about something.
I've removed extraConfig as well, but it's removal is not mutually exclusive with the other changes if you still want to keep it due to the reasons you listed earlier. I just don't want to have to do breaking changes to the options in a later commit to add support for modules and includes, since we're already doing breaking changes here. That being said, I'm pretty confident that this covers all features in both the MIT and Heimdal implementations, and that the format won't be changing anytime soon considering how old and stable these programs are.
Hope you don't mind
To run tests:

nix-build -A nixosTests.kerberos.mit
nix-build -A nixosTests.kerberos.heimdal (nvm, broken due to some outdated crypto defaults)
nix-build -A nixosTests.krb5.example-config

@2xsaiko I wrote earlier that moving the format generation to it's own file wasn't really necessary for this PR, but after having looked deeper into the manpages, I think we should do it anyway.

This looks really good! I think this is definitely worth doing, especially since additionally security.ipa also uses this format.

Moved all includes and modules to the top of the config file, as specified in https://linux.die.net/man/5/krb5.conf

It just says they must occur at the beginning of a line, as far as I can see. And I feel like I've seen example files where there was an include at the bottom when I originally wrote this. Not sure if it makes a difference either way though where you put them.
Also, what do you think about removing the inner attrset for the actual config and instead using freeformType in the submodule to flatten the structure? I feel like these are going to be very rarely used so I kinda don't like having the additional nested set just for these.
As far as I know there's no [modules] section or any of the others, so it wouldn't conflict.
let
  section = attrsOf ...;
in submodule {
  freeformType = section;
  options = {
    includes = mkOption { ... };
    includedirs = mkOption { ... };
    modules = mkOption { ... };
  };
}

I didn't know about this when I wrote this module, maybe I would have then used it in the first place to add includes and modules support :^)

[...] security.ipa also uses this format

Huh, I didn't know that. If other related security programs are reusing this format as well, maybe it actually would make sense to put it into pkgs-lib?

It just says they must occur at the beginning of a line, as far as I can see

Right, I think I misread it. But it also says "Included profile files are syntactically independent of their parents, so each included file must begin with a section header.", so I guess it doesn't really matter?

what do you think about removing the inner attrset for the actual config and instead using freeformType in the submodule to flatten the structure?

Sounds great!

Personally usually I'd also put it there because it really doesn't hurt to have it and it's like 5 extra lines which does a great deal for external extensibility of the module. I've been hit by it missing a couple times already...
But I feel like now with the addition of include being properly supported (thanks @h7x4!) that should cover all use cases of extraConfig while being way less sketchy (in terms of merge order potentially changing the meaning of the file contents).

Feel free to mark my comments as resolved

Result of nixpkgs-review pr 273807 run on x86_64-linux 1

7 packages built:

citrix_workspace (citrix_workspace_23_09_0)
citrix_workspace_23_02_0
citrix_workspace_23_07_0
heimdal
heimdal.dev
heimdal.info
heimdal.man

@ofborg build kerberos.mit kerberos.heimdal nixosTests.krb5.example-config pam-file-contents nfs4.kerberos

...no --keep-going?
@ofborg test kerberos.mit krb5.example-config pam-file-contents nfs4.kerberos

Also built the manual, lgtm

Thanks @pbsds, @2xsaiko, and @h7x4!


SqlServerStringReplaceTranslator fails with Sequence contains more than one matching element on the latest runtime

Running:
https://github.com/aspnet/SignalR/tree/dev/samples/ChatSample
fails with the following exception:
info: Microsoft.EntityFrameworkCore.Storage.IRelationalCommandBuilderFactory[1]
      Executed DbCommand (103ms) [Parameters=[@__get_Item_0='?' (Size = 450)], CommandType='Text', CommandTimeout='30']
      SELECT TOP(1) [e].[Id], [e].[AccessFailedCount], [e].[ConcurrencyStamp], [e].[Email], [e].[EmailConfirmed], [e].[LockoutEnabled], [e].[LockoutEnd], [e].[NormalizedEmail], [e].[NormalizedUserName], [e].[PasswordHash], [e].[PhoneNumber], [e].[PhoneNumberConfirmed], [e].[SecurityStamp], [e].[TwoFactorEnabled], [e].[UserName]
      FROM [AspNetUsers] AS [e]
      WHERE [e].[Id] = @__get_Item_0
fail: Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware[0]
      An unhandled exception has occurred: The type initializer for 'Microsoft.EntityFrameworkCore.Query.ExpressionTranslators.Internal.SqlServerStringReplaceTranslator' threw an exception.
System.TypeInitializationException: The type initializer for 'Microsoft.EntityFrameworkCore.Query.ExpressionTranslators.Internal.SqlServerStringReplaceTranslator' threw an exception. ---> System.InvalidOperationException: Sequence contains more than one matching element
   at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source, Func`2 predicate)
   at Microsoft.EntityFrameworkCore.Query.ExpressionTranslators.Internal.SqlServerStringReplaceTranslator..cctor()
   --- End of inner exception stack trace ---
   at Microsoft.EntityFrameworkCore.Query.ExpressionTranslators.Internal.SqlServerStringReplaceTranslator.Translate(MethodCallExpression methodCallExpression)
   at System.Linq.Enumerable.SelectListIterator`2.MoveNext()
   at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable`1 source, Func`2 predicate, Boolean& found)
   at Microsoft.EntityFrameworkCore.Query.ExpressionTranslators.RelationalCompositeMethodCallTranslator.Translate(MethodCallExpression methodCallExpression)
   at Microsoft.EntityFrameworkCore.Query.ExpressionVisitors.SqlTranslatingExpressionVisitor.VisitMethodCall(MethodCallExpression methodCallExpression)
   at System.Linq.Expressions.MethodCallExpression.Accept(ExpressionVisitor visitor)
   at Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor.VisitWhereClause(WhereClause whereClause, QueryModel queryModel, Int32 index)
   at Remotion.Linq.QueryModelVisitorBase.VisitBodyClauses(ObservableCollection`1 bodyClauses, QueryModel queryModel)
   at Remotion.Linq.QueryModelVisitorBase.VisitQueryModel(QueryModel queryModel)
   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.VisitQueryModel(QueryModel queryModel)
   at Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor.VisitQueryModel(QueryModel queryModel)
   at Microsoft.EntityFrameworkCore.Query.Internal.SqlServerQueryModelVisitor.VisitQueryModel(QueryModel queryModel)
   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.CreateAsyncQueryExecutor[TResult](QueryModel queryModel)
   at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQueryCore[TFunc](Object cacheKey, Func`1 compiler)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.ExecuteAsync[TResult](Expression query)
   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryable`1.System.Collections.Generic.IAsyncEnumerable<TResult>.GetEnumerator()
   at System.Linq.AsyncEnumerable.<Aggregate_>d__6`3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Identity.EntityFrameworkCore.UserStore`9.<GetRolesAsync>d__55.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Identity.UserManager`1.<GetRolesAsync>d__109.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Identity.UserClaimsPrincipalFactory`2.<CreateAsync>d__13.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Identity.SignInManager`1.<CreateUserPrincipalAsync>d__24.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Identity.SecurityStampValidator`1.<ValidateAsync>d__3.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.Cookies.CookieAuthenticationHandler.<HandleAuthenticateAsync>d__12.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.<InitializeAsync>d__52.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware`1.<Invoke>d__18.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()
fail: Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware[0]
      An exception was thrown attempting to execute the error handler.

Runtime version: 2.0.0-beta-001846-00

How can I get copy of latest runtime? It seems that there are multiple overloads of string.Replace now. We need to find the exact one which we can translate.

@smitpatel - I just git clean -xdf the repo and ran build.cmd

@smitpatel to look at this as a priority since it is blocking music store.

Is there a workout for this, I can't use my app atm

@harisbotic Avoid using the latest runtime?

won't help, I have latest version and one before that and neither works.
so its not only in the latest runtime, and I can't find a place to download older ones.
The only link I know is this one
https://github.com/dotnet/core-setup#daily-builds

@harisbotic I meant; Just use the release version from NuGet!

...
I though it had to be 2.0.0-* runtime because its netcoreapp2.0
It works with 1.1.1, thank you very much @ErikEJ


Error during export

Interface of Notion's share form doesn't allow to choose Integration as client.
So I create Global Web Share Link for Root page and specify it
https://rain-puppy-6dd.notion.site/Root-Page-ID
Script output:
Loading access token from keyring
Logging into Google Keep
Password:
Authenticating, this may take a while...
Authentication is successful, saving token in keyring
Token saved. Have fun with other commands!
Logging into Notion
Traceback (most recent call last):
  File "C:\Users\Sergey\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\notion_client\client.py", line 116, in _parse_response
    response.raise_for_status()
  File "C:\Users\Sergey\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\httpx\_models.py", line 749, in raise_for_status
    raise HTTPStatusError(message, request=request, response=self)
httpx.HTTPStatusError: Client error '400 Bad Request' for url 'https://api.notion.com/v1/pages'
For more information check: https://httpstatuses.com/400

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\Sergey\Downloads\gkeep2notion-main\gkeep2notion-main\gkeep2notion.py", line 372, in <module>
    create_page(notion, notes)
  File "C:\Users\Sergey\Downloads\gkeep2notion-main\gkeep2notion-main\gkeep2notion.py", line 168, in create_page
    notion_page = notion.pages.create(parent=page.parent,
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Sergey\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\notion_client\api_endpoints.py", line 197, in create
    return self.parent.request(
           ^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Sergey\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\notion_client\client.py", line 192, in request
    return self._parse_response(response)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\Sergey\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\notion_client\client.py", line 124, in _parse_response
    raise APIResponseError(response, body["message"], code)
notion_client.errors.APIResponseError: body failed validation. Fix one:
body.parent.database_id should be defined, instead was `undefined`.
body.parent.page_id should be a valid uuid, instead was `""`.

Seems like Notion has changed sharing and integrations. Could you check if the new instructions from https://github.com/trustmaster/gkeep2notion/pull/14 solve your issue?

Now that I look at this ticket again, the problem was in the root page URL: https://rain-puppy-6dd.notion.site/Root-Page-ID. This tool currently doesn't support custom domains, it needs a URL from https://notion.so to work. A workaround is to translate your public site link into a regular notion.so link, as the only part needed is the ID of the page.


Config for zoom level on high DPI systems

I could not find a zoom level setting so that high DPI machines could effectively use the app. Is there a plan to add this feature?

`
// Create the browser window and disable node.js (it is needed to work with pre-compiled js of external url)

mainWindow = new BrowserWindow({width: 900,

                             height: 600,

                             webPreferences: {nodeIntegration: false},

                             show: false

mainWindow = new BrowserWindow({width: 1800,

                             height: 900,

                             webPreferences: {zoomFactor: 1.5, nodeIntegration: false},

                             show: false,
                            })

`
Reading the zoomFactor value from the config file would probably solve this issue.


RCTShadowView: use 1x alignment

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[x] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary
Passing 0 to YGConfigSetPointScaleFactor() turns off layout rounding entirely, which makes things look blurry.  While it would be nice to use the backing scale factor of the window, that would be a very involved change.  Instead, hardcode 1x alignment.  On 2x displays, this just means that views cannot sit on half-point boundaries.  (They're still rendered at 2x, though.)
Changelog
[macOS] [Fixed] - Use 1x frame alignment
Test Plan
Ran a React Native demo project on macOS 10.15.7 / MacBookAir6,2.
Before:

After:

With #1069 merged, is this still an issue @mhjacobson ?

/azp run


uv pip compile: resolves to unnecessarily old versions of yamlpath and ruamel.yaml

Hello, and thanks!
This is on Ubuntu 22.04.
requirements.in:
cattrs
nestedtext
plumbum
rich
ruamel.yaml
yamlpath

$ uv --version
uv 0.1.31
$ uv pip compile --no-header --annotation-style=line -U requirements.in -o uv.txt
$ pip-compile --version
pip-compile, version 7.4.1
$ pip-compile --no-header --annotation-style=line -U requirements.in -o pip-tools.txt
$ diff -u pip-tools.txt uv.txt

--- pip-tools.txt	2024-04-17 01:24:37.815365304 -0400
+++ uv.txt	2024-04-17 01:24:23.219134399 -0400
@@ -1,15 +1,16 @@
 arrow==1.3.0              # via inform
 attrs==23.2.0             # via cattrs
-cattrs==23.2.3            # via -r requirements.in
+cattrs==23.2.3
 inform==1.28              # via nestedtext
 markdown-it-py==3.0.0     # via rich
 mdurl==0.1.2              # via markdown-it-py
-nestedtext==3.6           # via -r requirements.in
-plumbum==1.8.2            # via -r requirements.in
+nestedtext==3.6
+plumbum==1.8.2
 pygments==2.17.2          # via rich
-python-dateutil==2.9.0.post0  # via arrow, yamlpath
-rich==13.7.1              # via -r requirements.in
-ruamel-yaml==0.17.21      # via -r requirements.in, yamlpath
+python-dateutil==2.9.0.post0  # via arrow
+rich==13.7.1
+ruamel-yaml==0.18.6       # via yamlpath
+ruamel-yaml-clib==0.2.8   # via ruamel-yaml
 six==1.16.0               # via inform, python-dateutil
 types-python-dateutil==<IP_ADDRESS>40316  # via arrow
-yamlpath==3.8.2           # via -r requirements.in
+yamlpath==3.4.0

I tried to put the verbose output here in a fold, but GitHub complained that the message body was too long.

Oh whoops, I misread the output and didn't realize ruamel.yaml was newer instead of older. Without looking right now, probably newer yamlpath versions hold back the ruamel.yaml dependency.
So this all makes sense and when I'm back at a computer I'll force a high minimum version for yamlpath and all should be good.
Thanks!

Yup, it's all cleared up now, thanks again.

No problem! Glad to help :)


Doomfist uppercut and slam keybind swapped

Our implementation of doomfist has slam bound on "shift" and uppercut bound on "e", but in OW1 the keybinds are the opposite. This is due to the fact the OW2 binds slam to "shift" by default.
The keybinds for two abilities will be swapped if possible in the future.

Fixed in Doomfist v0


potential todo list (shortcomings in bash introspection, portability problems, other project, semantic model)

I have a more or less complete list of essential introspection, which bash/POSIX shells are lacking and which are essential to get some form of weak coupling on executing scripts:
https://github.com/matu3ba/dotfiles_skeleton/blob/main/POSIXunsafe
You can not in a POSIX way reliably and sanely short

check if symbols is defined in a script.
https://stackoverflow.com/questions/18597697/posix-compliant-way-to-scope-variables-to-a-function-in-a-shell-script
detect if a script is being sourced
https://stackoverflow.com/questions/2683279/how-to-detect-if-a-script-is-being-sourced
get the current directory of the script.
https://stackoverflow.com/questions/29832037/how-to-get-script-directory-in-posix-sh
how to find files in PATH (standard is ambiguous how it should work)
https://unix.stackexchange.com/questions/85249/why-not-use-which-what-to-use-then

Please also be aware, that symlinks work differently on Windows and Linux, which make scripts utilizing them unportable.
(platform support is not yet documented)
Sidenode: Please use fn or fun instead of function to enable faster script writing.
It would also help visitors to understand the scope of the project, if you compare to other scripting projects intended as bash execution replacement:

https://skarnet.org/software/execline/ from skarnet.org (also known for the s6 supervision system)
etc

Finally, I want to ask about the intended execution semantics for 1. multi-processing (if and how scripts are aware of another, IPC communication between instances, resource handling to prevent deadlocks etc) and related 2. how the runtime is supposed to work.
Do you plan to stay "relative bare metal" and allow users to setup IPC routines or how should this be handled?
For understanding, it would help to provide intended use cases.

As all variables must be declared in Hush, this does not apply.
As there's no such thing as sourcing a script in Hush (only importing), this also does not apply.
This is interesting, and I guess we should implement it as an std function. It should be pretty easy to implement, as we have the call site SourcePos metadata.
Not sure we should implement this in Hush. Seems better to leave that problem to whereis or some other tool.

Using fn instead of function is appealing, maybe we can get to change it before exiting the alpha phase.
I have no plans to introduce multi processing in Hush, be it in the form of threads or IPC. As Hush is a shell language, heavy processing should be off-loaded to external applications written in more robust languages like Rust, and then be invoked from Hush. Languages like Python have much more competent designers/developers than me, and yet still they fail to make multi-processing robust. It is a insanely hard problem, and Hush will do better by just not implementing it.


Protractor should have a function/wrapper to convert object returned by  getText() or getAttributes('value') into string

A simple command as following:
var ele = element(by.id("someID"));
var url = ele.getText();                    //returns an object
Now, I want to use this url in forking a new browser instance. For example:
newBrowser = browser.forkNewDriverInstance();
newBrowser.get(url);
But in return I get an error saying that : "URL must be a string and not an object"
Desired Functionality:
newBrowser = browser.forkNewDriverInstance();
newBrowser.get(parseToString(url));

You cannot convert promises to strings synchronously, so this is not a valid feature request.
Please follow up with this on stack overflow. Github is meant for feature requests and issues only. Thanks!


turn off micro when capture screen

How we can disable microphone without muting other programs sounds?
rtmpDisplay.disableAudio() - disable fully audio

This is not possible. Other programs are captured with microphone. Audio is captured from speakers with the microphone.
https://developer.android.com/preview/features/playback-capture
This could be uselfull to replace microphone but only available with Android Q

Ty for anawer, waiting for Android Q


fix inventory permissions

Codecov Report
Patch and project coverage have no change.

Comparison is base (9a6a670) 83.81% compared to head (21b9465) 83.81%.

:exclamation: Current head 21b9465 differs from pull request most recent head fc03f5b. Consider uploading reports for the commit fc03f5b to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #96   +/-   ##
=======================================
  Coverage   83.81%   83.81%
=======================================
  Files          57       57
  Lines        1223     1223
  Branches      395      395
=======================================
  Hits         1025     1025
  Misses        173      173
  Partials       25       25

Files Changed
Coverage Δ

...nents/CompletedTaskDetails/CompletedTaskDetails.js
91.66% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

lgtm


Add visibility slot to Symbol model

In preparation for the visibility-based selective flagging of CPython symbols marked as static inline.
cf https://github.com/trailofbits/abi3audit/issues/85.

Thanks @nicholasjng!


[BUG] before_event not working with validate_on_save = True

I'm using this class to implement the ManifestVersion type in my application: https://python-semver.readthedocs.io/en/latest/advanced/combine-pydantic-and-semver.html
The problem is that Beanie saves this value as an array in the database and when loading the document from the database, it is not able to convert it back, ie. it's not able to load that document again.
I fixed this by triggering the @before_event decorator, though not ideal, it works. But, it doesn't work when validate_on_save=True is set. It seems that validate_on_save is triggered after @before_event.
class UserModel(BeanieDocument):
    name: str
    version: ManifestVersion

    @before_event(Insert, Replace)
    def semver_to_string(self):
        self.version = str(self.version

    class Settings:
       validate_on_save = True

This wouldn't be an issue for me if there is a possibility to tell Beanie explicitly to convert certain datatypes when inserting/replacing them in the database (after validation). Now it decides that some objects should be an array, while it works better when stored as string.

Hi! Thank you for the report. I'll check it during the next bug fixing session


Update for Xcode 13

[x] Update tests to pass in Combine compatibility mode
[x] Fix OpenCombine so all tests pass
[x] Implement the new APIs
[ ] Write tests for the new APIs
[x] Enable testing with Swift 5.5 on Ubuntu (when Swift 5.5 docker image is released)

@broadwaylamb would you mind if I merge this as is, without tests? I'm thinking of tagging a new version real soon and it would be great to have this included.

@MaxDesiatov to be honest, I'm not sure this implementation works properly.

ok, I'll postpone this then

@MaxDesiatov I'm trying to get async tests to work with Wasm, and I'm stumbled upon the following issue: apparently, the swiftwasm fork of XCTest doesn't have anything async-related — XCTestExpectation, XCTWaiter, etc.
Swiftwasm's Foundation doesn't have RunLoop, either.
So, is there any way for me to wait on a Task (the stdlib class) in a blocking manner, so that I could call an async closure inside a sync one and wait for it to return?

Unfortunately, currently the answer is no, I think. AFAIU Swift 5.5.0 didn't have support for async tests. Some later 5.5.x release of upstream Swift apparently added support for async tests, but I didn't merge that into our fork yet. And that merge is unlikely to be trivial. IDK if it makes sense to do it in 5.5 point release of SwiftWasm at all, we may just do it for 5.6 only.
I think for now it's best to exclude async tests with #if !os(WASI).
@kateinoigakukun WDYT? Am I wrong in saying there's no easy way around this right now?

Also, be careful with Windows 5.5.2, there are some regressions there https://github.com/MaxDesiatov/swift-windows-action/issues/1

sigh…


remove RD tibi-covaci

Docs Build status updates of commit 75b1a1b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

rd/index.yml
:warning:Warning

Details

rd/map.yml
:warning:Warning

Details

rd/media/profiles/tibi-covaci.jpg
:white_check_mark:Succeeded

rd/tibi-covaci.yml
:white_check_mark:Succeeded

rd/toc.yml
:white_check_mark:Succeeded

rd/index.yml

Line 0: [Warning] Invalid file link:(~/about).
Line 0: [Warning] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning] Invalid file link:(~/map).

rd/map.yml

Line 0: [Warning] Invalid file link:(~/about).
Line 0: [Warning] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning] Invalid file link:(~/map).

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


AP-3721: Remove No response textarea from Nature of Urgency question

What
Link to story
Because the input field is no longer required, remove the additional_information value from the form, controller, and view
Update the locale files to remove entries that will no longer be needed
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

Do we care about the additional information providers have given for existing applications, or can we subsequently remove the column completely?


TypeError: Cannot read properties of undefined (reading 'code_snippet')

Something went wrong in Ignition!
An error occurred in Ignition's UI. Please open an issue on [the Ignition GitHub repo] and make sure to include any errors or warnings in the developer console.

TypeError: Cannot read properties of undefined (reading 'code_snippet')
    at XE (http://<IP_ADDRESS>:8001/login:52:249981)
    at mi (http://<IP_ADDRESS>:8001/login:52:70706)
    at Ql (http://<IP_ADDRESS>:8001/login:52:122308)
    at xc (http://<IP_ADDRESS>:8001/login:52:109797)
    at _c (http://<IP_ADDRESS>:8001/login:52:109725)
    at Cc (http://<IP_ADDRESS>:8001/login:52:109588)
    at Nc (http://<IP_ADDRESS>:8001/login:52:106582)
    at hc (http://<IP_ADDRESS>:8001/login:52:103958)
    at nu (http://<IP_ADDRESS>:8001/login:52:119816)
    at http://<IP_ADDRESS>:8001/login:52:121187

User Agent
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.109 Safari/537.36

After migrate laravel to 9 from 8. This is the error i am getting.

Thanks for reporting this issue! We've managed to reproduce it and will soon release a fix.
Since multiple issues were posted for this bug, I'll go ahead and close this one and post further updates in #59

Thanks for your help!
The issue has since been resolved in https://github.com/spatie/ignition-ui/commit/dbfc5255e8add6919e5b25ec061e89b133d86b79. A new version of spatie/ignition has been tagged and running composer update in your project will automatically pull this in.

Ok. Actually this issues occured because after upgraded to laravel 9, google-drive package i had installed was not working anymore.


setup.py will crash

A few grammar not compatiable with python3.x like:
(1)service_list.keys().index(service) AttributeError: 'dict_keys' object has no attribute 'index'
(2) after encode('utf-8') python will not treat it as str
And this script create 3 subnets across 3 AZs, so if your target region only have 2 AZs, it will be failed.

please use python 2


Bugfix/show DRM acceptance page before viewing content in full screen

#1160
Checklist

[x] the [contributor license agreement][] is signed
[x] commit message follows [commit guidelines][]
[ ] tests are included
[ ] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
This bug was firstly caused by LegacyContentApi failing to parse the URL, which resulted in Item version being -1.
After above issue was fixed, the next issue was the page stayed in Summary page rather than going to the DRM acceptance page. After lots of investigations, I found the key thing is to make sure RootItemFileModel has a proper file name which can be extracted from the URL and is used to determine what viewers to be used in the full screen mode.
After fixing the second issue, there was a third issue. Once DRM terms had been accepted, although the page showing contents was opened, the page looked very bad because CSS styles applied to this page were never included in Legacy.scss.
Once the third issue was fixed, this bug was completed fixed. So I reverted workaround added in DRMTest.

Once the third issue was fixed, this bug was completed fixed.

Gah! Fair effort @PenghaiZhang
Ye olde can o' worms.


Routing with render function instead of component does work?

I'm not 100% sure this is related to this project specifically but as i'm using it, considered asking here.
So i've created a new route for a test that use the render function instead of component, and that function just doesnt seem to be called for some reason
This doesnt work (in routes.jsx file):
<Route path="/home" render={() =>
Home
} />

This does work:
<Route path="/home" component={() =>
Home
} />

Any explanation? It might be somehow related to the SSR but i dont know where to look for this for how to debug it.

ok..... just noticed that we are actually using react-router v3 so there is no render prop on Route 😭
Shame on me


have question about the training process

In dataloader, it "select 64 frames from total original frames, proportionally" in every video sequence, but how to predict the total count with only 64 frames?
for example, in a situp video, if the sampled 64 frames are all "sit" state by coincidence, then we can infer the count is 0, while the ground truth is not

Sorry for the late reply.
This is a limitation of our methods, we have discussed this in our paper.
The 64 frame samples are follow the frame selection of the former work RepNet(CVPR20).
If you have any other questions about TransRAC, feel free to reopen the issue.


fix: Set rootProject.name to Gradle approved characters

Gradle wants the root project name to be from a small set of characters that, specifically, doesn't include " ".
This isn't a problem when this is a standalone project, but it is a problem if you try and include this code in another project as an "included build" -- for example, if you're developing changes to the library.
So rename to "android-connector" to match the module name.

Thank you


README update for Turkish Translation

This PR is in regards to the Turkish translation information in the main README file of the project. Please see my announcement in 'Community Translation Accouncements" : https://github.com/vuejs/vuejs.org/issues/2015#issuecomment-475966588
ONLY the README.md file should be merged into the main project. I couldn't submit only the related file, independantly from other files already translated.
In my update, I insert the Turkish language information in the alphabetical order (after Spanish and before Vietnamese). The information contains a link to my blog so that people can learn more about the primary maintainer of this translation fork as well as a direct link to the Turkish translation repo.
Thank you!

I will prepare a cleaner pull request


method

Hello,
Please, add the following functionality
// the neural network training method to use. Options are 'sgd', 'adagrad', 'windowgrad', 'nesterov'
method = 'adadelta'

Where we can add this method ?


Develop

Closes #19 with an update to alpine caddy 0.9.3

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: princemaple:x: Killian BrackeyKillian Brackey seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.


UI Enhancement: Change default sort order of Execution Stats bar chart

This is regarding the bar charts visible on the "Performance" tab along with the itemized detail pop-up.
For the "Execution Stats" bar chart, I think it would be helpful if the sections of the bars were sorted so that the largest sections were on the bottom.
As an example, the "Waits" bar chart graph is already set up this way. But it seems the "Execution Stats" bar chart graph is instead sorted alphabetically, which can make it somewhat difficult to follow.
The only reason I thought of this is because I just recently started using the DBADashReports, and when looking at the Procedure Stats report, it too is sorted by percentage rather than alphabetically and it is easier to see what's going on.
DBADashGUI - Performance - Execution Stats:

DBADashReports - Procedure Stats:

Thanks for the suggestion - makes sense.
Note: The SSRS reports are no longer being developed/tested.  They might provide a starting point to build on though if you want some SSRS reports.

Included in 3.11 🚀


TSW-71918: Deploy-CAM.ps1 now takes to top entry for AzureRM and Azure

When doing the Confirm-ModuleVersion check AzureRM and Azure takes only the top entry of the object for the comparison. At this point $AzureRMModule and $AzureModule will have contents, so this should always return a result for comparison.

Are we guaranteed the arrays are in the order we want or can/should we do a sort?


Error when I run android

Has anyone had the same problem?

ignite doctor
Computer
  Platform: darwin

Ignite
  Version: 1.13.0
  Path: /usr/local/bin/ignite

Node
  Version: v7.2.0
  Path: /usr/local/bin/node

NPM
  Version: 4.1.1
  Path: /usr/local/bin/npm

Yeoman
  Version: 1.8.5

React Native CLI
  Version: 2.0.1

App
  React Native Version: 0.38.0

"React native uses 23.0.2. maps uses 23.0.3. and some other 3rd party libs use 23.0.1." - skellock

Thanks!


Leer novelas

Source name
Nova encuentra tus novelas ligeras
Source link
https://novelasligeras.net/
Source language
Español
Other details
Si
Acknowledgements

[x] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

Mihon/Tachi can't do novels


Shiny.Localization

Description of Change
New plugin for Shiny called Shiny.Localization.
Its main goal is to manage various text resources type (resx, json, xml, database, etc), from various origin (assembly, web service, database, etc), for various cultures, all mixed if needed.
I implemented some tests but most of all a pretty exhaustive documentation (input/docs/localization/gettingstarted.md) wich should make you understand the idea.
I also implemented the plugin into Shiny Sample wich will result in another PR to demonstrate its functionalities.
Issues Resolved
No issues but a new feature
API Changes
No change for existing APIs but some more for Localization
Platforms Affected

All

Behavioral Changes
None as it would be a plugin
Testing Procedure
Just run the Sample and pick Localization then change the language from the picker
PR Checklist

[x] Rebased on top of the target branch at time of PR (well I used merge for this one, sorry about that)
[x] Changes adhere to coding standard

I really appreciate the PR.  However, I really wish you had have opened an enhancement issue to see if this was a feature set the project would want to take on.  This was not on the roadmap.
The reason why I'm hesitant to take this on is because right now, I am the only developer on the project and this isn't something I necessarily want to support going forward.

I'm sorry about the missing issue, that was something I was wondering for.
Then I realized I realy need it in my job, so I just developed it :)
I tryed to create it your way, your style. Except maybe the option builder (for fluent and infinite settings purpose), everything is written like other plugins are (well I hope so), so you could understand quickly what is done and what it is about.
When built, I published it as a private Nuget package on our Azure DevOps repo.
Finaly, when all done, I created this PR in case you think it could be part of the original Shiny project. About this, I could recreate the PR but on a dedicated branch if you wan't to play with it but the sandbox way.
If not, but think it could be useful for others, we could also publish it on public Nuget with a reshaped namespace like Shiny.Extensions.Localization or Shiny.External.Localization or whatever suit to you. A namespace allowing other developers to extend Shiny features without expending your amont maintaining work.
If not at all, I'll keep it for my personnal use, nevermind, don't be afraid to hurt me :)
At least it's already useful for my colleagues and me.
You decide :)

I think this is a great chunk of code.  I think you might actually be doing yourself a bit of a disservice releasing this under Shiny (or attaching shiny to it at all).  I think you could easily make this an abstraction of it's own for any .NET ecosystem.
If you'd still like to keep it under the Shiny name, we can discuss some different options.

I think you're right.
If it could be useful for shiny users, it could be for non-shiny to.
I think I'll reshape it as a standalone project like Plugin.Localizer, without any Shiny dependencies.
Then maybe, I'll send you a PR for a Shiny.Integration.Localizer, wich could provide some exclusive features like autoinitialization on a shiny background job.
Sounds good to you?

Just to let you know that, as discussed, this project is now published as Localizr on nuget and availble here https://github.com/Respawnsive/Localizr
There's no Shiny integration package as there's no Shiny specific features actualy.
We could just call services.AddLocalizr...
Regards


Fix minor bullet and caps in Editorially Chosen Content

Fix second level bullets not rendering
@erhopf FYI

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/MicrosoftDocs/azure-docs
Private repo URL: https://github.com/MicrosoftDocs/azure-docs-pr
We have closed this pull request. Please resubmit the updates through the private repo. For more information, [see the contributor guide].(https://docs.microsoft.com/en-us/contribute/)


[Question] Loading py2devtools.py json into chrome developer tools timeline

Hi,
Loading the JSON performance data produced by py2devtools.py into Chrome developer tools timeline isn't working for me.
Right click timeline, "Load Timeline data" and load JSON created by py2devtools.py results into nothing.
The error I'm getting:
[4763:4763:1230/195530:ERROR:CONSOLE(417)] "TracingStartedInPage event not found.", source: chrome-devtools://devtools/bundled/timeline_module.js (417)

Perhaps I'm overlooking something? Any ideas?

I think by the timeline they meant chart option from Profiles tab.
I was able to get the timeline chart by loading the file into Profiles tool from Chrome Dev Tools.
Then timeline-chart can be obtained by changing Heavy(Bottom Up) option to Chart.

Yeah that's it. Thanks the tip!


[Issue] Panic when start

Hi there, see your post in Reddit so I give it a try.
Unfortunately, It crashed on first start.
thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: DisplayParsi
ngError', /Users/anon/.cargo/registry/src/index.crates.io-6f17d22bba15001f/kbt-1.0.0/src/linux.rs:25:51
                                                                                note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I use macOS M1, install with cargo.

Only Linux running XOrg is supported now. Created a ticket for MacOS -> https://github.com/bloznelis/kbt/issues/8


Add option to restart failed tasks

I frequenty face the issue that a failed completed task needs to be restarted. Currently it is only possible from code, but in a cloud environment is it not convenient. I propose to add an admin action to restart completed jobs.
I also added filtering options to list views of the admin pages.

Coverage decreased (-0.6%) to 76.131% when pulling 84f7e5c657fb7ce810a2bccd105c452b9cee7b16 on hodossy:master into e04d44c07110443bbdeb8cb14effbe497ddbb98d on arteria:master.

This looks super helpful. Anything I can do to get it merged?

This looks super helpful. Anything I can do to get it merged?

Hi @schmave
This could be of interest: https://github.com/arteria/django-background-tasks/issues/267


[C++][Parquet] Boost 1.66 compilation fails on Windows on linkage stage

Boost's autolinking should be disable on compilation with MSVC, since it causes linkage with shared import libs, instead of expected static. Following error occurs:
LINK : fatal error LNK1104: cannot open file 'boost_filesystem.lib'
Environment: Windows
Reporter: Max Risuhin / @maxhora
Assignee: Max Risuhin](https://issues.apache.org/jira/secure/ViewProfile.jspa?name=Max+Risuhin) / @maxhora
PRs and other links:

GitHub Pull Request #437

Note: This issue was originally created as PARQUET-1210. Please see the migration documentation for further details.

Uwe Korn / @xhochy:
Issue resolved by pull request 437
https://github.com/apache/parquet-cpp/pull/437


Import hint is not shown when typing a name that is exported from a TypeScript local package

VSCode Version: 1.20.1.
OS Version: Windows 7 Professional SP1.

Steps to Reproduce:

Create "package1" folder and run "npm init" inside. Use all defaults, but for main entry point use "index.ts".
Add index.ts to the folder and add simple content there:

export function sayHello():string
{
    return "Hello";
}

Go one folder up and create "package2" folder (it must be in the same folder as "package1").
Run "npm init" inside "package2". Use all defaults, but for main entry point use "index2.ts".
Add empty "index2.ts" file.
Inside "package.json" for "package2" add the following depedency:

"dependencies": {
    "package1": "file:../package1"
}

Inside "package2" folder run "npm install". Make sure that "package1" link was added inside node_modules.
Open "package2" folder in VS Code.
Open "index2.ts" file for editing and start typing "sayHello". Notice that editor doesn't suggest you to import sayHello from "package1", but ideally should to. index.ts file from package1 is written in TypeScript so editor should have all the required information for the hint.

Is it a place for improvement? How can I make VS Code showing me hints? I want to have a multi-package workspace and having no hints for imports makes it hard to develop.
Thanks in advance.

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

This issue was moved to Microsoft/TypeScript#22252


Update examples/friends

This fixes running the app on master.

@michalmuskala I've added one more commit: adding examples/friends/.formatter.exs and formatting that app

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Add tests for all functions in utils.py

In particular, test copy_file_structure and all its options.


k3s on Android(arm or arm64)

run k3s on Android(arm64), and get the following tip:

Failed to find memory cgroup, you may need to add "cgroup_memory=1 cgroup_enable=memory" to your linux cmdline (/boot/cmdline.txt on a Raspberry Pi)"

lowmemorykiller is used for memory control on Android, cgroup memory is disable on Android
how can I run k3s on Android?
1.enable cgroup memory on Android?
2.remove the following code, and compile k3s?
if !strings.Contains(string(cgroups), "memory") {
    msg := "ailed to find memory cgroup, you may need to add "cgroup_memory=1
    cgroup_enable=memory" to your linux cmdline (/boot/cmdline.txt on a Raspberry Pi)"
    logrus.Error("F" + msg)
    return errors.New("f" + msg)
}

which is the better way?

@seiyaa If we remove that check the pods will still fail to launch because they will try to use the memory cgroup.  We would have to patch across the board and remove and reference to memory.  I'd be interested if this is the only problem.  I believe there is another issue with the cpuset cgroup not having the cpuset. prefix in all the files.

@ibuildthecloud thanks
cat /proc/self/cgroup on all my chhip boards(including hisilicon/mstar/mediatek), these boards have cpuset, but not have memory
I will try to add cgroup memory

Having memory limits for containers is very common when running kubernetes. Since this flag is part of the kernel, I think one would need to flash a modified Android OS that allows memory and CPU cgoups. Your debug message is geared towards raspberry pi devices. Is there an Android developer group that could point us to steps to get this enabled for Android. I'm fairly sure the support has been available for quite a while now.

hey any progress on this ??

@dur3x @erikwilson @borg286 @seiyaa @kityun Looks like quite a few folks are interested in making k3s work on android (including myself).  Has anyone made progress getting it to run?  perhaps issue links to other projects?

In theory it should be possible for k3s to run on Android, since it is a Linux system. I think the caveat is that it probably will not, now or ever (without extensive changes), run on a stock system. To achieve this you will likely need a custom kernel with root access.

Closing due to age. Every time we've looked at Android it appears that (rooting issues aside) the kernel is missing critical features on most Android devices.

This is a challenge but not impossible.
https://wiki.postmarketos.org/wiki/File:K3s_cluster_built_with_hammerhead_and_docker.jpg

The above link is to an image, but here's the associated writeup: https://wiki.postmarketos.org/wiki/User_talk:Docker


Bump go dependencies

Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test unit-tests_istio

integ-pilot-k8s-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test unit-tests_istio

integ-pilot-k8s-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test integ-pilot-k8s-tests_istio

integ-cni-k8s-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test integ-cni-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot-k8s-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test integ-pilot-k8s-tests_istio

integ-cni-k8s-tests_istio
431988be1e388e27d43b751e4994111a42bfbc34
link
true
/test integ-cni-k8s-tests_istio

unit-tests_istio
5a501f216281b78914516ba08225920e59f46f86
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@nmittler ptal


hey guys，where i can find the firmwares changlog

thx u

We don't have stable releases yet, so the best place to keep up with changes is to watch commits on the master branch: https://github.com/ah-/anne-key/commits/master


⚠ Slack has degraded performance

In 59ea450, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 195 ms

Resolved: Slack performance has improved in 1b49b4b after 4 hours, 37 minutes.


[DOCS] Document how SDK properties affect accessibility in design system

Problem/Concern
The goal of this issue is to write documentation around how shades, chroma, and other SDK properties affect the resulting accessibility of a design system.  This issue has the further goal of Identify what, if any, of this new documentation should be available to the user through the UI.
Related to #750

@aaronreed708 Mind if i Join this convo and help sorting this out ?

Hi @vinfinity7, sorry for the delay in replying.  That would be wonderful if you could take a stab at this.  If you would like, we could set up a call with yourself, myself and @lwnoble to collaborate on what information should be covered and where in the documentation it should live.  And what of this information we could re-use in contextual help inside Theme Builder.

Yes I would Love to be a part of such a call , lets make it happen . We can set up this call after 12:30 PM IST 4 May , Looking forward to work with yall !

@vinfinity7 I can't make this weekend work.  Are you available during the week at all?  If so, what time of day is generally good for you?

due to my university examinations , my schedule is running a bit tight nowadays ,I will be available after 12:30 PM IST 4 May onwards onto the next week , Lets make this happen !


feat: vanilla-extract utils

Not sure about your intention to have a dedicated package for having utils for vanilla extract but i'm working on an internal project with kobalte and this could be useful to others too 👀
This pr introduces a new package @kobalte/vanilla-extract that may exports some utils for vanilla-extract, just like the tailwind ones.
Currently I've added a componentStateStyles method that allows to compose the style for kobalte data-attributes selectors. This allows to write the selectors through a configuration object which has the vanilla-extract type constraints, instead of writing it from scratch (eg. &:not([data-hover]) or [data-theme=dark] &:[data-hover]))
Example
// Vanilla
const btn = style({
  selectors: {
    "&[data-hover]": {
      // styles
    },
    "&:not([data-active])": {
      // styles
    },
    ".parentClass &[data-active]": {
      // styles
    },
    ".parentClass &[data-hover]": {
      // styles
    }
  }
});

// With utils
const btn = style([
  componentStateStyles({
    hover: {
      // styles
    },
    active: {
      not: {
        // styles
      }
    }
  }),
  componentStateStyles(
    {
      hover: {},
      active: {
        // styles
      }
    },
    { parentSelector: ".parentClass" }
  )
]);

If something is not clear, or you think the API can be improved or missing, just tell me 😄

Thanks, i will publish it soon.


Product export command: --force does not have an effect

Description:
The --force parameter for the product-export:generate command does not always enforce product export generation.
Instead, if the file is still existing and the generated_at timestamp is younger than current_timestamp - interval it will not force re-generation of the export.
This is caused by the ProductExportFileHandler::isValidFile() function not taking the --force parameter into account.
Expected result: The --force parameter always creates new export files regardless of the cache / generated_at timestamp.
Actual result: The --force parameter has no effect.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-34007.
[created from NEXT-34007, comment 485960]

[public] Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know. Thank you again for your valuable feedback.
[created from NEXT-34007, comment 501233]


agent.setFollowupEvent purges agent.add

I'm trying to make my bot say some stuff and later trigger an Intent with an Event trigger. But, every time I have something like:
agent.add('I will say something here!');
agent.setFollowupEvent('SOME_EVENT');

The agent.add command is skipped over and the bot goes directly to the intent triggered by SOME_EVENT.
If I remove the line of code with agent.setFollowupEvent, it works fine.
Is this a bug or am I misusing the api?
Cheers!

This is not a bug in the library but the way Dialogflow works.  Follow up events are a way to redirect from the currently match intent to another intent.  The currently matched intent does not have a chance to respond before it is redirected.  This means you will only have one response from the intent that is redirected to last.  You can't compile response from multiple intent responses by redirecting through multiple intents with follow up events.
When a followup event is triggered from the webhook, the event specified in the webhook response is triggered. Triggering the event through a webhook sends the request back through Dialogflow for matching without responding to the user. The response to the user is sent as if the user triggered the event that was triggered in the webhook.  After the second intent is matched by the event, Dialogflow will do any necessary work (like webhook calls or additional prompts for required parameters) and send a response back as if the second intent was the originally matched intent.

A user sends a request to Dialogflow.
"Intent 1" is matched by Dialogflow.
A webhook request is sent to your fulfillment server, which indicates that intent 1 was matched.
Your fulfillment server responds to the webhook request with a followup event response.
Dialogflow re-matches intent 2 based on the followup event sent in the webhook response.
Dialogflow sends a response to the user based solely on intent 2 being matched. Dialogflow may send another webhook call indicating intent 2 was matched, or send a prompt for unfulfilled required parameters (if configured) for intent 2.

Can you provide some sample code for adding a followup event? I tried to generate an event in following ways, I'm not getting any error, but the followup intent is not triggered
agent.setFollowupEvent('CUSTOMER_NAME_REQUESTED');
OR
agent.setFollowupEvent({ "name": "CUSTOMER_NAME_REQUESTED", "parameters" : { "received": "false"}});
Please help

@juliejoseph3 Many of the Dialogflow samples have followup events like here.

I did the same in code and created an intent with the event 'CUSTOMER_NAME_REQUESTED', but the followup intent is not getting triggered. Also, No error shown
What could be the reason?

Honestly, it could be a number of things but I think you'll find it very helpful going through the sample to get a better idea of triggering intents in general.

Thankyou!

Is there a way to do setfollowup  with agent.add()?

agent.setFollowupEvent({ "name": "CUSTOMER_NAME_REQUESTED", "parameters" : { "received": "false"}});

after two days of the struggle, this works for me. Thanks a lot. You saved me.

@juliejoseph3 Many of the Dialogflow samples have followup events like here.

does that fact that line 71 puts the call to setFollowupEvent inside a variable assignment create a different result than just saying agent.setFollowupEvent('call_transfer_event')? If so, how

Hey, @davidwkaiser did you figure out the reason behind the assignment?

Hey, @davidwkaiser did you figure out the reason behind the assignment?

No, I didn't. I figured out a workaround and kept moving...

While setting a followup event, if I do not provide a agent.add() before agent.setFollowupEvent, cloud console shows an error - no responses defined for platform dialogflow_console.
  function setFollow(){
   // throws error without agent.add
   agent.add("hello");
   agent.setFollowupEvent({"name": "myFollowup", "parameters": {"myname": "John"}});
  }

What's the reason behind this behaviour?

The above function is working fine in my code. I need to get some data from an API and set the followup event using the data. When I add agent.setFollowupEvent inside then, it is not working.
function setFollow(agent) {
  agent.add("hello");
  axios.post(url, body, {headers})
    .then (({ data }) => {
      agent.add("setting followup event");
      agent.setFollowupEvent({ "name": data.followupEventInput_name, "parameters" : { "name": data.name}});
    })
}

The API call is working correctly, and its also receiving an appropriate response. But the followup event is not being set. The first agent.add("hello") works (which is not the intended behaviour when there is a followup event).
How to solve this?

Just in case you didn't solve the issue, axios call is asynchronous, so function setFollow() just ends with "hello" message before receiving the data. I htink you could force waiting for axios result by defining setFollow as async function and use await.
async function setFollow(agent) {
  agent.add("hello");
  await axios.post(url, body, {headers})
  ...


Stuck in repeated "Clearing RX buffer."

Hey @TillFleisch ,
I've got a weird one - I keep getting the following in my logs
[LD2450:276]: Clearing RX buffer. - repeated
[LD2450:258]: Sensor is not sending updates, attempting to leave config mode. - occasionally
[LD2450:131]: Sending command timed out! Is the sensor connected? - occasionally
The weird thing is the sensor was working perfectly and then after a while (not being touched) it starts failing and giving all ZERO values for my sensors, which aligns with the logs.
I thought perhaps I'm running out of ram so I added this to my sensors
  - platform: debug
    free:
      name: "Heap Free"

to see if I'm running out of ram and I consistently have 10s of thousands of Bytes of ram

Is it possible there is a memory leak somewhere that is causing LD2450 get stuck in this loop?
Things I've tried,

Rebooting esp32 device (power off and on)
Updating firmware
Increase uart -> rx_buffer_size to 512
Set logger -> baud_rate to 0

Things I've verified:

I'm getting 5v from GND -> VCC on the LD2450 and between my ESP32
I'm getting 3.3v from GND -> TX as well as GND -> RX and GND -> 3v3
I'm getting 3.3v from GND on LD2450 to TX as well as RX on ESP32

Here is my config: https://github.com/bsell93/esphome/blob/main/bedroom-thermostat.common.yaml
Any assistance is appreciated 😄

My first guess would be a power issue. Seeing that you're using 5V for the sensor, let's assume that this is not the case.
The Clearing RX buffer. message is contained in the logs, when the RX buffer of the UART port has been filled up. For some reason, on some ESPs new data will only be put into the RX buffer once it has been emptied completely. This can lead to un-processed messages. In complex ESPHome configurations, particularly ones with (compnoents) which use delays, the buffer can be filled up while other components are being processed. Based on the length of your configuration file alone, this may be the case here.
I would try the following trouble-shooting steps in your situation:

try a 'blank' yaml configuration which only contains the ld2450 component

this can help rule out issues caused by other components

try increasing the rx_buffer_size (see UART configuration)

The  Sensor is not sending updates log entry could be related (caused by) to the buffer issue. If this is not the case, it's probably a power issue which could be caused by other HW components. The sensor may struggle with noisy power, like when using 3.3V.
A memory leak is possible, though I would expect some kind of slope in memory usage. Does the esp crash/reboot? Does the loop time (debug component) correlate with the log messages (this would be in-line with the buffer thing)?
I have not yet tried your configuraiton. I may update this post once I do.

Thanks for the quick reply!
I will try a minimal config next to see if I can isolate the issue that way
I just tried setting the rx_buffer_size to 512 - that didn't seem to make a difference.
I also noticed a warning saying that the logger was using UART and suggested to set baud_rate to 0 for the logger - which I did. That didn't seem to make a difference. So I'm trying taking out the logger altogether to see if that helps.

Does the esp crash/reboot?

No - the esp32 appears to stay online

Does the loop time (debug component) correlate with the log messages (this would be in-line with the buffer thing)?

I've not tried checking that - will check that as well

Removing logger made no difference
Minimizing the config to just be the LD2450 has also made no difference.

I'm not ruling out the fact that it could be the power source - I'm using a AC -> DC step down converter. So I'm converting 120v AC to 5v DC. Specifically this: https://bityl.co/Qn4F (variant is 5V 700mA (3.5W))
The odd thing is it was working for days and then just randomly stopped and has not been working since.
I have two identical devices that are presenting identical behavior.

Okay - So to verify the power source theory I plugged the esp32 into usb and the sensor is now working as expected. 😢
So I'm guessing the step-down module is the source of the issue.

Going to try this power module and see if it does the trick https://amz.run/9J6v
I will close this issue since this is clearly not an issue with the codebase.
Thanks @TillFleisch for being my rubber ducky 🦆  😄
ps. hopefully this thread will help someone who may run into the same issue. 😄

Depending on what else is connected to the power supply, adding a couple of smoothing capacitors could help resolve the issue. (if you happened to have some laying around)

Thank you for the suggestions!
For the record:
I ended up trying it both ways just to be sure.
I added a 100µF Electrolytic Capacitor and that did the trick with the cheaper aliexpress stepdown module. (link to the capacitors I used: https://amz.run/9K2o)
I also tried using the above HLK PM01 (without capacitors) step down module in place of the aliexpress module and that did the trick as well.


COR-643 Bugfix on alt names pre-compute

Changes

Updated stopwordsStep.apply to remove stopwards for an alt name
Added in unit tests to confirm stopwordsStep.apply

Why these changes are being made

There was a small bug identified in the previous Pull request which actually prevented alt names from being pre-computed

Testing

Properly confirmed debug logs to confirm alt names are being updated

Confirmed via UI that results are now lower as expected

Codecov Report

Merging #376 (31b9351) into master (29897c9) will decrease coverage by 0.02%.
The diff coverage is 60.00%.

:exclamation: Current head 31b9351 differs from pull request most recent head 1b349a3. Consider uploading reports for the commit 1b349a3 to get more accurate results

@@            Coverage Diff             @@
##           master     #376      +/-   ##
==========================================
- Coverage   72.34%   72.31%   -0.03%
==========================================
  Files          33       33
  Lines        2513     2518       +5
==========================================
+ Hits         1818     1821       +3
- Misses        488      489       +1
- Partials      207      208       +1


📊 Add USGS minerals data

Main changes
Sorry, this PR has snowballed a bit! Here's a summary of the main changes:
Data changes

Added USGS data on minerals.
Fixed various issues on BGS data on minerals.
Created steps to combine both, and build an explorer.
Moved steps about minerals from energy to a new minerals namespace (with its own dag yaml file).

ETL changes

Added an --explorer flag to etl run, that creates an explorer and writes a tsv file to owid-content. We discussed this option in the data-architecture call and agreed that it's a move in the right direction. The implementation is not optimal, though:

Currently, --explorer sets an environment variable to 1. It would be better to handle this differently.
In theory, the new explorers step depends on the grapher://grapher step, so that's what should be added to the dag. However, this fails because GrapherStep does not have a checksum_output() method implemented, which makes the ETL break. For now, I used the data://grapher step as a dependency instead.

Improved the Explorer object, so that it can handle catalog-path-based explorers. It can also easily translate from variable-id to catalog-paths (so we could use it for all existing indicator-based explorers).
Added an EXPLORERS_DIR variable to .env.example. This variable is necessary if you want to create explorers from ETL, and your local owid-content is not in the same folder as etl (unlikely).

For reviewers
This PR is not yet ready to be merged, but I expect that more and more small issues related to minerals data are going to come up. So it would be better to merge this PR soon, and then have additional PRs to fix issues and improve metadata.

@Marigold could you please review the ETL changes (etl/command.py, etl/config.py, etl/explorer_helpers.py, etl/helpers.py)?

Hmmm, I guess this was closed automatically because of Mojmir's fix on a comment from this PR.

@pabloarosado fixed the issue with unit tests on master.

Thanks @lucasrodes for the suggestions!

@pabloarosado fixed the issue with unit tests on master.

Thanks @Marigold for the fix!


🐛 Fix the spec.URL in workspace using canonicalPath

The spec.URL in workspace should use canonicalPath

Summary
The spec.URL of workspace should use canonicalPath instead of absolute path.
Related issue(s)
Fixes https://github.com/kcp-dev/kcp/issues/2737

/hold for testing
this is still need to be confirmed whether it is the appropriate approach.

/unhold

Code LGTM. @sttts does this violate any tenets for workspace URLs?


Fixed syntax highlighting in readme

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe: documentation

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

Other information:

Thank you very much for the PR! :+1:


Modify the MempoolPayload data type

When originally implementing #514 (PR #554), I wasn't exactly aware of how this MempoolPayload data type was going to be used and I defined it incorrectly. The mempool needs to be able to hold individual ledger objects (transactions, delegation certs, update proposals, and update votes) while the *Payload data types I originally utilized hold collections of the aforementioned objects.

It seems that this should be sufficient for https://github.com/input-output-hk/ouroboros-network/issues/592. Ready for review.

bors r+


A11y: on:mouseover must be accompanied by on:focus with Svelte v3.44

I updated to Svelte v3.44 then get warnings on build.

Could you assist with providing a repro?
Each icon generated by this library has <!-- svelte-ignore a11y-mouse-events-have-key-events --> prepended to the source.
The Svelte REPL example uses Svelte v3.44.1, and no a11y warnings are logged in the console.

Sorry, it was my mistake, my project was referencing 2.0.0.
I don't see any issues with 2.1.0


Add dependency graph visualisation and measurement

max height of tree from :app is now 4
Resolves #193
Current dependency graph:

This PR was released with 0.22.1


[InputAdornment] IconButton is not aligned with Input when using it as InputAdornment

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://codesandbox.io/s/youthful-johnson-qhr1ql?file=/demo.tsx
you can demonstrate it just change props endAdornment into startAdornment on [this example.]
I think it only happens when Iconbutton is used as inputAdornment in Input, FilledInput, OutlinedInput.
(https://mui.com/material-ui/react-text-field/#input-adornments)

Current behavior 😯
I don't think Icon Button's padding is appropriate compared with padding of endAdornment.
padding is larger than using just Icon as inputAdornment
Expected behavior 🤔
No response
Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

It was happened because I pass end value to position prop of InputAdornment when using it as startAdornment.


SavingCallback and solver options

Hi,
The following worked for me in Julia 0.7
f = (du, u, p, t) -> du[1] = u[1]
prob = ODEProblem(f,[1.0],(0.0,1.0))
out = SavedValues(Float64, Vector{Float64})
scb = SavingCallback((u,t,integrator)->[2*u[1]], out, saveat = [0.0,0.25,0.5,0.75,1.0])
sol = solve(prob,CVODE_BDF(),  save_start=false, save_end=false, save_everystep=false, callback=scb)

now it produces an error:
ERROR: BoundsError: attempt to access 0-element Array{Float64,1} at index [0] which points to
https://github.com/JuliaDiffEq/Sundials.jl/blob/master/src/common_interface/integrator_utils.jl#L33
Without saving options everything workes fine:
solve(prob,CVODE_BDF(), callback=scb)
It seems to be related to these changes
https://github.com/JuliaDiffEq/OrdinaryDiffEq.jl/pull/514
However my initial setup workes fine with other solvers but Sundials. Should I use some other combination instead of save_start=false, save_end=false, save_everystep=false?

Was just a bug, fixed by that commit.

Thanks!


fix!(coverage): use transformMode based source maps

Description

Fixes https://github.com/vitest-dev/vitest/issues/3888
Fixes https://github.com/vitest-dev/vitest/issues/3251

This introduces breaking change as coverage providers now expect Vitest to pass transformMode in AfterSuiteRunMeta. But if this is included in v1 releases/beta releases, users won't notice anything as long as they don't mix vitest@v1 and<EMAIL_ADDRESS>packages.
I also ran into https://github.com/bcoe/c8/issues/172 in test/workspaces. V8 reports two different counts of blocks (branches) when run with different inputs.
https://github.com/vitest-dev/vitest/blob/9112cc96b4fe7f9db096ca83bee81ec7165e0710/test/workspaces/src/math.ts#L2-L9
This can't be fixed on our side - at least not without AST analysis or with hacky solutions like https://github.com/istanbuljs/istanbuljs/issues/531#issuecomment-656826893. As solution for now I simply set test/workspaces to use coverage.provider: 'istanbul'. Below is minimal reproduction of this issue showing how raw V8 reports show different count of branches:

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. If the feature is substantial or introduces breaking changes without a discussion, PR might be closed.
[x] Ideally, include a test that fails without this PR but passes with it.
[x] Please, don't make changes to pnpm-lock.yaml unless you introduce a new test example.

Tests

[x] Run the tests with pnpm test:ci.

Documentation

[ ] If you introduce new functionality, document it. You can run documentation with pnpm run docs command.

Changesets

[x] Changes in changelog are generated from PR name. Please, make sure that it explains your changes in an understandable manner. Please, prefix changeset messages with feat:, fix:, perf:, docs:, or chore:.

I marked this PR to fix https://github.com/vitest-dev/vitest/issues/3251 as well. Just to be sure - @sheremet-va can two workspace projects use different Vite plugins? Or are Vite plugins the same for each project? Only the test environment can cause different transforms between projects?

can two workspace projects use different Vite plugins?

Yes, they can and they do.

can two workspace projects use different Vite plugins?

Yes, they can and they do.

I see, vitest.workspace.ts can even contain inline plugins like:
export default defineWorkspace([
  { test: { name: 'Project #1' }, plugins: [/* Project specific plugins */] },
  { test: { name: 'Project #2' }, plugins: [/* Project specific plugins */] },
])

I'll remove https://github.com/vitest-dev/vitest/issues/3251 from linked issues. For that we'll need to pass the project name or some kind of ID in the AfterSuiteRunMeta.

Added fix for #3251 as well. test/workspaces contains good test case for this where two projects run identically but transform the source file differently.


Issue a warning when a parameter is not consumed by a task

Description
After consuming the parameters I add a check to ensure that the task consumed all given parameters. If it is not the case a warning is raised.
Motivation and Context

In one of our workflows, several tasks have similar parameters but not all. This confused some users that tried to use such parameter with irrelevant tasks and the users were confused. Adding warnings could help users to understand what's happening. It can also help finding typos in parameter names (if they have a default value luigi does not complain and just use the default value).
Have you tested this? If so, how?

I added a unit test.

With pleasure :-)
Do you know when the next release is planned? The last one is pretty old and there are several things in the master branch we are interested in.

@adrien-berchet , the current release is quite old. I haven't done a Luigi release myself in years, but I can try to get one out. If so, it'll likely be 3.1.0.
Out of curiosity, are y'all currently using 3.0.3, master, or a fork?

Ah cool, that would be great, thanks!
Currently we use the 3.0.3. But we have a repository (https://github.com/BlueBrain/luigi-tools) in which we add everything we need and then we submit PRs here for the generic features that might be interesting for others.

I've gotten the release started at https://github.com/spotify/luigi/pull/3171
I'll have to get a Spotify employee to approve the version bump before i can merge. At that point, i'll proceed with updating PyPi (hopefully i still have access to do so).

Ok, thank you very much!


Question 28, You can extend the Number prototype

I think this statement is falsy:

Note that you can't do this with numbers, since JavaScript doesn't automatically convert numbers to an object (unless you use new Number(2))

Example
Number.prototype.leet = () => console.log(1337);
42..leet() // logs 1337

You're right, my bad


Helpers

{{#textareaAutocomplete settings=settings id="msg"}}
{{/textareaAutocomplete}}
Hi. How would I write something like as above with Meteor Jade?

+textareaAutocomplete(settings=settings id="msg")


Trying to get the pipe table to work

Maybe a N00b question, but I'm trying to get a pipe table to work.
And I cannot find any C# example code.
Tried this:
var h = "a|b|b";
h += "--|--|--";
h += "1|2|3";
var r = MarkDig.Markdown.ToHtml();

but r results in:

a|b|c--|--|--1|2|3

documentation : here is not really helping...
Update:
Tried this, but also not working:
   var p = new Markdig.MarkdownPipelineBuilder().UsePipeTables().Build();
   var h = "a|b|c";
   h += "--|--|--";
   h += "1|2|3";
   var r = Markdig.Markdown.ToHtml(h,p);

What am I missing?

You'll need to add some new lines there - you're currently trying to render a|b|b--|--|--1|2|3

Thanks to MihaZupan...
   var p = new Markdig.MarkdownPipelineBuilder().UsePipeTables().Build();
   var h = "a|b|c" + Environment.NewLine;
   h += "--|--|--" + Environment.NewLine;
   h += "1|2|3" + Environment.NewLine;
   var r = Markdig.Markdown.ToHtml(h,p);

returns:

a
b
c

1
2
3


How to Convert a CSharp Project to JavaScript on a Mac?

Hi there, I would like to convert this to JavaScript, but I am on a Mac and as far as I can tell this tool JSIL only works on Windows. Am I wrong about that? How can I convert a CSharp project like the linked one (with dozens of files and imports) on a Mac to JavaScript?
Thanks!

Is there a specific reason?
3 suggestions:

Don't.
Migrate slowly using TypeScript.
See if you can add on a Blazor app and then leverage the C# natively using Blazor/WebAssembly.

JSIL is fully compatible with mono and the sandbox on the website is running on a Linux server. However, the project is unmaintained so I wouldn't advise using it.


feat(#165): add pytrees to requirements.txt

Description
add the python pytrees package to the docker environment. The decision to use it was done in #163. Quite a small change, but the process of reading through the docs and finding the right script for testing took some time.
Fixes #165
Time invested
Josef Kircher: 1 h
Type of change

New feature (non-breaking change which adds functionality)

Does this PR introduce a breaking change?
no
Most important changes
add package to requirements.txt delete old package from dockerfile
to test:

b5 update
b5 run
separate terminal: b5 shell
run py-trees-demo-behaviour-lifecycle

this will let a behavior count to 3 and stop, behaviors are comparable to states
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works (might be obsolete with CI later on)
[x] New and existing unit tests pass locally with my changes (might be obsolete with CI later on)

Just for our information: https://py-trees.readthedocs.io/en/devel/demos.html#module-py_trees.demos.lifecycle


Add Missing BodyPart Hediff for No-XXX-Genes

I got asked in Discord to add a Missing Body Part Hediff for the genes that remove a genital type.
Without it, there is an issue in OTY-Sized Apparel, according to the Discord.
Relevant Files:

[ ] No Anus
[ ] No Breasts
[ ] No Penis
[ ] No Vagina

Behaviur should be the same everywhere.
I am just not ... 100% sure if this will have other follow up issues e.g. with genes that regenerate missing bodyparts.
But I like the people making drawing mods so if this means that things re-grow then people should not combine these genes. Maybe I can make a <exclusiontag> too for common ones from Alpha Genes etc.

No Breasts just remove things back to 0.0 so they were not affected.
Important side note:
I first had to remove the missing-part hediff before re-adding the genitalia. Otherwise it would give me an error if I first added the part.


ECP Milestone (ST-PR-10-1380)

Due: 2020-09-30

[ ] Implement, release and integrate for testing of any additional benchmarks to help identify and track performance bottlenecks.

[ ] Explore additional app-centric issues (bugs and performance issues) and understand and address impacts based on complexities not possible to capture with lower-level benchmarks.

[ ] The Legion project currently has a CI testing system that is used internally for validating software updates.  Extensions to this system are needed to test at larger scale.  We will either undertake this within the project or explore integration with an ECP-wide CI system when that becomes available.

Completion Criteria:

[ ] Documented details of identified overheads, resulting fixes and resulting improvements in performance.

[ ] Software releases with corresponding performance issues addressed.

[ ] Software releases with bug fixes.

[ ] Improved testing infrastructure (including tracking performance characteristics where and when possible).

Whatever we did or did not do, this is project is long since over.


fix(root-layout): full height support

fix(root-layout): full height support

:hammer_and_wrench: Type Of Change

[x] Fix
[ ] Feature
[ ] Refactoring
[ ] Documentation

:book: Description
Been getting some requests for this so decided to fix it right now since it was pretty low effort.
Root layout component was not taking up the full viewport by default and consumers were having to pass in a bunch of utility classes to fix issues that this component should already handle. Added some flexbox classes to handle the sizing of the main content pane and render at full viewport height.
:bulb: Context
This component should have taken up the full viewport from the start, since it's only meant to be used at the root. Decided to implement it directly instead of making it a prop.
:pencil: Checklist

[x] I have reviewed my changes

:crystal_ball: Next Steps
release for implementation in web-clients

Adding Mika/David/Alex since they have all asked for this at some point


Using annotations with untargetted bindings results in no implementation error

I have the following situation where the code is throwing error because of no binding existence. In my scenario I used annotation. Following what's  described in the documentation and I quote,

When specifying binding annotations, you must still add the target binding, even it is the same concrete > class. For example:

bind(MyConcreteClass.class)
.annotatedWith(Names.named("foo"))
.to(MyConcreteClass.class);
bind(AnotherConcreteClass.class)
.annotatedWith(Names.named("foo"))
.to(AnotherConcreteClass.class)
.in(Singleton.class);

there is a mismatch.
public class Main {
    public static void main(String[] args) {
        Injector injector = Guice.createInjector(new CalculatorModule());
        SimpleCalculator calculator = injector.getInstance(SimpleCalculator.class);
        calculator.calculate(4,6,"+");
    }
}

public class SimpleCalculator {

    private Calculation addition;

    @Inject
    public SimpleCalculator(@Addition Calculation addition) {
        this.addition = addition;
    }

    public Integer calculate(Integer first,Integer second,String sign) {
        if("+".equals(sign)){
            return addition.applyOn(first, second);
        }
        return 0;
    }
}

@BindingAnnotation
@Target({
ElementType.METHOD,
ElementType.FIELD,
ElementType.PARAMETER,
ElementType.CONSTRUCTOR})
@Retention(RetentionPolicy.RUNTIME)
public @interface Addition {}

@ImplementedBy(Sum.class)
public interface Calculation {
    Integer applyOn(Integer first, Integer second);
}

public class CalculatorModule extends AbstractModule {
    @Override
    protected void configure() {
        bind(Sum.class).annotatedWith(Addition.class).to(Sum.class);
    }
}

public class Sum implements Calculation {
    @Override
    public Integer applyOn(Integer first, Integer second) {
        return first+second;
    }
}

There is the following error when I try to run the application:

Exception in thread "main" com.google.inject.ConfigurationException: Guice configuration errors:

No implementation for bug.Calculation annotated with @bug.Addition() was bound.
while locating bug.Calculation annotated with @bug.Addition()
for parameter 0 at bug.SimpleCalculator.(SimpleCalculator.java:13)
while locating bug.SimpleCalculator

1 error
at com.google.inject.internal.InjectorImpl.getProvider(InjectorImpl.java:1004)
at com.google.inject.internal.InjectorImpl.getProvider(InjectorImpl.java:961)
at com.google.inject.internal.InjectorImpl.getInstance(InjectorImpl.java:1013)
at bug.Main.main(Main.java:12)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:134)

The dependency is @Addition Calculation, not @Addition Sum.  So either of these will work:
bind(Calculation.class)
        .annotatedWith(Addition.class)
        .to(Sum.class);

or
bind(Calculation.class)
        .annotatedWith(Addition.class)
        .to(Calculation.class);

The second one only works because of the @ImplementedBy on Calculation.  Best practices would be to remove @ImplementedBy and use the first approach.

I will disagree with your solution. I posted a quote and as you can see they are binding a concrete class. Your bind is a normal Linked Binding. This is not related to the problem

You're free to disagree with me but I'm not wrong :)
There's a third solution too: Change SimpleCalculator to inject @Addition Sum.
Regardless, this is working as intended, it's not a bug in Guice.

When specifying binding annotations, you must still add the target binding, even it is the same concrete > class. For example:
bind(MyConcreteClass.class)
.annotatedWith(Names.named("foo"))
.to(MyConcreteClass.class);
bind(AnotherConcreteClass.class)
.annotatedWith(Names.named("foo"))
.to(AnotherConcreteClass.class)
.in(Singleton.class);
This is from the documentation, as you can see the bind and to are the same . And it's written Concrete Class it's not Interface so.

Okay i checked it and  it works
public class CalculatorModule extends AbstractModule {
    @Override
    protected void configure() {
        bind(Calculation.class).annotatedWith(Addition.class).to(Calculation.class);
    }
}

By concrete class i thought there are talking about implementation. My bad :).
Thanks


Update a nonexistent proc call to the new version

When trying to use this as a dependency, it failed, because the newer version of pixie I had installed had the proc usePng which appears to supersede usePngRaw. As such, this PR exists.

Actually, should the pixie version be the minimum required to have decodePng: Png, or is the newest version fine?


Fix "processFluidItems" sending null fluids and improve

https://github.com/Buuz135/Industrial-Foregoing/blob/432e14276c69de527e1d3ed4ff7525d74b83890e/src/main/java/com/buuz135/industrial/utils/ItemStackUtils.java#L190-L209

There are still some issues with this. Let me know if you want me to open a new issue about this.

First issue, if the machine tries to move less fluid than the cell will accept (it only accepts 1000, and returns 0 otherwise) the cell is moved into the output slot.

Second issue, if there is more than 1000mb in machine all the fluid is used up, even though the cell returns that only 1000mb was used.


:ghost: Bump github action use to current version

Issues identified in warnings on the checks/action summary page.
Changes:

codecov/codecov-action to version @v4

Replaced with #1780


Update card-demo.html

#7241 was a little heavy handed in removing all md prefixes.

I signed it.