Is that normal we can't access to the model when using repeating section?

Question
Is that normal we can't access to the model when using repeating section?
Maybe I'm using wrong the repeating section but in v5 it worked well.
https://stackblitz.com/edit/angular-vdxzfi?file=src/app/app.component.ts
(check the console. model is null)

Hi @kenisteward thank you for your response.
For the first point I used the tutorial and I haven't see that the model was initialized, sorry.
For the second point, effectively when I add defaultValue to the field the hide function works now for the second field.
But I tried to use the other method to avoid adding defaultValue to every field. And it's doesn't seem to work. The model is still undefined.
FormlyModule.forRoot({
  types: [
    {
      extends: 'formly-group',
      defaultOptions: {
        defaultValue: {}
      }
    }
  ],
})

Stackblitz updated: https://stackblitz.com/edit/angular-vdxzfi?file=src/app/app.component.ts

It can be done as follow:
        {
          name: 'repeat',
          component: RepeatTypeComponent,
          defaultOptions: {
             fieldArray: {
               defaultValue: {}
             },
          }
        },

Have a look for the three different ways to set the defaultValue for array-types at https://github.com/ngx-formly/ngx-formly/issues/3345#issuecomment-1165644688

Thank you @kenisteward and @FritzHerbers for your support and examples. It works now :)

Hi @kenisteward and @FritzHerbers, sorry to bother you again with my issue.
I have another form where I use my repeat component but now I want to hide all the field group depending on another field for example.
When I add the hide expression, the default value is not take in count and the model stay null.
Stackblitz updated: https://stackblitz.com/edit/angular-vdxzfi?file=src/app/app.component.ts

Can you check the following solution instead of default value:
hide: (field) => {
- return !field.model.task1;
+ return !field.model?.task1;
}

Thank you very much @aitboudad it works perfectly :smiley:


Add .standalone class to  when in standalone mode

Closes #6810
Add a standalone class to the html <body> element when in standalone mode, so that one can add custom dashboard CSS that only applies in standalone mode.

Codecov Report

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

@@            Coverage Diff             @@
##           master   #6811       +/-   ##
==========================================
+ Coverage   56.03%   73.4%   +17.37%
==========================================
  Files         527      75      -452
  Lines       23286   10105    -13181
  Branches     2788       0     -2788
==========================================
- Hits        13049    7418     -5631
+ Misses       9827    2687     -7140
+ Partials      410       0      -410

Impacted Files
Coverage Δ

superset/assets/src/components/Checkbox.jsx

...ations/deckgl/layers/Polygon/PolygonChartPlugin.js

...ets/src/dashboard/components/dnd/DragDroppable.jsx

...c/visualizations/deckgl/layers/Polygon/Polygon.jsx

...ssets/src/visualizations/presets/MapChartPreset.js

superset/assets/src/components/EditableTitle.jsx

...assets/src/visualizations/Iframe/transformProps.js

superset/assets/src/setup/setupPlugins.js

...t/assets/src/components/InfoTooltipWithTrigger.jsx

.../src/dashboard/components/UndoRedoKeylisteners.jsx

... and 442 more

Continue to review full report at Codecov.

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

@raphaelmerx , Could you please show an example of css for standalone chart.
When I use standalone=True I lose the label colors  , do you know how to fix this?

@raphaelmerx , could you please show an example of .standalone use in CSS.
Whenever I "explore" a chart from a dashboard I loos all CSS properties defined in the dashboard.
For instance, I am trying to increase the font size of dist_bar plot.
It works well in the Dashboard context but when I click on explore chart (standalone or not), the normal font size comes back. Do you have any suggestion for this? Thank you

@lilila I have a superset URL for a standalone dashbaord that looks like https://example.com/superset/dashboard/10/?standalone=true. This URL is used in an iframe that is present on my site.
Then when i open that dashbaord URL (without the ?standalone=true) in a separate tab, and go to Edit dashboard > Edit CSS, some of my CSS only targets the dashboard in standalone mode e.g.
.standalone .dashboard-component-chart-holder {
    border: 1px solid;
}

@raphaelmerx Thank you for your answer, so still is is a style sheet applied to the entire dashboard. I wish I find a way to keep the dashboard's style on a single chart when I go to the Explore Chart view. Basically I display iframed charts inside presentation (made with reveal.js) and I wish they look the same in the dashboard and in the presentation.
If you have any tips for this I would be glad.


Starscream's version is too low

The version used by Starscream is 3.1.2 and the current version 4.0.4 is very different, causing the upgrade to encounter major obstacles，please change the WebSocketConnecting protocol，Make the upgrade go smoothly。thanks

@lvzhehappy did you faced any problems with implementing WebSocketConnecting for Starscream 4.0.4?

@flypaper0 yeah, Starscream 4.0.4 has changed more, didn't implementing WebSocketConnecting.We have re-updated the websocket in our projects,so that can compatible with WebSocketConnecting

Good to hear! We are recommend to use Starscream v3, because v4 had some issues when we tested it. But I believe it's also possible to implement WebSocketConnecting for v4 if you really need it

@lvzhehappy Would you mind sharing what you did to support compatibility with Starscream 4? I'm a bit stuck at the moment.
@flypaper0 Do you have any guidance as well? It looks like Starscream 4 has been out for two years now, with other projects like web3swift supporting it. Would love to see v4 or alternative support!

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

customize starscream like this:
in Websocket file
// MARK: - EngineDelegate
public func didReceive(event: WebSocketEvent) {
callbackQueue.async { [weak self] in
guard let s = self else { return }
s.delegate?.didReceive(event: event, client: s)
s.onEvent?(event)
        //custom add ======================================
        switch event {
        case .connected:
            self?.mutex2.lock()
            self?.connected = true
            self?.mutex2.unlock()
            self?.onConnect?()
        case .disconnected:
            self?.mutex2.lock()
            self?.connected = false
            self?.mutex2.unlock()
            let error = WSError(type: .serverError, message: "Invalid HTTP server", code: 10000)
            self?.onDisconnect?(error)
        case .error(let error):
            self?.mutex2.lock()
            self?.connected = false
            self?.mutex2.unlock()
            self?.onDisconnect?(error)
        case .cancelled:
            self?.mutex2.lock()
            self?.connected = false
            self?.mutex2.unlock()
            self?.onDisconnect?(nil)
        case .text(let string):
            self?.onText?(string)
        case .ping:
            break
        case .pong:
            break
        case .reconnectSuggested:
            break
        case .binary, .viabilityChanged:
            break
        }
        //===================================================
    }
}

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。


[stable/kured] Upgrade to latest version 1.3.0

New version of Kured has been released. This PR updates the corresponding chart.

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @jlegrone

/ok-to-test

/lgtm


Eliminate empty destructors

Empty destructors (e.g. https://github.com/zhouguiyun-uestc/ParallelFlowDirections/blob/master/src/common/grid.cpp#L11-L14) can be replaced with ClassName::~ClassName() = default in the header, or eliminated entirely.

https://github.com/zhouguiyun-uestc/ParallelFlowDirections/blob/master/src/common/tile_info.cpp#L9-L11

And constructors: https://github.com/zhouguiyun-uestc/ParallelFlowDirections/blob/master/src/flowdir/GridCell.cpp#L5-L7

https://github.com/zhouguiyun-uestc/ParallelFlowDirections/blob/master/src/flowdir/GridCell.cpp#L14-L16

https://github.com/zhouguiyun-uestc/ParallelFlowDirections/blob/master/src/flowdir/consumer_2_producer.cpp#L2-L8

Done!

Thanks!


missing JPMS module name, should set java.xml.bind

The jaxb-api/src/main/java/module-info.java file is not used, which means we are missing the JPMS information in the generated MANIFEST.MF.
We could instead copy the following line from [https://github.com/jboss/jboss-jaxb-api_spec/blob/master/pom.xml#L83] I think into jaxb-api/pom.xml, then build and check jaxb-api/target/classes/META-INF/MANIFEST.MF for module name.

Should the Automatic-Module-Name be jakarta.xml.bind ?

Please use java.xml.bind.  The rule that we have been following is to keep the EAP 7.2 (Automatic-Module-Name) which is java.xml.bind.

Fixed


YAML multiple setting

Rust Version
rustc 1.23.0 (766bd11c8 2018-01-01)
Affected Version of clap
andre@ANDREW-DESKTOP:/mnt/c/Users/andre/Projects/cargo-mpirun$ grep clap Cargo.lock
 "clap 2.29.4 (registry+https://github.com/rust-lang/crates.io-index)",
name = "clap"
"checksum clap 2.29.4 (registry+https://github.com/rust-lang/crates.io-index)" = "7b8f59bcebcfe4269b09f71dab0da15b355c75916a8f975d3876ce81561893ee"

Expected Behavior Summary
Setting multiple settings in cli.yml via setting results in each setting being used.
Note: settings works as expected
Actual Behavior Summary
Setting multiple settings in cli.yml via setting results in only the last specified setting being used.
Steps to Reproduce the issue
cli.yml
name: example
version: "0.1"
author: Foo B<EMAIL_ADDRESS>setting: UnifiedHelpMessage
setting: DeriveDisplayOrder
args:
    - z-arg:
        long: z-arg
        value_name: Z_VALUE
    - a-arg:
        long: a-arg
        value_name: A_VALUE
    - z-flag:
        long: z-flag
    - a-flag:
        long: a-flag

main.rs
#[macro_use]
extern crate clap;
use clap::App;

fn main() {
    let cli = load_yaml!("cli.yml");
    let _matches = App::from_yaml(cli).get_matches();
}

Expected help output:
GT C:\Users\andre\Projects\clap-example [master +4 ~0 -0 !]> cargo run -- --help
   Compiling clap-example v0.1.0 (file:///C:/Users/andre/Projects/clap-example)
    Finished dev [unoptimized + debuginfo] target(s) in 1.34 secs
     Running `target\debug\clap-example.exe --help`
example 0.1
Foo B<EMAIL_ADDRESS>
USAGE:
    clap-example.exe [OPTIONS]

OPTIONS:
        --z-arg <Z_VALUE>
        --a-arg <A_VALUE>
        --z-flag
        --a-flag
    -h, --help               Prints help information
    -V, --version            Prints version information

Actual help output:
GT C:\Users\andre\Projects\clap-example [master +4 ~0 -0 !]> cargo run -- --help   Compiling clap-example v0.1.0 (file:///C:/Users/andre/Projects/clap-example)
    Finished dev [unoptimized + debuginfo] target(s) in 1.33 secs
     Running `target\debug\clap-example.exe --help`
example 0.1
Foo B<EMAIL_ADDRESS>
USAGE:
    clap-example.exe [FLAGS] [OPTIONS]

FLAGS:
        --z-flag
        --a-flag
    -h, --help       Prints help information
    -V, --version    Prints version information

OPTIONS:
        --z-arg <Z_VALUE>
        --a-arg <A_VALUE>

Ah yes, you're correct. The fix is to use settings (plural), but I know the docs aren't clear on that.
settings:
    - DeriveDisplayOrder
    - UnifiedHelpMessage

Although this could be fixed, the fix will be made moot once we move to serde in v3.
I'm going to close this for now, but using settings doesn't work for you please feel free to re-open and we can look into this further.


add epsilon() function

It seems to have been lost altogether at some point.  Perhaps for naming?
As Aki points out, we should fix definition of epsilon() to read
Machine epsilon (the difference between 1 and the least value greater than 1 that is representable).

Are there other constants like this from std::numeric_limits<double> that we should add?

We have machine_precision() now.


Feature Request: Record Timer

Not sure how much work this would take but would it be possible to have a record time? Like if I can tell livestreamer to stop saving to file after 1 hours. There is already a timer for how long the video has been recording for, it doesn't seem like it should be that hard to have it kill the program after x time.
-T 60 (min) or -T 3600 (sec)
I tried looking for a batch script option for this but there doesn't seem to be a very efficient command. I guess I could mess with cygwin but having an option built in would be more convenient.

There's a posting in the google forum for livestreamer with a 'patch' that provides a timer.
See https://groups.google.com/forum/#!topic/livestreamer/BMlxr9wQCkc
It would be very useful to have this capability within the core/main release as other forks are behind the main code base.
Not suggesting necessarily that the patch is appropriate to just go into the main code base as is, but might be a useful starting point.
Appreciate all your hard work. Keep it up :)

Yes, it would be great to have an option to allow Livestreamer to record/capture a file for X time. The fork that was created looks like a simple solution.

I tried looking for a batch script option for this but there doesn't seem to be a very efficient command.

Batch file:
start livestreamer.exe<EMAIL_ADDRESS>best -o 01.ts
timeout /t 1800
taskkill /im livestreamer.exe /f

I'm on Win 7 and currently use the Windows Task Scheduler to start the batch job and also to end it after x minutes/hours. It works well for me, although as I say having an internal Livestreamer record timer would still be useful.


Identify menu item with text

The closed menu only shows the icon of an item, the idea is to show a text to help identify the item. Just like we have in the "back" link on top.
example:

The issue was raised by Fernanda Passos from UX team.

One option to resolve, it would be adding the title attribute on the link.


[Bug]: Dependencies missing from py_venv

What happened?
When creating a venv with the py_venv rule, none of the libraries installed with pip are exported.
Version
Development (host) and target OS/architectures: Ubuntu x86_64
Output of bazel --version: bazel 7.0.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file: aspect_rules_py = 0.5.0 or HEAD, rules_python = 0.29.0
Language(s) and/or frameworks involved:
Python
How to reproduce
Minimal example:
MODULE.bazel:
bazel_dep(name = "aspect_rules_py", version = "0.5.0")

bazel_dep(name = "rules_python", version = "0.29.0")

python = use_extension("@rules_python//python/extensions:python.bzl", "python")

python.toolchain(
    python_version = "3.10",
    is_default = True,
)

use_repo(python, "python_versions")

register_toolchains("@python_versions//:all")

pip = use_extension("@rules_python//python/extensions:pip.bzl", "pip")

pip.parse(
    hub_name = "pip",
    python_version = "3.10",
    requirements_lock = "//:requirements_lock.txt",
)

use_repo(pip, "pip")

requirements_lock.txt:
pyserial==3.5 \
    --hash=sha256:3c77e014170dfffbd816e6ffc205e9842efb10be9f58ec16d3e8675b4925cddb \
    --hash=sha256:c4451db6ba391ca6ca299fb3ec7bae67a5c55dde170964c7a14ceefec02f2cf0
    # via -r requirements.txt

BUILD:
load("@aspect_rules_py//py:defs.bzl", "py_venv")
load("@pip//:requirements.bzl", "all_requirements")

py_venv(
    name = "venv",
    deps = all_requirements,
)

Commands:

bazel run //:venv
.venv/bin/python

import serial (or inspect .venv and see that pyserial is missing)

Any other information?
No response

This is the same as #248


[WIP] Adds support for using custom ways to load the vulkan dynamic libraries.

These changes are backwards compatible with older versions.
Signed-off-by: Hal Gentz<EMAIL_ADDRESS>

This change is

@MaikKlein 0.1.9 is out now, should be good for review/merging.

Everything else looks good, thanks for the PR. Could you also quickly explain the use case for the portability lib? I assume you are going to create a custom entry?

@MaikKlein One example is using RTLD_NEXT or RTLD_DEFAULT: https://github.com/gfx-rs/gfx/pull/2221/files#diff-44198a1b71ecb41e695db8b6967cd689R90
For why someone might want to do this, I refer you to this wonderful gem I found: https://github.com/amonakov/on-wrapping/blob/master/interposers-discussion.asciidoc

@MaikKlein Can you release 0.24.4 on crates.io?


Fix git email address parking possibility

Fix users being able to block a git_email by adding it and never confirming it, thus disallowing any other users to add it.
Now only allows adding if the user hasn't already added it or it's not confirmed yet.
Initially had a alter table drop constraint for the unique, but SQLite doesn't know constraint and only drop index.
Then SQLite automatically names the indexes (not very pretty), and the index to remove was sqlite_autoindex_git_email_2.
I thought, oh hey, not very nice, but oh well, add another DROP index if exists sqlite_autoindex_git_email_2, but that results in: index associated with UNIQUE or PRIMARY KEY constraint cannot be dropped
tl;dr: need to drop the unique index and have to delete the DB's, and start up again because of SQLite and this change being needed

As a user should theoretically be allowed to send another confirmation in case he hasn't received one. I implemented the remove git-email fn also to remove the entry in the user_emails_sent table. This seems to be the simplest solution to go around this behaviour of only allowing one email per type to get sent to a user.
If you have a different idea/approach, please let me know. But this should be fine at least for the go-live :)

one has to locally execute the command: alter table git_email drop constraint git_email_email_key; in the DB shell to remove the constraint, or delete and setup the DB new


JavaScript detail modal

In this project:

I have added blur background to the modal
I have added a detail modal
I have made detail modal dynamic
I have made detail modal responsive

Added all recommended Optional suggestion


Videochat 2 IT dataset size

I know that's a long shot, but has anyone downloaded the whole dataset and can tell me how much GB/TB I can expect it to be? Thank you

kinetics 400 436G / 600  727G / 700 867G
VideoChatGPT ~150G
M3IT 29.8 G
NExTQA  23G
CLEVERER 11.5G
YouCook2(only videochat2 processed) 60G
TextVR 5.4G
TGIF  11.8M
EgoQA(only videochat2 processed)  3.68G
WebVid(non-official) 20TB
[VideoChat-Conversation-video] (https://huggingface.co/datasets/ynhe/videochat2_data/blob/main/videochat2_conversation_videos.zip) 17.2G

Thank you so much ☺


Should exception be marked as observed?

Hi Todd,
It looks like the code does the opposite of what is recommended in the comments. Should I remove that call to SetOberved.
 /// <summary>
        /// Event Handler delegate to log if an unobserved task exception occurs.
        /// </summary>
        /// <param name="sender">the sender</param>
        /// <param name="e">the exception details</param>
        /// <remarks>
        /// We intentionally do not mark the exception as Observed, which would prevent the process from being terminated.
        /// We want the unobserved exception to take out the process. Note, as of .NET 4.5 this relies on the ThrowUnobservedTaskExceptions
        /// runtime configuration in the host App.Config settings.
        /// </remarks>
        private static void TaskScheduler_UnobservedTaskException(object sender, UnobservedTaskExceptionEventArgs e)
        {
            ServiceEventSource.Current.UnobservedTaskException(e.Exception?.GetType().Name, e.Exception?.Message,  e.Exception?.StackTrace);

            AggregateException flattened = e.Exception?.Flatten();
            foreach (Exception ex in flattened?.InnerExceptions)
            {
                ServiceEventSource.Current.UnobservedTaskException(ex.GetType().Name, ex.Message, ex.StackTrace);
            }

            // Marking as observed to prevent process exit.
            e.SetObserved();
        }

Submit a PR with the fix

Ok great. See https://github.com/toddabel/PQService/pull/6.


feat(core): pass session summary to plugin fns

This PR adds metadata to all callbacks in plugins so that a user can identify which session each callback is being made for.

I'm going to go ahead and merge, although it does seem like a docs update is needed.


Any plans on supporting yolov5 instance segmentation ?

As stated in the title,do you have any plans on supporting yolov5 or yolov7 instance segmentation models ?

@UygarUsta99 It's under development


Heartbeat & Namespace Flag

Description

added heartbeat to config for long running services
added namespace cli flag so that namespace in component config can be overridden at cli level

Codecov Report

Merging #3 (5692661) into master (7b5d461) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #3   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines            6         6
=========================================
  Hits             6         6

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 7b5d461...5692661. Read the comment docs.


able to list favo dishes and remove them form local storage

able to list favo dishes and remove them form local storage

finished adding the logic to list and add item in local storage and also bug were you cant remove item is fixed


feature support SHOW CHARSET statement.

Proposed changes
SHOW CHARSET
Charset|Description  |Default collation |Maxlen|
-------+-------------+------------------+------+
utf8mb4|UTF-8 Unicode|utf8mb4_general_ci|4     |
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

@morrySnow PTAL

run buildall

run buildall


[Material] RaisedButton text does not default to ColorScheme.onPrimary

Function getTextColor in ButtonThemeData on the case ButtonTextTheme.normal returns black or white. Material Guidelines expect onPrimary since the fill color is primary.
Details: Make backwards compatible with existing behavior.

@HansMuller

How is this not expected? Doesn't onPrimary mean white or black depending on the Brightness of the primary color?

onPrimary should be customized to fit in with the other colors in your scheme. Here's how the designer of Shrine did it:

Oh OK, seems like I need to take a closer look at Color Schemes again

Any progress on this? #38646 needs this to be fixed, since AlertDialogs no longer respect normal theming, and rely solely on ColorSchemes now.

Any update on this ?

CC @darrenaustin

is there a way to track the changes made in the framework regarding the colorScheme ?

@willlarche, @Levi-Lesches, @bitsydarel - it's been a long time since this issue was opened and I apologize for the lack of activity. If you're still interested in the topic, there's a wide-ranging proposal for updating the button widgets and their themes in #54776. The new proposal does bind the default button colors to the overall theme's ColorSchme, per the Material Design spec.
I'm going to close this issue in favor of #54776.


augment downloadFile to include cacert in request if provided #239

New Pull Request
For more information, see the CONTRIBUTING guide.
Prerequisites
Please make sure you can check the following boxes:

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project
[x] All new and existing tests passed

Type(s) of Changes
Contribution Type
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue, please reference the issue id)
[x] New feature (non-breaking change which adds functionality, make sure to open an associated issue first)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes

Description
When testing the new feature #239, we weren't able to publish pilet from npm because our private npm registry require the request to include cacert
[0068] Failed to download via HTTP: self signed certificate in certificate chain.

This change should resolve the cert issue.
Remarks
I'm not sure how to test this change locally... 🤞

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


Fix/issue 642

Codecov Report

Merging #652 into rusk-ipc will increase coverage by 0.77%.
The diff coverage is 60.23%.

@@             Coverage Diff              @@
##           rusk-ipc     #652      +/-   ##
============================================
+ Coverage     54.96%   55.74%   +0.77%
============================================
  Files           156      159       +3
  Lines          9689    10451     +762
============================================
+ Hits           5326     5826     +500
- Misses         3723     3914     +191
- Partials        640      711      +71

Impacted Files
Coverage Δ

pkg/config/consts.go
0.00% <ø> (ø)

pkg/core/consensus/collector.go
0.00% <0.00%> (ø)

pkg/core/consensus/committee/handler.go
80.00% <0.00%> (-8.89%)
:arrow_down:

pkg/core/consensus/header/header.go
24.75% <0.00%> (ø)

pkg/core/consensus/reduction/testutil.go
46.42% <0.00%> (-1.72%)
:arrow_down:

pkg/core/data/transactions/provider.go
0.00% <0.00%> (ø)

pkg/gql/handler.go
0.00% <ø> (ø)

pkg/gql/httpserver.go
65.11% <0.00%> (ø)

pkg/gql/query/mempool.go
30.00% <0.00%> (-1.04%)
:arrow_down:

pkg/gql/query/transactions.go
63.86% <0.00%> (+2.07%)
:arrow_up:

... and 68 more

Continue to review full report at Codecov.

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


Is there an idiomatic way to insert from select?

Is there a way to do the following SQL in requery without having to load the select into memory?
INSERT INTO table1(column1, column2)
  SELECT
    column1, column2
  from table2

Currently this isn't captured in the query api but should be easy enough to add


Module React.ui not found

I simply created a new project using reapp. After navigating into the project directory and running reapp run, I get the following error in the terminal,
ERROR in ./~/reapp-kit/dist/index.js
Module not found: Error: Cannot resolve module 'reapp-ui/all' in /home/jay/RT/node_modules/reapp-kit/dist
 @ ./~/reapp-kit/dist/index.js 43:18-41

ERROR in ./~/reapp-kit/dist/index.js
Module not found: Error: Cannot resolve module 'reapp-ui/helpers/Theme' in /home/jay/RT/node_modules/reapp-kit/dist
 @ ./~/reapp-kit/dist/index.js 63:13-46

ERROR in ./~/reapp-kit/themes/ios.js
Module not found: Error: Cannot resolve module 'reapp-ui/themes/ios' in /home/jay/RT/node_modules/reapp-kit/themes
 @ ./~/reapp-kit/themes/ios.js 1:17-47

ERROR in ./~/reapp-kit/dist/index.js
Module not found: Error: Cannot resolve module 'reapp-ui' in /home/jay/RT/node_modules/reapp-kit/dist
 @ ./~/reapp-kit/dist/index.js 11:10-29

ERROR in ./~/reapp-kit/dist/lib/theme.js
Module not found: Error: Cannot resolve module 'reapp-ui' in /home/jay/RT/node_modules/reapp-kit/dist/lib
 @ ./~/reapp-kit/dist/lib/theme.js 6:9-28

ERROR in ./~/reapp-kit/dist/lib/Reapp.js
Module not found: Error: Cannot resolve module 'reapp-ui/all' in /home/jay/RT/node_modules/reapp-kit/dist/lib
 @ ./~/reapp-kit/dist/lib/Reapp.js 5:17-40

Apologies, I'm not fully used to the new release process with reapp-ui, this should be fixed.

Works now, thanks Nate.

what is the solution?

@xinghengwang - Try npm install -g reapp
There were some updates that should have fixed this...
I can work with you if this doesn't work... I'm on the reapp gitter often, so just PM me there.

yes, that is what I did. Still issues of module not found. I'll private
message you on Gitter. Thanks for willing to help.
On Thu, Jun 23, 2016 at 6:31 AM, ryd0rz<EMAIL_ADDRESS>wrote:

@xinghengwang https://github.com/xinghengwang - Try npm install -g reapp
There were some updates that should have fixed this...
I can work with you if this doesn't work... I'm on the reapp gitter often,
so just PM me there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reapp/reapp/issues/54#issuecomment-228050323, or mute
the thread
https://github.com/notifications/unsubscribe/AGW5v_vHVb_FPrKkCb-d0zejBckhQChPks5qOoqwgaJpZM4EH1xW
.


Don't fail DELETE if instance was not provisioned due to terminal failure

This PR is a

[ ] Feature Implementation
[x] Bug Fix
[ ] Documentation

What this PR does / why we need it:
DELETE after terminal provisioning failure (e.g. 400 from broker) fails. This PR fixes this.
Which issue(s) this PR fixes
Fixes #2467
Please leave this checklist in the PR comment so that maintainers can ensure a good PR.
Merge Checklist:

[ ] New feature

[ ] Tests
[ ] Documentation

[ ] SVCat CLI flag
[ ] Server Flag for config

[ ] Chart changes
[ ] removing a flag by marking deprecated and hiding to avoid
breaking the chart release and existing clients who provide a
flag that will get an error when they try to update

/ok-to-test

/approve
/lgtm


Provide a named listener annotation for Message Throws

Description
Very often a message is thrown but there is a associated implementation java class being called.  We should not assume that the use of Message Throws are going to be associated with the Events add on.  A message throw could be "send an email".
It looks like we ~could use a ProcessEventListener, but it would be a little more "clean" if we could do something like
(not suggesting these should be the specific words used, I am only suggesting the spirit of the ability, and the finer naming details can be worked out during dev)
...
    @Override
    @Event("MyMessageName", "OptionalProcessDefinitionId")
    public void event(EventThrow event) {
         // my impl details for what to do when the BPMN Event Throw occurs
    }
...

Implementation ideas
No response

An for that, you should provide an email addon ;)
Thats the idea of Kogito, for any event publishing functionality, you should provide and addon that implements EventPublisher interface.
And addon is just an additional maven depedency you include in your pom.xml that can be as simple as an implementation of EventPublisher that prints the event to the console
Thats the way Kogito is intender to be extended.

@fjtirado does Kogito support multiple event publishers in parallel?

yes, Kogito loads all beans implementing EventPublisher interface and iterated through them


response 200 ok. captcha not ready ?

why is that ?

II have encountered the same problem. Have you solved it?

Could you please provide more info on the problem: an example of your code, expected result and actual result.


addGeoJSON

Hi, I started thinking of https://github.com/StephanGeorg/staticmaps/issues/81 and have made a method addgeoJSON. The method accepts the following object as a parameter:
{geoJSON, lineOptions, markerOptions, polygonOptions}. geoJSON is a geoJSON object as specified in RFC 7946. The line-, marker- and polygonOptions are the same as specified otherwise but do not have the coordinates property. The addGeoJSON should accept all possible geoJSON types. Only the options that are present in the geoJSON are required (for instance a geoJSON with only linestrings do not need to have the markerOptions and polygonOptions). I have manually tested that the function shoul work as intended for all geoJSON types.
The main con that I see with my implementation is that the user an not stylize individual lines, markers, or polygons but instead all lines, markers and polygons have a shared style but I thought that this implementation is the most straightforward.

Hi, I  started thinking of #81 and have made a  method addgeoJSON. The method accepts the following object as a parameter:
{geoJSON, lineOptions, markerOptions, polygonOptions}. geoJSON is a geoJSON object as specified in RFC 7946. The line-, marker- and polygonOptions are the same as specified otherwise but do not have the coordinates property. The addGeoJSON should accept all possible geoJSON types. Only the options that are present in the geoJSON are required (for instance a geoJSON with only linestrings do not need to have the markerOptions and polygonOptions). I have manually tested that the function shoul work as intended for all geoJSON types.
The main con that I see with my implementation is that the user an not stylize individual lines, markers, or polygons but instead all lines, markers and polygons have a shared style but I thought that this implementation is the most straightforward.

Thanks @leoalho for the PR. Currently I'm very busy so it will take a bit to have a look at it. Could you @kuuup-at-work pls review this and give your feedback? Would be awesome. Thanks guys.

Hi.
Thanks for your work @leoalho.
Your current version does not work for me. The rendered image is empty and there are no error messages. It's hard to tell if I made a mistake or if it doesn't work because of your changes because there are no automatic tests.

Hi, thank you both @kuuup-at-work and @StephanGeorg for commenting. I'll naturaly add the appropriate automatic tests once I've gotten green light that my implementation works as desires, which it does not seem to do at the moment.
Could you @kuuup-at-work  want to share how you tried to render the map? And did was the render completely empty, ie. without any basemap either?
If it helps at all here is an example rendering and associated code that I did when manually testing a geoJSON object consisting of a "GeometryCollection" type.  Please note in the code, that I used a relative path to the dist of my fork of staticmaps.

const StaticMaps = require("../staticmaps/dist/staticmaps");
const path = require("path");

const geoJSON = {
  type: "GeometryCollection",
  geometries: [
    {
      type: "MultiPolygon",
      coordinates: [
        [
          [125.6, 10.2],
          [125.7, 10.2],
          [125.7, 10.1],
          [125.6, 10.1],
          [125.6, 10.2],
        ],
        [
          [125.65, 10.19],
          [125.61, 10.11],
          [125.69, 10.11],
          [125.65, 10.19],
        ],
      ],
    },
    {
      type: "LineString",
      coordinates: [
        [125.6, 10.25],
        [125.7, 10.25],
      ],
    },
  ],
};

const lineOptions = {
  color: "#00ff00",
  width: 3,
};

const polygonOptions = {
  color: "#ff0000",
};

const render = async (zoom, center, geoJSONOptions) => {
  const options = {
    width: 600,
    height: 400,
  };
  const map = new StaticMaps(options);

  map.addGeoJSON(geoJSONOptions);

  await map.render(center, zoom);

  const fileName = `testi${Date.now()}`;
  await map.image.save(path.join("images", `${fileName}.png`));
};

render(10, [125.65, 10.1], {
  geoJSON: geoJSON,
  lineOptions: lineOptions,
  polygonOptions: polygonOptions,
});

Hi @kuuup-at-work, have you had the time to check my last comment?


Error !!! Create channel failed make: *** [Makefile:35: start-network1-local] Error 1

While trying to install Weaver DLT I ran into this problem.
I have tried reinstalling Ubuntu, all prerequisites, changing time (docker showed a different time, thought that might've been the issue).
Any help would be much appreciated.

Hi @Aishwarya-Kulshrestha , we have fabric-certs checked-in to the repo for deterministic demonstration. But these fabric-certs have expired, need to create them again, we will fix this as soon as possible.

PR #304 will fix this.

Awesome, thanks!
On Mon, 8 Aug, 2022, 21:41 Sandeep Nishad, @.***> wrote:

Closed #300
https://github.com/hyperledger-labs/weaver-dlt-interoperability/issues/300
as completed via #304
https://github.com/hyperledger-labs/weaver-dlt-interoperability/pull/304
.
—
Reply to this email directly, view it on GitHub
https://github.com/hyperledger-labs/weaver-dlt-interoperability/issues/300#event-7148458890,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASMFYUKGBEOH5HZUULLWV23VYEWSBANCNFSM55HVRTGQ
.
You are receiving this because you were mentioned.Message ID:
<hyperledger-labs/weaver-dlt-interoperability/issue/300/issue_event/7148458890
@github.com>


Fixed grammar, made default usage clearer.

I missed the line about removing .custom.semantic.json, thought it should be spelled out a bit more clearly. I read the docs 2 times before, and posted an issue on Stack Overflow, only to find I needed to remove the above mentioned file.

Thank you! I seemed to have messed up the merge (with conflicts) as it doesn't show up here :
Changes are in the readme though ;)


I would like to work on this issue. @niftynei

Go for it @AmoliShah!

@timabbott Changes are to be made in RecyclerMessageAdapter.java activity. Right?

@AmoliShah No. it is for gsm. Check it out here

ok thanks. @saketkumar95

@AmoliShah Are you working on this?


XOpenDisplayFailed

Is there an existing issue for this?

[X] I have searched the existing issues.

What's the problem?
thread 'main' panicked at 'Failed to initialize any backend! Wayland status: XdgRuntimeDirNotSet X11 status: XOpenDisplayFailed', /root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/linux/mod.rs:719:9
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
How did you install the app?
installed from crates.io
Operating System
Linux
Additional context
OS: CentOS Linux release 7.9.2009 (Core)
cargo install<EMAIL_ADDRESS>

The same issue was solved in #11:

It's strange, but... I deleted my ~/.cargo folder and build every cargo app I have [...] now sniffnet run with libpcap blessing without problem. Huh. I tried in alacritty and wezterm.

It's not a definitive solution but deleting that folder and rebuilding the app solved the issue in the past.
If it doesn't solve the issue for you, can you also please try to download the provided DEB package from the release page?


how to see which option the user voted for in pollUpdateMessage

Does anyone have a practical example of how to do this without using example.ts? the messages.update controller is not capturing pollMessage changes

read this, It solved that problem for me
#164
https://github.com/WhiskeySockets/Baileys/issues/164#issuecomment-1611851211

read this, It solved that problem for me #164
#164 (comment)

thanks, it worked here for min, somehow "store" makes it possible to vote updates in messages.update


Password Expriration

It would be helpful to add a link to this documentation also https://docs.microsoft.com/en-us/microsoft-365/admin/manage/set-password-expiration-policy?view=o365-worldwide

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

ID: decdefc9-3082-1350-520c-ce226be74a6e
Version Independent ID: 6457d5c9-87b0-bd1d-0a8e-7e5d9326851b
Content: Basic Security Set Up for Microsoft 365
Content Source: Community/basic-security-set-up-for-microsoft-365.md
Service: security
GitHub Login: @helloitsliam
Microsoft Alias: efrene

@DeanGross - We welcome pull requests from smart people like you!
@helloitsliam- Want to check on this?

@sympmarc I have created PR #431 to resolve this.

Hey @sympmarc, I actually created a different PR, that adds a missing license, and retains the original password expiration link plus the additional one. PR 432

@helloitsliam thanks for adding the missing licence, PR #431 already has those links however.


Research: Newborn Screening Test Order & Results Process

Story
Sentence
As the CDC TI Team,
In order inform future research needs,
I need a deeper understanding of the newborn screening test order & results domain.
Acceptance Criteria

[ ] Develop and document an understanding the Newborn Screening Domain
[ ] Document the workflow of newborn screening test order and results for at least two states (e.g. GA)
[ ] Document at least two of the national polices or laws related to newborn screening test order and results

Tasks

[ ] Document high level understanding of Newborn screening
[ ] Document Georgia PHL current state process based on their website
[ ] Define a second priority State for analysis of current state
[ ] Define a second priority State for analysis of current state

Additional Context
Definition of Done

[ ] TBD.

Some research and high level workflow documented in Lucid here


question about data augmentation

Hi, I am new to pytorch and want to make sure. You use this line to ensure that the augmentations, e.g. randomcrop(), randomflip(), colorjitter(), applied to all frames are identical?
https://github.com/tarun005/FLAVR/blob/8896a0f6e60a6c7b8b08aa973b79b230053690a4/dataset/vimeo90k_septuplet.py#L66

According to my experiments, augmentations applied to each frame are different and the difference among frames are huge. I am surprised that this model should converge in such extreme conditions. I will try to train the model to see if I can reproduce the reported metrics.

Closing this due to no further update.


Active:active is enabled by default

Active:active support was recently added to the server chart, and the docs indicate it is disabled by default:
https://github.com/aquasecurity/aqua-helm/commit/e688b2789620078e1a9bb931c6ab0c9559a22a13#diff-7d62b4cde47a08800e3b7212ca2dd397R81
However the default value for activeactive is actually true:
https://github.com/aquasecurity/aqua-helm/commit/ce60ce8e7fdaf83413a226846da242d7679756b5#diff-0fd8ccf1d8b0f5309cd540e606d67248R17
The default value should be false, as this is an unwanted new setting for existing consumers of the chart.

Looks like this was resolved in https://github.com/aquasecurity/aqua-helm/commit/75856ffe541c0d3d1f9e2029be79cfb2d8483e6e for 5.0

@cdunford Thanks for reporting the issue.
The issue is fixed in the latest chart so moving the issue to closure.


Patch for AIMA4e reimplement Fig 4.5 Simulated-Annealing from AIMA3e branch

Issue #224
Added more generic support , dropped action interface in preference to type parameter.
Will be trying to improve documentation. @ctjoreilly

Closing as this is a mixed pull request again (i.e. not specific to a particular task). Will be integrating pull request #142 as the initial implementation of simulated annealing on the aima4e branch.


add PhysicalExpr to support, Selective Aggregates feature using FILTER clause

Which issue does this PR close?
#2214
Add support for adding filter to aggregation.
eg: AGGREGATE(input) FILTER (WHERE condition)

Closes #2214
What changes are included in this PR?

ExprWithFilter  is added so that expr can be evaluated based on the given filter.
ExprWithFilter is used in aggregate functions if it contains filter
filtered column name will be added to the TableScan's projection by making expression visitors to visit aggregate function filters

@andygrove it's done. Please have a look and let me know :)

I am not sure what the current status of this PR is and it is not passing CI, so marking it as "draft" -- please mark it as ready for review when it is

@alamb I've fixed the test. Yess, it's ready for review.

@alamb PTAL when you can. LGTM but I have not been working so closely with physical execution lately.

I am sorry I don't have time to review this PR 😭

Hey @alamb
No worries, I’ll close this for now. We can rework this when this is relevant again or someone can take this work again and patch it

Thanks and sorry @poonai


Include prettier in npm test run

Applied to all files

Thanks for the real world usage. I filed a few issues on prettier repo for things we can improve!


Change bot message for notifying reviewers.

/cc @RByers @gsnedders.
Fixes w3c/web-platform-tests#4228.

Deployed this. LMK if it helps of if there are other things you want to change.


Fix NMFOptimizer's limited hyperparameter name

This PR fixes bug related to NMFOptimizer class.

Change limited hyperparamter name from top_k to n_components

Codecov Report

Merging #81 (0fbda8c) into main (77e31a0) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
- Coverage   95.68%   95.66%   -0.03%
==========================================
  Files          68       68
  Lines        3549     3549
==========================================
- Hits         3396     3395       -1
- Misses        153      154       +1


Issue with Zoom after running mkcert

After running mkcert to install localhost certs , I am now getting this Zoom popup macOS 11.2, which continues to appear no matter what I click:

Any ideas? Thanks in advance!

Hmm, installing mkcert itself should not (and does not on my machine) cause that. Did you also enable an intercepting proxy and configure it to use the mkcert root? That would explain the popup.

That indeed would be the issue., makes sense.
Closing


feat: gemini support

Key Changes

Added Google's Gemini models:

Chat: gemini-1.5-pro
Apply: gemini-1.5-flash, gemini-1.5-flash-8b
Embedding: gemini/text-embedding-004

Improved provider architecture:

Added model validation across providers
Standardized error handling
Moved common OpenAI-compatible code to shared module

Database updates:

Added provider prefixes to vector tables (e.g., openai/, ollama/)
Added Gemini embeddings table

Breaking Changes

Embedding model names now require provider prefix
Database migration required for vector table renames

Closing in favor of #120 which contains a reimplementation of these changes.

Closing in favor of https://github.com/glowingjade/obsidian-smart-composer/pull/120 which contains a reimplementation of these changes.


Detailed documentation for local setup and deploys

Depends on #100 - add detailed documentation on running setup locally, as well as deploying to cloud infrastructure.

This is now replaced by #372


Timezone vendor not installing correctly

Dump from when I try to install to updated core:
pi@raspberrypi:~/test_openaps/tutorial $ openaps vendor add openapscontrib.timezones
No module named recurrent
openapscontrib.timezones doesn't seem to be an importable python module
If it is a python module, try using --path to influence
PYTHONPATH

Any luck with this? I'm running into the same issue - slightly different output - but same end results:
pi@raspberrypi:~/Documents/tutorial-hello $ openaps-vendor add openapscontrib.timezone
No module named openapscontrib.timezone
openapscontrib.timezone doesn't seem to be an importable python module
If it is a python module, try using --path to influence
PYTHONPATH

I'm not familiar enough with openaps yet to really know how to troubleshoot, so I'm trying to ensure that I don't skip steps to avoid them coming back to haunt me later!

@djjdub  Page is gone, you still have a list somewhere?


visualisation of transformer attention

Hi,
When trying to access the transformers internal outputs, in an attempt to visualize attention, I am unable to access these layers on the normal tensorflow way using e.g. "model.get_layer('transformer').get_layer('decoder').get_layer('layer_5').multihead_attn.output". It seems the graph is not fully connected as it should be. I have found a couple of fixes that allowed me to access these layers succesfully:

define pos_encoding with boolean version of your masks as it is now (masks = tf.zeros((tf.shape(x)[0], tf.shape(x)[1], tf.shape(x)[2]), tf.bool);   pos_encoding = self.pos_encoder(masks);  but pass the masks to the transformer as tf.float32 instead of tf.bool.
pass the query to he transformer as self.query_embed(tf.constant(1.0)) instead of self.query_embed(None); in the FixedEmbedding Layer, return tf.multiply(self.w, x) instead of returning self.w; passing 'None' as argument causes the graph to break.
in the transformer code, make sure to put 'memory' as the first argument in the call to the decoder; put 'target' as the second argument: inputs to call() should always be the first argument; when you put 'memory' as the second argument, the graph breaks apart; in case of multiple inputs, it's even better to put them in a list.
Applying these changes to the code allowed me to access the multihead attention output using model.get_layer('transformer').get_layer('decoder').get_layer('layer_5').multihead_attn.output

Hey, could you provide an example code ? thanks in advance


No Response error

Hi there,
I've had a lot of fun with this project, however, I am having a problem both with this plugin: every now and then, when I open the Home app, all devices appear as No Response. I believe other people have had similar problems with this plugin  #17). All it takes is closing and re-opening the app for the messages to reset.
This problem is becoming increasingly annoying however, and this is why I am seeking help. Note that this problem only occurs when using this plugin; furthermore, when this plugin is isolated on my test "Home" with no other plugins, the issue does not occur. Thus, it seems the problem only occurs when in the same "Home" as multiple other accessories, and, at that, only every now and then (I'd say 1 in 5 times or if I haven't opened the Home app in a few minutes).
After running Homebridge in debug mode (DEBUG=* homebridge -D), something strange happens. Whenever I get the No Response error in the Home app, the following 5 lines always appear at the end of the debug log (where <IP_ADDRESS> is the local IP of my iPhone):
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP Response is finished +3ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] Writing pending HTTP event data +1ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] Client connection closed +113ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP connection was closed +2ms
  EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP server was closed +1ms

These lines never appear in this order anywhere else as long as there is no No Response error (the first 2 lines appear frequently by themselves elsewhere, but the last 2/3 only appear during this error). Any ideas on what might be causing the plugin to make this happen?
Thank you in advance for your help,
Kind regards, Tom

@Tommrodrigues I this problem persist after your pull request ? I updated npm as +<EMAIL_ADDRESS>version.
Best,

No, the issue has been around since I first became aware of the plugin. However, I feel the issue may not be due to the plugin as the issue is noted elsewhere due to network problems (like here).
Thanks anyway


Memory Leak: GMGridView never gets deallocated

GMGridView never gets deallocated because of a retain cycle with GMGridViewCell.
If you repeatedly push/pop it with a UINavigationController it will never dealloc and the memory usage increases as more GMGridView (and related) objects get alloced.
It's due to a reference to self in the cell.deleteBlock in newItemSubViewForPosition:. Updating to weakSelf fixes it.
So happy to have this resolved...

Good catch! Thank you.

not working for me. I've have the same problem. I put in each cell, a UIImageView, but when i repeatedly push/pop it from UiNavigationController, memory remain allocated.

I'm not able to reproduce the issue using the latest GMGridView code using ARC with iOS 4.3 and iOS 5.
I created a sample project that you can use as a starting point to isolate the issue. It's a simple UINavigationController that pushes/pops one of the view controllers from the GMGridView example project:
https://github.com/soleares/GMGridViewMemoryTest
Here's what I recommend:

Run the sample in Instruments with the Allocations instrument and verify that the GMGridView (and related objects) is deallocating. You should see the '# Living' column increase to 1 when you push, and the object should go away when you pop.
Slowly add in any custom code from your project and see if/when the issue occurs.

Hope that helps.

Is this still a bug in devel? We really should remove this branch and just use main, it's way to confusing otherwise.


[Concurrency] Menu query doesn't handle concurrency

_contentManager.Query().ForType("Menu").Join<TitlePartRecord>().OrderBy(x => x.Title).List())

In MainMenuService.

For reference while trying to understand why this query is locking:
https://www.brentozar.com/archive/2018/04/how-entity-framework-and-nhibernate-can-cause-poison-resource_semaphore-waits/
http://sqlapproach.blogspot.com/2012/08/active-processes.html

I am trying those tests.
I am using the 1.10.x branch, and a fresh tenant. I try to create contents of type Page. The definition of this ContentType has been changed, so it only contains: CommonPart, TitlePart, MenuPart. I did that because we know there is a concurrency issue with the MenuPart, and I did not want to have other stuff making things even more complex.
Concurrent creation of two items fails, as we knew it would. Here's the full error from the logs:
2018-05-10 09:06:01,454 [38] Orchard.ContentManagement.Drivers.Coordinators.ContentPartDriverCoordinator - DevNoLaser - ERROR GenericADOException thrown from IContentPartDriver by Orchard.Core.Navigation.Drivers.MenuPartDriver
NHibernate.Exceptions.GenericADOException: could not execute query
[ SELECT this_.Id as Id11_3_, this_.Number as Number11_3_, this_.Published as Published11_3_, this_.Latest as Latest11_3_, this_.Data as Data11_3_, this_.ContentItemRecord_id as ContentI6_11_3_, titlepartr2_.Id as Id136_0_, titlepartr2_.Title as Title136_0_, titlepartr2_.ContentItemRecord_id as ContentI3_136_0_, contentite1_.Id as Id7_1_, contentite1_.Data as Data7_1_, contentite1_.ContentType_id as ContentT3_7_1_, contenttyp6_.Id as Id13_2_, contenttyp6_.Name as Name13_2_ FROM Orchard_Framework_ContentItemVersionRecord this_ inner join Title_TitlePartRecord titlepartr2_ on this_.Id=titlepartr2_.Id inner join Orchard_Framework_ContentItemRecord contentite1_ on this_.ContentItemRecord_id=contentite1_.Id left outer join Orchard_Framework_ContentTypeRecord contenttyp6_ on contentite1_.ContentType_id=contenttyp6_.Id WHERE contentite1_.ContentType_id = @p0 and this_.Published = @p1 ORDER BY titlepartr2_.Title asc ]
  Name:cp0 - Value:1  Name:cp1 - Value:True
[SQL: SELECT this_.Id as Id11_3_, this_.Number as Number11_3_, this_.Published as Published11_3_, this_.Latest as Latest11_3_, this_.Data as Data11_3_, this_.ContentItemRecord_id as ContentI6_11_3_, titlepartr2_.Id as Id136_0_, titlepartr2_.Title as Title136_0_, titlepartr2_.ContentItemRecord_id as ContentI3_136_0_, contentite1_.Id as Id7_1_, contentite1_.Data as Data7_1_, contentite1_.ContentType_id as ContentT3_7_1_, contenttyp6_.Id as Id13_2_, contenttyp6_.Name as Name13_2_ FROM Orchard_Framework_ContentItemVersionRecord this_ inner join Title_TitlePartRecord titlepartr2_ on this_.Id=titlepartr2_.Id inner join Orchard_Framework_ContentItemRecord contentite1_ on this_.ContentItemRecord_id=contentite1_.Id left outer join Orchard_Framework_ContentTypeRecord contenttyp6_ on contentite1_.ContentType_id=contenttyp6_.Id WHERE contentite1_.ContentType_id = @p0 and this_.Published = @p1 ORDER BY titlepartr2_.Title asc] ---> System.Data.SqlClient.SqlException: Transaction (Process ID 68) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
   in System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection, Action`1 wrapCloseInAction)
   in System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning(TdsParserStateObject stateObj, Boolean callerHasConnectionLock, Boolean asyncClose)
   in System.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
   in System.Data.SqlClient.SqlDataReader.TryHasMoreRows(Boolean& moreRows)
   in System.Data.SqlClient.SqlDataReader.TryReadInternal(Boolean setTimeout, Boolean& more)
   in System.Data.SqlClient.SqlDataReader.Read()
   in NHibernate.Driver.NHybridDataReader.Read()
   in NHibernate.Loader.Loader.DoQuery(ISessionImplementor session, QueryParameters queryParameters, Boolean returnProxies, IResultTransformer forcedResultTransformer)
   in NHibernate.Loader.Loader.DoQueryAndInitializeNonLazyCollections(ISessionImplementor session, QueryParameters queryParameters, Boolean returnProxies, IResultTransformer forcedResultTransformer)
   in NHibernate.Loader.Loader.DoList(ISessionImplementor session, QueryParameters queryParameters, IResultTransformer forcedResultTransformer)
   --- Fine della traccia dello stack dell'eccezione interna ---
   in NHibernate.Loader.Loader.DoList(ISessionImplementor session, QueryParameters queryParameters, IResultTransformer forcedResultTransformer)
   in NHibernate.Loader.Loader.ListIgnoreQueryCache(ISessionImplementor session, QueryParameters queryParameters)
   in NHibernate.Impl.SessionImpl.List(CriteriaImpl criteria, IList results)
   in NHibernate.Impl.CriteriaImpl.List(IList results)
   in NHibernate.Impl.CriteriaImpl.List[T]()
   in Orchard.ContentManagement.DefaultContentQuery.Slice(Int32 skip, Int32 count) in C:\Sviluppo\Orchard.Sources.Github\Orchard\src\Orchard\ContentManagement\DefaultContentQuery.cs:riga 191
   in Orchard.ContentManagement.DefaultContentQuery.ContentQuery`1.Orchard.ContentManagement.IContentQuery<T>.List() in C:\Sviluppo\Orchard.Sources.Github\Orchard\src\Orchard\ContentManagement\DefaultContentQuery.cs:riga 292
   in Orchard.Core.Navigation.Services.MainMenuService.GetMenus() in C:\Sviluppo\Orchard.Sources.Github\Orchard\src\Orchard.Web\Core\Navigation\Services\MainMenuService.cs:riga 64
   in Orchard.Core.Navigation.Drivers.MenuPartDriver.Editor(MenuPart part, Object shapeHelper) in C:\Sviluppo\Orchard.Sources.Github\Orchard\src\Orchard.Web\Core\Navigation\Drivers\MenuPartDriver.cs:riga 40
   in System.Dynamic.UpdateDelegates.UpdateAndExecute3[T0,T1,T2,TRet](CallSite site, T0 arg0, T1 arg1, T2 arg2)
   in Orchard.ContentManagement.Drivers.ContentPartDriver`1.Orchard.ContentManagement.Drivers.IContentPartDriver.UpdateEditor(UpdateEditorContext context) in C:\Sviluppo\Orchard.Sources.Github\Orchard\src\Orchard\ContentManagement\Drivers\ContentPartDriver.cs:riga 62
   in Orchard.ContentManagement.Drivers.Coordinators.ContentPartDriverCoordinator.<>c__DisplayClass11_0.<UpdateEditor>b__0(IContentPartDriver driver) in C:\Sviluppo\Orchard.Sources.Github\Orchard\src\Orchard\ContentManagement\Drivers\Coordinators\ContentPartDriverCoordinator.cs:riga 63
   in Orchard.InvokeExtensions.Invoke[TEvents](IEnumerable`1 events, Action`1 dispatch, ILogger logger) in C:\Sviluppo\Orchard.Sources.Github\Orchard\src\Orchard\InvokeExtensions.cs:riga 17

Just the query (@p0 = 1 and @p1 = True):
SELECT
	this_.Id as Id11_3_,
	this_.Number as Number11_3_,
	this_.Published as Published11_3_,
	this_.Latest as Latest11_3_,
	this_.Data as Data11_3_,
	this_.ContentItemRecord_id as ContentI6_11_3_,
	titlepartr2_.Id as Id136_0_,
	titlepartr2_.Title as Title136_0_,
	titlepartr2_.ContentItemRecord_id as ContentI3_136_0_,
	contentite1_.Id as Id7_1_,
	contentite1_.Data as Data7_1_,
	contentite1_.ContentType_id as ContentT3_7_1_,
	contenttyp6_.Id as Id13_2_,
	contenttyp6_.Name as Name13_2_
FROM
	Orchard_Framework_ContentItemVersionRecord this_
	inner join Title_TitlePartRecord titlepartr2_ on this_.Id=titlepartr2_.Id
	inner join Orchard_Framework_ContentItemRecord contentite1_ on this_.ContentItemRecord_id=contentite1_.Id
	left outer join Orchard_Framework_ContentTypeRecord contenttyp6_ on contentite1_.ContentType_id=contenttyp6_.Id
WHERE
	contentite1_.ContentType_id = @p0
	and this_.Published = @p1
ORDER BY titlepartr2_.Title asc

Following what was done at https://www.brentozar.com/archive/2018/04/how-entity-framework-and-nhibernate-can-cause-poison-resource_semaphore-waits/ , I copied that query in SQLQueryStress, and ran it at 50 iterations on 50 threads.
The table for the WAIT_STATS of the results for that procedure had no RESOURCE_SEMAPHORE line:

Next, I tried running sp_BlitzFirst while running the concurrency scripts we have been using for our tests. I ran EXEC sp_BlitzFirst @ExpertMode = 1, @Seconds = 30 on sql server while launching teh ParallelTest.js as many times as I could (basically, I launch that script again right after it finishes).
The following image is the results from that:

Here we have that row with wait_type == LCK_M_S that was absent when running the queries from SQLQueryStress, with more than 2 seconds waited each time.
We still do not have the RESOURCE_SEMAPHORE waits, so I would guess that we are seeing something different form what's in the post that proposed the testing procedure.

the second result for querying
nhibernate LCK_M_S
on google is #4566 , so I am checking that

I am trying to see the information from sp_BlitzLock (one of the scripts from the sources @sebastienros posted).

Note that all the deadlock reported there are the ones I caused earlier by launching the scripts for 2 concurrent page creations (see two comments above for my test conditions).
I don't know why we would have a bunch of deadlocks related to TitlePartRecord and some to ContentItemVersionRecord. Looking at the other results for this stored procedure, it looks like the locks were on TitlePartRecord for the first 9 deadlocks, on ContentItemVersionRecord for the other 13 deadlocks afterwards.
The answer (https://stackoverflow.com/a/5353251/2669614) tells that automatic primary keys are often troublesome. Since in the results here we see them as "involved" in the deadlocks, they may be on to something.

I am checking that right now.

@sebastienros could you point me to wherever the NHibernate configuration is handled right now, so I don't have to search for it myself?

I don't think I know enough about NHibernate to figure out how to do the configuration for this in a timely manner. Perhaps someone could step in?
I would be adding the stuff from that SO answer in AbstractDataServicesProvider, or in SqlServerDataServicesProvider, where I would override AlterConfiguration.
I don't know what else I would need.
Likely something to update all existing tables in the db, so that the current Ids are not primary key anymore, but rather "become" computed from the hilo table. This should happen in such a way that the value in the hilo table is already not 1, but rather a value preventig the newly created ids to clash with pre-existing ones.
Also, changes to how new tables are created, so that wherever in a migration we define a column as PrimaryKey(), it does what's necessary for the hilo table and script to do their thing.

Looking at the code, I have more comments on this. What I say here may be wrong, so correct me as needed:

Orchard.ContentManagement.Records.ContentPartAlteration sets the mappings for the Id of ContentPartRecord and ContentPartVersionRecord to be generated as a "copy" of the id of the ContentItemRecord or ContentItemVersionRecord respectively. This may mean that we only need the hilo generation of the Id for those two, and for any record that is not a ContentPartRecord nor a ContentPartVersionRecord. What I mean is that we would need the hilo generation for all those columns in the db that we currently define somewhat like this: .Column<int>("Id", column => column.PrimaryKey().Identity()). I think the change should be in whatever handles the booleans set from .Identity() (is it Orchard.Data.Migration.Interpreters.DefaultDataMigrationInterpreter?).

I am not sure whether we should be thinking about setting this up only for MSSQL Server. While this is an issue we found there, I would think that its solution would not create any issue on other dbms. It seems to me that having this solution coexist with what's currently in live code would create a lot more complexity than "simply" using hilo for every system. This of course assumes that other dbms have no issue with that approach, and I don't see why they would.

I found a workaround for the issues here.
Since the tools report that the SELECT statements are deadlocking because sql server is too pessimistic and locks the entire tables for TitlePartRecord and ContentItemVersionRecord, I wrote an interceptor that adds "WITH (NOLOCK)" to the SELECT statements for those tables. This appears to fix things for the specific case of the issue here, and lets me create as many concurrent pages as I want (Page is defined to have CommonPart, TitlePart, MenuPart).
I'll do a PR here, to share this code, but I don't think it should be merged yet, because I am not sure this approach is correct and does not break something else. Also, this code is WIP: at the very least, if we agree it works, I would add an interface for providers to add further table names.


edgemesh-server inner ip process problem

What happened:
anticipation：
pulic ip ：<IP_ADDRESS>
inner ip ：<IP_ADDRESS>
install：helm install edgemesh --set server.publicIP=<IP_ADDRESS> build/helm/edgemesh
actual results：
tunnel.go:14] Tunnel server addr : /ip4/<IP_ADDRESS>/tcp/20004/p2p/QmQ7CWWuw4aPj82uHwK248suTMoYdLzvsyL9qn3D5995YB
tunnel.go:14] Tunnel server addr : /ip4/<IP_ADDRESS>/tcp/20004/p2p/QmQ7CWWuw4aPj82uHwK248suTMoYdLzvsyL9qn3D5995YB
tunnel.go:14] Tunnel server addr : /ip4/<IP_ADDRESS>/tcp/20004/p2p/QmQ7CWWuw4aPj82uHwK248suTMoYdLzvsyL9qn3D5995YB
What you expected to happen:
edgemesh-server can get the  right address or process with better compatibility
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

EdgeMesh version:10dada6234c7edbccfc5069abfa63da27c53788c

Kubernetes version (use kubectl version):

KubeEdge version(e.g. cloudcore --version and edgecore --version):1.8.2

Cloud nodes Environment:

Hardware configuration (e.g. lscpu):
OS (e.g. cat /etc/os-release):
Kernel (e.g. uname -a):
Go version (e.g. go version):
Others:

Edge nodes Environment:

edgecore version (e.g. edgecore --version):
Hardware configuration (e.g. lscpu):
OS (e.g. cat /etc/os-release):
Kernel (e.g. uname -a):
Go version (e.g. go version):
Others:

/assign @khalid-jobs

We need to support multi advertise address for edgemesh and figure out how the libp2p fetch the ip

/kind feature

It seems that the listener addr is depends on https://github.com/kubeedge/edgemesh/blob/main/server/pkg/tunnel/module.go#L68, if we use <IP_ADDRESS>, the edgemesh-server will listen on all ip in the host. And the https://github.com/kubeedge/edgemesh/blob/main/server/pkg/tunnel/tunnel.go#L13 will only fetch the loopback address and the globalUnicast address which also the can be fetch from routing.Route(net.Ipv4zero) (github.com/google/gopackage/routing)

How to Solve These Problems

[x] deduplicate the edgemesh-server address. https://github.com/kubeedge/edgemesh/pull/195
[ ] EdgeMesh-Server provide advertise address configuration rather than publicIP, so the user can config the ips according their network.

/close


[CH] No need clone const column in some cases

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #3875)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
CI
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

No big deal.


ActionCable support for real-time management

I love the manage rake task and I use it when I'm in a rut, but what would be really great would be if it could give me management advice when I don't yet realize that I need to be managed.  I know that adding ActionCable support is well beyond the scope of a rake task, but it might well be the feature that causes this gem to really take off! 🚀

👍  to this.

Thus far, we have only asked tcramer to write rake tasks. Given how well tcramer does at writing rake tasks, we could entertain PRs that add ActionCable support. If we do pursue this, we should prolly add a test suite next and hook up CI, etc.?

@cjcolvar I'm inclined to punt on this until at least v1.0.0. Tcramer currently has no UI and no assets, so there's no good mechanism for testing an AC implementation at the moment. I won't close this as #wontfix, but would happily welcome PRs!

Adding to Wishlist milestone.


Smoke Example - Unrecognized attribute

(venv) PS S:\Tools\genesis> py .\smoke.py
[Taichi] version 1.7.2, llvm 15.0.1, commit 0131dce9, win, python 3.10.5
[Genesis] [21:47:00] [INFO] Running on [NVIDIA GeForce RTX 3090 Ti] with backend gs.cuda. Device memory: 23.99 GB.
[Genesis] [21:47:01] [DEBUG] [Taichi] Starting on arch=cuda
[Genesis] [21:47:01] [INFO] 🚀 Genesis initialized. 🔖 version: 0.2.0, 🌱 seed: 0, 📏 precision: '32', 🐛 debug: False, 🎨 theme: 'dark'.
Traceback (most recent call last):
  File "S:\Tools\genesis\smoke.py", line 140, in <module>
    main()
  File "S:\Tools\genesis\smoke.py", line 87, in main
    sf_options=gs.options.SFOptions(
  File "S:\Tools\genesis\venv\lib\site-packages\genesis\options\options.py", line 27, in __init__
    gs.raise_exception(f"Unrecognized attribute: {key}")

[Genesis] [21:47:01] [ERROR] Unrecognized attribute: res
[Genesis] [21:47:01] [INFO] 💤 Exiting Genesis and caching compiled kernels...

The smoke example throws the following error:
[Genesis] [21:47:01] [ERROR] Unrecognized attribute: res

@PingchuanMa can you take a look?

Yo thanks for trying out the latest feature. The smoke example only works in the main branch though and hasn't been pushed to pypi yet. If you want to try it now please build genesis from scratch by pip install -e ., or just simply wait for the next push to pypi ;)

sick! thanks for the prompt response. I'm gonna pull latest and rebuild.

Yo thanks for trying out the latest feature. The smoke example only works in the main branch though and hasn't been pushed to pypi yet. If you want to try it now please build genesis from scratch by pip install -e ., or just simply wait for the next push to pypi ;)

Dos this repo supports the editable pip install now? Since I didn't see the setup.py script is included in the repo.

Yo thanks for trying out the latest feature. The smoke example only works in the main branch though and hasn't been pushed to pypi yet. If you want to try it now please build genesis from scratch by pip install -e ., or just simply wait for the next push to pypi ;)

Dos this repo supports the editable pip install now? Since I didn't see the setup.py script is included in the repo.

We followed the modern practice using pyproject.toml [1].


Fix sync activity update zombie

Coverage decreased (-0.03%) to 66.548% when pulling 193c7d20dc4b34b4a889fc267da1401d0016aa7d on fix-sync-activity into d4308b9eb690ad0a1c7a31610285db41d75d85fe on master.


Links in README are broken

Links in the README sections CM-Well Tutorial and Developer Resources are all returning 404's.
Cheers!

Hi,
please see https://cm-well.github.io/CM-Well/ which contains all the updated docs.
Please notice to upper tabs. The tutorial and the developer guide are there.
The README should be updated soon.
Thanks for reporting.

@nruppin let's remove those links or fix them. Maybe we need to rewrite entire root Readme.md now after the move to github.io ?


Support for MemSQL Database in Flyway

MemSQL is a in-memmory and disk based distributed and relational type database which is highly scalable.
MemSQL uses Mysql jdbc connector. We use flyway for database migrations for DB2 and MySQL and would like to add MemSQL to it which is going to be part of our datasystems.
Please see pull request for implementation details.

Duplicate of #1317


Through Hydra Cli rest, we can see its load balancing and sending requests to running instances. But in typical web application to call hydra services, how can i configure to call with domain we specify without port?

Through Hydra Cli rest, we can see its load balancing and sending requests to running instances. But in typical web application to call hydra services, how can i configure to call with domain we specify without port?

@ChandrewR Let us know whether the existing documentation is unclear.  We'll be happy to improve it.
https://www.hydramicroservice.com/docs/hydra-core/messaging.html

Hi Carlos, Thanks for the reply..But what I am looking is how to call load balanced microservices in hydra. For example, i have a microservice runnning on two instances - http::7000/api and http::7001/api. I need to add domain and configure these two instances to 80 port to call as http://myservicename/api..So it will get load balanced..Can you help me on this please? I dont see any docs for this..

I see. You could use something like NGinX.  However, we use HydraRouter for this purpose.
https://www.hydramicroservice.com/docs/tools/hydra-router/

I have tried adding externalRoutes and didn't work..I made it as http as well and called the api as http://test-service.com/processfileapis/v1/getfilestatus and it didn't work. In doc, I'm seeing [get]/.... and is it mandatory to mention request type?

Not sure about this externalRoutes comes with Hydra-route(or i should implement NGinX and then use this approach) and how to use it to load balance? I couldn't see any sample project in Git Repo as well.. Can you please help sharing any examples, if you have?

@ChandrewR Hydra services can communicate with each other using internal load balancing which doesn't require a port.  You can do that using hydra.sendMessage and hydra.makeAPIRequest.  However, if you're sending a message from outside of hydra, as with an incoming web API request from a client on the Internet, then you'll need a tool which can route to one of several hydra enabled services.
NGinx and other load balancers have this support, as does Hydra-Router.  This is more easily done when the hydra-services are running on different machines with their own IP addresses. That way the port address is the same for each service instance of the the same type.
Another popular way of handling all this is to use a Docker Swarm or Kubernetes cluster which has internal support for load balancing requests to containers.
All of this is more of a networking issue rather than a Hydra specific issue, as Hydra has support so that it works in all of the above situations.
I notice based on your posts times that we may be in compatible timezones.  Let me know if you'd like me to setup a zoom conference call to we can speak about the above. I've love to see you up and running. If you'd like to do that send me an email to<EMAIL_ADDRESS>so we can setup a time.  We'll then posts our findings here for future visitors.

Thank you Carlos. I will try setup with Nginx for load balancing and post the updates here..If I'm unable to make it, will send you an email to pitch in conference call. Tons of thanks for your time and answers so far.

Hi Carlos - I'm done. Its up and running with load balanced with NGINX setup..Thank you!!


⚠ AGEG website (prod) has degraded performance

In 00a88d5, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 4030 ms

Resolved: AGEG website (prod) performance has improved in fd293d1.


MB-10086: Add shipment weight to NTS-release

Description
This pull request creates a simple ShipmentWeightInput component (just a single heading, label, and number input text box), and adds it to the ServicesCounselingShipmentForm only when that component is being used for NTS-release shipments. This allows an office user to add and/or edit a shipment's weight (which is represented in the back end by the primeActualWeight field).
Reviewer Notes
Ensure that new component is showing up correctly -- only in the NTS-release view.
Setup
yarn storybook

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Closes MB-10086.


Do you have a document on the "Vue CRUD communication specification"?

Great work.
Do you have a document on the "Vue CRUD communication specification"?

Hi, thanks.
I don't quite understand what you mean. Unfortunately, I'm just starting my adventure with GitHub and contributing to Open Source tools. Do you mean a communication tool such as Skype or Discord? Or maybe about the forum related to the project? I will be grateful if you advise me.


nasin-nanpa-4.0.0-Helvetica.otf renders 'm' as nothing

No idea why.
I'm considering trying to solve this by just changing nasin nanpa's name fields to match Helvetica's, rather than using FontForge to merge the fonts.

Well I was able to generate a nn-helv without the missing m, but I still have no idea why it was ever happening. All I did was enable "Compact Encoding (hide unused glyphs)", then generated the otf file, and it was fine. Not sure if this was just general FontForge weirdness, or if enabling that setting is actually what solved it. This fix will be included in version 4.0.1.


Display tree in alphabetical order

Is it possible to add a flag to sort alphabetically when using the tree feature? Currently files are first shown and only then inner directories. I would like to replicate tree default behaviour:

I thought that maybe lsd --tree --group-dirs=none would achieve such behaviour but it didn't.

Hi @petobens,
Thanks for the report. It seems that the --group-dirs=none doesn't have the correct behavior indeed.


[CORE] Update vendors 2019-11-17

tinymce 5.0.16 -> 5.1.1 https://www.tiny.cloud/get-tiny/self-hosted/
select2 4.0.10 -> 4.0.12 https://github.com/select2/select2/releases
select2 bootstrap4 theme 1.4.2 https://github.com/ttskch/select2-bootstrap4-theme/tree/master/dist
jstz build 2015-11-04 -> 2019-09-09 https://github.com/pellepim/jstimezonedetect
lineawesome 1.1.0 -> 1.2.1 https://github.com/icons8/line-awesome (Too many breaking changes)

bootstrap 4.3.1 -> 4.4.0 https://github.com/twbs/bootstrap/releases

Updated

TinyMCE v5.1.3
select2 v4.0.12

I need to check Bootstrap 4.4 and the icon themes, maybe for Bludit v4.

Updated:

jstz build 2015-11-04 -> 2019-09-09 https://github.com/pellepim/jstimezonedetect

The only missing is line-awesome and I feel is not a priority, are just icons. I going to close the issue to clean up a little, but feel free to create a new one with new updates of vendors.


Problem reading vendor extensions applied of properties defined by an external reference

I noticed this problem a while ago. I did report it to the penapi-generator/issues/14327 . At the time it wasn't a big issue and I suspected it was a problem with the swagger parser. Recently, it has become something of a more important issue for us, so I looked into how to fix it. Fixing it is not too difficult, what's difficult is deciding on if it should be fixed, or not.
All the following will refer to version 2.1.14 when mentioning line numbers. I figure this is fixed, where the current development branch is not.
I will explain the problem  in the context of the parser. Starting with the schema definition, where we deserialize this simple model description
test_message:
   type: object
   properties:
      primitive:
         type: integer
         x-format: u32
         x-maitred-index: 0
      in_line:
         type: object
         x-maitred-index: 1
         properties:
            id:
               description: Vehical registation ID.
               type: string
               x-maitred-index: 0
         description: in_line description
      ref_obj:
         $ref: '#/ref_obj'
         x-maitred-index: 2
         description: ref_obj external description

ref_obj:
   type: object
   required:
      - id
   description: ref_obj internal description
   properties:
      id:
         description: Vehical registation ID.
         type: string
         x-maitred-index: 0

When you are deserializing the parameters and you call io.swagger.v3.parser.util.OpenAPIDserializer.getSchema() an the node generated for the last parameter ref_obj, you send in the following JsonNode
{
    "description":"ref_obj external description",
    "x-maitred-index":2,
    "$ref":"net_path.yaml#/net_path"
}

The getSchema() method recognizes this as a ref and on line 2760 enters an if statement to load the referenced object. All that's fine, but once that's done, on line 2799 the loaded schema is returned. This means that the description is ignored, which it probably should be, but also the vendor extension field is ignored. This I would argue is a a error. The vendor extension on this parameter are important, at least to us.
Among the other fields that are also ignored are:

default
nullable

These might also be important.
A quick hack fix is easy. Just replace line 2799 with
Map<String, Object> extensions = getExtensions(node);
if (extensions != null && extensions.size() > 0) {
	schema.setExtensions(extensions);
}
return schema;

I am sure there is a much better solution, but this would fix our problem.
The big issue is that this change would have an effect on openapi-generator, and probably all other generators. Specifically, it would add to the vendor generated fields. In the case of openapi-generator it will only append extra fields, so in all probability no one would notice it, but there is no guarantee that other generators would not be effected more drastically.
These parameter fields on reference are a complex issue. Some of them can be ignored, but some of them have meanings that make sense, so I think they should be included.
So, is this a change that can be made, or not?

Maybe it's possible that schema also has extensions, using schema.setExtensions(extensions); should not overwrite them.

What I am proposing does not effect the assignment relative to the original call to setExtensions when loading the referenced object. It refers to the interaction between the values assigned to a property and the values the object got from its original assignment from setExtensions.
It is true it is possible this problem exists in some systems that use the parser,  hence why I posed this as a question. But to clarify my original post,  this issue does not apply to the openapigen generator. The issue is, are other generators effected? Also, as I point out below the issue is not  so simple, so I think you will have to do something about how this  operation works to clarify the specification regardless.
Let me explain why I don't think it effects openapigen.
What the parser does is build a map with a reference value to represent  this entry. Specifically, the property value contains a reference to the object, not a copy of it. At the moment it does not add the property external definitions, so you just have the reference. Then this generated structures is used by the openapigen generator.
The generator processes this structures into a JSON like map and then performs the following steps:

it loops over the property fields of an object, loading the  values of the property, in the case of interest this is only the reference link.

Then, in this case, it recognizes it as a referenced object.

It then goes and loads the referenced object, including the values set by setExtensions and in the case where the external  values of the referenced object exist it does the following:

if the original property object does have a null external object (true because it was not loaded), it creates an empty map.
now it loops over the external value of the ref object and uses the Java  Map.put() method to insert these external value into the local external map.

The put method overwrites existing values (it does return the replaced value, but its not used here).
The result is that the external values of referenced object would  overwrite any added from the property values, so from the users point of view all they would see is more entries added to what existed before, but  no change to the original values. So its not going to effect openapigen. I think this is what should happen, but also with a warning telling the user that the value has been overwritten.
HOWEVER, this only happens if the implementation follows this pattern,  if there is a generator out there that depends on the parser, that  implements this interaction in the reverse order, it would see the  effect you mentioned. Possible even worse, Its behavior would be  undefined, if the implementation had been more carefully built and the  overwritten values are checked and processes. But having implementation  dependent behavior is a set of problems that will blow up in the future  regardless, so its worth looking into it anyway.
The problem is not really a bug, or an implementation issue, its a  design specification issue. I originally thought that the best thing to  do would be to add an entry in the external map the used a key of  something like"REF_VALUES", or anything that does not start withx-,  and then make the value be the external value map of the referenced  object. However this gets into issues about the protocol specification, potential problems with validation functions  and the generators would need to learn to use the new feature, so its a  none starter of a solution. I only mention this to demonstrate its a specification problem.
In addition, its not necessary to change the parser to do this, because  the objects the parser generates already have the implicate reference  definition built in, its just missing the other data. The reason I think you will have to do something  about it, is that, as happens a lot, there is a difference in the  behavior depending on if the object is a reference, or if it is defined  "in line". If you look at my original posting,  openapi-generator/issues/14327, you can see it works correctly (from my perspective) for the in line  case, and not for the referenced case. It should at least be consistent.
In addition as I pointed out there are other fields that are also  ignored. Specifically, "default" and "nullable". I think default  will only be a problem for externally defined Enums, but "nullable"  is an issue for general objects as well.
As I said I think the problem here is in the specification definition,  and there is a problem if you change things, and one if you don't. The  formal behavior of the specification, as it stands is ambiguous and that has to be changed somehow.
For my part I think the current behavior is just wrong from a logical perspective.

Hello, thanks @mwilby for the bug report.
I've been encountering the same issue, any update or workaround on this matter ?


Add an editor config file

Add an editor config file: http://EditorConfig.org
This will make it easier for people to contribute

Do you mean add the editor config file into this project? Maybe I can help.

Yes, into this project, to be used while working on ElixirLS


SDN-4194: Add a script to find OVN perf problems

(Unreasonably long poll intervals, but not only). It may be used to analyse a live cluster or a must-gather to find disturbing signs of OVN overload that are logged as WARNING messages.

/lgtm


add cargs

Adds a package used for parsing flags easily in C and C++

error C2220: the following warning is treated as an error

use io.replace or add_patches to disable this flags, https://github.com/likle/cargs/blob/f422cb2ceb40aeff440a63813a7d6bc8817dea81/cmake/EnableWarnings.cmake#L6

fixed and pushed, thanks!


MB-7448: add the ability to deploy to niprnet ecs services

Description
This adds the ability to deploy mirrors of the app and app-client-tls ECS services to support NIPRNet ingress and eventually egress once that is ready.
Specifically, this creates the environment variable files, relevant deploy scripts, and adds task definitions to the ecs-deploy command.
We cannot test this in experimental because there is no BCAP connection to the experimental account. We have been treating staging as an experimental environment for everything related to the BCAP connection. I ran these scripts against the staging environment and they worked. The services started and stabilized in the RUNNING status in ECS. Health checks for the NIPRNet ALB (8443/tcp) and NLB (9443/tcp) indicate the services are both healthy.
Setup
make build_tools
aws-vault exec transcom-gov-milmove-stg -- ./scripts/deploy-app-niprnet stg
aws-vault exec transcom-gov-milmove-stg -- ./scripts/deploy-app-client-tls-niprnet stg

Code Review Verification Steps

[ ] If the change is risky, it has been tested in experimental before merging.
[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)

References

Jira story for this change

Fails

:no_entry_sign:
Yarn Audit Issues Found:
high - Regular Expression Denial of Service
Package trim-newlines
Patched in >=3.0.1 <4.0.0 || >=4.0.1
Dependency of node-sass
Path node-sass > meow > trim-newlines
More info https://npmjs.com/advisories/1753
high - Denial of Service
Package css-what
Patched in >=5.0.1
Dependency of react-scripts
Path react-scripts > @svgr/webpack > @svgr/plugin-svgo > svgo > css-select > css-what
More info https://npmjs.com/advisories/1754
high - Denial of Service
Package css-what
Patched in >=5.0.1
Dependency of react-scripts
Path react-scripts > optimize-css-assets-webpack-plugin > cssnano > cssnano-preset-default > postcss-svgo > svgo > css-select > css-what
More info https://npmjs.com/advisories/1754
high - Denial of Service
Package css-what
Patched in >=5.0.1
Dependency of react-scripts
Path react-scripts > html-webpack-plugin > pretty-error > renderkid > css-select > css-what
More info https://npmjs.com/advisories/1754
4 high vulnerabilities and 0 critical vulnerabilities found

Messages

:book:
:link: MB-7448

  Generated by :no_entry_sign: dangerJS against cd45bd95a82147487fe88c1eb03afad98181cf67


[PROTOCOL] Scrape IDSA protocol lists (clinical)

https://www.idsociety.org/public-health/COVID-19-Resource-Center/covid19-expanded-resource-center/#Clinical
Note: this is going to be a bit of a rabbit hole, since each link opens up another resource to scrape / standardize, and may induce duplication with other resources already scraped.

Where possible, add in Protocol Categories and Protocol Settings (probably all/mostly clinical?)


Crash in case of virtual type conflicts

Just pass the following piece of code to any Nit engine and see it crash:
class A
    type X: Numeric
    fun foo: X do return 42
end
class B
super A
    redef type X: Int
end
class C
    super A
    redef type X: Float
end
class D
    super B
    super C
end
var y = new D
print y.foo

@egagnon: So, we can not refer to this case while discussing about virtual types support for now. :smile:

Virtual types checks are not that nice in fact. The crash here is explicit by an abort since the conflict is detected. What is missing is 1. how to present the; 2. how to return the chain of methods and gracefully try to continue.
This is hard to do because the detection is quite deep in the semantic analysis.
In order to solve the bug I need to think what is the best error message to present to the programmer.
Is the bug in class D ? is the bug in y.foo ?

One way we can deal with this case for now is to force the developer to redefine X in D (in this particular example, no definition of D::X will be accepted in practice). However, if one day, we implement intersection types (which would solve a lot of problem for what I’m working on), we would be able to implicitly define D::X as the intersection of Int and Float and the error would be caught by the implicitly cast on the return value of foo.


erro sintax

https://github.com/PauloSergio-0/System-Car/blame/bcaad3f33af9a4d4b6cbb30d6ba642a2027d4aef/src/modules/System_Cadastro/Cadastro_car.py#L147
in the code the dataframe filtering was closed with "]" instead of ")"

Is don't error syntax


bug: It seems that using this plugin causes some display errors in the .sh files.

Check the following:

[X] I have checked the behavior after setting "vscode-neovim.neovimClean" in VSCode settings.
[X] I have read the vscode-neovim docs.
[x] I have searched existing issues.

Neovim version (nvim -v)
v0.9.5
Operating system/version
windows 11 23H2 with WSL <IP_ADDRESS>
Describe the bug
When extension is activated, there are some strange things in .sh files.
I have a simple .sh file
pyinstaller -F ../saveyourbug.py    \
--add-data "../config.yml;."        \
--distpath "../build/"              \
--specpath "../temp/"               \
--workpath "../temp/"               \
--optimize 2                        \
--icon="../resource/logo.icns"      \
--log-level="ERROR"                 \
--clean                             \

but if I open it with vscode neovim extension, it will be like this

But if I try to copy those things and paste, it will come back to the origin content
Steps To Reproduce

enable VSCode Neovim extension
reload window
open a .sh file in windows

Expected Behavior
plain text

poor english sry😭

Confirmed

Debugged for 2 hours.  Nvim combined the dashes at the beginning of these lines with the status column, causing them to be mistakenly removed


Adds API to create JMeter threads on the fly

Adds a method in a thread group to add new threads dynamically.
Feature can be use to adjust thread count:

in case of low load
dynamic scenarii
...

To prevent code duplication, thread "make"/register/start has been moved to a new method, and a field has been added to track group number.

Sample script:
JMeterExampleScript.zip

New commit is done => Makes API thread creation returns the new JMeterThread instance

Thanks for contribution.
I reviewed the patch. It doesn't break current code but I think there may be an issue in :
       Map.Entry<JMeterThread, Thread> entry = allThreads.entrySet().iterator().next();
This may not work in case call happens when all threads of current thread have ended. allThreads will be empty.
I think it might be safer to store ListenerNotifier and TestTree in ThreadGroup so that they can be used in creation without using allThreads.
I'll create a bugzilla and attach updated patch to it.

See:
https://bz.apache.org/bugzilla/show_bug.cgi?id=60530


red message on crypto when we reload the token screen

When testing on iphone I had this error message a lot.
Apparently it happened in two occasions:

when downloading some pictures (maybe token picture, not sure) url of the download is empty,
When updating contracts: see following picture


Cannot detect imported path does not exist

xx.aa is a non-existent path, ruff indeed cannot resolve to this path, but did not give any hint.
I don't know the specific rules of grammar detection, but I think this is a feature that can be supported?
Thanks!

A minimal code snippet that reproduces the bug.

from xx.aa import Config
Config()

The command you invoked (e.g., ruff /path/to/file.py --fix), ideally including the --isolated flag.

ruff check .

The current Ruff settings (any relevant sections from your pyproject.toml).
none

The current Ruff version (ruff --version).
ruff 0.0.282

We wrote a Python module resolver that can support this in theory, but the details are hard to get right, since we might need to know about your virtualenv to know if the module is third-party.

On second thought, we could just detect whether this package (xx) is first-party, and then see if the module exists within the first-party package, and raise if not.


x/tools/go/packages: go1.10 behavior emits incorrect Imports

$ git sync e104082
$ go build ./go/packages/gopackages
$ PATH=$HOME/go1.10/bin ./gopackages -test fmt
Go package "fmt": ...
Go package "fmt [fmt.test]": ...
Go package "fmt_test [fmt.test]":
        package fmt_test
        has no exported type info
        file /usr/local/google/home/adonovan/go1.10/src/fmt/example_test.go
        file /usr/local/google/home/adonovan/go1.10/src/fmt/fmt_test.go
        file /usr/local/google/home/adonovan/go1.10/src/fmt/scan_test.go
        file /usr/local/google/home/adonovan/go1.10/src/fmt/stringer_test.go
        import "bufio"
        import "bytes"
        import "errors"
        import "fmt [fmt.test]"  # should say "fmt" => "fmt [fmt.test]"
        ...
        import "testing" # should say: import "testing" => "testing [fmt.test]"
        import "testing/iotest" # should say: import "testing/iotest" => "testing/iotest [fmt.test]"
        ...

Given that we only support Go 1.14 and Go 1.13 currently, going to close this issue. Even if gopls commits to supporting older Go versions, we won't go back before 1.11.


Update minimum DPF core requirement to 0.8.0

Update the minimum requirement for ansys-dpf-core to 0.8.0 in
minimum_requirements.txt, to match the pyproject.toml definition
>=0.8,<1.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (aa0651e) 91.73% compared to head (567e0de) 91.73%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #374   +/-   ##
=======================================
  Coverage   91.73%   91.73%
=======================================
  Files          47       47
  Lines        3038     3038
=======================================
  Hits         2787     2787
  Misses        251      251

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


select language

please add a menu to select language

So right now the /collection API endpoint I'm using for scryfall only returns the english cards.  What do you think of showing localized previews on hover?  Like the card name would be english in the list, but when you hover over it you'd see it in your preferred language (if it's available)

Would love to see that!


Stuck installing framework

tried on kali linux wsl and debian both return same issue, it just starts installing, never completes
![](file:///C:/Users/k97al.DESKTOP-PB3AS4M/AppData/Local/Packages/Microsoft.ScreenSketch_8wekyb3d8bbwe/TempState/Screenshot%202020-12-26%20130143.png)

multiple people have claimed the same thing, so that makes it not invalid so, hope you get it fixed! great project!