Bkgnd layer v2

I rebased the previous pull request to master. There are no changes needed for ilmControl implementation. Therefore, I dropped the third patch.

hi @eucan
we should remover the waston_view_set_mask call because src width and height are 1x1 so we we can just omit it.
and to see the background layer from the first frame we need to set the output_mask for the surface correctly this will be done by the weston_view_update_transform api, please find a patch for this:
diff --git a/weston-ivi-shell/src/ivi-controller.c b/weston-ivi-shell/src/ivi-controller.c
index 4133ca3..6d95c88 100644
--- a/weston-ivi-shell/src/ivi-controller.c
+++ b/weston-ivi-shell/src/ivi-controller.c
@@ -909,10 +909,9 @@ set_bkgnd_surface_prop(struct ivishell *shell)
     weston_log("set_bkgnd_surface_prop: x:%d y:%d s_width:%d s_height:%d d_width:%d d_height:%d\n",
                x, y, src_width, src_height, dest_width, dest_height);

-    weston_view_set_mask(view, 0, 0, src_width, src_height);
     wl_list_insert(&view->geometry.transformation_list,
                    &shell->bkgnd_transform.link);
-
+    weston_view_update_transform(view);
     weston_surface_schedule_repaint(w_surface);
 }

other wise it looks good


2 new queries

2 new queries
Network related queries

Hey @fskelly it looks like you dropped the schema line introduced by Jelle

new PR coming


New Appearance

To be as similar as possible to the public version. This requirement is related to all the requirements labelled with UI for the milestone 0.1.5.
Some general instructions

When the user selects the option (Layers/Entities/Models/..), the form should have
at the top an empty editbox with a button NEW next to it. The layer/entity/... is created filling the editbox and the button new (no a window is open). Similar to the public instance: http://riscossplatform.ow2.org/riscoss/wiki/githubanalyser/view/RISCOSSPlatformLayers/
just after the list/tree of existing layers/entities/models
in each row a DELETE button, like in entities/models... If is not possible for tree is not a problem because we will have a DELETE button for form on the right
When athe user selects one entity/layer/... a form on the right is shown with the details. This form will contains SAVE and DELETE buttons

User Stories

#61 layer management
#62 entity management
#63 models management
#64 risk configuration management
#58 risk session management

Added ~54 label

Added ~76 label

mentioned in issue #62

Removed ~76 label

mentioned in issue #63

Is this closeable?

mentioned in issue #64

No, this issue is not a requirement, some of the sub-issues are not yet implemented. Maybe it should be an issue without milestone.

Milestone removed

all the user stories closed, some issues related on the new UI are still ongoing, but the general change is already finished


Add docs for k8s secrets

@anthonychu - let me know if you have any comments on these changes :-)

Thanks @stuartleeks


Look into possibly running on WebWorker Thread

This could be especially very useful for our boot sequence, as the UI thread would be freed up for doing the Javascript part.
There would then be the need for a "ready event".
┆Issue is synchronized with this Asana task
┆Attachments: MicrosoftTeams-image.png

Never heard of anyone trying to run webassembly in the serviceworker. Did it work ? What was the purpose?
(Normally I use a webworker for webassembly in cases where there are long running operations that would block the UI if running on the main thread).

Fun fact: we probably meant Web worker - as the idea came from you :)
Second, its not supposed to be closed. So thanks :)

Great :-) Interesting project you got here btw! Looking forward to see it evolve :-)

Thanks. Next week this part will see a lot of love - and also getting the documentation to a decent level

Note to self:
After initial investigations, there could be benefits for us with this; especially on the startup. We now have the startup down to a couple of seconds for the WASM part and we could shave off a second or so from the total if the JavaScript part was running in parallel.
The consequence of doing this however is that all interop with the main UI thread would have to go through PostMessages, which is fine and wouldn't cause too much overhead - we would have to rewrite our interop layer. If we chose to follow through with this, we should do it in conjunction with #39. Our use of IndexedDB seems fine, for modern browsers - as it is supported on Web Worker threads; https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Functions_and_classes_available_to_workers


🔧 History component Amount token

PR type

[x] Bugfix

What's new?

[x] PR closes #4193
[x] display right token in History Table

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Screenshot

lgtm


Wrong UI duration

When I set
SlideCountdown(
duration: Duration(seconds: 50),
)
the countdown starts from 48

@farhanfadila1717 same


[W5.8b][T09-A2]LIWEN LAI

add some NFRs

Actually there is no NFR in AB2 developper guide. The exercise is about AB3.
@Livian1107 Some comments added. Please close the PR after reading comments.


Better handling of cors_enable=True + added CORSMiddleware

Although similar to https://github.com/falconry/falcon/issues/1930, this issue proposes to improve not only documentation, but also provide a clearer indication to the developer who has happened to pass this unsupported combination of parameters.
Maybe we could emit a warning, silently disable the implicitly added middleware component if we detect that any instance of CORSMiddleware is already present, or even raise an Exception? (Although the latter might be seen as a breaking change -- discuss).

connected to https://github.com/falconry/falcon/pull/1976
Probably the best choice here is to deprecate the flag, since any other case would not be immediately clear to understand.
An alternative may be to raise a deprecation warning and the raise an exception in the next major

See also https://github.com/falconry/falcon/discussions/2095. Rescheduling for 4.0.

I went ahead and decided to simply raise a ValueError in this case.

makes sense


Added Sentiment Analysis Endpoint to API

The pull-request was co-authored by Mitch Hollberg
Description
The team added an endpoint to the DS API for future sentiment analysis usage.  Right now, this endpoint will take any input text, call vaderSentiment's SentimentIntensityAnalyzer, and will return 'positive' if the sentiment analysis compound score is greater than 1 and will return 'negative' otherwise.  Eventually this will be helpful when linked to meeting notes and will be able return a negative or positive prediction of the how the meeting went.
vaderSentiment was added to the requirements.txt file.
API version was updated.
Type of change
Please delete options that are not relevant.

[ x] New feature
[ x] This change requires a documentation update

Checklist:

[x ] My code follows PEP8 style guide
[ x] I have removed unnecessary print statements from my code
[ x] I have made corresponding changes to the documentation if necessary
[ x] My changes generate no errors
[ x] No commented-out code
[ x] Size of pull request kept to a minimum
[ x] Pull request description clearly describes changes made & motivations for said changes

https://www.loom.com/share/d269ef774b414893b926e00ca5eb962c

Good thinking about creating this feature. I can't think of any changes or improvements that can be made from it. well done!


DBZ-1782 DBZ-1807 DBZ-1930 Document new devel features

Agh, didn't mean to merge this one yet. Will send a follow-up.


error on running example

(gpu) fridavs@fridavs-pc:~/Desktop/PC/deepgaze/examples$ python ex_pnp_head_pose_estimation_webcam.py
The video source has been opened correctly...
Estimated camera matrix:
[[554.2563   0.     320.    ]
[  0.     554.2563 240.    ]
[  0.       0.       1.    ]]
Traceback (most recent call last):
File "ex_pnp_head_pose_estimation_webcam.py", line 321, in
main()
File "ex_pnp_head_pose_estimation_webcam.py", line 168, in main
my_cascade.findFace(gray, True, True, True, True, 1.10, 1.10, 1.15, 1.15, 40, 40, rotationAngleCCW=30, rotationAngleCW=-30, lastFaceType=my_cascade.face_type)
File "/home/fridavs/anaconda3/envs/gpu/lib/python3.7/site-packages/deepgaze/haar_cascade.py", line 88, in findFace
self._findFrontalFace(inputImg, frontalScaleFactor, minSizeX, minSizeY)
File "/home/fridavs/anaconda3/envs/gpu/lib/python3.7/site-packages/deepgaze/haar_cascade.py", line 149, in _findFrontalFace
flags=cv2.cv.CV_HAAR_SCALE_IMAGE
AttributeError: module 'cv2.cv2' has no attribute 'cv'
(gpu) fridavs@fridavs-pc:~/Desktop/PC/deepgaze/examples$

Hi @tensor2flow
This is an issue of compatibility between the version of OpenCV you have installed on your system and the version of OpenCV required by Deepgaze.
My suggestion is to switch to the version 2.0 of Deepgaze. Simply clone the repository and then checkout 2.0 to load the updated version. After this you can install again the library. Version 2.0 is compatible with both OpenCV 2 and 3.


Announce 4.147

This post will be only published after the build success

https://hhvm.com/api/build-status/4.137.0
All targets are published

https://hhvm.com/api/build-status/4.137.0
All targets are published

It seems some deb packages are missing from https://dl.hhvm.com/debian/pool/main/h/hhvm/
bin/build-on-aws 4.147.0 debian-11-bullseye


[comet-parquet-exec] Add Native Scan to CometReadBenchmark

I still need to convince myself that CometNativeScan is being generated for the scenarios when it's enabled.
CometReadBenchmark-jdk11-results.txt

I am curious what's going on in this scenario in particular:
Sorted Lg Str Filter Scan (50.0% zeros):  Best Time(ms)   Avg Time(ms)   Stdev(ms)    Rate(M/s)   Per Row(ns)   Relative
------------------------------------------------------------------------------------------------------------------------
SQL Parquet - Spark                                 380            441          40          2.8         362.6       1.0X
SQL Parquet - Comet                                 233            241          10          4.5         221.8       1.6X
SQL Parquet - Comet Native Scan                     161            209          37          6.5         153.5       2.4X

I'll check if the benchmarks use the settings from CometTestBase or the global defaults. If so, then @parthchandra is right.


Clicking an interactive tooltip trigger parent onClick

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
If an interactive tooltip is child of a button and is clicked, it triggers the onClick of the parent button.
Expected Behavior 🤔
Interactive tooltip should not trigger any onClick.
Steps to Reproduce 🕹
Steps:

Wrap an interactive tooltip in a Button and add an onClick
Click on the tooltip

Running sandbox: https://codesandbox.io/s/material-demo-forked-zo0hv?file=/demo.js
Your Environment 🌎

`npx @material-ui/envinfo`
System:
    OS: macOS 11.3.1
  Binaries:
    Node: 12.16.1 - ~/.nvm/versions/node/v12.16.1/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.8 - ~/.nvm/versions/node/v12.16.1/bin/npm
  Browsers:
    Chrome: Version 92.0.4503.0 (Official Build) dev (x86_64) [used]
    Edge: 90.0.818.56
    Firefox: 85.0.2
    Safari: 14.1
  npmPackages:
    @material-ui/core: ^4.9.14 => 4.10.0
    @material-ui/icons: ^4.9.1 => 4.9.1
    @material-ui/lab: ^4.0.0-alpha.55 => 4.0.0-alpha.55
    @material-ui/styles:  4.10.0
    @material-ui/system:  4.9.14
    @material-ui/types:  5.1.0
    @material-ui/utils:  4.9.12
    @types/react: ^17.0.3 => 17.0.3
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    typescript: ^4.2.3 => 4.2.4

Out of curiosity why are you adding the Tooltip on the text inside the button? Why not wrapping the Button with Tooltip? Something like https://codesandbox.io/s/material-demo-forked-34x67

This is expected behavior for React portals in general, and Tooltip specifically. As Marija suggested, you should wrap your Button instead which is also important for a11y concerns (labelling, tooltip open on button focus etc)


GraphiQL use static schema

Part of https://app.asana.com/0/1201910659736061/1202865485101500/f
Notes:

The big challenge here is juggling a 40 MB schema file that is difficult to generate
My original plan was to host the schema separately (i.e., static.medplum.com), but CORS became a mess
So, instead, I decided to host the schema together with the graphiql web app
Due to the complexity of generating the schema, I decided it was out of scope to make it work with Vercel hosting
So, instead, I fell back to AWS S3/CloudFront, so we have more control over the storage and caching

Coverage remained the same at 94.129% when pulling cc9d14fe8cf9408a94c51771aa29aeb0346c70cf on cody-graphiql-static-schema into fbf086ce9f7227eff3c6d045197537663e0a2287 on main.


BREAKING CHANGE: update internal modules to latest, replace github_organization with github_owner, bump Terraform version to >=0.13.1

Description

add data source for using correct partition based on where Atlantis is provisioned for the default ECS task execution policy ARN
update sub-modules to use latest versions; GitHub version need to be updated for deprecating changes
bump min Terraform version to 0.13.1+ due to module and provider support
replace github_organization with github_owner due to deprecation in the GitHub provider
switch policy ARN attachment from count to for_each
correct github-complete example project so it works correctly again (trusted principals and trusted entities)

Motivation and Context
Closes #190
Closes #192
Closes #201
Breaking Changes

Yes; support for Terraform v0.12.x is dropped, and GitHub provider min version supported raised to v4.8.x for deprecation of github_organization

How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects

complete example updated and validated with test projects
also fixed/updated GitHub webhook example project

I know the PR is not ready but trusted_principals and trusted_entities are rather handy to have. Good night :)

Thanks, @bryantbiggs !
https://github.com/terraform-aws-modules/terraform-aws-atlantis/releases/tag/v3.0.0


Modifier -> SwiftUI -> App extension: dynamicIsland(verticalPlacement:)

Doc: https://developer.apple.com/documentation/swiftui/view/dynamicisland(verticalplacement:)

[ ] Swift implementation
[ ] Elixir implementation

Live Activities cannot be created with LiveView Native.

We won't support this one for now


Tests for changes to Signing and Publishing process

https://github.com/dotnet/arcade/pull/6206 will be introducing a change in logic for Signing and Publishing. We need to ensure we get tests around this.

Old issue, probably not going to be done at this point. Can revisit making tests if/when this code gets touched again.


working with Harbor and a Self Signed Certificate

Describe the problem or bug
I have an installation of harbor with a self-signed certificate.  When trying to use pushrm I am getting an invalid cert (note: push works for the image as i have configured docker to ignore).  Is there a way to tell pushrm to ignore the error as well?
docker-pushrm version
pushrm*     Push Readme to container registry (Christian Korneck, 1.9.0)
Docker CLI version and platform
Client: Docker Engine - Community
Version:           20.10.17
API version:       1.41
Go version:        go1.17.11
Git commit:        100c701
Built:             Mon Jun  6 23:02:57 2022
OS/Arch:           linux/amd64
Context:           default
Experimental:      true
Server: Docker Engine - Community
Engine:
Version:          20.10.17
API version:      1.41 (minimum version 1.12)
Go version:       go1.17.11
Git commit:       a89b842
Built:            Mon Jun  6 23:01:03 2022
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.6.8
GitCommit:        9cd3357b7fd7218e4aec3eae239db1f68a5a6ec6
runc:
Version:          1.1.4
GitCommit:        v1.1.4-0-g5fd4c4d
docker-init:
Version:          0.19.0
GitCommit:        de40ad0
if possible: registry server version
self-hosted harbor version 2.0.0
exact command that you're running
docker pushrm --debug myimage:latest --providerharbor2
debug output
Publishing Image Readme
DEBU root cmd init config
DEBU home dir: /home/ravenwolf.org/sanderson
DEBU subcommand "pushrm" called
DEBU Using target: harbor.internal-sites.org/images/traefik:latest
DEBU using README file: README.MD
DEBU server: harbor.internal-sites.org
DEBU namespace: images
DEBU repo: traefik
DEBU tag: latest
DEBU repo provider: harbor2
DEBU Harbor2.GetAuthident called
DEBU no credentials found in env vars. Trying Docker credentials store
DEBU Using config file: /home/localuser/.docker/config.json
DEBU util.GetDockerCreds called
DEBU tried candidate harbor.internal-sites.org: found credentials for user localuser
DEBU Using Docker creds: localuser ********
DEBU Harbor2.Pushrm called
DEBU Put "https://harbor.internal-sites.org/api/v2.0/projects/images/repositories/traefik": x509: certificate is valid for 4e001b526386b34480ab04d1538a8d04.3026041018f2cef38123353136fd9210.traefik.default, not harbor.internal-sites.org
DEBU error pushing README, error creating http request
ERRO error pushing readme to repo server. See error message below. Run with "--debug" for more details.

error pushing README, error creating http request

There are currently no docker-pushrm flags that you could set to trust a custom server certificate or allow insecure connections. You could however add the public cert of your server to your OS'es trusted CA certs and it should just work.
This is in line with how several other tools in this space behave:
https://github.com/genuinetools/img#using-self-signed-certs-with-a-registry
Please let me know if this doesn't work for you for some reason or if you need more infos.

Understood - maybe this could be a feature request?

maybe this could be a feature request?

Sure. Are there any particular reasons why making the self-signed cert trusted on the machine where you want to run docker-pushrm isn’t practical?
And are you looking to pass the cert to the cli or are you just looking for a convenience --allow-insecure flag?

A simple --allow-insecure would be perfect.  The other option could be to pull from the global docker daemon.json file "insecure-registries" property?

thanks, these are good thoughts, I will keep this open as a feature request


[PLUGIN REQUEST] grug-far

Field
Description

Plugin
grug-far

Homepage
https://github.com/MagicDuck/grug-far.nvim

Nixpkgs
false

Extra Information
it is a nice plugin for doing global find/replace utilizing ripgrep, I'm not sure if you want to have ripgrep installed as part of enabling the plugin though. I put nixpkgs as false for now, but ripgrep is available in nixpkgs

@tknawara Currently available in nixpkgs.  vimPlugins.grug-far-nvim. It would if I could use this instead of nvim-spectre


Improve type checking in constructors in new data model

Typeguard currently raises an error on the first wrong type it encounters. It would be more helpful if it returns information on all the wrong types among arguments passed by the user.
Additionally, as we create the setters dynamically, the argument name is always value. This should be improved as well.

Fixed via https://github.com/spdx/tools-python/pull/349


⚠ Bofrost.at has degraded performance

In babef3f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1329 ms

Resolved: Bofrost.at performance has improved in 5b63e62.


Use guzzle http client instead of file_get_contents

Many servers have allow_url_fopen disabled so they cant use file_get_contents to load apple keys.

+1 or better yet just use PSR-18 interface https://www.php-fig.org/psr/psr-18/ so any library can be used for request

similar PR from me: https://github.com/patrickbussmann/oauth2-apple/pull/17
@williamxsp @patrickbussmann for me it is important to call Guzzle with custom configuration and because of that guzzle client needs to be injectable (guzzle client with custom config is defined elsewhere)


Add navbar links to get help

@lognaturel, @issa-tseng, what do you think about adding link(s) to the navbar dropdown that link to the docs and/or the forum? One possibility is that these links could go between "Edit profile" and "Log out".
Also CC @florianm, who made a related comment on the forum.

Links to server diagnostics or docs/forum would be great!
e.g.

a hint to open the browser console might save a round trip to the forum
an expanding section in error messages to "view details" which I can copy straight away would save a round trip to the browser console and could include details about central version, deployment, issue with certificates etc (similar to R sessionInfo())
if error is known / occurs often due to incorrect config / relevant to server maintainers: a link to the FAQ or a preformed link to search the forum for that error.

Thanks, @florianm! Some of these ideas wouldn't be easy to implement today, but I like the idea of improving the user experience around errors. I'm going to throw out a couple of thoughts related to how Frontend handles errors today, then I'll show a screenshot of what I'm currently thinking.
Currently Frontend doesn't have information about the deployment, including the version of Central. Frontend could fetch /version.txt, and we could try to set up ops to provide Frontend other information about the deployment, but that's not something we have set up today. (Also linking to the related https://github.com/getodk/central/issues/59.)
There's a way in Frontend to read the details from a Backend error and display them in a human-readable way. The aim is for the Frontend error message to contain all the information that the user needs. For example, we use that mechanism in the modal that's used to create a form or upload a new form definition. However, I think there are probably more places we could use that mechanism: https://github.com/getodk/central/issues/144 may be an example of this.
Here are a couple of screenshots of what I think might be a quick win. I've updated the navbar to link to the docs, the forum, and /version.txt. We already link to the docs and the forum from the Central homepage, but this change would make those links accessible in more places.
Version 1: Add the links to the existing dropdown:

Version 2: Add a new help menu:

As an ODK Central user who just experienced an error, my first thoughts are "what's wrong exactly" and "where can I find a solution". The question mark icon in Version 2 is the most logical symbology for me to open, more than my profile/name.
Shower thought: "log" + "audit" + "console" = Win 10-like notification stream? Let this area display errors (with a link to the forum that contains a few targeted keywords) and make it unobtrusive yet discoverable.

the latter feels closer to me but I don't like the order and placement.
can we mirror the left side a little and do something like
[? v] [lang v] [space] [name v]


Unit tests/inject test clientset

What type of PR is this:
/kind code-refactoring
/area testing
What does this PR do / why we need it:

Add the possibility to inject (fake) dependencies when running unit tests on the CLI part.
Add unit tests for testing odo anlyze CLI and replace integration tests

Which issue(s) this PR fixes:
Fixes #6888
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

/override OpenShift-Integration-tests/OpenShift-Integration-tests
[FAILED] [423.437 seconds]
E2E Test starting with non-empty Directory [It] should verify developer workflow from non-empty Directory
/go/odo_1/tests/e2escenarios/e2e_test.go:197
[...]
  [odo]  âš   Unable to attach or mount volumes: unmounted volumes=[m2-suypzb-app-vol], unattached volumes=[odo-projects odo-shared-data m2-suypzb-app-vol kube-api-access-v6dgn]: timed out waiting for the condition

/override Kubernetes-Integration-Tests/Kubernetes-Docs-Integration-Tests
  [FAILED] registry_list_output.mdx
  Expected
      <string>:   strings.Join({
        	... // 108 identical bytes
        	"                        VERSIONS\nnodejs-react  StagingRegistry  ",
        	"       React is a free and open-source front-en...  2.0.2, 2.1.0",
      + 	", 2.2.0",
        	"\nnodejs-react  DefaultDevfileRegistry  React is a free and open-",
        	"source front-en...  2.0.2, 2.1.0\n```\n",
        }, "")

  to be empty
  In [It] at: /go/odo_1/tests/documentation/command-reference/doc_command_reference_init_test.go:189 @ 06/13/23 14:54:19.598


Not thread safe?

Hello,
in my current use case it would be cool if I can use multi threading/multi processing, cause I have a lot of calculations and my GPU could handle it.
However, I get strange results by using threading. Here is a small script to reproduce
from thundergbm import TGBMRegressor
from sklearn.datasets import load_boston
from sklearn.metrics import mean_squared_error
from multiprocessing.pool import ThreadPool as Pool
import functools
import numpy as np

def main():
    print("Without parallel threads: " + str(calc(2, 1)))
    print("With parallel threads: " + str(calc(2, 2)))

def calc(num_repeats, pool_size):
    x, y = load_boston(return_X_y=True)
    x = np.repeat(x, 1000, axis=0)
    y = np.repeat(y, 1000)

    x = np.asarray([x]*num_repeats)
    y = np.asarray([y]*num_repeats)

    pool = Pool(pool_size)
    func = functools.partial(fit_gbdt, x=x, y=y)
    results = pool.map(func, range(num_repeats))
    return results

def fit_gbdt(idx, x, y):
    clf = TGBMRegressor(verbose=0)
    clf.fit(x[idx], y[idx])
    y_pred = clf.predict(x[idx])
    rmse = (mean_squared_error(y[idx], y_pred)**(1/2))
    return rmse

if __name__ == '__main__':
    main()

Sometimes I get the error:
2019-10-14 10:48:22,422 FATAL [default] Check failed: [error == cudaSuccess]  an illegal memory access was encountered
2019-10-14 10:48:22,426 WARNING [default] Aborting application. Reason: Fatal log at [/thundergbm/include\thundergbm/util/device_lambda.cuh:49]
2019-10-14 10:48:22,434 FATAL [default] Check failed: [error == cudaSuccess]  an illegal memory access was encountered

and sometimes bad results:
Without parallel threads: [0.011103539879039557, 0.011174528160149052]
With parallel threads: [0.04638805412265755, 4.690559078455652]

Multi processing does work, but only if I'm not returning a TGBM instance.
Returning the instance would be the best solution but dosn't work at all cause TGBM is not picklable.
I'm using Windows 10 with CUDA 10.
From my experience it's sometimes hard to do multithreading with CUDA (=> tensorflow) but multiprocessing should be okay if the object is pickelable.
Maybe it is possible to make tgbm pickelable or find the bug which causes multi threading to crash.
Many thanks!

Thanks. We will look into this problem and get back to you if we have any update.

@Tripton Just some quick update. You may use the exact method for the tree_method option. We are working hard to locate the bug, which appears to be quite challenging due to massive parallelism in ThunderGBM with multithreading outside of it.

Hi @Tripton, thanks for your report. ThunderGBM is thread-safe now. I have run a dozen times of your code on our server, and there is no error shown. You should reinstall the library and have a try. Thanks.

this issue should be solved now. so we like to close it.

I would like to reopen this issue. I made some small modifications to the above code:
`from thundergbm import TGBMRegressor
from sklearn.datasets import load_boston
from sklearn.metrics import mean_squared_error
from multiprocessing.pool import ThreadPool as Pool
import functools
import numpy as np
def main():
print("Without parallel threads and 1 gpu: " + str(calc(2, 1, 1)))
print("With parallel threads and 1 gpu: " + str(calc(2, 2, 1)))
print("Without parallel threads and 2 gpu: " + str(calc(2, 1, 2)))
print("With parallel threads and 2 gpu: " + str(calc(2, 2, 2)))
def calc(num_repeats, pool_size, n_gpus):
x, y = load_boston(return_X_y=True)
x = np.repeat(x, 1000, axis=0)
y = np.repeat(y, 1000)
x = np.asarray([x]*num_repeats)
y = np.asarray([y]*num_repeats)

pool = Pool(pool_size)
func = functools.partial(fit_gbdt, x=x, y=y, n_gpus=n_gpus)
results = pool.map(func, range(num_repeats))
return results

def fit_gbdt(idx, x, y, n_gpus):
clf = TGBMRegressor(verbose=0, n_gpus=n_gpus)
clf.fit(x[idx], y[idx])
y_pred = clf.predict(x[idx])
rmse = (mean_squared_error(y[idx], y_pred)**(1/2))
return rmse
if name == 'main':
main()
`
Now sometimes I see output like this:
In [2]: %run test_tgbm.py                                                                                Without parallel threads and 1 gpu: [0.011102704273042477, 0.01117481052674395] With parallel threads and 1 gpu: [0.01117491826081946, 0.011103542490388574] Without parallel threads and 2 gpu: [0.01239784807141135, 0.012399129722859907] Segmentation fault (core dumped)
Also, I see that after it runs, maybe after a crash, gpu memory is not released:
`Fri Jun 12 10:05:34 2020
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 440.64.00    Driver Version: 440.64.00    CUDA Version: 10.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  TITAN X (Pascal)    On   | 00000000:03:00.0 Off |                  N/A |
| 29%   42C    P8    10W / 250W |   5747MiB / 12196MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
|   1  TITAN X (Pascal)    On   | 00000000:05:00.0  On |                  N/A |
| 31%   45C    P8    19W / 250W |    260MiB / 12190MiB |      0%      Default |
+-------------------------------+----------------------+----------------------+
+-----------------------------------------------------------------------------+
| Processes:                                                       GPU Memory |
|  GPU       PID   Type   Process name                             Usage      |
|=============================================================================|
|    0     27141      C   /home/xxxxxxxxx/venv/tf2/bin/python3            5707MiB |
|    1      1574      G   /usr/lib/xorg/Xorg                            14MiB |
|    1      1660      G   /usr/bin/gnome-shell                          49MiB |
|    1      2644      G   /usr/lib/xorg/Xorg                            92MiB |
|    1      2822      G   /usr/bin/gnome-shell                          98MiB |
+-----------------------------------------------------------------------------+`

Hi @civilinformer, thank you for your feedback. Your test results show that there might still be some bugs of thread-safety on ThunderGBM. We will conduct further tests on the ThunderGBM and fix the thread-safety issue in a better way. And we will get back to you if there is any update. Thank you.


Patch Release 1.2.1

Description

Improvements

Change line length to 100, the previous lentgh was too long

Bug Fixes

Removed staking from Jailed Validator
Default the users hash amount to 0

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer

We want to allow Jailed staking, so closing this and will work on supporting jailed staking instead.


Include read only mode on TextareaEditor

Resolve x-team/auto-ui#143

Thanks for including the build, forgot that :)


update for executors, prep for 0.2

Planning to tag 0.2.0 after merging.

I was about to comment on this. Do we ever provide this decoding anywhere in the docs:
https://github.com/pangeo-data/rechunker/blob/12b9a05dd1c4855968e5c36ff00a334749b9f7e9/rechunker/api.py#L149-L173
I think not.

It's not in the API docs, for the rechunk function: https://rechunker.readthedocs.io/en/latest/api.html#rechunk-function

I'll add a note. Best place is probably in rechunker.rechunk?


Animated Path2D doesn't update PathFollow2D progress when scene is running.

Tested versions
Reproducible in 4.2 stable, 4.0 stable, 4.1.2
System information
ubuntu 22.04
Issue description
When a Path2D curve is animated, the PathFollow2D progress is not updated when the scene starts and retains its position. In the editor, animation works fine and PathFollow2D moves according to the updated curve.
One exception. This works fine if "Visible Paths" is enabled in Debug.
The problem can be solved by listening to the curve.changed signal and setting the same progress manually. Still looks like a bug, especially since Progress_ratio is updated (without updating the node position) but Progress is not.
https://github.com/godotengine/godot/assets/72736632/3f8f5a67-fc4a-42cd-91da-6b500e6afd8e
Steps to reproduce

Play animation in the editor. PathFollow2D moves along an animated curve.
Start the scene. PathFollow2D is not moving.
Enable Visible Paths in the Debug and run scene. PathFollow2D moves.
Disable Visible Paths. Uncomment line 12 in the script and run the scene. PathFollow2D moves.

Minimal reproduction project (MRP)
AnimPath.zip

Can I fix this one or is there someone working on it already?

I got a fix, but the main branch is seriously broken, so we have to wait. I will do PR when I can test it.


correct port in Readme and do not require evolver.yml by default

i just had a shot at starting evolver-ng from scratch, hit same error as Ezira (refs #140 )
I also got the:  FileNotFoundError: [Errno 2] No such file or directory: 'evolver.yml' error.
To fix - I toggled the flag: https://github.com/ssec-jhu/evolver-ng/blob/main/evolver/settings.py#L20. to False as @jamienoss  suggested.
Then i went to the front end i was able to edit and save the configuration to a sane state...

BTW I was able to figure out why the codecov test was not running since we upgraded.
Should we:

make @imaitland admin so that they can admin merge their PRs?
unmark that test as required?

BTW I was unable to figure out why the codecov test was not running since we upgraded.
Should we:
1. make @imaitland admin so that they can admin merge their PRs?

2. unmark that test as required?

Probably want to keep codecov as a blocker... make me admin.

I'll go ahead and merge this in


Replace Config with explicit register types.

Do we need a higher MSRV?

const_panic is in 1.57. I have replaced the unreachable! with a default match case instead for now.

Uh, sorry but commit fc24b44 contains way too many fiddly changes for me to review. It will take me a while to go over it. A mitigation would be splitting into several commits, if that is something you would be willing to do.

Sure, I'll try to split some of this out.

I extracted some of this out into https://github.com/eldruin/lsm303agr-rs/pull/19.

@eldruin I think this should be good to go now. Don't think I can split this further.


bug: group concurrency not being respected

Node.js version
14
NPM version
latest
@checkly/cli version
latest
Steps to reproduce
Bug report: Group Concurrency setting not respected
A group check's concurrency setting is not respected when set. There's a timeout in the setup for 15 seconds for 8 checks. We should see these checks occurring over at a minimum of a two minute time period, but we see that it all happens in one.
Reproduction
https://github.com/modern-sapien/checkly-repro-error/tree/api-setup-failure
What is expected?
Concurrency is respected
What is actually happening?
Concurrency is not respected
Any additional comments?
No response

@tnolet has this been picked up by anyone and maybe just not resolved here?

@tnolet has this been picked up by anyone and maybe just not resolved here?

@modern-sapien no, this ticket has not been addressed yet.

@clample I talked to @tnolet how this behavior should look like. He wants to have a concurrency within the groups but 2 separate groups would run in parallel. Feel free to reach out to him if you have any questions


feat: swap UI improvements

Summary
Bunch of UI improvements. One commit - one improvement.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Get player input

Hi a question im using this lib for a plugin, how i can make user just click and have a to type? Calling an event inside the gui or idk

I don't understand your question. In general, I don't understand the whole sentence.
Please revise your sentence again so that I can help you.

For example a user click in the stone block then the gui close and a title saying "input your message in chat" the user type in chat the sentence.

You can give the stone block an InventoryClickEvent and then close your title and inventory inside it (player.closeInventory())

Yes thats what i mean but i can call some even that runs only when the user clicks it?

I don't quite understand what you are trying to accomplish.
Do you want to do it like this?
You click on an item, it follows that the inventory closes and the player can make his input in the chat.
Once he has made his input, should the inventory open again?

found this https://www.spigotmc.org/threads/player-chat-inputs.437565/ works fine for who is wondering


improve release/downloadIstioCandidate

Please provide a description of this PR:
improve release/downloadIstioCandidate

I would also update downloadIstioCtl.sh

Local testing with commands like ISTIO_VERSION=1.11.0 TARGET_ARCH=armv8 ./testing.sh (I had copied the file to testing.sh) work as expected.
We need to be careful of these files as they back the istio.io URLs.


massive water craters in weird places

Describe the bug
there are massive holes in the ground with stone rims that have water sitting deep at the bottom
To Reproduce
Describe the steps it takes to reproduce the bug
Example:

Join the game
Execute command /tpll 38.678039072306305, -77.25716596226385
compare to a map and terra121 original branch

Expected behavior
i expect it to behave like terra121 and not put massive random holes in the ground
(ye i did just test both mods with freshly generating worlds)
Screenshots

look around this isn't the only one in this area
and i'm saying this doesn't happen in terra121 but does in terra++
Error Logs
n/a
Terra121 or Terra++ (please complete the following information):

i talked about it

Version (please complete the following information):

im using my own custom bundle of mods all having to do with terra121 and building only but that's nnot the problem here

another example that is far less severe than the craters that are next to each other, but still equally as strange in my opinion
/tpll 38.68454315076617, -77.26091994159331

notice in an actual map that this seriously doesn't happen
there is no pushed in terrain like this and terra121 generates it correctly
i would love to use this great generator but it's making earth harder to build on than terra121 atm...

another example that is far less severe than the craters that are next to each other, but still equally as strange in my opinion
/tpll 38.68454315076617, -77.26091994159331

notice in an actual map that this seriously doesn't happen
there is no pushed in terrain like this and terra121 generates it correctly
i would love to use this great generator but it's making earth harder to build on than terra121 atm...

1:
You are running an unreleased, untested and unsupported version of Terramap, please don't report bugs to other projects like this when using it.
It is very unlikely to be the cause of the bug here, so I will just ignore it, but really, please don't, you could be waisting people's time with irrelevant issues.
2:
Which version of terra++ are you running?
It is still a work in progress with regular updates and changes, especially surrounding water handling right now. We can't help if we don't know exactly what which version you have.
3:
As I said, it's a work in progress, so for the moment you still need to be ready to deal with bug when using it (not that we want you to, but you should always plan for the worst).

1:
You are running an unreleased, untested and unsupported version of Terramap, please don't report bugs to other projects like this when using it.
It is very unlikely to be the cause of the bug here, so I will just ignore it, but really, please don't, you could be waisting people's time with irrelevant issues.
2:
Which version of terra++ are you running?
It is still a work in progress with regular updates and changes, especially surrounding water handling right now. We can't help if we don't know exactly what which version you have.
3:
As I said, it's a work in progress, so for the moment you still need to be ready to deal with bug when using it (not that we want you to, but you should always plan for the worst).

i had absolutely no idea terramap could affect generation. this is obviously news to me. i’m about to try without it, even though terramap now requires this project as a dependency...

i just used the latest release terra++ as of yesterday from jenkins and every other mod was built from source (terraplusplus-0.1.405-1.12.2.jar) (build #36 doWater always enabled) (this has been the latest release for three days)

i had absolutely no idea terramap could affect generation. this is obviously news to me. i’m about to try without it, even though terramap now requires this project as a dependency...

i just used the latest release terra++ as of yesterday from jenkins and every other mod was built from source (terraplusplus-0.1.405-1.12.2.jar) (build #36 doWater always enabled) (this has been the latest release for three days)

i removed your mapping application @SmylerMC just to generate the world and this is still definitely a valid, useful bug.

i removed your mapping application @SmylerMC just to generate the world and this is still definitely a valid, useful bug.

This seems to be a problem with the OSM Data, but it probably will be fixed in the next TerraPlusPlus update. The branch Fast-OSM is currently being worked on by DaPorkchop, and it's close to being finished. That will include better data to generate the world with.

This seems to be a problem with the OSM Data, but it probably will be fixed in the next TerraPlusPlus update. The branch Fast-OSM is currently being worked on by DaPorkchop, and it's close to being finished. That will include better data to generate the world with.

i had absolutely no idea terramap could affect generation

It cannot, to clarify a bit: I consider TM to be still in beta and I often let Master in a terrible state, please just don't report issues to other projects when using an "unofficial" build.

i had absolutely no idea terramap could affect generation

It cannot, to clarify a bit: I consider TM to be still in beta and I often let Master in a terrible state, please just don't report issues to other projects when using an "unofficial" build.

This issue also affects the fast-osm branch. Looks like it comes from in the Mapzen dataset. ( Here is one corresponding tile ).
It's probably related with this special check in terra121.

This issue also affects the fast-osm branch. Looks like it comes from in the Mapzen dataset. ( Here is one corresponding tile ).
It's probably related with this special check in terra121.

unable to reproduce using the latest terra++, cc and cwg from jenkins (no other mods):

what projection are you using? have you changed the configuration for overpass or elevation datasets at all?

unable to reproduce using the latest terra++, cc and cwg from jenkins (no other mods):

what projection are you using? have you changed the configuration for overpass or elevation datasets at all?

i'm using build the earth upright model and no changes to other configs

i'm using build the earth upright model and no changes to other configs

uhh i'm also not using minecraft bundled java because of the cert problem i have no idea if that's an issue
i'm using latest java or i don't get trees
i'm aware of letsencryptcraft, i don't have it

uhh i'm also not using minecraft bundled java because of the cert problem i have no idea if that's an issue
i'm using latest java or i don't get trees
i'm aware of letsencryptcraft, i don't have it

i've deleted the config and all other mods, so the config is fresh, and created a brand new world with the bte model and i have the same results as earlier. given i'm using latest java 8

i've deleted the config and all other mods, so the config is fresh, and created a brand new world with the bte model and i have the same results as earlier. given i'm using latest java 8

using bundled java yields exact same results for me.

using bundled java yields exact same results for me.

Got this with the latest commit on fast-osm:

Got this with the latest commit on fast-osm:

@DaMatrix could i get ur terrapp config

@DaMatrix could i get ur terrapp config

i used all the default settings, and the bte preset.

i used all the default settings, and the bte preset.

are you absolutely positive you aren’t using a custom elevation server that’s newer than master

are you absolutely positive you aren’t using a custom elevation server that’s newer than master

@DaMatrix look at the tile I linked above

@DaMatrix look at the tile I linked above

i saw that, but that still doesn't explain why it works perfectly fine for me at that same location :thonk:

i saw that, but that still doesn't explain why it works perfectly fine for me at that same location :thonk:

ok, but i did fresh minecraft and java install

ok, but i did fresh minecraft and java install

I can replicate the same issue with t++ 579.


fix: update attached event's details

Fixes attached event's details.
Screenshot

Signed-off-by:Qiuxia Fan<EMAIL_ADDRESS>

Fixed


Exposing http client for govcr

Hi,
This lib is working nicely for me at the moment.  One small improvement I'd like to make is to expose the httpClient pointer within your client struct.  The main reason is that I want to use the govcr library within my tests.
Happy to take any comments and refactor as you see fit to achieve this goal.
Best Regards,
Mike

I'm having second thoughts about this.  While it is interesting to record and checkout the responses from the winrm service, I'm finding it difficult to use this for my test fixtures.
The <a:messageId> element contains a uuid that changes with each request, making it very difficult for govcr to match the request with a previously recorded track.

Hi @HairyMike ,
I'm not sure I understand your use case.
If you want to mock a WinRM server, you might be inspired by @dylanmei winrmtest.
Otherwise let me know what you want to do, I might be able to help :)
Thanks,

Hey @masterzen,
The plan was to record some real world data using govcr and play it back during my tests.  I'll have a closer look at winrmtest.  Thanks for that 👍


testmodel: what about docunits to test model-related services?

Just create standalone pre-constructed models already filled with entities that are exposed as attributes.
Then use these models to test various services.
In the prof-of-concept PR, I just added 2 simple docunits.

Cool, +1

reroll: 1. call then 'standalone models'; 2. create a subclass per configuration.

Re +1


Feature request: save-data header

This will help save more data on supported sites.

Great idea!
Background information: https://www.ctrl.blog/entry/http-save-data.html
Source code of the extension itself: https://github.com/da2x/save-data/blob/master/background.js


Notify user if their tag may be dropped

Due to the implementation of the no_std backend, it is possible for a tag created by the user to be dropped before it is notified. This makes it  poor idea to use the tag for, e.g. sending data over a channel. Therefore this PR adds a parameter for the with_tag closure that gives the user information on whether or not the tag may be dropped.

Superseded by #75


Gem has runtime dependency that monkey-patches String class

The rake tasks use colorized output (e.g. https://github.com/neo4jrb/neo4j-rake_tasks/blob/master/lib/neo4j/rake_tasks/neo4j_server.rake#L37) which looks cool.
The downside is that this is achieved by including the colored gem, which monkey-patches the String class to add methods like cyan.
Because neo4j-rake_tasks is a runtime dependency of neo4j-core the monkey-patching flows through into any ruby project that uses Neo4J via neo4j-core.
Monkey-patching a Ruby core class is not required to achieve the cool output and it would be better to do this in a more hygienic way that doesn't have potential unintended consequences.
The same effect could be achieved with (for example) the rainbow gem without the side-effects.

This is 100% fair!  I'll make this change shortly.
Separately, though, I would like to remove neo4j-core's dependency on the neo4j-rake_task command, because not everybody uses the rake tasks to set up / manage Neo4j servers.

Released as 0.7.12, thanks!

👍 I was going to submit a PR when I had time, but you beat me to it!


Is there a way to trap unhandled intents?

I got my first round trip working between my test Alexa skill and my alexa-app handler code.  Unfortunately, the intent I defined in the management console was not one I had a handler for since it was a custom intent name I created.  I saw this in the management console test results:
{
  "version": "1.0",
  "response": {
    "directives": [],
    "shouldEndSession": true,
    "outputSpeech": {
      "type": "SSML",
      "ssml": "<speak>Sorry, the application didn't know what to do with that intent</speak>"
    }
  },
  "sessionAttributes": {}
}

Is there a way to define an error handler function to catch unhandled intents that will be fired if an INTENT is received?  If it weren't for that result above, I wouldn't have known about the unhandled INTENT since I didn't see anything in the Console/Log window of my Node.JS IDE.

Recently, Amazon has released a new intent for the purpose of handling phrases that shouldn't be handled via AMAZON.FallbackIntent (only for US English). As for handling unimplemented intents, alexa-app will throw a "NO_INTENT_FOUND" error and you can customize the error message here.


Validation database Vs Issues tracker

The Issue tracker is filled with [Incomplete] Issues which are mostly redundant to the N entries in the VUID database.
One Issue per-VU is unsustainable, and Issue should probably be open only if someone thinks the particular VU validation is extra important. So,
I suggest this [v. 2]:

Make one meta-Issue for all of this

perhaps more; e.g. one for core commands, more for extensions (so it has more definite target for close)
it should contain info such as link to the db, what relevant Issues should contain, which labels to use, and link to filtered and sorted by +1 Issue list

Encourage people to make Incomplete Issues (iff they want to prioritize one VU validation), and +1 vote on them
Link to the meta-Issue heavily, notably from Known Issues section in the documentation

👍  Issues tagged Incomplete receive a higher priority than issues marked  in the database, but this effort would not change that priority.  See issue #1612 for an example of how this might look.

An Enhancement tracking issue would also help with issue clutter.

Yes, except the curated list should not be needed, I think, if some discipline is maintained.
I.e.

such Issues having proper Label
people instructed to only open such Issues for voting purposes (emphasize that we already know all the holes in the layer impl)
people encouraged to add +1 to such issues
such Issues being well searchable (e.g. contain all appropriate VEID codes, and perhaps VUID codes too, and all relevand cmd, struct and param names)

In that case it is possible to make GitHub do all the work, e.g.:
https://github.com/KhronosGroup/Vulkan-LoaderAndValidationLayers/issues?q=is%3Aopen+is%3Aissue+label%3Aincomplete+sort%3Areactions-%2B1-desc

@krOoze, maybe we're already where you want to go.  We rarely open issues (at least internally) against outstanding incomplete VU checks.  The issues that ARE opened, by Google, LunarG, or anybody else, are the highest priority behind bugs.  We could fold this information into the database, but it is captured here and we have medium-term plans to obsolete the database and replace it with a dynamic tool to give the same information.  I do like the idea of folding all of the INCOMPLETE issues into a single tracking issue, but maybe at the cost of some priority, age, etc. Enhancements I might tackle this way though.  Would you be comfortable closing this issue, knowing that we do not create tracking issues for all outstanding VUIDs?

Would you be comfortable closing this issue, knowing that we do not create tracking issues for all outstanding VUIDs?

"we"? What about the general public.
Anyway, it was just an idea. I am always comfortable with it closing as long as you are not interested in implementing it.
It is mostly about esthetics (or OCD?) than anything. The Issue is that there are Issues that practically serve no purpose (and also just contain unnecessary info, where usually just VUID would suffice). And they are bit of a bait; to get rid of one (for the Achievement you know), you would have to fix it -- but then again there might be checks in the DB that probably have better benefit / cost.

@krOoze, I'm totally with you on the OCD -- it is more than a little frustrating to constantly work this list and to have the issue count inexorably inch upwards.  Our stance on the general public creating missing VUID issues is that they must have encountered them In Real Life, making them perhaps more useful than purely technical checks.  Less than a handful of issues are from folks just pointing out the obvious (thank goodness).  And we value all of your ideas -- keep them coming.


Generating Math and Code sample

Could you please provide the code for generating samples of Math.Find, Math.Calc, Code.RUN and Code.debug?
I want to generate some test sample with shorter length, since my model only support 32k context length.
Thanks a lot!

we have updated the synthetic data code for task math_find, math_calc and code_run. As the examples of the task code_debug are almost annotated by humans, there is no end-to-end generation code.


New package: Electrum v0.1.0

Registering package: Electrum
Repository: https://github.com/brainandforce/Electrum.jl
Created by: @brainandforce
Version: v0.1.0
Commit: adc3c928f3bcf969c7f2c93335a4cde82f6d756b
Reviewed by: @brainandforce
Reference: https://github.com/brainandforce/Electrum.jl/commit/adc3c928f3bcf969c7f2c93335a4cde82f6d756b#commitcomment-115011625
Description: A Julia library for working with crystal structures and associated data with the goal of making theory development easier.

[noblock]
For reference, I messaged in the  #pkg-registration channel of the Julia Slack and was instructed to wait 3 days before manually requesting a merge.