Support SNI mode in TCP

From Kirill Gavrilov on Slack:
HTTP mode with sni means that you will have to terminate TLS (im probably wrong here)? I would not like to terminate but pass-through still able to make rules based on sni.
Here is an examples with ssl/tls passthrough in TCP mode with sni acl https://www.haproxy.com/documentation/aloha/7-0/deployment-guides/tls-layouts/

Hi @tamal. Thank you for the issue created. Loadbalancing traffic based on sni doesn't require SSL termination. That's what sni was designed for. An ability to detect sni in haproxy is achieved through the extension.

TCP logic:
spec.TLS found and noTLS == false ---> bind ssl crt crt_file_path  // TLS terminated at haproxy
spec.TLS found and noTLS == true  ---> bind // no tls termination at haproxy
spec.TLS ! found and  >1 TCP backend for same port  ---> bind ssl  // SSL nto terminated but SNI used

validate that all matching backends have noTLS = false

  mode tcp
  tcp-request inspect-delay 5s
  tcp-request content accept if { req_ssl_hello_type 1 }

  use_backend app1-servers if { req_ssl_sni -i app1.example.com }
  use_backend app2-servers if { req_ssl_sni -i app2.example.com }
  use_backend app3-servers if { req_ssl_sni -i app3.example.com }

spec.TLS ! found and 1 backend svc for port         ---> bind  // example: ssh, mysql
host:
tcp:
port: <------------
backend:
service:
port:

--------------------------------------
ssl-passthrough =>> no TLS termination (converts to S-2 or S-3, noTLS = true)

host found in spec.TLS && noTLS = false =>>> terminate TLS (S-1)

--------------------------------------

S-1:
host found in spec.TLS && noTLS = false =>>> terminate TLS
#(host) == 1:
#(host) > 1:

(
S-2:
host found in spec.TLS && noTLS = true =>>> Do not terminate TLS
#(host) == 1:
#(host) > 1:

S-3:
host ! found in spec.TLS =>>> Do not terminate TLS
)

--------------------------------------

- Same port, S-1 and (S-2 or S-3) (one host wants to terminate TLS, one host does not want to terminate TLS): **Validation Error**

- #(S-2 or S3) >= 1 and host is set : OK
{
	== 1: use backend

	> 1: use SNI

		mode tcp
		tcp-request inspect-delay 5s
		tcp-request content accept if { req_ssl_hello_type 1 }
		use_backend app1-servers if { req_ssl_sni -i app1.example.com }
		use_backend app2-servers if { req_ssl_sni -i app2.example.com }
		use_backend app3-servers if { req_ssl_sni -i app3.example.com }
}

- #(S-2 or S3) >= 1 and (host is not set for one or more): **Validation Error**

- #(S-1) == 1 and #(S-2 or S-3) == 0  `bind ssl crt path_to_crt_file`
{
	host == "xyz.com"   >>>> OK   `use backend  xyz`

	host == `*.xyz.com` >>>>> OK  `use backend  xyz`

	host == `*`         >>>>>> OK `use backend  xyz`
}

- #(S-1) > 1 and #(S-2 or S-3) == 0 : ? (check)  `bind ssl crt path_to_crt_dir`
{
	host == "xyz.com", "abc.com"

		use_backend app1-servers if { req_ssl_sni -i xyz.com }
		use_backend app2-servers if { req_ssl_sni -i abc.com }

	host == `*.xyz.com`, `*.abc.com`, `hij.com`

		use_backend xyz-servers if { ***_end(req_ssl_sni) -i .xyz.com }
		use_backend abc-servers if { ***_end(req_ssl_sni) -i .abc.com }
		use_backend hij-servers if { req_ssl_sni -i hij.com }

	host == `*` >>>> **Validation Error**
}

- #(S-1) >=1 and ssl-passthrough == **Validation Error**
{
	host == "xyz.com"
	host == `*.xyz.com`
	host == `*`
}


Terminal: problems when pasting multiple lines of text

VSCode Version: Code - Insiders 1.11.0-insider (207686d256dafb78b247357ad8b96a95218a22a5, 2017-03-17T08:14:15.805Z)
OS Version: Windows_NT ia32 10.0.15048
Extensions:

Extension
Author
Version

typings-installer
benjaminromano
0.2.0

vscode-eslint
dbaeumer
1.2.7

theme-dracula
dracula-theme
1.8.0

tslint
eg2
0.8.1

vscode-great-icons
emmanuelbeziat
1.1.44

Theme-azure
gerane
0.0.2

docthis
joelday
0.4.3

theme-material-theme
jprestidge
1.0.1

vs-color-picker
lihui
0.6.0

Theme-MaterialKit
ms-vscode
0.1.1

debugger-for-chrome
msjsdiag
2.7.0

view-in-browser
qinjia
0.0.5

java
redhat
0.0.13

ActiveFileInStatusBar
RoscoP
1.0.2

vscode-hexdump
slevesque
1.2.1

eval
Stormspirit
0.0.6

omnipascal
Wosi
0.13.0

Steps to Reproduce:

open a terminal and run "bash"
run nano foo with a text file containing some lines of text
press ctrl+v (or right-click) to paste some lines of text

What's expected:
The text should be appended before the cursor, and the existing text should be there after the cursor.
What happens:
Existing text disappears, and it seems some extra newlines are added when pasting text.
Note: there is no such problem when doing the exact same steps with CMD.exe
This happens on insider builds, no idea if it happens with standard builds as well.

More information. Here is what happens when I try to paste this text
function start_agent {
    echo "Initialising new SSH agent..."
    /usr/bin/ssh-agent | sed 's/^echo/#echo/' > "${SSH_ENV}"
    echo succeeded
    chmod 600 "${SSH_ENV}"
    . "${SSH_ENV}" > /dev/null
    /usr/bin/ssh-add;
}

In the middle of a file containing this text:
existing file

with some text

@Tyriar Any update on this bug ?

@nico-onmap Thanks for the ping. I've validated it's definitely an upstream issue. I tried to fix it a couple of days ago but didn't get anywhere. I'd guess it's related to node-pty or winpty as different shells appeared to handle it differently. It also seemed somewhat inconsistent.
No plans to invest further in fixing it just yet.

This is probably an xterm bug, not a node-pty or winpty bug.  I placed these two lines in an untitled editor window:
a
b

I then set the environment variable WINPTY_DEBUG=trace,input, started VSCode, and copy-and-pasted the two lines above into VSCode's integrated terminal.  This appeared in the winpty trace:
[55993.086 winpty-agent.exe,p5460,t3560]: input chars: a^M^Jb^M^J (61 0D 0A 62 0D 0A)
[55993.086 winpty-agent.exe,p5460,t3560]: keypress: A ch='a'
[55993.086 winpty-agent.exe,p5460,t3560]: keypress: RETURN ch=0xd
[55993.086 winpty-agent.exe,p5460,t3560]: keypress: LCtrl-RETURN ch=0xa
[55993.086 winpty-agent.exe,p5460,t3560]: keypress: B ch='b'
[55993.086 winpty-agent.exe,p5460,t3560]: keypress: RETURN ch=0xd
[55993.086 winpty-agent.exe,p5460,t3560]: keypress: LCtrl-RETURN ch=0xa

The clipboard has CRLF-delimited lines, because we're on Windows.  However, in the VT/100 input format, CR means Return, and LF means Ctrl-Return.  Something (probably xterm) is copying the clipboard contents into the pty, and that something should normalize LF and CRLF into CR.
I described this bug in detail the last time I saw it: https://github.com/JetBrains/jediterm/issues/136.

@rprichard thanks! This is likely the culprit then which replaces all end of line chars with \r\n on Windows: https://github.com/Microsoft/vscode/blob/84b2a1ecb6a342d7d82605a330581f06cb726fc3/src/vs/workbench/parts/terminal/electron-browser/terminalInstance.ts#L426

Yeah, I think your change will fix the issue.  The sendText function looks like it might also need a change.  Maybe something like this?
	public sendText(text: string, addNewLine: boolean): void {
                text = _sanitizeInput(text);
		if (addNewLine && text.substr(text.length - 1) !== '\r') {
			text += '\r';
		}
		this._process.send({
			event: 'input',
			data: text
		});
	}

FYI @Krzysztof-Cieslak, I think you may have run into these issues before? It would be a good idea to test these changes in ionide-vscode-fsharp in tomorrow's Insiders build.

I've certainly experienced aspects of this issue from the Python extension when running selections of code in the terminal.

@Tyriar I was unable to reproduce the problem anyway, so have no way of testing this. But thanks for letting me know :-)


"Zu weinig Arbeitsspecher" message when loading MSCP driver

I am trying to load the MSCP driver and get the above error message after the driver loads, but before the frame for imaging appears on the read/write images dialog.  It will keep happening when I hit "Reset & open device".
Machine is an 11/23+ with ODT and 1,5MB of RAM

Julian,
this is new.
Can you perform the following test with SimH?

setup a simh machine with MSCP controller
(rq0 in the example script)
Start the "pdp11.exe" of the PDP11GUI distribution, run the
script and do "sim> set console stdio=23"
connect PDP11GUI to "Simh over telnet"
Open the MSCP disk as on the real 11/23+

What OS are you using, and which version of PDP11GUI ?
thanks,
Joerg
Am 24.07.2016 um 02:23 schrieb julianwolfe:

I am trying to load the MSCP driver and get the above error message
after the driver loads, but before the frame for imaging appears on the
read/write images dialog. It will keep happening when I hit "Reset &
open device".
Machine is an 11/23+ with ODT and 1,5MB of RAM
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/j-hoppe/PDP11GUI/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFBnP1x7O5-kCZd0e-WgSmxBozxgbmwSks5qYrBngaJpZM4JTeSz.

; a sample UNIBUS machine, with all disk&tape devices PDP11GUI can copy.
; for all devices:
;	unit 0 is attached to an useful media and is write protected
;       unit 1 is attached to "test" and read/write
; M7554       KDJ11-DA    Q   J11 CPU 15MHz, 512-Kbyte RAM, 2 SLUs, LTC
; M7554		Refs: EK-KDJ1D-UG
set cpu 11/44 4M
set cpu hist=100
set throttle 2m
set clk 50hz
; to make gkermit work
set tti 8b
set tto 8b
;set cons telnet=23
; disable all extra devices by default, will enable later
set ptr disabled
set ptp disabled
set lpt disabled
set rha disabled
set cr disabled
set rp disabled
set rk disabled
set xu disabled
set xq disabled
set rq disabled
set rl disabled
set rx disabled
set ry disabled
set tm disabled
set tq disabled
set ts disabled
set hk disabled
set vh disabled
; DLV11 Nachbau MDB
;M7940       DLV11       Q   Serial Line Unit (SLU, Async)
;M7940		Refs: MP-00055, EB-20175-20, uNOTE N#033
;set ttix lines=1
;attach ttix 2000
;
; M8061       RLV12       Q   RL01/RL02 disk control
; M8061		Refs: EK-RLV12-UG, EK-RLV12-TD, MP-01282, ZJ285-RZ
; M8061		Refs: uNOTE N#041
; unit 0,1 = RL=2, unit 2,3= rl01
set    rl  enabled
set    rl0 rl02
attach rl0 xxdp25.rl02
set    rl0 locked
set    rl1 rl02
attach rl1 test.rl02
set    rl2 rl01
attach rl2 random.rl01
set    rl2 locked
set    rl3 rl01
attach rl3 test.rl01
; M7555       RQDX3       Q   MFM Winchester and floppy disk controller
; M7555                       (RX50/RX33/RD50-54/RD31/RD32/RD33)
; M7555		Refs: EK-RQDX3-UG, uNOTEs N#041, N#043
;
; unit 0,1 hd: seagate st 225 = rd31-a  (25MB)
; unit 2,3  floppy: rx33
set    rq  enabled
set    rq0 rd52
attach rq0 ultrix31.rd52
set    rq0 locked
set    rq1 rd52
attach rq1 test.rd52
set    rq2 rx33
attach rq2 random.rx33
set    rq2 locked
set    rq3 rx33
attach rq3 test.rx33
set    rp  enabled
set    rp0 rm03
attach rp0 211bsd.rm02
set    rp0 locked
set    rp1 rm03
attach rp1 test.rm02
set    rk  enabled
attach rk0 unix0_v6.rk05
set    rk0 locked
attach rk1 test.rk05
set    hk  enabled
set    hk0 rk06
set    hk1 rk07
attach hk0 test.rk06
attach hk1 test.rk07
;set    rx  enabled
;attach rx0 random.rx01
;set    rx0 locked
;attach rx1 test.rx01
set    ry  enabled
set    ry0 double
attach ry0 test.rx02_dd
set    ry0 locked
set    ry1 single
attach ry1 test.rx02_sd
; paper tape
set    ptr enabled
; paper tape basic
attach ptr DEC-11-AJPB-PB.ptap
set    ptp enabled
! del test.ptap
attach ptp test.ptap
; set one DZ11 8 line async mux
; accessible via telnet to <IP_ADDRESS>:4000...4007
set dz enabled
set dz lines=8
set dz 8b
set dz vector=300
attach -am dz 4000
set stdio telnet=23


e6 not working

Hi!
I can't import the e6 module...
Error: Could not find module webstomp-client imported from plhw-client/services/realtime-stomp
npm install webstomp-client --save-dev
ember serve

// services/realtime-stomp.js
import webstomp from 'webstomp-client';

...

Works fine for me. Why are you saving in --save-dev mode?

@basz sorry to read that but it works for me too.

nvm. solved it somehow. can't remember what went wrong. thx


Question about the chosen of language code

I noticed that when using get_tasks(), the language code of C-MTEB tasks is cmn but not zho. However the definition classes of those tasks are in the 'zho' directories, which confused me a little at the beginning because I cannot get those tasks by specifying languages='zho'.
So I'm wandering how is this designed. Appreciate any idea, discussion, or explanation!

Based on git blame, these were mapped to "Mandarin Chinese" (cmn), whereas zho maps to "Chinese". I don't see an issue aligning the lang codes in the tasks to the folder name, i.e. updating all relevant lang codes to zho.
@ZiyiXia thanks for catching this. Feel free to open a PR!
CC @KennethEnevoldsen just in case I missed something obvious or there was a reason I wasn't aware of.

Got it. Keeping cmn now make sense to me. Happy to discuss in the future when there's demand to align the lang codes. Thanks!

Sounds good. I'll close this for now, but feel free to reopen whenever this topic comes back.

I believe "cmn" is a more specific language than "zho", which also refers to "cmn", "yue".

I would accept either "cmn" and "yue" OR "zho," but I would probably not use both. Would love the opinion of a Chinese speaker on this as I don't have much experience on the differences.


Pre-requirements not fullfilled

Describe the issue
We are trying to activated the Contracts Management form here: https://lookbook.microsoft.com/details/a8fc15e1-2bbe-43b9-98a0-99d3a997589c
Expected behavior
We configured the Microsoft Snytax Pay-Go as descriped at the documentation https://learn.microsoft.com/en-us/microsoft-365/syntex/syntex-azure-billing
Screenshots

Details (please complete the following information):

PnP Correlation ID: [Provide a correlation ID for the provisioning action]
Reference Template: https://lookbook.microsoft.com/details/a8fc15e1-2bbe-43b9-98a0-99d3a997589c
Primary language of the target tenant: english
Are you using a Tenant Global Admin account to provision the template: Yes

Additional context

Hi, Thanks for using the Look Book site. About your issue, you should move to the out of the box templates available in the product. We just disabled the templates for Syntex in the Look Book site.

Where can i find this out of the box templates?

Hi @PaoloPia where can I find this out of the box templates?


updating required :

Module compiled with Swift 4.0 cannot be imported in Swift 4.1:

@mohammadkalim Have you got any updates on this issues "Module compiled with Swift 4.0.2 cannot be imported in Swift 4.1:"

We have updated our SDK. Please update the pod and try again. If you still face the problem please get back to us.


Additional annotation for xanthine (CHEBI:15318)

It would be useful if the below details could be added to the xanthine (CHEBI:15318) entry.
A has functional parent relationship with guanine (CHEBI:16235) should be added.
Chemical details, including synonyms (2,6-dioxopurine) and SMILES can be imported from its PubChem entry: https://pubchem.ncbi.nlm.nih.gov/compound/1188.
The below reference could be added:
T Nguyen, D Brunson, C L Crespi, B W Penman, J S Wishnok, S R Tannenbaum. DNA damage and mutation in human cells exposed to nitric oxide in vitro Proceedings of the National Academy of Sciences Apr 1992, 89 (7) 3030-3034; DOI: 10.1073/pnas.89.7.3030.

I have added the guanine functional parent relationship, synonym (2,6-dioxopurine) and the reference provided to this entry. Please be aware that the xanthine (CHEBI:15318) entry is a chemical class with no definitive structure hence a SMILES or InChi cannot be provided. Several different tautomers of xanthine can exist such as 7H-xanthine (CHEBI:48517) and 9H-xanthine (CHEBI:17712) which belong to this class.


As a user I want to see when a news article was last modified so that I can know if the article has been updated

Acceptance criteria

[x] news article shows the last modified date alongside the publish date

In order to ensure good quality we should (Keep the ones we need for this US)

[ ] Perform integration testing
[ ] Perform functional testing
[ ] Extra focus on usability
[ ] Test for portability
[ ] Test for accessibility
[ ] Create automated acceptance test/smoke test
[ ] Perform performance and load test
[ ] Perform user testing
[ ] Perform security testing

Testing, testing, testing, this is the QA department speaking!
We are sorry to inform you @SvSven  that this did not, repeat DID NOT, pass the QA department with regard to small devices.

We know you can do better than this! ✺◟( ͡° ͜ʖ ͡°)◞✺

Good to see that my test of the QA process was a success! (☞ﾟヮﾟ)☞

I have included 'This is a testing page' into the visual tests. The tests include screen width 375 and 1920.


How do I know when tap is done?

UPDATE: The answer to the question is this: Call tap.end() after you've loaded all the tests via tap.test(), and then catch completion in the callback for tap.tearDown(cb).
I'm running tap in a forked process. I have to call process.exit(0) because a node-forked process won't terminate on its own. But I can't figure out when to do this. Neither the 'complete' event nor the 'end' event is called to report completion. My setImmediate() function is called before most of the tests run. I'm piping the tap output to my own Writable, but I can't figure out how to detect completion even without piping. (And end() is not called on my Writable.)
How do I learn that tap is done so I can call process.exit()? Thanks for your help!

HI @isaacs. I don't have a node-tap test script for you, but you can look over the code in my 'fork-tests' branch. I've been exploring variations without success. subtap.js forks _runfile.js. These are the two relevant files.

I thought to write a unit test for you but realized that I have no idea what should signal completion. Which of the following should it be? Or what else?

tap.on('complete', theEnd)
tap.on('end', theEnd)
tap.pipe(writable).end()
tap.pipe(writable).on('finish', theEnd)
setImmediate(theEnd)

I can't wait on process.on('beforeExit') or process.on('exit') because those events do not occur in a node-forked process until process.exit() is explicitly called.

Maybe tap needs an idle event to signal that no tap-related work is currently pending. Tap can't know whether another test file is about to be loaded, but it should know whether it's presently waiting on anything.
Is there any way for me to get the size of the next-tick queue?

Well, I've monkey-patched tap to count outstanding tests and exit when all have ended. I don't know how generally it works, because I reverse-engineered more from behavior than from source. (I have poor eyesight at 47 and can't track two-space indentations well.) I've tested it for explicit calls to t.end(), for deferred plans, and for returned promises. Here's the relevant code.

Drats. I think I see a much easier way -- watching the TAP output. tap knows when it's hit the end and says so. I just have to look for that mark in the output.

So, before you go reinventing wheels, it seems like this project is reproducing a lot of stuff that's already in node-tap, in the sense that node-tap already has a bin that spawns child processes and can present output as nested tests.  (Try running tap test/*.js --reporter=tap.)  Instead of using ICP to communicate bailout and completion, it just looks at the tap output.  There's even a builtin function, tap.spawn, which will run a child test as a sub-process and handle its output.
If your goal is to run subtests that have ICP capabilities, you can do tap.spawn(process.execFile, [theTestFile], { stdio: [ 'pipe', 'pipe', 'pipe', 'ipc' ] }), but I don't know offhand if there's a way to get at the child process object.  It wouldn't be hard to add support for that, maybe a callback that gets the child process object as an argument once it's set up and you're at that point in the test set.
If you are dead set on going your own way (which is totally fine, allowed, and can be a source of much goodness), then take a look at what the code in tap's bin/run.js is doing.
The reason that tap's bin runs tests as a standalone child proc, rather than using require() is that it ensures that the test can be run identically with the bin runner or by typing node test.js, with a "fresh" environment where the test itself is the require.main entry point.  This also obviates the need for a "when to close" indication, because Node closes when the event loop is empty, and the exit handler in lib/root.js prints out the final stuff.
One option, if you're going to load the test into the current process, is to use tap.tearDown(someFunction), which will require tap to exit once there's nothing pending.  (Otherwise it would be impossible to do things like shut down a server at the end of your test run.)  So you could just have your child proc do tap.tearDown(function () { process.exit() }).  No monkey-patching necessary, it does exactly what you're describing here, and is supported and tested already :)  http://www.node-tap.org/api/#tteardownfunction

Thanks @isaacs. I already have a simple solution in the works. I need IPC because I'm communicating more than bailout and completion. My alternative was to add TAP comments to the stream and read data from them.
I was also making a mistake by not calling tap.end() when I knew I wouldn't be queuing any more tests. If I do that, I get tap 'complete' and 'end' events, though I'm yet sure of the difference.
tap.tearDown() and the various emitted events don't seem to be documented outside the source, and I'm having trouble reading indentation in the source, so I'm kinda going blind here.
Thanks for your help. It looks like I now have several solutions.

Yes, calling tap.end() explicitly is also an option.
The public API (including the tearDown function) is documented at http://www.node-tap.org/api/
What are you finding is missing from the docs that would be helpful?  I don't have docs for all emitted events, because that is an internal API that isn't really supported publicly.  Apart from the standard Node.js readable stream interface, it only emits result, complete, and bailout events, which are likely what you'd expect.
As far as the indentation, I use the standard linter on this project, which is very popular, so it is probably worthwhile to get familiar with it, since you'll surely encounter it a lot.

Woo! It works, no monkey patching, and it's simple. Thank you!
Looking through the docs now, I see that all the information I needed was there. It's hard to retrospectively know what threw me, but I'm guessing:

Virtually no examples show .end() being called on tap itself, even though it makes sense because the docs clearly state that the module returns a Test object.
The tearDown() docs say "Run the supplied function when t.end() is called", which (as stated) is not quite the behavior I want. I know I can't call process.exit() in place of my tap.end(). It seems that tearDown() instead runs when all outstanding tests have completed.

Even so, I see now that there was enough information in the docs to have suggested that I experiment with this solution. The docs are pretty darn good.
Thank you again for all the help you've given me!

Awesome!  Yeah, I guess it should be clearer that it's not "when t.end() is called" but rather, "when the test ends".  (It'll also run if you use a plan, for example, or set autoend: true.)  I'll update the docs to correct that.
Hey, one thing that jumps out at me, is that you're doing process.send(...) and then immediately calling process.exit(0).  I think that's a race condition, since IPC is asynchronous.  It's probably hard to see in simple cases, but if you had a lot of stuff going on in that channel, it might result in the child process exiting before the message has been fully written to the channel.  It'd be safer to have the parent process send a reply indicating that it got the message, or even just have the parent process kill the child process when the message is received.

Thanks for that tip. I should have caught that. But tap has an on-exit handler that kicks in after the kill signal, and that handler is not happy:
/Users/joe/repos/qloader/node_modules/tap/lib/test.js:321
      throw er
      ^

Error: test after end() was called
    at Test._onTimeout (/Users/joe/repos/qloader/node_modules/tap/lib/test.js:177:5)
    at EventEmitter.<anonymous> (/Users/joe/repos/qloader/node_modules/tap/lib/root.js:125:7)
    at emitTwo (events.js:106:13)
    at EventEmitter.emit (events.js:191:7)
    at emit (/Users/joe/repos/qloader/node_modules/signal-exit/index.js:68:11)
    at process.listener (/Users/joe/repos/qloader/node_modules/signal-exit/index.js:82:7)
    at emitNone (events.js:86:13)
    at process.emit (events.js:185:7)
    at processEmit (/Users/joe/repos/qloader/node_modules/signal-exit/index.js:146:32)
    at Signal.wrap.onsignal (internal/process.js:201:44)

So kill it with SIGKILL, rather than SIGTERM.  (SIGTERM is for timeouts, so tap reads that as a test timing out, but still, it looks like you may have found a bug.)

Thank you. SIGKILL did the trick.

My new tool subtap breaks going from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>It appears that root-level assertions run in my fork but that subtests aren't allowed to run. It's probably something I'm doing in appropriately and I'll continue to look at it.

This is the only code that seems relevant to the problem, in root.js:
  if (!tap._ended) {
    tap._onTimeout(extra)
  } else {
    console.error('possible timeout: SIGTERM received after tap end')
    if (extra.handles || extra.requests) {
      delete extra.signal
      if (!extra.at) {
        delete extra.at
      }
      var yaml = require('js-yaml')
      console.error('  ---\n  ' +
               yaml.safeDump(extra).split('\n').join('\n  ').trim() +
               '\n  ...\n')
    }
    process.exit(1)
  }

My first real clue: the TAP output under version 7.0.0 indents root-level subtest names with four spaces, while before version 7.0.0, these names are not indented. When I run the test file directly with node, the indentation is correct. The problem has something to do with forking or my monkey patch.

CORRECTION! I had that backwards. tap 7.0.0 is outputting subtest names on the left margin. tap-parser does not like that. It has nothing to do with my software.

Problem solved. You also updated tap-parser. If I update tap-parser too, my tool works with 7.0.0.
Unfortunately, my tool is designed to work with the user's existing installation of tap. In order for people to be able to use my tool, I'll have to force them to upgrade tap to 7.0.0. I'm not sure how reasonable that is to ask of a test harness.

@jtlapp Just use the latest tap-parser. It properly handles both the indented and unindented child test formats, and should be 99% backwards compatible except in some very rare edge cases. It also handles "buffered" subtest formats that the Test2 cpan module uses.


int32_publisher not building for esp32c3

Hardware description: ESP32C3-Wroom-02
RTOS: freertos
Installation type: Standard
Version or commit hash: Humble

Steps to reproduce the issue
In the terminal:
1)Select int32_publisher example
2) idf.py set-target esp32c3
3)idf.py menuconfig (I used Ethernet as Transport)
4)idf.py build
Expected behavior
100% successful build
Actual behavior

Additional information

same problem

@Apurva2510 Please write log-files, or better copy-paste into
code blocks

for others to

find the problem via Google. Google indexes text not images.
to read.


how to handle multiple generated forms

I am trying to generate particular form for a set of words. Mostly everything produces good results, but sometimes there are two results, such as here:
uralicApi.generate("rakennusteline+N+Pl+Ela", "fin")
[('rakennustelinehistä', 0.0), ('rakennustelineistä', 0.0)]

It appears the first result is in some local dialect, and the second form is more correct literary form. Why is this? Is there a way to limit the output to just literary form?

Either open an issue in the Finnish transducer in https://github.com/giellalt/lang-fin or I am sure we can help through Rootroo for a fee https://rootroo.com


useAnimationReverse stores wrong prefers-reduced-motion setting

On first render window matches false, which is initially stored in local storage.
On second render, window matches true, but is not stored in local storage, meaning "allowAnimations" in useAnimationReverse will always be false
Also instead of throwing an error, maybe reject the promise instead, to avoid try/catching or errors in console.

This was (hopefully) fixed in 1.3.4


bump version to 0.20.3-alpha for development

Changes Unknown when pulling 6f62e0fae46c8b402b56c8664ce652c91dd4aadf on neozwu:bump-version-for-dev into ** on GoogleCloudPlatform:master**.


Protection against SQL-injection

Issue AmpersandTarski/Ampersand#271 claims that the generated web-application is not protected against SQL-injection.
Since the application is generated, an effective way to do this is to escape all input fields. The Ampersand-application should use ANSI-SQL all the way, so the (simple) ANSI escape mechanism is sufficient for this purpose: Simply encode all ' (single tick) characters with '' (two single ticks).
(see also https://www.owasp.org/index.php/SQL_Injection_Prevention_Cheat_Sheet)

Our prototypes are protected against malicious user input. Everything is escaped. We can do better with SQL Prepared statements, but for now we're good.


For the pkl CLI, please provide an installer package for Homebrew

Subject says all.
Homebrew (https://brew.sh/) is a very popular package manager for macOS where you can find almost any tool you might ever want to install.

We already started the process of releasing Pkl to homebrew. Should come soon.

Submitted here: https://github.com/Homebrew/homebrew-core/pull/161655
Looks like we need to re-submit as a cask.

I don't think cask is the right approach, something like the below should work
class Pkl < Formula
  desc "Configuration as code language with rich validation and tooling"
  homepage "https://pkl-lang.org"
  url "https://github.com/apple/pkl/archive/refs/tags/0.25.1.tar.gz"
  sha256 "f60412679a9a8a1740e81cbed89a3ca9ddc9aa2cf0c487ff8a8a9fce70c0bf4a"
  license "Apache-2.0"

  depends_on "openjdk@17" => :build

  def install
    ENV["JAVA_HOME"] =<EMAIL_ADDRESS>    task_os = OS.mac? ? "mac" : "linux"
    task_arch = Hardware::CPU.arm? ? "Aarch64" : "Amd64"
    task_name = ":pkl-cli:#{task_os}Executable#{task_arch}"

    system "git", "apply", "patches/graalVm23.patch" if OS.mac? && Hardware::CPU.arm?

    system "./gradlew", task_name
    os = OS.mac? ? "macos" : "linux"
    arch = Hardware::CPU.arm? ? "aarch64" : "amd64"
    bin.install "pkl-cli/build/executable/pkl-#{os}-#{arch}" => "pkl"
  end
end

However, the problem I am having is with homebrew's shimmed cc. It is expecting the environment variable HOMEBREW_RUBY_PATH to be present, and something is wiping out that value in the task process before the cc invocation. I tried to see where it was getting removed from the environment but was unable to figure it out

It looks like the submission has been stalled on the Homebrew side, is that the status?

It looks like the submission has been stalled on the Homebrew side, is that the status?

there is a new submission https://github.com/Homebrew/homebrew-core/pull/162857

@pavelzw It looks like you (we) are just waiting for someone to approve the merge?

Yeah, right now we are waiting for a homebrew maintainer to approve and merge the PR.

This is now available on homebrew! brew install pkl will install the CLI.
We are adding instructions to our website in https://github.com/apple/pkl/pull/270.


Metrics for vmq webhook

Environment

VerneMQ Version: 1.12
Runtime: Docker

We are missing metrics about how vmq webhook pools are used, the response latency from webhook endpoint. It will be helpful to have these metrics exposed so that we can fine tune the setup.

Implemented by https://github.com/vernemq/vernemq/pull/1991


Added not_allow_values rule

Introduced a new NotAllowValues rule that indicates the set of values which are not allowed. This rule was added to full.json, full.php, and full.yml files. Accommodating this new rule required the creation of corresponding classes and test files. The enhancement provides a more robust and configurable data validation effort.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

5 of 6 (83.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 94.296%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Rules/Cell/NotAllowValues.php
5
6
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
777

Relevant Lines:
824

💛  - Coveralls


Update coveralls dep to fix warning

Coveralls relies on request which is using an outdated dep, once it's updated this repo should update it's Coveralls version.
Ref: https://github.com/nickmerwin/node-coveralls/issues/149

Seems fine to me.  Patch welcome.

Seems fine to me.  Patch welcome.

(a) Coveralls is still using request4.5 years after this issue was posted!
(b) Coveralls most recent update was more than a year ago (May 1, 2020).
(c) Coveralls has a fork that is removes request and is being maintained slightly more actively (with their most recent update being Jan 18, 2021).
So, I'd love to see this issue re-opened with the goal of replacing the unmaintained and clearly out-of-date coveralls library with coveralls-next (https://github.com/jtwebman/coveralls-next).


BPF map dump_interrupts should not be considerd as cilium_datapath_errors

A user got confused by the following metric:
# HELP cilium_datapath_errors_total Number of errors that occurred in the datapath or datapath management
# TYPE cilium_datapath_errors_total counter
cilium_datapath_errors_total{area="conntrack",family="ipv4",name="dump_interrupts"} 515950

dump_interrupts happens when a BPF entry gets removed while dumping the map is in progress. This is not an error, thus we should move it to some other counter which doesn't have error in its name.

Having many of this (~400K) in my dashboard.

bump maybe

@brb regarding the DatapathErrors counter, it seems to be used only for the dump_interrupts.
Wouldn't it be simpler to rename DatapathErrors and all the string prints by removing the error part? Subsystem will stay the same.
Also, I can take care of this one.

@brb regarding the DatapathErrors counter, it seems to be used only for the dump_interrupts.
Wouldn't it be simpler to rename DatapathErrors and all the string prints by removing the error part? Subsystem will stay the same.
Also, I can take care of this one.

@youssefazrak Your suggestion makes sense. Just make sure that we don't forget to document the metrics name in the upgrade guide.

@youssefazrak Your suggestion makes sense. Just make sure that we don't forget to document the metrics name in the upgrade guide.


Use emptyDir to store bundle content rather than configmaps

Signed-off-by: Joe Lanford<EMAIL_ADDRESS>Closes https://github.com/operator-framework/rukpak/issues/296

invalidates a store key when we intercept the deletion timestamp

I'd agree that a finalizer-free implementation would be nicer.
I figure there's a decent possibility we'd miss deletion events and we'd slowly accrue zombie files. I suppose we could solve that by running a goroutine that periodically re-lists from the cache and deletes bundle files for anything not found in the cache.
Let me give that a quick spin.

I figure there's a decent possibility we'd miss deletion events and we'd slowly accrue zombie files. I suppose we could solve that by running a goroutine that periodically re-lists from the cache and deletes bundle files for anything not found in the cache.

Something else we may need to consider is a partial cache write where we encounter an error while storing objects, but I think this is only a problem with the bundle controller right now, and it's not entirely clear to me how you'd be able to do this without introducing extra complexity (maybe through channels/signals/etc.?)

I suppose we could solve that by running a goroutine that periodically re-lists from the cache and deletes bundle files for anything not found in the cache.

I looked into this a bit and realized that we'd have to expand the storage.Storage interface to include a List or Reap or Prune method. And that implies that a storage implementation would have to be able to list all of the files. For a mounted filesystem implementation, that's fairly cheap, but for something like S3, that would require at least 1 extra API call (maybe more depending on pagination requirements) per whatever the polling interval is.
I'm tempted to leave the finalizer in place for now to keep the storage interface minimal, and then revisit this question separately. Thoughts on that?

Does it make sense to track whether we need to re-evaluate our storage interface down the line?

Yeah I'll make some new tracking issues that cover:

Are we okay with using finalizers; what are the alternatives and pros/cons?
Are we okay with the storage interface? Should we keep it minimal? If it expands what are the implications?


Add a stroke parameter to Editor.text

Here is an example by editing welcome_image2.py.
Before
background.text(
    (400, 260), "WELCOME", color="white", font=poppins, align="center"
)

After
background.text(
    (400, 260), "WELCOME", color="white", font=poppins, align="center", stroke=2
)

Can you update it to stroke_width because other methods that implement stroke has it named stroke_width so it will be match for all

Actually, it's not stroke_width only. It's also about the color. Currently I have it so that you pass a tuple (width, color) to stroke (and if you pass only a integer, it is interpreted as (width, "black"). Do you want them to be separated like stroke_width and stroke_fill (as it is in PIL)?

OK. What about now?


Existing wallet cannot be imported

Existing wallet can not be imported. When will this be added?

Closing this one in favour of https://github.com/pradel/blockstack-wallet/issues/36
I am planning to add this next week.

Closing this one in favour of https://github.com/pradel/blockstack-wallet/issues/36
I am planning to add this next week.

Hey @vysegrad you can now download the latest version where you can import a wallet and that support mainnet. https://github.com/pradel/blockstack-wallet/releases/tag/v0.5.0

Hey @vysegrad you can now download the latest version where you can import a wallet and that support mainnet. https://github.com/pradel/blockstack-wallet/releases/tag/v0.5.0


Problem adding a youtube channel to my profile

First of all, sorry if this isn't the place, or if this already has a solution.
I wanted to add my youtube account to my profile.
It requests to write only my ID/username, but later I found that the web puts that id/name after a "https://www.youtube.com/user/[xxx]" url.
My problem is that (if i'm not wrong) the newest accounts, as mine, instead of that order, they use a "https://www.youtube.com/c/[xxx]" url, from channel, so if you check the link from my profile it goes to nowhere, not my profile, because of that "/user/" and "/c/" difference.
Could be any option or solution to this?

Do you know if the old accounts also work with the second URL? In that case I can just replace the URL.

Maybe the option will be to have : "https://www.youtube.com/username", and youtube just use the correct url I think, I have two old accounts with "/user/", someone can confirm it work? ;-)

"Do you know if the old accounts also work with the second URL? In that case I can just replace the URL."
No, they don't. :/ It needs to be exactly the correct URL. I checked different famous and older channels and it must be /user/ tag in that case.
Maybe the only way to resolve this is allowing to paste the entire URL about Youtube, or having different options when you select the plataform in our settings (Youtube/c/ and Youtube/user/ or something like that), but I don't know.
Btw, thanks for answering. Loving the site.

I think the best solution that doesn't change the database too much is to have 2 different kind of links:
YouTube (created after )
YouTube (older)
What do you think?
@Plusmine I'm glad you like it, thanks for using it :)

"""I think the best solution that doesn't change the database too much is to have 2 different kind of links:
YouTube (created after )
YouTube (older)
What do you think?"""
Perfect, too. If you can do that, it could work.
I didn't think about it, and thanks!

I think the simpliest solution si to just enter our username number, it will be easier and take less place to database too, more benefits !

"In case of: I suggested the "https://www.youtube.com/username" for tthe database use, and it simply redirect, I don't find another way more easy than that IMHO. ^^"
I just tried this, and yep, it works too!
Thanks, YoannGo.


fix(heni):update url and endpoint

Closes #ISSUE_NUMBER_GOES_HERE
Description
......
Changes

High level
changes that
you made

Steps to Test

Steps
to
test

Quality Assurance

[ ] If a new adapter was made, or an existing one was modified so that its environment variables have changed, update the relevant <ADAPTER_PACKAGE>/schemas/env.json and <ADAPTER_PACKAGE>/README.md
[ ] If a new adapter was made, or an existing one was modified so that its environment variables have changed, update the relevant infra-k8s configuration file.
[ ] The branch naming follows git flow (feature/x, chore/x, release/x, hotfix/x, fix/x) or is created from Clubhouse/Shortcut
[ ] This is related to a maximum of one Clubhouse/Shortcut story or GitHub issue
[ ] Types are safe (avoid TypeScript/TSLint features like any and disable, instead use more specific types)

I updated the DEFAULT_BASE_URL with the new one which does not require a collection name. So I had to remove that param completely and amend some changes. I also removed the integrations test since I'm working on the nock implementation instead. While trying to run tests, I was facing an error `Test suite failed to run
Cannot find module '@chainlink/ea-bootstrap' from '/Users/marcghannam/external-adapters-js/packages/sources/heni/test/unit'`

Not sure if I'm missing dependencies.


How can I use it in vim?

How can I use it in vim?
If not, is there plan to support vim?

I'd be happy to have a Vim client. This looks to be the most popular plugin for creating LSP clients in Vim. Seeing as I don't use Vim myself I don't think I'm the right person to implement the client though. I'd be happy to accept a PR that introduces Vim support 😊

vim-lsp is quite good.

Fixed by @JakobGM in #19


Use updatecli@main config as a fallback

Sets the fallback for check-spelling for when it can't find its configuration.
Test
To test this pull request, you can run the following commands:
Refresh a currently unhappy PR (e.g. https://github.com/updatecli/updatecli/pull/1319)
Additional Information
When someone creates a PR from before check-spelling (v0.0.21) has been added, check-spelling doesn't have access to its configuration (unless you tell it what fallback to use). Using the main branch is a pretty sound fallback (any future branches for releases will almost certainly be long past the addition of check-spelling), and I don't expect someone to be creating a PR from before the addition of check-spelling into a branch with check-spelling.
Tradeoff
It looks a bit odd
Potential improvement
I should have caught that the value wasn't quite right, sorry, I'm human :).
In a future version of check spelling, I hope that check-spelling won't be confused and lose track of its configuration. I can't remember if I've fixed this, but I'll definitely make an effort to ensure the behavior is reasonable.

I should have caught that the value wasn't quite right, sorry, I'm human :).
Likewise

Thank you for the contribution


Environment variable fixes

Allows environment variables to be used in custom components in development mode, loads .env file in cli and keeps VITE_ prefixed variables.  Fixes #2550

Patch to fix #2550

This looks good to me, if you can add a changeset, we can get this merged and released!

Thanks @owenmorris ! This will be released today or tomorrow


[CELEBORN-1328] Introduce ActiveSlotsCount metric to monitor the number of active slots

What changes were proposed in this pull request?
Introduce ActiveSlots metric to represent the disk resource demand currently in the cluster.
Why are the changes needed?
It's recommended to introduce ActiveSlots metric to represent the disk resource demand currently in the cluster.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
In our test cluster (we can see the value of activeSlots increases and then back to 0 after the application finished, and slotsAllocated is increasing all the way).

@CodingCat, I don't think that we need to decrease the SlotsAllocated. SlotsAllocated only represents the allocated slots, not the state of slots. BTW, what's the expr of this metric? Is the expr``increase(metrics_SlotsAllocated_Count[1h])?

@CodingCat, I don't think that we need to decrease the SlotsAllocated. SlotsAllocated only represents the allocated slots, not the state of slots. BTW, what's the expr of this metric? Is the expr``increase(metrics_SlotsAllocated_Count[1h])?

I mean what type of information we are expecting to get from the number of allocated slots ? I don't think it helps with anything for operations with an only increasing number...
what do you mean by expr?

@CodingCat, you could refer to grafana/celeborn-dashboard.json with increase(metrics_SlotsAllocated_Count[1h]) expr.

@CodingCat, I suggest that you could introduce ActiveSlots metric to represent the number of active slots instead of decreasing the SlotAllocated metric. IMO, the SlotAllocated metric should not decrease for releasing slots, but the ActiveSlots could. WDYT? cc @FMX, @onebox-li.

Thanks for the mention, +1 for a new ActiveSlots metrics. I think the current SlotAllocated metrics is also useful to observe the cluster slots' distribution. FYI (the history change) https://github.com/apache/incubator-celeborn/pull/437.

@onebox-li @SteNicholas thanks for the suggestion! updated

@CodingCat, could you update the test picture of this pull request?

@CodingCat, could you update the test picture of this pull request?

sure, updated

@SteNicholas would you please give another review of this?

updated

@CodingCat, thanks for the updates. Could you also update monitoring.md, METRICS.md and celeborn-dashboard.json?

@CodingCat, thanks for the updates. Could you also update monitoring.md, METRICS.md and celeborn-dashboard.json? cc @FMX.

updated, for METRICS.md, are you referring to configuration/metrics.md? that seems to be an abandoned doc?

@CodingCat, I just mean the METRICS.md file to document the metrics.

@CodingCat, I suggest that you could introduce ActiveSlots metric to represent the number of active slots instead of decreasing the SlotAllocated metric. IMO, the SlotAllocated metric should not decrease for releasing slots, but the ActiveSlots could. WDYT? cc @FMX, @onebox-li.

I agree with you.


pyglider additions, adding argopandas as submodule

I'm confused why we would add argo stuff to this repo?

Hi Jody,
I am not completely sure if adding it as a submodule in this way is needed, but I am using the argo package to extract the argo trajectories for the home page map, and adding it as a subdirectory within pyglider is what worked best for me.
Best,
Julia Putko

On Apr 28, 2022, at 1:58 PM, Jody Klymak @.***> wrote:
I'm confused why we would add argo stuff to this repo?
—
Reply to this email directly, view it on GitHub https://github.com/c-proof/pyglider/pull/51#issuecomment-1112648464, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATOOXHZLSD6EXMQXZ7OPTKTVHL3XLANCNFSM5UTI7HFQ.
You are receiving this because you authored the thread.


Add shim READMEs to service subfolders pointing to the documentation site

Description

We've moved most of our documentation to the documentation site (documentation/ and docs.openverse.org), however we've historically had README.md files at the following top level folders:

catalog: https://docs.openverse.org/catalog/index.html
api: https://docs.openverse.org/api/index.html
ingestion_server: https://docs.openverse.org/ingestion_server/index.html
frontend: https://docs.openverse.org/frontend/index.html

We should add a shim README which points to the documentation page for each service (also linked above).
Additional context

Related to #3204

Hi i will like to contribute to this issue please assign me. Thanks

Sure @AlajeBash, please take this up, thanks!


docadd krb and some ertificates FAQs

Proposed changes
add some security docs

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


TTC downstream task

Hi there,
do you plan to release code (and data, or example data) for the table type classification downstream task?
Thanks!

Hello @HaoAreYuDong
Can table-type classification runner files be made available?
Regards,
Harsha Kokel


Add ConsoleLink when running on OpenShift

This is a bit more complicated than you would expect because we're (currently) a namespace-scoped operator and the ConsoleLink CRD is cluster-scoped. This means we have to use ClusterRole/Bindings, and cannot rely on Kubernetes GC to remove the link when the operator is deleted.
The Makefile is modified to query the current context's namespace, and substitutes that into the ClusterRoleBinding in order to locate the ServiceAccount we create. The version of the OpenShift API is bumped as well since the version currently pulled in doesn't have support for ConsoleLinks in the namespace dashboard area. In the Administrator view, you should see the link appear when navigating to the ~Home -> Overview~ Home -> Project -> (project name) pane on a namespace that has the operator deployed. In the Developer view, you can find it in the Project tab.
Since ConsoleLinks are cluster-scoped, I used the GenerateName option to avoid potential naming conflicts with other namespaces. The mapping between Container JFR instances and ConsoleLinks is done using a pair of labels. In order to delete the ConsoleLink when the ContainerJFR CR is deleted, I added a finalizer. This works similarly to how we use finalizers for Recordings. I also did a bit of refactoring to avoid duplication of finalizer-related functionality.
Fixes: #140

When I make image it modifies deploy/crds/rhjmc.redhat.com_containerjfrs_crd.yaml (it removes line 13 & 14), not sure if that's because of some difference on my machine or if it should be changed.

When I make image it modifies deploy/crds/rhjmc.redhat.com_containerjfrs_crd.yaml (it removes line 13 & 14), not sure if that's because of some difference on my machine or if it should be changed.

I think this is an unrelated side-effect of the Operator SDK upgrade.

Ok, no prooblem, it looks good, works as expected!


Installed SQLite to test database connection

Uninstalled MySQL and followed tutorial for installing and configuring SQLite on http://www.tutorialspoint.com/sqlite/sqlite_perl.htm.
Created a test program to set up a connection to the SQLite database.

Upon testing connection received the following error:
DBD-mysql installed ,but still error “install_driver(mysql) failed: Can't locate DBD/mysql.pm in @INC”

Found suggestion that the database wasn't fully installed, http://stackoverflow.com/questions/17144583/dbd-mysql-installed-but-still-error-install-drivermysql-failed-cant-locate.
ran the following commands:
cpan DBD::mysql
perl Makefile.PL
make
make test
make install

Currently receiving another error:
install_driver(SQLite) failed: Can't locate DBD/SQLite.pm in @INC (you may need to install the DBD::SQLite module) (@INC contains: /etc/perl /usr/local/lib/perl/5.18.2 /usr/local/share/perl/5.18.2 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.18 /usr/share/perl/5.18 /usr/local/lib/site_perl .) at (eval 6) line 3.
Perhaps the DBD::SQLite perl module hasn't been fully installed,
or perhaps the capitalisation of 'SQLite' isn't right.
Available drivers: DBM, ExampleP, File, Gofer, Proxy, Sponge.
at test.pl line 53.

I get error when I try to run it in the command line.


Ankara Yildirim Beyazit University added

Institution/School Name
Ankara Yıldırım Beyazıt Üniversitesi
Instiution/School Website
https://aybu.edu.tr
Email Domains
aybu.edu.tr, ybu.edu.tr
Email Users
Please place an x between the square brackets if yes

[X] Students
[X] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[X] Graduate School
[X] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@Bariskau I am sorry, the domain ybu.edu.tr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Support floatingLabelText or hintText property

Subj.
Material UI DatePicker and TimePicker have such a nice property - it's rather painful to lose it.

:tada: Fixed in 0.1.9

Cool, thx!


[ENH] n_jobs/_n_jobs, parameter in classifiers

Describe the feature or idea you want to propose
so currently many sub classes of BaseClassifier have n_jobs as a constructor parameter, declared in onstructor and all have cut and paste docstring. BaseClassifier has a parameter _n_jobs, only ever set to 1 and used in default fit_predict. This is all a legacy of some bad code I wrote a few years ago I fear.
A complexity of this is that n_jobs needs to but mutable to stop algorithms that use prange automatically using all threads.
Describe your proposed solution
So I think n_jobs should just be a BaseClassifier parameter (and hence a single docstring), pass it up through super init, then the classes that need to change it declare a copy in _n_jobs locally and alter that.
 However, with the general discussion about how to handle different backends, it might be got to resolve it
Additional context
No response

May be related to #1797


Wire up related links view from reaction

Along with https://github.com/artsy/reaction/pull/3020, this PR fixes PURCHASE-1612.
It migrates the related links over to reaction and eagerly renders it so that it'll show up in our SEO results.

Important to know that it's currently not being stitched in the traditional way. There is the possibility that this implementation will still take too long to render and not show up in our SEO results. I based this change on the fact that the artwork grid (which is rendered in the same way) is showing up in our SEO results.
If reviewers think it necessary, I can move to a more traditional stitched approach which would ensure SSR though.
Note: CI for this PR will fail until https://github.com/artsy/reaction/pull/3020 is merged.

Interesting that the artist grid is showing up in our SEO results, but the previous artist/gene links (which were also client-side rendered) were not. Do you have an idea as to why that is?

I'm uncertain. It could be a timing issue. It's hard to say though.

@zephraph Heads up the reaction-side has been merged/released!
I think this is good to go. Feel free to merge when 🍏.


Tracking Issue: RKE1 Airgap clusters require additional documentation for vSphere node driver

The error reported:
Error creating machine: RHEL ssh command error: command: sudo -E yum
install -y curl err: exit status 1 output: Loaded plugins:
fastestmirror, langpacks

this error originates from: https://github.com/rancher/machine/blob/v0.15.0-rancher73/libmachine/provision/redhat_ssh_commander.go#L15-L42
CentOS leverages the RedHat provisioner under the hood, which in-turn leverages the systemd provisioner.
https://github.com/rancher/machine/blob/v0.15.0-rancher73/libmachine/provision/centos.go#L13-L17
https://github.com/rancher/machine/blob/v0.15.0-rancher73/libmachine/provision/redhat.go#L81-L102
https://github.com/rancher/machine/blob/v0.15.0-rancher73/libmachine/provision/systemd.go#L21-L34
https://github.com/rancher/machine/blob/v0.15.0-rancher73/libmachine/provision/custom_script.go#L19-L22
A workaround of disabling all yum repositories with yum-config-manager --disable \* or removing them entirely using the commands found further below was advised but not implemented in the VM template.
There was a secondary issue reported after a workaround was implemented for the curl issue:
 2022/03/25 18:30:32 [ERROR] cluster [c-XXXXX] provisioning: Failed to set up SSH tunneling for host [x.x.x.x]: Can't retrieve Docker Info: error during connect: Get "http://%!F(MISSING)var%!F(MISSING)run%!F(MISSING)docker.sock/v1.24/info": can not build dialer to [c-XXXXX:m-YYYYY] (edited)

This issue is caused by the dialer not having an existing session. https://github.com/rancher/rancher/blob/v2.6.3/pkg/dialer/factory.go#L277-L296
The issue was resolved by switching to using the Rancher docker installation scripts. We suspect that the existing 19.03.11 docker installation was not properly configured post-installation. There may have been additional issues with the existing docker installation due to OS hardening with selinux but this was not able to be verified.
At this time, the only action items are to additionally document that any vSphere templates being used for airgapped rke1 and rke2 clusters should not have public repositories configured as the time of VM provisioning. In the case of centos 7 it will cause failures if any yum install commands are run as yum attempts to build the cache for the public repos and fails to connect due to the airgap state.
The two solutions are:

remove all yum repositories when creating the golden image/template

yum remove epel* -y
rm -f /etc/yum.repos.d/*
yum clean all
rm -rf /var/cache/yum

using SUSE Manager or RedHat Satellite/Spacewalk to serve repositories inside the air-gapped network. This is not in the scope of Rancher support but should be noted for airgap.

RKE1 Windows is EOL.


Fixed: wiki103, Word Formality, wikiHow step-goal, Style adaptation

Skipped Summarization RLUE and NER
@danyaljj

Thanks! Feel free to merge this when it turns green.


armv6 and armv5?

Is there a simple way to do that? Can one APK target multiple architectures or do separate builds need to be set up?

use NDK 16 ?
Note: Historically the NDK supported ARMv5
(armeabi), and 32-bit and 64-bit MIPS, but support for
these ABIs was removed in NDK r17.

https://developer.android.com/ndk/guides/abis?hl=en


Syntax update

Parameter lists in lambda must be surrounded by round brackets.
Examples:
() -> 123                 # lazy term
(x) -> 456                # one-arg
(x, y) -> 789             # two-arg
(x, y, z) -> + x (+ y z)  # three args

This will probably require a major rewrite of the parser but it clarifies many things in the syntax.

Do not make round brackets for parameter lists optional: that would only complicate the syntax. "There should be one obvious way to do it."

Round brackets followed by an arrow is still complicated for the parser and potentially ambiguous. A leading keyword is the best solution. fn will be used as leading keyword.
Examples:
fn -> 123                  # lazy term
fn x -> 456                # one-arg
fn x, y -> 789             # two-arg
fn x, y, z -> + x (+ y z)  # three args


[Feature Request] Add converter to StoreBuilder

Some of the samples demonstrate adding a converter via a StoreBuilder but it looks like that's only possible from MutableStoreBuilder now, is that right?

https://www.reddit.com/r/androiddev/comments/13mb2sz/comment/jkzutgk/?utm_source=share&utm_medium=web2x&context=3
RealStore uses the concept of converter but currently it's only possible to configure using MutableStoreBuilder

Maybe storebuilder.withConverter?

I would like to add a comment as I think this is my case.
Now I am using Shop4 and the SourceOfTruth interface in it has the following signature: SourceOfTruth<Key, Input, Output>, which allows me to use different types of reading and writing in SourceOfTruth (When I receive data from the network, I modify it and write it to the database, but for reading I already get a different type (a complex combined query in the database), but there is no such possibility in Store5, due to its signature for SourceOfTruth is SourceOfTruth<Key : Any, Local : Any>), which does not allow switching to it.
Could a potential Converter cover this case?

I would like to add a comment as I think this is my case.
Now I am using Store4 and the SourceOfTruth interface in it has the following signature: SourceOfTruth<Key, Input, Output>, which allows me to use different types of reading and writing in SourceOfTruth (When I receive data from the network, I modify it and write it to the database, but for reading I already get a different type (a complex combined query in the database), but there is no such possibility in Store5, due to its signature for SourceOfTruth is SourceOfTruth<Key : Any, Local : Any>), which does not allow switching to it.
Could a potential Converter cover this case?

The documentation shows this as being included already, but as of 5.0.0-beta1 it's not. Note in the docs the return type is MutableStore yet the builder used is StoreBuilder.
https://mobilenativefoundation.github.io/Store/mutable-store/building/builder/

Hello! apologies for this breaking change, I'm at a conference this week but will take a look over weekend/next week. I think this was an unintended breaking change from an alpha where we had mutable store being a behavior or store rather than a proxy that you wrap a store with. I'll touch base with other maintainer and figure out a path forward. Apologies for the mess :-)

I don't believe anyone is and contributions are incredibly appreciated


Add CI - Build Distribution

Now the project is becoming more popular, it might be handy to include some files to keep de contributors at the same page.
Therefor I've added...

an .editorconfig file to maintain coding-style.
a code of conduct file.
a contributing file which contains information and rules about contributing.
a workflow to build the distribution files when a commit is pushed to the main branch.

Notes:

For the workflow, you've to create a classic Personal Access Token.
This token ONLY needs scope public_repo.

For the workflow, you've to create a secret with the name WORKFLOW_GIT_ACCESS_TOKEN and give it the value of the token, created above.

Please read topic Prevent changes to the distribution directory of CONTRIBUTING.md
Since the added workflow builds the distribution for us, we do NOT want to commit anything to directory dist or its contents.

That's a personal choice.
As soon as it expires, the script stops working until you've created a new one and updated the secret.
IMO, I think it's ok to set no expiration in this case.


feat: Use REST

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

@schmidt-sebastian this is cool, is your intention to default to gRPC, and document REST?

@bcoe I haven't decided yet. We need to fix one more issue with queries and then compare performance data to see what mode should be the default.

@MarkDuckworth It works now! When I throw new Error('...') in<EMAIL_ADDRESS>and run npm run system-test:rest, only a handful of tests that actually require gRPC fail, most tests pass.

Hello! Sorry for reviving an old pull request, but was this data published anywhere?

We need to fix one more issue with queries and then compare performance data to see what mode should be the default.

Performance data was not published. From what I have seen, internal tests and customer reports indicate a speedup when using preferRest in the cold start of functions. However, I cannot give specific numbers.
We did not change the default mode, which is still gRPC.


Where do the object_points come from and which in which units are they measured?

Hi @lincolnhard,
first - thanks for the nice project.
I would like to know where do the values for the object_points come from and in which units are they?

Me too. The link in ReadMe http://aifi.isr.uc.pt/Downloads/OpenGL/glAnthropometric3DModel.cpp is not available. Could you please provide the original 3D face model points? I would really appreciate that. Thanks a lot! @lincolnhard

@kirito12138 I had forked/modified this a while ago - here's an annotated version:
object_pts_source = [
    [-7.308957, 0.913869, 0.000000],
    [-6.775290, -0.730814, -0.012799],
    [-5.665918, -3.286078, 1.022951],
    [-5.011779, -4.876396, 1.047961],
    [-4.056931, -5.947019, 1.636229],
    [-1.833492, -7.056977, 4.061275],
    [0.000000, -7.415691, 4.070434],
    [1.833492, -7.056977, 4.061275],
    [4.056931, -5.947019, 1.636229],
    [5.011779, -4.876396, 1.047961],
    [5.665918, -3.286078, 1.022951],
    [6.775290, -0.730814, -0.012799],
    [7.308957, 0.913869, 0.000000],
    [5.311432, 5.485328, 3.987654],
    [4.461908, 6.189018, 5.594410],
    [3.550622, 6.185143, 5.712299],
    [2.542231, 5.862829, 4.687939],
    [1.789930, 5.393625, 4.413414],
    [2.693583, 5.018237, 5.072837],
    [3.530191, 4.981603, 4.937805],
    [4.490323, 5.186498, 4.694397],
    [-5.311432, 5.485328, 3.987654],
    [-4.461908, 6.189018, 5.594410],
    [-3.550622, 6.185143, 5.712299],
    [-2.542231, 5.862829, 4.687939],
    [-1.789930, 5.393625, 4.413414],
    [-2.693583, 5.018237, 5.072837],
    [-3.530191, 4.981603, 4.937805],
    [-4.490323, 5.186498, 4.694397],
    [1.330353, 7.122144, 6.903745],
    [2.533424, 7.878085, 7.451034],
    [4.861131, 7.878672, 6.601275],
    [6.137002, 7.271266, 5.200823],
    [6.825897, 6.760612, 4.402142],
    [-1.330353, 7.122144, 6.903745],
    [-2.533424, 7.878085, 7.451034],
    [-4.861131, 7.878672, 6.601275],
    [-6.137002, 7.271266, 5.200823],
    [-6.825897, 6.760612, 4.402142],
    [-2.774015, -2.080775, 5.048531],
    [-0.509714, -1.571179, 6.566167],
    [0.000000, -1.646444, 6.704956],
    [0.509714, -1.571179, 6.566167],
    [2.774015, -2.080775, 5.048531],
    [0.589441, -2.958597, 6.109526],
    [0.000000, -3.116408, 6.097667],
    [-0.589441, -2.958597, 6.109526],
    [-0.981972, 4.554081, 6.301271],
    [-0.973987, 1.916389, 7.654050],
    [-2.005628, 1.409845, 6.165652],
    [-1.930245, 0.424351, 5.914376],
    [-0.746313, 0.348381, 6.263227],
    [0.000000, 0.000000, 6.763430],
    [0.746313, 0.348381, 6.263227],
    [1.930245, 0.424351, 5.914376],
    [2.005628, 1.409845, 6.165652],
    [0.973987, 1.916389, 7.654050],
    [0.981972, 4.554081, 6.301271],
    [0.000000, 1.916389, 7.700000],
]

points_map = {
    # object_pt_id: image_pt_id
    0: 17,  # start jawline - right
    1: 16,
    2: 14,
    3: 13,
    4: 12,
    5: 10,
    6: 9,  # chin center
    7: 8,
    8: 6,
    9: 5,
    10: 4,
    11: 2,
    12: 1,  # end jawline - left
    43: 49,  # start mouth - left corner
    42: 51,
    41: 52,
    40: 53,
    39: 55,  # mouth - right corner
    46: 57,
    45: 58,
    44: 59,  # end mouth
    54: 32,  # start nose - left
    53: 33,
    52: 34,
    51: 35,
    50: 36,  # end nose - right
    58: 31,  # nose tip (new point)
    13: 37,  # start left eye - left corner
    14: 38,
    16: 39,
    17: 40,  # left eye - right corner
    18: 41,
    20: 42,  # end left eye
    25: 43,  # start right eye - left corner
    24: 44,
    22: 45,
    21: 46,  # right eye - right corner
    28: 47,
    26: 48,  # end right eye
    33: 18,  # start left eyebrow - left corner
    32: 19,
    31: 20,
    30: 21,
    29: 22,  # end left eyebrow - right corner
    34: 23,  # start right eyebrow - left corner
    35: 24,
    36: 25,
    37: 26,
    38: 27,  # end right eyebrow - right corner
}

Thank you so much for your help!

i wonder where did the "object_pts_source" come from? what are those number mean?
@naftalibeder naftalibeder

@kirito12138 I had forked/modified this a while ago - here's an annotated version:
object_pts_source = [
    [-7.308957, 0.913869, 0.000000],
    [-6.775290, -0.730814, -0.012799],
    [-5.665918, -3.286078, 1.022951],
    [-5.011779, -4.876396, 1.047961],
    [-4.056931, -5.947019, 1.636229],
    [-1.833492, -7.056977, 4.061275],
    [0.000000, -7.415691, 4.070434],
    [1.833492, -7.056977, 4.061275],
    [4.056931, -5.947019, 1.636229],
    [5.011779, -4.876396, 1.047961],
    [5.665918, -3.286078, 1.022951],
    [6.775290, -0.730814, -0.012799],
    [7.308957, 0.913869, 0.000000],
    [5.311432, 5.485328, 3.987654],
    [4.461908, 6.189018, 5.594410],
    [3.550622, 6.185143, 5.712299],
    [2.542231, 5.862829, 4.687939],
    [1.789930, 5.393625, 4.413414],
    [2.693583, 5.018237, 5.072837],
    [3.530191, 4.981603, 4.937805],
    [4.490323, 5.186498, 4.694397],
    [-5.311432, 5.485328, 3.987654],
    [-4.461908, 6.189018, 5.594410],
    [-3.550622, 6.185143, 5.712299],
    [-2.542231, 5.862829, 4.687939],
    [-1.789930, 5.393625, 4.413414],
    [-2.693583, 5.018237, 5.072837],
    [-3.530191, 4.981603, 4.937805],
    [-4.490323, 5.186498, 4.694397],
    [1.330353, 7.122144, 6.903745],
    [2.533424, 7.878085, 7.451034],
    [4.861131, 7.878672, 6.601275],
    [6.137002, 7.271266, 5.200823],
    [6.825897, 6.760612, 4.402142],
    [-1.330353, 7.122144, 6.903745],
    [-2.533424, 7.878085, 7.451034],
    [-4.861131, 7.878672, 6.601275],
    [-6.137002, 7.271266, 5.200823],
    [-6.825897, 6.760612, 4.402142],
    [-2.774015, -2.080775, 5.048531],
    [-0.509714, -1.571179, 6.566167],
    [0.000000, -1.646444, 6.704956],
    [0.509714, -1.571179, 6.566167],
    [2.774015, -2.080775, 5.048531],
    [0.589441, -2.958597, 6.109526],
    [0.000000, -3.116408, 6.097667],
    [-0.589441, -2.958597, 6.109526],
    [-0.981972, 4.554081, 6.301271],
    [-0.973987, 1.916389, 7.654050],
    [-2.005628, 1.409845, 6.165652],
    [-1.930245, 0.424351, 5.914376],
    [-0.746313, 0.348381, 6.263227],
    [0.000000, 0.000000, 6.763430],
    [0.746313, 0.348381, 6.263227],
    [1.930245, 0.424351, 5.914376],
    [2.005628, 1.409845, 6.165652],
    [0.973987, 1.916389, 7.654050],
    [0.981972, 4.554081, 6.301271],
    [0.000000, 1.916389, 7.700000],
]


[Bug] Child components with the same ref name are not found

I noticed that child components that use the same name for a ref cause the main ref to no longer be found.
Example scenario
export const ParentComponent = defineComponent({
  name: 'parent-component',
  refs: {
    someChild: refComponent(ChildComponent, { ref: 'some-child'  }),
  }
});

export const ChildComponent = defineComponent({
  name: 'child-component',
  refs: {
    // This component also has a ref that is named `some-child`
    someOtherChild: refComponent(SomeOtherChild, { ref: 'some-child' }),
  }
});

Result
The following error is displayed for the ParentComponent in the console:

Component not found [data-ref="some-child"]

Hello! With this example, I was unable to reproduce this bug. Is it still relevant? If you have time, can you please share the details of this case @larsvanbraam? Thank you!

Added bugfix/9-child-components-with-the-same-ref-name-are-not-found with a story, but it's not showing the issue.
I need more info to reproduce and fix this.

@ThaNarie @jyggiz I'm also no longer able to reproduce the original issue, weird! 🤔
I was able to locate another weird issue where the refElement returned the same element as the refs.self.element when they both use the same data-ref value. I've updated the branch to demonstrate the issue!

I was able to locate another weird issue where the refElement returned the same element as the refs.self.element when they both use the same data-ref value.

This is by design (and requested as a convenience feature), to keep a data-ref working when moving it to the data-component element. It's slightly confusing when both parents and components can target the same element when they use the same ref, and it gets priority.
Changing it to only use it as fallback could result in weird behaviour with optional refs (sometimes falling back to the root element, and sometimes selecting the correct child elements).

Ok, then I'll close and mark this one as invalid!


Remove stack-based out-of-place Jacobian and Hessian?

Currently, out-of-place Jacobians and Hessians use stack to put together the batches of JVPs/VJPs/HVPs, and there is a possible resizing at the end.

Advantages: preserves the native types of matrix-vector products
Drawbacks: can cause type instability (in the closure) and unwanted allocations (when concatenating or resizing)

Is there any drawback to defining a buffer as jac = similar(x, length(y), length(x)) or hess = similar(x, length(x), length(x))?
The only one I see is that the operators would turn SArrays into MArrays

Closing because it would impose mutability where I've made sure to avoid it


[SPARK-27040][SQL] Avoid using unnecessary JoinRow in FileFormat

What changes were proposed in this pull request?
When reading files with empty partition columns, we can avoid using JoinRow.
How was this patch tested?
Existing unit tests.

Test build #102974 has started for PR 23953 at commit fd691ae.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8443/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8445/
Test PASSed.

Test build #102976 has started for PR 23953 at commit 2d9f62c.

Test build #102974 has finished for PR 23953 at commit fd691ae.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102974/
Test PASSed.

Test build #102976 has finished for PR 23953 at commit 2d9f62c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/102976/
Test PASSed.

LGTM, merging to master!


[config/rbac] Add RayJob roles.

What type of PR is this?
/kind bug

What this PR does / why we need it:
[config/rbac] Add RayJob roles.
Which issue(s) this PR fixes:

Fixes #825
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/lgtm
/approve


How to query against a related model?

I have a set of models that are related like this:

Listing belongsTo Unit
Unit belongsTo Property
Property belongsTo Neighborhood

I need to query all of the listings withRelated: ['unit.property.neighborhood'] based on neighborhood.id. I'm not having any luck and I'm having a lot of trouble running down any documentation or even discussion about it. If I take out my attempt to filter, my listings come back just fine.
My models look like this:
// models/listing.js
var Listing = App.extend({
    tableName: 'listings',

    unit: function() {
        return this.belongsTo('Unit', 'unit_id');
    },
});

// models/unit.js
var Unit = App.extend({
    tableName: 'units',

    /**
     * Returns the unit's property.
     *
     * @return {Object}
     */
    property: function() {
        return this.belongsTo('Property');
    },

    /**
     * Returns the listings for a given unit.
     *
     * @return {Array}
     */
    listings: function() {
        return this.hasMany('Listing');
    },
});

// models/property.js
var Property = App.extend({
    tableName: 'properties',

    neighborhood: function() {
        return this.belongsTo('Neighborhood');
    },

    units: function() {
        return this.hasMany('Unit');
    },
});

// models/neighborhood.js
var Neighborhood = App.extend({
    tableName: 'neighborhoods',

    properties: function() {
        return this.hasMany('Property');
    },
});

My latest attempt at getting this to work from my listings route is in this gist.
I try not to use issues for support requests, but I'm not sure where else to go. This seems like functionality that almost sure exists so I must be setting something up incorrectly.

You should use knex (i.e. model.query) to do a join in order to perform the listing filtering by neighborhood id. The way you're handling it in the gist is not using a valid column name, even if there were joins involved. And Bookshelf mostly doesn't use joins where you might expect because it's not possible to properly join and denormalize data without knowing the schema. That's the most significant upcoming change that needs to happen.


[fix-13504] Display the current project name on the left corner of the page

Purpose of the pull request
[fix-13504] Display the current project name on the left corner of the page.
Releated PR: #13504
After fixing, the result would be as follows:

Brief change log
[fix-13504] Display the current project name on the left corner of the page.
Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@songjianet @Amy0104 Could you please do me a favour? This is my first PR. I do not see what this error means.

@SbloodyS I have updated my codes. And could you please help to label this issue for passing CI/CD's label check.

@SbloodyS I have updated my codes. And could you please help to label this issue for passing CI/CD's label check.

Done.

@SbloodyS @songjianet @Amy0104 Sorry to disturb and I would like to know why it stuck here. Could you please help me out?

I have rerun it.


feat: rename icon to logo slot and add content logo

Since Content is an official module, it makes sense to keep the logo, also it will simplify a lot the examples.

Will update the content logo, the proportion are not good:

https://nuxt-ui-components-as2ke3vq8-nuxt-js.vercel.app/example
https://nuxt-ui-components-as2ke3vq8-nuxt-js.vercel.app/example-content

Yes we can, they are not going to be many official mobiles anyway :blush:
Also, what’s wrong since the components are loaded only when used? The example component is aimed to be used for exemples anyway (like most the UI component library so far…)


TimeSpan deserialization does not work

https://deck.net/57067763f18bfd70093f8b65dfcae5c4
https://dotnetfiddle.net/Widget/HmlufY
using System;
using Newtonsoft.Json;

public class Program
{
	public static void Main()
	{
		TimeSpan ts = (TimeSpan)JsonConvert.DeserializeObject("\"00:00:01\"", typeof(TimeSpan));

		Console.WriteLine(ts);
	}
}

System.Exception: uncaught exception: System.ArgumentException: Could not cast or convert from String to System.TimeSpan
ctor@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:88705
ctor@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:91585
DeserializeObject@https://deck.net/resources/js/bridge/newtonsoft.json.min.js?16.8.0:7:28103
Main@https://deck.net/RunHandler.ashx?h=-1165200509:9:72
init/</<@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:40146
i@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:5505
i@https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:27146
add@https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:27450
n.fn.ready@https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:29497
n.fn.init@https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:24943
n@https://cdnjs.cloudflare.com/ajax/libs/jquery/2.2.4/jquery.min.js:2:405
ready@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:5554
init/<@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:40122
init@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:40108
Bridge.assembly@https://deck.net/resources/js/bridge/bridge.min.js?16.8.0:7:40762
@https://deck.net/RunHandler.ashx?h=-1165200509:4:1

Would the above issue be causing the issue I'm experiencing with DateTimeOffset deserialization failing with argument out of range exception for offset (which is a TimeSpan)?
https://deck.net/ba083a00c886571deec195adc6497b53

Hello, @FasettoAndrew!
Your deck is not triggering any exception to me. So can't say the issue is related or not to this one. As far as the original issue is concerned, it looks like removed from the recent pull request to this issue.
Please double check the deck you provided, if you can truly reproduce the issue, or have more information on how to actually reproduce it, create a new issue reporting it so we can check it out.

By going to my deck.net link I still see the issue. I have included a screenshot of the deck.net page so you can see what version of bridge it is built with.

@FasettoAndrew
It is different issue.
I got the following:
Argument_UTCOutOfRange, Parameter name: offset
My and deck servers are most likely in different time zones.
So create new issue as it is recommended and reference this one.


Updating HoloLens RS4 Preview doc

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/CONTRIBUTING.md
:white_check_mark:Succeeded
View

mixed-reality-docs/creating-3d-models-for-use-in-the-windows-mixed-reality-home.md
:white_check_mark:Succeeded
View

mixed-reality-docs/hololens-rs4-preview.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/MarkdownPoster.pdf
:white_check_mark:Succeeded
View

mixed-reality-docs/images/ResearchModeDevPortal.png
:white_check_mark:Succeeded
View

mixed-reality-docs/implementing-3d-app-launchers.md
:white_check_mark:Succeeded
View

mixed-reality-docs/research-mode.md
:white_check_mark:Succeeded
View

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


Jiggle physics work fine in unity editor, but don't work at all in game.

Creating a playermodel mod for Lethal Company - I added some jiggle physics to the model, and it works perfectly fine in the unity editor, but it just won't work in game. I've tried multiple different combinations of settings and such to no success. Is this a known issue? is there a fix for it? because I can't find one.

Don't install from git, use the one bundled with the Character Replacement SDK

I had actually installed from git (spent about 2 hours trying to figure out why the console was spitting out tons of errors) but have since removed the second (redundant) install before I was having this issue. Is there something else I need to do to fully remove it that I'm not aware of? clean caches or something like that? make a fresh project? I assumed I was successful in removing it because the console isn't giving any errors anymore.

Found the problem (or at least a fix). I deleted the Model Replacement Properties (Character Replacement SDK component) from my playermodel and then went through the process of setting up the playermodel again.


Add Ajay to others & add .pypirc to .gitignore

Add Ajay Bandaru to Authors
Summary
This PR adds 'Ajay Bandaru' to the authors list in the pyproject.toml file and adds '.pypirc' to '.gitignore'
Changes
Ajay to Authors
.pypirc to .gitignore
Related Issues

Closes #20: Add Ajay to Authors

We'll continue the remaining work for the API Key in the CONTRIBUTING guide issue.


difficult to see the number of crystal shards needed

if possible could you change the text outline or move it slightly higher than the item icon as it is too difficult to see the text for the number of crystal shards needed. thank you

I am working on this. The solutions that I have so far change the font too much and end up causing issues with large numbers so I will continue to work on fixing this hopefully without having to change how things look too much.

Just commenting here to see if this is still being worked. Thanks for all the hard work.

impossible to see, plugin is not usable at all. Just change the color to red. Of either the shards or the letters, that will solve the problem. Would love to be able to use this plugin.


Fix translations and warnings in calendar pages

Ref. #4723

Fixes #6390

Will this also address the problem that the calendar popup event has a wrong date format? It should use the sites default date format.

@Hypocrite could you describe the exact steps to follow to show that dialog (starting from a clean installation of concrete5)?

I have manually merged this. I had to in order to fix conflicts.
I don't know why GitHub doesn't seem to want to automatically mark these as merged when I do them via the command line to fix conflicts. Frustrating.


Code quality (part 1)

Here are some contributions to improve the code quality by fixing reported violations.

I updated my branch to account for @t-8ch rework of StashClient + have dropped the fix to the false-positive vulnerability reported


Non native arith

Codecov Report
Base: 50.62% // Head: 43.11% // Decreases project coverage by -7.50% :warning:

Coverage data is based on head (fa93b9a) compared to base (408d3ff).
Patch coverage: 0.00% of modified lines in pull request are covered.

:exclamation: Current head fa93b9a differs from pull request most recent head c1c7b2a. Consider uploading reports for the commit c1c7b2a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage   50.62%   43.11%   -7.51%
==========================================
  Files          29       29
  Lines        4482     5309     +827
==========================================
+ Hits         2269     2289      +20
- Misses       2213     3020     +807

Impacted Files
Coverage Δ

circuit/src/wrong/native.rs
0.00% <0.00%> (ø)

circuit/src/wrong/rns.rs
0.00% <0.00%> (ø)

protocol/src/utils.rs
29.31% <0.00%> (-39.44%)
:arrow_down:

protocol/src/epoch.rs
93.93% <0.00%> (-3.04%)
:arrow_down:

protocol/src/peer/pubkey.rs

protocol/src/peer/opinion.rs

protocol/src/behaviour/req_res.rs

protocol/src/behaviour/mod.rs

protocol/src/node.rs

protocol/src/peer/mod.rs

... and 7 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


v1.2.0 failed to install

When I tried to install v1.2.0 it failed

and
yamboy1@computer:~$ apm install atom-discord
Installing atom-discord to /home/yamboy1/.atom/packages ✗
undefined
`

Could you try
$ git clone https://github.com/HeloWorld017/atom-discord.git && cd atom-discord/ && apm link
?

It installs, I haven't checked whether it works in discord yet

However it give me this error:
Set up discord RPC.
{ Error: connect ENOENT /run/user/1001/snap.atom/discord-ipc-0
    at Object.exports._errnoException (util.js:1022:11)
    at exports._exceptionWithHostPort (util.js:1045:20)
    at PipeConnectWrap.afterConnect [as oncomplete] (net.js:1090:14)
  code: 'ENOENT',
  errno: 'ENOENT',
  syscall: 'connect',
  address: '/run/user/1001/snap.atom/discord-ipc-0' }

And the Rich Presence still doesn't work

Hmm... I think it may error of

But it won't be that part because I'm using Linux

ENOENT means No such file or directory

NVM IT WORKS!!!!!!
I think that my discord was broken.


Citation in README.md is wrong

Ah! Solved it (https://github.com/neulab/guided_summarization/commit/98f845c16bb308c7d2972c646cb77c6f073c3b11), thanks!


Version 2: Rebirth

Changing all aspects of the library. It was useless because of the requirements and ugly return types.

The class Stringy is now JString.
All return types of the methods that before return Stringy now returns a normal php string

Fix concat method


icon RenderRebuild mode works for growing but not shrinking

Unclear why -- some kind of min size factor somewhere.
Also button labels go to multi-line during zooming -- prior size alloc is being used?

This is better now but there are still some issues in which it does not work correctly

One case where this doesn't work correctly is when you move a window from a 1080p monitor to a 4k monitor (the icons stay small). Also, we need to figure out our 4k scaling approach more broadly.


Regenerate trace client

This PR was generated using Autosynth. :rainbow:
Here's the log from Synthtool:
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/google-cloud-clients/google-cloud-trace/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:9aed6bbde54e26d2fcde7aa86d9f64c0278f741e58808c46573e488cbf6098f0
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/devtools/cloudtrace/artman_cloudtrace_v1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/java.
synthtool > Running java formatter on 24 files
synthtool > Running java formatter on 1 files
synthtool > Running java formatter on 15 files
synthtool > Running generator for google/devtools/cloudtrace/artman_cloudtrace_v2.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/java.
synthtool > Running java formatter on 24 files
synthtool > Running java formatter on 1 files
synthtool > Running java formatter on 16 files
synthtool > Cleaned up 0 temporary directories.
synthtool > Wrote metadata to synth.metadata.

Combined into a single pull request #5850


resize_image_with_crop_or_pad should work with batch of images

This is a repost of https://github.com/tensorflow/tensorflow/issues/2284 per comment from @mrry (CC @ziky90).
resize_image_with_crop_or_pad should be modified to take either a single image or a batch of images. Right now the signature of the function specifies:
    image: 3-D tensor of shape `[height, width, channels]`

For a reference of what the api should look like, see resize_images(images,....):
    images: 4-D Tensor of shape `[batch, height, width, channels]` or
            3-D Tensor of shape `[height, width, channels]`.

Some commentary:

docs for resize_images say: " Resized images will be distorted if their original aspect ratio is not the same as size.  To avoid distortions see [resize_image_with_crop_or_pad]." However the method referenced cannot be currently used in some cases for which resize_images works due to the above mentioned limitations.
the entire operation can be implemented using slice operations on the tensor batch. This is what resize_image_with_crop_or_pad does but it has the following logic which would be nice not to copy and paste (there is some subtlety with round down vs up):

  width_diff = target_width - width
  offset_crop_width = max_(-width_diff // 2, 0)
  offset_pad_width = max_(width_diff // 2, 0)

  height_diff = target_height - height
  offset_crop_height = max_(-height_diff // 2, 0)
  offset_pad_height = max_(height_diff // 2, 0)

the fact this one image op method does not handle batches leads to confusion and various SO posts. People even resort to solving the problem using all sorts of crazy solutions (e.g. tf.while_loop).
this method is useful in combination with concatenation operations. For example see the Lasagne ConcatLayer.

@shlens Can you comment?  Ages ago we stripped most of these image preprocessing ops down to 3-D only; what do you think about people raising them back up again?
@cancan101 The motivation for 3-D only is that image processing pipelines that take images of different sizes have to handle separate images as separate tensors anyways, at least until after the resizing step happens.  And this op is the resizing step.

@girving Where I am using this, and likely others as well (this was the same use case that Lasagne handles) is in the NN portion graph itself when doing an upsample. I want to upsample and then concatenate across channels but I need to take a center crop to make sure spatial dimensionality  matches.

@cancan101 Okay, I think fixing this is quite reasonable.  However, if it's fixed it should work for any rank >= 3.  This shouldn't be much more complicated than doing it for just 3D and 4D, since the higher ranks all mean map.

@girving I think adding support for 4D sounds reasonable -- especially if it leads to consistency in the API across image resizing functions.

@cancan101 Great, we're all in agreement that this is a good change.  Would you be interested in submitting a PR?

Is it okay to change the api of this method? Perhaps with some sort of deprecation to take in images rather than image?

@cancan101 Please leave it image, then document that image can be higher rank.


Figure out why we're leaking npiperelay.exe instances

Something isn't being torn down correctly and we're ending up leaking npiperelay.exe instances, we probably need to see if the associated socat is alive and if it's not, trash everything via taskkill.exe /F /IM npiperelay.exe

Possibly fixed by b68520a914509

I'm not sure if it's the same leak that this issue was originally referring to, but I believe I found a potential source of leaks with my own setup.
Basically, I noticed that my WSL2 environment would eventually just hang on any network requests indefinitely, and looking in Task Manager revealed it was because there were too many instances of ssh-agent and npiperelay. Running the command taskkill.exe /F /IM npiperelay.exe would temporarily fix things.
VS Code has a setting called git.autofetch, which (I believe) by default calls git fetch every 3 minutes when in a workspace with a git repository. I set up a small shell script that used tasklist.exe to watch for how many instances of npiperelay and ssh-agent were running every mintue, and lo and behold, every 3 minutes both numbers went up by one.
So for me, I had a leak caused by git.autofetch in VS Code.


Setting DEBUG to False crashes server

Setting debug to false in settings.py crashes the server (not usable)

This only occurs in local hosting. Setting debug to False in production (and now chlambase dev) works as expected

Django does not host static files in production mode. You are supposed to use a provider like Nginx or Apache to serve the static files, hence django will not work with DEBUG = False in a local environment. Marking as closed


python312Packages.k5test: 0.10.3 -> 0.10.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.k5test is: Library for setting up self-contained Kerberos 5 environment
meta.homepage for python312Packages.k5test is: https://github.com/pythongssapi/k5test
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.10.4 with grep in /nix/store/cl0a691nzv7xv6x23a3ss1wxpapmmvqq-python3.12-k5test-0.10.4
found 0.10.4 in filename of file in /nix/store/cl0a691nzv7xv6x23a3ss1wxpapmmvqq-python3.12-k5test-0.10.4

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.k5test
python312Packages.k5test

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cl0a691nzv7xv6x23a3ss1wxpapmmvqq-python3.12-k5test-0.10.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.k5test https://github.com/r-ryantm/nixpkgs/archive/7eb582f8aff1309df0e8817426c7faa6fe65c1e6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/7eb582f8aff1309df0e8817426c7faa6fe65c1e6#python312Packages.k5test

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cl0a691nzv7xv6x23a3ss1wxpapmmvqq-python3.12-k5test-0.10.4
ls -la /nix/store/cl0a691nzv7xv6x23a3ss1wxpapmmvqq-python3.12-k5test-0.10.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

4 packages built:

python311Packages.k5test
python311Packages.k5test.dist
python312Packages.k5test
python312Packages.k5test.dist

Maintainer pings

Result of nixpkgs-review pr 297364 run on x86_64-linux 1

4 packages built:

python311Packages.k5test
python311Packages.k5test.dist
python312Packages.k5test
python312Packages.k5test.dist


Rename folders.

Signed-off-by: immmus<EMAIL_ADDRESS>

@immmus I am not sure if you noticed, I spotted some other inconsistencies after my last review.

@immmus I am not sure if you noticed, I spotted some other inconsistencies after my last review.

Hi @davidradl, Yes, I saw, I'm just a little busy for now, I'll be back here a little later. Let's finish this for now pr -> https://github.com/odpi/egeria/pull/3366

@immmus I am not sure if you noticed, I spotted some other inconsistencies after my last review.

Hi @davidradl, Yes, I saw, I'm just a little busy for now, I'll be back here a little later. Let's finish this for now pr -> #3366

Hi @immmus, sure - I have made review comments on that pr.

@immmus I am not sure if you noticed, I spotted some other inconsistencies after my last review.

Hi @davidradl, Yes, I saw, I'm just a little busy for now, I'll be back here a little later. Let's finish this for now pr -> #3366

Hi @immmus, sure - I have made review comments on that pr.

I made the necessary changes yesterday