Load balancer with availability set

If I don't have a load balance pointing an availability set of two VMs, how do I access 2nd VM in case of 1st VM went down?
Could you please explain or refer some doc?

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

ID: 948b29e3-d488-0a0e-89c7-42849a93522d
Version Independent ID: 8c986086-9343-4b99-ae53-f4fd1159d4f8
Content: Manage the availability of Windows VMs in Azure
Content Source: articles/virtual-machines/windows/manage-availability.md
Service: virtual-machines-windows
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@ramadarla Lets assume that you have an availability set with 2 VM's. If one of the VM is down, traffic from the Load balancer will be forwarded to the healthy VM.
You can access the unhealthy VM in the same way we are accessing other VM's.
With availability sets, your vms will be placed in different update and fault domains..
If this didn't answer your question, Please elaborate you issue a bit.

Thank you for the response Jakaruna.
Understood. So that confirms using Load Balancer is mandatory to access VMs behind an Availability set.

From: jakaruna-MSFT<EMAIL_ADDRESS>Sent: 19 December 2018 20:04
To: MicrosoftDocs/azure-docs
Cc: ramadarla; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Load balancer with availability set (#20986)
@ramadarlahttps://github.com/ramadarla Lets assume that you have an availability set with 2 VM's. If one of the VM is down, traffic from the Load balancer will be forwarded to the healthy VM.
You can access the unhealthy VM in the same way we are accessing other VM's.
With availability sets, your vms will be placed in different update and fault domains.https://docs.microsoft.com/en-us/azure/virtual-machines/windows/manage-availability#configure-multiple-virtual-machines-in-an-availability-set-for-redundancy.
If this didn't answer your question, Please elaborate you issue a bit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/20986#issuecomment-448616702, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AelNhZNgEu62GMcH2DXrzpqnln9U6Q0Wks5u6k5_gaJpZM4ZZvoE.

@ramadarla No.  You can login to the Availability set vm directly.
The ip and port to use to login differs based on your environment.
If your environment don't have VPN or Express route, Then

if your vm has  public ip you can access it directly
If your vm don't have a public ip, Then you can enable port forwarding on the load balancer. With port forwarding each VM which is added to it can be accessed with the load balancer ip and a different port for each VM.

If your environment has a VPN or express route

You can access the VM with the private IP itself.
If port forwarding is enabled in load balancer, You can access it with the load balancer ip and random port.

Traffic flow to the VM will work from load balancer irrespective of VPN or Express route. Only requirement is that the VM should have the port open to which the load balancer sends traffic .

@ramadarla Did you tried the suggestions?

Thank you, Apologies for the delay. Bare with me, I am slow in understanding.
Let's say, I have created an availability set placed VM1 and VM2 in it. No VPN or ER, I will access both the vms over internet using Public IPs, both got two different public IPs.
I am running replica of VM1 website on VM2 as well. I use public IP of VM1 to access the this website over internet. Now, How I will access VM2 in case of VM1 is down for some host failure,
-- How the VM2 will switch to serve the same VM1 website with out load balancer
(Since I am using PIP it's a public facing load balancer).


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 405

Questions ?
Please refer to the Backport tool documentation


HOME environment variable not set on Windows machines

Describe the bug
On this page of the documentation, the HOME environment variable is listed as a default variable which should be set "in all runner environments", however it is not when using windows-latest environments.
Area for Triage: JavaScript and Node.js

Question, Bug, or Feature?: Bug

Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Ubuntu 20.04 LTS
[ ] Windows Server 2016 R2: haven't tested with this one
[x] Windows Server 2019

Expected behavior
The HOME environment variable should be set, or the documentation should be edited.
Actual behavior

A description with steps to reproduce the issue. If your have a public example or repo to share,
please provide the link.

In a Javascript action, check the keys of the process.env dictionary.

Hello, thank you for report.
Actually, HOME variable doesn't exist in Windows by default. Instead it has HOMEPATH and USERPROFILE (contains the same value).
We will consider adding variable for consistency or updating documentation

Hello, we wouldn't like to add new variable to Windows environments. We will update documentation to avoid confusing.


When run with device_preview, has errors.

About
A while ago, I opened an issue about this, but I didn't get any feedback.
More details can be seen here.
How could I help in this case? It's a solution that interests me a lot, that I've tried to solve but haven't found ways.

Related to

https://github.com/Codelessly/ResponsiveFramework/issues/122

Thank you for reopening. I definitely want to make sure this library works with DevicePreview as I use that library quite extensively.
In fact, DevicePreview helps me test to make sure ResponsiveFramework is working correctly.
I've been focused on Rewriting this library for the v1 release, which, good news is now released!
I'll make sure to follow up here and update with an example.

sensational, @rayliverified. Thanks for response.
I can also test if it persists in the most current ones. but only in about 7 hour, approximately. :)

@rayliverified yes this error persists.

You can check my branch with current changes (including the migration to latest packages versions).
Here is my PR with migration (see only lib folder).
To see error or suggest anything, is only clone this repository. :)


feat: port Analysis.Convex.Hull

[x] depends on: #3061

This PR/issue depends on:

leanprover-community/mathlib4#3061
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib4#3061
By Dependent Issues (🤖). Happy coding!

bors merge


QUICKSTEP-89 Add support for common subexpression.

This PR adds support for common subexpression elimination in Quickstep. In summary we apply two kinds of optimizations:
(1) Query rewriting for aggregate expressions, e.g.
SELECT SUM(x), SUM(x)*2, AVG(x), COUNT(x) FROM s;

will be rewritten as
SELECT sum_x, sum_x * 2, sum_x / count_x, count_x
FROM (
  SELECT SUM(x) AS sum_x, COUNT(x) AS count_x
) t;

(2) Common subexpression labeling with per-work-order memoization table to cache result column vectors to avoid redundant computation. For example, the evaluation of expressions in
SELECT (x + 1) * (y + 2) * 3, (x + 1) * (y + 2), x + 1
FROM s;

will essentially look like:
t1 = x + 1
t2 = t1 * (y + 2)
Out1 = t2 * 3
Out2 = Reference(t2)
Out3 = Reference(t3)

Here t1, t2, Out1, Out2, Out3 are per-work-order ColumnVector's.
This PR together with a (later) new aggregation hash table implementation will improve the performance of TPC-H Q01 from ~11.5s to ~5.3s, with scale factor 100 on a cloud lab machine.

Note: For optimization (1), note that it is not free-of-cost to "re-project" the columns, so it may not worth doing the transformation in situations where a group-by aggregation has large number of groups. E.g., it may actually slow down the query to rewrite
SELECT SUM(a), SUM(b), SUM(c), SUM(d), SUM(a)
FROM s
GROUP BY x;

as
SELECT sum_a, sum_b, sum_c, sum_d, sum_a
FROM (
  SELECT SUM(a) AS sum_a, SUM(b) AS sum_b, SUM(c) AS sum_c, SUM(d) AS sum_d
  FROM s
  GROUP BY x;
) t;

if the number of distinct values of attribute x is large -- in that case, we avoid one duplicate computation of SUM(a), but introduce 5 extra expensive column copying due to the intermediate relation t.
Thus currently we employ a simple heuristic that if either
(1) The estimated number of groups for the Aggregate node is smaller than the specified FLAGS_reuse_aggregate_group_size_threshold.
or
(2) The ratio of (# of eliminable columns) to (# of original columns) is greater than the specified FLAGS_reuse_aggregate_ratio_threshold.
then we perform the transformation.

@jianqiao This PR has a bug and some issues to fix.

Excellent work @jianqiao. LGTM. Merging.
Side note: One of the jobs timed out, so we should consider removing some of the tests from the type system once you are done with the new type system.

@zuyu -- can you expand on the bugs? The tests all passed (except for the one that timed out).

@pateljm This is a minor bug not caught by the tests. See the comment.

Will fix the issues in one PR shortly after.

I have not done one pass. May have some more comments coming.


caret icon Fn missing in 0.14.1

however the version 0.12.0 contain the caret Fn

Sorry for opening the issue,it's my mistake.


Move test iterator types into separate header file

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
468

Relevant Lines:
468

💛  - Coveralls


[Storage] Fix more flaky lease tests

Fixing another flaky test around lease timing. Giving an extra 2 seconds for lease to expire.

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


[Master] Cannot make editor plugin using tool keyword GDScript

Godot version
4.0.dev 9444a3f01d3a8c4f49ddf24e3acc65b399ee0609
System information
Linux binogure 5.14.0-2-amd64 #1 SMP Debian 5.14.9-2 (2021-10-03) x86_64 GNU/Linux
Issue description
Cannot create in GDScript n editor plugin with the tool keyword
Steps to reproduce

Launch Godot
Create a project
Create a scene and attach a GD script
Paste the following code:

tool
extends Control
[...]

Minimal reproduction project
No response

tool is no longer a keyword, you should use the @tool annotation. See https://docs.godotengine.org/en/latest/tutorials/scripting/gdscript/gdscript_basics.html#annotations

Then there is an issue with the documentation since this recommand to use tool and not @tool:
https://docs.godotengine.org/en/latest/tutorials/plugins/editor/making_plugins.html

The documentation for Godot 4.0 is WIP, this page hasn't been updated yet. You can open an issue about it on godot-docs.

Also, I was wondering, is it possible to just say something like that in Godot:
`tool` is deprecated and has been replaced by `@tool`

Instead of:
Unexpected identifier


Publish on F-Droid

Title, proprietary dependencies would need to be replaced, though.

I will publish the next version


Feature/really really mergeme

Please attach WIP tag when this PR is still work in progress.
Fixes # .
Changes proposed in this pull request:

Checklist:

[ ] Added related tests
[ ] Made corresponding changes to the documentation

@anz-bank/sysl-developers

@springwiz merge this and then make the templating work again.


Verify headers first, to avoid large block denial of service

Motivation
In https://github.com/ZcashFoundation/zebra/pull/1328 we created a first version to handle AdvertiseBlocks messages, which downloads each full block that is advertised to us.
The procedure above exposes us to a DoS risk, because we blindly download an entire block whenever anyone tells us to.  This is in contrast to the sync process, where we generally download blocks from different peers than the ones who told us about them.
This process can also result in a lot of duplicate block notifications, as a block is broadcast across the network.
Initial Design
So as a follow-up, we want to:

deduplicate inbound block requests, and
support header-only verification of inbound blocks.

Alternative Fix
We might want to implement rate-limit inbound messages or inbound data instead.
Summary

[ ] Deduplicate inbound block requests
[ ] Download only the block header
[ ] Pre-verify the proof of work in the block header, using a new HeaderVerifier
[ ] If successful, download and verify the full block

Full blocks are verified by the BlockVerifier.
Gossiped blocks are deduplicated, downloaded header-first, and then pre-verified by the HeaderVerifier. The HeaderVerifier uses the state service's VerifyDifficultyThreshold request for contextual verification (see #1036):
If the header is valid, the full block is downloaded and verified by the BlockVerifier.
Detailed Design
Create an Inbound Downloads service, as described in https://github.com/ZcashFoundation/zebra/issues/1082#issuecomment-729950699.
Split a HeaderVerifier service out of the BlockVerifier service.
Make the difficulty adjustment contextual verification function available via a State service VerifyDifficultyThreshold request. This request should take a block header, and return Ok if:

the parent block has been verified, and
the block header passes the relevant consensus rules.

Once the VerifyDifficultyThreshold request request is implemented, it can be called by the HeaderVerifier.
TODO

[ ] Design RFC
[ ] Implementation
[ ] Tests

Additional context
https://github.com/ZcashFoundation/zebra/issues/1082, particularly the sketch of deduplication in https://github.com/ZcashFoundation/zebra/issues/1082#issuecomment-729950699
https://github.com/ZcashFoundation/zebra/pull/1328

I'm a little confused since this doesn't match my recollection of what we discussed in the meeting -- I thought that the idea was just to allow verification of the proof-of-work in advance of downloading the full block, rather than doing complete verification of the header.  I think that this is sufficient for the goal and considerably simpler.  I'm not sure, for instance, that we should create a CoinbaseVerifier, since that verification shouldn't be done outside of block verification, so separating it from the block::Verifier introduces additional complexity without having any reuse benefit.
I don't think that this needs to be part of the first alpha release milestone -- we don't need it to participate in the network, since we can just download full blocks when they're advertised to us.

I'm a little confused since this doesn't match my recollection of what we discussed in the meeting -- I thought that the idea was just to allow verification of the proof-of-work in advance of downloading the full block, rather than doing complete verification of the header. I think that this is sufficient for the goal and considerably simpler.

I thought it was simpler to split the block verifier, but I think you're right, maybe we just need to split out (parts of) the header verification.
But since we're only doing semantic verification, there isn't actually a huge difference between "header verification" and "proof of work verification". Here's a list of each field in the header that's semantically verified using the BlockVerifier:
nTime - local node time check
nBits - proof of work
nNonce - proof of work
solutionSize - proof of work
solution - proof of work
Since the time check is cheap, we might as well do it when verifying downloaded headers.

I'm not sure, for instance, that we should create a CoinbaseVerifier, since that verification shouldn't be done outside of block verification, so separating it from the block::Verifier introduces additional complexity without having any reuse benefit.

I think you're right about the CoinbaseVerifier.
Do you think we should create a HeaderVerifier, or skip the time check, and create a DifficultyVerifier?

I don't think that this needs to be part of the first alpha release milestone -- we don't need it to participate in the network, since we can just download full blocks when they're advertised to us.

Sounds good.

This still needs to be done for security, but we're unlikely to do it any time soon.


In certain cases Observable could not be cancelled

  implicit val scheduler = Scheduler(ExecutionContext.global)

  val source = Observable(1).flatMap(_ => Observable.interval(1.seconds).map(_ => "tic"))

  val task = source.foreachL(println).runAsync

  scheduler.scheduleOnce(1, SECONDS, { () =>
    println("Cancelling")
    task.cancel()
  })

  Await.result(task, Duration.Inf)

Cancellation for Observable.interval will not be happen. I believe that due following code:
monix.reactive.internal.operators.ConcatMapObservable.ConcatMapSubscriber#signalFinish
    private def signalFinish(ex: Option[Throwable]): Unit =
      stateRef.getAndSet(WaitComplete(ex)) match {
        case WaitOnNextChild(_) =>
          // In this state we know we have no active task,
          // and we know that the last `onNext` was triggered
          // (but its future not necessarily finished)
          // so we can just signal the completion event
          if (ex.isEmpty) sendOnComplete() else out.onError(ex.get)

        case Active(_) | WaitComplete(_) | Cancelled =>
          // We either have an active task, or this is a duplicate
          // event, hence we cannot signal the completion signal here.
          ()

        case WaitOnActiveChild =>
          // Something is screwed up in our state machine :-(
          reportInvalidState(WaitOnActiveChild, "signalFinish")
      }

If I correctly understand, that this fragment is called after Observable(1) completion and current Active(cancellable) will be substituted with WaitComplete(ex), that will make impossible to cancel observable in future

@YulawOne you are right 😒
This was due to a recent change to optimize concatMap and unfortunately this wasn't caught by any tests.
I'll do an emergency release tomorrow.

@YulawOne you pinpointed the problem, that's impressive :-)
Sorry for any inconvenience this may have caused.

@alexandru Thanks and thank you for fast response.

@YulawOne same logic was applied in mapTask and that had to be fixed as well. Also added tests to guard against regressions.
Thanks a lot for the report. I'm postponing the release for tomorrow since I have another issue I want to solve.

@YulawOne sorry for the wait, this was fixed in today's release 2.2.2


Unraid; Will only build from Timed URL

Bug description
When I install this on Docker, it will not build if I use just user/pass.  If I put in the timed URL it will build, but does not store in the Cache folder, so on restart for any reason I have to recreate the timed URL.
Steps to reproduce

Install on Unraid
Put in timed url and user/pass and cache dir in /Data folder
let build
wait for timed url to expire
make no changes but restart docker

Expected behavior
I expect that on a restart it will simply come back up and work. I should not have to have rebuild the docker ever time.
Container metadata
com.foundryvtt.version = "12.331"
org.opencontainers.image.authors =<EMAIL_ADDRESS>org.opencontainers.image.created = "2024-08-16T01:48:32.610Z"
org.opencontainers.image.description = "An easy-to-deploy Dockerized Foundry Virtual Tabletop server."
org.opencontainers.image.licenses = "MIT"
org.opencontainers.image.revision = "61a1ac11d7bd298b93fa6cebb102158ddb1d020e"
org.opencontainers.image.source = "https://github.com/felddy/foundryvtt-docker"
org.opencontainers.image.title = "foundryvtt-docker"
org.opencontainers.image.url = "https://github.com/felddy/foundryvtt-docker"
org.opencontainers.image.vendor = "Geekpad"
org.opencontainers.image.version = "12.331.0"

Relevant log output
Successful setup:

< etag: "a0413e8a134d6db99581d20ec5a8088f-28"

< last-modified: Thu, 15 Aug 2024 15:26:54 GMT

< vary: Accept-Encoding

< cache-control: max-age=14400

< cf-cache-status: MISS

< accept-ranges: bytes

< server: cloudflare

< cf-ray: 8bce3c29ab1b477e-DFW

<

{ [5 bytes data]

 13  221M   13 30.2M    0     0  33.9M      0  0:00:06 --:--:--  0:00:06 33.9M
 37  221M   37 84.1M    0     0  44.5M      0  0:00:04  0:00:01  0:00:03 44.5M
 60  221M   60  134M    0     0  46.7M      0  0:00:04  0:00:02  0:00:02 46.7M
 89  221M   89  198M    0     0  50.8M      0  0:00:04  0:00:03  0:00:01 50.8M
100  221M  100  221M    0     0  52.5M      0  0:00:04  0:00:04 --:--:-- 52.5M
* Connection #0 to host r2.foundryvtt.com left intact
Entrypoint | 2024-09-02 07:36:58 | [info] Installing Foundry Virtual Tabletop
Entrypoint | 2024-09-02 07:36:58 | [debug] Checking mime-type of release file.
Entrypoint | 2024-09-02 07:36:58 | [debug] Found mime-type: application/zip
Entrypoint | 2024-09-02 07:36:58 | [debug] Extracting release file.
Entrypoint | 2024-09-02 07:37:02 | [debug] Installation completed.
Entrypoint | 2024-09-02 07:37:02 | [info] Preserving release archive file in cache.
Entrypoint | 2024-09-02 07:37:02 | [debug] CONTAINER_CACHE_SIZE is not set. Skipping cache cleanup.
Entrypoint | 2024-09-02 07:37:02 | [debug] Patching GUI update and configuration messages.
Entrypoint | 2024-09-02 07:37:02 | [info] Not modifying existing installation license key.
Entrypoint | 2024-09-02 07:37:02 | [info] Setting data directory permissions.
Entrypoint | 2024-09-02 07:37:02 | [debug] Setting ownership of /data to foundry:foundry.
Entrypoint | 2024-09-02 07:37:07 | [debug] Completed setting directory permissions.
Entrypoint | 2024-09-02 07:37:07 | [info] Starting launcher with uid:gid as foundry:foundry.
Entrypoint | 2024-09-02 07:37:07 | [debug] Waiting for child pid: 75 to exit.
Launcher | 2024-09-02 07:37:07 | [debug] Ensuring /data/Config directory exists.
Launcher | 2024-09-02 07:37:07 | [info] Generating options.json file.
Launcher | 2024-09-02 07:37:07 | [info] Setting 'Admin Access Key'.
Launcher | 2024-09-02 07:37:07 | [info] Starting Foundry Virtual Tabletop.
FoundryVTT | 2024-09-02 14:37:08 | [info] Running on Node.js - Version 18.20.3
FoundryVTT | 2024-09-02 14:37:08 | [info] Foundry Virtual Tabletop - Version 12 Build 331
FoundryVTT | 2024-09-02 14:37:08 | [info] User Data Directory - "/data"
FoundryVTT | 2024-09-02 14:37:08 | [info] Application Options:
{
  "awsConfig": null,
  "compressSocket": false,
  "compressStatic": true,
  "cssTheme": "foundry",
  "fullscreen": false,
  "hostname": null,
  "hotReload": false,
  "language": "en.core",
  "localHostname": null,
  "passwordSalt": null,
  "port": 30000,
  "protocol": null,
  "proxyPort": null,
  "proxySSL": false,
  "routePrefix": null,
  "sslCert": null,
  "sslKey": null,
  "updateChannel": "stable",
  "upnp": false,
  "upnpLeaseDuration": null,
  "world": null,
  "adminPassword": "••••••••••••••••",
  "deleteNEDB": false,
  "noBackups": false,
  "serviceConfig": null
}
FoundryVTT | 2024-09-02 14:37:08 | [error] Software license verification failed. Please confirm your Foundry Virtual Tabletop software license
FoundryVTT | 2024-09-02 14:37:08 | [info] Server started and listening on port 30000
FoundryVTT | 2024-09-02 14:37:09 | [info] Created client session e22a9063902861a18b9ef169
FoundryVTT | 2024-09-02 14:37:16 | [warn] Could not reach IP discovery service
FoundryVTT | 2024-09-02 14:37:23 | [info] Created client session a82d92e224e89cbe5ff6eac7
FoundryVTT | 2024-09-02 14:37:26 | [info] License signature successfully created. Thank you and please enjoy Foundry Virtual Tabletop.
FoundryVTT | 2024-09-02 14:37:26 | [info] Software license verification succeeded
FoundryVTT | 2024-09-02 14:37:35 | [info] Administrator authentication successful for session e22a9063902861a18b9ef169
FoundryVTT | 2024-09-02 14:37:36 | [warn] The system "wfrp4e" is using "gridDistance" which is deprecated in favor of "grid.distance".
Deprecated since Version 12
Backwards-compatible support will be removed in Version 14
FoundryVTT | 2024-09-02 14:37:36 | [warn] The system "wfrp4e" is using "gridUnits" which is deprecated in favor of "grid.units".
Deprecated since Version 12
Backwards-compatible support will be removed in Version 14
FoundryVTT | 2024-09-02 14:37:42 | [info] Saved application configuration options to /data/Config/options.json

10 minutes later restarted the container:

< etag: "a0413e8a134d6db99581d20ec5a8088f-28"

< last-modified: Thu, 15 Aug 2024 15:26:54 GMT

< vary: Accept-Encoding

< cache-control: max-age=14400

< cf-cache-status: MISS

< accept-ranges: bytes

< server: cloudflare

< cf-ray: 8bce3c29ab1b477e-DFW

<

{ [5 bytes data]

 13  221M   13 30.2M    0     0  33.9M      0  0:00:06 --:--:--  0:00:06 33.9M
 37  221M   37 84.1M    0     0  44.5M      0  0:00:04  0:00:01  0:00:03 44.5M
 60  221M   60  134M    0     0  46.7M      0  0:00:04  0:00:02  0:00:02 46.7M
 89  221M   89  198M    0     0  50.8M      0  0:00:04  0:00:03  0:00:01 50.8M
100  221M  100  221M    0     0  52.5M      0  0:00:04  0:00:04 --:--:-- 52.5M
* Connection #0 to host r2.foundryvtt.com left intact
Entrypoint | 2024-09-02 07:36:58 | [info] Installing Foundry Virtual Tabletop
Entrypoint | 2024-09-02 07:36:58 | [debug] Checking mime-type of release file.
Entrypoint | 2024-09-02 07:36:58 | [debug] Found mime-type: application/zip
Entrypoint | 2024-09-02 07:36:58 | [debug] Extracting release file.
Entrypoint | 2024-09-02 07:37:02 | [debug] Installation completed.
Entrypoint | 2024-09-02 07:37:02 | [info] Preserving release archive file in cache.
Entrypoint | 2024-09-02 07:37:02 | [debug] CONTAINER_CACHE_SIZE is not set. Skipping cache cleanup.
Entrypoint | 2024-09-02 07:37:02 | [debug] Patching GUI update and configuration messages.
Entrypoint | 2024-09-02 07:37:02 | [info] Not modifying existing installation license key.
Entrypoint | 2024-09-02 07:37:02 | [info] Setting data directory permissions.
Entrypoint | 2024-09-02 07:37:02 | [debug] Setting ownership of /data to foundry:foundry.
Entrypoint | 2024-09-02 07:37:07 | [debug] Completed setting directory permissions.
Entrypoint | 2024-09-02 07:37:07 | [info] Starting launcher with uid:gid as foundry:foundry.
Entrypoint | 2024-09-02 07:37:07 | [debug] Waiting for child pid: 75 to exit.
Launcher | 2024-09-02 07:37:07 | [debug] Ensuring /data/Config directory exists.
Launcher | 2024-09-02 07:37:07 | [info] Generating options.json file.
Launcher | 2024-09-02 07:37:07 | [info] Setting 'Admin Access Key'.
Launcher | 2024-09-02 07:37:07 | [info] Starting Foundry Virtual Tabletop.
FoundryVTT | 2024-09-02 14:37:08 | [info] Running on Node.js - Version 18.20.3
FoundryVTT | 2024-09-02 14:37:08 | [info] Foundry Virtual Tabletop - Version 12 Build 331
FoundryVTT | 2024-09-02 14:37:08 | [info] User Data Directory - "/data"
FoundryVTT | 2024-09-02 14:37:08 | [info] Application Options:
{
  "awsConfig": null,
  "compressSocket": false,
  "compressStatic": true,
  "cssTheme": "foundry",
  "fullscreen": false,
  "hostname": null,
  "hotReload": false,
  "language": "en.core",
  "localHostname": null,
  "passwordSalt": null,
  "port": 30000,
  "protocol": null,
  "proxyPort": null,
  "proxySSL": false,
  "routePrefix": null,
  "sslCert": null,
  "sslKey": null,
  "updateChannel": "stable",
  "upnp": false,
  "upnpLeaseDuration": null,
  "world": null,
  "adminPassword": "••••••••••••••••",
  "deleteNEDB": false,
  "noBackups": false,
  "serviceConfig": null
}
FoundryVTT | 2024-09-02 14:37:08 | [error] Software license verification failed. Please confirm your Foundry Virtual Tabletop software license
FoundryVTT | 2024-09-02 14:37:08 | [info] Server started and listening on port 30000
FoundryVTT | 2024-09-02 14:37:09 | [info] Created client session e22a9063902861a18b9ef169
FoundryVTT | 2024-09-02 14:37:16 | [warn] Could not reach IP discovery service
FoundryVTT | 2024-09-02 14:37:23 | [info] Created client session a82d92e224e89cbe5ff6eac7
FoundryVTT | 2024-09-02 14:37:26 | [info] License signature successfully created. Thank you and please enjoy Foundry Virtual Tabletop.
FoundryVTT | 2024-09-02 14:37:26 | [info] Software license verification succeeded
FoundryVTT | 2024-09-02 14:37:35 | [info] Administrator authentication successful for session e22a9063902861a18b9ef169
FoundryVTT | 2024-09-02 14:37:36 | [warn] The system "wfrp4e" is using "gridDistance" which is deprecated in favor of "grid.distance".
Deprecated since Version 12
Backwards-compatible support will be removed in Version 14
FoundryVTT | 2024-09-02 14:37:36 | [warn] The system "wfrp4e" is using "gridUnits" which is deprecated in favor of "grid.units".
Deprecated since Version 12
Backwards-compatible support will be removed in Version 14
FoundryVTT | 2024-09-02 14:37:42 | [info] Saved application configuration options to /data/Config/options.json

Code of Conduct

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

And I've manually checked - there's nothing in my Cache folder.  At least not the path that's assigned in the Unraid configs.


icsharpcode.ILSpy version <IP_ADDRESS>45

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Fixes https://github.com/microsoft/winget-pkgs/issues/108304

Automatic Validation ended with:

Exception during executable launch operation System.AggregateException: One or more errors occurred. (The specified executable is not a valid application for this OS platform.)  ---
System.ComponentModel.Win32Exception (216): The specified executable is not a valid application for this OS platform.


FEAT: Have an Appendix/Talks page

We could have an appendix or a presentations page that would have a markdown of the presentations we have done in the past and other materials we have created. For example the one for the Scientific Communication Session.

I close this issue as it's redundant with #8.


Make FreeRTOS_OutputARPRequest() available for application code

Description

The function FreeRTOS_OutputARPRequest() was developed to be used by both the IP-task, as well as the application. However, when the application calls it, xNetworkInterfaceOutput() may land in a configASSERT(), because only the IP-task is allowed to call that (non re-entrant) function.
Solution:
    if( xIsCallingFromIPTask() != 0 )
    {
        /* Only the IP-task is allowed to call this function directly. */
        xNetworkInterfaceOutput( pxNetworkBuffer, pdTRUE );
    }
    else
    {
        /* Let the IP-task send the packet. */
    }

A new event-type is added: eNetworkTxEvent, which is sent as:
IPStackEvent_t xSendEvent;
    xSendEvent.eEventType = eNetworkTxEvent;
    xSendEvent.pvData = ( void * ) pxNetworkBuffer;
    xSendEventStructToIPTask( &xSendEvent, ( TickType_t ) portMAX_DELAY );

The IP-task will release the network buffer once the packet has been sent.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is Linted.

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

/bot run checks

/bot run checks


bugfix/AB#25658-RefactorRabbitMQ - Fix connections left open - singleton

Ready for Review
https://blog.devops.dev/robust-rabbitmq-implementation-for-asp-net-core-5672693544a2

At a high level:

the Unity.RabbitMQ project seems out of place in the solution structure the rest of the projects under src make up the clean architecture layers
RabbitMQ seems a bit too 'baked in' for example, in the EmailQueueService it directly calls on the RabbitMQ interface to queue messages, normally you would create an abstraction layer on top of the message brokering such that you are agnostic to which queuing mechanism is used under the hood and call / use that abstraction - this way we could swap out RabbitMQ for Kafka and would only be config changes and no code changes
I see a few naming standard issues like InvoiceService.cs line 137 parameter PaymentRequestId should be paymentRequestId

Fixed the cap case on line 137.
I could use some help in how we should refactor the reference RabbitMQ project

Absolutely, decoupling your application from specific message brokers like RabbitMQ by introducing an abstraction layer is a good practice. This approach enhances flexibility, allowing you to switch between different message brokers (such as RabbitMQ, Kafka, Azure Service Bus, etc.) more seamlessly without needing to rewrite significant portions of your codebase. Here’s how you can refactor your EmailQueueService to use an abstraction for the message queue:
Step-by-Step Refactoring
Define an Interface for Message Queue Operations:
Create an interface that defines the operations you need for interacting with your message queue. This interface will be implemented separately for each message broker you intend to support.
csharp
Copy code
public interface IMessageQueueProvider
{
Task PublishMessageAsync(TMessage message);
}
Here, TMessage represents the type of message you want to publish (in your case, EmailMessages).
Implement RabbitMQ Message Queue Provider:
Implement the IMessageQueueProvider interface specifically for RabbitMQ. This implementation will use RabbitMQ-specific libraries and configurations.
csharp
Copy code
public class RabbitMQMessageQueueProvider : IMessageQueueProvider
{
private readonly IQueueProducer _queueProducer;
public RabbitMQMessageQueueProvider(IQueueProducer<TMessage> queueProducer)
{
    _queueProducer = queueProducer;
}

public async Task PublishMessageAsync(TMessage message)
{
    // Assuming _queueProducer is properly configured to publish to RabbitMQ
    _queueProducer.PublishMessage(message);
    await Task.CompletedTask; // or await something async if needed
}

}
In this implementation, _queueProducer corresponds to your RabbitMQ-specific implementation of IQueueProducer.
Refactor EmailQueueService to Use Abstraction:
Modify EmailQueueService to accept an IMessageQueueProvider instead of directly interacting with RabbitMQ:
csharp
Copy code
public class EmailQueueService : ApplicationService
{
private readonly IMessageQueueProvider _messageQueueProvider;
private readonly ILogger _logger;
public EmailQueueService(IMessageQueueProvider<EmailMessages> messageQueueProvider,
                         ILogger<EmailQueueService> logger)
{
    _messageQueueProvider = messageQueueProvider;
    _logger = logger;
}

public async Task SendToEmailDelayedQueueAsync(EmailNotificationEvent emailNotificationEvent)
{
    try
    {
        var message = new EmailMessages
        {
            TimeToLive = TimeSpan.FromMinutes(1),
            EmailNotificationEvent = emailNotificationEvent
        };

        await Task.Delay(TimeSpan.FromMilliseconds(FiveMinutesInMilliSeconds * (emailNotificationEvent.RetryAttempts + 1)));

        await _messageQueueProvider.PublishMessageAsync(message);
    }
    catch (Exception ex)
    {
        var ExceptionMessage = ex.Message;
        _logger.LogError(ex, "SendToEmailDelayedQueueAsync Exception: {ExceptionMessage}", ExceptionMessage);
    }
}

public async Task SendToEmailEventQueueAsync(EmailNotificationEvent emailNotificationEvent)
{
    try
    {
        var message = new EmailMessages
        {
            TimeToLive = TimeSpan.FromMinutes(1),
            EmailNotificationEvent = emailNotificationEvent
        };
        await _messageQueueProvider.PublishMessageAsync(message);
    }
    catch (Exception ex)
    {
        var ExceptionMessage = ex.Message;
        _logger.LogError(ex, "SendToEmailEventQueueAsync Exception: {ExceptionMessage}", ExceptionMessage);
    }
}

}
Now, EmailQueueService is agnostic to the underlying message queue implementation. It relies on IMessageQueueProvider to handle the specifics of publishing messages.
Dependency Injection Configuration:
Register RabbitMQMessageQueueProvider (or other implementations for different brokers) and EmailQueueService with your dependency injection container. Here's an example using ASP.NET Core's built-in DI:
csharp
Copy code
services.AddSingleton<IMessageQueueProvider, RabbitMQMessageQueueProvider>();
services.AddTransient();
This ensures that EmailQueueService receives an instance of IMessageQueueProvider that is implemented specifically for RabbitMQ.
Benefits
Flexibility: You can now switch the message broker implementation by simply changing the registration in your DI container without modifying EmailQueueService.
Testability: You can easily mock IMessageQueueProvider for unit testing EmailQueueService without needing a live message broker.
Code Maintainability: The abstraction layer isolates message queue-specific logic, making your codebase cleaner and easier to understand.
By following this approach, your application gains significant flexibility and maintainability, adhering to best practices for dependency inversion and separation of concerns.


Update raycast condition and action with new fields.

This PR updates the raycast condition and action with new fields.

distance is now optional, where if this and the below fields aren't set, it will get the entity's block reach/entity attack range (with Reach Entity Attributes compat).
block_reach (double) a distance that is only effective with block raycasts
entity_distance (double) a distance that only is effective with entity raycasts.
direction (normalised vector) if set, the direction that the ray will travel in, if not set, the direction will be the entity's rotation vector as it was without this field.
space (space) the space of the direction, if the direction field isn't set, it won't do anything.

I need to fix the base block reach value. I'll do so when I can.


Recommended recipe for noisy 5K hours noisy training data

Hello guys,
I am working with inhouse 5K hours noisy training data. What is the most recommended recipe for my case? Meantime I am using Zipformer from Tedlium recipe.
Thanks.

https://github.com/k2-fsa/icefall/tree/master/egs/librispeech/ASR/zipformer
It is our best-performing recipe.


Removes default CDN host

We'll now trust the tiler and use whatever CDN configuration we get (or nothing, if cdn_url is empty).

sounds good!

Thanks!


Add pyodide execution support

As discussed in our meeting today, here's my prototype of adding live code execution to the docs along with my notes:
Live Code Execution

Since docs are already based on docusaurus there's no point switching to some other solution
Docusaurus offers react-py for pyodide based execution
Together with Ace editor it's simple to integrate live code execution

mistralai pyodide support

Currently mistralai depends on pydantic-core==2.20.1
Pydantic does publish emscripten wheels :

https://githubproxy.samuelcolvin.workers.dev/pydantic/pydantic-core/releases/download/v2.20.1/pydantic_core-2.20.1-cp311-cp311-emscripten_3_1_46_wasm32.whl

However they are compiled with emscripten 3.1.46, current version of pyodide are compiled with empscripten 3.1.58
We can patch react-py with old version of pyodide (0.25.0)
mistralai uses httpx which is not yet supported in pyodide https://github.com/koenvo/pyodide-http/issues/37

There's some efforts to remedy this, specifically https://github.com/encode/httpx/discussions/2994 suggests adding a transport layer based on urllib3 to httpx. This could already be patched in mistralai itself if we detect we are running in a pyodide context.

API Token Problem

Since pyodide runs in the browser users could access any tokens that are used
We could set up a backend proxy server that interacts with Mistral API
How do we authenticate with this proxy server?

Docs pages currently do not require login or OAuth
OAuth tokens probably best and safest way to avoid abuse
User authenticates and is given an access_token, we somehow pass the access_token through to the proxy server which checks rate limits and then forwards the request to Mistral API

ToDo

[ ] Add theme handling to the code editor (i.e. switch the ace theme based on dark/light mode)
[ ] Add a patch to support httpx requests in pyodide to the Python SDK

The patch looks something like this (
import httpx
import urllib3
urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)
import json

class URLLib3Transport(httpx.BaseTransport):
    def __init__(self):
        self.pool = urllib3.PoolManager()

    def handle_request(self, request: httpx.Request):
        body = json.loads(request.content.decode('utf-8'))  # Decode the data and load as JSON
        urllib3_response = self.pool.request(request.method, str(request.url), body=request.content, headers=dict(request.headers))  # Convert httpx.URL to string
        content = json.loads(urllib3_response.data.decode('utf-8'))  # Decode the data and load as JSON
        stream = httpx.ByteStream(json.dumps(content).encode("utf-8"))  # Convert back to JSON and encode
        headers = [(b"content-type", b"application/json")]
        return httpx.Response(200, headers=headers, stream=stream)

httpx_client = httpx.Client(transport=URLLib3Transport())

client = Mistral(api_key=api_key, client=httpx_client)

Oh, one more thing, you have to manually patch the pyodide version by editing:
/node_modules/react-py/dist/workers/service-worker.js
and changing the pyodide version to 0.25.0.


New constants for unfinished/planned broadcasts

New presets/variables and updated actions/feedbacks that will allow us having static buttons with planned Broadcasts names/states displayed using ${unfinished_0}, ${unfinished_state_0}, etc... without the need of reconfiguring companion buttons before action in case operators need to see dashboard with current broadcasts status.
Broadcast list menu in actions/feedbacks extended with constants like "Unfinished #0", "Unfinished #1", so you dont need to constantly changing YT broadcast in selectbox.
Completed broadcast is not removed from unfinished list, but stays until "Reload everything from YouTube" action called.
The size of "Unfinished list" is configurable.

issue started in https://github.com/bitfocus/companion-module-youtube-live/issues/62

@jendamozna, you may want to post a message on Slack to get the update above merged.

@jendamozna, I tried the merged functionality and found some small issues:

The feedbacks, i.e. "YT: Broadcast status" and "YT: Health of stream bound to broadcast" does not seem to work when feedbacks for the "unfinished/planned #n" streams are selected - Is this a bug?
The order for the streams seems to be in the order they have been created in Youtube Studio, not in a chronological order. However, Youtube seems to set the order in this way as the actual names for upcoming streams also show up in the same order as unplanned #0, #1, #2 etc. so I don't consider this to be a bug, more something that is good to know.

Good to know that feedbacks were working while developing, then there is some hope that it can be fixed.

@jendamozna I tested the new features and I had issue with the "finish broadcast" action, used with the unfinished constants. I had the following log:

instance(YT2): Action failed: Error: Action has unknown broadcast ID - not found or invalid
I checked the corresponding variable in the module variables list, and the good broadcast was associated.

Hi, thanks for feedback! This pull request was buggy. Issues with feedbacks and actions should be fixed in next merge. Currently there is pending merge request https://github.com/bitfocus/companion-module-youtube-live/pull/66
Mentioned bug is fixed there.

Thank you for your answer and your reactivity to fix the bugs!
I'll wait for the merge.


Version mismatch: 1.9.10714.2 Extension installer ships with vsix Manifest indicating version 1.8.10107.10 in Visual Studio 2015

If you download the current (1.9.10714.2) extension from https://visualstudiogallery.msdn.microsoft.com/1ec7db13-3363-46c9-851f-1ce455f66970 and install it into visual studio 2015, the extension will show as version 1.8.10107.10 in visual studio and a permanent 'Update Available' notification  will be displayed.
Checking the extensions manifest in C:\Program Files (x86)\Microsoft Visual Studio 14.0\Common7\IDE\Extensions\Microsoft\CodeContracts\1.9.10714.2 shows the following content in extension.vsixmanifest:
<?xml version="1.0" encoding="utf-8"?>
<PackageManifest Version="2.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns="http://schemas.microsoft.com/developer/vsx-schema/2011" xmlns:d="http://schemas.microsoft.com/developer/vsx-schema-design/2011">
    <Metadata>
        <Identity Version="1.8.10107.10" ...

A quick check with 7-zip inside the Contracts.devlab9ts.msi installer confirms that the file VS14extension.manifest is the only manifest indicating a faulty version number (VS10, VS11 & VS12 manifests all show the correct 1.9.10714.2 version).
This needs an update to the installer with a correct manifest.

Probably duplicate of #250 and #161


Remove incorrect obsolete from WindowsSpeechInputProvider

Overview
When the registrar was removed from the constructor in https://github.com/microsoft/MixedRealityToolkit-Unity/pull/6457, it looks like the new constructor was accidentally also marked with Obsolete. This PR removes that. The old constructor is still obsolete.

/azp run


Analyzer does not recognize wrong subtype

This issue was originally filed by<EMAIL_ADDRESS>
What steps will reproduce the problem?
abstract class ConnectionBuilder {
  WebSocketConnection build(final EventBus eventbus);
  ...
}
class ConnectionBuilderToServer implements ConnectionBuilder {
  ...
  WebSocketConnection build(final EventBus eventbus) {
  ...
}
class ConnectionBuilderForChat implements ConnectionBuilder {
  ...
  WebSocketConnection build(final EventBus eventbus) {
  ...
}
class ChatPresenter extends Presenter {
  final ConnectionBuilderForChat  _connectionbuilder;

  ChatPresenter(this._connectionbuilder);
  ...
  }

void main() {
    final ConnectionBuilder cbuilder = (new ConnectionBuilderToServer(uriprovider.forChat()));

    !!!!! cbuilder is the wrong type - it should be ConnectionBuilderForChat
    !!!!! This leads to an Runtime Exception:
    !!!!! Exception: type 'ConnectionBuilderToServer' is not a subtype of type 'ConnectionBuilderForChat' of '_connectionbuilder@0x3cfc28ed'.
    !!!!! but should be recogized by the analyser

    final Presenter presenter = new ChatPresenter(cbuilder, new ChatView(_eventbus), _eventbus );
}
What is the expected output? What do you see instead?
Error in Editor, but leads to an Exception
What version of the product are you using? On what operating system?
Dart Editor version 0.5.16_r23799
Dart SDK version <IP_ADDRESS>_r23799
Please provide any additional information below.

The override is invalid in Dart 2 without the covariant keyword on the argument which should serve as a signal that there is risky stuff happening.
As far as I know we don't have any realistic plans to start treating an explicitly typed variable as anything other than it's static type. Note that you can use var to get an inferred type which might help here.


Post drawing changes

Hey again!
By now this is my third post on this project, and I have to say that ever since I first started using it over 6 months ago, I have grown more and more dependent on it rather than the normal MATLAB toolkit, so once again - great job with this project. You are doing a service to mankind (or at least the niche that does statistical analysis using MATLAB 😶).
In working this much with this toolbox, I have come to know some of it's limitations - a big one being the inability to make major changes to design after drawing a gramm object.
An example of my current use case - I want to save two versions of a specific graph, one with a title and another with no title.
If this were any normal figure, I would have simply drawn the figure sans title, saved the figure, added a title, then saved the figure again under a different name. In gramm this is much more difficult, as I cannot actually use an existing gramm object to completely redraw and rearrange a figure.
I have also tried copying the gramm object, opening a new figure, and drawing, but I understand now that the gramm object is connected to a specific figure, and cannot be simply copied around.
So I would like to know if you could think up of a workaround for this issue, namely to either be able to completely redraw a figure using an existing gramm object after adding a title and such or to simply copy an existing gramm object into a new one, which can be drawn separately from the original (perhaps this is already implemented but I didn't figure out how to do so?).
In any case, keep up the good work.

Hi, thanks for the kind comment :) I'm well aware that the trade-off for a lot of gramm features is limited customization.
However, doing what you want is completely possible, as gramm objects (at least since the Nov. 1 commit) should be fully copyable at any point. You have to copy them explicitly using copy() though, I use it a lot in the examples.m file.
So in pseudo-gramm code:
g=gramm(...)
g.facet_grid(...)
g.geom_point(...)
%call any gramm method except draw() and update()

gt=copy(g);

figure
g.draw()

gt.set_title(...)
figure
gt.draw()

Let me know if that works!

Well what do you know!
http://imgur.com/a/xWEH6
Too bad I already finished this specific keynote, but since it's an automated system that makes the keynote, I'll implement it for the next iteration!
BTW, Maybe the cheatsheet should add some kind of intro section explaining the behavior of gramm objects, specifically these sort of things which might exist in the examples page but aren't necessarily too obvious when browsing through it.
Thanks again for such swift responses!

Nooo, the imgur link doesn't work!
I'll see how I can add that to the documentation, it's true that the example page is a bit long now. Maybe just mentioning copy() in the cheat sheet?

imgur link is updated =]
I was originally thinking about simply mentioning simple actions like copy() in the cheatsheet but having remembered that it took me a while to get a grasp of how to use the cheatsheet and gramm in general, I think that a short intro of the steps required to make a gramm would actually help a great deal.

Kind of an extended version of the "workflow" section in the realm page (main page on GitHub)?

Something like that.
To be honest I only just now noticed that there's a "Workflow" section in the read! But well, that might just show that it should be available on either the cheatsheet or examples as well.
Also, I think a brief paragraph explaining how gramm works (i.e. gramm creates a MATLAB object containing the data for x,y axis grouping and other attributes, and then you can use this object's subroutines to choose how to display that data) might be in order, as the gramm code works quite differently from what MATLAB users are used to.

Hi, good point about gramm not being "standard" Matlab, and explanation might also help on this.
If you have time to help on this that would be great! Maybe by submitting some more explanations to the first section of the examples.m file or the readme.md file, which then I could include at the beginning of the cheat sheet (the original document is a .numbers table that is not on github). I'd have to find a way to refer to the color coding that is used in the cheat sheet (which I need to improve).

Hi, another gramm user opened a issue specifically for discussing tutorials/help, you can join the discussion if still interested: https://github.com/piermorel/gramm/issues/31


"TypeError: Cannot read property 'end' of undefined"

Despite the same error message, I don't think this is a dupe of https://github.com/palantir/tslint/issues/3244 as that appears to have been fixed in 5.8.0.

Bug Report

TSLint version: 5.8.0
TypeScript version: 2.6.2
Running TSLint via: CLI

TypeScript code being linted
export interface SectionPreviewAction = {
  type: 'section-preview/REPLACE_THUMBNAIL_SUCCESS',
  sectionId: string,
  thumbnailUrl: string
};

(yes, this is known bad/invalid code - existed briefly as part of a small refactor)
with tslint.json configuration:

{
  "extends": ["tslint-config-airbnb", "tslint:recommended", "tslint-react"],
  "rules": {
    "eofline": true,
    "function-name": [true, {
      "function-regex": "^[a-zA-Z][\\w\\d]+$"
    }],
    "quotemark": [true, "single", "jsx-single", "avoid-template", "avoid-escape"],
    "import-name": [false],
    "interface-name": [true, "never-prefix"],
    "jsx-boolean-value": [true, "never"],
    "ordered-imports": [true, { "import-sources-order": "any", "named-imports-order": "case-insensitive" }],
    "trailing-comma": [true, {"multiline": "never", "singleline": "never"}],
    "variable-name": [true, "ban-keywords", "check-format", "allow-pascal-case"],
    "typedef": [
      true,
      "parameter",
      "arrow-parameter",
      "property-declaration",
      "call-signature",
      "arrow-call-signature"
    ],
    "max-line-length": [true, 100],
    "object-shorthand-properties-first": [false],
    "indent": [2, "spaces"]
  }
}

Actual behavior
TypeError: Cannot read property 'end' of undefined
    at OneLineWalker.check (/Users/dylan/dev/cala/studio/node_modules/tslint/lib/rules/oneLineRule.js:141:31)
    at cb (/Users/dylan/dev/cala/studio/node_modules/tslint/lib/rules/oneLineRule.js:92:27)
    at visitNodes (/Users/dylan/dev/cala/studio/node_modules/typescript/lib/typescript.js:12699:30)
    at Object.forEachChild (/Users/dylan/dev/cala/studio/node_modules/typescript/lib/typescript.js:12877:24)
    at OneLineWalker.walk (/Users/dylan/dev/cala/studio/node_modules/tslint/lib/rules/oneLineRule.js:138:19)
    at Rule.AbstractRule.applyWithWalker (/Users/dylan/dev/cala/studio/node_modules/tslint/lib/language/rule/abstractRule.js:31:16)
    at Rule.apply (/Users/dylan/dev/cala/studio/node_modules/tslint/lib/rules/oneLineRule.js:34:21)
    at Linter.applyRule (/Users/dylan/dev/cala/studio/node_modules/tslint/lib/linter.js:177:29)
    at /Users/dylan/dev/cala/studio/node_modules/tslint/lib/linter.js:119:85
    at Object.flatMap (/Users/dylan/dev/cala/studio/node_modules/tslint/lib/utils.js:151:29)

ERROR: src/redux/section-previews/actions.ts[8, 18]: An empty interface is equivalent to `{}`.

Expected behavior
ERROR: src/redux/section-previews/actions.ts[10, 14]: statements are not aligned
ERROR: src/redux/section-previews/actions.ts[11, 17]: statements are not aligned
ERROR: src/redux/section-previews/actions.ts[10, 12]: Missing semicolon
ERROR: src/redux/section-previews/actions.ts[11, 15]: Missing semicolon
ERROR: src/redux/section-previews/actions.ts[11, 23]: Missing semicolon
ERROR: src/redux/section-previews/actions.ts[12, 2]: Unnecessary semicolon

— this is the output from tslint 5.5.0, the previous version we used

Fix is up at #3538
Suprisingly TypeScript parses this declaration really bad.


Add --bind-address CLI option

Currently NDM accepts --port CLI option overriding default (:9090) listening port.
It would be handy to accept --bind-address option also to make NDM listenting on a desired address, not <IP_ADDRESS>/::.

@vibornoff  Thanks for raising the issue.  We will work on including --bind-address CLI option in NDM.
A patch for doing this is also welcome.

@akhilerm May I work on this?

@aswathkk  yes. you can take it up.

An issue #324  is raised to remove the unused flags.

NDM has a grpc endpoint which listens on 9115, and can be configured using api-service-address flag


[BUG] Set-PnPPage translation a full width section page

Reporting an Issue or Missing Feature
Issue: Set-PnPPage cmdlet error when trying to translate a page with a full width section.
Expected behavior
I expect the Set-PnPPage cmdlet to successfully copy the translated page without errors.
Actual behavior
When running the command:
Set-PnPPage -Identity "Yo.aspx" -Translate -TranslationLanguageCodes 1036 -LayoutType "Article"

I receive the following error:
2024-10-04T14:23:15Z   [Error]   ERROR: You cannot host text controls inside a one column full width section, only an image web part or hero web part are allowed

Exception             :
    Type        : System.Management.Automation.PSInvalidOperationException
    ErrorRecord :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : You cannot host text controls inside a one column full width section, only an image web part or hero web part are allowed
            HResult : -2146233087
        CategoryInfo          : InvalidOperation: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : InvalidOperation
    TargetSite  :
        Name          : ProcessRecord
        DeclaringType : [PnP.PowerShell.Commands.Base.PnPConnectedCmdlet]
        MemberType    : Method
        Module        : PnP.PowerShell.dll
    Message     : You cannot host text controls inside a one column full width section, only an image web part or hero web part are allowed
    Source      : PnP.PowerShell
    HResult     : -2146233079
    StackTrace  :
   at PnP.PowerShell.Commands.Base.PnPConnectedCmdlet.ProcessRecord() in D:\a\powershell\powershell\src\Commands\Base\PnPConnectedCmdlet.cs:line 101
   at PnP.PowerShell.Commands.PnPSharePointCmdlet.ProcessRecord() in D:\a\powershell\powershell\src\Commands\Base\PnPSharePointCmdlet.cs:line 126
   at System.Management.Automation.Cmdlet.DoProcessRecord()
   at System.Management.Automation.CommandProcessor.ProcessRecord()
CategoryInfo          : InvalidOperation: (:) [Set-PnPPage], PSInvalidOperationException
FullyQualifiedErrorId : InvalidOperation,PnP.PowerShell.Commands.Pages.SetPage
InvocationInfo        :
    MyCommand        : Set-PnPPage
    ScriptLineNumber : 107
    OffsetInLine     : 5
    HistoryId        : 1
    ScriptName       : C:\home\site\wwwroot\translation\run.ps1
    Line             :     Set-PnPPage -Identity $pageName -Translate -TranslationLanguageCodes $lcid -LayoutType $pageSource.LayoutType

    Statement        : Set-PnPPage -Identity $pageName -Translate -TranslationLanguageCodes $lcid -LayoutType $pageSource.LayoutType
    PositionMessage  : At C:\home\site\wwwroot\translation\run.ps1:107 char:5
                       +     Set-PnPPage -Identity $pageName -Translate -TranslationLanguageCo …
                       +     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    PSScriptRoot     : C:\home\site\wwwroot\translation
    PSCommandPath    : C:\home\site\wwwroot\translation\run.ps1
    InvocationName   : Set-PnPPage
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, C:\home\site\wwwroot\translation\run.ps1: line 107

Despite using only an image in the full width section.
Steps to reproduce behavior

Create a page in a site with translation enabled in any language.
Create a full width section and add an image WebPart in it.
Attempt to translate the page using the following command:

Set-PnPPage -Identity "yourpage.aspx" -Translate -TranslationLanguageCodes 1036 -LayoutType "Article"

Error Message

What is the version of the Cmdlet module you are running?
"2.99.2-nightly" (also tried "1.12.0")
Which operating system/environment are you running PnP PowerShell on?

[X] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[X] Azure Functions
[ ] Other : please specify

This is the stacktrace:

at PnP.Core.Model.SharePoint.PageWebPart.ToHtml(Single controlIndex)
at PnP.Core.Model.SharePoint.CanvasColumn.ToHtml()
at PnP.Core.Model.SharePoint.CanvasSection.ToHtml()
at PnP.Core.Model.SharePoint.Page.ToHtml()
at PnP.Core.Model.SharePoint.Page.SaveAsync(String pageName)
at PnP.Core.Model.SharePoint.Page.Save(String pageName)
at PnP.PowerShell.Commands.Pages.SetPage.ExecuteCmdlet() in PnP\powershell\src\Commands\Pages\SetPage.cs:line 152

So the issue is more related to PnP.Core than PnP.PowerShell by the look of it.

It seems that image webpart is missing the isFullWidth property somehow. See below how properties.TryGetProperty("isFullWidth", ...) returned false:

Thank you !It would be great if we could fix/disable this condition. I'm having trouble dodging this bug on my end. 😄

Tried with 2.99.80 and it works correctly without throwing any error anymore:

Given a page at /SitePages/Test.aspx , it created /SitePages/sv/Test.aspx when running Set-PnPPage -Identity "Test.aspx" -Translate -TranslationLanguageCodes 1053 -LayoutType "Article"

Thanks @jackpoz for confirming the fix, going to close this as it has been fixed in the latest nightly builds.


Confusion about data sampling strategy for scaling analysis in paper

Hi，thanks for your remarkable NDT2 work. I am a newcomer for neuron science. I have some confusions about figure 4 in paper as showed below, could you please help me figure it out?

In this figure, pretraining trials have different size, what is the sampling strategy for different trial size?
Does multi-task for 200k size contain all the data? would more data help improve performance?

Thanks^_^

data sampling details was in the code.


Test for 100% test coverage

removes coveralls, no need to show off if we have 100% anyway

View rendered README.md

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Invalid code in example "add custom Property"

the code says this:  !itemProperties.ContainsKey("customProp")
it should be   !itemProperties.Properties.ContainsKey("customProp")

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

ID: 26b3e91f-c240-fffa-6ae9-d676deb7ad4c
Version Independent ID: bbfa099e-3ed6-09ea-fbde-f2a3ed13139a
Content: Filtering and preprocessing in the Azure Application Insights SDK - Azure Monitor
Content Source: articles/azure-monitor/app/api-filtering-sampling.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@greektreat Thank you for bringing this to our attention. I have assigned this to the content author to evaluate and update as appropriate.

This has been fixed and the change will go live tomorrow. Thanks again for bringing this to our attention.

#sign-off

#please-close


Please reopen #5256

Please reopen #5256
I would like to make comment on that issue.

Is there any reason to force connect and withRouter order? Or please give official docs into the issue comment because that thread is the first result from google

It would be very pleasure if there is eslint plugin to force the correct order

PS: I took me hours to debug before finding out the issue. Unfortunately, the issue was locked

I have figured that @connect blocks inner component from re-render. Thus, component will never get url change update


Remove renter's merkle roots from ram

Instead of having all the merkle roots of every contract in memory, the renter should load them from disk if needed.

This PR is dependent on PR#19

Went through all the testing, mostly looks good. Want to make sure everything is being persisted correctly.
Are there any tests that would make sense to add to the siatest package? I want to make sure we have coverage for this, though I'm thinking right now that our existing siatest tests are probably good enough. The only other thing I can think that we'd need to check is the ACID guarantees, but we are working on that in a different set of pull requests.


Enable backlight without underglow LEDs.

Originally posted by @carlvr in https://github.com/foostan/crkbd/discussions/115

I was wondering if it's possible to enable the Backlighting LEDs without soldering the Underglow LEDs, by directly connecting some pins. From what I understand the LEDs are chained, so that the Backlighting LEDs only work when the Underglow LEDs are also installed, since they are coming first in this chain. But by adding a connection between some pins of the first Underglow LED and the first Backlighting LED, it should be possible to power all the Backlighting LEDs, right?

this is exactly what i did... i soldered the backlight without soldering the underglow leds... i bridged them and changed the firmware to have the correct support for RGB Matrix
i can think of two solutions here

Change the order of the leds, so that the first ones to solder are the backlight and then the underglow
Add some pads that can be shorted to skip the underglow leds (instead of having to manually bridge them)

The undergrow led is deprecated
https://github.com/foostan/crkbd/releases


Configuration obtaining problem

Can I get the Settings of the players in the demo? Screen Settings, arm stretches, etc

I don't think this is possible as I think these are not stored in demos.


Add a Toggle to the Status Bars w/ a Delay to show only in Combat

This is to only show the Status Bars only when in Combat, and hides it based on the delay set.
Closes #7436
Toggle Demo: https://gyazo.com/ee277e5f0d4bfad598a7a1049d972a67
Delay Demo: https://gyazo.com/421e2a4a725725f1b7b5e62f96a6a8a3
Note: I know my diffs are messed up - I have updated my .gitconfig to match the settings that deathbeam provided to me to fix this for any future commits.

Is this still coming or was it cancelled? Forgive my curiosity :D

Is this still coming or was it cancelled? Forgive my curiosity :D

This will probably have to be adjusted to the new statusbars i'm currently proposing, see #7730 it's a rework of the current statusbars with added functionality.

Is this ever making it to the live build? Its been almost a year and I still really want this feature, those bars still feel a bit in my face when using the interface "bottom line stones" and just skilling, etc.

To this day I still really want this option to be a reality, but if getting it to the main Runelite client is proving to be a challenge, could it perhaps be considered for the Plugin Hub?


docs: Mention that a kv-store is optional with k8s.

This change is

test-docs-please

Coverage decreased (-0.007%) to 45.507% when pulling e6ff6f1407468882537de0b2eefe4a153a0a1ced on pr/jrajahalme/docs-relax-kvstore-requirement into 9d8114f04c5dfd6e545e873f7eca665b35cff3f6 on master.

test-docs-please


[wallet-ext] new settings menu + lock account

Description
Describe the changes or additions included in this PR.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] protocol change
[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

@Nikhil-Mysten It will be helpful to update the description maybe with some screenshots as well


UF-295 - Upgrading lucene to 5.3.1

See also:

https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/pull/169
https://github.com/uberfire/uberfire/pull/337
https://github.com/uberfire/uberfire-extensions/pull/258
https://github.com/droolsjbpm/kie-wb-common/pull/337
https://github.com/droolsjbpm/jbpm-designer/pull/220
https://github.com/droolsjbpm/jbpm-form-modeler/pull/46
https://github.com/droolsjbpm/jbpm-console-ng/pull/395
https://github.com/droolsjbpm/drools-wb/pull/156
https://github.com/droolsjbpm/kie-wb-distributions/pull/261

+1 to merge (when @porcelli has commented on @mrietveld's question in https://github.com/uberfire/uberfire-extensions/pull/258)


Document "fast" insert

The example could be based on Plausible's in-memory buffer:

https://github.com/plausible/analytics/blob/master/lib/plausible/event/write_buffer.ex
https://github.com/plausible/analytics/blob/master/lib/plausible/session/write_buffer.ex
https://github.com/plausible/analytics/blob/master/lib/plausible/ingestion/write_buffer.ex

We use an ETS-based double buffer for our inserts to Kafka (not open source so I can't directly show you) but it is based on the buffer that OpenTelemetry uses for spans. Performance is really good. Maybe if I have some free time I could put together a PR for plausible implementing something like that?
https://github.com/open-telemetry/opentelemetry-erlang/blob/main/apps/opentelemetry/src/otel_batch_processor.erl

👋 @andyleclair
Oh, nice. I'll need to try it out and compare to our current approach. We can document both!


Added Muslim Aid bake sale fundraising event

Describe your changes
I added the recent bake sale for Muslim Aid to the fundraising section, and updated the fundraising total accordingly.
Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my changes
[x] I have made corresponding changes to the documentation and comments where neccessary
[x] If the changes affect the website, I have tested the branch using bundle exec jekyll serve on my local machine

@RalphLynch Can you please assign yourself as an assignee and Nathan or me to the PRs as reviewers? We won't see it otherwise. Thanks

Hi, sorry I took a while to get back to you, I've just been a bit busy with coursework. Also, apologies for not assigning you to the review earlier, it must have slipped my mind while making the PR.

@RalphLynch Thank you. Approved.


mr_list_test: mrListStateClosed fix

mrListStateClosed test is failing with not finding the output "!5 closed
mr".  This occurs because the output contains only the first 20 closed
MRs.  If all the closed MRs were listed the test will pass.
Modify the mrListStateClosed test to list all closed MRs.
Signed-off-by: Prarit Bhargava<EMAIL_ADDRESS>

This fixes the remaining test failure.

This fixes the remaining test failure.


docs: expand contributing documentation

Summary

works on expanding the contributing documentation a bit.
Checklist

[ ] If code changes were made, then they have been tested

[ ] I have updated the documentation to reflect the changes
[ ] I have formatted the code properly by running task lint
[ ] I have type-checked the code by running task pyright

[ ] This PR fixes an issue
[ ] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[x] This PR is not a code change (e.g. documentation, README, ...)

All in all a nice guide, though it should be split into one or multiple separate files instead of keeping everything in CONTRIBUTING.md, maybe even as part of the sphinx docs. Additionally, the larger codeblocks could be hidden in <details> blocks, though that won't be a necessity if the content is split into smaller chunks.

I think if we split it that might be a second pull request, not entirely sure.


[Social] Issue signin with apple

When I try to sign in with apple, I keep getting the following error:
response_mode must be form_post when name or email scope is requested.

this the authorization url:
https://appleid.apple.com/auth/authorize?client_id=XXX&response_type=code&redirect_uri=https://local.api.ambrosia.events/api/auth/callback/apple&scope=email%20name%20openid&state=XXX```

When I append `&response_mode=form_post` to the url manually, the error is no more. But then it fails on the callback url.

<img width="1343" alt="image" src="https://github.com/user-attachments/assets/654811e1-f9b0-44e8-afcb-ea7a40195092">

Hey can you please check if v0.8.2-beta.2 resolves the issues

Not working. You wanna add response_mode=form_post and keep response_type=code, or change it to response_type=code id_token
https://developer.apple.com/documentation/sign_in_with_apple/request_an_authorization_to_the_sign_in_with_apple_server

but again even after changing it manually on my end, I keep getting

apple is sending a POST request to the callback url with a state and code

changing this line to ["GET", "POST"] should do the trick I think.

@Bekacru I was able to move a little further on my end, and make it work.
changing callback.ts to
method: ["GET", "POST"],
		body: z.object({
			state: z.string(),
			code: z.string().optional(),
			error: z.string().optional(),
		}),

and updating any call from c.query to c.body.
But then this will break other oauth integrations.
Maybe updating better-call to add queryOrBody might solve that but I'm not sure if this the right route to go. or adding another metadata attribute that will take the body and pass it as query when post request?

Yeah, I was planning to see if there's a way to avoid the POST request (i.e., not using form_post as the response mode). But if that's necessary, we can make both the body and the query optional, and check inside the handler to make sure that at least one of them is provided.

I have a PR here if that's the direction we wanna go. https://github.com/better-auth/better-auth/pull/521/


Need to update GUI license that appears in the installer.

@chrisdembia Can this be closed or moved to Need verifying based on the auto generated builds? Also do we need to change again to include reference to JxBrowser? Thanks,

This cannot be closed yet, as #412 is not yet merged.

Also do we need to change again to include reference to JxBrowser?

This is what we say in #412 for the mac installer:

OpenSim includes a number of subcomponents with separate copyright notices and
license terms. Your use of these components is subject to the terms and
conditions of their respective licenses, which can be found in the LICENSE.txt
and NOTICE.txt files in the installation (<.app>/Contents/Resources/OpenSim).

I'm fine with adding the JxBrowser note here.
I'm not sure how to address this issue for the Windows build.

As of now the Windows build does not have an installer, just a zip file so the Notice file should be enough. If/when we create an installer we'll point it to the same file. Will leave open until we decide on windows installer.

@chrisdembia I think this can be closed since #412 has been merged.


[otbn,rtl] Fix overflow in stack_top_idx

We use next_stack_top_idx to derive next_top_valid_o, which says
whether the stack has data in it. This check is based on seeing
whether the index just above the top of the stack is nonzero or not.
We were using StackDepthW bits and got an overflow when the stack
filled up.
To see the bug (in a rather roundabout way), run e.g. util/dvsim/dvsim.py hw/ip/otbn/dv/uvm/otbn_sim_cfg.hjson --tool=xcelium -i otbn_single --fixed-seed<PHONE_NUMBER> with current master. The symptom is a failed assertion in the prefetcher, which turns out to be because we take a loop back-edge when the prefetcher doesn't think there's a loop. That, in turn, is because the prefetcher's notion of whether there's a loop or not comes from next_top_valid_o.

@prajwalaputtappa: You might want to grab this (it was causing ~1 failure in 20 for otbn_single)


KAFKA-14462; [13/N] CoordinatorEvent and CoordinatorEventProcessor

Adds CoordinatorEvent, CoordinatorEventProcessor, and MultiThreadedEventProcessor.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@kirktrue Thanks for your comments. I have addressed them.


Route Chassis request depending on OneView resource

I think we can use a query against the Index Service to figure out our route into OneView.  So we could do something like:
/rest/index/resources?filter=uuid=
This will return an array of members but the Resource in the array will have a category (ie server-hardware or enclosures) that we can use to build our route.

I could not find this mapped into hpOneView, so I opened an issue asking for help... :D

Hey @danielreed... a new hpOneView version with the index resource implementation has just been published. Are you going to implement this routing or is this card just related to the investigation?


Update README.md

Line 55, pod 'Alamofire', ~> '1.1' throws a CocoaPods error.
Proper format appears to be " pod 'Alamofire', '~> 1.1' "

:+1:
Thanks, @rborkow.


Make libvirtError public to allow clients to discern between different error types

This PR supersedes https://github.com/digitalocean/go-libvirt/pull/123 incorporating the feedback given in that PR.
This PR is needed to continue the experimental migration of terraform-provider-libvirt to pure-go, in order to be able to use the Terraform registry. https://github.com/dmacvicar/terraform-provider-libvirt/pull/813
What was done:

Rename the generated libvirt.Error to libvirt.remote_error
Rename libvirt.libvirtError to libvirt.Error
Expose all Error enum values
Expose only error Code and Message
(I am not convinced this is a good idea, but it was given as a guideline)

I regenerated the files against libvirt 7.1.0

Note:

I could still make .Message private, as it is accessible via .Error() too

// cc @MalloZup @kskewes @benlemasurier  @trapgate @vtolstov

@trapgate or your team/maintainers any chance you could review this ? that would help us a lot to move forward.. thank you in advance

thx @dmacvicar @trapgate


Add cost per token for APIs

I'm wondering if it would make sense to add the cost per token as metadata for embedding APIs so people can compare right at the leaderboard, which API solution might be the most cost effective for them.

IMO pricing can change often and without warning. It is also dated, so would that mean each time pricing is changed, it should be tracked? or will only the latest pricing is fetched and shown?
I'd defer this unless there's strong signals from user asks, as there are more and higher priority items for us to take on. Though I'd say this could be something contributed from the community.

I will close it for now, if someone wants to reopen it feel free to do so


KAFKA-6181 Examining log messages with {{--deep-iteration}} should show superset of fields

Printing log data on Kafka brokers using kafka.tools.DumpLogSegments --deep-iteration option doesn't print all the fields.
Adding missing fields in the log data on kafka brokers
Adding following fields in the deep-interation option:
baseOffset
lastOffset
baseSequence
lastSequence
producerEpoch
partitionLeaderEpoch
size
crc
thanks,
Nikhil
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

FAILURE
7975 tests run, 5 skipped, 2 failed.
--none--

SUCCESS
8083 tests run, 5 skipped, 0 failed.
--none--

SUCCESS
8083 tests run, 5 skipped, 0 failed.
--none--

Failure unrelated in the first build,
@guozhangwang @dguy @bbejeck @ybyzek
Can you please review my pull request?
Thank you

As an improvement, you could use string templating as it is done higher in the class.


It is not working on server

After reading the README and changing the admin textarea to summernote, it works very well when uploading image in local environment.
I tried applying the exact same code to the aws instance, but I get a 'Failed to save attachment' error in summernote's "SummernoteUploadAttachment".
I'm getting an error in the "#callback save method with attachment parameters as kwargs" section, but I'm not sure what the problem is.
The current server is in user - nginx - uwsgi - django format.

Did you run python manage.py migrate in production before uploading image files?

I just upgraded to <IP_ADDRESS> and I'm having the same problem as @harrimkim, uploads are not working and I'm getting an error in production.
My production setup is user - nginx -gunicorn -django. BUT if I try using the built-in Django web server instead of gunicorn on the production server uploads work again and I don't get the error. Any ideas?

@harrimkim Please reopen this again with more information and envs.

im not familar with github. how to reopen this issue?.
and @patabongo i dont think my problem is similar with you because i already check it before.

@harrimkim Could you share your AWS information? And if you're using https://sentry.io/, this would be the best option to catch error on production.


Lazy.nvim setup

I believe the lazy.nvim setup in the README is currently wrong.
This instead works as a basic setup
  {
    "nvim-zh/colorful-winsep.nvim",
    config = true,
    event = { "WinNew" },
  },

thank you


Fixes model usage in validation code.

Fixed model usage in validation.
Removed duplicate code in logging validation progress.
Fixed whitespaces.

@Liuftvafas thank you very much for your contributions! Would you mind sending a signed copy of the Contributor License Agreement as described there?
You're right that the usage of a function parameter nn_model and global variable model to designate the same object in Validation() is definitely inconsistent. Did you see an actual issue there, or as you suggesting this change merely to improve readability/robustness? If anything, I'd rather stick to using nn_model in this function as I think this would go in the direction of a cleaner separation of namespaces so that eventually we can move the code to another module, if we wish to.
Along the same lines, perhaps network, validationConfusion, validationBatchSize and valSize could be made function parameters. If I haven't missed any other globals, that would make the function Validation() independent of global variables, which I think would be a good thing. Then the pull request and commit could be renamed to something like "Refrain from using globals in Validation()".
Do you think you could submit the whitespace changes as a separate commit in order to make it easier to see where the actual change lies?
Also, do you think you could submit the in-place multiplication change in test.lua as a separate pull request (as this isn't well captured by the title of this pull request).

OK, I will update the code to use more function parameters both in Validation and Train functions to get rid of global variables.
I'm not sure how to remove the in-place multiplication from this pull request (I've already submitted it as separate pull request). Should I close this pull request and restart local/global variables refactoring from scratch in the new one?

Hey @Liuftvafas you can create a local branch of yours and cherry-pick the patches you want to submit in this Pull Request then force push to your Liuftvafas:master branch. You may do something like:
git checkout origin/master
git checkout -b dev_branch
git cherry-pick <SHA1>
git push --force https://github.com/Liuftvafas/DIGITS.git HEAD:refs/heads/master

I am re-opening this Pull Request so we keep the history of comments.


Bumped required psycopg2 version to 2.8.4.

psycopg2 2.8.4 is the first release to support Python 3.8.
I would like to backport this patch to the 4.0.x branch.

make sense

Thanks y'all for checking :+1:


Remove old getPort function

thaliMobile - Linting
thaliMobileNativeWrapper - This was originally going to be a 'universal'
function to hide the differences between Connect and MultiConnect but in
the end we decided to just have thaliMobile call directly to Connect and
MultiConnect since anyway they had different behaviors that we didn't think
were worth papering over.

This change is

PR is added to the queue for testing as 2. task. (f1f0fb5)

PR is added to the queue for testing as 3. task. (a3b8872)

Reviewed 1 of 2 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Test (Success) 86776290 build is completed (f1f0fb5)
See https://github.com/ThaliTester/TestResults/tree/86776290f1f0fb5_Remove_old_getPort_function_yaronyg/ for the logs

Test 86776290f1f0fb5(f1f0fb5) has failed
See https://github.com/ThaliTester/TestResults/tree/86776290f1f0fb5_Remove_old_getPort_function_yaronyg/ for the fail logs

Test (Success) 86776290 build is completed (a3b8872)
See https://github.com/ThaliTester/TestResults/tree/86776290a3b8872_Remove_old_getPort_function_yaronyg/ for the logs

Test 86776290a3b8872(a3b8872) has failed
See https://github.com/ThaliTester/TestResults/tree/86776290a3b8872_Remove_old_getPort_function_yaronyg/ for the fail logs


tls_leak_main_thread_allowed: make test check target_thread_local

Instead of ignoring the test entirely on some targets, make the test check the target_thread_local flag to determine whether thread_local! statics can be tracked by Miri and hence can have main-thread-TLS leaks ignored.

@bors r+

:pushpin: Commit e8778342e7533b823bf9728d25aba8fcc91eccb1 has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit e8778342e7533b823bf9728d25aba8fcc91eccb1 with merge ccb7f07a1d564d942ab297f29a2223f16a2fcfd7...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing ccb7f07a1d564d942ab297f29a2223f16a2fcfd7 to master...


Notes

Hey @disintegrator,
this looks really awesome!
I tried to improve it, but LinguiJS is missing few features:

to support reloading of messages without page refresh @lingui/loader need to receive a parameter, because for initial render we need compileNamespace=window.LINGUI_INIT, but for dynamically loading catalogs we need regular cjs or es export.

lingui extract --watch (or lingui watch) would be very useful to automatically extract messages.

i18n object could have method onLocaleRequest, where you could register callback (e.g. dynamic load of catalog) if locale is changed. Then we could use i18n.activate('en') to change active locale

So, I think for now you did the best what you can. I'll take a look next week how we could improve LinguiJS API to provide smoother implementation in SSR apps.

Can't believe I forgot to say this sooner... Thank you for checking this work out! It's really great to have your support 🙂

@disintegrator @tricoder42 Hi both!
Thanks so much for your work!

I've raised at PR at #3 that relies in a hack to fix the current example, but might be a helpful part of the journey to work out a better approach (it seems there was a change in Lingui recently which has broken this example, but this addresses it, in a slightly terrible way but that I think is helpful to highlight the problem).
What's great about Lingui and this example repo is that it works really well with both isomorphic rendering and serverless (and is much less opinionated that other projects, which have dependancies like express, which in turn impact bundle functions and being able to run serverless on the edge, etc).
Perhaps either Linqui could generate multiple outputs to facilitate easier isomorphic rendering + SPA support, or export a UMD module, or just a script that works in an isomorphic way by checking it's environment (I have had success with both approaches on different modules).

I'm happy to contribute to other items on this list if we can wrangle the this example back to a working state! I'm also hoping there is something I am missing about exports that someone can suggest when they see the problem. :-)

BTW, as announced at ReactEurope by @timneutkins, Next.js will have /api/ route support, which I think will be great for making it easy to facilitate loading catalogs in a Single Page App.
e.g. You could call an endpoint via fetch and have your app fetch and load a new catalog at run time without an app reload.
When this is working again (and the functionality settled) it would be great to see this example in next.js/examples, especially as it works so well with serverless deployments.


Facing error while installing another package or update

I am using Laravel 7.

Facing same issue. Tried in Laravel 6 and 8 same error


Fix memory bloat in CI

What was wrong?
The DeprecationWarnings in the Hardfork.discover method were causing significant memory bloat resulting in a crash in the CI.
How was it fixed?
Migrating to the latest APIs for loading the modules removes these warnings.
Cute Animal Picture

Temporarily disabled the optimised tests in CI as we wait for the updated optimised implementation for cancun. See issue


Swiping doesn't work

I believe this is because the X-Auth-Token header doesn't get set.
Please don't ask how I found this out

It seems that X-Auth-Token does get set right in the beginning, when the code authenticates. Nonetheless the swipes do not work, an error {"status":"401","error":""} appears.


Delete the Operator’s ClusterServiceVersion (CSV) mentions namespace, but does not apply here

"
Step 3: Delete the Operator’s ClusterServiceVersion (CSV)
The CSV contains all the information that OLM needs to manage an operator, and it effectively represents an operator that is installed on cluster. By deleting a ClusterServiceVersion, OLM will delete the resources it created for the operator such as the deployment, RBAC, and any corresponding CSVs that OLM “Copied” into other namespaces watched by the operator.
If you wish to look up a list of ClusterServiceVersion in a specific namespace to see which ClusterServiceVersion you need to delete, you can use the example kubectl command:
$ kubectl get clusterserviceversion -n
Example output
NAME                        DISPLAY              VERSION   REPLACES                    PHASE
foo                         Foo Operator         1.0.0                                 Succeeded
You can delete the ClusterServiceVersion in the namespace that the operator was installed into using this command:
kubectl delete clusterserviceversion  -n
"
this is a cluster object. namespace can be omitted here correct?

If i read https://docs.openshift.com/container-platform/4.6/operators/operator_sdk/osdk-generating-csvs.html#osdk-suggested-namespace_osdk-generating-csvs
it mentions OLM is default namespace for the operator, and if you want the operator to be deployed in an alternative namespace there is option:

metadata:
  annotations:
    operatorframework.io/suggested-namespace: <namespace>

you might be correct though, however, also in regards of the naming of the object, where other objects starting with Cluster are Cluster wide and not per namespace.
If what you say is correct, than whoever chose the naming of this object, chose it in a manner that is pretty confusing.

But It seems you are correct. closing this. ❌


Why can't we rewrite this pattern like this?

for (int i=0; i < retryCount; i++)
  {
    try
    {
      // Call external service.
      var status = await TransientOperationAsync();

      if(status == 'success')
           return true;

       throw;
    }
    catch (Exception ex)
    {
      currentRetry++;

      if (currentRetry > this.retryCount || !IsTransient(ex))
      {
        // If this isn't a transient error or we shouldn't retry,
        // rethrow the exception.
        throw;
      }
    }

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

ID: 497c1436-d32c-1fb5-3209-22058fece4b2
Version Independent ID: 4818d7be-c866-39bc-f2c3-d7af59d2cd4e
Content: Retry
Content Source: docs/patterns/retry.md
Service: guidance
GitHub Login: @dragon119
Microsoft Alias: pnp

The first throw isn't inside a catch block. Also you have a possible case where i == retryCount, and you don't re-throw the exception but you exit the loop, swallowing the exception.

My idea is to replace infinite loop (;;) with  finite loop(int i=0; i < retryCount; i++)
@MikeWasson , Thanks for the Answer, however I got same output for both the method.


Added GSSOC'23 banner and back to top part

Related Issue

This is with reference to issue #160

Proposed Changes

Changing the banner in README to GSSOC'23
Adding back to top feature at the end of the README

Checklist

[x] ✅ My code follows the code style of this project.
[x] 📝 My change requires a change to the documentation.
[x] 🎀 I have updated the documentation accordingly.
[x] 🌟 ed the repo

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Summary of the Changes
Previously the README had banner  that displayed  "Hacktoberfest," which is incorrect for the project. It needs to be updated to reflect "GSSoC 23," aligning with the GirlScript Summer of Code (GSSoC) branding. Hence, I updated it to the GSSOC'23 Banner .
Added the extra feature of Back to top feature at the end of README, so that it is easy for users to navigate through the README without scrolling much.

Link to the Issue : https://github.com/agamjotsingh18/codesetgo/issues/160#issue-1722127713
Output Screenshots
| Screenshot #1 |
|

|Updated Banner|
|Screenshot #2 |

| Added back to top at the end of README|

Can you add the images too

Can you add the images too in the readme

Can you please specify which images you want me to add to the README

Website snippet, you can take the current one which is present in the readme

Website snippet, you can take the current one which is present in the readme

like should I cover all the pages or just the home page is enough?? And should it be like a walk through

Just take the one that is present currently

Just take the one that is present currently

okayy will work on that and it will cover all the sections of the website right??

Anything else to add? @agamjotsingh18


Failed to load mods

On Manjaro Linux navigating to the Mods tab it just shows "Failed to load mods" with no additional info anywhere. I ran it through a terminal to check if that shows anything but no errors there either.

I had the same issue. Downgrading the version to 1.29.1 (Properties -> Betas -> legacy1.29.1) fixes it.

@GalecO1987  What if I got the app from Meta Quest Link? Can't find any downgrade option


convert quantities before they might be strings in to_table

trying to u.Quantity with a string, for example, u.Quantity([m42.ra, '']),
yields a TypeError:
    print(u.Quantity([m42.ra, '']))
  File "~/.local/lib/python3.8/site-packages/astropy/units/quantity.py", line 382, in __new__
    raise TypeError("The value must be a valid Python or "
TypeError: The value must be a valid Python or Numpy numeric type.

this is exactly what happens in to_table() with transitions or unused slots
shown, because of the append append('') in scheduling.py.
so instead, do the u.Quantity conversion earlier where we know the type is
(should be!) a Numpy numeric (or other appropriate type).
this is, in a way, a rewrite of #370: really just moves the change there up a few lines. i tested that it works by hand on a table with TransitionBlock; what would yield the aforementioned TypeError on master now produces a table that can be printed:
     target         start time (UTC)         end time (UTC)      duration (minutes)      ra         dec      configuration
--------------- ----------------------- ----------------------- ------------------- ----------- ----------- ----------------
            M42 2021-02-14 01:30:00.000 2021-02-14 02:23:20.000  53.333333333333414 83.82208333 -5.39111111 {'filter': 'Ha'}
TransitionBlock 2021-02-14 02:23:20.000 2021-02-14 02:23:29.136 0.15226966667478692                                       []
             M1 2021-02-14 02:23:29.136 2021-02-14 03:43:29.136   80.00000000000003 83.63308333     22.0145 {'filter': 'Ha'}
TransitionBlock 2021-02-14 05:53:29.136 2021-02-14 05:53:59.688   0.509194929995882                                       []
            M51 2021-02-14 05:53:59.688 2021-02-14 07:13:59.688   80.00000000000003  202.469575 47.19525833               {}

i'm not sure how to run the tests here, or i'd go ahead and add a test that to_table should work with examples from the docs :D

This looks great, thanks @iximeow!


feat: support dry run create and update policy and provider

Closes: https://github.com/odpf/guardian/issues/128
Proton PR: https://github.com/odpf/proton/pull/221

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 52 (63.46%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.006%) to 74.875%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

api/handler/v1beta1/policy.go
1
3
33.33%

api/handler/v1beta1/provider.go
2
6
33.33%

core/policy/service.go
5
11
45.45%

core/provider/service.go
25
32
78.13%

Files with Coverage Reduction
New Missed Lines
%

core/policy/service.go
1
62.7%

Totals

Change from base Build<PHONE_NUMBER>:
-0.006%

Covered Lines:
6124

Relevant Lines:
8179

💛  - Coveralls