Use config.base when starting dev server (v2)

Hi,
currently when making a static build using a custom base path in doczrc.js, it is possible to run the site e.g. on github pages as stated in the docs.
But that base path is also used for local development while the docz serve command will always open the browser on /.
What do you think of using the base also for starting the local browser?
https://github.com/doczjs/docz/blob/221d448f861a90037a3f35d475e3fcac1a6e2e98/core/docz-core/src/bundler/machine/services/exec-dev-command.ts#L8-L21
Besides that, why is this behavior implemented in a customized way like that even with gatsby under the hood (instead of using gatsby serve)?
Seems like the people over at gatsby had a similar issue: https://github.com/gatsbyjs/gatsby/issues/5271

Hello,

But that base path is also used for local development while the docz serve command will always open the browser on /.

I'm a bit confused here, I think you mean the docz dev command ? The code you're pointing to runs when you call docz dev and the docz serve command doesn't open your browser.
For the rest, I'll assume you mean dev, please correct me if I'm missing something.

But that base path is also used for local development while the docz dev command will always open the browser on /.

Yep this definitely adds friction to the user

What do you think of using the base also for starting the local browser?

I like that ! Under the hood the dev command is very simple :
It cds into .docz directory and calls yarn dev --port 3000 which maps to  gatsby develop --port 3000 waits until port 3000 is open and then opens a browser with the provided url
gatsby develop doesn't support opening the browser in a specific path, if I'm not mistaken, but since we're using a custom open browser method it shouldn't be too hard to open a url built with the base config argument.
Are you're interested in taking this on ?

I'm a bit confused here, I think you mean the docz dev command ?

Yeah sorry, my bad. The whole mapping stuff is messing with my head 😄

but since we're using a custom open browser method it shouldn't be too hard to open a url built with the base config argument.

Thats right! Something like http://${args.host}:${args.port}${args.base ? args.base : ''} should do the job.
BUT:
Fiddling around with base path brought some other insights:

using conifg.base generates the whole page in a sub directory and does not only prefix the link targets
instead using {prefixPath: '/custom/'} in gatsby-config.js with gatsby build --prefixPaths does the job well

So I'm not even sure whether either config.base works like expected and is just meant for another usecase or has some issues in the current implementation?
Anyways for my desired effect with the prefixed link targets the generated .docz/package.json should include --prefixPaths option, when pathPrefix is specified in gatsby-config.js.

Anyways for my desired effect with the prefixed link targets the generated .docz/package.json should include --prefixPaths option, when pathPrefix is specified in gatsby-config.js

So you're suggesting,

For docz dev config.base is ignored and the docs are served to base "/"
For the build process we add --prefixPaths if a prefixPath variable is specified in gatsby-config.js
Deprecate config.base in favor of using pathPrefix in gatbsy-config.js

I think it would be easier to use config.base only, to decide if we should use pathPrefix or not when building, to help people migrating from v1.
We'd have to remove the behavior you described :

using config.base generates the whole page in a sub directory and does not only prefix the link targets

And pass the config object the build command
From here:
https://github.com/doczjs/docz/blob/7b31589cdbd2f434917f4e833537c4c5452dee1f/core/docz-core/src/commands/build.ts#L19
to here :
https://github.com/doczjs/docz/blob/7b31589cdbd2f434917f4e833537c4c5452dee1f/core/docz-core/src/bundler/build.ts#L12-L13
and change the command from yarn build :
- spawn.sync('yarn', ['build'], { stdio: 'inherit' })
+ spawn.sync('yarn', ['build', config.base?'--prefixPaths':''], { stdio: 'inherit' })

And last, add pathPrefix to the generated gatsby config file :
https://github.com/doczjs/docz/blob/7b31589cdbd2f434917f4e833537c4c5452dee1f/core/docz-core/templates/gatsby-config.tpl.js
You're right this is much bigger than I assumed 😄

Wasn't suggesting anything - just mentioning. But your approach sounds reasonable.
So following things to do (worked quite well with the list the last time):

Prevent config.base from influencing the output structure
Pass config to bundler/build.ts
If config.base is set, apply option --prefixPaths to build command (and thus to gatsby build)
Copy config.base as pathPrefix to gatsby-config.tpl.js

I'll see what I can do :)

Awesome ! Take your time and no pressure at all :)

👍 Merged and released as 2.0.0-rc.40

Does this change break the gatsby-theme-docz use case where you want to add docz to an existing Gatsby site? For example, a gatsby-config.js like:
module.exports = {
  siteMetadata: {
    title: 'Some Gatsby site',
    description: 'This is a Gatsby site that has other pages, not just docz',
  },
  plugins: [
    {
      resolve: 'gatsby-theme-docz',
      options: {
        title: 'Some Gatsby Site Docz',
        description: 'Docz for Components used to build the other pages in the Gatsby site',
        base: '/components/',
      },
    },
    // ... other gatsby plugins and themes here
    //  These other plugins and themes might create other pages.
  ],
}

Prior to 2.0.0-rc.40, this would result in all of the docz pages living under /components/, along side any other Gatsby pages. Importantly, this meant that there was no danger of overlap in pages between docz and any other plugins/themes. Now, the docz pages are being flattened into the Gatsby site, resulting in collisions, the most likely of which is the docz index overwriting the Gatsby site's index page (conventionally located at /src/pages/index.jsx).
Is there a way to meet this use case after rc.40?


Create a NativeModule to control swiping gestures

Thanks, looks good now from my point of view.
Please squash commits.

What happend here? why is it now 134 files changed?

^ @tbergq it's solved.

^Fixed


Gold/silver price tracker

Real-time access to market prices for gold, silver, and other precious metals, and instant updates on the current market prices for gold, silver, and other precious metals.
Historical Data: Access historical price data to analyze trends and make informed decisions.
Customizable Alerts: Set personalized alerts based on price thresholds to stay informed of significant market movements.
please assign me this under GSSoc'24
@Sulagna-Dutta-Roy

Already added to my repository #507. Closing this


Bug: Lexical Conversion to markdown conversion and vice versa not working properly for multiple combination of Bold, Italic and strikethrough with space seperated

Lexical version: 0.20.0
Steps To Reproduce

https://playground.lexical.dev/
Click on bold -> Write "Test" -> Provide space -> Click on Italic -> Write "Text"
Click in markdown converter
Click on lexical view
Text will be distorted and unable to parse converted markdown in lexical

Link to code example:

The current behavior

The expected behavior
After conversion from markdown to lexical, it should be like below and value should be intact and same as before

Impact of fix
We have started using this library for our critical application but lexical library is not working for basic operations mentioned above which is not expected. Can you please try to fix this issue asap as users already started complaining about this issue in production?

I think this is a duplicate of #7009


chore(master): release 1.4.12

:robot: I have created a release beep boop
1.4.12 (2023-09-12)
Miscellaneous

deps: update docker/login-action action to v3 (f7a2445)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/docker-rabbitmq/releases/tag/v1.4.12 :sunflower:


Failed auto installation of iscsi on Debian 11 bullseye

Hello,
The auto installation failed on node running Debian 11. In the pod I had a "sudo yum not found" error.
I saw that the "ID_LIKE" field does not exists in debian 11 /etc/os_release file, there is only ID=debian.
https://github.com/longhorn/longhorn/blob/184ad405847788a19642aa88a36033cf8c956cee/deploy/prerequisite/longhorn-iscsi-installation.yaml#L8
root@awtprdkubworker01:~ # cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

As per this manpage: https://manpages.debian.org/testing/systemd/os-release.5.en.html ID should be used first and if not recognized, fallback to ID_LIKE.
Ping @innobead

On debian11 node, the ID_LIKE is not set
root@ku70-master:~# cat /etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"

Will fix it by following the spec
Build scripts and similar should check this variable if they need to identify the local operating system and the value of ID= is not recognized. Operating systems should be listed in order of how closely the local operating system relates to the listed ones, starting with the closest. This field is optional.

PR: https://github.com/longhorn/longhorn/pull/3330

Verified by @longhorn/qa already


Fix Option for CVE-2024-4068: Memory Exhaustion in Braces

Proposing as a possible solution if it fits.
Issue Description:  "braces" fails to limit the number of characters it can handle, which could lead to Memory Exhaustion. In "lib/parse.js," if a malicious user sends "imbalanced braces" as input, the parsing will enter a loop, which will cause the program to start allocating heap memory without freeing it at any moment of the loop. Eventually, the JavaScript heap limit is reached, and the program will crash.
To address this issue I added a check for imbalanced braces after the maxLength check. When tested it no longer leads to "Javascript heap out of memory" and will instead throw syntax error when imbalanced braces are used similar to when input is above the max length.

Yes, hope this gets published soon.

When this PR will get merged?

it needs tests.

Was there a reason why this PR was closed instead of merged? I see tests weren't added either

@quiquetubi no particular reason. I was not in the mood of keeping it open.


Bootstrap Express.js committers

With the moving of this repository into the expressjs GitHub organization and the three GitHub organizations expressjs, pillarjs, and jshttp, there is no organized list of committers/TC for this new super structure.
In order to get this bootstrapped, we are moving forward with the plan agreed upon in the latest Express.js meeting (https://github.com/expressjs/discussions/issues/1):
Data will be collected by analyzing the commits in all the repositories in the above three repositories. From this data of existing committers, the new agreed upon contributing guide (https://github.com/expressjs/express/blob/master/Contributing.md) will be applied to the list to create a group of committers for Express.js (see section "Becoming a Committer").
This new committers list will then also become the Express TC group, and from this point forward, on-boarding new committers to this list will follow the contributor guide and the set of committers that are the TC will be managed according to the contributor guide.
@mikeal has volunteered to adapt an existing Node.js script to collect the set of data to bootstrap the committers from.

With issue #2922 the TC has been bootstrapped!
Initial TC members (in no particular order):

@hacksparrow
@jonathanong
@dougwilson
@crandmck
@blakeembrey
@niftylettuce
@LinusU
@Fishrock123
@troygoode


Adding notebooks for the SEA tutorial

Added notebook on working with GINI files (and as an intro to Siphon)
Updated radar server notebook

Incorporated out of band comments from @gobie28.


How to compile on AVR (Arduino)?

Hi there,
I'm a total Rust novice, so please forgive my ignorance. I was excited to find your u16 math functions, but I'm getting an error when trying to build for arduino.
Starting from the rust arduino example https://github.com/avr-rust/blink/ , I add cichlid to Cargo.toml:
 [dependencies]
 ruduino = "0.2"
+cichlid = "0.2.1"

Then, I build according to the instructions in the repo:
rustup override set nightly

# Compile the crate to an ELF executable.
cargo build -Z build-std=core --target avr-atmega328p.json --release

That produces the following error:
info: using existing install for 'nightly-x86_64-unknown-linux-gnu'
info: override toolchain for '/home/oliver/code/rust/blink' set to 'nightly-x86_64-unknown-linux-gnu'

  nightly-x86_64-unknown-linux-gnu unchanged - rustc 1.48.0-nightly (d006f5734 2020-08-28)

   Compiling bitflags v0.7.0
   Compiling compiler_builtins v0.1.32
   Compiling core v0.0.0 (/home/oliver/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/core)
   Compiling lazy_static v0.2.11
   Compiling cichlid v0.2.1
   Compiling xml-rs v0.4.1
   Compiling xmltree v0.4.0
   Compiling avr-mcu v0.2.6
   Compiling ruduino v0.2.0
   Compiling rustc-std-workspace-core v1.99.0 (/home/oliver/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/library/rustc-std-workspace-core)
error[E0463]: can't find crate for `std`
  |
  = note: the `avr-atmega328p-13777972876759820720` target may not be installed

error: aborting due to previous error

For more information about this error, try `rustc --explain E0463`.
error: could not compile `cichlid`.

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: build failed

I found this pretty surprising, because it seems like you specifically support no-std, but I can't quite figure out how to make it work. Any advice would be appreciated.
Thanks!
Oliver

I realize now that I just needed to opt-in to the no-std feature for cichlid using the extended dependency syntax in Cargo.toml instead of the shorter version I used above. I hope this helps another beginner!
 [dependencies]
 ruduino = "0.2"
+
+[dependencies.cichlid]
+version = "0.2.1"
+features = ["no-std"]


Add a switch to activate New UI

Can we let the users decide to activate or not the new UI feat(ui): landing page new design #11630, by adding a switch in the user preference page ? Also, we can move the language setting to this preference page.

@louie007 we won't be adding any switch for this. Any reason for this?

@louie007 if you have any feedback please provide us on the Slack (https://slack.open-metadata.org) but maintaining 2 designs and layouts is just too much work and impossible to get things right.
Based on community feedback we are moving with this design.

@louie007 if you have any feedback please provide us on the Slack (https://slack.open-metadata.org) but maintaining 2 designs and layouts is just too much work and impossible to get things right. Based on community feedback we are moving with this design.

Thanks for your response @harshach, I think some folks will prefer the classic design like me. As your said maintaining 2 designs is difficult, I totally agree, and will check the community feedback soon.


Logging a message only to one log

How can a given transport once it logged a message to keep it only for itself and not pass it along to other transports? I want to prevent copying the same log message into multiple transports, if at least one handled it.
Here is my configuration:
        new winston.transports.DailyRotateFile(
          {
              name: 'mainlog', level: 'info', handleExceptions: true,
              filename: activeConfig.logger.main,
              maxsize: 100000000, maxFiles: 10,
              datePattern: '.yyyy-MM-dd'
          }
        ),
       new winston.transports.DailyRotateFile(
            {
                name: 'applicationlog', level: 'verbose',
                filename: activeConfig.logger.application,
                maxsize: 100000000, maxFiles: 10,
                datePattern: '.yyyy-MM-dd'
            }
        )

What I want is when I log a message with level 'error' to only go to the 'mainlog', not to both files - 'mainlog' and 'applicationlog'. The current behavior writes messages with 'info' and above to both places.

You need to make sure that the level you are logging to is correct. Only log messages with a level greater than the level set on your transports will flow through.


Deprecate the use of the display keyword for datasets

Following on from the dataset API discussion in #35, I noticed that many of the datasets contain a display kwarg that is typically unimplemented. In the few cases where it is implemented (e.g. iris), I could not find any references in the code base where display=True. A couple of issues briefly mention the use of display=True but it isn't really clear what the purpose of display is (see, for example https://github.com/slundberg/shap/issues/1359)
Perhaps we should consider removing this keyword and its existing implementations?

It's used in a couple of places, e.g. https://shap.readthedocs.io/en/latest/example_notebooks/tabular_examples/tree_based_models/NHANES I Survival Model.html
From the code comments, it's just for having human-readable feature values in the shap plots. E.g. Female/Male instead of True/False.

That said:
I'm ambivalent about removing the display parameter, for now. It does help to keep the notebook examples a little shorter, but I can understand the sentiment to remove it, it's just that the display parameter isn't really relevant for many of the datasets.

Ah of course. I just did a quick check using GitHub referencing and it must have only looked through the Python files. I forgot to check the notebooks.
This is pretty low priority so I don't have a strong opinion. Considering that the datasets are only really there for demonstrative purposes, is there really a need for display=True? Why not make each dataset return the human readable display version by default?


merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Coverage remained the same at 14.035% when pulling 4dbd74581d0bb2b96e1f089b75dfd2a2fbbaf337 on template-sync-1 into c42342764ed977012605b6bfee1c6c684b1bd6a4 on master.


solves issue Add a map showing all the countries that fellows are loc…

…ated in #10
Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

We have added a map to show the locations of all the pod members as a pin. On clicking this pin, a popup will be displayed showing a one-liner description about the respective member.
This is important as a visual representation of location is a better representation of the location and is a pleasing UI experience for the viewers.
We have implemented the above by using Mapbox GL JS to create an interactive map and place interactive pins and popups.   Attached below is how the map looks on the website.

May be we can add images of all pod members on their respective locations..Except that pr looks clean

Yeah even I thought that too but decided to push this much for this issue and opened a separate issue for that.

@yashika51 I think the PR is ready to be merged

@AnushaNathRoy conflicts :)

@yashika51 I think it is ok now


Use absolute URLs in API section

We use relative links in our API section for submodules for example. However, these won't work if user comes directly to /docs/api (without trailing /). The proper solution here should be to use absolute links.
Repro:
Click this link without trailing slash and try to navigate to a submodule - should result in 404.

Working on this

Looks like the docs/api files are no longer in the repo ? I was able to setup the docs on my local system and make the changes, but no way to track the file changes

Hi @JacksonIsaac the resulting md files are generated using pdoc3, thus not part of the repo. Changes need to be made here. You can test it out using cd py && make docs which will run the python docs build step.

I tried testing by 'make docs', and the build directory does get created, but the sub-folders are empty. The log does show the files being listed but something is going wrong perhaps. Could you maybe guide me please on this ? :)
Here's my output log: https://privatebin.net/?89b64ef4945fa8a5#8pM5tqdzdm9HL7g7a58MnJQQ4uLWADEfcTVJ3UGSwXTD

build directory does get created, but the sub-folders are empty

Yep, this is expected. If you have a closer look at the make target you can notice that the built assets are then moved over to website/docs/api folder. That's where you can find the results.
Note: Once your fix is ready, you need to run docs website to verify everything works as expected.

I tried few different approaches for this,
added target="_self" in  tags, or href="./{}", but both doesn't seem to lead me anywhere. Any more ideas perhaps ? :)
I am assuming it can be something on link(m) https://github.com/h2oai/wave/blob/master/py/docs/templates/text.mako#L94 that I can try, or trying to handle submodules, but not sure how this works

Seems like this is the function you need to look into.

I have been researching a bit, and also tried few different ways to try solve this, without much luck.
Also, at pdocs for Doc.url:https://pdoc3.github.io/pdoc/doc/pdoc/#pdoc.Doc.url I see it is about relative URL. Couldn't find much about absolute URL (External object has a .url which returns absolute URL). Not sure if I am looking at the right place.
Do you perhaps any guidance on this ? :)

I would start by looking into the correct docs. We use pdoc3, not pdoc :) I would first aim for some simple solution that pdoc3 would provide for free. If not successful, I would look at the code around and see if I can construct the absolute URLs manually which should basically be not so hard.


Fixes for #253 and #254

Issue #, if available:
#253
#254
Description of changes:
Updated WebApplication\ 1_StaticWebHosting\README.md file to clarify two source code repo options and updated Amplify Console instructions to align with latest web UI.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Chris Munns is going to fix #254 while I'm going to submit a separate PR for #253


Doc updates & additions for SPFx v1.16 release

Category

[x] Content fix
[x] New article

Related issues

n/a

What's in this Pull Request?
Updated docs to reflect changes in upcoming SPFx v1.16 release, including:

update supported node versions
update how all release versions are listed in TOC
update getting started with SPFx dev env to be current with
update guidance on obtaining teams context
add doc on creating MS Teams app for use in MS Teams, Outlook & Office

include docs stating how to get started and use it within your tenant

Learn Build status updates of commit c305492:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/spfx/release-1.16.md
:warning:Warning
View
Details

docs/images/msteams-app-leadassistdashboard-install-apps.png
:white_check_mark:Succeeded
View

docs/images/msteams-app-leadassistdashboard-installed-app-sharepoint-01.png
:white_check_mark:Succeeded
View

docs/images/msteams-app-leadassistdashboard-installed-app-teams-01.png
:white_check_mark:Succeeded
View

docs/images/msteams-app-leadassistdashboard-installed-app-teams-02.png
:white_check_mark:Succeeded
View

docs/spfx/office/overview.md
:white_check_mark:Succeeded
View

docs/spfx/set-up-your-development-environment.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

docs/spfx/release-1.16.md

Line 66, Column 7: [Warning: bookmark-not-found] Cannot find bookmark '#sdk-versions' in 'spfx/release-1.16.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit a3e10a2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/images/msteams-app-leadassistdashboard-install-apps.png
:white_check_mark:Succeeded
View

docs/images/msteams-app-leadassistdashboard-installed-app-sharepoint-01.png
:white_check_mark:Succeeded
View

docs/images/msteams-app-leadassistdashboard-installed-app-teams-01.png
:white_check_mark:Succeeded
View

docs/images/msteams-app-leadassistdashboard-installed-app-teams-02.png
:white_check_mark:Succeeded
View

docs/spfx/integrate-with-teams-introduction.md
:white_check_mark:Succeeded
View

docs/spfx/office/overview.md
:white_check_mark:Succeeded
View

docs/spfx/release-1.16.md
:white_check_mark:Succeeded
View

docs/spfx/set-up-your-development-environment.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Azure DevOps CLI issue in Windows Server 2019

Description
Running az pipelines pool list --organization $Organization --pool-name $PoolName in Windows throws the following exception:
I'm currently at commit 956f9257505d3d10e63220e93da37dfd37a55778  (not too far behind).
2020-09-28T02:48:48.3895140Z ##[error]UnexpectedError: The command failed with an unexpected error. Here is the traceback:
[WinError 5] Access is denied: 'C:\\Program Files\\Common Files\\AzureCliExtensionDirectory\\azure-devops\\azure_devops-0.18.0.dist-info'
Traceback (most recent call last):
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-2gf5i3x9\knack\cli.py", line 215, in invoke
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cep_dxyk\azure\cli\core\commands\__init__.py", line 509, in execute
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cep_dxyk\azure\cli\core\__init__.py", line 436, in load_command_table
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cep_dxyk\azure\cli\core\__init__.py", line 307, in _update_command_table_from_extensions
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-cep_dxyk\azure\cli\core\extension\__init__.py", line 156, in get_metadata
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-2izf0p5u\pkginfo\wheel.py", line 16, in __init__
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-2izf0p5u\pkginfo\distribution.py", line 108, in extractMetadata
  File "C:\Users\VSSADM~1\AppData\Local\Temp\pip-unpacked-wheel-2izf0p5u\pkginfo\wheel.py", line 33, in read
PermissionError: [WinError 5] Access is denied: 'C:\\Program Files\\Common Files\\AzureCliExtensionDirectory\\azure-devops\\azure_devops-0.18.0.dist-info'
To open an issue, please run: 'az feedback'

I'm unsure if it's during the installation of the extension, but we (almost) follow the exact same pipelines as per this repo, except we publish to Shared Image Gallery.
The same PowerShell function in Ubuntu 20.04 works perfectly.
Area for Triage:

Deployment/Release
Scripting and command line

Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] ~macOS 10.15~ (we do not have this image to try)
[ ] ~Ubuntu 16.04 LTS~ (we do not have this image to try)
[ ] ~Ubuntu 18.04 LTS~ (we do not have this image to try)
[ ] Ubuntu 20.04 LTS
[ ] ~Windows Server 2016 R2~ (we do not have this image to try)
[X] Windows Server 2019

Expected behavior
CLI has the appropriate permission to the tool.
Actual behavior
CLI doesn't have appropriate permission to the tool.
Repro steps
A description with steps to reproduce the issue. If your have a public example or repo to share,
please provide the link.

Step 1 - Azure Powershell in Azure Pipelines (Release)

Write-Output "Authenticating to Azure DevOps"
$env:AZURE_DEVOPS_EXT_PAT = $AccessToken; # PAT that has appropriate permissions to our Azure DevOps instance

Write-Output "Find Pool"
$pools=$(az pipelines pool list --organization $Organization --pool-name $PoolName);

Step 2
Error mentioned above.

@martingemme - Confirmed in the packer file - Windows2019-Azure.json
* Bad script
'/agent/_work/1/s/virtual-environments/images/win/scripts/Installers/Install-AzureDevSpacesCli.ps1':
stat
/agent/_work/1/s/virtual-environments/images/win/scripts/Installers/Install-AzureDevSpacesCli.ps1:
no such file or directory

Hello, @martingemme
Could you please provide image version which you are using? I can't reproduce the issue:
Current image version: '20200920.1'

Current image version: '20200927.0'

I'm unsure which image version, we're using our own yyyy.mm.ddBuildNumber scheme.
As stated I'm currently at commit 956f9257505d3d10e63220e93da37dfd37a55778 (not too far behind).
Is there something I could try? We have the same scripts

I updated our repo with the latest and will post back if main (at the time of writing https://github.com/actions/virtual-environments/commit/854d90d2058963f1e3cc0c7c404857aa9c898910) has fixed it.

Unfortunately still the same issue. I'm at 854d90d

@martingemme , Could you please provide output with whoami /groups?

@al-cheb

@al-cheb

@martingemme, Try to provide full permissions:
- name: grant Full Access to AZURE_EXTENSION_DIR directory
   run: |
         icacls "$env:AZURE_EXTENSION_DIR" /q /c /t /grant Everyone:F

Unfortunately, the command didn't work. Thanks for the help so far!

@martingemme, As  we can see above your token has Mandatory Label\Medium Mandatory Level = standard user rights. You should run process with Mandatory Label\High Mandatory Level=administrator rights. During image generation we disable UAC - https://github.com/actions/virtual-environments/blob/9110b389998666dc0c6a8e9abd9400248bad8dba/images/win/scripts/Installers/Initialize-VM.ps1#L28

We're using VMSS with the Re-image turned on in Azure DevOps. Therefore we do not go into the agents and there is actually in the script the execution of Disable-UserAccessControl at https://github.com/actions/virtual-environments/blob/9110b389998666dc0c6a8e9abd9400248bad8dba/images/win/scripts/Installers/Initialize-VM.ps1#L69
(I believe) The Task Scheduler is also managed by Azure DevOps (and installed by Azure DevOps), so I cannot change on an instance basis when they are always re-imaged after each build.
I verified and in our repo, we have that same line in script that should disable UAC. Perhaps it is an issue that should be fixed from the Azure team?

I confirm that the user is not running the powershell script with admin rights.
The question I thus have if I built the image from your source: Why doing az pipelines list fails because az cli can't access a directory? shouldn't any user be able to run the az cli? why do I need admin rights to run az cli?

whoami /groups

$folderName = "C:\Program Files\Common Files\AzureCliExtensionDirectory\azure-devops"
Write-Output $folderName
(get-acl $folderName).access | ft IdentityReference,FileSystemRights,AccessControlType,IsInherited,InheritanceFlags -auto

The question I thus have is if I built the image from your source:
Why doing az pipelines list fails because az cli (and its extensions) can't access a directory?

Shouldn't any user be able to run the az cli and its extensions?
Why do I need admin rights to run az cli and its extensions?

This is bizarre as the user is in the Administrators group, and the folder has full control for administrators (the group). We're not sure why this is going on.

UAC - https://docs.microsoft.com/en-us/windows/security/identity-protection/user-account-control/how-user-account-control-works
When an administrator logs on, two separate access tokens are created for the user: a standard user access token and an administrator access token. The standard user access token contains the same user-specific information as the administrator access token, but the administrative Windows privileges and SIDs are removed. The standard user access token is used to start apps that do not perform administrative tasks (standard user apps). The standard user access token is then used to display the desktop (explorer.exe). Explorer.exe is the parent process from which all other user-initiated processes inherit their access token. As a result, all apps run as a standard user unless a user provides consent or credentials to approve an app to use a full administrative access token.
That's why you don't have full access, because you token is Mandatory Label\Medium Mandatory Level = standard user rights.
In our agents as you can see with disabled UAC we have Mandatory Label\High Mandatory Level=administrator rights:

@al-cheb Thanks for the input!
For the people that would like to fix it for their own, we have a VMSS so the fix was to disable (or toggle off) "Configure VMs to run interactive tests" like so:

Thanks @al-cheb again!

@martingemme, Thank you for providing resolution.
Put small notes - https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/scale-set-agents?view=azure-devops
Configure VMs to run interactive tests (Windows Server OS Only) - Windows agents can either be configured to run unelevated with autologon and with interactive UI, or they can be configured to run with elevated permissions. Check this box to run unelevated with interactive UI. In either case, the agent user is a member of the Administrators group.


fix cocometric if result not exists

Motivation
When evaluating a small number of images, inference results may not be obtained, and cocometric may not terminate normally.
Modification
Fixed to use empty COCO class when the inference result json file is empty.
BC-breaking (Optional)
nothing
Use cases (Optional)
Since it is a minor case, it seems that just modifying the unit test is sufficient.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMPreTrain.
The documentation has been modified accordingly, like docstring or example tutorials.

 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.


#74 reproduced and fixed bag

Many thanks for the PR, let me find a reviewer for it

@pinaf please review, thanks

@dmzaytsev look great!

@rultor merge

@rultor merge

@pinaf Thanks for your request. @yegor256 Please confirm this.

@pinaf thanks!

@dmzaytsev two comments from me

@yegor256 @pinaf review again please

@dmzaytsev see my comment above

@pinaf @yegor256 If you have any more comments let me know please

@dmzaytsev i'm good.
@rultor merge

@dmzaytsev i'm good.
@rultor merge

@pinaf I see you're talking about me, but I don't understand it. If you want to say something to me directly, start a message with @rultor

@rultor merge

@rultor merge

@pinaf Thanks for your request. @yegor256 Please confirm this.

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 20min)

@pinaf paid, thanks, added 19 mins to your account, payment AP-3HM86940MA9346400, 46 hours and 4 mins was spent
you're getting extra minutes here (c=4)
+19 added to your rating, current score is: +5746

@rultor deploy

@rultor deploy

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor deploy

@davvd Done! FYI, the full log is here (took me 12min)


Create child wraps using type(self)

Hi,
I think it would be nice to use type(self) in the _get_or_create_child_wrap method. This way subclasses of Wrap will return instances of the current subclass.

Can you change type(self) to self.__class__?

Done :)

Thanks!


Webapp gets incomplete data from the MariaDB for Jenkins

Tried to build a simple syslog config using this JSON data:
{"syslog": [{"loghost": "<IP_ADDRESS>","facility": "daemon","level":"info"},{"loghost": "<IP_ADDRESS>","facility": "any","level": "warning"}]}

Run write_yaml_files.py from within the jenkins container to generate ./group_vars/spines.yaml:
syslog:
- loghost: <IP_ADDRESS>
  facility: daemon
  level: info
- loghost: <IP_ADDRESS>
  facility: any
  level: warning

And use this J2 template to build the Junos config:
system {
  syslog {
    {% for host in syslog %}
      host {{ host.loghost }} {
        {{ host.facility }} {{ host.level }};
      }
    {% endfor %}
  }
}

If running from the shell, the build process works correctly. If running from the Webapp, Ansible fails with this error:
AnsibleUndefinedVariable: 'ansible.parsing.yaml.objects.AnsibleUnicode object' has no attribute 'loghost'"

This error is because the YAML file was not fully built by the "Build" job. However, the Excel data is all correctly stored in the "ngcn_worksheets" table inside the "Sites" database (running in the MariaDB container). So somehow, this data is not getting extracted correctly by the webapp when it runs the Jenkins job.

Confirming issue. MariaDB has both rows of data but the data.json that is generated when the ansible-playbook is run  run by jenkins only gets the last row of data.
Tested against ansible 2.10 as well just in case it is some oddball issue with ansible. Same result.

Yes that was my understanding as well, pretty sure the issue is around views.py where the webapp gets the info from mariadb and generates the data.json.

Confirmed based on Debug info that data is being truncated in views.py code. Output in log shows data set at line 207 but my 924 the extra rows are gone. Querying Maria DB directly using the same selector/order by as the python code returns the data as expected. So something from the data retrieval into json file is losing fidelity. This might be being done by built-in functions so maybe a bug upstream (json library maybe?)

Potential workaround for now...for multi-line entries like syslog you can create a second tab called anything (I chose syslog2) and the yaml files get created properly and the data is there in the debug log for the data.json file being pulled by views.py.
Generated Yaml file on test data:
OS_version: 18.1R3-S5.3
healthbot_device_group: dc1-spines
loopback_ip: <IP_ADDRESS>
management_interface:
int: em0
ip: <IP_ADDRESS>
mask: 16
syslog:
facility: any
level: warning
loghost: <IP_ADDRESS>
syslog2:
facility: daemon
level: info
loghost: <IP_ADDRESS>
Ansible still doesn't like "loghost" as a variable name, for some reason. I suspect it might be unrelated to this.

Thanks Matt, I've tried your workaround and I can confirm that it works, thank you very much. We can caveat this behaviour in the docs for now and move on. Whether it's an ER or a bug I don't know, I'll defer to you since you spent longer looking into the code than I did :-)

Doing a basic test on the example data definitely shows that this problem is not caused by yaml2xls:
test.yaml
syslog:
- loghost: <IP_ADDRESS>
  facility: daemon
  level: info
- loghost: <IP_ADDRESS>
  facility: any
  level: warning

yaml2xls.py test.yaml test.xlsx produces a file test.xlsx
then you can convert this file back to yaml by doing
xls2yaml.py test.xlsx tmp and that produces:
tmp/test.yaml
---
syslog:
- loghost: <IP_ADDRESS>
  facility: daemon
  level: info
- loghost: <IP_ADDRESS>
  facility: any
  level: warning

Which has slightly different ordering but nothing important has changed.

This seems to be resolved by an undocumented feature in NITA. If we change the column header on syslog to be host+.facility and host+.level the views.py subroutines use ordered_dicts as fields instead of strings. So we go from this:
('syslog', OrderedDict([('loghost', '<IP_ADDRESS>'), ('facility', 'daemon'), ('level', 'info')]))])),
To this:
('syslog', OrderedDict([('loghost', '<IP_ADDRESS>'), ('host', [OrderedDict([('facility', 'any'), ('level', 'warning')]), OrderedDict([('facility', 'daemon'), ('level', 'info')])])]))])),
Then after that I believe it should just be updating the j2 template to iterate through each element like the evpn_port template does in the examples. If you look at the example files for the data center use case evpn_port tab there is "+" on the tab name too. I assume it’s doing some sort of larger grouping that may not matter for the syslog example (I’m almost sure that’s it honestly). However, this worked without the tab name being "syslog+". Something to explore further while working on developer's guide.

This sounds like a documentation bug against nita-yaml-to-excel


Enable the StorageAccount Encryption and Enforce HTTPS

What this PR does / why we need it:
The PR enables the encryption at rest and encryption in transit for the StorageAccount provisioned from acs-engine for Kubernetes.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
It's a requirement from the internal security review of CloudShell.
Special notes for your reviewer:
Release note:

@robbiezhang,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@anhowe , to also TAL

@robbiezhang , looks the e2e tests failed. AndI took a look at the error, it is something we fixed yesterday. So you should just need to rebase on the master, the test should pass afterwards.

I guess for this scenario, an E2E test makes more sense?

This is the following of this closed PR #406.
Good idea to add "supportsHttpsTrafficOnly" property.
I agree with @anhowe, enabling this feature via a flag would be great.
Thx a lot.

@robbiezhang, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot


Minor copy edits, APA bib style/capitalization, and serial commas

Cc: https://github.com/openjournals/joss-reviews/issues/1700

Thank you so much Jed @jedbrown for identifying and fixing these issues. We'll look at your other comments posted in JOSS asap.
@moustachio-belvedere @ab2425


Checker does not work in latest pyflakes==2.2.0

steps to reproduce the bug
Write very simple but not correct code, and execute :SyntasticCheck. I got

After downgrade pyflakes to pyflakes==1.6.0, I got the correct result.

And the debug information was
"main.py" [変換済] 1L, 19C
syntastic: 6.538853: g:syntastic_version = '3.10.0-7 (Vim 800, Linux, GUI)'
syntastic: 6.539005: &shell = '/bin/bash', &shellcmdflag = '-c', &shellpipe = '2>&1| tee', &shellquote = '', &shellredir = '>%s 2>&1', &shelltemp = 1, &shellxquote = '', &autoc
hdir = 0, &shellxescape = ''
syntastic: 6.539558: UpdateErrors: default checkers
syntastic: 6.539782: CacheErrors: default checkers
syntastic: 6.540696: g:syntastic_aggregate_errors = 0
syntastic: 6.540878: getcwd() = '/home/nakagawa/Desktop'
syntastic: 6.548083: CacheErrors: Invoking checker: python/pyflakes
syntastic: 6.548628: SyntasticMake: called with options: {'errorformat': '%E%f:%l: could not compile,%-Z%p^,%E%f:%l:%c: %m,%E%f:%l: %m,%-G%.%#', 'defaults': {'text': 'Syntax er
ror'}, 'makeprg': 'pyflakes main.py', 'env': {'TERM': 'dumb'}}
syntastic: 6.888344: system: command run in 0.339400s
syntastic: 6.888474: checker output: ['main.py:1:1 ''numpy as np'' imported but unused', '']
syntastic: 6.888769: raw loclist: []
syntastic: 6.888914: getLocList: checker python/pyflakes returned 1
syntastic: 6.888999: python/pyflakes raw: []
syntastic: 6.889108: quiet_messages filter: {}
syntastic: 6.889186: getLocList: checker python/pyflakes run in 0.340994s
syntastic: 6.889415: CacheErrors: Invoking checker: python/pycodestyle
syntastic: 6.889822: SyntasticMake: called with options: {'subtype': 'Style', 'errorformat': '%f:%l:%c: %m', 'makeprg': 'pycodestyle main.py', 'env': {'TERM': 'dumb'}}
syntastic: 7.148497: system: command run in 0.258406s
syntastic: 7.148621: checker output: ['']
syntastic: 7.148832: raw loclist: [{'lnum': 0, 'bufnr': 0, 'col': 0, 'valid': 0, 'vcol': 0, 'nr': -1, 'type': '', 'pattern': '', 'text': ''}]
syntastic: 7.149000: getLocList: checker python/pycodestyle returned 0
syntastic: 7.149082: python/pycodestyle raw: [{'lnum': 0, 'bufnr': 0, 'col': 0, 'valid': 0, 'vcol': 0, 'nr': -1, 'type': 'E', 'pattern': '', 'subtype': 'Style', 'text': ''}]
syntastic: 7.149205: quiet_messages filter: {}
syntastic: 7.149282: getLocList: checker python/pycodestyle run in 0.259791s
syntastic: 7.149472: aggregated: {'_sorted': 0, '_name': '', '_owner': 1, '_columns': 1, '_rawLoclist': []}

Since the reason is likely changes of pyflakes output from above debug message, I checked it.
pyflakes==2.2.0
$ pyflakes main.py
main.py:1:1 'numpy as np' imported but unused

pyflakes==1.6.0
$ pyflakes main.py
main.py:1: 'numpy as np' imported but unused

the version of Vim you are using
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Mar 18 2020 18:29:15)

the version of syntastic you are using
Syntastic version: 3.10.0-7 (Vim 800, Linux, GUI)
Info for filetype: python
Global mode: active
Filetype python is active
The current file will be checked automatically
Available checkers: flake8 pep8 pycodestyle pyflakes python
Currently enabled checkers: pyflakes pycodestyle

Please try b3e2d847.

Works fine! thank you.


Remove dependency on django.contrib.sites

What was wrong
django-plans assumes that django.contrib.sites is installed. The documentation at https://django-plans.readthedocs.io/en/latest/integration.html doesn't mention anything about having it in INSTALLED_APPS
What this changes
When the Site model can't be imported, the item description will not contain the current site's name.
How this fixes it.
Since django.contrib.sites is only used for populating the item description with the site name, falling back to not have the site name in the description mkaes django.contrib.sites an optional requirement.
Alternatively, we could fix the issue just on the documentation side by adding 'django.contrib.sites' to https://github.com/cypreess/django-plans/blob/master/docs/source/integration.rst

@fcurella it makes sense to make the Site optional and not have it mandatory, especially since it's only using it for populating the name.
I'll merge this into master.
Thanks you for finding this out and making the patch.


[Feature Request] Please return to the old v3.18 GUI - the new 3.19+ GUI is confusing

The new GUI introduced in version 3.19 is confusing - I can't tell which DInput source is outputting to which XOutput source. Before, in v3.18, I could easily tell: This controller is currently being emulated, there's the emulation On/Off switch. Now, it's not so obvious, as there are XOutput controllers listed as "Emulated" even if my DInput controller is disconnected.
Any chance you could revert to the old simple UI?
Many thanks!

The new UI was needed to have the ability to combine multiple input devices to a single output. E.g. now you can create a controller even if you do not have enough axes or buttons, you can fill the desired inputs from other devices.
So currently the input devices (with the input settings), and the emulated devices (with the mappings) must be in a separate section. You can still rename the emulated devices, so you will know which is using which device.
So this is not a single input to single output connection anymore. I cannot see any good way of displaying the information better, but I am open for design ideas. I am planning to make the UI most comfortable, because I feel that the UI is just too much, and got complicated.


[OpenEBS-Install]: Openebs install job need to be run in sequence

This Issue is for:

Executing openebs-install jobs in sequence

Details:
Sequence of jobs:
1  - ` openebs-install-check`
2  -  `openebs-control-plane-test`
3  -  ` openebs-data-plane-test`
4 -   `openens-reinstallation`
5 -  `openebs-resource-limit`

https://github.com/mayadata-io/oep-e2e-gcp/pull/83


CM-286 Informacion de enfermedad no es populada durante busquedas de …

…bovinos

Refer to this link for build results (access rights to CI server needed):
http://utils.brazilsouth.cloudapp.azure.com:8080/job/ci/job/continuous-integration-bovine/61/

Refer to this link for build results (access rights to CI server needed):
http://utils.brazilsouth.cloudapp.azure.com:8080/job/ci/job/continuous-integration-bovine/62/


Issue with plan if not capacity reservation provided

Description
Running terraform plan cased the issue
If your request is for a new feature, please use the Feature request template.

[x] ✋ I have searched the open/closed issues and my issue is not listed.

⚠ Note
Before you submit an issue, please perform the following first:

Remove the local .terraform directory (! ONLY if state is stored remotely, which hopefully you are following that best practice!): rm -rf .terraform/
Re-initialize the project root to pull down modules: terraform init
Re-attempt your terraform plan or apply and check if the issue still persists

Done! But not helps.
Versions

Module version: 4.1.3
Terraform version: 1.2.6
Provider version(s): 4.23.0

Reproduction Code [Required]
module "ec2_data" {
  # https://registry.terraform.io/modules/terraform-aws-modules/ec2-instance/aws/latest
  source  = "terraform-aws-modules/ec2-instance/aws"
  version = "~> 4.0"
  # insert the 10 required variables here
  create = true

  name                        = "${local.pre-fix}"
  ami                         = data.aws_ssm_parameter.ubuntu_latest_ami_arm64.value
  instance_type               = "t4g.micro"
  cpu_credits                 = "unlimited"
  subnet_id                   = module.vpc.public_subnets[0]
  vpc_security_group_ids      = [aws_security_group.ssh.id]
  associate_public_ip_address = true
  ebs_optimized               = true
  iam_instance_profile        = aws_iam_instance_profile.ec2.name
  key_name                    = aws_key_pair.ec2.key_name
  user_data                   = null
  disable_api_termination     = true
  monitoring                  = false
  root_block_device = [
    {
      volume_type = "gp2"
      volume_size = 50
    }
  ]

}

Steps to reproduce the behavior:
terraform init
terraform plan
Expected behavior
plan output
Actual behavior
 Error: Invalid combination of arguments
│
│   with module.ec2_data.aws_instance.this[0],
│   on .terraform/modules/ec2_data/main.tf line 7, in resource "aws_instance" "this":
│    7: resource "aws_instance" "this" {
│
│ "capacity_reservation_specification.0.capacity_reservation_target": one of
│ `capacity_reservation_specification.0.capacity_reservation_preference,capacity_reservation_specification.0.capacity_reservation_target`
│ must be specified

Terminal Output Screenshot(s)

Additional context
Downgrade to 4.1.2 fixes the issue

I was able to reproduce.
In fact the default value of the capacity_reservation_specification should remain null instead of {}
@bryantbiggs : I can revert this back to what it was originally

let me take a deeper look locally

ran into the same error today and confirmed @savroman's note with downgrading to 4.1.2 worked for me as well.

looks like the module testing should be improved.

looks like the module testing should be improved.

sponsorships are always welcomed so we can dedicate more time

This issue has been resolved in version 4.1.4 :tada:


A few misc items fixed

ohhh come on! There is nothing like "too many!! :laughing:
Great cleanup, thanks!!


Potential flakiness with () as map value type

Tried making a pseudo-hashset in the runtime with
map T::AccountId => ()

The basic usage was inserting a new account ID into the map and using contains_key to check membership. It passed unit tests ok and sending extrinsics worked well. Sometimes the membership check would just fail, even though it was correct in previous blocks, this would create forks and panics with state must match that calculated in the test network.
In the end I changed map AccountId => bool and it appears to work correctly now.
Maybe this is a well known or expected footgun. Could prevent () as map value if so, otherwise it seems like it might be a sneaky 🐛

Yeah it is not really supported https://github.com/paritytech/subport/issues/7

🙏 ok seems we can close this. Some macro lint may help here for future users


Need a Test plan

We need a test plan for this project. This can be used in part 3.
If anyone has anything to add to the test plan , please add it.

I will go ahead and create an initial test plan


“Dictionary source” model

We need a model that stores bibliographic information about a dictionary source. Its primary key will be the source abbreviation (e.g., CW, MD). It will contain the following fields (like a bibliographic reference!):

[x] Abbreviation (primary key)
[x] Title
[x] Authors
[x] Publisher
[x] City
[x] Year
[x] Editor

The Definition models must have a many-to-many relationship to dictionary sources. These are citations.
For the first iteration, there is no need to automatically import the bibliographic information from the XML; all that's needed is the ability to modify the fields in the admin interface.
Addresses user story:

As an Administrator, I want to integrate words from multiple dictionary sources.

Here are the first two:
Abbreviation (primary key): CW
Title: Cree : nêhiyawêwin : itwêwina / Cree : Words
Author(s): NA
Editor/Compiler: Arok Wolvengrey
Publisher: Canadian Plains Research Center
City: Regina, Saskatchewan
Year: 2001
URL: NA
Non-academic citation: nêhiyawêwin : itwêwina / Cree: Words. Compiled by Arok Wolvengrey. Regina: Canadian Plains Research Center, 2001.
Academic citation: Wolvengrey, Arok. (2001). nêhiyawêwin : itwêwina / Cree: Words.  Regina: Canadian Plains Research Center.
Abbreviation (primary key): MD
Title: Maskwacîs Dictionary of Cree Words / Nehiyaw Pîkiskweninisa, Maskwachees Cultural College, Maskwacîs, 2009.
Author(s): NA
Editor/Compiler: Maskwachees Cultural College
Publisher: NA
City: Maskwacîs, Alberta
Year: 2009
URL: NA
Non-academic citation: Maskwacîs Dictionary of Cree Words / Nehiyaw Pîkiskweninisa, Maskwachees Cultural College, Maskwacîs, 2009.
Academic citation: Maskwachees Cultural College. (2009). Maskwacîs Dictionary of Cree Words / Nehiyaw Pîkiskweninisa, Maskwacîs, 2009.

This is done! Antti, if you login to your admin account, should should see the two DictionarySources with the correct information.


Data Tile Source Error

I am trying to copy the Data Tile example from openlayers examples in my Angular 13 project. But I am getting an error for source as DataTile.
When hovered over the source, it shows following msg.
"
Property 'getContextOptions' is missing in type 'DataTileSource' but required in type 'TileSource'.ts(2741)
Source.d.ts(38, 5): 'getContextOptions' is declared here.
BaseTile.d.ts(21, 5): The expected type comes from property 'source' which is declared here on type 'Options'
"
https://openlayers.org/en/latest/examples/data-tiles.html

Issue resolved. I was importing the wrong module for TileLayer
Correct Module: import TileLayer from 'ol/layer/WebGLTile';


Bug in Solve_ivp with BDF and Radau solvers and numpy arrays

It seems Scipy integration module solve_ivp when used with BDF and Radau solvers appends rounded numbers to the array. This will then cause an error of inconsistent dimensional further on in my code.
I wrote a simple example of a system with (2,1)-dimensional state (i.e. a second order system), which prints the state y as it runs in the function f(t,y), at one point, it appends the rounded number and gave me
[[ 13.67999979  13.67999999]
 [-15.99       -15.98999976]]

The second column is unnecessary and these added numbers will cause error in my original code.
Reproducing code example:
from numpy import array, matmul
from scipy.integrate import solve_ivp
from numpy import size
def sys(t,x,A,xeq,matmul,size):
    if size(x)==4:
        print(x)
    xdot=matmul(A,x-xeq)
    return xdot

T=10
A=array([[-1,2],[-2,-6]])
x0=array([13.67999999,-15.989999999])
xeq=array([[0.99999999],[2.999999999]])
fun=lambda t, x: sys(t,x,A,xeq,matmul,size)
sol=solve_ivp(fun,[0,T],x0,method='BDF',vectorized=True)

The output is:
[[ 13.67999979  13.67999999]
 [-15.99       -15.98999976]]

Scipy/Numpy/Python version information:
1.1.0 1.14.4 sys.version_info(major=3, minor=6, micro=5, releaselevel='final', serial=0)

@sajad-872797 does't look like any of A, x0, xeq are added to:
...
[[ 13.67999979  13.67999999]
 [-15.99       -15.98999976]]

In [11]: x0
Out[11]: array([ 13.67999999, -15.99      ])

In [12]: A
Out[12]:
array([[-1,  2],
       [-2, -6]])

In [13]: xeq
Out[13]:
array([[0.99999999],
       [3.        ]])

x0 is an array with shape (2,), so x in fun will (normally) have shape (2,) also. xeq has shape (2,1), so x - xeq will will have shape (2, 2) (because of numpy's broadcasting).  Is that what you intended?
I suspect the case where x.size is 4 has something to do with your use of the option vectorized=True.  The docstring says vectorized means "Whether fun is implemented in a vectorized fashion. Default is False.", but I'm not really sure what that means.  I haven't dug into the code, but that might mean that the solver calls your function with an array of x vectors.   This option needs better user documentation.
Try solving the problem using vectorized=False in solve_ivp, taking into account my comment above about the shape of x - xeq.

@rgommers @WarrenWeckesser
Please note that vectorized=True is mean to able us to do matrix multiplication and write the system as natural as possible. Also, x0 has to be an array of shape (2,), and x0 is not going to be directly inserted into f(t,y). The latter gets a (2,1) and yields a (2,1) array. In addition, if you just change BDF to any of RK45 or RK23 the code works just fine.

I think this works the way you expect.  I made the changes I suggested above, and also removed some arguments from sys and tweaked the whitespace (sorry, can't help it).
from numpy import array, matmul
from scipy.integrate import solve_ivp
from numpy import size

def sys(t, x, A, xeq):
    if size(x)==4:
        print(t)
        print(x)
    xdot = matmul(A, x - xeq)
    return xdot

T = 10
A = array([[-1, 2],
           [-2, -6]])
x0 = array([13.67999999, -15.989999999])
xeq = array([0.99999999, 2.999999999])
fun = lambda t, x: sys(t, x, A, xeq)
sol = solve_ivp(fun, [0, T], x0, method='BDF')

Apparently vectorized=True can result in the x argument being shape (2, 2).  I assume this is meant to be interpreted as multiple x vectors.  When vectorized=False, the argument x is always just a single vector.

@WarrenWeckesser
For this simple example, it does not matter matmul(A,x-xeq) is a row vector or a column vector. But I can imagine for an involved example, we need to reshape x for calculations. Anyway, this is perhaps a minor bug which can be avoided just by not using vectorized when BDF and Radau solvers are being selected.

I'm not convinced it is a bug at all.  It might just be a misunderstanding of the vectorized argument (not surprising, given that it isn't really explained in the docstring), combined with attempting to treat arrays with shape (2,) and (2,1) as being the same.  solve_ivp expects your state vector to have shape (2,), not (2, 1).   As I noted above, your xeq, has shape (2, 1), so the subtraction x - xeq results in shape (2, 2) (i.e. it is no longer something you would consider a "vector").  Then matmul(A, x - xeq) has shape (2, 2), which I think is not what you want.

It is bug. Forget about xeq. Run the code with matmul(A,x) you will get the same result. As I said, the solver append the rounded state to the state and adds to its dimension.

As I said, the solver appends the rounded state to the state and adds to its dimension.

And as I said, that appears to be a consequence of using vectorized=True; it is passing two state vectors at which you are to evaluate the system.
@nmayorov, could you describe (in a little more detail than what is in the current docstring) how the vectorized option is supposed to be handled by the user's code?  (Even better, could you update the docstring in a pull request, perhaps expanding on how and why to use vectorized in the Notes section?)

Hey guys, sorry for the confusion.
What vectorized means is explained for fun:

Right-hand side of the system. The calling signature is fun(t, y).
Here t is a scalar, and there are two options for the ndarray y:
It can either have shape (n,); then fun must return array_like with
shape (n,). Alternatively it can have shape (n, k); then fun
must return an array_like with shape (n, k), i.e. each column
corresponds to a single column in y. The choice between the two
options is determined by vectorized argument (see below). The
vectorized implementation allows a faster approximation of the Jacobian
by finite differences (required for stiff solvers).

(Not sure where did ; come from).
So yes, vectorized means that that you need to compute multiple f for multiple y (useful for faster finite difference).
I guess we should add reference to fun in vectorized description. Will it be sufficient?

Thanks!  Sorry I didn't notice that explanation under fun.
@sajad-872797, that explains it.  When vectorized is True, your function might be passed an array of vectors.

I believe this bug is bigger than appending rounded numbers to the array. I'm working with a very large system of ODEs (1000+ equations), and have found that when I use the vectorized=True option for solve_ivp the solver will attempt to pass a (n,n) matrix into the right-hand-side function, where n is the number of variables (n=1180 for the ODE system I'm working with).
I also agree that the documentation for the "vectorized" option is entirely unclear.

Why should there be a difference in the way BDF and Radau implemented with that of RK45 and RK23. Users should be able to easily change the method to observe speed, convergence, and accuracy of each method. Nonetheless, if there is difference, it should be explicitly stated. I will try to design a simple system of ODEs for which switching from RK45 to BDF yields an error of inconsistent dimension.

Any progress on this?

@TejasAvinashShetty and @sajad-872797 : The difference is that RK23 and RK45 are explicit methods, there never arises the need to evaluate the ODE function at multiple state vectors at the same time point. All that vectorized does is change the type of the state from (n,) to (n,1), that is, from array to proper column vector.
On the other hand, BDF and Radau are implicit methods that use a Newton-like iteration to solve the resulting non-linear system. From time to time this method needs to compute or refresh the Jacobian, if that is done via divided differences then the ODE function needs to be evaluated at n slight variations of the current state, which can be done "in parallel".
If this is properly implemented, one call with all of these vectors at once and using the numpy methods for matrix multiplications and simultaneous column operations can be substantially faster than a loop over all these state vectors separately. It might be questionable if this gives much of a speed-up in total when taken together with all the other steps without update of the Jacobian.

It seems that this was always a documentation issue. Please see if gh-17836 would have prevented the misunderstanding.
A related feature request is for solve_ivp to allow multidimensional x0 (e.g. shape (2, 1) in this case). I wrote about how that feature can be accomplished in gh-13348. It really is closely related even though the use case there appears different. (The user wants to "vectorize"  in the sense of solving several problems in parallel.) I'll open a feature request for that and probably submit a PR, since I've had that need before and apparently others have, too.


Microsoft.Insights/diagnosticSettings not deployed against existing App GW resource

Bicep version
0.19.5
Describe the bug
When instrumenting for Microsoft.Insights/diagnosticSettings with the scope of existing app gw resource, no diagnostic settings get deployed
To Reproduce
var appGWDeployMode  = 'existing'

resource applicationGateWayExisting 'Microsoft.Network/applicationGateways@2023-02-01' existing = if (appGWDeployMode == 'existing') {
  name: appGWName
}
resource appGWNewDiagnosticExistingSettings 'Microsoft.Insights/diagnosticSettings@2021-05-01-preview'= if (appGWDeployMode == 'existing')  {
  name: '${logAnalyticsWorkspaceName}-DiagnosticSetting'
  scope: applicationGateWayExisting
  properties: {
    workspaceId: logAnalyticsWorkspaceId
    logs: [
      {
        categoryGroup: 'allLogs'
        enabled: true
      }
    ]
    metrics: [
      {
        category: 'AllMetrics'
        enabled: true
      }
    ]
  }
}

Additional context
this works as expected if targeting non -existing bicep resource
we have to use the existing resource construct due to https://github.com/Azure/bicep/issues/2316
we have pre-existing App GW instances and wanted to add diagnostic settings hence this existing resource construct

@VaclavK please could you open a support case for this?


Invalidate Configuration Cache when Isolated Projects option changes

Fixes #26049
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

Superseded by #26204


import error

When I try importing the flask_awscognito library in my init.py module I get the below error. Any ideas what I am doing wrong?
from flask_awscognito import AWSCognitoAuthentication

def create_app():
    app = Flask(__name__)
    app.config['SECRET_KEY'] = 'xxxxx'
    app.config['SQLALCHEMY_DATABASE_URI'] = f'sqlite:///{DB_NAME}'
    db.init_app(app)

    from .views import views

    #cognito auth
    app.config['AWS_DEFAULT_REGION'] = 'us-east-1'
    app.config['AWS_COGNITO_DOMAIN'] = 'cognitodomain.com'
    app.config['AWS_COGNITO_USER_POOL_ID'] = 'us-east-1_XXXXXX'
    app.config['AWS_COGNITO_USER_POOL_CLIENT_ID'] = 'YYYYYYYY'
    app.config['AWS_COGNITO_USER_POOL_CLIENT_SECRET'] = 'ZZZZZZZZZ'
    app.config['AWS_COGNITO_REDIRECT_URL'] = 'http://localhost:5000/aws_cognito_redirect'
    aws_auth = AWSCognitoAuthentication(app)

Error: While importing 'main', an ImportError was raised:

Traceback (most recent call last):
  File "/home/pbarone/docker/myflaskapp/.venv/lib/python3.10/site-packages/flask/cli.py", line 219, in locate_app
    __import__(module_name)
  File "/home/pbarone/docker/myflaskapp/main.py", line 1, in <module>
    from website import create_app
  File "/home/pbarone/docker/myflaskapp/website/__init__.py", line 5, in <module>
    from flask_awscognito import AWSCognitoAuthentication
  File "/home/pbarone/docker/myflaskapp/.venv/lib/python3.10/site-packages/flask_awscognito/__init__.py", line 1, in <module>
    from flask_awscognito.plugin import AWSCognitoAuthentication
  File "/home/pbarone/docker/myflaskapp/.venv/lib/python3.10/site-packages/flask_awscognito/plugin.py", line 3, in <module>
    from flask import _app_ctx_stack, abort, request, make_response, jsonify, g
ImportError: cannot import name '_app_ctx_stack' from 'flask' (/home/pbarone/docker/myflaskapp/.venv/lib/python3.10/site-packages/flask/__init__.py)

The code is now around four years old, and it's not something I've looked into in a while. My guess would be in that time it looks like these libraries have changed, and the example code probably needs a little massaging to work in 2024.


Update urls.py.

django.conf.urls.patterns() is deprecated and will be removed in Django 1.10

@TetraEtc Thanks, good catch :)


[SPARK-20606][ML] ML 2.2 QA: Remove deprecated methods for ML

What changes were proposed in this pull request?
Remove ML methods we deprecated in 2.1.
How was this patch tested?
Existing tests.

Test build #76479 has started for PR 17867 at commit 4922c03.

Test build #76479 has finished for PR 17867 at commit 4922c03.

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

Merged build finished. Test PASSed.

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

Hm, isn't it sudden to remove deprecated methods after 1 minor release?
Also why does this not remove the methods from subclasses? if they're still needed by implementations, then does it make sense to deprecate/remove them?

@srowen These methods were deprecated when 2.1 releasing, and we usually remove methods deprecated in last feature/minor release (when 2.1 releasing, we remove methods deprecated in 2.0, see #15913). We only remove the setter methods from the trait, which cause them be remove from the model classes who extend them. Since it does not make sense to have it in the model classes, and actually we never use them and mark them deprecated for one minor release cycle. You can refer the discussion here . Thanks.

@srowen Any more comments?

I do see methods in the code that are deprecated yet retained for a few minor releases -- generally I'd expect we only actually remove them in a major release because it is a breaking change for any caller. I guess that's my primary question: is this binary/source compatible?

Yeah, that's because we didn't always do that in time, especially for spark.mllib packages which has been in maintenance mode. But for spark.ml package, I think we always keep to remove methods deprecated in last feature/minor release. And we have already warned This method is deprecated and will be removed in 2.2.0 in API documents, it may confuse users if we don't take action follow the API annotation. I'm prefer to merge this as the old convention, and start the new convention in the following release which we should add annotation like deprecated and will be removed in 3.0.0. What do you think about it? Thanks.

I'll merge this to catch 2.2, and update user guide to reflect this change as usual, since this is one of the ML QA tasks which blocks the 2.2 release. If there are more comments, I can address them in follow-up work. Thanks.

I actually think @srowen has a good point that we should maintain more stable minor releases.  I'd be in support of reverting this patch and changing the deprecation comments to say the items will be removed in "3.0.0" instead of 2.x minor versions.  Is that OK with you @yanboliang ?
Note: I also noticed that MLWriter.context does not appear as deprecated in the API docs because the deprecated implementation is overridden with a non-deprecated method.  (That was my bad for committing that PR.)

I'm OK to revert this change and change the deprecation comments to say the items will be removed in 3.0.0 instead of 2.x. Will do them in two follow-up PRs. Thanks.

I sent #17946 to change the deprecation comments to say the items will be removed in 3.0.0 instead of 2.x, please feel free to review and comments. @srowen @jkbradley


n edit dim should default to 3 in 3D view

I've noticed that I often forget to change n edit dims to 3 when I switch to 3D view, resulting in painting some thin 2D slices in that view. I find 2D painting from 3D quite unintuitive (?), so I want to discuss the idea of automatically switching n_edit_dims to 3 when switching to a 3D canvas.
@psobolewskiPhD @kephale @kevinyamauchi any thoughts? I feel like you've done more 3D editing than most. 😊

I find it hard to paint in 3D, so not much to on.
I think in general yes, seems good?

Like in a empty labels over a 3D image i can't paint at all.
But if I open Balls 3D sample which has 3D labels, I can paint with nedit dim 3. Which looks good:

But in reality:

So if y'all have advice, I'm all ears.
That said, the eraser in 3D is better, but always seems hyper aggressive, where I scar/scratch objects that are behind what I am trying to erase.

(1) indeed, erasing can be destructive if you don't have a clear line of sight both in front and behind the target. Maybe we can add a guard that during drag only erases the first label it touches?
(2) "printing" in 3D is just a toy, for sure. But a workflow I use relatively often is separating with the eraser then filling. The (current default) 2D fill gives you a very underwhelming single plane fill. When you then fix it to 3D you then have to fill two segments (above and below that one plane).

@jni so lets say toggle to 3d sets neditdim to 3. what about when toggle back? set to 2?
But what if it was already 3? I actually use neditdim 3 in 2D viewer more often than in 3d, so do we need to keep the state of the property to set it back or not?

Good Q! 😓
Here's a follow-up: do you think 2D editing in 3D view is useful ever? If not, we could have the "desired" n-edit dim be whatever, and the effective n-edit-dim be max(n_edit_dim, viewer.ndim.ndisplay)...


Documentation for upgrading influxdb from 1.2.2 to 1.6

URL for relevant page?
N/A
What products and version are you using?
Influxdb 1.2.2
Where did you look before opening the issue?
https://docs.influxdata.com/influxdb/v1.6/administration/upgrading/
Referred above documentation for upgrade instructions. I could not find steps for upgrading from 1.2.2 to 1.6 version. Please suggest.

@nsphaniraj The instructions are listed on that page, but there are some typos and information that need to be updated. The instructions provided in this section cover the upgrade process you're looking for:
https://docs.influxdata.com/influxdb/v1.6/administration/upgrading/#upgrading-influxdb-1-0-1-4-to-1-5-x
I've noted the changes that need to be made to this page and am closing this issue for now.


Use proper intra doc links for macro generated code

doc like https://github.com/paritytech/substrate/blob/62f011ca8d0491f6be77b3dbe0074eb060245d08/frame/support/procedural/src/construct_runtime/expand/origin.rs#L160 doesn't link to the item.
We should make a correct item link.

I'm currently learning Rust and would like to contribute, would I be able to help with this issue?

Is this issue done? @thiolliere can I pick the issue up?

anybody can submit a PR which solve this issue, they can also ping my name on the PR top message.
On master I still see those which could have a intra doc link:
frame/support/procedural/src/construct_runtime/expand/origin.rs:185:			/// Create with system none origin and `frame-system::Config::BaseCallFilter`.
frame/support/procedural/src/construct_runtime/expand/origin.rs:189:			/// Create with system root origin and `frame-system::Config::BaseCallFilter`.
frame/support/procedural/src/construct_runtime/expand/origin.rs:193:			/// Create with system signed origin and `frame-system::Config::BaseCallFilter`.
frame/support/procedural/src/construct_runtime/expand/origin.rs:219:			/// Convert to runtime origin, using as filter: `frame-system::Config::BaseCallFilter`.
frame/support/procedural/src/construct_runtime/expand/origin.rs:251:			/// `frame-system::Config::BaseCallFilter`.
frame/support/procedural/src/construct_runtime/expand/origin.rs:314:			/// Convert to runtime origin using `frame-system::Config::BaseCallFilter`.
frame/support/procedural/src/construct_runtime/expand/origin.rs:78:		/// Origin is always created with the base filter configured in `frame_system::Config::BaseCallFilter`.

can you check the PR i created @thiolliere

@thiolliere i have updated the PR i opened. could you take a look now.

Fixed by #10271.


Need a mechanism to auto-publish this code to Google Apps Script

https://developers.google.com/apps-script/add-ons/how-tos/publish-add-on-overview

https://tidisventures.com/blog/a-complete-guide-to-publishing-a-google-workspace-add-on-google-app-script


Course Connection: no way to discern a Faculty CC record from a Student CC record

Users may create a Student or Faculty Course Connection

Having created the Course Connection record, there is no way to tell a Student CC record from a Faculty CC record without tracking down the status of the Contact. The nature of CC record Student or Faculty should be visible on the CC record.

Totally agree, but since the record types are unmanaged we cannot package the layout change. We will need to tell the users to add it to the layout and to the related lists in the installation instructions. I believe Paul already has a note for this regarding the docs. Does that sound right, @paulmalenke?

The column is there in the layout's related list in the Trial. There's nothing for us to do in the package, since we cannot make layout changes related to record types that are unmanaged.


Change the defauilt connect_ports docs to reflect the code

[approve ci docs]

Going to need @zwoop  on this -
checking for sphinx version >= 1.7.5...
Found Sphinx version (1, 7, 0, 'final', 0)
sphinx version is older than 1.7.5

[approve ci docs]


The initialize method for Application initializer 'register-google' should take only one argument

it's happening if:
DEBUG: -------------------------------
vendor.js?v=1.4.38:1505 DEBUG: Ember      : 2.2.0
vendor.js?v=1.4.38:1505 DEBUG: Ember Data : 2.2.0
vendor.js?v=1.4.38:1505 DEBUG: jQuery     : 2.1.3
vendor.js?v=1.4.38:1505 DEBUG: -------------------------------
vendor.js?v=1.4.38:1246 DEPRECATION: The `initialize` method for Application initializer 'register-google' should take only one argument - `App`, an instance of an `Application`. [deprecation id: ember-application.app-initializer-initialize-arguments] See http://emberjs.com/deprecations/v2.x/#toc_initializer-arity for more details.
        at HANDLERS.(anonymous function)
...

how can I resolve that?

dear, any progress here? I don't like yellow messages I can't resolve and its eating up my console space :)

@patroqueeet I'll fix this today.

that would be really great!

@patroqueeet this should be fixed on the last version.


Add RN support

@arahansen you had a project going on with this right?

Yes, prototype is working on this branch: https://github.com/gymnastjs/gymnast/tree/ahansen/native?files=1

Closing since we aren't actively working on it. We can reopen if needed


Missing edge categories?

@kevinschaper reports that kg-phenio may be missing edge categories.

[x] Check if edge categories are present in the KGX output.
[ ] If not, investigate the transform and the PHENIO build.
[ ] If so, ensure they are consistent with expectations (e.g., in expected column, etc)

KG-Phenio has edge properties, they're just quite minimal. This is the header:
id	subject	predicate	object	category	relation	knowledge_source

Not much going on there!
For comparison, here's the Monarch graph heading:
id	original_subject	predicate	original_object	category	aggregator_knowledge_source	primary_knowledge_source	publications	qualifiers	provided_by	has_evidence	stage_qualifier	relation	knowledge_source	negated	frequency_qualifier	onset_qualifier	sex_qualifier	evidence	subject	object

Not all properties will be necessary for PHENIO, but the knowledge sources can certainly be expanded.

The merged Upheno mapping table also needs a knowledge_source added, but that can just be added at transform time as a KGX argument.

Also want:

synonyms and xrefs (see https://github.com/monarch-initiative/monarch-ingest/issues/409 and #79)
obsoletions (#90)
other provenance details contained in PHENIO but not included in the KG (PMIDs? versions?)

I am not sure we have an implemented strategy for populating edge categories when going from owl->kgx
This could be done in kgx by inference,
  gene to phenotypic feature association:
    is_a: association
    exact_mappings:
      - WBVocab:Gene-Phenotype-Association
    defining_slots:
      - subject
      - object
    mixins:
      - entity to phenotypic feature association mixin
      - gene to entity association mixin
    slot_usage:
      subject:
        range: gene or gene product
        description: "gene in which variation is correlated with the phenotypic feature"
        examples:
          - value: HGNC:2197
            description: "COL1A1 (Human)"
      object:
        range: phenotypic feature

however I would do this with linkml:classification_rules now


[Select] opens while scrolling

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

Current Behavior 😯
When I start scrolling on a SelectField it immediately opens the Select-Options.
Expected Behavior 🤔
Start Scrolling on a SelectField should not open the Select-Options.
Steps to Reproduce 🕹
Steps:

Place some <SelectField>s
Mouse Down on <SelectField>
Start Scrolling

Context 🔦
This is due to this change: #17883
SelectInput should open onClick instead of onMouseDown
Especially annoying for users on smaller Touch-Screen-Devices with many SelectFields on a larger Page, because they can not really scroll without opening a SelectField
Your Environment 🌎

Tech
Version

Material-UI
v4.9.5

React
v16.13.0

I have to examine how <select> behaves on touch devices. But on desktop devices SelectInput absolutely should open on mousedown because that's how <select> works.

I can't reproduce the issue.

You're absolutely right, it works correctly for desktop and mobile. It was  a problem with my custom device, that has a Touch-Interface, but handles Events in a desktop way (especially mousedown). My solution was using a Controlled Open Select.

@sebastian-ludwig Awesome, we are happy to hear it :)


resolve paths

Should resolve correctly relative paths in the composed files.
Rules

@import,
anything containing url keyword (@font-face, background and etc).

Related to https://github.com/sullenor/postcss-modules-resolve-imports/issues/13

solved in https://github.com/sullenor/postcss-modules-resolve-imports/commit/6f6fa14d0ad3d77fa991eb04b49b966698e3b764


Application CRUD commands

New commands implemented:

[x] application list
[x] application create
[x] application show
[x] appliation apply
[x] application delete
[x] application suspend
[x] application resume

Fixes #45
Missing integration tests

ready for review @miguelsorianod

changes applied


Add nodejs, npm, yarn to web container

The Problem/Issue/Bug:
People want node, npm, and yarn for their web container.
How this PR Solves The Problem:
Add it, from https://github.com/drud/docker.nginx-php-fpm-local/pull/68
Manual Testing Instructions:
Use this (or just set your config.yaml to use webimage: drud/nginx-php-fpm-local: 20180416_add_node
ddev ssh in there and do something useful.
Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:

I checked this in two installations - everything works fine


Ability for translator to pick up functions not nested inside classes.

Title. I use a few global common functions that are selectively imported into my components. I would like some of these functions to include translations. It currently isn't possible because the node scan only picks up on classes, then scans and sees if the constructor contains the I18n polyfill name. We can do the same thing for functions by scanning their parameters.

@ocombe do you know if another release will be out for angular 6 with this included, or will the next release be angular 7? Thanks!!!


[FEAT] Develop API Endpoints for Notification Settings (retrieval) - Backend

Description
Develop server-side logic to retrieve current notification settings.
Acceptance Criteria

The endpoint should be accessible at  /api/v1/notification-settings.
The endpoint should accept HTTP POST requests.
Requests to the endpoint must include a valid authentication token in the Authorization header. Authorization: Bearer <token>
Given a POST request with user data (userId & token), the system should validate and sanitize the incoming data.

Purpose
Provide a backend service that allows users to retrieve their notification preferences securely.
Requirements

[ ] Develop server-side logic to retrieve current notification preferences.
[ ] Ensure user exists and is authorized for notification settings they want to retrieve.
[ ] Retrieve and return user notification settings when user exists and is authorized.
[ ] Return appropriate error codes and responses in error scenarios

Expected Outcome

Allow retrieving notification settings with appropriate validation and security measures.

Status Codes

200: User settings data found
400: Bad Request
401: Unauthorized
404: User not found
500: A server error occurred

Endpoint
[POST] /api/v1/notification-settings
Request:
Headers:
{
  "Content-Type": "application/json",
  "Authorization": "Bearer <JWT_TOKEN>"
}

Body:
{
  "userId": "String"
}

Responses:
Successful response
{
  "message": "string",
  "success": true,
  "status_code": 200,
  "data": {
    // JSON object containing user notification settings
  }
}

## Example User Settings
{
  "email": false,
  "push-notification": true
}

Error response
{
  "message": "string",
  "success": false,
  "status_code": int
}

Testing
Write unit tests that cover

[ ] Verifying user authentication and authorization to retrieve notification settings
[ ] Appropriate response and status code is returned for various scenarios
[ ] User notification settings data is fetched and returned correctly

How are you using post for getting?

@markessien I corrected it to use GET now.


add density as an argument to the histogram command

Describe your changes
Add density as an argument to the histogram command
Checklist before requesting a review

[x] Performed a self-review of my code
[x] Formatted my code with pkgmt format
[x] Added tests (when necessary).
[x] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://jupysql--597.org.readthedocs.build/en/597/

closing due to inactivity


How to edit options? Where's manual?

Hi. I want to turn off autoprefixer.
{
    "hayaku_CSS_prefixes_disable": true
}

Where and how to edit that? Why not to write this in documentation? You that Sublime is confusing, right?

My bad, found

Why is there three settings — to let you make overrides in ST-way. Sublime Text have different scopes for preferences: User's, Syntax Specific and Project, when you declare any settings in the Preferences, all those settings are merged together. Although, each one specific setting is completely overriden by the setting with the same name in the next scope, so there is no simple way to merge things using one name. That's why Hayaku provides you with three settings: you can use hayaku_user_dict in your User/Preferences.sublime-settings, then add some other things using hayaku_syntax_dict in your Syntax Specific settings, like in user/Stylus.sublime-settings, and, finally, make overrides for your specific project in your *.sublime-project files.


Please add Kanata as input source

Pretty much the title; fwiw, I fully switched to Kanata because:

Same feature-set as QMK/ZMK, also with fine-tuned tap-hold behaviors
Runs on OS-level, so as well on your notebook keyboard
One file runs on all OSes (Win, Mac, Linux)
Extremely elegant, terse, compact, lisp-like config lang
Fastest iteration thanks to hot-reload your config; updating Katana is one git pull && cargo build
Actively maintained

It would be great if Kanata configs could be parsed too, so we could automate the process of keymap drawings just based on the config.

While I know of Kanata I haven't used it myself, I'll have a look. I'd like to support it but it mainly depends on how hard the parsing will end up being.

After an initial glance, here is a question (for you or anyone else that would like to visualize a Kanata map): What physical layout should be used?
Kanata lets you easily map just a subset of the keys, and it is not associated with a specific keyboard shape. My initial thought would be to always use a ANSI 104 keyboard as the physical layout, since that could be considered the "generic" keyboard and we know where each keycode goes, at least assuming US keyboard layout. In this case, we could show the full keyboard with the src keys replaced with the mapped ones, or, we could leave the non-mapped spots empty.
I can imagine people would still want to select their own physical layout. But without knowing their firmware mapping, we wouldn't know which key positions are overriden by the Kanata keymap. So I can't see how that would work without additional input data.

Thanks for looking into it!

What physical layout should be used?

I can talk only for my case but it's a typical 60% US ANSI or US mapped ISO because this is also what you find also on notebooks.
Reason is that I don't looe any muscle memory when switching to a notebook. I don't have perfect key placements like on a Corne or thumb keys but in terms of finger movement and overall feel I come very close with my mapping (it's basically a 40% on a 60% with tons of magic). And again, without efficiency losses when switching back and forth a notebook and any external, fancy mechboard.

I can imagine people would still want to select their own physical layout.

Yes, even when using it with such thing as a Corne, it would offer faster iteration than both QMK and ZMK and maybe also better control of tap and hold than QMK. And much terser confgs.

So I can't see how that would work without additional input data.

What about letting the user import KLE data and only for the physical matters, so positions and og keycodes.
Not sure if this all fits anymore into the scope of your tool which I just found yesterday but Kanata is really onto something. I use one file for all my devices with both macOS and Windows pulled from a central storage, it's a dream. And again, these iteration times are necessary, you code, feel that something is wrong, switch to your kanata file, change and code on. This is a matter of a few seconds and also for complex remappings.
Having now a tool that could create wonderful pics on save (or on git push haha) would be sick and I see a future where everyone has their layout published with auto-generated layout maps from your tool.

I can talk only for my case but it's a typical 60% US ANSI or US mapped ISO because this is also what you find also on notebooks.

I imagine so, the problem is it doesn't cover all possible src keys, e.g. if people use f-keys in their defsrc. Maybe I'll do a staged approach, where it defaults to a 60% and picks larger layouts (65%, TKL, 104) if it encounters a key not covered by the smaller layout.

What about letting the user import KLE data and only for the physical matters, so positions and og keycodes. This would need to be done only once since this doesn't change once a user settled with one physical layout (for a while at least haha).

That would be possible in theory, but I'd like to not deviate from the current physical layout paradigm. But that doesn't contain the OG keycodes info, of course. Given that I imagine laptop keyboard use case will be dominant, it might not be worth investing much time into custom physical layouts, at least initially. Feedback is welcome on that.
The good news is that the keymap seems relatively easy to parse using the existing pyparsing dependency, so that's great.

Related discussion for anyone else who checks this issue: https://github.com/jtroo/kanata/discussions/1003

fyi, I answered in the other thread...


[Bug]: graph does not update on tab-switch

Description
I'm trying to switch between the hour_to_show setting in the graph card by tabs.
The following config renders 24h in every tab, instead of 1/24/168 as set in the tab
The entities are identical in each tab and work as expected, but are cleared here for easier reading
Environment
- OS: kubernetes on ubuntu
- Browser: Homeassistant app on android
- Home Assistant Version: 2024.12.5 / Frontend 20241127.8
- Tabbed Card Version: 0.3.2

Reproducible configuration
type: custom:tabbed-card
options: {}
tabs:
  - card:
      type: history-graph
      entities:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
    hours_to_show: 1
    attributes:
      label: HOUR
  - card:
      type: history-graph
      entities:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
    hours_to_show: 24
    attributes:
      label: DAY
  - card:
      type: history-graph
      entities:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
        - entity:
          name:
    hours_to_show: 168
    attributes:
      label: WEEK

Additional context
No response

Wrong user code, not a bug.
hours_to_show is not indented.

Thanks, works!  Happy Holidays <3


Markdown linting and EOLs consistency

Target components
devutils, docs
Why & who
Internal/external devs, ReMark and the documentation website which has some trouble rendering different ways of splitting the lines when parsing our .md sources.
Feature or enhancement

There's no New Line consistency enforced. (\n mixed with \r\n)
Spacing is inconsistent. (some files ending with 0, 1 or 2 new lines; trailing spaces on non-empty lines; useless empty lines)
Everything relies on human opinions, place where we need a tool to automate the prettying and enforce a standard through linting.

Solution & alternative

All IDEs we use should strip trailing spaces and end the file with only one mandatory NL. (same standard for Python code) — easily configurable in IDE’s settings
A Markdown prettier and linter should be automated in our Invoke command stack. (extension)
Git attributes configuration to normalize the line endings, so we also stay consistent with what we end-up in the repo vs. what everyone has locally.

Slack public thread -- Linear
Slack private thread

Linking here some relevant work from Kari:

The docs breaking symbols removal: https://github.com/robocorp/robocorp/compare/master...doc-fixes
lazydocs improvements: https://github.com/robocorp/lazydocs/pull/2


minor changes and next version of DriverData, v1.1, submission item for hackathon

Why is the change being made?

Adding additional functionality to DriverData

What has changed to address the problem?

Finished work on the recursive search function, added it as version1.1
Added option 8 and functioning logic to search the data structure for a value

Other concerns

Tests with unit test are overly-complex for my code and none are working yet. I might need to refactor to make it testable.

Related sprint issue, TT, and/or CM

Hackathon

Related design document link:
Please select a mode of operation:
 1. Enter 1 for data viewing by key.
 2. Enter 2 to view available keys.
 3. Enter 3 to view a sample calculation.
 4. Enter 4 to view all available data in readable format.
 5. Enter 5 to view total combined miles driven for both drivers in all data.
 6. Enter 6 to view a range of n to k sorted example miles data.
 7. Enter 7 view total combined time driven.
 8. Enter 8 to search the data for a value.

clean. makes me miss python.
if i really had to nit, i'd say maybe some enums (does python have enums?) might be better than ints for your operationMode() conditionals, but that's mostly me looking for something to comment. i didn't run over the logic with a fine comb, but it reads clearly and seems good to me. if you'd like to go over tests some time, i'm due for some collab work and this seems very interesting.
p.s. i definitely appreciate the way you wrote your 'reasoning' notes. that's an interesting practice and seems great.

I've been thinking about a better way to run the operation mode, especially as more are added. I'll have to look into enum. As for testing, I have some work to do there so any help would be welcomed!


Fix tests

The tests are broken for one reason or another. This really needs to be fixed so bugs can be caught.

Fixed in c482b30!