[TACHYON-1251] Add configuration parameter for MAX_TRY in LeaderInquireClient.java

Please review
https://tachyon.atlassian.net/browse/TACHYON-1251

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6716/
Test PASSed.

Thanks for the patch!
Could you also update docs/Configuration-Settings.md to explain this newly added configuration parameter?

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6717/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6718/Failed Tests: 1org.tachyonproject:tachyon-integration-tests: 1tachyon.client.IsolatedTachyonFileSystemIntegrationTest.unlockBlockTest3
Test FAILed.

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6719/
Test PASSed.

Thanks for the improvement! I left a few comments.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6739/Failed Tests: 1org.tachyonproject:tachyon-servers: 1tachyon.master.block.BlockMasterTest.detectLostWorkerTest
Test FAILed.

Jenkins, test this please.

Jenkins, test this please.

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6747/
Test PASSed.

+1 LGTM

I think the doc variable needs to be renamed. Thanks!

@shuhuai007 I think @gpang has two comments you should address. Otherwise it looks good to me.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6768/
Test PASSed.

@shuhuai007 , thanks for the fix! Could you fix merge conflicts before I merge your PR?
Thanks

@apc999
I have merged the conflicts.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6799/Failed Tests: 1org.tachyonproject:tachyon-servers: 1tachyon.master.file.FileSystemMasterTest.deleteDirTest
Test FAILed.

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6803/
Test PASSed.

+1 LGTM. @gpang , could you take a look?

LGTM


Remove built-in kubectl, filters, defaultNamespace, fix list bugs

Description
Changes proposed in this pull request:

Remove built-in kubectl, filters, defaultNamespace, fix list bugs

Testing
Tested e2e.
Related issue(s)
Fix #1003

After rebase, I see that the name in the permission error is the one that we "hardcoded" on the backend. What should I do? Replace that in e2e test assertion? or maybe we should remove the hardcoded one and instead use the same name that was configured for group 🤔 ?

@mszostok yes, unfortunately K8s API returns just the username for authorization erorrs, without group name 😞 I think for tests we can use, just to make sure
Both options work for me (updating assertions or adding static user name for test cases), but I think the one with updating username in e2e-test-values.yaml for tests will be a bit better to debug if something goes wrong? 🤔


[SE-0258] Determine mutating-ness of accessors based on the storage declaration

With property wrappers, the mutating-ness of the accessors can be determined
non-locally by the property wrapper type itself. Ensure that the storage
declaration is consulted to determine the mutating-ness of accessors,
eliminating a crash involving willSet/didSet and property wrappers.
Fixes rdar://problem/51669930.

@swift-ci please smoke test

@swift-ci please test source compatibility


chore: bump gradle memory

This PR resolves []
Description
Follow up from https://github.com/artsy/eigen/pull/10485 bumps gradle memory

#nochangelog

Warnings

:warning:

✅ No changelog changes

  Generated by :no_entry_sign: dangerJS against c13082d9e1cf7801dd8fed1885876aab1d121370


Get all entries

Hello there . I would like to request a feature called "Get All Entries".
It will be like you can get all Keyvalue entries in Array of the database. It should be like
[
{ "Key": "value" },
{ "Key2": "val2"},
]

Hi @MEZBATHEKING - Thanks for the question and the only way to do this is to use the iterator method. Here
https://github.com/jaredwray/keyv/tree/main/packages/keyv#iterator


add protractor ci?

Protractor + Travis CI + Sauce Labs

At the moment we don't have plans to run the e2e tests in CI, however, if you have some time you're welcome to contribute with that!

OK, but I need a SAUCE_USERNAME and SAUCE_ACCESS_KEY (open source)
Like it

#734

@Shyam-Chen any updates here?

Is still failures
Failed: Component returned failure code
result

Here's a related issue. Do you think it can help? https://github.com/angular/protractor/issues/2297

I don't know, either
I think it's no crash
In my Angular 1 project is successfully executed

I finished
result
version: rc.1

Awesome job! I see only one e2e test running. Do you know what is causing this?

That is my simple project.
I'll wait for this merger (#808), and then start working

@Shyam-Chen awesome! Looking forward for it!


Add my username

What does this PR do?

Fixes #(issue)

Type of change

Bug fix (non-breaking change which fixes an issue)
Chore (refactoring code, technical debt, workflow improvements)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

How should this be tested?

[ ] Test A
[ ] Test B

Mandatory Tasks

[ ] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

I haven't read the contributing guide
My code doesn't follow the style guidelines of this project
I haven't commented my code, particularly in hard-to-understand areas
I haven't checked if my PR needs changes to the documentation
I haven't checked if my changes generate no new warnings
I haven't added tests that prove my fix is effective or that my feature works
I haven't checked if new and existing unit tests pass locally with my changes

@FabianCristancho You were not supported to commit package-lock.json and yarn.lock file. Fix it

Why fail it?

Close this pr and create a new pr (this time without those yarn and lock files) @FabianCristancho


Publish latest release to NPM as other packages are breaking

Please answer these questions before submitting a bug report.
What version of OpenTelemetry are you using?
0.10.2
What version of Node are you using?
14
What did you do?
Cannot install 0.10.2 as it's not published on the npm resgistry though other packages already depend on it like @azure/core-http which causes packages like azure-iot-device and azure-iot-device-mqtt to fail
If possible, provide a recipe for reproducing the error.
What did you expect to see?
It should install and run fine
What did you see instead?
npm install fails because this dependency hasn't been published
Additional context
Add any other context about the problem here.
Please publish the latest version to npm

It might has connection with the npm issue. For more information, see https://status.npmjs.org/


Hi team I want to highlight a particular point in chart.js is it possible below is the example the data is taken from the table and is it possible for the x and y interception at the dataset image

Hi team I want to highlight a particular point in chart.js is it possible below is the example the data is taken from the table and is it possible for the x and y interception at the dataset

Please ask support / questions on StackOverflow per our issue guidelines


terraform-docs 0.8.2

Hi,
I would like to suggest adding this great tool to the rig.
Thanks
Marcel

Fails on package glide. Seems to be the issue #479.

might be due to some flaky network issues last night, as https://github.com/fishworks/fish-food/pull/598 passed 6 hours ago. I'll re-run the build for now.


Sorting of work experiences is wrong

On users#show the work experiences should be sorted vice the versa. The current position should be on top.

This was fixed in51dd0aabefa280313c9330db76bf195531f79919


Animations on Mobile

Great script.
Is there anyway to disable the animations on mobile devices?  I'd rather have no animations for users with a screen width of 800px or below for example.
Thanks a lot

@attilaolah, thanks, it worked for me.

Its run slow in Iphone.! Hepl

it's not working on mobile device please help me

Try this guys It worked for me.
new WOW({mobile:false}).init();


Document that time extensions will be limited to one target

Thought of this in https://github.com/rust-lang/docs.rs/issues/1001, it seems like a good policy.
r? @pietroalbini

If we're going to have a policy, then a method of "enforcement" should probably be implemented because there's nothing stopping someone from getting an increase and then switching to a higher number of targets later on (either forgetting that they have an increase or maliciously)

Actually this can be enforced in a database constraint too, let me do that.

@Kixiron there is already a way to enforce it: update sandbox_overrides set max_targets = 1 where timeout_seconds is not null;. But that requires manual action - if I add a database constraint I can't forget to run that command.

I wouldn't put a hard constraint on the database schema. What we could do is change the limits code so that when the number of targets is not overridden, it defaults to n when the timeout is not overridden and to 1 when it is.

I think displaying to the user on the builds page that their builds are limited is important too if we aren't

I think displaying to the user on the builds page that their builds are limited is important too if we aren't

We already do: https://docs.rs/crate/haxcel/0.1.0/builds

I mean specially limited, like in the situation where we extend build time in exchange for a single target (only if we don't already)

Do you think that's necessary? We already document the effective limits build page and the policy on /about/builds. Putting in on a successful build seems like overkill.

I mean on a crate-by-crate basis, if I own the crate foo-bar-baz and I get time extended, when I look at my builds page I should see

Limit
Setting

Maximum rustdoc execution time
10 hours

Maximum number of build targets
1

That's what it says already, that's the link I gave.

The limits for that page are calculated the same way they're calculated for builds, they don't show exactly what's in the database.

@pietroalbini is this waiting on anything?

Looks good to me.


soc/dts: riscv: telink_b91: drivers: serial/pinctrl: telink_b91: boar…

…ds: riscv: tlsr9518adk80d: Introduce TLSR9518ADK80D board win retention memory layout.
The board is available during build as tlsr9518adk80d_retention.
This board supports only retention RAM - first 64KB but supports deep-sleep retention feature.
Telink board tlsr9518adk80d which uses b91 chip has big power consumption during suspend mode ~40..50uA.
But just few uA during deep-sleep mode. Deep-sleep on b91 chip produces reset
after leave this mode it makes a troubles for application developers.
So retention deep-sleep mode is implemented which behaves same as suspend
but decreases power consumption to few uA.
Signed-off-by: Andrii Bilynskyi<EMAIL_ADDRESS>Note!
plese proceed with PR https://github.com/telink-semi/hal_telink/pull/1 before.
After that west.yml should be updated with appropriate revision of hal_telink
Test project is prepared in branch retention_test. There is only one commit on top of this branch which updates blinky project.
To check build it using:
west build -p auto -b tlsr9518adk80d samples/basic/blinky - SUSPEND
west build -p auto -b tlsr9518adk80d_retention samples/basic/blinky - DEEP-SLEEP
both project print incremental counter to console.

Maybe, we have to update the board documentation with the retention info (boards\riscv\tlsr9518adk80d\doc\index.rst).


calling force-clip squad to the rescue

I guess the latest sales broke due way too long identity
plus would be nice to rework previous CSS hacks more sophisticated with clip )

identity could be found here
https://kodadot.xyz/rmrk/gallery/10812187-66241b171becfd500e-EGG-KU_EGG_371-0000000000000371

maybe fixed

yeah most probably yes, thank you!


Change-based action matrix + dynamic versions

Hatch build does set a new tag:
$ git describe --tags
rym-token-v0.1.2-11-g56c522d

However, this tag does not appear to be used:
$ cat rym/token/_version.py
# file generated by setuptools_scm
# don't change, don't track in version control
__version__ = version = '0.1.2'
__version_tuple__ = version_tuple = (0, 1, 2)

I'm at a loss.


[RNG] Device API headers and tests migration

Description
Added

includes for RNG Device API
uniformm.cpp example
moments and skip_ahead tests

Checklist
All Submissions
All unit tests have passed

[x] Do all unit tests pass locally? Attach a log.
[ ] Have you formatted the code using clang-format?

New interfaces

[x] Have you provided motivation for adding a new feature as part of RFC and
it was accepted? # (RFC)
[x] What version of oneAPI spec the interface is targeted? - 1.3
[ ] Complete New features checklist

New features

[ ] Have you provided motivation for adding a new feature? - It was added to provide RNG Device Interfaces to oneMKL open-source.
[ ] Have you added relevant tests? - An example and a few tests have been added.

Output:

@andreyfe1, @paveldyakov, please take a look


Wiring SharedMemory with it's corresponding bindings

I think this is more or less what we need to have an initial support for SharedMemory.
I'll be adding tests and examples if you think I'm heading into the right direction.

I have a "test case" with this code:
        import wasmtime

        engine = wasmtime.Engine()

        wasi_config = wasmtime.WasiConfig()
        wasi_config.inherit_stdout()

        store = wasmtime.Store(engine)
        store.set_wasi(wasi_config)

        wasm_file = 'my_wasi_program.wasm'
        with open(wasm_file, 'rb') as f:
            wasm_bytes = f.read()

        module = wasmtime.Module(engine, wasm_bytes)

        linker = wasmtime.Linker(engine)
        linker.define_wasi()

        memory_type = wasmtime.MemoryType(wasmtime.Limits(0, 4000), shared=True)
        memory = wasmtime.SharedMemory(engine, memory_type)

        linker.define(store, "env", "memory", memory)

        instance = linker.instantiate(store, module)
        ...

This is throwing an error coming from the wasmtime runtime:
ty = <wasmtime._types.MemoryType object at 0x101e30580>

    def __init__(self, engine: Engine, ty: MemoryType):
        """
        Creates a new shared memory in `store` with the given `ty`
        """

        sharedmemory = ffi.wasmtime_sharedmemory_t()
        ptr = POINTER(ffi.wasmtime_sharedmemory_t)()
        error = ffi.wasmtime_sharedmemory_new(engine.ptr(), ty.ptr(), byref(ptr))
        if error:
>           raise WasmtimeError._from_ptr(error)
E           wasmtime._error.WasmtimeError: shared memory must have the `shared` flag enabled on its memory type

which kind of make sense, because there's no support for the "shared" flag in the wasmtime_memorytype_new exported in the C-API. I also have a patch a there (will PR soon), and made some progress, but then a clone() of the shared memory type is panicking, so still trying to figure out why.

Looks good to me! I think the lack of read/write on the Rust side of things is probably an oversight and so might be worth adding here eventually, but I think it's nonetheless reasonable to start here. Before landing this'll want to have a few smoke tests too to ensure that everything is wired up right

Do you know  how can I generate the _bindings.py file out of the wasmtime C-API? I have the latter cloned, modified to add the 5th argument (bool shared) to the MemoryType, and I'm able to build an generate the .dlyb (on my mac).

It's unfortunately not easy at this time. Basically ./ci/download-wasmtime.py will place all the headers/etc into a location and what you'll want to do is overwrite/link the in-tree headers in the same location. From there you can also replace the *.dylib with what you built locally, and that way you can test locally.

Ok, I'm resuming this, now that we have all the pieces in place for the wasmtime runtime.

Sounds good! As a heads up CI here pulls from a release branch of Wasmtime but you should be able to pull it from the "dev" release which should have all the latest changes.

After pulling lastest changes from the c-api and regenerating the bindings, I realized that I need to integrate other changes not related to the memory as well (because regenerating the bindings pull all the changes, of course). This is fine, I can do it here as well, I wouldn't expect the PR to grow a lot.

Ah yeah no worries, happy to review that here too!

Feel free to hack around things here and/or split out the update to Wasmtime 27. I'm busy this week but I can more thoroughly dig in next week

I'm having a Seg. Fault after writing a test, still debugging it (been busy this past weeks!). Once I have fixed it, I'll update the branch and I think we should be ok to merge this.

It looks like mypy is reporting some type errors which might be the reason for the faults? If mypy passes it may be easier to debug perhaps

Yep, mypy has indeed captured all the problems.
I added some unit tests too. I used the same testing structure that exists around Memory, it's a good start but maybe looks insuficient. Will try to add some acceptance tests in another follow up PR, with the intention to merge this asap if you find it ready.
Thanks Alex!

Looks good to me! Could you go ahead and add the tests here? I'd like to make sure to at least exercise some basics in CI even if more advanced testing is coming later

Of course that I forgot to git add the test file xDDD
Done!

I don't have a "production" use-case for this just yet, we are investigating the potential fit for some use-cases we have, but releasing would make things a little bit easier moving forward, so it's up to you :)

Sounds good! I'll do that in https://github.com/bytecodealliance/wasmtime-py/pull/261


error：command  ‘gcc’ failed with exit status 1

Hi, When I execute "python setup.py build_ext --inplace" there is one error like follow picture, I want to know why it happened

Use the pytorch version described in the requirement.txt

thanks，it worked


Can I inquire about some training details about armo-rm？

I'm currently working on reproducing the training of NVIDIA's multi-objective architecture reward model. What are some questions about the training details of ARMO-RM? I'm using Mean Squared Error (MSE) as the loss function, and I'm normalizing the model's output and the scores of each dimension to a range of 0-1 for training. Is this approach feasible? Additionally, how should the multi-objective reward model be evaluated on the reward benchmark after training? Should I use a weighted sum of the outputs from each dimension to obtain a final score as the overall reward? Is the weight a hyperparameter that I need to set myself?

@Haoxiang-Wang hi haoxiang, could you comment on it?

Okay, thank you for your explanation.

 I’m currently working on training a multi-objective reward model using the  qwen-1.8B-sft model, and I'm utilizing two datasets: helpSteer and helpSteer2. After running the reward bench, I obtain a matrix of reward scores across different dimensions and then use a neural network to train my weight vector
x. However, the results on the reward bench are not as promising as I had hoped.
This makes me wonder if, given the relatively small size of the model, a much larger preference dataset might be necessary to train a robust reward model effectively. Relying solely on the helpSteer dataset may not be sufficient to achieve optimal performance. Based on your experience, could you please share your thoughts on this?

 I’m currently working on training a multi-objective reward model using the qwen-1.8B-sft model, and I'm utilizing two datasets: helpSteer and helpSteer2. After running the reward bench, I obtain a matrix of reward scores across different dimensions and then use a neural network to train my weight vector x. However, the results on the reward bench are not as promising as I had hoped. This makes me wonder if, given the relatively small size of the model, a much larger preference dataset might be necessary to train a robust reward model effectively. Relying solely on the helpSteer dataset may not be sufficient to achieve optimal performance. Based on your experience, could you please share your thoughts on this?

My guess is that if you only use help steer, there is no need to use a dynamic gating network. This is because the prompts are i.i.d.. The MoE aggregation trick only helps when the preference datasets are diverse.
Also, the number of samples in the help steer seems to be limited. You may want to first train a BT model as the starting point of armo training. See details in our technical report.

Agreed with @WeiXiongUST
If you are only training on HelpSteer, you can simply use a fixed gating weight, following the practice of the reward modeling in HelpSteer2: https://arxiv.org/abs/2406.08673

Code released!


fix(deps): update module golang.org/x/crypto to v0.10.0

This PR contains the following updates:

Package
Type
Update
Change

golang.org/x/crypto
require
minor
v0.9.0 -> v0.10.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.

:tada: This PR is included in version 1.10.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[5.9] Make FormRequest::validationData() public

I have action classes something like SaveCreditCourseAction($request) that will accept the form request and creates or updates a course from the form request.
I'm also overriding the validationData() method on my form requests, and i have quite a few changes that i'm making to the data.
Within the action i want to use the $this->request->validationData().
Since it's protected i cannot do that.
This PR is targeted for 5.9, to avoid breaking applications on 5.8 that are overriding the validationData() on their form requests.
A note should be added to the upgrade guide from 5.8 to 5.9 about updating the visibility of validationData() from protected to public

Many ways to handle this without dropping this method into the public space.  But 🤷🏻♂


[Relay] make simplify inference iterative

I was hitting a stack overflow in this pass in a very large model, so I converted it to MixedMode. It's a fairly simple traversal, so it was a simple change.
Thanks!
cc @ZihengJiang @siju-samuel @zhiics

plz fix ci

Thanks @mbrookhart


⚠ Bofrost.lu has degraded performance

In 3a7cc8c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 824 ms

Resolved: Bofrost.lu performance has improved in 222fcf7 after 8 minutes.


If text contains Chinese or Japanese text, the output notification becomes "garbled"

Example result with message as "虛實"

Possible solution: https://github.com/go-toast/toast/issues/18#issuecomment-532605509

how to fix it

This issue has been fixed in version 0.3.1.
Thanks to @WingDust for the PR containing the fix!
785d319a916ab03d2d110b8dafcc585b3ac9b7a8 & 4d30f3b8266d4e4184c5c15e33518fef597ff1b1


ocrd-anybaseocr-binarize handles multiple output groups not correct

ocrd-anybaseocr-binarize -m weigel/data/mets.xml -I OCR-D-IMG -O OCR-D-IMG-BIN-IMG,OCR-D-IMG-BIN -w weigel/data
create the following files:
weigel/data/OCR-D-IMG-BIN:
OCR-D-IMG-BIN_0001.png  OCR-D-IMG-BIN_0002.png  OCR-D-IMG-BIN_0003.png  OCR-D-IMG-BIN_0004.png
weigel/data/OCR-D-IMG-BIN-IMG,OCR-D-IMG-BIN:
OCR-D-IMG-BIN-IMG,OCR-D-IMG-BIN_0001.xml  OCR-D-IMG-BIN-IMG,OCR-D-IMG-BIN_0002.xml  OCR-D-IMG-BIN-IMG,OCR-D-IMG-BIN_0003.xml  OCR-D-IMG-BIN-IMG,OCR-D-IMG-BIN_0004.xml

Commit: 092183d

:+1:  Works fine for binarization and deskewing.
Please fix it also for dewarping.

Commit: 2dc4de2
Can this issue be closed now?

:+1:


Revert "Implement update operation"

Reverts homedepot/hackathon-purple-parrots#5

Codecov Report

Merging #16 into master will decrease coverage by 0.78%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
- Coverage   90.23%   89.45%   -0.79%
==========================================
  Files          20       19       -1
  Lines         256      237      -19
  Branches       18       14       -4
==========================================
- Hits          231      212      -19
  Misses         21       21
  Partials        4        4

Flag
Coverage Δ

#api
95.91% <100%> (-0.67%)
:arrow_down:

#ui
84.89% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

api/src/routes/wish.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Create copy_catalog_to_tob.sh

This script copies catalog files (no sensitive DNA info) from the bioheart bucket into TOB so that it can be used for the TOB genotyping run.

Looks fine, but should this end up in cpg-common at some point so that it can be a single source for all projects that need it?


Pull request 22145 doesn't update PyTado

https://github.com/home-assistant/home-assistant/pull/22145 does not update PyTado version.
/usr/local/lib/python3.7/site-packages/PyTado/interface.py still points to wrong API endpoint.

Hi was just about to post that Tado is still not working on 0.90.0b7 but wonder if this is the same issue.
The errors in the log are...
Error 401 on https://auth.tado.com/api/v2/homes/{home_id}/mobileDevices?username={username}&password={password}.
2019-03-20 11:37:26 ERROR (SyncWorker_8) [homeassistant.components.tado] Unable to connect to mytado with username and password
2019-03-20 11:37:26 ERROR (MainThread) [homeassistant.setup] Setup failed for tado: Component failed to initialize.
2019-03-20 11:37:28 ERROR (MainThread) [homeassistant.components.device_tracker] Error setting up platform tado

I have the same errors, but it seems to be a change on Tado’s side (something with authentication).
Changing line 115 and 140 in PyTado/interface.py from my.tado.xxxxx to auth.tado.xxxxx fixes half of the problems. Heating will work, though device tracking is still broken for me.

Still not working with 0.90 GM.

Apparantly 0.90.0 should have fixed this. However device tracking still isn’t working for me. Are there any workarounds perhaps? Btw it seems almost no one has this problem as this is the only post I could find about the device tracking part.

PLEASE do not remove our issue template.
@grinco please show me the evidence your PyTado still point to wrong URL, and define what is "wrong"

0.90 still point to the "wrong" url: "https://my.tado.com/oauth/token"
the correct one must be "https://auth.tado.com/oauth/token".

As I saw in PyTado repo, its code already changed to auth. on 0.2.8 tag
https://github.com/wmalgadey/PyTado/blob/95afdde4370c0e363b1d325b00258317a3cec8fc/PyTado/interface.py#L115
Unless they released wrong version.

Unless they released wrong version.

The issue I saw for the device tracker is it just changed api end point to auth to all API calls. Let's assume PyTado is right about API endpoint, then device tracker should not change the endpoints.
https://github.com/home-assistant/home-assistant/blob/01d8b5831ea6fd97dac1beea99844754c4520857/homeassistant/components/tado/device_tracker.py#L54-L58
Moreover, if my understanding is right by following all discussion about this integration, device tracker should 1) update to use oauth too, the username/password way need change 2) this logic should rely on the implementation of PyTado, e.g. device tracker should not define the URL in the first place.
** I do not have that device, everything I write might be wrong entirely **

Unless they released wrong version.

Then please open an issue to their repo.

I would, but I don't know how to do that :(

I download it from PyPi, and verified they indeed released wrong version.
I submitted one issue in their repo. https://github.com/wmalgadey/PyTado/issues/13

Ok, I have changed the following lines back from https://auth.tado.com/ to https://my.tado.com.

home-assistant/homeassistant/components/tado/device_tracker.py
Lines 54 to 58 in 01d8b58

Seems to be working again. So what I understand from all this is that the only thing that should have been changed are line 115 and 140 in the ~/site-packages/PyTado/interface.py but not these ones.
I'd love to make a pull request but I really wouldn't know how :(, will test it some more now and figure out how to make a pull request :).

So I have done a bit of testing, seems the device tracker is working again and all my automations that depended on it work again as well. So it seems that everything is working again e.g. operation modes and device tracking. However I found another error which it will throw at you if you try to set the operation mode to "off" or "timer" it will throw the following error:
`HTTP Error 422:
Traceback (most recent call last):
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/components/websocket_api/commands.py", line 122, in handle_call_service
connection.context(msg))
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/core.py", line 1138, in async_call
self._execute_service(handler, service_call))
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/core.py", line 1160, in _execute_service
await handler.func(service_call)
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/helpers/entity_component.py", line 188, in handle_service
self._platforms.values(), func, call, service_name
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/helpers/service.py", line 314, in entity_service_call
future.result()  # pop exception if have
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/helpers/service.py", line 328, in _handle_service_platform_call
await getattr(entity, func)(**data)
File "/usr/lib/python3.6/concurrent/futures/thread.py", line 56, in run
result = self.fn(*self.args, **self.kwargs)
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/components/tado/climate.py", line 225, in set_operation_mode
self._control_heating()
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/components/tado/climate.py", line 343, in _control_heating
self._store.set_zone_overlay(self.zone_id, CONST_OVERLAY_MANUAL)
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/homeassistant/components/tado/init.py", line 122, in set_zone_overlay
self.tado.setZoneOverlay(zone_id, mode, temperature, duration)
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/PyTado/interface.py", line 287, in setZoneOverlay
data = self._apiCall(cmd, "PUT", post_data)
File "/home/homeassistant/homeassistant/lib/python3.6/site-packages/PyTado/interface.py", line 81, in _apiCall
response = self.opener.open(req)
File "/usr/lib/python3.6/urllib/request.py", line 532, in open
response = meth(req, response)
File "/usr/lib/python3.6/urllib/request.py", line 642, in http_response
'http', request, response, code, msg, hdrs)
File "/usr/lib/python3.6/urllib/request.py", line 570, in error
return self._call_chain(*args)
File "/usr/lib/python3.6/urllib/request.py", line 504, in _call_chain
result = func(*args)
File "/usr/lib/python3.6/urllib/request.py", line 650, in http_error_default
raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 422:
`

Thanks @awarecan for this link. The thing is I'm a hassio user and I don't think that I can change the file.

The issue is that PyTado isnt maintained.
The repo with the fix is named Python-tado on pypi
https://pypi.org/project/python-tado/

So PyTado is not used anymore. Python-Tado is what is being used now.
so @grinco no it does not update PyTado. It replaces PyTado with Python-Tado
and @jimbob1001  That seem to be another issue. The auth.tado.com adress should work.

So should/can we hassio users do. Just sit there and wait for a fix in the next release?

@bhaap we are using python-tado

I updated python-tado (aka PyTado) to 0.2.9 which finally has the right changes.
PS: PyTado was already taken on pyPi, and I couldn't get in contact with the maintainer!

What we can do as a workaround while the fix comes to homeassistant?

@rafuz you can change the file manually. It will stay there as long as you don't install it again.
I guess the fix will also be backported to a 0.90.1


possible size conversion issue

Hi,
using LFS 1.0.0. When I compare this output to that from df and df -h the output from lfs seems wrong. total size is not 13G and the used from df is 8.5G but 9.1G on LFS.
┌──┬─────┬──────────────┬────┬────┬────┬─────────┬─────┬────┬──────────────────┐
│id│ dev │  filesystem  │disk│type│used│  use%   │avail│size│mount point       │
├──┼─────┼──────────────┼────┼────┼────┼─────────┼─────┼────┼──────────────────┤
│38│259:8│/dev/nvme0n1p1│SSD │xfs │9.1G│71% ███▌ │ 3.8G│13G │/                 │
│43│259:4│/dev/nvme5n1  │SSD │xfs │127M│ 1%      │  12G│12G │/var/local1       │
│44│259:6│/dev/nvme7n1  │SSD │xfs │106M│ 1%      │  11G│11G │/var/store1       │
│46│259:0│/dev/nvme1n1  │SSD │xfs │7.4G│77% ███▉ │ 2.3G│9.7G│/usr/local/Store  │
│45│259:1│/dev/nvme2n1  │SSD │xfs │518M│16% ▊    │ 2.7G│3.2G│/var/log          │
│47│259:5│/dev/nvme6n1  │SSD │xfs │306M│14% ▊    │ 1.8G│2.1G│/tmp              │
│42│259:2│/dev/nvme4n1  │SSD │xfs │ 36M│ 3% ▏    │ 1.0G│1.1G│/home             │
└──┴─────┴──────────────┴────┴────┴────┴─────────┴─────┴────┴──────────────────┘

→ df | grep /$
/dev/nvme0n1p1  12570604 8865848   3704756  71% /

→ df -h | grep /$
/dev/nvme0n1p1   12G  8.5G  3.6G  71% /

Filesystem      Size  Used Avail Use% Mounted on
devtmpfs        7.8G     0  7.8G   0% /dev
tmpfs           7.8G   12K  7.8G   1% /dev/shm
tmpfs           7.8G  900K  7.8G   1% /run
tmpfs           7.8G     0  7.8G   0% /sys/fs/cgroup
/dev/nvme0n1p1   12G  8.5G  3.6G  71% /
/dev/nvme1n1    9.0G  6.9G  2.2G  77% /usr/local/Store
/dev/nvme7n1     10G  101M  9.9G   1% /var/store1
/dev/nvme5n1     11G  122M   11G   2% /var/local1
/dev/nvme4n1   1014M   35M  980M   4% /home
/dev/nvme6n1    2.0G  292M  1.8G  15% /tmp
/dev/nvme2n1    3.0G  489M  2.6G  16% /var/log
tmpfs           1.6G     0  1.6G   0% /run/user/0
tmpfs           1.6G     0  1.6G   0% /run/user/1002

To compare with lfs, don't use df -h (which is the old 1024 based format) but df -H which uses the ISO 1000 based format.

AWS EC2 shows their sizes as GiB. Then to use the json output I have to do conversions in any case. Will it be possible to add a switch to print out the sizes differently as an option please?

Will it be possible to add a switch to print out the sizes differently as an option please?

You mean when you output as JSON ?

In JSON I could even output the two values, there's no real cost.

Can the switch be available for regular use and json output?
I had a look at df again. I've always used df -h. Interesting that df -h and df -H both prints G or M without distinction. I'm using EC2 and ideally my sizes from EC2 should match the sizes from the host without needing to convert in between.
I had a search on their mailing list and found this .
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=47023#16
Found this statements from LA Walsh
Side rant:
 Using decimal prefixes with a binary unit (1B=2**3 bits)
 defeats the purpose of using a common multiplier for metric.
 Since computers use base-2, similar prefixes should be used.
 Just because the disk-industry bought and paid for the
 ruling to use base-10 doesn't mean that memory comes in
 units of 1-million, 1-billion or 1-trillion bytes or
 that disk space is organized in decimal units.

I mailed the group and asked if they could print DB and GiB accordingly.

I won't personally go and have a fight with the old guys still arguing that base-2 makes sense for units...
If you want to avoid ambiguity when reading the JSON, just multiply blocks by bsize, the result will be in bytes


style_links.js not updated after removing some projects

Issue Type:

[x] Bug Report

Describe the bug

After commit #720 for solving the issue #718 some projects were removed but the style_links.js was not updated hence it yet  appear on the site
Alarm_clock, Pokemon generator,Candy Crush Clone, yet appears on search its index.html being deleted

Possible solution

Remove the links from style_links.js of all those removed projects
or replace the projects from non-plagiarised projects with similar concepts(I am ready to take this up too!)

Screenshot

@arpit456jain assign it to me I will do it


Add the ability to easily format the text included in the AfterInstallMessage and highlight the fragments

Large text without $C(13,10) becomes unreadable

I'd expect that you could put in a newline with XML entities e.g.,


[hotfix][doc] Correct code sample issues in Create a TableEnvironment…

… chapter in table/common.md
What is the purpose of the change
Correct code sample issues in Create a TableEnvironment chapter in table/common.md
Brief change log

docs/content/docs/dev/table/common.md

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

@paul8263 , IMO, you could change the Chinese docs together for this change.

@flinkbot run azure


Missing arrow in tooltip.

Hi Nicolas.
I'm not sure if this is the place to say this but I wanted to say thanks very much for taking the time to create these awesome accessible plugins.
I am having some problems however with the 'jquery-accessible-simple-tooltip-aria' plugin.
I copied and pasted your code as you included on the http://a11y.nicolas-hoffmann.net/simple-tooltip/ webpage but for some reason in Firefox v47.0, Chrome v51.0 and Internet Explorer 11.0 I see the black tooltip boxes but the arrow is not aligned properly and appears inside the tooltip, slightly covering the first letter appearing in the tooltop box.
It seems to appear fine in your samples on your actual http://a11y.nicolas-hoffmann.net/simple-tooltip/ webpage so perhaps another style on that page is making them look as intended.
I am including a link to Codepen where I pasted your examples for you to check out. The only modifications I made to the code was spacing and line breaks.
http://codepen.io/c0brarules/pen/BzpMqW
Could you please let me know what changes I need to make to have the arrow appear as it does on your actual sample page?
Thanks again.
Dwayne

Hi @c0brarules,
thanks for your message. ;)
It took me some time to understand where the problem was, as you've done perfectly! :)
In fact, it is my mistake: it comes from a piece of code I have included in the CSS for the demo, but not in the documentation. It is the box sizing, try adding this in the demo:
html { box-sizing: border-box; }
*, *:before, *:after {
  box-sizing: inherit;
}

And everything will be displayed fine. I will update the documentation to avoid this issue again.
Thanks,
Nicolas

Hi Nicolas.
Wow, thanks very much for the quick response. I appreciate that.
I'm glad I could help and yes, that did the trick.
:-)
I do have another question for you though if you don't mind.
Is there a way to make it so that the Tooltip will
automatically/intelligently detect the four edges of the browser window so
that the Tooltip will switch from right/left/bottom/top based on how close
it appears to the edge/viewport of a webpage? That way it is always
displaying itself in full view of the user. I would still want to specify a
default, but it would be overwritten based on distance to edge/viewport of
the browser window.
In other words, if I have a long sentence that goes all the way from the
left side of the browser window to the right side of a browser window it
would be nice if the Tooltip would move from it's default right side of the
sentence to either below or above it. Otherwise what happens is it gets
hidden and can't be scrolled to see the entire Tooltip.
I am including a screenshot for you as well as the CodePen example using
your default code examples.
http://codepen.io/c0brarules/pen/BzpMqW

I know you supplied in your example the media-queries for tablets/mobile
(thanks by the way) for the Tooltip to switch below the text rather than to
the right of it however I want this to occur regardless of the screen size.
I know that may be a lot to ask for and if you don't have time to show me
what would need to be changed, could you please at least show me what to
change in the CSS to make the tooltip always appear above the hovered text,
rather than to the right of it?
Thanks again for your time.

c0brarules

On Mon, Jun 27, 2016 at 3:47 AM, Nicolas Hoffmann<EMAIL_ADDRESS>wrote:

Hi @c0brarules https://github.com/c0brarules,
thanks for your message. ;)
It took me some time to understand where the problem was, as you've done
perfectly! :)
In fact, it is my mistake: it comes from a piece of code I have included
in the CSS for the demo, but not in the documentation. It is the box
sizing, try adding this in the demo:
html { box-sizing: border-box; }
*, *:before, *:after {
box-sizing: inherit;
}
And everything will be displayed fine. I will update the documentation to
avoid this issue again.
Thanks,
Nicolas
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nico3333fr/jquery-accessible-simple-tooltip-aria/issues/5#issuecomment-228675639,
or mute the thread
https://github.com/notifications/unsubscribe/AMQbknZo3qIi3o7lFAAh5yjr_MbZv13Bks5qP4AEgaJpZM4I-rxR
.

Hi Nicolas.

Hi again !

Wow, thanks very much for the quick response. I appreciate that.
I'm glad I could help and yes, that did the trick.
:-)

Cool :)
Sorry for the delay for this answer, very busy :-\

I do have another question for you though if you don't mind.

Is there a way to make it so that the Tooltip will automatically/intelligently detect the four edges of the browser window so that the Tooltip will switch from right/left/bottom/top based on how close
it appears to the edge/viewport of a webpage? That way it is always displaying itself in full view of the user. I would still want to specify a default, but it would be overwritten based on distance to edge/viewport of the browser window.
In other words, if I have a long sentence that goes all the way from the left side of the browser window to the right side of a browser window it would be nice if the Tooltip would move from it's default right side of the sentence to either below or above it. Otherwise what happens is it gets
hidden and can't be scrolled to see the entire Tooltip.

I've already thought to this feature, but I never had the time to implement it. My tooltip is for rather "controlled" environment. I hope to add it in a not-too-far future :)
> I know you supplied in your example the media-queries for tablets/mobile
(thanks by the way) for the Tooltip to switch below the text rather than to
the right of it however I want this to occur regardless of the screen size.

I know that may be a lot to ask for and if you don't have time to show me
what would need to be changed, could you please at least show me what to
change in the CSS to make the tooltip always appear above the hovered text,
rather than to the right of it?

It is a pure CSS issue :) so I've made an example here : http://codepen.io/nico3333fr/pen/RRpvrd

Thanks again for your time.

You're welcomed.

Hi Nicolas.
I can believe you are quite busy and that is why it means even more to me
that you were willing to take the time to help me out.
I work at the University of Guelph in Ontario Canada and I help build
courses for students taking courses online. I really appreciate the time
you have taken to help me, and for all the accessible plug-ins you have
created that will be such a big help for the students with disabilities
that will be able to benefit from them when I implement them into the
courses.
I used the CSS you modified for me and from that I was able to create an
additional style that allows me to specify that the Tool-tip appears below
the hovered text. So now I can select above or below the hovered text as
required which should get me by until you have the time to implement the
new feature of having them appear automatically as needed.
:-)
Thanks again and have a great day!

Dwayne

On Wed, Jun 29, 2016 at 9:54 AM, Nicolas Hoffmann<EMAIL_ADDRESS>wrote:

Hi Nicolas.
Hi again !
Wow, thanks very much for the quick response. I appreciate that.
I'm glad I could help and yes, that did the trick.
:-)
Cool :)
Sorry for the delay for this answer, very busy :-\
I do have another question for you though if you don't mind.
Is there a way to make it so that the Tooltip will
automatically/intelligently detect the four edges of the browser window so
that the Tooltip will switch from right/left/bottom/top based on how close
it appears to the edge/viewport of a webpage? That way it is always
displaying itself in full view of the user. I would still want to specify a
default, but it would be overwritten based on distance to edge/viewport of
the browser window.
In other words, if I have a long sentence that goes all the way from the
left side of the browser window to the right side of a browser window it
would be nice if the Tooltip would move from it's default right side of the
sentence to either below or above it. Otherwise what happens is it gets
hidden and can't be scrolled to see the entire Tooltip.
I've already thought to this feature, but I never had the time to
implement it. My tooltip is for rather "controlled" environment. I hope to
add it in a not-too-far future :)
> I know you supplied in your example the media-queries for tablets/mobile
(thanks by the way) for the Tooltip to switch below the text rather than to
the right of it however I want this to occur regardless of the screen size.
I know that may be a lot to ask for and if you don't have time to show me
what would need to be changed, could you please at least show me what to
change in the CSS to make the tooltip always appear above the hovered text,
rather than to the right of it?
It is a pure CSS issue :) so I've made an example here :
http://codepen.io/nico3333fr/pen/RRpvrd
Thanks again for your time.
You're welcomed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nico3333fr/jquery-accessible-simple-tooltip-aria/issues/5#issuecomment-229362849,
or mute the thread
https://github.com/notifications/unsubscribe/AMQbks4eBvftSXNzz7uKEGa1qQiRvfqCks5qQnkmgaJpZM4I-rxR
.


Update helm chart to align with k/charts feedback

What this PR does / why we need it:
This PR brings the helm chart in line with what is found in kubernetes/charts (changes contained here: https://github.com/kubernetes/charts/pull/3416/commits/008d52cbc4c2b7d2b4981f53214e15779e085fb7).
It also addresses some feedback (https://github.com/kubernetes/charts/issues/3513) in order to improve our post-installation documentation by providing links to how to configure issuers/ingress-shim.
Release note:
Remove default resource requests in Helm chart. Improve post-deployment informational messages.

/retest

Build infrastructure problems after changes over the weekend :tada:
/retest

/retest

/retest

/retest

/retest


Fix backslash bug

This PR aims to fix #11. I've also added tests, but still I'm not 100% sure that it won't break anything. I tried to make sure that the replacement phase titled "unescape double-quotes" does not mess with the escaped backslashes that are followed with a double quote.
Testing this with inline strings was so difficult for me, so created a folder named "tests" and put all the relevant files into that folder.

I'm not totally confident in this change, but it's passing all current tests, so I'll merge. Thanks @monoblaine :)
This fix should probably be applied to https://github.com/sindresorhus/to-double-quotes too.

Would be really nice if someone could look into https://github.com/sindresorhus/to-single-quotes/issues/13.


Swagger CSS not loading properly.

Hi guys,
When I enter "swagger project edit" command, style problem occurs on my project.

It should be like this.

How can I solve this?
Thanks.

This is an issue related to swagger-node, not swagger-core. At the moment, it's not updated to the latest version of the Editor. Best solution is to use the editor externally.


Confusing term "isolation class" in APF doc

This is a Bug Report

Problem:
Someone reading https://kubernetes.io/docs/concepts/cluster-administration/flow-control/# was not able to discern the intended meaning of the term "isolation class".  See https://kubernetes.slack.com/archives/C01G8RY7XD3/p1661569471734649 .
This term is only used twice, and is used in a way that is synonymous with another term already introduced ("priority level").  I think it would be clearer to replace "isolation class" with "priority level".
Proposed Solution:
Replace "isolation class" with "priority level".
Page to Update:
https://kubernetes.io/docs/concepts/cluster-administration/flow-control/#

This has been there for a while.

/assign

/triage accepted

This is what I originally posted in the #api-priority-and-fairness Slack channel. Thanks for filing this on GitHub, Mike.


Must use import to load ES Module in Nextjs

I'm trying to use livekit-react in nextjs and throws me this error:
Server Error
Error: Must use import to load ES Module: C:\projectFolder\node_modules\livekit-client\dist\livekit-client.umd.js
require() of ES modules is not supported.
require() of C:\projectFolder\node_modules\livekit-client\dist\livekit-client.umd.js from C:\projectFolder\node_modules\livekit-react\dist\index.js is an ES module file as it is a .js file whose nearest parent package.json contains "type": "module" which defines all .js files in that package scope as ES modules.
Instead rename livekit-client.umd.js to end in .cjs, change the requiring code to use import(), or remove "type": "module" from C:\projectFolder\node_modules\livekit-client\package.json.

And when i removed "type": "module" in the package.json of livekit-client, it was able to run
"livekit-client": "^0.18.3",
"livekit-react": "^0.7.1",
"next": "^12.1.5",
To reproduce:

Create next app
Use basic template of livekit-react for connecting to a room
Go to the page and see error

hi @Dolir,
thanks for the report. we are currently ironing out the last bits to make the client sdk work across all frameworks. the next livekit-client update will have a fix for it.

@Dolir could you try with version 0.18.4-RC5 and see if it fixes your issues?

Yes, now it works


Running in parallel

📚 Description
Closes: https://github.com/TypedDevs/bashunit/issues/354
🔖 Changes

Add parallel running: -p|--parallel
Add possibility of disabling parallel execution with --no-parallel

✅ To-do list

[x] I updated the CHANGELOG.md to reflect the new feature or fix
[x] I updated the documentation to reflect the changes

🖼 Screenshots
Running the entire test suite sync (~43s)

Running the entire test suite in parallel (~15s) - 65.12% improment! 🚀

Running parallel tests verbose

Documentation

The tests are not good in alpine

I would rather not keep the parallel testing as default behavior but as it was until now.
This is the default behavior on PHPUnit, Pest, JUnit, among others. However I found that Jest run them in parallel "when it make sense" based on some heuristic -size of tests, number of tests, etc... but I find this too complicated to get into this right now.
Aiming for simplicity, I am in favor of:
If you want parallel tests you can enable with -p|--parallel or in your .env BASHUNIT_PARALLEL_RUN=true


Upgrade to latest nightly and remove workaround for not picking up changes.

This PR upgrades to the latest nightly wrapper and removes the workaround introduced by https://github.com/gradle/gradle/pull/20366, since the problem has been fixed in Gradle.

Your PR is queued. See the queue page for details.


PPTP-868 : Generic retry functionality available via Retry trait. Non…

… Repudiation Service submission call will be retried (as per configuration) on 500 errors.

Quick commit review.
There seems to be a lot of detail on the commit subject.
My suggestion is to use the commit body for implementation details or explain why, how and what.
PPTP-868 Generic retry functionality available via Retry trait.

Non Repudiation Service submission call will be retried (as per configuration) on 500 errors.

We should loosely follow https://chris.beams.io/posts/git-commit/#seven-rules

Quick commit review.
There seems to be a lot of detail on the commit subject.
My suggestion is to use the commit body for implementation details or explain why, how and what.
PPTP-868 Generic retry functionality available via Retry trait.

Non Repudiation Service submission call will be retried (as per configuration) on 500 errors.

We should loosely follow https://chris.beams.io/posts/git-commit/#seven-rules
how to do this on the command line:
git commit --amend

Apologies @PauloPortugal; I remember you specifically asking for this.


[docs] update plugin docs

Description:

Refactors and updates the plugin README.
Moves part of the README to a new how-to for creating new plugin apps.

Issue:
https://github.com/openedx/edx-django-utils/issues/276
Merge checklist:

[ ] All reviewers approved
[ ] CI build is green
[ ] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:
No new version needed. Just doc changes.

Note: I tabbed-in a big section that is meant to move to the how-to, so either move the code or view the diff without whitespace changes to see the recommended changes. This is just to document proposed changes for https://github.com/openedx/edx-django-utils/issues/276, so I'm closing this for now. Feel free to reopen or start a fresh copy off this branch.

A few bits of cleanup, but otherwise looks good.

The :doc: syntax didn't seem to work, but text <path/to/doc>_ did

The tricky bit is that the .rst links only work on GitHub, whereas the :doc: links only work on Read The Docs (or if you use make docs). I think we should prefer the latter.


Hiding literature evidence from data download section

Describe the bug
There are 4 chips being displayed in the Target - Disease evidence data download instead of 2.
These are different download options:
One set is for the evidence 23.06/output/etl/parquet/evidence and
the other set is for the literature evidence 23.06/output/etl/parquet/literature/evidence
We want to hide the the 2 chips containing the literature evidence
Expected behaviour
Only 2 chips pointing to parquet and json evidence files should be displayed. Need to change a boolean in the config file that displays these.

I don't know how we are handling the nested literature datasets, but we want to say "include_in_fe": false, in the next dataset
https://github.com/opentargets/ot-ui-apps/blob/c2b9c4ba31eb0eee053b9e53c283b7a3858cb8e7/apps/platform/src/pages/DownloadsPage/dataset-mappings.json#L4

just a thought: instead of hiding literature evidence, can we add another row on download page:

target-disease evidence (lIterature) with the two relevant json and parquet files?

Or is the plan not to provide these literature evidence files to users at all?

The Target-Disease evidence row contains all evidence including literature. By including another row we are artificially duplicating the data.

@ehsanb29 is looking into this. He is also going to work on the order of chips - json and parquetfor all rows

@chinmehta working on the order of the chips

Works correctly in the latest dev with literature evidence not present and ordered chips.


Handle both string and Color as color definitions

Fixes #3011

I think this is ok. @simonbrunel thoughts?

Having all these conversions to color, then to string (in controllers), then to color again and finally to string (in elements) is not good at all for performance. Duplicating that .rgbaString() is not good for compression, neither for maintenance. I would rather add a color helper to make sure that the behavior is consistent and avoid code duplication, i.e. helper.normColor() returning the correct rgba string. If the issue is that we want a string color at the end, we should avoid converting it to color if it's a string?!
I would not normalize color twice, in the controllers and in the elements.  Ideally we should normalize colors only once and only user inputs and consider everywhere else rgba values.
Anyway, I don't think that the solution is to make all these conversions, so not sure about merging that PR.

I agree on performance, but if elements require colors to be in strings, it'll need to be specified in the docs, IMO. The way it is here, there's the advantage of working well with user created controllers (although this argument is debatable).
An alternative solution is to fix Color -  new Color(colorA) === Color(colorA) === colorA is really the problem here.

Do we really need/want to allow Color object in Chart options / data? I don't think we initially embedded the color lib to expose it to the user but to make easier color manipulation internally (transition, hover style, etc.). So another solution would be to make that lib private and only accept CSS colors (as strings) from user inputs. Not really a deprecation if it currently doesn't work with Color object. That may be my favorite alternative! @etimberg, @tannerlinsley,  @zachpanz88 , what do you think?

My 2 cents: making the lib private isn't really necessary (it can be quite useful to whoever looks hard enough :) )
Removing Color as a 'format' in the docs is a nice solution.

I'm ok with making the color dependency private. I think the reason that it doesn't work is due to the transition function that does the animation. There is no special case for color objects in the models. See https://github.com/chartjs/Chart.js/blob/master/src/core/core.element.js#L29

I guess transition is called only for animation, so in case of no animation, it will not work better to have a special case here. Anyway, I don't see the benefit to allow Color objects in options/data, if users want to manipulate colors, they can still use that lib and call to*String() before sending the value to the chart.
Should we remove Color as accepted format from the documentation and consider (in controllers and elements) that any color values are strings?

Lets remove it from the docs then. Maybe we can add a note that colors should be strings (though CanvasPattern also works)


Kubernetes v1.8.4 support

https://github.com/kubernetes/kubernetes/releases/tag/v1.8.4

@jackfrancis looks like we missed 1.8.3, should I add it too while I'm at it?

Also 1.7.11 just got released

Since we missed the window when v1.8.3 was latest 1.8, we don't have to retroactively add it in acs-engine.


QEMU: include more test runs for all bar one cross compilation target

For the cross compiles where the tests couldn't be run, most are capable of being run when statically linked.  For these, a shared with FIPS build but not test run is also included to maximise compilation coverage.  The builds take a couple of minutes so the impact of these extra jobs isn't great.

[ ] documentation is added or updated
[x] tests are added or updated

I suppose we want to merge  #16133 first and then rebase this one on top?

I can do that.


support ssh git urls

So far the bot has been tested against public https repos or private repos using https and an oauth token. We should add support for private repos using ssh deploy keys that can be git-secret encrypted into a config repo to be deployed via ocd-environment-webhook.

This is done it just needs documenting


Installing Docker on PC

In the system setup documentation, under Installing the Docker, can I install Docker on a PC instead of on an Nvidia Jetson board?
can I install Docker on a PC instead of on an Nvidia Jetson board?
thanks!

Hi, yes, you can. Clone the repository and open it with your VSCode. Assuming docker Dev Containers extension is installed (in addition to docker engine and NVIDIA container toolkits installed system-wide), you should be able to simply press 'ctrl+shift+p' and run the Rebuild and Reopen in Container command. It will build the docker image and attach your vscode environment into it. Please let me know if it works for you here.

Hi, yes, you can. Clone the repository and open it with your VSCode. Assuming docker Dev Containers extension is installed (in addition to docker engine and NVIDIA container toolkits installed system-wide), you should be able to simply press 'ctrl+shift+p' and run the Rebuild and Reopen in Container command. It will build the docker image and attach your vscode environment into it. Please let me know if it works for you here.

I've followd your instructions. When I run the Rebuild and Reopen in Container command, I am asked for a template.
How should I configure it?
By the way, I prefer to run the docker on Pycharm. Are the instructions different?
Thanks!


Double-check if override-client-name is correctly supported

Maybe we can close the issue right now, but want to double check first :)

Called with --override-client-name=FooOverride and didn't find it in the codemodel.

Whoa, I didn't even know that was there.  Definitely a bug.


Bump Spark version to 3.2

Putting this PR on hold until EMR supports Spark 3.2

EMR 6.5.0 was released on January 20 but it only supports Spark 3.1.

EMR 6.6.0 with Spark 3.2.0 support is out.
https://docs.aws.amazon.com/emr/latest/ReleaseGuide/emr-spark.html


option to display location

These are incredibly captivating, but I sometimes find myself frustrated wondering where in the world I'm looking at.  Would help reduce my stress if there was an option to display that information (in the lower left corner or some such out of the way place) while the screen saver was running.

Whoops - dupe of #58


Summary is not generated

Describe the Issue
When running an Audit, the summary file is not created. I'm pretty sure this happens during a remediation as well.
Expected Behavior
A Summary file(s) is created and saved on the target machine.
Actual Behavior
All of the summary steps are skipped:
TASK [/home/jenkins/workspace/peline_VZWAN-55977-add-cis-audit/audit/RHEL7-CIS : Post Audit | ensure audit files readable by users] ***
 skipping: [<IP_ADDRESS>] => (item=/opt/WELKTX08LAB-mnbbkhb-4140-RHEL7-CIS_post_scan_1706303543.json)
 skipping: [<IP_ADDRESS>] => (item=/opt/WELKTX08LAB-mnbbkhb-4140-RHEL7-CIS_pre_scan_1706303543.json)
 skipping: [<IP_ADDRESS>]

 TASK [/home/jenkins/workspace/peline_VZWAN-55977-add-cis-audit/audit/RHEL7-CIS : capture data /opt/WELKTX08LAB-mnbbkhb-4140-RHEL7-CIS_post_scan_1706303543.json] ***
 skipping: [<IP_ADDRESS>]

 TASK [/home/jenkins/workspace/peline_VZWAN-55977-add-cis-audit/audit/RHEL7-CIS : Capture post-audit result] ***
 skipping: [<IP_ADDRESS>]

 TASK [/home/jenkins/workspace/peline_VZWAN-55977-add-cis-audit/audit/RHEL7-CIS : Post Audit | capture data /opt/WELKTX08LAB-mnbbkhb-4140-RHEL7-CIS_post_scan_1706303543.json] ***
 skipping: [<IP_ADDRESS>]

 TASK [/home/jenkins/workspace/peline_VZWAN-55977-add-cis-audit/audit/RHEL7-CIS : Post Audit | Capture post-audit result] ***
 skipping: [<IP_ADDRESS>]

 TASK [/home/jenkins/workspace/peline_VZWAN-55977-add-cis-audit/audit/RHEL7-CIS : Show Audit Summary] ***
 skipping: [<IP_ADDRESS>]

 PLAY RECAP *********************************************************************
 <IP_ADDRESS>             : ok=17   changed=0    unreachable=0    failed=0    skipped=35   rescued=0    ignored=0

Control(s) Affected
Audits and Summary
Environment (please complete the following information):

branch being used: devel

@uk-bolly I would really appreciate your input on this! Thanks!

hi @mcascone
Apologies for the delay, could you let me know what command you are using to run the playbook, as it appears the audit is not set to run at all.
e.g.
-e "{'run_audit': true }" -e "{'setup_audit': true }"

Kind regards
uk-bolly

hi @mcascone
We have cut a new release in the main branch and the devel is updated. This works through as expected.
https://github.com/ansible-lockdown/RHEL7-CIS/releases/tag/1.3.0
I will therefore close this issue, please feel free to feedback if you are still seeing problems.
many thanks
uk-bolly


Logitech G305 not recognized due to missing ID and error

Information

ratbagd version (ratbagd --version): 0.12
Distribution: Manjaro
Kernel version (ex. uname -srmo): Linux 5.3.18-1-MANJARO x86_64 GNU/Linux
Piper 0.3

Describe the bug
A recently purchased G305 is not recognized by libratbag and consequently does not show up in piper.
To Reproduce
Steps to reproduce the behavior:

launch libratbag
launch piper
look at the oops screen

Logs
$ sudo ratbagd --verbose=raw
Initializing libratbag
ratbag debug: New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: No data file found for 046d:c53f
ratbag debug: New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: No data file found for 046d:c53f
ratbag debug: New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: No data file found for 046d:c53f
DBus server ready

I went ahead and added the mentioned ID, this is the output:
$ sudo ratbagd --verbose=raw
Initializing libratbag
ratbag debug: New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: device assigned driver hidpp20
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw0'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw1'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: - HID report ID 02
ratbag debug: - HID report ID 03
ratbag debug: - HID report ID 04
ratbag debug: - HID report ID 08
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw2'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: - HID report ID 10
ratbag debug: - HID report ID 11
ratbag debug: - HID report ID 20
ratbag debug: - HID report ID 21
ratbag debug: hidpp: device supports short reports
ratbag debug: hidpp: device supports long reports
ratbag raw: hidpp write:  11 ff 00 18 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
ratbag raw: hidpp read:   10 ff 8f 00 18 01 00
ratbag debug:     HID++ error from the device (255): ERR_UNKNOWN (01)
ratbag debug: New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: device assigned driver hidpp20
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw0'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw1'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: - HID report ID 02
ratbag debug: - HID report ID 03
ratbag debug: - HID report ID 04
ratbag debug: - HID report ID 08
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw2'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: - HID report ID 10
ratbag debug: - HID report ID 11
ratbag debug: - HID report ID 20
ratbag debug: - HID report ID 21
ratbag debug: hidpp: device supports short reports
ratbag debug: hidpp: device supports long reports
ratbag raw: hidpp write:  11 ff 00 18 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
ratbag raw: hidpp read:   10 ff 8f 00 18 01 00
ratbag debug:     HID++ error from the device (255): ERR_UNKNOWN (01)
ratbag debug: New device: Logitech USB Receiver
ratbag debug: Using data directory '/usr/share/libratbag'
ratbag debug: device assigned driver hidpp20
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw0'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw1'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: - HID report ID 02
ratbag debug: - HID report ID 03
ratbag debug: - HID report ID 04
ratbag debug: - HID report ID 08
ratbag debug: hidraw info: bus 0x03 vendor 0x46d product 0xffffc53f
ratbag debug: Logitech USB Receiver is device '/dev/hidraw2'.
ratbag debug: Parsing HID report descriptor
ratbag debug: Parsing HID report descriptor
ratbag debug: - HID report ID 10
ratbag debug: - HID report ID 11
ratbag debug: - HID report ID 20
ratbag debug: - HID report ID 21
ratbag debug: hidpp: device supports short reports
ratbag debug: hidpp: device supports long reports
ratbag raw: hidpp write:  11 ff 00 18 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
ratbag raw: hidpp read:   10 ff 8f 00 18 01 00
ratbag debug:     HID++ error from the device (255): ERR_UNKNOWN (01)
DBus server ready

Piper still shows nothing and I don't know how to debug this error.

You need Linux 5.4.


Add get capital config request

The /sapi/v1/capital/config/getall route return the wallet status for all coins Binance, useful to know the withdrawal status on different networks per example.

Hello. Sorry, not very active anymore with my projects.


prefer accessing ENV directly from code

By accessing your secrets and passing on the command-line, there is a small chance they can leak in a shared virtual machine. See https://docs.github.com/en/actions/security-guides/encrypted-secrets#using-encrypted-secrets-in-a-workflow

Thanks! I inadvertently fixed this, when I broke my ankle and rewrote this in Rust for fun. But! Hopefully you're good with your fork as well.


Removes Incremental Accounts Hash featurization

🚧 Since this impacts the ability to open snapshots created before IAH was enabled, we'll wait a while before merging 🚧
Problem
The Incremental Accounts Hash feature has been enabled on all clusters. Its featurization code can now be removed.
Summary of Changes
Remove the featurization code.
Fixes #30819
Originally from https://github.com/solana-labs/solana/pull/34957

Codecov Report
Attention: Patch coverage is 95.83333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 81.6%. Comparing base (186befe) to head (d8b4f70).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##           master      #88     +/-   ##
=========================================
- Coverage    81.6%    81.6%   -0.1%
=========================================
  Files         867      867
  Lines      368767   368828     +61
=========================================
+ Hits       300967   300986     +19
- Misses      67800    67842     +42


How to fail with an NSError?

An URLSession data task can fail with an error according to:
open func dataTask(with request: URLRequest, completionHandler: @escaping (Data?, URLResponse?, Error?) -> Swift.Void) -> URLSessionDataTask

Is there any way to fail with an NSError at the moment? I know that Nocilla for example is capable of failing like this.
If not i'd like to rise the proposal:
server["some/path"] = { _ in
    return HttpResponse.fail(NSError.notConnected)
}

This is an example where the OS is throwing a notConnected in case the customer does not have an internet connection. (Used in testing)
I'd like to see this in Swifter too.

My fault got that Nocilla is doing something completely different.


support for multithreading

Is your feature request related to a problem? Please describe.
it would be nice if the documentation could be improved for the multi-threading. The two examples in the documentation do not seem to run and the appear to have some confusing logic.
Describe the solution you'd like
Update or removal of the multithreading examples in the documentation
Describe alternatives you've considered
None
Additional context
I wanted to solve the same problem over and over again with different input parameters and collect the best 10 results along with the optimized values. It was not clear if this was possible with cvxpy.

CVXPY is not threadsafe. Can you link to the offending documentation?

Here:
https://www.cvxpy.org/examples/applications/consensus_opt.html?highlight=multiprocessing
https://www.cvxpy.org/tutorial/intro/index.html?highlight=multiprocessing

You can definitely spawn multiple processes, each of which solves its own problem, since processes don’t share memory. That’s just standard Python, and is not CVXPY specific.
It’s possible the examples don’t work, though. Can you paste the entire error messages you saw when trying to run them?

OK, thanks I will work on these later.

When I need parallelism with CVXPY I just use Dask. They have a “delay” feature where you pass it a reference to a function handle (can be an instance method, like “.solve” for a CVXPY Problem). You can specify that the parallelism should be localized to different threads.
https://docs.dask.org/en/latest/delayed.html
Sent with GitHawk

@rileyjmurray : can you post a short snippet how you/to use dask with .solve()?

This is a code snippet I've actually used.
import dask

def ecossolve(cvxprob):
    cvxprob.solve(solver='ECOS')
    return cvxprob

def parallel_ecossolve(problist):
    dasklist = [dask.delayed(ecossolve)(prob) for prob in problist]
    results = dask.compute(*dasklist, scheduler='processes')
    return results

I don't remember exactly why I created the ecossolve function. I probably needed it because of something with how dask.delayed(fxn)(arg) passes arg to fxn, but there is also a chance that I wrote awkward code based on a poor understanding of dask.


Add install instruction to package sidebar

What
Copiable command line instruction that will install the package you're looking at.
Behaviour

Triple click or drag to select text.
Doesn't select > symbol.

Notes

Tripple click always selects extra new line due to setting css overflow, not sure if or how this should be solved.
Could be made more interactive by actually copying on click.

Preview

Fixes
#27

Install instructions have been added in the new "About" section of each package, for example:
https://package.elm-lang.org/packages/elm/file/latest/about
Note that triple click seems to work fine to select it (at least it does on Linux with Firefox for me).

The current implementation also fits more text, which is an issue with having it in the sidebar.
Triple click selects a newline in Safari on macOs but not in Firefox, not sure if this is an issue worth fixing.


tensorflow Multi-GPU vs CPU mirroredstrategy

Ask everyone for help..
I am so confused why gpu is slower than cpu on any codition i try...
I want to use six GPU with mirroredstrategy to reduce the training time..
I follow below steps.
https://keras.io/guides/distributed_training/
I got the bad performance with the machine
GPU: 6*GeForce RTX 2080 TI (10GB)
CPU: Intel(R) Xeon(R) Silver 4110 CPU @ 2.10GHz
run on the Docker container:
(1)tensorflow version 2.0.0
(2)cuda 9.0
(3)cudnn 7.6
(4)nvidia-driver 410.78 - install on server
====Test1=======
CPU,batch=64,samples:575478
Epoch 2/500 575478/575478 [===] - 16s 28us/step - loss: 0.0735 - val_loss:
====Test2=======
GPU,batch=64,samples:575478
Epoch 1/500
1498/1498 [===] - 60s 40ms/step - loss: 0.0907 - val_loss: 0.0619
Epoch 2/500
1498/1498 [===] - 32s 21ms/step - loss: 0.0592 - val_loss: 0.0522
my_mini_batch = BATCH_SIZE_PER_REPLICA * strategy.num_replicas_in_sync(GPU)
384 = 64*6
[enter image description here][1]
Why so slower than CPU?
enter code here
#import tensorflow_datasets  as ds #for debug
os.environ["TF_FORCE_GPU_ALLOW_GROWTH"] = "true"
os.environ['CUDA_VISIBLE_DEVICES'] = '0,1,2,3,4,5'
gpu_table = ['/gpu:0','/gpu:1','/gpu:2','/gpu:3','/gpu:4','/gpu:5']

strategy = tf.distribute.MirroredStrategy(devices=gpu_table)

TrainDataPath = './Train'
TestDataPath = './Test'

def Load_Data(InputPath):
 Data_Total = np.array([])
 Label_Total = np.array([])
 folder_content = glob.glob(InputPath+'/'+'*_Label*')
 print('folder content=',folder_content)
 for File in folder_content:
	Label = np.load((File))
	Label_Total = np.append(Label_Total, Label)
	Feature = np.load([F for F in glob.glob(InputPath + "/" + File.split('/')[-1].split('Label')[0] + '*') if 'Label' not in F][0])
	Data_Total = np.append(Data_Total, Feature)

Label_Total = Label_Total.reshape(-1, 1)
Data_Total = Data_Total.reshape(-1, 1,3, 29)
return Data_Total, Label_Total

Train_Data, Train_Label = Load_Data(TrainDataPath)
Test_Data, Test_Label = Load_Data(TestDataPath)

def get_dataset():#for debug

 print('get_dataset()')
 print("Training_Data.shpae=", Training_Data.shape)

 global my_mini_batch
 my_mini_batch = 1
 BATCH_SIZE_PER_REPLICA = 64

 my_mini_batch = BATCH_SIZE_PER_REPLICA * strategy.num_replicas_in_sync
 print('my_mini_batc=',my_mini_batch)
 print('BATCH_SIZE_PER_REPLICA=',BATCH_SIZE_PER_REPLICA)
 print('strategy.num_replicas_in_sync=',strategy.num_replicas_in_sync)

 Train_Dataset_1 = tf.data.Dataset.from_tensor_slices((Training_Data[:, :,:,0:5], Training_Data[:, :,:,5:15], Training_Data[:, :,:,15:25], Training_Data[:,:,:,25:])).batch(my_mini_batch).repeat()
 Train_Dataset_2 = tf.data.Dataset.from_tensor_slices(Training_Label).batch(my_mini_batch).repeat()

 global train_steps
 global valid_steps
 global test_steps
 train_steps = (int)( Training_Data.shape[0] / my_mini_batch)
 print('train_steps='+str(train_steps))

 valid_steps = (int)(Val_Data.shape[0] / my_mini_batch)
 print('valid_steps=' + str(valid_steps))

 test_steps = (int)(Test_Data.shape[0] / my_mini_batch)
 print('test_steps=' + str(test_steps))

 Valid_Dataset_1 = tf.data.Dataset.from_tensor_slices((Val_Data[:, :,: ,0:5], Val_Data[:, :,:,5:15],
 Val_Data[:, :,:,15:25], Val_Data[:, :, :,25:])).batch(my_mini_batch).repeat()
 Valid_Dataset_2 = tf.data.Dataset.from_tensor_slices(Val_Label).batch(my_mini_batch).repeat()

 Test_Dataset_1 = tf.data.Dataset.from_tensor_slices((Test_Data[:, :,:,0:5], Test_Data[:, :,:,5:15],
 Test_Data[:, :,:,15:25], Test_Data[:, :,:,25:])).batch(my_mini_batch).repeat()
 Test_Dataset_2 = tf.data.Dataset.from_tensor_slices(Test_Label).batch(my_mini_batch).repeat()

 print('Replicas: ', strategy.num_replicas_in_sync)
 print("Num GPUs Available: ", len(tf.config.experimental.list_physical_devices('GPU')))

 Train_Dataset_final = tf.data.Dataset.zip((Train_Dataset_1,Train_Dataset_2))
 Valid_Dataset_final = tf.data.Dataset.zip((Valid_Dataset_1,Valid_Dataset_2))
 Test_Dataset_final = tf.data.Dataset.zip((Test_Dataset_1,Test_Dataset_2))

 return Train_Dataset_final, Valid_Dataset_final, Test_Dataset_final

with strategy.scope():
 PalmTh = 0.5
 A_layer1_Filters= 2
 B_layer1_Filters = 2
 C_layer1_Filters = 2
 D_layer1_Filters = 2

 L = 0.005
 F = 12
 Epochs = 500
 EarlyStopPatience = 10
 ChangeLrPatience = 8
 ChangeLrFactor = 0.9

 Test_Para = []

 A_IN_Cell = Input(shape=(1, 3, 5), name = "Cell")
 PX = Input(shape=(1, 3, 10), name = "X")
 PY= Input(shape=(1, 3, 10), name = "Y")
 PZ = Input(shape=(1, 3, 4), name = "Z")

 L1= Convolution2D(filters = A_layer1_Filters,
	     kernel_size = 3,
	     strides = 1,
	     padding = 'valid',
	     data_format = 'channels_first',
	     use_bias = True ,
	     name = 'Conv1_Height_Cell', activity_regularizer=regularizers.l2(0.00001))
 (A_IN_Cell)

 LH1 = Activation(custom_HardTanh)(L1)
 LH1_out= Flatten()(LH1)

 L2= Convolution2D(filters = B_layer1_Filters,
	     kernel_size = 3,
	     strides = 1,
	     padding = 'valid',
	     data_format = 'channels_first',
	     use_bias = True ,
	     name = 'Conv1_ProjectionX', activity_regularizer=regularizers.l2(0.00001))
 (PX)

 LH2= Activation(custom_HardTanh)(L2)
 LH2_out= Flatten()(LH2)

 A_Convolution1 = Convolution2D(filters = C_layer1_Filters,
	     kernel_size = 3,
	     strides = 1,
	     padding = 'valid',
	     data_format = 'channels_first',
	     use_bias = True ,
	     name = 'Conv1_ProjectionY', activity_regularizer=regularizers.l2(0.00001))
 (PY)

 A_Hidden1 = Activation(custom_HardTanh)(A_Convolution1)
 A_Out = Flatten()(A_Hidden1)

 Centroid_Convolution1 = Convolution2D(filters = D_layer1_Filters,
	     kernel_size = 3,
	     strides = 1,
	     padding = 'valid',
	     data_format = 'channels_first',
	     use_bias = True ,
	     name = 'Conv1_Centroid', activity_regularizer=regularizers.l2(0.00001))(PZ)

 Centroid_Hidden1 = Activation(custom_HardTanh)(Centroid_Convolution1)
 Centroid_Out = Flatten()(Centroid_Hidden1)

 ConcatentaLayer = concatenate([LH1_out,
	       LH2_out,
	       A_Out,
	       Centroid_Out]
	       )

 DenseLayer1 = Dense(F, use_bias = True, activation=None,
 activity_regularizer=regularizers.l2(0.00001))(ConcatentaLayer)
 DenseLayer1 = Activation(custom_HardTanh)(DenseLayer1)
 Output = Dense(1, use_bias = True, activation='sigmoid')(DenseLayer1)

 model = Model(inputs=[A_IN_Cell , PX, PY, PZ],
 outputs=[Output])

 adam = optimizers.Adam(lr=L)

 model.compile(optimizer = adam, loss = 'binary_crossentropy')
 model.summary()

 change_lr = ReduceLROnPlateau(monitor='val_loss', factor=ChangeLrFactor,
			      patience=ChangeLrPatience, min_lr=0.00005)
 EarlyStop = EarlyStopping(monitor='loss', patience = EarlyStopPatience, verbose=2, mode='min')

 Training_Data, Val_Data, Training_Label, Val_Label = train_test_split(Train_Data, Train_Label,
 test_size=0.1)
 print('Train_Data follow=',Train_Data.shape)
 print('Training_Data follow=',Training_Data.shape)

 train_dataset, val_dataset, test_dataset = get_dataset()
 print('type(train_dataset)=', train_dataset)

#beside scope
history = model.fit(train_dataset, epochs=Epochs,
                steps_per_epoch=train_steps,
                validation_steps = valid_steps,
                validation_data=val_dataset)

@harrypotter02
I ran the code shared and face different error, please find the gist here , share a colab gist with the error faced.(select change run time and gpu)

@Saduf2019  I Appreciate your help very much.
I want to ask some question.
(1) I want to know whether tensorflow 'MirroredStrategy' can speed up the training speed.
my example is follow the link
https://www.tensorflow.org/api_docs/python/tf/distribute/MirroredStrategy
(2)How to speed up the training speed(double/triple) by  MirroredStrategy.
I follow the link implement https://www.youtube.com/watch?v=bRMGoPqsn20
My exsample code is word on colab or github gist,
but you need download the training data and put on the google drive space.
//colab
https://colab.research.google.com/drive/1ldJvdk6wfu-fXBb2iBjKe0q1ZnExGG17?usp=sharing
//github gist
https://gist.github.com/harrypotter02/0cc6ffe3bf7c520207dc7be96b1e8b66
I got the bad performance with the machine
GPU: 6*GeForce RTX 2080 TI (10GB)
CPU: Intel(R) Xeon(R) Silver 4110 CPU @ 2.10GHz
run on the Docker container:
(1)tensorflow version 2.0.0
(2)cuda 9.0
(3)cudnn 7.6
(4)nvidia-driver 410.78 - install on server
====Test1=======
CPU,batch=64,samples:575478
Epoch 2/500 575478/575478 [===] - 16s 28us/step - loss: 0.0735 - val_loss:
====Test2=======
GPU,batch=64,samples:575478
Epoch 1/500
1498/1498 [===] - 60s 40ms/step - loss: 0.0907 - val_loss: 0.0619
Epoch 2/500
1498/1498 [===] - 32s 21ms/step - loss: 0.0592 - val_loss: 0.0522
my_mini_batch = BATCH_SIZE_PER_REPLICA * strategy.num_replicas_in_sync(GPU)
384 = 64*6
[enter image description here][1]
Why so slower than CPU?

Other experiment.
-------Lib1 S-----------
Add mode samples for test.
But the six GPU slower than CPU..WHY..
Windows CPU , batch=64
training samples 1726436
Epoch 1/500 1726436/1726436 [=======] - 62s 36us/step - loss: 0.0888 - val_loss: 0.0685
Epoch 2/500 1726436/1726436 [=======] - 66s 38us/step - loss:
Windows GPU , batch=64
training samples 1726436
4495/4495 [=====] - 122s 27ms/step - loss: 0.0644 - val_loss: 0.0518
Epoch 2/500
4495/4495 [=====] - 93s 21ms/step - loss: 0.0466 - val_loss: 0.0442
nvidia -graph
https://imgur.com/svhbZsd
-------Lib1 E-----------

Hi @harrypotter02, in response to your first question, yes using MirroredStrategy can speed up training. However, sometimes you might need to do some performance debugging to make sure your program can make the best use of your accelerators. We are working on a guide to help with this type of debugging, but as a first step here are some things you can try:

Run an experiment to see if you are getting any speedup just using a single GPU (no MirroredStrategy) compared to just CPU
Run an experiment to see if you are getting any speedup using 2 GPUs and MirroredStrategy (it's possible that the overhead of synchronizing 6 GPUs is too high given your dataset/model size)
Follow this guide for using the Tensorboard Profiler, which can help you identify what could be causing the bottleneck. Often a slowdown like this indicates that your program in input bound.

Lastly, looking at your code I noticed you're using a combination of tensorflow and Keras. tf.distribute.MirroredStrategy() is tested with tf.keras and you'll notice that in the guide you are following imports from tensorflow import keras
I would suggest modifying your code so that you are only importing from tf.keras and not native keras.


989 feature request qdrant update

Description
refactor qdrant module and add some features

Added class QdrantManager to manage global collections' CRUD operations.
Added an explicit QdrantStorage.close_client() method to prevent concurrency conflicts in local storage access.
Added QdrantStorage.update(), which allows updating the vector itself or updating the payload.
Added filter functionality to QdrantStorage.query(), enabling searches using QdrantClient’s native Filter support.

Motivation and Context
close #989

[ ] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds core functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (update in the documentation)
[ ] Example (update in the folder of example)

Implemented Tasks

[ ] Subtask 1
[ ] Subtask 2
[ ] Subtask 3

Checklist
Go over all the following points, and put an x in all the boxes that apply.
If you are unsure about any of these, don't hesitate to ask. We are here to help!

[ ] I have read the CONTRIBUTION guide. (required)
[ ] My change requires a change to the documentation.
[ ] I have updated the tests accordingly. (required for a bug fix or a new feature)
[ ] I have updated the documentation accordingly.

Looks good but what do we must have QdrantManager? Can you explain?

Thanks @lightaime
QdrantManager is designed to provide users with the ability to manage collections so that users can control the collections in their qdrant database in their business logic.
The function of QdrantStorage is CRUD for records in a single collection, such as add() → add a record into collection. Its processing scope should only be collection.
In the original design, QdrantStorage had private functions to operate collections, such as _create_collection() and _delete_collection. If they were directly converted to public functions, there would be security risks.

Refactored association #1147


Gale excerpts and articles only (not full works) show the words "GALE url" before DocID on item detail pages

Full work, how it should look:
Excerpt, with added words:

Looks good. Tested with a Gale full work, an excerpt, and an article (and CW & CB links) and "GALE url" has been successfully removed.


Add links to 1st Gossoms End Scout's github resources

This PR adds a wordpress theme from 1st Gossoms End Scouts (and moves the organisation link out of currently empty)

@AA-Turner Could I get a review on this? Thanks

Every line needs to end with two spaces please


Running GMD in docker with KVM

Hi!
Looks like you have made a great effort exploring gradle managed devices in github. I was wondering if you have some experience getting the same thing up and running on a self hosted docker machine supporting KVM?
I have a sample project here trying to make it work
https://github.com/warting/gradle-managed-devices-docker
The error I receive is:
Caused by: java.lang.IllegalStateException: Gradle was not able to complete device setup for: dev30_google_apis_x86_Nexus_4
The emulator failed to open the managed device to generate the snapshot.
This is because the emulator closed unexpectedly, try updating the emulator and
ensure a device can be run from Android Studio.
        at com.android.build.gradle.internal.AvdSnapshotHandler.generateSnapshot(AvdSnapshotHandler.kt:264)
        at com.android.build.gradle.internal.AvdManager$loadSnapshotIfNeeded$1.invoke(AvdManager.kt:214)
        at com.android.build.gradle.internal.AvdManager$loadSnapshotIfNeeded$1.invoke(AvdManager.kt:194)
        at com.android.build.gradle.internal.AvdManager$runWithMultiProcessLocking$1.accept(AvdManager.kt:113)
        at com.android.build.gradle.internal.AvdManager$runWithMultiProcessLocking$1.accept(AvdManager.kt:113)
        at com.android.builder.utils.SynchronizedFile.doActionWithMultiProcessLocking(SynchronizedFile.java:273)
        ... 35 more

Hi!
I have tried to use self hosted runners. Please see the documentations About self-hosted runners.
I have no experience with docker machine. How do you want to use docker machine and what result do you want achieve?

Hey, the problem is not that it is docker or self hosted runners. The problem is running the android emulators in nested virtualisated environments. Docker machine is just one common way of creating them :)

@warting Looks like now we can run android emulators in Ubuntu runners. We can enable the nested virtualization
        if: contains(matrix.os, 'ubuntu')
        run: |
          echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' | sudo tee /etc/udev/rules.d/99-kvm4all.rules
          sudo udevadm control --reload-rules
          sudo udevadm trigger --name-match=kvm


How to Scale and Rotate the texture region got from Image File.

Versions

Python: 3.7
OS: Windows
Kivy: 1.11.1
Kivy installation method: Spyder Anaconda

Description
I want to rotate the texture region obtained from the image file. I have a piece of code in Pygame which I want to convert into Kivy.
Code and Logs
A code in PyGame:
full_sprite =  pygame.image.load("./resources/images/full-sprite.png").convert_alpha()
rect = pygame.Rect(181, 1050,50,50)
cropped = full_sprite.subsurface(rect).copy()
pig_image = pygame.transform.scale(cropped, (30,30))
angle_degrees = math.degrees(pig.body.angle)
img = pygame.transform.rotate(pig_image, angle_degrees)
w,h = img.get_size()

This is how much I converted into Kivy but it is not working. I am not able to find any method to scale and rotate the texture region.
full_sprite = Image(source='resources/images/full-sprite.png', allow_stretch=True)
full_sprite.texture = full_sprite.texture.get_region(181, 1050,50,50)
pig_image = full_sprite.texture
self.angle_degrees = math.degrees(pig.body.angle)
img = full_sprite.texture.Rotate(set(self.angle_degrees))

The provision of solution for this issue will be a great help to me.

No one helped me on support channels so far.


Mention (in README) that the browser bundle supports AMD

There are some old docs floating around the net suggesting you need to build with python, if you google for highlight.js AMD.

We actually need a few words in the README describing our relationship with various build systems and distribution means. I'll use this open issue as a reminder to do that and mention AMD too. Thanks!

I added being able to use both AMD and CommonJS in the README. All that's left is left is the russian translation.


Support select multiple options in select menu components

https://discord.com/channels/730045476459642900/1199012215336992951

This can be useful for e.g. role selects. To not create a text response for each selected option the user would disable the default response for each options and then add a "finally" response to the select menu itself.
Or would the default response be smart enough to figure that out by itself?


Users have to manually install extensions

Right now, users have to manually run just install-beyond-extensions after logging in. This is not great! Instead, we should add them to Yafti once it supports installing GNOME extensions.

Blocked by https://github.com/ublue-os/yafti/issues/110

we could try using the "beyond first boot" script to install these extensions - like, we could just run the custom.just's install-beyond-extensions recipe on it then by writing a "firstboot-done" on /etc/ublue-os/beyond-first-boot (or something like that) we can verify whether or not the script ran and if run accordingly
i'd just make a pull request for that right now, but I don't know if that's something that makes sense for this project..?

What I did in #87 (because the extension was not packaged anywhere) was just included the extension in tree. This could be an option, similar to how Inter is handled, though it seems a little odd and will not benefit from updates without OS updates.

I've thought about trying this, put the extensions on the disk and then let them update, but not sure how that would work long-term. I think they'd autoupdate themselves right away?

@castrojo if you include the extension files directly in the image, they show as "system" installed extensions, and so don't get updated e.g. from the Extensions Manager app. Whatever files are included in the image is what will be run.
So the nice side is that everyone on the same commit of the OS is running the exact same extension code, but the downside is that extension updates can only come with an OS update/reboot. I think that's... fine? You don't get a new GNOME Shell update without rebooting into a new commit of the OS, either.

Yeah, that's probably better than having to remind people to run a just command.