RecursionError at query-signature

I seem to be running into a bug but I'm not really sure on why this fails at the point where it fails.
RecursionError at /django_query_profiler/7c1ed3533d2a4c91aa3736812dffe345/QUERY_SIGNATURE
maximum recursion depth exceeded while calling a Python object

This is in the file: django_query_profiler/templates/django_query_profiler_level_query_signature.html line 177.
If I expand the if statement with {% if query_signature and query_signature.analysis.visible_in_ui %},
this works fine, just does not show the analysis name.

We are using the Moz parser to parse the sql, and this particular line goes to that code path.  Looks like there is a Recursion error in that code path
Recently fixed this issue with try-except block here
Looks like we should not only catch ParseException, but Exception base class
The idea with this code is to help with recommendations - whether its a simple case of applying select_related or prefetch_related;  Its better to show no recommendations than fail
Let me try to take a stab at it today.

Can you try with the latest version (0..8).  I believe it should fix this issue;

Can you try with the latest version (0..8). I believe it should fix this issue;

Yes this works 🙂 Thanks for fixing this that fast 🙏🏻


build.rs: print rustc stderr if exit status != 0

I was trying to run benchmarks locally with rustc-perf and found that many of them failed to build with a message from libc's build.rs "Failed to get rustc version." I made this change locally to help debug, and I think it would be generally useful. In my case it quickly revealed that rustc was failing to find libLLVM and so rustc --version was emitting nothing on stdout.
I think this may have been part of what was intended with #3000 and might help debug https://github.com/rust-lang/crater/issues/663.

@bors r+

:hourglass: Testing commit 88e27b8df27d0aaa1ade562b01224cdfd6ead8d8 with merge 557f644b3ce773ea4f8e0b089d988d81c735206a...

:broken_heart: Test failed - checks-actions

@bors r+

:pushpin: Commit 52808cef25e8404488e5e8dec9330c90b59df6d0 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit 52808cef25e8404488e5e8dec9330c90b59df6d0 with merge f3d760791ec35a056de43495dfc03873d150ef83...

:broken_heart: Test failed - checks-actions

Note that you saw the build on PRs, not bors. The bors' build is here: https://cirrus-ci.com/build/5602401026048000
The FreeBSD failure happens on your PR because your branch is old, it's fixed on master so rebasing should resolve it.
The actual error is here (you can find the link from bors comment): https://github.com/rust-lang/libc/actions/runs/4601182601/jobs/8128728585#step:4:237
It's a network failure which means we should: @bors retry

:hourglass: Testing commit 52808cef25e8404488e5e8dec9330c90b59df6d0 with merge b9f037292b144a416a454ca457c6d6f94662a59e...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing b9f037292b144a416a454ca457c6d6f94662a59e to master...


BUG: Adding new columns doesn't work ass expected

Modin version checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest released version of Modin.

[X] I have confirmed this bug exists on the main branch of Modin. (In order to do this you can follow this guide.)

Reproducible Example
import modin.pandas as pd
test = pd.DataFrame({"a":[1,2,4],"b":[0,17,5]})
|    |   a |   b |
|---:|----:|----:|
|  0 |   1 |   0 |
|  1 |   2 |  17 |
|  2 |   4 |   5 |

test.loc[test["a"].isin([1,4]),"c"] = np.nan
|    |   a |   b |
|---:|----:|----:|
|  0 |   1 | nan |
|  1 |   2 |  17 |
|  2 |   4 | nan |

Issue Description
Hello,
I don't know if I m doing it wrong but when I try to create a new columns with a filter condition using loc it doesn't work.
It return me the dataframe with the last column changed.
Expected Behavior
I expected something like this :

a
b
c

0
1
0
nan

1
2
17
nan

2
4
5
nan

Error Logs

Replace this line with the error backtrace (if applicable).

 The root cause is actually a known issue, #3764: assigning to a nonexistent column with loc should add that column to the dataframe. In this case column "c" doesn't exist in test, but assigning to it with test.loc[test["a"].isin([1,4]),"c"] = np.nan should create c and assign np.nan to it. I've marked this issue as a duplicate of #3764, so I'll close this one.
#3764 is a really important issue for Modin and we are actively working on a fix, #3765. Please follow progress on the thread for #3764.


Compilation failed

I want to build winafl, but Compilation failed
winafl and DynamoRIO are in the same directory
cmake DynamoRIO
PS D:\dynamorio\build64> cmake -G"Visual Studio 17 2022" -A x64 ..
-- Selecting Windows SDK version 10.0.22000.0 to target Windows 10.0.19043.
-- The C compiler identification is MSVC 19.36.32537.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: D:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- The CXX compiler identification is MSVC 19.36.32537.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: D:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Version number: 9.93.19549
-- Found assembler: D:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.36.32532/bin/Hostx64/x64/ml64.exe
-- Found message compiler: D:/Windows Kits/10/bin/10.0.22000.0/x64/mc.exe
-- Found Perl: D:/Strawberry/perl/bin/perl.exe (found version "5.32.1")
-- Performing Test cxx17_available
-- Performing Test cxx17_available - Failed
-- Targeting subsystem 5.02
-- PT related libraries only supported on Linux x86_64
-- For dbghelp, choosing among:
-- Looking for MFC
-- Looking for MFC - not found
-- MFC not found: disabling DRstats
-- Could NOT find ZLIB (missing: ZLIB_LIBRARY) (found version "1.2.11")
-- Performing Test implicit_fallthrough_avail
-- Performing Test implicit_fallthrough_avail - Failed
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
CMake Warning at CMakeLists.txt:1863 (message):
  doxygen not found: documentation will NOT be built

*** NOT building documentation *** (must re-enable BUILD_DOCS manually if fix up the component paths)
-- WARNING: vera++ not found: disabling code style checks
-- WARNING: Could not find Qt 5: DrGUI will NOT be built
--   Point CMake variable Qt5Widgets_DIR at the Qt5WidgetsConfig.cmake directory
-- Configuring done
-- Generating done
-- Build files have been written to: D:/dynamorio/build64

PS D:\dynamorio\build64> cmake --build . --config RelWithDebInfo
MSBuild version 17.6.3+07e294721 for .NET Framework

  Generating ../events.h
  MC: Compiling D:/dynamorio/core/win32/events.mc
............................
.....................

cmake winafl
PS D:\winafl\build64> cmake -G"Visual Studio 17 2022" -A x64 .. -DDynamoRIO_DIR=D:\dynamorio\build64\bin64\cmake
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 2.8.12 will be removed from a future version of
  CMake.
  CMake that the project does not need compatibility with older versions.

-- Selecting Windows SDK version 10.0.22000.0 to target Windows 10.0.19043.
  By not providing "FindDynamoRIO.cmake" in CMAKE_MODULE_PATH this project
  has asked CMake to find a package configuration file provided by
  "DynamoRIO", but CMake did not find one.

  Could not find a package configuration file provided by "DynamoRIO" with
  any of the following names:

    DynamoRIOConfig.cmake
    dynamorio-config.cmake

  "DynamoRIO_DIR" to a directory containing one of the above files.  If
  "DynamoRIO" provides a separate development package or SDK, be sure it has
  been installed.

CMake Error at CMakeLists.txt:153 (message):
  DynamoRIO package required to build

-- Configuring incomplete, errors occurred!
See also "D:/winafl/build64/CMakeFiles/CMakeOutput.log".

I want to ask what's wrong

Can you verify that the directory D:\dynamorio\build64\bin64\cmake exists and that it contains DynamoRIOConfig.cmake

I searched in /bin64, but did not find cmake
how do i do it

I find it. D:\dynamorio\build64\cmake And winafl also compiled successfully
Think you!


Adding draft code for Layer_version resource

Description of changes:
Adding draft code for Layer_version resource. This contains hook code and e2e tests.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all

/test all


Built lib returns warnings and not render anything when imported in a Vue-cli project

Version
5.0.0-alpha.4
Environment info
System:
OS: Linux 5.8 Ubuntu 20.10 (Groovy Gorilla)
CPU: (8) x64 Intel(R) Core(TM) i7-10510U CPU @ 1.80GHz
Binaries:
Node: 12.18.2 - /usr/bin/node
Yarn: 1.22.10 - ~/.npm-packages/bin/yarn
npm: 6.14.8 - /usr/bin/npm
Browsers:
Chrome: 88.0.4324.150
Firefox: 85.0.1 npmPackages:
@vue/babel-helper-vue-jsx-merge-props: 1.2.1
@vue/babel-helper-vue-transform-on: 1.0.2
@vue/babel-plugin-jsx: 1.0.3
@vue/babel-plugin-transform-vue-jsx: 1.2.1
@vue/babel-preset-app: 5.0.0-alpha.4
@vue/babel-preset-jsx: 1.2.4
@vue/babel-sugar-composition-api-inject-h: 1.2.1
@vue/babel-sugar-composition-api-render-instance: 1.2.4
@vue/babel-sugar-functional-vue: 1.2.2
@vue/babel-sugar-inject-h: 1.2.2
@vue/babel-sugar-v-model: 1.2.3
@vue/babel-sugar-v-on: 1.2.3
@vue/cli-overlay: 5.0.0-alpha.4
@vue/cli-plugin-babel: ~5.0.0-alpha.4 => 5.0.0-alpha.4
@vue/cli-plugin-eslint: ~5.0.0-alpha.4 => 5.0.0-alpha.4
@vue/cli-plugin-router: 5.0.0-alpha.4
@vue/cli-plugin-vuex: 5.0.0-alpha.4
@vue/cli-service: ~5.0.0-alpha.4 => 5.0.0-alpha.4
@vue/cli-shared-utils: 5.0.0-alpha.4
@vue/compiler-core: 3.0.5
@vue/compiler-dom: 3.0.5
@vue/compiler-sfc: ^3.0.4 => 3.0.5
@vue/compiler-ssr: 3.0.5
@vue/component-compiler-utils: 3.2.0
@vue/preload-webpack-plugin: 1.1.2
@vue/reactivity: 3.0.5
@vue/runtime-core: 3.0.5
@vue/runtime-dom: 3.0.5
@vue/shared: 3.0.5
@vue/web-component-wrapper: 1.3.0
eslint-plugin-vue: ^7.2.0 => 7.6.0
vue: ^3.0.4 => 3.0.5
vue-eslint-parser: 7.5.0
vue-hot-reload-api: 2.3.4
vue-loader: 16.1.2 (15.9.6)
vue-style-loader: 4.1.2
vue-template-es2015-compiler: 1.9.1
npmGlobalPackages:
@vue/cli: 5.0.0-alpha.4

Steps to reproduce
Building a vite app and defining the mode lib with the standards of the doc
Building a vue-cli app with Vue3 presets
Including the lib to the app
What is expected?
The lib components appear on the screen
What is actually happening?
Nothing appears, just some console warnings :
runtime-core.esm-bundler.js?7572:38 [Vue warn]: resolveDirective can only be used in render() or setup().
runtime-core.esm-bundler.js?7572:38 [Vue warn]: withDirectives can only be used inside render functions.
[Vue warn]: Invalid VNode type: Symbol(Fragment) (symbol)
at
at

The lib works perfectly with a second Vite built project as App
This lib depends on Primevue component framework. When I don't use it, it works everywhere.
I tried with Vue-cli 4.5.11 and 5.x, same behaviour

Thanks for submitting this issue! Due to our limited time, we ask you to include a reproduction link to a minimal full reproduction of your problem (for example in a GitHub repository) so we can find what is causing the issue. Thank you for your understanding!

I found my issue, on my tests in local, I had a node_modules with my lib.


Update setup.go to skip alias tests for COS images

COS has ip_alias=False in our instance_config file, we can skip the alias tests on COS.

/ok-to-test


Elasticsearch Client Plugin Contribution

Title: Elasticsearch Client Plugin Contribution
Link: https://github.com/bbossgroups/pinpoint/tree/elasticsearch-567/plugins/elasticsearch-bboss
Target: Elasticsearch Rest Client BBoss
Supported Version: pinpoint 1.9.0
Description: This agent plugin is worked with elasticsearch rest client BBoss,BBoss is a best Java Highlevel Rest client for ElasticSearch.
ServiceTypes: [9201, "ElasticsearchBBoss"],[9202, "ElasticsearchBBossExecutor"]
Annotations: [971, "es.args"],[972, "es.url"],[973, "es.dsl"],[974, "es.action"],[975, "es.responseHandle"],[976, "es.version"]
Configurations: List of configuration keys and description the plugin adds.
profiler.elasticsearchbboss.enabled=true //enable or disable this plugin, default true.
profiler.elasticsearchbboss.recordResult=false //record or disable record method return value ,default false.
profiler.elasticsearchbboss.recordArgs=true //record or disable record method args,default true.
profiler.elasticsearchbboss.recordDsl=true //record or disable record dsl,default true.
profiler.elasticsearchbboss.recordESVersion=true //record or disable record ES Version infomation,default true.
profiler.elasticsearchbboss.recordResponseHandlerClass=false //record or disable record response handle class ,default false.
profiler.elasticsearchbboss.maxDslSize=50000 //set the max length dsl script,over this size dsl will be substring.

[#5411] Add histogram for Alias #5551
@RoySRose

Hello, @yin-bp
Before I take a look at the code.
May I ask you why commit [#4558] Fix SpanChunk handle of Collector is included in this PR?
Commits from other contributors will not be included if you try to Rebase instead of Merge.

Hello, @yin-bp
Before I take a look at the code.
May I ask you why commit [#4558] Fix SpanChunk handle of Collector is included in this PR?
Commits from other contributors will not be included if you try to Rebase instead of Merge.
@RoySRose
Sorry，I don't know the rule before. I'll submit another PR soon.


make socks5 http request after creating server

https://stackoverflow.com/questions/59357669/python-pproxy-make-sock5-http-request-after-creating-proxy-using-ssh

Can you try the CLI pproxy instead of using API?


Add a remote certificate

I have a certificate in S3 and I have an error in the PKPass constructor. I guess it is normal because file_exists does not work with remote files but is there a solution to add a remote certificate ?

True, I guess file_exists doesn't work, so the library would require code changes, unless of course you would be okay with temporarily storing the certificate locally and then removing it after use, which I guess file_get_contents internally would also do when using a remote file URL.
I would be okay with accepting a PR to change the library so that it accepts remote URLs for the certificate.

Fixed in #83.


Test PodDisruptionBudgets

build-depends: https://github.com/dci-labs/example-cnf-config/pull/96

This is to fix pod-disruption-budget tnf test
Config provided handles the case where we only have one replica deployed, where we don't want to have a node that is impossible to drain

pod-disruption-budget test is now passing: https://www.distributed-ci.io/jobs/41f63c56-44e3-43c8-85c0-3535751e02f6/tests/5eb2475b-4b87-4a35-9fae-a71026c9cd84 , this is ready for review

pod-disruption-budget test is now passing: https://www.distributed-ci.io/jobs/41f63c56-44e3-43c8-85c0-3535751e02f6/tests/5eb2475b-4b87-4a35-9fae-a71026c9cd84 , this is ready for review

Just one question: I checked the job log and I see the task Create PodDisruptionBudget for controller-manager pods that corresponds to change in https://github.com/dci-labs/example-cnf-config/pull/96. But I do not see where the Create PodDisruptionBudget for * tasks of this PR#61 are executed. Are those not tracked during the job execution? Maybe it's in another place :thinking:

pod-disruption-budget test is now passing: https://www.distributed-ci.io/jobs/41f63c56-44e3-43c8-85c0-3535751e02f6/tests/5eb2475b-4b87-4a35-9fae-a71026c9cd84 , this is ready for review

Just one question: I checked the job log and I see the task Create PodDisruptionBudget for controller-manager pods that corresponds to change in dci-labs/example-cnf-config#96. But I do not see where the Create PodDisruptionBudget for * tasks of this PR#61 are executed. Are those not tracked during the job execution? Maybe it's in another place 🤔

You're right, in the job, we can only see the task from example-cnf-config, which is directly included in the hooks because, for the controller-manager pods, we need to do it from an external point of view.
For the other pods (LoadBalancer, TestPMD and TRex), you can see how they're created in the controller-manager logs, e.g. testpmd-lb-operator-controller-manager-5578fb6c98-s2fkw.log
:
...
--------------------------- Ansible Task StdOut -------------------------------

TASK [loadbalancer : Create PodDisruptionBudget for LoadBalancer pod] **********
task path: /opt/ansible/roles/loadbalancer/tasks/main.yml:67

-------------------------------------------------------------------------------
{"level":"info","ts":"2024-03-04T18:55:59Z","logger":"proxy","msg":"Cache miss: policy/v1, Kind=PodDisruptionBudget, example-cnf/lb-app-pdb-min"}
{"level":"info","ts":"2024-03-04T18:55:59Z","logger":"proxy","msg":"Injecting owner reference"}
{"level":"info","ts":"2024-03-04T18:55:59Z","logger":"proxy","msg":"Watching child resource","kind":"policy/v1, Kind=PodDisruptionBudget","enqueue_kind":"examplecnf.openshift.io/v1, Kind=LoadBalancer"}
{"level":"info","ts":"2024-03-04T18:55:59Z","msg":"Starting EventSource","controller":"loadbalancer-controller","source":"kind source: *unstructured.Unstructured"}
{"level":"info","ts":"2024-03-04T18:55:59Z","logger":"proxy","msg":"Cache miss: policy/v1, Kind=PodDisruptionBudget, example-cnf/lb-app-pdb-max"}
{"level":"info","ts":"2024-03-04T18:55:59Z","logger":"proxy","msg":"Injecting owner reference"}
{"level":"info","ts":"2024-03-04T18:55:59Z","logger":"runner","msg":"Ansible-runner exited successfully","job":"2591084995821321510","name":"lb","namespace":"example-cnf"}

----- Ansible Task Status Event StdOut (examplecnf.openshift.io/v1, Kind=LoadBalancer, lb/example-cnf) -----

...


[Request-For-External] Table 37 "Sales Line" - FindOrCreateRecordByNo

Hi, could you please change the local FindOrCreateRecordByNo procedure of table 37 (Sales Line) to a global one?
    local procedure FindOrCreateRecordByNo(SourceNo: Code[20]): Code[20]
    var
        Item: Record Item;
        FindRecordManagement: Codeunit "Find Record Management";
        FoundNo: Text;
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeFindOrCreateRecordByNo(Rec, xRec, CurrFieldNo, IsHandled);
        if IsHandled then
            exit("No.");

        GetSalesSetup;

        if Type = Type::Item then begin
            if Item.TryGetItemNoOpenCardWithView(
                 FoundNo, SourceNo, SalesSetup."Create Item from Item No.", true, SalesSetup."Create Item from Item No.", '')
            then
                exit(CopyStr(FoundNo, 1, MaxStrLen("No.")))
        end else
            exit(FindRecordManagement.FindNoFromTypedValue(Type.AsInteger(), "No.", not "System-Created Entry"));

        exit(SourceNo);
    end;

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


fix to conjugacy in detecting indexed node names

Fixes #590.
Letting tests run.

Tests passed.  Merging, and closing #590.  Should be good to go @perrydv


Minor suggestion: Firebase Crashlytics plugin: Crashlytics() is exposed

Crashlytics() is exposed, and there is Crashlytics.instance
If i understand correctly, the constructor should be private so that devs do not try to use it on accident, and of course it doesn't achieve the right result.

@neiljaywarner please file the issue to https://github.com/FirebaseExtended/flutterfire/issues and close this issue.

Hi @neiljaywarner
for what I can see the issue is related to a FlutterFire plugin,
please open the issue in the dedicated github.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
thank you


GROOVY-10223: support conversion of Optional to array/collection

Adds support for assignment of java.util.Optional to arrays or collections (without explicit .stream().toArray() or .stream().toList() or .asCollection()) -- and by extension an iterator() method and for-each iteration.  The array or collection is empty if optional is empty or has one element if valued.
Object[] array = optional
List<Object> list = optional
Iterator<Object> iter = optional.iterator()
for (value in optional) { /*...*/ } // ... executed only if value is present

I didn't think it was necessary to extend this to OptionalInt, OptionalLong and OptionalDouble.
Does not support spread (aka def list = [*optional] or m(*optional)).
https://issues.apache.org/jira/browse/GROOVY-10223

And this kind of thing is not supported without getAt(int) or getAt(Integer):
def opt = Optional.of('')
def (String x) = opt
assert x == ''

@paulk-asert I agree, it seems the extension method Optional<T> collect(Optional<S>,Closure<T>) is not that valuable.  If deprecated, it would not resolve as an extension, which would then cause List<T> collect(Object,Closure<T>) to be selected instead.  I doubt any code that makes use of this is prepared for a list return value.

@paulk-asert I agree, it seems the extension method Optional<T> collect(Optional<S>,Closure<T>) is not that valuable. If deprecated, it would not resolve as an extension, which would then cause List<T> collect(Object,Closure<T>) to be selected instead. I doubt any code that makes use of this is prepared for a list return value.

Yes, it would be a breaking change. We'd deprecate it now in Groovy 4 and remove in Groovy 5 with explanation in release notes. Folks wanting to prepare could potentially test with -Dgroovy.extension.disable=collect but that isn't very selective.
We can explain the behavior either way, I just think the explanation is much simpler if it behaves like all the other fallback variants.


TCP_KEEPIDLE is not available on maOS, this allows for local developm…

Description
TCP_KEEPIDLE is not available on maOS, this allows for local development on it without issues
Verification
Executed athenad on macos successfully

Thanks for the PR, however I'm going to close this since we don't have anything in CI testing this. We may revisit macOS support once we get the lines and dependencies down some more.


Cannot Use Visual Studio 2017 Add Controller Feature

If you open the project file in Visual Studio right click the controllers folder and chose add new Controller, you will get an error that states "No executable found matching command dotnet-aspnet-codegenerator".
In order to fix this issue, the following lines need to be added to the CSProj file:
    <ItemGroup>
        <DotNetCliToolReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Tools" Version="1.0.0" />
    </ItemGroup>

Stack Overflow Question for Reference:
http://stackoverflow.com/questions/41450148/no-executables-found-matching-command-dotnet-aspnet-codegenerator

Thanks very much for raising this point. I did some investigation to track down what's going on.
The templates here are trying to be consistent with the default mvc template, which doesn't include the Microsoft.VisualStudio.Web.CodeGeneration.Tools package by default either. I'm not totally sure why it doesn't, but it doesn't, and we want to stay consistent with that.
However, when I click "Add Controller", I don't get the error you describe. Instead VS gives a prompt like this:

... which sorts out the package dependencies.
I'm not sure why you're not getting that prompt. Perhaps you just need to update to a newer version of VS2017, but in any case it's a separate question.
Since we're trying to be consistent with the mvc template, and since VS2017 has a built-in system for setting up this dependency (albeit not working in your case, but that's a separate issue), I'll close this. Hope that seems reasonable, and thanks again for bringing this up!
For anyone else who gets the error (rather than the VS prompt that you should get), please see @jkruer01's manual workaround above.

Thanks @SteveSandersonMS!


How to map a Cosmos UDF to an entity property

I am wondering whether there is an attribute or a fluent API method that would allow me to tell EF Core to map an entity property to a result of a UDF invoked with that document as a parameter so that a context of the collection could be used to compute the value of the property from other documents in the collection.
Specifically, something like this:
SELECT {{ firstName: item.FirstName, lastName: item.LastName, test: udf.test(item) }} FROM AppDbContext item
As opposed to something like this which I assume the Cosmos provider already generates:
SELECT {{ firstName: item.FirstName, lastName: item.LastName, test: item.Test }} FROM AppDbContext item
Further technical details
EF Core version: Microsoft.EntityFrameworkCore.Cosmos 2.2.4
Database Provider: Microsoft.EntityFrameworkCore.Cosmos
Operating system: Windows 10
IDE: VS Code

This is in the master list for Cosmos in issue #12086. Keeping this open to track the specific work, now referenced from the master list.

I am adding infrastructure support for this in query pipeline.


Extending OpenClinicaODMFunctions and OpenClinicaSoapWebservice

Extending OpenClinicaODMFunctions:

A new class method is introduced for the classes to be able to add new items/groups/forms/events/subjects to the ODM structure
The ocODMtoXML function is now able to add Form Status attribute to the XML

OpenClinicaSoapWebservice

The class is now able to create new subject requests for studies configured to collect year of birth only.

Thanks very much for these suggestions, I have incorporated them and done a general clean up of the existing code. I've created a new repository for this client at: https://github.com/lindsay-stevens-kirby/openclinica_webservices_php.


dhcp redirecting client to tftpd running on other machine in lan

Hi,
now luci supports only using tftp server using dnsmadq running on openwrt itself.
However there is an uci solution:
uci add_list dhcp.@dnsmasq[0].dhcp_boot='pxelinux.0,<name of external tftp pxe server>,<IP of external tftp pxe server'
But if you change something in dhpc via luci above dirty-hack solution is overwritten.

Just put pxelinux.0,<name of external tftp pxe server>,<IP of external tftp pxe server into the Network boot image field in Network -> DHCP and DNS -> TFTP Settings.


Automated cherry pick of #8609

Cherry pick of #8609 on cloud.
/cc  @hmhealey

/release-note-none


Tutorial Link is not working

https://developer.yuneec.com/documentation/120/Live-Video-Streaming
=> The website encountered an unexpected error. Please try again later.

@petergerten thanks for the feedback, we're checking this.

@petergerten the docs are working again. Thanks again for the note.


how to config lowdelayP(IPPP)

Current default configure is random access. But I need low delay P.

There is low delay B configured in the cfg/experimental directory (its IBBBBBBB). You can adapt it to produce P frames only. You can also copy over the old 4-frame LDP pseudo GOP configuration from VTM, VVenC understands the semantics.
Still, to run LD or AI configs with VVenC, you need to use the expert encoder and please mind that the configurations are experimental and not officially supported as of now.

Thanks a lot. I modified cfg/experimental/lowdelay_fast.cfg and it works for me.


Using vue-cli with a custom registry or behind proxy.

What problem does this feature solve?
Hi,
I am trying to use vue-cli in my company, but i just discovered it make a request to internet in the file @vue/cli/lib/util/getPackageVersion.js.
I'm behind a proxy and we use Nexus OSS as registry to use NPM.
Is it possible to get the registry configured in my .npmrc?
What does the proposed API look like?
const { request } = require("@vue/cli-shared-utils");
const fs = require("fs");
const join = require("path").join;
module.exports = async function getPackageVersion(id, range = "") {
const registry = (await require("./shouldUseTaobao")())
? https://registry.npm.taobao.org
: (await getRegistry());
let result;
try {
result = await request.get(
${registry}/${encodeURIComponent(id).replace(/^%40/, "@")}/${range}
);
} catch (e) {
return e;
}
return result;
};
async function getRegistry() {
const execa = require("execa");
const registry = (await execa("npm", ["config", "get", "registry"])).stdout;
return registry || https://registry.npmjs.org;
}

vue create -r
There's already an option for this, see the docs or run vue create --help

In v4 the logic has been fixed, now it is:

if the useTaobaoRegistry config is true, then use the taobao registry (in case novice users in China haven't manually set the registry before).
then fall back to the result of npm config get registry (which can be set in ~/.npmrc) (or yarn config get registry, or pnpm, depending on user's default package manager).

So I think this issue could be considered fixed.


Add funds date defaulting to previous day

When you use add funds there is a date field, which should default to the current day. It's defaulting to the previous day for me.

This could be due to the date line, but this feature should always reference local time.

This probably affects the confirm contribution modal as well; we're using a function named getCurrentDateInUTC that returns the UTC date rather than the one using the local timezone. Should be a simple fix.


Spi stall fix2

Reverts changes to core state variables made in #7377

No more "actual" or "prestall" state

Still correctly solves errors noted in #7281 with the following changes:

Adds signal command_ready_int to internally track non-stall-condition values for command_ready_o
Internal signal new_command now depends on command_ready_int, not command_ready_o
command_ready_o is masked to always output 0 during a stall condition.
All registers and output signals are now enabled only when !stall
The name of the state register has been renamed to simply "state_d/q"

This results in a far simpler scheme for managing the FSM, with fewer coverage points to worry about.  The previous version, though functional, left some uncertainty as to how to establish coverage points for the "actual" and "prestall" states.

Removes corresponding TODOs from the documentation

Signed-off-by: Martin Lueker-Boden<EMAIL_ADDRESS>Note to Reviewers: In order to make the review easier this PR is currently organized into three commits:

Reversion of #7377 (unchanged, no real need to review)
Renaming spi_host_st_d/spi_host_st_q to state_d/state_q
Actual alternative bug fixes to handle #7281 in a more simple fashion

All three will be squashed before merging.

Note: this has also been simulated against Rasmus' draft testbench (which is available as a snapshot here)  to confirm that this still satisfies #7281.

Thanks @tjaychen!


Server Sent Event (SSE) helper

I would be nice to have a server sent event helper that passes control to a handler.
The sent event helper must be able to detect when the client drops/closes the connection, then drops the handler for that connection and not give an error, stating that a message was trying to be sent to a client connection that does not exist anymore.
A simple example using the native Deno http can be found at https://crux.land/61HZ4a

to detect the client connection is lost or close the browser tab, maybe with clearInterval and send callback to server.
here, my simple code
import { NHttp } from<EMAIL_ADDRESS>
const app = new NHttp();

const html = `<!doctype html>
<title>SSE Demo</title>

Events will appear here:

`;

class SSEResponse extends Response {
    constructor(data: any, opts: ResponseInit & {
        timer?: number,
        onClose: (err?: any) => void,
        onComplete: () => void,
        maxData?: number
    }) {
        let timer = opts.timer || 1000;
        opts.headers = new Headers({
            "Connection": "Keep-Alive",
            "Content-Type": "text/event-stream",
            "Cache-Control": "no-cache",
            "Keep-Alive": `timeout=${Number.MAX_SAFE_INTEGER}`,
        });
        let id = -1;
        let interval = 0;
        const body = new ReadableStream({
            start(controller) {
                interval = setInterval(() => {
                    id += 1;
                    controller.enqueue(
                        new TextEncoder().encode(
                            `event: message\nid: ${id}\ndata: ${JSON.stringify(data)}\n\n`,
                        ),
                    );
                    if (id === opts.maxData) {
                        opts.onComplete();
                        clearInterval(interval);
                    }
                }, timer);
            },
            cancel(error) {
                opts.onClose(error);
                clearInterval(interval);
            },
        });
        super(body, opts);
    }
}
app.get("/sse", ({ respondWith }) => {
    const onClose = (error: any) => {
        console.log(error.message);
    }
    const onComplete = () => {
        console.log('Complete Task');
    }
    const maxData = 10;
    const data = { name: 'sample' };
    respondWith(new SSEResponse(data, { onClose, onComplete, maxData }));
});
app.get("/html",  ({ respondWith }) => {
    respondWith(new Response(html, {
        headers: new Headers({
            'content-type': 'text/html'
        })
    }));
});

app.listen(3000);

please correct if I wrong

This works great. Thank you.

You're welcome


Add nil properties to outbound JSON

Include nil properties on the outbound JSON. Otherwise updating some value to nil won't be noticed, at least for some APIs.

Sorry I was going to wait for nit to be fixed. Did I miss that?

Ok, sorry! I've fixed the indentation.

Thanks!

Thank you!


somalifuscatorv2 full deobfuscation

https://www.youtube.com/watch?v=AhkB1R-yG_c

https://www.youtube.com/watch?v=AhkB1R-yG_c

That's only for symbol deobfuscating. Let's see you do the entire thing.

Let's see it deobf V3 lmao

https://www.youtube.com/watch?v=AhkB1R-yG_c

That's only for symbol deobfuscating. Let's see you do the entire thing.

Some of the obfuscation method's are not in the free version.

Let's see it deobf V3 lmao

If you don't mind giving me a obfuscated SomalifuscatorV3 file, I'll gladly try and see if I can deobfuscate it.

https://www.youtube.com/watch?v=GNdxYgCnmys
My deobfuscator still work's with somalifuscator v2 (max settings). Also I am not using smart screen since it doesn't really work but it seem's not really that important.

Let's see it deobf V3 lmao

If you don't mind giving me a obfuscated SomalifuscatorV3 file, I'll gladly try and see if I can deobfuscate it.

Why video is hidden.


[WFLY-18718] Correct line endings in merged licenses.xml file

Issue: https://issues.redhat.com/browse/WFLY-18718

Thanks @spyrkob


Support for reordering of independent operations

This is work package 5 in our 2020 roadmap.
Goal
Given two consecutive row-wise operations operating on disjoint columns, the user should be able to swap them, by drag and drop. This would not change the effects of the workflow: the state of the table would be the same after the two operations.
Background
This feature is not very useful by itself to users: currently, people rarely care about how tidy their history looks like. In some cases it should allow users to undo an operation that is buried in the history and keep the ones after, by dragging it down. This is probably a marginal use case.
However, this feature is a crucial first step towards:

executing independent operations in parallel (such as fetching URLs and reconciling another column at the same time)
multi-user support, to let users work on independent parts of the project simultaneously

Work plan
This relies on #2252 as a prerequisite.
Then, we need to add some backend logic to:

Check if two row-wise column-based operations can be swapped;
Perform the swap, by generating the new Change objects;
Potentially update the project state if the history cursor was between the two operations exchanged.
This should all be exposed in a new command.

Frontend-side, we should add some light jQuery code to handle the drag and drop in the history. We might want to signal in some ways that operations can be swapped, although this should become clearer with graphical visualization (#2253).

Can we write down currently what the criteria rule would be for the Check that the operations could be swapped?

Yes: the second operation should not depend on any column that the first one writes to.

Hmm, just thinking off the cuff here, but I wonder if the Pregal API might help in some fashion directly in this issue.  Or generally as a property graph and using parts of Spark's GraphX.

I have changed my mind about this: I don't see it as a meaningful first step towards concurrent operations. As it stands, it does not strike me as matching a user need either. I have found myself in the situation of realizing that I should have done a particular operation earlier, but it does not seem like a good UX to do the operation later and then use some history reordering feature to put it back into place: instead, a way to insert an operation earlier in the history would be more useful.
For instance, one thing I would find more useful would be that if I try to apply a new operation while having undone some operations before, I am given a choice between erasing the future operations (which what happens currently) and inserting the operation without discarding the future operations, when that is possible (which could still mean deleting some future entries or re-running some long-running future operations).

@wetneb So is the better way to:

a rollback to a history point
insert some operation
replay the history forwards from after that

Yes exactly, I think that would be a more common use case.

So is the better way to:

a rollback to a history point
insert some operation
replay the history forwards from after that

I think (from my experience) the last step needs to be:

SELECTIVELY replay the history forwards from after that

i.e. decide which of the steps are replayed and which are not.
The reason for this is usually it's not just that a step was missed (although that sometimes happens) but that a section of the process was wrong in retrospect, and the need is to do a different set of steps at that stage of the process (rather than just adding in additional steps as Thad's description suggests to me)

Interesting, so you'd prompt the user to select which of the future steps should be retained? Perhaps somewhat in the spirit of the current "Extract" dialog, with checkboxes for each future operation maybe?

Yes exactly that. It's not exactly a common occurrence but what I've had to do in the past is:

Extract the steps I need to preserve and save the JSON to a file
Step back in the history before whatever issue I had
Correct (or just skip) the processing I need to do at that stage
Apply the steps I extracted earlier

Agree.  Almost like PAUSING [ || ] the future steps until I am ready to PLAY [ > ] them again.

@ostephens we are planning a similar sort of functionality not for inserting, but deleting operations while preserving the future ones. As part of that, we plan that OpenRefine would be able to indicate which future operations can be kept, which ones will be discarded and which ones will need recomputing.
For instance, if you are inserting an operation that modifies a column, a future operation that reconciles that column would need re-computing (since the data it runs on was modified), and OpenRefine would automatically trigger that re-computation.
Similarly, if you delete an operation that creates a new column, any operation that reads data in that column would be dropped (and transitively so: if any future operation depend on those newly-dropped operations, they would get dropped too).
With this knowledge in mind, do you think we should still give the user the option to select which of the remaining operations should be kept? Or is it sufficient to keep all the operations that can be kept based on columnar dependencies between operations?
Since we are planning to add support for deleting an history entry, I wonder if your interest in selecting which future operations to retain could be satisfied by:

retaining as many operations as we can when the user inserts an operation
let them delete any future operations that are not needed later on, using the dedicated button

I think that should make for simpler UIs since we wouldn't need to prompt the user for as much information as they insert an operation.
@cooperzoe has been working on mock-ups on this UI, so once we are ready to share them it might make things clearer.

Thanks @wetneb. As I noted above this sort of workflow isn't a common occurrence for me so it's hard to be confident about what will/won't be appropriate (it is probably over a year since I last needed to do this type of operation). However, my initial feeling is that what you are describing would be sufficient, at least for the vast majority of situations


Solution plugins

[x] moved solution steps plugin from edtr-io

[x] added possibility

[ ]  smart convert: convert two-column layout plugins to solution steps and explanations

What's the state of this PR? CI is failing


USB flash drive apparently mounted but the mount point does not exist

$ date ; uptime ; freebsd-version ; uname -a
Mon  5 Sep 2016 10:30:46 BST
10:30a.m.  up  1:12, 2 users, load averages: 0.57, 0.91, 0.79
12.0-CURRENT
FreeBSD momh167-gjp4-hpelitebook850g2-trueos.university.brighton.ac.uk 12.0-CURRENT FreeBSD 12.0-CURRENT #1 ec1c56b(drm-next-4.7): Thu Aug 25 23:03:05 UTC 2016     root@gauntlet:/usr/obj/usr/src/sys/GENERIC  amd64
$ mount | grep msdosfs
/dev/da1s1 on /media/DOLLY (msdosfs, local)
$ cd /media/DOLLY
cd: /media/DOLLY: No such file or directory
$

Limited workaround:
sudo mount at /mnt so the result is
/dev/da1s1 on /mnt (msdosfs, local)
However: when mounted in that way, the user can not remove files from the file system – should that be a separate bug report?

Output from gpart show:
$ date ; uptime ; freebsd-version ; uname -a
Fri  9 Sep 2016 13:49:34 BST
 1:49p.m.  up  4:52, 3 users, load averages: 0.85, 0.79, 0.75
12.0-CURRENT
FreeBSD momh167-gjp4-hpelitebook850g2-trueos.university.brighton.ac.uk 12.0-CURRENT FreeBSD 12.0-CURRENT #1 ec1c56b(drm-next-4.7): Thu Aug 25 23:03:05 UTC 2016     root@gauntlet:/usr/obj/usr/src/sys/GENERIC  amd64
$ gpart show /dev/da2
=>      1  7814303  da2  MBR  (3.7G)
        1        1       - free -  (512B)
        2  7814302    1  fat32  (3.7G)

This almost sounds like its mounted to "/media/DOLLY ", notice the extra whitespace? Wonder if thats how the label is setup? Either way, if it says its mounted there, it should be, otherwise we have a FreeBSD mount bug.


Clicking on an alias in a message causes "Cannot rejoin an empty room" every time

This only happens on riot-android. I have v0.8.1 (google play) installed.
I can type /join #theroom:domainc.om, but tapping the alias in a message always gives me "Cannot rejoin an empty room".

Another victim

Also getting this on clicking a room in a pill.

I installed version v0.8.1 of Riot-Android and I can not reproduce the bug. In any case, I managed to join the room by clicking on its alias in a message, even if I left the room or if I rejected an invitation.
If you are on a server other than matrix.org, I'm afraid that the problem comes from server federation.

I am on a different server, but it has always worked until recently without server changes. Every other client is fine, so there seems to be a bug with how riot-android does the request.

Sorry, I couldn't reproduce this bug even with the pills. Could you create a rageshake please, so that we will can analyze the logs ?

sent (although it took about 15 minutes to send). Should be from @travis:t2l.io

Thanks, the room alias is null in the logs. I will analyze that.

The original error of the join request is No known servers. The code translates it to Cannot rejoin an empty room, which is not helpful.
But knowing this, the bug looks like https://github.com/vector-im/riot-web/issues/4251, which has been fixed by joining by the room alias instead of the room id.
We need to understand why Riot-Android does not use the room alias to join the room.

After analysis, it turns out that this bug is related to https://github.com/vector-im/riot-android/issues/2098. In this case, the code does not go through the right place and that's why the room alias is null.


[windows] Release mode opens non-native file picker.

Debug mode: the file picker is the native (Explorer.exe)
Release mode: the file picker is non-native (swing)
Tested on only one machine:
OS: windows 10
Koltin: 1.8.10
Compose: 1.4.1

nativeDistributions { includeAllModules = true                 /.../ }

My app was crashing with this error A call to chooseDirectoryNative failed: Could not initialize class org.lwjgl.system.MemoryStack. I tried #40 but didn't work. But setting nativeDistributions { includeAllModules = true /.../ } worked.


Move-DbaDbFile fails to move the datafile when attempting on a secondary AG.

Verified issue does not already exist?
Yes
What error did you receive?
Move-DbaDbFile -SqlInstance sqlNode02 -Database myDB -FileType Data -FileDestination E:\SQLData -DeleteAfterMove -Verbose
VERBOSE: [15:02:07][Get-DbaDbFile] Querying database [myDB]
VERBOSE: Performing the operation "Setting database myDB offline" on target "myDB".
VERBOSE: [15:02:07][Set-DbaDbState] Setting database [myDB] to OFFLINE
WARNING: [15:02:07][Edit-DatabaseState] Failed to set 'myDB' to OFFLINE
WARNING: [15:02:07][Move-DbaDbFile] Setting database Offline failed!
VERBOSE: [15:02:07][Test-PSRemoting] Testing sqlNode02
VERBOSE: Performing the operation "Copying file L:\SQLLog\myDB.mdf to E:\SQLData\myDB.mdf using remote PS on
sqlNode02" on target "myDB".
The process cannot access the file because it is being used by another process.
+ CategoryInfo          : InvalidOperation: (:) [Start-BitsTransfer], Exception
+ FullyQualifiedErrorId : StartBitsTransferCOMException,Microsoft.BackgroundIntelligentTransfer.Management.NewBitsTransferCo
mmand
+ PSComputerName        : sqlNode02
VERBOSE: [15:02:30][Move-DbaDbFile] File myDB.mdf was copied successfully
VERBOSE: Performing the operation "Executing ALTER DATABASE query - ALTER DATABASE [myDB] MODIFY FILE (name=myDB, filename='E:\SQLData\myDB.mdf'); " on target "myDB".
WARNING: [15:02:30][Move-DbaDbFile] ERROR: | To use ALTER DATABASE, the database must be in a writable state in which a checkpoint can be executed.
Steps to Reproduce
# provide your command(s) executed pertaining to dbatools
# please include variable values (redacted or fake if needed) for reference

Move-DbaDbFile -SqlInstance sqlNode02 -Database myDB -FileType Data -FileDestination E:\SQLData -DeleteAfterMove -Verbose
Are you running the latest release?
Yes
Other details or mentions
TBH, I am reporting this based on one line in the help, "It can also be used to move database files on an AG secondary."
I am assuming that it means what it means and hence I set out to try it on a test 2017 SQL Cluster on Windows 2016 Server env.
There is the long way: https://www.sqlshack.com/a-walk-through-of-moving-database-file-in-sql-server-always-on-availability-group/
But as I am finding my way around dbatools I was willing to give this a try.
Sure enough it fails as expected when trying todo this without following the proper steps in the link provided above.
Trying to move the datafile on a secondary AG Node will not be possible unless the database can be set to OFFLINE etc.... I do not doubt its current process (if it was on a stand-along server). BUT, I seriously doubt its Availability Group credentials, even if its on Primary Node.
I do not see any checks for the database being part of an AG nor do I see anything referring to checking if its the primary node, etc...
IMHO, the script should bomb out if it encounters any failure to set the database OFFLINE, which it didnt.
Happy to discuss more...
What PowerShell host was used when producing this error
Windows PowerShell ISE (powershell_ise.exe)
PowerShell Host Version
Name                           Value

PSVersion                      5.1.14393.4467
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.4467
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
SQL Server Edition and Build number
Microsoft SQL Server 2017 (RTM-CU24) (KB5001228) - 14.0.3391.2 (X64)
            Apr 28 2021 10:32:18

            Copyright (C) 2017 Microsoft Corporation

            Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2016 Standard 10.0 <X64> (Build 14393: ) (Hypervisor)

Thank you for the report @Littlebobbytable5 - can you try with powershell.exe and let us know if you encounter the same error?

Just did, makes NO difference sadly, and I wasn't expecting any either. Because the source code does not handle Availability Groups. I've gone through the source code and unless I am missing something very obvious there is NO test to see if the DB id participating in AG or if the SQL instance is Primary or not.

Thank you for testing! @ClaudioESSilva - can you please check this out? Was it expected that it would work on the secondary? If not, we can update the docs.

Thanks mate.
PS: https://github.com/sqlcollaborative/dbatools/blob/d2f368169408697d089fab85eda2cc9b1f516d5f/functions/Move-DbaDbFile.ps1#L182
The check here to see if the DB has gone offline just rams through, even when there is exception (see my errorlog above). It should, technically, stop executing the function IMO, but I shall leave that to the author to decide.
Once again, thank you.


AUTH-471: pkg/operator/status - add EvaluationConditionDetected

Cherry-pick of https://github.com/openshift/library-go/pull/1634
What
Add ClusterFleetEvaluationType to status controller.
Why
It is a valid ClusterOperator status, but doesn't have a mechanism to be pulled from Operator's status into ClusterOperator's status.

/lgtm


add ios-specific text-input.json

This PR removes the active state from all text-input components except for iOS, which needs to distinguish between user/system focus.

I've updated this PR to only include the active state for iOS


about the viewer

First of all, thank you for sharing the code and models !
I installed the corresponding dependencies as you suggested，There was an error in executing the show() function in visualization.py:

I find that the problem is self.server=None，

My current version of the related library is as follows:
numpy=1.22.4
itkwidgets=1.0a49
I suspect this problem is a library version issue, do you have any suggestions？

Hi, Jianjun,
You may troubleshoot this issue through two steps:

Check whether the imjoy-jupyterlab-extension has been installed successfully. The ImJoy logo should present as:

Install the customized itkwidgets from https://github.com/henguo/itkwidgets.git, when this is installed, the version of itkwidgets should not be 1.0a49.

Thank you for your advice, I give it a try~

First of all, I installed the official version of itkwidgets (https://github.com/InsightSoftwareConsortium/itkwidgets), "visualization.ipynb" of CT-SAM3D can run.
However, there is an error: "Viewer" object has no attribute "points". I assume that the official version of itkwidgets library does not support such interaction as adding points. I need to install the custom version of itkwidgets library as you suggested, My installation method is : "pip install git+https://github.com/henguo/itkwidgets.git"

Now, I have two questions:
First, with this customized version of the itkwidgets library, the image display interface is gone

Second, the implementation of "pip install numpy = = 1.22.4",  some  library version are not compatible, such as: scipy, zarr, matplotlib, etc. Do you have to use numpy version 1.22.4?

The first issue is that the imjoy-jupyterlab-extension is not activated, as evidenced by the absence of the ImJoy logo in the toolbar. After installing the customized itkwidgets, the imjoy-jupyterlab-extension should still be activated.
The second issue can be resolved by installing a compatible version of numpy in your runtime environment. Version 1.22.4 works for us, but we have not tested other versions.


Add unittest support and fix directory

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

Project coverage is 100.00%. Comparing base (2c23357) to head (350e490).
Report is 1 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##            master       #11   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9        10    +1
  Lines          212       244   +32
=========================================
+ Hits           212       244   +32

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


chore: bdd-nh-1590658749 to 0.0.1

chore: Promote bdd-nh-1590658749 to version 0.0.1

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

Needs approval from an approver in each of these files:

OWNERS

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


Add ProjectName as a property on team member so it can be used further down the pipeline

Add ProjectName as a property on team member so it can be used further down the pipeline.
For example:
Get-Project | Get-Team | Get-TeamMember | select projectName,team,displayName | Sort-Object -Property ProjectName,Team,displayName
This now gives a useful overview of team members in each team project / team in an account.

I plan to publish this weekend. Thanks for the continued support.


Revert inplace merge

From my point of view, using this solution to control UT is a bit tricky. The UT is IoTDBMergeTest.java
If it's just a CI problem, then it's approve here

From my point of view, using this solution to control UT is a bit tricky. The UT is IoTDBMergeTest.java
If it's just a CI problem, then it's approve here

From my point of view, using this solution to control UT is a bit tricky. The UT is IoTDBMergeTest.java
If it's just a CI problem, then it's approve here

From my point of view, using this solution to control UT is a bit tricky. The UT is IoTDBMergeTest.java
If it's just a CI problem, then it's approve here


Resize area overlaps window contents when using custom title bar style on Windows

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.70.0 (user setup)
OS Version: Windows 11 (Windows_NT x64 10.0.22621)

When the window is not in fullscreen mode and the Window: Title Bar Style setting is set to "custom", it causes the resize area of the window to slightly overlap with the contents of the window. This does not happen when it is set to "native". It's a small issue but I sometimes scroll through files by clicking on or dragging the scrollbar and it causes me to accidentally resize the window every so often. I used the Steps Recorder to get the following screenshots.
Title Bar Style set to "custom":

Title Bar Style set to "native" (expected behaviour):

Duplicate of https://github.com/microsoft/vscode/issues/102023


Save header format has changed for 2.0+

In particular, it appears the array of SaveHeaderMods are not being parsed correctly anymore

waiting for update ❤


Vickel: Watch handmadenhahem.com

Vickel requests the watch of the watch_keyword handmadenhahem\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

handmadenhahem\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Nisse Engström in Charcoal HQ


Add docstrings/.rst for query_compiler/parsers of cuDF backend

Query compiler and text parsers for cuDF backend should be documented (docstrings/.rst docs) fill in the gaps:
https://github.com/modin-project/modin/blob/master/docs/flow/modin/backends/index.rst#L49

Hi @prutskov has this issue been resolved?


Support Lab color

The Lab color space is part of CSS color 4, and is implemented in Safari already. There are also some operations, like color distance, nicely defined in Lab space.
It's be great to support the lab() syntax in parsing, and toLab() and toLabString() methods.

Actually landing to all major browsers:


Westlicht Internal clock offset

Hello, I'm having a clock issue with my brand new Performer (not assembled by me but by a professional, SMD already soldered by machine);
I was testing the sequencer by simply sending a gate from the performer to a kick drum and syncing it up by ear with ableton's metronome at the same tempo (125bpm) since Live offers a way to sync up things manually, I've noticed that the performer's clock would go off grid after a few bars and slowly going offset after a bit. I repeted the same test with metronome from ornament and crime and the kick would stay perfectly on grid after minutes. I also repeted the test by sending clock from O_c to the performer and still the clock seems to be unstable...What could this problem be related to?
Thanks a lot in advance.

I've also noticed the clock output can be quite unstable.

I'd guess it's not being smoothed out so well.
The gate outputs are super stable though, so sacrificing one of them to use as a clock usually works.

Actually scratch that, increasing the the output pulse time seems to have fixed it


<EMAIL_ADDRESS>ion-content background can't be set with css background property

Bug Report
Ionic Info
Ionic:

   ionic (Ionic CLI)          : 4.1.2 (C:\Users\{myname}\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework            : @ionic/angular 4.0.0-beta.12
   @angular-devkit/core       : 0.7.4
   @angular-devkit/schematics : 0.7.4
   @angular/cli               : 6.1.5
   @ionic/ng-toolkit          : 1.0.7
   @ionic/schematics-angular  : 1.0.5

Capacitor:

   capacitor (Capacitor CLI) : 1.0.0-beta.8
   @capacitor/core           : 1.0.0-beta.8

System:

   NodeJS : v8.10.0 (C:\Program Files\nodejs\node.exe)
   npm    : 6.2.0
   OS     : Windows 10

Describe the Bug
Background color of ion-content can not be set with background or background-color css property.  To style the background --background property has to be set.
Because of Background color delegation to ion-scroll component the background property of ion-content was changed to --background. Therefore css won't overwrite its value.
Steps to Reproduce
Steps to reproduce the behavior:

Go to any page
Add css class with background property to ion-content
Still displaying white background
set --background as property in css class instead of background
background color is working

Expected Behavior
Background can be styled by overwriting background property on ion-content

Take a look at this #15841
This is not an issue because Ionic uses css variables now.
You can also change the background of ion-content by adding color to it:
<ion-content color="primary">

Thank you for the issue. This is not a bug, but rather the desired behavior for a few reasons.

The inner scroll element needs to have the background in order to work with fullscreen mode.

The ion-content element uses Shadow DOM which means the inner scroll element needs to use CSS variables in order to change its background from the parent ion-content.

For consistency. All elements with the color attribute should be overridden using --color or --background. This is because the color attribute on elements will take precedence over any --color or --background variables set. This is on purpose because if you were to do the following:
ion-content {
 --background: blue;
}

It would be a quick way to set all of the ion-content elements to a blue background, unless they have the color attribute applied. The color attribute overrides the background property directly so if you were to set it like the following example, and if we had the background applied to ion-content instead of the inner scroll element, there would be conflicts in what takes precedence (in our testing the color attribute was not taking priority).
ion-content {
  background: blue;
}

We decided that the colors set by the color attribute should always take higher precedence, because if you don't want those colors then the color attribute shouldn't be used on the element.

I recommend reading through our Theming documentation for more information on setting colors and CSS variables and if you have any feedback for ways to improve our documentation, please create an issue on the ionic-docs repository. Thanks!


Add tests for few json options

Signed-off-by: Niranjan Artal<EMAIL_ADDRESS>This fixes https://github.com/NVIDIA/spark-rapids/issues/4614 and partially addresses https://github.com/NVIDIA/spark-rapids/issues/4612.
Added tests to verify allowNumericLeadingZeros and allowUnquotedControlChars options. Have documented the differences between Spark and plugin.

build


failed to install bpf tools permission denied error: 13

⚠ Don't use issues for support questions ("How does x feature work in Anchor? How to do x in Anchor?"). Use the discord instead https://discord.gg/GVRw4kHB (but don't ping maintainers). ⚠
I am getting error
BPF SDK: /home/sushant/.local/share/solana/install/releases/stable-157aa16c6b8b3f3d965680c87a8703047e475881/solana-release/bin/sdk/bpf
Failed to install bpf-tools: Permission denied (os error 13)
I have followed https://buildspace.so/p/build-solana-web3-app/lessons/LEa3ce13e3-d075-4914-9023-b9820e8f9319. While running the command anchor test, I am getting error. Please help.

https://github.com/project-serum/anchor/issues/1003


Release: 1.4.1

@Kong/team-docs please approve this metadata bump but merge it only once Kong 1.4.1 is released! Thanks!

@Kong/team-docs this is good to go from our end! (no idea what's the cause of the CI failure)


support for break / continue in blocks

[uservoice](https://touchdevelop.uservoice.com/forums/402381-code-the-micro-
bit/suggestions/15725448-add-the-ability-to-break-from-for-loops)

[ ] add "break" and "continue" block under loops
[ ] support blocks -> JS conversion
[ ] support JS -> blocks conversion
[ ] add help pages

It is not clear to me that this is an improvement. I'd like to have a discussion around criteria for what sort of language constructs we support in blocks. Before we do this stuff, I'd like to discuss simple functions.

I don't see lot of value in this either. This seems to be similar to ask for functions in blocks. I am closing this for now as we don't have any immediate plan.


internal/poll: CopyFileRange returns EPERM on CircleCI Docker Host running 4.10.0-40-generic

What version of Go are you using (go version)?

$ go version
go version go1.15 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Docker Host running Ubuntu 18.04 on Linux Kernel 4.10.0-40-generic

This is the Go env of the container trying to build:

go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/circleci/.cache/go-build"
GOENV="/home/circleci/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/circleci/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/circleci/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/circleci/project/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build612039276=/tmp/go-build -gno-record-gcc-switches"

What did you do?

What did you expect to see?
Docker image should have been built correctly, and my binary compiled successfully with Go 1.15.
What did you see instead?
An error like this for every binary in my project:
/usr/local/go/pkg/tool/linux_amd64/link: cannot write /tmp/go-link-518048351/000003.o: write /tmp/go-link-518048351/000003.o: copy_file_range: operation not permitted
/usr/local/go/pkg/tool/linux_amd64/link: cannot write /tmp/go-link-518048351/000035.o: write /tmp/go-link-518048351/000035.o: copy_file_range: operation not permitted
The command '/bin/bash -exo pipefail -c go install -mod=vendor -v ./...' returned a non-zero code: 2

Details from my investigation:
I build my binaries on CircleCI in a Docker container. CircleCI allows the user to pin a specific Docker engine, but they default to a version (17.11.0-ce) that runs on a Docker Host powered by Linux Kernel 4.10.0-40-generic.
Despite the above kernel does support the directive copy_file_range, for some restrictions possibly setup by the vendor, the Go linker gets this error:
usr/local/go/pkg/tool/linux_amd64/link: cannot write /tmp/go-link-518048351/000003.o: write /tmp/go-link-518048351/000003.o: copy_file_range: operation not permitted

This seems to correspond to the EPERM error.
I've filed an issue report with the vendor asking why this is happening and I'm waiting for a confirmation: is this a scenario that we would like to take into account, and fallback as well to the other approach?
I assume the EPERM can be returned also in the case of the user not having access to a file, therefore between the two branches of this switch I guess this scenario would be more similar to the second case where we might want a one-off fallback.
I spawn up a VirtualBox environment and tried to build the same code that breaks on CircleCI on three different kernels and all the builds were successful, proving that the issue I've encountered is probably related to some limitation/sandboxing artificially imposed by the vendor:

4.15.0-45-generic <- the officially LTS supported by Ubuntu 16.04
4.10.0-40-generic <- the one used by CircleCI
4.4.232-0404232-generic<- the latest prior to the introduction of copy_file_range

Unfortunately I can't really reproduce the exact setup as CircleCI has its own internal AMIs/images that are not accessible to me, but I asked for the Kernel details and they told me the Docker Host that breaks my linker/builds runs on 4.10.0-40-generic.
The above has been posted as a comment on https://github.com/golang/go/issues/40731 but for easier traceability I've opened a dedicated issue.

/cc @ianlancetaylor
@thoeni thanks for raising a new issue. Would you be able to run some experiments to try to isolate the problem. The main one I'm thinking is operation not permitted is what is says -- the user running that build isn't able to write to /tmp. Would you be able to rejig your image build process to work under something from https://hub.docker.com/_/golang. It looks like those images are build against a deb based system so there shouldn't be many surprises. I'd be interested in knowing if a different base image produced the same permission errors.

Also paging @tianon who knows more about this stuff than the rest of us put together.

/cc @ianlancetaylor
Would you be able to rejig your image build process to work under something from https://hub.docker.com/_/golang.

I originally started from 1.15.0-alpine3.12 which is where I first encountered the error, and raised the issue with CircleCI: they then asked me to try their own container in order to make sure it was not related to the container, and the error seems to happen regardless.
I will try to create a public project with a simple app and the .circleci configuration that triggers it to allow people to reproduce it on CircleCI, but my build breaks all the time regardless of the docker image used for the build, and the way I resolved it was migrating to a different CircleCI remote docker engine (18.06.0-ce) which runs on Kernel 4.15.0-1014-gcp. This allows the build to work fine on any container I tried.
Hope this helps, but I'll try to create a test project to allow others to reproduce it.

Given the error (and what's already been ruled out 💪), my initial guess would be seccomp or apparmor related -- does CircleCI use a profile for either of those that's more restrictive than Docker's defaults?  Maybe some extra capabilities they drop?

It looks like we're hitting a similar issue like #40731. AFAIK, processes running inside Docker may return EPERM for unimplemented or unsupported syscalls due to its default seccomp policy (we had this case a few times in x/sys/unix already). I've sent https://golang.org/cl/249257 with a possible fix.
/cc @ianlancetaylor

Also, 17.11.0-ce (and Linux 4.10) are both pretty old in terms of containers. 😅

@gopherbot please consider backporting this to 1.15.
The use of copy_file_range is new in 1.15 and this causes failures doing file system operations in containers running on older kernels which do not provide the copy_file_range syscall.

Ahh, so this could also be related to the version of libseccomp that their build of 17.11.0-ce is compiled against. 😅
(See also https://github.com/moby/moby/issues/40734 for a recent example of newer kernel APIs being blocked by default thanks to that.)
If you can run with the equivalent of --security-opt seccomp:unconfined, that would be a good test to confirm.

If you can run with the equivalent of --security-opt seccomp:unconfined, that would be a good test to confirm.

@tianon I can definitely try. Just to clarify, since my build is failing on docker build, can --security-opt seccomp:unconfined be used on build or only on run?

Sorry, got the answer already:
#!/bin/bash -eo pipefail
docker build --security-opt seccomp:unconfined -f Dockerfile -t "${CIRCLE_PROJECT_REPONAME}:${CIRCLE_SHA1}" .

Error response from daemon: The daemon on this platform does not support setting security options on build
ERRO[0000] Can't add file /root/project/go.sum to tar: io: read/write on closed pipe
ERRO[0000] Can't close tar writer: io: read/write on closed pipe

Exited with code exit status 1

It should work on build (definitely does in newer versions) but I'm not 100% certain how far back it goes.

fyi, I followed this advice from https://discuss.circleci.com/t/cgo-docker-build-failing-at-link-stage-operation-not-permitted/37100 and it fixed my similar issue:
      - setup_remote_docker:
            version: 18.09.3


How to allow only trusted domains ?

CORS (Cross-Origin Resource Sharing) Misconfiguration.
The application does not validate cross origin resource sharing requests from another origin based on a
whitelist. Access is granted to any arbitrary domain based on the origin header. Cross origin resource
sharing is a mechanism that allows restricted resources on web page to be requested from another
domain outside the domain for which the resources originated.
https://track-sdk-eu.customer.io/api/v1/customers/
We recommend allowing only selected, trusted domains in the "Access-Control-Allow-Origin" header.
Prefer whitelisting domains over blacklisting or allowing any domain (Do not use * wildcard nor blindly
return the origin header content without any checks.). Also, we recommend to ensure that the CORS
header only list trusted domain especially when credentials are allowed.   How we only allow

Hello, Rao. Thanks for contacting us today.
Where are you seeing this error message? In a web browser? In a development console?
Are you seeing this error after installing our Flutter SDK in your mobile application? If so, what steps have you taken to get this error?
Have a great rest of your day!

This step is from pen testing report.


Add an install spinner

@Raynos

 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.


'zip' is not recognized as an internal or external command, operable program or batch file.

basically what the title says

Hi @serekAlmette,
Sorry for the late response.  the zip command should be installed on your system.
For Ubuntu apt install zip.
Or you can just manually zip the build folder and then change the extension to OPK.
Read more of OPK package: https://overwolf.github.io/start/submit-your-app#how-to-create-an-opk-package


Adds special case for origin of blob to the compatibility URL.

Fixes #7089

/botio-windows preview

/botio-linux preview

but I haven't got the necessary hardware to test that it actually fixes the referenced issue.

I tried using polyfill with blob URL on IE9, and it worked as expected.


GuildMemberRemove broken?

Same issue as #793 but just for the remove event. I've enabled intents and I've allowed the code to use intents..
Not sure what is going on

Any update?

Same issue as #793 but just for the remove event. I've enabled intents and I've allowed the code to use intents.. Not sure what is going on

Are you sure you enabled intents before Opening the session?

Same issue as #793 but just for the remove event. I've enabled intents and I've allowed the code to use intents.. Not sure what is going on

Are you sure you enabled intents before calling Open on the session?

Just tested this and it worked when I kicked a User. Thanks!

Since issue is resolved. I'm going to close this.

@FedorLap2006 I believe that also what fixed my issue, sorry for not closing when I figured it out a long time ago.


Stunts Not Reset on Catch Your Breath use

First off a good adaptation of the Breathless System it will give me a good way to easily play my own Breathless game "Cargo".
I have only come across one issue, when you Catch Your Breath it correctly resets the skills to the default value, however, if you have used your stunt it does not reset your stunt to unused.

Thanks, this should be fixed when 0.2.2 rolls out later today


[Question] Support data type

Hi now I'm using to floor in my flutter application
What't kind of data type can I use in entity?

int -> SQLITE INTEGER?
bool -> INTEGER?
string -> TEXT?
datetime -> TEXT?

And also Uint8List ->BLOB, if you are using the master branch.

The project's README has just been updated with a "supported types" section.

@vitusortner Thank you


File Upload destination, database, and server.

From what I gather, the current file upload facility in Parse Server saves a file to Hosted Parse (the same way it always has been), however work is underway for it to be saved to Amazon S3 (with the gridstore adapter #113 ).
This is while my database is hosted somewhere else (eg mongolab) and my instance of Parse server is hosted in another place (eg heroku).
The cost and workload of having 3 separate hosting paths to maintain is a little extreme for small businesses like mine.
I have seen that Google Datastore is a proposed option for database implementation in #20 .
What I would like to see is perhaps the option or capability of having my File Storage, NoSQL Database and Parse Server all on Google services (Cloud Storage, Datastore, App Engine) for example.
This minimises cost and hassle on my part as a client developer with limited backend experience.
How feasible is this to be implemented within the phasing out year?
Thanks. :)

The default in parse-server is to store the files in GridStore, which is part of your Mongo installation.  See GridStore.js.  S3 has just been added as an adapter.  We welcome other file adapters to support other use cases.  I expect many to be added in the near future.


Fix visualization bug in navbar-lang-selector

It solves #931
Evidence

Hello @92nqb, thank you for your contribution!
For anyone's information:
Currently, there is a open PR (#930) that also fixes #931 by modifying SCSS: see here.
(CC: @cjyabraham)

This solution might be slightly better than mine so I'm fine with it being merged.

@92nqb Thank you for the contribution !
(Thanks to @cjyabraham @jihoon-seo for the PR coordination)
/lgtm


Customize fields in Quotation & Sale Order

Add a field to the Quotation and Sale Order, The options in that field will be set to Thai language.
http://redmine.ecosoft.co.th:10083/issues/5871

No need to use translate field (field should not have the translate icon)

I have made changes to the customization field in Quotation, and Sale Order and added fields Allow Standard Type and Allow Custom Type in Item Doctype.


upload task attachment UI

#REF: #MOBILEAPPS-1438

 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.Ankit Goyal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Loader_5

Loader_5
Design a loader animation like the above image
Create a file "loader_5.html" in the modules/loader folder.
Use simple HTML and CSS and JAVASCRIPT to create the loader animation. You can use the interface development https://kwickerhub.com (This project) or you can write the lines of code yourself.
Step by Step Guide.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "loader_5.html" in the "modules/loader" folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder  ••
"modules/loader/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Make you Pull Request...
Good-luck.

Hello @MaverickDe  , can u please assign this to me ...i am  interested to contribute

@MaverickDe Can you please assign to me

@MaverickDe  can you please assign this to me I am interested


Eagerly launch Singularity tasks for each module build

Executor has been updated to wait for the build to enter the launching state before continuing, but this will reduce the overhead by having the docker container pulled and launched, JVM started, etc. so once the build enters the launching state we can go right away
@jonathanwgoodwin @gchomatas

:ghost: sounds good!


switch tipranks to api

Hello @hristo-mavrodiev! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file yahoostats/evaluator.py:

Line 4:80: E501 line too long (91 > 79 characters)

In the file yahoostats/requests_stats.py:

Line 276:80: E501 line too long (90 > 79 characters)
Line 286:80: E501 line too long (82 > 79 characters)
Line 294:80: E501 line too long (91 > 79 characters)
Line 324:80: E501 line too long (82 > 79 characters)
Line 326:80: E501 line too long (89 > 79 characters)
Line 327:80: E501 line too long (97 > 79 characters)
Line 328:80: E501 line too long (99 > 79 characters)
Line 329:80: E501 line too long (91 > 79 characters)
Line 335:80: E501 line too long (89 > 79 characters)
Line 338:80: E501 line too long (81 > 79 characters)
Line 343:80: E501 line too long (83 > 79 characters)
Line 356:80: E501 line too long (86 > 79 characters)
Line 357:80: E501 line too long (101 > 79 characters)
Line 357:102: W291 trailing whitespace
Line 358:80: E501 line too long (93 > 79 characters)
Line 358:94: W291 trailing whitespace
Line 359:80: E501 line too long (86 > 79 characters)


Updates to Wordle text and yellow highlight

Updated the Wordle text to slightly more friendly interaction.
Changed yellow highlight to not over-highlight letters (don't count dupes of "correct letter in correct spot" as "correct letter in incorrect spot").

Bump, would be good to get this landed sooner than later


Add cancelling status for pipelineruns

Fixes

https://issues.redhat.com/browse/HAC-2644
Description

Adds Cancelling status to the pipeline runs that are already cancelled and still finishing up the current tasks (this could be finally tasks as well). Doing this will prevent the user from cancelling the pipelinerun again and again and provides the actual state of the pipelinerun.
Type of change

[x] Bugfix

Screen shots / Gifs for design review

Before:
https://user-images.githubusercontent.com/9964343/208898535-614c1bca-a0f2-405e-af6f-6ea146961ce4.mp4
After:
https://user-images.githubusercontent.com/9964343/208898548-15296b4a-1f57-403a-a945-129454b383c9.mp4
How to test or reproduce?

Stop/Cancel a running Pipeline, it should mark the pipelinerun status as cancelling and stop/cancel actions in the dropdown should be disabled.

Unit Tests
  pipelineRunStatus
    ✓ should return null if an invalid pipelineruns (2 ms)
    ✓ should return Pending status for pipelinerun status  with type as "Succeeded" & Pending condition (1 ms)
    ✓ should return Running status for pipelinerun status with type as "Succeeded" & status as "Unknown"
    ✓ should return Succeeded status for pipelinerun status with type as "Succeeded" & status as "True"
    ✓ should return failed status for all the failed pipelineruns" (1 ms)
    ✓ should return Cancelled status for pipelinerun status with reason as "StoppedRunFinally" and "CancelledRunFinally" (1 ms)
    ✓ should return Cancelling status for pipelinerun which is cancelled but finishing the current execution (1 ms)
    ✓ should return Skipped status for pipleinerun with reason as "ConditionCheckFailed" (1 ms)
  pipelineRunFilterReducer
    ✓ should return a valid status
    ✓ should return "-" for pipelinerun status with empty conditions (1 ms)
  containerToLogSourceStatus
    ✓ should return waiting status
    ✓ should return restarting status
    ✓ should return running Status (1 ms)
    ✓ should return terminated Status
  getRunStatusColor
    ✓ should return the default case (1 ms)
    ✓ should result the message, pftoken and label colour for all the handled statuses (2 ms)
  pipelineRunStatusToGitOpsStatus
    ✓ should return the default case
    ✓ should return the valid gitops statuses (3 ms)

Browser conformance:

[x] Chrome
[x] Firefox
[x] Safari
[ ] Edge

cc: @christianvogt

/lgtm
/approve


feat: Protogalaxy folding of challenges

We need to fold challenges as well. This means a refactor around how relation parameters are specified
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit c4d8d963 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,444
179,588
716,132

l1_rollup_calldata_gas
222,888
868,196
3,449,468

l1_rollup_execution_gas
841,975
3,595,304
22,204,837

l2_block_processing_time_in_ms
996 (-2%)
3,826 (-1%)
15,020 (-1%)

note_successful_decrypting_time_in_ms
328 (-1%)
1,015
3,852 (-3%)

note_trial_decrypting_time_in_ms
30.0 (-14%)
109 (+1%)
137

l2_block_building_time_in_ms
8,764 (-2%)
35,011 (-2%)
154,782 (-1%)

l2_block_rollup_simulation_time_in_ms
6,479 (-2%)
25,840 (-2%)
102,956 (-2%)

l2_block_public_tx_process_time_in_ms
2,251 (-2%)
9,053 (-2%)
51,119

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
11,448 (+2%)
21,918 (+1%)

note_history_successful_decrypting_time_in_ms
2,366 (+1%)
4,842 (+6%)

note_history_trial_decrypting_time_in_ms
121 (-1%)
146 (-20%)

node_database_size_in_bytes
1,663,859
1,201,644

pxe_database_size_in_bytes
27,188
54,187

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
46.4 (-1%)
61,697
18,841

private-kernel-ordering
23.4 (+1%)
24,233
8,089

base-rollup
849 (-1%)
639,797
811

root-rollup
37.6 (-1%)
4,072
1,097

private-kernel-inner
41.8
81,504
18,841

public-kernel-private-input
47.0 (-1%)
41,455
18,841

public-kernel-non-first-iteration
30.4 (-2%)
41,497
18,841

merge-rollup
0.848
2,592
873

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
8,723
27,508


Update to Kubernetes v1.14.4 with necessary fixes

Update to Kubernetes v1.14.4 with necessary fixes.
Please don't forget, the current CKA, as of now, is on Kubernetes v1.14!

This guide does not follow CKA but it has been updated to 1.15.3.


Implement "Add Open" code fix

(It pops up assertion dialogs in Debug mode, so I recommend to test it in Release mode)

💋

(can you do the same for suggestions?)

@forki How they are presented? Just embedded into error messages as text?

@forki BTW, "Maybe you want ..." part looks unnatural in the code fix dialog :(

@Krzysztof-Cieslak parses the error message in ionide and gives suggestions from it. There is probably a better way for VS.

Regarding automatic open - @Krzysztof-Cieslak is implementing the very same feature for Ionide. can we find a way to share the code between these two. I mean afaik ionide needs it in fsautocomplete. How can we move forward?

We both ported the same my code from VFPT. I added the core logic into VFT-FCS, so it will be available in FCS package eventually.

@dotnet-bot test this please

@nosami Yeah, this Code Fix UI is cool :)

This is fantastic.

The C# equivalent looks like:

I prefer the C# wording, and I like that it offers the ability to add a using, add the full typename and also to generate a new type.  It seems that if we have a fixer, we want one that offers a choice of fixes.
Kevin

I prefer the C# wording, and I like that it offers the ability to add a using, add the full typename and also to generate a new type. It seems that if we have a fixer, we want one that offers a choice of fixes.

About wording, it's the error message which our Diagnostics provider gives, we cannot control it.
This PR does provide two choices: open namespace/module or add full symbol name. About generating a type, I'm not sure it's necessary, I've never even thought about it.

Same for me, I cannot think of a time where that was ever the solution I wanted for that kind of error state

We could discuss what could or should be in the code fix UI in a separate discussion issue.  For now, I think that the two namespace suggestion options are great experience-wise and fix a big problem today.  The others are more along the lines of us looking to achieve experience parity with C#, and I'd rather have that as a part of a broader discussion.

Let's start with the two we have. Can always iterate later to make it more powerful. The whole generate type in C# came as part of VS2010 IIRC for the TDD support it was pushing. I'm not sure how relevant it is for F#, plus you'd need to offer multiple types - Records, DUs and Classes I suppose.

Sounds like a plan
@vasily-kirichenko thanks for this, it's nice
Kevin


newRidgeState is not exported from react-ridge-state with latest version

Hi,
With the latest release 4.2.4 I get this error during build and dev mode. Reverting back and pinning to 4.2.2 and it works again.
Log is from a github action docker build, but local log is the same. Checked over multiple systems as well.
 > [web-builder 6/6] RUN yarn build:
#23 0.440 yarn run v1.22.15
#23 0.488 $ react-scripts build
#23 2.448 Creating an optimized production build...
#23 39.99 Failed to compile.
#23 39.99
#23 39.99 Attempted import error: 'newRidgeState' is not exported from 'react-ridge-state' (imported as 'newRidgeState').
#23 39.99
#23 39.99
#23 40.05 error Command failed with exit code 1.
#23 40.05 info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
------
error: failed to solve: executor failed running [/bin/sh -c yarn build]: exit code: 1
Error: buildx failed with: error: failed to solve: executor failed running [/bin/sh -c yarn build]: exit code: 1

4.2.6 should be uses.

Thanks for getting back. Tried that version and it's still the same issue.
Attempted import error: 'newRidgeState' is not exported from 'react-ridge-state' (imported as 'newRidgeState').
Using it like the docs says, and as before on 4.2.2 with import { newRidgeState } from "react-ridge-state";
There must be something you're missing with how it's built now and the stuff you changed for the build. I don't know much about building modules so can't really help out with that.

Should work now

It added more bytes to bundle though but I think it's too early to use too much modern javascript in final bundle so I compile with tsc now instead of esbuild


Bug in mxnet.contrib.text.utils.count_tokens_from_str

mxnet.contrib.text.utils.count_tokens_from_str(source_str, token_delim=' ', seq_delim='\n', to_lower=False, counter_to_update=None)[source]
the description is
Counts tokens in the specified string.
For token_delim=’’ and seq_delim=’’, a specified string of two sequences of tokens may look like:
token1token2token3<sd>token4token5<sd>

But if one use "*" as a token_delim,executingtext.utils.count_tokens_from_str("*".join(s),"*"), Error occurs:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "d:\program files\python37\lib\site-packages\mxnet\contrib\text\utils.py", line 69, in count_tokens_from_str
    re.split(token_delim + '|' + seq_delim, source_str))
  File "d:\program files\python37\lib\re.py", line 213, in split
    return _compile(pattern, flags).split(string, maxsplit)
  File "d:\program files\python37\lib\re.py", line 286, in _compile
    p = sre_compile.compile(pattern, flags)
  File "d:\program files\python37\lib\sre_compile.py", line 764, in compile
    p = sre_parse.parse(p, flags)
  File "d:\program files\python37\lib\sre_parse.py", line 930, in parse
    p = _parse_sub(source, pattern, flags & SRE_FLAG_VERBOSE, 0)
  File "d:\program files\python37\lib\sre_parse.py", line 426, in _parse_sub
    not nested and not items))
  File "d:\program files\python37\lib\sre_parse.py", line 651, in _parse
    source.tell() - here + len(this))
re.error: nothing to repeat at position 1

if one really wanna regard "*" as a delim they should use "\\*" rather than "*"
more, using "\\" as a token delim will get a unexpect response.

Thank you for submitting the issue! @sandeep-krishnamurthy requesting this be labeled.

Try using "\\" as the the token delim when you want to use '' or '\' as delim

@vandanavk You're right, but the document is wrong.
document just say that:
Counts tokens in the specified string.
For token_delim=’’ and seq_delim=’<sd>’, a specified string of two sequences of tokens may look like:
token1token2token3<sd>token4token5<sd>
But if we use token_delim=’*’ and seq_delim=’\n’, the function will throw an error
many people knows regular expression, but not all the people will translate  should be treated as a regular expression.

The documentation update (https://github.com/apache/incubator-mxnet/pull/11800) has been merged. Can this be closed? @Neutron3529  @sandeep-krishnamurthy

@vandanavk
thanks


Mermaid uses ESM modules now. Update so mermaid loads again.

Since Mermaid has moved to ESM module, the former loading sequence no longer works.
This updates the code generation to use 'import'. The tests are updated and pass. It is deployed  (for now) on NPM as @rwk/typedoc-plugin-mermaid.

LGTM!

@all-contributors please add @BobKerns  for code, bug.


Remove deprecated gulp-util module

The gulp-util module is marked deprecated on npm and as such causes install warnings. It should be super easy to swap out; see here for more information and equivalent functionality. If I have time I will probably do a PR for this one but feel free to take it now since I may not for a while.

Good idea! PR would be lovely too, definitely! 👍

I definitely will PR if I have time! Which I may or may not since I'm in the middle of final exams ¯_(ツ)_/¯

Oh stressful! Well focus on that, and holiday season first!
Good luck with exams and take the time you need!


Allow streamItemReceived to get raw Data [Feature Request]

Feature Request
Allow streamItemReceived to get raw Data as the incoming stream message. I understand this library was made to only allow materialized objects to come in as stream messages but I think there are valid use cases for getting the raw data.
The FAQ's suggest sending a String from the backend so the swift client can get the raw String and convert to Data. However, I don't have control over what the backend sends and additionally I don't think suggestion is an efficient solution for production.
Use Cases

Ability to conditionally decode data. An example is that each incoming message has a type field that tells the client what object to attempt to decode. A workaround is creating a large object with all possible fields as nullable but this is difficult to manage and basically makes you re-create decoding errors if expected fields are null. It's much cleaner if we let Decodable do this for us.
Ability to specify a different decoder for different objects. Some objects may decode decode things differently than others. An example could be some object decode dates in one way while others do not. Currently it does not seem possible to specify a decoder on per message basis. Exposing the raw data allows clients to decide how to process the data.
Ability to decode data however the client wants, currently only JSON decoding is possible

Additional context
I created a simple escape hatch to get the raw data here: https://github.com/aj-bartocci/SignalR-Client-Swift/commit/61e52b02436a6c5a10a0ce01c6b7fd9c3029c9ad
It's not very elegant but it's just a quick fix for myself at this time. It would be great to have the ability to access the raw data with this library. Happy to help out in adding it if wanted.

Thanks for opening the issue but this is not something that I am planning to support.
If you want to use own encoding/decoding you probably could use byte arrays. If you don't have control over the server, hijacking the protocol does not seem to be the right way. If the server sends objects of the same type with properties encoded differently then I don't quite understand how it can work in other strongly typed clients like C# and I would say the problem is on the server side. I also don't think any of the other clients expose this information.
Non-JSON protocol require support on the server side. AFAIU, the only non-JSON  protocol supported by the server is MessagePack. Implementing support for MessagePack should be done by implementing the HubProtocol protocol.


[1.13] Update vendored swarmkit to b5f07ce49c66d2f5feee83998b23d4c905b78155

This fix update swarmkit to b5f07ce49c66d2f5feee83998b23d4c905b78155
The following changes have been included:

Fix missing IPAM options in swarm network mode (docker/swarmkit#1789)

The above PR is related to docker PR #29074 and docker issue #29044.
/cc @aaronlehmann
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

LGTM


Header Menu (Eureka Demo, ...) jumping left and right when each menu button is clicked

If you click "About" the whole header jumps to the right, if you click "Posts" the whole header jumps back to the left, ... Maybe the problem is content width.

@natasak I will check and fix this issue in the next few days, thank you for your feedback!

@natasak  This bug has been fixed in 6638e3a.
You can checkout master to preview it or wait for the next release which may be released in days.
Thank you for your feedback to Eureka!


Onboard connectors

Hi guys.
First of all thanks a lot for great collector distribution.
In contrib collector there is a connector functionality available.
Althougt it is current in development phase it would be great to have it in this distribution.

@cpheps, any way I could help?

@SHaaD94 I've actually got this on my list for today. I think it's simple enough to get it into our next relesae.


IOTSTOOL-1891 Notification retries for failed requests

Added notification mechanism based on Polly
https://github.com/App-vNext/Polly
Now retries for HTTP Status 500 without stopping the notifications up to ~2 mins

Codecov Report

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

@@            Coverage Diff             @@
##           master     #209      +/-   ##
==========================================
- Coverage   25.12%   25.04%   -0.09%
==========================================
  Files          97       97
  Lines        3089     3099      +10
  Branches      456      454       -2
==========================================
  Hits          776      776
- Misses       2277     2287      +10
  Partials       36       36

Impacted Files
Coverage Δ

...edCloudSDK/Connect/Api/ConnectApi.Notifications.cs
0.96% <0%> (-0.11%)
:arrow_down:

Continue to review full report at Codecov.

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