please release a 1.0.5 that updates ms to v2.0.0

debug 1.x is widely depended on still, and its getting flagged over and over for bringing in older versions of ms that had security vulnerabilities reported against it.
Its a false positive, but a whole lot of build chains and users of debug, directly or indirectly, would be very, very, appreciative if you release a 1.x update to debug that uses the same version of ms you use in master, 2.0.0. Its API compatible with the ms used now for 1.x, at least for the humanize call debug makes (the incompatibility - and the vulnerability - is around a parse function that debug doesn't use).
I can't PR the change because I'd need a 1.x branch to target, but its this:
debug ((1.0.4) *%) % git diff
diff --git a/package.json b/package.json
index 4ee4dc5..b348b90 100644
--- a/package.json
+++ b/package.json
@@ -16,7 +16,7 @@
     "Nathan Rajlich<EMAIL_ADDRESS>(http://n8.io)"
   ],
   "dependencies": {
-    "ms": "0.6.2"
+    "ms": "^2.0.0"
   },
   "devDependencies": {
     "browserify": "4.1.6",

Please see v1.0.5.

Thank you!


Testing units are shown repeatedly on Testing Panel

Hello!
I've seen strange behavior in the UI when we have a workspace with multiple folders.
A new tree of tests is fed on the Testing panel for every folder added to the workspace.
For example, there are 12 folders in this workspace:

So, there are 12 trees in Testing Panel, each root with its 12 folders! 🤣

I want to move to the Native adapter, and actually the base code is complete, but just do not have the time to add the tests. I will try to finish that some time soon, and afterwards you can check out again to see if this is an extension or API problem.

An update after a long time: now that this extension is using the native Testing API of VSCode, I will assume all previous issues with unknown/unreproducible errors to be stale. Please feel free to reopen if you can reproduce them clearly with the new version, v0.6.0.

It works, not as I expected, but this may be an issue for the VSCode devs. The unit tests are all there, not repeated, but also not divided by folder :-)


Error in Running the notebook 03_aml_training.ipynb

Dear Farm Vibes - Team,
I successfully ran the notebooks 01_dataset_generation.ipynb and 02_visualize_dataset.ipynb. But I am facing the error in running the 03_aml_training.ipynb. Anyone could assist me in this regard.
Error Log:
Converting CDLMask CRS from EPSG:5070 to EPSG:32611
Converting CDLMask resolution from 30.0 to 10.0
Traceback (most recent call last):
  File "03_aml_training.py", line 210, in <module>
    save_chips_locally(data.train_dataloader(), os.path.join(AML_DATASET_DIR, "train"))
  File "/home/azureuser/farmvibes-ai/notebooks/crop_segmentation/notebook_lib/modules.py", line 31, in save_chips_locally
    batch = next(iter(dataloader))
  File "/home/azureuser/.local/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 628, in __next__
    data = self._next_data()
  File "/home/azureuser/.local/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1333, in _next_data
    return self._process_data(data)
  File "/home/azureuser/.local/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1359, in _process_data
    data.reraise()
  File "/home/azureuser/.local/lib/python3.8/site-packages/torch/_utils.py", line 543, in reraise
    raise exception
AttributeError: Caught AttributeError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/home/azureuser/.local/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 302, in _worker_loop
    data = fetcher.fetch(index)
  File "/home/azureuser/.local/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 58, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/azureuser/.local/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 58, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/azureuser/.local/lib/python3.8/site-packages/torchgeo/datasets/geo.py", line 876, in __getitem__
    samples = [ds[query] for ds in self.datasets]
  File "/home/azureuser/.local/lib/python3.8/site-packages/torchgeo/datasets/geo.py", line 876, in <listcomp>
    samples = [ds[query] for ds in self.datasets]
  File "/home/azureuser/farmvibes-ai/notebooks/crop_segmentation/notebook_lib/datasets.py", line 293, in __getitem__
    sample = super().__getitem__(query)
  File "/home/azureuser/.local/lib/python3.8/site-packages/torchgeo/datasets/geo.py", line 427, in __getitem__
    data = self._merge_files(filepaths, query, self.band_indexes)
AttributeError: 'CDLMask' object has no attribute 'band_indexes'

Hi, @hnlf123. Thank you for using FarmVibes.AI.
I couldn't reproduce the error with the notebook and environment provided in the repo.
The error is coming from torchgeo, so I imagine you are running the notebook with a different version than the one specified by the conda environment (torchgeo==0.3.0).
Please, could you verify if you have created the crop_seg environment and have activated it?
$ conda env create -f ./crop_env.yaml
$ conda activate crop-seg

Hi, @hnlf123, were you able to verify the version of torchgeo? Does the error still persist?

Closing this for now. Please, let us know if the error still persists.


Backlight?

I was wondering if this board really has a backlight or not. The reference image in the store product page mentions IO33 as "Backlight" but below code does not seem to work:
#define BACKLIGHT 33

pinMode(BACKLIGHT, OUTPUT);
digitalWrite(BACKLIGHT, HIGH);

Does this board not have a backlight or am I doing something wrong? Thanks in advance!

There is a connector in order to connect a backlight. But the backlight itself is not included.

Do you know anywhere one might find a compatible one?

Do you know anywhere one might find a compatible one?

It seems that the connector is compatible with the "front-light" GooDisplay (small flat cable). The e-paper display + front-light is available on Aliexpress : https://fr.aliexpress.com/item/1005006277614640.html  (not tested)


improve get result from constant_col when expr is const

Proposed changes
const expr have execute in open function.
so in execute function could get result directly.
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


use the aud from the subject token as the audience in the federated token request

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

@yangminzhu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
d5375f6494c92254fbbad390ce401a76073a5b74
link
/test lint_istio

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

@yangminzhu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
d5375f6494c92254fbbad390ce401a76073a5b74
link
/test lint_istio

gencheck_istio
d5375f6494c92254fbbad390ce401a76073a5b74
link
/test gencheck_istio

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


Introduce new Lite strategies.

Why
Currently, to create a new strategy, a user needs to understand various inner workings of Hummingbot and to maintain a minimum of 5 files:
[strategy]_config_map.py
Details configurations needed for a strategy.
conf_[strategy]_TEMPLATE.yml
Formats a configuration file with comments and version number.
conf_[strategy].yml
A configuration file, created automatically by the 2 files above. A user can change the setting either by using config command or by updating the file manually
start.py
Initialises the strategy, connectors and market_info objects.
[strategy].py
The main strategy file which users need to care about the most.
What
The epic is to introduce new lite strategies which simplifies strategy creation, import and start process. It reduces number of related files needed to just 2 files and provides basic lite strategy examples.
Related Issues
#4716
#4717
#4718

I am working on a Strategy Definition Language for the definition and auto-generation of new strategies.
https://github.com/klpanagi/hbot-strategy-dsl
A code generator is implemented for the generation of all 5 files, based on input strategy models. Furthermore, a CLI is provided which allows for building new strategy using a local repo of hummingbot client.
Currently, I am working on also initialize markets in the auto-generated [strategy].py file.
I might also missing documentation currently, but I will work on it in the next days.


Ubuntu 22.04 horrible, laggy performance

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

- VS Code Version: 1.92
- OS Version: Ubuntu 22.04 (Linux Mint 21.3)
Description:
The performance is horrible. I can only work on it for about 5 minutes before my whole computer freezes from how bad VSCode is. I used to be on windows using the same amount of apps etc and never had this issue. I have also disabled hardware acceleration and use of GPU still VSCode is unusable.
I have also done a performance monitor through the debugger and every time I press 'save' VSCode crashes without fail, so I can't even save that.
Edit: Here is the .dmp file from the crash mentioned above
dmp.zip
Steps to Reproduce:

Run two local servers on your machine, pgAdmin, and a few other apps of your choice in the background
Start typing, scrolling, using extensions in VSCode etc...
Watch your CPU memory usage skyrocket for no apparent reason and your computer become unresponsive

Here is the .dmp file for the crash when I am trying to save the performance monitor file in debug mode

I have solved this. For anyone on Ubuntu who encounters this frustrating issue and may be new to Linux like me, try lowering your swappiness. This controls the tendency for your computer to move processes from your RAM to the disk (disk is slower than RAM) and only tends to pick up when you are using majority of your memory.

Pull up terminal and run cat /proc/sys/vm/swappiness which should show 60
Paste this into terminal xed admin:///etc/sysctl.conf
Scroll to the bottom and paste this line and make sure to save the file afterwards vm.swappiness=10  then reboot your computer

Before I encountered these issues with VSCode I had it set to 30 but 10 seems to work well with my laptop.


Suppliers: Add a new page 'Create new supplier + belongs to camp + enter to the city' [Feature]

As a Gatter, When I'm in the event and I have a supplier that I have to let in, I would like to have a One page that will create-match-enter the supplier to the city.
Please create a new page that will have have 3 functions in one,

Add a supplier to the db
match the new supplier to a camp
enter the supplier to the city

In the end of the page, after clicking 'save' will popup a window with a msg that will warn the gatter that he is about to let the supplier to enter to the city in red
(In Red: אנא שים לב, הינכם עומדים להכניס ספק לתוך העיר)
A link to this page will be in a button name 'צור ספק חדש'
location of the button will be in the bottom of the Gatter-supplier page, see screenshot
https://spark.staging.midburn.org/he/gate/suppliers
Roles: Only 'Admin' and 'Gater' roles will have access to this page

This should be (in theory) a rare flow.
Most (all) suppliers should be created and assigned before the event, since they have the whole thing with the contract they need to sign and all.
Though if needed, this functionality is available.


fcitx-qt5: fix build

Things done

[ ] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Without this change, the install phase fails with the following message:
CMake Error at platforminputcontext/cmake_install.cmake:50 (file):
  file INSTALL cannot copy file
  "/tmp/nix-build-fcitx-qt5-1.0.5.drv-0/fcitx-qt5-1.0.5/build/platforminputcontext/libfcitxplatforminputcontextplugin.so"
  to
  "/nix/store/5ixiz6661qki5a3crlv4q74hc6l8za7g-qtbase-5.5.1/lib/qt5/plugins/platforminputcontexts/libfcitxplatforminputcontextplugin.so".
Call Stack (most recent call first):
  cmake_install.cmake:39 (include)

@ttuegel I am not yet familiar with multiple outputs, is this the right way to address this problem?

Looks good to me!


add node variants to elixir & python

[x] I've run make from the root directory to see all Dockerfiles are created successfully.
[x] I've updated the documentation if necessary.

The elixir community, especially the Phoenix part thereof, heavily relies on
node and other tools written in Javascript.
This PR relates to (and closes) #90 & #58.
we move the ruby & php "specific" code into generate.sh and generate node and
browsers, node & node-browsers images for elixir and python.

Thanks - this looks great 👍

how long until these images will be available on docker hub?

@igalic they should be available as this workflow completes

that work flow is either not available to outside users, or in mobile, or maybe i just have to login, or maybe i should stop checking github on my phone 😅


KOGITO-4094 Avoid to deploy artifacts if author is not kiegroup and no artifacts repository has been given

https://issues.redhat.com/browse/KOGITO-4094
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The (build) Apps check has failed. Please check the logs.

The (build) Apps check has failed. Please check the logs.

The (build) Runtimes check has failed. Please check the logs.

The (build) Runtimes check has failed. Please check the logs.

The (build) Apps check has failed. Please check the logs.

The (build) Apps check has failed. Please check the logs.

The (build) Examples check has failed. Please check the logs.

tested all paths ok

tested all paths ok

The (build) Runtimes check has failed. Please check the logs.

The (build) Runtimes check has failed. Please check the logs.

jenkins rerun runtimes tests


Remove return types from Verify methods

It might be good to move the validation with return types to extension methods.


[BUG]: require("used-pm") does not return function

Description of the bug
https://github.com/mheob/used-pm/blob/f7240436632735fdab258c5fc8e1456c9e2a0db1/README.md?plain=1#L35-L41
When this code is executed, the variable currentPackageManager is expected to be assigned a function.
However, when I tried<EMAIL_ADDRESS>the following values were assigned:

Therefore, it is not possible to execute a code like currentPackageManager().
Instead, I have to write currentPackageManager.default().
Steps To Reproduce

Go into the new directory and create a package.json file with the following contents:
{
  "scripts": {
    "foo": "node bar.cjs"
  }
}

Run npm install<EMAIL_ADDRESS>

Create a bar.cjs file with the following contents:
const currentPackageManager = require('used-pm');
console.log(currentPackageManager());

Run npm run foo - This command should fail with the following error:
$ npm run foo

> foo
> node foo.cjs

.../foo.cjs:2
console.log(currentPackageManager());
            ^

TypeError: currentPackageManager is not a function
    at Object.<anonymous> (.../foo.cjs:2:13)
...

Additional Information
There is no easy fix for this.
If you use export =, you can build the correct CommonJS module.
see: How to make TypeScript generate a CommonJS valid module? - Stack Overflow
However, if you use export =, the following error will be thrown and the ESM module build will fail:
$ tsc -b ./tsconfig.esm.json
src/index.ts:6:1 - error TS1203: Export assignment cannot be used when targeting ECMAScript modules. Consider using 'export default' or another module format instead.

  6 export = (): PackageManager | undefined => {
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  7  const userAgent = process.env.npm_config_user_agent;
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
...
 19  };
    ~~~
 20 };
    ~~

Found 1 error.

The solution I came up with is to build ESM first, then rewrite export default to export = in the src/index.ts file, and finally build CommonJS.
However, there may be a better way.

Note
If you want to rewrite the dist/cjs/index.js file instead of the src/index.ts file, don't forget to rewrite the dist/cjs/index.d.ts file as well. These files should be edited like this:
dist/cjs/index.js
 "use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = () => {
+module.exports = () => {
     const userAgent = process.env.npm_config_user_agent;
     if (!userAgent)
         return;

dist/cjs/index.d.ts
     version: string;
 }
 declare const _default: () => PackageManager | undefined;
-export default _default;
+export = _default;

This bug has been fixed in<EMAIL_ADDRESS>Thank you so much!


Fixing problem that default search did not apply "notTerminated", whi…

…ch used to be the default.

:shipit:


browser result UintArray[5722] OK, nodejs crashes index.js:70:22 with same data

While on my Chrome console, I copy-pasted the base64js.min.js,
called base64.js.toByteArray on my copy-pasted data from file and obtained the beautiful Uint32Array [5722] I expected,
This simple node script crashes with that same data,
var fs = require('fs')
var base64js =  require('base64-js')
fs.readFile('./test', (err, data) => {
  if (err) throw err;
  // var byteArray = base64js.toByteArray(data) // crashes at index.js:70:22
  console.log(base64js.byteLength(data)) // returns successfully 5722
})

I tried every encoding for the file without any change, and the file is successfully parsed by a  C# .NET Convert.FromBase64String(File.ReadAllText( call.
Anyone has an idea?

Pleas apologies the issue, I solved myself the problem with codificating my text file in utf-8 w/BOM and passed 'utf8' as encoding to fs.readFile.


Feature/7640 penrollment deletion

Fixes W-007640: What should we do with the related Affiliation when a Program Enrollment record is deleted?
I have added an additional second custom setting to allow the admin to specify which status to change the related Affiliation to.

Hey @kbromer, let us know if you'd like to merge this, or if I should ask @daveperry. Thanks.


Add RISC-V Support

As discussed here https://github.com/vitejs/vite/discussions/13198 it seems there is no risc-v support for this in https://unofficial-builds.nodejs.org/
I am not sure whats the root cause of this, but it seems this package is possible - even if not as performant - to work in 100% nodejs without native binaries, so it should be possible to add support for risc-v using that.

Can you try the same case in the latest version of Node.js? This should have been fixed in https://github.com/nodejs/node/pull/43612.

Indeed everything is working using node 20


Laravel 5.1+ PHP7 Error - Uncaught ReflectionException: Class log does not exist

This is a shameless duplicate of #11724 (which has been locked, otherwise I'd comment there).
There, it was suggested that you:

Mess with the error handler.

In case it saves anyone else some time, what this means is that you need to edit vendor/laravel/laravel/src/Illuminate/Foundation/Http/kernel.php and replace the contents of  function reportException($e) with dd($e).  You'll now see your error, rather than laravel's failure to log it.
In my case, my MySQL driver config in config/database.php contained a reference to the constant PDO::MYSQL_ATTR_MAX_BUFFER_SIZE which is only available in some builds of PDO.

Reopening for comments.
If I understand correctly:
In PHP 5, these code errors cause fatal PHP errors - and helpful error messages.
In PHP 7, these code errors trigger exceptions, which laravel catches - but fails to handle - hence no helpful error message.

This is a "nofix" atm, sorry. All these issues are caused by user issues each time rather than actual framework bugs though, so it's not critical we fix it.

I definitely agree that it is very annoying that we can't handle errors early in the booting any better.

it is very annoying that we can't handle errors

Not even something along these lines (which works for my, based on very limited testing...)
        try {
            $this->app['Illuminate\Contracts\Debug\ExceptionHandler']->report($e);
        } catch (Exception $e) {
            dd($e);
        }

It's not suitable for production apps though, and could leak information to the user, as well as not showing as an error 500, infact, a 200 OK.

@fisharebest that helped me get to the route of the real problem... in my case, some pieces of cut and paste PHP in an untitled file had somehow saved to config/untitled.php
It would seem to match with other people's experience of having had to remove php functions ( url() etc ) from within config files.

I also came across this error while running a simple artisan command php artisan make list.
@GrahamCampbell what about changing only Console\Kernel.php to
    protected function reportException(Exception $e)
    {
        try {
            $this->app['Illuminate\Contracts\Debug\ExceptionHandler']->report($e);
        } catch (Exception $ex) {
            dd($e);
        }
    }

This way, nothing would be exposed to the user, since only the devs can run console commands. The
Http\Kernel.php would remain the same. In mine opinion this would save a lot of debugging time since those errors usually happen because of having functions in the configuration files instead of giving a message that the log class cannot be found.


Updates on the overrides

https://phabricator.endlessm.com/T32401

Excellent. And checks have passed.

Yeah, I waited for them to pass before bothering a review :)


change database

Hi this is a great user system, I am trying to implement a system when the users are on one database and when the user logs in the part of the information will be where the rest of the data is stored
//something like this
protected function setUser()
{
$account = $this->CI->db->get_where("accounts", array("id" => $this->user->account_id))->row(0);
$_SESSION['db_name'] = $account;
var_dump($_SESSION['db_name']);
    die();

.....................
return true;
}
the var_dump give this
object(stdClass)#27 (6) { ["id"]=> string(1) "1" ["company"]=> string(10) "management" ["database"]=> string(21) "management_qpuv0o2y3r" ["user_limit"]=> string(1) "1" ["created_at"]=> NULL ["updated_at"]=> NULL }

see https://stackoverflow.com/questions/62763008/codeigniter-getting-database-after-login/62860442#62860442
I managed to a temporary solution


Can local APIs such as ollama be supported?

Can local APIs such as ollama be supported?

maybe it could be something more general, like enter a API endpoint, and API key. Then for local ollama you would just enter localhost.  May also work for openrouter that is another popular request, need to look into this.

maybe it could be something more general, like enter a API endpoint, and API key. Then for local ollama you would just enter localhost. May also work for openrouter that is another popular request, need to look into this.

When Ollama encounters cross-origin resource sharing (CORS) issues, it is recommended to use a proxy
Step 1: Set up a local proxy server (for example, http-proxy-middleware is used in Node.js).
Step 2: Set up a proxy to forward requests to Ollama servers.
Code example:
const { createProxyMiddleware } = require('http-proxy-middleware');
app.use('/api', createProxyMiddleware({
target: 'http://localhost:11434',
changeOrigin: true,
}));
Potential side effect: additional latency due to proxy requests.
I personally use Windows and found potential problems, which require complicated settings, but there are not many problems when using a proxy server.
ps: I used to modify the server CORS configuration and Google's CORS Unblock extension to make the web page work.


Deta.sh preset

Deta.sh is a pretty cool (and as free as it gets) personal-ish cloud for developers 🚀
Website →
Documentation →
It seemingly doesn't support ESM, what is a good way to approach it?

data.sh seems SO cool! I can't wait to try it. Thanks for introducing
Regarding ESM, we might create a CJS entry and use dynamic import() to initialize the entry. Not sure have to try.

the startup has been changed to https://deta.surf/


Does not work on 1.19.1-rc1 (fabric)

When trying this out in 1.19.1-rc1 (fabric) the game fails to open. Here's a link to my logs:
https://paste.ee/p/EsrvC

You need Fabric API.

Yes, first - you need that. Second - the mod doesn't officially support 1.19.1 RC; it will most likely work, but there are a couple important changes that will be introduced as part of official release for 1.19.1.


Allow creation of a server lock, which has no limits and is never deleted

Any suggestions?

Either this or having a permission node that would be given to admins that allows them to bypass the lock limits


[FEA] Finish LIKE support

Is your feature request related to a problem? Please describe.
LIKE currently only works for string literal patters. It would be nice if we could support a column of patters too, but this is not very commonly used.

This is likely to be a really low priority, because I don't know of any queries where the LIKE pattern is non-scalar.

#10618 is a case where someone was requesting LIKE with a column of patterns.  libcudf has support for this, see https://github.com/rapidsai/cudf/blob/branch-24.10/cpp/include/cudf/strings/contains.hpp#L169-L208.  We need to add JNI bindings and update GpuLike to use it, but it should be relatively straightforward to do so.


npm, metadata & badges

updates package.json to be more modern
remove lint.sh script in place of npm test
add badges for npm, travis, license

+1


Vulnerability roundup 113: deno-1.19.1: 1 advisory [10.0]

search, files

[ ] CVE-2022-24783 CVSSv3=10.0 (nixos-unstable)

CVE details
CVE-2022-24783
Deno is a runtime for JavaScript and TypeScript. The versions of Deno between release 1.18.0 and 1.20.2 (inclusive) are vulnerable to an attack where a malicious actor controlling the code executed in a Deno runtime could bypass all permission checks and execute arbitrary shell code. This vulnerability does not affect users of Deno Deploy. The vulnerability has been patched in Deno 1.20.3. There is no workaround. All users are recommended to upgrade to 1.20.3 immediately.

Scanned versions: nixos-unstable: ff9efb0724d.
Cc @06kellyjac

#167500 in master atm

It only affected unstable and the channel has picked up the change, so this is fixed.


[6.0] - Multiple OS Testing With Travis CI - Windows Symlink

Laravel Version: 6.0
PHP Version: 7.2+
Database Driver & Version: (not relevant)

Description:
Sorry to open this issue again, but I'm hoping to make some final headway here. This has been brought up time and time again across Laravel and PHP.
Reference Pull Request
Symfony Reference (mentioned below)
I'm going to quote myself from the reference PR:

I've been working on getting Travis CI working with a Windows instance (on the 6.0 branch), and this bug is basically a non-starter. Is there any interest in fixing it or having it fixed? (The short term would be a PHP_OS check and skip - I'm also open to a PHP version check as well in case internals ever gives Windows some love).
To be honest - there are several issues around this set of tests causing problems with Windows testing, but I don't want to waste any more of my time if there is no chance or interest in fixing this. Symlink is basically broken on Windows unless you want to run your application/testsuite as Administrator (or something equally elevated).
Symfony works around this issue here it seems.

I understand the hesitation with taking ownership of more code that hasn't been written by the team, but maybe having a testsuite across all of the OSes would ease that burden?
Either way the functionality of symlink is inconsistent at the very core of PHP (or rather Windows implementation of symlink does not fully follow the *nix implementation).
Steps To Reproduce:
See my Travis CI report here (Line 449)
EDIT: I understand that I'm not at 100% with the Windows instance (still need to get Redis tests working), but I figured I would address this in the meantime.

Here is an example of what I am proposing short term.

I'm basically at a point where the only known issues are the issues that dismiss the Windows environment.
Also, there is a MacOS consideration for pcntl here, but ignored here.
.travis.yml
Windows PHP 7.2 Results
Windows PHP 7.3 Results

Heya, I'm gonna have to close this as a duplicate of https://github.com/laravel/framework/issues/27004
Appreciating all the info though. Maybe cross-post everything on the other issue? Thanks!


JSON field removes empty objects

Bug Report
This is reopening of #2189. As after<EMAIL_ADDRESS>update the empty objects in JSON fields get stripped upon saving the document. I tried clean payload install and the issue is still present.
As the commited fix disabled moongose minimize option I have tested to update the field with local API and it keeps the empty object correctly.
This works (test2: {} is present on next load of the document):
payload.update({
        id: "64005e1b6811e12c7f73d111",
        collection: 'pages',
        data: {
          test: {
            test1: 1,
            test2: {}
          }
        }
      })

Therefore now I suspect it's caused by some processing on Payload's API layer. While sending an update request through REST or GQL it strips the object. (The shape of request's payload coming from admin is correct).
Steps to Reproduce

Have JSON field on Collection
Try to store:

{
   "test": 1,
   "test2": {}
}

See the output immediatelly being changed upon saving to:

{
   "test": 1
}

<EMAIL_ADDRESS>

Hey @chladog, I attempted to recreate this issue this morning and was unable to do so. You can see in the screen recording that for me it is saving the empty object properly.
Can you let me know if anything might be different with your config?
In addition to the recording, I setup a test on a branch with these additional tests: code comparison
https://user-images.githubusercontent.com/30633324/223184024-d43fce61-309f-43ee-81fe-8b7dfcbcdb83.mp4

Hello @JarrodMFlesch,
ahha, I tracked the problem...:
So the value gets stored correctly in the DB actually and on next refresh it even renders correctly, however if you have versioning on a collection enabled and then save the updates on collection item the object is missing in update response body and thats what admin renders resulting in shape described above.
If I remove from collection config following:
  versions: {
    drafts: {
      autosave: false,
    },

    maxPerDoc: 50,
  },

then it works as in your video.

Thanks for the help narrowing it down here @chladog. Fix incoming with #2248 👍


API Endpoint: Get Logs

Add an API endpoint that can fetch logs the user has access to.
This is related to #41 , since we will want to allow the user to specify the context that they want the logs from.
E.g., get all the course logs, or get the logs about this user.

Implemented.


interface for adding files to an experiment

It would be very useful if one could add files to an experiment that can be then be stored by the observers alongside the information for the run.
I can think of several different types of files to add:

dependencies:
static files:
dynamic files:

Dependencies
This includes additional source-files etc.. They would be part of the info about the experiment. So they could potentially be shared among many runs.

they exist as files already, so you want to specify them as a path
they should be added as part of the setup before initialization and of course also before the run is started
a possible interface could be a simple ex.add_dependency("/my/path/filname.foo")

Static Files
Files that are written once and never changed, but belong to the run not to the experiment. For example a report that is written after the run is finished would fall in this category. They might be a special case of Dynamic Files.
Dynamic Files
Files that are updated continuously. So they should be updated as part of the heartbeat. Examples include current weights or log files.

they are part of the run, so the interface should be part of the run object.
they might not exist yet, so it would be convenient to have an interface that provides a temporary named file to write to that is also tracked and deleted afterwards. Something like f = run.create_tracked_file(filename_in_db)
optionally you might want to create the file yourself. Then a simple run.track_file(filename, filename_in_db=None) might be sufficient.
Remembering the file handler in client code would be tedious. So maybe there should be a method f = run.get_tracked_file(filename_in_db)  that returns the file if present or otherwise creates it?

dynamic files are not covered yet, but all the other cases have been implemented now


DB correction + minor graphic fix

Resolved some error with the new db fields.
Removed boolean type as it doesn't exist in the sqlite plugin: bool is not a supported SQLite type. Use INTEGER and 0 and 1 values. (source)

I'm getting this on Mac OS:

I/flutter (14356): *** WARNING ***
I/flutter (14356):
I/flutter (14356): Invalid argument true with type bool.
I/flutter (14356): Only num, String and Uint8List are supported. See https://github.com/tekartik/sqflite/blob/master/sqflite/doc/supported_types.md for details
I/flutter (14356):
I/flutter (14356): This will throw an exception in the future. For now it is displayed once per type.

Is your commit related?

I'm getting this on Mac OS:

I/flutter (14356): *** WARNING ***
I/flutter (14356):
I/flutter (14356): Invalid argument true with type bool.
I/flutter (14356): Only num, String and Uint8List are supported. See https://github.com/tekartik/sqflite/blob/master/sqflite/doc/supported_types.md for details
I/flutter (14356):
I/flutter (14356): This will throw an exception in the future. For now it is displayed once per type.

Is your commit related?
EDIT: I'm pretty sure this is an old Warn, so the correct question should be: is your commit intended to solve also this?

Yes, the commit solves also that problem, i think it shows up to you for the cached db, if you remove and reinstall the app it shouldn't appear again, at least for me that worked fine.


update to squid 3.5 (latest)

Hi, thx for your amazing containers,
could you please update this one to the last version ?
I'm facing security issues with this one (its a 2013 release ^^" )

➤ Sameer Naik commented:
@neolectron soon enough I will be switching to ubuntu:xenial base. You can expect an updated squid soon.

➤ Dashamir Hoxha commented:
Squid container based on Ubuntu 16.04: https://github.com/docker-scripts/squid

➤ Vincent Voyer commented:
Hi @sameersbn, any news on having squid 3.5? What are the required changes?

➤ Dashamir Hoxha commented:
The required changes are switching to ubuntu:16.04 (aka ubuntu:xenial).


Compilation fails on GCC8+

Noticed the project is failing compilation on GCC8, the default on Ubuntu flavors 19+. I tried to solve them, but there are quite a few and in places that have non-trivial fixes. Not urgent, creating to track.
I also noticed that we're pulling in third-party sources into main compilation. This will most likely cause issues since different projects assume different compiler flags. For example, the madoka project fails compilation on GCC8+, too. We should probably be building them separately and linking them in to save ourselves the headache.

I suspect this will not be heavily investigated until next summer when we'd want to move the default environment to the next Ubuntu LTS release (20.04) and whatever its default toolchain versions (GCC, Clang, LLVM) are and fix any of the compatibility issues in one shot then.

Fixed by #642. As long as GCC 7 is the only version in CI though, we're likely to continue breaking compatibility.


Deploy Container JFR with TLS using service serving certificates

This PR aims to support TLS in Container JFR when deployed in OpenShift from the operator. The idea is to use the default router wildcard certificate for the Container JFR route, but with TLS termination set to re-encrypt instead of edge. The internal service traffic is encrypted using a key-pair/cert created by the OpenShift service serving certificates mechanism [1]. The created certificate is signed by OpenShift's service CA, which is already trusted by the router. These keys/certs get converted from PEM format keys/certs into PKCS12 keystores/truststores using an init container and are passed to Container JFR using a mounted emptyDir volume [2].
This seems to work, except for enabling TLS for the remote JMX connection. After passing -Djavax.net.debug=ssl,handshake to Container JFR, I get the following error:
javax.net.ssl|ERROR|1C|RMI TCP Connection(16)-<IP_ADDRESS>|2020-09-18 20:33:23.111 GMT|TransportContext.java:318|Fatal (CERTIFICATE_UNKNOWN): Extended key usage does not permit use for TLS client authentication (
"throwable" : {
  sun.security.validator.ValidatorException: Extended key usage does not permit use for TLS client authentication
      at java.base/sun.security.validator.EndEntityChecker.checkTLSClient(EndEntityChecker.java:245)
      at java.base/sun.security.validator.EndEntityChecker.check(EndEntityChecker.java:146)
      at java.base/sun.security.validator.Validator.validate(Validator.java:277)
      at java.base/sun.security.ssl.X509TrustManagerImpl.validate(X509TrustManagerImpl.java:313)
      at java.base/sun.security.ssl.X509TrustManagerImpl.checkTrusted(X509TrustManagerImpl.java:222)
      at java.base/sun.security.ssl.X509TrustManagerImpl.checkClientTrusted(X509TrustManagerImpl.java:123)
      at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.checkClientCerts(CertificateMessage.java:1267)
      at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.onConsumeCertificate(CertificateMessage.java:1186)
      at java.base/sun.security.ssl.CertificateMessage$T13CertificateConsumer.consume(CertificateMessage.java:1163)
      at java.base/sun.security.ssl.SSLHandshake.consume(SSLHandshake.java:392)
      at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:444)
      at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:422)
      at java.base/sun.security.ssl.TransportContext.dispatch(TransportContext.java:183)
      at java.base/sun.security.ssl.SSLTransport.decode(SSLTransport.java:164)
      at java.base/sun.security.ssl.SSLSocketImpl.decode(SSLSocketImpl.java:1144)
      at java.base/sun.security.ssl.SSLSocketImpl.readHandshakeRecord(SSLSocketImpl.java:1055)
      at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:395)
      at java.base/sun.security.ssl.SSLSocketImpl.ensureNegotiated(SSLSocketImpl.java:709)
      at java.base/sun.security.ssl.SSLSocketImpl$AppInputStream.read(SSLSocketImpl.java:792)
      at java.base/java.io.BufferedInputStream.fill(BufferedInputStream.java:252)
      at java.base/java.io.BufferedInputStream.read(BufferedInputStream.java:271)
      at java.base/java.io.DataInputStream.readInt(DataInputStream.java:392)
      at java.rmi/sun.rmi.transport.tcp.TCPTransport$ConnectionHandler.run0(TCPTransport.java:716)
      at java.rmi/sun.rmi.transport.tcp.TCPTransport$ConnectionHandler.lambda$run$0(TCPTransport.java:677)
      at java.base/java.security.AccessController.doPrivileged(Native Method)
      at java.rmi/sun.rmi.transport.tcp.TCPTransport$ConnectionHandler.run(TCPTransport.java:676)
      at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
      at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
      at java.base/java.lang.Thread.run(Thread.java:834)}

I then verified that these service serving certificates do not support TLS client authentication. [3][4] If we don't set com.sun.management.jmxremote.ssl.need.client.auth=true, then the JMX communication works, but this is not ideal. I'm marking this PR as a draft for now while we explore solutions to allow mutual TLS authentication for JMX connections. One obvious solution is to create a self signed cert just for client authentication, but there may be better solutions.
[1] https://docs.openshift.com/container-platform/4.5/security/certificates/service-serving-certificate.html
[2] https://developers.redhat.com/blog/2017/11/22/dynamically-creating-java-keystores-openshift/
[3] https://github.com/openshift/origin/issues/10085
[4] https://github.com/openshift/service-ca-operator/issues/62

This has been superseded by #115


Align all array items

A handful of arrays had been aligned, this just makes them all the same.

Ping.

The build is failing. Additionally, I do not agree with vertical alignment for reasons that are described here: http://en.wikipedia.org/wiki/Programming_style#Vertical_alignment
Thanks for your eagerness to contribute, but I'm declining this pull request. Sorry.


Add timeline viewer sharing feature

Resolves #49
Added new flag shareWithTimelineViewer enabling which trace data will be stored to your gdrive and timeline opened automatically in a browser.
Example
https://chromedevtools.github.io/timeline-viewer/?loadTimelineFromURL=https://drive.google.com/file/d//0B0c67TI7mLzEMFNhSENiVDhHWEE/view?usp=drivesdk

Oh, totally forgot about permissions sharing, now it's ready :)

@paulirish ready for one more look :)

lgtm.
we can do the upload/view split in another PR.
sweet!


Provide a strategy pattern to determine if the request needs to be loadbalanced

I am working on a custom implementation of discovery client/ service registry/ load balancer
Current load balancer design appears to give no way to distinguish between a regular request, i.e. "http://example.com" and a load balanced one i.e. "lb://example.com".
Ideally there would be a way to do the following:

Wire up a strategy pattern that would be injected into LoadBalancerClient automatically that would help determine if the request needs to be load balanced or not, and if not i.e. "http://example.com" would be executed as is. Currently all requests go through the entire chain of load balancing logic and if no ServiceInstance is found, it throws an exception. Example: https://github.com/spring-cloud/spring-cloud-commons/blob/087b94a905c5428c89e4db22a6a284999d5d0a7d/spring-cloud-loadbalancer/src/main/java/org/springframework/cloud/loadbalancer/blocking/client/BlockingLoadBalancerClient.java#L79

Provide an easy way to (perhaps this is simply a documentation issue) to  route all such requests (ones identified in step 1 for load balancing) to a generic ServiceInstanceListSupplier that is not unique to a specific service. Currently each service must have its own supplier defined, or at least clients need to be defined manually. This might work for small use-cases, but is not ideal for larger projects.

I have considered avoiding using the cloud-load-balancer abstraction all-together, but that will force me to re-implement other useful features of the abstraction, such as retry support, blocking, webflux, and gateway support.
Thanks for your consideration.

I wanted to add to this, I think I may have solved part of the problem, specifically, item 2.
It seems that annotating a config class with @LoadBalancerClients without any values results in all requests generating a new load-balancer through the LoadBalancerClientConfiguration, however, this means I am unable to provide my own load balancer and forced to use the RoundRobinLoadBalancer provided in the reactorServiceInstanceLoadBalancer method here: https://github.com/spring-cloud/spring-cloud-commons/blob/087b94a905c5428c89e4db22a6a284999d5d0a7d/spring-cloud-loadbalancer/src/main/java/org/springframework/cloud/loadbalancer/annotation/LoadBalancerClientConfiguration.java#L65
Is there a way I can keep all the benefits that this class provides, while overriding the type of LoadBalancer it returns?
Thanks!


Add function that receives coordinates of a point (x,y) and determines is that point belongs to the shape.

After that modify all mouse related event emitters like "click", "mousedown", "mouseover" etc. to trigger only if mouse cursor is exactly inside shape's polygon.


[Snyk] Security upgrade urllib3 from 1.26.6 to 1.26.18

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

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

requirements.txt

Vulnerabilities that will be fixed
By pinning:

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

496/1000  Why? Recently disclosed, Has a fix available, CVSS 4.2
Information Exposure Through Sent Data SNYK-PYTHON-URLLIB3-6002459
urllib3:  1.26.6 -> 1.26.18
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.
Check the changes in this PR to ensure they won't cause issues with your project.

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Picked up in #703


Added draw CMU UC using leaflet in js

This is going to be superseded by PR with react based osm-viewer.

Superseded by: https://github.com/geotrellis/geotrellis-osm-elevation/pull/15


Add thread decorator in the exchange request as per spec

"~thread": { "pthid": "" },
https://github.com/hyperledger/aries-rfcs/tree/master/features/0023-did-exchange
Implement the following spec:
https://github.com/hyperledger/aries-rfcs/tree/master/features/0023-did-exchange#correlating-requests-to-invitations

#718


Add page on Eye2Sky network

https://www.dlr.de/ve/en/desktopdefault.aspx/tabid-17889/28318_read-73347/

This issue is better described in #28


Xposed not working anymore

It complaints Xposed module not enabled like https://github.com/KieronQuinn/DiscoverKiller/issues/17.
Discover killer 3.0
Google <IP_ADDRESS>.arm64
Lsposed 1.9.1(6900)

You already linked the duplicate, nothing has changed #17.

It's a regression after updating Google app.

No it isn't, nothing has changed in the Google app. This happens occasionally to some, it's always the local Xposed setup which has failed.

"Uninstall updates" does solve the issue. After uninstallation, Google version is <IP_ADDRESS>.arm64.


Better errors for insufficient stack or memory size

https://jolt.a16zcrypto.com/usage/troubleshooting.html#insufficient-memory-or-stack-size
Currently, the tracer will error out with nondescript errors like Unknown memory mapping 7FFFFE04.
The tracer should detect when we have exhausted the stack and/or heap, and throw a descriptive error if so

Hey! Can I take this up?

Yeah go for it!

@ameya-deshmukh Any update here?

Hey @moodlezoup! Sorry, couldn't find time to work on this. Can I grind this out over the weekend and give you an update by early Monday ET?

Yeah that'd be great! Lmk if you need any pointers

@ameya-deshmukh @moodlezoup I would like to have a try if you fine with that

Hey @4rgon4ut sure! I didn't get the time to work on this issue, all yours to take

@moodlezoup I have two questions:

Would you like it to be panics with specified messages (based on the case) or new trap type here
Does tracer/src/emulator/cpu.rs is the right place to add related tests?

@moodlezoup I have two questions:

Would you like it to be panics with specified messages (based on the case) or new trap type here
Does tracer/src/emulator/cpu.rs is the right place to add related tests?

Thanks for taking this!

just a better panic message is fine
I think perhaps tracer/src/emulator/mmu.rs may be a better place for tests

@moodlezoup does stack grows downwards, from higher stack addr down to the DRAM_BASE ? ref
/// DRAM base address. Offset from this base address
/// is the address in main memory.
pub const DRAM_BASE: u64 = 0x80000000;

Yes, I believe so


Flexible number format in _get_parameter_array

Close #952 by implementing an iterative process.
Ok, it might be slow sometimes, but I think it is a reasonable approach for this case. I setup an upper limit at F60.12 for the float format.

I believe most of the cases it will just go on the first iteration. It might not be necessary so many steps though. But as you said, it will be mostly a "one-time" thing.


Bypass ssl certificate issue

is it possible to bypass ssl certificate issue with using this library like -K in curl command?

I think by the client does that by default when your endpoint begins with 'https'.
https://github.com/hawkular/hawkular-client-java/blob/master/src/main/java/org/hawkular/client/core/jaxrs/RestFactory.java#L121


Undefined behavior when calling vsnprintf() leads to empty error messages

LMS64CProtocol.cpp convertStatus executes the following code.
static int convertStatus(const int &status, const LMS64CProtocol::GenericPacket &pkt)
{
    if (status != 0) return ReportError(EIO, GetLastErrorMessage());

This causes a call to vsnprintf with the target string and the format string being the same buffer.
const char *lime::GetLastErrorMessage(void)
{
    return _reportedErrorMessage;
}
int lime::ReportError(const int errnum, const char *format, va_list argList)
{
    _reportedErrorCode = errnum;
    vsnprintf(_reportedErrorMessage, MAX_MSG_LEN, format, argList);
    return errnum;
}

According to "man vsnprintf" this is undefined behavior.
C99  and  POSIX.1-2001 specify that the results are undefined if a call to sprintf(), snprintf(), vsprintf(), or vsnprintf() would cause copying to take place between objects that overlap (e.g., if the target string array and one of the supplied input arguments refer to the same buffer).  See NOTES.
On Ubuntu 17.04 this can cause empty error messages when loading an ini file fails due to some error

Some other code executes
ReportError(-1, "RegistersTest() failed - %s", GetLastErrorMessage());

which might also cause problems.

This seems to be fixed in current LimeSuite


scripted tests are not independent on windows

steps
Since 1.4.0, the scripted tests for paradox at appveyor fail with:
[info] Running 19 / 19 (100.00%) scripted tests with RunFromSourceMain
[info] Running paradox/custom-configuration
[info] Running paradox/custom-directive
[info] Running paradox/default-theme
[info] [info] welcome to sbt 1.4.0 (Oracle Corporation Java 1.8.0_162)
[info] [info] loading settings for project sbt_9f2594d5-build from plugins.sbt ...
[info] [info] loading project definition from C:\Users\appveyor\AppData\Local\Temp\1\sbt_9f2594d5\project
[info] [info] loading settings for project docs from build.sbt ...
[info] [info] set current project to Paradox Default Theme Test (in build file:/C:/Users/appveyor/AppData/Local/Temp/1/sbt_9f2594d5/)
[info] [error] Unexpected top-level pages (pages that do no have a parent in the Table of Contents).
[info] [error] If this is intentional, update the `paradoxRoots` sbt setting to reflect the new expected roots.
[info] [error] Current ToC roots: [index.html, sub/unindexed.html, test.html]
[info] [error] Specified ToC roots: [index.html, sub/unindexed.html]
[info] [error] Paradox failed with 1 errors
[info] [error] (Compile / paradoxMarkdownToHtml) com.lightbend.paradox.sbt.ParadoxPlugin$ParadoxException
[info] [error] Total time: 1 s, completed Oct 5, 2020 1:54:39 AM
[error] x paradox/default-theme
[error]  Cause of test exception: {line 1}  Command failed: paradox failed

The paradox/default-theme test fails because it finds an unexpected test.html. test.html is an expected output of the previous scripted test, paradox/custom-directive.
problem
Artifacts from one scripted test appear to leak into the next. I'm not sure yet if it's the sources or the target resources - I don't have a windows machine handy to easily test this with, and it works fine on linux...
expectation
I expected scripted tests to be independent.

This looks like a problem with paradox, not sbt. That being said, scripted tests generally run in alphabetical order on windows and in a random order on other platforms which is probably why you see the problem only on windows.

I'll see if I can come up with a more targeted reproducer (or indeed confirm this as a problem with paradox, though I have my doubts ;) )

Ok. I stand corrected. For performance reasons, the scripted plugin now reuses the same directory for all of the tests. To make this work, between each test, it blows away all of the files in the test directory and then calls reload. The problem is that sbt can't actually delete some of the files on windows, like index.html, because some other process has a handle to them. Here is an example stack trace:
java.nio.file.FileSystemException: C:\Users\micro\AppData\Local\Temp\sbt_c46b041\src\docs2\paradox\index.md: The process cannot access
the file because it is being used by another process.

        at sun.nio.fs.WindowsException.translateToIOException(WindowsException.java:86)
        at sun.nio.fs.WindowsException.rethrowAsIOException(WindowsException.java:97)
        at sun.nio.fs.WindowsException.rethrowAsIOException(WindowsException.java:102)
        at sun.nio.fs.WindowsFileSystemProvider.implDelete(WindowsFileSystemProvider.java:269)
        at sun.nio.fs.AbstractFileSystemProvider.deleteIfExists(AbstractFileSystemProvider.java:108)
        at java.nio.file.Files.deleteIfExists(Files.java:1165)
        at sbt.scriptedtest.ScriptedTests.$anonfun$runBatchedTests$8(ScriptedTests.scala:245)

Does paradox shell out to another process during scripted tests? If so, it may be necessary to ensure that process has exited when the tests are complete.
It wasn't necessarily intentional that we changed the default behavior in 1.3.13 vs 1.4.0. I am looking into why that is and if there is a way to override it.

The problem is that sbt can't actually delete some of the files on windows, like index.html, because some other process has a handle to them

I noticed the same thing. Turns out we were incorrectly holding on to them, so it was also a problem in paradox ;).

Here is an example stack trace

Hmm, I don't see those in the paradox appveyor output at all - were those hidden by default?

I am looking into why that is and if there is a way to override it.

Great! In the mean time I fixed the unclosed stream so this is not an urgent problem for me anymore. Do you want to keep this open anyway?

Hmm, I don't see those in the paradox appveyor output at all - were those hidden by default?

I added a stack trace print to a local sbt build so they wouldn't be visible to you.

Great! In the mean time I fixed the unclosed stream so this is not an urgent problem for me anymore. Do you want to keep this open anyway?

Yes, let's leave this open in case someone else has a similar problem and can't work around it.

Now that #6054 is merged, there is a workaround to the problem of sbt being unable to delete some files on windows. Builds can add scriptedBatchExecution := false to their project to restore the pre-1.4.0 semantics where each scripted test gets a fresh sbt instance (which means the tests tend to be much slower but each test is completely isolated).


Incorrect command for setting up blueprint in management group scope

I believe target scope assignment for management group scope is incorrect. it needs to be
az blueprint create --management-group MyManagementGroup  --name MyBlueprint \ --description "blueprint contains all artifact"  --target-scope managementGroup
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: e584f1d8-f5c1-51e8-b0bf-38dd19995d12
Version Independent ID: 7f7930db-3630-7ecb-1e28-b82a98d4b35c
Content: az blueprint
Content Source: latest/docs-ref-autogen/blueprint.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

@TharinduK It can be either managementgroup or subscription.
https://docs.microsoft.com/en-us/cli/azure/blueprint?view=azure-cli-latest#az-blueprint-create
The scope where this blueprint definition can be assigned.
accepted values: managementGroup, subscription
default value: subscription
Its mentioned here as well
https://docs.microsoft.com/en-us/azure/governance/blueprints/overview#blueprint-assignment
Each Published Version of a blueprint can be assigned (with a max name length of 90 characters) to an existing management group or subscription. In the portal, the blueprint defaults the Version to the one Published most recently. If there are artifact parameters or blueprint parameters, then the parameters are defined during the assignment process.
**Assigning a blueprint definition to a management group means the assignment object exists at the management group. The deployment of artifacts still targets a subscription**. To perform a management group assignment, the [Create Or Update REST API](https://docs.microsoft.com/en-us/rest/api/blueprints/assignments/createorupdate) must be used and the request body must include a value for properties.scope to define the target subscription.


HUE-9646 [Hue] Docker image size reduction for Hue, Hue Load Balanc…

…er images

And for info which final size do you get?


I can’t sort the root directory.

How do I make it work in the root directory? Currently, I can’t sort the root directory.

thanks! you help me


In AWS::B2BI::Transformer, the status is created as 'inactive' even though the status is created as 'active'.

Name of the resource
AWS::B2BI::Transformer
Resource Name
AWS::B2BI::Transformer
Issue Description
In AWS::B2BI::Transformer, the status is created as 'inactive' even though the status is created as 'active'.
This causes the error "Transformer status must be 'active'" in AWS::B2BI::Capability.
The resource is not added in the list of Name of the resource.
Expected Behavior
As stated in the Issue Description
Observed Behavior
As stated in the Issue Description
Test Cases
As stated in the Issue Description
Other Details
https://github.com/aws-cloudformation/cloudformation-coverage-roadmap/issues/1878

At this moment, this is currently by design. An AWS::B2BI::Transformer is created in an inactive state by default and must be updated to an active state in a separate action. However, we realize that this is not a good customer experience for customers trying to create B2BI CloudFormation resources. We are discussing this concern internally.

We have now added support for customers to create active B2BI transformers in CloudFormation. When creating a transformer, please specify the status field with the value active to create an active transformer. Feel free to reopen this issue if there are any issues or concerns.


Query cache for ArangoDB

Discussion here: https://groups.google.com/forum/?fromgroups#!topic/arangodb/Tx-P57xT8DU
In a nutshell, this is a feature request to implement a query cache, which may or may not be implemented in the future.
Essentially, all queries would check the cache first, before processing (if required). Making changes to the collection would invalidate cache entries.
As Jan has pointed out in the discussion, a cache is useful, but it depends on your use-case. In some cases, it can cause performance issues. Hence, this is why this is "uncertain" at the moment.
Another option would be to cache the execution plan instead of the query + result set. This may or may not be useful depending on how efficient ArangoDB is at creating the execution plan.

I just stumbled upon this issue and found it can be closed now.
There is a query result cache available starting with ArangoDB 2.7.0.


Fixed: Move processmon to internal  package

This PR creates an internal package called processmon as this is not consumable outside trireme lib.

Codecov Report

Merging #392 into master will increase coverage by 0.27%.
The diff coverage is 61.61%.

@@            Coverage Diff             @@
##           master     #392      +/-   ##
==========================================
+ Coverage   55.16%   55.43%   +0.27%
==========================================
  Files          51       50       -1
  Lines        6314     6263      -51
==========================================
- Hits         3483     3472      -11
+ Misses       2507     2465      -42
- Partials      324      326       +2

Impacted Files
Coverage Δ

trireme.go
46.37% <0%> (-1.15%)
:arrow_down:

enforcer/proxy/enforcerproxy.go
78.41% <100%> (+2.61%)
:arrow_up:

internal/processmon/processmon.go
58.64% <58.64%> (ø)

cmd/remoteenforcer/remoteenforcer_linux.go
54.15% <72.72%> (-1.6%)
:arrow_down:

cmd/remoteenforcer/stats.go
26.78% <75%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[BUG]

Library name and version
Cognitive - Form Recognizer V2 and V3
Describe the bug
This happens with the preview version as well as the stable version.
Last line  with one cell in a table is missing
Expected behavior
The last row of the table contains only one cell. nevertheless it should belong to the table detected by FormReconizer.
Actual behavior
Last line  with one cell in a table is missing
Reproduction Steps
File to submit to Form Recognizer Studio
Table2.pdf
Environment
Using Form Recognizer Studio, Layout or General document.

Hi @cstfr.  Thank you for reaching out and we regret that you're experiencing difficulties.   Unfortunately, this repository is only able to assist with packages for the Azure SDK for .NET.   Form Recognizer Studio is not something that we have insight into.   I'd recommend opening an Azure Support Ticket which would be the best path for assistance.
I'm going to close this out, but please feel free to unresolve if I've misunderstood what help you're seeking.


Remove slavebook worker

For now, this is an experimental branch aimed to remove SlaveBook worker.
Related issue: https://github.com/rubykube/peatio/issues/2144
Some context:

Every Matching::Engine has a Matching::OrderBookManager that owns two in-memory Matching::OrderBook: one for bid orders and one for ask orders
Everytime an OrderBook is modified it calls its broadcast method that puts a payload to the :slave_book AMQP queue
SlaveBook is a worker that listens to that queue and builds a copy of Matching::Engines' OrderBookManager by reading the payloads it receives from the queue
SlaveBook has a thread that writes its "depths" (a compact and accumulated version of its OrderBookManagers' OrderBook PriceLevels) to Rails.cache on keys like "peatio:#{market.id}:depths:{market.side}s" every 3 seconds, even when nothing has been modified
On the other hand, global_state daemon, every 5 seconds, triggers a publish event that reads depths from Rails.cache and sends and update message to users' websockets, also even when nothing has been modified.

Pull Request https://github.com/rubykube/peatio/pull/1894 rewrote SlaveBook's get_depth method to fetch data from database making SlaveBook's OrderBookManagers pointless and also the :slave_book queue and the OrderBook.broadcast method.
So the point is: can we remove SlaveBook worker and reduce the amount of data being sent to users?
This PR tries to do that by:

Removing SlaveBook Worker
Giving Matching::OrderBook the methods to build its "depths" and write them to Rails.cache
Making Matching::OrderBookManager, responsible for checking if its OrderBooks have been modified and then trigger the publishing event

More specifically this PR:

Refactors Matching::OrderBook to use mutexes to protect its @limit_orders and @market_orders (we'll see later there will be a concurrency)
Replaces Matching::OrderBook.broadcast method by an @updated attribute that is set to true when some of its orders is modified (add/remove/update)
Gives build_depth and write_depth_to_cache methods to Matching::OrderBook
Gives Matching::OrderBookManager a thread that checks its OrderBooks for changes (every 500ms) and if so then it triggers the publish event (this thread reading from OrderBook competes with Matching::Engine modifying the OrderBook, so that's why the mutexes)
A mandatory update can also be configured just in case (it was hardcoded every 30s)
Add code to prevent leaking threads
and finally, remove SlaveBook worker and its related stuff

So it is expected that depths will be written to Rails.cache, and users will receive update events on their websockets, only when some OrderBook has been updated.
I'd appreciate any suggestions and discussion on this topic :)

Hello, Thanks we noticed this issue, and what we did, is query the database from global_state.
You may not notice the change, but I will think @ysv will be able to show you.

@elias19r I will dig into your code.
Idea looks interesting for me.
We already did kind of hotfix for this issue
Thanks for your contribution to Peatio !
RubyKube team really appreciate your work

nice work
i love it 👍

@elias19r I had time to check your pr. Unfortunately it doesn't coincides with our view of solving this problem because:

Your approach makes Matching::OrderBook slower

Our plan is to support stop hidden orders which will appear in Matching::Orderbook but we will not show them for enduser so for the future reasons it's better to keep them in separate places

Also this approach makes complicated to read and understand Orderbook

For now we are kind of happy with single select per 3 second in SlaveBook but for sure we have this issue in mind and going to work on it soon.
Thanks for your contribution to Peatio !
RubyKube team really appreciate your work.
For now I'm closing this PR
Put your comments in #2144


[sil-combine] Eliminate a dead parameter.

We pass in deadEndBlocks into scCanonicalize's constructor now.

@swift-ci smoke test


get_signature_requests doesn't return list_info

The get_signature_requests method for the signature_request/list endpoint doesn't return the list_info parameters as listed in the API docs

Resolved in https://github.com/HelloFax/hellosign-ruby-sdk/pull/56


fix(bash): Further isolate bash runs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
0

Relevant Lines:
13

💛  - Coveralls


[SPARK-29143][PYTHON][ML] Pyspark feature models support column setters/getters

What changes were proposed in this pull request?
add column setters/getters support in Pyspark feature models
Why are the changes needed?
keep parity between Pyspark and Scala
Does this PR introduce any user-facing change?
Yes.
After the change, Pyspark feature models have column setters/getters support.
How was this patch tested?
Add some doctests

Merged build finished. Test PASSed.

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

Test build #111246 has started for PR 25908 at commit 2ced6ea.

Test build #111246 has finished for PR 25908 at commit 2ced6ea.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class LSHParams(JavaParams, HasInputCol, HasOutputCol):
class LSH(JavaEstimator, LSHParams, JavaMLReadable, JavaMLWritable):
class LSHModel(JavaModel, LSHParams):
class BucketedRandomProjectionLSHParams(JavaParams):
class BucketedRandomProjectionLSH(LSH, BucketedRandomProjectionLSHParams,
class BucketedRandomProjectionLSHModel(LSHModel, BucketedRandomProjectionLSHParams, JavaMLReadable,
class ImputerParams(JavaParams, HasInputCols, HasOutputCols):
class Imputer(JavaEstimator, ImputerParams, JavaMLReadable, JavaMLWritable):
class ImputerModel(JavaModel, ImputerParams, JavaMLReadable, JavaMLWritable):
class MaxAbsScalerParams(JavaParams, HasInputCol, HasOutputCol):
class MaxAbsScaler(JavaEstimator, MaxAbsScalerParams, JavaMLReadable, JavaMLWritable):
class MaxAbsScalerModel(JavaModel, MaxAbsScalerParams, JavaMLReadable, JavaMLWritable):
class MinMaxScalerParams(JavaParams, HasInputCol, HasOutputCol):
class MinMaxScaler(JavaEstimator, MinMaxScalerParams, JavaMLReadable, JavaMLWritable):
class MinMaxScalerModel(JavaModel, MinMaxScalerParams, JavaMLReadable, JavaMLWritable):
class OneHotEncoderParams(JavaParams, HasInputCols, HasOutputCols, HasHandleInvalid):
class OneHotEncoder(JavaEstimator, OneHotEncoderParams, JavaMLReadable, JavaMLWritable):
class OneHotEncoderModel(JavaModel, OneHotEncoderParams, JavaMLReadable, JavaMLWritable):
class RobustScalerParams(JavaParams, HasInputCol, HasOutputCol):
class RobustScaler(JavaEstimator, RobustScalerParams, JavaMLReadable, JavaMLWritable):
class RobustScalerModel(JavaModel, RobustScalerParams, JavaMLReadable, JavaMLWritable):
class StandardScalerParams(JavaParams, HasInputCol, HasOutputCol):
class StandardScaler(JavaEstimator, StandardScalerParams, JavaMLReadable, JavaMLWritable):
class StandardScalerModel(JavaModel, StandardScalerParams, JavaMLReadable, JavaMLWritable):
class VectorIndexerParams(JavaParams, HasInputCol, HasOutputCol, HasHandleInvalid):
class VectorIndexer(JavaEstimator, VectorIndexerParams, JavaMLReadable, JavaMLWritable):
class VectorIndexerModel(JavaModel, VectorIndexerParams, JavaMLReadable, JavaMLWritable):
class Word2VecParams(JavaParams, HasStepSize, HasMaxIter, HasSeed, HasInputCol, HasOutputCol):
class Word2Vec(JavaEstimator, Word2VecParams, JavaMLReadable, JavaMLWritable):
class Word2VecModel(JavaModel, Word2VecParams, JavaMLReadable, JavaMLWritable):
class PCAParams(JavaParams, HasInputCol, HasOutputCol):
class PCA(JavaEstimator, PCAParams, JavaMLReadable, JavaMLWritable):
class PCAModel(JavaModel, PCAParams, JavaMLReadable, JavaMLWritable):
class RFormulaParams(JavaParams, HasFeaturesCol, HasLabelCol, HasHandleInvalid):
class RFormula(JavaEstimator, RFormulaParams, JavaMLReadable, JavaMLWritable):
class RFormulaModel(JavaModel, RFormulaParams, JavaMLReadable, JavaMLWritable):
class ChiSqSelectorParams(JavaParams, HasFeaturesCol, HasOutputCol, HasLabelCol):
class ChiSqSelector(JavaEstimator, ChiSqSelectorParams, JavaMLReadable, JavaMLWritable):
class ChiSqSelectorModel(JavaModel, ChiSqSelectorParams, JavaMLReadable, JavaMLWritable):

Merged build finished. Test PASSed.

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

_CountVectorizerParams and _StringIndexerParams have _ . Shall we follow the same convention? @zhengruifeng

Test build #111304 has started for PR 25908 at commit 74de7a4.

Merged build finished. Test PASSed.

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

Test build #111304 has finished for PR 25908 at commit 74de7a4.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class LSHParams(HasInputCol, HasOutputCol):
class BucketedRandomProjectionLSHParams():
class IDFParams(HasInputCol, HasOutputCol):
class IDF(JavaEstimator, IDFParams, JavaMLReadable, JavaMLWritable):
class IDFModel(JavaModel, IDFParams, JavaMLReadable, JavaMLWritable):
class ImputerParams(HasInputCols, HasOutputCols):
class MaxAbsScalerParams(HasInputCol, HasOutputCol):
class MinMaxScalerParams(HasInputCol, HasOutputCol):
class OneHotEncoderParams(HasInputCols, HasOutputCols, HasHandleInvalid):
class RobustScalerParams(HasInputCol, HasOutputCol):
class StandardScalerParams(HasInputCol, HasOutputCol):
class VectorIndexerParams(HasInputCol, HasOutputCol, HasHandleInvalid):
class Word2VecParams(HasStepSize, HasMaxIter, HasSeed, HasInputCol, HasOutputCol):
class PCAParams(HasInputCol, HasOutputCol):
class RFormulaParams(HasFeaturesCol, HasLabelCol, HasHandleInvalid):
class ChiSqSelectorParams(HasFeaturesCol, HasOutputCol, HasLabelCol):

Merged build finished. Test PASSed.

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

Similar to the other PR, this PR adds setters/getters to feature models.

@huaxingao

Shall we follow the same convention?
I prefer to follow it in the following PRs.

Test build #111437 has started for PR 25908 at commit 425959e.

Merged build finished. Test PASSed.

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

Test build #111437 has finished for PR 25908 at commit 425959e.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class _LSHParams(HasInputCol, HasOutputCol):
class LSH(JavaEstimator, _LSHParams, JavaMLReadable, JavaMLWritable):
class LSHModel(JavaModel, _LSHParams):
class _BucketedRandomProjectionLSHParams():
class BucketedRandomProjectionLSH(LSH, _BucketedRandomProjectionLSHParams,
class BucketedRandomProjectionLSHModel(LSHModel, _BucketedRandomProjectionLSHParams, JavaMLReadable,
class _IDFParams(HasInputCol, HasOutputCol):
class IDF(JavaEstimator, _IDFParams, JavaMLReadable, JavaMLWritable):
class IDFModel(JavaModel, _IDFParams, JavaMLReadable, JavaMLWritable):
class _ImputerParams(HasInputCols, HasOutputCols):
class Imputer(JavaEstimator, _ImputerParams, JavaMLReadable, JavaMLWritable):
class ImputerModel(JavaModel, _ImputerParams, JavaMLReadable, JavaMLWritable):
class _MaxAbsScalerParams(HasInputCol, HasOutputCol):
class MaxAbsScaler(JavaEstimator, _MaxAbsScalerParams, JavaMLReadable, JavaMLWritable):
class MaxAbsScalerModel(JavaModel, _MaxAbsScalerParams, JavaMLReadable, JavaMLWritable):
class MinHashLSH(JavaEstimator, _LSHParams, HasInputCol, HasOutputCol, HasSeed,
class _MinMaxScalerParams(HasInputCol, HasOutputCol):
class MinMaxScaler(JavaEstimator, _MinMaxScalerParams, JavaMLReadable, JavaMLWritable):
class MinMaxScalerModel(JavaModel, _MinMaxScalerParams, JavaMLReadable, JavaMLWritable):
class _OneHotEncoderParams(HasInputCols, HasOutputCols, HasHandleInvalid):
class OneHotEncoder(JavaEstimator, _OneHotEncoderParams, JavaMLReadable, JavaMLWritable):
class OneHotEncoderModel(JavaModel, _OneHotEncoderParams, JavaMLReadable, JavaMLWritable):
class _RobustScalerParams(HasInputCol, HasOutputCol):
class RobustScaler(JavaEstimator, _RobustScalerParams, JavaMLReadable, JavaMLWritable):
class RobustScalerModel(JavaModel, _RobustScalerParams, JavaMLReadable, JavaMLWritable):
class _StandardScalerParams(HasInputCol, HasOutputCol):
class StandardScaler(JavaEstimator, _StandardScalerParams, JavaMLReadable, JavaMLWritable):
class StandardScalerModel(JavaModel, _StandardScalerParams, JavaMLReadable, JavaMLWritable):
class _VectorIndexerParams(HasInputCol, HasOutputCol, HasHandleInvalid):
class VectorIndexer(JavaEstimator, _VectorIndexerParams, JavaMLReadable, JavaMLWritable):
class VectorIndexerModel(JavaModel, _VectorIndexerParams, JavaMLReadable, JavaMLWritable):
class _Word2VecParams(HasStepSize, HasMaxIter, HasSeed, HasInputCol, HasOutputCol):
class Word2Vec(JavaEstimator, _Word2VecParams, JavaMLReadable, JavaMLWritable):
class Word2VecModel(JavaModel, _Word2VecParams, JavaMLReadable, JavaMLWritable):
class _PCAParams(HasInputCol, HasOutputCol):
class PCA(JavaEstimator, _PCAParams, JavaMLReadable, JavaMLWritable):
class PCAModel(JavaModel, _PCAParams, JavaMLReadable, JavaMLWritable):
class _RFormulaParams(HasFeaturesCol, HasLabelCol, HasHandleInvalid):
class RFormula(JavaEstimator, _RFormulaParams, JavaMLReadable, JavaMLWritable):
class RFormulaModel(JavaModel, _RFormulaParams, JavaMLReadable, JavaMLWritable):
class _ChiSqSelectorParams(HasFeaturesCol, HasOutputCol, HasLabelCol):
class ChiSqSelector(JavaEstimator, _ChiSqSelectorParams, JavaMLReadable, JavaMLWritable):
class ChiSqSelectorModel(JavaModel, _ChiSqSelectorParams, JavaMLReadable, JavaMLWritable):

Merged build finished. Test PASSed.

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

Test build #111493 has started for PR 25908 at commit 50e4467.

Merged build finished. Test PASSed.

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

Test build #111493 has finished for PR 25908 at commit 50e4467.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class MinHashLSH(LSH, HasInputCol, HasOutputCol, HasSeed, JavaMLReadable, JavaMLWritable):

Merged build finished. Test PASSed.

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

@srowen @zhengruifeng @huaxingao Could you explain the rationale behind 425959e? Leading underscore doesn't really have the same semantics as Scala package private (or private modifier in general).
However, when used in context which have direct impact on user facing API, it is quite annoying, as it requires manually segregation from truly internal components. Even if the goal is simple API parity it makes sense to take such things into account. Otherwise the whole thing could be just code generated. Just saying...

If the general question is, why hide things? just the usual software design idea, to only expose what you intend to support as a user or developer API. I'm not making a specific judgment about whether these should be exposed or not, just don't know yet.
If the question is, can we make this more extensible, I also don't have a strong opinion but am not against it, but think that can be considered separately.
There are also going to be a few follow-on PRs to this one to further remove some setters.

@zero323
If your question is specific for this leading underscore I just added, there are two reasons:

To keep consistent with the previous implementation in _CountVectorizerParams and _StringIndexerParams
Since Python doesn't really have private modifier, seems to  me the closest one is to have leading underscore specified in PEP 8 convention. https://www.python.org/dev/peps/pep-0008/
_single_leading_underscore: weak "internal use" indicator. E.g. from M import * does not import objects whose names start with an underscore.

I am neural on this leading underscore. Since I am following  Bryan's convention in  _CountVectorizerParams, I would like to check with him to see if there are any other reasons for this leading underscore.  Thanks in advance @BryanCutler

If the general question is, why hide things? just the usual software design idea, to only expose what you intend to support as a user or developer API. I'm not making a specific judgment about whether these should be exposed or not, just don't know yet.

Not at all @srowen. It is a specific question.  These mixins are used only to augment public API, and as such directly affect user's code (for example method resolution order), so it is a bit fuzzy why indicate them as internal.
From one hand their usage is limited to specific Estimator / Model pair, so there is really little value in exposing these to the end user. From the other, hand limited scope suggests little potential for change, especially one that doesn't affect public API, hence there seems to be no little harm in exposing these (the problem is similar to SPARK-7146).
In contrast to SPARK-29212, I am indifferent here and just trying to get a better feeling how the API evolves and what drives certain decisions.

@huaxingao. I understand leading underscore usage and I've seen Bryan's work, but for now it was an exception in pyspark.ml. We have a bunch of fairly isolated *Params but nothing indicates these are considered internal. Hence I was curious why the sudden change.

Test build #111754 has started for PR 25908 at commit bc25799.

Merged build finished. Test PASSed.

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

Test build #111754 has finished for PR 25908 at commit bc25799.

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

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

Test build #111767 has started for PR 25908 at commit c305a43.

Test build #111767 has finished for PR 25908 at commit c305a43.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class _LSH(JavaEstimator, _LSHParams, JavaMLReadable, JavaMLWritable):
class _LSHModel(JavaModel, _LSHParams):
class BucketedRandomProjectionLSH(_LSH, _BucketedRandomProjectionLSHParams,
class BucketedRandomProjectionLSHModel(_LSHModel, _BucketedRandomProjectionLSHParams, JavaMLReadable,
class MinHashLSH(_LSH, HasInputCol, HasOutputCol, HasSeed, JavaMLReadable, JavaMLWritable):
class MinHashLSHModel(_LSHModel, JavaMLReadable, JavaMLWritable):

Merged build finished. Test FAILed.

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

Test build #111768 has started for PR 25908 at commit 64fca95.

Merged build finished. Test PASSed.

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

Test build #111768 has finished for PR 25908 at commit 64fca95.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class BucketedRandomProjectionLSHModel(_LSHModel, _BucketedRandomProjectionLSHParams,

Merged build finished. Test PASSed.

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

Merged to master

Thanks all for your help!


Appending to local list after merging

Ive added the appending of merged content to the end of local list

verified and closing


Action composition?

In the JS-land Redux, one of the convenient things is having various patterns that allow you to dispatch actions based on actions dispatched. e.g thunks and more complex concepts like rx , sagas, epics in an asynchronous way that involves side effects.
What would be the equivalent here?

Hey Danny, could you give me a concrete example of what you would like to achieve?

At the very minimum, side effects in UI. e.g:
Launch an action that triggers an async function , respond with dispatching error/success actions.

And how is this usually accomplished? I guess an interceptor between dispatchers and reducers that filter out "fake" actions and trigger side effects? You could do this by implementing trait Reducer for some type that wraps your actual state. The library doesn't currently provide any convenience utilities for this though :(


Hostname verification fails behind a corporate proxy when connecting to Azure Service Bus with AMQP Web Sockets

Actual Behavior
Note: I actually reported that before in (https://github.com/Azure/azure-service-bus-java/issues/377) and was redirected here for issues with version 2.0.0. The response claims it's not a bug, but I feel the statement was "It's not a bug in 1.2.x because we don't do AMQP.".

I'm using com.microsoft.azure:azure-servicebus:2.0.0 behind a company proxy with TransportType set to AMQP_WEB_SOCKETS
I'm getting a hostname verification exception: the exception is just a "GeneralSSLProblem" but when I debug into it I can see that the client expects the server to have the name of the proxy, which leads to a hostname verification problem.
I can use the switch -Dcom.microsoft.azure.servicebus.ssl.verifymode=verifyCertificateOnly to make it work.

Analysis
I tracked that down to the class com.microsoft.azure.servicebus.amqp.ConnectionHandler which initializes the SslPeerDetails like that
SslPeerDetails peerDetails = Proton.sslPeerDetails(this.getOutboundSocketHostName(), this.getOutboundSocketPort());

Since the OutboundSocketHostName is the name of my proxy, that seems to be the root cause of my troubles. I changed the value in the debugger to the proper service bus address, and that worked.
I also noticed that in version 1.2.x (master1 branch of the old repository), the code is different: here it uses this.messagingFactory.getHostName() which seems the proper solution for me.
Reproduction
I assume you would be able to reproduce that, when setting up your own local proxy and trying to connect to Azure Service Bus using AMQP_WEB_SOCKETS.
Versions

OS platform and version:  2.0.0 on Windows (Oracle Java SE 1.8)

Hi @ChristianSchenk711 , thanks for reporting this. CCing @ramya-rao-a to help route this further.

tagging @yvgopal @nemakam to follow up on the customer-reported issue based on the CODEOWNERS file.  @yvgopal it looks like you already commented on https://github.com/Azure/azure-service-bus-java/issues/377

I have already commented about this issue on the thread https://github.com/Azure/azure-service-bus-java/issues/377. It is not a bug. You got it working with the appropriate flag, to be used specifically in cases like yours.
Verifying only the certificate, disregarding the name of the host presenting the certificate, is vulnerable to man in the middle attack.

So if I use a company proxy, I have to use the flag verifyCertificateOnly and then I'm vulnerable to man-in-the-middle attacks. So if this doesn't work, what is the way to securely connect to the Service Bus if I am behind a company proxy?

Honestly I don't have an answer. I don't know how it is done. But I think your company proxy should accept the certificate from service bus and present its own certificate to the client. I am no authority on this. If you have a suggestion, I will happily consider it.


:on-event-create does not work

When using :on-event-create with Click&Hold the function is not fired.
Affected version 3.5.1
3.4.1 is fine
https://codepen.io/vipw/pen/zYrBByO

Hey @vip9,
I just added :drag-to-create-event="false" and then it started to work.
Do you need the drag-to-create-event property to be true?
Hope it helps!

Thanks, I do not use drag to create for now. The trick did the job.
But found out (even on ver 5.4.2), that when created event, there's no "split" property in selectedEvent object.

Hey @vip9,
I am not sure if I understood your issue.
I think you need, always when you create an event, set the right split to it. Then it will have a specific split.
In your codepen, you haven't set the split to the event 'Swimming lesson'. Do it as a static way:
events: [
      {
        start: '2020-06-13 19:30',
        end: '2020-06-13 23:00',
        title: 'Swimming lesson',
        split: 1
      }
],

Or dynamically (in your controller, for example):
$events['split'] = the.id.of.splitDays;
Note: maybe you will have some issues about the split ordering. I was using the
$events['split'] = the.id.of.splitDays;
with the splitDays id being the same as the id of the split database table (that in my case is courts (sports)). The problem comes when you remove a court from your database, then vue-cal loses the reference to the splitDays id. This is, if you have splitDays: 1, 2, 3 and courts relating these splitDays, it will work, but if you have splitDays: 1, 3, it will lose the reference, because it needs the splitDay 2. This means the court 2 was deleted and vue-cal cannot handle with ids 1, 3 only. So if you add a new court to the database with id 4, when you reference splitDays 4 to the court 4, it will mess everything (1, 3, 4 cannot be handled by vue-cal).
Vue-cal needs the splitDays array to be in sequence.

The problem is not with viewing, but when creating new one.
In new one event  onEventCreate (event: any, deleteEventFunction: any) {;
event has no property "split".
Updated the codepen with latest vue-cal version.

Maybe @antoniandre can explain why it has no split in your case.
But, a workaround is to change :on-event-create to @cell-click="onEventCreate".
Then you have the date and split properties.
You can try to create an event (documentation example) by setting a ref="vue-cal" to your vue-cal instance and then:
$refs.vuecal.createEvent(
    $event,
    120,
    { title: 'New Event', class: 'blue-event' }
)

Hope it helps.

Thanks @giulianimartini for the help.
Thanks @vip9 for reporting, I believe this is a regression.
Will release a fix soon.

Hi @vip9,
As @giulianimartini said, :drag-to-create-event="false" is needed. It is not a trick: the default event creation method has changed since version 3.5.0 in favor of drag creation.
Regarding the rest of the problem, this was indeed a regression brought by the new drag creation method. I released a fix in version 3.5.3 and your codepen example is now working.
Thanks for reporting the issue.
Please reopen if not solved.


[SPARK-43979][SQL][FOLLOWUP] transformUpWithNewOutput should only be used with new outputs

This resubmits https://github.com/apache/spark/pull/42408 to 3.5

What changes were proposed in this pull request?

This is a followup of https://github.com/apache/spark/pull/41475 . It's risky to use transformUpWithNewOutput with existing attribute ids. If the plan contains duplicated attribute ids somewhere, then we will hit conflicting attributes and an assertion error will be thrown by QueryPlan#transformUpWithNewOutput.
This PR takes a different approach. We canonicalize the plan first and then remove the alias-only project. Then we don't need transformUpWithNewOutput anymore as all attribute ids are normalized in the canonicalized plan.
Why are the changes needed?

fix potential bugs
Does this PR introduce any user-facing change?

No
How was this patch tested?

existing tests

cc @yaooqinn

Thank you for the quick fix, @cloud-fan .


Add unit tests for PomUtil.java

I've analysed your codebase and noticed that org.xujin.moss.client.endpoint.dependency.util.PomUtil is not fully tested.
I've written some tests for the methods in this class with the help of Diffblue Cover.
Hopefully, these tests will help you detect any regressions caused by future code changes. If you would find it useful to have additional tests written for this repository, I would be more than happy to look at other classes that you consider important in a subsequent PR.

Made requested changes, plus renaming test and replacing unicode with string characters, and some further tidy-ups.

I'd remove the .java from the title as well. Usually we try to fit the qualified classname in the title but this one looks quite long.

Commit message amended, pull request message amended, commit history squashed.
Unable to get diffblue cover to generate tests for other classes. Possibly a platform issue.

Updated JUnit version, removed unused dependencies, fixed indentation to match rest of project, refactored test slightly.


changed default image pull policy to if-not-present

kpt fn commands support command flag called image-pull-policy that determines image fetch behavior for KRM functions. Current default value is AlwaysPull which means kpt will always try to pull the image using docker pull even if the function image is present locally. In addition, current code also calls docker inspect to check if the image is present locally for ifnotpresent.
This PR does the following:

Changes the default value for image-pull-policy to if-not-present i.e. pull image only if it doesn't exists locally.
Removes the docker pull and docker image inspect calls instead relays the image-pull-policy flag to docker run command as is. This will eliminate two docker calls and thus, make the fn render and fn eval faster.
To support the above implementation, we had to filter out docker CLI command  (especially progress messages). This was needed because output from executing KRM function and docker run will be emitted on same stderr output stream.

Code changes LGTM