Typo in 4.1

Hi,
I think there is a typo in chapter 4.1.
Original: Bayesian statistics the probability of data under a specified model (P|D() is a number that expressed what is sometimes referred to as the absolute evidence
Correct: Bayesian statistics the probability of data under a specified model P(D|H) is a number that expressed what is sometimes referred to as the absolute evidence

Hi, I only see it misses a closing parenthesis in the online version. Are you using a pdf? I do not k ow why the math does not render for you.

From: PetrPalisek @.>
Sent: Sunday, October 22, 2023 11:40:54 PM
To: Lakens/statistical_inferences @.>
Cc: Subscribed @.***>
Subject: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Hi,
I think there is a typo in chapter 4.1.
Original: Bayesian statistics the probability of data under a specified model (P|D() is a number that expressed what is sometimes referred to as the absolute evidence
Correct: Bayesian statistics the probability of data under a specified model P(D|H) is a number that expressed what is sometimes referred to as the absolute evidence
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYJCOY3FVHNMBE7HG33YAWHGNAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2TMMBWGYZDGOA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi,
here's a screenshot.
￼
It's the web version: https://lakens.github.io/statistical_inferences/04-bayes.html
And I am using Safari.
P.

On 23. 10. 2023, at 13:51, Daniel Lakens @.***> wrote:
Hi, I only see it misses a closing parenthesis in the online version. Are you using a pdf? I do not k ow why the math does not render for you.

From: PetrPalisek @.>
Sent: Sunday, October 22, 2023 11:40:54 PM
To: Lakens/statistical_inferences @.>
Cc: Subscribed @.***>
Subject: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Hi,
I think there is a typo in chapter 4.1.
Original: Bayesian statistics the probability of data under a specified model (P|D() is a number that expressed what is sometimes referred to as the absolute evidence
Correct: Bayesian statistics the probability of data under a specified model P(D|H) is a number that expressed what is sometimes referred to as the absolute evidence
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYJCOY3FVHNMBE7HG33YAWHGNAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2TMMBWGYZDGOA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775023543, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOTGMWIYWGDWNSHSSBKMOQTYAZK27AVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGAZDGNJUGM.
You are receiving this because you authored the thread.

Thanks! Then I guess it does not work on your safari setup. It shows up fine on my phone and desktop.  I have never heard this before so it might be some specific settings on your end, as Safari is used more widely, and a lot of students have used this book. There is nothing I can do about it, I fear.

From: PetrPalisek @.>
Sent: Monday, October 23, 2023 2:14:13 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Comment @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Hi,
here's a screenshot.
￼
It's the web version: https://lakens.github.io/statistical_inferences/04-bayes.html
And I am using Safari.
P.

On 23. 10. 2023, at 13:51, Daniel Lakens @.***> wrote:
Hi, I only see it misses a closing parenthesis in the online version. Are you using a pdf? I do not k ow why the math does not render for you.

From: PetrPalisek @.>
Sent: Sunday, October 22, 2023 11:40:54 PM
To: Lakens/statistical_inferences @.>
Cc: Subscribed @.***>
Subject: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Hi,
I think there is a typo in chapter 4.1.
Original: Bayesian statistics the probability of data under a specified model (P|D() is a number that expressed what is sometimes referred to as the absolute evidence
Correct: Bayesian statistics the probability of data under a specified model P(D|H) is a number that expressed what is sometimes referred to as the absolute evidence
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYJCOY3FVHNMBE7HG33YAWHGNAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2TMMBWGYZDGOA.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775023543, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOTGMWIYWGDWNSHSSBKMOQTYAZK27AVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGAZDGNJUGM.
You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775059132, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYO432XYZZMMLCQ6UMDYAZNRLAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA2TSMJTGI.
You are receiving this because you commented.Message ID: @.***>

Ps your screenshot is also not visible to me.

Yeah, might be! Good thing the rest of the book works.
Here is the screenshot: https://ibb.co/2NT1qmY

We have misunderstood each other. In your copy pasted issue the H0 was missing from the formula! Now I see that is not at all what you meant!

From: PetrPalisek @.>
Sent: Monday, October 23, 2023 2:24:23 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Comment @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Yeah, might be! Good thing the rest of the book works.
Here is the screenshot: https://ibb.co/2NT1qmY
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775076379, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYN7XUURPK6XSLVR4FTYAZOXPAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA3TMMZXHE.
You are receiving this because you commented.Message ID: @.***>

So, I think I do not understand what your issue is. Can you explain? The screenshot has H0 in there. And yet, that is the change you made in your issue. I am confused what your problem is. Can you explain it like I am 4?

From: Daniel Lakens @.>
Sent: Monday, October 23, 2023 2:27:31 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Your activity @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
We have misunderstood each other. In your copy pasted issue the H0 was missing from the formula! Now I see that is not at all what you meant!

From: PetrPalisek @.>
Sent: Monday, October 23, 2023 2:24:23 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Comment @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Yeah, might be! Good thing the rest of the book works.
Here is the screenshot: https://ibb.co/2NT1qmY
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775076379, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYN7XUURPK6XSLVR4FTYAZOXPAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA3TMMZXHE.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775081463, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYIT6SCD5TVM22QLCPTYAZPDHAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA4DCNBWGM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Ah! Yes I get it now. Will fix it. Was on my phone traveling to Germany, so difficult to get it.

From: PetrPalisek @.>
Sent: Monday, October 23, 2023 4:20:44 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Comment @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
:D
Ok, sure!
In the book (the third line of the 4.1 chapter) I see the following expression: P|D(H0) I think it is wrong because it is supposed to denote the probability of data given H0, i.e.: P(D | H0).
So now the part is "data under a specifed model (P|D(H0))" while it should be "data under a specified model P(D | H0)".

On 23. 10. 2023, at 14:29, Daniel Lakens @.***> wrote:
So, I think I do not understand what your issue is. Can you explain? The screenshot has H0 in there. And yet, that is the change you made in your issue. I am confused what your problem is. Can you explain it like I am 4?

From: Daniel Lakens @.>
Sent: Monday, October 23, 2023 2:27:31 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Your activity @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
We have misunderstood each other. In your copy pasted issue the H0 was missing from the formula! Now I see that is not at all what you meant!

From: PetrPalisek @.>
Sent: Monday, October 23, 2023 2:24:23 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Comment @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Yeah, might be! Good thing the rest of the book works.
Here is the screenshot: https://ibb.co/2NT1qmY
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775076379, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYN7XUURPK6XSLVR4FTYAZOXPAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA3TMMZXHE.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775081463, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYIT6SCD5TVM22QLCPTYAZPDHAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA4DCNBWGM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775085351, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOTGMWMMK2TOKZOH7DOCHNLYAZPLVAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA4DKMZVGE.
You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775314390, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYO73YJYNFGDK4OTONLYAZ4LZAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGMYTIMZZGA.
You are receiving this because you commented.Message ID: @.***>

Great! :)
Thanks.

On 23. 10. 2023, at 16:29, Daniel Lakens @.***> wrote:
Ah! Yes I get it now. Will fix it. Was on my phone traveling to Germany, so difficult to get it.

From: PetrPalisek @.>
Sent: Monday, October 23, 2023 4:20:44 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Comment @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
:D
Ok, sure!
In the book (the third line of the 4.1 chapter) I see the following expression: P|D(H0) I think it is wrong because it is supposed to denote the probability of data given H0, i.e.: P(D | H0).
So now the part is "data under a specifed model (P|D(H0))" while it should be "data under a specified model P(D | H0)".

On 23. 10. 2023, at 14:29, Daniel Lakens @.***> wrote:
So, I think I do not understand what your issue is. Can you explain? The screenshot has H0 in there. And yet, that is the change you made in your issue. I am confused what your problem is. Can you explain it like I am 4?

From: Daniel Lakens @.>
Sent: Monday, October 23, 2023 2:27:31 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Your activity @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
We have misunderstood each other. In your copy pasted issue the H0 was missing from the formula! Now I see that is not at all what you meant!

From: PetrPalisek @.>
Sent: Monday, October 23, 2023 2:24:23 PM
To: Lakens/statistical_inferences @.>
Cc: Lakens, Daniël @.>; Comment @.>
Subject: Re: [Lakens/statistical_inferences] Typo in 4.1 (Issue #102)
Yeah, might be! Good thing the rest of the book works.
Here is the screenshot: https://ibb.co/2NT1qmY
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775076379, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYN7XUURPK6XSLVR4FTYAZOXPAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA3TMMZXHE.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775081463, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYIT6SCD5TVM22QLCPTYAZPDHAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA4DCNBWGM.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775085351, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOTGMWMMK2TOKZOH7DOCHNLYAZPLVAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGA4DKMZVGE.
You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHubhttps://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775314390, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABRQPYO73YJYNFGDK4OTONLYAZ4LZAVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGMYTIMZZGA.
You are receiving this because you commented.Message ID: @.***>
—
Reply to this email directly, view it on GitHub https://github.com/Lakens/statistical_inferences/issues/102#issuecomment-1775332501, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOTGMWI7DLVHRIENAYTC44LYAZ5K7AVCNFSM6AAAAAA6LFQNQSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZVGMZTENJQGE.
You are receiving this because you authored the thread.


OOM server error

I have a basic usecase counter, however because GetEncoding is so expensive, my server with 50M of memory gets an OOM error immediately with only about 7 go routines calling it at the same time. It would be great if this was optimized or if the tkm could be shared and re-used
func countTokens(messages []openai.ChatCompletionMessage) int {
	tkm, err := tiktoken.GetEncoding(tiktoken.MODEL_CL100K_BASE)
	if err != nil {
		panic(err)
	}

	tokensPerMessage := 3
	var tokenCount int
	for _, message := range messages {
		tokenCount += tokensPerMessage
		tokenCount += len(tkm.Encode(message.Content, nil, nil))
		tokenCount += len(tkm.Encode(message.Role, nil, nil))
	}
	tokenCount += tokensPerMessage // every reply is primed with <|start|>assistant<|message|>

	return tokenCount
}

In order to allow multiple Encodings to be initialized concurrently, I did not lock getEncoding() but only used sync.Once to ensure the same instance is returned for the same Encoding name.
Now it seems that getEncoding() is a very expensive operation and perhaps should not be allowed to execute concurrently, as it will allocate too much memory during execution.

@pkoukk thank you so much for the quick response and enhancement. Very cool.
I just want to clarify that with this update, I can generate a single encoding in the main.go file during server initialization and pass it to the handler so that requests and threads created within the request can reuse the same encoding maps concurrently?

Sure, of course you can.
You also can initialization tiktoken in go routines.
Now 10 threads only need allocate about 10M of space, which is 1/10 of the original.


TF-Lite Micro: port Silicon Labs STK3701A to Micro Speech example

This ports the Silicon Labs STK3701A development board to the TF-Lite Micro Micro Speech example

@sldriedler Thank you for your contribution. Can you please sign CLA? Thanks!

Still waiting on the legal team.. w


Priority settings in uptime kuma

The priority settings is not working in uptime kuma. No matter set it on 1-5 it still sends notification with the medium priority.
Is there any way to fix it? I looked at the docs but couldn't find anything there.

That seems to be an issue with Uptime Kuma, and not ntfy. Please open an issue in their issue tracker.


Some bugfixes, M3U standard, Phoniebox Editions

@MiczFlor
I wrote a upgrade guide into UPGRADE.md. Please merge it into develop branch. This update should be done by ALL USERS, which are on 1.1.7 (with or without spotify) because i improved handling and built the "Editions" we spoke about.
It's late now, i hope i got all things from my E-Mail into this version.
IMPORTANT: i tested recursive folders and they are working very fine in 1.1.7 (and this 1.1.8-beta).
the only issue i know is that i can't use the idle shutdown. Or i haven't understand this function ;-)


Group 9.14 behavior when multiple US Core-defined v2 scopes apply to the resource

Hello,
USCDIv3 defines the Health Concern data element as part of the Problems data class. US Core 6 introduced granular scopes of:

Condition.rs?category=problem-list-item
Condition.rs?category=health-concern

These concepts can overlap, but across SMART, US Core, CCG, and HTI-1, authorization server behavior for overlapping granular scopes is not formally defined.
For example, consider a Condition with a category of both health-concern and problem-list-item, and an app authorized for patient/Condition.rs?category=health-concern but not for patient/Condition.rs?category=problem-list-item.
The g(10) Test Kit Group 9.14 tests, as currently implemented, require the Condition with both categories (above) to be shared with the app, but –

It’s unclear that this is the "correct" behavior. The opposite behavior is also justifiable
There’s no normative or regulatory language defining this expectation

This has sparked a conversation at chat.fhir.org which hasn't yet reached consensus: https://chat.fhir.org/#narrow/stream/179175-argonaut/topic/authorization.20of.20a.20resource.20that's.20in.20multiple.20categories
Ideally, US Core will patch a clear requirement to US Core 6.1 as part of the resolution to https://jira.hl7.org/browse/FHIR-46748, by defining normative requirements for multiple SMART granular scopes applied to a single FHIR resource. Until then, the extent of guidance to implementers is this section of the SMART App Launch spec:

As such, clients can attempt to perform FHIR operations based on the scopes they are granted — but depending on the details of the underlying permission system (e.g., the permissions of the approving user and/or permissions assigned in a client-specific policy) these requests may be rejected, or results may be omitted from responses.

Given the current ambiguity, and ultimate responsibility of the authorization server to determine appropriateness of sharing data, we suggest that test group 9.14 be modified to not mandate behavior around resources to which multiple granular scopes could apply.
This is detail is important, and also time-sensitive given certification timelines, so we’d appreciate any determination or clarification on this behavior.
Test Group 9.14, for reference: https://inferno.healthit.gov/suites/g10_certification/k32hPWG2xla#9.14

Hey inferno folks, FYI - CGP just voted on this jira -- https://jira.hl7.org/browse/FHIR-46748, setting a direction for implementers of US Core to return resources which are described by multiple OAuth scopes, for which the client is authorized for only some of.

The test kit's current behavior matches the above Jira's clarification, so closing this issue


Can I have a client for windows and Mac OS as well?

Hi can I have a client for windows and Mac OS as well，how do I use docker, can you give me a example within the document?
（有win和Mac 客户端？另外 docker 怎样使用也不清楚）

Not sure what you mean but it already works on Windows and macOS.
If you want a GUI, we have no plans to develop it yet. Consider contacting authors of existing tools for hysteria support.


Requiring esm doesn't do anything

Description
It seems that requiring esm in the sidebar config doesn't do anything while it works fine from console.
https://github.com/standard-things/esm
   trying to searching for tests using these settings:

    mocha path: mocha
    test files location: ./esnext/**/*.spec.js
    files to ignore: **/.git/**/*,**/node_modules/**/*
    environmets: {}
    requires: ["esm"]
    options:  {}

   if you find anything wrong please change those default settings
____________________________________________________________________________
********/Cluster.spec.js:1
(function (exports, require, module, __filename, __dirname) { import { PlanarCluster } from "./Cluster.js";
                                                                     ^

Steps to Reproduce

Install esm: npm i --save esm
Paste into file with a test:

import * as path from "path";

Run test through sidebar

Same example will run fine from console: mocha -r esm esnext/**/*.spec.js
Expected Results
Shouldn't crash
Actual Results
Crashes because requiring esm doesn't seem to do anything
Versions

VScode: v1.28.0
Mocha SideBar: v0.20.18
Node Version: v8.9.3

OS version

[x] Linux
[x] Mac
[x] Windows

Same here, did you find a workaround?

I create .babelrc like this
{
  "presets": [
    [
      "@babel/preset-env",
      {
        "targets": "> 0.25%, not dead"
      },
    ]
  ],
  "exclude": "node_modules/**",
}

then the mocha-sidebar can work .
But when it paused on breakpoint , that line didn't highlighted, and debug-siderbar tell me that file was skipped by smartStep.

+1

+1


build: remove luet and old elemental

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:
We want to bump elemental to the newer version and drop luet.
Solution:
update the build flow to match the new elemental binary
Related Issue:
None
Test plan:

Make sure the installation work as usual
Make sure the cluster operation as susal

Additional Context:
It depends on the harvester-isntaller PR https://github.com/harvester/harvester-installer/pull/606

@mergifyio backport v1.2


hsla vs rgba

Hi Philip!
Why do you prefere use background: hsla(0, 0%, 0%, 0.1) insted of background: rgba(0, 0%, 0%, 0.1) like here? Performance?
Thanks a lot

No, not performance. I prefer hsl because it's easier for humans to read and understand. This Stack Overflow post summarizes it well:
https://stackoverflow.com/questions/26059228/css-hsl-or-rgba-colors

Ok, thanks 😊


Score on leaderboard should say 0 not None

On leaderboard, teams with 0 solves show up as "None". Should say 0.
Almost certainly handled at views.py - https://github.com/PuzzleTechHub/myus/blob/main/myus/myus/views.py#L169
Most likely fix was to use Coalesce, but doesnt work -
            .annotate(score=Coalesce(Sum('puzzle__points'), 0)) (instead of line 183).

Hi @TheOriginalSoni , I was looking at the issue and wanted to know if anyone is taking it so I can take it myself.

Nobody has taken it yet, so absolutely feel free to take it up! We are also mostly coordinating over the PTH general discord if you wanna pipe over there/take up our roles

Hey @TheOriginalSoni, I fix the issue but I'm unable to submit the PR, can I get some help for that?

You should be able to click on "Create New Pull request" on https://github.com/PuzzleTechHub/myus/pulls once you upload the changes to your fork of this repo. Ping me on discord or give screenshots if you aren't sure.

@TheOriginalSoni I created the [PR}(https://github.com/PuzzleTechHub/myus/pull/36)

Looked good, tested it and it works.  Merged. Thanks @AlejandroBlanco2001


PIP-240: A new API to unload subscriptions

Motivation
We sometimes try to unload the topic to resolve some consumption-stop issues. But the unloading topic will also impact the producer side.
Goal
Providing a new API to unload the subscription dimension triggers reconnection of all consumers on that subscription and reconnection is guaranteed by the client. The API will be used in these ways:

unload special subscription of one topic(or partitioned topic)
unload all subscriptions of one topic(or partitioned topic)
unload subscriptions of one topic(or partitioned topic) by regular expression

If a reader's subscription name is not set, a random subscription name prefixed with 'multiTopicsReader-' or 'reader-' will be used, and users can uninstall these subscriptions using regular expressions.

In addition to triggering consumer disconnection, Unloading Subscribers will restart the Dispatcher, which resets the redeliver message queue and delayed message queue in the Broker's memory, which can help resolve issues caused by an abnormal dispatcher state. However, the execution flow of Unloading Subscribers does not include a restart of the Managed Cursor related to this dispatcher; if there is a problem with the cursor, we can only rely on the unload topic to solve it.
Note: From the client's perspective, this connection may be shared by consumers, producers, and transactions, so Unloading Subscribers maybe impact the producer and transaction.
These scenarios are not supported

Functions message-dedup, geo-replication, and shadow-topic also read messages from the topic, but Unloading subscribers will not support triggering restarts of these three functions( because the cursor is used directly to read the data in these scenarios, not the consumer or reader ).
The Compression task(subscription name is __compaction) also use a reader to read data, but Unloading Subscribers does not support it because this task creates a new reader each time it starts.
Do not support all topics related to Transaction features.

__transaction_buffer_snapshot works with the task TB recover,  and this task will create a new reader each time they start.
__transaction_pending_ack works with the task Transaction Pending Ack Store replay,  and this task will use managed cursor directly to read data.
__transaction_log_xxx works with the task Transaction Log, which will use managed cursor directly to read data.
transaction_coordinator_assign No data will be written on this topic.

Special system topic supports
The system topic __change_events is used to support topic-level policies, there may also be some message delivery issues in this scenario, so Unloading Subscribers will support this topic.
API Changes
For persistent topic
pulsar-admin persistent unload {topic_name} -s {sub_name}

For non-persistent topic
pulsar-admin non-persistent unload {topic_name} -s {sub_name}

Explain the param -s

set param -s to special sub name to unload special subscription
set param -s to ** to unload all subscriptions under this topic
set param -s to regexp to unload a batch subscriptions under this topic

In my experience if a subscription is stuck, the problems are:

the client has some problems - 99.9%
there is a bug in Pulsar - 0.1%

Unloading a topic is an operation that triggers some reset of the state on both the broker and the clients and this usually TEMPORARY unblocks the subscription.
If there is a problem we should spend time on investigating the problem and not in adding this kind of tools.

I think that the terminology of unload is wrong. These are reset operations.

Hi @eolivelli

If there is a problem we should spend time on investigating the problem and not in adding this kind of tools.

You are right. When a user encounters a problem, it often takes a while actually to solve the root cause. It is important to provide a tool to recover services quickly, and cmd 'unload topic' is often used to solve problems temporarily.
But now a topic can be used by multiple teams, such as Business-team A to produce messages, Business-team B and C to consume messages (each using a different subscription name), and Data-team D to read messages using reader api (using a random subscription name). When there is a consumption problem in subscription B, if we do unload the topic, this will affect teams A, B, C, D, and when we provide a new API to reset subscribers, the impact can be controlled only to affect team B.
So I think a new API is necessary。
Thanks

Hi @dave2wave

I think that the terminology of unload is wrong. These are reset operations.

already change unload to reset

As much as I also like unloading topics to resolve mysterious issues, I'd rather pulsar not add additional "band-aid/manual reset" command functionality to work around either misbehaving clients or broker state-corrupting bugs. I think doing that has a short-term benefit of providing a tool for expert operators to use to resolve issues, but has a long-term cost of making brand new adopters of pulsar find it less reliable than they want it to be.
I'm not saying "fix the bugs instead" (though we should also fix things!), but rather that building more support code for stuff like this may come at a time/complexity cost with actually fixing the issues that would make a "reset" necessary.


static fonts from single-master sources have overlaps, skewing glyf ttx diff

Following #971, ttx_diff.py now invokes fontmake -o ttf instead of -o variable when the input source contains only one master, in order to avoid fontmake adding unnecessary empty/noop variable tables (https://github.com/googlefonts/fontmake/issues/1125).
But fontmake -o ttf also passes the source through the ufo2ft's remove overlaps filter (using booleanOperations by default, or skia-pathops if requested via --overlaps-backend pathops).
fontc being variable-first font compile does not attempt nor support removing the overlaps (as it would destroy master compatibility for a VF target).
Therefore comparing fontc-built single-master sources vs fontmake-built ones will usually produce several glyph diffs in glyph-related tables like glyf, maxp, etc. because the number of outline points when glyphs contain overlapping contours will be different between the two fonts.
For example, take this test font, containing only one glyph "T" with overlapping contours, and compare it with ttx_diff.py:
SingleMaster.glyphs.zip
--- build/default/fontc.ttf
+++ build/default/fontmake.ttf
@@ -12,12 +12,12 @@
     <!-- Most of this table will be recalculated by the compiler -->
     <tableVersion value="1.0"/>
     <fontRevision value="1.0"/>
-    <checkSumAdjustment value="0x0"/>
+    <checkSumAdjustment value="0xd88e9083"/>
     <magicNumber value="0x5f0f3cf5"/>
     <flags value="00000000 00000011"/>
     <unitsPerEm value="1000"/>
     <created value="Mon Sep 30 16:38:24 2024"/>
-    <modified value="Tue Oct  1 10:17:24 2024"/>
+    <modified value="Tue Oct  1 10:17:25 2024"/>
     <xMin value="26"/>
     <yMin value="-200"/>
     <xMax value="574"/>
@@ -53,7 +53,7 @@
     <!-- Most of this table will be recalculated by the compiler -->
     <tableVersion value="0x10000"/>
     <numGlyphs value="3"/>
-    <maxPoints value="10"/>
+    <maxPoints value="8"/>
     <maxContours value="2"/>
     <maxCompositePoints value="0"/>
     <maxCompositeContours value="0"/>
@@ -169,12 +169,10 @@
         <pt x="26" y="723" on="1"/>
         <pt x="574" y="723" on="1"/>
         <pt x="574" y="578" on="1"/>
-        <pt x="363" y="578" on="1"/>
-        <pt x="408" y="623" on="1"/>
+        <pt x="408" y="578" on="1"/>
         <pt x="408" y="0" on="1"/>
         <pt x="209" y="0" on="1"/>
-        <pt x="209" y="623" on="1"/>
-        <pt x="254" y="578" on="1"/>
+        <pt x="209" y="578" on="1"/>
         <pt x="26" y="578" on="1"/>
       </contour>
       <instructions/>

Why not add --keep-overlaps to the fontmake call? It won't do any harm on the variable font build.

of couse we can do that for the time being, until we have remove overlaps in rust, but the comparison would not be fair/realistic, since most static builds do/should have overlaps removed.

after applying #979, ttx_diff'ing the above SingleMaster.glyphs test file yields "output is identical", though it feels a bit like cheating to me :)
I defer to @rsheeter

@raphlinus just noted https://github.com/GraphiteEditor/Graphite/tree/master/libraries/path-bool is now a thing, seems to be a port of a js pathops library. This one is new to me.


add mooSushiWFTM-LINK

Contracts are manually tested (as per https://github.com/beefyfinance/beefy-contracts/blob/master/readme.md).
Ownership is transferred.
This is my first PR for beefy, so feel free to tell me to do any changes that are needed and a bit more caution before merging might be required :-)
Vault: 0x912d3e6c4d3CE89Ac85e3FbB22631D5E85D5158a
Strategy: 0xF6075D0815d2a540651127cBCDdC01b5AA20672c
Want: 0x1Ca86e57103564F47fFCea7259a6ce8Cc1301549
PoolId: 7

should have harvestOnDeposit=true and new vaults to be placed on top below maxi and boosts.
(i will fix it)

thanks, acknowledged.


feat!: removes Prove from the NMT wrapper

Overview
Closes #1547
There is no usage of the Prove API of the NMT Wrapper in Celestia-node and Celestia-app as of the following commits:
https://github.com/celestiaorg/celestia-node/commit/0d4e4bb8c74baac814f7b32a87ac80a0aa7a4088 and https://github.com/celestiaorg/celestia-app/commit/3767e3eb35230eb50e44c7c2f8b49ed3701d346b.
Checklist

[x] New and updated code has appropriate documentation
[x] New and updated code has new and/or updated testing
[x] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[x] Linked issues closed with keywords

Codecov Report

Merging #1645 (034e1c9) into main (3767e3e) will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1645      +/-   ##
==========================================
+ Coverage   50.96%   50.98%   +0.01%
==========================================
  Files          92       92
  Lines        5751     5749       -2
==========================================
  Hits         2931     2931
+ Misses       2520     2518       -2
  Partials      300      300

Impacted Files
Coverage Δ

pkg/wrapper/nmt_wrapper.go
80.43% <ø> (+3.35%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Developer instructions fail on mac

Following the instructions https://a-ev.github.io/DACE.jl/dev/tutorials/setting-up-your-development-environment/ fails when running using DACE with error
ERROR: LoadError: UndefVarError: `StdFill` not defined

Should be working now with this additional step on mac: https://a-ev.github.io/DACE.jl/stable/tutorials/setting-up-your-development-environment/#On-Mac-OS-X-only:-build-our-own-version-of-libcxxwrap-julia


misc: add user-defined chokidar files to watch (2)

@Atinux @clarkdo This is me picking up on this PR, which seems abandoned. I've refactored CLI tests a bit and added a nuxt dev test to make sure the custom chokidar watchers were working.
cc @SabatinoMasala

Codecov Report

Merging #3633 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #3633   +/-   ##
=======================================
  Coverage   99.19%   99.19%
=======================================
  Files          18       18
  Lines        1122     1122
  Branches      302      302
=======================================
  Hits         1113     1113
  Misses          9        9

Continue to review full report at Codecov.

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


[FEATURE]

Description
Thinking it is redundant to leave the Pic-Text Label as "Drag & drop your files or browse", maybe it could be 'Browse files' on smaller screen sizes.
Screenshots

It's great having you contribute to this project by creating an issue Make sure you are assigned to this before you work on it and you read the Contributing.md file, Thank you! Welcome to the community :nerd_face:

Sure, Dunsin

I use a library for this, I don't understand the issue here actually

I use a library for this, I don't understand the issue here actually

Maybe he wants to say that we can change text from 'drag & drop files' to only 'browse files', for small screen devices.

I use a library for this, I don't understand the issue here actually

Maybe he wants to say that we can change text from 'drag & drop files' to only 'browse files', for small screen devices.
Because we can't actually drag files from small screen devices.

I see

yeah, I meant the text of the label.

We use file pond library for the input, if you find a way to change it based on viewport then go for it.

Ok

@clinton-lynx Maybe you can refer to this section of File Pond's documentation for the task!

@Dun-sin if nobody is working on this issue, could you please assign this to me?

:tada: This issue has been resolved in version 2.6.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update autograd_cn.rst

优化背景描述

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Unable to find the documentation for cloud deployment

Hi Team,
I am implementing the same project for the City of Erfurt. Currently, I am trying to deploy the backend (fargate cluster). But I am unable to do so. I want to ask how you deploy the code on the aws cloud?
I didn't find any ecs-cli or aws-cli commands in the repository. Please guide me in the right direction. I am trying to deploy the app on aws directly. I have a postgres db ready. But I am unable to find the documentation for the deployment over the cloud.
Regards
Bilal Zaidi

Hi,
We are using terraform to deploy to fargate and to provision the S3. See terraform/ecs-harvester in this repo.
But actually having GitHub action would be enough in this case.

Hi,
Thanks for the reply. I have deployed the backend using terraform scripts. But the ECS task is not able to connect with databases. I have Terraform 0.13.5 . If I create a new task it ends and the following error can be found in cloudwatch.

Does the DB exist?
Are they in the same VPC?
Is the DB ingress restricted?
Can you run the harvester.py from your local machine?
AWS config is kind of black magic. Maybe running it from GitHub actions is easier.

Yes, It was the issue because the database was not created by terraform. while debugging I was able to create the DB.
Thank you so much for the response.

@syedmbzaidi You are welcome. The documentation is not that great.  The schema for the DB can be found in here  which might be a little outdated but still should work. We added some extra columns there are pretty berlin specific.


Caliper zookeeper setup problem

I use caliper to test hyperledger fabric, and try to use zookeeper type to do distributed test.But i got a problem.
Context

firstly, i use command:
caliper zooservice start

it looks like good.
to run a local docker zookeeper nodes.
And then,run command:
caliper zooclient start --caliper-zooaddress <IP_ADDRESS>:2181 --caliper-workspace ~/Documents/caliper --caliper-networkconfig packages/caliper-samples/network/fabric-v1.4/2org1peergoleveldb/fabric-go-tls.yaml

no more log output, it's strange.
and the zookeeper's logs output:

when i run benchmark test , caliper told me no nodes register in zookeeper.so ....
Expected Behavior

Actual Behavior

Possible Fix

Steps to Reproduce

Existing issues

[ ] Stack Overflow issues
[ ] GitHub Issues
[ ] Rocket Chat history

Context

Your Environment

Version used: v0.1.0
Environment name and version (e.g. Chrome 39, node.js 5.4): node version:v8.16.0
Operating System and version (desktop or mobile): ubuntu 19 or centos 7 both have these problem
Link to your project:

@TopJohn The zookeeper-based clients will be removed in v0.2.0 and a new solution will take its place in v0.3.0
Until then, try scaling your local clients (and resources).


Play Audio File

Play an audio file in the program.

git fetch origin
git checkout 13-play-audio-file


[RFC] Golden Path to plugin development

Need
While our documentation is slowly getting better, there is a need for a more step-by-step tutorial that guides a new plugin developer through the full process of building a shipping a plugin. Inside Spotify we call these tutorials Golden Paths:
https://engineering.atspotify.com/2020/08/17/how-we-use-golden-paths-to-solve-fragmentation-in-our-software-ecosystem/
Proposed solution
Create a "Golden Path to plugin development" with the following outline (details are out of scope for this RFC):
Part 1: Creating a new plugin

What to build (use PagerDuty example)
Creating a new plugin
Finding your plugin

Part 2: Building the UI

Layout and design
Testing

Part 3: Fetch external data

Get a token from PagerDuty
Add to the proxy
Storing token and/or secrets

Part 4: Add plugin to Service Catalog

Add plugin to App
Adding navigation and tab
Link plugin to Component definition (annotations:) should this association be elsewhere? 🤔
Integrate with a Backstage Software Template

Part 5: Publish your plugin

Distribute plugin via NPM
Add plugin to Marketplace

We're still planning on this one 😄  Perhaps with some targeted high-value cases (catalog ingestion, for example).


Adding listicle feature along with the plugins and MU plugins

Adding listicle feature along with the plugins and MU plugins

Visual regression test failed!

Lighthouse test passed! The score of 51 isn't less than the acceptable score of 17 (5 less than the score of 22 on the master branch)
View the full Lighthouse performance report for pull/15 and compare it to the Lighthouse performance report for the master branch


Next.js v6/7 _app.js causing error when deploying

I've upgraded my app to use v6.1.2 Next.js to use _app.jsx so that I can use next-ga for Google Analytics. When I deploy I get this error message:
{ Error: (client) ./pages/_app.jsx
Module build failed: Error: Cannot find module 'babel-plugin-module-resolver' from '/home/nowuser/src'

This problem persists when I use _app.js instead of _app.jsx.
Upgrading to v7.0.0 gives a slightly different error message:
{ Error: (client) ./pages/_app.js
Module build failed (from ./node_modules/next/dist/build/webpack/loaders/next-babel-loader.js):

Can someone please help with this issue?

Thank you so much for the effort you have put into this issue! 😊
However, since we completely re-worked the entire codebase of Now CLI and this issue does not apply to the new code anymore, I'm closing it now.
You can read more about our biggest release yet here: https://zeit.co/blog/now-2


Unable to train to convergence (small dataset)

Hi nice work with x-clip. Hoping to play around with it and eventually combine it into your DALLE2 work.
Currently having some trouble training on roughly 30k image-text pairs. Loss eventually goes negative and starts producing Nan's. I've dropped learning rate down (1e-4) and I'm clipping gradients (max_norm=0.5).
Any thoughts on what are sane training params/configs on such a small dataset using x-clip?

I'm noticing the same on my own dataset of ~175k text-image pairs, so maybe it's not a dataset size issue?
To add my own data of what I've tried so far to resolve the NaN issue:
I originally tried training with a learning rate of 1e-4 and a training batch size of 32. Training would run for about 8 hours or so on my 3090, but eventually NaNs would consistently take over, no matter how much I rolled back to a prior checkpoint. I noticed that the loss value would also have dramatic swings while training (+/- 3.0 between consecutive training iterations, it was hard to tell if the loss was actually trending any direction).
I've tried the following so far:

Set is_multiview=true on the constructor, & supplying aug_text and aug_image while training

I generated 32 augmentations for both image and text before passing it into the model.

I don't think it's documented anywhere, so if you want to do this yourself, aug_text and aug_image should be a tensor with the shape [augment_count, batch_size, ... ]. Alternatively you can pass a list of tensors the same shape as text or image, instead of a single tensor.

This required using a different tokenizer since the SimpleTokenizer was slowing down training at this scale. (I wound up using huggingface's tokenizers, which was very straightforward to train and integrate into x-clip)
To keep everything in memory on my gpu, I had to drop the training batch size down to 16.
This didn't seem to help training stability very much, but I did start seeing the model produce some early results. It seemed to be able to identify some of the most common tokens in my training images. However, I didn't test things very thoroughly at this point, so I may have just have picked a good couple of images to manually inspect the model on.
Unfortunately, the model never really progressed past that. Performance started to degrade, and then NaN loss values started happening again.

Increasing the batch size to 64 (+ 32 augmentations for each image and text), dropping learning rate to 1e-5

This is my current attempt I'm training.
I had to implement gradient accumulation for the training, since there was no hope of keeping everything in memory anymore.
This seems to have increased the stability of the loss value, there's no dramatic swings anymore (it tends to keep to a range of about +/- 0.25), and the loss has been steadily trending towards 0.
Results from this model have not been as good, at least not so far. I'm guessing that's because of the lowered learning rate.

I can update here if I manage to get things working well. I think I have the learning rate too low now, but I'll give it some time to see if things work out.

@jacobwjs hi Jacob! thanks for trying to train this! I've added numerically stable attention and some extra layernorms (recommended by a recent paper) to see if that helps with stability https://github.com/lucidrains/x-clip/commit/9d9bae0fbff65015d0227873df1b42bdeadd7be9 do let me know if it does or does not work, and we can try some other avenues

@jacobwjs hi Jacob! thanks for trying to train this! I've added numerically stable attention and some extra layernorms (recommended by a recent paper) to see if that helps with stability 9d9bae0 do let me know if it does or does not work, and we can try some other avenues

Beautiful. Will kick off training tonight and get back to you when I have results.

I'm noticing the same on my own dataset of ~175k text-image pairs, so maybe it's not a dataset size issue (or 200k is also not enough)?
To add my own data of what I've tried so far to resolve the NaN issue:
I originally tried training with a learning rate of 1e-4 and a training batch size of 32. Training would run for about 8 hours or so on my 3090, but eventually NaNs would consistently take over, no matter how much I rolled back to a prior checkpoint. I noticed that the loss value would also have dramatic swings while training (+/- 3.0 between consecutive training iterations, it was hard to tell if the loss was actually trending any direction).
I've tried the following so far:
* Set `is_multiview=true` on the constructor, & supplying aug_text and aug_image while training

  * I generated 32 augmentations for both image and text before passing it into the model.

    * I don't think it's documented anywhere, so if you want to do this yourself, aug_text and aug_image should be a tensor with the shape `[augment_count, batch_size, ... ]`. Alternatively you can pass a list of tensors the same shape as `text` or `image`, instead of a single tensor.
  * This required using a different tokenizer since the SimpleTokenizer was slowing down training at this scale. (I wound up using [huggingface's tokenizers](https://github.com/huggingface/tokenizers), which was very straightforward to train and integrate into x-clip)
  * To keep everything in memory on my gpu, I had to drop the training batch size down to 16.
  * This didn't seem to help training stability very much, but I did start seeing the model produce some early results. It seemed to be able to identify some of the most common tokens in my training images. However, I didn't test things very thoroughly at this point, so I may have just have picked a good couple of images to manually inspect the model on.
  * Unfortunately, the model never really progressed past that. Performance started to degrade, and then NaN loss values started happening again.

* Increasing the batch size to 64 (+ 32 augmentations for each image and text), dropping learning rate to 1e-5

  * This is my current attempt I'm training.
  * I had to implement gradient accumulation for the training, since there was no hope of keeping everything in memory anymore.
  * This seems to have increased the stability of the loss value, there's no dramatic swings anymore (it tends to keep to a range of about +/- 0.25), and the loss has been steadily trending towards 0.
  * Results from this model have not been as good, at least not so far. I'm guessing that's because of the lowered learning rate.

I can update here if I manage to get things working well. I think I have the learning rate too low now, but I'll give it some time to see if things work out.

Yes definitely interested. Please share how it goes.

My attempt with the larger batch size is still going without any NaNs so far in about 62 hours of training on my 3090. Currently the loss is hovering around -0.21, so it's possible they're just around the corner.
I'll let you know if and when they show up, but I would possibly recommend trying your training with a similar batch size if you haven't already. It's possible you may need a larger batch size than 64, because I'm also passing in the augmentations for each image. I guess my effective batch size would be 64*32=2048 with the augments.

@Netruk44 Thanks for the update. I've tried larger batch sizes (e.g. 384) and same issue. The fact that the loss is negative is still concerning. Still curious where you end up once training finishes however.
@lucidrains
Everything converges when I turn off "decoupled_contrastive_loss". This seems to be the culprit.

@jacobwjs Unfortunately, my machine power-cycled itself for some reason, so training on my x-clip model has stopped for now. I wanted to test out lucidrain's imagen model with my text-image dataset. I might come back to this, though.
For this model, as far as I could tell the loss never converged. It continued down below -1.0, but I never saw a NaN. Previous models would occasionally see a NaN loss while training before it got to the point where I couldn't train the model at all anymore due to the NaNs. So if the 384 batch size didn't help you, maybe the augmentations or the 1e-5 loss value helped to stabilize things for me. I also set text_rotary_pos_emb = True, but I don't think that affected the convergence much.
 I'd be curious to know how well the model works for you.

@jacobwjs Unfortunately, my machine power-cycled itself for some reason, so training on my x-clip model has stopped for now. I wanted to test out lucidrains's imagen model with my text-image dataset. I might come back to this, though.
For this model, as far as I could tell the loss never converged. It continued down below -1.0, but I never saw a NaN. Previous models would occasionally see a NaN loss while training before it got to the point where I couldn't train the model at all anymore due to the NaNs. So if the 384 batch size didn't help you, maybe the augmentations or the 1e-5 learning rate helped to stabilize things for me. I also set text_rotary_pos_emb = True, but I don't think that affected the convergence much.
 I'd be curious to know how well the model works for you.

Thanks for following up. Bummer on the lost run, but hopefully you have some checkpoints for when/if you come back to training.
I assumed most would be interested in getting imagen up and running. Good luck with it, and I'll be following along there as well.
I'll tweak a few more parts of x-clip before kicking off "real" training. Will get those results here eventually. Until then...


I need a "Worker" node configured with PENTAHO PDI ( CARTE ). I have tried to create an image from apache/dolphinscheduler-worker, but with no success.

I am testing version 3.1 of DS, and the changes introduced are impressive. I would like to make a POC with DS to introduce it in my organization, but I need your help.
I need help to have a "worker" node that has PENTAHO CARTE installed (in order to invoke the ETL process execution via HTTP). An example of an image with CARTE is "nentangso/pentaho-kettle-carte:latest"
Excuse me, but I need your help. I have tried to search for manuals, blogs, without success. I think DS is an excellent product (it should be more popular than Apache Airflow!), but I need CARTE integration. Can you help me, provide a model image, or provide documentation to achieve it?

Please don't open pointless pull request, use GitHub issues as you already opened one I'll close this.


Revert "MGMT-16807: set correct no proxy and proxyStatus"

This reverts commit 4ea5d9ec458e27175d107aa2c647c59c519b2d4d.
Reverting cause of rbac issue, will reopen with the fix

/override ci/prow/integration
/override ci/prow/ibu-e2e-flow


Missed Updating DatabaseOperation in previous Pull

Missed Updating DatabaseOperation in previous Pull

Hi @dominiqa, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;


Update docker-compose and .env-example

The docker-compose example is severely outdated and the past configuration used a docker image for the dashboard that did not allow you to configure the URLs to the various API endpoints if you self-hosted your own dashboard publicly.
The newest dashboard image allows you to do this so the docker-compose has been updated to use this image and the env variables have been updated accordingly.
Other variables have been updated in the docker-compose to support self-hosting a public instance.
A commented traefik configuration in the docker-compose for the dashboard service also allows the user to configure basic auth to protect a publicly-facing dashboard.

Thanks! Forgot to mention you need to run it in the root of the project. Merging.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f7c2148) 86.84% compared to head (5b4efd8) 86.80%.
Report is 14 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2397      +/-   ##
==========================================
- Coverage   86.84%   86.80%   -0.05%
==========================================
  Files          85       85
  Lines        9353     9328      -25
  Branches      495      489       -6
==========================================
- Hits         8123     8097      -26
- Misses       1230     1231       +1

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

Happy CNY. Is the CI something others such as myself can help with? Seems to be vercel-side and I can't find any CI docs on the subject

The issue is that pnpm e2e fails. I dropped a comment. Thanks and sorry this fell through the cracks.

Should the pnpm lock be committed or gitignored or just not added?

well e2e works on my system. looks like problem with github actions

well e2e works on my system. looks like problem with github actions

It doesn't work for me though. Just checking out your branch and running pnpm e2e leads to various issues:

First the .env file reference in the docker-compose file doesn't exist.
Changing the reference to .env.example leads to issues with the jwt secret being too short
After fixing that things still don't work, I suspect due to the values of NHOST_HOST, PROXY_HOST and PROXY_URL.

First the .env file reference in the docker-compose file doesn't exist.

We expect the user to copy the .env file from the .env.example per the docker-compose README.
The e2e tests currently run/don't run on my computer because the afterAll test in docker-compose.test.ts is called right before the it() tests and runs at the same time as the beforeAll. This leads is to complete before the it() tests leading to the graphql-engine container going down/becoming unhealthy before all the it() tests complete.
Commenting
afterAll(async () => {
      // * Stop docker compose
      await promisifiedExec('docker compose -f docker-compose.yaml --env-file .env.example down')
      }, 5 * 60 * 1000)

allows the tests to pass. afterAll seems like it's only supposed to run after all the tests have executed but it's currently not doing that.

allows the tests to pass. afterAll seems like it's only supposed to run after all the tests have executed but it's currently not doing that.

afterAll runs after all the tests have run. If that's not the case for you it most likely means beforeAll failed and the tests are being skipped. You can verify on the branch:
(main) ❯ pnpm e2e

><EMAIL_ADDRESS>e2e /Users/dbarroso/workspace/nhost/nhost/examples/docker-compose
> vitest run

 RUN  v0.32.4 /Users/dbarroso/workspace/nhost/nhost/examples/docker-compose

 ✓ test/docker-compose.test.ts (5) 72454ms
   ✓ docker-compose should start, work and stop (5) 72454ms
     ✓ Hasura
     ✓ Hasura auth (retry x44)
     ✓ Hasura storage
     ✓ Serverless functions
     ✓ Dashboard

 Test Files  1 passed (1)
      Tests  5 passed (5)
   Start at  08:02:07
   Duration  72.66s (transform 23ms, setup 0ms, collect 29ms, tests 72.45s, environment 0ms, prepare 43ms)

However, on your branch things don't work for the reasons I explained here:
https://github.com/nhost/nhost/pull/2397#issuecomment-1938176555

Ok, with the suggestion I just added to your docker-compose.yaml pnpm e2e works fine without any change.

Sooo the current code is good?

Yes, if you accept my suggestion I will do a last check to make sure pnpm e2e works and if it does I will merge it. Thanks!

Great job, thanks a lot for the contribution.


hello world mod crashes GD instantly

Platform
Windows
Geode Version
v0.6.1
Mods Installed

devtools v0.0.5
fps bypass v1.0.0
the basic mod generated with geode new

Steps to Reproduce

Install the SDK and loader
Create new mod with geode new
Compile mod
Launch GD

Crash Report
https://pastebin.com/HJtyqAt2 - crashlog
Last log in geode/log is empty
Additional Information
I'm guessing I did something wrong while installing the SDK, no other idea why this happens.

Have you tried to remove the other mods to see if it's some issue related to them?

yep, it's definetly the mod that I compile that crashes the game

seems like youre compiling in debug, you have to compile in either release or relwtihdebinfo, otherwise std::string and other stl containers will crash

yeah, that was it. thanks a lot!


Version installation Error

I am running  jupyterlab version 3.0.14 and I am receiving the following error during installation:
Extension Installation Error
An error occurred installing jupyterlab-execute-time.
Error message:
The extension "jupyterlab-execute-time" does not yet support the current version of JupyterLab.
Conflicting Dependencies:
JupyterLab                  Extension      Package

=3.0.9 <3.1.0              >=2.0.2 <3.0.0 @jupyterlab/application
=3.0.9 <3.1.0              >=2.0.2 <3.0.0 @jupyterlab/notebook
=3.0.5 <3.1.0              >=2.0.1 <3.0.0 @jupyterlab/settingregistry

Please see new install directions (use pip): https://github.com/deshaw/jupyterlab-execute-time#install


Move to .NET Core 1.0.0 and new version of MongoDB Client

Based on #5, cleaned up some things and made the build working (hopefully, let's see what travis CI thinks).
When this is merged, #5 should be merged automatically as well :tada:, thanks @mattwhetton for doing most of the work 😄

🎉 🎉 🎉 💃 💃 💃 👯 👯 👯
all green, :shipit:


Categorize badges and maybe preview page?

It could be useful to put the images/SVGs into separate folders based on what they are for.
f.e. spigot or plugins containing the badges for platforms supported or downloads for badges about where you can download them from.
As an example: badges/downloads/modrinth_vector.svg

Another thing that could be nice to add is a page to preview the badges. Something like a PREVIEW.md that would display the badges and also show their Markdown-code to use.
Would allow people to more easily know what badges look like.
If wanted can I work on this myself and PR it later on.

This may break some of the badges people are currently using, but I could give it a try. Maybe I'll keep a legacy badges folder, and then have a new assets folder which is the one that should be used.

Should this be counted as documentation? Or moved somewhere else.

This is what I have done for organization

This may break some of the badges people are currently using, but I could give it a try. Maybe I'll keep a legacy badges folder, and then have a new assets folder which is the one that should be used.

Maybe use GitHub pages for this?
Like have the new badges on a github.io domain while keeping the old ones where they are right now.
Another alternative would be to create a new branch as the default to use while putting the current one as a separate branch. That way should image links remain the same, while you can provide a new format.

Should this be counted as documentation? Or moved somewhere else.

Would say it counts towards documentation, yeah.

I think going the github pages way works out, because I was going to make a website for this anyway (not a picker though)


when is the embedding generated by llama-index for the input text

Hi,
Below is the code that I am using to do inference on Fastchat LLM.
from llama_index import GPTListIndex, SimpleDirectoryReader, GPTVectorStoreIndex, PromptHelper, LLMPredictor
from langchain.embeddings.huggingface import HuggingFaceEmbeddings
from llama_index import LangchainEmbedding, ServiceContext
from transformers import T5Tokenizer, T5ForConditionalGeneration

model_name = 'lmsys/fastchat-t5-3b-v1.0'
tokenizer = T5Tokenizer.from_pretrained(model_name)
model = T5ForConditionalGeneration.from_pretrained(model_name)

from transformers import pipeline

pipe = pipeline(
    "text2text-generation", model=model, tokenizer=tokenizer,
    max_length=1024, temperature=0, top_p = 1,no_repeat_ngram_size=4, early_stopping=True
)

from langchain.llms import HuggingFacePipeline
llm = HuggingFacePipeline(pipeline=pipe)

embed_model = LangchainEmbedding(HuggingFaceEmbeddings())

# set maximum input size
max_input_size = 2048
# set number of output tokens
num_outputs = 512
# set maximum chunk overlap
max_chunk_overlap = 20
# set chunk size limit
chunk_size_limit = 512
prompt_helper = PromptHelper(max_input_size, num_outputs, max_chunk_overlap)

service_context = ServiceContext.from_defaults(embed_model=embed_model, llm_predictor=LLMPredictor(llm), prompt_helper=prompt_helper, chunk_size_limit=chunk_size_limit)

# build index
documents = SimpleDirectoryReader('data').load_data()

new_index = GPTListIndex.from_documents(documents, service_context=service_context)

# query with embed_model specified
query_engine = new_index.as_query_engine(
    retriever_mode="embedding",
    verbose=True,
    #streaming=True,
    similarity_top_k=1
    #service_context=service_context
)

response = query_engine.query("sample query question?")

Here the "data" folder has my full input text in pdf format, and am using the llama_index and langchain pipeline to build the index on that and fetch the relevant chunk to generate the prompt with context and query the FastChat model as shown in the code.
I want to understand when does llama_index generate the embeddings for the input text from the "data" folder.
is it at the time of indexing new_index = GPTListIndex.from_documents(documents, service_context=service_context) the embeddings are generated for all the nodes/chunks in the input text of document
or
at the time of query query_engine.query("sample query question?")  when the relevant chunk/node is to be fetched with similar embeddings as that of input prompt.
Please help me understand at what point does llama_index generate the embeddings.

Hi @logan-markewich , Thanks for the response.
In the llama_index documentation here  , it says that for List Index, the embeddings are generated during query() and not during index construction.
Actually my goal is to generated the embeddings during index construction, assuming it will reduce the inference time during query.
Please have a look and let me know your thoughts.

@suraj-gade ah I missed that you were using a list index. You likely be more interested in using GPTVectorStoreIndex


Adding content to Food Ordering website

It was already been completed before assigning to me. So, I made change which increases user experience.

Close #2806

Checklist:

[✅] I have mentioned the issue number in my Pull Request.
[✅] I have commented my code, particularly in hard-to-understand areas
[✅] I have created a helpful and easy to understand README.md

@MohitGupta121 kindly, review this please.


Comparison to -1

I saw a lot of index comparisons like (-1 < index). Is there any case Array#indexOf returns a value lower than -1 or NaN?

Never mind I read the arrow wrong.

Just for the record, indexOf always returns either the index the element was found at, or -1. Never less than -1, and never NaN.


ORM changes and games.py functions

Describe Your Changes

App is now on debug mode
Imported ORMs from database, not schemas
Added db.query(..) where necessary to make API work
Removed a problematic foreign key

Non-Obvious Technical Information
Integrated game is WIP
Checklist Before Requesting a Review

[x] The code runs successfully.

HERE IS SOME COMMAND LINE OUTPUT

Check latest changes @michplunkett

Doneee 💀, check latest changes in game logic tomorrow @michplunkett


Combine import- Contact address company address not import by excel

Combine import - Contact address company address not import by excel

https://docs.google.com/spreadsheets/d/1FYva6vw75VYtSaBWV2F9wj7lP4XHNkaX8llh6EMzBQ8/edit?usp=sharing
this is excel sheet


Error raised when I tried to deploy 3Bot on threebot_deploy pacakge

Description
I got this error:

Steps to reproduce
Go through the 3bot chatflow.
Seems to happen when the user has a pool that requires an extension to deploy the solution.

@m-motawea please link the pr the fixed this

verified as below steps
1. Install threebot deploy
2. Create a threebot deploy instance with some expiration date
3. Delete it.
4. Create another with expiration date after the first one.
5. Make sure in the chatflow it says that it found an old pool that needs extension.


Tooltip

Please review:
Screenshot here to show all are working.

Coverage remained the same at 100.0% when pulling 6315c8c060d0b3065cb0652b471285ec1e1b014d on tooltip into 68a54617e618d7b3664c70d7ece1fcf1c0e4c6fc on master.


1.17

this must be updated to 1.17 kspigot (remove shadow plugin, add library in plugin.yml...).
see KSpigot Docs for more

Oh yeah I see now, PaperMc is finally stable, I have the Project already 1.17 ready on my Pc I was wating for Paper to be stable. Will push in a few minutes

Update to 1.17


[FancyZones] support multiple zones snapping in app history

FZ now supports to snap a window to two or four adjacent zones, but it doesn't persist this information in the app history, therefore the Move newly created windows to their last know zone doesn't work in this case.
Prerequisite: move the app history to its own json file https://github.com/microsoft/PowerToys/issues/2579

Fixed in 0.19.0, please visit https://github.com/microsoft/PowerToys/releases/ for the latest release


login_required error in web browser console when calling signinRedirectCallback

Hi team. Thank for your library. I'm using your lib in my project.
I have issue with refeshToken. After user login, following error occurred twice in the web browser console:
ErrorResponse: login_required
    at ResponseValidator._processSigninState (http://localhost:4200/vendor.js:112250:13)
    at http://localhost:4200/vendor.js:112141:15
    at Generator.next (<anonymous>)
    at asyncGeneratorStep (http://localhost:4200/vendor.js:116388:24)
    at _next (http://localhost:4200/vendor.js:116410:9)
    at http://localhost:4200/vendor.js:116417:7
    at new ZoneAwarePromise (http://localhost:4200/polyfills.js:13207:21)
    at http://localhost:4200/vendor.js:116406:12
    at ResponseValidator.validateSigninResponse (http://localhost:4200/vendor.js:112154:7)
    at http://localhost:4200/vendor.js:112897:32

I can't fix it. Can you help me. Thanks.
Sorry about my english.

Looks like your IdP is not happy with the refresh token you pass back...

Looks like your IdP is not happy with the refresh token you pass back...

How can I fix it? It only keycloak config right?


Adapt testing to the new way of patching the driver

Change required for napalm-automation/napalm-base#110

Coverage increased (+0.2%) to 79.699% when pulling 8fb0faeeebb6abded85f2b92318c4d75d1336991 on dbarrosop/updated_test_framework into 3e488fb8946859e85dc22e832db5c0e7f135fb08 on develop.


Packaging bug for USI

File           : GameData/UmbraSpaceIndustries/Kolonization/CHANGELOG.txt
Installing Mod : USI-UKS-Shared <IP_ADDRESS>
Owning Mod     : USI-Kolonization
CKAN Version   : v1.16.0-0-g7206ce5 (beta)

As in previous issues: uninstall and re-install USI Kolonization

Thanks. I've tried uninstalling and reinstalling various bits, but it seems that uninstalling everything associated with USI does the trick.


feat: airtable oauth2 integration

Describe your changes
This PR adds Airtable OAuth2 integration.
Scopes docs: https://airtable.com/developers/web/api/scopes
Airtable example oauth2 workflow: https://github.com/Airtable/oauth-example/blob/main/index.js
Issue ticket number and link
FI-4
FI-51
Checklist before requesting a review

[x] I have performed a self-review of my code
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics?
[ ] Will this be part of a product update? If yes, please write one phrase about this update.

Hi Bowen, for now, the scope can be fixed, we will introduce var when we add Nodes
Ok, we will add a test branch later

@ybw0014 Hi Bowen, do you test this function to get accesstoken? if yes, I will merge this code

still have some issues, gonna fix it later

need some extra changes.

The redirect works as expected, but the callback doesn't seem to print any logs or have a HTTP response? Is that only me or the setup is wrong.

CacheModule.register() is fantastic!

The redirect works as expected, but the callback doesn't seem to print any logs or have a HTTP response? Is that only me or the setup is wrong.

Same error: app-1    | [Nest] 80  - 01/09/2024, 5:02:01 AM   ERROR [AuthCallbackController] Error exchanging code for token
Investigating

payload looks good:
{"client_id":"xxxxxxxx","client_secret":"xxxxx","redirect_uri":"https://localhost:3000/oauth/callback","grant_type":"authorization_code","code":"xxxxx","code_verifier":"xxxxxx"}
DId you check the official docs?

app-1    | {"error":"invalid_request","error_description":"The Content-Type of your request must be "application/x-www-form-urlencoded"."}

after add x-www: {"error":"invalid_client","error_description":"Invalid client credentials."}

The callback is fixed now.

Is that due to
  Authorization: `Basic ${encodedCredentials}`,
 "Content-Type": "application/x-www-form-urlencoded",

Fixed refresh token, tested and it works.

many thx!


ctr images import : no descriptor found for reference object

when i use the command "ctr images import xxx/calico-node:v1.1.3 XXX_calico-node.tar" ,i got the error "ctr: no descriptor found for reference object "v1.1.3""
is this a bug for containerd or my error ?

I guess this is because your index.json does not contain v1.1.3 reference.
You might need to specify --ref-object sha256:deadbeef.. in such a case.

Are you using cri-containerd? If yes, you may want to use cri-containerd load instead.

@Random-Liu
Thanks very much. you solved my question.

So, you were trying to import a Docker image rather than an OCI image?
It is not supported in ctr images import now.

If the archive content is conformant to https://github.com/opencontainers/image-spec , the archive is an OCI image.
Otherwise it is likely a Docker v1 image.


WETH deposit succeeds on low-level call but fails on method call

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (6b3db2a 2022-08-19T00:24:19.967244Z)
What command(s) is the bug in?
forge test
Operating System
macOS (Apple Silicon)
Describe the bug
Run the following code against a mainnet fork
pragma solidity ^0.8.16;

import { Test } from "forge-std/Test.sol";

interface IWETH {
    function deposit() external payable returns (uint);
    function transfer(address to, uint value) external returns (bool);
    function withdraw(uint) external returns (uint);
}

contract DebugMainnet is Test {
    address holder = 0xEA674fdDe714fd979de3EdF0F56AA9716B898ec8;
    IWETH weth = IWETH(0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2);

    function test() external {
        vm.startPrank(holder);
        assertGt(holder.balance, 100 ether);
        // This succeeds
        address(weth).call{value: 1 ether}(abi.encodeWithSelector(0xd0e30db0));
        // This fails
        weth.deposit{value: 1 ether}();
    }
}

You'll notice that the low-level call succeeds while the high-level call fails. This is bizarre because they're both calling the same functionality. Any clue what might be going on? Since WETH.deposit() is obviously a legal method, there must be something wrong in forge.
The full test log is below
Running 1 test for test/DebugMainnet.t.sol:DebugMainnet
[FAIL. Reason: EvmError: Revert] test() (gas: 35871)
Traces:
  [35871] DebugMainnet::test()
    ├─ [0] VM::startPrank(0xea674fdde714fd979de3edf0f56aa9716b898ec8)
    │   └─ ← ()
    ├─ [6874] WETH9::deposit{value:<PHONE_NUMBER>000000000}()
    │   ├─ emit Deposit(dst: 0xea674fdde714fd979de3edf0f56aa9716b898ec8, wad:<PHONE_NUMBER>000000000)
    │   └─ ← ()
    ├─ [2074] WETH9::deposit{value:<PHONE_NUMBER>000000000}()
    │   ├─ emit Deposit(dst: 0xea674fdde714fd979de3edf0f56aa9716b898ec8, wad:<PHONE_NUMBER>000000000)
    │   └─ ← ()
    └─ ← "EvmError: Revert"

Test result: FAILED. 0 passed; 1 failed; finished in 1.67s

Failing tests:
Encountered 1 failing test in test/DebugMainnet.t.sol:DebugMainnet
[FAIL. Reason: EvmError: Revert] test() (gas: 35871)

Able to reproduce this as well.

You have the WETH interface as
function deposit() external payable returns (uint);

but it should be
function deposit() external payable;

i.e. deposit doesn't return anything, so it reverts trying to decode the return data


Virtual console at startup

Per the confusion in https://github.com/tmpvar/jsdom/pull/991#issuecomment-76561496, we should probably add some kind of option that allows setting up a virtual console at startup time. Not sure exactly what it would look like.
Alternately we could delay firing the console events for a tick?

Instead of receiving a virtualConsole, could you build one yourself and pass it in as a config option?
var myConsole = new (require('events').EventEmitter)();

myConsole.on("log", function (message) {
  console.log(message);
});

jsdom.env({
  virtualConsole: myConsole
}};

On a different note, would it be useful to skip a layer of abstraction and let you just pipe output to a writable stream?
jsdom.env({
  sendOutputTo: process.stdout
}};

Instead of receiving a virtualConsole, could you build one yourself and pass it in as a config option?

I think that was what I was envisioning in the OP, actually.

On a different note, would it be useful to skip a layer of abstraction and let you pipe output to a writable stream?

Not really. The events are pretty important, especially given the upcoming #1039. And it's an easy enough adapter to write. I think it'd just be more code to support that kind of thing.

var myConsole = new (require('events').EventEmitter)();

Although this isn't great; we'd want jsdom.createVirtualConsole() probably.


Pummel podman

Type of change
Select the type of your PR

Enhancement / new feature

Description
Add an arbitrary cheap API call to try and ensure the container engine API is available.
Additional Context
When running on podman on linux repeated executions of TemplateTest fail in the Versions test (this doesn't happen when run in isolation?!?!). This is currently a tactical hack to ensure the API is available.
Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

I think this helped us un-cover the root cause but isn't in itself useful so I'm closing it.


More options for inserting based on siteId

I was just looking at the case for inserting an identifier at the same position as the previous identifier, but with a greater site id:
      } else if (diff === 1 && siteID > prevSiteID) {
        return [[prevInt, siteID]];

This makes sense, but is there not also a similar case:
      } else if (diff === 1 && siteID < nextSiteID) {
        return [[nextInt, siteID]];

And a final case (which would actually have to come first in the conditional):
      } else if (diff === 0 && siteId > prevSiteId && siteID < nextSiteID) {
        return [[nextInt, siteID]]; // Or equivalently use prevInt

Do these all actually make sense? If the existing case works, I can't see why the new ones wouldn't.
Practically I don't know whether these will actually come up that often - in the original logoot paper I don't think any of these options are deliberately used on an individual site, but they could still happen from concurrency so I can't see any harm in using them except performance. Actually the algorithm in the original logoot paper doesn't seem to work because of an off by one error and a more important logical problem with generating the line positions (as far as I can tell!).

I was also working through the effect of this, and it's interesting that the implementation needs the site id to be > 0, so as not to conflict with the min and max positions used. I think it would be possible to permit arbitrary site ids by just enforcing that all identifiers other than the min and max positions' identifiers have a pos at least 1. This is another way of ensuring that the min identifier is less than or equal to all identifiers, so we can always generate a position before any other position. This is only slightly less efficient than requiring site id to be positive, but a lot less likely to go wrong as a result of someone not realising site id must be positive?

It's been over a year since I've looked into this code. I'll have to see if I can find time to look into this at some point. There was a brief period where I was investigating various text CRDTs for use in Canvas (now shut down), and at that time came to the conclusion that Logoot was a no-go for us, but I learned a lot from this making implementation.
On your first comment: I'm not sure if it really makes a difference. I believe that both versions (with your conditionals and without) are both correct. One just results in longer identifiers in some cases.
On your second comment: This is probably a good idea, but I wasn't really looking towards productionization, yet, when I originally wrote this. There is much more that would have to happen, and I don't think Logoot is suited to collaborative document editing, in the first place, for most use cases. It was really intended for editing in environments where users don't edit the same line at the same time, which isn't practical in my experience. No character-wise CRDT is, in my opinion. I think a string-aware CRDT (if there even is a good one) or good, old-fashioned operational transformation end up working better, in practice.

Ah that's interesting, I think I'm just re-treading the same path you followed a year ago... I'd like to support collaborative text editing as well, and Logoot seemed like a relatively simple and established way to do this. Looking at the implementation now makes me think it's not so simple (all the edge cases!), and I'm not sure I've found anyone who is actually actively using Logoot. On the other hand I do have a use case for just rearranging/editing lists where I think Logoot would be much better - it has much shorter sequences with much less editing than a collaborative string, so maybe that is still practical.
I'm guessing that your conclusion that Logoot isn't suited is from the overhead, or also from problems with intent of editing? As far as a string-aware CRDT goes, I guess Logoot could maybe be used on a word level, at least for English.


Courts: Clarify code of conduct

Code of conduct should make the distinction between bribery attempts which can be provably associated with a specific party (eg the bribe is signed by the party and communicated through evidence) versus a bribe that is made indirectly and communicated out-of-band.
Code of conduct should also have a clause related to credible commitments among jurors.

Are referendums possible or is the code of conduct set in stone?

Possible, but there may be some bureaucratic checks and balances because...

The code of conduct is an agreement between a juror and court/network
Individiual agreements reference an arbitration provider (eg the court/network)

So in order to change the code of conduct, the network would need to propose a new code of conduct, and jurors would need to accept it. In practice this might look similar to a court forking and jurors that opt in to the new agreement transferring their reputation from the old court to the new court. Then the network would determine which courts make up the court hierarchy.
Parties to other agreements could mutually agree to use the new court or the old court, If they originally specified in the agreement that they want to use the "network" as their arbitration mechanism then by default the agreement would use the new court, but the default doesn't matter so much because in order for the court to do anything a dispute has to be raised by either counterparty, and they can simply mutually agree to switch arbitration mechanisms before raising a dispute.


The TypeScript language service died 5 times right after it got started.

Issue Type: Bug
Upon starting I receive the message:
The TypeScript language service died 5 times right after it got started. The service will not be restarted.
Additionally I am not able to Open a Folder, not when I start nor by clicking the "Open Folder" button and selecting a folder.
I am running on a MacBook Pro and the error started suddenly this morning. I have started both normally "code ." and "code . --disable-extensions"
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:17:14.222Z)
OS version: Darwin x64 19.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700HQ CPU @ 2.60GHz (8 x 2600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (3.15GB free)

Process Argv
.

Screen Reader
no

VM
0%

Extensions: none

When this happens, there should be a button in the popup that helps you report an issue that includes the error stack from the TypeScript service. Please try using that
/needsMoreInfo

Sorry, I didn's see that button. A reboot seems to have fixed the problem.


Support microsecond timestamps in strftime()

This commit adds %f specifier for microsecond timestamp. This is
similar to python's %f extension:
https://docs.python.org/3/library/datetime.html#strftime-and-strptime-format-codes .
This closes #1921.

Checklist

[x] Language changes are updated in docs/reference_guide.md
[x] User-visible and non-trivial changes updated in CHANGELOG.md
[x] The new behaviour is covered by tests

Will need to update #1711 with this PR


get error when Convert from CPU to GPU

Hi, guys:
I am using keras, but when I was trying to change to GPU from CPU which CPU works well, the error shows.. Actually I definitely run the cuda example with instruction correctly.
I followed the instruction to change the GPU conf:
openmp=False
device = gpu
floatX = float32
allow_input_downcast=True
[lib]
cnmem = 0.8
[blas]
ldflags= -lopenblas
[nvcc]
fastmath = True
from CPU:
[global]
openmp=True
device = cpu
floatX = float32
allow_input_downcast=True
[blas]
ldflags= -lopenblas
ye@ye-desktop:~$ gedit ~/.theanorc
ye@ye-desktop:~$ python
Python 2.7.12 (default, Jul  1 2016, 15:12:24)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.

import keras
Using Theano backend.
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/keras/init.py", line 2, in
from . import backend
File "/usr/local/lib/python2.7/dist-packages/keras/backend/init.py", line 61, in
from .theano_backend import *
File "/usr/local/lib/python2.7/dist-packages/keras/backend/theano_backend.py", line 1, in
import theano
File "/usr/local/lib/python2.7/dist-packages/theano/init.py", line 42, in
from theano.configdefaults import config
File "/usr/local/lib/python2.7/dist-packages/theano/configdefaults.py", line 16, in
from theano.configparser import (AddConfigVar, BoolParam, ConfigParam, EnumStr,
File "/usr/local/lib/python2.7/dist-packages/theano/configparser.py", line 84, in
theano_cfg.read(config_files)
File "/usr/lib/python2.7/ConfigParser.py", line 305, in read
self._read(fp, filename)
File "/usr/lib/python2.7/ConfigParser.py", line 512, in _read
raise MissingSectionHeaderError(fpname, lineno, line)
ConfigParser.MissingSectionHeaderError: File contains no section headers.
file: /home/ye/.theanorc, line: 1
'openmp=False \n'

According to the error, I pip installed ConfigParser, really appreciate your help.!

In the config file for GPU, you are missing that at the top. You have it in
the CPU config file.
[global]
Le 11 oct. 2016 01:06, "Ye Wang"<EMAIL_ADDRESS>a écrit :

Hi, guys:
I am using keras, but when I was trying to change to GPU from CPU which
CPU works well, the error shows.. Actually I definitely run the cuda
example with instruction correctly.
I followed the instruction to change the GPU conf:
openmp=False
device = gpu
floatX = float32
allow_input_downcast=True
[lib]
cnmem = 0.8
[blas]
ldflags= -lopenblas
[nvcc]
fastmath = True
from CPU:
[global]
openmp=True
device = cpu
floatX = float32
allow_input_downcast=True
[blas]
ldflags= -lopenblas
ye@ye-desktop:~$ gedit ~/.theanorc
ye@ye-desktop:~$ python
Python 2.7.12 (default, Jul 1 2016, 15:12:24)
[GCC 5.4.0 20160609] on linux2
Type "help", "copyright", "credits" or "license" for more information.
import keras
Using Theano backend.
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python2.7/dist-packages/keras/init.py", line 2, in
from . import backend
File "/usr/local/lib/python2.7/dist-packages/keras/backend/init.py",
line 61, in
from .theano_backend import *
File "/usr/local/lib/python2.7/dist-packages/keras/backend/theano_backend.py",
line 1, in
import theano
File "/usr/local/lib/python2.7/dist-packages/theano/init.py", line 42,
in
from theano.configdefaults import config
File "/usr/local/lib/python2.7/dist-packages/theano/configdefaults.py",
line 16, in
from theano.configparser import (AddConfigVar, BoolParam, ConfigParam,
EnumStr,
File "/usr/local/lib/python2.7/dist-packages/theano/configparser.py",
line 84, in
theano_cfg.read(config_files)
File "/usr/lib/python2.7/ConfigParser.py", line 305, in read
self._read(fp, filename)
File "/usr/lib/python2.7/ConfigParser.py", line 512, in _read
raise MissingSectionHeaderError(fpname, lineno, line)
ConfigParser.MissingSectionHeaderError: File contains no section headers.
file: /home/ye/.theanorc, line: 1
'openmp=False \n'
According to the error, I pip installed ConfigParser, really appreciate
your help.!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fchollet/keras/issues/4017, or mute the thread
https://github.com/notifications/unsubscribe-auth/AALC-3foW2JK5XJV3Np2Ywizpk_0xgpKks5qyxlqgaJpZM4KTP4D
.

It works. Thanks so much!


Custom Error Message Not Displaying

Hello,
I'm having an issue where my custom error message is not displaying on one of my properties.  Instead I'm getting a generic 'Property' must not be empty error.  Here is my rule:
RuleFor(x => x.Mortgage.MonthlyPayment).NotEmpty().When(x => x.Plan.DisabilityPlan != DisabilityPlan.Empty && x.Plan.DisabilityPlan != DisabilityPlan.None).WithMessage("Please enter a monthly payment");

Even if I take off the conditional .When(), the custom message isn't displaying correctly.  However, if I move this rule into a custom validator for the child object like so, RuleFor(x => x.Mortgage).NotNull().SetValidator(new MortgageValidator()); , it works but then I can't use the conditional since it requires a property from the parent object.
I've tried passing the property I need to check into the child validator like so RuleFor(x => x.Mortgage).NotNull().SetValidator(x => new MortgageValidator(x.Plan.DisabilityPlan)); and then use that in the .When() method but that doesn't work since that only takes a lambda.
Any help with this would be greatly appreciated.

Thanks for the quick reply Jeremy.  It is indeed an MVC web project.  I thought that may be the issue but since I have other fields that are value types that give the correct error message, I thought it may be something else.   Those other value type fields have the validation rules set in the child objects custom validator without a conditional instead of in the parent.
All that said, I went into the view and surrounded the field in question with @{ Html.EnableClientSideValidation(false); } and `@{ Html.EnableClientSideValidation(true); } and it seemed to fix my issue.  Seems a little hacky to me, however.

Yes it's one of the limitations of MVC's built-in validation as there's no way to override the behaviour. The usual workaround is to make the value type nullable.

Agreed.  Unfortunately, it isn't using a view model so that's easier said than done.  Anyway, I think you solved the issue so thanks for you help!


copy-modules-webpack-plugin throw an error with AOT build on Angular project

What are you trying to do?

I am trying to "ng build -c dev" an angular project with the plugin.
The build fail when it start the copy-modules-webpack-plugin with an error like this:
95% emitting WebpackCopyModulesPluginAn unhandled exception occurred: ENOENT: no such file or directory, stat '/path-to-my-project/src/app/+ui-kit/table/basic-table.component.scss.shim.ngstyle.js'

What feature or behavior is this required for?

Build with AOT angular project

How could we solve this issue? (Not knowing is okay!)

Don't know why or how yet... still looking. If anyone can help 😃

Anything else?
It works well when I remove the AOT option, but I need to build with AOT in order to optimise the rendering of my app.

Could you post your webpack configuration?  I'll admit I'm not specifically familiar with the angular CLI tools but if I can see your configuration that may help.

yes for sure.
here's my only webpack conf :
const CopyModulesPlugin = require('copy-modules-webpack-plugin');

module.exports = {
  plugins: [
    new CopyModulesPlugin({
        destination: 'webpack-modules'
    })
  ]
};

here's my angular.json :
{
  "$schema"<EMAIL_ADDRESS>  "version": 1,
  "newProjectRoot": "projects",
  "projects": {
    "or-angular": {
      "root": "",
      "sourceRoot": "src",
      "projectType": "application",
      "prefix": "app",
      "schematics": {
        "@ngrx/schematics:action": {
          "creators": true
        },
        "@ngrx/schematics:effect": {
          "creators": true
        },
        "@ngrx/schematics:reducer": {
          "creators": true
        }
      },
      "architect": {
        "test": {
          "builder": "@angular-builders/jest:run",
          "options": {
            "no-cache": true
          }
        },
        "build": {
          "builder": "@angular-builders/custom-webpack:browser",
          "options": {
            "es5BrowserSupport": true,
            "outputPath": "dist",
            "index": "src/index.html",
            "main": "src/main.ts",
            "polyfills": "src/polyfills.ts",
            "tsConfig": "src/tsconfig.app.json",
            "assets": [
              "src/favicon.ico",
              "src/assets",
              "src/assets/**/*.svg",
              "src/assets/mocks/**/*.html"
            ],
            "scripts": [],
            "styles": [
              "src/app/_fonts.scss",
              "src/styles.scss"
            ],
            "stylePreprocessorOptions": {
              "includePaths": [
                "src/app/core/style",
                "src/app/"
              ]
            },
            "customWebpackConfig": {
              "path": "./webpack.config.js"
            }
          },
          "configurations": {
            "production": {
              "deployUrl": "/etc/clientlibs/AppRoot/js/",
              "fileReplacements": [
                {
                  "replace": "src/environments/environment.ts",
                  "with": "src/environments/environment.prod.ts"
                }
              ],
              "optimization": true,
              "outputHashing": "all",
              "sourceMap": false,
              "extractCss": true,
              "namedChunks": false,
              "aot": true,
              "extractLicenses": true,
              "vendorChunk": false,
              "buildOptimizer": true
            }
          }
        },
        "serve": {
          "builder": "@angular-builders/custom-webpack:dev-server",
          "options": {
            "browserTarget": "or-angular:build",
            "proxyConfig": "proxy.conf.js",
            "customWebpackConfig": {
              "path": "./webpack.config.js"
            }
          },
          "configurations": {
            "production": {
              "browserTarget": "or-angular:build:production"
            }
          }
        }
      }
    }
  },
  "defaultProject": "or-angular"
}

the only webpack conf that i have is the one as suggested in the readMe.md of this repo


EffectSomeAsync complains that Func<T, Task> is not awaited

Task thing(value) => Task.FromResult(value);
Option.Some("value").AsAsync().EffectSomeAsync(value => thing(value)); // gets green underline saying the result isn't awaited.
Option.Some("value").AsAsync().EffectSomeAsync(async value => await thing(value)); // this is okay though.

It looks like there are some missing overloads for this. It should have an overload like the one below:
public static async Task<Unit> EffectSomeAsync<T>(this Task<Option<T>> optional, Func<T, Task> doWhenSome)

Completed on EffectImprovements branch.


Multiple Packages With Binaries Found

Having an odd issue which I can't figure out. If I install my Rust application with:
cargo install --git https://github.com/mmstick/parallel

I get this error:
error: multiple packages with binaries found: generate-files, libc-test, parallel

Yet if I install the application with this command:
cargo install parallel

It compiles perfectly.

Ah yeah if a git repository has multiple crates inside of it all with binaries Cargo has to know which one is actually being installed. The package on crates.io, however, is specific, so that's why that works instead.
Does this work for you?
cargo install --git https://github.com/mmstick/parallel parallel

That does work perfectly.

The RFC is still indicating to use -p https://github.com/rust-lang/rfcs/blob/master/text/1200-cargo-install.md
@alexcrichton

https://doc.rust-lang.org/cargo/commands/cargo-install.html says
--bin name...
Install only the specified binary.
--bins
Install all binaries.

neither of these worked for me, still giving the error about wanting to know which of multiple packages to install, but passing the binary name as an argument like suggested in this issue did work.

@spease this is a fairly old issue and it is unclear if its related to this issue.  If this appears to be a bug, it might be good to create a new issue and include full reproduction steps.  If this is a question on how to use Cargo, you might want to post on https://users.rust-lang.org/

Sorry, maybe it's not clear how I arrived at this ticket.
If I run
CARGO_NET_GIT_FETCH_WITH_CLI=true cargo install --git "ssh://user@host/home/user/repo"

I get Multiple Packages with Binaries Found.
If I run
CARGO_NET_GIT_FETCH_WITH_CLI=true cargo install --git "ssh://user@host/home/user/repo" --bin app

which is what the --help suggests is the right option for what I'm trying to do, I get the same error. That's when I started googling and found this ticket.
If I use the suggested solution in this ticket
CARGO_NET_GIT_FETCH_WITH_CLI=true cargo install --git "ssh://user@host/home/user/repo" app

then I get
failed to load manifest for workspace member...

Caused by:
  failed to load manifest for dependency `dependency`

I don't know if this is the best ticket, but it seems like this is broadly the same usability issue.

As you highlighted, something is going on with dependency but I can't really say much without a reproduction case.

Figured it out.  The issue was that an underlying dependency was outside the project tree as I had to modify a third-party library and the PR never got accepted or rejected, so I never updated the dependency to a permanent value. Opened #14924 to discuss making the error clearer.