Exclude directories from processing

Description
A small feature request
While adding Paket to our solution, we noticed that it runs through all subdirectories to scan for paket.references etc. There are two inconvenient things here:

It also parses the .svn directory, where it should never ever touch anything anyway.
we have some code generation directories that contain '.project' files, which Paket dutifully warns about.

None of these issues is critical; the .svn directory contains essentially guids as filenames, and the '.project' files are not really touched because there is no paket.references in these directories or above. However, it seems nice to be able to exclude directories from the automatic search of "paket install" (or, alternatively, include them, we have just two that are relevant).
For example, running something like "paket install --include-dir=src'.

I think the .svn could be easily fixed as a "up-for grabs" PR. You could scan the repository for ".git" and just add ".svn" in those places.

There seems to be a workaround for the second part in Issue #918. Of course, I did not search for closed issues...


test: repro panic for tracing

I was able to repro panic for #73 again with high-concurrency. Command to run was:
cargo run --features stdout-tracing  -- --cluster --port 8000 ---count 1000000 --pool 5 --tracing --concurrency 100000 pipeline
It doesn't require any actions with redis itself (restart/failvor isn't needed) and looks related to back-pressure logic

Yeah, I was hoping this wouldn't actually repro but it was on my list to check with a very high concurrency. The fix is relatively straightforward so I should have a fix up later today.

Yes, I was thinking exactly about this behavior when looking at the code.

Ok if you have some time try it with the changes in https://github.com/aembke/fred.rs/pull/89. If that doesn't work please let me know.

Looks fixed! 👍
At least it's not reproducible with current example


2024-04-30 — 1 new commit(s) made to reference.md in Google Transit's repo

Commit Nº 1
AUTHOR × DATE

isabelle-dr ( isabelle-dr )
April 29, 2024   @   04:24 AM ETC (08:24 AM UTC)

MESSAGE
Editorial changes: rendering of locations.geojson description + updating booking_rule_id description (#441)
* Update reference.md
* Update reference.md
* Update gtfs/spec/en/reference.md
Co-authored-by: Tzu-Jen Chan<EMAIL_ADDRESS>---------
Co-authored-by: Tzu-Jen Chan<EMAIL_ADDRESS>
FILES CHANGED

https://github.com/google/transit/commit/ca4b879ea9719b9f80dbd3de9ecf128d809c4c1b

https://github.com/google/transit/commit/ca4b879ea9719b9f80dbd3de9ecf128d809c4c1b

Editorial change with no change to validation rules needed.


cannot import name '_DataLoaderIter'

I ran the code in pytorch 1.2.0
How to deal with "cannot import name '_DataLoaderIter'"?
Thanks!

Did you solve this problem somehow?

please help me . I alse hava this problem!!!

I also have this problem


Core: Remove cache in all tosfs APIs

Considering the cons are larger than pons, we decide to remove the cache in all the tosfs APIs. For more detail pls see:https://github.com/volcengine/tosfs/pull/19#discussion_r1726589299

linked PR is #29


WSL 2 conversion fails without any actionable details

Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)

PS C:\Users\Flynn Buckingham> [System.Environment]::OSVersion.Version

Major  Minor  Build  Revision
-----  -----  -----  --------
10     0      18917  0

What you're doing and what's happening:

I'm attempting to migrate my WSL Ubuntu (1) over to (2), but the migration fails without any key indicators (besides a vague error message). While I'd love to have WSL 2 working, the lack of actionable output makes me a bit uneasy. (Also, I have restarted twice as of writing, after enabling the WSL 2 feature)
PS C:\WINDOWS\system32> Get-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux

FeatureName      : Microsoft-Windows-Subsystem-Linux
DisplayName      : Windows Subsystem for Linux
Description      : Provides services and environments for running native user-mode Linux shells and tools on Windows.
RestartRequired  : Possible
State            : Enabled
CustomProperties :
                   ServerComponent\Description : Provides services and environments for running native user-mode Linux
                   shells and tools on Windows.
                   ServerComponent\DisplayName : Windows Subsystem for Linux
                   ServerComponent\Id : 1033
                   ServerComponent\Type : Feature
                   ServerComponent\UniqueName : Microsoft-Windows-Subsystem-Linux
                   ServerComponent\Deploys\Update\Name : Microsoft-Windows-Subsystem-Linux

PS C:\WINDOWS\system32> wsl --list --verbose
  NAME      STATE           VERSION
* Ubuntu    Stopped         1
PS C:\WINDOWS\system32> wsl --set-version Ubuntu 2                                                                      Conversion in progress, this may take a few minutes...
For information on key differences with WSL 2 please visit https://aka.ms/wsl2
A device attached to the system is not functioning.
PS C:\WINDOWS\system32> wsl --list --verbose
  NAME      STATE           VERSION
* Ubuntu    Stopped         1

What's wrong / what should be happening instead:

It should at least be giving me actionable error information; but ultimately should be working.

I'm seeing this as well with both Debian and Ubuntu:
PS C:\WINDOWS\system32> Enable-WindowsOptionalFeature -Online -FeatureName VirtualMachinePlatform

Path          :
Online        : True
RestartNeeded : False

PS C:\WINDOWS\system32> wsl -l
Windows Subsystem for Linux Distributions:
Debian (Default)
Ubuntu
PS C:\WINDOWS\system32> wsl --set-version Debian 2
Conversion in progress, this may take a few minutes...
For information on key differences with WSL 2 please visit https://aka.ms/wsl2
Unspecified error
PS C:\WINDOWS\system32> wsl --set-version Ubuntu 2
Conversion in progress, this may take a few minutes...
For information on key differences with WSL 2 please visit https://aka.ms/wsl2
Unspecified error
PS C:\WINDOWS\system32> wsl --list --verbose
  NAME      STATE           VERSION
* Debian    Stopped         1
  Ubuntu    Stopped         1
PS C:\WINDOWS\system32>

At least I can still access WSL using V1 after the failed conversion. I was surprised by the ability to auto convert my V1 WSL to V2, this will be really nice when it works.

Hey @flynnham I had the same message as this and I resolved it by removing Acrylic DNS. Maybe see if you have that installed or something similar?

@kevin-coyle That seems to have done the trick, though I am 1100% certain I wouldn't have figured this out intuitively.

What's Acrylic DNS?

Thank for filing, let's track install / conversion failures with #4112.  Agreed, the error messages could be better, and when the root causes are identified I'll add some better errors.

@haydennyyy Acrylic DNS is like a supercharged override fot eh hosts file found in system32. I use it to enable me to develop websites on my Windows 10 machine using wildcard domains.
It looks like WSL currently has issues where programs that are altering the underlyuing windows networking tools, it causes it to fail. I'd try and seeing if you have any network altering programs running (like some antivirus programs, DNS resolvers, other network altering services) that are breaking it.

Well, I've never installed that...

Having the same problem. Worked for a few days after the initial install and then produced the same error message everyone else is getting. After the latest Windows update WSL2 started magically working again for a day or two then went back to throwing an error. Not sure where to find any useful logs, though.


fix(table-sheet): 修复明细表 tooltip 展示了错误的汇总数据的问题

👀 PR includes

🐛 Bugfix

[x] Solve the issue and close #0

📝 Description

🖼 Screenshot

Before
After

🔗 Related issue link

🔍 Self-Check before the merge

[ ] Add or update relevant docs.
[ ] Add or update relevant demos.
[ ] Add or update test case.
[ ] Add or update relevant TypeScript definitions.

:tada: This PR is included in version @antv/s2-v1.53.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-react-v1.45.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error in Modelica Mechanics

Modified by dietmarw on 1 Jun 2010 08:51 UTC
There seems to be a problem in Modelica.Mechanics.MultiBody.Parts.BodyCylinder
The error is in the calculation of r_CM which is set to
r_CM[3]=Modelica.Math.Vectors.normalize(r)*length/2

but must be
r_CM[3] = (r + r_shape) / 2;

Reported by erik.aberg on 1 Jun 2010 08:38 UTC
One of our customers suspects that there is a bug in Modelica Mechanics.
Best regards,
Erik Åberg
Original message:
The problem is in Modelica.Mechanics.MultiBody.Parts.BodyCylinder
The error is in the calculation or r_CM which is set to
r_CM[3]=Modelica.Math.Vectors.normalize(r)*length/2
but must be
r_CM[3] = (r + r_shape) / 2;

Migrated-From: https://trac.modelica.org/Modelica/ticket/373

Modified by dietmarw on 1 Jun 2010 08:51 UTC

Comment by otter on 26 Jul 2010 19:35 UTC
Fixed in c1a246c2b5e864994fef16814fb1a3e255d9cd93.
The correct formula is:
rCM = r_shape + Modelica.Math.Vectors.normalize(lengthDirection)*length/2
The same error was present in BodyBox and was also corrected.


plato.inspect does not recognize files with '.es' extension.

I have a hybrid project, some files are es6 (file extension .es) and some are ES5 ( file extension .js).
When I run plato analysis, only the *.js files are analyzed, not the *.es files.  I am able to get jshint, jscs and eslint to all load and analyze the *.es files, but not plato...
I saw an issue that was closed earlier stating that it would be in v2.0, when will that be out?  Is it possible to get this fix into 1.3.x ?
Thanks

https://github.com/es-analysis/plato/issues/184


Fixes spec error if date not loaded

When date is not loaded by default, this triggers an wrong number of arguments(3 for 0) exception.
Ensure date is present in the particular test.

:+1:

:+1:


Add INA219, current and voltage meter

Add INA219

See also
https://github.com/bareboat-necessities/bbn-m5atomS3-lite/issues/2

https://github.com/SolderedElectronics/Soldered-INA219-Current-Sensor-Arduino-Library


Fix link to flowlint-comments

The current link is incorrect and ends up on a 404. This change should fix that.

Whoops. Thanks for the fix!


Is it possible to get a natural log on a bigInt instance?

Hi there! I notice there's no log() function in the docs. is it possible to get a natural log on a bigInt instance?

Added it: https://github.com/peterolson/BigInteger.js/pull/27

This function returns a native JavaScript (floating point) number, which is unexpected in a big integer library because

the result is not usually an integer
the result cannot be arbitrarily large

Similar to the Math.exp function, this seems to be outside the scope of a big integer library. Do you have any examples of other integer libraries providing such a function?

Of course, that would break chaining. I see your point. I was really looking for a good mechanism to handle log() for big numbers but I do see your point. Perhaps a stand alone function outside this library would be considered more appropriate.

// Función logaritmo natural, take a look, please indent, the problem is the adjust at the end but it works fine, and exact. Bultet.
// Bien, ten presente, puede que quede mejor: { Log.N(X) := Log.2(X) / Log.2(N) } no lo he probado, yet.
Function Logarithm(a : Bigint) : Bigint; Overload;
Var
PA, SA : Bigint;
Begin
If (a = Zero) Or (a = One) Then Result := Zero Else
If (a = Two) Then Result := One Else
If MenorQue(a, IntToStr(Maxint)) Then
Result := IntToStr(Trunc(Ln(StrToInt(a))))
Else
Begin
PA := IntToStr(Trunc((Length(a) - 1)/Log10(Exp(1))));
SA := Logarithm(Divide(a, Exponent(PA)));
Result := Sumar(PA, SA)
End;
// Ajuste
While MenorQue(Exponent(Sumar(Result, One)), a) Do Increment(Result);
End;


Document the regex_set module

r? @emilio

@bors-servo: r+

:pushpin: Commit 735fb48 has been approved by emilio

:hourglass: Testing commit 735fb48 with merge f99d129...

:sunny: Test successful - status-travis


Scan a File without Storing file into disk

I want to scan a file, without storing the same on the disk.
As you know that the file which we are plan to scan, which may be vulnerable to attack, if we store the same there might be a chance of trigger.
Hence trying to avoid it and exploring an option to scan the file in memory as a stream.
Is that possible?
Thanks

You can use ClamClient.SendAndScanFileAsync which has overloads for a byte array or stream.

Thanks for the quick response.


Added command for installing K8s dev tools.

Thank you for contributing to the Universal Blue project!
Please read the Contributor's Guide before submitting a pull request.
Closes #308

Would you mind adding the Kubernetes.Brewfile here too?
Plus we'd avoid having bluefin and bazzite diverge, k8s nerds can be happy on both, thanks!

Would you mind adding the Kubernetes.Brewfile here too?
Plus we'd avoid having bluefin and bazzite diverge, k8s nerds can be happy on both, thanks!

Where exactly, it seems like you forgot to paste the link?

https://github.com/ublue-os/bluefin/blob/main/system_files/shared/usr/share/ublue-os/homebrew/kubernetes.Brewfile

I can do a follow up on this one with the bluefin/bazzite-cli ones so we can centralize.

ideally i would have liked it to use the install- verb, however it will get too wordy and it is doing something specific that will be fine without a verb.


Linebreaks Option for WYSIWYG

Add option for WYSIWYG widget.
http://imperavi.com/redactor/docs/settings/special/#setting-linebreaks

The annoyance specifically, for me anyways, would deal with the paragraphize setting.
I can teach a client to shift+enter to avoid creating new paragraphs upon new lines, if need be.
It's the automatically wrapping text initially in <p> tags that's a pain in the arse.
http://imperavi.com/redactor/docs/settings/clean/#setting-paragraphize


Linux packagers would hate static linking and would always rely on the system wide installation of dependencies

... it would be cool to be able to configure dependencies to use the system wide installations.
I did something like that for caveexpress: https://github.com/mgerhardy/caveexpress/tree/master/cmake see cp_add_library and cp_find

Sure, we could use cmake-options to select between the system-provided lib vs. local source.
I'm not sure where to put the cmake-logic, either 'centralized' into the oryol cmake-files, or decentralized into the cmake-files of fipsified projects. I guess both have their advantages vs disadvantages...
Doing this decentralized in the fipsified libs would require some extension to fips to prevent redundant helper cmake function in every fipsified lib (e.g. fips-glfw would come with a cmake-option whether to link the github version or the system-installed version).
I'm not a big fan of this feature but understand why 'Linux system builders' would want that :)
I think there's an underlying fips feature here which has to do with a more flexible dependency definition (I think we discussed in a separate ticket a while ago)... going into the direction of separating the fips-cmake-definitions of fipsified project from the actual github dependency import (so no longer using git submodule to embed the github project, and probably allowing easier overrides to use a different branch/tag/github repo or local version).


Ghost <-> Ghost Admin Repo Split Tooling

This issue is intended to be used for tracking all the tasks that remain in terms of needing tooling post repo split. @kevinansfield please feel free to edit this message as needed!
There are some small and some big tasks remaining as part of the repo split. They fall roughly into two categories - build tooling (largely being done by @kevinansfield at the moment but I'm sure he'd love some help) and release tooling, which is largely about making sure we can credit contributors to Ghost-Admin. We'd really really ❤  some help to do this.
Release Tools:
Build About Page

[ ] Move buildAboutPage & update it to count contributors from both repos

The buildAboutPage task currently lives in Ghost's grunt file, but as it creates templates and files for the client, it should really live in the Ghost-Admin grunt file. This task needs updating to pull down top contributors from both repositories, and combine the data. It's probably going to get much bigger, and might need moving out to its own repository.
Contributor list

[ ] Create a tool (grunt task?) to output the list of contributors for the release post.

Right now, we use this git alias to output a nice comma separated list of people who contributed, ordered by commits (not including merges):
contribs = "!f() { git shortlog -sn --no-merges $1.. | awk '{printf (\"%s %s, \", $2, $3)}'; }; f"
This approach isn't going to work now that Ghost-Admin is split out, so we need some other way to grab the names to put them in the release process.
Build Tools & Cleanup:

[ ] Add bower to Ghost-Admin
[ ] Add a new task (grunt init or grunt build?) to Ghost-Admin grunt file for installing dependencies
[ ] Remove bower from Ghost (PR open - #6910)
[ ] Remove old ember-related JSCS rules from Ghost (PR open - #6910)
[ ] Add a new task (grunt build ?) to Ghost which does NOT do submodule deps but does build everything.

New grunt task for managing both repos
To explain the idea behind the new grunt build (or whatever it gets named) task, it should be for developers (where grunt init will be only for installing now) to setup their environment for developing. Ideally it would take two PR numbers, to checkout & build specific PRs for both the Ghost and Ghost admin repos.
E.g. for scheduling, I might want to run grunt build --g=6861 --ga=25. This would set Ghost to PR 6861 and Ghost-Admin to PR 25. It would then run all the build tasks so that your repository was setup ready to test out the scheduler feature. Not that for this to work, it would probably require an update to your git config for both repos, adding fetch = +refs/pull/*/head:refs/remotes/upstream/pr/* to upstream as described here.

Just as a thought (since we're doing contribs for ghost and ghost-admin now), do we want to add contribs for Casper as well or no?

Another thought (something that should be added to the buildAboutPage task): Because we do a lot of PR's that are from GreenKeeper, it shows up as a top contributor. That's somewhat misleading, so we might want to add a clause to prevent that particular committer from showing up.

@ErisDS @kevinansfield I discovered something whilst working on this that will make the numbers skewed a bit.
There are duplicate commits between the two Ghost-Admin and Ghost repositories (the ones pertaining to the client), and because the commit shas are different, there's no easy way to compare the two commits to see if they are the same short of comparing the commit messages.
This will only occur until 90 days after the split, but I guess my question is should I have the commit messages be compared (would take edits to top-gh-contribs) or just go with the skewed number counts (for this release only most likely)?

This will only occur until 90 days after the split, but I guess my question is should I have the commit messages be compared (would take edits to top-gh-contribs) or just go with the skewed number counts (for this release only most likely)?

I'm happy to live with the duplication as its ownly short term, I think it's a fiddly thing to fix?

@ErisDS yeah trying to fix it is probably more trouble than it's worth :smile:


PEOPLE WANT TO KNOW!

Yes, yes, we want to know what plans the esteemed Showdown has about continuing the modding of UCP and the game of Stronghold Cruzader 1 itself?
I, and we will all be very happy if you tell us
What will be real (specific) in the new fashion?
What are you really planning to do yet?
Will you take into account the opinions and ideas of the community on certain issues? (Well, I have a lot of such ideas in my discussion thread)))).
Yes, sorry for the translation through the translator.
I will be very grateful to the author of the mod and those who really help him if you tell as much as possible about your mod and about the nearest plans. =)

They gonna add new bot in upcoming Patch,  Richard LemonHarth, he is weaker than Rat but more Agressive than Caliph, he attacks with mass Shields and Woodcutters (backed up with Hunters)

Sh0wdown is currently not working on the project, but new devs are. I think there are many hints scattered around the different issues what comes next, and with the latest commits you might guess what actually is going to be implemented ;)
The problem with stating what changes come when is, that people get their hopes up and if things don't go as planned, people get sad/upset/angry... this is why noone of the devs makes any fixed statements. Sh0wdown left some big shoes to fill, but I guarantee it, we will do so ;)
And I come out of cover: Release is very soon!

@J-T-de Thank you for not quitting your work  patch.
@ilosevsrat :)
@Russianstory Зайди в Дискорд группу там вся инфа по Патчу и последние Тестовые сборки. Да и попроси у Krarilotus или Kimberly чтобы они тебе дали роль тестера !

@Russianstory i would love to see you on the discord server of the patcher. You can find an invitation link on the front page of the Wiki or on the readme when you open the github page itself.

@Krarilotus I D LIKE TOO))) where is ?

@Russianstory

@Krarilotus I D LIKE TOO))) where is ?

You can find an invitation link to the discord on the front page of the Wiki or on the github wiki page

just click on that :) you are wellcome!

I think this can be closed now that the a new release is out.


[FP]: flexmark-ext-yaml-front-matter identified as go yaml project

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:yaml_project:yaml:1.33:::::::*
CVE
CVE-2021-4235, CVE-2022-3064
ODC Integration
{"label"=>"Gradle Plugin"}
ODC Version
7.4.4
Description
flexmark-ext-yaml-front-matter identified as go yaml project

Related to https://github.com/jeremylong/DependencyCheck/issues/5233 - will no doubt get many FPs here across libs.


Added missing unittest_cmake.sh

#1408
@mcallegari

I am not so clear how engine/test/qlci18n works.
Because qlci18n_fi_FI.ts_ is required for testing qlci18n and qlci18n_fi_FI.ts is required for compiling the target and now both are omitted from Git.
Tested on my side and it looks like working when I manually added these 2 files.
It looks like make check for Windows is missing in the todo list. Don't we need it anymore? It looks like that it was there in the original "qlc.pro" file.
I manually modified the 1st line of resources/fixtures/scripts/fixtures-tool.py from #!/usr/bin/env python to #!/usr/bin/env python3, otherwise it did not work. Not sure if python3 is default in your develop environment.

coverage: 28.053%. remained the same when pulling 33a72c4244c97dab7dfcd5e0c937680849061b83 on Itay-Lifshitz:cmakebuild into 76221757f8e6ff247b101736d5cd60e00235cef5 on mcallegari:cmakebuild.

@Itay-Lifshitz there's never been test units on Windows. They're executed only on Linux which is the reference OS (that's why I insisted to have a build working there first)
As for i18n I'll check later what was the intention of that test.
Thanks

@mcallegari You can review this PR anyway, I added the missing shell script file.

Looks good, thanks
Next is creating Debian package
See: https://karthikkalyanaraman.medium.com/creating-debian-packages-cmake-e519a0186e87
Seems like adding the following and using cpack will do the trick
SET(CPACK_GENERATOR "DEB")
SET(CPACK_DEBIAN_PACKAGE_MAINTAINER "KK") #required
INCLUDE(CPack)

@mcallegari

I was wondering how the original create-deb.sh worked.
It simply runs dpkg-buildpackage -rfakeroot -I.git --jobs=auto and then it compiles, tests, builds packages and signs automatically.
Where are these operations defined?
And I also got the following error when I tried to build debian package using the old create-deb.sh. How can I solve it?

I think dpkg-buildpackage recognized it was a qmake project and built it that way.
As I mentioned in my previous comment, with cmake we need to use cpack.
I'm gonna give it a try when I have some time

I think dpkg-buildpackage recognized it was a qmake project and built it that way. As I mentioned in my previous comment, with cmake we need to use cpack. I'm gonna give it a try when I have some time

I think I found it. Please refer to this link
It says Ubuntu is referring to debian directory when it is building a package.
Working on it now.

@Itay-Lifshitz please wait, I'm working on it right now.
I'll push some changes soon

@Itay-Lifshitz please wait, I'm working on it right now. I'll push some changes soon

OK, please let me know the updates when you push them.

I just pushed. A debian package is created, but I'm not sure if it is a proper one.
I think dependencies need to be fixed.
You can work on AppImage creation if you want

OK, will work on appimage.


[AUTOMATED] Update release.yml with npm environment

This PR was created automatically, as part of the efforts to address PHC-2100.

@ynotdraw I requested you for review because you are a top contributor. See https://lifeomic.slack.com/archives/C3NA7J4E8/p1642178097007500 for more info.

Closing this PR since my commit came through unsigned. I will re-open a PR with a signed commit, and re-request review.


Make the generator universal and fixed minimum OS

Fixes #798. Part of #223.
For local development of the generator we use macos_command_line_application instead of apple_universal_binary + swift_binary. This is to keep configurations the same between the generator and the tests, which makes the Xcode development experience better. If we used swift_binary or apple_universal_binary in xcodeproj, then the macos_unit_test transition (which is used to be able to set a minimum os version on the tests) will create slightly different configurations for our swift_librarys. Maybe https://github.com/bazelbuild/bazel/issues/6526 will fix that for us.

I'm currently working on reducing the fixture churn. Trying to minimize the impact of Starlark hash changes.


Some icons have unnecessary and disturbing title attributes

which makes it unpossible to overwrite a title attribute around the svg markup in inline rendering mode.

please define "some"

I will come up with a PR


etcd pods goto Completed state

vagrant@kube1 ~]$ kubectl get pods -ngcs
NAME                                   READY   STATUS      RESTARTS   AGE
csi-attacher-glusterfsplugin-0         2/2     Running     0          45m
csi-nodeplugin-glusterfsplugin-bh5ww   2/2     Running     0          45m
csi-nodeplugin-glusterfsplugin-d7db4   2/2     Running     0          45m
csi-nodeplugin-glusterfsplugin-vfd2v   2/2     Running     0          45m
csi-provisioner-glusterfsplugin-0      3/3     Running     0          45m
etcd-b2d2czvkq2                        0/1     Completed   0          14m
etcd-operator-7cb5bd459b-hl7hs         1/1     Running     0          51m
etcd-p5gk77fp5h                        0/1     Completed   0          14m
gluster-kube1-0                        1/1     Running     0          49m
gluster-kube2-0                        1/1     Running     1          49m
gluster-kube3-0                        1/1     Running     0          49m
[vagrant@kube1 ~]$
Normal   Pulled     15m                 kubelet, kube1     Container image "quay.io/coreos/etcd:v3.3.8" already present on machine
Normal   Created    15m                 kubelet, kube1     Created container
Normal   Started    15m                 kubelet, kube1     Started container
Warning  Unhealthy  13m (x2 over 14m)   kubelet, kube1     Liveness probe failed: rpc error: code = 2 desc = oci runtime error: exec failed: container_linux.go:247: starting container process caused "process_linux.go:110: decoding init error from pipe caused "read parent: connection reset by peer""
Warning  Unhealthy  13m (x17 over 14m)  kubelet, kube1     Readiness probe failed: rpc error: code = 2 desc = oci runtime error: exec failed: container_linux.go:247: starting container process caused "process_linux.go:110: decoding init error from pipe caused "read parent: connection reset by peer""
[vagrant@kube1 ~]$
Warning  Unhealthy  15m                 kubelet, kube2     Liveness probe failed: rpc error: code = 2 desc = oci runtime error: exec failed: container_linux.go:247: starting container process caused "process_linux.go:110: decoding init error from pipe caused "read parent: connection reset by peer""
Warning  Unhealthy  14m (x18 over 16m)  kubelet, kube2     Readiness probe failed: rpc error: code = 2 desc = oci runtime error: exec failed: container_linux.go:247: starting container process caused "process_linux.go:110: decoding init error from pipe caused "read parent: connection reset by peer""
[vagrant@kube1 ~]$
I have the commits:
commit d40e12a4c69256194d2a198c07469fde50b2725e
commit c164b49befa86f71caaeac1baa4e6bfa5928a25d

Looks like the errors are due to docker, https://github.com/openshift/origin/issues/21590
build docker-1.13.1-88 seems to have the fix, will verify the same and close the issue.

I ran into this as well. The fix I went with was continuing to use the 1808.01 version of the centos/atomic-host Vagrant box. The 1811.01 version is the one that has the bad Docker version.


ffuf: init at 1.1.0

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@htr please resolve the merge conflict.

@htr please resolve the merge conflict.

closing, ffuf has been merged since then

closing, ffuf has been merged since then


Add manifest validation pipeline for Azure Devops

Today we don't have any protection against merging PRs with faulty manifests in the gitops repositories. This PR adds a pipeline that can be used for PR build validation in Azure Devops. The pipeline performs a kustomize build but can be further extended in the future, e,g, linting.

@bittrance Yes you are right that we will do this for Github as well. Will be done in a separate PR though


Rename error interface getStatusCode

Codecov Report

Merging #61 into master will not change coverage.
The diff coverage is 50.00%.

@@            Coverage Diff            @@
##             master      #61   +/-   ##
=========================================
  Coverage     98.78%   98.78%
  Complexity      116      116
=========================================
  Files            20       20
  Lines           328      328
=========================================
  Hits            324      324
  Misses            4        4

Impacted Files
Coverage Δ
Complexity Δ

src/Error/RequestHeaderException.php
66.66% <0.00%> (ø)
5.00 <1.00> (ø)

src/Error/ErrorListener.php
100.00% <100.00%> (ø)
4.00 <0.00> (ø)

Continue to review full report at Codecov.

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


Fix C# editor dialogs

Use EditorInterface to open C# editor dialogs.
Ensure C# editor dialogs are open after EditorProgress finishes.

Fixes the error about dialogs mentioned in https://github.com/godotengine/godot/pull/80260#issuecomment-1736076386.

Thanks!


Code review

Great job with the project the app looks nice, I really like the like button functionality :+1:
There are some suggestions you can implement to make it more accessible:
You can rewrite the form validation each time by emptying the innertext inside so that it is in a single line, rather than coming up multiple times.

I've fixed it, thank you so much for your review. I'll be closing this issue. ✅


Headrooms

Objective of this pull request: Add headrooms (min and max capacities of a grid node). Required for network congestion management in the PT demo
@ArnaudDebray could you please create a new PR or update this one to inform that only the DSO can create or update the headrooms. The FSP should not be able to see the headrooms (impact on bidding strategy)

Hello. I will close this PR since we had the confirmation today that the headrooms are not required in the Portuguese demo


Update backup ALIASES

Fedora 35 has been released.

Backup alias definitions now reflect the official release of Fedora 35.

I added a changelog note, even though this is the backup solution.

I added a changelog note

where?


refactor keyname in SongRec.js

There is a mistake with the key name, my refactoring key names were not included in the last pr.
Bug fixed now

@mukhsadr Please, review this PR. It does not appear the songs are changing as the weather changes


"Docker run" on docker-in-docker container fails

The codespaces universal/kitchensink image provides sanity tests of the form docker run IMAGE ... ....  I caught and can repro docker-in-docker (and subsequently kitchensink) get these permission denied errors from the docker-init.sh. They are wrapped in SudoIf statements.  Strange because I can spin this up in remote containers just fine, but can repro with a docker run.
https://github.com/microsoft/vscode-dev-containers/blob/master/script-library/docker-in-docker-debian.sh#L144-L158

@Chuxel long shot, but have you seen anything like this before?
cc/ https://github.com/microsoft/vssaas-planning/issues/1588

@joshspicer, is it because we need to run the container with --privileged?

@joshspicer, is it because we need to run the container with --privileged?

Yep 😅.  Swear i'd tried that earlier, I think i'd placed that flag in the wrong order.

Yep 😅.  Swear i'd tried that earlier, I think i'd placed that flag in the wrong order.


provide viewsets for relations

For starters ListView and DetailView should be sufficient

basic listview is implemented; what is needed is a generic way to create filterforms with autocompletes (the current version only provide default drop-downs which will cause performance issues on bigger data sets)


[DO NOT MERGE] Test RC provider packages for https://github.com/apach…

[DO NOT MERGE] Test RC provider packages for https://github.com/apache/airflow/issues/33751

@vatsrahul1001 we also need to update the constraint file.

@vatsrahul1001 we also need to update the constraint file.

why constraint file? this PR is to run a test against the new provider rc. we use constraint file only to run test for airflow 2.2.5


fix: clean client cache when feature flags change

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Magic flag?

-w? It's short for --workspace-root, i.e. where the script lives.

Ah - helpful that it's listed there and not https://pnpm.io/cli/run 🤦🏻♀


[W7.8a][T09-1]Sean Tan Rui Xiang

added command that lists all persons in address book but sorted alphabetically by name.

Okay, according to the ivle announcement and the github LO pages for AB3, theres AB3 enhancement, but i can't for the life of me find the LO code. :/


refactor: introduce @validate_body_root_model

Add @validate_body_root_model which is a drop-in replacement for Sanic's @validate when validating a RootModel or a model deriving from it.
The decorator should be replaced by @validate once https://github.com/sanic-org/sanic-ext/issues/198 is fixed.
/deploy

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

39 of 40 (97.5%)  changed or added relevant lines in 5 files are covered.
7 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.02%) to 90.473%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

components/renku_data_services/base_api/misc.py
12
13
92.31%

Files with Coverage Reduction
New Missed Lines
%

components/renku_data_services/base_api/pagination.py
1
84.21%

components/renku_data_services/users/db.py
1
94.18%

components/renku_data_services/storage/blueprints.py
2
95.24%

components/renku_data_services/crc/blueprints.py
3
93.5%

Totals

Change from base Build<PHONE_NUMBER>5:
0.02%

Covered Lines:
9344

Relevant Lines:
10328

💛  - Coveralls

You can access the deployment of this PR at https://renku-ci-ds-423.dev.renku.ch

Tearing down the temporary RenkuLab deplyoment for this PR.


Redis backups

This PR adds the redis backups to the saas-operator as a new api with a new controller. An example of the custom resource with the status:
apiVersion: saas.3scale.net/v1alpha1
kind: ShardedRedisBackup
metadata:
  finalizers:
    - saas.3scale.net
  name: backup
  namespace: default
spec:
  dbFile: /data/dump.rdb
  historyLimit: 2
  pause: false
  pollInterval: 10s
  s3Options:
    bucket: my-bucket
    credentialsSecretRef:
      name: aws-credentials
    path: backups
    region: us-east-1
    serviceEndpoint: http://minio.default.svc.cluster.local:9000
  schedule: '* * * * *'
  sentinelRef: sentinel
  sshOptions:
    port: 2222
    privateKeySecretRef:
      name: redis-backup-ssh-private-key
    sudo: true
    user: docker
  timeout: 5m0s
status:
  backups:
    - message: backup scheduled
      scheduledFor: "2023-09-20T08:33:00Z"
      shard: shard02
      state: Pending
    - message: backup scheduled
      scheduledFor: "2023-09-20T08:33:00Z"
      shard: shard01
      state: Pending
    - backupFile: s3://my-bucket/backups/redis-backup_shard02_2023-09-20T08:32:01Z.rdb.gz
      backupSize: 234
      finishedAt: "2023-09-20T08:32:12Z"
      message: backup complete
      scheduledFor: "2023-09-20T08:32:00Z"
      serverAlias: redis-shard-shard02-1
      serverID: <IP_ADDRESS>:6379
      shard: shard02
      startedAt: "2023-09-20T08:32:01Z"
      state: Completed
    - backupFile: s3://my-bucket/backups/redis-backup_shard01_2023-09-20T08:32:01Z.rdb.gz
      backupSize: 232
      finishedAt: "2023-09-20T08:32:12Z"
      message: backup complete
      scheduledFor: "2023-09-20T08:32:00Z"
      serverAlias: redis-shard-shard01-1
      serverID: <IP_ADDRESS>:6379
      shard: shard01
      startedAt: "2023-09-20T08:32:01Z"
      state: Completed

Features:

Backups are triggered at the same time for all redis shards. Even though this does not ensure perfect consistency among shards, it's way better than the current scenario.
Backups are fully executed from the operator. It only requires the redis instance to have ssh, gzip and awscli tools installed. An ssh user and private key must be provided for access to the instance
The s3 service endpoint can be injected in the custom resource to point to a minio instance or any other s3 compatible api.
The following metrics are published for backups: size, failure_count, success_count, duration.
Backup status is stored in the custom resource status. Errors are reported in the logs, in the status and in the metrics, greatly improving backup observability.
Automated e2e test that ensures backups do work.
BGSave command is directly executed instead of being scheduled (as we were doing up until now) so this removes the need to manually configure the RO slave in each shard.
The controller automatically looks for the RO slave in the shard by inspecting the Sentinel status.
Backups can be paused using the pause flag in the custom resource.

Limitations:

The backups are each run by its own goroutine. The problem with this approach is that anything that kills or terminates the saas-operator pod instantly kills all running backups. I decided not to wait for running backups to finish before the controller shutting down as this could mean waiting for a long time. I don't think this limitation has a huge impact as the saas-operator is very stable in production and if a backup is terminated ahead of time, the next one will just trigger soon enough. The way to overcome this limitation would be to run each backup in a Job, but this greatly complicates things, specially reporting the status of the backup back to the controller.

/kind feature
/priority important-soon
/assign

/retitle Redis backups

@slopezz @raelga the PR is now ready for review

@slopezz @raelga final commits to fix some bugs are in place and tested in staging env:

fixed memory issue (not closing ssh connections properly)
fixed cleanup of redis backup files in redis instances
added some validation to the required Secrets that the backup uses
improved the AWS client configuration (extracted functionality to util package)
fixed a minor problem with the schedule of backups when the schedule is changed

/approve

Related to https://github.com/3scale/platform/issues/868


Update Twitter Logo in the footer section to 'X'

Description
The current logo of the Twitter icon on the website needs to be updated to the new 'X' logo. This change will ensure that the website remains up-to-date with the latest branding of Twitter. The new 'X' logo reflects the current design and branding guidelines of Twitter.
Proposed Solution:
Replace the current Twitter logo with the official 'X' logo of Twitter.
Update the website's CSS or image asset to reflect the new logo.
Additional Information:
Twitter's official branding guidelines can be found here.
The 'X' logo image file can be obtained from the official Twitter branding resources.
Attachments:

I want to work on this , can you assign me this?

@Debaditya-Som yeah fix this.

@Debaditya-Som yeah fix this.

Okay I'm on it

Sending a pr request right now

@Debaditya-Som yeah fix this.

I have sent a PR solving the above mentioned issue , thank you


Govendor version

Govendor doesn't know about its own version and thus can't tell the user if they're using a different version than someone else.

govendor -version

Looks like I was on an older govendor after doing a go get github.com/kardianos/govendor which didn't have it.
$ govendor -version
Error: Unknown command "-version"
govendor: copy go packages locally. Uses vendor folder.
govendor init
	Creates a vendor file if it does not exist.

govendor list [options] ( +status or import-path-filter )
	List all dependencies and packages in folder tree.
	Options:
		-v           verbose listing, show dependencies of each package
		-no-status   do not prefix status to list, package names only

govendor {add, update, remove} [options] ( +status or import-path-filter )
	add    - Copy one or more packages into the vendor folder.
	update - Update one or more packages from GOPATH into the vendor folder.
	remove - Remove one or more packages from the vendor folder.
	Options:
		-n           dry run and print actions that would be taken
		-tree        copy package(s) and all sub-folders under each package

		The following may be replaced with something else in the future.
		-short       if conflict, take short path
		-long        if conflict, take long path

govendor migrate [auto, godep, internal]
	Change from a one schema to use the vendor folder. Default to auto detect.

Expanding "..."
	A package import path may be expanded to other paths that
	show up in "govendor list" be ending the "import-path" with "...".
	NOTE: this uses the import tree from "vendor list" and NOT the file system.

Flags
	-n		print actions but do not run them
	-short	chooses the shorter path in case of conflict
	-long	chooses the longer path in case of conflict

"import-path-filter" arguements:
	May be a literal individual package:
		github.com/user/supercool
		github.com/user/supercool/anotherpkg

	Match on any exising Go package that the project uses under "supercool"
		github.com/user/supercool/...

	Match the package "supercool" and also copy all sub-folders.
	Will copy non-Go files and Go packages that aren't used.
		github.com/user/supercool/^

	Same as specifying:
	-tree github.com/user/supercool

Status list used in "+<status>" arguments:
	external - package does not share root path
	vendor - vendor folder; copied locally
	unused - the package has been copied locally, but isn't used
	local - shares the root path and is not a vendor package
	missing - referenced but not found in GOROOT or GOPATH
	std - standard library package
	program - package is a main package
	---
	outside - external + missing
	all - all of the above status

Status can be referenced by their initial letters.
	"st" == "std"
	"e" == "external"

Status can be joined together with boolean AND and OR
	govendor list +vendor,program +e --> (vendor AND program) OR external

Ignoring files with build tags:
	The "vendor.json" file contains a string field named "ignore".
	It may contain a space separated list of build tags to ignore when
	listing and copying files. By default the init command adds the
	the "test" tag to the ignore list.

If using go1.5, ensure you set GO15VENDOREXPERIMENT=1

Examples:
	$ govendor list -no-status +local
	$ govendor list +vend,prog +local,program
	$ govendor list +local,!prog

had to re-compile from master source.


Symfony 5 without FosUserBundle

I'm currently migrating my project to Symfony 5.
FOSUserBundle being deprecated, I've unplugged it from my project.
But now I've an error at that line:
https://github.com/kevinpapst/TablerBundle/blob/5110ae631e47581880c668d456c20a9dd3d2bafa/templates/security.html.twig#L68-L70

❔ Who define that variable ❔
If I change the line by {% if csrf_token | default(true) %}, it works.
What is the purpose of that "if" condition ?

Weird, that should check if the csrf_token() method is available.
I removed FOSUserBundle as well and have no problem with that line, but still running SF4 (not migrated to 5 yet).

We can probably remove that code entirely. I just know that I had users that had the opposite problem. Without if the code failed when trying to run csrf_token method, which wasn't available in their context. But that was quite some time ago.
I cannot reproduce either case 🤷

that should check if the csrf_token()

That what I thought..
I've tried to found the "best" way to check if a function/filter exist in twig and I don't really found something useful..

We can probably remove that code entirely.

Instead of completely removing it, we can convert the if by a block.
That allow devs to override that block to remove the csrf if they want to 🤔

Did you try if csrf_token is defined ?
If the defined check does not work, make it a block. I don't really care, because I truly hope that no one ever uses a login form with csrf token...

Yeah I've already tried it, and it returns false..
What is your twig version installed ?
root@bd73a300ef88:/var/www/html# composer show twig/twig
name     : twig/twig
descrip. : Twig, the flexible, fast, and secure template language for PHP
keywords : templating
versions : * v3.3.4

name     : twig/twig
descrip. : Twig, the flexible, fast, and secure template language for PHP
keywords : templating
versions : * v3.3.3

But the csrf_token() method is coming from the framework or security bundle, not sure without looking it up.

I wasn't looking for the csrf_token() method, but the way twig interpret the if condition but you are n V3 too.
Anyway, like you,
I don't get why people doesn't want to use the csrf, but I can switch it into block.

Which seems to be some old code that is not required. I have to delete it. As you have to change the template anyways: choose the solution you want. Removing the if is a stronger statement for best security practices. Having a block makes it more configurable. I would merge both...


docs: Make sure all partner packages have README

Privileged issue

[X] I am a LangChain maintainer, or was asked directly by a LangChain maintainer to create an issue here.

Issue Content
Add a good README to each of the libs/partners packages. See langchain-google-vertexai for a good reference.

I'd be happy to contribute to this.

I'd also like to contribute to this!

I'd be happy to contribute. I checked some of them and looks like nvidia-trt is missing a readme but has docs/. OpenAI only has no example usage but links to comprehensive examples. Should these be expanded or are they OK as is?

Happy to contribute to this.


Some cleanup for unstable docs

This cleans up some issues with the unstable docs. See each individual commit for a description.

@bors r+

:pushpin: Commit c515e4a833d28b08ebe903c6543e3a5d671f7158 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit c515e4a833d28b08ebe903c6543e3a5d671f7158 with merge 2dcdf1e2e4dbb22164c3b26eb6c667f48359684d...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 2dcdf1e2e4dbb22164c3b26eb6c667f48359684d to master...


Adding contributors to MAPLE

Adding contributors by using bot

@all-contributors
please add @ for .
please add @ for .
please add @ for .
please add @ for .
please add @ for .

https://github.com/all-contributors
please add @hshareef9 for code.
please add @MarkTrepanier for code.
please add @akh4rf for code.
please add @oherma01 for code.
please add @OmegaDeanGrey for code.

@all-contributors
please add @hshareef9 for code.
please add @MarkTrepanier for code.
please add @akh4rf for code.
please add @oherma01 for code.
please add @OmegaDeanGrey for code.

npm run fix


[GH4576] Add options to reporter config which will be passed down to reporter init function.

This PR introduces the options property under the reporter config section.
The purpose of this new property is to pass down configuration to the custom reporter without using ENV variables.
fixes #4576

Hi @DamianOsipiuk,
Thank you for your desire to improve and contribute to TestCafe. I apologize for the delayed response.
We discussed your pull request with the team and decided not to accept it due to the lack of command-line and 'configuration file' interfaces support. At present, there are not many customer requests for the 'Add capability to set-up reporter options' feature.
For now, let's leave it as is. The recommended way to pass custom options to a reporter is to use environment variables.


Download option in addition to INSTALL as part of PUBLIC distribution

The solution I'd like
When .IPA files are distributed to PUBLIC access group, there is only INSTALL option against each .IPA file. It would be highly helpful to have DOWNLOAD option for the clients to download the app and distribute using their MDM tools.
Alternatives
Right now I have go to the releases and download the .IPA from there and give it to the clients to upload into their MDM.
Additional context

Hi @rughvi, this one is on our iteration plan for this month already. We'll get it shipped next week most likely.

Changes are in prod, you should be able to download the ipa on any client browser

Hi team,
I still can't see the download option on MAC desktop.
Expected : On iPad/iPhone INSTALL option, on MAC / WINDOWS desktop DOWNLOAD option.
Actual : On MAC desktop it says 'This app is not compatible with MAC OS' but still shows INSTALL option.

This was implemented for Android and currently does not work for iOS. Reopening this issue and creating a bug for it.


REACT: Added Home Page

Issue: #357
I have added Home Page in src/pages/HomePage.

@anushbhatia Please check the file structure

@anushbhatia Please check the file structure

@smaranjitghose Let me know if any changes needed.
And please change the label from SWoC2021 to DWOC and Level-x.
Thank you!

@smaranjitghose Let me know if any changes needed.
And please change the label from SWoC2021 to DWOC and Level-x.
Thank you!

Don't use the same template as the current project. Create each component from scratch and then we can proceed each element along with it.

What do you mean by from scratch?

Don't use the same template as the current project. Create each component from scratch and then we can proceed each element along with it.

What do you mean by from scratch?

Image location should be modified  src/assets/images for team images
Use black colour everwhere for the colour scheme in header footer and main gradient on home page too.

@anushbhatia Ok, I am fixing point 1. But point 2 is invalid. Refer #357 . I was told to make it similar to basic HTML, CSS, and JS website version.

Image location should be modified  src/assets/images for team images
Use black colour everwhere for the colour scheme in header footer and main gradient on home page too.

@anushbhatia Ok, I am fixing point 1. But point 2 is invalid. Refer #357 . I was told to make it similar to basic HTML, CSS, and JS website version.

Image location should be modified  src/assets/images for team images
Use black colour everwhere for the colour scheme in header footer and main gradient on home page too.

@anushbhatia Ok, I am fixing point 1. But point 2 is invalid. Refer #357 . I was told to make it similar to basic HTML, CSS, and JS website version.

I guess what @anushbhatia means is you have exactly copy pasted the css and js from the base version and rendered it into react components and pages. It was required to have a similar UI but not the same messy code

Image location should be modified  src/assets/images for team images
Use black colour everwhere for the colour scheme in header footer and main gradient on home page too.

@anushbhatia Ok, I am fixing point 1. But point 2 is invalid. Refer #357 . I was told to make it similar to basic HTML, CSS, and JS website version.

I guess what @anushbhatia means is you have exactly copy pasted the css and js from the base version and rendered it into react components and pages. It was required to have a similar UI but not the same messy code

@smaranjitghose Please check once again. It was not just a copy/paste work. I have spent more than half of my day to migrate the whole HomePage to React.

@smaranjitghose Please check once again. It was not just a copy/paste work. I have spent more than half of my day to migrate the whole HomePage to React.

Here, in Home Section, previously bootstrap was used for CSS. Whereas I have used my own custom CSS.
I agree that I have copied CSS from the base branch in many places because I want to make the HomePage look simliar to the basic version of the website.
@smaranjitghose
And please tell me, in which Component did you find the messy code?

Here, in Home Section, previously bootstrap was used for CSS. Whereas I have used my own custom CSS.
I agree that I have copied CSS from the base branch in many places because I want to make the HomePage look simliar to the basic version of the website.
@smaranjitghose
And please tell me, in which Component did you find the messy code?

@smaranjitghose and @anushbhatia at least reply.

@smaranjitghose and @anushbhatia at least reply.

Convert the complete theme to black and white colours.

Convert the complete theme to black and white colours.

@anushbhatia example?

@anushbhatia example?

https://leerob.io/

https://leerob.io/

@anushbhatia It will be better if this is done in a separate issue.

@anushbhatia It will be better if this is done in a separate issue.

Ok then do it after i merge this PR

Ok then do it after i merge this PR


Don't minify the Action's standalone JavaScript bundle

Checklist

[x] I left no linting errors in my changes.
[x] I tested my changes.
[x] I updated the documentation according to my changes.
[x] I added my change to the Changelog.

Description
A suggestion to stop minifying the Action's standalone JavaScript bundle.
Doing this removes a dependency (with corresponding benefits), reduces the build time (and by extension CI times), and makes the standalone more human-readable (and by extension potentially makes reviewing it's changes possible, though at >220000 probably pretty difficult in most cases).

I'm considering this superseded by #730 as it addresses one of the main concerns this Pull Request is about:

[...] makes the standalone more human-readable (and by extension potentially makes reviewing it's changes possible, though at >220000 probably pretty difficult in most cases).

in a different way. By signing the build output in the CI, users can have increased confidence the build was legitimate. Plus, for even greater confidence they can pull down the repository and build themselves.


Terminal crashes when trying to run it as administrator from a non administrator windows user

Windows Terminal version (or Windows build number)
Microsoft Windows [Version 10.0.19042.868], Windows Terminal [Version 1.6.10571.0]
Other Software
No response
Steps to reproduce

have a windows user with admin rights and a windows user without admin rights
log in to the user without admin rights
try to run the new windows terminal as an administrator
when the promt to give the admin password: do that
the app crashes

Expected Behavior
I expect to open the windows terminal in administrator mode
Actual Behavior
The windows terminal crashed with the below error message.
Error message:

Thanks for the report! This currently isn't possible for applications installed from the Store. If you want to run the Terminal as an administrator, then you'll need the account the Terminal is installed for to also be an admin.
This is actually already being tracked by another issue on our repo - please refer to #4217 for more discussion.
/dup #4217


Errors during importing KST to SunriseWorkbench

Dear Dr. Safeea,
I’m following your KST tutorials and I encounter some problems.
I’m now trying to import KST to SunriseWorkbench (Version 1.7). But there are some error icons near the copied java files. I have added the Direct Servo Motion Extension and Smart Servo Motion Extension in the StationSetup.cat, however, the errors still exist. The specific error messages are listed in the attachment below.
I have tried to select different media flange when establishing a new project and to use different KST version from KST_1.6 to KST_1.7, but they are no use. Could you help me analyze the reasons of the problems that I have? I really don’t know how to deal with it.
Thanks a lot.
Tian Xu.
Error Messages.pdf

Hello Tian,
After looking at the PDF that you provided, it seems to me that the cause of the errors is due to two main reasons:
1- Media flange: your robot has an electric flange, so make sure that you are using the Sunrise Server with the name universal_no_flange
2- Direct/Smart servo libraries: your Workbench is not recognizing the Direct/Smart servo libraries.  Now, if you followed the steps in the YouTube video (Add DirectServo and SmartServo libraries to Sunrise.Workbench), then make sure that you tick the boxes near the DirectServo and SmartServo in the (StationSetup.cat) file. Sometimes the errors do not disappear immediately after you modify the (StationSetup.cat) file. In such a case, you have to save the project and normally the errors will disappear.
Try previous steps, if they did not work attach a photo of the (StationSetup.cat) file.
Cheers

Dear Dr. Safeea,
1-    We’re sure that we use the ‘KST_1.6_iiwa14R820_Universal_noFlange’.
2-    We have added the two libraries to sunrise workbench (version 1.7). Besides, we indeed ticked the boxes. The photo of our (StationSetup.cat) file. was shown in Fig.1 in the attachment.
Whether the errors are caused by our low version of sunrise workbench, 1.7? Today, I tried to use another class names to replace the original class names in your toolbox, for example,
‘import com.kuka.roboticsAPI.motionModel.DirectServo’  instead of
‘import com.kuka.connectivity.motionModel.directServo.DirectServo’
‘import com.kuka.roboticsAPI.motionModel.IDirectServoRuntime’ instead of
‘import com.kuka.connectivity.motionModel.directServo.DirectServo’
‘import com.kuka.roboticsAPI.motionModel.ISmartServoRuntime’  instead of
‘import com.kuka.connectivity.motionModel.smartServo.ISmartServoRuntime’
‘import com.kuka.roboticsAPI.motionModel.ServoMotion’  instead of
‘import com.kuka.connectivity.motionModel.smartServo.ServoMotion’
‘import com.kuka.roboticsAPI.motionModel.SmartServo’  instead of
‘import com.kuka.connectivity.motionModel.smartServo.SmartServo’
After these replacement, the errors in the java files disappear, however, when I try to synchronize the project, another error appears, which is shown in Fig.6 in the attachment..
Do you think if my method of revising the class names are right? Besides, if our sunrise workbench version 1.7 are okay for the KST? And what should I do to deal with the original errors and the new error?
I’m really sorry to disturb you so much and thank you for your reply and help very much!
Tian Xu.
------------------ 原始邮件 ------------------
发件人: "Mohammad<EMAIL_ADDRESS>发送时间: 2019年9月15日(星期天) 下午5:06
收件人<EMAIL_ADDRESS>抄送: "天涯芳草"653219708@qq.com;"State<EMAIL_ADDRESS>主题: Re: [Modi1987/KST-Kuka-Sunrise-Toolbox] Errors during importing KSTto SunriseWorkbench (#7)
Hello Tian,
After looking at the PDF that you provided, it seems to me that the cause of the errors is due to two main reasons:
1- Media flange: your robot has an electric flange, so make sure that you are using the Sunrise Server with the name universal_no_flange
2- Direct/Smart servo libraries: your Workbench is not recognizing the Direct/Smart servo libraries. Now, if you followed the steps in the YouTube video (Add DirectServo and SmartServo libraries to Sunrise.Workbench), then make sure that you tick the boxes near the DirectServo and SmartServo in the (StationSetup.cat) file. Sometimes the errors do not disappear immediately after you modify the (StationSetup.cat) file. In such a case, you have to save the project and normally the errors will disappear.
Try previous steps, if they did not work attach a photo of the (StationSetup.cat) file.
Cheers
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

Dear Dr. Safeea,
1-    We’re sure that we use the ‘KST_1.6_iiwa14R820_Universal_noFlange’.
2-    We have added the two libraries to sunrise workbench (version 1.7). Besides, we indeed ticked the boxes. The photo of our (StationSetup.cat) file. was shown in Fig.1 in the attachment.
Whether the errors are caused by our low version of sunrise workbench, 1.7? Today, I tried to use another class names to replace the original class names in your toolbox, for example,
‘import com.kuka.roboticsAPI.motionModel.DirectServo’  instead of
‘import com.kuka.connectivity.motionModel.directServo.DirectServo’
‘import com.kuka.roboticsAPI.motionModel.IDirectServoRuntime’ instead of
‘import com.kuka.connectivity.motionModel.directServo.DirectServo’
‘import com.kuka.roboticsAPI.motionModel.ISmartServoRuntime’  instead of
‘import com.kuka.connectivity.motionModel.smartServo.ISmartServoRuntime’
‘import com.kuka.roboticsAPI.motionModel.ServoMotion’  instead of
‘import com.kuka.connectivity.motionModel.smartServo.ServoMotion’
‘import com.kuka.roboticsAPI.motionModel.SmartServo’  instead of
‘import com.kuka.connectivity.motionModel.smartServo.SmartServo’
After these replacement, the errors in the java files disappear, however, when I try to synchronize the project, another error appears, which is shown in Fig.6 in the attachment..
Do you think if my method of revising the class names are right? Besides, if our sunrise workbench version 1.7 are okay for the KST? And what should I do to deal with the original errors and the new error?
I’m really sorry to disturb you so much and thank you for your reply and help very much!
Tian Xu.
Error messages2.pdf

Dear Tian,
Unfortunately, I could not find an answer to the issue. The KST was tested on Sunrise.OS/workbench <IP_ADDRESS> on iiwa 7R800 which is the only KUKA robot we have in the university. Probably you can ask KUKA for support?
In case you get it to work, please share with us the solution.
Don't worry, you are not disturbing, good luck
Regards

Dear Dr. Safeea,
Thank you for your reply.
A good news! We have solved our previous problems. Now we share the solutions with everyone here.
Step 1: We install the ‘Direct Servo Motion Extension’ library to our project because we miss this library. (For other people, please make sure that the ‘Direct Servo Motion Extension’ library and the ‘Smart Servo Motion Extension’ are both installed in the project.)
Step 2: We comment the following syntaxes,
‘import com.kuka.connectivity.motionModel.directServo.DirectServo’ and ‘import com.kuka.connectivity.motionModel.directServo.IDirectServoRuntime’ in the BackgroundTask.java and MatlabToolboxServer.java;
‘import com.kuka.connectivity.motionModel.smartServo.ISmartServoRuntime’, ‘import com.kuka.connectivity.motionModel.smartServo.ServoMotion’ and ‘import com.kuka.connectivity.motionModel.smartServo.SmartServo’ in the SmartServoWithImpedence.java.
Step 3: Some errors still exist because now we have not imported the direct servo and smart servo library in the project. At this moment, we just put the mouse for the error code line to use the suggested import by the sunrise. Then, the errors will disappear.
Step 4: We modified the package name to ‘application’, not the ‘lbrExampleApplications’, because our codes are all in the ‘application’ package. Other people may not need to conduct this step. If you want to do this like us, please remember to also modify the names, such as ‘application’ instead of ‘lbrExampleApplications’, in first line in all codes of the KST.
Done.
Our workbench version is 1.7. And all the libraries version that we use and the specific discriptions are shown in the attachment below.
Thank you for your reply and help all the time!
Tian Xu.
Solution for our problems.pdf

Hi Mr Tian
Please, how did you install : Direct Servo Motion Extension and Smart Servo Motion Extension?
and please,  if you still have the file maybe you can share it with us and we are really appreciate it.
Thank you
Maitham

Hi Mr Tian
Please, how did you install : Direct Servo Motion Extension and Smart Servo Motion Extension?
and please, if you still have the file maybe you can share it with us and we are really appreciate it.
Thank you
Maitham

Hello everyone,
Are these software libraries in all iiwa or do I need to ask kuka for these two software installations?
when i go to the software tag there I cannot see Direct Servo Motion Extension nor Smart Servo Motion Extension?
thank you for your help.

If anyone is looking for these packages, apparently is part of the sunrise.servoing tecnology package.
You need to buy from kuka.

Hello,
If you do not have the Sunrise.Servoing librarry, you can check the following package IIWAContorlToolbox  which works without requiring the servo librarry.
Cheers

Hello,
If you do not have the Sunrise.Servoing librarry, you can check the following package IIWAContorlToolbox  which works without requiring the servo librarry.
Cheers

Oh yeah I downloaded that, but haven't tried it yet. I was looking at the code and wondering what differences there are?
Can I still control the robot through Matlab? I'm guessing no, right?
What I wanted to try were using remote controller to move the robot or try an implementation like you did with hand sensor collision avoidance, but i think that is impossible without the technology.
Thank you for your help and support. This repo that you made is god send haha
It would be helpful to know what limitations there are between your two packages IIWAContorlToolbox  and KST


InstantiateTemplateResponse can return an empty email

In CreateCustomEmail.cs, the InstantiateTemplateResponse  can return an empty email if the template is not set up correctly.
I think there needs to be a check on the email.Id to make sure it is not equivalent to Guid.Empty.

If the template is incorrect, then there shouldn't be a point going any further.
In GetEmailInstantiateFromTemplate, it will always log
TraceService.Trace("Email Template has been instanciated correctly. Email ID:{0}", email.Id); whether or not a template was found or not.
If it can find a template, a valid Guid is stored in email.Id, otherwise it is Guid.Empty. (It's been 4 months, but I am pretty sure this is correct)
I believe that if email.Id is Guid.Empty, then the email has not been instantiated correctly and it should be handled properly (via logging or exception) in GetEmailInstantiateFromTemplate or later on the creation of the email.
Again, it's been 4 months since I reported this, but I think I had the incorrect name, and this would have helped troubleshoot my issue a lot easier.

That makes some sense, but the InstantiateTemplateRequest takes an ID for the Template, not a name. If the name of the template was not found, that would appear in the RetrieveEmailTemplate method -- and in that case, an exception is thrown with a pretty meaningful message.
            if (templatesCollection.Entities.Count == 0)
                throw new Exception("A template couldn't be found with name " + templateName);
            else if (templatesCollection.Entities.Count > 1)
                throw new Exception("There is more than one template with name " + templateName);
            else
            {
                TraceService.Trace("Email Template has been retrieved correctly");
                return templatesCollection.Entities[0];
            }

I haven't run this recently either, but my understanding is that Guid.Empty is the normal value for the returned email entity, because it doesn't create the record in the system. Guid.Empty in this case isn't an error indicator, it's always the ID of any record that doesn't exist.
I know that because I just had to fix a bunch of javascript in a 2011-2016 migration because it used getId() ==null to detect when it was on the create form instead of using getFormType() == 0.

What you said makes sense. I'll investigate further later with an incorrect template name again and see what I get in the logs...


Cannot find name ... errors when using programmatically with declarations

c:/Users/felix/git/Projekte/registerex/node_modules/typedoc/lib/converter/types/intrinsic.d.ts(5,105): error TS2305: Module 'ts' has no exported member 'IntrinsicType'.
c:/Users/felix/git/Projekte/registerex/node_modules/typedoc/lib/converter/types/intrinsic.d.ts(6,45): error TS2305: Module 'ts' has no exported member 'IntrinsicType'.
c:/Users/felix/git/Projekte/registerex/node_modules/typedoc/lib/converter/types/intrinsic.d.ts(7,44): error TS2305: Module 'ts' has no exported member 'IntrinsicType'.
c:/Users/felix/git/Projekte/registerex/node_modules/typedoc/lib/output/events.d.ts(20,15): error TS2304: Cannot find name 'HandlebarsTemplateDelegate'.
c:/Users/felix/git/Projekte/registerex/node_modules/typedoc/lib/output/utils/resources/templates.d.ts(4,20): error TS2304: Cannot find name 'HandlebarsTemplateDelegate'.

Seems like there are some dependencies typedoc can't live without - should the declarations be published on typings/registry instead?

Having simillar issue, where you able to fix this?

Just ignoring it for now. @blakeembrey could you comment on this?

My issue is opened in https://github.com/TypeStrong/typedoc/issues/268
Also have been ignoring it as the app runs fine, but because I  am using with angular2 there is actually a big list of things I need to include, so its quite a list of warnings;P

Most likely both are from global definitions. At least the Handlebars one is. It may be possible to switch to using Typings, or to @types when 2.0 is stable.

@blakeembrey I tried installing the definitions with typings, but typedoc is still using tsd: https://github.com/TypeStrong/typedoc/blob/master/tsd.json. Would be nice if we could add a typings.json until @types is ready.

I believe there's a PR for @types, finishing that up ASAP.

These errors might go away if you
npm install --save-dev @types/handlebars

@christopherthielen It doesn't fix the private properties we have exposed from TypeScript though, which are the first three errors.

oh, right, now I understand.  sorry for the noise!


Adding pyspark example

Description
Adding an example Jupyter Notebook to demonstrate how we've implemented our experimental pyspark integration. Feel free to read through the notebook and suggest anything that should be different.
General Checklist

[x] Tests added for this feature/bug
if it was a bug, test must cover it.
[ ] Conform by the style guides, by using formatter
[x] Documentation updated
[ ] (optional) Please add a label to your PR

This looks great. Couple things to check on:

Can this have an Open in Collab button?
Check to see if there is anyway to resolving the warning/errors. Especially the two blanks ones.

I also agree with Melly's suggestions, but we can do in a follow up too. The warnings I think Melly is referring to are the "red" cell output in the notebook example.
Also you might try this command to install whylogs[spark]:
!pip install "whylogs[spark]" --pre

FYI you can't run Pyspark in Collab. They don't have Java


Add Spanish locale

🔗 Linked issue

No issue linked.

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[X] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Adds Spanish as a new Locale language.
📝 Checklist

[ ] I have linked an issue or discussion.
[X] I have updated the documentation accordingly.

The component https://github.com/nuxt/ui/pull/2577 is on the way, and it will also require a translation.
inputNumber: {
  increment: 'Increment',
  decrement: 'Decrement'
},

Please comment on this PR with how you would translate these two words. If this PR is merged sooner, I’ll add the translation myself.

Thanks @eduayme! 😊


Add type stubs for JPype internal customizers

This is a first version of type stubs for the customizers that ship with JPype.
Ideally the classes should be re-organized so there is exactly one customizer python module per Java Package, but this would most certainly interfere with #828, so I didn't do it in this branch.

@marsher I have no objections to stubs, but I don't understand how to properly test this particular PR.   Is this safe to release with 1.2 or do we need additional time to discuss?

I also do not have a clue, how to test these other than "visual inspection".

@michi42 - any reason to not generate the stubs for _jpype also? (it is no more private than other things here)
In general, is there a reason not to do inline type annotations where feasible (i.e. not in compiled extensions) now that JPype is Py3.5+?

@michi42 - any reason to not generate the stubs for _jpype also? (it is no more private than other things here)
In general, is there a reason not to do inline type annotations where feasible (i.e. not in compiled extensions) now that JPype is Py3.5+?

This PR was only to introduce stubs for the JPype customizers which will be used in the Java stubs generated by stubgenj. Of course it would be nice to have stubs (or inline type annotations) for the JPype python API as well, but I considered it out of scope of this PR.
The main reason why I did stub files instead of inline annotations is https://github.com/jpype-project/jpype/issues/714#issuecomment-668265961 (and following) - in the future, JPype will support loading customizers from JAR files, and the internal customizers may be moved to the JAR package shipped with JPype as well.
In this case, IDEs would not have access to the python source or stub files, so stubgenj will have to copy them into the generated stub directory.
For pyi files, this is straightforward - just copy the file next to the "init.pyi" of the concerned Java package generated by stubgenj. For inline annotations, it's a bit more complicated. If there is no pyi file, stubgenj tries to call mypy stubgen (if available) to generate a stub, which will also extract typing information from inline annotations, if any (see https://gist.github.com/michi42/2110a8615a0e917a13ec6748c6168735#file-stubgenj-py-L126). But that process has a lot that can go wrong or not optimal, so I would consider it only a fall-back of last resort.

This PR was only to introduce stubs for the JPype customizers which will be used in the Java stubs generated by stubgenj. Of course it would be nice to have stubs (or inline type annotations) for the JPype python API as well, but I considered it out of scope of this PR.
The main reason why I did stub files instead of inline annotations is https://github.com/jpype-project/jpype/issues/714#issuecomment-668265961 (and following) - in the future, JPype will support loading customizers from JAR files, and the internal customizers may be moved to the JAR package shipped with JPype as well.
In this case, IDEs would not have access to the python source or stub files, so stubgenj will have to copy them into the generated stub directory.
For pyi files, this is straightforward - just copy the file next to the "init.pyi" of the concerned Java package generated by stubgenj. For inline annotations, it's a bit more complicated. If there is no pyi file, stubgenj tries to call mypy stubgen (if available) to generate a stub, which will also extract typing information from inline annotations, if any (see https://gist.github.com/michi42/2110a8615a0e917a13ec6748c6168735#file-stubgenj-py-L126). But that process has a lot that can go wrong or not optimal, so I would consider it only a fall-back of last resort.

I have no objections to stubs, but I don't understand how to properly test this particular PR.

I also do not have a clue, how to test these other than "visual inspection".

I just answered a question on SO on how to test the stubs. As you might expect, it is pretty noisy with these stubs, but could be a way forwards in the future.
For example, just running this on one module:
$ python -m mypy.stubtest jpype._jcollection
error: failed mypy build.
jpype/_jcollection.pyi:12: error: Class jpype._jcollection._JCollection has abstract attributes "__iter__"
jpype/_jcollection.pyi:12: note: If it is meant to be abstract, add 'abc.ABCMeta' as an explicit metaclass
jpype/_jcollection.pyi:17: error: Argument 1 of "__contains__" is incompatible with supertype "Container"; supertype defines the argument type as "object"
jpype/_jcollection.pyi:17: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:17: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:21: error: Signature of "__getitem__" incompatible with supertype "list"
jpype/_jcollection.pyi:21: error: Signature of "__getitem__" incompatible with supertype "MutableSequence"
jpype/_jcollection.pyi:21: error: Signature of "__getitem__" incompatible with supertype "Sequence"
jpype/_jcollection.pyi:27: error: Signature of "__setitem__" incompatible with supertype "list"
jpype/_jcollection.pyi:27: error: Signature of "__setitem__" incompatible with supertype "MutableSequence"
jpype/_jcollection.pyi:29: error: Argument 1 of "__delitem__" is incompatible with supertype "list"; supertype defines the argument type as "Union[int, slice]"
jpype/_jcollection.pyi:29: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:29: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:29: error: Return type "E" of "__delitem__" incompatible with return type "None" in supertype "list"
jpype/_jcollection.pyi:29: error: Signature of "__delitem__" incompatible with supertype "MutableSequence"
jpype/_jcollection.pyi:33: error: Signature of "index" incompatible with supertype "list"
jpype/_jcollection.pyi:33: error: Signature of "index" incompatible with supertype "Sequence"
jpype/_jcollection.pyi:37: error: Return type "_JList[E]" of "insert" incompatible with return type "None" in supertype "list"
jpype/_jcollection.pyi:37: error: Return type "_JList[E]" of "insert" incompatible with return type "None" in supertype "MutableSequence"
jpype/_jcollection.pyi:39: error: Return type "_JList[E]" of "append" incompatible with return type "None" in supertype "list"
jpype/_jcollection.pyi:39: error: Return type "_JList[E]" of "append" incompatible with return type "None" in supertype "MutableSequence"
jpype/_jcollection.pyi:47: error: Argument 1 of "__iadd__" is incompatible with supertype "list"; supertype defines the argument type as "Iterable[E]"
jpype/_jcollection.pyi:47: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:47: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:47: error: Argument 1 of "__iadd__" is incompatible with supertype "MutableSequence"; supertype defines the argument type as "Iterable[E]"
jpype/_jcollection.pyi:65: error: Generic tuple types not supported
jpype/_jcollection.pyi:69: error: Argument 1 of "__contains__" is incompatible with supertype "Mapping"; supertype defines the argument type as "object"
jpype/_jcollection.pyi:69: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:69: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:69: error: Argument 1 of "__contains__" is incompatible with supertype "Container"; supertype defines the argument type as "object"
jpype/_jcollection.pyi:79: error: Signature of "__getitem__" incompatible with supertype "tuple"
jpype/_jcollection.pyi:79: error: Signature of "__getitem__" incompatible with supertype "Sequence"

With the following tweaks:
diff --git a/jpype/_jcollection.pyi b/jpype/_jcollection.pyi
index 7e811332..90409608 100644
--- a/jpype/_jcollection.pyi
+++ b/jpype/_jcollection.pyi
@@ -14,29 +14,29 @@ class _JCollection(Collection[E]):

     def __delitem__(self, i: E) -> None: ...

-    def __contains__(self, i: E) -> bool: ...
+    def __contains__(self, i: E) -> bool: ...  # type: ignore[override]

 class _JList(List[E]):
-    @overload
+    @overload  # type: ignore[override]
     def __getitem__(self, ndx: int) -> E: ...

-    @overload
+    @overload  # type: ignore[override]
     def __getitem__(self, ndx: slice) -> E: ...

-    def __setitem__(self, ndx: int, v: E) -> None: ...
+    def __setitem__(self, ndx: int, v: E) -> None: ...  # type: ignore[override]

-    def __delitem__(self, ndx: int) -> E: ...
+    def __delitem__(self, ndx: int) -> E: ...  # type: ignore[override]

     def __reversed__(self) -> Generator[E, None, None]: ...

-    def index(self, obj: E) -> int: ...
+    def index(self, obj: E) -> int: ...  # type: ignore[override]

     def count(self, obj: E) -> int: ...

-    def insert(self, idx: int, obj: E) -> '_JList'[E]: ...
+    def insert(self, idx: int, obj: E) -> '_JList'[E]: ...  # type: ignore[override]

-    def append(self, obj: E) -> '_JList'[E]: ...
+    def append(self, obj: E) -> '_JList'[E]: ...  # type: ignore[override]

     def reverse(self) -> None: ...

@@ -44,9 +44,9 @@ class _JList(List[E]):

     def pop(self, idx: int = ...) -> E: ...

-    def __iadd__(self, obj: List[E]) -> '_JList'[E]: ...
+    def __iadd__(self, obj: List[E]) -> '_JList'[E]: ...  # type: ignore[override]

-    def __add__(self, obj: List[E]) -> '_JList'[E]: ...
+    def __add__(self, obj: List[E]) -> '_JList'[E]: ...  # type: ignore[override]

     def remove(self, obj: E) -> None: ...

@@ -62,11 +62,11 @@ class _JMap(Mapping[K, V]):

     def __setitem__(self, ndx: K, v: V) -> None: ...

-    def items(self) -> Set['_JMapEntry'[K, V]]: ...
+    def items(self) -> Set['_JMapEntry']: ...

     def keys(self) -> Set[K]: ...

-    def __contains__(self, item: V) -> bool: ...
+    def __contains__(self, item: V) -> bool: ...  # type: ignore[override]

 class _JSet(Set[E]):
@@ -76,7 +76,7 @@ class _JSet(Set[E]):
 class _JMapEntry(Tuple[K, V]):
     def __len__(self) -> int: ...

-    def __getitem__(self, x: int) -> Union[K, V]: ...
+    def __getitem__(self, x: int) -> Union[K, V]: ...  # type: ignore[override]

 class _JIterator(Iterator[E]):

I can drop the errors to a single thing for jpype._jcollection:
$ python -m mypy.stubtest jpype._jcollection
error: failed mypy build.
jpype/_jcollection.pyi:12: error: Class jpype._jcollection._JCollection has abstract attributes "__iter__"
jpype/_jcollection.pyi:12: note: If it is meant to be abstract, add 'abc.ABCMeta' as an explicit metaclass

I don't know how to fix that one though - it isn't immediately clear where the issue is coming from, and some JPype digging would be needed to figure it out (I suspect one of you will look at it and instantly know the reason).
Proposal: In order to avoid these stubs becoming obsolete/incorrect, we introduce CI to run the stubtest command on them to confirm that they represent reality.

I have no objections to stubs, but I don't understand how to properly test this particular PR.

I also do not have a clue, how to test these other than "visual inspection".

I just answered a question on SO on how to test the stubs. As you might expect, it is pretty noisy with these stubs, but could be a way forwards in the future.
For example, just running this on one module:
$ python -m mypy.stubtest jpype._jcollection
error: failed mypy build.
jpype/_jcollection.pyi:12: error: Class jpype._jcollection._JCollection has abstract attributes "__iter__"
jpype/_jcollection.pyi:12: note: If it is meant to be abstract, add 'abc.ABCMeta' as an explicit metaclass
jpype/_jcollection.pyi:17: error: Argument 1 of "__contains__" is incompatible with supertype "Container"; supertype defines the argument type as "object"
jpype/_jcollection.pyi:17: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:17: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:21: error: Signature of "__getitem__" incompatible with supertype "list"
jpype/_jcollection.pyi:21: error: Signature of "__getitem__" incompatible with supertype "MutableSequence"
jpype/_jcollection.pyi:21: error: Signature of "__getitem__" incompatible with supertype "Sequence"
jpype/_jcollection.pyi:27: error: Signature of "__setitem__" incompatible with supertype "list"
jpype/_jcollection.pyi:27: error: Signature of "__setitem__" incompatible with supertype "MutableSequence"
jpype/_jcollection.pyi:29: error: Argument 1 of "__delitem__" is incompatible with supertype "list"; supertype defines the argument type as "Union[int, slice]"
jpype/_jcollection.pyi:29: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:29: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:29: error: Return type "E" of "__delitem__" incompatible with return type "None" in supertype "list"
jpype/_jcollection.pyi:29: error: Signature of "__delitem__" incompatible with supertype "MutableSequence"
jpype/_jcollection.pyi:33: error: Signature of "index" incompatible with supertype "list"
jpype/_jcollection.pyi:33: error: Signature of "index" incompatible with supertype "Sequence"
jpype/_jcollection.pyi:37: error: Return type "_JList[E]" of "insert" incompatible with return type "None" in supertype "list"
jpype/_jcollection.pyi:37: error: Return type "_JList[E]" of "insert" incompatible with return type "None" in supertype "MutableSequence"
jpype/_jcollection.pyi:39: error: Return type "_JList[E]" of "append" incompatible with return type "None" in supertype "list"
jpype/_jcollection.pyi:39: error: Return type "_JList[E]" of "append" incompatible with return type "None" in supertype "MutableSequence"
jpype/_jcollection.pyi:47: error: Argument 1 of "__iadd__" is incompatible with supertype "list"; supertype defines the argument type as "Iterable[E]"
jpype/_jcollection.pyi:47: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:47: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:47: error: Argument 1 of "__iadd__" is incompatible with supertype "MutableSequence"; supertype defines the argument type as "Iterable[E]"
jpype/_jcollection.pyi:65: error: Generic tuple types not supported
jpype/_jcollection.pyi:69: error: Argument 1 of "__contains__" is incompatible with supertype "Mapping"; supertype defines the argument type as "object"
jpype/_jcollection.pyi:69: note: This violates the Liskov substitution principle
jpype/_jcollection.pyi:69: note: See https://mypy.readthedocs.io/en/stable/common_issues.html#incompatible-overrides
jpype/_jcollection.pyi:69: error: Argument 1 of "__contains__" is incompatible with supertype "Container"; supertype defines the argument type as "object"
jpype/_jcollection.pyi:79: error: Signature of "__getitem__" incompatible with supertype "tuple"
jpype/_jcollection.pyi:79: error: Signature of "__getitem__" incompatible with supertype "Sequence"

With the following tweaks:
diff --git a/jpype/_jcollection.pyi b/jpype/_jcollection.pyi
index 7e811332..90409608 100644
--- a/jpype/_jcollection.pyi
+++ b/jpype/_jcollection.pyi
@@ -14,29 +14,29 @@ class _JCollection(Collection[E]):

     def __delitem__(self, i: E) -> None: ...

-    def __contains__(self, i: E) -> bool: ...
+    def __contains__(self, i: E) -> bool: ...  # type: ignore[override]

 class _JList(List[E]):
-    @overload
+    @overload  # type: ignore[override]
     def __getitem__(self, ndx: int) -> E: ...

-    @overload
+    @overload  # type: ignore[override]
     def __getitem__(self, ndx: slice) -> E: ...

-    def __setitem__(self, ndx: int, v: E) -> None: ...
+    def __setitem__(self, ndx: int, v: E) -> None: ...  # type: ignore[override]

-    def __delitem__(self, ndx: int) -> E: ...
+    def __delitem__(self, ndx: int) -> E: ...  # type: ignore[override]

     def __reversed__(self) -> Generator[E, None, None]: ...

-    def index(self, obj: E) -> int: ...
+    def index(self, obj: E) -> int: ...  # type: ignore[override]

     def count(self, obj: E) -> int: ...

-    def insert(self, idx: int, obj: E) -> '_JList'[E]: ...
+    def insert(self, idx: int, obj: E) -> '_JList'[E]: ...  # type: ignore[override]

-    def append(self, obj: E) -> '_JList'[E]: ...
+    def append(self, obj: E) -> '_JList'[E]: ...  # type: ignore[override]

     def reverse(self) -> None: ...

@@ -44,9 +44,9 @@ class _JList(List[E]):

     def pop(self, idx: int = ...) -> E: ...

-    def __iadd__(self, obj: List[E]) -> '_JList'[E]: ...
+    def __iadd__(self, obj: List[E]) -> '_JList'[E]: ...  # type: ignore[override]

-    def __add__(self, obj: List[E]) -> '_JList'[E]: ...
+    def __add__(self, obj: List[E]) -> '_JList'[E]: ...  # type: ignore[override]

     def remove(self, obj: E) -> None: ...

@@ -62,11 +62,11 @@ class _JMap(Mapping[K, V]):

     def __setitem__(self, ndx: K, v: V) -> None: ...

-    def items(self) -> Set['_JMapEntry'[K, V]]: ...
+    def items(self) -> Set['_JMapEntry']: ...

     def keys(self) -> Set[K]: ...

-    def __contains__(self, item: V) -> bool: ...
+    def __contains__(self, item: V) -> bool: ...  # type: ignore[override]

 class _JSet(Set[E]):
@@ -76,7 +76,7 @@ class _JSet(Set[E]):
 class _JMapEntry(Tuple[K, V]):
     def __len__(self) -> int: ...

-    def __getitem__(self, x: int) -> Union[K, V]: ...
+    def __getitem__(self, x: int) -> Union[K, V]: ...  # type: ignore[override]

 class _JIterator(Iterator[E]):

I can drop the errors to a single thing for jpype._jcollection:
$ python -m mypy.stubtest jpype._jcollection
error: failed mypy build.
jpype/_jcollection.pyi:12: error: Class jpype._jcollection._JCollection has abstract attributes "__iter__"
jpype/_jcollection.pyi:12: note: If it is meant to be abstract, add 'abc.ABCMeta' as an explicit metaclass

I don't know how to fix that one though - it isn't immediately clear where the issue is coming from, and some JPype digging would be needed to figure it out (I suspect one of you will look at it and instantly know the reason).
Proposal: In order to avoid these stubs becoming obsolete/incorrect, we introduce CI to run the stubtest command on them to confirm that they represent reality.


Fix /etc/default/mscs loading

/etc/default/mscs was sourced 'after' default variables were used
making overrides not to work. /etc/default/mscs is now sourced twice,
after LOCATION definition and after other variables definition so all
variables can be overrided. A better fix is let for later.

This change was merged with commit e0ee185be9d2d1c8f143ea734e326b64d463ff53.  Removing duplicate pull request.