Tutorial is incompatible with default authorization servers - okta-hosted-login

https://github.com/okta/samples-php/tree/develop/okta-hosted-login
This tutorial uses a default authorization server, which works in a free okta dev account. However it depends on okta-jwt-verifier-php which is incompatible with default authorization server in a production account.
It is unclear to me how that is possible, but I am looking for a recommendation on how to set up authentication with OIDC, OKTA, PHP, and the default authorization server, in a production environment.
Can you point me to documentation, or a code example that will work for my use case?
Thanks,
Aidan.

@bretterer @vijetmahabaleshwar-okta @ivanezeigbo-okta is it possible to get pointed in the right direction?

@agawronski, Unfortunately, the PHP libraries for JWT Verifier are not compatible with the default authorization server. The packages were built for the custom authorization server setup.
Since this ticket was created, we have announced that our PHP libraries are in security path only mode and will be archived soon. If you have any other questions, please feel free to reach out on our developer forum or<EMAIL_ADDRESS>


win32: Drop support for XP and Vista

Microsoft has already dropped support for Windows 7 and earlier.
Vim 9 will be a good timing for dropping support for old Windows.
This patch drops support for XP and Vista, because Windows 7 may still
have quite a few users,

I wonder what the advantage is of dropping XP support.

We can stop using dynamic loading for some APIs, and it makes the source code simpler.
However, the most important purpose of this patch is the preparation for the future improvement.
Writing a new patch for old platforms requires extra efforts. #5946 is a good example.

Python 2 is deprecated.  Still, many people use Python 2 scripts.
IE11 has been dropped long ago, still about 1% of people use it.
Windows XP is not supported for some time.  Still quite a few people use
Windows XP.
Not sure how accurate it is:
google.com/search?q=how+many+people+run+windows+xp suggests there are
140 million users.
Vim has a long history supporting old setups.  It's hard to decide where
to drop support, but I do believe that Windows XP is still relevant.
Unless it's no longer possible to run the current version of Vim on
Windows XP.  I have to admit I haven't tried recently.
--
"Computers in the future may weigh no more than 1.5 tons."
Popular Mechanics, 1949
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

FWIW, I am in favor of dropping Windows XP support. I checked a couple of other projects:

git-for-windows - dropped support in September 2016
Notepad++ - dropped support around January first 2018 (according to a forum entry with release 7.5.1, but couldn't find the release date of 7.5.1)
Chrome support was dropped in 2016 according to a Google blog post
Mozilla announced Firefox has dropped support with Release 52.9 ESR, which was released in summer 2018

So how about dropping Windows XP support with the next major release then?

Codecov Report

Merging #6091 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6091      +/-   ##
==========================================
- Coverage   87.34%   87.33%   -0.02%
==========================================
  Files         142      142
  Lines      157190   157190
==========================================
- Hits       137294   137276      -18
- Misses      19896    19914      +18

Impacted Files
Coverage Δ

src/netbeans.c
74.66% <0.00%> (-0.82%)
:arrow_down:

src/testing.c
92.15% <0.00%> (-0.68%)
:arrow_down:

src/os_unix.c
70.85% <0.00%> (-0.14%)
:arrow_down:

src/search.c
91.39% <0.00%> (-0.12%)
:arrow_down:

src/sign.c
94.94% <0.00%> (-0.09%)
:arrow_down:

src/gui.c
63.92% <0.00%> (-0.06%)
:arrow_down:

src/ex_getln.c
91.44% <0.00%> (-0.05%)
:arrow_down:

src/gui_gtk_x11.c
58.85% <0.00%> (ø)

src/window.c
89.74% <0.00%> (+0.03%)
:arrow_up:

src/if_xcmdsrv.c
88.90% <0.00%> (+0.52%)
:arrow_up:

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 ebe9d34...c457685. Read the comment docs.

Let's keep XP support for half a year, and then we can reconsider.  It's usually better to remove support after a major release, so perhaps after Vim 9 is out (and stable).  Let's close this now, there is no point in trying to keep the PR in sync.

It's usually better to remove support after a major release, so perhaps after Vim 9 is out (and stable).

Okay, agreed.

there is no point in trying to keep the PR in sync.

I will keep this patch up to date locally.


Support for null-safe equality comparison at Spark-Elasticsearch datasource.

For Spark 1.5, EqualNullSafe can be pushed down as a filter operator.
(https://issues.apache.org/jira/browse/SPARK-9814)
I think this can be easily done adding combination of equality comparison query and missing filter adding a case at ElasticsearchRelation.translateFilter() in org.elasticsearch.spark.sql.DefaultSource.scala
Could I open a PR for this?

I missed the new filter. Sure, go ahead with a PR (take a look at StringStartsWith and co to add the case in a backwards compatible way with Spark versions lower than 1.5).
Thanks,

Hi,
Thanks for the PR. It is merged in master and 2.x
I've added a null check for EqualsTo - for NullSafeEquals it doesn't seem to make sense considering nulls are handled by the operator directly (based on what I've seen in the Spark code).
The same seems to apply for String operators - in fact is null is passed, the whole filter can't really work properly since the regex becomes global (every string starts or ends with null for example).
Cheers,

Yeap I agree. I think this wouldn't be a big deal.
Thanks :)


Console process order changed in 15063's v2 console making npm harder to close

In 15063's v1 console, and any console in Windows 10.0.14393 and older, Windows seems to order the processes from newest-to-oldest for GetConsoleProcessList (probably the opposite of the attachment order).  When a console is closed, Windows considers each process, in order, and delivers it a CTRL_CLOSE_EVENT event.  Windows gives the process 5 seconds to exit, after which point it terminates the process.
In 15063's v2 console, the order of the process list has reversed and is now oldest-to-newest.  Maybe the change was intentional?  Regardless, it affects npm (the node package manager), as detailed here: https://github.com/Microsoft/vscode/issues/26807#issuecomment-304503591.
The test case given on https://github.com/Microsoft/vscode/issues/26807 was fairly easy for me to reproduce, and it doesn't require VSCode.  Installing node.js adds node.bat and npm.cmd to the PATH.  Run the commands in an ordinary console, and instead of "pressing the garbage bin button", try to close the console window.  In my experience, the first click leaves the youngest node.exe process running, and the second click finishes the job.  (VSCode's "garbage bin button" eventually turns into winpty posting a WM_CLOSE message, which kills winpty-agent.exe.  The console window survives, but it's hidden, so the node.exe is leaked.)
Starting in Windows 8, when the console is delivering CTRL_CLOSE_EVENT events, it seems to abort the operation if a process exits on its own accord.  Windows 7, on the other hand, skips over dead processes.
The combination of the 15063 v2 change and the Win8 change breaks npm, which has a process tree like so:

A: node.exe (npm run, owns Job1)

B: cmd.exe (assigned to Job1)

C: node.exe (tsc --watch)

Windows delivers CTRL_CLOSE_EVENT to A(node.exe).  node.exe turns it into a libuv-emulated SIGHUP signal, which is sent to Node's main thread and ignored.  After 5 seconds, Windows kills A(node.exe).  Killing A(node.exe) destroys Job1, which destroys B(cmd.exe).  Apparently the console then aborts the close operation because B(cmd.exe) is missing.
Previously, Windows signaled C(node.exe) first, and after C exited, the other two processes would definitely exit.
Also: There seems to be a race condition between process cleanup (in the kernel?) and the close event signaling (in conhost.exe?), and sometimes the console close operation is able to skip over already-dead processes.
I wrote a test program demonstrating things discussed in this report -- closetest.cc, https://gist.github.com/rprichard/7ec3fe1b199f513bee82fea196a82a79.  It has notes at the top of the file and a --help option.

To see the change in signaling order (EXAMPLE 1 in closetest.cc), run Sysinternals dbgview.exe, then closetest.exe without arguments.

To create a console that's hard to close (EXAMPLE 2), run closetest.exe -d alternate --gap -n N.  In Windows 8 and up, you'll have to close the console N + 1 times.  In Vista or Windows 7, just once is enough.

To see the race condition (EXAMPLE 3), try:

closetest.exe -d [backward/forward] -m job -n 10 --alloc 1, or
closetest.exe -d [backward/forward] -m job -n 50

On a v1 or <= 14393 console, use -d backward.  On the v2 15063 console, use -d forward.

The closest analog to npm's issue is closetest.exe -d forward -m job -n 2 --alloc 100.

Windows build number:  10.0.15063.332

This wasn't intentional and I'm making a change to put the process order back to how it was.
Basically, what happened was that I decided to use std::list instead of NT Driver LIST_ENTRY for the process list.
Somehow the "InsertHeadList" command from the NT LIST_ENTRY structures got converted into a .push_back instead of a .push_front in the new std::list usage. Probably my fault.  Sorry about that.
The fix should make it into Insiders builds within the next few weeks. Thanks for your report!

Hey @rprichard,
I'm interested in using your code sample above at  https://gist.github.com/rprichard/7ec3fe1b199f513bee82fea196a82a79 as a part of our test suites to prevent further regressions in this area, but I didn't see any licensing information. Would you be able to provide the code under an open source license? Generally we prefer a permissive license such as MIT, but please choose whatever license fits best for you. You can take a look at https://choosealicense.com/ if you need help choosing a license.
Thanks,
--Michael

@miniksa That sounds good to me.  I'm traveling at the moment, but I'll add an MIT license file once I have the chance.

@miniksa I updated the Gist by adding an MIT License block at the front (revisions 2 and then 3).

@rprichard Thank you very much!

Fixed in 16257.

This issue was moved to Microsoft/console#95


when run  ./scripts/cli/start_cli_testnet.sh error

error: failed to run custom build command for grpcio-sys v0.4.4
Caused by:
process didn't exit successfully: /usr/libra/target/debug/build/grpcio-sys-18712f499919ddc1/build-script-build (exit code: 101)
--- stdout
cargo:rerun-if-changed=grpc_wrap.cc
cargo:rerun-if-changed=grpc
cargo:rerun-if-env-changed=GRPCIO_SYS_USE_PKG_CONFIG
cargo:rerun-if-env-changed=CARGO_CFG_TARGET_OS
running: "cmake" "/root/.cargo/registry/src/github.com-1ecc6299db9ec823/grpcio-sys-0.4.4/grpc" "-DgRPC_INSTALL=false" "-DgRPC_BUILD_CSHARP_EXT=false" "-DgRPC_BUILD_CODEGEN=false" "-DgRPC_BENCHMARK_PROVIDER=none" "-DCMAKE_INSTALL_PREFIX=/usr/libra/target/debug/build/grpcio-sys-c998cb3e25589d95/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC -m64" "-DCMAKE_CXX_COMPILER=/usr/bin/c++" "-DCMAKE_BUILD_TYPE=Debug"
-- Configuring done
-- Generating done
-- Build files have been written to: /usr/libra/target/debug/build/grpcio-sys-c998cb3e25589d95/out/build
software version
go version go1.10.8 linux/amd64
libprotoc 3.8.0

check your rustup toolchain versions please.
https://github.com/libra/libra/issues/99

I have to upgrade go to the latest version, I was using go 1.9.2 and having the same error.

@horca17 latest nightly version fails to build.
You can do exact rust toolchain version like this way.
cargo clean --package libra_swarm
rustup toolchain install nightly-2019-05-22-x86_64-unknown-linux-gnu
rustup override set nightly-2019-05-22
cargo run -p libra_swarm -- -s

libprotoc 3.8.0
go1.12.5
rustup 1.18.3
rustc 1.36.0-nightly
ok~

libprotoc 3.8.0
go1.12.5
rustup 1.18.3
rustc 1.36.0-nightly
ok~

cmake >= 3.3


The Endless Dungeons license

https://sourceforge.net/projects/endlessdungeons/ does not seem to have a clear indication of the license next to the sourcecode.
Contact the author(s) and ask for clarification / possibility to add (a suitable) license information.

I sent a message to the maintainer of the project (https://sourceforge.net/u/litmusdragon/profile/).

I got a reply:

My intention was that the sourcecode could be freely modified and distributed, for non-commercial purposes only. I would also appreciate an attribution somewhere in either the documentation or in-game text. So a CC-by-Attribution license would be most appropriate.

CC-BY is not non-commercial, so it would have to be more like CC-BY-SA-NC if I'm not mistaken. Furthermore, CC is not a good choice for code. GPL seems like the more appropriate choice for the requirements outlined. Will contact again and ask for clarification.

Got another reply:

You are correct, I misunderstood the license. In that case I believe CC BY-NC-SA 2.0 as defined here is the option that best suits my intention: https://creativecommons.org/licenses/by-nc-sa/2.0/
I'm fine with you putting it on Github.

I guess, this means that the author insists of using a CC license because he said so even after I explicitely mentioned to him that even CC does not recommend to use CC as license for open source code.
On the other hand, we already have such cases. I will simply comply and put the project on Github with the permission of the author.

Project has been imported and licensed at https://github.com/Trilarion/endlessdungeons

Added entry


feature: use neovim virtual text to display tree titles and transclusions

This would need decent plaintext output from forester

We should use treesitter to get a hold of the AST and use lua to call forester to query the title.


Add name arg to MobileNet V1 and V2

Followup from #19695 and 2b7120b.

Codecov Report
Attention: Patch coverage is 0% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 22.95%. Comparing base (310c275) to head (05904ff).

Files
Patch %
Lines

keras/src/applications/mobilenet.py
0.00%
3 Missing :warning:

keras/src/applications/mobilenet_v2.py
0.00%
3 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           master   #19712       +/-   ##
===========================================
- Coverage   78.52%   22.95%   -55.57%
===========================================
  Files         498       16      -482
  Lines       45769     2230    -43539
  Branches     8456      369     -8087
===========================================
- Hits        35942      512    -35430
+ Misses       8091     1708     -6383
+ Partials     1736       10     -1726

Flag
Coverage Δ

keras
?

keras-jax
?

keras-numpy
?

keras-tensorflow
?

keras-torch
?

keras.applications
22.95% <0.00%> (?)

keras.applications-numpy
22.95% <0.00%> (?)

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

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


Strange LIMIT behavior with multiple joins/subqueries

CrateDB version: 2.2.3
JVM version: Java(TM) SE Runtime Environment (build 1.8.0_144-b01)
OS version / environment description: Ubuntu 16.04
Here's the large query I have:
SELECT agg_total,
       agg_unique,
       agg_median,
       agg_sum,
       id,
       title,
       table_B._id AS internal_id
FROM
(
    SELECT col_A,
           percentile(col_E, 0.5) AS agg_median,
           SUM(col_E) AS agg_sum
    FROM table_A
    WHERE col_B = 'value'
          AND col_C IS NULL
          AND col_D != 'Unknown'
          AND col_E IS NOT NULL
          AND col_F = 'value_1'
    GROUP BY col_A
) group_A
INNER JOIN (
    SELECT COUNT(*) AS agg_total,
           COUNT(DISTINCT amplitude_id) AS agg_unique,
           col_A
    FROM table_A
    WHERE col_B = 'value'
          AND col_C IS NULL
          AND col_D != 'Unknown'
          AND col_F = 'value_2'
    GROUP BY col_A
) group_B ON group_A.col_A = group_B.col_A
INNER JOIN table_B ON group_A.col_A = table_B._id
WHERE agg_total >= 0
      AND type = 'type_1'
      AND "exists" = true
ORDER BY agg_unique DESC
LIMIT X;

And here's the strange behavior:

If the LIMIT is 100, this query returns 3 rows (the max)
If the LIMIT is 25, this query returns 1 row
If the LIMIT is 2, this query returns no rows
If the first INNER JOIN is changed to a FULL OUTER JOIN, the LIMIT behaves as expected (LIMIT 1 = 1 row, LIMIT 2 = 2 rows, LIMIT 3+ = 3 rows)

Am I just crazy and don't understand SQL? I would assume the LIMIT would limit the number of final rows returned, but it's almost like the LIMIT is running...inside the subqueries?
I tried running an EXPLAIN and I don't really understand the output. I can email it to you if you'd like.

@SneakyMax can you please email me the outputs of explain for both INNER JOIN and FULL OUTER JOIN? marios [ at ] crate.io

@SneakyMax Would also be helpful if you could provide the table schemas and sample data.

On it


Templates try to load url from future, which causes error in Django 1.9

I'm using  Django==1.9.10, so I'm getting such errors.

TemplateSyntaxError at /checkout/paypal/preview/2/
'url' is not a valid tag or filter in tag library 'future'

All because of {% load url from future %} line e.g. here and here.
Do you intend to remove those lines according to stackoverflow?
Edit: I'm using django-oscar-paypal==0.9.7.

Yeah I came across this error too and found the easy solution was to override the templates in my own template folder. So i just forked the project from my lib and added it to my template folder and just deleted the line on express and payflow templates. It's a tad hacky but it works.

This is due to deprecated template tags. If you look in Pull Request section of this repository, you will find that the contributions to fix it have been sitting there for a long time. The same is true of all other django-oscar-* modules. For some reason, the owners of this project seem VERY slow, if at all, at accepting incoming changes.

i think that this particular project was actually abandoned.. There seems to be no additions and any sorta improvements around it.

I've addressed this and other issues in  #159.

@solarissmoke  thanks man this will be of great help.. cheers


Implement new user login workflow

per Figma: https://www.figma.com/file/wo6SjH2DBuXf4DgL8rDBOX/PREreview?node-id=361%3A485

Closed by #147


feat: introduce bundle optimizer

Fix #2133

@mgechev , nice result.

Yeah, not impressive but it's still something :-)
It drops mostly the useless metadata, that's why it's applicable only for aot mode.


Expose revision count on post page

We have revision information on most posts since we started autoflagging.
Would it be possible to expose this on the post page and on /review, so that you don't have to leave metasmoke to determine whether or not the post is vandalism?

Similar FR to expose the count in the webapi:
https://github.com/Charcoal-SE/metasmoke/issues/135

Thoughts about where this could be shown? I'm leaning toward a tooltip, as it's not information one needs often, but I'm not sure where.

Maybe in the bottom-right corner next to the user info? E.g.

reported 8 minutes ago | edited 3 minutes ago | posted by user353809 (1)

We don't have revision timestamp data, only revision count, which'd complicate that.

reported 8 minutes ago | edited once | posted by user353809 (1)


chore(main): release 1.2.3

:robot: I have created a release beep boop
1.2.3 (2024-02-14)
Bug Fixes

prerelease on (b3bcb92)
rc1 update (#15) (6a741c9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ixxeL-actions/actions/releases/tag/v1.2.3 :sunflower:


No system node pool by default ?

"When you create a new AKS cluster, you automatically create a system node pool with a single node." So I'm expecting to have as least one system pool on all my clusters, especially the new created (using CLI 2.4.0 but with aks preview enabled)
I just created an AKS cluster, and the default nodepool (name nodepool1) is a "user" nodepool when I'm using CLI az aks nodepool show

{
"agentPoolType": "VirtualMachineScaleSets",
"availabilityZones": null,
"count": 1,
"enableAutoScaling": true,
"enableNodePublicIp": false,
"id": "/subscriptions/6c9dc160-dca4-47cb-9e76-ed16a4b4edb1/resourcegroups/rg-aks-test/providers/Microsoft.ContainerService/managedClusters/aks-test/agentPools/nodepool1",
"maxCount": 3,
"maxPods": 110,
"minCount": 1,
"mode": "User",

with kubectl, I don't have the specific label either (kubernetes.azure.com/mode: system).

apiVersion: v1
kind: Node
metadata:
annotations:
node.alpha.kubernetes.io/ttl: "0"
volumes.kubernetes.io/controller-managed-attach-detach: "true"
creationTimestamp: "2020-04-27T14:09:39Z"
labels:
agentpool: nodepool1
beta.kubernetes.io/arch: amd64
beta.kubernetes.io/instance-type: Standard_DS2_v2
beta.kubernetes.io/os: linux
failure-domain.beta.kubernetes.io/region: westeurope
failure-domain.beta.kubernetes.io/zone: "0"
kubernetes.azure.com/cluster: MC_rg-aks-test_aks-test_westeurope
kubernetes.azure.com/role: agent
kubernetes.io/arch: amd64
kubernetes.io/hostname: aks-nodepool1-17697912-vmss000000
kubernetes.io/os: linux
kubernetes.io/role: agent
node-role.kubernetes.io/agent: ""
storageprofile: managed
storagetier: Premium_LRS

Then if I update the nodepool mode to system, the mode appears in json (CLI) and also as a label with kubectl

{- Finished ..
"agentPoolType": "VirtualMachineScaleSets",
"availabilityZones": null,
"count": 1,
"enableAutoScaling": true,
"enableNodePublicIp": false,
"id": "/subscriptions/6c9dc160-dca4-47cb-9e76-ed16a4b4edb1/resourcegroups/rg-aks-test/providers/Microsoft.ContainerService/managedClusters/aks-test/agentPools/nodepool1",
"maxCount": 3,
"maxPods": 110,
"minCount": 1,
"mode": "System",

BUT if I try to modify again the mode to user, I get the following error:

"Operation failed with status: 'Bad Request'. Details: Must define at least one system pool."

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

ID: 225b99ee-4c4d-54c2-cafa-ddcd286438da
Version Independent ID: ca9410a8-8ddb-5528-a0ac-0963458f190e
Content: Use system node pools in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/use-system-pools.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

I created an issue (https://github.com/Azure/AKS/issues/1583), because I'm not sure documentation is faulty

@lgmorand Thank you for your question!
As you have already opened an issue on AKS https://github.com/Azure/AKS/issues/1583, requesting you to please continue the discussion there.
Since this channel is for driving improvements towards MS Docs, and you mentioned this issue is not related directly to the document. I will proceed to close this issue now.


bugfix: plugin node-status and heartbeat don't have schema.

Fix #1245.

where is heartbeat plugin? @membphis

where is heartbeat plugin? @membphis

sorry for this, added.


kubemci commands have a litany of testing flags

This is due to k8s.io/ingress-gce/pkg/loadbalancers having a dependency on the testing package. This needs to be fixed, and that repo revendored.

[x] Remove k8s.io/ingress-gce/pkg/loadbalancers dependency on testing package.
[x] Revendor k8s.io/ingress-gce/pkg/loadbalancers

Thanks for finding this out @perotinus!
Sent https://github.com/kubernetes/ingress-gce/pull/69 to remove ingress-gce/pkg/loadbalancers dependency on testing

This is now fixed.


Remove mprotect()/rwx-related patch entry from bazel/v8.patch

A patch is introduced by https://github.com/envoyproxy/envoy/pull/23257 to fix  MemoryAllocator::PartialFreeMemory() on v8 <IP_ADDRESS> (https://github.com/envoyproxy/envoy/blob/ad800292a630c273fb548ac6d320522ffdbd2a8a/bazel/repository_locations.bzl#L1053). See https://github.com/envoyproxy/envoy/issues/23243 for more details.
That patch is copied from https://chromium-review.googlesource.com/c/v8/v8/+/3700352. That commit on v8 upstream is included in 10.5 (can be checked from 10.5-lkgr: https://chromium.googlesource.com/v8/v8.git/+/refs/heads/10.5-lkgr/src/heap/memory-allocator.cc#407) hence when we adopt 10.5 or higher, we should remove that patch (probably most of the patches will be removed/adapted anyway).

Seems like we are aiming for 10.7: https://github.com/envoyproxy/envoy/pull/23257#pullrequestreview-1122924600

@PiotrSikora I tried to upgrade v8, however, I stumbled upon: https://chromium-review.googlesource.com/c/v8/v8/+/3834031. Do you think we need to have this enabled by default too (hence reverting that commit) + adding a flag to enable V8_USE_ZLIB?
Note: zlib is enabled by default in 10.7-lkgr: https://github.com/v8/v8/blob/6c8b357a84847a479cd329478522feefc1c3195a/BUILD.gn#L373-L376.

@dio you can ignore it, snapshot compression was never enabled in the Bazel build anyway. Did you run into any issues because of that change?

@PiotrSikora nope. I asked since previously we patched zlib header path.

Did you run into any issues because of that change?

No. Without zlib we're good. Thanks!

Submitted https://github.com/proxy-wasm/proxy-wasm-cpp-host/pull/310 to have proxy-wasm-cpp-host to use newer V8, <IP_ADDRESS> (with some updated --test_timeouts).

Seems like https://github.com/proxy-wasm/proxy-wasm-cpp-host/pull/310 is merged (the updated v8 version is v<IP_ADDRESS>). I can set up my own bucket at first to serve the required snapshot for submitting a PR (to upgrade the v8 version for Envoy), but then we need someone who has access to provide the snapshot on [https://storage.googleapis.com/envoyproxy-wee8/](https://storage.googleapis.com/envoyproxy-wee8 (ref: https://github.com/envoyproxy/envoy/blob/e5ec3dad31a0588227586750f77cae7d93c3cdf1/bazel/repository_locations.bzl#L1058) using https://storage.googleapis.com/envoyproxy-wee8/wee8-fetch-deps.sh.
cc. @envoyproxy/wasm-dev

@mpwarres should have access to generate and upload them.

@mpwarres should have access to generate and upload them.

Ack, ran out of time to do this today, but will get to it tomorrow.

The following files have been added to the https://storage.googleapis.com/envoyproxy-wee8 bucket:
v8-<IP_ADDRESS>.tar.gz
v8-<IP_ADDRESS>-deps.sha256
chromium-base_trace_event_common-521ac34ebd795939c7e16b37d9d3ddb40e8ed556.tar.gz
chromium-icu-20f8ac695af59b6c830def7d4e95bfeb13dd7be5.tar.gz
chromium-zlib-f48cb14d487038d20c85680e29351e095a0fea8b.tar.gz

Thanks, @mpwarres!


Milestone 5: Desktop Version Portfolio 💻🛠

In this milestone

I kindly suggest you to consider the use of media query in your project.
Attached is a link that may be of help

https://github.com/microverseinc/curriculum-html-css/blob/main/responsivness.md


How to display all the charts as coverflow with  fusioncharts Jquery?

I am using fusioncharts-jquery to display my chart, and now I want to display all my charts as coverflow, is there any way?
Thanks!

Hello,
Yes, you can render FusionCharts as coverflow using fusioncharts.jqueryplugin.

Replace coverflow  element with  (container to render FusionCharts).
Define unique ID for each container elements.
On document ready use the insertFusionCharts method to render the chart.

I have also attached a sample for your reference.
coverflow.zip

Thanks Rohit! It looks very neat!
Joyce


Audit field type registry bug

Summary
In class AuditFieldTypeRegistry method validateAuditType works incorrectly.
Method validateAuditType checks whether provided audit type exists and if it's exist -> exception is raised:
        if (!empty(static::$auditTypes[$auditType])) {
            throw new \LogicException(sprintf('Unknown audit type %s.', $auditType));
        }

Should be:
        if (empty(static::$auditTypes[$auditType])) {
            throw new \LogicException(sprintf('Unknown audit type %s.', $auditType));
        }

Steps to reproduce
Use method addType to add new type to auditable fields.
Actual Result
Exception Unknown audit type is raised
Expected Result
New audit type should be added
Details about your environment

OroPlatform version: 4.2.6
PHP version: 8.0

Hi @andriusvo,
You can extend the Oro\Bundle\DataAuditBundle\Model\AuditFieldTypeRegistry to override the maps as they are stored in protected properties

@anyt yes, but this is a bug and in any case should be fixed ;)

The data audit feature in the oro/platform package supports the limited list of field types. If you add an unsupported type - the validation triggers an error, that is, as I understand, expected behavior.

@anyt Please check if statement one more time. In Oro code this if statement says that: If provided $auditType is not empty (== is found) - thrown an exception) And there should be exactly the opposite - `If provided $auditType is empty (== found) - thrown an exception.

I see there is a unit test for this method
\Oro\Bundle\DataAuditBundle\Tests\Unit\Model\AuditFieldTypeRegistryTest::testAuditFieldTypeRegistry.
Could you, please, change it to reproduce the bug?

It seems the validation message is missliding. According to the test, it throws an exception if you add the type that is already in the map.

Ok, let me give a real example for you. In AuditFieldTypeRegistry we have $typesMap where doctrine type is mapped to audit type. And now, f.e. I want to add integer_array (Doctrine type) to as simplearray (audit type). How I will achieve this? By writing this code:
        if (false === AuditFieldTypeRegistry::hasType(IntegerArrayType::TYPE)) {
            AuditFieldTypeRegistry::addType(IntegerArrayType::TYPE, 'simplearray');
        }

Now, method validateAuditType will be called and will be checked if audit type simplearray exists, and if exists -> the exception is thrown <---- Here is a bug! Should be the opposite, if the audit type is not found, only then the exception should be thrown. We are not checking duplicates of mapping here, we are checking only if this audit type exists.


Thumbs Swiper isn't working well in Shopify hydrogen

Check that this is really a bug

[X] I confirm

Reproduction link
https://hydrogen-a5ca48aa978097b02d14.o2.myshopify.dev/products/socks
Bug description
I am developing the Shopify hydrogen theme using RemixJS now.
The thumbs swiper is working at localhost, but it isn't working well at production mode.
import React, {useEffect, useState} from 'react';
import type {LinksFunction} from '@shopify/remix-oxygen';
import {Swiper, SwiperSlide} from 'swiper/react';
import {Image} from '@shopify/hydrogen';
import {FreeMode, Navigation, Thumbs} from 'swiper/modules';
import {MediaConnection} from '@shopify/hydrogen/storefront-api-types';
import type {Swiper as TSwiper} from 'swiper';
import styles from './desktopMediaGallery.css';

type TProps = {
  media: MediaConnection;
};

export const links: LinksFunction = () => [
  {
    rel: 'stylesheet',
    href: styles,
  },
];

export const DesktopMediaGallery: React.FC<TProps> = (props: TProps) => {
  const [thumbsSwiper, setThumbsSwiper] = useState<TSwiper | null>(null);
  const {media} = props;

  return (

      <Swiper
        onSwiper={(swiper) => setThumbsSwiper(swiper)}
        height={261}
        spaceBetween={26}
        slidesPerView={3}
        freeMode={true}
        loop={true}
        watchSlidesProgress={true}
        direction={'vertical'}
        modules={[FreeMode, Navigation, Thumbs]}
      >
        {media.nodes.map((node, index) => (
          <SwiperSlide key={index}>
            <Image src={node.previewImage?.url} width={'90%'} />
          </SwiperSlide>
        ))}
      </Swiper>

        <Swiper
          spaceBetween={30}
          slidesPerView={1}
          loop={true}
          thumbs={thumbsSwiper ? {swiper: thumbsSwiper} : undefined}
          modules={[FreeMode, Thumbs]}
          className="mySwiper2"
        >
          {media.nodes.map((node, index) => (
            <SwiperSlide key={index}>
              <Image src={node.previewImage?.url} />
            </SwiperSlide>
          ))}
        </Swiper>

            <Image
              src="https://crownandpaw.com/cdn/shop/files/icon-1_1_100x.png?v=1657724047"
              height={35}
              width={35}
            />

Made in USA

            <Image
              src="https://crownandpaw.com/cdn/shop/files/icon-2_1_100x.png?v=1657724064"
              height={35}
              width={35}
            />

Tracked shipping

            <Image
              src="https://crownandpaw.com/cdn/shop/files/icon-3_1_100x.png?v=1657724084"
              height={35}
              width={35}
            />

5 Star Reviews

  );
};

How to fix it?
Expected Behavior
No response
Actual Behavior
No response
Swiper version
10.1.0
Platform/Target and Browser Versions
Windows Chrome 115.0.5790.171
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Do you want to ask a question? Are you looking for support? Stack Overflow or Swiper Discussions is the best places for getting support

Please, don't use GitHub issues for questions


SharePoint SE Installation Fails within W2022 Server Core

Dear,

May be because of SE setup.exe now another additional argument must be added "/IAcceptTheLicenseTerms".
Otherwise the script raise an exception saying the setup installer requiers a reboot.
https://github.com/brianlala/AutoSPInstaller/blob/b14be321bbd3f431b81608e0c0f658f55e89e3ac/SP/Automation/AutoSPInstallerModule.psm1#L996

Because of the way how the server core is working, we cannot create shortcut , the conditional statement at the following line should also check if the current machine is a server core:
https://github.com/brianlala/AutoSPInstaller/blob/b14be321bbd3f431b81608e0c0f658f55e89e3ac/SP/Automation/AutoSPInstallerMain.ps1#L205

Note: How to retrieve the server product info with the server level : Determining Whether Server Core Is Running

In addition the config XML for server core must change as following:

Display Level must be none
<Display Level="none" CompletionNotice="No" AcceptEula="Yes"/>
(https://github.com/brianlala/AutoSPInstaller/blob/b14be321bbd3f431b81608e0c0f658f55e89e3ac/SP/Automation/AutoSPInstallerModule.psm1#L316)

USINGUIINSTALLMODE must be "0"
<Setting Id="USINGUIINSTALLMODE" Value="0"/>
https://github.com/brianlala/AutoSPInstaller/blob/b14be321bbd3f431b81608e0c0f658f55e89e3ac/SP/Automation/AutoSPInstallerModule.psm1#L321

Hi @vherrlein and @brianlala ,
I have to install sharepoint SE on Windows Server 2022.
@vherrlein : Does the script work with your last posted modification?
@brianlala  : Is it possible to use your script for the installation of SharePoint "SE" now?
Best regards

Yes, SE and Server 2022 are both supported for AutoSPInstaller.
Note though that https://autospinstaller.com has not been updated yet to support building XML configs for the most recent changes, but the scripts themselves will work on the latest version/OS.


Remove Marionette dependency

There's not really anything Marionette-specific in here. I'd just use a good ol' JavaScript function for your 'Route' object, then mix in the Backbone goodies like extend and Events.
If, in your own applications, you use Marionette features like triggerMethod, you could add them to your application by extending the Route class. The benefit to this change is that it makes the library usable by more of the Backbone community.
This issue competes with https://github.com/samccone/backbone.blazer/commit/7faf9a7e968d7bc1b6e04c36ff359b4250a1fe6f.

resolved here: https://github.com/Betterment/backbone.blazer/pull/12


[feat] Support 'approve check' chatops command

close #247
Changes

Support 'approve check' chatops to sync approve status
Submitter Checklist

[ ] Docs included if needed
[x] Tests included if needed
[x] Follows the good commit messages standard

/test

/test

/test

/test

/test

/test

/test

/test


manifest.json file listens for compilation issues

The manifest.ts file has added listening, but the contents modified after starting local development are not updated in the compiled dist file
It is hoped that the contents of the manifest.ts file can be modified during the development process to monitor the simultaneous update of the manifest.json file in the dist file after recompilation

@wangxinyugithub We're not listening for files outside of src, but i think also about listening for manifest or other core files.
Are you want to open PR with that change?

Now i have a think.
This template is created for solved issue with working on yours plugins, not the core code, manifest is outside of src.
We must think about it, maybe it's good idee.
Let's wait whats @Jonghakseo write :)

I'll delegate the change detection for local files to the filesystem and apply the changes detected by the plugin to the completion of the build. While I'm at it, I'll also look at this issue.

#265
I was hoping to be able to handle the issue content while doing the above, but it's more complicated than I thought, so I'll cover that separately.
I need to create a new manifest when I change the manifest, which is inconvenient because I have to reimport the changed manifest.ts file, but I think I can do it, just be a bit patient.

[Enhance] Modify to let the plugin do the build completion detection not file system. #265

I was hoping to be able to handle the issue content while doing the above, but it's more complicated than I thought, so I'll cover that separately.
I need to create a new manifest when I change the manifest, which is inconvenient because I have to reimport the changed manifest.ts file, but I think I can do it, just be a bit patient.

As you write, it's not working now, it detect changes in manifest, but after reload the content inside output manifest.json is old

This is a complicated issue. I used dynamic import to re-import the manifest at the end of the build, but the module system recognized it as a cached module already, so the changes to the contents are not reflected.
I should probably replace the manifest file with js (keeping the type as JSDoc) and use require, but that doesn't seem easy since vite already works as an esm. :(

@all-contributors please add @wangxinyugithub  for bug


Add ability to feed multiple input files to nickel export

Add ability to feed multiple input files to nickel export.
#37
Contains changes from #38.

Because of the squash merging, git doesn't pick up that the version update changes have been done already. Could you rebase this PR?

Done -> once we merge this, I will rebase the #29 as well.


cuda error?

UserWarning: A NumPy version >=1.16.5 and <1.23.0 is required for this version of SciPy (detected version 1.24.3)
warnings.warn(f"A NumPy version >={np_minversion} and <{np_maxversion} is required for this version of "
compute FID between two folders
Found 8091 images in the folder /dataset/flickr/images/
FID  :   0%|                                                                                | 0/506 [00:02<?, ?it/s]
Traceback (most recent call last):
File "/home/m11113013/ProjectCode/MasterProject4/model/metric.py", line 11, in
score = calculate_fid(p1, p1)
File "/home/m11113013/ProjectCode/MasterProject4/model/metric.py", line 4, in calculate_fid
return fid.compute_fid(x_dir, y_dir, mode='clean', num_workers=0, batch_size=16)
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 478, in compute_fid
score = compare_folders(fdir1, fdir2, feat_model,
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 269, in compare_folders
np_feats1 = get_folder_features(fdir1, feat_model, num_workers=num_workers,
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 147, in get_folder_features
np_feats = get_files_features(files, model, num_workers=num_workers,
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 119, in get_files_features
l_feats.append(get_batch_features(batch, model, device))
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/fid.py", line 88, in get_batch_features
feat = model(batch.to(device))
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/features.py", line 25, in model_fn
def model_fn(x): return model(x)
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 168, in forward
outputs = self.parallel_apply(replicas, inputs, kwargs)
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 178, in parallel_apply
return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/parallel/parallel_apply.py", line 86, in parallel_apply
output.reraise()
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/torch/_utils.py", line 461, in reraise
raise exception
RuntimeError: Caught RuntimeError in replica 1 on device 1.
Original Traceback (most recent call last):
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/parallel/parallel_apply.py", line 61, in _worker
output = module(*input, **kwargs)
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl
return forward_call(*input, **kwargs)
File "/home/m11113013/miniconda3/envs/pytorch/lib/python3.8/site-packages/cleanfid/inception_torchscript.py", line 54, in forward
features = self.layers.forward(x2, ).view((bs, 2048))
RuntimeError: The following operation failed in the TorchScript interpreter.
Traceback of TorchScript, serialized code (most recent call last):
File "code/torch/torch/nn/modules/container/___torch_mangle_9.py", line 45, in forward
_17 = self.mixed_10
_18 = self.pool2
input0 = (_0).forward(input, )
~~~~~~~~~~~ <--- HERE
input1 = (_1).forward(input0, )
input2 = (_2).forward(input1, )
File "code/torch.py", line 74, in forward
_22 = self.stride
_23 = self.padding
x3 = torch.conv2d(x, _21, None, [_22, _22], [_23, _23], [1, 1], 1)
~~~~~~~~~~~~ <--- HERE
x4 = _20(x3, self.mean, self.var, None, self.beta, False, 0.10000000000000001, 0.001, )
x5 = torch.torch.nn.functional.relu(x4, False, )
Traceback of TorchScript, original code (most recent call last):
File "C:\Users\tkarras\Anaconda3\lib\site-packages\torch\nn\modules\container.py", line 117, in forward
def forward(self, input):
for module in self:
input = module(input)
~~~~~~ <--- HERE
return input
File "c:\p4research\research\tkarras\dnn\gan3support\feature_detectors\inception.py", line 28, in forward
def forward(self, x):
x = torch.nn.functional.conv2d(x, self.weight.to(x.dtype), stride=self.stride, padding=self.padding)
~~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
x = torch.nn.functional.batch_norm(x, running_mean=self.mean, running_var=self.var, bias=self.beta, eps=1e-3)
x = torch.nn.functional.relu(x)
RuntimeError: cuDNN error: CUDNN_STATUS_NOT_INITIALIZED
from cleanfid import fid
def calculate_fid(x_dir, y_dir):
return fid.compute_fid(x_dir, y_dir, mode='clean', num_workers=0, batch_size=16)
if name == "main":
p1 = '/dataset/flickr/images/'
score = calculate_fid(p1, p1)
print(score)
python version: 3.8.16
pytorch version: 1.12.1
cuda version:11.3

Could you provide more details about your system?
It looks like your machine does not have a GPU?


CLOUDSTACK-4374 - As a Developer I want to have HA enabled for routers that are part or a redundant network or VPC

This PR fixes the issue we were facing with crashed redundant routers not being picked up by the HA monitor due to an IF statement in the code.

Redundant Routers are not the same as Haigh Available. Having a router HA is actually saying that the given router will be controlled by the High Availability monitor. Hence fix any problem we might face.

Ping @remibergsma @DaanHoogland @miguelaferreira @borisroman @michaelandersen
One test failed due to a network hiccup. I will run only that test again and put the results here later tonight.
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : EXCEPTION ===
ERROR

Environment

1 KVM host on CentOS 7.1
Management Server on CentOS 7.1
Agent + Common RPMs built from source

Tests executed

nosetests --with-marvin --marvin-config=/data/shared/marvin/mct-zone1-kvm1-ISOLATED.cfg -s -a tags=advanced,required_hardware=true smoke/test_vpc_redundant.py smoke/test_routers_iptables_default_policy.py smoke/test_routers_network_ops.py smoke/test_vpc_router_nics.py component/test_password_server.py smoke/test_router_dhcphosts.py smoke/test_loadbalance.py smoke/test_internal_lb.py smoke/test_ssvm.py smoke/test_vpc_vpn.py smoke/test_network.py

Results

Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_isolate_network_FW_PF_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_isolate_network_FW_PF_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_03_RVR_Network_check_router_state | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check the password file in the Router VM ... === TestName: test_isolate_network_password_server | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === TestName: test_router_dhcphosts | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : EXCEPTION ===
ERROR
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok
Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test for port forwarding on source NAT ... === TestName: test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: test_02_port_fwd_on_non_src_nat | Status : SUCCESS ===
ok
Test for reboot router ... === TestName: test_reboot_router | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_1_static_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_2_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 35 tests in 22944.728s

FAILED (errors=1)
Waiting for data... (interrupt to abort)

This where and why the vpc vr refactor started, isn't it? code lgtm, let's have a regression test and put it in!

Ping @DaanHoogland @remibergsma @miguelaferreira @borisroman @davidamorimfaria
Tests have been executed again and all looks fine!
[root@cs1 integration]# less /tmp//MarvinLogs/test_ssvm_AOL76R/results.txt
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 8 tests in 1901.959s

OK
/tmp//MarvinLogs/test_ssvm_AOL76R/results.txt (END)

I have started the integration tests.

LGTM :+1:
Ran tests on Ubuntu 14.04
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_isolate_network_FW_PF_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_isolate_network_FW_PF_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_03_RVR_Network_check_router_state | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check the password file in the Router VM ... === TestName: test_isolate_network_password_server | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === TestName: test_router_dhcphosts | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok
Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
Test for port forwarding on source NAT ... === TestName: test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: test_02_port_fwd_on_non_src_nat | Status : SUCCESS ===
ok
Test for reboot router ... === TestName: test_reboot_router | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_1_static_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_2_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 35 tests in 18635.596s

LGTM based on these tests:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true \
component/test_password_server.py \
smoke/test_vpc_redundant.py \
smoke/test_routers_iptables_default_policy.py \
smoke/test_routers_network_ops.py \
smoke/test_vpc_router_nics.py \
smoke/test_router_dhcphosts.py \
smoke/test_loadbalance.py \
smoke/test_internal_lb.py \
smoke/test_ssvm.py \
smoke/test_vpc_vpn.py \
smoke/test_privategw_acl.py \
smoke/test_network.py

Result:
Check the password file in the Router VM ... === TestName: test_isolate_network_password_server | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network and check default routes ... === TestName: test_02_redundant_VPC_default_routes | Status : SUCCESS ===
ok
Create a redundant VPC with two networks with two VMs in each network ... === TestName: test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policy on RouterVM ... === TestName: test_02_routervm_iptables_policies | Status : SUCCESS ===
ok
Test iptables default INPUT/FORWARD policies on VPC router ... === TestName: test_01_single_VPC_iptables_policies | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_isolate_network_FW_PF_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_isolate_network_FW_PF_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false | Status : SUCCESS ===
ok
Test redundant router internals ... === TestName: test_03_RVR_Network_check_router_state | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test nics after destroy ... === TestName: test_01_VPC_nics_after_destroy | Status : SUCCESS ===
ok
Create a VPC with two networks with one VM in each network and test default routes ... === TestName: test_02_VPC_default_routes | Status : SUCCESS ===
ok
Check that the /etc/dhcphosts.txt doesn't contain duplicate IPs ... === TestName: test_router_dhcphosts | Status : SUCCESS ===
ok
Test to create Load balancing rule with source NAT ... === TestName: test_01_create_lb_rule_src_nat | Status : SUCCESS ===
ok
Test to create Load balancing rule with non source NAT ... === TestName: test_02_create_lb_rule_non_nat | Status : SUCCESS ===
ok
Test for assign & removing load balancing rule ... === TestName: test_assign_and_removal_lb | Status : SUCCESS ===
ok
Test to verify access to loadbalancer haproxy admin stats page ... === TestName: test02_internallb_haproxy_stats_on_all_interfaces | Status : SUCCESS ===
ok
Test create, assign, remove of an Internal LB with roundrobin http traffic to 3 vm's ... === TestName: test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80 | Status : SUCCESS ===
ok
Test SSVM Internals ... === TestName: test_03_ssvm_internals | Status : SUCCESS ===
ok
Test CPVM Internals ... === TestName: test_04_cpvm_internals | Status : SUCCESS ===
ok
Test stop SSVM ... === TestName: test_05_stop_ssvm | Status : SUCCESS ===
ok
Test stop CPVM ... === TestName: test_06_stop_cpvm | Status : SUCCESS ===
ok
Test reboot SSVM ... === TestName: test_07_reboot_ssvm | Status : SUCCESS ===
ok
Test reboot CPVM ... === TestName: test_08_reboot_cpvm | Status : SUCCESS ===
ok
Test destroy SSVM ... === TestName: test_09_destroy_ssvm | Status : SUCCESS ===
ok
Test destroy CPVM ... === TestName: test_10_destroy_cpvm | Status : SUCCESS ===
ok
Test Remote Access VPN in VPC ... === TestName: test_vpc_remote_access_vpn | Status : SUCCESS ===
ok
Test VPN in VPC ... === TestName: test_vpc_site2site_vpn | Status : SUCCESS ===
ok
test_01_vpc_privategw_acl (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_01_vpc_privategw_acl | Status : SUCCESS ===
ok
test_02_vpc_privategw_static_routes (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_02_vpc_privategw_static_routes | Status : SUCCESS ===
ok
test_03_rvpc_privategw_static_routes (integration.smoke.test_privategw_acl.TestPrivateGwACL) ... === TestName: test_03_rvpc_privategw_static_routes | Status : SUCCESS ===
ok
Test for port forwarding on source NAT ... === TestName: test_01_port_fwd_on_src_nat | Status : SUCCESS ===
ok
Test for port forwarding on non source NAT ... === TestName: test_02_port_fwd_on_non_src_nat | Status : SUCCESS ===
ok
Test for reboot router ... === TestName: test_reboot_router | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_1_static_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_2_nat_rule | Status : SUCCESS ===
ok
Test for Router rules for network rules on acquired public IP ... === TestName: test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 38 tests in 21378.096s

OK

And:
nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=false \
smoke/test_routers.py \
smoke/test_network_acl.py \
smoke/test_reset_vm_on_reboot.py \
smoke/test_vm_life_cycle.py \
smoke/test_service_offerings.py \
smoke/test_network.py \
component/test_vpc_offerings.py \
component/test_vpc_routers.py

Result:
Test router internal advanced zone ... === TestName: test_02_router_internal_adv | Status : SUCCESS ===
ok
Test restart network ... === TestName: test_03_restart_network_cleanup | Status : SUCCESS ===
ok
Test router basic setup ... === TestName: test_05_router_basic | Status : SUCCESS ===
ok
Test router advanced setup ... === TestName: test_06_router_advanced | Status : SUCCESS ===
ok
Test stop router ... === TestName: test_07_stop_router | Status : SUCCESS ===
ok
Test start router ... === TestName: test_08_start_router | Status : SUCCESS ===
ok
Test reboot router ... === TestName: test_09_reboot_router | Status : SUCCESS ===
ok
Test reset virtual machine on reboot ... === TestName: test_01_reset_vm_on_reboot | Status : SUCCESS ===
ok
Test advanced zone virtual router ... === TestName: test_advZoneVirtualRouter | Status : SUCCESS ===
ok
Test Deploy Virtual Machine ... === TestName: test_deploy_vm | Status : SUCCESS ===
ok
Test Multiple Deploy Virtual Machine ... === TestName: test_deploy_vm_multiple | Status : SUCCESS ===
ok
Test Stop Virtual Machine ... === TestName: test_01_stop_vm | Status : SUCCESS ===
ok
Test Start Virtual Machine ... === TestName: test_02_start_vm | Status : SUCCESS ===
ok
Test Reboot Virtual Machine ... === TestName: test_03_reboot_vm | Status : SUCCESS ===
ok
Test destroy Virtual Machine ... === TestName: test_06_destroy_vm | Status : SUCCESS ===
ok
Test recover Virtual Machine ... === TestName: test_07_restore_vm | Status : SUCCESS ===
ok
Test migrate VM ... === TestName: test_08_migrate_vm | Status : SUCCESS ===
ok
Test destroy(expunge) Virtual Machine ... === TestName: test_09_expunge_vm | Status : SUCCESS ===
ok
Test to create service offering ... === TestName: test_01_create_service_offering | Status : SUCCESS ===
ok
Test to update existing service offering ... === TestName: test_02_edit_service_offering | Status : SUCCESS ===
ok
Test to delete service offering ... === TestName: test_03_delete_service_offering | Status : SUCCESS ===
ok
Test for delete account ... === TestName: test_delete_account | Status : SUCCESS ===
ok
Test for Associate/Disassociate public IP address for admin account ... === TestName: test_public_ip_admin_account | Status : SUCCESS ===
ok
Test for Associate/Disassociate public IP address for user account ... === TestName: test_public_ip_user_account | Status : SUCCESS ===
ok
Test for release public IP address ... === TestName: test_releaseIP | Status : SUCCESS ===
ok
Test create VPC offering ... === TestName: test_01_create_vpc_offering | Status : SUCCESS ===
ok
Test VPC offering without load balancing service ... === TestName: test_03_vpc_off_without_lb | Status : SUCCESS ===
ok
Test VPC offering without static NAT service ... === TestName: test_04_vpc_off_without_static_nat | Status : SUCCESS ===
ok
Test VPC offering without port forwarding service ... === TestName: test_05_vpc_off_without_pf | Status : SUCCESS ===
ok
Test VPC offering with invalid services ... === TestName: test_06_vpc_off_invalid_services | Status : SUCCESS ===
ok
Test update VPC offering ... === TestName: test_07_update_vpc_off | Status : SUCCESS ===
ok
Test list VPC offering ... === TestName: test_08_list_vpc_off | Status : SUCCESS ===
ok
test_09_create_redundant_vpc_offering (integration.component.test_vpc_offerings.TestVPCOffering) ... === TestName: test_09_create_redundant_vpc_offering | Status : SUCCESS ===
ok
Test start/stop of router after addition of one guest network ... === TestName: test_01_start_stop_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test reboot of router after addition of one guest network ... === TestName: test_02_reboot_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to change service offering of router after addition of one guest network ... === TestName: test_04_chg_srv_off_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test destroy of router after addition of one guest network ... === TestName: test_05_destroy_router_after_addition_of_one_guest_network | Status : SUCCESS ===
ok
Test to stop and start router after creation of VPC ... === TestName: test_01_stop_start_router_after_creating_vpc | Status : SUCCESS ===
ok
Test to reboot the router after creating a VPC ... === TestName: test_02_reboot_router_after_creating_vpc | Status : SUCCESS ===
ok
Tests to change service offering of the Router after ... === TestName: test_04_change_service_offerring_vpc | Status : SUCCESS ===
ok
Test to destroy the router after creating a VPC ... === TestName: test_05_destroy_router_after_creating_vpc | Status : SUCCESS ===
ok

----------------------------------------------------------------------
Ran 41 tests in 8923.743s

OK

After all these tests, all routers created had the HA bit set:
MariaDB [cloud]> select name,ha_enabled from vm_instance where name like 'r-%';
+----------+------------+
| name     | ha_enabled |
+----------+------------+
| r-4-VM   |          1 |
| r-6-VM   |          1 |
| r-7-VM   |          1 |
| r-12-VM  |          1 |
| r-13-VM  |          1 |
| r-18-VM  |          1 |
| r-19-VM  |          1 |
| r-23-VM  |          1 |
| r-24-VM  |          1 |
| r-28-VM  |          1 |
| r-30-VM  |          1 |
| r-31-VM  |          1 |
| r-32-VM  |          1 |
| r-34-VM  |          1 |
| r-35-VM  |          1 |
| r-37-VM  |          1 |
| r-38-VM  |          1 |
| r-40-VM  |          1 |
| r-44-VM  |          1 |
| r-45-VM  |          1 |
| r-49-VM  |          1 |
| r-53-VM  |          1 |
| r-56-VM  |          1 |
| r-59-VM  |          1 |
| r-67-VM  |          1 |
| r-69-VM  |          1 |
| r-70-VM  |          1 |
| r-73-VM  |          1 |
| r-74-VM  |          1 |
| r-75-VM  |          1 |
| r-78-VM  |          1 |
| r-79-VM  |          1 |
| r-80-VM  |          1 |
| r-81-VM  |          1 |
| r-85-VM  |          1 |
| r-87-VM  |          1 |
| r-89-VM  |          1 |
| r-91-VM  |          1 |
| r-92-VM  |          1 |
| r-94-VM  |          1 |
| r-96-VM  |          1 |
| r-98-VM  |          1 |
| r-101-VM |          1 |
| r-106-VM |          1 |
| r-108-VM |          1 |
| r-110-VM |          1 |
| r-111-VM |          1 |
| r-113-VM |          1 |
| r-115-VM |          1 |
| r-117-VM |          1 |
| r-118-VM |          1 |
| r-119-VM |          1 |
| r-120-VM |          1 |
| r-124-VM |          1 |
+----------+------------+
54 rows in set (0.00 sec)

Awesome!


fix: cmyk calculations - incorrect blue value calculations

Small fix for CMYK value calculations: in clue color calculations, magenta value was used instead of black
Proper algorithm:

Code change checklist

[x] I have ensured that all methods and functions are fully documented using doxygen style comments.
[x] My code follows the coding style guide.
[x] I tested that my change works before raising the PR.
[x] I have ensured that I did not break any existing API calls.
[x] I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


/modules/prism_statehandler.php calls to a commented out function

I noticed that line 331 in /modules/prism_statehandler.php calls to an undefined function whenever a player presses Shift+I in-game, causing PRISM to restart.

Alright, I'll fix in the next nightly. So, that should be about 20 minutes from now.

Do you mean, https://github.com/Dygear/PRISM/blob/master/modules/prism_statehandler.php#L339
Your plugin should have function called onButtonRequest, if it does not, then the if method_exists function should save PRISM from crashing when players use Shift + I. I'll see if I can reproduce, but I have to fix my router first that just shat the bed.

CTRL + F this line: "ButtonManager::clearButtonsForConn($BFN->UCID);" in prism_statehandler.php.
Sorry for providing the wrong line :)

Fixed in 4269f812cd.


add support for --token-file

To be able to access the k3s cluster remotly, the bosh release should map the --token-file k3s server arg.

https://github.com/cloudfoundry-incubator/kubo-release/blob/master/jobs/kube-apiserver/templates/config/tokens.csv.erb
https://kubernetes.io/docs/reference/access-authn-authz/authentication/#static-token-file

https://github.com/cloudfoundry-incubator/kubo-release/blob/master/jobs/kube-apiserver/templates/config/tokens.csv.erb
https://kubernetes.io/docs/reference/access-authn-authz/authentication/#static-token-file


Updade contribute-pictograms, text edits

Remove "(coming soon)" from the Contribution process section, as per issue: https://github.ibm.com/brand/bxd/issues/80

@janchild please review. thanks!

I am not sure why vercel is not uploading the preview. Can you try to push again?

@laurenmrice the branch had a conflict with the master, it should be ok now. Thanks!


Fully out of place sparse Jacobians

At the moment, jacobian for AutoSparse calls jacobian!, which makes it incompatible with StaticArrays. But then again is it really interesting to have a "sparse static matrix"?

Done


Add procedure in documentation on how to change IP to peers

Documentation lack of clear info on how to migrate a peer on a new IP, could you add the necessary steps?
Or in alternative add a new command in horcrux binary to quickly perform this operation?
Thanks.

To change the DNS/IP of a cosigner:

update config files on each cosigner
bring all cosigners down
remove the .horcrux/raft directory on all cosigners
restart all cosigners

https://github.com/strangelove-ventures/horcrux/pull/213


Update testcoverage build dependencies

Current rust nightly seems to be based on LLVM 18:
$ rustc +nightly --version --verbose
rustc 1.79.0-nightly (aa1c45908 2024-04-06)
binary: rustc
commit-hash: aa1c45908df252a5b0c14e1bcb38c6c55ae02efe
commit-date: 2024-04-06
host: x86_64-unknown-linux-gnu
release: 1.79.0-nightly
LLVM version: 18.1.2

coverage: 59.812% (-0.02%) from 59.834%
when pulling 5807d872a368195385f878e7c64a25a8b6a38fb1 on dennisschagt:update-testcoverage-build-dependencies
into c990101d486a69f2666a2928a554e23fec9419d9 on newsboat:master.


FA - update to 4.4

http://fontawesome.io/whats-new/

Yes please. This is the only reason I'm not using fontello to bundle my current icons.

+10000

4.5 is now out

versioning in general would be an absolute killer-feature!

Yep, 4.5.

Yep, 4.5.

Is there an ETA on this? The old version of the Google+ logo is haunting me :ghost:

+1 using lot of new icons here =D
TY for fontello.

Yesterday 4.6 was released leaving it even more outdates.
I hadn't noticed that you can upload your own font, but  today I tested it for up to date Google icons, works good enough while we wait for an update.

Aha! Hadn't thought of that :thought_balloon: @ineentho - nice workaround

Title should now read update to 4.6 due to the new Instagram logo.

4.6 would indeed be great. Maybe an issue need to be added to fontello/awesome-uni.font: transformed Font Awesome, since this is the Git submodule Fontello is importing internally. Or instructions on how to update that submodule to future Font Awesome versions, so that the community can help and push a PR...

@puzrin Is this you would accept a PR for? Or something you prefer to update yourself, usually? Thx.

If anyone has time for that, any help will be appreciated. Repo is here https://github.com/fontello/awesome-uni.font, see details in makefile.
Actually, PR needs 2 files: updated original svg font & config.yml. Of cause, in buildable state.

@puzrin sounds good. Would you mind considering accepting or rejecting @mareksuscak's fontello/awesome-uni.font#5 before I start -- I work on a mac, and it seems it is fixing something to the Mac build (unclear). Thanks.

@sebastienbarre i think no - it includes unnecessary changes in binary files.

single glyph fix can be included to your update
makefile changes are useful, but let make it saparate
fontforge can be replaced with svg2ttf and install instructions will not be needed anymore.

I short: there are some useful things to extract, but PR needs cleanup.

Yup i fixed the makefile.

@mareksuscak thanks! I can accept PR immediately if you could remove autobuilded files from the first commit.

Cool i can do that but right now im away from computer so will do that later, hopefully today.

https://github.com/fontello/awesome-uni.font/pull/6

What's the current status on this? I noticed in font awesome some icon names are also changed (e.g. "fa-clock" is now "fa-clock-o") and I'd love to get the latest version just for this.
Thank you very much, keep up the good work! :+1: :smile:

I hadn't noticed that you can upload your own font, but today I tested it for up to date Google icons, works good enough while we wait for an update.

@ineentho Unfortunately this doesn't work very well, because you can only upload SVG fonts and the glyph names are missing, so you would have to enter all glyph names one-by-one! :disappointed:

@digeomel Yea, but I only needed a few icons so that worked for me. Of course I'd rather see the font updated instead of doing manual work.

Hopefully I can start working on this over the weekend...

@puzrin Just a quick update; I'm in the process of writing the PR to update to Font Awesome 4.6.3. The build process in awesome-uni.font seems broken for OSX, though I suspect it's broken on Unix too:
awesome-uni.font (fa4.6.3)$ awesome-uni.font/node_modules/.bin/svgo -f /var/folders/9g/_h11phvx4vv0h85qt0swr4xh0000gn/T/fstools.d02c0e --config awesome-uni.font/node_modules/svg-font-create/svgo.yml
awesome-uni.font/node_modules/svg-font-create/svgo.yml:1
(function (exports, require, module, __filename, __dirname) { # replace default config
                                                              ^
SyntaxError: Unexpected token ILLEGAL

This is a problem with SVGO, which has been reported here and here. It is a tad unclear where the fix can be found, the latter issue mentions 0.5, but it seems fixed inh 0.4 too. For the sake of getting this done quickly, I'll update the dependency to SVGO to the latest 0.6.6 in my PR.

@sebastienbarre remove it at all.

I'm not sure how to partially revert a commit that way.
The "only" files that were modified are:
config.yml
DEVELOPMENT.md
package.json
src/original/fontawesome-webfont.svg

and src/svg/ajust.svg which was deleted, because it was misspelled (should be adjust.svg)

Updated to 4.6.3, big thanks to @sebastienbarre for the help.


Remove dead code from background activity service

Goal
Removes some dead code from the BackgroundActivityService that is only called from tests.

Searching 5.x, it seems this was only ever called from tests.

Searching 5.x, it seems this was only ever called from tests.

Adding specific public functions just for testing is the absolute worse......


Put #[cfg] in backticks.

The bare [cfg] makes rustdoc nightly complain.

With the current nightly it won't even document the crate, could this be merged?

alex will be back at the end of the month, this will have to wait until then, sorry for the inconvenience

@goffrie thank you for the PR. The other PRs also bumps the version and it will be quicker to merge that one and just do a new release afterwards. Would you be willing to close this PR so that there is only one about this issue open? (might help speed things up once alex is back, his TODO list will be... pretty large...).

For anyone who want the create docs in the meantime cargo doc --no-deps will document only your own crate.


Add support for Electron versions from 15 to 17

Changes

Changed "electron" peer dependency version from ^14.0.0 to >=14.0.0 <18.0.0 in @itwin/core-backend and @itwin/certa packages. This allows users to choose between several major Electron versions.
Increased "electron" dev dependency to ^17.0.0 to use latest supported electron version in development.

Considerations in supporting multiple major versions of Electron
Electron breaking changes
Electron breaking changes list.
From what I can tell non of the breaking changes on the list break our API or behavior.
Node versions
Electron supported Node versions per Electron version:

Electron version
Node version

14.0.0
14.17

15.0.0
16.5

16.0.0
16.9

17.0.0
16.13

Since we support >=14.17 and >=16.13 Node versions, this should be fine (unless Electron requires exact Node version ?).
Testing
Changes were tested with Electron versions 14.2.9, 15.5.7 and 17.4.11. full-stack-test tests with electron as a dependency pass and appui, display and ui test apps seem to work as intended.

This PR looks good to me. Any reason it's still sitting in a draft state?

There is a problem with Certa tool not working properly with debugging on Electron 19. Working on that right now.

@calebmshafer, @aruniverse
Support for Electron 18 and 19 is still blocked by AsyncLocalStorage problem. It seems to be a V8 bug (Issue #35043). My recommendation is to add support for Electron versions 15, 16 and 17 for now. Once the V8 fix is available, we should revisit Electron support once again.

@calebmshafer, @aruniverse
Support for Electron 18 and 19 is still blocked by AsyncLocalStorage problem. It seems to be a V8 bug (Issue #35043). My recommendation is to add support for Electron versions 15, 16 and 17 for now. Once the V8 fix is available, we should revisit Electron support once again.

@GytisCepk That sounds good for now. Can you add a note in the NextVersion.md to explain it?

Taking display-test-app for a spin (node 16.16.0).

I am enqueuing an ImageTests run against this branch. Please do not complete this PR until the results are analyzed.

display-test-app appears to be working normally and ImageTests reports no problems.

@calebmshafer @wgoehrig this is awaiting your review.


测试PR

[This message is created by hyperf-bot]
Hi @limingxinleo, this is a READ-ONLY repository, please submit your Pull Request to hyperf/hyperf repository, this Pull Request will close automatically.


Memory overhead of sourcemapLocations is high

We were profiling the memory use of our rollup builds, and a surprising amount is due to sourcemapLocations. As far as we can tell, each AST node ends up adding two locations (https://github.com/rollup/rollup/blob/121a7f41fb88a294d9680c3fa11f4f72ebf9e9ff/src/ast/nodes/shared/Node.ts#L113-L114). Here's a screenshot from heap dump from the rollup process:

sourcemapLocations is an object being used as a set, if it's changed to a real Set it becomes ~4x smaller (guessing this is because it avoids the implicit stringification of object keys):

@Rich-Harris is this something you're open to? If so, happy to send a PR.

I am not a contributor to magic-string, but I believe that if your change doesn't alter how magic-string works and all the tests passes, providing at the same time less memory overhead, surely your PR is welcome!

@mihaip A bit set ought to use considerably less memory than a set. Could you profile this patch?
diff --git a/src/MagicString.js b/src/MagicString.js
index 18bf1f4..e138492 100644
--- a/src/MagicString.js
+++ b/src/MagicString.js
@@ -16,4 +16,21 @@ const warned = {
 };

+const BitSet_BITS = 32;
+
+class BitSet {
+	constructor(arg) {
+		this._bits = arg instanceof BitSet ? arg._bits.slice() : [];
+	}
+	clone() {
+		return new BitSet(this);
+	}
+	set(n) {
+		if (n >= 0) this._bits[Math.floor(n / BitSet_BITS)] |= 1 << (n % BitSet_BITS);
+	}
+	get(n) {
+		return !!(this._bits[Math.floor(n / BitSet_BITS)] & (1 << (n % BitSet_BITS)));
+	}
+}
+
 export default class MagicString {
 	constructor(string, options = {}) {
@@ -31,5 +48,5 @@ export default class MagicString {
 			filename:              { writable: true, value: options.filename },
 			indentExclusionRanges: { writable: true, value: options.indentExclusionRanges },
-			sourcemapLocations:    { writable: true, value: {} },
+			sourcemapLocations:    { writable: true, value: new BitSet },
 			storedNames:           { writable: true, value: {} },
 			indentStr:             { writable: true, value: guessIndent(string) }
@@ -44,6 +61,6 @@ export default class MagicString {
 	}

-	addSourcemapLocation(char) {
-		this.sourcemapLocations[char] = true;
+	addSourcemapLocation(pos) {
+		this.sourcemapLocations.set(pos);
 	}

@@ -122,7 +139,5 @@ export default class MagicString {
 		}

-		Object.keys(this.sourcemapLocations).forEach(loc => {
-			cloned.sourcemapLocations[loc] = true;
-		});
+		cloned.sourcemapLocations = this.sourcemapLocations.clone();

 		cloned.intro = this.intro;
diff --git a/src/utils/Mappings.js b/src/utils/Mappings.js
index 6026200..8bf3d21 100644
--- a/src/utils/Mappings.js
+++ b/src/utils/Mappings.js
@@ -29,5 +29,5 @@ export default class Mappings {

 		while (originalCharIndex < chunk.end) {
-			if (this.hires || first || sourcemapLocations[originalCharIndex]) {
+			if (this.hires || first || sourcemapLocations.get(originalCharIndex)) {
 				this.rawSegments.push([this.generatedCodeColumn, sourceIndex, loc.line, loc.column]);
 			}

Thanks @kzc, that is a lot letter. Opened a PR at #169


/waiter-async endpoints should not propagate authorization header to backend

They currently pass along whatever authorization header was provided by the client. This was discovered when some of our tests attempted to pass the authorization header on /waiter-async routes, and the header was forwarded to kitchen, which then failed to authenticate because it expects the WAITER_USERNAME and WAITER_PASSWORD it was configured with.

This was resolved by the choice of removing hop-by-hop headers.


[Search] Add multi-vector search support

Packages impacted by this PR
@azure/search-documents
Issues associated with this PR
#26531
Describe the problem that is addressed by this PR
Adds multi-vector support through Swagger regeneration

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-search-documents


[Bug]  I am having trouble making a discord integration

General

[X] I have updated to the latest version of MySpeed.
[X] My bug has not been reported yet.

The Bug
I installed speedtest on my raspberry pi4 using casaos and wanted to get notified for each speedtest on discord, but i cant get past the section that asks for webhook URL. the box just stays red and i cant save.. I dont really know if it is a bug or i am just bad, but would really appreciate some help :D!

What device are you using to access the page?
In the browser
Which operating system is your MySpeed instance running on?
Linux

@lasuaa Can you tell me how you resovled this? I am also facing the same problem on casaOS...


Return only primal when applying non-differentiable methods to Dual

Addresses #167. Opening this as a draft since it would require an entire rewrite of the overloading structure.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 14.28571% with 12 lines in your changes missing coverage. Please review.

Project coverage is 39.01%. Comparing base (8251cb3) to head (0ff7b71).

Files
Patch %
Lines

src/overloads/conversion.jl
22.22%
7 Missing :warning:

src/overloads/gradient_tracer.jl
0.00%
3 Missing :warning:

src/overloads/hessian_tracer.jl
0.00%
2 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (8251cb3) and HEAD (0ff7b71). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (8251cb3)
HEAD (0ff7b71)

4
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #169       +/-   ##
===========================================
- Coverage   91.65%   39.01%   -52.65%
===========================================
  Files          37       36        -1
  Lines        1786     1525      -261
===========================================
- Hits         1637      595     -1042
- Misses        149      930      +781

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


Multiple Ambassador deployments fails with CRDs

Describe the bug
Deploying multiple Ambassador configurations with Helm doesn't seem to work with CRDs.
To Reproduce
Create a new helm chart called ambassador-external with a requirement to stable/ambassador and specify the env variable AMBASSADOR_ID.
Create a new helm chart called ambassador-internal with a requirement to stable/ambassador and specify the env variable AMBASSADOR_ID.
Also set the following in values.yaml:
crds:
    create: false

Deploy the helm chart ambassador-external. This deploy succeeds and creates the CRDs and other Kubernetes resources.
Deploy the helm chart ambassador-internal. This deploy succeeds but generates the following errors in the ambassador POD:
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: looking up ID for namespace default
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: cluster ID URL is d6e_id://262a6283-55e4-11e9-85da-023e14333eae/ambassador-internal
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: CRD type definition not found for modules.getambassador.io
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: Main CRDs are not available. To enable CRD support, configure the Ambassador CRD type definitions and RBAC, then restart the Ambassador pod.
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: Main CRDs are not available. To enable CRD support, configure the Ambassador CRD type definitions and RBAC, then restart the Ambassador pod.
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: Main CRDs are not available. To enable CRD support, configure the Ambassador CRD type definitions and RBAC, then restart the Ambassador pod.
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: Main CRDs are not available. To enable CRD support, configure the Ambassador CRD type definitions and RBAC, then restart the Ambassador pod.
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:18 kubewatch [9 TMainThread] 0.72.0 DEBUG: Main CRDs are not available. To enable CRD support, configure the Ambassador CRD type definitions and RBAC, then restart the Ambassador pod.
ambassador-internal-5588b698cf-dtxtl ambassador 2019-07-12 06:36:19 kubewatch [9 TMainThread] 0.72.0 DEBUG: cluster ID is e013e215-43fd-5daf-93d3-73a32c864846 (from namespace default)

ambassador-internal-5588b698cf-dtxtl ambassador 2019/07/12 06:36:20 aggregator: watch hook stderr: 2019-07-12 06:36:20 watch-hook ERROR: -global-: <RichStatus BAD error=Ambassador could not find core CRD definitions. Please visit https://www.getambassador.io/reference/core/crds/ for more information. You can continue using Ambassador via Kubernetes annotations, any configuration via CRDs will be ignored... hostname=ambassador-internal-5588b698cf-dtxtl version=0.72.0>

Expected behavior
I would expect that when executing helm release for the 2d helm chart that it would succeed without errors and use the existing CRDs which were created by the first helm chart.
Currently, when deploying Mapping objects with an ambassador_id that points to ambassador-internal, this doesn't get picked up by the ambassador POD.
Versions (please complete the following information):

Ambassador: 0.72.0
Kubernetes environment: AWS provisioned with kops
Version: v1.12.9

I hit the same issue, tried applying CRDs again, just got response that CRDs are configured but error didn't go away.

It should by setting
crds:
  create: true
  keep: true
  enabled: true

in the first release. Then using
crds:
  created: false
  keep: true
  enabled: true

In all subsequent releases.
I would love some feedback if that is not the case


10/25/2018 21:40 : Create Pull Request for branch prTesting_2018102521 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon.md
:white_check_mark:Succeeded

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


[BFCL] Reproduction GPT-4o-2024-08-06 (prompt) does not match the official score

Hello, in Berkley -function-call-leaderboard, when I ran GPT-4o-2024-08-06 (prompt), I only got 32.37 points, the official score is 53.66 points, probably because my json has' ' 'json representation, Broken code? Could you please provide the official json result of GPT-4o-2024-08-06 (prompt)

Hi @liuxize-dhu,
Thanks for the issue.
I have attached the result we used to compute the current score on the leaderboard (the rest category result file has been processed to have the API keys redacted). The single-turn results are generated in #603 while the multi-turn results are generated in #646.
I have also dispatched a complete regeneration for GPT-4o-2024-08-06 (prompt), and the model does seem to have a significant performance drop, similar to what you claimed.
Examine the two versions' result files for the simple category (1st image is the version we use, 2nd image is the version from regenerate), we can see that the later version contains many more entries that have formatting issues, which would cause decoder failure.
For example, for simple_22, the later version has:
```json\n[{\"name\": \"math.gcd\", \"parameters\": {\"num1\": 12, \"num2\": 15}}]\n```

This is wrong, as it is not the format we asked for in the system prompt; the model is not following instructions.
Below is the expected output format.
[math.gcd(num1=12, num2=15)]

I will do a re-generate for all the single_turn data, and update the leaderboard with the most up-to-date score shortly.
gpt-4o-2024-08-06_result.zip

Hi I had tried quite a number of models versus the gpt-4o-mini as a benchmark on langgraph. However, the result of the output was quite mediocre at best compared to even hermes-3-llama-3-1-8b-tools? I am not doubting the benchmarks but I am puzzled why it does not even do a tool call. While the results from openai and anthropic are similar and good,  those from llama, mistral, qwen and others have varying degree of unwanted result, none that I had experimented come close to even the gpt-4o-mini.  Unfortunately, gorilla-llm/gorilla-openfunctions-v2-gguf possibly gave one of the worst results.  I am really puzzle to the Whys and How it can be make better? Hope you can point me in the right direction.

Hi I had tried quite a number of models versus the gpt-4o-mini as a benchmark on langgraph. However, the result of the output was quite mediocre at best compared to even hermes-3-llama-3-1-8b-tools? I am not doubting the benchmarks but I am puzzled why it does not even do a tool call. While the results from openai and anthropic are similar and good, those from llama, mistral, qwen and others have varying degree of unwanted result, none that I had experimented come close to even the gpt-4o-mini. Unfortunately, gorilla-llm/gorilla-openfunctions-v2-gguf possibly gave one of the worst results. I am really puzzled to the Whys and How it can be made better? Hope you can point me in the right direction.

Would you mind open a separate discussion for this?


Make root_files configurable

Language server
No response
Requested feature
It'd be nice if we can configure the root_files of LSP servers via .setup():
https://github.com/neovim/nvim-lspconfig/blob/f3195835c0447ee2c80152b893ab51ca162b04a9/lua/lspconfig/server_configurations/lua_ls.lua#L3-L11
It might be a tricky to implement though, since we need to make sure that the root_dir function refers to the same table when it was defined. Any ideas?
Other clients which have this feature
No response

We encourage users to gradually use vim.lsp.start. although it may be necessary to manually handle the trouble of starting, re-using and stopping the server now. But we will add more support in the core. lspconfig is currently only a repo collected as server configuration. More features and suggestions may not be considered. Sorry.

Welp, tried to make a PR to improve the root detection for lua_ls and it's also rejected: https://github.com/neovim/nvim-lspconfig/pull/2820
I am not sure what a user should even do here. Are they expected to copy-paste the entire file to their config just to add 10 chars? If lspconfig is just a "a repo collected as server configuration", then why not remove .setup() altogether and just make everything static and unconfigurable?
This overall situation has been really frustrating. Apparently users are expected to use vim.lsp.start, which makes this whole thing even more confusing. Is lspconfig going to be deprecated?
Because the current feature parity between lspconfig and vim.lsp.start is not even close, and I don't really see any signs that it would change. No client re-use, no status menu, no stop/start/restart interface.
And let's say that we now have feature parity. What will happen to lspconfig then? Is every user now expected to write hundreds or even thousands of lines inside their local config just to set up every single LSP client from a scratch?
I just don't see a clear path forward here.

You can define your own root_files in the root_dir function. lspconfig is not deprecated. Because it can export the repo as server configuration. Pass it to vim.lsp.start. In the future we hope to have the unified vim.lsp interface take over. For example vim.lsp.config vim.lsp.start, etc.


Add /help Command to magic_commands.py

The magic_commands.py script lacks a /help command that would list all available commands for the user. This enhancement would improve user experience by providing an easy way to view all commands. The /help command should list all commands with a brief description of their functionality.

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

Users currently have no way of knowing all the available commands in the magic_commands.py script without looking at the source code or documentation.
Describe the solution you'd like

I would like a /help command to be added to the magic_commands.py script. This command should list all available commands along with a brief description of their functionality.
Describe alternatives you've considered

An alternative could be to include a comment at the top of the magic_commands.py script that lists all the commands. However, this would not be as user-friendly as a /help command.
Additional context

This enhancement would improve the user experience by providing an easy way to view all commands without needing to refer to the documentation or source code.

This would be a good addition

I'll get on this as soon as i complete it since its too easy. Beside this, I've got some amazing commands and features which we can integrate like a text-to-speech module, etc. I will create the issues soon!

That's great


Send topic updates after claiming partition 0 immediately

This is done for performance since claiming topics can take some time.

Got it! Thank you!


As a user I want to be able to search into the json string

Search through the json as a regular string
Search for specific child nodes in a xpath like notation

54c478b2417172aeca7639d8e66234015cd8adc4