Simplified pip env setup

This PR simplifies the project setup and updates the requirements file to ensure compatibility and ease of installation. The following changes have been made:
Updated requirements.txt:

Specified minimum versions for all dependencies to ensure compatibility.
Fixed the Cartopy version to resolve the libgeos issue when installing via pip.
Included torch-geometric directly in the requirements file for straightforward installation.
Tested the setup on both CPU and GPU environments to ensure proper functionality.

Updated README.md:

Revised the installation instructions to reflect the simplified setup process.
Provided clear steps for setting up the project environment using the updated requirements.txt.

This should simplify the installation of the environment. The installation is now fully flexible without pinned deps. For papers and such, pinned envs can still be exported of course.

Better solution proposed here: https://docs.google.com/document/d/1csgMjyuE1Up2m262e-N7qMPmNJ-zHLguGyrgvnjTuBE/edit
handle package dependencies with pdm rather than pip
what: put all dependencies in pyproject.toml with pdm and sync pyproject.toml to requirements.txt for pip-based installs
why: handling dependencies by manually keeping requirements.txt updated is tedious and errorprone task, requirements.txt don't fully describe a package (version, description) and are therefore no longer the recommended way of defining python packages.

@leifdenby I added you as a reviewer here, so that you can take what is useful from this PR but then please implement the more robust variant without requirements.txt file mentioned above. You can either use this branch or supercede this PR after your work is complete. I will give you access to the MeteoSwiss fork, since that is where this PR-branch lives.

I agree that moving dependencies from requirements.txt into pyproject.toml is good. However, I do not understand why one could not just install these using pip? I am not familiar with pdm, but is my understanding correct that this would introduce an additional tool that people have to use for installing requirements? It seems unnecessary to me to introduce additional tools, especially ones that ML practitioners are unlikely to be familiar with.
I agree that requirements handling can be somewhat tedious, but I don't think the project is at a stage that this is a problem yet.

I agree that moving dependencies from requirements.txt into pyproject.toml is good. However, I do not understand why one could not just install these using pip? I am not familiar with pdm, but is my understanding correct that this would introduce an additional tool that people have to use for installing requirements? It seems unnecessary to me to introduce additional tools, especially ones that ML practitioners are unlikely to be familiar with.

Yes, I agree adding another tool isn't something we should just do for the sake of it, and I think we can actually make it optional. By this I mean that the package dependencies will be defined in pyproject.toml (rather than requirements.txt) and this could in theory still be updated by hand, and we can then sync this automatically to a requirements.txt-file if needed (this is done by a pre-commit hook). Newer versions of pip can install directly from pyproject.toml, and so using pdm to manage and install dependencies would be optional.
For someone not using pdm the process of setting up a new development environment would be:

Clone repo git clone http://github.com/<username>/neural-lam and change directory cd neural-lam
Optionally create your virtualenv with your favourite tool, maybe mkvirtualenv ... and then activate ...
Install neural-lam editable with pip so that changes you make are reflected in the site-packages version (actually this is simply a link) pip install -e .
Edit and run neural-lam: python -m neural_lam.train_model --config-file config.yaml
Manually install a dependency with pip and edit pyproject.toml by hand, i.e. python -m pip install ... and edit the pyproject.toml file
Stage, commit and push!

I like using a package manager like pdm (other options are pipenv, poetry, uv) because it makes the development process easier by 1) automatically updating package versions in pyproject.toml to ensure versions work together when I add a new package, 2) allowing me to "dry run" any dependency modification without clobbering my current env (this has hurt me quite a few times with pip) and 3) handling creating and activating virtuals envs
So with pdm the development process would be:

Clone repo git clone http://github.com/<username>/neural-lam and change directory cd neural-lam
Create virtual env pdm venv create and either activating it pdm venv activate ... or making it the default when using pdm run python ... with pdm use ...
Install package pdm install
Add dependency pdm add ... (this is where you can do  pdm add --dry-run ... to see what would change before you install a package), or remove one with pdm remove .... You can also add dev dependencies separately (which won't be included in wheels) with pdm add --dev ... or dependency groups with pdm add --group visualisation matplotlib for example (if we didn't want visualisation tools installed by default)
Edit and run neural-lam: python -m neural_lam.train_model --config-file config.yaml (if using activated virtualenv) or pdm run python -m neural_lam.train_model --config-file config.yaml
Stage, commit and push!

Thanks for clarifying and outlining that Leif. I think that all sounds good, and the actual change is just to move dependencies to pyproject.toml. Then users can use whatever tool they want to handle their environment. We could keep sync a requirements.txt as well, but I would not mind just requiring people to have a new enough version of pip to install directly from pyproject.toml.
Will see what I end up doing myself going forward. The pdm workflow seems convenient as well.

Remember to update the changelog before you merge @sadamov! I think I would call this a new (very useful!) feature. A few things also change here (where constants are stored and thus how they are accessed in the code)

Can we close this if it is fully superseded by #37 ?

Superceded by #37


Mandation information missing

See e.g. DIDS data set - imaging activity group. Mandation information missing (many data sets) M/R/O

request status update on this one - not sure why it is in September release - must be fixed for first go-live

This is the text on the data set that I will patch in the September release.

Patched in the September release, needs checking on the September beta version.

Fixed


fix(es/minifier): Do not add vars if eval exists

Description:
And drop inlined paramters from IIFE inliner.
Related issue:

Closes #8886

Actually name mangler cannot work properly in presence of eval, because it cannot modify/add/remove a name even theoretically

I see. But there's still lots of changes in absence of eval, why?

@Austaras Nice catch! Thank you. I was confused about the changes.

Wait, I see the problem now.

This code maybe not of best quality, but I hope you get what I mean.

Thank you so much 👍


Intellisense autosuggest not working properly

I have updated to the newest verison 1.43.1 , the feature of auto-completion of javascript is not working properly it's always on relaoding , it just works on the first time for the first 2 seconds and doesn't work anymore et keep reloading.

I have the same problem for typescript.

I've tried to test if it work with other version like 1.31 but still don't work properly

/needsMoreInfo

The Auto-Completion of Javascript stopped working , you can see here an example :

The Suggestions of variables , functions , ... doesn't appear anymore.
Thank you in advance

Does this reproduce with all extensions disabled?

I disabled all my extensions and now it works fine 💯  , thank you for you help !

If you figure out which extension caused this, please file a bug against them so they can fix the issue


add Admin Panel

admin panel is important for content management  ,
when user login and the user is admin ,
the user or admin in this case must have admin panel ,
admin panel for posting College information like
title .description , image ,
and admin panel for post other component's  information .

please assign this issue for me as GSSoC Contributor


T279547 Gutenburg custom format - form

Phabricator Ticket: https://phabricator.wikimedia.org/T279547
Previous POC PR: https://github.com/wikimedia/wikipediapreview-wordpress/pull/19
Notes to this branch,

Once the preview link is being selected from the Gutenberg toolbar, and it enables the Wikipedia preview link
temporarily button, Add and Remove, as the action button
temporarily solution for the visible state, for example forceFormVisibleValue ( will remove in the next stage)

We will finish the following issues when we are in the next stage - search and preview

when to show edit form? when to show preview form? what's the internal state to control it?
the search list will replace the temporarily Add button
the preview UI's Delete button will replace the Remove button

Please ignore the current CI error and sometimes js linter error due to the unknown package-lock json dependencies version reason.
Please test the branch carefully,
The latest change includes the file/folder structure that

the main written javascript code is under src folder, and npm run start to observe any changes in folder src
build serve as the compiled folder ( npm run start )
assets is the folder we store the final output assets when we publish

some weird behaviour I never notice before, the edit form doesn't appear at the wrong place

Hover the text
Then, Select the preview link

Try hover another text
Then, Select the preview link

some weird behaviour I never notice before, the edit form appear at the wrong place

I had noticed that too, in my case it was only after resizing the screen that would throw it out of place. I think the arrow (noArrow={ false }) is what makes it seem more out of place.

Please note that since i18n support is merged already. This PR should use the proper localization function instead of hardcoding English strings.

I think I was able to get rid of the forceFormVisibleValue hack and still have the form show and hide correctly. Please test carefully!

I think I was able to get rid of the forceFormVisibleValue hack and still have the form show and hide correctly. Please test carefully!

Amazing, and It works very well!

One option to feature toggle is to wrap the including of the gutenburg script with if ( $_GET[ 'wikipediapreview_gutenburg' ] ) in php


Check if the String is a Palindrome

Before jumping into Pr first comment for assign.
Problem statement : A palindrome is a word, phrase, number or other sequence of characters which reads the same backward or forward, such as madam or kayak.
Write a function that takes a string and determines whether it's a palindrome or not. The function should return a boolean (true or false value).
Probelm Link : https://edabit.com/challenge/FSeXe93hxXFyJQpzN
Choose one Cateorgy - Medium
question category : strings, validation

Before submitting a PR please Check some details.

[ ] fileName formet should be in camelcase
example- returnTheNextNumber.py
[ ]  Choose label for Problem , example- "good first issue", "easy"
[ ]  In file there should be problem statement and link to problem

/* Problem-Task : This program will add two numbers
 * Problem Link : https://edabit.com/challenge/SFzHtm63XT6EYNHWY
*/
int main() {
    int bla;
    return 0;
}

This repository is vaild for HacktoberFest2020
Read before making PR - How to contribute on Github
Good Luck,

please assign me

Hey @Nusratillo777 . Could you please assign me this?

Hey can you assign me this problem?

Hey!!
Could I be assigned a problem too???

Can you please assign this problem to me, I will surely meet the program requirements.

I'll be glad to work on this issue.

I'll be glad to work on this

Could you please assign me for this task ?


you cannot split an xarray.DataArray image into channels via add_image / channel_axis

🐛 Bug

When you have an image with channel data, it is possible to split the channels into separate image layers via the channel_axis argument. This works for Numpy but triggers a dimensionality error for xarray.DataArray.
Notably, the bug occurs with pyimagej multichannel images.
To Reproduce
   import napari
   import imagej
   ij = imagej.init('sc.fiji:fiji:2.1.1')
   dataset = ij.io().open("Image 27.czi")  # any image would do
   image = ij.py.from_java(dataset) # yields xarray.DataArray z, c, y, x
   viewer = napari.Viewer()
   viewer.add_image(image, channel_axis=1) # buggy line
   viewer.add_image(image.values, channel_axis=1) # DataArray converted to numpy, works

Error trace: https://pastebin.com/Kf5A5FAp
Environment

napari 0.4.8 installed via conda

Additional context
https://forum.image.sc/t/napari-should-i-reorder-the-dimensions-of-an-array-created-via-ij-py-from-java/53436

Ok, thanks good to know - we should really get better at testing xarray support!!

Fixed by https://github.com/napari/napari/pull/2780


eagerAlgorithm doesn't work with pathAndFetchById

If use eagerAlgorithm with patchAndFetchById:
const application = await Application.query()
        .patchAndFetchById(applicationId, {status: value})
        .eagerAlgorithm(Application.JoinEagerAlgorithm)
        .eager('participant')

then application doesn't contains eagers properties.
DB: Postgres

Yep, it's not supported to do that. It's just as readable (and only one line more) to split that into two queries. Your code would anyway execute a separate update and a select query.
await Application.query().findById(applicationId).patch({ status: value })
await Application.query().findById(applicationId).joinEager('participant')


Enforce that Shoot nginx-ingress addon and Seed VPA are not enabled for ManagedSeeds

How to categorize this PR?

/area quality
/kind enhancement
What this PR does / why we need it:
This PR enforces the following validations for ManagedSeeds:

The Shoot nginx-ingress addon to be not enabled for ManagedSeeds. The addon is deprecated and supported only for Shoot clusters with evaluation purpose. Instead the Gardener managed Seed ingress controller or the self-managed ingress controller should be used.
The Seed VPA to be not enabled for ManagedSeeds. https://github.com/gardener/gardener/issues/6305 outlines the motivation for it. Instead the Shoot VPA should be used.

Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/6305
Release note:

The Gardener API server now enforces the following configuration options for ManagedSeed resources:
1. The vertical pod autoscaler should be enabled from the Shoot specification.
2. The nginx-ingress addon should not be enabled for a Shoot referred by a ManagedSeed.

Before upgrading to this version of Gardener make sure that all ManagedSeeds and the Shoots they refer to conform the newly enforced configuration options.

/test pull-gardener-e2e-kind

/retest

And one question - does it make sense to also document in the release note the proper order in which to edit Shoots and ManagedSeeds that do not conform to the new requirements (if this order matters)? E.g. first the Shoot should be edited to disable nginx and enable Shoot VPA, then the ManagedSeed should be adapted to disable the Seed VPA.

I think the order should not matter. But good point, I will a sentence to the release note of type "Before upgrading to this version of Gardener make sure that all ManagedSeeds and the Shoots they refer to conform the newly enforced configuration options ."

/lgtm


minor cleanups to hide extra tree and command pallet items

Hiding Data source from Tree
Hiding Exclude item command from command pallet

Pull Request Test Coverage Report for Build 233506318

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

Totals

Change from base Build 233489466:
0.0%

Covered Lines:
18911

Relevant Lines:
44282

💛  - Coveralls


Remove functions from keywordCase docs

sql-formatter v15 included changes which excluded functions from keywordCase

https://github.com/sql-formatter-org/sql-formatter/releases/tag/v15.0.0

Thanks

Glad to help, thanks for the review and merge!


chore(deps): update common-dev-assets digest to 669c646

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
c3d897c -> 669c646

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

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


Axes

Intro
This is a meta-issue to collect thoughts about implementing axes.
Axes are important for downstream code like Fastplotlib. I think it makes sense to support the building blocks in pygfx itself, so that axes can be composed and configured with relative ease.
Things that make up axes
Strict:

Viewport (in 2D the axes is drawn over/around the scene).
Lines for the axes.
Ticks.
Tick labels.
Grid.

Somewhat broader:

Title, x-label, y-label.
Legend.
Colorbar.
UI elements.

Things to solve or account for

Tick spacing.
Log scale.
Polar scale.
Map projection.
Layout (i.e. positioning title and labels)
Twin axes (Left and right side with diff y-scales)

Links
Related issues and prs:

#492
#683
#582
#118

Techniques:

Nicolas' shaders for infinite grids: https://jcgt.org/published/0003/04/01/

Implementations:

Axis in visvis
Axis in vispy

Example from visvis (we can do better 😉):

Example from vispy:

I will update this text as I collect more info and converge on a plan.

Do you also intend to support 3D rendering of grids (in world space with a perspective camera)?
And rendering grids and axes on deformed surfaces?

We've done some brainstorming on legends and a very primitive legend currently exists in fpl (most experimenting with things, we're far from happy with it): https://github.com/fastplotlib/fastplotlib/issues/403
I can help out with legends later in the summer and contribute that to pygfx instead if that makes sense. Anyways I'm guessing the "Strict" things you pointed out would come first.

Do you also intend to support 3D rendering of grids (in world space with a perspective camera)?

Yes, definitely that. And axii too (works in visvis).

And rendering grids and axes on deformed surfaces?

Nicolas' paper on grids (same paper that we used for markers) shows grids for polar plots and map projections. Is that what you mean with deformed surfaces?

We've done some brainstorming on legends and a very primitive legend currently exists in fpl (mostly experimenting with things, we're far from happy with it): https://github.com/fastplotlib/fastplotlib/issues/403

👍 that will be helpful.

Log scale

Would this require a camera matrix with a log function along x, y, and/or z instead of a static projection matrix?

Yes, probably.

Like this: https://github.com/pygfx/pygfx/discussions/601

I also wanted to share this article with you since it's pretty amazing: https://bgolus.medium.com/the-best-darn-grid-shader-yet-727f9278b9d8#5ef5

Incidentally, I've been working on a simple selection helper (mimicking the implementation of the TransformGizmo) for a different project. It would need a bit of clean-up but I thought this might be useful in this context. Is there appetite for a PR?
https://github.com/pygfx/pygfx/assets/7161148/6bd2e33c-438c-4da5-be86-ed4a1a120ea9

Idea for later: grid that stays "fixed" but changes major and minor boxes w.r.t. the current camera? Kind of like matplotlib's 3D grid.

That's indeed the plan :)  next step, I think, is a mechanism+api to determine step sizes based on camera.


Applications page fix

Fix key error causing build to break

Closed this PR to make another PR branching from master.


Use JavaEnumField in the inspection result JSON or remove JavaEnumField completely

Need more investigation to see which is better:

Use JavaEnumField
Completely remove JavaEnumField from both of UI and backend

Both involves changing serialized inspection result contract, i.e. UI change must be involved at the same time.
Also see (#13):
https://github.com/atlasmap/atlasmap/issues/13#issuecomment-350062836
https://github.com/atlasmap/atlasmap/issues/13#issuecomment-350101751
https://github.com/atlasmap/atlasmap/issues/13#issuecomment-350104146

It turned out JavaEnumField represents an instance value of the enum, where the enum class is represented by the JavaField and it has a list of JavaEnumField to hold its values. #1594 would also follow the same manner.


Could not resolve all dependencies for configuration ':app:analyticsDebugRuntimeClasspath'.

It probably network issue. Retry may fix this or try using VPN.


have at least one script ready to run in scripts/.

It is turned out that that script might be subster

It's now done ✅ with 7fe697d at afccat.js


Screen cleared after only pressing return

In 0.5.0, the screen would clear only after actually running a command. If I just pressed return one or more times, lines would be added as they would be in a zsh without almostontop. This seemed the correct and unsurprising behavior.
In 0.6.0, pressing return clears the screen, but leaves two prompt lines on the screen. (Just press return once and you'll see -- let me know if you need a screen shot.)
It seems that the plugin should do one or the other -- either clear the screen after an empty command, leaving only one line with the current prompt, or allow multiple empty commands to stack up before clearing the screen for a real command.

Hi @jslabovitz , thanks for the report. Not sure I'm getting you. Could you please send screenshot (or asciinema cast)?

Yeah, something has changed here. I run this plugin along side auto-ls and now when I press enter, it's clearing the auto-ls; which wasn't normal

This was the old behaviour:
https://asciinema.org/a/zzNwMZgghJ5hPOrTmUcm0CPKN

This is the new behaviour:
https://asciinema.org/a/JRkX5V7A4skg5TtKpzG49mQF9

I don't mind if it clears the screen on the "enter" key, but I'd love for auto-ls to still function with this

Hi @rawkode , could please put auto-ls plugin load prior to almostontop and check again.

@Valiev Apologies for the delay, I took a lot of time off for the holidays.
I've put auto-ls before almostontop, but I'm still seeing the ls aspect cleared.
##
# Always Clear Screen & Run `ls`
##
AUTO_LS_COMMANDS=(exa git-status)

zplugin ice lucid pick"auto-ls.zsh"
zplugin load "desyncr/auto-ls"

auto-ls-exa () {
  exa
  [[ $AUTO_LS_NEWLINE != false ]] && echo ""
}

zplugin ice lucid pick"almostontop.plugin.zsh"
zplugin load "Valiev/almostontop"

This is my config

@rawkode @jslabovitz closing due to inactivity


(docs) Update or fix links, especially to plugins.

Many links point to the projects.puppetlabs.com wiki, instead of the
migrated equivalent on docs.puppetlabs.com. Point these links to the
best places.
Also, several links point to paths on marionette-collective.org. Point
these links to docs.puppetlabs.com/mcollective.

Rebased.


Docker image shows "Invalid host header"

Similar to issue #301. When I deploy my app in docker from local, I am able to see its up and running on localhost:. When I try to deploy it on kubernetes environment, it returns invalid host header.
I am using React>17 and nx for microfront end.
docker file is -
build environment
FROM node:14.18.1-buster-slim
ARG APPNAME
WORKDIR /home/${APPNAME}
COPY package.json package.json
RUN npm install --registry=
RUN npm i -g @nrwl/cli --registry=
COPY . ./
Building Specific app
RUN echo ${APPNAME}
RUN nx build ${APPNAME}
RUN groupadd -g 10000 10000 &&
useradd -u 10000 -g 10000 -m -d /home/${APPNAME} -s /bin/bash 10000 &&
mkdir -p /opt/dummy/logs &&
chown -R 10000:10000 /opt
EXPOSE 4200
CMD nx serve ${APPNAME} --host=<IP_ADDRESS> --disableHostCheck

I don't think this issue is still relevant.
If it is, please open a new issue.
The current docker image uses nginx so you can update the configuration as needed.


merge publish.md update

Docs Build status updates of commit 17402d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/publish.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Doc detail template: Author field needs to be conditional

Currently, even when field needs to be blank, the word "By" still shows up on the page preview

Part of this requires work from content. In the CMS, you'll have to click the trashcan so that there's no authors being added, right now a blank author is being added. The dash is a front-end problem and I'll still fix it.

Thanks! I'll go through and make those updates to the doc detail pages.
On Mon, Feb 1, 2016 at 2:15 PM, Kimberly Munoz<EMAIL_ADDRESS>wrote:

Part of this requires work from content. In the CMS, you'll have to click
the trashcan so that there's no authors being added, right now a blank
author is being added. The dash is a front-end problem and I'll still fix
it.
[image: image]
https://cloud.githubusercontent.com/assets/1860176/12728058/fee9d8be-c8ed-11e5-9ee5-167da90cfade.png
—
Reply to this email directly or view it on GitHub
https://github.com/cfpb/cfgov-refresh/issues/1391#issuecomment-178135994
.

Pushed up a fix to take care of a rogue dash that still appears!


399 | Fixed bug on barcode scanner crash

Fixes #399
Type
Bug fix
Checklist

[x] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[x] I have added any strings visible on UI components to the strings.xml file
[x] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

Pull Request Test Coverage Report for Build #206

0 of 0   changed or added relevant lines in 0 files are covered.
41 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.7%) to 85.616%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/org/smartregister/fhircore/viewmodel/PatientListViewModel.kt
1
97.37%

src/main/java/org/smartregister/fhircore/activity/QuestionnaireActivity.kt
2
70.0%

src/main/java/org/smartregister/fhircore/activity/PatientListActivity.kt
9
76.32%

src/main/java/org/smartregister/fhircore/fragment/PatientListFragment.kt
29
61.11%

Totals

Change from base Build #205:
-0.7%

Covered Lines:
1250

Relevant Lines:
1460

💛  - Coveralls


Add a namespace for AttributeValue xsi:type value

This fixes the issue when integrating with onelogin's python3-saml
client:
Element '{urn:oasis:names:tc:SAML:2.0:assertion}AttributeValue',
attribute '{http://www.w3.org/2001/XMLSchema-instance}type': The QName
value 'xs:string' has no corresponding namespace declaration in
scope.

Oops! This fixes the same issue as #336, but in a different way.

Either https://github.com/rohe/pysaml2/pull/336 or this pr should be merged, not both. both look good to me :+1:

I think it's cleaner to just conditionally add this attribute, as is done in #336.

This PR here sets self._extatt['xmlns:xs'] in addition to self.extension_attributes["xmlns:xs"]. In my tests (when I was working on #336) this was not required and it is too long ago for me to understand the difference. @ashimaathri are you able to explain, hopefully having this at the top of your head? :-)

@jgehrcke it looks like a fallback. The access that ashima is doing looks consistent with other code in here... https://github.com/rohe/pysaml2/blob/master/src/saml2/saml.py#L241
self._extatt gets set on the object init to be empty. self.extension_attributes gets set to this empty object. https://github.com/rohe/pysaml2/blob/master/src/saml2/saml.py#L140 (i am guessing this is for legacy use of this object)
self.extension_attributes is from SamlBase, the class this extends. https://github.com/rohe/pysaml2/blob/master/src/saml2/init.py#L302

@jgehrcke Thanks for you comment. I've updated my code to conditionally add this attribute. And it looks like @jaustinpage has answered your other questions. I'm ok with either PR getting in as long as the issue is fixed.


Error if host has more than 8 CPUs with virt guest

Describe the issue
I have a problem installing windows 11, when I try to start the VM it gives me an error  message (qemu-aarch64-softmmu: Number of SMP CPUs requested (16) exceeds max CPUs supported by machine 'mach-virt' (8))
Configuration

UTM Version: 2.2.4 (36)
OS Version: macOS Big Sur V. 11.6
Intel or Apple Silicon? 2,3 GHz 8-Core Intel Core i9

Debug log
Running:  -L /private/var/folders/2b/plq7c1w11tdgwzw6mv61pw1m0000gn/T/AppTranslocation/96037664-92D5-49D4-8042-296ED65940FD/d/UTM.app/Contents/Resources/qemu -S -qmp tcp:<IP_ADDRESS>:4000,server,nowait -nodefaults -vga none -spice "unix=on,addr=/Users/serajburshan/Library/Group Containers/WDNLXAD4W8.com.utmapp.UTM/14A2D25C-6951-4952-8F51-2A14DB87812C.spice,disable-ticketing=on,image-compression=off,playback-compression=off,streaming-video=off,gl=on" -device virtio-ramfb-gl -cpu cortex-a72 -smp cpus=16,sockets=1,cores=8,threads=2 -machine virt,highmem=off -accel tcg,tb-size=4096 -drive if=pflash,format=raw,unit=0,file=/private/var/folders/2b/plq7c1w11tdgwzw6mv61pw1m0000gn/T/AppTranslocation/96037664-92D5-49D4-8042-296ED65940FD/d/UTM.app/Contents/Resources/qemu/edk2-aarch64-code.fd,readonly=on -drive "if=pflash,format=raw,unit=1,file=/Users/serajburshan/Library/Containers/com.utmapp.UTM/Data/Documents/Windows 11.utm/Images/efi_vars.fd" -boot menu=on -m 16384 -device intel-hda -device hda-duplex -name "Windows 11" -device qemu-xhci,id=usb-bus -device usb-tablet,bus=usb-bus.0 -device usb-mouse,bus=usb-bus.0 -device usb-kbd,bus=usb-bus.0 -device ich9-usb-ehci1,id=usb-controller-0 -device ich9-usb-uhci1,masterbus=usb-controller-0.0,firstport=0,multifunction=on -device ich9-usb-uhci2,masterbus=usb-controller-0.0,firstport=2,multifunction=on -device ich9-usb-uhci3,masterbus=usb-controller-0.0,firstport=4,multifunction=on -chardev spicevmc,name=usbredir,id=usbredirchardev0 -device usb-redir,chardev=usbredirchardev0,id=usbredirdev0,bus=usb-controller-0.0 -chardev spicevmc,name=usbredir,id=usbredirchardev1 -device usb-redir,chardev=usbredirchardev1,id=usbredirdev1,bus=usb-controller-0.0 -chardev spicevmc,name=usbredir,id=usbredirchardev2 -device usb-redir,chardev=usbredirchardev2,id=usbredirdev2,bus=usb-controller-0.0 -device nvme,drive=drive0,serial=drive0,bootindex=0 -drive "if=none,media=disk,id=drive0,file=/Users/serajburshan/Library/Containers/com.utmapp.UTM/Data/Documents/Windows 11.utm/Images/Windows11_InsiderPreview_Client_ARM64_en-us__22454.VHDX,cache=writethrough" -device usb-storage,drive=drive1,removable=true,bootindex=1 -drive if=none,media=cdrom,id=drive1 -device virtio-net-pci,mac=7E:E2:B0:35:8F:BD,netdev=net0 -netdev vmnet-macos,mode=shared,id=net0 -device virtio-serial -device virtserialport,chardev=vdagent,name=com.redhat.spice.0 -chardev spicevmc,id=vdagent,debug=0,name=vdagent -device virtserialport,chardev=charchannel1,id=channel1,name=org.spice-space.webdav.0 -chardev spiceport,name=org.spice-space.webdav.0,id=charchannel1 -uuid 14A2D25C-6951-4952-8F51-2A14DB87812C -rtc base=localtime
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: Started vmnet interface with configuration:
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: MTU:              1500
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: Max packet size:  1514
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: MAC:              12:75:7f:40:ae:a3
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: DHCP IPv4 start:  <IP_ADDRESS>
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: DHCP IPv4 end:    <IP_ADDRESS>
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: IPv4 subnet mask: <IP_ADDRESS>
qemu-aarch64-softmmu: -netdev vmnet-macos,mode=shared,id=net0: info: UUID:             4CB43171-F1ED-4FF3-9826-B67E175D7E93
qemu-aarch64-softmmu: Number of SMP CPUs requested (16) exceeds max CPUs supported by machine 'mach-virt' (8)

What's happening here is a bug in the way UTM configures itself automatically: it takes the number of logical CPU cores your Mac has and tries to give that many to the VM, regardless if the VM system can take it.
To work around this issue, please go to the VM configuration → System → Advanced and type "4" in the CPU Cores field. Then enable "Force Multicore" to improve performance.

I will leave this Issue open and change the title to reflect the source of the problem.

Thinking of making two changes. Let me know if there’s any objections:

New installs and those who never touched the options will have the “Use performance cores by default” option set. This way people won’t have weird performance inbalances.
If the host cpu cores are > the max for virt, we set it to max (8). Ideally we get this info from QEMU but it doesn’t exist outside of the source code so we’ll have to manually code in a list of popular machines and their max cpu count.

Sounds reasonable to me.

@osy I've been looking into this, and it seems the 8 is only a hard limit with default QEMU settings (specifically, GICv2 - explainer see below). Here's the quote from the QEMU source that determines how many CPUs are supported for virt:
    /* The maximum number of CPUs depends on the GIC version, or on how
     * many redistributors we can fit into the memory map.
     */
    if (vms->gic_version == VIRT_GIC_VERSION_3) {
        virt_max_cpus =
            vms->memmap[VIRT_GIC_REDIST].size / GICV3_REDIST_SIZE;
        virt_max_cpus +=
            vms->memmap[VIRT_HIGH_GIC_REDIST2].size / GICV3_REDIST_SIZE;
    } else {
        virt_max_cpus = GIC_NCPU;
    }

    if (max_cpus > virt_max_cpus) {
        error_report("Number of SMP CPUs requested (%d) exceeds max CPUs "
                     "supported by machine 'mach-virt' (%d)",
                     max_cpus, virt_max_cpus);
        exit(1);
    }

QEMU Source qemu/hw/arm/virt.c lines 1956-1970

GIC is the Generic Interrupt Controller, available in QEMU in version 2 and 3. (QEMU gic-version parameter; defaults to 2).
GIC version 2 only supports max. of 8 CPUs:
/* Maximum number of possible CPU interfaces, determined by GIC architecture */
#define GIC_NCPU 8

QEMU source qemu/include/hw/intc/arm_gic_common.h lines 32-33

GIC version 3 has a dynamic CPU limit, see the first quote.
For testing, I added the ,gic_version=3 parameter to the QEMU Machine Properties in UTM 2.4.0 and started a virt machine I had previously created. It uses hvf, as I am testing on M1. The result was that the EFI BIOS seemed to hang at "Start boot option". It did not show the 3s timeout bar and pressing esc did not go to the EFI menu. Nothing appears in the Debug Log:
Running:  -L /Applications/UTM/UTM.app/Contents/Resources/qemu -S -qmp tcp:<IP_ADDRESS>:4000,server,nowait -nodefaults -vga none -spice "unix=on,addr=/….spice,disable-ticketing=on,image-compression=off,playback-compression=off,streaming-video=off,gl=on" -device virtio-ramfb-gl -cpu host -smp cpus=2,sockets=1,cores=2,threads=1 -machine virt,highmem=off,gic_version=3 -accel hvf -accel tcg,tb-size=512 -boot menu=on -m 2048 -name "Ubuntu 20.04 ARM 2.1.2" -device qemu-xhci,id=usb-bus -device usb-tablet,bus=usb-bus.0 -device usb-mouse,bus=usb-bus.0 -device usb-kbd,bus=usb-bus.0 -device ich9-usb-ehci1,id=usb-controller-0 -device ich9-usb-uhci1,masterbus=usb-controller-0.0,firstport=0,multifunction=on -device ich9-usb-uhci2,masterbus=usb-controller-0.0,firstport=2,multifunction=on -device ich9-usb-uhci3,masterbus=usb-controller-0.0,firstport=4,multifunction=on -chardev spicevmc,name=usbredir,id=usbredirchardev0 -device usb-redir,chardev=usbredirchardev0,id=usbredirdev0,bus=usb-controller-0.0 -chardev spicevmc,name=usbredir,id=usbredirchardev1 -device usb-redir,chardev=usbredirchardev1,id=usbredirdev1,bus=usb-controller-0.0 -chardev spicevmc,name=usbredir,id=usbredirchardev2 -device usb-redir,chardev=usbredirchardev2,id=usbredirdev2,bus=usb-controller-0.0 -device usb-storage,drive=drive0,removable=true,bootindex=0,bus=usb-bus.0 -drive if=none,media=cdrom,id=drive0 -device virtio-blk-pci,drive=drive1,bootindex=1 -drive "if=none,media=disk,id=drive1,file=/…/disk-0.qcow2,cache=writethrough" -drive "if=pflash,media=disk,id=drive2,file=/…/edk2-aarch64-code.fd,cache=writethrough" -drive "if=pflash,media=disk,id=drive3,file=/…/edk2-arm-vars.fd,cache=writethrough" -device virtio-net-pci,mac=…,netdev=net0 -netdev user,id=net0 -uuid … -rtc base=localtime
WARNING: Image format was not specified for '/Users/chris/Library/Containers/com.utmapp.UTM/Data/Documents/Ubuntu 20.04 ARM 2.1.2.utm/Images/edk2-aarch64-code.fd' and probing guessed raw.
         Automatically detecting the format is dangerous for raw images, write operations on block 0 will be restricted.
         Specify the 'raw' format explicitly to remove the restrictions.
WARNING: Image format was not specified for '/Users/chris/Library/Containers/com.utmapp.UTM/Data/Documents/Ubuntu 20.04 ARM 2.1.2.utm/Images/edk2-arm-vars.fd' and probing guessed raw.
         Automatically detecting the format is dangerous for raw images, write operations on block 0 will be restricted.
         Specify the 'raw' format explicitly to remove the restrictions.

Note the missing lines gl_version 30 - es profile enabled and WARNING: running without ARB/KHR robustness in place may crash that would usually appear. To verify that our build of QEMU supports GICv3, I enabled "force slower emulation" and started the VM again:
Running:  -L /Applications/UTM/UTM.app/Contents/Resources/qemu -S -qmp tcp:<IP_ADDRESS>:4000,server,nowait -nodefaults -vga none -spice "unix=on,addr=/….spice,disable-ticketing=on,image-compression=off,playback-compression=off,streaming-video=off,gl=on" -device virtio-ramfb-gl -cpu cortex-a57 -smp cpus=2,sockets=1,cores=2,threads=1 -machine virt,highmem=off,gic_version=3 -accel tcg,tb-size=512 -boot menu=on -m 2048 -name "Ubuntu 20.04 ARM 2.1.2" -device qemu-xhci,id=usb-bus -device usb-tablet,bus=usb-bus.0 -device usb-mouse,bus=usb-bus.0 -device usb-kbd,bus=usb-bus.0 -device ich9-usb-ehci1,id=usb-controller-0 -device ich9-usb-uhci1,masterbus=usb-controller-0.0,firstport=0,multifunction=on -device ich9-usb-uhci2,masterbus=usb-controller-0.0,firstport=2,multifunction=on -device ich9-usb-uhci3,masterbus=usb-controller-0.0,firstport=4,multifunction=on -chardev spicevmc,name=usbredir,id=usbredirchardev0 -device usb-redir,chardev=usbredirchardev0,id=usbredirdev0,bus=usb-controller-0.0 -chardev spicevmc,name=usbredir,id=usbredirchardev1 -device usb-redir,chardev=usbredirchardev1,id=usbredirdev1,bus=usb-controller-0.0 -chardev spicevmc,name=usbredir,id=usbredirchardev2 -device usb-redir,chardev=usbredirchardev2,id=usbredirdev2,bus=usb-controller-0.0 -device usb-storage,drive=drive0,removable=true,bootindex=0,bus=usb-bus.0 -drive if=none,media=cdrom,id=drive0 -device virtio-blk-pci,drive=drive1,bootindex=1 -drive "if=none,media=disk,id=drive1,file=/…/disk-0.qcow2,cache=writethrough" -drive "if=pflash,media=disk,id=drive2,file=/…/edk2-aarch64-code.fd,cache=writethrough" -drive "if=pflash,media=disk,id=drive3,file=/…/edk2-arm-vars.fd,cache=writethrough" -device virtio-net-pci,mac=…,netdev=net0 -netdev user,id=net0 -uuid … -rtc base=localtime
WARNING: Image format was not specified for '/…/edk2-aarch64-code.fd' and probing guessed raw.
         Automatically detecting the format is dangerous for raw images, write operations on block 0 will be restricted.
         Specify the 'raw' format explicitly to remove the restrictions.
WARNING: Image format was not specified for '/…/edk2-arm-vars.fd' and probing guessed raw.
         Automatically detecting the format is dangerous for raw images, write operations on block 0 will be restricted.
         Specify the 'raw' format explicitly to remove the restrictions.
gl_version 30 - es profile enabled
WARNING: running without ARB/KHR robustness in place may crash

This time, the VM started up normally, just slower due to emulation instead of hvf virtualization. I tried again with 12 CPU cores, to verify we can fix this issue if we get GICv3 working with hvf. It worked fine, the only change in the log is the smp cpus=12,sockets=1,cores=12,threads=1.

In conclusion, it seems that (at least on my M1) with hvf mode and the included EFI BIOS, you can't boot a VM using QEMU GICv3. When you disable HVF, it works.
Machine: MacBook Pro M1
OS: 12.1 beta
UTM: 2.4.0 from GitHub

@conath the first search for qemu gicv3 hvf gets this<EMAIL_ADDRESS>and if you look at the end of the first block of > 's you can see: - missing GICv3 support

Okay thanks for the find, I'll take out the 8 core on hvf code since that could change in the future with GICv3. But I'll keep in default to using pcores only which has the side effect of working on new M1 Macbooks which has 8 pcores.

@osy do you want to apply the patch linked to by ktprograms? Then we could set GICv3 as default. That would actually solve the underlying issue which will otherwise appear again when Apple Silicon with more than 8 cores is introduced in the future.

What do you mean? The patch says it's missing GICv3

No, I mean this one.

Since @agraf removed that patch in v7, I assume it's not ready.

@agraf

I'd suggest to leave it at GICv2 for now until we sort out GICv3 support upstream.

Has there been any movement on this, or is it a lost cause (use Apple Virtualization Framework instead)?
I looked at the recent QEMU changelog, and see GICv3 mentioned quite a bit, but don't understand enough to be able to tell if this is related to the CPU count issue or not.

The patch works, but upstream is unhappy with the way it integrates in the rest of the code base, so I removed it to get anything upstream at all. Given that there are no Macs out with more than 8 p cores, I'd suggest to leave it at GICv2 for now until we sort out GICv3 support upstream.

16 pcores has been here for a while :)

The QEMU issue to track GICv3 support which would enable -smp > 8 is this: https://gitlab.com/qemu-project/qemu/-/issues/743. I would appreciate Tested-by / Reviewed-by tags on the mailing list to push it forward :).

I'll open a new issue to track this so we don't forget.


cityandcountry

Create cityandcountryproperties

@gr00nd Good luck on your coding adventure, but this is definitely not the project to try and make your first contribution.

sorry that was ai generator mistake. I deleted it,


Adds an the option to have heard about underdogs from a mentor or mentee

Description
Adds Mentor or Mentor to the answers to the question 'How did you hear about underdog dev?' on both mentor and mentee intake forms. Commented the dialog.js file out, errors (not my code nor part of this ticket).
Video Link
[Loom Video](Add your loom video link here)
Trello Link
Paste your trello card link here, deleting this example
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Great work on the selections, the  "Do not wish to share" option is a good addition, giving a user-friendly experience.


Make it compatible with forms API (RC4)

make it compatible with tslint too? :)

Please merge it.

I would love to, but I need CI build fix

Sorry, I don't use TypeScript but I've commited some tslint fixes

That seems to work well. But 2 remarks:

When doing this.form.value, an array is set of both the id and the text. I think it should be an array of just id instead.
One slightly inconsistent point is when multiple is set to false, the value is set within an Array of 1 object. Logically, this should not be an array but a simple entry with the "id" as value.

Either way, this pull request makes this plugin useful, looking forward to seeing that merged :)

Hi :) Anything new on this one? :)

@otelnov please check this one

Any progress on this PR?

I've tested version from @chymz fork on RC4 and it looks like it's working good with new forms using FormBuilder :)

So does that means that you are about to merge it?  Maybe you could provide kind of a roadmap of when you will merge it.  Couple of weeks?  Couple of months?  Looks like many people are waiting for this new version of your module.  Good for you that so many peoples are using it. Thanks!

Update for rc6 comitted

Awesome, I will merge and publish it asap

Sorry for the inconvenience. Any news about this PR?

Thank you!
Merged as a part of another PR


Missing APM transactions since upgrading to HotChocolate 12.0.0-rc.3 and latest

I've noticed that since I've upgraded to latest 12 version of HotChocolate, specifically since RC 3 I get some differences in Elastic APM.
before RC 3:

after RC 3:

The data still seems to be corelated for Gateway and subgraph.
Not sure if this is an issue with ElasticApm.GraphQL.HotChocolate or HotChocolate introduced a breaking change.

Hi and thanks for reporting. Please try with 2.0.0-preview.1.

I think this is fixed, please create a new one if not.
2.0.0-preview.2 has also the 12 stable release from HC

2.0.0-preview.2 has also the 12 stable release from HC

@glucaci, can you check to publish status of the new version as it's still missed at nuget.org.
Thanks in advance

Done
https://www.nuget.org/packages/ElasticApm.GraphQL.HotChocolate/2.0.0-preview.2


Plugin System Proof of Concept

Hi,
I had created a proof of concept for the Plugin system. The sample plugin can be found at https://github.com/suhastech/bundler-sample-plugin
It is still early stage. Only subcommand module has been implemented.
Currently in the pipeline: Source and Lifecycle.

@suhastech the next step is probably to write a bunch of tests for this, as I don't see any at the moment?

@suhastech this needs a rebase.
I'd also like to see significantly more test coverage, particularly unit tests.

:umbrella: The latest upstream changes (presumably #4016) made this pull request unmergeable. Please resolve the merge conflicts.

:umbrella: The latest upstream changes (presumably #4077) made this pull request unmergeable. Please resolve the merge conflicts.

@all Is this still on the agenda?

@suhastech this is still on the agenda eventually, but we're focusing on launching 1.12 and the new index on the server for right now


Fix 64735 ServletContext.addJspFile() always fails with SecurityManager

Many thanks for this. I particularly like the way you implemented the unit test. I am currently leaning towards packaging the test slightly differently so I'll apply this manually rather than just clicking merge.

Thanks. Applied manually so I could make a few minor tweaks (mainly for consistency with existing tests). Also back-ported to 8.5.x and 7.0.x,


Fix typo in Venv warning message.

Simple fix of typo in warning message:
old: "Attemping"
new: "Attempting"

Thanks!


Replaced slang terms

There is no such thing as the "Exchange Online admin" role.  There is no place in these docs for writers' comfort-zone slang.  Readers came here for a reason, we shouldn't steer them wrong with terminology that is not going to be found anywhere within the products that they've come here to learn about.
Also, updated the recommendation to also assign the "Service Support Administrator" (not "Service admin") role so that it is clearly just that, a recommendation.  It's not mandatory to do that and it's subjective.

@JohanFreelancer9 I think I've done so correctly.  Here's hoping.  Just let me know if I need to redo or do anything more.  Thanks!

@efrene Please merge this PR. Thanks


linux-iot2050:fix: couldn't connect to phy

When booting up,it would show the error "couldn't connect to phy".
as it trigger the deferred probe mechanism,it would probe later
But there is a chance of error, causing the MAC to fail to link with
the phy normally.Resulting in the ethernet can not work
How to reproduce:
1.power on and get the ip then ping router
2.reboot
3.waiting bootup,and repeat 1~3
Repeat about 500 times,it would cause the error.
Compile the mdio into kernel to fix this issue.
Signed-off-by: chao zeng<EMAIL_ADDRESS>

LGTM at this moment.
Long term speaking, the defered probe should not trigger kernel oops even if the mdio is modularized.

This makes no sense to me yet: We didn't build that MDIO driver, but Ethernet generally still worked?! Also, this driver seems to be involved only in the CPSW interface, not the PRU one. Otherwise, the prueth driver would do a select TI_DAVINCI_MDIO.
I strongly suspect this is papering over the actual problem.

This makes no sense to me yet: We didn't build that MDIO driver, but Ethernet generally still worked?! Also, this driver seems to be involved only in the CPSW interface, not the PRU one. Otherwise, the prueth driver would do a select TI_DAVINCI_MDIO.
I strongly suspect this is papering over the actual problem.

looks actually Prueth is using this DAVINCI_MDIO: https://github.com/siemens/meta-iot2050/blob/65a3ba902b3ada8a34acba9331abf839d6073ca9/recipes-kernel/linux/files/patches-5.10/0085-net-ti-icssg-prueth-Add-ICSSG-ethernet-driver.patch#L119

OK, now it now starts to make sense. Hope TI is fixing this before upstreaming that code....

Yep, we will inform TI regarding this issue.


Voice Identification using Classification Algorithms

Formatting and structure checks

[X] Have you used the correct folder and file structure?
[X] Is your article properly formatted in Markdown?
[X] Have you used our preferred heading structure?
[X] Does your article follow any of the suggested structure formats?
[X] Is your article over 750 words?
[X] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.
[X] Does your article provide enough value and detail about your topic?
[X] Can your article be understood by beginners?
[X] Have you included a hero image that is 600x400 pixels and have the copyright to use it?

Grammar & plagiarism checks

[X] Have you spellchecked and grammar checked your article?
[X] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit.
[X] Have you checked your article for readability?
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[X] Are your code snippets properly formatted for syntax highlighting
[X] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[X] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

This closes #4077

@Bayler  I have updated the changes you requested.

@JOEOBY Welcome to our Eng-Ed Engineering Program. Please take note of all the commits I made in this article so that you don't repeat the same mistakes in your next article.

@JOEOBY
Thank you for your patience as our team has been working diligently to work through the recent increased volume of articles coming in.
We are going through are ready articles in the order they were received and working on improving our review time as well.
🚀

jumping on this one

@JOEOBY
Not sure if there has some files that have not been pushed?
I tried to check it out and it showed 8 files to push and 9 to pull via my vscode. ??
Not sure how or why?
I tried to check out another PR and had not such issue.
cc @Bayler any idea?

@JOEOBY
Not sure if there has some files that have not been pushed?
I tried to check it out and it showed 8 files to push and 9 to pull via my vscode. ??
Not sure how or why?
I tried to check out another PR and had not such issue.
cc @Bayler any idea?

@hectorkambow everything seems okay from my end. Please check to confirm whether you've pushed all files @JOEOBY

@hectorkambow  i have rechecked and all files are in line from my side.
Thank you.
cc @Bayler

jumping on this one

@section-engineering-education/content-moderator-team Can someone try to preview this when the they have a chance?
I could not get it to render?

@Bayler
I didn't see a plagiarism check on this PR - please lets be sure we are always performing a plagiarism check. I did the whole final review fixed more thing before I realized. Not a good use of our time to be doing reviews on PR that will be closed.
@JOEOBY
On that note - many sentences and paragraphs seem to be paraphrased from other sources.
We only accept unique and original content to be published.
Voice Identification using Classification Algorithms #4629.pdf


Constraint Propagation in CSPs

@sharanraja Deadline Wednesday morning after the lecture.

I have the code ready for AC3 and AC1 but I am unable to run the doc even after installing all the required modules from requirements.txt.

Added AC1 and cleaned up the Constraint propagation section


Assignment 2

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
Added code for assignment 2
What did you learn from the changes you have made?
learnt about the K nearest neighbours algorithm
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
there are other classifier methods
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

How were these changes tested?
Running the notebook.
A reference to a related issue in your repository (if applicable)
Checklist

[X ] I can confirm that my changes are working as intended

A2 marked as completed!


Workshop

[ ] Identify interns to do a workshop
[ ] prep files for workshop
[ ] Coordinate a time for the workshop
[ ] Run the workshop
[ ] Make notes after the workshop synthesizing the information

Reached out to Sam Terrell about getting interns to do a workshop for an hour or two


install-poetry breaks with Poetry 1.2.0

Poetry 1.2.0 changes a lot of things about how path and virtualenvs are handled, and it breaks install-poetry:
▶ Run snok/install-poetry@v1
  with:
    virtualenvs-create: true
    virtualenvs-in-project: true
    installer-parallel: true
    version: latest
    virtualenvs-path: {cache-dir}/virtualenvs
  env:
    DJANGO_ENV: LOCAL
    pythonLocation: /opt/hostedtoolcache/Python/3.6.15/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.6.15/x64/lib
Run $GITHUB_ACTION_PATH/main.sh
  $GITHUB_ACTION_PATH/main.sh
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    DJANGO_ENV: LOCAL
    pythonLocation: /opt/hostedtoolcache/Python/3.6.15/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.6.15/x64/lib
    VERSION: latest
    VIRTUALENVS_CREATE: true
    VIRTUALENVS_IN_PROJECT: true
    VIRTUALENVS_PATH: {cache-dir}/virtualenvs
    INSTALLER_PARALLEL: true
    INSTALLATION_ARGUMENTS:

Setting Poetry installation path as /home/runner/.local/

Installing Poetry 👷

Retrieving Poetry metadata

# Welcome to Poetry!

This will download and install the latest version of Poetry,
a dependency and package manager for Python.

It will add the `poetry` command to Poetry's bin directory, located at:

/home/runner/.local/bin

You can uninstall at any time by executing this script with the --uninstall option,
and these changes will be reverted.

Installing Poetry (1.[2](https://github.com/critizr/critizr-django-1.7/runs/8108219212?check_suite_focus=true#step:7:2).0)
Installing Poetry (1.2.0): Creating environment
Installing Poetry (1.2.0): Installing Poetry
Installing Poetry (1.2.0): Creating script
Installing Poetry (1.2.0): Done

Poetry (1.2.0) is installed now. Great!

You can test that everything is set up by executing:

`poetry --version`

/home/runner/work/_actions/snok/install-poetry/v1/main.sh: line [3](https://github.com/critizr/critizr-django-1.7/runs/8108219212?check_suite_focus=true#step:7:3)3: poetry: command not found
/home/runner/work/_actions/snok/install-poetry/v1/main.sh: line 3[4](https://github.com/critizr/critizr-django-1.7/runs/8108219212?check_suite_focus=true#step:7:4): poetry: command not found
/home/runner/work/_actions/snok/install-poetry/v1/main.sh: line 3[5](https://github.com/critizr/critizr-django-1.7/runs/8108219212?check_suite_focus=true#step:7:5): poetry: command not found
/home/runner/work/_actions/snok/install-poetry/v1/main.sh: line 3[7](https://github.com/critizr/critizr-django-1.7/runs/8108219212?check_suite_focus=true#step:7:7): poetry: command not found

Installation completed. Configuring settings 🛠

Done ✅

If you are creating a venv in your project, you can activate it by running 'source .venv/bin/activate'. If you're running this in an OS matrix, you can use 'source $VENV' instead, as an OS agnostic option

▶ Run poetry install --no-interaction --no-root
/home/runner/work/_temp/63f8c552-7bf4-4d02-bd7b-8b02d[9](https://github.com/critizr/critizr-django-1.7/runs/8109330298?check_suite_focus=true#step:9:10)c8b5c0.sh: line 1: poetry: command not found
Error: Process completed with exit code 127.

Would you mind sharing your workflow configuration (just the install-poetry step)?
I don't think it breaks the action. AFAICT lockfiles generated by < 1.2 are incompatible with 1.2 and vice versa, so if your lockfile was generated with 1.1.X you need to specify a 1.1.X version in your action run.
    - uses: snok/install-poetry@v1
      with:
        version: 1.1.14

Should work, while
    - uses: snok/install-poetry@v1

Will not.
Does that make sense, or do you think it's broken for 1.2 with 1.2 lockfiles?

Hello :)
I pasted the output of the action (and then you have the action's configuration as well). It's the poetry command itself that is not found. See the original message.

Ah yes. And I see you've specified latest as the version. Does it work if you change it to 1.1.14?

Yes

Ok great. If you want to use 1.2.0 in your github actions run you first need to upgrade Poetry to 1.2.0 locally, then run poetry update or poetry lock --no-update to make the file compatible with 1.2.0 in the workflow run. After doing that, upgrading to 1.2.0 in the action should work.

Hello. You got it. Sorry I should not have cut the Github Actions output: the error we get is poetry: command not found. You can see this error (somewhat hidden) in GHA output. I think Poetry 1.2.0 changed the location where poetry is installed.

I see. That's unfortunate if that's the case. I'm not able to look at this right now, but PRs are welcome 👍

Also seeing this issue 👍

https://github.com/snok/install-poetry/issues/89

I added two comments in the PR, but would be good to get your take too @deronnax. I tried updating two projects to Poetry version 1.2.0, and neither one had any problems running the action. Just to understand what is broken, what is different between these two workflows and yours?

https://github.com/snok/flake8-type-checking/runs/8122396279?check_suite_focus=true
https://github.com/snok/asgi-correlation-id/actions/runs/2966986335

@sondrelg no clue, our workflow is pretty standard, my bet is that it's all related to the dropping of Python 3.6 from Poetry 1.2.0. We will pin the Poetry version and wait until we move to the next python version to see if it reproduces, which I bet it won't.
In between, thank you very much for your time

To avoid the brownout, add the following to the poetry install step until you upgrade to 1.2.0
        env:
          GET_POETRY_IGNORE_DEPRECATION: 1

#90 Attempting a fix here, ~unsure how to test this in a Github Action~
edit:// this seems to fix it - there was a trailing / in the path, by accident. I have a separate issue now of Poetry 1.2.0 dropping support for Python 3.6, but that's unrelated smiley

https://endoflife.date/python
Projects are dropping python3.6 because it is no longer considered secure. Tools and libraries dropping support forces OS distributions to release newer, supported versions of python on their platform. This is a good thing, because it is intended to result in a more secure internet and better safety for user's machines.

I guess you gotta make sure to have
[build-system]
requires = ["poetry-core>=1.1.0"]

More on --> https://github.com/python-poetry/poetry/blob/master/pyproject.toml#L47

Same issue here, but only with macos-latest ❌
On the other hand no problem with windows-latest and ubuntu-latest ✅
Here is the link to the yml file: https://github.com/Musaefendic/icao-emissions/blob/96d41895570a1aefe21cab13f32f906f44d6f6ab/.github/workflows/tests.yml
2022-09-29T16:05:45.3245480Z ##[group]Run poetry install --no-interaction
2022-09-29T16:05:45.3245930Z poetry install --no-interaction
2022-09-29T16:05:45.3754580Z shell: /bin/bash --noprofile --norc -e -o pipefail {0}
2022-09-29T16:05:45.3754920Z env:
2022-09-29T16:05:45.3755490Z   pythonLocation: /Users/runner/hostedtoolcache/Python/3.9.10/x64
2022-09-29T16:05:45.3755920Z   PKG_CONFIG_PATH: /Users/runner/hostedtoolcache/Python/3.9.10/x64/lib/pkgconfig
2022-09-29T16:05:45.3756330Z   Python_ROOT_DIR: /Users/runner/hostedtoolcache/Python/3.9.10/x64
2022-09-29T16:05:45.3757220Z   Python2_ROOT_DIR: /Users/runner/hostedtoolcache/Python/3.9.10/x64
2022-09-29T16:05:45.3757620Z   Python3_ROOT_DIR: /Users/runner/hostedtoolcache/Python/3.9.10/x64
2022-09-29T16:05:45.3757940Z   VENV: .venv/bin/activate
2022-09-29T16:05:45.3758180Z ##[endgroup]
2022-09-29T16:05:45.4574200Z /Users/runner/work/_temp/7e911813-9a38-4bc5-8835-86f3f94d1643.sh: line 1: poetry: command not found
2022-09-29T16:05:45.4602330Z ##[error]Process completed with exit code 127.

did I set up my .yml wrong?

@Musaefendic is it your intention not using poetry v1.2.1 ? I can see this https://github.com/Musaefendic/icao-emissions/blob/96d41895570a1aefe21cab13f32f906f44d6f6ab/.github/workflows/tests.yml#L48
You can drop the version and the action will automatically setup the latest version

good catch! It is an error after several unsuccessful attempts.
I just removed the line version: 1.1.14 but the result is the same
✅ windows-latest and ubuntu-latest
❌ macos-latest
i got the same error:
Run poetry install --no-interaction
[...]
/Users/runner/work/_temp/6c346071-2b8c-41b1-b4f2-75377cb405b1.sh: line 1: poetry: command not found
Error: Process completed with exit code 127.

I wonder if your action is caching a really old version 1, and it's not using the newest version.
Maybe try uses<EMAIL_ADDRESS>You can also try activating it manually

@Musaefendic also, restore venv cache BEFORE installing poetry.

And if it still doesn't work I'd start with
      - run: |
          ls ./local/bin

And I'm most sure snok/install-poetry@lint could work, because he added the path to the final commands.

And if it still doesn't work I'd start with this after installing poetry
      - name: Check environment
        run: |
          echo $PATH
          echo $VENV
          ls ./local/bin

/Users/runner/.local//bin:/Users/runner/hostedtoolcache/Python/3.9.10/x64/bin:/Users/runner/hostedtoolcache/Python/3.9.10/x64:/usr/local/lib/ruby/gems/2.7.0/bin:/usr/local/opt/ruby@2.7/bin:/usr/local/opt/pipx_bin:/Users/runner/.cargo/bin:/usr/local/opt/curl/bin:/usr/local/bin:/usr/local/sbin:/Users/runner/bin:/Users/runner/.yarn/bin:/Users/runner/Library/Android/sdk/tools:/Users/runner/Library/Android/sdk/platform-tools:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/usr/bin:/bin:/usr/sbin:/sbin:/Users/runner/.dotnet/tools:/Users/runner/.ghcup/bin:/Users/runner/hostedtoolcache/stack/2.9.1/x64

.venv/bin/activate

ls: ./local/bin: No such file or directory
Error: Process completed with exit code 1.

@Musaefendic also, restore venv cache BEFORE installing poetry.

good intuition! if I delete Load cached poetry installation,  all workflows pass ✅
any idea how to purge the cache?

To clear the GH Actions cache, it's a new feature since June 22. See this SO post https://stackoverflow.com/a/64819132
Clearing the cache
curl \
    -H "Accept: application/vnd.github.v3+json" \
    -H "Authorization: token *********************" \
    https://api.github.com/repos/Musaefendic/icao-emissions/actions/caches
{
  "total_count": 0,
  "actions_caches": [

  ]
}

1st CI Run after clearing the cache.
Link to my tests.yml

✅ ubuntu-latest
✅ macos-latest
✅ windows-latest

cache after this CI run
curl \
    -H "Accept: application/vnd.github.v3+json" \
    -H "Authorization: token ****************" \
    https://api.github.com/repos/Musaefendic/icao-emissions/actions/caches
{
  "total_count": 5,
  "actions_caches": [
    {
      "id": 21,
      "ref": "refs/heads/master",
      "key": "venv-macOS-3.9.10-7f3e4de67ec2fafad1743c550f0d8aaf4038b5f0f03ecfe5275d285833296cbf",
      "version": "25e65b70d167147605b4332eb57b6d34df8e0303b88c4de252efb18cf3ed148a",
      "last_accessed_at": "2022-09-29T21:31:08.286666700Z",
      "created_at": "2022-09-29T21:31:08.286666700Z",
      "size_in_bytes": 71408887
    },
    {
      "id": 20,
      "ref": "refs/heads/master",
      "key": "poetry-0",
      "version": "d6b93696bc837862e84764c8431f7c282393336a702ec638c820588088bfd7a1",
      "last_accessed_at": "2022-09-29T21:30:09.980000000Z",
      "created_at": "2022-09-29T21:30:09.980000000Z",
      "size_in_bytes": 30
    },
    {
      "id": 19,
      "ref": "refs/heads/master",
      "key": "venv-Windows-3.9.10-14a2a15fefb761fe78d797839a035d718998d5ff92247101add1124f1e5af187",
      "version": "519c93c80eb6dc7871185acd9844f0af47c509ad7b0399902b51e465753dba51",
      "last_accessed_at": "2022-09-29T21:29:51.203333300Z",
      "created_at": "2022-09-29T21:29:51.203333300Z",
      "size_in_bytes": 75356150
    },
    {
      "id": 18,
      "ref": "refs/heads/master",
      "key": "poetry-0",
      "version": "f4b5713a4657ae30a7c2804b2ad5dbc790b3736f71a74a904bcab1f478b0ed29",
      "last_accessed_at": "2022-09-29T21:27:04.993333300Z",
      "created_at": "2022-09-29T21:27:04.993333300Z",
      "size_in_bytes": 25628232
    },
    {
      "id": 17,
      "ref": "refs/heads/master",
      "key": "venv-Linux-3.9.10-7f3e4de67ec2fafad1743c550f0d8aaf4038b5f0f03ecfe5275d285833296cbf",
      "version": "25e65b70d167147605b4332eb57b6d34df8e0303b88c4de252efb18cf3ed148a",
      "last_accessed_at": "2022-09-29T21:27:00.680000000Z",
      "created_at": "2022-09-29T21:27:00.680000000Z",
      "size_in_bytes": 82591676
    }
  ]
}

2nd CI RUN
same tests.yml

✅ ubuntu-latest
❌  macos-latest
✅ windows-latest

I'm on holidays almost without internet for a few days, but should have time to review a PR if anyone has time to investigate :+1: Otherwise, I'll be happy to look at this on Tuesday 🙂

And if it still doesn't work I'd start with this after installing poetry
      - name: Check environment
        run: |
          echo $PATH
          echo $VENV
          ls ./local/bin

/Users/runner/.local//bin:/Users/runner/hostedtoolcache/Python/3.9.10/x64/bin:/Users/runner/hostedtoolcache/Python/3.9.10/x64:/usr/local/lib/ruby/gems/2.7.0/bin:/usr/local/opt/ruby@2.7/bin:/usr/local/opt/pipx_bin:/Users/runner/.cargo/bin:/usr/local/opt/curl/bin:/usr/local/bin:/usr/local/sbin:/Users/runner/bin:/Users/runner/.yarn/bin:/Users/runner/Library/Android/sdk/tools:/Users/runner/Library/Android/sdk/platform-tools:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/usr/bin:/bin:/usr/sbin:/sbin:/Users/runner/.dotnet/tools:/Users/runner/.ghcup/bin:/Users/runner/hostedtoolcache/stack/2.9.1/x64

.venv/bin/activate

ls: ./local/bin: No such file or directory
Error: Process completed with exit code 1.

I wonder if your action is caching a really old version 1, and it's not using the newest version.

good intuition! if I delete Load cached poetry installation,  all workflows pass ✅
any idea how to purge the cache?

You could always use a different cache key or add a restore key that's different than what it's using now. Poetry version changed but your pyproject.toml didn't, so the next change to your pyprotect.toml would have done it also.

I'm having the exact same issue (exit code 127, only on macOS-latest) with the following workflow. Is there something I should change?
  push:
    branches: [main]
  pull_request:

jobs:
  build:
    runs-on: ${{ matrix.os }}
    strategy:
      fail-fast: false
      matrix:
        python-version: ["3.10"]
        os: [ubuntu-latest, macOS-latest, windows-latest]
    env:
      POETRY_VIRTUALENVS_IN_PROJECT: true
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-python@v2
        with:
          python-version: ${{ matrix.python-version }}

      - name: cache poetry install
        uses: actions/cache@v2
        with:
          path: ~/.local
          key: poetry-1.2.1-0

      - uses: snok/install-poetry@v1
        with:
          version: 1.2.1
          virtualenvs-create: true
          virtualenvs-in-project: true

      #  Cache your dependencies (i.e. all the stuff in your `pyproject.toml`)
      - name: cache venv
        uses: actions/cache@v2
        with:
          path: .venv
          key: venv-${{ runner.os }}-${{ matrix.python-version }}-${{ hashFiles('**/poetry.lock') }}
      - run: poetry install --no-interaction --no-root
        if: steps.cache-deps.outputs.cache-hit != 'true'
      - run: poetry install --no-interaction
      - run: poetry run flake8 src/ tests/
      - run: poetry run pytest```

I'm having the exact same issue (exit code 127, only on macOS-latest) with the following workflow. Is there something I should change?
  push:
    branches: [main]
  pull_request:

jobs:
  build:
    runs-on: ${{ matrix.os }}
    strategy:
      fail-fast: false
      matrix:
        python-version: ["3.10"]
        os: [ubuntu-latest, macOS-latest, windows-latest]
    env:
      POETRY_VIRTUALENVS_IN_PROJECT: true
    steps:
      - uses: actions/checkout@v2
      - uses: actions/setup-python@v2
        with:
          python-version: ${{ matrix.python-version }}

      - name: cache poetry install
        uses: actions/cache@v2
        with:
          path: ~/.local
          key: poetry-1.2.1-0

      - uses: snok/install-poetry@v1
        with:
          version: 1.2.1
          virtualenvs-create: true
          virtualenvs-in-project: true

      #  Cache your dependencies (i.e. all the stuff in your `pyproject.toml`)
      - name: cache venv
        uses: actions/cache@v2
        with:
          path: .venv
          key: venv-${{ runner.os }}-${{ matrix.python-version }}-${{ hashFiles('**/poetry.lock') }}
      - run: poetry install --no-interaction --no-root
        if: steps.cache-deps.outputs.cache-hit != 'true'
      - run: poetry install --no-interaction
      - run: poetry run flake8 src/ tests/
      - run: poetry run pytest```

fixed it by adding
env:
  POETRY_VERSION: 1.2.1

And replacing
key: poetry-1.2.1-0

with
key: poetry-cache-${{ runner.os }}-${{ matrix.python-version }}-${{ env.POETRY_VERSION }}


Reference UIManager directly, not via NativeModules

Fixes initialization bug where NativeModules.UIManager has not yet been initialized before render is called and constants are accessed.
While debugging today, I was hitting a crash caused by the failure of UIManager to have fully intialize various propeties, including the constant values, of RCTVideo, which meant that this call:
    const RCTVideoInstance = this.getViewManagerConfig('RCTVideo');

would return null, and this call would crash:
RCTVideoInstance.Constants.ScaleToFill;
By referring to UIManager instead of NativeModules.UIManager, we seem to force initialization of the constants in time for their access within the render method of Video.js
This project also appears to be the only one on the internet that accesses UIManager via NativeModules.
This is related to: https://github.com/react-native-video/react-native-video/issues/2295

This prevents the project from working on react-native-windows in release mode, since in release mode react-native-windows run the UIManager as a TurboModule rather than a NativeModule.

At a high level, I think the approach used by the package needs to change, which will avoid this problem altogether.
Today, these View Configs are read in order to get constants from native.
https://github.com/react-native-video/react-native-video/blob/6af61a35dab0bc6df15286632bcb616b1d28e925/Video.js#L280-L290
The downside of this approach is that on startup of the app React Native has to do reflection do read the values of these constants from native to pass them to JS. To avoid this performance issue, I'd avoid reading the view config for constants at all.
Instead of reading the constants to be able to just pass back to native as a prop (presumably some integer), I'd recommend just using a string.
Consider changing that code to:
if (resizeMode === VideoResizeMode.stretch) {
  nativeResizeMode = 'ScaleToFill';
} else if (resizeMode === VideoResizeMode.contain) {
  nativeResizeMode = 'ScaleAspectFit';
} else if (resizeMode === VideoResizeMode.cover) {
  nativeResizeMode = 'ScaleAspectFill';
} else {
  nativeResizeMode = 'ScaleNone';
}

And then you can do the parsing on the native side.
This performance optimization you can make now, but it will actually be required with the new React Native architecture. Natively calculated view configs will no longer exist, so there will no longer be View Manager constants. Those will need to get defined in JS (we recommend doing it with strings like the example above).

not stale.
@esilverberg Do you know what needs to happen to push for this change?

I am afraid not; I have no direct involvement with this project. I just happened to discover this bug over one year ago and patched it in my own branch.

I'm confused by the comment from @acoates-ms. Is this fix going to break Windows?

Merged via another PR.


fatal: Authentication failed  on git push azure in tutorial 'Deploy your application code to Azure'

After failing to go through the Python Web App from VSCode tutorial (see this issue ) I am trying to continue using the CLI option, with instructions from 3 - Deploy your application code to Azure, "deploy using local git", "azure CLI".
git push azure master:master (as per branch name) hangs with no message for a while, then:
fatal: Authentication failed for 'https://azwebapp-tut-jm.scm.azurewebsites.net/azwebapp-tut-jm.git/'

Contrary to what the documentation states, no username / password (which I have) is ever prompted for.
Other repro information
git version 2.35.1
in case that matters, Debian package
Package: openssh-client
Version: 1:8.9p1-3

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

ID: af57ca1d-6109-b46b-ec0e-50fb35d8981a
Version Independent ID: 2f24c98c-d89f-8693-2618-d49b42b94b51
Content: Quickstart: Deploy a Python (Django or Flask) web app to Azure - Azure App Service
Content Source: articles/app-service/quickstart-python.md
Service: app-service
GitHub Login: @DavidCBerry13
Microsoft Alias: daberry

Nevermind, got confused by the VSCode behavior of a GUI prompt at the top instead of directly in the terminal windows...

Reopening. Still fails  when using the password obtained from az webapp deployment list-publishing-credentials, with the same autheitication failure message.

@jmp75 - Thanks for letting us know about this.  Can I confirm you are on a Linux workstation and that you were never prompted for the git credentials?

@DavidCBerry13  I am working from a Debian Wormwood (testing) setup indeed. I am prompted for the password contrary to what I initially said (not a user name though, so I assume this is actually more like a personal authentication token). I've updated the issue report to be clearer.
I should probably restart from scratch using only one method i.e. the az CLI. It looked like I could swap from the VScode based tutorial I got stuck with, but maybe I am missing something and barking up the wrong tree.


Default of None converts non-optional fields to optional

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
It looks like Pydantic pydantic treats fields with an default Value of None as optional even if they are not marked as such.
I also tried setting validate_all = True and the behavior is the same.
This behavior seems strange to me, and conflicts with the documentation here.
Changing None to a different invalid value would make the example in the documentation behave as described.
import pydantic

class MyModel(pydantic.BaseModel):
    int_with_default_int: int = 123
    int_with_default_none: int = None
    class Config:
        validate_all = True

MyModel(int_with_default_none=None) # this is accepted
MyModel() # this is also accepted
MyModel(int_with_default_int=None) # this is correctly rejected

Environment
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
pydantic version: 1.9.1
pydantic compiled: True
install path: /home/___/.venv/lib/python3.10/site-packages/pydantic
python version: 3.10.5 (main, Jun  8 2022, 09:26:22) [GCC 11.3.0]
platform: Linux-5.18.11-xanmod1-x86_64-with-glibc2.33
optional deps. installed: ['typing-extensions']

This is the documented behaviour of pydantic v1.x.
It's not a bug, but the behaviour will change in pydantic V2.

Thanks for the (very) quick response!
I searched and couldn't find this mentioned in the documentation, maybe it was removed as it is planned to change?
The bit at https://pydantic-docs.helpmanual.io/usage/validators/#validate-always mentions different behavior so it should probably be changed.


Prevent syncing while libfuzzer is running

Summary of the Pull Request
What is this about?
PR Checklist

[x] Applies to work item: #194
[ ] CLA signed. If not, go over here and sign the CLI.
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Info on Pull Request
What does this include?
Validation Steps Performed
How does someone test & validate?

When run as part of an ensemble, this can prevent fuzzer that doesn't find crashes from learning of new inputs from the ensemble.  I have a suggestion for how to handle that.  Let's talk on Monday.

Performing integration testing


Docs: Vitest docs are both incorrect and incomplete

What is the improvement or update you wish to see?
Fix docs around vitest CLI and improve docs around caching and file storing.
Is there any context that might help us understand?
In the Turborepo x Vitest there are these commands:
{
  "scripts": {
    "test": "vitest",
    "test:watch": "vitest --watch"
  }
}

Which are wrong.
vitest does watch the files, while vitest run run the tests and then stop (thus being suitable for CI). Reference.
Am i misunderstanding something?
Also, vitest has docs for caching and docs for storing output files and  coverage files.
it would be nice if Turborepo mentioned and gave guidance about these docs, here.
Does the docs page already exist? Please link to it.
https://turbo.build/repo/docs/guides/tools/vitest

We break these up into two different commands since a bare vitest command has different behavior in different environments, as you've noted. That means we need two separate turbo tasks, as noted on the documentation.
You're right that the vitest invocations could be updated for better clarity! We can also add a link to the storing of coverage files documentation, since that's a common ask.
Let's avoid linking to that caching doc, since it looks like that happens by default, and I don't want to confuse folks about Turborepo caching vs. Vitest caching. Power users can reach for Vitest's caching options if they need them.
I'm happy to review a PR for this, if you'd like to make one! If not, I can get it handled. Let me know. 👍


Fix Rubocop offenses

Coverage decreased (-0.005%) to 98.209% when pulling 083795cff1f467df2a57e59bb59d92e0bd4b6ad6 on mb-fix-rubocop into f28a54a9ba6960ebcae23b4d3b95cdafa6ee4c7b on master.


Added function to write values to ini file

Function now takes in arguments for header, section, and value, and writes them to ini file. Resolved #22

You need to use the ptree allready present in the private class member variables.  Also add comments.  Also change the class name to something that reflects both reading and writing.

Getting closer. I thought we decided to just pass one string that already includes the "Header.Value" that is how the other (read) function works.  In addition, a writeMap() function needs to be created.
Lastly, I think a more appropriate name would be INIParser as opposed to INI handler.


Refactor alert

https://trello.com/c/4SR4Nbxi/1061-refactor-add-alert
Description
This PR refactors the add alert workflow to follow our existing pattern of using forms and service classes to make changes to resources rather than callbacks and legacy audit log logic.
This change also removes the wizard flow from the alerts controller.

Checklist:

[ ] Have you documented your changes in the pull request description?
[ ] Does the change present any security considerations?
[ ] Is any gem functionality overloaded? Eg: Devise controller methods being overloaded.
[ ] Has acceptance criteria been tested by a peer?
[ ] Has the CHANGELOG been updated? (If change is worth telling users about.)

General testing (author)

[ ] Test without JavaScript
[ ] Test on small screen

Accessibility testing (author)

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable CSS - does content make sense and appear in a logical order?

Bug spotted: If submitting the form with default values, the following is displayed:


How do I...

Sorry if this has been asked before.
I'm trying to simulate the vscode experience in nvim and there are fundamentally two features I miss:

Smarter autocompletion: the sorting of my current autocomplete suggestions is alphabetical. The one in vscode is by recent use. Is it possible to enable this?
Populate symbols: when I currently go into :CocList > symbols, it is empty. This extension claims to offer the possibility to find symbols at least in the buffer. Vscode's Go To Symbol is very powerful, can I emulate this?

Thanks.

It is sorted like vscode.
Type characters to search symbols,you have to use interactive mode

I'm trying the following command:
:CocList --interactive symbols
and I receive:
[coc.nvim] Error on "CocList symbols": Workspace symbols provider not found for current document
I have tried :CocList symbols --interactive but it says the list only works in interactive mode.

but it says the list only works in interactive mode.

It's documented.

Workspace symbols provider not found for current document

You have to use it within document handled by tsserver.

How can I tell a file is being handled by tsserver? I am running the command on a typescript file.

https://github.com/neoclide/coc.nvim/wiki/Debug-language-server#using-output-channel

Hey thanks, the symbols filter suddenly started working and it's quite nice. Is there a way to start the preview right away? That'd be neat.
I'm also trying to make outline list command work but it just stays there empty. The server shows a successful response but the list is empty. Any clues?

any solution for this error?

@aigo-tech configure  language server that support workspace symbols for your buffer.


Players models equipping with different suits that selected

Sometimes when equipping a suit from the rack, the actual suit that makes it onto the player doesn't match the one selected. It is a different suit one or more spaces down the rack. Noticed during online multiplayer with both clients having the same mods installed. I have verified the correct suit name is indicated beneath the pointer when making selection.

I haven't seen this issue myself, so I'm inclined to believe it's from another mod.


cant compile bitcoin-qt at master because of missing locales

It is all in the title.
I do following:
ubuntu-Delly:litecoin nmelzer [master] $ qmake
Project MESSAGE: Building with UPNP support
Project MESSAGE: Building with UPNP support
Removed plural forms as the target language has less forms.
If this sounds wrong, possibly the target language is not set or recognized.
RCC: Error in 'src/qt/bitcoin.qrc': Cannot find file 'locale/bitcoin_de.qm'
RCC: Error in 'src/qt/bitcoin.qrc': Cannot find file 'locale/bitcoin_es.qm'
RCC: Error in 'src/qt/bitcoin.qrc': Cannot find file 'locale/bitcoin_nl.qm'
RCC: Error in 'src/qt/bitcoin.qrc': Cannot find file 'locale/bitcoin_ru.qm'
ubuntu-Delly:litecoin nmelzer [master] $ make
  [...]

/usr/bin/rcc -name bitcoin src/qt/bitcoin.qrc -o qrc_bitcoin.cpp
RCC: Error in 'src/qt/bitcoin.qrc': Cannot find file 'locale/bitcoin_de.qm'
make: *** [qrc_bitcoin.cpp] Fehler 1

Since coblee merged pull request #13 this is also closed...


.NET Core

Is MagicOnion technically not able to run on  .NET Core ( or the 2.0 beta )?

Currently no.
But I'm planning to move the project to Visual Studio 2017 soon.
At that time, it may be made as a netstandard 1.5(includes .NET Core) project.

Hey @neuecc, is this still true?

It is still the .NET Framework.
I have to make many modifications around reflection.
Please wait until the beginning of next month because I need a little time.

I've released 0.5.0 and it supports .NET Core.


Detect system, use it to pick the right extension for the shared library

The shared library name is different on macox. This PR adds a build config which allows to pick the right name.

thanks for the fix. In fact, since the ~from workaround is needed only for Toplevel (afaik), we could also add something like this
let from = if !Sys.interactive
   then Some (Dl.(dlopen ~filename:"libSDL2_image-2.0.so.0" ~flags:[RTLD_NOW]))
   else None

let foreign name typ = foreign name typ ?from

Then for any system has yet another different filename, it should still work at least for compiled programs.
(of course we can combine this with your fix)

Btw since you have a mac, could you have a look at this other config: https://github.com/sanette/tsdl-ttf/pull/1  ?

on my system, if I ignore the ~from argument of foreign, it compiles fine, and I can link the library to other programs.
I only get an error when doing #require "tsdl-image"; from the toplevel.
What you are saying is that you get an error without the ~from, outside of toplevel; is this correct?

put in another way, did you have the same issue with version 0.2?

on my system, if I ignore the ~from argument of foreign, it compiles fine, and I can link the library to other programs. I only get an error when doing #require "tsdl-image"; from the toplevel. What you are saying is that you get an error without the ~from, outside of toplevel; is this correct?

Confirmed. I have updated the PR to override foreign only when running as top-level!

sounds good. Thank!

let from = if !Sys.interactive

in fact this was a bad idea of mine... If you have a "toplevel file" like this one
#use "topfind";;
#thread;;

Printf.printf "Sys.interactive = %b\n" !Sys.interactive;;

#require "tsdl-image";;

Tsdl_image.Image.(init Init.empty) |> ignore

and execute this with ocaml, the bug reappears because it is not considered interactive...
$ ocaml bug.ml
Sys.interactive = false
Exception:
Dl.DL_error
 "/home/san/.opam/4.08.1/lib/stublibs/dllctypes-foreign_stubs.so: undefined symbol: IMG_Init".
File "./bug.ml", line 1:
Error: Reference to undefined global `Tsdl_image'


Heroku Deployment

Any insight on how to deploy with heroku and not have the slug size get huge?
I suspect some of the native binaries that need to get built are the issue.

Hey @joegaudet, feel free to dig into this and submit a PR.


Exercise keys logic

Implement logic for UI, read developer guide implementation for more details.
Supports Clear command as well, some light testing done.

Codecov Report
Base: 70.77% // Head: 70.59% // Decreases project coverage by -0.17% :warning:

Coverage data is based on head (fc15712) compared to base (9f2966b).
Patch coverage: 67.79% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #107      +/-   ##
============================================
- Coverage     70.77%   70.59%   -0.18%
- Complexity      391      404      +13
============================================
  Files            71       72       +1
  Lines          1249     1306      +57
  Branches        123      127       +4
============================================
+ Hits            884      922      +38
- Misses          324      342      +18
- Partials         41       42       +1

Impacted Files
Coverage Δ

src/main/java/gim/logic/LogicManager.java
71.42% <0.00%> (-3.58%)
:arrow_down:

src/main/java/gim/model/Model.java
100.00% <ø> (ø)

src/main/java/gim/ui/MainWindow.java
0.00% <0.00%> (ø)

src/main/java/gim/ui/SavedExerciseListWindow.java
0.00% <0.00%> (ø)

.../main/java/gim/model/exercise/ExerciseHashMap.java
63.63% <68.00%> (+0.22%)
:arrow_up:

src/main/java/gim/logic/commands/ClearCommand.java
100.00% <100.00%> (ø)

src/main/java/gim/model/ExerciseTracker.java
94.11% <100.00%> (+1.26%)
:arrow_up:

src/main/java/gim/model/ModelManager.java
100.00% <100.00%> (ø)

src/main/java/gim/model/exercise/ExerciseKeys.java
100.00% <100.00%> (ø)

... and 1 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Feature]: Improvising DOB's calendar

[x] Would you like to take up this task?
[x] Are you from GSSoC'23?

✨
Feature request

Is your feature request related to a problem?
No, want to improve Date of birth's calendar in signup card

Describe the solution you'd like
Scene- Movies app has a basic calendar in DOB in sign up form, It would look great to add a new style calendar

Describe alternatives you've considered
Yes, I've thought of another way to do this. Just by giving date, month and year in number format instead of having the present calendar

Screenshot

I can work on both the proposed solution and the alternative solution. Its upto you to tell me on which one should i work
I believe I can do this.
THANK YOU!

Hey @sourabhsikarwar, Can I work on this? You labeled but didn't assign this task to me 😅.

@sourabhsikarwar I can add a more attractive yet simple calendar.

Hey @anamika7153 , I've already linked a previous issue's PR if @sourabhsikarwar just reviews and tells me what to do I can work on this.
please check out other issues.


test: add missing tests

Description
Add more tests.
Checklist
All checkboxes must be checked before this PR can be reviewed.

[ ] Have you tested your feature/bug for responsive layout?
[ ] Have you tested your feature/bug in multiple browsers/devices via BrowserStack?
[ ] Has this PR been pulled and tested by someone other than yourself?

Closes
#9

Codecov Report

Merging #10 (42a9f96) into main (347ce85) will increase coverage by 4.95%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   81.70%   86.66%   +4.95%
==========================================
  Files           7       10       +3
  Lines          82       90       +8
  Branches       18       18
==========================================
+ Hits           67       78      +11
+ Misses         10        5       -5
- Partials        5        7       +2

Impacted Files
Coverage Δ

src/mockData/blocks.headings.ts
100.00% <ø> (ø)

src/mockData/blocks.paragraph.ts
100.00% <ø> (ø)

src/mockData/defaults.ts
80.00% <50.00%> (-20.00%)
:arrow_down:

src/mockData/blocks.child-page.ts
100.00% <100.00%> (ø)

src/mockData/blocks.todo.ts
100.00% <100.00%> (ø)

src/mockData/blocks.toggle.ts
100.00% <100.00%> (ø)

src/utils/convert-blocks.ts
83.33% <100.00%> (+11.11%)
:arrow_up:

... and 1 more


[AutoPR billing/resource-manager] Updated the operation id for Billing spec

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5727

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/3853