Installing extension on remote host takes forever

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.58 and 1.59
OS Version: RHEL 8.4

Steps to Reproduce:

ssh to some remote web server (in my case it is my academic institution's High power computing server)
once the vs code is initialized for the server, go to extensions and try installing any extension on the remote host. On clicking install on , it shows installing for a very long time.
I tried installing python extensions, C++ extensions, all of them take too long time. So, long that I have to close the host connection.
Upon reestablishing the connection I can see the extensions installed(sometimes but not always)

Can you please provide following logs while you reproduce the issue?

View -> Open View… -> Log (Shared)
View -> Open View… -> Log (Window)

View -> Open View… -> Log (Remote)

@sandy081
log(window) is empty
log(main)
[2021-08-20 14:18:57.708] [main] [info] window: using vscode-file:// protocol and V8 cache options: bypassHeatCheck
[2021-08-20 14:18:57.742] [main] [info] update#setState idle
[2021-08-20 14:19:03.878] [main] [info] window: using vscode-file:// protocol and V8 cache options: bypassHeatCheck
[2021-08-20 14:19:27.744] [main] [info] update#setState checking for updates
[2021-08-20 14:19:27.827] [main] [info] update#setState idle
[2021-08-20 14:24:30.570] [main] [info] window: using vscode-file:// protocol and V8 cache options: bypassHeatCheck

log(shared)
[2021-08-20 14:19:08.627] [sharedprocess] [info] [logs cleanup]: Removing log folders '20210820T121451, 20210820T121615'
[2021-08-20 14:19:28.626] [sharedprocess] [info] [code cache cleanup]: Starting to clean up old code cache folders.
[2021-08-20 14:19:28.626] [sharedprocess] [info] [storage cleanup]: Starting to clean up storage folders.
[2021-08-20 14:19:28.627] [sharedprocess] [info] [storage cleanup]: Deleting storage folder<PHONE_NUMBER>825.
[2021-08-20 14:19:28.627] [sharedprocess] [info] [storage cleanup]: Deleting storage folder<PHONE_NUMBER>994.
[2021-08-20 14:19:28.627] [sharedprocess] [info] [storage cleanup]: Deleting storage folder<PHONE_NUMBER>283.
[2021-08-20 14:19:28.628] [sharedprocess] [info] [storage cleanup]: Deleting storage folder<PHONE_NUMBER>957.
[2021-08-20 14:19:28.628] [sharedprocess] [info] [storage cleanup]: Deleting storage folder<PHONE_NUMBER>249.
[2021-08-20 14:19:28.628] [sharedprocess] [info] [storage cleanup]: Deleting storage folder<PHONE_NUMBER>415.
[2021-08-20 14:19:28.628] [sharedprocess] [info] [storage cleanup]: Deleting storage folder<PHONE_NUMBER>441.
[2021-08-20 14:19:38.626] [sharedprocess] [info] [language pack cache cleanup]: Starting to clean up unused language packs.

logs(remote server)
[2021-08-20 14:24:45.308] [remoteagent] [info]

*
* Visual Studio Code Server
*
* Reminder: You may only use this software with Visual Studio family products,
* as described in the license https://aka.ms/vscode-remote/license
*

[2021-08-20 14:24:45.308] [remoteagent] [info] Extension host agent started.
[2021-08-20 14:24:45.548] [remoteagent] [info] [<IP_ADDRESS>][e10b9382][ManagementConnection] New connection established.
[2021-08-20 14:24:45.565] [remoteagent] [info] [<IP_ADDRESS>][972019e4][ExtensionHostConnection] New connection established.
[2021-08-20 14:24:45.570] [remoteagent] [info] [<IP_ADDRESS>][972019e4][ExtensionHostConnection] <12127> Launched Extension Host Process.

logs(extension host)
[2021-08-20 14:24:31.470] [exthost] [info] extension host started
[2021-08-20 14:24:31.483] [exthost] [info] ExtensionService#_doActivateExtension ms-vscode-remote.remote-ssh {"startup":false,"extensionId":{"value":"ms-vscode-remote.remote-ssh","_lower":"ms-vscode-remote.remote-ssh"},"activationEvent":"onResolveRemoteAuthority:ssh-remote"}
[2021-08-20 14:24:31.483] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/private/.vscode/extensions/ms-vscode-remote.remote-ssh-0.65.7/out/extension
[2021-08-20 14:24:46.420] [exthost] [info] eager extensions activated
[2021-08-20 14:24:46.425] [exthost] [info] ExtensionService#_doActivateExtension vscode.microsoft-authentication {"startup":false,"extensionId":{"value":"vscode.microsoft-authentication","_lower":"vscode.microsoft-authentication"},"activationEvent":"onAuthenticationRequest:microsoft"}
[2021-08-20 14:24:46.425] [exthost] [info] ExtensionService#loadCommonJSModule file:///usr/share/code/resources/app/extensions/microsoft-authentication/dist/extension.js
[2021-08-20 14:24:46.432] [exthost] [info] ExtensionService#_doActivateExtension ms-vscode-remote.remote-containers {"startup":false,"extensionId":{"value":"ms-vscode-remote.remote-containers","_lower":"ms-vscode-remote.remote-containers"},"activationEvent":"onStartupFinished"}
[2021-08-20 14:24:46.432] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/private/.vscode/extensions/ms-vscode-remote.remote-containers-0.191.1/dist/extension/extension
[2021-08-20 14:24:46.450] [exthost] [info] ExtensionService#_doActivateExtension ms-vscode-remote.remote-wsl {"startup":false,"extensionId":{"value":"ms-vscode-remote.remote-wsl","_lower":"ms-vscode-remote.remote-wsl"},"activationEvent":"onStartupFinished"}
[2021-08-20 14:24:46.450] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/private/.vscode/extensions/ms-vscode-remote.remote-wsl-0.58.2/dist/extension
[2021-08-20 14:24:46.929] [exthost] [info] ExtensionService#_doActivateExtension vscode.github-authentication {"startup":false,"extensionId":{"value":"vscode.github-authentication","_lower":"vscode.github-authentication"},"activationEvent":"onAuthenticationRequest:github"}
[2021-08-20 14:24:46.929] [exthost] [info] ExtensionService#loadCommonJSModule file:///usr/share/code/resources/app/extensions/github-authentication/dist/extension.js

log(Remote extension host)
[2021-08-20 14:24:46.573] [exthost] [info] extension host started
[2021-08-20 14:24:46.648] [exthost] [info] ExtensionService#_doActivateExtension vscode.debug-auto-launch {"startup":true,"extensionId":{"value":"vscode.debug-auto-launch","_lower":"vscode.debug-auto-launch"},"activationEvent":"*"}
[2021-08-20 14:24:46.648] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/maths/private/.vscode-server/bin/3866c3553be8b268c8a7f8c0482c0c0177aa8bfa/extensions/debug-auto-launch/dist/extension
[2021-08-20 14:24:46.658] [exthost] [info] ExtensionService#_doActivateExtension vscode.git {"startup":true,"extensionId":{"value":"vscode.github","_lower":"vscode.github"},"activationEvent":"*"}
[2021-08-20 14:24:46.658] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/maths/private/.vscode-server/bin/3866c3553be8b268c8a7f8c0482c0c0177aa8bfa/extensions/git/dist/main
[2021-08-20 14:24:46.757] [exthost] [info] ExtensionService#_doActivateExtension ms-vscode.cpptools {"startup":true,"extensionId":{"value":"ms-vscode.cpptools","_lower":"ms-vscode.cpptools"},"activationEvent":"*"}
[2021-08-20 14:24:46.757] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/maths/private/.vscode-server/extensions/ms-vscode.cpptools-1.5.1/dist/main
[2021-08-20 14:24:47.113] [exthost] [info] ExtensionService#_doActivateExtension ms-vscode.js-debug {"startup":false,"extensionId":{"value":"ms-vscode.js-debug","_lower":"ms-vscode.js-debug"},"activationEvent":"onCommand:extension.js-debug.clearAutoAttachVariables"}
[2021-08-20 14:24:47.113] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/maths/private/.vscode-server/bin/3866c3553be8b268c8a7f8c0482c0c0177aa8bfa/extensions/ms-vscode.js-debug/src/extension.js
[2021-08-20 14:24:47.390] [exthost] [info] ExtensionService#_doActivateExtension vscode.github {"startup":false,"extensionId":{"value":"vscode.github","_lower":"vscode.github"},"activationEvent":"*"}
[2021-08-20 14:24:47.390] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/maths/private/.vscode-server/bin/3866c3553be8b268c8a7f8c0482c0c0177aa8bfa/extensions/github/dist/extension.js
[2021-08-20 14:24:56.889] [exthost] [info] ExtensionService#_doActivateExtension vscode.emmet {"startup":false,"extensionId":{"value":"vscode.emmet","_lower":"vscode.emmet"},"activationEvent":"onStartupFinished"}
[2021-08-20 14:24:56.890] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/maths/private/.vscode-server/bin/3866c3553be8b268c8a7f8c0482c0c0177aa8bfa/extensions/emmet/dist/node/emmetNodeMain
[2021-08-20 14:24:56.919] [exthost] [info] ExtensionService#_doActivateExtension vscode.merge-conflict {"startup":false,"extensionId":{"value":"vscode.merge-conflict","_lower":"vscode.merge-conflict"},"activationEvent":"onStartupFinished"}
[2021-08-20 14:24:56.919] [exthost] [info] ExtensionService#loadCommonJSModule file:///home/maths/private/.vscode-server/bin/3866c3553be8b268c8a7f8c0482c0c0177aa8bfa/extensions/merge-conflict/dist/mergeConflictMain

There is some vscode merge conflict in logs (remote extension). I cannot comprehend what it is but I guess it has something to do with the issue.

Have you tried installing the extension when these logs are captured? It seems there is no activity of installing extensions.

Yes, I was installing the Python extension while I was copying the logs, I noted the time when I clicked the install button but nothing was added to logs after that time instance.

Weird, is the connection to remote active? Are there any errors in the devtool/console?

Yes, the connection with the remote is active and all the logs are getting updated in the .vscode-server directory on the host

Do you have this setting set - remote.downloadExtensionsLocally ?

That option wasn't checked earlier but I checked it and retried everything. This time I can see the log for extension installation
logs(Remote Server)
[2021-08-20 16:06:32.621] [remoteagent] [info]

*
* Visual Studio Code Server
*
* Reminder: You may only use this software with Visual Studio family products,
* as described in the license https://aka.ms/vscode-remote/license
*

[2021-08-20 16:06:32.622] [remoteagent] [info] Extension host agent started.
[2021-08-20 16:06:33.489] [remoteagent] [info] [<IP_ADDRESS>][7d303744][ManagementConnection] New connection established.
[2021-08-20 16:06:33.519] [remoteagent] [info] [<IP_ADDRESS>][d25bf354][ExtensionHostConnection] New connection established.
[2021-08-20 16:06:33.525] [remoteagent] [info] [<IP_ADDRESS>][d25bf354][ExtensionHostConnection] <12111> Launched Extension Host Process.
[2021-08-20 16:08:41.939] [remoteagent] [info] Installing extension: ms-python.python
[2021-08-20 16:08:43.217] [remoteagent] [info] Installing extension: ms-python.vscode-pylance
[2021-08-20 16:08:50.463] [remoteagent] [info] Installing extension: ms-toolsai.jupyter
[2021-08-20 16:10:49.284] [remoteagent] [error] Error while preparing to install dependencies and extension packs of the extension: ms-python.python
[2021-08-20 16:10:49.288] [remoteagent] [error] Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1146:16)
[2021-08-20 16:10:49.289] [remoteagent] [error] Failed to install extension: ms-python.python connect ETIMEDOUT <IP_ADDRESS>:443
[2021-08-20 16:10:50.432] [remoteagent] [error] Error while preparing to install dependencies and extension packs of the extension: ms-python.vscode-pylance
[2021-08-20 16:10:50.432] [remoteagent] [error] Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1146:16)
[2021-08-20 16:10:50.433] [remoteagent] [error] Failed to install extension: ms-python.vscode-pylance connect ETIMEDOUT <IP_ADDRESS>:443
[2021-08-20 16:10:58.994] [remoteagent] [info] Extracted extension to /home/maths/integrated/mt6130583/.vscode-server/extensions/.bf0359ef-936d-4cff-8785-fbeb716de151: ms-toolsai.jupyter
[2021-08-20 16:10:59.023] [remoteagent] [info] Renamed to /home/maths/integrated/mt6130583/.vscode-server/extensions/ms-toolsai.jupyter-2021.8.1236758218
[2021-08-20 16:10:59.030] [remoteagent] [info] Extracting completed. ms-toolsai.jupyter
[2021-08-20 16:10:59.040] [remoteagent] [info] Extension installed successfully: ms-toolsai.jupyter

It seems like your remote has network connection issues, so I would recommend to check above setting so that extensions are downloaded locally and streamed to remote.

Error while preparing to install dependencies and extension packs of the extension: ms-python.python
Above error is already fixed and available in next release - https://github.com/microsoft/vscode/issues/130962

@sandy081  did you check the IP address mentioned in the logs? <IP_ADDRESS>
it belongs to Microsoft and if you try to reach it, it shows

Our services aren't available right now

We're working to restore all services as soon as possible. Please check back soon.
0FIofYQAAAABP7/H30+eeQr0v4iSmgqOyREVMMDFFREdFMDMxMQBFZGdl

So, are you sure that there is issue at my host's network?

May be with your DNS as I do not think Marketplace service is up and running. So can you please check if the remote machine has any proxy set up or DNS resolving to wrong IP addresses.


248 release changes

Bring internal changes external. Fixes #1218

Hi, @lukewrites! Thanks for opening this PR!

In the file tdvt/tdvt/setup_env.py:

Line 269:40: W605 invalid escape sequence '\s'
Line 269:24: W605 invalid escape sequence '\s'
Line 268:24: W605 invalid escape sequence '\s'
Line 267:24: W605 invalid escape sequence '\s'

For reference: Error Code Reference, PEP8, Python code style


i18n(ru):  Upgrade v2 and Blog tutorial translation

What kind of changes does this PR include?

Translated content

Description

Translation to russian Migration guide and Blog tutorial

LGTM

@SonOfStep @dezzerlol Можете взглянуть?

Fix all


Unjail event field is incorrect

Describe the bug
Unjail event field is incorrect
To Reproduce
https://github.com/forbole/big_dipper/blob/master/imports/api/transactions/server/methods.js#L82
In fact unjail event is like this in v0.37：
    "events":[
        {
            "type":"message",
            "attributes":[
                {
                    "key":"action",
                    "value":"unjail"
                },
                {
                    "key":"module",
                    "value":"slashing"
                },
                {
                    "key":"sender",
                    "value":"xxxxx"
                }
            ]
        }
    ]

Ref https://github.com/cosmos/cosmos-sdk/blob/master/x/slashing/handler.go#L39


Error extracting & loading with tap Zuora (tap Zuora is broken for users)

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/433
Originally created by @valexieva on 2019-02-28 21:40:51

What is the current bug behavior?
(What is happening now)
I am trying to extract and load some of the test data in Zuora, but I get the below error even though I have the right credentials (please see second meltano vault in 1Password)
tap_zuora.client.ApiException: 401: b'{\n  "success" : false,\n  "reasons" : [ {\n    "code" : 90000011,\n    "message" : "this resource is protected, please sign in first"\n  } ]\n}'

What is the expected correct behavior?
I should be able to extract and load data if our configs and login information is correct
Steps to reproduce
Create meltano project, fill in configs, source .env and run the elt
Relevant logs and/or screenshots
Running extract & load...
[MainThread][WARNING][2019-03-08 00:03:21,932] Could not find tap.properties.json in /Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/tap.properties.json, skipping.
[MainThread][WARNING][2019-03-08 00:03:21,932] Could not find state.json in /Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/state.json, skipping.
[MainThread][WARNING][2019-03-08 00:03:21,939] No state was found, complete import.
INFO Sending version information to singer.io. To disable sending anonymous usage data, set the config parameter "disable_collection" to true
INFO Starting discover
INFO GET: https://rest.zuora.com/v1/describe
CRITICAL 401: b'{\n  "success" : false,\n  "reasons" : [ {\n    "code" : 90000011,\n    "message" : "this resource is protected, please sign in first"\n  } ]\n}'
Traceback (most recent call last):
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/bin/tap-zuora", line 11, in <module>
    load_entry_point('tap-zuora==1.1.5', 'console_scripts', 'tap-zuora')()
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/lib/python3.7/site-packages/singer/utils.py", line 192, in wrapped
    return fnc(*args, **kwargs)
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/lib/python3.7/site-packages/tap_zuora/__init__.py", line 124, in main
    do_discover(client, force_rest)
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/lib/python3.7/site-packages/tap_zuora/__init__.py", line 71, in do_discover
    catalog = {"streams": discover_streams(client, force_rest)}
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/lib/python3.7/site-packages/tap_zuora/discover.py", line 168, in discover_streams
    for stream_name in discover_stream_names(client):
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/lib/python3.7/site-packages/tap_zuora/discover.py", line 87, in discover_stream_names
    xml_str = client.rest_request("GET", "v1/describe").content
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/lib/python3.7/site-packages/tap_zuora/client.py", line 85, in rest_request
    return self._request(method, url, headers=self.rest_headers, **kwargs)
  File "/Users/virjiniaalexieva/projects/zuora/.meltano/extractors/tap-zuora/venv/lib/python3.7/site-packages/tap_zuora/client.py", line 75, in _request
    raise ApiException(resp)
tap_zuora.client.ApiException: 401: b'{\n  "success" : false,\n  "reasons" : [ {\n    "code" : 90000011,\n    "message" : "this resource is protected, please sign in first"\n  } ]\n}'
[MainThread][ERROR][2019-03-08 00:03:22,732] Could not select stream, catalog file is missing.

Possible fixes
(If you can, link to the line of code that might be responsible for the problem or suggest a fix)
Further regression test
(Ensure we automatically catch similar issues in the future)
/label ~bug
/label ~"To Do"

View 19 previous comments from the original issue on GitLab


Small readme fix

Hi @jpsim,
thank you for open sourcing this ❤
I spotted a tiny typo in the readme, hope you don't mind the PR 🙏🏻
Wish you a nice day :)

Thanks! How’s you even find this? I haven’t shared it anywhere yet.

Thanks! How’s you even find this? I haven’t shared it anywhere yet.

How’s you even find this?

It's a secret! 😄
Just kidding, It was right at the top of my GitHub feed (when you open github.com, you see all activity of the people you follow there):

How’s you even find this?

It's a secret! 😄
Just kidding, It was right at the top of my GitHub feed (when you open github.com, you see all activity of the people you follow there):

Ah right. I forget that GitHub has “social” features sometimes.

Ah right. I forget that GitHub has “social” features sometimes.


Bit scrappy and needs some tests

Description
User removal support
Issues Resolved
#55
Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable

You have a ruby syntax error @damacus

Seems like this needs a rebase. Or perhaps it has been superseded by other efforts?

          2 Errors

:no_entry_sign:
Please add labels to this Pull Request

:no_entry_sign:
Please include a CHANGELOG entry.

          1 Warning

:warning:
This Pull Request is probably missing tests.

  Generated by :no_entry_sign: Danger

Hey,
Thank you for the PR. We are closing due to inactivity.
If you feel this feature should be merged in, please reopen and rebase as your contributions are always welcome.
Sous Chefs


Remove Actor concept and some other improvements with bitboard

Remove Actor
Situation.generateMovesAt now only generate valid moves
Move destFrom test functions to ChessTest class
Use bitboard to optimize InsufficientMatingMaterial and other draw conditions
Add some helper functions for Board like kingsAndBishopsOnly
Remove threatens tests because we have Bitboard.pieceAttacks functions

Very cool!


Clarify that Fabric is included by default

Q
A

content fix?
Yes

New article?
No

Related issues?
No

What's in this Pull Request?
Clarifies that Fabric Core styles (typography, colors, icons, etc.) are loaded onto the page by default. Web parts do not need to import anything to make use of these styles.

Hi @mikewheaton, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Mike Wheaton). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

thx Mike, good clarification.


Scheduling shortcuts broken with Todoist Version 968

Typing t followed by any of the single-character shortcuts (c, r, s, t, w) no longer works.

Also having this issue

+1

+1

Yep I'm seeing this too, will hopefully have time to fix it this evening
On Thu, Sep 5, 2019 at 9:30 AM jamesx4<EMAIL_ADDRESS>wrote:

+1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mgsloan/todoist-shortcuts/issues/85?email_source=notifications&email_token=AAAFQKTR5U7Y7M3AP66WYT3QIEQ3DA5CNFSM4ITUQ5AKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD57SHUI#issuecomment-528425937,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAFQKTNZRSM6LKRRVBAMPTQIEQ3DANCNFSM4ITUQ5AA
.

Thanks for reporting!  I've uploaded version 61 with a fix

Thanks for reporting! I've uploaded version 61 with a fix

Thanks for fixing so fast!!  I've really come to depend on the shortcuts in Todoist!


Variable card heights in a single scrollable container issue

I created a sample sandbox for you to playaround with that is similar to my code. I have a list of cards in a single scrollable container that need to be sorted vertically.
Each card has height auto assigned to them. That is because each card will have different fields inside depending on the "slotType" in my case.
My problem is that when i got to sort, you can see the index number shift and I can tell it is because of my height auto and the transform trying to calculate its correct position. It does fix itself when the drop ends.
Does anyone know how to keep those numbers fixed or move with the transition of the cards to keep them aligned.
Sandbox:
https://codesandbox.io/s/friendly-https-crhckj?file=/src/App.tsx
ScreenRecord:
https://github.com/clauderic/dnd-kit/assets/102334688/c64d9615-4257-445b-b4ec-9b71e80ef747

Fixed this issue actually by using newIndex returned from the useSortable hook instead of using the index from the map i was using


Import hadoop version 2.8.0

Import-only, so merging (and ignoring test failures, as they're handled in another PR, already)


Ref calendarElement is null causes "Cannot read property 'removeEventListener' of null" #204

https://github.com/Kiarash-Z/react-modern-calendar-datepicker/issues/204

I am also facing the same issue. Could you please resolve this issue?

Hope this PR will merge ASAP.

Hi! It's solved this issues?

I just fork this repo and have custom build to resolve this issue
https://github.com/Cow258/react-modern-calendar-datepicker/tree/built

Thank you! I look forward to the resolution of the issue.


Switch to prebuilt docker image for Linux build on Travis

The Linux Travis build now runs all stack commands in a Docker image we built for this purpose, haskell-lavello, which contains all the prebuilt dependencies we need for the project.
This means that adding new dependencies to the build is now free, at the small upfront cost of pulling the image from Docker Cloud (~7mins).
Note that the osx build still uses native stack commands, but build times there seem to be manageable so far.

timed out?

passed, we could merge this


res.redirect error

Question
Hello guys I get an error while doing res.cookie and res.redirect in the success message.
My Code:
const loginPageShow = async (req, res, next) => {

    try {
        const youtube = await new Innertube();

        var creds = '';

        youtube.ev.on('auth', (data) => {

            if (data.status == 'AUTHORIZATION_PENDING') {

                res.render('user/Login', { layout: '../layouts/General', title: `Youtube Abone Hilesi`, description: ``, keywords: ``, data })

            } else if (data.status == 'SUCCESS') {

                function credsSend() {

                    return creds = data.credentials;
                }
                credsSend()

                         const datax = {
                             time: Date(),
                             userId: data.credentials.access_token,
                         }
                         const jwtToken = jwt.sign(datax, process.env.JWT_SECRET_KEY, { expiresIn: '1d' });

                         res.clearCookie('access_token');

                         res.cookie('access_token', jwtToken, { expires: new Date(Date.now() + 900000) })
                         //console.log(req.cookies.access_token);
                         res.redirect(`https://twitter.com`);

            }

        });

        await youtube.signIn(creds);

       async function check() {

            const uyeVarMiFilter = { channel_id: (await youtube.account.getInfo()).channel_id };
            const uyeVarMi = await User.find(uyeVarMiFilter).countDocuments();
            if (uyeVarMi > 0) {
                //const delUser = await User.findOneAndDelete({ channel_id: (await youtube.account.getInfo()).channel_id })
                const guncelBilgiler = {
                    name: (await youtube.account.getInfo()).name,
                    email: (await youtube.account.getInfo()).email,
                    channel_id: (await youtube.account.getInfo()).channel_id,
                    subscriber_count: (await youtube.account.getInfo()).subscriber_count,
                    photo: (await youtube.account.getInfo()).photo[0].url,
                    access_token: creds.access_token,
                    refresh_token: creds.refresh_token,
                    expires: creds.expires,
                }
                await User.findOneAndUpdate(uyeVarMiFilter, guncelBilgiler)
            } else {
                const filter = { access_token: creds.access_token };

                const newUser = new User({
                    name: (await youtube.account.getInfo()).name,
                    email: (await youtube.account.getInfo()).email,
                    channel_id: (await youtube.account.getInfo()).channel_id,
                    subscriber_count: (await youtube.account.getInfo()).subscriber_count,
                    photo: (await youtube.account.getInfo()).photo[0].url,
                    access_token: creds.access_token,
                    refresh_token: creds.refresh_token,
                    expires: creds.expires,
                    tActive: '1'
                });
                await newUser.save();
            }
        } check();

    } catch (error) {
        console.log(error);
    }

}

Error Code:
node:internal/errors:464
    ErrorCaptureStackTrace(err);
    ^

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
    at new NodeError (node:internal/errors:371:5)
    at ServerResponse.setHeader (node:_http_outgoing:576:11)
    at ServerResponse.header (/Users/canyesilyurt/Desktop/ytPanel/node_modules/express/lib/response.js:794:10)
    at ServerResponse.append (/Users/canyesilyurt/Desktop/ytPanel/node_modules/express/lib/response.js:755:15)
    at ServerResponse.res.cookie (/Users/canyesilyurt/Desktop/ytPanel/node_modules/express/lib/response.js:884:8)
    at ServerResponse.clearCookie (/Users/canyesilyurt/Desktop/ytPanel/node_modules/express/lib/response.js:827:15)
    at EventEmitter.<anonymous> (/Users/canyesilyurt/Desktop/ytPanel/src/controllers/homeController.js:45:30)
    at EventEmitter.emit (node:events:532:35)
    at Timeout._onTimeout (/Users/canyesilyurt/Desktop/ytPanel/node_modules/youtubei.js/lib/core/OAuth.js:120:18)
    at processTicksAndRejections (node:internal/process/task_queues:96:5) {
  code: 'ERR_HTTP_HEADERS_SENT'
}
[nodemon] app crashed - waiting for file changes before starting...

Other details
No response
Checklist

[X] I am running the latest version.
[X] I checked the documentation and found no answer.
[X] I have searched the existing issues and made sure this is not a duplicate.
[X] I have provided sufficient information.

This error appears to be unrelated to YouTube.js and rather an error in your Express.js logic.

(Also just a little tip, refactoring your getInfo() calls into a single variable will most likely improve your performance drastically and might even help some issues you might encounter down the line, because each call does a network request)

This error appears to be unrelated to YouTube.js and rather an error in your Express.js logic.

When I checked, it should redirect in the success loop but it doesn't. The problem goes away when I don't use res.render in the AUTHORIZATION_PENDING loop, but if I remove the res.render command, I have no idea how to show the code to the user.

When I checked, it should redirect in the success loop but it doesn't. The problem goes away when I don't use res.render in the AUTHORIZATION_PENDING loop, but if I remove the res.render command, I have no idea how to show the code to the user.

You cannot send a page to the user (in the AUTHORIZATION_PENDING callback) and then update the headers afterwards, HTTP doesn't allow this as the body in sent after the headers (which contain both the cookies and redirect location).
The best way will be to use clientside javascript to handle this.

When I checked, it should redirect in the success loop but it doesn't. The problem goes away when I don't use res.render in the AUTHORIZATION_PENDING loop, but if I remove the res.render command, I have no idea how to show the code to the user.

You cannot send a page to the user (in the AUTHORIZATION_PENDING callback) and then update the headers afterwards, HTTP doesn't allow this as the body in sent after the headers (which contain both the cookies and redirect location).
The best way will be to use clientside javascript to handle this.

How do I do this, can you show a source?
youtube.ev.on('auth', (data) => {
      switch (data.status) {
        case 'AUTHORIZATION_PENDING':
          console.log(`
                Hello! On your phone or computer,
                go to ${data.verification_url} and enter
                the code ${data.code}.
              `);
              next();
          break;

          case 'SUCCESS':
            console.log('bbbb');
          break;
      }
    });

How can I log in other than this code?

Is it possible to access data.code without using this function?
youtube.ev.on('auth', (data) => {
      switch (data.status) {
        case 'AUTHORIZATION_PENDING':
          console.log(`
                Hello! On your phone or computer,
                go to ${data.verification_url} and enter
                the code ${data.code}.
              `);
              next();
          break;

          case 'SUCCESS':
            console.log('bbbb');
          break;
      }
    });


Codeunit 5005271 "Create Delivery Reminder" Onprem

Hi Al-team,
why is the function UpdateLines marked with OnPrem? Can you make this callable for the cloud?
[Scope('OnPrem')]
    procedure UpdateLines(DeliveryReminder: Record "Delivery Reminder Header")
    begin
        DeliveryReminderLine.Reset();
        DeliveryReminderLine.SetRange("Document No.", DeliveryReminder."No.");
        OK := DeliveryReminderLine.Find('-');
        while OK do begin
            OK :=
              (DeliveryReminderLine.Type = DeliveryReminderLine.Type::" ") and
              (DeliveryReminderLine."Attached to Line No." = 0);
            if OK then begin
                DeliveryReminderLine.Delete(true);
                OK := DeliveryReminderLine.Next <> 0;
            end;
        end;
        OK := DeliveryReminderLine.Find('+');
        while OK do begin
            OK :=
              (DeliveryReminderLine.Type = DeliveryReminderLine.Type::" ") and
              (DeliveryReminderLine."Attached to Line No." = 0);
            if OK then begin
                DeliveryReminderLine.Delete(true);
                OK := DeliveryReminderLine.Next(-1) <> 0;
            end;
        end;

        InsertBeginTexts(DeliveryReminder);
        InsertEndTexts(DeliveryReminder);
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[RFC] CORS Support

Looking towards 4.0... I was wondering about including CORS support built-in for Slim users and I want to hear your feedback on it.
Ideally it would be something that you can either turn ON or OFF, so that you are not forced to use it.
The idea comes from the few projects that already address this issue.
[1] https://github.com/palanik/CorsSlim
[2] http://joshuasampia.com/2015/11/05/cors-slim-php-setup/
... etc

I don't think this belongs in the core of Slim, but can be part of Slim Skeleton.

Are there any things within Slim that make supporting CORS harder than it could be?

I don't think we should add this on Slim.
A 'simple' middleware could add CORS Support.
Maybe we only need a Middleware/Addon section on slim website to index these useful things

@mathmarques @akrabat
The problem with CORS is that in the pre-flighted requests you need to know what endpoints are registered.
Example:
Resource: /v1/user

Supports: GET, POST, PUT

The preflight OPTIONS /v1/user should return a header with Access-Control-Allow-Methods: POST, GET, PUT
Doing this in a middleware... I have no idea how that could be done.
It makes most sense to do it at the router level, as it can inspect the Dispatcher to find routes LIKE this one and construct a HTTP METHOD List for this resource.

@silentworks Not sure if it should land in core myself, but its something that I deal with frequently. Perhaps a CorsRouter might be more useful.

I'm maybe wrong but any middleware can access to the routes registred in the router, no ?

@lalop true... in order to figure out what ROUTE you are in you will need to use the determineRouteBeforeDispatch setting.

Should we be organising the information in the router, so that getting this information out is reasonably easy?

Adding a route middleware and the route information as attribute to the request should do the trick I suppose

I think CORS does not belong to core.There already are couple of middlewares,  I including mine which uses neomerx/cors-psr7 for heavy lifting. However the OPTIONS problem @geggleto mentioned is real.

The main problem I had in implementing CORS in Slim was (is) the absence of the Origin header in $request->getHeader*()
Same thing happened with Authorization (for OAuth2). I had to use a SetEnvIf in htaccess to have access to it.
A way to add "whitelisted headers" in Headers' $special array before the createFromEnvironment() would be great, IMHO.
Right now the call to it is hardcoded in Request, so you can't even override it in Application. You need to extend Request and rewrite its createFromEnvironment from scratch.
While I'm on it, a way to extend Request's $validMethods would be nice too, as I had (still, for CORS) to use PROPFIND (or create a new INFO method) where I used OPTIONS, because the latter is hijacked by CORS.

The Authorization header thing is not related to Slim. It is considered a PHP feature. If you send anything else than a valid Authorization: Basic header it will not be available to PHP without the .htaccess workaround. Or it is, but only using apache_request_headers() which is Apache specific.

Sorry, forget what I said about the Options header. It's a bug in my Postman. The Origin header gets correctly parsed by Slim.
It would still be nice to have a way to customize $validmethods. :)

The more I think about the OPTIONS problem the more I think we should have a different Router to handle it. If we had a CORS Router that could track routes and the method they are mounted to it would be rather trivial to solve the options problem with CORS. This wouldn't be a BC break as we are adding a new Router for our users to use, and most importantly it would be optional.
It would simply require a replace in the container... or we could even have it as a configuration option when constructing the container.
The CORSRouter would then extend the Slim basic router and would overload the methods that register routes. A new list would be created to track the routes, with Pattern being the key and verbs added on.
a simple getter to the router with a pattern would suffice to get a list of VERBS supported by your application.
//...  Rough Pseudo Code
protected $patterns = [];

public function getSupportedVerbs($pattern) {
    return $this->patterns[$pattern];
}

protected function addSupportedVerb($pattern, $verb) {
    $this->patterns[$pattern][] = $verb;
}

public function get(....) {
    $this->addSupportedVerb($pattern, 'GET');
    parent::get(....);
}

//...

I found a way for middleware to get the defined verbs for current routes pattern with the code below. Note that this is not always the the same as all the defined verbs for current routes uri. However it is close enough for most use cases.
require __DIR__ . "/vendor/autoload.php";

$app = new Slim\App([
    "settings"  => [
        "determineRouteBeforeAppMiddleware" => true,
    ]
]);

$app->add(function($request, $response, $next) {
    $pattern = $request->getAttribute("route")->getPattern();

    $methods = [];
    foreach ($this->router->getRoutes() as $route) {
        if ($pattern === $route->getPattern()) {
            $methods = array_merge_recursive($methods, $route->getMethods());
        }
    }

    $response->write(print_r($methods, true));
    return $next($request, $response);
});

$app->get("/api/{id}", function($request, $response, $arguments) {
});

$app->post("/api/{id}", function($request, $response, $arguments) {
});

$app->map(["DELETE", "PATCH"], "/api/{id}", function($request, $response, $arguments) {
});

$app->run();

%curl --include http://localhost:8080/api/xxx
HTTP/1.1 200 OK
Content-Type: text/html; charset=UTF-8
Content-Length: 76

Array
(
    [0] => GET
    [1] => POST
    [2] => DELETE
    [3] => PATCH
)


Consolidate lists

Hello,
Thank you for this work. It is very interesting to me and I would like to make use of it.
Is there any way to consolidate all the separate lists into 1 single list to add to pihole rather than adding what looks to be 50-100 separate entries? Or is this something that can be setup somehow?
Kind regards
Peter

Saw you updated the lists... just wondering if you had a chance to check my query out?

Hi Peter, and sorry for not answering faster, but I have been busy with other projects.
I think there must be a way to compile it all into 1 single list. The reason I have them separate is that there are different types of threats, and some need more maintenance than others and also to make it more streamlined for myself and how I work.
The lists here are at this point only for testing, and to keep it simple when adding/removing lists to the systems I play around with I chose to have each threat on its own separate list.
I'm no super user as to Github and coding yet, so can't say how it should be done.

No problem thank you


Netflix is blocked on ATV

Energized Protection - block ⚡

Let's make an annoyance free, better open internet, altogether!

Issue Submit Form

  Provide the following info properly, which will help us to resolve your issue quickly.

Issue(s):
Type x in between [ ] and make sure there isn't any space between brackets. Example; for Your Selected Issue(s), type like this - [x]
You can select more than one category of issues if you need to!

[x] Whitelist
[ ] Blacklist
[x] App Broken
[ ] Website Broken
[ ] Request
[ ] Other Issue

Pack(s):
Write the name of the pack(s) you are using.

Pack(s) Name: Ultimate AND Basic

Extension(s):
Also name the extension(s) if you are using any. If you don't then leave it blank.

Extension(s) Name:

Domain(s):
If you are submitting this issue for whitelist/blacklist issue, send us the domain(s) for whitelisting/blacklisting here. Kindly use the Code Tag to prevent tracking.

Domain(s):

ios.prod.http1.netflix.com

Your Config:
Just to ensure there is no issues or conflicts with other app/software/magisk module/extension/source list.
Make sure you are running Energized Protection Service only.

Client: NextDNS

Version: No version as it comes from NextDNS filter list.

Detail:
Write us a lil bit more about your issue or query. You can attach any screen shot or log of the issue or advert, this will help to highlight it.

Your Issue Detail: Connection to Netflix on the Apple TV isn't working anymore because the above request is blocked.

Thank you for making Energized Protection great, with your kind help!
A project by Ador with ❤

I also saw the same issue on my Pihole.

I also saw the same issue on my Pihole.

I whitelisted it till the list get updated, if it gets updated...

Seriously, I really don't see why reporting issues. There is no response and no action to solve the problems.
Energized lists are now opted out.


remove callback called before item is removed

Hi Dragosu,
I built a tree with a menu to add/remove item.
For each change, I dump tree content with method serialize of the acitree API.
I noticed that if I remove an item than the remove callback is called before the item is really removed.
According to the acitree source code, we can see the following scheduling of methods calls :
// just remove the item
this._success(item, options);
this._removeItem(item);
Could you please confirm or infirm the problem ?
If it's confirmed than could you help me to solve it ?
Thank you in advance for your response.
Best regards,
Franck

Hi, you are right, the element is removed from the DOM after the .success function call. I think there was a reason why I did it this way, hard to tell that I don't remember why :) You can try to remove the element first or, if something goes wrong, detach first then remove in the end. I think it depends on what logic is implemented around aciTree in the success callback ...


Can't run verification scripts: 'ptr_fun<int, int>' is deprecated

I tried to install OpenVino based on the official guide for macOS Mojave (with Xcode 11 beta 3). When I would like to run the verification scripts/sample applications I encounter an error: 'ptr_fun<int, int>' is deprecated. Here is the full message:
-- Generating done
-- Build files have been written to: /var/root/inference_engine_samples_build
[ 10%] Built target format_reader
[ 20%] Built target gflags_nothreads_static
[ 96%] Built target ie_cpu_extension
[100%] Building CXX object classification_sample/CMakeFiles/classification_sample.dir/main.cpp.o
In file included from /opt/intel/openvino_2019.1.144/deployment_tools/inference_engine/samples/classification_sample/main.cpp:16:
/opt/intel/openvino_2019.1.144/deployment_tools/inference_engine/samples/common/samples/common.hpp:64:72: error:
      'ptr_fun<int, int>' is deprecated [-Werror,-Wdeprecated-declarations]
    s.erase(s.begin(), std::find_if(s.begin(), s.end(), std::not1(std::ptr_f...
                                                                       ^
/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/functional:1097:1: note:
      'ptr_fun<int, int>' has been explicitly marked deprecated here
_LIBCPP_DEPRECATED_IN_CXX11 inline _LIBCPP_INLINE_VISIBILITY
^
/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/__config:1101:39: note:
      expanded from macro '_LIBCPP_DEPRECATED_IN_CXX11'
#  define _LIBCPP_DEPRECATED_IN_CXX11 _LIBCPP_DEPRECATED
                                      ^
/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/__config:1090:48: note:
      expanded from macro '_LIBCPP_DEPRECATED'
#    define _LIBCPP_DEPRECATED __attribute__ ((deprecated))
                                               ^
In file included from /opt/intel/openvino_2019.1.144/deployment_tools/inference_engine/samples/classification_sample/main.cpp:16:
/opt/intel/openvino_2019.1.144/deployment_tools/inference_engine/samples/common/samples/common.hpp:65:63: error:
      'ptr_fun<int, int>' is deprecated [-Werror,-Wdeprecated-declarations]
    s.erase(std::find_if(s.rbegin(), s.rend(), std::not1(std::ptr_fun<in...
                                                              ^
/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/functional:1097:1: note:
      'ptr_fun<int, int>' has been explicitly marked deprecated here
_LIBCPP_DEPRECATED_IN_CXX11 inline _LIBCPP_INLINE_VISIBILITY
^
/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/__config:1101:39: note:
      expanded from macro '_LIBCPP_DEPRECATED_IN_CXX11'
#  define _LIBCPP_DEPRECATED_IN_CXX11 _LIBCPP_DEPRECATED
                                      ^
/Applications/Xcode-beta.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/__config:1090:48: note:
      expanded from macro '_LIBCPP_DEPRECATED'
#    define _LIBCPP_DEPRECATED __attribute__ ((deprecated))
                                               ^
2 errors generated.
make[3]: *** [classification_sample/CMakeFiles/classification_sample.dir/main.cpp.o] Error 1
make[2]: *** [classification_sample/CMakeFiles/classification_sample.dir/all] Error 2
make[1]: *** [classification_sample/CMakeFiles/classification_sample.dir/rule] Error 2
make: *** [classification_sample] Error 2
Error on or near line 230; exiting with status 1
Horvaths-MacBook-Pro:demo root#

Is there any known solution for this?

Hi @Hordon13, we do not checked compilation with Xcode beta versions. So errors may occurred.
Particularly Xcode 11 have changed Apple Clang. Specific item from Xcode Release Notes:

Deprecation warnings will be issued when standard library facilities that were deprecated in the active Standard version are used. (46881474)

To eliminate this error you have to remove all inclusion of std::ptr_fun from sample code.
samples/common/samples/common.hpp:
inline std::string &trim(std::string &s) {
-    s.erase(s.begin(), std::find_if(s.begin(), s.end(), std::not1(std::ptr_fun<int, int>(std::isspace))));
-    s.erase(std::find_if(s.rbegin(), s.rend(), std::not1(std::ptr_fun<int, int>(std::isspace))).base(), s.end());
+    s.erase(s.begin(), std::find_if(s.begin(), s.end(), [](int c) { return !std::isspace(c); }));
+    s.erase(std::find_if(s.rbegin(), s.rend(), [](int c) { return !std::isspace(c); }).base(), s.end());
     return s;
 }

Also you may encounter with error: explicitly defaulted move assignment operator is implicitly deleted [-Werror,-Wdefaulted-function-deleted]. To avoid this you may just delete from ie_layers.h all lines like XXXLayer& operator = (XXXLayer &&) = default; or switch off "-werror" compilation flag.
Official support of Xcode 11 will be added once it will be publicly released.

Hi @AlexPeskov, thank you very much for the detailed answer! I got a better understanding now about the problem, I should suspect that the problem is with the Xcode beta, thanks again!

Hello,
Please am facing exactly the same issue. What is the solution to this?


Fix doc comments for CDK resource methods

There are still lots of TODO comments.
cc @mitchdenny

I think I have these now.


SSH transport should support other use cases (validate fingerprints, keys, agents, etc.) for security

Is your feature request related to a problem? Please describe.
The SSH transport doesn't check fingerprints and other non-trivial SSH use cases.
Describe the solution you'd like
The toolkit should support other use cases (validate fingerprints, keys, agents, etc.) for security. Being able to trust a fingerprint would make attackers' lives harder, for instance.
Describe alternatives you've considered
The toolkit gets passed an object with state and handlers defined already, like the Python toolkit needs a Paramiko session object instead of making one for you.
Additional context
I'm developing the SSH feature for the PHP toolkit, so it would be wise to have both parity and security features be consistent.

Describe alternatives you've considered
The toolkit gets passed an object with state and handlers defined already, like the Python toolkit needs a Paramiko session object instead of making one for you.

PR #314 adds support for passing an existing odbc and idb-connector objects for those transports. Being able to pass in a ssh client object instead of creating one for the ssh transport makes sense too.

Describe alternatives you've considered
The toolkit gets passed an object with state and handlers defined already, like the Python toolkit needs a Paramiko session object instead of making one for you.

PR #314 adds support for passing an existing odbc and idb-connector objects for those transports. Being able to pass in a ssh client object instead of creating one for the ssh transport makes sense too.

Found some  ssh2 connect config options that would help verify the host key and pass in an ssh agent. In theory this should work with the existing code just need pass this along these confis options on the Connection.transport options object. Still need to test these options out to confirm things are working properly.
hostHash - string - Any valid hash algorithm supported by node. The host's key is hashed using this
algorithm and passed to the hostVerifier function as a hex string. Default: (none)

hostVerifier - function - Function with parameters (hashedKey[, callback]) where hashedKey is a string
hex hash of the host's key for verification purposes. Return true to continue with the handshake or false
to reject and disconnect, or call callback() with true or false if you need to perform asynchronous
verification. Default: (auto-accept if hostVerifier is not set)

...

agent - string - Path to ssh-agent's UNIX socket for ssh-agent-based user authentication.
Windows users: set to 'pageant' for authenticating with Pageant or (actual) path to a cygwin
"UNIX socket." Default: (none)

agentForward - boolean - Set to true to use OpenSSH agent forwarding<EMAIL_ADDRESS>for the life of the connection. agent must also be set to use this feature. Default: false

Found some  ssh2 connect config options that would help verify the host key and pass in an ssh agent. In theory this should work with the existing code just need pass this along these confis options on the Connection.transport options object. Still need to test these options out to confirm things are working properly.
hostHash - string - Any valid hash algorithm supported by node. The host's key is hashed using this
algorithm and passed to the hostVerifier function as a hex string. Default: (none)

hostVerifier - function - Function with parameters (hashedKey[, callback]) where hashedKey is a string
hex hash of the host's key for verification purposes. Return true to continue with the handshake or false
to reject and disconnect, or call callback() with true or false if you need to perform asynchronous
verification. Default: (auto-accept if hostVerifier is not set)

...

agent - string - Path to ssh-agent's UNIX socket for ssh-agent-based user authentication.
Windows users: set to 'pageant' for authenticating with Pageant or (actual) path to a cygwin
"UNIX socket." Default: (none)

agentForward - boolean - Set to true to use OpenSSH agent forwarding<EMAIL_ADDRESS>for the life of the connection. agent must also be set to use this feature. Default: false

Example of host verification:
https://github.com/mscdex/ssh2/issues/985#issuecomment-806266569
https://github.com/mscdex/ssh2/issues/985#issuecomment-809583839


GROOVY-8220: SC GroovyCastException on parameter flow typing

GROOVY-8157 introduced flow typing for parameters and this fix is
required in order to track their assignments in if branches for
temporary type assignments.

Interesting solution. I also did something similar but then backed it out and had a parallel ifElseForWhileAssignmentTrackerParams. But your approach is simpler. I'll have a think about whether I see any cases that might be handled differently.

I will probably merge this PR in preference to mine (PR#606). Just wondering whether wrapping is needed in the case of a closure shared variable? I haven't had time to think it through myself yet.


API design

Sketching out an API, we can change this along the way:
model to ml-enabler

[x] POST /ml-enabler/register/<model_name> - may not implement now, but should have this endpoint to register models
[x] POST /ml-enabler/<model_name>/ with a payload of JSON tile data

ml-enabler to TM

[x] GET /ml-enabler/models/ - fetch all available models
[x] GET /ml-enabler/<model_name>/predictions/bbox - get aggregated prediction for the bbox
[x] POST /ml-enabler/<model_name>/predictions/project - with the TM project geojson as the payload will return aggregated prediction per task

cc @wronk @batpad @xamanu

Thanks, @geohacker. Makes a lot of sense, for the first API definition. What do you think @fitoria and @JorgeMartinezG?

All the above endpoints are done -  https://github.com/hotosm/ml-enabler/blob/master/ml_enabler/init.py#L38-L44


Unable to do codegen from devnet addresses

I assume this would be for both the tool and the source generator?
Thinking of a network switch that defaults to mainnet and accepts both dev and test net.

I also assume there is no need for custom RPC nodes?

I assume this would be for both the tool and the source generator?
Thinking of a network switch that defaults to mainnet and accepts both dev and test net.

Yes, this seems like the best option. Don't think custom could be needed since it's only used to get the IDL

I've added this to the tool. But realised that its impossible to the source generator with the current schema, and changing it is not trivial.
Would just adding this to the tool be okay?

I've added this to the tool. But realised that its impossible to the source generator with the current schema, and changing it is not trivial.
Would just adding this to the tool be okay?

Yes, for now this would suffice.

Another possible idea: to have the tool being able to output the IDL? This would considerably speed up some code generation if you use it from the .json file after downloading it instead of from the address

This also seems like a great idea!

Closed by #7


Aliasing modules?

I'm using module feature and I think it's great. One limitation that I've encountered is that modules cannot be aliased but it can be really useful in some scenarios.
mod frontend
mod backend

alias f := frontend

gives the following error
error: Alias `f` has an unknown target `frontend`
 ——▶ Justfile:4:7
  │
4 │ alias f := frontend
  │       ^

Not sure if it's planned or not.

I think this would be reasonable. It might be pretty rough in terms of implementation, but I definitely see the value of the feature.


Alpha release

Alpha version ships with:

all the requirements in the Wiki’s Roadmap for this version;
a new architecture (now customizable, usable through npm);
build process;
regressive CSS tests (visual);
customizable flags;
source maps;
i18n (implem, docs, samples);
a11y baseline (implem, docs);
recommendations for fonts;
default styles for RTL and CJK scripts;
various styles improvements;
various bug fixes;
EPUB compatibility doc;
various corrections in docs;
fonts for dyslexia.

Quick question for you, @llemeurfr : how do we version this one? Like Readium Desktop (alpha-1.0.0) or do we stick to 0.8.0? Maybe you want to keep consistency.
I wouldn’t necessarily be comfortable with the former since build and tests are still rough around the edges, comments are not removed in the built stylesheets yet, it would not clearly be representative of the current experimental status of vertical-writing, etc., so the latter makes most sense to me but I don’t have a strong opinion about that.
Also, I guess that versions can be pushed to master relatively quickly after being pushed to develop from now on, which is quite a milestone.

Don’t merge until issue #31 is fixed.

Re the version, the semantic versioning we have decided to use for iOS/Android/Desktop code would lead to 1.0.0-alpha.1.
I don't see any issue adopting that numbering for Readium CSS, alpha meaning "rough around the edges".

OK thanks let’s use this then. I’ll commit the version change on the develop branch.

As far as I’m concerned, this is good to go as an alpha – even caught a late issue related to table in safeguards.
I’ll let you review, especially the “admin” stuff (templates in .github folder, readme, package.json, etc.) before merging, @llemeurfr, if it’s ok for you. I’ll also get back to you tomorrow to deal with the communication around this release (updated article, etc.).
From now on, we’ll be able to use a “beta release” milestone, and deal with more specific issues.

The templates in .github are fine. The readme has been properly updated (esp. section Testing).  I can't judge the versioning contraints in package.json.

Ok thanks!
I’ll get back to you by mail for the blogpost announcing the release, which has been updated + I need to check if links are working as expected since I anticipated their url in the draft. Not urgent though, it can wait one day or two.


Problem with stepping

Hi,
i found that the stepping option does not work as it should, or it could work better :).
I want to use it with two dayparts , the first with full hours and the second with full hours but starting from half hours, i explain.
Some reservation can start at 10:00, 11:00 12:00 and so on for a period of 60 min each, so i m using the options stepping with a value of 60 minutes wich from what I understood it give the possibility to go from 10:00 to 11:00 etc ... so here its working perfect.
The problem i have is for the period wich must start from 10:30 to 11:30, 11:30 to 12:30 etc, on those dayparts i tried with the stepping option of 60 minutes with a start date of [date] 00:30 or [date] 09:30 and it does not work.
The script start to 10:00 every time i put a stepping of 60 minutes it  doesnt want to work with a stepping of 60 min starting from hal hours .
Did i missed something ?

Hi, Any idea of getting this option work as it should ?

I have the same issue with smaller values: step 15 mins starting at 10 mins will not work; start time must be multiple of step, in my case: 0, 15, 30, 45 will work.


Does anyone have alternative pretrained model?

Hi, I hope to reproduce the results with 36 features.
However, the link for the alternative pretrained model is gone.
Can anyone share the 'resnet101_faster_rcnn_final_iter_320000.caffemodel' ?
I would really appreciate your help.
Thank you!

.


Added CORS Library and modularized route handling using controllers

Added CORS from npm to avoid writing middleware for all the requests
Modularized route handling function as a class

Stopped with the Post routes, let me know if I'm heading in the right direction to continue

@psnmissaka Friendly ping 😄

Will close this pull request and create the issue


Location: Unrecognized location type near

There are several SciStarter opps with locations showing as "Unrecognized location type near"
E.g. https://beta.sciencenearme.org/plastic-pirates-go-europe-1
https://beta.sciencenearme.org/rhs-cellar-slug-survey
https://beta.sciencenearme.org/monitoring-at-water-and-feed-stations-post-bushfire-1

Fixed.


VmaQuickSort is never used

VmaQuickSort is only defined if the macro VMA_SORT hasn't been, so that it can be used for implementing it:
https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator/blob/5ab8c1752ab9720b7a38a3bc72ebcf17ef3ce6f1/include/vk_mem_alloc.h#L3415
However, if not defined by the user, VMA_SORT will currently always get defined to std::sort as this comes before VmaQuickSort:
https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator/blob/5ab8c1752ab9720b7a38a3bc72ebcf17ef3ce6f1/include/vk_mem_alloc.h#L2799-L2801
While VmaQuickSort (and VmaQuickSortPartition) could be deleted because of this, the other alternative would be to remove the use of std::sort. Then, as far as I can see, <algorithm> would only be used for std::min and std::max, which makes it appealing to delete this header include altogether.

Also, it looks like <utility> is only included for std::swap and std::forward.
Since this is also a big header, I believe it would be nice to replace it with a custom VmaSwap implementation (if VMA_SWAP isn't user-defined) and a new VMA_FORWARD macro following this: https://www.foonathan.net/2020/09/move-forward/

Thank you for this suggestion. I agree that VmaQuickSort and VmaQuickSortPartition are unused and can be removed, which I just did. User of the library can provide his own implementation of VMA_SORT, but when he doesn't, the default is to use std::sort. Custom sorting algorithm was a legacy of some old code.
I don't want to reimplement custom sorting, std::forward, and other basic facilities just to remove #include of some standard library header and save some milliseconds of compilation time. I know this is a common approach among C++ programmers, but I think the standard library is carefully designed and well tested, so I would rather rely on it than implement everything on my own.


Change model for messages with multiple image previews to stack images horizontally with scrollbar

I'm not 100% sure this will be good UX, but it might be (and solves several problems), and we won't really know until we try.
Zulip's model for image previews is designed around a few goals:

Not having images be so giant that they crowd out text (which is very common in other chat platforms with full-width image elements -- a square image can take ~50% of the screen)
Avoiding the message feed jumping around randomly as images load (which requires that we can communicate to the browser via just HTML+CSS how much height the image element will consume before it is loaded).  This can be quite problematic, since for scrolling convenience, we display in the DOM hundreds of messages of scrollback at a time, so it's very possible for a message well outside of view with 5 images to result in jumping scroll position.
Avoiding creating extremely tall Zulip messages, which are not a great UX for keyboard users (one can scroll down partway through a message with e.g. Space, but overall if a message is multiple screens tall, it's a bit annoying).

The current design related to this is as follows:

Image previews are 100px in height in CSS (And then can be whatever width that scales them to) -- this achieves the anti-crowding goal
We show at most one image preview per line (this avoids the jumping -- since the browser doesn't need to know how the width of the images relates to the width of the message area and thus whether we'll end up wrapping to multiple lines).
We show at most 5 image previews per message, to avoid creating extremely tall messages and also avoid unintended expensive behavior for rare automated bot messages (E.g. email gateway, etc.) that might have 100 links to images in them.

We've discussed options around this many times, but a new solution came up here: https://chat.zulip.org/#narrow/stream/92-learning/topic/Does.20using.20more.20than.205.20links.20remove.20image.20preview.3F/near/1148468
Specifically, we could just have all inline image previews going to the end of a message (or paragraph) be placed in a horizontal row together, with a horizontal scrollbar setup so that the browser knows that area will be 100px in height regardless of how many images we have there (avoiding the jumping bug).
And then we could raise the 5 to something like 10-15 (with maybe also a "show the first N" behavior, rather than "If more than N, show none") easily enough.
We'd need mobile responsive CSS to avoid an enormous horizontal scrolling region (for mobile web it should just be a vertical stack).
We can try doing this with just CSS, but it seems likely to me that we'll need to adjust the HTML markup to create a div to be the scrolling container for multiple images.  I suppose we could try doing any HTML transformation in rendered_markdown.js rather than having the markdown processor generate the outer div; that might be better for prototyping since it doesn't adjust the database layout and would safer to test-deploy on chat.zulip.org to play with.
https://zulip.readthedocs.io/en/latest/subsystems/markdown.html has useful context.

@zulipbot claim

We implemented an alternative version of this as https://github.com/zulip/zulip/pull/21068 in the 5.0 release.


More false positive for unused imports

package com.acme

import com.acme.cathedral.TheDome   // here
import com.acme.bazar.SomeException // and here

/**
 * Do something.
 * @throws SomeException when ...
 * @see TheDome.someMethod
 */
fun doSomething() {
    TODO()
}

Looks related to #201

Thanks for the reporting!
Will look at this issues asap.

There seems to be 2 issues here:
First: Detekt doesn't account for Tags in KDoc at the moment. I'm looking into a fix right now.
Second: If you reference classes from KDoc you should put them in square brackets for detekt to pick them up. (see also: https://kotlinlang.org/docs/reference/kotlin-doc.html#linking-to-elements)
/**
 * Do something.
 * @throws [SomeException] when ...
 * @see [TheDome.someMethod]
 */

I put #458 together which should solve the issue. However you would still need to correctly link to the classes with [SomeException]. I took over the code from this issue as a test case for the rule with the added square braces.
https://github.com/arturbosch/detekt/pull/458/files#diff-80fb3301450b6c6c9842db6565cef77dR123

Wow that was fast, thanks!
Right, this made me realize that the square braces are required for dokka to generate proper links.
That's perfect!


IBLs use clamped HDR?

The glTF Sample Viewer seems to have removed higher exposures from its packaged IBLs. Could this be a bug?

https://github.khronos.org/glTF-Sample-Viewer-Release/

Model = MetalRoughSpheres

Active Environment = Studio Neutral

Open a second window, and load another instance of the sample viewer

Model = MetalRoughSpheres

Drag-and-drop the Neutral.HDR (https://github.com/KhronosGroup/3DC-Certification/blob/main/models/Neutral.hdr)

Open a third window, and load the Babylon.js Sandbox (https://sandbox.babylonjs.com/)

Drag-and-drop the MetalRoughSpheres.glb (https://github.com/KhronosGroup/glTF-Sample-Models/blob/master/2.0/MetalRoughSpheres/glTF-Binary/MetalRoughSpheres.glb)

Drag-and-drop the Neutral.HDR (https://github.com/KhronosGroup/3DC-Certification/blob/main/models/Neutral.hdr)

Open a fourth window, and load the Enterprise PBR Sample Renderer (https://dassaultsystemes-technology.github.io/dspbr-pt/)

Drag-and-drop the MetalRoughSpheres.glb (https://github.com/KhronosGroup/glTF-Sample-Models/blob/master/2.0/MetalRoughSpheres/glTF-Binary/MetalRoughSpheres.glb)

Drag-and-drop the Neutral.HDR (https://github.com/KhronosGroup/3DC-Certification/blob/main/models/Neutral.hdr)

Compare the four results.

The included IBL for Studio Neutral has a lower dynamic range than the original HDR when manually loaded, or when loaded into other renderers.

Moved this to the main repository, here: https://github.com/KhronosGroup/glTF-Sample-Viewer/issues/349


Cannot build out of the box

When I clone and try to build the project (either in Visual Studio or via the build.cmd) the build fails with errors that the class PackageExplorer.Resources is missing.
To solve it I had to right click Resources.resx, click Properties and here set Custom Tool namespace to  "PackageExplorer.Resources"
I guess you forgot to check this setting in to the .csproj file?

All of the files are checked in. What version of VS are you using? Do you have automatic package restore enabled? The resx files get set by targets that come down during restore.

As far as I can tell, this was caused by https://github.com/NuGetPackageExplorer/NuGetPackageExplorer/commit/6531156bbb8f2cfeb6331db4e09d161020e33506. I have just created https://github.com/NuGetPackageExplorer/NuGetPackageExplorer/pull/360, which should fix this.

Yep, setting CustomToolNamespace in the csproj file fixes it.


Contribution of the channels to the transition events (magnitudes) are now computed on the maximum probability at each trial

Up to now the contribution of each channel to the transition event is computed as follows:
for comp in range(self.n_dims):
    magnitudes[event,comp] = np.mean(np.sum( \
        eventprobs[:,:,event]*self.data_matrix[:,:,comp], axis=0))
# Scale cross-correlation with likelihood of the transition
# sum by-trial these scaled activation for each transition events
# average across trials

But averaging the probabilities in this way :

slows down the estimation process as the mean position is more influenced by the gamma distribution
forces us to create a new data representation (see Issue #62 )
Based on simulations seems to actually misestimate the event timings (see last point)

A new proposition is to base the computation of the contributions to the max probabilities
if self.em_method == "max":
    #Take time point at maximum p() for each trial
    #Average channel activity at those points
    event_values = np.zeros((self.n_trials, self.n_dims))
    for trial in range(self.n_trials):
        time = self.starts[trial]+ np.argmax(eventprobs[:, trial, event])
        event_values[trial] = self.events[time]
    magnitudes[event] = np.mean(event_values, axis=0)

Latest commits in Branch #Improving-EM-estimation by default uses the max method as it seems the best both in terms of correctness and speed:

This also has as a consequence that backward_estimation() yields more coherent event repartition, example on tutorials 3 data:
with the max method:

With the mean method


Run all microbenchmarks

Two things:
Re-land: Uninstall microbenchmarks before running them.
Flakes in #153828 stem from adb saying the app isn't installed, but then failing to install wtih -r. Several other tests uninstall the app before trying to run it. Previous fix called uninstall between tests, but iOS takes 12 to 13 seconds to perform uninstall / install, which timed out the test.  Just uninstall the one time since we only care about any lingering apps with different keys.
Potential solution #153828
Make things go fast
Instead of installing 21 different compilations of the same app to get results; compile and run them together. Locally on Mac+iOS, this should takes ~3 minutes instead of ~15 minutes.

> time ../../bin/cache/dart-sdk/bin/dart bin/test_runner.dart test -t microbenchmarks_ios -d <my device>

  25.68s user 9.63s system 17% cpu 3:19.04 total

Does this README need an update? https://github.com/flutter/flutter/blob/master/dev/benchmarks/microbenchmarks/README.md
It'll be interesting to see whether there are shifts in the metrics in the benchmarks that come later in the sequence. That would indicate that we still have some post-install or first-run warmup happening somewhere.

Does this README need an update? https://github.com/flutter/flutter/blob/master/dev/benchmarks/microbenchmarks/README.md
It'll be interesting to see whether there are shifts in the metrics in the benchmarks that come later in the sequence. That would indicate that we still have some post-install or first-run warmup happening somewhere.

I'll update it.  I ran the benchmarks against a mokey and an iphone se and they were comperable to skia perf numbers.

Does this README need an update? https://github.com/flutter/flutter/blob/master/dev/benchmarks/microbenchmarks/README.md
It'll be interesting to see whether there are shifts in the metrics in the benchmarks that come later in the sequence. That would indicate that we still have some post-install or first-run warmup happening somewhere.

Done.

Reason for revert: A different benchmark was using the microbenchmark parser and timing out.


Data in model needs to be derived into separate stores

This should actually continue down into a specific cell so the whole table doesn't need to update, just the effected regions.

I need to redesign the interface around that to return row stores (a simple projective operator on the full data set used to created a derived store that only fires an even if it has changed compared to the previous value when an event from the parent store fires) and cell stores that are then used only run the synthetic field derivations if there was actually a change.


Remove outdated warning when using screen texture in GL Compatibility renderer

This fixes an unreported bug. But everytime a user uses the screen_texture they would get this warning which is not accurate anymore since we added screen support early in the 4.1 dev cycle.
Since https://github.com/godotengine/godot/pull/72361 was cherrypicked, we should cherrypick this as well

Thanks!


Power Switch breaks the render

I had an issue with a blueprint book not rendering, after narrowing it down to which blueprint was causing the issue, I went through and found a two item combo that breaks rendering.
A powerswitch, with a substation, connected via a copper wire.
Example:
https://teoxoy.github.io/factorio-blueprint-editor/?source=https://pastebin.com/LbdUgad8
Image of the blueprint in the pastebin link above is attached.
The concrete is not in the blueprint.

Error:
Array(3) [ {…}, {…}, {…} ] main.js:43:4643
[
{
"keyword": "additionalProperties",
"dataPath": ".blueprint.entities[0].connections",
"schemaPath": "#/properties/entities/items/properties/connections/additionalProperties",
"params": {
"additionalProperty": "Cu1"
},
"message": "should NOT have additional properties"
},
{
"keyword": "additionalProperties",
"dataPath": "",
"schemaPath": "#/oneOf/1/additionalProperties",
"params": {
"additionalProperty": "blueprint"
},
"message": "should NOT have additional properties"
},
{
"keyword": "oneOf",
"dataPath": "",
"schemaPath": "#/oneOf",
"params": {
"passingSchemas": null
},
"message": "should match exactly one schema in oneOf"
}
]

Sorry for the late response. I didn't get an email notification for your issue for some reason.
I see what's causing the issue. I never tested that configuration before, it should be an easy fix tho.
Thanks for the report.


istioctl: use configmap for inject configuration, if present

If the inject config file arg is not used, check for a configmap to
use for the sidecar inject configuration.  Adds a new param for the
inject configmap name (default 'istio-inject').

Removing the default configmap (istio-inject) and making the feature opt-in would seem best.

I'll submit with opt-in changes and will return err if user sets it and there's a failure (missing or invalid configmap).
RE: the overriding the hub, tag, etc options, I can add a help comment.  Should the same comment be added to the --injectConfigFile help since the configmap option has the same behavior with respect to that (overriding any other inject config --hub, etc)?

A comment on precedence would be good, i.e. --injectConfigFile and --injectConfigMapName before flags/params. You could also throw an error if user specified file and configmap name at the same time, similar to the mesh file and configmap name.

@tiswanso: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
ef6f558c31aecbd030d76a529fb8d3c8a5520a7b
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tiswanso: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
fccc1e58bd99591d592f9ef136763d321b1a0e58
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tiswanso: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@tiswanso: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tiswanso: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test istio-pilot-e2e

prow/istio-unit-tests.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test istio-unit-tests

prow/e2e-smoke.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test e2e-smoke

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@tiswanso: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test e2e-smoke

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@tiswanso: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test e2e-smoke

prow/istio-pilot-e2e.sh
7b4deab5135248ef911b5c875b9caaffab76537c
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Anything left to do on this review?  @ayj -- looks like the PR system still thinks your requested changes are unresolved.
Also, with the e2e fails do I need to rebase or should I just keep mashing recheck?

Can you try rebasing?

@tiswanso: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
a4eff732780b32ed7d11cc059100558a7228c935
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/ok-to-test

/retest

* (auth infra) routing-rules run 0 routing all traffic to c-v1 with shadow policy attempt 0 expected 100 requests (+/-5) to reach v1 => Got 51
* (auth infra) routing-rules run 0 routing all traffic to c-v1 with shadow policy attempt 0 expected 0 requests (+/-5) to reach v2 => Got 44
* (auth infra) routing-rules run 0 routing all traffic to c-v1 with shadow policy attempt 1 expected 100 requests (+/-5) to reach v1 => Got 50
* (auth infra) routing-rules run 0 routing all traffic to c-v1 with shadow policy attempt 1 expected 0 requests (+/-5) to reach v2 => Got 50
* (auth infra) routing-rules run 0 routing all traffic to c-v1 with shadow policy attempt 2 expected 100 requests (+/-5) to reach v1 => Got 50
* (auth infra) routing-rules run 0 routing all traffic to c-v1 with shadow policy attempt 2 expected 0 requests (+/-5) to reach v2 => Got 50

Likely fixed by https://github.com/istio/istio/pull/3442. I'm not sure if Circle auto-rebases, in which case you might need to do that yourself.

thanks @ayj  -- will rebase manually

@tiswanso: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
a4eff732780b32ed7d11cc059100558a7228c935
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[FEATURE] Adding a Workflow for greetings to new issue and PRs

This workflow runs whenever a new issue or pull request is opened.
Greet User: It uses the actions-ecosystem/action-create-comment action to post a customized comment directly on the issue or pull request.
Personalized Message: The message greets the user by username and dynamically adjusts based on whether it’s an issue or pull request. It also includes a link to the contribution guidelines for new contributors.
Benefits of the Greeting Workflow

Improves User Experience
Encourages Quality Contributions
Reduces Common Questions

Hello @yashksaini-coder please add gssoc ext and level labels

Hello @yashksaini-coder please add gssoc ext and level labels

this project is not in gssoc-ext


Changelog window opens too small

Pretty self explanatory with the title.

I guess, such size will be enough.

Looks great to me.
On Oct 30, 2017 5:24 AM, "MasterPetrik"<EMAIL_ADDRESS>wrote:

[image: 2017-10-30_090233 win7 mastercore]
https://user-images.githubusercontent.com/22294259/32163313-fa0fb258-bd6c-11e7-8e94-1b89fa46e470.png
I guess, such size will be enough.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Bionus/imgbrd-grabber/issues/1126#issuecomment-340387866,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS2t8J5mwcTkSARFpGrsg2Py9UAZsDE6ks5sxZXLgaJpZM4QKgde
.

@Bionus is this normal, that when I'm on nightly version, I get update message, but when I try to update it sends me to version download page on Github in browser instead of autoupdate?

Yes, nightly doesn't update itself for the moment.


Button (and Other Components?) Aren't Exported

Button, and probably other components, are missing from your NPM package.

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

Current Behavior 😯
import { Button } from 'material-ui';

... doesn't work.
Expected Behavior 🤔
import { Button } from 'material-ui';

... should work.
The current list of components which are exported are:

'AppBar',               'AutoComplete',     'Avatar',
'Badge',                'BottomNavigation', 'BottomNavigationItem',
'Card',                 'CardActions',      'CardHeader',
'CardMedia',            'CardTitle',        'CardText',
'Checkbox',             'Chip',             'CircularProgress',
'DatePicker',           'Dialog',           'Divider',
'Drawer',               'DropDownMenu',     'FlatButton',
'FloatingActionButton', 'FontIcon',         'GridList',
'GridTile',             'IconButton',       'IconMenu',
'LinearProgress',       'List',             'ListItem',
'makeSelectable',       'Menu',             'MenuItem',
'MuiThemeProvider',     'Paper',            'Popover',
'RadioButton',          'RadioButtonGroup', 'RaisedButton',
'RefreshIndicator',     'SelectField',      'Slider',
'Subheader',            'SvgIcon',          'Step',
'StepButton',           'StepContent',      'StepLabel',
'Stepper',              'Snackbar',         'Tabs',
'Tab',                  'Table',            'TableBody',
'TableFooter',          'TableHeader',      'TableHeaderColumn',
'TableRow',             'TableRowColumn',   'TextField',
'TimePicker',           'Toggle',           'Toolbar',
'ToolbarGroup',         'ToolbarSeparator', 'ToolbarTitle'

Steps to Reproduce 🕹
Steps:

npmi material-ui
node
Object.keys(require('material-ui'))
4.Button (and possibly others) are missing

Context 🔦
The normal way Node programmers access components is by importing them.  Node has a standard package format where packages expose their values via exports, and while this package does use that system for most of its components, at least one (Button) is missing.
Your Environment 🌎

Tech
Version

Material-UI
^0.20.2

React
^16.9.0

Browser
Chrome

TypeScript
ewww

There is no Button Component in v0. You can use FlatButton, RaisedButton, FloatingActionButton, or Icon Button.
https://v0.material-ui.com/#/components/flat-button

I don't know what "v0" is, but Button is on the front page of your site!
  import React from 'react';
  import Button from '@material-ui/core/Button';

  const App = () => (
    <Button variant="contained" color="primary">
      Hello World
    </Button>
  );

@machineghost We don't support 0.20.2 anymore.

But it's on the front of your site!?! Maybe take it down then?

@machineghost Don't worry, hopefully, it will make sense.

@machineghost The install instructions are also on the homepage.

I think at some point you have to ask, "should I continue arguing with a total Internet stranger who has volunteered to tell me something I may not have realized about my library and use that feedback in a productive way (say by making your front page clearer ...)" or ... "should I keep arguing with an Internet stranger?"
Personally I picked another UI framework, one that can handle the incredible complexities of both using NPM properly and showing how to use the library on the front page of the site.  You do what you want to do with your library.

@machineghost Could you share a screenshot of this page of the documentation, Thanks? Here is mine:

@machineghost See https://material-ui.com/api/button/#import. You need to install https://www.npmjs.com/package/@material-ui/core, not https://www.npmjs.com/package/material-ui

If you move one panel to the left on the homepage, you'll find the install instructions. You'll need to uninstall v0 first.

Thanks, but again I moved on to a framework that only had one (straightforward) package to install.
If you care about other users like me, perhaps you should be more clear that the package "material-ui" has nothing to do with material-ui.com on that front page.

At this point I can only assume you're trolling.

At this point I can only assume you're trolling.

😆
@material-ui/core is the name of one package. @material-ui is an organization scope name.


Check current exchange price

Check on start, and recheck every 15 minutes
From here https://www.cryptocompare.com/api/#-api-data-price- (we'll need to put a link to CryptoCompare on about page, going to be a separate issue)

oh... this is current pr #120 -- i can modify to use that API if you prefer

great! that works too


Access control for custom endpoints

Documentation Issue
We have the following setup: We have one auth enabled collection "User". All other collections are not auth enabled, but we restrict access to the auto-generated CRUD endpoints via the access: { read: .., create: ...} configuration where we provide functions which check if the user is authenticated for the "User" collection.
Is it possible to somehow limit the access for custom endpoints in collections which are not auth enabled?
The documentation lacks information about this: https://payloadcms.com/docs/rest-api/overview#custom-endpoints.
The request object that was provided to the custom endpoint handler contained no authenticated user object. I assume the user was missing, because Passport would not kick in here to set the user on the request, when the endpoint is configured to not require authentication.
Additional Details
In case it is currently not possible to restrict access to custom endpoints the same way we can do this for the standard CRUD endpoints, this would be feature request.

Hey @bfoese — did you ever solve this? I believe Dan is correct in that you may not be supplying credentials: 'include' within your fetch request, as custom endpoints do indeed go through our auth middleware prior to executing your endpoint logic, so the user object should indeed be present on your req.
I will convert this to a discussion so that we can continue the conversation there!


404 getting started page

On the repo, clicking on this link its giving me 404 not found. I don't know where to start

Where did you find the link you posted above?
use: https://substrate.dev/docs/en/knowledgebase/getting-started/
And in case you are unsure, use https://substrate.dev/ 😀


Add a feature -> max_potency_ratio

I think that is a good way to reach people you
really want to follow.
In previous version, it would follow if user's ratio
les than min_previous_ratio. I.E. Follow = 100, Following = 10000
its ratio will be 0.01. I think following that user is not a good idea.
So basically, we will be able to control the scale that we really want.
It works properly for me.
If you accept this change, there should be a field in filter.json
"max_potency_ratio":
{
"skip_business": true,
"skip_non_business": false,
"min_followers": 30,
"max_followers": 1000,
"min_followings": 30,
"max_followings": 1000,
"min_potency_ratio": 0.3,
"max_potency_ratio": 1,
"follow_private_or_empty": true
}
this is my settings, by looking that, it wont follow a user which has
300 follower, 1000 followings -> ratio 0.3

Thanks! A very good idea!!
Can you please format that code using this: https://pypi.org/project/black/
It's very simple! You have to install it and then run from the command line "black nameoffile.py" or "black folder/" and you're done!
Thanks :)

start over :)


Feature-gate database connectors in Prisma Engines

Problem
Prisma Client Rust depends on a fork of the engines, which build every database connector by default. I tried implementing sql and mongodb features but it required so many changes that I decided it would be better to be done officially so that updating my fork doesn't introduce huge merge conflicts. Even better would be if there were individual features per database - mysql, postgresql, mongodb etc.
Suggested solution
Reorganise the Prisma engines code to allow for each database connector to be included with a Cargo feature.
Alternatives
I do it myself and have to deal with huge merge conflict resolution each time Prisma Client Rust wishes to update the engines version it uses.
Additional context
I'd expect that this would not be on the roadmap for a long time as it has little to no benefit for Prisma Client JS, but it would be greatly appreciated if this was even considered. I'd be willing to work on it with you guys if needed.

I just opened PR #3157 about separating the sql and monogdb features.

FWIW I have put support for each database behind a feature at Brendonovich/prisma-engines.
Compiling query-core with only the sqlite feature is now 30% faster.


Fix broken link to the Acceptable Formulae docs

This link died as this formula was deprecated, sorry and thanks anyway!


fix(db-postgres): nested localized arrays

Description
Fixes a bug with nested arrays within either localized blocks or arrays.

@jmikrut Would it be possible to backport this fix to v2?

Hey @Janisvb — well, this is only one of MANY Postgres fixes / improvements that we've made since 2.0, which have only been made on 3.0. If we wanted to cherry-pick this PR in specific, we could, but we'd also need to bring over this PR as well:
https://github.com/payloadcms/payload/pull/7900
That PR was the bulk of the work involved in fixing the issue. The PR we're commenting on was just follow-up fixes.
I think the better idea would be to update your project to 3.x beta honestly, but if you were interested in replicating this fix for 2.x, we would merge it!