[Website] Update additional case studies

add hugo to the additional cards layout, so they could be clickable and lead to certain webpages
add link to the akvelon logo description
add optional property hasLink  to md logos  in frontmatter. If you need your logo to be a link on hover, add this property to your case-study. If you need card description to be shown on hover, skip this property

before:

after:

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Verified, works as expected. Reviewed and tested by QA

@pabloem, @aijamalnk, could you, kindly, help with review of PR?

taking a look...

I don't see the links appearing here: http://apache-beam-website-pull-requests.storage.googleapis.com/23555/case-studies/index.html
maybe it's because we've not added links yet?

correct, this is to add hasLink parameter, actual links for logos haven't been part of this PR

lgtm


Flaky test "Can perform a rolling update with subgroupsize and MaxSurge set"

Log: https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_lws/217/pull-lws-test-e2e-main-1-31/1834756867335655424
https://github.com/kubernetes-sigs/lws/pull/217#issuecomment-2350768092
/assign @Edwinhr716

Test follows the same checks and format as the other rollingUpdate + MaxSurge tests. The same line that fails https://github.com/kubernetes-sigs/lws/blob/main/test/e2e/e2e_test.go#L184, works on other MaxSurge tests https://github.com/kubernetes-sigs/lws/blob/main/test/e2e/e2e_test.go#L138. The test itself checks if MaxSurge still works if subgroupsize is set. The only thing that subGroupSize sets is labels, so there's no reason for it to fail in this case but succeed in others, which means it is a race condition.
Not sure what the solution would be here which would not make it different from other maxSurge tests.

Test isn't failing anymore, wdyt of closing the issue? @ahg-g


Add GATT busy checks

I've triggered a bug where I'm sending multiple GATT operations at once. I'm doing an MTU exchange request and a discover services request at once. Other BLE stacks I've used will return a BUSY error for the second request. Nimble doesn't have these checks. Is this by design or would a PR be accepted to add these?

Hi @tannewt ,

Is this by design or would a PR be accepted to add these?

Since esp-nimble is taken from upstream mynewt nimble, request to please raise one PR upstream , so it can be taken up in all upcoming releases too going forward.

Done! https://github.com/apache/mynewt-nimble/issues/1790


Setup Dev AWS account for operations-engineering

Environment details
Operations Engineering need a sandbox AWS environment to test AWS infrastructure operations.

ideally admin access to core AWS services.
Need to be able to test automatic Route 53 operations.

Application Name
operations-engineering
Description of application
Operations Engineering need a sandbox AWS environment to test AWS infrastructure operations.

ideally admin access to core AWS services.
Need to be able to test automatic Route 53 operations.

GitHub team slug
operations-engineering
GitHub code owner team slug
No response
Environments

[X] Development
[ ] test
[ ] Preproduction
[ ] Production

Environment access level Development
sandbox
Environment access level Test
No response
Environment access level Preproduction
No response
Environment access level Production
No response
application
operations-engineering
business-unit
operations-engineering
infrastructure-support
<EMAIL_ADDRESS>owner
<EMAIL_ADDRESS>Subnet sets

[X] General
[ ] Isolated

How do users connect to the application
Over the public internet
Additional features
Please check any additional features required

[ ] Additional VPC Endpoints
[ ] Extended DNS Zones
[ ] Other - please specify

Other information
No response
Definition of Done
Definition of Done

[X] account created
[X] customer informed

@connormaglynn - can we discuss this

New environment setup is complete.


CSS generation issues

I found this project after searching for a more up-to-date Sass implementation from Delegate.SassBuilder.
I'm using CSS isolation in Blazor, per:
https://docs.microsoft.com/en-us/aspnet/core/blazor/components/css-isolation?view=aspnetcore-5.0#css-preprocessor-support
Is there a way to ensure that the Blazor build waits on the Sass compilation to complete first?
It seems that the Blazor build continues without waiting for the CSS files to be generated first (and thus missing any changes in that build).
Additionally, CSS isn't re-generated (possibly due to the hash check) when CSS files are manually deleted.

Hi @cpboyd
I'm also using this with CSS isolation in Blazor, and it should wait for the Sass compilation.
How are you building your solution? With VS, or CLI dotnet build?
Do you perhaps have steps to replicate your issue?

I am seeing the same issue. I have to run a dotnet build first. and the a dotnet publish else the dotnet publish command wont find the compiled css files.

Thanks @Thnslj
I'm following your steps with the below commands:
dotnet new blazorwasm
dotnet add package LibSassBuilder
echo "body { background-color: blue; }" > wwwroot/test.scss
dotnet publish -o output

On both Windows & Ubuntu this works and I can see Sass compile files as part of my dotnet publish output:
jvr@JvR-XPS:~/sourcecode/cssgenerationissues$ dotnet publish -o output
Microsoft (R) Build Engine version 16.8.0+126527ff1 for .NET
Copyright (C) Microsoft Corporation. All rights reserved.

  Determining projects to restore...
  All projects are up-to-date for restore.
  Sass compile files
  Sass files compiled
  cssgenerationissues -> /home/jvr/sourcecode/cssgenerationissues/bin/Debug/net5.0/cssgenerationissues.dll
  cssgenerationissues (Blazor output) -> /home/jvr/sourcecode/cssgenerationissues/bin/Debug/net5.0/wwwroot
  Optimizing assemblies for size, which may change the behavior of the app. Be sure to test after publishing. See: https://aka.ms/dotnet-illink
  Compressing Blazor WebAssembly publish artifacts. This may take a while...
  cssgenerationissues -> /home/jvr/sourcecode/cssgenerationissues/output/
jvr@JvR-XPS:~/sourcecode/cssgenerationissues$ ls output/wwwroot/
_framework                      cssgenerationissues.styles.css.br  favicon.ico.br  index.html.br  test.css
css                             cssgenerationissues.styles.css.gz  favicon.ico.gz  index.html.gz  test.css.br
cssgenerationissues.styles.css  favicon.ico                        index.html      sample-data    test.css.gz
jvr@JvR-XPS:~/sourcecode/cssgenerationissues$ cat output/wwwroot/test.css
body{background-color:blue}
jvr@JvR-XPS:~/sourcecode/cssgenerationissues$

Which platform & versions are you using?
Maybe try send me your console output if possible?

I was running on Windows 10 with the Visual Studio "run" button.
At the moment, I'm not able to use the dotnet CLI to add packages due to 401 errors from private sources (apparently resolving in VS doesn't necessarily add them for the CLI).
I do recall seeing it actually run and update the individual file... but the merged test.styles.css or whatever wouldn't contain the updates (perhaps due to some parallel build process).

Were you editing the file while the app was running? Can you share your build output?
Also maybe try disabling any extra VS extensions you might have to see if that makes a difference?
When I edit the file between VS runs, my output looks like this:

I'm really struggling to replicate this. Last resort if you can maybe share a github repo to reproduce your situation?

After setting up the project from scratch, I wasn't able to reproduce so I'll close this for now.
Thanks for looking into it.


Add a newline between assertThrows and the following assertions:

Overview
Add a newline after assertThrows with an expression lambda and the following assertions,
so that each statement appears on its own line.
Note
com.google.errorprone.BugCheckerRefactoringTestHelper#addOutputLines does not seem to care about any whitespaces, therefore, I couldn't include a failing test.

We usually run a formatter (e.g.) on any modified lines after running refactorings, instead of trying to make sure the suggested fixes are well formatted. Is that an option for you?

com.google.errorprone.BugCheckerRefactoringTestHelper#addOutputLines does not seem to care about any whitespaces, therefore, I couldn't include a failing test.

By default it does an AST diff and ignores whitespace, but there's an example of a whitespace-sensitive test here:
https://github.com/google/error-prone/blob/2623f228cece3a24fd7ccb77be478c4e81f2883d/core/src/test/java/com/google/errorprone/bugpatterns/MultiVariableDeclarationTest.java#L63

We usually run a formatter (e.g.) on any modified lines after running refactorings, instead of trying to make sure the suggested fixes are well formatted. Is that an option for you?

I see, that makes sense! I certainly do not propose to properly indent the code, but having two statements on a single line results in quite a long line in a patch. Auto-formatting might not work if a user has a build configuration that checks that that the formatting is valid, but does not enforce a canonical formatting. In this case running a formatter that cannot operate on changed lines only might result in unrelated changes.

By default it does an AST diff and ignores whitespace, but there's an example of a whitespace-sensitive test here:

Thank you for the reference! Do you think it is worth including a whitespace-sensitive test in this case or would it be too fragile? I've attached a patch, if it's OK, I would push it to this branch.

positiveMatchingExceptionMessage test
Index: core/src/test/java/com/google/errorprone/bugpatterns/ExpectedExceptionRefactoringTest.java
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
--- core/src/test/java/com/google/errorprone/bugpatterns/ExpectedExceptionRefactoringTest.java	(date<PHONE_NUMBER>000)
+++ core/src/test/java/com/google/errorprone/bugpatterns/ExpectedExceptionRefactoringTest.java	(date<PHONE_NUMBER>000)
@@ -16,6 +16,8 @@

 package com.google.errorprone.bugpatterns;

+import static com.google.errorprone.BugCheckerRefactoringTestHelper.TestMode.TEXT_MATCH;
+
 import com.google.errorprone.BugCheckerRefactoringTestHelper;
 import java.io.IOException;
 import org.junit.Test;
@@ -74,6 +76,51 @@
         .doTest();
   }

+  @Test
+  public void positiveMatchingExceptionMessage() throws IOException {
+    testHelper
+        .addInputLines(
+            "in/ExceptionTest.java",
+            "import static org.hamcrest.MatcherAssert.assertThat;",
+            "import static org.hamcrest.Matchers.nullValue;",
+            "import java.util.Arrays;",
+            "",
+            "import org.hamcrest.CoreMatchers;",
+            "import org.junit.Rule;",
+            "import org.junit.Test;",
+            "import org.junit.rules.ExpectedException;",
+            "class ExceptionTest {",
+            "  @Rule ExpectedException thrown = ExpectedException.none();",
+            "  @Test",
+            "  public void test() throws Exception {",
+            "    thrown.expectMessage(nullValue(String.class));",
+            "    thrown.expect(NullPointerException.class);",
+            "    Arrays.asList(null);",
+            "  }",
+            "}")
+        .addOutputLines(
+            "out/ExceptionTest.java",
+            "import static org.hamcrest.MatcherAssert.assertThat;",
+            "import static org.hamcrest.Matchers.nullValue;",
+            "import static org.junit.Assert.assertThrows;",
+            "",
+            "import java.util.Arrays;",
+            "import org.hamcrest.CoreMatchers;",
+            "import org.junit.Rule;",
+            "import org.junit.Test;",
+            "import org.junit.rules.ExpectedException;",
+            "class ExceptionTest {",
+            "",
+            "  @Test",
+            "  public void test() throws Exception {",
+            "",
+            "    NullPointerException thrown = assertThrows(NullPointerException.class, () -> Arrays.asList(null));",
+            "assertThat(thrown.getMessage(), nullValue(String.class));",
+            "  }",
+            "}")
+        .doTest(TEXT_MATCH);
+  }
+
   @Test
   public void noThrowingStatements() throws IOException {
     testHelper

Do you think it is worth including a whitespace-sensitive test in this case or would it be too fragile?

Thanks, I think skipping the test is OK in this case. I'll merge this change momentarily.


Add Operator to Contour Docs Site

Please describe the problem you have
projectcontour.io is the home to Contour's official documentation. These docs should be updated to include the operator.

[x] Operator installation, including any CRDs (if needed). Should the operator be an option in the Getting Started section?
[x] Deploy an instance of Contour. Should the operator be an option in the Deployment Options section? Note: The uninstall step should include deleting ns/contour-operator when using the operator.
[ ] Contour configuration reference. Similar to Contour's config reference.
[ ] Update the supported versions matrix to include the operator.
[x] Upgrades- Add content to this or this. Note: Contour will not support upgrades for the v1alpha1 release.
[ ] Generate Contour API reference similar to Contour's API reference docs.
[x] Update the troubleshooting guide with steps on how to troubleshoot the operator.
[x] Does the release process doc need to be updated with operator-specific steps/section?
[x] Update the contributing guide with steps on how to contribute.

/cc @stevesloka @jpeach @Miciah

Please include a blog post/segment in an a new release blog post :)

https://github.com/projectcontour/contour/pull/3138 adds operator install to Contour's quickstart.

https://github.com/projectcontour/contour/pull/3139 updates Contour's contrib page.

Troubleshooting guide: https://github.com/projectcontour/contour/pull/3168

Here is the operator-related content for Contour's v1.11.0 release.

Release process doc update: https://github.com/projectcontour/contour/pull/3169

This issue moved to the v1.12.0 milestone to resolve the remaining documentation requirements.

This issue moved to the v1.12.0 milestone to resolve the remaining documentation requirements.

I moved this to 1.14, as I don't think we'll finish up the outstanding in time. Definitely need to devote some resources to it though.


Can't load NES ROM. Fades to black and then back to menu.

Only output from emulationstation is (without the periods):
==============================================.
==============================================.
Log file is:
lvl2:   EmulationStation - v2.0.1a, built Jul 21 2015 - 19:40:05
lvl2:   Creating surface...
lvl2:   Created window successfully.
lvl2:   Added known joystick USB,2-axis 8-button gamepad   (instance ID: 0, device index: 0)
lvl2:   Checking available OpenGL extensions...
lvl2:    ARB_texture_non_power_of_two: MISSING
lvl2:   Loading system config file /home/pi/.emulationstation/es_systems.cfg...
lvl2:   Parsing XML file "/home/pi/.emulationstation/gamelists/nes/gamelist.xml"...
lvl1:   File ""/home/pi/games/ROMs/NES/1942 (U) [!].nes"" does not exist! Ignoring.
lvl2:    req sound [basic.launch]
lvl2:      (missing)
lvl2:   Attempting to launch game...
lvl2:           ~/games/RetroArch/retroarch -L ~/games/RetroArch/cores/libretro-fceumm.so /home/pi/games/ROMs/NES/1942.nes
lvl2:   Creating surface...
lvl2:   Created window successfully.
lvl2:   Added known joystick USB,2-axis 8-button gamepad   (instance ID: 0, device index: 0)
lvl2:   Added known joystick USB,2-axis 8-button gamepad   (instance ID: 0, device index: 0)
I don't even know where to start with no actual error messages.
This is on my RPi2 with Raspbian (not RetroPi), compiled ES from the RPI-standalone tutorial on the site. RetroArch runs the game fine outside of ES, if I just run "~/games/RetroArch/retroarch -L ~/games/RetroArch/cores/libretro-fceumm.so /home/pi/games/ROMs/NES/1942.nes" it starts up no problems.

Doesn't look like an ES issue to me. Try with full paths without the tilde ~

No change. Appreciate the assistance, though.

post a log again please - (via pastebin or similar)

and your emulationstation config please.

            <!-- A short name, used internally. Traditionally lower-case. -->
            <name>nes</name>

            <!-- A "pretty" name, displayed in menus and such. -->
            <fullname>Nintendo Entertainment System</fullname>

            <!-- The path to start searching for ROMs in. '~' will be expanded to $HOME on Linux or %HOMEPATH% on Windows. -->
            <path>~/games/ROMs/NES</path>

            <!-- A list of extensions to search for, delimited by any of the whitespace characters (", \r\n\t").
            You MUST include the period at the start of the extension! It's also case sensitive. -->
            <extension>.nes .NES</extension>

            <!-- The shell command executed when a game is selected. A few special tags are replaced if found in a command:
            %ROM% is replaced by a bash-special-character-escaped absolute path to the ROM.
            %BASENAME% is replaced by the "base" name of the ROM.  For example, "/foo/bar.rom" would have a basename of "bar". Useful for MAME.
            %ROM_RAW% is the raw, unescaped path to the ROM. -->
            <command>/home/pi/games/RetroArch/retroarch -L /home/pi/games/RetroArch/cores/libretro-fceumm.so %ROM%</command>

            <!-- The platform to use when scraping. You can see the full list of accepted platforms in src/PlatformIds.cpp.
            It's case sensitive, but everything is lowercase. This tag is optional.
            You can use multiple platforms too, delimited with any of the whitespace characters (", \r\n\t"), eg: "genesis, megadrive" -->
            <platform>nes</platform>

            <!-- The theme to load from the current theme set.  See THEMES.md for more information.
            This tag is optional. If not set, it will default to the value of <name>. -->
            <theme>nes</theme>
    </system>

and
/home/pi/games/RetroArch/retroarch -L /home/pi/games/RetroArch/cores/libretro-fceumm.so /home/pi/games/ROMs/NES/SMB3.nes

def works from console?
If you can't get this sorted btw you could just use retropie (you can customise/trim it down as needed).

Yep, copy/pasted it to make sure. Maybe something from the manual compile? I don't have much else on the RPi, wouldn't be hard to format/stick RetroPi on to get it working and then add the other things I'm using (LAMP stack, etc.).

This is happening to me too on Windows 8.1. My config:
<system>
	    <name>nes</name>
	    <fullname>Nintendo Entertainment System</fullname>
 	   <path>C:\Games\RETRO\Nintendo NES</path>
 	   <extension>.nes .NES .fds</extension>
 	   <command>C:\Games\RETRO\Emulators\RetroArch\retroarch.exe -L C:\Games\RETRO\Emulators\RetroArch\cores\nestopia_libretro.dll "%ROM_RAW%"</command>
 	   <platform>nes</platform>
 	   <theme>nes</theme>
	</system>


Include Kotlin multiplatform test folders to default exclude configuration

The default-detekt-config.yml has several default 'excludes' configurations to exclude test sources.
excludes: '**/test/**,**/androidTest/**,**/*.Test.kt,**/*.Spec.kt,**/*.Spek.kt'
However, this does not cover test folders in Kotlin multiplatform projects, which are named **/<platform-target>Test/**, e.g., **/jsTest/** and **/commonTest/**. I reuse the majority of defaults, but using detekt in a multiplatform project requires me to override all default excludes with my own:
excludes: '**/commonTest/**,**/jvmTest/**,**/jsTest/**,**/test/**'
Therefore, it would be nice to either add the default multiplatform test folders to default-detekt-config.yml, or alternatively, to override the 'default' excludes in one place.

Maybe **/*Test/**, or would that be too broad?

'**/test/**' is already too broad... adding **/*Test/** doesn't seem like a huge change. This will help to android and it's flavours. Where the tests of a flavour are placed in **/flavourTest/**.
Any way, I think that for your use case something like #2581 would be really handy too.

I think adding **/commonTest/**,**/jvmTest/**,**/jsTest/**,**/iosTest/** makes more sense, since this is really common in Kotlin multi-platform.
If everyone agrees, I'll submit a PR.

@schalkms I tried making a PR before but couldn't find the sources to modify. :) I presumed that default file was generated?

One needs to edit 1 file and 1 unit test. The .yml file is auto-generated. I modified the 2 files quickly and submitted the PR.


docs: update readme instructions regarding nextclade full run

Description of proposed changes
This adds a description of the new workflow for full Nextclade runs.
More convenient to read here:
https://github.com/nextstrain/ncov-ingest/tree/docs/nextclade-full-run#refreshing-clades-nextclade-full-run
Related issue(s)

Related to #218 #221
Testing
Careful read

I cancelled the triggered automatic checks, because there are no code changes in this PR

I cancelled the triggered automatic checks, because there are no code changes in this PR

This can be automated by a conditional run of the ingest workflows [doc]
on:
  push:
    paths-ignore:
      - '**.md'

I suspect this can be beneficial in the other repos as well.

Thanks @ivan-aksamentov! I edited it here and there and replaced some somewhat vague instructions with more explicit instructions from the PR docs.
Happy for it to be merged if you don't dislike any of my edits.


Fix parsing of oid_section in config

Parsing the 'sn = ln, 1.2.3' form would segfault. Do not rely on the
previous behavior of OBJ_create(), which did not dereference the 'ln'
argument immediately.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This would demonstrate the problem:
a.cnf:
openssl_conf = x
[x]
oid_section = new_oids
[new_oids]
xxsn = xxln, <IP_ADDRESS>.5.6.7

$ OPENSSL_CONF=a.cnf openssl version

Would you mind also having a look at OBJ_create_objects (found in crypto/objects/obj_dat.c)?  That's the only other use of OBJ_create I could find that doesn't look safe...

I've marked this for 1.1.0 as well as master.

Sure, pushed a new commit.

ping @rhenium  can you look at @levitte 's comment?

This issue does not seem to be a problem any more as of commit d9ac82a28d1. Therefore I am closing this.


Please make ports configurable

I may have existing services already running and may not want to shut them down in order to develop.
Please make this here: https://github.com/amazeeio/pygmy/blob/master/lib/pygmy/haproxy.rb#L17 configurable

Also, as a developer I cannot rely on being able to bind privileged ports due to not having privilege escalation

FYI:
Ports 80 and 443 are the only ports which will work because of the haproxy configuration.

https://github.com/amazeeio/docker-haproxy/blob/master/haproxy.tmpl#L29
https://github.com/amazeeio/docker-haproxy/blob/master/haproxy.tmpl#L51

An example of exposing a different port can be found in the link below, but pygmy would still need to be wired up accordingly.

https://github.com/fubarhouse/docker-haproxy/commit/70317d602268a3dfae1f9df832b7162a946a0b1e

I could see if I can open a PR to switch to https (443) instead of 80 - but those are your only choices with the configuration of what is available.

That could delegate to a variable as well. Further down you can see where
you’re pulling in the Amazee env variables.
Port 443 is still a privileged port and in a development environment we may
not be able to bind these ports either for preference or because of not
having access rights to do so.
On Thu, 5 Dec 2019 at 7:48 am, Karl Hepworth<EMAIL_ADDRESS>wrote:

FYI:
Ports 80 and 443 are the only ports which will work because of the haproxy
configuration.

https://github.com/amazeeio/docker-haproxy/blob/master/haproxy.tmpl#L29
https://github.com/amazeeio/docker-haproxy/blob/master/haproxy.tmpl#L51
An example of exposing a different port can be found in the link below,
but pygmy would still need to be wired up accordingly.

fubarhouse/docker-haproxy@70317d6
https://github.com/fubarhouse/docker-haproxy/commit/70317d602268a3dfae1f9df832b7162a946a0b1e

I could see if I can open a PR to switch to https (443) instead of 80 -
but those are your only choices with the configuration of what is available.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amazeeio/pygmy/issues/46?email_source=notifications&email_token=AAJM5JVYH7WQNJ5ESOEWZJDQXAJSLA5CNFSM4JGCD472YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF6ODWQ#issuecomment-561832410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJM5JQQ6U7CCBJUEVVAQKTQXAJSLANCNFSM4JGCD47Q
.

Check out the PR - if you use that version you can run pygmy up and it will only bind to port 80.
pygmy up --port 443 will run the container on port 443 for HTTPS.
You may need to remove the containers if they've already been created prior to use though - pygmy stop -d

Subscribing as I'd like to see configurable ports too if that's somehow possible.

https://github.com/pygmystack/pygmy supports this


fixed about page and blog page layout

fixes #14

can you provide a little more info here? we don't currently have a blog page or about page, so i'm not sure what this is doing.

/about and /blog route


"weed shell" is always connected, preventing interactive use

"weed shell" always fails with the message that a shell is already connected, making it not possible to issue commands to seaweedfs over said shell.
This happens in all instances -even right after a reboot- as long as "weed server" is running. From what I can tell from the logs, "weed server" seems to automatically start & connect an client shell over ipv6.
master.go:137] Start Seaweed Master 8000GB 1.75 grpc server at <IP_ADDRESS>:19333
master_grpc_server.go:264] + client master@<IP_ADDRESS>:4414
master_grpc_server.go:264] + client filer@<IP_ADDRESS>:1051
[...]
master_grpc_server.go:264] + client shell@[::1]:35914
configuration.go:25] Configure filer for leveldb2

"ss -p" also confirms that a "weed" process is connecting from [::1]:35914 to [::1]:19333 and also from [::1]:19333 to [::1]:35914, so I assume "weed shell" is probably correct in detecting an already connected shell.

I learned that having a [master.maintenance] section in the master.toml is what apparently responsible. Regardless if sleep_minutes is on a long interval it seems to always keep a shell connected. Commenting out this section works around the issue for now.  It however certainly feels like it should be possible to have both automatic maintenance commands run on the master server and also interactively use a weed shell.
It maybe should even be possible to connect multiple weed shells (like before 1.74) and run commands like "volume.list" or "fs.du" or "collection.list" from them. If certain commands happen to interfere with each other, why not instead just block them from running at the same time?

I learned that having a [master.maintenance] section in the master.toml is what apparently responsible. Regardless if sleep_minutes is on a long interval it seems to always keep a shell connected. Commenting out this section works around the issue for now. It however certainly feels like it should be possible to have both automatic maintenance commands run on the master server and also interactively use a weed shell.
It maybe should even be possible to connect multiple weed shells (like before 1.74) to run commands like "volume.list" or "fs.du" or "collection.list" from them. If certain commands happen to interfere with each other, why not instead just block them from running at the same time?

try to use a separate weed shell to run the admin scripts for now.

temporarily disabled this restriction in 1.76.


[WIP] Update user JSON responses to match JSONAPI 1.0 spec

 Purpose
References Issue #3074. API v2 should be formatting JSON responses to adhere to the JSONAPI v1 specification http://jsonapi.org/. The JSONAPI v1 spec was released 5-29-15. This PR modifies the  user  response to match the new spec. Other responses will be formatted in subsequent PR's.
 Changes
Condenses  user  properties into an attributes field and formats links and relationships. Modifies tests to handle new response formatting.
 Users
New formatting:
![screen shot 2015-07-17 at 11 39 44 am](https://cloud.githubusercontent.com/assets/9755598/8751019/a19a8f80-2c78-11e5-8c2f-2a62150f2970.png)
Old formatting:

 Node contributors
New formatting:
![screen shot 2015-07-17 at 11 40 08 am](https://cloud.githubusercontent.com/assets/9755598/8751028/ae714636-2c78-11e5-95ae-52d6f26910d1.png)
Old formatting:

 Side Effects
API is now returning users with different formatting.
 TODO
- [ ] Make sure formatting is correct
- [ ] :warning: Figure out filtering issues caused by changes to formatting. TestNodeContributorFiltering and TestUsers are failing

Regarding filtering issues:
Filtering a user on fullname, for example, is failing due to formatting changes.  Exception is KeyError fullname api/base/filters.py in convert_key.
Node filtering on title, is working.  I think this is because title, category and description are still serializer fields themselves(write-only). fullname, given_name, etc. are absorbed into the attributes field. and aren't serializer fields themselves?

Moved to #3683


"Jump to" feature/shortcut

Is your feature request related to a problem? Please describe.

When you have a lot of tabs it can be hard to locate a tab purely from looking at them and then clicking on the right one.
Describe the solution you'd like

I would like a quick navigator popup that can be activated with a keyboard shortcut that finds tabs for me, and can activate the found tab when pressing enter.
Similar to the navigator you can activate with Ctrl-K on discord:

Describe alternatives you've considered

Additional context

This would go nicely with #1484 if properly implemented. More navigation options with less clutter should be one of the priorities in my opinion.

I think this can been closed with the merge of #1588.


cant resize image

target_shape=(160, 160),
(1,2160,2)  VS (1, 11000, 2)
160x160        VS       360x360
because the anchor is alwalys  11000

Yes, if you want to change the input size, you need to change it the corresponding feature_map_size.
# this is the input size with 260.
feature_map_sizes = [[33, 33], [17, 17], [9, 9], [5, 5], [3, 3]]
# if your input is 160
feature_map_sizes = [[20, 20], [10, 10], [5, 5], [3, 3], [2, 2]]

how to define feature_map_sizes with image size?

With different input size, the feature map size for locate is different. I am sorry we didn't edit our codes to adapt different input size. The main reason is that we train the model with input size 260.


Errors in running the simple example provided by the tutorial website

When running the simple example:  an ideal Re0.5W0.5 solution, errors always occurs as:
"
(sqsgen) Qiangs-MacBook-Pro:file qiangbai$ sqsgen run iteration re-w.first.yaml
AttributeError: 'tuple' object has no attribute 'class'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/Users/qiangbai/anaconda3/envs/sqsgen/bin/sqsgen", line 11, in
sys.exit(cli())
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/click/core.py", line 1130, in call
return self.main(*args, **kwargs)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/click/core.py", line 1055, in main
rv = self.invoke(ctx)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/click/core.py", line 1657, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/click/core.py", line 1404, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/click/core.py", line 760, in invoke
return __callback(*args, **kwargs)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/sqsgenerator/commands/common.py", line 88, in _inner
result = f(*args, **kwargs)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/sqsgenerator/commands/common.py", line 130, in _dummy
settings = process_settings(settings, params=process, ignore=ignore)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/sqsgenerator/settings/readers.py", line 296, in process_settings
settings[param] = processor(settings)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/sqsgenerator/settings/functional.py", line 43, in _wrapped
try: processed_value = f(settings)
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/sqsgenerator/settings/readers.py", line 103, in read_structure
structure = Structure(lattice, coords, species, (True, True, True))
File "/Users/qiangbai/anaconda3/envs/sqsgen/lib/python3.10/site-packages/sqsgenerator/core/structure.py", line 40, in init
super(Structure, self).init(lattice, frac_coords, symbols, pbc)
SystemError: <method-wrapper 'iter' of list object at 0x1130e1080> returned a result with an exception set
"
conda version: conda 4.12.0
python version: 3.10
OS: Mac OS 12.5.1

@woshijamess many thanks for bringing the problem up!
It seems that there's a problem with the call to the extension module! Looks like it fails to create the underlying object
You could verify this by executing:
import numpy as np
from sqsgenerator.core.core import Structure
Structure(np.eye(3), np.array([[0.0,0.0,0.0]]), ["Fe"], (True, True, True))

and see what the exception is, that would help to examine the problem further!
There is one strange thing however. You stack trace states
structure.py line 40: super(Structure, self).init(lattice, frac_coords, symbols, pbc). However, it line 40 should be super(Structure, self).__init__(lattice, frac_coords, symbols, pbc), hence the underscores (__) are missing.
Were there changes made to the code?

Thanks for your reply.
"

import numpy as np
from sqsgenerator.core.core import Structure
Structure(np.eye(3), np.array([[0.0,0.0,0.0]]), ["Fe"], (True, True, True))
AttributeError: 'tuple' object has no attribute 'class'

The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "", line 1, in
SystemError: <method-wrapper 'iter' of list object at 0x1114f10c0> returned a result with an exception set
"
Still the tuple error.
Also, recheck the code.
Yes, it is "super(Structure, self).init(lattice, frac_coords, symbols, pbc)"

Yes, it is "super(Structure, self).init(lattice, frac_coords, symbols, pbc)" It has the underscore.

Yes, it is "super(Structure, self).""init""(lattice, frac_coords, symbols, pbc)" It has the underscore but vanish when reply.

The bug should be fixed in the master branch. The also the Anaconda packages should be fixed, as a new build is available!
@woshijamess again many thanks for bringing the issue up!

Thanks very much for your hardworking even on weekends. It is fixed now.


Multi authors and no post author page

Multiple authors now display on post cards. I moved the location of the 'X min read' text to make the cards look more regular.
I resolved the issue with author pages not generating if the author didn't have any posts. We may want to edit the look of these pages still.
#113
#32

Thanks @mgilbert1! Merging. Heads up that to link the issues you need the Fixes #113, fixes #32 text in the initial PR comment (I linked them manually).


[ContainerApps] Fix polling logic for long running operation

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

ContainerApps


watchOS support

Probably shouldn't merge this until Xcode 7.0 is released.

This PR changes the “iOS Deployment Target” to 9.0, from 8.0 (IPHONEOS_DEPLOYMENT_TARGET = 9.0).
Unless I'm missing something that's iOS 9-specific in the changes, the deployment target can/should remain at 8.0.

I have proposed a correction to the deployment target issue with PR #111 .

@soffes since the Xcode 7 GM is out, it might be a good time to merge this and create a new release / upload a framework binary to the release for Carthage

Bump

+1


Cleaning up Hydrate binds

Consider the following code:
Hydrate(workspace){
  [OnEvent "ChildAdded"] = function()
    -- do whatever
  end
}

This might be part of a ViewportFrame wrapper Component, to give a realistic use-case.
This connection is forced to be around forever, because Hydrate does not provide a way to disconnect it. Unlike New, things passed to Hydrate can't always be :Destroyed to clean up their connections.

Contingent on #171

Closing this issue; scopes solved this, but Hydrate isn't long for this world anyway.


Deploy documentation using Github pages

Switch from netlify to GitHub pages and update some readme.md that was missed.
You can check it publishing to my forked repo gh-pages branch @dgreene1
https://github.com/yhy-1/react-accessible-treeview/tree/gh-pages

@dgreene1  I tried using the deploy command but it does not recognize GitHub action for  GIT_USER and GIT_PASS. I don't think we want to set our email and password. Third-party looks like a better option.


How to use this Node?

Hi,
I am trying to use this node. Maybe I dont know what I am doing. I bring in the output Aws Iot mqtt node. Then I put in what I think is the necessary information it is asking for by clicking the new and put in the information needed such as Endpoint, Thing type, "Things Shadow", Cert folder location. There is something that I am not doing right. I am getting an error "Error: Invalid "keyPath" option supplied.".
If you can help me with that I really appreciate that. I have attached what I am doing and the error I am getting to the right.
Thanks

You are missing client id input. Checkout the readme for details

Hmm a little bit more info please. Cause I did use the clientID but still didnt get it working.

It’s not an issue about the source code.


PIR does not work

It seems that in V2, the PIR is not connected to ground, is this normal ?

Closing. #24 is the same but better with more input.


Refactoring m2m

The core.py file has 3 different classes. It would be great to have 3 separate files and import all in the core.py file.

🎉 🎉 🎉


[Docs] Removal of additional  from the home page

Current State
There is an additional </div> in the home page of https://docs.meshery.io/

Desired State
Deletion of the unwanted  from the home page

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

If granted, I would love to work on this issue

Here you go @Debaditya-Som

Thanks @yash37158 :)

@Debaditya-Som Thank you for opening this.
It's taken care of in a pr that's now merged :
https://github.com/meshery/meshery/pull/9155#issuecomment-1776602350
Are we good to close this issue as completed @Debaditya-Som ?

@Debaditya-Som Thank you for opening this.
It's taken care of in a pr that's now merged :
https://github.com/meshery/meshery/pull/9155#issuecomment-1776602350
Are we good to close this issue as completed @Debaditya-Som ?

Okay cool


iOS 9 beta 3 issues

Hi!
It is a issue with the beta 3 of iOS9.
In Mapper.swift on line 327. I think T should be renamed to Element?
I having some issues with running the tests to validate if it fixes the build issues. Got it working
Thanks.

Fixed in https://github.com/Hearst-DD/ObjectMapper/pull/151


MudThemeProvider render 2 times lists of component

Bug type
Component
Component name
MudThemeProvider
What happened?
when I extend the mudselect list it appears twice, once below mudselect and once based on where the MudThemeProvider tag is located. I tried to change the location of the tag but the bug follow it as you can see from the pictures. I found the same problem with the datepicker.

Expected behavior
the list of components should be displayed only once.
Reproduction link
https://try.mudblazor.com/snippet/wYmcYGvunMQgxXZC
Reproduction steps

add MudSelect
add MudTheme (not immediately below the component, let some space)

Relevant log output
no error output

Version (bug)
6.0.6
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Microsoft Edge
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Thank you! and sorry for the false report.


Floating point literals are modified in generated native shader code

Godot version
v4.1.rc2.official [46424488e]
System information
Godot v4.1.rc2 - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2060 (NVIDIA; <IP_ADDRESS>02) - Intel(R) Core(TM) i5-10400F CPU @ 2.90GHz (12 Threads)
Issue description
The native shader generated from (for example) a canvas_item shader has some modified floating point litterals. For example, in the following function, the "43758.5453" becomes "43758.5".
float rand(vec2 x) {
return fract(cos(mod(dot(x, vec2(13.9898, 8.141)), 3.14)) * 43758.5453);
}
It is probably not a problem in most cases, but Material Maker generates both Godot shaders to preview results and compute shaders to render them to textures (and the end results are not the same). I can workaround the problem (by making sure float literals are not "too precise") for shaders that are provides with Material Maker, but it will remain for user-defined shaders.
Steps to reproduce
Open the attached project
Open shader_example.tres material
Open the shader in the shader editor
Inspect native code for the material and select the fragment tab

Minimal reproduction project
floating_point_literal_bug.zip

Possible duplicate of https://github.com/godotengine/godot/issues/77325, though this one is better framed so we might want to close 77325 and transfer its current conclusions here.


OTLP_EXPORTER_OTLP_HEADERS fail to parse with key=value format

Relevant code: https://github.com/Pondidum/Trace/blob/81804b6269802539fed209a7eb9f97bcca9ccfba/tracing/otlp_exporter.go#L44-L56
The OTLP Exporter Configuration Reference shows that these headers should be in key=value format.

Fixed in 71a83b8872f8016fced693e648993bd51e19cb87!  Thank you for the report and docs links!


Not just another custom column widths PR

See https://github.com/box/spout/pull/715

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

72 of 93 (77.42%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 96.61%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Spout/Writer/Common/Manager/WorkbookManagerAbstract.php
2
5
40.0%

src/Spout/Writer/ODS/Manager/WorksheetManager.php
1
5
20.0%

src/Spout/Writer/ODS/Manager/Style/StyleManager.php
14
28
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
2451

Relevant Lines:
2537

💛  - Coveralls

Thank you @aphofstede

No problem. Hope this project gets the maintenance it deserves.

Are there docs for this feature?
Also, is it really so that column width can be set only before any data is written to the spreadsheet? Seems so - and that's kinda bummer, as it doesn't allow setting column widths to adjust them to longest value written to each column. Having to know these values before write results in very unoptimal code when eg. writing data from iterator.
Is it an XLSX limitation?

See description of the original PR. Since the maintainers had declined multiple PRs already we wanted to run this implementation by them first before documenting and building further on top of this.
If this fork gets much traction we should add docs and revisit an autosizing option.

Hi @BelegUS it is my plan to postpone every actual fwrite as late as possible.
Problem is that Microsoft Office don't like having <cols> section after <sheetData> one, even though XML standard allow random order of elements.
So we can't actually write column widths after data processing.
The only solution would be to process data, get the file pointer back to the top and overwrite a dummy portion of the file just like it was proposed here https://github.com/box/spout/pull/866/files#diff-f1797564831f00a171c0864b91771b3ba9a4b1b2c7980c7a91dff07d79f55bc3R130-R134
But that's a nightmare for maintenance reasons, I'm not incline to implement that at the moment.

Hi, just like ODS already did it, with https://github.com/openspout/openspout/pull/45 now also XLSX writes the rows in a temporary file and only after that it writes all the remaining data, thanks to the speed of stream_copy_to_stream.
So now column with, alongside auto-sizing, could be easily supported and already achievable by user code :rocket:


Update mapsturcture decode and remove unused config

Description
Update mapsturcture decode and remove unused config
Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

[ ] Code compiles correctly.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #396 (84b8349) into main (8cd293b) will decrease coverage by 0.13%.
The diff coverage is 100.00%.

:exclamation: Current head 84b8349 differs from pull request most recent head 0fc217d. Consider uploading reports for the commit 0fc217d to get more accurate results

@@            Coverage Diff             @@
##             main     #396      +/-   ##
==========================================
- Coverage   41.82%   41.68%   -0.14%
==========================================
  Files          73       72       -1
  Lines        5640     5637       -3
==========================================
- Hits         2359     2350       -9
- Misses       2943     2949       +6
  Partials      338      338

Flag
Coverage Δ

unittests
41.68% <100.00%> (-0.14%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

client/config/peerhost.go
35.48% <100.00%> (ø)

pkg/util/net/iputils/ip_utils.go
61.22% <0.00%> (-4.09%)
:arrow_down:

client/daemon/peer/peertask_base.go
32.76% <0.00%> (-1.70%)
:arrow_down:

client/daemon/peerhost_stub.go


Connect to peer device without using Controller+PASE

Problem
Currently the only streamlined method for connecting to a device is to use the DeviceCommissioner::PairDevice(). This is not feasible for certain peer-to-peer connections such as a device sending QueryImage to an OTA Provider server device (such a device should not be dependent on Commissioning code).
Proposed Solution
Create some API for initiating a SecureSession to a given Node ID for a given Fabric. This would start a CASE session, and should also probably tie into the existing Device class.

Just realized that #7976 was filed earlier and describes the same issue. Closing this as a duplicate


remove 4.1 warnings

Need curl installed in the CircleCI test environment, for 'swift test' to run, judging from the logs.


[feature request]please reserve UnitedDeployment resource for version compatibility.

What would you like to be added:
We should reserve UnitedDeployment resource in yurt-app-manager for one or two versions time, not rename it to YurtAppSet.
Why is this needed:
If end user is using UnitedDeployment, but UnitedDeployment disappeared suddenly, end users maybe fell confused and can not keep the compatibility for original workloads.
others
/kind feature

/assign @hxcGit


Exporter: Add table cached ratio

We need to calculate the and export cached ratio percent for each table
| disk_mapped                 | 85964091 |
| disk_mapped_cached          | 32768    |

Finished in https://github.com/manticoresoftware/manticoresearch-prometheus/commit/0ea46fa162caf99db9524112abbe732b264953dc


feat: add layer override

Add JLCPCB Layer Override JlcLayerOverride and JLCLayerOverride. These can take top and bottom values.
This will enable the use of footprints that are placed on one side, but ha actual mounting on the other side (such as hot swaps for MX switches).

Closes #64

Would it work in conjunction with the JLCPCB Rotation Offset?

I can do a proper test tonight, but it should :) I'll let you know.

Any updates?


New package: XMLParser v0.1.4

Registering package: XMLParser
Repository: https://github.com/baxmittens/XMLParser.jl
Created by: @baxmittens
Version: v0.1.4
Commit: e2a2ed0b230e90d67059af475b504fe5f408f8fa
Reviewed by: @baxmittens
Reference: https://github.com/baxmittens/XMLParser.jl/issues/1
Description: A lightweight XML parser in the julia language

Have you checked out https://github.com/joshday/XML.jl
[noblock]

@Moelf

Have you checked out https://github.com/joshday/XML.jl

Actually, I did not know that. It seems to pretty new. I will have a look. Should we continue here anyway?
[noblock]

Your call. I just think the effort can be combined here.
[noblock]

@Moelf
Then I think we just continue here. But I agree with you and I will consider your suggestion. Thank you


Remove communication for demo protocols

Purpose
We agreed offline to remove communication layer for simple protocols that were initially included in emmy for demo purposes, as they complicate necessary refactorings that are common at this stage.
Scope
This PR cleans up definitions in the protobuf package:

Protocol service (dedicated to running interactive protocols for demo purposes) is removed.
All message types that are not used in pseudonym system scheme or in QR proofs are removed.
A new service called QRProofs with RPCs QR and QNR is added.
Enumerations are removed, because there will be no need to differentiate protocols with respect to SchemaType and SchemaVariant (they were only used in the protocols that were removed in this PR).

In addition, it removes all the communication code of the protocols included for demonstration purposes (schnorr, schnorr_ec, pedersen, pedersen_ec and cspaillier). The only thing we wanted to keep communication-wise were qr proofs, which are now a part of a separate RPC service, so some changes were required in QR/QNR client and server code.
All the client subcommands of the emmy CLI corresponding to demo protocols were removed. A deprecation notice is added to the instructions of the emmy client in the README.

@miha-stopar As discussed yesterday, I also removed QR and QNR. I got myself into a bit of a merge hell, so I had to reset history. Everything is in a single commit now - all that has changed from yesterday is the removal of QR and QNR client-server code + removal of appropriate potobuffer service and message types.


HST Imager GUI version on Mac will not proceed past Terms and Conditions

Running the Mac version (on Sonoma) for the latest release will not allow me to proceed past the Ts and Cs. I get to the screen asking me to agree or disagree but I cannot click on Agree to continue.
The button presses themselves seem to work as pressing on Disagree exits the application.

Running the Mac version (on Sonoma) for the latest release will not allow me to proceed past the Ts and Cs. I get to the screen asking me to agree or disagree but I cannot click on Agree to continue.
The button presses themselves seem to work as pressing on Disagree exits the application.

hello, the some issue with windows version, i try portable & setup versions both the some issue,  the button "AGREE" not work.

Hi, I released a new Hst Imager v1.2.396 with fixes for license not disappearing when clicking agree button.


Add liveliness and readiness endpoints for the install CNI process

Adds /healthz and /readyz endpoints mentioned in this comment.

/healthz indicates that the installer binary is running
/readyz indicates that the installer binary has installed the configuration, and that CNI is ready to start processing pods. We can use this in CNI race condition mitigations.

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

@bcheung: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
a274dd3c383592b4c738d59a5bc04ec525f04527
link
/test release-notes_istio

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


Hypha will only create invoices in IntAcct, what info is needed from IntAcct?

After the walk-through of how OTF uses IntAcct in practice it has become clear that that a lot of the vendor and project set up must be done in IntAcct manually by OTF finance people.
Bank information does not need to be in either Hypha or IntAcct since payments are done in a completely separate system.
My suggestion is that we provide a nice view for finance that display all the needed vendor and project information. Making it easy to copy and paste information from Hypha into IntAcct.
Finance then add the following data to Hypha:

Vendor ID
Project ID
Contract amendment number

Hypha can use this information for two purposes:

Create invoices in IntAcct via API calls.
Create the contract number since it is --< amendment-number>

Tasks:

We need to decide in what step Finance will add the information.
We need to make banking information fields optional so OTF can turn them off.
?

A small summary of the yesterday's call with Finance.
Finance will put the Contract Project ID in Hypha(This is the final project for which they generate invoices). I am hoping this ID can also be used to get deliverables of the project. I will check with the APIs doc.
While creating an invoice in Hypha..we will generate a unique Invoice Identifier. (Finance will let us know about the format of this identifier)
We will use contract project id provided above to generate an invoice in SageIntAcct, also updating unique invoice identifier there in a field Finance will let us know about.
Now the SageIntAcct invoice generated above will go through different processes for it to be paid. In this process, entity of the originally generated invoice also gets changed. According to finance the originally generated invoice won't be related in any way to final entity which is getting paid.
We need something constant throughout this process..to be able to update the invoice status back to Hypha. Finance will use the Hypha's unique invoice identifier throughout the invoicing process.
We will have a cron job which checks all the final paid invoicing entity in SageIntAcct(We also need to know what is that entity called in SageIntAcct). Using the unique invoice identifier we will find out the related Hypha invoice and update the status.


Better icon

Do you need any help at all with the UI, more specifically a good icon? You'd keep all rights.
Some of my stuff:
https://play.google.com/store/apps/developer?id=Daniel+Watson

About UI, getting #43 in a mergeable state would be a great start.
About the icon, I'm not very sure. I do like the idea of having a differentiation from the official client but I don't know what you have in mind. Open a PR with what you're thinking and let's take it from there?

Here's something I threw together. If you like it, it would be fun to help with other parts of the UI.
Feedback welcome :)

Here's something I threw together. If you like it, it would be fun to help with other parts of the UI.
Feedback welcome :)

I like it! I'll have my testers share their opinion on the icon and get back to you with what they say.

@MSF-Jarvis Awesome. PR incoming

After the honest and true feedback the PR was closed, although I think a lot of work has been put into this rewrite and it deserves its own icon, color scheme, etc.
https://github.com/MSF-Jarvis/wireguard-kt/pull/48#issuecomment-426053001


cmd: Add 'version' option

This 'version' option prints the fakemachine version. Fix #200

CI fails because of https://github.com/go-debos/fakemachine/pull/218


[chore] added test for default parentSelector.

Acceptance Checklist:

[x] All commits have been squashed to one.
[x] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[x] If the commit message has [changed] or [removed], there is an upgrade path above.

Coverage remained the same at 86.721% when pulling 242f8039240ff1e6f7b4155dd8739ec11d29df88 on diasbruno:chore/added-test-for-default-parent-selector into f47e79db83ef6e855195447640005e1b796aaec4 on reactjs:master.


Adds 1k.lom.me

I am seeing 929 -> https://gtmetrix.com/reports/1k.lom.me/34WUWhiP/

That's the redirect to https. (I'm not serving it on http, instead redirecting to https.)
Testing https://1k.lom.me gives 628 → https://gtmetrix.com/reports/1k.lom.me/kgrnBGZ9/
(Somehow an extra byte has snuck into the headers since the last check. Updating.)


fix: Link to kerberos constrained delegation

Seems like a copy/paste of a partial link thing that I do sometimes too.
/cc @KathrynEE

@nschonni Thank you for your contribution, I'm merging this and it should be live shortly.


Python bouncing rocket

First of all thank you so much for creating this project.
python main.py doesn't work for me so I'm using python3 main.py. It successfully creates a game but then the champion never leaves the base. A python rocket bounces forever. Problem with openCV?
Is there anyway to resolve this? I'm running macOS Monterey with Python 3.8.9.
Any help is highly appreciated.

managed to fix the problem, but after starting a game, the champion never leaves the base still.

2022-08-27 23:25:20,098 - LPBv2.Caller - ERROR - Cannot connect to host www.leaguepybot.dorsk.dev:443 ssl:True [SSLCertVerificationError: (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1125)')] (caller.py:37)

Thank you so much!! This is the first time I've seen bot working on mac version; I'll read and learn from your code.


Formsy.Form onChange is fired twice

Hi, facing an issue with Formsy.Form fires onChange twice even though my Formsy.Mixin.setValue is invoked only once.
Might not be a big perf impact, but doesn't seem necessary either, or is it?

I think that is the onChange event from the input. Just add stopPropaganation:
  changeValue: function (event) {
    event.stopPropagation();
    this.setValue(event.currentTarget.value);
  },

Nope, that didn't help. And i don't think it could, it reacts solely to setValue, no?
Well, at least, this.isChanged() second argument on second change is false, so i can check on that and prevent unnecessary store updates.

I a little changed login example:
In Input.js:
  changeValue(event) {
    console.log('Change!');
    this.setValue(event.currentTarget[this.props.type === 'checkbox' ? 'checked' : 'value']);
  },

In app.js:
  onChange() {
    console.log('Changed!');
    console.log(arguments);
  },
  render() {
    return (
      <Form onChange={this.onChange} ...

I see in console:

When I changed changeValue to:
  changeValue(event) {
    event.stopPropagation();
    console.log('Change!');
    this.setValue(event.currentTarget[this.props.type === 'checkbox' ? 'checked' : 'value']);
  },

I see:

If this does not help, can you show your code?

My Formsy.Form.onChange does not get fired with SyntheticEvent, it is (object, bool) both times.
input.js
onChange: function(e) {
  e.stopPropagation(); // having this here does not affect anything for me
  this.props.onChange(e.currentTarget.value); // fired once
}

input_formsy.js
onChange: function(value) {
  this.setValue(value); // fired once
}

some_form.js
onChange: function(data, isChanged) {
  actions.updateSomeForm(data); // fired twice
}

actually, i do not see this behaviour anymore... probably was some temp binding issue.. or whatever.
Anyway, thanks for the time spent on investigation.

I have the same issue as the OP.
Using e.stopPropagation fixed the issue.

Im running into the same issue.  I think its because of the render function's <form {...this.props} onSubmit={this.submit}>.  The props are not having Formsy-only props filtered out (such as onChange).

I have a PR for a fix #258.


Issue 205: Add generators to create random insert/retract/fire patter…

…ns for testing

I have responded to the comments here.  There is still some investigation to do on whether using :scope for dependencies has the impact desired on the build.

@mrrodriguez , I think this should be ready for merge now with my most recent change.  Could you look over this and see if you have any further concerns?

+1 looks good

Merged. Thanks!


Add to docs how to get shareable link

I didn't know that I can share my work through a link on excalidraw.
My suggestion is to add one of the following:
"To get a shareable link for your work on excalidraw, export your work and press the link icon 🔗"
Or

add this GIF

if you would like me to upload the GIF please tell me which export settings work for GitHub so that it's visible

Fixed by a3de975ecebca910d7afc92dfa196d998788cc75. Thanks for the suggestion @m-abdelwahab 🎉


Exposed public interface for writable loader

Fixes #49.
As we mentioned a while ago, I moved this decision out to a separate PR. Good news is that this will hopefully be the final one to reach a baseline of functionality and close #49 :)
Tried to match the interface for make_loader; Right now the WritableLoader object is pretty bare bones, but can be extended to do other things in the future, like the regular Loader object.
Also rolled in a couple of very small fixes; let me know if you prefer these in a separate PR.
Let me know what you think!

Codecov Report

Merging #90 into master will decrease coverage by 12.66%.
The diff coverage is 40.74%.

@@             Coverage Diff             @@
##           master      #90       +/-   ##
===========================================
- Coverage   92.44%   79.77%   -12.67%
===========================================
  Files           6        8        +2
  Lines         635      806      +171
===========================================
+ Hits          587      643       +56
- Misses         48      163      +115

Impacted Files
Coverage Δ

apacheconfig/__init__.py
100% <100%> (ø)
:arrow_up:

apacheconfig/wloader.py
31.13% <36%> (ø)

apacheconfig/flavors.py
100% <0%> (ø)

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 22fff01...6b53b08. Read the comment docs.

do we need two loaders - writable and non-writable?

I think it would be good to keep them separate for the time being, then slowly migrate all the functionality of the original into the new one.

The one pypy Travis failure seems like a network flake:
https://travis-ci.org/etingof/apacheconfig/jobs/623029025
So I think this is good to go for a last review!

It seems the new code is slightly lacking unit test coverage.

It seems the new code is slightly lacking unit test coverage.

Hmm, I added some tests since codecov was run. How can I re-run the codecov tool?

Hmm, I added some tests since codecov was run. How can I re-run the codecov tool?

I do not see how... It feels like it does analysis immediately, at viewing time. Not sure...

Thank you!


[WIP] Convert CI to Docker

Convert Travis CI to run all tests in the mtmiller/octave Docker image instead of the native Travis Ubuntu environment.
Addresses #204

This exposed an interesting bug, fixed in 0cbb7d7. Should apply the same fix to octsympy.

So that was easy :grin:. This is a good POC and starting point for what a more thorough test matrix can look like once mtmiller/octave provides more Octave versions.

@catch22 @cbm755 Merge now? Or wait until I provide at least one other (older or newer) Octave version to test against?

I saw this in the Travis log:

make: Entering directory '/doctest'
/bin/bash: git: command not found

I tried this locally:

docker run --volume=${PWD}:/doctest mtmiller/octave ls /doctest
cannot open directory '/doctest': Permission denied`

(as root or nonroot)

There is a loss of test coverage with merging now but I don't mind: its was sort of undefined which versions were run before with "ppa" anyway.

I saw this in the Travis log:

make: Entering directory '/doctest'
/bin/bash: git: command not found

Yeah, that's already https://gitlab.com/mtmiller/docker-octave/issues/8

I tried this locally:

docker run --volume=${PWD}:/doctest mtmiller/octave ls /doctest
cannot open directory '/doctest': Permission denied`

(as root or nonroot)

Hmm, cannot reproduce here. Maybe a Fedora difference? SELinux? Feel free to report with your configuration, I can try to replicate.

There is a loss of test coverage with merging now but I don't mind: its was sort of undefined which versions were run before with "ppa" anyway.

Yeah, that's why I'm ok with leaving this open until we get at least two Octave versions covered, whatever those versions happen to be.

Ok, let's leave this open for now.  Once you add one more Octave version, I'm +1 to merge.

sorry I broke this: feel free to drop my test-bist target from travis if you need this working again...

This sounds like a good plan of action to me, too.

This is ready from my side, builds are happy. Please review and feel free to merge if you agree.


⚠ Bofrost.be has degraded performance

In 7968950, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 978 ms

Resolved: Bofrost.be performance has improved in ba63ce0.


Git on-premises how to docs

There are three VC specific documents added:

Work with Git On-Premises Version Control Server
Solve Known Git Issues
Work with On-Premises Version Control Server

The latter one act as a root document for both Git and SVN On-Premises how-to's

 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.Maksym Vyhovskyi 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.


Archaic brick recipe bug(?)

When you try to craft archaic bricks with more than 1 archaic brick on exchange table the result is not added to the total number of bricks on exchange table.
To recreate it follow this steps:

Try to craft archaic bricks. At the end of process there will be 5 bricks on the table.
Do not take the result and start crafting process again.
At the end there will be the same amount of bricks. Another ingridients was consumed as usual.

Fixed in version 1.2.0


“Learn More” alignment not fit to screen on CELO Rewards Program page from Earn Rewards pills.

Frequency: 100%
Repro on build version: Android Internal Build V1.9.2(1004294334)
Does not repro version Test Flight Build V1.9.2(44)
Repro on devices: Samsung Galaxy A5(7.0),Samsung Galaxy J5 Prime (8.0)
Does not repro devices Samsung Galaxy A21s(10.0), iPhone 8(14.2)
Testing Account:<PHONE_NUMBER> / Backup Key
Repro Steps:

Launch the application and login with a test account.
Tap on Hamburger Menu.
Tap on “Earn Rewards” pills.
Now the user should be on the “CELO Rewards Program” screen.
Observed that,
Current Behavior:  “Learn More” button alignment does not fit to screen.
Expected Behavior:  Alignment should be proper.
Impact: Bad user experience.
Attachment: LearnMoreButtonNotFitToScreen.mp4

Hi  I have verified this issue using latest Android Internal Build V1.11.1(1004294339) & observed that issue is fixed now.
Verified on Devices: Samsung Galaxy A5(7.0), Samsung Galaxy J5 Prime (8.0)
Attachment: LearnMoreAllignment.mp4


fix typo

pulled from https://github.com/influxdata/docs.influxdata.com/pull/625
When I was in academia, people were pretty serious about data being plural, but a google search suggests that either is acceptable. I'll leave the grammar decisions up to you.

@jademcgough Thanks!


DeploySPLists is creating lists but not columns

Describe the bug
When I run the DeploySPLists flow it runs successfully and creates the lists, however when opening the lists in SharePoint all of the CI_***** lists they do not contain any columns other than "title".
Expected behavior
I was expecting in the CI_configAdminSetup and other lists to populate columns for which the Admin app can submit data.
Screenshots
If applicable, add screenshots to help explain your problem.

Interface(please complete the following information):

Browser  - Chrome
Build - 80.0.3987.122 (Official Build) (32-bit)

Additional context
n/a

you might look at modify the view - I thought this first as well - you can add the fields to the view

As above, open the list and select "Edit Current view" from the "All item" menu.
I have added the following columns to my view.

Thank you, simple!


Incorrect stacking of events that are close in time

Hi,
I have the following trace that gets rendered incorrectly in speedscope
[
{ "name": "A", "ph": "X", "ts":<PHONE_NUMBER>551429.079, "dur": 34670882.955, "pid": 1033891, "tid":<PHONE_NUMBER>50624, "args": {}},
{ "name": "B", "ph": "X", "ts":<PHONE_NUMBER>997066.682, "dur":   476375.838, "pid": 1033891, "tid":<PHONE_NUMBER>50624, "args": {}},
{ "name": "C", "ph": "X", "ts":<PHONE_NUMBER>473442.569, "dur":   300492.826, "pid": 1033891, "tid":<PHONE_NUMBER>50624, "args": {}}
]

The event B ends at<PHONE_NUMBER>473442.520, and event C starts at<PHONE_NUMBER>473442.569 (49ns later), so C should be on the same stack, but just after B.
However, this is how it gets visualized in speedscope:

And in perfetto.dev:

My hunch is that this could be due to a double precision issue.

is this why mine looks like this?

@occivink Yeah, I think your suspicions are correct:

I'm not super stoked at the idea of used arbitrary precision floating point for all of the parsing for trace file import, since I'm assuming that would be a non-trivial perf hit for massive files. I wonder how Perfetto deals with this
@jespersh I think what you're looking at is probably unrelated -- the display in that profile doesn't look like a bug to me

@jlfwong I'd think that the brown AdMain.Program.Main() shouldn't be chopped into pieces as the program never leaves the Main() method

I'm not super stoked at the idea of used arbitrary precision floating point for all of the parsing for trace file import

I don't know much about the internals of speedscope, but what about changing the stacking logic when event A ends at the same time as event B starts? Right now speedscope considers that B is a child of start, would it be an issue to consider it a sibling instead?


Something is very wrong with search

When trying to search for bash this morning (as in programs.bash), I got hardly any results. I'm not getting many results for any search terms.

May be the API has changed. I'll have a look ASAP


Create unit tests for the linter

Add unit tests for the linter and all linter checks.

PR #93 is taking care of this. While we don't have 100% coverage yet, we do have at least one test for each check.


Link to dev build doesn't work in readme

https://www.openfl.org/builds/lime

Fixed in 10b8f962cdf6f84d7ec764194b6538f98ee72281.


fix: process resource api does not response sub_domain field

This PR is for:

Server

Affected branches

main
v6.5

 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.zhengya 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.


2024-10-25_01-35-01_master-next_python3-botocore

Automatically created.

Successfully created backport PR for kirkstone-next:

#10196

Successfully created backport PR for scarthgap-next:

#10197

Successfully created backport PR for styhead-next:

#10198


Don't bail on a (static adapter) build after the first 404

Describe the problem
It's extremely tedious (especially when converting a Sapper app with lots of old, broken links) to

Run a build (with the static adapter)
See a single 404 error, which ends the build
Fix that single error
Repeat the above for all 37 404s.

Describe the proposed solution
It would be far better if the first encounter with a 404 didn't immediately bail on the rest of the adapter phase of the build. Sapper used to give a full list of all files, including links that 404'ed. It would be nice if SvelteKit either did the same, or at least produced a full list of 404s instead of showing you one error for each build attempt.
Alternatives considered
I mean, yeah. I can just run the build one time for every single error as described above, but that takes an order of magnitude more time/compute and attention. It's nuts.
The other alternative is to stay on Sapper.
Importance
would make my life easier
Additional Information
...failure number 23...

...fix the code for 5 seconds...
...run npm run build...
...81 seconds later...
...failure number 24...

...fix the code for 5 seconds...
...run npm run build...
...81 seconds later...
...failure number 25...

(you get the picture, and 81 seconds is accurate 😆)

Ok. I found that prerender: {force: true} has the intended effect. I'll open another issue for the naming of that option, as I saw it before, but it didn't make sense that this would be what I wanted.


[request] add dump buttons to chemical infuser

currently since theres no way to draw the gases out of its input tanks, if you put the wrong gas into it, you have to destroy it and craft a new one because you just cant empty the input tanks

The Guage Dropper is the tool your are looking for to empty random tanks in the machines.

ah, I didnt  know that thanks a bunch.  I was getting frustrated because wrong fuilds/gass ended up in wrong machine and had to build new ones after that.  closing this now.


Fix Either/FilterValue compiler error

It looks like in old versions of persistent (e.g. https://hackage.haskell.org/package/persistent-2.9.2/docs/Database-Persist-Types.html), Filter had these constructors:
data Filter record = forall typ. PersistField typ => Filter
    { filterField  :: EntityField record typ
    , filterValue  :: Either typ [typ] -- FIXME
    , filterFilter :: PersistFilter -- FIXME
    }
    | FilterAnd [Filter record] -- ^ convenient for internal use, not needed for the API
    | FilterOr  [Filter record]
    | BackendFilter
          (BackendSpecificFilter (PersistEntityBackend record) record)

but in newer versions, it changed (https://hackage.haskell.org/package/persistent-2.10.4/docs/Database-Persist-Types.html#t:Filter):
data Filter record = forall typ. PersistField typ => Filter
    { filterField  :: EntityField record typ
    , filterValue  :: FilterValue typ
    , filterFilter :: PersistFilter -- FIXME
    }
    | FilterAnd [Filter record] -- ^ convenient for internal use, not needed for the API
    | FilterOr  [Filter record]
    | BackendFilter
          (BackendSpecificFilter (PersistEntityBackend record) record)

resulting in this compiler error:
/Users/maximiliantagher/Documents/Clones/Haskell/persistent-iproute/Database/Persist/IP.hs:42:71: error:
    • Couldn't match expected type ‘FilterValue IP’
                  with actual type ‘Either IP b0’
    • In the second argument of ‘Filter’, namely ‘(Left ip)’
      In the expression:
        Filter
          (unsafeCoerce field :: EntityField record IP)
          (Left ip)
          (BackendSpecificFilter ">>")
      In an equation for ‘>.>.’:
          field >.>. ip
            = Filter
                (unsafeCoerce field :: EntityField record IP)
                (Left ip)
                (BackendSpecificFilter ">>")
   |
42 | field >.>. ip = Filter (unsafeCoerce field :: EntityField record IP) (Left ip) (BackendSpecificFilter ">>")
   |                                                                       ^^^^^^^

FilterValue is defined like this:
data FilterValue typ where
  FilterValue  :: typ -> FilterValue typ
  FilterValues :: [typ] -> FilterValue typ
  UnsafeValue  :: forall a typ. PersistField a => a -> FilterValue typ

So, my reading of the situation is that it needed a third constructor, so what was previously Left/Right is now FilterValue/FilterValues, and just a switchout is necessary. I compiled with:
extra-deps: [persistent-2.10.4]

to confirm this fixes the compilation error.
I haven't tested this at runtime though.

Oh wait, I need to either use CPP or add a constraint on using a more recent persistent

Separately, would you support editing package metadata to require < 2.10.0 for persistent? https://hackage.haskell.org/package/persistent-iproute-0.2.3/persistent-iproute.cabal/edit

Separately, would you support editing package metadata to require < 2.10.0 for persistent? https://hackage.haskell.org/package/persistent-iproute-0.2.3/persistent-iproute.cabal/edit

Just did it.
Would you squash this PR into a single commit please?

Oh, you can squash right from the Github UI now:

Squashed and released!


Fix/claim node conflicts

Avoid conflicts between blank nodes when serializing VC to be stored in the cognitarium.
Details
When serializing VC rdf before storing in the cognitarium, in the purpose of making them totally isolated from others we ensure a hierarchy based on blank nodes instead of named nodes.
Regarding the claim reference, as it is linked to its subject (a named node) we change it by a static blank node _:c0, the resolution of its related subject being allowed as we forge the predicates dataverse:credential#subject & dataverse:credential#claim for the separation.
By introducing the static _:c0 blank node we're exposed to potential conflicts with existing blank nodes being in the claim. This PR brings here the renaming of those blank nodes prefixing them by _:b{} to avoid any possible conflict.

Summary by CodeRabbit

New Features

Enhanced handling of blank node identifiers in queries, improving the accuracy and efficiency of data retrieval.

Bug Fixes

Fixed an issue where incorrect attribute values and RDF node references were used, ensuring data integrity and consistency.

Refactor

Optimized the handling of blank nodes and identifier issuing to reduce unnecessary cloning, enhancing performance.
Refactored the DataverseCredential structure for better clarity and maintainability.

Documentation

Added documentation for the new Unexpected error variant, aiding in error handling and debugging.

size-limit report 📦

Path
Size

target/wasm32-unknown-unknown/release/okp4_objectarium.wasm
440.47 KB (0%)

target/wasm32-unknown-unknown/release/okp4_law_stone.wasm
654.01 KB (0%)

target/wasm32-unknown-unknown/release/okp4_cognitarium.wasm
835.63 KB (+0.19% 🔺)

target/wasm32-unknown-unknown/release/okp4_dataverse.wasm
673.86 KB (+0.46% 🔺)

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


bug(mat-dialog): First click in mat-dialog not working

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/edit/angular-pefszd
Steps to reproduce:

Click to open dialog.
Click big button in dialog.
click handler is executed only on second click

Expected Behavior
Alert from (click) handler
Actual Behavior
Alert opens only the second time the button is pressed.
It works fine if any of these changes are done:

ngFor over an array field, instead of a method returning the array.
element in the array is a number instead of an object
remove the ok button

Environment

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

It's because you're returning a new array every time fun is called. This means that whenever any event happens, you get a completely new button. The first time the click doesn't work, because moving focus causes it to be recreated in the middle of the click sequence. Closing since this isn't an issue in Material.


Red mark on bottom right when using nvim 0.1.3

environment

vim: neovim 0.1.3
vim-airline: 168dd7970a95c0c5049fec0b101b7f53b6a22469
OS: OSX 10.11.4
terminal: iTerm2

actual behavior
There's red mark on bottom right when I upgraded my neovim to 0.1.3. Its not there when I'm using 0.1.2.
Here's my configuration
" ----- vim-airline/vim-airline settings -----
let g:airline_theme = 'papercolor'
set laststatus=2
let g:airline_powerline_fonts = 1
let g:airline_detect_paste=1
let g:airline#extensions#tabline#enabled = 1

expected behavior
The red mark should be hidden.
https://github.com/neovim/neovim/issues/4564

This is not an issue, but a limitation of how airline is currently building the status line. Please track progress here : https://github.com/vim-airline/vim-airline/issues/157

I see. I'll close this issue then.


proxy::sessions::tests::same_address_uses_different_sockets deadlocks

What happened:
❯ for i in {1..100}; do cargo test --color=always --package quilkin --lib proxy::sessions::tests::same_address_uses_different_sockets; done
    Finished test [unoptimized + debuginfo] target(s) in 0.19s
     Running unittests src/lib.rs (target/debug/deps/quilkin-98ba5c957339e5bd)

running 1 test
test proxy::sessions::tests::same_address_uses_different_sockets ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 111 filtered out; finished in 0.00s

    Finished test [unoptimized + debuginfo] target(s) in 0.20s
     Running unittests src/lib.rs (target/debug/deps/quilkin-98ba5c957339e5bd)

running 1 test
test proxy::sessions::tests::same_address_uses_different_sockets ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 111 filtered out; finished in 0.00s

    Finished test [unoptimized + debuginfo] target(s) in 0.20s
     Running unittests src/lib.rs (target/debug/deps/quilkin-98ba5c957339e5bd)

running 1 test
test proxy::sessions::tests::same_address_uses_different_sockets has been running for over 60 seconds

# Deadlock, never finishes.

What you expected to happen:
The tests to pass consistently.
How to reproduce it (as minimally and precisely as possible):
See above.
Anything else we need to know?:
Environment:

Quilkin version: dev
Execution environment (binary, container, etc): shell.
Operating system: Debian
Custom filters? (Yes/No - if so, what do they do?): No
Log(s): N/A
Others: N/A

I'll look into it tomorrow, I should be able to debug this .

I'll look into it tomorrow, I should be able to debug this .

If I have time, I'll have a little poke around too, will post here if I find anything. 👍🏻

fixed by #826


Fix document link in vcxagency-node/README.md

Signed-off-by: InGee Kim<EMAIL_ADDRESS>

@ingee thanks for fixes! Thanks to your PR I've also discovered problem building PRs from forked repositories. It should be fixed now, can you please try to rebase this on latest master?


[FLINK-22857][table] Support evaluating expressions in SpecializedFunction

What is the purpose of the change
This enables to evaluate arbitrary expressions in SpecializedFunction. It allows to access all built-in functions including code generated ones during runtime. It enables writing UDFs in pure Java code without adjusting the code generation at various locations. As a proof-of-concept and reference implementation this PR also adds the ARRAY_CONTAINS function to both Table API and SQL.
Brief change log

Various refactorings around context instances
New APIs for SpecializedFunction
ARRAY_CONTAINS for Table API and SQL

Verifying this change
This change added tests and can be verified as follows: FunctionITCase, CollectionFunctionsITCase
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

Thanks for the review @matriv. I addressed your comments.


fix: add prefer global

:tada: This PR is included in version 1.0.46 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


[SUGEST] - A sugest for reduce further the script

Implemented map() to work for convert string to char. Has a revert, I was implement es6 too but I see that es5 is proprosital.

Hey, thanks~!
I had this initially, but the manual loop is much faster as it only traversing the string once. I felt that the significant performance increase was worth the additional three bytes. What do you think?
Here's a before/after comparison:
// Before:   259B gzip
encrypt   x 288,653 ops/sec ±0.38% (93 runs sampled)
decrypt   x 405,658 ops/sec ±1.73% (93 runs sampled)

// After:  256B gzip
encrypt   x 241,159 ops/sec ±0.38% (94 runs sampled)
decrypt   x 353,091 ops/sec ±1.48% (97 runs sampled)

Hey, thanks~!
I had this initially, but the manual loop is much faster as it only traversing the string once. I felt that the significant performance increase was worth the additional three bytes. What do you think?
Here's a before/after comparison:
// Before:   259B gzip
encrypt   x 288,653 ops/sec ±0.38% (93 runs sampled)
decrypt   x 405,658 ops/sec ±1.73% (93 runs sampled)

// After:  256B gzip
encrypt   x 241,159 ops/sec ±0.38% (94 runs sampled)
decrypt   x 353,091 ops/sec ±1.48% (97 runs sampled)

Indeed, always will be while and for structure more fast, but i think, that the code is more readable using map or/and forEach. In this case, if you wanna brush bits, is sure use while for this.


rewrite with scalajs

https://github.com/Jarlakxen/atom-scalajs-seed

Great! What is the benefit of using coffeescript instead of all scalajs?

What is the benefit of using coffeescript instead of all scalajs?

Just alignment with Atom community. Examples, apis and such are all coffeescript. Of course possible to use scala.js but not really any big benefit since it's just glue code for the atom apis.''
In the case of vscode, there's a lot of benefit in using typescript on the other hand since all the apis are typescript and you get intellisense and stuff when creating the glue code. Would be a lot of upfront work to make scala.js type interfaces for all typescript apis of vscode. I briefly looked at https://github.com/sjrd/scala-js-ts-importer before. Dunno. Think I want the other way around, create scala.js code for the ensime bindings and then auto-generate ts typings and use that in the vscode glue and just coffeescript cowboy in atom.

@fommil @hedefalk I'll close this for now (I'm going to get an overview of the open issues, we can always reopen this on a later date)

This one we need to keep open because it's going to be done by the sponsored student in the next few weeks :smile:

@fommil Rewrite of ensime-client or ensime-atom? If ensime-atom then I would've liked to know that before I started hacking around on it 😂

we haven't decided anything yet and I don't really know the tech stack but I don't think anything you'd do will be wasted: you'll learn weird things that need to be worked around and we probably need to leave the atom integration part in whatever the native language is (same for vscode). Plus it could be months before a rewrite would be ready for use.
If you want to get a head start with the scalajs rewrite then picking up the atom-scalajs-seed would probably be the best thing!

@fommil Alright, I'll keep on poking around and provide any knowledge I might pick up along the way to the student 👍

If you want to take a look, here is what I have right now https://github.com/Jarlakxen/atom-scalajs-seed/tree/ensime. Most of the code in api and sexp is the code as in ensime-server. The only changes are:

Because there is no java.io.File, I wrote my own File ( a very small version )/
Move the logging to slogging
Rewrite all the file IO operation using node binding ( https://github.com/scalajs-io/nodejs )

The tests are also the same as in ensime-server, but for some reason some of the are failing. I wasn't able to run the test in debug mode to trace whats going on.

@Jarlakxen most of the failed tests are caused by https://github.com/scala-js/scala-js/issues/3125 which is fixed in the next version of scala-js
The two remaining I believe are caused by the Generic/Shapeless code, I'm looking into it but it's quite hard to parse.

@Jarlakxen @fommil I managed to pass all the tests with minimal changes to @Jarlakxen's code so there is no major issue preventing us to reuse the code from ensime-server in a scala-js client library.
Not sure where it should sit though, references to a file api make it depend on node-js but I'm not sure it's really useful. I believe we should make it as generic as possible (node & browsers), maybe someone will want to build a webapp with such a client.

Oh! That's wonderful!
I was pretty sure that the problem it was due to a scala-js issues. I'm a little bit curiose, what was the problem with Shapeless?
I like the idea of a client agnostic of the environment ( node, browser and probably even jvm =P ), there is some parte of the code that rely on file operations (like looking for the java home, checking files, read the .ensime, etc ...) but maybe is not very difficult to provide this things as parameters of the client =D

@Jarlakxen The problem with shapeless was just that a SexpFormat[RawFile] wasn't provided so labelledProductFormat was used instead (it fits the implicit but with less priority). The code is here https://github.com/jfraudeau/atom-scalajs-seed/tree/ensime if you want to see.

Nice. We're currently doing some work in the server to replace File with RawFile and (when I finish my book) I know how to dramatically speed up the compile. See stalactite.

Now that they annouced Atom IDE (https://blog.atom.io/2017/09/12/announcing-atom-ide.html) probably the rewrite could be really simple and small ( just a couple of lines ). The key part is going to have a language server protocol implementation of Scala, like a standalone version of https://github.com/dragos/dragos-vscode-scala.

Nice!

Interesting! That points me again in the direction that it would probably be a good idea to consolidate efforts with @dragos. I've been away from the game too long now so don't really know the status of things…

sponsored developer is building language server into ensime after adding sbt 1.0 support to the plugin :wink:

Sounds great!


enable experimental quic feature

 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.


Metrics events - add utils to support mixpanel integration and persisting user settings in shell.

In standup I heard that we wanted to reach consensus on something? I don't see any details on what is being asked. Looking at the code, this seems great to me. I think adding events to every command is the only way to go. Can't wait to see this live!
Looks like we've got some linting issues, FYI

Merge conflict 🎗

@janedegtiareva looks like it's asking CI to opt in and that's why the build is breaking for this PR.

@janedegtiareva we for sure need to figure out how to not show this in CI - we will have this problem in all examples.

@janedegtiareva we for sure need to figure out how to not show this in CI - we will have this problem in all examples.

This was temporary not working due to changing some things due to comments. Back to working again.

Merge conflict 🎗

resolved

@janedegtiareva looks like it's asking CI to opt in and that's why the build is breaking for this PR.

Yeah this was temporarily broken due to commit /addressing comments churn. Back to working now.


Get rid of travis CI and switch to github actions

Need to switch this repo to github actions so it no longer depends on travis CI.

done!  i think  all we  have to do now is delete the travis  file!!

actually  we  also need to create the  conda build  for  windows, mac  and linux  so  reopening! docker build is done.


edit requests with status

What does this PR do?

this PR a requester should get her request to see if she gave the right information about the travel request
a requester may update and edit her request.

Description of Task to be completed?

a user will be able to edit his request while it is still open

How should this be manually tested?

copy this link to be cloned in terminal git clone https://github.com/Soma-Technologies-Inc/visitAfrica-backend.git`
Run npm install To install the packages needed in project dependencies
Run npm run statDev To start the application
run inthe postman
Run npm test to run test

Screenshots (if appropriate)

Questions:
N/A

can you please squash your commits


Parse AzureBlobFileSystem account_name in URI

Before this PR, one needs to manually specify account_name to storage_options when opening a file from Blob Storage with fsspec.
import fsspec

# ok
with<EMAIL_ADDRESS>account_name="") as f:
   ...

# fails
with<EMAIL_ADDRESS>as f:
   ...

However, the account name can be part of the URI, as specified by Azure's doc.
This PR allows the user not to specify account_name to storage_options when it is in the URI
import fsspec

# ok
with<EMAIL_ADDRESS>account_name="") as f:
   ...

# ok
with<EMAIL_ADDRESS>as f:
   ...

Thanks for the contribution!


Overriding vhost-templates global

Is it possible to override the vhost-templates global, so it will be used for all projects?

At the moment you can only alter general server settings. For that see cfg/apache-2.4 directory which also includes an example.
Is this sufficient or do you need to alter the vhost itself in general?
If so, could you give some examples about what settings you require.

One can override the vhost-templates for each project like you described in the issue https://github.com/cytopia/devilbox/issues/142
I would like to do something like that, but global, so i don't need to put .devilbox folder inside the projects folder.

As said, this is currently not possible. Depending on the options:

If yours are better default option than current present, I will add them to: https://github.com/devilbox/vhost-gen
If yours are very specific to your local dev environment I will have to implement a way to generally overwrite the defaults

I think, it would be nice, if all projects are also accessible over www.project.loc, not only over project.loc. But maybe other users don't want this behaviour like me.

@mavimedia OK.
So this is not (yet) officially ported, but as a quick work-around you can do the following:

In the devilbox root directory, create a new directory vhost-gen/
Copy all three templates files into the new vhost-gen/ dir from: https://github.com/devilbox/vhost-gen/tree/master/etc/templates
Add a new volume mount as shown below into docker-compose.yml

  httpd:
    ...
    volumes:
      ...
      - ${DEVILBOX_PATH}/templates:/etc/vhost-gen/templates
      ...

This way you should be able to edit the templates globally for all vhosts from within the devilbox project.
Please let me know how this works out for you.

It works, thanks!
But in ypur example it should be
httpd:
...
volumes:
...
- ${DEVILBOX_PATH}/vhost-gen:/etc/vhost-gen/templates
...

Yes you are right. I will fix my comment above for others to better see how it works.

The whole thing is going to be documented with new documentation setup: https://github.com/cytopia/devilbox/issues/157

This will be fixed and documented here:
https://github.com/cytopia/devilbox/pull/298

@mavimedia feature will be very soon fully integrated and documented