ExC template

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@purplecabbage let's close this one, the template as been added to generator-aio-app in https://github.com/adobe/generator-aio-app/pull/4.
This will get automatically part of aio once #113  is merged.
I've moved the UI as is, however as of know if the ui is not running in exc there is an error and it shows a blank screen to the user. I feel this might be confusing for new comers?

wdyt about changing the UI template to mock the exc runtime object when not running in the exc shell, so that the user does not get a blank screen?

This has been quashed by the new generator module
#113


Update PHP deps with fixed Critical Vulns

This pathfinder fork has some pretty old deps worth updating

Can we do something about it ?

Yes, however it's not very easy to solve as code needs a lot of restructuring to behave with PHP 8+.
I've tried numerous solutions to get us there (or at least as close as possible), and with Alpine no longer publishing PHP 7.x in their repositories that puts us in a rough spot.
One possible direction would be to switch to for example Ubuntu 20.04 LTS which will provide us with backported security updates until 2025-04-01. But it requires some rework of the structuring, as service related paths are different between the distros.
I do however have this on my scope, and have already re-worked substantial parts which addresses majority of the above and will prioritize for the next release which I hope we can release within a few weeks.


disableable assertions

state.js needs some assertions because it interfaces with parts of V8
debug API that are volatile across versions. However, we don't want to
be running these assertions in real user code. debug-assert module
allows us to run with assertions enabled in the CI.

I think we also need the environment variable set on mocha.

I think we also need the environment variable set on mocha.

Yes. Users running mocha manually (rather than run-test.sh) will need to make sure that the env. var. is defined. The purpose of the test is the make sure that our CI always runs with the env. var. is defined.
I am not sure if you are requesting a change or making an observation. Can you approve or elaborate on what change you would like to see?

Complete mistype. Meant to say appveyor. Should the environment variable be set on appveyor?

Ah, so appveyor doesn't use run-test.sh? Ofcourse it doesn't. Dx

PTAL.


swipe fixes & little optimizations

The animation is now working when swiping with speed > FLICK_SPEED, and on tap on left/right arrows.

I added various style modifications

paper-ripple implemented !

Fix #51

Thanks for the PR. This includes a huge number of changes, many of which are unrelated, so I can't really merge it all at once. There are some commits that I don't think should be applied, however I want to look into your changes in more detail. Could you create pull requests for individual bugfixes and/or features?


Split unique into separate functions

This PR

resolves gh-212 by splitting unique into separate functions with a fixed number of arguments, thus addressing the concerns regarding unique's polymorphic behavior.

splits unique in the following functions:

unique_all(x, /): returns (1) unique values, (2) indices of first occurrence, (3) reverse indices, and (4) counts. This is the equivalent of the previous unique API with return_index, return_inverse, and return_counts kwargs as True.
unique_inverse(x, /): returns (1) unique values and (2) reverse indices. As discussed in the OP for gh-212, this addresses the fairly common use case of setting return_inverse to True in the previous API.
unique_values(x, /): only returns (1) unique values. This is the most common use of the previous API.

chose to retain return_index as a return value in unique_all as this returned array does have value in being able to construct the array of unique values and because deriving this returned array is somewhat tedious to construct via the objects in this specification otherwise.

eliminates polymorphic return behavior by removing kwargs from the respective APIs which affect which and how many arrays are returned.

adds guidance on expected output array shapes.

requires that named tuples be returned.

does not use the function name unique in order to avoid breaking backward compatibility in array-specification implementations.

I suggest not mixing the "are NaN's unique" question with the discussion on unique being polymorphic and hence this PR to split it into three separate functions. They are orthogonal problems. Could you move your question to the NaN-unique issue @honno?

This PR was discussed during the 7 October 2021 consortium meeting. No objections were raised and the proposed changes were seen as positive. Will merge. Any needed changes can be addressed in subsequent PRs.


Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Name of the service
Whimsical
URL of the service
https://whimsical.com/
GitHub URL to a recipe that has already been created by someone else or you
No response
Features you want implemented
Currently I've added as custom website but it's showing address bar.

Address bar can be disabled - please check the settings


Adding workbook as a part of the solution

Adding workbook as a part of the solution.

@NikTripathi - This workbook can be good addition to the solution.
Request your review for the merge.
Thank you.

@v-spadarthi - Could you please let me know for any changes else help with approval & merge.

Thank you for the approval & merge @v-spadarthi  & @NikTripathi


Make Double Pause and Double Resume a No-op.

This fixes bug https://github.com/XamlAnimatedGif/XamlAnimatedGif/issues/170
Calling pause twice dereferences an active TaskCompletionSource
By checking for and completing the TaskCompletionSource before creating a new one, we don't double stop the async thread if someone makes an error and calls pause twice.
This fixes Issue: https://github.com/XamlAnimatedGif/XamlAnimatedGif/issues/170 https://github.com/XamlAnimatedGif/XamlAnimatedGif/issues/170
From the discussion section of the bug;
I'll do a pull request on the first of the expected behavior; _animator.Play(); resumes animation even if you have paused it twice.
Sure, someone can implement a Boolean, or check _animator.IsPaused first to know if the animation is already paused, however, the documentation isn't clear enough to know that you must do that and calling a double Pause would break the threading model.
When it doesn't work, it's really tough for someone figure out why calling resume isn't working. I just assumed that it was buggy for several months and tried to use it as little as possible.
There isn't any message or other notification that you broke it.
Furthermore, in other frameworks, if you call pause twice or resume twice, the result is it pauses or resumes respectively. The double action is a no-op.
This is the second attempt at the PR after the maintainer requested changes.  Instead of clearing the TaskCompletionSource before making a new one, we're just checking for if the video IsPaused in the Pause and Resume methods returning per guidance by the maintainer.

Thanks @Teravus!


Organize code as libraries/modular

The way I see it, you've now got duplicate code in the repository. With the 'main program' and classes it depends on in the same file, but multiple times.
My guess is this is for distribution reasons (maybe even start up time) ?: so you can download a single file and run it. I've done this myself in the past, so I get why you'd want to do it.
Would it be an idea to organize things in such a way that you can have a script that builds that one file from separate files, maybe based on a specification file to only have the features you want ? And then include at least one ready build file in the Git repo, so you'd always have a single file ready for distribution.

well, its no longer a single file (yes it was few updates ago)
now i'm distributing each updates via pip
this isn't js or node so that you can build it in a single file and share. Standard python package build requires multiple files.
dev-src folder: where the python and html (web view) are separately updated (for the ease  of dev.
src: where after development files are place in 1 file (manually, because there are many left over test lines in dev-src files, so can't use any build.py yet. Sorry)

pyroboxCore.py: the server handler I made for use (like django  and flask)
server.py: the file that handles the IO
init.py: to make package accessible via import
main.py: the reason you can call youtube_dl via shell without actually running python

So all these files are necessary.
if you want to try, install via pip install pyrobox
also https://github.com/RaSan147/pyrobox/releases

https://github.com/RaSan147/pyrobox/releases/tag/v0.5
this was the last time i made single file (gets messy when things gets large)

oh my bad (english not 1st language)
maybe you are talking about same class or functions exist multiple times?
well, because i'm doing some performance test and thus made multiple similar things (and yes it worked)
i'll clean up soon, thanks for the headsup

well most of the functions depend on each other and share variables (config stuffs)
so it'd take time if i try to separate these things. 😅

i'll fix it today

the only issue keeping me away from doing so is relative import issue. Packages use relative import
from .exceptions import LimitExceed
but in dev env
ImportError: attempted relative import with no known parent package
its kinda painful to update them every time. So made it one file

okk i tried and i'm retreating, big mistake
most of the data is entangled with each other and and untieing them made a huge mess, i'm leaving this issue open
if i have more time and a clear mind and ofcourse more EXP, i'll retry

@Lennie  I tried my best (can do better with a little more time)


chore: release master

:robot: I have created a release beep boop
configs: 1.14.0
1.14.0 (2023-06-02)
Features

test release wtf (a9cd8b9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wholesome-ghoul/allaround-configs/releases/tag/configs-v1.14.0 :sunflower:


detectPublicIPv4: try to detect ipv4's from interfaces with a default gateway first

detectPublicIPv4: try to detect ipv4's from interfaces with a default gateway first

adds github.com/vishvananda/netlink to get at the routing tables
if none found, old logic is kept (gets the first IPv4 from the first interface that has one)

Signed-off-by: Ricardo Pardini<EMAIL_ADDRESS>

Yay got it green, finally. Sorry for the delay...


Update system requirements page

See https://trello.com/c/n5cfCmiN/43-sys-requirements
I've updated the page to remove the pointless minimum and use a more recently suggested spec as an example. Also improved the layout.
Staged at https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/system-requirements.html
I could add the last few examples from the sysadmin questionnaire results to https://www.openmicroscopy.org/site/support/omero5.3-staging/sysadmins/server-setup-examples.html but given they are all 5.0.x and the most recent is 1.5 years old, I'm really not sure it is worth it now.

 Thank you.


docker image is dead after starting

when running docker run -ti --rm sunflowerpockets/ukrainian-warship:latest
Acquiring targets...
⣽panic: Could not get targets...

goroutine 1 [running]:
github.com/seedpockets/ukrainian-warship/cmd.Kill(0x200)
        /app/cmd/kill.go:105 +0x309
github.com/seedpockets/ukrainian-warship/cmd.AutoKill(0xc0001bdd88)
        /app/cmd/kill.go:73 +0x8f
github.com/seedpockets/ukrainian-warship/cmd.glob..func1(0xc26460, {0x8733fa, 0x0, 0x0})
        /app/cmd/kill.go:43 +0x2f
github.com/spf13/cobra.(*Command).execute(0xc26460, {0xc5fdc8, 0x0, 0x0})
        /go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:860 +0x5f8
github.com/spf13/cobra.(*Command).ExecuteC(0xc266e0)
        /go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:974 +0x3bc
github.com/spf13/cobra.(*Command).Execute(...)
        /go/pkg/mod/github.com/spf13/cobra@v1.3.0/command.go:902
github.com/seedpockets/ukrainian-warship/cmd.Execute()
        /app/cmd/root.go:46 +0x25
main.main()
        /app/main.go:21 +0x17

I'm currently working on a fix for this.

I rewrote the kill command, this should solve your issue.


The json output of typedoc is inconsistent with html output

I want to output a json file for azure-event-hub package and only output those types or classes export form index.d.ts.
So I use this command, but none of those  types/classes is generated.
Typedoc   \typings\lib\index.d.ts --json api.json --ignoreCompilerErrors --includeDeclarations
If I use the following command to generate html output, it works as what I want
Typedoc   \typings\lib\index.d.ts --out  --ignoreCompilerErrors –includeDeclarations
Could you check why the output of json and html are inconsistent?

Thanks for the bug report. The json output could use improvement. It would be helpful if there was a small project where we could reproduce this.

Please just use npm package azure-event-hub to reproduce it.

npm install azure-event-hub
under the package azure-event-hub, execute typedoc  by specifying source  \typings\lib\index.d.ts
Thank you!


chore(starknet_l1_provider_types): add crate

Only move, no logic changes, this is a setup for upcoming infra support.

This change is


Add extra acl, gce and ec2 config variables

Changes:

Add retry_join_ec2 variables (Fixes #192)
Add retry_join_gce variables
Add acl_enforce_version_8 variable
update README to show existing acl capabilities

ping @savagegus ? :)

Is @savagegus still there? There's a number of issues like this one that are pending merges or were merged but never released. #184, #183.
Does anyone else but @savagegus have admin rights to this repo?

@savagegus Hi! Can this be merged soon?

@aairey Can you make sure that start_join is not defined when ec2 or gc2 options are used?

@aairey sorry my bad, i can make consul_join_at_start: false :)

@mattfinlayson is this still relevant?

I haven't used this in many years, so I don't know.

Same
Sent from my iPhone

On Jul 14, 2022, at 10:33 PM, Nimrod Wandera @.***> wrote:

I haven't used this in many years, so I don't know.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


Update D2 Admin url and online demo

url https://github.com/d2-projects/d2-admin
demo https://d2admin.fairyever.com

I'm sorry for my previous behavior, but now I've put my project in its original place, just changed the name and link address.


How to add a devserver before or after hook？

How to add a devserver before or after hook？
const webpackMock = (config) => {
  console.log(config);
  if(!config.devServer) { config.devServer = {}; }
  config.devServer.before =function (app) {
    apiMocker(app, path.resolve('./mock/index.js'))
  }
  return config;
}
module.exports = [
  webpackMock
]

It doesn't work

Apologies that I'm only just now seeing this! Try removing the braces around webpackMock. Should work fine. Let me know if you run into more trouble.


Undefined index: Authorization in /home/www/dyndns/update.php on line 13

I have a small issue with this and I think I need some help to troubleshoot it.
The script works great in almost anyway I tried, except with my Draytek router when using https.
http works fine though, curl request works fine too, but with my router it won't work when I select https protocol and logs the error in the title to error_log on the server Undefined index: Authorization in /home/www/dyndns/update.php on line 13 - line 13
The same error is logged in error_log if I access the web page from the browser without any auth parameters.
Here's a screenshot of the config:

By just changing the connection type from http to https the error is logged and the update fails.
Any clue how I could debug this?
Thank you!
P.S: I'm wondering if https cannot be used with Basic Auth and maybe I must change from Basic auth to URL (like curl) along these lines: https://superuser.com/a/1545384

A quick test using hardcoded plaintext credentials as parameters in the url worked with https on the router.
$requestUser = $_GET['usr'];
$requestPassword = $_GET['pwd'];

/update?hostname=host.domain.com&usr=myuser&mypassword=4vM1YDTduA0ut&myip=<IP_ADDRESS>"
I need a better way to test why it won't work with Basic Auth + SSL.

A quick test using hardcoded plaintext credentials as parameters in the url worked with https on the router.
$requestUser = $_GET['usr'];
$requestPassword = $_GET['pwd'];

/update?hostname=host.domain.com&usr=myuser&pwd=mypassword&myip=<IP_ADDRESS>"
I need a better way to test why it won't work with Basic Auth + SSL.

Sorry for the delay. I'll need to look at and think about what the limitations to your application actually are. Supplying credentials in a GET request is insecure, even over HTTPS. what you COULD do, and what I'm actually doing right now for my own use with another person is creating a unique URL similar to the test client. Simply hitting that URL will update with the credentials provided in the PHP file.
I can show an example when I get mine working.

Okay, I have an update. While scripting I noticed that the request probably isn't being made properly if you get the
Undefined index: Authorization in /home/www/dyndns/update.php on line 13
We can fix this by fixing the request. To do this, we need to issue the correct CURL command or fix something else.
in the short term you can try the preconfigured URL approach I just uploaded. It isn't perfect security, but it should work in the short term.
As far as working with your specific router, I'm interested in looking at the options you have available...does it use ddclient under the hood?

Supplying credentials in a GET request is insecure, even over HTTPS. what you COULD do, and what I'm actually doing right now for my own use with another person is creating a unique URL similar to the test client.

Thanks for supplying that script, I might use that approach in the future although it introduces a lot of rigidity to the process.
At the moment I don't really mind using the GET parameters in the URL, mainly because their only purpose it to match a hostname using a user/password combination, nothing sensitive about them.

As far as working with your specific router, I'm interested in looking at the options you have available...does it use ddclient under the hood?

That's a good question. I don't think it uses ddclient, but I also am not sure what it uses. The only thing I can see is the config file when I backup the router configuration, and goes something like this:
config 'profile' 'ddns1'
	option 'uciid' '1'
	option 'ifname' 'wan1'
	option 'policy' 'first'
	option 'wild_card' 'disable'
	option 'backup_mx' 'disable'
	option 'mail_extender' ''
	option 'isShow' 'true'
	option 'acme' '0'
	option 'last' ''
	option 'rfilter' 'isShow'
	option 'rvalue' 'true'
	option 'provider' 'CUSTOMIZED'
	option 'provider_host' 'ddns.domain.com'
	option 'auth_type' 'basic'
	option 'login_name' 'myuser'
	option 'login_pw' 'mypassword'
	option 'ip_source' '0'
	option 'time_interval' '1'
	option 'exp_date' ''
	option 'domain_name' 'host.domain.com'
	option 'server_resp' 'good'
	option 'server_type' 'dynamic'
	option 'service_api' '/update?hostname=host.domain.com&myip=###IP###'
	option 'connection_type' 'https'
	option 'status' 'enable'
	option 'force_update' '0'

Do you have a valid certificate for https for your server hosting this script? I’m not sure of the behaviour of the certificate is not valid.

Yes, the certificate on the server is valid. I think this might just be a bug with the router, although I'm not sure how the request changes between plain http and https.

Yes, the certificate on the server is valid. I think this might just be a bug with the router, although I'm not sure how the request changes between plain http and https.

what options are available under "Service Provider" and "Service Type"? My code expects "dyndns2" protocol, so maybe you need to correct something in that selection.

what options are available under "Service Provider" and "Service Type"? My code expects "dyndns2" protocol, so maybe you need to correct something in that selection.

I assume it uses dyndns2, because it works fine in http mode. Service provider lists a dozen ddns providers, dyndns, no-ip, etc. Service type has 3 options, dynamic, static, custom. Selecting any of the doesn't seem to allow access to further options, I'm unsure what it is used for. Using http it works with dynamic out of the box.

Hmm, is your cPanel using Apache or Lightspeed for the web server or something else? The code uses
$requestAuthString = apache_request_headers()['Authorization'];
so when https is selected this header appears to not be present. Maybe you can dump the entire apache_request_headers() variable to see how it differs between http and https?

Apparently the router uses wget (at least for https) and doesn't use Basic Auth at all in https mode, so it is a bug that can only be worked around with the URL solution.
Here's an example of the current behavior:
Draytek DDNS client (http)
Array
(
    [Host] => ddns.domain.com
    [X-Real-IP] => _host-public-ip_
    [X-Forwarded-By] => _ddns-server-ip_:80
    [Connection] => close
    [Authorization] => Basic _HASH_
    [Content-Type] => application/x-www-form-urlencoded
    [Cache-Control] => no-cache
    [User-Agent] => Vigor2960 series
    [Content-Length] => 0
)

Draytek DDNS client (https)
Array
(
    [Host] => ddns.domain.com
    [X-Real-IP] => _host-public-ip_
    [X-Forwarded-By] => _ddns-server-ip_:443
    [X-Forwarded-Port] => 443
    [X-Forwarded-Proto] => https
    [Connection] => close
    [User-Agent] => Wget/1.16 (linux-gnu)
    [Accept] => */*
    [Content-Length] => 0
)

Indeed it does seem like a bug. That's a really weird bug. Why should it matter http or https? Oh well, have any suggestions for alternative work around I can help with?

Oh well, have any suggestions for alternative work around I can help with?

Not really, no, I think there are already quite a few acceptable alternatives to basic auth. In any case, I'm happy with the way it is, this was just one of those minor nuisances that I wanted to try to figure out if I was doing something wrong which apparently wasn't the case.
Thanks for all the suggestions, and mainly the development of the core script itself, it's extremely useful.

as an alternative you might consider looking at https://www.dnsomatic.com/ . I used them for a bit when I did ddns through Google and somewhere else. dnsomatic can be configured to look like dyndns2 protocol, so you could try that one and see if you have different results.


Windows build of version 6.0.2

Just wondering if there is a windows build of this latest version somewhere?
Thanks

Hi, 6.0.2 is same as 6.0.1, the only difference is the website mentioned in the software has been updated to "github.com/adriancs2/cti-text-encryption". In old version, it mentioned "ctiencrypt.codeplex.com".

You may however download the latest version at here:
https://github.com/adriancs2/CTI-Text-Encryption/releases/tag/37a41f2

thanks so much


directories shouldn't be created before ceph installs

Bug Report
What happened: ceph-ansible creates directories and sets permissions before installing Ceph
What you expected to happen: Directory creation and permission setting should happen by the packaging itself. If the packaging rules aren't setting paths or permissions correctly, it is a bug in Ceph, not ceph-ansible
How to reproduce it (minimal and precise): This task is running:
ceph-config : create ceph initial directories owner={{ ceph_uid }}, group={{ ceph_uid }}, mode=493, state=directory, path={{ item }}] Which does things like:
changed: [mon0] => (item=/var/lib/ceph/bootstrap-osd) => {
    "changed": true,
    "gid": 167,
    "group": "ceph",
    "item": "/var/lib/ceph/bootstrap-osd",
    "mode": "0755",
    "owner": "ceph",
    "path": "/var/lib/ceph/bootstrap-osd",
    "secontext": "system_u:object_r:ceph_var_lib_t:s0",
    "size": 6,
    "state": "directory",
    "uid": 167
}

Environment:

OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Docker version if applicable (e.g. docker version):
Ansible version (e.g. ansible-playbook --version):
ceph-ansible version (e.g. git head or tag or stable branch):
Ceph version (e.g. ceph -v):

This is only needed for containers, let me fix this. Thanks.


Return body after client error.

Hello ,
Why Guzzle didn't return result even I get a client error .
E.g : I may have 400 bad request but :

HTTP 400 BAD REQUEST
Content-Type: application/json
Vary: Accept
Allow: POST, OPTIONS

{
-----------------------------------------------------------------
|    "errors": "amount or message must be set",                 |
|    "error_code": "missing_field"                              |
-----------------------------------------------------------------
}

Syntext used :
$response = $this->client->post($Endpoint, ['query' => array_merge($data , $Query)]);
Sometimes I may need this result to identify the problem ....

Thanks , I think I solved this with :

	 catch (Exception $e) {
		echo $e->getResponse()->getBody();
	}


Add Multiple vnet in single firewall

is this possible to add multiple vnet in single firewall?

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

ID: 6bba1b4d-cf65-c0a2-9bb6-4ff7896f0a15
Version Independent ID: 7ab0db77-e78d-de58-369f-641782327264
Content: Azure Firewall FAQ
Content Source: articles/firewall/firewall-faq.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

@cyrusvivek
Thanks for your feedback! We will investigate and update as appropriate.


About Dataset Padding

Thanks for providing this beautiful code and documentation!
I have been reading the implementation of Dataset and SequenceSampler in the Colab example and I have a question about it.
def create_sample_indices(
        episode_ends:np.ndarray, sequence_length:int,
        pad_before: int=0, pad_after: int=0):
    indices = list()
    for i in range(len(episode_ends)):
        start_idx = 0
        if i > 0:
            start_idx = episode_ends[i-1]
        end_idx = episode_ends[i]
        episode_length = end_idx - start_idx

        min_start = -pad_before
        max_start = episode_length - sequence_length + pad_after

        # range stops one idx before end
        for idx in range(min_start, max_start+1):
            buffer_start_idx = max(idx, 0) + start_idx
            buffer_end_idx = min(idx+sequence_length, episode_length) + start_idx
            start_offset = buffer_start_idx - (idx+start_idx)
            end_offset = (idx+sequence_length+start_idx) - buffer_end_idx
            sample_start_idx = 0 + start_offset
            sample_end_idx = sequence_length - end_offset
            indices.append([
                buffer_start_idx, buffer_end_idx,
                sample_start_idx, sample_end_idx])
    indices = np.array(indices)
    return indices

I understand that we need to pad $T_o-1$ (pad_before) steps before an episode because we need to make sure the first action is predicted conditioning on observation of length $T_o$. But why do we need to pad after an episode for $T_a-1$ (pad_after) steps? What is the problem if we make the max_start to be episode_length-sequence_length?

Hi @yuanfei-Wang, in several of our simulation datasets, it is desireable if the robot stays static once the task is finished (in particular the Push-T task which terminates immeidately after the T is inside the goal region). If set we max_start to episode_length-sequence_length or equiveltenly set pad_after=0, the last few observations of the dataset will never get supervised on, which will lead to performance degredation on certain benchmarks. In particulary for the Push-T task, if we set pad_after=0, the poliy sometimes will suddenly make an undesired move right before completing the task, since the observations right before completed the task are out of distributions.

Thanks for your insightful explanation! Now I only have a minor question. Why is pad_after set to be $T_a-1$ instead of other values? Is there any specific reason or did you try other options?

Hi @yuanfei-Wang, we would like to have the policy to observe all images in the training dataset. In order to do this, need to set pad_after to T_a - 1 (since the first action step is aligned with the last observation step). In most of our simulation benchmarks, it is desired for the robot to stay still after finishing the task, which is what actions will be set inside paddings ("same" padding for positional actions). If this behavior is not desired, you should set pad_after to 0.

Hi @cheng-chi, now I understand why pad_after is set to $T_a-1$. Thanks for your detailed explanation!

Hi @cheng-chi, I think if we set pad_after = Ta - 1, the last few observations are still not supervised (specifically, last horizon - To - Ta + 1 obs) because only the first To obses are used for conditioning. Why not set pad_after to horizon - To (then the last sample conditions on obs[-To:], i.e., all obses are leveraged) or horizon - To - Ta + 1 (then first Ta actions in the last sample are still valid)?

Hi @cheng-chi, I think if we set pad_after = Ta - 1, the last few observations are still not supervised (specifically, last horizon - To - Ta + 1 obs) because only the first To obses are used for conditioning. Why not set pad_after to horizon - To (then the last sample conditions on obs[-To:], i.e., all obses are leveraged)?

I have the same question.

@ManUtdMoon @tongzhoumu You're right, but I think due to them defining Ta = horizon - To + 1 for their experiments, setting pad_after=Ta-1 is equivalent to setting pad_after=horizon-To.
As far as I can tell, though, you seem to be correct that setting pad_after=horizon-To is better.


I can't open the website of dataset.

Could you share other links?

Hi @fxy2012 ,
The links work fine for me. Are you in China? Since I am hosting the data in Dropbox, maybe the website is blocked by the firewall. In any case, you should be able to open  the link of the data in a standard browser. You can just copy everything after wget and paste in a browser. e.g., for human seg:
https://github.com/ranahanocka/MeshCNN/blob/29c30a4d6dee123cacbb662e3f5666420c0e94c5/scripts/human_seg/get_data.sh#L8

Thank you for your reply. So can the codes support pytorch 1.0 with cuda9.0?
I ran with pytorch 1.0, cuda9.0, but the following error occurred.
TypeError: can't convert np.ndarray of type numpy.bool_. The only supported types are: double, float, float16, int64, int32, and uint8.

Hi @fxy2012 ,
The newest version of the code uses pytorch 1.2, you can install the environment as in the readme.
Alternatively, you can check out an older version of the code which used Pytorch 1.0 (i.e., run git checkout cf93099)


Integrate experimental-foveation branch

Integrates the commits from:
https://github.com/bjoeris/bigwheels/tree/experimental-foveation
into BW main's branch. It contains the following commits

Add util functions to fill foveation pattern bitmaps
Add BlitImage (Vulkan Only)
Add foveation benchmark
Add multisample support for ShadingRatePattern.

Thanks Jorge! Any chance we can split this into separate PRs for each commit/meaningful change instead? This much code is a bit daunting to review.

I have split the PR into the original commits.
The last one also included my won addition to integrate.
I am looking into the shader compilation problem on Linux

Since you already have this in separate commits, would it be possible to split into PRs?
It would be much easier/faster to review, and address review feedback, feedback is supposed to be addressed as separate commits.

I follows Aliya's advice, I update this PR to contain only one of the CL in the original merge.
I create the nexts ones when this gets merged.

Thanks Jorge for taking care of merging these.
I don't remember exactly what benchmarking you're planning to do, so I just want to make sure these functions are actually needed. I added them specifically to test FDM and VRS independent of OpenXR foveation. These let the application use it's own density patterns.
In normal OpenXR foveation, the foveation pattern comes from OpenXR, via the application via XrSwapchainImageFoveationVulkanFB in the next chain of xrEnumerateSwapchainImages. For this use case, we would want to build a ShadingRatePattern around the density image from OpenXR.

Hi Benson,
Yes, the idea is to port all your changes that lead to the foveation_benchmark app.
To my undestanding these is a CL that is part of four required ones (correct me if I am worng here)
I want to integrate all four of these:

Add util functions to fill foveation pattern bitmaps
Add BlitImage (Vulkan Only)
Add foveation benchmark
Add multisample support for ShadingRatePattern.
Since they are al required to get the app working.


Keyboard Manager - Remapping F4 & F6 also remaps the Surface Vol. hardware buttons

ℹ Computer information

PowerToys version: 27.1
PowerToy Utility: Keyboard Manager
Running PowerToys as Admin: n/a
Windows build number: [run "winver"] Win 10/20H2

📝 Provide detailed reproduction steps (if any)

… On a Surface Pro 7, remap the volume up/down buttons (f4/f6) to page up/down
… Press volume up or down button on the hardware buttons (on the top of the device, near the power button)
… Observe that volume up & down also do page up & down.

✔ Expected result
Hardware volume buttons should still change the volume; F4 & F6 should be correctly remapped.
❌ Actual result
Both F4 & F6 AND hardware volume up & down now are remapped to the same value
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

@notandrew can you take a screenshot of your mappings just for sanity.  Only laptop i have with hardware buttons is my Wife's SP2 so kind of want to do minimal take over of that :)

Turns out, a Surface Pro 2 keyboard doesn't have volume up / down keys.

I just tried this on my Surface Pro 4 and it works (funny trick, too). The thing is that the physical hardware buttons trigger a simple [VolUp] or [VolDown] and PowerToys would have to differentiate them from (also physical) keys on a keyboard. I'm not sure of the inner workings, but I'm thinking that might be difficult.
Google found me the same question in Auto Hotkey forum, without any answer.
Apparently there's an app that remaps the physical volume buttons on your smartphone as wel.

Also, it looks like the 'brightness up' button isn't remappable to F2. Is this intentional/known? (I never want to adjust brightness but daily, I want to rename things and can never figure out if I need the Fn button pressed or not to trigger it)

The brightness key on a surface pro? If I'm right that's for the keyboard backlight and as far as I know that is handled by the actual keyboard. So, I'm afraid we can't do much with it.

@notandrew issue is a lot of those keys do 'special' things that are outside the VKEY range.  Office key is an example of that.  Our code just doesn't see them.

@notandrew issue is a lot of those keys do 'special' things that are outside the VKEY range.  Office key is an example of that.  Our code just doesn't see them.


[Tooltip] Not being placed correctly when used with FormControlLabel checkbox

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example: https://stackblitz.com/edit/react-qqwxar-c3zrv8?file=demo.tsx
Steps:

Pass bottom-end to placement in tooltip
Pass bottom-start to placement in tooltip
compare both

Current behavior 😯
After passing position parameters in the tooltip both position start and position end is working same.

Expected behavior 🤔
position start and position end should not be same.
Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Can I work on it?

@gzrae  can I work on this?

@gzrae  @siriwatknp  can I work on this?

This should work. Looks like a bug.

hi @ZeeshanTamboli
if anyone is not working on it, can you please assign this issue to me?

Hi @OmPr366 , just try adding a sx={{ml:0}} in your FormControlLabel may be tooltip is not getting enough room to show the tooltip

And another way to fix this issue is to wrap div into Grid and display them as flex and direction column

@Vivek-Prajapatii In the CodeSandbox you provided, "Bottom End" placement doesn't work as expected.

@Vivek-Prajapatii Thanks for providing the solution. I do see that as a workaround, though. Why was margin-left: -11px initially included? I found a similar problem with margin-right at https://github.com/mui/material-ui/issues/34039. So similarly, here, it would be a breaking change to remove it now. Lets keep it tracked it for v6.

@Vivek-Prajapatii Thanks for providing the solution. I do see that as a workaround, though. Why was margin-left: -11px initially included? I see that the values got changed in #8173 and #15097. But why was it added in the first place? There must be some reason. I think it got turned into a bug now. I found a similar problem with margin-right at #34039. Like there, similarly here, it would be a breaking change to remove it now. Lets keep it tracked it for v6.

sure @ZeeshanTamboli , thank you

@OmPr366 @ZeeshanTamboli we would like to pick this up

@gitstart, as mentioned earlier, this change might be breaking, so it's better suited for v6. However, if you have a non-breaking solution, please go ahead and create a PR.


Develop API

All URIs should adhere to the URL standards for GOV.UK.
API technical and data standards (v2 - 2019)
Catalogue Endpoints

[ ] Endpoint to return a list of all RDF graphs uploaded to the repos (perhaps searchable?)

requires paging

[ ] Endpoint to return a list of the historic versions uploaded for a particular RDF graph

requires paging

[ ] Endpoint to upload a versioned CSV-W to the repository (An RDF version of this should be automatically generated on upload).
[ ] Endpoint to upload a versioned RDF file to the repository.

N.B. When uploading, nothing should be returned in the 'live' listing endpoints until the whole ingestion/indexing process has been completed.
Content Acquisition Endpoints

[ ] Endpoint to download the RDF (in ttl format) for a given version (0.1) of a graph (some-graph-identifier)

http://some-base-url/graphs/some-graph-identifier/0.1 with no Accept header OR Accept: text/plain (until we develop a suitable human-friendly web page) OR Accept: text/ttl
http://some-base-url/graphs/some-graph-identifier/0.1/ttl with any or no Accept header
http://some-base-url/graphs/some-graph-identifier/0.1/rdf with any or no Accept header
We could add more formats of RDF in the future?

[ ] Endpoint to download the CSV (if available) for a given version (0.1) of a graph (some-graph-deitnfier)

http://some-base-url/graphs/some-graph-identifier/0.1 with Accept: text/csv
http://some-base-url/graphs/some-graph-identifier/0.1/csv with any or no Accept header

[ ] Endpoint to download the CSV-W metadata JSON (if available) for a given version (0.1) of a graph (some-graph-deitnfier)

http://some-base-url/graphs/some-graph-identifier/0.1 with Accept: application/csvm+json
http://some-base-url/graphs/some-graph-identifier/0.1/csvw with any or no Accept header

[ ] The above content acquisition endpoints should accept latest as a version, e.g. http://some-base-url/graphs/some-graph-identifier/latest. This should return the data associated with the latest version uploaded.
[ ] http://some-base-url/graphs/some-graph-identifier should perma-redirect to http://some-base-url/graphs/some-graph-identifier/latest

Do we want to support the memento protocol?

Do we want to automatically generate a CSV-W from a given RDF? (Though clearly not something we're building now.)

Do we want to automatically generate a CSV-W from a given RDF? (Though clearly not something we're building now.)

I don't think so. There's no way to know we'd be making a sensible CSV-W. Which records do we decide are the ones from the RDF which should be present in the CSV and which ones need to go in the CSV-W? How do you 'correctly' decide?


Hotfix: Added missing commands to the Settings page

Before:
In the Settings page (Cmd + , on Mac), certain git commands were missing in the search and were also marked as not recognized when using them:

Now:
The Settings page correctly returns the missing git commands and they are recognized by the editor:

@wabirached, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

Those settings are gone, that's why they are reported missing.

@joaomoreno even the enableLongCommitWarning one? I see it being used in the code, so does that mean that it's always enabled and cannot be turned off?

We're currently missing that feature, needs reimplementation: #21144


respect SUPRESS_EMBEDS flag

does discord not have message flag static class somewhere? I don't like magic numbers

i don't think they do have such a class. Even discord itself just uses magic numbers. Here's an example discord's code that checks flags:
https://gitdab.com/Juby210/discord-jadx/src/branch/alpha/app/src/main/java/com/discord/models/message/Message.java#L618
and i've checked, there's no hasEmbedSuppressed function or anything similar.

Even discord itself just uses magic numbers

Because the java compiler inlines final ints


Include FetchContent/CPM packages in sdist

Hey there,
thanks for the effort to create this package.
At the moment I am using legacy scikit-build in my project and I am using CPM to get build dependencies.
Since there was a requirement in our group to be able to build the package without Internet connection, I created a custom sdist command, to download the dependencies before creating the dist: setup.py
import os
import shutil
import subprocess
import tempfile

import setuptools
import skbuild

class sdist(setuptools.command.sdist.sdist):
    """Custom sdist command to include deps in source distribution."""

    def run(self):
        here = os.getcwd()
        dep_dir = os.path.join(here, "<dep-folder>")
        # remove potentially existing dir
        shutil.rmtree(dep_dir, ignore_errors=True)
        # download dep by calling cmake (CPM will download correct version)
        cmake = skbuild.constants.CMAKE_DEFAULT_EXECUTABLE
        with tempfile.TemporaryDirectory(dir=here) as tmp_dir:
            subprocess.run(
                (cmake, f"-B{tmp_dir}", f"-DCPM_SOURCE_CACHE={here}"),
                capture_output=True,
                cwd=here,
            )
        # CPM stores deps in folder with git-hash as name
        if not os.path.exists(dep_dir) or len(os.listdir(dep_dir)) != 1:
            shutil.rmtree(dep_dir, ignore_errors=True)
            raise RuntimeError("could not download deps.")
        hash_dir = os.path.join(dep_dir, os.listdir(dep_dir)[0])
        # copy contents to '<dep-folder>' top-folder
        shutil.copytree(
            hash_dir, dep_dir, copy_function=shutil.move, dirs_exist_ok=True
        )
        shutil.rmtree(hash_dir)
        # run sdist
        super().run()
        # remove deps dir again
        shutil.rmtree(dep_dir)

skbuild.setup(
    ...
    cmdclass={"sdist": sdist},
)

I know this is kind of hacky, but it does exactly what I want. CPM is instructed in the CMakeLists file to use the local folder for the dependency if it is present.
Do you think, there is an option to create workflows like these with scikit-build-core? My approach depends on setuptools, so when there is setuptools support in the future it could be possible to use it with scikit-build-core as well, right?
Or do you have any opinion if this could be done better?
Cheers,
Sebastian

Yes, I think there should be such an option in the future. Designing it will be tricky, so I'll probably work on finishing editable installs and getting a few more low-lying fruit before trying to tackle this properly. I think these are the initial ideas:

Running cmake in the sdist step - also useful for storing CMakeConfig's (pybind11). Easy enough to add an option, but what gets run and stored?
Providing a hook or an option to run a hook executing Python. Probably will do several things (like selecting what to do if there's a failure)
Plugin system for looks like CPM. Providing the cmake files is easy, the harder thing is integrating it with (the above?)

You should be able to do this already with git submodules & force-include... Oh, I haven't added force-include yet, never mind. I expect we'll have hatchling's force-include in the future.
Of course, the setuptools support should be able to do this as well. skbuild.constants.CMAKE_DEFAULT_EXECUTABLE is the only thing that might change above. The main thing here is that cmake/ninja is not required for the isolated SDist step, so whatever config we select above should also enable cmake/ninja in the SDist step.

when there is setuptools support in the future

Well, technically, the support is there now, it's just very experimental, a bit incomplete, and subject to change. ;)

Thanks for your insights!
So, skbuild.constants.CMAKE_DEFAULT_EXECUTABLE isn't something I should rely on? Is there already a future-proof way to get the cmake executable?

I don't think it will go away, it will likely be a backward-compat item in scikit-build.

Bundling dependencies like that might be some licensing issues. There is also the issue of which library is picked depending on LD_LIBRARY_PATH values and equivalent logic of the OS. Why are you not building the upstream packages there as static libraries?

This is possible I believe, in the last version or two, with sdist.cmake = true.


Adds message_transform link from SFTDataset docstring to docs

Context
What is the purpose of this PR? Is it to

[ ] add a new feature
[ ] fix a bug
[x] update tests and/or documentation
[ ] other (please add here)

Fixes https://github.com/pytorch/torchtune/issues/1331
Changelog
What are the changes made in this PR?

Adds message_transform link from SFTDataset docstring to docs

Test plan
Please make sure to do each of the following if applicable to your PR. If you're unsure about any one of these just ask and we will happily help. We also have a contributing page for some guidance on contributing.

[x] run pre-commit hooks and linters (make sure you've first installed via pre-commit install)
[ ] add unit tests for any new functionality
[x] update docstrings for any new or updated methods or classes
[ ] run unit tests via pytest tests
[ ] run recipe tests via pytest tests -m integration_test
[ ] manually run any new or modified recipes with sufficient proof of correctness
[ ] include relevant commands and any other artifacts in this summary (pastes of loss curves, eval results, etc.)

UX
If your function changed a public API, please add a dummy example of what the user experience will look like when calling it.
Here is a docstring example
and a tutorial example

[x] I did not change any public API
[x] I have added an example to docs or docstrings

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 23.85%. Comparing base (5d1866f) to head (4502312).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2219   +/-   ##
=======================================
  Coverage   23.85%   23.85%
=======================================
  Files         344      344
  Lines       20658    20658
=======================================
  Hits         4928     4928
  Misses      15730    15730

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Build is failing

Gif of wierd pub key error while trying to run my own redmine stack.

W: GPG error: http://deb.debian.org/debian bookworm InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 0E98404D386FA1D9 NO_PUBKEY 6ED0E7B82643E131 NO_PUBKEY F8D2585B8783D481
E: The repository 'http://deb.debian.org/debian bookworm InRelease' is not signed.
W: GPG error: http://deb.debian.org/debian bookworm-updates InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 0E98404D386FA1D9 NO_PUBKEY 6ED0E7B82643E131
E: The repository 'http://deb.debian.org/debian bookworm-updates InRelease' is not signed.
W: GPG error: http://deb.debian.org/debian-security bookworm-security InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY 54404762BBB6E853 NO_PUBKEY BDE6D2B9216EC7A8
E: The repository 'http://deb.debian.org/debian-security bookworm-security InRelease' is not signed.
E: Problem executing scripts APT::Update::Post-Invoke 'rm -f /var/cache/apt/archives/*.deb /var/cache/apt/archives/partial/*.deb /var/cache/apt/*.bin || true'
E: Sub-process returned an error code

I released that yesterday and it built without any GPG errors, I just kicked off another build and still no errors.  I suspect that maybe this is something to do with your network, perhaps a PROXY or company firewall issue?
#7 [ 2/10] RUN apt-get update  && DEBIAN_FRONTEND=noninteractive apt-get install --no-install-recommends -y       supervisor logrotate nginx mariadb-client postgresql-client ca-certificates sudo tzdata       imagemagick subversion git cvs bzr mercurial darcs rsync locales openssh-client       gcc g++ make patch pkg-config gettext-base libc6-dev zlib1g-dev libxml2-dev       default-libmysqlclient-dev libmariadb-dev libpq5 libyaml-0-2 libcurl4 libssl3 uuid-dev xz-utils       libxslt1.1 libffi8 zlib1g gsfonts vim-tiny ghostscript sqlite3 libsqlite3-dev  && update-locale LANG=C.UTF-8 LC_MESSAGES=POSIX  && gem install --no-document bundler -v 2.4.22  && rm -rf /var/lib/apt/lists/*
#7 0.529 Get:1 http://deb.debian.org/debian bookworm InRelease [151 kB]
#7 0.554 Get:2 http://deb.debian.org/debian bookworm-updates InRelease [52.1 kB]
#7 0.563 Get:3 http://deb.debian.org/debian-security bookworm-security InRelease [48.0 kB]
#7 0.621 Get:4 http://deb.debian.org/debian bookworm/main amd64 Packages [8787 kB]
#7 0.700 Get:5 http://deb.debian.org/debian bookworm-updates/main amd64 Packages [12.7 kB]
#7 0.722 Get:6 http://deb.debian.org/debian-security bookworm-security/main amd64 Packages [137 kB]
#7 1.444 Fetched 9188 kB in 1s (9833 kB/s)
#7 1.444 Reading package lists...
#7 1.841 Reading package lists...
#7 2.233 Building dependency tree...
#7 2.318 Reading state information...

Note you shouldn't need to build the image yourself.  It is pushed to the docker repo and can just be fetched from there.  Just remove the "build:" parameter from the docker-compose.yml file.

This forum post seems to indicate there might be issues with docker versions before 20.10.18-1.25.4
https://forums.debian.net/viewtopic.php?p=776923#p776923

@jcormier appreciated.
Thank you for your quick response.
I tried it in my clean vm, and can confirm it ran fine without the errors, and with up-to-date versions.
Just fetch and run.
Probably some issue on my workstation due to some rules and older versions of Ubuntu and Docker.


Reuse FZ bug with taskbar

I remember in the early versions of PT, when the taskbar can sit on zones.
This bug can be used to imitate 'RocketDock' appearance. Create a floating zone for taskbar (with a small distance below the screen and a percent of horizontal width - to be specific).
There will be no need for additional software to enhance the desktop. This can make a lot of different setups.

@gczark
tracked in https://github.com/microsoft/PowerToys/issues/503


Add Modal functionality using JS

Add functionality to the modal:

make the modal appear on the screen when the user clicks on the pokeball

make the modal disappear if the user clicks anywhere outside the modal

make the modal store the user input (artist name) into local storage

close the modal when the user has submitted the artist name and show results

Multiple pulls are referenced in this issue as the modal caused us some...problems.


'method' object not subscriptable when loading a config file with 'strategy' argument supplied to Trainer

Bug description
I use Weights & Biases with PL all the time and it works great.
    wandb_logger = WandbLogger(
        project="Project Name",
        config="configs/config.yaml",
        log_model=True,
        mode="offline",
    )
    config = wandb_logger.experiment.config
    wandb_logger.experiment.log_code(".")
    model_name = wandb_logger.experiment.name

    # Load dataset
    dataset = MSDataModule(
        config["dataset_path"],
        config["batch_size"],
    )

The above works without any problems when the 'strategy' argument is not supplied to Trainer:
    trainer = pl.Trainer(
        accelerator="gpu",
        devices=[1, 2, 3],
        precision=16,
        max_epochs=config["epochs"],
        callbacks=[checkpoint_callback, lr_monitor, model_summary],
        gradient_clip_val=1.0,
        logger=wandb_logger,
    )

But if I manually specificy the strategy (and it doesn't matter what it is) as follows:
    trainer = pl.Trainer(
        accelerator="gpu",
        devices=[1, 2, 3],
        precision=16,
        strategy='ddp'
        max_epochs=config["epochs"],
        callbacks=[checkpoint_callback, lr_monitor, model_summary],
        gradient_clip_val=1.0,
        logger=wandb_logger,
    )

I get an error saying:
Exception has occurred: TypeError
'method' object is not subscriptable
File "/home/......py", line 201, in
config["dataset_path"],

I thought for there was another variable in the namespace when I specify 'strategy' also called config, but even if I change the W&B config variable name to something else I get the same error.

### Environment

#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow): Trainer
#- PyTorch Lightning Version (e.g., 1.5.0): 1.7.7
#- PyTorch Version (e.g., 1.10): 1.12
#- Python version (e.g., 3.9): 3.9
#- OS (e.g., Linux): Linux
#- CUDA/cuDNN version: 11.6

maybe you are missing a comma here:
    trainer = pl.Trainer(
        accelerator="gpu",
        devices=[1, 2, 3],
        precision=16,
        strategy='ddp' # <- hereeeeeee
        max_epochs=config["epochs"],
        callbacks=[checkpoint_callback, lr_monitor, model_summary],
        gradient_clip_val=1.0,
        logger=wandb_logger,
    )

Good find! But no that was just a typo here because I typed it in manually. My actual code has the comma :-)

can you share the complete stack trace?

@TKassis The error message shows that this happens in /home/timothy/Documents/CelestialAI/celestialai/train_foundation.py. This is your code, right? Please take a look at line 201 in that file. The variable config seems to be a method, but the author of the code is accessing it as a dictionary.

config is actually a dictionary, but for some reason when I used the 'strategy' argument suddenly it becomes interpreted as a method. I couldn't fix the issue, I think it might be WandB related. I ended up just not using the returned dictionary from WandB. It might be something related with using WandB with multiple GPUs (although no issue with the default 'ddp' strategy unless explicitly mentioned). Anyway, I'm closing this for now as I found a workaround.


Add new data model - FreeFloatObserved

hello, I would like add some data-models to Fiware about:
FreeFloatObserved in Transportation
Thanks in advanced

i upload the wrong file.


Orchestrator: Use show slaves command for discovery

Orchestrator documentation recommends setting this option to false. However, I found that with mysql-operator setups, discovery only works with this command. The other option (discovery by IP address), does not work.

Please let me know what is your case in which orchestrator doesn't discover new nodes. I know there is a bug and should be fixed. The controller tries to discover nodes in SyncOrchestratorStatus but only the first time.

@AMecea as mentioned, I have observed this issue with mysql clusters newly created with the operator. This may not be related to existing clusters when operator restarts/upgrades.

FWIW, I found that deploying v0.1.12 of images results in this behavior. The latest images work fine.


HTML video with twig code is missing

To whom it may concern
I'm trying to add a video on a page by using twig code below:

it works on my localhost test but there are 404 error on server side testing.
on HTML, the video element's source is:

in addition, the video is missing with url, 'myurl.com/user/pages/mypage/video.mp4' meanwhile, it works with 'myurl.com/mypage/video.mp4'.
I'm using the latest version GRAV and Microsoft Azure for hosting.
Am I missing some configuration detail?

Cannot see the twig code.. please use code blocks.

oh sorry the twig code was
![](video.mp4)

and what is the HTML output on each of your instances, local and remote?

Here is the HTML output

<video controls="1" alt=""><source src="/user/pages/09.test/fsp-small.mp4">Your browser does not support the video tag.</video>


[SPARK-5127][MLlib] Fixed overflow when there are outliers in data in Logistic Regression

gradientMultiplier = (1.0 / (1.0 + math.exp(margin))) - label
However, the first part of gradientMultiplier will be suffered from overflow
if there are samples far away from hyperplane, and this happens
when there are outliers in data.
As a result, we use the equivalent formula but more numerically stable.
val gradientMultiplier =
  if (margin > 0.0) {
    val temp = math.exp(-margin)
    temp / (1.0 + temp) - label
  } else {
    1.0 / (1.0 + math.exp(margin)) - label
  }

Test build #25153 has started for   PR 3928 at commit ee7a9d8.

This patch merges cleanly.

Here, does overflow matter? 1.0 / (1.0 + math.exp(10000)) still comes out 0.0, just like math.exp(-10000) / (1.0 + math.exp(-10000))

Yeah, in this case, math.exp(x) will be infinity when x > 720, so 1.0 / (1.0 + math.exp(10000)) will always be zero.
In MLOR,
multiplier = exp(margins(i)) / (1 + \sum_k exp(margins(k))) -  \alpha(label) * \delta_{label}{i+1}

where \alpha(label) = 1 if label != 0;
           \alpha(label) = 0 if label == 0, and
           \delta_{i}{j} = 1 if i == j;
           \delta_{i}{j} = 0 if i != j

the numerator and dominator can both go infinity, which will case issue. For binary one, @srowen you are right, it's not a concern. I'll close it.

Test build #25153 has finished for   PR 3928 at commit ee7a9d8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25153/
Test PASSed.


Implement real storage backend

Implement another storage.Client that uses actual database backend, whether it is SQL or NoSQL.
Requirements

Implement storage.Client using some sort of external persistent storage

Open Questions

SQL or NoSQL? After deciding that, I need to decide which product of either type to use

It’s probably best to wait until after implementing #22 since that will be a major storage refactor


Scale from centre

is it possible to scale the text is from centre?

Hi @JayParikh20! Thanks for the question, it's been a long time since I worked on this. I'm not sure if that was supported originally, but I can look into it this weekend.

Since the scaling based on your pinch only changes the text size, it should appear to grow from the center as along as the text itself is centered inside the parent.
I did this by setting android:layout_centerHorizontal="true" inside the layout file. Is this the behavior you were looking for?
https://drive.google.com/file/d/1axYr7Cskd1nw7l97eDUZnfFzQkqTFKBe/view?usp=sharing

Additionally, using android:gravity="center" will keep the text centered as it scales up, too.

Yes, i was looking for something similar. Thanks for your help!

Great! Thanks for nudging this old library though. I'm gonna freshen up these dependencies, get rid of the Java modules, and release it with a 2020 feel. 😆 So keep an eye out for that.


rtl8821CU not working

usb wifi device COMFAST 811AC not working.
it is  rtl8821CU chip.
could you please add this driver to the image?
thanks 😸

Noted. Thanks for your report.

Please try the new image.

thanks a lot. 😁


Add Github Actions workflows for nightly macOS and Linux builds

This PR adds 3 GA workflows, each for a particular operating system and architecture:

nightly-ubuntu-x86_64.yml
nightly-ubuntu-i386.yml
nightly-macos-x86_64.yml

They run ./scripts/do-all-unix.sh for every combination of build type (debug or release), library type (shared or static), and SSL support (on, off).
They are scheduled to run every night and can also be triggered manually.
See the runs on my fork:

ubuntu-x86_64: https://github.com/yemreinci/hazelcast-cpp-client/actions/runs/597264394
ubuntu-i386: https://github.com/yemreinci/hazelcast-cpp-client/actions/runs/597562963, https://github.com/yemreinci/hazelcast-cpp-client/actions/runs/598727691
macos-x86_64: https://github.com/yemreinci/hazelcast-cpp-client/actions/runs/597852623

Note: I'll send workflow(s) for nightly Windows builds in a separate pull-request.

Linux test PASSed.

Windows test PASSed.

Linux test PASSed.

Windows test PASSed.

Your fork shows a failed build (nightly-Ubuntu-i386 (Release, STATIC, ON))? https://github.com/yemreinci/hazelcast-cpp-client/runs/1974293859?check_suite_focus=true
Well, they are flaky. Sometimes others fail, but this one passes.

Can we write the variables explicitly at the CMake file start so that I can match easily the run output and the provided build config?
I can add a workflow step that prints out every matrix parameter. Will that work? Sth like:

OS: Ubuntu
architecture: i386
build_type: STATIC
...

Can you separate build, test and verify into 3 steps?
OK.

Linux test PASSed.

Windows test PASSed.

Windows test FAILed.

Linux test PASSed.

Building part is divided into 3 steps: Build&install, test, and verify.
Links to most recent runs:

ubuntu i386: https://github.com/yemreinci/hazelcast-cpp-client/actions/runs/620243954
ubuntu x86_64: https://github.com/yemreinci/hazelcast-cpp-client/actions/runs/620244233
macos x86_64: https://github.com/yemreinci/hazelcast-cpp-client/actions/runs/620244491

Windows test PASSed.

Linux test PASSed.

step that prints out every matrix parameter. Will that work? Sth like:

I want to see this: When I expand the Build&Install log, at the very start I like to see the hazelcast build related cmake variable values, namely, CMAKE_BUILD_TYPE, WITH_OPENSSL, BUILD_SHARED_LIBS. I want to verify easily that the build is building with the expected parameters (I can see matrix parameters in the job name but that is not what I am asking for). Is this more clear?

Windows test PASSed.

Linux test PASSed.


Use image instead of heroImageURL in ViewingRoomApp as well

I also tried to update the image for those 2 test rooms in production but it will not save for one of the rooms for some reason. But don't think it matter for that code. All actual rooms seem to work just fine.

Going to go ahead and merge as the one comment I left is non-blocking.


Update all actions to latest

This will get rid of all the warnings on builds we have.

Looks good to me. From a quick glance at the changelogs, there does not seem to be breaking changes in these actions, apart from using Node 16 instead of 12.


fix: Auth backend which doesn't fully support custom user model

Fix multiple issue with auth backend. It now supports custom default user manager, support custom username field with help of get_by_natural_key and doesn't use both email and username for getting user if it is not intended choice for custom UserModel instead use get_by_natural_key and at last protect from timing attack as mentioned at https://code.djangoproject.com/ticket/20760

Since email login test was failing. I have removed that test. After this fix PR test is not valid test since get_by_natural_key only support username by for default user model. So this PR may be breaking change for package?

Codecov Report

Merging #32 (56e466c) into master (4d1313c) will decrease coverage by 4.23%.
The diff coverage is 61.53%.

@@             Coverage Diff             @@
##            master      #32      +/-   ##
===========================================
- Coverage   100.00%   95.76%   -4.24%
===========================================
  Files            5        5
  Lines          109      118       +9
===========================================
+ Hits           109      113       +4
- Misses           0        5       +5

Impacted Files
Coverage Δ

impostor/backend.py
93.42% <61.53%> (-6.58%)
:arrow_down:

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 4d1313c...56e466c. Read the comment docs.

@avallbona I have updated PR with your suggestion. As well as added some test and logs for new authentication fail cases.

@avallbona I have updated PR again since earlier update had issue with exception not catched and logged

@avallbona Can you review latest changes? This PR is stale for nearly 3 months after last review with no progress

@avallbona Can you review latest changes? This PR is stale for nearly 3 months after last review with no progress

I'll do it this weekend

@avallbona Can you review latest changes? This PR is stale for nearly 3 months after last review with no progress

LGTM, will merge it very soon and add a few more tests. Thanks.


Need way to remove codons and UGOs

There doesn't seem to be a way to remove either codons or UGOs from a cell.
For codons, I would suggest "remove" and then "RGL<>"
For UGOs, I would suggest "remove" and then "Weak Loc"
Additionally, it would be neat if "remove" "inwards" removed all food, waste, and UGOs from a cell.

Not quite the same as moving, but you could store a none codon in memory and then write it to a codon to make it functionless


[Refactor](table function) Decouple vectorized table functions from non-vectorized ones

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 36.48 seconds
load time: 487 seconds
storage size:<PHONE_NUMBER>9 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230110144707_clickbench_pr_76935.html


How many images do we need to train a restyle model?

Just like the title, maybe silly but very important!

It depends on the domain and data. For faces, we used all 70,000 FFHQ images. For the cars domain, we used the ~8,000 training images from Stanford Cars. It all depends on the difficulty of the domain, but the more data, the better.

Thank you! It would be helpful!

And I wonder how many epochs do you train for FFHQ? Last time I put in 4000 images and trained for 200 epochs, but the result turns out not too good.

We don't measure the training, but total training steps. If I remember correctly, our models were trained for approximately 100,000 iterations.


Removed @ts-expect-error for middleware in upload-builder.ts

Use a single function instead of interpreting from multiple signatures and use arguments from the Args type so it's still customizable.
It's also possible to use the code below, but it would give an error to the user if they wrote this .middleware((req) => {...}) so they would need to write .middleware((req, _res) => {...}) if they didn't need the response when using next-legacy.
type MiddlewareFn<
  TOutput extends Record<string, unknown>,
  TRuntime extends string
> = Args<TRuntime>["res"] extends never
  ? (req: Args<TRuntime>["req"]) => MaybePromise<TOutput>
  : (
      req: Args<TRuntime>["req"],
      res: Args<TRuntime>["res"]
    ) => MaybePromise<TOutput>;

But the one plus would be it doesn't allow the .middleware((req, _res) => {...}) when using next or server, while in the one committed _res is of type never so it's not really usable.

There seems to be some edge cases that isn't covered here.
Check out the tests locally (especially the upload-builder.test.ts) to see what's wrong.
One of the faults is that it's now okay to pull the res even if it's never. Maybe not a big deal? But we're also always expecting 2 arguments when callign the middleware now

I've changed to the other MiddlewareFn type I spoke about above. And all the tests pass now.
I did test the old one yesterday, but seems sleepiness got the better of me, as I've misunderstood the number of tests cases to the number of tests passed.
Also, I've rearranged the types and renamed the Args type to MiddlewareFn.
never is still used, but now it signals that the function only has one argument not two.
I've tried to remove never, but then res is implicitly any, I think, and doesn't work with extends so it can't figure out that it should only be one argument and not two.


[Brainstorm] Analytics troubleshooting

Lots of users are emailing asking about the accuracy of analytics. I haven't found any real problems so far, but the nature of analytics is that they're opaque. It would be helpful to have a tool for understanding whether listen or subscriber counts are accurate.

It would be useful to warn users when FeedBurner hits are detected
It would be useful to count listens and subs without user agent strings

Analyze drops in subscriber counts

Detect "rainbow bumps" in listens by episode


Upgrading from 3.5.3 to 3.5.7 fails

Neo4j version: 3.5.3 -> 3.5.7
Operating system: Debian
API/Driver: Cypher
Environment: Official neo4j container running on kubernetes 1.11.8

Expected behavior:
Use a stateful set when deploying neo4j, deploy neo4j at version 3.5.3, upgrade the stateful set to 3.5.7 and everything should work.
Actual behavior:
After upgrading the container continually crashes with the following logs:
Active database: graph.db
Directories in use:
  home:         /var/lib/neo4j
  config:       /var/lib/neo4j/conf
  logs:         /logs
  plugins:      /var/lib/neo4j/plugins
  import:       /var/lib/neo4j/import
  data:         /var/lib/neo4j/data
  certificates: /var/lib/neo4j/certificates
  run:          /var/lib/neo4j/run
Starting Neo4j.
2019-07-08 19:33:11.901+0000 WARN  Unknown config option: causal_clustering.discovery_listen_address
2019-07-08 19:33:11.904+0000 WARN  Unknown config option: causal_clustering.raft_advertised_address
2019-07-08 19:33:11.904+0000 WARN  Unknown config option: causal_clustering.raft_listen_address
2019-07-08 19:33:11.904+0000 WARN  Unknown config option: ha.host.coordination
2019-07-08 19:33:11.905+0000 WARN  Unknown config option: causal_clustering.transaction_advertised_address
2019-07-08 19:33:11.905+0000 WARN  Unknown config option: causal_clustering.discovery_advertised_address
2019-07-08 19:33:11.906+0000 WARN  Unknown config option: ha.host.data
2019-07-08 19:33:11.906+0000 WARN  Unknown config option: causal_clustering.transaction_listen_address
2019-07-08 19:33:11.929+0000 INFO  ======== Neo4j 3.5.7 ========
2019-07-08 19:33:11.942+0000 INFO  Starting...
2019-07-08 19:33:13.002+0000 ERROR Failed to start Neo4j: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabase@38604b81' was successfully initialized, but failed to start. Please see the attached cause exception "Unable to obtain lock on store lock file: /data/databases/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)". Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabase@38604b81' was successfully initialized, but failed to start. Please see the attached cause exception "Unable to obtain lock on store lock file: /data/databases/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)".
org.neo4j.server.ServerStartupException: Starting Neo4j failed: Component 'org.neo4j.server.database.LifecycleManagingDatabase@38604b81' was successfully initialized, but failed to start. Please see the attached cause exception "Unable to obtain lock on store lock file: /data/databases/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)".
	at org.neo4j.server.exception.ServerStartupErrors.translateToServerStartupError(ServerStartupErrors.java:45)
	at org.neo4j.server.AbstractNeoServer.start(AbstractNeoServer.java:187)
	at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:124)
	at org.neo4j.server.ServerBootstrapper.start(ServerBootstrapper.java:91)
	at org.neo4j.server.CommunityEntryPoint.main(CommunityEntryPoint.java:32)
Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.server.database.LifecycleManagingDatabase@38604b81' was successfully initialized, but failed to start. Please see the attached cause exception "Unable to obtain lock on store lock file: /data/databases/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)".
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:473)
	at org.neo4j.kernel.lifecycle.LifeSupport.start(LifeSupport.java:111)
	at org.neo4j.server.AbstractNeoServer.start(AbstractNeoServer.java:180)
	... 3 more
Caused by: java.lang.RuntimeException: Error starting org.neo4j.graphdb.facade.GraphDatabaseFacadeFactory, /data/databases
	at org.neo4j.graphdb.facade.GraphDatabaseFacadeFactory.initFacade(GraphDatabaseFacadeFactory.java:230)
	at org.neo4j.graphdb.facade.GraphDatabaseFacadeFactory.newFacade(GraphDatabaseFacadeFactory.java:146)
	at org.neo4j.server.database.CommunityGraphFactory.newGraphDatabase(CommunityGraphFactory.java:41)
	at org.neo4j.server.database.LifecycleManagingDatabase.start(LifecycleManagingDatabase.java:90)
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:452)
	... 5 more
Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.kernel.internal.locker.StoreLockerLifecycleAdapter@54eb2b70' was successfully initialized, but failed to start. Please see the attached cause exception "Unable to obtain lock on store lock file: /data/databases/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)".
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:473)
	at org.neo4j.kernel.lifecycle.LifeSupport.start(LifeSupport.java:111)
	at org.neo4j.graphdb.facade.GraphDatabaseFacadeFactory.initFacade(GraphDatabaseFacadeFactory.java:225)
	... 9 more
Caused by: org.neo4j.kernel.StoreLockException: Unable to obtain lock on store lock file: /data/databases/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)
	at org.neo4j.kernel.internal.locker.StoreLocker.storeLockException(StoreLocker.java:116)
	at org.neo4j.kernel.internal.locker.StoreLocker.unableToObtainLockException(StoreLocker.java:109)
	at org.neo4j.kernel.internal.locker.StoreLocker.checkLock(StoreLocker.java:97)
	at org.neo4j.kernel.internal.locker.GlobalStoreLocker.checkLock(GlobalStoreLocker.java:61)
	at org.neo4j.kernel.internal.locker.StoreLockerLifecycleAdapter.start(StoreLockerLifecycleAdapter.java:36)
	at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:452)
	... 11 more
2019-07-08 19:33:13.003+0000 INFO  Neo4j Server shutdown initiated by request

This seems to be the same as issue #207 which was caused by some necessary changes to file permissions that I did for 3.5.6. There were a lot of complaints about neo4j containers messing with file permission access on mounted folders, which was fixed in 3.5.6.
I wrote a justification for the change at the time here: https://github.com/neo4j/docker-neo4j/issues/131#issuecomment-489612338
I have questions for anyone who is hitting this issue:

What folders are being mounted when this problem happens?
what user is the owner of the host folder(s) and what user is docker running as?
does running docker with --user="$(id -u):$(id -g)"  fix the problem?
does running with --user="0:0" fix the problem? (this seemed to work for another user: https://github.com/neo4j/docker-neo4j/issues/207#issuecomment-558149437)

OK nevermind, I think I've figured this out.
I've been able to reproduce this by mounting the /data folder on neo4j:3.5.3 and writing some small amount of data. Then closing the container and mounting the same folder to /data on neo4j:3.5.7.
The entrypoint script is supposed to check if the /data folder is writable and then chown -R neo4j:neo4j if not (and we're not running as any specific user).
The problem seems to be that the /data/databases folder is owned by the correct neo4j group but not the neo4j owner. HOWEVER instead of having permissions 777 like /data it has permissions 755 which prevents group write access. Our writability check checks to see if the running user has group access  but doesn't actually check that the folder has permissions allowing group write.
I'll create a pull request to fix it.


Closes #19: Bubble cannot start on android 14

This change adds a type declaration in the Android manifest, which is required for apps targeting Android 14.
If an app that targets Android 14 doesn't define types for a given service in the manifest, then the system will raise MissingForegroundServiceTypeException upon calling startForeground() for that service.
Read more here: https://developer.android.com/about/versions/14/changes/fgs-types-required
Related Issues:

Closes #19
Fixes #14 (already closed without being resolved)

Hi @dhiaCodes , can you help me with this.
Is there any way I can edit my code to fix this issue. Right now my app crashes when the target sdk is 34.


[Question][AzureDevOps] Pipeline run retention

Hi - The ADO project I'm working with has a pipeline rention policy of 3 previous runs. When running the data collection in DevLake, any runs that have expired or been deleted from ADO are removed, which leads to the DORA dashboard having no/less data.
Is there any way for me to configure DevLake to retain any pipeline metrics even if the actual pipeline run in ADO has been removed? Thanks for your help.

@benjaminrmoss  You can run your project in incremental mode. It will only collect data that changes therotically since last successful sync.

@d4x1 Thanks for the reply. Would I set this in the project configuration settings? I can't see any option for it here:

Maybe this feature isn't supported for ADO yet? https://devlake.apache.org/docs/Overview/SupportedDataSources/#azure-devops

@benjaminrmoss  You can try azuredevop_go, it can collect data incremantally.


Add new ts0601_smoke model to quirk

Added model from this issue: https://github.com/zigpy/zha-device-handlers/issues/1491
Device doesn't report battery status, concluded it simply doesn't do it because it has now PowerConfiguration cluster.

Codecov Report

Merging #1503 (acdbc5d) into dev (16f298e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev    #1503   +/-   ##
=======================================
  Coverage   80.36%   80.36%
=======================================
  Files         227      227
  Lines        6704     6704
=======================================
  Hits         5388     5388
  Misses       1316     1316

Impacted Files
Coverage Δ

zhaquirks/tuya/ts0601_smoke.py
66.66% <100.00%> (ø)

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.37%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5388

Relevant Lines:
6704

💛  - Coveralls

Isort dont like your spitted line.
     ZONE_STATUS,
     ZONE_TYPE,
 )
-
-from zhaquirks.tuya import (
-    TuyaManufCluster,
-    TuyaManufClusterAttributes,
-)
+from zhaquirks.tuya import TuyaManufCluster, TuyaManufClusterAttributes

 _LOGGER = logging.getLogger(__name__)

Delete lines starting with - and adding  theline with + (without the + at the start).

@MattWestb Thats weird, because I installed isort an let it run on the file...


FIX the implementation guide, tell the user he have to use the key 'form' in his state

In the documentation it's stated that it's "recommended" to use "form" as key in the state.
I think this is not a recommendation but an obligation, otherwise connectReduxForm will fail
https://github.com/erikras/redux-form/blob/master/src/connectReduxForm.js#L5

It's not an obligation.
You can use any key as long as you pass it into the form prop to reduxForm. But if you use the connectReduxForm decorator, it will force the key to be form, since it's doing the connect()ing for you, too.

Ok, I got your point. Thanks for your quick answer.
In that case a simple remark: in the "Implementation Guide", it's unclear that connectReduxForm() is optional. In STEP 1 it's stated "it's recommended to use the key 'form'" (but it's not an obligation) and in STEP 2 "Wrap your form component with connectReduxForm()".
I would say that in the context of this implementation guide, either you tell explicitly that the first step is an obligation if you don't do the connect()-ing by yourself, either you don't tell anything and let the user discover later in the doc how he can customize that key.
Otherwise, (dumb) people like me will just fail your implementation guide by using another key ;)
Thanks,

Thanks for your input. The more clarity the better.

tell the user he have to

he or she.

Thankfully docs can be written in the second person.


Added methods to stop sounds that are already playing.

API | Common
Fixes #1527

Thanks, @Cybermaxke.


ARROW-10251: [Rust] [DataFusion] MemTable::load() now loads partitions in parallel

For the TPCH benchmark with --mem-table this gave me ~10x speedup in load times. fyi @jhorstmann


Tensorboard SummaryWriter

I am trying to run one code that I forked from here. This uses tensorboard as given in the following line:
logger = tensorboard.SummaryWriter('./log')
This throws an error saying :

AttributeError: module 'tensorboard' has no attribute 'SummaryWriter'

Any suggestions what should I do next?

@akhilesh-pandey I suggest https://github.com/awslabs/mxboard, the usage is quite similar. But make sure you have install mxboard or dmlc/tensorboard first.


Looks like the documentation site is broken?

404 error thrown on the JS bundle. just raising up - maybe needs a re-deploy :)

I was wrong. Closing!

@lucastobrazil <3 :)

@rwieruch hello old friend! Just circling back on this one, someone else in my team noticed the issue too. I think it was the "MyGlue" Chrome extension which we had installed (it is a password manager). Just FYI that seemed to be what caused the issue. No idea how/why but writing it here in case anyone else ever experiences it. Hope you're well Robin!


Kernel SHAP notebooks

Adds notebooks demonstrating how to compute Shapely values for Autogluon-Tabular predictions in: examples/tabular/interpret/
as well as pointer to these notebooks in Tabular tutorial FAQ.

Job PR-720-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-720/1/index.html


cms integration

can you add some cms integration  in the astro website mentioned

Can you add other git-based CMS like DecapCMS?


added base test

added examples for a basic test and the cmake target

@exoticDFT we good? if this passes should i merge?


Refactor and add shortcuts

Rework of the volumio meta driver. This includes a few more buttons (repeat, shuffle) and an ability to browse the full directory accessible by Volumio and define shortcuts to them. These shortcuts then become visible in the directory.
Signed-off-by: Mark Herwege<EMAIL_ADDRESS>

Merge with mherwege version


Parameterized  + Parallelized

This may not be officially support by Jest but I can run some tests multiple times in a forEach, for loop, etc. e.g.:
versions.forEach(version=> {
    describe(version, function(
        test("something", function() {
            /* test something */
        });
        test("something else", function() {
            /* test something else */
        });
    ) {});
});

However, this does not allow me to run said tests in parallel because there is no way, from what I can tell, to split this into multiple files.
If my tests are IO bound or would otherwise benefit from parallelization it would be great to have a way to tell jest to run some tests in parallel either through multiple config files that specify which tests to run with set-up files, etc. to set the parameters for each run or through a Jest API (e.g. jest.forEach, jest.fork, etc.).
Is this already supported in some way that I have overlooked? If not, how might this be added? Thank you.

There's a test.concurrent which runs your tests in parallel inside the same file, but it's not officially supported and I'd advice against it, because you're on your own taking this path (however Yarn codebase uses it, so you can check how they manage this API).
To benefit parallelization, you need to split your tests across multiple files, e.g. by having a template like this and generating the files before the test run (which then could be git ignored if you don't want them inside the source).
Also, please respect the issue template.
This is not a help forum, please ask such questions on forums like Stack Overflow or Discord.

Thank you @thymikee. My apologies for disrespecting the issue template. In my mind I was thinking I was asking for a new feature but I also knew it might be there so I see clearer now that this would have been better asked on Stack Overflow or Discord. Thanks.


Stubbed mix-in for applications to use in their own functional tests

Example: we might want to provide "preview" functionality for a publishing feature, but we don't want to write out own stub that mirrors all the behaviour of GDS::SSO users in our functional test setup method.
If you could provide one for gem consumers that'd be awesome.

🌸spring cleaning 🌸Closing this issue because of inactivity. Feel free to reopen!


Added CustomUrlCategory object type to objects.py

Resolves enhancement request #129 .

I need this updating. FYI, codes are exactly same that I am using with modifying.

This pull request has been outstanding for a while now.  Is there anything that I need to work on specifically to get it reviewed and approved?


WADO retrieve instance giving 406

Hi, I am trying to test the wado-rs functionality for retrieve instance, the functionality works fine on postman/browser. While testing using dicomweb-client I am getting the following error.
http://<IP_ADDRESS>:8000 "GET /api/internal/dicomweb/ec712b95-a5fb-43bc-9de1-3ae5024e7c39/wadors/studies/<IP_ADDRESS>543.6003.1719516381.65153.16773.3015806397/series/<IP_ADDRESS>543.6003.2297816378.33047.16917.2271512738/instances/<IP_ADDRESS>543.6003.3368520785.218.17135.699537048 HTTP/1.1" 406 57
DEBUG 2020-06-30 05:03:10,741 api 77<PHONE_NUMBER>93320  request status code: 406
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/code.py", line 90, in runcode
    exec(code, self.locals)
  File "<console>", line 1, in <module>
  File "/usr/local/lib/python3.7/site-packages/dicomweb_client/api.py", line 2084, in retrieve_instance
    return self._http_get_multipart_application_dicom(url)[0]
  File "/usr/local/lib/python3.7/site-packages/dicomweb_client/api.py", line 1045, in _http_get_multipart_application_dicom
    response = self._http_get(url, params, headers)
  File "/usr/local/lib/python3.7/site-packages/dicomweb_client/api.py", line 695, in _http_get
    response.raise_for_status()
  File "/usr/local/lib/python3.7/site-packages/requests/models.py", line 941, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 406 Client Error: Not Acceptable for url: http://<IP_ADDRESS>:8000/api/internal/dicomweb/ec712b95-a5fb-43bc-9de1-3ae5024e7c39/wadors/studies/<IP_ADDRESS>543.6003.1719516381.65153.16773.3015806397/series/<IP_ADDRESS>543.6003.2297816378.33047.16917.2271512738/instances/<IP_ADDRESS>543.6003.3368520785.218.17135.699537048

This is how I am testing the wado retrieve instance
from dicomweb_client.api import DICOMwebClient
study_instance_uid = '<IP_ADDRESS>543.6003.1719516381.65153.16773.3015806397'
series_instance_uid = '<IP_ADDRESS>543.6003.2297816378.33047.16917.2271512738'
sop_instance_uid = '<IP_ADDRESS>543.6003.3368520785.218.17135.699537048'
client = DICOMwebClient(url="http://<IP_ADDRESS>:8000/api/internal/dicomweb/ec712b95-a5fb-43bc-9de1-3ae5024e7c39", wado_url_prefix="wadors")
client.retrieve_instance(study_instance_uid=study_instance_uid, series_instance_uid=series_instance_uid, sop_instance_uid=sop_instance_uid)

The response from the endpoint has content_type multipart/related;type=application/dicom. Let me know if I am doing something wrong here.

@srinivasaravind5 you closed the issue. Where you able to solve it?


Changes required to support Hexagon SDK 3.0

The inc and lib directories were renamed to incs and libs in Hexagon SDK 3.0.
This requires an updated cmake_hexagon and some changes to
PX4. Both Hexagon SDK 2.0 and 3.0 are supported with this patch.
Signed-off-by: Mark Charlebois<EMAIL_ADDRESS>

@julianoes Can you please test this when you have a chance?
@LorenzMeier When this is merged I can migrate all the documents to use the installer for Hexagon SDK 3.0. Its a MUCH simpler setup and can be fully automated (after the SDK is downloaded from QDN). Its already available at: https://github.com/ATLFlight/cross_toolchain/tree/sdk3

I merged the needed commit in DriverFramework and benched tested this on Snappy. It seems to be working so far.

Rebased.

@mcharleb Can you prepare a PR against PX4/Devguide and then we switch this all at once?

@LorenzMeier See https://github.com/PX4/Devguide/pull/169 for the updated documentation you requested.

Thanks! Rebased and applied.


Approach to fixing #1174

In this approach we track a new bit of state within the File transport: rotatedWhileOpening which indicates that enough data was written while no files were open to cause rotation.
When this flag is set we trigger the rotation in _createStream. Not 100% sure this is the right approach, but it does solve the problem at hand. Going to sleep on it and decide the right path forward.

@DABH if there are multiple rotates during open all the content will be written to the first file.


Mediainfo shows inconsistent results w/ aws presigned URL file (ts file)

Environment:
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.4 LTS
Release:	20.04
Codename:	focal

mediainfo --version
MediaInfo Command line,
MediaInfoLib - v22.03

Have also seen same behavior with Mediainfo (v19)
This happens when I run it the first time:
mediainfo 'https://mediainfo-debug.s3.us-east-1.amazonaws.com/stereo_en-US-0-new3.ts?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIA3IVUTRF6WLZA6VNT%2F20220509%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220509T213345Z&X-Amz-Expires=604800&X-Amz-SignedHeaders=host&X-Amz-Signature=2d3ef95bc204e6cc4354dd098bb1cf464e20b5c6ad55305b2f04274f5eb21008'
General
Complete name                            : https://mediainfo-debug.s3.us-east-1.amazonaws.com/stereo_en-US-0-new3.ts
Format                                   : ADTS
Format/Info                              : Audio Data Transport Stream
File size                                : 1.03 MiB
Overall bit rate mode                    : Variable
FileExtension_Invalid                    : aac aacp adts

Audio
Format                                   : AAC LC
Format/Info                              : Advanced Audio Codec Low Complexity
Format version                           : Version 4
Codec ID                                 : 2
Bit rate mode                            : Variable
Channel(s)                               : 2 channels
Channel layout                           : L R
Sampling rate                            : 48.0 kHz
Frame rate                               : 46.875 FPS (1024 SPF)
Compression mode                         : Lossy
Stream size                              : 1.03 MiB (100%)

This is what happens when I run it the second time:
mediainfo 'https://mediainfo-debug.s3.us-east-1.amazonaws.com/stereo_en-US-0-new3.ts?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIA3IVUTRF6WLZA6VNT%2F20220509%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220509T213345Z&X-Amz-Expires=604800&X-Amz-SignedHeaders=host&X-Amz-Signature=2d3ef95bc204e6cc4354dd098bb1cf464e20b5c6ad55305b2f04274f5eb21008'
General
ID                                       : 1 (0x1)
Complete name                            : https://mediainfo-debug.s3.us-east-1.amazonaws.com/stereo_en-US-0-new3.ts
Format                                   : MPEG-TS
File size                                : 1.03 MiB
Duration                                 : 1 min 0 s
Overall bit rate mode                    : Variable
Overall bit rate                         : 144 kb/s

Audio
ID                                       : 256 (0x100)
Menu ID                                  : 1 (0x1)
Format                                   : AAC LC
Format/Info                              : Advanced Audio Codec Low Complexity
Format version                           : Version 4
Muxing mode                              : ADTS
Codec ID                                 : 15-2
Duration                                 : 1 min 0 s
Bit rate mode                            : Variable
Channel(s)                               : 2 channels
Channel layout                           : L R
Sampling rate                            : 48.0 kHz
Frame rate                               : 46.875 FPS (1024 SPF)
Compression mode                         : Lossy

Menu
ID                                       : 4096 (0x1000)
Menu ID                                  : 1 (0x1)
Format                                   : AAC
Duration                                 : 1 min 0 s
List                                     : 256 (0x100) (AAC)
Service name                             : Service01
Service provider                         : FFmpeg
Service type                             : digital television

For some reason, mediainfo does not like this ts file and is giving different results when run. The first mediainfo results does not have duration yet the second one does.
note: presigned url used in example is available up to 7 days from this issue submission.

This is due to the raw ADTS parser catching the stream before the TS parser, random behavior is due to random packet sizes from the server (this is normal), we have to better manage such case and the raw ADTS parser should wait before catching the stream in case the TS can also catch it, TS should have priority there.

Warning: free support is set to low priority, we don't know when we have time to handle it.
If you desire to accelerate the development, don't hesitate to contact us for a quotation.

Hello Jerome, I appreciate your feedback. I think our team can figure out a workaround for the randomness behavior. If we have any other requests, I will be sure to reach out to you on the contact form.
Much appreciated.

Hello Jerome, I appreciate your feedback. I think our team can figure out a workaround for the randomness behavior. If we have any other requests, I will be sure to reach out to you on the contact form.
Much appreciated.

What was your workaround, if you don't mind? Having a similar issue with ffmpeg in a Lambda using a presigned URL

My workaround was to not use the presigned URL and rely on the having the file locally before running Mediainfo.


Improve helm configuration

This allows the cronjob to be disabled (for example to allow CR to be deployed in another manner) and makes the alertmanager address configurable. The defaults for both are to retain the existing behaviour
Checklist

[x] Update changelog in CHANGELOG.md.

Thank you for the contribution @paddatrapper


ContribFest: Link to test docs is broken in DEVELOPMENT.md file

The link to the test docs from the DEVELOPMENT.md file points to ./test/README.md which does not exists.
Should we use the one from https://github.com/knative/eventing (core) here?
@lionelvillard @aslom @psschwei

I think the link definitely need to be fixed. We may need to create new README if there are differences in tests?

Got it. I’ll work on this. Please assign this to me.