Update Copyright year

Describe the bug

We just need to change the copyright year from 2020 to 2022 which is written on the sign-up card.
Add any other context about the problem here.

If it is a valid issue can I work on this issue?

Yep sure you can work on this issue cause it is still open.
But before that please confirm on slack whether the issue is valid .
Cause I have observed many organizations and Inc update their copyright year.
Example: Github

I have fixed this issue and made a Pull Request. Kindly review it.


fix(payment): STRIPE-0000 bump checkout-sdk-js v1.309.5

What?
Bump checkout-sdk-js version
Why?
Fixes for the tickets:
STRIPE-160
STRIPE-167
STRIPE-183
STRIPE-193
STRIPE-195
Testing / Proof
@bigcommerce/checkout @bigcommerce/apex-integrations

Thanks @BC-AOrtiz can you please link the SDK prs in this bump. Since this is an open source repository and folks outside BC don't have access to JIRA.

Thanks @BC-AOrtiz can you please link the SDK prs in this bump. Since this is an open source repository and folks outside BC don't have access to JIRA.

Sure, let me add the PRs related

We just identified an issue in SDK.
Could you please bump SDK to v1.309.6 so that CI checks can pass?


TypeError: flameChart.setData is not a function

The example of README doesn't worked,
import FlameChart from 'flame-chart-js'

throw no default error, so
import { FlameChart } from 'flame-chart-js'

Then flameChart = new FlameChart({...}), and it works, but flameChart.setData is undefined

Oops, sorry, the readme is a little outdated.  You can use the setNodes method to set some data like this in the example: https://github.com/pyatyispyatil/flame-chart-js/blob/096773686ca188b634c19cc481dd31019a3c8333/example/src/components/charts/flame-chart-wrapper.tsx#L96
I will update readme soon.)

How to update the data of FlameChartContainer? Or just remove/destroy and renew?

It seems that we can use FlameChartPlugin.setData to update the data. So, how can we remove FlameChartPlugin from FlameChartContainer?

Method setData from FlameChartPlugin works the same as method setNodes from FlameChart class (method setNodes is a wrapper over method setData from FlameChartPlugin). Are you sure you need to remove this plugin?)
In general, if you need some kind of customized view, then you should use the container without everything and fill it with plugins yourself, as is done in the FlameChart class.

Oh, okay. You can do the same as in this examples:
https://github.com/pyatyispyatil/flame-chart-js/blob/master/example/src/components/charts/custom-flame-chart.tsx
https://github.com/pyatyispyatil/flame-chart-js/blob/master/example/src/components/charts/flame-chart-container-wrapper.tsx


Update atom.desktop.ini

In this pull I am editing the Categories and MimeType fields, for these reasons respectively:

Deleted the GNOME category as, as that's for GNOME core apps, which Atom is not.
MimeType should include all default (i.e., with the default bundled packages) supported file types for the editor. Most importantly, it would be delightful if one could open up project folders with Atom using your file manager's context menu.

I think this is a very good job, but I still want to study harder


[webapi] Update Presentation API tests

These tests are followed specification: https://w3c.github.io/presentation-api/
(Presentation API W3C Editor's Draft 22 December 2015)
Impacted TCs num(approved): New 27, Update 0, Delete 0
Unit test Platform: Crosswalk Project for Android 19.48.485.0
Unit test result summary: Pass 11, Fail 7, Blocked 9
BUG=https://crosswalk-project.org/jira/browse/XWALK-5686

@chunywang could you please provide testing results based on latest Crosswalk Project library? Thanks.

Result for  Presentation API cases on Crosswalk for Android(19.48.485.0 ),
(Pre-Condition: Turn on Settings -> Developer options -> Simulate secondary displays)

PresentationRequest.getAvailability() can work
PresentationAvailability.value are always false, so onchange eventhandler can't be fired
PresentationRequest.start(), reconnect() throw an InvalidAccessError exception

Please don't merge the PR as there are some case errors. i will update the cases as soon as possible

(Pre-Condition: Turn on Settings -> Developer options -> Simulate secondary displays)

This shall be documented in REAMDE.md file.


seeds: add violet subdomain dummy data

resolves #628

add violet subdomain
seed to look like violet.restarone.solutions
add skeleton fake data (can be augmented by a faker library as a next step)

Great first effort @yashkir -- lets use Faker along with the other changes I suggested above. After you attach screenshots of the Freshly Seeded State :seedling: we can deploy this to staging

Added Faker and removed the subdomain. Should be fairly straightforward to extend these with fancier markup/date if needed:

@yashkir Faker looks good! Can you attach some pictures of the web pages? Eg: violet.restarone.solutions landing page

@yashkir Faker looks good! Can you attach some pictures of the web pages? Eg: violet.restarone.solutions landing page

As mentioned in the review reply above, the images are not imported and show as blank. We could leave this as is, or import a minimal set (e.g. just the violet rails/restarone logos).

@yashkir good call on it bloating source. But I think we should include it and strip it later (as part of a different task) during deployment.
Can you go ahead and include the assets as well? So it looks just like violet.restarone.solutions

@yashkir Since CI is passing and it looks like you have linked assets, I'm creating a base branch for you. Please use that branch for changes (if any) moving forward 👍


Ability to customize styles without creating a new theme

Problem
As a user of this library I would like to customize the styles using some external CSS.
Suggested Solution
Inside typedoc.json add a property — customCSS where any file path for CSS can be added.
{
  "customCSS": "../theme/style.css"
}

I was trying to move the menu bar from right to left, and this works. Just need to create style.css and define the css rules inside the class that we want to modify.


bigint type is missing

I tried to create a table using Perique_Migrations with a foreign key reference to wp_posts(ID) but I noticed that there are only unsigned_int and unsigned_medium.

now merged, again sorry for neglecting this and thanks for the PR


Fix exception message in Consul.Builder#withMultipleHostAndPort

The message states that the blacklistTimeInMillis must be positive, but it accepts zero as well as positive numbers.
For now, fix the error message to reflect the actual logic. Later, we may want to change the logic so it accepts only positive values.
While maybe this could technically be called a 🐛 , I've labeled it as an enhancement instead, since it does not actually affect the way the code works in any way.

Also see discussion #319


RosNavClient::send_pose() return Ok even if timeout!!

It's not clear what should send_pose() returns, but i think it's a bug.

[ ] Make it clear what should send_pose() returns
[ ] fix it

In the #144, there are discuss about the naming of send_pose()
Navigation::send_pose() -> Navigation::send_goal_pose()

send_pose() is not send_goal_pose().
It's actually move_to.

We decided to quit to use await/async and use wait().
So, it's send_goal_pose(), not move_to() now.


Second click on item removes mm-active class

version:
@version 3.0.4
Description
When i click on item second time on the same item mm-active class gets removed. This is unexpected behavior. It's breaks requirements to show active page. When menu item doesn't have sub-menu & user clicks on same menu-item it triggers area-expand behavior. which should not be the case since item doesn't contain sub-menu.

I'm also having this problem. My quick solution is by doing this
$('#metismenu li a').on('click', function(){
    $(this).attr('aria-expanded', 'true')
    $(this).parent().addClass('mm-active')
})

It does not remove active menu when clicking outside the metis menu, which I assume a bug. However for my project it is not bother me at all

@AmirinNS,
I am also facing the same issue in the active parent class.  Submenus are collapsing but active class not coming on the parent.
BTW your solutions helped me at this time.
Thanks.

@LL,
Problem solved. It was conflicting with another tab.
Thanks.


ItemsRepeater has incorrect render result when scroll back to previous contents

Describe the bug
Some text will disappear when scrolling back to the previous elements in ItemsRepeater
To Reproduce

Create an ItemsRepeater with ItemTemplate.
Scroll up and down to see items disappear

Expected behavior

Elements should be the same when scrolling up and down

Screenshots

Desktop (please complete the following information):

OS: Windows 11 22H2
Version: Avalonia Latest Nightly Build

Any updates for this issue?


test(pallet-api/fungibles): ensure codec indexes

Codecov Report
Attention: Patch coverage is 95.43269% with 19 lines in your changes missing coverage. Please review.

Project coverage is 51.51%. Comparing base (4b9da05) to head (4f5bfc6).
Report is 3 commits behind head on daan/api.

Files with missing lines
Patch %
Lines

pallets/api/src/fungibles/tests.rs
95.48%
0 Missing and 18 partials :warning:

pallets/api/src/fungibles/mod.rs
93.75%
1 Missing :warning:

@@             Coverage Diff              @@
##           daan/api     #288      +/-   ##
============================================
+ Coverage     48.89%   51.51%   +2.61%
============================================
  Files            47       47
  Lines          4532     4785     +253
  Branches       4532     4785     +253
============================================
+ Hits           2216     2465     +249
  Misses         2254     2254
- Partials         62       66       +4

Files with missing lines
Coverage Δ

pallets/api/src/mock.rs
100.00% <ø> (ø)

runtime/devnet/src/config/api/versioning.rs
90.60% <100.00%> (ø)

pallets/api/src/fungibles/mod.rs
92.89% <93.75%> (+7.41%)
:arrow_up:

pallets/api/src/fungibles/tests.rs
97.04% <95.48%> (-2.96%)
:arrow_down:


Using the nodemcu reset mechanism

Hi,
is it possible to use the nodemcu auto-reset mechanism, that uses RTS and DTR, also in this WebSerial API? Would be nice to use this tool also with boards that don't have the reset and boot buttons.
Best Christian

we dont have that, if you can submit a PR to do it, it would be great! (we're using it with the ESP32-S2 which does not have auto-reset)


complex support for tf.math.argmin and tf.math.argmax

This PR tries to address the issue raised in #41437
tf.math.argmin and tf.math.argmax don't support complex
The issue was that there aren't kLt, kGt, kLe and kGe ComparisonDirection in opcode kCompare of complex. Besides, MinValue and MaxValue in xla/literal_util.cc didn't support the minimum and maximum value for complex.
This PR adds complex support for tf.math.argmin and tf.math.argmax.

In numpy, it looks like np.argmax is defined as the lexi order of the tuple (real_part, Imag_part) based on https://github.com/numpy/numpy/issues/10469
The following verifies:
>>> import numpy as np
>>> np.argmax([1+0.5j, 1-0.5j])
0
>>> np.argmax([1-0.5j, 1+0.5j])
1

Should the behavior match numpy instead? /cc @tensorflow/api-owners

+1 to @yongtang 's comment

+1. In Math there is no comparison between two complex numbers, so if we want to implement that support then we should make sure to follow existing practices

Argmax returns the indices of the maximum values ignoring NaNs.
In numpy, np.argmax return the first number that is NAN when there are nans in array, and np.nanargmax returns the indices of the maximum values ignore nan. Currently, the behavior of argmax in tensorflow ignores nans.
Should tensorflow be consistent with numpy at this point?
Numpy 1.18.1, Python3.8:
import numpy as np
print(np.argmax([complex(3,4), complex(1, np.nan), complex(np.nan, 2)]))

Output
1

However, the numpy documentation does not seem to mention it, this may be undefined behavior?
In addition , when I try to implement complex support for argmax and argmin, I found it is strange that lhs_value and rhs_value seem to be swapped in tensorflow/compiler/xla/client/lib/arithmetic.cc:CreateMinMaxComputation, when the input is complex type. This causes my argmax always choose the last nan if there are nans in tensor...

Let's also add some test where we compare several numbers. Scenarios I would include:

Numbers with same real part
Numbers with same imaginary part
Numbers all around the complex space (maybe some roots of unity?)

It would be nice to have somewhere centrally in the Tensorflow documentation what the expected behavior on complex numbers is and why. That principle would then determined the behavior of all operations. For example this choice of using the real component would suggest preserve the
elementary operations (addition, multiplication) but use the real component norm as norm function as a general principle for complex numbers. But there is at least one paper on 'complex neural networks', like this one: https://arxiv.org/abs/1705.09792. And poking around on stackoverflow some frustration about incomplete support for complex numbers https://stackoverflow.com/questions/43934487/minimizing-loss-in-a-complex-valued-network-in-tensorflow

I try to add test cases in tensorflow\compiler\tests\argminmax_test.py. However, it seems that there are some problems in the existing code, the following code cannot pass even in the master branch.
def testNaN(self):
    minmax_types = self.all_types & {np.int32, np.int64}
    for dtype in self.float_types:
      for output_type in minmax_types:
        # argmax NAN tests (ignore nan in tensor)
        self._assertOpOutputMatchesExpected(
            math_ops.argmax,
            axis=0,
            output_type=output_type,
            op_input=np.array([np.nan, 6.0, 7.0, -1.0, 4.0, np.nan, -50.0], dtype=dtype),
            expected=output_type(2))

Output
AssertionError:
Arrays are not equal

not equal lhs = array(6)
not equal rhs = array(2)
Mismatched elements: 1 / 1 (100%)
Max absolute difference: 4
Max relative difference: 2.
 x: array(6)
 y: array(2)

@agarwal-ashish any suggestions on what the semantic should be?

@agarwal-ashish any suggestions on what the semantic should be?

@wangpengmit  Can you please take a look on this PR ? Thanks!

I try to add test cases in tensorflow\compiler\tests\argminmax_test.py. However, it seems that there are some problems in the existing code, the following code cannot pass even in the master branch.
def testNaN(self):
    minmax_types = self.all_types & {np.int32, np.int64}
    for dtype in self.float_types:
      for output_type in minmax_types:
        # argmax NAN tests (ignore nan in tensor)
        self._assertOpOutputMatchesExpected(
            math_ops.argmax,
            axis=0,
            output_type=output_type,
            op_input=np.array([np.nan, 6.0, 7.0, -1.0, 4.0, np.nan, -50.0], dtype=dtype),
            expected=output_type(2))

Output
AssertionError:
Arrays are not equal

not equal lhs = array(6)
not equal rhs = array(2)
Mismatched elements: 1 / 1 (100%)
Max absolute difference: 4
Max relative difference: 2.
 x: array(6)
 y: array(2)

Is it because all_types has unsigned integers?

@jstzwj  Can you please address Ubuntu Sanity errors? Thanks!

Thanks for making this contribution!
I think it might be nicer for the complex argmin/argmax to explicitly extract the real/complex components rather than defining minimum and maximum for complex values. I think this should result in the same performance but avoids having to define an ordering of complex elements at the HLO level.

@jstzwj  Any update on this PR? Please. Thanks!

I agree with @majnemer - unless we explicitly want to support the same semantics as numpy, it's probably best not to implement argmin/argmax for complex types, but instead force the user to be explicit about the semantics.
Defining argmin/argmax for complex will force us to also define these semantics elsewhere, for all backends (XLA-HLO, Eigen, etc...).  In Eigen we will not implement complex comparison functions like this, since there is no mathematical basis for it - so TF will need to maintain its own internal workaround.

@jstzwj Any update on this PR? Please. Thanks!

@jstzwj Can you please resolve conflicts? Thanks!

@jstzwj  Any update on this PR? Please. Thanks!

It has been 14 days with no activity and the awaiting response label was assigned. Is this PR still valid? Assigning the stalled label. Please comment to reassure me that this is still being worked on.

@jstzwj Any update on this PR? Please. Thanks!

It has been 28 days that this pull-request has stalled. Please create a new pull-request with the requested changes.


Make CloudWatch logs retention longer than 1 day

So that we can go back and inspect older events

Set to 30 days.


[Figma] Insight Report : Cell

Description
Fill out the Cell insight report from your findings from the audit
The insight report will be part of our decision making framework and is intend to:

Document all findings from the component audit
Confirm as many component details as possible
Mitigate component inconsistencies across Figma, Mobile and Extension

Include your thoughts on component name, description, api and any comments or topic to discuss relating to the component or it's make up. We will review the audit and insight report in our Wednesday technical sync to finalize the details of the component for all platforms.
Technical Details
The insight report should include the following for the component

name
description
variants/props
requirements (optional)
discussion/questions (optional)

Acceptance Criteria

name, description variants/props is filled out and matches with other platforms where possible

References

[FigJam]https://www.figma.com/file/YmFeHzySNf6jzbowCoY0qr/Cell-Audit?node-id=0%3A1&t=Q8fSXZxdSWbvYkze-1
Read exercised #05 Identify Existing Paradigms in Design and Code and #06 IdentifyEmergingandInteresting Paradigms in Design and Code in the Design System in 90 Days workbook

Hey team! Please add your planning poker estimate with Zenhub @Akatori-Design @garrettbear @brianacnguyen


Fix dependency bug and add documents

Fix bug: change zeppelin-interpreter/pom.xml, package the interpreter with dependencies.
Add documents: EH and CN
Remove the duplicate README in zeppelin-interpreter/

To be discussed:
As the Github image website githubusercontent.com is unstable, Shall we create an image directory for all used images and refer to them with relative links? just like what  does.
For example, the images in UserGuide/Ecosystem Integration/Grafana.md may not display normally now.

As the Github image website githubusercontent.com is unstable, Shall we create an image directory for all used images and refer to them with relative links? just like what  does.

I think it may cause another problem. When we clone the code from the IoTDB repository, we have to download these images to our local computers. These images may take a lot of disk space....

As the Github image website githubusercontent.com is unstable, Shall we create an image directory for all used images and refer to them with relative links? just like what  does.

I think it may cause another problem. When we clone the code from the IoTDB repository, we may have to download these images to our local computers. These images may take a lot of disk space....

I understand and quite agree that we should separate image resources from the code repository.
The change has been done.
Just a few extra ideas. github.com  and githubusercontent.com  have different web links and ips, thus users may be able to view the former but blocked by the latter. In the future, we could consider storing our images in another github.com repository, instead of githubusercontent.com , although it's really convenient.


Resolving the text overflow in Divs ( "What comes after implementation" and "About" section)

Fixes
fixes #76 by @shubhanshu02
Description
The PR resolves the text overflow occuring in the Div section i.e

What comes after implementation
About

When viewed on devices with a width between 768px to around 1300px.
Technical Details
changed some hardcoded widths and heights from these cards , for the pt-card and pt-text-card classes:
Tests

Open the "ABOUT" section of the page.
Adjust the width size between 768px to 1300px.
Notice that the text overflow which was occuring previously is rectified.
Follow step 1 and 2 for "WHAT COMES AFTER IMPLEMENTATION" section.

Screenshot

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@renuka142 The PR seems good. Have a look upon the required changes.

Hello @Cronus1007 , sure i will do the above changes.
Thankyou for your remarks!

@renuka142 Have a look about this ss

.

Okay , the two cards needs to be aligned.


Consider using typerep-map for shakeExtra field in ShakeOptions

I've noticed that ShakeOptions has shakeExtra field of type HashMap TypeRep Dynamic
https://github.com/ndmitchell/shake/blob/cff1873a33af6d2388b810ccfb5cda470e38cdea/src/Development/Shake/Internal/Options.hs#L215
Maybe for performance reasons it could be better to use typerep-map:

https://kowainik.github.io/posts/2018-07-11-typerep-map-step-by-step

@ndmitchell That's okay! If the performance is really not a problem then it's better to not introduce new dependencies 👍


Items / paragraphs and buttons re-alignment

Could I work on this particular section to re-align some items? : >

Okay...make sure you did not disturb any other section!! @DEV-YJY

Created PR #13 for this.
Have a look and let me know what you think!

Merged Bruh!! @DEV-YJY


Possibly incorrect theme typings for styled function

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
https://mui.com/guides/interoperability/#theme mentions that you can
const CustomizedSlider = styled(Slider)(
  ({ theme }) => `
  color: ${theme.palette.primary.main};

  :hover {
    color: ${darken(theme.palette.primary.main, 0.2)};
  }
`,
);

However the theme leads to https://github.com/mui-org/material-ui/blob/master/packages/mui-system/src/createTheme/createTheme.d.ts

Expected behavior 🤔
I would expect Theme["palette"] to reference Palette in https://github.com/mui-org/material-ui/blob/master/packages/mui-material/src/styles/createTheme.d.ts
Steps to reproduce 🕹
Steps:

Context 🔦
No response
Your environment 🌎

`npx @mui/envinfo`
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Nevermind, VSCode was autocompleting "wrong" styled function
I had
import { styled } from "@mui/system";
but its supposed to be
import { styled } from "@mui/material/styles";


Dev opt inference

这个PR要做的:

[x] 使用mock功能来支持导入transformers仓库
[x] 单机单卡的opt推理
[x] 单机多卡的opt 模型并行推理
[x] 和hugging face的正确性验证

文档说明:
Mock transformers
This is an application of mock transformers, which can perform distributed inference in LiBai with model under the transformers.
Supported Model

OPT: tensor parallel

Environment
Before running the scripts, make sure to install the library's dependencies:
Install libai
libai installation, refer to Installation instructions
# create conda env
conda create -n libai python=3.8 -y
conda activate libai
# install oneflow nightly, [PLATFORM] could be cu117 or cu102
python3 -m pip install --pre oneflow -f https://staging.oneflow.info/branch/master/[PLATFORM]
# install libai
git clone https://github.com/Oneflow-Inc/libai.git
cd libai
pip install pybind11
pip install -e .

All available [PLATFORM]:

Platform
CUDA Driver Version
Supported GPUs

cu117
>= 450.80.02
GTX 10xx, RTX 20xx, A100, RTX 30xx

cu102
>= 440.33
GTX 10xx, RTX 20xx

cpu
N/A
N/A

Install transformers
refer to transformers installation
python3 -m pip install "transformers>=4.26"

Notes

You need to register a Hugging Face account token and login with huggingface-cli login

python3 -m pip install huggingface_hub

If no command available in the PATH, it might be in the $HOME/.local/bin

 ~/.local/bin/huggingface-cli login

distributed infer OPT
An reimplement of OPT distributed inference in LiBai

 opt inference
Tensor Parallel
Pipeline Parallel

  Support
✔
-

for tensor_parallel=2, run command in libai_root
bash tools/infer.sh projects/mock_transformers/dist_infer_opt.py 2

modify the infer code dist_infer_opt.py according to your own needs:
...
if __name__ == "__main__":
    # set dist config
    parallel_config = DictConfig(
        dict(
            data_parallel_size=1,
            tensor_parallel_size=2, # modify it according to your own needs
            pipeline_parallel_size=1, # set to 1, unsupport pipeline parallel now
            pipeline_num_layers=None,
            )
    )
    dist.setup_dist_util(parallel_config)
    ...
    # initial and load model
    model = AutoModelForCausalLM.from_pretrained("facebook/opt-125m").half() # change your model type 125m~66b
    model._apply(dist.convert_to_distributed_default_setting)
    # initial tokenizer
    tokenizer = AutoTokenizer.from_pretrained("facebook/opt-125m", use_fast=False) # change your model type  125m~66b

阔以加一下License，然后init_env.py的潜力很大，感觉之后可以mock很多任务来兼容

阔以加一下License，然后init_env.py的潜力很大，感觉之后可以mock很多任务来兼容

加了


mitm-socket CertsIpcHandler.onError

run example error
env:
node: 14.17.1
secret-agent: ^1.4.1-alpha.5
LSB Version:	core-9.20170808ubuntu1-noarch:security-9.20170808ubuntu1-noarch
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.4 LTS
Release:	18.04
Codename:	bionic
network location: chinese
yarn add secret-agent success
code:
import secretAgent from "secret-agent";

const { Agent } = secretAgent;

(async () => {
  console.time("agent");
  const agent = new Agent({
    humanEmulatorId: "basic",
    showReplay: false,
  });
  await agent.goto("https://example.org");
  await agent.waitForPaintingStable();
  const title = await agent.document.title;
  await agent.close();
  console.timeEnd("agent");

  console.log("Retrieved from https://example.org", {
    title,
  });
})();

run: node main.js
err:
2021-06-23T12:53:19.148Z ERROR [/home/ss/node_modules/@secret-agent/mitm-socket/lib/CertificateGenerator] CertsIpcHandler.onError {
  error: 'Error: spawn<EMAIL_ADDRESS>ENOENT',
  context: {},
  sessionId: undefined,
  sessionName: undefined
} Error: spawn<EMAIL_ADDRESS>ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:269:19)
    at onErrorNT (internal/child_process.js:467:16)
    at processTicksAndRejections (internal/process/task_queues.js:82:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn<EMAIL_ADDRESS>  path<EMAIL_ADDRESS>  spawnargs: [
    '{"storageDir":"/tmp/.secret-agent","mode":"certs","debug":false,"ipcSocketPath":"/tmp/sa-ipc-certs-fbf922f0-d421-11eb-8010-838a796da3e0.sock"}'
  ]
}
WARNING: Secret-Agent is being run under "root" user - disabling Chromium sandbox! Run under regular user to get rid of this warning.
2021-06-23T12:53:19.277Z ERROR [/home/ss/node_modules/@secret-agent/mitm-socket/lib/MitmSocketSession] ProxyIpcHandler.onError {
  error: 'Error: spawn<EMAIL_ADDRESS>ENOENT',
  context: { sessionId: 'fbfec840-d421-11eb-8010-838a796da3e0' },
  sessionId: 'fbfec840-d421-11eb-8010-838a796da3e0',
  sessionName: 'default-session-2'
} Error: spawn<EMAIL_ADDRESS>ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:269:19)
    at onErrorNT (internal/child_process.js:467:16)
    at processTicksAndRejections (internal/process/task_queues.js:82:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn<EMAIL_ADDRESS>  path<EMAIL_ADDRESS>  spawnargs: [
    '{"rejectUnauthorized":false,"clientHelloId":"Chrome88","tcpTtl":64,"tcpWindowSize":65535,"mode":"proxy","debug":false,"ipcSocketPath":"/tmp/sa-ipc-proxy-fc0e0a80-d421-11eb-8010-838a796da3e0.sock"}'
  ]
}

I think this is because you weren't able to install the library it needs per #261.

@blakebyrnes Sorry, these two running environments are different. I run this abnormal code normally in MAC, but the above error occurred in Ubuntu.
And from the error message, I can't confirm whether it is the running problem of dependency or the dependency is not completely installed, but there is no error in yarn install

Ah, ok. Something seems to have failed in this install too. Good point about error message - will improve that.


Unable to get clientIP

I am using the example "Add true clientIP to request". I have followed the example and deployed a cloudfront example but I am not able to see the ip as part of the request.
function handler(event) {
var request = event.request;
var clientIP = event.viewer.ip;
console.log("test");
//Add the true-client-ip header to the incoming request
request.headers['true-client-ip'] = {value: clientIP};

return request;

}

I was able to fix this


has:image broken?

I've just discovered has:image and has:attachment. But in our own realm has:image gives me nothing, while a lot of images show up in has:attachment. On CZO has:imagegives me two messages, and again has:attachment gives me a lot of messages that I would say has an image.
I figure an image counts as an attachment, but images aren't properly detected.

I can confirm this is broken.  Some quick testing suggests the API is working (Network tab shows results) but the UI is blank, which suggests a web frontend bug.


Fix type null for nullable type parameters not added as type

I've started using PHP 7.1 nullable types and noticed, that Sami is not able to parse them. I saw the issue #264, but it does not fix the issue, at least not for parameters.
For example function setAvatar(?string $data) will result in a type of string, and not string|null as expected. The PHP documentation for that element is @param string|null $data.
This PR fixes this by adding the null type to the type array when resolving hints.

Thank you @CharlotteDunois.


Orbit around the sRGB 3D gamut in CIE XYZ space

As a CG enthusiast,
I want to orbit around the sRGB 3D gamut in CIE XYZ space
so that I can better how display colors work.

This issue was broken down into smaller yet less specific issues


Code review comments

Hi Jay,
I reviewed your code and made some comments.
Let me know if you have any questions.
Hope this helps :)
Thanks,
Chie

The not loading in your browser thing sounds like a local problem. I have tested this in firefox, chrome, and safari without any issues using the gh-pages version which is hosted at  https://jayzamazing.github.io/Pirate-Bartender-Project . I would try clearing your cache and trying again or updating your browser. If the issue persists, please provide additional information such as system you are running this on, browser, and version of browser.

I will look into the other issues, but for these issues I will work on them rather than merging this pull request.

Closed by mistake without addressing issues.

Added comments to style.css, made comments inline in app.js. If the browser is still not reloading the app.js file open a bug rather than a pull request and provide more details. Closing this pull request without merging.

ohh yea, I can see the website now.

👍


Search in Docs does not work.

What happened:
The search in the docs does not give any results even for terms which are present in the documentation.
What you expected to happen:
To give result for the keywords in the docs.
Where can this issue be corrected? (optional)

How to reproduce it (as minimally and precisely as possible):

This issue should be opened in litmus-doc repository.


Int32 overflow in sparse_reshape on Windows

System information

OS Platform and Distribution: Windows 10 64-bit
TensorFlow installed from: Source
TensorFlow version: 1.14.0
Python version: 3.6.7

Describe the current behavior
When a large sparse tensor is reshaped using sparse_reshape on Windows it fails, due to the fact that it uses np.prod to determine whether the number of elements in the old and new SparseTensor are the same. This can fail on Windows, due to the fact that numpy converts python ints to dtype np.int32 on Windows, which will overflow for large dimensions.
Describe the expected behavior
sparse_reshape should work similarly on both Windows and Unix-based systems.
Code to reproduce the issue
import tensorflow as tf
tf.enable_eager_execution()

idxs = tf.constant([[1, 0, 1],
                             [1, 4, 2]], dtype=tf.int64)
values = tf.constant([12, 42])

A = tf.sparse.SparseTensor(idxs, values, (8, 42000, 40000))
tf.sparse.reshape(A, (8, 40000, 42000))

Output:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\thomasoerkild\Anaconda3\envs\tf_gpu\lib\site-packages\tensorflow\python\ops\sparse_ops.py", line 769, in sparse_reshape
    reshaped_size))
ValueError: Cannot reshape a tensor with 555098112 elements to shape [8, 40000, 42000]<PHONE_NUMBER>0 elements).

Other info / logs
The issue might be resolved, by forcing the shape inputted to np.prod to be of type np.int64.

@ThomasOerkild,
Sorry for the late response. This issue was fixed.
import tensorflow as tf

idxs = tf.constant([[1, 0, 1],
                             [1, 4, 2]], dtype=tf.int64)
values = tf.constant([12, 42])

A = tf.sparse.SparseTensor(idxs, values, (8, 42000, 40000))

B = tf.sparse.reshape(A, (8, 40000, 42000))

print(B)

Output:
SparseTensor(indices=tf.Tensor(
[[    1     0     1]
 [    1     3 34002]], shape=(2, 3), dtype=int64), values=tf.Tensor([12 42], shape=(2,), dtype=int32), dense_shape=tf.Tensor([    8 40000 42000], shape=(3,), dtype=int64))

Thank You.


Doesn't it work if the parent has child processes?

~/reptyr-master$ ./reptyr 946227
[-] Process 946489 (gmake) shares 946227's process group. Unable to attach.
(This most commonly means that 946227 has sub-processes).
Unable to attach to pid 946227: Invalid argument
Regards,
Manju

It may work better with -T, which uses a different strategy to attach.
But the default mode of operation cannot attach processes with children.
On Tue, Jun 20, 2017, 08:41 Manjunatha S M<EMAIL_ADDRESS>wrote:

~/reptyr-master$ ./reptyr 946227
[-] Process 946489 (gmake) shares 946227's process group. Unable to attach.
(This most commonly means that 946227 has sub-processes).
Unable to attach to pid 946227: Invalid argument
Regards,
Manju
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nelhage/reptyr/issues/84, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABBVZhYASfxqkfJVpkCz9c5yirZknk9ks5sF-gogaJpZM4N_xxT
.

For me, scala programs run using sbt fail to re-attach despite trying -T on parent sbt process.

now neovim and vim8 support job, so they have suprocesses.
when I use this tools I get:
[-] Process 8411 (python2) shares 8407's process group. Unable to attach.
(This most commonly means that 8407 has suprocesses).
Unable to attach to pid 8407: Invalid argument

same with rsync :-(

I have the same problem when I start a command with sudo that spans a new subprocess with the actual command

did you try the -T command? see https://github.com/nelhage/reptyr/issues/24
it worked for me on raspbian lite where I had started a process over ssh and then ran
sudo reptyr PID -T

note that there seemed to be no response in the terminal on the target machine for a few minutes, until it finally began printing the stdout
Many thanks @nelhage for reptyr and providing the -T -- now I can leave for and not worry about restarting an already 10-hour make process on an rpi zero...

I'm trying to recover a process still running in a docker container and started through "docker exec" (the global window has been lost, but the execution is still running in the background, and i would have been willing to recover the output as it's a very long processing).
By default (without any paramters), reptyr fives the same error messages about child processes, but when using the -T feature, i get something else, does anyone have an idea about it ? (of course i'm still logged inside the container as the same user, which in my case - this is a test container - is the root account (UID=GID=0)
root@7de93b6ab1af:~# reptyr 4863 -T
Unable to attach to pid 4863: Operation not permitted
The kernel denied permission while attaching. If your uid matches
the target's, check the value of /proc/sys/kernel/yama/ptrace_scope.
For more information, see /etc/sysctl.d/10-ptrace.conf

(i have the value 1 in the referenced file, but i'm not sure what it may mean)
Maybe it's just related to how docker works ? (that is, reptyr inside the container is trying to access some processes informations only owned by the underlying kernel ? (hence the errors)

@SR-G It might be due to not starting the container with --privileged (or maybe --cap-add=SYS_PTRACE is sufficient), since reptyr seems to depend on ptrace. I wonder if it might be possible to call reptyr from the host directly, but I wasn't successful when trying in a DinD container... Would be cool to have support for this, though :)

@SR-G Not sure if this still is a topic for you, probably not, but the value in the referenced file (ptrace_scope) should be 0 AFAIK.

Any update on this error?
Child is not connected to a pseudo-TTY. Unable to steal TTY.

Maybe add a hint that -T might work to the "This most commonly means that" message ?

did you try the -T command? see #24
it worked for me on raspbian lite where I had started a process over ssh and then ran
sudo reptyr PID -T

note that there seemed to be no response in the terminal on the target machine for a few minutes, until it finally began printing the stdout. just writing this to encourage patience :)
Many thanks @nelhage for reptyr and providing the -T -- now I can leave for and not worry about restarting an already 10-hour make process on an rpi zero...

Hi @michaelpalumbo! I've tried the command and it does not work, do you have any idea? Thanks!
> sudo reptyr 1290042 -T
Unable to attach to pid 1290042: Permission denied

the process (1290042) is created by:
sudo cp /deva /devb

I think this issue is the one currently preventing me from using this with the GCC testsuite on the GCC compile farm; the -T option doesn't help...


Citizens of Earth Demo [PCSE00562]

App summary

App name: Citizens of Earth Demo
App serial: PCSE00562
App version: 1.00

Vita3K summary

Version: v0.1.6 2977-3a9892fe
Build number: 2957-4343481c
Commit hash: Vita3K/Vita3K@3a9892fe
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: User01
OS: Windows 10
CPU: AMD Ryzen 5 2600 Six-Core Processor
GPU: NVIDIA GeForce RTX 3060 Ti/PCIe/SSE2
RAM: 8 GB

Issues

The game crashes are the first tutorial fight.
Minor graphical issues with text misalignment on the virtual keyboard and other text prompt having a line on the keys not present in the console.
An also a graphical issue with the fast-forward feature
Screenshots

Log
vita3k.log
Recommended labels

Ingame -
Crash
Graphical bug

as of https://github.com/Vita3K/Vita3K/commit/5cb4c46c17acf1eff13e6c4ea96f3dd98e026320, the game's status is https://github.com/Vita3K/compatibility/issues/1123.
Screenshots

Log
vita3k.log
Recommended Labels

Ingame +
graphics bug

App summary

App name: Citizens of Earth Demo
App serial: PCSE00562
App version: 1.00

Vita3K summary

Version: v0.2.0
Build number: 3691
Commit hash: https://github.com/vita3k/vita3k/commit/31826697
CPU backend: Dynarmic
GPU backend: Vulkan

Test environment summary

Tested by: Benjamin Toledo
OS: Windows 11
CPU: AMD Ryzen 5 4600G
GPU: AMD Radeon(TM) Graphics
RAM: 15 GB

Issues

No issues

Screenshot

Log
vita3k.log
Recommended labels

Playable


new_feed

don't show subscribed feeds
limit feeds list by 30

closed by https://github.com/recoilme/verysmartdog/issues/42


Team prop detection

Scan team prop at the start of the match and use an as init-task

Detection is working, need to make routes for this now (#49)


[.NET][WPF][HTML] Fix bleed bugs

Fixes this issue: https://github.com/Microsoft/AdaptiveCards/issues/2680
How verified
WPF
Nested.ColumnSet.Bleed.json

Nested.ColumnSet.Bleed.Complex.json

ComprehensiveTest.json

Column.ColumnSet.Container.Bleed.json

HTML
The tests cards were run and it gave this results

ColumnColumnSetContainer.Bleed.json

Could you update the description to contain a brief description of what the issues were?


Cannot install plugin

In Obsidian 1.1.12 I cannot install the plugin from inside Obsidian, I get Failed to install plugin "Simple Note Review".
This happens in my Vault, and inside the Sandbox Vault.

@Signynt should be fixed by now.


Cards globally accessible

The Credit Card objects could be accesible outside the plugin scope in order to extend them if needed, to retrieve the Card object and perform custom actions, etc.
I have not thought about all implications about this change, but opened this pull request to discuss it :-)

Thank you for this recommendation. I implemented it (with tests and documentation) in c16dd65e5aa3e2c23504267cba072b3abb7cdacd.


zf3 support

hey, any news for zf3 support ?

nothing from side, I am not using Zend at all personally


delayed estrous cycle MP:0009008 - placement

delayed estrous cycle MP:0009008 (syn: delayed onset of estrous cycling) should be made a child of delayed sexual maturation MP:0001938
delayed estrous cycle MP:0009008
DEF: onset of the estrous cycle of females occurring at a later than expected day/age --> get rid of day
SYN: delayed estrus --> delayed estrus cycle
SYN: delayed oestrous --> delayed oestrous cycle
SYN: delayed onset of estrous cycling
Currently, EQ uses:
'has part' some
(delayed and ('characteristic of' some estrus) and ('has modifier' some abnormal))
EQ should be using estrous cycle GO:0044849 instead of estrous GO:0060209.
Need to import estrous cycle GO:0044849

Create a new term delayed estrus under abnormal estrus
DEF: increase in the time between between the estrus phases of the estrous cycle in females

Add delayed estrus phase as a synonym.


Fix hardcoded execution context

The default execution context (scala.concurrent.ExecutionContext.global)
was hardcoded in multiple places in the code. This is not only
inflexible, but problematic for Play applications, since it will cause
code inside futures to not use the expected environment, like the custom
class loader.
As as solution, introduce an executionContext member to
RuntimeEnvironment, defaulting to the global context, that can be
overridden by users as desired.
Services that need the contexts have been refactored to receive them
once as a constructor parameter in their default implementations, or
whenever possible, take them from another parameter that already has one
(like another service). The only service that was not linked to any
other and needs it's own definition of executionContext is
AuthenticatorStore. To mitigate issues, it will get a context from the
environment in the default implementation (through an implicit
parameter), or default to the global context if not other override
exists

Sorry for the large changeset, but the uses were scattered all round the code and I needed to do a bit of refactoring to properly pass the context from service to service.

First let me say this is a great idea, this has been bothering me for a long time too :)
I have some comments I will put directly in the changeset.

I pushed up a new version with some of the suggested fixes. I went with the abstract class solution for AuthenticatorStore, at least temporarily, since it ended up much simpler. I also cleaned up some formatting issues, the compile-time formatter shouldn't need to fix anything anymore.

cool :) thanks for all the work !

thanks a lot @danielkza.


clientv3: set current revision to create rev regardless of CreateNotify

Turns out the optimization to ignore setting the init rev for
current revision watches breaks some ordering assumptions. Since
Watch only returns a channel once it gets a response, it should
bind the revision at the time of the first create response.
Was causing TestWatchReconnInit to fail.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@94f6a11). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #7804   +/-   ##
=========================================
  Coverage          ?   75.63%
=========================================
  Files             ?      331
  Lines             ?    26148
  Branches          ?        0
=========================================
  Hits              ?    19778
  Misses            ?     4940
  Partials          ?     1430

Impacted Files
Coverage Δ

clientv3/watch.go
93.7% <100%> (ø)

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 94f6a11...ca4acce. Read the comment docs.

we do not really say any thing about the ordering between watch and other calls from one client. but yea, i agree that this optimization might not worth the effort. the side-effect might be confusing.

lgtm

@heyitsanthony Do we backport this as well? This is following change after https://github.com/coreos/etcd/pull/7795?

@gyuho yeah


volume not listing if created in run command

Steps:

rio run -v test2:/data nginx
rio volume

Results:  test2 is not being listed.   Should be listed.

This is currently creating an ephemeral volume, which makes sense if defined in yaml but doesn't match user expectation like in docker run.  So for the CLI only, -v foo:/path should first create a volume and then use that volume for the container to be run.  When the container is deleted, the volume stays around unless deleted manually.

No longer an option


my 2 hotspot is on denylist

i need help plz my 2 hotspot is on denylist      bobcat:Funny orange moth                             nebra out door:Fresh purple octobus                I don't have others.

This is not where you should submit denylist removal requests. That needs to be done at Crowdspot: https://docs.helium.com/mine-hnt/crowdspot/#submit-a-removal-request


Fix "Already-Registered" error when calling same participant multiple times

Purpose
Fixes #15362
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Added a issue for track that https://github.com/ballerina-platform/ballerina-lang/issues/18844

Codecov Report

Merging #18841 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18841   +/-   ##
=======================================
  Coverage   19.92%   19.92%
=======================================
  Files          52       52
  Lines        1355     1355
  Branches      208      208
=======================================
  Hits          270      270
  Misses       1072     1072
  Partials       13       13

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 112ca24...dd064f0. Read the comment docs.

This issue can be found in local participants and we need to handle in all places used for remote participants. Hence reopen the issue.


Make it clearer that submit leaves edit mode, or make it an option to continue editing

I started filling the first form (documentation, license, etc.), then submitted.
Wanted to continue with next form (next tab), but the form is locked (mouse changes to forbidden sign when hovering over the radiobuttons).
I could not find out why. Do I have to do something? Is it some processing happening in the background and I just have to wait? For how long?
https://bestpractices.coreinfrastructure.org/projects/271
thanks for your project, hopefully it will be used a lot! :)

Oh, found out. By submitting, one leaves editing mode.
One has to click on "Edit" (see top border) again.

Right! Is there a way we could make that clearer?

Maybe 2 buttons below the form:
a) Save (and leave edit mode)
b) Save and continue editing

The CII grant request form has a pair of buttons at the botton:
"Save & Continue Editing" | "Save & Exit"
That might be a useful model/example; it seems easy to understand.  Here is its HTML (using Bootstrap):
<div class="button-group">
 <input type="submit" class="button btn btn-default" name="save-changes" value="Save & Continue Editing" />
 <input id="next-submit-button" type="submit" class="button button-primary button-highlight btn btn-primary" name="submitbutton" value="Save & Exit" />
  </div>

We just added an extra button to save and continue in PR #631
Thanks again for the feedback!


[bulk] validation: validation edits JM 5/12a

This PR fixes 60 issues.

Global effort to fix build validation errors
@Rick-Anderson
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you'd like to see. Thank you!


Issue with the .js file "src.e31bb0bc.js"

When I'm clicking on the Preview camera and microphone button it is not working. But it worked in the case of your main app without the screen share feature.
The error it is showing is in the auto generated file "src.e31bb0bc.js".

Resolved by re-cloning.


Don't poll on Inotify File descriptor

The current logic polls on the inotify filedescriptor by a sleep and read loop: https://github.com/erikzenker/inotify-cpp/blob/master/source/Inotify.cpp#L194. This mechanism should be replaced by an event based approach.

Resolved by #39


Footer component

Feature request 🚀

[x] I will create Pull Request
[x] It's just a suggestion

Example
Refer This for Example
I needed this component so I tried and have 1 example ready:

So let me know if it's needed I would start working on it.

Looks very nice 👍 , I think this component can be merged.

Please see https://github.com/geist-org/react/pull/505#issuecomment-863481366.


[BUG] server crash when someone register

Versions

EasyAuth 3.0.20
Fabric API 0.97.1+1.20.4

Describe the bug
server crash
To Reproduce
Steps to reproduce the behavior:
1.use register command
2.Server crash
Expected behavior
duno
Additional context
none
Logs
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x0000000000000eb6, pid=1, tid=97
#
# JRE version: OpenJDK Runtime Environment Temurin-17.0.12+7 (17.0.12+7) (build 17.0.12+7)
# Java VM: OpenJDK 64-Bit Server VM Temurin-17.0.12+7 (17.0.12+7, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, serial gc, linux-amd64)
# Problematic frame:
# C  [jna16170577377298990446.tmp+0x523a]
#
# Core dump will be written. Default location: Core dumps may be processed with "/usr/share/apport/apport -p%p -s%s -c%c -d%d -P%P -u%u -g%g -- %E" (or dumping to /home/container/core.1)
#
# An error report file with more information is saved as:
# /home/container/hs_err_pid1.log
#
# If you would like to submit a bug report, please visit:
#   https://github.com/adoptium/adoptium-support/issues
# The crash happened outside the Java Virtual Machine in native code.
# See problematic frame for where to report the bug.
#

This should help you:
https://github.com/NikitaCartes/EasyAuth/wiki/GLIBC-problems


Fix incorrect UseCookieAuthentication code snippet

Replace lambda argument to app.UseCookieAuthentication() with instantiation of a new CookieAuthenticationOptions() object.

Hi @diddledan, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

Maybe related to #1472

@diddledan, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

Thanks!


Mobile monitoring dashboard

The Google Cloud app's rendering of our main monitoring dashboard isn't great. It doesn't have the layout or ordering, and some graphs don't show up at all. Maybe make a mobile-specific dashboard?

Done! https://console.cloud.google.com/monitoring/dashboards/builder/f3261458-ec57-4614-8247-fd572509b5e2?project=bridgy-federated


E-01-S01 Database design

For Bloodconnect, we need to design a single table design for dynamodb that serves, our business cases. To learn about bloodconnect please check the readme file here. It has those details. Apart from that, I am also attaching the details that was designed earlier.
Figma design
Acceptance criteria:

Documentation update.
Single table design that serves bloodconnect project. (Database : dynamodb)

Helpful resource:
Here is a talk that i found very helpful DAT410

@Shahed-Chy-Suzan Please share the doc here


ESP32: Wait till NimBLE host-cotroller is synced

Problem

If NimBLE host-controller is not synced then BLE manager fails to configure advertising data. This is observed when ESP32 is configure to run at 80 MHz CPU frequency.

Change overview

Waiting on a semaphore to till NimBLE host-controller sync completes.
Some fixes in temperature-measurement-app/esp32 readme file.
Added a config option to add 8K RTC memory to heap in unicore mode in temperature-measurement-app/esp32

Testing

Build, flash and monitor all-clusters-app/esp32 at 80 MHz CPU frequency and BLE manager was able to configure the advertising data.

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

Thanks a lot @shubhamdp! LGTM

Can this be reviewed and merged? cc @woody-apple


Pub: Fix package project duplicates and other issues

Please see the commit messages for details.

I have added a fix for a subversion test.

Merged, as test failures in DotNetFunTest are unrelated to the change.


Don't try to use the -C switch for ps in OS X

The -C switch in the BSD-flavored version of ps in OS X doesn't do
what it does in Linux:
-C      Change the way the CPU percentage is calculated by using a
        ``raw'' CPU calculation that ignores ``resident'' time (this normally
        has no effect).

$ ps -C Google\ Chrome -o pid= | head -1
ps: illegal argument: Google Chrome

This causes tests for process('AppName') resources to fail. This workaround
does get ugly, but yields the expected output:
$ ps -A -c -o command | grep Google\ Chrome
Google Chrome
Google Chrome Helper
Google Chrome Helper

$ ps -A -c -o pid=,command | grep -E -m 1 ^\ *[0-9]+\ +Google\ Chrome$
99159 Google Chrome

$ ps -A -c -o pid=,command | grep -E -m 1 ^\ *[0-9]+\ +Google\ Chrome$ | \
    awk '{print $1}'
99159

Thanks.

Released as v2.42.2.


Add divider for menus

As far as I can see there is no divider css styles for menus.
Can this be added ...
Regards

Can you provide a reference in the specification for where this is defined?

Sure
General divider spec
https://www.google.com/design/spec/components/dividers.html
Usage in a menu
https://www.google.com/design/spec/components/menus.html#menus-usage
Regards

What about just using an
 or a border-bottom for those purposes?

Doing a modifier against menu items with border-bottom styling seems best here.

I agree that border-bottom would work here and It was something I considered, but I think it is also unintuitive to many.
Also, dynamically adding items to menus in groups may become somewhat confusing and more difficult to handle. In this case I expect a  maybe better placed but at least there would be options.
However, as a stop gap, until a proper solution in line with the specs can be provided, I am personally happy to go with this.
Regards

The logistics between adding in a horizontal row dynamically or the modifier class is the same and not more difficult in any case that comes to mind. Somehow, you are going to need to detect when to add either and then carry out the task, so which is done doesn't seem like an issue.
Dividers can't live on its own. The way to implement it depends upon the component being targeted. Abstracting it out doesn't make sense. The spec pages are not a one-to-one thing. There is a subheader page, but we won't implement that as its own component.

A standalone menu/list divider class--that doesn't piggy back a list item with content--would be so much easier for looping, IMHO. Sounds like a pull request opportunity for me... :o]

Can you provide an example where it would make looping easier? I'm not sure how it being a class vs a dedicated node is going to make a difference.

We have some dynamic menus built from values in the database. After I loop through those, I want to add a full-bleed divider and add a few more items to the menu. Is there a way provided to add standalone list item for a menu divider?

 with no content has a lot of space (for the menu item content), and

 has half space before and after the divider.
If there's not already something provided, I'll just add CSS to the hr version to shrink the height to 1px.

There is no node to make a divider. It's just a modifier class on an existing node. Even with the given case I'm still not seeing how having a dedicated node is going to help at all.

It's a personal code-style thing. I'd rather not add the divider class to the previous , and I don't want to add logic to the loop to add the divider class to other menu items at certain points. I like dropping this in when I want a divider

.

I've found that using an
 would be fine, but something is missing, since it isn't rendered.
For some reason, the hr in material.grey-orange.min.css (which is probably missing in other ones like material.indigo-pink.min.css) would have only these properties:
hr {
    display: block;
    height: 1px;
    border: 0;
    border-top: 1px solid #ccc;
    margin: 1em 0;
    padding: 0;
}

I think these should be added:

width: 100%; - responsive, solves the problem (probably 90% would be nice for hr, while 100% could be set for .mdl-cell hr)
margin: 1.5em auto - 1.5em would give more space, while auto would center it

The hr styling is from the reset. We won't allow any changes to that internally since it could have adverse affects on existing sites.

Here is an option that seems to be inline with the style guide...
<Menu id="nav-menu">
    {items.map(
        option => [
            <MenuItem key={option.primary}>
                <ListItemIcon>{option.icon}</ListItemIcon>
                <ListItemText primary={option.primary} />
            </MenuItem>,
            option.isDivider ? <Divider /> : null,
        ]
    )}
</Menu>


Random vhost for testing

We should start using a random vhost for test runs rather than trying to clean up an existing one.
reset_rabbit_connections doesn't work properly because changes in the rabbit management interface aren't reflected immediately, so if the vhost is dirty there's nothing we can do about it.
It would be better to use a random vhost (unless requested otherwise) and still fail the test at the end if there are connections left over. reset_rabbit_connections should be removed.

Hey @mattbennett. Why can't you use something like rabbitmqctl delete_vhost test at the end of a test run?

It's definitely preferable to use an API rather than wrapping a command line tool. Also I strongly suspect that rabbitmqctl either directly calls the API itself, or they both invoke the same underlying code.

by "something like" i meant whatever underneath this command line call. what i'm curious about is why we can't create a test vhost and then kill it at the end of the run - seems much tidier than creating random ones.

rabbitmqctl seems to be a wrapper around the erlang interpreter (https://github.com/rabbitmq/rabbitmq-server/blob/master/scripts/rabbitmqctl) so there's no benefit in going to what's "underneath" rabbitmqctl (same problems of the binary perhaps changing between platforms, not being on the path etc.)
As I said I suspect the fact that connections aren't immediately closed is just how things are implemented inside RabbitMQ, and doesn't have anything to do with the management API per se.
I agree that it would be nicer to use and then clean the same vhost between tests, but it doesn't seem to be possible, at least reliably.

Fixed in #288


⚠ GLDDNS Server (ns2.glddns.com) has degraded performance

In bfeec9d, GLDDNS Server (ns2.glddns.com) (ns2.glddns.com) experienced degraded performance:

HTTP code: 200
Response time: 432 ms

Resolved: GLDDNS Server (ns2.glddns.com) performance has improved in 3114a9d after 11 minutes.


Update about.md for labop.io with current elected leadership

@photocyte it looks like this PR is ready to merge?  are you planning on making further updates, or have you just been waiting for an approval?

@bbartley Was waiting for an approval. But seems implicit so will merge.


Update LMM token

hi，now i commited the picture and contract json file, please check it and accept it ,thanks.

I have updated the website address. Please re-verify it.thanks.

Please remove .gitignore file


New parameter on Queue, textChannel

Add a new parameter on queue, textChannel . It is the channel in which the play comnand was used first. i think this is useful if we want to send every success message (rather than the default distube  event) to the channel, in which the bot is emiting distube event. We can use something like queue.textChannel.send(".. your message bla bla...")

You can do it by adding queue.textChannel = message.channel in play events

You can do it by adding queue.textChannel = message.channel in play events

Ok, thanks for help 🙂

Ok, thanks for help 🙂


[FEATURE] Volume expansion

What feature you'd like to add:
https://github.com/container-storage-interface/spec/blob/release-1.6/spec.md#controllerexpandvolume
Currently,

modifying a JuiceFS PVC's storage request has no effect, and
modifying the PVC's underlying PV causes mount failures:

MountVolume.MountDevice failed while expanding volume for volume "pvc-acb59dca-c2e2-40f6-9821-fa27d47dce0d" : Expander.NodeExpand found CSI plugin kubernetes.io/csi/csi.juicefs.com to not support node expansion

Why is this feature needed:
Dynamically resizing volumes is extremely helpful when expanding storage pools.

Dear friends,
Any forecast for adding resizing to the JuiceFS CSI driver?
On the other hand, it should first support the size concept because whatever pvc size you provide, it is possible to create as much data on s3.


Add build time deps to tools package

My editor was complaining that many of the build-time packages that we use for protobuf generation were not used in the project. T
This PR imports them in the tools package so that they show up and are tracked correctly in go.mod and go.sum

LGTM
I like the changes in the APIs.

FYI those are auto-generated. I guess bumping the version added some sort of annotation.


[SPARK-25534][SQL] Make SupportWithSQLConf trait

What changes were proposed in this pull request?
Currently, Spark has 6 withSQLConf function. This PR aims to remove duplicated and inconsistent withSQLConf code and reduce them to the following three meaningful implementations.

SupportWithSQLConf.scala: The one which was used in PlanTest.
ExecutorSideSQLConfSuite.scala:  The one which doesn't throw Exception on StaticConf changes.
SQLTestUtils.scala:  The one which changes the active session like the following.

  protected override def withSQLConf(pairs: (String, String)*)(f: => Unit): Unit = {
    SparkSession.setActiveSession(spark)
    super.withSQLConf(pairs: _*)(f)
  }

How was this patch tested?
Pass the Jenkins with the existing tests.

Could you review this PR, @cloud-fan , @gatorsmile , and @wangyum ?

Test build #96579 has started for PR 22548 at commit bc4d71d.

Merged build finished. Test PASSed.

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

We have duplicated withTempPath, withTempTable too. Shall we pull them out as well? The trait name can be SQLHelper

+1, Sure, @cloud-fan !

It turns out that withTempTable cannot be here together because it requires spark.catalog.
  def withTempTable(tableNames: String*)(f: => Unit): Unit = {
    try f finally tableNames.foreach(spark.catalog.dropTempView)
  }

Since it requires function signature changes, shall we postpone withTempTable as another PR?

sure

Merged build finished. Test PASSed.

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

Test build #96586 has started for PR 22548 at commit 0827ede.

Test build #96586 has finished for PR 22548 at commit 0827ede.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait PlanTestBase extends PredicateHelper with SQLHelper
trait SQLHelper
abstract class CheckpointFileManagerTests extends SparkFunSuite with SQLHelper

It passed. Merged to master.

Merged build finished. Test PASSed.

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


Adaboost - Documentation

As this issue hadn't been raised earlier. I would like to work on this issue.

Task : Create detailed documentation of the Adaboost ML algorithm by updating readme.md
Level-1 : 10 points

I would like to contribute could you please assign me this task.  I am a gssoc participant.

@ashwani-rathee i am a gssoc participant and i would love to contribute to this project and would try my best to explain it in the best possible way

@ashwani-rathee kindly assigned the issue as I can submit the pr.

Hii...I am a GSSOC'21 participant, and I would like to work on this issue. Kindly assign me this issue.

@ashwani-rathee I would like to contribute could you please assign me this task. I am a GSSoC'21 participant.

@surbhi2408 would you like to take this up?

@surbhi2408 would you like to take this up?

Yes, sure

@surbhi2408 Sure,go ahead

Merged


Update Liberty OpenStack-Ansible SHA

Update OpenStack-Ansible SHA in preparation for the release of r12.2.3.


Multiple Calls to OpenAM Server for Cookie Name

It seems that the gem makes at least 4 calls to the server to fetch the cookie name for the OpenAM Token. Is there a reason that value isn't saved in an instance variable the first time the call is made?

Hi,
yeah, looks like a bug, and an easy win to save a few requests. I pushed a fix for this, and I'll get it published to rubygems shortly.


Support for PNPM (astro-i18next throws with "module is not defined")

When I try to use <Trans />, <HeadHrefLangs />, <LanguageSelector />, interpolate, etc I get the following error :
module is not defined

/@fs/my/local/directories/turborepo-astro-nestjs/node_modules/.pnpm/deepmerge@4.2.2/node_modules/deepmerge/dist/cjs.js:133:1

ReferenceError: module is not defined
    at /my/local/directories/turborepo-astro-nestjs/node_modules/.pnpm/deepmerge@4.2.2/node_modules/deepmerge/dist/cjs.js:133:1
    at instantiateModule (file:///my/local/directories/turborepo-astro-nestjs/node_modules/.pnpm/vite@3.1.6/node_modules/vite/dist/node/chunks/dep-db16f19c.js:53436:15

I'm using pnpm and turborepo, reproduction (error at apps/frontend/src/layouts/Layout.astro line 2 & 19):
https://github.com/Alexandre-Fernandez/turborepo-astro-nestjs
Astro app is located in /apps/frontend, commands to get the error :

pnpm install
pnpm dev --filter frontend
localhost:4000

EDIT:
This might happen because it's trying to load from the monorepo root /node_modules instead of its corresponding workspace /apps/frontend/node_modules.
I say that because I get a different error (another missing dependency) when I install<EMAIL_ADDRESS>in the root package.json (pnpm add<EMAIL_ADDRESS>-w).
This only happens with astro-i18next and not with i18next.

Apparently the problem are not monorepos but PNPM.
I made a new reproduction here :
https://github.com/Alexandre-Fernandez/astro-i18next-issue-48
If you install with npm install it works, but if you install with pnpm install you get the module not defined error.

Workaround
Add this file to your root :
.npmrc
public-hoist-pattern[]="*deepmerge*"
public-hoist-pattern[]="*proload*"
public-hoist-pattern[]="*locale-emoji*"

While I haven't used the module yet, I'm a big fan of both astro and pnpm so I had a quick look at the source.
First thing I notice is these 2 imports:

cli/utils.ts:import { AstroI18nextConfig } from "types";
cli/generate.ts:import { AstroI18nextConfig } from "types";

Both should probably import from "./types" (or ./types.ts - I never know if the extension is required).
Other than that, I see these imports, all from dev deps. Probably should be regular deps:
cli/utils.ts:import { fdir, PathsOutput } from "fdir"; // dev dep
index.ts:import { AstroIntegration } from "astro"; // dev dep
cli/middlewares.ts:import { MiddlewareFunction } from "yargs"; // dev dep
cli/index.ts:import yargs from "yargs"; // dev dep
cli/index.ts:import { hideBin } from "yargs/helpers"; // dev dep
cli/utils.ts:import fsExtra from "fs-extra"; // dev dep
cli/utils.ts:import ts from "typescript"; // dev dep

With those fixes, pnpm might be happy without resorting to public-hoist-patterns.


exclude embedded ACS AEM Commons classes from javadoc

@kwin i tested your changes on 6.5.13 and things seem to work. Think its ready for a release, or any other lagging changes you want to get in?


Where is old documentation for version 3.xx?

Why is there no link to documentation for current version 3.xx? I know next version is awesome but as long as it's not officially released, please keep everything for current version's users.
Thanks,

Sorry, it's inconvenient, but reinstating the old version isn't possible at this time due to having lost Ben as a maintainer. We'll get 4.0 out as soon as possible. Subscribe to #408 for updates.


Images not loading in beta skillmap

Tested with https://arcade.makecode.com/beta--skillmap#github:https://github.com/shakao-test/demo/sample
Could be because GitHub is currently experiencing issues.

was indeed github issues


Individually set OPENSSL_INCLUDE_DIR and OPENSSL_LIB_DIR instead of OPENSSL_DIR

This makes cross compatible with a wider range of openssl versions.

Nice! Do you know what's the oldest version of openssl that can be supported with this patch?
Also, could you add a test for this in ci/script.sh. Something like this:
# you'll need to `cargo install cargo-clone -f` in ci/install.sh first
$ cargo clone openssl --vers 0.6.0

$ cd openssl

$ cross build --target $TARGET

The oldest version supporting this appears to be 0.3.6.

I can confirm that this supports openssl down to version 0.4.0.
But openssl 0.4.0 is not compatible with current rust versions any more (it was released around the time std::io was reorganized, and depends on std::old_io).

The pull request was updated with a test, against openssl-sys 0.5.5.

# you'll need to `cargo install cargo-clone -f` in ci/install.sh first

Do you mean in trust's install.sh script?

:pushpin: Commit 0d53943 has been approved by japaric

@homunkulus r+
Thanks, @jedisct1!

:hourglass: Testing commit 0d53943 with merge 0d53943...

Do you mean in trust's install.sh script?

There's no install.sh here, I forgot. I added that to Travis' install phase.

:broken_heart: Test failed - status-travis

Looks like that newer versions of openssl need the old OPENSSL_DIR variable. So I just opened #55 readding that variable.
Closing this PR in favor of that one.


Initialize column names for some plan node according to their dependencies

To Simplify Project/Dedup/Filter contruction:

For dedup/filter, set the input's columns as their output columns by default
For project, set the yield columns as its output columns when calling constructor.

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


Question: how to have per user folders

similar to FTP, each user gets own private folder
when user tester logins, I can share a folder "/davshare/tester" with the user for esample
is there a way to configure wsgidav to do this?

This is not yet implemented, sorry.


NDT mapping determinism and accuracy criteria

I have two question marks about NDT algorithm used in autoware:

is NDT implementation deterministic? So by giving always the same point clouds and init_guess to ndt.align, is it 100% sure that the output will be the same?
I got some differences in results depending on the conditions i use ndt, so i wanted to make sure that normally we should have deterministic NDT.

As i get some z-drift with ndt, I try to trace the ndt accuracy. Is fitness_score a robust quality criteria? i saw in a recent question that transform_probability should not be used. so is it the same for fitness_score? What other variable could be used if fitness is not good?

thanks!

is NDT implementation deterministic?
Since the convergence degree varies depending on the initial value, it is not definitive. The initial value also varies depending on the execution time

Is fitness_score a robust quality criteria?
It is a very difficult problem to decide the numerical value representing the quality of ndt. It is necessary to decide at what time the quality is bad or good. However, ndt often becomes unstable when the yaw rate is high, and fitness score seems to be consistent with instability.
He is trying to introduce new rating values (@YamatoAndo )

thanks for the answers. When you say the initial value, do you mean the initial guess provided to ndt or do you talk about internal ndt parameter initialization (in this case, where is it defined?) ?
I agree it seems complex to get proper metrics about map quality, and fitness score is not enough. Nice to hear that some work is done on this part! @YamatoAndo  can we have some first hints about the future metrics that we will get in a futur release? :)

When you say the initial value, do you mean the initial guess provided to ndt or do you talk about internal ndt parameter initialization (in this case, where is it defined?) ?

It mean the initial guess provided to ndt. (not parameter)

@yukkysaito @YamatoAndo Thanks for your answer! Do you have new results about the rating values of NDT matching?

Autoware has moved from GitHub to GitLab.

Follow the support guidelines and ask your question at ROS Answers.


Build Layout.js lib into Bangle.js firmware

Right now, every app that uses Layout has to include its own copy. Parsing it in initially takes 25ms by itself, and then executing it to render on something like a navigation message takes 240ms when running from internal flash would be 220ms.
... so if we built it into the Bangle.js firmware we'd save around 45ms loading something like a navigation message.
The reason we didn't build it in initially was that it was changing rapidly, but the last real change was made a year ago so I think we'd be safe.
Of course if it were to need changes after this, realistically we're looking at waiting until the next Bangle.js firmware release before those changes could go it...
Any thoughts?

Just to add to this, we also have some "ram" keywords to aid performance from external flash. When running on internal flash those could be removed and would likely get it running even faster.

every app that uses Layout has to include its own copy

Could we get around this by loading it on as a module, like what we do with clock_info, So the JS file sits on the device itself, rather than a Modules.addCached(...) ?
But I suppose that wouldn't address the performance, and I'd be please with a 25ms speed up, so +1 from me on that.

Yes, it could sit on the device but you don't really get all the speed benefits then.
Ok, just done this with ef5c7c8a38868d50fdea5fa32d04ba540b1e3c20 - but it needs an updated BangleApps to realise that Layout exists and to not re-upload it
I also improved some other stuff, but now to get Messages to the point where it displays a message we're down from 331ms in 2v24 to 308ms in 2v24.128
Not quite as much as I'd have liked, but still good!


js-test-repo to 0.0.4

Promote js-test-repo to version 0.0.4

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@abayer: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

promotion-build
c83a850e7268587d4a5d606829232de3140e7e7c
link
/test this

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Remove version: Cube.CubePlatform version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://download.cubejoy.com/app/<IP_ADDRESS>/CubeSetup_v<IP_ADDRESS>.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 923)


[Question] Why is "ConditionalExpression" ignored in the "indent" config?

This does not match the style guide and seems generally like an unintuitive choice. What's up wit dat?

After playing around with it some more, I think I misunderstood what this does. I previously thought ignoring a node meant that ESLint would apply the default rules to those nodes, but now I am under the impression it means to apply no rules to the node. My question remains, however, along with an additional one:
There doesn't seem to be an indent config option for ConditionalExpression (or at least I can't find one). This seems like a gross oversight; I can't get my ternary expressions to match with everything else, and I'm not sure why this would be excluded.

I guess the second question might be good to ask the ESLint project. Please excuse the asking of an irrelevant question.


popup doesn't follow to draggable elements

I have a draggable (jquery ui draggable ) elements on my web page,  the popup does not follow to elements during drag, please provide option to fix it  thanks.

I have no plans to fix this, but I will happily accept a pull request if
it's not too intrusive.
On 24 Feb 2015 21:24, "Ashot"<EMAIL_ADDRESS>wrote:

I have a draggable (jquery ui draggable ) elements on my web page, the
popup does not follow to elements while dragging . plese provide option to
fix it thanks.
—
Reply to this email directly or view it on GitHub
https://github.com/rubenv/angular-rt-popup/issues/8.


Drag&Drop & Async Loading broken

Using Drag & Drop to move a node into a new parent node that has not been expanded and support Async Loading, will cause that the node shows only the moved node and not all the childrens.
You can verify this problem in your online demo (https://angular2-tree.readme.io/docs) following these steps:

Open the demo at: https://angular2-tree.readme.io/docs
Drag the node "root2" to "asyncroot"
Expand "asyncroot"

You will see that "asyncroot" now contains only "root2" as children instead "root2" beside the original children.
Please fix it!

Hi,
if you support async loading you should override the drop action (like saving the data to the remote server on drop) using options.actionMapping.mouse.drop


I'd like to take a look


GH2714: Squash warning when skip package version check

Fixes #2714.

@gitfool your changes have been merged, thanks for your contribution 👍


[Feature] support swagger UI to viewing API

Feature Description (功能描述)
Hugegraph is inconvenient to view and test API, so I suggest use swagger to manage the API.
Use swagger UI view API like this:

And test API like this:

As can be seen from the above, it is very convenient. So I have a PR for it 1880
Shall we need it？

@JackyYangPassion @SunnyBoy-WYH could also test & add the basic doc in website, and link this PR to finish the doc


Change preferences on GitHub

@AlZorba it's done,
no related code
What was done/set :

✅ Automatically delete head branches
✅ Allow squash merging ONLY
✅ squash commit message : Default to pull request title and description
Dev is now the default branch
I protected main, we will see how it works


Generated project requires node v4 to ng serve

We should be explicit about this dependency in the docs since the generator does not actually require v4 and it is easy to assume that things are just broken if you don't catch the warning..

Would updating the package.json engine help any here?
{ "engines" : { "node" : ">=4.1.x" } }


Initial path types

Describe your changes
This PR introduces types that are needed for motion planning.
As an initial suggestion I've decided to treat dual arm configurations as an array np.hstack((joints_left, joints_right)). The alternative is treating them as the tuple (joints_left, joints_right). Both options have pros and cons we should consider.
Additionally, for the paths I've provided both list and array types. We should consider whether both are actually necessary.
Considerations

DualJointConfigurationType being the hstack or two JointConfigurationType is similar to how Drake combines all state of a plant. E.g. calling sceneGraphCollisionChecker.CheckConfigCollisionFree expects a (12, ) numpy array for my dual UR5e scenes.
Stacking might require code like this: joints_left, joints_right = joints_dual[:6], joints_dual[6:] or np.split(joints_dual, 2). I'm not sure if this or better or worse than unpacking a tuple joints_left, joints_right = joints_dual. The tuple approach could support two arms with different #DoFs.
The list/array types are somewhat redundant, but some path operations require arrays, such as np.diff(path_array, axis=0).
How we will handle additional DoFs, such as trajectories for gripper position?

Checklist

[ ] Have you modified the changelog?

I strongly like the name path you use if there is no attached temporal dimension.
Going from paths to trajectories; trajs are indexed by time, this could be a function or dict. You might also support this instead of splitting this up into TimePathType and TimePathArrayType (or at least that is how I interpret these types).
Additional DoFs for grippers -> I also thought about initially supporting this but our Robotiq grippers don't like high frequency commands and I don't see an immediate use-case on our Schunk gripper so I think we can ignore that for now.
I like the tuple approach more than the stacked numpy array approach. An alternative might be a list of joint names and a list of joint values.

The tuple approach is what I've currently implemented in my planner interfaces (see except below). The "downside" of this approach is that to extract that path for only the left or right arm, you need things like this:
path_left = [joint_left for joint_left, _ in path]
path_right = [joint_right for _, joint_right in path]
path_left_resampled = resample_path(path_left, n_servos)
path_right_resampled = resample_path(path_right, n_servos)
path_resampled = list(zip(path_left_resampled, path_right_resampled))

class SingleArmMotionPlanner(abc.ABC):
    """Base class that defines an interface for single-arm motion planners. """

    @abc.abstractmethod
    def plan_to_joint_configuration(
        self,
        start_configuration: JointConfigurationType,
        goal_configuration: JointConfigurationType,
    ) -> Union[List[JointConfigurationType], None]:
        raise NotImplementedError

class DualArmMotionPlanner(abc.ABC):
    """Base class that defines an interface for dual-arm motion planners.  """

    @abc.abstractmethod
    def plan_to_joint_configuration(
        self,
        left_start_configuration: JointConfigurationType,
        right_start_configuration: JointConfigurationType,
        left_goal_configuration: Union[JointConfigurationType, None],
        right_goal_configuration: Union[JointConfigurationType, None],
    ) -> Union[List[Tuple[JointConfigurationType, JointConfigurationType]], None]:
        raise NotImplementedError

I agree that the distinction between path and trajectory is important and should be very clear, I'm not convinced of the need for a  List and Array variant of all the variables, that seems confusing to me and also harder to maintain?
Wrt dual-arms, I would not provide separate types because this feels hard to scale to me (what about grippers etc?). Either we rely on the user knowing the degrees of freedom of its system and interpreting the configurations correctly, or we use some kind of naming?
Regarding Path -> Trajectory, I can live with a (named) Tuple for this, though I could also live with a dict from 'pose/config' to timestamp.
We should maybe also anticipate the need for specifying velocities/accelerations/torques in trajectories.
In general it might not be a bad idea to take a look at the relevant ROS messages? :slightly_smiling_face: e.g. https://docs.ros2.org/foxy/api/trajectory_msgs/index-msg.html

For reference, ROS does it like this:

JointTrajectoryPoint: flat array for positions/velocities/... with all DoFs stacked together and a time value.
JointTrajectory: ~list of JointTrajectoryPoint and list of names for the DoFs

I don't really understand their docs for "MultiDOFJointTrajectoryPoint`, to me it seems like that's a trajectory of SE(3) poses.

I've pushed a much-simplified proposal. I've chosen to use 2D arrays for paths for the simple reason that you can have both row-wise access (e.g. to execute a servoJ()) and column-wise access (e.g. to plot with matplotlib).
I've also removed the dual arm types as this would indeed scale badly to gripper DoFs, mobile base DoFs, etc. However, we have to consider what the consequences are for the DualArmMotionPlanner interface. Currently I've used this pattern for the plan_() functions:
(start_left, start_right, goal_left | None, goal_right | None) -> list[(config_left, config_right) | None

Using flat arrays as arguments here would mean we can't use None anymore to signal that no planning should be done for that arm. However, we could also add boolean arguments for the same functionality, something like:
(start_joints, goal_joints, plan_for_left=True, plan_for_right=True) -> 2D-array

The DualArmMotionPlanner would however need to know how to unstack the joints, e.g. to plan for a single arm or to call the inverse kinematics of the left/right arm.

I've pushed a new iteration of the types with JointTrajectory and DualArmJointTrajectory dataclasses. There's two things to consider here:

How will we handle additional DoFs, e.g. grippers, mobile bases?
positions is a required attribute of JointTrajectory, velocities, accelerations and efforts are optional. Does this cover all use cases? Might someone ever want a JointTrajectory without positions and only velocities, and do we have to support that in this dataclass?

@tlpss @adverley @m-decoster

Some proposed changes in commit 3ca5049:

Separate paths and trajectories
This reduces code duplication in the DualArmTrajectory and should scale better to other set-ups
It also allows for easier addition of grippers as illustrated
Make positions an optional attribute (user responsibility to make sure at least one JointPathType is valid)

Do we also want to support derivatives of the PoseTrajectory (and how)? @tlpss

I really like the proposal in Mathieu's last commit, and think we are getting close to a final design. It also matches well with the SingleArmMotionPlanner and DualArmMotionPlanner interfaces I'm working with right now. Their plan function signatures would have to be changed slightly to something like this:
SingleArmMotionPlanner
def plan_to_joint_configuration(self,
    start_joints: JointConfigurationType,
    goal_joints: JointConfigurationType)
    -> SingleArmTrajectoryType | None:

DualArmMotionPlanner
def plan_to_joint_configuration(self,
    start_joints_left: JointConfigurationType,
    start_joints_right: JointConfigurationType,
    goal_joints_left: Optional[JointConfigurationType],
    goal_joints_left: Optional[JointConfigurationType]
) -> DualArmTrajectory | None:

Many planners however don't consider time, so this should come this the warning/caveat that these trajectories might not always be safe to directly execute.

@adverley  one disadvantage now is that the time parameterization can be different for the left and right path, which is something we don't want to support I think?
IMO a trajectory can have only 1 time parameterization. We split the path DOFs for readability, not because we want to interpret them as separate trajectories I think?
And I tend to agree that accessing the data is convoluted.

We also want to support force trajectories e.g. for a force controllable gripper. I see only two options:

Andreas' comment: times are grouped with positions & it's derivatives (velocities etc.), this means left/right arm/left can all have their own timestamps and duration
Mathieu's commit: times are a top-tevel attribute of SingleArmTrajectory & DualArmTrajectoryand the derivatives for each model/DoFs are grouped in a JointPathContainer or similar. This means that the user must ensure that the length of all paths matches the len(times).

one disadvantage now is that the time parameterization can be different for the left and right path, which is something we don't want to support I think?

@tlpss If we want to group times, velocities, etc., how do you propose to make sure that the time parameterization is the same for the left and right path? Because the velocities can be different for left and right. I also don't see another option than the two @Victorlouisdg mentions.

VL does not like these big names.

@adverley @Victorlouisdg Not sure if this matches your preferences for naming conventions, but we could do something like
@dataclass
class JointTrajectory:
	positions: JointPathType
	times: TimesType
	velocities: Optional[JointPathType]
	acc: Optional[JointPathType]
	efforts: Optional[JointPathType]

@dataclass
class SingleArmTrajectory:
	arm: JointTrajectory
	gripper: Optional[JointTrajectory]

@dataclass
class DualArmTrajectory:
	left: SingleArmTrajectory
	right: SingleArmTrajectory

because it is implied that the left member of a dual arm trajectory refers to the left arm, which makes accessing members a bit cleaner:
dual_arm_trajectory.left.arm.positions

It's terse, but implicit.
But: this brings me to another issue: how do you define which arm is left and which is right? ;) Would a 2-tuple be an option? Or do we just leave it to the programmer to define left and right in their setup?

@m-decoster  Ah yes  I now realize that I made a weird/wrong assumption. In this case I have to say I like your original proposal most:

class JointPathContainer:
    positions: Optional[JointPathType] = None
    velocities: Optional[JointPathType] = None
    accelerations: Optional[JointPathType] = None
    efforts: Optional[JointPathType] = None

class SingleArmTrajectory:
    path: JointPathContainer
    gripper_path: Optional[JointPathContainer]
    times: TimesType  # time (seconds) from start of trajectory

@dataclass
class DualArmTrajectory:
    path_left: JointPathContainer
    path_right: JointPathContainer
    gripper_left: Optional[JointPathContainer]
    gripper_right: Optional[JointPathContainer]
    times: TimesType  # time (seconds) from start of trajectory

Regarding the Left/right, yeah I think we want to distinguish them somehow. Left/right seems easier to work with than first/second for example, though I can imagine there are cases where you don't have a 'left' and 'right' arm.  I don't have a strong opinion on that, and would suggest we just go for something and start integrating using the types.  We can make changes along the way  if needed?

LGTM! Maybe with gripper_path_left in the DualArmTrajectory to be consistent with gripper_path in SingleArmTrajectory (my bad in my commit) - but just gripper_left works too.

I think the current proposal is really great. There's just one detail, do we make times a required attribute without default value or an optional attribute with None as default value? (Currently it's different between PoseTrajectory and the JointTrajectory classes.)
I prefer making it required in all Trajectory classes. This way consumers don't need to worry about checking whether each trajectories has times or not. A sensible default for times is np.linspace(0, 1, len(path)). We could set this in the dataclass __post_init__() method if None was passed to the constructor.

Would there be a use case for having such a default time parameterization? IF not, I think it would be semantically more consistent to have 'actual' timings for trajectories.

I've made times required for all Trajectory dataclasses and have given it no default value. I believe this was the last thing to decide, so this PR is now ready to merge.


Rooms heading added in rooms.hbs

Light Theme

Dark Theme

[Current coverage][cc-pull] is 19.39%

Merging [#450][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #450   diff @@
==========================================
  Files             3          3
  Lines           330        330
  Methods          34         34
  Messages          0          0
  Branches         41         41
==========================================
  Hits             64         64
  Misses          266        266
  Partials          0          0

Powered by Codecov. Last updated by [26ac03f...38e995a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/fossasia/open-event-webapp/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/fossasia/open-event-webapp/compare/26ac03f8f2b4f9c8d3834774edcd2a6887fdae13...38e995a2a398f7187b5a9b59db420d9e5d582773?src=pr
[cc-pull]: https://codecov.io/gh/fossasia/open-event-webapp/pull/450?src=pr


Improvements to (the tests for) the SystemNodeProvider class.

Prompted by #185 (written by my friend and colleague @mjrider) I would like to suggest a few improvements to the Ramsey\Uuid\Provider\Node\SystemNodeProvider class and the test it comes with:

The SystemNodeProviderTest mocks (part of) the class it is testing. This is a testing anti-pattern. The main reason seems to be that, instead of changing the code to be testable, the test was changed (for the worse).
I would suggest this change to fix that. (Taking it one step further might be a bit too much.)
All tests have @runInSeparateProcess and @preserveGlobalState disabled annotations. The only reason for this seems to be the use of a static variable in the getNode function.
By replacing the static variable with a class variable this issue can easily be resolved.
The static variable in the SystemNodeProvider::getNode() function should be class-level, not function-level

The code I link to is a suggestion, if there are any reasons I have missed, the code can be changed accordingly.
Also, as a matter of priority, I'll wait with opening a PR for this until the changes in #185 have been merged to master, as that code is in the system and these changes are more for the benefit of the tests.

Edit: After a discussion with @mjrider I understand that having $node be static (i.e. shared across several instances of the same class) is desirable for performance by avoiding calling the functionality more than once. I have updated my suggestion to change the level of static variable.

What you've described here sounds good to me. Even the "taking it one step further" with the constructor is good, IMO. If you open this as a PR, I'll accept it. Thanks!

As the accompanying pull-request has been merged I am closing this issue.


[HttpClient] Update UI to show basic activity

Update UI to render basic http client data in line with current Execution tab:

[ ] Update execution tab to show basic http client requ/res data (i.e. url, method, status code, duration, etc)
[ ] Get feedback/review of final UI design
[ ] Add unit tests**

** Note we currently don't have unit tests for the UI. This point will need to reviewed depending on the state of the client unit tests.
Design:
Currently the design shows IO operations occurring inline where they occur. Update the UI to interlace this summary information within the scope of where it occurs.

Here is how things look currently (note, still need to get a bit more data in, etc):


A minimum working version of pulse simulator

The codes contain a single-qubit Schordinger Equation solver (sesolver-based) pulse simulator.

Thank you @jinleic for the contributions! Could you also add some tests to the pulse module?