[BUG] Parser should correctly identify empty tuple.

Pocketpy main
>>> empty_tuple = ()
  File "<stdin>", line 1
    empty_tuple = ()
                   ^
SyntaxError: expected an expression, got )

>>> list_of_empty_tuples = [(),()]
  File "<stdin>", line 1
    list_of_empty_tuples = [(),()]
                             ^
SyntaxError: expected an expression, got )

Python
>>> empty_tuple = ()
>>> len(empty_tuple)
0

Kindly let me know if this is intended or it should be fixed.

This is intended. Empty tuple does not make sense.

I understand. Closing !


Web UX: Collection view scrolling

There is currently no obvious way to scroll the collection view on web, landscape when the scroll region goes horizontal.
Ideally, the mouse scroll wheel would still navigate this view even though it is horizontal because the scrollbars are hidden, and otherwise the view relies on click and drag.
Additional details:

Shift + Scroll works for hz scrolling on macOS version, and web version in Safari
Not working in Chrome (118.0.5993.88)

@jaredbell Considering we've fixed #147 which turns scrollbars on permanently for web/desktop, do you think this still needs additional scrollwheel support?


Crash if swipe to delete (actually hide) token in Wallet tab if that token has been hidden before

@James-Sangalli This is observable in both master and multi-chain, but have just applied to fix in the multi-chain WIP branch because the fix is actually more complicated for master (the class has no primary key defined!) than multi-chain. So avoiding the work unless it's necessary.


Unable to narrow down type out of a union type (of interfaces) when a distinguishing property is checked

https://flow.org/try/#0PQKgBAAgZgNg9gdzCYAoVBLAdgFwKYBOUAhgMZ5gAaYA3qmMQFxgDOOB2A5mKgL7qZchEuTABNWvSZgsAVwC2AI0IAaeouZylhHv3RRZWUjgxwsYKAApiBTs0oAfMQEpmAfgl0AkBis3OAHSKzpJgYeHAwGAA6hSkxOZYiAwsLDo4ABbEOAy2YBgsYHBQYDgAngAOFGL04WG1dZFgACoZBawZcLIwACZghARwBGDK8bJpuYHEYFmFOHAjFMWllRQABgByCsoEa2CWAPIA0s4N4f4BxAEsFTAYOJYA5I+ndfVvYE2t7Syd3X0bA7NfoEQbDUbEcYUBAUNgYGAwMAAaySSEy2Um+UKy3KVXE+wAYgBBACSABlXm8CHgcLICOZ-Gd+PwgA
I describe the bug with comments in the example above.
cc: @tootallnate

Ignore me, I missed the whole part about https://flow.org/en/docs/types/unions/#toc-disjoint-unions-with-exact-types


Add back button to desktop threads

Noticed this got dropped in the layout refactor. This keeps the logic, but doesn't gate it based on breakpoint.

Overdue - we should also think about what screens we want to make this sticky on web


fix: improve spanish translation

Motivation
I'm a spanish native speaker and I found some diacritic errors when using the current translation. Also, I've translated some values that were in english.
Have you read the Contributing Guidelines on pull requests?
Yes.

Just to clarify my change proposal: esta and está may seem pretty similar, but their meanings are completely different:
With tilde, is a form of the verb to be:
🇪🇸 El gato está debajo de la mesa.
🇺🇸  The cat is under the table.
Without it, it's a pronoun:
🇪🇸 Editar esta página.
🇺🇸 Edit this page.

@faloi thanks!


Please add select option for KDE or Gnome

i have been using KDE for years and i hate bloated gnome

Due to the nature of how the image is built this isn't possible currently.
TBS, we use gnome partially because it uses less space on the image, so switching to KDE would cause more "bloat".
We're considering if we should switch to KDE in the future though, but t he earliest version that might happen would be v47. For now I'm going to close this.


Tensorflow-lite gpu output is corrupted when using opencl backend

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Android version:9, Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: OnePlus3, GPU:Adreno 530
TensorFlow installed from (source or binary): tensorflow-lite-gpu 1.15, tensorflow-lite-gpu:0.0.0-nightly from 'https://mvnrepository.com'
TensorFlow version (use command below): 1.14, 1.15

Describe the current behavior
I'm using tflite-gpu in my android application for semantic segmentation. Using my tflite model(all gpu supported ops) i'am able to get proper output with CPU version and tensorflow-lite-gpu 1.14 ;but when i use nightly or 1.15 it loads up opencl backend and gives corrupted output. This backend seems to take longer time to start up(5-10s); however it seems to be faster than corresponding opengl version. When i run the model to get a image output (float:0-1) there seems to be random rectangular blanks within the output. The input is 256x256x3(float) and output os 256x256x1(float). However.  i 'am not facing this issue using a different model with input size 128; even though i use the same back-end.
Describe the expected behavior
The tflite model should produce correct output with opencl backend like the  opengl version, regardless of input size.
Other info / logs
I'am getting correct output for model with 128 input size regardless of backends and devices; but for the model with 256 output size i'am not getting proper output with opencl backend (gpu-nightly and gpu-1.15)
Models.zip
Only the 'opencl gpu delegate with this 256 input sized-model' produces this corrupted output; other versions (CPU, Opengl-GPU, 128 input-model with Opencl etc.) seems to produce correct result without the rectangular blanks.

I forwarded this to our OpenCL engineer.  He's looking.

Btw, I spy 3 black rectangles (1 in the background).  Is it always the same location, or is it random?

Hm, we cannot reproduce this bug with Adreno 530 (Samsung S7 Edge; sorry, we don't have OnePlus3).  Do you see this on every frame or only occasionally?

It appears randomly on most of the frames. I'am able to run it at around 12 fps using opencl backend.The phone uses OxygenOS 9.0.6 and is based on android Pie.These rectangles appears  in both  foreground and background regions of the segmentation map(model is about portrait segmentation). Also, is it normal for the model to take this much time for initialization?

@anilsathyan7
Engineer suspects some problems in GPU/CPU synchronization when fetching output.  Unfortunately, he wasn't able to reproduce the problem on his device and is now on vacation until mid Jan.  Do you have the luxury of checking things on another device?
By the way, I heard your network has transpose convs in them.  There's been a commit 020e0de6b0c6741acae0735c41f9dff463230aa8 which should speed things up a bit.

Sure, i will check it on other devices. Currently i'am using java float array of shape [1][257*257] to get the flattened output from model. I also saw the recent android support library that tf has released, for handling I/O. Is it still in development phase or can we use it for  production?
Thanks

@anilsathyan7
While I don't have experience with the android support library you linked, things under experimental  are... well... experimental.
The APIs and/or implementation details may change, and in worst case, they may not survive experimental and may be killed.  In that sense, I wouldn't say it's safe to use it for production.  But if you're willing to eat up that development cost, go ahead :P  Sorry for not being able to give you a definitive answer there; I'm not directly part of the TFLite org and don't know their plans at the sub project level.

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


picture upload

add opportunity to select multiple pictures to upload via strg

This should also be supported via drag'n'drop #34


Footers

Hello,
Which file do i need to modify to change the footer on the main communications portal site ?
Thanks!

@gristy58 check out the following closed issue:
https://github.com/SharePoint/sp-starter-kit/issues/84
./provisioning/hubsite.xml in particular. You can also modify the collabFooter SPFx extension found in solution/src/extensions/collabFooter if you want to.

Hi, is there a way to remove the portalfooter completely?

@ruben1980 absolutely. Easiest way is to not activate it via a custom action in the provisioning template. Open https://github.com/SharePoint/sp-starter-kit/blob/master/provisioning/hubsite.xml in a text editor, look at line 500 (Currently) and remove the line:
<pnp:CustomAction Name="PortalFooter"... />
That will not deactivate the footer on an existing site though, for that you will need to use PowerShell.

As @eoverfield just mentioned, you'll need to modify the hubsite.xml file or use powershell to do it. I have written about it, and code is available on this post https://beaucameron.net/2018/07/19/customizing-the-pnp-starter-kit-for-office-365-part-1-portal-footer/

Thank you verry much for your support!

Has anyone successfully managed to get the portalfooter into its own package to deploy? I am currently trying to do this but I am running into an error when I try to do a gulp bundle --ship.
Its complaining about :
Error - [webpack] 'dist': ./lib/common/myLinks/MyLinksDialog.js Module not found: Error: Can't resolve 'MyLinksStrings' in 'C:/dev/SharePoint Dev/git/sp-starter-portal-footer/lib/common/myLinks'
I am not sure why its looking for that as a module.

Yes I have successfully moved it out into it's own package. MyLinks is what is used in both the portal footer and the collab footer. You can find it in the common folder https://github.com/SharePoint/sp-starter-kit/tree/master/solution/src/common/myLinks
This is what is used when managing the personal links in both of the footer extensions. You'll need to pull this into your solution, and update your references accordingly.
It's currently imported in the PortalFooterApplicationCustomizer.ts on lines 21 & 22.

Yup, I brought across the common folder contents (mylinks). Refactored MyLinksDialog.tsx using the new CommandBar (icons in items are now iconProps). I got the references for mylinks in PortalFooterApplicationCustomizer.ts as well.
I will push the change to my repository if you would like to look at the code.
One difference I noticed in the Lib folder is in MyLinksDialog.js, my file doesnt have "use strict"; at the top.
Following this, I saw in my file:
import styles from './MyLinks.module.scss';
import * as strings from 'MyLinksStrings';
Whereas for the sp-starter-kit, the same lib file has:
Object.defineProperty(exports, "__esModule", { value: true });
...
var MyLinks_module_scss_1 = require("./MyLinks.module.scss");
var strings = require("MyLinksStrings");
I am not sure why thats happening to be honest

I think I might have found a clue. At the top of the MyLinksDialog.tsx file is this line:
import * as strings from 'MyLinksStrings';
I think this might be the culprit because when Gulp generates the JS files, something in the new libraries is not reading this reference correctly.
I found this from this error message generated:
Field 'browser' doesn't contain a valid alias configuration
Digging further

@elpinto007 I have made a comment on your new post #122 , please check it out :)

Closed as this was answered.


ICorsPolicyProvider url restriction

HI Guys!
Sorry to bother byt i've run into an issue trying to use your IdentityServer in my project. To give you a better overview of my current problem my setup is:

i have a asp core RTM (but that problemy doesn't mater) user restricted Web API with you IdentityServer4 setup as in the sample for SPA.
Asp Core ui project with is hosted on different domain the the server. This one is just serving a static files with Angular2 website that is using your js client to authenticate users and request the api with the access token.

So the issue is that in the base implementation of your ICorsPolicyProvider you are allowing only specific paths to be used with CORS. Those paths live in Constants.RoutePaths.CorsPaths read only array.
All that will cause all other request to the api to be Unauthorized.
So the question what is the best work around this issue?
Shell i make my own custom implementation of a ICorsPolicyProvider and register it into a a standard Asp Core IoC container?
Or is there a way of specifying that other urls are valid?
Or maybe i'm just and ideat and that's not how we want to interact with the restricted web API?
Hope my description is precised enough.

the CORS design is not done yet - we know there are issues.

give input if you like #167


Add centos9 support to rancher-selinux

This PR adds  centos-stream9 (EL9) support to rancher-selinux .

image: quay.io/centos/centos:stream9

Here is the outcome of the policy compilation and installation, after the build:
bash-5.1# cat /etc/*release
CentOS Stream release 9
NAME="CentOS Stream"
VERSION="9"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="9"
PLATFORM_ID="platform:el9"
PRETTY_NAME="CentOS Stream 9"
ANSI_COLOR="0;31"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:centos:centos:9"
HOME_URL="https://centos.org/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux 9"
REDHAT_SUPPORT_PRODUCT_VERSION="CentOS Stream"
CentOS Stream release 9
CentOS Stream release 9
bash-5.1# ls
CODEOWNERS		   Dockerfile.centos9.dapper		Dockerfile.microos.dapper  README.md
Dockerfile.centos7.dapper  Dockerfile.centos9.dapper2218901142	LICENSE			   policy
Dockerfile.centos8.dapper  Dockerfile.centos9.dapper2819780759	Makefile
bash-5.1# cd policy/
bash-5.1# ls
centos7  centos8  centos9  microos
bash-5.1# cd centos9/
bash-5.1# ls
rancher-selinux.spec  rancher.fc  rancher.te  scripts
bash-5.1# make -f /usr/share/selinux/devel/Makefile rancher.pp
Compiling targeted rancher module
Creating targeted rancher.pp policy package
rm tmp/rancher.mod.fc tmp/rancher.mod
bash-5.1# semodule -i rancher.pp
libsemanage.semanage_rename: WARNING: rename(/var/lib/selinux/targeted/active, /var/lib/selinux/targeted/previous) failed: Invalid cross-device link, fall back to non-atomic semanage_copy_dir_flags()

Tracking issue in Rancher https://github.com/rancher/rancher/issues/43146.

rancher-selinux test on Centos9 (RHEL9) was successful.
Environment and versions used:

Server: Centos Stream 9 x86_64
Cluster: rke1-v1.26.8 (standalone with etcd/controplane/worker)
kubectl: v1.28.2
helm: v3.13.1
rancher: v2.7.6

Generate rancher-selinux rpm:

Generate the rancher-linux rpm (use the centos-stream9 branch), and copy the generated files locally from the container (will use it later to install it on the Centos9 server).

> podman run --rm -it -v $(pwd):/code quay.io/centos/centos:stream9
# yum install -y epel-release && yum -y install container-selinux selinux-policy-devel yum-utils rpm-build git
# ./policy/centos9/scripts/build

Procedure to test rancher-selinux rpm:

Spin up a Centos9 instance and install the requirements (rke, kubectl, helm, docker and SELinux packages)

  1  yum update -y
  2  yum install -y epel-release && yum -y install container-selinux selinux-policy-devel setools
  3  curl -LO "https://dl.k8s.io/release/$(curl -L -s https://dl.k8s.io/release/stable.txt)/bin/linux/amd64/kubectl"
  5  mv kubectl /usr/bin/
  6  chmod +x /usr/bin/kubectl
  8  yum install -y yum-utils
  9  yum-config-manager --add-repo https://download.docker.com/linux/centos/docker-ce.repo
 10  yum install docker-ce docker-ce-cli containerd.io docker-buildx-plugin docker-compose-plugin
 12  curl -LO https://github.com/rancher/rke/releases/download/v1.4.10/rke_linux-amd64
 14  chmod +x rke_linux-amd64
 15  mv rke_linux-amd64 /usr/bin/rke

Create a RKE1 cluster, use the RKE install minimum conf

@andypitcher I believe we can merge this PR now and generate a testing RPM, wdyt?
Thanks for adding the comment above. I believe that it should be pasted as is in https://github.com/rancher/rancher/issues/43146, as it will help QA when testing this.


Clean Up Sonarcloud Quality Gate Issues

Description
An attempt to clean up some Sonarcloud Quality Gate issues:

Main issues addressed:

Adding alt tags to images
Removing un-used imports
Simplifying multiple imports from the same source
Switching to string from String
Switching to 'boolean' from 'Boolean'
Removing unexpected empty Constructors

Additional:

When you visit https://qa.dockstore.org/docs you are redirected to https://docs.dockstore.org/en/stable/.html instead of https://docs.dockstore.org/en/stable/, removed + .html to fix redirect:
https://github.com/dockstore/dockstore-ui2/blob/429bd41bede34cfda0118280ee1c4f935648308a/src/app/docs/docs.component.ts#L69
Removed a chunk of old css in starredentries.component.scss to style elements/classes that no longer exist (checked history to verify)
Removed src/organizations.scss, file has no references made to it and history states this file was brought over from the previous UI1 repo in 2017

Review Instructions
All tests should be passing.  There should be less Sonarcloud Quality Gate issues than in the current screenshot.
Issue
SEAB-5338
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that your code compiles by running npm run build
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] If this is the first time you're submitting a PR or even if you just need a refresher, consider reviewing our style guide
[x] Do not bypass Angular sanitization (bypassSecurityTrustHtml, etc.), or justify why you need to do so
[x] If displaying markdown, use the markdown-wrapper component, which does extra sanitization
[x] Do not use cookies, although this may change in the future
[x] Run npm audit and ensure you are not introducing new vulnerabilities
[x] Do due diligence on new 3rd party libraries, checking for CVEs
[x] Don't allow user-uploaded images to be served from the Dockstore domain
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.
[x] Check whether this PR disables tests. If it legitimately needs to disable a test, create a new ticket to re-enable it in a specific milestone.

A few more placeholders, for example
https://github.com/dockstore/dockstore-ui2/blob/SEAB-5338/sonarcloud-quality-gate-bugs/cypress/e2e/group2/organizations.ts#L89

Looks like remaining places are using to test for regex, still updated to replace final instances of placeholder.com with gravatar


chore(cd): added assets to sem-release git plugin

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

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


feat: process workspace member glob

Should fix #10.
@Stupremee could you please test this?

Yes, it works!


export_fig broken in MATLAB pre-release 2016b

export_fig is not working for me in MATLAB 2016b pre-release.
The line below works fine in 2016a.
I do have Ghostscript installed.
The problem seems to be that using_hg2 no longer works:
>> plot(rand(4)); using_hg2(gcf)
ans =
  logical
   0

and indeed if I force using_hg2 to return true then export_fig works.
Here is the error message:
>> plot(rand(4)); export_fig('s:\out.pdf')
export_fig error. Please ensure:
  that you are using the latest version of export_fig
  and that you have Ghostscript installed
  and that you do not have multiple versions of export_fig installed by mistake
  and that you did not made a mistake in the expected input arguments

If the problem persists, then please report a new issue.

Error using fix_lines
Too many output arguments.
Error in print2eps (line 399)
        fstrm = fix_lines(fstrm);
Error in export_fig (line 635)
                print2eps(tmp_nam, fig, options, p2eArgs{:});

>> ver
----------------------------------------------------------------------------------------------------
MATLAB Version: <IP_ADDRESS>0522 (R2016b) Prerelease
MATLAB License Number: Prerelease
Operating System: Microsoft Windows 7 Home Premium  Version 6.1 (Build 7601: Service Pack 1)
Java Version: Java 1.7.0_60-b19 with Oracle Corporation Java HotSpot(TM) 64-Bit Server VM mixed mode
----------------------------------------------------------------------------------------------------

Fixed

Thanks!


Provide a convenience function which returns geographic data points on the original coordinate reference system

Is your feature request related to a problem? Please describe.
At the moment there is an inconsistency between what the to_points method returns from GribField objects vs NetcdfField objects, in that the netCDF objects will return the x and y points of the data in the data's coordinate reference system, whereas GribFields always return an array of lat/lon points.
Describe the solution you'd like
I think that to_points should always return the geographic x and y points of the data in the data's original CRS. We should also perhaps offer a to_latlon (or similar) method, which explicitly does the translation to latitudes and longitudes.
Describe alternatives you've considered
No response
Additional context
No response
Organisation
ECMWF

I would say the solution is that for now, in those circumstances, we throw exception in to_points() and when the grid library is there, we then do the right thing.

I to like the distinction of to_points and to_latlong as @JamesVarndell described.

Currently to_points returns a dict as {"lat": ndarray, "lon": ndarray} . What would be the keys in the new version?

I will then add to_latlon to return the latitudes and longitudes for GRIB fields. We still need to figure out what keys to_points should return for GRIBs where to original crs is known.

This was fixed by #102


Add possibility to prefix specific frameworks

It was already mentioned in #148 by @Mihailoff , I think it's a useful case to direct whole frameworks to specific paths instead of overriding specific modules on at a time.
{
    "extra": {
        "installer-paths-prefix": {
          "framework-folder": ["framework:wordpress"]
        }
    }

}

Actually duplicated it myself with #325 :(


Hi I am trying to install Deuteros on my mac but facing issues in installation

After the installation we have this massage ¨Your installation may require additional configuration steps.
Append the following to your DYLD_LIBRARY_PATH environment variable:
/Applications/MATLAB/MATLAB_Runtime/R2022b/runtime/maci64:/Applications/MATLAB/MATLAB_Runtime/R2022b/bin/maci64:/Applications/MATLAB/MATLAB_Runtime/R2022b/sys/osmaci64:/Applications/MATLAB/MATLAB_Runtime/R2022b/extern/bin/maci64
If MATLAB Runtime is to be used with MATLAB Production Server, you do not need to modify the above environment variable.In the following directions, replace MR/R2022b by the directory on the target machine where MATLAB is installed, or MR by the directory where the MATLAB Runtime is installed.
If the environment variable DYLD_LIBRARY_PATH is undefined, set it to the following string:
MR/R2022b/runtime/maci64:MR/R2022b/sys/os/maci64:MR/R2022b/bin/maci64
If it is defined, set it to the following:
${DYLD_LIBRARY_PATH}:MR/R2022b/runtime/maci64:MR/R2022b/sys/os/maci64:MR/R2022b/bin/maci64¨
after resolving this with ¨
(base) ritwikmaity@Ritwiks-MacBook-Air ~ % echo $DYLD_LIBRARY_PATH
(base) ritwikmaity@Ritwiks-MacBook-Air ~ % export DYLD_LIBRARY_PATH=MR/R2022b/runtime/maci64:MR/R2022b/sys/os/maci64:MR/R2022b/bin/maci64
(base) ritwikmaity@Ritwiks-MacBook-Air ~ % echo $DYLD_LIBRARY_PATH
MR/R2022b/runtime/maci64:MR/R2022b/sys/os/maci64:MR/R2022b/bin/maci64
(base) ritwikmaity@Ritwiks-MacBook-Air ~ % ¨
The programme wont run and reports ¨Cannot locate a valid install area¨

Sorry for the late reply. Are you still experiencing an issue with the installation? To be honest, I have no idea why that error is showing, as installation should be quite straightforward (albeit slow) with the MATLAB installer, nor have I encountered the error on any other device running MacOS.
This seems difficult to debug from my end, but if possible, could give me more information regarding which version of Deuteros you're trying to install, and the specification of your hardware, including which version of MacOS you're running, and whether it's running as a virtual machine, etc.
Thanks,
Andy

I have the same issue on MacOS.
I'm trying to install Deuteros 2.4.2 on MacOS 12.6.7 not running as a virtual machine. Could you help?


Drop travis CI

Circle CI builds take about 1 minute per platform
AppVeyor builds take about 45 seconds per platform
TravisCI builds take 3-6 minutes or more per platform.
TravisCI's build environment is way more out of date than CircleCI and AppVeyor.

In addition TravisCI's cache mechanism is completely different from CircleCI and is soley based on branch, language, and compiler instead of the much more useful directory paths.
https://docs.travis-ci.com/user/caching/

There is one cache per branch and language version/ compiler version/ JDK version/ Gemfile location/ etc.
Only modifications made to the cached directories from normal pushes are stored.

I would like to eventually drop TravisCI. It's slow and more difficult to maintain. However, it does have automatic setup for Pull Request builds. A PR build is what you'd get after merging the branch to master, which is important to check in cases where master has received merges from other independent branches, and so has diverged from the branch point of the current branch.
We could potentially do the PR builds from CircleCI for Linux and MacOS, though we'd need to manually set that up over there. I'm not currently sure of the details. I had in the past looked into how to get a PR build checkout. I never really investigated if anything special needed to be done for notify hooks.
AppVeyor does automatic PR builds for Windows.

We could potentially disable the branch builds (aka push builds) for TravisCI. That is now sufficiently covered by CircleCI.
Curiously, I noticed the branch builds were not running when I spot checked a few recent PRs. Might be an account/security related cause. It could be that only PR builds run when the push comes from a user account outside the organization.

Should we move forward with removing TravisCI? Seems all the rest are working.

Not yet. TravisCI is still doing the PR builds, which CircleCI hasn't yet been configured to do.
There is also the issue of building with both GCC and Clang, which I don't think was sorted out yet.
We're close, but not quite there yet.

Bleh. Welp, I'll leave it alone then for now.

I noticed we are no longer building on TravisCI, so I guess it was dropped at some point. We still have the .travis.yml file present though. Perhaps we should remove that file and close this issue.

For reference, I stumbled upon this comment about TravisCI being turned off:
https://github.com/lairworks/nas2d-core/pull/159#issuecomment-488162841


Convert *.rb to *.js

I can't use this plugin, because github forbid plugin on free account. Can u convert this plugin to JS file??

Unfortunately, that's not possible. The AMP pages must be generated on server, not client-side. Thus the Javascript would not work as intended.
One alternative is to build the site locally and only commit the generated site to your Github Pages repositoty. Alternately you might look into other hosting options that allow running Jekyll plugins.


Fix tests, remove unnecessary exit-clause

Fixes #19

IssueHunt Summary

Referenced issues
This pull request has been submitted to:

#19: Fix tests

IssueHunt has been backed by the following sponsors. Become a sponsor

Travis is still failing.

Closing in favor of https://github.com/sindresorhus/timed-out/pull/24


[ADD] Cardano support

Please add Cardano and token $ADA to the app.

It won't be possible to add ADA for now because they have its own curve.
https://github.com/vultisig/vultisig-ios/pull/1631


Bug: Could Not Fetch User Details in Navbar Due to Wrong JSON Configuration

When we login into the application, the Navbar must print Hello "Username" but instead, it's printing only Hello. Similarly to profile pic and role. It's happening due to the wrong JSON configuration. The current one is user.username, user.role, user.profilePic but instead the working one should be user.others.username, user.others.role, and user. others.profilePic.
Steps to reproduce the behavior:

Go to Login Page
log in to the System
In the Navbar, you will see only Hello but no username, etc are printed due to the wrong JSON configuration.

The expected behavior is to see the correct user role, profilePic, and username.
Current -

Expected -

Hey @mohityadav0903, please assign it to me. II would like to work on it for GSSoC '22.

hey @Shubham29012001 you can start working on it.

Hey @Dcode-k, I have raised a pull request to fix this bug issue.


Metadata Updates for DCP3 Catalog

This ticket is currently blocked by
https://app.zenhub.com/workspaces/dcp-ingest-product-development-5f71ca62a3cb47326bdc1b5c/issues/ebi-ait/dcp-ingest-central/77 and https://app.zenhub.com/workspaces/dcp-ingest-product-development-5f71ca62a3cb47326bdc1b5c/issues/ebi-ait/dcp-ingest-central/204
Details in the google doc<EMAIL_ADDRESS>Dataset/group this task is for:

A cell atlas of human thymic development defines T cell repertoire formation
HumanThymicDevelopment
Transcriptomic characterisation of haematopoietic stem and progenitor cells from human adult bone marrow, spleen and peripheral blood
AdultHemOrgans
Single-cell transcriptional landscape of human embryonic limb development
EmbryonicHindlimb
A human single cell atlas of the substantia nigra reveals novel cell specific pathways associated with the genetic risk of Parkinson’s disease and neuropsychiatric disorders
HumanBrainSubstantiaNigra

Description of the task:
For each individual dataset:

Work out what are the required changes to the metadata
Perform changes in ingest UI
Confirm changes
Export dataset submission
Fill in Broad import form, notifying that we are requesting an import
Broad notifies us as UCSC that import and snapshot is completed
UCSC notifies us that indexing is complete
Final checks

Some of these changes are required as a result of this issue: https://github.com/ebi-ait/hca-ebi-wrangler-central/issues/260 in which we made ad-hoc changes to the data in the staging area, and these changes were not reflected in ingest.
We have now confirmed that to be displayed as the Data Curator Role, a wrangler needs the ontology label (data curator (EFO:0009737)
Acceptance criteria for the task:
The above four projects are updated with correct metadata in the dcp3 catalog
HumanThymicDevelopment to display Marion Shadbolt as the data curator
AdultHemOrgans to display Ray Stefancsik as the data curator
EmbryonicHindlimb to display Enrique Sapena Ventura as the data curator
HumanBrainSubstantiaNigra to display 10X 3' v2 instead of 10X 3' v3 as the technique used

Closing this issue as superceeded by dataset specific ticksts


Cmake errors when building with the Dawn backend

I'm trying to use the Dawn backend in my project. However when running cmake -B build I get a serious of errors which seem to indicate missing directories.
output:
-- Dawn: using SPIRV-Headers at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/vulkan-deps/spirv-headers/src
CMake Error at build/_deps/dawn-src/third_party/CMakeLists.txt:23 (add_subdirectory):
  add_subdirectory given source
  "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/vulkan-deps/spirv-headers/src"
  which is not an existing directory.

-- Dawn: using SPIRV-Tools at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/vulkan-deps/spirv-tools/src
CMake Error at build/_deps/dawn-src/third_party/CMakeLists.txt:48 (add_subdirectory):
  add_subdirectory given source
  "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/vulkan-deps/spirv-tools/src"
  which is not an existing directory.

-- Dawn: using Abseil at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/abseil-cpp
CMake Error at build/_deps/dawn-src/third_party/CMakeLists.txt:67 (add_subdirectory):
  add_subdirectory given source
  "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/abseil-cpp"
  which is not an existing directory.

-- Dawn: using Vulkan-Headers at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/vulkan-deps/vulkan-headers/src
CMake Error at build/_deps/dawn-src/third_party/CMakeLists.txt:72 (add_subdirectory):
  add_subdirectory given source
  "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/vulkan-deps/vulkan-headers/src"
  which is not an existing directory.

-- Found PythonInterp: /usr/bin/python3 (found version "3.10.6")
-- Dawn: using python at /usr/bin/python3
-- Dawn: using jinja2 at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/jinja2
Traceback (most recent call last):
  File "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/generator/dawn_version_generator.py", line 18, in <module>
    from generator_lib import Generator, run_generator, FileRender
  File "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/generator/generator_lib.py", line 94, in <module>
    import jinja2
ModuleNotFoundError: No module named 'jinja2'
CMake Error at build/_deps/dawn-src/generator/CMakeLists.txt:65 (message):
  Dawn: Failed to get the dependencies for Dawn version based utilities.
  Base args are
  '/usr/bin/python3;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/generator/dawn_version_generator.py;--template-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/generator/templates;--root-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src;--output-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-build/gen;--dawn-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src;--jinja2-path;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/_deps/dawn-src/third_party/jinja2'.
Call Stack (most recent call first):
  build/_deps/dawn-src/src/dawn/common/CMakeLists.txt:15 (DawnGenerator)

-- Configuring incomplete, errors occurred!
See also "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn/build/CMakeFiles/CMakeOutput.log".

things I've checked:
I've also tried building the upstream Dawn project and it builds and runs demo code successfully. That's as far as my cmake knowledge will take me however.

Are these the very first line of CMake's output? It seems that the dependencies are not cloned.
The upstream Dawn projects requires the installation of an extra tool (depot_tools) to handle this, but it is overkill so I replaced it with this script but apparently in your case it did not run. Maybe the beginning of the log can hint why (you have Python installed, right? Upstream Dawn requires it too anyways).
Whatever the problem is with this script, the error message should indeed be more informative, let me know the extra log and I'll work on it!

@eliemichel
No sorry.

-- Dawn: using SPIRV-Tools at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/vulkan-deps/spirv-tools/src
CMake Error at build/_deps/dawn-src/third_party/CMakeLists.txt:48 (add_subdirectory):
  add_subdirectory given source
  "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/vulkan-deps/spirv-tools/src"
  which is not an existing directory.

-- Dawn: using Abseil at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/abseil-cpp
CMake Error at build/_deps/dawn-src/third_party/CMakeLists.txt:67 (add_subdirectory):
  add_subdirectory given source
  "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/abseil-cpp"
  which is not an existing directory.

-- Dawn: using Vulkan-Headers at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/vulkan-deps/vulkan-headers/src
CMake Error at build/_deps/dawn-src/third_party/CMakeLists.txt:72 (add_subdirectory):
  add_subdirectory given source
  "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/vulkan-deps/vulkan-headers/src"
  which is not an existing directory.

-- Found PythonInterp: /usr/bin/python3 (found version "3.10.6")
-- Dawn: using python at /usr/bin/python3
-- Dawn: using jinja2 at /home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/jinja2
Traceback (most recent call last):
  File "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/generator/dawn_version_generator.py", line 18, in <module>
    from generator_lib import Generator, run_generator, FileRender
  File "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/generator/generator_lib.py", line 94, in <module>
    import jinja2
ModuleNotFoundError: No module named 'jinja2'
CMake Error at build/_deps/dawn-src/generator/CMakeLists.txt:65 (message):
  Dawn: Failed to get the dependencies for Dawn version based utilities.
  Base args are
  '/usr/bin/python3;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/generator/dawn_version_generator.py;--template-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/generator/templates;--root-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src;--output-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-build/gen;--dawn-dir;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src;--jinja2-path;/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/_deps/dawn-src/third_party/jinja2'.
Call Stack (most recent call first):
  build/_deps/dawn-src/src/dawn/common/CMakeLists.txt:15 (DawnGenerator)

-- Configuring incomplete, errors occurred!
See also "/home/bobajeff/projects/webgpu_native_exp/LearnWebGPU_Dawn_bug_report/build/CMakeFiles/CMakeOutput.log".

For what it's worth I managed to get the upstream Dawn to work with the guide by manually fetching it adding it as a sub-directory in my project and linking in webgpu_dawn instead of webpgu in my project.
workingproject.zip

Thanks for the details! I just updated the dawn branch to better find Python when fetching dependencies, could you try again from a fresh clone?

Thanks! It's working now.

Great thanks for your feedback!


Allows to override the bbox

This PR adds a new user option (override_bbox) that allows to override the default use of the bounding box to generate the image, using the center instead.
Plus: wiki updated.
@CartoDB/frontend / @javisantana: review this, please :)

:+1:

Thanks!

as a side comment I would store test viz.json locally, we had problems with that in the past

Should we update the docs too?

Yeah, I did that too, look at the description :)

Description mentions wiki page not official docs

Ah, ok, there's a ticket for that: https://github.com/CartoDB/cartodb.js/issues/371


Alignment issue with longer lists

When using longer lists, alignment seems to break down.

Happening to me as well. @adityamr15 any idea how to fix this?

I thought it was gonna happen to me as well. But apparently it's because the component does not update due to enough reasons...
When you increase the amount elements in props.items the individual heights seem to not be recalculated, simply reloading the app (or just add/remove a hook call like useState in the parent component to make it rerender) worked for me!


Create jquery.js

What the program do ?

In what programming language it is written?

jquery.js

This pull request exeeds the max additions limit. Please fix it and reopen


If rune create is running in detach mode, then runelet does not use l…

…ogpipe.
Signed-off-by: jack.wxz<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add support to private-bridge

to support a private-bridge

create-private-bridge command has been added
private-bridge network type has been added
create,clone and prepare modified to support the new network-type
several other minor changes to support the new network type (list, info, ecc.)

Tests don't show any regression, so I'll merge it


Restart just Discord Integration

SUGGESTION
In order for the Rich presence to show up in discord, the client must be opened before webstorm. A feature suggestion would be to allow the plugin to be restarted by itself, instead of having to close and reopen the entire IDE. This would allow one to launch the discord client after webstorm has been launched, and still use the rich presence without having to relaunch the IDE.
Note: This was an unresolved issue #80 but was closed without a fix

In order for the Rich presence to show up in discord, the client must be opened before webstorm.

This is not exactly true.

A feature suggestion would be to allow the plugin to be restarted by itself, instead of having to close and reopen the entire IDE.

The plugin already does that, every 20 seconds until it gets an answer from the Discord client.

Note: This was an unresolved issue #80 but was closed without a fix

The issue (restarting the integration/resetting the connection) was fixed, it seems like it just doesn't fix the connetion.

Anyway, we're currently in the process of moving the new GameSDK which hopefully fixed the problems some people experience.


Adding the Code World, No Blanket research group at VT

Requesting to add the Code World, No Blanket research lab at Virginia Tech to the NRV Dev group page.

Thanks for adding this! Sorry it's been so long!


KAFKA-10357: Use validate and setup during internal topics creation

For KIP-698, we introduce new setup and validation methods for
internal topics. This PR uses the newly added methods and removes
the old methods.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Call for review: @rodesai @guozhangwang


Global PDF title needs to be changes

@AnjuJoon , could you please change the global Python PDF book title to "Lectures in Quantitative Economics with Python".

updated pdf sent to Matt


Client Allocation Request for: This is an organization

Client Allocation Questions
Core Information

Name: This is an organization
Website / Social Media: https://organization-website.com, (@organization, Twitter)
Region: Asia excl. Greater China
DataCap Requested: 420 GiB
Addresses to be Notarized: f27182818284590452353602874
Notary Requested: Claudia Richoux

This user’s identity has been verified through filplus.storage


[Bug]: Selecting datasources using keyboard shortcut selects previous entry and not selected entry, query template does not load

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Issue # 1: Selecting datasources from keyboard shortcut from the entity explorer ends up picking the previous entry and not the selected datasource.
Issue # 2: Creating queries this way leads to some query templates not loading at all (i.e. wait time is infinite). User has to refresh page to see the query template else the app shows a Loading..... placeholder that does not change (See at at 41s  mark of the attached video)
Steps To Reproduce

Use the entity explorer shortcut to select a datasource
Use your keyboard keys to scroll down and select a datasource, preferably to the next page on the dropdown to consistently repro this issue - observe the selected datasource's template does not open up, instead it's the option listed just before the selected option. (Issue # 1)
Some queries take a long time to load templates (wait time is infinite), only refreshing the page loads the template as expected (Issue # 2)

https://www.loom.com/share/2c5064caabf148338fd79b8f54a83a71
Public Sample App
No response
Version
Cloud

Issue 3 is the same as reported in https://github.com/appsmithorg/appsmith/issues/15127.


[8.x] Add a BladeCompiler::renderComponent() method to render a component instance

This PR is similar to the functionality introduced in #40425.
This new method allows you to call BladeCompiler::renderComponent($component), where $component is an instance of Illuminate\View\Component, and get the HTML of the rendered component returned. Alternatively you may call the renderComponent method from the Blade facade: Blade::renderComponent($component).
Example
class HelloComponent extends Component
{
    private $name;

    public function __construct(string $name)
    {
        $this->name = $name;
    }

    public function render()
    {
        return "Hello, $this->name";
    }
}

Blade::renderComponent(new HelloComponent('Claire'));

// Returns 'Hello, Claire'

Rationale
Components are a great way to encapsulate parts of the view. Rendering a component programatically is useful if you want to render a component's HTML directly into a response. For example, if you wanted to dynamically update a component via XMLHttpRequest.
Currently to achieve this you would need to create a new view partial to return in the response, for the sole purpose of rendering the component:
<x-hello-component :name="$name"/>

return view('partials.hello-component', ['name' => 'Claire']);

The Blade::render() function makes this a bit easier, as you could render the component as a Blade string into the response, rather than needing to resort to a partial:
return Blade::render('<x-hello-component :name="$name"/>', ['name' => 'Claire']);

However, this still doesn't have optimal ergonomics, especially with increasing numbers of properties. The renderComponent method cuts out the intermediate step and makes this simple:
return Blade::renderComponent(new HelloComponent('Claire'));

Note that if needed, attributes can be set on the component instance like so:
return Blade::renderComponent((new HelloComponent('Claire'))->withAttributes(['class' => 'red']));

Implementation
The implementation is based on the renderComponent method of the Illuminate\View\Concerns\ManagesComponents trait.

I'm not following how
return Blade::renderComponent(new HelloComponent('Claire'));

is better than
return Blade::render('<x-hello-component name="Claire"/>']);

I'm also struggling to understand how returning a component is better than returning a partial?

Couldn't you also just
return (new HelloComponent('Claire'))->render();

if you're going to instantiate the class anyway?

@browner12 For your first question, you can't always want to interpolate prop values into a template string. Imagine if one of the props the HelloComponent takes is an object, like a Model instance:
$user = User::find(1);
return Blade::render('<x-hello-component user="'.$user.'"/>'); // doesn't work!

The next best thing is to pass that data into the BladeCompiler::render method. But imagine if there are a lot of props we need to pass in:
return Blade::render('<x-hello-component :user="$user" :foo="$foo" :bar="$bar" :qux="$qux"/>', ['user' => $user, 'foo' => $foo, 'bar' => $bar, 'qux' => $qux]);

This works, but it's not very ergonomic – there's a lot of repetition. We also lose the benefit of type hinting from the component's constructor parameters.
Regarding partials – they are a valid way to go about rendering a component. But it requires going to the effort of creating a new template file for every component you want to render, and again, the repetition of passing data into the view and then into the component.
You can't call render directly on a component instance, because it doesn't simply return a string. It can return any one of the following:

A blade template string
A View instance
A Htmlable instance
A closure that accepts data and returns any one of the above.

So, some logic is needed to determine what's been returned, and how to convert it into a string. This makes up the bulk of the renderComponent method.

Cool, thanks @tobyzerner 👍


Format using black

Figure I'm on a roll of suggesting cosmetic rather than substantive changes so may as well continue.
Problem
As more people contribute to this repo it will become more and more difficult to maintain a consistent code style and keep everything readable.
Proposed Solution
Enforce all code to be autoformatted by https://github.com/psf/black. This is nice because unlike tools like pep8 or flake8 etc it will actually modify the code to conform to the style guide for you.
Additional context
One of the few configurable options is the line length, so this woud not constrain line length.
I got hooked on this when we added it to jupyterlab-git and now I use it in all my personal projects. Here is an example PR of formatting everything + adding a formatting check to tests + adding a precommit git hook to autoformat https://github.com/jupyterlab/jupyterlab-git/pull/763
It also is supported in editors and there is even an extension integrating it into jupyterlab https://jupyterlab-code-formatter.readthedocs.io/en/latest/index.html

I've never used jupyterlab, so I don't really understand everything you're suggesting here. Does it work with Pycharm? Are there additional setup/configuration things that would be needed to do it? Can it be applied to notebooks that people PR as well?
I think an autoformatter is great in principle, though I'm hesitant to implement anything that will add extra steps for novice python users to make contributions. I have had the experience before of contributing to an open source project and having to figure out how to use their formatting scripts to do so, which I found quite annoying.
Especially considering the relatively small size of the Python codebase in this project, I would want to make sure this doesn't introduce any barrier of entry to contributing

I've never used jupyterlab, so I don't really understand everything you're suggesting here. Does it work with Pycharm?

Ah, my bad. Jupyterlab was a bit of red herring, I mentioned it because it's surprising that it works for notebooks because black is designed to work on files. black is first and foremost a command line tool, you can run it python -m black . (or black .) from the command prompt and it will correctly format all the files in that directory. There are also lots of integrations into IDEs e.g. for pycharm: https://black.readthedocs.io/en/stable/editor_integration.html#pycharm-intellij-idea

Can it be applied to notebooks that people PR as well?

Yes, though this is maybe something that would be better to do manually. I use https://jupyterlab-code-formatter.readthedocs.io/en/latest/index.html and there's an option to format the entire notebook

I think an autoformatter is great in principle, though I'm hesitant to implement anything that will add extra steps for novice python users to make contributions. I have had the experience before of contributing to an open source project and having to figure out how to use their formatting scripts to do so, which I found quite annoying.

I would want to make sure this doesn't introduce any barrier of entry to contributing

I've been there, the flake8 linting errors when contributing to matplotlib can be extremely confusing. This can definitely be implemented in such a way that it doesn't introduce a barrier.  Essentially the direction would be:

make your PR and get the content reviewed
at the end of PR ask them to do:

pip install black
black . # from command line

this can also be set up so that it runs every time you git commit but thats not necessary.
This could also be something that just gets done every now and again as a one-off

Got it. Yeah that sounds like a good idea then. Certainly at minimum as something that can be done manually from time to time, and then on top of that if there's some mechanism to automatically suggest to contributors to do it when PRing. Would also be good for the project's swag to have one of those

Would also be good for the project's swag to have one of those

That's honestly the strongest possible argument for it :)

there's some mechanism to automatically suggest to contributors to do it when PRing

Using pre-commit will do this.
You can make the contributing install instructions look something like
git clone <your fork>
cd pycro-manager
pip install -e ".[dev]"
pre-commit install

and then everything will be set up.

And you can add an automatic check on to run on PRs using github actions. see here https://github.com/ianhi/mpl-interactions/blob/04eb663503dcd80556c2a5e7ac74348464af2621/.github/workflows/test.yml#L52

Sounds great, go for it!


Xandra.Cluster does not reconnect

Automatic reconnection after a Cassandra restart does not work with Xandra.Cluster (it does with DBConnection.Poolboy).
Environment

Elixir & Erlang versions (elixir --version): Erlang/OTP 20 [erts-9.3], Elixir 1.6.3

Operating system: macOS 10.13.3

Current behavior
iex> {:ok, conn} = Xandra.start_link(nodes: ["localhost"], pool: Xandra.Cluster, underlying_pool: DBConnection.Poolboy, pool_size: 2)
Xandra.execute(conn, "SELECT key FROM system.local LIMIT 1", [], pool: Xandra.Cluster)
iex> {:ok, #Xandra.Page<[rows: [%{"key" => "local"}], more_pages?: false]>}

If I now shut down the local Cassandra instance, I get the following.
iex(3)> 2018-03-20 18:39:07.587 module=Xandra.Cluster.ControlConnection pid=<0.524.0> [debug]: Received STATUS_CHANGE event: %Xandra.Cluster.StatusChange{address: {127, 0, 0, 1}, effect: "DOWN", port: 9042}
iex> Xandra.execute(conn, "SELECT key FROM system.local LIMIT 1", [], pool: Xandra.Cluster)
** (Xandra.ConnectionError) action "checkout from cluster #PID<0.520.0>" failed with reason: not connected to any of the nodes
    lib/db_connection.ex:934: DBConnection.checkout/2
    lib/db_connection.ex:750: DBConnection.run/3
    lib/db_connection.ex:1141: DBConnection.run_meter/3
    lib/db_connection.ex:644: DBConnection.execute/4
    lib/xandra.ex:802: Xandra.execute_without_retrying/4

After restarting Cassandra, nothing happens. Trying to execute the above query again I always get the same connection error.
Expected behavior
I would expect the same behaviour as with DBConnection.Poolboy, i.e. connection errors from the logger when Cassandra is down but automatic reconnection once Cassandra is available. From my understanding of DBConnection and the Xandra documentation this is the expected behaviour.
In more detail:
When I shut down a local Cassandra installation on a DBConnection.Poolboy pool I get the following errors:
2018-03-20 18:43:01.934 module=DBConnection.Connection pid=<0.524.0> [error]: Xandra.Connection (#PID<0.524.0>) disconnected: ** (Xandra.ConnectionError) action "ping" failed with reason: socket is closed
2018-03-20 18:43:01.942 module=DBConnection.Connection pid=<0.524.0> [error]: Xandra.Connection (#PID<0.524.0>) failed to connect: ** (Xandra.ConnectionError) action "TCP connect" failed with reason: connection refused
...

When I restart Cassandra the error messages stop and I am able to execute the above query successfully.

With the switch to DBConnection 2.x and the reimplementation of Xandra.Cluster as its own process (see #140) I revisited this issue and tested if connection recovery is still a problem.
TL;DR
Reconnection is still buggy with the new Xandra.Cluster implementation.
Environment

Elixir & Erlang versions (elixir --version): Erlang/OTP 21 [erts-<IP_ADDRESS>], Elixir 1.8.2
Xandra version: 766cdcb482ccf85bb9a9348f0cd513932b0aec33
Operating system: macOS 10.14.6

Current behaviour
When starting a connection against a local Cassandra instance and then stopping the instance queries will fail with an error. This is as expected. When restarting the Cassandra isntance and waiting queries against the connection still fail, i.e. the connection does not revocer. This is bad!
Steps to reproduce
$ iex -S mix
iex(1)> {:ok, conn} = Xandra.Cluster.start_link([])
{:ok, #PID<0.206.0>}
14:54:19.780 [debug] Control connection for <IP_ADDRESS>:9042 is up
14:54:19.791 [debug] Started connection to {127, 0, 0, 1}
14:54:19.792 [debug] Discovered peers: []
iex(2)> Xandra.Cluster.execute!(conn, "SELECT data_center FROM system.local") |> Enum.at(0)
%{"data_center" => "datacenter1"}

# here I stop the local Cassandra instance

14:54:29.795 [debug] Received event: %Xandra.Cluster.StatusChange{address: {127, 0, 0, 1}, effect: "DOWN", port: 9042}
iex(3)> Xandra.Cluster.execute!(conn, "SELECT data_center FROM system.local") |> Enum.at(0)
** (Xandra.ConnectionError) action "checkout from cluster #PID<0.206.0>" failed with reason: not connected to any of the nodes
    (xandra) lib/xandra/cluster.ex:344: Xandra.Cluster.execute!/3

# here I restart the local Cassandra instance

14:54:54.876 [debug] Discovered peers: []
iex(3)> Xandra.Cluster.execute!(conn, "SELECT data_center FROM system.local") |> Enum.at(0)
** (Xandra.ConnectionError) action "checkout from cluster #PID<0.206.0>" failed with reason: not connected to any of the nodes
    (xandra) lib/xandra/cluster.ex:344: Xandra.Cluster.execute!/3

Expected Behaviour
I would expect the connection to revover, i.e. after Cassandra is available again we should be able to successfully issue queries through the connection. This is the  behaviour a "normal" (non-clustered) connection exhibits.

I tested reconnection with a Xandra.Cluster connection with two nodes and only restarting one node, which works!
$ iex -S mix
iex(1)> {:ok, conn} = Xandra.Cluster.start_link(nodes: ["<IP_ADDRESS>", "<IP_ADDRESS>"], autodiscovery: false, load_balancing: :priority)
{:ok, #PID<0.206.0>}
16:40:11.515 [debug] Control connection for <IP_ADDRESS>:9042 is up
16:40:11.528 [debug] Started connection to {65, 12, 123, 11}
16:40:11.528 [debug] Control connection for <IP_ADDRESS>:9042 is up
16:40:11.528 [debug] Started connection to {65, 12, 123, 10}
iex(3)> Xandra.Cluster.execute!(conn, "SELECT data_center, listen_address FROM system.local") |> Enum.at(0)
%{"data_center" => "DC1", "listen_address" => {65, 12, 123, 10}}

# here I stopped <IP_ADDRESS>

16:40:34.152 [debug] Received event: %Xandra.Cluster.StatusChange{address: {65, 12, 123, 10}, effect: "DOWN", port: 9042}

16:40:34.164 [debug] Received event: %Xandra.Cluster.StatusChange{address: {65, 12, 123, 10}, effect: "DOWN", port: 9042}

iex(5)> Xandra.Cluster.execute!(conn, "SELECT data_center, listen_address FROM system.local") |> Enum.at(0)
%{"data_center" => "DC1", "listen_address" => {65, 12, 123, 11}}

# here I restarted <IP_ADDRESS>

16:41:27.899 [debug] Received event: %Xandra.Cluster.StatusChange{address: {65, 12, 123, 10}, effect: "UP", port: 9042}
iex(6)> Xandra.Cluster.execute!(conn, "SELECT data_center, listen_address FROM system.local") |> Enum.at(0)
%{"data_center" => "DC1", "listen_address" => {65, 12, 123, 10}}

Observations:

During the downtime of <IP_ADDRESS> the query is issued to <IP_ADDRESS>. This is good!
After restarting <IP_ADDRESS> (and waiting a bit) the query is issued against <IP_ADDRESS> against, indicating that the reconnection was successful!

I had a look at the code and did some testing and I think I know what the problem is:

if a node goes down only the control connection process survives, the whole connection pool is terminated (see Xandra.Cluster.handle_status_change(state, %{effect: "DOWN", address: address}))
if the node comes back the control connection process re-establishes the connection to the node, but the pool is not restarted
the pool is only restarted if an "STATUS_CHANGE=UP" event is received
the "STATUS_CHANGE=UP" event can only be received on a different, pre-exisiting connection that is still connected to a Cassandra node
this means that re-establishing connectivity via the control connection does not restart the pool
the fix is to restart the pool if the control connection process re-establishes connectivity to the Cassandra node

@jvf do you think it makes sense to re-establish the connection to the pool when the control connection reconnects only if we have a single node? Because it seems like if we have two we might want to wait for STATUS_CHANGE=UP to know that the other node actually re-joined the cluster?

@whatyouhide I think that this this a general problem, not only when using Xandra.Cluster with a  single node. Consider the following:

create Xandra.Cluster with two nodes
stop and restart both nodes simultaneously
only one node will reconnect correctly

I have seen this during testing
theoretically (depending on timing) no connection gets re-established correctly (I think this is unlikely though, I have not seen this in practice)

I think loosing part of the endpoints like this would be undesirable

However, I am pretty sure that a Cassandra Node only starts accepting client connections once it has fully joined the ring. I think I red that somewhere, I will try to find a good reference for this. If this is true it is safe to view successful establishment of the control connection as equivalent to STATUS_CHANGE=UP.

The best reference I found is from the protocol spec. It states that receiving a READY message "Indicates that the server is ready to process queries". A READY message is received in response to a STARTUP message as well as in reponse to a REGISTER message. Both of these messages are sent while establishing the control connection.
This is supported by what we can observe in the system.log when restarting a node <IP_ADDRESS>:
INFO  [main] 2019-09-06 12:58:14,413 YamlConfigurationLoader.java:89 - Configuration location: file:/etc/cassandra/cassandra.yaml
...
INFO  [HANDSHAKE-/<IP_ADDRESS>] 2019-09-06 12:58:24,807 OutboundTcpConnection.java:560 - Handshaking version with /<IP_ADDRESS>
...
INFO  [main] 2019-09-06 12:58:25,979 StorageService.java:1449 - JOINING: Finish joining ring
...
INFO  [main] 2019-09-06 12:58:26,053 StorageService.java:2292 - Node /<IP_ADDRESS> state jump to NORMAL
...
INFO  [main] 2019-09-06 12:58:34,502 Server.java:156 - Starting listening for CQL clients on /<IP_ADDRESS>:9042 (unencrypted)...

I think with this it is safe to assume that (re)stablishing the control connection means that the node is safe to be used as endpoint.


Contributor Recommendability

This issue is created to collect comments about the rolling release of "______"
This metric can be found here: https://docs.google.com/document/d/1P-Xf2tdXEwg9FH6hANet68paUzadvahI3dW4IGuxk0g/edit#heading=h.wmfp9nxbbdwc
Note: This metric is being developed using a Google Doc, and the "Convert to Markdown" add on.
See all release candidates: https://chaoss.community/metrics/
CHAOSS Metric Quality Checklist
This checklist is used for new and updated metrics to ensure we follow CHAOSS quality standards and processes. Below checklist items don’t have to be completed all at once: create the metric release candidate issue first and then start working on the checklist.
Process

[x] Create the “review issue” in the authoring WG’s repo for comments during review period and paste this template in
[x] Create pull request to edit or add metric to WG’s repo (after checking Content Quality and Technical Requirements below)
[x] Add the new metric or metric edit to release notes issue in working group repo
[x] Update the Metrics Spreadsheet
[ ] Create issue in CHAOSS/Translations repository to kick-off translation to other languages (please use the the translation issue template)
[x] "Metric Candidate Release" label added to the metric release candidate issue.
[ ] Metric was added to website

When above steps are completed:

[ ] Announce new/updated metric on mailing list, newsletter, community Zoom call, and Twitter. This can be coordinated with the community manager.

Content Quality

[x] Required headings are filled in, including Questions.
[x] Description provides context to metric
[x] Objectives list sample uses for the metric and desired outcomes
[x] If any, DEI uses of the metric are included Objectives
[x] Optional headings that have no content were removed
[x] Contributors section lists those contributors that want to be named
[x] The name of the metric is the same in (1) metric heading, (2) metric file name, (3) focus area, (4) metrics spreadsheet, (5) “review issue”, (6) translation issue, and (7) website

Technical Requirements

[x] Message in the metric markdown file that the metric will be part of the next regular release is at top of page and the links are correct (this is in the metric template)
[x] Metric file name is the full metric name and only contains lower case letters and hyphens (“-”) for spaces
[x] Images are included using markdown and relative links (as described in the metrics template)
[x] Images have at least one empty line above and below them
[x] Ensure images are placed in image folder and followed naming convention
[x] If new focus area is created, ensure focus area is added to wg repo readme and focus area folder readme
[x] Within the focus area, add the metric in the table and provide the link to the metric and metric question
[x] Ensure tables within metric are converted as image and placed in the image folder (both original MD and screenshotted PNG format) and follow the naming convention
[x] No HTML code in the metrics markdown file

@sgoggins Is this part of the current release or next release.

@sgoggins Is this part of the current release or next release.

This metric was submitted after the metric freeze and should be considered for the next release: 2022-03.

The name of this metric is inaccurate. It should be Project Recommendability.
Although, recommendability, in general, is a difficult word for a metric title so I would re think it.

@klumb name is correct. here is the correct link to the document finalized in last value working group meeting.
https://docs.google.com/document/d/1NIUIBGHJDwhht9lXt66uoSP4CfKvD4AHwEsVa4bDKZE/edit

@klumb I thought you are saying the name is incorrect in the document. After rereading your comment, I assume you are proposing the new name i.e. Project Recommendability. We can consider that and discuss it in our upcoming value meeting.

@klumb I thought you are saying the name is incorrect in the document. After rereading your comment, I assume you are proposing the new name i.e. Project Recommendability. We can consider that and discuss it in our upcoming value meeting.

Yes, I think this metric really needs a new name.

Please create an issue for this metric in the translations repo

@vinodkahuja could you do this?

Translation request done

I have updated translation.

This metric is ready for release.


authentication?

Hi :) love the idea of a Barchart API. I installed the package from pip. Following your simple example code, I'm getting an HTTP 401 error which would indicate an invalid API key. I tried explicitly passing my API key as a parameter, same thing. My API key does work in a standard web request. Any ideas?
Thanks! - Benn

Sorry @beifert
I wasn't aware it was now pip installable
This project is not a Barchart project. This project was forked from femtotrader’s project and extended/modified.
https://pypi.python.org/pypi/barchart-ondemand-client-python/0.0.4
see https://github.com/lanshark/barchart-ondemand-client-python
Pinging @lanshark
I wasn't aware of this fork.

@beifert can you post some of your code?  Preferably on my fork: github.com/lanshark/barchart-ondemand-client-python

Hello @lanshark,
I'd be very pleased to receive a PR from you to merge into https://github.com/femtotrader/barchart-ondemand-client-python
I also would like to tell you that changing licence from BSD to MIT is not fair (and I'm not sure that it's even legal to do this without author consent)
Being original author, I'd be very pleased to be author of
https://pypi.python.org/pypi/barchart-ondemand-client-python/0.0.4
I will of course, accept you as co-author because you wrote some interesting improvement, and as a maintainer if you are interested.
Kind regards

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.


(General) Generate sidebar with template service

Look into using the HTML service's template functionality to create the ESD sidebar. This will break up the current Sidebar.html file into a simple HTML template file, a CSS file, and a JavaScript file then assemble them together when needed. Should make editing code a little easier.

Example of loading files via templates can be seen here.


MsSql watcher v1.3.2 - net461 lib not published in nuget

I am just working to build a monitoring system using Warden. Observing an issue that the published libs on Nuget for MsSql watcher v1.3.2 only has the lib for  netstandard1.6 and net461 lib missing.
I also checked the prior version v1.3.1 and libs for both netstandard1.6 and net461 are there.
Could you please check and publish the missing lib?

Hi, apologies for the late reply. I don't have the access to the Windows machine, thus I did not include net461 + I'm moving this project to the .NET Core in general. However, I'll try to resolve this issue and also include the full .NET framework (hopefully by tomorrow).

I think it would be worthwhile to add a CI integration using Appveyor with the purpose of building the nupkg artifacts for all the targeted .Net frameworks. You can use the created artifacts to publish Nuget packages. This way you don't need a Windows machine to get things going.
I can certainly help with creating the Appveyor config which will ease the process for you.


forge config: Fix compiling issue on mac

this fixes an error when compiling the app on macOS
An unhandled rejection has occurred inside Forge:
Error: /var/folders/9b/419m4hqs453_18_fftblwxhm0000gn/T/electron-packager/darwin-arm64/roblox-rpc-darwin-arm64-AXDf61/Electron.app/Contents/Resources/app/node_modules/register-scheme/build/node_gyp_bins/python3: file "../../../../../../../../../../../../../Library/Frameworks/Python.framework/Versions/3.8/bin/python3.8" links out of the package

Works pretty well for MacOS, thanks for the PR, will merge it in later.


Date off by one day

Using prisma 2.24.1 and MySQL 8.0.25. Whenever I insert a date value, it gets stored in the database with a day off by one, for example:
Insert Tue May 30 2006 00:00:00 GMT+0200 (Central European Summer Time)
Database: 2006-05-29
I am using the DATE data type of MySQL. I also tried DATETIME to no avail.
My setup is:
myTable (
  id int primary key autoincrement,
  ...
  birthdate date,
  ...
)

schema.prisma according to prisma db pull
model myTable {
  id         Int       @id @default(autoincrement()) @db.UnsignedInt
  ...
  birthdate  DateTime? @db.Date
  ...
}

Insert
await prisma.$transaction(
    myTable.map((obj) => {
      const dataObject = {
        ...
        birthdate: Date object, containing the date,
        ...
      };
      return prisma.myTable.create({
        data: dataObject,
      });
  })
);

Any hints are welcome

The issue is unrelated to prisma, a funny behaviour of date-fns seems to create an off-by-one date.

The issue is unrelated to prisma, a funny behaviour of date-fns seems to create an off-by-one date.


Make Perspective Call From Java Servlet

This PR adds the code for handling the Perspective call given text and a language from the JS code. Currently it will return a null pointer exception because the JS code is not implemented yet on this branch. Otherwise, it return a variety of attributes; we can determine exactly which we want later on. Many of the attribute are only available in English or a select few number of languages.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it! and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

@googlebot I signed it!

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.
ℹ Googlers: Go here for more info.

@googlebot I fixed it.


Missing dependencies when used from NPM

Here are my steps to get the demos running:
npm install webvr-boilerplate
cd node_modules/
cd webvr-boilerplate/
python -m SimpleHTTPServer

I assume there's no build? there's no grunt/gulp, absolutely nothing in readme.md that hints at building.

Alright so THREE is missing and some other things.
Same errors. Still missing THREE.
npm install --save three

GET http://localhost:8000/node_modules/es6-promise/dist/es6-promise.js
(index):73 GET http://localhost:8000/node_modules/three/three.js
VRControls.js:6 Uncaught ReferenceError: THREE is not defined(…)(anonymous function) @ VRControls.js:6
VREffect.js:12 Uncaught ReferenceError: THREE is not defined(…)(anonymous function) @ VREffect.js:12
(index):99 Uncaught ReferenceError: THREE is not defined(…)
Module issues.
I'm missing something obvious that's undocumented. Please help?

Hi Michael, there actually is a build step. You can run npm build. Although the build is checked in, as are all of the node depsendencies.


Fix titles overflowing graph nodes

Closes #2614

"The text-overflow property only affects content that is overflowing a block container element in its inline progression direction (not text overflowing at the bottom of a box, for example)."
I shamefully broke this rule when carrying over my old node styling to the new graph node component
https://developer.mozilla.org/en-US/docs/Web/CSS/text-overflow
adding spans was also convenient for the MUI tooltip which expects a component that's not customized

Now the tooltips are displayed for all nodes, event those with short captions, where they are of no use. Can we somehow display them only after some caption length cut-off.
Is it possible to make the tooltip text bigger? It is tiny

Changed from .675rem (mui default) to 1rem what do you think?

I think it looks better, the old version was almost unreadable on my laptop.

Great - tooltips won't show unless the title is longer than 20 characters (might still get some cases where there's an unnecessary tooltip but I wanted to be safe) as well

I think you can safely make the check 25 characters, there is still some margin.
I can see all of the tooltips at once. I thought only one if the element is hovered is supposed to be displayed?

From the acceptance criteria: "As a user when I hover over the node, I will see the full name as a tooltip."


Envision Pause Button

A pause button is useful to help users of Envision survey the state of a frame of a SMARTS simulation. Current behaviour of envision allows scrubbing but not pausing of the simulation visualisation.

closed with #585


HTTPRequest: Expose and add methods to make working with headers easier

Closes 4734
Exposes has_header() and get_header_value() methods and adds get_dictionary_from_headers method.

Due to my idiocy I accidentally made a merge commit and now I cannot squash.
The last commit how ever has all the relevant commits squashed if that is helpful.

Should be fine in 63713


[Ingestion] [DBT] Discrepancy between "dataProcessInstance" model (not supporting "status" aspect) and source_helpers.auto_status_aspect() leads to 500 error in GMS

Describe the bug:
There is a discrepancy between DataProcessInstance entity that does not support "status" aspect, while, when created through Python-based ingestion source API layer, such aspect is automatically added by source_helpers.auto_status_aspect(...) function:
def auto_status_aspect(stream: Iterable[MetadataWorkUnit]) -> Iterable[MetadataWorkUnit]:
    """
    For all entities that don't have a status aspect, add one with removed set to false.
    """
    ...

This leads to such 500 error on the triggered REST POST call to datahub-gms:
2023-04-04 10:08:21,068 [qtp2106620844-121] INFO  c.l.m.r.entity.AspectResource:145 - INGEST PROPOSAL proposal: {aspectName=status, systemMetadata={lastObserved=1680602901026, runId=postgres-2023_04_04-09_58_20}, entityUrn=urn:li:dataProcessInstance:f8be5e45f445824568745dbd12710c17, entityType=dataProcessInstance, aspect={contentType=application/json, value=ByteString(length=18,bytes=7b227265...6c73657d)}, changeType=UPSERT}
2023-04-04 10:08:21,069 [qtp2106620844-121] INFO  c.l.m.filter.RestliLoggingFilter:55 - POST /aspects?action=ingestProposal - ingestProposal - 500 - 1ms
2023-04-04 10:08:21,069 [qtp2106620844-121] ERROR c.l.m.filter.RestliLoggingFilter:38 - Rest.li error:
com.linkedin.restli.server.RestLiServiceException: java.lang.RuntimeException: Unknown aspect status for entity dataProcessInstance
    at com.linkedin.metadata.restli.RestliUtil.toTask(RestliUtil.java:42)
    at com.linkedin.metadata.restli.RestliUtil.toTask(RestliUtil.java:50)
    at com.linkedin.metadata.resources.entity.AspectResource.ingestProposal(AspectResource.java:158)
...
Caused by: java.lang.RuntimeException: Unknown aspect status for entity dataProcessInstance
    at com.linkedin.metadata.entity.EntityService.validateAspect(EntityService.java:904)
    at com.linkedin.metadata.entity.EntityService.ingestProposal(EntityService.java:840)

with this exception thrown by this EntityService::validateAspect(...) method unsurprisingly failing to validate this DataProcessInstance entity against its schema.
Expected behavior:
I don't know if this wrong behaviour is due to the fact that DataProcessInstance entity should support status aspect or if it's the Python-based ingestion source API layer that should not automatically add one while calling datahub-gms for this particular entity (other ones might be impacted), but for sure DataProcessInstance handling through this Python-based ingestion framework should work fine without such 500 error on datahub-gms due to such discrepancy in metadata-model assumptions.
Additional context:
Occurs on DataHub v<IP_ADDRESS>
Re-opened by #7748

@sanromeo we recently added status aspects to dataProcessInstances (https://github.com/datahub-project/datahub/pull/10757). I believe the actual issue is that we released this change as part of v<IP_ADDRESS> CLI, instead of holding this change until 0.13.4.x for a corresponding 0.13.4 server release.
Will need to think about the best way to solve this.


is the get_total_active_balance() at current epoch designed even handling previous justification?

at version 4b4fc32,  beacon_chain, in methodprocess_justification_and_finalization(), there is:
 if get_attesting_balance(state, matching_target_attestations) * 3 >= get_total_active_balance(state) * 2:
the get_total_active_balance is not from previous epoch, I wonder if it is designed on purpose?
Maybe I don't understand it correctly but I am still wondering if it would be wrong in these cases:
Case 1: some validators exit at current epoch:
suppose we have 2 validators, A and B, each with 32 eth, both were active validators, and they voted to previous epoch, but A, B voted at two branches. And  only B exited at current epoch.
A--votes--->state at prev epoch-------state at current epoch(b exited)--------> we are here
B
|
votes
|
+---->another root at prev epoch
then:
total active balance:32 eth(b exited so her stake is not counted)
attesting balance: 32 eth(A's stake)
and then root at prev epoch is justified
But the root at prev epoch is not supposed to be justified because the  it only got 1/2 votes?
Case 2: some validators become active at current epoch
even if the root at previous epoch is supposed to be justified, it will never  be justified because the total balance were diluted by new validators at current epoch.
Even it could turn a justification from true to false, though it is not allowed and our spec doesn't handle it.

The change in the validator set is bounded slot to slot so the marginal difference (except in extreme cases of small amounts of validators) is very low and translates into a only a small marginal degradation in the max accountable safety of that epoch.
essentially this slightly affects the max security but in practice is so small that this approximation is fine and tracking historic total balances (in our calculation) was not worth added consensus complexity


Enabel serve-static options

Enable the possibility to set max-age for static files

What will this do if no configuration is passed in?
How does it interact with the existing express.static call?
And does it make sense to configure appy for better production static file
serving? nginx is always going to outperform it, for sure... that's why our
standard recommendation for Apostrophe involves a frontend proxy:
http://apostrophenow.org/howtos/production.html
On Tue, Jan 26, 2016 at 7:01 AM, Thomas König<EMAIL_ADDRESS>wrote:

Enable the possibility to set max-age for static files
You can view, comment on, or merge this pull request online at:
https://github.com/punkave/appy/pull/9
Commit Summary

Include serve-static module to make it possible to set the
Cache-Control-Header of static files
Merge pull request #1 from
sueddeutsche/feature-SZINT-2792-serve-static-max-age

File Changes

M appy.js https://github.com/punkave/appy/pull/9/files#diff-0
(17)

Patch Links:

https://github.com/punkave/appy/pull/9.patch
https://github.com/punkave/appy/pull/9.diff

—
Reply to this email directly or view it on GitHub
https://github.com/punkave/appy/pull/9.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Hey Tom!
If one wouldn't pass the serveStaticOptions to appy, nothing changes. Static files would be served with a Cache-Control-Header which says: max-age: 0.
You are right, serving static files with nginx would be a way much better solution, but unfortunately we are not able to this in our case. So the only way to cache these files is to do this over the CDN, and therefore it is necessary to set the Cache-Control-Header  to max-age: 31536000.

For bc reasons I'm leery of switching to a different module to provide
static file serving. appy is based on Express 3, which does not use
serve-static under the hood. However, express.static in Express 3 does
take an optional "options" object, and that object does support a MaxAge
option, check it out in the Express source (it's under-documented on the
site).
So a more minimal and safe change would be to add a staticOptions property
that gets passed as the second argument to express.static if it exists.
On Wed, Feb 10, 2016 at 2:59 AM, Thomas König<EMAIL_ADDRESS>wrote:

Hey Tom!
If one wouldn't pass the serveStaticOptions to appy, nothing changes.
Static files would be served with a Cache-Control-Header which says: max-age:
0.
You are right, serving static files with nginx would be a way much
better solution, but unfortunately we are not able to this in our case. So
the only way to cache these files is to do this over the CDN, and
therefore it is necessary to set the Cache-Control-Header to max-age:
31536000.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/appy/pull/9#issuecomment-182245386.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Hey Tom,
you're right. That would be easier. Although we need to pass an other object to the express.static call...


Animations: arbitrary properties

Hi guys,
Just wanted to check if there is any planned work for supporting arbitrary properties' animation.
While this addition seems quite trivial, it's really handy for many cases when a non-transform paraneters get animated. The immediate example is light intensity, which is integer and cannot be animated using current functionaly.
I would be happy to add this possibility to animation system and issue a pull request :)

We have thought about it. There are few vectors of approaching it. Would be great to get some ideas brainstormed first.
Considerations from me:

How is animation data produced, is it timeline with curve? What about positional data, is it splines, how they are authored?
We've got curves, they are good way to store value changing over time. And with curveset you can do vecs and colors. Will the system be a "link" that binds property and curve together and provides playing API?

As you've probably noticed I usually refer to how stuff works in Unity when there's a similar functionality :)
So, in Unity we have animation curves, which seem to be Cubic Hermite spline. Each curve controls a scalar value (with the exception for boolean properties of course), meaning even positions and rotations are animated on a per-component basic. This gives a very straightforward and universal approach to animating properties as creating the math for controlling curve shapes for scalar values allows to re-use the same logic for animating both transforms and user-defined properties.
The only tricky thing to consider is, of course, rotations: while you can interpolate Euler angles pretty safely for rotating around 1 axis, for the case of needing perfectly smooth rotations a spherical interpolation has to be utilised. However, spherical interpolation doesn't allow for rotations of over 180 degrees around a given axis, meaning it's required to maintain intermediate keyframes for such cases. In case of Unity this is done by the editor – whichever type of interpolation you choose, internally the curve just controls different properties of a rotation (either quat's coords or euler angles coords), with the missing keyframes injected when needed. So it's really up to the way the curve data is going to be produced.
As for built-in curves of playcanvas, I am afraid they have a critical limitation – I don't see it supporting custom tangents / control points on the curve, so they probably have to be extended. The other concern I can immediately see is the way the curve computes its values: it seems to be quite computational-heavy (I refer to https://github.com/playcanvas/engine/blob/master/src/math/curve.js#L107), so for animations that run at FPS rate it might require some optimisations.

@slimbuck + @ellt92: assigning this to you guys for now. Do we have more engine PRs coming related to this? If so, please link them to auto-close this issue. Otherwise, if we already have our implementation merged, feel free to just go ahead and manually close this. 😄

Closing this as the anim component provides the ability to animate any arbitrary component property of type float / vec / quat / color. A demo can be seen here: https://playcanvas.github.io/#/animation/component-properties


width and height with 0px

I'm trying to make a dynamic image of the server be shown with zoom or it is there but it is not shown
                    <ngx-image-zoom
                        [thumbImage]="content.items[0].mediaPath"
                    ></ngx-image-zoom>

Hi, I just released 0.5.0 with Angular9 support where I tweaked the image sizing a bit. Does the problem still exist in that version?
And when you say dynamic image, do you mean dynamic in that it's generated by server code rather than being a static image? Or do you mean dynamic in that you change what image is being shown?

The 2 ways both server and how to change it.
I will test the new version. Thanks

I had the same problem, so I downloaded the library's to figure it out ...
And, I had to change this line in the ngx-image-zoom.component.html to a fixed size ...
<div #zoomContainer class="ngxImageZoomContainer" [style.width.px]="this.thumbWidth" [style.height.px]="this.thumbHeight">
I don't think that the method responsible for assigning the width and the height is working properly.

Could you fix this problem?
To make it work we are editing subject file in mode_modules.
Looking forward to your update.

Could you fix this?
We are editing subject file under node_modules. Future updates will overwrite this change.

Any Update on this? or any workaround, i am stuck because of this

Hi, This is working for me...
its a manual solution, but its urgent now:
I'm using "ngx-image-zoom": "^0.6.0", Angular 9 and DOM elements:
in my html:
<lib-ngx-image-zoom id="elevatezoomBig" mode="hover" [thumbImage]="imgPrincipal" [fullImage]="imgPrincipal"></lib-ngx-image-zoom>
in my TS file:
 ngAfterViewInit() {     var c = document.getElementById("elevatezoomBig").children[0];     c.removeAttribute("style");     c.setAttribute("style", "imgContainer"); }
... and in my css file:
.imgContainer{   width: 100% !important;   height: auto !important; }
i'm not sure if is a best practice, but is a solution for me
Regards,

My workaround was:
Going to styles.css file in your angular project and add:
.ngxImageZoomContainer {
    width: auto !important;
    height: auto !important;
}


Add possibility to configure options validation in order to validate options at the start up time (not at runtime).

From @MaKCbIMKo on Monday, February 20, 2017 9:50:01 AM
This PR introduces the possibility to 'initialize' options at the start up time. It will prevent possible config issues that might occur in runtime because of incorrect config file.
Also, it allows to use initializers to write some custom logic to instantiate options objects.
Copied from original issue: aspnet/Options#171

From @dnfclas on Monday, February 20, 2017 9:50:05 AM
Hi @MaKCbIMKo, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

From @dnfclas on Monday, February 20, 2017 10:02:42 AM
@MaKCbIMKo, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

From @HaoK on Tuesday, March 7, 2017 1:19:24 PM
Take a look at https://github.com/aspnet/Options/issues/161 which is tracking some features which would be useful for options to provide for the Auth 2.0 work, named options will add a bit of a wrinkle since there'd need to be the ability to target All instances vs named instances, vs the default instance.  We haven't decided that we are bringing back named options yet, but its something to consider

From @MaKCbIMKo on Tuesday, March 14, 2017 11:21:37 AM
@HaoK, I've modified (a lot) of validation stuff. Moreover, I took a look on feature (about multi-tenant) and found that it might require some validation 'on creation' time. I also added that (wasn't not a big deal).
About multi-tenancy: it all depends on how we will configure all these 'names', where we will store cached instances (for different names), but I think it won't be so difficult to modify current validation to work in a new way.
But, only if you have some roadblocks that you still not sure (or don't know) how to break...

From @HaoK on Tuesday, March 14, 2017 11:27:10 AM
Thanks, I'll try to take a look this week

From @HaoK on Wednesday, March 15, 2017 5:16:25 PM
So I'm going to be starting a PR for https://github.com/aspnet/Options/issues/161, it will be a pretty lightweight feature to start, I'll submit it by EOD tomorrow, so you can provide input and insight since you've been playing around with validation for a while.  The initial critical feature is named options support, so the first iteration of the PR will likely have very minimal validation support (think Action<TOption>).

From @MaKCbIMKo on Thursday, March 16, 2017 6:13:16 AM
Ok, waiting for your PR.

From @HaoK on Thursday, March 16, 2017 12:22:25 PM
Take a look at https://github.com/aspnet/Options/pull/176

From @MaKCbIMKo on Thursday, March 16, 2017 1:19:58 PM
@HaoK - I reviewed https://github.com/aspnet/Options/pull/176 and left a comment.
What I found is that your PR might reduce the amount of changes and be some kind of 'base' for this PR. (see the comments for https://github.com/aspnet/Options/pull/176).

From @MaKCbIMKo on Wednesday, March 29, 2017 2:30:21 PM
@HaoK - Is it fine to base this PR over your initial Options 2.0 changes? Or there more changes are comming?
Maybe something in my PR need to be reconsidered...

From @HaoK on Wednesday, March 29, 2017 2:43:03 PM
Feel free to rebase if you want, there likely will be more changes coming, but it won't be for a few weeks

From @HaoK on Wednesday, April 26, 2017 1:00:28 PM
See https://github.com/aspnet/Options/pull/187, adding an Initialize set of actions that runs after Configure for fixup, you could potentially use these for validation as well.  Auth is going to run validation outside of options since it needs to run every request potentially, so it will only use this feature for fixup

From @MaKCbIMKo on Thursday, April 27, 2017 6:51:51 AM
@HaoK - At first view, 'Initialization' step might be used for validation, but this validation will be applied all the time. Is that fine? What if want to validate only one time at start up (is that real case ?) ?

From @MaKCbIMKo on Thursday, April 27, 2017 6:54:26 AM
update:  I implemented a common validation (for unnamed options).
Faced some troubles with named options: If we want to validate the options at start up, then we need to know all 'names'. It means that we need to store somewhere all names. But it argues with ability to inject the options that wasn't configured at all (because of IOptions default feature).
Still thinking about that.

From @MaKCbIMKo on Monday, June 26, 2017 9:36:54 AM
@HaoK - I have added a possibility to validate all named options. Please take a look.
p.s. there is some problems with Travis CI check (Error: Unable to locate libunwind. Install libunwind to continue) - not sure how to solve that weather the problem with my changes.

From @MaKCbIMKo on Tuesday, July 18, 2017 2:23:25 AM
Changes from dev branch has been merged. @HaoK - may I ask you to review the PR?

From @MaKCbIMKo on Tuesday, July 18, 2017 2:26:27 AM
About 'AppVeyor build failed' - seems like it faced the exception during downloading "https://dotnetcli.azureedge.net/dotnet/Sdk/2.0.0-preview3-006785/dotnet-sdk-2.0.0-preview3-006785-win-x64.zip" (I managed to download it locally). then it switched to alt download link "https://dotnetcli.azureedge.net/dotnet/Sdk/2.0.0-preview3-006785/dotnet-dev-win-x<IP_ADDRESS>-preview3-006785.zip" - which seems to be not working.
@HaoK - maybe we just need to try to re-run the AppVeyor build?

From @MaKCbIMKo on Friday, August 25, 2017 2:41:11 AM
@HaoK - did you have a chance to take a look at this PR?

From @HaoK on Friday, August 25, 2017 2:57:31 AM
I took a look, but we haven't signed up to do this work yet, so drilling into the details/implementation is a bit premature until we actually are trying to get this feature implemented.

From @MaKCbIMKo on Friday, August 25, 2017 3:01:57 AM
Do you mean that this feature is not in roadmap?
Can we do anything in order to sign this up?

From @HaoK on Friday, August 25, 2017 3:05:09 AM
Yes, feedback/comments in https://github.com/aspnet/Options/issues/227 saying why you want/need this feature will help get it prioritized higher/sooner.