refactor(EKS/LegacyClusterAPI.CreateCluster): used CreateNodePoolWorkflow for node pool creation

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
-

License
Apache 2.0

What's in this PR?

Refactored the legacy EKS ClusterAPI.CreateCluster (and subsequently EKS CreateInfrastrcutureWorkflow) implementations to use the recently introduced CreateNodePoolWorkflow for node pool creation.
Why?

in order to move towards a unified node pool creation solution, deduplicate business logic code and simplify expectations.
Additional context

Advised to review by commit.
Another PR is coming where the legacy EKS ClusterAPI.UpdateCluster implementation is refactored accordingly, but it requires a bit more refactoring due to me missing a chance of refactor when the DeleteNodePoolWorkflow was introduced, so I decided to split the two apart into their own PRs for agility reasons.
Checklist

[ ] Implementation tested (with at least one cloud provider)
[X] Code meets the Developer Guide
~OpenAPI and Postman files updated (if needed)~
~User guide and development docs updated (if needed)~
~Related Helm chart(s) updated (if needed)~

To Do

[ ] Manual test.

I have added a commit making label set creation optional in the CreateNodePoolWorkflow, because the order of execution in a ClusterAPI.CreateCluster call understandably causes the node pool creation to run before the node pool label set operator is installed.
So label set creation should be turned off for cluster creation cases and on for other cases including NodePoolAPI.CreateNodePool.
In case of cluster creation, the cluster creation workflow makes sure the node pool labels are synced to the existing node pools once the operator is installed, so no additional operations are required then.

Had to fix the node instance role handling making sure to always use the resource ID only, stripping the paths specified for explicit node instance role ID cluster creation.


ws on port 800 doesn't work, ws :8888 does

Environment

VerneMQ Version: 1.2.0
OS: Ubuntu 17.10

Expected behavior

Configured listener.ws.default = <IP_ADDRESS>:800
Vernemq should listen to port 800 for websocket connection

Actual behaviour

It did not. Two developers here actually spent 9 hours each trying to figure out why the project didn't work when ran from browser with ws, instead of tcp, like from the node-clients, android, etc. Using the "idiot-method", I followed the examples, and baam, it worked when using 8888.

@ozra thanks for your effort & info, great to see you using VerneMQ.
The problem has to do with Linux privileged ports (< 1024), I think; unless I misunderstand what you wish to achieve.

Aha, I'll forward that information! This was clearly a lack of knowledge in the team and on my part - I'll close the issue in shame and gratitude XD.

It's certainly not your fault :)
& it's an example from the template: https://github.com/erlio/vernemq/blob/master/apps/vmq_server/priv/vmq_server.schema#L524
I'll add a note about that right there or in the docs.
Thanks again for bringing it to our attention!


lazy_static is blocking

As discussed in https://github.com/crossbeam-rs/crossbeam/pull/122#issuecomment-288436809, lazy_static is blocking.  It is used for the global registries, the garbage queue, and the global epoch.  It would be great if accessing them is non-blocking.

It's only blocking in the slow path due to Once, it only does a seqcst load on the fast-path  (https://github.com/rust-lang/rust/blob/c7104be1a37daa2f7ec114913247dc93524fd48e/src/libstd/sync/once.rs#L220)

Should we close this? (based on the above comment)

I feel we cannot easily change the current state of affair.  I'm closing this now, but please feel free to reopen it.


chore: add relevant information to logs

what do you think about adding a timestamp (like Date.now())?

The logger already prints the timestamps


VR Camera Control Improvements

Applying VR offset after occlusion check eliminates weird behavior when clipping camera into parts
Changed IsVREnabled references to new VREnabled
Changed pitch min/max to 30 degrees instead of 80, goes with following change
Negated pitching of the camera after VR offset by a variable fraction, set to 50% right now.

We CANNOT accept commits from an unknown author. You need to set up your git username and email and then fix-up your commits before we can consider merging the pull request. If you aren't sure how to do it there is a guide on the git first-time setup page: https://git-scm.com/book/en/v2/Getting-Started-First-Time-Git-Setup

When fixing up old commits that are still pending in pull requests squash them down using rebase and the 'fix-up' command

Not sure what happened with the author; SourceTree seems to have lost my configuration at some point. Will fix.


How to change text

Hey! I was wondering how to change the text in the lobby. I saw the "lobby_text_bc.png" image but had no idea what to edit. Any help would be appreciated!

For performance reasons the text is baked into the scene. If you select the text in the editor, you can see that several pieces like the Lobby, Roster Panel, etc. are baked into a single mesh called lobby_text_sm. If you want to change any part of this, you'll need to replace this asset. We suggest using TextMesh Pro which is included in the project under the Assets directory.


GitHub login not connecting to existing accounts properly

@gutierri reports:

I connected with my github account that has the same email I've ever had on the site (CodeCombat), but after connecting a new account which had no relation to the existing was created.

Not sure if this is proprosital, but if it is, it would be interesting to connect with an account that email there is already quite interesting adicionair just another means of connection and not create a new account.

Also got another report:

I think your github integration might be broken? I logged in with it as I'd forgotten my password, but it's logged me in as someone called 'Wolfenswan'? (That's not my account...)

Sounds like the same thing as #1749. Let's just fix it as part of re-enabling it with #1764.


LJ Dataset Silence

I downloaded the LJ dataset and trained with it, it seems to have about 1 second of silence at the end of each file, which is causing a flat line in my alignment
Did anyone else run into this problem? Did you clean the LJ dataset before training?

See https://github.com/Yeongtae/tacotron2/blob/monitoring_metric/preprocess_dataset.py
I'm preparing a pull request for monitoring the sequence to sequence model and preprocessing dataset.

thank you

@Yeongtae
I used your scripts, and check the files, there is no silence in beginning and end, but the predicted and target mel still show this at the end

@yondu22
Train again with preprocessing.

@Yeongtae I did run your prepprocessing and checked the files, it was great it removed the beginning and end silence. I read on the interenet that the yellow part in the mel frequency is from some files that are padded?

It seems that your model fails to gate prediction to find the end point of prediction.
Would you like to know the reason why your model fails?

@Yeongtae yes please tell me, i just used the lj dataset from the original source, and then ran your preprocess and use the default hyper paramaters

@yondu22
To learn the gate prediction layer, there is a need to prepare data(char, mel) for the endpoint, such as end char '~' at the end of all utterance and silent Mel-spectrograms.
These data are represented by using Encoder and Prenet of Tacotron2.
If there is no data for the endpoint, It means there is no clue for the endpoint.
So I recommend you to use '~' and small silence audio padding, such as 5 times of hop size.
It makes your model able to predict the end point.

oh what about the punctuation in the data like . ! and ? do these work for endpoints?

@yondu22
They work for it.
But the model cant predict endpoints with out them.

you should double check the audio created from your preprepossing on the original LJ dataset, it decreased the volume and made them sound robotic with some static noise

you should double check the audio created from your preprepossing on the original LJ dataset, it decreased the volume and made them sound robotic with some static noise

Thanks. I will check it.

These are example audios of my preprocessing.
wavs.zip
Is there any problem?

Volume decreases because of preemphasis.
In general, it makes the noise of the original audio less.

ok check out one example from the LJ dataset text but different voice, before and after using the processing
LJ028-0270|This was said by a Babylonian who thought that a mule would never foal.|This was said by a Babylonian who thought that a mule would never foal.
processing.zip

ok check out one example from the LJ dataset text but different voice, before and after using the processing
LJ028-0270|This was said by a Babylonian who thought that a mule would never foal.|This was said by a Babylonian who thought that a mule would never foal.
processing.zip

You can skip the preemphasis without using -p argument.

Closing due to inactivity.


Build 1st layer of TCF modal with stacks

Is your feature request related to a specific problem?
As part of TCF requirements, we need a "first layer" of the modal which shows stacks (groups of purposes)
Describe the solution you'd like
Will fill in more after some team discussion...
Describe alternatives you've considered, if any
A description of any alternative solutions or features you've considered.
Additional context
Design: https://www.figma.com/file/dmEwdK3xZwjKfGVQ9t66xe/Fides-v.2-Master-Working-Files?type=design&node-id=7781-250419&mode=design&t=PGSiiqpK8HENjzjY-0

closing


[bug] Ensure Z-Wave JS external config directory exists

Version
Checklist:

[ ] I am not using HomeAssistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog problem was not mentioned there.

Build/Run method

[X] Docker
[ ] PKG
[ ] Snap package
[ ] Manually built (git clone - yarn install - yarn run build )

zwavejs2mqtt version: 4.2.0
zwave-js version: 7.4.0
Describe the bug
The .config-db directory is not populated. This is because nothing creates the directory. ZWave-JS expects the directory to be created already. After manually creating the directory in my store volume, it gets populated.
To Reproduce
Steps to reproduce the behavior:

Run the docker image
Notice the .config-db is not created or populated.

Expected behavior
The container startup scripts should create the directory automatically, or upstream zwave-js driver should do it.
Additional context
None.

For me, rolling back to 4.1.0 does not fix my problem, but 4.0.0 does.


Fix typos in create-node-lambda.js

Current coverage is 43.50% (diff: 100%)

Merging #49 into master will not change coverage

@@             master        #49   diff @@
==========================================
  Files            37         37
  Lines          1602       1602
  Methods         166        166
  Messages          0          0
  Branches        156        156
==========================================
  Hits            697        697
  Misses          905        905
  Partials          0          0

Powered by Codecov. Last update c39dfab...c2ddd9a


[Task] shenyu-client-grpc test case

Description
Add test case for shenyu-client-grpc
Task List

[ ] Add test case for JsonForwardingServerCall
[ ] Add test case for JsonServerCallListener
[ ] Add test case for JsonServerServiceInterceptor
[ ] Add test case for GrpcServerRunner
[ ] Add test case for GrpcClientEventListener

Hello, I would like to try this, can I attempt this? if so please assign it to me, I'll try to complete it


[AutoPR compute/resource-manager] Images swagger

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

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


Update UI to allow users to upload their own documents.

[x] Create a new page for Documents.

[x] Add upload button for users to upload (limited to PDFs).
[x] Add a table / list of existing documents and their processing status.

[ ] Use PrepareDocs Azure Function endpoint to upload, embed, and index documents.

Fixed by #147


Terminal Plus will not let me type text or connect to my terminal

Uninstall and reinstall completed

Tried:   apm install terminal-plus( nothing )

All I see is the yellow terminal and blinking cursor

I just installed terminal-plus on a fresh install of atom and I'm getting exactly this behavior. just getting a blank terminal with blinking cursor and no shell prompt or error.

Same here on Windows 7. Tried uninstalling and reinstalling from both Atom and command line. Is this a permissions thing (E.g. running as admin?)

It works for me with atom 1.6.2, but it does not work (same behavior reported by others) with 1.9.0-beta0.

Try https://github.com/platformio/platformio-atom-ide-terminal

I also have this issue, and using the above link does not help.
With terminal-plus I get a blank screen, with platformio-atom-ide-terminal I get the following error:
execvp(3) failed.: No such file or directory

See issue here
Anyone know how to address this issue?

Same problem here.

Same here.

same here

Here's a temporary workaround: https://github.com/jeremyramin/terminal-plus/issues/402

@claudiodangelis thank you so much!

Uninstalled package

Same issue 👎
Uninstalled package

This package is a fork of terminal-plus and doesn't exhibit this bug ( tested on Mac running High Sierra)
https://atom.io/packages/tokamak-terminal

your terminal might be outdated , update it and restart atom

In Mac

Open Atom and use shortcut to open setting
Cmd  ,
At Search Package, type
terminal-plus
Click disable
At Search Package, type
platformio-ide-terminal
Install platformio-ide-terminal and click Enable
Restart Atom

Try uninstall platformio-ide-terminal and install it again ....and also check shell override field for any leading or trailing spaces.


Getting rollback error when dropping an svs file for patch creation

Hi,
First of all thank you for creating this wonderful utility that speeds the workflow from UI--->GPU-->UI. It is perfect, and really nice to see features like intelligent patch suggestion. Kudos! I hope it will work for me.
I installed in a new conda environment and started the app. Then connecting to port 5555, created a project and tried to drop a file. Thats when I see this error in the logs (the file drop is not successful). These are the last few lines in the log. Any pointers what might be wrong?
I tried the docker route too, but there is some error with that also.
Thanks,
Darshat
FROM job
WHERE job."projId" = ? AND job."imageId" IS NULL AND job.cmd = ? AND job.status IN (?, ?)
LIMIT ? OFFSET ?
2022-01-27 21:24:21,420 [INFO] ('1', 'make_embed', 'RUNNING', 'QUEUE', 1, 0)
2022-01-27 21:24:21,426 [INFO] SELECT count() AS count_1
FROM (SELECT job.id AS job_id, job."projId" AS "job_projId", job."imageId" AS "job_imageId", job.cmd AS job_cmd, job.params AS job_params, job.status AS job_status, job.retval AS job_retval, job.start_date AS job_start_date
FROM job
WHERE job."projId" = ? AND job."imageId" IS NULL AND job.cmd = ? AND job.status IN (?, ?) ORDER BY job.id ASC
LIMIT ? OFFSET ?) AS anon_1
2022-01-27 21:24:21,426 [INFO] ('1', 'make_embed', 'RUNNING', 'QUEUE', 1, 0)
2022-01-27 21:24:21,427 [INFO] ROLLBACK*

Thanks for your interest in our tool!
The subject of your issue is a bit curious: "dropping an svs file", as
quick annotator does not yet support SVS files directly : )
You should extract the associated tiles as png/tif and upload those; we
provide a script for exactly this purpose.
Have you seen this part of the wiki?
https://github.com/choosehappy/QuickAnnotator/wiki/Image-List-Page
On Thu, Jan 27, 2022 at 9:04 AM darshats @.***> wrote:

Hi,
First of all thank you for creating this wonderful utility that speeds the
workflow from UI--->GPU-->UI. It is perfect, and really nice to see
features like intelligent patch suggestion. Kudos! I hope it will work for
me.
I installed in a new conda environment and started the app. Then
connecting to port 5555, created a project and tried to drop a file. Thats
when I see this error in the logs (the file drop is not successful). These
are the last few lines in the log. Any pointers what might be wrong?
I tried the docker route too, but there is some error with that also.
Thanks,
Darshat
FROM job WHERE job."projId" = ? AND job."imageId" IS NULL AND job.cmd = ?
AND job.status IN (?, ?) LIMIT ? OFFSET ? 2022-01-27 21:24:21,420 [INFO]
('1', 'make_embed', 'RUNNING', 'QUEUE', 1, 0) 2022-01-27 21:24:21,426
[INFO] SELECT count() AS count_1
FROM (SELECT job.id AS job_id, job."projId" AS "job_projId",
job."imageId" AS "job_imageId", job.cmd AS job_cmd, job.params AS
job_params, job.status AS job_status, job.retval AS job_retval,
job.start_date AS job_start_date
FROM job
WHERE job."projId" = ? AND job."imageId" IS NULL AND job.cmd = ? AND
job.status IN (?, ?) ORDER BY job.id ASC
LIMIT ? OFFSET ?) AS anon_1
2022-01-27 21:24:21,426 [INFO] ('1', 'make_embed', 'RUNNING', 'QUEUE', 1,
0)
2022-01-27 21:24:21,427 [INFO] ROLLBACK*
—
Reply to this email directly, view it on GitHub
https://github.com/choosehappy/QuickAnnotator/issues/16, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACJ3XTDOVAMFQZ6L2QUOH3DUYFUIZANCNFSM5M6LFHBQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Great, I'm happy to know its not some version issue. I will try this out and revert back. Thanks for your prompt response!

It is working now. This is a gem of a tool! It allows to reuse all the IP we have in python with a minimal setup. Code is neatly factored, and the db based workflow engine is really perfect to bridge UI to GPU. Thanks for doing this!
We will try to see if an auth head can be put on this so multiple annotator scenarios can be supported.

Thanks for the feedback!
Multi-class is somewhere on our development list, but it will be an undertaking and we're waiting to see how well adoption of the tool is to determine if the investment is worthwhile
Practically speaking, my preference lately has in fact been to make numerous binary classifiers (1 vs all), i.e., 1 "segmentor" per target class.
The logic there is multi-fold:

its easier to lock down a single classifier when it works, and then move on to the next class without impacting the first class
training tends to be faster and converge more quickly
usually the best way to improve the model requires getting more samples of a specific sample, and "fully annotating" each every ROI for each multi-class quickly becomes onerous. If one only annotates one class and labels the rest as "unknown", this would work but very quickly you end up with a large amount of computation time which isn't involved in backpropogation, leading to 100h of training time that could have been equally accomplished with 1-2hs of a targeted model
when (eventually) someone wants to add yet another class to their dataset, instead of fully retraining a large model with a large dataset in a multi-class situation, they only need to train a smaller model (since less complexity is involved in detecting a single class), with less data, and can do it more quickly without "disturbing" previously validated models

Some quick thoughts, but this is the motivation of why quick annotator was initially designed to use a single class instead of multi class


ubuntu xenial warning during activate gpg-check for yum-repos

ubuntu 16.04
ansible 2.1.2
dev-sec.os-hardening 3.1
Result:
TASK [dev-sec.os-hardening : activate gpg-check for yum-repos] *****************
[DEPRECATION WARNING]: Skipping task due to undefined Error, in the future this will be a fatal error.: 'dict object'
has no attribute 'stdout_lines'.
This feature will be removed in a future release. Deprecation warnings can be disabled
by setting deprecation_warnings=False in ansible.cfg.

Thanks for reporting this @ahrib! The referenced commit fixes this.


Rename -es5 to -legacy, to reduce confusion over what's actually supported  (issue 12976)

Please note that this will also require some edits of the Wiki.
Fixes #12976

Note: when this gets merged, also merge https://github.com/mozilla/botio-files-pdfjs/pull/29.

Looks good to me too. Thanks!


Correction for raw affiliation Univ Rennes, INSERM, BRM (Bacterial Regulatory RNAs & Medecine),Rennes,France,F-35000

Correction needed for raw affiliation Univ Rennes, INSERM, BRM (Bacterial Regulatory RNAs & Medecine),Rennes,France,F-35000
raw_affiliation_name: Univ Rennes, INSERM, BRM (Bacterial Regulatory RNAs & Medecine),Rennes,France,F-35000
new_rors: 015m7wh34;02vjkv261;031y0x195
previous_rors: 015m7wh34;02vjkv261
works_examples: W4389077574
contact: 2ce0a338128ebb0976b3cd0207eeb6db:8b7c5d0a2b686572ccb99c4e3ca2ad71 @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


TokenHelper in AppForSharePointWebToolkit fails to validate host URL with port number

Category

[ ] Question
[ ] Typo
[x] Bug
[ ] Additional article idea

Expected or Desired Behavior
CreateAppEventClientContext should work when using ACS and nonstandard port for provider hosted add-in (like when using Visual Studio Development Server).
Observed Behavior
If using non default port for provider hosted add-in (for example VS Development Server), you will get
an AudienceUriValidationFailedException with the message: "localhost" is not the intended audience "b99d1203-1862-4c3a-9947-45663236d2ee/localhost:44349@12768036-4522-497f-833f-7ad2d7e89856" when creating an App Event Client Context (TokenHelper.CreateAppEventClientContext)
Steps to Reproduce
Create a new SharePoint provider hosted Add In project using ACS and select either SharePoint 2013 or SharePoint 2016 as the minimum version (I have not testet for SharePoint Online, but suspect the same issue there as well).
Enable Handle Add-in Installed.
Set a breakpoint on line 130 in TokenHelper.cs (inside method ReadAndValidateContextToken on the line that starts with: if (StringComparer.OrdinalIgnoreCase.Equals(token.Audience, principal)))
Debug the project.
The breakpoint will be hit and the principal does not match token.Audience because of missing port number.
Creating a token like the one in the default HomeController.cs works as expected with the port number applied.
The problem lies in that the method CreateAcsClientContextForUrl calls ReadAndValidateContextToken only including the host and not the port number.
Not sure if this is the right channel, but I get no response from contacting the AppForSharePointWebToolkit NuGet package owners (which contains the TokenHelper.cs)
Workaround is to manually modify the TokenHelper but it would be best to have this fixed in the NuGet package.

@jensotto - Thanks for your feedback.  Token helper is provided as a starting point that works with the default behavior.  If you have a non standard site (such as one that uses a different port), you can modify token helper to accommodate this change.  It lives in your solution and is fine to be different.

@mcmynn83 didn't you read my explanation? I don't have a non standard site. I'm using the default and standard developer web server provided by Visual Studio. This does not work, and it should. It works for all situations except when you try to create an app-only context. This should work as this is what is provided by Microsoft.

@mcmynn83 the end result is that you can't debug a project with the current tokenhelper. Is that what you want developers to struggle with?

@mcmynn83 you state that the default behaviour works, but that is NOT correct. The default behaviour requires tokenhelper to include the port number as Visual Studio Development Server uses non default ports, and it's impossible for the user to modify that. Based on that your conclusion is wrong and this issue should therefore be reopened.

Let's reopen this for further investigation and proper testing also with SharePoint Online. @jensotto there are now quite a few different statements in the comments here and would appreciate following clarifications, which would speed up the possible resolution - if possible. We can surely do the end-to-end testing also on our side if needed - but getting input for following would be helpful

In the original description, you did not mention app-only, but then mentioned following in the follow-up comment - "It works for all situations except when you try to create an app-only context". So the issue is only with app-only then?

You are 100% correct that the IIS Express is using non-standard ports and if that's an issue, we absolutely should ensure that this works properly. I did not yet test the issue personally due time constraints, but would be highly surprised if we have changed anything in this implementation since 2013 and never remember to run into this personally when doing similar development... obviously situation might have changed, just puzzled on the issue which we regardless need to investigate and potentially fix.

@VesaJuvonen I could probably have been a bit clearer in the original post. You are right in that it affects the app-only context. The problematic method is the TokenHelper.CreateAcsClientContextForUrl method. This method is used in two other methods: TokenHelper.CreateRemoteEventReceiverClientContext and TokenHelper.CreateAppEventClientContext
If you use any of these two app-only context methods you will have the issue with missing port number.The reason the method does not work properly is the call to the TokenHelper.ReadAndValidateContextToken
method. This method accepts a parameter named appHostName. In the description it clearly states that this must include the port number: "(...)The URL authority, consisting of  Domain Name System (DNS) host name or IP address and the port number, to use for token audience validation. (...)"
This is not done in the TokenHelper.CreateAcsClientContextForUrl method as only the host name is included due to the usage of OperationContext.Current.IncomingMessageHeaders.To.Host.
TokenHelper.ReadAndValidateContextToken is correctly used in SharePointContext.CreateSharePointContext. Here appHostName is correctly set by using the property httpRequest.Url.Authority.
In light of this I suggest changing OperationContext.Current.IncomingMessageHeaders.To.Host in TokenHelper.CreateAcsClientContextForUrl to OperationContext.Current.IncomingMessageHeaders.To.Authority


support codecov

Closes #81
📑 Description

More setting related to CodeCov ( https://app.codecov.io/ ) needs to be done on repo author's side. @ProKil

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed
[x] Branch name follows type/descript (e.g. feature/add-llm-agents)
[x] Ready for code review

ℹ Additional Information

hi @lwaekfjlk I have added a CODECOV_TOKEN to the secrets, and codecov asks me to add a new github action:
- name: Upload coverage reports to Codecov
  uses<EMAIL_ADDRESS>  with:
    token: ${{ secrets.CODECOV_TOKEN }}

Could you help with adding this github action?

ok, will do later.

@ProKil Please check the codecov report.

Thanks! Would this be displayed on the readme.md?

for badge, should look something like this:


[User Story] - notation verify - remote artifacts

As a verifier, I want to verify remote artifact integrity with remote signatures using my local trust policy and store.
Note: verify with local key, verify remote artifacts.  Verify local artifacts out of scope RC-1.
Implementation items to complete story:

[ ] https://github.com/notaryproject/notation/issues/228

[ ] https://github.com/notaryproject/notation/issues/216

Migrating this item to https://github.com/notaryproject/roadmap/issues/41 so closing it out.


Run Queue Without Job chaining

Requirements
Please take note of our contributing guidelines: https://docs.laravel-excel.com/3.1/getting-started/contributing.html
Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
Mark the following tasks as done:

[x] Checked the codebase to ensure that your feature doesn't already exist.
[x ] Checked the pull requests to ensure that another person hasn't already submitted the feature or fix.
[ ] Adjusted the Documentation.
[ ] Added tests to ensure against regression.

Description of the Change
This adds a concern which will dispatch queued jobs without chaining.
Why Should This Be Added?
It provides a solution for two issues I ran into with chaining. (1) For extra large files being processed the size of the chain exceeded the size limits of the queue. (2) Chaining runs each job subsequently so you lose the speed gains from running concurrent jobs.
Benefits
Your processing of a file can run jobs concurrently and thus be a lot faster.
Possible Drawbacks
Because the jobs can be run concurrently the AfterImportJob is ignored.
Verification Process
I've tested it and it works well. Because it's a new concern and the limited scope in which it works it does not break anything else.
Applicable Issues
None

Debating here if the name should change to ShouldQueueWithoutChaining or QueueWithoutChaining since it could effectively replace ShouldQueue.

Can you provide some tests and a documentation update, thanks.

Ping @jonrobinson

@patrickbrouwers Yeah sorry, crazy last few weeks. Will try to take care of this soon. One caveat I'm just realizing now is that we can't specify the queue when I add this.
Ideas on how to solve that?
The only thing I can think of at the moment is adding a 4th parameter to the queue() function which is the queue name.

@jonrobinson no problem!
Perhaps you could continue this PR: https://github.com/Maatwebsite/Laravel-Excel/pull/2513 and pass those kind of queue props to the actual jobs?

@patrickbrouwers ah yes. I think that would work!

@patrickbrouwers A little late but we got it done if you want to review.

@jonrobinson no worries, sometimes things get in the way :) I'll have a look on monday!


bucketing info parameter does not accept camel cased column names

Describe the bug
Describe the bug
The awswrangler.s3.to_parquet takes the name of the column to bucket the data. And fails if a column name is camel case or contains caps.
My assumption is, somewhere before the bucketing step, we are doing a to_lower operation, and then applying group by, which means we should also apply the to_lower operation to the bucketing column name.
How to Reproduce

Pass a df with came cased column names.
Pass a camel case column name to bucketing_info.
Expected behavior
Expected behaviour should be to handle the bucketing column name irrespective of camel case.

Expected behavior
No response
Your project
No response
Screenshots
No response
OS
Mac
Python version
python3.8
AWS SDK for pandas version
2.20.1
Additional context
No response

This should have been fixed in 2.21.0 as mentioned here


Update gleam-for-elixir-users.md

Todo

[ ] Flow Control

[ ] Case
[ ] Try

[ ] Type aliases
[ ] Custom types

[ ] Records
[ ] Unions
[ ] Opaque custom types

[ ] Modules

[ ] Imports
[ ] Nested modules
[ ] First class modules

Would be good to explain that Gleam has exhaustiveness checking here

The idea of using let assert is to let the readers know the equivalent feature in Gleam. However, I will address the exhaustiveness checking you mentioned and explain to the readers a more "Gleam" way of thinking instead of the "Elixir" way.


param usecols in read_csv is not valid

System information

OS Platform and Distribution: Linux Ubuntu 16.04
Modin installed from (source or binary):pip install modin
Modin version:0.5.0
Python version:3.7

Describe the problem

My raw data:
$ cat batch_instance1.csv
ins_74901673,task_LTg0MTUwNTA5Mjg4MDkwNjIzMA==,j_217,10,Terminated,673795,673797,m_2637,1,1,13,16,0.02,0.02
ins_815802872,M1,j_1527,1,Terminated,158478,158520,m_3430,1,1,3,19,0.13,0.18
ins_564677701,M1,j_2014,1,Terminated,372602,372616,m_1910,1,1,87,116,0.04,0.05
ins_257566161,M1,j_2014,1,Terminated,372602,372615,m_2485,1,1,91,123,0.05,0.05
ins_688679908,M1,j_2014,1,Terminated,372602,372615,m_993,1,1,93,141,0.05,0.05
ins_929638393,M1,j_2014,1,Terminated,372603,372615,m_2808,1,1,100,137,0.05,0.05
ins_1349024140,M1,j_2014,1,Terminated,372603,372617,m_3736,1,1,82,111,0.05,0.05
ins_330247444,M1,j_2014,1,Terminated,372603,372617,m_1176,1,1,84,110,0.05,0.05
ins_833551291,M1,j_2014,1,Terminated,372602,372614,m_2682,1,1,90,159,0.05,0.05
ins_833550789,M1,j_2014,1,Terminated,372603,372619,m_3625,1,1,78,105,0.05,0.05

Source code / logs

import modin.pandas as pd
First,  I read csv file with columns 0 and 1
df = pd.read_csv("/state/partition1/alibaba_clusterdata_v2018/batch_instance1.csv", header=None, usecols=[0,1])
df.head()

0 | 1
-- | --
ins_74901673 | task_LTg0MTUwNTA5Mjg4MDkwNjIzMA==
ins_815802872 | M1
ins_564677701 | M1
ins_257566161 | M1
ins_688679908 | M1

While, I read csv file with columns 0 and 7
df = pd.read_csv("batch_instance1.csv", header=None, usecols=[0,7])
df.head()

The output is the same!
0 | 7
-- | --
ins_74901673 | task_LTg0MTUwNTA5Mjg4MDkwNjIzMA==
ins_815802872 | M1
ins_564677701 | M1
ins_257566161 | M1
ins_688679908 | M1

Addition, I test with pandas API.
import pandas as pd
df = pd.read_csv("/state/partition1/alibaba_clusterdata_v2018/batch_instance1.csv", header=None, usecols=[0,7])

df.head()

And off course, the pandas API output is right.
0 | 7
-- | --
ins_74901673 | m_2637
ins_815802872 | m_3430
ins_564677701 | m_1910
ins_257566161 | m_2485
ins_688679908 | m_993

Hi @axiqia, thanks for posting this!
I am able to reproduce the issue, it is the combination of header=None and usecols that results in this behavior. If you use the parameter usecols without header=None the behavior is correct. It seems that header=None somehow is overriding the usecols indicies, I will have to take a look into that.
This is a pretty subtle bug, thanks for reporting!

Hi @axiqia, thanks again for bringing up the issue!
The issue should be resolved on master. You can install from master using pip install git+https://github.com/modin-project/modin


Is there a version for Dot Net Core?

Is there a version which can be used with Dot Net Core?

Possible duplicate of #94 and #98

Yes, #master currently works with .net standard 1.2 and up


[Feature Request] Heading Search -> Unfold Fold Headers

This FR was generated by a comment that is here
📎 https://github.com/tadashi-aikawa/obsidian-another-quick-switcher/issues/28#issuecomment-1120189569
@tadashi-aikawa !!!!
I have one more idea about this FR to share with everybody
It's very simple to explain -> If a header is FOLD, when we select a Header from the Modal, it will open it !
My Workflow
I have a bunch of notes that I call cheatsheet for a quick access of the documentation from every workflow that I have in my vault
🎥 A Example from my sources capture workflow

Each one of the headers are folded when I open the Note
My Idea is after selecting Video from the Modal, AQS will open the folded Header!
I don't know if is feasible but would be great :)

🆘 On 4.10.0 When We have a H3 inside a H2, it will open H2 when I select an H3
🎥 Take a look Open Chicago Bulls (H3) opens NBA (H2)

Thanks for reading this !
Have a great day 😀

It seems to be difficult..

It seems to be difficult..

In my user mind I think this would be a difficult one :) (especially on Reading Mode) :)
Feel absolutely free to close this FR whenever you want :)
Thanks for trying @tadashi-aikawa !!!!! 😀

If there were an API that tells the Fold/Unfold status, it would be easy, but I have not found such an API.
So I'll leave this issue 👍


fix(error-equals): update error equals schemas to have the correct syntax

Here is the syntax for declaring string[] in json schema: https://json-schema.org/understanding-json-schema/reference/array.html
The current definitions before this change would produce a schema that was an array of objects. { [key: string]: any }[] when used with the https://github.com/bcherny/json-schema-to-typescript package
The current schemas dont seem to be used in the repo anywhere, and definitely not in the tests, so this wouldnt have been caught here. However the downstream package asl-types does depend on these schemas, which is where i saw this issue.
I have validated in that package that changing items -> item did in fact fix the conversion process.
Here is the issue created in that package https://github.com/fourTheorem/asl-types/issues/7
And here are the docs for ErrorEquals https://docs.aws.amazon.com/step-functions/latest/dg/concepts-error-handling.html

Thank you for the very well detailed bug report and fix @Crazyglue 👌
I just added a test to check that we weren't validating item types on master, and released a v1.7.1


Feature/87-Volunteers Formation

Is your feature request related to a problem? Please describe.
Not applicable
Describe the solution you'd like
The screen of volunteers formation should be access once the form of the volunteers it is completed. This screen should show the 3 different videos that the applicant for volunteer should watch.
Also create the second form needed for the volunteers and check if the first one works.
Describe alternatives you've considered
Not applicable
Additional context
Any doubt with the design have a look on the mock ups.

Story Points: 8
See the discussion in Frontend S2
Powered by Parabol

Story Points: 8
See the discussion in Frontend S3
Powered by Parabol


typo fixes: make man more reliable

Signed-off-by: Aaron.L.Xu<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Can you squash and provide an accurate commit message?
Thanks!

@liker12134
Just a confirmation: are you @xulike666 ?
(I think it is OK to use multiple GitHub accounts, but just confused)

@AkihiroSuda, sorry for the confusion and it‘s my personal private account. 😄

ping @xulike666 (@liker12134)


Change the text font size and font color from navigation/menu

font-size: 16px;
font-weight: bold;
color: #7B573F

That isn't Cinzel Bold - that is a browsers simulating boldness because the Bold version wasn't provided

hmm, meaning we shouldn't really use bold throughout the site right?

the branding isn't properly defined - if you wish to use bold cinzel, then add it to the site and use the proper bold version of it, not the generic browser version

do you have that font?

https://fonts.google.com/specimen/Cinzel?query=cinz


[FEATURE] Mechanism to allow sub-accounts to request perimeter FW/ALB flow updates

Mechanism to allow sub-accounts to request perimeter FW/ALB flow updates

Most of this functionality is being delivered by functionality in the second half of #505 "Challenge: How do we route inbound traffic?".  If direct perimeter requests from sub-accounts are actual customer requirement, will deliver using future functionality with Service Catalog #503 and mechanism implemented to make #738 Self-Service VPC possible.  Many customers want to approve new ingress flows through an existing ticketing system anyway.


Add confirmation prompt when selecting the "clear list" button

The "Clear List" and "Add All to Cart" options are very close together. It would be easy to accidentally hit "Clear List" when you actually mean to "Add All to Cart". A good solution would be to add a prompt asking the user to confirm they would like to clear the list.
Perhaps with some fun added wording:
All done with your shopping and ready to clear your list?
<Yes, Clear the list!> <No, I want to keep my list.>
Add green to Yes option, and red to No option for visibility sake and good UI.
If you don't like this suggestion, please move the clear list option to the three dots or to another area of the screen away from the "Add All to Cart" button.

Looks like this was fixed in the latest update. Woot.

Accidentally fixed in 932646d65592b86e238bdc2382df2775264f90b5 :tada:


Define and create workflows for the branch and release policy

Things can be done

[x] Create protected branches, and define branch policy.
[x] Create binaries and attach the a draft release note.
[x] Create docker images and publish it to docker.io.
[ ] ...

Notes
Re release notes, please apply the same style as in https://github.com/superfluid-finance/protocol-monorepo/releases/tag/subgraph%40v1.1.0

@hellwolf this issue can be closed


Move SV loading to Luigi task

Google Drive link for the genome SV datasets.

The drive folder for both gCNV and SV is here.


Hide left arrow on load

I'm being nit-picky, but when I enable arrows, I'm seeing one on the left on initial load. It doesn't do anything, and I don't want it to. Wondering if it could just be hidden until the user clicks right one time.
Is this something I should implement on my end?

It is ok to be nit-picky - I have given this particular issue some thoughts in the process. It would be easy to remove using js, but I also want it all to be very simple. But I am sure I can come up with a lightweight solution to this.

It could also go to last slide when you slide left on first slide...?

This feature has been added.
You can add slider-nav-autohide to the swiffy-slider instance and that is enough. When the JS loads, it will detect which slides are visible and set slider-item-first-visible or slider-item-last-visible classes on the swiffy-slider instance.
If you know that the first item is visible on initial load (usually will be), always apply slider-item-first-visible to avoid CLS and flickering.

...

    <button type="button" class="slider-nav"></button>
    <button type="button" class="slider-nav slider-nav-next"></button>

See an example in the configurator:
https://swiffyslider.com/configuration/?slider-nav-visible=slider-nav-visible&slider-nav-autohide=slider-nav-autohide&slider-item-first-visible=slider-item-first-visible


Unable to set a property with false value

I have a problem with draggable property which might be true by default, e.g. on <img>.
Problem is with this line: https://github.com/developit/preact/blob/master/src/dom/index.js#L77, if I set property to false preact adds it and then immediately removed via removeAttribute. The workaround is to use 0, e.g. draggable={0} this way it isn't being caught by the problematic line and is converted to false to property assign by the browser.

Hmm, yeah that's tricky.  I am trying to remember why the property setter would care about invoking removeAttribute in the first place. Any guesses?

@developit maybe something related to <select>/<option> and selected attribute, it was always a mess what attr or prop takes priority in the browser, not sure. Also maybe for hydration, i.e. SSR?

Just encountered this. Seems like that behavior was added in https://github.com/developit/preact/commit/460099f3bd6b96dab8eeaba3b7ebd7fd80d66016.

I also had this issue. We should remember that JS has true, false and null, therefore explicitly falsy values shouldn't be removed as long as there is not specification on HTML that the absence of a property defaults it to false (and draggable is an example that defaults to true).

I think a potential solution to this problem would be checking if the current node's property value is the same as the value to be set before removing the attribute, I think this will help us avoid the pitfall if the attribute's default value is not false or null. This might add a few overheads especially for the SSR cases, but it will make the components less dependent to the browser's implementation of the DOM elements.
I can make a PR if this sounds like a plan.

Related: https://github.com/developit/preact/issues/58
Linking because that one comes up when searching for preact draggable false.

Just checked with our code in master and this is resolved :tada: Setting draggable to false works :+1:


High CPU load (Virtual Machine Worker Process + System) when session is IDLE

Version
Win: 10.0.22622.575  WSL: <IP_ADDRESS> Kernel: <IP_ADDRESS>
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
Install WSL2/Ubuntu and start a session. Obeserve in Windows Task manager high load on both "Virtual Machine Worker Process" and "System". Both taking 15% CPU each for a combined 30+% CPU load. After wsl --shutdown the issue is gone.
Expected Behavior
No significant CPU usage when Linux/WSL is IDLE.
Actual Behavior
This is on my Surface Pro X (aarch64) - other ARM machine users for instance with the new Thinkpad X13S have reported similar obervations.
Both "Virtual Machine Worker Process" and "System". Each taking  roughly 15% CPU for a combined 30% CPU load.
Linux System is idle:
wsl --system -d Ubuntu Top
PID USER      PR  NI    VIRT    RES  %CPU  %MEM     TIME+ S COMMAND
1 root      20   0    2.1m   1.5m   0.0   0.0   0:00.02 S /init
2 root      20   0    2.2m   1.6m   0.0   0.0   0:00.01 S  - /init    27 root      20   0    2.3m   0.1m   0.0   0.0   0:00.00 S      - /init
28 root      20   0    2.3m   0.1m   0.0   0.0   0:00.07 S          - /init    30 wslg      20   0    9.5m   4.4m   0.0   0.1   0:00.09 S              - -bash
7 root      20   0    2.1m   0.2m   0.0   0.0   0:00.00 S  - /init     8 root      20   0   45.9m   7.3m   0.0   0.1   0:00.01 S      - /usr/bin/WSLGd
11 wslg      20   0  674.9m  34.3m   0.0   0.4   0:00.55 S          - /usr/bin/weston --backend=rdp-backend.so --modules=wslgd-notify.so --xwa+    15 wslg      20   0   18.5m   7.2m   0.0   0.1   0:00.03 S              - /usr/libexec/weston-rdprail-shell
31 wslg      20   0   44.7m  17.7m   0.0   0.2   0:00.04 S              - /usr/bin/Xwayland :0 -rootless -core -listen 37 -wm 38 -terminate -n+    16 wslg      20   0    2.0m   1.5m   0.0   0.0   0:00.00 S          - /init /mnt/c/Program Files/WindowsApps/MicrosoftCorporationII.WindowsSub+
17 message+  20   0    7.8m   3.3m   0.0   0.0   0:00.02 S          - /usr/bin/dbus-daemon --syslog --nofork --nopidfile --system    18 wslg      20   0  230.1m   7.2m   0.0   0.1   0:00.06 S          - /usr/bin/pulseaudio --log-target=file:/mnt/wslg/pulseaudio.log --load=mo+
22 wslg      20   0    7.6m   0.3m   0.0   0.0   0:00.00 S  - /usr/bin/dbus-daemon --syslog --fork --print-pid 6 --print-address 8 --session    93 root      20   0    2.1m   0.1m   0.0   0.0   0:00.00 S  - /init
94 root      20   0    2.1m   0.1m   0.0   0.0   0:00.00 S      - /init    95 wslg      20   0    6.4m   2.1m   0.0   0.0   0:00.02 R          - top
Diagnostic Logs
WslLogs-2022-08-24_17-14-22.zip

Much appreciated 👍

I assume this has not been reviewed by WSLG developers yet? Let me stress, that this is a very serious issue, which makes WSL almost un-usable on many if not most ARM devices. Issue is, that, this constant 30+% load on the system affects responsiveness, battery drain and available processing resources - you essentially have to shutdown WSL when not in use.
Let me know if you need more logs - as I initially only uploaded the WSL logs.

Same here on a Windows on ARM - Qualcomm 8cx gen3 (Thinkpad X13s)
Wondered why runtime is so bad until I noticed the 30% CPU penalty.
Need to shutdown WSL if not absolutely needed.

@puzzone, @Gerdya, to narrow down the cause, would you please disable WSL GUI support by below, and check if you observe the issue? Please set below in .wslconfig, thanks!
[wsl2]
guiApplications=false

For further details, please refer https://docs.microsoft.com/en-us/windows/wsl/wsl-config.

Issue still present!

@dtgeorge, would you please collect WSL log by following instruction at 8) at https://github.com/Microsoft/WSL/blob/master/CONTRIBUTING.md ? thanks!

WslLogs-2022-09-12_23-15-31.zip

@dtgeorge, thanks for the log. your log doesn't exhibit the issue @benhillis noted at https://github.com/microsoft/wslg/issues/819#issuecomment-122775983, so next step would be collecting more detailed log, would you please do below?

extract log.wprp from log.zip
make sure WSL is idle and CPU high load issue is happening.
from run admin privileged cmd.exe, run wpr -start log.wprp!GPUView.Verbose -filemode
wait 3 sec, then wpr -stop output.etl
please share output.etl with us.

Thanks!

I did upload both, logs and output.etl (output.zip)
WslLogs-2022-09-12_23-47-21.zip
output.zip

@hideyukn88 , does @Gerdya logs enough?

Hi all,
please find my logs with wslg dis- and enabled below.
And big thanks for your help!
wslg-DISABLED.zip
wslg-ENABLED.zip

@hideyukn88 , does @Gerdya logs enough?

Yes, the output.etl is what we need, thanks!

Now we understood the issue and the fix is being made. The fix will be at Windows OS side (not WSL/WSLg), thus ETA for fix is still unknown, once I know more, will post here, thanks!

@hideyukn88 Thanks for root-causing the issue!

Looks like it is fixed in the dev channel build 25211
Preview Build 25211 Release Notes
Hopefully beta channel is next.

Is there deployment ETA for the normal Win 11 build? Still an issue on Pro X.

It is been a month since the dev channel fix. Nothing on the other channels as far :(

Seems to have been fixed!
Edition	Windows 11 Pro Version	22H2 OS build	22621.900 Serial number<PHONE_NUMBER>53 Experience	Windows Feature Experience Pack 1000.22638.1000.0

Idle CPU usage was around 1%-3% with Ubuntu 22.04 running along with a bunch of other apps (Firefox, etc.). Excellent!

Still experiencing this issue Edition	Windows 11 Pro
Version	22H2
Installed on	8/28/2022
OS build	22621.755
Experience	Windows Feature Experience Pack 1000.22636.1000.0

To add, I have a Surface Pro 9 ARM variant, Virtual Machine Worker Process is using 12.5% CPU and System using 12.5% while Debian on the WSL side is fully idle.

@hideyukn88 @gerdya To add, I have a Surface Pro 9 ARM variant, Virtual Machine Worker Process is using 12.5% CPU and System using 12.5% while Debian on the WSL side is fully idle.


Example parameters

Hi, I found your research quite interesting! Could you provide some example sets of parameters that you know to work well with these example datasets?

Hi, @bradasteiner @houssamzenati
python3 main.py gan mnist run --nb_epochs=100 --label=9 --w=0.5 --m='cross-e' --d=2 --rd=123
Here is example.

(Net_Trim) thomas@beast:~/virtualens/Efficient-GAN-Anomaly-Detection$ python3 main.py gan mnist run --nb_epochs=100 --label=9 --w=0.5 --m='cross-e' --d=2 --rd=123

|_     .|_   _|     / ___ .   .'    '.
| | . \ | |      |_/___) |  |  .--.  |   | |  | | | |   _   .'____.'  | |    | |  _| |_.' /_| |__/ | / /_____  _|  --'  |
|.'|__| |__|()'..'
[09:35:07 INFO @AnomalyDetection] Running script at gan.run_mnist
/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/h5py/init.py:36: FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type.
from ._conv import register_converters as _register_converters
[09:35:10 INFO @GAN.train.mnist.cross-e.9] Building training graph...
[09:35:10 WARNING @GAN.train.mnist.cross-e.9] The GAN is training with the following parameters:
Batch size:  100
Starting learning rate:  1e-05
EMA Decay:  0.999
Weight:  0.5
Method for discriminator:  cross-e
Degree for L norms:  2
Anomalous label:  9
[09:35:12 INFO @GAN.train.mnist.cross-e.9] Building testing graph...
[09:35:14 WARNING @tensorflow] From /home/thomas/virtualens/Efficient-GAN-Anomaly-Detection/gan/run_mnist.py:170: calling norm (from tensorflow.python.ops.linalg_ops) with keep_dims is deprecated and will be removed in a future version.
Instructions for updating:
keep_dims is deprecated, use keepdims instead
[09:35:14 WARNING @tensorflow] From /home/thomas/virtualens/Efficient-GAN-Anomaly-Detection/gan/run_mnist.py:211: Supervisor.init (from tensorflow.python.training.supervisor) is deprecated and will be removed in a future version.
Instructions for updating:
Please switch to tf.train.MonitoredTrainingSession
[09:35:15 INFO @GAN.train.mnist.cross-e.9] Start training...
2018-06-19 09:35:15.309253: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1356] Found device 0 with properties:
name: GeForce GTX TITAN X major: 5 minor: 2 memoryClockRate(GHz): 1.2155
pciBusID: 0000:0a:00.0
totalMemory: 11.92GiB freeMemory: 353.06MiB
2018-06-19 09:35:15.543079: E tensorflow/core/common_runtime/direct_session.cc:154] Internal: failed initializing StreamExecutor for CUDA device ordinal 1: Internal: failed call to cuDevicePrimaryCtxRetain: CUDA_ERROR_OUT_OF_MEMORY; total memory reported:<PHONE_NUMBER>6
[09:35:15 ERROR @AnomalyDetection] Failed to create session.
Traceback (most recent call last):
File "main.py", line 41, in run
mod.run(args.nb_epochs, args.w, args.m, args.d, args.label, args.rd)
File "/home/thomas/virtualens/Efficient-GAN-Anomaly-Detection/gan/run_mnist.py", line 329, in run
train_and_test(nb_epochs, weight, method, degree, random_seed, label)
File "/home/thomas/virtualens/Efficient-GAN-Anomaly-Detection/gan/run_mnist.py", line 214, in train_and_test
with sv.managed_session() as sess:
File "/home/thomas/.pyenv/versions/3.5.0/lib/python3.5/contextlib.py", line 59, in enter
return next(self.gen)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/training/supervisor.py", line 1000, in managed_session
self.stop(close_summary_writer=close_summary_writer)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/training/supervisor.py", line 828, in stop
ignore_live_threads=ignore_live_threads)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/training/coordinator.py", line 389, in join
six.reraise(*self._exc_info_to_raise)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/six.py", line 693, in reraise
raise value
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/training/supervisor.py", line 989, in managed_session
start_standard_services=start_standard_services)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/training/supervisor.py", line 726, in prepare_or_wait_for_session
init_feed_dict=self._init_feed_dict, init_fn=self._init_fn)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/training/session_manager.py", line 279, in prepare_session
config=config)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/training/session_manager.py", line 180, in _restore_checkpoint
sess = session.Session(self._target, graph=self._graph, config=config)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1569, in init
super(Session, self).init(target, graph, config=config)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 639, in init
self._session = tf_session.TF_NewDeprecatedSession(opts, status)
File "/home/thomas/.pyenv/versions/Net_Trim/lib/python3.5/site-packages/tensorflow/python/framework/errors_impl.py", line 516, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.InternalError: Failed to create session.
[09:35:15 ERROR @AnomalyDetection] Uhoh, the script halted with an error.


Use Django ORM query in Page._update_descendant_url_paths.

I'm in doubt about MSSQL, but for the other backends, it can be expressed in Django.

In the README-file, there's no word about MSSQL.

Wagtail supports Django 1.8.x, 1.10.x and 1.11.x, on Python 2.7, 3.4, 3.5 and 3.6. Supported database backends are PostgreSQL, MySQL and SQLite.

Thanks @atombrella! There's actually a very similar PR already in the queue for reviewing: #3496, currently blocked by our lack of testing environment for MSSQL. (We don't officially support MSSQL at the moment, but we're working towards doing so, and there are several existing sites running on that setup - so we need to be careful not to break those.)

An alternative could be to keep the proposed change, backed by the mssql check?

Addressed via #3496, with an exception added for MSSQL as you suggest. Thanks again!


Escape path in batch script

Summary
In the batch activator script, we need to escape double quotes with an additional double quote (https://stackoverflow.com/a/562080/3549270).
See #9424.
Test Plan
Unclear.

I failed to test this since double quotes are not allowed in file names (https://learn.microsoft.com/en-us/windows/win32/fileio/naming-a-file#naming-conventions)


Always include note labels in .scl output

frostburn — Today at 8:58 AM

Scala files are basically just lists of cents. If you ignore the headers it's pretty human-readable if you ask me.

zastari — Today at 8:58 AM

the main complaint for things like 159 edo is not having the step values commented in for you so you'd need to do something like turn on line numbers in a text editor

The solution is to add tuning table note labels after the cent lines in the file output.
Key colors can be listed in a comment after the main output.

Vynce — Today at 9:02 AM

then you could have a file with your cents, the note "name" and scale degree, and maybe even a few selected JI near-equivalencies ... locally.

frostburn — Today at 9:04 AM

That might get cluttered, but I'll think about offering the option to export fully annotated .scl.

Make .scl export a proper modal, basically.

So what do we need in this modal:

[checkbox] include interval expression (like 41\159 for an edo)
[checkbox] include the color
[checkbox] include scale degree
[checkbox] include closest JI intervals

[???] something about choosing them

Their order though might be an issue. Maybe something like a format string with variables would be acceptable? (Or it can be an advanced option which is filled in by tweaking simple options from above.)
Not assigning myself just yet because I have no clue what exactly is the target behavior (and look) (and yeah we can ignore JI approximations at first, at least I won’t touch them right away).

Are we doing this in the version 2 series where the interval labels are somewhat nebulous and annoying to work with? In SW3 I would add checkboxes for scale.labels, scale.colors and scale degrees. A proficient SonicWeaver could put the closest JI intervals into the label strings (or that could be a separate issue about a modification modal that just does helpful labeling stuff).

I was about targeting SW3. What do you think about ordering, can it be fixed (and which one, then) or how to allow changing it? Via a list with 6 entries.

I suggest 3 checkboxes for degrees, labels and colors in that order. It's a good start and an UX designer can always tweak the order later and devs can add more options in the future.

Most text editors can show line numbers so power users can either delete the boilerplate or learn how to program custom labels using SonicWeave. Closing as good enough.

Sorry I didn’t proceed myself 😶


Upgrading from 0.3.3

I can't find any documentation on upgrading Restic. Will my repos be backwards compatible? Is there anything I should know before making this change?

Hey, you can use restic directly, in regards to the structure of the repo, nothing has changed. Except when you're using a repo stored in s3, the directory names have changed, but the old ones still work just fine. You can use the new migrate command if you'd like to migrate to the new names. Otherwise, have a look at the release notes of each release from 0.3.3 on.
I'm going to close this issue as your question is answered, please feel free to leave a comment! Especially, let us know if something does not work :) thanks!


rgb.measure failing when px.radius = 1

I ran the tri.surf function where num.passes = 3 to eliminate the issue identfied when num.passes = 1.
When 3 rounds of triangulation were performed (successfully), I measured the values using rgb.measure. If px.radius = 1, it fails to measures RGB values for the pixels. Both the perimeter and interior points are NaN.
uncalib_RGB = rgb.measure(imagedir = "C:/Users/jennv/OneDrive/Mitch Error/unwarped_images/", image.names =  specimen.factors[,2], delaunay.map = specimen.sampling.template, px.radius = 0, linearize.color.space = FALSE)

uncalib_RGB_px1 = rgb.measure(imagedir = "C:/Users/jennv/OneDrive/Mitch Error/unwarped_images/", image.names =  specimen.factors[,2], delaunay.map = specimen.sampling.template, px.radius = 1, linearize.color.space = FALSE)

uncalib_RGB_px2 = rgb.measure(imagedir = "C:/Users/jennv/OneDrive/Mitch Error/unwarped_images/", image.names =  specimen.factors[,2], delaunay.map = specimen.sampling.template, px.radius = 2, linearize.color.space = FALSE)

This is the same issue that caused issue #7 , so I'm going to close it here and address it there. circle.coords(1) and circle.coords(0) produce the same output of (0,0), which will measure the exact pixel value of each triangulated point.


[PROPOSAL] Release 2.8.3

What/Why
What are you proposing?
In a few sentences, describe the feature and its core capabilities.
If possible, can we release 2.8.3 with #838 (#833 should be avoided since we are only using 2.x) it is a change that would enable a new feature for our customers.
Is that an alright process to go through? @dblock @VachaShah

Another request for 2.8.3 release: https://github.com/opensearch-project/opensearch-java/pull/841
Working on the release.

https://github.com/opensearch-project/opensearch-java/releases/tag/v2.8.3

2.8.3 is released.


UI: Move menu to position subtitles

The position menu should be placed below the video and subtitle area. User can either insert style information or position (region). In this menu he can select a region or add a new region. There are also two buttons to allow dragging and resizing with the mouse.

Fixed in release 0.4.0

resolved by d8bf384ef856d6a1e6ddce7b621e6f44a96e2fce


Origin/Faction/Role from existing characters vanished

Describe the bug
Origin/Faction/Role from existing characters vanished. The character I created yesterday with the generator does not have this problem
To Reproduce
Dont know if it is reproducable anymore. If you want an export write me on Discord
Screenshots

Version Numbers
ImpMal 2.0.1

Fixed via https://github.com/moo-man/WarhammerLibrary-FVTT/commit/571e4e949de1d0c1f08b487b97bd7b4385bdbfcf


pull request for e2e test at 11/12/2016 21:15:10

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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.


Teamspeak downloading doesn't work & Ability to reinstall servers if installation failed.

So I tried installing Teamspeak on the server, and it failed. I have no clue how to fix this :P. Here is the error code: A fatal error was encountered while attempting to create a server. { Error: (HTTP code 500) server error - json: cannot unmarshal number 65099999.99999999 into Go struct field HostConfig.Memory of type int64
Also I was wondering if you can add the ability to reinstall servers if it failed.
Cheers.

Please post the full error.

What value did you enter for the server memory?

60 mb. It shouldn't need much?

Fixed it. I changed a bit of the script to download the latest version, and removed the variable.


Issues with Search Results in Update FolderItems.ps1

Hello David,
Hope you are doing good.
I have below issues below.
When i run the below command
$items = .\Update-FolderItems.ps1 -Mailbox $Mailbox -FolderPath "WellKnownFolderName.MsgFolderRoot" -ProcessSubfolders -Office365 -OAuth -OAuthTenantId $tenantId -OAuthClientId $clientId -OAuthRedirectUri $redirectUrl -OAuthSecretKey $secretKey -SearchFilter "subject:Invoice"  -ListMatches
Issue 1 :
When i check $items | fl it only shows the subject of the emails and other fields are empty, like timedaterecieved, sender, recipient.
Issue 2:
When i am satisfied with the search results in -listmatches , how can pass the values of $items with -delete -harddelete without running the entire command which will perform the search again.
Thank you in Advance

Closing issue as handled outside Github.


Model channel objects

Codecov Report

Merging #1 (6fefaf6) into main (6d3611d) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              main        #1    +/-   ##
==========================================
  Coverage   100.00%   100.00%
==========================================
  Files            5        12     +7
  Lines          102       269   +167
  Branches        10        19     +9
==========================================
+ Hits           102       269   +167

Flag
Coverage Δ

unittests
100.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/picartoapi/model/channel.py
100.00% <100.00%> (ø)

src/picartoapi/model/channel_stub.py
100.00% <100.00%> (ø)

src/picartoapi/model/chat_settings.py
100.00% <100.00%> (ø)

src/picartoapi/model/description_panel.py
100.00% <100.00%> (ø)

src/picartoapi/model/language.py
100.00% <100.00%> (ø)

src/picartoapi/model/multistream.py
100.00% <100.00%> (ø)

src/picartoapi/model/thumbnails.py
100.00% <100.00%> (ø)

tests/model_builder_test.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d3611d...6fefaf6. Read the comment docs.


Add sample to create the client via dynamic proxy

Description
Add a sample on how to create the OpenSearchClient through DynamicProxy. This allows users to add custom logic around the client operations for cross-cutting concerns such as added logging, exception handling, or introducing delays and failures for testing purposes.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hey @HEskandari, I believe this example does not belong here. Any kind of proxying / class rewrite / instrumentation ...  is not the concern of OpenSearch Java Client but the framework or application that uses it (hopefully, for reasons). In this regard, the OpenSearch Java Client  APIs are no different than any other APIs out there.

@reta does it hurt to have it here? should it just be a SO or maybe someone's blog post? I'd be happy to have it on mine ;)

Sure @dblock , but do you really want that? There are few things to keep in mind: JDK releases easily break that (cglib has not been updated for the last 2 years, 3.3.0 was out in 2019). This is very shaky ground for the library whose business is not that ...

Yeah, I think I agree :(

Moved the example here for whoever is interested. Closing.

Thanks for contributing the sample @HEskandari !


[FAB-17103] Add upgrade network option to operator

Adding upgrade network option to operator. For example:
go run main.go -i <network-spec-file> -a upgradeNetwork
Above command will take down existing fabric network, upgrade
all peer db's, relaunch fabric network using 2.0 images, update
capabilities for orderer system channel and all application
channels, update policies for consortium on orderer system
channel, update policies for each organization, app policies,
acls on all application channels.
Signed-off-by: Surya Lanka<EMAIL_ADDRESS>

@bharadwajambati95


Update google-cloud-trace to 2.39.0

About this PR
📦 Updates com.google.cloud:google-cloud-trace from 2.3.7 to 2.39.0
Usage
✅ Please merge!
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.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-trace" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.google.cloud", artifactId = "google-cloud-trace" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #356.


Release sdk/resourcemanager/costmanagement/armcostmanagement v0.1.0

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run go - armcostmanagement - ci

/azp run prepare-pipelines

/azp run go - costmanagement - ci

/azp run go - armcostmanagement - ci

/check-enforcer override


Add image support for uikit feature backends

This builds off #46.
I wanted to add some simple tests so I added Image support for iOS/uikit and put a simple test in with cargo-dinghy. I'm trying to avoid too much of the objc for the future merge conflicts but some I'm sure will happen.
The image used in the test-data is the favicon from simlay.net because I didn't know what else to use.

This is awesome! At a glance I don't see anything I'm concerned with, but I'll likely take another look tomorrow and then look at merging.
If this also makes it so I don't need to deal with #46 further then I'll close that once this gets merged as well.

I've gone ahead and reviewed the objc2 PRs with the aim to try and get them merged, after which I think I'll merge this and the iOS fix (and cut a 0.4 most likely, just due to the objc2 changes).
That said, this could probably get merged sooner without too much hassle - just don't want to make too much extra work for @madsmtm existing PRs.
(Your work is, as always, appreciated greatly!)

Merged - thanks again for all of this!


Silent flux failure while using to(), "internal error not returned to client"

I am attempting to side load ~2GB worth of backed up data from a temporary bucket to an existing bucket in my InfluxDB instance. To do this I am running the following flux query:
      from(bucket: "tmp")
      |> range(start: -1y)
      |> filter(fn: (r) => r._measurement !~ /event_logs/)
      |> to(bucket: "dest")
      |> map(fn: (r) => ({"_value": 1, "_time": r._time, "_measurement": r._measurement}))
      |> group(columns: ["_measurement"])
      |> count()

While running this query it sometimes appears the influx query CLI command hangs. During this I can see my instances memory usage and CPU usage drop (as if the query stopped, the influx instance usually maxes a single core and uses >8Gi of memory during this time) and I find this entry in the logs:
ts=2023-01-20T16:44:26.130913Z lvl=warn msg="internal error not returned to client" log_id=0fTme62G000
handler=error_logger error="context canceled"

Any ideas on how to make this restore process more reliable (and possibly faster)?
Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

influx backup ~2GiB worth of data from a single bucket
influx restore into a temp bucket
Attempt to use Flux to sideload points into an existing bucket with data in it.

Expected behavior:
Sideloading to complete.
Actual behavior:
Indefinite hang.
Environment info:

System info: Linux 5.4.0-1098-azure x86_64 - Azure AKS, Standard_E4as_v4 node
InfluxDB version: InfluxDB v2.6.0 (git: 24a2b621ea) build_date: 2022-12-15T18:47:00Z
Other relevant environment details: N/A

Config:
{
        "assets-path": "",
        "bolt-path": "/var/lib/influxdb2/influxd.bolt",
        "e2e-testing": false,
        "engine-path": "/var/lib/influxdb2",
        "feature-flags": null,
        "flux-log-enabled": true,
        "hardening-enabled": false,
        "http-bind-address": ":8086",
        "http-idle-timeout":<PHONE_NUMBER>0000,
        "http-read-header-timeout":<PHONE_NUMBER>0,
        "http-read-timeout": 0,
        "http-write-timeout": 0,
        "influxql-max-select-buckets": 0,
        "influxql-max-select-point": 0,
        "influxql-max-select-series": 0,
        "instance-id": "",
        "log-level": "info",
        "metrics-disabled": false,
        "nats-max-payload-bytes": 0,
        "nats-port": 4222,
        "no-tasks": false,
        "pprof-disabled": false,
        "query-concurrency": 1024,
        "query-initial-memory-bytes": 0,
        "query-max-memory-bytes": 0,
        "query-memory-bytes": 0,
        "query-queue-size": 1024,
        "reporting-disabled": false,
        "secret-store": "bolt",
        "session-length": 1440,
        "session-renew-disabled": false,
        "sqlite-path": "/var/lib/influxdb2/influxd.sqlite",
        "storage-cache-max-memory-size":<PHONE_NUMBER>,
        "storage-cache-snapshot-memory-size": 26214400,
        "storage-cache-snapshot-write-cold-duration": "10m0s",
        "storage-compact-full-write-cold-duration": "4h0m0s",
        "storage-compact-throughput-burst": 50331648,
        "storage-max-concurrent-compactions": 0,
        "storage-max-index-log-file-size": 1048576,
        "storage-no-validate-field-size": false,
        "storage-retention-check-interval": "30m0s",
        "storage-series-file-max-concurrent-snapshot-compactions": 0,
        "storage-series-id-set-cache-size": 0,
        "storage-shard-precreator-advance-period": "30m0s",
        "storage-shard-precreator-check-interval": "10m0s",
        "storage-tsm-use-madv-willneed": false,
        "storage-validate-keys": false,
        "storage-wal-fsync-delay": "0s",
        "storage-wal-max-concurrent-writes": 0,
        "storage-wal-max-write-delay":<PHONE_NUMBER>00,
        "storage-write-timeout":<PHONE_NUMBER>0,
        "store": "disk",
        "testing-always-allow-setup": false,
        "tls-cert": "",
        "tls-key": "",
        "tls-min-version": "1.2",
        "tls-strict-ciphers": false,
        "tracing-type": "",
        "ui-disabled": false,
        "vault-addr": "",
        "vault-cacert": "",
        "vault-capath": "",
        "vault-client-cert": "",
        "vault-client-key": "",
        "vault-client-timeout": 0,
        "vault-max-retries": 0,
        "vault-skip-verify": false,
        "vault-tls-server-name": "",
        "vault-token": ""
}

Do you ever see any data show up in the new bucket or does all the data from tmp stay missing?
A workaround you could try is to use the stop parameter to the range and write multiple smaller batches.

Yep, using the stop parameter to create smaller chunks in sequence has been our workaround.
The bigger problem for us is that the HTTP response is never sent and the client appears to hang, so our scripts spin forever before timing out. Do you have any methods of debugging why that might be the case?

1+ year old issue. any insights? I'm getting this same error when doing 10KB data loads from Telegraf.


feat: add cognito_refresh_callback and underlying plugin methods

Addresses:

https://github.com/mblackgeo/flask-cognito-lib/issues/38

Depends on:

https://github.com/mblackgeo/flask-cognito-lib/pull/41

ToDo:

[x] Test coverage
[x] Docs update

@mblackgeo This one is ready for review

Had to make some adjustments in https://github.com/mblackgeo/flask-cognito-lib/pull/39/commits/1e36de647c490c2949301739c23009946e3eea31

Noticed a mistake, as refresh flow does not provide new refresh_token each time. Cleaned that up.
That lead to necessity  of having separate AWS_COGNITO_REFRESH_COOKIE_AGE_SECONDS

My testing on the real app went good. I was able to log it, refresh access token multiple times, log out

Guys thanks so much for this, it's been something I've been considering for a while. I'd like to fully understand how to implement it, am I correct in thinking that a refresh will not be called automatically and the cognito_refresh_callback needs to be manually called, or is it done automatically - somewhere perhaps I missed in a brief review?
My initial thoughts were to maybe create a custom decorator similar to the current @auth_required() which if the access is expired then attempts a refresh thus (or have I got the wrong end of the stick and there's a simpler approach?):
def my_auth_required(groups: Optional[Iterable[str]] = None, any_group: bool = False):
    def wrapper(fn):
        @wraps(fn)
        def decorator(*args, **kwargs):
            with app.app_context():
                if cfg.disabled:
                    return fn(*args, **kwargs)

                access_token = request.cookies.get(cfg.COOKIE_NAME)
                refresh_token = request.cookies.get(cfg.COOKIE_NAME_REFRESH)
                valid = False

                try:
                    # Attempt to verify the access token
                    claims = cognito_auth.verify_access_token(
                        token=access_token,
                        leeway=cfg.cognito_expiration_leeway,
                    )
                    valid = True
                except (TokenVerifyError, KeyError):
                    # Access token is invalid, try to refresh it
                    if refresh_token:
                        try:
                            new_tokens = cognito_auth.exchange_refresh_token(refresh_token)
                            access_token = new_tokens['access_token']  # Assume new access token is obtained
                            # Verify the new access token
                            claims = cognito_auth.verify_access_token(
                                token=access_token,
                                leeway=cfg.cognito_expiration_leeway,
                            )
                            # Update the cookies with new tokens
                            request.cookies.set('access_token', access_token)
                            if 'refresh_token' in new_tokens:
                                request.cookies.set('refresh_token', new_tokens['refresh_token'])
                            valid = True
                        except Exception as e:
                            valid = False

                # Check for required group membership if token is valid
                if valid and groups:
                    if any_group:
                        valid = any(g in claims.get("cognito:groups", []) for g in groups)
                    else:
                        valid = all(g in claims.get("cognito:groups", []) for g in groups)

                    if not valid:
                        raise CognitoGroupRequiredError

                if not valid:
                    raise AuthorisationRequiredError

                return fn(*args, **kwargs)

        return decorator

    return wrapper

Hi @planestoner,

I'd like to fully understand how to implement it, am I correct in thinking that a refresh will not be called automatically and the cognito_refresh_callback needs to be manually called, or is it done automatically - somewhere perhaps I missed in a brief review?

Any automation would be inappropriate as library then would dictate how the application should behave. As I mentioned in https://github.com/mblackgeo/flask-cognito-lib/pull/39#discussion_r1567221835

I have a React front end, that ~5 min before expiration shows user a modal with the button to prolong the session.
If user clicks it - refresh flow happens.
If not - user will be kicked-off.

My initial thoughts were to maybe create a custom decorator similar to the current @auth_required() which if the access is expired then attempts a refresh thus (or have I got the wrong end of the stick and there's a simpler approach?):

It's up-to-you. I just want to warn you that request.cookies.set('access_token', access_token) is not secure, see https://github.com/mblackgeo/flask-cognito-lib/pull/39/files#diff-5123dc7c7226ad96083297a735e20f71e19ad4b103579db333801451a9f9b6faR60-R82


pkg issue in running with native addons on Linux

I am using google-gax module and it has grpc_node.node native addon.
I have placed this addon at the root level with my executable.
On windows when I run my executable everything works fine.
But when I run the linux build on Linux (x86_64) it gives me following error:
Failed to load gRPC binary module because it was not installed for the current system\nExpected directory: node-v57-linux-x64-glibc\nFound: [node-v57-win32-x64-unknown]\nThis problem can often be fixed by running  \"npm rebuild\" on the current system\nOriginal error: /home/ec2-user/project/demo/grpc_node.node: invalid ELF header",   "stack": "Error: Failed to load gRPC binary module because it was not installed for the current system\nExpected directory: node-v57-linux-x64-glibc\nFound: [node-v57-win32-x64-unknown]\n   This problem can often be fixed by running \"npm rebuild\" on the current system\nOriginal error: /home/ec2-user/project/demo/grpc_node.node: invalid ELF header\n       at Object.<anonymous> (/snapshot/project/node_modules/grpc/src/grpc_extension.js:53:17)\n ",
I tried doing "npm rebuild" but it did not solve the issue.
I ran following command:
pkg  . -t node8-linux -d
Thanks for this amazing module.

I am using google-gax module and it has grpc_node.node native addon.
I have placed this addon at the root level with my executable.
On windows when I run my executable everything works fine.
But when I run the linux build on Linux (x86_64) it gives me following error:
Failed to load gRPC binary module because it was not installed for the current system\nExpected directory: node-v57-linux-x64-glibc\nFound: [node-v57-win32-x64-unknown]\nThis problem can often be fixed by running \"npm rebuild\" on the current system\nOriginal error: /home/ec2-user/project/demo/grpc_node.node: invalid ELF header", "stack": "Error: Failed to load gRPC binary module because it was not installed for the current system\nExpected directory: node-v57-linux-x64-glibc\nFound: [node-v57-win32-x64-unknown]\n This problem can often be fixed by running \"npm rebuild\" on the current system\nOriginal error: /home/ec2-user/project/demo/grpc_node.node: invalid ELF header\n at Object.<anonymous> (/snapshot/project/node_modules/grpc/src/grpc_extension.js:53:17)\n ",
I tried doing "npm rebuild" but it did not solve the issue.
I ran following command:
pkg . -t node8-linux -d
Thanks for this amazing module.

grpc_node.node is different for different platforms. Fixed by creating one for linux.
But file names are same between all platform i.e. for Linux it is "node-v57-linux-x64-glibc/grpc_node.node" and for windows "node-v57-win32-x64-unknown/grpc_node.node".
How to place these native addons inside different folder at root level with executables so that based on the current platform it picks up the valid one?


No module named 'nmap'

I'm running Kali Linux 2018.1 on Virtual Box 6.1.6
Here is error code:
root@kali:~/socialfish# python3 SocialFish.py
Traceback (most recent call last):
File "SocialFish.py", line 6, in
from core.scansf import nScan
File "/root/socialfish/core/scansf.py", line 1, in
import nmap
ModuleNotFoundError: No module named 'nmap'
tell me if you need anymore info.
please help if there is any way, thanks.

I'm running Kali Linux 2018.1 on Virtual Box 6.1.6
Here is error code:
root@kali:~/socialfish# python3 SocialFish.py
Traceback (most recent call last):
File "SocialFish.py", line 6, in
from core.scansf import nScan
File "/root/socialfish/core/scansf.py", line 1, in
import nmap
ModuleNotFoundError: No module named 'nmap'
tell me if you need anymore info.
please help if there is any way, thanks.

This is a common error for anyone using Oracle VM VirtualBox, I don't personally know if you can fix it.

Make sure to install nmap:
sudo apt-get update && sudo apt-get install nmap
And all the Python modules:
sudo -H pip3 install -r requirements.txt and it should work, I loaded my VirtualBox just to test this for you and it works

Make sure to install nmap:
sudo apt-get update && sudo apt-get install nmap
And all the Python modules:
sudo -H pip3 install -r requirements.txt and it should work, I loaded my VirtualBox just to test this for you and it works

im having the same issue and i have checked already to see if it is installed or not and it clearly is. Any solutions?

Edit requirements.txt and change python-nmap  to python3-nmap

sudo -H pip3 install -r requirements.txt

Thanks, it worked for me


Adds commit log guidelines

I've noticed myself repeating instructions around the change log. This places guidance in the CONTRIBUTING section, as that's slightly more relevant than HACKING as it already includes policy such as license and code style.

@marcingrzejszczak fyi I'm logging this here as I think formally I was telling people individually.

NetflixOSS » feign » feign-pull-requests #238 SUCCESS
This pull request looks good


Secure Workspace public access

Customer wants to secure public access to his workspace
He needs to edit Firewall, App Gateway and guacamole app service configuration.
Following best practice to make changes to the solution, this implies to create 3 new forked templates that he has to manage.
IF TRE was making incoming traffic to App Gateway and Guacamole go through Azure Firewall, this would allow customers more flexibility and the ability to make the platform completely secure with changes only on the firewall level.

We should consider exposing Apache Guacamole workspace services using private endpoints via Firewall or App GW (as it has a cert etc).
We can't do this with things like Azure ML as we need to use the service DNS names.
@chboudry has conditional access IP restrictions for AAD on the workspace been considered?
I will rename this item as think it's specifically for Apache Guacamole?

Yes, that would be exclusive to Guacamole.
I'm not sure to follow the rest, the scenario we focus on is network security on who can access guacamole:
I don't think IP restrictions for AAD is matching the scenario.
Also, I don't understand the DNS note, there is not issue using a public DNS name and forbid public traffic at the same time.