Print type tags in Debug::print

Motivation
easier to read
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
WIP

Incomplete but I'm trying to print strings in Debug-- i need to create some test cases

I think @vgao1996 (who added the VM debug printing logic IIRC), @runtian-zhou, or @tnowacki will be able to suggest a way to test this.

I don't have a nice suggestion for testing this at the moment. IIUC the debug printing logic is gated under the testing flag so you definitely can perform a one-shot testing by using --features testing for cargo. I'm not sure how to write a persistent unit test for this. Maybe @vgao1996 may have a better clue?

Sorry for the late reply and I welcome this change!
You can add your new test cases to https://github.com/move-language/move/tree/main/language/tools/move-cli/tests/sandbox_tests/print_values and run env UPBL=1 cargo test to update the .exp files. This should also take care of your existing test failure.

Is this still active?


warp10-quantumviz broken in Firefox

Here's a graph in Firefox:

And here's the same graph in Chromium:

The code is:
<!DOCTYPE html>
<html>
    <head>
        <meta charset="utf-8">
        <title>Infra dashboard</title>
        <script src="https://cdn.cityzendata.net/quantumviz/latest/webcomponentsjs/webcomponents-lite.js"></script>
        <link rel="import" href="https://cdn.cityzendata.net/quantumviz/latest/warp10-quantumviz/warp10-quantumviz.html">
    </head>
    <body>
        <warp10-quantumviz host="https://warp10.gra1.metrics.ovh.net/api/v0">
            [My WarpScript]
        </warp10-quantumviz>
    </body>
</html>

This bug has been witnessed just now on Firefox Quantum (57.0.1, GNU/Linux) with a blank profile.

OK, I've just replicated the bug, going to deal with it

Fixed on 2.1.0-b6. There is still a problem with c3 implementation, we will need to see it with @Giwi but for the moment I revert to DyGraphs implementation.

Awesome, thanks! 😁


Session: don't return undefined if a response is required

Sequel to #16773
This doesn't yet fix SignatureHelp, since that doesn't return an array. I'm not sure if VSCode is depending on us sending back an "error" response if the user isn't currently at a signature.

Looks like the build is failing. Can you take a look?

Fix is #17685, will merge once that's in

@rbuckton Build is passing now, good to go?

@andy-ms It might be safest to revert this (and hold off on https://github.com/Microsoft/TypeScript/pull/17727) until we further discuss the implications for clients. (cc: @minestarks, @rbuckton)

Might be better to wait for release-2.5 and remove this from that, but not from master.

I think release-2.5 exists now.  The plan is to revert this there and submit https://github.com/Microsoft/TypeScript/pull/17728 to master (and close https://github.com/Microsoft/TypeScript/pull/17727)?


Fix ref-schema imports

Handle sh:prefixes when importing the ref-schema

The one thing that seems to have been dropped is the ontology declaration.
This causes (me) problems with imports, since there is no name for the graph. Easy enough to fix, right?
In Brick.ttl I see:
<https://brickschema.org/schema/Brick> sh:declare [ sh:namespace "https://brickschema.org/schema/Brick#"^^xsd:anyURI ;
            sh:prefix "brick" ],
        [ sh:namespace "http://www.w3.org/ns/shacl#"^^xsd:anyURI ;
            sh:prefix "sh" ],
        [ sh:namespace "http://data.ashrae.org/standard223#"^^xsd:anyURI ;
            sh:prefix "s223" ],
        [ sh:namespace "http://www.w3.org/2002/07/owl#"^^xsd:anyURI ;
            sh:prefix "owl" ],
        [ sh:namespace "https://brickschema.org/schema/Brick/ref#"^^xsd:anyURI ;
            sh:prefix "ref" ],
        [ sh:namespace "http://www.w3.org/1999/02/22-rdf-syntax-ns#"^^xsd:anyURI ;
            sh:prefix "rdf" ],
        [ sh:namespace "http://www.w3.org/2000/01/rdf-schema#"^^xsd:anyURI ;
            sh:prefix "rdfs" ] .


Import Asahi JW March

@horak could you check to see if Asahi has its March data?
If not, please let me know, I will contact them.
It is April now, and they mentioned about changing their policy on April 1st.

Looks like they still uploaded it, I backed it up and will handle the upload next week.

Thanks Alex. I will email Ashahi later in this month to find out what we need to do after this last batch.


Set up a staging environment for app

GitHub Pages only hosts from a single branch.  Set up a new repo for NIEM Toolbox staging, so in-progress features can be tested without disrupting the production tool.

Using fork as staging site: https://cdmgtri.github.io/niem-toolbox/


Pipe stdout to stdin

Hi,
I am trying to do this:
func main() {
      output, error := exec.Command(
            "expect.exe",
            "test.lua"
      ).Output()

      fmt.Println(string(out))
}

It seems like the main.exe does not print the output, but freezes there. Hence I would like to know if expect.go is able to pipe the stdout to stdin of another process?

os/exec".Cmd.Output() waits until the command terminates.
And expect() watches the console directly and ignores stdout and stderr.
You had better to use "os/exec".Cmd.StdoutPipe().
https://golang.org/pkg/os/exec/#Cmd.StdoutPipe

Thanks. I will try it in a short while and let you know the result.

Hi,
I wrote a very simple and lousy to test your POC. It didn't print out the stdout.
package main

import (
	"fmt"
	"os/exec"
	"time"

)

func main() {

	cmd := exec.Command(
		"expect.exe",
		"test2.lua",
		"<IP_ADDRESS>",
	)

	readStdout := make([]byte, 65536)

	stdout, err := cmd.StdoutPipe()

	if err != nil {
		panic(err)
	}
	startErr := cmd.Start()

	if startErr != nil {
		panic(startErr)
	}

	read, ioError := stdout.Read(readStdout)

	if ioError != nil {
		panic(ioError)
	}

	//if waitError := cmd.Wait(); waitError != nil {
	//	panic(waitError)
	//}
	time.Sleep(time.Second*10)
	fmt.Println(string(read))

}

stdout.Read() may wait until expect.exe outputs 65536 bytes data or closes its stdout.
And I am afraid that It does not work as expected whether it is fixed or not.
The line without CRLF(= password prompt) may not be output(flushed) by plink.exe when stdout is redirected.
Because it is buffered until CRLF is written in plink.exe.
So my function expect() sees console directly (not redirected stdout) .


Replying to a comment and a post is not working

Replying to a comment and a post is not working

Fixed in 4.0.1 and now in the JSON Configuration as well :)

Nice! Just implemented this update and it works like a charm!
Thank you!
Juan Carlos Chomali
iOS & Web Developer
On Mon, Feb 23, 2015 at 2:12 PM, Ben Gordon<EMAIL_ADDRESS>wrote:

Fixed in 4.0.1 and now in the JSON Configuration as well :)
Reply to this email directly or view it on GitHub:
https://github.com/bennyguitar/libHN/issues/31#issuecomment-75573330

No problem man, you should be using 4.0.2 now since I noticed a problem with HNComments of the Job type.


Guice creation errors, failed to proxy mesosphere.marathon.MarathonScheduler to support a circular dependency

Environment:
Marathon: 0.9.1
Mesos: 0.23.0
Zookeeper: 3.5.0-alpha
3 AWS EC2 instances(OS: centos 7.0)
I deploy this environment on AWS using Chef. After the deployment completes, the Marathon service on at least one EC2 instance(not limited to the EC2 instance has the leader of Mesos Master) is not available. I cannot access the RESTful API of Marathon.
From the log("/var/log/message")
Aug 12 06:32:32 vos-node-03 marathon[2559]: [2015-08-12 06:32:32,282] INFO Session establishment complete on server localhost/0:0:0:0:0:0:0:1:2181, sessionid = 0x34f209c213e0007, negotiated timeout = 40000 (org.apache.zookeeper.ClientCnxn:1235)
Aug 12 06:32:32 vos-node-03 marathon[2559]: [2015-08-12 06:32:32,428] INFO Registering in Zookeeper with hostPort:vos-node-03:8080 (mesosphere.marathon.MarathonModule:207)
Aug 12 06:32:32 vos-node-03 marathon[2559]: [2015-08-12 06:32:32,582] INFO Now standing by. Closing existing handles and rejecting new. (mesosphere.marathon.event.http.HttpEventStreamActor:100)
Aug 12 06:32:32 vos-node-03 marathon[2559]: [2015-08-12 06:32:32,584] INFO http_endpoints(List(http://<IP_ADDRESS>:8088/api/v1/marathon_event/)) are specified at startup. Those will be added to subscribers list. (mesosphere.marathon.event.http.HttpEventModule:61)
Aug 12 06:32:32 vos-node-03 marathon[2559]: Exception in thread "main" com.google.inject.CreationException: Guice creation errors:
Aug 12 06:32:32 vos-node-03 marathon[2559]:
Aug 12 06:32:32 vos-node-03 marathon[2559]: 1) Tried proxying mesosphere.marathon.MarathonScheduler to support a circular dependency, but it is not an interface.
... ...

More detail log please refer to https://gist.github.com/howardzhang/02c8184f803d3effefa9
Does anyone get any information on this?

Hi @howardzhang, thanks for reporting this.
Can you tell us the command line parameters that Marathon is started with?

Hi @kolloch, thanks for your attention.
Below is the command line parameters.
/usr/bin/marathon mesosphere.marathon.Main --event_subscriber http_callback --http_endpoints http://<IP_ADDRESS>:8088/api/v1/marathon_event/ --zk zk://localhost:2181/marathon --master zk://localhost:2181/mesos

Hi @howardzhang, we have released two release candidates that should fix your problem:

v0.10.1-RC1
v0.9.3-RC1

It would be great if you could confirm if these fix your issue.
Thanks a lot!

Danke Peter!  Wollte ich am Montag machen - die Message in Slack war nur eine Info, falls einer fragt. Wir müssen nochmal diskutieren, ob so ein fix einen RC Cycle durchlaufen muss.
Schönes Wochenende!
Matthias

Am 14.08.2015 um 15:57 schrieb Peter Kolloch<EMAIL_ADDRESS>Hi @howardzhang, we have released two release candidates that should fix your problem:
v0.10.1-RC1
v0.9.3-RC1
It would be great if you could confirm if these fix your issue.
Thanks a lot!
—
Reply to this email directly or view it on GitHub.

Wollte keine "offizielle" Release vor dem WE machen.
On Fri, Aug 14, 2015 at 4:14 PM, Matthias Veit<EMAIL_ADDRESS>wrote:

Danke Peter! Wollte ich am Montag machen - die Message in Slack war nur
eine Info, falls einer fragt. Wir müssen nochmal diskutieren, ob so ein fix
einen RC Cycle durchlaufen muss.
Schönes Wochenende!
Matthias

Am 14.08.2015 um 15:57 schrieb Peter Kolloch<EMAIL_ADDRESS>Hi @howardzhang, we have released two release candidates that should fix
your problem:
v0.10.1-RC1
v0.9.3-RC1
It would be great if you could confirm if these fix your issue.
Thanks a lot!

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/mesosphere/marathon/issues/1984#issuecomment-131118029
.

Hi @kolloch, Great. I will let you know if this issue occurs again.
And thanks for the quick fix.


[Community Vote] Removing HttpRouter or Gorilla/mux from Go-Blueprint

Tell us about your feature request
The project is getting pretty big, and we want to remove one framework, so we are leaving it up to the community to decide which one should be removed. Please give us your thoughts and votes:
HttpRouter 👍
Gorilla/mux  👎
After 3 weeks, the poll will be closed, and one of the frameworks will be sent to oblivion.
Disclaimer

[X] I agree

I can't vote for either because it depends on the use case. Both frameworks require implementation from scratch, unlike others that come with everything pre-built, allowing you to focus solely on your specific web development needs.


Fix filter the policy messages

Signed-off-by: Zixuan Liu<EMAIL_ADDRESS>Motivation
Message<PulsarEvent>.getValue() sometimes returns null in SystemTopicBasedTopicPoliciesService.notifyListener(), so we need to skip the messages that do not belong to the policy type, this problem can cause the policy service to fail to work.
Modifications

Checks the Message<PulsarEvent>.getValue() value.

Verifying this change

[x] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): no
The public API: no
The schema: no
The default values of configurations: no
The wire protocol: no
The rest endpoints: no
The admin cli options: no
Anything that affects deployment: no

Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?

[ ] doc-required
[x] no-need-doc
[ ] doc

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Windows: refactor initialization of OpenGL context into its own function

Closing since OpenGL is now beyond the scope of baseview.


get-devicemanagement_devicecompliancepolicysettingstatesummaries returns Setting Name as Setting and ID

This is what the command returns with no parameters:
id                       : DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned
setting                  : DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned
settingName              : DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned
platformType             : all
unknownDeviceCount       : 0
notApplicableDeviceCount : 0
compliantDeviceCount     : 7
remediatedDeviceCount    : 0
nonCompliantDeviceCount  : 0
errorDeviceCount         : 0
conflictDeviceCount      : 0

The response section of the docs says the ID should be a UID: https://docs.microsoft.com/en-us/graph/api/intune-deviceconfig-devicecompliancepolicysettingstatesummary-get?view=graph-rest-1.0#response-1
{
  "value": {
<EMAIL_ADDRESS>"#microsoft.graph.deviceCompliancePolicySettingStateSummary",
    "id": "7474d6d5-d6d5-7474-d5d6-7474d5d67474",
    "setting": "Setting value",
    "settingName": "Setting Name value",
    "platformType": "iOS",
    "unknownDeviceCount": 2,
    "notApplicableDeviceCount": 8,
    "compliantDeviceCount": 4,
    "remediatedDeviceCount": 5,
    "nonCompliantDeviceCount": 7,
    "errorDeviceCount": 0,
    "conflictDeviceCount": 3
  }
}

Oh fun. Graph Explorer returns the same. The value returned for ID doesn't work appended to the url either.
        {
            "id": "DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned",
            "setting": "DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned",
            "settingName": "DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned",
            "platformType": "all",
            "unknownDeviceCount": 0,
            "notApplicableDeviceCount": 0,
            "compliantDeviceCount": 7,
            "remediatedDeviceCount": 0,
            "nonCompliantDeviceCount": 0,
            "errorDeviceCount": 0,
            "conflictDeviceCount": 0
        },

https://graph.microsoft.com/beta/deviceManagement/deviceCompliancePolicySettingStateSummaries/DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned and https://graph.microsoft.com/v1.0/deviceManagement/deviceCompliancePolicySettingStateSummaries/DefaultDeviceCompliancePolicy.RequireDeviceCompliancePolicyAssigned both fail, suggesting that's not the ID that should be provided - is Graph itself broken then?


Unable to hydrate a foreign key in relation

Hello,
I have this database schema (link to db diagram).
Each tables has composite (compound) primary keys with foreign keys.
I would like to create fixtures for the Table C.
According to the documentation, I guess that one solution would be:
App\Entity\TableC:
  TableC{1..100}:
    cpk1: <numberBetween()>
    bfk1: '<current()>->tableB->bfk1'
    bfk2: '<current()>->tableB->bfk2'
    afk1: '<current()>->tableB->afk1'
    afk2: '<current()>->tableB->afk2'
    tableB: '@TableB<current()>'

But that is not working.
In order to get it working, I have:

Add a relation to Table A in Table C:

    /**
     * @ORM\ManyToOne(targetEntity=TableA::class)
     * @ORM\JoinColumns(
     *     @ORM\JoinColumn(name="AFK1", referencedColumnName="APK1"),
     *     @ORM\JoinColumn(name="AFK2", referencedColumnName="APK2"),
     * )
     */
    public $tableA;

Update the fixture:

App\Entity\TableC:
  TableC{1..100}:
    cpk1: <numberBetween()>
    bfk1: '<current()>->tableB->bfk1'
    bfk2: '<current()>->tableB->bfk2'
    afk1: '<current()>->tableB->afk1'
    afk2: '<current()>->tableB->afk2'
    tableB: '@TableB<current()>'
    tableA: '@TableA<current()>'

To be honest, I don't like this solution, this is why I found it ugly (see commit).
I succeeded to do it successfully for Table A and B (see PR) but not for Table B and C. I'm trying to find a better solution, but I'm running out of ideas.

Whilst not impossible I don't think the solution is ideal. You are running into a problem here with Alice because it alice does two very distinct steps: creating the objects and then persisting them.
As a result, when you need to work with DB auto-generated IDs you are screwed: because you rely on the object to be persisted in a specific order before being able to manipulate it to create another object.
So usually there is two fixes:

Do what you do but with non DB auto-generated IDs, e.g. pre-defined UUIDs
Do not use IDs but objects, i.e; you don't base tableB->bfk1 to TableC but tableB itself; I'm however not sure how you tell Doctrine that you want to persist only the ID for it though...

Whilst not impossible I don't think the solution is ideal. You are running into a problem here with Alice because it alice does two very distinct steps: creating the objects and then persisting them.
As a result, when you need to work with DB auto-generated IDs you are screwed: because you rely on the object to be persisted in a specific order before being able to manipulate it to create another object.
So usually there is two fixes:
* Do what you do but with non DB auto-generated IDs, e.g. pre-defined UUIDs

* Do not use IDs but objects, i.e; you don't base `tableB->bfk1` to `TableC` but `tableB` itself; I'm however not sure how you tell Doctrine that you want to persist only the ID for it though...

Dear @theofidry,
There are no auto-generated IDs.
See the SQL dump of the schema: https://del.dog/domippafir.txt
Do you have a code example under the hand? Or a link to documentation on this?

Based on your input, I did the following changes:
diff --git a/fixtures/tableb.yaml b/fixtures/tableb.yaml
index dfc375d..6ae46f1 100644
--- a/fixtures/tableb.yaml
+++ b/fixtures/tableb.yaml
@@ -2,6 +2,6 @@ App\Entity\TableB:
   TableB{1..100}:
     bpk1: <numberBetween()>
     bpk2: <numberBetween()>
-    afk1: '<current()>->tableA->apk1'
-    afk2: '<current()>->tableA->apk2'
+    afk1: '<current()>->tableA'
+    afk2: '<current()>->tableA'
     tableA: '@TableA<current()>'
diff --git a/fixtures/tablec.yaml b/fixtures/tablec.yaml
index 6dba5db..d273e14 100644
--- a/fixtures/tablec.yaml
+++ b/fixtures/tablec.yaml
@@ -1,8 +1,8 @@
 App\Entity\TableC:
   TableC{1..100}:
     cpk1: <numberBetween()>
-    bfk1: '<current()>->tableB->bfk1'
-    bfk2: '<current()>->tableB->bfk2'
+    bfk1: '<current()>->tableB'
+    bfk2: '<current()>->tableB'
     # For afk1 and afk2, we should be able to use:
     # '<current()>->tableB->afk1'
     # without creating the field $tableA in Table C, because those fields
@@ -10,7 +10,6 @@ App\Entity\TableC:
     # In order to get it working, I added the field $tableA in Table C
     # and it seems to work. I don't like at all that solution, I'm looking
     # for an alternative.
-    afk1: '<current()>->tableB->afk1'
-    afk2: '<current()>->tableB->afk2'
+    afk1: '<current()>->tableB'
+    afk2: '<current()>->tableB'
     tableB: '@TableB<current()>'
-    tableA: '@TableA<current()>'
\ No newline at end of file
diff --git a/src/Entity/TableC.php b/src/Entity/TableC.php
index dad4f54..2ccefca 100644
--- a/src/Entity/TableC.php
+++ b/src/Entity/TableC.php
@@ -68,13 +68,4 @@ class TableC
      * )
      */
     public $tableB;
-
-    /**
-     * @ORM\ManyToOne(targetEntity=TableA::class)
-     * @ORM\JoinColumns(
-     *     @ORM\JoinColumn(name="AFK1", referencedColumnName="APK1"),
-     *     @ORM\JoinColumn(name="AFK2", referencedColumnName="APK2"),
-     * )
-     */
-    public $tableA;
 }

Then I recreated the schema and created fixtures.
relation between Table A and B seems to work pretty fine. However, between B and C:

As you can see, the data in columns AFK1 and AFK2 in TABLE_C are wrong. They are supposed to contains random numbers, not indexed numbers.


Option to only download samples / thumbnails?

Although I've searched all the options in Grabber I wasn't able to find a "download thumbnails" or "download samples" option. I am a deep-learning researcher and the ability to download low-quality alternatives is very convenient when batch downloading > 20k images.
It would be awesome if this could be supported officially.
Excuse me if this feature already exits and I was incapable of finding it,
Can
Temporary solution for windows users:
Byte-patch [Grabber.exe + 0xD7507] to 0xC
(Build: 25 May)

If I may ask, What are you trying to teach an AI with images downloaded from boorus?

@brodycas3 auto tagging pictures & feature extraction.
So far in my experiments I was able to do searches in untagged pictures like:
get_vector("blond") + get_vector("fang") or get_vector(picture) - get_vector("naked"). Quite fun actually.

That seems pretty interesting.

@can1357 maybe you will be interested about already existed tag logging feature https://github.com/Bionus/imgbrd-grabber/issues/874
and do you heard about Hidrus project?(Sorry, Bionus) It's anime DB with cloud tags storage, marking by community. There are exist almost 15 GB hash-tags info(downloads separately) about almost all images from all boorus
https://github.com/hydrusnetwork/hydrus
And moreover, there is already existing anime images tag predicting deep AI neetwork:
http://demo.illustration2vec.net/#
And it works almost perfect:

@MasterPetrik
I didnt know about the tag logging feature nor about the Hidrus project; good to know about!
I'll definitely let you know if I have some notable result.
Illust2Vec seems to work quite good though it's quite impressive.

looks like they even giving they Pre-trained model for free:
http://illustration2vec.net

There is no current way to download thumbnails. There is an option in the settings to download samples but I can't guarantee you it's working properly on all source.
I could add a new CLI option to download or return the URLs of thumbnail/samples though. Depending on the sources you're using, you could also directly build these URLs from the full image URL.
If the source is danbooru-based for example, the image https://danbooru.donmai.us/data/93e6d0d665c373bd6c5c9df8088b501e.jpg will have a thumb located at https://danbooru.donmai.us/data/preview/93e6d0d665c373bd6c5c9df8088b501e.jpg: simply replace /data/ with /data/preview/. Same thing for samples, except you have to replace /data/ with /data/sample/sample-, and not all images have samples.

The bytepatch I do changes:
https://github.com/Bionus/imgbrd-grabber/blob/88bfbd66b2693387f05bf0f33d2bb7ad8737dd2a/lib/src/models/image.cpp#L203
m_url = m_fileUrl.toString(); -> m_url = m_previewUrl.toString();
and it seems to work fine for most images. It'd be really cool if you could add it @Bionus.
Where's the download samples option thought I cant seem to be able to find it 😕

The bytepatch I do changes

You also have m_sampleUrl if I remember correctly.

It'd be really cool if you could add it @Bionus.

I'll add it as a CLI option in the next update. Since it seems you're building from source (:question:) you can grab the develop branch when you see one of my commits referenced in this issue.

Where's the download samples option thought I cant seem to be able to find it 😕

If you don't check the first box ("Download original images") it should download samples when available.

I was byte-patching the already-built executable file because I do not like to deal with Qt.
Anyway, thanks a lot for considering the addition of this feature :)


Check to see if KeyboardEvent.code is supported

R: @beaufortfrancois etc.
Just to make it play a little nicer on unsupported browsers.

:shipit:


Enable issue template for better creation of issues

What changes are proposed in this pull request?
Enable issue template for better creation of issues
How is this patch tested?
N/A
Release Notes
Is this a user-facing change?

[x] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release notes for MLflow users.

(Details in 1-2 sentences. You can just refer to another PR with a description if this PR is part of a larger change.)
What component(s), interfaces, languages, and integrations does this PR affect?
Components

[ ] area/regression: Sklearn regression pipeline example
[x] area/build: Build and test infrastructure for MLflow pipeline example

How should the PR be classified in the release notes? Choose one:

[ ] rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
[x] rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
[ ] rn/feature - A new user-facing feature worth mentioning in the release notes
[ ] rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
[ ] rn/documentation - A user-facing documentation change worth mentioning in the release notes

Could you add examples how these templates look like?

It should be similar to one from mlflow: https://github.com/mlflow/mlflow/issues/new/choose
Just removed some sections that didn't seem relevant.

I'm a bit hesitant to duplicate the MLflow issue guideline here. Can you make changes to guide users to report to the main MLflow repo about execution issues? It's easier to track instead of asking the OSS team to look MLP templates for issue tracking as well. Imagine we have 100 templates published next year. Who's going to check all those template repos for issue report?

+1


Export quantized model

Hi Neta,
I looked into the doc and find out that in https://nervanasystems.github.io/distiller/design/index.html#quantization, it is mentioned 'We also provide a mechanism which takes an existing model and automatically replaces required operations with quantized versions.' Does this mean we can export this model with such 'quantized versions' operation?
If it is not supported in current Distiller, can you kindly suggest how I can export it?
Thanks so much.

Hi Neta,
I tried to export a non-quantized model to ONNX as advised, however, got a RuntimeError 'ONNX export failed: Couldn't export operator aten::softmax'.
I know this layer is added in Distiller, and I checked the corresponding 'torch/onnx/symbolic.py', the 'softmax' definition is there. Do you have any idea about this error? Thanks.

Hi @cattpku ,
Please share your command-line, so I can try to recreate this problem.
I think that using
softmax = torch.nn.Softmax(dim=-1)

is probably more correct, but I want to see why it fails for you before making changes to the code.
BTW, you can also try to change it to -1 and report if this fixes the fail you see.
Thanks
Neta

Hi Neta,
I am not using command-line, as I put the whole code in Pycharm  to run it. I just set the following args:
--data:  my own dataset path
--arch:  my own network definition
--summary:  default = 'onnx'
I tried to change it to -1 as suggested, it went well, no error occurred.


Validate cluster role ID

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #3034

License
Apache 2.0

What's in this PR?
Validate cluster role ID in case of EKS
Checklist

[x] Implementation tested (with at least one cloud provider)
[x] Code meets the Developer Guide

Tested this locally, it works as expected.


List Navigation in Mac

Describe the bug
List Navigation is not working in Mac
To Reproduce
Install mac app, press cmd + 1-9, nothing happened.
Expected behavior
Jump to specific list.
Screenshots

Technical Info (please complete the following information)

OS: Mac
Ao Version: 5.6.0

Additional context
Add any other context about the problem here.

Issue fixed on f25061c.
New 6.0.0 release is now out. Full changelog & binaries are available on the GitHub Releases page.


ESP32-S3-BOX-3

Since yesterday HA/Esphome wants me to upgrade to 2023.11.3 but current GitHub version is .2
There is a typo in the yaml :
package_import_url: github://esphome/firmware/voice-assistant/esp32-s3-box.yaml@main
It should be
package_import_url: github://esphome/firmware/voice-assistant/esp32-s3-box-3.yaml@main
Then when trying to update , it t also throw errors about the Casita PNG files not found , it works when installing through Esphome.io/USB .

Update error log :
INFO ESPHome 2023.11.3 INFO Reading configuration /config/esphome/esp32-s3-box-3-5ab118.yaml... INFO Updating https://github.com/esphome/esphome.git@pull/5230/head WARNING GPIO0 is a strapping PIN and should only be used for I/O with care. Attaching external pullup/down resistors to strapping pins can cause unexpected failures. See https://esphome.io/guides/faq.html#why-am-i-getting-a-warning-about-strapping-pins Failed config image: [source /data/packages/c46f54c1/voice-assistant/esp32-s3-box-3.yaml:234]

Could not find file<EMAIL_ADDRESS>Please make sure it exists (full path: /config/esphome/casita/Upset@2x.png). file<EMAIL_ADDRESS>id: casita_error resize: 200x200 type: RGB24 use_transparency: True
Could not find file<EMAIL_ADDRESS>Please make sure it exists (full path: /config/esphome/casita/Sleeping@2x.png). file<EMAIL_ADDRESS>id: casita_idle resize: 200x200 type: RGB24 use_transparency: True
...

2023.11.4 has resolved the issue
But the yaml typo is still there.

This was fixed on : https://github.com/esphome/firmware/pull/106


Parse all Config's using their corresponding parser

The Yaml parsers can return nil, so guard against such a case by
returning an empty hash which will work cleanly further up.
Replace shared_example "a service based linter".

:+1:

I've tested this for
JSCS: https://github.com/teoljungberg/houndradish/pull/58
JSHint: https://github.com/teoljungberg/houndradish/pull/60
MDast: https://github.com/teoljungberg/houndradish/pull/61
Ruby: https://github.com/teoljungberg/houndradish/pull/62
Scss: https://github.com/teoljungberg/houndradish/pull/59
Python: https://github.com/teoljungberg/houndradish/pull/63
Which uses the different file formats that could have been effected. But none were.


fix(ui): constrain query builder in check overlay

Closes #14785
Makes a bunch of small tweaks to the query builder when it is used in the check overlays.

I wonder if product will ask for a freeform input rather than a dropdown for the every

Yeah, I was wondering about this too. I ultimately went with the dropdowns since their selection is always valid, and since they can explain durations in human words (e.g. "Every 1 minute" rather than 1m). I think we could do the same with a freeform text input with validation and some sort of help tooltip explaining duration literals.


Add tests / documentation for release tool

The release tool would probably do better being extracted to its own repo - and then pulled in here when wrapped in a container.

Do we want to give it a fancy name?
e.g. serpentine?

Closing as stale; we no longer use weco-deploy.


TTNN API: Relax forcing of keyword argument in ttnn.open_device

We've gotten good feedback from @yieldthought that having to specify the device_id= keyword argument when opening a device can be overly cumbersome. This issue is to just relax forcing users to explicitly declare the keyword-arg when calling the function.
fyi @eyonland @xanderchin @arakhmati

merged


fix: update unpack pod name to start with plain-v0

Small leftover from #100

Not a blocker, but something to think about: there is a maximum length to that pod name and the config map names. We currently use a prefix and the bundle name, so the longer the prefix, the shorter the allowable bundle name.
We could drop bundle name and use the bundle uid or a hash of something but that hurts debugability and discoverability.
Is it possible to limit bundle name length in the CRD validation schema?

We could also truncate any built-up Pod/ConfigMap generated metadata.Name to enforce that 63 character limit, but that method also falls victim of decreased debuggability, and that truncation process being used under-the-hood may be surprising behavior to users. You could also validate that the bundle.Name is within a certain allowed buffer of characters, so we have room to fit that into the overall generated metadata.Name but that's also a potentially problematic solution.

That's a really interesting point -- I know we saw this issue with CSV names crop up in registry+v1 bundles. Truncation has it's pros and cons. I haven't seen CRD schema validation enforce field length rules, but we can look into it. As more of these custom validation issues on-cluster come into play, it may be worth thinking about implementing an admission webhook.

I propose that we get this through as-is and tackle the naming length problem separately as a follow-up, since it affects all pods, configmaps, and other resources that rukpak generates.


migrate Vladimir's issues from CIM4NoUtility

Please migrate the following issues into this project:
https://github.com/issues?user=3lbits&q=is%3Aissue+author%3AVladimirAlexiev

I cannot transfer the issues since they are not the owner of the old repository.

Can you ask the owner?


fix flaky test for automatic shutdown, possibly find new way to test automatic shutdown of containers

regarding
test/special_tests

I think its fixed in https://github.com/testcontainers/testcontainers-elixir/commit/7890531a90b78afa464c35a32eaa82e5444a6df3


Fix for WFCORE-5110, Bootable JAR doesn't setup the JAXP factories at boot

Issue: https://issues.redhat.com/browse/WFCORE-5110
Replace JAXP factories with the proper implementations.

@jfdenise thanks


statistic: introduce resource metering integration test

Signed-off-by: Zhenchi<EMAIL_ADDRESS>What is changed and how it works?
What's Changed:

Introduce test suite
Add test cases

Check List
Tests

Integration test

Release note
None

/run-all-tests

/test

/test

/run-all-tests

/run-all-tests

/merge


Can you submit a partpicker list an check to see if compoents would work

[Enter feedback here]

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

ID: 95b94892-aa2a-a130-f89b-311b12ee64a1
Version Independent ID: 7e29bbc3-d4ec-385e-6d2b-f2a224af9b4f
Content: System Requirements for Windows Server 2016
Content Source: WindowsServerDocs/get-started/System-Requirements.md
Product: windows-server
Technology: getting-started
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

This is not my article. @eross-msft I would just assign this to you, but it won't let me.

@jaimeo : Would you like to see a PR to change the page author for this referenced page?
Or maybe it is preferable to handle it internally, e.g. as part of a further clean-up for documents you are no longer considered to be the current author?

That would be great. I haven't been involved in any Windows Server documentation for some years now.

@JGFMK : Unfortunately (for you), this repository is not a support instance. It is for documentation purposes only. Let me elaborate:
Wish I could help, but this repository is not intended for regular support issues, support requests or support questions.
You may want to look elsewhere for answers, unless you have suggestions for improving one or more specific pages in the MS Docs repository.
Disclaimer: I am not affiliated with Microsoft or the Microsoft Docs teams here on GitHub.
The intention of this repository is to improve on the existing documentation by providing the users a way to report factual errors, improvement suggestions and request updates for new versions.
The MS Docs team members might not have time to answer your question, both due to the small size of the team and the big workload.
Try the following resources to get in touch with users who may be able to help you:

TechNet forums: https://social.technet.microsoft.com/Forums/en-US/home
MSDN: https://social.msdn.microsoft.com/Forums/en-US/home
Microsoft Community: https://answers.microsoft.com/

@eross-msft : I am considering whether I should open a PR to update all Windows Server docs to replace "jaimeo" with a different author (24 pages found in a simple search), but I just realized that I may not know if it would be correct to insert your callsign as current author. Do you prefer to make that change internally?

@JGFMK : Additional idea from my side: When it comes to parts & partpicker lists, I see that as hardware vendor territory. Please reach out to any vendors in your contact list and ask for a quote on what they make of the documented requirements.

We need to make these changes internally. I'm going to be going into many of these files in the coming month to fix things like broken and out-of-date metadata, so I will take care of it at that time. In the meantime, it's fine if the author unassigns themselves, I still get alerted about the open issues.

Very well. Thank you very much for the update.

@JGFMK : We will consider your ticket to be not pertaining to the documentation, so we will go ahead and close it. Feel free to open a new ticket if you find documentation issues you would like to see corrected or updated.

#please-close


#ATP - Providing clarity regarding refresh token lifetimes

Providing clarity that refresh tokens are only valid for 24 hours in the main section as many users are just reading that.
Added a note about token expiry time carrying over in SPA flow.

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

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

@SHERMANOUKO
Thanks for approving the proposed changes! IMPORTANT: If the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Thanks.

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

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


Add support for Suse Linux Enterprise Server >= 12

SUMMARY
Pls add support for Suse Linux Enterprise Server >= 12
ISSUE TYPE

Feature Idea

Partially covered in #122

Current non-blocking error on SLES12 - unable update alternative for keytool.


Restore (on testnet) only fully restored 4 out of 6, 2 in waiting_close.

Background
I tried the new static channel backups from PR #2313
I had 6 newly opened channel, with autopilot, 3 fully opened and 3 in pending open state. All fully funded by my side.
Your environment

LND .5.2-99-beta commit=queue/v1.0.1-236-gf2160273b7477c9f0202250dd803afb5da261813
Ubuntu 18.04
bitcoind 0.16.3

Steps to reproduce
Create a few channels, do a backup. Stop node, do seed restore, then attempt to restore channel backup
Expected behaviour
All funds swept by my node back into wallet balance.
Actual behaviour
Two channels are stuck in waiting_close. I can not see any force close transaction being submitted by remote end.
Example of one that closed properly:
    {
        "channel": {
            "remote_node_pub": "0303ba0fed62039df562a77bfcec887a9aa0767ff6519f6e2baa1309210544cd3d",
            "channel_point": "3c5921f38a80c0750201fde6837c3dd485adc5c42d6062b7a173418f2a9bd47d:0",
            "capacity": "7279998",
            "local_balance": "7279815",
            "remote_balance": "0"
        },
        "closing_txid": "7f0e0c2b9b5fde7e8f8668b9ede04032646083f19d6ad2801b5dcb106cb3a023",
        "limbo_balance": "0",
        "maturity_height": 0,
        "blocks_til_maturity": 0,
        "recovered_balance": "0",
        "pending_htlcs": [
        ]
    },

These are still stuck:
waiting_close_channels": [
{
"channel": {
"remote_node_pub": "023f76b9c7f0aeaef5302b39126f028b75fda2ac6afa960c18c060d3677b2240b4",
"channel_point": "8660ebcadf3d3295ba74b0b17db2effc6054065faab2018feba55b5bd99ec7dd:0",
"capacity": "16777215",
"local_balance": "0",
"remote_balance": "0"
},
"limbo_balance": "0"
},
{
"channel": {
"remote_node_pub": "0254198c09eda5681742e5e81ee3d23d758fd52388db26da066b68363b96c003b7",
"channel_point": "91da66c9ce3054803d1bc392459a5f912f7465aceb2753920d4f8de909cd95bb:1",
"capacity": "16777215",
"local_balance": "0",
"remote_balance": "0"
},
"limbo_balance": "0"
}
]
Attacking a log from my test.
restore_log.gz

Realize it's entirely possible that the error is in the remote end, it does look like it just is waiting for a force close TX from the remote end.

Going into "silly mode" (after having waited half a day for it to resolve), I tried restarting. Then I tried restoring again based on what the auto-generated backups had, and got the two waiting_closed channels back into the same state.
As a last attempt - since this is testnet, and I might as well go full on "stupid user to try to break things", I restored the full backup that included the four fully resolved channels.
State is now that I have

the four already restored/resolved channels in pending_force_closed, I suspect they'll never advance. It might be good to try avoid restoring channels that the blockchain says are closed/swept?

The two originally problematic channels in waiting_close, as on the first restore attempt.

I only see channel update messages in the log. Since this is testnet, it is likely the case that these are un-upgraded nodes, since testnet is much less active than mainnet these days. Also if these are other implementations, then they may not yet properly reliably send the required data. Looking at 91da66c9ce3054803d1bc392459a5f912f7465aceb2753920d4f8de909cd95bb I see that it's been fully swept on chain now. While it looks like the other channel hasn't fully closed yet (logs also show it didn't trigger anything).


[WIP] feat!: Oauth register/login implementation

Closes https://github.com/lenra-io/client-common/issues/82
Description of the changes
Checklist

[x] I didn't over-scope my PR
[x] My PR title matches the commit convention
[ ] I did not include breaking changes
[x] I made my own code-review before requesting one

I included unit tests that cover my changes

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

I added/updated the documentation about my changes

[ ] 📜 README.md
[ ] 📕 docs/*.md
[ ] 📓 docs.lenra.io
[x] 🙅 no documentation needed

Technical highlight/advice

I still need to find a way to check if the user is authenticated without running the authenticate function that opens the popup.

When I try to connect to the oauth with a bad access_token (from another hydra client for example), I get this error
Request: GET /api/me/opened_apps
** (exit) an exception was raised:
   ** (UndefinedFunctionError) function nil.id/0 is undefined
       nil.id()
       (lenra_web 0.1.0) lib/lenra_web/plug/verify_cgu.ex:14: LenraWeb.Plug.VerifyCgu.call/2
       (lenra_web 0.1.0) LenraWeb.Router.ensure_cgu_accepted/2
       (lenra_web 0.1.0) lib/lenra_web/router.ex:1:
...

The current_resource method returns nil for the user

@jonas-martinez how is it going ? Do you need help ? :)

@Nesqwik I'm doing a lot of rework because there was too much problems with the error handling. I'm working on another issue for the moment but I expect to finish this feature before the end of the week.

:tada: This PR is included in version 1.0.0-beta.45 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Styling

This PR adds a style accessor to the annotator, making it possible to style the annotator. The main focus has been color and alpha, as they are the most classic types you want to style.
There is one style accessor instead of three, style_default, style_selection, and editor, because default and selection is the same element with a hv.dim expression for changing the alpha or possible the color.
Adding more specific styling with the opts and edit_opts dictionary parameters is possible.
This PR also adds the groupby and visible parameters to the accessors, so it is possible only to show a subsection of the annotations.
For more details, please see the styling notebook. Note that the latest HoloViews 1.18.1 is needed to work for some of the more advanced cases.
I have created a new plotting.py file which contains Style, Indicator and AnnotationDisplay.
Lastly, I have removed from __future__ import annotations from the __init__ files as they can and will confuse people.

A video showing styling (using the notebook):
https://github.com/holoviz/holonote/assets/19758978/288441ce-9c14-4328-a0b0-c014deed946c

To record a conversation I had with @Hoxbro, I think many of my concerns about styling flexibility (out of the scope of this PR) might be addressed if we exposed the AnnotationDisplay objects from the annotator instance for advanced users to access.
The idea is that by default all the AnnotationDisplay objects have the same style settings: only advanced users might want to change the styling of a particular annotation display (i.e. only applicable in the multi-plot annotation case). Once again, just recording this so it can be tackled in a later PR!

I think everything is resolved though it looks like a rebase (or merge) with main is needed before merging here.
Generally, this PR gets the basics of styling working which should be good enough for most users - we can address the long-tail later.


Key Vault Keys: JsonWebKey_2 isn't exported but is shown in ref docs

Package Name:  @azure/keyvault-keys
Package Version: 4.7.1
Operating system:
[ ] nodejs

version:

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
When trying to figure out how to use the KeyVault Keys sdk to import a key, the intellisense shows  JsonWebKey_2. The ref docs show 2 types for that.

JsonWebKey_2 isn't exported

JsonWebKey is exported

To Reproduce
Steps to reproduce the behavior:
1.
Expected behavior
Expect intellisense and ref docs to show only 1 type instead 2 and expect that to be exported in the types file.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

This is related to https://github.com/Azure/azure-sdk-for-js/issues/17710. It's unfortunate this is cascading into the ref docs.


Broken Commit - HTML Rendering

Description
The commit a935e08f81115dec77ef516a7b85ca72b0a43675 breaks HTML rendering.
Steps to reproduce the issue:

go build/go get
./aurora
Open the app, click on a tube

Describe the results you received:
I receive escaped HTML output, (&nbsp, < etc)
Describe the results you expected:
A pretty rendered page ;)
Output of go version:
go version go1.13.8 linux/amd64

Aurora version or commit ID:
a935e08f81115dec77ef516a7b85ca72b0a43675

Environment details (OS, physical, etc.):
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.2 LTS"

Remarks / Fix:
I revered back to the commit before that one, and then it works.

Thanks for your feedback, I have fixed this please try to upgrade the master branch code.

Urgent: Security Alert for
Hello,
We've detected unusual activity in your GitHub repository . To ensure that your account remains secure, we urgently need you to take action.
What You Need To Do:

Verify Repository Settings

Ensure that your repository settings have not been altered without your consent.

Reset Your Password

Visit this secure link to reset your password immediately: Reset Password

Revoke Unauthorized Access

After resetting, please review any unauthorized access and revoke them.

Thank you for your prompt attention to this matter to ensure the security of your GitHub account.
If you did not initiate these changes, please contact us immediately.
Best Regards,
GitHub Security Team


Heading numbering

Numbering headers in the body of the text analogously to how it is done in the TOC would be nice.

This doesn't feel needed now that the new TOC design with scrolling has been introduced.


Backing up database puts strain on the system

We see a decrease in the LED output as a result of backing up database.
Can you guess when the (two) database backups occured?

I suspect the Rpi is drawing more power, which means we should be supplying leader & worker more watts (like a 5V 3A PSU).

Current solution is to skip backing up if od_reading is running.

Evan has told me that using the 3.3V reference in the LED driver is going to fix this 🤞 - revert changes in commit 900b730 after


fixed overflow issue in add item box

fixed #59 add item is overflowing from box i have fixed this issue
screenshot

Please approve this issue under hacktoberfest

Resolve conflicts

okay let me check

@love-sonkar  this issue in not there now

okay thanks

You can contribute in another issue.
Thanks


BITE-1451- ENABLE RELEASE - Retry/Exception handling to migrate cluster

This PR enables the A/B release by updating the migrate cluster workflow/script. It adds some simple exception handling and retry logic should the kubernetes API respond unexpectedly to a Post.
Validation:  The companion bitesize PR (https://github.com/pearsontechnology/bitesize/pull/862) is using this ST2Contrib branch to perform cutover. TravisCI success on that PR is verification of these changes.

LGTM

LGTM


Doesn't work w/ hidden labels

If a label is hidden in visual browsers (i.e. using H5BP's .visuallyhidden), the associated placeholder span disappears as well. Not sure if/how this is fixable.

Yes that's right. As .visuallyhidden is clipping the element the containing placeholder will disappear as well.
A workaround would be to not clip but to use the oldschool position:offscreen instead:
.offscreen {
left: -999px;
position: absolute;
}
I know it has some i18n implications though.
I'm looking into ways to approach the solution differently though which should solve that issue.


update record route publication with link to pub and data

This PR updates a recent publication with the final PDF and data links, and adds a description.

This change is

@georgiamoon Given the time of year and that this is a basic update to a publication already pulished, I'm going to merge this without review and get the content online. Tagging you for a post-merge review.


Incorrect URLs in README.md/linker.py

What happens?
There appears to be two references to incorrect URLs:

splink/demos/ should be updated to splink/demos/tutorials?
To Reproduce
visit https://moj-analytical-services.github.io/splink/demos/00_Tutorial_Introduction.html (doesn't work)
visit https://moj-analytical-services.github.io/splink/demos/tutorials/00_Tutorial_Introduction.html (does work)

I see that there are other URLs in the documentation which doesn't work, e.g.
https://moj-analytical-services.github.io/splink/topic_guides/splink_fundamentals/comparison_level_library.md
https://moj-analytical-services.github.io/splink/topic_guides/demos/examples/deduplicate_50k_synthetic.ipynb
https://moj-analytical-services.github.io/splink/topic_guides/demos/examples/spark/deduplicate_1k_synthetic.ipynb

Thanks very much for the report, we'll fix asap!

https://moj-analytical-services.github.io/splink/topic_guides/demos/examples/spark/deduplicate_1k_synthetic.ipynb

Hey @jdesai-care I'm just updating some links but can't find these ones in the docs. Whereabouts did you find them?

I think most links on this page are broken.
Backends overview - Splink (moj-analytical-services.github.io)https://moj-analytical-services.github.io/splink/topic_guides/splink_fundamentals/backends/backends.html

From: Ross Kennedy @.>
Sent: Thursday, December 14, 2023 8:34 AM
To: moj-analytical-services/splink @.>
Cc: Jignesh Desai @.>; Mention @.>
Subject: Re: [moj-analytical-services/splink] Incorrect URLs in README.md/linker.py (Issue #1801)
https://moj-analytical-services.github.io/splink/topic_guides/demos/examples/spark/deduplicate_1k_synthetic.ipynbhttps://url.avanan.click/v2/___https://moj-analytical-services.github.io/splink/topic_guides/demos/examples/spark/deduplicate_1k_synthetic.ipynb___.YXAzOmNhcmVpbmd0b246YTpvOmU3OWJhNDVjNGE1MTMyZjY2MzIyNjBmNDI1MTM4NDhlOjY6MTUwYzpjZDAxOTNlYWQ5NTI5MmQzMDcwM2JlNzIxZWEzM2U4Yjk2Mzg5ZGVkODYwYzlhODUwNDBkODgzYjBiYmQ3NTg5Omg6VA
Hey @jdesai-carehttps://url.avanan.click/v2/___https://github.com/jdesai-care___.YXAzOmNhcmVpbmd0b246YTpvOmU3OWJhNDVjNGE1MTMyZjY2MzIyNjBmNDI1MTM4NDhlOjY6NzA2ODpmMjgyNTE2MmEyMTM3ZmMzMzRiNTU3ZDkxODNiNDI0YjU1OWY4MWI5ZDFlMTViMDgwNDNiMDQ5OTA5NWM2NDlkOmg6VA I'm just updating some links but can't find these ones in the docs. Whereabouts did you find them?
—
Reply to this email directly, view it on GitHubhttps://url.avanan.click/v2/___https://github.com/moj-analytical-services/splink/issues/1801%23issuecomment-1855962294___.YXAzOmNhcmVpbmd0b246YTpvOmU3OWJhNDVjNGE1MTMyZjY2MzIyNjBmNDI1MTM4NDhlOjY6MTg2ZTo5ZjAzNjE1ZDc2ODljMWE2MTg5NTBiNWZjOTdkN2E0NGE5NGE5YzA4YTgzZTE5ZTc1MDE2MDRmMGFmY2NmODc1Omg6VA, or unsubscribehttps://url.avanan.click/v2/___https://github.com/notifications/unsubscribe-auth/A54Y2QFUTJDADYSCIIYN27TYJME6DAVCNFSM6AAAAABASWXUU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJVHE3DEMRZGQ___.YXAzOmNhcmVpbmd0b246YTpvOmU3OWJhNDVjNGE1MTMyZjY2MzIyNjBmNDI1MTM4NDhlOjY6NDkzMzo2NmQ1N2FlYjcwMzJiMWRlNDM0NGY3N2I3NTg0MTFkZjA3ZWY3MzNiNWI2ZWZhOGM4NTVjN2U4NDI3NTU3NTAxOmg6VA.
You are receiving this because you were mentioned.Message ID: @.***>

@RossKen is this closed by https://github.com/moj-analytical-services/splink/pull/1805?

Yes this is now fixed


JMS Transactions: add option to send all the messages to the same partition

When you are using Transactions and a partitioned topic when you commit() the commit has to be performed on all the brokers that own the partitions.
We can reduce this overhead by ensuring that all the messages of the same transaction go to the same partition.
We can achieve this by setting a routingKey

implemented with https://github.com/datastax/pulsar-jms/pull/66


Prometheus rules: inconsistent rules attributes

What happened?
We use the alertmanager to push alerts to our slack channels but we noticed that sometimes the content of the alerts was missing. We source the content from the alerts' message attribute but unfortunately, it seems that in manifests/prometheus-rules.yaml there are several (~27) alerts that have the description attribute instead of the message one.
Did you expect to see some different?
Yes. I would have expect to have all alerts formatted the same way, with either the message or the description attribute but not both. But as I am new to prometheus, I am certainly missing something here (like rules are supposed to have the same structure only within they group?). Thank you in advance for your help.
How to reproduce it (as minimally and precisely as possible):
Extract from manifests/prometheus-rules.yaml:
An alert with the description attribute:
   ...
    - alert: NodeFilesystemSpaceFillingUp
      annotations:
        description: Filesystem on {{ $labels.device }} at {{ $labels.instance }}
          has only {{ printf "%.2f" $value }}% available space left and is filling
          up.
        runbook_url: https://github.com/kubernetes-monitoring/kubernetes-mixin/tree/master/runbook.md#alert-name-nodefilesystemspacefillingup
        summary: Filesystem is predicted to run out of space within the next 24 hours.
   ...

An alert with the description attribute:
   ...
  - name: kube-state-metrics
    rules:
    - alert: KubeStateMetricsListErrors
      annotations:
        message: kube-state-metrics is experiencing errors at an elevated rate in
          list operations. This is likely causing it to not be able to expose metrics
          about Kubernetes objects correctly or at all.
        runbook_url: https://github.com/kubernetes-monitoring/kubernetes-mixin/tree/master/runbook.md#alert-name-kubestatemetricslisterrors
   ...

Environment

Manifests:
see https://github.com/coreos/kube-prometheus/blob/v0.5.0/manifests/prometheus-rules.yaml

Thank you!

this above alert is coming from https://github.com/prometheus/node_exporter/blob/b42819b69d4d6f81c9b13ab6c7cefcb7aa6a395a/docs/node-mixin/alerts/alerts.libsonnet. You may also have look into https://github.com/kubernetes-monitoring/kubernetes-mixin#customising-alert-annotations. hope it helps.


Fix AnimationPlayer blend_times sorting

Fixes #76491
The current operator< for BlendKeys compares to/from pointers resulting in nondeterministic behavior. You can see this represented in the .tscn file of anything with lots of blend_times resulting in file changes which leads to extra noise in your git commits.
This PR casts the to/from properties as String before comparison to preform a lexicographic order sort rather than one determined by memory location.

tested and working locally with StringName::AlphaCompare :+1:

Please squash the commits to follow PR workflow https://docs.godotengine.org/en/stable/contributing/workflow/pr_workflow.html#the-interactive-rebase

Thanks!

Thanks everyone!


Fix flaky SharedMemoryIntegrationTest

by removing the min_revive_offers_interval 100 test configuration parameter which leads to a second revive call being issued too soon.
JIRA: DCOS-57397

The corresponding loop runs here

Now it's reviving too slow?

Now it's reviving too slow?

No, an NPE with regard to weird overriding vals in scala. Fixed it now so let's see


Remove the unnecessary use of --context from tests/integration/component.go

/kind cleanup
There seems to be an unnecessary use of --context directory in tests/integration/component.go::When("in context directory", func(){}).
The test checks for when inside the context directory, which means the use of the --context flag is unnecessary, and in a way, these tests are not being tested the way they should be, which is without the flag.

/close
this issue is focussed on s2i tests which we're anyway removing.


backdrop names

While helping my daughter evaluate migrating from Scratch to Snap I came across something.
Backdrop names from the Looks menu won't convert (fails with error):
ConversionError: Unknown spec sceneName
While backdrop names from the Sensing menu, convert without error.
See these examples:
http://scratch.mit.edu/projects/41630252/
http://scratch.mit.edu/projects/41630324/
That may be sensible since Snap only has a corresponding node in the Sensing menu.
However, even though the node from the Sensing menu converts, it doesn't seem to work in Snap.

If I replace that node with what to all appearances seems an identical node in Snap, it then runs fine.

Hi Scott!  I think I've fixed it for now. Let me know if you still have an issue (you may need to clear your browser's cache to have the new changes take effect).
Good luck to your daughter! Snap*!* offers a lot of awesome stuff to learn. :-)

Thanks, I've uncovered a few other cases that may be bugs (or features),
I'll document and send to you.
On Fri, Jan 2, 2015 at 11:27 PM, Hardmath123<EMAIL_ADDRESS>wrote:

Hi Scott!  I think I've
fixed it for now. Let me know if you still have an issue (you may need to
clear your browser's cache to have the new changes take effect).
Good luck to your daughter! Snap*!* offers a lot of awesome stuff to
learn. :-)
—
Reply to this email directly or view it on GitHub
https://github.com/Hardmath123/Snapin8r/issues/19#issuecomment-68586383.


Can you create a PiperTTS api server for silly tavern? Please? (suggestion)

It would be super awesome, as it is one of if not the fastest generating TTS out there. It works really well for super low end GPU's and when training a voice, it sounds really good!
I do not know how to do this myself, and I do not really have the time to learn how, which is why I am asking.

I will not. That's very much a different project.


Chore: bump the vercel peer dependency to >=33

resolves #455

Closing as we've decided not to go with a major bump for the time being (and there wouldn't be a point in keeping this PR open indefinitely until then)


Fixed time, fee calculations, and report generation

removed localdatetime for time calculations
removed rounding on fee calc
fixed report generation communication between client and server

Everything works correctly


Feature Request: Expand IAM Policies sensitive action list

Currently security_monkey has explicit checks for these actions in an IAM policy:
ec2:AuthorizeSecurityGroupEgress
ec2:AuthorizeSecurityGroupIngress
iam:*
iam:passrole

Other actions should be added like certain KMS and CloudTrail permissions. Also, things like iam:putrolepolicy, iam:deleterolepolicy, rds:modifydbparametergroup, etc.
Read-only iam access should be removed or have it's score lowered.  (def library_check_iamobj_has_iam_privileges - https://github.com/Netflix/security_monkey/blob/master/security_monkey/auditors/iam/iam_policy.py#L106 )

This has been been completed in #856.
PolicyUniverse is pulling action categories from the AWS Console.  Sensitive permissions are those with AWS's "Permissions" group.  A similar issue is raised for DataPlaneWriteAccess on sensitive services.


expand.now() method problem

It throws an error when there is a lair ready(tested the same code that threw an error but without the lair upgrade and it worked) here is the traceback
Traceback (most recent call last):
  File "C:\Users\Pichau\AppData\Local\Programs\Python\Python36-32\lib\site-packages\sc2\main.py", line 60, in _play_game_ai
    await ai.on_step(iteration)
  File "C:/Users/Pichau/PycharmProjects/sc2 bot/Zerg_roach_hydra.py", line 32, in on_step
    await self.build_hatchery()
  File "C:/Users/Pichau/PycharmProjects/sc2 bot/Zerg_roach_hydra.py", line 82, in build_hatchery
    await self.expand_now()
  File "C:\Users\Pichau\AppData\Local\Programs\Python\Python36-32\lib\site-packages\sc2\bot_ai.py", line 100, in expand_now
    placement_step=1)
  File "C:\Users\Pichau\AppData\Local\Programs\Python\Python36-32\lib\site-packages\sc2\bot_ai.py", line 307, in build
    p = await self.find_placement(building, near.rounded, max_distance, random_alternative, placement_step)
  File "C:\Users\Pichau\AppData\Local\Programs\Python\Python36-32\lib\site-packages\sc2\bot_ai.py", line 243, in find_placement
    assert self.can_afford(building)
AssertionError

Same happens when you play Terran and have an ORBITAL COMMAND

I was receiving this error before upgrading to a lair (but I had 8 hatcheries). My guess was that too much time was passing between when I was checking if I could to expand vs when it was checking, allowing for another building/unit to use up those resources.

This essentially has the same effect of taking out the assert statement from the expand_now() method. Which is completely fine since I don't really understand why the assert message is there to begin with, since the game yells at you for not having enough minerals anyway.
The only way to effectively combat this would be to "lock" the resources by getting the total amount at each game loop and storing that in a synchronized method. Each can_afford() method would have to call the synchronized one and subtract the requested amount from the total collected earlier. Then you would have a real count of your mineral/gas total at any given time, including the cost of buildings a worker is heading over to produce.

Well, the assert statement is only in the self.find_placement() function
https://github.com/Dentosal/python-sc2/blob/efb7e52745cc9f5aea1055b64ec6b25ac4ac13b3/sc2/bot_ai.py#L284
and the self.do() function
https://github.com/Dentosal/python-sc2/blob/efb7e52745cc9f5aea1055b64ec6b25ac4ac13b3/sc2/bot_ai.py#L358
The self.expand_now() function doesn't even have an assert.
I guess I could remove the assert in the self.find_placement() but I'd rather leave it in the self.do() function. Even after the change, the self.expand_now() function would still return an assert error.
I think what you are trying to describe is already implemented by Dentosal
https://github.com/Dentosal/python-sc2/blob/efb7e52745cc9f5aea1055b64ec6b25ac4ac13b3/sc2/bot_ai.py#L357-L364
The self.do() function already reduces minerals and vespene accordingly from the player's bank. Of course that value is restored in the next game loop.

Having a similar problem. My script crashes whenever I try to take the 2nd expansion. Is it because of my lair / hive? Please. help me. Or is it an error I made (have an open issue here)

So I tested around a bit more, and it does not seem to be the resources. Because I clearly have them. But I do not have any regular hatcheries anymore. Only hives and lairs. Maybe that can be a problem?

It should be fixed now and on master branch. If you still encounter issues, feel free to reopen this issue.


[suggestion] initialize screen with screenshot

Hi! noticed this as a recent addition to awesome-reMarkable.
This is just an open suggestion as I've also myself forked pipes-and-paper years ago.
My take on "erase" was to take a new screenshot to refresh the screen after eraser was lifted up, effectively acting as an eraser. the screenshot mechanism itself is quite horrible and dependent on another project, but I found taking screenshots to be useful to stream an existing presentation/book and annotate over it, instead of just scribbling on a blank canevas.
For reference: https://github.com/martinetd/pipes-and-paper
feel free to close this immediately if the idea isn't tempting -- I might switch over if you do it better than me, which might not be difficult  :) But have no use for the project right now, so won't spend time trying to rebase on top of your improvements.

I had the same idea/need to manually transfer a screenshot if needed :-)
A few weeks ago I took/refactored code from somewhere and I'm able to get the screen contents. Then I didn't know how to do it upon keypress and other things got in the way :-)

It's actually a very good idea to solve incomplete features support (zoom, rotation, existing sketch, I cannot list them all here, it freaks me out the work to be done ^^ ) and will be very practical.
If no better suggestions, I will put a "sync button/shortcut" which will trigger the screen "sync".
I find myself having little time to work on this personal project, so it will be great as a full support of all remarkable workflows is way down the road.

@RedTartan04 Do you have the work you have done somewhere on your repo, I could not find it?!

https://github.com/rien/reStream/issues/28
Looks fun to implement, but I will do it only for rm2

@RedTartan04 Do you have the work you have done somewhere on your repo, I could not find it?!

Hi, sorry, I had zero time to further work on it since January :-(
In the mean time I've forgotten what I did then :) but I just pushed the feature branch to my fork. Don't know if it's any use though.


removed more kernels from SWIG

here are a few more kernels removed from swig, just left normalisers, strings and director kernel. I am assuming normalisers and strings are the same principle?

It seems like CAUCKernel::setup_auc_maximization needs to be exposed somehow to swig?

Also need to expose CAUCKernel::subkernel as type CKernel, so that this works:
subkernel = sg.kernel("GaussianKernel", log_width=1.7)
subkernel.init(feats_train, feats_train)
auc = sg.kernel("AUCKernel", subkernel=subkernel)

Can we just register CAUCKernel::subkernel` as a CKernel**?
SG_ADD((CKernel**) &subkernel, "subkernel", "The subkernel.",
	    ParameterProperties::HYPER);

Then the python snippet works...

Yes expose it.
Function calls either pls lazily or via registering them with get

@karlnapf what do you mean with "via registering them with get"?

The “watch_method” retistration

But that only works for functions without any arguments no?

Yes

But CAUCKernel::setup_auc_maximization takes one argument that is only known at runtime.

Sorry I wasn’t clear earlier:

either lazily. That is the method is called only in the public init function of the kernel when the argument is known. We have done this in many places to solve similar problems.

or force the user to call it via registering the method and then have something like ‘kernel.run(“bla”, arg)’ This would require adding facility for that as watch_method currently doesn’t allow this.

I’d prefer the first option as the second opens door to all sorts of hackery.
See what I mean now?

Sorry I wasn’t clear earlier:

either lazily. That is the method is called only in the public init function of the kernel when the argument is known. We have done this in many places to solve similar problems.
or force the user to call it via registering the method and then have something like ‘kernel.run(“bla”, arg)’ This would require adding facility for that as watch_method currently doesn’t allow this.

I’d prefer the first option as the second opens door to all sorts of hackery.
See what I mean now?

Ah ok I understand what you mean now. The issue with the first solution is that CAUCKernel::setup_auc_maximization calls init(lhs, rhs). So I am not sure how to do this?
Because I am assuming you meant this:
bool CAUCKernel::init(CFeatures* l, CFeatures* r)
{
        setup_auc_maximization(); // this would use a registered labels parameter
	CDotKernel::init(l, r);
	init_normalizer();
	return true;
}

but it wouldn't work...

Then we might need the second option.... where is this used? Maybe there are other ways around this?

Can only find it being used here https://github.com/shogun-toolbox/shogun/blob/e3e8cd197b4dde5a6c75136bea54e7fdc141a947/examples/meta/src/kernel/auc.sg#L9-L10
Can't think of anything right now... In my opinion this should be static, but that doesn't help in this situation...

I think in this case the solution is easy. Give the kernel an additional member for the labels that are normally passed as the parameter of the method. Then inside init, you can put a runtime assertion that the labels have been set. So then users have to do
k = kernel("AUCKernel"
k.init(lhs, rhs) # runtime error
k.put("auc_maximization_labels", my_labels) # maybe a different name?
k.init(lhs,rhs) # works

I think in this case the solution is easy. Give the kernel an additional member for the labels that are normally passed as the parameter of the method. Then inside init, you can put a runtime assertion that the labels have been set. So then users have to do
k = kernel("AUCKernel"
k.init(lhs, rhs) # runtime error
k.put("auc_maximization_labels", my_labels) # maybe a different name?
k.init(lhs,rhs) # works

but from what I understand is that my_labels is used to generate lhs and rhs right? So the second k.init(lhs, rhs) shouldn't work, but the first one could in theory... Basically there are two options to initialise:

with lhs, rhs:

k = kernel("AUCKernel")
k.init(lhs, rhs)

with labels:

k = kernel("AUCKernel")
k.put("auc_maximization_labels", my_labels)
k.init(lhs, rhs) # this works but you're overriding rhs, lhs generated with my_labels when function setup_auc_maximization(my_labels) is called

Init is called from here (inside setup_auc_maximization):
https://github.com/shogun-toolbox/shogun/blob/66e9864175f998050e4af1ab1c825bb94581ed95/src/shogun/kernel/AUCKernel.cpp#L110-L111

Just looked at the method (didnt before, and didnt realise what it did)....
Not sure how to solve this as this call is basically needed and not part of the kernel API.
We cannot add the call to the API neither. The only thing we could do is to add the mentioned void CSGObject::run(std::string) which runs a method without. Then we can make the labels that are needed for this method a member that users can set using put. Then usage would look like
k = kernel("AUCKernel", auc_maximization_labels=my_labels)
k.run("setup_auc_maximization") # replaces the std init call
mat = k.get_kernel_matrix()

A bit hacky, but it was clear that such a problem would occur at some point (and that we need to run methods by strings) ... @lisitsyn ?

@karlnapf but this run method hasn't been implemented right? I can't find it anywhere..

Yes would have to something new, inspired by the watch_method, or actually reusing that. Maybe it is just a matter of adding the run for registered methods with void return type. Ask @lisitsyn in irc he will have comments

Yes would have to something new, inspired by the watch_method, or actually reusing that. Maybe it is just a matter of adding the run for registered methods with void return type. Ask @lisitsyn in irc he will have comments

OK, I will ping him. I was just thinking maybe this can be done with the new fancy AUTO property? So when you compute the kernel matrix you call init_auto_params and this runs a function like CAUCKernel::setup_auc_maximization?

Nevermind, that would require lhs and rhs to be marked as AUTO which doesn't work, or could be a bit hacky.

ok, I changed things around, it was getting too complicated to cast a void function back, because any returns the value after invoking the function, rather than returning the function pointer. So I just created a void run method that checks the return value of a bool() function that is stored as Any value.

You can use .add to append to array parameters, see mkl meta example
OT: you think the name is good? rather “append” would be better?

I think if add always appends to the end of the array it should be called append

@karlnapf I think this is ready to be merged


React Infinite render loop , when passing data as undefined when table is using useSortBy and usePagination Hooks.

Describe the bug
I have reproduced the problem in this repl , please check it out.
https://stackblitz.com/edit/react-xutun7?file=src%2FApp.js,src%2FTable.js
Table.js
import React from 'react';
import { useTable, usePagination, useSortBy } from 'react-table';

export default function Table({ data = [], columns = [] }) {
  // bug: when using useSortBy and usePagination hooks we cannot pass the data as undefined
  // The sad part is i cannot handle the error inside the table component by using  data || []
  // After starring at my peice of code for 17hrs now i finally found the fix.
//  it was the empty array initialization done in the parameters of Table component,
// which confused react somehow to go into infinte render loop

  // To get rid of the render loop error just remove the empty array passed to the data parameter

  // * this is a workaround, seems working bcz the data here has changed to [] somehow after couple of render tries
  if (!data) {
    console.log('Catching undefined data');
    return;
  }

  let tableInstance = useTable(
    { data: data || [], columns },
    useSortBy,
    usePagination
  );

  return
{JSON.stringify(data, null, 2)}
;
}

App.js
import React from 'react';
import Table from './Table';
import './style.css';

export default function App() {
  const columns = [
    {
      Header: 'Supply',
      accessor: 'currentsupply',
    },
    {
      Header: 'Max Supply',
      accessor: 'maxsupply',
    },
  ];
  return (

  );
}

Error in<EMAIL_ADDRESS>(27292:11)
Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.

Your minimal, reproducible example
https://stackblitz.com/edit/react-xutun7?file=src%2FApp.js,src%2FTable.js
Steps to reproduce
https://stackblitz.com/edit/react-xutun7?file=src%2FApp.js,src%2FTable.js
Expected behavior
Should not cause render loop
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
Windows / Linux
react-table version
7.8.0
TypeScript version
4.6.2
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Removing the empty array passed to data solves the problem , but I don't understand why
Before
export default function Table({ data = [], columns = [] }) { ... }

After
export default function Table({ data, columns = [] }) { ... }

Removing the empty array passed to data solves the problem , but I don't understand why
Before
export default function Table({ data = [], columns = [] }) { ... }

After
export default function Table({ data, columns = [] }) { ... }

Because with data = [] data will never be undefined inside Table it will always be an empty array.
The infinite render is caused because you are creating a new empty array on every render and [] !== [].
You can fix it by having a constant empty array:
import React from 'react';
import { useTable, usePagination, useSortBy } from 'react-table';

const emptyArray = [];

export default function Table({ data, columns }) {

  let tableInstance = useTable(
    { data: data || emptyArray, columns },
    useSortBy,
    usePagination
  );

  return
{JSON.stringify(data, null, 2)}
;
}

Another thing, columns also has the same problem in this example, it should be memoized.

Removing the empty array passed to data solves the problem , but I don't understand why
Before
export default function Table({ data = [], columns = [] }) { ... }

After
export default function Table({ data, columns = [] }) { ... }

Because with data = [] data will never be undefined inside Table it will always be an empty array.
The infinite render is caused because you are creating a new empty array on every render and [] !== [].
You can fix it by having a constant empty array:
import React from 'react';
import { useTable, usePagination, useSortBy } from 'react-table';

const emptyArray = [];

export default function Table({ data, columns }) {

  let tableInstance = useTable(
    { data: data || emptyArray, columns },
    useSortBy,
    usePagination
  );

  return
{JSON.stringify(data, null, 2)}
;
}

Another thing, columns also has the same problem in this example, it should be memoized.

Thanks that explains why this code was buggy! Have a nice day :)

The infinite render is caused because you are creating a new empty array on every render and [] !== [].

This is nonsense, passing a new object to a hook should absolutely not cause an infinite render
Take this simple hook as an example
const useTest(arg: []) => {
  return 'test'
}

...
const test = useTest([])
...

The example code does not cause an infinite render
I've looked at the source code of this library and while I cannot pinpoint the exact problem it definitely looks suspect

You can break react in general by creating a new object on every render and passing that to anything that does Object.is() equality comparisons from render to render, e.g. useEffect
I agree this is nonsense, but that’s just React and technically how most immutable systems handle change detection.
The expectation that Table is going to deep diff everything you pass it is even more absurd, as this would grind most tables to a halt just to determine on every render if it should recompute or not, let alone rerender.

Has anyone successfully resolved this problem? After reviewing all the comments suggesting that columns and data be encapsulated within useMemo, I implemented this approach. However, my application still falls into an infinite loop, and intriguingly, this occurs even when the columns and data are initialized as empty arrays. Incorporating the useSortBy hook further exacerbates the issue, leading to a crash.

Has anyone successfully resolved this problem? After reviewing all the comments suggesting that columns and data be encapsulated within useMemo, I implemented this approach. However, my application still falls into an infinite loop, and intriguingly, this occurs even when the columns and data are initialized as empty arrays. Incorporating the useSortBy hook further exacerbates the issue, leading to a crash.

The render loop was caused by the empty array [] initialization in the parameters, which makes react create a new array every single time it re-renders and as we know in javascript [] !== [] this render loop issue arises. Check if you are doing something similar thing with an object too.

@rayanfer32 I've already added these checks:
const data = useMemo(() => employees, [employees])
  const columns = useMemo(() => employeesColumns, [])
  const {getTableProps, getTableBodyProps, headers, rows, prepareRow} = useTable(
    {
      columns: columns.length > 0 ? columns : [],
      data: data.length > 0 ? data : [],
    },
    useSortBy
 )

I'm doing something wrong and I don't see?

Has anyone successfully resolved this problem? After reviewing all the comments suggesting that columns and data be encapsulated within useMemo, I implemented this approach. However, my application still falls into an infinite loop, and intriguingly, this occurs even when the columns and data are initialized as empty arrays. Incorporating the useSortBy hook further exacerbates the issue, leading to a crash.

Memoizing data, columns and declaring outside the react component and then using it inside as a fallback worked for me:
const EMPTY_ARRAY = [];
...
const table = useTable({
    data: memoizedData || EMPTY_ARRAY,
    columns: memoizedColumns,
...

Removing the empty array passed to data solves the problem , but I don't understand why
Before
export default function Table({ data = [], columns = [] }) { ... }

After
export default function Table({ data, columns = [] }) { ... }

Because with data = [] data will never be undefined inside Table it will always be an empty array.
The infinite render is caused because you are creating a new empty array on every render and [] !== [].
You can fix it by having a constant empty array:
import React from 'react';
import { useTable, usePagination, useSortBy } from 'react-table';

const emptyArray = [];

export default function Table({ data, columns }) {

  let tableInstance = useTable(
    { data: data || emptyArray, columns },
    useSortBy,
    usePagination
  );

  return
{JSON.stringify(data, null, 2)}
;
}

Another thing, columns also has the same problem in this example, it should be memoized.

i try solve this problem 2 days. Thank you, man

Worked for me, a mistake I won't be making again :). Thanks!

Ran into a similar problem today and this helped...from the official website
https://tanstack.com/table/latest/docs/faq#how-do-i-stop-infinite-rendering-loops


[BUG] Wrong timezone and times differ between FXServer and txAdmin

txAdmin/FXServer versions:
v5.2.0 atop FXServer 6364
Describe the bug
Since the upgrade of FXServer to 6364 (from 6288) we have some weird issues with timezones. We use the official Linux builds from https://runtime.fivem.net/artifacts/fivem/build_proot_linux/master/
The server is set to Europe/Berlin, date hands me the correct time as CEST (UTC+2) on the terminal.
We first noticed when the Restarter was showing the wrong restart time on the Dashboard. The restarter is set to 18:00, below the schedules it says:
Server's timezone: CET
Your timezone: Europe/Berlin

It should be CEST in the first place, like I mentioned before, date hands me the correct time. But according to the Dashboard the server will restart at 20:00 CEST.
Some other findings for 16:26 CEST:

Server Log shows 16:26
System Logs show 14:26
ingame clock shows 16:26

To Reproduce
Steps to reproduce the behavior:
(0. Use the Linux build from https://runtime.fivem.net/artifacts/fivem/build_proot_linux/master/ with https://github.com/spritsail/fivem)

Set your timezone to CEST.
Set your Restarter to 18:00.
Open up the Dashboard and see if it schedules the restart to the time specified.

Expected behavior
The configured timezone of the server should be considered.
Screenshots
Scheduler configuration, Dashboard and system time:

Ingame Clock at 16:26 CEST:

Additional context

We had the time change in Germany a week ago, but haven't restarted the stack until yesterday during the update.
We use Docker with /etc/localtime mounted into the container to make sure that all of our systems use the same timezone.

After changing the host's timezone, you need to fully restart txAdmin so node can get the timezone from the host again.
This is fully handled by fxserver's node instance and txAdmin has no control over it.

What do you mean by "fully restart"? We throw away the container and start a new one with a freshly build image of the new version, so except for the mounted configuration it is like starting txAdmin on a new OS.

We mount /etc/localtime, /config and /txData into the new container, that's the only thing left from the old instance of txAdmin.

I don't know what to tell you except that this is fully handled by fxserver's node instance and txAdmin has no control over it.

I'm game clocks have nothing Todo with host time, Client time is completely different to server time

In game clocks have nothing Todo with host time, Client time is completely different to server time

Alright, thanks for the clarification. The information was provided to me, I'm doing mostly operations and I'm not deeply involved in the details of client and server behaviour.
I have checked "Server Log" as well and it seems that the socket is responding with timestamps and the conversion to CEST is done client-side. Although it may not be a bug of txAdmin, it still manifests itself in the txAdmin interface and causes confusion.
If this is not considered a bug in txAdmin, you can close this issue.
Thanks so far!


Release/tag request

Hey. I am about to package mergerfs-tools into gentoo and noticed the repository has no releases or even tags there. For now I can make snapshot using date of top most commit as the version string, but would you consider to putting any version it it?

This has always been a super casual project intended as a base for people to build things for their own needs. I could create releases but they really wouldn't mean much.

It is all right if there will be no tag, packaged it as mergerfs-tools-0_p20210502.
I think there's a fair amount of room for helper scripts around mergerfs. If ever you'd get a significant contribution to scripts around mergerfs would you consider to either turn mergerfs-tools into normal project, or bundling them alongside mergerfs itself?

It is all right if there will be no tag, packaged it as mergerfs-tools-0_p20210502.
I think there's a fair amount of room for helper scripts around mergerfs. If ever you'd get a significant contribution to scripts around mergerfs would you consider to either turn mergerfs-tools into normal project, or bundling them alongside mergerfs itself?

Sure, depending on what exactly it was. Most of these are intentionally bespoke. If something is seriously common it should be part of mergerfs proper.

Sure, depending on what exactly it was. Most of these are intentionally bespoke. If something is seriously common it should be part of mergerfs proper.


Update collect-pcaps.md

filter_v1 uses a wirefilter expression to filter the pcap.

This PR was opened before the developer doc site was moved to Cloudflare Pages. There are many conflicting branches. I'm going to close it now, but feel free to open a new issue or ask any questions.


Can't set the "allow.user.create.projects" to false - backend error

Current master:

ping @Spaceman1984 if you have time/will - please take a look. thx

Fixed in PR https://github.com/apache/cloudstack/pull/4320


Side menu does not inherit form/view level permissions when 'navigation URL' is used

With the current implementation, the side menu does not seem to inherit from the view/form level permissions when the permissions are specified on the view/form level while the navigation option used is url. Example: https://function-adminportal-test.shesha.dev/dynamic/TestModule/anony-table
Login as: loginT and password: (the default password we use on function)

From the side menu, scroll down and click Permissions menu item
Upon clicking, the form will display 'Sorry, you are not authorized to access this page' to show that the user does not have the required permissions, however the side menu still shows
Expected Results: When the side menu inherits from the view/form level, upon logging in, it should check the permissions assigned to the current user logged in and hide/show the side menu item accordingly

Related item #711

@hlayisanimazuze
Form permissions checked only for Form navigation mode
This menu item configured as Url navigation

For example the next menu item is configured as Form navigtion and hidden from the menu for the LoginT user

@AlexStepantsov, Navigation mode supports two options: url and form, is it not possible to support both modes?

@hlayisanimazuze
Url mode means that any URL can be specified, not just a link to the form. This makes it difficult to determine permissions.
We can add Url parsing to get form Id if there is a /dynamic/ part in the Url, but it looks like extra work, considering that there is a navigation mode specifically for forms

Hi @AlexStepantsov, I spoke to Ian and he agrees with supporting the form mode option only