Grammar railroad diagram

With the transformed EBNF shown bellow from the README, we can copy and paste it on https://www.bottlecaps.de/rr/ui in the tab Edit Grammar then switch to the tab View Diagram to see the railroad diagram.
In doing so I noticed that the <arm> nonterminal is not defined.
datatype      ::= "datatype" "(" ( derive_clause "," )? datatype_name ( "," variant )+ ")"
record        ::= "record" "(" ( derive_clause "," )? record_name ( "," field )* ")"
datatype_name ::= ident
record_name   ::= ident

variant       ::= "(" variant_name ( "," type )* ")"
field         ::= "(" type "," field_name ")"
variant_name  ::= ident
field_name    ::= ident

derive_clause ::= "derive" "(" deriver_name ( "," deriver_name )* ")"
deriver_name  ::= ident

match         ::= "match" "(" lvalue ")" ( arm )+
matches       ::= "MATCHES" "(" expr "," ident ")"
if_let        ::= "ifLet" "(" lvalue "," variant_name "," ident ( "," ident )* ")" stmt
of            ::= "of" "(" variant_name ( "," ident )* ")" stmt
otherwise     ::= "otherwise" stmt

And here I'm also including the _ for the of:
datatype      ::= "datatype" "(" ( derive_clause "," )? datatype_name ( "," variant )+ ")"
record        ::= "record" "(" ( derive_clause "," )? record_name ( "," field )* ")"
datatype_name ::= ident
record_name   ::= ident

variant       ::= "(" variant_name ( "," type )* ")"
field         ::= "(" type "," field_name ")"
variant_name  ::= ident
field_name    ::= ident

derive_clause ::= "derive" "(" deriver_name ( "," deriver_name )* ")"
deriver_name  ::= ident

match         ::= "match" "(" lvalue ")" of+ otherwise?
matches       ::= "MATCHES" "(" expr "," (ident | "_") ")"
if_let        ::= "ifLet" "(" lvalue "," variant_name "," ident ( "," ident )* ")" stmt
of            ::= "of" "(" variant_name ( "," ident )* ")" stmt
otherwise     ::= "otherwise" stmt

Here is another one with a match_ident nonterminal that seems to accomplish the intended usage:
datatype      ::= "datatype" "(" ( derive_clause "," )? datatype_name ( "," variant )+ ")"
record        ::= "record" "(" ( derive_clause "," )? record_name ( "," field )* ")"
datatype_name ::= ident
record_name   ::= ident

variant       ::= "(" variant_name ( "," type )* ")"
field         ::= "(" type "," field_name ")"
variant_name  ::= ident
field_name    ::= ident

derive_clause ::= "derive" "(" deriver_name ( "," deriver_name )* ")"
deriver_name  ::= ident

match         ::= "match" "(" lvalue ")" of+ otherwise?
matches       ::= "MATCHES" "(" expr "," ident ")"
if_let        ::= "ifLet" "(" lvalue "," variant_name "," ( "," match_ident )+ ")" stmt
of            ::= "of" "(" variant_name ( "," match_ident)+ ")" stmt
otherwise     ::= "otherwise" stmt
match_ident ::= ident | "_"


FreeBSD init: Remove unnecessary daemon -u option

1. What does this change do, exactly?
Fixes #1923 by removing unnecessary -u ${caddy_user} option for the daemon command in the init script for FreeBSD.
2. Please link to the relevant issues.
#1923
3. Which documentation changes (if any) need to be made because of this PR?
None
4. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I am willing to help maintain this change if there are issues with it later

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thank you!


[ISSUE] Error: unexpected error handling request: json: cannot unmarshal number into Go struct field APIErrorBody.error_code of type string.

Description
Error: unexpected error handling request: json: cannot unmarshal number into Go struct field APIErrorBody.error_code of type string. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues.
Then the Error message is returned and the github workflow fails.
Is it a regression?
Did this work in a previous version of the SDK? If so, which versions did you try?
I appeared to work in development last week on May22 without returning the error.  On getting the error I tried both the current release, cli/0.220.0 and 0.219.0, still getting error.   Error does not occur in terminal usage, only in Github action.
Debug Logs
GET /api/2.0/preview/scim/v2/Me
> * Host:
> * Accept: application/json
> * Authorization: REDACTED
> * Traceparent: 00-a63c735a684241e3fa6d83ff720ad9b1-e9a9ebb5027dc722-01
> * User-Agent: cli/0.220.0 databricks-sdk-go/0.41.0 go/1.21.10 os/linux cmd/bundle_deploy auth/pat cicd/github
< HTTP/2.0 [40](https://github.com/<ORG>/<REPO>/actions/runs/9274716226/job/25517703872#step:8:41)3 Forbidden
< * Content-Length: 70
< * Content-Type: application/json; charset=utf-8
< * Date: Tue, 28 May 2024 18:44:[43](https://github.com/<ORG>/<REPO>/actions/runs/9274716226/job/25517703872#step:8:44) GMT
< * Server: databricks
< * Vary: Accept-Encoding
< * X-Databricks-Reason-Phrase: Invalid access to Org:<PHONE_NUMBER>454685
< {
<   "error_code": 403,
<   "message": "Invalid access to Org:<PHONE_NUMBER>[45](https://github.com/<ORG>/<REPO>/actions/runs/9274716226/job/25517703872#step:8:46)4685"
< }

Other Information
Running in Github Action workflow
Additional context
Add any other context about the problem here.

I'm getting a similar issue with a terraform plan:
Planning failed. Terraform encountered an error while generating this plan.
╷
│ Error: cannot read storage credential: unexpected error handling request: json: cannot unmarshal number into Go struct field APIErrorBody.error_code of type string. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:
│ │ GET /api/2.1/unity-catalog/storage-credentials/mi_storage_credential │ > * Host:  │ > * Accept: application/json │ > * Authorization: REDACTED │ > * User-Agent: databricks-tf-provider/1.41.0 databricks-sdk-go/0.38.0 go/1.21.9 os/linux terraform/1.7.0 resource/storage_credential auth/pat │ < HTTP/2.0 403 Forbidden │ < * Content-Length: 45 │ < * Content-Type: application/json; charset=utf-8 │ < * Date: Wed, 29 May 2024 13:54:33 GMT │ < * Server: databricks │ < * Vary: Accept-Encoding │ < * X-Databricks-Reason-Phrase: user not found │ < { │ <   "error_code": 403, │ <   "message": "user not found" │ < } │
│
│   with databricks_storage_credential.external_mi,
│   on metastore.tf line 46, in resource "databricks_storage_credential" "external_mi":
│   46: resource "databricks_storage_credential" "external_mi" {
│
╵
╷
│ Error: cannot read grants: unexpected error handling request: json: cannot unmarshal number into Go struct field APIErrorBody.error_code of type string. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:
│ │ GET /api/2.1/unity-catalog/permissions/metastore/ad5554ac-50e9-4a55-aa36-fd3d87fba036 │ > * Host:  │ > * Accept: application/json │ > * Authorization: REDACTED │ > * User-Agent: databricks-tf-provider/1.41.0 databricks-sdk-go/0.38.0 go/1.21.9 os/linux terraform/1.7.0 resource/grants auth/pat │ < HTTP/2.0 403 Forbidden │ < * Content-Length: 45 │ < * Content-Type: application/json; charset=utf-8 │ < * Date: Wed, 29 May 2024 13:54:42 GMT │ < * Server: databricks │ < * Vary: Accept-Encoding │ < * X-Databricks-Reason-Phrase: user not found │ < { │ <   "error_code": 403, │ <   "message": "user not found" │ < } │
│
│   with databricks_grants.external_mi,
│   on metastore.tf line 57, in resource "databricks_grants" "external_mi":
│   57: resource "databricks_grants" "external_mi" {
│
╵

@MarcusCanto That sounds like a different issue. Could you open up a new issue for it? Thanks!
@madengr00 I checked and while totally not descriptive, the error could indicate that there are IP access lists for the workspace and that the caller's origin IP is not allowed. Could you check if this is a possible explanation?

When I was trying to pass the databricks bundle deploy command in my Git Action,  I was using
[[ "$DATABRICKS_BUNDLE_ENV" == "dev" ]] && databricks bundle deploy --target dev || [[ "$DATABRICKS_BUNDLE_ENV" == "prod" ]] && databricks bundle deploy --target prod
Perhaps what is happening here is that it was successfully executing the first one, but could not get past the second half since I do not pass both sets of credentials, only the set for the environment being executed on.
I will close this out and do some additional debugging on my end.


Fully support BigDecimal

All the same functions equivalent to existing BigInteger support
toBigDecimal(limit: Int) for decimal approximations having a limit on repeating decimals in addition to toBigDecimal()

It may prove challenging to provide toBigDecimal(limit: Int).  See https://softwareengineering.stackexchange.com/a/192077


Ensure we are validating all fields of a block

Some validation is missing when checking blocks. Off the top of my head we should check:

That the gas price in transactions is at least the minimum gas price.
That the block gas limit is respected (maybe this is already validated when executing the transactions, we should check)
That the gasUsed in the block is truthful

Review all fields of a block to work out what we need to validate :)

Here are a few more:

the timestamp is plausible
the transactions root reflects the transactions in the block
the receipts root reflects the receipts generated after executing the transactions
the state root reflects the state after executing the transactions
the size of extra data is limited

I left out the logsBloom we will also need to check once we implement light clients, and the prevRandao field once we implement it.


PR: Fix test workflow syntax and update README (CI, docs)

[ ] Fix tests workflow
[ ] Add info about pre-commit in the README and development environment setup with environment.yml

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@e9c77e8). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head d577784 differs from pull request most recent head 5a54a15. Consider uploading reports for the commit 5a54a15 to get more accurate results

@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage        ?   73.51%
=======================================
  Files           ?        8
  Lines           ?      287
  Branches        ?       31
=======================================
  Hits            ?      211
  Misses          ?       62
  Partials        ?       14

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


Typo on new-mod page

It says join is in the cradshfiter chat room on this page: http://stats.zomis.net/io-web/new-mod

It took me a good minute to find the typo! Thanks I'll correct it tonight!

Will be fixed on the next update of the site


Error displaying widget

Describe the bug
When I run tutorial notebooks and the result should be visualized, it gives me an error: "Error displaying widget". For queries without a path, it gives tabs "Console" and "Query Metadata" without any problems.
To Reproduce
I run it on JupyterLab 3.2.8 (I tried it with 3.4.2 but the result was the same). Python version 3.9.7
Expected behavior
I would love to see the results from the queries. Any idea of what could help will be appreciated.
Screenshot

I have the same problem. This is what I tried:
mamba create -n test12 python=3.8 (which installed 3.8.13)
conda activate test12
pip install rdflib==5.0.0
pip install markupsafe==2.0.1
pip install graph-notebook
pip install jupyterlab==3.2.8
The installation of jupyerlab returned the error:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
graph-notebook 3.3.0 requires nbconvert==5.6.1, but you have nbconvert 6.5.0 which is incompatible.
I then ran
pip install nbconvert==5.6.1
and got
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
jupyter-server 1.17.0 requires nbconvert>=6.4.4, but you have nbconvert 5.6.1 which is incompatible.
I then ran
pip install jupyter_server==1.13.4
and I got no more conflicts. Afterwards, I loaded the magics, configured the Neptune details and seeded/loaded the air routes data
If I execute
%%gremlin
g.V().groupCount().by(label).unfold()
I get results
If I execute
my_node_labels = '{"airport":"city"}'
my_edge_labels = '{"route":"dist"}'
%%gremlin -p v,oute,inv -d $my_node_labels -l 20 -g country -de $my_edge_labels
g.V().has('code','LHR').outE().inV().path().by(valueMap(true)).limit(5)
I get 'Error displaying widget'
I tried the same procedure for python versions 3.6.13, 3.7.13, 3.9.7 (for this I also had to install matplotlib) and the results were the same. I attached a file with the results of pip freeze
req.txt

Hi @anezkakot, thank you for the bug report, and apologies for the late response.
I spent some time trying to reproduce this bug on JupyterLab 3.2.8 following the installation info provided (also that from @alexandrututunaru), but have not yet been successful.
To help better understand the root cause, could either of you also provide:

The output of !jupyter labextension list
The JavaScript errors displayed in console of your browser's web developer tools, after running a query where the widget failed to display

For reference, these configurations on Python 3.6.13 and 3.9.7 display the graph widget correctly on my local setup:
freeze_3.6.13_v3.3.0_noconflicts.txt
freeze_3.9.7_v3.3.0_noconflicts.txt

Closing due to >3 months of inactivity. Please re-open this issue if you are able to provide additional information that can help reproduce the error.

I'm having this same problem when I try to use jupyterlab > 4 .
It works
jupyter==1.0.0
jupyter-client==7.1.2
jupyter-console==6.4.3
jupyter-contrib-core==0.3.3
jupyter-contrib-nbextensions==0.5.1
jupyter-core==4.9.2
jupyter-highlight-selected-word==0.2.0
jupyter-latex-envs==1.4.6
jupyter-nbextensions-configurator==0.4.1
jupyter-server==1.13.1
jupyterlab==3.2.8
jupyterlab-server==2.10.3
jupyterlab-widgets==1.1.0
pandas==1.5.3
graph-notebook==3.8.2
ipykernel

This generates error Error displaying widget
jupyter==1.0.0
jupyter-client==8.3.0
jupyter-console==6.4.3
jupyter-contrib-core==0.3.3
jupyter-contrib-nbextensions==0.5.1
jupyter-highlight-selected-word==0.2.0
jupyter-latex-envs==1.4.6
jupyter-nbextensions-configurator==0.4.1

jupyter-core==5.3.1
jupyter-server==2.7.0
jupyterlab==4.0.2
jupyterlab-server==2.23.0

jupyterlab-widgets==1.1.0
pandas==1.5.3
graph-notebook==3.8.2
ipykernel


fix: button white-space style

Types

[x] 💄 Style Changes

Background or solution

🤖 Generated by Copilot at b186183

Allow button text to wrap to the next line if it exceeds the available width, preserving the whitespace and line breaks (link)

Changelog

🤖 Generated by Copilot at b186183
Changed the white-space property of the button component to allow text wrapping. This improves the responsiveness and accessibility of the button component in packages/components/src/button/style.less.

/backport to v2.25


Document sharing workflow experiment

This is a separate workflow immediately following #41
GOAL
I have just uploading notes from a meeting and would like to alert the other meeting attendees so they can review the notes.
Step 1
I'm looking at my file within a list of other files.

Nothing obvious to do except click the triangle button so I do that. I see the content of my .txt file in a box within the file structure.

Observation: I am still in this view where I can see the entire folder structure of team.inrupt.net, and my file within in it. Is this useful to me? I'm trying to do something specifically with this file, so I don't really care what folder I'm in.
Step 2
There do not appear to be any tools to help me get a URL for this file that I can share with others, or to share it with them directly from this databrowser screen.
Ah! But then I happen to hover over the area next to the filename and see a list of icons appear. I hover again over an icon of a lock and see that a popup dialogue box calls it "sharing." That seems promising. Click.

Hmm ok, I see information about the default sharing settings. That's useful. It appears that the people I want to share the file with already have access (because I know they're part of the "inrupt team"). But why does it say Inrupt Team twice?
Step 3
So it's good that I know the people I want to share with already have access, but I still need to alert them to the fact that I recently uploaded this document. Otherwise they're never going to look at this file way down in this folder. Maybe this little circle icon with the arrow next to "Inrupt Team" will help me share a link to the file with that team? Click.

Oh no. Where am I now? I'm at https://team.inrupt.net/profile/card#me
This appears to be some sort of blank profile page for an indeterminate entity known as Inrupt Team. Not what I was looking for.
Step 4
Because I personally have been here before and know this little trick, I drag the filename out of the folder structure and into my browser bar. This provides me with a direct URL to the document that I can copy and paste in other channels to alert the team.

@InruptKelly Is there a specific issue you are reporting, something that is not working or are you just recording your observations?

https://github.com/solid/mashlib/issues/41


Missing source code for TournamentModule.dll for BWAPI version 4.2.0

The source code is missing for TournamentModule.dll for BWAPI version 4.2.0, i.e. the file bwapi-data/TournamentModule.dll contained in server/required/Required_BWAPI_420.zip. For the other versions it is located at e.g. src/tournamentmodule/BWAPI_412
Please could it be added? The reasons are for transparency and because I would like to build my own variant of TournamentModule.dll for BWAPI 4.2.0 that runs on Windows XP SP3 (FYI, BWAPI 4.2.0 doesn't work on Windows XP SP3 and I've already proposed a fix to BWAPI in https://github.com/bwapi/bwapi/issues/733).  Thanks

I believe it's identical to the 4.1.2 version, just compiled in 4.2.0, but @richard-kelly could you add it?

@davechurchill that's correct. I'll create a version named 4.2.0 for clarity.

Thanks, please feel free to close this issue.  I just successfully used it to build and test my own variant of TournamentModule.dll for a modified version of BWAPI 4.2.0 that runs on Windows XP SP3.
Note: the injectory.x86.exe file included in StarcraftAITournamentManager doesn't run on Windows XP SP3 (due to the same issue described in https://github.com/bwapi/bwapi/issues/733) but I was able to use Chaoslauncher instead.

We should just update the instructions to say Windows 7

Sorry, closed by misclick.

By the way, the stuff about Windows XP SP3 was just FYI - I wasn't pushing for Windows XP SP3 support.

I just think that advertising Windows 7 is the easiest option rather than worrying about XP versioning :)

@richard-kelly Has this been added? If so can you close the issue

Yes, the change was made.


🆕 Software Suggestion | Add a Chromium based browser

Category:
Browser Recommendations For Desktop
Browser Recommendations For iOS
Why I am making the suggestion
In some use cases (often while working for a company) you just cant go around the fact that u have to use a Chromium based browser.
For whatever the reason may be.
I personally dont know if there are any good Chromium based browsers, but i can imagine people having the need of using them and visit Privacytools.io for some guidance.
I don't think that they should be recommended based on that in the past, i didn't see any recommendation for it (i think at least), but i hope they maybe fit into the critera of "worth mentioning".
Edit:
Fixed some typos.
Also im sorry this should have the enhancement label instead of software suggestion.

If I'm not wrong, there can't be a truly Chromium based browser on iOS because Apple forces developers to use their WebKit engine, so I guess it's kind of pointless to recommend other browsers that aren't as private as Firefox if they are going to behave pretty much the same way.
As for desktop browsers, at some point Brave was listed, but the developers asked it to be removed because followers of PT used to go to their repository and ask them to change or add X or Y and they didn't want to handle those people. Also, it's CEO it's a bit of a homophobic, and their whole cryptocurrency thing is a bit shady. As for the rest of the Chromium based stuff out there, it's really hard to maintain a private fork because of how it's released, you need to remove all of Google's blobs while also maintaining a browser up to date (which is a really hard work). The only more or less good options are Brave, Iridium, ungoogled Chromium and maybe Vivaldi (its CSS is closed source, though), but I'm not sure if they are updated so often.

It would make sense to recommend a Chromium browser since they're so popular, but there are basically no good browsers from a privacy perspective. Most people would probably agree that the best is Ungoogled Chromium, but it requires manual updates. This is a problem, especially for the average user, since browsers are the type of software that need to remain up-to-date. Brave is seen as dodgy by many people due to the affiliate link scandal, the cryptocurrency stuff, etc. However, it's still a lot better than many other browsers. I've read that Vivaldi uses some analytics service that gets pinged every 24 hours (see the privacy policy), and it can't be built from source. The other Chromium browsers are either closed source or not updated regularly enough to be recommended.

Although I agree with @samuel-lucas6 I think listing Ungoogled-Chromium as an option, with the disclaimer, is not a bad idea. In the same terms, a user not being able to find a Chrome replacement, after hearing how much faster and awesome Chrome is, might just keep using Chrome instead of trying out another option if there is not one listed. Open to objections on this, of course.

@wylel I don't think that'll work. PrivacyTools' target demographic is average users.
They won't be able to build Ungoogled Chromium, and recommending them to download binaries from a wide varity of sources is just asking for trouble. (It's even a warning in the official repository)
Again, we're targeting average users. The user experience is just too poor to recommend.


[Bug]: Reopening #5849 Puppeteer does not work on RHEL 8.1 (inside a podman)

Steps to reproduce
puppeteer stopped working once we migrated our build servers from RHEL7 to RHEL 8. Please let us know how to fix this .
Failed to launch the browser process!     /var/jenkins_home/workspace/acceptance-tests/node_modules/puppeteer/.local-chromium/linux-722234/chrome-linux/chrome: error while loading shared libraries: libXss.so.1: cannot open shared object file
Expected results
Failed to launch the browser process!     /var/jenkins_home/workspace/acceptance-tests/node_modules/puppeteer/.local-chromium/linux-722234/chrome-linux/chrome: error while loading shared libraries: libXss.so.1: cannot open shared object file
Actual results
Failed to launch the browser process!     /var/jenkins_home/workspace/acceptance-tests/node_modules/puppeteer/.local-chromium/linux-722234/chrome-linux/chrome: error while loading shared libraries: libXss.so.1: cannot open shared object file

You need to install all missing system dependencies including libXss as the error indicates. See https://pptr.dev/troubleshooting#running-puppeteer-in-docker and https://pptr.dev/guides/system-requirements I am not sure if Chrome is officially supported on RHEL but here is a list of dependencies required on rpm-based systems https://source.chromium.org/chromium/chromium/src/+/main:chrome/installer/linux/rpm/dist_package_provides.json;drc=2e9ec273b0df7d904c20579380822ff349d30ca3.


NIFI-13970 added disclaimer to source and key assemblies

Summary
NIFI-13970
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[ ] Apache NiFi Jira issue created

Pull Request Tracking

[ ] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[ ] Pull Request based on current revision of the main branch
[ ] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

🤦 Doh!   on it.     Why didn't someone else less thorough review this.....

Rebased to latest support/nifi-1.x and added registry assembly and force pushed.

Minor nitpick, but the last line is missing a line ending in the DISCLAIMER files.
In the Unix/-like world, a line in a text file is ending by a line ending \n by definition. On Windows, \r\n is more like a line separator, so adding it after the last line is not needed. I prefer the Unix way. Normally text editors on Linux (and I think macOS too) add this automatically, but some IDEs follow the Windows convention, except using a Unix \n line ending character as their line separator when on Linux/macOS.

i added a commit to ensure each disclaimer ends with a newline.

Merged to support 1.x branch in https://github.com/apache/nifi/commit/ca10d7abc00060453bdb0451f6b7fe3f5503c058
Merged to support 1.28 branch in https://github.com/apache/nifi/commit/26e4598abb64d79f3c927c3fc8949c62f024c05e


Added function call exception, thrown if parameter count doesn't match

 Type visitor: Check number of parameters along with their types in function calls


Saving to correct path

I'm using meteor and meteor environment.
The file I want resides here:
/Users/username/2emp/latest.xml
Meteor in development bundles the file here:
/Users/username/2emp/.meteor/local/build/programs/server/latest.xml
When using this.download(this.getCurrentUrl(), "latest.xml") a couple of things I don't understand.
If I pass it the full path, it downloads it correctly.
How can I access a global variable from with spooky.then function? Do I have to pass it to spooky.on and then do something with the download?

You need to pass yours vars from the node context to the casperJS context. Spooky provide this helper :
spooky.then([{
  myvar: myvar // Warning : variables are pass as value with json serialization
}, function(){
  myvar.myproperty// is now available
}])


Add stress tests for settings table

[x] Add a stress integration test that creates X collections in parallel in X goroutines.
[ ] As the test will always fail and implementation is not in scope, either skip the test or not merge it.

[upd] the problem with collection creation is an easy one - when we query CreateDatabaseIfNotExists from msg_create we don't consider a case when DB already exists as normal. But we should consider it normal as it's ok if the DB is created from one of the parallel requests. The fix is in #1316.

I also fixed simple test failures occurred on my local env by adding transactions to SchemaStats usage.


Calendar synced from outlook in .ics format broken in new update.

Opening settings to add new .ics format calendars has no dropdown box to add URL
I'm on Windows 10 Pro.
Obsidian 1.0.3 version.

This seems to be a duplicate of #206. Please follow along there for progress on this bug. Thank you for submitting!


Enhance secrets documentation

How to categorize this PR?

/area documentation
/kind enhancement
Which issue(s) this PR fixes:
Fixes a part of #3405
Release note:

NONE

Thanks for your reviews @vpnachev and @rfranzke. PTAL.


Reduce number of unnecessary copies in ipset code

We on several occasions do unnecessary copies. For example, in _sync_to_ipset, programmed_members is discarded and replaced with a copy of members. This leads to churning of the heap, which has to have a CPU and possible occupancy cost.
We should see if we can cut a few of those copies.

I might not do this, but I have a couple of hours and an idea for how to do it simply, so will own it until I either do it or decide that it's better to leave to somebody else.


"Help wanted" Bar chart Time scaling graph not showing correctly

For a statistics overview people can select a daterange.
I need the chart to plot the results according to a timestamp.
I use the type: time
here are some options:
                            scales: {                                 yAxes: [{                                    type: 'linear',                                }],                                 xAxes: [{                                     type: 'time',                                     time: {                                         tooltipFormat: 'lll',                                         minimumTimeUnit: '{{ $readingData['minimumTimeUnit'] }}',                                         min: {{$period->start->getTimestamp()*1000}},                                         max: {{$period->end->getTimestamp()*1000}},                         }                                 }]                             }
Current Behavior
You see I have only a couple of timestamps and it creates a bar with information that is not true.
I want only bar at the correct timestamp, not spreading weird:s
The bar chart currently

the bar chart data

Context

Want to let people see statistics per day, week, month, year etc
Environment

Chart.js version: ^2.7.1
Vue-Chart : 2.8.7

I believe this is a duplicate of a couple of other issues. Not sure of the numbers off the top of my head

I think I'm having the same issue. It happens when using bar chart with linear (time) distribution. It's possible to set barThickness properly manually but then you lose responsiveness.
Desired behavior would be that bar widths are calculated automatically but all bars would use the same width. And that width would be the width of the smallest bar in the current implementation.

#4855 seems to talk about the same issue.


Resolves pcfens#266

initial commit of filebeat::cloud hash that allows cloud.id and cloud.auth in the filebeat.yml for major version 6

Ah, it does seem to be working now after I resolved my firewall issues (my first test was prior to getting my firewall opened). Will stick with the hash since it's the pattern you've set up
I'm using hiera for all the config and it looks like this:
filebeat::cloud:     id: 'hiscox:<MY REALLY LONG HASH>'     auth: "elastic:%{lookup('<MY EYAML ENCRYPTED SECRET>')}"
This is working great and overrides the host provided in the output.elasticsearch as specified in the filebeats documentation
@pcfens Anything else you'd like me to do for the PR?

Nope - we're all set - I'll try and create a new release on the forge soon. Thanks for all your help on this.

I just pushed this to the forge as 4.8.0. Thanks again!


Allow creating checkout without email

Do not require email for checkoutCreate mutation.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-3143 Allow to create checkouts without email


How to execute POST example?

Hi,
I am new to nginx and lua, POST section seems not working for me, after i put openresty nginx.conf,
GET options with
curl http://localhost/hello works for me and give me output as "someone said hello"
curl http://localhost/hello/nur gives output as hello nur,
but how should i test the POST part?
I tried several way but please show me how can i test it exactly, i tried below:
curl -X POST --data '{ "comment" = "fascinating"}' http://localhost/app/2/comment
nothing shows.
//Regards
Nur

what does your route for /app/2/comment look like?

Hi,
It is now working as below:
curl -X POST --data 'comment=fascinating' http://localhost/app/2/comment
comment fascinating created on app 2
How can we able to get URL args like below:
http://localhost/app/4/comment?comment=test
?
My route for /app/2/comment is below:
local router = require "router"
local r = router.new()
r:match({
POST = {
["/app/:id/comment"] = function(params) ngx.print("comment " .. params.comment .. " created on app " .. params.id) end
}
})
local ok, errmsg = r:execute(
ngx.var.request_method,
ngx.var.request_uri,
ngx.req.get_uri_args(),  -- all these parameters
ngx.req.get_post_args(), -- will be merged in order
{other_arg = 1})         -- into a single "params" table
if ok then
ngx.status = 200
else
ngx.status = 404
ngx.print("hellopost says - not found!")
ngx.log(ngx.ERR, errmsg)
end
//Regards
Nur


Add format & stdin options

Hi,
I add formatter and stdin options to this cli command line interface.
I think it would help if we can specify lint result output format to JSON format.
Usage shown as blow:
Lints html files with htmllint.
Usage: cli.js [OPTIONS] [ARGS]

Options:
  --help             Show help                                         [boolean]
  --version          Show version number                               [boolean]
  --rc               path to a htmllintrc file to use (json)     [default: null]
  --format           output formatter (default: simple)      [default: "simple"]
  --cwd              path to use for the current working directory
                                                                 [default: null]
  --stdin-file-path  read html content from stdin and specify its file path
                                                                 [default: null]

Nice! I think the --stdin-file-path option is kind of confusing, since there's not a real file involved (the path is just used as a name for output, right?). It's also kind of strange that it works with no file name given but then uses the name "true". I will probably change it to --stdin and use a default filename of "stdin".
The --format option should also list possible values in the help. It would be nice if it got these dynamically from lib/formatters, but I doubt there will be too many formatters so that's probably overkill.

I use the option --stdin-file-path by extracting HTML content from other kinds of language (like PHP). I think it can help us to identify the original source file name (like extension, etc.) for rules and plugins so that they could be enabled or not through the config file based on the path pattern or the file name extension.

I definitely agree that the name option to --stdin-file-path is useful. It's just that calling it --stdin-file-path rather than --stdin is confusing: it seems like it's purpose is just to set the file path for stdin and there should be some other option to enable it in the first place.
I changed the option name to --stdin and changed the help text to something I find clearer. I also changed the behavior when --stdin is used without a value so that the filename "stdin" is used, rather than "true". Does that work for you?

Oh I see, I misunderstood before. That’s work for me.


References are not showing to newly created elements

When a reference to an instance of X is chosen and a new X instance is created, the new instance cannot be selected as it does not show in the dropdown.
Probably a too enthusiastic optimisation / caching.

Fixed


Support for simplified and traditional  Chinese

Issue
The library does not handle properly languages that are coded by iOS in three components. I have tried to include translation for both simplified and traditional Chinese. The language variable is set as follow:

zh-Hans-US for simplified Chinese
zh-Hant-US for the traditional Chinese

However, I cannot create json resource with the name zh-Hans-US / zh-Hant-US as last component encode the regional settings. In my case  it is US. I have tried  to name files: lang-zh-Hant.json  and lang-zh-Hans.json but it results in selecting default language as the library always fall back to the single component file name (lang-zh.json) which also cannot be found.
Localize Environment

Localize version:
Xcode version:
Swift version:
Platform(s) running Localize:
macOS version running Xcode:

Localize Config
/// Please put here your config

What did you do?
ℹ Please replace this with what you did.
What did you expect to happen?
ℹ Please replace this with what you expected to happen.
What happened instead?
ℹ Please replace this with of what happened instead.

It works for me.
Make sure you included Localize.update(provider: .json) if you want to use JSON files.
I'm using lang-zh-Hant-HK.json and it works just fine.

Correct, it would work but the problem that I point here is that you have
to include as many json files as regions that your users may be located (HK
in your case).  In reality you may need several files to cover people
located in other part of the world who just selected simplified Chinese as
a language for the interface. Therefore, I would propose that the file name
should be lang-zh-Hant.json. (without HK at the end)
On Thu, Jan 3, 2019 at 4:59 PM Jan Schwenzien<EMAIL_ADDRESS>wrote:

It works for me.
Make sure you included Localize.update(provider: .json) if you want to
use JSON files.
I'm using lang-zh-Hant-HK.json and it works just fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/andresilvagomez/Localize/issues/48#issuecomment-451187472,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACGg65TFP5d5unhB6fpmOkkwRzbPjIpjks5u_ijigaJpZM4ZY0Ne
.


The -antigen-use-prezto function silently affects how Zsh loads dotfiles

Prezto, unfortunately, does not provide an option to change its own dot directory. So I see why, to get it to work inside Antigen, the function -antigen-use-prezto does
export ZDOTDIR=$ADOTDIR/repos

Without it, pmodload would look for Prezto modules in ~/.zprezto instead of ~/.antigen/repos/.zprezto and therefore would fail.
The problem is that ZDOTDIR is used by Zsh itself to determine where its startup/login/logout files are. For an example of what can happen, try this:
# assuming $ZDOTDIR is unset initially, which is the default
echo 'echo "this should run on login"' > ~/.zlogin
echo 'echo "this should not run on login"' > ~/.antigen/repos/.zlogin
source antigen.zsh
antigen use prezto
zsh --login

You'll notice that the correct file is ignored, and the stray file is run in its place.
An additional wrinkle is that a few of the Prezto modules also use ZDOTDIR, e.g. for locating HISTFILE, and these probably should be in ~ as intended, and not in ~/.antigen/repos. In fact, it seems like pmodload is the only place where ZDOTDIR needs to be changed.
I've raised an issue with Prezto (sorin-ionescu/prezto/issues/920) about implementing some kind of PDOTDIR parameter. But in the meantime, maybe it wouldn't be a bad idea to re-implement pmodload on the Antigen side. If that seems like something you all would want to do, I can go ahead with a pull request to show you what I have in mind.
Also, in the current version, the line ZDOTDIR=$ADOTDIR/repos/ should be ZDOTDIR=$ADOTDIR/repos to avoid ending up with double slashes.

As noted, this is an upstream issue. Anything we can do to antigen are mostly hacks to make it work.
At least from the top of my mind I can't think a non-hacky way to solve it.
If you got an idea hack away and send your PR :)

Here's what I did to st up Prezto. I changed the -antigen-use-prezto function to this:
-antigen-use-prezto () {
    export ZDOTDIR=$ADOTDIR/repos
    antigen-bundle sorin-ionescu/prezto
}

It sets ZDOTDIR first which fixes some problems inside prezto when prezto isn't running from it's intended location (cc: @sorin-ionescu Please, we need these solved in one go: https://github.com/sorin-ionescu/prezto/issues/920, https://github.com/sorin-ionescu/prezto/pull/761, https://github.com/sorin-ionescu/prezto/pull/271, https://github.com/sorin-ionescu/prezto/issues/1025, https://github.com/sorin-ionescu/prezto/pull/791, basically make sure a new variable PDOTDIR based on ZDOTDIR is used everywhere).
Then I linked ~/.zshrc to ~/.antigen/repos/.zshrc and linked all ~/.antigen/repos/.z* files to the respective ones in ~/.antigen/repos/.zprezto/runcoms/.
After that, the only thing left to get prezto working is to put this in .zshrc:
    source ~/path/to/antigen.zsh
    antigen use prezto
    antigen apply

If antigen is doing export ZDOTDIR=$ADOTDIR/repos (which defeats the purpose of doing antigen use prezto because the expectation of antigen use prezto is that prezto will simply work without further tweaking and not having to compensate for a modified ZDOTDIR), the README may as well simply tell the user to set ZDOTDIR to /path/to/antigen/repos manually, and how to act accordingly, by moving all ~/.z* files to the new directory (or linking to them).

Done in v1.1.0.


TypeSpec Requirements for Release Planner

This epic will track the TypeSpec requirements for the Release Planner.
TypeSpec Onboarding (API Readiness milestone)

[ ] https://github.com/Azure/azure-sdk-tools/issues/4050
[ ] https://github.com/Azure/azure-sdk-tools/issues/5227

Design your API and describe using TypeSpec

[ ] https://github.com/Azure/azure-sdk-tools/issues/5621

TypeSpec Validation and API Review

[ ] https://github.com/Azure/azure-sdk-tools/issues/5622

@maririos I created this epic to group together all of the requirements for TypeSpec for the release planner. I think this will make it easier to track progression and understand where we may be blocked (i.e. process not defined, ...).

I see another epic related to API process: https://github.com/Azure/azure-sdk-tools/issues/4053 should we close it and include the relevant content here?

It looks like there are no active isues here so we can close this epic.


[Feature] Send messages selectively when sending mails

We have too many issue updates every day. I think it would be better if we had a choice when send emails, because some discussions are very important, while some are not. Is it OK to implement it through the comment area @oss-bot?

我们每天有大量的issue更新，我觉得如果发送邮件时有一个选择会更好，因为有些讨论是很重要的，而有些并不重要，实现方式通过评论区@oss-bot的方式是否OK？或者是否可以等回复数超过一个阈值(再把本次issue/PR的完整回复发一遍)，实现上可能有难度，仅是建议，也不会前端，很尴尬

I believe that's exactly the problem that @clr-apache is facing every day!

only send issue with specified label?

Assignees：Mingyu

Filter unimportant issues by filtering tags for issue name.

I think we can make the rule config file. For user to make their own rule of which message should be send by the bot.


fix: add back default value of null for actitivy tracker crn input var

Description
In https://github.com/terraform-ibm-modules/terraform-ibm-cos/issues/130 the default value of var.activity_tracker_crn was incorrectly removed.
Types of changes in this PR
No release required

[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI-related update (pipeline, etc.)
[ ] Other changes that don't affect Terraform code

Release required

[ ] Bug fix (patch release (x.x.X): Change that fixes an issue and is compatible with earlier versions)
[ ] New feature (minor release (x.X.x): Change that adds functionality and is compatible with earlier versions)
[ ] Breaking change (major release (X.x.x): Change that is likely incompatible with previous versions)

Release notes content
Replace this text with information that users need to know about the bug fixes, features, and breaking changes. This information helps the merger write the commit message that is published in the release notes for the module.

Checklist for reviewers

[ ] The PR references a GitHub issue.
[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

Merge actions for mergers

Merge by using "Squash and merge".

Use a relevant conventional commit message that is based on the PR contents and any release notes provided by the PR author.
The commit message determines whether a new version of the module is needed, and if so, which semver increment to use (major, minor, or patch).

:tada: This PR is included in version 5.3.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Broken control on unplug

Checklist

[x] I have searched for existing issues
[x] I have looked through the wiki
[x] I have updated MonitorControl to the latest version

Describe the bug
App works great when plugged. However, brightness control buttons stop working for the internal monitor when I unplug both external monitors. To fix the issue, I have to quit from the monitorcontrol app.
To Reproduce
Steps to reproduce the behavior:

Install the app
Plug your mac to external monitor
Unplug the monitor
Try using the brightness buttons

Expected behavior
It should continue working
Environment Information (please complete the following information):

macOS version: Catalina
MonitorControl version: 2.1.0
Monitor(s): Asus PB298Q and Dell U2711b monitors

You're right, I'm only seeing your issue now but I have run into this issue myself. It's probably a duplicate of #327, so let's continue tracking this on that issue.


the parser chokes on a single dot

In the PyDigger project I monitor PyPI and analyze recent releases. Among other things I use requirements-parser to parse the requirements.txt file.
Recently I encountered this project https://github.com/Te-k/pycrtsh  where the "requirements.txt" file contains a single dot"
.

the parser raises an exception.
Is this a bug in requirements.parse or is having a single dot really incorrect?
See:
>>> import requirements
>>> for req in requirements.parse(".\n"): print(req)
...

Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/requirements.py", line 98, in __init__
    req = REQUIREMENT.parseString(requirement_string)
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 1654, in parseString
    raise exc
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 1644, in parseString
    loc, tokens = self._parse( instring, 0 )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 3417, in parseImpl
    loc, exprtokens = e._parse( instring, loc, doActions )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 3739, in parseImpl
    return self.expr._parse( instring, loc, doActions, callPreParse=False )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 1402, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 3400, in parseImpl
    loc, resultlist = self.exprs[0]._parse( instring, loc, doActions, callPreParse=False )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 1406, in _parseNoCache
    loc,tokens = self.parseImpl( instring, preloc, doActions )
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/pyparsing.py", line 2711, in parseImpl
    raise ParseException(instring, loc, self.errmsg, self)
pkg_resources._vendor.pyparsing.ParseException: Expected W:(abcd...) (at char 0), (line:1, col:1)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.9/site-packages/requirements/parser.py", line 87, in parse
    yield Requirement.parse(line)
  File "/usr/local/lib/python3.9/site-packages/requirements/requirement.py", line 254, in parse
    return cls.parse_line(line)
  File "/usr/local/lib/python3.9/site-packages/requirements/requirement.py", line 232, in parse_line
    pkg_req = Req.parse(line)
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/__init__.py", line 3139, in parse
    req, = parse_requirements(s)
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/__init__.py", line 3084, in parse_requirements
    yield Requirement(line)
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/__init__.py", line 3094, in __init__
    super(Requirement, self).__init__(requirement_string)
  File "/usr/local/lib/python3.9/site-packages/pkg_resources/_vendor/packaging/requirements.py", line 100, in __init__
    raise InvalidRequirement(
pkg_resources.extern.packaging.requirements.InvalidRequirement: Parse error at "'.'": Expected W:(abcd...)

Thanks for taking the time to share this @szabgab.
A single dot in a requirements.txt can be used when dependencies are defined in another place (e.g. setup.py in the example you share).
Agree that this project shouldn't choke on this however, so will look to make an improvement.


Update Luanti (formerly Minetest) standard to 5.10.0

Minetest has been renamed to Luanti with the release of version 5.10.0.
This updates the standard to reflect that: It is now called luanti (with minetest still supported for compatibility). Most importantly, we now recommend using the core instead of the minetest namespace in code, though the latter is still being supported for an indefinite amount of time. (The core namespace alias has essentially always worked, but was only used internally and undocumented until now.)
It also updates the standard to contain new Lua API functions listed in https://dev.minetest.net/Changelog#5.9.1_.E2.86.92_5.10.0.

Thanks for the contributions. I split your commit up into one to rename and move things around and another to update the actual contents of the standard, but the end result should be the same as you submitted.

I split your commit up into one to rename and move things around and another to update the actual contents of the standard, but the end result should be the same as you submitted.

That makes sense. I've looked at the two commits and confirmed via git diff that the end result is indeed the same. Thanks for splitting it up.

nice, looks good :+1:


State mixin causes jest to fail

Jest seems to fail on the State mixin which uses this.context heavily.

So it does not appear to be possible to test any React component that involves the State mixin.
The class under test that is failing is defined like so:
var Router = require('react-router');
module.exports = React.createClass({
  mixins: [Router.State],
  ...
});

You can force a context on the rendered component under test like so:
var Context = {
  getCurrentParams: function() {
    return { moduleName: 'Test' };
  }
};

var Downstream = React.withContext(Context, function() {
  return TestUtils.renderIntoDocument(<DownstreamComponent/>);
});

FYI, withContext will get deprecated in next versions of React, but you'll be able to do the same by just wrapping your component into component with context.

@gaearon could you give some more information on the wrapping would work exactly? Maybe using the above example?

@alengel any luck?

var Context = {
  getCurrentParams: function() {
    return { moduleName: 'Test' };
  }
};

// React 0.13
var Wrapper = React.createClass({
  childContextTypes: {
    getCurrentParams: React.PropTypes.func.isRequired
  },
  getChildContext: function() {
    return Context;
  },
  render: function() {
    // React 0.13 uses owner-based context, so Wrapper has to render children here
    return this.props.render();
  }
});

var Downstream = TestUtils.renderIntoDocument(
  <Wrapper render={function() { return <DownStreamComponent />; }} />
);

// React 0.14?
var Wrapper = React.createClass({
  childContextTypes: {
    getCurrentParams: React.PropTypes.func.isRequired
  },
  getChildContext: function() {
    return Context;
  },
  render: function() {
    // context is based on parent, so it doesn't matter who created the element
    return this.props.children;
  }
});

var Downstream = TestUtils.renderIntoDocument(
  <Wrapper><DownStreamComponent /></Wrapper>
);

Thanks for that @taurose!

@taurose You just saved me a huge amount of time. Thank you very much!
If you're in Australia sometime, I'll shout you a beverage of your choice.


Option topology

WARNING: --topology net30 support for server configs with IPv4 pools will be removed in a future release. Please migrate to --topology subnet as soon as possible.

Option add to server.conf

Close


Enhance motion extension field

Power user features for motion detail.
Enhances the motion extension field to make it even more easily
accessible.
Also allows to quickly jump between motions using:
Meta + Alt(left) + ArrowKeyLeft/Right
@emanuelschuetze please test and roll out

There is no placehoder (e.g. "[motion:42]") after selecting a motion from the drop down list
in the recommendation extension field component.

It works for new motions only. But if you want to change a motion: remove the input and try to change the select field -> no placehoder is set in the input field. The select field should be always reset.

solved. Had one unsub too much


Knative Serving operator not updating config-certmanager values

Describe the bug
Knative Serving operator is not updating config-certmanager values.
Expected behavior
Setting the spec.config.certmanager should update config-certmanager ConfigMap as per the documentation.
To Reproduce
Update the spec of the Knative Operator:
spec:
  config:
    autoscaler:
      enable-scale-to-zero: "false"
    certmanager:
      issuerRef: |
        kind: ClusterIssuer
        name: letsencrypt-dns-issuer
    domain:
      app-staging.sidetrek.com: ""
    gc:
      max-non-active-revisions: "20"
      min-non-active-revisions: "5"
      retain-since-create-time: disabled
      retain-since-last-active-time: disabled
    network:
      autoTLS: Enabled
      httpProtocol: Redirected
  high-availability:
    replicas: 2

All other ConfigMaps are updating, but not the certmanager ConfigMap:
apiVersion: v1
data:
  _example: |
    ################################
    #                              #
    #    EXAMPLE CONFIGURATION     #
    #                              #
    ################################

    # This block is not actually functional configuration,
    # but serves to illustrate the available configuration
    # options and document them in a way that is accessible
    # to users that `kubectl edit` this config map.
    #
    # These sample configuration options may be copied out of
    # this block and unindented to actually change the configuration.

    # issuerRef is a reference to the issuer for this certificate.
    # IssuerRef should be either `ClusterIssuer` or `Issuer`.
    # Please refer `IssuerRef` in https://github.com/jetstack/cert-manager/blob/master/pkg/apis/certmanager/v1alpha1/types_certificate.go
    # for more details about IssuerRef configuration.
    issuerRef: |
      kind: ClusterIssuer
      name: letsencrypt-issuer
kind: ConfigMap
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","data":{"_example":"################################\n#                              #\n#    EXAMPLE CONFIGURATION     #\n#                              #\n################################\n\n# This block is not actually functional configuration,\n# but serves to illustrate the available configuration\n# options and document them in a way that is accessible\n# to users that `kubectl edit` this config map.\n#\n# These sample configuration options may be copied out of\n# this block and unindented to actually change the configuration.\n\n# issuerRef is a reference to the issuer for this certificate.\n# IssuerRef should be either `ClusterIssuer` or `Issuer`.\n# Please refer `IssuerRef` in https://github.com/jetstack/cert-manager/blob/master/pkg/apis/certmanager/v1alpha1/types_certificate.go\n# for more details about IssuerRef configuration.\nissuerRef: |\n  kind: ClusterIssuer\n  name: letsencrypt-issuer\n"},"kind":"ConfigMap","metadata":{"labels":{"app.kubernetes.io/managed-by":"pulumi","networking.knative.dev/certificate-provider":"cert-manager","serving.knative.dev/release":"v0.26.0"},"name":"config-certmanager","namespace":"knative-serving"}}
  creationTimestamp: "2021-10-26T04:16:54Z"
  labels:
    app.kubernetes.io/managed-by: pulumi
    networking.knative.dev/certificate-provider: cert-manager
    serving.knative.dev/release: v0.26.0
  name: config-certmanager
  namespace: knative-serving
  resourceVersion: "100586"
  uid: e5cfa2e9-3bd2-4024-8977-61c26ea6c983

Knative release version
Knative serving version: 0.26.0

@seunggs Sorry for the late response. I am not sure if you have figured it out or not, but if you would like to update the configmap in a certain yaml, you need to make sure that knative operator INSTALLS this yaml.
Like in https://github.com/knative-sandbox/net-certmanager/releases/download/knative-v1.0.0/net-certmanager.yaml, the configmap is available in this yaml, but this yaml is not managed or installed by knative operator by default.
One approach is that you can specify this yaml file's link in the spec.additionalManifests section in the serving CR, so that operator can install it as a custom manifests. In that case, you would be able to configure the config map with the serving CR as you expected.
For doc regarding custom manifests: https://knative.dev/docs/install/operator/knative-with-operators/#create-and-apply-the-knative-serving-cr  and click on the Tab "install customized knative serving".

Hi, thanks for responding. Just to confirm, then if I were to specify https://github.com/knative/net-certmanager/releases/download/v0.26.0/release.yaml in spec.additionalManifests section in the serving CR, do I not need to install this yaml separately? i.e. serving CR will install it?

When operator has this specified links in spec.additionalManifests, operator will install them for you. no need to apply separately any more.

/reopen

/reopen

/remove-lifecycle stale


Support ARM arch

Nowadays is common the scenario when users have an ARM machine with Mac M1. KEDA supports ARM64 but this example is build with non ARM64 images, so basically those users cannot test and play with KEDA locally. It'd be nice if we added the needed changes to make this demo runnable on ARM64

@JorTurFer I can work on this issue.

Sorry @rodrigoechaide didn't notice you also flagged you'd help with this. Saw someone on slack asking for this. I pushed a version of the container to GitHub registries and linked to this repo - should support both linux/arm64 and linux/amd64 archs. Feel free to check out PR if you didn't already start on this

That would be awesome. Was trying to find out the best way to get rabbitmq in the cluster last night on my M1 but didn't find a helm chart to do it - not sure if thinking is we create our own deploy.YAML to just use the rabbitmq image directly but would love some help there.

@jeffhollan What I did was to use the bitnami helm chart and modify the image to the rabbit official image that does support ARM. At least with that change rabbitmq was up and running but I couldn't go further with the demo because the rabbitmq-consumer was failing because of the lack of support of ARM. I will continue testing the full demo now that you introduce the changes to support ARM ❤.


Problem with right click for spell check

[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary
nothing happen when rigth click on a (bad) word to check spell
Environment

Operating System:
Mattermost Desktop App version: 4.6.2
Mattermost Server version: 5.33.0

Steps to reproduce
Just right click
Expected behavior
Propositions of goog words
Observed behavior
on 2 differents computers, same version of mattermost desktop, one work only
Maybe a problem of configuration ? of dependencies ? i don't know.
Thanks

on 4.7 we are changing how the dictionary works, and it uses either the operating system one (in mac) or the hunspell one (win/linux)
as far as I've tested it worked in this new version (we recently published beta 2 if you want to try)
closing as we won't be fixing it for 4.6.2, feel free to reopen if you think we need to work on it.


Query about using bluesky in AI-assisted ATC research

If this is not the appropriate channel for a general question, please point me in the right direction.
I am in the early stages of a small research project to design/implement an ATC system where some sort of external AI engine is integrated to assist a human controller in making decisions.  It seems to me that through the self-contained plugins, a skilled Python develop could notionally write a plugin to tie in such an external resource.  Could anyone close to the project confirm/deny that bluesky would be a good tool for this type of project?
Eric

Hi Eric,
You can indeed use Bluesky also for real-time applications, and it has also already been used in combination with (external) machine learning / reinforcement learning algorithms several times. If you look around at the different forks of bluesky on github you will find several implementations of bluesky combined with python RL libraries. Note that the graphical interface of BlueSky is not an ATCo screen: you might have to make one yourself if you need more than just a top-down representation of traffic. That is also possible as an external application to bluesky. You can find an example of an external client in the wiki.

Great.  Your response is very helpful for my planning.  I hope to contribute to the project in some way throughout the research.
Thank you.


Package and publish Evidence extension v1.0.0 release

[x] Bump this extension version to 1.0.0 and use major/minor versioning thereafter
[x] Add Evidence Commands and extension demo gif to readme.md
[x] Update changelog.md with new features and tickets info from this new ext. repo
[x] Update extension features list in docs
[x] Change Issues link in docs to point to this repository issues
[x] Change git url and issues link in package.json to point to this repository
[x] Add Evidence ext. keywords to this repository tags to make it discoverable on github
[x] Add brief v1.0.0 release intro to the changelog.md
[x] Rename this VS Code extension repository to evidence-vscode and update repository references in package.json
[x] Make this repository public for v1.0.0 release (@mcrascal)
[x] Change demo and commands images to use absolute url for display in VS Code marketplace after this repository is made public (@RandomFractals)
[x] Tag evidence-vscode-v1.0.0 release and create release notes with .vsix attachment (@RandomFractals)

Published: https://github.com/evidence-dev/evidence-vscode/releases/tag/v1.0.0
@mcrascal Please published attached v1.0.0 vsix to VS Code Marketplace.


npm start is not working

npm start is not working, need to test and adjust README accordingly

any updates on this?


MGHImageIO class Doxygen formatting issue

Description
The itk::MGHImageIO class has some formatting issue derived from its Doxygen documentation: the Doxygen page menu is displayed in a single column vs. in a single row, and without the expected background/highlighting.
Expected information
The Doxygen page menu is displayed in a single row.
Actual information
he Doxygen page menu is displayed in a single column and without the expected background/highlighting.
Versions
master.
Additional Information

Might be related to how the Doxygen pages are built for remote modules, as a local Doxygen build of the remote seems OK:


Update demo to latest version, create image

Demo is at [1].
There should also be a reference to the demo in the manual.
[1] http://belaban.blogspot.com/2020/12/running-jgroups-raft-as-service.html

Just checked the demo and everything is working. Also checked with JGroups 5.2.12.Final and JGroups-RAFT 1.0.10.Final.

Thanks


[Fix #3271] Fix bad auto-correct for Style/EachForSimpleLoop cop

There is a bug in Style/EachForSimpleLoop, where it auto-corrects with the wrong number of iterations. Consider:
(0..10).each { ... }

This is an inclusive range of 0 through 10, i.e. 11 iterations. This incorrectly gets auto-corrected to:
10.times { ... }

It turns out the tests for this auto-correct are incorrect, and missing test cases for the exclusive range cases.
This fix adds test coverage, and makes the auto-correct behave well for inclusive ranges.

:+1: Looks good!

👍


Use zero-install lightweight DB instead of MongoDB

Fixed. We now just write the latest snapshot to disk as JSON.


routes created by the exposecontroller now include namespace in the hostname, so create the redirects with it included

[merge]

Is having the namespace in the route what you wanted? I think I just copied over whatever it did before but not sure.

Yeah we do want the namespace in the hostname.  Actually, it's related to this issue here https://github.com/fabric8io/fabric8/issues/6360


Bug when hitting back button

In the middle of this gif, I hit a link to go to another page, and quickly hit the back button. When i am returned to the original page, the Simplebar scroll becomes buggy where the vertical scroll doesn't work anymore and the container (which doesn't have any x-overflow) suddenly has horizontal scroll.
Using simplebar in a pretty straightforward way - initializing it in my JS as so new SimpleBar($('.left-nav-brand-list')[0], {autoHide: false});
I also tried removing the jquery and just initializing it with data-simplebar and am seeing the same error.

Suspicion is that this is related to the fact that when hitting the back button, the simplebar DOM markup is still there causing incorrect height/width calculations. For example, if I go through the same flow, and instead of hitting the back button, I navigate to the previous page via the URL bar, this problem doesnt occur. Is there a workaround for this?

That's pretty hard to say without knowing your environment. This can come from multiple things.
I guess that you're working in a single page application environment? When you go to the previous page via the URL bar, it forces a page reload, so the plugin is initialised properly.
But when you hit back button, if no page reload occurs, you need to make sure the plugin is initialised properly based on what's in the DOM. You can't init Simplebar on an element that has already been initiated before. You'll have to refresh your element to it's original DOM shape.

Yeah turns out its a rails + turbolinks related issue with hitting the back button pulling up a cached version of the DOM, and the JS reinitializing simplebar even though it already exists in the cached version.
Is there a way to destroy the DOM for simplebar? Don't see anything in the documentation but wanted to check.
Thanks @Grsmto !

There is no way currently to "clean" the DOM from the Simplebar elements. That's something I thought about but didn't implement it until I actual have a good reason to do so.
That may probably be a good idea to add such a function to Simplebar. In the meanwhile you can probably woraround that by cleaning the DOM manually, or maybe keep a copy of the element before Simplebar is added to it and re-apply it when page is left. So when the cached version of the DOM kicks in, it's still the original DOM. It's quite a pain to do to be honest but that's the only solution I see...
Or Simplebar should be able to re-attach itself. Maybe try the { wrapContent: false } option!
It's a pretty weird behaviour to cache the DOM this way tho!

Got it, using the wrapContent option combined with removing the vertical and horizontal tracks before reinitializing Simplebar seems to allow it to reattach itself successfully. Kind of a hacky solution but a bit better than cleaning the DOM manually.
Thanks!


Saving transaction of annotations gets a long time and does not even happens sometimes!

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Steps to Reproduce
Just go ahead and try to save an image with a lot of annotations(after days of working).
Expected Behavior
saving data lasts 2-3 minutes at the worst case or saving the data locally somewhere, preventing data from getting lost.
Possible Solution
Persist the data locally (in the localstorage) or saving it real time(while annotating) on the database.
Context
Our annotators that use this project version 2.5 have complained that saving annotations takes a lot of time to happen and sometimes they does not even get saved after waiting for a long time (about 15 mins!) and all their work gets disappeared.
Environment
The project is deployed according to the instructions in the documentation.

I'm interested to know in which part of the source code that state of the annotation component gets handled. Can somebody give me a clue?

Hello,
We do not support 2.5 anymore. It is too old. Try the latest release first.


[Feature] Implement functionality for SupportsConcurrentConnection

Feature description
As mentioned in bugs #27694 and #24080 the SDK does not implement the GeneralCommissioning cluster attribute SupportsConcurrentConnection which according to [1] Section 11.9.5 is a Mandatory attribute.  The text below uses the example of Wi-Fi and BLE as the technology run in a non-concurrent mode.
[1] Section 5.5 indicates two modes, concurrent connection and non-concurrent connection commissioning flow. A Commissioner and Commissionee with non-concurrent connection capability cannot be simultaneously connected to both the operational network (e.g Wi-Fi) and the commissioning channel (BLE).
The MSC for the use of non-concurrent mode is shown in [1] Section 5.5, Figure 31. As stated in section 5.5 for non-concurrent mode the commissioning terminates after successful trigger of step 12 ConnectNetwork on the BLE commissioning channel and no ConnectNetworkResponse will be received by the controller on the commissioning channel  as the BLE link will have been disconnected.
This feature request is to modify the commissioner (chip-tool) to request the SupportsConcurrentConnection attribute and if disabled to immediately move from the CommissioningStage::kWiFiNetworkEnablestep to start the discovery on the operational network.
On the commissonee this feature request is to change the code (some of which is platform dependent) to wait (if the SupportsConcurrentConnection  is disabled) until the ConnectNetwork has been received and the commissioning channel stopped before configuring the operational network.
There is an implementation under construction in this forked repository, using the Linux chip-tool and lighting-app as the development vehicle on a Raspberry Pi 4.
https://github.com/simonhmorris1/connectedhomeip/tree/non-concurrent-mode
References
[1] Matter Specification R1.1
Platform
all
Platform Version(s)
No response
Anything else?
No response

Completed as part of https://github.com/project-chip/connectedhomeip/pull/30201
and subsequently as part of https://github.com/project-chip/connectedhomeip/pull/31739


Poem zooming

Here's my first swing at the poem zooming effect! I've done some refactoring to keep it as performant as I can (although I suspect this chain-of-transactions pattern I'm naively writing could be done more formally and performantly).
That said, the maths on the zooming isn't working, so you'll notice the poem is zooming off screen right now. Would love a second pair of eyes on this when you have some time, @andrewpbray; you might have a better intuition than me! It's at about L241 of scroller-init.js.
There's another issue that's a bit of a pin:
Unlike Chrome, Safari appears to masterise text before you apply a transform to it (in some circumstances—I've occasionally managed to get sharp text, mostly when I'm not using a translation, but it's tricky to reproduce!), so small poems that get scaled (scale >> 1) up appear pixelated.
When I look at how Reveal's auto sizing works, they do the opposite: start big and (mostly) scale down. They specify a "natural" width and height and the scaling based off that. So if your natural height and width are high (eg. 1400px), most of the scaling is downward.
So we could specify a large font size for poems and then scale them down.
The trouble is, when a poem is too big for the screen and is scaled down, it still reserves layout space based on its original size. That causes other stickies to miss when it comes to their margins and be pushed down (unless we just decide to apply the transform to all the stickies?).

Oops, this was meant to merge into your PR! Closing!


docs: installation guide

Description
This is the ag CLI installation and upgrade guide.
Related issue(s)
Fixes #810

Hey @alequetzalli, @derberg, @pratik2315, @Annysah, @starlightknown, @nelsonmic, @thulieblack the installation guide is ready for review. I'll appreciate any feedback/criticism I get 😄

/rtm


Finalize the branch for publication

@fractaldragonflies, please see what I have been doing here.

Adding CONTRIBUTORS.md with our names (I forgot your John E, you may want to add that).
Fixing one Line that prevents using concepticon v3.0.0, by just pulling the Concepticon Gloss labels from Concepticon and not from WOLD or IDS.
Adjusting the NOTES.md which appears in README.md after runnin cldfbench lexibank.makecldf.
Modifying metadata.json, which now gives our EACL paper as the "how to quote the dataset", which I find okay.
Adding sources for IDS and WOLD. For WOLD, we add the WOLD itself and Miller2020, since this provides the orthoprofile for WOLD, and for IDS we add Miller2023 (the new EACL paper) as it provides orthoprofiles for Spanish.

Note that in this form, we can easily make a version 1.0, and this version will then be automatically uploaded to Zenodo, where we get a DOI, and with this DOI we can then reference the data in the paper (no zips for ACL, only links to Archives!).

I tested how everything runs, and it is fine. I suggest you also test one more time. It is anyway fast.

If the Markov branch is not really important for the paper, I'd leave it out for now, and rather link this with Zenodo (which I'll do, once you tell me it is good to go) and rather concentrate on adding a few more lines to the paper. All initial ideas, you could MENTION them there, and then, you'd be the one who has articulated them first, which is probably useful.

I tested how everything runs, and it is fine. I suggest you also test one more time. It is anyway fast.

I will test out in this final form... just to be sure there are no surprises in my environment.  Of course using a virtual environment.

If the Markov branch is not really important for the paper, I'd leave it out for now, and rather link this with Zenodo (which I'll do, once you tell me it is good to go) and rather concentrate on adding a few more lines to the paper. All initial ideas, you could MENTION them there, and then, you'd be the one who has articulated them first, which is probably useful.

Let's wait to integrate the Markov Chain stuff.  I  put a mention in the updated paper as a footnote to conclusion. Not sure if that is really 'acceptable' for conference paper, but it was a way to avoid having to write into methods section, include in software, and add new results stuff, when already at 5 page updated paper limit.

Testing install.

Tried using the makevirtualenv command, but not unknown command. Did an install of virtualenvwrapper, but still no go, so something a bit more complicated.  Left aside and use my conda commands.
Haven't merged yet, so copied down zip of 'final' and renamed to sabor.  I ignored git checkout v1.0 command for now.
Install OK, and download OK.
Fails on makecldf due to reference to glottolog and concepticon databases. Here is message.

(testsabor) johnmiller@Johns-M1-Fractal-Dragon sabor % cldfbench lexibank.makecldf lexibank_sabor.py --glottolog=../glottolog --concepticon=../concepticon-data --clts=../clts --glottolog-version=v4.7 --concepticon-version=v3.0.0 --clts-version=v2.2.0

Error initializing catalog glottolog
invalid repository path: ../glottolog

Hmm... I had previously (2 years ago?) installed with 'catconfig', but at application support level for johnmiller. cldfbench knows where to look for the catalog and pulls in the appropriate files.
OK.  I'll modify command for my test for now in order to verify that the app does function.
Seems we would need to install these databases if

@fractaldragonflies, please see what I have been doing here.

Adding CONTRIBUTORS.md with our names (I forgot your John E, you may want to add that).
Fixing one Line that prevents using concepticon v3.0.0, by just pulling the Concepticon Gloss labels from Concepticon and not from WOLD or IDS.
Adjusting the NOTES.md which appears in README.md after runnin cldfbench lexibank.makecldf.
Modifying metadata.json, which now gives our EACL paper as the "how to quote the dataset", which I find okay.
Adding sources for IDS and WOLD. For WOLD, we add the WOLD itself and Miller2020, since this provides the orthoprofile for WOLD, and for IDS we add Miller2023 (the new EACL paper) as it provides orthoprofiles for Spanish.

VERY Pretty...  looks professional!

Sorry, this was written without checking against it: please replace (!) glottolog/concepticon/clts by x-repos, so --glottolog-repos, --concepticon-repos, --clts-repos.

This is also described in cldfbench --help.

Using my conda commands, and with the exception of  git checkout v1.0 command, this install works fine after installing repositories one directory above the installed sabor directory.  Links to Glottolog, Concepticon, CLTS work fine.  I may not have paid attention to previously to your detail instructions on using this command!
I haven't installed the necessary path info to get the mkvirtualenv command work.  But I assume that professionals that use other env tools will be able to figure it our.
WRT specifying v1.0, what do we need to add?  [setup.py needs to be updated to this version.  But there must be something in the git database itself?]
On my mac, install of glottolog, clts, concepticon files is made automatically to application support directory just by using cldfbench catconfig.  Quite convenient as I avoid multiple copies of databases and makecldf command doesn't even need to refer to repositories.  Very cool.  And easy to update too.

To make a version 1.0, I need to first link this repository with Zenodo (via our Zenodo account) and then, we do a release on git, where we specify the version. This is what we need now. If you think we are good to go, I'd do that tomorrow.


Teardown adapter method on waterline collection is taking wrong arguments.

This piece of code that attaches non-CRUD adapter methods onto a waterline collection does not apply the correct arguments specifically for the teardown method.  Most adapter methods begin with a connection then a collection/tableName argument, which is what is assumed here.

var args = [conn, tableName].concat(Array.prototype.slice.call(arguments));

But the teardown method is an exception this rule.  It takes a connection, but not a table name.  What ends up happening is that you get a type error: TypeError: string is not a function
It's trying to call the table name string as the callback since the arguments are all out of whack.

@devinivy, is this still an issue?

I do believe so, yup!

This is causing issues in one of the projects I am working on. See Buto/sails-oracledb-sp#3
The first argument to teardown is always null
Sails 0.11.0
Node v0.12.7

I don't have time to make a PR for this at the moment, but it should be a fairly simple change.  See the original post for details.

So we just need to tap into it and if the method is teardown only pass the connection? Sounds easy enough, I can knock this out.

Currently connection is undefined
My adapter only needs connection. It appears that the other adapters only need connection also


Add button to trigger re- classification of signs in signbank

Add button to trigger re- classification of signs in signbank

Done


Update README.md

Token must be set inside the middleware function, because the token declaration in the previous version is only initiated on page load. This causes bugs where the authorization header isn't changed on a login / logout until a page refresh.

@jarnojellesma good catch Thanks


Deprecate Makefile

Currently the Makefile assumes that GOPATH is set to a single directory, instead there could be multiple directories it could be set to concatenated with :
https://github.com/drone/drone/blob/master/Makefile#L49
Can the GOPATH be replaced with anything that works also for this case? Thanks!

I'm re-naming the issue. I think the ideal solution would be to make drone go gettable so that we don't need a makefile or scripts that rely on the GOPATH for the build process.

I would suggest fileb0x instead of bindata :)

@tboerger Good suggestion https://github.com/UnnoTed/fileb0x

I'm using https://github.com/gobuffalo/packr lately. Just to put it on the list too.

Thanks, I will definitely check out packr and filebox for drone-ui 😄
For drone/drone we actually don't store any binary assets. They are all text assets related to database queries, migration and templates. So I'm using https://github.com/bradrydzewski/togo
This is a tool of my own creation that handles embedding plain text files as plain text (strings) and generating the relevant template, migration, sql code. This is an example of what generated template code looks like, for those interested:
package template

import "html/template"

// list of embedded template files.
var files = []struct {
	name string
	data string
}{
	{
		name: "en.tmpl",
		data: en,
	}, {
		name: "es.tmpl",
		data: es,
	},
}

// T exposes the embedded templates.
var T *template.Template

func init() {
	T = template.New("_").Funcs(funcMap)
	for _, file := range files {
		T = template.Must(
			T.New(file.name).Parse(file.data),
		)
	}
}

//
// embedded template files.
//

// files/en.tmpl

var en = `<html>
<body>
Hello world
</body>
</html>
`

// files/es.tmpl

var es = `<html>
<body>
Hola Mundo
</body>
</html>
`

merged #2040


az aks get-credential needs subscription access

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command

Describe the bug

To Reproduce

Expected behavior

Environment summary

Additional context

az aks get-credential does not work if we have access to the AKS resource / RG only and not the subscription. It shows this error.
az aks get-credentials --resource-group "rg" --name "aks" --subscription "xxx"
Subscription 'xxx' not recognized.
Subscription 'xxx' not found. Check the spelling and casing and try again.
Is this the expected behavior? If so, is it called out somewhere?

route to CXP team

@uthsab Thanks for reaching out to us and reporting this issue. Please run the below command first to set the right subscription and then you can run the az aks command. That should help.

az login
az account set --subscription "XXXXXXX-XXXX-XXXX-XXXX-XXXXX"
az aks get-credentials --name MyManagedCluster --resource-group MyResourceGroup

Hope this helps.

Yes, it is working for me.
I had tried that earlier but was seeing the same error. I was suspecting that is because I have access to the AKS cluster RG only. not the entire subscription.
Apparently it was some cache issue. Az logout an az login again helped refresh the permissions.
Thanks for the help.

Hi @navba-MSFT, @yonzhan
it still happens.
Reproduce:

Create Service Principal
Grant access inside Kubernetes RBAC - not in the Azure RBAC
Login to via az cli az login --service-principal --username "xxx" --password "xxx" --tenant "xxx" --allow-no-subscriptions
az aks get-credentials --resource-group XXX --name XXX --subscription XXX
Error: Subscription 'XXX not recognized. Subscription 'XXX' not found. Check the spelling and casing and try again.

I guess it happens because service principal doesn't have access to any subscription inside Azure.


🐛 Error running reset-user-efs-home

Describe the bug.
There is an error when attempting to run the "Reset home directory" helm chart via the Control Panel UI. The error may be related to a failure to update the helm repo, as the sentry log shows that the "mojanalytics/reset-user-efs-home" chart is not found.
The full stack trace of the error can be found in sentry:
https://ministryofjustice.sentry.io/issues/3941226983/?query=is%3Aunresolved&referrer=issue-stream&stream_index=17
Note: in sentry there are multiple Helm related errors, that all may be linked
To Reproduce
Log in to control panel
Go to the tools page
Click the "reset home directory" link
Reset home directory
To a user it appears it completed successfully. This is because the task is run in the background via django-channels. So check Sentry to see the stack trace of the background task
Expected Behaviour
The task to run the helm helm chart should run successfully, and the users home directory is reset. The bug is fixed when:
the steps to "reset home directory" can be completed via Control Panel (as steps above) and there is no error reported in sentry
the commands defined in the helm chart are completed. This can be checked by connecting to the rstudio tools pod and checking that the files have been moved to the backup folder (defined in the helm chart command)
Additional context
Ticket originally captured in the Control Panel repo here https://github.com/ministryofjustice/analytics-platform-control-panel/issues/1207

Dupliate of https://github.com/ministryofjustice/data-platform/issues/1750

The root cause identified for this bug was that the name of the helm chart for reset-user-efs-home was released under a old name reset-user-home but in cpanel it uses reset-user-efs-home


Improve debug configuration providers

The following improves the debug configuration providers:

Fix a bug in the creation of debug configurations for  cppdbg and cppvsdbg
Register cppdbg and cppvsdbg providers separetly
Only register the providers if there is a debugger extension installed.

Here is an example of providing debug configuration for a debugger that doesn't exist:

I am not sure how to handle the case of a missing debugger extension:

Only register the provider if the extension exists
Always register the providers, but only provide debug configurations when the extension exists.

What line does the undefined come from?

What line does the undefined come from?

undefined comes from registering a debug configuration provider using vscode.debug.registerDebugConfigurationProvider with a debugger type that is unknown, for example registering it for the lldb debugger type when this debugger does not exists.
Debuggers are registered by extensions like CodeLLDB here registering a new debugger with type: lldb and label: LLDB.

undefined comes from registering a debug configuration provider using vscode.debug.registerDebugConfigurationProvider with a debugger type that is unknown, for example registering it for the lldb debugger type when this debugger does not exists.

Does this mean that we should check for the existence of the debugger before registering the provider?

undefined comes from registering a debug configuration provider using vscode.debug.registerDebugConfigurationProvider with a debugger type that is unknown, for example registering it for the lldb debugger type when this debugger does not exists.

Does this mean that we should check for the existence of the debugger before registering the provider?

This is what this PR does here: https://github.com/mesonbuild/vscode-meson/pull/256/commits/fb4d6a80fe18d367470fff9b5fb7cb87013e28c9
My doubt is whether to do that check when registering the provider or when listing configurations in MesonDebugConfigurationProvider.provideDebugConfigurations for the corner case in which a user has the meson extension installed and initialized and they install the extension providing the debugger support afterwards, which would require restarting vscode. Supporting that corner would make the code less obvious, and I am unsure about the real benefits.

I don't think we need to solve the corner case. I like what you have. Are you ready for me to merge it?

I don't think we need to solve the corner case. I like what you have. Are you ready for me to merge it?

It's ready to be merged :) Thanks for the review


Color Picker add an amplifier

📝 Add an amplifier when Color Picker is activated.
Color Picker is very useful, but sometimes the data of a certain pixel cann't be accurately extracted just by the cursor of the mouse. If a local amplifier can appear in the corners when Color Picker is activated, it can be selected more conveniently.

Have you tried zooming in (scroll wheel)? Does that meet your needs?

Have you tried zooming in (scroll wheel)? Does that meet your needs?

Have you tried zooming in (scroll wheel)? Does that meet your needs?

Wow~ That's so cool, I gave it a try and it worked super well. I didn't know that before. Thx.

Have you tried zooming in (scroll wheel)? Does that meet your needs?

Wow~ That's so cool, I gave it a try and it worked super well. I didn't know that before. Thx.


Module '"algoliasearch/lite"' has no exported member 'SearchClient'.ts(2305)

Line 1:
import type { SearchClient } from 'algoliasearch/lite';
In this file:
https://github.com/algolia/autocomplete/blob/next/examples/react-instantsearch/src/components/Autocomplete.tsx
is giving me the following error:
Module '"algoliasearch/lite"' has no exported member 'SearchClient'.ts(2305)
How to fix?
I'm using the following npm packages:
"@algolia/autocomplete-js": "1.17.7",
"@algolia/autocomplete-plugin-query-suggestions": "1.17.7",
"@algolia/autocomplete-plugin-recent-searches": "1.17.7",
"@algolia/autocomplete-theme-classic": "1.17.7",
"algoliasearch": "5.15.0",
"react-instantsearch": "7.13.8",
"typescript": "5.7.2"

This example still has algoliasearch v4. Either:

change to algoliasearch v4 and use the example as-is
change import type { SearchClient } from 'algoliasearch/lite' to import type { LiteClient as SearchClient } from "algoliasearch/lite"; and use v5 lite

Will leave this issue open until we update the examples to use v5 to avoid confusion

Thanks, I will try the second option in the coming days.

The following seems to be working - just updated my code:

change import type { SearchClient } from 'algoliasearch/lite' to import type { LiteClient as SearchClient } from "algoliasearch/lite"; and use v5 lite


Confusing Example 2- Dates are different

Example 2 is labelled as: Get all the alerts that created after 2018-10-20 00:00:00
Yet the code related shows:
HTTP GET  https://api.securitycenter.windows.com/api/alerts?$filter=alertCreationTime+gt+2018-11-22T00:00:00Z
Looks like the GET request is pulling data created after 2018-11-22, not 2018-10-20 as the label suggests

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

ID: ba741c25-911f-7766-5025-70cca2fc65ea
Version Independent ID: 45808cc4-01b7-3033-dd69-cdf032467064
Content: OData queries with Microsoft Defender ATP - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/exposed-apis-odata-samples.md
Product: w10
Technology: windows
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Resolved by PR #8364 - ticket can be closed.


As a user, I want to have the Kerth winners listed on the website, not just on the archives

The Kerth winners have historically been listed only on the archives. It would be more consistent to have a copy located on the Kerth website and just link to the Archive instead.

[ ] Import the Kerth winners for 1998 - 2016 to the Kerth website and provide back links.
[ ] Insert links to the Archive for the Kerth winners of 2017ff

relates to #15

relates to #17


Draft: 163 generator add the feature tweak generator

Codecov Report

Merging #195 (37197be) into development (17ea311) will decrease coverage by 0.02%.
The diff coverage is 77.95%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@               Coverage Diff               @@
##           development     #195      +/-   ##
===============================================
- Coverage        77.73%   77.71%   -0.02%
===============================================
  Files               38       43       +5
  Lines             1334     1499     +165
===============================================
+ Hits              1037     1165     +128
- Misses             297      334      +37

Impacted Files
Coverage Δ

src/CounterfactualExplanations.jl
100.00% <ø> (ø)

src/evaluation/benchmark.jl
93.33% <ø> (-1.34%)
:arrow_down:

src/generators/Generators.jl
100.00% <ø> (ø)

src/models/model_utils.jl
79.31% <ø> (+17.24%)
:arrow_up:

src/models/nondifferentiable/mlj_tree.jl
68.08% <68.08%> (ø)

src/counterfactuals/functions.jl
76.66% <70.00%> (-0.39%)
:arrow_down:

src/generate_counterfactual.jl
83.33% <75.00%> (+2.08%)
:arrow_up:

src/models/differentiable/evotree_model.jl
75.75% <75.75%> (ø)

src/generators/non_gradient_based/functions.jl
77.08% <77.08%> (ø)

src/data_preprocessing/utils.jl
60.46% <100.00%> (+21.17%)
:arrow_up:

... and 6 more

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more