VS Code's debug console cannot display complete data.

Environment data

VS Code version: 1.55.0 (user setup)
Extension version (available under the Extensions sidebar):  Python Extension version:2021.3.680753044
Python version (& distribution if applicable, e.g. Anaconda):Python3.8.3
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: none
Relevant/affected Python-related VS Code extensions and their versions: Jupyter: 2021.5.702919634  Pylance: 2021.3.4
Value of the python.languageServer setting: Pylance

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
The debug console can display complete data:

Actual behaviour
Only part of the data is shown:

Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

code:

my_list = []
for i in range(0,100000):
    my_list.append(i)
print(my_list)

Run the above code, click the run button and then the data is displayed completely, but click F5 to debug the code, the debug console only displays part of the data.

Related Links: Running python script in VSCode produces truncated output

Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

none

Same issue here, I could reproduce. It's imo probably best that there would be at least a setting for allowing complete output.


UIKit Lightbox close button not working properly in presence of VUIkit CSS

It looks like the VUIkit CSS for the modal transitions is breaking the regular UIKit Lightbox because it uses a normal modal under the covers.
With the changes by VUIKit to the transition CSS for modals, a CSS transition event that is assumed to happen on a normal modal is not firing. As a result, the close button doesn't work properly because it's trying to hook onto this transition event that is not firing (but does fire later after some other transition occurs).
To recreate:

Set up VUIkit.
Set up a Lightbox per UIkit documentation with a group (two images).
Trigger the lightbox.
Click the close button - nothing happens.
Navigate to the second image.
After the second image loads/animates in, the lightbox closes.

Would it be possible to get some native support for Lightboxes in VUIkit? Or barring that, could we get some CSS for the modal transitions that doesn't interfere with the operation of the stock Lightbox component from UIkit?

Thanks for the feedback @davidkhess. Using Vuikit in combination with UIkit was never the goal, thus those compatibility tests were ignored. Anyway, Vuikit next version will rely entirely on UIkit default CSS and that kind of issues should be gone. Meantime, the only solution I could think of is to create a separate UIkit theme with a different prefix, or use UIkit Modal instead.
Note that Vuikit next is being deeply refactored, it will take some time to get a public release ready.

Thanks for the info. The plans for the next version sound good - since VUIkit is just a subset of what's possible with UIkit, it makes it harder to justify using it if it means the rest of UIKit becomes unsupported – and possibly broken. It sounds like that will not be the case with the next version.
I ended up lucky in this case and could remove VUIkit from the page in question.


⚠ Cioos National CKAN has degraded performance

In 7d42b49, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 9231 ms

Resolved: Cioos National CKAN performance has improved in 3c04388 after 10 minutes.


Add optional gzip flag to handler

Overview
The previous routing gzipped all routes and overwrote the Content-Length header. This header was needed for the metastore copy and caused it to break clustering. This commit fixes this issue by adding a gzipped flag to the route struct.

+1

Pity gzipping support isn't somehow more transparent. +1 if it works.


built-in rules: response-contains-header -- force toLowerCase on comparison

RFC2616 section 4.2 indicates all headers are case-insensitive. I believe following the spec will provide consistent behavior for users of this rule.
https://www.rfc-editor.org/rfc/rfc2616.html#section-4.2

Thank you!


fix: pluralize table name when building join table index query

Problem

update forms are generated with import statements for non-existent join table query
Solution

pluralize the join table name when building the query name to get the correct import
Additional Notes

Links
Ticket

GitHub issue:
Other links
Verification
Manual tests

Automated tests

[x] Unit tests added/updated
[ ] E2E tests added/updated
[ ] N/A - (provide a reason)
[ ] deferred - (provide GitHub issue for tracking)

Housekeeping

[ ] No non-essential console logs
[ ] All new files contain license notice

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #1070 (0416fd3) into main (d066faa) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main    #1070   +/-   ##
=======================================
  Coverage   93.88%   93.88%
=======================================
  Files         124      124
  Lines        5840     5841    +1
  Branches     1775     1775
=======================================
+ Hits         5483     5484    +1
  Misses        339      339
  Partials       18       18

Files Changed
Coverage Δ

...act/lib/forms/form-renderer-helper/relationship.ts
93.04% <100.00%> (+0.03%)
:arrow_up:

Continue to review full report in Codecov by Sentry.

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


dart2js: identical(NaN, NaN) should be true.

The spec now says explicitly that identical(c1, c2) is true if both c1 and c2 are doubles representing NaN.
Dart2js generates code that makes
 var n = double.NAN;
 print(identical(n, n));
print false. The code seems to be:
  function() {
    $.print(0 / 0 === 0 / 0);
  }

That depends entirely on the meaning of "-double.NAN". We haven't specified it, so we would assume some kind of IEEE-754 related behavior. IEEE-754 2008 says that operations using quiet NaNs should return one of the incoming NaNs, so returning the "-NaN" value for the minus operation on NaN isn't really proper.
When compiling to JS we inherit the JS behavior. JS doesn't specify what to do in cases like this - the spec only has one NaN value and doesn't give a way to distinguish them, but typed arrays may allow you to see the difference anyway. Implementations ... differ :)
So, if we return the negative NaN for -double.NaN then it's not identical to double.NaN, but nothing says we have to do that, and the double spec suggests otherwise.
The test is actually comparing identical(double.NaN, double.NaN) which must always be true - it's two references to the same object (double.NaN is a constant, not a getter). The test that identical(0.0/0.0, double.NaN) is not required to be true, though - we may pick another NaN representation.

Imho the correct behavior is to look for the bit-pattern. This guarantees that writing a value into a byte-buffer will give the same value if two values are identical.
It's unlikely that dart2js will correctly implement the spec. Since 'identical' is (unfortunately) frequently used for performance-critical code it's probably too expensive to compare the bit-pattern (by writing into a byte-array).

On Mon, Jun 15, 2015 at 8:34 AM, Florian Loitsch<EMAIL_ADDRESS>wrote:

Imho the correct behavior is to look for the bit-pattern. This guarantees
that writing a value into a byte-buffer will give the same value if two
values are identical.
It's unlikely that dart2js will correctly implement the spec. Since
'identical' is (unfortunately) frequently used for performance-critical
code it's probably too expensive to compare the bit-pattern (by writing
into a byte-array).

Agree with bit-pattern, disagree with using it in performance-critical
code: for VM  in most cases '==' is faster  than 'identical' (because of
special number checks needed).

Srdjan

—
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/sdk/issues/11551#issuecomment-112111760.


Validation layers have multiple bugs with destroyed render passes

The validation layers incorrectly assume that the render pass that a pipeline was created from must be valid or even the same at the time of vkCmdBindPipeline.
This is wrong. The spec specifically says that the render pass may be deleted or the pipeline used with another compatible render pass.
E.g. in CmdBindPipeline:
auto rp_state = GetRenderPassState(dev_data, pipe_state->graphicsPipelineCI.renderPass);
if (rp_state) {
addCommandBufferBinding(&rp_state->cb_bindings, {HandleToUint64(rp_state->renderPass), kVulkanObjectTypeRenderPass}, cb_state);
}
You cannot assume that the renderPass used to create the pipeline (graphicsPipelineCI) is related to the render pass used in the draw.
I have found similar bugs in other places.

@tobine, I thought this was recently addressed... Maybe e03f0c649d ?

I'm using <IP_ADDRESS> layers, this might be fixed in top of tree.

Yes, all the renderPass lifetime bugs should be fixed in ToT


Added the Doxygen API documentation to the repository

Added the API documentation to the repository.

Windows test FAILed.

Linux test PASSed.


Generated links problem !!

i have this code :
<%  current_user.locals.each do |l| -%>

		<%= link_to l.name.capitalize, local_path(l) %>

<% end %>

which generate :

		Coffe bar

		Itaila restaurante

but when i go to each links and use back button to return to home page i got now

		Coffe bar

		Itaila restaurante

2 generated links pointed on the same page now.
How i can resolve this problem please help me i have been on this problem for too long time.
Thanks to all the team for the great job you have done !

Yep, you guys got it :)


GWEN is failing while executing system process

Steps to reproduce -

System process step is failing with error --> And I execute system process "cmd.exe /c echo Hi > test.txt"
This was working on GWEN 3.2.0

Env -
GWEN version - 3.31.4
Chrome driver - Latest

Issue was that windows commands were being execute like unix commands. Fixed. Will be available in next release.

Fix released in gwen-web v3.31.6. Please close when ready.

Tested on GWEN 3.31.6, working as expected.


IDSCPv2 examples failing to establish provider-consumer connection

Describe the bug
When trying to run the new IDSCPv2 examples available here, we're not able to establish the connection and send data between the provider and consumer.
When running the provider before the consumer, we're getting the following exception:
org.apache.camel.FailedToCreateProducerException: Failed to create Producer for endpoint: idscp2client://consumer-core:29292?connectionShareId=pingPongConnection&sslContextParameters=%23clientSslContext.
Reason: de.fhg.aisec.ids.idscp2.error.Idscp2Exception: Connecting TLS client to server failed

When running the consumer before the provider, there seems to be a problem with the DAPS certification and the connection is being immediately closed before any data is sent after throwing the following exception:
javax.net.ssl.SSLPeerUnverifiedException: Hostname verification failed. Peer certificate does not belong to peer host

We pulled the last version of the repository this morning and rebuilt all docker images using the buildx/docker-buildx.sh script.
To Reproduce
Steps to reproduce the behavior:

Clone this repository
Go to examples/example-idscp and run the ttp docker container.
Go to examples/example-idscp2 and run the consumer and provider docker containers.

Expected behavior
The provider and consumer should be sending alternating "ping" and "pong" messages every 10 seconds, which does not happen.
Screenshots and logs

Log file when running provider before consumer (stack trace at line 1242): karaf.log
Log file when running consumer before provider (stack trace at line 1307): karaf.log

Environment (please complete the following information):

OS: Docker Desktop on Windows 10
Environment: Docker version
Version: a90f87c

Additional context
N/A

I checked this today, and it's working flawlessly. But there has indeed been an issue with the hostname verification in IDSCP2 which caused the example to fail, maybe you missed my patch for just some hours or days.
Could you please re-check with the latest develop and report back if it works?

Thanks for looking into it. Indeed I tried pulling again from develop and now the consumer and provider are able to connect without a problem. I will close this issue.

Glad to hear that!
Today I realized that my latest IDSCP2 protocol/example adaptations contained an error that causes an indefinite loop, patch goes to develop in a few minutes.


[rcore] Feature request/discussion: MaximizeWindow() on HTML5 to maximize the canvas to fill the browser viewport?

Currently calling MaximizeWindow() on HTML5/WASM does nothing but print a message to the console:
WARNING: MaximizeWindow() not available on target platform

I want maximize window to maximize the canvas to fill the viewport of the browser. Note that this is different from fullscreen which also hides the browser chrome like the Navigation bar etc..
I want something like this in rcore_web.c:
EM_JS(void,  EmscriptenMaximizeWindow (), {
    var canvas = document.getElementById('canvas');
    canvas.style.width = "100%";
    canvas.style.height = "100%";
    canvas.style.position = "absolute";
    canvas.style.top = "50%";
    canvas.style.left = "50%";
    canvas.style.transform = 'translate(-50%,-50%)';
});

void MaximizeWindow(void)
{
    EmscriptenMaximizeWindow();
}

Or alternatively an option that disables canvas sizing on InitWindow so the canvas can be layouted using CSS in the shell.html file.
See also: https://www.reddit.com/r/raylib/comments/1g4gg27/raylib_wasm_how_to_resize_the_canvas_to_fill_the/
If this is something you do not want to support you can close this issue.

MaximizeWindow() is easy to implement. However it shouldn't be done with canvas.style as it will force scaling upon the window (canvas) which isn't the maximizing standard behavior for a window. Here's a diff that also includes the RestoreWindow() implementation and integrates with the WindowSizeCallback:
diff --git a/src/platforms/rcore_web.c b/src/platforms/rcore_web.c
index 35c15a6b..e556f9fc 100644
--- a/src/platforms/rcore_web.c
+++ b/src/platforms/rcore_web.c
@@ -74,6 +74,8 @@
 typedef struct {
     GLFWwindow *handle;                 // GLFW window handle (graphic device)
     bool ourFullscreen;                 // Internal var to filter our handling of fullscreen vs the user handling of fullscreen
+    int unmaximizedWidth;               // Internal var to store the unmaximized window (canvas) width
+    int unmaximizedHeight;              // Internal var to store the unmaximized window (canvas) height
 } PlatformData;

 //----------------------------------------------------------------------------------
@@ -317,7 +319,16 @@ void ToggleBorderlessWindowed(void)
 // Set window state: maximized, if resizable
 void MaximizeWindow(void)
 {
-    TRACELOG(LOG_WARNING, "MaximizeWindow() not available on target platform");
+    if (glfwGetWindowAttrib(platform.handle, GLFW_RESIZABLE) == GLFW_TRUE)
+    {
+        platform.unmaximizedWidth = CORE.Window.screen.width;
+        platform.unmaximizedHeight = CORE.Window.screen.height;
+
+        const int tabWidth = EM_ASM_INT( { return window.innerWidth;  }, 0);
+        const int tabHeight = EM_ASM_INT( { return window.innerHeight; }, 0);
+
+        if (tabWidth && tabHeight) glfwSetWindowSize(platform.handle, tabWidth, tabHeight);
+    }
 }

 // Set window state: minimized
@@ -329,7 +340,10 @@ void MinimizeWindow(void)
 // Set window state: not minimized/maximized
 void RestoreWindow(void)
 {
-    TRACELOG(LOG_WARNING, "RestoreWindow() not available on target platform");
+    if (glfwGetWindowAttrib(platform.handle, GLFW_RESIZABLE) == GLFW_TRUE)
+    {
+        if ( platform.unmaximizedWidth && platform.unmaximizedHeight ) glfwSetWindowSize(platform.handle, platform.unmaximizedWidth, platform.unmaximizedHeight);
+    }
 }

 // Set window configuration state using flags

Test example
#include "raylib.h"
int main(void) {
    InitWindow(800, 450, "test");
    SetTargetFPS(60);

    SetWindowState(FLAG_WINDOW_RESIZABLE);

    while (!WindowShouldClose()) {

        if (IsKeyPressed(KEY_ONE)) { MaximizeWindow(); }
        if (IsKeyPressed(KEY_TWO)) { RestoreWindow(); }

        BeginDrawing();
        ClearBackground(RAYWHITE);
        DrawText("test", 20, 20, 20, BLACK);
        EndDrawing();
    }
    CloseWindow();
    return 0;
}

Let me know if you want me to PR this.

Let me know if you want me to PR this.
I'll have to try it out first. I'll try it tomorrow.

For convenience: https://github.com/asdqwe/raylib-fork/tree/add-maximize-restore-web

However it shouldn't be done with canvas.style as it will force scaling upon the window (canvas) which isn't the maximizing standard behavior for a window.

I agree. It is just that I did not notice because I was using the 3D camera. The behaviour of MaximizeWindow() should be consistent across platforms.

I'll have to try it out first.
Well, when I tried it at first there was a tiny margin around the canvas element and so it has scrollbars. I eventually figured out that it was because my browser adds an 8px margin around the HTML body and shenanigans like that. It works fine now.

I think the ability to maximize a window is useful. It is a bit less attractive as a feature to me now because disabling the margins on body and canvas elements is an extra thing to do.
The other solution would be to have a flag/mode for raylib, where InitWindow does not change the width and height of the canvas so the canvas can be layouted statically and maximized using CSS. This would have the advantage of allowing a raylib app be in a blog post or in a html table or something. The disadvantage of this would be that it is not possible to maximize restore the canvas during the runtime of the raylib app.

Let me know if you want me to PR this.
I want it, but I cannot speak for others in the raylib community.

PS:
One question: why did you call assign the innerWidth/innerHeight of window to a variable called tabWidth/tabHeight as I don't quite get the "tab" naming?

It is a bit less attractive as a feature to me now because disabling the margins on body and canvas elements is an extra thing to do.

I believe that's more or less unavoidable when deploying for web. Some degree of shell file CSS customization always ends up being necessary to fit how it will be deployed.

The other solution would be to have a flag/mode for raylib, where InitWindow does not change the width and height of the canvas so the canvas can be layouted statically and maximized using CSS.

I'm not sure if that can be done. The initial width and height are required to setup the viewport, render, fbo and screen size which most of the library/render depends on.

I want it, but I cannot speak for others in the raylib community.

Cool, I'm send the PR for review.

One question: why did you call assign the innerWidth/innerHeight of window to a variable called tabWidth/tabHeight as I don't quite get the "tab" naming?

The window.inner* is the web browser's tab (inner) "area". Which is the actual usable area the canvas can fill without overflow.

The window.inner* is the web browser's tab (inner) "area". Which is the actual usable area the canvas can fill without overflow (note: if the page is inside an iframe, it will be the iframe dimension).

Makes sense now.
In my weird personal use of the english language the word "tab" means a navigational element at the top not the document view area below it. Maybe you can argue that a tab is actually the size of the navigational element plus the document area, so I guess the tab is larger than the view area as it surrounds it. Actually they are inspired by tabs in filing cabinets which stick to the documents, you see the size of the tab is the small thing that sticks out plus the bit that sticks to the paper card.
You see I would have called it innerWidth or viewportWidth. But you see I am overthinking things about a detail that does not matter.


Sidebar "slide in" animation sometimes stops before the end

Bug Description
The animation doesn't finish and leaves a lot of space on the left:

How To Reproduce
Not sure how, last time I found it was simply by loading this URL: https://review-python-editor-next.microbit.org/less-reference/?reference=accelerometer
Expected behavior
"Accelerometer" slide-in animation to finish and place the content to the final position.
Screenshots
At the description.
Environment

OS: macOS
Browser: Chrome
Version: 103

Additional context
N/A

We see this from time to time when refreshing URLs that specify a reference section.
It's currently hard to get hold of these URLs from our beta deployment because of the iframe-embedding the versioner uses.
We should definitely fix this if we start linking to the docs ourselves or remove the iframe (which we hope to do at some point).


Download stops after first subreddit/user when providing a list

The scripts exits after completing the first subreddit or user when providing a list of subreddits/users.
The only special settings I used ist to set the number of posts to max (0 at the propmt). At least it should be higher than the available posts of the subreddit.
I identified the issue is on this condition:
https://github.com/josephrcox/easy-reddit-downloader/blob/9c92f76e906c1f19ee7e8643233571fdc92beb25/index.js#L1080
With those changes it worked for me, but not sure if it's fixed correctly at all:
		if (numberOfPostsRemaining()[0] === 0 ||
			(lastAPICallForSubreddit && lastPostId === currentAPICall.data.children[responseSize - 1].data.name) ||
			(numberOfPostsRemaining()[1] === responseSize && responseSize < 100)) {

Also the amount of posts of a subreddit or user has affects to the condition, if it has more than 100 or not.
Once workarounded this condition, I also got stuck here in the console, which actually makes no sense when running with testing mode params:
https://github.com/josephrcox/easy-reddit-downloader/blob/9c92f76e906c1f19ee7e8643233571fdc92beb25/index.js#L1121
				} else if (!testingMode && !config.download_post_list_options.enabled) {

@knoxitus Do you mind submitting a PR? The lines got switched up with the latest PRs merged in, but if you can get that – I can test and review!


[BUG] mne.label.find_pos_in_annot doesn't check coord_frame

It doesn't seem like a good idea to allow any position to be passed to find_pos_in_annot without checking that it is in the mri coordinate frame. If someone has the wrong coordinate frame, the error will be silent and they will get the wrong ROI which to me seems like a pretty good risk for incorrect results that could be avoided by enforcing the coordinate frame or maybe rather making this a class function of Dipole.

I don't think Dipole would be the right subclass, in general the point you're talking about could be anything (iEEG sensor, part of a mesh, an elementary current dipole, etc.). I think we should do two things:

Document clearly that if an ndarray or list-like is passed, these must be in the MRI coordinate frame
Add support for other objects as we need them, such as DigPoint instances (?), where we can check the coordinate frame

FWIW Dipole always lives in the head coordinate frame, so if we wanted to add support for that in (2) then we'd have to have a trans argument in that function, if we don't already.
But really I think we should do the doc update (1) first and then let actual use cases guide (2)

Maybe it should be in a mixin for things with positions?

What objects + use cases do you have in mind?

I think just DigMontage and Dipole unless you know more that have positions... now that I say that it might be better just to make an inst that takes either

Dipole is always in head coords and I don't think it's a common use case anyway. Let's just start from montage (and not worry about Dipole) since it's the use case we actually want to support for iEEG?


No "GraalVM" debugger launch environment available

The docs for the GraalVM VS Code extension have several sections on debugging various Truffle languages. In each of them, there's some variation of the following:

To debug a Ruby application running on GraalVM, creating a launch configuration for the application is necessary. To do so, open the application project folder in VS Code (File > Open Folder), then switch to the Debug view by clicking on the "bug" icon in the left-hand side panel. The newly opened window will suggest you create a launch.json file. If debugging is not yet configured (no launch.json has been created), select GraalVM from the list of available debug environmnets [sic].

However, there is no "GraalVM" option available in the list of debug environments. It appears that either the option is missing or the documentation is out of date.

There is this list of GraalVM Launch configurations available when you click the Add Configuration in launch.json

Is not this what you are looking for and it also starts with GraalVM text


Fix correction of example code for components catchAll

Warning: Functions are not valid as a React child. This may happen if you return a Component instead of  from render. Or maybe you meant to call this function rather than return it.

Hey @nrikiji ,
Congratulations and many thanks. Wouldn't it be better if we change CustomErrorPage from React.FC to React.Element? What do you say?
like this,
const CustomErrorPage =
Page not found
;

const App: React.FC = () => {
    return (
        <Refine
            ...
            catchAll={CustomErrorPage}
        />
    );
};

const CustomErrorPage =
Page not found
;

const App: React.FC = () => {
    return (
        <Refine
            ...
            catchAll={CustomErrorPage}
        />
    );
};

Thank you. This description is very clear.


ReportingGroup v ReportGroup wording inconsistency

in the UI there is "REporting group" and also "Report Group" - should be "Report Group" consistently

Closed by #44


[BUG] - Low priority - DNSZone - complex first NS hostname

Bug Report
Description
Kubebuilder validation restricts the spec.primaryNS.hostname field to a simpler hostname format, preventing the creation of more complex hostnames required for certain DNS setups.
Steps to Reproduce
Please provide detailed steps to reproduce the issue:

Create a DNSZone resource with a complex hostname.cat << EOF | kubectl apply -f -
---
apiVersion: monkale.monkale.io/v1alpha1
kind: DNSZone
metadata:
 name: bubble-lab
 namespace: kube-system
spec:
 connectorName: coredns
 domain: "bubble.lab"
 primaryNS:
   hostname: master-0.k3s
   ipAddress: "<IP_ADDRESS>"
 respPersonEmail<EMAIL_ADDRESS>EOF

ErrorThe DNSZone "bubble-lab" is invalid: spec.primaryNS.hostname: Invalid value: "master-0.k3s": spec.primaryNS.hostname in body should match '^[a-zA-Z0-9]([-a-zA-Z0-9]*[a-zA-Z0-9])?(\\.[a-zA-Z0-9]([-a-zA-Z0-9]*[a-zA-Z0-9])?)*$'

Expected Behavior
The system should allow complex hostnames such as master-0.k3s for the primaryNS.hostname field to enable more detailed DNS configurations.
Actual Behavior
The current validation rules restrict the hostname format, resulting in an error when trying to apply more complex hostnames.
Environment
All environments where the CoreDNS Manager Operator is deployed.
Additional Context
The validation rules enforced by Kubebuilder do not permit the creation of more complex hostnames. For instance, the requirement is to create an NS record pointing to master-0.k3s.example.com and subsequently create an A record for it.
Example
<lab>root@core-k3s-lb-0:~# cat << EOF | kubectl apply -f -
> ---
apiVersion: monkale.monkale.io/v1alpha1
kind: DNSZone
metadata:
  name: bubble-lab
  namespace: kube-system
spec:
  connectorName: coredns
  domain: "bubble.lab"
  primaryNS:
    hostname: master-0.k3s
    ipAddress: "<IP_ADDRESS>"
  respPersonEmail<EMAIL_ADDRESS>> EOF
The DNSZone "bubble-lab" is invalid: spec.primaryNS.hostname: Invalid value: "master-0.k3s": spec.primaryNS.hostname in body should match '^[a-zA-Z0-9]([-a-zA-Z0-9]*[a-zA-Z0-9])?(\\.[a-zA-Z0-9]([-a-zA-Z0-9]*[a-zA-Z0-9])?)*$'
<lab>root@core-k3s-lb-0:~# cat << EOF | kubectl apply -f -
> ---
apiVersion: monkale.monkale.io/v1alpha1
kind: DNSZone
metadata:
  name: bubble-lab
  namespace: kube-system
spec:
  connectorName: coredns
  domain: "bubble.lab"
  primaryNS:
    hostname: master-0
    ipAddress: "<IP_ADDRESS>"
  respPersonEmail<EMAIL_ADDRESS>> EOF
dnszone.monkale.monkale.io/bubble-lab configured

The Kubebuilder validation prevents the use of more complex hostnames. The objective is to create an NS record pointing to master-0.k3s.example.com and then create an A record for it.

Resolved in 1.0.3


fix(v2client): increases static-route list limit

Description
This commit removes the list limit for v2 static routes API.

:tada: This PR is included in version 7.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix pipeline failures

No real changes. Simple change to Drone file to mirror the workaround in rancher/rancher:
https://github.com/rancher/rancher/blob/58bfcb19605d51bf24f0aa5bf1d60a68b1ae2a27/.drone.yml#L948-L951

Fix can be seen in Drone.
In one of the last successful PRs (https://github.com/rancher/wins/pull/147), drone shows the following in the clone step:

Initialized empty Git repository in C:/drone/src/.git/
From https://github.com/rancher/wins
 * branch            main       -> FETCH_HEAD
 * [new branch]      main       -> origin/main
Branch 'main' set up to track remote branch 'main' from 'origin'.
Switched to a new branch 'main'
From https://github.com/rancher/wins
 * branch            refs/pull/147/head -> FETCH_HEAD
Updating 01290ed..c0ee559
Fast-forward
 go.mod | 2 +-
 go.sum | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

In the failing case (https://github.com/rancher/wins/pull/151), even though the PR passed you can see that drone failed to have a starting point:
Initialized empty Git repository in C:/drone/src/.git/
fatal: no path specified; see 'git help pull' for valid url syntax
error: pathspec 'main' did not match any file(s) known to git
From https://github.com/rancher/wins
 * branch            refs/pull/151/head -> FETCH_HEAD

In my PR, you can see this come back in the Drone run:
Initialized empty Git repository in C:/drone/src/.git/
From https://github.com/rancher/wins
 * branch            main       -> FETCH_HEAD
 * [new branch]      main       -> origin/main
Branch 'main' set up to track remote branch 'main' from 'origin'.
Switched to a new branch 'main'
From https://github.com/rancher/wins
 * branch            refs/pull/153/head -> FETCH_HEAD
Updating 3898f7a..02da848
Fast-forward
 .drone.yml | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

The reason why this only fails after the PR is merged is probably cause Drone falls back to the PR's git repository when evaluating against a push. But as you can see in the Drone tag pipeline:
Initialized empty Git repository in C:/drone/src/.git/
fatal: no path specified; see 'git help pull' for valid url syntax
error: pathspec 'FETCH_HEAD' did not match any file(s) known to git


Enable the new model serialization for Appconfiguration

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.


Wallet seems to silently fallback to an network when given an unknown network

While working on subnet, I happened to call callContract with a network address that I didn't previously set on my wallet.
In the transaction pop-up, instead of warning or suggesting to add the network, the wallet silently falls back to testnet. Which I think is misleading.
Expect behaviour
Either warn the user the they need to add the network or directly suggest to add it.
Note: In the future, since more dApps will interact with subnet, it would be good to have an API to:

check if a network is set in the wallet
directly add a network from a call to the extension wallet
So that dApps could have a button to "Add subnet to your wallet"

Fixed


Upgrade SLF4J version to 2.0.10

Update pom.xml to upgrade SLF4J's version to 2.0.10

It is not required to update the version of SLF4J in POM.


Loading external html

I'm maybe overcomplicating things here but I've been thinking of redoing the code that it loads in the animations from external html files.
That way, streamers can do a slight tweak to the main twitchpopups.html file to say which pages are to be loaded in for which commands, and then they can make a separate page for each animation.
Imagine you had a dozen animations, maybe even twenty or thirty. You wouldn't want all those animations coded in the main page would you?
Anyway, I tried it and I ran into one issue after another.
Before anybody goes ahead and attempts it, what are your thoughts?

For a project like this I'd put them all into a file like animations.js and load that in via a <script src="./animations.js">

Roughly speaking, what would the process be in creating new divs and so on with an external javascript file? Can anything be done, am I limited only by my imagination?

javascript loaded through script src has the same functionality as inline javascript aye, it's just put in a separate file to make things cleaner
If Slugworth wants a gobstopper, he can have one

I see, I see.

Bit of discussion in the Discord too today about ways to split things up to make it more manageable for developers and usable for streamers which includes this kind of thing too. One idea:
main.htm
main.js -- triggers ticks and on message but only really calls the handlers
settings.js -- holds constants bg colours etc
handlers folder
--| alert.js
--| hotseat.js
--| new handlers added here
--| animation.js
styles folder
--| main.css -- holds base css styles
--| animations.css
.. this would probably be a good opportunity to also do some general clean-up on formatting, comments, indentation, etc.

It's easier to generate HTML from other JavaScript files with something like Element.insertAdjacentHTML() (or have other files provide HTML as a string for the main page to insert), as iframes are a ballache in terms of communication.
Here's an example of alerts being defined in its own file, then hotseats added in a different files, where hotseats reuses the animation created by alerts:
https://codesandbox.io/s/twitchpopups-modules-0zx5w
tmi.min.js seems to make CodeSandbox brick itself every other time I open it, so here's a separate version too if that won't run: https://twitchpopups-modules.netlify.app/

Bit of discussion in the Discord too today about ways to split things up to make it more manageable for developers and usable for streamers which includes this kind of thing too. One idea:
main.htm
main.js -- triggers ticks and on message but only really calls the handlers
settings.js -- holds constants bg colours etc
handlers folder
--| alert.js
--| hotseat.js
--| new handlers added here
--| animation.js
styles folder
--| main.css -- holds base css styles
--| animations.css
.. this would probably be a good opportunity to also do some general clean-up on formatting, comments, indentation, etc.

I have the following: https://github.com/FunkotronKing/TwitchPopups/tree/feature/moduler-structure
Its early days but appears to be working, I want to get the animation code from Gantee and see how it fits with this structure.


fix(discord.js-utilities): adjust type guards for DJS v13

is2g @kyranet you better properly review this because you messed up the types hard

Tests are broken because this is suddenly no longer working with latest DJS dev.
@kyranet pls help. It was your way to checking for DM permissions.
const DMAvailablePermissions = new Permissions(
    ~new Permissions([
        //
        'ADD_REACTIONS',
        'ATTACH_FILES',
        'EMBED_LINKS',
        'READ_MESSAGE_HISTORY',
        'SEND_MESSAGES',
        'USE_EXTERNAL_EMOJIS',
        'VIEW_CHANNEL'
    ]).bitfield & Permissions.ALL
);

    const resolved = new Permissions(['SEND_MESSAGES', 'ATTACH_FILES']);
    const resolvedIncludesServerPermissions = Boolean(resolved.bitfield & DMAvailablePermissions.bitfield);

resolvedIncludesServerPermissions  is suddenly false and not true?

Superseded by #147


MCRIB atlas not downloaded to atlas folder

Hi,
Have installed the feature/MCRIB_integrated branch using the setup.sh script.
Now when I am running.
$mirtk neonatal-segmentation rawdata/sub-PK343/ses-MR1/anat/sub-PK343_ses-MR1_run-001_T2w.nii.gz 40 -atlas MCRIB -d derivatives/neonatal-segmentation_DrawEM_MCRIB/sub-PK343/ses-MR1 -t 10
ls: cannot access '/home/finn/Software/DrawEM/build/MIRTK/Packages/DrawEM/atlases/M-CRIB_2.0/T2': No such file or directory
DrawEM multi atlas  v1.3 (branch version: 820308892ccaceb821a17a8875a6abb3e19bed05)

it cannot find the MCRIB-atlas. In the appropriate location I only have the other atlases
finn@laptop:~/Software/DrawEM/build/MIRTK/Packages/DrawEM$ ls atlases/
ALBERTs  MNI  non-rigid-v2

I have previously downloaded it and have it in another location, but the folder names do no follow your convention, and comparing with the ALBERTs atlas I don't have any gm-posteriors etc etc.
How do I proceed?
Thanks!
Finn

Have downloaded and built DrawEM after MCRIB was merged into the master-branch. As before the $DRAWEMDIR/atlases does not contain any MCRIB-atlas.
However, from the configuration-script $DRAWEMDIR/parameters/MCRIB/configuration.sh I can set the relevant paths
export ATLAS_T2_DIR=$DRAWEMDIR/atlases/M-CRIB_2.0/T2
export ATLAS_TISSUES_DIR=$DRAWEMDIR/atlases/M-CRIB_2.0/tissues
export ATLAS_SEGMENTATIONS_DIR=$DRAWEMDIR/atlases/M-CRIB_2.0/segmentations

I have done this manually (or actually in first play-around just making the equivalent folder structure in $DRAWEMDIR/atlases/M-CRIB_2.0 and filling it with the relevant files from the M-CRIB_2.0 atlas.)
I am still not there and have some questions.

The M-CRIB_2.0 atlas does not contain a any tissue segmentations.
Can these be found somewhere?
If not, would it be possible to generate these? I guess again from one of the variables in the $DRAWEMDIR/parameters/MCRIB/configuration.sh (like $ALL_LABELS_TO_TISSUE_LABELS) and using mirtk padding command? However, I am not so used to using this. Advice would be appreciated!

I would be good to get confirmed which T2 you are expecting (raw or preprocessed).

Thanks in advance!
Finn

Hello Finn,
did you manage to resolve this? I am currently dealing with the same problem and would be thankful for any advice :)
Best,
Andrea

Hi Andrea,
No, not yet. Have been in contact with Antonios @amakropoulos, but as of until some months ago, the licensing thing had not yet been completely resolved.
Cheers
Finn

Hello,
It seem we need to cross our fingers for the speedy resolution then :) and to come up with some alternative solution until then.
Thanks again.
Have a lovely day,
Andrea

I am having a similar issue.
Also hoping for a speedy solution...


Correct warnings about deprecated parameter

Haproxy is emitting a warning: "The $bind_options parameter is deprecated; please use $bind instead"
This occurs even when bind_options is NOT being used.
See issue #555

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.ram900 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.

Signed Contributor License Agreement as Hawkeye-7.

Hi @hawkeye-7, the PR looks good to me, however, it needs a quick rebase before I can merge. Hopefully, CI hits green too as the RedHat 9 failures seemed to be transient.

Is there any progress on this, it is only a small change, but it will help to keep messages out of our puppetserver.logs?

I don't know why it didn't make the last release. It should be easier to get trivial changes like this through.

I don't know why it didn't make the last release. It should be easier to get trivial changes like this through.

I guess that @LukasAud is requesting a rebase so that the CI can run on this PR: while the change seems okay, a passing test suite makes us sure we are not introducing some regression due to some broken existing behavior, presence of weird chars (e.g. non-breakable spaces), etc.
See https://github.com/puppetlabs/puppetlabs-haproxy/pull/557#pullrequestreview-1523345167 for rebase instructions.

ram900@hawkeye7:~/git$ cd puppetlabs-haproxy/
/Users/ram900/git/puppetlabs-haproxy
ram900@hawkeye7:~/git/puppetlabs-haproxy$ git fetch origin
ram900@hawkeye7:~/git/puppetlabs-haproxy$ git rebase origin/main
Current branch main is up to date.
ram900@hawkeye7:~/git/puppetlabs-haproxy$ git push -f
Everything up-to-date
ram900@hawkeye7:~/git/puppetlabs-haproxy$

The PR now has 3 commits, the 2 last one being merge commits.  After a successful rebase, your are supposed to have no merge commits.
I tried to replicate what might be your setup to hopefully make the rebase instruction copy/pastable:
$ git clone https://github.com/hawkeye-7/puppetlabs-haproxy
$ cd puppetlabs-haproxy
$ git remote add puppetlabs https://github.com/puppetlabs/puppetlabs-haproxy
$ git fetch puppetlabs
$ git remote -v
origin	https://github.com/hawkeye-7/puppetlabs-haproxy (fetch)
origin	https://github.com/hawkeye-7/puppetlabs-haproxy (push)
puppetlabs	https://github.com/puppetlabs/puppetlabs-haproxy (fetch)
puppetlabs	https://github.com/puppetlabs/puppetlabs-haproxy (push)
$ git log --format=oneline puppetlabs/main..main
0232792b5e636cf8d4c2418568a4bb1b53a41e70 (HEAD -> main, origin/main, origin/HEAD) Merge branch 'main' into main
4f918cbb96ec5979deba7f764af4047e33425926 Merge branch 'main' into main
8aa521cf389bba5c6ae60d1fa479a6b0915c7399 Suppress spurious warning about bind_options

Here we are on top of origin/main, and we want the commits to be on top of puppetlabs/main.
First, we make sure that the local repo know what is the current state of the puppetlabs remote:
$ git fetch puppetlabs

Then, we can do the actual rebase of the current branch on top of the reference one:
$ git rebase puppetlabs/main
$ git log --format=oneline puppetlabs/main..main
514095b88cce20b8d571479c3c5dbb423b68549a (HEAD -> main) Suppress spurious warning about bind_options

Finally we push the local branch to your fork to update the PR:
$ git push -f

Run with the new instructions. Your rebase has been done.


Changed to allow Nitro to be integrated into the CI system

These changes are required to allow veracruz to run inside a docker container for the Nitro environment. This is a partner to the PR (https://github.com/veracruz-project/veracruz-docker-image/pull/10) in the veracruz-docker-image repo. They should be merged at the same time.

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: c0e991ddf9e530f5f93618ae9b4957b170835470
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: f0e6cce8133d0a4c1d795e3257cf45719432c243
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

This is all good from my test run. I have rebased and updated the nitro_instruction.md.

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 95865c3c1666082527680dbeafb3452aa78c8f87
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: Veracruz
Commit ID: 4583760ddf7622c2ce2a07ef127f666181603444
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

What is the status of this pull request?

This PR is superseded by #146. I will close it.


Env file should be simple key value pairs

It is a common patter to use simple key values for envitonment parameters instead of  complex objects or json.

How would that work for a list of storage accounts?  The only time I've come across lists of stuff in env vars is like PATH which has colons separating each entry but that feels weird for a list of sets of info

or is the point here to not specify such information in the env var but instead somewhere else?  the env var is convenient when deploying IQEngine through an azure app service because you just set the value in the app service settings, i.e. the string from the .env file can be copy/pasted into the app service setting.

The idea it is that configuration data could live in a config object that could be stored in a databased or file. environment settings are normally used for smaller starter things.

Ok, we'll start with a file (I assume json file?), because currently IQEngine is database-less, which isn't always going to be the case, but I want to make sure once we add the database its done with as much planning as possible, and for a specific reason (not just to hold these basic settings).


Introduce Raw Socket public API

subj
Does it sound good idea to introduce such API?
Current Socket struct is pretty much for internal use, but what about letting user access it by making it public?

I started to work on it on my own https://github.com/DoumanAsh/lazy-socket.rs
So, but if you're going to consider introducing such API, let me know, i would gladly help :)

Over time I've come to think that we may inevitably provide an API like this, yeah. I'd prefer to take such a step lightly as it can have ramifications beyond just the current TCP/UDP test case, but would be good to keep in mind!

@alexcrichton JFYI i finished, more or less, Socket struct itself.
It is just thin wrappers over socket methods and a bit of convenience methods.
It still requires some touch up of internals, but so far seems to work more or less on Windows and Linux.
Address representation is done through net::SocketAddr which should be ok i suppose.

net2 is deprecated and under sustaining maintenance now, accepting only bugfixes and patches to enable new rust platforms.


Easier documentation for Box Api

The Box API documentation is not straightforward and segmented into two parts.
https://material-ui.com/components/box/
https://material-ui.com/system/typography/#api
There is a typography component within the components, however the system/typography uses the Box API instead. This is sort of confusing.

[x] This is not a v0.x issue.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Context 🔦
I have upgraded a project to material ui v4. And there were many visual aesthetic changes. But I will focus on the Box API.
For the Box api, is it meant to replace Typography component? Since it has nearly all the same features. If not, can we have some examples that use both <Box/> and <Typography/>
? Something like so <Box fontWeight="bold"><Typography></Typography></Box>

I am trying to leverage all CSS features into the Box component but find it still a little magical.

@tingaloo You can think of the Box component as CSS utility helper. It's not meant to replace any component. The documentation is fragmented because these are two different building blocks. The system can be used outside of the Box component, you could build your own components with it.


Antivirus yells at me whenever I try to load the modrinth page.

Every time I load the modrinth page for Pl3xMap v2, I get
An infected file attempted to run on your device.
Threat name: Exploit.CVE-2016-3714.Gen.3
Path: /Users/**user**/Library/Caches/Firefox/Profiles/2b33tdxu.default-release/cache2/entries/4CD8796841042CB58C4020B7B5F22E40EDCA947C=>(URL)
We deleted the file to prevent malicious commands from being executed on your device.
from BitDefender.
This does not happen on ANY other webpage I have encountered.

Is this happening for any other pages on modrinth, or is it very specifically Pl3xMap's only?

Actually, It does look like it comes from the wakatime badge I was using. I have removed it from all locations.
Thanks for the report ^_^


ResourceRelationship.relatedResourceID - update usage notes

See meeting notes from March 31st 23

[ ] Update the notes to make it clear that this term should be a PID for the objectgroup or recordLevel class is a property of.

This seems reasonably clear already, so closing issue


added estimate_resource_usage.py

Adding estimate_resource_usage.py. This notebook implements some tools to estimate the relationship (function) that maps the size of inputs to required disk and resource usage for individual tasks within a workflow.
As a binary file, the notebook does not produce a meaningful diff, but rather one should look at the added file directly.

I believe I've addressed all comments. There are no longer any file dependencies, and I uncommented the save line.


Update chat.md

When I followed this tutorial, I got error IndexError because the first choices response don't contain any element. So I add try and except
📚 Context
Just add try except, so that others can copy and play 😄
🧠 Description of Changes

add try except on response.choices[0].delta.get("content", "")

Revised:
Insert screenshot of your updated UI/code here
Current:
Insert screenshot of existing UI/code here
💥 Impact

Size:

[x] Small
[ ] Not small

🌐 References

[ ] Notion

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

Hi @klurpicolo 👋
#884 updates the tutorial to use OpenAI's latest chat completions API. We were unable to reproduce the IndexError with the latest version, so we're closing this PR. If you're still running into the problem, please create a new issue or reply to this PR with a prompt that reproduces the issue.
🎈


[Bug] Loading state bar keeps going after everything is loaded.

I noticed this on the login page.
Even after the page finish loading, the loading bar on top keeps going forever.

Fixed by removing pace.js in https://github.com/Laravel-Backpack/CRUD/pull/5115


Debug viewlet panes out of order and imporperly rendered

Notice the breakpoints pane is untitled. Ran into this while debugging a node app using the debugger terminal @connor4312

We saw this before when the welcome view was initially added. I don't think we ever got a consistent repro, does it happen reliably for you?
cc @isidorn

No, I think @eamodio ran in to this as well in the timeline pane and the issue was the one pane gets added first and gets the viewlet stuck in the single pane mode. But it doesn't reliably repro. I've only seen it the one time and I'm not quite sure how I got there.

Oh, okay. Maybe it's not specific to the debug view then and adding the welcome view just triggered the same behavior somehow.

@JacksonKearl Are you still in the state? Can you get more info about what views are "supposedly" visible by right-clicking the header of the viewlet?


Test flags

Pull Request Template

[x] Initial implementation
[x ] Refactoring
[ ] Unit tests
[ ] Integration tests

Pull Request Test Coverage Report for Build 171

57 of 107 (53.27%)  changed or added relevant lines in 10 files are covered.
5 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.3%) to 76.099%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/smclient/clientConfig.go
2
3
66.67%

internal/cmd/info/info.go
4
6
66.67%

internal/cmd/login/login.go
10
12
83.33%

pkg/httputil/httputil.go
25
27
92.59%

internal/configuration/configuration.go
12
14
85.71%

pkg/auth/oidc/oidc.go
2
8
25.0%

internal/cmd/smctl.go
0
8
0.0%

main.go
0
13
0.0%

internal/cmd/commander.go
0
14
0.0%

Files with Coverage Reduction
New Missed Lines
%

internal/cmd/commander.go
1
56.0%

pkg/smclient/clientConfig.go
4
17.65%

Totals

Change from base Build 166:
-1.3%

Covered Lines:
987

Relevant Lines:
1297

💛  - Coveralls


Create stream method

Add a stream method to Query and Connection that returns results as an observable.
Fixes #11

Coverage increased (+0.7%) to 95.574% when pulling 9ffedae743acdcfbd8c6567e78f9c0db72bc401d on create-stream-method into eb1fb48eac6d5a7c39aca6200068b632c55dba7a on master.


Stylelint is broken and can't be installed. Please help

What minimal example or steps are needed to reproduce the bug?
npm init stylelint
npx: installed 76 in 8.406s
/home/db/.npm/_npx/583809/lib/node_modules/create-stylelint/create-stylelint.mjs:5
import process from 'node:process';
^^^^^^^
SyntaxError: Unexpected identifier
at Module._compile (internal/modules/cjs/loader.js:723:23)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
at startup (internal/bootstrap/node.js:283:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)
What minimal configuration is needed to reproduce the bug?
1- Fresh Ubuntu 20.04.5 LTS
2- sudo apt-get install nodejs
3- sudo apt-get install npm
4- npm init stylelint
How did you run Stylelint?
npm init stylelint
Which Stylelint-related dependencies are you using?
None
What did you expect to happen?
To install stylelint
What actually happened?
Error
Do you have a proposal to fix the bug?
No response

@crawlchange From the error message, .mjs and ESM seem unsupported. What is the Node.js version?


Ability to Change Kodi Controls

Using a RG353M on ArkOS v2.0 (08/23/2024)
I am trying to use my device as a portable media player; however, the current kodi controls will turn up and down my volume any time I put my RG353M in my pocket (due to the right thumbstick handling volume control). Could we have a change in controls so that it can recognize the volume buttons as we have seen across the rest of ArkOS (just utilizing the volume buttons)? That would be great. If not, a easy way to modify the kodi controls is needed.

Kodi has it's own controls that's setup separately from the OS controls.  You can try modifying /home/ark/.kodi/userdata/keymaps/remote.xml file and deleting the following entries
      <volumeplus>VolumeUp</volumeplus>
      <volumeminus>VolumeDown</volumeminus>

The volume keys will still control the volume of the OS so should work fine.


Update PNPM in CI workflows

This pull request updates PNPM in the CI workflows.

coverage: 100.0%. remained the same
when pulling b90b199423340a0050252060a8d39f551a36a5c7 on update_pnpm_on_ci
into 78c86ba0130ebbf8233165c2a0aee405f6cd7f2f on master.


Compat for Julia 1.7 and switch from Roations.jl to Quaternions.jl

Codecov Report

Merging #10 (fc1265c) into main (efab5f2) will increase coverage by 0.01%.
The diff coverage is 88.29%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   92.13%   92.14%   +0.01%
==========================================
  Files          81       81
  Lines        3816     3821       +5
==========================================
+ Hits         3516     3521       +5
  Misses        300      300

Impacted Files
Coverage Δ

src/Dojo.jl
100.00% <ø> (ø)

src/bodies/set.jl
94.91% <ø> (ø)

src/contacts/nonlinear.jl
100.00% <ø> (ø)

src/contacts/utilities.jl
0.00% <ø> (ø)

src/joints/impulses.jl
100.00% <ø> (ø)

src/joints/joint.jl
89.39% <ø> (ø)

src/joints/limits.jl
100.00% <ø> (ø)

src/joints/rotational/constructor.jl
100.00% <ø> (ø)

src/joints/rotational/impulses.jl
100.00% <ø> (ø)

src/joints/rotational/input.jl
43.58% <ø> (ø)

... and 28 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 efab5f2...fc1265c. Read the comment docs.

Is it possible to remove dependency on Rotations? What are the benefits to using Quaternions.jl over the implementation in Rotations.jl?

Completely removing the dependency on Rotations.jl is possible and requires rewriting just a few small functions (rand, or some mappings for MeshCat/CoordinateTransformations). So yes, makes sense!
The issue with the new Rotations.jl versions is that a QuatRotation needs to be unit length, and is changed to a Quaternion when it is no longer unit length. We are doing a lot of multiplications of quaternions with scalars, for example for angular velocities, which no longer gives us a unit length quaternion and would require adding a number of functions / special casing.


How to add less plugin for single vue component file

I use vue-cli to generate my project. And I use less. I need to use a less plugin. But error occurred.
Plugin.install is not a function.
I noticed that less-loader use loader-utils to get options. And in loader-utils, the query is a string like this  ?{"plugins":[{}],"sourceMap":false}. So the loader can't get the correct plugin instance (the install function is in the prototype object).
I found that if I use use option, the query is an object. I am not familiar to webpack. So how can I fix this.
similar ref
P.S. less-loader v4.0.1 not this bug

I suggest you to ask your question on the forum, the Discord server, gitter or StackOverflow.
I think this is more related to vue-loader


Not entirely pointy on v1.0.1-a.14

Slight distraction at the corners using Paper Cut theme on Zen 1.0.1-a.14

It should be fixed on paper cut v1.0.3 👍


♻ Move internal variable for errors in jsonable_encoder to put related code closer

Moved the declaration of errors list to the first except clause in fastapi/encoders.py

Cool, thanks! :coffee:


[WIP] CursorLine and CursorColumn externalized

I'll start this off by saying I have no idea how to get rid of duplicate commits and also the older ones. I tried a generic rebase to upstream/master, which failed, and then I went in and somehow duplicated the commits that I actually made... :(
Any help with this would be appreciated.
As for the actual content, I'd like your input on what could be done better and how. I didn't know exactly what direction to go, so I just kind of imitated the surrounding code.

As far as the duplicate commits go, next time try creating a new branch and creating a Pull Request from that branch.  A Pull Request includes your full branch, not specific commits, and this Pull Request is from your master branch, which has all your previous commits as well.
In the future, before committing your changes, create a new branch
git checkout -b my-new-feature

Then add your changes and commit
git add <changed files>
git commit -m "with a message"

Note that when pushing changes to a new branch, your origin repo won't know about that branch yet.  So tell git to create a corresponding branch on its server along with your push.
git push --set-upstream origin my-new-feature

Now when you login to GitHub it should tell you "hey! you pushed to a new branch! do you want to Compare & Pull Request?".  That compare should only show the changes pushed to this new branch.

Also, since you said you couldn't get the rebase to work, here are the steps I follow to rebase.  Hopefully this will work for you next time.
When there are changes to extr0py's main repo you'll need to pull them into your fork.
git fetch upstream

This will download the changes from extr0py's main repo to your disk.
Then tell git to apply those changes to your local repo.
git rebase upstream/master

This will apply extr0py's changes to your local repo on disk.
Then tell git to push those changes to your GitHub fork.
git push

After that, if you login to GitHub it should tell you that your branch is "even" with extr0py's branch.

I tried to rebase to upstream master but there were conflicts,  and when I
resolved them it said there weren't any changes...  Git is 2 hard 4 me.
On Feb 13, 2017 11:43 AM, "keforbes"<EMAIL_ADDRESS>wrote:

Also, since you said you couldn't get the rebase to work, here are the
steps I follow to rebase. Hopefully this will work for you next time.
When there are changes to extr0py's main repo you'll need to pull them
into your fork.
git fetch upstream
This will download the changes from extr0py's main repo to your disk.
Then tell git to apply those changes to your local repo.
git rebase upstream/master
This will apply extr0py's changes to your local repo on disk.
Then tell git to push those changes to your GitHub fork.
git push
After that, if you login to GitHub it should tell you that your branch is
"even" with extr0py's branch.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/extr0py/oni/pull/218#issuecomment-279447763, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AICBCfXyU7laQ4TLP4loWPi7pT0fTx8Bks5rcIgPgaJpZM4L-l9E
.

While it might say there aren't any changes, the line above probably said "your branch is ahead of origin/master by  commits".  That's where git push would push the rebase without adding any new "changes".

I've made a cleaner version over in #219


failed to create new symlink

The code runs on Windows as a normal user, but the following error occurs:
failed to rotate: failed to create new symlink: symlink \opt\log.2019042614 /opt/log.2019042614_symlink: A required privilege is not held by the client.
Here's my code:
package main
import (
	"fmt"
	"encoding/json"
	"github.com/lestrrat-go/file-rotatelogs"
	"github.com/rifflock/lfshook"
	"github.com/sirupsen/logrus"
	"os"
	"time"
)

var log = logrus.New()

func ConfigLog()  {
	log.Formatter = &logrus.JSONFormatter{}
	log.SetOutput(os.Stdout)
	log.SetLevel(logrus.InfoLevel)
	log.SetReportCaller(true)
}

func newLfsHook(logLevel string, maxRemainCnt uint) logrus.Hook {
	var logName = "/opt/log"
	writer, err := rotatelogs.New(
		logName+".%Y%m%d%H",
		rotatelogs.WithLinkName(logName),
		rotatelogs.WithRotationTime(time.Hour),
		rotatelogs.WithRotationCount(maxRemainCnt),
	)

	if err != nil {
		log.Errorf("config local file system for logger error: %v", err)
	}

	level, err := logrus.ParseLevel(logLevel)

	if err == nil {
		log.SetLevel(level)
	} else {
		log.SetLevel(logrus.InfoLevel)
	}

	lfsHook := lfshook.NewHook(lfshook.WriterMap{
		logrus.DebugLevel: writer,
		logrus.InfoLevel:  writer,
		logrus.WarnLevel:  writer,
		logrus.ErrorLevel: writer,
		logrus.FatalLevel: writer,
		logrus.PanicLevel: writer,
	}, &logrus.TextFormatter{DisableColors: true})

	return lfsHook
}

func main() {
	ConfigLog()
        log.AddHook(newLfsHook("info", 5))
	log.WithFields(logrus.Fields{
		"a": "astring",
		"b": 10,
	}).Info("Test Msg.")
}

But running as an administrator, it can pass.
Can you help me?

You don't have permissions to write to /opt/log?
I don't think it's anything this library has to do with.

The Microsoft Windows 10 has supported the file simple link.
经本人亲测，WIN10是支持符号链接的，代码：
func fileLink() {
	src := "Y:\\test.txt"
	target := "Y:\\test_link.txt"
	err := os.Link(src, target)
	if err != nil {
		fmt.Println("os.Link error: ", err)
	}
}

program run results:
运行结果：

Same issue on windows.

右键以管理员身份运行IDE


Added 'About', 'Contact' and 'FAQ' Pages

Added Added 'About', 'Contact' and 'FAQ' Pages in Footer
Created Page Routing using React Router DOM and now user can easily jump from one page to another (only from footer section)
Would like to work on entire Footer pages under GSSOC '24, if assigned ;)

https://github.com/Curious-Ecosystem/Curious-Connect/assets/132436131/e022ec07-dfea-4b42-a344-bef16159ffa0

@yashmandi you are in GSSOC Group? Please ping us

yes, i have sent a request to join your whatsapp community

Please join the call @yashmandi

Hi @thecuriousteam , i have made the necessary changes in folder structure.

Move this inside the footer folder brother

updated 👍


Implements attribute label restrictions

This implements content restrictions on attribute labels, in an effort to make attribute referencing on flows easier for a user.
While doing so, this also updates marshmallow to its latest version, and normalizes outputs (mainly of error results) to be sent as content-type: application/json.
This is connected to dojot/dojot#417

Right now, the policy (at least mine), was to limit configuration "as needed".
Since attribute labels are the ones that are actively used across dojot (in flows, and data publishing), it made sense to restrict those.
Device and template labels are currently not used anywhere but on data display (GUI), and thus need not be fettered with system-level restrictions. I for one think that in the future, we should replicate the ID vs label behavior that's implemented for devices and templates, but that would probably be best done on its own PR, since this will probably end up being merged to 0.2.x branch as well in the near future.


Optimize the getBucket method

What changes are proposed in this pull request?
Use ListStatusPartial instead of listStatus when recursive is set to true.
Why are the changes needed?
Using ListStatusPartial can avoid fetching a large amount of useless data, and can also relieve the pressure on the client and server.
Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including
No.

is this changing the default behavior? Changing the current behavior should be avoided usually.

The result returned is unchanged, just to avoid requesting a lot of useless data.

@Jackson-Wang-7  Please take a look

The result returned is unchanged, just to avoid requesting a lot of useless data.

The caller would see less data, am I right? If this is the case, that means the behavior is changed.

The original logic would first request all the data before performing the filter, including limiting the number, so that the excess data would be discarded. The current implementation is only to avoid requesting excess data, but the actual returned result is the same.

When the delimiterParam is not empty also can use listStatusPartial, i will finish it.

This loop here https://github.com/Alluxio/alluxio/blob/79f43ef82036d90077c1c502e86edfd780255b9f/core/server/master/src/main/java/alluxio/master/file/DefaultFileSystemMaster.java#L1132-L1134 kill the performance of recursive listings, since it traverses the entire subtree on every call. But in the vast majority of cases it does nothing, there are only a few specific cases where this will trigger a metadata sync.
Eventually we plan to fix the metadata sync and remove this logic, but for now, I wonder if we can add a flag to disable this loop if the user doesn't want to metadata sync? wdyt @lucyge2022

since the syncMetadata is called before doing anything in liststatus call on the target listing path, what's the ramification by turning this off? what are the scenarios that the list will list a stale view from UFS?

This loop here
https://github.com/Alluxio/alluxio/blob/79f43ef82036d90077c1c502e86edfd780255b9f/core/server/master/src/main/java/alluxio/master/file/DefaultFileSystemMaster.java#L1132-L1134
kill the performance of recursive listings, since it traverses the entire subtree on every call. But in the vast majority of cases it does nothing, there are only a few specific cases where this will trigger a metadata sync.
Eventually we plan to fix the metadata sync and remove this logic, but for now, I wonder if we can add a flag to disable this loop if the user doesn't want to metadata sync? wdyt @lucyge2022

since the syncMetadata is called before doing anything in liststatus call on the target listing path, what's the ramification by turning this off? what are the scenarios that the list will list a stale view from UFS?

A big picture question I'd like to ask is, how's the recursive ls call used? And what do you say the percentage of using ls vs delimiter and ls recursively from your application?

@lucyge2022 There are various usage scenarios, and I have not considered this issue from the usage scenarios. I refactored this method to use listStatusPartial which I think makes more sense. This avoids the unnecessary overhead of fetching large amounts of unneeded data.

@Jackson-Wang-7 Can you help review this code.

@lucyge2022 Please help take a look!

One question, how much performance gain from this change? Had you measured this part?

We recently added this flag to disable the areDescendantsLoaded check
https://github.com/Alluxio/alluxio/blob/master/core/transport/src/main/proto/grpc/file_system_master.proto#L234-L239
I wonder if we can add this as a parameter? Or maybe a property key?
If it is disabled there are some cases where a nested directory might not load metadata, but it will let the partial listing have decent performance until the metadata sync code is updated.

since the syncMetadata is called before doing anything in liststatus call on the target listing path, what's the ramification by turning this off? what are the scenarios that the list will list a stale view from UFS?

The scenarios are pretty difficult to explain, but I can try.
The basic idea would be that a metadata load happened on a nested directory, but only at 1 level. Then when the root directory is listed recursively, and the time based sync is disabled, normally it will traverse the tree to find that this folder has not loaded its metadata and perform a load metadata on the root. If this is disabled, then it will not perform this check.

@yyongycy I have tested to get 1.5w pieces of data with recursive, and the performance has improved by nearly 40%.

@tcrain I will add a property for areDescendantsLoaded in getBucket. In fact, in the scenarios we need, sync metadata is a must, of course I have to consider more general scenarios.

@lucyge2022 @yyongycy Any other suggestions? I think the problem can be solved step by step, and the problem of pulling all files every time can be solved first, which is a very serious problem.

sure agreed, will take a look again


"Upgrade to Tensorflow 2.0" documentation lists wrong TF version

System information

TensorFlow version: 1.12
Doc Link: https://github.com/tensorflow/docs/blob/master/site/en/r2/guide/upgrade.md

Describe the documentation issue
The document says that Note: tf_upgrade_v2 is installed automatically by pip install for TensorFlow 1.12 and later. but looking at the setup.py files this console command is not installed by 1.12, but will be installed in 1.13
We welcome contributions by users. Will you be able to update submit a PR (use the doc style guide) to fix the doc Issue?
No

This has been fixed.


Private repo for benchmarking backend infra

This should include

Deployment descriptors;
Grafana dashboard for automatic deployment;
Aux scripts
Scripts for persistence layer (InfluxDB) backing up.

https://github.com/o1-labs/benchmark-infra
Backups are configured for entire cloud provider VM.
Grafana dashboard will be added as part of the work on #1488.


Add example for multiple top level APIs

I created an example for people who want to have more than one top level API, not just children. As far as I can tell the only major drawback is that the does not preserve CLI functionality. Happy to make any changes needed for code style or anything else to make it better fit or more understandable.

Thanks @cody-code-wy for the contribution! I'll keep #25 open and resolve it once I add CLI support


Remove useless resources

It solves:
Error:(10, 9) Attribute application@icon value=(@mipmap/ic_launcher) from AndroidManifest.xml:10:9 is also present at me.alexrs:recyclerview-renderers:1.0.4:13:9 value=(@drawable/ic_launcher)

Merged!


"Your web browser blocked Rocket.Chat from opening a new tab.": video chat opens in default browser instead of application window

Search before asking

[X] I had searched in the issues and found no similar issues.

Operating System

[ ] macOS
[X] Windows
[X] Linux

Operating System Version
Windows 10, Fedora Linux 39
It happens on the web browser too?
No, it just happens on the Desktop app
Rocket.Chat Desktop App Version
3.9.10
Rocket.Chat Server Version
6.4.8
Describe the bug
when this setting is enabled rocket chat:

after restarting rocketchat desktop app (sometimes multiple restarts required) rocket.chat desktop opens the video call link in my default browser and not in the application window:

clearing the data directory of rocket chat completely fixes this issue for one or two starts of rocket chat.
How to Reproduce
set open video chat in application window to true
open a video chat (in my case jitsi)
restart rocket chat desktop
open a video chat
last 2 steps my be repeated until the problem occurs (i have this every one or two days)
Describe your Expected behavior
rocketchat opens video in app window
Anything else
No response
Are you willing to submit a code contribution?

[ ] Yes, I am willing to submit a Pull Request!

Hey @stylefish, thanks for sharing your experience and point out the bug.
@jeanfbrito, I would be delighted to work on this issue.
I will be updating you with the status of the progress shortly.
Prisha Gupta

Please update the server to 6.6.0.


Work CI-CD

Description

Improve update-dependencies timeschedule.
Use a better time schedule for the cron.

Motivation and Context

The current timeschedule is less likely to have support from core team members.
Linked to https://github.com/nanoframework/Home/issues/1565
How Has This Been Tested?

Screenshots
Types of changes

[ ] Improvement (non-breaking change that improves a feature, code or algorithm)
[ ] Bug fix (non-breaking change which fixes an issue with code or algorithm)
[ ] New feature (non-breaking change which adds functionality to code)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Config and build (change in the configuration and build system, has no impact on code or features)
[ ] Dependencies (update dependencies and changes associated, has no impact on code or features)
[ ] Unit Tests (add new Unit Test(s) or improved existing one(s), has no impact on code or features)
[ ] Documentation (changes or updates in the documentation, has no impact on code or features)

Checklist:

[ ] My code follows the code style of this project (only if there are changes in source code).
[ ] My changes require an update to the documentation (there are changes that require the docs website to be updated).
[ ] I have updated the documentation accordingly (the changes require an update on the docs in this repo).
[ ] I have read the CONTRIBUTING document.
[ ] I have tested everything locally and all new and existing tests passed (only if there are changes in source code).
[ ] I have added new tests to cover my changes.

Summary by CodeRabbit

Chores

Updated scheduling for dependency update workflows to run at new times on Mondays and Wednesdays.
Reduced timeout for dependency update jobs from 180 minutes to 15 minutes across multiple workflows.
Adjusted timeout for one workflow to 120 minutes, considering additional processing time.

@coderabbit full review

LGTM. Care to follow coderabbitai suggestions and fix the yaml formatting on the cron expressions? 😉

I had already done them but code rabbit doesnt pick them up until you do another full review (initiated).

@CodeRabbit full review


Variable for Require GPS Hardware

Platforms affected
Android
Motivation and Context
Currently installing this plugin, has the sideeffect of your App only beeing installable on Android devices with GPS hardware. We have clients with tablets without GPS hardware.
closes #187
Description
Added the variable GPS_REQUIRED so when installing the pluging, its optional if you want to require GPS Hardware on Android devices. Currently an app with this plugin, cannot run on devices without GPS Hardware, because of this setting in the plugin.xml
<uses-feature android:name="android.hardware.location.gps" />
Also added a description in the readme, under Android Quirks, as this only affects Android atm.
Naming of the variable is all caps, as is standard with install time variables, for editing the AndroidManifest.xml.
This PR is trying to finish the work in #171
Testing
Testing installing the plugin with and without the variable, and checked the resulting androidmanifest.xml and tested the resulting apk's.
Checklist

[ ] I've run the tests to see all new and existing tests pass
[ ] I added automated test coverage as appropriate for this change
[] Commit is prefixed with (platform) if this change only applies to one platform (e.g. (android))
[x] If this Pull Request resolves an issue, I linked to the issue in the text above (and used the correct keyword to close issues using keywords)
[x] I've updated the documentation if necessary

why isn't this merged ? this or #171 @breautek

why isn't this merged ? this or #171 @breautek

Two main reasons:

Because both PRs introduce preference in a UPPERCASE_FORMAT, which doesn't match existing format cordova uses for preferences, which in a PascalCase format. I'm hesitate to add inconsitencies.
Generally speaking, we need at least two positive votes and 0 negative votes from apache members for a PR to be accepted. So even if I agreed with this PR, I need one more member to review before I can merge.

Actually variables are defined in UPPERCASE_FORMAT, as it can be seen in different plugins as well :

cordova-plugin-facebook4
cordova-android-firebase-gradle-release
cordova-android-play-services-gradle-release
cordova-android-support-gradle-release
cordova-plugin-admob-free

example

    <plugin name="cordova-plugin-admob-free" spec="^0.21.0">
        <variable name="ADMOB_APP_ID" value="ca-app-pub-7369058349181167~8774507330" />
    </plugin>
    <plugin name="cordova-android-support-gradle-release" spec="^3.0.1">
        <variable name="ANDROID_SUPPORT_VERSION" value="28.+" />
    </plugin>
    <plugin name="cordova-android-play-services-gradle-release" spec="^2.0.0">
        <variable name="PLAY_SERVICES_VERSION" value="+" />
    </plugin>
    <plugin name="cordova-android-firebase-gradle-release" spec="^2.0.0">
        <variable name="FIREBASE_VERSION" value="+" />
    </plugin>
    <plugin name="cordova-plugin-facebook4" spec="4.1.0">
        <variable name="APP_ID" value="1302211316473586" />
        <variable name="APP_NAME" value="Mploy" />
        <variable name="FACEBOOK_HYBRID_APP_EVENTS" value="false" />
        <variable name="FACEBOOK_ANDROID_SDK_VERSION" value="4.38.1" />
    </plugin>

why isn't this merged ? this or #171 @breautek

Two main reasons:

Because both PRs introduce preference in a UPPERCASE_FORMAT, which doesn't match existing format cordova uses for preferences, which in a PascalCase format. I'm hesitate to add inconsitencies.
Generally speaking, we need at least two positive votes and 0 negative votes from apache members for a PR to be accepted. So even if I agreed with this PR, I need one more member to review before I can merge.

Actually as @jony89 also states UPPERCASE_FORMAT is the norm for this sort of setting, that effects the native build.

Those aren't apache's codebases.

@sithwarrior can you please change rename the variable to GPSRequired so we can progress with this merge?
It is a crucial one.

Those aren't apache's codebases. I'm talking about consistency within the apache's repositories. Third-party plugins may choose whatever format they like to use. If you look at any other apache cordova plugin, you will see preference names defined in PascalCase format.

@breautek Thats not the case for variables set at install time, that set native build settings, as mentioned earlier.
Here from the cordova docs
https://cordova.apache.org/docs/en/latest/plugin_ref/spec.html
API_KEY
plugman --platform android --project /path/to/project --plugin name|git-url|path --variable API_KEY=!@CFATGWE%^WGSFDGSDFW$%^#$%YTHGsdfhsfhyer56734
Here from the apache cordova camera app
<preference name="ANDROID_SUPPORT_V4_VERSION" default="27.+"/>
Here from the apache cordova media app
<preference name="KEEP_AVAUDIOSESSION_ALWAYS_ACTIVE" default="NO" />
Also it dosent work with PascalCase .
If you install a plugin with --variable switch, which we need here, it automaticly becomes UPPER_CASE when saved in config.xml, and then crashes on build, because it dosent match the PascalCase in the plugin.xml

Also it dosent work with PascalCase .
If you install a plugin with --variable switch, which we need here, it automaticly becomes UPPER_CASE when saved in config.xml, and then crashes on build, because it dosent match the PascalCase in the plugin.xml

I'll try to take a look later tonight. Is there a reason why we are using a --variable switch instead of a config.xml preference?

@erisu can we get a review on this? Right now i have to do hacky stuff to get my project built correctly, and this would fix that.

Besides the small README suggestion, LGTM!
I've done a quick test, the AndroidManifest file is changed as expected.
Unfortunately, my android devices all have GPS, so I can't really test this besides building the APK.

This was the extent to my testing. And I'm not entirely sure if the phone will let you install an APK even if you lack required features or not. This may also just be a google play store where the listing is simply hidden.
So once the README has been updated as per your suggestion I'll merge this PR.

Luckily Martin set the checkmark for "allow edit by maintainers". I've applied my small readme suggestion. Once the CI is green, feel free to merge.

All green. Thanks @timbru31 for taking the time to review :tada:

This also closes https://issues.apache.org/jira/browse/CB-14255

Great stuff @breautek  Any Idea, when this will be included in a new release, so we can use it, in our apps?

We generally do not give any ETAs on new releases. You can always install the master with cordova plugin add https://github.com/apache/cordova-plugin-geolocation.git though.

Still no eta on a proper release?
I know I can install using the master branch, but that's basically the same thing, and setting your build setting to nightly, and sidesteps semver.


Smsubham patch 1

Tried to implement - Fig 11.5 Hierarchical-Search issue #35 . Feedback is expected so that i can improve my code and better serve the community.

Please try to compile and test your code locally before submitting pull requests, i.e. your SearchTest class is not declared properly:
[javac] /home/travis/build/aimacode/aima-java/aima-core/src/main/java/aima/core/search/SearchTest.java:1: error: class, interface, or enum expected
[javac] SearchTest {
In addition this request is not related to a specific issue and will not be accepted.

OK ... Thanks for your feedback.


not working with laravel application ...I am trying to use  it on laravel v5.3

Hi @yash2code,
What kind of errors did you get?
You can use hotel add -o dev.log 'laravel command' to log output to dev.log or you can see them on http://hotel.dev.

@yash2code What command are you using to add the app to hotel?
hotel add 'php artisan serve --port=$PORT'
hotel add 'php -S <IP_ADDRESS>:$PORT -t public'
Both work for me from the application root directory with Laravel 5.5


connect to DB in onMessage function

I'm trying to connect to my own (mysql) DB whenever onMessage is fired. Like this:
public function onMessage(ConnectionInterface $from, $msg) {
$con=\mysqli_connect("1xx.xxx.xxx.xxx","root","pass","dbname");
if (mysqli_connect_errno($con))
{
echo "Failed to connect to MySQL: " . mysqli_connect_error();
}
$query = "SELECT * FROM users" or die("Error in the consult.." . mysqli_error($con));
}
Problem is, that I always get error like:
Fatal error: Call to undefined function mysqli_connect()
So instead I tried using PDO, with similiar results (I get error, that I don't have installed PDO drivers, altough I have)
Note: mysqli and pdo functions are working correctly on my server. I'm using ubuntu, php5, apache.
I think it has to do something with namespace, maybe composer, but I'm quite newbie to this stuff and don't know where to look. Do you have any suggestion, where could be problem?

If so how can I log those messages into db


Python implementation of resolver

Match the resolver spec: https://buf.build/regen/regen-ledger/docs/main:regen.data.v1#DefineResolver
Resolver should accept the IRI of the content hash, return content or 404.
For graph data, resolvers should use HTTP accept header to negotiate the RDF serialization format
RDF serialization formats/media types: https://docs.aws.amazon.com/neptune/latest/userguide/sparql-media-type-support.html

The resolver endpoint & serialization to the requested format is implemented. I don't think there is much more to do here, perhaps improve the rdflib serialization as we learn more about the library.


Add rel=nofollow links to external sites?

@zachleat Not sure if this is something we want to consider.
Not that I imagine people are submitting random sites to the leaderboard just to get a referral link bump.
Possibly:
<a href="..." rel="ugc nofollow" >

Via https://developers.google.com/search/docs/advanced/guidelines/qualify-outbound-links

rel values
NOTES

rel="ugc"
We recommend marking user-generated content (UGC) links, such as comments and forum posts, with the ugc value.If you want to recognize and reward trustworthy contributors, you might remove this attribute from links posted by members or users who have consistently made high-quality contributions over time.

rel="nofollow"
Use the nofollow value when other values don't apply, and you'd rather Google not associate your site with, or crawl the linked page from, your site. For links within your own site, use the robots.txt disallow rule.

Ah hmm, do you mean from the author pages? https://www.11ty.dev/authors/zachleat/
We do include rel="sponsored" for the opencollective facepiles:

Are there any downsides to giving referral bumps for folks listing their sites?

Yeah, I think I was thinking more about the sites themselves:
$$("div.sites-data-container > a[href^='http']").forEach(link => {
  console.log(`${link.textContent} => ${link.getAttribute("href")}`);
});

/** OUTPUT
11ty.dev => https://www.11ty.dev/
Filament Group => https://www.filamentgroup.com/
fontspeed => https://www.fontspeed.io/
NEJS CONF 2018 => https://2018.nejsconf.com/
NEJS CONF 2019 => https://2019.nejsconf.com/
NEJS CONF History => https://nejsconf.com/
Netlify => https://www.netlify.com/
Zach Leatherman => https://www.zachleat.com/
*/

(Plus possibly all the other 11 sites under DEMOS, EXAMPLES, AND COMMUNITY LINKS)
Probably not a big downside/risk to having a bunch of external links, but curious if Google penalizes you for pages with dozens of outbound links.


Leetcode Logo

Thank you for your contribution!  Before sending this Pull Request, please confirm the following:

[X] You have read the contributing guidelines
[X] The icon's size is under 1,024 Bytes
[X] The layout of the SVG looks like this including newlines

<svg xmlns="http://www.w3.org/2000/svg"
aria-label="..." role="img"
viewBox="0 0 512 512"><path
d="m0 0H512V512H0"
fill="#fff"/> ... </svg>

If you have done the above, please send the Pull Request.

@kycilius this is very nearly ready to be accepted!
Take a look at the start of your SVG.  It looks like this:
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" aria-label="LeetCode Logo">

Now take a look at the message you confirmed above.
Do you see the difference?
Your logo needs to be resized for a 512 viewbox. It also needs a white background.


Support array of default enums

Related to https://github.com/luisrudge/prisma-generator-fake-data/issues/15

released in https://github.com/luisrudge/prisma-generator-fake-data/releases/tag/v0.14.0


NSFW commands not working,

Hi, I imported the bot via your hosting, mainly for the use of NSFW as a joke for my mates. I enabled the NSFW module but it still doesn't work? Help would be amazing!

Only ~boobs and ~butts work atm due to spam. I have to limit the usage somehow in the near future and then i'll reenable them.

Oh ok


WIP: bos-tasks-wljcc: Update translations from Fedora Zanata

Update translations from Fedora Zanata

[ ] po-refresh

po-refresh in progress on bos-tasks-wljcc.
Log: http://fedorapeople.org/groups/cockpit/logs/po-refresh-435-20181126-054747/


Fixed NeptuneLogger support across different versions of neptune package

What does this PR do?
Fixes #18555

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

📚  Documentation preview 📚 : https://pytorch-lightning--19131.org.readthedocs.build/en/19131/

let's update or rebase after #19265 lands :flags:

Closing in favor of https://github.com/Lightning-AI/pytorch-lightning/pull/19265


Perception: use std sort

The code LGTM.
But please fix the build.

@KaWaiTsoiBaidu
Done. Sorry for the late reply.


OpenAPI - v1 grants schema

Ticket #114
Description

Adds an initial schema for external consumers of grant data

Testing
Automated and Unit Tests

[ ] Added Unit tests

Manual tests for Reviewer

[ ] Added steps to test feature/functionality manually

Checklist

[ ] Provided ticket and description
[ ] Provided testing information
[ ] Provided adequate test coverage for all new code
[ ] Added PR reviewers

@TylerHendrickson wdyt of adding "required" fields to the grant schema at the moment? There are many fields that are conditionally required based on whether the grant is posted/forecasted/etc. I'm wondering whether we should let this schema be flexible and only require the grant_id.

@TylerHendrickson wdyt of adding "required" fields to the grant schema at the moment? There are many fields that are conditionally required based on whether the grant is posted/forecasted/etc. I'm wondering whether we should let this schema be flexible and only require the grant_id.

We only ingest posted grants (and ignore all forecasts) – does that change anything for you? I wonder if a separate ForecastedGrant type would be warranted if/when we decide to include them.

We only ingest posted grants (and ignore all forecasts)

Is this determination done in the new grants-ingest? Want to confirm since the old scraper does not seem to separate them out.

We only ingest posted grants (and ignore all forecasts)

Is this determination done in the new grants-ingest? Want to confirm since the old scraper does not seem to separate them out.

Good question! This may indicate an oversight in early planning stages that should be revisited, but we'd made this decision early on that was informed by the "bug" in the old grant scraper that ignores updates to previously-scraped grants.
That said, this should be fairly easy to remedy if we do think this was a bad decision – the SplitGrantsGovXMLDB Lambda function only processes XML nodes after checking that they are OpportunitySynopsisDetail_1_0 – we should be able to expand that to include OpportunityForecastDetail_1_0 nodes as well.

@TylerHendrickson I updated to make specific fields as required based on OpportunitySynopsisDetail_1_0. I think this should be sufficient for the initial iteration until we learn more.
I feel it may be better to create a separate one for the forecasted grants in the future when we decide to support it.

@as1729 What do you think of adding a revision field of type ULID?

This is a great idea. I was not familiar with ULIDs, but the property of being sortable is great for the revisions use-case.
Will update shortly.

I think I relied a bit too much on the github suggestion commit here, but I pulled the code down and confirmed that the alignment is correct on all the required properties. The extra category on OpportunityMilestones also seems to be removed.


Untangle vector tile feature reprojection

When using an ol.format.MVT with featureClass set to ol.Feature, OpenLayers thinks it has to reproject features, even though they are drawn in the same projection ('EPSG:3857').
This change makes it so reprojection happens later in the renderer, where more information is available than in the feature format. In the future, this change here will also make it easier to transform features to a different projection, even if the data projection is in tile-pixels units.

LGTM @ahocevar I put a small question in the diff


DEPR: remove order kw from .factorize(), xref #6930

xref #6930

Looks good

Merged that with the script, hopefully everything went well.


Add new category "Guides" to reference topics

I've never worked with Ruby and I'm not very sure about the changes, however, it seems to work (see screenshots below). I've just grepped the codebase for occurrences of topic names ("Patching", "Debugging") and added "Guides" were it seemed reasonable. I've skipped lib/constants.rb because it is about the actual commands. Also, I didn't add new css rules, because there is no icon for the new category.
This PR is for issue #675.
I'm not sure in which column to put "Guides"
Topic dropdown:

Reference landing page (with "Debugging" moved to the end of the first column):

@rybak Thanks for looking into this, and sorry for the slow response (I've been traveling and am still clearing the backlog).
It looks like you're touching the right places. I think the third column is a good place to put it; I actually wonder if it could go at the top of the column, though. Most of those guides are way more interesting than administration commands.
This also makes the third column a bit longer than the others. Maybe we could move one of the shorter groups from the third column over to the second to balance it more (e.g., "server admin").
The table of contents itself looks nice, but I don't think the links actually work (because it is gitmodules and not git-modules, for example). I'm not sure if that will make things tricky with the man macro that drops the git- prefix from the link name.

As far as the icon goes, I think it would be OK to ship this without one. The originals were done by @jasonlong, I think. @jasonlong: any interest in doing a "Guides" icon for https://git-scm.com/docs?

but I don't think the links actually work

I just pushed up two commits to deal with this. It is a little funny, though. Arguably gitignore should have git at the front. But gitrevisions should not. So maybe we would be better off handling each case manually. I dunno.

I didn't actually create any of those icons. I used an icon library of some sort, but I'm not sure which one at this point. I can find something that matches though and get it added.

@jasonlong Thanks. I think we can move the rest of this forward without the icon for now, and add it in later. So no particular rush.

@peff Yeah, I'd say go ahead with this PR and I'll make a followup pass to add the icon.

I tried moving the guides to the third column:

@rybak Thanks. I pushed up one more commit that bumps the guides up in the two-column form, as well.
I think the list of included guides could perhaps stand to be updated, but I see you pulled the list straight from git help --guides. It seems like we should probably just follow that, and any opinions I have on the matter can be taken upstream. :wink:
I think that makes this ready to merge, then; let me know if you have any objections to the two-column update I pushed. If not, I'll merge and deploy in a day or so.