Set language_in and language_out options to ECMASCRIPT5_STRICT

Fixes #3341

What are the effects of using that?

"use strict"; is added by the compiler to the output file.
According to [developer.mozilla.org] (https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Strict_mode): "strict mode code can sometimes be made to run faster than identical code that's not strict mode."

LGTM. Why is this still wip?

@gberaudo just tell me that we need to make sure that projects builded along with ol3 still works

These projects use their own, separate, configuration files so it should be OK.

I also wanted to test if the mode is really activated because the documentation says that:
"To invoke strict mode for an entire script, put the exact statement "use strict"; (or 'use strict';) before any other statements."
And it's not our case, we have a wrapper around the code (from #3039)

The strict mode is not activated by the browser because the statement is not the first one in the function.
We currently have:
(function(root, factory) {
    if (typeof define === "function" && define.amd) {
        define([], factory);
    } else if (typeof exports === "object") {
        module.exports = factory();
    } else {
        root.ol = factory();
    }
}(this, function() {
    var OPENLAYERS = {};
    'use strict';
    var l, aa = aa || {}, ba = this;
    function m(b) {
...

But it should be:
(function(root, factory) {
    if (typeof define === "function" && define.amd) {
        define([], factory);
    } else if (typeof exports === "object") {
        module.exports = factory();
    } else {
        root.ol = factory();
    }
}(this, function() {
    'use strict';
    var OPENLAYERS = {};
    var l, aa = aa || {}, ba = this;
    function m(b) {
...

Second issue (described here).
In the compiled code, there's var ba = this statement (that's goog.global = this) but this is prohibited is strict mode unless the this is explicitly bound to the function.

Closing in favor of #3366


The CF template uses a lambda which runs on Node Js 14.x runtime which is already deprecated. Can you guys please update this to Node 20.x

The CF template uses a lambda which runs on Node Js 14.x runtime which is already deprecated. Can you guys please update this to Node 20.x .
As of now I have tried running it with Node 16.x and it worked after adding the  axios and crypto, which are dependent packages of the lambda.
code provided via blog is in V2 SDK
Can you please refactor the code for V3 SDK and migrate to NodeJs 20.x

Had a similar issue - just updated the lambda code to support node 20.x, see:
https://github.com/aws-samples/aws-codepipeline-bitbucket-integration/pull/29

Thanks for your contribution


boards: arm: add Electronut labs papyr (nrf52840_papyr)

Added config for https://docs.electronut.in/papyr/
Signed-off-by: Tavish Naruka<EMAIL_ADDRESS>

Codecov Report

Merging #13208 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13208      +/-   ##
==========================================
+ Coverage   48.76%   48.76%   +<.01%
==========================================
  Files         319      319
  Lines       46864    46864
  Branches    10832    10832
==========================================
+ Hits        22853    22854       +1
  Misses      19381    19381
+ Partials     4630     4629       -1

Impacted Files
Coverage Δ

arch/posix/core/posix_core.c
91.91% <0%> (+1.01%)
:arrow_up:

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 8d86773...5e22c85. Read the comment docs.

@galak can this be added to release 1.14.0 milestones?

@anangl @galak can you take a look?

The only blocker is the simultaneous use of spi0 and i2c0. Other comments are only suggested corrections or improvements.

Thanks @anangl, don't know how I missed that! I've enabled I2C_0, SPI_1, PWM_0 instances also as you suggested.


package request: vignette

Project description
https://github.com/vignette-project/vignette

The open source VTuber toolkit. heart

It's a .NET Core 5 application afaict. Not sure what our packaging story for those applications look like...
Metadata

homepage URL:  https://github.com/vignette-project/vignette
source URL: https://github.com/vignette-project/vignette
license: NPOSLv3
platforms: ?

Hm, maybe I'm not so interested, I can't tell but it doesn't seem like there's actually anything there there yet.


Fix being kicked from the world above 30 million blocks

Currently it's not possible to teleport up this high due to #552, but this can be reproduced clearly with a NBT editor. Flying above this height will return the player to the multiplayer screen, instead of allowing the player to just move freely (especially since blocks will probably be allowed to exist up here once #525 is fixed).

This also affects vanilla: https://bugs.mojang.com/browse/MC-123587
Closely relates to #535 and can probably be fixed alongside it.

Are you sure it's an CC world??
Using /tp ~ 30000000 ~ as well as /tp ~ 3E8 ~ works perfectly finr, even /tp ~ 1E9 ~ works as it should

Oh wait it's 1.16.1 sorry then I ran the test on 1.12.2
Maybe next time mention it so people don't have to inspect the images first

21w13a fixes this entirely: https://bugs.mojang.com/browse/MC-219901


new ai tools

dancing characters
<EMAIL_ADDRESS>

claude 3 vs gpt 4 vs custom gpt vs combo* for longer narrative - use opus, not sonnet - sonnet is bad at code

combo idea: gpt 4 has images and the code compiler so maybe claude 3 is just for narrative but technical roles happen in gpt 4? feels too complicated but a third party system calling them could be more seamless

https://www.tiktok.com/@techfren/video/7359110416607300871
leonardo ai style reference: https://twitter.com/LeonardoAi_/status/1781663639986196708
leonardo ai vs scenario<EMAIL_ADDRESS>Gligen https://www.tiktok.com/@exalclaw/video/7351497753664458027
codex https://www.tiktok.com/@gregqualls/video/7328033307479002414?lang=en
Lorekeeper https://lorekeeper.com/ - "stick FEEDBACK in account settings on LoreKeeper and it will give you 15 dollars of credits. also please come give some feedback"

luma dream machine kinda slaps
https://lumalabs.ai/dream-machine

https://www.fables.gg/

https://www.tiktok.com/t/ZTNJg5Nhv/

app.meshy.ai

https://research.nvidia.com/labs/dir/magic3d/

https://www.rosebud.ai/ai-game-creator

https://arcane.land/

https://aidungeon.com/renaissance

Ai dungeon vs novel ai
https://www.reddit.com/r/AIDungeon/s/mFXrD7RbIA

Related to licensing
https://www.reddit.com/r/AIDungeon/s/4EhoMmDLKB

Very weird
https://x.com/dr_cintas/status/1806682769927397813?s=46&t=wr67JgytWZYNZwXKRQGQKQ

AI video short
https://x.com/minchoi/status/1808183585775706236?s=46&t=wr67JgytWZYNZwXKRQGQKQ

https://www.tiktok.com/t/ZTNTLYrrT/

https://x.com/WonderDynamics

https://x.com/emmanuel_2m/status/1831745413424296178
https://x.com/emmanuel_2m/status/1831407957084664077

https://x.com/emmanuel_2m/status/1832449779802185734?s=46&t=wr67JgytWZYNZwXKRQGQKQ

https://x.com/liamesp/status/1869319333954089218?s=46&t=wr67JgytWZYNZwXKRQGQKQ

https://x.com/minchoi/status/1869905351161852081


fix: cos_bucket

:tada: This PR is included in version 1.1.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


refactor(semantic): remove ModuleRecord from Semantic

#7548  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


tf.keras.mixed_precision causes error when using tf.custom_gradient

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 22.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.12.0
Python version: 3.9
Exact command to reproduce: python test.py

Describe the problem.
I implemented a custom keras layers with some custom gradient functions (@tf.custom_gradient). It works, but when keras mixed precision policy is set, it will fail with error @tf.custom_gradient grad_fn must accept keyword argument 'variables', since function uses variables. I think it may because mixed precision policy wraps Variable into AutoCastVariable.
Describe the current behavior.
Please run the attached py script. It's a toy example that I defined a custom dense layer, whose call function will call another tf custom gradient function. It will fail in eager mode because of the error I described above. It can pass if you delete tf.keras.mixed_precision.set_global_policy(policy)
Describe the expected behavior.
It should be no functionality issues when keras mixed precision is turned on.
Contributing.

Do you want to contribute a PR? (yes/no):
Yes, if I know how to fix.

Standalone code to reproduce the issue.
import tensorflow as tf
import numpy as np

@tf.custom_gradient
def custom_bmm(x, w):
  result = tf.raw_ops.BatchMatMulV2(x=x, y=w)
  def grad(upstream):
    dx = tf.raw_ops.BatchMatMulV2(x=upstream, y=w, adj_y=True)
    dy = tf.raw_ops.BatchMatMulV2(x=x, y=upstream, adj_x=True)
    dy = tf.math.reduce_sum(dy,axis=0)
    return dx, dy
  return result, grad

class custom_layer(tf.keras.layers.Layer):
  def __init__(self, num_outputs):
    super(custom_layer, self).__init__()
    self.num_outputs = num_outputs

  def build(self, input_shape):
    self.kernel = self.add_weight("kernel",
                                  shape=[int(input_shape[-1]),
                                  self.num_outputs])

  def call(self, inputs, training=None):
    return custom_bmm(inputs, self.kernel)

policy = tf.keras.mixed_precision.Policy('mixed_bfloat16')
tf.keras.mixed_precision.set_global_policy(policy)

inputs = tf.keras.Input(shape=(1,3,))
x = custom_layer(4)(inputs)
model = tf.keras.Model(inputs=inputs, outputs=x)

result = custom_layer(4)(np.array([[[1.,2.,3.]],[[4.,5.,6.]]]).astype(np.float32))
print(result)
result = model(np.array([[[1.,2.,3.]],[[4.,5.,6.]]]).astype(np.float32))
print(result)

Source code / logs.
2023-05-18 21:40:18.428090: I tensorflow/tsl/cuda/cudart_stub.cc:28] Could not find cuda drivers on your machine, GPU will not be used.
2023-05-18 21:40:18.456493: I tensorflow/tsl/cuda/cudart_stub.cc:28] Could not find cuda drivers on your machine, GPU will not be used.
2023-05-18 21:40:18.456768: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI AVX512_BF16 AVX_VNNI AMX_TILE AMX_INT8 AMX_BF16 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-05-18 21:40:18.909846: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
Traceback (most recent call last):
File "test.py", line 37, in
result = custom_layer(4)(np.array([[[1.,2.,3.]],[[4.,5.,6.]]]).astype(np.float32))
File "python3.9/site-packages/keras/utils/traceback_utils.py", line 70, in error_handler
raise e.with_traceback(filtered_tb) from None
File "test.py", line 25, in call
return custom_bmm(inputs, self.kernel)
TypeError: Exception encountered when calling layer 'custom_layer_1' (type custom_layer).
@tf.custom_gradient grad_fn must accept keyword argument 'variables', since function uses variables: [<tf.Variable 'custom_layer_1/kernel:0' shape=(3, 4) dtype=float32, numpy=
array([[-0.57051015,  0.01343554, -0.798573  , -0.41828716],
[-0.4713288 ,  0.35838473, -0.6395558 , -0.00528103],
[ 0.7071755 , -0.44282293, -0.41892177, -0.65287685]],
dtype=float32)>]
Call arguments received by layer 'custom_layer_1' (type custom_layer):
• inputs=tf.Tensor(shape=(2, 1, 3), dtype=bfloat16)
• training=None

Hi, the workaround is fine for me


Fix/reverse pitch

Description

Fixed reversed pitch angle calculation
Fixed extra comma of eagleye_pp csv

Step to Test

Compile
Run eagleye_rt or pp to check pitch angle
Run eagleye_pp and check that there is not comma at end of line

@rsasaki0109 Changed base branch from main-ros1 to develop-ros1


Can't define recursive regex match expressions

You cant define recursive regex patterns to match,
this would be useful when doing something, that might need matched parenthesis and you are already using a begin/end or this method is not applicable.
Example
{
    "$schema": "https://raw.githubusercontent.com/martinring/tmlanguage/master/tmlanguage.json",
    "scopeName": "source.test",
    "name": "A test",
    "patterns": [
        {
            "include": "#my-recursive-using-thing"
        }
    ],
    "repository": {
        "my-recursive-using-thing": {
            "match": "(\\((?:[^()]+|(?1))\\s+or\\s+(?:[^()]+|(?1))\\))\\s+([_-]?[A-Za-z][0-9A-Z_a-z-]*)(;)",
            "captures": {
                "1": {
                    "patterns": [
                        {
                            "include": "#my-recursive-thing"
                        }
                    ]
                },
                "2": {
                    "name": "entity.name"
                }
            }
        },
        "my-recursive-thing": {
            "match": "(\\(([^()]+|(?1))\\s+(or)\\s+((?1)|[^()]+)\\))",
            "captures": {
                "1": {
                    "patterns": [
                        {
                            "include": "#my-recursive-thing"
                        },
                        {
                            "match": "[_-]?[A-Za-z][0-9A-Z_a-z-]*",
                            "name": "storage.type"
                        }
                    ]
                },
                "2": {
                    "name": "keyword.operator.or"
                },
                "3": {
                    "patterns": [
                        {
                            "include": "#my-recursive-thing"
                        },
                        {
                            "match": "[_-]?[A-Za-z][0-9A-Z_a-z-]*",
                            "name": "storage.type"
                        }
                    ]
                }
            }
        }
    }
}

should match both:
(foo or bar) one_test;
(foo or (another_foo or bar)) test;

This uses the PCRE recursion feature, which is currently not supported and cannot easily written as a begin/end matcher.
Solutions
Use PCRE(2) like mainline vs-code for lookups
Additional Benefits:

You can possibly allow predefinitions of often used templates to avoid unnecessary repetition,
like in this example [_-]?[A-Za-z][0-9A-Z_a-z-]* could be predefined with a name and all further references could just use (?&name) to use them

If my memory is correct, recursive back references should work. We've done balanced parentheses using this in the C++ syntax. (I'm not a vs code contributor btw)
It's not PCRE though, it's Oniguruma.
https://macromates.com/manual/en/regular_expressions
So the (?1) shouldn't work as a subexpression or a backreference. The syntax for subexpression is \g<1> but I don't remember testing it in VS Code.
The subexpression might be screwing up the whole execution, making it look like the backref doesn't work.

I'll try that then, did not know of the onigoruma dialect.

(?1) will cause an error and silently fail

List of all Oniguruma's expresses can be found in the docs section on its github
https://github.com/kkos/oniguruma/blob/master/doc/RE

I think this issue can be closed


[SPARK-14544] [SQL] improve performance of SQL UI tab

What changes were proposed in this pull request?
This PR improve the performance of SQL UI by:

remove the details column in all executions page (the first page in SQL tab). We can check the details by enter the execution page.
break-all is super slow in Chrome recently, so switch to break-word.
Using "display: none" to hide a block.
using one js closure for  for all the executions, not one for each.
remove the height limitation of details, don't need to scroll it in the tiny window.

How was this patch tested?
Exists tests.

cc @zsxwing

Looks good!

Test build #55550 has started for PR 12311 at commit b25701f.

LGTM

Test build #55550 has finished for PR 12311 at commit b25701f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #55575 has finished for PR 12311 at commit b25701f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #2776 has started for PR 12311 at commit b25701f.

Test build #2776 has finished for PR 12311 at commit b25701f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #2778 has started for PR 12311 at commit b25701f.

Test build #2778 has finished for PR 12311 at commit b25701f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

@zsxwing Could you help me to fix the streaming test suite? It seems related to this PR.

Test build #55622 has started for PR 12311 at commit a8f209a.

Test build #55622 has finished for PR 12311 at commit a8f209a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #55638 has started for PR 12311 at commit a8f209a.

Test build #55638 has finished for PR 12311 at commit a8f209a.

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

Merged build finished. Test PASSed.

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

Merging this into master, thanks!


feat: add support for nonce

This PR adds a CSP nonce if given in $global. fixes: https://github.com/marko-js/express/issues/8

Description
The midstream redirect script should include a CSP nonce if one exists in $global

Since we don't have a handle to $global in the redirect function, I can't really think of a better way to add the nonce in without either

changing the signature of res.redirect (not ideal)
adding a new variable to req or res which might break others

Motivation and Context
We need a CSP Nonce for our security settings

Screenshots (if appropriate):
Checklist:

[x] I have updated/added documentation affected by my changes.
[x] I have added tests to cover my changes.

cc @DylanPiercey

I signed the CLA but still showing an error

@snyamathi thanks for the reminder and the PR. I went ahead and improved/simplified a few things and will get out the patch shortly 😄

Got it - yeah I like the approach using a symbol to store the cspNonce on the res object - LGTM and much easier.

Awesome. I'm just working to figure out what's wrong with the format CI. Once I get that fixed I'll get this merged.

Seems like github just doesn't like having a commit action from a fork so 🤷. Thanks again for the PR, just going to go ahead and merge!


Make Ansible Automation Platform chart modular

What is this PR About?
Remove the OperatorGroup and Subscription Template from chart in favor of using helm chart designed for the purpose as a dependency
How do we test this?
Run the chart ;)
cc: @redhat-cop/day-in-the-life

Im good with this change Kuba, just looks like the CI is failing for some reason

Yeah, and I'm trying to work out why it fails:) Helm chart renders correctly

Yeah, and I'm trying to work out why it fails:) Helm chart renders correctly

hey  @jfilipcz - did you get any further with the conf- test failing ? cheers

Yeah, should be solved by https://github.com/redhat-cop/bats-library/pull/36

Or not, there seems to be other test problem with the fact we're using sub-chart

@eformat , tests were fixed, so the thing is ready to be merged


Feature Request: Arbitrary interval support

I work in software and we organize our work into sprints, which are generally 2 weeks, but might be longer if a major holiday falls in the middle.
I'd love to be able to organize my notes by sprint - so using an arbitrary length of time.
Using a command would start a new sprint, which would increment a sprint number, create a new note with the corresponding number in the sprints folder.
Obviously I can accomplish this manually, but would it would be great to have tie-ins with commands.

I was coming to add this same request.
I would love to help this happen, but before that @liamcain , is this something you would like to see on this plugin?


Add Solution to LC Problem - 2032

Related Issue
Fixes #3480 Issue
Description
Two Out of Three
Given three integer arrays nums1, nums2, and nums3, return a distinct array containing all the values that are present in at least two out of the three arrays. You may return the values in any order.
Type of PR

[ ] Bug fix
[x] Feature enhancement
[x] Documentation update
[ ] Security enhancement
[ ] Other (specify): _______________

Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have followed the code style guidelines of this project.
[x] I have checked for any existing open issues that my pull request may address.
[x] I have ensured that my changes do not break any existing functionality.
[x] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[x] I have read the resources for guidance listed below.
[x] I have followed security best practices in my code changes.

Resources for Guidance
Please read the following resources before submitting your contribution:

[ ] Code Harbor Hub Community Features
[ ] Docusaurus Documentation
[ ] React.js Documentation
[ ] Markdown Guide
[ ] MDX Documentation
[ ] Mermaid Documentation

Resources for DSA Features on Our Site:

[ ] Tabs: Switch Languages
[ ] Line Numbering
[ ] Math Equations

Contribution Guidelines
Thank you for considering contributing to our project! To ensure smooth collaboration and effective contribution management, please adhere to the following guidelines:
Issue Creation

Limit on Issues:

Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.

Contribution Levels

Basic Contributions:

This project is primarily focused on documentation. Most of the setup has been completed, so contributors will generally need to work on basic code tasks, such as writing tests.
For these tasks, issues will be assigned the level1 label.

Acknowledging Hard Work:

If a contributor puts in significant effort on a task, the issue will be upgraded to level2. This is our way of recognizing and appreciating extra effort.

Feature Additions and Component Work:

Contributors working on new features or components using JSX/TSX will be assigned a level based on the complexity and quality of their work.
The more complex and valuable the contribution, the higher the level assigned.

Level Definitions

Level1:

Tasks are straightforward, such as fixing minor bugs, writing tests, or making simple documentation updates.

Level2:

Tasks require more effort, such as addressing complex bugs, improving existing features, or making substantial documentation improvements.

Level3:

Tasks are highly complex and involve significant new feature development, major refactoring, or extensive contributions to the project’s core components.

We look forward to your contributions and appreciate your effort in helping us improve the project!

I have done all the changes you suggested to me
Kindly view them and let me know if there are any mistakes

@sreevidya-16, Please compare your code with our format
---
id: enter-id-for-routes
title: Title of Problem
sidebar_label: Number of Problem
tags: [tag1, tag2, ...., tagn]
description: Write description for SEO in few words and in only string and can't use any HTML tags in this line
sidebar_position: enter position number
---

## Problem Statement

### Problem Description

### Examples

### Constraints

## Solution of Given Problem

### Intuition and Approach

### Approaches

## Video Explanation of Given Problem

<Tabs>
  <TabItem value="en" label="English">
    <Tabs>
      <TabItem value="javascript" label="JavaScript">
        <LiteYouTubeEmbed
          id="video_id"
          params="autoplay=1&autohide=1&showinfo=0&rel=0"
          title="Problem Explanation | Solution | Approach"
          poster="maxresdefault"
          webp
        />
      </TabItem>
      <TabItem value="python" label="Python">
        <LiteYouTubeEmbed
          id="video_id"
          params="autoplay=1&autohide=1&showinfo=0&rel=0"
          title="Problem Explanation | Solution | Approach"
          poster="maxresdefault"
          webp
        />
      </TabItem>
      <TabItem value="java" label="Java">
        <LiteYouTubeEmbed
          id="video_id"
          params="autoplay=1&autohide=1&showinfo=0&rel=0"
          title="Problem Explanation | Solution | Approach"
          poster="maxresdefault"
          webp
        />
      </TabItem>
    </Tabs>
  </TabItem>

  <TabItem value="hi" label="Hindi">
    <LiteYouTubeEmbed
      id="video_id"
      params="autoplay=1&autohide=1&showinfo=0&rel=0"
      title="Problem Explanation | Solution | Approach"
      poster="maxresdefault"
      webp
    />
  </TabItem>
</Tabs>

---

Authors:

{['github_username'].map(username => (
 <Author key={username} username={username} />
))}

and resolve your issue what's wrong in your code.


Fix tests and produce proper error from StatusList.decode.

Codecov Report

Merging #8 (8b38c21) into main (1ed2b73) will decrease coverage by 0.74%.
The diff coverage is 85.71%.

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

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   96.29%   95.55%   -0.75%
==========================================
  Files           3        3
  Lines         135      135
==========================================
- Hits          130      129       -1
- Misses          5        6       +1

Impacted Files
Coverage Δ

StatusList.js
92.30% <83.33%> (-7.70%)
:arrow_down:

main.js
95.83% <100.00%> (-0.14%)
:arrow_down:

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 1ed2b73...a78233e. Read the comment docs.


Add new icons

Below mentioned icons are needed in IRIS

 Mobile.svg
 Desktop.svg
 Attendee.svg
 TextAlignLeft
 TextAlignCenter
 TextAlignRight
 NumberedList

cc: @seanmcintyre

Hello. This is not the appropriate channel to request new icons be added to Iris. Please ask your designer to bring them to the next Design Systems Review meeting.


A11y_AzureDataStudioStayHealthy_Welcome_Resources_Keyboard:The tab focus is not visible after deploy a server button.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags:#A11y_AzureDataStudioStayHealthy;#A11ySev2;#A11yTCS;#A11yMAS;#NonBenchmark;#DesktopApp;#MAS2.4.7;#SQL Azure Data Studio;#FTP;#Keyboard;#STP;#Win32;
Environment Details:
Application Name: Azure Data Studio
Version: 1.19.0-insider
Commit: e8dc0d1
Date: 2020-05-27T17:24:57.733Z
VS Code: 1.46.0
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18363
Additional Details:
MAS References:MAS2.4.7
Repro Steps:
1 . Launch Azure Data Studio Insiders application.
2 . If Welcome screen is not shown on application launch, then navigate to Help --> Welcome.
3 . Navigate till Resources section "History tab" using tab key,
Actual:
The tab focus after deploy a server button is not visible and when screen reader is on the invisible focus make an announcement of a radio button and on the next tab key the focus is on the history tab under Resources section.
Expected:
The tab focus after deploy a server button should be on the history tab and the screen reader should make the announcement of it when in focus under Resources section
User Impact:
Keyboard users will not know where the focus has been moved too.
Attachment link for Reference:
10621A11y_AzureDataStudioStayHealthy_Welcome_Resources_Keyboard.zip

@v-bbrady please tag this issue in the pr where you are adding tabindex=-1 for the invisible element

Version: 1.43.0-insider (user setup)
Commit: b4d5a08199e9e0cb0138ce540c086ac32d072019
Date: 2023-03-07T06:31:08.002Z
VS Code: 1.67.0
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.25300
#closed;
Video 3-07 at 17.19.webm


[Snyk] Fix for 1 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

661/1000  Why? Recently disclosed, Has a fix available, CVSS 7.5
Allocation of Resources Without Limits or Throttling SNYK-JAVA-COMNIMBUSDS-6247633
no.ks.fiks:maskinporten-client:  3.0.1 -> 3.1.9
Yes
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Vulnerabilities that could not be fixed

Upgrade:

Could not upgrade<EMAIL_ADDRESS>to<EMAIL_ADDRESS>Reason could not apply upgrade, dependency is managed externally ; Location: https://maven-central.storage-download.googleapis.com/maven2/io/micronaut/security/micronaut-security-bom/3.11.0/micronaut-security-bom-3.11.0.pom

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Allocation of Resources Without Limits or Throttling

Will open a new snyk, PR. Close the outdated once.


[Feature Request] Bert Model support

🚀 Feature

Motivation

I want to do a project based on  mlc-llm, and I think maybe add a new architecture (like bert) to it according to the tutorialhttps://github.com/mlc-ai/notebooks/blob/main/mlc-llm/tutorial_add_new_model_architecture_in_tvm_nn_module.ipynb  would be possible. I wonder if there is any big problem? I noticed that almost all models that already exist are decoder-only, so maybe to add a model totally different is not easy? It will be a big help if you can tell me it is not a good idea or it is possible to follow the tutorial, thank you very much!!!
Alternatives

Additional context

encoder-decoder models like bert cannot use KVCache, which is definitely out of our scope. If you want to run bert model, TVM is a good choice instead of MLC-LLM

Thank you very much!! It is a big help!


no event is emitted for docker load

Output of docker version:
$ docker version
Client:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   99b71ce
 Built:        Fri Mar 11 18:42:00 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.3
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   99b71ce
 Built:        Fri Mar 11 18:42:00 2016
 OS/Arch:      linux/amd64

Output of docker info:
$ docker info
Containers: 36
 Running: 31
 Paused: 0
 Stopped: 5
Images: 289
Server Version: 1.10.3
Storage Driver: aufs
 Root Dir: /nail/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 970
 Dirperm1 Supported: false
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: null host bridge
Kernel Version: 3.13.0-83-generic
Operating System: <unknown>
OSType: linux
Architecture: x86_64
CPUs: 24
Total Memory: 125.9 GiB
Name: dev34-devc
ID: VKMX:YMJ2:3NGV:5J6I:5RYM:AVBK:QPOZ:ODYE:VQ2D:AF2J:2LEM:TKTE
WARNING: No swap limit support

Additional environment details (AWS, VirtualBox, physical, etc.):
This is a physical ubuntu Lucid (I know, ancient) box with a heavily upgraded kernel.
Steps to reproduce the issue:

Run this script

#!/bin/bash
set -eux
show-events() {
docker events --since "$(date -d '10 seconds ago' +%s)" --until "$(date +%s)"
echo end
echo
}
show-events
docker pull alpine
docker save alpine > alpine.tar
docker rmi -f alpine
cat alpine.tar | docker load
docker run --rm alpine echo ok

**Describe the results you received:**

Output:

2016-04-12T11:02:29.701790619-07:00 image pull alpine:latest (name=alpine)
2016-04-12T11:02:29.880773197-07:00 image untag sha256:d7a513a663c1a6dcdba9ed832ca53c02ac2af0c333322cd6ca92936d1d9917ac (name=sha256:d7a513a663c1a6dcdba9ed832ca53c02ac2af0c333322cd6ca92936d1d9917ac)
2016-04-12T11:02:29.880937546-07:00 image delete sha256:d7a513a663c1a6dcdba9ed832ca53c02ac2af0c333322cd6ca92936d1d9917ac (name=sha256:d7a513a663c1a6dcdba9ed832ca53c02ac2af0c333322cd6ca92936d1d9917ac)
2016-04-12T11:02:30.129892929-07:00 container create 818db8f87879781e7f4d227cc9d5d17b06d1c2b2a305c85bac6bf84d5353200d (name=evil_bhaskara, image=alpine)
2016-04-12T11:02:30.132069682-07:00 container attach 818db8f87879781e7f4d227cc9d5d17b06d1c2b2a305c85bac6bf84d5353200d (image=alpine, name=evil_bhaskara)
2016-04-12T11:02:30.434189124-07:00 network connect 93178695d5a46bd7627a3e8d91b9d891958d2f12f3f1a541c1b2036a44649df3 (container=818db8f87879781e7f4d227cc9d5d17b06d1c2b2a305c85bac6bf84d5353200d, name=bridge, type=bridge)
2016-04-12T11:02:30.434797885-07:00 container start 818db8f87879781e7f4d227cc9d5d17b06d1c2b2a305c85bac6bf84d5353200d (image=alpine, name=evil_bhaskara)
2016-04-12T11:02:30.646233214-07:00 container die 818db8f87879781e7f4d227cc9d5d17b06d1c2b2a305c85bac6bf84d5353200d (image=alpine, name=evil_bhaskara)
2016-04-12T11:02:30.967371724-07:00 network disconnect 93178695d5a46bd7627a3e8d91b9d891958d2f12f3f1a541c1b2036a44649df3 (name=bridge, type=bridge, container=818db8f87879781e7f4d227cc9d5d17b06d1c2b2a305c85bac6bf84d5353200d)
2016-04-12T11:02:31.037112757-07:00 container destroy 818db8f87879781e7f4d227cc9d5d17b06d1c2b2a305c85bac6bf84d5353200d (image=alpine, name=evil_bhaskara)

**Describe the results you expected:**

There should be *some* event between the above "image delete" and "container create" to represent the docker load.

From https://docs.docker.com/engine/reference/commandline/events/
it seems docker not have load events designed. Need some cores to confirm if it is work as design

Looks indeed that we have an event for import, but not for load.
@calavera any objections against adding that as an event?

"image create" would seem appropriate, triggered in both cases

Full disclosure: I've worked around this issue, and so no longer intend to patch it.
We've transitioned from a s3 cp | docker load strategy to docker run registry:2; docker pull; docker stop. This new way is a bit more complex, but avoids the above issue (since we now trigger the "pull" event), while being a bit more network efficient.

Thanks for letting us know @bukzor. I'll leave this issue open, because possibly it makes sense to implement it, but happy to hear @calavera's thoughts on this

my opinion is that every action should generate an event.

Yes, good ways to improve such image event. I could take a look at it and see what can add

#dibs  Let me know if any option against it. @thaJeztah

No objections @HackToday, I just wanted to check with @calavera first, to be sure there wasn't a specific reason why it was missing in the first place :smile:


(alldup) Failing Validator due to invalid URL's

Chocolatey Version
1.3.0
Chocolatey License
None
Package Version
4.5.28
Current Behaviour
Package is failing Validator due to invalid licenseUrl. The releaseNotes URL is also invalid.
Expected Behaviour
Package passes Validator.
Steps To Reproduce
See moderation message on the package page.
Environment
Package Validator.

Chocolatey Log
N/A

Anything else?
No response

Re-Opening as this issue is known as a issue that needs to be fixed, and is tracked in a parent issue.

@pauby I see you have commits on this issue. To me they are looking good so I would like to know if this still up for grabs?

There is a PR up for it and I think there is an unresolved comment on there that I've not had time to get back to.


Reconciler for OSM CRDs

The goal of this GitHub issue is to create a feature flagged feature where Open Service Mesh Control plane will deploy, manage, and maintain CRDs. This feature will help customers who want OSM to maintain the consistency of the system. In the case of an accidental deletion of the CRDs or mutations, which could adversely affect the service mesh, OSM Control Plane would restore the artifacts and keep the mesh operating.
Tasks :

[X] Move CRD deployment from helm to osm-bootstrap
[ ] Build a reconciler client in OSM that watches for CRD updates/deleted
[ ] Add reconciliation logic for OSM to manage CRDs
[ ] Migrate existing mutating webhook reconciler to this framework

Tasks :

[ ] Move CRD deployment from helm to osm-bootstrap
[ ] Build a reconciler client in OSM that watches for CRD updates/deleted
[ ] Add reconciliation logic for OSM to manage CRDs
[ ] Migrate existing mutating webhook reconciler to this framework

@snehachhabria could you confirm this behavior is opt in and not the default experience?

@snehachhabria could you confirm this behavior is opt in and not the default experience?

Yes the reconciliation process will be configurable and will be turned off by default


Process multiple css queries

One document could contain multiple points of interests for user.
Currently, the API takes one html and one css, and returns relevant results.
Need to supports multiple css in single API call.
Every css should be unique, and used as a key for response.
Like:
css1: .key
css2: .value

   Key
   Value

[
  {
    "key": ".key",
    "values": [ { "value": "Key" } ]
  },
  {
    "key": ".value",
    "values": [ { "value": "Value" } ]
  }
]

Introduce a method which takes an array of css.
Get the JsonArrays via same underlying calls.
Generate a new JsonObject with "key" around.
Put this object into result array

Looks like a day-two.
2SP

[Test]
    public void Demo1()
    {
        var css = ".target";
        var html = """

                 Two
                 Three
                 Four

  Key 	  Value

  Width 	  2

  Height 	  3

        """;

        string expected = """
        [
            {
                "key": ".target",
                "values":
                [
                    { "value": "Two" },
                    { "value": "Three" },
                    { "value": "Four" },
                    { "value":
                        {
                            "headers": ["Key", "Value"],
                            "values": [
                                ["Width", "2"],
                                ["Height", "3"]
                            ]
                        }
                    }
                ]
            }
        ]
        """;

        var actual = new Scrapper()
            .Scrap(html, css)
            .AsJson()
            .ToJsonString();

        Helpers.AssertJson(expected, actual);


No Content with AJAX

Hi everyone,
I have some issue with dataTable and AJAX.
There is my configuration :  http://zupimages.net/viewer.php?id=17/27/blsv.png
When I display my data with the success of my Ajax request, i have all of my row. But datatable just display "No data available in the table".
There is something I miss ?
Thanks for your help.

Thanks for your question. However, per the Contributing document and new issue template text, please post support requests in the DataTables forums. Please also ensure that you link to a running test case showing the issue so it can be debugged.


Reduce unnecesary IOBufferBlock allocation

MIOBuffer::write_avail() adds IOBlocks if needed. As @shinrich pointed at #5894 , some of them are unnecessary. It looks like the root cause is MIOBuffer::add_block always overwrite the writer->next if the "next" is writable.
This appends new IOBufferBlock in the tail of the block list for the general fix.
Unit tests will be added.

Downloading 10MB file on HTTP/1.1 over TLS.
Allocs     | Frees      | Size In-use          | Avg Size   | Location
-----------|------------|----------------------|------------|--------------------------------------------------------------------
       800 |        800 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:6500

With this change
Allocs     | Frees      | Size In-use          | Avg Size   | Location
-----------|------------|----------------------|------------|--------------------------------------------------------------------
       321 |        321 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:6500

Looks good.  By not adjusting the _writer pointer, you won't create as many partially filled blocked.

Comparison of allocations while downloading 10MB file
Before
HTTP/1.1 over TLS
Allocs     | Frees      | Size In-use          | Avg Size   | Location
-----------|------------|----------------------|------------|--------------------------------------------------------------------
         1 |          1 |                    0 |          0 | memory/IOBuffer/Http1ClientSession.cc:188
         1 |          1 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:5982
       800 |        800 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:6521
         1 |          0 |                 8192 |       8192 | memory/IOBuffer/Http1ServerSession.cc:78
         1 |          1 |                    0 |          0 | memory/IOBuffer/./P_SSLNetVConnection.h:195
         2 |          1 |                  256 |        256 | memory/IOBuffer/SSLSessionCache.cc:158
                                          8448 |            | TOTAL

HTTP/2
Allocs     | Frees      | Size In-use          | Avg Size   | Location
-----------|------------|----------------------|------------|--------------------------------------------------------------------
         1 |          1 |                    0 |          0 | memory/IOBuffer/UNKNOWN-LOCATION
         1 |          1 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:5982
      1118 |       1118 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:6521
         1 |          0 |                 8192 |       8192 | memory/IOBuffer/Http1ServerSession.cc:78
        11 |         11 |                    0 |          0 | memory/IOBuffer/Http2ClientSession.cc:206
       640 |        640 |                    0 |          0 | memory/IOBuffer/Http2ClientSession.cc:210
       644 |        644 |                    0 |          0 | memory/IOBuffer/./Http2ClientSession.h:102
         1 |          1 |                    0 |          0 | memory/IOBuffer/./P_SSLNetVConnection.h:195
         2 |          1 |                  256 |        256 | memory/IOBuffer/SSLSessionCache.cc:158
                                          8448 |            | TOTAL

After
HTTP/1.1 over TLS
Allocs     | Frees      | Size In-use          | Avg Size   | Location
-----------|------------|----------------------|------------|--------------------------------------------------------------------
         1 |          1 |                    0 |          0 | memory/IOBuffer/Http1ClientSession.cc:188
         1 |          1 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:5982
       322 |        322 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:6521
         1 |          0 |                 8192 |       8192 | memory/IOBuffer/Http1ServerSession.cc:78
         1 |          1 |                    0 |          0 | memory/IOBuffer/./P_SSLNetVConnection.h:195
         2 |          1 |                  256 |        256 | memory/IOBuffer/SSLSessionCache.cc:158
                                          8448 |            | TOTAL

HTTP/2
Allocs     | Frees      | Size In-use          | Avg Size   | Location
-----------|------------|----------------------|------------|--------------------------------------------------------------------
         1 |          1 |                    0 |          0 | memory/IOBuffer/UNKNOWN-LOCATION
         1 |          1 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:5982
       322 |        322 |                    0 |          0 | memory/IOBuffer/HttpSM.cc:6521
         1 |          0 |                 8192 |       8192 | memory/IOBuffer/Http1ServerSession.cc:78
         5 |          5 |                    0 |          0 | memory/IOBuffer/Http2ClientSession.cc:206
       640 |        640 |                    0 |          0 | memory/IOBuffer/Http2ClientSession.cc:210
       644 |        644 |                    0 |          0 | memory/IOBuffer/./Http2ClientSession.h:102
         1 |          1 |                    0 |          0 | memory/IOBuffer/./P_SSLNetVConnection.h:195
         2 |          1 |                  256 |        256 | memory/IOBuffer/SSLSessionCache.cc:158
                                          8448 |            | TOTAL

Looks good, But my question is why _writer is not the tail of a IOBufferBlock chain. the _writer should always move to the end when you are append the block. Otherwise the data might be polluted when water_market is set to not 0.

@scw00 The answer is, as @SolidWallOfCode pointed out on slack, _writer must always be the first writable block in the chain.

[approve ci autest]

@masaori335 @SolidWallOfCode The _writer may be the last readable block and _writer->next is nullptr. The last readable block may not be writeable.

Right, that's possible.

The _writer always be the last written or appended block.

Pushed new changes to follow @oknet's advice on slack.

Do nothing when the next block of the current writer exists instead of using add_block_at_tail().
Update unit tests
Comparison is the same as previous change ( 322 allocations of memory/IOBuffer//HttpSM.cc:6521 )

[approve ci autest]

I think this might be a candidate for 8.0.6 as well.

Cherry-picked to v9.0.x branch.


feat: add eslint + prettier for auto code format and style

add tools eslint and prettier
prettier was executed in the code
prettier config on package.json
updating types based on @typescript-eslint/recommended suggestions

:100:


Enable security flow to get only the security cr name via swagger file in user namespace

Description:
This should enable the security flow as follows.

Get the name of the security cr from the swagger. (ex: security:
- basicauth:
- read
- write)
Get all securities defined in swagger and get crs from user namespace and update config files and mount certificates for all of securities.
Modify the swagger by adding securitySchemes for defined securities.

Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

resloved in https://github.com/wso2/k8s-apim-operator/pull/69


Use async find provider only for file scheme

This pull request modifies the ExplorerView to utilize the async find provider exclusively for resources with the 'file' scheme. This change ensures that the find results provider is only instantiated when all roots support the file scheme.
fixes #230483

Instead of withdrawing Explorer async find for every scheme other than file I suggest you consider adding a temporary window-scoped setting allowing users to turn it off. This will provide a solution for users of FileSystemProviders for which there is not yet a FileSearchProvider, and should also placate users working with ordinary files (i.e. file scheme) who are unhappy with the loss of highlight-mode finding in Explorer tree.
Perhaps make the setting default in such a way that the new async find provider is on for file scheme and off for all other schemes.
Ideally you'd be able to detect whether a FileSearchProvider for a given scheme is (a) available, (b) contributed but not yet initialized, or (c) not contributed. But if that's infeasible in time for a recovery release then the setting would suffice, plus it would bring a benefit for discontented file-scheme users.

Can you now use what @andreamah added in #230731?

Not for the recovery release as it requires more changes which could be risky. If I use it, it will currently only work for file scheme and maybe remote, but not for extensions as the register late. Will think of a fix for insiders


Debug using devfile

User Story
As a developer I want to debug my component so that I can resolve bugs and use it for other cases.
Acceptance Criteria

[ ] It should have a command to start the component in debug mode
[ ] It should port-forward the given port or auto select a default one
[ ] It should have a command to turn off debug mode for the component

Links

Feature Request:
Related issue:

/kind user-story

I managed to get debug working with a devfile for a nodejs component.
Using https://github.com/openshift/odo/pull/2735, I changed the devrun command to what we are using in the odo-init image https://github.com/openshift/odo-init-image/blob/d64042928158c8d975b690ac9df28731c2ddd28b/language-scripts/nodejs/dev-run#L71. Then I triggered port forwarding using oc and used the normal vscode steps to achive it.
So we would require a devdebug step in the devfile which would kill the current running application and start it in debug mode. Then the odo port-forward would be used and would replace the oc port-forward step used above.

But I don't expect all devfiles to have a devdebug step. What should we do in such a scenario? @kadel @girishramnani

in my opinion, this would work similar to current components where certain images support debug and other dont, we can decide on a name of the step like you mentioned devdebug here and then check which of the devfiles from the registry have this step defined. Based on this we show the Supported and Unsupported devfiles.

in my opinion, this would work similar to current components where certain images support debug and other dont, we can decide on a name of the step like you mentioned devdebug here and then check which of the devfiles from the registry have this step defined. Based on this we show the Supported and Unsupported devfiles.

OK. Maybe we can inform the user that he can add a devdebug step manually in the devfile. Also we can ask the user he wants to manually add a devdebug step. If he says yes, we can just add a devdebug basic template in devfile.

After some discussion with @kadel , we have decided to name the step debug or debugrun in the devfile. And instead of having a dedicated command, we will start the debug mode when the user executes odo push --debug.
The user can also declare a env variable in the devfile to trigger debug mode.
@girishramnani @elsony WDYT?

After some discussion with @kadel , we have decided to name the step debug or debugrun in the devfile. And instead of having a dedicated command, we will start the debug mode when the user executes odo push --debug. This will execute the debugrun step in the devfile.
The user can also declare a env variable in the devfile to trigger debug mode.
@girishramnani @elsony WDYT?

After today's cabal, this LGTM and makes sense. If a user wants their application to initially run in debug mode, it makes sense to pass in --debug to odo push.

After some discussion with @kadel , we have decided to name the step debug or debugrun in the devfile. And instead of having a dedicated command, we will start the debug mode when the user executes odo push --debug. This will execute the debugrun step in the devfile.
The user can also declare a env variable in the devfile to trigger debug mode.
@girishramnani @elsony WDYT?

Yes, sounds good to me.

@girishramnani @kadel How will we detect the remote port of the debugrun step?

we need to allow the user to set the remote debug port if needed which means the debugrun command should support env interpolation i.e.  a sample devrun command node —inspect $DEV_PORT main.js the DEV_PORT can be env variable expose on the s2i script ( for now as we still plan to use it ). This can be a possible solution.

We should default to something but the user shouldn’t need to provide it from the get go as they can rely on conventions.

remote port should only be in the local env if it’s set specifically by the user.


Adds dependency retrieval workflow

Resolves #390
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

I'm running into some odd behaviour when I run retrieval locally, maybe you can help me out.

I deleted a version in the buildpack.toml
cd into dependency
run make retrieve buildpackTomlPath=$PWD/../buildpack.toml output=$PWD/metadata.json
it fails out with:
2022/10/07 12:29:28 no license file was found exit status 1 make: *** [retrieve] Error 1

Hey, I just re-tried the same experiment but now instead of failing it seemingly hangs

@ForestEckhardt thanks for making those changes. It runs faster now, and looks like most of that time comes from A) license generation and B) artifact validation, both of these areas are important and we can't really cut down on.
The code is reasonable, well-organized, and tested. It also passes validation and the metadata looks good to me.

I ran into an issue with using the generated metadata.json with jam update-dependencies and I realized that the metadata you generate has the ID dotnet-core-sdk, whereas the buildpack.toml constraints expect dotnet-sdk. Can you fix that?

Sorry forest, I almost missed it but can you update the .syncignore to exclude the new workflow and include the old update-dependencies workflow please?


fix: pass worker name to syncAssets in dev

This fix passes the correct worker name to syncAssets during wrangler dev. This function uses the name to create the backing kv store for a Workers Sites definition, so it's important we get the name right.
I also fixed the lint warning introduced in https://github.com/cloudflare/wrangler2/pull/321, to pass props.enableLocalPersistence as a dependency in the useEffect call that starts the "local" mode dev server.

I'm working on an implementation of act for ink-testing library too, so that should make testing dev possible.


[BUG] parse_json processor doesn't handle pointers with destinations

Describe the bug
When using a pointer and a destination together with the parse_json processor, the output data in the destination contains an incorrect nested key.
To Reproduce
Steps to reproduce the behavior:
pipelines.yaml:
parse-json-pipeline:
  source:
    http:
  sink:
    - stdout:
  processor:
    - parse_json:
        source: "message"
        destination: "target"
        pointer: "outer_key/inner_key"

Input:
JSON='[{"message": "{\"outer_key\": {\"inner_key\": \"1220\"}}"}]'
curl -k -XPOST -H "Content-Type: application/json" -d "${JSON}" http://localhost:2021/log/ingest

Output:
{"message":"{\"outer_key\": {\"inner_key\": \"1220\"}}","target":{"inner_key":"1220"}}

Expected behavior
Expected / desired output:
{"message":"{\"outer_key\": {\"inner_key\": \"1220\"}}","target":"1220"}

Environment (please complete the following information):
opensearchproject/data-prepper:latest
tag:2c474f8f05fb
data-prepper 2.5
Additional context
When doing complex conversions (e.g. input JSON to SS4O or OTEL schema), users need the ability to perform standard JSON re-mapping from a source schema into a destination schema. This currently isn't supported by DataPrepper.

To achieve the desired result, you can use this configuration.
processor:
    - parse_json:
        source: "message"
        pointer: "outer_key/inner_key"
    - rename_keys:
        entries:
          - from_key: "inner_key"
            to_key: "target"

Closing as this is not a bug


client function in ReplicatedClients is O(n)

Is there any reason why a vector instead of a hash map is used here?
https://github.com/projectharmonia/bevy_replicon/blob/c740527a268a3a902db5852f2cefa879d5cb4cc5/src/core/replicated_clients.rs#L21-L25
I have the following use-case: Entities & Players have a transform and move around. When entities get near players they should become visible. Given I have a lot of entities moving around, it seems more realistic that the entity notifies the player if it gets near enough. Therefore I would like to lookup ClientVisibility efficiently.
The required function is change_visibility(entity, client_id, state);. We can implement it the following:

Just call replicated_clients.client(client_id).visibility_mut().set_visibility(entity, state).
Call replicated_client.visibility_mut().set_visibility(entity, state) when iterating over all clients using replicated_clients.iter_mut(). Since the number of roaming entities is large it is not visible to do this without caching proximity changes during iterating the entities. We could use a HashMap<ClientId, Vec<(Entity, bool)>> which seems unnecessary difficult considering the underlying structure of ReplicatedClients.

I suggest making Clients a HashMap to an enable a broader range of use-cases. I am open do fix this myself if there is no real reason against this suggested approach.

Hi! I used a Vec since I heavily iterate over the clients inside replication system. So HashMap will affect the replication performance.
But I also see your problem. I need to think about it. Maybe you could iterate over clients first and cache the necessary data on your side?

bevy_replicon_attributes might be a good solution to your use-case. You'd probably want to do perf tests comparing it with a raw approach. And also perf test both with few and many clients.

What about players moving frequently between sector borders. In my indented approach I would make entities inside a 3x3 range visible and entities outside a 5x5 range invisible. This is possible using your approach when the player keeps ownership over all sectors in a 3x3 range (keep in mind that all these 9 sectors need to lookout for a 3x3 range). Since I consider this a hotpath it would be good to have more control over the implementation.

Hi! I used a Vec since I heavily iterate over the clients inside replication system. So HashMap will affect the replication performance.
But I also see your problem. I need to think about it. Maybe you could iterate over clients first and cache the necessary data on your side?

I think O(1) access is very useful since getting entities by id is also used inside the replicon. I would suggest one of the following approaches.

Gauge the performance benefit of using Vec over HashMap negligible. When we consider that networking & physics are included this might be true.
We can store a HashMap alongside the Vec in ReplicatedClients mapping client id -> index in Vec. We assume that connecting and disconnecting is relatively rare. Adding is trivial. Removal already has enough overhead making an update to the HashMap negligible.
We can use the indexmap crate. It provides a HashMap with Vec performance iteration.
We can keep order of clients in tact allowing to search using binary search.

I think the least intrusive and easiest to implement is probably 4. We keep the data structure in tact and just change only the methods add, remove & get. What do you think?

We can store a HashMap alongside the Vec in ReplicatedClients mapping client id -> index in Vec. We assume that connecting and disconnecting is relatively rare. Adding is trivial. Removal already has enough overhead making an update to the HashMap negligible.

I would go with this one. It's simple as 4, but more performant for lookups and doesn't require a separate crate.

@phisn could you submit a PR?

Will implement it in the next month.


Updates to Updates not showing unless you hard refresh

Publish a new Update or delete an old draft
Go back the to Collective page and scroll to the Updates section to see the results of your handiwork
The new Update does not show up
Do a hard refresh
See the new Update

It seems like a very natural path to post a new Update and then go back to your page to have a look. Can we refresh the page automatically or something?

This one is something that I've seen several times but somehow forgot to create an issue as and when I saw it. Thanks @alanna. 👍🏽
I think this is due out the Apollo cache policy we have (defaulting to cache-first) for updates. My thinking is that we should probably cache-and-network for updates since I suppose this can be quite a common flow and keeping the cache up-to date makes sense. 🤔
@Betree : I've made a PR for this, let me know if you disagree. 😄


Add docs publish CI job

Summary
This commit adds a new github workflow that will publish documentation
to https://qiskit.org/partners/ionq on new tag push events.
Details and comments

Ok, I've removed the on hold label here because I've updated the trigger to only occur on tags. The first run uploaded it for us since the release is already live: https://qiskit.org/documentation/partners/ionq/


Validation of periodic builds setup on Nordix Infrastructure for selected ETOS repos

Description
Periodic builds are currently being setup for selected ETOS projects/repositories on Nordix Infrastructure as part of Eiffel Infrastructure and CI/CD establishment work Phase 2. List of projects is

etos-api
etos-environment-provider
etos-suite-runner
etos-suite-starter

It is important the build process is reviewed by the project to ensure it is correct.
The temporary jobs for these projects are available on Nordix Jenkins. (Jobs named tmp-etos-*)
The builds for etos-test-runner-containers will be worked on once the build process of above projects/repos are reviewed.
Here are more details.

The job created for these projects on Nordix Jenkins run against the master branches of the git repositories and attempts building container images using podman
The jobs regularly poll the repo for commits and get triggered if one or more commits are found. Please note that this can be adjusted depending on the project needs and triggers can be updated such as to trigger based on new PRs.
Upon completion of the builds, the container images will be pushed to eiffel project on Nordix Container Image Registry (Please reach out to @fdegir if you need account on the registry.)

Build process.

When the jobs get triggered, project repos are cloned and the last commit on the tip of master branch is checked out
Once the clone is ready, jobs proceed with podman build using the command below
podman build --log-level error --tag registry.nordix.org/eiffel/<etos-repo-name>:latest .
Image tag is set to latest since this is a build against the tip of master branch
The complete build script is available on Nordix CICD Repository.
Upon completion of the build, the container image is pushed to eiffel project on Nordix Container Image Registry
The image is available for anonymous users and can be pulled using either one of the commands below

podman pull registry.nordix.org/eiffel/<etos-repo-name>:latest
docker pull registry.nordix.org/eiffel/<etos-repo-name>:latest

What we need from you is

Look at the build logs of the temporary jobs and let us know if you notice anything that looks strange
Review the change on Nordix Gerrit that creates the real jobs. (you need to login to Nordix Gerrit using your Google account in order to be able to review the code)
Correct the build process or the command used for the project build if they are incorrect
If possible, pull the image and verify it works

podman pull registry.nordix.org/eiffel/<etos-repo-name>:latest
docker pull registry.nordix.org/eiffel/<etos-repo-name>:latest

Provide your opinion about what type of builds your project needs - PR validation before merge, builds after merge, builds after release, periodic builds and anything else you may think if.

Please note that upcoming phases will enable testing as well and we will reach out to you when the time comes.
Motivation
Setup builds for the project
Exemplification
N/A
Benefits
Project will have automated builds available
Possible Drawbacks
N/A

Thank you.
I cannot see the tmp-etos-* jobs in Jenkins.
At the moment we want to be responsible for our own release work when it comes to production ETOS so nothing should be done on releases.
When it comes to the jobs I would like to see docker built after merge and published with the :dev tag as this is how we work today. This could be changed to "latest" if we want to @fredjn  @andjoe-axis  has to chip in on that.
As I wrote in the gerrit review: We added another dockerfile in the etos-environment-provider today.

I cannot see the tmp-etos-* jobs in Jenkins.

Sorry, my mistake.
The temporary jobs were created manually in a wrong way and they were deleted by the job updating jobs.
They are there now: https://jenkins.nordix.org/view/Eiffel/

At the moment we want to be responsible for our own release work when it comes to production ETOS so nothing should be done on releases.

Got it.]
But still, would you want to have new set of jobs, -release-ubuntu2004-master, which gets triggered when you do a release on GitHub so the jobs could build the new image, apply the tag to them based on what you used in GitHub during the release, and push them to Nordix Harbor eiffel project?
Or, do you plan to build/tag/push those release images manually?

When it comes to the jobs I would like to see docker built after merge and published with the :dev tag as this is how we work today. This could be changed to "latest" if we want to @fredjn @andjoe-axis has to chip in on that.

The tag :dev works as well. Another thing that comes to my mind is that if we could somehow align the tags used for images that are built after merges for all the projects, this could help us when we want to bring up a Kubernetes cluster and test those versions. Unfortunately I haven't received any feedback from the maintainers of other projects with regards to the builds we created.

As I wrote in the gerrit review: We added another dockerfile in the etos-environment-provider today.

Thanks for the review. I'll fix this in the change on Gerrit.

An update; the jobs for subset of ETOS repos are created https://jenkins.nordix.org/view/Eiffel/ and the images built/tagged/pushed to eiffel project on Nordix Harbor
The next change creates jobs for etos-test-runner-containers.
Once all these jobs are in place, the intention is to update their triggers to run after merge which I'll need you help to configure repos and enable webhook and give permission to Nordix Infra user.

For now, we will create everything for our production releases ourselves (including the building and pushing of images).
We probably want to change this in the future and move over step by step to nordix :)
The ":dev" tag is better than ":latest" because it's clearer that it's not considered a stable image. ":latest" should be used for the latest production/stable release.
We will start looking at the jobs in jenkins to make sure it works. Thank you so much for doing this work!

@fdegir
The test-runner-containers build requires build-args input for building. Is this possible? If so, can we add these ourselves? Can we add them as an input parameter with default values?

@t-persson yes, it is possible. The jobs currently use the common job template and scripts.
We can update the common job template and set versions of all tools as parameters which become available to build scripts as environment variables assuming setting these parameters do not cause issues for other ETOS builds.
If so, could you please pass the values for below variables so we can update the jobs?

ETR_PYTHON_VERSION
GO_VERSION
PYTHON_VERSION
RUST_VERSION

We pass the values as --build-args to the docker build process.
As I said in the other issue over at the community repository I think that we should not automate those builds. Better for us to run them manually with input parameters ourselves.
DEBIAN_RELEASE=buster
GO_VERSION=1.15.3 (latest version on golang.org)
PYTHON_VERSION=3.8.3 (not latest, but the one we've verified)
RUST_VERSION=1.47.0 (latest on https://www.rust-lang.org)
ETR_PYTHON_VERSION=3.8.3 (not latest, but the one we've verified)

thanks @t-persson. will adjust jobs and send a review for this.

This works.


Failing test

Hey,
There seems to be a failing test.
  1 failing

  1) serveIndex(root)
       should treat an ENAMETOOLONG as a 414:
     Error: expected 414 "URI Too Long", got 400 "Bad Request"
      at Test.assert (/usr/lib/nodejs/supertest/lib/test.js:205:15)
      at Server.assert (/usr/lib/nodejs/supertest/lib/test.js:132:12)
      at emitCloseNT (net.js:1629:8)
      at process._tickCallback (internal/process/next_tick.js:63:19)

Hi @utkarsh2102 sorry I haven't been watching this repo (fixed now, I subscribed for notifications -- oops!). Yes, this was an issue for Node.js 10+ due to the change in allowed HTTP header size (and thus URL size). The master branch has this resolved, though. It was just an issue with the tests, so no release was made.
Sorry for any confusion!


Invoke arrow function

Is there some way to call an arrow function from InvokeVoidAsync? something like this:
JSRuntime.InvokeVoidAsync(() => alert('testing'))();");
This could be use full to execute js code not defined in any js files. I think this could be specially use full when you create a class library with Razor components which need to invoke js code.

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

ID: aada0049-99a8-2f91-5d60-6970474e2371
Version Independent ID: c29fb1a8-028f-6efd-f1bc-e889fb362311
Content: Call JavaScript functions from .NET methods in ASP.NET Core Blazor
Content Source: aspnetcore/blazor/call-javascript-from-dotnet.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @zachneu ... The signatures require a function identifier. See the API :point_right: https://docs.microsoft.com/dotnet/api/microsoft.jsinterop.jsruntimeextensions.invokevoidasync
You can open product feedback on it with engineering/management on the engineering repo :point_right: https://github.com/dotnet/aspnetcore/issues


added in rudimentary specificity in terms of the url loader to use

Let me preface this with the fact that python is not something I have much experience in at all. That being said, this PR addresses #586 and allows for overriding the unstructuredURLloader with selenium or playwright by means of environment variables (ONLY_SELENIUM / ONLY_PLAYWRIGHT). I figured environment variables would make this easier to implement in other areas/scripts, some kind of getter pattern may be even better but this works.
added in modifications to the get_docs_with_score to help increment down slightly faster instead of -10 each time (honestly not sure if this is working, but starting at 1000 and then decrementing 10 each time is not great if your store only has say 50 docs, it's a workaround to a workaround), as well as added in rudimentary specificity in terms of the url loader to use

Thanks!  I posted changes that account for this.


Implementation of model-based RL algorithm Dreamer

Thanks for the amazing project!
Can you provide the implementation of model-based RL algorithm Dreamer from paper Dream to Control: Learning Behaviors by Latent Imagination? Implementation from ray-project/ray is too cumbersome.

Thanks for your interests. We are planning to add supports for DreamerV3 (e.g. #652). You can keep track of this issue for further updates.

Thanks!

Thanks for your interests. We are planning to add supports for DreamerV3 (e.g. #652). You can keep track of this issue for further updates.
Thanks!

Thanks for your interests. We are planning to add supports for DreamerV3 (e.g. #652). You can keep track of this issue for further updates.

Thanks!

We have implemented dreamerv3 and tested its performance in #652.


Expected database doesn't work with transactional and delete

When using @Transactional, TransactionDbUnitTestExecutionListener.class and @ExpectedDatabase, updating a row makes the expected database to always fail.
Here is a exemple :
@RunWith(SpringJUnit4ClassRunner.class)
@WebAppConfiguration
@ContextConfiguration(classes = MvcUnitTestConfiguration.class)
@DatabaseSetup("uuidtest.xml")
@DatabaseTearDown(value = "uuidtest.xml", type = DatabaseOperation.DELETE)
@TestExecutionListeners({ DependencyInjectionTestExecutionListener.class,
        DirtiesContextTestExecutionListener.class,
        TransactionDbUnitTestExecutionListener.class })
@Transactional
@TransactionConfiguration(defaultRollback = true)
public class TestDUITransactional {

@Autowired
private WebApplicationContext   wac;
@Autowired
TempUuidDaoI tempUuidDaoI;

private MockMvc mockMvc;

@Before
public void setup() {
    this.mockMvc = MockMvcBuilders.webAppContextSetup(this.wac).build();
}

@Test
@ExpectedDatabase(value = "testDelete.xml", assertionMode = DatabaseAssertionMode.NON_STRICT_UNORDERED)
public void testDeleteUuid() throws Exception {
    Integer nb = tempUuidDaoI.getAll().size();
    tempUuidDaoI.delete(tempUuidDaoI.get(1));
    assertEquals(nb-1, tempUuidDaoI.getAll().size());
}

@Test
@ExpectedDatabase(value = "testUpdate.xml", assertionMode = DatabaseAssertionMode.NON_STRICT_UNORDERED)
public void testUpdateUuid() throws Exception {
    TempUuid tu = tempUuidDaoI.get(1);
    tu.setUuid("4b2f02ca-d857-4c95-b7fa-ee6283ce0d66");
    tempUuidDaoI.update(tu);
    tu = tempUuidDaoI.get(1);
    assertEquals("4b2f02ca-d857-4c95-b7fa-ee6283ce0d66", tu.getUuid());
}

@Test
@ExpectedDatabase(value = "testInsert.xml", assertionMode = DatabaseAssertionMode.NON_STRICT_UNORDERED)
public void testInsertUuid() throws Exception {
    Integer nb = tempUuidDaoI.getAll().size();
    TempUuid tu = new TempUuid();
    tu.setUuid("4b2f02ca-d857-4c95-b7fa-ee6283ce0d67");
    tempUuidDaoI.insert(tu);
    assertEquals(nb+1, tempUuidDaoI.getAll().size());
}

}
In this code, only the update doesn't work.
The original db :
<?xml version="1.0" encoding="UTF-8"?>
<dataset>
    <PDI.TEMP_UUIDS tuu_id="1" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d64" />
    <PDI.TEMP_UUIDS tuu_id="2" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d65" />
</dataset>

testDelete.xml :
<?xml version="1.0" encoding="UTF-8"?>
<dataset>
    <PDI.TEMP_UUIDS tuu_id="2" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d65" />
</dataset>

testUpdate.xml :
<?xml version="1.0" encoding="UTF-8"?>
<dataset>
    <PDI.TEMP_UUIDS tuu_id="1" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d66" />
    <PDI.TEMP_UUIDS tuu_id="2" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d65" />
</dataset>

testInsert.xml :
<?xml version="1.0" encoding="UTF-8"?>
<dataset>
    <PDI.TEMP_UUIDS tuu_id="1" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d64" />
    <PDI.TEMP_UUIDS tuu_id="2" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d65" />
    <PDI.TEMP_UUIDS tuu_id="3" uuid="4b2f02ca-d857-4c95-b7fa-ee6283ce0d67" />
</dataset>

I've  checked that the methods to update/delete/insert work, as does the dao. I use postgresql for anything else than unit testing for the project. I use hibernate H2 for testing.
Any idea on where does this come from ?

Hi nyddogghr,
The issue that you described is very interesting. I created a sample application that shows this problem: https://github.com/ekravchenko/testingzone/tree/master/spring-dbunit-issue75
In the sample application I'm using same entity and dataset files that you provided in the example. Instead of DAO I'm using spring-data-jpa CrudRepository but that shouldn't matter cause I could reproduce the issue you described.
If I understand you right testInsert and testDelete work but testUpdate fails. The root of the problem is the @Transactional annotation which causes the tests to be wrapped in the transaction. DBUnit checks the expected dataset however data is not inserted/updated cause it only happens at the end of transaction. If this statement is correct then none of your tests should work.
The reasonable questions is 'why testInsert and testDelete work then'? They work because you have the following line in tests:
assertEquals(nb-1, tempUuidDaoI.getAll().size());

If you don't configure FlushMode then the default value of AUTO will be used. So when you fire tempUuidDaoI.getAll() at the end of the test it makes hibernate flush before transaction ends, thus making changes visible for DBUnit checks.
Re-write testUpdateUuid in the following way and you will be surprised to find it working:
@Test
@ExpectedDatabase(value = "testUpdate.xml", assertionMode = DatabaseAssertionMode.NON_STRICT_UNORDERED)
public void testUpdateUuid() throws Exception {
TempUuid tu = tempUuidDaoI.get(1);
tu.setUuid("4b2f02ca-d857-4c95-b7fa-ee6283ce0d66");
tempUuidDaoI.update(tu);
tu = tempUuidDaoI.get(1);
assertEquals("4b2f02ca-d857-4c95-b7fa-ee6283ce0d66", tu.getUuid());
tempUuidDaoI.getAll();
}

The last line will force the flush.
As a workaround I would suggest flushing manually at the end of the test. session.flush() or entityManager.flush(). I wouldn't rely on AUTO flush cause that's not intuitive.
However it would be awesome if this problem could be fixed on the tesing framework level so that users wouldn't have to worry about it.

Hi,
Thank you for your anwser.
I have retried this, with your correction, but I'm not able to fix this. I even tried flushing manually :
    @Test
	@ExpectedDatabase(value = "testUpdate.xml", assertionMode = DatabaseAssertionMode.NON_STRICT_UNORDERED)
	public void testUpdateUuid() throws Exception {
		TempUuid tu = tempUuidDaoI.get(1);
		tu.setUuid("4b2f02ca-d857-4c95-b7fa-ee6283ce0d66");
		tempUuidDaoI.update(tu);
		tu = tempUuidDaoI.get(1);
		assertEquals("4b2f02ca-d857-4c95-b7fa-ee6283ce0d66", tu.getUuid());
		tempUuidDaoI.getAll();
		tempUuidDaoI.flush();
	}

But I still have the same error.
What tempUuidDaoI.flush() does is simply getSession.flush(). So it should have fixed it. I remember I had also tried this with no result.
So far, I have no anwser, I had to use another way to do what I wanted to do (use jdbc template to avoid the @Transactionnal annotation).
Thanks again even if I can't confirm this is fixed

HI nyddogghr,
Could you please send a link to your sample application in Git? That would really help in sorting this issue out.
Thanks,
YK

Hi,
Sorry but fro some reason I'm not able to create a sample application for this. I've tried your sample that indeed works, but I have used another way to get past this.
Once again, thank you for your help.
I'm closing this since it seems it comes from my project only.

I have the same trouble :( Without flush it does not work

Hi,
with the help of flush I was able to make an integration test work on service-level, e.g., see: service integration test
However, it does not work on Mvc-level, when testing the controller, e.g., see: controller integration test
Do you have any idea what is the issue with the MockMvc? Can you advice?
Thanks in advance!


Modify ermrest/hatrac ACL to support the new webauthn

ERMrest

[ ] Adjust the ACL storage schema for new catalog deployments as needed
[x] Adjust internal code that intersects ACLs with webauthn.context.attributes
[x] Adjust internal code that exposes webauthn.context as PostgreSQL session parameters for row-level security scenarios
[ ] Update row-level-security documentation to reflect examples using new session parameters and recommended storage representation for security identifiers/attributes in application schemas There are no changes for application schemas.

Hatrac

[ ] Adjust the ACL storage schema
[x] Adjust internal code that  intersects ACLs with webauthn.context.attributes
[x] Adjust internal code that injects webauthn.context.client into new ownership ACLs

Rather than branching, I patched ermrest/master and hatrac/master to have compatibility with both webauthn/master and webauthn/globus_auth by detecting whether the webauthn API is returning bare strings or dictionaries for the client context values.
The ACL storage needs a little more thought.  I'd like to come up with a similar backwards and forwards compatible code patch to ease transition of existing deloyments.

In ermrest, ACLs are stored in the catalog "meta" table which is a key-value store with text type values (with a full ACL spread across multiple rows). If we changed the value type to jsonb I could branch on that. But, this would also affect other future use of metadata...
In hatrac, ACLs are stored in dedicated columns of type text[] containing a full ACL. If we change the ACL type to jsonb[] I could branch on that.

After further discussion with @ljpearlman we decided not to change ACL storage formats. This opens a lot more questions than it answers and there is no reason to pursue this right now.

Reassigning to @ljpearlman for review.  If she wants to deploy latest master hatrac and ermrest with any permutations of security that she is testing, they should all work.

We're already running this on several servers so I am going to call it done and reviewed.


Type converters

Didn't find possibility to use type converters.
As an example:
in a DB I have a column with varbinary(char(1)) type. And this should be converted into boolean in entity.
In Telerik Open Access I have possibility to use something like the below:
HasProperty(x => x.SomeProperty).WithConverter<BitTypeToBooleanConverter>();

Do we have something like that in EF7?

This isn't supported yet... but we have https://github.com/aspnet/EntityFramework/issues/242 tracking this scenario (which is part of a bigger area of work tracked by https://github.com/aspnet/EntityFramework/issues/240)


Vec<T> can't be used for #[ruma_api(query)] fields

The client-server spec has at least one endpoint where a query parameter can be used multiple times. This is not denoted specifically in the query string, however, as is frequently done elsewhere.
# Common notation
?param[]=value1&param[]=value2
# Matrix notation
?param=value1&param=value2

serde_urlencoded, which we are currently using, does not support this. serde_qs seems to only support the common notation. We might have to roll our own (the new ruma-serde crate would probably be a good place for it).

Superseded by #49.


ModuleNotFoundError: No module named 'auto_editor'

Bug description
I got "zsh: command not found: auto-editor" on my mac and then I followed "manually adding the path" (I got "No such file or directory" error when try to run ls /usr/local/lib/python3.12/site-packages/ | grep auto )
Then I got this error when trying to run auto-editor
Traceback (most recent call last):
File "/usr/local/bin/auto-editor", line 5, in
from auto_editor.main import main
ModuleNotFoundError: No module named 'auto_editor'
What's your platform?
Traceback (most recent call last):
  File "/usr/local/bin/auto-editor", line 5, in <module>
    from auto_editor.__main__ import main
ModuleNotFoundError: No module named 'auto_editor'

What command did you use
auto-editor

What properties does your input video have?
auto-editor video_name.mp4 --export final-cut-pro
Comments
No response

auto-editor with Python3.12 is not ready yet. Trying using 3.11

Nice, It worked. thank you!
By the way, when I use "--export final-cut-pro" and try to import the XML file to Final Cut Pro. I got error
'The XML document sent from application “Final Cut Pro” could not be imported. DTD version is not recognized.'
But it work well when I just try to export as a new video.

I'm still having this issue even in python 3.10.14


"coverage html" fails on systems with FIPS enabled

Is your feature request related to a problem? Please describe.
My development system recently got an "upgrade" which enabled FIPS and has thus blocked all access to creating MD5 hashes. Because of this, coverage html does not working anymore. Instead I get a stacktrace that looks like this (I can't copy and paste the exact error because it's on a lab machine that is not connected to the internet):
  File: "....../venv/lib64/python3.6/site-packages/coverage/html.py" line 461, in check_global_data
    m = Hasher()
  File: "....../venv/lib64/python3.6/site-packages/coverage/misc.py" line 203, in __init__
    self.md5 = hashlib.md5()
ValueError: unknown reasons

Describe the solution you'd like
It would be nice if this class used sha256 instead of md5 for the hashing algorithm.
Describe alternatives you've considered
We have not explored any other alternatives other than manually modifying the code to work.
Additional context
The workstation I'm using is configured as follows:
CentOS 7.9
Python 3.6
coverage==5.5.0
pytest==6.2.4
pytest-cov==2.12.1
I will create a PR for this as the changes seemed pretty straightforward and non-impacting.

Fixed in 367f7c45.

This is now released as part of coverage 6.0b1.

This is now released as part of coverage 6.0.


ntr: long polar microtubules at interphase

a child of
long polar microtubules
(note this is different from
long mitotic spindle microtubules protruding beyond spindle pole body, as it occurs during interphase, not as a consequence of spindle elongation...it appears to be the source of the cytoplasmic microtubules in this mutant)
"Many MT bundles projected outside the body of the nucleus (Figure 4E). Consistent with this, we observed that the nu- clear envelope formed thin protrusions that grew and shrank (Figure 4, G and H, Supplementary Movie 8). Such intranuclear MTs and nuclear envelope distortions were not observed in wild-type cells. Several criteria confirmed that these cells with intranuclear MTs were in interphase and not in mitosis: these cells did not exhibit cytokinetic actin rings or separated SPBs and were often shorter than mitotic cells (Figure 4, B and H; our unpublished observations)."
PMID:15772152

"Thus, interphase intranu- clear MTs that pierce the nuclear envelope contribute to the formation of a significant fraction of the cytoplasmic MTs, but there are probably additional sources as well."

Polar microtubules are part of a spindle, by definition (in GO CC and generally). Is it part of this phenotype that the cells have spindles in interphase, or are these MTs that connect to an SPB despite not being part of a spindle?
The way they did the experiment I suspect the latter, but shout if I missed something.

OK I think it is the latter
so would it be OK to just call them
long nuclear microtubules protruding through nuclear envelope at interphase?

yep, I think that would be basically OK

long nuclear microtubules protruding through nuclear envelope during interphase FYPO:0004724
nuclear envelope protrusion present during interphase FYPO:0004725

edit file: e78131ae9f80d87fc081f0db57734c7aa001fec8
release: 812c9c7c3fd5bdfc78999933ff1a79bc9634fc7c


Feature Request: Built-in API provider for custom IconifyJSON

Motivation
As is, iconify is great for free, public icon sets. For commercial purposes, you may want to use commercially licensed icon pack - the most popular example being Font Awesome Pro. Iconify already provides a way of converting custom icon packs into IconifyJSON definition. However, Including the whole IconifyJSON in the app is costly, it can easily weigh over 100 kB (raw). Therefore, iconify by default uses API provider to deliver only the requested icon definitions on demand. It is currently not possible to easily host your own icon packs definitions.
Feature Request
Since nuxt already hosts server routes, it would be great if the module could provide a streamlined method of hosting Iconify API provider. This way, the end-user could easily use their non-free icon packs benefiting from the on-demand architecture of iconifi's loadIcon.
Example module configuration could look like this:
defineAppConfig({
  nuxtIcon: {
    customCollections: [
      resolve('./assets/icons/collections/my-icons.json')
    ]
  }
})

To achieve this, the module would have to expose http API conforming to the Iconify API standard. The API is very straight forward
Example definition:
{
  "prefix": "my-icons",
  "lastModified":<PHONE_NUMBER>,
  "icons": {
    "a": { "body": "<path d=\"_path_a_\"/>" },
    "b": { "body": "<path d=\"_path_b_\"/>" },
    "c": { "body": "<path d=\"_path_c_\"/>" }
  },
  "width": 24,
  "height": 24,
}

Example API response to GET /api/iconify-provider/my-icons.json?icons=a,c
{
  "prefix": "my-icons",
  "icons": {
    "a": { "body": "<path d=\"_path_a_\"/>" },
    "c": { "body": "<path d=\"_path_c_\"/>" }
  },
  "width": 24,
  "height": 24,
}

Module would have to register the custom provider with iconify library:
import { addAPIProvider } from '@iconify/vue'
addAPIProvider('nuxt', { resources: ['/api/iconify-provider'] })

And the icons should be usable as per example:
<icon name="@nuxt:my-icons:a"/>
<icon name="@nuxt:my-icons:c"/>

Any chance we could get this soon with the new version now done?

I believe you could do that already:
https://github.com/nuxt/icon/blob/cb44eede9993ed287a41c47b63d790d17a0d37a7/src/types.ts#L66-L70

@antfu any chance you could provide a sample showing how to load icons from a custom json file?

hi @antfu pls reopen because this not implemented
https://iconify.design/docs/libraries/tools/import/json.html
https://github.com/nuxt/icon/blob/548c3bb105448747f1eaac5653c85a40ec3cd51f/src/collections.ts#L33

@reslear
It works. It consumes awful lot of memory during build, but works. I never got to debugging the memory issue properly, but here is the demo showcasing importing collection as IconifyJSON: https://stackblitz.com/edit/github-rpgvsa?file=nuxt.config.ts. If the possible memory issue rings a bell, I'd appriciate 15 minutes of thought - maybe it is trivial to fix. Otherwise I'll maybe make proper reproduction someday 😉

@Aareksio I don't want to generate a ts file


dns timout

Would you have any recommendations for those getting the following response from the DNS queries:
$ kubectl exec -ti $POD_NAME -- nslookup kubernetes
;; connection timed out; no servers could be reached
command terminated with exit code 1

Replace kubedns with coredns

On Sep 3, 2018, at 1:32 PM, Giovanni Salvador<EMAIL_ADDRESS>wrote:
Would you have any recommendations for those getting the following response from the DNS queries:
$ kubectl exec -ti $POD_NAME -- nslookup kubernetes
;; connection timed out; no servers could be reached
command terminated with exit code 1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@gio-salvador
Have you tried the solution proposed here.
This issue may be a duplicate

@omarrayward is right, it´s just the busybox image is crap - see this fix https://github.com/kelseyhightower/kubernetes-the-hard-way/pull/384

KubeDNS has been replaced with CoreDNS and all examples have been tested with the new configuration and verified to be working with the latest release of this guide.


Mitosis to generate components at the correct path

Currently, when generating components with Mitosis outside of the will place the file in an undesired path:
{
    dest: "../react/src/components"
}

will place the components in packages/react/src/components/react/src.
Research if there is a way around this.

Fixed by using getTargetPath


fix: pin eyeglass to 2.2.1

Closes #197
Description
Pegs our Eyeglass dependency to 2.2.1 because subsequent versions have a bug that breaks the build. (I have raised an issue with them, and once it's fixed, we should un-peg and upgrade this to the latest version.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


update gh run list columns

Fixes #22 and adds a note about the required gh version

sorry it is not working for me you get wrong id after that i use gh 1.12.1
I think we don't need to get value from age or elapsed  because we don't display it on table

I dropped the new fields, I'm also on 1.12.1, so not sure what issue you're seeing.
diff --git a/lua/telescope/_extensions/gh_make_entry.lua b/lua/telescope/_extensions/gh_make_entry.lua
index be67b87..aba1adb 100644
--- a/lua/telescope/_extensions/gh_make_entry.lua
+++ b/lua/telescope/_extensions/gh_make_entry.lua
@@ -50,6 +50,7 @@ function gh_make_entry.gen_from_run(opts)
   return function (entry)
     if entry == '' then return nil end
     local tmp_table = vim.split(entry,"\t");
+    print(vim.inspect(tmp_table))
     local active = tmp_table[1] or ""
     local status = tmp_table[2] or ""
     local description = tmp_table[3] or ""

If you try that patch, which index are you seeing the id field in?

I am using gh version 1.12.1 (2021-07-01), and for me local id = tmp_table[7] or "" is working fine.
BTW, we can do something like this to support both versions:
    local id = ""
    if #tmp_table > 8 then
        id = tmp_table[7]
    else
        id = tmp_table[#tmp_table]
    end

An elegant solution @sbulav, much better than what I was thinking for parsing version numbers. I've pushed that fix.


HIVE-24945: Support vectorization for lead/lag functions

What changes were proposed in this pull request?
Implement vectorized evaluators for lead and lag functions.
Why are the changes needed?
For better vectorization coverage (performance).
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Performance tested on cluster, added new qtests + old qtests got vectorized without query result change.

@ramesh0201: is there a chance that you can review this too? (much simpler than HIVE-24761 :) )

@abstractdog  Looks great except the two comments.

Also probably need to rebase to latest branch and update the q.out file as it doesn't seem to contain the deleted rows query patch

@ramesh0201 : rebased and fixed patch passed precommit testing, PR changes are in https://github.com/apache/hive/pull/2278/commits/81592ac9909d1e2875dfdcb2e2e86f1f597062b3

+1. LGTM.


Vscode safe write from which?

Otherwise, in the real-time compiler will cause the compiler is not successful.
https://webpack.js.org/guides/development/#adjusting-your-text-editor

We do not have atomic saving in VS Code. File writes go directly into the file.

@bpasero
Vscode save the file there is a certain problem, there are many developers have the problem. see:
https://github.com/sass/node-sass/issues/1894#issuecomment-309023744

@sbfkcel we use fs.writeFile(), if there is an issue with that I suggest to report this to node.js directly.

@bpasero The problem should be above the node fs module.
Think you~~ 👍

I was facing the same problem too. It all became fine once I changed my tool from node-sass-chokidar to node-sass. 😕

I do not understand.


Landing page shows interview prep as a cert

On the home page, it lists the coding interview prep section as a certification....

Maybe remove that since there's no cert for it?

Yes, you are correct. I will defer to @QuincyLarson for the correct categorization for this.

We could change the font size of this. Then it would be visually distinct from the other items in this list.

We can indent the list belonging to same category. And make change in font style
Like this way:-
Certifications:
       Responsive Web Design
      JavaScript Algorithms and Data Structures
      Front End Libraries
      Data Visualization
      APIs and Microservices
      Information Security and Quality Assurance
Coding Interview Prep

I'm not sure I like either of these solutions - it seems like it might just make a user ask themselves something like: "what's with that last one?" - And not be quite clear enough that it is not a cert. What about something a little more specific...
Certifications:
   Responsive Web Design
  JavaScript Algorithms and Data Structures
  Front End Libraries
  Data Visualization
  APIs and Microservices
  Information Security and Quality Assurance
Additional Challenges:
  Coding Interview Prep


Fix crash for when a PR is closed and branch deleted but we pull anyway

Coverage decreased (-0.05%) to 55.813%
ha, ok, I'm updating the config to not fail the build


[tool_crash] FormatException

Command
flutter build ipa --export-method ad-hoc --verbose

Steps to Reproduce

...
...
...

Logs
FormatException
#0      _Utf8Decoder.convertChunked (dart:convert-patch/convert_patch.dart:1859:7)
#1      _Utf8ConversionSink.addSlice (dart:convert/string_conversion.dart:314:28)
#2      _Utf8ConversionSink.add (dart:convert/string_conversion.dart:310:5)
#3      _ConverterStreamEventSink.add (dart:convert/chunked_conversion.dart:72:18)
#4      _SinkTransformerStreamSubscription._handleData (dart:async/stream_transformers.dart:1

Closing this as a duplicate of https://github.com/flutter/flutter/issues/81666.
Kindly follow up in the linked issue for updates. Please upvote the issue as well.
Thank you


[backport] Avoid non-differential point in TestTriplet

Backport of #8376

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 7a800703e351e55b8954d8bedc1d1d8ba55e9980, target branch v6) succeeded!

Jenkins, test this please.

Jenkins CI test (for commit 7a800703e351e55b8954d8bedc1d1d8ba55e9980, target branch v6) succeeded!

Jenkins, test this please.

Jenkins CI test (for commit 7a800703e351e55b8954d8bedc1d1d8ba55e9980, target branch v6) succeeded!

LGTM.


Update [Blockchain.com blockchain.com] #custodian #wallet #exchange #memo-required GARAR5…GZYSLN

Address: GARAR5QR7WRL24MQMSO4INWV7C5SE4EE2YVXTLD6ORONYFHSUAGZYSLN
Tags: custodian wallet exchange memo-required
Title: Blockchain.com  Domain: blockchain.com
Requested by @btfanwn746

@btfanwn746
I'm not sure what you are reporting. This account has already been labeled/tagged. If you are having trouble seeing tags on stellar.expert you may have to refresh your screen and/or reduce the number of stellar.expert tabs you have open.  You might also try a different browser and/or restart your machine.
Looking to the future with you,
Rob

Hello @btfanwn746
You can reopen this issue with new information that we should be aware of. It has been over a week with no reply
Looking towards the future with you,
Rob