HA System Log fills up with python crashes.

can you clean up the code so it silently just logs the error, and not clog up the system log with these types of errors.
2023-08-09 12:45:40.158 ERROR (SyncWorker_8) [icloud3] HTTPSConnectionPool(host='p27-fmipweb.icloud.com', port=443): Max retries exceeded with url: /fmipservice/client/web/refreshClient?clientBuildNumber=2021Project52&clientMasteringNumber=2021B29&ckjsBuildVersion=17DProjectDev77&clientId=0a7fd768-36a6-11ee-80cd-000c29e890fa&dsid=11318756 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f75fa22a610>: Failed to establish a new connection: [Errno -3] Try again'))
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/urllib3/connection.py", line 174, in _new_conn
conn = connection.create_connection(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/urllib3/util/connection.py", line 72, in create_connection
for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/socket.py", line 962, in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
socket.gaierror: [Errno -3] Try again
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/urllib3/connectionpool.py", line 714, in urlopen
httplib_response = self._make_request(
^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/urllib3/connectionpool.py", line 403, in _make_request
self._validate_conn(conn)
File "/usr/local/lib/python3.11/site-packages/urllib3/connectionpool.py", line 1053, in _validate_conn
conn.connect()
File "/usr/local/lib/python3.11/site-packages/urllib3/connection.py", line 363, in connect
self.sock = conn = self._new_conn()
^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/urllib3/connection.py", line 186, in _new_conn
raise NewConnectionError(
urllib3.exceptions.NewConnectionError: <urllib3.connection.HTTPSConnection object at 0x7f75fa22a610>: Failed to establish a new connection: [Errno -3] Try again
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/requests/adapters.py", line 486, in send
resp = conn.urlopen(
^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/urllib3/connectionpool.py", line 798, in urlopen
retries = retries.increment(
^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/urllib3/util/retry.py", line 592, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='p27-fmipweb.icloud.com', port=443): Max retries exceeded with url: /fmipservice/client/web/refreshClient?clientBuildNumber=2021Project52&clientMasteringNumber=2021B29&ckjsBuildVersion=17DProjectDev77&clientId=0a7fd768-36a6-11ee-80cd-000c29e890fa&dsid=11318756 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f75fa22a610>: Failed to establish a new connection: [Errno -3] Try again'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/config/custom_components/icloud3/support/pyicloud_ic3.py", line 152, in request
response = Session.request(self, method, url, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/requests/sessions.py", line 589, in request
resp = self.send(prep, **send_kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/requests/sessions.py", line 703, in send
r = adapter.send(request, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/site-packages/requests/adapters.py", line 519, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='p27-fmipweb.icloud.com', port=443): Max retries exceeded with url: /fmipservice/client/web/refreshClient?clientBuildNumber=2021Project52&clientMasteringNumber=2021B29&ckjsBuildVersion=17DProjectDev77&clientId=0a7fd768-36a6-11ee-80cd-000c29e890fa&dsid=11318756 (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7f75fa22a610>: Failed to establish a new connection: [Errno -3] Try again'))
2023-08-09 12:45:40.186 ERROR (SyncWorker_8) [icloud3] * iCloud Location Request Error > An error occured refreshing the iCloud Location Data. . • iCloud may be down, or. • iCloud Account needs to be reauthorized, or. • There may be an internet connection issue. iCloud3 will try to access the iCloud Account again later. (Failed to establish a new connection (Status Code -2))

Does the iCloud Location Request Error line (the last one in the above comment) occur a lot of times or just once?

Just emailed you the full log from HA.

sorry missed a log

On 9 Aug 2023, at 14:07, Gary Cobb @.***> wrote:
Does the iCloud Location Request Error line (the last one in the above comment) occur a lot of times or just once?
—
Reply to this email directly, view it on GitHub https://github.com/gcobb321/icloud3_v3/issues/179#issuecomment-1671295766, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAATA4AIFSR3P3V7TAM25ELXUODSVANCNFSM6AAAAAA3J66AAY.
You are receiving this because you authored the thread.

￼￼

but yes, i see it quite a lot recently.

found an older log that show more info
￼

On 9 Aug 2023, at 14:07, Gary Cobb @.***> wrote:
Does the iCloud Location Request Error line (the last one in the above comment) occur a lot of times or just once?
—
Reply to this email directly, view it on GitHub https://github.com/gcobb321/icloud3_v3/issues/179#issuecomment-1671295766, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAATA4AIFSR3P3V7TAM25ELXUODSVANCNFSM6AAAAAA3J66AAY.
You are receiving this because you authored the thread.

I made a few changes to the modules that handle communicating with Apple iCloud Web Servers so the detail error messages are not added to the log files. I have not tested them since they deal with internet connections issues.
Unzip the file below into the icloud3/support directory and restart HA. Let me know if just the iCloud3 message displays on the Event Log and the HA log without all the other stuff and I'll incorporate it into the final v3 release.
pyicloud_ic3-rc2.zip

thanks for the reply, i have loaded the files, I will keep you updated how it goes over the few hours.

hi,
ye this is much better, no more python crashes.
Thanks.
2023-08-10 07:56:45.446 ERROR (SyncWorker_5) [icloud3] andys_iphone_icloudv3 iCloud Location Request Error > An error occured refreshing the iCloud Location. iCloud may be down or there is an internet connection issue. iCloud3 will try again later. (cannot access local variable 'response' where it is not associated with a value)

Good


Dashbord service should not startup in metasrv and datanode

What type of bug is this?
Unexpected error
What subsystems are affected?
Datanode, Meta
What happened?
The #1239 embeds the dashboard service and listens on the :4000 port during the HTTP server start-up process.
It functions properly in standalone mode. However, in distributed mode, the frontend/metasrv/datanode services also start the dashboard service due to the common implementation of the HTTP server. If I run the distributed mode on the same host, the service fails to start due to the 4000 port being used by other components.
The dashboard service should be exported by the frontend and it's not necessary to be started by the metasrv and datanode services, which are not direct components used by the user.
What operating system did you use?
macOS
Relevant log output and stack trace
No response
How can we reproduce the bug?
Follow the instruction to deploy the cluster on the host by using the latest binary(v0.2.0-nightly-20230403), it will fail by the 4000 port is already used when starting datanode.

https://github.com/GreptimeTeam/greptimedb/pull/1256 adds http server to datanode, looks like we have to update the docs, to start datanode by --http-addr=xxx

@MichaelScofield The --http-addr can resolve the port confliction, but it seems the /dashboard endpoint is still working:
$ ./greptime metasrv start --store-addr <IP_ADDRESS>:2379 --server-addr <IP_ADDRESS>:3002 --http-addr <IP_ADDRESS>:14000

$ curl -XGET http://localhost:14000/dashboard/

IMO, we should:

Add the user configuration to determine whether the component should start with /dashboard;
When in distributed mode, the metasrv and datanode should disable /dashboard;

make sense, I will look into it


Enable SystemD support in xwayland-satellite

As the flake for xwayland-satellite does it, we might do it as well.

Hey, so it just so happens to be the case that i was actually working on this earlier today. I didn't push it then, because i didn't get a chance to test that the builds still succeeded. But i've tested it now, and my changes work fine.
This PR:

doesn't install the systemd user service
adds an unnecessary dependency on systemd
breaks the xwayland-satellite-stable package (see failed check above; this is because the systemd feature doesn't exist in the released version)

As such, i've just pushed the changes i already made earlier today, because they work fine and i've tested them.
https://github.com/sodiboo/niri-flake/commit/c8a91169d21f16a9eb046c25a7964348e9726bb6 supersedes this PR.

I'm really sorry for the not so great PR (I don't really write much packages), although SystemD was enabled so I'm happy anyway.

I'm really sorry for the not so great PR

Hey, it's okay! I would've been happy to review this and comment on what needs changes; it just so happens to be the case that I already made those changes prior to this PR being opened, so it's simply less effort to use that than to iterate on this one. You don't have to apologize for submitting a PR that wasn't merged; this isn't an inconvenience to me or anyone else.

although SystemD was enabled so I'm happy anyway.

yay!


Upgrade GitHub actions from Node 16 to Node 20

Node 16-based actions are EoL, we need to upgrade to Node 20-based (LTS) actions.

Some actions still need updating:

actions/configure-pages@v3
actions/upload-artifact@v3
actions/deploy-pages@v2

Pushed out to get the v3.4 features released quicker.


Preserve imageOpacity when assigning to Wheel.items

The imageOpacity property wasn't previously passed through when initializing Item objects, so it would be ignored.
One can work around this by setting imageOpacity manually, as in the following example, but it seems like this property is simply missing from the setter and should be included with the rest of the fields.
var wheel = new Wheel(container, { items })

for (const i in items) {
  wheel.items[i].imageOpacity = items[i].imageOpacity
}

Thanks @evhan!


Do not mess input urls order (fix #100)

Nope, not gonna work, same thing, non-deterministic order since forEach is parallel and is basically a race condition between the elements it returns.
We could maybe do this?
position = videoGuids(indexOf(guid));
metadata[position] = {
            date: date,
            totalChunks: totalChunks,
            title: title,
            playbackUrl: playbackUrl,
            posterImage: posterImage
        };

parallel? what you mean?
forEach runs sequentially as per docs

It is sequential but not ordered sequential. Just try it out you'll see.

i don't get it
https://jsfiddle.net/wx28bukL/
from what i found, forEach is deterministic and it runs in ascending order (index)
we have a numeric index, so i really don't see how this can return the array in a wrong order

You are missing an await. If you don't await a promise then indeed it is deterministic but try it awaiting something.

ok, the issue is await and the forEach callback

reusable fn, we may need it in future
original idea was to prototype Array, but it turns out this is much simpler and cleaner then typescript mess

LGTM. Ship it. :shipit:


Convert PHP Array Config to PHP ConfigBuilder

Since Symfony 5.3 there exist a new way of configuring packages. The config builders:
https://symfony.com/blog/new-in-symfony-5-3-config-builder-classes
It would be great as we are currently having done the step from yaml to php configs. To do the next step from PHP array configs to PHP config builder configs.
Not sure how easy possible without knowing the Builder class and only the array this is even possible.

I had the same thought... and also not sure if it's possible. Well, those config builders are built from the config tree/Configuration classes... so in theory it might be possible.
But these seems important. If we're going to allow PHP config, I think the config builders need to be used from day 1 or never at all. Switching to it later would be a second nightmare of updating docs, getting people to migrate again and trying to get recipes:update to work.

Understand your point, I added it also to the release milestone. I already had a quick chat with @Nyholm about the ConfigBuilders as I think he knows most about them. He already mention that the yaml has some shortcuts, if he got some time he will add his thoughts about it.

There is still a long way in front of us but already created a prototype to convert PHP Array Configs to ConfigBuilder.
First results:
 <?php

 declare(strict_types=1);
+use Symfony\Config\FrameworkConfig;

 use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;

-return static function (ContainerConfigurator $containerConfigurator): void {
-    $containerConfigurator->extension('framework', [
-        'session' => [
-            'handler_id' => null,
-            'cookie_secure' => 'auto',
-            'cookie_samesite' => 'lax',
-            'storage_factory_id' => 'session.storage.factory.native',
-        ],
-    ]);
+return static function (FrameworkConfig $frameworkConfig): void {
+    $frameworkConfig->session()->handlerId(null)->cookieSecure('auto')->cookieSamesite('lax')->storageFactoryId('session.storage.factory.native');
     if ($containerConfigurator->env() === 'test') {
-        $containerConfigurator->extension('framework', [
-            'session' => [
-                'storage_factory_id' => 'session.storage.factory.mock_file',
-            ],
-        ]);
+        $frameworkConfig->session()->storageFactoryId('session.storage.factory.mock_file');
     }
 };

Test Repo: https://github.com/alexander-schranz/prototype-config-builder-refractor


Rayable trait

Instead of passing arguments to the ray() function, you may now add the Rayable trait to a class and call it fluently.
use Spatie\Ray\Support\Rayable;

class User
{
    use Rayable;
}

$user = new User;
$user->ray()->doSomething();

I thought this was a neat idea! Do you plan on continue polishing this, or is it ready for review?

Hey Freek, I'm sorry it took me this long to respond. The feature is ready, but I couldn't get the test to work with GitHub Actions. I think there's some metadata in the snapshot that differs.

Ok, I'll take care of this and finish the feature!


Feature/less package

Added less loader. User options and most of the code are the same as scss package for easier usage.

This would be very useful to me. Can we get this vetted and merged?

Im waiting for any comment from maintainers then im read to add tests and example.

I'm going to close this in favor of #1150


Revert  control Blazor guidance

NOTE: There is deactivated Link component content in the Class Libraries topic to update/reactivate for 6.0.
When components for influencing <head> elements come back, revert changes made on  #20318.
TOC Built-in components entries
- name: Link
  href: blazor/fundamentals/additional-scenarios.md#influence-html-head-tag-elements
- name: Meta
  href: blazor/fundamentals/additional-scenarios.md#influence-html-head-tag-elements
- name: Title
  href: blazor/fundamentals/additional-scenarios.md#influence-html-head-tag-elements

Release notes ...
### Influence the HTML head in Blazor apps

Use the new `Title`, `Link`, and `Meta` components to programmatically set the title of a page and dynamically add links and meta tags to the HTML head in a Blazor app. For more information, see <xref:blazor/fundamentals/additional-scenarios#influence-html-head-tag-elements>.

Also, update the following line ...

*This section applies to the upcoming ASP.NET Core 5.0 release of Blazor WebAssembly and Blazor Server.*

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

ID: fc56d254-09c5-993b-2559-76af9a3f97ed
Version Independent ID: a0bb183f-7e4d-ba39-c94c-cf60e951c7a9
Content: ASP.NET Core Blazor hosting model configuration
Content Source: aspnetcore/blazor/fundamentals/additional-scenarios.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

@captainsafia ... Will the work for bringing back <head> control in Blazor apps be done for the 6.0 release? I just want to get the labeling right for this docs issue.

@guardrex It's on the roadmap for 6.0 but prioritized pretty low. I would tentatively put this on-hold until we confirm we are doing this for 6.0.

Note to self:
https://github.com/dotnet/aspnetcore/blob/main/src/ProjectTemplates/Web.ProjectTemplates/content/BlazorServerWeb-CSharp/Pages/_Layout.cshtml#L15

This was resolved by #22931, #22929, and #22931.


area light attributes

area lights block sun light, this should not be the case, always turn off cast shadows.
Add primvis, visible in reflections, visible in refractions etc.

Done and added texturing.


Why non English character can not be used as taxonomy name

I created a taxonomy using a Chinese word. But when CreateTermContentType for this taxonomy, the name of it will be ToSafeName().
public string GenerateTermTypeName(string taxonomyName) {
            var name = taxonomyName.ToSafeName() + "Term";
            int i = 2;
            while (_contentDefinitionManager.GetTypeDefinition(name) != null) {
                name = taxonomyName.ToSafeName() + i++;
            }

            return name;
        }

After this operation, the non English character and non digital character will be removed, then the name become empty. This will causes #7893.
However, the taxonomy can be created successfully as well as the Term content type for this taxonomy. The only problem is that when create a new term for the Term content type, the edit page is empty.
repro step:
1: Create a taxonomy with non English character and non digital character.
2: Create a new term for this taxonomy
3: Editing page is empty.

Because they are used for technical names. You can use any character in the display name.


"Fury" spell targetting caster on a horse

Hello,
We have Magic version 7.9.2-SNAPSHOT-363ebb5 on a Minecraft 1.15.2 paper server.
Casting the Fury spell while on a horse will also target and hurt the caster. I think it is because it is targetting the horse itself and the player happens to be in the way, but ideally there should be an exception for a hourse mounted by the caster.
This is not an urgent issue, but thank you for looking into this.

Thanks! This seems to have revealed some consistency issues with how the "target self" flag works. I'll get it fixed up for the next release.
I'm also going to add a flag to skip targeting mounts, which will be on by default.
So the real fix here is having it avoid targeting your mount.
But if you're standing right near an enemy you can get hit with the projectile accidentally.

I completely agree that you could be hurt by your projectile if you're in the way, it has always been that way and makes total sense. As you said the real problem here is it targets your mount, which means you get damaged 100% of the time even with no enemies nearby...

Agreed, that's been fixed :)
Spells now avoid targeting the caster's mount by default.

Great, that works with latest version indeed. Thank you !


Enhance parameter types

Introduce parameterType.KEYS for multiple keys
It represents an array of keys, or jsPsych.ALL_KEYS and jsPsych.NO_KEYS
Replace ALL_KEYS and NO_KEYS with strings
"ALL_KEYS" is now used instead of jsPsych.ALL_KEYS and "NO_KEYS"
instead of jsPsych.NO_KEYS
Use enum for parameter types and remove JsPsych.plugins
const parameterType is now enum ParameterType and can only be
accessed by importing it.

enum docs: https://www.typescriptlang.org/docs/handbook/enums.html
Three reasons why I think removing JsPsych.plugins makes sense:

The main reason for plugins was to hold plugins, which it doesn't do anymore. Removing it makes the breaking change more obvious.
All remaining members of the plugins object were parameterType and universalPluginParameters, which were only used within the JsPsych class itself.
Plugins have to access parameter types statically anway.

@becky-gilbert Despite the discussed changes, merging this would require replacing parameterType with ParameterType in all plugins.

@bjoluc sounds good, thanks for explaining these changes!
I'm going to assume that this will be merged and switch to using ParameterType and "ALL_KEYS"/"NO_KEYS" in the modularization-plugins branch. But I'll keep an eye on this PR in case the implementation changes.


Disable html autocomplete

Hello,
This is more like a question. How can I disable autocomplete for forms generated by antd-schema-form
Thanks

You can try custom components, and then set autocomplete="off" for the form controls.

You can try custom components, and then set autocomplete="off" for the form controls.


Add AppVeyor config

Thought it would be nice to ensure it works for Windows folk as well.

:ship: :it:


Cake slices use wrong naming convention

What is currently used:
"[INSERT FLAVOUR] Cake Slice"
How all other cake slices are named:
"Slice of [INSERT FLAVOUR] Cake"

Fixed in 1.1.0


Failed when install this package

I have installed VS2015 and VS2017 locally, but I could not install node-sass which depend on node-gyp which depend on Microsoft's windows-build-tools. So I install this package, but it gave me the following errors.
C:\Users\Liu.D.H>npm install --global --production windows-build-tools
npm WARN npm npm does not support Node.js v10.10.0
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 4, 6, 7, 8, 9.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>postinstall C:\Users\Liu.D.H\AppData\Roaming\npm\node_modules\windows-build-tools
> node ./dist/index.js

Downloading python-2.7.14.amd64.msi
[>                                            ] 0.0% (0 B/s)
Downloaded python-2.7.14.amd64.msi. Saved to C:\Users\Liu.D.H\.windows-build-tools\python-2.7.14.amd64.msi.
Downloading BuildTools_Full.exe
[>                                            ] 0.0% (0 B/s)
Downloaded BuildTools_Full.exe. Saved to C:\Users\Liu.D.H\.windows-build-tools\BuildTools_Full.exe.

Starting installation...
Please restart this script from an administrative PowerShell!
The build tools cannot be installed without administrative rights.
To fix, right-click on PowerShell and run "as Administrator".
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `node ./dist/index.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!       Extracting by manifest.
657 silly tarball no local data for<EMAIL_ADDRESS> There is likely additional logging output above.
797 verbose exit [ 1, true ]

From the log, I could not find something useful.

As stated in the program output:
The build tools cannot be installed without administrative rights.
To fix, right-click on PowerShell and run "as Administrator".```

@schuch, thanks, I will try and report later.

@schuch I installed it with administrative rights. It used to work, but after switching versions with nvm (using nvm use ...) and then coming back to my main version (with nvm), node-gyp fails, despite I install windows-build-tools again (with administrative rights).
See here report (from error and below):
  File "C:\Users\afil\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\gyp\pylib\gyp\common.py", line 393, in close
    os.rename(self.tmp_path, filename)
WindowsError: [Error 183] Cannot create a file when that file already exists
gyp info ok
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (C:\Users\afil\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\configure.js:336:16)
gyp ERR! stack     at ChildProcess.emit (events.js:182:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:237:12)
gyp ERR!  System Windows_NT 10.0.17134
node-pre-gyp ERR! command "C:\\Program Files\\nodejs\\node.exe" "E:\\R\\exclusivi-panel\\node_modules\\grpc\\node_modules\\node-pre-gyp\\bin\\node-pre-gyp" "install" "--fallback-to-build" "--library=static_library"
node-pre-gyp ERR! cwd E:\R\exclusivi-panel\node_modules\grpc
node-pre-gyp ERR! node -v v10.6.0
node-pre-gyp ERR! node-pre-gyp -v v0.7.0
node-pre-gyp ERR! not ok
Failed to execute 'C:\Program Files\nodejs\node.exe C:\Users\afil\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js configure --fallback-to-build --library=static_library --module=E:\R\exclusivi-panel\node_modules\grpc\src\node\extension_binary\no

@schuch
I tried executed the command with administrator permission. But it gave me the same error.
C:\WINDOWS\system32>npm install --global --production windows-build-tools
npm WARN npm npm does not support Node.js v10.10.0
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 4, 6, 7, 8, 9.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>postinstall C:\Users\Liu.D.H\AppData\Roaming\npm\node_modules\windows-build-tools
> node ./dist/index.js

Downloading python-2.7.15.amd64.msi
[============================================>] 100.0% of 20.25 MB (1.72 MB/s)
Downloaded python-2.7.15.amd64.msi. Saved to C:\Users\Liu.D.H\.windows-build-tools\python-2.7.15.amd64.msi.
Downloading vs_BuildTools.exe
[============================================>] 100.0% of 1.12 MB (639.91 kB/s)
Downloaded vs_BuildTools.exe. Saved to C:\Users\Liu.D.H\.windows-build-tools\vs_BuildTools.exe.

Starting installation...
Launched installers, now waiting for them to finish.
This will likely take some time - please be patient!

Status from the installers:
---------- Visual Studio Build Tools ----------
2018-10-08T14:32:27 : Verbose : Closing installer. Return code: 1.
2018-10-08T14:32:27 : Error : Please update Visual Studio Installer before proceeding.
2018-10-08T14:32:27 : Verbose : Closing installer. Return code: 1.
2018-10-08T14:32:27 : Error : Please update Visual Studio Installer before proceeding.
2018-10-08T14:32:27 : Verbose : Closing installer. Return code: 1.
------------------- Python --------------------
Action start 14:25:37: SetODBCFolders.
Action ended 14:25:37: SetODBCFolders. Return value 1.
Action start 14:25:37: MigrateFeatureStates.
Action ended 14:26:05: MigrateFeatureStates. Return value 1.
Action start 14:26:05: InstallValidate.

Could not install Visual Studio Build Tools.
Please find more details in the log files, which can be found at
C:\Users\Liu.D.H\.windows-build-tools

In my case it is yarn that fails, when trying to use node-gyp.
windows-build-tools did install OK.

Haven't looked closely at the error messages reported here, but I did notice the OP is using npm 5.x with Node 10.x. Although I'm not aware of any particular problem with 5.x, I'd recommend upgrading to npm v6.x to isolate the problem as that is what normally ships with 10.x.

You can also try setting the --debug flag (or DEBUG=* environment variable, as described in the README.md file) to get more useful troubleshooting information.


Optimize Code Structure: Externalize CSS for 404, Index, Privacy, and successDelete PHP Pages

Related Issue
Closes: #336
Description of Changes
Externalized CSS for 404, Index, Privacy, and successDelete PHP Pages Cleaned up codebase for better maintainability and performance. Your collaboration is valued! 🌟📝🚀
Checklist:

[x] My code adheres to the established style guidelines of this project.
[x] I have conducted a self-review of my code.
[x] I have included comments in areas that may be difficult to understand.
[ ] I have made corresponding updates to the project documentation.
[x] My changes have not introduced any new warnings.

@apoorvaron Review some of this changes

@Nishitbaria Changed files only for 404 page

For Privacy also improved responsiveness
https://github.com/apoorvaron/Shorty/assets/85815172/82ac1f04-9eb7-4096-af15-be0d50227fd6

@apoorvaron Check the final


[Snyk] Upgrade aws-sdk from 2.1611.0 to 2.1613.0

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to upgrade aws-sdk from 2.1611.0 to 2.1613.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.

The recommended version was released on 21 days ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Pretty sure we can actually remove aws-sdk - don't need any direct connection to S3/SQS in this form

Closing as we will never merge this


Uncaught TypeError: Cannot instantiate an arrow function

When using parcel v2, or esbuild the generated JS code, causes the above error when calling AbsintheSocket.create.
I was after some trial and error able to make it working again for esbuild by using require instead of import.
// import * as AbsintheSocket from "@absinthe/socket";
const AbsintheSocket = require('@absinthe/socket');

This did not work with parcel however.
I cloned José Valim's esbuild test repo to replicate the error, https://github.com/Hermanverschooten/phx_esbuild_demo, branch test_absinthe

I'm also seeing this issue.
@Hermanverschooten Your require fix works for me; thanks for posting it!

@Hermanverschooten Shot in the dark, but you wouldn't have any idea why using esbuild's inject feature might cause this bug to come back even if you're using the require trick, would you? I'm at a loss.

@neurodynamic No I am sorry, no clue. Sorry about the delay, I was traveling. But if you find out, please share.

I gave up on Parcel long time ago and since moving to the esbuild hex package and using the require solution I have had no more issues.

I'm still seeing this issue. even with the require method. I'm using @absinthe/socket-apollo-link version 0.2.1. phoenix version 1.7.1. Anything else I need to do?

@RichMatthews  I am still using the require method, I must say my phoenix is still at 1.6.15 for that project.


feat: added dict.cc extension

Description
Simple translation using dict.cc.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Hi @pernielsentikaer, thanks again for the feedback. I applied it here: https://github.com/raycast/extensions/pull/1358/commits/6dd79f84656f25ef610e54cde78c59602ac31475. Please, let me know what you think.

Thanks for the updates @natterstefan 🔥

Thanks for your review and patient help.


feat(uninstall): add --skip-registry-secret option and make uninstall remove registry secret #1859

Release Note
NONE

As I got 2 approvals let me go ahead and merge it.


macOS installation

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Dear Team I could not install the mzmime3 on my computer (macOS system). It gives me a corrupt file message. I open my security settings but not working.

Hey,
did you try this:
https://github.com/mzmine/mzmine3#running-on-macos
We are still in the process to sign MZmine. So future releases will be easier to install.

Closing due to inactivity. In case you are still facing issues we can reopen and discuss

I tried every way but couldn't get it to install. It keeps giving the following message. thank you very much for your interest.
Last login: Fri Mar 25 22:21:07 on ttys000
(base) sahabettinselek@Sahabettin-MacBook-Pro ~ % sudo xattr -cr MZmine.app
Password:


Unable to delete users

Whenever I try to delete a user, I get an error page stating:
Call to undefined method Illuminate\Database\Query\Builder::sync()
I am currently using the latest version of the master branch.

Umm try: composer update --no-dev --no-interaction

Did not find any update and did not fix the problem, sadly :(

Did you try php artisan migrate ?

Yields Nothing to migrate

This is a confirmed bug. I will push a hotfix shortly


sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) FATAL: sorry, too many clients already

Hi, we using a k8s cluster on which we installed airflow2 via helm chart.
We're trying to run 300 tasks on airflow - and got this error: sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) FATAL: sorry, too many clients already
i have tried the next configurations on helmchart value.yaml file:
env:

name: "AIRFLOW__CORE__SQL_ALCHEMY_POOL_SIZE"          ===> after this change airflow ui taking longer time to refresh pages
value: "1000"
postgresql:
enabled: true
postgresqlExtendedConf:
maxConnections: "1000"

This is the postgres template values (airflow/charts/postgres/values.yaml):
resources:
requests:
memory: 256Mi
cpu: 250m

@krestix This issue is caused by airflow tasks all creating their own connections to the metadata database (Postgres), exceeding whatever max_connections is set on your Postgres instance.
There are a few options to fix it:

We could implement PgBouncer in the chart, to pool the connections:

However, this would not work with MySQL, and I am not sure if there is an alternative (or if this issue can even occur in MySQL)

Increase the max_connections of your Postgres
Reduce the number of connections which airflow is making:

There are a few configs for airflow that do this, like AIRFLOW__CORE__SQL_ALCHEMY_POOL_SIZE
However, after you have a large number of tasks, there is no avoiding lots of connections, due to how airflow is designed.

@krestix I do wonder if solving https://github.com/airflow-helm/charts/issues/233 (allowing Redis as a result_backend) will fix this issue for you?

@thesuperzapper
Hi, we use KubernetesExecutor  -
after using pgbouncer the issue with "no client connections" fixed
but we faced another problem
if we run many (8000 for exm.) jobs - most of them crash and loose the connection to the network.


Kubernetes deployment not accessible

On following the README in scalable folder. The deployment that is created is not accessible. None of the ports are listening and the logs are not getting populated. Cannot seem to find the problem.
Please let me know if you get any information.

Hi @salonijuneja27 - we’ll do a review, and in order to best do so, it would be helpful if you could share information about your environment (kubernetes deployment mechanism, version, etc), any anchore configurations files that you can share, and any logging and command output to show us your process thus far (starting up, determining that logs are not being populated, etc).
Best Regards,
Dan

@salonijuneja27 , there are two deployments created, one is the core services deployment which is exposed via the core service definition core service definition. The other is the analyzer deployment, which does not get an endpoint/service definition because those pods only pull tasks from apis exposed by the core.
To use the system, you configure your client to access the core service dns name and port.

Closing due to no response/activity in a long time.


UAT: Case witness and evidence number

reference -- https://docs.google.com/spreadsheets/d/1HPBO1MB62iDAP3pcvl2LWUuVcYyNBR20etZtWcxPIFU/edit?gid=0#gid=0
to align with all other numbers, suggest following change
numbers after 'Prosecution (Complainant) Artifact/Evidence' will all follow the same format as that followed by application, order, hearing etc. so the format will be
format: CNR-P[X]
example: KLKM520000012024-P1
special instruction: when used in front end, like for the judgement order, remove CNR part and just the suffix
same will be done for numbers from row 17 - 24.

@Ramu-kandimalla the ticket is still valid if this has not been addressed.

Is the numbering suggested done for various artifacts/evidence as mentioned in the XLS
In Judgement order and case bundle (not sure how the numbering will be displayed in case bundle - will need to with Abhishek once)


Add query to Route class

Move the query function from its own function to the route class and add docs.

:tada: This issue has been resolved in version 1.3.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Send verification code before login?

I'm building a django application in which the user will provide instagram username and password. The application will find companies from google maps and then will look for them in instagram and send some message. The problem is the django app is synchronous, I mean, when the user clicks on the submit form button with username and password, the execution of the login method will break when asking the verification code send via email.
Is there a way to send the verification code before login? So the user will be able to fill the username, password and verification code fields together?

https://github.com/adw0rd/instagrapi/blob/master/instagrapi/mixins/totp.py

@adw0rd thanks for reply.
In the (https://adw0rd.github.io/instagrapi/usage-guide/totp.html) you sent, when calling  cl.login(USERNAME, PASSWORD) it will ask for the verification code before enabling the TOTP. How to avoid the ask for the verification code and move forward to enable TOTP?

No way, disable 2fa yourself, after which you can log in and enable totp

Thanks again @adw0rd. One more question: I've wrote the code to use google places api and I am able to get name, phone and website from places. But how can I find the companies in the instagram and be sure I am finding the same company that I found at google places? Is it possible to search on instagram by anything besides the name? I tried to search for phone and mail at instagram for web, with no success.

I don't know how to find a place by email or company phone number


KeyError from Typing while importing TestSuite

Hi, first time posting an issue so let me know if there's more info needed.
Evidently version: 0.4.36
Python version: 3.11.0
I'm trying to import TestSuite but keep running into the following issue:

I have tried installing this fresh and I'm still running into the same issue.
Would anyone be able to help solve this?
Thanks

Hi @brandoneh,
1. Reproducible example. Could you share the exact code you are running, is this:
pip install evidently
from evidently.test_suite import TestSuite

?
2. Environment. What is your Python environment - e.g. Jupyter notebook, Colab?

Hi, I'm running into this problem on Databricks, but am also seeing the issue using vscode, poetry and pyenv locally to replicate it. The commands I've run are as below:
poetry new evidently_test
cd evidently_test
pyenv local 3.11.0
poetry shell
poetry add evidently
python
>> from evidently.test_suite import TestSuite

Let me know if I need to provide or clarify anything else. Thanks

Any news on this issue?

The same issue with Runtime Databricks 15.4. I have two potential solutions:

Downgrade evidently==0.4.30
Switch runtime to 14.3 LTS


Synology camera shows correct state based on is_streaming and is_recording flag

Description:
Updated Synology camera to show correct state based on is_streaming and is_recording flag
Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

I'm closing this one as I've submitted another PR which includes features added here: https://github.com/home-assistant/home-assistant/pull/9583


Support style for Extension python markdown

Python Markdown Extensions
https://pythonhosted.org/Markdown/extensions/index.html#officially-supported-extensions

We use Markdown-it for the markdown preview. Here's our current markdown extension story: #22916 Please take a look at the API and let me know if you are interested in writing an extension


Loss is nan, stopping training

I have  an issue with the training, for the 2 datasets, I get the same error: Loss is nan, stopping training.

I just changed the parameters of training. It was mainly an issue related to my GPU.


chore: revert jellyfish bound

This reverts commit f8825059dd2a6d75e7c6fed17cd34f72d4b515bd.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

https://github.com/jamesturk/jellyfish/issues/217 has been closed and there should now be a compatible windows release of jellyfish. We should be able to remove the bound.

Windows tests are still failing with
Collecting jellyfish>=0.8.9 (from bigframes==1.27.0)
  Downloading jellyfish-1.1.2.tar.gz (364 kB)
  Installing build dependencies: started
  Installing build dependencies: finished with status 'done'
  Getting requirements to build wheel: started
  Getting requirements to build wheel: finished with status 'done'
  Preparing metadata (pyproject.toml): started
  Preparing metadata (pyproject.toml): finished with status 'error'
  error: subprocess-exited-with-error

  Preparing metadata (pyproject.toml) did not run successfully.
  exit code: 1

  [6 lines of output]

  Cargo, the Rust package manager, is not installed or is not on PATH.
  This package requires Rust and Cargo to compile extensions. Install it through
  the system's package manager or via [https://rustup.rs/](https://www.google.com/url?q=https://rustup.rs/&sa=D)

  Checking for Rust toolchain....
  [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed


#44 Create the Nimblehq GitHub example project

https://github.com/nimblehq/flutter_templates/issues/44
What happened 👀
Officially, there is no example project to show how we arrange our codebase yet.
The example application should show the list of our organization's Github repositories, so the project name could be nimblehq_github.
The main purposes of this project are:

How we work with riverpod.
How we arrange the MVVM layers (currently a resource to refer to could be varun-matchmaking-flutter).
How we use retrofit or dio to make the network request.

Insight 📝
I have created the example app based on flutter_templates init script:

The project name: nimblehq_github
The app name: Nimblehq Github
The package identifier:

Android: co.nimblehq.example.nimblehq_github
iOS: co.nimblehq.example.nimblehq-github

In order to pass the CI, I have config analysis_options.yaml to exclude examples directory out of flutter analyze.
Proof Of Work 📹

@manh-t So we bring everything from the template to example, and when modifying/adding sth in template, we have to do the same with example 🤔

@doannimble yes, please read this discussion for more details

We will not perform any action to this PR until #58 is clear 🙇


Markup to identify registries

Many specs either contain an inline registry or link to an external registry, e.g. https://encoding.spec.whatwg.org/index-windows-1252.txt and https://storage.spec.whatwg.org/#registered-storage-endpoints
The TAG has been discussing how this proliferation of registries both makes them hard to find for authors and difficult to process in an automated fashion (due to the many different formats used to encode the data).
One thought is to include some standardized markup to identify inline registries and external registry URLs in specs. This would allow a tool to scrape specs and maintain a centralized list of what registries exist and where they can be found.

The big problem, I assume, would be associating all the additional data a registry has, beyond just the name.  I presume it would just have to be a full "row" of data, uninterpreted.
Similarly, I assume that the only "special markup" that could really be imposed would be a requirement that they be a table, with an identifying class or attribute that external tools could find.
Is there more data that would need to be included?

For a first pass, we don't have to markup the actual registry data, just the fact that a registry exists, and some metadata about the registry, e.g. its name and URL.
For inline registries the URL would be the anchor of the registry markup, for external registries the markup would have a way to specify the URL.
We can work on standardizing the actual registry data later.

Note that the Process CG has been working on defining what registries are from a process standpoint. That wouldn't put any particular requirement on bikeshed, but it might inform what you want to do.
The proposal isn't final, but you can read up on variants currently under investigation at: https://github.com/w3c/w3process#user-content-branches-under-development
I'll also give a talk on this very topic at the upcoming AC meeting. This should soon be available with synchronized subtitles (in various languages) but until then, the slides / recording / transcripts are available at https://florian.rivoal.net/talks/tpac2020/process2021/


Don't collapse TOC

I'd love for my table of contents in the side bar to always be expanded fully. Is there a config parameter to achieve this, or, if not, could one be created?

Found it.
tocFolding = false either in the front matter or the config/_default/params.toml


Does not seem to work with headless mode

This is related to issue #921 which is now closed, however I am still experiencing this problem. In headless mode, it still does not seem to work.

Puppeteer version: 1.10.0
Platform / OS version: Windows 10 x64
Node.js version: 8.9.1

Here is the entire code:
const puppeteer = require('puppeteer');

(async function main() {

    const browser = await puppeteer.launch({
        headless: true,
        userDataDir: "./tmp",
        args: [
            '--disable-infobars',
            '--window-position=0,0',
            '--ignore-certificate-errors',
            '--ignore-certificate-errors-spki-list'
        ]
    });

    console.log("going to page");

    const page = await browser.newPage();

    console.log("created new page");

    page.goto('https://web.whatsapp.com/', {
        waitUntil: 'networkidle2',
        timeout: 0
    }).then(async function (response) {

        console.log("opened site");

        await page.waitFor(30000);

        console.log(await page.content());

        await page.screenshot({
            path: 'test.png'
        });

        await browser.close();
    })

})();

This works fine, site loads fine and I can do everything fine. The tmp user directory is also filled with stuff after running above code with headless set to false. But as soon as I set headless to true while keeping tmp directory intact with previous session, it just loads blank page, seems like nothing gets called after statement const page = await browser.newPage() because last message I see on screen is just going to page.
Came to know about using absolute path so I tried using userDataDir: "C:\\Dev\\tmp" too but in this case, response is  and screen generated is white empty one. The expected result is to see whatsapp web scan qrcode page.
Thanks

I have this issue regardless of headless mode or not. It seems like the first .goto() call is ignored (Chromium stays on about:blank) but making a second .goto() call with the same parameters begins navigation as expected.

@wildlyinaccurate For me on Win10 x64, even that doesn't work. I can make it pass beyond new page by specifying absolute url but then it does not go to actual url and yields empty page.

While #921 got fixed, there's still no 100% interop between headless and headful profiles:

#1268 - Headless can't read LocalStorage written from Headful
#1270 - Headless can't read indexedDB written from Headful and vice versa

I'd expect certain things to be falling apart while these things aren't fixed. Also, It might be that whatsapp has some headless checks in place to prevent automation - these fall outside of the scope of the project.
Closing this in favor of #1268 and #1270 - please upvote those to stay in the loop.


Docs/azure pipelines

Add documentation on how to run dagger in Azure Pipelines
Closes: #2847

@grouville  Pulled in the latest changes from main.

@grouville Pulled in the latest changes from main.

I meant git pull --rebase, sorry 😇

completed the rebase

hmmm i must have not done that right, now there are additional modified files, Taking a look.

Okay now the pull request is clean. @grouville


Maven not finding version 5

I tried rebuilding my project with 5.0.0 but it wasn't finding it.  All earlier versions work fine.  Is there time needed before it become available via Jitpack?

https://jitpack.io/com/github/thomasdarimont/embedded-spring-boot-keycloak-server/5.0.0/build.log
The build failed at jitpack build.
@thomasdarimont Could you fix or release a new version?
Thanks

https://jitpack.io/#thomasdarimont/embedded-spring-boot-keycloak-server

Sorry for the delay. It seems that the git environment on jitpack.io is broken and thus the build fails.
I just released 5.0.1 with an updated git-commit-id-plugin for a try to fix the issue.

It seems that the git-commit-id-plugin uses an unsupported parameter for the git log command.
Downgraded the git-commit-id-plugin to 4.0.4 with a try to workaround the issue, see 3339a755681b4ed54ff0a091cb355dab1f222a7b.
Just released 5.0.2 lets see if this works.

3339a75 works :)
The release 5.0.2 seems to work fine :)


Inform about silent supergroup migrations

I think it would be worth mentioning that telegram groups are migrated automatically behind the scenes to supergroups under certain conditions. If not aware this can lead to some unexpected behavior of the session plugin.

I believe if we want to share the same session data for both chats it is required to somehow store the link between the two chat ids. However, to my knowledge this link is only temporarily provided by telegram. The message type has two properties that contain the link info migrate_to_chat_id and migrate_from_chat_id.
This in combination with the chat.id of the update gives the link between the old group and the new super group. When I experimented with the migration behavior I could only get these properties (migrate_to_chat_id and migrate_from_chat_id) once though. Don't know how this is designed but for sure you can not access this information for all future messages.
Once this information is catched by the bot I think one solution could be to add a value (e.g. migrated_to) for the old session object (old key) that contains the new key. And additionally a new session object (with the new key) with the information for the old session (e.g. migrated_from). And then always when loading session data check for the migrated_from field and point to that one instead, i.e. the new session object is only a pointer towards the old session.
What do you think? Or would there be a reason to maintain two sessions for both chat ids and then apply some other logic to combine them later on?
btw: should we continue this here or create an issue for this discussion?

There is a way to link one chat to the other, which is what you outlined. However, this doesn't mean that automatic session migrations from one chat to the other are possible. This is because new messages can be received by the bot from both chats concurrently for a short period of time. This leads to data races which cannot be fixed without help from the bot developer. That's what I meant when I said that we have to

explain why no solution exists

How should we proceed with this?

Do you have more official information on this? I can try to put this conversation into a comprehensive text, explaining the two bullet points. But I can't find much info on this topic so I'm not certain about what exactly to explain.

Here is the Bot API dev himself confirming the data races: https://t.me/tdlibchat/44915

Here is the Bot API dev himself confirming the data races: https://t.me/tdlibchat/44915

Thanks for the info. Do I understand correctly that we can hope that this will be fixed eventually (https://t.me/tdlibchat/42933)?

Here is the Bot API dev himself confirming the data races: https://t.me/tdlibchat/44915

Thanks for the info. Do I understand correctly that we can hope that this will be fixed eventually (https://t.me/tdlibchat/42933)?

No. Read the respective message thread, it continued with https://t.me/tdlibchat/44906.

Okay, I see. Added some more explanation about this problematic with two possible (inconsistent workarounds). Should we put more details or keep it on this level?

Thanks for the detailed feedback!
For 1. and 2.: Yes, this is something I was wondering as well. I think it's worth making it a section. For the placement within the documentation: Is there any page / section about general caveats / good to know things? Something a bit like in the "Advanced/Deployment Checklist" but more general. If not, maybe it's worth opening a new page? Or just in the Overview page?
Do you have other caveats / tips in mind that are worth noting? Maybe also just general Telegram behavior that one might stumble over while developing a bot. This could all go to the decided place then.
And then in the Session Page we just put a short warning box linking to the explanation.
The suggested other workarounds are added in the new commit. If the content is fine like this, it is now just a matter of finding a place for it and prettifying the language / style :)

Nothing is remaining except my take over.

Awesome!

@Madnex could you fix the formatting issues?

What exactly do I need to change? I don't see where the format breaks?

Check the CI task

You can run the formatting script to fix it automatically :)

See https://github.com/grammyjs/website#checking-correct-formatting-of-docs

Thanks, got it I think 👍

Awesome, thanks!

https://github.com/grammyjs/website/pull/628


Grant access to secret store from rhtap-promotion-staging namespace

In 37533f6aee79b78dea775ce2b4ed16813b93818e, promotion component was created and 2 external secrets added in the rhtap-promotion-staging namespace.
These external secrets were not able to get value from the secret store because secret store access is limited to a list of namespaces.
Add the rhtap-promotion-staging namespace in the list of allowed ones using a patch only applied in staging, as we do not want that in production overlay.

/lgtm


modules/users-groups: add shell example

closes #9872
Cannot use the example attribute, because using $ results in:
Example: "${pkgs.bash}/bin/bash"
Not sure if it’s an escaping bug in the docbook tranformation.
cc @peti

But if you use shell like zsn or fish, you must also do programs.zsh.enable = true;

You are right, this needs some more thought.
We’ve got an enum type, haven’t we? So we could make it an either (enum [ … ]) str and define values for shells.

I added the escaping and a note that the shell might need to be enabled. @jagajaga @peti

looks great!


Change python version to 3.9.* in "environment.yml"

Hey @klamann -- as discussed, here is the brief patch to use python version 3.9.x when using conda as package manager. Cheers, David.

My pleasure! Thanks for the fast review and the correction with the white space :)


Docker and Docker Compose Support

Docker
The Dockerfile and .dockerignore will create a docker image and automatically run the Thread Watcher bot upon start (along with registering the slash commands) when using the following command inside the root folder of the repo:

docker build -t thread-watcher bot/

To then run the resulting docker image, use the following command:

docker run --name thread-watcher -d thread-watcher

Docker Compose
If you'd prefer to use Docker Compose, the included docker-compose.yml will build and launch the container by using the following command inside the root folder of the repo:

docker-compose up -d

#LGTM! Thank you very much


Add support for special comments in multiline functions

This is a starter diff to deal with #282, and I'm happy to iterate on it.
We want to ensure that type comments (https://www.python.org/dev/peps/pep-0484/) for Python 2 are formatted in a valid way, so that we can use Black in our organization.

Pull Request Test Coverage Report for Build 839

17 of 17 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 91.658%

Totals

Change from base Build 838:
0.04%

Covered Lines:
2791

Relevant Lines:
3045

💛  - Coveralls

Pull Request Test Coverage Report for Build 839

17 of 17 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 91.658%

Totals

Change from base Build 838:
0.04%

Covered Lines:
2791

Relevant Lines:
3045

💛  - Coveralls

Pull Request Test Coverage Report for Build 839

17 of 17 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 91.658%

Totals

Change from base Build 838:
0.04%

Covered Lines:
2791

Relevant Lines:
3045

💛  - Coveralls


Specific the first page

Normally, we just need the first page for thumbnail.
It's inconvenient and poor performant if generate all pages once you need only one page.
Can you add more parameter? Or can I contribute to your project.

I'd be open to working on this as well if you're open to it.
Users may be able to specify a particular page or all.


Skip consent for login and logout is missing for application in the rest api

Description:
Currently, we only providing the following configuration through rest API to skip consent

However, in the management console, we can skip the consent for login and logout separately

We need to update the rest API to support it.

@mefarazath was it not allowed from API intentionally or do we need to fix?

I think this improvement came after the REST APIs were done. Omitting this wasn't intentional.

@AnuradhaSK let's update the docs on this as well. https://github.com/wso2/docs-is/blob/master/en/docs/develop/restapis/applications.yaml#L2077

Also this will have a migration impact, if someone migrates from 5.10.0 to 5.11.0 which requires them to regenerate clients. so let's create a migration issue as well.


gddo: display gddo version on godoc.org page

for bug reports (well, mostly) it would be very interesting what kind of version of gddo is currently serving godoc.org.
either in the "about" page or, rather, in one of the footer or header panels.

Dupplicate of #177

Was this closed as "wontfix" or was this actually implemented? /cc @adg


[Web] flutter run in offline mode fails with Failed to load resource: net::ERR_INTERNET_DISCONNECTED

Goal: When working offline e.g. on the go in a train, or with an unstable internet connection, we want to be able to run and develop flutter web apps.
Steps to reproduce:

flutter create --offline .
Disable your network and flutter run on chrome
Chrome devtools console shows this: Failed to load resource: net::ERR_INTERNET_DISCONNECTED for<EMAIL_ADDRESS>
Expected:
flutter run, runs a flutter web application which can be executed without remote dependencies. Either by forcing the renderer to be html instead of webkit (which is reported to be working) or by providing a locally cached version of canvaskit.js when flutter create is run with --offline
This issue has already been reported multiple times was triaged and closed without resolving it, reraising the issue since it's not yet properly addressed.
https://github.com/flutter/flutter/issues/85624
This is also related:
https://github.com/flutter/flutter/issues/70101

Hi @dkbast, Thanks for filing the issue. I was able to reproduce the issue on stable and the master channel. Even the counter app fails to run offline no errors are logged to the flutter console and the app shows white screen. However the chrome devtools shows the error Failed to load resource: net::ERR_INTERNET_DISCONNECTED

logs
mahesh@Maheshs-MacBook-Air-M1 hello_world % flutter create --offline .
Recreating project ....
  lib/main.dart (created)
Running "flutter pub get" in hello_world...                      1,013ms
Wrote 4 files.

All done!
In order to run your application, type:

  $ cd .
  $ flutter run

Your application code is in ./lib/main.dart.

mahesh@Maheshs-MacBook-Air-M1 hello_world % flutter run -d chrome
Launching lib/main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...             12.9s
This app is linked to the debug service: ws://<IP_ADDRESS>:61433/SOOA6aI22BE=/ws

flutter doctor -v
[✓] Flutter (Channel master, 2.6.0-12.0.pre.789, on macOS 12.0.1 21A559 darwin-arm, locale en-GB)
    • Flutter version 2.6.0-12.0.pre.789 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 6ae721de3d (8 days ago), 2021-11-21 15:22:37 -0500
    • Engine revision ad00d12833
    • Dart version 2.16.0 (build 2.16.0-35.0.dev)
    • DevTools version 2.8.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • B622CBB7-F906-4FA7-8F49-FACEAAC905AB • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.0.1
      21A559 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome
      96.0.4664.55

• No issues found!

[✓] Flutter (Channel stable, 2.5.3, on macOS 12.0.1 21A559 darwin-arm, locale en-GB)
    • Flutter version 2.5.3 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 18116933e7 (6 weeks ago), 2021-10-15 10:46:35 -0700
    • Engine revision d3ea636dc5
    • Dart version 2.14.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.1, Build version 13A1030d
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.28.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • B622CBB7-F906-4FA7-8F49-FACEAAC905AB • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.0.1
      21A559 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome
      96.0.4664.55

• No issues found!

cc: @yjbanov

@yjbanov can you take a look at this? I would expect the latest tool to use the cipd version.
cc @zanderso

CanvasKit is now bundled with the app (you will find it under build/web/canvaskit/), but by default we still use unpkg.com. To use the bundled version add this to the <head> tag of the index.html (or whatever host page the app is using):
<head>
  <script>
    window.flutterConfiguration = {
      canvasKitBaseUrl: "/canvaskit/"
    };
  </script>
</head>

(the exact value of canvasKitBaseUrl may vary, depending on the baseUrl and server configuration; "/canvaskit/" work for the default app template served using flutter run)
Fallback fonts still come from fonts.google.com though, so there's work to do for out-of-the-box offline support (see https://github.com/flutter/flutter/issues/60069#issuecomment-994103190).
Deduplicating with: https://github.com/flutter/flutter/issues/60069


Rework onboarding for CM

Ask user to give access for location and CoreMotion during the onboarding flow.
Don't allow user to proceed and use the app without.

[ ] Remove asking permission to send notifications and replace it with CM permission.
[ ] Ask permission to send notifications on the second day (24h hours after installing the app).

I've noticed this is not correctly align in all phones. Should I try to do it better or is it ok like this?

Don't you think it's super weird we have some parts translated and some are not? I would say we should pick one and stick to it. And for now it would make sense to stick to English.

I'll check, but I'm afraid there's nothing we can do about it. The first part of the message is in whichever language the user's phone is.
Other than that, what about my question?

Ok. About your question - let's keep it on hold until we have new copy for both screens. Then i'll check the designs and confirm with you :)

Step3: location
For iOS 10 and iOS11

Step4: motion data

No text at the bottom for location in iOS 11?

No text

👍


Updated GitHub Issue Classification Tutorial to v0.11

Updated GitHub Issue Classification Tutorial to v0.11

Closing due to duplicate #692


[BUG] Incorrect exception type thrown for blobcontainerclient.createifnotexists()

Library name and version
Azure.Storage.Blobs v12.13.0
Describe the bug
We are trying to create a container client from a storage account that doesnt exist in the following way.
storageAccountConnectionString is dummy here.
BlobServiceClient blobServiceClient = new BlobServiceClient(storageAccountConnectionString);
BlobContainerClient containerClient = blobServiceClient.GetBlobContainerClient("blobName");
containerClient.CreateIfNotExists();
We are getting 'The remote name could not be resolved' exception as expected.
Ideally we expect AzureRequestFailed exception as per Link, but we are getting System.Aggregate Exception with 6 internal retries.
Can you please help why we are not observing AzureRequestFailed exception?
Expected behavior
We are expecting AzureRequestFailed exception
Actual behavior
We are observing System.Aggregate exception
Reproduction Steps
Use dummy storageAccountConnectionString and implement the following code.
BlobServiceClient blobServiceClient = new BlobServiceClient(storageAccountConnectionString);
BlobContainerClient containerClient = blobServiceClient.GetBlobContainerClient("blobName");
containerClient.CreateIfNotExists();
Environment
No response

Because the storage account does not exist (or is a dummy in this case), when we go to send the request we will get back a Bad Connection attempting to request to a URL endpoint that does not exist. That doesn't necessarily mean that the storage account (or container) doesn't exist, it means that the connection or authentication was bad.
We cannot throw back a false if we were not able to initialize a connection to the service or it the authentication was invalid. It doesn't actually necessarily mean the storage account (or account does not exist), because this behavior would be created if you didn't have internet connection or if you gave us a malformed URL.
If you're trying to see if the storage account exists I would suggest using the Management SDK. And then check if the container exists. Unfortunately there's no way to do this in one API call since each type of resource lives in a different REST API surface.

We are not trying to see if the storage account exists. We are bothered with the type of the exception the SDK is giving. When we perform exception.GetType(), we expect AzureRequestFailed exception as per Link, but we are getting System.Aggregate Exception

True, the documentation should be updated to state that "A RequestFailedException will be thrown if a storage service error is thrown." Instead of any failure. The error you're getting is thrown by Azure.Core underneath.

Can you please summarize what all types of exception can occur apart from AzureRequestFailedException during blobcontainerclient.createifnotexists() call so that we can handle those error types separately?

The ask is to go through all the documentation and every API, and all the dependencies and see what exceptions each exception is thrown, all their error codes (including general exceptions from System)? That seems a little excessive.
I empathize with the intention to cover all exception cases, however for us to document every possible exception case that does not get thrown by our library but by a dependency, seems a lot to maintain as dependencies can add more exception error codes. We do document our dependencies is the Nuget library, and those dependencies do document the exceptions.
We will take it into consideration to how to best document all the exceptions that can be thrown, to best help with exception handling.

Ok. Thanks for your help


add niteshseram's data

@schmelto , when someone adds their own file, do they leave an empty line at the end?

@schmelto , when someone adds their own file, do they leave an empty line at the end?

dont think so


2 borders in borderRadius style

Description
I am applying this style when I need radius but its showing two radius in Android working fine in iOS.
container: {
      marginVertical: 8,
      borderRadius: 10,
      paddingHorizontal: 17,
      marginRight: 10,
      width: 160,
      backgroundColor: Colors.white,

      borderWidth: 0.3,
      borderColor: Colors.primary,

      shadowColor: Colors.primary,
      shadowOffset: {
          width: 0,
          height: 2,
      },
      shadowOpacity: 0.25,
      shadowRadius: 3.84,

      elevation: 5,
  },

Version
0.67.3
Output of npx react-native info
error Unrecognized command "info".
info Run "react-native --help" to see a list of all available commands.
Steps to reproduce
Follow given style
Snack, code example, screenshot, or link to a repository

@work4m it's due to shadow or elevation. Could you please try removing shadow and elevation styles.


Combine configuration

Combine config.py and config.env somehow, or differentiate their purposes better.

Idea:
decentralize configuration by container
use the envfile attr of docker-compose service declarations


Return NotFound in mgmt API requests when tenant is not present in the pageserver

Describe your changes
Add Error enum for tenant state response to allow better error handling in mgmt api
Issue ticket number and link
#2238
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Shouldn't we also change some mgr::get_tenant usages, such as let tenant = mgr::get_tenant(tenant_id, false).await?; in the tenant_status handler?
I see that we have around more than one (3?) different errors to indicate that the tenant is absent internally.
Does not look fun, might be better to push the "missing tenant error" variant somewhere lower, on the mgr level?

Looks like the tests are failing because of changed strings in allowed_errors.
Were the string changes on purpose or was this just about returning 404?

Shouldn't we also change some mgr::get_tenant usages, such as let tenant = mgr::get_tenant(tenant_id, false).await?; in the tenant_status handler?
I see that we have around more than one (3?) different errors to indicate that the tenant is absent internally. Does not look fun, might be better to push the "missing tenant error" variant somewhere lower, on the mgr level?

done

Lets add a test to detach that checks the return code to be 404

@LizardWizzard there is already one at test_tenant_detach_smoke: line 266


Failed to render the page

I have followed the Get Started page to set up the first site and have added index.md under "pages" folder that has "## Hello World". Everything seems to be working until yarn next. I see some warning showing in the start-up of the dev server.
❯ yarn next
yarn run v1.22.19
warning package.json: No license field
$ /home/jhonghee/workspace/web-workspace/default-site/node_modules/.bin/next
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
info  - automatically enabled Fast Refresh for 1 custom loader
event - compiled client and server successfully in 1681 ms (154 modules)
wait  - compiling /_error (client and server)...
event - compiled client and server successfully in 70 ms (155 modules)
warn  - Fast Refresh had to perform a full reload. Read more: https://nextjs.org/docs/basic-features/fast-refresh#how-it-works
Error: apply() is only allowed in ready status (state: idle)
    at http://localhost:3000/_next/static/chunks/webpack.js?ts=1669951074125:548:21
wait  - compiling / (client and server)...
event - compiled client and server successfully in 930 ms (161 modules)

And the rendered page of "http://localhost:3000" has javascript as follows.

Running environment is Ubuntu 22.04. Node version is v18.12.1.
I've also tested the exact same set up on Windows 11 and got the same result.
While I was adding dependencies, I saw following warning.
(base)  jhonghee    default-site  ﮫ6.856s⠀   yarn add next nextra react react-dom nextra-theme-docs                                                                      pwsh   100  21:14:26 
yarn add v1.22.19
info No lockfile found.
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning "nextra ><EMAIL_ADDRESS>has unmet peer dependency "webpack@>=4".
warning "nextra-theme-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ^17.0.0".
warning "nextra-theme-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "nextra-theme-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "nextra-theme-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "nextra-theme-docs ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS><=99".
warning "nextra-theme-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "nextra-theme-docs ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "nextra-theme-docs > @reach/skip-nav ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning "nextra-theme-docs > @reach/skip-nav ><EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...

What am I doing wrong?

I've played around with the dependencies and found a working combinations.
From
    "next": "^13.0.6",
    "nextra": "^1.1.0",
    "nextra-theme-docs": "^1.2.6",

to
    "next": "12.1.6",
    "nextra": "2.0.0-beta.41",
    "nextra-theme-docs": "2.0.0-beta.41",

Please test it with latest 2.0 version of Nextra, keep in mind that Nextra and Nextra docs theme need to have the same version.


Support insecure registry option in "publish an image" step

publish an image  step should provide option for user to specify if the chosen registry is insecure or not.

If a user want to deploy a workload using images from insecure registry, they need to properly configure docker daemon on their hosts beforehand.
Now publishing a image to an insecure registry results in a failure in auth.
The publish image step is run in dind way, we can add an allow insecure registries option in the step and pass it to that docker daemon if we want.

This is causing me trouble at the moment. I've set up an insecure registries in the daemon.json on my hosts, but when it tries to do the publish an image step the authentication fails.

I ended up just securing the registry

Starting from v2.1.0, UI does not expose the option, but you can set environment variable for the push image step as follows:
- publishImageConfig:
      dockerfilePath: ./Dockerfile
      buildContext: .
      tag: xxx
      pushRemote: true
      registry: xxx
    env:
      PLUGIN_INSECURE: "true"

For more available environment variables please refer to the docs.

@gitlawr
Hi, could you tell me where can I find the environment variables for the plugin.
There are only 5 environment variables in this page.
https://rancher.com/docs/rancher/v2.x/en/k8s-in-rancher/pipelines/#build-and-publish-images
https://rancher.com/docs/rancher/v2.x/en/k8s-in-rancher/pipelines/#environment-variables

@samsiu2014 It's using the drone docker plugin under the hood: https://github.com/drone-plugins/drone-docker/blob/master/cmd/drone-docker/main.go

@gitlawr Thanks a lot. You're my life saver!


Perform Node version check

After setting up my Neovim config on my laptop, I noticed I kept getting warnings like Client [number] quit with exit code 2 and signal 0. At first I was quite lost, since my config is based off of LazyVim and I'm quite new to Vim, so I didn't really know where to start to troubleshoot, but I wrote a little bit of code and noticed I wasn't getting Copilot suggestions.
To make a long story short, I realized my default Node version was v16. Fixing default Node version to v20 fixed it, but the warning message was difficult to figure out. I (maybe naively) feel like a quick node --version check before calling node would make the experience a little be nicer. I also recognize that the README specifically says that Node needs to be > 18.x, so feel free to close this if it's out of scope.
Potentially related: #232

Should be fixed with https://github.com/zbirenbaum/copilot.lua/commit/3665ed0f3ef3ad68673df7195789d134d0d1fdb0


Raspberry Pi Implementation

Hi @atainter,
After trying this on my mac I tried to install it on my Pi but it won't work.
Did you use a Pi to do this or your mac? Can this be implemented on a Pi or not? If so how, because when I try to clone it on the Pi it says that I don't have rights and when I download the Zip it can't decode it, why is that?
Thanks

I'm not sure if the setup script works on a Pi or not. You need to add an ssh key to github to fix the permissions problem.

Sweet thanks

On Nov 27, 2016, at 2:01 PM, Aaron Tainter<EMAIL_ADDRESS>wrote:
I'm not sure if the setup script works on a Pi or not. You need to add an ssh key to github to fix the permissions problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/HackerHouseYT/AI-Smart-Mirror/issues/15#issuecomment-263143686, or mute the thread https://github.com/notifications/unsubscribe-auth/AWbTEU7iSVq9QKPwcyFmdPUtuoxqQjKxks5rCeGUgaJpZM4K9M9a.

You can also use if from http://
git clone https://github.com/HackerHouseYT/AI-Smart-Mirror.git

@carruthjr
Hi can you please tell me if this works on Pi. I cant figured it out.

I made it work on my Raspberry Pi 3 but it´s extremely slow so that it´s actually not worth it for me... if someone knows how to make it work fluently please tell me! :)

nicolasfraikin  do you have your installation steps? to make it work on raspberry pi 3?


TER-363 add landing zone example

fix blocking issues in the cli

fix error where a provider block comes up with empty file location
fix error where the dir doesn't exists when the generated code is empty
fix issue where tr_gen_local gets overridden intermittently.
add flag to skip generating env files
add flag to ignore unimplemented dependencies
update relative paths to module sources

@kanaksinghal I have updated the test and also ran this PR against the test.
PR- https://github.com/cldcvr/terrarium-api-automation/pull/21
Result- https://dash.codepipes.io/org/1ff7b99c-42da-4259-84bb-6b7c439a39af/projects/31185cf6-dd27-46c9-8b7b-90c3f6db72a1/apps/6ccb92f7-48bb-4ee5-a931-f209b1168bac/integrations/f0855e64-bbd5-4e16-a105-070a93ce41de/job/15a214d7-cce5-47c3-8c2e-e64a0693dfe5


Raise Assert.Ignore when step throws NotImplementedException

Context: Fluent Api
If a story fails because a step throws a not implemented exception it would be nice if this translated to an ignored result in the test framework the story is running in. It currently causes the whole test to fail and also prevents any further steps from being run. If a step fails for a different reason it does not prevent other steps from being run and we can see the result of those steps.
Is there a specific reason it was done this way? I have seen the IgnoreStepAttribute but that doesn't appear to work for the fluent api. The above approach worked really well in StoryQ which is why I am suggesting it. I understand the project is not currently being maintained but if there is an agreement on the above then I could put together a PR.

Hi Matt. I don't use the Fluent Api myself, but happy to help you try to get this PR added. I agree that having a not implemented exception translate into an ignored result in the test framework is the desired behaviour.

That sounds great, I will see if I can come up with something.


config.log: Permission Denied

I get this error when I run sudo npm i nodegit -g on Mac OS 10.3.3.
Here is the log:

I had to login with root user to execute node-git pre-install/install scripts.

I would not consider having to login as root a good solution for the problem :/
Any other idea?


[SPARK-13180] Protect against SessionState being null when accessing HiveClientImpl#conf

See http://search-hadoop.com/m/q3RTtFoTDi2HVCrM1 for related stack trace

Test build #50703 has started for PR 11066 at commit 6ee7d98.

Test build #50703 has finished for PR 11066 at commit 6ee7d98.

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

Merged build finished. Test PASSed.

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

Do you have suggestion on which class to put the common code in ?
Thanks

@tedyu lets create a HiveConfUtil object in the client package, put the method in and use that in both places.

Test build #50749 has started for PR 11066 at commit aff16d6.

Test build #50749 has finished for PR 11066 at commit aff16d6.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #50750 has started for PR 11066 at commit 438e2a3.

Test build #50750 has finished for PR 11066 at commit 438e2a3.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #50751 has started for PR 11066 at commit 0a43c19.

Test build #50751 has finished for PR 11066 at commit 0a43c19.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #50754 has started for PR 11066 at commit f20a61f.

Test build #50754 has finished for PR 11066 at commit f20a61f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

This error surfaced:
[info]   2016-02-04 10:15:21.319 - stderr> Exception in thread "main" java.lang.LinkageError: loader constraint violation: when resolving method "org.apache.spark.sql.hive.      client.HiveConfUtil$.conf()Lorg/apache/hadoop/hive/conf/HiveConf;" the class loader (instance of org/apache/spark/sql/hive/client/IsolatedClientLoader$$anon$1) of the current    class, org/apache/spark/sql/hive/client/HiveClientImpl, and the class loader (instance of sun/misc/Launcher$AppClassLoader) for resolved class, org/apache/spark/sql/hive/client/ HiveConfUtil$, have different Class objects for the type /hadoop/hive/conf/HiveConf; used in the signature

I wonder if rev 1 should be used:
the utility class only saves less than 10 lines of code
intricacies were introduced w.r.t. class loading

@hvanhovell
What do you think ?

Test build #50827 has started for PR 11066 at commit 8b9e24c.

Test build #50827 has finished for PR 11066 at commit 8b9e24c.

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

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

Merged build finished. Test PASSed.

@davies
Can you take a look ?

@tedyu This seems like a workaround. Have you find the root cause? Maybe there is a place that we didn't set the SessionState for current thread.

From HiveContext.scala , line 552 (the last line below):
   * 1. create a new SessionState for each HiveContext
   * 2. when the Hive session is first initialized, params in HiveConf will get picked up by the
   *    SQLConf.  Additionally, any properties set by set() or a SET command inside sql() will be
   *    set in the SQLConf *as well as* in the HiveConf.
   */
  @transient
  protected[hive] lazy val hiveconf: HiveConf = {
    val c = executionHive.conf

If I read the doc correctly, the above call is supposed to create new SessionState which is what the PR is doing.

The doc is out-updated, HiveConf is created in HiveClientImp.scla LINE 107 (when created HiveClientImp obj).

The user was using 1.6.0 where there was no HiveClientImp.scala

Thanks for the pull request. I'm going through a list of pull requests to cut them down since the sheer number is breaking some of the tooling we have. Due to lack of activity on this pull request, I'm going to push a commit to close it. Feel free to reopen it or create a new one. We can also continue the discussion on the JIRA ticket.
Note: Looks like this code path has been rewritten heavily and the problem might not apply anymore.


Allow storing different content for different areas of the site

Yeah this one! Please up this!

Is there any update of this?

Im trying to make my version of this. If this Repo still active, i can make a PR


Update endpoint-slices.md

fix translation errors about "Distribution of EndpointSlices" section.

Hi, we do not accept part of changes when much inconsistency exists. Please try to run this command to check difference with en:
./scripts/lsync.sh content/zh-cn/docs/concepts/services-networking/endpoint-slices.md

The output may be similar to:

Hi, we do not accept part of changes when much inconsistency exists. Please try to run this command to check difference with en:
./scripts/lsync.sh content/zh-cn/docs/concepts/services-networking/endpoint-slices.md

The output may be similar to:

the command executed result:
 $ ./scripts/lsync.sh content/zh-cn/docs/concepts/services-networking/endpoint-slices.md
content/zh-cn/docs/concepts/services-networking/endpoint-slices.md is still in sync

Is it ok?

Run the command in the main branch to find differences, and modify all zh text that is different from en.

/hold

Run the command in the main branch to find differences, and modify all zh text that is different from en.
If not, you may need to check it line by line manually.

@windsonsea @tengqm Hi， I have merged the latest content of the main branch and modified the Chinese translation of the corresponding content.

Squash your commits into one please.

Squash your commits into one please.

done.

Hey @YunWZ,
It's been a while since the last update, would you like to address issues in the previous comments so that we can move forward?

Hey @YunWZ,
It's been a while since the last update, would you like to address issues in the previous comments so that we can move forward?

done.

/unhold

Squash your commits into one please.

The commits are 8 now.

Thanks
/lgtm
/approve