Express v5 and body-parser v2 not compatible with serverless-express

Hello! It seems like express v5 and its bundled body-parser v2 do not work with serverless-express. The body-parser dependency has changed its internals to use "on-finished" dependency to check if the body is parsed already. It does so by checking if the request stream has "completed" set to true. The requests from serverless-express are always "completed" when it reaches the express app, i.e. the body-parser thinks the body is already parsed and skips parsing e.g. JSON content.
Code that always sets request to completed:
https://github.com/vendia/serverless-express/blob/f3f9c50f6dfcdbaf84af6ca64e77af1ba8b34bf0/src/request.js#L20
The "on-finished" code that checks if the request body is already parsed:
https://github.com/jshttp/on-finished/blob/1111fe8e913debaf3da9bd4f6bda216ef36097fa/index.js#L76
Here's the change in body-parser which start using the "on-finished":
https://github.com/expressjs/body-parser/compare/1.20.0...v2.0.0-beta.1#diff-c0749b151a7ec0288a6a4c849ff5215ec9ddeef8fa799a5cecef4b320a7dcf00R100
Here's example code:
import express from 'express'; // ^5.0.0-beta.1
import serverlessExpress from '@vendia/serverless-express'; // ^4.8.0

const app = express();
app.use(express.json());
app.post('/example', (req, res) => {
  console.log('body', req.body);
})

export const handler = serverlessExpress({ app });

Calling POST /example prints a "Buffer" object. If you enable debugging for body-parser (set env flag DEBUG=body-parser:*), then it prints out "body-parser:json body already parsed".
How can this be solved? I am afraid that if the request has "completed" to false, it might have unexpected consequences as well.

Oh no, are they actually going to release express v5? The best thing about Express is how it never changes and code that integrates with it never has to be updated :)


Hello, has SciSharp/NumSharp stopped development and maintenance?

Hello, has SciSharp/NumSharp stopped development and maintenance?

I think so,it's so sad.

Any news about this topic?


Adding my own greeting

amazing

Hi Alex, thanks for your contribution, can you please re-base your solution. Seems like there is a conflict

Ok, check it out now

Thanks


Demo configuration showing multiple user nodegroups and jupyterlab profiles using them

This has been documented here https://github.com/Quansight/qhub-ops/blob/master/qhub_ops/template/{{ cookiecutter.project_name }}/docs/maintinance.md and included in the datum deployment.


error while publishing docker image to ecr repository

Pushing the docker image to ECR repository...
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error saving credentials: error storing credentials - err: exit status 1, out: error storing credentials - err: exit status 1, out: The stub received bad data.``
wanotif1balisoketcom could not be published as ASP.NET Core App to Amazon ECS using AWS Fargate: Failed to login to Docker due to the following reason:
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Error saving credentials: error storing credentials - err: exit status 1, out: error storing credentials - err: exit status 1, out: The stub received bad data.``
wanotif1.balisoket.com failed to publish to AWS.

Windows Version: 10
Visual Studio Version 2019:
AWS Toolkit for Visual Studio Version:

I've created https://github.com/aws/aws-dotnet-deploy/issues/754. The Visual Studio Toolkit wraps the .NET Deployment Tool, so any changes to behavior here will have to originate in that repository.
https://stackoverflow.com/questions/60807697/docker-login-error-storing-credentials-the-stub-received-bad-data does mention a possible workaround for the same error message involving Docker configuration.
How are your AWS credentials configured?

Issue with similar error is reported at https://github.com/aws/aws-cli/issues/5636.

Based on testing with Get-ECRLoginCommand PowerShell Cmdlet, the token/password obtained:

For non-SSO based profile is approx. 2165 characters long.
For SSO based profile (after aws sso login) is approx. 2277 characters long.

(Get-ECRLoginCommand -ProfileName <<SSO-Profile-Name>>).Password | docker login --username AWS --password-stdin <<account-id>>.dkr.ecr.us-east-2.amazonaws.com works fine.
Tried using SSO profile with dotnet aws deploy --profile sso-profile-name --region us-east-1, worked fine.

@fpranadi Good evening. Could you please review https://github.com/aws/aws-toolkit-visual-studio/issues/320#issuecomment-1433723105 and respond with requested information. If this is no longer an issue, please share the root cause so that it is helpful to other users experiencing similar error and confirm if this issue could be closed.

This issue has grown stale. Without additional steps to reproduce or supporting details, we aren't able to investigate this issue further. If you are still encountering this problem, please open a new issue and provide supporting details.


[Enhancement] ArmorEquip Trigger

Description: Applies child components when placing armor into the armor slots and items into the offhand slot.

mana: true/false
cooldown: true/false
#tooltip for armor-slot: 1 = Head, 2 = Chest, 3 = Leggings, 4 = Boots, 5 = Offhand, 6 = Mainhand
armor-slot: 1-4

Add this trigger to the trigger mechanic

Solved by 93cb2cef1796ee107a0fcd321748c7f3d22c7523, https://github.com/promcteam/promccore/commit/f40819dcc73bc6e7d0f5edce18dae0153eff2ce3 and https://github.com/Travja/editor/pull/142/commits/f3f17ff7b70804a77a6d42dbf673bd94d45a612f


Mobile - Update content for earning feature

Describe the task
Update content for earning feature

[x] Message in case Enter Amount > Available Balance (Available Balance > 0)

Actual:

Expect: Show popup error:

Header: Insufficient balance
Content: Insufficient balance. Amount must be smaller than available balance
Button: I understand

[x] Unstaking period info

Actual: Bifrost 7 days
Expect:

Bifrost: "up to 7 days"

@Sokol142196 Error message when Selected pool is undefined need time to verify. Please move this issue to new task

I moved this issue to https://github.com/Koniverse/SubWallet-Mobile/issues/1470

Version test: https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-315-1468-merge-2024-02-15T10-42-42.ipa
Device test: IP 11

[ ] Update unstaking period info for Liquid staking

Version test: https://file.cdmteck.com/s/DB9JLAnjJczopcC/download?path=%2F&files=SubWalletMobile-build-315-1468-merge-2024-02-16T09-34-00.ipa
Device test: IP 11
Validation passed.


NETOBSERV-637 UI Consistency: Topology styling

Blocked by https://github.com/netobserv/network-observability-console-plugin/pull/215 & https://github.com/netobserv/network-observability-console-plugin/pull/222

Filtering color

Decorators sizing / placement

Old "info" decorator has been removed in favor of side panel for better UX

Decorators tooltips

Node background fix on dark theme

Edges barely visible on dark theme

Unknown / external square shape

/ok-to-test

awesome, looks much better with the dark theme!
/lgtm

awesome, looks much better with the dark theme! /lgtm

:+1: I have rebased for merge. No changes on my side

/lgtm

/lgtm

/approve


Clone breaks BT

Attempting to integrate this project with GDExt I discovered the following behavior.
By simply modifying the game_tick function in the Simple NPC Behavior example to have the following:
let status = bt.clone().tick()...

instead of
let status = bt.tick()...

I encountered an infinite loop that develops.
reached main loop...
NPC is alive...
NPC has action points: 3
Performing action: run. Action points: 2
Performing action: shoot. Action points: 1
NPC has action points: 1
Performing action: run. Action points: 0
Cannot perform action: shoot. Not enough action points.
NPC does not have actions points left...
Rested for a while... Action points: 1
reached main loop...
NPC is alive...
NPC has action points: 1
Performing action: run. Action points: 0
Cannot perform action: shoot. Not enough action points.
NPC does not have actions points left...
Rested for a while... Action points: 1
reached main loop...
NPC is alive...
NPC has action points: 1
Performing action: run. Action points: 0
Cannot perform action: shoot. Not enough action points.
NPC does not have actions points left...
Rested for a while... Action points: 1
reached main loop...

Minimal code example here. You also need to derive(clone) for your blackboard
My operating assumption was that clone semantics couldn't induce this kind of error, but turns out I don't understand it as well as I thought. Is there a way to resolve this where clone could be used?
(also I see this is on your kanban board when I get my gdext implementation working, would you like a copy?)

If you clone the behavior tree at that point it basically means that the simulation starts all over again infinitely. Notice that the initial behavior tree lives in the main function outside the "game loop" and you are cloning that again and again.
When this behavior tree is ticked inside the game_tick, it stays executing the behavior tree inside the game_tick function until the behavior tree reaches the Rest action. At this point, Running status is returned which means the behavior tree exits the tick function and returns Running status. This is how the RepeatSequence is programmed to work. If you return Running during RepeatSequence's sequence actions, the execution is returned out of the tick function with Running status. When you call tick again, the execution returns at the same action where Running was previously returned.
Since you are ticking a cloned version of the behavior tree, these changes get lost once you exit the game_tick function leaving the original initial behavior tree unchanged with no tick methods called onto it.

Makes sense. I anticipated clone would clone all internal state in addition to the structure or give a mutable reference to the same data structure. But I understand it working like this. I'll work around this. Thanks for following up.


Asset converter writes images to descriptor set before layout transitions, raising validation errors

Describe the bug
Calling reserve on an asset converter, passing as assets

An ICPUDescriptorSet with one descriptor binding of type ET_SAMPLED_IMAGE (likely same thing happens with ET_STORAGE_IMAGE or ET_COMBINED_IMAGE_SAMPLER).
An ICPUImageView bound to the aforementioned binding

will call updateDescriptorSets when creating the corresponding IGPUDescriptorSet. However, the layout transition for the image from UNDEFINED to the expected final layout happens later when calling convert.
Steps to Reproduce
As an example, set up an ICPUDescriptorSet with a single binding of type ET_SAMPLED_IMAGE and use getDescriptorInfos to bind an ICPUImageView to it. Then add the descriptor set as an input asset for a converter and call reserve on the converter.

The VUID of the Validation LAyer you're getting is not about the current layout of the image, but the layout the descriptor info states
4149 and 4150 are about invalid layouts in the descriptor info.
My guess is that you're probably providing an invalid enum here: https://github.com/Devsh-Graphics-Programming/Nabla/blob/e8f4dca1724b907826f2b9ca1f238ad114716218/src/nbl/video/utilities/CAssetConverter.cpp#L1708C50-L1708C58

which comes from here: https://github.com/Devsh-Graphics-Programming/Nabla/blob/e8f4dca1724b907826f2b9ca1f238ad114716218/src/nbl/video/utilities/CAssetConverter.cpp#L533C82-L533C93

nice repro, will investigate!

The VUID of the Validation LAyer you're getting is not about the current layout of the image, but the layout the descriptor info states
4149 and 4150 are about invalid layouts in the descriptor info.
My guess is that you're probably providing an invalid enum here: https://github.com/Devsh-Graphics-Programming/Nabla/blob/e8f4dca1724b907826f2b9ca1f238ad114716218/src/nbl/video/utilities/CAssetConverter.cpp#L1708C50-L1708C58

I think I know what caused it.
its an ICPUImageVIEW not a plain image thats written into the desc-set, wrong extra parameter pack handling.


Problems while testing locally

Hi,
I cloned the project and after npm install and npm run dev-app, I was able to run the server on my machine. But when I try to test it by opening it in an incognito window of Chrome, it looks for others for a few seconds and creates a new session because it cannot find any.
How should I go on with testing it?

Instead of using /webpack-dev-server , go to http://localhost:8080/pad/CeaselessPad2864

okay I got it
thanks


When full ledger support?

I click Import Hardware Wallet, select a ledger and nothing happens. I still view the account created with the generated seed.
Is Sollet ready to work with Ledger?

I think it only works with macOS at the moment.

I think it only works with macOS at the moment.

Okay. I am on Windows 10 using Chrome maybe that's the reason. When should I expect it work with this setup?

I don't think any of the sollet developers have access to a windows machine, so it's not immediately on our roadmap. PRs are welcome, though.

It is becoming an increasingly higher barrier of entry for ledger users on Windows to be able to access Solana dApps because they do not have a wallet solution (as seen on Raydium.io)
Just letting you know that maybe you could coordinate the team to have this feature soon? Many are needing it.
But of course, no pressure, your decision. Solflare.com seems to be an alternative fow now.
Thanks


Contribute Buttons before 20th of July 🚨

Discussed in https://github.com/Spyware007/Animating-Buttons/discussions/1346

Originally posted by Spyware007 June 19, 2023
Thank you so much for all the contributions! ✨
We will soon hit 200+ 🌟 , and more than 400+ buttons have been added! 🚀
📌 I will request all the contributors who had created issues for adding a new button should add it before the 20th of July.
📌 We will shift the data to Firebase and then you will be able to sign in and add button from the website itself!
[NOTE]
No Issues for adding new buttons will be accepted after 20th July
PS: New UI 👀

Please check #1346

Sorry for the inconvinience, but we are closing all the issues! Please open a new issue if you want to contribute anything. Also now you can create a new button from the website itself! You will get points for those too! I will add the instructions on how to create a new button soon and create an empty pr for the same to get points!


[WIP] Do not create Level 0 seciton in Markdown when the input document start from a section

Ref #636

Coverage increased (+0.05%) to 90.557% when pulling fb11f94613d6b7e2a81c6d436c46a554373b7cab on takahi-i:fix-voidsection into e1ae7ee3dbae61c811cee04972e72946c85df6a5 on redpen-cc:master.

Coverage increased (+0.03%) to 90.54% when pulling 5ed81190702de81b0a7b25615ced4b9478196e13 on takahi-i:fix-voidsection into e1ae7ee3dbae61c811cee04972e72946c85df6a5 on redpen-cc:master.


Update composer.json

Update composer.json to latest available versions.
No conflicts found.
Is there any reason to stick to the "old" versions even if newer ones are available w/o BC breaks?
Cheers,
LT

Thanks for your PR.
Just to clarify: this tutorial can never be kept updated for every package version used in the composer.json file. The versions that are currently noted are the versions after installing the 1.0.3 release of the Zend\Expressive skeleton. If there will be newer releases we might update the versions as well on a regular basis.
To clarify this I added a note at the end of the part1 doc.
https://github.com/RalfEggert/zend-expressive-tutorial/blob/part1/doc/book/part1.md#compare-with-example-repository-branch-part1
Closing this PR now.

Hello Ralf,
thanks for your explanation.
My understanding was that this tutorial is not yet complete as some discussed parts such as the pull requests from matthew weier o'phinney and doctrine, etc. are not yet implemented.
Cheers,
LT.

Regarding Matthews PRs. They won't be added to this tutorial. The idea is to provide two different flavours of the tutorial. This one concentrates on the usage of Zend Framework components, while another tutorial will use other components and Doctrine and stuff.


Flex

Don't forget to add your info to the Cargo.toml authors field if you want!

Don't forget to add your info to the Cargo.toml authors field if you want!

This is very nice of you! Will do


Support PHP advanced version.

This has been implemented in the 2.0 branch of assetic-php/assetic. Until we tag 2.0, you can pull it in by using "assetic/framework": "dev-2.0/dev" in your composer.json files.


Flow doesn't recognize that destructured properties in function parameters are read-only with regards to the object

Flow version: v0.97.0
When an object is destructured in the parameters of a function, I would prefer if flow Flow didn't produce errors that result from the possibility of re-assigning properties on the object.  Having to mark Maybe properties as covariant can be somewhat tedious, and solving the error can be quite counter-intuitive for those unfamiliar with it since the function can't easily produce the error being guarded against.
Try Flow
I recognize that a reference to the object is technically reachable through the arguments object, but accessing arguments is now generally agreed as bad practice in most applications. Accomodating such use cases could involve annotation escape hatches, more strict application of type-checking when the arguments token is present in a function, or a flow config setting.

While I see how this can be inconvenient in your case :(, this is not a bug, but an explicit design decision in Flow.
myFunc (which takes the argument without the read-only / covariance annotation) could delete the key thing from the object, which is exactly what Flow's sound type system defends against.

Can you explain how it could delete the key without accessing arguments?

Close since the behavior is expected. We only use the declared signature to decide the type of the function.

My issue is that if it were actually using my declared signature as given in the Try Flow link, it shouldn't behave this way.


Parameterize doobie sql interpolator, so that it's not hard-coded using the pSql interpolator

Fixed as of 460999870c9918b4c4ff0f641165083e0634179c


Update runtime_identifier.md

I changed the "disables entity shadows" to "makes the shadows small" on the trident and the arrow

Hey! Thanks for the PR. Sadly it has a file conflict. Can you fix the merge conflict? You can PM me if you need help with it.

I'm sorry, I tried to fix the conflict, but I'm a failure of a git user 😢
Added manually.


tests intermittently hit timeouts in travis

We may need to bump up the default test timeout to something greater than 5 seconds due to the occasional slowness in Travis' infrastructure.
For example the following run had two tests fail due to timeouts:
https://travis-ci.org/juttle/juttle/jobs/98195108

I'm looking into this will have a PR shortly to either increase the 5s timeout to something more reasonable or modify the tests to be better behaved.

@demmer something else must have happened here since I've ran this test numerous times locally and never seen a runtime over 75ms (the default --slow value). Could this timeout have been triggered by the other bug we saw this run that you're fixing due to the bad parsers tests that were leaving lingering callbacks and intermittently failing ?

Oh. That could maybe be the case. I'll put a PR up for that shortly and then we can maybe just close this issue if it doesn't pop up again.

Bumped up the timeout and haven't seen any more sporadic failures do to slower runtimes in travis. Closing this for the time being.


Form Component and Authentication Pages

Deliverables:

Dynamic Form Components - Base Input, Base Text Area and Base Select Fields
Responsive Authentication Pages - Signup & Signin

Note:
Please this is a joint task. Ibukun and Evidence, I expect you both to work together to close this. Thank you.

Hi, Ibukun
Please on this issue, you will have to cleanup the auth forms - spinners and error messages. Also, we have to re-introduce the signup options and cleanup for bugs on the auth side.


Add test_configparser from CPython 3.8.7

This adds over 300 tests that (should) already pass.

For reference (troubleshooting CI failures):

https://github.com/RustPython/RustPython/actions/runs/580161752
https://github.com/RustPython/RustPython/actions/runs/580337246
https://github.com/RustPython/RustPython/actions/runs/582168247


Update zio, zio-test, zio-test-sbt to 1.0.0-RC18

Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 1.0.0-RC17 to 1.0.0-RC18.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Applied Migrations

github:zio/zio/CurriedAssert

labels: library-update, semver-pre-release, scalafix-migrations

Superseded by #98.

Superseded by #98.


clean up

:+1: Looks like this has been introduced during a merge, see https://github.com/laravel/framework/commit/20e17b73732605fac37533421fd75c38b13b4540 and https://github.com/laravel/framework/commit/8fc1942fcb11fdd5f501a959accd23f3c10ca1e7.


Achievement Page update #6

on this link https://recodehive.github.io/awesome-github-profiles/pages/githubbadge.html
Under How to get: Mention like the below👇🏻

Merged a pull request without a review
You can raise a PR here here (https://github.com/recodehive/machine-learning-repos)
Watch the Video Tutorial: https://youtu.be/GnHNScuGKrg
Note: hyperlink the above link.

Achievement Page update #6

Please assign me for this

Resolved the issue
file:///home/devalnath/Pictures/Screenshots/Screenshot%20from%202024-10-08%2015-14-29.png

Resolved the issue

Bro , jis issue mai assign hue ho wahi krna chahiye na sirf 🫠

Bro , jis issue mai assign hue ho wahi krna chahiye na sirf 🫠

I don't know bro, the maintainers weren't replying and I forked the whole repo, I was just trying to do the easy fixes when I can

assign me if still available

im assigning to @JomanaMahmoud to make equal opportunity to everyone.

Please assign me the task

@devalcodes why are you raising the Pr which is not assigned to you.
i was trying to give equal opportunity for everyone here. and I have already assigned you couple of issue in this repo.
this is not good practice.


no unnecessary .0

I would prefer not to make this change because I want this crate to be nothing more than a port of dtoa.h. If you get this change accepted upstream, I will follow.
It looks like there are mainstream languages on both sides of this.

.0

CLISP
Haskell
Java/Scala/Clojure
Python
Ruby

no .0

D
Go
JavaScript
Perl
PHP


benchmarks: avoid use of deprecated pandas API

The append method of DataFrame was deprecated in pandas 1.4.0.
Here, we update to use pandas.concat instead (as recommended)

Is it alright if we move this to 22.08?

Yes

replaced by #339


Emit list of modules to retime

This is a summary of the discussion at the Hurricane-2 debrief meeting.
A Firrtl pass should emit a list of modules for which the designer intent is to retime them using the synthesis tools.  Such intent would be described in an annotation of the module in the Chisel source.
Whether this emits a simple list of names, or a properly-formatted (and potentially tool-specific) list of .tcl commands, is not decided.  But the designer intent should certainly be encoded in the source and passed through to the backend somehow.

See https://github.com/freechipsproject/rocket-chip/pull/1333

Consider in the future whether to move this to chisel without rocket


[wrt][xwalk-3599] Update ssh connection code in run_test.py

Add the code for ssh connection when test device
type is ssh

Impacted tests(approved): new 0, update 264, delete 0
Unit test platform: [Tizen]
Unit test result summary: pass 264, fail 0, block 0
Bug: xwalk-3599

LGTM


CLI does not start browser on port 4200 with active reload

Please provide us with the following information:

OS?

Windows 7, 8 or 10. Linux (which distribution). Mac OSX (Yosemite? El Capitan?)

OSX Yosimite
Versions.

Please run ng --version. If there's nothing outputted, please run in a Terminal: node --version and paste the result here:

angular-cli: 1.0.0-beta.19-3
node: 6.9.1
os: darwin x64

Repro steps.

Was this an app that wasn't created using the CLI? What change did you do on your code? etc.

All of it was cli.
The log given by the failure.

Normally this include a stack trace and some more information.

EHEARD-M-N20B:my-dream-app peteheard$ n6076ms building modules                   2ms add01675ms1320ms asset41ms emittingHash: 28ce1f0add2ae1d07014
Version: webpack 2.1.0-beta.25
Time: 9685ms
           Asset       Size  Chunks             Chunk Names
  main.bundle.js    2.71 MB    0, 2  [emitted]  main
styles.bundle.js    10.2 kB    1, 2  [emitted]  styles
       inline.js    5.53 kB       2  [emitted]  inline
        main.map    2.81 MB    0, 2  [emitted]  main
      styles.map    14.1 kB    1, 2  [emitted]  styles
      inline.map    5.59 kB       2  [emitted]  inline
      index.html  477 bytes          [emitted]
Child html-webpack-plugin for "index.html":
         Asset     Size  Chunks       Chunk Names
    index.html  2.81 kB       0
webpack: bundle is now VALID.

Mention any other details that might be useful.

Thanks! We'll be in touch soon.

The ng serve command does not open the browser by default. You need to add the argument -open to it, like ng serve -open.

Oh. That document is very outdated!
It doesn't even have the serve command!
Might be a good idea to remove it, redirect to readme or something.
// CC @filipesilva

An outdated doc file is a separate issue from what this issue is though.

Do I have to make a new issue or can I rename this one then?
What does //CC @filipesilva mean?

It's like a "mention" in twitter/facebook :)
Since he hasn't replied anyway, I expect it'll be a bit until there's talks about it. Maybe close this issue and create another one, linking to this issue for context.
Thanks a lot man!

"start": "ng serve -open" or  "start": "ng serve -opn" opens the browser before the compilation is done and moreover, it opens an empty browser window. Does anyone have a precise solution?


Revert "Master to Live"

Reverts MicrosoftDocs/office-docs-powershell#1617

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

.openpublishing.publish.config.json
:white_check_mark:Succeeded

exchange/docfx.json
:white_check_mark:Succeeded

exchange/docs-conceptual/exchange-eop/connect-to-exchange-online-protection-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-eop/exchange-online-protection-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/connect-to-exchange-online-powershell/connect-to-exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/connect-to-exchange-online-powershell/mfa-connect-to-exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/disable-access-to-exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/connect-to-exchange-servers-using-remote-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/control-remote-powershell-access-to-exchange-servers.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/exchange-cmdlet-syntax.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/exchange-management-shell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/find-exchange-cmdlet-permissions.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/open-the-exchange-management-shell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/recipient-filters/filter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/recipient-filters/recipient-filters.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/recipient-filters/recipientfilter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/use-update-exchangehelp.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/office-365-scc/connect-to-scc-powershell/connect-to-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/office-365-scc/connect-to-scc-powershell/mfa-connect-to-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/office-365-scc/office-365-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/active-directory/active-directory.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AdvancedThreatProtectionDocumentDetail.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AdvancedThreatProtectionDocumentReport.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/advanced-threat-protection.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/antispam-antimalware/Get-MailboxJunkEmailConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/antispam-antimalware/antispam-antimalware.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/client-access-servers/client-access-servers.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/client-access/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/Set-OwaMailboxPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/client-access.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/client-access/set-CASMailboxPlan.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/connected-accounts/connected-accounts.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/database-availability-groups/database-availability-groups.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/devices/devices.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/email-addresses-and-address-books/email-addresses-and-address-books.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/encryption-and-certificates/encryption-and-certificates.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/exchange.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/federation-and-hybrid/federation-and-hybrid.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/mail-flow/mail-flow.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/mailbox-databases-and-servers/mailbox-databases-and-servers.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/mailboxes/Set-Clutter.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/mailboxes.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/move-and-migration/move-and-migration.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/organization/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/organization.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/policy-and-compliance-audit/policy-and-compliance-audit.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/policy-and-compliance-content-search/policy-and-compliance-content-search.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/policy-and-compliance-dlp/policy-and-compliance-dlp.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

.openpublishing.publish.config.json
:white_check_mark:Succeeded

exchange/docfx.json
:white_check_mark:Succeeded

exchange/docs-conceptual/exchange-eop/connect-to-exchange-online-protection-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-eop/exchange-online-protection-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/connect-to-exchange-online-powershell/connect-to-exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/connect-to-exchange-online-powershell/mfa-connect-to-exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/disable-access-to-exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-online/exchange-online-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/connect-to-exchange-servers-using-remote-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/control-remote-powershell-access-to-exchange-servers.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/exchange-cmdlet-syntax.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/exchange-management-shell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/find-exchange-cmdlet-permissions.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/open-the-exchange-management-shell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/recipient-filters/filter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/recipient-filters/recipient-filters.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/recipient-filters/recipientfilter-properties.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/exchange-server/use-update-exchangehelp.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/office-365-scc/connect-to-scc-powershell/connect-to-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/office-365-scc/connect-to-scc-powershell/mfa-connect-to-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/docs-conceptual/office-365-scc/office-365-scc-powershell.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/active-directory/active-directory.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AdvancedThreatProtectionDocumentDetail.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/Get-AdvancedThreatProtectionDocumentReport.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/advanced-threat-protection/advanced-threat-protection.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/antispam-antimalware/Get-MailboxJunkEmailConfiguration.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/antispam-antimalware/antispam-antimalware.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/client-access-servers/client-access-servers.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/client-access/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/Set-OwaMailboxPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/client-access/client-access.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/client-access/set-CASMailboxPlan.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/connected-accounts/connected-accounts.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/database-availability-groups/database-availability-groups.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/devices/devices.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/email-addresses-and-address-books/email-addresses-and-address-books.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/encryption-and-certificates/encryption-and-certificates.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/exchange.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/federation-and-hybrid/federation-and-hybrid.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/mail-flow/mail-flow.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/mailbox-databases-and-servers/mailbox-databases-and-servers.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/mailboxes/Set-Clutter.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/mailboxes/mailboxes.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/move-and-migration/move-and-migration.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/organization/Set-OrganizationConfig.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/organization/organization.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/policy-and-compliance-audit/policy-and-compliance-audit.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/policy-and-compliance-content-search/policy-and-compliance-content-search.md
:white_check_mark:Succeeded

exchange/exchange-ps/exchange/policy-and-compliance-dlp/policy-and-compliance-dlp.md
:white_check_mark:Succeeded

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


Add support for rolling updates when cluster config changes are applied

For example, the user may wish to patch the cluster spec with a new container image. When this happens, ideally the operator would reconcile the new spec with the runnings pods by spinning up new pods, draining traffic from the old ones, then removing the old pods. This would primarily be useful for long-running clusters (e.g., Ray Serve) that ideally should be upgraded with zero downtime.
cc @Jeffwan

Attach some offline discussion here. I think this is a valid use case especially for long running clusters.
Live upgrade need support of both control plane and data plane.

Control plane requires non-stop head service and graceful handling of those inflight requests,
Data plane needs similar mechanism like Kubernetes drain node in order to migrate actors to other nodes. I think some work might be done on Ray core part to support this.

As @DmitriGekhtman said, A fully general rolling upgrade is going to be hard achieve for a while.
We can do some investigations and collect requirements for Ray core.

It's tracked elsewhere.


Dump

Proposed changes

Re-structured optimizerFORCES function
Added variant to only dump the solver object into a MAT file without generating code
Added option to perform some basic anonymization on the problem data
Minor cosmetics

Reviewer
@kostas-l

@GianUlli ready to be merged into master from our side. Can you please do so and also tag it as release 0.1.19? Thanks.

OK, I reran all tests and they pass, ready to be merged from my point of view.


feat: add new roadmap page

Summary | Résumé
Add a new roadmap page.

LGTM! @adorayi are we linking to this page from somewhere else in the site?

No need to link it for now, we can share the direct url for now and then consider the full site structure another time.


2.0-wip: Long Links Overflow Dropdown Menu

In 2.0, dropdown links have an added white-space: nowrap defined here. Additionally, the dropdown menu itself is set to have max-width: 220pxdefined here
These two don't play nice in cases where the link text is wider than 220px. The overflow spills out of the dropdown onto the page behind. On mouse-over, the overflow is hidden.
I tried removing the nowrap from links, but that causes the dropdown to wrap well before the max-width is hit (seemingly at the min-width). I couldn't figure out how to get it to wrap at max-width.

Same problem here.
In my case I just removed the max-width as having more than 220px is not a problem for me.

Removed the max-width to resolve this.


Is there a way to extract positions/postings information from a search result?

I am trying to highlight search results within their original documents for presentation. In order to do that I need to get the raw position information for each term involved in the search, but have been unable to find a way to do that. I'm not even sure that information is kept around when a search is completed. The document text is indexed with STORED & POSITION options.

Maybe the code from SnippetGenerator is helpful?

Yes, I did look at the SnippetGenerator source and it was helpful. I was kind of hoping there was an API I had missed which generates a list of positions given a query and a document id as parameters. But, I think I see a path to building that kind of functionality by using an InvertedIndexReader to extract the position information for each search term within that document and then merging those results.


Ensure that transactions are persisted to configuration databases when making changes to Chocolatey GUI settings/features

Starting with LiteDB 5.x.x, the ways that transactions are handled in the database has changed.  When a transaction hasn't been persisted to the main data.db file, it exists as a transaction within a data-log.db file.  To ensure that these transactions make their way into the main data.db file, we need to make sure to dispose of the database correctly once everything is completed.
Not doing this leads to transactions not being committed properly to the database, and as a result, inconsistencies in how the application will work.

I have a big data-log.db file 1.31GB and my Chocolatey GUI is v<IP_ADDRESS>, can I delete this db file? If not, how can I clean it? Thank you.


Add comma to prevent build error when react.gradle is uncommented

Thanks for submitting a PR! Please read these instructions carefully:

[ ] Explain the motivation for making this change.
[ ] Provide a test plan demonstrating that the code is solid.
[ ] Match the code formatting of the rest of the codebase.
[ ] Target the master branch, NOT a "stable" branch.

Motivation (required)
At the moment, if you uncomment the react specific code in build.gradle you'll get a compilation error. Adding a comma prevents this error.
Test Plan (required)
No tests are necessary. This is a simple fix.
Next Steps
Sign the CLA, if you haven't already.
Small pull requests are much easier to review and more likely to get merged. Make sure the PR does only one thing, otherwise please split it.
Make sure all tests pass on both Travis and Circle CI. PRs that break tests are unlikely to be merged.
For more info, see the "Pull Requests" section of our "Contributing" guidelines.

@facebook-github-bot shipit

Hmm... I really just added a comma. Not sure how that would cause the builds to fail.


IANA code for TLV

2 IANA codes are support for TLV:
typedef enum
{
    LWM2M_CONTENT_TEXT      = 0,        // Also used as undefined
    LWM2M_CONTENT_LINK      = 40,
    LWM2M_CONTENT_OPAQUE    = 42,
    LWM2M_CONTENT_TLV_OLD   = 1542,     // Keep old value for backward-compatibility
    LWM2M_CONTENT_TLV       = 11542,
    LWM2M_CONTENT_JSON_OLD  = 1543,     // Keep old value for backward-compatibility
    LWM2M_CONTENT_JSON      = 11543
} lwm2m_media_type_t;

The OLD value (1542) is supported if LWM2M_OLD_CONTENT_FORMAT_SUPPORT compilation flag is used.
If the Wakaama Linux client connect to a server and received a READ command on /3 (example), even if the LWM2M_OLD_CONTENT_FORMAT_SUPPORT compilation flag is used, the coding value for content-type is the new one
Example
00000000 67 45 89 11 EE 2C 3C CD  59 36 7D C2 2D 16 FF 08
00000010 00 8E C8 00 14 4F 70 65  6E 20 4D 6F 62 69 6C 65
00000020 20 41 6C 6C 69 61 6E 63  65 C8 01 16 4C 69 67 68
00000030 74 77 65 69 67 68 74 20  4D 32 4D 20 43 6C 69 65
00000040 6E 74 C8 02 09 33 34 35  30 30 30 31 32 33 C3 03
00000050 31 2E 30 86 06 41 00 01  41 01 05 88 07 08 42 00
00000060 0E D8 42 01 13 88 87 08  41 00 7D 42 01 03 84 C1
00000070 09 64 C1 0A 0F 83 0B 41  00 00 C8 0D 08 00 00 00
00000080 00 AA A0 F1 72 C6 0E 2B  30 31 3A 30 30 C8 0F 0D
00000090 45 75 72 6F 70 65 2F 42  65 72 6C 69 6E C1 10 55

1st line,: 2D 16
Better to send in this case 06 06 (1542)
Old value is not supported in lwm2m_data_serialize

For me, the LWM2M_OLD_CONTENT_FORMAT_SUPPORT means that wakaama handles both the new and the old values.
If the request contains an accept option with (only) the old value, the response's content-type will be the old one.
If there is no accept option in the request, the response will use the new value.
If I understand correctly, what you are looking for is to always use the old values and discard the new one. In that case, it would be better to have:
typedef enum
{
    LWM2M_CONTENT_TEXT      = 0,        // Also used as undefined
    LWM2M_CONTENT_LINK      = 40,
    LWM2M_CONTENT_OPAQUE    = 42,
#ifdef LWM2M_OLD_CONTENT_FORMAT_ONLY
    LWM2M_CONTENT_TLV       = 1542,
    LWM2M_CONTENT_JSON      = 1543
#else   // LWM2M_OLD_CONTENT_FORMAT_ONLY
    LWM2M_CONTENT_TLV       = 11542,
    LWM2M_CONTENT_JSON      = 11543
#ifdef LWM2M_OLD_CONTENT_FORMAT_SUPPORT
    LWM2M_CONTENT_TLV_OLD   = 1542,     // Keep old value for backward-compatibility
    LWM2M_CONTENT_JSON_OLD  = 1543,     // Keep old value for backward-compatibility
#endif  //  LWM2M_OLD_CONTENT_FORMAT_SUPPORT
#endif  // LWM2M_OLD_CONTENT_FORMAT_ONLY
} lwm2m_media_type_t;

Regards,

OK
From my point of view, I thought that the LWM2M_OLD_CONTENT_FORMAT_SUPPORT flag meant that only old code for TLV/JSON are supported.
Your remark makes sense: close the ticket and the pull request


deployer: Support resuming deployments

The main changes here are determining what job events to wait for based on currently running jobs, and ensuring the deployer reconnects to the controller when waiting for job events.
Apologies for the big diff, but I did not commit along the way and splitting things out would take time.
Closes #799.

Closing this in favour of smaller incremental patches.


Add more checks to K frontend

[ ] Check that the left-hand side of any function definition rule is a function-like pattern.
[x] Check that no two productions with the symbol attribute have the same klabel attribute or the name of any builtin symbol in KORE.
[ ] Check that collection patterns are valid.
[ ] Check that module names are unique.
[ ] Check that the configuration declaration does not refer to functions requiring context.
[ ] Check that hooked sorts do not declare constructors
[ ] Check that hooks are declared correctly.

[ ] Check that an ensures clause is either #True, true, or contains at least one free variable.
[ ] Check that the user does not declare productions of sort K that are not functions.
[ ] Warning if the user extends the isSort predicates (except KResult)
[x] Check that rule labels are unique.
[x] Check that axioms do not contain function symbols on lhs (except for at the top)
[ ] Check that bracket attribute is used correctly.
[ ] Check that contexts don't contain rewrites outside of K cell or rewrites whose LHS is not a HOLE or whose RHS does not contain a HOLE
[ ] Check that contexts either mention no cells or a k cell.
[ ] Check that the BOOL module is imported by the main module of the definition
[ ] Check that the user doesn't import any modules in kast.k except BASIC-K

There is definitely a check that the user doesn't use klabels or productions that weren't declared anywhere, do you have an example where this is failing?

I don't remember the context, it only happens in specific situations. When I'll encounter this issue again I'll post an example.

A more helpful inner parser would go a long way. For example, if I accidentally screw up a closing cell in the configuration, like so
<foo> 0 <foo>

the error can look something like this:
[Error] Compiler: Had 215 parsing errors.
[Error] Inner Parser: Parse error: unexpected token '<'.
	Source(/home/hjort/code/wasm/ewasm-semantics/deps/wasm-semantics/.build/defn/llvm/wasm.k)
	Location(19,24,19,25)
[Error] Inner Parser: Scanner error: unexpected character sequence '/'.
	Source(/home/hjort/code/wasm/ewasm-semantics/deps/wasm-semantics/./.build/defn/llvm/test.k)
	Location(112,65,112,66)
[Error] Inner Parser: Scanner error: unexpected character sequence '/'.
	Source(/home/hjort/code/wasm/ewasm-semantics/deps/wasm-semantics/./.build/defn/llvm/test.k)
	Location(116,70,116,71)
...

If I misspell a sort name the error points me to a later part of the line:
rule ( module OID:OptionaId DEFNS ) => ...

Above an l is missing, easy to miss. Error message:
Error] Compiler: Had 1 parsing errors.
[Error] Inner Parser: Parse error: unexpected token 'DEFNS'.
	Source(/home/hjort/code/wasm/ewasm-semantics/deps/wasm-semantics/.build/defn/llvm/wasm.k)
	Location(1512,37,1512,42)
make: *** [Makefile:114: .build/defn/llvm/test-kompiled/interpreter] Error 113

I find these kinds of error messages are common. They describe how the system failed internally rather than try to point at plausible mistakes.
I find these are no big deal after you get used to K and know what to look for and when a rule is malformed, but for someone starting out they are super annoying because they misdirect attention and don't really teach you anything about what you should be doing when you are unsure yourself. I remember quite a lot of time being spent when I just started with K struggling to understand why my programs didn't kompile correctly.

I have been aware of the issue when the configuration fails to parse for some time and just never got around to fixing it. Fortunately, there is an easy fix; simply don't parse any rules if there are configurations that didn't parse.
This second issue, with the error message not being very informative, I am interested in investigating, but you will have to give me a minimal example because there is some interaction between different productions that is causing the behavior you see.

Closing this as there's been no meaningful update in 4 years.


LTO(2/3): Enable thin LTO

This is an experiment to get measurements of CI run times, with and without LTO enabled. See discussion on Zulip.
This PR will establish the time it takes for CI to compile and run with thin LTO.

Too bad, there are mingw compile failures.


Hangs when multiple calls made to failover_storage_group_srdf() with different SG names

Thanks Paul for your reply in advance!!!
I have a following call in my code with different group_name & same rdf_number & opt as:
replication.failover_storage_group_srdf(storage_group_id=self.group_name, srdf_group_number=self.rdf_number, failover_options=opt, _async=False)
I am calling above function at the same time for 6 storage groups. It is giving me exception:
"Bad or unexpected response from the storage volume backend API: Error PUT symmetrix resource. The status code received is None and the message is None."
Also error message:
ERROR:PyU4V.rest_requests:The PUT request to URL https://<IP_ADDRESS>:8443/univmax/restapi/92/replication/symmetrix/000297900701/storagegroup/SG_dl360g10-141_142_R6/rdf_group/101 timed-out, but may have been successful. Please check the array. Exception received: HTTPSConnectionPool(host='<IP_ADDRESS>', port=8443): Read timed out. (read timeout=120).
Looks like the API call hung caused the problem. I then used API to set timedout as below did not help me.
self.conn = PyU4V.U4VConn(username=self.username, password=self.password, server_ip= self.server_ip, port=self.port, verify=self.verify,array_id=self.array_id)
self.conn.set_requests_timeout(500)
Am I missing something?
NOTE: We have both Gatekeeper & Unisphere available.
Thanks,
Sacheen Birhade

Sacheen, you are issuing calls for 6 different groups at the same time,  Try staggering the failovers, put a wait in doing them one after the other.
The error code is a generic one coming from the Unipshere server not from PyU4V,   I suspect your Unipshere server is busy and replying back with the generic error.  Another thing you could do is issue the calls asyncronously and check the job ids after,  Check the examples in https://pyu4v.readthedocs.io/en/latest/programmers_guide_src/provisioning.html#a-synchronous-provisioning-creating-storage-for-a-host  it's the same concept for all calls that support Asyncronous Execution, this is an ideal candidate for this type of operation.  You won't be getting timeouts or overloading the server.

@Sacheen123 we have enabled the Discussions feature on GitHub for questions and Q&A,  for non PyU4V issues and general discussions if you want to create discussions we can tackle on best effort basis.
Let me know if the Asynchronous calls resolved your issue, I'm confident that this would

Thanks Paul for your help. I have fixed this issue using exclusive lock through my python code.
In my case multiple calls to the same API were through different thread, so file based exclusive lock helped me to send one request at a time. By that time all other threads were waiting.
flow:
take exclusive lock()
all to api ()
release exclusive lock()
You were really helpful to me. Thanks. best wishes.

Nice trick 👌
Sent from Outlook for Androidhttps://aka.ms/AAb9ysg
Internal Use - Confidential

From: Sacheen123 @.>
Sent: Friday, July 12, 2024 10:58:25 AM
To: dell/PyU4V @.>
Cc: Martin, Paul @.>; Assign @.>
Subject: Re: [dell/PyU4V] Hangs when multiple calls made to failover_storage_group_srdf() with different SG names (Issue #191)
[EXTERNAL EMAIL]
Thanks Paul for your help. I have fixed this issue using exclusive lock through my python code.
In my case multiple calls to the same API were through different thread, so file based exclusive lock helped me to send one request at a time. By that time all other threads were waiting.
flow:
take exclusive lock()
all to api ()
release exclusive lock()
You were really helpful to me. Thanks. best wishes.
—
Reply to this email directly, view it on GitHub [github.com]https://urldefense.com/v3/__https://github.com/dell/PyU4V/issues/191*issuecomment-2225235643__;Iw!!LpKI!nE7fM6-szl7eqsRCEKKsLR-oT3Cwbnt2sGeqEzrpd41-qDT3zo87garz8r7r8daGrrDNflr5oBxIUarObJmPXhyL$, or unsubscribe [github.com]https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AF7FWPIJFL4G64U7CDGVWODZL6SEDAVCNFSM6AAAAABKNP2DPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVGIZTKNRUGM__;!!LpKI!nE7fM6-szl7eqsRCEKKsLR-oT3Cwbnt2sGeqEzrpd41-qDT3zo87garz8r7r8daGrrDNflr5oBxIUarObOT6n2FT$.
You are receiving this because you were assigned.Message ID: @.***>


Javascript React Emmet is not working

VSCode Version: 1.7.1
OS Version: macOS 10.12.1

Steps to Reproduce:

Open up a .jsx file
Pressing tab now no longer autocomplete jsx syntaxes.

Did I miss something on the changelog? I just updated it and suddenly it stops working. Running with --disable-extension still does not fix the problem. Apparently no issue about this has been raised in here so I'm quite confused right now whether it's an intended change or a local problem. Thanks.
Edit
<EMAIL_ADDRESS>does not have this problem.

Duplicate of  #14962
@kairxa Looks like this will be fixed with the upcoming 1.7.2 release

closing as dup


Cache invalidation with gatsby apps

** Please describe which feature you have a question about? **
Delta Deployments
** Provide additional details**
I have a gatsby-amplify app. If I remove pages, aka paths from my gatsby app and access them afterwards they are not accessible on my local builds but on AWS the CDN serves the old version of the app when accessing these routes. As soon I access a valid route, it switches to the new version of the app.
My build settings:
version: 0.1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - yarn install
    build:
      commands:
        - yarn build
  artifacts:
    baseDirectory: public
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

where yarn build references "gatsby build",

Hi @ohlr . This seems to be similar to https://github.com/aws-amplify/amplify-console/issues/604 . We have a feature request added to our backlog to address this. Let me know if this helps

@abhi7cr looking forward to the feature.. closing the issue as it's a duplicate. Thank you for the fast response.


Make it possible to use generic Context types

Currently there can only be one concrete Context type per #[union_fn]. However, especially in wasmi the respective Context type is generic over some lifetimes and a user defined HostState. Therefore we cannot make use of this macro library until it supports generic Context types.
One design solution is to allow GAT syntax for the Context type such as:
#[union_fn]
trait Foo {
    type Context<'a, T> = MyContext<'a, T>;
}

Technically we will require some changes to the union_fn traits in order to support generic Context types. Namely CallWithContext and IntoOpt will be required to be generic over the Context type. Furthermore we can no longer have an associated Output type on the UnionFn trait since it might be generic.
With respect to the generated (hidden) types there also need to be some changes:

The user facing enum and args types needs no changes since they are already decoupled from the context type.
The optimized #[union_fn] type needs to be generic over the Context type because its function pointers will be different for each generic invocation..
The delegator and impls types need to be generic over the Context type since they are defining methods taking the generic Context type as parameter.

Traits
Given the above summary we will require the following changes to our base traits in the union_fn crate:
/// This trait is still going to be implemented on the generated `Enum` and `Opt` types.
pub UnionFn {
    type Output;
    type Args;
}

/// Implemented by `Enum` and `Opt` if they define a `Context` type or a generic `Context` type.
pub trait CallWithContext<Context>: UnionFn {
    fn call(self, ctx: &mut Context) -> <Self as UnionFn>::Output;
}

/// Same as `CallWithContext` but for `#[union_fn]` types that do not have a `Context`.
pub trait Call: UnionFn {
    fn call(self) -> <Self as UnionFn>::Output;
}

/// Implemented by `Enum` type for traits that define a (generic) `Context` type.
pub trait IntoOpt<Context> {
    type Opt;
    type Delegate;
    type Impls;

    fn into_opt(self) -> Self::Opt;
}

/// Sentinel type for `IntoOpt` trait impls without `Context` type.
pub enum NoContext {}

This way we no longer can access the Context type via <EnumType as UnionFn>::Context as we do so far. However, this is not an issue since we can simple use the literal Context tokens given by the user instead. No associated type access needed in order to maintain syntactical cleanliness.

A problem with this design is that generically different Context types will require different optimized Opt types due to requiring different underlying function pointers. With respect to wasmi it would not be very useful to have such a design since then the wasmi bytecode is directly and structurally dependent on the host data type which would not make sense.
Therefore this issue is not very relevant for wasmi and we need to find a better solution to make wasmi engine executor code completely independent of the user provided host data type. Technically this shoud be possible since the host data type is not really needed where interaction with the bytecode is required.


BIP39 Wallet

Implement a way to backup/restore keys using a seed phrase (BIP39).
BIP39 covers more than seed phrases, it also covers HD wallets (BIP32), which enables generating multiple keys using the same seed-phrase. The same seed-phrase can be used to recover all the keys that were generated with it.
We could do something similar to Cosmos gaiacli:
# produce a seed phrase, generate the fist key following BIP39(index 0),
# prompt password to encrypt key, save encrypted keyfile.
monetd keys add [name]

# prompt seed phrase, and recover the i'th key
# prompt password to encrypt key, save encrypted keyfile
monetd keys add [name] --recover --index [i]

So for example:
monetd keys add key0
monetd keys add key0-recovered --recover

Produces two identical keys key0 and key0-recovered provided that the seed-phrase given to the second command is the same as the seed-phrase produced by the first command.
Then, create a second key using the same seed-phrase:
monetd keys add key1 --recover --index 1

And recover it:
monetd keys add key1-recovered --recover --index 1

key1-recovered equals key1.

If we allow ourselves to deviate from gaiacli's API, we could have:
To generate a new keyfile from a new seed-phrase:
monetd keys new [name]

To generate a new keyfile from an existing seed-phrase (which it prompts for interactively):
monetd keys new [name] --seed

To recover a key from a seed-phrase:
monetd keys recover [name]

At this stage, there would be no support for HD wallets. Only one key per seed-phrase.
Full BIP39 would be implemented in monetcli later.


Tweak debug log

@vladyslav-iosdev seems like you do get assigned as reviewer automatically. Just don't add anyone when creating the PR.


update github action scripts to test on supported platforms (multi-repo)

dependent on #238 (determine list of supported platforms)
repositories:

wasmcloud/wasmcloud-otp
wash
examples
capability providers
interfaces

superceded by wasmcloud #245


[epic] Build benchmarks

Build a set of simulators/datasets to benchmark autoemulate.
Let's aim at having 2-3 datasets from 3 different fields. If we only include small-medium sized datasets, it should be ok to include them in autoemulate.

[x] biomedical (cardiac)

[ ] engineering

[ ] climate change (putatively)

[x] #136

What are the steps needed here? How can we bring this to a finish line over the next three sprints? (Or however many sprints we think we'll need for this issue.)

Added some more specifics. Should be doable in one sprint.

Great - thanks! I think we still need to break it down into several issues, unless this is not an epic but rather a single issue now that it has been refined? Let's chat about it in today's planning meeting.

An example for how to write something for loading data, if we need it (i.e. if there are bigger data packages that we need to load) can be found in the DeepSensor codebase. I liked how @tom-andersson made it easy to load external datasets for the package.

Great that we have multiple datasets across domains that can be easily loaded, is the actual benchmarking also documented somewhere already?

notes from aug 12 discussion
dimensions to benchmark

fit
stability (eg of eigenvalues)
rank order of parameters

potential dataset for benchmarking for engineering application https://www.researchgate.net/publication/380289575_A_Comprehensive_Dataset_of_the_Aerodynamic_and_Geometric_Coefficients_of_Airfoils_in_the_Public_Domain


Memory handling

Memory handling in SMP has changed compared with JSP, yet scripts are in the dark over the SMP memory status. The particular problem is ensuring OOMs can't be triggered on systems with low amounts of RAM (perhaps as little as 250MB for all SMP panels) while allowing those with plenty of RAM allocated to SMP to use it. Perhaps "% SMP memory used", which could be dynamically called,  or other SMP memory info could be provided to scripts?

Well, I thought of that, but what could the end user (e.g. you) do with this information?

Some of my scripts use a dynamic image cache.
So something like:
if ("% SMP memory used" < 50) add_img_to_cache();
else trim_img_cache() then add_img_to_cache();

Alternatively, but perhaps less effective, would be to know the maximum heap size. Then:

Set a limit to the image cache size, e.g. 10-20% of maximum heap size.
Script keeps a running total of the size of the image cache in memory.

Then something like:
if (image_cache_size_in_memory < allowed_image_cache_size) add_img_to_cache();
else trim_img_cache() then add_img_to_cache();


Add support for installation from url

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
12fb7c9f935a6f59ef16b7dddc55d1046ff352d9
link
/test operator-lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm, the downloaded files are the istio release tar and that's where the signature comes from. The istio/installer charts and operator profiles are being bundled into the istio release tar.
There's very little code required to implement this and it's a useful shortcut, according to UX feedback.

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-test
c6f3f231e95715e3e30494d60b4ba882cc04603c
link
/test operator-test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-test
c6f3f231e95715e3e30494d60b4ba882cc04603c
link
/test operator-test

operator-lint
bcae7979b1ee326add1d3fc26f04064dd733c30d
link
/test operator-lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@richardwxn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint
a879494fc3e19cfee26c4e751f3095428ccc758d
link
/test operator-lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Any link to UX feedback ? What kind of testing do we expect/need to do ?
My concern is that instead of focusing on the core
'operations/upgrade/canary', a lot of the code and testing has to be
spent on non-critical features - longer term, after operator starts
supporting it's specific use cases it may be ok
to add more stuff, but right now downloading releases should be P4 at best.
And there are implications - it's not 'cost free' even if code to download
a file is easy.  For example kustomize and is
taking the approach of using git repos - why would we use .tar.gz and .sha
file ? I was not planning to have
.tar.gz and sha for the installer - but a rolling release expecting tools
to use git to merge.
Keep in mind most prod users are patching or adjusting our templates - a
.tar.gz doesn't work well with this
pattern. And for beginners we already have an install solution that is
super easy, with kubectl -k
On Tue, Aug 13, 2019 at 5:09 PM Xinnan Wen<EMAIL_ADDRESS>wrote:

@richardwxn commented on this pull request.
In cmd/mesh/manifest.go
https://github.com/istio/operator/pull/186#discussion_r313661562:

@@ -89,6 +93,27 @@ func genManifests(inFilename string, setOverlayYAML string) (name.ManifestMap, e
if err != nil {
return nil, err
}

// for installation path contains version number only
isp := mergedICPS.InstallPackagePath
if version.IsVersionString(isp) {

mergedICPS.InstallPackagePath = fmt.Sprintf(helm.InstallationPathTemplate, isp, isp)

}
if util.IsHTTPURL(mergedICPS.InstallPackagePath) {

uf, err := helm.NewURLFetcher(mergedICPS.InstallPackagePath, "")

it is actually using the same directory instead of creating new ones with
random prefix. You can check the logic at NewURLFetcher. To avoid
confusion, I will update the naming and comment.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/operator/pull/186?email_source=notifications&email_token=AAAUR2X24EHHA6LPUFROJLLQENEKFA5CNFSM4ILG2I72YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCBPFAEY#discussion_r313661562,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAUR2SAYCZIFBUVYTN55WTQENEKFANCNFSM4ILG2I7Q
.

@costinm rather than holding up this PR, let's discuss it in UX group. I don't have an objection to a different approach to publishing. I do think that many users will benefit from having a single line install that points to a URL and given that the manifest commands are now in istioctl it makes a lot of sense to me to have the charts in there as well. But this is still alpha so we can get more feedback, evaluate and make a decision.

I am not saying having a 'download' feature is bad.
I am holding releasing this feature as implemented because:

downloading .tar.gz + .sha is ancient - kubectl is using git ( so is
golang ).
the .sha has no purpose anyways - it provides no security. If you
download a file and the sha from the same place you don't verify any
provenance.
It was used for mirroring - but the sha would be verified against the main
site. But we don't plan to support mirroring the .tar.
it has impact on release process - more complications to publish the
installer as .tar.gz, securing the distribution channel, etc
what is the target audience for this feature ? Beginners who just want a
demo can use plain kubectl - which does download for you from git.
Advanced users will want to have the charts checked into their own CI/CD.
And downloading the istio release including both
istioctl + the charts + examples + other tools is not the hardest part.

We also publish a .deb file that includes istioctl and can also include the
templates - and a .rpm is supported ( not part of release, but it's
getting fixed). We can also publish a mac package if someone wants. And
that is the proper way to distribute binary packages with
extra data.
What next, turn istio into a package manager with "apt-get update" support
?
On Wed, Aug 14, 2019 at 10:41 AM Martin Ostrowski<EMAIL_ADDRESS>wrote:

@costinm https://github.com/costinm rather than holding up this PR,
let's discuss it in UX group. I don't have an objection to a different
approach to publishing. I do think that many users will benefit from having
a single line install that points to a URL and given that the manifest
commands are now in istioctl it makes a lot of sense to me to have the
charts in there as well. But this is still alpha so we can get more
feedback, evaluate and make a decision.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/operator/pull/186?email_source=notifications&email_token=AAAUR2TMSLYHV2E47GC6SILQEQ7S5A5CNFSM4ILG2I72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4JRYWI#issuecomment-521346137,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAUR2RAEF5C2U7AMVYKRI3QEQ7S5ANCNFSM4ILG2I7Q
.

@costinm How is doing a download through a specified URL different from a user doing exactly the same thing, with an equally useless sha? Why prevent a useful shortcut from going in?
The format discussion is orthogonal. When we have a format for the charts that's more modern it will be easy and backwards compatible to add it as a target for installPackagePath.

Re SHA: The purpose of the SHA is to allow mirroring - in the old days
binary releases were mirrored by volunteers, over FTP or HTTP, and
the only safe way to download the binary was to check the SHA of the
download against the published SHA in the official web site.
You would go to apache.org, copy the SHA, select a mirror in your region
operated by a 3rd party, get the file - and verify that the
downloaded file was not altered.
We do not do this. Our release is available over https ( which has its own
security ), not from mirrors. It is absurd to check the SHA
of a file if it is downloaded from the same location with the binary - any
hacker can modify both the binary and sha at the same time.
Regarding 'useful shortcut going in' - it is called feature bloat and
complexity. All users have multiple tools to download - from browser
to curl/wget. Re-implementing curl is not providing value to users - quite
the opposite. It's another thing they need to learn and be
confused about. Not to mention concerns over security - few people want
their prod servers to download stuff from internet that runs
with cluster priviledges.
And besides: user already has to download a .tar.gz (or deb, rpm, mac pkg)
to get istioctl. What is the point to download a .tar.gz to
download another one - when the original packet includes the templates
already.
kubectl and go do download, as well as helm - but they do it because they
implements a dependency management system.
I certainly hope we are not going to implement a 'package management
system' and start downloading other operators - instead
of implementing the logic to upgrade and operate istio. And that is the
reason I'm concerned about this direction and focus.
On Thu, Aug 15, 2019 at 3:27 PM Martin Ostrowski<EMAIL_ADDRESS>wrote:

@costinm https://github.com/costinm How is doing a download through a
specified URL different from a user doing exactly the same thing, with an
equally useless sha? Why prevent a useful shortcut from going in?
The format discussion is orthogonal. When we have a format for the charts
that's more modern it will be easy and backwards compatible to add it as a
target for installPackagePath.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/operator/pull/186?email_source=notifications&email_token=AAAUR2WWL6EHU35FCKARGS3QEXJ6PA5CNFSM4ILG2I72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4NFLIA#issuecomment-521819552,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAUR2TJJBM5QPS2EQQOVNTQEXJ6PANCNFSM4ILG2I7Q
.


installGitPreCommitHook task fails when using Git worktree

So there's a problem, directory created by a worktree command doesn't have .git directory.
ChatGPT suggested this fix, and it appears to be working for me. @Jolanrensen what do you think?
val installGitPreCommitHook by tasks.creating(Copy::class) {
    doNotTrackState(/* reasonNotToTrackState = */ "Fails on TeamCity otherwise.")

    val gitHooksDir = File(rootProject.rootDir, ".git/hooks")

    if (gitHooksDir.exists()) {
        from(File(rootProject.rootDir, "gradle/scripts/pre-commit"))
        into(gitHooksDir)
        fileMode = 755
    } else {
        logger.lifecycle("'.git/hooks' directory not found. Skipping installGitPreCommitHook task.")
    }
}

Thanks, AI overloards :)


How does the extension of the Animated API work?

As it is right now, animating markers result in smooth animations when zommed out and very jumpy when zoomed in. I suspect this is because the animation of the marker is done by "stepping" towards the new latitude/longitude, however I can't find it here. When zoomed in the steps are more notable since they cover more of the screen and thus the result becomes very jumpy as opposed to when zoomed out the steps in the animation are very small and look more smooth.
Would it be possible to instead animate to the relative position of the marker on the screen? That way the animation would be smooth regardless of zoom.
Any ideas on how this would be achieved?

I'm also interested to hear how the region is animated in Objc and Java land.

I will close this for now as it's an old issue. Please comment if you want it re-opened but first have a look at the latest code, maybe you can find what you want to know.


More realistic GraphQL vs REST explanation

Hey there! I'm making this PR with the best of intentions, but I'm sure it'll be taken as aggressive. Not the goal.
Recently I've been trying to explain to people the differences between GraphQL and REST and debunk some of the "GraphQL is magically better" stuff going around. Your introduction is furthering the issue.
Generally, GraphQL and REST are just different, and pretending that one is better is going to confuse people.

self-checks embedded on the ground level of your backend architecture

What does this mean

reusable API for different client versions and devices, i.e. no more need in maintaining "/v1" and "/v2"

This was never recommended in REST, in fact the opposite was recommended. Evolvability is king for both.

easily generated documentation and incredibly intuitive way to explore created API

Yeah! Love it, but their schemas are based on concepts like JSON Schema, JSON LD, OpenAPI, etc, and exploring the API is what HATEOAS provides, which is a REST concept.

once your architecture is complete – most client-based changes does not require backend modifications

👍
I had a go at making it a bit more accurate for you, I hope you like it!

@philsturgeon Thank you for your suggestion, I agree with most of your changes but from your permission would like keep some things as they were. I'll put it out there and if you could take a look – I'd appreciate the feedback.
My reasoning is that at this point of time I'd rather have pros exaggerated on purpose to persuade more people to try GraphQL since I experienced its superiority over REST.

Hey! Thanks for getting back to me.
I know you enjoyed using it, and I can tell, but I think what you're suggesting is intentionally misleading people. Emphasising only the pros, ignoring the cons, and failing to advertise differences is going to trick people into using something that might not be right for them.
By all means have a go and I can disagree and who cares because I'm just a person on the internet, but I'd really ask you to keep it realistic and accurate.
--
Phil Sturgeon
Sent from my iPhone and there's probably typos because I'm probably at the pub.

On Feb 5, 2017, at 02:11, Alexandr Viniychuk<EMAIL_ADDRESS>wrote:
@philsturgeon Thank you for your suggestion, I agree with most of your changes but from your permission would like keep some things as they were. I'll put it out there and if you could take a look – I'd appreciate the feedback.
My reasoning is that at this point of time I'd rather have pros exaggerated on purpose to persuade more people to try GraphQL since I experienced its superiority over REST.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey there! It looks like the README is still word for word the same. Maybe you could merge this to get you close, then tweak it to add a few notes about the positive experiences you've had using GraphQL on top of that as a new section?

I agree with Phil's improvements. They are a much more accurate sell for using GraphQL.
Scott

I have to say I agree with Phil's comments. Intentionally leading someone towards graphql isn't doing anyone service. It's not a one size fits all, and depending upon your application needs, REST may work great for you.

I also agree with this PR. GraphQL is not a silver bullet and certainly nothing you want to recommend without context. Its an alternative, sometimes superior solution to a certain problem space, but not all.

I've written a little more on representing state in GraphQL and REST, which might help you visualize the important difference between the two approaches.
https://philsturgeon.uk/api/2017/06/19/representing-state-in-rest-and-graphql/
Please accept this PR.

Thanks @philsturgeon , I'm merging this but making my own changes. I appreciate all the effort but some things I just want to present in a slightly different light. I also firmly believe that GraphQL will become the primary protocol/approach for APIs until something new will render it obsolete and that there's no real reason to have REST and GraphQL except the case when you're dealing with the legacy code or 3rd party services and so on — when you cannot really change or afford changing the API.


Clean-css breaks SVG with PNG fallback (Multiple background-image)

Hello,
I have the following code, which is a known "trick" to provide a fallback for svg. (Who said IE8 ? :))
.myclass {
  background-image: url(...png image...);
  background-image: url(...svg image...), none;
}

Unfortunately, the png background gets eaten during the minification, so, no fallback available.
Any way to get around it?
Thank you for your work :)

Will be a part of clean-css 3.2 due in coming days. Sorry it took a bit longer but it was hard to tweak without rewrite of the internals which is now complete.

@jakubpawlowicz Lol, please don't be sorry. Thank you very much for your work :)

Thanks! :)


static-builds: run docker commands using sudo

normal users might not have the correct permissions to run
docker without sudo.
In addition, as docker will run with sudo, fix permissions
on the qemu and nemu files.
Fixes: #544.
Signed-off-by: Salvador Fuentes<EMAIL_ADDRESS>

/test

timeout in 18.04 CI - rebuild nudged.


[Auto-generated] Check & update dependencies (pyproject.toml)

Update dependencies (pyproject.toml)
Automatically created PR based on the 'CI - Check pyproject.toml dependencies' workflow in SINTEF/ci-cd.

Codecov Report
Patch and project coverage have no change.

Comparison is base (d4f2338) 67.10% compared to head (3eb7ca6) 67.10%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@                  Coverage Diff                   @@
##           ci/dependency-updates      #94   +/-   ##
======================================================
  Coverage                  67.10%   67.10%
======================================================
  Files                         14       14
  Lines                       1146     1146
======================================================
  Hits                         769      769
  Misses                       377      377

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

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


Use correct ACTIVE_CONTENT_LOCALE name in wagtailConfig stubs

Fix up from #11166

See source config usage here:
https://github.com/wagtail/wagtail/blob/3d5c20863cd895a337c3b08345d1ab0c4cff966f/wagtail/admin/templatetags/wagtailadmin_tags.py#L967

Small typo fix, only in JS unit tests, I will merge once CI passes.
FYI @laymonage


How to use celestial body generator?

I keep getting this error whenever I update any values in the generator or click one of the generate options:
IndexOutOfRangeException: Invalid kernelIndex (0) passed, must be non-negative less than 1. ComputeHelper.GetThreadGroupSizes (UnityEngine.ComputeShader compute, System.Int32 kernelIndex) (at Assets/Script Utilities/ComputeHelper.cs:109) ComputeHelper.Run (UnityEngine.ComputeShader cs, System.Int32 numIterationsX, System.Int32 numIterationsY, System.Int32 numIterationsZ, System.Int32 kernelIndex) (at Assets/Script Utilities/ComputeHelper.cs:15) CelestialBodyShading.GenerateShadingData (UnityEngine.ComputeBuffer vertexBuffer) (at Assets/Celestial Body/Scripts/Shading/CelestialBodyShading.cs:45) CelestialBodyGenerator.GenerateTerrainMesh (UnityEngine.Mesh& mesh, System.Int32 resolution) (at Assets/Celestial Body/Scripts/CelestialBodyGenerator.cs:240) CelestialBodyGenerator.HandleEditModeGeneration () (at Assets/Celestial Body/Scripts/CelestialBodyGenerator.cs:115) CelestialBodyGenerator.Update () (at Assets/Celestial Body/Scripts/CelestialBodyGenerator.cs:48)
I just attached the script to the Mesh Holder under Constant Companion, then selected Constant Companion at the body.
What is the correct way to use it?
also, is there anyway to resize the planets/moons

This error isn't related to generating celestial bodies, really. Its roots is in one of the compute shaders not compiling. Judging by the name Constant Companion, it is a MoonShading.compute shader. (located in Assets/Scripts/Celestial/Shaders/Compute/Shading)
Possible solution: at line 23, change this
float sphereNoise = 1.#INF;
to this
float sphereNoise = 3.402823466e+38F;
To change the planets' size, select the planet you want to resize, then in Inspector in Celestial Body (Script) change its radius.
P.S. Also guessing by what you described, CelestialBodyGenerator isn't supposed to be added manually as a script to a celestial body - it's a script that will get attached to an object automatically afterwards in the process of creating a Solar System. What you really should be doing is creating new (or editing existing) Celestial Body Settings. There is a scene Planet to make that process easier.


Improvement/centralize makefile

Updated Makefile to build the logger then the kmyth tool
Removed the logger-specific Makefile
Updated doc to reflect CentOS 8
Removed the OpenSSL dir options needed for CentOS 7

This should address what we wanted with issue #4


StartedTags StoppedTags timezone wrong

I have a StartedTags and StoppedTags set using the {timezone} place holder.
I have DefaultTimezone set to UTC.
I have created a Schedule with a timezone setting of America/New_York.
When the instance was stopped using that schedule it was successfully tagged with:
ScheduleMessage: Stopped on 2019/05/22 at 22:55 UTC
You can see that the {timezone} placeholder inserted UTC for the timezone when I think it should have used 18:55 America/New_York.  It seems like the {timezone} placeholder and the time value is using the default timezone rather than the timezone from the Schedule.
Could this be modified to use the timezone setting from the Schedule if there is one?

Bij design the date and time are always in UTC timezone.

Ok, thanks for looking.
I just thought it made sense that if a Schedule had a timezone defined, and the instance was started or stopped based on that timezone then the time in the tags would reflect that timezone setting.
Not a big deal though, just a small, visual preference.

Close as resolved.


Can't scroll in social menu if Sidekick is opened

Maybe a conflict with other programs running, I'll try to reproduce.

Can't reproduce.


Add log4j2 hooks to standard java logging

As suggested by @housejester  in https://github.com/druid-io/druid/pull/1140

probably will need merge conflict resolution after https://github.com/druid-io/druid/pull/1140 goes in

There was some internal discussion on if the log4j2 log manager could have the same performance impact as doing an slf4j bridge
I'm not convinced slf4j bridge for jul is the best option right now solely for the sake of arbitrary compatibility without some performance tests to make sure it doesn't tank logging performance.


Bug Report: When loading the third song, it crashes without giving any error messages

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
64
What version of the game are you using? Look in the bottom left corner of the main menu.
1.5.4 EK Kade Engine
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
No I haven't, I didn't know what was the bug since it crashes without giving error messages
Please describe your issue. Provide extensive detail and images if possible.
I was adding dialogues in the fnf mod then I went to test it, everything was fine until the third song loads, when the third song loads, it crashes, I checked mistakes in the source code but I didn't end up finding one
How do I fix this?

I am experiencing this as well.


unresponsive actions in beta 6

List of actions that seem to do nothing with beta 6:

Add Color Wheels Effect
Add Color Board Effect
Add Color Curves Effect
Add Hue Sat Effect
Toggle Color Corr Effects on/off
Toggle Fullscreen
Disable
tbc

These are Keyboard Shortcut actiosn. Do you actually have something assigned to "Add Color Wheels Effect" in the Final Cut Pro Command Editor? If not, you should have seen a macOS notification like this:

I'm going to close this issue for now, as I'm ASSUMING that you are just using a Command Set that doesn't have shortcuts assigned to these actions?


More intrusive snippet suggestions after latest insiders update

VSCode Version:
1.29.0-insider
931c22033db3cc8d5dff265889307ed4076ac2d3
x64

OS Version: Ubuntu 18.04.1 LTS

Steps to Reproduce (just one example below):

Type first three characters of a jsDoc or tsDoc comment (/** [*/] <- section in square brackets auto-completed, great!)
Now press enter. If you have custom snippets, one of your custom snippets will be selected, even though it is clearly a comment. In a previous version, this didn't happen.

Another example is that the user setting snippet-suggestions: top setting also makes snippet suggestions much more aggressive than before. They crowd out all other auto-complete suggestions even though the correlation between what I am typing and the actual suggestion is very loose.

/duplicate of #60746


Cannot keep another pod in same space as the one running hsds

Hi,
I observed that the following line throws an error if we have any pod apart from hsds running in our namespace
https://github.com/HDFGroup/hsds/blob/master/hsds/basenode.py#L305
Kindly look into the same.
Thanks,
Akshay

Does this update fix the issue: https://github.com/HDFGroup/hsds/commit/6cc3a032d07fbb1380ee3360c89cf5f052c8fcc1?

Issue fixed. Thank you.


App logs only tail with follow param set

Viewing logs for an app in the dashboard is broken due to the changes in how the follow param is handled by /apps/:app_id/log (only lines added after opening the stream are displayed).

This also effects the command line, which used to stream the previous logs also.

This should probably updated on the server end then as clients expect to receive the full log


chore: update readme w/ available modules

Updates the README.md file with all the available PS modules.
Closes #296

Aha! Let me change that. Thx!

Thx for the reminder! Wasn't paying attention. 😏


Medication

Medication
As soon as the seeding data is available, the mapper for the Firestore document for the MedicationDetails object must be created and the observed data used in the ViewModel.
:gear: Release Notes

MedicationCard
MedicationProgressBar
MedicationStatusIcon
MedicationList
DosageInformation
MedicationViewModel, MedicationScreen, MedicationUiState and MedicationUiStateMapper
MedicationRepository

:books: Documentation

Dark Mode
Light Mode

:pencil: Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

Thanks for the feedback and for pointing out the behaviour of the progress bar, @PSchmiedmayer. Is this a typical behaviour for iOS, @nriedman? I haven't seen this dynamic behaviour for Android, but the Compose Progress Bar doesn't even offer labels at all. I would have to implement this customised.

This looks great! Nice work @Basler182.
I did end up implementing custom styling for the dynamically aligned progress bar @Basler182. Having the label left aligned is also not default iOS behavior, but since I already implemented custom styling, switching to having it left aligned would be very simple. I like how left aligned looks, and I think it does a good job communicating the key points of the bar too -- I can go ahead and incorporate that into my next PR. Let me know what you think @PSchmiedmayer!

@nriedman @Basler182 I agree; the left alignment communicates this well. Nothing to change in a rush @nriedman, but I can see this as a nice option or flag to pass into the progress view that indicates if the label should be rendered underneath the current progress or left aligned. We can then set it to true for now but still have the logic in place if we would
a) Move it to SpeziViews or other packages
b) Change our mind later on
For Android: Left alignment looks great; let's stick with that!

Hey @Basler182, you can have a look at my commits but I am summarizing here what I applied so far after aligning with @pauljohanneskraft

Add component to read localized strings from firebase responses and using it for videos and medicationRecommendations
Retrieve medicationRecommendations from firebase
Introduced an empty screen state
Adaption of dose values after syncing with @pauljohanneskraft (see MedicationUiStateMapper#dosageValues
Refactor of composables and medication details model. Introduced new ui models that represent exactly what we display in a card. The ui model is created once in the mapper when we retrieve updates from firebase repository. MedicationUiStateMapper is now responsible to calculate the info to display in the card. The benefit of this approach is that the ui data is created once. So far we were doing the calculations inside composables e.g. summing the dosages. We now save these operations in case of recompositions and activity recreations in case of configuration changes

It looks very good from my perspective so far. We still need to add and extend unit tests for these components:

MedicationRecommendationMapper
MedicationRepository
MedicationUiStateMapper
MedicationViewModel

I can add those during my flight back or next time I will be able to work again (latest next weekend). Or we create an issue for the unit tests and merge this PR as is.

@eldcn those are great changes. 🚀  i just looked at it and from my side it looks great too. 👍  i would also say if you add the tests on the flight it's great, if not it's no problem, then we merge the state as it is and create the issue so one of us can catch up when we have time

@Basler182 @eldcn Would be great if we can merge this PR by the meeting tomorrow.
Please also check with @pauljohanneskraft to send you the testing credentials so we can test that that Android app displays the content as expected on Android for a demo tomorrow. It would be great to do the demo from an actual device with Bluetooth connections & the Firebase Datastore 👍


v3.3.1  (IDFGH-4915)

i used by git.And now,i try to run again from cmd,its same erros as follows:
C:\msys32\esp-idf-v3.3>install.bat
Installing ESP-IDF tools
Installing tools: xtensa-esp32-elf, esp32ulp-elf, cmake, openocd-esp32, mconf, ninja, idf-exe, ccache
Skipping<EMAIL_ADDRESS>(already installed)
Skipping<EMAIL_ADDRESS>(already installed)
Skipping<EMAIL_ADDRESS>(already installed)
Skipping<EMAIL_ADDRESS>(already installed)
Skipping<EMAIL_ADDRESS>(already installed)
Skipping<EMAIL_ADDRESS>(already installed)
Skipping<EMAIL_ADDRESS>(already installed)
Skipping<EMAIL_ADDRESS>(already installed)
Setting up Python environment
fatal: No names found, cannot describe anything.
Traceback (most recent call last):
File "C:\msys32\esp-idf-v3.3\tools\idf_tools.py", line 1358, in
main(sys.argv[1:])
File "C:\msys32\esp-idf-v3.3\tools\idf_tools.py", line 1354, in main
action_func(args)
File "C:\msys32\esp-idf-v3.3\tools\idf_tools.py", line 1144, in action_install_python_env
idf_python_env_path, _, virtualenv_python = get_python_env_path()
File "C:\msys32\esp-idf-v3.3\tools\idf_tools.py", line 840, in get_python_env_path
idf_version_str = subprocess.check_output(['git', '--work-tree=' + global_idf_path, 'describe'], cwd=global_idf_path, env=os.environ).decode()
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\subprocess.py", line 411, in check_output
**kwargs).stdout
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\subprocess.py", line 512, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['git', '--work-tree=C:\msys32\esp-idf-v3.3', 'describe']' returned non-zero exit status 128.

@zhy102 You don't have a git installed for Windows. The one installed in MSYS32 is not working from Windows command line.
You can use the Windows tool istaller to install everything you need:
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/get-started/windows-setup.html#esp-idf-tools-installer

I'm linking here my previous comment: https://github.com/espressif/esp-idf/issues/6705#issuecomment-797962639

I am sorry, but it still  when  i  run install.bat from Windows command line. Later versions of ESP-IDF support powershell as well.
---Original---
From: "Roland @.>
Date: Mon, Mar 15, 2021 18:13 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [espressif/esp-idf] v3.3.1  (IDFGH-4915) (#6707)
I'm linking here my previous comment: #6705 (comment)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Have you tried the Windows tool installer linked above?

I'm closing this in favor of https://github.com/espressif/esp-idf/issues/6775