ReservoirWeightGeneration_TIMIT notebook dependencies not available

Hello PyRCN Team,
I followed the basic tutorial and got it to work.

I want to adapt the TIMIT notebook to my dataset since I'm doing classification over spectrogram images as shown below.

Using a sliding window with 75% overlap, the inputs are [samples*sliding windows, timestamps in window, frequency]. You'll notice four "sub-bins" per dashed window in the predictions.
However, I can't seem to import the dependencies shown below.

My versions are up to date

Please update the dependencies. Thanks.

Dear @alvin-garcia,
thanks for bringing up this issue. In fact, I was already notified about this before. The SeqToSeqESNClassifier is no more required and now included in the ESNClassifier. You can have a look on the updated version of the notebook in the dev-Branch. After releasing the next version of PyRCN, this will be available on the main branch.

Dear @alvin-garcia,
with the release of the new version of PyRCN, we updated everything. Can you please confirm whether this works now and close this issue if so?

Dear @renierts,
Thanks for looking into this issue. I need help getting the RCN code to run successfully. I noticed the TIMIT example was removed from the examples, so instead I will attach what I really want to train.
rcn_spec.pdf
There's an issue in ~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/echo_state_network/_esn.py, with ndim on line 205. Wrapping the input using np.asarray created a new error. Not sure how to proceed?

Hi @alvin-garcia,
thanks for the hint. I noticed from your appended example that you want to train the ESN with multiple spectrograms (each spectrogram is a sequence).
As shown in the documentation, this can be done by passing numpy arrays to the ESN.
I think, the solution to your problem is to change a few lines of code in cell 12 your jupyter notebook:
There, you have the following lines:
X_train = [x for x in x_train_arr]
y_train = [y for y in y_train_arr]
X_test = [x for x in x_val_arr]
y_test = [y for y in y_val_arr]

you should change these lines of code to the following:
X_train = np.empty(shape=(x_train_arr.shape[0], ), dtype=object)  # shape is the number of sequences/spectrograms
y_train = np.empty(shape=(y_train_arr.shape[0], ), dtype=object)  # shape is the number of sequences/spectrograms
X_test = np.empty(shape=(x_test_arr.shape[0], ), dtype=object)  # shape is the number of sequences/spectrograms
y_test = np.empty(shape=(y_test_arr.shape[0], ), dtype=object)  # shape is the number of sequences/spectrograms
for k, (x, y) in enumerate(zip(x_train_arr, y_train_arr)):
    X_train[k] = x  # x has the shape (n_samples, n_features)
    y_train[k] = y
for k, (x, y) in enumerate(zip(x_test_arr, y_test_arr)):
    X_test[k] = x  # x has the shape (n_samples, n_features)
    y_test[k] = y

This creates a numpy array of dtype object (similar as a list). PyRCN will then iterate across each element in this numpy array.

Ah interesting solution, yes that worked. Thanks!
However, now I'm getting another error, which I'm trying to figure out right now.
I understand the k_in parameter, not sure why there would be a divide by zero in this part of the code?
Fitting 5 folds for each of 200 candidates, totalling 1000 fits
[Parallel(n_jobs=-1)]: Using backend LokyBackend with 128 concurrent workers.
[Parallel(n_jobs=-1)]: Done 194 tasks      | elapsed:    5.8s
[Parallel(n_jobs=-1)]: Done 544 tasks      | elapsed:    8.0s
[Parallel(n_jobs=-1)]: Done 1000 out of 1000 | elapsed:   11.0s finished
---------------------------------------------------------------------------
ZeroDivisionError                         Traceback (most recent call last)
<ipython-input-13-0267e97c79d3> in <module>
     35 base_esn = ESNClassifier(**initially_fixed_params)
     36
---> 37 sequential_search = SequentialSearchCV(base_esn, searches=searches).fit(np.asarray(X_train[:10]), np.asarray(y_train[:10]))
     38 dump(sequential_search, "first_try.joblib")  # TODO: undo the "large" settings!

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/model_selection/_search.py in fit(self, X, y, groups, **fit_params)
    190                 self.all_multimetric_[name] = result.multimetric_
    191                 self.estimator = result.best_estimator_
--> 192         self._run_search(evaluate_candidates)
    193         return self
    194

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/model_selection/_search.py in _run_search(self, evaluate_candidates)
    136         evaluate_candidates: Callable
    137         """
--> 138         evaluate_candidates(self.searches)
    139
    140     def fit(self, X: np.ndarray, y: Optional[np.ndarray], *,

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/model_selection/_search.py in evaluate_candidates(searches)
    175                     result = search(
    176                         self.estimator, params, refit=True, **kwargs[0])\
--> 177                         .fit(X, y)
    178                 else:
    179                     result = search(

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/sklearn/utils/validation.py in inner_f(*args, **kwargs)
     70                           FutureWarning)
     71         kwargs.update({k: arg for k, arg in zip(sig.parameters, args)})
---> 72         return f(**kwargs)
     73     return inner_f
     74

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/sklearn/model_selection/_search.py in fit(self, X, y, groups, **fit_params)
    763             refit_start_time = time.time()
    764             if y is not None:
--> 765                 self.best_estimator_.fit(X, y, **fit_params)
    766             else:
    767                 self.best_estimator_.fit(X, **fit_params)

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/echo_state_network/_esn.py in fit(self, X, y, n_jobs, transformer_weights)
    738             self._check_if_sequence(X, y)
    739         if self.requires_sequence:
--> 740             self._input_to_node.fit(X[0])
    741             self._node_to_node.fit(self._input_to_node.transform(X[0]))
    742             self._check_if_sequence_to_value(X, y)

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/base/blocks/_input_to_node.py in fit(self, X, y)
    132                 n_features_in=self.n_features_in_,
    133                 hidden_layer_size=int(self.hidden_layer_size), fan_in=fan_in,
--> 134                 random_state=self._random_state)
    135         if self.predefined_bias_weights is not None:
    136             assert self.predefined_bias_weights.shape == (

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/base/_base.py in _uniform_random_input_weights(n_features_in, hidden_layer_size, fan_in, random_state)
    192     input_weights = _uniform_random_weights(
    193         n_inputs=n_features_in, n_outputs=hidden_layer_size, k_in=fan_in,
--> 194         random_state=random_state)
    195     return input_weights
    196

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/base/_base.py in _uniform_random_weights(n_inputs, n_outputs, k_in, random_state)
    164
    165     if k_in < n_outputs:
--> 166         return _make_sparse(k_in, dense_weights, random_state)
    167     else:
    168         return dense_weights

~/.conda/envs/tf2-gpu/lib/python3.7/site-packages/pyrcn/base/_base.py in _make_sparse(k_in, dense_weights, random_state)
     94     nr_entries = int(n_inputs * k_in)
     95     indices = np.zeros(shape=nr_entries, dtype=int)
---> 96     indptr = np.arange(start=0, stop=(n_inputs + 1) * k_in, step=k_in)
     97     dense_weights = dense_weights.flatten()[:nr_entries]
     98

ZeroDivisionError: division by zero

Great that it works.
Sure, I will not close the issue before you notify me that everything works. You can close it by yourself as well, if you want.


Netflix cannot be started: script.module.win_inet_pton not available

Bug report
Your Environment

Netflix add-on version: 1.12.3-Leia
Operating system version/name: Raspbian/Kodi 18.7
Device model:  Raspberry Pi 3

Used Operating system:

[ ] Android
[ ] iOS
[ ] Linux
[ ] OSX
[x] Raspberry-Pi
[ ] Windows

Describe the bug

I am using Kodi 18.7 on Raspbian OS on a Raspberry Pi 3. Both is completely new installed and nothing changed. When I want to install the Netflix addon, there is a missing dependency, which cannot be installed: script.module.win_inet_pton (not available). Therefore, also Netflix cannot be installed. How can I fix this?
Expected behavior

Netflix should install and start
Actual behavior

Netflix could not be installed or started
Steps to reproduce the behavior

Install Raspbian on a Raspberry Pi 3
Install Kodi (apt install kodi)
Try to install Netflix

Possible fix

Debug log

The debug log can be found here:

https://paste.kodi.tv/forucujugo.kodi
Additional context or screenshots (if appropriate)
Other information

Screenshots

script.module.win_inet_pton

This is not directly used by this add-on, some dependency could use this

ImportError: cannot import name Padding

This often means could be installed other version of required python "pycrypto" module,
there are a lot of info to this in the already closed issues threads, or you can try follow the Wiki install instructions
for other help, ask on Kodi forum or RPI forum

Hello, just a message to tell you that i'm having exactly the same issue : script.module.win_inet_pton not available


WIN11 HOME

Requirements

[X] I have searched the issues of this repository and believe that this is not a duplicate
[X] I have confirmed this bug exists on the latest version of the app

Platform
Windows
Description
"App not compatible with WINDOWS version"
Steps to reproduce
Go to app file to run installer. Right click, run as admin, it tries to run as ifd WIN8, only older versions of win in drop down. same incompatible message.
Expected behavior
A clean and easy install.
Actual behavior
message pops up incompatible with my win version, but i'm running win11 home.
Screenshots or additional context
No response

I'm sorry, I'm not sure what you mean. Could you maybe send a screenshot?

Somehow the fifth time i troubleshot it, installer worked.


fix: allow non_snake_case and dead_code lints to run within component functions

Currently, the non_snake_case and dead_code lints will not run within component functions because the component macro emits allow attributes for the two lints, which allows room for users to make mistakes. This PR fixes the problem by converting the function name to snake case before prefixing it with __ and dropping the allow attributes for the two lints from the macro.
I have run cargo check on the library and all examples and see no additional warnings. We can have the CI confirm this.

The server_fns_axum example failed to build because there are multiple component and server functions with the same name but in different case (rkyv_example and RkyvExample, for example). I changed component and server macros to emit different prefixes (__leptos_component_ and __leptos_server_) to address the problem.

I don't know why, but I might've run into a cargo fmt bug touching seemingly unrelated code. I thought it would be better not to include those formatting changes, but apparently the CI would not be happy with that! So I included the changes and force-pushed again. Hopefully everything is OK now.

Oh, nevermind, I see what happened here. Perhaps we can have a #[allow(non_snake_case, dead_code)] in the declaration, and another #[warn(non_snake_case, dead_code)] in the body to bring back the lint instead of changing the name of the fn or struct?
Not sure if that would work, but worth a shot


Update to remark 14

https://github.com/vercel/next.js/discussions/27758#discussioncomment-1262647

@leerob is the source for https://nextjs.org/learn/basics/dynamic-routes/render-markdown open in a repo somewhere, or will your team take care of changing that?


Editable address and map for storeDetail

TODO:

[ ] show list of possible addresses (maybe using https://github.com/perliedman/leaflet-control-geocoder)
[ ] replace angular-xeditable with custom code for more fine-grained control

Current coverage is 98.25% (diff: 93.42%)

Merging #198 into master will decrease coverage by 0.25%

@@             master       #198   diff @@
==========================================
  Files           100        104     +4
  Lines          1479       1550    +71
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1457       1523    +66
- Misses           22         27     +5
  Partials          0          0

Powered by Codecov. Last update bdcf458...3d5e1c7


docs: readme updates

Consider updating this as well: https://github.com/lockbook/lockbook/blob/master/public_site/index.html#L32


[Bug]

Reproducible in vscode.dev or in VS Code Desktop?

[X] Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

[ ] Not reproducible in the monaco editor playground

Monaco Editor Playground Code
var jsCode = [
	'"use strict";',
	'function Person(age) {',
	'	if (age) {',
	'		this.age = age;',
	'	}',
	'}',
	'Person.prototype.getAge = function () {',
	'	return this.age;',
	'};'
].join('\n');

var editor = monaco.editor.create(document.getElementById('container'), {
	value: jsCode,
	language: 'javascript'
});
var editor1 = monaco.editor.create(document.getElementById('container1'), {
	value: jsCode,
	language: 'javascript'
});
var myCondition1 = editor.createContextKey(/*key name*/ 'myCondition1', /*default value*/ false);
var myCondition2 = editor.createContextKey(/*key name*/ 'myCondition2', /*default value*/ false);

editor.addCommand(
	monaco.KeyCode.Tab,
	function (e) {
		// services available in `ctx`
 console.log(111,e)
	},
	'myCondition1 && myCondition2'
);
editor1.addCommand(
	monaco.KeyCode.Tab,
	function (e) {
		// services available in `ctx`
 console.log(222,e)
	},
	'myCondition1 && myCondition2'
);
myCondition1.set(true);

setTimeout(function () {
	alert('now enabling also myCondition2, try pressing Tab!');
	myCondition2.set(true);
	// you can use myCondition2.reset() to go back to the default
}, 2000);

Actual Behavior
Only editor1 has addCommand  (only console 222)
Expected Behavior
all has that command  (can console  111 and can console 222 in different editor)
Additional Context
These two commands should not correspond to each other, but the latter will replace the former, resulting in no addCommand in the editor
I want Perform different operations in different editors （console 111 console 222）

I think this is a duplicate of https://github.com/microsoft/monaco-editor/issues/2947

I think this is a duplicate of #2947

Yes  thank you very much

Closing as duplicate of #2947


”Laza”, ”Bejenești”, ”Râșnița”, ”Sauca” are listed in the drop-down menu

Description:
When user writes ”Laza”, ”Bejenești”, ”Râșnița”, ”Sauca” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Laza"

Expected results
The location appears in the drop-down menu as "Sat Laza, VASLUI (Laza)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bejenești"

Expected results
The location appears in the drop-down menu as "Sat Bejenești, VASLUI (Laza)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Râșnița"

Expected results
The location appears in the drop-down menu as "Sat Râșnița, VASLUI (Laza)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Sauca"

Expected results
The location appears in the drop-down menu as "Sat Sauca, VASLUI (Laza)".


maliput_object: 0.1.1-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository maliput_object to 0.1.1-1:

upstream repository: https://github.com/maliput/maliput_object.git
release repository: https://github.com/ros2-gbp/maliput_object-release.git
distro file: foxy/distribution.yaml
bloom version: 0.11.1
previous version for package: 0.1.0-1

maliput_object
* Fixes include folder installation. (#41 <https://github.com/maliput/maliput_object/issues/41>)
* Uses ros-action-ci for main workflow. (#40 <https://github.com/maliput/maliput_object/issues/40>)
* Contributors: Franco Cipollone

CC: @tfoote


Initial work towards autogenerating the list of functions (documentation)

Currently generates something like this which can obviously be improved:

[ ] Group by module
[x] Add parameter names to the signature

This is now a lot nicer with the new (revived) function-signature printing


Format Document crashing

Steps to reproduce
Open any powershell script
Press Alt+Shift+F or right click and select 'Format Document'
Expected behavior
Format document
Actual behavior
Powershell Extension crashes
Turned on verbose logging and this is what it outputs:
8/16/2018 2:07:32 PM [VERBOSE] - Formatting entire document - file://(FILEPATH)/RemoveAllNetworkPrintersv2.ps1...
8/16/2018 2:07:32 PM [VERBOSE] - Document formatting failed in 0.064: Error: Connection got disposed.
8/16/2018 2:07:35 PM [NORMAL] - powershell.exe terminated or terminal UI was closed
Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      4.0
WSManStackVersion              3.0
SerializationVersion           <IP_ADDRESS>
CLRVersion                     4.0.30319.42000
BuildVersion                   6.3.9600.18728
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0}
PSRemotingProtocolVersion      2.2

(Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
I can't seem to get this command to work???

Windows 7.
Additional Notes
Running Invoke-Formatter command brings up another error:
Invoke-Formatter : The term 'Invoke-Formatter' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is
correct and try again.

At \\servername\path\test2.ps1:2 char:1
+ Invoke-Formatter -ScriptDefinition (get-Content $scriptPath) -Verbose
+ ~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (Invoke-Formatter:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

I think the issue is the PSScriptAnalyzer module is not installed correctly. So I have tried to manually import it. Using command 'Import-Module .\PSScriptAnalyzer.psm1'
I then try to run the Invoke-Formatter command again and get this issue:
Invoke-Formatter : Cannot convert 'System.Object[]' to the type 'System.String' required by parameter 'ScriptDefinition'. Specified method is not supported.
At \\server\path\test2.ps1:2 char:36
+ Invoke-Formatter -ScriptDefinition (get-Content $scriptPath) -Verbose
+                                    ~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Invoke-Formatter], ParameterBindingException
    + FullyQualifiedErrorId : CannotConvertArgument,Microsoft.Windows.PowerShell.ScriptAnalyzer.Commands.InvokeFormatterCommand

Interestingly when I run the Get-Module command after I do the manual import it lists PSSCriptAnalyzer as installed but the version number 0.0
> Get-Module

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Management     {Add-Computer, Add-Content, Checkpoint-Computer, Clear-Content...}
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Security       {ConvertFrom-SecureString, ConvertTo-SecureString, Get-Acl, Get-AuthenticodeSignature...}
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Object...}
Script     0.2.0      PowerShellEditorServices.Commands   {ConvertFrom-ScriptExtent, ConvertTo-ScriptExtent, Find-Ast, Get-Token...}
Script     0.2.0      PowerShellEditorServices.VSCode     {Close-VSCodeHtmlContentView, New-VSCodeHtmlContentView, Set-VSCodeHtmlContentView, Show-VSCodeHtmlContentView...}
Script     0.0        PSScriptAnalyzer                    {Get-ScriptAnalyzerRule, Invoke-Formatter, Invoke-ScriptAnalyzer}

Please let me know if I need to provide anything further

@Xaliba101 Were you running your PowerShell commands inside VSCode's integrated PowerShell terminal (which you should in this case) or in a 'normal' PowerShell window?
Can you please provide the output of the following 2 commands when being executed inside VSCode's PowerShell terminal:
Get-Module -ListAvailable PSScriptAnalyzer
(Get-Module -ListAvailable PSScriptAnalyzer).path

We should first find out what your environment issues are but at a later stage, I am pretty sure that a Install-Module PSScriptAnalyzer -Scope CurrentUser will resolve it (but this would be a workaround and should only be done at the end once we know what the problem is).
cc @rjmholt @tylerl0706

@Xaliba101 I think above you've imported the wrong module -- ScriptAnalyzer.psm1 is a script module, but most of PSScriptAnalyzer comes in binaries. You need to find the directory containing PSScriptAnalyzer.psd1 and import that (the directory).
Anyway, I've just opened a PR to fix a possible PSES crash we were seeing.
The output in your latest comment looks like you haven't got PSScriptAnalyzer installed on your system anywhere. The version used by the VSCode extension should be at ~/.vscode/extensions/ms-vscode.powershell-2.0.0/modules/PSScriptAnalyzer/. So you can try importing from that path, or try Install-Module -Scope CurrentUser PSScriptAnalyzer and then try @bergmeister's suggestions again.
I suspect the empty output you saw before was due to using the PowerShell terminal provided by VSCode and not the PowerShell Integrated Console provided by the PowerShell VSCode extension.

Documenting my steps here:
Install-Module didn't work. I believe it was only available in powershell verison 5.0 and above.
This makes things interesting as I wanted to keep my PS version the same as whats across my network. I couldn't think a way around this so I have manually updated to PS version 5.1.
> Install-Module -Scope CurrentUser PSScriptAnalyzer
NuGet provider is required to continue
PowerShellGet requires NuGet provider version '<IP_ADDRESS>' or newer to interact with NuGet-based repositories. The NuGet provider must be available in 'C:\Program Files\PackageManagement\ProviderAssemblies' or 'C:\Users\he55766a\AppData\Local\PackageManagement\ProviderAssemblies'. You can also install the NuGet provider by running 'Install-PackageProvider -Name NuGet -MinimumVersion <IP_ADDRESS> -Force'. Do you want PowerShellGet to install and import the NuGet provider now?
[Y] Yes [N] No [S] Suspend [?] Help (default is "Yes"):

Again, didn't really want to say yes but I did anyway.
This failed as trying to access the internet and our proxy has blocked it.
Yup, and I can't work out how to manually install it offline

Can you see if PSScriptAnalyzer exists here:
~/.vscode/extensions/ms-vscode.powershell-2.0.0/modules/PSScriptAnalyzer/

Yes its there

What happens when you try to import that path and then run Get-Module?

Here is my results:
PS C:\> cd C:\Users\he55766a\.vscode\extensions\ms-vscode.powershell-1.8.3\modules\PSScriptAnalyzer\1.17.1
PS C:\Users\he55766a\.vscode\extensions\ms-vscode.powershell-1.8.3\modules\PSScriptAnalyzer\1.17.1> import-module
cmdlet Import-Module at command pipeline position 1
Supply values for the following parameters:
Name[0]: psscriptanalyzer
Name[1]:
PS C:\Users\he55766a\.vscode\extensions\ms-vscode.powershell-1.8.3\modules\PSScriptAnalyzer\1.17.1> gmo

ModuleType Version    Name                                ExportedCommands
---------- -------    ----                                ----------------
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Management     {Add-Computer, Add-Content, Checkpoint-Computer, Clear-Content...}
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Security       {ConvertFrom-SecureString, ConvertTo-SecureString, Get-Acl, Get-AuthenticodeSignature...}
Manifest   <IP_ADDRESS>    Microsoft.PowerShell.Utility        {Add-Member, Add-Type, Clear-Variable, Compare-Object...}
Script     0.2.0      PowerShellEditorServices.Commands   {ConvertFrom-ScriptExtent, ConvertTo-ScriptExtent, Find-Ast, Get-Token...}
Script     0.2.0      PowerShellEditorServices.VSCode     {Close-VSCodeHtmlContentView, New-VSCodeHtmlContentView, Set-VSCodeHtmlContentView, Show-VSCodeHtmlContentView...}
Script     1.17.1     psscriptanalyzer                    {Get-ScriptAnalyzerRule, Invoke-Formatter, Invoke-ScriptAnalyzer}

PS C:\Users\he55766a\.vscode\extensions\ms-vscode.powershell-1.8.3\modules\PSScriptAnalyzer\1.17.1>

The Format Document worked.
Closed VSCode and re-opened and ran 'gmo'
The module is not installed.
Does that mean it only installs on that session?

What do you see here?

Does it say "PowerShell Integrated Console" or "powershell.exe"?

Same as you "PowerShell Integrated Console"

Interestingly it appears to be working now. Even after closing app and reopening it. Even after restart

So I guess what I have done to get this resolved is:

Update powershell to version 5.1
Browse to path '~/.vscode/extensions/ms-vscode.powershell-*/modules/PSScriptAnalyzer/' in VSCode
Perform import-module psscriptanalyzer

@rjmholt Should we close the issue then?

@bergmeister yes, I think we've established pretty firmly that this is/was a PSES issue.


Can I do ratelimit based on sub of jwt token?

Can I do ratelimit based on sub of jwt token?
Description:

I would like to do rate limit based on the sub of the jwt token. Every user with valid token will be allowed a maximmum number of limits per second/minute etc.

It is not supported now.   Jwt_authn filter and rate-limiting filter are two independent filters, their data are not shared.
A valid JWT payload is written to dynamic Metata,  so rate limiting filter could potentially pick up that data and use it.

Thanks @qiwzhang . It is working now
making this change in rate limit filter passed the JWT token to rate limit service
  rate_limits:
               - stage: 0
                 actions:
                    - {request_headers:  { header_name: "Authorization", descriptor_key: "token"}}

Change in Rate limit service:
domain: apis
descriptors:
  - key: token
    rate_limit:
      unit: day
      requests_per_unit: 5

I have a similar requirement any idea how we can pass jwt payload scope as a header for ratelimit filter to access

The problem is u can add it as payload but it base64 encoded

Same issue for me. The solution maybe:
Add a envoyfilter lua before ratelimit, we need decode the token by ourselves and add header from token.
Then , do ratelimit.

Yes, istio/envoy already supported this function, and need not use additional envoyfilter.

@zakhenry  is this a local rate limit or global? If local, it's per pod, right...which means as pods scale up, so does the rate limit. Have you been able to implement this at the service level, so it aggregates across pods? Also, when you say "jwt filter", do you mean a separate section of the same EnvoyFilter as the rate_limits block? Or another object entirely? I'm trying to do the same thing.

@jbilliau-rcd it is global as this is is accompanied with defining the rate_limit_service on the filter envoy.filters.http.ratelimit. Your rate limit service is an external service that can front multiple proxies.
The jwt filter bit is https://www.envoyproxy.io/docs/envoy/latest/api-v3/extensions/filters/http/jwt_authn/v3/config.proto


an universal node able to toggle the visibility of its children

Operating system or device - Godot version:
Official 2.0.3stable win64
Issue description (what happened, and what was expected):
Let's imagine I have a branch in my tree that mixes 2D, 3D and Control nodes.
If the parent node is a 3D node and if I hide it, then only 3D children will be hidden too. All other 2D and Control children will remain visible.
It could be handy  to have an "universal" Node that could hide/show its children even if they are of an other type.
Steps to reproduce:
Link to minimal example project (optional but very welcome):

How about Node? :)
More seriously, it sounds like a bug if hiding a parent node does not hide its children, it should not be dependent on their node type IMO.

Yeah, I first thought that Node would be what I needed, but it does not have visibility methods.
Also, I don't think it would wise to add them to it, because Node is used as base class for special nodes like AnimationPlayer, EditorPlugin, SamplePlayer, etc which does not need visibility toggle.
As for wether it is a bug or not, I think it was already working like this in 1.0.

Or maybe, instead of a "universal node", there could be "inherit visibility state from parent node" added to Spatial and CanvasItem ?

Both CanvasItem and Spatial have a NOTIFICATION_VISIBILITY_CHANGED, but
they have different values :p
On 20 June 2016 at 11:30, SuperUserNameMan<EMAIL_ADDRESS>wrote:

Or maybe, instead of a "universal node", there could be "inherit
visibility state from parent node" added to Spatial and CanvasItem ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/5314#issuecomment-227158941,
or mute the thread
https://github.com/notifications/unsubscribe/AGVmPUsbrETUg05c0gtsmk7y8rrOemvxks5qNqP9gaJpZM4I5vbN
.

Hm indeed:

Spatial: https://github.com/godotengine/godot/blob/0dfc4a2029e7a39ce521cc81da94fdaf73341ab2/scene/3d/spatial.h#L133-L140
CanvasItem: https://github.com/godotengine/godot/blob/d4359b2d09625695a972a40ce76faf90b38f4e59/scene/2d/canvas_item.h#L167-L176
Control (commented out, as it inherits CanvasItem, so should be dropped): https://github.com/godotengine/godot/blob/55b83157e70a34a933a2a73f14a0052a832d0287/scene/gui/control.h#L215-L228

Discussed this a bit on IRC with @reduz:
[17:05] <reduz> Akien, i can't see how that is a bug, and much less an useful feature
[17:06] <Akien> reduz: Well if you hide a node, all its children should be hidden no?
[17:06] <reduz> all the children of the same type
[17:07] <reduz> 3D and 2D functions are not supposed to be mixed like this, I think that's more confusing than helping if it happens
[17:08] <reduz> IMO It's safer to assume that there is no relation between 2D and 3D to understand how they work
[17:08] <reduz> and given this is a very very special case, it should be handled as such with a custom script
[17:09] <Akien> I don't have much experience with 3D, so I might be mistaken. I was supposing that one would want Control children of Spatial nodes to be hidden
[17:10] <Akien> e.g. a kind of icon overlay on top of a 3D character
[17:10] <Akien> But I guess such GUI elements don't work like that in 3D :)
[17:10] <reduz> yeah, they are completely unrealted
[17:10] <reduz> you need to put a viewport to show GUI elements in 3D

@SuperUserNameMan Could you give an example of why you want this feature?

@akien-mga :
Sure.
I need to be able to switch between different branches that each contains a different User Interface and a different 3D or 2D content.
Root:Node
    + BranchA:UniversalNode
        + 3dContent:Spatial
        + 2dContent:Node2D
        + UI:Control
    + BranchB:UniversalNode
        + 3dContent:Spatial
        + 2dContent:Node2D
        + UI:Control
    + BranchC:UniversalNode
        + 3dContent:Spatial
        + 2dContent:Node2D
        + UI:Control

Only one branch is visible at a time, and the content of other hidden branch are paused.
None of the branches share a common 3d or 2d content or a common UI.
They only share the root script.
And indeed, my current workaround is to use a simple Node with a custom script :
extends Node

export var visible = true;

func set_hidden(state): #{
	visible = not state;
	for child in get_children(): #{
		if child.has_method("set_hidden"): #{
			child.set_hidden(state);
		#}
	#}
#}

func _ready(): #{
	set_hidden(not visible);
#}

... but it is only useful at runtime. Not into the editor.

@SuperUserNameMan you can add tool keyword on the top, as well as setters/getters to visible (and restart the editor if you are on stable 2.0.x), and then this should work in editor as well.

@kubecz3k : great thanks !
When I tried the tool mode and setget before, I did not think i had to restart the editor... so it did not work.
But now it works :+1:

With the plugin system in 2.1 you can create a custom Node type that has such functionality.

No more nodes please, unless absolutely necessary.

Not sure what next with this issue, from what reduz said, how it works currently is a design decision and not a bug (seems fine for me)... @SuperUserNameMan are you happy with your current approach?

If you only want one of the three branches to be active at a time, why don't you create a node that contains all three as references but only attaches one of them as a child at a time?
As long as you don't have to change the visibility very often it might even be better for the performance to detach nodes from the tree rather than hide them.

@Warlaan : yeah that could be an other solution. I'm not really concerned about performance in this case, but I'll try it if I find that my current approach appears to be less adapted.
@kubecz3k : well, the scripted approach works so far. So I'm not sad.
As for this issue, I did not consider it as bug. The "universal node" was a feature proposal.
If i'm the only creature of the universe to think it could have been a handy feature to add, I'm okay with that.

@SuperUserNameMan Well I don't think you are the only one, but since only some smaller % of projects will need to use this I think it's great candidate for small yet nice plugin for upcoming asset sharing platform :)

Use Node2D as a parent for 2D components and Spatial as a parent for 3D components and don't mix 2D and 3D. It will work fine.   Also as a general rule I don't use generic Node as a parent to 2D or 3D nodes because Node is pretty limited and generic misses position or rotation can't be translated etc.

@kubecz3k thanks, I was looking for this feature and was about to open an issue.
I tried the same approach as SuperUserNameMan but did not restart the editor.
Is it a bug that a print inside the setter is not showing in the output(tool/editor mode)?

@AlexHolly I think it's more a design decision than something else. Know for sure that CoreDevs are aware of this fact and it will be addressed at some point. Think I will close this issue, since we all know how to do this stuff at this moment, and with release of 2.1 it will be possible for community to create very simple plugin and share it with ease with other users :)


Cannot scroll terminal on Windows with libertyRun and libertyDev

By default, Gradle has a continuously refreshing progress bar in the console.  On Windows, this prevents you from scrolling up in the terminal.  This particularly affects long running operations like libertyRun and libertyDev.
For libertyRun, you could still look at the server logs.
For libertyDev, it has additional output like compile result messages and test result messages that will not be accessible once they are scrolled out of the screen.
The workaround is to run Gradle with --console=plain, or add org.gradle.console=plain to gradle.properties e.g. in the project directory or Gradle user home directory.

FYI, @cherylking.

We should investigate and come up with a plan to address or mitigate the effects of this, even if this is a Gradle limitation.

I think we should just document something on https://github.com/OpenLiberty/ci.gradle/blob/master/docs/libertyDev.md to mention the --console=plain workaround


Update config_automation.yml

Purpose/implementation Section
What changes are being implemented in this Pull Request?
I saw this comment: https://github.com/fhdsl/NIH_Data_Sharing/pull/92#issuecomment-1449296689
And was like, hmm the image was updated on dockerhub and emojifont should be there. Then I realized this course has its config set to use the old name version of the docker image. How many other courses are also doing this?

Purpose/implementation Section
What changes are being implemented in this Pull Request?
I saw this comment: #92 (comment)
And was like, hmm the image was updated on dockerhub and emojifont should be there. Then I realized this course has its config set to use the old name version of the docker image. How many other courses are also doing this?

ooh that makes sense - thanks for noticing that!

Yeah there might be quite a few...


Lot of errors at Django S3 example

The first error I've got was at line 54: request_payload = json.loads(request.body), saying that the parameter must be 'str', not 'bytes'. I've fixed it writing this instead: request_payload = json.loads(request.body.decode()).
Then I've got one at line 113: policy = base64.b64encode(json.dumps(policy_document)). It said that a bytes-like object was required, not 'str'. So I changed that line to policy = base64.b64encode(json.dumps(policy_document).encode()), and it seemed to be fixed.
Now I'm getting an error at the next line, 114: signature = base64.b64encode(hmac.new(settings.AWS_CLIENT_SECRET_KEY, policy, hashlib.sha1).digest()).  At some point of the hmac.new function I get this: key: expected bytes or bytearray, but got 'str'.
I'll try to fix it, but please tell me if I'm doing something wrong, maybe there's a better way of doing it.

Now I've converted from byte to str or backwards in every part of the code where I've got errors, but the signature is not correct, because I get a 403 (Forbidden) response from my bucket.

I don't have any working knowledge of Django. If you are seeing clear issues with the code, a pull request fixing the issues with explanations of the errors and fixes would be much appreciated. Any other issues you see that you are not able to immediately fix can be discussed in that PR as well.

@andresespinosapc did you find a solution?

It sounds like you are using Python 3 and the example is for Python 2

In my case I had to use v4 signature to make the signing work. I've modified the signature functions in the Django-s3 folder as follows
import base64, hmac, hashlib, json, sys, binascii

# Key derivation functions. See:
# http://docs.aws.amazon.com/general/latest/gr/signature-v4-examples.html#signature-v4-examples-python
def sign(key, msg):
    return hmac.new(key, msg.encode('utf-8'), hashlib.sha256).digest()

def getSignatureKey(key, dateStamp, regionName, serviceName):
    kDate = sign(('AWS4' + key).encode('utf-8'), dateStamp)
    kRegion = sign(kDate, regionName)
    kService = sign(kRegion, serviceName)
    kSigning = sign(kService, 'aws4_request')
    return kSigning

def sign_policy_document(policy_document, client_secret):
    """ Sign and return the policy doucument for a simple upload.
    http://aws.amazon.com/articles/1434/#signyours3postform
    """
    # client_secret = bytearray(client_secret, 'utf-8')
    dateStamp = get_condition(policy_document['conditions'], 'x-amz-date')
    dateStamp = dateStamp[:8]

    import logging
    logger = logging.getLogger(__name__)
    logger.error(client_secret)

    # london
    regionName = 'eu-west-2'
    serviceName = 's3'
    policy = base64.b64encode(
      bytearray(
        json.dumps(policy_document), 'utf-8'
      )
    )
    sigKey = getSignatureKey(client_secret, dateStamp, regionName, serviceName)
    signature = sign(sigKey, policy.decode('utf-8'))

    return {
        'policy': policy.decode("utf-8"),
        'signature': binascii.hexlify(signature).decode('utf-8')
    }


Twoslash output is inaccessible

Hey there 👋 I noticed that currently TwoSlash produces inaccessible output.
This issue is not a comprehensive accessibility audit, but I wanted to draw attention to some key issues that would need addressing to make TwoSlash code blocks usable by more users.
The references here are to Web Content Accessibility Guidelines (WCAG) 2.2
Key accessibility failures
Guideline 2.1 Keyboard Accessible

All functionality of the content is operable through a keyboard interface without requiring specific timings for individual keystrokes

Twoslash tooltips are only available via mouse/pointer hover, so fails most of WCAG Guideline 2.1 regarding keyboard accessibility: it should be possible for users to access the content of tooltips using only the keyboard.
Guideline 4.1 Compatible

Maximize compatibility with current and future user agents, including assistive technologies.

Twoslash tooltips are not available to screen readers: navigating a code sample using Twoslash, there is no way for screenreaders to access the content of tooltips.
Other details
There are some smaller things like the following, but these are kind of moot given the bigger issues:
1.4.13 Content on Hover or Focus

A mechanism is available to dismiss the additional content without moving pointer hover or keyboard focus, unless the additional content communicates an input error or does not obscure or replace other content;

Tooltip content is not dismissible via a mechanism other than moving the pointer’s hover position.

I don't know if it's an explicit project goal that Twoslash output must not include JavaScript, but the library currently relies entirely on CSS for functionality. As this issue outlines, that makes the content of tooltips inaccessible to users who navigate via keyboard or screen readers.
How possible would it be to keep the current CSS-only foundation that is currently in place, but use the forthcoming Popover API to provide a more accessible experience in browsers that support it? There could be some challenges matching the hover behavior with the Popover API without additional JavaScript to implement imperative popover toggling, but 1) encoding the tooltips in the markup would support screen readers and 2) JavaScript would likely be necessary to support the roving tabindex pattern anyway.

I think this energy is probably worth putting into the new version of twoslash which might have more of a way to customize how the twoslash annotations are turned into HTML due to it being a shiki transformer: https://github.com/twoslashes/twoslash

I think this energy is probably worth putting into the new version of twoslash which might have more of a way to customize how the twoslash annotations are turned into HTML due to it being a shiki transformer: twoslashes/twoslash

Ah, thank you! I missed the note in the README about the new repo. Moved my issue over to: https://github.com/twoslashes/twoslash/issues/32


Add deceased vs age graph in deep dive section.

Please add a graph for number of people deceased vs their age in the deep dive section.

I would very much like to work on this issue, but I checked the API for deaths. It doesn't have the age of all dead people.

Duplicate issue (https://github.com/covid19india/covid19india-react/issues/967) and PR for this issue were already made.

Duplicate of #967


use correct types for webSocket

I'm not 100% sure about the correct place to address https://github.com/cloudflare/workerd/issues/2723. Appreciate some feedback.
Fixes https://github.com/cloudflare/workerd/issues/2723

Hmm although this was merged and seems to be included in v1.20240919.0, I'm still seeing the old type in<EMAIL_ADDRESS>


Error reinstalling bash - Error 0x80070005

Hi guys,
I've been trying bash for a couple of weeks and I literally 'Messed around' with it to test my full Linux stack and whether it could work with my workflow or not.
I've got to the point, though, when I broke too many things therefore I decided to reinstall it from scratch.
lxrun /uninstall worked as expected but when I now run lxrun /install, this is the output I get
C:\WINDOWS\system32>LxRun.exe /install
-- Beta feature --
This will install Ubuntu on Windows, distributed by Canonical
and licensed under its terms available here:
https://aka.ms/uowterms

Type "y" to continue: y
Error: 0x80070005

Also tried to do the same thing opening bash directly but when I type y the bash window disappear!
Thanks in advance and congrats for the amazing job you're doing!

Try lxrun /uninstall /full /y and try installing via bash.exe?

Hi @fpqc, tried it already! No joy :(

https://github.com/Microsoft/BashOnWindows/issues/410#issuecomment-221463487 maybe!
https://github.com/Microsoft/BashOnWindows/issues/453

@davidedimauro88 - Which Windows build are you on?

@davidedimauro88 - Which Windows build are you on? Can you try this with Windows build 14352?

@sunilmut - I'm on the 14352 already!

@davidedimauro88 - Sorry to hear that it is still not working for you. Can you try these steps and see if that helps?

lxrun /uninstall /full /y
Reboot
Open an admin prompt or through explorer, delete all the content under the %localappdata%\lxss directory.
Install using bash.exe or "LxRun.exe /install"

Hi @sunilmut - it worked!!!
Awesome, thanks so much!

C:\Users\pstejska\AppData\Local\lxss\home\pstejska\android>rmdir /s /q .
.\external\chromium_org\third_party\jsoncpp\overrides\src - The directory is not empty.
.\external\chromium_org\third_party\skia\gm\rebaseline_server\testdata\outputs\expected\compare_rendered_pictures_test.CompareRenderedPicturesTest.test_endToEnd_withImageBaseGSUrl\compare_rendered_pictures.json - The file name is too long.
.\external\chromium_org\third_party\webrtc\modules\audio_processing\aecm - The directory is not empty.
.\external\chromium_org\v8\third_party - The directory is not empty.
.\external\clang\test\Driver\Inputs\basic_linux_libcxx_tree\usr\include\c++ - The directory is not empty.
.\external\clang\test\Driver\Inputs\fedora_18_tree\usr\lib\gcc - The directory is not empty.
.\external\clang\test\Driver\Inputs\mips_cs_tree\lib\gcc\mips-linux-gnu\4.6.3\include-fixed\mips16\soft-float - The directory is not empty.
.\external\clang\test\Driver\Inputs\mips_fsf_tree\mips-mti-linux-gnu\lib\mips64\fp64 - The directory is not empty.
.\external\clang\test\Driver\Inputs\mips_fsf_tree\sysroot\mips32\el\fp64\usr - The directory is not empty.
.\external\clang\test\Driver\Inputs\ubuntu_13.04_multiarch_tree\usr\include\x86_64-linux-gnu\c++\4.7 - The directory is not empty.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.pde.api.tools.ee.cdcfoundation10_<IP_ADDRESS>901281111\org\eclipse\pde\api\tools\internal\api_stubs\CDC-1.0_Foundation-1.0.zip - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.pde.api.tools.ee.cdcfoundation11_<IP_ADDRESS>901281111\org\eclipse\pde\api\tools\internal\api_stubs\CDC-1.1_Foundation-1.1.zip - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.pde.api.tools.ee.javase16_<IP_ADDRESS>901281111\org\eclipse\pde\api\tools\internal\api_stubs\JavaSE-1.6.zip - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.pde.api.tools.ee.osgiminimum10_<IP_ADDRESS>901281111\org\eclipse\pde\api\tools\internal\api_stubs\OSGi_Minimum-1.0.zip - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.pde.api.tools.ee.osgiminimum11_<IP_ADDRESS>901281111\org\eclipse\pde\api\tools\internal\api_stubs\OSGi_Minimum-1.1.zip - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.pde.api.tools.ee.osgiminimum12_<IP_ADDRESS>904031014\org\eclipse\pde\api\tools\internal\api_stubs\OSGi_Minimum-1.2.zip - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.pde.build_3.6.1.R36x_v20100823\data - The directory is not empty.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\model\BuildResultsProperties.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\model\ComponentResultsElement.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\model\ConfigResultsElement.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\model\PerformanceResultsElement.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\model\ScenarioResultsElement.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\ui\PerformanceResultsPerspective.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\ui\PerformanceResultsPreferenceInitializer.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\ui\PerformanceResultsPreferencePage.java - The file name is too long.
.\external\eclipse-basebuilder\basebuilder-3.6.2\org.eclipse.releng.basebuilder\plugins\org.eclipse.test.performance.ui\src\org\eclipse\test\internal\performance\results\utils\IPerformancesConstants.java - The file name is too long.
.\external\google\market_licensing\sample\src\com\example\android - The directory is not empty.
.\external\libunwind\aux - The system cannot find the file specified.
.\external\linux-tools-perf\perf-3.12.0\arch\hexagon\include\uapi - The directory is not empty.
.\external\okhttp\android\test - The directory is not empty.
.\external\square\dagger\compiler\src\it\default-package-injected-type\src\main - The directory is not empty.
.\external\square\dagger\compiler\src\it\inject-parameterized-type\src - The directory is not empty.
This seems to be why. The %appdata%\home\username\android\external has this problem.

@davidedimauro88 - Good to know that your issue is resolved. We had a bug in WSL that was leaking handles to directory. During bash instance start, we would try to delete these directories and that would fail (because of leaked handles) and cause the instance to not start. This has been fixed in our dev branch and should hit the release branch soon.

Reopening this until the fix hits the insider builds.

@sunilmut thanks for the update and the help! will be waiting for the new build then.
Thanks again

This should be fixed in Build 14371 and probably even earlier than that.  If possible @davidedimauro88 could you let us know if this is resolved for you?

Hey @benhillis, I just tried to replicate the issue again and it seems to be fixed now. Thanks a lot!

@davidedimauro88 Thank you very much for confirming!

This error happened to me after the latest update on the release branch OS Build 14393.693 The instructions provided by @sunilmut fixed it, but requires losing all of the data which is not acceptable solution for most people:
lxrun /uninstall /full /y
Reboot
Open an admin prompt or through explorer, delete all the content under the %localappdata%\lxss directory.
Install using bash.exe or "LxRun.exe /install"

@NoFlames - You should also just be able to remove %localappdata%\lxss\temp

I am reopening this issue. Before using the workaround, please collect traces as we would like to get to the bottom of this issue. Agreed with @NoFlames that lostig user data is not an acceptable solution.

@sunilmut - If it helps, this started happening to me after I modified the file permissions so I could see hidden files.  After I reverted that change, the issue no longer existed - so it may be an issue with windows modifying file attributes in a way that the bash shell doesn't like.

@sunilmut unfortunately this procedure doesn't work on my system, still getting error 0x80070005
It worked well until yesterday, I don't know what has changed...
My build is 15063.540 (Windows 10 1703)

@lornova Had the same problem after I made the mistake of trying to access WSL files from Windows-Explorer.  After manually deleting these from an elevated-CMD window, I was able to completely delete %appdata%\lxss. After that, running Bash.exe worked fine.

@lornova had the same issue after updating and rebooting Windows (I don't think updating had anything to do with it). After doing @sunilmut procedure (which was very hard due to a lot of files having excessive path names) and extra cleaning %localappdata%\lxss I was able to reinstall using bash.exe. Last week before rebooting I used VSCode to edit some files on lxss which eventually broke access from Ubuntu, it looks like I broke the filesystem. From now on I won't edit lxss files from Windows.

Closing this issue as it is stale. If you come across this again we will re-open and repro.


s/member_round_trip_time/peer_round_trip_time/

/cc @brancz
The example alerts make reference to etcd_network_member_round_trip_time_seconds_bucket, but the codebase only contains etcd_network_peer_round_trip_time_seconds_bucket (in rafthttp/metrics.go).   Thanks!
per https://travis-ci.org/coreos/etcd/jobs/319104998#L879

Yes as it is it's a mistake, thanks for noticing @tomwilkie! Not sure how that got in there, seems like that metric never existed :sweat_smile:.


[WIP] Visiting Controllers Monitoring Tool

Resolves CORE-45

Tests perhaps pending - trying to keep the work below the estimate.

@AxonC I've made some minor amends.
However, some of this code is very heavy on DB usage. My dev env has 1 user with 500 rows of networkdata_atc and the code executed over 1000 queries on this page.
Can we look to optimise please?


dnscrypt-proxy IPv6 Windows 10 problems

Hi,
I want to use dnscrypt-proxy with IPv6 on Windows 10 x64. I have followed your Configuration guide, however I am encountering multiple problems with IPv6:

When I test C:\Program Files (x86)\bitbeans\Simple DNSCrypt\dnscrypt-proxy>.\dnscrypt-proxy.exe dnscrypt-proxy.conf with IPv4 settings, everything succeeds, but when I change to IPv6 I am not able to connect by any means. Please, see below:

When I change config to LocalAddress [::FFFF:<IP_ADDRESS>]:53, I get following error: [ERROR] Unable to bind (UDP) [Cannot assign requested address [WSAEADDRNOTAVAIL ]], despite that I am running COMMAND with elevated privilages.

When I change config to LocalAddress ::FFFF:<IP_ADDRESS>:53 (without brackets), I am getting: syntax error line 70: [:FFFF:<IP_ADDRESS>:53]

When I change config to LocalAddress [::1]:53, and invoke any IPv4 resolver, everything seems to work:
04/22/17 14:22:00 [INFO] + DNS Security Extensions are supported
04/22/17 14:22:00 [INFO] + Namecoin domains can be resolved
04/22/17 14:22:00 [INFO] + Provider supposedly doesn't keep logs
04/22/17 14:22:00 [NOTICE] Starting dnscrypt-proxy 1.9.4
04/22/17 14:22:00 [INFO] Loading plugin [C:\Program Files (x86)\bitbeans\Simple DNSCrypt\dnscrypt-proxy\libdcplugin_example_cache.dll]
04/22/17 14:22:00 [INFO] Loaded plugin: [A basic DNS cache]
04/22/17 14:22:00 [INFO] Generating a new session key pair
04/22/17 14:22:00 [INFO] Done
04/22/17 14:22:00 [INFO] Server certificate with serial #12345 received (removed for security reasons)
04/22/17 14:22:00 [INFO] This certificate is valid
04/22/17 14:22:00 [INFO] Server key fingerprint is XXXX: (removed for security reasons)
04/22/17 14:22:00 [NOTICE] Proxying from [::1]:53 to XXX.XXX.XXX.XXX:PORT (removed for security reasons)
but
are not resolving IPv6 addresses. They do resolve names to IPv4 addresses, but not IPv6 addresses or IPv6 names.

a) When I change config to LocalAddress [::1]:53, and invoke any IPv6 resolver, I get every time (with multiple different resolvers tried), following error:
04/22/17 14:25:48 [INFO] + Provider supposedly doesn't keep logs
04/22/17 14:25:48 [NOTICE] Starting dnscrypt-proxy 1.9.4
04/22/17 14:25:48 [INFO] Loading plugin [C:\Program Files (x86)\bitbeans\Simple DNSCrypt\dnscrypt-proxy\libdcplugin_example_cache.dll]
04/22/17 14:25:48 [INFO] Loaded plugin: [A basic DNS cache]
04/22/17 14:25:48 [INFO] Generating a new session key pair
04/22/17 14:25:48 [INFO] Done
04/22/17 14:26:03 [ERROR] Unable to retrieve server certificates
04/22/17 14:26:04 [INFO] Refetching server certificates
04/22/17 14:26:19 [ERROR] Unable to retrieve server certificates
04/22/17 14:26:22 [INFO] Refetching server certificates
And it never connects. Despite that its corresponding IPv4 server connects instantly.
I tried also to manually specify from .cvs file
ProviderName
ProviderKey
ResolverAddress
also to no avail.

I am wondering what am I doing wrong, or what I set up wrong or did not set up, but should in config file.
I have well working IPv6 and have no issues with it otherwise, but I am unable to get it working with dnscrypt-proxy.
Any advice will be greatly appreciated.
Sincerely,
Zenith-Nadir

[::FFFF:<IP_ADDRESS>] is just a different way to write <IP_ADDRESS>, i.e. this is an IPv4 address. If you already have something listening to <IP_ADDRESS>, you can't have something else listening to the same IP.
You don't need to have the proxy listen to a local IPv6 address if it's already listening to an IPv4 address. Just use [::FFFF:<IP_ADDRESS>] as the resolvers address to use for IPv6 queries in the Windows settings.
The resolvers themselves don't care about the type of IP address clients connect to. They will respond to "what's the IPv6 address for that name" even if queries are sent using IPv4. So, if a resolver is reachable using both IPv4 and IPv6, using IPv6 is often just slower and less reliable.
Unable to retrieve server certificates means that a regular, standard, non-encrypted DNS query sent to that resolver never got a response. This happens even before anything is sent using DNSCrypt. Check that your router/firewall/antivirus properly support IPv6 and don't block port 443.

Hello jedisct1,
Thank you for such a quick response, and your interest to assist with my issues. THANKS!
Regarding ::FFFF:<IP_ADDRESS>, I know it is an old transition IPv4 to IPv6 notation.
My machine responds to pings to ::FFFF:<IP_ADDRESS> and displays <IP_ADDRESS>
Otherwise netstat -an does not show anything listening on ::FFFF:<IP_ADDRESS> address, neither ::1, there are several instances listening on <IP_ADDRESS>, but none on port 443, only one on port 53 while running dnscrypt-proxy, and none without dnscrypt-proxy running, but this does not seem to cause any problems anyway.
When I test dnscrypt-proxy using LocalAddress <IP_ADDRESS>:53, I get the same results: IPv4 responses are instant from responsers using :443 port (nothing blocked), but when I specify any IPv6 responders all of them are Unable to retrieve server certificates.
Am I missing something? If I query IPv4 or IPv6 responders on <IP_ADDRESS> shouldn't they respond in IPv4?
Also, I specifically chosen :443 specified responters and IPv4 do respond, and all IPv6 are: Unable to retrieve server certificates.
Where am I making mistake?
I will try to disable all firewalls, but this will have to wait till tomorrow or later.
Thank you very much again for your assistance.
Sincerely,
Zenith-Nadir

Hello jedisct1,
It appears it was my router's ip6tables firewall allowing only udp port 53 for DNS.
Turns out that I had to permit not only port 443, but any port remote resolver is listening to.
Since, this does not seem to be any standard, normalized port, and there are resolvers listening on different ports (1053, 2053, 5353, 5443, etc), I had to allow those ports to be able to get resolvers to connect.
Thanks for your assistance and hints.
This would be very helpful for troubleshooting, if mentioned in Configuration github web page, and not only 443 port, but others, too. Especially for strict ip6tables or other firewalls.
Thanks again
Very Sincerely,
Zenith-Nadir


Readme & license

Took a stab at a basic readme and added an MIT license. Look good?

:+1: This is awesome :smile:


While testing I cant see the jquery correctly working on Firefox nor Chrome

Hi! Im just wondering why I can't see the popdown effect while testing my website on Firefox and Chrome? I can only see it on IE. I've tried many things changing the order of the scripts and re checking they are correctly linked but I get the same result: the window shows up but it loops on the loading stage and doesn't show the content.

I hope this is not a silly issue.
Thank you very much for your time.
CPE. María Inés Calderón Zetter.

Hello,pleass do element inspect and share the console screenshot, the backend maybe returning internal server error (500). Or you can also try to visit the link first without Popdown and see if it shows up.


Edit README.md

Separate URL to apksigner documentation from command
Capitalize APK in sentence
Add link to CONTRIBUTING.md
Turn into code block
Delete commented-out lines

Great, but can you revert the code block for the verification info of AppVerifier? I think it is more readable that way.

Ok, I've made the requested change.

thanks


[Language]: [User Registration] : [Change 'Door number to Door number/House number']

Describe the bug
It is showing Door number only when we are registering a citizen
*
Expected behavior
Change 'Door number to Door number/House number
Screenshots

Hi @devarajd94, can you please confirm this.


Imba todomvc benchmark is cheating :(

(Posted this comment on HN, but figured I'd copy it here for discussion.)
I just spent an hour looking at the Imba benchmark. Yep, it's cheating. Which is a shame, because I really like the framework as a whole.
The vast majority of the speedup comes from a single sneaky line of code. The majority of their "Everything" benchmark's time is spent in the reorder step. They've implemented this as "remove a random todo, render, push the removed todo back onto the end, render." The Imba implementation, and it alone, caches the rendered todo view, so that they can re-use it once the todo is reinserted.
This single optimization is responsible for the vast bulk of their claimed speed. Removing it puts Imba only 2x faster than React, not 60x.
If you want to try it yourself, look at line 55 of app.js. Change:
    res.push((this['_' + todo.id] = this['_' + todo.id] || t$('todo')).setObject(todo).end());

... to:
    res.push(t$('todo').setObject(todo).end());

Furthermore, this isn't a caching strategy you'd want to use in a real app. It holds onto all DOM nodes ever created, thereby leaking quite a lot of memory.
Again, I think Imba is cool, and fast, just not otherworldly fast. I hope this was just an "oops!" and not an intentional misrepresentation.

Thank you for taking the time to read through the code and trying to understand. What you did is turn caching off completely. I'm actually very suprised that Imba could still be faster in that case, I would have predicted it to be much slower than React and Mithril, as it doesn't reuse DOM nodes at all.
It is true that creating a million todos would create a leak in Imba, unless you manually prune the cache (for now). I will answer this in more detail, but before that, I suggest you change the line to:
res.push((this['_' + i] = this['_' + i] || t$('todo')).setObject(todo).end());

With this change you will only ever cache as many nodes as there are visible tasks, yet not remove caching all together. This is not how you would usually do it in Imba though, the 'correct' way would be the way it is done in the benchmark.

If you had an app where you expected to have millions of todos I agree that you would not want to cache all nodes if they have been shown at least once. In those cases you would be much better off by doing  something like this. But again, the benchmark is mostly trying to measure the time it takes to render the whole app when nothing has changed (in which case the this['_' + i] caching would be just as fast (and not create any additional tags). It just looked a bit boring when the benchmark appears to do nothing at all.

I have now actually commited this change to the benchmark. You are right that it might actually be better to not keep cached nodes for individual tasks hanging around. This is even simpler. Do you still think this is sneaky? If not, I'm happy to tell you that the performance actually increases on all benchmarks but 'everything', where it is the same. So 50x faster than React on my mbp here.

@curveship Do you have any other comments regarding this? Otherwise I guess I should close the issue, but I do want to make sure we're seeing eye to eye :)

Hi Sindre -- Sorry for the slow reply.  We're under a crunch at work right now.
Also I, too, need to apologize for coming across as unduly harsh on HN.  "Cheat" and "sneaky" have a usage in benchmarks, but they also carry a moral accusation.  I meant much more the former than the latter, but that wasn't clear enough.
I've been looking more at Imba, and I continue to find cool stuff.  I've been playing around with my own framework, and there's a lot of food for thought in Imba.  I also understand better why you were frustrated by my removal of caching altogether.  You've gone to great lengths with Imba to separate out the minimum work needed to do an update, and removing all caching defeats those efforts.  So no caching isn't "the Imba way."  At the same time, tuning the caching strategy to the benchmark doesn't seem totally fair either.  It means that the results aren't representative of behavior in general.
My take on what would make the benchmark more "fair":

only hold onto the current live todo views.  You have to admit that holding onto the rendered todo views when the original todo has been removed from your model (which usually means "gone from the universe") seems like it's tuned to the particular behavior of your benchmark, where such a todo suddenly reappears.  This is a more conservative strategy than what you've got right now (hold onto max number of todos), but I think it makes a greater case for Imba's speed.  I'm much more impressed by someone saying "we're 8x faster with a conservative implementation" than "we can tune to the benchmark and get 60x."

there should be a benchmark for rendering genuinely new todos.  DOM creation is an important part of a framework and should be tested.  (Also, you're really fast here, though not 60x, so you might as well show that.)

if you're going to use Imba's performance strategies (caching and minimum update work), then you should be fair and use React's as well (shouldComponentUpdate).  I don't follow the argument that you had to slow down React to be fair to Mithril (?), especially when Mithril has its analogue, { subtree: 'retain' }.

I did these changes with my clone of the benchmark and came up with Imba being about 7x faster than React (Chrome/win7).  I think that's a) awesome, and b) probably more indicative of real world performance.
More generally, I have a couple comments on the benchmark as a whole:

I'm not sure I follow the value in a "full update" benchmark.  We usually need two things: render the initial full view, and update as necessary based on changes.  What's the need for testing full update speed?

I liked that the original todomvc benchmark tested render and event-handling time as well.  Different frameworks interact with the DOM more or less efficiently.  The React folks claim that they do some expensive JS calcs (like Levenshtein distance array mutations) in order to minimize DOM mutations, as that produces an overall faster experience. You're effectively penalizing them for that here.

Again, I think Imba is very cool, even if I'm critical of this benchmark and its relevance to Imba's speed in general usage.
(If you're curious: you beat my framework by 2x.  I'm faster on rename and toggle, but you're 3x faster on reorder.  My algorithm for reconciling a node's children is still fairly naive.)

@somebee do you plan to address the points made by @curveship on Aug 22nd?
I'm interested in your response, specifically to "being fair to React"

if you're going to use Imba's performance strategies (caching and minimum update work), then you should be fair and use React's as well (shouldComponentUpdate). I don't follow the argument that you had to slow down React to be fair to Mithril (?), especially when Mithril has its analogue, { subtree: 'retain' }.

Hey. I haven't taken the time to respond to @curveship yet, but thanks for reminding me @jrschumacher.
I think most of the discussion here revolves around the fact that we still have crazy bad documentation and no real good description of our caching, why it is so fast, and why it matters for real world applications. The docs/site will be improved a lot in a week or two :) "Unchanged render" was originally the only benchmark, and still the only one I find enlightening, but since nothing happens visually while it runs, I decided to add a few that were a bit more 'visual'. Looking back, these should probably never have been added at all. I see that I still have work to do on explaining why this is the case.
I'll start with a comment I made over at the react discussion forums.

First things first, I had (amateurishly) absolutely no idea how fast Imba would be compared to regular virtual dom implementations. The way we do rendering in Imba was made long before we ever started thinking about benchmarking anything, and the TodoMVC app was made as a reference implementation, and then benchmarked.
The optimizations are extremely simple. but very difficult to do in React, since they require caching to happen inline in the code where you generate the components (see code). Since React is not intended to require JSX, it is not feasible to ask js-developers to manually cache all nodes etc. The approach is very different from React.
I am absolutely sure that the React way is better in many ways. It is easier to mess things up in Imba if you don't fully understand how the caching works. Since Imba does not do any 'intelligent' virtual dom diffing it would vastly decrease performance if you start mixing uncached nodes into these views. Imba really lacks documentation now, but we are working on it. I am confident that the performance difference is in fact real, and not only in special benchmarks, but in real world applications. I don't expect to convince any of you today, but I'm hoping to improve the documentation and get more examples out there to show how all this works. If you have an application which would create thousands of new views all the time, you will manually need to prune the cached nodes from time to time. We are working on automated ways to do this, but in all our projects we have developed using Imba over the last few years, we have never had a real world need for this.

Only hold onto the current live todo views

You have to admit that holding onto the rendered todo views when the original todo has been removed from your model (which usually means "gone from the universe") seems like it's tuned to the particular behavior of your benchmark, where such a todo suddenly reappears.  This is a more conservative strategy than what you've got right now (hold onto max number of todos), but I think it makes a greater case for Imba's speed.  I'm much more impressed by someone saying "we're 8x faster with a conservative implementation" than "we can tune to the benchmark and get 60x."

This is a valid point, and I did address the concern, which was the fact that the 'Everything' benchmark at one point removes a task and forces a rerender before appending it again. This was unintentional (and has been corrected). The change does not really affect the results.
There should be a benchmark for rendering genuinely new todos.

DOM creation is an important part of a framework and should be tested.  (Also, you're really fast here, though not 60x, so you might as well show that.)

You are more than welcome to add a benchmark that creates tons of tasks, but IMHO it would tell us very little of value. I've stated multiple times that I think all these frameworks are plenty fast if you only render on changes, and make sure to only render/resolve the changed parts. A benchmark for inserting tasks would to a much larger degree test the performance of DOM creation (in the browser). Imba is still much faster here, but how interesting is it to see how long it takes to insert 10 million tasks? Let's say you are organising your todos (in the real world) for 10 minutes, using a task manager. During these 36000 frames (assuming 60FPS) - how many dom insertions do you think were spent per frame on average? This is the type of micro benchmark that says very little about performance in real apps.
The 'Unchanged render' benchmark says a whole lot more. What it shows is that rerendering the whole view is now so inexpensive that it is possible to render on every single frame, without any need for listeners, bindings, dependency tracking etc. Keeping the view 'in sync' is no longer an issue. It is like rendering templates on the server - where you always know the whole state/story on render.
shouldComponentUpdate

if you're going to use Imba's performance strategies (caching and minimum update work), then you should be fair and use React's as well (shouldComponentUpdate).  I don't follow the argument that you had to slow down React to be fair to Mithril (?), especially when Mithril has its analogue, { subtree: 'retain' }.

React and Imba can both optimize using shouldComponenUpdate (in Imba you would add the same logic inside node#commit). Utilising shouldComponenUpdate (in React and Imba) requires additional logic by the developer, not the framework. This is not an optimisation at the framework level - but an endpoint where the developer can write their own unique optimisations. Adding your own checks around the app for whether any state has changed is inelegant, cumbersome, and bug prone. In my opinion it defeats lots of the purpose of any virtual dom, if you are expected to do your own diffing before sending it through.  Also, the relative performance difference is about the same when turning it on, and since the original Mithril TodoMVC does not use subtree: retain, I have not taken the time to implement it there.
In addition, the way shouldComponentUpdate was implemented in the TodoMVC example (see here) it does not rerender unless the whole task object has been replaced. So in the original react example you would need to create a new task-object whenever you wanted to change anything on a task (title, completed-state etc). The lowlevel actions of this benchmark changes properties like task.title directly.
full update

I'm not sure I follow the value in a "full update" benchmark.  We usually need two things: render the initial full view, and update as necessary based on changes.  What's the need for testing full update speed?

Yes, but how do we know what to update, based on what changes? Are you showing the time elapsed since a task was created somewhere, do you have complex conditions that control whether a certain view is displayed or not? Frameworks solves this in many different ways, and most of them require the use of listeners, bindings, and all sorts of tracking - that the developer has to deal with (like shouldComponentUpdate). From my own experience, it is very easy to introduce bugs where the view is not fully in sync with the state of the application. This is in fact one of the main advantages of React - that the 'correct' way to sync the view is to call render on the top-level node, and let React take care of only rendering (as in touching the DOM) where there are actual changes. This makes development much easier, since you can practically call render whenever you know that something has changed. When your app grows you still need to add shouldComponentUpdate guards etc to keep from resyncing everything, because the virtual dom is not blazingly fast even when it does not touch the dom. My claim is that Imba is so much faster at this, that you don't need any logic to keep parts of the app from rendering, nor do you need to track when things change. You can render every frame. Think about how you would write a server side template in a synchronous language/framework. It is very easy to make sure the whole view is in sync.

@somebee thanks for linking to that response as well as your other points.
Side note: have you seen vuejs' performance comparison update http://vuejs.org/perf/. Seems like they had similar issues

@jrschumacher Thank you for linking to that :) To clarify for others who might read this.

Due to internal implementation differences, frameworks that uses async rendering (e.g. Vue, Om, Mercury) gains the advantage by skipping part of the calculations that happened in the same event loop. The real world user experience doesn’t demonstrate such dramatic difference.

This was one of my main concerns with the mentioned benchmark as well. In this benchmark everything happens synchronously, and all implementations are forced to do the same amount of synchronous updates. This can be seen by the counter in the title (Todos #number#) which is incremented on every full render.  As we state in our todomvc-render-benchmark readme.md:

There has been a TodoMVC benchmark floating around earlier. It mainly tested the performance of your browser, by creating fake events and navigating the dom to insert, complete, and remove todos. Some frameworks such as elm and mithril outperformed React by a lot. The (only) reason for this was that they used asynchronous rendering, while React rendered / synchronized the whole view every time a todo was inserted, completed, and removed. Making rendering in React asynchronous would take 2 lines of code, and bring its performance up to the others.

Just curious @somebee what your take is on React's design decision to not make rendering asynchronous and faster? Most frameworks tout their superiority to React in terms of speed, yet, here the claim is React is purposefully keeping things slower, but to someone like me, I can't wrap my head around why. I know React is backed by the geniuses at facebook, so I've been wondering about this ever since I've been shopping around for a framework to use for my next project...

@unabst Thankfully, React has not made such a design decision, only the React implementation of TodoMVC. So if you use React, it is up to you whether you want to do rendering asynchronously (usually with requestAnimationFrame) or force synchronous rendering whenever something has changed. The same goes for Imba. In my own projects I usually render the whole application on every frame, but you could render as a reaction to certain events etc.

Ah, thank you. It has to do with the benchmark then, which makes much more sense. I am just curious, is there anything that would stop facebook from using Imba for the front end? Is there something about React that makes it worth the weight and complexity in certain scenarios (like say, for serving facebook content)? I guess I should really be reading the React docs, but they seem bias towards React :)

Is there something about React that makes it worth the weight and complexity in certain scenarios

No.


Align Grid in Memorize activity

The grid is not vertically aligned.

Also in mobile view, which is used by most of the users, it looks much worse.

After change: -

The grid doesn't need to be aligned.
There is a limit to responsive in Sugarizer, see here.


Android Basics: Activities and intents: Needs clarification on needed imports

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-activities-intents?hl=en&continue=https%3A%2F%2Fcodelabs.developers.google.com%2F%3Fcat%3Dandroid#6
In which task and step of the codelab can this issue be found?
Steps 5-9
Describe the problem
The documentation lacks info about the imports needed to run the app. I walked through the steps and when I clicked run- the project failed because several imports were missing (import android.content.Intent, import android.net.Uri etc). Each step should mention an import if needed for the app to run.
Versions
Android Studio version: Arctic Fox 2020.3.1
API version of the emulator: Pixel 5 API 30

@dm36
Thank you for reaching out to us and sharing your feedback. How to enable auto imports, please refer to the codelab:
https://developer.android.com/codelabs/basic-android-kotlin-training-create-dice-roller-app-with-button?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-kotlin-four%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-training-create-dice-roller-app-with-button#3.


Noto siddham cannot display on ms word and adobe illustrator properly

@caominhsg : thank you for your feedback... Could you try the font that could be downloaded from https://github.com/googlefonts/noto-fonts/tree/master/hinted/ttf/NotoSansSiddham ?
If you still have issues with this new font, could you send me photos of what wrong? and also write (or copy and paste) here the text in Siddham?
Thank you for your help...

I had installed noto font siddham and here is result
On notepad is fine and show properly but on the Adobe Illustrator is not, MS Word take long time to render and show up when I paste into ms word
I convert text from Pali to Siddham at address  https://aksharamukha.appspot.com/converter (this is not my website)
here is few pali text for you to check and test
TIṢṬA TIṢṬA
SAMAYAM ANUPĀRAYA
TATHĀGATA-HṚDAYA ŚUDDHE
VYAVAROKAYA TUMAṂ
AṢṬA BHI_ MAHĀ-DĀRUṆA BHAYEBHYAḤ
SARA SARA _PRASARA PRASARA
SARVA VARAṆA VIŚODHANI
SAMANTĀ KĀRA MAṆḌARA VIŚUDDHE
VIGATE VIGATE VIGATA
MARE VIŚODHANI
KṢIṆI KṢIṆI
SARVA PĀPA VIŚUDDHE

This isn't a font problem, but a software problem. I will check and report the Illustrator problem.

Thanks so much

Closing as this is Adobe's issue, not Noto's.


PHP typing else  adds garbage code

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug
When inputting PHP code for a if/else statement, when type 'else' and hit enter, VSC inserts garbage code and screws the formatting up which I have to fix.  It looks like this:
if (...)
{
}
else {
	# code...
}

I want it to look like this:
if (...)
{
}
else
{
	# code...
}

Or even better would be for it to not insert anything at all.  I have gone through the settings, disabled all extensions, and it still does it.  So at this point, I have to conclude that it is a bug of some sort.
 Are you triggering a snippet? Or some auto format?

I figured it out.  It was a snippet.  A feature that I was not aware of.
Ok to close.


Vuejs-templates v1.2.3, v1.2.4 s open the blank with afari browser, who knows what to do?

Can you be a  bit more precise?

Which project setup (answers)?
What safari version?
Any console errors? What does the HTML look like?
...?

safari version 10.2, there is no console error, I guess the ios version may be too low ....

I don't own an iOS device, so I can't debug this, sorry.

Alright, thanks for your reply

Closing for inactivity.


Tag more filter examples as optional

Closes #262, if desired...

[x] tag more bullet-pointed filter examples as OPTIONAL on the same line where it has already been indicated as optional in the text.
[ ] ~re-tagged one :filter: as a :filter-fragment:~
[x] fixed a spelling typo

I haven't been able to spot any more filters that are optional, but are not tagged as such, so I think this is ready for review. We are now validating this list as of Materials-Consortia/optimade-python-tools#213.

Looks good.
Great work with the validator! It seems sensible to push for including it in the main repository to travis-validate changes to the specification, right?

To clarify, we are validating our default server implementation against all the mandatory :filter: examples in the spec (and hopefully soon will be doing the same with optional filters). I'm not sure we should go as far as validating the spec with the same tool, but we could discuss this in the next meeting!


[FLINK-25178][table] Throw exception when managed table sink needs checkpointing

What is the purpose of the change
In the past, many users encountered the problem that the sink did not output because they did not open the checkpoint. For the managed table: The planner will throw an exception if the checkpoint is not turned on. (Later we can add public connector interface, including Filesystem, Hive, Iceberg, Hudi need it). PlannerBase adds the check whether the managed table sink is enabled to checkpointing.
Brief change log

PlannerBase adds the checkManagedTableCheckpointing to check whether the managed table sink is enabled to checkpointing. If the managed table sink disables checkpointing, this could throw the TableException.

Verifying this change
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@JingsongLi , thanks for detailed review. I have addressed above comments and added the managed table sink test. Please take a look.

@JingsongLi , thanks for the review. I have addressed your comments and added the tests above mentioned. Please check again.


Changing the fetchmail config file has no effect.

Only the first launch of the container with fetchmail enabled changes the config file, changing it later in is impossible without manually deleting files in the container and relaunching it.
The following part should be removed from setup-fetchmail to resolve this:
if [ -f "$RC" ]; then
	echo "The Fetchmail configuration is file '$RC' is already generated."
	exit 0
fi

Merged.