Docker containers should run with non root users

A docker best practice is to create a new user and group without super user privileges and run your application with them. We should do that here, too.
Tasks:

[ ] research docker best practices
[ ] update the oba/Dockerfile file with the result of research into how to best run OBA server container without super user privileges.
[ ] update the bundler/Dockerfile file with the result of research into how to best run transit data bundler container without super user privileges.
[ ] Validate that everything works by creating a new bundle, launching the oba_app container, and running bin/validate.sh

Docs:
The official Docker instructions on this topic are extremely brief (and frankly kind of unhelpful), but they represent a good jumping off point to find other guides on how to do this around the web: https://docs.docker.com/develop/develop-images/instructions/#user

To run Docker containers without superuser privileges, you can follow these best practices:
Create a non-root user: Use the USER directive in your Dockerfile to create a non-root user for running the container. Ensure this user has the necessary permissions to execute the required commands.
Use a non-privileged port: Avoid using privileged ports (ports below 1024) for your applications, as these require superuser privileges. Use ports above 1024 instead.
Minimize the use of sudo: Limit the use of sudo within your Dockerfile, as this requires superuser privileges. Try to perform the necessary actions without elevated permissions.

We can also use these practices to run Docker containers without superuser privileges:

Create a Non-Root User in Dockerfile: Docker containers, by default, run as root, which is equivalent to the root user of the host machine. To mitigate this, you can create a non-root user in your Dockerfile and switch to it before executing your application.

Use USER Instruction: The USER instruction in Dockerfile specifies the default user for the container. This user will be used for any RUN, CMD, and ENTRYPOINT instructions that follow in the Dockerfile.

Least Privilege Principle: Opt to create a dedicated user and a dedicated group in the Docker image for the application. Use the USER directive in the Dockerfile to ensure the container runs the application with the least privileged access possible.

Leverage User Namespaces: Docker allows us to leverage user namespaces to provide additional isolation for users within the container.

@aaronbrethorst can I start work on this issue?

Use Goss or Docker Bench Security:
Tools like Goss or Docker Bench Security can be incorporated into your Dockerfile to perform security checks and ensure best practices. These tools can help you identify potential security issues, including running containers as the root user.
Docker Security Scanning:
Docker provides security scanning tools that can analyze your Docker images for vulnerabilities. By addressing these vulnerabilities, you enhance the overall security of your containers.
AppArmor or SELinux:
AppArmor or SELinux can be used to enforce security policies on containers. These mandatory access control systems allow you to define fine-grained security policies, restricting the capabilities of containers.
Privilege Separation:
Consider breaking your application into multiple containers with different responsibilities. For example, have a separate container for a web server and a separate container for a database. This way, each container runs with the minimum required privileges for its specific role.
Docker Security Tools:
Explore other security-focused tools such as Clair for vulnerability analysis, Trivy for container scanning, and Dive for exploring and analyzing container images. These tools can help you identify and address security concerns.
Multi-Stage Builds:
Use multi-stage Docker builds to separate the build environment from the runtime environment. This can reduce the number of unnecessary tools and libraries in the final image, potentially minimizing security risks.
@aaronbrethorst can I start work on this issue?

Assigned.


refactor(db/schema): optimize ca cert validation

Summary

Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

localize some functions
replace string.gmatch to string.find
to_hex() will never return nil, we should check x509:digest()

Can you please clean up the commit history and combine your commit into a single commit with the appropriate commit header?

Done. please check it again, thanks.

@hbagdi i don't see version compat in this PR, could you double check?

merging this in as is, if there's anything we missed here, let's fix in a different PR.

@hbagdi i don't see version compat in this PR, could you double check?

As far as I can see, there isn't one required.


link to Story

hi everybody!
I am trying to go from 4 branches in my story to another story. It is supossed the "link to" button in the down right corner does it. but when I train it only appear the text from my branch but it does not go to the next story.
I am using debian and firefox to test it

Hi @pcolina,
To debug, check the following:

What is the first event in avisos? (It should be an action or bot utterance)
What do you find in the tracker instead of that first event, right after the utterance event that maps to "buenas tardes"? What policy predicted it? (You can check this in Incoming > Conversations and toggle the debug mode [🐞]).

The first thing there was in my second  story it was a user payload, and this was the reason it didn't work fine. thanks a lot !

Glad you were able to resolve your issue!! 🚀


why doesn't ResxToJS use the same structure as the RequireJSNet?

Hi
I have just adopted RequireJSNet for typescript code in my multi-page MVC app. It works great and I have all my TS files being loaded appropriately now.
My next step was to move on to i18n. I was surprised to see how ResxToJS works. I was expecting it to follow a similar convention RequireJSNet in that I expected it to create a file of translations per page. But it appears to be creating a single large file of translations for the entire solution.
Am I missing something with how it works? I have only read this page so far: http://requirejsnet.veritech.io/i18n.html
If that is really how it works, I will try my hand at forking the project to implement the following. if anybody sees something wrong with the following approach, please let me know. I would very much like to hear from people who have better ideas. :)
The goal is to create a file per MVC controller action in  RequireJS i18n format. My expectation is that these files will be quite small and we wont have to take the hit of loading the entire language for pages that only require a couple strings. I further expect that quite a bit of internationalization will be done on the server in the Razor view, but that there are times when it makes more sense to do it in the Typescript code (ie, you are providing different options on the screen based on model context).
Given that, I am thinking that I will look for a {action}-strings.json file sitting next to the page bootstrap javascript/typescript file. For example with an MVC controller called Home and an Action called Index, I would expect the following directory structure:
Scripts
    Controllers
        Home
            Index.ts
            Index-strings.json

This keeps the strings next to the code that is going to use them.
The format of the json file would be fairly simple:
{[
   ResoruceName1,
   ResourceName2
]}

ResourceName1 would be a resource name from your resource.resx file.
The rest of the configuration of ResxToJS would be the same:

a config file to say which languages to build

the output would go to the Nls directory.

The difference with the output would be that instead of one file called app-resx.js, there would be one file per controller action called {controller}-{action}-resx.js.
Am I missing something? Or are there suggestions that could make it a better solution?
Would others use this or am I the only one that thinks loading many smaller files on demand is better than loading a monster sized file on first load and hoping that the browser keeps it in cache so that it doesn't get re-loaded every so often or on every site visit.
Greg

Did you try out your fork?
As you didnt expect there to be very many translations per page I guess loading a single translation bundle for the entire site makes sense from a caching point. However, if the translations become largw I think it would make more sense to bundle parts with their respective areas, or one per page as you suggested.
Myself I dont need i18n at all, but would like all these issues to move forward or get closed...

HI @aquamoth
No I didn't get a chance to branch and try something like I was describing. I would still like to, but I have a very important deadline coming up that I have to put all my effort into.


uncover not working when an element is partially out of the slides

if the element is only a tiny bit on the lines then the object appear on each slides

Can you elaborate on that? I'm afraid I don't really understand what issue you are describing.


Testing..

Testing 123...

Thank you for creating this issue!
I'll get back to you as soon as I can.
Have a great day!
--
This comment was created automatically.

Comment with API.
Thank you!


Error when trying to denoise

Updated to the latest version and ran into this error while trying to denoise a spectrum. Got the same error trying to repeat a denoising for a previously successful spectrum.
Traceback:

1. denoise_spectrum(wavelength, flux, variances, masks, compute_uncertainties = FALSE)
2. lapply(X = 1:length(df_list), FUN = function(X) tf_obj[[X]][["x_eval"]])
3. FUN(X[[i]], ...)```

Fixed in version 0.3.0


Add skeleton code for monomorphization solver.

Description
Add skeleton code for monomorphization solver.
This code is not used in this PR---this PR just adds this as the skeleton code for how it will be used. To review this PR I would like feedback on overall design of the monomorphization module and solver system. The actual logic in terms of how individual constraints will resolve will come in a later PR that y'all can review and give feedback on as well :)
This PR is also broken out like this to reduce PR diff and merge conflicts...
Checklist

[x] I have linked to any relevant issues.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

@emilyaherbert how should this PR be reviewed? is this code not functional or exercised right now?

@mohammadfawaz I updated the PR description!


Add validation to the frontend text box to only receive urls input

Currently the text box receives any kind of input and send a call to backend. Only allow Youtube urls to be accepted

Hi @Maleehak can I work on this ?

Sure, assigning this issue to you

Hi @Maleehak one more question you want only you tube videos or youtube shorts too?

@Bhanubediya , for now the backend works for Youtube videos only so what you did is fine. The search bar looks better now too IMO. Kindly check why the left section image is breaking.

@Bhanubediya , for now the backend works for Youtube videos only so what you did is fine. The search bar looks better now too IMO. Kindly check why the left section image is breaking.

Thanks just wanted to share one thing that I think the summarize button which is in yellow should be disabled initially and it should be enabled only when the user pastes a valid url.
Just an idea. using bootstrap for future front end development would be great.

Good suggestion. I'll create another issue with your feedback or you can work on it as part of this change.

PR merged. Closing this issue. Thanks for your help.


Weird error related to iframe on Windows 7 & Firefox 26

Dear Sinon guys,
I have a unit test relating to create an iframe. The unit test is working fine in Mac for whatever version of firefox, also working fine on Chrome.
However, it failed on Windows 7 & Firefox 26, and the failures are random for each running.
And I tried on Windows 7 & Firefox 35, the issue can not be reproduced anyhow.
Appreciate if you can let me know the root cause, and also the fixing if there is one.

You have given us very little information to work with, I can't imagine anyone being able to figure out what the issue is from your description.
Feel free to re-open this issue with more information, see the the guide to reporting issues for Sinon.JS.


(AzureCXP) Updated Muti tenant

Updated Muti tenant while creating the logic app
resolves MicrosoftDocs/azure-docs#80296

@MayankBargali-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@MayankBargali-MSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.


just question ,is thanos' object storage support hdfs，like hdp,cdh?

Thanos, Prometheus and Golang version used:

Object Storage Provider:
What happened:
What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Full logs to relevant components:

Anything else we need to know:

It is not supported. You can find all supported storage backend https://github.com/thanos-io/thanos/blob/master/docs/storage.md#configuration here.

HDFS is not object storage, but distributed filesystem. However, technically with proper implementation Thanos could work over HDFS with something like https://github.com/colinmarc/hdfs. Help wanted for contributing this (:

HDFS is not object storage, but distributed filesystem. However, technically with proper implementation Thanos could work over HDFS with something like https://github.com/colinmarc/hdfs. Help wanted for contributing this (:

i dont konw the different about object storage and distributed filesystem, and recently i study on the monitor,and i do a plan that flink+consul+pushgateway+prometues for monitor,but it cant fix the large storage,so i found the thanos  ,whether thanos and hdfs can fix the large storage

It is not supported. You can find all supported storage backend https://github.com/thanos-io/thanos/blob/master/docs/storage.md#configuration here.

i saw that, but i wanna konw whether thanos and hdfs (or something base on hdfs) can work together?coz hdfs can save larger data,and monitor need the history data


[TW#15166] Re-Declaration of list_node

I have a protocol handler that will process messages that are sent both over bluetooth and over wifi.
A recent change in de100327 has meant that there is now a re-declaration of list_node causing my code not to compile. (as seen below).
In file included from C:/0000-Working/ESP32/esp-idf/components/bt/bluedroid/osi/include/fixed_queue.h:23:0,
                 from C:/0000-Working/ESP32/esp-idf/components/bt/bluedroid/hci/include/hci_layer.h:24,
                 from C:/0000-Working/ESP32/esp-idf/components/bt/bluedroid/device/include/controller.h:28,
                 from C:/0000-Working/ESP32/esp32.core/main/gatt_client.h:16,
                 from C:/0000-Working/ESP32/esp32.core/main/protocol.c:41:
C:/0000-Working/ESP32/esp-idf/components/bt/bluedroid/osi/include/list.h:108:7: error: 'list_node' redeclared as different kind of symbol
 void *list_node(const list_node_t *node);
       ^
In file included from C:/0000-Working/ESP32/esp-idf/components/tcpip_adapter/include/tcpip_adapter.h:48:0,
                 from C:/0000-Working/ESP32/esp-idf/components/esp32/include/esp_event.h:23,
                 from C:/0000-Working/ESP32/esp-idf/components/esp32/include/esp_wifi.h:69,
                 from C:/0000-Working/ESP32/esp32.core/main/protocol.c:18:
C:/0000-Working/ESP32/esp-idf/components/lwip/include/lwip/apps/dhcpserver.h:182:2: note: previous declaration of 'list_node' was here
 }list_node;
  ^

code worked fine before this commit.

@igrr @projectgus
Just wondering if there are any comments on this.
Perhaps re-naming one of these to a more file specific name as not to conflict with the other?

Hi @lucazader ,
Thanks for reporting this. It's definitely a bug that two structures have such a generic name. Working on a fix now.
Angus

Hi @projectgus
Thanks for the reply.
I look forward to working with the extra stuff in the 3.0 branch again!

Hi @projectgus
Just checking if this is still in the to-do queue, or if it is done waiting to be merged into the github master?
Currently this is the only thing preventing us from switching over to the v3.0 for development

Hi @lucazader ,
Sorry for the delay. The fix for this was merged internally yesterday, it should appear on github in the next 24 hours or so (this issue will be updated when it does.)
Angus


Remove transactionWithContext

If we dont need it
It seems likely that this is something handled by SqlDelight by now, but we should make sure we understand if it's needed

I think this is fine to leave as-is. It's a helper function that encapsulates switching threads and launching a transaction, which we're using for all of our database operations since we always do both those operations at the same time.


bug: Error: 10

Plugin version: 0.3.11
Platform(s): Android
Current behavior: When signing in with Google, after choosing the right Google account I get the following error:
Error: 10:
    at Object.cap.fromNative ((index):426)
    at <anonymous>:1:18

Expected behavior: Sign in successfully.
Steps to reproduce:

Install plugin.
Call FirebaseAuthentication.signInWithGoogle()
Choose account

This happens both when skipNativeAuth is true and false, running on Android Simulator
Capacitor doctor:

Latest Dependencies:

  @capacitor/cli: 3.3.0
  @capacitor/core: 3.3.0
  @capacitor/android: 3.3.0
  @capacitor/ios: 3.3.0

Installed Dependencies:

  @capacitor/cli: 3.2.4
  @capacitor/android: 3.2.4
  @capacitor/core: 3.2.2
  @capacitor/ios: 3.2.4

Hi @yaron1m,
thank you for your request.
I have never tested the plugin in the Android emulator. I will give this a try this weekend.
The error looks like error code 10 (DEVELOPER_ERROR) from Google Play Services.
Please have a look at this comment.
Make sure your app is configured correctly in the Google API Console/Firebase Console (specifically the SHA1).
Let me know if this helps you.
I will get back to you as soon as I have tested the plugin in the emulator.

Quick update: I tested it on an Android emulator. It worked without any problems.

The issue was indeed with the SHA1.
After updating it in Firebase issue was resolved.
Thanks!


cache Docker images to speed up build

No significant performance was detected. With the switch to Testcontainers Cloud, we anyway get performance improvements.


[Feature Request] ConvTranspose2D

PyTorch Navie Implementation: https://github.com/pytorch/pytorch/blob/main/aten/src/ATen/native/NaiveConvolutionTranspose2d.cpp
PyTorch Navie Cuda Implementation: https://github.com/pytorch/pytorch/blob/main/aten/src/ATen/native/cuda/NaiveConvolutionTranspose2d.cu
I guess so this is the metal implementation of Conv Tranpose in PyTorch:
https://github.com/pytorch/pytorch/blob/015bd0e0a189f929e469c6bc75fe1541c18a014d/aten/src/ATen/native/mps/operations/Convolution.mm#L709

I can take this up @awni. Let me know if I can go ahead with the implementation of this feature request

Wow that would be amazing! Are you comfortable writing Metal / kernels on the GPU?

I am new to GPU programming and I'm skilling up on Metal, but I'd love to collaborate on this.
I actually thought about implementing ConvTranpose2D feature in python, but I just realized that it's better to implement it using Metal first and then python. Thanks for clarifying.


PowerToys Run Search Error

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
I tried using run for the first time after updating from version 0.61 to 0.66
After opening, I typed two letters (te), the error appeared on the second key press
The following error dialogue popped up twice with identical contents and prompting this error form

Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.19045.0
IntPtr Length: 8
x64: True
Date: 22/01/2023 19:53:46
Exception:
System.InvalidOperationException: Dispatcher processing has been suspended, but messages are still being processed.
   at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)

And generating the following error log file
2023-01-22.txt
✔ Expected Behavior
It to search
❌ Actual Behavior
It crashed
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/18166

For any users that are able to replicate this consistently, can you please update to 0.68.1 and disable thumbnail generation to verify if it fixes the issue? Please let us know on #18166!


fix error on merging findings due to django3 changes

fixes https://github.com/DefectDojo/django-DefectDojo/issues/4586

@37b Thanks. We (not you) should have picked this up earlier. Can you look at removing that extra file that came along in the commit?

Hi! I just manually applied this fix on my DefectDojo instance because the merging feature is critical for us. However, there is a bug in the fix. See PR #5144 for details.


Add common oauth ultilities for 3pi support

Add common structs: client auth, token exchange requests and responses
Add token exchange helper methods to generate grpc_httpcli_request and request body

@markdroth @jiangtaoli2016 @ZhenLian
Please take a look at this PR which adds common oauth utilities for 3p identity support.
It seems I don't have write access so I can't add the proper labels. Can someone help add them? Thanks!

Using C++ style for this new code does not depend on the style of any existing code.  As I said, we are in the process of migration, so there is a bunch of existing code that has not yet been migrated.  But I do not want to make that process harder by adding more code using the wrong style that will have to be migrated later.
It's fine to create a class whose name is in C++ style and that inherits from another class whose name is not yet in C++ style.  It's also fine to use C-style names if you are implementing an interface whose API has not yet been converted to C++ style naming (e.g. grpc_call_credentials).  Those are uses of existing code that you can't avoid.  But for anything that does not need to use C style in order to make those kinds of interactions work, it should be done in C++ style.
I'm not sure why it's useful for you to see docs about our migration to C++, but you're certainly welcome to view them.  Here are the relevant docs:

gRFC L6: Allow C++ in gRPC Core Library
gRFC L59: Allow C++ standard library in gRPC Core Library
Moving gRPC core to C++

In the particular case of grpc_oauth2_token_fetcher_credentials and of the method names in the grpc_call_credentials API, they should have been converted to C++ style names when the code was coverted to C++.  @soheilhy, you did that conversion in #17291, which was merged in Dec. 2018.  You told me at that time that you were going to update the names to C++ style in a subsequent PR, but you never did.  Are you still going to do it, and if so when?  If not, I will try to find time to do it myself.

I was worried about the mix of C++11 and C styles so I was thinking converting the style at the same time may be better. But what you said makes sense. I'll update this PR in C++ styles.
I was wondering if there is migrated code(instead of docs) I can refer to, so I can learn more about the C++11 style and adhere that. The docs and project board helps though.
I'll update this PR later. Thanks!

You can look at pretty much any of the completed code in https://github.com/grpc/grpc/projects/9 for examples.
See also https://github.com/grpc/grpc/blob/master/doc/cpp-style-guide.md, which basically just points to Google's OSS C++ style guide.

@soheilhy, you did that conversion in #17291, which was merged in Dec. 2018. You told me at that time that you were going to update the names to C++ style in a subsequent PR, but you never did. Are you still going to do it, and if so when? If not, I will try to find time to do it myself.

My apologists, I probably wouldn't have the cycles to do so.  IIRC, I had a patch ready but there were complication regarding Asylo imports and namings, and I had to give on that patch due to copybara headaches and conflicts.

According to the discussion @markdroth , the OAuth related methods should be encapsulated in the base credentials class. Please see the changes in https://github.com/grpc/grpc/pull/24208 and I'm going to close this issue.


VectorHand - OverflowException: Value was either too large or too small for an Int16.

Issue:
Use FillBytes to encode your hand(s) and get the error:
OverflowException: Value was either too large or too small for an Int16.
from line:
LeapMotion/Core/Scripts/Encoding/VectorHand.cs:239)
Reproduce:

Start a new scene, remove the main camera, drag Leap Rig prefab into scene.
Add the following script to the Leap Rig Game Object

using Leap;
using Leap.Unity;
using Leap.Unity.Encoding;
using UnityEngine;

public class LeapTestVector : MonoBehaviour
{
    LeapXRServiceProvider leap;
    VectorHand vectorHand = new VectorHand();

    void Start()
    {
        leap = GetComponentInChildren<LeapXRServiceProvider>();
        leap.OnUpdateFrame += OnUpdateFrame;
    }

    private void OnUpdateFrame(Frame obj)
    {
        byte[] leftHandBytes = null;
        byte[] rightHandBytes = null;
        foreach (Hand hand in obj.Hands)
        {
            vectorHand.Encode(hand);
            if (hand.IsLeft)
            {
                leftHandBytes = new byte[86];
                vectorHand.FillBytes(leftHandBytes);
            }
            else
            {
                rightHandBytes = new byte[86];
                vectorHand.FillBytes(rightHandBytes);
            }
        }

    }
}

Move the Leap Rig's X or Z Position to 8 or -8.
See errors.

Am I using VectorHand wrong? Above is just a snippet of the code I'm using, I know I can tidy it up, I'm sending the byte array over the network to show hands on other clients...

If we encode the VectorHand with Int32 instead of Int16 for Palm Position then we need 6 more bytes, which I'm fine with.
VectorHand.cs:
193:
public int numBytesRequired { get { return 92; } }
216:
BitConverterNonAlloc.ToInt32(bytes, ref offset))
261:
BitConverterNonAlloc.GetBytes(Convert.ToInt32(palmPos[i] * 4096f),

Added this as a potential bug to our backlog. Are you saying that the Int16 isn't sufficient to encode the palm position and therefore you need to increase its size to an int32? If so, the other changes you make are needed to get it to stream correctly now the size is larger.

Tracked as a breaking feature reques/bug. UNITY-762. Closing here.

Still not fixed? It occurs when I try to encode a replicated hand via PostProcessor.

Still not fixed? It occurs when I try to encode a replicated hand via PostProcessor.

Hmm, it seems we have marked this as a ticket we won't do. I will re-open it, though can't promise when it will get looked at 😬


Exclude root project from type-safe project dependencies

In Gradle the rootProject is special and accessible from every project.
Expected Behavior

Skip root project for type-safe project dependencies
Current Behavior

With type-safe project dependencies, the root project raises several issues with some provided names.
Context

The root project is in the same hierarchy like its subprojects. Therefore it's not possible to name the root project Lib and have a subproject :lib as a subproject.
The root project name is also used by other tools (like Intellij) to display it to the user, but it's not possible to use a readable name including spaces   or other characters not mathing the projects pattern.

Steps to Reproduce

enableFeaturePreview 'TYPESAFE_PROJECT_ACCESSORS'

rootProject.name = 'Lib' // or 'Some lib'

include ':lib'

Thank you for providing a valid reproducer.
The issue is in the backlog of the relevant team but the existence of a workaround makes it non-critical so it might take a while before a fix is made.

Note: the source of the problem is a name clash between project name and subproject name.
* What went wrong:
org.gradle.api.internal.catalog.GeneratedClassCompilationException: Unable to compile generated sources:
  - File RootProjectAccessor.java, line: 29, method getLib() is already defined in class org.gradle.accessors.dm.RootProjectAccessor
> Unable to compile generated sources:
    - File RootProjectAccessor.java, line: 29, method getLib() is already defined in class org.gradle.accessors.dm.RootProjectAccessor

@ov7a, can I contribute a fix? It seems like just RootProjectAccessorSourceGenerator and its usages should be deleted.

Related issues that could be resolved by this proposal:

https://github.com/gradle/gradle/issues/18971
https://github.com/gradle/gradle/issues/16780
https://github.com/gradle/gradle/issues/28644

@osipxd Sorry for the late reply. The team responsible for this area should make a decision. They have been informed. Response time may vary.

Yet another problem with the root accessors is that changing the root project's name does not make the accessors to be regenerated. More surprisingly, the state accessors continue to work, and the builds are successful.

Related: https://github.com/gradle/gradle/issues/16608

can I contribute a fix?

@osipxd, this issue is still relevant and we still plan to work on it. We need to come up with a design first before we proceed to the implementation, so we don't accept PRs in that area at the moment.


Undulators: SRW library (srwlib) not present in Linux

I would not try to distribute SRW now. Unless it is easy to pack the libraries in xoppy. You decide. In the future we will see.  I wrote the srundplug to compare also with SRW. I think it is not a major problem to see that error message. If you want we can hide this call, but can be useful if you have SRW installed (like me).

I will just send a better message


useAsyncData does not pass errors from server to client-side

Environment

Operating System: Linux
Node Version:     v14.18.1
Nuxt Version:     3.0.0-27294839.7e5e50b
Package Manager<EMAIL_ADDRESS>Bundler:          Vite

Reproduction
https://stackblitz.com/edit/github-ygz9df?file=app.vue
Describe the bug
On server side the error content is rendered  as expected while the client side says it's null.
Additional context
No response
Logs
No response

anyone getting how to solve the same issue with onRequestError ?
seems there is no way to get response status with this handler, for example when the server return a 401


feat: add authentication to diagnostics

Issue

Link: https://github.com/NebraLtd/hm-diag/issues/39
Summary: add authentication to diagnostics

How

To prepare for future updates and new features, authentication is added to diagnostics page.
Note that this is a major update and will need thoroughly testing and documenting (for support purposes) whilst on testnet. Customers should be notified of the change via a blog post and email as well after we merge to production.
Screenshots

References

https://github.com/NebraLtd/hm-diag/issues/39
https://github.com/NebraLtd/hm-diag/issues/445
https://github.com/NebraLtd/hm-diag/issues/446
https://github.com/NebraLtd/hm-config/pull/225
Checklist

[x] Tests added
[x] Cleaned up commit history (rebase!)
[x] Documentation added
[x] Thought about variable and method names

Note that this is a major update and will need thoroughly testing and documenting (for support purposes) whilst on testnet. Customers should be notified of the change via a blog post and email as well after we merge to production.

Just an FYI on this part @ChristopherRush @WayneNebra @KevinWassermann94

@shawaj I'll get it tested thoroughly today

@robputt @shawaj We can merge this to testnet right now, correct?

@KevinWassermann94 yep all good

As a suggestion, since the MAC addresses typically appear with colons, and the Nebra Dashboard reports them with colons as well, to avoid users having to copy/paste from the Nebra Dashboard into notepad or something, then deleting the colons, then copying/pasting again into the password field, I'd suggest allowing colons in the password field and stripping them out before determining whether the password is valid.
This would also allow the removal of the "without any colons" wording on the login page.

Not a bad suggestion @Ryan-Goldstein - will see what we can do with @robputt
Could you put this as an issue on hm-diag repo?
https://github.com/NebraLtd/hm-diag/issues


Is this repo still maintained?

It looks like this repo has no active maintainer anymore, which is a bit sad. Are you getting back any time soon?

Sorry, currently I don't have much time to maintain this plugin because I don't use it. I would really like to get some help with this.

@FWeinb You should state that you're looking for collaborators to help maintain this project near the top of your readme.md

Sorry for the long phase of silence. I added it to the readme and would really like to get someone who would take over this.

I’ve been recommending this plugin and I’ll be giving the source a few more passes. If I can help, I will.

Thanks! I added you as a collaborator. You should have commit rights now.

I want to help where I can. Thanks @FWeinb for your work so far, love the product!!

I'd respectfully give feedback this be closed by OP or maintainer. I hadn't contributed for a while, and as I scanned down the issues list, I definitely saw this and wondered myself. I had faith having seen that @jonathantneal appears to sort of still maintain,  and coded up my PR, but I could see it dissuading others. This is a useful plugin and my company has used it in our SVG icon process for 3 years now. We've even had recent success with exporting from Sketch which simply requires adding the Sketch svgo compressor plugin to generate clean raw SVGs, and grunt-svgstore works fine!


Doc fix: Replace SINGLE with PLURAL in taxonomy template file names

Related to issue https://github.com/spf13/hugo/issues/1546

 You should sign our Contributor License Agreement in order to get your pull request merged.

This PR should be reverted (my mistake). See https://github.com/spf13/hugo/issues/1546#issuecomment-153753810

WE can close this. We haven't merged it yet, so we don't need to revert it.


Use pytest fixtures for anndata tests

referenced in #1574

I tried factoring out the model training into a fixture, but that resulted in test_anndata taking 4->14 seconds to run

I tried factoring out the model training into a fixture, but that resulted in test_anndata taking 4->14 seconds to run

did you use the fixture on every test, or was it only used for tests that did this anyways? If its the latter I dont understand why the runtime would increase so much.

It was only for the tests that required a model to be trained.


Update debian arm64 Helix queue image

Without this we get errors like https://dev.azure.com/dnceng/public/_build/results?buildId=302761. I talked with @MattGal and this is due to the move from Python2 to Python3.
Do we have some process for updating those docker tags in a more centralized way?

Without this we get errors like https://dev.azure.com/dnceng/public/_build/results?buildId=302761. I talked with @MattGal and this is due to the move from Python2 to Python3.
Do we have some process for updating those docker tags in a more centralized way?

THere isn't anything currently.  I built the feature earlier this year and it's only been in major use for 4-5 months; we could certainly think about some Maestro-like way of flowing updates but it's important to not break users and thus we'd have to ensure anything being auto-flowed like this was not breaking things.  With JSON like this (https://github.com/dotnet/versions/blob/master/build-info/docker/image-info.dotnet-dotnet-buildtools-prereqs-docker-master.json) being automatically published with every new image it becomes trivially easy to implement such a thing, we just have to decide to do it.
Also, I love how you specify your queues using ITaskItems, I may show this to some CoreFX dudes who have giant lists.

What do you mean about the ITaskItems? I don't actually work in this area, was just annoyed by the failures, so I could probably learn from it too lol.

What do you mean about the ITaskItems? I don't actually work in this area, was just annoyed by the failures, so I could probably learn from it too lol.

Sorry too much jargon, I just mean the things inside an ItemGroup in msbuild (https://docs.microsoft.com/en-us/dotnet/api/microsoft.build.framework.itaskitem?view=netframework-4.8 ) .  CoreFX has a giant semicolon-delineated list in yaml for these which is getting out of control, major props whoever wrote this props file.

Props to @pakrym :)

Seems like I fixed arm64, but problems remain in x64?

I suspect this or something else is also breaking Mission Control. Or perhaps Mission Control is no longer involved in Helix runs. https://mc.dot.net/#/user/ryanbrandenburg/builds doesn't show anything of interest. @MattGal?
If this is correct, please remove the https://mc.dot.net/#/user/$(your user name)/builds mention in https://github.com/aspnet/AspNetCore/blob/release/3.0/docs/Helix.md

Seems like I fixed arm64, but problems remain in x64?

Hard to tell w/ limited information in latest build. Think I've seen similar "unable to parse response" errors in other PRs.
See also #13031

I suspect this or something else is also breaking Mission Control. Or perhaps Mission Control is no longer involved in Helix runs. https://mc.dot.net/#/user/ryanbrandenburg/builds doesn't show anything of interest. @MattGal?
If this is correct, please remove the https://mc.dot.net/#/user/$(your user name)/builds mention in https://github.com/aspnet/AspNetCore/blob/release/3.0/docs/Helix.md

Seems like I fixed arm64, but problems remain in x64?

Hard to tell w/ limited information in latest build. Think I've seen similar "unable to parse response" errors in other PRs.
See also #13031

Yes moving from Mission Control -> Azure DevOps is intentional and updated in docs via PRs like https://github.com/dotnet/arcade/pull/3578.
We certainly didn't intend for the change to flow to everyone before communicating it more broadly, however, it just is a natural side effect of a newer Helix Job API version that, once adopted by the Arcade Helix SDKs, made calculating such URLs no longer possible, and the automatic flowing of dependencies was just faster than expected.


[WIP] Apply commonRegex in adaptive dialog

Apply commonRegex in adaptive dialog "RegexRecognizer"

close for duplicated


throws error when validating solo side

https://github.com/tjmehta/homebridge-eightsleep-platform/issues/3

fix in v1.3.2


Switch curl to using the multi-handle & workers

We've noticed the performance of the plugin is horribly slow -- to the point where we don't have enough cores in machines to have the origin keep up with the caches (why the caches are pushing so hard is a separate story).
A few judicious uses of pstack indicates that almost all the CPU time is spent in TLS handshakes.  It turns out that the curl_easy_* APIs have no connection reuse unless you reuse the handle itself.  So, every single request is a separate TLS connection to the upstream HTTP / S3 server.  That is indeed, no fun.
This takes motivation (and code) from https://github.com/PelicanPlatform/xrdcl-pelican to have all the requests handled in a set of separate worker threads.  The request object enters a queue, is processed by a worker, and then, on completion, the original thread is notified.
This means there's probably 5-ish connections held open to AWS (one per thread) instead of many thousands of short-lived ones.

I built this and ran my stress test on it to see if maybe it fixed the failures problem as a side effect. Quite the opposite! The existing code base fails at something like a 0-20% rate, which high levels of variability. This update is failing at more like a 90% or more rate. I don't have a ton of time this afternoon to dig deeper, but have a bunch of time tomorrow to see if this might help us uncover the failure problem(s).

241016 rich log.txt


Custom Links with # are not processed correctly

Using the following custom link results in the rendered link on the page being malformed:
[custom_links.nodes.0]
title = "Open in Launch"
href = "https://hostname/automation/#/service/catalog/consume/deployment?search=%22{{name}}%22"
new_tab = true

The resulting link rendered on the page points to http://localhost:8080/ui/nodes/%22https://hostname/automation/.  URL's without the # character are processed correctly and function as expected.

This sounds like what I fixed in https://github.com/drewhammond/chefbrowser/pull/400 but never cut a release. Will have this out later today in 0.1.7. Thanks!

No, thank you!

Yeah, that's basically what it was doing under the hood. The fix in #400 isn't perfect because ideally I'd like to be able to support both inline comments and quoted strings containing #, but it'll do for now.
Released in 0.1.7!


Show your own game status somewhere

Currently you can't see your own game status, perhaps a notification or status indicator of some sort would be helpful.

Partially added support with the change in #4; might look into adding it to the user list at a later time.


remove mention of nursery from contribute/index.md

The mathlib-nursery repo is no longer in active use. I suggest removing the text that mentions it.

bors merge

bors merge

Patrick merge
? :)


Mixed versions of release and prerelease are misordered

http://www.nuget.org/api/v2/FindPackagesById()?id='EntityFramework'&$orderby=Version
The order is ascending but 4.3.0 should be after 4.3.0-beta1 (since release versions are always greater than prerelease).

V2 API is in maintenance. Please use NuGet V3 API.

V2 API is in maintenance. Please use NuGet V3 API.


Header issue

ISSUE:
The example at: https://design-system.service.gov.uk/components/header/ for the service name example will put the service name underneath the GOV.UK header instead of alongside it. This is because the header code relies on a closing and opening div being on the same line.
EXPECTED BEHAVIOUR:
The header should work correctly regardless of whether a div is on the same line or not.
REPRODUCE:
Open your test page here in Chrome: https://design-system.service.gov.uk/components/header/with-service-name/index.html
Inspect element.
To the left of: govuk-header__container govuk-width-container on the three dots right click and 'edit as HTML'
Go down to this line:

Move the second opening div <div class=... to a new line and click on the page.
The service name moves to the next line.

I see a few possible solutions:

Use floats and clearfix
Add comments into the template itself
Don't pretty print html
Do the font-size trick https://css-tricks.com/fighting-the-space-between-inline-block-elements/#article-header-id-3

It feels like 1. would be the most robust

Please don't do 2 or 3....


Allow to set SELinux mode through global.components.selinux.mode.

What does this PR do?
Towards: https://github.com/giantswarm/giantswarm/issues/30605
Allows to set SELinux mode (enforcing, permissive or disabled) through global.components.selinux.mode.
Default value is permissive (current flatcar default).

[x] CHANGELOG.md has been updated

@stone-z Please take a look, we will be able to set SELinux to enforcing, permissive (current default) or disabled through helm values (once this is merged and cluster-$provider charts pick up the new version and update their schema, e.g., https://github.com/giantswarm/cluster-aws/pull/610).
Do we also want to allow to set the type of protection (targeted, strict, mls, mcs) through values?
(Changes will be reflected in the /etc/selinux/config file)


feature/MODELINKS-248 Do not show empty arrays for extended mapping

Purpose
Update serialization for AuthorityDto to not expose empty saftBroaderTerm, safrNarrowerTerm, saftEarlierHeading, saftLaterHeading in kafka messages.  It makes including extended mapping not noticeable for users who doesn't use this functionality.
Approach
add com.fasterxml.jackson.annotation.JsonInclude.Include.NON_EMPTY annotation for saftBroaderTerm, safrNarrowerTerm, saftEarlierHeading, saftLaterHeading fields.
Changes Checklist

[ ] API Changes: Document any API paths, methods, request or response bodies changed, added, or removed.
[ ] Database Schema Changes: Indicate any database schema changes and their impact. Confirm that migration scripts were created.
[ ] Interface Version Changes: Indicate any changes to interface versions.
[ ] Interface Dependencies: Document added or removed dependencies.
[ ] Permissions: Document any changes to permissions.
[ ] Logging: Confirm that logging is appropriately handled.
[ ] Unit Testing: Confirm that changed classes were covered by unit tests.
[ ] Integration Testing: Confirm that changed logic was covered by integration tests.
[ ] Manual Testing: Confirm that changes were tested on local or dev environment.
[ ] NEWS: Confirm that the NEWS file is updated with relevant information about the changes made in this pull request.

Related Issues
List any Jira issues related to this pull request.
Learning and Resources (if applicable)
Discuss any research conducted during the development of this pull request. Include links to relevant blog posts, patterns, libraries, or addons that were used to solve the problem.
Screenshots (if applicable)
If this pull request involves any visual changes or new features, consider including screenshots or GIFs to illustrate the changes.

Tests?

Added.


Refactor Aggregator implementation

This PR refactors the aggregator functions visit implementation. The visiting implementation is moved to a new class called StreamingAggregatorArrayBuilder

Codecov Report

Merging #13582 into master will increase coverage by 3.84%.
The diff coverage is 79.16%.

@@             Coverage Diff              @@
##             master   #13582      +/-   ##
============================================
+ Coverage      60.5%   64.34%   +3.84%
  Complexity      426      426
============================================
  Files          2157     2153       -4
  Lines        104599    94567   -10032
  Branches      13421    12473     -948
============================================
- Hits          63285    60847    -2438
+ Misses        36048    28815    -7233
+ Partials       5266     4905     -361

Impacted Files
Coverage Δ
Complexity Δ

...lang/compiler/desugar/StreamsPreSelectDesuagr.java
93.86% <0%> (-2.37%)
0 <0> (ø)

...inalang/compiler/desugar/StreamingCodeDesugar.java
96.9% <100%> (+0.16%)
0 <0> (ø)
:arrow_down:

...mantics/analyzer/StreamsQuerySemanticAnalyzer.java
86.22% <12.5%> (-1.26%)
0 <0> (ø)

...piler/desugar/StreamingAggregatorArrayBuilder.java
92.59% <92.59%> (ø)
0 <0> (?)

...allerinalang/net/jms/LoggingExceptionListener.java
50% <0%> (-50%)
0% <0%> (ø)

...a/org/ballerinalang/bre/bvm/WorkerDataChannel.java
79.01% <0%> (-4.94%)
0% <0%> (ø)

...rinalang/runtime/threadpool/ThreadPoolFactory.java
61.9% <0%> (-4.77%)
0% <0%> (ø)

...org/ballerinalang/bre/bvm/WaitCallbackHandler.java
82.88% <0%> (-2.71%)
0% <0%> (ø)

...ain/java/org/ballerinalang/net/websub/hub/Hub.java
69.53% <0%> (-1.57%)
0% <0%> (ø)

...e/src/main/java/org/ballerinalang/bre/bvm/BVM.java
81.21% <0%> (-0.04%)
0% <0%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


Visibility of "Hide search matches" button

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
Currently - light mode - not overing

Currently - light mode - overing

📝 Steps to reproduce
Visit:
https://mapdl.docs.pyansys.com/version/stable/mapdl_commands/post1/_autosummary/ansys.mapdl.core.Mapdl.set.html#set

Ha, I was about to post the same!

📝 Steps to reproduce
Visit:
https://mapdl.docs.pyansys.com/version/stable/mapdl_commands/post1/_autosummary/ansys.mapdl.core.Mapdl.set.html#set

At least for me simply going to that page doesn't show the button. I can reproduce it with

search for a term (e.g. "test")
go to one of the suggested locations


Got error when training ssd300MobileNet with imagenet

Hi @tanakataiki ,
Thanks a lot for such a great work!
I am trying to train your ssd300MobileNet model with ImageNet dataset and got the following error:

ValueError: Error when checking target: expected predictions to have 3 dimensions, but got array with shape (32, 1000)

Do you have idea how to fix this issue?  Is there any chance of missing Flatten layers before predictions layer (or after softmax)?
Best,
Hung

How did you solve this issue?


Can't import functions with 64bit parameters

I want to import functions from my wasm guest. I use the same mechanism used by this repo, but it seems like there is a limit on how many bits you can have in your parameters/returns.
Here is a repro:
https://github.com/mhmd-azeez/Int64Repro

Moved it to https://github.com/dotnet/runtime/issues/90786


[SPARK-28635][SQL] create CatalogManager to track registered v2 catalogs

What changes were proposed in this pull request?
This is a pure refactor PR, which creates a new class CatalogManager to track the registered v2 catalogs, and provide the catalog up functionality.
CatalogManager also tracks the current catalog/namespace. We will implement corresponding commands in other PRs, like USE CATALOG my_catalog
How was this patch tested?
existing tests

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13805/
Test PASSed.

cc @rdblue @brkyvz

Test build #108722 has started for PR 25368 at commit de95b81.

Test build #108722 has finished for PR 25368 at commit de95b81.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class CatalogManager(conf: SQLConf) extends Logging

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108722/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13837/
Test PASSed.

Test build #108756 has started for PR 25368 at commit e85f495.

Test build #108756 has finished for PR 25368 at commit e85f495.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108756/
Test PASSed.

@cloud-fan, how will users set the current catalog using the catalog manager?
We want to support changing the current catalog, right? Like this:
USE CATALOG my_catalog;

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13880/
Test PASSed.

@rdblue we need a separated PR to implement USE CATALOG my_catalog, which needs to add a new SQL syntax and the related logical/physical plans. This PR is more of a code refactor, and I've added the "currentCatalog" and currentNamespace in the CatalogManager.

Test build #108800 has started for PR 25368 at commit 73fdb22.

Test build #108800 has finished for PR 25368 at commit 73fdb22.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108800/
Test PASSed.

Test build #108860 has started for PR 25368 at commit a440694.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13933/
Test PASSed.

Test build #108860 has finished for PR 25368 at commit a440694.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108860/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/13942/
Test PASSed.

Test build #108869 has started for PR 25368 at commit a440694.

Test build #108869 has finished for PR 25368 at commit a440694.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108869/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14019/
Test PASSed.

Test build #108946 has started for PR 25368 at commit b07790d.

Test build #108946 has finished for PR 25368 at commit b07790d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/108946/
Test PASSed.

@brkyvz @rdblue any more comments?

Test build #109099 has started for PR 25368 at commit 9186620.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14166/
Test PASSed.

Test build #109099 has finished for PR 25368 at commit 9186620.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109099/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14213/
Test PASSed.

Test build #109144 has started for PR 25368 at commit 45cbbd0.

Test build #109144 has finished for PR 25368 at commit 45cbbd0.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109144/
Test FAILed.

retest this please

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14237/
Test PASSed.

retest this please

Test build #109168 has started for PR 25368 at commit 45cbbd0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14244/
Test PASSed.

Test build #109168 has finished for PR 25368 at commit 45cbbd0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109168/
Test FAILed.

LGTM pending tests

retest this please

Test build #109184 has started for PR 25368 at commit 45cbbd0.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14256/
Test PASSed.

Test build #109184 has finished for PR 25368 at commit 45cbbd0.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109184/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14272/
Test PASSed.

Test build #109202 has started for PR 25368 at commit 45cbbd0.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14291/
Test PASSed.

Test build #109224 has started for PR 25368 at commit 45cbbd0.

Test build #109224 has finished for PR 25368 at commit 45cbbd0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109224/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14442/
Test PASSed.

Test build #109380 has started for PR 25368 at commit 4895a6e.

Test build #109380 has finished for PR 25368 at commit 4895a6e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class FileCommitProtocol extends Logging
class BindingParquetOutputCommitter(
class PathOutputCommitProtocol(
class DecisionTreeParams(Params):

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109380/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/14454/
Test PASSed.

Test build #109392 has started for PR 25368 at commit 4895a6e.

Test build #109392 has finished for PR 25368 at commit 4895a6e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class FileCommitProtocol extends Logging
class BindingParquetOutputCommitter(
class PathOutputCommitProtocol(
class DecisionTreeParams(Params):

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/109392/
Test PASSed.

thanks for the review, merging to master!

@cloud-fan, sorry for the delay reviewing this. +1 from me.


Builder mounts detected

I followed the steps to install synth in a jail (running Freenas and set it up in a jail)
When I run upgrade-system or status I get this message:

Builder mounts detected; attempting to remove them automatically ...
The attempt failed. Check for stuck or ongoing processes and kill them.
After that, try running Synth again or just manually unmount everything
still attached to /usr/obj/synth-live

I have no idea on how to proceed from here. Any tips?

maybe this FAQ entry has a clue?
https://github.com/jrmarino/synth#synth-fails-when-i-run-it-inside-a-jail--what-can-i-do

I already set those options.
But I found the solution. When cancelling a run using ctrl+c it will "lock" the file in question so other jails can't run synth at all. Letting a command finish properly will not give the error, so I guess it is not freeing up something?

You're never supposed to use control-C.  There's a control-Q option for a graceful shutdown.  I'm sure that's documented in man page and other places.


feat: use openDrawer param on application page to open sidebar

Description :sparkles:

Open sidebar with openDrawer param on application view
Add link to the icon indicating there's new messages
If there are new messages, add openDrawer param in the text and icon link

Codecov Report

Merging #2143 (f4f25f6) into develop (2e4d8c1) will decrease coverage by 2.66%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #2143      +/-   ##
===========================================
- Coverage    20.65%   18.00%   -2.66%
===========================================
  Files          577      122     -455
  Lines        11918     2905    -9013
  Branches      2887      727    -2160
===========================================
- Hits          2462      523    -1939
+ Misses        9052     2271    -6781
+ Partials       404      111     -293

see 455 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Automated: Bump Docker images

Hello! This is an automated pull request.
I have looked for the following Docker images, and updated your files referencing any version(s) of these images:

qlikcore/engine:12.612.0
qlikcore/licenses:3.5.0

You're not obligated to win. You're obligated to keep trying to do the best you can every day. (Marian Edelman)

Coverage remained the same at 89.802% when pulling 69c7741d6292e66ed1957a6cdbff57e42f153e45 on docker-images/1584284410 into a63f7431ee79cd38e8e3403dea616805a0e2a2a3 on master.


How do we want Undo/Redo behave?

As discussed in #91 undo/redo may be triggered on Surface level, but actually needs to be interpreted on app level, in the presence of multiple panels (not necessary in single surface scenarios).
I.e., to be able to give visual feedback for an undone change, it might be necessary to open a specific panel.
Having this behavior requires that every document change is able to carry application specific data, which is also interpreted when a change is replayed.
We should discuss here what we want, and what alternatives are there, and for what 'price'.

I would suggest to trigger this on the controller level:
controller.executeCommand('undo');

Then we can define hooks for undo in the app-specific controller implementation. We considered passing a context when constructing a controlller, that we can then call ctx.setState() on for switching panels etc.

Or adding Controller.create(proto) to create a derived Controller conveniently.

For my use-case in Dabble, I want to have undo work similar to IntelliJ's. I'm not defining an implementation but a user-facing description of what happens.
When a user calls undo once, the cursor takes them to the place of the last operation. When calling undo a second time the last operation gets undone. If the cursor is already in place the operation just gets undone. This would allow undo operations to happen across documents or locations within a document while still providing the user insight into what is happening. If you simply undo the last operation the user may not know exactly what was undone if it is off the screen when they call undo.
This may need to be handled by the app, but it will need help from the commands by having them store the last surface and cursor location as well as making that information visible to the app so that it may take the user to that surface/location.

@jacwright I think that is exactly what we want to do
I think we are all good, as we have undo and redo modeled as ControllerCommand which is working on a app level. So we should be able to achieve all we need.
WIth realtime collab there are some more challenging issues regarding undo in presence of other users' changes - but that is a different story.


Add options to refresh Cards that rely on server-side info

Anything that uses auto-complete will have this option.

Missed the second milestone.

Implementing this will depend on whether or not the app is robust enough. If it always loads, we won't need it.

This could be done more seamlessly by regularly updating the headers.
My real concern is that rebuilding the autocomplete will get expensive.

Cards are now refreshed continuously since #144.


Develop

Add DF profiler

Coverage increased (+0.2%) to 16.239% when pulling ca5cc4593425002ce7b16e4e0b5a039f850d71cf on develop into 44a15c65f9ae60d60b6ab12de43d8bf91a31abea on master.


Web (HTTP) Seeding

WIP (do not merge). See https://github.com/feross/webtorrent/issues/67
Depends on this PR: https://github.com/feross/bittorrent-swarm/pull/19
I started working on a web seed implementation. The implementation right now is a bit naive (it maps every bittorrent protocol requests to HTTP Range requests) and could probably be optimized.

[x] downloads from web seeds listed in url-list
[ ] test in web browser
[ ] test what happens when regular peers and web peer

@feross could you just take a quick look at this and let me know if there's any chance to get this merged? And if I'm heading in the right direction...

@feross this would be quite handy.

@olalonde - you're a hero! Thanks for this PR! Please accept my sincere apologies for the delay in getting this merged. This was a much needed feature.
It only works in node right now, and it only supports BEP19, and the approach isn't very efficient with so many separate requests, but that's all okay for now. We have a starting point to work from now!

This is released as 0.51.0.

Great, @feross. Was starting to wonder if you'd ever take a look at the PR :+1:. I'll probably try to improve it in coming days/weeks as I need the functionality for a project I'm working on. I'll try to catch you on IRC if I need some advice.

Sounds great!

Hi there! I'm also interested in support for web seeding. I didn't try this out yet, but from reading the code, it seems this only supports single-file torrents, is that correct? Because if I'm reading BEP19 correctly, one would need to translate from pieces to offsets into specific files for multi-file torrents, which I don't see being done here.

@samcruz Yeah, that's not implemented yet.

Does this mean, the testing indicated above hasn't been done before merging?

Great work @olalonde! I'm trying to test this out but I'm getting an error
When I run the test file download-webseed-torrent.js in node I get:
TAP version 13
# Download using webseed (via .torrent file)

TypeError: Cannot read property 'length' of undefined
    at WebConn.Wire.handshake (/home/jake/webtorrent/node_modules/bittorrent-swarm/node_modules/bittorrent-protocol/index.js:190:39)
    at WebConn.<anonymous> (/home/jake/webtorrent/node_modules/bittorrent-swarm/lib/webconn.js:26:10)
    at WebConn.emit (events.js:106:17)
    at WebConn.Wire._onHandshake (/home/jake/webtorrent/node_modules/bittorrent-swarm/node_modules/bittorrent-protocol/index.js:391:8)
    at WebConn.<anonymous> (/home/jake/webtorrent/node_modules/bittorrent-swarm/node_modules/bittorrent-protocol/index.js:665:12)
    at WebConn.Wire._write (/home/jake/webtorrent/node_modules/bittorrent-swarm/node_modules/bittorrent-protocol/index.js:555:10)
    at doWrite (_stream_writable.js:226:10)
    at writeOrBuffer (_stream_writable.js:216:5)
    at WebConn.Writable.write (_stream_writable.js:183:11)
    at write (_stream_readable.js:585:24)

@jakefb Hey! Were you by chance using a magnet link when you got that error? I fixed an issue with that in https://github.com/feross/webtorrent/commit/cb5c9a430c53d5cfd5e7db2ef2e0bdba7ec2013d and released it as 0.53.4. Please update and give it another try!


D13 self issued

D13 states: Note that credentials utilized within one or more BPIs may be self-issued.
Isn't that what CR4>D11 implies though? All credentials must be generated by the principal owner.

Question answered. Closing issue


win64 adoptOpenjdk run dacapo--tradesoap complaint java.lang.NoClassDefFoundError

Platform:
windows x64
Architecture:
x86-64
Please describe the issue that you are having with our binaries and if possible how to recreate it.
binary:https://netcologne.dl.sourceforge.net/project/dacapobench/9.12-bach-MR1/dacapo-9.12-MR1-bach.jar
run command:
D:\software\jdk\adoptOpenjdk\jdk-<IP_ADDRESS>-hotspot\bin\java.
exe -jar dacapo-9.12-bach.jar tradesoap
error log:
Using scaled threading model. 48 processors detected, 48 threads used to drive the workload, in a po
ssible range of [1,128]
Exception during initialization: java.lang.reflect.InvocationTargetException
java.lang.reflect.InvocationTargetException
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
        at org.dacapo.daytrader.Launcher.initialize(Launcher.java:70)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:498)
        at org.dacapo.harness.Tradesoap.prepare(Tradesoap.java:45)
        at org.dacapo.harness.Benchmark.run(Benchmark.java:144)
        at org.dacapo.harness.TestHarness.runBenchmark(TestHarness.java:218)
        at org.dacapo.harness.TestHarness.main(TestHarness.java:171)
        at Harness.main(Harness.java:17)
Caused by: java.lang.NoClassDefFoundError: Could not initialize class org.apache.geronimo.system.pro
perties.JvmVendor
        at org.apache.geronimo.system.logging.log4j.Log4jService.logEnvInfo(Log4jService.java:641)
        at org.apache.geronimo.system.logging.log4j.Log4jService.doStart(Log4jService.java:599)
        at org.apache.geronimo.gbean.runtime.GBeanInstance.createInstance(GBeanInstance.java:998)
        at org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.
java:268)
        at org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:102)
        at org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(GBeanInstanceState.ja
va:124)
        at org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanInstance.java:555)
        at org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(BasicKernel.java:379)
        at org.apache.geronimo.kernel.config.ConfigurationUtil.startConfigurationGBeans(Configuratio
nUtil.java:456)
        at org.apache.geronimo.kernel.config.ConfigurationUtil.loadBootstrapConfiguration(Configurat
ionUtil.java:208)
        at org.apache.geronimo.kernel.config.ConfigurationUtil.loadBootstrapConfiguration(Configurat
ionUtil.java:167)
        at org.dacapo.daytrader.DaCapoServerRunner.initialize(DaCapoServerRunner.java:38)
        ... 14 more

Hi @yansendao Can you check with the Azul or Amazon OpenJDK distribution?  I suspect this might be a bug in Apache Geronimo as it may only accept Oracle as the vendor...

Actually, chances are this might be because of #1061

From a quick glance, it looks like org.apache.geronimo.system.properties.JvmVendor does check System.getProperty("java.vm.vendor"), and if it does not find expected values...
else {
// should never happen
return new String("Unknown JVM detected - " + getFullName());
}
So the code does seem to only look for a restricted set of vendors (sun, apache, ibm, ibmHybrid), as @karianna suggests, likely that the JvmVendor code could use an update.

Hi @yansendao Can you check with the Azul or Amazon OpenJDK distribution? I suspect this might be a bug in Apache Geronimo as it may only accept Oracle as the vendor...

Amazon seems to be OK.

Hi @yansendao . A PR which might have resolved this (#1020) was merged so could you give this another go with one of our latest JDK's https://adoptopenjdk.net/releases.html?variant=openjdk8&jvmVariant=hotspot

Closing due to lack of requested feedback. We're happy to reopen the issue if feedback is provided by the OP (or anybody else) at a later date.


onEndReached prop issue

I can't use onEndReached prop because the FlatList do not get that prop.
Can anyone add this line to Flatlist?
onEndReached={this.props.onEndReached}
https://github.com/peacechen/react-native-modal-selector/blob/930572dd70ed84b93b0a58788aed8f3aaa23c328/index.js#L318

Thanks for reporting that oversight. Please submit a PR to add that prop. Remember to add it to the Readme as well.

@peacechen But what PR I need to submit? I don't know.

@peacechen sure! here the PR
https://github.com/peacechen/react-native-modal-selector/pull/182


Add Gabrielaromanc's file

COMMITED

created a branch, created a file and made a commit, and opened a pull request


Update gem to support the latest Solidus version(4.4)

Summary
This pull request resolves compatibility issues with Solidus version updates by adding necessary gem dependencies, modifying forms for proper field rendering, fixing jQuery issues with subscription creation, and updating permission sets. These changes align Solidus Subscriptions with the latest Solidus architecture.
Enhancements:

Form Rendering Restriction

Updated the subscription fields form to render only when the associated product is subscribable.

jQuery Script Fix

Corrected the script used for creating subscription line items to ensure the correct subscribable_id is stored.

Permission Set Updates

Introduced new privilege and category methods for the permission sets to maintain compatibility with the latest Solidus versions.

Fixed breadcrumbs issue

https://github.com/solidusio/solidus_subscriptions/issues/304

Wrap deface overrides in a module

Zeitwerk expects a class or module correctly named in these files in newer versions of Solidus using Zeitwrek.

Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:

[x] I have written a thorough PR description.
[x] I have kept my commits small and atomic.
[x] I have used clear, explanatory commit messages.

The following are not always needed:

📖 I have updated the README to account for my changes.
📑 I have documented new code with YARD.
🛣 I have opened a PR to update the guides.
✅ I have added automated tests to cover my changes.
📸 I have attached screenshots to demo visual changes.

@jarednorman we still struggle with SQLite testing, everything else looks ok. I think the test routine for SQLite is a little flawed. I have the impression it's more SQLite than us.

@jarednorman you are getting soft

#305 cross reference


Generalize/replace Scalarep to allow formal functions: needed for symbolic algerba

At present scala objects such as Long are bound to HoTT types using scala-rep (should describe in the wiki). This has a form of combinators, so that functions between types with scala representatives are represented by scala functions. This is useful in defining, for example, recursion and induction for natural numbers.

In many situations, we would like several different scala types to bind to a HoTT type, with new cases mixed in later. For example this is useful for symbolic algebra.

The weakness of the present structure is that we have a single case class ExtendedFunction that gives formal extensions, containing information about the scala representation. Instead we should have a more general FormalExtension case class, which gives a function depending on a sequence of objects of scalla type Term => Option[Term] (or refinements of this). A scalarep should give a case class extending Term => Option[Term], with an attribute indicating from which scala term it came.

Not clear that multiple representation should be mixed like this. Instead have an implicit based design. Separate issue.


429 additional report metrics

Not sure what will happen if input reports are not using the same Pxx (usually 50/95/99/99.9), I will check this.

Now that we have the Percentiles column set in the table, we can remove the Median column in the Runtime column set.

In case there is no data available to show in a table, the then empty table misses some cells in the table body and footer.

Add an asterisk to the 1/h and 1/d column headers in the Bandwidth table to indicate these values are projected.

Add an asterisk to the 1/h and 1/d column headers in the Bandwidth table to indicate these values are projected.

This applies to diffreport as well as loadtest report, right?

Add an asterisk to the 1/h and 1/d column headers in the Bandwidth table to indicate these values are projected.

This applies to diffreport as well as loadtest report, right?

Right.

While we are at it, could you please add an explanation for the two asterisks at the Distinct** column to the help text of the Requests page? Have a look at the load test report and borrow the text from there.

In config/xsl/loadreport/util/timer-row.xsl, we show the "Distinct" column for requests only. Please simplify the template by using a plain xsl:if instead of xsl:choose.

In case there is no data available to show in a table, the then empty table misses some cells in the table body and footer.

I don't understand, what's wrong with this? Or is the html wrong and Firefox is just filling the gaps itself so that it looks ok?

In case there is no data available to show in a table, the then empty table misses some cells in the table body and footer.

I don't understand, what's wrong with this? Or is the html wrong and Firefox is just filling the gaps itself so that it looks ok?

See below:


Revise the whole library and check for improvements

[x] Decide about semantic versioning
[x] Check for improvements in the HTTP connection class

Done internally


Merged the latest changes for easymesh

Merged the latest changes in the EasyMesh code base.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Behanan:x: behanansajuBehanan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverity detected 1 issue; a security concern.

Orchestration of configuration


Specific Configs to GethDebugTracerConfig + generic config build method for GethDebugTracingOptions

Motivation

Simpler conversion from specific tracer config to GethDebugTracerConfig
Single config build method on GethDebugTracingOptions that covers all configs

Solution

Impl From for CallConfig, PreStateConfig, MuxConfig
Introduce generic build method for trace configs on GethDebugTracingOptions

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

cc @mattsse

@halo3mic are you still on this? Else I can take this over, we'd like to wrap it up


Change Varbits.java to include bolt pouch

Add varbits for bolt pouch quantities and bolt types.

Do you need this for an external plugin?

Yes, it is for an external plugin

I think we would not mind having a bolt pouch overlay in master. If you are set on making an external plugin you can use client.getVarbitValue that we made so external plugins do not need to touch the varbits file.

If you would like the whole bolt pouch overlay plugin in master I can modify this pull request to include the entirety of it instead of just the varbits or make another pull request. The plugin is working and based on the rune pouch plugin.

I am unsure whether we would like this as its own plugin or combined with the rune pouch plugin but I think it would be fine in master.

This definition can be hard-coded into the external plugin or introduced with a main repo feature which uses it. Since there's no real reason to have this added in isolation, I am closing this PR.


dependency patching

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
1.14.0 could stand some patching on it's dependencies. Please see the following scan results:
Scan results available on request
Affected area/feature
Software Compositional Analysis

@GeoffMillerAZ can you please include the dependencies you'd like to see patched?

@GeoffMillerAZ If the list/scan result is sensitive, perhaps you can post on in internal issue? Or should we just close this out for now?


feature request: build_with_source_file

In program we have create_with_source and create_with_source_file, but only build_with_source. Could we get a build_with_source_file too? Thanks!

Fixed by #739


Use Nipoppy schema to validate proc status TSV for bagel derivatives command

Following #369, we should also consider requesting that Nipoppy keep a copy of a reference example processing status file in some repo they can easily update (maybe pipeline-catalog? or in the nipoppy repo itself). We can simply grab that file and use it in our tests.
This way, if the schema changes, presumably that example file will change, and our tests should start failing - so we should catch incompatibilities straight away.

Currently blocked by #180, due to Nipoppy schema being written in v2 (and we would have to use their model as a module, not jsonschema, due to the internal validation they have)

Tracked while we discuss how to do this

Decision: We will implement the following as part of a new issue: grab example imaging status file from a nipoppy maintained repo (e.g. catalog) and use it as a test target. Keep this example in sync with upstream nipoppy (e.g. submodule) so tests start breaking when we go out of sync


Add FromStr for DynamicColor and static colors

This makes it easier to parse them from a string with more concise code and not requiring a separate import of parse_color.

If we like this idea, then I'd add tests to the PR.

This would improve code in Vello's pico_svg as well as my current patches to svgtypes.


Guest mode and sandbox before honeypot for safe interaction

08062021
Security measure to avoid attacks and prevent data loss.

(virtualization software)
redns
encfs
dnscrypt
httpseverywhere
opendns

added a proxy before firewall (redns)
for something more


Add Latin american spanish suggested adjustements

As requested by @ericherman , I am submitting here my suggestions for the Spanish translation. I hope that you find them useful.

Wow, @ejortegau thanks a lot for this extensive work!
It might take us some time to review it, though.

Thanks again for the time you have invested in this pull request, @ejortegau !
After reviewing your suggestions and discussing them with some members of the Public Code Foundation, I have decided to proceed as follows. Some of your suggestions (typos, better ways to refer to specific terms, etc.) are naturally going to be accepted. On the other hand, those referring to terms used in Latin America will unfortunately not be incorporated at this time. However, given the time it must have taken you to review this pull request, I would like to encourage you (if you feel like it, of course!!) to create another independent pull request with the Latin American Spanish version!


curl

IPFS integration in curl is coming! https://github.com/curl/curl/pull/8805
As of this writing, most of it is there and done.
Only thing left to do i making a couple testcases.

Completed as of Sept. 23, 2023. To be released on version 8.4.0 which is probably going to be released somewhere mid-late november.


Atom 1.0.10/11 broken in Ubuntu 14.04?

Hi
I cannot get Atom 10.0.11 to run on my Ubuntu 14.04 VM. I downloaded the .deb package and installed with dpkg -i as many times before.
It doesn't work and I do not get any reasonable error messages displayed:
~  atom -f [20921:0906/171447:ERROR:browser_main_loop.cc(170)] Running without the SUID sandbox! See https://code.google.com/p/chromium/wiki/LinuxSUIDSandboxDevelopment for more information on developing with the sandbox on. App load time: 247ms [20921:0906/171449:INFO:CONSOLE(46)] "Window load time: 694ms", source: file:///opt/atom/resources/app.asar/static/index.js (46) [20921:0906/171527:ERROR:channel.cc(305)] RawChannel read error (connection broken)
But the application remains unusable and has graphical issues.

Any ideas how this issue could be resolved?
Thanks

Try running as sudo.

Same result

@binaryanomaly follow here mate https://github.com/atom/atom/issues/8579

Thanks. Interestingly I have no issues with my native 15.10 installation but that may not be a stable reference, though...

@binaryanomaly could be it, i'm running 15.04 x64.


Not able fetch  API endpoint request

Describe the bug
I have used axios to gte the data from Lokalise API end points and  I get the
ERROR ----> Error: socket hang up     at connResetException (https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:6:6855)     at TLSSocket.socketOnEnd (https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:6:910485)     at TLSSocket.EventEmitter.emit (https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:6:131338)     at endReadableNT (https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:6:540012)     at Object.callback (https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:15:188119)     at https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:15:167184     at _0x16827a (https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:15:167218)     at MessagePort._0x231308.port1.<computed> (https://nextjs-ctyzc2.w.staticblitz.com/blitz.6495216afc69f31e3592b868d6f1407d8619435d.js:15:167376) {   code: 'ECONNRESET',   config: {
Link to the blitz that caused the error
https://stackblitz.com/edit/nextjs-ctyzc2?file=lokalise.js
Steps to reproduce

Run this Stackblitz
Create a Lokalise API token - https://docs.lokalise.com/en/articles/1929556-api-tokens
Create a .env file in the root and add LOKALISE_API_KEYvariable.
run node lokalise.js

Expected behavior
Fetch data without issue.
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform

OS: macOS
Browser: Chrome
Version: 99.0.4844.74

Additional context
No response

@anilkk Thanks for the report. This is most likely due to a CORS error, which we cannot work around given the security model of the browser.

No problem :+1:  I'm going to close this for the time being. We do keep the CORS issue in our minds and StackBlitz might change in the future to unblock more use cases, but there is nothing immediately actionable.


mdx-converter: Create the microservice workspace under services/mdx-converter

no longer in scope.


⚠ Samsung has degraded performance

In 356779a, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1636 ms

Resolved: Samsung performance has improved in 4a82d3e.


mode is not set so webpack defaults to production

mode is not for dll so webpack it to production which results in minified dll, Instead mode should either "inherited" or set to "none" so webpack is not minifying dll in all cases automatically.
now i'm doing this:
new AutoDllPlugin({
       config: { mode:"none" },
       plugins: [...

to avoid unneeded minification.

Oh, that's very important for React projects because the minified version doesn't show full error messages. Tks!

This totally saved me. Should be added to documentation!


Prepare a UI

Make a simple and basic UI for the interface of the web app like chatGPT home page, In this remove the typing section.

Hey @mohsinansari0705  can you assign this issue to me?

Hey @mohsinansari0705 I have sent the PR request pls check it....

@snehautekar05 As you have recently pull your changes but due to some sort of issues, I have close your PR.
I review your contribution and glad to say that it a awesome.
Now you have to make a new PR to be merged to resolve this issue.


Report a possible race that uses goroutines on shared variables.

What did you see instead? Under which circumstances?
At https://github.com/prometheus/prometheus/blob/main/tsdb/db_test.go#L708
https://github.com/prometheus/prometheus/blob/a7d499e19a932f4f7cf14e8b8c935dd43d2a9830/tsdb/db_test.go#L708
Within a loop "for _, c := range cases {", variable db is shared by all the loop iterations since it is defined outside the loop:
db := openTestDB(t, nil, nil)
for _, c := range cases {
		db, err = Open(snap, nil, nil, nil, nil)
		defer func() { require.NoError(t, db.Close()) }()
}

All the defer statements within the loop may refer to the last value updated by the loop. As a consequence, the last db may be closed multiple times, while the other dbs are not closed.
More information on the bug can be found at https://github.com/golang/go/wiki/CommonMistakes#using-goroutines-on-loop-iterator-variables.
What did you expect to see?
A fix is to add db as a parameter to the closure so that db is evaluated at each iteration and placed on the stack for the goroutine rather than being shared.
db := openTestDB(t, nil, nil)
for _, c := range cases {
		db, err = Open(snap, nil, nil, nil, nil)
		defer func(db *DB) { require.NoError(t, db.Close()) }(db)
}

Another fix is to introduce a local storage for db:
db := openTestDB(t, nil, nil)
for _, c := range cases {
		db, err := Open(snap, nil, nil, nil, nil)
		defer func() { require.NoError(t, db.Close()) }()
}

Found by #deepgo.

Thanks for the report. We should not override the db since it is used for delete at the start of the loop, and we cannot do a := later in the loop with db, so a suggested way is to rename the db in the loop to something like newDB. Additionally, we should have a db.Close() for the one opened outside the loop.

I can work on this @codesome

@girishramnani please go ahead!


firewall: show "log" icon for RFC 1918 and IANA rules

Show a "log" icon if the RFC 1918 rule or the reserved IANA rules
are configured to be logged.
Fixes parts of #495

Merged this manually, thanks! Commit author/email all good now.


Analytics : How is share performing

Hi,
Noticed we have segment.io for analytics, we should somehow expose some of the data to help us make data-driven development. I want to know how is the share on surah pages performing, are they getting a decent number of clicked?
data-metrics-event-name="Share:Twitter"
data-metrics-event-name="Share:Facebook"

@sabeurthabti good question. We are not using Segment.io or Mixpanel unfortunately since, well, we won't be able to afford it. We have so many users and events that we really quickly surpass the free tier

@sabeurthabti send me an email at<EMAIL_ADDRESS>


#21 Remove all Native modules before move to Elm v0.19

No Native modules allowed in Elm v0.19
So before migration, we need to get rid of all the "Native" modules.
We could do it using "ports" but I want to experiment with the idea of using XHR request interceptor
to treat Bowser as a node.js server with expres.js like API for Elm (see /client/ElmExpress.js).
// index.js
const ElmExpress = require('./ElmExpress')
ElmExpress.create(window, router => {

      router.get('localStorage', (req, res) =>
          res.text(window.localStorage.getItem(req.params.get('key'))))

      router.post('localStorage', (req, res) =>
          res.ok(window.localStorage.setItem(req.params.get('key'), req.body)))
})

It is very easy to use. Any request to elm: will be intercepted by the browser.
fetchAll : (Result Error String -> Msg)  -> Cmd Msg
fetchAll tagger =
    Http.send tagger <|
        Http.getString "elm:localStorage?key=starred.eventTypes"

saveAll : (Result Error String -> Msg)  -> String -> Cmd Msg
saveAll tagger data =
    Http.send tagger <|
        Http.post "elm:localStorage?key=starred.eventTypes" (Http.stringBody "" data) string

Native Ace rich editor replaced by Web Component (custom element)
Request for element "width" replaced by parsing event.target.clientWidth
internalLink moved to the Routing.Helpers module to break circular dependancy.
link replace by internalLink
I use the new version of elm-format. Unfortunately, it produces different formatting, resulting in too many lines changed.

:+1:

👍

:+1:


Update secret.md

We had to escape a password that ended with = to make the secret work with our pod.
So i updated the section about special characters to also contain the character =.

Hello @cristobal . Thanks for opening this pull request.
Page preview:
https://deploy-preview-20645--kubernetes-io-master-staging.netlify.app/docs/concepts/configuration/secret/

/assign

Update covers another special character case. Very helpful and page renders great.
Great job and thank you for your contribution.
/lgtm
/approve

Thanks for the positive feedback 🙂


Buttons of founders in Team section not working

Describe the bug
All the socials of both the founders are not working as they should.
Expected behaviour
They should redirect to their socials but instead are opening the Home section!!!
Screenshots

Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@SahityaRoy Please assign me this issue under GSSOC'23.


Fail to install due to missing ?

I followed the detail instructions on https://github.com/ethereumjs/testrpc/wiki/Installing-TestRPC-on-Windows
yes I am a windows user, no I couln'd pin it down to windows
Expected Behavior
testrpc is runnable
Current Behavior
npm install -g ethereumjs-testrpc exits with an Error
Possible Solution
I haven't the faintest
Steps to Reproduce (for bugs)

do as suggested
run npm install -g ethereumjs-testrpc
the error shows in the screen log ...

C:\Users\Jelle\AppData\Roaming\npm\node_modules\ethereumjs-testrpc\node_modules\scrypt>if not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\npm\bin\node-gyp-bin\....\node_modules\node-gyp\bin\node-gyp.js" rebuild )  else (node "" rebuild )
Warning: Missing input files:
C:\Users\Jelle\AppData\Roaming\npm\node_modules\ethereumjs-testrpc\node_modules\scrypt\build..\scrypt\win\include\config.h
Die Projekte in dieser Projektmappe werden nacheinander erstellt. Um eine parallele Erstellung zu erm▒glichen, m▒ssen Sie den Schalter "/m" hinzuf▒gen.
VCBUILD : error : project file 'C:\Users\Jelle\AppData\Roaming\npm\node_modules\ethereumjs-testrpc\node_modules\scrypt\build\copied_files.vcproj' was not found or not a valid project file. [C:\Users\Jelle\AppData\Roaming\npm\node_modules\ethereumjs-testrpc\node_modules\scrypt\build\binding.sln]
Your Environment
Windows 10 German Version, Visual Studio 8 Express, etc a shown in the installation instructions

I have the same problem... really frustrating.

Install https://github.com/felixrieseberg/windows-build-tools and then testrpc

@fanatid I have just tried, nothing changes, it still does not work

@fanatid Thank you for sharing your valuable expertise. Felix Riesebergs script, that you recommend, was smart to discover that I lack administrator access rights when running my GitBash. When I start GitBash as an admin then testrpc installs OK. Problem solved! (for me)
To the good people of testrpc.

Can all of you give the TestRPC beta a try? First uninstall the normal one, if installed (or partially installed):
$ npm uninstall -g ethereumjs-testrpc

And then install the beta version:
$ npm install -g ethereumjs-testrpc

The beta version significantly improves installation and should bypass your installation issues.
Since this is a beta version, I'd very much appreciate feedback on how it works for you and if you run into any errors during use.

Sorry for keeping you waiting. I was on a trip. Will do soon.
@tuytuy how did the beta version work for you?

@tcoulter I reinstalled and experienced no problems. All worked fine when I used the beta installer.


MemoryPKG documentation is corrupted

Please look at MemoryPKG_user_guide.pdf
It looks that most of it is just copy&paste from AlertLogPKG documentation

I can confirm; looks like some build script somewhere got tied in knots.

Thanks.  Found the correct file and updated the master and dev.