MINOR: Add validation to the get_resource_config method in GrpcPlacementService

What type of enhancement is this?
Refactor
What does the enhancement do?
Add validation to the get_resource_configmethod in GrpcPlacementService and fill in the test case.
Tips:
you can implement the ValidateExt trait on GetResourceConfigRequest, then invoke validate_ext function to achieve the validation functionality.

you can locate the ValidateExt trait in the file: src\placement-center\src\server\grpc\validate.rs
you need to add validation to the node_list method in this file:  src\placement-center\src\server\grpc\service_placement.rs
you need to fill in the test case in this file: src\grpc-clients\tests\placement_test.rs

reference:

https://github.com/robustmq/robustmq/pull/548)https://github.com/robustmq/robustmq/pull/548

Implementation challenges
No response

mark


Add typescript typings

I have added Typescript typings to the project.

Would you like to merge this PR @styfle?

We don’t have admin permissions.
But I also don’t think this package is relevant anymore.
https://twitter.com/styfle/status/1105843173933436934?s=20


GIMME_ARM should be GOARM in .travis.yml

This project: https://github.com/sselph/scraper/blob/master/.travis.yml
    - GIMME_OS=linux GIMME_ARCH=arm GIMME_ARM=6

Some other project: https://github.com/rackspace/rack/blob/master/.travis.yml
  - GIMME_OS=linux GIMME_ARCH=arm GOARM=6
  - GIMME_OS=linux GIMME_ARCH=arm GOARM=7

Seems like gimme doesn't check for GIMME_ARM https://github.com/travis-ci/gimme/blob/master/gimme

Thanks! I updated the file.


feat: date sorting

Closes #41

Let's put it at the bottom of the facets on the sidebar. Authors should remain at the top.

Let's put it at the bottom of the facets on the sidebar. Authors should remain at the top.

Alright


Fixed static media failure on /graphql/ and /admin/ pages.

Closes #5610
What's Changed

Screenshots

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example App Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

Looks solid to me, thanks for adding the test case! Can we get that part ported over to develop as well easily enough?

Checking the test case now

Looks solid to me, thanks for adding the test case! Can we get that part ported over to develop as well easily enough?

Checking the test case now

https://github.com/nautobot/nautobot/pull/5613


Correct sizing of header back button image on iOS

Summary
This fixes the slight blurring of the back icon in stack navigator on iOS due to a mismatch between absolute pixel sizes of the icon (and the @2x, @3x, etc) and the stylesheet width and height.
The alternative here would be to regenerate all the iOS icons at the correct resolution (20x12 @ 1x), but I believe @grabbou has the master file (7a20389e04a41de65e145a456c7dcacb787aad24).
Test plan
Before:

After:

For reference, the @1x size of the asset is 21x13. https://github.com/react-community/react-navigation/blob/master/src/views/assets/back-icon%401x.ios.png

is there a way to set the height of the header of a stack navigator?
StackNavigator({...routes},{
headerStyle:{
height: 20
}
});


isBotBlockedBy function

Feature Request

I have:

searched for such a feature request (https://github.com/yagop/node-telegram-bot-api/labels/enhancement) and found none

Introduction
I needed to message all Bot users but I get sometimes an error of Bot blocked by user! so it'd be helpful to have a isBotBlockedBy(userID) function.
Example
if ( !isBotBlockedBy(userID) ) {
bot.sendMessage(userID, ' hi ');
}

You need to setup a local DB or similar that you store the userID of the users + the status (block or not).
Actually you can do:
bot.sendMessage(userID, 'hi').catch(err => {
// If the Telegram bot api return an error, you can handle it and put the user your db the list of blocked
})
Recomended for future, listen this event for know in real time when a user block, not block, change persions of your bot...:
bot.on("my_chat_member", (msg) => {
 // You can know if a user block or unblock your bot (When the user perform this action). Is a telegram update event
})

Note: For get this update you need to put in the list of "allowed_updates". Reference: https://github.com/yagop/node-telegram-bot-api/issues/923#issuecomment-996038501

that's so helpful. thank you, alot!


create Chatbot project

I created a simple Chatbot program that takes input from the user and responds to the questions. Things this project cover are taking input, structures, dictionaries, string interpolation and some built-in functions such as rand(), occursin() etc.

Thanks! Okay, I added to the README.

Thanks. Hoping to contribute more in the future!


Use "base" tsconfig with overrides when neccessary?

https://www.typescriptlang.org/docs/handbook/tsconfig-json.html#tsconfig-bases

Please don't remove the issue template, it is there for a reason.
This sounds like a duplicate of #634 / #753 . Given the removal of the template and lack of details, it is difficult to interpret if you meant use an existing base, e.g. @tsconfig/recommended or for TSDX to have its own base, i.e. @tsdx/tsconfig. Per the linked issues, I plan to have @tsdx/tsconfig that all the templates will extend. It will itself extend @tsconfig/recommended.
@tsconfig/recommended is itself only a few settings and isn't currently commented well, so I don't think changing to that in isolation (without @tsdx/tsconfig) is worthwhile. I actually wrote the code for that but decided against it and to wait until @tsdx/tsconfig exists, especially as that will make for a more intuitive dependency structure.


Add set of icons (read carefully steps)

Close #227

Braulio, the icon is called globe in the issue, the copilot suggests the internet and by searching for the icon they usually call it the internet, do we leave it like that?


create business registry, registry search, personal property registry, sbc-connect (as connect) directories

Completed in #42


End User Documentation

This goes along with issue #61, we only need it to be documented well enough for Erik and Todd to be able to use it.


Fix example EXT bug

Description
Adds missing " to the EXT pizza example

That bug has already been fixed in the following PR along with one other bug.


Implements onRefresh event at SummaryProductFragment

Implements onRefresh event to allow pull-down refresh when browsing a expanded view of a scanned product.

It works! But it doesn't query for product state, only for robotoff questions... Strange..

Thank you! 🥳

Can I close the issue related to this PR or do we need something else done?


Discrepancy between host.json log filters and dotnet log filters should be documented

Having failed to understand this and spent a while figuring it out, I would like to save someone else the pain.
E.g. Attempting to get logs at trace level, setting a log filter in host.json does not work:
{
    "version": "2.0",

    "logging": {
        "logLevel": {
            "default": "Trace"
        },
        "applicationInsights": {
            "samplingSettings": {
                "isEnabled": false
            }
        }
    }
}

Instead you must do:
            var host = new HostBuilder()
                .ConfigureFunctionsWorkerDefaults()
                .ConfigureLogging(logging =>
                {
                    logging.AddFilter("System.Net.Http", LogLevel.Trace);
                })
                .ConfigureServices(s =>
                {
                    s.AddHttpClient();
                    // etc
                })
                .Build();

Same with the "default": "Trace" key and logging.SetMinimumLevel(LogLevel.Trace), although the latter causes my function to not start correctly and time out, the first one does not enable trace level logging globally.
This should be documented in the "Logging" section.

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

ID: 6ad5f362-cbfb-0621-4389-6378f50359e4
Version Independent ID: ebf1a8f3-ab9d-d295-e3a4-a982ed9df1e9
Content: .NET isolated process guide for .NET 5.0 in Azure Functions
Content Source: articles/azure-functions/dotnet-isolated-process-guide.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hello @allanlw,
Thank you for your feedback! We will review and update as appropriate.

@allanlw Just want to say THANK YOU. I was going nuts trying to stop the ocean of Microsoft logs in my function (I use EF Core, so every query command etc...), and just couldn't get it to work via either host.json or local.settings.json ("logging:loglevel:..." etc). Magically I clicked the link at the very bottom of the doc to get to this page. Yes you absolutely saved lots of pain :).
I sincerely hope .NET 5 Azure Function will get some first-class treatment (Visual Studio integration etc) and updated docs from Microsoft soon.

Thanks @allanlw for reporting this.
@brettsam do we not still support host.json settings to control logging in .NET isolated, or is there a bug somewhere?

@ggailey777 @brettsam Thanks for looking into this. While you're at it, I just want to let you know that IConfiguration DI via local.settings.json also seems to be broken, which means the Options pattern described in this doc does not work.  See this.

@ggailey777  @brettsam  @felinepc Can confirm this problem re: IConfiguration. This is currently a show stopper for my efforts to port to Core 5.  Is there a workaround? I have code that needs to work in both MVC and Azure Functions.

@kinetiq I had to make a few duplicate service configuration code files and use Environment.GetEnvironmentVariable just for the Azure Function project.
I also configured some options directly via services.AddOptions<TOptions>().Configure(options =>{...}).
Thankfully my project is still small. I can see this being a problem for larger projects that already make heavy use of the IOptions pattern.

@felinepc  Thanks for getting back to me on that. I'm only having this issue in one place... I think what I'm going to do is try and add calls to Environment.GetEnvironmentVariable in the event that IConfiguration can't find a value. I have to make sure this works in both MVC and Azure Functions, which is tricky, although in the previous version of AF this was working fine.

I just chatted with @brettsam and there are 2 points that need to be clarified in the docs:

The custom dependency injection model that we have for .NET in-process (C# class library functions) doesn't apply at all to .NET isolated (.NET 5.0) functions. Totally ignore the old DI article and follow the guidance here, which relies on normal ASP.NET Core DI model. I'll add a note to the Functions DI article to make this clear.

Log filter settings defined in host.json only apply to the functions runtime host. Because .NET class libraries are loaded into the host process, those rules get applied automatically. However, when running out-of-process, the .NET worker doesn't have access to those filters. Because host.json is just a config file, you can load this file as a JSON configuration provider. In theory, this would give the worker process access to the same logging rules as the runtime. We'll try to get an example of how to do this published ASAP. In the meantime, the approach that @allanlw suggests works, it's just not in-sync with host.json.

@ggailey777 DI itself registered via .ConfigureServices in isolated Azure Functions does work. The problem is specific to IConfiguration and config/options binding.
To put it more clearly, in traditional ASP.NET Core we have appsettings.Environment.json files in which we can enter configuration variables, and either access them via either IConfiguration or IOptions (if binding is configured).
Where do these configurations should go now with out-of-process Azure Functions?
P.S. I recommend to have all the updated docs related to out-of-process Azure Functions with .NET 5 added or referenced in this .NET 5 Azure Functions guide, which is to date the most specific official doc related to this product.

Yes, to be extremely specific: in the previous version of Azure Functions, if I injected IConfiguration, it would automatically pick up values in local.settings.json, in the Values array. In Production, it looks like it would also pick up environment variables.
That could be useful in cases where code needed to be shared between MVC  and Functions... It was possible to drop the value in local.settings.json and appsettings.Environment.json and everything would just work.
However, I can see that this behavior is at odds with MVC configuration, where you have to manually tell it to use appsettings.json.  Seems like it might be best long term to bring all this into parity. Of course, backwards compatibility problems abound...

@kinetiq Hey looks like I just missed mhoger's reply here. Adding
.ConfigureAppConfiguration(c =>
{
    c.AddEnvironmentVariables();
})

made IConfiguration pick up values in local.settings.json again (still have to be under Values section).
They stated that they'll add this by default in the next release.
Sorry I think I may have driven this particular issue off-topic, which is related to logging configuration.

@felinepc I did, thanks! Works great!

@allanlw @ggailey777 @brettsam  After getting the DI issue sorted out, I conducted extensive testing on the logging configuration. Here's what I found (its current behavior is very quirky):

The "default": log level actually can ONLY be set in host.json. Configuring it  via .AddFilter has no effect.  Furthermore, when it is set to Information or above in host.json, it applies globally AND overrides all other more verbose log level configurations such as Trace.

For example: If default is set to Information, then logging.AddFilter("System.Net.Http", LogLevel.Trace) will have no effect. Similarly, if default is set to Warning, then all other Information level logs will be suppressed.

If default is set to Trace in host.json, it enables trace level logging ONLY for the Azure Function host related logs (trigger, operation etc). It does not apply globally, so other unconfigured logs will NOT show trace.

However, since Trace is already the most verbose, other log configurations set to Trace, such as the  logging.AddFilter("System.Net.Http", LogLevel.Trace) configuration above, will work.

All other non default log configurations are completely ignored in host.json, regardless if .AddJsonFile is used. These configurations can only be set via .ConfigureLogging as posted by @allanlw. Again they will only work if they're no more verbose then the default set in host.json, which defaults to Information if unset.

Hopefully this helps. @allanlw let me know if these findings match yours.

What @ggailey777 says matches my understanding of the issue. I don't really care where I configure log filters (json or in the code) but the documentation right now is IMO confusing if not outright wrong.

@allanlw What I meant was, from what I observed in my test, this section in your host.json:
"logging": {
        "logLevel": {
            "default": "Trace"
        }

actually did have effects, though it by itself wouldn't enable your other trace logs.
If you don't mind doing a test, change the default level above to Information or Warning, or remove this section completely from your host.json, you should see the logging.AddFilter("System.Net.Http", LogLevel.Trace) configuration in your .ConfigureLogging method stop working.
The behavior of this default filter configuration and its "maximum verbosity" effects on the rest of the filters was described in my previous post.
Furthermore,  loading host.json as json configuration provider seems to have no effect, as suggest by @ggailey777, as the non-default log filters appear to only work if configured via ConfigureLogging.


[BUG] Two way binding is not working for NSTextfield

Describe the bug
When two way binding a ViewModel to an NSTextField. The ViewModel is never updated. However the NSTextField is updated.
Steps To Reproduce

Create a ReactiveViewController.
Bind in the constructor

this.Bind(ViewModel, vM => vM.ClientId, v => v.ClientIdTextField.StringValue);

Expected behavior

When changing the value of an NSTextField. The ViewModel's setter should be hit.
Environment

OS:       MacOS 10.15
Version   11.4.1
Device:   Macbook Pro 16

Additional context
It seems extremely similar to an earlier reported issue. https://github.com/reactiveui/ReactiveUI/issues/1603 . Which was fixed may 2018. However I have been reading the docs over and over. And however it is missing some details about Xamarin.Mac implementations. I think I got it right. I'll post the full ViewController code below.
// This file has been autogenerated from a class added in the UI designer.

using System;
using Moneybird.MacOs.ViewModels.Preferences;
using ReactiveUI;

namespace Moneybird.MacOs
{
    public partial class PreferencesViewController : ReactiveViewController<GeneralPreferencesViewModel>
    {
        public PreferencesViewController(IntPtr handle)
            : base(handle)
        {
            this.Bind(ViewModel, vM => vM.ClientId, v => v.ClientIdTextField.StringValue);
            this.Bind(ViewModel, vM => vM.ClientSecret, v => v.ClientSecretTextField.StringValue);
        }

        public override void ViewDidLoad()
        {
            base.ViewDidLoad();
            ViewModel = RxApp.SuspensionHost.GetAppState<AppState>().GeneralPreferencesViewModel;
        }
    }
}

Using the WhenActivated method has fixed my issue now. I had read before it was good practise, but not mandatory (https://reactiveui.net/docs/handbook/data-binding/#types-of-bindings). But it seems it was mandatory in this case. I just didn't know how it was supposed to be implemented, but the project readme had an good example here https://github.com/reactiveui/ReactiveUI#binding-our-viewmodel-to-the-platform-specific-ui .

@Dylan-DutchAndBold Quick question I was going to ask.  Sometimes (depending on how you construct objects) you have to manually set the BindingContext (ViewModel = new ViewModel()) in the view.  Generally, I put a breakpoint in the ViewModel constructor to verify that it is getting hit when I run into issues like the one you described (literally did this before I saw the notification of this issue).

@RLittlesII Thanks. I did put breakpoints on the getter and setter and noticed that only the getter was being hit, except for the setter being hit by Akavache once. I'm glad it is working now.


Graal 21.3.4 fixing 18 vulnerabilities

Upgrade Graal from 21.2.0 to 21.3.4.
This fixes 18 vulnerabilities in org.graalvm.sdk:graal-sdk: https://security.snyk.io/package/maven/org.graalvm.sdk:graal-sdk/21.2.0
This pull request is against the master branch because the development branch is already at <IP_ADDRESS>.

@julianladisch  the develop branch is "what counts". if it helps, use 1.3.0.RC2 in maven central that passed basic security checks. when we release 1.3.0 final, master will be brought in line. can I close this and the issue ?

and of course for a PR to be accepted, CI has to pass. also read: https://github.com/karatelabs/karate/issues/2009

I don't know what to do about the failing test JsCallTest.testParallel.


Steady maven plugin is throwing UnsupportedOperationException

Describe the bug
When running mvn -Dsteady steady:report it throws an UnsupportedOperationException due to an invalid environment variable key
IMPORTANT: please ensure that you do not refer to any internal or confidential information from your company/organization here. Also, please provide only references to public (open-source) projects on which the issue can be reproduced. Please do not include in the issue description any reference or link to non-public or otherwise proprietary information.
To Reproduce
Steps to reproduce the behavior:
In case of bugs happening on the client (when performing scans):

Clone a java repo
Add profile as per guideline -> https://eclipse.github.io/steady/user/tutorials/java_maven/
Run plugin mvn -Dsteady steady:report
See error

Expected behavior
No error should be thrown
Steady version

Steady version 3.2.5

In case of bugs happening on the client (when performing scans)

OS/version: Windows 10
Java JVM/version or Python version: JDK 11
Steady client (Maven plugin, Gradle plugin, CLI, setuptools plugin): Maven plugin
Console log(s) (with confidential information removed)
2023-04-24 11:27:30,224 [main] [INFO ] shared.util.VulasConfiguration - Added configuration [MapConfiguration] from source [Transient-Config-Layer]
2023-04-24 11:27:30,232 [main] [INFO ] shared.util.VulasConfiguration - Added configuration [SystemConfiguration] from source [System-Properties]
2023-04-24 11:27:30,232 [main] [WARN ] shared.util.VulasConfiguration - Configuration key [ProgramFiles(x86)] removed due to illegal characters
[WARNING] Error injecting: org.eclipse.steady.java.mvn.MvnPluginReport
com.google.inject.ProvisionException: Unable to provision, see the following errors:

Error injecting constructor, java.lang.UnsupportedOperationException
at org.eclipse.steady.java.mvn.MvnPluginReport.(Unknown Source)
while locating org.eclipse.steady.java.mvn.MvnPluginReport
...
Caused by: java.lang.UnsupportedOperationException
at java.util.Collections$UnmodifiableMap.remove (Collections.java:1460)
at org.apache.commons.configuration.MapConfiguration.clearPropertyDirect (MapConfiguration.java:218)
at org.apache.commons.configuration.AbstractConfiguration.clearProperty (AbstractConfiguration.java:505)
at org.eclipse.steady.shared.util.VulasConfiguration.sanitize (VulasConfiguration.java:306)
at org.eclipse.steady.shared.util.VulasConfiguration.addConfiguration (VulasConfiguration.java:272)
at org.eclipse.steady.shared.util.VulasConfiguration.appendInitialConfigurations (VulasConfiguration.java:219)

URL of a public repo that can be used for reproducing (minimal example if possible)

Additional context
Add any other context about the problem here.

The problem lies in VulasConfiguration call to System.getenv() which returns an UnmodifiableMap
sanitize method attempts to clear any unsupported key (e.g., ProgramFiles(x86))
Additionally, there's a ConcurrentModificationException thrown when deleting and iterating through keys. This may also be replicated in testSanitize unit test by adding a new property at the end. For example pc.setProperty("new invalid key", "bar");

Hi @the-brownstone,
the report goal generates a report based on the application data available in the backend as a result of running the other goals. It looks to me that you run such goal without any other first. The minimal requirement for generating a report is the execution of mvn -Dsteady compile steady:app [1] [2].
[1] https://eclipse.github.io/steady/user/manuals/analysis/#bill-of-material-analysis-app
[2] https://eclipse.github.io/steady/user/tutorials/java_maven/#app

@serenaponta thank you for highlighting that. However, upon running command mentioned above, it seems the same error is appearing due to unsupported environment variable key of ProgramFiles(x86).
Here's the shortened and redacted log file:
2023-04-25 09:53:08,598 [main] [INFO ] shared.util.VulasConfiguration - Added configuration [MapConfiguration] from source [Transient-Config-Layer]
2023-04-25 09:53:08,613 [main] [INFO ] shared.util.VulasConfiguration - Added configuration [SystemConfiguration] from source [System-Properties]
2023-04-25 09:53:08,613 [main] [WARN ] shared.util.VulasConfiguration - Configuration key [ProgramFiles(x86)] removed due to illegal characters
[WARNING] Error injecting: org.eclipse.steady.java.mvn.MvnPluginBom
com.google.inject.ProvisionException: Unable to provision, see the following errors:

Error injecting constructor, java.lang.UnsupportedOperationException
at org.eclipse.steady.java.mvn.MvnPluginBom.(Unknown Source)
while locating org.eclipse.steady.java.mvn.MvnPluginBom

1 error
at com.google.inject.internal.InternalProvisionException.toProvisionException (InternalProvisionException.java:226)
at com.google.inject.internal.InjectorImpl$1.get (InjectorImpl.java:1053)
at com.google.inject.internal.InjectorImpl.getInstance (InjectorImpl.java:1086)
at org.eclipse.sisu.space.AbstractDeferredClass.get (AbstractDeferredClass.java:48)
...
Caused by: java.lang.UnsupportedOperationException
at java.util.Collections$UnmodifiableMap.remove (Collections.java:1460)
at org.apache.commons.configuration.MapConfiguration.clearPropertyDirect (MapConfiguration.java:218)
at org.apache.commons.configuration.AbstractConfiguration.clearProperty (AbstractConfiguration.java:505)
at org.eclipse.steady.shared.util.VulasConfiguration.sanitize (VulasConfiguration.java:306)
at org.eclipse.steady.shared.util.VulasConfiguration.addConfiguration (VulasConfiguration.java:272)
at org.eclipse.steady.shared.util.VulasConfiguration.appendInitialConfigurations (VulasConfiguration.java:219)
at org.eclipse.steady.shared.util.VulasConfiguration. (VulasConfiguration.java:140)
at org.eclipse.steady.java.mvn.AbstractVulasMojo. (AbstractVulasMojo.java:91)
at org.eclipse.steady.java.mvn.MvnPluginBom. (MvnPluginBom.java:35)
...
[IJ]-1-MojoFailed-[IJ]-source=CLI-[IJ]-goal=app-[IJ]-id=::jar:1.0.0-[IJ]-error=Failed to execute goal org.eclipse.steady:plugin-maven:3.2.5:app (default-cli) on project : Execution default-cli of goal org.eclipse.steady:plugin-maven:3.2.5:app failed: Unable to load the mojo 'app' (or one of its required components) from the plugin 'org.eclipse.steady:plugin-maven:3.2.5'
[IJ]-1-ProjectFailed-[IJ]-id=::jar:1.0.0
[IJ]-1-SessionEnded-[IJ]-id=unknown
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  7.801 s
[INFO] Finished at: 2023-04-25T09:53:08+08:00
[INFO] ------------------------------------------------------------------------
[IJ]-1-Unknown-[IJ]-event=org.apache.maven.execution.DefaultMavenExecutionResult@2d07aacc
[ERROR] Failed to execute goal org.eclipse.steady:plugin-maven:3.2.5:app (default-cli) on project : Execution default-cli of goal org.eclipse.steady:plugin-maven:3.2.5:app failed: Unable to load the mojo 'app' (or one of its required components) from the plugin 'org.eclipse.steady:plugin-maven:3.2.5': com.google.inject.ProvisionException: Unable to provision, see the following errors:
[ERROR]
[ERROR] 1) Error injecting constructor, java.lang.UnsupportedOperationException
[ERROR]   at org.eclipse.steady.java.mvn.MvnPluginBom.(Unknown Source)
[ERROR]   while locating org.eclipse.steady.java.mvn.MvnPluginBom
[ERROR]   at ClassRealm[plugin>org.eclipse.steady:plugin-maven:3.2.5, parent: jdk.internal.loader.ClassLoaders$AppClassLoader@512ddf17] (via modules: org.eclipse.sisu.wire.WireModule -> org.eclipse.sisu.plexus.PlexusBindingModule)
[ERROR]   while locating org.apache.maven.plugin.Mojo annotated with @com.google.inject.name.Named(value="org.eclipse.steady:plugin-maven:3.2.5:app")
[ERROR]
[ERROR] 1 error
[ERROR]       role: org.apache.maven.plugin.Mojo
[ERROR]   roleHint: org.eclipse.steady:plugin-maven:3.2.5:app
[ERROR] -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/PluginContainerException

@the-brownstone, thanks for reporting and for the detailed description of the issue. This is now fixed in master (version 2.3.6-SNAPSHOT) and will be part of the next release.


Publish for other versions

I am using "play 2.3 + scala.2.11", and there aren't for this combination a repository for this.
Can you publish for this and others combinations in https://oss.sonatype.org/content/repositories/snapshots/com/balihoo/ ?

+1


Workaround on http://bugs.python.org/issue27931

Not sure that is best idea, but I use it on my project.

Merged via #90


Update containers, nf trace compatible now. Some to biocontainers.

This branch has a the following:

Moved over to quay.io (dockerhub has pull limits now)
Migrated some tools to biocontainers, less maintaince work
Added the tools required for Nextflow monitoring and trace

TODO: complete the build.py command to push all the containers.
I would like to use gh actions to push the containers when
we get a new release.
NOTE: it was built on top of https://github.com/EBI-Metagenomics/emg-viral-pipeline/pull/80
Should close https://github.com/EBI-Metagenomics/emg-viral-pipeline/issues/50

This one also closes #74


ENH: allow iMath to use FlatStructuringElement instead of BinaryBallStructuringElement

I was looking into iMathFunctions and it seems as though iMathME and iMathMD use balls of a certain radius, where the radius is defined by the user.
I don't know ITK enough to implement (hence the ENH), but I think using FlatStructuringElement may allow for more flexible kernels (I'm mainly interested in box kernels) for dilation/erosion/smoothing.  If you can smooth/erode/dilate with a box kernel using ANTsR otherwise, this may be a moot point.

A quick look at the documentation suggests that this should be a pretty straight forward implementation. I can take a closer look this week.

Ok, finally made the updates for this. I need to update the relevant code in ANTsR to take advantage of the new options, but all of the updates to ANTs code should be done.

is the idea to add this to antsr?

Yes, working on that addition now.
On Fri, Jun 17, 2016 at 4:15 PM stnava<EMAIL_ADDRESS>wrote:

is the idea to add this to antsr?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/327#issuecomment-226870667, or mute
the thread
https://github.com/notifications/unsubscribe/AAfWiJr6L6h6sWpM3lmhTjHIcH6JkyeQks5qMwBNgaJpZM4Ir2s-
.

the iMath in ANTs is really nothing more than a testing ground for
functions that will end up in ANTsR. It makes the development process
faster: write and test the function in ANTs then once that is good, all
that is needed is a wrapper in ANTsR and updating the ANTs commit version
used in ANTsR
On Fri, Jun 17, 2016 at 4:17 PM Jeffrey Duda<EMAIL_ADDRESS>wrote:

Yes, working on that addition now.
On Fri, Jun 17, 2016 at 4:15 PM stnava<EMAIL_ADDRESS>wrote:

is the idea to add this to antsr?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/327#issuecomment-226870667, or mute
the thread
https://github.com/notifications/unsubscribe/AAfWiJr6L6h6sWpM3lmhTjHIcH6JkyeQks5qMwBNgaJpZM4Ir2s-
.

yeah, plus updating iMathOperations csv file and man file
brian
On Fri, Jun 17, 2016 at 4:19 PM, Jeffrey Duda<EMAIL_ADDRESS>wrote:

the iMath in ANTs is really nothing more than a testing ground for
functions that will end up in ANTsR. It makes the development process
faster: write and test the function in ANTs then once that is good, all
that is needed is a wrapper in ANTsR and updating the ANTs commit version
used in ANTsR
On Fri, Jun 17, 2016 at 4:17 PM Jeffrey Duda<EMAIL_ADDRESS>wrote:

Yes, working on that addition now.
On Fri, Jun 17, 2016 at 4:15 PM stnava<EMAIL_ADDRESS>wrote:

is the idea to add this to antsr?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/327#issuecomment-226870667, or
mute
the thread
<
https://github.com/notifications/unsubscribe/AAfWiJr6L6h6sWpM3lmhTjHIcH6JkyeQks5qMwBNgaJpZM4Ir2s-

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/327#issuecomment-226871683, or mute
the thread
https://github.com/notifications/unsubscribe/AATyfuIRa704OkHMShrtaXHgKBPAdEKzks5qMwFugaJpZM4Ir2s-
.

and updating vignette
On Fri, Jun 17, 2016 at 4:21 PM stnava<EMAIL_ADDRESS>wrote:

yeah, plus updating iMathOperations csv file and man file
brian
On Fri, Jun 17, 2016 at 4:19 PM, Jeffrey Duda<EMAIL_ADDRESS>wrote:

the iMath in ANTs is really nothing more than a testing ground for
functions that will end up in ANTsR. It makes the development process
faster: write and test the function in ANTs then once that is good, all
that is needed is a wrapper in ANTsR and updating the ANTs commit version
used in ANTsR
On Fri, Jun 17, 2016 at 4:17 PM Jeffrey Duda<EMAIL_ADDRESS>wrote:

Yes, working on that addition now.
On Fri, Jun 17, 2016 at 4:15 PM stnava<EMAIL_ADDRESS>wrote:

is the idea to add this to antsr?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/327#issuecomment-226870667,
or
mute
the thread
<

https://github.com/notifications/unsubscribe/AAfWiJr6L6h6sWpM3lmhTjHIcH6JkyeQks5qMwBNgaJpZM4Ir2s-

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/327#issuecomment-226871683, or
mute
the thread
<
https://github.com/notifications/unsubscribe/AATyfuIRa704OkHMShrtaXHgKBPAdEKzks5qMwFugaJpZM4Ir2s-
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/327#issuecomment-226872028, or mute
the thread
https://github.com/notifications/unsubscribe/AAfWiPhjt1Gee9XB9FKv5NHMAZJQjFdBks5qMwHLgaJpZM4Ir2s-
.

Ok, just added this to ANTsR, see the new "morphology" function for details. The structuring elements are also available via iMath, but the morphology function provides a more "R-like" interface.

looks good - would be nice to have more meaningful example(s) in  ?morphology


support reading of properties on implicit block mapping keys

now , the following yaml document can be loaded
&anchor foo:
  foo: bar
  *anchor: duplicate key
  baz: bat
  *anchor: duplicate key

Note that the example is invalid YAML.
There needs to be a space after an alias key before the colon (because a colon can be part of an anchor/alias).
It should be:
  *anchor : duplicate key

pyyaml parses this, however http://ben-kiki.org/ypaste/ (which is specified as reference parser) does not.
Gonna check it again later, but I think we should require a space after colon there?

pyyaml and libyaml can parse it, but that it actually a bug.

And what about object / array anchor merging ?
Is it a valid syntax for Yaml 1.2 ?
"@figen/config-eslint@*":
  dependencies: &eslint
    "eslint": "8.55.0"

"@figen/config-typescript@*":
  dependencies: &typescript
    "typescript": "^5.3.3"

"figen@*":
  dependencies:
    << : {*eslint, *typescript}

Correct would be:
<< : [*eslint, *typescript]

The syntax itself is correct. The parser doesn't know anything about << - it is just a string at this point.
The constructor detects it as a merge key, and that depends on library if it allows this for YAML 1.2. It should not be enabled by default.


Serializing a map with non-string keys will silently generate invalid JSON

For example, a HashMap<u32, u32> will produce invalid JSON when serialized.
At a minimum, the json serializer should error rather than produce invalid JSON.
Ideally however, it would fall back to serializing the map as a list of pairs:
[[0, 1], [2, 0], ...]

we had this issue before with rustc-serialize... I wasn't aware we also had it in serde...
probably we just need an intermediate serializer here: https://github.com/serde-rs/serde/blob/master/serde_json/src/ser.rs#L262 that is passed to the MapVisitor, to check for stringable keys and error else. Fallback to serializing the map as a list of pairs isn't really doable. You should be serializing an iterator to get that behavior.

Good idea @oli-obk! let me whip that up...


Registers @.everyone for a class when anyone registers

https://github.com/joey-colon/lion/blob/master/src/services/class.service.ts

Blocks HEAD from deployment

Blocks HEAD from deployment


Add yearly-weights parameter

Description of the parameter
yearly-weights offers the possibility of assuming that the Monte Carlo years simulated in the ANTARES study are not equally probable. The most representative years may be given greater weight than those that are less representative. The yearly-weights points to a vector (ω_1,…,ω_n ), with n the number of Monte-Carlo years in the study, which is used to evaluate the expected production costs.

With cost_i the production cost of the i-th Monte Carlo year.
Developments
--> Add the yearly-weights  parameter in the settings.ini file in the user/expansion/ folder.
--> This is not a mandatory parameter, it may not appear in the settings.ini (as the additional-constraints parameter), therefore empty by default
--> Assert that the value filled is a string specifying the name of a file
--> Assert that this file exists, is located in the user/expansion/ folder of the ANTARES study.
--> Read and assert that this file contains one column with numerical values (double)

--> Assert that  (all(yearly_weights >= 0))
--> Assert that (sum(yearly_weights) > 0)
--> Assert that the length of the column is equal to the number of Monte-Carlo years in the ANTARES study (the value of the i-th row is the weight of the i-th Monte Carlo year)
If there is a playlist in the study, assert that length(yearly_weights) == length(antares::playlist) and read the weights
If there is no playlist in the study, assert that length(yearly_weights) == length(antares::nbyears) and read the weights
Else stop("Number of yearly weights does not correspond to number of MC years in the Antares study")
Warning : it is up to the user to check the coherence between the weights and the Monte carlo years in the Antares study !
--> If a weight is egal to 0 : don't modify the Antares playlist (MPS of this file are thus generated) and try to modify the lp_namer so that the year does not appear in the structure.txt and is not taken into account in the in the optimization block.
--> Create a weights.txt file in the same location as the structure.txt file and with a similar structure with two columns :
Add one line per sub-problem (so per year and per week) : in the first column add the name of the mps problem (as in the structure.txt file) and in the second column, repeat the weight of the year
Add an extra line with the name WEIGHT_SUM and the sum of the weights

--> Edit the SLAVE_WEIGHT parameter in the options.txt file ( /lp/ folder) with weights.txt instead of CONSTANT :
SLAVE_WEIGHT                   weights.txt
-->Take into account Xavier's pull request for the optimization
https://github.com/AntaresSimulatorTeam/antares-xpansion/pull/91
Example study to test  :
xpansion-test-01-weights.zip

Some questions :

How are the .mps file indicated in the created weight.txt ?

with suffix :
problem-1-1-AAAAMMDD-hhmmss.mps 1
problem-1-2-AAAAMMDD-hhmmss.mps 1
problem-1-24-AAAAMMDD-hhmmss.mps 1

without suffix :
problem-1-1-AAAAMMDD-hhmmss 1
problem-1-2-AAAAMMDD-hhmmss 1
problem-1-24-AAAAMMDD-hhmmss 1

What order should be used for .mps listing ? same order as the mps.txt file created for lp_namer ?

They're indicated without suffix, you have an example in data_test\mini_network/slave_weights.txt.
The order doesn't matter, as you can see in BendersOptions.cpp:82, the weights are read in a std::map<std::string,double>, where the key is the name of the problem.

@MathildeFranconRTE  there is some python code that support yearly_weights option but not yearly-weights.
Can you confirm yearly-weights must be used ?

Yes, I confirm yearly-weights

We're doing some refactoring so there is no problem for changing the name.


Fix, avoid ending up with the wrong merged head values for each token

(originally added as a comment to your youtube video, minor edits here)
Hi! Great series. About the problem with empty text evaluation, I'm not sure about this, but you can try to revert the permutation of dimensions 1 and 2 before reshaping values, at transformer.py:93.
        values, attention = scaled_dot_product(q, k, v, mask)  # value.shape: (B, 8, 200, 64)
        values = values.permute(0, 2, 1, 3)                                # <--- insert this
        values = values.reshape(batch_size, sequence_length, self.num_heads * self.head_dim) # reshaping to (B, 200, 8 * 64)

We need to switch dim 1 & 2, to avoid ending up with the wrong merged head values for each token. E.g., given this code:
        before, attention = scaled_dot_product(q, k, v, mask)  # before.shape: (B, 8, 200, 64)
        after = before.reshape(batch_size, sequence_length, self.num_heads * self.head_dim) # reshaping to (B, 200, 8 * 64)

Without permute(), then the values 'after[B,0,:]', which should be the merged heads for token 0, would be the values from 'before[B,0,0:8,:]', which would be the first head from the 8 first tokens. I´m unsure, but if I am right (I´m still a beginner), that sounds wrong :P
The training result is different, but I don't know if it is correct.
Anyway, thanks for your great ml videos. Always fun watching your content.
Result:
Epoch 6
Iteration 0 : 1.5735915899276733
English: hes a scientist.
Kannada Translation: ಇವರು ಸಂಶೋಧಕ ಸ್ವಭಾವದವರು.
Kannada Prediction: ಇವರು ನಂಬ್ಧಿ್ಮಾಥಾವಗಿ ಾೆ.
Evaluation translation (('should we go to the mall?',)?) : ('ಈಗಾಗಲೇ ಅವರು ಹೇಗೆ ಹೇಗೆ?',)
(feel free to close/delete this pr, I created this due to youtube seemingly blocked my comment there)

sorry, didnt see the already submitted issue, closing pr :)

Thanks so much for the comment! I actually saw this PR and the issue at the same time. I hadn't realized the issue was posted for as long as it was. That was my fault. Regardless, I'll be giving that issue as well as this PR a shout out in my next video. It helps that multiple people saw the same issue. Thanks again for the assist here; and for being a part of the community :)


iOS ReferenceError w/ GADInterstitialDelegate

I'm seeing the following error on app startup:
app/tns_modules/nativescript-admob/admob.js:22:71: JS ERROR ReferenceError: Can't find variable: GADInterstitialDelegate

Sounds like the pod isn't installed (which should happen automatically). What pod --version are your running?


Add hibernate feature

https://rambox.pro/#feature-details/hibernation

I have now implemented this feature in 666ec21. The hibernation can be enabled in the settings and will kick in after 5 minutes - we need to see it this is too short.

@vantezzen Notifications still work in hibernation?

No - during hibernation the service won't be loaded and thus won't be able to send you notifications.


[release-2.1] cherry-pick: Add end_key limit to kv_scan interface (#3720)

What have you changed? (mandatory)
This PR cherry-picks #3720 to release-2.1 branch.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
By CI
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No.
Does this PR affect tidb-ansible update? (mandatory)
No.
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

Please fix conflict.

LGTM.

Can this be merged?

@zhangjinpeng1987 PTAL


Option to hide own character model at certain phases of Warden

Specifically p3 and p4 of warden, but having it be a checkbox for which phases overall might be nice

I understand the desire to declutter during hectic phases, but am not interested in adding this feature to the plugin. It is likely better served as a separate plugin on its own, perhaps as a hotkey.


[zh-cn] Translate mutating-webhook-configuration-v1.md

Trablate content/zh-cn/docs/reference/kubernetes-api/extend-resources/mutating-webhook-configuration-v1.md
https://github.com/kubernetes/website/issues/34733

/retitle [zh-cn] Translate mutating-webhook-configuration-v1.md

preview: https://deploy-preview-34991--kubernetes-io-main-staging.netlify.app/zh-cn/docs/reference/kubernetes-api/extend-resources/mutating-webhook-configuration-v1/

@donghui12 Please use the https://deploy-preview-34991--kubernetes-io-main-staging.netlify.app/zh-cn/docs/reference/kubernetes-api/extend-resources/mutating-webhook-configuration-v1/ URL for preview rather than other software.

I don't know it about this。

I don't know it about this。

After adding empty lines, the problem has been fixed.
/lgtm
/approve


az feedback does not request feedback

The az feedback command asks 2 questions:

would you recommend us 1-10.
what do we do well.

One would expect a 3rd question , what do we do "not so well" which would afford the user an opportunity to vent some frustration or report an issue. Unless you only want positive feedback, that is :-)

Hi @mkiernan the questions asked in the latest az feedback are:
"How likely is it you would recommend our Azure CLI to a friend or colleague?"
"What changes would we have to make for you to give us a higher rating?"
Which version of the CLI are you using?

@tjprescott I just ran az feedback on master and got the following:
$ az feedback
We appreciate your feedback! This survey is only two questions and should take less than a minute.

How likely is it you would recommend our Azure CLI to a friend or colleague? [0 to 10]: 10

What do we do really well?

If you would like to join our insiders program and receive tips, tricks, and early access to new features, let us know by leaving your email address (leave blank to skip):

Thanks for your feedback!

Nevermind... Enter 0 and get this:
$ az feedback
We appreciate your feedback! This survey is only two questions and should take less than a minute.

How likely is it you would recommend our Azure CLI to a friend or colleague? [0 to 10]: 0

What changes would we have to make for you to give us a higher rating?

If you would like to join our insiders program and receive tips, tricks, and early access to new features, let us know by leaving your email address (leave blank to skip):

Thanks for your feedback!

@twitchax what do you think about having 3 questions?

rating prompt
what do you like
what do you not like

After the score, we ask one further question. Depending on the score you give, the question differs.
If score is 10, we ask What do we do really well?.
If score < 10, What changes would we have to make for you to give us a higher rating?.

Got it. What I didalso  was gave it a '10', but I also wanted to write a feedback comment. I think it would be best to ask the question even if the user gives a 10.

I think it may be best to go with @devigned's suggestion here.

"How likely is it you would recommend Azure CLI to a friend or colleague?"
"What does the Azure CLI do well?"
"Do you have any suggestions for improvements to the Azure CLI?"

We will fix this as part of the effort to make az feedback better


Fixed 'sql error 30082'

I am work on moving a node.js app from mongo DB to DB2, so I found your repo. you are awesome and the toolkit really helped. Thank you!
But I found it cannot connect to db2 with right configurations. There is an error:
SQL error 30082- SQL30082N Security processing failed with reason "17"
I don't know whether it is because of the different versions of DB2.
I'm connecting to the db2 in container setup with ibmcloud/db2-s390x image and after updating the db config map, it works.
Please take a look.

Thanks for the correction, sorry I was a little busy before and cannot merge in time.


add custom srcdir and update build docs

with current setup the images are already create in docs/images, but the html file is not
add a nbPostInit custom include (requires a nimib more recent than 0.1, taken current head but fixed commit) that adds a nbSrcDir (books dir). It also renames nbDoc.filename to the relative position with respect to srcDir. In this way also html file is written in docs folder
added a .gitgnore that will ignore everything in docs by default. modified the build script so that this gitignore is removed when building docs. also removed rsync since it is not needed anymore (everything is written in docs folder already)

I think I might add a srcDir mechanism like the above in future versions of nimib to support this workflow out of the box but probably the default srcDir will be named "src".

Many thanks for helping out! 😄 The only problem I seem to have is that Nimble isn't cooperating really. It complains about nimib@#486c22d9cd9c40f32f00efe9fd46630b9bd4d3c7, but when I changed it to nimib#486c22d9cd9c40f32f00efe9fd46630b9bd4d3c7 it works.

always forget that syntax... and I actually looked it up! https://github.com/nim-lang/nimble#nimble-instal

Haha same, I only ever use package@#head so it seems natural to just replace head 🙃
I tried fixing the path at the top of the page by inserting this code into nbPostInit:
nbDoc.context["here_path"] = (nbThisFile.relativeTo nbSrcDir).string

But it doesn't seem to have any effect. I was looking at this piece of code where title is set to context["here_path"]. Any ideas why it doesn't work? 😅

Oh I see why now: https://github.com/pietroppeter/nimib/blob/4926b61190416ab7b52a42cb5644fffd3163b44d/src/nimib.nim#L51-L54
Defaults is initialized before nbPostInit is run.

yes, you are right. You should also add a doc.context["title"] = nbDoc.context["here_path"] to ovveride the default (or directly override the title). Probably the template itself can be improved (to use here_path if a title is not found). In fact also the home link is wrong and it should change to nbDoc.context["home_path"] = (nbHomeDir.relativeTo nbThisDir).string (and this does not require to ovveride other stuff). Will add a commit with this stuff.

had to add also a nbDoc.partials["header_center"] = "<code>" & nbDoc.context["title"].castStr & "</code>" which clearly shows that templates are not being used correctly there, will have to fix that!

Awesome! I feel ready to merge this now unless you have anything more to add 😄

had to add also a nbDoc.partials["header_center"] = "" & nbDoc.context["title"].castStr & "" which clearly shows that templates are not being used correctly there, will have to fix that!

I'm still trying to wrap my head around how Nimib works internally, so correct me if I'm wrong, do I understand it correctly that nbDoc.partials["header_center"] should be a mustache template instead of this manual construction. That way nbDoc.context["title"] won't be used for rendering until at the very end and hence our changes to it in nbPostInit would be visible?

nothing more to add!
yes, you are right, nbDoc.partials["header_center"] should be a template accessing the here_path (more than accessing directly title, which is also used in the head section).
the idea in nimib is that rendering is done only at the end during the save phase and everything can be customized between the nbInit and nbSave. Also, some part of the customization can be done directly through template files without changing the nim file (for example you can add a header_center.mustache file to override the template for all documents.

mmh, the CI for building docs it seems it ran fine but I do not see the changes reflected in the final result...

Hmm yeah me neither... I checked the logs and saw this:
Installing nimib@any version
Downloading pietroppeter/nimib using git
  Verifying dependencies for<EMAIL_ADDRESS>
So it seems it ignores the hash for some reason :/
Okay nice, good to know 👍
That idea really makes for a fully customizable experience and it reminds me of Nim's philosophy that one isn't dependent on the compiler (or Nimib in this case) to have all the bells and whistles you need because there is nothing stopping you from doing whatever you want. 😄 Oh that's cool, I assume it's good if you want to share it across multiple files?

Hmm yeah me neither... I checked the logs and saw this:
Installing nimib@any version
Downloading pietroppeter/nimib using git
  Verifying dependencies for<EMAIL_ADDRESS>
So it seems it ignores the hash for some reason :/

Ok, I will release a new nimib and send a follow up PR.

Okay nice, good to know 👍
That idea really makes for a fully customizable experience and it reminds me of Nim's philosophy that one isn't dependent on the compiler (or Nimib in this case) to have all the bells and whistles you need because there is nothing stopping you from doing whatever you want.

yes, nimib tries to follow nim philosophy as far as I am able to understand it :). The realization that I should just go the nim way ("this is the way") is what actually made me come up with a library and an api instead of a binary file (which will probably eventually come to ease up configuration and scaffolding).

😄 Oh that's cool, I assume it's good if you want to share it across multiple files?

Yes, it could be used to replace the header with some simple navigation common to all pages for example.

I restarted the Action and now it installed the correct version... No idea what happendde

Ok, I will release a new nimib and send a follow up PR.

Greatly appreciated 🙏

yes, nimib tries to follow nim philosophy as far as I am able to understand it :). The realization that I should just go the nim way ("this is the way") is what actually made me come up with a library and an api instead of a binary file (which will probably eventually come to ease up configuration and scaffolding).

Sounds like a really good realization :) And yes some kind of binary that could replace our nimble genbook for example or more generally a mdBook is a nice feature to have in the future.

Yes, it could be used to replace the header with some simple navigation common to all pages for example.

That's neat!


Initial Webpack DLL Support

[x] Enables Webpack DLL for faster Hot Reloading/Reloading

@jhen0409 couldn't get modules: false to work without breaking hot reloading. Lets save that for another PR


[WIP] CI using Travis

.travis.yml will launch a docker image (build+run), for instance build

Looks like caching is not possible for docker:

Docker images are not cached,

Ready to be merged, see https://travis-ci.org/jgphpc/SPH-EXA_mini-app/jobs/600871087


Shell scripts should have a common locale

We should insure all code assures the same locale (which should only mean shell scripts) as it impacts sort order and regex (and probably extglob). Probably documentation and eventually some linter/unit test mechanism.

Agreed. Could you suggest a code template or another mechanism to make this easier to remember & opt-in? Would you be willing to explore adding a unit test and/or GitHub pre-commit hook to catch this?

Bats maybe? https://github.com/sstephenson/bats
Though maybe closer to code smell thing?
I'm not a fan of using cloud services on repos but I could look into what
TravisCI can do if we want?
I'd expect to only see service and build shell scripts,  so there should be
few locations to have bats look if that's the choice. I guess just grep -qE
"^[[:space:]](export )?LC_ALL=C[[:space:]]$"
! grep -qE "^[[:space:]]*(export )?LC_ALL=[^C]"
... probably limit that grep to the first 10 lines or so? That or test for
the sourcing of a central script that everyone knows about and to use that
never gets touched and contains all your sanity - not exactly the same as
seeing green tests for me though.
Also, bats brings some niceties, but if we prefer to keep test stuff in
house, that's not a big deal.
On Thu, Jun 2, 2022, 20:07 nbpion @.***> wrote:

Agreed. Could you suggest a code template or another mechanism to make
this easier to remember & opt-in? Would you be willing to explore adding a
unit test and/or GitHub pre-commit hook to catch this?
—
Reply to this email directly, view it on GitHub
https://github.com/trustOHSNAP/docs/issues/3#issuecomment-1145540546,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWOP2YBFLAUF6SJO5N7CDVNFZHTANCNFSM5XW3DRBQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Expand LXD driver functionality

As the title states I expanded the LXD driver. The default Docker driver doesn't cut it for me because of the missing systemd. Took a try with gdraheim/docker-systemctl-replacement but it comes short sometimes, not suitable for stable testing.
What has changed:

Mainly more mappings of the lxd_container module to allow more flexible usage
The lxd_container option profiles defaults to the default profile when not provided(source) therefore now standard omitted
The prepare playbook needed work since Python is not always installed in LXC containers. Also installation of some basic packages (based on the Dockerfile in the repo). Which for Gentoo took some bash magic for the Ansible required gentoolkit.
Moved the create, destroy and prepare playbooks to the provisioner/ansible/playbooks directory
Fixed a little unrelated typo in molecule init template docs

Testing done:

Ran tox -e py27-ansible26-functional -- -k lxd -v which passed successfully
Created a test repo and did a travis build against all the aliases on the official LXC repo see the travis build. Only UbuntuCore and Arch failed. UbuntuCore is special so that is fine, Arch fails because of this issue.

Two points for discussion:

Do I need to update the LXD driver tests to reflect these new changes? I thought about it but in my opinion it doesn't add much since it only implements the lxd_container module
The lxd_container option source is a dict in the lxd_container module but not in molecule. I thought about changing this but it would break backwards compatibility since alias moves into this dict

Do I need to update the LXD driver tests to reflect these new changes? I thought about it but in my opinion it doesn't add much since it only implements the lxd_container module

Looked into this with help of your comment, I think I implemented it all. Most if it was similar to the vagrant|docker implementation, only the devices argument which has a nested dict where the keys can differ was difficult. See the lxd API docs and the lxd devices docs

The lxd_container option source is a dict in the lxd_container module but not in molecule. I thought about changing this but it would break backwards compatibility since alias moves into this dict

Had some inspiration and moved the source options to a dict with some help of Jinja2. This is the same as the lxd_container module and keeps things more easy for the end user. The part about breaking backward compatibility was not true since there was no variables before (used a custom create.yml myself, that got me confused).

Is there a reason why there is a test/resources/playbooks/lxd? Couldn't we just point to the molecule/provisioner/ansible/playbooks/lxd in molecule.yml for testing? This would dedup these playbooks.

@retr0h Do you know when you have time to review the latest changes? In no means I want to rush you, just curious if everything is alright now :)

@wilmardo the model unit tests are failing.

Also, run tox -e format to reformat the code with yapf.

@wilmardo v2 model unit tests still failing
test/unit/model/v2/test_platforms_section.py::test_platforms_lxd[_model_platforms_lxd_section_...] FAILED                                       [ 62%]
test/unit/model/v2/test_platforms_section.py::test_platforms_lxd_has_errors[_model_platforms_docker_error...] FAILED                            [ 64%]

@retr0h Yeah sadly I know, I struggled with it yesterday but there seems to be an issue on my own laptop since the master branch is failing too. Hopefully this evening I have time to retry on my desktop on which I know it worked before.
Any tips on debugging the tox errors? They seem very non descriptive to me (first time using tox).
Also are there any useful tox commands I (or any other contributor) need to know? I might expand the Testing section of the docs some more, for example only yesterday I found out the schema is Cerberus. Also the LXD test commands are not longer valid (v1 legacy?).

I suggest just running py.test.
$ py.test test/unit/model/v2/ -vvv

You can then see which test failed.

Fixed the unit tests! Pytest helped a lot, I will try to create a PR for some more documentation about testing.
The most of the issue was that in test_platforms_lxd_has_errors the items in the source variable are alphabetically sorted and source itself becomes a dict in a list for some reason.
Thanks for the pointers!

Merging.  @wilmardo is our resident LXD driver community  member.


Failed to parse 1 export - PalWorld

Failed to maintain binary equality error.
Sample Object included.
BP_BuildObject_BaseCampWorkHard.zip

Mappings.zip
Might need the mappings after looking at other issues, added.

Failed to maintain binary equality error.
Sample Object included.
BP_BuildObject_BaseCampWorkHard.zip

Same as u. Please let me know if there is any solution. Thank you

To parse this asset with the latest commit, you need to also export the asset /Game/Pal/Blueprint/MapObject/Components/BP_InteractableBox and place it somewhere where UAssetAPI can access it. You can either reconstruct the Content directory tree or simply place this asset in the same folder as the BP_BuildObject_BaseCampWorkHard asset


Component unmounting after every route change

Let' say we have the following code. I am trying to figure out is the component unmounting previuos component and mount new  will be triggered every time we click on the Link with new route?
Profile, Home, Dasboard class components in which I added console logs for componentDidMount and componentWillUnmount lifecycle hooks. Every transition leads to unmount previous component and mount new. Is it expected behavior? (I added the Dashboard route with function in component attribute -  intentionally because it behaves the same like other and I don't see a difference)

      <Router>

                <Link to="/">Home</Link>

                <Link to="/profile">Profile</Link>

                <Link to="/dashboard">Dasboard</Link>

          <Switch>
            <Route path="/" exact component={Home} />
            <Route path="/profile" render={(props) => <Profile {...props} />} />
            <Route path="/dashboard" component={() => <Dasboard />} />
          </Switch>

      </Router>

The Switch only renders the route that matches. When you navigate to a new location, the old location's matched route will unmount and the new location's matched route will mount.
If you have a nested route (e.g. /profile/user) where the Profile component renders a switch to differentiate between /profile and /profile/user, then navigating between those locations will not unmount the Profile.
function Profile({ match }) {
    return (
      <Switch>
        <Route exact path={match.path} component={BaseProfile} />
        <Route exacth path={`${match.path}/user`} component={User} />
      </Switch>
    );
}

@pshrmn Thanks!


Nix 2.4.7 issue with nix-commands for multiple output path derivations.

Describe the bug
This one's a real oddball.
I first noticed it on a fresh Debian VM. I reproduced the issue on my NixOS box, and 2 separate Debian 11 VMs that were created explicitly for this purpose.
So with a fresh box, adding only curl, sh <( https://nixos.org/install; ) --daemon; ( currently v2.7.0 ) and then adding experimental-features = nix-command flakes to ~/.config/nix/nix.conf, certain packages which have multiple output paths will fail to install with nix profile install nixpkgs#jq, but work fine with nix-env -i jq.
Verifying and repairing the full store had no effect. Forcing --no-registries had no effect. Forcing refresh and everything else I could think of - no luck.
I did the full trace, tried wiping out Nix and starting over. When it kept popping up I spun VMs to make certain. A partial trace for the failure is below, but suffice to say that's it's lengthy and not really interesting to dig through.
What is interesting though is that this works:
nix profile install nixpkgs#jq.{bin,doc,man,dev,lib,out}
More interesting still is that at any given member of the output chain, it always almost gripes that the previous member is unavailable or within ~2 members. This tells me "references between output paths are not properly caught by the generic builder".
Steps To Reproduce
Steps to reproduce the behavior:
$ sh <( curl -L https://nixos.org/install; ) --daemon;
...

$ cat <<EOF > ~/.config/nix/nix.conf
system = x86_64-linux
experimental-features = nix-command flakes
build-users-group = nixbld
EOF

$ nix profile install nixpkgs\#legacyPackages.x86_64-linux.jq    \
              --refresh --impure --verbose --show-trace --debug  \
              --sync-before-registering;
...
instantiated 'hook' -> '/nix/store/80mc0ycwsg25ll205jr403w96c6vq731-hook.drv'
instantiated 'CVE-2019-13232-1.patch' -> '/nix/store/2jaq3wj2schdhdhr9f099wbfngs68m75-CVE-2019-13232-1.patch.drv'
instantiated 'CVE-2019-13232-2.patch' -> '/nix/store/r0m0zn5s8jk11nlwg0lsasvsp25p4iza-CVE-2019-13232-2.patch.drv'
instantiated 'CVE-2019-13232-3.patch' -> '/nix/store/icqkc30jw51bwgdyfk14wpy5i2pm2ap0-CVE-2019-13232-3.patch.drv'
instantiated 'unzip60.tar.gz' -> '/nix/store/mxycw8zj6d5rarx1fhxkkb1lacyd9gch-unzip60.tar.gz.drv'
instantiated 'unzip-6.0' -> '/nix/store/749svxn13d6bsg6p4chbs5av6p206rcl-unzip-6.0.drv'
instantiated 'source' -> '/nix/store/wzc8pwx06x4f567bscx2g1652in05s2x-source.drv'
instantiated 'onig-<IP_ADDRESS>' -> '/nix/store/h24s8rf1i2ibq2ch790yfm7fjas9qxdv-onig-<IP_ADDRESS>.drv'
instantiated 'hook' -> '/nix/store/8sv7g5b2r684hzx324zb8cv63s0gyg5s-hook.drv'
instantiated 'hook' -> '/nix/store/wdpi0jzxa59qclw916c1dvda9qz9bygi-hook.drv'
instantiated 'patchutils-0.3.3.tar.xz' -> '/nix/store/p513asp5216fpa1wsr18qf0wr7vg2w74-patchutils-0.3.3.tar.xz.drv'
instantiated 'patchutils-0.3.3' -> '/nix/store/y1mnhbn2pyx5jn782wv84p97xpx5d62c-patchutils-0.3.3.drv'
instantiated 'fix-tests-when-building-without-regex-supports.patch' -> '/nix/store/dadkz8vawvm8dmiy5aj1vlg8rqs8f3k8-fix-tests-when-building-without-regex-supports.patch.drv'
instantiated 'source' -> '/nix/store/0img5x7lnkxk306ny7ivypjx957lwsgb-source.drv'
instantiated 'jq-1.6' -> '/nix/store/p2rd53ylw9ydffrf23r2lk1riqdy2vy3-jq-1.6.drv'
evaluating uncached attribute legacyPackages.x86_64-linux.jq
evaluating uncached attribute legacyPackages.x86_64-linux.jq.outputName
error: opening directory '/nix/store/li593k0xg3drb4bsh3l0avjq0nz0gq05-jq-1.6': No such file or directory
download thread shutting down

Expected behavior
jq outputs should install all outputs "to the left of" the named outPath in this case out.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
I tried a million of the old tricks to repair/ixup the store, restart the daemon, etc, etc. Honestly I think the most likely situation is that something has changed in the latest nix-v2.4.7 primops, causing misbehavior in this edge case.
jq was not the only effected package, but I only encountered issues with packages who define multiple output paths.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
$ nix-shell -p nix-info --run "nix-info -m";
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.0-13-amd64, Debian GNU/Linux, 11 (bullseye), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.7.0`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nix-2-4-7-issues-handling-multiple-outputs/18597/1


Fix components documentation example.

todo in this example is undefined as written, due to the destructuring. This patch fixes it.

@lambdatastic Looks good, thanks! 👋

@lukejacksonn @jbucaran
Does Travis CI only trigger when JS files change? It should fire on any change 🤔
Example here: I only changed the README and Travis CI still triggered
I'll open up an issue (maybe we hit a limit of some sort?)!


Landsat products projection problem in southern hemisphere

arosics version: Any
Python version: Any
Operating System: Any

Description
I tried to co-register a Landsat-8 collection-2 level-2 surface reflectance image to a Sentinel-2 MSI level-2 image, both taken in South Africa via Google Earth Engine. But I got this error:
RuntimeError: Input projections are not equal. Different projections are currently not supported. Got WGS 84 / UTM zone 36S vs. WGS 84 / UTM zone 36N.
I found that while GEE Sentinel-2 products have proper UTM zone designation for northern or southern hemisphere, the GEE Landsat-8 collection surface reflectance products are all designated by UTM north projections even if the scene is for the southern hemisphere (see https://gis.stackexchange.com/questions/269004/wrong-projection-from-landsat-8-surface-reflectance-collection-in-gee). Although they modified the false_northing parameter to place the image correctly on display, but it makes the AROSICS program to fault.
P.S. I checked Landsat 8 TOA products and it had the same problem.

Well, the only thing I could do here is to somehow read the false-northing parameter to correct for the hemisphere. But I have to check if that makes sense and can be implemented robustly.
The output of gdalinfo /path/to/file/with/wrong/hemishere might help with that.


RFC: allow [TEST] label for conventional commit

Right now we only allow

performance, enhancement, bug, chore, documentation, dependencies

However, sometimes I want to add a PR that only has tests.
I experience this a lot in Spark Connect when I realize a feature
doesn't need any extra implementation because it has already been
implemented by another PR.
I want to add the test PR to clarify that we have completed implementing df.{feature}.
Proposal: Add a new type of commit that is only for tests. Probably: [TEST]
https://github.com/Eventual-Inc/Daft/blob/ec39dc01ff6879ef2cff43d8f577cacfaaca1cf7/.github/release-drafter.yml#L3-L28
https://github.com/Eventual-Inc/Daft/blob/ec39dc01ff6879ef2cff43d8f577cacfaaca1cf7/.github/workflows/pr-labeller.yml#L50

Not particularly opposed to this, but is there a reason you wouldn't include your tests with the relevant feature or bug fix PR? You can also consider using [CHORE]

Go for it. Also [CHORE] is probably fine.

Not particularly opposed to this, but is there a reason you wouldn't include your tests with the relevant feature or bug fix PR? You can also consider using [CHORE]

https://www.conventionalcommits.org/en/v1.0.0/

types other than fix: and feat: are allowed, for example @commitlint/config-conventional (based on the Angular convention) recommends build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others.

People often have test (although not required).
Chore is probably the closest we have right now
Here is a subjective cheat sheet someone made https://gist.github.com/Zekfad/f51cb06ac76e2457f11c80ed705c95a3?utm_source=chatgpt.com
They define:

chore: Changes which doesn't change source code or tests e.g. changes to the build process, auxiliary tools, libraries
test: Adding missing tests or correcting existing tests

I think I agree with @universalmind303 here. If we want to take any actions, we should probably go with #2867.
Closing in favor of that issue


Pin http_file and http_archive resources in WORKSPACE

This was a tidy up after noticing warnings when running bazel

Thanks~


Maccy After being used in the background for many times, the memory occupation increases linearly

Operating environment

Device: MacBook Pro 14`2021
macOS: Monterey 12.3
Maccy Version: 0.22.2

Describe the problem
After running Maccy for a long time and using it for many times, the memory occupation of the application will gradually increase (it seems to grow linearly and there is no sign of release). It can be recovered after restarting the application.
I hope the author can fix this problem, thank you.

Device: MacBook Air (M1)
macOS: Monterey 12.3.1.
Maccy Version: 0.22.2
In my case Maccy takes up 1.38 GB.

The way Maccy uses Core Data could probably be improved to avoid memory bloat - right now every new copy is kept in-memory so if you copy large images (for example) - it can increase over time. I need to see if there is a way to make Core Data work in a different way by offloading unused items from memory.

Alex thx for this.
From the behaviour I've seen on my machine, it appears that references even to deleted items are being retained. The memory usage remains the same even after deleting all items by doing a 'Clear'.
Shutting down & restarting Maccy is the only way to reclaim the memory.

What's the status of this issue?

No progress at the moment.

Is it the same how Microsoft Windows store (Windows+V) clipboard items, via memory?
I'm thinking if there should be an option to store locally or via memory.

Maybe users have set a large history size? And a lot of images copied.(Just random guess)

Maybe users have set a large history size? And a lot of images copied.(Just random guess)

It's not the case: I can reproduce with small text entries also. Just wait some time and the memory increase progressively

it often gets dangerously close to 2gb...

cring😭 1.2GB

 cring😭 1.2GB
3.8GB now

 cring😭 1.2GB

3.8GB now

@tony-swg Do you copy lots of images or just text?

@tony-swg Do you copy lots of images or just text?

yes, both

After updating my system to Sonoma and Maccy to latest version, with each opening (command shift c) the memory usage increases by around 300kb

Same issue

4 GB...

Folks, I get it - there is no need to post screenshots of RAM usage. It would be more helpful if someone would actually suggest ways to reduce the memory overhead or, ideally, send a PR.

Folks, I get it - there is no need to post screenshots of RAM usage. It would be more helpful if someone would actually suggest ways to reduce the memory overhead or, ideally, send a PR.

I have no idea what cause this problem. But I suggest you could publish a build with regular relaunch the main program for quick fix.

Has this been fixed yet?

Memory leak is draining my Ram!

I suggest a temporary solution - restart the program every hour using crontab.
$ crontab -l
* 1 * * * killall Maccy; /Applications/Maccy.app/Contents/MacOS/Maccy &

To add a task to the crontab use crontab -e

For me the RAM usage was ~12 GBs, Quitting didn't help.
So I deleted everything and reinstalled it.

I appreciate this software and hope this issue can be resolved promptly. As this software has been developed and maintained by the author for a long term, very few people are familiar with this project besides the author. While waiting for contributions from the open source community may not be the most effective solution for a quick fix, I also hold hope that a kind soul might come forward to address this issue.

Is it tried to free some history from memory (and save it to disk) when the memory usage exceed a limit? If not, I think this can be a temporary solution.
And maybe it's possible to ask Microsoft that how win + v works when user copying lots of image.

Folks, please share how things work for you on Maccy 2.0+

Folks, please share how things work for you on Maccy 2.0+

I'd say it's definitely better, well done with the rewrite and thank you so much for all the efforts!
I'd still say that it's way over what I'd expect, and when restarting it falls down to ~50MB, so maybe not quite solved?

Mine is the same

A little more robust workaround than https://github.com/p0deje/Maccy/issues/384#issuecomment-2143459955 that avoids killing Maccy while currently open:
https://gist.github.com/kra-mo/e404d594d3d1c18e4db37013c9bec694
Just put it in ~/Library/LaunchAgents ^^

I noticed that the Memory issue appeared once I closed the lid of my macbook. Before that it was under 70 MB which is very reasonable. Maybe when the laptop wakes up from sleep ?

Hi. I've noticed that the memory usage shot up to 600MB+ after my macbook pro wake up from sleep. I had checked the memory usage 5 minutes before and Maccy was idling at a very reasonable 70 MB. Maybe this helps ?

This still an issue using over 6 GB on my M3 MacBook Pro:


Update canonical JSON library

@lukehinds pointed this out in the OpenSSF slack.

I don't know if this is the right channel, but wanted to discuss use of canonical-json , its what's used to canonicalise SLSA attestations (cjson) in the verifier code base. As someone developing an app that validates SLSA I elected to use the same library to parse out payload metadata. The thing is the project is hosted here: https://github.com/docker/go and it basically curls down the source files and un-tars them into the repo. The repo has not been updated in 8 years, so its pulling down 1.16 code and 1.16 is EOL. https://github.com/docker/go/blob/d30aec9fd63c35133f8f79c3412ad91a3b08be06/Makefile#L13-L18

The go-securesystemslib/cjson library is supported by the in-toto/TUF teams and while it doesn't look to have a large test suite or handle all edge cases, it would work well enough for our purposes as we only care about canonicalizing a DSSE envelope.
https://github.com/slsa-framework/slsa-verifier/blob/b01cb9d69cb87adbd1491f55ff9e54b399ed560d/verifiers/internal/gha/rekor.go#L326

go-securesystemslib/cjson is also used by cosign and intoto

https://pkg.go.dev/github.com/secure-systems-lab/go-securesystemslib/cjson?tab=importedby


Introduce blocker webhook to delay workspace creation until real webhooks are created

What does this PR do?
This PR introduces a blocker webhook that delays workspace creation until real webhooks are created. Basically what will happen is if you try and do oc apply -f samples/cloud-shell.yaml you'll get a response of:
Error from server (InternalError): error when creating "samples/cloud-shell.yaml": Internal error occurred: failed calling webhook "mutate.che-workspace-controller.svc": Post https://workspace-controller.che-workspace-controller.svc:443/mutate?timeout=30s: dial tcp <IP_ADDRESS>:8443: connect: connection refused.
until the actual webhook server is connected. Once the webhook server is started then you'll be able to create a dev workspace successfully.
What issues does this PR fix or reference?
https://github.com/eclipse/che/issues/17200
Is it tested? How?

Webhooks enabled, then create the cert gen image make docker_cert then the regular controller image make docker then make deploy. Right after you run make deploy start doing oc apply -f samples/cloud-shell.yaml

/retest

/retest

locally tests are finished successfully after specifying image built from that PR.
I'm merging it and e2e tests are going to be improved to avoid such failure in the scope of https://github.com/eclipse/che/issues/17264


Polling should not start for analysis nodes that don't have a URL yet

Closes CartoDB/cartodb#7319.
@viddo, this is the replacement for CartoDB/cartodb.js#1260. Basically, analysis nodes might be created/defined using vis.analysis.analyse, but they might not have been sent to the Maps API for some reason (eg: they are not linked to a layer/dataview yet) so they should not be polled.
cc: @xavijam @juanignaciosl

👍


Update integrate-ibmi-apps-distributed-program-calls.md

IMB i connector ---> IBM i connector
It seems as a typo.

Looks good. Thanks @shigeooooooooooo


bug(mat-expansion-panel): Nested expansion panels will break the lazy rendering

Reproduction
Use StackBlitz to reproduce your issue:
https://components-issue-yzmq99.stackblitz.io
Steps to reproduce:

Open Expansion Panel Header A
Look at the text displayed under the second level expansion panels. Here you can see the content of the expansion panel of Item A1

Expected Behavior
Each nested expansion panel should only display and render its own content when opened (lazy rendering).
Actual Behavior
By nesting two expansion panels, the content of the <ng-template> tag is already displayed when the outer expansion panel is opened.
Environment

Angular: 9.1.2
CDK/Material: 9.2.1
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): macOS

Duplicate of https://github.com/angular/components/issues/14365.


[#480] Fix inheritance problems regarding ambiguous constructors

Description

Related Issue

#480
Motivation and Context

Great that this is fixed 👍

Merged b900a81c80fb57d9e117d05e1140f725ff38c45b


GDPR Webhook Integration

Our app was rejected by Shopify's automated submission test immediately after submission with the same reason of

App must verify the authenticity of the request from Shopify.
Expected HTTP 401 (Unauthorized), but got HTTP 405 from https://8235cf20c428.ngrok.io/webhook/gdpr/shop_redact. Your app's HTTPS webhook endpoints must validate the HMAC digest of each request, and return an HTTP 401 (Unauthorized) response when rejecting a request that has an invalid digest.

Error I received
TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received undefined in
at Hmac.update (internal/crypto/hash.js:84:11)
at receiveWebhookMiddleware( \node_modules@shopify\koa-shopify-webhooks\build\cjs\receive.js:32:63 )
at dispatch( \node_modules@shopify\koa-shopify-webhooks\node_modules\koa-compose\index.js:42:32 )
at bodyParser \node_modules\koa-bodyparser\index.js:95:11)
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at \node_modules\koa-mount\index.js:58:5

You can check the code here.
https://github.com/akeans-mgs/mgs_testing

This line appears to map to passing the secret into createHmac. - The error being that the value is undefined rather than a string. That value comes from what what you set as secret  when calling receiveWebhook({secret}).
In your code that seems to be the value process.env.SHOPIFY_API_SECRET. Are you sure your're setting that environment variable correctly? I suspect you're not, and you're doing receiveWebhook({secret: undefined}), which is an error.

@BPScott
Hi,
Thanks for your quick replay. I am sure that, environment variables are added correctly.
https://github.com/akeans-mgs/mgs_testing/blob/main/server/server.js#L22
in this line also, i am using process.env.SHOPIFY_API_SECRET to initialize shopify context.
And My App GDPR Webhook endpoints are like below

@BPScott
Thanks for reviewing.
You can clone the repo : https://github.com/akeans-mgs/mgs_testing.git
And check it out too.

You can clone the repo

This wouldn't help. I don't know what value you set the SHOPIFY_API_SECRET environment variable to (and neither should you tell me), thus I wouldn't be able to test this.
What happens if you change the line https://github.com/akeans-mgs/mgs_testing/blob/main/server/server.js#L85:
- secret: process.env.SHOPIFY_API_SECRET,
+ secret: process.env.SHOPIFY_API_SECRET || '123456',

Do you get the same or a different error?
What is output if you do console.log({secret: process.env.SHOPIFY_API_SECRET}) at some point in your code?
I remain convinced that this is erroring because you're passing undefined into receiveWebhook's secret key.

@BPScott
I have tried to output the console.log({secret: process.env.SHOPIFY_API_SECRET}) at some point in server.js file. it returns the value of SHOPIFY_API_SECRET in .env
I have changed the line as you said  https://github.com/akeans-mgs/mgs_testing/blob/main/server/server.js#L85
I received the same error.

TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received undefined
at Hmac.update (internal/crypto/hash.js:84:11)
at receiveWebhookMiddleware (D:\xampp\htdocs\2021\shopify-app\mgs-testing\mgs_testing\node_modules@shopify\koa-shopify-webhooks\build\cjs\receive.js:32:63)
at dispatch (D:\xampp\htdocs\2021\shopify-app\mgs-testing\mgs_testing\node_modules@shopify\koa-shopify-webhooks\node_modules\koa-compose\index.js:42:32)
at bodyParser (D:\xampp\htdocs\2021\shopify-app\mgs-testing\mgs_testing\node_modules\koa-bodyparser\index.js:95:11)
at processTicksAndRejections (internal/process/task_queues.js:95:5)
at D:\xampp\htdocs\2021\shopify-app\mgs-testing\mgs_testing\node_modules\koa-mount\index.js:58:5

I asked you to clone the repo and create a dummy shopify app  from your end to test the webhook code.

Hi All,
Just waiting for your response. kindly give me any update.

It looks like the error is caused by the cleartext rawBody being passed to the HMAC digest rather than the key itself. When creating the HMAC with an undefined cleartext we get the same message
> createHmac('sha256', '123456').update(undefined, 'utf8').digest('base64')
Uncaught:
TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received undefined
    at __node_internal_captureLargerStackTrace (node:internal/errors:464:5)
    at new NodeError (node:internal/errors:371:5)
    at Hmac.update (node:internal/crypto/hash:105:11) {
  code: 'ERR_INVALID_ARG_TYPE'
}

Whereas if we specify an undefined key we get a slightly different error message:
> createHmac('sha256', undefined).update('asdf', 'utf8').digest('base64')
Uncaught:
TypeError [ERR_INVALID_ARG_TYPE]: The "key" argument must be of type string or an instance of ArrayBuffer, Buffer, TypedArray, DataView, KeyObject, or CryptoKey. Received undefined
    at __node_internal_captureLargerStackTrace (node:internal/errors:464:5)
    at new NodeError (node:internal/errors:371:5)
    at prepareSecretKey (node:internal/crypto/keys:570:11)
    at new Hmac (node:internal/crypto/hash:132:9)
    at createHmac (node:crypto:162:10) {
  code: 'ERR_INVALID_ARG_TYPE'
}

This would suggest that the API key is likely being set correctly, and instead the rawBody value which should be accessible on the request from koa-bodyparser is not being properly set.

The parsed body value is only available if the content type that's provided specifies that the request is either json or form.
Can you try adding a console.log to output the value of ctx.request.type before passing it into the receiveWebhook and let us know what it says? I can see from the stack trace that you're running this application behind XAMPP and it seems possible that the configuration of that web server is stripping or modifying the content-type header.

I have the same problem and this is not only for GDPR webhook but for all types of webhooks.
I noticed that receiveWebhook function from koa-shopify-webhooks uses rawBody from ctx.request to generate Hmac and this undefined and causes this error:
const {rawBody} = (ctx as ParsedContext).request;

const generatedHash = createHmac('sha256', secret)
   .update(rawBody, 'utf8')
   .digest('base64');

I am using koa-better-body npm package and I checked ctx.request.type and it is application/json .
Does someone resolved this problem?


[FEA] Faster transition from columns to rows

Is your feature request related to a problem? Please describe.
The current columnar to row conversion code pulls back columnar data to the CPU and then walks through it.  We have seen a lot of issues with this and the cache.  It can be a real performance issue.
In the past we tried to create the unsafe row format for fixed width types using a cuda kernel and it worked.  The problem was that it used up a lot of memory. It might be good to explore some kind of a hybrid approach where we can create a more compressed row based format that can very easily be expanded out into unsafe row on the fly on the CPU, possibly using some code generation.

Duplicate of #507


Enable saving, loading and pushing for multiple Keras models in one repository

I started working on this previously, so it was another branch, will definitely fork for all my next PRs. This PR solves #533
push_to_hub_keras() and save_pretrained_keras() now supports multiple models through passing them via a dictionary, like so:
(sharing only the parts affected)
push_to_hub_keras(
    {"model_1": model_1, "model_2": model_2},
    repo_path_or_name=f"{WORKING_REPO_DIR}/{REPO_NAME}",
    config={
                "model_1": {"num": 7, "act": "gelu_fast"},
                "model_2": {"num": 7, "act": "gelu_fast"},
            },
    multiple_model_save_args={
        "model_1": {"save_traces": False},
        "model_2": {"save_traces": True},
    },
)

So model can be either a Keras model or dictionary of Keras models. You can apply different save args for different models through multiple_model_save_args (which is completely optional) and config works the same as model dict.
This creates a directory like so:

You can load with from_pretrained_keras and it returns a dict of models if there are multiple models in the repository:
models_dict = from_pretrained_keras(dir)
Which looks like the one we've pushed:
{"generator": some keras model, "discriminator": other keras model}
Model card has a shared part at the top:

And model specific parts at the bottom:

See this example repository.
I wrote four tests: from_pretrained_keras that checks if the values in returned dictionary are Keras model classes, save_pretrained_keras that checks if the model folders are there and the required files are inside model folders, and two tests forpush_to_hub_keras that checks if the pushed models are there and another one for if the kwargs are injected into tf.keras.models.save_model() for each model.
There seem to be many commits because it was another branch based on model card PR, but there actually is two commits on top of that.
There's no deprecation whatsoever but I might add some warnings for when people are pushing only one model and are passing values to multiple_model_save_args and others.

@nateraw I saw you noted down to get rid of config for from_pretrained_keras() would you like me to do that? I think it doesn't make any sense and not used anyway.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

"I love resolving merge conflicts" said by no one. ever.

I wonder if we can use builder to get rid of dict of dicts.

@osanseviero don't you need discriminator if you'd like to train again, I never needed to do that so I thought I'd ask. I don't know why but as of now, tf.keras.models.save_model doesn't support serial graph saving that's why I did that. If you want I can look for a fix for this.

Yes, I've seen people saving a model as a single one with TF before, so I would maybe try that way.

https://stackoverflow.com/questions/57812324/saving-and-loading-multiple-models-with-the-same-graph-in-tensorflow-functional for example

pinging @ariG23498 here 🤩

What is the update on this? I am sorry I could not reply any sooner.


Added aswf-docker ideas

@aloysbaillet , can you please add the DCO sign-off and re-push? It won't let me merge without it.

Of course, sorry about that!


topics: add open-policy-agent

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

For the sake of transparency, I am an active contributor to the OPA project, but not a maintainer/creator/contractor/employee. I wasn't sure if that constituted an "affiliation" per the verbiage above.
Which change are you proposing?

[x] Curating a new topic or collection

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Open Policy Agent is a widely adopted, Cloud Native Computing Foundation graduated project.

My local ruby environment is wonky right now, so I'm relying on the PR for CI.  Once someone approves the workflow run I'll check the results and make any necessary tweaks.

Thanks very much for your contribution, @gshively11! The video looks great. 💖


17.03 cherry picks

https://github.com/docker/docker/pull/29497 (6138a02c8739045a2f6dec85fd0d89db9bed43e5)
https://github.com/docker/docker/pull/30776 (f5116c680207a12ecd9c4fa654a1a5707e7ac89c)
https://github.com/docker/docker/pull/30893 (eaffb3099a4ed2701fc0934b39b7dcdd1ca62219)
https://github.com/docker/docker/pull/30223 (0566f3ffc46fbdf020e957fc30f56960152e057e)
https://github.com/docker/docker/pull/31055 (7761c69e239171c6b388d53d03619eacc49338a0)
https://github.com/docker/docker/pull/30113 (e74623d283111dc6492b1a97a8299a9377794b99)
https://github.com/docker/docker/pull/31006 (a12454d7a964b99921adabdb8f741da5a94a0e66)
https://github.com/docker/docker/pull/31070 (39851eac0c655d0c1911db58d4f630e2da233cac)
https://github.com/docker/docker/pull/31103 (e1839ab95341fe8c40d8e8e0112fcfab38c92f3f)
https://github.com/docker/docker/pull/31141 (6486924f0b6464a90c9c2d251a3ab0a96302dce9)
https://github.com/docker/docker/pull/29702 (22472c8be50ca0ddaf8207d46c6a9f5e792bd0af)

ping @vieux @cpuguy83 @vdemeester ptal

LGTM


how do I stop it from auto launching on osx?

I would like keybase to not autolaunch, but I do not see a preference for this or an uninstall option.
my log id: 839d1d1792e6521349d04d1c
-brewster
<EMAIL_ADDRESS>

@brewsterkahle See #3311

thank you.  I think I have now uninstalled for now.
-brewster
On 12/5/16 12:54 AM, Jesse Jackson wrote:

@brewsterkahle https://github.com/brewsterkahle See #3311
https://github.com/keybase/client/issues/3311
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/issues/5012#issuecomment-264798520,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACk_2Yy9aBBd6DJPWEm-22gUT2GcB40cks5rE9FfgaJpZM4LDNu5.


As a user, I want to be able to add an inventory and associate it with myself or any of my organizations

Inventory.owner_id and Inventory.owner_type = [:user, :organization] which is therefor a "polymorphic" rails association could be an option.

I'm working on this right now

I assume this is only organizations you own, right?  You shouldn't be able to add any inventory to any organization you are just a part of.

I don't know, it could make sense someday to have either/or based on a flag.

o.k. I'll keep it as just owner for now.  Perhaps in the future we can put in a organization level like for inventories (read, write, admin).  If you have admin access you can create an inventory and associate it to the organization.

I tried to make an array with the current user and the owned inventories of the user for the owner of the inventory but wasn't able to get it to work.  Is that possible?
For the time being I am falling back to nested routes to grab the owner from the url.

You might need to make user.all_inventories that merges org invs and user.inventories (or something similar)

Don't think you understand my question.  User.inventories will already grab all inventories it owns (and is part of since we will add the user to inventory.users on create).
My issue is assigning an inventory an owner either through the form or on create.
Do we want to nest the inventories in this project?  For example if a user creates an inventory do we want it to live in /users/user_id/inventories/inventory_id.  Or just /inventories/inventory_id.  This is how Github works with repositories with either /organization or /user and I think it makes sense.
If that is the case, you can ignore my question since I was trying to get the owner functionality working in the /inventories/new route and wasn't sure how to pass in the owner in the form.


feature(ci): Create main.yml

Fixed #9

I think it would be better to just run the builds separately in parallel.


Add hascolumn() template function

I added a template function named hascolumn().
For example, if you want to update only the updated_at column
if the table has updated_at column.
// templates/mysql.type.go.tpl

/ Update updates the {{ .Name }} in the database.
func ({{ $short }} *{{ .Name }}) Update(db XODB) error {
	...

  {{ if hascolumn .Fields "updated_at" }}

  {{$short}}.UpdatedAt = time.Now().Unix()

  {{ end }}

  ...
}

I rewrote this, as what was submitted would work when column names need to be escaped.
I also added a "hasfield" template helper func. I hope these will be able to help you on custom templates.


Code snippets

Share and comment code snippets.

Looks like markdown supports code highlighting: https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet#code, so I think #43 will solve this one too.

Yep. This might be related!

Done.


doc tweaks

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d698cba) 98.23% compared to head (504c548) 98.23%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #9   +/-   ##
=======================================
  Coverage   98.23%   98.23%
=======================================
  Files           1        1
  Lines         113      113
=======================================
  Hits          111      111
  Misses          2        2

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


Missing Icon in Latest Version: TextBox

Describe the bug
I don't know if this is the right place to report this, but I'm using this library and noticed that the TextBox icon listed on the website as available since 1.4 seems to be missing. I was using 2.0.15 but upgraded to 2.1.7 to make sure it wasn't a version issue.
I don't think I'm missing something, but if I am or if this is the wrong place, please let me know.
Steps to Reproduce
Attempt to import the TextBox icon from the package.
Expected behavior
You can successfully import it, because it exists.
Screenshots

Context (please complete the following information):

OS: MacOS Sonoma
Browser/Environment: Chrome
Library: @phosphor-icons/react
Version: 2.1.7

Actually nevermind, I was capitalizing it as "TextBox" when it's actually "Textbox" ugh.


Show : winrt::hresult_error  when using WebView2

[Enter feedback here]

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

ID: 6747d603-e773-dec7-7580-f18f4365cec9
Version Independent ID: 5f7cd45b-5739-ad85-4b61-c9f94921df98
Content: Get started with WebView2 in WinUI 3 (Windows App SDK) - Microsoft Edge Development
Content Source: microsoft-edge/webview2/get-started/winui.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

forget to do Step 0 .....  after install edge beta, fix it.