Added Ukrainian translation

@Nollasko I've created a PR for you. Please verify everything's correct and in place, and let me know once you do so.

Please review the new changes, and let me know what you think.
If you'd like to test the built application from this PR, you can download the Windows build from the CI artifacts, at the bottom of the page here: https://github.com/DevilXD/TwitchDropsMiner/actions/runs/5298922029?pr=230

There are strings that can't be translated due to their absence in .json file

That's intentional. The About section is my own personal section, written with my own words. Translating it would be like "putting words I never said into my mouth", which is why there's no need nor way to translate those, and it should stay like that.
The "Language" option is untranslatable to allow people to find the language option and change it, even if they'd switch to a language they don't otherwise speak. Some apps translate the language change option too, and once the language switches, the user would be then unable to find the language change option again and change back otherwise.
The Twitch code message that's printed into the chat is meant to be translated at some point in the future - I haven't gotten around to making a translation key for it yet. For now, it can't be translated.

I've pushed out more changes as requested. Is it good now?

@DevilXD found one more untranslatable string.

Everything else seems right, u can close this.

found one more untranslatable string.

It's from an unfinished, in-progress feature that I haven't ended implementing properly yet. Again, translation to-be-added later. The dev version is not stable for a reason, nothing is guaranteed to work as it should.

Everything else seems right

Merging then. Thank you for your contribution 🙂


Add a plugin for (or at least a port of) security/tailscale

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[X] When the request is meant for an existing plugin, I've added its name to the title.

Is your feature request related to a problem? Please describe.
OPNsense is including a number of VPN/mesh options but is ignoring the currently best available solution Tailscale.
Right now the only widely used mesh VPN/overlay network/soft SDN I have found in large environments was Tailscale. Installing and maintaining it (it is actively developed with a lot of attractive features like mulvad VPN integration) is a bit of a nightmare because it requires golang so compiling it yourself from ports is time and resource consuming.
Please consider adding a plugin for it or at least add the package to the OPNSense repository.
Describe the solution you'd like
Right now the only widely used mesh VPN/overlay network/soft SDN I have found in large environments was Tailscale. Installing and maintaining it (it is actively developed with a lot of attractive features like mulvad VPN integration) is a bit of a nightmare because it requires golang so compiling it yourself from ports is time and resource consuming.
Please consider adding a plugin for it or at least add the package to the OPNSense repository. Adding the Port "headscale" might be a good idea, too, although it is lacking all the additional services that make Tailscale interesting.
Describe alternatives you've considered
Right now I'm adding it from ports but upgrading is cumbersome.

Why did a search using the search field not find  that entry? Proves that adding the package name to the subject header is helpful.

likely because it was asked at their end 🙂


tunnels

Add support for tunnels on roads and railways

I've been playing a bit with this issue: the current style does mention bridges, but draws them the same as the roads/rails. If I create a style for tunnels eg:
[tunnel=1] {
::case {line-color: #f00;}
}
I get this result:
http://dl.dropbox.com/u/175548/tunnelproblem.png
As you can see the red bits of 'Mauritskade' are a tunnel which should go under 'Rhijnspoorplein' However Mapnik doesn't know that they are of a different height and connects them. If you create a separate tunnel layer and put it under the roads layer you get the same style artefact as in the standard OSM style:
http://www.openstreetmap.org/?lat=52.359903&lon=4.908984&zoom=18&layers=M
any ideas how to solve this?

@ajashton - is this still relevant?

Migurski came up with a solution: https://github.com/migurski/HighRoad though it is meant for osm2pgsql style imports, my feeling is that it shouldn't be too hard to adept for imposm style imports

I've been making lots of progress on tunnels, bridges, and railways in the variations branch (with some inspiration from HighRoad). I'll be merging this into the main branch soon.

thanks, looking forward to your work


cap() returns always 0

I am not sure why this is happening, as I haven't digged deeper, however a call to cap() always returns 0.
This makes a lot of code just hang and breaks badly.

Probably due to #2?


New strings PT-BR

Added the strings of: https://github.com/DistributedCollective/Sovryn-frontend/pull/1640
Please merge @soulBit

Could you please merge this on master @creed-victor? Thanks.


docs(workbox): v4x renames addRequest to pushRequest

As of workbox v4, addRequest no longer exists.

Renames addRequest -> pushRequest.

Target Live Date: NA

[ ] This has been reviewed and approved by (NAME)
[ ] I have run npm test locally and all tests pass.
[ ] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

I signed it!

@jeffposnick Thanks!
Another thing, In docs it says:

⚠ DO NOT USE CHROME DEVTOOLS OFFLINE ⚠ The offline checkbox in DevTools only affects requests from the page. Service Worker requests will continue to go through.

But I was able to test it offline checkbox in DevTools and it worked like charm. So, is it still needed?

I believe https://bugs.chromium.org/p/chromium/issues/detail?id=938186 is related, and that just got fixed a few days ago. Are you testing using Chrome Canary by any chance?
CC: @philipwalton

@jeffposnick I am on Chromium stable (73.0.3683.103) which probably didn't contain this fix.

:tada: This has been pushed live to https://developers.google.com/web/


dnscrypt-proxy2: 2.0.45 -> 2.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/DNSCrypt/dnscrypt-proxy/releases.
meta.description for dnscrypt-proxy2 is: "A tool that provides secure DNS resolution"
meta.homepage for dnscrypt-proxy2 is: "https://dnscrypt.info/"
meta.changelog for dnscrypt-proxy2 is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/yl5bki5kya6z53kk9q6531j40cnz3d1s-dnscrypt-proxy2-2.1.0/bin/dnscrypt-proxy passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 2.1.0 with grep in /nix/store/yl5bki5kya6z53kk9q6531j40cnz3d1s-dnscrypt-proxy2-2.1.0

found 2.1.0 in filename of file in /nix/store/yl5bki5kya6z53kk9q6531j40cnz3d1s-dnscrypt-proxy2-2.1.0

directory tree listing: https://gist.github.com/6439391c98de768c94fffc4c6c169606

du listing: https://gist.github.com/79bfa58ad9ce4a42d4bfdb2d2863673c

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

3 package rebuild(s)

3 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
dnscrypt-proxy2
nixos-install-tools
tests.nixos-functions.nixos-test

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yl5bki5kya6z53kk9q6531j40cnz3d1s-dnscrypt-proxy2-2.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A dnscrypt-proxy2 https://github.com/r-ryantm/nixpkgs/archive/e239b5b539b5732f95e9253d09349a627b0f0df7.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yl5bki5kya6z53kk9q6531j40cnz3d1s-dnscrypt-proxy2-2.1.0
ls -la /nix/store/yl5bki5kya6z53kk9q6531j40cnz3d1s-dnscrypt-proxy2-2.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

nixos-install-tools

1 package built:

dnscrypt-proxy2

Maintainer pings
cc @Atemu @waynr @kalbasit for testing.

Result of nixpkgs-review pr 134102 at e239b5b5 run on aarch64-linux 1

2 packages built successfully:

dnscrypt-proxy2
nixos-install-tools

Result of nixpkgs-review pr 134102 run on x86_64-linux 1

1 package blacklisted:

tests.nixos-functions.nixos-test

1 package failed to build:

nixos-install-tools

1 package built:

dnscrypt-proxy2


Re-Did Kene101 Power Module

I rebuilt kene101’s power module. However, the client_detail.php code
in the WIP was much different than the code of the current version.
This code works with the current 2.2.0 - but not haven’t been able to
test on the WIP. All that was throwing me is that I don’t see the
bluetooth tab-pane anymore.

I think you need to update your repository. It is very behind the current munkireport repo.

My problem too.
Does this mean the Power Module has been completed?
My apologies and embarrassment for starting, and receiving Arjen’s good advice & comment and not completing.
If not complete I will continue.

—
Reply to this email directly or view it on GitHub.

Hi ken, your changes have been pulled and merged here: #205. Thanks!


return as html

Can u return pure html code

Set return type: 'string' and it will return a Promise where the string is HTML code


fix (x/y)ticklabelaling

Oops...for some reason I read labelalign when it should have been ticklabelalign.  My bad, sorry about that :D


K Largest element in array using heap

Find all the k largest elements are present in an array/vector. It will use heap data structure for solving.
example:-
arr=[1,2,3,4,5,6] and k=3
then the output will be:
[4,5,6] which are the 3 largest elements present in the array.

Languages and Assignees:

[x] Python: @aastha01chauhan
[x] Java: @PrabhuRaamDev
[ ] C: @
[x] C++: @Priya272

Notes
This code will be helpful as it will help us to find the range of all greater no present in an array.

@kaustubhgupta  please assign this issue to me!

Please assign this issue to me in java and/or python

I would like to contribute in java.Kindly please assign me

@aastha01chauhan @PrabhuRaamDev Pls check your language assignments


OCM-5715 | fix: add error message when supplying worker disk size on unsupported version

Related issue: OCM-5715

changes look good. for me
just wonder whether we can move the common func such as IsSettingRootDiskSizeSupported() into ocm-common so that it can be used in both client/server side.

/lgtm changes look good. for me just wonder whether we can move the common func such as IsSettingRootDiskSizeSupported() into ocm-common so that it can be used in both client/server side.

Considering we have multiple common functionality around this I'd say we should elect what needs to be transferred and capture it in an epic so we can have better tracking


[R] Parameter nround(s) in documentation

The parameter to set the maximum number of boosting iterations is defined as nrounds in the R-Package.
However in the documentation they are multiple instances of where it is just used as nround.
This works because R is doing argument matching.
Of course one can rely on it but it's not the best way and a bit confusing for beginners. That's why it should be nrounds everywhere.

Thanks for your report. Yes, the documentation should be updated. We'll gladly consider a pull request that fixes the typo.


Information Retrieval (2/5) [wip]

Before submitting

[X] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[X] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Implements Mean Reciprocal Rank for Information Retrieval. More info here.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @lucadiliello! Thanks for updating this PR.

In the file tests/retrieval/test_mrr.py:

Line 20:34: E261 at least two spaces before inline comment
Line 54:1: W293 blank line contains whitespace


Custom alias in Webpack error

This is related to the issue titled "Bug with custom alias in Webpack" #405. That issued is locked, so I had to create this one.
I read through that entire history and the difference for me is that I do not have any ES6 features in my webpack config. Furthermore, one alias (Router) works, but the other (CSS), does not.
As the others stated, eslint runs with no errors on CLI and webpack has no issues loading this import on my dev server.
Let me know what other information I can provide.
Error

webpack config
const webpack = require('webpack');
const UglifyJsPlugin = require('uglifyjs-webpack-plugin')
const path = require('path')

module.exports = {
  mode: 'production',
  devtool: 'source-map',
  entry: {
    ['2_main']: './src/client/index.jsx', // eslint-disable-line no-useless-computed-key
  },
  output: {
    path: `${__dirname}/public/scripts/`,
    publicPath: '/scripts/',
    filename: '[name].bundle.[contenthash].js',
  },
  module: {
    rules: [
      { test: /\.jsx?$/, use: 'babel-loader', exclude: /node_modules/ },
      {
        test: /\.md$/,
        use: [
          { loader: 'babel-loader' },
          { loader: 'react-markdown-loader' },
        ],
      },
      { test: /\.css$/, use: ['to-string-loader', 'css-loader'] },
    ],
  },
  optimization: {
    minimizer: [new UglifyJsPlugin()],
    runtimeChunk: {
      name: '0_runtime',
    },
    splitChunks: {
      cacheGroups: {
        vendor: {
          test: /[\\/]node_modules[\\/]/,
          name: '1_vendor',
          chunks: 'all',
        },
      },
    },
  },
  plugins: [
    new webpack.optimize.OccurrenceOrderPlugin(),
    new webpack.DefinePlugin({
      'process.env.NODE_ENV': '"production"',
    }),
  ],
  resolve: {
    alias: {
      Router: path.resolve(__dirname, 'src/client/router'),
      CSS: path.resolve(__dirname, 'src/client/css'),
    },
    extensions: [
      '.js',
      '.jsx',
      '.css',
    ],
  },
}

@polizz can you please provide me with a GitHub repository that I can clone that demos this. Would help tracking down why this fails for you?


docs: Add documentation for aya-gen

This change adds a short description of aya-gen, what does it do and how
to use it. It also uses an example eBPF program which is using
task_struct from its own vmlinux.rs module.
Signed-off-by: Michal Rostecki<EMAIL_ADDRESS>

@dave-tucker done, PTAL


Add french number tests

This PR adds french number tests.
Added

French number tests

Fixed
N/A
Breaking
N/A

When I run the tests, two strings look like equal, but when I serialize there is some weird invisible characters that seems to be included in the output of the NumberFormatter.
Can you please tell me if you have the same errors @michael-rubel ?
   FAIL  MichaelRubel\Formatters\Tests\LocaleNumberFormatterTest
  ✓ can format number based on pl locale
  ✓ can format number based on en locale
  ⨯ can format number based on fr locale

  ---

  • MichaelRubel\Formatters\Tests\LocaleNumberFormatterTest > can format number based on fr locale
  Failed asserting that two strings are equal.

  at tests/LocaleNumberFormatterTest.php:97
     93▕             'number' => 10000,
     94▕             'locale' => 'fr',
     95▕         ]);
     96▕
  ➜  97▕         $this->assertEquals('10 000,00', $result);
     98▕
     99▕         $result = format(LocaleNumberFormatter::class, [
    100▕             'number' => 10000.00,
    101▕             'locale' => 'fr',
  --- Expected
  +++ Actual
  @@ @@
  -'10 000,00'
  +'10 000,00'

  Tests:  1 failed, 15 passed
  Time:   6.53s

When I serialize both values, here is what I get (first is my string, second is the NumberFormatter output):
  --- Expected
  +++ Actual
  @@ @@
  -'s:9:"10 000,00";'
  +'s:11:"10 000,00";'

Hello @khalyomede
It adds NBSP character to the output number.
You could see it if you use PHPStorm:

Scrutinizer stands "You shall not pass" on this test despite it works on GitHub Actions and locally. I tried to clear the Scrutinizer's cache but got no luck. I've reported this issue (probably encoding-related) so we'll wait for their response first.

Scrutinizer has solid trouble with narrow break space.
I had to trim all special characters for this particular test.


feat: Support CDK v2

Upgrade projen to latest version
Upgrade CDK dependencies to v2
Adopt tests to new assertions lib

BREAKING CHANGE: Not compatibe with CDK v1 anymore
Fixes #11

Hey! Just a status update: Sorry I did not have time last weekend to check this out and I'm traveling due to work next few days. I'll try to take a look into this at the end of this week or the weekend latest.
The multi-branch / multi-version (to support both v1 and v2 CDK) deployment seems like a good idea, but I need to take a look into that.


No run Install on Windows 10 Pro + IIS

Describe the bug
I have downloaded the file publish-win-x64.zip and created a website in Windows 10 Pro + IIS + IP: PORT. I have also installed dotnet-hosting-3.1.4-win.exe. But when launching in the browser http://<IP_ADDRESS>:9696/init and configuring the connection to the SQLEXPRESS database generates an error: either a trusted connection or also the remote connection.
To Reproduce
Steps to reproduce the behavior:

Download file publish-win-x64.zip. Copy / Paste all files to folder c:\WWW\MixCoreDemo
Install  dotnet-hosting-3.1.4-win.exe.
Create a web site pointing to that folder with IP:PORT, and configure AppPool like UNMANAGED CODE.
In Firefox Dev browser point to http://<IP_ADDRESS>:9696/init, and fill the form with the installation data requested by the application
Using Sql Server Management Studio create a sqlExpress database with the same name that the application shows, for example, 2405-mix-cms
Click Next button

Expected behavior
Complete installation step 1.
Screenshots

Desktop (please complete the following information):

OS: Window 10 Pro x64 1809 updated to date
Browser: Firefox Dev 77

Additional context
Database installation is successful only when IIS is stopped and run as administrator Mix.Cms.Web.exe.

@devhomecali  could you help reattach the image files? Thanks a lot.

@Smilefounder now the images are ok. Thanks.

Hi @devhomecali , I just want to double-check if you have installed localdb SQLExpress in your local machine or server?


feat(ONYX-1458): add missing tracking data

The type of this PR is: Feat

This PR resolves ONYX-1458
Description

Add missing tracking data for tracking that will be added in this PR https://github.com/artsy/eigen/pull/11328
Saves:

subtitle link

    action: tappedLink,
    context_module: saves,
    context_screen_owner_type: saves,
    type: "link",
    subject: signalYourInterestToGalleries

read more link

    action: tappedLink,
    context_module: saves,
    context_screen_owner_type: savesInfoModal,
    type: "link",

Favorites:
    action: tappedInfoBubble,
    context_module: follows,
    context_screen_owner_type: follows,
    subject: "followsHeader",

Alerts List:

title

    action: tappedInfoBubble,
    context_module: alertsList,
    context_screen_owner_type: alerts,
    subject: "alertsHeader",

Activity link

    action: tappedLink,
    context_module: alertsList,
    context_screen_owner_type: alertsInfoModal,
    type: "link",

PR Checklist (tick all before merging)

[x] If I've added a new file to the tree I've exported it from the common index.ts
[x] I've added comments with examples for any new interfaces and ensured that they're in the docs
[x] No platform-specific terminology has been used outside of click and tap (platform is inferred by the DB storing events)

:rocket: PR was released in v4.221.0 :rocket:


restructuring into cmd and pkg main folders

moved main.go to cmd/beanstalkg
moved architecture, backend, and operation to cmd/beanstalkg

did change some paths, not sure if I did cover all

This is intended to follow https://peter.bourgon.org/go-best-practices-2016/ . @stephenwithav please comment on whether this makes sense to you.

@stephenwithav modified;
guys, anything you need me to change?

@heinrich10 everything looks good to me.

@stephenwithav thanks.
Guys, have to remove the last line in .gitignore, any reason why you added it in the first place?
it ignores the cmd/beanstalkg/main.go file. added it back now

@heinrich10 beanstalkg was added to ignore the beanstalkg executable.  Instead of removing it, please change it to /beanstalkg, which will work as intended.
https://git-scm.com/docs/gitignore

@heinrich10 does this build and run without issue now?

@vimukthi-git it's going to need rebasing again since #81 was merged.

@stephenwithav I meant independent of that, if this branch works if I check out.

@vimukthi-git it does.  The only change needed is to re-add beanstalkg to .gitignore as /beanstalkg.

@vimukthi-git integration test is running normally, I changed the path to execute $GOPATH/bin/beanstalkg
@stephenwithav seems like we cannot import getConfig, so I temporarily copied the files to main.go, I will create an issue for this, please help to resolve, thank you very much

@heinrich10 All lowercase identifiers are private, meaning they're only accessible inside their own package.
Revert to a1e6cf, change getConfig to GetConfig, and you'll be good to go. :)

@vimukthi-git @stephenwithav finally able to change it. probably test will fail due to directory structure, should we merge first?

I'll test it over the weekend or on Monday.  I suffered a seizure on Wednesday, so I need to reorient myself.

@stephenwithav no worries, hope you are fine now. please have more rest.


Add stdexec threading backend

A new threading backend based on stdexec, which is proposed as the prototype for the new std::execution parallelism model for the C++ standard.
It supports asynchronous execution with scheduler backends based on coroutine thread pools or GPU schedulers, and could be an alternative or basis for #2831.
Description
A new MultiThreader class that that uses stdexec/execution.hpp
Impact analysis
CPU-based parallism could be improved and this could provide GPU parallelism support throughout the toolkit.
Expected behavior
If benchmarking succeeds and support is available across platforms and toolchains, use the new MultiThreader class as the default with the exec/static_thread_pool.hpp. Optionally provided support for NVIDIA GPUs if available.
Versions
Target for ITK 6.
Additional Information
Eventually the header-only library could be vendored in the ITK source tree in ThirdParty/.

@N-Dekker you might be interested in this :-D


Device freezes with GTK 1 is already in use

wpa: GTK 1 is already in use in entry 3, it may be an attack, ignore it.

Refs: https://esp32.com/viewtopic.php?t=10295

Traceback (most recent call last):
  File "main.py", line 56, in <module>
  File "main.py", line 47, in main
  File "aqi.py", line 362, in Run
  File "aqi.py", line 320, in GetData
  File "m5stickc.py", line 246, in GetURI
  File "nurequests.py", line 28, in text
  File "nurequests.py", line 20, in content
KeyboardInterrupt:
MicroPython v1.12-574-ge2b117096-dirty on 2020-06-05; M5Stick with ESP32

    @property
    def content(self):
        if self._cached is None:
            try:
                self._cached = self.raw.read()     <<<<

So read needs a timeout.

https://docs.micropython.org/en/latest/library/uselect.html
Example with socket here


Is CC1101 Raspberry Pi SPI Modul V0.3 supported?

https://shop.codm.de/module/3/cc1101-raspberry-pi-spi-modul-v0.3?c=5
Thanks!
Jorge

No it is not. It only works with the RF modules listed in the README and sold by eQ3. However, using the CUxD Addon software it might be supported somehow.


Should return index in onChange

Hello
Thanks for the perfect DropDown.
I think the DropDown's onChange function should return the index value too onChange={(item,index)=>{}}

hi @ydv0121 ,
You can filter the data list to find the index.

this actually prevents using async options which would be cool with multi dropdown search... search is not only convenience of the front, it could be used to optimize stuff. would love this feature

Please implement this feature if possible. I am also looking for it. Instead of looping through the list


Fix readme requirement name

I've checked on both SPM master and the swift-5.2 branch:
it seems that the Package.Dependency.Requirement name is always .upToNextMinor(..).

Absolutely right, thanks @zntfdr!


support service principal

https://learn.microsoft.com/ja-jp/azure/developer/python/sdk/authentication/local-development-service-principal?tabs=azure-cli


Apache Echart plugin allow to execute any arbitrary javascript function which can be used to perform unauthorized actions

We are using Apache Echart latest version and we observed when we add below code in Function section it gets executed.

Do we have any solution to prevent the execution of such a code?
Server: Grafana 9.5.3 OSS

+1

+1

@ajinkya-mulay Please take a look at the Security Guidelines: https://volkovlabs.io/plugins/security/
Javascript in Grafana runs in a Sandbox, which prevent some malicious executions. If it's a concern, please use native plugins without JS customizations.

@mikhail-vl Thanks for the response. As per security guidelines admin and editor is allowed to add the javascript code. but editor can also add the such a arbitrary functions. is there any plan to validate the code return in the textbox before running it?
alternatively from where I can take the native plugins? which is exactly same as this plugin functionality wise?

@ajinkya-mulay We don't plan to validate code, which as I mentioned executing in a Grafana's Sandbox. Apache Charts panel used for a wide range of use cases and validating code can limit its existing functionality.
Regarding, native panels, there is no one panel which can provide the same functionality as Apache ECharts. Depends on the use case, you can use Trend, 3D, Time Series, Pie Chart.
Alternatively, you may look at Plotly panels, they also use JavaScript and have the same security concerns.


Add Tensorflow eager execution mode example.

Add an example code of Tensorflow eager execution mode under examples/.

Would like to pick this up.

Okay, thanks!

Merged: https://github.com/pfnet/optuna/pull/369


Got error when opening atest-desktop on Windows

Description:

What issue is being seen? Describe what should be happening instead of
the bug, for example: api-testing should not crash, the expected value isn't
returned, etc.
As mentioned in the title.

Repro steps:

Include sample requests, environment, etc. All data and inputs
required to reproduce the bug.
Build the desktop package with the following command:

cd console/atest-desktop/
npm i
npm run package

then open it from console/atest-desktop/out/atest-desktop-win32-x64/atest-desktop.exe

Note: If there are privacy concerns, sanitize the data prior to
sharing.

You can pay attention to this line https://github.com/LinuxSuRen/api-testing/blob/b40eea5afaca9e873ece02e54df652d802af1cda/console/atest-desktop/main.js#L18
Environment:

Include the environment like api-testing version, os version and so on.

Logs:

Include the access logs and the api-testing logs.

kindly to ask if there is somebody working for this issue?if not,i'm glad to undertake it

Thanks for your attention. Please go ahead.
/assign @DWJ-Squirtle

I see, the issue seems to be related to the line of code ‘app.dock.setIcon(path.join(__dirname, "/api-testing.png"))’. This code is specific to macOS systems and cannot be used on Windows.

It should be.

It should be.

Simply commenting out that line of code should not affect the operation, but the exe file still cannot be started and there is no prompt. I want to know where the log file is so I can further check the problem. I hope you can tell me.

You can add a condition check. For instance: process.platform === 'darwin'. You can find the log file below:
log.transports.file.resolvePathFn = () => path.join(atestHome, 'log.log');


https://legend.finos.org/docs/getting-started/maven-install.md ==> 404

This link is on the main installation page.
Maven Install

This option is for software developers who want to build and run Legend from source. Instructions can be found [here](https://legend.finos.org/docs/getting-started/maven-install.md).

@saraswat Thanks for spotting this! We will fix this broken link. The Maven installation guide is right below so there's no need to click on that link to go anywhere


How to deal with the re-implementation of numpy functions depending of C functions?

As per title, is there / what is the procedure to re-implement numpy functions that depend on C functions, e.g. numpy.nonzero, depending on PyObject_NonZero?

In general, JAX implements all of its NumPy operations in terms of the primitive operations in lax (or in terms of other NumPy operations).
Rather than explain in detail, I thought it was probably quicker just to implement it:
https://github.com/google/jax/pull/1905
Take a look: does it make sense how it works?
Note that nonzero is unusual in the JAX world because its output shape is data-dependent. This means it cannot be jitted unless we add more support for dynamic shapes to JAX. However, it's probably still useful to have an implementation even if it cannot be jitted.
I hope that helps!

Thanks @hawkinsp, that example implementation is very clear.
Are you maintaining a list on non jittable methods, or just implementing it with those limits?


Added solution to N Queen problem in Java

Description
Added the solution to the N Queen problem in the Java language. I tried to print all possible solutions to this problem using recursion and backtracking. I have explained the code using comments wherever required.
Fixes #227
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

@surajm-333 what is the other file added?

@surajm-333 what is the other file added?

Other file is the class file @Amit366

@surajm-333 add everything in one file


TypeError: document.getText is not a function

Hi,
thank you for the plugin, it seems very cool.
I have an error when I run (coc-codeaction-line) then Add docstring for "Line or Selected" by pydocstring:
Error on request (doCodeAction): Vim(return):Error invoking 'doCodeAction' on channel 3 (coc):
TypeError: document.getText is not a function

note that CocCommand pydocstring.runFile works well.
Config:
nvim : v0.8.0-dev+448-g7b2b44bce
coc-pydocstring: 0.7.3
coc is up to date
doc is installed by CocCommand pydocstring.install

Thanks for the issue report. I am unable to reproduce this issue in my environment.
Can you give me the Pythoh code to reproduce?

a simple code with which I can reproduce :
def hello(param: str = "world"):
    print(f"hello {param}")

steps:

set cursor on the 'h' of 'hello'
trigger (coc-codeaction-line)  (I use :Telescope coc code_actions for that)
the floating window with codeactions-line is shown
it contains the line Add docstring for "Line or Selected" by pydocstring

select this line then press <CR>
the error happens

is the function document.getText must be in CocCommand ? I don't have it :

Maybe you have some dependency that I didn't have installed...

Does it work correctly when run without telescope?

No it fails by the same way when I call (coc-codeaction-line) :

[coc.nvim] Error on notification "doCodeAction": TypeError: document.getText is not a function
But it works well when I call CocCommand pydocstring.runFile for the whole file.

I cannot reproduce this in my environment.
DEMO (mp4):
https://user-images.githubusercontent.com/188642/177046257-c60bbdc4-6d1c-41cd-8307-42be60938f8e.mp4

I have fix the problem by changing two lines in function "activate"
I am not really confortable with js and COC lib and I'm not able to explain the bug, but now it seems to works well.
  context.subscriptions.push(import_coc4.commands.registerCommand("pydocstring.runAction", async (document, range) => {
    const doc = import_coc4.workspace.getDocument(document.uri);
    let edits;
    const code = await doFormat(context, outputChannel, doc.textDocument, range); // document => doc.textDocument
    if (!range) {
      range = fullDocumentRange(document);
      edits = [import_coc4.TextEdit.replace(range, code)];
      if (edits) {
        return await doc.applyEdits(edits);
      }
    }
    if (doc.textDocument.getText() === code) { // document => doc.textDocument
      return;
    }
    edits = [import_coc4.TextEdit.replace(range, code)];
    if (edits) {
      return await doc.applyEdits(edits);
    }
  }, null, true));

Thanks for looking into the fixes. However, I think it is probably occurring because of the particular way you are executing CodeAction.
Would the same error occur if you execute a code action in your environment with the following mappings?
Example key mapping:
nmap <silent> ga <Plug>(coc-codeaction-line)
xmap <silent> ga <Plug>(coc-codeaction-selected)

OK !
I already tried by this way but I had the same issue.
but I removed a plugin that wrapped coc-action (weilbith/nvim-code-action-menu) and now it works (but not with telescope).
But maybe I could be interesting to make coc-pydocstring compatible with theses plugins.
Thanks for your help 👍🏽 .


Crashes.GetErrorAttachments doesn't upload data, not documented

Crashes.GetErrorAttachments - ErrorAttachmentLog.AttachmentWithText does not upload data.
I'm providing correct file path to the file, but the web portal only shows the "key".
Is there a functional example that actually proves that this functionality works?
2)
The documentation is really slim and could use additional details for these methods. For example:
ErrorAttachmentLog.AttachmentWithBinary - what are the allowed values for contentType? contentType should be an enum instead.
3)
ErrorAttachmentLog.AttachmentWithText - should not require a file. That should be in the ErrorAttachmentLog.AttachmentWithTextFile.
Instead, ErrorAttachmentLog.AttachmentWithText should take a dictionary style Key-Value pairs, or an actual Dictionary<string,string>
While it is nice to have ability to upload a picture, in practice what we want is to tag crashes with data that is relevant, so for example:

unique app run id as a foreign key in the non MS hosted analytics DB;
app version string;
string representation of unique app state that would help track down the issue.
Saving this data to disc in a text file so that Crashes engine can read text file is very impractical; which is why passing strings to ErrorAttachmentLog is much nicer.

@dstulic I'll have the PM team take a look. In the mean time, if you're having trouble getting something to work, just reach out to our support team, they'll help you out. App Center support is provided directly from within the App Center portal. Any time you need help, just click the blue chat button in the lower-right corner of any page and our dedicated support team will respond. For additional information, see the App Center Help Center.

@jwargo I would, but it takes them a day to respond and quality of response is really low... :/

@dstulic well, that's the only way to get support. The comments feature of our documentation is intended to enable customers to suggest changes to the docs or ask clarifying questions about the docs. I've asked the PM team to take a look at your questions and address any potential docs issues.

@jwargo i understand your point of view, and I am talking to support team as well. So while that is slowly moving, I decided to ask you to improve on documentation. Specifically, I've asked you to:

provide a working example of AttachmentWithText
provide information for ** contentType** in AttachmentWithBinary
and suggested to add a string key value pairs as an attachment; as opposed to files. Here I provided you with a real world example of what I would like to upload: unique app run id as a foreign key in the non MS hosted analytics DB, and app version - both as strings.
These are the needs of my project, which is currently published to 100K+ users. I suspect that there will be other developers that have similar needs. My hope is that I can help you improve your service including documentation aspects, before many other developers jump in to use it.

Hi @dstulic,
Thanks for your feedback! I plan to improve our documentation this week so stay tuned :) It's really helpful to hear this kind of feedback directly from our users and I appreciate you taking the time to write this up.

@dstulic Here is the documentation for AttachmentWithText  API - https://docs.microsoft.com/en-us/appcenter/sdk/crashes/xamarin#add-attachments-to-a-crash-report
We support all the binary content like png, jpeg, gif, zip, doc etc. for AttachmentWithBinary API.

@dstulic it's been 18 days since the last response, so I'm going to close this issue.

@dstulic Have you fixed the issue? I'm facing the same issue right now and I'm really interested in working example.
Could you please post it here or describe whether file should be located?

I've found an answer. I don't know how to use AttachmentWithText but after the reading https://www.respawnsive.com/playground-part-3-catch-it-trace-it-log-it-and-upload-it I realised I'm able to use AttachmentWithBinary and just send bytes of the file.
I hope it'll help the others


ES index stopped working on Legacy

After updating Legacy with latest Arches Core dev/7.4.x the ES reindex stopped working. It started consuming a huge amount of resources and then was killed. Need to determine why this is happening. and resolve.
Currently reverted to an older version of the Arches Core for database reindex. Seems to work fine after DB reindex is complete.

This has been traced back to a specific Arches Core commit. Currently working with the Arches Core dev team to rectify. For the moment have reverted the change in a single file to allow reindexing to continue.

Fix was merged and is now part of dev/7.4.x


Allowing special characters in request url

From @joetherod on April 25, 2018 19:6
In MVC 4.5, there are web.config params to allow special characters in the url request.

Currently, in MVC Core app that runs locally in Linux via Docker for Windows, fails on HTTP 400 when the url contains a accented character. This application was originally written in MVC 4 and was ported to MVC core. So, the request works in MVC 4 on Windows. Now it doesnt locally and when hosted in Linux via Docker.
eg Request: www.contoso.com/search/Paris%2C Île-de-France Region
Original string was encoded in javascript before sending request, "Paris, Île-de-France Region"
The request doenst even make it to the app or any middleware that I have.
And the application breaks the website, it doenst come up at all. It is like stuck once I request this url.
Copied from original issue: aspnet/aspnet-docker#414

Likely a duplicate of https://github.com/aspnet/KestrelHttpServer/issues/1144

Can you share a Fiddler or Wireshark trace to confirm?

in which version will this issue get fixed? any plan?

Hi. We're closing this issue as no response or updates have been provided in a timely manner and we have been unable to reproduce it. If you have more details and are encountering this issue please add a new reply and re-open the issue.


Jupyter notebooks changes auto sync to exported HTML/Python/PDF

Would be nice if exported HTML/Python/PDF are auto synced with the notebooks changes, perhaps when the files share the same names and live in the same folder. Could also be an option when exporting.

Thanks. This should be a dupe of https://github.com/microsoft/vscode-jupyter/issues/8978


NIFI-8971 Removed support for legacy attributes for NiFi 2.0

Summary
NIFI-8971 includes the following changes...
NIFI-8971 Removed nf.file.name, nf.file.path, content-encoding, content-type from MergeContent, UnpackContent, ListenHTTPServlet
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[NIFI-8971] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-0000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-0000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[x] Build completed using mvn clean install -P contrib-check

[x] JDK 8
[x] JDK 11
[ ] JDK 17

Licensing

[x] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[x] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[x] Documentation formatting appears as expected in rendered files

Thanks for the contribution @aakash17kc!
As mentioned in the associated Jira issue, these changes should be part of NiFi 2.0. At this time, however, there is no timeline set for the next major release.  For that reason, this falls into the category of issues to resolve eventually, but not yet ready to be incorporated.
There are many other open issues, however, so feel to ask on the NiFi dev mailing list of Slack channel if you have questions.


Provide commands instead of mappings by default

There is quite some asymmetry between functionality that is provided by python functions, functionality that is provided by Vim commands and stuff that is accessible through mappings.
I suggest we provide a uniform set of Vim commands (or Vim functions, I don't mind, they might be easier when handling arguments) that provide all things that the user should be able to with vdebug.  The documentation should then encourage the user to only use these. The python bindings should be hidden internally (I think they already are). If we provide commands for all functionality the plugin will also be easier to explore for new users: They can type :Vdebug<TAB> and see what commands are suggested – and be sure that that is all you can do with the plugin!
If we want to provide default key mappings they should map to the aforementioned vim commands/functions directly! That way they are easier to understand for users. Also they are easier to mimic in case a user wants some custom mappings.
Currently we have g:vdebug_keymap which could be used for custom mappings. But the keys in that dict are used nowhere else in the public interface. We could keep this dict and try to adjust it to the new commands, or we can replace it with a simple boolean var like g:vdebug_use_default_mappings which the user can set to false in order to just map stuff individually.
The help file also lists several commands in section 4.3 that are not available (for me?). For example VdebugCommandStepOver which is only accessible via <F2> (but as mentioned above the key for that is called step_over and not VdebugCommandStepOver).
What do you think about providing a uniform interface and better exploreability for vdebug? What names would you suggest for the commands?

<Plug> mappings would be a good call here


Update awesome.yaml

Fix missing repos.
It was a mistake. I added the URLs and not the repos.  Apologies

👀


Update to Rails 6 (WIP)

Checklist:

I have performed a self-review of my own code,
I have commented my code, particularly in hard-to-understand areas,
I have made corresponding changes to the documentation,
I have added tests that prove my fix is effective or that my feature works,
New and existing unit tests pass locally with my changes ("bundle exec rake"),
Title include "WIP" if work is in progress.

Resolves #1181
Description
Start with rails app:upgrade. However, this inflicts many changes, most of which are unnecessary or clobber needed app functionality or setup. So I went through all changes and discarded such changes, confirming the app still runs and tests pass.
I reviewed https://edgeguides.rubyonrails.org/6_0_release_notes.html for breaking changes and search the app for any occurrences; there were none.
I also discarded most formatting / text / commenting changes to keep the PR easier on the eyes.
For a review of the changes that are caused by rails app:upgrade, reference https://github.com/rubyforgood/diaper/pull/1204
Labelled WIP because I think it would be wise to manually test various app setups and features in addition to relying on rspec tests, and I am not fluent enough in this app to do so with confidence.
I attempted to remove action_text, which is now included in Rails, but ran into issues with the action_text file not being included, so punted on that for now.
Type of change

Software upgrade

How Has This Been Tested?
Tests pass. Poked around app. Should poke around more.

Closing -- multiple tests now failing (108 failures) after actually upgrading gems (not just running app:update.

Repoened: most failures were due to an ActionView change that break rspec <4. Upgraded to rspec4.0.0.beta3 to fix; this decision should be weighted against the need to upgrade to Rails 6. Comment from rspec folks is that 4 will be released soon. See https://github.com/rspec/rspec-rails/issues/2086


Fix log contacts bug

The log contacts page now shows "anonymous" when trying to log contacts instead of contact name

Thanks for the eagle-eyes @lisabang  👀 .
Keep it up and when you spot something that needs fixing make a ticket here:
https://github.com/ContactTracing-app/Frontend/issues/new/choose
👍


disqus.js was not copied into static folder

make command is running find, and the code is getting inject into the html.
the disqus.js file however, is not showing anywhere.

This is my repo
https://github.com/gtalarico/revitpythonwrapper
Thanks for the great pkg!

Fixed by making my own disqus.js with universalcode from admin page.

Hmm, seems like if you comment out html_static_path = ['_static'] in https://github.com/gtalarico/revitpythonwrapper/blob/master/docs/conf.py#L183 the file is copied over. That's the default directory so that should be a quick fix for you.
I feel like this may still be a bug in this extension. I'll look into handling this case so that people with custom static folder names can still use my extension.
Thanks for reporting this!

Commenting out html_static_path = ['_static'] did not work. I had to create my own disqus.js file and include it in my _static folder like @gtalarico suggested to get Disqus working.

No longer able to reproduce, even with the old version of Sphinx and pip installing your requirements file from 2017. Reopen if this is still an issue.


README - list new columns

@kschmid I will improve README to answer your questions. If you still have questions, throw them at me.

I tagged you because you seemed confused how this  caching worked.

I can allow changing cache location, like with yfinance.

yeah, was a bit confused with how it works with yfinance_cache, but also with caching in general. (Very new to the python ecosystem)
I found the possibility to move caches around rather helpful. In particular while experimenting.
(And I am personally not a fan of software which simply puts large junks of data somewhere. )

And I am personally not a fan of software which simply puts large junks of data somewhere.

What I do is normal, and cache must go somewhere. Many programs on Linux dump data in the same folder as YFC, and some more than YFC e.g. PIP's cache. Can't speak for Mac but probably same - maybe you should look.


Create a user search history function

Add drop down list and dynamically add value functions

ah silly me @kevinvongmany, I tested the code and played around with it and commented out a piece of code that clear out the the existing value when adding the a new value in. So the it will not loop again when add new value


Fixed a bug when decoding a key with one character only

I found an issue with the convertFromCapitalized key decoding strategy which threw an error when the key had just one characted. This PR should fix the issue.

Thank you @TheFlow95, great catch! I've also added a test that reproduces the issue.


a question about training on the charades-sta dataset

hi, authors, great works, now I want to train the model on the charades-sta dataset, and I find that you provide an 'opt.json' file about hyper-parameters in https://github.com/jayleicn/moment_detr/issues/11, in the configure file, you set the parameter 'clip_len' is 2, what does it mean?

ok, got it thanks!!!


Update automatic to <IP_ADDRESS>4

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 73e314c086c9198d0a0175175f38aef515861da1 to keep commit history cleaner. Your contribution is still credited to you.


feat: hot server reload & debugging over docker

Changes proposed in this pull request

adds hot sever reload on save to auth/backend docker services
exposes debugger with instructions on how to use in most development-environment-agnostic way

Context

fixes #2650
fixes #2672
Checklist

[x] Related issues linked using fixes #number
[ ] Tests added/updated
[x] Documentation added
[x] Make sure that all checks pass
[ ] Bruno collection updated

It's not a factor in these changes, but I noticed we have docker-compose in our services, like rafiki/packages/backend/docker-compose.yml. Why?
As far as I can tell they are unused. Do we need them? Should I get rid of them?

I considered adding a CI step for checking that frontend starts OK with the production image. This is not trivial to setup since frontend depends on quite a few things (kratos, backend db, etc.). It would also be completely redundant if we
add any sort of frontend e2e test as part of this issue: https://github.com/interledger/rafiki/issues/2627

using ts-node instead of explicit build step. Never really liked the idea but its the more common use-case with nodemon. Tried it briefly but ran into some typescript errors and didn't pursue it further.

@BlairCurrey curious to hear your thoughts on debugging the complied JS files or the TS files directly.
I tried using ts-node-dev in this commit, and it ended up working quite well with the following launch.json for the cloud-nine-backend debugging (with the remote root pointing to the volume you exposed:
    {
      "name": "Attach to docker (cloud-nine-backend)",
      "type": "node",
      "request": "attach",
      "port": 9229,
      "address": "localhost",
      "localRoot": "${workspaceFolder}",
      "remoteRoot": "/home/rafiki/",
      "restart": true
    },

Let me know what feels better/what issues you had when trying TS, but it was quite nice debugging the files we end up writing.

using ts-node instead of explicit build step. Never really liked the idea but its the more common use-case with nodemon. Tried it briefly but ran into some typescript errors and didn't pursue it further.

@BlairCurrey curious to hear your thoughts on debugging the complied JS files or the TS files directly.
I tried using ts-node-dev in this commit, and it ended up working quite well with the following VS code launch.json for debugging cloud-nine-backend (with the remote root pointing to the volume you expose):
    {
      "name": "Attach to docker (cloud-nine-backend)",
      "type": "node",
      "request": "attach",
      "port": 9229,
      "address": "localhost",
      "localRoot": "${workspaceFolder}",
      "remoteRoot": "/home/rafiki/",
      "restart": true
    },

Let me know what feels better/what issues you had when trying TS, but it was quite nice debugging the files we end up writing.

Thanks for sharing this - seems promising. I like that we dont have to copy files every chance (since they arent being wiped out by building).


feature(Order/SupClosed): Add lemmas for inserting upper or lower bounds into a Sup/Inf closed set

Add lemmas for inserting upper or lower bounds into a Sup/Inf closed set.
e.g. When s is sup closed, so is s with ⊤ or ⊥ appended. Similarly for s inf closed.

I thought I was going to need this in #18705, but I found a better way. This PR may still be of independent interest though?

Oh, my maintainer delegate didn't go through.
maintainer merge

bors r+


Improve Accessibility score

Describe the bug
Fix the following:

Image elements do not have [alt] attributes
Links do not have a discernible name
Background and foreground colors do not have a sufficient contrast ratio.
[id] attributes on active, focusable elements are not unique
Heading elements are not in a sequentially-descending order

To Reproduce
Steps to reproduce the behavior:

Go to DevTools
Run Lighthouse analyser
List of issues are under "accessibility"

Expected behavior
Increase Accessibility score by fixing the issues
Screenshots

Please, Assign me this task.


.mock doesn't exist

💬 Questions and Help
Hi, I'm new to Jest. I was following the code from es6-class-mocks.
And there's something that doesn't work in my code.
// package.json
"devDependencies": {
  "@types/jest": "^24.0.18",
  "jest": "^24.9.0",
  "ts-jest": "^24.1.0",
  "typescript": "^3.6.3"
},

// jest.config.js
module.exports = {
  roots: ["<rootDir>/src"],
  transform: {
    "^.+\\.tsx?$": "ts-jest"
  }
};

I'm using jest with TypeScript. And these below are the related codes about my question.
// Sport.ts
export default class Sport {
  name: string;

  constructor() {
    this.name = "Sport";
  }

  printName() {
    return this.name;
  }
}

// Sport.spec.ts
import Sport from '../Sport';
jest.mock('../Sport');

beforeEach(() => {
  // This gives me an error
  Sport.mockClear();
});

So, Sport.mockClear() gives me an error, Property 'mockClear' does not exist on type 'typeof Sport'.ts. And Sport doesn't even have mock property.
So I found this issue ts-jest issue, and here's my new code.
// Sport.spec.ts
import Sport from '../Sport';
import { mocked } from 'ts-jest/utils';
jest.mock('../Sport');

beforeEach(() => {
  // This works
  mocked(Sport).mockClear();
});

it('Constructor is called', () => {
  new Sport();
  expect(Sport).toHaveBeenCalledTimes(1);
});

it('Mock is clear', () => {
  // Succeess
  expect(Sport).not.toHaveBeenCalled();
});

Question
So, I wonder why Sport.mockClear is error and is it the only way to use ts-jest/utils?

This is a TypeScript error. One thing you could do is to cast the type to jest.Mock. Please ask on TS forums about it though.


Integer Unification for Ruby 2.4.0+

Ruby 2.4.0 unifies Fixnum and Bignum into Integer.
https://bugs.ruby-lang.org/issues/12005
Fix following deprecated warnings in Ruby 2.4.0-preview3.
warning: constant ::Fixnum is deprecated
Thanks.

The build failure is unrelated and it will be solved by #85, maybe.

LGTM... test coverage seems to be pre-existing.
https://github.com/ipaddress-gem/ipaddress/blob/master/test/ipaddress/prefix_test.rb#L76-L89

Thanks!

This goes back a while, but appears there hasn't been a release including this change?


refactor shap-ttolbar and add support for remote plugin

Description

main.py as the main entry point
moved test assests to stock-plugins root (stock-plugins/user_defined_files)
refactored the folder and files structure.
Users can now execute the plugin as a module
Plugin will now accept comma separated assets paths (data,model,ground_truth), model classifications, ground truth value,plugin args

Type of Change

refactor: Code refactoring without changes in functionality

perf: Performance improvements

How to Test
Execute the below commands to run unit tests -
cd aiverify/stock-plugins/aiverify.stock.shap-toolbox/algorithms/shap_toolbox/
pytest . -vv

Checklist
Please check all the boxes that apply to this pull request using "x":

[x]  I have tested the changes locally and verified that they work as expected.
[x]  I have added or updated the necessary documentation (README, API docs, etc.).
[x]  I have added appropriate unit tests or functional tests for the changes made.
[x]  I have followed the project's coding conventions and style guidelines.
[x]  I have rebased my branch onto the latest commit of the main branch.
[x]  I have squashed or reorganized my commits into logical units.
[x]  I have added any necessary dependencies or packages to the project's build configuration.
[x]  I have performed a self-review of my own code.
[x]  I have read, understood and agree to the Developer Certificate of Origin below, which this project utilises.

Screenshots (if applicable)
[If the changes involve visual modifications, include screenshots or GIFs that demonstrate the changes.]
Additional Notes
[Add any additional information or context that might be relevant to reviewers.]

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.

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

Developer's Certificate of Origin 1.1

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

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

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

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

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

@imda-peckyoke  / @timlrx : Tests for multiclass_non_pipeline  is failing. This will be revisited later. For now, I have commented this test.
Ref- https://github.com/aiverify-foundation/aiverify/pull/323/files#diff-0a6151e73ecee45f03ebdc7c1c53deb3ce79b44657b279d466b331da60c6e9cbR107


SocketRocket: In debug mode.  Allowing connection to any root cert

Hi,
After losing out a user and then login the same user Simperium for Mac OS goes into a loop with Socket Error "Invalid Certificate"
To replicate all without restarting the App:

Connect with a user with call like [self.simperium authenticateWithAppID:@"paper-video-6fd" token:[notification object]];  the token is valid.
Logout user with [self.simperium signOutAndRemoveLocalData:NO completion: ^{
}];
Connect the same a user with call like [self.simperium authenticateWithAppID:@"paper-video-6fd" token:[notification object]];  the token is valid.

The connection looks OK but the NSLOG keeps printing out : "SocketRocket: In debug mode.  Allowing connection to any root cert".

(void)webSocket:(SPWebSocket *)webSocket didFailWithError:(NSError *)error  starts looping the error is  #23556 (Invalid server cert) .

I'm I doing something wrong when disconnecting the client?

@mmilen hello there!. There's nothing wrong with the sign out invocation. I'll try to reproduce this on my end.
I remember seeing this error in console... long ago, while testing Simplenote in the iOS Simulator. It got fixed on its own, after performing a fresh install (Simulator Reset, actually). Sadly, i never got to trigger it again.
Is it possible you're using a VPN / debugging software, such as Charles?

Thanks for the immediate response ( rare occurrence nowadays ) !!!
I'm using stock Mac OS 10.8.5 with XCode 5.1.1 , both my project and
Simperium project targets are set to Mac OS 10.8 .
Milen Milkovski
CEO / Lead Developer
L'Atelier D'Ema
<EMAIL_ADDRESS>On 2015-08-12 3:01 PM, Jorge Leandro Perez wrote:

@mmilen https://github.com/mmilen hello there!. There's nothing
wrong with the |sign out| invocation. I'll try to reproduce this on my
end.
I remember seeing this error in console... long ago, while testing
Simplenote in the iOS Simulator. It got fixed on its own, after
performing a fresh install (Simulator Reset, actually). Sadly, i never
got to trigger it again.
Is it possible you're using a VPN / debugging software, such as Charles?
—
Reply to this email directly or view it on GitHub
https://github.com/Simperium/simperium-ios/issues/492#issuecomment-130412788.

No problem sir!. I'll try to reproduce the same under Mountain Lion. Would it be possible to test the exact same code you're running in Yosemite?

Will try in the next couple of days and will let you know.
Milen Milkovski
CEO / Lead Developer
L'Atelier D'Ema
<EMAIL_ADDRESS>On 2015-08-12 3:14 PM, Jorge Leandro Perez wrote:

No problem sir!. I'll try to reproduce the same under Mountain Lion.
Would it be possible to test the exact same code you're running in
Yosemite?
—
Reply to this email directly or view it on GitHub
https://github.com/Simperium/simperium-ios/issues/492#issuecomment-130415812.

Perfect, thank you!

Just build the project in  Yosemite, and everything is working perfectly
fine .
Milen Milkovski
CEO / Lead Developer
L'Atelier D'Ema
<EMAIL_ADDRESS>On 2015-08-12 3:38 PM, Jorge Leandro Perez wrote:

Perfect, thank you!
—
Reply to this email directly or view it on GitHub
https://github.com/Simperium/simperium-ios/issues/492#issuecomment-130422531.

Cool. Are you running 10.8 in a Virtual Machine? (i'll try that setup, since i'm already running Yosemite).

Can you please verify if there's no proxy setting that might be interfering?
Is the library effectively not sync'ing anything?

@mmilen hello there!
May i ask you if you have some extra debug info // still experiencing this glitch?. I've been unable to reproduce on my end.
Thanks in advance!

It is still there. With my game it would be extremely rare that the user
will logout, so no big deal. Let me know if I can test something to help
you out.
Milen Milkovski
CEO / Lead Developer
L'Atelier D'Ema
<EMAIL_ADDRESS>On 2015-09-01 11:43 AM, Jorge Leandro Perez wrote:

@mmilen https://github.com/mmilen hello there!
May i ask you if you have some extra debug info // still experiencing
this glitch?. I've been unable to reproduce on my end.
Thanks in advance!
—
Reply to this email directly or view it on GitHub
https://github.com/Simperium/simperium-ios/issues/492#issuecomment-136765392.


Added additional attributes to OwnTracks device_tracker

Description:
Adds three relevant attributes to OwnTracks devices

vel: Current speed of the device
tid: Tracker ID of the device to distinguish it from other devices of the same user
addr: Can be used by OwnTracks recorder to provide reverse geocoding for the lat/lon coordinates

Checklist:
If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Hi @binarybucks,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

With 0.50.1, I am seeing tid and vel, but am not seeing addr in the Owntracks device tracker. Just wanted to check if there is anything else that is required to enable addr.

addr is currently not added automatically. The attribute can be used by e.g. ot-recorder to provide this data.

Both vel and id were automatically added, so I was thinking that addr should be the same. How can user access this information?

@binarybucks I have been reading this pull request with some interest. I have installed ot-recorder and am receiving rev-geo information but cannot see how you add this to the MQTT payload for each device. Are you using an Lua script to do this?


Fixed installation error caused by old SSL module

This PR fixes an issue which happens on Python 2 platforms that have an outdated ssl module. These older ssl modules can cause some insecure requests to succeed where they should fail and secure requests to fail where they should succeed.
Source: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
Fixes #2222

getting this on installation:
Check if urllib3[secure] is installed...............[FAIL]


Large files, option to only return the column mapping insted of full content

Hi there.
We've already built an (ugly) but efficient piece of code that is able to read a csv file (first x lines) and then allow the user to tell what is the data type of each columns. Then we send the untouched file, with the mapping to the backend that does the import.
Indeed, with huge files (million lines) it would surcharge the browser to read the entire file and return the result.
I propose an option to return only the mapping instead of the full result set.
I can work on it and propose an implementation as I plan to replace our little piece of code with your library. (We already use it in some parts of our apps where the files uploaded are not that big).

Hi @pintaf! For large file use cases, I was thinking about implementing streaming so the backend could receive the data in chunks. We could think about adding support for just sending the mapping (and the file data) but this might be tough to support with future updates to add a review screen and validations. Let me know your thoughts.

First I know you mainly use this for your business in TableFlow, so indeed it's important to limit the options, otherwise that can quickly become a spaghetti bowl. Thanks for open sourcing this part BTW.
Indeed, with just the mapping, you would not be able to do validation. But it it feasible to open a 1Million lines excel in the browser and check every line for validation ? Sure, there are possibly ways to chunk the process to avoid high Ram usage, but I wonder if that's not going to transform the end user machine in a heater.
Nice idea the streaming, but as per current lib design, this seems out of the scope here no ? currently the lib only return the result, and it's up to the lib user to do what they want with results ?
Maybe there could be two processes:

Normal one, with review allowing to correct some data (maybe) and output being an object with corrected data
Mapping only, where you skip the review and validation and get either the whole data as a JS object or mapping plus file object.

From my point of view, there is not enough power in our computers OR we don't want to vamparize the computer resources with large files, especially if you have mobile support in mind.
If you want, I can come up with a configuration object proposal and result proposal depending on the options ?

@pintaf

We've already built an (ugly) but efficient piece of code that is able to read a csv file >(first x lines) and then allow the user to tell what is the data type of each columns.

Could you please point me to the this code in your repo?
When i open the [link] (https://github.com/CustomsBridge/)  , i see two public repositories.
Thanks!

It's not public, it's made in vueJS, but if you want I can share it with you.


[fixed] Added brokerConfig search functionality by value (Issue/3163)

[ ] Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
In BrokerConfigTab, I have added search by value functionality and basically changed the method getData by filter out brokerconfig name or value.
Closes #3163
Is there anything you'd like reviewers to focus on?
BrokersConfigTab.java and Configs.tsx are changed. Please refer those.
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[x] No need to
[x] Manually (please, describe, if necessary)
[x] Unit checks
[x] Integration checks
[x] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

I think the search text should be case insensitive. keyword.toLocaleLowerCase()

Yeah. You are right.

@malavmevada there are some linter issues which prevent the build from succeeding, otherwise looks good

@malavmevada there are some linter issues which prevent the build from succeeding, otherwise looks good

@Haarolean Actually I'm very new to this and because of that I made many small mistakes btw thank you. Do i need to change anything now ?

@Haarolean There are some issues regarding formatting the code in Linter. What should I do now ?

@Haarolean There are some issues regarding formatting the code in Linter. What should I do now ?

I guess fixing line breaks would be nice :)
https://github.com/provectus/kafka-ui/pull/3804/files

@Haarolean Linter issue is resolved.

@malavmevada thank you, will be merged soon


ansible-conjur-host-identity docs are migrated

The docs for https://github.com/cyberark/ansible-conjur-host-identity have been migrated, pending the migration of the project and tests themselves to this collection.
AC:

[ ] Migrate the docs for the role to the collection, and revise to be based on how the role works within the collection
[ ] Update the Conjur docs card cyberark/conjur-docs#895 to include what the docs should say now

@izgeri's gist from the last update might be a useful baseline, and there are also some Word docs linked in the docs issue that may help

Note: cyberark/conjur-docs#895 has been migrated to cyberark/conjur-docs#1015
I don't know if anyone has written good docs yet on best practices for collection usage, and the README in this repo also looks a little sparse. It might be good to write up a gist for what you think the online docs should have, given that:

Ansible 2.9 and earlier have a built-in lookup plugin (though it isn't being maintained, and our collection will be)
Ansible 2.10 will have a reference to our collection lookup plugin built-in
There is also a built-in lookup plugin in Ansible Tower / AWX

I think in all cases, I'd like to recommend people invoke the collection lookup plugin specifically, since that is our official lookup plugin that we maintain / test / etc.
It is ok if the README here remains sparse, and we link people to the Conjur docs. Or we could really populate this README with detailed instructions, and have an overview in the docs that links to GitHub. I leave it to you and @suefran to figure out - I will be happy if the end result is more comprehensive, clear guidelines for using our Ansible tools.

Closing, as we have done as much of this as we can at this point.


Broken interactive window test 'Raising an exception from within a function has a stack trace'

https://github.com/microsoft/vscode-jupyter/actions/runs/3565779955/jobs/5991373158
https://github.com/microsoft/vscode-jupyter/actions/runs/3568555079

@amunger looks like the tests are failing consistently, hence looks more like a break in the code.,
You might want to look into this sooner than later, hence assigned to nov and added the bug label
Please re-adjust as you see fit

I'll take a look, but it actually doesn't seem to have failed on main for a few weeks

never mind, looks like the latest ipykernel version breaks our regex again

verification steps:
create and save a python file
create a python environment and install ipykernel  6.18.1 (pip install ipykernel==6.18.1) run the file in the interactive window with something that will throw e.g. print(x)`
ensure the stacktrace has links back to the errored line in the file


"Follow" button email subject line needs fixed

The email subject line you get when questions open in a project you follow is "Questions have changed status".
The way this previously worked was it said "New questions have opened in [project name]".
The way it previously worked is much clearer and much more likely to get someone to check out the email, so it would be great to bring that subject line back.

Also if it sends that notification about any status change, and not just questions opening, that should probably be fixed too, though I have no idea if that's the case, just implied by the subject line.


Typo

It should be persistent instead of persistant.

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

ID: ed330edc-c3ab-d053-faad-4dc5da373b48
Version Independent ID: 625ab8a7-8379-0a1c-c633-fa7bf80f161e
Content: Introduction to Azure Kubernetes Service
Content Source: articles/aks/intro-kubernetes.md
Service: container-service
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

Thanks for the feedback! We are currently investigating and will update you shortly.

@zhusulai thanks for this find! I have corrected the doc. Once the PR merges this issue will close and the changes will go live in a few hours.


SchedMD slurm image does not exist

The documentation suggests this is the most recent slurm image:
schedmd-v5-slurm-22-05-6-hpc-centos-7

But I cannot find it when building a packer image:
* googleapi: Error 404: The resource 'projects/spherical-berm-323321/global/images/family/schedmd-v5-slurm-22-05-6-hpc-centos-7' was not found, notFound
.
.
* googleapi: Error 404: The resource 'projects/ml-images/global/images/family/schedmd-v5-slurm-22-05-6-hpc-centos-7' was not found, notFound

The packer default hpc-centos-7 still exists. Is slurm on GCP still supported?

Hi Tristan -
That is in fact the most recent image documented by slurm-gcp for v5. This should work for any module in the format of schedmd-slurm-gcp-v5-*.
If you are using modules in the format of SchedMD-Slurm-on-GCP-*, you need the most recent v4 image which is schedmd-slurm-21-08-8-hpc-centos-7, which are documented here.
I see that packer is searching in spherical-berm-323321 and ml-images, but the source project needs to be schedmd-slurm-public which is where they are hosted by schedmd until you create a local copy in your project.
If you are able to send me part or all of your blueprint, I can try to reproduce and help you find the problem.

Thanks, working for me now


Added Ukrainian translation

Just a quick translation to Ukrainian

Added, thanks! Will deploy to the production API next week; we don't like making changes on Fridays.
(FYI, "Ukranian" has the language code "uk"; "ua" is the country code for Ukraine. I have adjusted this in your commit.)


Wasm:support initialization of md arrays

Adds a test (previously failing) and support for initializing multidimensional arrays such as
        double[,] doubles = new double[2, 3]
        {
            {1.0, 1.1, 1.2 },
            {2.0, 2.1, 2.2 },
        };

Fixes #7921

Thanks


Laravel 8.73.1 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v8.73.1 branch.
Before merging, you need to:

Checkout the shift-ci-v8.73.1 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:information_source: The following Laravel packages were updated as they are known to be compatible with Laravel 8.73. Nonetheless, you should review the CHANGELOG to ensure no additional changes are needed.

[ ] thujohn/twitter

:warning: The Shift build servers run PHP 8. If you are still running PHP 7.4 or lower you should run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP 7 environment.


Associations

Thank you for reviewing my project! On this PR, I completed...

Create Rentals and Books classes
Add associations: the person has many rentals, books have many rentals, rentals have book and person.

Your review and suggestions on how to improve the project will be greatly appreciated. ⭐

I don't understand what you want to say, can you provide me some examples
On Tue, 14 Mar 2023 at 14:46, Caleb Moses HYMELE T. <
@.***> wrote:

@.**** requested changes on this pull request.
Hi @asifaminisonline https://github.com/asifaminisonline ,
Good job so far. !
There are some issues that you still need to work on to go to the next
project but you are almost there!
[image: mario_ladders]
https://user-images.githubusercontent.com/20563252/121369265-4e418880-c901-11eb-848c-1eff06d5a4cf.gif
Highlights Required Changes ♻

You really did a great job but there are some features you need to
improve

Please check the comments below

I'm sure you can do it just keep it up
Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop
the approval of this PR. However, I strongly recommend you to take them
into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something
is not 100% clear.
Please, remember to tag me in your question so I can receive the
notification.
@.*** https://github.com/m05e5*
Please, do not open a new Pull Request for re-reviews. You should use the
same Pull Request submitted for the first review, either valid or invalid
unless it is requested otherwise.
As described in the Code reviews limits policy
https://microverse.zendesk.com/hc/en-us/articles/1500004088561 you have a
limited number of reviews per project (check the exact number in your
Dashboard). If you think that the code review was not fair, you can request
a second opinion using this form https://airtable.com/shrQAqnBwek5a0O0s.
In book.rb
https://github.com/asifaminisonline/OOP-school-library/pull/3#discussion_r1135225490
:

@rentals = []

end
+end

Hi great job so far. As part of the requirements, you are supposed
to create a has-many relationship side of Book and Rental (a book has many
rentals). So you are supposed to have a method to add rentals in the book
class

In person.rb
https://github.com/asifaminisonline/OOP-school-library/pull/3#discussion_r1135228617
:

end
+end

Hi great job so far. As part of the requirements, you are supposed
to create a has-many relationship side of Person and Rental (a person has
many rentals). So you are supposed to have a method to add rentals in the
person class

—
Reply to this email directly, view it on GitHub
https://github.com/asifaminisonline/OOP-school-library/pull/3#pullrequestreview-1338792367,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A26GIDQG35GHWSUKCMONNG3W4AZPZANCNFSM6AAAAAAV2EQWRI
.
You are receiving this because you were mentioned.Message ID:
@.***>


build_SYS_str_reasons: Fix a crash caused by overlong locales

Applications using OpenSSL crash in openSUSE upon initialization under Russian locales.
See https://bugzilla.opensuse.org/show_bug.cgi?id=1135550.
This happens due to several factors.
Apparently it gets exposed by commit e3b35d2b29e9446af83fcaa534e67e7b04a60d7a which we apply in openSUSE.
The crypto/err/err.c 4 kB SPACE_SYS_STR_REASONS isn't enough for some locales. The Russian locales consume 6856 bytes, Ukrainian even 7000.
build_SYS_str_reasons() contains an overflow check:
  if (cnt > sizeof(strerror_pool))
      cnt = sizeof(strerror_pool);

But it no longer works since commit 9f15e5b911ba6053e09578f190354568e01c07d7, where cnt is incremented once more after the condition.
The added cnt++; at the end of the VMS hack results in openssl_strerror_r() being called with a size parameter equal to -1, meaning SIZE_MAX. This leads to an unbounded OPENSSL_strlcpy() writing after the array.
(gdb) bt
#0  OPENSSL_strlcpy (dst=0x7ffff7840e81 <ex_data+1> "", src=0x7ffff71630e4 "\265-сокету", size=18446744073709551554) at crypto/o_str.c:86
#1  0x00007ffff76ccef0 in openssl_strerror_r (errnum=88, buf=0x7ffff7840e43 <prompt_string+35> "Операция для сокета применена к ", <incomplete sequence \320>, buflen=18446744073709551615) at crypto/o_str.c:244
#2  0x00007ffff768f84f in build_SYS_str_reasons () at crypto/err/err.c:223
223                 if (openssl_strerror_r(i, cur, sizeof(strerror_pool) - cnt)) {
(gdb) p sizeof(strerror_pool)
$10 = 4096
(gdb) p cnt
$11 = 4097

Invalid memory is overwritten, making the program crash later.

nice debugging and writeup!

@levitte Does your approval still hold?
@bernd-edlinger Do you approve also for 1.1.1?

yes.

@levitte ping? I'd like to push this but it requires re-approval.

@openssl/omc Or anybody else from OMC

I am going to merge.

[master fac9200a88] build_SYS_str_reasons: Fix a crash caused by overlong locales
[OpenSSL_1_1_1-stable a069bdf0aa] build_SYS_str_reasons: Fix a crash caused by overlong locales


New package: CAT v0.1.0

Registering package: CAT
Repository: https://github.com/fadihamad94/CAT-Journal
Created by: @fadihamad94
Version: v0.1.0
Commit: 100eb39faa44da65ee65dc8a4627684c7ea86cd6
Git reference: HEAD
Description: This package is the implementation of a simple and practical adaptive trust-region method for finding stationary points of nonconvex functions with L-Lipschitz Hessians and bounded optimality gap.
Release notes:

Release notes:

First version of the CAT Solver

We decided to name the package CATrustRegionMethod and we included in the package readme an explanation of the name.
We opened another pull request. Can you please close this pull request.


Print out links to the pull requests created

At the end of a PR run, it would be great if the links were printed from the PRs that were created. So the results could look like this:
$ turbolift create-prs
  OK   Reading campaign data
  OK   Pushing changes in org/repo-1 to origin
  OK   Creating PR in org/repo-1
  OK   PR created: https://github.com/org/repo-1/pull/163
  OK   Pushing changes in org/repo-2 to origin
  OK   Creating PR in org/repo-2
  OK   PR created: https://github.com/org/repo-2/pull/163
  OK   Pushing changes in org/repo-3 to origin
  OK   Creating PR in org/repo-3
  OK   PR created: https://github.com/org/repo-3/pull/163
  OK   turbolift create-prs completed (3 OK, 0 skipped)

Love the tool, and find it very useful!

Thanks for your feedback @gavingmiller. Definitely a good addition to the feature set

I've also come

Sorry, fat fingers!

@gavingmiller Sorry for the extremely long time to resolve this, but we think this is now finally addressed. We took a slightly different approach, but now have the turbolift pr-status --list command which includes URLs for all PRs. Hopefully you're still finding turbolift useful and can benefit from this!
Thanks


[BEAM-11354] Allow DoFn itself to be used as the restriction provider.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @boyuanzz

It seems like we want to do the similar thing to java? Java SDK uses DoFn as both RestrictionProvider and WatemarkEstimatorProvider. If so, it would be better for python sdk to either do the same thing as java sdk or keep using RestrictionTracker and WatermarkEstimatorProvider classes.

Yes, this is more similar to Java. Good point about doing this for the
watermark provider too--I'll do that. (Removing the old way would be
backwards incompatible and a more invasive change.)
On Wed, Nov 25, 2020 at 4:20 PM Boyuan Zhang<EMAIL_ADDRESS>wrote:

It seems like we want to do the similar thing to java? Java SDK uses DoFn
as both RestrictionProvider and WatemarkEstimatorProvider. If so, it
would be better for python sdk to either do the same thing as java sdk or
keep using RestrictionTracker and WatermarkEstimatorProvider classes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/beam/pull/13431#issuecomment-734004182, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADWVAJXBI64PMALKQCNTF3SRWNORANCNFSM4UDA4RDQ
.

Thanks for the explanation. I'm kind of concerned about keeping both APIs. It seems like not clear enough and might be confusing for beam users. Besides, we also need to update related documentation if we decide that way.

I've updated the documentation.
Java provides meaning to these specific methods by using annotations, but I still think we want a type in Python (unless we wanted to make all DoFns into RestrictionProviders and WatermarkEstimatorProviders, with their corresponding default methods). Not being able to bind these things based on DoFn properties is very constraining though (as you discovered as well).

Drive by comment: Java users who want to build instances of DoFns programmatically (such as DSL authors) suffer when annotations are the only way to define something. Ideally, we would have both.

When I looked at the programming guide, it pointed to the python docs for the details, which have been updated.

When I looked at the programming guide, it pointed to the python docs for the details, which have been updated.

I mean we may also want to provide the code example that is using DoFn as the RestrictionProvider and WatermarkEstimatorProvider.


Enables building on Solaris

Using golang.org/x/sys/unix instead of syscall in osfs/os_posix.go makes this project buildable on OpenIndiana 18.04 (OpenSolaris), most probably on Solaris too. See: https://github.com/golang/go/issues/11113

Merged https://github.com/src-d/go-billy/pull/65
Thanks


From a Bower install, foundation.scss doesn't compile with critical partials like _grid.scss?

I have only attempted this from a bower install of foundation 5.3.3. When compiling from foundation.scss to foundation.css the generated css completely leaves out critical includes like grid. I went even further and attempted to generate a partial grid.css from _grid.scss grid.css and it generated an empty file?
I'm getting no sass errors so not really sure what's going here

I realize this issue is closed, but I was led to this thread from the Foundation forums.  I'm using Sass 3.4.18 (Selective Steve) to compile my scss to css.  As stated in this thread, the critical components are not included in the final compiled css.  I can downgrade to 3.2 and it works.  I should note that my foundation version is the latest 5.5.2—I thought this issue might have been fixed by now, but it has not.  Normally I would look into some kind of log, but am not getting any error messages as sass is reporting no errors.   Any suggestions for debugging for troubleshooting?

@ariestav - my suggestion would be to use gulp (gulpjs.com) - I've started using it a while back and it's just so much easier to manage everything - you don't have to worry about all these problems any more - simply instal all gulp components via npm, setup gulpfile.js and you're ready to roll. You can install foundation via bower and then add add whatever you need to gulpfile to process.

@sebastiansulinski has the right idea. I too have been digging into gulp and use it for everything, including image optimization, full css and js concatenation / minification, and browser-sync with my lamp vagrant stack using box.scotch.io. Fantastic workflow, super quick, and painless. Pipeline assets / npm is definitely where web design is at right now when used correctly and in moderation.


Flag low coordination numbers for lanthanide/alkaline/alkaline earth

Feature description
Lanthanides like to have many neighbors and we now many instances in CoRE where they have surprisingly few neighbors.
Implementation idea
def _check_metal_coordination(site, cn: int) -> bool:
    # Lanthanides like to have many neighbors
    # Low coordinatio number is usually only
    # possible with really bulky ligands
    # for this reason, a Lanthanide with low
    # coordination number, e.g., <= 4 can be considered "interesting"
    if site.specie.is_lanthanide():
        if cn <= 4:
            return True

    # Also for the alkaline/alkaline earth metals,
    # I would find a low coordination number surprising
    elif site.specie.is_alkali() or site.specie.is_alkaline():
        if cn <= 2:
            return True

    return False

Alternatives

If I recall correctly, I think many of the instances of the low coordination numbers in the CoRE MOF database for the lanthanides are cases where terminal O were removed as "solvent" even though they might be oxos or hydroxyl groups. For instance, the uranyl node has the motif O=U=O, but it's easy to accidentally flag those O atoms as water and remove them.
The check for the alkali(ne earth) metals is also important but probably a bit tougher. Li-exchanged MFU-4l for instance is three-coordinate, and a quick ConQuest search of the CSD MOF subset points to a lot of hits with low-coordinate alkali(ne earth) metals. Whether those are all physical or not though, I'm not entirely sure.

If I recall correctly, I think many of the instances of the low coordination numbers in the CoRE MOF database for the lanthanides are cases where terminal O were removed as "solvent" even though they might be oxos or hydroxyl groups. For instance, the uranyl node has the motif O=U=O, but it's easy to accidentally flag those O atoms as water and remove them.

Indeed, we're currently working on a ML-based approach to find those + charged cases, this model flagged a lot of lanthanides that were low coordinated and which I found suspicious. Our main challenge is currently to systematically assess if those cases are physical or not (rather than relying on my guts ...).
But I guess you're write one should remove this check for now.

Sweet! Best of luck to you and the team on that! I think the ML-based approach is probably the smartest. You'll surely flag them pretty effectively that way. This is all going to be great for the community because we need some good tools out there for this, even if they'll never be perfect.
My gut instinct is that the simple heuristic you had/have here for low coordination number checker is probably perfectly suitable for the lanthanides/actinides in most cases. I saw that max(coord. num.) was set to 4 for the alkali metals in the commit I accessed, which made me think of the Li-MFU-4l, so I figured I'd just share that counter-example.


[docs] Demo SearchIcons crashes

[ ] I have searched for similar issues in this repository and believe that this is not a duplicate.

Steps to Reproduce

Visit https://mui.com/components/material-icons/
??
demo SearchIcons crashes

Your Environment

Tech
Version

MUI
v5.2.5

netlify deploy
https://61c098e520eae9000736c0df--material-ui-docs.netlify.app

Browser
Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.110 Safari/537.36 Edg/96.0.1054.62

Fixed with #30319 - we'll soon release a new docs version with it.


fix: fixed mocha grep regex instantiation, closes #3949

Description of the Change
There is a problem in the instantiation of RegExp in mocha grep
The problem is explained in https://github.com/mochajs/mocha/issues/3949
Why should this be in core?
Without this passing regex-like string to grep will not work correctly
Benefits
Enhancing grep option
Applicable issues
This is a bug fix

Coverage increased (+0.07%) to 92.464% when pulling 07f0a19de0b5fea08f8eec53059ee7f2539a6b13 on mrastiak:issue/3949 into 9ea45e7557a193c4b69816085397a6655fb5bc06 on mochajs:master.

Some examples would help to understand this issue. We have several unit tests for Mocha#grep() which seem to work fine.
Please advice which test constellations fail with the current version of this function, thanks.


Finding Linked List is Palindrome or Not

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes ofcourse!!
Motivation
(Please outline the motivation for the proposal.)
Its a Classical Problem which is often asked in Interviews!!
Problem Statement:
Given a linked list you need to find whether the given linked list is Palindrome or Not
Input: 1->2->1
Output: Palindrome
Input:1->2->3
Output: Not Palindrome
Input: 1
Output: Palindrome(a no is always palindrome)
Pitch
(Please explain why this feature should be implemented and how it would be used.)
This problem will help developers realize how Linked List data structure can be used in various Set of problems :)

@siddharth25pandey @HarshCasper @iamrajiv  can u please assign me the issue !!
Would love to work on C

@siddharth25pandey @iamrajiv @HarshCasper , I would like to contribute for the issue in Java/C++ , please  assign this task to me

@siddharth25pandey @iamrajiv @HarshCasper I m a GSSOC'21 participant, I would like to contribute to this issue in Java, please assign this issue to me.

@siddharth25pandey @HarshCasper @iamrajiv can u please assign me the issue !!
Would love to work on C as a part of GsSoc21

@sumitgsh Hey, can you tell me which DSA you are going to implement as already this kind of issue someone else already solving.

@siddharth25pandey @iamrajiv @HarshCasper , I would like to contribute for the issue in Java/C++ , please assign this task to me

@nishantsahu961 Hey, can you tell me which DSA you are going to implement as already this kind of issue someone else already solving and also you can open a new issue with the same concept but with implementation in a different programming language. We will assign it to you then. Happy Coding!😊

@siddharth25pandey @iamrajiv @HarshCasper I m a GSSOC'21 participant, I would like to contribute to this issue in Java, please assign this issue to me.

@cintamg Hey, can you tell me which DSA you are going to implement as already this kind of issue someone else already solving and also you can open a new issue with the same concept but with implementation in a different programming language. We will assign it to you then. Happy Coding!😊

@ankitaggarwal23 I am going to implement this using a doubly-linked list, in Java.

@sumitgsh @cintamg @nishantsahu961
Check whether it is already there in the repo under the subsection of programming lang you are willing to contribute.
If, yes, drop it and open a new issue with a new and unique idea.
If no, open a new issue by mentioning the link to this issue and the programming language you would want to contribute in.
Issues will be assigned on a first come first serve basis.
Happy Coding!😊

@ankitaggarwal23
Can u tell me how I can mention the link to this issue while creating the new issue?

@sumitgsh Hey, can you tell me which DSA you are going to implement as already this kind of issue someone else already solving.

@ankitaggarwal23  I would be using two pointers approach and will STACK as well.
I had seen someone else is working on the issue, but I want to work using C language, as no one is working on that(Even I mentioned in the first comment of the issue)
Others who are working on the same are:
Issue no:#2951(working in C++)
Issue no: #2891(Working in java)

@sumitgsh Hi, I assigned you. Happy Coding!😊