Handle * dbname without throwing error

Description of your changes
Currently, if you pass dbname * you get an error:
ERROR 1221 (HY000): Incorrect usage of DB GRANT and GLOBAL PRIVILEGES

This checks if the value you pass is exactly *, and if it is, leaves it alone (to be not quoted by mysql), and keeps quoting the remaining parameters passed.
Fixes #79
I have:

[x] Read and followed Crossplane's contribution process.
[ ] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
I am new to go, and not entirely sure how to go about testing this properly.
I have tested the resultant SQL manually.
The "old" version with the quoted * throws the error, and the version without it quoted does not throw it.

Examples of queries that error:
GRANT PROCESS ON `*`.* TO 'datadog'@'%';

REVOKE PROCESS ON `*`.* FROM 'datadog'@'%';

To fix them, remove the backticks.

@alexbowers you'll also want to --signoff all of your git commits.
More details in the DCO Details link https://github.com/crossplane-contrib/provider-sql/pull/84/checks?check_run_id=5888447437

I receive the following error when trying to run make and make reviewable:
WARN [runner] Can't run linter goanalysis_metalinter: bodyclose: failed prerequisites<EMAIL_ADDRESS>analysis skipped: errors in package: [/Users/alexbowers/Development/opensource/provider-sql/pkg/controller/mysql/mysql.go:26:2: could not import github.com/crossplane-contrib/provider-sql/pkg/controller/mysql/grant (/Users/alexbowers/Development/opensource/provider-sql/pkg/controller/mysql/grant/reconciler.go:291:3: dbname declared but not used)]]
WARN [runner] Can't run linter unused: buildir: analysis skipped: errors in package: [/Users/alexbowers/Development/opensource/provider-sql/pkg/controller/mysql/grant/reconciler.go:291:3: dbname declared but not used /Users/alexbowers/Development/opensource/provider-sql/pkg/controller/mysql/grant/reconciler.go:316:3: dbname declared but not used -: could not load export data: no export data for "github.com/crossplane-contrib/provider-sql/pkg/controller/mysql/grant"]
ERRO Running error: buildir: analysis skipped: errors in package: [/Users/alexbowers/Development/opensource/provider-sql/pkg/controller/mysql/grant/reconciler.go:291:3: dbname declared but not used /Users/alexbowers/Development/opensource/provider-sql/pkg/controller/mysql/grant/reconciler.go:316:3: dbname declared but not used -: could not load export data: no export data for "github.com/crossplane-contrib/provider-sql/pkg/controller/mysql/grant"]
15:52:32 [FAIL]
make[2]: *** [build/makelib/golang.mk:194: go.lint] Error 1
make[1]: *** [build/makelib/common.mk:357: lint] Error 2
make: *** [build/makelib/common.mk:425: reviewable] Error 2

If that resolves the same problem, go for it.

Hi, this is still a problem

https://github.com/crossplane-contrib/provider-sql/pull/97/ is about to fix this, just kicked off the CI for it.


tooltip: fix empty template

close https://github.com/CartoDB/cartodb/issues/7736
it was setting an empty template (i.e. '') thus not rendering properly the tooltips, this is needed for the click infowindows, but not really for the tooltips at this point
CR @gfiorav

Hey @matallo I'm not sure I can do a frontend CR! Maybe ping someone from the frontend team, I'm just a casual visitor 😅

/cc @alonsogarciapablo

👍

🚀


Docs: move quickstart project info into interface-specific docs

This PR streamlines the quickstart flow by:

Moving the example project exposition into collapsible sections of each API-specific doc
Altering the quickstart left menu page order to reflect the linear flow among pages
Expanding each page's "next steps" to reinforce flow

The PR also removes the main menu "API" link to the Python pdocs and provides additional information on the Python reference page that links to the API docs.

make sure that the base url that we link everywhere still works

I intentionally kept that base URL page /quick_start in place so it would continue to work, and as far as I can tell it will.
Here's the base URL: https://sqlmesh.readthedocs.io/en/stable/quick_start/
Here's the local URL for updated version: http://<IP_ADDRESS>:8000/quick_start/


Android Studio can't find fabric.properties file

Issue

I forked the repository, cloned into a local directory and imported the project in Android Studio 3.6.1
While importing, Android Studio throws an error:

Caused by: java.io.FileNotFoundException: C:\Users\chinm\Desktop\Open Source\Kickstarter\android-oss\app\fabric.properties (The system cannot find the file specified)

Hi @Grandolf49, thanks for checking out the repo!
That's expected because it sounds like you haven't run make bootstrap in step 4 of the README. Let me know if you're still running into issues after!


Show the devcontainer label value specified in portsAttributes within the port notification instead of "your application"

With any port forwarding command, the devcontainer launches the output that looks similar to:

Hidden under the "ports" tab is the label associated with the forwarded application, e.g.

This yields a moment of confusion as to what "your application" is.
Thus, it would be great if the label name in portsAttribute could be specified in place of "Your application".
So, the proposal is if the following entry in devcontainer.json is present:
  "portsAttributes": {
  	"8787": {
  		"label": "RStudio",
  		"requireLocalPort": true
  	}
  },

We would want the notification panel to show the "RStudio" value in the label field instead of "Your application":

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

I don't know that all labels will make sense as a substitute for "Your application...". I will change it to be "Your application (RStudio)..."

@alexr00 thank you so much for working on the issue and submitting a fix! :)


TextEdit[] returned from provideDocumentFormattingEdits doesn't behave correctly?

I'm trying to determine the root cause of https://github.com/Microsoft/vscode-go/issues/16, but I don't see any problem with the TextEdit object I'm providing back to the provideDocumentFormattingEdits.  It seems that in some cases Code will corrupt the buffer in odd but deterministic ways.
I haven't been able to pin down the exact incorrect behaviour, but two example:
Example 1
Before text:
package caddy\n\nfunc main() {\n\tfmt.Println("Hello World! :)")\n}\n

Then replace the full range with:
package caddy\n\nimport "fmt"\n\nfunc main() {\n\tfmt.Println("Hello World! :)")\n}\n

The result will be
package caddyimport "fmt"func main() {\n\tfmt.Println("Hello World! :)")\n}\n

Example 2
Before text:
package caddy\n\nfunc main() {\n    fmt.Println("Hello World! :)")\n}\n

Then replace the full range with:
package caddy\n\nimport "fmt"\n\nfunc main() {\n\tfmt.Println("Hello World! :)")\n}\n

The result will be
package caddy\n\nimport "fmt"\n\nfunc main() {\n\tfmt.Println("Hello World! :)")\n}\n

Note that the only difference between these examples is the "\t" vs "    " in the original text.  The result in Example 1 has corruption in the string result in a seemingly random location.
Two question:

Is this a bug in the extension API, or am I somehow calling it wrong?
If it's a bug, is there any workaround?

@alexandrudima @bpasero Any thoughts on this?  Any chance there is a workaround?  This is currently blocking addressing the top piece of feedback on vscode-go and I'm not sure how to workaround it.

TL;DR; It is an issue in the formatCommand.ts edit trimming. A fix is on the way in the November recovery drop.
The format command tries to do some smart front/rear trimming of edits to reduce edit ranges. E.g.
Given a buffer containing "some cool cool text"
replace [0,19] with "some cool cool text" => no-op
replace [0,19] with "something interesting" => replace [4,19] with "thing interesting"
replace [0, 19] with "some interesting text" => replace [5,14] with "interesting"
It had a bug when prefix & suffix overlap. e.g.:
replace [0, 19] with "some cool and cool text" => invalid edit

You can either wait for our recovery drop due end of this week or early next week or you can try to massage the edits on your side such that there is no prefix & suffix overlap. Removing the same prefix would already work.

616b9ad


Roll Flutter Engine from 9f247021387a to 3d3a1b89e696 (1 revision)

https://github.com/flutter/engine/compare/9f247021387a...3d3a1b89e696
2023-02-24<EMAIL_ADDRESS>Roll Skia from aa36fbbd0c1d to 3f396dc3ef80 (1 revision) (flutter/engine#39849)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Update gui_base_class.py

Codecov Report

Merging #43 (b90e207) into main (902d0bc) will not change coverage.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##              main       #43   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           71        71
  Lines         3899      3899
=========================================
  Hits          3899      3899

Impacted Files
Coverage Δ

ScenarioGUI/gui_classes/gui_base_class.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Enforce valid associations for StudentSchoolYear

Notes

#264

This might be a nice time to add in the gem in https://github.com/studentinsights/studentinsights/pull/241, and then the constraints just for these columns.  That way we can start ticking off the remaining constraints rather than trying to do them all in one batch.


16.03 moreutils 0.57 is a 404 link

Basic info
Linux toadman 4.3.3 #1-NixOS SMP Thu Jan 1 00:00:01 UTC 1970 x86_64 GNU/Linux
15.09.1180.e8e1cb8 (Dingo)
nix-env (Nix) 1.10
"16.03.468.2d840da"
Describe your issue here
Trying to upgrade to 16.03 on my laptop I get stuck here.  It looks like the newer version 0.58 is in master already.  seems like most easy way to fix it will be to pull that change into 16.03  otherwise one could change the link to http://archive.ubuntu.com/ubuntu/pool/universe/m/moreutils/ which also has a copy of the tarball that moreutils is looking for.

The moreutils changelog only lists two minor changes in 0.58, so I think we can safely backport it to stable.


[TF 2.0 API Docs] tf.data.Dataset.map() example has web render issues.

URL(s) with the issue:
https://www.tensorflow.org/versions/master/api_docs/python/tf/data
https://www.tensorflow.org/versions/master/api_docs/python/tf/data/Dataset
https://www.tensorflow.org/versions/master/api_docs/python/tf/data/TextLineDataset
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/data/ops/dataset_ops.py
Description of issue (what needs changing):
The map() method call for each of these classes has an example which is not being rendered properly.
This is what is intended by the docstring
   # NOTE: The following examples use `{ ... }` to represent the
    # contents of a dataset.
    # Each element is a `tf.Tensor` object.
    a = { 1, 2, 3, 4, 5 }
    # `map_func` takes a single argument of type `tf.Tensor` with the same
    # shape and dtype.
    result = a.map(lambda x: ...)

This is what gets rendered to the web.  It looks like the back tick references have an issue.
# NOTE: The following examples use `{ ... }` to represent the
# contents of a dataset.
# Each element is a tf.Tensor object.
a = { 1, 2, 3, 4, 5 }
# `map_func` takes a single argument of type tf.Tensor with the same
# shape and dtype.
result = a.map(lambda x: ...)

Submit a pull request?
No, the docstring looks correct.  I don't know how to fix the error.

Why are there reference links inside the code after all? I'm confused.

The code creates documentation via doc strings.

I don’t know if the code links work or not.  That is a good point.

The issue also exists in r2.0 https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/data/Dataset#for_example_5

Regarding the links in the code block, we auto-link backticked TF symbols, so it's probably written as `tf.Tensor`. We probably shouldn't expand links in code blocks, if we can detect that.

Regarding the links in the code block, we auto-link backticked TF symbols, so it's probably written as tf.Tensor. We probably shouldn't expand links in code blocks, if we can detect that.

Could we check for "#" in the same line? In that way we could know that it is a code comment and thus, not link it.

Yes, that would work.

I would like to work on it if that is okay with you and if you could point me in the right direction as I don't know where this is done.

Hey, you can definitely work on it. Thanks.
To start, changing the regular expression here (https://github.com/tensorflow/docs/blob/master/tools/tensorflow_docs/api_generator/parser.py#L118) to not consider lines that start with '#' should solve it.
Good luck and let me know if you have any questions :)

I have opened a PR. I believe that should do the work.
Thanks!

I think this issue could be closed now as it is solved!

Thank you, Anestis!


JBPM-4850: Add warning when clicking reset default filters

+1 thanks @nmirasch

@nmirasch just a suggestion, but what do you think if we change the reset default filters icon to this one: http://fontawesome.io/icon/filter/ ?

@nmirasch hi, do you have an update on this? please don't forget that we need it on the 6.4.x branch as well, when ready.


fix(a11y): add tabstop on info panel

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

My previous PR removed the tab stop on the Code panel in the mobile layout because it was unnecessary. This PR adds a tab stop on the Info panel because it is definitely necessary in several situations.

Some Info panels do not have any focusable elements in them and thus it is currently impossible for a keyboard user to scroll the content of the Info panel when needed. See the Stand in Line challenge.
As suggested by the ARIA Authoring Practices Guide, unless the first element in the tab panel is focusable then the tab panel itself should be a tab stop. This will be particularly helpful for screen reader users as it will guarantee they always get dropped into the tab panel at the very beginning of the content.

You should just be able to toggle the "draft" feature while you are still working on something :)

My bad. It's ready for review.


Auto run dcos auth login when token expired

if core.dcos_auto_login is set.
$ grep auto ~/.dcos/dcos.toml
dcos_auto_login = true

$ dcos node
If your browser didn't open, please go to the following link:

    http://<redacted>/login?redirect_uri=urn:ietf:wg:oauth:2.0:oob

Enter OpenID Connect ID Token: <paste token>
Login successful!
  HOSTNAME         IP                          ID
<IP_ADDRESS>  <IP_ADDRESS>  74df965e-a46e-46a9-8d2a-639db0af0e25-S0
<IP_ADDRESS>  <IP_ADDRESS>  74df965e-a46e-46a9-8d2a-639db0af0e25-S4
<IP_ADDRESS>   <IP_ADDRESS>   74df965e-a46e-46a9-8d2a-639db0af0e25-S2
<IP_ADDRESS>   <IP_ADDRESS>   74df965e-a46e-46a9-8d2a-639db0af0e25-S3

Can one of the admins verify this patch?

@tamarrow: Addressed both comments. Thanks!

@tamarrow: Any other changes needed here?

Oy. I wish we could get rid of these CCM errors. I don't know what CCM is, but it seems to fail a lot.

Ha yea... it stands for Cluster Create Manager. It's a wrapper around aws, but I hear it should get re-written soon.

ok to test

[auth_token_auto_login f3881bc] Remove unnecessary imports
1 file changed, 4 deletions(-)

Huh?

Fixed:
dcos/http.py:177:80: E501 line too long (81 > 79 characters)

with 059ee29.

jwt is already installed in setup

but I reproduced the error locally:
$ git checkout 059ee29d90a62c03f413f671efd222669b5dcfd4
...
HEAD is now at 059ee29... Fix line too long error

$ tox -e py35-unit
GLOB sdist-make: /Users/abramowi/dev/git-repos/dcos-cli/setup.py
py35-unit inst-nodeps: /Users/abramowi/dev/git-repos/dcos-cli/.tox/dist/dcos-SNAPSHOT.zip
py35-unit installed: appdirs==1.4.3,coverage==4.3.4,dcos===SNAPSHOT,jsonschema==2.6.0,mock==2.0.0,packaging==16.8,pager==3.3,pbr==2.0.0,prettytable==0.7.2,py==1.4.32,Pygments==2.2.0,pyparsing==2.2.0,pytest==3.0.7,pytest-cov==2.4.0,requests==2.13.0,six==1.10.0,sseclient==0.0.14,teamcity-messages==1.21,toml==0.9.2
...
=================================================================================================== ERRORS ===================================================================================================
__________________________________________________________________________________ ERROR collecting tests/test_marathon.py ___________________________________________________________________________________
ImportError while importing test module '/Users/abramowi/dev/git-repos/dcos-cli/tests/test_marathon.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
.tox/py35-unit/lib/python3.5/site-packages/_pytest/python.py:418: in _importtestmodule
    mod = self.fspath.pyimport(ensuresyspath=importmode)
.tox/py35-unit/lib/python3.5/site-packages/py/_path/local.py:662: in pyimport
    __import__(modname)
.tox/py35-unit/lib/python3.5/site-packages/_pytest/assertion/rewrite.py:216: in load_module
    py.builtin.exec_(co, mod.__dict__)
tests/test_marathon.py:9: in <module>
    from dcos import http, marathon, rpcclient
.tox/py35-unit/lib/python3.5/site-packages/dcos/http.py:7: in <module>
    from dcos.auth import header_challenge_auth
.tox/py35-unit/lib/python3.5/site-packages/dcos/auth.py:7: in <module>
    import jwt
E   ImportError: No module named 'jwt'
_______________________________________________________________________________ ERROR collecting tests/test_packagemanager.py ________________________________________________________________________________
ImportError while importing test module '/Users/abramowi/dev/git-repos/dcos-cli/tests/test_packagemanager.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
.tox/py35-unit/lib/python3.5/site-packages/_pytest/python.py:418: in _importtestmodule
    mod = self.fspath.pyimport(ensuresyspath=importmode)
.tox/py35-unit/lib/python3.5/site-packages/py/_path/local.py:662: in pyimport
    __import__(modname)
.tox/py35-unit/lib/python3.5/site-packages/_pytest/assertion/rewrite.py:216: in load_module
    py.builtin.exec_(co, mod.__dict__)
tests/test_packagemanager.py:5: in <module>
    from dcos import packagemanager
.tox/py35-unit/lib/python3.5/site-packages/dcos/packagemanager.py:8: in <module>
    from dcos import cosmos, emitting, util
.tox/py35-unit/lib/python3.5/site-packages/dcos/cosmos.py:3: in <module>
    from dcos import config, http, util
.tox/py35-unit/lib/python3.5/site-packages/dcos/http.py:7: in <module>
    from dcos.auth import header_challenge_auth
.tox/py35-unit/lib/python3.5/site-packages/dcos/auth.py:7: in <module>
    import jwt
E   ImportError: No module named 'jwt'
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 2 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
========================================================================================== 2 error in 1.01 seconds ===========================================================================================
ERROR: InvocationError: '/Users/abramowi/dev/git-repos/dcos-cli/.tox/py35-unit/bin/py.test -p no:cacheprovider -vv --cov /Users/abramowi/dev/git-repos/dcos-cli/.tox/py35-unit/lib/python3.5/site-packages/dcos tests'
__________________________________________________________________________________________________ summary ___________________________________________________________________________________________________
ERROR:   py35-unit: commands failed

$ grep -c 'jwt' setup.py requirements.txt
setup.py:0
requirements.txt:0

https://github.com/dcos/dcos-cli/blob/master/cli/setup.py#L69

Oh I see.
But:
$ grep -i -c 'jwt' setup.py requirements.txt
setup.py:0
requirements.txt:0

abramowi at abramowi-macOS in ~/dev/git-repos/dcos-cli (059ee29...●)
$ ag jwt dcos/auth.py
7:import jwt
21:       'acsjwt' (DC/OS acs auth), 'oauthjwt' (DC/OS acs oauth), or
33:                       if auth_type.rstrip().lower().startswith("acsjwt") or
34:                       auth_type.rstrip().lower().startswith("oauthjwt")),
42:                   "'{}', DC/OS only supports ['oauthjwt', 'acsjwt']".format(
89:    jwt token encoded with private key).
257:        'token': jwt.encode(
323:            if auth_scheme == "oauthjwt":
325:            # auth_scheme == "acsjwt"

so dcos/cli has the dependency on jwt, but I think dcos should have it, since it's used in dcos/auth.py.
In fact, it looks like it's only used in dcos/auth.py and not used at all in dcos/cli:
$ ag --ignore=tests --ignore=env --ignore=venv jwt --ignore='*.egg-info'
cli/setup.py
69:        'PyJWT==1.4.2',

dcos/auth.py
7:import jwt
21:       'acsjwt' (DC/OS acs auth), 'oauthjwt' (DC/OS acs oauth), or
33:                       if auth_type.rstrip().lower().startswith("acsjwt") or
34:                       auth_type.rstrip().lower().startswith("oauthjwt")),
42:                   "'{}', DC/OS only supports ['oauthjwt', 'acsjwt']".format(
89:    jwt token encoded with private key).
257:        'token': jwt.encode(
323:            if auth_scheme == "oauthjwt":
325:            # auth_scheme == "acsjwt"

so I think the dependency should be moved. See https://github.com/dcos/dcos-cli/pull/937

Another CCM failure.

retest

Hmmm I thought my other change would fix the CI failure. Any other ideas?

For those watching at home who didn't see the commit 649bf1a, it turned out that the test failure was because of a circular import and it's fixed by 649bf1a.
I also just removed two changes that aren't necessary in d58336b and 7b06b03.

CCM cluster error

Ooh CI on this one finally went green! 🎉

🎉


Docker run fails

When I use the docker run command, I get this failure:
docker run -e TIINGO_API_KEY=xxx  -p 8080:8080 --name robinhood omdv/robinhood-portfolio:ubuntu
Status: Downloaded newer image for omdv/robinhood-portfolio:ubuntu
Traceback (most recent call last):
  File "app.py", line 8, in <module>
    from backend.backend import BackendClass
  File "/robinhood-portfolio/backend/backend.py", line 5, in <module>
    from backend.robinhood_data import RobinhoodData
  File "/robinhood-portfolio/backend/robinhood_data.py", line 4, in <module>
    from Robinhood import Robinhood
ImportError: No module named 'Robinhood'

Running on Mac OSX 10.13.6 (High Sierra)
Docker version 19.03.2, build 6a30dfc

I tried this a couple of days ago too, in digital ocean, and had the same problem. At first we couldn't figure out what was going on because it seemed like the image wasn't even running, but it was the script inside the image that was halting somehow with errors, but I don't know enough about python to diagnose it. That all said, however, I would be very very appreciative if anyone figures out how to actually get this going!!!

Hi, thanks for submitting. I am not using it for few years already, but tried to rebuild docker image and found that Robinhood module is replaced with pyrh. Will try to update everything. If module API hasn't changed it should not take long.

<3 27.04.2020, 04:45, "Oleg Medvedev"<EMAIL_ADDRESS>Hi, thanks for submitting. I am not using it for few years already, but tried to rebuild docker image and found that Robinhood module is replaced with pyrh. Will try to update everything. If module API hasn't changed it should not take long.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Is there a reason why the previous working docker image isn't on dockerhub? That way a historical working version would be available for everyone instead of having to build one today that doesn't work anymore.

Hey, quick update. First, I don't know why docker image was changed - most likely I triggered the build during last commit and you were the first who noticed. And I didn't have a proper CI/CD.
Now, as for fixing it. I am not using it myself for 3+ years. Started looking at the code - it is quite messy and uses obsoleted features of pandas. To be honest I have limited interest/time to fix it. Plus I don't think a lot of people use it. Will try to salvage and migrate it to Jupyter notebook inside docker first, which will make it easier to tinker with. Sorry. Will likely take a week or longer.

That's OK. I'm not in a hurry. I've never used it so I only have the screenshot to go by. It looked cool so I thought I'd give it a shot.
I did find a similar project written in Ruby on Rails that works that you might be interested in.
Thank you for making this project and sharing it with the world. Much appreciated!

Well, I am staying tuned, as this package has a very unique functionality of portfolio analysis and visualization, which neither countless others which I have looked at do, nor the nonetheless interesting RoR one which @frakman1 has found above! :(
Please keep us posted, and if you have any more diagnostic updates as far as what is wrong, or what impacts that would have, even though I have very limited python knowledge, it might enable myself or someone entirely different to patch it up! Perhaps even if some features will be missing or broken, if I could even get some of this functionality from something that can run, I would be so grateful!

Hi @omdv , was wondering if there's any update on this, or suggested hacks that one might be able to use to get this running?

Hi @rbdl1. Sure, I made changes couple of weeks ago, but was sitting on them to add some testing. Check out the updated version.
Done:

migrated away from Pandas panels finally and upgraded to latest versions
reworked all calculations to get rid of tables and HDFstore, which will allow to use smaller version of docker images
cleaned up code in few places

What was not done:
I haven't updated the actual flask portion, so currently the only way to use it is via Robinhood jupyter notebook. I'll update instructions and will recompile docker image with jupyter notebook and proper conda environment.
May rework flask part later. Will update docker image and instructions later next week.

@omdv, You are a legend. <3

Hey, was just wondering if there's any updates on this? :)

@rbdl1, I plan to update readme to indicate that it was converted to Jupyter notebook. Other than that - I don't think I will touch the flask part.

@omdv Okay, so what does this mean from an end-user then from an end-user perspective, then? Is this currently usable as is, or not quite yet? Thanks for working on it

Hi, sorry for delay. It was usable since my previous reply. I added instructions. Will add docker way as well, although it may be an overkill if it runs just jupyter.


Feature: Ctrl+Shift+V to paste into the selected folder

Resolved / Related Issues

[X] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes #11776

Validation
How did you test these changes?

[X] Did you build the app and test your changes?
[X] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Sure


Converting an empty list to relational algebra throws

In hybrid mode (where we convert Morel to expressions to relational algebra) a query on an empty list gives an error:
= Sys.set ("hybrid", true);
val it = () : unit
= from e in [] yield {a = 1, b = 2};
bad INTERSECT/UNION/EXCEPT input count

The problem is that we are trying to create a Calcite Union relational operator with zero inputs.

Fixed in 78d3c33.


If these are not commits from upstream kfp, then please ensure
you adhere to the commit checker formatting

commitchecker verson unknown
Validating 0 commits between 49cab500c6a7788e0be83082da235e0d213bc4b4...7e7a993f9d98373a85f2193cde35945c923a739f

A set of new images have been built to help with testing out this PR:
API Server: quay.io/opendatahub/ds-pipelines-api-server:pr-103
DSP DRIVER: quay.io/opendatahub/ds-pipelines-driver:pr-103
DSP LAUNCHER: quay.io/opendatahub/ds-pipelines-launcher:pr-103
Persistence Agent: quay.io/opendatahub/ds-pipelines-persistenceagent:pr-103
Scheduled Workflow Manager: quay.io/opendatahub/ds-pipelines-scheduledworkflow:pr-103
MLMD Server: quay.io/opendatahub/mlmd-grpc-server:latest
MLMD Envoy Proxy: registry.redhat.io/openshift-service-mesh/proxyv2-rhel8:2.3.9-2
UI: quay.io/opendatahub/ds-pipelines-frontend:pr-103

Commit Checker results:
**NOTE**: These are the results of the commit checker scans.
If these are not commits from upstream kfp, then please ensure
you adhere to the commit checker formatting

commitchecker verson unknown
Validating 0 commits between 49cab500c6a7788e0be83082da235e0d213bc4b4...7e7a993f9d98373a85f2193cde35945c923a739f


Implement Python SDK

Description

A Python SDK should be created that mirrors the node/ts SDK.
Task overview:

Only the eval command should be implemented during the first stage.
Only the Configu store should be implemented during the first stage.

Motivation

Expanded functionality for potential users of Configu.

@rannn505 Please fill the issue with more details 🙏

Strategy and Tasks
first stage will be setting up the project skeleton with all relevant utilities and workflows compliant with PEP8
second stage is implementing @config/ts in python
Project Setup Tasks

[ ] Create new branch and merge current draft to it
[ ] Subscribe to https://pypi.org/
[ ] Setup poetry for dependencies and publishing
[ ] Setup testing engine/platform for the project and the types
[ ] Setup lints/lint-staged for python
[ ] Create / Edit README for the project - How to install and run the project
[ ] Update Python SDK row in main project README
[ ] Find most fitting Git Action to build and publish the project

@ShriekinNinja @RichardAkman have you thought about auto-generating the Java and Python SDKs?
buildwithfern.com


[Stdlib] Fix requirements on FlattenCollectionIndex.

FlattenCollectionIndex recently became a typealias, but overstated its
requirements.

@swift-ci please test and merge

Oops. Thanks Doug.


Edit Config in Pinot UI does not work

I cannot create server pool using Edit Config button in Pinot UI.
The rest API results in 400 Bad Request with below exception:
Cannot deserialize value of type `org.apache.pinot.spi.config.instance.InstanceType` from String "server": not one of the values accepted for Enum class: [BROKER, SERVER, CONTROLLER, MINION]
 at [Source: (org.glassfish.jersey.message.internal.ReaderInterceptorExecutor$UnCloseableInputStream); line: 4, column: 11] (through reference chain: org.apache.pinot.spi.config.instance.Instance["type"])
```

Edit Config button

Edit Config showing type as `server`

https://github.com/apache/pinot/blob/436968e080e89cdf4e08ab064e47cc70b1eb6491/pinot-controller/src/main/resources/app/interfaces/types.d.ts#L286 file has:
  export const enum InstanceType {
    BROKER = "broker",
    CONTROLLER = "controller",
    MINION = "minion",
    SERVER = "server"
  }

but, https://github.com/apache/pinot/blob/fb043dab7d5fef3ef726896d2f53e45a2fd69f22/pinot-spi/src/main/java/org/apache/pinot/spi/config/instance/InstanceType.java#L22 has:
public enum InstanceType {
  CONTROLLER, BROKER, SERVER, MINION
}

https://github.com/apache/pinot/pull/12210/files#diff-4e48b36a5089784afbf9656197aa7494cd41e8de31841cbdc7d3db9a20f6edcdR286 PR introduced the change.

https://github.com/apache/pinot/pull/12678 to fix the issue

Adding @jadami10 and @jayeshchoudhary for review

Edit Config showing SEVER instead of server:

Pool creation works using UI:


Leon hotword not working

I am not able to use the Leon hot word at all so I cannot say "Leon" to do something I manually have to press the button

Hello @Architsakri 👋,
Thanks for opening this issue. Could you please try with the develop branch instead and let us know about the result?
Also, please provide more details as per requested in the issue template.

I close this as no answer has been provided. Feel free to reopen.

I also had this problem. Using the button everything is ok. Saying "Leon" nothing happens.
I am on the dev branch.
Hotword console log:

Connected to the server
Waiting for hotword...

Leon server log:

Connected
Type: hotword-node
Socket id: F3-ojBZYyocdvQ6fAAAd

The web browser shows only 1 socket connection (webapp). Should I see another socket connection for hotword?


i was getting this error when executing docker run -p 8080:8080 -p 50000:50000 jenkins PS C:\WINDOWS\system32> docker run -p 8080:8080 -p 50000:50000 jenkins C:\ProgramData\chocolatey\lib\docker\tools\docker.exe: error during connect: Post http://%2F%2F.%2Fpipe%2Fdocker_engine/v1.39/containers/create: open //./pipe/docker_engine: The system cannot find the file specified. In the default daemon configuration on Windows, the docker client must be run elevated to connect. This error may also indicate that the docker daemon is not running. See 'C:\ProgramData\chocolatey\lib\docker\tools\docker.exe run --help'.

Issue with the Docker memory configuration.


Worst visibility, not able to see the description

[Enter feedback here]

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

ID: e6254da8-fbe7-cc2a-2c10-1ba918a61039
Version Independent ID: 7d3100b1-8ab0-f17f-6494-d183bdeb9081
Content: MSBuild Command-Line Reference - MSBuild
Content Source: docs/msbuild/msbuild-command-line-reference.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@vishwa-lumen -- thank you for your feedback.  Please consider these resources:
Please consider these resources:

Ask a question or find an answer on the Microsoft Q&A page for Visual Studio.
Report product issues to the engineering team via the Report a problem tool that appears both in the Visual Studio Installer and in the Visual Studio IDE.
Track product issues and engineering team responses in the Visual Studio Developer Community.
Check the Visual Studio MSDN Forum.
Stack Overflow
Reddit

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Another Language Label Error

Ma"di' (PH 932) incorrectly links to Glottolog Madi (Papua New Guinea) (madi1261).  It actually should link to Glottolog Ma'di (madi 1260).  The reference cited for the Phoible entry is of the African Ma'di language, not the New Guinea language.
https://www.degruyter.com/view/title/16629?language=en

@Taivo55 -- thanks for pointing out this mistake in Glottocode assignment.
I have updated it here:
https://github.com/phoible/dev/pull/258
and when we release the next version of phoible CLDF and phoible online, it will be update there as well.


[Bug]

[REQUIRED] Please fill in the following fields:

Unity editor version: 2019.4.22f
Firebase Unity SDK version: 7.1.0
Source you installed the SDK: Unity Package Manager
Other Firebase Components in use: Message,Crashlytics, Dynamic Links, Remote Config
Additional SDKs you are using: Facebook, AdMob, IronSource, Applovin SDK, Helpshift, OneSignal
Platform you are using the Unity editor on: WEindows
Platform you are targeting: Android\IOS
Scripting Runtime: IL2CPP

[REQUIRED] Please describe the issue here:
When we upgraded to version 7+ we have started get crashes in users. By logs I can assume that problem rise at the push notifications initialize, but not assured
Logs in Crashlytics:
crash_1.txt
crash_2.txt
crash_3.txt

Hi @Vesterix,
I think this issue is going to be a little too broad to tackle with just the crash logs. We'll need to do some work here to isolate the cause. With that, I'd like you to provide the following:

Problematic Firebase product
A minimal, reproducible example of your project
Confirmation that you are getting NDK crash reports

Additionally, please make sure that you're using the latest version of the Unity editor so that your implementation can have the latest updates and bug fixes. The latest version as of writing is Unity 2021.1.0.

@paulinon
Unity discourages us to use latest TECH release (2021.1.0) in production.
All LTS versions are stable releases and get bugfixes and latest updates.


Update performance test results for Ballerina v1.0.4

Purpose
This PR updates the performance test results for Ballerina v1.0.4.
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #20470 into ballerina-1.0.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.0.x   #20470   +/-   ##
================================================
  Coverage            20.51%   20.51%
================================================
  Files                   52       52
  Lines                 1394     1394
  Branches               216      216
================================================
  Hits                   286      286
  Misses                1095     1095
  Partials                13       13

Continue to review full report at Codecov.

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


Update bts_key_info to do private key -> address / pubkey conversions

Should also provide secure input for private keys to keep them from showing up in .bash_history

Superseded by: #1205


Update landscape-standard.adoc

Updated W3C table
Updated MISB table
Added 3GPP table
Added MIV table
Added CIPA table
Added ISO table
Added IEEE table (incomplete)

@3DXScape Please give me write access if you'd like to merge these changes. Thanks.


Update teams.md

Adding descriptions for targeting

Docs Build status updates of commit 0260aa1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/teams.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@ayrtonsoftware forget it, moved to the next review, and I see that you are adding them.
Thank you for your collaboration.

@get-itips

@ayrtonsoftware

@ayrtonsoftware Gentle reminder

Docs Build status updates of commit c32f817:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/teams.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@Kateyanne Please help.


unable to run demo

When I run npm run demo I get the following output:
><EMAIL_ADDRESS>demo C:\digipolis\prototypes\zoid-demo
> echo 'Navigate to http://localhost:1337/demo/iframe/index.htm or http://localhost:1337/demo/popup/index.htm' && serve . --port 1337 --silent

'Navigate to http://localhost:1337/demo/iframe/index.htm or http://localhost:1337/demo/popup/index.htm'
'serve' is not recognized as an internal or external command,
operable program or batch file.

Because of this I'm unable to run the demo.

you should install serve package like this:
npm install -D serve
and update the demo script in package.json to ..... && serve . -l 1337


SMB security settings configuration from Azure Portal

SMB security settings section mentions that security configurations can't be modified from the portal. However, it is already possible. Please consider to include Portal steps as well

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

ID: 459ae273-9261-cb21-318b-f20071b7fb6a
Version Independent ID: d3d3bb89-b29e-4b73-1436-8b88f91d8886
Content: SMB file shares in Azure Files
Content Source: articles/storage/files/files-smb-protocol.md
Service: storage
Sub-service: files
GitHub Login: @khdownie
Microsoft Alias: kendownie

@aamorale thanks for catching this. @SaiKishor-MSFT - I will update with portal instructions and then update/close this issue.

We are waiting for a bug to be fixed in how the portal displays security settings. When it is fixed I will add the portal instrux.

@aamorale and @SaiKishor-MSFT the dev team fixed the portal bug and I have added the portal instructions to the page:
https://docs.microsoft.com/en-us/azure/storage/files/files-smb-protocol?tabs=azure-portal#smb-security-settings
This issue can now be closed.

#please-close


Research guide v1

Good work Zhen!

Thanks so much zhen. this looks great.

👌

Just added links to the wiki courses pages for CS1989. gonna merge now


Markdown/bbcode or other markup language support

What are you requesting?
It would be great if the pyfancy parsing also had basic markdown/bbcode or other markup language support.
Related #36
How could this be implemented.
pyfancy("#Bold").output()

pyfancy().parse("[color=red]RED![/color]")

@jonathan50  @TheMonsterFromTheDeep @baranskistad @joker314

How would we do different size text? (i.e. #, ##, ###, etc.)

Good question... I will try to do research... but that might not be possible

It is mostly OS specific... So that might not be possible

So maybe only support bold, underline, italics, code, etc?


Code Editor - Use ESM Version

Use ESM Version
Create builder for Angular CLI (like ng-packager)
Refactor component code

First pass in https://github.com/Teradata/covalent/tree/feat/monaco-esm


Client.Source text is probably wonky

SourceURL seems to work well but I imagine Source needs some special escaping to not break the Prolog format client.

fixed in v0.2.0


add node 18 to matrix

Add node v18 to matrix & remove deprecated v12.

 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.

@jaffrepaul

Thanks for addressing the issue https://github.com/cypress-io/github-action/issues/623 I reported!

@jaffrepaul Are these tests actually running on Node 18? I see the 14, 16, and 18 tests showing v16 in the Cypress test output as the Node version for testing.

Perhaps this would be a good time to update the following?
uses: actions/setup-node@v2 to v3
uses: actions/checkout@v2 to v3
The older versions produce deprecation warnings regarding Node.js 12.

Perhaps this would be a good time to update the following?
uses: actions/setup-node@v2 to v3 uses: actions/checkout@v2 to v3
The older versions produce deprecation warnings regarding Node.js 12.

@MikeMcC399. I'm hoping to update a number of outdated deps throughout this action soon, including these bumps. That will be treated as a separate effort but I don't mind including these here since it's very related 👍

@jennifer-shehane I believe that output is just noting the node version of the GH hosted runner (2.299.1) that runs the workflows. The tooling there can change over time. We do specify a v16 runtime in the meta data of our action as required. This is also subject to change once 16 reaches EOL.
As you mentioned the workflow summary shows the node version in the setup & echo in each step.  Tho it is curious that the output follows those.

@jaffrepaul
There is a tag
https://github.com/cypress-io/github-action/releases/tag/v5.0.0
but no tag
https://github.com/cypress-io/github-action/releases/tag/v5
so specifying cypress-io/github-action@v5 doesn't work.
"Unable to resolve action cypress-io/github-action@v5, unable to find version v5"
Is it intentional not to have released a generic v5 version at the moment?

@jaffrepaul

The v5 tag issue is settled. You can now pull in github-action@v5.

Thank you very much. I confirm that this now works!

Digging into the caching issue.

That's good! I set up two tests, one with and one without using the action on https://github.com/MikeMcC399/cypress-test-tiny. They use latest versions of everything Ubuntu 22.04, Node,js 18, Cypress 11.2.0 and github-action@v5
The two logs are:

https://github.com/MikeMcC399/cypress-test-tiny/actions/runs/3604760548 (showing the issue)
https://github.com/MikeMcC399/cypress-test-tiny/actions/runs/3604752964 (for comparison, running Cypress directly)

Let me know if you prefer to have a separate issue to track this.

@MikeMcC399 separate issue please 👍

@jaffrepaul

The Node,js version selection problem is described in the new issue https://github.com/cypress-io/github-action/issues/642

I hope this is helpful!

I don't think this is a cache issue. See https://github.com/cypress-io/github-action/issues/642#issuecomment-1336202107.

@jennifer-shehane

Are these tests actually running on Node 18? I see the 14, 16, and 18 tests showing v16 in the Cypress test output as the Node version for testing.

This is a old known issue. See https://github.com/cypress-io/github-action/issues/489


Ensure that saveDocument works if there's no /ID-entry in the PDF document (issue 13279)

First of all, while it should be very unlikely that the /ID-entry is an indirect object, note how we're using Dict.get when parsing it e.g. in PDFDocument.fingerprint. Hence we definitely should be consistent here, since if the /ID-entry is an indirect object the existing code in src/core/writer.js would already fail.
Secondly, to fix the referenced issue, we also need to check that the /ID-entry actually is an Array before attempting to access its contents in src/core/writer.js.
Drive-by change: In the xrefInfo object passed to the incrementalUpdate function, re-name the encrypt property to encryptRef since its data is fetched using Dict.getRaw (given the names of the other properties fetched similarly).
Fixes #13279

/botio unittest

/botio integrationtest

While PDF document without /ID is quite rare, as the issue shows they unfortunately do exist.
Given that the patch is small, and in my opinion quite safe, this might be the sort of thing that we should consider uplifting (once it's landed of course)?


Solve error when project_folder has spaces

If project_folder has spaces then Shell Exec will fail when trying to
change the current directory to the project folder. For example:
project folder = /Users/myuser/Google Drive/myproject
Will fail because Shell Exec will try to execute:
cd /Users/myuser/Google Drive/myproject && command
And the folder "/Users/myuser/Google" won't be found.
Solve this error by wrapping project_folder between quotes

Good! tks.


Make ZOPFLI_MASTER_BLOCK_SIZE an option

I have a program that creates large PNGs (up to 48 MiB of raw pixel data, but usually 12 or 16 MiB) and uses Zopfli as the deflater for PNG encoding. At these sizes, compression ratios are usually between 99.4% and 99.9%. I suspect the compression would go much faster if I could specify a larger ZOPFLI_MASTER_BLOCK_SIZE, e.g. 4 MiB, so that compression operated on fewer blocks. I'm currently using only about 20GiB RAM on a c7g.16xlarge (which has 124GiB).

Indeed a bigger master block size could help in this case. Have you tried using the DeflateEncoder wrapped on a BufWriter of a custom, large capacity? Currently, the compress function is just a more ergonomic frontend for that.

Thanks; I'll suggest that to the maintainer of oxipng.

I'm trying to use a BufWriter in a fork of Oxipng, but it's failing an end-to-end test. Any clue what I might be doing wrong?
impl Deflater for BufferedZopfliDeflater {
    fn deflate(&self, data: &[u8], max_size: &AtomicMin) -> PngResult<Vec<u8>> {
        #[allow(clippy::needless_update)]
        let options = Options {
            iteration_count: self.iterations,
            maximum_block_splits: self.max_block_splits,
            ..Default::default() // for forward compatibility
        };
        let mut out = Cursor::new(Vec::with_capacity(self.output_buffer_size));
        let mut buffer = BufWriter::with_capacity(
            self.input_buffer_size,
            DeflateEncoder::new(
                options,
                Default::default(),
                &mut out,
            ),
        );
        let result = (|| -> io::Result<()> {
            buffer.write_all(data)?;
            buffer.into_inner()?.finish()?;
            Ok(())
        })();
        result.map_err(|e| PngError::new(&e.to_string()))?;
        let out = out.into_inner();
        println!("Compressed {} -> {} bytes", data.len(), out.len());
        if max_size.get().is_some_and(|max| max < out.len()) {
            Err(PngError::DeflatedDataTooLong(out.len()))
        } else {
            Ok(out)
        }
    }
}

Update: found the issue - Oxipng is using zlib_compress rather than compress.

How would I wrap zlib_compress this way? That, not compress, is what Oxipng is using.

Even copying the zlib_compress code into my code doesn't work, because HashingAndCountingRead is in a private crate.

Is it possible zlib_compress could support this as well? That, not compress, is what Oxipng is using.

It'd be necessary to redesign its API, which is a breaking change, but other than that yes, it's possible and doable.

The API has been redesigned in v0.8.0 to provide GzipEncoder and ZlibEncoder structs, analogous to DeflateEncoder. Using these structs, users can tune block sizes and other advanced compression options for each of the supported compression formats. Therefore, I consider this issue resolved :tada:


Legacy of Dreams is broken

https://ptr.d3planner.com/321783042
The damage buff and damage reduction for the Legacy of Dreams legendary gem aren't factoring in every slot, and the slots that are counted change depending on which jewelry slot the Legacy of Dreams occupies.
If it's in the amulet, only the head, shoulders, and neck slots are counted. If it's in the left ring, every slot except the right ring, main hand, and off hand are counted. If it's in the right ring, every slot except the main hand and off hand are counted.

Oh that's unfortunate, guess I can't do legendary bonuses like that because they're evaluated before it's finished checking all items. Will have to think of a better way to do it.

This should work now.

I'm getting this when I socket a legendary gem in any jewelry slot:
ReferenceError: getGemValue is not defined 3 core.js:4:8480
    applyGems https://www.d3planner.com/core.js:4
    each https://www.d3planner.com/script.js:1
    applyGems https://www.d3planner.com/core.js:4
    computeStats https://www.d3planner.com/core.js:4
    getStats https://www.d3planner.com/core.js:4
    t https://www.d3planner.com/core.js:14
    a https://www.d3planner.com/scripts/data.js:1


[Bug] Tailscale doesn't connect after phone reboot with Tailscale connection on

Is this a support request?

[X] This is not a support request

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
After rebooting phone (without disabling connection) or force closing the android app it doesn't connect again (stuck at starting...)
In headscale logs after successful authorisation I see 2024-05-08T08:07:39+02:00 INF home/runner/work/headscale/headscale/hscontrol/poll.go:657 > node has an open stream(0x4000a8c480), rejecting new stream node=localhost node.id=6 omitPeers=false readOnly=false stream=true
After force closing the app and restarting headscale I'm able to log back in.
Expected Behavior
The app should connect again.
Steps To Reproduce

Connect to tailnet
Reboot phone without disabling tailscale connection

Open app after restart and try to connect again.

Environment
- OS: android 14
- Headscale version: v0.23.0-alpha11
- Tailscale version: 1.65.182-t80df8ffb8-g6a15347453c (new beta app)

Runtime environment

[X] Headscale is behind a (reverse) proxy
[X] Headscale runs in a container

Anything else?
No response

There is a change in Tailscale 1.66.3 that may solve your issue.
https://tailscale.com/changelog#client

"Improved the VPN service connection logic, especially when rebooting the device with Always-On VPN enabled."


NASA Picture of the Day API Integration

Feature Request for NASA Picture of the Day API Integration
Problem:
I'm always looking for interesting and educational content to share with my Discord server members. I think space and astronomy are fascinating topics that many people enjoy.
Solution:
I'd like to integrate the NASA Picture of the Day API into my Discord bot. This API provides a new image or video of our universe each day, along with a brief explanation written by a professional astronomer. The bot could automatically fetch the picture of the day and post it in a specific channel, or it could provide it in response to a user command.
Alternatives:
I've considered using other astronomy APIs, but the NASA Picture of the Day API is widely recognized and provides high-quality content. Another alternative would be to manually search for interesting space images and post them, but this would be more time-consuming and less consistent.
Additional Context:
I think this feature would be a great addition to the bot because it provides a daily dose of interesting and educational content. It can spark conversations about space and science, and it's a great way to engage with the community.

APOD Doku
Laut Doku ist es möglich, die APOD von bis zu 1995 zu bekommen.

NON-COMMERCIAL USE
For educational or informational purposes
NASA content – images, audio, video, related media and files used in the rendition of 3-dimensional models, such as texture maps and polygon data in any format – generally are not subject to copyright in the United States. You may use this material for educational or informational purposes, including photo collections, textbooks, public exhibits, computer graphical simulations and Internet Web pages. This general permission extends to personal Web pages.

News outlets, schools, and text-book authors
News outlets, schools, and text-book authors may use NASA content without needing explicit permission, subject to compliance with these guidelines. NASA content used in a factual manner that does not imply endorsement may be used without needing explicit permission. NASA should be acknowledged as the source of the material. NASA occasionally uses copyright-protected material of third parties with permission on its website. Those images will be marked identified as copyright protected with the name of the copyright holder. NASA’s use does not convey any rights to others to use the same material. Those wishing to use copyright protected material of third parties must contact the copyright holder directly.

I think its okey since we are Non-Commercial ?
But we can still add the Copyright parameter to the api, to respect the Photograpers

I was trying to add a filter depending on the lincese, but it turns out, the copyright parameter for the api only returns the names.
Tom Abel & Ralf Kaehler (KIPAC, SLAC), AMNH
A Twitter bot outputs the name and picture, if that is not enought, thenn i will delete the APOD complety from the Bot.
https://twitter.com/bot_apod/status/1356512834117459970

in the def get_apod will get the response as json.
In that response you can get data of the response.

# Gets the Picture of NASA A Picture Of the Day
def get_apod():
    api_key = os.getenv('NASA_API_KEY')
    url = f"https://api.nasa.gov/planetary/apod?api_key={api_key}"
    response = requests.get(url)

    # Check if request was successfull
    if response.status_code == 200:
        data = response.json()

        # Extract the copyright information
        copyright = data.get('copyright', None)    <=========GETS COPYRIGHT FIELD
        print(copyright)

    else:
        print("Error when Get APOD Picture.")
        return None

When tested on 02.11.2023 the console output the name: Marcelo Rivera
In the offical Api documentation  , you can get a list with the returned fields.

I will try to move the apod to a own class (with python OOP that you suggestet once) and "turn off the feature" until we find a way to ignore copyrighted Pictures

i also would like to move this issue to disscusion.
If yes then react with thumps Up


feat(data-table): ng7 data-table component

Description

Data Table, Sorting, Column Resizing
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots:
Before (If applicable):

After:

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

Checklist:

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

question about this .. can I put a custom component in a cell?  Not sure from all I read

question about this .. can I put a custom component in a cell? Not sure from all I read

@mathsusan You can, one of the examples I added has our avatar component in a custom cell


[e2e]: container-based push main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8549095002
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container-based.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container-based.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Thu Apr  4 04:06:13 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8578583733
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container-based.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container-based.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Sat Apr  6 04:38:43 UTC 2024
Tests are passing now. Closing this issue.


Upgrade The Unarchiver to v3.10

Thank you for the contribution. It was merged directly as 266588cd7e6b2eb0992fa51f21e1712309c630b9 to keep commit history cleaner. Your contribution is still credited to you.


Problem: maintaining single .sql migration files

They make it difficult to compose final shippable SQL scripts for every particular version. They also grow out of proportion.
Solution: write them incrementally and compose on psql/test/packaging

For the record, @burntcarrot also suggested this on Discord:

I was thinking in a similar way, but I had an idea of introducing "block"-based extension creation. Each block will have the procedures related to it with incremental migration. Each extension has a tiny config; I don't have a concrete structure for the config, but it at least will specify what blocks are a part of this extension. And there would be an optional config in the project's root directory, which specifies the available extensions and their path.
A CLI or a script can check these directories (good time to build a omni CLI!), package the migrations to a single SQL file. The artifacts are then either stored in the extensions' directories or it could be stored in a different path.
One advantage of using this approach is that when we develop blocks/extensions which are experimental, we can modify the config to ignore the experimental blocks/extensions when we build for a push on the main branch.

And there was a note that there isn't necessarily a conflict between the ideas (one can grow into another one)


Only include sys/sysctl.h on for Apple

glibc deprecated that header in version 2.30, and removed it in 2.32.

@pierremoreau This is currently marked as a draft. Are you expecting a review? Do you anticipate further changes?

I wanted to take a deeper look at those headers and check that those are even needed for Apple (test_common/harness/testHarness.cpp definitely needs it, but it didn't look like the other did) and if other sys/*.h headers could be removed as well as no longer needed, but ran out of time.
Maybe this can be reviewed as-is as it is enough to make it compile against recent glibc, and open an issue to keep track of the other bits?

Sounds good, thanks! I'll open another PR with the remaining changes.


typo: "resect" instead of "reset"

The following is written twice:

[...] self service password resect (SSPR) experience [...]

Given the contest, the word "resect" is likely a misspelling of "reset".

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

ID: ba12051f-3f69-3dae-1779-d8e54f0fc598
Version Independent ID: 489c7916-0ca7-7d7a-09f0-d9bc05274551
Content: Troubleshoot custom policies and user flows in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/troubleshoot.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

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


Baal: Split Shares into new contract

[x] Make contract updates
[x] Can we get a peer review to help solidify the choice?

https://github.com/HausDAO/Baal/tree/feature/sharesToken

Moloch Mystics are working on a few code changes. Hoping to get it good enough to lock.

Relates to: https://github.com/HausDAO/daohaus-monorepo/issues/44

Creating new issues for code updates and deployments

https://baal-docs.vercel.app/baal-cli

Going to close this issue.
We will track the updates Bill is making here:
https://github.com/HausDAO/daohaus-monorepo/issues/56


Access PIM activity and audit history via API,PWSH?

In the article you describe how to access activity and audit history for Azure resource roles in Privileged Identity Management via the Azure AD Admin Center.
However we would like automate generating activity and audit history reports. Could you add information how to access the information via API or Powershell?
I could not find this documented anywhere else.
Thanks and best regards,
Joerg

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

ID: b9dcb3c8-7a2a-098a-41bb-937483596a92
Version Independent ID: 577311fd-9d22-0270-d39e-0047ca63fb11
Content: View audit report for Azure resource roles in Privileged Identity Management (PIM) - Azure AD - Microsoft Entra
Content Source: articles/active-directory/privileged-identity-management/azure-pim-resource-rbac.md
Service: active-directory
Sub-service: pim
GitHub Login: @curtand
Microsoft Alias: curtand

@joergrenn Thank you for your feedback, let me investigate and revert back.


Feat: Better error messaging when a component is not found by a flavored component exists

Is your feature request related to a problem? Please describe.
As Ezra when I try to create a package that is importing components, but I don't set the flavor or the architecture required by the component I find the error message zarf gives confusing.
Failed to create package: component "istio-controlplane" not found in "../../src/istio"
You can see an example of this by running zarf package create with no parameters on this package https://github.com/defenseunicorns/uds-core/tree/main/packages/standard
Describe the solution you'd like

Given I have a zarf package importing flavored components
When I run zarf package create with no parameters
Then I get an error message signalling that it's possible a component exists it, but it is not compatible.

Found the relevant code here: https://github.com/defenseunicorns/zarf/blob/1ec88490121cf8ec1ae0ce402691fc3c8f663681/src/pkg/packager/composer/list.go#L205
Are you suggesting changing the error message itself or adding new code to track if an incompatible component might be missing?

Came up with this message:
componentNotFound := "Component %q not found in %q satisfying %q. It's possible a component exists but it is not compatible with the current architecture or flavor."
If this sounds good, I'll create a PR for it.

Sorry @chrishorton on second thought the brevity of the first message is probably better, going to close this.


Macro to destructure term App as if it were a list

In preparation for actual lists

A function application is modelled as a function with zero or  more arguments. This is currently represented as a function with a Cons list. This would be nicer to work with if it was actually a list. This would also make it easy to support Lisp syntax.
(f( a b c ))

vs

(f a b c)

Rethinking this, I still prefer the AST to be Cons based, but it is also nice to know the syntax form for use in preprocessor.

Maybe there is a way to add a marker to App to tell if this expression is a Cons root


Show a list of other user responses after submission

After a user submits a prompt to be saved show a list of submissions from other users

Added in v1.4.0


cusparse_build_version outputs version greater than latest cusparse version

Description
Hi There,
I'm trying to run JAX on a fresh VM on cloud infrastructure. The error is on the first line when printing the environment info
CUDA backend failed to initialize: Found cuSPARSE version 12103, but JAX was built against version 12200, which is newer. The copy of cuSPARSE that is installed must be at least as new as the version against which JAX was built. (Set TF_CPP_MIN_LOG_LEVEL=0 and rerun for more info.)

This seems like a bug in the code that checks the cuSPARSE version because the latest version of cuSPARSE is <IP_ADDRESS>.
As seen on my system, I have this version:
$ python3 -m pip list | grep cusparse
nvidia-cusparse-cu12     <IP_ADDRESS>

And using the jax function after opening a python3 shell, you can see it finds this version, but cuts off the middle section.
>>> from jax._src.lib import cuda_versions
>>> cuda_versions.cusparse_get_version()
12103

Should this instead read 1220103?
The build version expected by JAX is somehow larger than this.
>>> from jax._src.lib import cuda_versions
>>> cuda_versions.cusparse_build_version()
12200

I suspect this is supposed to read 1220000 instead? Either way, this version as currently depicted doesn't seem to exist and is definitely larger than the most recent cuSPARSE version available with pip, so this seems like a bug in the cuda_versions.cusparse_build_version() function.
If there is anything else I can provide to help track this down, let me know. I'm trying to help some researchers get JAX running on GPUs on Jetstream2, and I cannot get past this roadblock.
System info (python version, jaxlib version, accelerator, etc.)
>>> import jax; jax.print_environment_info()
CUDA backend failed to initialize: Found cuSPARSE version 12103, but JAX was built against version 12200, which is newer. The copy of cuSPARSE that is installed must be at least as new as the version against which JAX was built. (Set TF_CPP_MIN_LOG_LEVEL=0 and rerun for more info.)
jax:    0.4.25
jaxlib: 0.4.25
numpy:  1.26.4
python: 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0]
jax.devices (1 total, 1 local): [CpuDevice(id=0)]
process_count: 1
platform: uname_result(system='Linux', node='eagerly-healthy-wallaby', release='5.15.0-97-generic', version='#107-Ubuntu SMP Wed Feb 7 13:26:48 UTC 2024', machine='x86_64')

$ nvidia-smi
Tue Mar  5 18:40:36 2024
+---------------------------------------------------------------------------------------+
| NVIDIA-SMI 535.129.03             Driver Version: 535.129.03   CUDA Version: 12.2     |
|-----------------------------------------+----------------------+----------------------+
| GPU  Name                 Persistence-M | Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |         Memory-Usage | GPU-Util  Compute M. |
|                                         |                      |               MIG M. |
|=========================================+======================+======================|
|   0  GRID A100X-40C                 On  | 00000000:04:00.0 Off |                    0 |
| N/A   N/A    P0              N/A /  N/A |      0MiB / 40960MiB |      0%      Default |
|                                         |                      |             Disabled |
+-----------------------------------------+----------------------+----------------------+

+---------------------------------------------------------------------------------------+
| Processes:                                                                            |
|  GPU   GI   CI        PID   Type   Process name                            GPU Memory |
|        ID   ID                                                             Usage      |
|=======================================================================================|
|  No running processes found                                                           |
+---------------------------------------------------------------------------------------+

JAX's cusparse_version just returns CUSPARSE_VERSION as defined in the cusparse C headers. The definition looks something like this depending on which CUDA version you're using:
#define CUSPARSE_VER_MAJOR 12
#define CUSPARSE_VER_MINOR 1
#define CUSPARSE_VER_PATCH 3
#define CUSPARSE_VER_BUILD 153
#define CUSPARSE_VERSION (CUSPARSE_VER_MAJOR * 1000 + \
                          CUSPARSE_VER_MINOR *  100 + \
                          CUSPARSE_VER_PATCH)

This would become 12103. 12200 would come from version 12.2.0. If JAX is reporting a different CUDA version than in header files that you see on your system, then it is likely accessing a different CUDA installation than you are expecting. For example, if you install pytorch via pip, it will bundle its own CUDA headers that may be higher up in your LD_LIBRARY_PATH than the other headers you're looking at.
Does that help answer your question?

Ah, I see what you mean. I was confused because the 103 of 12103 happened to match the 103 of <IP_ADDRESS>. It turns out these are not correlated. So even though the latest version of cusparse I have through pip is <IP_ADDRESS>1, jax must be pulling the cusparse version from elsewhere. Likely I have cusparse <IP_ADDRESS> installed somewhere, though where I'm not sure yet. I'll go digging for that and see if I can remove it from the path.
Confirmed. Cusparse was installed elsewhere. I removed it from LD_LIBRARY_PATH and all is good now.


Unable to convert example of useSwr hook to use react-query's useMutation hook?

I tried changing the useSwr hook to use:
import { useMutation } from 'react-query';

const { data: user, mutate: mutateUser } = useMutation<User>('/api/user');

But I get an error stating:

No mutationFn found

Unable to solve this. I tried adding mutationFn inside useMutation like but it is static:
const { data: user, mutate: mutateUser } = useMutation<User>('/api/user', {
    mutationFn: () => {
      const u: User = {
        isLoggedIn: false,
        avatarUrl: '',
        login: '',
      }
      return Promise.resolve(u)
    },
  })

If I use a dynamic request using something like axios then do I need to put the fetchJson from login.tsx to mutationFn hook?
Basically, I want to use react-query instead of swr & unable to find a solution?
I'm pretty sure it's really simple but haven't been able to make it work.
Stackblitz Demo → https://stackblitz.com/edit/next-iron-session-with-react-query?file=lib%2FuseUser.ts (go to /login route, enter a github username like vvo & click Login to see the error in console)
Repo → https://github.com/deadcoder0904/next-iron-session-with-react-query

Found the answer → https://stackoverflow.com/a/72507830/6141587


(maint) Eliminate redundant sort_by

When sorting an array, just call sort instead of sort_by with a
block returning each element. Also enable the rubocop check for this.
See https://github.com/bbatsov/rubocop/blob/master/manual/cops_performance.md#performanceredundantsortby
Follow up from PR #6372

CLA signed by all contributors.


Add ublox evk nina b1 board

I have only done some manual tests to verify that serial, MSD and debugging works since we don't have the Keil uvision compiler. The binaries have been built with ARMCC(make_armcc project) instead but the automatic tests seems quite tricky to run without uvision.

/morph test


Re-enable godot-cpp CI.

Whit https://github.com/godotengine/godot/pull/59207 and https://github.com/godotengine/godot-cpp/pull/726 merged, it should work again.

Thanks!


Error message for obsoleted protocol composition syntax should say "types" instead of "protocols"

The following syntax error is emitted when the parser spots obsoleted protocol composition syntax:
typealias C = protocol<P, Q> // error: 'protocol<...>' composition syntax has been removed; join the protocols using '&'

The problem with this message is that "protocols" is not an accurate or prudent description of the components at this compilation stage:

Protocol compositions are not limited to protocols.
A component can be written in terms of a member type, such as a nested typealias.
The parser has no knowledge of whether the components are legal, namely, whether they really reference protocols or classes.

A more neutral "types" is preferable.

Hmm, I'm not sure that "types" is the right word to use here. The things that are joined together with & in a composition are constraints. Compositions can be used for existential types, or for generic requirements on type parameters. We've been trying to guide people away from thinking of protocols and other constraints as types.

Good point, wording them as constraints (or type constraints per the language guide) sounds better.

So we change the word "types" to "type constraints"?
Also is it the right way to do it, as I've done ? (because it is failing the smoke test)
@AnthonyLatsis @hborla

So we change the word "types" to "type constraints"?

Sounds good to me. It’s totally fine to do this in the same pull request btw, no need to close it.

Also is it the right way to do it, as I've done ? (because it is failing the smoke test)

Yeah, you simply forgot to update a test somewhere — check out the logs.

There are a ton of places with this error:
include/swift/AST/DiagnosticsParse.def:      "'protocol<...>' composition syntax has been removed; join the protocols using '&'", ())
test/Parse/swift3_warnings_swift4_errors_version_4.swift:let z: protocol<P1, P2> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}}
test/decl/inherit/inherit.swift:struct S7 : protocol<P, Q> { }  // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}}
test/type/protocol_composition.swift:  var x2 : protocol<SuperREPLPrintable, FooProtocol> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{12-53=SuperREPLPrintable & FooProtocol}}
test/type/protocol_composition.swift:  var _ : () -> protocol<FooProtocol, REPLPrintable> = { return_superPrintable() } // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{17-53=FooProtocol & REPLPrintable}}
test/type/protocol_composition.swift:var y : protocol<P5, P7>= 17 // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{9-26=P5 & P7=}} expected-error {{'=' must have consistent whitespace on both sides}}
test/type/protocol_composition.swift:var z : protocol<P5, P7>?=17 // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{9-27=(P5 & P7)?=}}
test/type/protocol_composition.swift:typealias B1 = protocol<P1,P2> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{16-31=P1 & P2}}
test/type/protocol_composition.swift:typealias B2 = protocol<P1, P2> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{16-32=P1 & P2}}
test/type/protocol_composition.swift:typealias B3 = protocol<P1 ,P2> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{16-32=P1 & P2}}
test/type/protocol_composition.swift:typealias B4 = protocol<P1 , P2> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{16-33=P1 & P2}}
test/type/protocol_composition.swift:typealias C1 = protocol<Any, P1> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{16-33=Any & P1}}
test/type/protocol_composition.swift:typealias C2 = protocol<P1, Any> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{16-33=P1 & Any}}
test/type/protocol_composition.swift:typealias F = protocol<Any, Any> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{15-33=Any & Any}}
test/type/protocol_composition.swift:typealias J = protocol<P1, P2>.Protocol // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{15-31=(P1 & P2)}}
test/type/protocol_composition.swift:typealias K = protocol<P1, P2>? // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{15-32=(P1 & P2)?}}
test/type/protocol_composition.swift:typealias L = protocol<(P1), P2> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{15-33=(P1) & P2}}
test/type/protocol_composition.swift:  // expected-error@-1 {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{26-43=P1 & P2>}}
test/type/protocol_composition.swift:  // expected-error@-1 {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{26-45=P1 & (P2)>}}
test/type/protocol_composition.swift:struct S_45712<T: protocol<P1, P3>> {} // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{19-36=P1 & P3>}}
test/type/protocol_composition.swift:func f1_45712<U where U: protocol<P1, P3>>(_: U)  {} // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}} {{26-43=P1 & P3>}} // expected-error {{'where' clause}}
test/type/protocol_types.swift:struct Struct5<T : protocol<Pub, Bar>> { } // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}}
test/type/protocol_types.swift:struct Struct6<T> where T : protocol<Pub, Bar> {} // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}}
test/type/protocol_types.swift:typealias T2 = protocol<Pub , Bar> // expected-error {{'protocol<...>' composition syntax has been removed; join the protocols using '&'}}

Have done the necessary changes.

Have you checked in with @IceCurrent? They might still be interested in wrapping this up.

I didn't check with him, but looks like he closed his pull request around 2 months back and didn't revert, so I just did the changes.

Also, I actually by mistake committed and pushed changes of this issue in the same branch of the previous pull request, and since the branch is same, it's reflected in the previous issue too...
What's the best I can do to resolve this...

Soft-reset those changes, stash them, then unstash them in a new branch, then force-push the other branch.

Thanks!

@AnthonyLatsis I have committed the changes and also did the testing, can I open a pull request?

I cannot forbid you to, but I will suggest to at least try and reach out to the person that last worked in this. We do not want fixing issues to be a race between contributors (unless it is a race that has been agreed on 😅).

Lol😂, ok sure I will ping him, but I doubt he is gonna reply!

Also, I see the commits of my ongoing pull request in the new pull request, is that normal to happen, because when I researched, it seemed that it happens a lot and it's a Git issue?

That means that your new branch branches off of the branch associated with the ongoing PR. You should set it to track main instead and rebase.

@AnthonyLatsis I have created a pull request, please review it.

@IceCurrent Are you interested in wrapping this up?


[Release] Java binaries are missing from artifactory

Describe the bug, including details regarding any error messages, version, and platform.
When uploading binaries to artifactory for release 11.0.0 I've realised we don't upload Java binaries to artifactory on releases:

Should Java binaries be uploaded to artifactory? From my understanding all released packages should go to Artifactory + any other distribution channel.
Component(s)
Release

@kou shouldn't those artifacts be there?

We don't use Artifactory for .jars.
See "Publish Maven artifacts" in https://arrow.apache.org/docs/dev/developers/release.html#post-release-tasks for .jars.

ok, I thought we also should upload them to artifactory as we do with Python but I see the difference being jars are still uploaded them to apache.org when Python wheels would only be uploaded to PyPI if we would not upload them to artifactory.
I am closing this then. Thanks for clarifying.

Yes. https://repository.apache.org/ is one of the official distribution channels: https://infra.apache.org/release-publishing.html#distribution
And  it supports distributing signature and checksums that are required for each artifact: https://infra.apache.org/release-distribution.html#sigs-and-sums
.e.g.:  https://repository.apache.org/content/repositories/staging/org/apache/arrow/arrow-memory/11.0.0/
So we can use https://repository.apache.org/ for .jars.


model local function example model compilation failure

@MikeHolman constructed a model maf_gpt2_tiny_display_name.onnx with model local functions which fails to compile because of type and shape inference problems.
I put a copy of the model on a git branch:
https://github.com/sorenlassen/onnx-mlir-einsum/tree/sorenlassen/maf_gpt2_tiny_display_name.onnx
You can download the model with the "view raw" link here  (save it to file).
It fails type inference in the builder:
$ onnx-mlir maf_gpt2_tiny_display_name.onnx
Assertion failed: (isa<To>(Val) && "cast<Ty>() argument of incompatible type!"), function cast, file Casting.h, line 566.

With -useOnnxModelTypes it can make it through the frontend
$ onnx-mlir maf_gpt2_tiny_display_name.onnx -useOnnxModelTypes -EmitONNXIR

but fails lowering to Krnl because shape inference leaves some types unranked

@sorenlassen can this issue be closed?


Failed to run a newly created dapp

Hello, newly created dapp cannot run with several erros:
WARNING in ../anchor/src/lottery-app-exports.ts 9:0-25
export 'IDL' (reexported as 'LotteryAppIDL') was not found in '../target/types/lottery_app' (module has no exports)

WARNING in ../anchor/src/lottery-app-exports.ts 12:52-73
export 'IDL' (imported as 'LotteryAppIDL') was not found in '../target/types/lottery_app' (module has no exports)

WARNING in ../anchor/src/lottery-app-exports.ts 16:21-34
export 'IDL' (imported as 'LotteryAppIDL') was not found in '../target/types/lottery_app' (module has no exports)

WARNING in ./src/app/lottery-app/lottery-app-data-access.tsx 26:51-60
export 'programId' (imported as 'programId') was not found in '@lottery-app/anchor' (possible exports: LOTTERY_APP_PROGRAM_ID, LotteryAppIDL, getLotteryAppProgram)

WARNING in ./src/app/lottery-app/lottery-app-data-access.tsx 40:4-13
export 'programId' (imported as 'programId') was not found in '@lottery-app/anchor' (possible exports: LOTTERY_APP_PROGRAM_ID, LotteryAppIDL, getLotteryAppProgram)

WARNING in ../node_modules/asn1.js/lib/asn1/api.js 21:12-42
Module not found: Error: Can't resolve 'vm' in '/Users/pplanel/src/pplanel/solana/lottery-app/node_modules/asn1.js/lib/asn1'
ERROR in ./anchor/src/lottery-app-exports.ts:5:10
TS2305: Module '"../target/types/lottery_app"' has no exported member 'IDL'.
    3 | import { PublicKey } from<EMAIL_ADDRESS>    4 | import type { LotteryApp } from '../target/types/lottery_app';
  > 5 | import { IDL as LotteryAppIDL } from '../target/types/lottery_app';
      |          ^^^
    6 |
    7 | // Re-export the generated IDL and type
    8 | export { LotteryApp, LotteryAppIDL };
ERROR in ./web/src/app/lottery-app/lottery-app-data-access.tsx:1:10
TS2305: Module '"@lottery-app/anchor"' has no exported member 'programId'.
  > 1 | import { programId, getLotteryAppProgram } from '@lottery-app/anchor';
      |          ^^^^^^^^^

Versions
anchor-cli 0.30.0
solana-cli 1.18.8 (src:e2d34d37; feat:3469865029, client:SolanaLabs)
rustc 1.79.0 (129f3b996 2024-06-10)

Steps to reproduce
npx create-solana-dapp lottery-app --yarn --preset react --anchor basic --anchor-program lottery-app
cd lottery-app
yarn
yarn run anchor build
yarn run dev

Running on @next same error:
 ⚠ ../anchor/src/lottery-app-exports.ts
export 'IDL' (reexported as 'LotteryAppIDL') was not found in '../target/types/lottery_app' (module has no exports)

Import trace for requested module:
../anchor/src/lottery-app-exports.ts
../anchor/src/index.ts
./components/lottery-app/lottery-app-data-access.tsx
./components/lottery-app/lottery-app-feature.tsx

../anchor/src/lottery-app-exports.ts
Attempted import error: 'IDL' is not exported from '../target/types/lottery_app' (imported as 'LotteryAppIDL').

Thanks for filing this issue @pplanel - I'll push up a fix.

This is fixed in #74 and published under the next tag:
npx create-solana-dapp@next lottery-app --yarn --preset react --anchor basic --anchor-program lottery-app


Ftrack: Don't force ftrackapp endpoint

Brief description
Auto-fill of ftrack url don't break custom urls. Custom urls couldn't be used as ftrackapp.com is added if is not in the url.
Description
The code was changed in a way that auto-fill is still supported but before ftrackapp is added it will try to use url as is. If the connection works as is it is used.
Additional info
Can't test it as I wasn't able run ftrack server locally and couldn't run redirect proxy either.
Testing notes:

Have ftrack server without ftrackapp in url
Set it in settings to be connected there
Run Tray and log in
Publish at least from one DCC

do you have an idea how to test this one?


Send data for Python Social Auth

Hello @PhilipGarnero thank for clarify me in the other issue. I use FIELDS_STORED_IN_SESSION in Python Social Auth, can I send this data for Python Social Auth when django-rest-framework-social-oauth2 library creates a new user using Python Social Auth?

Well, as django-rest-framework-social-oauth2  is designed for apis I don't see how you would use sessions in this case. But I think you could emulate your behavior by making two calls on the api using partial pipelines maybe ?
You could also try to retrieve your query params using the strategy.request object but I can't guarantee this will work as I didn't try that.

@PhilipGarnero Great! It's working!
In pipeline I got the variable using: backend.strategy.request_data()
Thanks you!


fix: re-add missing kong.ingressVersion template

What this PR does / why we need it:
This adds kong.ingressVersion template which was removed in #813.
Additionally it adds a tests that uses that ( by exposing Admin API service ).

Edit: Ah, we removed the function while it was still used in several places, right?

yes


after modifying the engine and execute felt build did not change in any way

After modifying the engine code and executing felt build, the product did not change in any way.
Am I not using the correct method?

I remove all src/out/host_debug_unopt and execute felt build again, it works,


Troubles getting syntax highlighting in heex files

I can not get syntax highlighting to work in VSCode with Lexical.
Is it not yet supported or am I missing some additional configuration for this?
BTW:
loving Lexical, had only minor issues so far!

Hi @mindreframer! Syntax highlighting should work as soon as you install the lexical-vscode extension and open an Elixir file (with a .ex, .exs or .eex extension).
Do you get any syntax highlighting at all in elixir files or is everything the default color? Do you have other extensions that might be conflicting with vscode-lexical?

@Blond11516 Thanks for the response!
I did figure it out yesterday! Syntax highlighting for Ex(s) files was working, but not for html.heex and also not in ~H""" """" blocks. I remember that ElixirLS had it directly in the extension, but it was taken out. Apparently the Phonenix (phoenixframework.phoenix) extension is now responsible for this, and it works now with Lexical and with ElixirLS. I was surprised, that this is not documented anywhere.
Please add this to the docs, so that other users avoid this confusion.

Glad to hear you figured it out!
I agree it can non-intuitive to have to install a separate extension for HEEX highlighting. I'll keep this issue open as a reminder to add something to the readme.


"Copy to monitors" button

It would be good to copy the folder path as well

It's not copying the Folder/Path from the Monitors tab to the Downloads tab when using the Copy to downloads option. #3118

Closing as duplicate of #3118 then


Remove usage of old ServiceHubServiceBase ctor.

The change wasn't exactly what was expected. The core ServiceHub is still what calls into our Stream, IServiceProvider overload (as intended) which means we needed to call down into the proper basetype constructor to start using Roslyn's non-obsolete one.
Had to manually start the StreamJsonRpc property to work around race conditions in the ServiceHub APIs.

#976
/cc @heejaechang so once this is in/inserted you'll be able to nuke your obsolete ctor.

👍

@NTaylorMullen - let's do an insertion into prerel once we're able, and let @heejaechang know when we have done


Some issues / suggestions

The "manage files" popup doesnt get refreshed immediately once a new file is added or removed for a tv show / movie.
Subtitles have an annoying black background around them once converted to .vtt format.
It would be nice to have a subtitle feature in the player to let the user get video subtitles from opensubtitles.com. Along with that an ability to upload multiple subtitles for a tv show/movie and let user select one of those.

Been loving streama so far, it could use some enhancements and improvements but its one of the best stream players i've seen thus far in terms of design. Hope support for it lasts a lot!

Hi and thanks for your feedback!

for me it updates. (See low quality gif: http://www.gfycat.com/QuestionableEasygoingAxolotl) How exactly have you noticed the issue?
Yea thats default html5 video subtitle behavior. In a future update i plan to use srt and generate the subtitle output with code instead of relying on html5 subtitles. Then Ill be able to give it custom design, too.
A lot of people have asked for this and it is definitely a top prio enhancement!


Find the performance bottleneck in Sprint 54/55 implementation

Tested sending a single block with 40k minimal block items from a client to the block node via publishBlockStream took ~35 secs. 35 sec latency is very far outside the target latency of less than a second.
I discovered 3 main issues:

client latency - Using ghz perf testing tool was taking ~15 sec. After creating a single input file with 40k block items this loaded in less than a second

Synchronous wait to persisting block items to the file system - ~10 secs. We were waiting to save each block item to the file system.

Sending a response for each PublishSubscribeRequest. - Remaining time spend sending 40k responses rather than a single PublishSubscribeResponse for the whole block.

After commenting out the code impacting #2 and #3 (and fixing #1) I found that the block node was able to consume 40k block items in less than a second. This puts us in the expected performance range for now. I created #168 to address #2 and #3 in code.


Not working with multiple instances on one page

Hi! It all works great when there's only one zoomist container on the page, but once I add multiple instances it doesn't seem to work. Would you able to help? Thanks.

Hey there,
this is the way for multiple zoomist
document.querySelectorAll('.zoomist-container').forEach(container => {
  new Zoomist(container, { ... })
})


Using Theano backend. Traceback (most recent call last):   File "C:\Users\liangxin\dive_into_keras\cnn.py", line 37, in      data, label = load_data()   File "C:\Users\liangxin\dive_into_keras\data.py", line 21, in load_data     imgs = os.listdir("./mnist") WindowsError: [Error 3] : './mnist/.'

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[ ] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[ ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Even with the terrible formatting:
The answer is that you need to download the dataset, and put it in the correct location.

keras_usage.zip
i do it as this,but it still has this  error  @patyork

You're 100% sure you extracted it to a folder called mnist/?

mnist/mnist/ is not mnist/
Extract it correctly and you'll be fine.

@patyork  Ok, i will try again. Thank you


Added CONTRIBUTING.md for contributing

Added CONTRIBUTING.md for contributing,
So people didnt messed up with the project

Yes, you can fix it if you want @nabijaczleweli

No, I can't; I ain't got a push access to yer fork

Done
@nabijaczleweli

:+1:

I don't really think it is necessary to list all the contributors, just my opinion.

Ok, You can remove it if you want to

I agree with @JuanPotato , there is no need to list the contributors in a file since the github contributors page will always be up to date and doesn't have to be maintained.

Ok guys,
I added things that you guys want,
The @landscape-bot is checking the commits that I made,
Cheers! :smile:

(Merge commits into one)

???

!!!

Just found it ironic that this PR doesn't follow the guidelines in the contributing file :laughing:
I can add it in without the PR, I want to give you credit, but 7 commits for one file is excessive, and I thought that you should follow one of the guidelines in the file and merge the commits.

This PR is just a huge trainwreck, ain't it?

It is a huge trainwreck.

I'm sorry, this PR is really a mess, I can make my own contributing.md or you can make a new clean PR @christoga

Thank you.