Add radd in Circuit

This commit adds radd in cirq.Circuit() because Moment + Circuit doesn't work even if Circuit + Moment works.

LGTM! Thanks for the contribution. We'll need an approval from someone with write access before it can be merged, but it shouldn't take too long.

I really appreciate for helping me ramping up for this first PR! :D

Please add tests for these lines;
Line  277 is new or changed but not covered:                  return NotImplemented
Line  286 is new or changed but not covered:              raise ValueError("Can't add circuits with incompatible devices.")

Nit @dabacon : You don't need to request changes that the build system is already enforcing.

Nit @dabacon : You don't need to request changes that the build system is already enforcing.

Perhaps not, but it is a helpful reminder for a first-time contributor.

Yeah, the explanation is certainly a good idea.

Yes, I need to learn more about github system because I am a novice. Thank you all for your kind comments. I will work on the coverage!

I have a question. I realized that radd doesn't require device checking codes inside because (1) add already has it and (2) moment + circuit is converted into cirq.Circuit(moment) + self, which is able to be concatenated with any device. (3) there is no way to set the device of moment, except (4) cirq.Circuit(moment, device=cg.Foxtail) explicitly, for example, then (5) it runs add, not radd.
In conclusion, may I just remove the device comparison codes inside radd?

Use backticks ` to avoid the underscores becoming bold __like_this__ instead of like_this.
Yes, if the other instance is a Circuit then __add__ will be used instead. You can start __radd__ with a comment # The Circuit case is handled by __add__, and then just do an insert at index 0.

Thank you, but I am stuck with coverage. I am very confused. Even if I added test codes, coverage still says my codes are not covered. On top of it, it also says my test codes are not covered! Why are my test codes recognized as tests? LOL. I just ran "./check/pytest-and-incremental-coverage" and the messages are:
Running pytest                                                                                [23/337]
ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --benchmark-skip
inifile: None
rootdir: /usr/local/google/home/jaeyoo/git/Cirq
Finished pytest
Running incremental-coverage
************* cirq/circuits/circuit.py (4 uncovered)
Line  276 is new or changed but not covered:         if not isinstance(other, (ops.Operation, Iterable)):
Line  277 is new or changed but not covered:             return NotImplemented
Line  279 is new or changed but not covered:         result = Circuit(other)
Line  280 is new or changed but not covered:         return result.iadd(self)
************* cirq/circuits/circuit_test.py (25 uncovered)
Line  252 is new or changed but not covered:     a = cirq.NamedQubit('a')
Line  253 is new or changed but not covered:     b = cirq.NamedQubit('b')
Line  255 is new or changed but not covered:     c = cirq.Circuit()
Line  256 is new or changed but not covered:     assert [cirq.X(a), cirq.Y(b)] + c == cirq.Circuit([
Line  257 is new or changed but not covered:         cirq.Moment([cirq.X(a), cirq.Y(b)]),
Line  258 is new or changed but not covered:     ])                                                                                [4/337]
Line  260 is new or changed but not covered:     assert cirq.X(a) + c == cirq.Circuit(cirq.X(a))
Line  261 is new or changed but not covered:     assert [cirq.X(a)] + c == cirq.Circuit(cirq.X(a))
Line  262 is new or changed but not covered:     assert [[[cirq.X(a)], []]] + c == cirq.Circuit(cirq.X(a))
Line  263 is new or changed but not covered:     assert (cirq.X(a),) + c == cirq.Circuit(cirq.X(a))
Line  264 is new or changed but not covered:     assert (cirq.X(a) for _ in range(1)) + c == cirq.Circuit(cirq.X(a))
Line  265 is new or changed but not covered:     with pytest.raises(AttributeError):
Line  266 is new or changed but not covered:         _ = cirq.X + c
Line  267 is new or changed but not covered:     with pytest.raises(TypeError):
Line  268 is new or changed but not covered:         _ = 0 + c
Line  271 is new or changed but not covered:     d = cirq.Circuit()
Line  272 is new or changed but not covered:     d.append(cirq.Y(b))
Line  273 is new or changed but not covered:     assert [cirq.X(a)] + d == cirq.Circuit([
Line  274 is new or changed but not covered:         cirq.Moment([cirq.X(a)]),
Line  275 is new or changed but not covered:         cirq.Moment([cirq.Y(b)])
Line  276 is new or changed but not covered:     ])
Line  277 is new or changed but not covered:     assert cirq.Moment([cirq.X(a)]) + d == cirq.Circuit([
Line  278 is new or changed but not covered:         cirq.Moment([cirq.X(a)]),
Line  279 is new or changed but not covered:         cirq.Moment([cirq.Y(b)])
Line  280 is new or changed but not covered:     ])

Thank you, but I am stuck with coverage. I am very confused. Even if I added test codes, coverage still says my codes are not covered. On top of it, it also says my test codes are not covered! Why are my test codes recognized as tests? LOL. I just ran "./check/pytest-and-incremental-coverage" and the messages are:
Running pytest [23/337]
ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --benchmark-skip
inifile: None
rootdir: /usr/local/google/home/jaeyoo/git/Cirq
Finished pytest
Running incremental-coverage
************* cirq/circuits/circuit.py (4 uncovered)
Line 276 is new or changed but not covered: if not isinstance(other, (ops.Operation, Iterable)):
Line 277 is new or changed but not covered: return NotImplemented
Line 279 is new or changed but not covered: result = Circuit(other)
Line 280 is new or changed but not covered: return result.iadd(self)
************* cirq/circuits/circuit_test.py (25 uncovered)
Line 252 is new or changed but not covered: a = cirq.NamedQubit('a')
Line 253 is new or changed but not covered: b = cirq.NamedQubit('b')
Line 255 is new or changed but not covered: c = cirq.Circuit()
Line 256 is new or changed but not covered: assert [cirq.X(a), cirq.Y(b)] + c == cirq.Circuit([
Line 257 is new or changed but not covered: cirq.Moment([cirq.X(a), cirq.Y(b)]),
Line 258 is new or changed but not covered: ]) [4/337]
Line 260 is new or changed but not covered: assert cirq.X(a) + c == cirq.Circuit(cirq.X(a))
Line 261 is new or changed but not covered: assert [cirq.X(a)] + c == cirq.Circuit(cirq.X(a))
Line 262 is new or changed but not covered: assert [[[cirq.X(a)], []]] + c == cirq.Circuit(cirq.X(a))
Line 263 is new or changed but not covered: assert (cirq.X(a),) + c == cirq.Circuit(cirq.X(a))
Line 264 is new or changed but not covered: assert (cirq.X(a) for _ in range(1)) + c == cirq.Circuit(cirq.X(a))
Line 265 is new or changed but not covered: with pytest.raises(AttributeError):
Line 266 is new or changed but not covered: _ = cirq.X + c
Line 267 is new or changed but not covered: with pytest.raises(TypeError):
Line 268 is new or changed but not covered: _ = 0 + c
Line 271 is new or changed but not covered: d = cirq.Circuit()
Line 272 is new or changed but not covered: d.append(cirq.Y(b))
Line 273 is new or changed but not covered: assert [cirq.X(a)] + d == cirq.Circuit([
Line 274 is new or changed but not covered: cirq.Moment([cirq.X(a)]),
Line 275 is new or changed but not covered: cirq.Moment([cirq.Y(b)])
Line 276 is new or changed but not covered: ])
Line 277 is new or changed but not covered: assert cirq.Moment([cirq.X(a)]) + d == cirq.Circuit([
Line 278 is new or changed but not covered: cirq.Moment([cirq.X(a)]),
Line 279 is new or changed but not covered: cirq.Moment([cirq.Y(b)])
Line 280 is new or changed but not covered: ])

Oh, I realized that I didn't install pytest-benchmark in my venv. As well, I missed registering upstream yet. Both make coverage happy & green now :D Thank you for your helps! Now I got it!

Missing coverage for two lines (See my other comment)

I've done. Could you give me LGTM?

Love the simpler implementation.

Automerge cancelled: Merging is blocked (I don't understand why).


Issue about knockout groups.

Hi guys!
I never saw this warning before, so I'm opening this issue:
[13:58:53.651] "Warning: TODO: Support knockout groups."
[13:58:53.651] "Warning: TODO: Support non-isolated groups."
Here is a link to the pdf with the error: http://dc230.4shared.com/download/XnQb9rEQ/LM-ED04.pdf

PDFX-ready_Output-Test_301d_X4.pdf
Adding a really good PDF-X4 support document to show in Adobe and PDF.JS (see Page 2 - #10 - for Knockout support).  I'll open another issue for overprint support.

https://support.mozilla.org/en-US/questions/1168544#answer-988671
I've got another instance of this affecting rendering of a PDF that works in all other browsers and Acrobat Reader.

Still reproducible with current Nightly flatpak.

I upgraded the pdfjs version to 2.5.207, the same problem will also occur

It has been sometime since I opened the bug :)
I found a new pdf file which also shows the warning using the current flatpak version of Firefox: 91.0.1
http://www.cs.cornell.edu/~ragarwal/pubs/blk-switch.pdf

I get this warning too. Don't have a clue about this.

me too.

22060_A1_01_Plans-1.pdf
I also have the problem with this pdf. This issue seems to be open for almost 10 years now. Could we imagine that you'll be able to fix that at some point.
Thanks for your work on that project anyway.


can't login and sync

I want to login and sync to vscode. But I get this error. what do I do? I get vscode from pacman.

and my PC info:

and vscode version:
Version: 1.73.1
Commit:<PHONE_NUMBER>f055b99068d3688932416f2346dd3b
Date: 2022-11-14T09:01:30.462Z
Electron: 19.1.4
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.0.9-arch1-1
Sandboxed: No

It looks like this is Code-OSS and not a VS Code build, and in that case Microsoft's services will not work without an official release. @rzhao271 do you happen to know if the pacman distribution is officially maintained by our team? It seems we do not have an official build for Arch Linux based on https://wiki.archlinux.org/title/Visual_Studio_Code but I want to make sure. Thanks!

I don't think the Arch package is maintained by us. Instead, users on distros that don't support Debian or RPM packages need to get the tar.gz file from the Downloads page.

Thanks @rzhao271, in that case @amirhossein-fzl I would recommend to use the tar.gz from our Downloads page in order to use our sync features.


disallowKeywords has no affect on 'null'

The setting "disallowKeywords": ["null"] has no affect.

That's because null is not a keyword it is a literal, you would need a special rule for that.
If you would like to implement it, let us know

I would like to implement it. Douglas Crackford makes a good case for using undefined in place of null. https://youtu.be/rhV6hlL_wMc?t=22m6s. I'll see if I can get a PR running sometime this week.


Collaborate with timvideos/litex-buildenv and litex-hub/linux-on-litex-vexriscv?

Hi!
We are working on getting the panologic-g2 into the timvideos/litex-buildenv repository and litex-hub/linux-on-litex-vexriscv repository (plus the general Migen/LiteX ecosystem).
This ecosystem supports Zephyr, MicroPython and Linux (on three/four architectures OpenRISC1000, RISC-V 32bit and soon PowerPC + SH4).
We have a tutorial for Linux on the Arty board and are working on getting the LiteX patches upstream.
#litex and #timvideos on irc.freenode.net is where we mostly hangout.
It would be awesome to collaborate more here!
Tim '@mithro' Ansell

I'd love that too, but last time I tried to get LiteX working, I got stuck in an endless loop of git submodules not being up to date and an almost complete lack of up-to-date "Getting Started" documentation.. except for some YouTube videos. But nobody has time to watch a video.
Have things changed?

We have had comprehensive set of tutorials in the wiki for at least a couple of years now. See the page at https://github.com/timvideos/litex-buildenv/wiki/HowTo-LCA2018-FPGA-Miniconf-VexRiscv-Renode or https://github.com/timvideos/litex-buildenv/wiki/HowTo-FuPy-on-iCE40-Boards example.
If you know what you are doing there is also the repo at https://github.com/litex-hub/linux-on-litex-vexriscv/blob/master/README.md#prerequisites
Please also do ask for help on IRC if you get stuck.


debug with chrome dev tool  issue

debug @latest
i hit with follow with node

DEBUG=picnicapp:* node --inspect-brk  ./bin/www

const debug = require('debug');
	log = require('debug')('picnicapp:stdout')
	debug.log = console.log.bind(console);
log("test custom console");

console  look like

picnicapp:stdout test custom console +0ms

anyone can help me ?

This seems to be an encoding error not related to debug module itself. Try the same by just using console.log().

although It's working linux console but not in chrome console. as you mention in readme

Web Browser
Colors are also enabled on "Web Inspectors" that understand the %c formatting option. These are WebKit web inspectors, Firefox (since version 31) and the Firebug plugin for Firefox (any version).

and my chrome support

console.log("%c this is blue text", "color:blue")

then this should work. i think chrome dev tool relativly new. improve "debug" to work with this powerful tool
thanks

Ya I see the problem when debugging a Node app with --inspect-brk. The problem is that I can't seem to figure out a way to change the arguments passed to console.log() depending on if it's the terminal or the web inspector.
If you can come up with any clever ways to handle that then I'm all ears.

Wrote a hack for you. Should work in any version of Node with adequate API support, failing back silently to the current functionality.
function formatInspector(...args) {
	// TODO format the message for the web inspector
	return `INSPECTOR: ${JSON.stringify(args)}`;
}

function formatNodeConsole(...args) {
	// TODO format the message for the Node.js console
	return `CONSOLE: ${require('util').inspect(args, {colors: true})}`;
}

function hookConsole(inspectFormatter, nodeFormatter) {
	const defaultConsole = console;
	const defaultStdout = console._stdout;
	const defaultStderr = console._stderr;

	try {
		const configBinding = process.binding('config');
		if (!configBinding.debugOptions.inspectorEnabled) {
			return console.log;
		}

		const inspectorConsole = global.console;

		const vanillaConsole = new (require('console')).Console(console._stdout, console._stderr);

		if (!(delete global.console)) {
			return console.log;
		}

		global.console = vanillaConsole;

		const nullStream = new (require('stream').Writable)();
		nullStream._write = () => {};

		inspectorConsole._stdout = nullStream;
		inspectorConsole._stderr = nullStream;

		const nodeOutput = (...args) => console.log(nodeFormatter(...args));
		const inspectorOutput = (...args) => inspectorConsole.log(inspectFormatter(...args));

		const inspectorBinding = process.binding('inspector');
		return inspectorBinding.consoleCall.bind(inspectorConsole, nodeOutput, inspectorOutput, {});
	} catch (err) {
		try {
			delete global.console;
		} catch (err) {}
		global.console = defaultConsole;
		global.console._stdout = defaultStdout;
		global.console._stderr = defaultStderr;
		return console.log;
	}
}

const debugWriter = hookConsole(formatInspector, formatNodeConsole); // Default is node console in this case

console.log('normal console log');
debugWriter('debug log message');

This is very cool @Qix- this will make me want to try out --inspect a lot more now! And ya when we add this we should definitely make sure normal console functions still proxy to the Web Inspector as well.

@TooTallNate added comments explaining how it works.

@TooTallNate did we ever merge my hack?

I'll investigate adding this in the 5.x release.

 You have got to be kidding me. They really did it.
Okay well case closed then...

@Qix- Hi, I have the log problem with chrome dev tool when I use this debug module.
Firstly, I like this Node JS Debug module and appreciate the work of those contributors!
Initially, I could not get the log in chrome dev tool when I use this debug module. Then I use the bind, it makes the log appear in chrome dev tool with the display problem like above.
I use the bind like below.
const debug = require('debug')('test:auth');
debug.log = console.log.bind(console);

I use --inspect to start my app.
nodemon --inspect  ./app.js

It shows like below, in VS Code terminal.

It shows like below, in Chrome DevTools.

How to solve the displaying color of the label and the time, in Chrome Devtools?
My version of the debug is "debug": "^4.1.1".
Another question is about the message of the where is the log from. For example, in the above screenshot from Chrome DevTools. auth.js:34 or common.js:116.
Is that possible to pass the message and display, when I use debug module? That will make the module better.
Thanks in advance!

That's expected. There's a nasty hack to fix that, but you're looking at output from a terminal app in the developer tools. There's nothing simple we can do.

2.6.8

4.3.4


Fix typo in config reference: window-decorations -> window-decoration

This is auto-generated so we should fix this upstream. I'll try to get to it.

@mitchellh this seems to be a common mistake; I'll work to make this a little bit more clear that this is a generated file: #183


Update examples and tutorials for transfer

The dispatchable transfer from the balances pallet has been removed in https://github.com/paritytech/polkadot-sdk/pull/1226. Any occurrences of it should be replaced by transfer_allow_death.

ping @Rashmirreddy


Don't check connector panes for VIs provided by VeriStand

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Replace the XPath-based VI discovery used in the connector pane test with the .NET parsing logic provided by the VeriStand API. This allows us to filter out system VIs much more easily.
Why should this Pull Request be merged?
The connector pane test is currently failing for Routing and Faulting due to attempting to find system VIs.
What testing has been done?
Ran unit tests, ran connector pane tests for Scan Engine EtherCAT and Routing and Faulting.

AssertConnectorPanesMatchExpected.vi is still used and in the lvlib, it is just rendering oddly in the diff. I have removed the other file. Good catch!

[ ] Move switch statement out of the loop.

[ ] Use a for loop to build the array of page VIs.


[Android Auto] Event.RemoteBrowse behaves weired

Screencast_TrackPlayer_AndroidAuto_RemoteBrowse.webm
Describe the Bug
The RemoteBrowse event doesn't fire for the first two elements (tabs in the example project, but also applies to list items) after an audio has played.
Steps To Reproduce

Open the example Project on Android Auto.
Start an audio
Switch between tabs. The first two don't trigger Event.RemoteBrowse

Replicable on Example App?
Yes

thats confusing... I dont have the dev setup on hand, will look over the weekend.
does android studio logcat emit
Timber.tag("GVA-RNTP").d("RNTP received loadChildren req: %s", parentMediaId) for the first two tabs?
I want to say podverse was fine, this is the demo I did and the podcast tab content update is done via RemoteBrowse
https://www.youtube.com/watch?v=QB-6yYE2LM8
where it was still updating after the podcasts were played

Sadly, loadChildren is not executed in said cases, so "RNTP received loadChildren req: %s", parentMediaId is not logged either.
It is interesting that it seems to work in podverse.
You're using slightly different versions of KotlinAudio and Trackplayer in podverse, right? Maybe one of the differences is causing this side effect.
I noticed the problem when I refactored our app. I wanted, similar to podverse, that initially only the basic navigation (the tabs) for android auto would be set in trackplayer and then the data would be updated on each RemoteBrowse event. Unfortunately, this does not work properly when the events are missing.

i wasnt able to reproduce this with the example app:(
I did synced up to RNTP's main again, as well as KA. maybe this did the trick?
podverse's KA is a bit customized mostly because podverse is staying at RNTP v4 rc04, though KA did have a few quite big metadata handling changes since...
pardon the poor quality
rntp.webm

Hey @lovegaoshi! To further isolate the problem I tested the current develop branch of podverse on my system. Here the problem also occurs. It shows the same wrong behavior as in my project and the example project. Since we are using the same Android Auto emulator, the problem could be due to the mobile device or the android version used. I am using a Pixel 5 and a Pixel 7a, both with the latest Android 14 for testing. Which Android version is running on your test device/emulator?

it was samsung S21 + android 13, here's pixel 7 + android 14 (UI1A.231005.007), still seems normal to me:( am I not reproducing it right?
pixel7

it was samsung S21 + android 13, here's pixel 7 + android 14 (UI1A.231005.007), still seems normal to me:( am I not reproducing it right?

In your screencasts everything works normally. This is really confusing.

from the podverse feedbacks so far i dont think this has been reported yet:( i will ask podverse to keep an eye out too

I did another test today just to make sure it wasn't my development setup. I deployed the app to my phone via the Play store and tried it with a real Android Auto device. Unfortunately, the issue occurs in this case as well :man_shrugging:.

just tried, tab location is not it:S still all seems normal

In my testing today, I found the following: When an audio is loaded from the rntp app, I get the "wrong" event behavior. When an audio is loaded from another app (e.g. Google podcasts), the events behave correctly with onClick. I made a screencast of this for clarification:
Video

oh ok! i did reproduce with https://github.com/schoetty/react-native-track-player/tree/dev-android-auto. still no idea why yet, but could you update/test RNTP dependency to https://github.com/lovegaoshi/react-native-track-player/tree/dev-android-auto ? that recently synced fork works fine on my end.

The tests today were at commit 34e5d259be518a55bf9909aaeb582cd1237ac61a. I have retested the current state (31170ae) but come to the same conclusion. I even cleared the gradle cache. But without success.

might be my gradle cache then, i'll load up on another computer later today

@schoetty I have bad and even worse news :(
The bad news is I did reproduce this issue, but couldnt get it "working" again on my samsung phone. my pixel is still fine.
And yes the issue is exactly as you described: when mediaSession is currently playing, the first 2 browsable items in root are automatically fired and the results are "cached," preventing further onloadchildren calls. when mediaSession is not playing anymore this is "released" and onLoadChildren is called again.
The even worse news is this is NOT specific to RN; I forked google's media player example repo UAMP and it's behaving the same. bad on samsung, good on pixel.
Now i'm believed this is some very obscure phone setting i'm not aware of...
ss
p

after sacrificing my car data I found out my samsung "works" again now by uninstalling and reinstalling android auto the app:/ win some lose some.

So, today I installed the latest beta version of the Android Auto app (10.7.134154) on my Pixel 5 and that has solved the problem with the events.


x/tools/internal/lsp: add tests for hover

Our test suite does not yet have any tests for the Hover feature.

Fixed with https://golang.org/cl/174380.


Avro support?

awswrangler.redshift.copy says "Load Pandas DataFrame as a Table on Amazon Redshift using parquet files on S3 as stage." but is it really limited to parquet or can it import any pandas df to redshift? ie my pandas df might be composed of Avro or json data

It can import any Pandas DataFrame. Parquet is only used as an intermediate step: DataFrame -> Parquet -> Redshift, meaning DataFrame will be converted to Parquet first, and then loaded into Redshift.


De-Helvetization of the mortgage API

The API resources of the Hypothekar API were already confirmed as an international ISO standard in July of this year, see 3rd entry at the table at the bottom of the following page:
https://www.iso20022.org/development/status-iso-20022-submissions?block_id=iso20022_submission_catalogue_all_submissions_block&prefix=BJAP
In order to propose the API spec as a whole as a standard, it still needs to be cleared of CH-specific artifacts.

Some months ago, SFTI's API resources of the Mortgage API were submitted to ISO 20022:
https://www.iso20022.org/development/status-iso-20022-submissions?block_id=iso20022_submission_catalogue_all_submissions_block&prefix=BJAP
In the meantime, these ressources are approved. Now, the API itself should be de-helvetized to be prepared for the mext steps. But as developments in ISO20022 are progressing rather slowly, the handling of de-helvetization is becoming less important. That is why this issue received the label "backlog".


Feature/templates api

Description

Related issues

Type of change

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

Changes Made

Testing

[x] Unit Tests
[x] Integration Tests

Mentions

I'm not sure why but so far I've got error 404 when doing POST to /templates/{schema} with draft JSON content.


Class members are accesible only from the class where they are defined.

Based on the https://github.com/vim/vim/discussions/13004 discussion, the following changes are made:

Static variables and methods are accessible only using the class name and inside the class where they are defined.
Static variables and methods can be used without the class name in the class where they are defined.
Static variables of a super class are not copied to the sub class.
A sub class can declare a class variable with the same name as the super class.
When a method or member is found during compilation, use more specific error messages.

This aligns the Vim9 class variable/method implementation with the Dart implementation.
Static variables in interfaces are not changed in this PR.

Just ran a few quick tests.
C extends B extends A. In class C, using A.svar, can read/write private members of A, tried that because I remembered you could do that before. Some might consider that a bug. Not sure what I think.
Looking good. I'll dig in more tomorrow.
Messages, I'm fine with the way they are, but some things that occurred to me.

In C, svar get message  Class member "svar" accessible only inside class "A",
wondering about working use A.svar and/or from class "A" in there somewhere.

With var oa = A.new() and doing oa.sval get ... accessible only using class "A"
wondering about ... only using A.svar"

With var oc = C.new() and oc.sval get the old favorite Member not found on object
wondering, since sval is in C's kind of parentage, if a message like in 1. is useful.

Just ran a few quick tests.

Thanks for testing the PR.

C extends B extends A. In class C, using A.svar, can read/write private members of A, tried that because
I remembered you could do that before. Some might consider that a bug. Not sure what I think.

I have updated the PR with a fix for this.

Looking good. I'll dig in more tomorrow.

I played around with vim including this PR. While trying stuff, ran into https://github.com/vim/vim/discussions/13041#discussioncomment-7004644; doesn't have anything to do with this PR.
Unless there's something particular you'd like explored, I'll wait for the merge.

On Thu, Sep 14, 2023 at 10:32 AM errael @.***> wrote:

I played around with vim including this PR. While trying stuff, ran into #13041
(comment)
https://github.com/vim/vim/discussions/13041#discussioncomment-7004644;
doesn't have anything to do with this PR.
Unless there's something particular you'd like explored, I'll wait for the
merge.

Thanks.  Did you see any issues with using class variables and methods in
script context, def function context, derived class, another class, using
an instance, instance method, class method and different access controls?
I know this is a long list. But there are too many scenarios where a class
variable and method can be used.
Thanks,
Yegappan

Duplicate method name for both object/class method not flagged as an error.
This output should be a dup name error, but it runs OK. I started putting the test code here, then checked, it's not caused by this PR.
https://github.com/vim/vim/discussions/13041#discussioncomment-7008328

Duplicate method name for both object/class method not flagged as an error.
This output should be a dup name error, but it runs OK. I started putting the test code here, then checked, it's not caused by this PR.

I have updated the PR with a fix for this issue also.

Did you take a peek at https://github.com/vim/vim/discussions/13041#discussioncomment-7004644? _F() in one class and  F()  in a child or base class gets no error.

Did you take a peek at #13041 (comment)? _F() in one class and F() in a child or base class gets no error.

This is now addressed in the updated PR.

Haven't found any more bugs with this PR (original comment went to wrong spot)

Thanks for testing the PR.

Coverity complains about this part here:
*** CID 1544700:  Control flow issues  (DEADCODE)
/src/vim9class.c: 1114 in add_classfuncs_objmethods()
1108                if (loop == 1 && STRNCMP(
1109                                    extends_cl->class_class_functions[i]->uf_name,
1110                                    "new", 3) == 0)
1111                    ++skipped;
1112                else
1113                {
>>>     CID 1544700:  Control flow issues  (DEADCODE)
>>>     Execution cannot reach the expression "extends_cl->class_class_functions" inside this statement: "pf = (loop == 1) ? extends_...".
1114                    ufunc_T *pf = (loop == 1
1115                                            ? extends_cl->class_class_functions
1116                                            : extends_cl->class_obj_methods)[i];
1117                    (*fup)[gap->ga_len + i - skipped] = copy_function(pf);
1118
1119                    // If the child class overrides a function from the parent

Hi Christian,
On Sat, Sep 16, 2023 at 4:20 AM Christian Brabandt <
@.***> wrote:

Coverity complains about this part here:

I have opened the PR https://github.com/vim/vim/pull/13103 to fix this.
Regards,
Yegappan

*** CID 1544700:  Control flow issues  (DEADCODE)
/src/vim9class.c: 1114 in add_classfuncs_objmethods()
1108                if (loop == 1 && STRNCMP(
1109                                    extends_cl->class_class_functions[i]->uf_name,
1110                                    "new", 3) == 0)
1111                    ++skipped;
1112                else
1113                {

CID 1544700:  Control flow issues  (DEADCODE)
Execution cannot reach the expression "extends_cl->class_class_functions" inside this statement: "pf = (loop == 1) ? extends_...".

1114                    ufunc_T *pf = (loop == 1
1115                                            ? extends_cl->class_class_functions
1116                                            : extends_cl->class_obj_methods)[i];
1117                    (*fup)[gap->ga_len + i - skipped] = copy_function(pf);
1118
1119                    // If the child class overrides a function from the parent


Don't replace data property directly

For issue https://github.com/pa7/heatmap.js/issues/219

Any timeline on when this could be merged and released?


Add option to set local time zone

Will be implemented together with a user management solution: https://github.com/RobinMeis/TTNmon-Backend/issues/2

Timezone can now be selected


feat: Add support for default refresh commands in DataLoaderView.

GitHub Issue: #

Proposed Changes

[ ] Bug fix
[x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build or CI related changes
[ ] Documentation content changes
[ ] Other, please describe:

Description

It's now possible to define a command provider to auto-populate DataLoaderView.RefreshCommand with a default value.

Impact on version

[ ] Major (Public API was modified.)

Public constructs (class, struct, delegate, enum, etc.) were removed or renamed.
Public members were removed or renamed.
Public method signatures were changed or renamed.

[x] Minor (Public API was extended.)

Public constructs, members, or overloads were added.

[ ] Patch (Public API was unchanged.)

A bug in behavior was fixed.
Documentation was changed.

[ ] None (The library is unchanged.)

Only code under the build folder was changed.
Only code under the .github folder was changed.

Checklist
Please check that your PR fulfills the following requirements:

[x] Documentation has been added/updated.
[ ] Automated Unit / Integration tests for the changes have been added/updated.
[ ] Updated BREAKING_CHANGES.md (if you introduced a breaking change).
[x] Your conventional commits are aligned with the Impact on version section.

Other information

is it big enough to be worthy of a note in the readme, probably here :https://github.com/nventive/Chinook.DataLoader/blob/main/README.md#define-default-behaviors


Sales Tracker Bar chart not visible to sales rep

 Description
When logged into the sales rep account, I was not able to see the Sales Tracker bar chart appear. and there is an error in the console.
 Steps to Reproduce

Log into a sales Rep account
The Sales Tracker does not render
Click on Month or Year, an error will appear in the console.

 Screenshots

This bug has been fixed.


[Feature Request] Additional options for color customization in the Armory

Is your feature request related to a problem? Please describe.
I apologize in advance if I missed it, but right now we have two possibilities:

To change the color of character item level ONLY if the "Show full item levels" option is enabled.
To change the background color behind character item level ONLY along with the color of the attribute titles.

Describe the solution you'd like
I would like to separately adjust the color of the character item level font and background.

That is only, because you can color the "Avarage" Color if you display both. if you only display this the string is controlled by blizzard.
My advice is, to just use "OUTLINE" as font style than it is readable:


Directory Manipulations

As far as I can see, There is no implementations of directory manipulations. I can see that given API is more about pipelines and one liners, but I can assure that it will be quite handy time to time and can be useful for anyone who will use script.
I would like to suggest PR that will include two new and a must Unix equivalent functions:

cd    - change directory
pwd - print working directory

Thanks for the issue @CSenshi! A couple of very interesting ideas there.
Could you give a short example program showing you would like to use these functions in script, supposing they were provided?

cd:
script.ChangeDir("/home/guest") - it would change your working directory to "/home/guest"

pwd:
script.pwd().stdout() - would print out your current working directory (e.g. after executing above command it would output "/home/guest")

P.S. I haven't yet thought about name of functions, so ChangeDir() and pwd() would do for now

But we already have os.Chdir() and os.Getwd(), don't we? What value would this add to that?

My bad, I read README.md file and thought you only had the commands that are written in "Quick start: Unix equivalents" paragraph.
I would like to contribute to this project, It's awesome! Where can I find the whole list of commands and see what's missing?

oh okay, I know that they are in os library, I thought you might want it in your script library as well. But as for now I Can see there is no real need for chdir, but pwd would be nice in script library because of pipes. There can be case where someone wants to do some operation on pwd-s output afterwards.
P.S. I would love to contribute to this project, because I think it has potential. I would like to hear If you have any other suggestions.

No problem, you're most welcome to contribute, and keep making suggestions! I'll close this for now, but keep the ideas coming.


Support of Terraform Module registry protocol

What problem are you facing?
After the investigation described in https://github.com/upbound/provider-terraform/issues/69#issuecomment-1411233709 we found a workaround, but it is limited, especially for the case of private terraform registries that usually require authentication:
The direct module source URL will expose the user/password in the YAML spec.
How could Official Terraform Provider help solve your problem?

Add support of TF Module registry protocol including authentication from Secret and the download URL discovery

Still relevant, full-fledged registry support is still desired by the customer:
https://www.notion.so/upbound/Terraform-Private-Registry-Support-f867920d848a43f69555d89b8860a1f4


dynamic config load and support multi config source ,eg mysql

Proposal
Use case. Why is this important?
“Nice to have” is not a good use case. :)
Hope to support dynamic configuration loading and multi-configuration data source support, for example, configuration changes can be realized by adding and deleting configurations through mysql

This is so far out of scope for Prometheus. Prometheus, as a monitoring solution, is started with files. However, you can define targets in many ways with service discovery, including generic SD like HTTP_SD.
I will move this to a github discussion, if other people want to comment.


Cannot see documentation for multiple targets

Package: https://swiftpackageindex.com/ChimeHQ/ChimeKit
My package has two targets with docc documentation. I'm using this .spi.yml:
version: 1
builder:
  configs:
    - documentation_targets: [ChimeLSPAdapter, ChimeExtensionInterface]

When I link the documentation link for the package, I only see docs for ChimeLSPAdapter. I see no listing for ChimeExtensionInterface. However, I can manually navigate to https://swiftpackageindex.com/ChimeHQ/ChimeKit/main/documentation/chimeextensioninterface.
Does the documentation hosting support viewing multiple targets like this? Is it the first one listed in .spi.yml that shows up at https://swiftpackageindex.com/ChimeHQ/ChimeKit/main/documentation ?

It's a bit hidden :)

Sorry about that Matt! This is a problem we're aware of.
The last two elements in the breadcrumb are both drop downs, but the UI isn't obvious enough. It's being discussed here: https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/discussions/2022

We're discussing improvements for this here: https://github.com/SwiftPackageIndex/SwiftPackageIndex-Server/discussions/2022

Ah! Haha ok well good to see you are both on it.
And just to confirm, the ordering is based on the target listing in the .spi.yaml? I'd like to control which one the user sees when clicking the package's documentation link.

Yes, that's correct!


Connectivity plugin: Potential memory leak on Android 6 and lower

The problem is in ConnectivityPlugin.java, lines 119 and 104.

I'm getting this as well. Lint is failing on release builds due to the connectivity plugin.

Since that PR is merged (thanks), I'm closing this issue.


Splitview

Don't hesitate to ask me any questions here.
Let me suggest to start by defining your widget model. I would suggest using a widget constructed like this:
class SplitView(DOMWidget):
    _model_name = Unicode('SplitViewModel').tag(sync=True)
    _view_name = Unicode('SplitViewView').tag(sync=True)

    left = Instance(Widget).tag(sync=True, **widget_serialization)
    right = Instance(Widget).tag(sync=True, **widget_serialization)

You can look at widgets libraries like https://github.com/QuantStack/ipytree or https://github.com/martinRenou/ipycanvas to get some ideas.

In case you use gitter: https://gitter.im/martinRenou

Thanks, after two years I've now used another approach:
Instead of widget-cookiecutter I've used a @magics_class in sw_cellmagic.py and took the JavaScript implementation of the split view widget from  https://github.com/NUKnightLab/juxtapose.


Fails in npm install

Hi, I am trying to install javascript sdk with npm command, and running into the issue below.
% npm config set @momento:registry https://momento.jfrog.io/artifactory/npm-public/
% npm install @momento/sdk
npm ERR! code FETCH_ERROR
npm ERR! errno FETCH_ERROR
npm ERR! invalid json response body at https://momento.jfrog.io/artifactory/npm-public/@momento%2fsdk/ reason: Unexpected token < in JSON at position 0

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ubuntu/.npm/_logs/2021-12-29T04_56_28_944Z-debug.log

For work around, I have succeeded to install it with downloading package from jfrog and installing from local file system with npm command.

@yuta-imai Thank you very much for reporting this! We found a typo in our instructions, sorry about that. The fix is merged now.


Reject function error 'Too many arguments to block call, expected 1, have 3

else if (target == RCTCameraCaptureTargetCameraRoll) {     [[[ALAssetsLibrary alloc] init] writeImageDataToSavedPhotosAlbum:imageData metadata:metadata completionBlock:^(NSURL* url, NSError* error) {       if (error == nil) {         resolve([url absoluteString]);       }       else {         reject(RCTErrorUnspecified, nil, RCTErrorWithMessage(error.description));       }     }];     return;   }
The reject function returns 'Too many arguments to block call, expected 1, have 3. The same thing happens for self.videoReject calls. 8 errors in total.
The errors occur in RCTCameraManager.m

Looks like RCTPromiseRejectBlock now only accepts 1 argument. I'm in RN 0.19. I'm not sure what version your supporting up to now but let me know if I should submit a PR fixing this by passing only the RCTErrorWithMessage to the reject fn. @lwansbrough

Actually, RCTPromiseRejectBlock recently moved from 1 to 3 arguments (https://github.com/facebook/react-native/commit/0c5f279c9df869db2afb3e1d92a54e6d930551ad), so this code is in line with the new syntax. But that change was prior to the release of 0.19, so I'm not sure why this wouldn't work with that version. :/

Not sure what's up or if my version is off but what I posted fixed my issue...
_(ツ)_/¯


search method expects 'queryString', doc lists 'query'

code: queryString
doc: query
i'm happy to just update the doc, but 'queryString` feels like a clumsy param name to me.

8e0eb62

Thank you.  As a naive user, reading and following the docs, I got bit by this just today.

sorry.  i just tagged v.1.0.2 and updated the live doc.


Minor Change in fq lint README

I recently figured out a minor misspelt flag in fq lint README
https://github.com/stjude-rust-labs/fq#validators
The README for Validator section states:
"Validate includes a set of validators that run on single or paired records. By default, records are validated with all rules, but validators can be disabled using --disable-valdiator CODE, where CODE is one of validators listed below."
Here, the --disable-valdiator flag is misspelled and needs to corrected to --disable-validator to avoid inconvenience to new users.
Cheers!

Thanks!


discovery adapter does not find devices

Is there a way to get discovery working when using docker?
I've tried to use smcroute but did not get it working.

Hi,
it's depending on you network. I think you have to run the container in MACVLAN mode to get discovery working.
But I never used the adapter. So no guarantee it then discovers every service...
Regards,
André

Hi,
it's depending on you network. I think you have to run the container in MACVLAN mode to get discovery working.
But I never used the adapter. So no guarantee it then discovers every service...
Regards,
André

Thx for the answer. Although I was hoping to get it working without MACVLAN. It just doesn't feel right to expose the container completely.

Thx for the answer. Although I was hoping to get it working without MACVLAN. It just doesn't feel right to expose the container completely.

I'm sorry, it's not my fault. It's a Docker thing.
But if you think about it's gonna be logical that you can't receive multicast packages to a container running in bridged mode. This not only affects the discovery adapter. There are some other Adapters which work with multicast packages to discover smart home devices.
If you don't want to run your container in MACVLAN or host mode, you can use a multihost slave like a raspberry pi to run adapters which need this kind of communication. Just think about.
If your question/ problem is solved, please close this issue. Thank you.
Regards,
André

I'm sorry, it's not my fault. It's a Docker thing.
But if you think about it's gonna be logical that you can't receive multicast packages to a container running in bridged mode. This not only affects the discovery adapter. There are some other Adapters which work with multicast packages to discover smart home devices.
If you don't want to run your container in MACVLAN or host mode, you can use a multihost slave like a raspberry pi to run adapters which need this kind of communication. Just think about.
If your question/ problem is solved, please close this issue. Thank you.
Regards,
André


Node.js LTS Support

We should support Node.js 4.x.x if the effort isn’t too crazy.

[ ] Remove unsupported ES6 features
[ ] Wire up CI testing to use Node.js 4.x.x

Node.js now shows 7.x.x as current, and Electron supports most of the ES6 features. We can safely require >6.x.x and mention it in docs.


Error handling in Form and Field

Added simple try catch error handling for DOMExceptions and normal Exceptions. Appologies if this merge request is not by standards, I couldn't find them anywhere.

👍


Select git workflow for development

Git Feature Branch
Gitflow

For Big Data Genomics organization projects, we use something like Git Feature Branch, with the following:

Intermediate/insignificant commits on pull request branches should be squashed
Pull requests are merged to master with the Github Rebase and Merge button
Pull request authors cannot merge their own pull requests
Pull requests cannot be merged until the CI build passes
Pull requests and commits are linked to issues whereever possible (e.g. Fixes #foo)
Releases use a process based on the Maven Release Plugin, which creates release tags and release branches

@heuermh - your comment in the initial code contribution about the branch from the repo instead of a fork (https://github.com/disq-bio/disq/pull/14#issuecomment-417726269) is related with this guidelines about the git-workflow.
Maybe we can somehow disable making branches in the repo, or enforce every commiter (including maintainers) to use their own fork. That will also keep the branchs in the repo clean and open only if necessary (e.g., master and gh-pages)

Decisions/recommendations made here should be added to CONTRIBUTING.md.

Fixed by #98


Minimum hardware required

What is the minimal hardware required to run a search ?  On CIFAR, a Nvidia GTX 1080 8Go doesn't  seems to be enough.  The GPU memory used varies a lot according to the model built. Is there a way to limit the model build space or something like that ?
  File "cifar.py", line 17, in <module> clf.fit(x_train, y_train, time_limit=12 * 60 * 60)
  File "/myPath/python3.6/site-packages/autokeras/image_supervised.py", line 238, in fit run_searcher_once(train_data, test_data, self.path, int(time_remain))
  File "/myPath/python3.6/site-packages/autokeras/image_supervised.py", line 41, in run_searcher_once searcher.search(train_data, test_data, timeout)
  File "/myPath/python3.6/site-packages/autokeras/search.py", line 177, in search self.metric, self.loss, self.verbose)]))
  File "/myPath/python3.6/site-packages/autokeras/search.py", line 280, in train verbose).train_model(**trainer_args)
  File "/myPath/python3.6/site-packages/autokeras/utils.py", line 124, in train_model self._train()
  File "/myPath/python3.6/site-packages/autokeras/utils.py", line 169, in _train loss.backward()
  File "/myPath/python3.6/site-packages/torch/tensor.py", line 93, in backward torch.autograd.backward(self, gradient, retain_graph, create_graph)
  File "/myPath/python3.6/site-packages/torch/autograd/__init__.py", line 90, in backward allow_unreachable=True)  # allow_unreachable flag
  RuntimeError: CUDA error: out of memory```

i think this is due to the accuracy of model below to some "initial threshold", resulting this program piled a large number of CNN layers in a single model and run out of graphics card memory. i used just 500 images to train a model and met this problem too.

I tried to set both width and depth to 4,(default are 1024 and 10),this problem still exists.


windows: fix regression when cancelling getaddrinfo

The req->addrinfow field contains the hints in that case, so we must not
free it.
This regression was introduced in f2bb8d3 by yours truly.
R=@piscisaureus
R=@bnoordhuis
This gives us a green build on Windows: https://jenkins-iojs.nodesource.com/view/libuv/job/libuv+any-pr+multi/60/

LGTM


Windows Version (Unable to confirm if installed Windows version is 10 or greater)

Steps to Reproduce

please check logs

...
...
...

Logs

Logs

Version: Windows 11 Professional
Version: 22H2
Operating system version: 22621.1194

How do I solve the following problems?

@danielesk
This has been fixed recently and should be available on latest master : https://github.com/flutter/flutter/issues/117890
Please switch to it to see the fix working.
flutter channel master
flutter upgrade
Closing as duplicate.


Setting Different Margins Bug

I had an issue with the free code camp mission above for a bit resetting my code worked to fix it but every time i load the page the first time that mission wont let me put in more than 3 sides of the margin.  It didnt stop me from finishing it but i just wanted to report the bug.

@Silverjayw thanks for the issue. Can you give us more information about the challenge you've talking about? It isn't clear from the post above what challenge you're referring to.


This sentence is a mess

Using precise X and Y coordinates for positioning glyphs, OpenType Layout fonts also can identify points for attaching one glyph to another to create cursive text and glyphs that need diacritical or other special marks.

"Fonts containing OpenType Layout tables can also specify how glyphs are to be attached to one another. X and Y coordinates are used to specify the attachment points between glyphs. This functionality can be used to attach diacritical marks to glyphs, as well as to create connected (cursive) text."
Clearer?

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

ID: 5b15ab9e-86c5-f6e0-0145-f9de77c60141
Version Independent ID: 0ebe3dec-cc24-b6a4-01b8-73bfdb79a71a
Content: Advanced typographic tables - OpenType Layout - Typography
Content Source: typographydocs/opentype/spec/ttochap1.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: PeterCon

Fixed for next version, using proposed text.


@aspnet/signalr - Unable to receive a message sent to concrete client

Describe the bug
I am unable to receive a message sent from AspNetCore WebApi via Microsoft.AspNetCore.SignalR Hub to concretely targeted client.
The issue I am encountering exists only, when I am trying to send the message to single client:
await Clients.Client("Valid connection Id").SendAsync("Client message", response);
I am successfully sending the same message passing the same flow, when sending it to all of the clients:
await Clients.All.SendAsync("Client message", response);
To Reproduce
The code pictures are just examples of what I am trying to do.
Back-end AspNetCore side

Front-end Angular side

Further technical details

ASP.NET Core version - netcoreapp3.0
Node.js version - 10.16.3
Angular version - 8.3.6
aspnet/signalr version - 1.1.4
Include the output of dotnet --info

The IDE (VS / VS Code/ VS4Mac) you're running on, and it's version
Microsoft Visual Studio Community 2019 - 16.3.5

First question I have to ask, how do you know you have a valid connection Id?

First question I have to ask, how do you know you have a valid connection Id?

Good question I could say. I am caching the connections to the memory cache and also a logger to the override Task OnDisconnectedAsync(Exception exception) method indicating, when some user gets disconnected.

And you're using Context.ConnectionId from inside the Hub to get the connection ID initially?

Context.ConnectionId

That's right.

I can't think of anything that would be wrong from the info provided. Would it be possible for you to make a minimal repro app and put it on github?

If you send a message directly to the Context.ConnectionId in OnConnectedAsync, does that work?

If you send a message directly to Context.ConnectionId in OnConnectedAsync, does that work?

No. I am not able to send any message to anyone from OnConnectedAsync. I have tried Client(Context.ConnectionId), All, Caller, but isn't it expected behavior? I am not able to send any message from OnConnectedAsync in other hubs as well.
I wrote an integration test, which shows exactly the same behavior. I am able to send a message, when I target all of the clients, but not able, when I select exact client. Therefore we are eliminating the possible issues in my client application and @aspnet/signalr library.

I can't think of anything that would be wrong from the info provided. Would it be possible for you to make a minimal repro app and put it on github?

Is there a small app netcoreapp3.0 with workin with example showing how to send private messages Clients.Client()

Sure, I created a super simple 3.0 app that sends to Clients.Client(Context.ConnectionId) in OnConnectedAsync.
https://github.com/BrennanConroy/3_0_Sample

Good job @BrennanConroy ! I have tested it and it is working perfectly. I created another branch of the your repo, where I cached in the memory the connections in OnConnectedAsync and send the message to the other user in SendMessage and it is working properly.
I noticed that you are using @microsoft/signalr, which is created for AspNetCore. However I am using @aspnet/signalr as I have described above. I guess that's my mistake and switching to<EMAIL_ADDRESS>will fix the issue.

However I am using @aspnet/signalr as I have described above. I guess that's my mistake and switching to<EMAIL_ADDRESS>will fix the issue.

Using @aspnet/signalr shouldn't cause any issues. I switched the sample to<EMAIL_ADDRESS>and it continued working as expected.

One more thing, not only sending to concrete client, but also OnDisconnectedAsync is not working on<EMAIL_ADDRESS>with netcoreapp3.0

What isn't working? You probably wont be able to send a message to the specific client that is closing because it is in the process of shutting down.
Could you please provide a sample app that shows these things not working properly?

I had an issue with authorization. Exactly the same as ddydeveloper had in https://github.com/aspnet/SignalR/issues/1764 . Hope that help to anyone.

            services
                .AddAuthentication("Bearer") // Just an example
                .AddJwtBearer(options =>
                {
                    options.Events = new JwtBearerEvents
                    {
                        OnMessageReceived = context =>
                        {
                            var accessToken = context.Request.Query["access_token"];
                            if (!string.IsNullOrWhiteSpace(accessToken))
                            {
                                context.Token = accessToken;
                            }

                            return Task.CompletedTask;
                        }
                    };
                });

I worked around the issue with Clients.Client(clientConnectionId) with Clients.User(staOpsConnections.Id). I guess I will come back, when I have to create a anonymous hub. I am sorry that I don't have enough time to provide you a sample repo of the issue.


Step08: Improve clarity around geolocation plugin + wrapper usage

Leaving a reminder here...
We denote in Step 08 of the v4 documentation - Using Geolocation - that we are making use of the Ionic Native wrapper + the Geolocation Cordova plugin.
However, while we point out that we're making use of both, we don't provide much detail as to how the wrapper improves the experience, nor how the plugin/wrapper work together. Probably worth calling out that errors with a particular "service" (like geolocation) are 99.99% of the time related to the plugin and not the wrapper.

Closing out w/ closure of #21


You have too many "likes"

Your README has a lot of "like"s. delete it.

I agree with you. I don't like am in disagreement with the decision to put in as many like instances of a four-letter word that starts with the letter L. This is like akin to people asking you to follow the law but breaking them at the same time.  Like give a reaction emoji of approval, comment, and subscribe to me on GitHub if you agree.

I'd, like, love to fix this, but like....
I really don't know how to like,... Phrase a normal sentence anymore


x/tools/gopls: references query does not count positional assignments to struct fields

Here're the steps to reproduce:

git clone https://github.com/ethereum-optimism/op-geth
git checkout optimism

Open with vscode, then cmd+left click disableTxPool field, it doesn't show the assignment to it here.
I guess vscode-go has a bug identifying positional assignments to struct field.

Thanks. This is a dupe of #66356.


Replace outdated stpmex link

Change Summary

Replaced outdated link to stpmex GitHub repository in V1.10 docs. The stpmex GitHub no longer exists, so the link now points to stpmex PyPI.
fixes #10908
Related issue number

fixes #10908
Checklist

[ ] The pull request title is a good summary of the changes - it will be used in the changelog
[ ] Unit tests for the changes exist
[ ] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[ ] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Fixed here: https://github.com/pydantic/pydantic/pull/11028


Make installer ask to add to windows startup

Nice work on this app! Works great on my Legion 5 pro with Windows 11!
What do you think about adding a step to the installer where it asks if you would like to autostart the app at windows login?

ThreeFingersDragOnWindows v2.0 just have been released. As the app is now a WinUI app that can be installed from the Microsoft Store or with a .Msix file, there is not really an "installer" anymore. However, this release now registers the app to run at system startup by default. This behavior can then be configured in the app Other Settings page.
See release v2.0.1
Enjoy!


Azure: Add approval job

This will secure from any PRs running from forks
on Azure without approval.
Relates-To: OLPEDGE-2823
Signed-off-by: Yaroslav Stefinko<EMAIL_ADDRESS>

Not relevant anymore.


impl achievement features

Description
Needs updated metadata, and mapper#1 on internal git
Related Issue

close #897
close #1907
Checklist

[X] The target PR branch is develop branch

Internal Task: https://git.snapgenshin.com/DGP-Studio/Snap.Resource.Management/pulls/1


Support for delay and/or stochastic delay problems

It would be helpful to apply GPUs for ensemble studies with the delay and stochastic delay problems.

These will be much more difficult because the history would then have to be accessible from the generated kernels which means that they will have to have access to the memory. There is an alternative formulation of GPU-able functions that might be able to handle this more directly though.

Maybe it's not the right level of abstraction, but I have had great success on GPU for fixed step solver (EM & Heun) with lots of delays (n^2 as in the issue in the SDDE repo), because the memory access to the delay buffer can be coalesced (a la SIMD) across solutions (assuming they have identical delays) and practically hides all the arithmetic, i.e. it's memory bound..  This doesn't do anything about the other issues related to delays like discontinuity propagation, but it could be a start.  I will try to come up with an example in Julia.

Yeah, that's something we can hack together in SimpleDiffEq and expose pretty quickly, and that would be fine. The adaptive higher order methods though have a lot more caching details in there though.


Prevent product name in header from wrapping

Currently the product name in the header wraps when the space shrinks which doesn't look great.

Adding white-space:nowrap prevents that so that the product name as a whole drops to a new line when space is shrunk.

GIFs

@dashouse you're right, something like WEEE service will break. hmmm

@igloosi You'll have to remind me about WEEE, but if it's a service it would be using the service name not the product name.

@dashouse https://weee.service.gov.uk/ yeah true

Cool, looks like your change only affects product name anyway which is fine, services are less likely to be only two words anyway so wrapping is ok

@dashouse I've also added some spacing for #891
does that look ok?

👍 LGTM

and Ipad Pro example if someone had both product and service name in there
before:

after:


Feature/ci linter docs gdev 1697

Proposed Changes

CI for schema linter
CI for checking-generating docs

The CI for the entity documentation will fail but there is now easy way to fix it in this PR. Let's try to merge it and fix it in a different PR.


 is not a valid SSR component

Any solution for this issue?
<Chart> is not a valid SSR component. You may need to review your build config to ensure that dependencies are compiled, rather than imported as pre-compiled modules

Duplicate of https://github.com/himynameisdave/svelte-frappe-charts/issues/14.
As of<EMAIL_ADDRESS>the <Chart /> component can never be a valid SSR component because it relies on the window object in the browser. Perhaps this will be fixed in<EMAIL_ADDRESS>which is currently under development.
Regardless though, this is not an issue with this Svelte wrapper library, but is an issue in the underlying frappe-charts library, so I'll be closing this. Feel free to file an issue over there.

Similar to #14, I fixed this with npm i --save-dev svelte-frappe-charts


docs: allow-scripts link is a 404

Under "How to secure your app against supplychain attacks" the link for @lavamoat/allow-scripts goes to https://github.com/LavaMoat/LavaMoat/blob/main/LavamoatAllowScripts which is a 404 page (for me anyway)
I wanted to send a tiny PR to fix that directly, but the README.md actually looks fine, defining the link like this:
1. disable/whitelist dependency lifecycle scripts (eg. "postinstall") via [@lavamoat/allow-scripts](LavamoatAllowScripts)
...
[LavamoatAllowScripts]: ./packages/allow-scripts

I see nothing wrong with that, so I don't know why this goes to the bad blog URL, instead of https://github.com/LavaMoat/LavaMoat/tree/main/packages/allow-scripts

ah! for url refs it should be [@lavamoat/allow-scripts][LavamoatAllowScripts] thanks!

Oh, right! I didn't notice the wrong braces 😅


"reputable news" link 404s

The link underlined by this image:

Leads to https://standwithukraine.pp.ua/WarNews.html, which 404s.

The link works when there hyphens in the URL ("stand-with-ukraine").

I don't see this bug when i navigate to https://stand-with-ukraine.pp.ua/ and click this link.
It also works from https://github.com/vshymanskyy/StandWithUkraine/tree/main/docs#readme
I don't think there's any real issue here

I don't get a 404, I just get a different website.

vs

i don't know why they did it...
On Wed, 13 Apr 2022, 6:21 pm Jacob Dreesen, @.***>
wrote:

I don't get a 404, I just get a different website.
[image: grafik]
https://user-images.githubusercontent.com/424602/163214407-805052bf-903c-431e-982e-369df35aa2ee.png
vs
[image: grafik]
https://user-images.githubusercontent.com/424602/163214511-ae4e0f5f-641e-4527-b26d-7e98178653e8.png
—
Reply to this email directly, view it on GitHub
https://github.com/vshymanskyy/StandWithUkraine/issues/112#issuecomment-1098183980,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALP3FFKKDYZV765Q73HAH3VE3RANANCNFSM5TIK7SNA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I don't know either, but they did. I assume others have too.
IMO redirecting standwithukraine.pp.ua to stand-with-ukraine.pp.ua is probably the best/simplest solution.

i'm not afilliated with http://standwithukraine.pp.ua/
On Wed, 13 Apr 2022, 7:41 pm Stephen Mathieson, @.***>
wrote:

I don't know either, but they did. I assume others have too.
IMO redirecting standwithukraine.pp.ua to stand-with-ukraine.pp.ua is
probably the best/simplest solution.
—
Reply to this email directly, view it on GitHub
https://github.com/vshymanskyy/StandWithUkraine/issues/112#issuecomment-1098265561,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALP3FHIRTJLN6BBD5K7BPDVE32MLANCNFSM5TIK7SNA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Oh weird. Looks like some sort of phishing scam. Yesterday, it was a copy of https://standwithukraine.pp.ua.

https://standwithukraine.pp.ua/ is a legit (but outdated) website of "Come Back Alive" Non-Government Charity Organization.
Which is a verified organization, and is present on our list as well :+1:

https://standwithukraine.pp.ua/ is a legit (but outdated) website of "Come Back Alive" Non-Government Charity Organization.
Which is a verified organization, and is present on our list as well :+1:

Interesting... When I was registering https://stand-with-ukraine.pp.ua, https://standwithukraine.pp.ua was not a thing and I was considering to register it myself as an alias. But decided that I probably don't need it. Maybe somebody from the charity decided to get it, after all.

After some verification, we consider standwithukraine.pp.ua as possible a phishing attempt.
ComeBackAlive org refused any relation to this website.
However, it looks like it was just used once in the VIM wiki. Currently, it's not mentioned anywhere else: seash results


Thoughts about Uproot4 ↔ boost-histogram/hist integration

Since this comes after reading and writing, it's a few steps removed.
However, I think that the histogram objects in Uproot4 will not be usable as such, but will consist entirely of conversion methods to NumPy-style, boost-histogram, and hist. They should reuse code as much as possible, so it might be hist-by-way-of-boost-histogram and/or aghast.
Mentioning @henryiii and @LovelyBuggies.

Actually, this conversation is happening here: https://github.com/scikit-hep/uproot/issues/502#issuecomment-656864720


Some unicode tags return empty slug

Certain unicode doesn't yield a usable slug.
>>> obj.tags.add('(ง’̀’́)ง')
>>> obj.tags.all()
[<Tag: ง’̀’́)ง>]
>>> obj.tags.all()[0].slug
u''

We've solved this internally by doing this:
from shortuuid import uuid

name = data['name'].encode('ascii', 'ignore').decode('utf-8')
if not taggit_slugify(name):
    data['slug'] = 'tag-{}'.format(uuid())

Seem reasonable?  If so, I can add it to this method and create a PR.

Do you have 'unidecode' installed?

I do not.

So if you install that, it might fix this issue. We should probably add a note about it in the docs somewhere....
Relevant PR: https://github.com/alex/django-taggit/pull/315

That seems to work fine, thanks!

Yes, installing "unidecode"  fixed issues.  (pip install unidecode)

I have installed unidecode, but my ciryllic tags still have empty slugs.

@AliIslamov could you provide a snippet of how you are creating your tags, along with an assertion that the slug is indeed the empty string?
In particular, could you try things like calling tag.slugify(tag_string) to confirm that the slugify method is returning an empty string?

I would like to add more tests or something here to work through this issue but am having a hard time conceptualizing at what part of the system this is failing

@AliIslamov could you provide a snippet of how you are creating your tags, along with an assertion that the slug is indeed the empty string?
In particular, could you try things like calling tag.slugify(tag_string) to confirm that the slugify method is returning an empty string?

I already have solved my problem.

Have written in settings.py:

TAGGIT_STRIP_UNICODE_WHEN_SLUGIFYING = True

Have installed unidecode:

pip install unidecode

Have written in models.py for my application:

from unidecode import unidecode

And now everything works well 👍

I would like to add more tests or something here to work through this issue but am having a hard time conceptualizing at what part of the system this is failing

We can conclude that while unicode is enabled by default, cyrillic tags get an empty slug.
The situation is solved by installing the unidecode module and importing it into the models.py file, in which the TaggableManager () field is present.
In my opinion it makes sense to add this instruction in documentation.


Added queries for SI Papers by id and Proposed Negative SI Papers by id

@katylouise second failed check was my mistake (the PR validation build referenced an older repository name).
Can you commit another to check?


cretae go file for "release"

hello
I want to make a golang file for "release".
This is my command:
go build -o testi.exe -ldflags "-H windowsgui"
What is the correct way?

We do questions in the forums, not in our bug tracker. See https://golang.org/wiki/Questions


[Bug Report] About input component in safari browser, delete character cursor jump problem

Element UI version
2.6.0
OS/Browsers version
macOS Mojave 10.14.1/  safari 12.0.1 (146<IP_ADDRESS>.1)
Vue version
2.6.8
Reproduction Link
https://jsfiddle.net/mmx38qxw/
Steps to reproduce
one：Type a character in the input box
two： Type a character in the input box
three：Delete the character
and then The cursor jumps to the end
What is Expected?
The cursor is positioned at the delete position
What is actually happening?
The cursor jumps to the end

duplicate of https://github.com/ElemeFE/element/issues/14540


Bugfix/home page typo

Bug fix

 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.:x: Zeno-Ta sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


DigitalOcean managed K8s LoadBalancer requires an exposed HTTP port for external healthchecks

In doc\jitsi-helm\templates\jvb\service.yaml, it seems that the following linea are required according to docs:
I am not sure whether it is only required in Digital Ocean only, or whether there are no side effects in other cloud providers. But, at least in DO, it works w/ the following additional lines in service.yaml.
    - port: {{ default 8080 .Values.jvb.livenessProbe.httpGet.port }}
      protocol: TCP
      name: tcp

Originally posted by @humble92 in https://github.com/jitsi-contrib/jitsi-helm/issues/80#issuecomment-1633345805

I added knobs to specify these additional ports and annotations for JVB service, so it should work well now.


Update args builder to respect WithAudio

Resolves https://github.com/mauricioabreu/mosaic-video/issues/1

@rafiramadhana did you run the test?
I just set up a test workflow. Would you mind to run the tests locally and rebase your branch with the main branch?

@rafiramadhana did you run the tests?

I only run the package's unit test

I've just set up a test workflow. Would you mind to run the tests locally and rebase your branch with the main branch?

ok, I will take a look

@mauricioabreu
the test runs fine on my local
➜  mosaic-video git:(1-with-audio-field) ✗ go test -v ./...
?   	github.com/mauricioabreu/mosaic-video	[no test files]
?   	github.com/mauricioabreu/mosaic-video/cmd	[no test files]
?   	github.com/mauricioabreu/mosaic-video/internal/config	[no test files]
?   	github.com/mauricioabreu/mosaic-video/internal/locking	[no test files]
?   	github.com/mauricioabreu/mosaic-video/internal/logging	[no test files]
?   	github.com/mauricioabreu/mosaic-video/internal/mocks	[no test files]
?   	github.com/mauricioabreu/mosaic-video/internal/mosaic/command	[no test files]
?   	github.com/mauricioabreu/mosaic-video/internal/uploader	[no test files]
=== RUN   TestBuildFFMPEGCommand
=== RUN   TestBuildFFMPEGCommand/Multiple_URLs_with_audio
=== RUN   TestBuildFFMPEGCommand/Multiple_URLs_without_audio
--- PASS: TestBuildFFMPEGCommand (0.00s)
    --- PASS: TestBuildFFMPEGCommand/Multiple_URLs_with_audio (0.00s)
    --- PASS: TestBuildFFMPEGCommand/Multiple_URLs_without_audio (0.00s)
=== RUN   TestGenerateMosaic
--- PASS: TestGenerateMosaic (0.00s)
PASS
ok  	github.com/mauricioabreu/mosaic-video/internal/mosaic	0.004s
=== RUN   TestGenerateMosaicWhenLockingFails
--- PASS: TestGenerateMosaicWhenLockingFails (0.00s)
=== RUN   TestGenerateMosaicWhenExecutingCommandFails
--- PASS: TestGenerateMosaicWhenExecutingCommandFails (0.00s)
PASS
ok  	github.com/mauricioabreu/mosaic-video/internal/worker	(cached)

I see the problem @rafiramadhana
We are using ElementsMatch from assert. It is not a good option here because the order of the parameters matter when we talk about FFMPEG. I think we need to change the way we assert the command is right (parameters, order, etc)

i see, wdyt about this?
i think if we are not planning to add more args (frequently in the near future), this is good enough
args := []string{
	"-loglevel", "error",
	"-i", cfg.StaticsPath + "/background.jpg",
	"-i", m.Medias[0].URL,
	"-i", m.Medias[1].URL,
	"-filter_complex", filterComplex,
	"-map", "[mosaico]",
}

if m.WithAudio {
	args = append(args, "-map", "1:a")
}

args = append(args, []string{
	"-c:v", "libx264",
	"-x264opts", "keyint=30:min-keyint=30:scenecut=-1",
	"-preset", "ultrafast",
	"-threads", "0",
	"-r", "24",
	"-c:a", "copy",
	"-f", "hls",
	"-hls_playlist_type", "event",
	"-hls_time", "5",
	"-strftime", "1",
	"-hls_segment_filename", segmentPattern,
	"-method", "PUT",
	"-http_persistent", "1",
	"-sc_threshold", "0",
	fmt.Sprintf("http://localhost:8080/%s", playlistPath),
}...)

return args

for the unit test, I will update them to also assert the args ordering

@rafiramadhana yes, this is good enough.

@rafiramadhana yes, this is good enough.

ready for review

@rafiramadhana thank you!

@rafiramadhana tests are failing. Could you check why?

@rafiramadhana tests are failing. Could you check why?

updated, ready to review

@rafiramadhana tests are still failing as you can see in the logs. Could you check why?

@rafiramadhana tests are still failing as you can see in the logs. Could you check why?

ah sry, i've pushed the fix. thanks!

Thank you @rafiramadhana
merged


Adding steering election blog tweet

cc: @kaslin

/lgtm
Thanks for putting this together, @rajula96reddy!


fix(webapp): Fix 989 issue

Motivation and Context
Allow the project to work anonymously
Description
Fix 'getLogger' error that prevent the webapp to start anonymously.

@microsoft-github-policy-service agree

Thanks for the fix!


Nested lists have too much whitespace between items

Bug Report
Environment
Zola version: 0.17.2
tabi commit: 0b6cb31bba927315c975dfe5bcf12b691a6e2573
Expected Behavior
See reproduction case and screenshot below. Zola supports the markdown syntax where adding an empty line between list items creates more space between items. This is achieved by adding a
 element in the
, which creates proper spacing in the nominal case thanks to the margin-bottom property on
.
It does not play nice with nested lists though, as there's too much space between the main item's
 and the sub-item
. In that case, I expect no spacing (or a small spacing) between these elements, while still having proper spacing between the sub-items and the next top-level item.
Not sure whether the fix is a CSS fix in tabi or a code fix in zola though.

for now I'm working around it by wrapping the list in a div and adding the following style: article div.compact-list p { margin-bottom: 0.4rem; }
ideally, we'd fix it with a li p:has( + ul) selector (to neutralize the margin if a p's next sibling is a ul),  but this does not work yet on Firefox
or the margin should be set on the
 element, but only in these "spaced-out lists" cases. Again, that would require a li:has(p) selector that does not work everywhere?
best option to me would be to neutralize the margin on li p, and add one on article li :last-child (to match the last element of li's, even if it's the ul)

Current Behavior
Tell us what happens instead of the expected behavior.

Step to Reproduce
# Lists with no spacing are unaffeced

1. first item that needs no context
2. second item that needs details:
    - step 1
    - step 2
3. last item

# Lists with spacing are affected

1. first item that needs no context

2. second item that needs details:
    - step 1
    - step 2

3. last item

Generates the following markup

   Lists with no spacing are unaffeced

first item that needs no context

second item that needs details:

step 1

step 2

last item

   Lists with spacing are affected

first item that needs no context

second item that needs details:

step 1

step 2

last item

Found a better solution while writing the issue: the following CSS gives the expected layout:
article li p {
    margin-bottom: 0;
}

article li :last-child {
    margin-bottom: max(2.3vmin,24px);
}

Does this solution change the appearance of lists without spacing? If everything else remains the same, could you do a PR?

It should not, as lists without spacing are simple
the text
 lists. I'll triple check and open a PR.

When I do this PR, should I factor this margin-bottom: max(2.3vmin,24px); value from p into a variable, so that it stays in sync?

Yes, please.


[Bug]: Error duplicating OOTB Custom Serving Runtimes several Times

Is there an existing issue for this?

[X] I have searched the existing issues

Deploy type
OpenDataHub core version (eg. v1.6.0)
Version
main
Current Behavior
Follow up of #1680
When creating a new cloned OOTB Custom Serving Runtime, we added this logic:
metadata.name -> ${metadata.name}-copy
If the metadata.name is not changed, it work the first time (there's no other Custom Serving Runtime with that name), we might need to indicate the user that they need to change both Display Name and metadata.name
As per @vconzola comment:

I'm guessing that I need to either change the display-name or the name (or both) in the YAML file. When I change only the display-name the Add button is enabled, but when I click it I get an error saying the serving runtime already exists. When I change the name in the metadata I get a 403 error, which is meaningless to me. So, I'm stuck.

Expected Behavior
Users are aware that they need to change Display Name and metadata.name
Steps To Reproduce

Go to Custom Serving Runtimes
Duplicate an OOTB Serving Runtime
Submit that Serving Runtime without changing any value
Duplicate again the Serving Runtime
Check that it will fail due to duplicate names

Workaround (if any)
Just changin the metadata.name is enough.
What browsers are you seeing the problem on?
No response
Anything else
No response

@vconzola I was trying to replicate your comment, but it's not giving me a 403 once I change the metadata.name, can you try that again an share the steps?

@vconzola @lucferbux We will need to together and discuss the next step.

Kubernetes does support name generation by setting the generateName field. So if we set generateName: something-copy- instead of name: something-copy it would result in a name of something-copy-h487z for example.
Unfortunately you cannot try this out today because we currently have custom validation that requires a name or otherwise results in an error:

We could generate the YAML as follows with the generateName property and a comment to guide the user if they want to specify a specific resource name:
  # <explain the result of generateName along with instructions on how to specify the name directly>
  generateName: ovms-copy-
  #name: ovms-copy

@lucferbux If you would like to include this issue in the 1.34 Release Notes, please provide the text for the "Known issues" section.

@andrewballantyne If you would like to include this issue in the 1.34 Release Notes, please provide the text for the "Known issues" section.

@andrewballantyne Suggested text -- let me know what you think:
"
ODH-DASHBOARD-1758 - Error duplicating OOTB custom serving runtimes several times
If you duplicate a model-serving runtime several times, the duplication fails with the Serving runtime name "" already exists error message.
"

@bredamc That works, thanks!

https://issues.redhat.com/browse/RHOAIENG-1207


⚠ ClamAV has degraded performance

In 5eb9a27, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in d97b35c.


sample json file to work with jmxtrans - elasticsearch combinaiton

can some one give me a sample json to work with jmxtrans-elasticsearch combination?  I could not find it in wiki documentation.

In the absence of documentation, the code always helps...

Thanks.  I checked the code and I could not understand what values to sent.  I am not good at java.  It will be helpful if any sample json file is provided.

The following properties are available on the elasticsearch writer:

typeNames
booleanAsNumber
rootPrefix
connectionUrl

The only one really specific to elasticsearch is connectionUrl, which is the URL to your elasticsearch instance.
So the output writer part of your config should be something like:
"outputWriters" : [ {
  "@class" : "com.googlecode.jmxtrans.model.output.elastic.ElasticWriter",
      "connectionUrl" : "http://your.elasticsearch.server.example.net"
  } ],

As always, feel free to start a wiki page and document whatever you learn!

Hello,
I am interested too to participate to this wiki page. I asked a question about Elastic and JmxTrans-agent. Another issue has been created into jmxtrans-agent project.
I like the two solutions.
Best regards,
Nicolas

This is a sample configuration file for sending a Kafka JMX metric to ElasticSearch:


Add file 0000-01-02-lucasssfigueredo.md

created a branch, created a file and made a commit, and opened a pull request

Merging my-slide branch to master


1.0.0

Version 1.0.0 of the Omnia React Native Mapbox Module with Navionics and Aeris Support.
Not really intended to be a release, but more of a tracker of diffs against main when made stable.

Apologies, meant to create this PR against our local version of main.


IPROTO-64 Annotation based marshallers should unmarshall empty repeat…

…able fields as empty collections instead of null
https://issues.jboss.org/browse/IPROTO-64

Integrated in master.


New DSL for numbers

Description
Introduce a new experimental DSL for building a holder from a number.
Checklist

[ ] Implement.
[ ] Test.
[ ] Refactor.
[ ] Update Work in progress section in changelog.

Reuse the defined DSL available in the kotools.types.number package.


New-PodeWebButton not working on Linux Container

Describe the Bug
Using New-PodeWebButton works as expected in a VM or physical machine as below:
 new-podewebbutton -Name test -ScriptBlock {                     Show-PodeWebToast -Message "Job finished successfully!" -Duration 10000                 }
Running this block of code in a container does not do anything.
Steps To Reproduce
Create New-PodeWebutton and run in container. It does not execute the code in the script block
Expected Behavior
I expect that the code in the scriptblock runs as it does on a vm or physical machine.
Screenshots
n/a
Platform

OS: container (linux)
Browser: Edge
Versions:

Pode.Web: [e.g. Pode.Web v0.8.3]
PowerShell: [e.g. PS7.x]

Additional Context
This has been tried on VM (works) / Podman(does not work)/ kubernetes (does not work)

I have been unable to reproduce this. Running in a container, the toast works fine.


Documentation page issue with iPad in horizontal orientation

The title says it all. When using an iPad to view the documentation (http://foundation.zurb.com/sites/docs/), the right hand side bar moves to the left when I scroll down (enough to hide the first <hr/>).

Same behaviour with: Safari, Google Chrome and Firefox.
With Opera mini and Brave, the element does not slides down.

Hi, does this still occur?

Closing as outdated as this does not happen in the v6 docs.


Slow startup times

Background
Hi everyone.
Since recent commits in master, lnd startup time is very slow compared to before.
This issue appears for me compiling lnd to Android (using the in-mem gRPC), but the same probably applies to just running it on a computer.
I'm compiling with go 1.13 and latest gomobile.
I've tested it on 2 different computers and I've tried on an isolated android app running only lnd.
I did some simple benchmark testing giving the following results:
For starting up + unlock wallet, the startup time before was around:
1.03-4s (avg 1.13s)
Now on latest master, the startup times is around:
3.59-10.71s (avg 5.03)
These high startup times basically make mobile wallets infeasible.
I've seen it going even higher than 10.71s.
Your environment

lnd master (as of writing 7fcac54)
Linux compiling aar file for Android
go 1.13 and latest gomobile

Steps to reproduce
Compile latest master, run lnd on testnet/mainnet with Neutrino.
The last known commit I'm absolutely certain works* without the slowdown regression is f60e4b1e14547a76747a012cff663e828c9da313
*) Unfortunately unless you have it in your go cache, compiling lnd at that commit doesn't seem to work anymore because of some revision issue with btcwallet, giving the following error when compiling:
<EMAIL_ADDRESS>invalid pseudo-version: revision is longer than canonical (284e2e0e696e)

I've been conducting my own testing and I have found out that one of the following dependencies causes the regression:
btcsuite/btcwallet or lightninglabs/neutrino.
These two commits caused the regression
According to my tests, these two commits caused the regression:
5fc003d6 and 5ecde6d5.
5fc003d6 updates btcd, neutrino and btcwallet.
5ecde6d5 makes use of new neutrino APIs.
Presumably there's a regression in neutrino.

Seems to be happening because of freelist disable in btcwallet: https://github.com/btcsuite/btcwallet/commit/e809748b1e5563ef90ec04246762e7b67f283706

Yes, as as discussed on the lnd slack, reverting btcsuite/btcwallet@e809748 puts startup times back to normal.

I did some testing with this, and it is definitely slower with disabled free list sync. A run with the freelist sync disabled (current master) looks like this:
2019-09-30 13:35:29.322 [INF] CHDB: Checking for schema update: latest_version=11, min_upgrade_version=9, db_version=11
2019-09-30 13:35:31.318 [INF] BTCN: Loaded 110 addresses from file '/root/.lnd/data/chain/bitcoin/testnet/peers.json'

while reverting the above mentioned btcwallet commit results in runs like this:
2019-09-30 13:37:41.144 [INF] CHDB: Checking for schema update: latest_version=11, min_upgrade_version=9, db_version=11
2019-09-30 13:37:41.165 [INF] BTCN: Loaded 110 addresses from file '/root/.lnd/data/chain/bitcoin/testnet/peers.json'

So opening the database goes from a few milliseconds to ~2 seconds in my case (testing on $5 DigitalOcean box), so I can definitely see it being a problem on slower hardware.
@hsjoberg Do you consistently get increased startup (look at the two log lines I posted above) up to 10s? I experienced this on first restart after switching between freelist/nofreelist, but next restart after the db was updated to the new setting was usually faster for me.

Also inspected the Lightning App running on Pixel 2 XL, seeing about 5s to open db:
2019-09-30 13:48:02.290 [INF] CHDB: Checking for schema update: latest_version=11, min_upgrade_version=9, db_version=11
2019-09-30 13:48:07.008 [INF] BTCN: Loaded 112 addresses from file '/data/user/0/engineering.lightning.LightningMainnet/files/data/chain/bitcoin/mainnet/peers.json'
``

Do you consistently get increased startup (look at the two log lines I posted above) up to 10s? I experienced this on first restart after switching between freelist/nofreelist, but next restart after the db was updated to the new setting was usually faster for me.

@halseth Yes I consistently* get increased startup time. I haven't seen it go down to levels it was before (~1.5s).
*) It is not always 10s though, it seems like every time I restart my phone (OnePlus 6), the startup time will be around 8s (for mainnet, ~12s testnet) for some reason.
After that for the next startup, it goes down to around 2.5s mainnet and 4s testnet
But I've also seen it going up to 10s occasionally after that as well, seemingly at random.

@hsjoberg are you seeing this with a new database or with an existing wallet?

@cfromknecht I started noticing it on an existing wallet, but I've checked and it happens from a complete clean install as well with no previous data.

Immediately after restart of my DO node, opening the db goes to ~9s. On the phone I couldn't see this jump after a restart.

compacting the db didn't make a difference either.

Most of the increased startup time comes from opening the neutrino db, according to my testing.

@hsjoberg Could you get a cpu & heap profile?

@Crypt-iQ I have mostly tested this on Android (compiled with gomobile) so far, where profiling probably will be more difficult.
I'll set up a proper dev environment for Go when I get my new computer.


Add indication that scan is in progress

Watch for data file creation, when a file appears create scan and use indeterminate progress indicator to indicate that the scan is in progress.

We might need to introduce a more explicit scan state.

this is being worked on with a JSON output file from the 4D Camera receivers. WIP

JSON status files make this unnecessary. This can be closed.


SCT-966 Finish AssemblyHomlogy tests.

Also a good bit of refactoring for testability & exception handling.

Codecov Report

Merging #67 into master will increase coverage by 5.46%.
The diff coverage is 86.95%.

@@             Coverage Diff              @@
##             master      #67      +/-   ##
============================================
+ Coverage     56.55%   62.01%   +5.46%
- Complexity      403      432      +29
============================================
  Files            57       59       +2
  Lines          1862     1864       +2
  Branches        315      315
============================================
+ Hits           1053     1156     +103
+ Misses          798      697     -101
  Partials         11       11


error when doing "docker-compose up"

I follwoed Docker Getting Start Guide, saw error when doing "docker-compose up"
ERROR: for wherehowsdocker_wherehows-mysql_1  Cannot start service wherehows-mysql: driver failed programming external connectivity on endpoint wherehowsdocker_wherehows-mysql_1 (33c6e19419e1edb7284cfeb2008ea43a7cf5ce5aa80c73b1d852952c24d61d83): Error starting userland proxy: Bind for <IP_ADDRESS>:3306 failed: port is already allocated
ERROR: for wherehows-mysql  Cannot start service wherehows-mysql: driver failed programming external connectivity on endpoint wherehowsdocker_wherehows-mysql_1 (33c6e19419e1edb7284cfeb2008ea43a7cf5ce5aa80c73b1d852952c24d61d83): Error starting userland proxy: Bind for <IP_ADDRESS>:3306 failed: port is already allocated
ERROR: Encountered errors while bringing up the project.
I have checked followings:

$ lsof|grep 3306 didn't return anything
tried to change 3306 in wherehows-mysql/Dockerfile AND docker-compose.yml to a random port #, I got mysql connection rejected error.

Please help, thanks

can anyone help?
I tried the same on 2 macbook, same error.

@richardxin Is there any existing mySQL installed on your local machine? You need to close it or disable 3306 port first before you start your docker


Alpine image deprecated

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
Why is the alpine version marked as deprecated in the wiki under the docker install instructions? it mentions dns cache issues, but I can't find any reference to it in the github issues. Should current alpine users move to the latest debian based version?
🐻 Uptime-Kuma Version
1.19
💻 Operating System and Arch
Debian 11 x86
🌐 Browser
Firefox 108
🐋 Docker Version
20.10.22
🟩 NodeJS Version
No response

https://github.com/louislam/uptime-kuma/issues/295
Error: EAI_AGAIN
I would recommend Debian based, because it also got more features such as Cloudflare tunnel and high performance health check (no cpu spike for slow devices)

Ah that makes sense. I never had any issues and it looks like it only affected hosts with kernel older than 5.x.
Will alpine be abandoned in the future and only debian build maintained? Or is it just not supported and use at your own risk?

For 1.X.X, I won't update Alpine dockerfile anymore, but I still build image for it.
Alpine based will be dropped completely in 2.0 in my plan.


Snakebite calling wrong python directory

Franks-MacBook-Pro:bin frankblau$ snakebite -version
-bash: /usr/local/bin/snakebite: /usr/local/opt/python/bin/python2.7: bad interpreter: No such file or directory
How do I tell snakebite where python is?

Seems there is a problem with your setup. Snakebite uses /usr/bin/env python to figure out where your python is. Does the command python work for you?


MathJax Live Demo

Hi,
Is there ( online ) tool to convert from MatJax to plain text  ?

It's not clear what you are after with "from MathJax to plain text". MathJax can convert TeX, MathML, or AsciiMath to either HTML with CSS or SVG; the Accessibility Extensions can additionally create textual descriptions from the input.


Fix dataloading for cpt

This PR fixes an issue introduced in this PR https://github.com/huggingface/alignment-handbook/pull/135 where "unused" columns would be removed. However, the definition of what "unused columns" are is not user-defined and therefore prone to unexpected side-effects. Moreover, this was only focused on instruction or DPO-like datasets, and did not account for pretraining datasets.
This PR makes sure that the user-defined text_column (that is used to collect the pretraining column) is not removed when run_cpt.py is used.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Added tvermaashutosh in the list

Related issue
What changes does this PR do?

Added tvermaashutosh in the list

Screenshots (If any)
Questions

@schmelto
Kindly review my PR and please merge it.


[Hacktoberfest] Isolate flow execution from business process and add flow execution log

1. Why is this change necessary?
Currently when a flow fails for any reason, the business process (request, queue message, etc.) which triggered it will also fail. This is due to the database transaction being marked for rollback only. In the worst case, this can lead to the store checkout failing for customers, which should always be avoided.
2. What does this change do, exactly?
This changes the flow executor to collect all flow triggers which are dispatched during the business process and then run them only after the business process has concluded. As this leads to the user having no visibility of failed flow runs (the originating process completed without errors), this PR also adds logging for every flow execution.
The logging of flow executions is implemented for both versions 6.6 and 6.7, however as changing the time of flow execution is a breaking change it is only added with 6.7.
3. Describe each step to reproduce the issue or behaviour.

Create a flow that in any way crashes (illegal state transition, infinite loop, etc.)
Execute a business process which triggers the broken flow

4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfill them.

https://github.com/shopware/shopware/pull/5744#issuecomment-2514580549
I don't really understand what Danger is on about, maybe i put the files in the wrong directory or something?
However the tests are there:

https://github.com/shopware/shopware/pull/5744/files#diff-db1f26a0fccb25eb3d48b912351ac05a3c4fcd1178cbae47171a42b6aa5d500d
https://github.com/shopware/shopware/pull/5744/files#diff-196a3f0aaed5b190159bc354a2f3f80ce1a9f4a6ecc4bb368f9d460ce98e3465
https://github.com/shopware/shopware/pull/5744/files#diff-59a2d003412911c63c1b24d5a6ad78690f6cf62453a2c194905f37e7550974e4


Fix some small issues

README.md has a wrong path to the cmd folder
The alpine image needs to run apk update before installing packages

Nice!
Btw: could you enable the Issues feature for this fork? That would enable discussion on topics that can not be handled through pull requests.


Update doc for Custom operator.

Update documentation for Custom operator.
@mli, @zackchase, @madjam, @nswamy, @Roshrini, @jiajiechen
Please review and merge.

Hi all what's the status on this update? I noticed some checks didn't pass. Can we resolve why this is happening?

Hi all. What's up with the checks not passing. Can we resolve and merge? Thanks for making the changes Indu!

random failures on traivs. forced.


Run commands with 'sudo' when registered user is not root

Let the user choose if he wants commands to be run as sudo or not.

http://askubuntu.com/questions/192050/how-to-run-sudo-command-with-no-password
Commands can fail when user asks to run them as sudo because of password prompt. Some configuration is needed on the server itself with sudoer configuration.

http://askubuntu.com/questions/470383/how-to-avoid-prompt-password-for-sudo