MessageInterceptorAdapter suspicious behavior

Got interesting output while debugging NB LwM2M client. So, with each re-transmission functions MessageInterceptorAdapter.onSent/onCancel... adds trigger action.
Add to LeshanServer

this.mLeshanServer.coap().getSecuredEndpoint().addInterceptor(messageInterceptorAdapter);

MessageInterceptorAdapter code.
private final MessageInterceptorAdapter messageInterceptorAdapter = new MessageInterceptorAdapter() {
        @Override
        public void sendRequest(final Request request) {
            request.setNanoTimestamp(System.currentTimeMillis());
            request.addMessageObserver(new MessageObserver() {
                @Override
                public void onTimeout() {
                    LOG.warn("timeout at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onSendError(Throwable error) {
                    LOG.warn("sent error {} at {} : {}", error, System.currentTimeMillis(), request);
                }

                @Override
                public void onRetransmission() {
                    LOG.warn("retransmission at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onResponse(Response response) {
                    LOG.warn("get response {} at {} : {}", response, System.currentTimeMillis(), request);

                }

                @Override
                public void onReject() {
                    LOG.warn("rejected at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onReadyToSend() {
                    LOG.warn("ready to send at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onDtlsRetransmission(int flight) {
                    LOG.warn("retransmit flight {}  at {} : {}", flight, System.currentTimeMillis(), request);
                }

                @Override
                public void onContextEstablished(EndpointContext endpointContext) {
                }

                @Override
                public void onConnecting() {
                    LOG.warn("connecting at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onComplete() {
                    LOG.warn("completed at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onCancel() {
                    LOG.warn("cancelled at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onAcknowledgement() {
                    LOG.warn("acknowledged at {} : {}", System.currentTimeMillis(), request);
                }

                @Override
                public void onSent(boolean retransmission) {
                    LOG.warn("sent at {} : {} : {}", System.currentTimeMillis(), request);
                }
            });
        }
    };

Output for one read request. Please focus on WARN SDProcessor$2$1.
Mar 10 17:29:42 iot bash[24969]: 2020-03-10 17:29:42,336 DEBUG Lwm2mHelper - Sending read to iot_sdl1bn_1 on /33755/0/0 at<PHONE_NUMBER>336

Mar 10 17:29:42 iot bash[24969]: 2020-03-10 17:29:42,337 WARN MyMessageTracer - sendRequest at:<PHONE_NUMBER>337 / false / CON / GET / 5556 / 207CCF176BF05B00 / Uri-Path: "33755", "0", "0" / null

Mar 10 17:29:42 iot bash[24969]: 2020-03-10 17:29:42,338 WARN SDProcessor$2$1 - ready to send at<PHONE_NUMBER>338 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:42 iot bash[24969]: 2020-03-10 17:29:42,338 WARN SDProcessor$2$1 - sent at<PHONE_NUMBER>338 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload : {}

Mar 10 17:29:46 iot bash[24969]: 2020-03-10 17:29:46,748 WARN SDProcessor$2$1 - retransmission at<PHONE_NUMBER>747 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:46 iot bash[24969]: 2020-03-10 17:29:46,748 WARN MyMessageTracer - sendRequest at:<PHONE_NUMBER>748 / false / CON / GET / 5556 / 207CCF176BF05B00 / Uri-Path: "33755", "0", "0" / null

Mar 10 17:29:46 iot bash[24969]: 2020-03-10 17:29:46,748 WARN SDProcessor$2$1 - ready to send at<PHONE_NUMBER>748 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:46 iot bash[24969]: 2020-03-10 17:29:46,748 WARN SDProcessor$2$1 - ready to send at<PHONE_NUMBER>748 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:46 iot bash[24969]: 2020-03-10 17:29:46,749 WARN SDProcessor$2$1 - sent at<PHONE_NUMBER>749 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload : {}

Mar 10 17:29:46 iot bash[24969]: 2020-03-10 17:29:46,750 WARN SDProcessor$2$1 - sent at<PHONE_NUMBER>750 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload : {}

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,567 WARN SDProcessor$2$1 - retransmission at<PHONE_NUMBER>567 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,568 WARN SDProcessor$2$1 - retransmission at<PHONE_NUMBER>568 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,569 WARN MyMessageTracer - sendRequest at:<PHONE_NUMBER>569 / false / CON / GET / 5556 / 207CCF176BF05B00 / Uri-Path: "33755", "0", "0" / null

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,569 WARN SDProcessor$2$1 - ready to send at<PHONE_NUMBER>569 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,569 WARN SDProcessor$2$1 - ready to send at<PHONE_NUMBER>569 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,570 WARN SDProcessor$2$1 - ready to send at<PHONE_NUMBER>569 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,570 WARN SDProcessor$2$1 - sent at<PHONE_NUMBER>570 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload : {}

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,571 WARN SDProcessor$2$1 - sent at<PHONE_NUMBER>570 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload : {}

Mar 10 17:29:55 iot bash[24969]: 2020-03-10 17:29:55,571 WARN SDProcessor$2$1 - sent at<PHONE_NUMBER>571 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload : {}

Mar 10 17:29:57 iot bash[24969]: 2020-03-10 17:29:57,339 WARN SDProcessor$2$1 - cancelled at<PHONE_NUMBER>339 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:57 iot bash[24969]: 2020-03-10 17:29:57,339 WARN SDProcessor$2$1 - cancelled at<PHONE_NUMBER>339 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:57 iot bash[24969]: 2020-03-10 17:29:57,339 WARN SDProcessor$2$1 - cancelled at<PHONE_NUMBER>339 : CON-GET    MID= 5556, Token=207CCF176BF05B00, OptionSet={"Uri-Path":["33755","0","0"]}, no payload

Mar 10 17:29:57 iot bash[24969]: 2020-03-10 17:29:57,340 DEBUG Lwm2mHelper - ReadRequest for iot_sdl1bn_1 on resource /33755/0/0 timeout! at<PHONE_NUMBER>340

Is this behavior is ok?

Is this behavior is ok?

I would say "yes".

The same the californium stack recall callback for retransmission. So for 3 retransmissions you will have 4 call to send request, ready to send and  send at

but you have more than 4 call ! The reason for each call of send request you add a new observer and so more logs

For the point 1. I think it's a correct behavior but you can open a discussion at californium project if you disagree.
For the point 2, this is bug probably because you didn't expect the 1. behavior :)

Got it,  thank you!


ProgressBar: fix Error color persistence after showError has been toggled

Description

Changing from Error to any other state other than Error will not set the color to the appropriate state but keep the Error color on the progress bar.
Motivation and Context

This PR fixes the Error color persistence after ShowError is set to false after being true.
How Has This Been Tested?

Manually.

/azp run


GEP-1619: Add to mkdocs

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Adds GEP-1619 to mkdocs which adds it to https://gateway-api.sigs.k8s.io/geps/overview/ webpage
Which issue(s) this PR fixes:

N/A
Does this PR introduce a user-facing change?:

NONE

/lgtm
/approve


Mock tests for the comment processor module are missing

The Comment Processor module introduced in #4  has only TODOs for mock tests. Add mock tests here:

[x] ~Put GitHub flow in distinct processor~ → already minimal
[x] Decide about good location for recaptcha check (form or processor?)
[ ] TestCommentProcessor

Decided to leave the recaptcha check where it is right now.
However, in a later version it might be beneficial to move it to the Processor, as recaptcha check and gettting the HEAD can be run in parallel to save on time.


Support 16nLC as a new device type.

Identify LC boards as a different device type.

Fixed via b736aae700947f0fbad695f4f6d819ba82743a80


UnsupportedDataConnectionTypes Data connections that are not supported on the new platform.

Trying to find out more detail on what this means
UnsupportedDataConnectionTypes | Data connections that are not supported on the new platform.
Does this mean that the InfoPath form could contain a SQL connection and that the data connection isnt supported on the new platform without additional infrastructure and licensing?

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

ID: 3c27cc7d-bb92-c1bc-4330-ae2bb54846bc
Version Independent ID: f14f15f3-e079-a6c6-d9ef-d7698f2e989f
Content: SharePoint Migration Assessment Tool - Migrate to Microsoft 365
Content Source: migration/overview-of-the-sharepoint-migration-assessment-tool.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

researching.

Hello.  Just following up.  Any more details?
Shane Ramos | Grp Mgr, Tech Architect
Enterprise Technology Architecture
Avanade Inc.
1191 Second Avenue Suite 100, Seattle, WA 98101
Cell: +1-206-239-6459;ext=21238
www.avanade.comhttp://www.avanade.com/
@.***
From: JoanneHendrickson @.>
Sent: Tuesday, July 27, 2021 12:30 PM
To: MicrosoftDocs/OfficeDocs-SharePoint @.>
Cc: Shane Ramos @.>; Author @.>
Subject: Re: [MicrosoftDocs/OfficeDocs-SharePoint] UnsupportedDataConnectionTypes Data connections that are not supported on the new platform. (#2883)
You don't often get email from<EMAIL_ADDRESS>Learn why this is importanthttp://aka.ms/LearnAboutSenderIdentification
This message is from an EXTERNAL SENDER - be CAUTIOUS, particularly with links and attachments.

researching.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SharePoint%2Fissues%2F2883%23issuecomment-887777017&data=04|01|shane.ramos%40avanade.com|96272dc6637e4b7f77b608d95134f8fa|cf36141cddd745a7b073111f66d0b30c|0|1|637630110233904232|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=QXSsHq2s1%2FEhpu1bAn7ailP1bK6I5i%2BoESU1jmBmKI0%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAU72HPGFJIVSMDM5Y4FQ5ODTZ4CMZANCNFSM5BC4AHNQ&data=04|01|shane.ramos%40avanade.com|96272dc6637e4b7f77b608d95134f8fa|cf36141cddd745a7b073111f66d0b30c|0|1|637630110233904232|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=r6Q9b%2FfTcGhmbBRVYfOIWdJAxrZZA%2FEBEqAO9hedG%2BY%3D&reserved=0.

Hi @spshane    I believe the error means that the InfoPath forms contain a data connection that is not supported on SPO; for example Soap Connection, REST Connection, RPC Connection and Ado Connection. The connection URL is usually reported. You should be able to check the URL to confirm which connection it is.


Move type definitions to devDependencies

Fixes #166 by avoiding dependencies on type definitions from
conflicting with other dependencies of dependent projects.

LGTM! Thanks @Vinnl I'll push a release in a few minutes!


results: permit a string or object as 1st arg

In #909 @celesteking volunteered to test.

give me some time (a week)

It logs:
[INFO] [896A7AD8-B0AA-46E7-9EB1-F1EF93486548] [core] { name: 'auth' }fail:mismatch
instead of [auth] fail:mismatch

@celesteking, correct, because as I pointed out in #909, the logger expects a plugin as the first argument. If you want ResultStore to work this way, then logger.js needs changing. If you're inclined to do that, I'll leave this PR open.


extra leading caret in structured_patterns

some structured_patterns in the nmdc-schema appear to start with a ^ and others don't , that means we have some structured_patterns that materialize to something starting with ^^
separated out from https://github.com/microbiomedata/nmdc-schema/issues/1869

classes.DataObject.slot_usage.was_generated_by
slots.was_generated_by.structured_pattern.syntax

PR appears to be approved. Can you close this issue by tomorrow @turbomam?

I am waiting on approval for several PRs from @naglepuff. I have slacked him, but haven't followed up recently.
@mslarae13 has requested that, in general, all berkeley-schema-fy24 migration stakeholder approve all PRs in the nmdc-schema, berkeley-schema-fy24 (and other) repos. In general, I think that's a good idea. We don't have any formal procedure for escalating other that to keep reminding the stakeholders or discussing the issue with project leaders. None of this is meant to say that @naglepuff is being especially difficult. The two of us just haven't had a meeting of the midns yet.
@aclum said that her approval can take the place of @mbthornton-lbl 's since he's out sick.


#1881 vision: include original Likelihood score

Add the original likelihood value into the response from the vision api
alongside the boolean values

I signed it!

Thank you very much :) I think the only thing missing before we merge is an update to the docs (in the JSDocs snippet above the method) and a test to confirm we're returning the original value. If you prefer, I'm happy to do these at a later time.


feat: format-painter

PR
PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our Commit Message Guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
Issue Number: #19
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Summary by CodeRabbit

New Features

Introduced a "Format Painter" feature, allowing users to copy and apply formatting between text selections.
Added a new sidebar entry for easy access to the Format Painter documentation.
Enhanced the text editing interface with a customizable toolbar in the new Format Painter component.

Documentation

Added comprehensive documentation for the Format Painter feature, including usage instructions and a demo.

Bug Fixes

Improved type safety and functionality within the Quill editor integration.

@kagol 感谢邀请。 缩进问题已解决，如果功能上不能满足需求我可以进行修改。


Implement a status pending.

Fixes problems introduced by #99.

Resort pending tasks by priorities.
Make _pytask/live.py more flexible to handle more statuses.

The executor from concurrent.futures are not reporting accurately: https://github.com/python/cpython/issues/81457.


Multiple Textures in a shader

I'm passing multiple textures to a fragment shader. I've verified that each texture loads correctly and each texture is bound to a different location. Adding both samplers together in the fragment shader does not yield the expected result. Instead of two different textures I see the same image superimposed on top of itself. I'm assuming it's something I'm doing wrong but there are no examples to draw from.
main.py
# Native Modules
import struct
import tkinter as tk

# External Modules
from lib import ModernGL
from lib.ModernGL import tk as ModernGL_tk
from lib.PIL import Image

ctx = ModernGL.create_standalone_context()

fbo = ctx.framebuffer(ctx.renderbuffer((512, 512)))

with open('shader/default_vertex.glsl') as f:
	v_sdr = f.read()

with open('shader/lut.glsl') as f:
	f_sdr = f.read()

prog = ctx.program([
	ctx.vertex_shader(v_sdr),
	ctx.fragment_shader(f_sdr),
	])

img1 = Image.open('texture/shady-forest.jpg')
texture1 = ctx.texture(img1.size, 3, img1.tobytes())
texture1.use(prog.uniforms['img'].location)

img2 = Image.open('texture/test_clut.ppm')
texture2 = ctx.texture(img2.size, 3, img2.tobytes())
texture2.use(prog.uniforms['clut'].location)

vbo = ctx.buffer(struct.pack('16f', -1.0, -1.0, 0.0, 0.0,
									-1.0, 1.0, 0.0, 1.0,
									1.0, -1.0, 1.0, 0.0,
									1.0, 1.0, 1.0, 1.0))
vao = ctx.simple_vertex_array(prog, vbo, ['vert', 'tex_coord'])

root = tk.Tk()
frame = ModernGL_tk.ModernGLCanvas(root, 512, 512)
frame.pack()

def draw():

	fbo.use()
	fbo.clear(0.94, 0.94, 0.94)
	vao.render(ModernGL.TRIANGLE_STRIP)

	fbo.read_into(frame.pixels)
	frame.update()
	frame.after(16, draw)

frame.after(16, draw)

root.mainloop()

shader/default_vertex.glsl
#version 330

in vec2 vert;
in vec2 tex_coord;
out vec2 v_tex_coord;

void main() {
	gl_Position = vec4(vert, 0.0, 1.0);
	v_tex_coord = tex_coord;
}

shader/lut.glsl
#version 330

uniform sampler2D img;
uniform sampler2D clut;

in vec2 v_tex_coord;
out vec4 color;

void main()
{
	color = texture(img, v_tex_coord) + texture(clut, v_tex_coord);
}

By default all the textures are bound to GL_TEXTURE0
I assume
print(prog.uniforms['img'].location) # prints 0
print(prog.uniforms['clut'].location) # prints 0

prog.uniforms['img'].location = 0
prog.uniforms['clut'].location = 1

texture1.use(0)
texture2.use(1)

your code will work if the lut.glsl uses layout(binding = ...) uniform sampler2D ...; for the uniforms. ( you will need 2 different binding points )
Layout_Qualifier_(GLSL)

I prefer setting those values from the code

I just noticed that you use the uniform location (prog.uniforms['img'].location)
You should use the value instead (prog.uniforms['img'].value)
The uniforms have different location (since they are 2 different uniforms) but they refference the same texture since their values are equal to 0.

prog.uniforms['img'].value = 0
prog.uniforms['clut'].value = 1

texture1.use(0)
texture2.use(1)

This solves the problem. Thanks for the direction.

Take a look at this new example I hope it will help.

I will highlight the difference in the docs. Thank you for reporting this issue.


Add a message for users running tf.js in node without the node backend

This PR adds a user-friendly message to remind people that we have node backend whenever they use tf.js in node with the vanilla cpu backend.
Also fixes a problem where a message saying document is not defined shows up at the top of the program. This was a side-effect of trying to access document when registering a WebGL backend.

DOC

This change is

Review status: 0 of 3 files reviewed at latest revision, all discussions resolved.

src/kernels/backend_cpu.ts, line 56 at r1 (raw file):

👋

What would this show up as in terminals that don't support unicode? Probably something like a question mark, as what I found when I tried it in non-unicode xterm. While I appreciate the lighthearted style, consider using something more robust.  :)

src/kernels/backend_cpu.ts, line 57 at r1 (raw file):

speed up

Grammar nit: "speech up" is transitive. Maybe say "speed things up" or "speech computation up."

src/kernels/backend_cpu.ts, line 57 at r1 (raw file):

 backend

Grammar nit: add comma after "backend".

src/kernels/backend_cpu.ts, line 59 at r1 (raw file):

`npm i @tensorflow/tfjs-node`

Not only this, but also something like import '@tensorflow/tfjs-node' is required in the source code - is that right?
Also, should you say something about CUDA support?

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 4 unresolved discussions.

src/kernels/backend_cpu.ts, line 59 at r1 (raw file):
Previously, caisq (Shanqing Cai) wrote…

`npm i @tensorflow/tfjs-node`

Not only this, but also something like import '@tensorflow/tfjs-node' is required in the source code - is that right?
Also, should you say something about CUDA support?

One idea, you can do a system call like which nvidia-smi to check whether nvidia-smi is on the path and if so, conditionally print some messages about CUDA tfjs-node.

Comments from Reviewable

Thanks for the review. PTAL.

Review status: 0 of 3 files reviewed at latest revision, 4 unresolved discussions.

src/kernels/backend_cpu.ts, line 56 at r1 (raw file):
Previously, caisq (Shanqing Cai) wrote…

👋

What would this show up as in terminals that don't support unicode? Probably something like a question mark, as what I found when I tried it in non-unicode xterm. While I appreciate the lighthearted style, consider using something more robust.  :)

It shows up as ö, which is not bad. Non-unicode terminals are getting more rare. I think we can afford some emoji in 2018.

src/kernels/backend_cpu.ts, line 57 at r1 (raw file):
Previously, caisq (Shanqing Cai) wrote…

speed up

Grammar nit: "speech up" is transitive. Maybe say "speed things up" or "speech computation up."

Done.

src/kernels/backend_cpu.ts, line 57 at r1 (raw file):
Previously, caisq (Shanqing Cai) wrote…

 backend

Grammar nit: add comma after "backend".

Done.

src/kernels/backend_cpu.ts, line 59 at r1 (raw file):
Previously, caisq (Shanqing Cai) wrote…
One idea, you can do a system call like which nvidia-smi to check whether nvidia-smi is on the path and if so, conditionally print some messages about CUDA tfjs-node.

Added CUDA. I did say "and import it in the program", but I added require('.....') to be more concise. Note I didn't use import '...' since that's es6 js. require() will work on all node versions. FYI, I didn't mention tf.setBackend('tensorflow') since that is going away in a upcoming PR (the import will do that for you).

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 4 unresolved discussions.

src/kernels/backend_cpu.ts, line 59 at r1 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
Added CUDA. I did say "and import it in the program", but I added require('.....') to be more concise. Note I didn't use import '...' since that's es6 js. require() will work on all node versions. FYI, I didn't mention tf.setBackend('tensorflow') since that is going away in a upcoming PR (the import will do that for you).

Leave which nvidia-smi for a future time. It will be tricky to conditionally import node utility libraries in tfjs-core that execute a system call while still making sure that the browserified bundle doesn't have those imports.

Comments from Reviewable

Thanks!

Review status: 0 of 3 files reviewed at latest revision, 4 unresolved discussions.

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 4 unresolved discussions, all commit checks successful.

src/kernels/backend_cpu.ts, line 59 at r1 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
Leave which nvidia-smi for a future time. It will be tricky to conditionally import node utility libraries in tfjs-core that execute a system call while still making sure that the browserified bundle doesn't have those imports.

CUDA is Linux only and still a pain to install. Lots of Linux users don't have a CUDA-capable card - so let's not sweat that here. I think if users know about CUDA + nvidia-smi (which is linux only) they'll be able to find @tensorflow/tfjs-node-gpu.
I'd advise dropping this for now.

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 4 unresolved discussions, all commit checks successful.

src/kernels/backend_cpu.ts, line 59 at r1 (raw file):
Previously, nkreeger (Nick Kreeger) wrote…
CUDA is Linux only and still a pain to install. Lots of Linux users don't have a CUDA-capable card - so let's not sweat that here. I think if users know about CUDA + nvidia-smi (which is linux only) they'll be able to find @tensorflow/tfjs-node-gpu.
I'd advise dropping this for now.

I'll still keep the tiny plug for CUDA (in case someone has it, they should know we support it) but not going into details. People that don't know what CUDA is will either 1) ignore the "if you have CUDA" message, or 2) go google it and learn about it, which is also good

Comments from Reviewable


A bug in Dockerfile.

Describe the bug

Should use >> instead of > !
Screenshots

So has it been fixed?


Create dedicated screen-capture extension.

A WBS browser extension for screen capture.
See:
https://chrome.google.com/webstore/detail/screen-capturing/ajhifddimkapgcifgcodmmfdlknahffk
https://github.com/muaz-khan/Chrome-Extensions/tree/master/desktopCapture

Chrome added native screen capture without extensions! Now available as the "Screen Capture" device.
Audio capture from applications, and Firefox support is pending.


build.sh: Set libdir for stp

On my Fedora, STP gets installed to install/lib64, but the script sets LD_LIBRARY_PATH to install/lib.
This should fix it.
Alternatively, you could stop fighting individual build scripts, let them install to the arch-specific libdir and use LD_LIBRARY_PATH='install/$LIB' (ld.so would resolve $LIB to the arch-specific name).

The other proposed solution seems also good, however, I'll merge the provided fix for now. Thanks.


Update deploy-a-dm-cluster-using-tiup-offline.md

First-time contributors' checklist

[x] I've signed Contributor License Agreement that's required for repo owners to accept my contribution.

What is changed, added, or deleted? (Required)

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v3.0 changes)
[x] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

/label needs-cherry-pick-2.0
/translation doing
/cc @CharLotteiu @lance6716
/uncc @qiancai

/lgtm

/label needs-cherry-pick-master
/unlabel needs-cherry-pick-2.0

/remove-label needs-cherry-pick-2.0

/merge

/remove-translation doing
/translation done


(MODULES-6592) Update DSC Resources to Oct-Nov-Dev 2017 Releases

This PR updates the DSC Resources listed below to the latest versions from the October, November, December, and February releases.
This PR also fixes the issue detailed in MODULES-6592 where the rename of the xSQLServer DSC Resource to SqlServerDSC causes problems with our build process. This has been avoided by the xSQLServer submodule reference being removed entirely and only the SqlServerDsc submodule remaining
This PR also includes several commits that fix minor issues in the build process that historically haven't been a large enough problem, but now prevented building DSC Resources for this release.
Normally this would be one commit with all changes included, but because of the SqlServerDSC rename each DSC Resource had to be committed separately to maintain the rename structure reflected correctly in git.
The following DSC Resources have been updated:

StorageDsc

SqlServerDsc

OfficeOnlineServerDsc

SecurityPolicyDsc

SharePointDsc

SystemLocaleDsc

xActiveDirectory

xAdcsDeployment

xCertificate

xComputerManagement

xDatabase

xDnsServer

xExchange

xFailoverCluster

xHyper-V

xNetworking

xPSDesiredStateConfiguration

xRemoteDesktopSessionHost

xSharePoint

xStorage

xTimeZone

xWebAdministration

[x] Update dsc.rake with comments describing that it is a workaround until we move to a solution that does not use the powershell/dscresources repo Not needed

[x] Add comment to dsc_resources_tags.yml that states xPowerShellExecutionPolicy needs to be pinned until https://github.com/PowerShell/xPowerShellExecutionPolicy/issues/14 is fixed

[x]  Test the tar with puppet module install, validate that all DSC files are present

Inside lib/puppet_x/dsc_resources:
PS C:\src\puppetlabs\modules\puppetlabs-dsc> gci -rec .\lib\puppet_x\dsc_resources\ | group extension

Count Name                      Group
----- ----                      -----
  690                           {AuditPolicyDsc, OfficeOnlineServerDsc, SecurityPolicyDsc, SharePointDsc...}
    8 .ResourceHelper           {SystemLocaleDsc.ResourceHelper, AdcsDeploymentDsc.ResourceHelper, CertificateDsc.ResourceHelper, Co...
  179 .psd1                     {AuditPolicyDsc.psd1, AuditPolicyResourceHelper.strings.psd1, MSFT_AuditPolicyCsv.strings.psd1, MSFT...
  464 .psm1                     {AuditPolicyResourceHelper.psm1, MSFT_AuditPolicyCsv.psm1, MSFT_AuditPolicyOption.psm1, MSFT_AuditPo...
  389 .mof                      {MSFT_AuditPolicyCsv.schema.mof, MSFT_AuditPolicyOption.schema.mof, MSFT_AuditPolicySubcategory.sche...
    2 .Util                     {OfficeOnlineServerDsc.Util, SharePointDsc.Util}
    1 .ContentDatabase          {SharePointDsc.ContentDatabase}
    1 .Farm                     {SharePointDsc.Farm}
    1 .ProjectServer            {SharePointDsc.ProjectServer}
    1 .Search                   {SharePointDsc.Search}
    1 .WebApplication           {SharePointDsc.WebApplication}
    1 .WebAppPolicy             {SharePointDsc.WebAppPolicy}
    1 .dll                      {ProjectServerServices.dll}
    1 .config                   {output.config}
   23 .cs                       {wcf.Admin.cs, wcf.Archive.cs, wcf.Calendar.cs, wcf.CubeAdmin.cs...}
   25 .ps1                      {Assert-HADC.ps1, Assert-ParentChildDomains.ps1, MSFT_xADCommon.ps1, GeneratexADRecycleBinSchema.ps1...
    5 .Common                   {CertificateDsc.Common, ComputerManagementDsc.Common, NetworkingDsc.Common, StorageDsc.Common...}
    1 .PDT                      {CertificateDsc.PDT}
   14 .mfl                      {MSFT_xPowerPlan.schema.mfl, MSFT_xArchive.schema.mfl, MSFT_xEnvironmentResource.schema.mfl, MSFT_xG...

Installed using puppet module install
PS C:\ProgramData\PuppetLabs\code\modules\dsc> gci -rec .\lib\puppet_x\dsc_resources\ | group extension

Count Name                      Group
---- ----                      -----
 690                           {AuditPolicyDsc, OfficeOnlineServerDsc, SecurityPolicyDsc, SharePointDsc...}
   8 .ResourceHelper           {SystemLocaleDsc.ResourceHelper, AdcsDeploymentDsc.ResourceHelper, CertificateDsc.Re...
 179 .psd1                     {AuditPolicyDsc.psd1, AuditPolicyResourceHelper.strings.psd1, MSFT_AuditPolicyCsv.st...
 464 .psm1                     {AuditPolicyResourceHelper.psm1, MSFT_AuditPolicyCsv.psm1, MSFT_AuditPolicyOption.ps...
 389 .mof                      {MSFT_AuditPolicyCsv.schema.mof, MSFT_AuditPolicyOption.schema.mof, MSFT_AuditPolicy...
   2 .Util                     {OfficeOnlineServerDsc.Util, SharePointDsc.Util}
   1 .ContentDatabase          {SharePointDsc.ContentDatabase}
   1 .Farm                     {SharePointDsc.Farm}
   1 .ProjectServer            {SharePointDsc.ProjectServer}
   1 .Search                   {SharePointDsc.Search}
   1 .WebApplication           {SharePointDsc.WebApplication}
   1 .WebAppPolicy             {SharePointDsc.WebAppPolicy}
   1 .dll                      {ProjectServerServices.dll}
   1 .config                   {output.config}
  23 .cs                       {wcf.Admin.cs, wcf.Archive.cs, wcf.Calendar.cs, wcf.CubeAdmin.cs...}
  25 .ps1                      {Assert-HADC.ps1, Assert-ParentChildDomains.ps1, MSFT_xADCommon.ps1, GeneratexADRecy...
   5 .Common                   {CertificateDsc.Common, ComputerManagementDsc.Common, NetworkingDsc.Common, StorageD...
   1 .PDT                      {CertificateDsc.PDT}
  14 .mfl                      {MSFT_xPowerPlan.schema.mfl, MSFT_xArchive.schema.mfl, MSFT_xEnvironmentResource.sch...

Built the module locally on my mac, copied to a test vm, installed and verified. Everything is 🎉


[storage] implement read_joint

use in get_joint network request

should be a simple wrapper for #34

added in #83
Duplicate of #34

Duplicate of #34


Automatically optimize image size

ML image sizes can be very large, particularly when using Nvidia base images.
I wonder if there are ways we can automatically reduce the size of the image? Some quick wins could be removing caches, and all that stuff.
However we can probably get bigger wins. Unlike Docker images generally, Cog images have a predictable layout with predictable stuff in them, so we could do clever things to reduce image size. Perhaps we can detect what things you're actually using in your code, and remove them if they aren't needed. Perhaps we can detect weight files / caches that aren't actually used in your code and remove them.

I looked into that a few months ago, by running a prediction, scanning the container file system, and removing any files with atime less than when the container was booted.
That approach didn't work because Docker didn't correctly update atime on file access.
There may be some less automatic approaches though like you say. For example, the nvidia docker images include a lot of development libraries that probably aren't used during prediction.

I looked into that a few months ago, by running a prediction, scanning the container file system, and removing any files with atime less than when the container was booted.
That approach didn't work because Docker didn't correctly update atime on file access.
There may be some less automatic approaches though like you say. For example, the nvidia docker images include a lot of development libraries that probably aren't used during prediction.

The Nvidia images do include a lot of development libraries. I've switched over to using the DockerHub PyTorch runtime containers which are much smaller. The only issue is some ML code assumes you have development libraries available (i.e. lazily compiling custom CUDA extensions first time you run the model). In some of those cases I'm using Docker multi-stage builds with a build container based on the PyTorch devel containers that have those tools to build as much as I can and then copy the artifacts over to the production runtime based container.

Docker-slim might be an option? If we knew the possible inputs / input types we could do some fuzzing against Cog's built-in HTTP server and use that as the docker-slim probe.
I'm not sure how well docker-slim would work on our images, but it'd be worth trying on a couple of models.


building and browsing site but cannot view examples

This must be a trivial problem.  After forking and cloning p5.js-website and following instructions
I can get website up and running however when I try to view one of the examples I get the following
Cannot GET /examples/motion-bouncy-bubbles.html
How do I correctly build the site with working examples to browse?

Is it only one of the examples or all of them?

All of them

I'm not able to replicate the problem. You should be able to just clone the repo, run npm install then grunt server and it should all work. If you are still having problem, can you check in the cloned folder and in dist/examples/ whether there are any files in there after running grunt or grunt server.

output from grunt server shows only one file generated in dist/examples/index.html and I have checked folder and its only the index.html file there.  No errors reported during running of grunt server

That's rather unusual, can you try starting from a fresh clone of the repo?

just directly cloned repo locally and built it
I do get some warnings and have run npm audit fix
still same outcome however

Maybe don't run npm audit fix, just clone the repo, run npm install then grunt server.

okay so cloning and building repo definitely fails.  However I have directly downloaded zip file and unpacked, run npm install and grunt server and it successfully builds and runs site with examples

I am also getting this type of error.
Cannot GET es/zh-Hans/examples/
but I go to the homepage in this and then come to the examples then this will not occur.

@mcuz There shouldn't be a different between cloning and downloading a ZIP in terms of building the site. I cannot say much else though as I cannot replicate this issue but if you manage to build the site sucessfully one way or another that's OK I guess.
@17bcs029 es/zh-Hans/examples/ is not a valid link (es being the Spanish translation and zh-Hans being the Chinese translation).

Git for Windows can change the end of line characters in files. This breaks findName() in src/data/examples/build_examples/build.js

@takyano That made sense, I think that should be it! Good work finding that.

yes I am using a windows platform (take it that you are all using linux or mac?).  so my settings for git should be checkout as-is commit as-is (would be good if someone could verify).  I will try settings and see if it fixes the problem.  @takyano well spotted if thats the case

verification: changed git autocrlf setting to false
cloned repo, npm install, grunt build, grunt server
site and examples now working!
thanks @takyano @limzykenneth


Tensor dimension error on tiiuae/falcon-7b

python -m helm.benchmark.run \
    --enable-huggingface-models tiiaue/falcon-7b \
    --run-specs mmlu:subject=anatomy,method=multiple_choice_joint,model=tiiaue/falcon-7b \
    --local --max-eval-instances 5 --suite tiiaue_falcon-7b \
    --local-path ./envs/tiiaue_falcon-7b \
    2>&1 | tee $tiiaue_falcon-7b_mmlu

results in the following tensor error

It would be nice to add the falcon models to the set of evaluations.

I've also had issues with it: https://github.com/huggingface/peft/issues/685 if I get it to work I will ping y'all.

it also has eos = pad token issue which might be an issue if you fine tune the model e.g., it (un) learns to predict eos: https://stackoverflow.com/questions/76633368/why-does-the-falcon-qlora-tutorial-code-use-eos-token-as-pad-token

We've add Falcon using Together's inference API.


cached resolution: don't include callers from evicted modules

This is the last of cached resolution fix for 0.13.9 series that I made last week, and released 0.13.9-M4.
Unfortunately the issue that a client was having did not resolve.
I am thinking about releasing 0.13.9 with it as is without going through another RC.
/review @jsuereth, @dwijnand

LGTM, shame to have a mutable Map floating around, but it's limited to the scope of the (rather large) method.


Use an external browser for vsc.view

I would like to use an external web browser while keeping the default behavior of View() function provided by this extension. However, the following option just falls back to the utils::View():
options(vsc.view = FALSE)

Could you please add an option argument to address this need?

What are you trying to view? A html file, a url, an htmlwidget, a shiny app, etc.?

Data.frame object using View() function like below:
View(airquality)

In your case, you need to make the data frame an htmlwidget to view it in a web browser.
You may try the following:
options(vsc.viewer = FALSE)
DT::datatable(airquality)

Thank you very much for your kind response, @renkun-ken. I was wondering if there would be any way to get View(...) to open in an external browser while keeping the default behavior of show_dataview(...) function defined within ~/.vscode-R/init.R.
I know, I could use DT::datatable(airquality) or gt::gt(airquality); however, I am looking for a way to open a data in an external browser via View(airquality).


FD Analysis - calculation of optical constants for sample and reference with thickness difference <0.1mm

When calculating the absorption coefficient of a sample of 3.15 mm thickness and a reference of 3.10 mm thickness the optical constants are not coming out correctly. In the time-domain tab I can see that the respective thickness information is entered correctly in the .thz file but on the frequency-domain tab in the box 'FD Data Analysis' the thickness is only shown as 3.1 mm in each case.
It looks like the calculation is carried out with the 3.1 mm thickness rather than the correct 3.10 and 3.15 mm thickness, i.e. the thickness difference between sample and reference of 0.05 mm.

The value display format of the thickness edit fields are now changed to '%6.3f,' allowing for the accuracy of 1 micron.


Tables v2

Front: https://github.com/katalysteducation/adaptarr-front/pull/27
I've realized that to allow user setting he's own [de]serialization rules we need only export those:
parseHtml,
deserializeRules: deserialize,
serializeRules: serialize,
render

So now on the front end we can use built int serializer or just create our own.
I've changed caption to be handled as text('caption') and the normalization should change it to figure/table_caption.

I'm not sure we want tables in admonitions or exercises, but this PR certainly shouldn't change default schema to include tables if they are being implemented outside cnx-designer.

In Introductory Statistics there is 89 tables inside notes and 214 tables inside exercises, so schema should be changed to accept those elements.
I don't see reason to create separate PR for updating schema for those elements or to do this from outside because it has to be here anyway.


Implement new rawStatusCode methods on ServerHttpResponse

The following change https://github.com/spring-projects/spring-framework/commit/547342b27d39efdb7ef6de068897b972e85c7b23 makes it possible to provide non-standard status codes as integer. However ArmeriaServerHttpResponse would need to implement the new methods in order to take advantage of this.

@trustin
May I try to solve this?
I would like to implement a setStatusCode(Integer statusCode) and then it will get non-standard HttpStatus by HttpStatus.valueOf(statusCode).
If anyone has a good idea, please let me know. 🙇♂

Spring framework has been released 5.2.4. However, Spring Boot is not released with it yet.


DNM fix memory buffer issues with system calls

...TBD...
just getting some CI results

Codecov Report

Merging #7049 into master will increase coverage by 18.66%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #7049       +/-   ##
===========================================
+ Coverage   58.62%   77.28%   +18.66%
===========================================
  Files         464       10      -454
  Lines       47430      273    -47157
  Branches     8787       30     -8757
===========================================
- Hits        27804      211    -27593
+ Misses      15795       42    -15753
+ Partials     3831       20     -3811

Impacted Files
Coverage Δ

tests/ztest/include/ztest_assert.h
25% <0%> (-8.34%)
:arrow_down:

tests/ztest/src/ztest.c
52.85% <0%> (-7.9%)
:arrow_down:

subsys/bluetooth/host/crypto.c

ext/lib/crypto/mbedtls/include/mbedtls/cipher.h

tests/lib/rbtree/src/main.c

subsys/net/ip/ipv6.h

subsys/net/ip/tcp.h

subsys/net/ip/net_core.c

subsys/net/ip/ipv6.c

include/arch/posix/asm_inline_gcc.h

... and 435 more

Continue to review full report at Codecov.

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

This is still very WIP but has most of the concepts discussed in #6972.

This is now ready for review.

Working through some CI issues:

Thread resource patch is pulling in mempool code that was previously discarded even if usused, need to figure out what's going on, causing a failure on grove light test on Quark D2000
Weird xtensa crash on a seemingly unrelated test
native_posix tests failing for some reason
tests/kernel/mem_protect/mem_protect failing on minnowboard

@dbkinder  updated

CC  @pizi-nordic

The CI failure is baffling me.
It is not SDK dependent. The ARC version of objdump is segfaulting when generating zephyr.lst.
From bisection the offending patch is the one that introduces k_queue support.
Really need to get this in, so I'm going to drop the k_queue patch from this PR and introduce separately while I figure this out.


integrations: Add grafana.

Grafana has a webhooks system, so this should be pretty straightforward:
https://grafana.com/docs/alerting/notifications/#webhook
You can follow the instructions for adding a new webhook integration at https://zulipchat.com/api/incoming-webhooks-overview

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

hi @rishig, I am new here, can you elaborate on how to solve this.

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

@illseption I have already done this and made a PR. You can look try another issue.

Ah okay. I thought the issue was free because you had been unassigned.

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

Hi @rishig
I am very new to the Zulip codebase but would like to make contributions. I am currently planning to integrate Grafana to Zulip. I should setup Grafana webhook first. And I am thinking that we do not need install Grafana in the vagrant env or I should install it in my local machine in order to test functionality. Any advice?
Thanks

Resolved.

claim

@zulipbot claim

#12951 added a commit to akashaviator/zulip that referenced this issue

@zulipbot abandon

@zulipbot claim

@zulipbot  abandon

@zulipbot claim

@zulipbot claim

@zulipbot claim

@zulipbot claim

Can somebody please guide me how to proceed with this issue? Thanks :)

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot  claim

@zulipbot claim

@zulipbot claim

My PR is ready for review.


Set default readOnly value to false

Fixes #4280.
Changes proposed in this pull request:

Set default readOnly value to false

Codecov Report

Merging #4483 into master will decrease coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4483      +/-   ##
============================================
- Coverage     55.36%   55.23%   -0.14%
  Complexity      330      330
============================================
  Files           952      953       +1
  Lines         17984    18026      +42
  Branches       3384     3408      +24
============================================
- Hits           9957     9956       -1
- Misses         7382     7425      +43
  Partials        645      645

Impacted Files
Coverage Δ
Complexity Δ

...ngjdbc/jdbc/adapter/AbstractConnectionAdapter.java
73.52% <ø> (-0.26%)
6 <0> (ø)

...ingsphere/sql/parser/visitor/SQLServerVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...hardingsphere/sql/parser/visitor/MySQLVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ngsphere/sql/parser/visitor/PostgreSQLVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...ardingsphere/sql/parser/visitor/OracleVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...phere/sql/parser/visitor/impl/SQL92DDLVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...here/sql/parser/visitor/impl/OracleDDLVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...hardingsphere/sql/parser/visitor/SQL92Visitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

...phere/sql/parser/visitor/impl/MySQLDDLVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

.../sql/parser/visitor/impl/PostgreSQLDDLVisitor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 9821

0 of 0   changed or added relevant lines in 0 files are covered.
721 unchanged lines in 14 files lost coverage.
Overall coverage decreased (-0.1%) to 58.876%

Files with Coverage Reduction
New Missed Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/ddl/constraint/ConstraintDefinitionSegment.java
1
0%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/util/IpUtils.java
3
76.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/sql/statement/ddl/CreateTableStatement.java
5
0%

shardingsphere-sql-parser/shardingsphere-sql-parser-engine/src/main/java/org/apache/shardingsphere/sql/parser/sql/segment/ddl/column/ColumnDefinitionSegment.java
6
0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQL92DDLVisitor.java
32
0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sqlserver/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/SQLServerDDLVisitor.java
58
0%

shardingsphere-sql-parser/shardingsphere-sql-parser-postgresql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/PostgreSQLDDLVisitor.java
60
0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/OracleDDLVisitor.java
65
0%

shardingsphere-sql-parser/shardingsphere-sql-parser-sql92/src/main/java/org/apache/shardingsphere/sql/parser/visitor/SQL92Visitor.java
75
0%

shardingsphere-sql-parser/shardingsphere-sql-parser-mysql/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/MySQLDDLVisitor.java
77
0%

Totals

Change from base Build 933:
-0.1%

Covered Lines:
10613

Relevant Lines:
18026

💛  - Coveralls


"CMakeLists.txt not found" when running idf.py

I'm running a develop shell directly from the flake ref as in the readme, but when I try to run an idf.py command, I get an error:
$ idf.py erase-flash
Executing action: erase-flash
CMakeLists.txt not found in project directory /home/jonah

Am I missing a step here?  FWIW, that path is not a Git repo.

idf.py commands only work in idf project directories.

I expect this isn't a bug in this project, so I will close.


Exchange: Quote not calculated on first attempt

Subject of the issue
When visiting the Exchange and selecting a base and quote token, and then clicking "Max" for the base token, the values for the quote do not calculate.
This only works if the user deletes the base values and enters a value manually.
Your environment

OP Mainnet
Chrome, latest

Expected behaviour
On click of "Max", both base and quote should be calculated immediately.
Actual behaviour
https://user-images.githubusercontent.com/548702/216041346-a692df20-840c-44e7-9686-815948b2e19f.mov
Steps to reproduce

Visit Kwenta
Connect wallet with some token holdings
Visit Exchange, select base token (of which wallet holds tokens) and quote token
Click MAX

This is more like an RPC issue, but returning a more helpful message to the user would be better.


Is recursion supported?

I'm new to JSON schema validation and am trying to use recursion for an array item type like the example below.
e.g.
{
    "$schema": "http://json-schema.org/draft-07/schema#",

    "definitions": {
        "person": {
            "type": "object",
            "properties": {
                "name": { "type": "string", "pattern": "^[\s\S]*$" },
                "children": {
                    "type": "array",
                    "items": { "$ref": "#/definitions/person" },
                    "default": []
                }
            }
        }
    },

    "type": "object",

    "properties": {
        "person": { "$ref": "#/definitions/person" }
    }
}

I have tried for days to fix this error but I keep on getting: [requireValidation] type should be specified at #/properties/children/items
Any advice on how to fix this please?
edit: I forgot to mention that I was using the parser api

@DeanRae Sorry for the long delay.

Parser uses strong mode by default, which imposes extra checks on the schema.
There was a misdetection in those extra checks in recursive refs, that is now fixed in v1.0.0-rc.4
It will still (deliberately) fail to compile that schema though, unless additionalProperties and required are specified:

{
    "$schema": "http://json-schema.org/draft-07/schema#",

    "definitions": {
        "person": {
            "type": "object",
            "required": [],
            "additionalProperties": false,
            "properties": {
                "name": { "type": "string", "pattern": "^[\s\S]*$" },
                "children": {
                    "type": "array",
                    "items": { "$ref": "#/definitions/person" },
                    "default": []
                }
            }
        }
    },

    "type": "object",

    "required": [],
    "additionalProperties": false,
    "properties": {
        "person": { "$ref": "#/definitions/person" }
    }
}

Also note that \ should be escaped in strings, so in order to write down ^[\s\S]*$ into a string, "^[\\s\\S]*$" should be used.

I'm going to close this as a7a5b55 (included in v1.0.0-rc.4) fixed the issue reported here, and the rest are non-issues.
Please feel free to comment if there are more concerns.


BUG: fix syntax error in action workflow

To address https://github.com/datalad/datalad-catalog/issues/151
see: https://github.com/datalad/datalad-catalog/pull/149#issuecomment-1176784340

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@35d1df8). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #154   +/-   ##
=======================================
  Coverage        ?   76.02%
=======================================
  Files           ?       21
  Lines           ?     1360
  Branches        ?        0
=======================================
  Hits            ?     1034
  Misses          ?      326
  Partials        ?        0

Flag
Coverage Δ

unittests
76.02% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


Add TypeScript support

Implement TypeScript support. Add tests (all are fine).
https://github.com/isagalaev/highlight.js/issues/484

any updates?


update fix vertx plugin NPE

Please answer these questions before submitting pull request

Why submit this pull request?

[x] Bug fix

[ ] New feature provided

[ ] Improve performance

Related issues
https://github.com/apache/skywalking/issues/3338

Bug fix

Bug description.
my project use vertx v3.4 ,the agent log show
java.lang.NullPointerException
at org.apache.skywalking.apm.plugin.vertx3.HttpServerResponseImplEndInterceptor.beforeMethod(HttpServerResponseImplEndInterceptor.java:39)

How to fix?
I review code and debug, the context is null, so I find there are not use class io.vertx.ext.web.impl.RouterImpl in my project
finally,I solve it by I create io.vertx.core.http.impl.ServerConnection enhance, it works healthy

New feature or improvement

Describe the details and related test reports.

/run ci

/run ci

There are no test case in this plugin

We need integration test case here, https://github.com/SkyAPMTest/agent-auto-integration-testcases

Your target branch is still wrong. And your ci fails.
Please fix.

@wentjiang Are you still following this?

@wu-sheng I think it is a usage question,but I not very be familiar with vertx, in my usage,I don't use the plugin enhance class , so I find an available class to enhance;I think this question need the plugin author to  judge;

If you have a new use case, the plugin should enhance. I closed this because you pr target is not master.

I know ,I change target to master branch

@wentjiang please submit a new pull request. And provide plugin tests for it.


Array concatenation

First, thanks for your share, I have a question about Array concatenation.
When i run it in chrome console:
const ArrayConcat = (arr, ...args) => [...arr,...args];
ArrayConcat([1], [1, 2, 3, [4]])

// The answer is => [1, [1, 2, 3, [4]]];
// This is different from the document. -_-

// see the print
const AC = (arr, ...args) => {
    console.log(arr, args);
};
AC([1], [1, 2, 3, [4]]);  // [1] [[1, 2, 3, [4]]]
AC([1], [1, 2, 3, [4]], [5, 6]); // [1] [[1, 2, 3, [4]], [5, 6]]

Is anything wrong with me?

There's nothing wrong with your browser. This is a very interesting bug, thanks for the heads up!

Nothing wrong with you. The [..arr, ..args] is simply the other way of writing Array.concat() for 2 arrays. The function in this state cannot make an array out of multiple subarrays.

Could https://lodash.com/docs/#flatten help in this situation?

Why we must pass the second parameter of function as a spread?
I want a flat array, but get multidimensional one.
const concArrays = (oneArray, ...anotherArray) => [...oneArray, ...anotherArray];
concArrays([1], [2, 3, 4]) // -> [1, Array(3)];

But if i will make the second parameter without spread all will be ok:
const concArrays = (oneArray, anotherArray) => [...oneArray, ...anotherArray];
concArrays([1], [2, 3, 4]); // -> [1,2,3,4]

As far as I am concerned, the concatenation is problematic for one main reason (apart from the obvious one that being it not working as expected): We have not really improved over the native Array.concat(), so what we are doing is exactly the same thing. lodash's concat accepts a first argument that can be something apart from an array, for example.
Now, this is a problem because we are showing off a snippet that does not work properly and does nothing that JS cannot natively achieve. Therefore I am closing all related issues and removing the snippet. Move this discussion in #100 where we decide which lodash functions are too much of a polyfill to be implemented and we can decide if we want to start rewriting this or drop it forever.


fix(jsx/dom): Do not call insertBefore if the element position does not change

Fixed the same problem as #2016 when one child is updated in an element having multiple children.
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

@usualoma
Thanks! I'll merge it now.


What command to issue for compiling?

Sorry for the silly question, but I've just installed D (dmd and ldc) for the first time. My Excel is 64 bit, and I copied in the example folder the right xlcall32 lib. But the command  'dub build --arch=x86_64' , from the example folder, ends with lot of errors, related to function xlld.wrap.wrap.stringAutoFreeOper not callable

should i use dmd or ldc?
should i first issue a command in the root folder? which?
which is the correct command from the example folder ?

Sorry, a nogc version update broke excel-d. To build the command after the fix, just use dub build --arch=x86_64 in the example directory.


[sysrst_ctrl] ec_rst_l_o released earlier than expected

Description
after disabling ec_rst_l_o override (PIN_OUT_CTL.EC_RST_L) post reset, it should be asserted for 2000 cycles (reset default of sysrst_ctrl.EC_RST_CTL)
but it is released earlier in some of the instances of sysrst_ctrl_combo_detect testcase. This is observed on PR https://github.com/lowRISC/opentitan/pull/17232
test can be reproduced by
 util/dvsim/dvsim.py hw/ip/sysrst_ctrl/dv/sysrst_ctrl_sim_cfg.hjson -i sysrst_ctrl_combo_detect  --tool xcelium --waves shm --fixed-seed 161966062
Error log:
UVM_ERROR @<PHONE_NUMBER> ps: (sysrst_ctrl_base_vseq.sv:36) [uvm_test_top.env.virtual_sequencer.sysrst_ctrl_combo_detect_vseq] Check failed (act_cycles inside {[exp_cycles - 3 : exp_cycles + 3]}) act(1395) vs exp(2000) +/-3

Thank you for finding this issue! Definitely not right.
I dumped the wave and this is what I found:
1). The counter "timer_cnt_q" has incremented from 0 -> 82 with one frequency, then with another frequency from 82 -> 2000.
2). When calculated the cycles, the testbench uses the current (slower) aon_timer period:
https://github.com/lowRISC/opentitan/blob/master/hw/ip/sysrst_ctrl/dv/env/seq_lib/sysrst_ctrl_base_vseq.sv#L25
I think the DV issue here is to assign the aon_clk frequency too late.
https://github.com/lowRISC/opentitan/blob/master/hw/ip/sysrst_ctrl/dv/env/seq_lib/sysrst_ctrl_base_vseq.sv#L55
I am going to make a PR to fix it. But please correct me if I missed anything here. (I am still pretty new to this block)

@cindychip , Good catch. Thanks for looking in to this.
The change ca52d2, will update the dut_init task for all the sequences.
It would be better to override the task in sysrst_ctrl_combo_detect_vseq as this is the one that requires the change

#17244 fixes the issue


Add body battery

hey @elliottyu19 thank you for fixing this, do you have the ability to ship the new release (0.0.14) as an actual release and push to pypi?   We are seeing errors related to this very often on our Terra endpoint.
thanks!


Added a simple keylogger

Pull Request for PyVerse 💡
Requesting to submit a pull request to the PyVerse repository.

Issue Title
Please enter the title of the issue related to your pull request.
Add a simple keylogger code

[x] I have provided the issue title.

Info about the Related Issue
What's the goal of the project?
The goal of the project is to create a simple keylogger
Describe the aim of the project.
This program is a basic keylogger that captures keystrokes and logs them into a text file until the Esc key is pressed

[x] I have described the aim of the project.

Name
Please mention your name.
Deanne Vaz

[x] I have provided my name.

GitHub ID
Please mention your GitHub ID.
https://github.com/djv554

[x] I have provided my GitHub ID.

Email ID
Please mention your email ID for further communication.
<EMAIL_ADDRESS>
[x] I have provided my email ID.

Identify Yourself
Mention in which program you are contributing (e.g., WoB, GSSOC, SSOC, SWOC).
Contributor at GSSoC and Hactkoberfest

[x] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
*Closes: #333

[x] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
This program is a basic keylogger that captures keystrokes and logs them into a text file. It uses the pynput library to listen to keyboard events. The keylogger logs both normal and special keystrokes into the file and stops when the Esc key is pressed.

[x] I have described my changes.

Type of Change
Select the type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how your changes have been tested.
Describe your testing process here.
Ran it locally on my system

[x] I have described my testing process.

Checklist
Please confirm the following:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

@UTSAVS26 please do review this I have another pending issue.

@UTSAVS26 please review this request as I have another pending issue to complete.


accesstoken in headers is not getting updated when token is renewed

open func startAuthorizedRequest(_ url: String, method: OAuthSwiftHTTPRequest.Method, parameters: OAuthSwift.Parameters, headers: OAuthSwift.Headers? = nil...) {
return self.client.request(url, method: method, parameters: parameters, headers: headers, body: body, success: success) { (error) in
switch error {
        case OAuthSwiftError.tokenExpired:
            _ = self.renewAccessToken(withRefreshToken: self.client.credential.oauthRefreshToken, headers: renewHeaders ?? headers, success: { (credential, _, _) in
                // Ommit response parameters so they don't override the original ones
                // We have successfully renewed the access token.

                // If provided, fire the onRenewal closure
                if let renewalCallBack = onTokenRenewal {
                    renewalCallBack(credential)
                }

                // Reauthorize the request again, this time with a brand new access token ready to be used.
                _ = self.startAuthorizedRequest(url, method: method, parameters: parameters, headers: headers, body: body, onTokenRenewal: onTokenRenewal, success: success, failure: failure)
            },
                                      failure: failure)

}
here after token renewing previous token is used in headers(i mean same headers of first request are getting passed in 2nd request) in -=self.startAuthorized() request
first request after token renewal is failing every time and getting
OAuthSwiftResponse:{
error = "invalid_request";
"error_description" = "Missing form parameter: grant_type";
}
This problem i am getting only when token expiration is handled by library when i am checking token expiration and sending request no error like this is coming.Please help.

i think this is because in OAuthSwiftHTTPRequest.swift
public mutating func updateRequest(credential: OAuthSwiftCredential)
line 406  self.urlRequest.allHTTPHeaderFields = requestHeaders + headers
requestHeaders contains Authorization:Bearer already and in headers also Authorization Bearer is present in library both are getting added .Please Help

Please help I am stuck only in this except this everything is working

hello sir,
i got this solved by adding this
if requestHeaders.count > 0{
headers.removeValue(forKey: "Authorization")
}
self.urlRequest.allHTTPHeaderFields = requestHeaders + headers
in public mutating func updateRequest(credential: OAuthSwiftCredential) {
}

Please do not add new message asking just for help. Do u know that people sleep and work :p
So you want authorization from requestHeaders, not headers with priority now
The question is why there is redundant header. I need to read the code later

hi i want to know whether this function will be called for making all headers or only when token is refreshed
/// Allow to customize computed headers
public protocol OAuthSwiftCredentialHeadersFactory {
func make(_ url: URL, method: OAuthSwiftHTTPRequest.Method, parameters: OAuthSwift.Parameters, body: Data?) -> [String: String]
}


Remove empty content encoding

This will help us fix the blank "Content-Encoding" file header in S3

:+1:


Security Group IDS have been added to the instances

chore: added security group ids for all instances

Approved.


Grab Additional Hardware Details in Redfish Script

There are some additional hardware details we can try to get from Redfish:

[ ] Graphics cards
[ ] Accelerators
[ ] NIC's
[ ] MAC Addresses
[x] BMC Addresses
[x] Physical Location (Cabinet, Rack, etc.)

Graphics cards may be under redfish/v1/Systems/{SystemName}/PCIeDevices, could possibly check the "name" or "description" field and parse under "graphics" or "video"


Some mini bugs and improve

Im using your latest version v11 and im facing some mini issues like

the sleep mode cant fully sleep like windows, its only turn off the screen, and the fans keep running too.
sometime the speakers cant output sound and need reboot

i saw that asussmc have updates on catalina so could you bring it back again ?
Thank you.

Next week I will send V12 .


[Storage-preview] Modify the processing logic of content-md5 and add related tests

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines
In Track 2 the content_md5 is eventually serialized to a base-64 string and added to the header:
self._serialize.header("file_content_md5", file_content_md5, 'bytearray') --> Serializer.serialize_bytearray

So before the SDK fixes the problem, we need to deserialize content_md5 into a bytearray manually

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

[x] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Storage-preview

Storage-preview


fix: github link in footer

Issue: Replace link for Github [footer]

I have updated the GitHub link in the config.yml file as per the instructions provided in the issue.

Previous Link: https://github.com/FunPythonEC/Hacktoberfest-GYE-2024

New Link: https://github.com/Hacktoberfest-GYE/Hacktoberfest-GYE-2024

resolve issue #14 thks @Yeasir-Hossain

Thanks @Yeasir-Hossain for the help.

You are most welcome @jhon-p16 and @davcortez. This is my first open source contribution so I started with beginner friendly issues. It means a lot.


[WIP] Add a runner template

This adds a template runner, which serves as a template for custom runners. The template runner is provided for convenience of new runner development only, it cannot be used as an actual runner.
part of runner spec implementation #66
depends on #73
closes #68

Features in this PR have been implemented in other PRs.


[BUG] k8s-ci failed in gremlin query stage

Describe the bug
See detailed report in https://github.com/alibaba/GraphScope/runs/7552550498?check_suite_focus=true
oading vertex labeled _ and edge labeled _: 100%|██████████| 10/10 [00:02<00:00,  4.07it/s]
2022-07-28 11:19:30,868 [INFO][coordinator:1186]: subgraph has been loaded
Error: -28 03:19:30,898 [ERROR][rpc:187]: Runstep failed with code: INTERACTIVE_ENGINE_INTERNAL_ERROR, message: Error occurred during preprocessing, The traceback is: Traceback (most recent call last):
  File "/home/graphscope/.local/lib/python3.8/site-packages/gscoordinator/coordinator.py", line 910, in _fetch_gremlin_result
    rlt = result_set.all().result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 444, in result
    return self.__get_result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 389, in __get_result
    raise self._exception
  File "/home/graphscope/.local/lib/python3.8/site-packages/gremlin_python/driver/resultset.py", line 90, in cb
    f.result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 437, in result
    return self.__get_result()
  File "/usr/lib/python3.8/concurrent/futures/_base.py", line 389, in __get_result
    raise self._exception
  File "/usr/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/graphscope/.local/lib/python3.8/site-packages/gremlin_python/driver/connection.py", line 83, in _receive
    status_code = self._protocol.data_received(data, self._results)
  File "/home/graphscope/.local/lib/python3.8/site-packages/gremlin_python/driver/protocol.py", line 138, in data_received
    raise GremlinServerError(message['status'])
gremlin_python.driver.protocol.GremlinServerError: 597: op type {class com.alibaba.graphscope.common.intermediate.operator.ScanFusionOp} returns {TableNotExistError, msg is table Str("\u{6}") does not exist}

Additional context
Add any other context about the problem here.

Have passed all ci tests before merged. No idea what is the issue? Hi a rerun anyway

Closing as rerun succeeded.

The k8s-test keeps failing with the exactly same reason:

https://github.com/alibaba/GraphScope/runs/7596818539?check_suite_focus=true
https://github.com/alibaba/GraphScope/runs/7596960999?check_suite_focus=true


Added support for "Refresh Ahead" caching strategy

The main changes are:

Updating the pipeline to allow for doing a background refresh of the cache
Remove the use of the EventAwareCacheWrapper for the posts public cache

Background refresh
This is just an initial implementation, and tbh it doesn't sit right with me that the logic for this is in the pipeline - I think this should sit in the cache implementation itself, and then we call out to it with something like: cache.get(key, fetchData) and then the updates can happen internally.
The cache-manager project actually has a method like this called wrap - but every time I've used it it hangs, and debugging was a pain, so I don't really trust it.
EventAwareCacheWrapper
This is such a small amount of logic, I don't think it's worth creating an entire wrapper for it, at least not a class based one. I would be happy to refactor this to use a Proxy too, so that we don't have to add methods to it each time we wanna change the underlying cache implementation.

Merged by accident 😬


Buttons that navigate to a new page will only load the footer

When pressing "get to know us" (Homepage), or "Get in touch"(About Page) they will navigate to the page but display only the footer. All browsers

using the stencil-route-link instead of the a tag seems to solve this issue solved in this commit

New homepage design no longer navigates to a new page and just scrolls to meet the team. Closing Ticket


HDDS-4713. Add build-helper-maven-plugin to interface-client module

What changes were proposed in this pull request?
In ozone/interface-client module, sources files are generated by protobuf  plugin. But when other modules depend on this module, idea always prompts "Cannot resolve symbol" message.

What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4713
How was this patch tested?
Intellig IDEA.

I found if run mvn install then the Intellij will be able to find such proto classes.
Does this build-helper-maven-plugin skip the step of mvn install in command line?

I found if run mvn install then the Intellij will be able to find such proto classes.
Does this build-helper-maven-plugin skip the step of mvn install in command line?

I found if run mvn install then the Intellij will be able to find such proto classes.
Does this build-helper-maven-plugin help skipping the step of mvn install in command line (thus better)?

Hi, @amaliujia, mvn install seams not work in my local machines(at least two computers, withing Intellij IDEA Ultimate 2020.3).
When using build-helper-maven-plugin, Intellij Idea works ok (need run mvn package )

I found if run mvn install then the Intellij will be able to find such proto classes.
Does this build-helper-maven-plugin help skipping the step of mvn install in command line (thus better)?

Hi, @amaliujia, mvn install seams not work in my local machines(at least two computers, withing Intellij IDEA Ultimate 2020.3).
When using build-helper-maven-plugin, Intellij Idea works ok (need run mvn package )

Actually I guess there are two things you can try:

Mark the directory as the generated source by intellij

It is possible that the intellij has marked the generated source, however the file size parameter is too small thus intellij does not load the specific large class/java file (the generated proto file sometimes can be very large).

Actually I guess there are two things you can try:

Mark the directory as the generated source by intellij

It is possible that the intellij has marked the generated source, however the file size parameter is too small thus intellij does not load the specific large class/java file (the generated proto file sometimes can be very large).

Thanks for your detail @amaliujia . You're right, we need increate the idea.max.intellisense.filesize.
By the way, if we use build-helper-maven-plugin, the idea will automatically recognizes the source files.

Thanks for your detail @amaliujia . You're right, we need increate the idea.max.intellisense.filesize.
By the way, if we use build-helper-maven-plugin, the idea will automatically recognizes the source files.

Though I haven't checked what build-helper-maven-plugin does, I believe Intellij does not require to manually mark those source directories (at least on version 2020) :-)

Though I haven't checked what build-helper-maven-plugin does, I believe Intellij does not require to manually mark those source directories (at least on version 2020) :-)

ok, thanks @amaliujia : )

ok, thanks @amaliujia : )


Update alerts-activity-log.md

Cross-posting relevant note from the concepts article.
https://docs.microsoft.com/en-us/azure/azure-monitor/platform/activity-log-alerts

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

#sign-off .  Looks good @dcbrown16 . Thanks for the change.


chore: Add go.work and remove replace directive

Fixes #
Description
Adds go.work in /test over replace directive. There are a couple reasons for this:

This is the recommended way to reference local modules after 1.18
This means that we can update our package dependencies go.mod and test/go.mod without them affecting each other (which has been a headache as of late)

How was this change tested?

make build
TEST_FILTER=Integration make e2etests

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

Release Note

None

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks like you need to check in test/go.work.sum


Allow for specification of custom code path

In specific scenarios, a user may wish to override the code property used when instantiating the SingletonFunction for this Construct. ECRDeploymentProps should hence be changed to allow this.
Background
CloudFormation templates can be synthesized from CDK equivalents, though this synthesis is lossy. Specifically, since CloudFormation is purely declarative, we are unable to invoke an action unlike CDK.
As a result, a CDK deployment may be able to build the code needed for a Lambda, but this cannot be done in the synthesized CloudFormation equivalent. A workaround might be to reference the code as a Docker image in a private ECR, but the image has to be made available pre-deploy. Lambda also cannot refer to public ECR images, even with the use of pull-through caches.
This is problematic for a maintainer of a CloudFormation template intended for non-technical audiences, who may only be familiar with interacting with AWS through the console.
Proposal
As a workaround, the maintainer may wish to provide the Lambda code for cdk-ecr-deployment on a public S3 bucket, so that the construct is usable even in a synthesized CloudFormation template. This would need a change to the construct to accept an optional code property, which can be used to override the equivalent property in SingletonFunction.

Once you are overwriting the code prop, what benefits does this package even provide to you?

This is problematic for a maintainer of a CloudFormation template intended for non-technical audiences, who may only be familiar with interacting with AWS through the console.

I don't think this is a use case this package set out to support originally. Its main purpose is to publish images that have just been build using CDK to a well-known location. Can you describe more how the scenario you describe would work in practice? What images are copied from where to where? Where is the source image coming from and how is it build?

Once you are overwriting the code prop, what benefits does this package even provide to you?

Allowing code to be specified allows me to host the bootstrap code found in this package within lambda/ somewhere else, specifically in a location supported by CloudFormation.
In the context of using synthesized CloudFormation,  a maintainer would continue to benefit from the constructs in the package, that declares the lambda containing the relevant bootstrap code (only this time provided through public S3 bucket), as well as the custom resource that invokes it.

I don't think this is a use case this package set out to support originally. Its main purpose is to publish images that have just been build using CDK to a well-known location.

This is understood; the issue proposes making a change so that even if the CDK templates are synthesized, there is a path to allowing this to function.
I am happy to fork this repo with our own changes so that it serves our own intended purpose.

Can you describe more how the scenario you describe would work in practice? What images are copied from where to where? Where is the source image coming from and how is it build?

Our main use case is - through CloudFormation synthesized from CDK for reasons described earlier - to deploy a Lambda which uses a Docker image for its code.
The code for the Lambda has to be a Docker image as it requires additional binaries not usually found in the default Lambda environment. In turn, CloudFormation cannot build images, and requires Lambda docker images to be hosted on private ECR.
The source Docker image is to be found on Docker Hub, prebuilt for the CloudFormation template. Destination would be a private ECR declared in the template.
An alternate approach of building the image from source using CodeBuild (because we are severely limited by CloudFormation and hence cannot build directly) was considered, but rejected, since we wish to keep deploy times and moving parts to a minimum.

We've elected to reuse the constructs from this repo through copying, as they are simple enough, and loading the bootstrap code as a .zip in a public S3 bucket. Thank you for considering this issue!


Running in background

If I run RAN in the background how can I stop it or start it again?

Stop: Use ps command to find the pid of ran, and then use kill command to stop it.
example:
ps -C ran -o pid= | xargs -I {} kill {}
Start ran in background:
ran -l &

perfect, also the below code also worked fine to stop the process
kill -9 $(lsof -t -i:8080)


pull request for e2e test at 12/11/2016 07:15:02

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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.


Extend builder to pass member access by dot '.' notation string

As a developer I want to be able to specify member to acces using string dot '.' notation.
Our frontend just send to us the member path to order on in dot notation form.
This also ease the way we access Json fields in our backend.
public class NestedJsonModel
{
    public int Id { get; set; }
    public NestedInnerJsonModel Inner { get; set; }
}

public class NestedInnerJsonModel
{
    public int Id { get; set; }
    public DateTime Created { get; set; }
    public JsonDocument Data { get; set; }
}

var context = query..KeysetPaginateQuery(
			b => b.Ascending("inner.data.nbString"),
			KeysetPaginationDirection.Forward);

For Json property we also need an extra parameter to convert the JsonElement to the value type.
It can also be use in other cases for type convertion.
MethodInfo converter = typeof(JsonElement).GetMethod(....);
var context = query..KeysetPaginateQuery(
			b => b.Ascending("inner.data.nbString", converter),
			KeysetPaginationDirection.Forward);

I'm hesitant about this as well. Not having proper indexing completely defeats the point of doing keyset pagination. Your code snippet above is saying you can create a keyset out of anything and everything, which doesn't really make sense.
If you have well known cases (which you can index properly), then I recommend your frontend send some kind of enum (or whatever you like) that identifies which case you want, and then switch on that and build your keyset. Having this dynamic string support will encourage creating keysets out of anything, which is something I don't want.


Is there an end or quit method?

The example has spooky.run(). How do we quit and ensure all spawned phantomjs / casperjs processes are also killed?

spooky.destroy() gives an exception and leaves phantomjs childs.


Collect coverage data when rendering widgets.

Resolves #94

Okay, this doesn't actually collect coverage on real code.
It seems to be because line_number_range(frame) bails out if the self object is an instance of django.template.base.Template, which the template used for rendering widgets is, even when using the django template backend/engine.
@nedbat do you recall why this is?
https://github.com/nedbat/django_coverage_plugin/blob/ca45852f7168d36b69e6a64f01d804b572775680/django_coverage_plugin/plugin.py#L233

Ah, think I figured that out - it's the nodes within a template that have the line numbers available.
The issue is that form templates use a different rendering thing, which is in a different python file, so is being omitted by file_tracer.

Ah, turns out it needs settings.FORM_RENDERER to be set to 'django.forms.renderers.TemplatesSetting'


Got into the same problem for several times.
Looks like Twitch is preventing bots from claiming?

Hello o/
This is a duplicate of #138 and is already fixed in the latest master, please see discussion there.


temporary-fix(big bundle & cannot-resolve-symbol): revert to non-optimized bundling until it's fixed in qwik

TLDR:  I think it would be best to keep the unoptimized bundle until a real fix can be made in the optimizer or until we're ready for v1 if the optimizer issue hasn't been fixed by then.
What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests

Description
I removed
      output: {
        preserveModules: true,
        preserveModulesRoot: 'packages/kit-headless/src',
      },

from the headless-kit's vite config, because it creates the namespace issues described in #519, for which there is no great workaround in terms of DX.
This brings back the big bundle issue, but the "big bundle" isn't so big on first load anyway, and subsequent requests will be loaded from the browser cache.
To make the bundle smaller, I removed country-list-json and libphonenumber-js dependencies and all related code (which didn't receive any attention for the past 6 months). We can always add it back later or create another package for those kinds of components if the bug hasn't been fixed by then.
Performance should be okay now (11kb on first load :point_down:)

:warning:  I reused some of the code from #511, updated to latest qwik version, and made sure that everything works in dev and preview (the workspace dependency was causing issues to the build step). I suggest to first merge this PR and then @shairez you can work on top of it for the utils package.
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

In case anyone's looking into this, I had a programming session with Shai and we think we know what's causing the namespace issues when using preserveModules, so we're holding off for now.

closing this in favor of the recent fixes


Update upgrade test

Incorporated few more scenarios for upgrade.
Signed-off-by: Khushboo<EMAIL_ADDRESS>

highlights from the suggested code snippet

Use fixtures to setup prerequisite

We will not need pre_upgrade, because those setup will be included in other tests.
Use module scoped fixtures will be more suite in here.

Use Class based to group tests, let verify points isolated.

We will not need post_upgrade, because those verify points should be able to verify independently and separately.
And use dependency make verify tests skipped when upgrade test failed.

Use fixture to store cluster states, share between tests.

Use state fixture to let the configured setup could be shared after upgraded.

Use single_node, any_nodes, three_nodes to mark test scope.

and we can filter tests to fit cluster's nodes, but it will need to write in other file, so not included in the code snippet.

The change will bring the flexibility for us to run upgrade tests like:
# to test single node upgrade
pytest -m "upgrade and single_node"
# to test negative upgrade
pytest -m "upgrade and negative"
# to test 3 nodes upgrade
pytest -m "upgrade and three_nodes"

We can also restructure test cases to group in a class marked with any_nodes to reduce test cases, but in here, I just demonstrate how could we use the mark as flexibility as possible.

import pytest

@pytest.fixture(scope="module")
def cluster_state(self):
    class ClusterState:
        pass

    return ClusterState()

@pytest.fixture(scope="module")
def cluster_prereq(self, api_client, cluster_state):
    # create cluster network
    # create a routable vlan
    cluster_state.clusternetwork = "cluster network name"
    cluster_state.vm_network = ("name", "vlanid", "cluster network name")
    cluster_state.vlan = "routable vlan id"

@pytest.fixture(scope="module")
def vm_prereq(self, api_client, cluster_state):
    cluster_state.old_target = "existing backup target" or ""
    # setup backup target
    # create new storage class
    # create 3 VMs:
    # - having the new storage class
    # - the VM that have some data written, take backup
    # - the VM restored from the backup
    # check VMs should able to reach each others (in same networks)

    cluster_state.storage_class = "Created storage class name"
    cluster_state.backup_name = "backup name from the VM"
    cluster_state.backup_target = "backup_target"
    cluster_state.vms = {
        "backed up": "backuped VM",
        "restored": "restored VM",
        "storage class": "VM having different storageclass"
    }

@pytest.mark.upgrade
@pytest.mark.any_nodes
@pytest.mark.negative
class TestInvalidUpgrade:
    def test_invalid_manifest(self):
        """
        Criteria: https://github.com/harvester/tests/issues/518
        """

    def test_degraded_volume(self):
        """
        Criteria: create upgrade should fails if there have any degraded volumes
        """

@pytest.mark.upgrade
@pytest.mark.single_node
class TestSingleNodeUpgrade:
    @pytest.mark.dependency(name="single_node_upgrade")
    def test_perform_upgrade(self):
        pass

@pytest.mark.upgrade
@pytest.mark.three_nodes
class TestThreeNodesUpgrade:
    @pytest.mark.dependency(name="three_nodes_upgrade")
    def test_perform_upgrade(self, cluster_prereq, vm_prereq):
        """
        - perform upgrade
        - check all nodes upgraded
        """

    @pytest.mark.dependency(depends=["three_nodes_upgrade"])
    def test_verify_logging(self):
        """ Verify logging pods and logs
        Criteria: https://github.com/harvester/tests/issues/535
        """

    @pytest.mark.dependency(depends=["three_nodes_upgrade"])
    def test_verify_network(self, api_client, cluster_state):
        """ Verify cluster and VLAN networks
        - cluster network `mgmt` should exists
        - Created VLAN should exists
        """

    @pytest.mark.dependency(depends=["three_nodes_upgrade"])
    def test_verify_vms(self, api_client, cluster_state):
        """ Verify VMs' state and data
        Criteria:
        - VMs should keep in running state
        - data in VMs should not lost
        """

    @pytest.mark.dependency(depends=["three_nodes_upgrade"])
    def test_verify_restore_vm(self, api_client, cluster_state):
        """ Verify VM restored from the backup
        Criteria:
        - VM should able to start
        - data in VM should not lost
        """

    @pytest.mark.dependency(depends=["three_nodes_upgrade"])
    def test_verify_storeage_class(self, api_client, cluster_state):
        """ Verify StorageClasses and defaults
        - `harvester-longhorn` should be settle as default
        - `longhorn` should exists
        """

@lanfon72 Thanks for the nice suggestion! I incorporated those in the PR.


Ray parallel fix

Description of changes:
cap dask and distributed to fix parallel. re-enable parallel
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Innixma There was some issue for the previous PR because I was using a branch that pushed the disabling code. This one should be good

Job PR-1417-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1417/1/index.html

Job PR-1417-2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-1417/2/index.html


docs: README remove old info

connexion 3 is 6 month old

Hi @raphaelauv, most incoming issues are still about migrating to connexion 3, so I would prefer to keep this in for now.


UnAckedMessageTracker: 3600 messages have timed-out... but they haven't?

Describe the bug
~I've deployed a topic into Pulsar 2.4.1 with default configuration for message TTL, which as I understand it should mean that unacked messages should be retained forever.~
UPDATE: I've realized the UnAckedMessageTracker does something different than I thought it did. The behavior is still unexplained though.
I've deployed a topic into Pulsar and my consumers are configured with ackTimeout = 60 minutes. However, my consumers always either negative ack or ack messages. There may be rare situations in which neither of these happen (application crash or network problem), but for some reason my consumers consistently and periodically report:
org.apa.pul.cli.imp.UnAckedMessageTracker : [ConsumerBase{subscription='my-sub', consumerName='my-consumer', topic='persistent://dev/my-ns/q'}] 3600 messages have timed-out

I've seen this on 3 separate environments, starting the topic and associated subscriptions from scratch. Subscriptions are Shared, and configured to start at position "earliest".
The value is always 3600. In all 3 environments, which have vastly different numbers of messages in the topic, the number is always 3600. In some environments I don't believe 3600 messages were even placed into the topic, so it shouldn't be possible for there to be 3600 unacked messages.
This doesn't actually appear to cause any problems -- everything seems to be working normally. Its just an annoying message to see.
To Reproduce
Steps to reproduce the behavior:

Create a topic with default settings, change retention size to -1 and retention period to 1 hour (I can't see how these settings would be relevant, but that's what my topics have).
Produce and consume some messages from the topic, ensure they are all acked or nacked.
Wait until the warning is reported.

Expected behavior
No warning.

I have seen the same.

will look into it.


Are Expired Ads Checked For?

Preloaded ads are typically expired after 1 hour, but I'm unable to find anything in the code handling expired ads.
Does this library correctly handle expired ads?

Actually I'm going to go ahead and close and reopen if I test and see an issue.


Not clustering custom markers without coordinate prop to direct children

Hi,
first of all thanks for your work.
I was a bit surprised to notice the following behavior. I have a MapScreen class, that renders Markers through another EventMarkerPin class (only showing parts of the code):
MapScreen:
class MapScreen extends Component<Props, State> {
  renderEventMarkerPins = () => {
    const {visibleEventIds} = this.props;
    return visibleEventIds.map(eventId => (
      <EventMarkerPin key={eventId} eventId={eventId} />
    ));
  };
  render() {
    return (
      <MapView>
        {this.renderEventMarkerPins()}
      </MapView>
    )
  }
}

EventMarkerPin:
class EventMarkerPin extends Component<Props> {
  render() {
    return (
      <Marker
        identifier={eventId}
        flat
        coordinate={coordinate}
        anchor={anchor}
        tracksViewChanges={false}
        onPress={this.openEvent}>
        {markerView}
      </Marker>
    );
  }
}

As you can see, I'm not passing any coordinate prop to EventMarkerPin, but I'm getting it with mapStateToProps:
const mapStateToProps = (state, ownProps) => {
  return {
    coordinate,
  };
};
export default connect(mapStateToProps)(withNavigation(EventMarkerPin));

Checking out your code, I found isMarker (https://github.com/venits/react-native-map-clustering/blob/master/lib/helpers.js) and realized, that I had to pass the coordinates to EventMarkerPin for it work.
It would be great if the package would support custom components as Markers that get their coordinates individually from the redux store, instead of having to pass them from the MapView level.

@tguelenman There was similar discussion on #134

Fixed in module version 3.2.0
Thank you for your contribution ❤

Out of curiosity, how was this fixed in 3.2.0?
It seems passing the coordinate prop directly to the custom marker is still necessary in order for clustering to work. Unfortunately, in my application, I'm fetching the markers' coordinates in the function body of the custom marker itself, and feeding the custom marker only an id which it uses to query our backend for the coordinate data. This is a necessary workaround for my particular use case.
Because I'm using a useQuery hook to fetch the coordinate for each map result, there really isn't anywhere higher in the component tree to perform that fetching because the results list will vary in length and I get RN errors for having varying numbers of hook calls between renders.


port: remove trusted URL logic

Ports changes from https://github.com/microsoft/botbuilder-dotnet/pull/4827

We can wait until the C# PR lands.