Add 'overlay' as an accepted value in the overflow options

Hi devs,
I noticed that you get a prop-types error in the console when you try to set the overflowY property of the table container to 'overlay'.

Sandbox
This error comes from overlay not being included in the array of the type definition of the options:
https://github.com/material-table-core/core/blob/4fd6f459b17be7efed77f2e3686307cbcac82a5d/src/prop-types.js#L342
While overlay is a deprecated value, it is still valid CSS. It is supported and also behaves differently in Chromium browsers, so I think that it would be valid to include it as an option in the prop-types file. What do you think?
I could open a PR with this change if you are ok with it.
Thanks for the great job in the library!

Hi, those prop types are really old and most likely need updating. So yes please open an PR, thanks !!

I just opened a PR for this, thank you!


Query: Add more tests for Queryable.Reverse

Add tests for

Reverse inside a subquery, i.e. the main query is being pushed down
Reverse is used in either side of join/leftjoin/selectmany
Reverse is used in a subquery in projection

Single result
Collection projection

Reverse is used in scalar subquery

OrderBy -> OrderBy -> reverse
OrderBy -> ThenBy -> reverse

I'll do it if no one minds.

@smitpatel I am new to CosmosDB and have some issues so I am asking for a bit of advice.
ss.Set<Employee>()
    .OrderBy(e => e.City)
    .ThenBy(e => e.EmployeeID)
    .Reverse()
    .Select(e => $"{e.EmployeeID}")

this query fails with {"code":400,"body":{"code":"BadRequest","message":"Message: {\"Errors\":[\"The order by query does not have a corresponding composite index that it can be served from.\"]}.
I believe it has to work on CosmosDB too; maybe I have to add a composite index, or choose another pair of columns? Or should I disable that particular test on Cosmos?

Disable test on cosmos for now. Issue number in skip message would be #17246


Make arches list space separated for easier looping/awking

Do you have some sample output to prove it still works? :slightly_smiling_face:

(I cancelled Travis so we don't waste their resources on building this)

$ # on branch "join"
$ ./generate-stackbrew-library.sh > buildpack-new
$ git checkout master
$ ./generate-stackbrew-library.sh > buildpack-old
$ diff buildpack-old buildpack-new
1c1
< # this file is generated via https://github.com/docker-library/buildpack-deps/blob/f243e010fec2494fa6f119298da2c37ef4031baf/generate-stackbrew-library.sh
---
> # this file is generated via https://github.com/docker-library/buildpack-deps/blob/ac51851510ce191b9808a2bb6780fff23aef5dd3/generate-stackbrew-library.sh

See also https://github.com/docker-library/buildpack-deps/pull/59.


Add super-linter?

Would you be open to adding https://github.com/super-linter/super-linter to this project?
Here is the output from running it on tip of this repo: https://github.com/smallstepman/cargo-dist/actions/runs/5763809454/job/15626410522#step:4:57

Wow that's a lot of linters. Seems like we'd need to spend quite a while tuning them to work right.

Whatever process it's using for picking out files isn't properly ignoring the .j2 files (so e.g. the shellchecker is running on installer.sh.j2 when it shouldn't be, we run shellcheck in ci post-expansion of the template, so i'm pretty confident it works).

textlint is the kind of pedantic I don't like in CI (spellcheck on technical documentation is Hell)

markdownlint has some good complains but needs some callibration (the long lines are intentional, idk about the various random style guides)

no idea what jscpd is or what it's trying to tell me, it seems like gibberish

seems to be recommending unconventional quoting of run in github ci ymls (minor, could take or leave it)

I'm not opposed to this but this is a lot to take in.

yea I agree there is a lot of stuff there :D

no idea what jscpd is or what it's trying to tell me, it seems like gibberish

jscpd tells you there is duplicated code, from their readme:

Copy/paste detector for programming source code, supports 150+ formats. Copy/paste is a common technical debt on a lot of projects. The jscpd gives the ability to find duplicated blocks implemented on more than 150 programming languages and digital formats of documents. The jscpd tool implements Rabin-Karp algorithm for searching duplications.

seems to be recommending unconventional quoting of run in github ci ymls (minor, could take or leave it)

The error is hard to read and doesn't point to the right place, but there are two different reasons:

it wants to see this cargo publish -p cargo-dist-schema --token ${CRATES_TOKEN} changed into this  cargo publish -p cargo-dist-schema --token "${CRATES_TOKEN}"
.github/workflows/release.yml:119:9: shellcheck reported issue in this script: SC2046:warning:10:42: Quote this to prevent word splitting [shellcheck] it want to change this           gh release upload ${{ github.ref_name }} $(cat uploads.txt) to this           gh release upload ${{ github.ref_name }} "$(cat uploads.txt)" (I don't have windows so I cannot verify if it still works there)

Whatever process it's using for picking out files isn't properly ignoring the .j2 files (so e.g. the shellchecker is running on installer.sh.j2 when it shouldn't be, we run shellcheck in ci post-expansion of the template, so i'm pretty confident it works).

textlint is the kind of pedantic I don't like in CI (spellcheck on technical documentation is Hell)

I excluded installer.sh.j2 from listing, disabled textlint and let the CI run again: https://github.com/smallstepman/cargo-dist/actions/runs/5765577745/job/15631804140

markdownlint has some good complains but needs some callibration (the long lines are intentional, idk about the various random style guides)

This will require more effort, but I can fix everything listed there (and leave long lines untouched and ignored by linter) - let me know

Whoops, sorry I totally let this slip -- could you open a PR with what you have so far so it's actively in the work queue?

sure! I won't have much time this week to finish this but I'll try to pick it up next week


Red Patch

Hello there! I've been working on your program over the last 4 weeks or so, I'm learning to code and I wanted to improve the program as I use Blender to edit videos often, now I'm ready to share it 😄 . Managed to add quite a bit:

[get_project_info.py] now reads project's info from blend file's ACTIVE SCENE, and reads total number of scenes

[get_project_info.py] now saves a copy of the json file in the scripts folder, this is necessary to avoid exceptions involving DoReadBlenderData() and invalid output paths

fixed a lot of unhanded exceptions, adding error messages

Render and join buttons are disabled if errors are detected

Complete UI overhaul, tooltips, new info and an icon cause why not?

CMD arguments! Running the following in a CMD window will automatically set to specified file:
BlenderRenderController.exe “filepath to .blend”

Thank you very much for your effort! :)
Right now I'm pretty busy, but I will look into this and merge as soon as possible!

I've just compiled it, looks really good! :)
I will hopefully be able to make a project that I could try it on tomorrow.


Can't click on test output in editor to open peek

Does this issue occur when all extensions are disabled?: Yes (except for test adapter being debugged)

VS Code Version:
Version: 1.65.0-insider
Commit: 0283cec6ec2c2d5ec39006a1048b8dc3ea8bc955
Date: 2022-02-08T05:16:19.099Z
Electron: 13.5.2
Chromium: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 5.15.19-1-MANJARO snap

Steps to Reproduce:

Run a test appending output at location in editor
Try clicking on output
Nothing happens

Alternative:

Hover over test output
Manage to get mouse to "Peek Test Output" and click it
Nothing happens

Above image is an example where a test is producing both output and testmessage from fail. The output at line 67 shows a cursor on hover but is not clickable, but the test message (red line) at line 74 is clickable.

Above image: Upon opening the test peek window by clicking on the error its possible to navigate to the ouput from there.
This example has both output and test message, but the problem is the same for a test with only output.

/assign @connor4312

I noticed an additional hover was displayed over the "Peek Test Output" link, showing the vscode command being run or something like that. Maybe it is helpful to solve this issue:

The actual test id is skyturns:controllers:playing:c-playing-buy-mushi-world:mushiflowers discount:should continue to playing on buy:


Proxying https traffic sets "Transfer-Encoding: chunked"?

First reported by @franziskagoltz.
Am not positive, but it looks like the conduit proxy is adding a "Transfer-Encoding: chunked" header when proxying my https request. Is that possible?
To test, I've been injecting the proxy into a linkerd config that forwards traffic to https://httpbin.org, available here:
proxy.txt (actually .yml but file extension not supported)
If I apply that config in minikube and curl the 'proxy-https' service, it successfully returns a 200:
$ kubectl apply -f proxy.txt
$ curl -sIH 'Host: httpbin.org' $(minikube service proxy-https --url)/get | head -n1
HTTP/1.1 200 OK

Whereas if I inject that config with conduit and try curling, it fails with a 501:
$ curl -sIH 'Host: httpbin.org' $(minikube service proxy-https --url)/get | head -n1
HTTP/1.1 501 Not Implemented

The 501 response body is:
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>501 Not Implemented</title>

Not Implemented

Chunked requests are not supported for server

Fwiw, I've tested this with conduit master before and after #397 merged, and it appears to behave the same.

I don't believe TLS is related, but rather that it's a GET request, and the way body is constructed and the given to hyper means it isn't able to detect that the body is empty.
There is a fix in hyper master to just assume no body on GET if no content length is set...

Is this a regression from 0.2, or was this the case for 0.2 too?

It's not a regression. Just likely no test case to see it.

There is a fix in hyper master to just assume no body on GET if no content length is set...

The RFC says: "The presence of a message body in a request is signaled by a Content-Length or Transfer-Encoding header field." So it isn't just GET and it isn't just Content-Length.

I realize that. It's a combination of how it's used in the proxy. By default, in hyper, a request doesn't have a body unless set. In the proxy, since it's translating from potentially an h2 stream, a body is added to the request. hyper sees the set body and assumes a transfer encoding if content length wasn't set.
It's a deficiency in the current API, where it's not possible to know if a set body is actually empty until after the headers are sent.
The body trait will gain a method to check for it being empty in the next breaking release. In the mean time, it ignores chunked bodies for GET, since that's probably not what you meant.

Thanks for the explanation -- have updated the title accordingly.


Fixes for multicluster builds

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

Test name
Commit
Details
Rerun command

unit-tests_istio
78f37e940e59580794b31a0bbbda33e944bb935b
link
/test unit-tests_istio

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

/test unit-tests_istio


brew list --installed-on-request -tr Can't sort by time

brew doctor output
Your system is ready to brew.

Verification

[X] My "brew doctor output" above says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update twice and am still able to reproduce my issue.
[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

brew config output
HOMEBREW_VERSION: 4.3.24
ORIGIN: https://github.com/Homebrew/brew
HEAD:<PHONE_NUMBER>62c32fc2365e8e9ff0b1507a98925e
Last commit: 4 days ago
Core tap JSON: 26 Sep 09:05 UTC
Core cask tap JSON: 26 Sep 11:26 UTC
HOMEBREW_PREFIX: /opt/homebrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 10
Homebrew Ruby: 3.3.4 => /opt/homebrew/Library/Homebrew/vendor/portable-ruby/3.3.4_1/bin/ruby
CPU: 10-core 64-bit arm_firestorm_icestorm
Clang: 16.0.0 build 1600
Git: 2.39.5 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 8.7.1 => /usr/bin/curl
macOS: 15.0-arm64
CLT: <IP_ADDRESS>.1.1724870825
Xcode: N/A
Rosetta 2: false

What were you trying to do (and why)?
run: brew list --installed-on-request -tr
What happened (include all command output)?
It's not chronological, it's still alphabetical

What did you expect to happen?
Sort by time
Step-by-step reproduction instructions (by running brew commands)
`brew list --installed-on-request -tr`

Yes, the short flags -l, -r, and -t only properly with each other and with --formulae or --casks (and no named args).
It would be nice to fix this. If not, then it should at least be documented.

It would be nice to fix this. If not, then it should at least be documented.

The fix here would be marking these options as conflicting.


[WIP KBFS-2244] jenkins: Move change checking to the Jenkinsfile

… script.

I only need one review, but PTAL.
https://ci.keyba.se/job/client/job/PR-7245/9/ ran quickly, then when I removed go/test/jenkins_test.sh, it runs slowly. Seems to work!

If my build passes, I'm going to merge without a review so CI speeds up for tomorrow; I'd appreciate a retroactive review anyway though.

? Doesn't this add a label in the pipeline (not blue ocean)
On Thu, Jun 1, 2017 at 11:05 PM John Zila<EMAIL_ADDRESS>wrote:

@jzila commented on this pull request.
In Jenkinsfile
https://github.com/keybase/client/pull/7245#discussion_r119773104:

                                 }},

                                 test_linux_js: { withEnv([
                                     "PATH=${env.HOME}/.node/bin:${env.PATH}",
                                     "NODE_PATH=${env.HOME}/.node/lib/node_modules:${env.NODE_PATH}",
                                 ]) {
                                     dir("shared") {

                                       stage("JS tests") {

It doesn't actually do anything inside the parallel node...
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/keybase/client/pull/7245#discussion_r119773104, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfhU4T0WKwoeP_DD6_vXhDw2NEebkvbks5r_3wHgaJpZM4Ntv8E
.

Oh neat, you are correct. All right I'll roll back that particular change.


Lack of documentation in examples

The examples are great, but there is a lack of documentation and source code.
It would be really cool to know, for example, the json structure used for the example "With json data".
Or how to implement resizable columns with example source code.

Yes I agree. Just some simple code to start us off. Please

I agree that there should be code connected to the examples (preferably very concise code).
Though for those willing to dig a little, all of these examples seem to be available in the /site/examples folder of this project.
For example, the JSON / Object structure is available here:
https://github.com/facebook/fixed-data-table/blob/master/site/examples/FakeObjectDataListStore.js

Yes just found them , thanks for the examples. A link from the website could have avoided the confusion tho

The link above is dead. It would be trivial to include the format of JSON used in the example, why not just make people's lives easier and show the format, in comments or something?

I believe you can find the same example here:
https://github.com/facebook/fixed-data-table/blob/master/examples/helpers/FakeObjectDataListStore.js


Make a call to graph service for user creation on invite request.

Description
Make a simple call to the graph service user creation on invitation request.
Motivation and Context
This is for the POC of inviting guest users.
How Has This Been Tested?
Tested with curl, but no unit tests since being a POC.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

Checklist:

[x] Code changes
[ ] Unit tests added
[ ] Acceptance tests added
[ ] Documentation ticket raised:

@Excds I'm happy to just merge this into the invitation branch ASAP. we should just continue in a single branch. Lets discuss how to move forward tomorrow. Maybe the invitation branch should just be made green, we merge it and then change the base of this branch to master?


Customer orders list order date in wrong locale

The javascript doesn't appear to put the order date in the correct locale.
From utils.js:
var timestamp = new Date(attrs.shopTimestamp);
element.html($filter('date')(timestamp, attrs.timeFormat));

Result in en locale:

please check, if your have installed the correct angular-locale_XX.js.
The problem is, that DRF returns the timestamp as isotimestamp which has to be converted on the client. This on the other side allows us to use libraries such as angular-timeago or ngtimeago.

Thanks for the tip! I fixed it with the following in the base template:
{% addtoblock "js" %}
  <script src="{% static 'node_modules/angular-i18n/angular-locale_'|add:LANGUAGE_CODE|add:'.js' %}"></script>
{% endaddtoblock %}

I will try to fix this with this https://github.com/jrief/django-angular/issues/305 issue.

I guess the advantage of the templatetag is it can check the existence of a file for each configured language 👍


Control curve degrees hard-coded to 1

I noticed the mgear.core.icon code creates the correct shapes.
However, Shifter seems to always create them with 1 degree.
The flower shape looks like a ninja star :smile:.
If there is a reason for this, should this be an option?

Could this be a configurable thing on the guide?
Let's say it defaults to 1 for performance reasons and an 'auto' option to get nice looking curves?

yes it can be exposed as an option


Event page

I'm not sure about event__cta price banner taking all width on tablet and especially on a desktop, but of course that's more a design question otherwise :shipit: :+1:


[Gally]: master <- dev

Automatically created by Git-Ally

:tada: This PR is included in version 2.11.24 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


test in style_transfer

In styleTransfer,when i use other BVHs,why cant run the style_transfer/demo.sh successfully?
is my skeleton in my BVH  is lack?  the skeleton must match it that you give?

Our style transfer framework deals with a fixed skeleton. To test our pretrained model, you need to first retarget your BVH to the skeleton described in style_transfer/global_info/rest.bvh. We'll provide more information on how to train the model with your own customized skeleton in the near future. Hope this would help.

in style transfer ，when i want to change content BVH, how can i do? you say the skeleton should same as   style_transfer/global_info/rest.bvh, how can we get it  from our own avis?

Extracting 3D motions from 2D videos (3D motion reconstruction/pose estimation) is a fundamental research problem in computer vision. And it's beyond the scope of our style transfer framework. We only support direct usage of 2D videos as the style input.

thanks a lot !


Added vulnerability file to represent lack of SELinux prior to 4.4

At the moment this "vulnerability" is showing on the lists of vulnerabilities for each version. Do we want to keep it like that (in that it is a way of elevating privileges) or not (given that it was a design feature and was known about without being regarded as a vulnerability)?

I don't think we want to show it on the lists and we only want to display it on the graph if the "root" node already exists. Regrettably that is likely to be horrible to implement so it might be better to leave it as is at the moment and implement such things later if sufficiently useful?


MSL: Track location component to match vecsize between shader stages.

Matching output/input struct member types between shader stages could fail if
a location is shared between members, each using different components of that
location, because the member vecsize was only stored once for the location.
Add MSLShaderInput::component member.
Use LocationComponentPair to key inputs_by_location, instead of just location.
ensure_correct_input_type() pass component value as well as location.
Any tests would require ability to pass location and component info. Not sure how to do that.
@cdavis5e Inviting your review here regarding impact on tessellation stages.

CI runs should be fixed if you rebase against master. Github Actions nuked Ubuntu 16.04 support.


Rate limit requests so as to not exceed API quota

On my first run I received this message about quotas. I verified it is not my google storage quota so it must be an API quota. Is there a way to adjust that in the cloud API manager? If not, is there a way to rate limit the application so it doesn't exceed this quota?
2018/11/24 10:36:26 failed uploading image
 --- at /home/nick/go/src/github.com/nmrshll/gphotos-uploader-cli/upload/fileUpload.go:33 (StartFileUploadWorker.func1) ---
Caused by: failed uploading image
 --- at /home/nick/go/src/github.com/nmrshll/gphotos-uploader-cli/upload/fileUpload.go:53 (FileUpload.upload) ---
Caused by: failed adding media [image name].jpg
 --- at /home/nick/go/src/github.com/nmrshll/google-photos-api-client-go/lib-gphotos/client.go:115 (Client.UploadFile) ---
Caused by: googleapi: Error 429: Quota exceeded for quota metric 'photoslibrary.googleapis.com/write_requests' and limit 'WritesPerMinutePerUser' of service 'photoslibrary.googleapis.com' for consumer 'project_number:367310083084'., rateLimitExceeded

Solved on


Enhance debuggability for tuners

The debuggability of tuners are very poor. Currently, the only way to debug a tuner is repeatedly running it (and possibly burning a lot of resources to run trials).
Also, debugging by launching full experiments is unfriendly because:

NNI messes with my stdout and stderr.
NNI messes with my working directory.
I don't know what NNI is doing to my environment when NNI launches my tuner.

The problem is even more serious when users write an advisor inheriting message dispatcher directly.
I suggest providing a python-native utility to mock the environment of NNI manager to launch the tuner. And user can easily put some pdb lines in it for debugging.

I'm working on a mechanism that you use the tuner process as main script, then inside it you use nnicli/nni.experiment to launch NNI manager. NNI manager will connect to the dispatcher IPC pipe created by nni.experiment, instead of launching dispatcher process.
In this case the tuner is expected to behave like a ordinary Python script. You can even use pdb to launch it.


Better support for monorepos

I believe that for monorepos we can greatly improve the DX by building a matrix of the packages and running each in parallel in on it's own. I also think that we can run lint in the same way. The main benefit of this in actions is that you can see specifically which one failed without having to open the logs.

In a monorepo, things should be run in topological order - only things that aren’t dependent on each other should run parallel. Unfortunately no workspace tools get this right yet.

Technically turborepo does depending on the setup. That said, what I am talking about here is more about a UX affordance than doing it strictly "correctly". I could be convinced, but I think that most situations would be better served by just doing it in parallel in a matrix and letting app devs worry about the interdependency of their tests than have the "managed workflow" try and get too fancy.

I mean, if we're letting app devs worry about it then they can do their own parallelization too :-p

I was hoping they don't need to worry about it? My goal was to add a better baseline for monorepos. When I said "letting them worry about it" was not trying to get fancy with topological order when 90% of test failures will not be because of an in-repo-dep failing in a hard to debug way. Just run it in parallel and that alone with satisfy the mass majority of cases.

And I guess also lastly I should address that publish is the only step that actually benefits from topological ordering, and this does not cover that.

Build steps do as well, as do if one of the monorepo packages is a dev tool that requires building, used in another one. It comes up quite a bit in my experience.

Hm, this is a typescript thing isn't it? I really should probably adopt a TS monorepo and try to actually get it into a state I can stand working in so I catch these things earlier. Anyway, I was thinking that this repo should not have opinions on how folks build, but just run npm run test and have the repo configure how it wants to build. I guess though that does mean npm run -w pkg test would need to know how to build the whole monorepo, so maybe that is too tough to achieve in a non opinionated way.

Is there even a common pattern in the ecosystem for how monorepo CI is supposed to work? Conventions, guidelines, etc? Most of the time I've heard devs say "just us nx" or smth along these lines, but that does not really do anything that's native to actions (i.e. it does not build a matrix - it just executes things in parallel).
And yeah, compiling TS ahead of the matrix and caching is a pre-requisite for any monorepo of decent size, by the looks of it. I don't know how monorepo people afford the Actions minutes 🤷♂
While there's definitely lots of scope for discussions and collecting best practices, my gut is that code wise this might need to start as some experiments.


Compilation error

engine-src/Game/LambdaHack/Client/UI/Frontend/Dom.hs:174:60: error:
    • Couldn't match expected type ‘K.PointUI’ with actual type ‘Point’

Some version bound is missing?

No, just between-releases bitrot (last tag should compile fine). Let me attempt compilation and fix...

Recompilation of deps takes forever, so I've just blindly attempted a hotfix. Please try. :)

Oh, and sadly you need to do
cp LambdaHack.cabal.flattened LambdaHack.cabal

because cabal with ghcjs don't yet handle internal libraries. But then it should compile. I does for me.

This works, thanks!


add integrate test for runtime.

What this PR does:
add integrate test for runtime.

Component List:

redis

Api List (not all features) :

hello api
state api
lock api
sequencer api

Please review and merge the #408 and #410 first


Python function samples are actually JavaScript

See here https://github.com/cdk-patterns/serverless/tree/main/the-saga-stepfunction/python/lambda_fns/
Are there any actual Python samples?

Hi,
All of the Lambda fn code starts out as TS/JS when the patterns launch and the plan is to slow migrate it to Python (and the other langs) through community contribution or as I have capacity. All the CDK code is in the language described but to launch all the languages at the same time with the lambda fns matching cdk language would make it a much larger maintenance overhead at launch and cause potential differences, plus you can use ts/js lambda fns with any lang of cdk. There are however already examples in there like https://github.com/cdk-patterns/serverless/blob/main/the-predictive-lambda/python/the_predictive_lambda/the_predictive_lambda_stack.py that use Python for the lambda fn


Python solution for Next Greater element in DSA 450

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes
Motivation
(Please outline the motivation for the proposal.)
Link to the problem : https://leetcode.com/problems/next-greater-element-i/
Pitch
(Please explain why this feature should be implemented and how it would be used.)
Using stacks

Please assign me this issue

You can always assign yourself using the command '/assign'.


cherry-pick: Disable legacy (v1) registries by default

Cherry-picking change: https://github.com/moby/moby/pull/33629
Deprecation of interacting with v1 registries was started in docker 1.8.3, which added a --disable-legacy-registry flag.
This option was announced to be the default starting with docker 17.06, and v1 registries completely removed in docker 17.12.
This patch updates the default, and disables interaction with v1 registres by default.

/cc @thaJeztah

LGTM
integration tests and unit tests are green


refactor(markInvalid): Refactored markInvalid()

PR related to this trello task --> https://trello.com/c/wHKRgUuc

:tada: This PR is included in version 1.11.0 :tada:
Your semantic-release bot :package::rocket:


[FEATURE] Sales Graph and Statistics

Description
Create a sales graph and statistics section to show stats of sales and usage of the application to the admin or designated staff member.
Screenshots

Additional information
No response

Closed with #35


Update JNI and cudf-py version to 23.02

Signed-off-by: Peixin Li<EMAIL_ADDRESS>fix https://github.com/NVIDIA/spark-rapids/issues/7064

convert to draft first, will trigger build when 23.02 JNI artifact is available

build

filed https://github.com/NVIDIA/spark-rapids/issues/7092
remove databricks in title for premerge run

build

build

build


No data returned

Spreadsheet id "1RXjhi32S2zIfFivybqL95tKybpN66z-JtzbtuZUdOZM", SheetName = "String"
are shared publicly
new PublicGoogleSheetsParser("1RXjhi32S2zIfFivybqL95tKybpN66z-JtzbtuZUdOZM", { sheetName: "Strings" }).parse().then(r => console.inf(r)) shows []
Interestingly, getSpreadsheetDataUsingFetch() returns:
google.visualization.Query.setResponse({"version":"0.6","reqId":"0","status":"ok","sig":"1458216511","table":{"cols":[{"id":"A","label":"","type":"string"},{"id":"B","label":"","type":"string"}],"rows":[{"c":[{"v":"key"},{"v":"value"}]}],"parsedNumHeaders":0}});

Nvm, spreadsheet were empty, works perfectly, thank you!


IGAPP-718: Add map translations

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

Please let me know if you think translations are missing/could be better. Also try to check that I did not break any keys/translations.

I'd do the professional translations in this issue.

Tested on android couldn't find any translation issues so far! Good work!

Translations have been exported and will be translated soonish. Once we get the results I'll merge this with the translations.

Translations for malte only languages will be added in a separate PR as that may take a while. Please have another superficial look at the translations.json if it looks legit. It would be nice if you could translate some strings with deepl/google translate as a sample.


commenter 0.0.1

New Pull Request Checklist
Please mark the points which you did / accept.

[X] I will not move an already shared step version's tag to another commit
[X] I read the Step Development Guideline
[X] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[X] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[X] I read and accept the Abandoned Step policy

@chrisfuller please refer this message: https://github.com/bitrise-io/bitrise-steplib/pull/2113#issuecomment-532139355

Closing as we did not receive a reply - @chrisfuller please let us know if we can help with anything!


Inactivity timeout

I'd like to know more about inactivity timeout, such as:

how long it it supposed to be?
I've noticed that if it timeouts in the middle of creating or amending the an item, the changes aren't saved - is it possible to have a timeout warning?
the orchestra never seems to timeout.

When you are logged in, if you did nothing else in the Mauro UI or Orchestrator (e.g. didn't click any buttons to perform an action), then you would be automatically logged out after 30 minutes.  This does work in the Orchestrator too - I've seen it happen. It is referred to as a "rolling" timeout though, meaning as long as you performed an action in Mauro (including fetching data), that keeps your login session "alive" and active, which will count 30 minutes from your last action.
Any requests to change that or add new features around that, such as your suggested warning, needs to be raised to @jamesrwelch  and the Mauro team, that is outside the scope of System C's work.

@jamesrwelch - as per Peter's message above, can you look at a warning message please?


fix: add missing useEffect dependencies

fixes repeated logging out with expired session token
evaluates the use of disable linting rules and reenables them where neccessary

I have been away from the react hooks for too long to really review this PR effectively. will let @daviroo or @Extheoisah to approve. maybe @samerbuna also would have a better opinion than mine


Support "people tracking" / "person tracking" for non-custom exceptions

I see in the source code that a person value can be passed for custom exceptions:
https://github.com/ForzaElixir/rollbax/blob/master/lib/rollbax.ex#L146
I'm wondering if its possible to configure Rollbax to populate Person/People information for exceptions which are not custom, as described here:
https://docs.rollbar.com/docs/person-tracking
i.e. something similar to the Ruby gem's notion of a person_method:
https://rollbar.com/docs/notifier/rollbar-gem/#person-tracking
Is it possible?

It's possible yes, but I have no ideas on the API and I don't have bandwidth to implement this right now. If you want to contribute this, I would suggest that you come up with a detailed proposal of how the API would look like and then if we want to move forward with that we'll go with the implementation. Wdyt?

I love the idea; I've added it to my backlog, but can't give a projection on when I might have something.  I think we can close this for now.


Make capabilities has_str case insensitive

Related djc/tokio-imap/issues/54

Looks good to me, thanks!


NixOS: Bluetooth not working (seems to be a bluez issue)

Describe the bug
I just recently set up my laptop using nixos-unstable. When I set hardware.bluetooth.enable = true; and run nixos-rebuild, I get the following error.
Apr 27 00:05:37 deckchair systemd[1]: Starting Bluetooth service...
Apr 27 00:05:37 deckchair systemd[161067]: bluetooth.service: Failed to set up mount namespacing: /run/systemd/unit-root/var/lib/bluetooth: No such file or directory
Apr 27 00:05:37 deckchair systemd[161067]: bluetooth.service: Failed at step NAMESPACE spawning /nix/store/cpswxwgcbxfbqydiya4xkcdwx3ndawyr-bluez-5.64/libexec/bluetooth/bluetoothd: No such file or directory
Apr 27 00:05:37 deckchair systemd[1]: bluetooth.service: Main process exited, code=exited, status=226/NAMESPACE
Apr 27 00:05:37 deckchair systemd[1]: bluetooth.service: Failed with result 'exit-code'.
Apr 27 00:05:37 deckchair systemd[1]: Failed to start Bluetooth service.

It looks like when my system built the bluez derivation, bluetoothd is missing.
I did find a similar issue, in the bluez project, but it seems unrelated to this specific issue, although strikingly similar.
Steps To Reproduce
Steps to reproduce the behavior:

Install Nixos
Set hardware.bluetooth.enable = true;
Run nixos-rebuild switch

Expected behavior
Bluetooth service starts and bluetooth devices are usable.
Notify maintainers
@ryantm (I didn't see any maintainers for Bluez in the metadata and your bot was the last to touch bluez)
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[grayson@deckchair:~/nixos-configs]$ nix-shell -p nix-info --run "nix-info -m"
this path will be fetched (0.00 MiB download, 0.00 MiB unpacked):
  /nix/store/lqslhy9ywfxkhipy28sm9qprlmb7dlm8-nix-info
copying path '/nix/store/lqslhy9ywfxkhipy28sm9qprlmb7dlm8-nix-info' from 'https://cache.nixos.org'...
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.34, NixOS, 22.05 (Quokka)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.0`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I have met the same error a few days ago, when reinstall my laptop with nixos-unstable channel. But it was easy to fix, Run sudo install -dm700 /var/lib/bluetooth and restart it works.
Maybe related with systemd. @dasJ

@zendo That workaround worked for me as well, thanks!

Why did I get pinged here? I'm not even part of the systemd team

I run tmpfs on root, and as /var/lib/bluetooth doesn't exist, I got the same problem
For now, @zendo 's workaround seems to work, in this declarative form:
{...}: {
  systemd.tmpfiles.rules = [
    "d /var/lib/bluetooth 700 root root - -"
  ];
  systemd.targets."bluetooth".after = ["systemd-tmpfiles-setup.service"];
}

Sadly systemd.targets."bluetooth".after does not solve the problem if you erase the whole root at boot time and have /var/lib/bluetooth be a symlink to a persisted location instead. Apparently there should be an environment variable for specifying the state directory but I haven't found a good way to set that on NixOS yet.

With https://nixos.wiki/wiki/Impermanence, bind-mount solves my issues:
  environment.persistence."/persistent" = {
    directories = [ "/var/lib/bluetooth" ];
  };

Sadly systemd.targets."bluetooth".after does not solve the problem if you erase the whole root at boot time and have /var/lib/bluetooth be a symlink to a persisted location instead. Apparently there should be an environment variable for specifying the state directory but I haven't found a good way to set that on NixOS yet.

@jansol
I am currently doing an install similar to yours (grahamcs zfs setup). I googled a bit around and added the following as workaround for the moment.
 fileSystems."/var/lib/bluetooth" = {
    device = "/persist/var/lib/bluetooth";
    options = [ "bind" "noauto" "x-systemd.automount" ];
    noCheck = true;
  };

which i found in a similar way on the internet
Maybe its helpful for you.

That does indeed work, thanks!

Heads-up, neither of these solutions worked for me (on unstable @ ac718d02867a84b42522a0ece52d841188208f2c). After doing some digging, I managed to find that the issue lies purely with the security defaults for the service file. @fogti filed https://github.com/systemd/systemd/issues/25097 upstream, which folks should follow.
In the mean time, both of the above solutions will work if you follow @fogti's suggestion and override StateDirectory so that systemd does not try to do anything special:
systemd.services."bluetooth".serviceConfig = {
  StateDirectory=""; # <<< minimal solution, applied in override.conf

  # Seems unnecessary, but useful to keep in mind if bluetooth
  # defaults get locked down even more:
  # ReadWritePaths="/persist/var/lib/bluetooth/";
};

An aside on the differing permissions in the above solutions
Just a small note on permissions because this is usually where I look for problems. Possibly one other person will be interested and everyone else will already know this. Both,
environment.persistence."/persistent" = {
    directories = [ "/var/lib/bluetooth" ];
};

and
systemd.tmpfiles.rules = [
    "d /var/lib/bluetooth 700 root root - -"
];

have the same permissions because StateDirectoryMode=0700 is set in the default bluetooth.service file -- systemd will adjust the state directory to 0700 from the perspective of the script.

I have got the same error a few days ago, when reinstall my laptop in nixos-unstable channel. But it was easy to fix, Run sudo install -dm700 /var/lib/bluetooth and restart then it works. Maybe related with systemd. @dasJ

Hi! Thank you, I am back from future (July 2023) to say that after my first reboot on a fresh install I had the same issue and this fixed to me. Thx.


Compile error re: TimeUtils.swift

Using 0.9.26 as installed from podfile () I ran into a compile error in Xcode 8.2. In the timeOfDayFromSecondsSince1970 func, I had to optionally cast hh to Int32. (Expedient fix just to continue compiling and testing).
    let hh = div(secondsSinceMidnight, secondsPerHour) as? Int32

    return (hh: hh!, mm: mm, ss: ss)

@cburrows:  Could you take a look at this?

Given this was 0.9.26, do we want to worry vs. just let it get fixed when are ready to cut a new release?

@stonetip:  I just tagged the current tip-of-tree as 0.9.28.  Could you try that version to see if you still get this error?  I'm confused why no-one else is seeing this, since I don't think anything there has changed since 0.9.26.

I haven't yet tried 0.9.28, However, I copied the function and constants over from TimeUtils.swift and the mod and div functions from MathUtils.swift to a playground and ran a test against the current epoch time. Of course, no problems there!
Oddly, earlier when tried "Jump to Definition" on div it was opening a Darwin class. I came back to the computer a few hours later and was taken to MathUtils.swift instead. I don't know if this has any bearing on the issue.
Regardless, I have to either modify the hh line to:
let hh = div(secondsSinceMidnight, secondsPerHour) as Int32
or
let hh = Int32(div(secondsSinceMidnight, secondsPerHour))
to compile a project targeting iOS 10.2 in Xcode 8.2.1 (8C1002)

I just merged PR #345 with a fix for this.  Thanks for pointing it out.


formula_desc_cop: tweak some rules.

Allow some specific lowercase words and provide an autocorrect for some of these rules.
As mentioned in https://github.com/Homebrew/brew/pull/2724.
CC @issyl0 as this was partly motivated by your PR.
CC @GauthamGoli for thoughts and review. Additionally, this did made me think we should probably have some of the stuff that can't be autocorrected as different rules otherwise RuboCop will think it's autocorrected them when it has not.

Amazing! It took a while to think of alternatives to "xUnit-based" and I definitely wasn't sure about capitalising "MacOS".

Yeah. We should separate rules with/without autocorrect method. In RuboCop, ideally, a single cop would check just one independent rule, but we are using it to our advantage when there is no autocorrect.

Yeah. We should separate rules with/without autocorrect method.

👍 will do that.

@GauthamGoli Updated, let me know what you think.

Looks cool!

@issyl0 Now this is merged brew style --fix should make your life easier!


setTimeout is undefined even when using ChakraCore js engine

I am using 'react-json-view' to display json.
whenever I try to use react-json-view in any one of my components I get this error -
'setTimeout' is not defined at Anonymous function
Below are my ConfigureService and Configure methods.
 public IServiceProvider ConfigureServices(IServiceCollection services)
        {
            services.AddJsEngineSwitcher(options => options.DefaultEngineName = ChakraCoreJsEngine.EngineName)
            .AddChakraCore();

            services.AddReact();
}

 public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
        {
            app.UseReact(config =>
            {
                config
                .SetLoadBabel(false)
                .AddScriptWithoutTransform("~/build/server.bundle.js")
                .SetReuseJavaScriptEngines(false);
            });
}

@tavisca-prathod that sounds a lot like #555, please try pinning react and react-dom to 16.4.0 until we can ship a fix on our end, or until React itself ships a new version


Enable publishing of native module.

Previously, the coreNative module was not published in the
+publishLocal command. Now, the root project aggregates coreNative
so that the module is correctly published. To make sbt test still
work like before, testing is disabled in Scala Native.

@olafurpg thanks a lot for your help 💯 I think this did the trick. In the end I deleted the root project again.

Awesome! The next step is to try a non snapshot release. They often fail because of random gpg issues

:clap:
Let's cut a 2.4.0-M1 ... @joroKr21 do you want to do the honours?

Sure! I just need to push a tag right? 🎉

Yes. I normally push the tag from the command line and write the release notes while waiting for the release to finish

Cool the release worked 🎉
The release notes will take a while. It's been so long since the last major release.

Very cool! Congrats 🎉 Now with releases requiring less ceremony it may be easier to write release notes in the future 😉
Writing release notes always takes time for large projects. I have found it helpful to use "release drafter" to get a quick summary of PRs that have been merged. You can set it up with GitHub Actions by copying these two files
https://github.com/scalameta/munit/blob/master/.github/workflows/release-drafter.yml
https://github.com/scalameta/munit/blob/master/.github/workflows/release-drafter.yml
It generates release templates like this https://github.com/scalameta/munit/releases/tag/v0.7.1 which you can then enrich with more details.


Include C32 procedures from Results sections

Per https://github.com/blue-button/bluebutton.js/pull/72#issuecomment-26114114

@thetylerhayes Have you guys done this on your fork / is it still useful / can we close it?

We haven't. Probably still useful though. My biggest question is whether or not procedures included in results sections are doubled-up from the procedures section. If they are, we don't want to dual import. I'll try to find a C32 sample file that has these and suss it out.

OK so I took a look at some C32 files. Sure enough some had procedures inside results. Typically it'd be structured something like this:
<entry>
    <organizer>
        <templateId root="2.16.840.1.1138<IP_ADDRESS>.32"/>

        <effectiveTime value="19760101010000"/>
        <component>
            <observation ... >
                <templateId root="2.16.840.1.1138<IP_ADDRESS>.31"/>
            </observation>
        </component>
        <component>
            <observation ... >
                <templateId root="2.16.840.1.1138<IP_ADDRESS>.31"/>
            </observation>
        </component>
        <component>
            <observation ... >
                <templateId root="2.16.840.1.1138<IP_ADDRESS>.31"/>
            </observation>
        </component>
        <component>
            <procedure ... >
                <templateId root="2.16.840.1.1138<IP_ADDRESS>.29"/>

            </procedure>
        </component>
    </organizer>
</entry>

In other words: a results element would parent a bunch of test results (<observation> tags) and also one <procedure>. And the procedure would have the same details as the parenting results element itself. Which makes sense.
Never found a situation where there would be a procedure inside a results element that also was its own procedure in the Procedures section. I'm sure some C32 file somewhere has that situation but I don't think it's a thing to worry about. Pull request incoming.

Decided to leave this alone based on the discussion in #137


Proxyman Linux requires WSL to run bash script to install android emulator cert

Description
When I attempt to install a cert to a running emulator I am presented with a popup that states "WSL is required" to which it says this is to run a bash script. This should not be an issue as I am on Ubuntu as my OS.
Steps to Reproduce
Below will be a screenshot showing my os details, and the WSL error popup.

Open proxyman
Open android emulator with Google API (not play store)
Open the menu Certificate > Install Certificate on Android > Emulators...
Ensure the certificate has been installed to OS (which it had been prior in case installing it prior is what makes this reproducible)
Select Override Emulator from step 2 on the certificate window.

The remaining steps were completed previously and is working on my Windows, and MacOS machines as well as working on a physical device on this very same machine I am having issues with right now.

Current Behavior
Being presented with a message stating WSL is required to override certificate for the android emulator even though I am on a linux machine.
Expected Behavior
I should be able to override the certificate on my Android emulator
Environment

App version: 2.16.0
Windows/Linux version: Xubuntu 24.04.1 LTS x86_64

Thanks, I suppose that the emulator override script couldn't work correctly on Ubuntu, we will fix it 👍


React integration

It would be good to have react integration:
import { state } from 'cygnals'
import { useCygnal } from 'cygnals/react'

const message = state('hello')

export default function() {
  const value = useCygnal(message)
  return <input value={value} onChange={e => message.set(e.target.value)} />
}

not sure about the useCygnal name... good because it can easily be used with other hooks without a name clash... bad because it's a strange word.

done in v0.4.0


Keep multiselect open after making choice

I want to merge this change because it prevents multiselect fields from closing upon making a selection.
Pull Request Checklist

[ ] This code contains UI changes
[ ] All visible strings are translated with proper context including data-formatting
[ ] Attributes [data-test-id] are added for new elements
[ ] Changes are mentioned in the changelog
[ ] The changes are tested in different browsers and in light/dark mode

Test environment config

API_URI=https://master.staging.saleor.cloud/graphql/

Task linked: SALEOR-3604 Multiselect dropdowns close after choosing a single option


Implement coprocessor access instructions

Implements coprocessor basic access instructions (MCR, MRC, MCRR, MRRC) that compile to a single instruction under --release profiles

CI tests:

rustfmt fails but functions are formatted the same as the rest of the assembly functions
stm32 target tests fail due to 3 probes being present: Need to repeat test

The aim of these is to expose a common method to interact through the ARM coprocessor interface. Separate libs can then implement the coprocessor driver's using these functions.
For example, I'm using it currently to interact with the LPC55S69 math coprocessor

If you want separate libs to later use these functions, don't they need to be pub instead of pub(crate)?

Yeah, good catch, that was there on the test library, forgot to remove it

Ok, it's now failing to compile because const ASM operands are unstable, needing #![feature(asm_const)], can this be enabled in the crate, or should this PR wait until ASM const is stabilized??

This is getting stabilized soon as per #93332

Likely in rust 1.82.0.

Hello
With the stabilization of Rust 1.82 asm_const has been stabilized. This should be enough to merge these instructions into cortex-m without breaking, although there is the caveat of a new minimum Rust version now (1.82).
Additionally, with the release of the new RP2350 chip, there is now another mainstream processor with coprocessors, which adds some necessity to the access to coprocessor instructions.
I would like some input from somebody in the team of how to proceed at this point, if I should close this PR and open a new one, keep this PR or maybe spin this functionality into a different crate.
Many thanks

which adds some necessity to the access to coprocessor instructions

The rp2350-hal uses inline assembler to make use of the instructions. Nicer APIs would be better though.


tilemap editor move_layer sorting error

Godot version
4.2 dev6
System information
Godot v4.2.dev6.mono - Windows 10.0.23560 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3050 (NVIDIA; <IP_ADDRESS>09) - 12th Gen Intel(R) Core(TM) i5-12490F (12 Threads)
Issue description
Create two new layers in the Tilemap editor and exchange the order of the two layers. The corresponding editor displays no sorting.
Only by editing the TileMap view or reopening the scene will it be displayed correctly.
https://github.com/godotengine/godot/assets/17066147/36a62aa1-8bad-4d2a-a102-a667c2229f70
Steps to reproduce

Create a new TileMap node
Create two new Layers
Exchange two layers
View the editor’s TileMap display

Minimal reproduction project
tilemap_test.zip

The problem comes from the fact Y-sorting is enabled on the TileMap node but not on any of the layer. This makes it so each layer is Y-sorted as a whole. Thus the system tries to Y-sort the layers which causes issues as they have the exact same position (more or less).
I added a warning in my local fork about that, but I guess I can PR it to master too.

问题来自于 TileMap 节点上启用了 Y 排序，但未在任何图层上启用。这使得每一层作为一个整体都是 Y 排序的。因此，系统尝试对导致问题的层进行 Y 排序，因为它们具有完全相同的位置（或多或少）。
我在我的本地分支中添加了一个关于这一点的警告，但我想我也可以通过 PR 来掌握它。

The problem comes from the fact Y-sorting is enabled on the TileMap node but not on any of the layer. This makes it so each layer is Y-sorted as a whole. Thus the system tries to Y-sort the layers which causes issues as they have the exact same position (more or less).
I added a warning in my local fork about that, but I guess I can PR it to master too.

This problem still occurs when Y sorting is not enabled in TileMap。


Incorrect extraction

As pointed at https://github.com/kevva/download/issues/42#issuecomment-85255868, extraction of files sometimes is incorrect, leading to don't maintain the execution bit and also missing files and folders.

The file modes should be fixed but the missing files and folders in the gcc package is due to some @LongLink issue. Open an issue at https://github.com/mafintosh/tar-stream about it.

Linux kernel also had (bigger) problems when extracting, so I think is also used there.

Files mode is fixed.

Issue for tar-stream at https://github.com/mafintosh/tar-stream/issues/35

Just published tar-stream 1.1.4 which has support for gnu long link/path.

Sick.

:clap:


Add arm64 build for Apple Silicon

Example PR created by this change: https://github.com/cagataygurturk/homebrew-bicep/pull/2
Fixes Azure/bicep#5378

@anthony-c-martin @bhsubra can this get reviewed and merged to help resolve backlogged enhancement issue https://github.com/Azure/bicep/issues/5378

I think this is going to be blocked by https://github.com/Azure/bicep/issues/6195 unfortunately.

Thanks for contributing this!

Thanks for contributing this btw! ❤

Was a pleasure 😊


msgpack-python is deprecated

The documentation states that msgpack-python is required for MsgPack support. The new package is msgpack.

Thanks, we should update the requirements files also.


Revert "Add validator to prevent discriminator and encodedname used together"

Reverts microsoft/typespec#3763
This is actually a pattern used for things like @odata.kind and would need more discussion to disallow

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/compiler

Show changes

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs


New feature: Install from the Unofficial Windows Binaries for Python Extension Packages

I have seen that many users have difficulty using the standard PIP repo. Maybe the binaries are corrupt or their system needs the wheel files shared on the link below. I suggest we add a new feature to give the users the opportunity to download from this repository too. This is the code I have developed but we can polish it to be more user-friendly. I have not seen any other package giving such opportunity to the users.
I tried to make system recognitions automatic and the user won't struggle opening a browser, download the wheel file, go to the terminal, etc. Everything with just a line of command is in their hands. Maybe a command like:

Archived: Unofficial Windows Binaries for Python Extension Packages
by Christoph Gohlke. Updated on 26 June 2022 at 07:27 UTC.

I tried to make system recognitions automatic and the user won't struggle opening a browser, download the wheel file, go to the terminal, etc. Everything with just a line of command is in their hands. Maybe a command like:
pip-review upep <- package-name -> <- package-version ->
If we like the feature, we may be able to make a mirror to the original link and make the 'pip-review' a semi-pip package.
def uPEP (pckg_name, pckg_version):

    '''
    Function to download Unofficial Windows Binaries for Python. Developer is not affiliated with the UCI, is not in any contract of any type with the developers of the following page, and is not maintainer of the packages presented in the link below.

    You may use this package and links downloaded at your own risk. There is no guarantee that the links work properly, or the files downloaed do not harm your system.

    Archived: Unofficial Windows Binaries for Python Extension Packages
    by Christoph Gohlke. Updated on 26 June 2022 at 07:27 UTC.
    '''

    webPath = "https://download.lfd.uci.edu/pythonlibs/archived/"
    webView = "https://www.lfd.uci.edu/~gohlke/pythonlibs/"

    import sys

    cpu_arch = ""
    for i in range (-7,-2):
        cpu_arch = cpu_arch + sys.version[i]

    if "32" in cpu_arch:
        cpu_mch = "win32"
    else:
        cpu_mch = "win_amd64"

    pthn_version = str(sys.version_info.major) + str(sys.version_info.minor)

    if sys.version_info.minor <=7:
        pthn_release = "cp" + pthn_version + "-cp" + pthn_version + "m"
        webAddress = webPath + "cp" + pthn_version + "/"

    else:
        pthn_release = "cp" + pthn_version + "-cp" + pthn_version
        webAddress = webPath + pthn_version + "/"

    webFile = pckg_name + "-" + pckg_version + "-" + pthn_release + "-" + cpu_mch + ".whl"

    import requests

    r = requests.get(webView)

    if webFile or re.sub("-", u"\u2013", webFile) in r.text:
        print ('Yes')
    else:
        print ('No')

    return webFile

input("Do you want to install the package?" )

if input == 'Y':

    import os
    os.system("python -m pip install "+ uPEP("xgboost", "1.6.1"))

else:

    print("Quit!")

How to install packages and from where is a responsibility that we defer to pip. Would pip's built-in options --index-url, --extra-index-url or --find-links not already do the job here? Run pip help install for documentation, section "Package Index Options".

This is what we read in the help:

-i, --index-url
Base URL of the Python Package Index (default https://pypi.org/simple). This should point to a repository compliant with PEP 503 (the simple repository API) or a local directory laid out in the same format.

Then the normal syntax will be something like this, without the freedom to call other options like the version or the architecture of the system.
python -m pip install -f 'https://download.lfd.uci.edu/pythonlibs/archived/' <- package ->
I think the existing feature in pip does not have all the freedom we can add with the proposed feature.

<- package -> may include the version number, like so: pip-review==1.3.0. I think the architecture can be set with other flags.
pip-review forwards all arguments it doesn't recognize to pip, so if you call something like pip-review --auto --extra-index-url 'https://download.lfd.uci.edu/pythonlibs/archived/', that will update packages from that unofficial Windows respository, in addition to PyPI.
If you are proposing to not only review updates, but also install new packages, then I say: let's not widen the scope of pip-review. Installing new packages is the business of pip. If pip is lacking in some way, contribute to pip instead!

Agreed Julian, you are right. We are making an interface for easier usage of the PIP. So I will mark this ticket for closure.


VisEditor 0.3.0 roadmap

[ ] Create TransformComponent, TintComponent etc. This will allow to mix SpriteComponnet with others types of Component like, MusicComponent, SoundComponent, TextComponent and others.
[ ] Remove EntityProxy - obscure way of manipulating different type of entities components basic properties like position, scale etc.
[ ] Refactor AssetComponent, initial idea is to create AssetReference allowing to store multiple VisAssetDescriptor, one system will them process them and create appropriate components.
[ ] Refactor ProtoComponents - transient components (non serializable) will be skipped for serialization but they should be recreatable from using components (like Postion, Scale etc.)
[ ] Remove "Component" from components classes names to decrease verbosity.
[ ] Deprecate content roots in favor of storing every assets in any directory. If some file is ambiguous we can store its type in metadata.
[ ] Move away from Kryo to JSON. JSON will allow for human readable scene files and simplify keeping scenes in VCS repo. Editor JSON files should be pretty printed.
[x] Remove editor beta update channel.
[x] Remove code support for old editor versions, 0.3.0 won't be able to load old projects without conversion anyway.
[x] Reorganize util package.
[x] Move SceneViewport
[ ] Write an project converter for VisEditor 0.2.5.

0.3.0 will be developed in separate branch, critical issues in 0.2.5 will be still fixed.
If you have any ideas or suggestions, please share it here or start a discussion on our forums.

Notes for project converter

EditorPositionComponent in Sound/Music replaced by PositionComponent

All done, finally.


Only supports square images?

may be only supports square images? because image size only one dim

We use Resize and CenterCrop to preprocess images. For images that are not square, we will process the part that gets the center square for calculation.


Implement tuition class and student Info Page

Added ViewCommand and ViewClassCommand

Updated GUI to fit all the new components

CommandResult has too many individual booleans to represent various actions to be taken by the UI, this solution is not scalable if we ever intend to add other UI Actions through commands. Let's instead change it to a single enum field to specify which UI Action.

The Class model.tuition.Student.java actually represents a list of students, and its name is a misnomer. Let's rename it to StudentList.java.

TuitionCard is meant to a quick overview of a Tuition Class to be displayed in a list, currently contains too much information. Let's move some of this information to the dedicated information page for users to view on demand. Information moved includes student list and remarks.

Moved some Student details from the PersonCard on the list to a detailed Info Page, accessed by the ViewCommand.

Codecov Report

Merging #42 (1184b0e) into master (0b5d1c8) will decrease coverage by 2.38%.
The diff coverage is 12.00%.

@@             Coverage Diff              @@
##             master      #42      +/-   ##
============================================
- Coverage     54.49%   52.11%   -2.39%
+ Complexity      438      436       -2
============================================
  Files            94      100       +6
  Lines          1868     1965      +97
  Branches        210      216       +6
============================================
+ Hits           1018     1024       +6
- Misses          807      897      +90
- Partials         43       44       +1

Impacted Files
Coverage Δ

.../seedu/address/logic/commands/AddClassCommand.java
0.00% <0.00%> (ø)

...eedu/address/logic/commands/AddToClassCommand.java
0.00% <0.00%> (ø)

...seedu/address/logic/commands/ViewClassCommand.java
0.00% <0.00%> (ø)

...java/seedu/address/logic/commands/ViewCommand.java
0.00% <0.00%> (ø)

...du/address/logic/parser/AddClassCommandParser.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
73.91% <0.00%> (-7.04%)
:arrow_down:

...in/java/seedu/address/logic/parser/ParserUtil.java
59.32% <0.00%> (ø)

...u/address/logic/parser/ViewClassCommandParser.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/ViewCommandParser.java
0.00% <0.00%> (ø)

...ain/java/seedu/address/model/tuition/Timeslot.java
8.51% <0.00%> (ø)

... and 16 more

Continue to review full report at Codecov.

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


Code checking image BLIT support interferes with image clearing

@aerofly
The following code in MVKImage is interfering with the ability of MVKCmdClearImage to clear an image (by rendering the clear color to it):
// If this format doesn't support being blitted to, and the usage
// doesn't specify use as an attachment, turn off
// MTLTextureUsageRenderTarget.
VkFormatProperties props;
_device->getPhysicalDevice()->getFormatProperties(getVkFormat(), &props);
if (!mvkAreAllFlagsEnabled(_isLinear ? props.linearTilingFeatures : props.optimalTilingFeatures, VK_FORMAT_FEATURE_BLIT_DST_BIT) &&
	!mvkIsAnyFlagEnabled(_usage, VK_IMAGE_USAGE_COLOR_ATTACHMENT_BIT|VK_IMAGE_USAGE_DEPTH_STENCIL_ATTACHMENT_BIT)) {
	mvkDisableFlag(usage, MTLTextureUsageRenderTarget);
}

This code appeared in this commit. Bizarrely...unless I am blind...I can't find that code in the PR that contains that commit...which I am very confused about.
What was the reason for this code? What problem is it trying to solve? Can it simply be removed to clear the way for image clearing? Or can we adjust it to include VK_IMAGE_USAGE_TRANSFER_DST_BIT usage in the test for attachment usage?

@billhollings
I am pretty sure, I did not add or modify the above code as my commit was just introducing the MVKVector class. I did not work on anything else.
However when sending the pull request with my MVKVector class, I made a few mistakes on updating my version of MoltenVK as you can see in the conversation history.
So my assumption is, due to this issue, this code somehow found it's way back into MoltenVK even though it was previously removed by you or somebody, but I also don't see it anywhere in the history?

@aerofly Thanks.
Okay...I think I've found the original commit.
@cdavis5e
On further examination...it turns out NOT to be the source of the error above...so we don't have to fix anything. But I am curious what this change was for. It is odd that we are turning off a Metal usage under the covers...rather than reporting via the Vulkan API...and I'm wondering if we should revisit this function in general. Do you recall why this change was made?

I was running the CTS, and it tried to blit to a VK_FORMAT_E5B9G9R9_UFLOAT image. Metal--or perhaps more precisely, the driver for my Radeon card--cannot render to this format. And I think it hit the case where we fall back to a render in vkCmdBlitImage().

@cdavis5e Thanks.
MTLTextureUsageRenderTarget is being set on CTS tests that are using VK_FORMAT_E5B9G9R9_UFLOAT_PACK32 as a BLIT source. So you turning it off under the covers doesn't affect that behaviour. And as you say...it won't ever be a BLIT destination...where removing MTLTextureUsageRenderTarget would cause a Metal validation error.


How to deploy with a single docker-compose file?

Hello. I love bobarr for being a one-stop-shop for movies/tv shows and have been trying to set it up on my home server for a while now.
What I would like to do, however, is to have all of the configuration in a single docker-compose file, instead of running the prescribed installation and start-up scripts. This allows me to better organize where certain files go where (e.g. config files in my cache drive). In addition, I also wanted to run bobarr with azinchen's nordvpn docker client, which I found to be the easiest way to have services run through my subscription of nordvpn.
I therefore took the provided bobarr/docker-compose.yml and started to add environmental variables from bobarr/.env and azinchen's nordvpn stuff.
Bear in mind that I'm basically a "script kiddie" when it comes to docker. I am okay with making small modifications to example compose files, but don't really understand how it all comes together behind the hood. I hence apologize in advance if I made any gross mistakes in the following compose file.
This is the current version of said compose file, which I am able to deploy. While I did not run into any error during the deployment, the server is not fully functional. I got as far as being able to add movies on the bobarr interface, but couldn't find the downloads starting on transmission.
I'd appreciate any feedback/tips to get this to work. Thanks!
version: "3.6"

volumes:
  web_build:
  api_build:
  api_node_modules:
  web_node_modules:
  db_data:
  redis_data:

networks:
  default:

services:
  api:
    container_name: bobarr-api
    environment:
      - ENV=production
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
      - LIBRARY_MOVIES_FOLDER_NAME=movies
      - LIBRARY_TV_SHOWS_FOLDER_NAME=tvshows
    image: iam4x/bobarr-api:latest
    command: yarn start:prod
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr:/usr/library
    ports:
      - 4000:4000
#     links:
#       - vpn:transmission
#     networks:
#       - default

  web:
    container_name: bobarr-web
#     environment:
#       - WEB_UI_API_URL=http://yourdomain.com/api
    image: iam4x/bobarr-web:latest
    command: yarn start
    restart: unless-stopped
    ports:
      - 3000:3000
#     links:
#       - vpn:transmission
#     networks:
#       - default

  postgres:
    container_name: bobarr-postgresql
    image: postgres:12-alpine
    environment:
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
    restart: unless-stopped
    volumes:
      - db_data:/var/lib/postgresql/data

  redis:
    container_name: bobarr-redis
    image: bitnami/redis:5.0.6
    environment:
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
    restart: unless-stopped
    volumes:
      - redis_data:/bitnami/redis/data

  jackett:
    image: linuxserver/jackett
    container_name: bobarr-jacket
    environment:
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
    restart: unless-stopped
    volumes:
      - /mnt/ssd0/appdata/jackett/config:/config
      - /mnt/ssd1/media/bobarr/jackett:/downloads
    ports:
      - 9117:9117
#     network_mode: service:vpn
#     depends_on:
#       - vpn

  flaresolverr:
    image: ghcr.io/flaresolverr/flaresolverr:latest
    container_name: bobarr-flaresolverr
    environment:
      - LOG_LEVEL=info
    restart: unless-stopped
    ports:
      - 8191:8191

  transmission:
    image: linuxserver/transmission
    container_name: bobarr-transmission
    environment:
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr/downloads:/downloads
      - /mnt/ssd0/appdata/transmission/config:/config
      - /mnt/ssd0/appdata/transmission/watch:/watch
    network_mode: service:vpn
    depends_on:
      - vpn

#   transmission-web:
#     image: dperson/nginx
#     container_name: bobarr-transmission-web
#     depends_on:
#       - transmission
#     environment:
#       - TZ=US/Eastern
#     ports:
#       - "9091:81"
#     command: -w "http://transmission:9091;/"
#     restart: unless-stopped

  vpn:
    image: azinchen/nordvpn:latest
    network_mode: bridge
    container_name: bobarr-nordvpnwireguard
    cap_add:
      - net_admin
    devices:
      - /dev/net/tun
    environment:
      - USER=[nordvpn-user]
      - PASS=[nordvpn-pass]
      - COUNTRY=[nordvpn-country]
      - GROUP=P2P
      - TECHNOLOGY=openvpn_udp
      - RANDOM_TOP=10
      - RECREATE_VPN_CRON=5 */3 * * *
      - NETWORK=<IP_ADDRESS>/24
      - OPENVPN_OPTS=--mute-replay-warnings
    ports:
#       - 9117:9117  ## Jackett
      - 9091:9091  ## Transmission
      - 51413:51413  ## Transmission
      - 51413:51413/udp  ## Transmission

Hi !
Here's what I suggest, and it seem to work. However, I'm not using the     image: azinchen/nordvpn:latest on my side. I'm using : image: ghcr.io/bubuntux/nordlynx
version: "3.6"
volumes:
web_build:
api_build:
api_node_modules:
web_node_modules:
db_data:
redis_data:
networks:
default:
services:
vpn:
image: azinchen/nordvpn:latest
network_mode: bridge
container_name: bobarr-nordvpnwireguard
cap_add:
- net_admin
devices:
- /dev/net/tun
environment:
- USER=[nordvpn-user]
- PASS=[nordvpn-pass]
- COUNTRY=[nordvpn-country]
- GROUP=P2P
- TECHNOLOGY=openvpn_udp
- RANDOM_TOP=10
- RECREATE_VPN_CRON=5 */3 * * *
- NETWORK=<IP_ADDRESS>/24
- OPENVPN_OPTS=--mute-replay-warnings
ports:
- 9091:9091  ## Transmission
- 51413:51413  ## Transmission
- 51413:51413/udp  ## Transmission
api:
container_name: bobarr-api
environment:
- ENV=production
- PUID=3001
- PGID=3001
- UMASK_SET=0002
- TZ=US/Eastern
- POSTGRES_DB=bobarr
- POSTGRES_USER=bobarr
- POSTGRES_PASSWORD=bobarr
- REDIS_PASSWORD=bobarr
- DEBUG_REDIS=false
- JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
- JACKETT_MANUAL_SEARCH_TIMEOUT=15000
- LIBRARY_MOVIES_FOLDER_NAME=movies
- LIBRARY_TV_SHOWS_FOLDER_NAME=tvshows
image: iam4x/bobarr-api:latest
command: yarn start:prod
restart: unless-stopped
volumes:
- /mnt/ssd1/media/bobarr:/usr/library
ports:
- 4000:4000
links:
- vpn:transmission
networks:
- default
depends_on:
- vpn
web:
container_name: bobarr-web
environment:
- WEB_UI_API_URL=http://yourdomain.com/api
image: iam4x/bobarr-web:latest
command: yarn start
restart: unless-stopped
ports:
  - 3000:3000

postgres:
container_name: bobarr-postgresql
image: postgres:12-alpine
environment:
- POSTGRES_DB=bobarr
- POSTGRES_USER=bobarr
- POSTGRES_PASSWORD=bobarr
restart: unless-stopped
volumes:
- db_data:/var/lib/postgresql/data
redis:
container_name: bobarr-redis
image: bitnami/redis:5.0.6
environment:
- REDIS_PASSWORD=bobarr
- DEBUG_REDIS=false
restart: unless-stopped
volumes:
- redis_data:/bitnami/redis/data
jackett:
image: linuxserver/jackett
container_name: bobarr-jacket
environment:
- PUID=3001
- PGID=3001
- UMASK_SET=0002
- TZ=US/Eastern
- JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
- JACKETT_MANUAL_SEARCH_TIMEOUT=15000
restart: unless-stopped
volumes:
- /mnt/ssd0/appdata/jackett/config:/config
- /mnt/ssd1/media/bobarr/jackett:/downloads
ports:
- 9117:9117
flaresolverr:
image: ghcr.io/flaresolverr/flaresolverr:latest
container_name: bobarr-flaresolverr
environment:
- LOG_LEVEL=info
restart: unless-stopped
ports:
- 8191:8191
depends_on:
- vpn
transmission:
image: linuxserver/transmission
container_name: bobarr-transmission
environment:
- PUID=3001
- PGID=3001
- UMASK_SET=0002
- TZ=US/Eastern
restart: unless-stopped
volumes:
- /mnt/ssd1/media/bobarr/downloads:/downloads
- /mnt/ssd0/appdata/transmission/config:/config
- /mnt/ssd0/appdata/transmission/watch:/watch
network_mode: service:vpn
depends_on:
- vpn
transmission-web:
image: dperson/nginx
container_name: bobarr-transmission-web
depends_on:
- transmission
environment:
- TZ=America/Toronto
ports:
- "9091:80"
- "4443:443"
command: -w "http://transmission:9091;/"
restart: unless-stopped
depends_on:
- vpn
links:
- vpn:transmission
networks:
- default

Hi @mune360! Thank you so much for looking into this!
I've managed to adapt my config to yours. I believe we're close! I think I'm still having trouble accessing transmission from behind the vpn through the reverse proxy. That part of my config is the same as yours, I believe.
version: "3.6"

volumes:
  web_build:
  api_build:
  api_node_modules:
  web_node_modules:
  db_data:
  redis_data:

networks:
  default:

services:

  vpn:
    image: azinchen/nordvpn:latest
    network_mode: bridge
    container_name: bobarr-nordvpnwireguard
    cap_add:
      - net_admin
    devices:
      - /dev/net/tun
    environment:
      - USER=[nordvpn_user]
      - PASS=[nordvpn_pass]
      - COUNTRY=[nordvpn_country]
      - GROUP=P2P
      - TECHNOLOGY=openvpn_udp
      - RANDOM_TOP=10
      - RECREATE_VPN_CRON=5 */3 * * *
      - NETWORK=<IP_ADDRESS>/24
      - OPENVPN_OPTS=--mute-replay-warnings
    ports:
      - 8081:80

  api:
    container_name: bobarr-api
    environment:
      - ENV=production
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
      - LIBRARY_MOVIES_FOLDER_NAME=movies
      - LIBRARY_TV_SHOWS_FOLDER_NAME=tvshows
    image: iam4x/bobarr-api:latest
    command: yarn start:prod
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr:/usr/library
    ports:
      - 4000:4000
    links:
      - vpn:transmission
    networks:
      - default
    depends_on:
      - vpn

  web:
    container_name: bobarr-web
#     environment:
#       - WEB_UI_API_URL=http://yourdomain.com/api
    image: iam4x/bobarr-web:latest
    command: yarn start
    restart: unless-stopped
    ports:
      - 3000:3000
#     links:
#       - vpn:transmission
#     networks:
#       - default

  postgres:
    container_name: bobarr-postgresql
    image: postgres:12-alpine
    environment:
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
    restart: unless-stopped
    volumes:
      - db_data:/var/lib/postgresql/data

  redis:
    container_name: bobarr-redis
    image: bitnami/redis:5.0.6
    environment:
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
    restart: unless-stopped
    volumes:
      - redis_data:/bitnami/redis/data

  jackett:
    image: linuxserver/jackett
    container_name: bobarr-jacket
    environment:
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
    restart: unless-stopped
    volumes:
      - /mnt/ssd0/appdata/jackett/config:/config
      - /mnt/ssd1/media/bobarr/jackett:/downloads
    ports:
      - 9117:9117
#     network_mode: service:vpn
#     depends_on:
#       - vpn

  flaresolverr:
    image: ghcr.io/flaresolverr/flaresolverr:latest
    container_name: bobarr-flaresolverr
    environment:
      - LOG_LEVEL=info
    restart: unless-stopped
    ports:
      - 8191:8191
    depends_on:
      - vpn

  transmission:
    image: linuxserver/transmission
    container_name: bobarr-transmission
    environment:
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr/downloads:/downloads
      - /mnt/ssd0/appdata/transmission/config:/config
      - /mnt/ssd0/appdata/transmission/watch:/watch
    network_mode: service:vpn
    depends_on:
      - vpn

  transmission-web:
    image: dperson/nginx
    container_name: bobarr-transmission-web
    depends_on:
      - transmission
      - vpn
    environment:
      - TZ=US/Eastern
    ports:
      - 9091:80
    command: -w "http://transmission:9091;/"
    restart: unless-stopped
    links:
      - vpn:transmission
    networks:
      - default

Taking a look at the bobarr-transmission-web > Logs, this is what I see.
2023/04/09 01:35:46 [emerg] 7#7: host not found in upstream "transmission" in /etc/nginx/conf.d/default.conf:43
nginx: [emerg] host not found in upstream "transmission" in /etc/nginx/conf.d/default.conf:43

Hi @mune360! Thank you so much for looking into this!
I've managed to adapt my config to yours. I believe we're close! I think I'm still having trouble accessing transmission from behind the vpn through the reverse proxy. That part of my config is the same as yours, I believe.
version: "3.6"

volumes:
  web_build:
  api_build:
  api_node_modules:
  web_node_modules:
  db_data:
  redis_data:

networks:
  default:

services:

  vpn:
    image: azinchen/nordvpn:latest
    network_mode: bridge
    container_name: bobarr-nordvpnwireguard
    cap_add:
      - net_admin
    devices:
      - /dev/net/tun
    environment:
      - USER=[nordvpn_user]
      - PASS=[nordvpn_pass]
      - COUNTRY=[nordvpn_country]
      - GROUP=P2P
      - TECHNOLOGY=openvpn_udp
      - RANDOM_TOP=10
      - RECREATE_VPN_CRON=5 */3 * * *
      - NETWORK=<IP_ADDRESS>/24
      - OPENVPN_OPTS=--mute-replay-warnings
    ports:
      - 8081:80

  api:
    container_name: bobarr-api
    environment:
      - ENV=production
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
      - LIBRARY_MOVIES_FOLDER_NAME=movies
      - LIBRARY_TV_SHOWS_FOLDER_NAME=tvshows
    image: iam4x/bobarr-api:latest
    command: yarn start:prod
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr:/usr/library
    ports:
      - 4000:4000
    links:
      - vpn:transmission
    networks:
      - default
    depends_on:
      - vpn

  web:
    container_name: bobarr-web
#     environment:
#       - WEB_UI_API_URL=http://yourdomain.com/api
    image: iam4x/bobarr-web:latest
    command: yarn start
    restart: unless-stopped
    ports:
      - 3000:3000
#     links:
#       - vpn:transmission
#     networks:
#       - default

  postgres:
    container_name: bobarr-postgresql
    image: postgres:12-alpine
    environment:
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
    restart: unless-stopped
    volumes:
      - db_data:/var/lib/postgresql/data

  redis:
    container_name: bobarr-redis
    image: bitnami/redis:5.0.6
    environment:
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
    restart: unless-stopped
    volumes:
      - redis_data:/bitnami/redis/data

  jackett:
    image: linuxserver/jackett
    container_name: bobarr-jacket
    environment:
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
    restart: unless-stopped
    volumes:
      - /mnt/ssd0/appdata/jackett/config:/config
      - /mnt/ssd1/media/bobarr/jackett:/downloads
    ports:
      - 9117:9117
#     network_mode: service:vpn
#     depends_on:
#       - vpn

  flaresolverr:
    image: ghcr.io/flaresolverr/flaresolverr:latest
    container_name: bobarr-flaresolverr
    environment:
      - LOG_LEVEL=info
    restart: unless-stopped
    ports:
      - 8191:8191
    depends_on:
      - vpn

  transmission:
    image: linuxserver/transmission
    container_name: bobarr-transmission
    environment:
      - PUID=3001
      - PGID=3001
      - UMASK_SET=0002
      - TZ=US/Eastern
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr/downloads:/downloads
      - /mnt/ssd0/appdata/transmission/config:/config
      - /mnt/ssd0/appdata/transmission/watch:/watch
    network_mode: service:vpn
    depends_on:
      - vpn

  transmission-web:
    image: dperson/nginx
    container_name: bobarr-transmission-web
    depends_on:
      - transmission
      - vpn
    environment:
      - TZ=US/Eastern
    ports:
      - 9091:80
    command: -w "http://transmission:9091;/"
    restart: unless-stopped
    links:
      - vpn:transmission
    networks:
      - default

Taking a look at the bobarr-transmission-web > Logs, this is what I see.
2023/04/09 01:35:46 [emerg] 7#7: host not found in upstream "transmission" in /etc/nginx/conf.d/default.conf:43
nginx: [emerg] host not found in upstream "transmission" in /etc/nginx/conf.d/default.conf:43

Hi. I made some progress on this. I am now able to access transmission through my VPN but am running into other problems. Adding a movie doesn't seem to do anything. When I try to add a torrent file manually by clicking on the movie card and then through Replace > select own .torrent, this is what I see:

In addition to the above, I am of course skeptical that other parts of bobarr is fully functional, so any help to fully set this up is much appreciated.
I know have two docker compose files -- one for bobarr and another one just for the VPN. I do everything on Portainer, and first deploy vpn.yml before deploying bobarr.yml.
bobarr.yml
version: "3.6"

volumes:
  web_build:
  api_build:
  api_node_modules:
  web_node_modules:
  db_data:
  redis_data:

networks:
  default:

services:

  api:
    container_name: bobarr-api
    environment:
      - ENV=production
      - PUID=3000
      - PGID=3000
      - UMASK_SET=0002
      - TZ=US/Eastern
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
      - LIBRARY_MOVIES_FOLDER_NAME=movies
      - LIBRARY_TV_SHOWS_FOLDER_NAME=tvshows
    image: iam4x/bobarr-api:latest
    command: yarn start:prod
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr:/usr/library
    ports:
      - 4000:4000
    external_links:
      - vpn:transmission
    networks:
      - default

  web:
    container_name: bobarr-web
#     environment:
#       - WEB_UI_API_URL=http://yourdomain.com/api
    image: iam4x/bobarr-web:latest
    command: yarn start
    restart: unless-stopped
    ports:
      - 3000:3000

  postgres:
    container_name: bobarr-postgresql
    image: postgres:12-alpine
    environment:
      - POSTGRES_DB=bobarr
      - POSTGRES_USER=bobarr
      - POSTGRES_PASSWORD=bobarr
    restart: unless-stopped
    volumes:
      - db_data:/var/lib/postgresql/data

  redis:
    container_name: bobarr-redis
    image: bitnami/redis:5.0.6
    environment:
      - REDIS_PASSWORD=bobarr
      - DEBUG_REDIS=false
    restart: unless-stopped
    volumes:
      - redis_data:/bitnami/redis/data

  jackett:
    image: linuxserver/jackett
    container_name: bobarr-jacket
    environment:
      - PUID=3000
      - PGID=3000
      - UMASK_SET=0002
      - TZ=US/Eastern
      - JACKETT_AUTOMATIC_SEARCH_TIMEOUT=120000
      - JACKETT_MANUAL_SEARCH_TIMEOUT=15000
    restart: unless-stopped
    volumes:
      - /mnt/ssd0/appdata/jackett/config:/config
      - /mnt/ssd1/media/bobarr/jackett:/downloads
    ports:
      - 9117:9117

  flaresolverr:
    image: ghcr.io/flaresolverr/flaresolverr:latest
    container_name: bobarr-flaresolverr
    environment:
      - LOG_LEVEL=info
    restart: unless-stopped
    ports:
      - 8191:8191

  transmission:
    image: linuxserver/transmission
    container_name: bobarr-transmission
    environment:
      - PUID=3000
      - PGID=3000
      - UMASK_SET=0002
      - TZ=US/Eastern
    restart: unless-stopped
    volumes:
      - /mnt/ssd1/media/bobarr/downloads:/downloads
      - /mnt/ssd0/appdata/transmission/config:/config
      - /mnt/ssd0/appdata/transmission/watch:/watch
    network_mode: container:nordvpnwireguard

vpn.yml
version: "3.5"
services:
  vpn:
    image: azinchen/nordvpn:latest
    network_mode: bridge
    container_name: bobarr-nordvpnwireguard
    cap_add:
      - net_admin
    devices:
      - /dev/net/tun
    environment:
      - USER=[nordvpn_user]
      - PASS=[nordvpn_pass]
      - COUNTRY=[nordvpn_country]
      - GROUP=P2P
      - TECHNOLOGY=openvpn_udp
      - RANDOM_TOP=10
      - RECREATE_VPN_CRON=5 */3 * * *
      - NETWORK=<IP_ADDRESS>/24
      - OPENVPN_OPTS=--mute-replay-warnings
    ports:
      - 9091:9091  ## bobarr-transmission
      - 51413:51413  ## transmission
      - 51413:51413/udp  ## transmission

solution?

Unfortunately, I didn't arrive at one and moved onto the standard arr stack instead.
Still interested in deploying this if someone can make it work!


Change tenant VPC cidr from /16 to /20 to increase number of supported tenants

Use Case
Boost currently limits the number of tenants to 255 due to IP blocks availability. Each VPC is create with a 10.X.X.X/16 cidr block hence only allowing 255 VPCs and therefore tenants to be created.
Proposed Solution
Change VPC cidr block to /20 to support 4080 tenants and 4096 IPs per VPC

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

Hi @somensi-aws , thanks for your feature request! I have a few things to mention:

Due to the current way that CIDR tracking is implemented with a DDB table, this feature request would incur a breaking change to existing SaaS Boost installations by requiring the table to be reformatted and (potentially) existing tenant VPCs to get recreated. I will mark this issue to reflect that.
One side effect of increasing the number of available VPCs for tenant usage is to decrease the number of IPs available to each tenant VPC. Because each SaaS Boost user's application's needs will differ, it would instead make more sense for the VPC CIDR configuration to be calculated based on a configured maximum number of required IP addresses for a given application at SaaS Boost start time. This way, everyone's use case will be covered by SaaS Boost out of the box. I will change the title of this feature to reflect that.

If you're interested in helping implement this feature request, please keep the above in mind.

In case the limit for number of tenants is increased, the APIs for Tenant management and Onboarding may need to be visited  as they do not currently support nextToken. The metrics queries against CloudWatch metrics may also timeout if the number of tenants increase.


Merge soniq into master

It doesn't really make sense to merge since I actually want to overwrite master with the soniq (development) branch.


Incompatible shapes in GPMC with multiple outputs

GPflow version : 2.1.4
tensorflow version : 2.3.1
Hi all,
I am trying to implement a constrained GP regression with a custom kernel. Two input dimensions and multiple output dimensions that are correlated with one another. I've implemented this as a subclass to MultiOutputKernel and confirmed that the shapes of kernel.__call__ are as they should be according to the docs. When I come to try and use this in either GPR or GPMC I run into inconsistent shape errors in conditionals.conditionals.py
I can reproduce this error by taking this tutorial and switching out the SVGP with GPMC as below.
     N = 100  # number of points
     D = 1  # number of input dimensions
     M = 15  # number of inducing points
     L = 2  # number of latent GPs
     P = 3  # number of observations = output dimensions

     def generate_data(N=100):
         X = np.random.rand(N)[:, None] * 10 - 5  # Inputs = N x D
         G = np.hstack((0.5 * np.sin(3 * X) + X, 3.0 * np.cos(X) - X))  # G = N x L
         W = np.array([[0.5, -0.3, 1.5], [-0.4, 0.43, 0.0]])  # L x P
         F = np.matmul(G, W)  # N x P
         Y = F + np.random.randn(*F.shape) * [0.2, 0.2, 0.2]
         return X, Y

     X, Y = data = generate_data(N)
     Zinit = np.linspace(-5, 5, M)[:, None]

     kern_list = [gpf.kernels.SquaredExponential() + gpf.kernels.Linear() for _ in range(P)]
     # Create multi-output kernel from kernel list
     kernel = gpf.kernels.SeparateIndependent(kern_list)
     # initialization of inducing input locations (M random points from the training inputs)
     Z = Zinit.copy()
     # create multi-output inducing variables from Z
     iv = gpf.inducing_variables.SharedIndependentInducingVariables(
         gpf.inducing_variables.InducingPoints(Z)
     )

     # below is different from the tutorial

     # m = gpf.models.SVGP(kernel, gpf.likelihoods.Gaussian(), inducing_variable=iv, num_latent_gps=P)
     m = gpf.models.GPMC(data=(X,Y),kernel=kernel, likelihood=gpf.likelihoods.Gaussian(), num_latent_gps=P)

     m.predict_f(X)

which fails with
InvalidArgumentError: Incompatible shapes: [20,2,2] vs. [20,20] [Op:AddV2]

originating at the line
~/virtualenvs/icf/lib/python3.7/site-packages/gpflow/conditionals/conditionals.py in _conditional(Xnew, X, kernel, f, full_cov, full_output_cov, q_sqrt, white)
       122         - variance: [N, R] (full_cov = False), [R, N, N] (full_cov = True)
       123     """
   --> 124     Kmm = kernel(X) + eye(tf.shape(X)[-2], value=default_jitter(), dtype=X.dtype)  # [..., M, M]
       125     Kmn = kernel(X, Xnew)  # [M, ..., N]
       126     Knn = kernel(Xnew, full_cov=full_cov)  # [..., N] (full_cov = False) or [..., N, N] (True)

Apparently the shape of kernel(X) is not as expected.
Thanks in advance!

I'm facing the same problem when trying to combine a SeparateIndependent kernel with a GPMC model. It is also an issue when using the SharedIndependent kernel. Any ideas on how to fix this issue?
Example:
import numpy as np
import gpflow as gp
gpflow.config.set_default_float(np.float64)
gpflow.config.set_default_jitter(1e-4)

J = 100  #number of time points
P = 5    #number of processes

def generate_data(J, P):
    Xs = np.linspace(0, 1, J)[:, None]
    kernel = gp.kernels.SquaredExponential(lengthscales = 0.2)
    cov = kernel(Xs)
    L = np.linalg.cholesky(cov + np.eye(J) * 1e-6)
    epsilon = np.random.randn(J, P)
    F = np.sin(Xs) + np.matmul(L, epsilon)
    return Xs, F

X, Y = generate_data(J, P)
data = (X, Y)

Zinit = np.linspace(0, 1, 10)[:, None]
Z = Zinit.copy()
iv = gp.inducing_variables.SharedIndependentInducingVariables(gp.inducing_variables.InducingPoints(Z))

# Example with shared independent kernel
k_shind = gp.kernels.SharedIndependent(gp.kernels.SquaredExponential(), output_dim = P)
m_shind1 = gp.models.SVGP(k_shind, gp.likelihoods.Gaussian(), inducing_variable=iv, num_latent_gps = P)
m_shind2 = gp.models.GPMC(data, k_shind, gp.likelihoods.Gaussian(), num_latent_gps = P)

m_shind1.log_posterior_density(data)
m_shind2.log_posterior_density()

# Examle with separate independent kernel
k_sepind = gp.kernels.SeparateIndependent([gp.kernels.SquaredExponential() for _ in range(P)])
m_sepind1 = gp.models.SVGP(k_sepind, gp.likelihoods.Gaussian(), inducing_variable=iv, num_latent_gps = P)
m_sepind2 = gp.models.GPMC(data, k_sepind, gp.likelihoods.Gaussian(), num_latent_gps = P)

m_sepind1.log_posterior_density(data)
m_sepind2.log_posterior_density()


[WIP] Runtime Capabilities Analysis (RCA)

Closing in favor of #1164 since the commit history and comments on this one is a bit verbose.
@sezna, I saw that you started reviewing. If you are still interested in reviewing this RCA, please take a look at #1164.


Ensure that no instance in the project overrides the project setting for enabling OSLogin(OSLogin needs to be enabled in project metadata for all instances)

Possible check to target the following resource

[ ] google_compute_instance

closing, tracking in spreadsheet


Fix all GHA linting

Linting processes to fix:

[x] EditorConfig
[x] Prettier
[x] nf-core lint

Linting (and CI) errors resolved in #3.


[HUDI-7498] Fix schema for HoodieTimestampAwareParquetInputFormat

Change Logs
Fix schema for HoodieTimestampAwareParquetInputFormat
Impact
Fixes field ordering
Risk level (write none, low medium or high below)
low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@codope Have you made more progress on this fix?

@codope Have you made more progress on this fix?

Not yet. We need to fix https://issues.apache.org/jira/browse/HUDI-7554 as well.


Decouple the current billing index from best block-number

Describe the bug
Currently, all network validators are required to have offchain worker enabled. This is because we depend on the best block number in retrieval contracts ids from the billing shards. Skipping subsets of contracts in the billing cycle can occur in cases such as:

Offchain worker is not enabled on one or more validators
Offchain workers are not triggered on a block import (this is on substrate, it is not guaranteed to trigger the hook every block).
Local key store has multiple signing/aura keys (some validators were incorrectly set up).

My suggestion is to store the current index in the smart contract pallet storage and increment it upon every successful billing cycle.
By decoupling the current billing index from best block-number we would ensure that all contracts billing are handled as long as there is at least one validator have offcahin worker enabled on the network and configured correctly .
if this implemented, The only side effect for less offchain workers would be prelonged cycle (less workers = more time) but this would be better than no billing done at all.
This is primarily an improvement to enable better error tolerance, but we can also plan and proceed in parallel with a more comprehensive redesign that involve zos nodes, gateways, and other components responsible for triggering the billing instead of the offchain workers if needed.

Consider research if we can use off-chain indexing (interesting feature) for storing last index off-chain instead of on-chain.

This can be done using the approach suggested here https://github.com/threefoldtech/tfchain/issues/932#issuecomment-1954991279

Closing this now as the concerns here was handled as part of https://github.com/threefoldtech/tfchain/issues/932


Gradle multi module projects

I couldn't really figure out how to use fxlauncher in a Gradle project containing multiple modules. I added the needed gradle code to the build.gradle file of the parent project but it doesn't copy the build files from the modules to the output directory. Could you help me out please?
P.S.: I am relatively new to Java and coding in general.

Hi @SirWindfield - I'm no Gradle expert myself, having only used it to create the sample for fxlauncher (Maven guy here :), but I'm sure I'll figure it out. The way I assemble the artifacts in build/libs might not be the best approach either, so let me have a look at this tomorrow and get back to you, OK?

It would be helpful if you can share the project or at least the project skeleton as you have it right now as well :)

Never mind, I just solved it. It works by default without changing anything specific in the gradle file itself. I just messed up and forgot to add a task for copying the files like you did in the example file. Sorry for the inconvenience 😞

Great :) No problem at all, I'll close the issue then. Please let me know if you run into any further issues.