[WIP] New tutorials on Learn

This PR and branch will cover the following tutorials:

pulumi fundamentals
aws fundamentals
azure fundamentals
gcp fundamentals
pulumi in practice
pulumi in operations

For those working on this, see https://github.com/pulumi/pulumi-hugo/blob/master/LEARN.md for how to add new topics, etc!
Also, please append /learn/pulumi-fundamentals/ to the preview URL to see what this looks like, etc
Signed-off-by: Matt Stratton<EMAIL_ADDRESS>

@cnunciato can you fix up the white space between the different modules? id like there to be some space between them

@cnunciato @nimbinatus i think we are going to need some indicator of level or similar on the cards. something to indicate that pulumi fundamentals on azure requires pulumi fundamentals first, in order to be successful.
@nimbinatus id also recommend adjusting the titles a bit, because i don't think repeating pulumi fundamentals makes it clear to folks where they should start.

PR here to adjust the horizontal spacing: https://github.com/pulumi/pulumi-hugo/pull/597

random thought: should we come up with another name other than module since module is what we use in our API ref docs?

Yes, I believe we’re going to be changing this to align with some new terminology — “tutorials” or “guides”, IIRC.

Before I forget, here's the current feedback from other reviews that needs to get addressed as well:

[ ] Lede copy needs to reflect explaining concepts of Pulumi through an real-life app
[ ] Subheader copy should reflect #1
[ ] Should we do other languages in the future?
[ ] We don't need to follow the tutorial->module->topics convention. We can create our own hierarchy for what makes sense. Christian can change the code to reflect whatever we decide
[ ] Does AWS, GCP, K8s, and Azure tutorials become their own "top level card" or are they embedded into the Fundamentals
[ ] About this module is great
[ ] Topics doesn't show a sequence. It almost seems like you can skip to #3.
[ ] Topic 2 - "ideal configuration of your infrastructure" - its not ideal but desired right?
[ ] Bring in the concept of state. The program produces a desired configuration graph
[ ] I really like how you make them run into an error in Topic 3 as a way to learn
[ ] name=f"backend-{stack}", - needs comma
[ ] The code sometimes has 5 tabs and sometimes 9
[ ] Why do we export the mongo container ID versus the front end one
[ ] Topic 4 - you can't get this display without running pulumi up first. "You can also list the current stack’s resources with pulumi stack:"
[ ]  Topic 4 - typo - "So what else can do with stack outputs? "
[ ] I would add a summary of what people learned at the end and then direct them to the cloud specific tutorials
[ ] You want to replace spara? "pulumi config set org spara"
[ ] Give some examples of something that is incredibly useful "These exported values are incredibly useful when using Pulumi stacks."
[ ] "Use the defaults, and ensure you use the prod stack, the same one you currently have up for the my-first-app project." - this is a bit confusing. basically accept all the defaults but the stack defaults and type in prod instead
[ ] stack_ref is the variable StackReference is the declaration
[ ] For the use-docker-id project, I am actually confused about the relationship of the prod stacks and the relationship of projects and stacks. Why does use-docker-id need to use a prod stack to be able to see to my-first-app? Doesn't <f"{org}/my-first-app/{stack}"> denote the stack to reference from?

@zchase imm gonna be updating that diagram this week. don't mind if that means i need to open another pr but wanted to give you a heads up.

@susanev sounds good --- I think we can wait for your work on the diagram before we ship this.

NOTE: This current stage is a partial commit. I have to step away, so there's some rough spots for me to smooth out (namely that there's three files that are basically the same thing in the Practice pathway now). Just wanted to get changes up there for people to work from while I go.

Sorry for the huge blob of text, but I took notes as I looked through this, so apologies that a few of these could be PR feedback on the specific lines instead of in this comment.
pulumi-fundamentals/

A Pulumi account and token should link to http://pulumi-hugo-origin-pr-552-f2c00dde.s3-website.us-west-2.amazonaws.com/docs/intro/console/accounts/#access-tokens
"As to skills, you should be able to" missing colon at the end
"MongoDB, ExpressJS, NodeJS, and React (the MERN stack)" -- re-order so that it's the same as MERN?

pulumi-fundamentals/create-a-pulumi-project/

"Infrastructure in Pulumi is organized into projects, the highest level component in a Pulumi ecosystem." --Infrastructure in Pulumi is organized into projects, the highest level component in a Pulumi ecosystem. I think we should avoid using the term 'component' in general unless we're referring to actual components. Suggestion: "Infrastructure in Pulumi is organized into projects, which represent a Pulumi program and its corresponding stacks" (we may even want to reference https://www.pulumi.com/docs/intro/concepts/#overview)

pulumi-fundamentals/create-docker-images/

"In Pulumi, you have resources managed by resource providers, and then you can group those resources to abstract them (such as a group of compute instances that all have the same configuration and implementation)." -> "In Pulumi, your resources are managed by resource providers. You can group those resources to abstract them (such as a group of compute instances that all have the same configuration and implementation) via component resources."?
"then move everything from the code repository directory to the app/ directory." -- end with a colon instead of period to indicate this paragraph describes the following block of terminal commands
"the ideal configuration" -> "the desired state"
"In this file, we import, or call, the main Pulumi package and the Docker provider." -- having the "or call" feels like it will only cause confusion/ambiguity
"we figure out which stack we’re on in the command line" -- Perhaps consider rephrasing to more around what stack we're operating against since we're not in a command line context and could be confusing to the reader? This will also help emphasize the difference b/t project and stack where we can talk about how the project (program) can do stack-specific things.
"If we tried to run pulumi up without the mongo_container running, we would get an error message." -- technically, this shouldn't be possible in the sense that pulumi up is what will try to enforce all the resources exist. we'd get an error if pulumi up couldn't create the container, but if it exists in state, it will work
"set up our Pulumi file" -> program?

It's probably worth noting at some point in this article the fact that we're mixing language constructs like os.getcwd in our program and that the full language ecosystem (including 3rd party libraries) is available to us. I also think it might be helpful to explain the typing support we have and how someone can take advantage of either their IDE completion or our documentation to determine the correct types for the inputs.
Secrets
One of the questions/concerns that usually comes up is "is secretness transitive?" i.e. if I create an output from a secret, is that output secret? We should show an example of how secrets 'taint' other outputs and preserve secretness.
Other
When we end, shouldn't the primary button take me back to the Learn homepage vs. the module's index? I've just completed the module, so I don't need to go back to the index.

This is great feedback, @leezen! We really appreciate it!

One thought for fundamentals; we should make sure to end the tutorial with a pulumi destroy (not sure we want to do a destroy of the stack, etc, as we will keep using that in future tutorials, but the local resources should get destroyed)

@nimbinatus or @zchase could one of you update the intro paragraph so its a bit more consumable? ill be updating the diagram later today to better fit in the space as well.

@nimbinatus for the fundamentals on cloud, is there any way we can name these starting with the cloud first? so azure & pulumi fundamentals, or similar? this will make it easier for folks to scan instead of starting them all with the same word.

@susanev definitely can do both :) Give me a bit

@cnunciato there seems to be something weird about the shadows on these lesson cards, they don't seem to be the same as the rest of the website. can this be fixed?

I just pushed up the preview for the updates to the main text and the renaming of the cloud pathways. One thing I'm running into is fitting/adjusting the wording on the meta images to match; the layout is a bit tight and would look a bit odd if I put the cloud first. Thoughts?

@nimbinatus im kinda confused about these cards and the titles. it seems like we have the titles twice on the card. is that intended?
i also think there's some layout and hierarchy issues on the cards. everything is big, the pulumi logo, the pulumipus, the heading, the repeated heading. i think we should reduce the size of the pulumi logo quite a bit at a minimum.
i could take a pass at redesigning the layout of these cards later today if you think that would be helpful, let me know.

@cnunciato weird gray like border on the top, heavier shadow on the bottom

Ah, compared to the marketing cards, I see.
The cards we use on marketing pages use a custom shadow that has an even spread, whereas the Tailwind utilities we use everywhere else (shadow, shadow-xl, shadow-2xl, etc.) use a graduated spread. Basically, we have two different "light" sources going on: on the marketing site, L+R seems to have designed for a light source that hits the page head-on, giving cards and things an even shadow on all sides, whereas everywhere else (via Tailwind's default settings), the light hits the page from above -- hence longer shadows below elevated items, but no shadows above them (since that wouldn't make physical sense). I threw a 1px gray border onto the cards to give these cards definition against the white background (without which you wouldn't be able to discern their upper edges), but I agree that doesn't look great.
So the main problem is that we have an inconsistent light source. I actually prefer L+R's treatment aesthetically, so we just have to decide -- do we want to light our pages from the front, or from the top? If it's the front, awesome, we just need to add some customization to Tailwind (shouldn't be a size-smallish thing to do that), and then design with this same light source in mind going forward. What do you think? I'll make a card with all the details if you agree.

ive created some recommended ux designs for the landing page and dropped it in yalls figma file. feel free to take and use what yall think will work and drop what wont. my core recommendations are to (1) consider the hierarchy of information when laying out the cards and (2) use the cloud logos more prominently

Thanks for the help, @susanev. I'm happy to take care of this.

@cnunciato i think @nimbinatus is going to make some decisions on what specific things they want from that and what they don't, so id hold off for now.

@nimbinatus @susanev checking in on the UI adjustments above. Do y'all want to move forward with this? Any changes to make first?

Congratulations! You’ve finished the Pulumi in Practice pathway! In this pathway, you learned all about stacks, outputs, and stack references so you can work in multiple environments. You also learned about secrets in Pulumi and testing with standard third-party frameworks. The best next step in your journey to learn Pulumi is to work on the Pulumi in Operations pathway.  - the link to Pulumi in Operations isn't correct

noticed that the button on the last pages is outlined instead of fill why did we make this choice?
also its a bit weird that the last buttons have the forward arrow since you are really going backward at that point
with only 2 cards i think it looks a bit strange that its left-aligned; we may want to center the pathways heading and the cards on the page

i only looked very quickly, though did try to scan every page and use every button. i do not feel comfortable with this merging until someone else reviews though.

Completely fair. I think there's going to be a lot of reviews for this (some folks have already reviewed the content, but explicit reviews are better).
Re the buttons, that was how it was when I picked this up. Maybe a question for @cnunciato?
And I can center the cards for now to patch it until the other pathways come out if you think that makes sense :)


Editable Parameter does not work

Hi,
I've been fighting this in v17.1 and it does not appear that passing a parameter to the editable function.
I've tried it in many locations, when creating the page, on init, on openpage, onaftergetrecord, even onaftergetcurrrecord.
Thanks
Yann

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

ID: aa18ed00-c64a-b31f-6824-fce50ab40319
Version Independent ID: 862ad8b6-574b-bc16-8633-549b9f8747e6
Content: Page.Editable Method - Business Central
Content Source: dev-itpro/developer/methods-auto/page/page-editable-method.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for your feedback. This could be a product bug. Please contact your usual support channel to file a bug. Thanks.


Support AD directory in path

e.g. it could support https://resources.azure.com/microsoft.com/subscriptions/etc... to make the directory explicit.

When the user is initially logging in, they will log into their default directory, that is whichever one AAD picks for them.
After that if we have a deep link, we will query ARM for the tenant Id for the given subscription. ARM has mapping of subscriptionId <-> tenantId, for all Azure Subscriptions.

If it's the same as the current tenantId, nothing to do, proceed with existing deep link logic.
If not, save the deep link, redirect to the correct tenant, redirect back to explorer, extract the saved deep link, and proceed as before

We don't have too.  The subscriptionId and tenantId are dependent and, given subscriptionId, there is a very quick and unauthenticated way to find out tenantId - little to no perf.   It will require at most one redirect to get the token (which is the same for Portal).   Having both in deep link is unnecessarily redundant.

Ok, closing as this appears unnecessary.


fix: add support for microsoft onelake

Made code change to test utils to handle abfss<EMAIL_ADDRESS>Reads/Writes need to use a fully qualified https or abfss (onelake.dfs.fabric.microsoft.com/ws-id/artifact-id

@vmuddassir-msft are you getting to a point to where you're ready to put this into Review?

@vmuddassir-msft are you getting to a point to where you're ready to put this into Review?

Delta-rs needs to be upgraded to the latest version of object_store v0.7.0 , in order to add Support for Microsoft Fabric / OneLake . I have upgraded the versions for object_store with other dependencies which reference object_store v0.6.1
I have facing an issue after the upgrade :

This is because datafusion seems to be internally using the dependency crate arrow-array v45 while the latest version is arrow-array = { version = "46.0.0" }

This is because datafusion seems to be internally using the dependency crate arrow-array v45 while the latest version is arrow-array = { version = "46.0.0" }

Yes, we need to wait for datafusion to upgrade versions of arrow before we can. Can you upgrade object store independent of arrow?

This is because datafusion seems to be internally using the dependency crate arrow-array v45 while the latest version is arrow-array = { version = "46.0.0" }

Yes, we need to wait for datafusion to upgrade versions of arrow before we can. Can you upgrade object store independent of arrow?
Unable to upgrade Object store  independently as this is also a dependency for parquet and the latest version of paraquet v46 has linked dependencies to the latest versions of all arrow related crates which includes arrow-array

@vmuddassir-msft understood. The next version of datafusion is out for vote, so it should be released sometime next week. Once that happens we should be able to get this PR in a merge-able state. :)

@vmuddassir-msft are you getting to a point to where you're ready to put this into Review?

@rtyler - PR is ready to be reviewed

@vmuddassir-msft is your expectation that the test will never be run? (in which case we should just delete it), or do you want to configure a bearer token in GitHub secrets so that at least CI can run?

Closing this in favor of the rebased #1642


MAB for Mutator selection only

Closing some old PRs. Sorry for the inactivity! Please just reopen if this is still relevant!


rust-lld: error: undefined symbol: __[start|stop]_linkm2_KNOWN_STUBS

This has been an issue before (https://github.com/dtolnay/linkme/pull/88), but now it has reached rustc beta: https://github.com/ruffle-rs/ruffle/actions/runs/9462151873/job/26064482962?pr=16448
  = note: rust-lld: error: undefined symbol: __start_linkm2_KNOWN_STUBS
          >>> referenced by 14g71b7tf2rrf8tb6zps16jjh
          >>>               ruffle_core-d15044b578f91d45.14g71b7tf2rrf8tb6zps16jjh.rcgu.o:(ruffle_core::stub::KNOWN_STUBS::h570647c79bce49d9) in archive /home/runner/work/ruffle/ruffle/target/ci/deps/libruffle_core-d15044b578f91d45.rlib
          >>> did you mean: __start_linkme_KNOWN_STUBS
          >>> defined in: <internal>
          >>> the encapsulation symbol needs to be retained under --gc-sections properly; consider -z nostart-stop-gc (see https://lld.llvm.org/ELF/start-stop-gc)

          rust-lld: error: undefined symbol: __stop_linkm2_KNOWN_STUBS
          >>> referenced by 14g71b7tf2rrf8tb6zps16jjh
          >>>               ruffle_core-d15044b578f91d45.14g71b7tf2rrf8tb6zps16jjh.rcgu.o:(ruffle_core::stub::KNOWN_STUBS::h570647c79bce49d9) in archive /home/runner/work/ruffle/ruffle/target/ci/deps/libruffle_core-d15044b578f91d45.rlib
          >>> did you mean: __stop_linkme_KNOWN_STUBS
          >>> defined in: <internal>
          collect2: error: ld returned 1 exit status

See also: https://github.com/dtolnay/linkme/actions/runs/9458387043/job/26053880019

RUSTFLAGS="-C link-args=-znostart-stop-gc" does indeed work around this.
Additional info here: https://lld.llvm.org/ELF/start-stop-gc

It seems to really be a dup of now-long-standing #49 (see that digit switch? what a coincidence!)
BTW, what was the platform in this case? (distro, libc, cc toolkit; with versions) I am wondering how to pinpoint the real culprit.

It seems to really be a dup of now-long-standing #49 (see that digit switch? what a coincidence!)
BTW, what was the platform in this case? (distro, libc, cc toolkit; with versions) I am wondering how to pinpoint the real culprit.

I have realised the main post had a ref to GHA running Ubuntu 22.04 which has clang in version 14  https://packages.ubuntu.com/jammy/clang and obviously runs glibc

Same issue here. link-args=-znostart-stop-gc as the error suggests works for me, as does link-dead-code.
Host triple: x86_64-unknown-linux-gnu
Arch Linux w/Rustup
rustc 1.85.0-nightly (a224f3807 2024-12-09)
glibc 2.40
clang/lld 18.1.8


add neurodata/hyppo

Hello! I'd like to add hyppo to this list, I am one of the maintainers and we do have a list of good first issues.
Please let me know if there is any feedback, I appreciate any reviews.

Hey, @sampan501! Thanks for your contribution!
BTW: You can also consider enabling DeepSource on your project, which gives you a dashboard of all code quality issues present in your code (just like this) so you can attract meaningful beginner contributions that can make your code better.
I hope this helps!


How to display error messages?

How to display error messages? For example, if a database or a collection exists.

Hi eldertrevisan,
For database and collection you can catch KeyError or arango.exceptions.NotFoundError:
try:
    a.database('missing_database')
except KeyError:
    print('db missing!')

For any request related errors you can do the following:
from arango.exceptions import RequestError
try:
    # something
except RequestError as err:
    print(err.message)
    print(err.error_code)
    print(err.http_code)
    print(err.url)
    print(err.method)

I am planning to improve the exceptions in the future
Jay

Hi eldertrevisan,
In the next release, ALL python-arango exceptions will subclass from ArangoError. I will also put up a documentation page soon so you have something to refer to. For now I will close this issue. Thanks.


Fix ESLint indentation warnings

This properly indents the code to fix 300+ ESLint warnings.
Six max-len warnings remain, but they're less straightforward to resolve, so I leave that up to you.

Coverage remained the same at 99.31% when pulling a21fbee1ab1a37b2affad45429829f05b2583fbb on TheDancingCode:fix-eslint-warnings into a58a58d95d004271f06c4df31138b859fe73b619 on adametry:master.

Coverage remained the same at 99.31% when pulling a21fbee1ab1a37b2affad45429829f05b2583fbb on TheDancingCode:fix-eslint-warnings into a58a58d95d004271f06c4df31138b859fe73b619 on adametry:master.

@adametry
Did you get a chance to look at this?

Is this the same error as this:
142:11  error  Expected indentation of 12 space characters but found 10  react/jsx-indent
'lint-app' errored after 5.46 s
ESLintError in plugin 'gulp-eslint'

Not exactly. This PR fixes the same kind of error, but then for the code found in this repo.


unit-test/test-continuable-single fails on gcc 8.2

@Naios
unit-test/test-continuable-single results in segmentation fault on gcc 8.2
Also tested with clang++ 7.0 and it passes, and it looks like gcc6 on Travis also passes thus I am not sure if it is a compiler regression or something like an undefined behavior (I will update the post after testing with gcc-trunk)
Output of running the tests (successfull tests are ommitted):

../test/unit-test/single/test-continuable-flat-variant.cpp:70: Failure
Expected equality of these values:
e.cast()
Which is: 4790218
CANARY
Which is: 373671
../test/unit-test/single/test-continuable-flat-variant.cpp:78: Failure
Value of: e.is()
Actual: false
Expected: true
[  FAILED  ] flat_variant_single_test.is_copy_constructible (0 ms)
../test/unit-test/single/test-continuable-flat-variant.cpp:121: Failure
Value of: variant.template is()
Actual: false
Expected: true
../test/unit-test/single/test-continuable-flat-variant.cpp:121: Failure
Value of: variant.template is()
Actual: false
Expected: true
../test/unit-test/single/test-continuable-flat-variant.cpp:121: Failure
Value of: variant.template is()
Actual: false
Expected: true
../test/unit-test/single/test-continuable-flat-variant.cpp:121: Failure
Value of: variant.template is()
Actual: false
Expected: true
../test/unit-test/single/test-continuable-flat-variant.cpp:121: Failure
Value of: variant.template is()
Actual: false
Expected: true
../test/unit-test/single/test-continuable-flat-variant.cpp:160: Failure
Value of: destroyed
Actual: false
Expected: true
[  FAILED  ] flat_variant_single_test.test_leak_regression (1 ms)
../test/unit-test/single/test-continuable-result.cpp:152: Failure
Value of: bool(e)
Actual: false
Expected: true
../test/unit-test/single/test-continuable-result.cpp:153: Failure
Expected equality of these values:
*e
Which is: 11712264
CANARY
Which is: 373671
../test/unit-test/single/test-continuable-result.cpp:154: Failure
Value of: e.is_value()
Actual: false
Expected: true

I also got several warnings on uninitialized variables but I am not sure if they are intentional or bugs.
../include/continuable/detail/utility/flat-variant.hpp:369:23: warning: ‘((void)+16)’ may be used uninitialized in this function [-Wmaybe-uninitialized]
../include/continuable/detail/utility/traits.hpp:99:35: warning: ‘((void)+8)’ may be used uninitialized in this function [-Wmaybe-uninitialized]
There should be a few more like these I can add those as well

Commit Hash
e23e363
Expected Behavior
Tests should pass
Actual Behavior
Some of the tests fail and program finishes with segmentation fault
Steps to Reproduce
cmake -G Ninja -DCMAKE_BUILD_TYPE=Release ..
ninja test
Your Environment

OS: Fedora 28
Compiler and version: GCC8.2
Standard library (if non default): Default

With g++9.1.0 also fails.
$ g++-9 -v
Using built-in specs.
COLLECT_GCC=g++-9
COLLECT_LTO_WRAPPER=/usr/local/Cellar/gcc/9.1.0/libexec/gcc/x86_64-apple-darwin18/9.1.0/lto-wrapper
Target: x86_64-apple-darwin18
Configured with: ../configure --build=x86_64-apple-darwin18 --prefix=/usr/local/Cellar/gcc/9.1.0 --libdir=/usr/local/Cellar/gcc/9.1.0/lib/gcc/9 --disable-nls --enable-checking=release --enable-languages=c,c++,objc,obj-c++,fortran --program-suffix=-9 --with-gmp=/usr/local/opt/gmp --with-mpfr=/usr/local/opt/mpfr --with-mpc=/usr/local/opt/libmpc --with-isl=/usr/local/opt/isl --with-system-zlib --with-pkgversion='Homebrew GCC 9.1.0' --with-bugurl=https://github.com/Homebrew/homebrew-core/issues --disable-multilib --with-native-system-header-dir=/usr/include --with-sysroot=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk
Thread model: posix
gcc version 9.1.0 (Homebrew GCC 9.1.0)


Adding Login and Signup pages with jwt authentication

I am interested in this task. Please assign it to me under gssoc.
I propose to:

Implement a Signup page where new users can register by providing necessary information such as username, email, and password.
Ensure that passwords are securely hashed before storing them in the database using bycrypt hashing.
Implement a Login page where existing users can enter their credentials to access the application.
Upon successful login, generate a JWT token and send it to the client to be stored securely (e.g., in local storage or cookies).
Implement middleware to validate JWT tokens for protected routes, ensuring that only authenticated users can access certain parts of the application.
Handle authentication errors gracefully and provide appropriate feedback to users.
Secure API endpoints by verifying JWT tokens before allowing access to sensitive data or operations
I have earlier also implemented this type of functionality in one of my projects.You can check out at https://github.com/ManpreetSingh18/tavPrasad

I have already done Login and Signup authentication.
Someone is already assigned to create the Login and Signup pages.
Once the pages are created I will assign them to you for backend integration.


6.0: [ClosureSpecializer] Bail on noncopyable argument.

Explanation: Fix a miscompile involving captured noncopyable value.
When the ClosureSpecializer pass runs, it produces new functions which are invoked later than the partial_apply which they replace.  This requires extending the lifetime of each captured value.  This is done by retaining the value and having the specialization release it.  That doesn't work for noncopyable values.  Retain has no effect on such values and the first release will destroy the value.  The result was that the value was destroyed when the specialized closure ran.
Here, this is fixed by bailing out of the optimization when one of the captures is noncopyable.
Scope: Affects noncopyable code.
Issue: rdar://129622373
Original PR: https://github.com/apple/swift/pull/74309
Risk: Low.
Testing: Added and updated tests.
Reviewer: Andrew Trick ( @atrick )

@swift-ci please test


fix oss build

for oss build, broken due to #2686.

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


Module(@typescript/vfs): lib files from cdn are broken

Improvement Idea:

Lib types fetched from @typescript/vfs' CDN are broken, they return 404 errors.
e.g. https://typescript.azureedge.net/cdn/4.7.4/typescript/lib/lib.es2020.full.d.ts

Why:

I'm not sure why it's broken but as there is no easy way to switch CDN's it's makes it very difficult to fetch lib.d.ts in WebWorkers.
Alternative ideas:

Allow the user to easily switch CDN's
Make the typescript CDN more reliable

Looking fine here?


Core dump when writing streamline file

To reproduce:

Spin up Vapor, open DUKU
Create and enable a flow renderer
Click "Write to file" in the appearance tab
Check the variable "P" and click "Write to File"
Check the variable "PB" and click "Write to File"
Vapor will crash

@sgpearse I cannot reproduce this crash on cisl-vapor. Maybe there are some missing steps in the sequence of actions?


[Issue] Microsoft.Web/sites does not respect ipSecurityRestrictions For Deny All

I recognize that this might not be the correct repository to submit this to, if I could be pointed in the right direction, I would appreciate it.
Describe the bug
I believe I have found a corner case in creating a new Microsoft.Web/sites resource in ARM wherein ipSecurityRestrictions is not respected when attempting to block all traffic when created without additional firewall rules.
To Reproduce
The following ARM Template will reproduce the issue; you may need to generate yourself a new GUID to allow the site name to be Globally Unique:
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
    },
    "variables": {
        "guidToUse": "f6358587-2652-41f2-92b0-932e0d9a50d8",
        "serverFarmName": "[concat(variables('guidToUse'), '-sf')]",
        "websiteName": "[concat(variables('guidToUse'), '-ws')]"
    },
    "resources": [
        {
            "type": "Microsoft.Web/serverfarms",
            "apiVersion": "2022-03-01",
            "name": "[variables('serverFarmName')]",
            "location": "northcentralus",
            "sku": {
                "name": "F1"
            },
            "properties": {
            }
        },
        {
            "type": "Microsoft.Web/sites",
            "apiVersion": "2022-03-01",
            "name": "[variables('websiteName')]",
            "location": "northcentralus",
            "dependsOn": [
                "[resourceId('Microsoft.Web/serverfarms', variables('serverFarmName'))]"
            ],
            "properties": {
                "siteConfig": {
                    "ipSecurityRestrictions": [
                        {
                            "ipAddress": "Any",
                            "action": "Deny",
                            "priority":<PHONE_NUMBER>,
                            "name": "Deny all",
                            "description": "Deny all access"
                        }
                    ],
                    "scmIpSecurityRestrictions": [
                        {
                            "ipAddress": "Any",
                            "action": "Allow",
                            "priority":<PHONE_NUMBER>,
                            "name": "Allow all",
                            "description": "Allow all access"
                        }
                    ]
                },
                "serverFarmId": "[resourceId('Microsoft.Web/serverfarms', variables('serverFarmName'))]"
            }
        }
    ]
}

Expected behavior
I would expect Access Restrictions for this deployed site to be set to Deny:

Additional context
Interestingly enough if you configure this via the Console like so:

These changes will apply:

Exporting an ARM Template from this site will generate a very similar template which if you attempt to redeploy does not work.
Furthermore, if you add a single rule this appears to work as well, I suspect that most people are adding in an allow rule of some sort which is why this was never caught in testing/is a corner case:
{
    "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
    "contentVersion": "<IP_ADDRESS>",
    "parameters": {
    },
    "variables": {
        "guidToUse": "f6358587-2652-41f2-92b0-932e0d9a50d8",
        "serverFarmName": "[concat(variables('guidToUse'), '-sf')]",
        "websiteName": "[concat(variables('guidToUse'), '-ws')]"
    },
    "resources": [
        {
            "type": "Microsoft.Web/serverfarms",
            "apiVersion": "2022-03-01",
            "name": "[variables('serverFarmName')]",
            "location": "northcentralus",
            "sku": {
                "name": "F1"
            },
            "properties": {
            }
        },
        {
            "type": "Microsoft.Web/sites",
            "apiVersion": "2022-03-01",
            "name": "[variables('websiteName')]",
            "location": "northcentralus",
            "dependsOn": [
                "[resourceId('Microsoft.Web/serverfarms', variables('serverFarmName'))]"
            ],
            "properties": {
                "siteConfig": {
                    "ipSecurityRestrictions": [
                        {
                            "action": "Allow",
                            "ipAddress": "<IP_ADDRESS>/32",
                            "priority": 100,
                            "name": "Allow Example.com's IP",
                            "description": "Test Allowing IP"
                        },
                        {
                            "ipAddress": "Any",
                            "action": "Deny",
                            "priority":<PHONE_NUMBER>,
                            "name": "Deny all",
                            "description": "Deny all access"
                        }
                    ],
                    "scmIpSecurityRestrictions": [
                        {
                            "ipAddress": "Any",
                            "action": "Allow",
                            "priority":<PHONE_NUMBER>,
                            "name": "Allow all",
                            "description": "Allow all access"
                        }
                    ]
                },
                "serverFarmId": "[resourceId('Microsoft.Web/serverfarms', variables('serverFarmName'))]"
            }
        }
    ]
}

@alex-frankel - Do you know where this should be filed?

This should be filed as an Azure support ticket so it can be routed to the Web Resource Provider team. Tagging @naveedaz / @seligj95 as FYI

I actually came across this today with a deployment I was working on. Please open the support ticket. I'll also try to route this internally.

Done I've got ticket<PHONE_NUMBER>003456 assigned let's see where it goes.

No need for a support ticket, @seligj95  @aolszowka. You cannot define a Deny All rule through the ipSecurityRestrictions property. To configure Deny All, you need to set ipSecurityRestrictionsDefaultAction to Deny

@madsd Interesting; can you point out where that is documented? I didn't see anything on either of these pages:
https://learn.microsoft.com/en-us/azure/templates/microsoft.web/sites?pivots=deployment-language-arm-template
https://learn.microsoft.com/en-us/azure/templates/microsoft.web/sites/config-web?pivots=deployment-language-arm-template
It looks like someone noticed it missing in the Bicep world 22 days ago: https://github.com/Azure/bicep/issues/784#issuecomment-1282208703
Should I open a support ticket on the documentation to see what happened?

Not sure how that part of the documentation is updated. It is a site config property. You can see the property in JSON view in the portal, or doing a GET REST call on a site resource

@madsd the documentation is based off of the PUT schema defined in the resource type's swagger spec. Should a PUT with those settings fail instead of silently ignoring the desired properties?

Deny All and Allow All cannot be configured in the rule list itself. It is based on the DefaultAction (if configured) or if the list has rules or not, if DefaultAction is not configured.

Based on this statement:

cannot define a Deny All rule through the ipSecurityRestrictions property

It sounds like this setting below is being ignored:
                "siteConfig": {
                    "ipSecurityRestrictions": [
                        {
                            "ipAddress": "Any",
                            "action": "Deny",
                            "priority":<PHONE_NUMBER>,
                            "name": "Deny all",
                            "description": "Deny all access"
                        }
                    ],

Would you say that is right?
If so, I am wondering if a PUT with those settings should fail. Today it appears to succeed, even though the setting is not being applied.

Closing the Loop on my end, support confirmed the above and opened up https://github.com/MicrosoftDocs/azure-docs/issues/101666.
As far as an error, that probably would be helpful but doesn't sound like its in the cards. Thank you for all your help.


Python shows Deprecation Warning for TLS

This issue is for a: (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce
Run the getting_started scenario for python
Any log messages given by the failure
 /workspaces/MqttApplicationSamples/scenarios/getting_started/python/getting_started.py:117: DeprecationWarning: ssl.PROTOCOL_TLSv1_2 is deprecated
Expected/desired behavior
As this repo offers guidance to use existing MQTT clients, we should make sure we are providing the best sample code. We should avoid any warning.
Language
Python
OS and Version?
CodeSpaces

this warning can be fixed by replacing
https://github.com/Azure-Samples/MqttApplicationSamples/blob/a5ce9d4b6fd7bee8bd1d21a953e248f69dbacaa6/scenarios/getting_started/python/getting_started.py#L117
with
context = ssl.SSLContext(ssl.PROTOCOL_TLS_CLIENT)
context.minimum_version = ssl.TLSVersion.TLSv1_2
context.maximum_version = ssl.TLSVersion.TLSv1_3

I was going to submit a PR, and then I realized that line is copied in all the other samples.
We should refactor the code to avoid duplication by providing common functions in /mqttclients/python so we can fix this bug by updating just one file.

This fix can't be centralized until #79. I could make this change manually in each place as well. Let me know.

This fix can't be centralized until #79. I could make this change manually in each place as well. Let me know.

Why? In the same way we have ConnectionSettings we can centralize how to configure the TLS connection. What's blocking?

I have made a PR in #80 to suppress the warning.
With regard to centralizing Python more, this will need to be a separate item.


Unable to run eureka server on Weblogic 12c

i'm unable to run eureka server on Weblogic 12c.
war file is deployed successfully on Weblogic 12c but I'm receiving following error in server logs:
[2015-12-10T13:36:46.135+05:30] [AdminServer] [ERROR] [] [com.netflix.eureka.PeerAwareInstanceRegistry] [tid: Eureka-EvictionTimer] [userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:8] [APP: eureka-server-1.1.37] The lease expiration has been disabled since the number of renewals per minute   is lower than the minimum threshold. Number of Renewals Last Minute : 0. The Threshold is 0.85 of total instances : 0
[2015-12-10T13:37:08.337+05:30] [AdminServer] [ERROR] [] [com.netflix.discovery.DiscoveryClient] [tid: DiscoveryClient_ServiceURLUpdater] [userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:9] [APP: eureka-server-1.1.37] DISCOVERY: invalid zone - defaultZone defaulting to defaultZone
[2015-12-10T13:37:11.702+05:30] [AdminServer] [WARNING] [] [com.netflix.discovery.DiscoveryClient] [tid: DiscoveryClient_CacheRefresher] [userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:6] [APP: eureka-server-1.1.37] Action: Refresh  => returned status of 404 from http://localhost/eureka/v2/apps/
[2015-12-10T13:37:11.704+05:30] [AdminServer] [ERROR] [] [com.netflix.discovery.DiscoveryClient] [tid: DiscoveryClient_CacheRefresher] [userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:6] [APP: eureka-server-1.1.37] Can't get a response from http://localhost/eureka/v2/apps/[[
Can't contact any eureka nodes - possibly a security group issue? java.lang.RuntimeException: Bad status: 404
at com.netflix.discovery.DiscoveryClient.makeRemoteCall(DiscoveryClient.java:833)
at com.netflix.discovery.DiscoveryClient.makeRemoteCall(DiscoveryClient.java:753)
at com.netflix.discovery.DiscoveryClient.getAndStoreFullRegistry(DiscoveryClient.java:615)
at com.netflix.discovery.DiscoveryClient.fetchRegistry(DiscoveryClient.java:562)
at com.netflix.discovery.DiscoveryClient.access$1200(DiscoveryClient.java:87)
at com.netflix.discovery.DiscoveryClient$CacheRefreshThread.run(DiscoveryClient.java:1316)
at java.util.TimerThread.mainLoop(Timer.java:555)
at java.util.TimerThread.run(Timer.java:505)
]]
[2015-12-10T13:37:11.704+05:30] [AdminServer] [ERROR] [] [com.netflix.discovery.DiscoveryClient] [tid: DiscoveryClient_CacheRefresher] [userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:6] [APP: eureka-server-1.1.37] DiscoveryClient_EUREKA/MACHINE_NAME - was unable to refresh it's cache! status = Bad status: 404[[
java.lang.RuntimeException: Bad status: 404
at com.netflix.discovery.DiscoveryClient.makeRemoteCall(DiscoveryClient.java:833)
at com.netflix.discovery.DiscoveryClient.makeRemoteCall(DiscoveryClient.java:753)
at com.netflix.discovery.DiscoveryClient.getAndStoreFullRegistry(DiscoveryClient.java:615)
at com.netflix.discovery.DiscoveryClient.fetchRegistry(DiscoveryClient.java:562)
at com.netflix.discovery.DiscoveryClient.access$1200(DiscoveryClient.java:87)
at com.netflix.discovery.DiscoveryClient$CacheRefreshThread.run(DiscoveryClient.java:1316)
at java.util.TimerThread.mainLoop(Timer.java:555)
at java.util.TimerThread.run(Timer.java:505)
]]
NOTE: MACHINE_NAME is a name of my windows machine which cannot be disclosed

You Eureka client/server configuration assumes that you run server on port
80, and that the context path is 'eureka' (http://localhost/eureka/v2/apps).
You are using probably default configuration hence localhost address is
used, not routable address of the host. Please consult
https://github.com/Netflix/eureka/tree/master/eureka-examples/conf for
example configuration. You need to change 'eureka.serviceUrl.default' to
your eureka deployment URL.
/Tomasz
On Mon, Dec 14, 2015 at 4:22 AM, omkar9999<EMAIL_ADDRESS>wrote:

i'm unable to run eureka server on Weblogic 12c
war file is deployed successfully on Weblogic 12c but I'm receiving
following error in server logs:
[2015-12-10T13:36:46135+05:30] [AdminServer] [ERROR] []
[comnetflixeurekaPeerAwareInstanceRegistry] [tid: Eureka-EvictionTimer]
[userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:8] [APP:
eureka-server-1137] The lease expiration has been disabled since the number
of renewals per minute is lower than the minimum threshold Number of
Renewals Last Minute : 0 The Threshold is 085 of total instances : 0
[2015-12-10T13:37:08337+05:30] [AdminServer] [ERROR] []
[comnetflixdiscoveryDiscoveryClient] [tid:
DiscoveryClient_ServiceURLUpdater] [userId: ] [ecid:
4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:9] [APP:
eureka-server-1137] DISCOVERY: invalid zone - defaultZone defaulting to
defaultZone
[2015-12-10T13:37:11702+05:30] [AdminServer] [WARNING] []
[comnetflixdiscoveryDiscoveryClient] [tid: DiscoveryClient_CacheRefresher]
[userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:6] [APP:
eureka-server-1137] Action: Refresh => returned status of 404 from
http://localhost/eureka/v2/apps/
[2015-12-10T13:37:11704+05:30] [AdminServer] [ERROR] []
[comnetflixdiscoveryDiscoveryClient] [tid: DiscoveryClient_CacheRefresher]
[userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:6] [APP:
eureka-server-1137] Can't get a response from
http://localhost/eureka/v2/apps/[[
Can't contact any eureka nodes - possibly a security group issue?
javalangRuntimeException: Bad status: 404
at
comnetflixdiscoveryDiscoveryClientmakeRemoteCall(DiscoveryClientjava:833)
at
comnetflixdiscoveryDiscoveryClientmakeRemoteCall(DiscoveryClientjava:753)
at
comnetflixdiscoveryDiscoveryClientgetAndStoreFullRegistry(DiscoveryClientjava:615)
at comnetflixdiscoveryDiscoveryClientfetchRegistry(DiscoveryClientjava:562)
at comnetflixdiscoveryDiscoveryClientaccess$1200(DiscoveryClientjava:87)
at
comnetflixdiscoveryDiscoveryClient$CacheRefreshThreadrun(DiscoveryClientjava:1316)
at javautilTimerThreadmainLoop(Timerjava:555)
at javautilTimerThreadrun(Timerjava:505)
]]
[2015-12-10T13:37:11704+05:30] [AdminServer] [ERROR] []
[comnetflixdiscoveryDiscoveryClient] [tid: DiscoveryClient_CacheRefresher]
[userId: ] [ecid: 4c164384-726a-4c35-a306-2cb86fb2103a-00000003,0:6] [APP:
eureka-server-1137] DiscoveryClient_EUREKA/MACHINE_NAME - was unable to
refresh it's cache! status = Bad status: 404[[
javalangRuntimeException: Bad status: 404
at
comnetflixdiscoveryDiscoveryClientmakeRemoteCall(DiscoveryClientjava:833)
at
comnetflixdiscoveryDiscoveryClientmakeRemoteCall(DiscoveryClientjava:753)
at
comnetflixdiscoveryDiscoveryClientgetAndStoreFullRegistry(DiscoveryClientjava:615)
at comnetflixdiscoveryDiscoveryClientfetchRegistry(DiscoveryClientjava:562)
at comnetflixdiscoveryDiscoveryClientaccess$1200(DiscoveryClientjava:87)
at
comnetflixdiscoveryDiscoveryClient$CacheRefreshThreadrun(DiscoveryClientjava:1316)
at javautilTimerThreadmainLoop(Timerjava:555)
at javautilTimerThreadrun(Timerjava:505)
]]
NOTE: MACHINE_NAME is a name of my windows machine which cannot be
disclosed
—
Reply to this email directly or view it on GitHub
https://github.com/Netflix/eureka/issues/719.

I am closing this issue. Please, reopen it if you wish to continue this discussion.


CircleCI

i ran it on my fork: https://circleci.com/gh/dmitriiabramov/jest/tree/master and it seems to be working really well.
I also noticed that we ran our tests twice on travis (npm i triggers prepublish that runs 'npm test'). Fixed that too.
I wish i could parallelize node 4 and node 5 on circle, but it's not that simple there

Jest has more than just one node_modules folder – every example and package might have one. We should probably delete them all :)

@cpojer that was a workaround for circleci cache that gets restored into jest/node_modules and breaks the build after switching to another nvm version :)
Since Travis is working now i'll close this PR, but if we have any issues with it again we can always reopen it


Add url omitempty option for RepoListCommitsOptions fields.

This will allow cleaner urls when most RepoListCommitsOptions options are unchanged from their defaults.

LGTM


Login/Sinup Functional

Make the Login and Signup Page functional

Log in still not quite there, but there is now an entry page to choose between log in and sign up, you can change between the sign in and up pages, and the sign up page signs you in, updates general state, and brings you to the song listing page.

Boom!


moved config to another module

Pull Request (PR) Template
Does this solve an existing Issue?

General refactoring

Put an x inside the applicable checkboxes [ ].

Primary type of change:

[ ] New feature.
[ ] Bug fix.
[ x] Refactor.
[ ] Documentation.
[ ] Build/ devops.
[ ] Test.
[ ] Other.

Checklist:

[x ] I have actually ran this code locally to make sure it works.
[ x] I have pulled and merged the latest main into my feature branch.
[ ] I have ran the existing tests, and inspected areas that my code touches.
[ ] I have updated the tests to include my changes.
[ ] I have included updates to the documentation.
[ ] I have built the documentation files make html.

Describe the env you tested on: OS, Python version, shell type (e.g. cli, ide, notebook).

Contains breaking changes:

[ ] Yes (explain what functionality it breaks).
[x ] No

When I clone this, aiqc.destroy_db() with rebuild=True does not work for me.

When I run aiqc.setup
---------------------------------------------------------------------------
ImproperlyConfigured                      Traceback (most recent call last)
<ipython-input-14-6ca6abb31d35> in <module>
----> 1 aiqc.setup()

~/Desktop/cloner/aiqc/aiqc/__init__.py in setup()
     58 #==================================================
     59 def setup():
---> 60 	setup_database([	File, Tabular, Image,
     61                         Dataset,
     62                         Label, Feature,

~/Desktop/cloner/aiqc/aiqc/configuration.py in setup_database(tablesToCreate)
    334         create_folder()
    335         create_config()
--> 336         create_db(tablesToCreate)

~/Desktop/cloner/aiqc/aiqc/configuration.py in create_db(tablesToCreate)
    295                 print(f"\n=> Info - skipping table creation as the following tables already exist.{tables}\n")
    296         else:
--> 297                 db.create_tables(tablesToCreate)
    298                 tables = db.get_tables()
    299                 table_count = len(tables)

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in create_tables(self, models, **options)
   3329     def create_tables(self, models, **options):
   3330         for model in sort_models(models):
-> 3331             model.create_table(**options)
   3332
   3333     def drop_tables(self, models, **kwargs):

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in create_table(cls, safe, **options)
   6663             safe = options.pop('fail_silently')
   6664
-> 6665         if safe and not cls._schema.database.safe_create_index \
   6666            and cls.table_exists():
   6667             return

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in database(self)
   5569         db = self._database or self.model._meta.database
   5570         if db is None:
-> 5571             raise ImproperlyConfigured('database attribute does not appear to '
   5572                                        'be set on the model: %s' % self.model)
   5573         return db

ImproperlyConfigured: database attribute does not appear to be set on the model: <Model: Algorithm>

When I run aiqc.destroy_db(confirm=True, rebuild=True) I get this error:
---------------------------------------------------------------------------
OperationalError                          Traceback (most recent call last)
~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in execute_sql(self, sql, params, commit)
   3142             try:
-> 3143                 cursor.execute(sql, params or ())
   3144             except Exception:

OperationalError: disk I/O error

During handling of the above exception, another exception occurred:

OperationalError                          Traceback (most recent call last)
<ipython-input-11-06a1dd422084> in <module>
----> 1 aiqc.destroy_db(confirm=True, rebuild=True)

~/Desktop/cloner/aiqc/aiqc/__init__.py in destroy_db(confirm, rebuild)
     70
     71 def destroy_db(confirm:bool=False, rebuild:bool=False):
---> 72 	destroy_database([	File, Tabular, Image,
     73                         Dataset,
     74                         Label, Feature,

~/Desktop/cloner/aiqc/aiqc/configuration.py in destroy_database(tablesToCreate, confirm, rebuild)
    326
    327                 if (rebuild==True):
--> 328                         create_db(tablesToCreate)
    329         else:
    330                 print("\n=> Info - skipping destruction because `confirm` arg not set to boolean `True`.\n")

~/Desktop/cloner/aiqc/aiqc/configuration.py in create_db(tablesToCreate)
    295                 print(f"\n=> Info - skipping table creation as the following tables already exist.{tables}\n")
    296         else:
--> 297                 db.create_tables(tablesToCreate)
    298                 tables = db.get_tables()
    299                 table_count = len(tables)

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in create_tables(self, models, **options)
   3329     def create_tables(self, models, **options):
   3330         for model in sort_models(models):
-> 3331             model.create_table(**options)
   3332
   3333     def drop_tables(self, models, **kwargs):

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in create_table(cls, safe, **options)
   6668         if cls._meta.temporary:
   6669             options.setdefault('temporary', cls._meta.temporary)
-> 6670         cls._schema.create_all(safe, **options)
   6671
   6672     @classmethod

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in create_all(self, safe, **table_options)
   5780     def create_all(self, safe=True, **table_options):
   5781         self.create_sequences()
-> 5782         self.create_table(safe, **table_options)
   5783         self.create_indexes(safe=safe)
   5784

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in create_table(self, safe, **options)
   5635
   5636     def create_table(self, safe=True, **options):
-> 5637         self.database.execute(self._create_table(safe=safe, **options))
   5638
   5639     def _create_table_as(self, table_name, query, safe=True, **meta):

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in execute(self, query, commit, **context_options)
   3154         ctx = self.get_sql_context(**context_options)
   3155         sql, params = ctx.sql(query).query()
-> 3156         return self.execute_sql(sql, params, commit=commit)
   3157
   3158     def get_context_options(self):

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in execute_sql(self, sql, params, commit)
   3148             else:
   3149                 if commit and not self.in_transaction():
-> 3150                     self.commit()
   3151         return cursor
   3152

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in __exit__(self, exc_type, exc_value, traceback)
   2914             new_type = self.exceptions[exc_type.__name__]
   2915             exc_args = exc_value.args
-> 2916             reraise(new_type, new_type(exc_value, *exc_args), traceback)
   2917
   2918

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in reraise(tp, value, tb)
    188     def reraise(tp, value, tb=None):
    189         if value.__traceback__ is not tb:
--> 190             raise value.with_traceback(tb)
    191         raise value
    192

~/.pyenv/versions/3.8.7/envs/jupyterlab3/lib/python3.8/site-packages/peewee.py in execute_sql(self, sql, params, commit)
   3141             cursor = self.cursor(commit)
   3142             try:
-> 3143                 cursor.execute(sql, params or ())
   3144             except Exception:
   3145                 if self.autorollback and not self.in_transaction():

OperationalError: disk I/O error

@Bennylave @sahilgupta2105 in the morning I am going to reset the branch to June 23.
https://github.com/aiqc/AIQC/commits/main
I am glad the initiative was taken to merge it, but I need to review it first for edge cases.


Require Audience check in Tornjak backend

This is to help support IAM clients that may call several Tornjak backends (resource servers) connected to the same IAM - especially for the Tornjak Manager

closed by #397


CI is red on recent PRs

Can someone with guide's backend experience give a look, please?

Spellchecks are failing:
docs/engineering/workflows/taskforce-warzone.md
     1:3-1:12  warning  `Taskforce` is misspelt                  taskforce      retext-spell
    1:13-1:20  warning  `Warzone` is misspelt                    warzone        retext-spell
     4:1-4:10  warning  `Taskforce` is misspelt; did you mean ?  taskforce      retext-spell
    4:11-4:18  warning  `Warzone` is misspelt; did you mean ?    warzone        retext-spell
    6:37-6:47  warning  `hackathons` is misspelt                 hackathons     retext-spell
  31:40-31:53  warning  `recalibration` is misspelt              recalibration  retext-spell

Fixed dependency problem on #35


[CCFPCM-350] Track TODO's

CCFPCM-0350
Objective:
Create a ticket and annotate any TODO's in the codebase with the correct ticket number

Looks good. Note that the comment removed from pos-deposit.service.ts was left in error from an earlier commit.

Sorry, I'm not sure what you meant here?


[bug] css file not sorted correctly

Your Environment

Prettier version:  3.3.3
node version: 20
package manager: pnpm@9
IDE: VScode,

Describe the bug
import '../styles/main.css'; didn't sort into the bottom
import { AppProps } from 'next/app';
import Head from 'next/head';
import { createTheme, StyledEngineProvider, ThemeProvider } from '@mui/material';
import { HTTPError } from 'koajax';
import '../styles/main.css';
import { configure } from 'mobx';
import { enableStaticRendering, observer } from 'mobx-react';

import { isServer } from '../models/Base';

import { Footer } from '../components/Layout/Footer';
import { MainNavigator } from '../components/Layout/MainNavigator';

To Reproduce
when I format with prettier, I got this below
import { AppProps } from 'next/app';
import Head from 'next/head';
import { createTheme, StyledEngineProvider, ThemeProvider } from '@mui/material';
import { HTTPError } from 'koajax';

import '../styles/main.css';

import { configure } from 'mobx';
import { enableStaticRendering, observer } from 'mobx-react';

import { isServer } from '../models/Base';

import { Footer } from '../components/Layout/Footer';
import { MainNavigator } from '../components/Layout/MainNavigator';

Expected behavior
import { AppProps } from 'next/app';
import Head from 'next/head';
import { createTheme, StyledEngineProvider, ThemeProvider } from '@mui/material';
import { HTTPError } from 'koajax';
import { configure } from 'mobx';
import { enableStaticRendering, observer } from 'mobx-react';

import { isServer } from '../models/Base';

import { Footer } from '../components/Layout/Footer';
import { MainNavigator } from '../components/Layout/MainNavigator';

import '../styles/main.css';

Screenshots, code sample, etc

Configuration File (cat .prettierrc, prettier.config.js, .prettier.js)
.prettierrc
{
    "plugins": [
        "prettier-plugin-css-order",
        "@ianvs/prettier-plugin-sort-imports",
        "prettier-plugin-tailwindcss"
    ],
    "singleQuote": true,
    "trailingComma": "none",
    "arrowParens": "avoid",
    "tabWidth": 2,
    "printWidth": 100,
    "importOrder": [
        "^react$",
        "^next\/?",
        "^@mui",
        "<THIRD_PARTY_MODULES>",
        "",
        "models\/",
        "api\/",
        "",
        "components\/",
        "^[.]",
        "",
        "^(?!.*[.]css$)[./].*$",
        ".css$",
        ""
    ]
}

Error log

Contribute to @ianvs/prettier-plugin-sort-imports

[ ] I'm willing to fix this bug 🥇

I've retrieved a couple of issues over at tailwind that conflict with the sort plugin, but they're all marked as fixed, and I'm not quite sure if the issue is related to regular expressions or the plugin itself

Hey @Soecka this is a recurring confusion, (and I think we need something more than documentation to explain it @IanVS)
In short, reorganizing CSS file-imports runs the risk of changing the CSS cascade which could change the look of your code. (Unless you have very specific conventions in how you write CSS).
And as a more general issue, imports without imported-names are by-definition being imported to cause a side-effect (whether it’s hooking in polyfills into global scope or something else). So it’s unsafe for this plugin to move stuff around an import that doesn’t import any variables (all imports can have side-effects, but compromising was necessary to make any tool like this one work at all).
Other similar tickets:

#157
#181
#186
#187

Thanks for the reply, I assume it was a tailwind plugin conflict, should have searched for related issues in this repository first!
Recheck the definition of side effect import
Thanks again

Thanks @fbartho.  I've pinned this issue, hopefully that will help a little.  We can also maybe update the issue template to mention this as well, because indeed it seems to trip up a lot of people.


[KOGITO-1852]: Update check for internal registry

https://issues.redhat.com/browse/KOGITO-1852
https://github.com/kiegroup/kogito-cloud-operator/pull/304 was not enough
we need also to confirm that the image pull secret is correctly attached to the pod...
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster

Wait. Is this a WIP?

@ricardozanini no worry, almost done.
@Sgitario it is totally because sometimes Openshift is too slow ...
The first fix was checking that the image pull secret was created into the namespace before creating the operator. But going further in the tests, into the Kogito Openshift, we noticed that sometimes it happens that, even if the secret is created, it is not attached to the pod when operator pod is created.
The idea of the fix is really to make sure that the secret is attached. If it is not, the pod is switched into ErrImagePull and we get a authentication required, and so the scenario will fail ...
So we have 2 things now that are done:

We wait for the image pull secret to be created.
Waiting for Kogito operator, we check the secret is attached to the pod.

workaround #2 will just almost never happen. It is mostly for really low clusters...

tested on Kogito Openshift. Took 4 hours but no ErrImagePull :)

Is this PR ready?


Changed the method of fetching item contents to innerText

We're searching for text, so it made sense to use the innerText instead of innerHTML.
Also, when using the library I came across the fact that, when you have, for example, an item with ampersand somewhere in the list, you'd find it when searching for "amp", because innerHTML returns an html-encoded &

Hmm, the thing is that we are not sure that the content is just text. It could be an image as well.
I know images and other HTML does not make sense when it comes to searching and sorting, but for showing e.g. image content in lists are innerHTML necessary.
But the amp-thing is disturbing, thought. :/


TOC request

I'm glad tag and categories were added to this theme, it enriches the reader's experience and completes the ongoing usability of this theme, while the rainbow banner looks great.
It would be great to see toc added in the future as well.

Thanks! I may implement this feature in the future.

Done! Check it out by updating the theme! 🎉


Configurable location for generated Festival Wristbands

Authorino Festival Wristbands are right now always passed back to client in a fixed HTTP response header X-Ext-Auth-Wristband. It would be nice have at least the name of this header configurable, but maybe extending the implementation to support Set-Cookie response headers as well.

Closing this as some configurability has been added already with #138.
About support for Set-Cookie HTTP header, better re-opening it in a separate issue if it ever becomes an actual RFE.


chore(deps): bump nodejs

Applied mkpr on undefined
sed s/16.7.[0-2]/16.8.0/

:tada: This PR is included in version 1.29.23 :tada:
The release is available on:

v1.29.23
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make errors descriptions more visually clear

You've all heard about syntax coloring it is something to make it easier for kinder-gardeners to do programming.
See it is quite hard to distinguish logical/context parts of the message and identify the problems from such output:

It would be good if error descriptions contained some highlighting of important parts like particular error messages (underlined on the picture), and maybe included code/signature highlighting as well.

@whitecolor thank you for opening this issue. Can you show me more about the context of your feature request? Is this in the output window?
If you could share something simple I could recreate what you are seeing that would be appreciated.

@waderyan Yes this is a popup that appears when I the mouse is over the errored code (undelined with red).

There is even some code highlighting in the popup =) But it is obviously is not enough.

@dbaeumer do you have an opinion on this?
I agree with OP that it would be good to do some highlighting in the popover view. Who would be best to send this to on the team? Is this strictly a UX issue?

Agree that the error message needs to be better understandable. Being able to colorize that however need some support from the tsserver. We can't simple take the error message and run a tokenizer over it. Will very likely produce a suboptimal coloring.
So in a first step we need to ask the tsserver to produce 'strucutured' error message and then we can try to improve the coloring and rendering on it. In the above example I could even imagine some sort of folding.

@dbaeumer thank you. Sending to TS.
Note there is a closely related issue here - https://github.com/Microsoft/vscode/issues/13070#event-808268456

This issue was moved to Microsoft/TypeScript#11334


Setting sesssiontimezone to UTC

This probably fixes the issue in the integration test.  Need too figure out how to add similar statements to production pools if that is indeed the fix

We could just make it part of the connection setup like the office id/user session. Yeah, it's a bit silly maybe, but it would make sure it was done.


Kf 4075 feat unit tests for jupyter scipy rock

Unit tests for Jupyter Scipy ROCK.
These tests verify that all artefacts are in place, that imports can be made in mounted/executed Python script, and that Jupyter Notebook Server is accessible.
Summary of changes:
- Tests for testing ROCK integrity.
- Commented out some imports in import test as TO-DOs to be addressed during integation stage. Those might not be required to be tested as imports. Integration should verify this.
- Changed /home/jovyan to be owned by daemon
- Added notebook server access test.
- Added unit test to test artifacts, imports, and access.
- Added tox.ini
Testing:
tox -e unit

i have only approve the rockcraft.yaml. I let other for the other files


Add .gitignore to aiken new project generation

What is your idea? Provide a use case.
When you run aiken new the project comes with a free .gitignore:
build
assets

aiken.lock

Why is it a good idea?
Most people don't need to include all their dependencies in the /build directory in their repo. /assets and aiken.lock are also often superfluous.
What is the current alternative and why is it not good enough?
You could add it by hand.
But I think most people are gonna want /build and /assets and aiken.lock ignored from their repo.

Cool. I'm curious why you'd want to keep it. I've seen people go both ways with cargo.lock and I've never given it the time to understand why.

@MitchTurner I believe it's related to reproducible builds or something, I've always committed the lock file for everything so I was under the impression it was a settled matter.


[1.0.0.rc1] validation: add linux resource check

Signed-off-by: Ma Shimiao<EMAIL_ADDRESS>Backported to v1.0.0.rc1 from 9cad4d8 #195 (cherry-pick applied
cleanly).
Signed-off-by: W. Trevor King<EMAIL_ADDRESS>

LGTM


It could be much more ratios and resolutions for SDXL

maybe it's important for this project.
previously my custom node also have only SDXL resolutions that stated in ComfyUI SDXL examples, but it turns out it much more than just that.
This is the full list of accepted aspect ratios and pixel size is derived from this SDXL paper :
https://arxiv.org/abs/2307.01952 https://arxiv.org/pdf/2307.01952.pdf
Page 17 "We use the following image resolutions for mixed-aspect ratio finetuning as described in Section 2.3"

Here's what I add
https://github.com/marhensa/sdxl-recommended-res-calc/issues/2
Maybe you could implement it in your https://github.com/evanspearman/ComfyMath/blob/main/src/comfymath/graphics.py

Thank you, this helps me a lot

I've added extended versions of these nodes that incorporate the resolutions in the paper. The original nodes have been left unchanged.


with and void* cast

Is there any way to have with's var be the correct type?
With -Wall is outputs a lot of warnings, the example show usage with an fd, maybe a with_fd could be created?
#define with(var, close, ...)					\
        let var = (__VA_ARGS__); \
        defer((close)(var))

Compiles without warnings

Is there any way to have with's var be the correct type?
With -Wall is outputs a lot of warnings, the example show usage with an fd, maybe a with_fd could be created?

This is certainly queer. with already uses typeof on GCC and C23, so it
should be fine. I cannot reproduce with GCC 14.2.1, what's your
compiler/version?

I get the following warnings
daemon.c:38:5: warning: cast to 'void *' from smaller integer type 'int' [-Wint-to-void-pointer-cast]
    with (fd, close, open(p, O_WRONLY)) {
    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
./pretty.h:136:26: note: expanded from macro 'with'
        for (void *var = (void*) (__VA_ARGS__),                 \
                         ^~~~~~~~~~~~~~~~~~~~~
daemon.c:38:11: warning: incompatible pointer to integer conversion passing 'void *' to parameter of type 'int' [-Wint-conversion]
    with (fd, close, open(p, O_WRONLY)) {
          ^~
./pretty.h:139:22: note: expanded from macro 'with'
             (close)(var), pretty_with_flag = (void*) 0)
                     ^~~
/usr/aarch64-linux-android/bin/../sysroot/usr/include/unistd.h:268:15: note: passing argument to parameter '__fd' here
int close(int __fd);
              ^
daemon.c:39:16: warning: comparison between pointer and integer ('void *' and 'int') [-Wpointer-integer-compare]
        if (fd == -1) {

I'm cross compiling for android with clang
$ /usr/aarch64-linux-android/bin/clang --version
Android (8490178, based on r450784d) clang version 14.0.6 (https://android.googlesource.com/toolchain/llvm-project 4c603efb0cca074e9238af8b4106c30add4418f6)
Target: aarch64-unknown-linux-android23
Thread model: posix
InstalledDir: /usr/aarch64-linux-android/bin

Oh I see, you're using Clang. Check whether
ffadd8c2b98f82681e7b4dc6c94ab2ed332b28ec fixes it for you.

Yes it works perfectly, thanks a lot


README.md: Remove to docs hosted on coreos.com

As these docs are outdated, this patch removes the link in the
README.md.

lgtm

Seeing this again while writing the Changelog: Why are they still online when we don't even link to them anymore?


Update docker demo to show the dashboard

need to add mysql container in the composer

duplicated with #151


[mono] Add Wasm sample using mono runtime

Summary
This PR looks to port over the wasm sample from dotnet/runtime to highlight the concept of building applications on top of the mono runtime.
There are couple dependencies such as dotnet-serve, a working browser, and dotnet sdk to run the sample with dotnet publish.
The sample will depend on a NuGet Package containing the files that build the app and embed the mono runtime into the app, and the code that will build that package can be found here https://github.com/dotnet/runtime/pull/44749

This PR will need to wait for the .NET SDK version where https://github.com/dotnet/runtime/pull/44749 gets applied. In doing so, the README can update to include the proper WasmAppBuilder task changes.

The relevant PRs that contain the WasmAppBuilder task changes have gone through, and changes have been made to the .csprojs and README.mds to reflect that.


[Enhancement]: partition plan set a stable pre-creation count as default

What would you like to be added?
partition plan set a stable pre-creation count as default.
such as

default  15 pre-creation count for daily parition
default  7 pre-creation count for weely/montly partition
default 3 pre-creation count for yearly partition

more, tip user why should we use a stable pre-creation strategy
Why is this needed?
product level partition maintain

fixed: https://github.com/oceanbase/odc/issues/1881#issuecomment-1994320009


examples for SO protein feature search

to go under the search box in the Protein motifs tab (protein domains, features and motifs):
"Examples: short motifs such as NLS or KEN box, signal sequence, cleaved region, helix"
(tangent from #1720)

Thanks Midori.
I've added the code and config in my local copy.  It will be released when I commit the other query builder changes.
It's configured like this in the main website config file:
           {
             "id": "SO_protein",
             "displayName": "protein motifs",
             "nodeType": "ontology",
             "ontologyName": "SO-protein",
             "topDescription": [
               {
                 "text": "Retrieve protein-coding genes based on features annotated using the protein sequence feature branch of the Sequence Ontology (SO)"
               }
             ],
             "extraDescription": "Examples: short motifs such as NLS or KEN box, signal sequence, cleaved region, helix"
           }

and looks like:

Looks good; thanks :)


Bump js-cookie version to 2.2.1

I hope this doesn't break anything.

Oops..

Weird, in circleci test-e2e-ssr failed, but in my machine is working fine. Here is the output.
$ jest test/module.test
 PASS  test/module.test.js (92.716s)
  basic
    √ sets SEO metadata properly (119ms)
    √ / contains EN text, link to /fr/ & link /about-us (23ms)
    √ /fr contains FR text, link to / & link to /fr/a-propos (15ms)
    √ /about-us contains EN text, link to /fr/a-propos & link / (16ms)
    √ /fr/a-propos contains FR text, link to /about-us & link to /fr/ (17ms)
    √ /fr/notlocalized contains FR text (12ms)
    √ /notlocalized & /fr/fr/notlocalized return 404 (12ms)
    √ route specifies options with non-supported locale (15ms)
    √ navigates to child route with nameless parent and checks path to other locale (70ms)
    √ navigates to dynamic child route and checks path to other locale (66ms)
    √ /dynamicNested/1/2/3 contains link to /fr/imbrication-dynamique/1/2/3 (25ms)
    √ /fr/imbrication-dynamique/1/2/3 contains link to /dynamicNested/1/2/3 (16ms)
    √ localePath returns correct path (55ms)
    √ switchLocalePath returns correct path (53ms)
    √ getRouteBaseName returns correct name (58ms)
    √ localePath, switchLocalePath, getRouteBaseName works from a middleware (11ms)
    √ redirects to existing route (56ms)
    √ fallbacks to default locale with invalid locale cookie (13ms)
    √ registers message using vueI18nLoader (12ms)
    posts
      √ /posts contains EN text, link to /fr/articles/ & link to /posts/my-post (22ms)
      √ /posts/my-post contains EN text, link to /fr/articles/mon-article & link to /posts/ (13ms)
      √ /fr/articles contains FR text, link to /posts/ & link to /fr/articles/mon-article (14ms)
      √ /fr/articles/mon-article contains FR text, link to /posts/my-post & link to /fr/articles/ (17ms)
    store
      √ injects $i18n in store (187ms)
      √ syncs i18n locale and messages (75ms)
  lazy loading
    √ shows fallback string (37ms)
  with empty configuration
    √ does not remove all routes (90ms)
  prefix_and_default strategy
    √ default locale routes / and /en exist (39ms)
    √ non-default locale route /fr exists (8ms)
    √ canonical SEO link is added to prefixed default locale (16ms)
    √ canonical SEO link is not added to non-prefixed default locale (14ms)
  no_prefix strategy
    √ sets SEO metadata properly (56ms)
    √ / contains EN text & link /about (20ms)
    √ /about contains EN text & link / (14ms)
    √ /fr/ returns 404 (6ms)
    √ localePath returns correct path (96ms)
    √ localePath with non-current locale triggers warning (57ms)
    √ fallbacks to default locale with invalid locale cookie (11ms)
  no_prefix strategy + differentDomains
    √ triggers warning (5625ms)
  invalid strategy
    √ triggers error on building (8381ms)
  dynamic route
    √ can access catch-all route for every locale (114ms)
  hash mode
    √ localePath returns correct path (without hash) (168ms)
  with router base
    √ localePath returns correct path (132ms)
  differentDomains enabled
    √ matches domain's locale (en) (43ms)
    √ matches domain's locale (fr) (13ms)
  parsePages disabled
    √ navigates to route with paths defined in pages option (93ms)
    √ navigates to route with paths disabled in pages option (16ms)

----------------|----------|----------|----------|----------|-------------------|
File            |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------------|----------|----------|----------|----------|-------------------|
All files       |    99.22 |    92.77 |      100 |    99.17 |                   |
 src            |      100 |    83.33 |      100 |      100 |                   |
  index.js      |      100 |    83.33 |      100 |      100 |     32,58,109,118 |
 src/helpers    |    98.81 |    96.61 |      100 |    98.72 |                   |
  components.js |       95 |      100 |      100 |       95 |                46 |
  routes.js     |      100 |    96.08 |      100 |      100 |             20,80 |
----------------|----------|----------|----------|----------|-------------------|
Test Suites: 1 passed, 1 total
Tests:       47 passed, 47 total
Snapshots:   0 total
Time:        93.093s
Ran all test suites matching /test\\module.test/i.
Done in 94.42s.

However, i got failure test when running browser.test.js. Here is the output.
$ jest test/browser.test
  console.info node_modules/tib/lib/tib-browser.js:201
    tib: Static server started on http://localhost:54551

 FAIL  test/browser.test.js (31.548s)
  puppeteer/core
    × navigate with SPA (2502ms)
    × APIs in app context work after SPA navigation (167ms)
  puppeteer/core (generate)
    × navigate to non-default locale (522ms)
  puppeteer/core (SPA with router in hash mode)
    √ navigates directly to page with trailing slash (521ms)
    √ navigates directly to page with query (173ms)

  ● puppeteer/core › navigate with SPA

    Evaluation failed: TypeError: Cannot read property 'then' of undefined

      at eval (eval at <anonymous> (:2:17), <anonymous>:1:39)
      at __puppeteer_evaluation_script__:2:60
      at ExecutionContext._evaluateInternal (node_modules/puppeteer-core/lib/ExecutionContext.js:122:13)
        -- ASYNC --
      at ExecutionContext.<anonymous> (node_modules/puppeteer-core/lib/helper.js:111:15)
      at DOMWorld.evaluate (node_modules/puppeteer-core/lib/DOMWorld.js:112:20)
        -- ASYNC --
      at Frame.<anonymous> (node_modules/puppeteer-core/lib/helper.js:111:15)
      at Page.evaluate (node_modules/puppeteer-core/lib/Page.js:863:43)
      at Page.<anonymous> (node_modules/puppeteer-core/lib/helper.js:112:23)
      at Proxy.runScript (node_modules/tib/lib/tib-core.js:49:22)
      at Proxy.runAsyncScript (node_modules/tib/lib/tib-webpage.js:86:17)
      at Proxy.navigate (test/browser.test.js:13:16)
      at Object.test (test/browser.test.js:52:16)

  ● puppeteer/core › APIs in app context work after SPA navigation

    Evaluation failed: TypeError: Cannot read property 'then' of undefined

      at eval (eval at <anonymous> (:2:17), <anonymous>:1:39)
      at __puppeteer_evaluation_script__:2:60
      at ExecutionContext._evaluateInternal (node_modules/puppeteer-core/lib/ExecutionContext.js:122:13)
        -- ASYNC --
      at ExecutionContext.<anonymous> (node_modules/puppeteer-core/lib/helper.js:111:15)
      at DOMWorld.evaluate (node_modules/puppeteer-core/lib/DOMWorld.js:112:20)
        -- ASYNC --
      at Frame.<anonymous> (node_modules/puppeteer-core/lib/helper.js:111:15)
      at Page.evaluate (node_modules/puppeteer-core/lib/Page.js:863:43)
      at Page.<anonymous> (node_modules/puppeteer-core/lib/helper.js:112:23)
      at Proxy.runScript (node_modules/tib/lib/tib-core.js:49:22)
      at Proxy.runAsyncScript (node_modules/tib/lib/tib-webpage.js:86:17)
      at Proxy.navigate (test/browser.test.js:13:16)
      at Object.test (test/browser.test.js:64:16)

  ● puppeteer/core (generate) › navigate to non-default locale

    Evaluation failed: TypeError: Cannot read property 'then' of undefined

      at eval (eval at <anonymous> (:2:17), <anonymous>:1:39)
      at __puppeteer_evaluation_script__:2:60
      at ExecutionContext._evaluateInternal (node_modules/puppeteer-core/lib/ExecutionContext.js:122:13)
        -- ASYNC --
      at ExecutionContext.<anonymous> (node_modules/puppeteer-core/lib/helper.js:111:15)
      at DOMWorld.evaluate (node_modules/puppeteer-core/lib/DOMWorld.js:112:20)
        -- ASYNC --
      at Frame.<anonymous> (node_modules/puppeteer-core/lib/helper.js:111:15)
      at Page.evaluate (node_modules/puppeteer-core/lib/Page.js:863:43)
      at Page.<anonymous> (node_modules/puppeteer-core/lib/helper.js:112:23)
      at Proxy.runScript (node_modules/tib/lib/tib-core.js:49:22)
      at Proxy.runAsyncScript (node_modules/tib/lib/tib-webpage.js:86:17)
      at Proxy.navigate (test/browser.test.js:13:16)
      at Object.test (test/browser.test.js:111:16)

----------------|----------|----------|----------|----------|-------------------|
File            |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------------|----------|----------|----------|----------|-------------------|
All files       |    85.94 |    66.27 |      100 |    85.12 |                   |
 src            |    90.91 |    54.17 |      100 |     90.7 |                   |
  index.js      |    90.91 |    54.17 |      100 |     90.7 |       39,40,76,78 |
 src/helpers    |    83.33 |    71.19 |      100 |    82.05 |                   |
  components.js |       95 |      100 |      100 |       95 |                46 |
  routes.js     |    79.69 |    66.67 |      100 |    77.59 |... 99,103,104,105 |
----------------|----------|----------|----------|----------|-------------------|
Test Suites: 1 failed, 1 total
Tests:       3 failed, 2 passed, 5 total
Snapshots:   0 total
Time:        31.875s
Ran all test suites matching /test\\browser.test/i.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
npm ERR! Test failed.  See above for more details.

I've updated yarn.lock and test running fine on my machine, and now I have no idea why the test is failed on circle ci.

No need to create separate PR for this. This is just a minor dependency update and is normally handled by automatic lock file maintenance PRs that happen every Friday (I can also retry the one that is currently open).

No, sorry, that's not entirely true. renovate only handles devDependencies.
For js-cookie, since the version is specified as a range (^2.2.0), it doesn't prevent anyone from using the latest version of that dependency. Is it really necessary to manually change it to require at least 2.2.1?

Although I see the point that explicitly upping it will force people to use the latest version which might be worth it if there is a security fix there.

BTW. There might be some issues with circleci. Will try to find out what's happening there.

@rchl Thanks for the response. This PR is not required though, I just think it would be nice if the library has the latest version of its dependencies. Feel free to close the PR, if you think it is not the important things that need to be working on right now. 👍

That's OK, I'll merge it since it seems to include security-related fix. Just fixing the testing issue first. Thanks. :)

Can you rebase your change on master? It should pass testing next time.

@rchl Now everything works fine. 👍

Thanks!


increased tls timeout to get test passing on arm.

[X] Documentation added (if applicable)
[X] Branch rebased on top of current master (git pull --rebase origin master)
[X] You have certified that the contribution is your original work and that you license the work to the project under the MIT license

Resolves:
TLS TImeout causing errors in arm build testing.
/cc @nats-io/core

Changes Unknown when pulling 56f1f8a52974c44c03734ece6e3ff060308794f2 on increase_tls_timeout_for_arm into ** on master**.

Changes Unknown when pulling cfe6d58792db16c0eb55f24409830a99ef5470bb on increase_tls_timeout_for_arm into ** on master**.


8266 refresh the connection every 70 minutes

I can connect to GCP with 8266 using this library no problem. However approximately every 70 minutes (one hour and 10 minutes) I see that there is a new connection open.
I monitored the serial monitor and looks like every 70 minutes, it acts like wifi connection was lost and the device re-connects to the WiFi thus recalls the connect functions. The logs are pasted below.
This happens regardless of the RSSI and approximately every 70 minutes. To me there is a setting in the mqtt header that refreshes the JWT every 70 minutes. Can anyone advise?
checking wifi...
connecting...Refreshing JWT
connected!

Set the expiration of the token when you create the JWT and the password should expire within 20 minutes of the expiration time.

Aha! thank you very much.
I increased it to 2 hours and now I see a new connection request every 130 minutes. I would like to increase it even higher - perhaps only one refresh per day. Do you see a problem with that? Why did you want to set the refresh timer to 1h to begin with?

The reason you set the refresh time (expiration of the JWT) is to reduce the impact of a credential being compromised (e.g. CSRF on the web). The maximum allowable duration for a token to be valid is 24-hours and there are advantages to extending the duration that tokens are valid (e.g. less power used for recalculating JWT) that come with trade-offs (impact of abuse in case of credential leak). One hour is a pretty typical expiration duration when using JWTs but there isn't any specific reason I chose that duration.  Also of note, there is 20 minutes of "clock drift" that is allowed on the JWT, so your expiration won't be exactly the difference between the issue time and expiration.

Very good. Thanks again for detailed response.


⚠ Samsung has degraded performance

In 2e18958, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2878 ms

Resolved: Samsung performance has improved in 49c9d64.


Set default locationName in GUI to avoid various issues when not set

This PR is to address issue #9 by setting the current directory into "Location" field in the GUI.

I knew there was something I had forgotten to do! LGTM, thanks!

I knew there was something I had forgotten to do! LGTM, thanks!


Remove Coffeescript

BREAKING CHANGES: Removed CoffeeScript support; .coffee files will no longer load
BREAKING CHANGES: Removed es2015.js file; import index.js file instead

Why remove this and break lots of plugins in the ecosystem? I don't see this tied to an issue with discussion or anything like that.

A few thoughts:
Version 10.0.5 still supports coffeescript. For those still using hubot and coffeescripts, they should set the hubot peer dependency to version 10.
I haven’t seen any  GitHub issues about breaking plugins. I’d love to know how many people are still using hubot. And hubot with coffeescripts.
Removing coffeescript was on the roadmap issue #1658
Some background:
Development on hubot stalled sometime around 2017. I asked to be a maintainer and was added to the repo this year (at least I think it was this year). My objective was to modernize the code base, and try to pick up where the other maintainers left off with the goals in the evolution repo.
in my research, I found that almost every repo I found was last updated many years ago. Leaving me with the impression that the hubot ecosystem has gone stale.
 Specifically reducing dependencies. I still get dependabot notifications for the yo generator repo. Gotta figure out what to do with that.
I think there’s better adoption for ES6, which is my motivation to modernize hubot and evolve it to a new vision having been left stale for so long.
With all those points in mind, I removed coffeescript in v11 as I try to incorporate some of the goals initially defined in the evolution repo and figure out where hubot now fits in todays universe.
Are you still using hubot? With coffeescripts? I’d love to hear your story.


Can't get D-Settlement console to run

I have placed the D-Settlement console in a folder (which is written in an *.env file), but everytime I try to execute the D-Settlement model, a window opens with the following error message:
"License not granted
Module(s) not found in the license file or borrowed modules, or license file not found.
Batch processing not allowed."
How to solve this?

Did you install the consoles as was described in https://deltares.github.io/GEOLib/latest/user/setup.html#console-usage Iseection "Consoles") and did you obtain a license from Deltares and installed it?

Yes I did. Yesterday I contacted Deltares and they give me the license file and I install it using "Add License File" program but it still gives me an error during executing the model using geolib. This time it gives the following error "Output file generated but parsing of Tutorial.sld failed"

The good news is that your system is working now. It seems however that the calculation was not successful. Check if the model that you created with GEOLib is correct. Did you study the tutorial (https://deltares.github.io/GEOLib/latest/community/tutorial_dsettlement.html)?

I did. In fact, I followed the tutorial step-by-step as in the link that you shared

I see that the inputfile (sli) is created with GEOLib version 0.2.3 and the output file with D-Settlement version <IP_ADDRESS>213.
Currently GEOLib works only with DSettlementConsole.exe version 19.1 (build 1.34009), see https://deltares.github.io/GEOLib/latest/user/setup.html.
At the end of this year we will release DSettlement 23.2 and update the GEOLib code to work with that version.
I am not sure about GEOLib version 0.2.3 but when you get the latest changes from the master you will get GEOLib version 1.3.1. If you follow the tutorial with the current master and run with DSettlementConsole.exe version <IP_ADDRESS>009 it should succeed. Please notice that for DSettlementConsole.exe version 19.1 you need a different license file than for the regular DSettlement.exe. Your company (as I see in the sld file) should have this license, since it participated in the GEOLib project. If you do not have DSettlementConsole.exe version 19.1 yet, you can download it from https://download.deltares.nl/geolib

Hi @zantDelta, which license file that you mean? Last week I contacted Deltares software support, and they give me 100389_S_00505696b3cd.lic but it still doesn't work. The license file that I got from https://download.deltares.nl/en/license-manager also doesn't work either. Upgrading to geolib v1.3.1 doesn't help either, so it must be really the problem of license.

Please ask Deltares software support for a license of DSettlementConsole.exe version 19.1

I presume the problem was solved, so I close this issue


Error: Undefined method "id" for nil:NilClass

Error: Undefined method "id" for nil:NilClass on creating a new resource

Logs for www-cronrobot
Started GET "/dashboard/resources/new" for <IP_ADDRESS> at 2021-09-11 16:02:04 +0000
Cannot render console from <IP_ADDRESS>! Allowed networks: <IP_ADDRESS>/<IP_ADDRESS>, ::1
Processing by Dashboard::ResourcesController#new as HTML
  User Load (0.2ms)  SELECT "users".* FROM "users" WHERE "users"."uid" = ? LIMIT ?  [["uid", "google-oauth2|109354136664780264056"], ["LIMIT", 1]]
  ↳ app/controllers/concerns/www_secured.rb:10:in `logged_in_using_omniauth?'
  Project Load (0.2ms)  SELECT "projects".* FROM "projects" WHERE "projects"."user_id" = ? ORDER BY "projects"."id" ASC LIMIT ?  [["user_id", 2], ["LIMIT", 1]]
  ↳ app/controllers/dashboard_controller.rb:15:in `ensure_project_selected'
  Project Exists? (0.3ms)  SELECT 1 AS one FROM "projects" WHERE "projects"."id" = ? LIMIT ?  [["id", 2], ["LIMIT", 1]]
  ↳ app/controllers/dashboard_controller.rb:29:in `ensure_project_selected'
  Project Load (0.1ms)  SELECT "projects".* FROM "projects" WHERE "projects"."id" = ? LIMIT ?  [["id", 2], ["LIMIT", 1]]
  ↳ app/controllers/dashboard_controller.rb:33:in `ensure_project_selected'
  Project Exists? (0.2ms)  SELECT 1 AS one FROM "projects" WHERE "projects"."user_id" = ? AND "projects"."id" = ? LIMIT ?  [["user_id", 2], ["id", 2], ["LIMIT", 1]]
  ↳ app/models/user.rb:17:in `can_access_project'
  Rendering layout layouts/dashboard.html.erb
  Rendering dashboard/resources/new.html.erb within layouts/dashboard
  Rendered dashboard/resources/new.html.erb within layouts/dashboard (Duration: 3.0ms | Allocations: 371)
[Webpacker] Everything's up-to-date. Nothing to do
  Rendered layouts/_flash_errors.html.erb (Duration: 0.2ms | Allocations: 24)
  Rendered layout layouts/dashboard.html.erb (Duration: 15.1ms | Allocations: 3791)
Completed 200 OK in 42ms (Views: 16.0ms | ActiveRecord: 1.2ms | Allocations: 6995)

@kb24x7 try to take the change on master https://github.com/cronrobot/cronrobot/commit/d32af3bb2baadc74ea1f44ed9ffcff21828064e1

@kb24x7
for the allowed error, to fix it add this in config/environments/development config.web_console.whitelisted_ips = %w( <IP_ADDRESS>/0 ::/0 )


Resolves #966

Fixing typo in Example 2 - DHSP to DHCP.
https://github.com/MicrosoftDocs/windows-powershell-docs/issues/966

@AndreaBarr Review needed for minor typo fix. Thanks.


Error in hive_flutter package

Steps to Reproduce
Cloned hive repo and executed pub get
These two files seems to be with errors:

hive_aes_native_cbc_cipher.dart
lines 57 and 77
hive_aes_native_gcm_cipher.dart same lines
The errors:
'HiveAesNativeGcmCipher.decrypt' ('Future<int> Function(Uint8List, int, int, Uint8List, int)') isn't a valid override of 'HiveAesCipher.decrypt' ('int Function(Uint8List, int, int, Uint8List, int)')
'HiveAesNativeGcmCipher.encrypt' ('Future<int> Function(Uint8List, int, int, Uint8List, int)') isn't a valid override of 'HiveAesCipher.encrypt' ('int Function(Uint8List, int, int, Uint8List, int)')

Maybe related
flutter packages pub run build_runner build
[INFO] Generating build script...
[INFO] Generating build script completed, took 273ms
[INFO] Precompiling build script......
[WARNING] ../flutter/.pub-cache/hosted/pub.dartlang.org/hive-2.2.3/lib/src/backend/vm/storage_backend_vm.dart:24:29: Error: Expected a class member, but got '6'.
final ReadWriteSync _sync;6
^
[INFO] Precompiling build script... completed, took 735ms
[SEVERE] Failed to precompile build script .dart_tool/build/entrypoint/build.dart.
This is likely caused by a misconfigured builder definition.
pub finished with exit code 78
Though i checked latest code and there is no 6 after that line in the file... something really strange is happening...

Cannot reproduce with version 2.2.2

Why is there a random 6?
Hauahauah

@kamimark maybe it's related to some changes that where committed between 2.2.2 and now. I've cloned the master branch and not the 2.2.3 on pub


The meaning of "normMean" of the segstats.py output

Hey. I am running the segstats.py to get the vol_seg.stats after running the segmentation only module. According to the output, there is a column "normMean". I wonder what is the meaning of this column specifically. Thanks.

Gotta you! Thanks for clarifying the difference of that to freesurfer output.

That is not really a difference to the freesurfer output. Freesurfer has the same output in mri_segstats in stats/aseg.stats or similar.

A follow-up question about the "--gt_nocc" in generate_hdf5.py. I am currently trying to modify the FastSurferCNN so I can train using my own data (seg was labeled using WM,GM, CSF etc.). Since I did not have the ground truth from Freesurfer-like output, is it reasonable to set the  gt_noncc as None?


shade everything but us in the shaded nio jar

Update our shaded jar example to actually shade everything except what it needs (google-cloud-nio). This allows this jar to be used with recent versions of Spark that also bring in a conflicting version of netty.

I ran the nio example and it still works fine:
$ java -cp google-cloud-contrib/google-cloud-nio/target/google-cloud-nio-0.4.1-SNAPSHOT-shaded.jar:google-cloud-contrib/google-cloud-nio-examples/target/google-cloud-nio-examples-0.4.1-SNAPSHOT.jar com.google.cloud.nio.examples.ListFilesystems
Installed filesystem providers:
file
jar
gs

cc: @jart @mziccard

Coverage remained the same at 84.568% when pulling e20811a915d73301fd52c80f0d3ca8ba38beb96a on jean-philippe-martin:shade_netty into 307f9f0457817d094734c3d1dd929b82b49883a2 on GoogleCloudPlatform:master.

@jart is this a LGTM for you?
@jean-philippe-martin While looking at this PR I noticed other dependency-related issues:

org.mockito:mockito-core dependency should have test scope
google-cloud-nio now depends on the whole google-cloud, we should depend just on google-cloud-storage
To make 2. effective we need to also remove google-cloud dependency from google-cloud-contrib main pom.xml, we can let each child module pull its own dependencies.

Does this make sense to you? We can take care in a different PR, I just wanted to bring this up.

LGTM

@mziccard that makes sense, let me work on those in a separate PR. I've filed #1332.

What is the name of the artifact that's going to have the shading?

It's google-cloud-nio-0.4.1-shaded.jar